Support CSS env function

This is very new and not supported in browsers as of now.
Specification for this can be found here - https://drafts.csswg.org/css-env-1/
Though PostCSS plugins have started supporting this function. Check postcss-env-function.
What I need to do is, support JS files, that will contain few variable exports, which should be added to cssvar intellisense.

For what it's worth I don't think this is really in the scope of css variable autocomplete, at least not until the spec has matured and is on path to be supported by browsers.

I am not planning to support it anytime soon 😅 .
I have just kept this issue open for my reference.
My first priority is to support https://github.com/willofindie/vscode-cssvar/issues/1, which requires a complete re-write of this plugin, and am not getting enough time to work on the same.
Anyways, thanks for your support @madeleineostoja 👍🏾

Deferring this task.


How to deal with InvalidToken?

I have users with invalidTokens, and I need to prevent some code if this happen.
So, how can I deal with it?
fail = FbGraph::User.me('invalid_token').fetch
if fail == "InvalidToken"
or something else... is it possible?
Thank you

rescue FbGraph::Unauthorized exception.

Can you be more specific, with an example? Thank you

You should read more about exceptions.
begin
    FbGraph::User.me('invalid_token').fetch
    # this code won't be executed if FbGraph raises an exception
    # code...
rescue FbGraph::Unauthorized => e
    redirect_to root_url, notice: e.message
end

thank you

thanks @niuage :)


MF-846 - Install python in docker build for aedes mqtt image

Signed-off-by: Ivan Milošević<EMAIL_ADDRESS>What does this do?
Include python in multi-stage docker build for aedes mqtt image
Which issue(s) does this PR fix/relate to?
Resolves #846

Codecov Report

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

@@           Coverage Diff           @@
##           master     #860   +/-   ##
=======================================
  Coverage   85.57%   85.57%
=======================================
  Files          72       72
  Lines        4708     4708
=======================================
  Hits         4029     4029
  Misses        443      443
  Partials      236      236

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 5ef6aa8...62351a4. Read the comment docs.


ERR_STREAM_WRITE_AFTER_END when service is unavailable

When the gateway routes to a route that is unavailable, the gateway crashes and return ERR_STREAM_WRITE_AFTER_END
Steps to reproduce the behavior:

Run the gateway normally with a route that doesn't exists like localhost:8081
Try to access the route calling it through the gateway, for example localhost:8081/v1/test/aa
See error

Expected behavior
The gateway should return 'Service unavailable' error
Returned behavior
The gateway return 'Service unavailable' error but the gateway crashes with stack:
events.js:353
      throw er; // Unhandled 'error' event
      ^

Error [ERR_STREAM_WRITE_AFTER_END]: write after end
    at writeAfterEnd (_http_outgoing.js:694:15)
    at ServerResponse.end (_http_outgoing.js:815:7)
    at node_modules/fast-proxy-lite/index.js:95:19
    at node_modules/fast-proxy-lite/lib/request.js:61:18
    at node_modules/pump/index.js:75:7
    at f (node_modules/once/once.js:25:25)
    at ClientRequest.<anonymous> (/node_modules/pump/index.js:31:21)
    at ClientRequest.f (node_modules/once/once.js:25:25)
    at ClientRequest.onerror (node_modules/end-of-stream/index.js:44:12)
    at ClientRequest.emit (events.js:388:22)
Emitted 'error' event on ServerResponse instance at:
    at writeAfterEndNT (_http_outgoing.js:753:7)
    at processTicksAndRejections (internal/process/task_queues.js:83:21) {
  code: 'ERR_STREAM_WRITE_AFTER_END'
}

Additional context
The main.ts file:
/**
 * The main class
 */
import * as gateway from 'fast-gateway'
const PORT = 8080

const server = gateway({
  routes: [
    {
      prefix: '/v1/test',
      target: 'http://localhost:8081'
    }
  ]
}).start(PORT).then(() => {
  console.log(`API Gateway listening on ${PORT} port!`)
})

package.json file:
{
  "name": "test",
  "version": "1.0.0",
  "description": "test",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "build-ts": "tsc",
    "build": "npm run build-ts",
    "start": "node dist/main.js"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "fast-gateway": "^3.0.0"
  },
  "devDependencies": {
    "@types/express": "^4.17.12"
  }
}

tsconfig.json file:
{
  "compilerOptions": {
    "outDir": "dist",
    "target": "es2017",
    "module": "commonjs",
    "sourceMap": true
  }
}

Hi @Gemerich, thanks for reporting. Can you please also describe the version of Node.js you are using and the Operating System?
I can't reproduce your issue as you describe it.
Thanks

Sorry for the delay! I got covid last week and was away from work.
My node version is 14.17.0 and I'm running on a MacOs BigSur 11.6

@jkyberneees I was able to replicate provided downstream exists and it crashes during the response, hence upstream to gateway leads to above error on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>IMHO it's probably because end is being called multiple times in<EMAIL_ADDRESS>I assume it's also present in fast-proxy-lite.
// index.js line 90
       if (!res.sent) {
            if (err.code === 'ECONNREFUSED' || err.code === 'ERR_HTTP2_STREAM_CANCEL') {
              res.statusCode = 503
              res.end('Service Unavailable')
            } else if (err.code === 'ECONNRESET' || err.code === 'UND_ERR_HEADERS_TIMEOUT' || err.code === 'UND_ERR_BODY_TIMEOUT') {
              res.statusCode = 504
              res.end(err.message)
            } else {
              res.statusCode = 500
              res.end(err.message)
            }
          }

@Gemerich Hope you're feeling better :)

Hi @iapain, very much appreciated. I will reproduce and push the fix ASAP.
@Gemerich Hope you're feeling better :) +1

Hey @iapain, @Gemerich, I have to admit I am still not able to reproduce the issue. Could you please detail the steps so I can reproduce?
Thanks in advance.

I think the catch here is to call a route that doens't exists. On the example I used the route is /v1/test and you should try to call /v1/test/aa. Your gateway returns Service Unavailble normally?

@Gemerich @jkyberneees TBH this happens randomly. Try bombarding gateway with multiple concurrent connections using bench marking tool.

Hi @Gemerich and @iapain, thanks for bringing more details into this issue. I have released a minor patch in this direction inside fast-proxy-lite.
Can you please try out v3.0.2 of fast-gateway? https://github.com/BackendStack21/fast-gateway/releases/tag/v3.0.2
I am still unable to repro this issue in Node v16.13.1
Curious to see if you are still able to reproduce this issue.
Regards

Hi @jkyberneees
Many thanks! It's fixed in v3.0.2 :)
Cheers!

Thanks @iapain , @Gemerich can you confirm this issue is now resolved on your side as well?


Spotify's blend playlists seem to throw 404s

Fetching tracks and metadata from Spotify's new blend playlists seem to confuse rspotify (we might want to add a ticket upstream too).

Same for Your Top Songs ... playlists, apparently, see also e1d771d (might be worth investigating in detail).


The State of Data Engineering 2023 / Einat Orr @ lakeFS blog

URL: https://lakefs.io/blog/the-state-of-data-engineering-2023/#h-observability
Review (1-2 sentences): It's time to reveal new State of Data Engineering with lakeFS!

Published in Digest #33.


Change IO#write, IO#skip, IO.copy to return Int64

This PR

Updates the unreleased changes in #9233 to return Int64 instead of UInt64.
Changes IO.copy to return Int64 for consistency
Updates IO::Stapled#write_byte, String::Builder#write_byte which were missing in #9233.

The dilema between using UInt64 or Int64 was due to:

OS / libc uses unsigned int under the hood in some places.

Odly, I was sure LibC.write returned unsigned, but I was wrong 🤷♂. I know we are not using the turned value directly in IO#write, but they are related.

Keeping consistency with existing IO.copy

But since mixing signed and unsigned types will impose some performance issues, we can avoid them if we discourage unsigned int in some cases like this one.

Heads up: it seems that this change broke sth on nightlies - see https://travis-ci.com/github/Sija/blurhash.cr/jobs/341785188

@Sija not by this PR. https://travis-ci.com/github/Sija/blurhash.cr/jobs/340619464 is also broken in nightlies. Some of the dependencies is still not up to date with IO breaking changes.


need some pointers please

Ok so i got my RP2040 watch from ali everything worked fine out of the box/bag.
But after trying to flash the usbvalve image to it or any uf2 image for that matter the usb device popped up and i dragged and dropped the uf2 on the watch and nothing happened!
So i tried numerous other uf2 files to no avail. now i tried using Arduino IDE to upload the raw data as a sketch and after hours of work i got it to finally upload to the watch i was so happy!
until i saw a black screen and nothing happening on the watch itself i unplugged it and plugged it back in now it identifies itself as a usb drive named MyDrive with two files on it AUTORUN.INF and README.TXT "that says ...nuke the entire site from orbit. It's the only way to be sure."
I really need some pointers if possible? i would love to use the watch as an USBValve.
But why cant i for the love of god not get the UF2 file to flash the device?
I have tried it with a regular pico and rp2040 with all kinds of uf2 images and that works fine!
A few days ago i used this very same setup to flash a PICO with Jrunner to flash my xbox 360 which went very well.
But somehow this device won't work properly.
p.s. using windows 11 also tried using my steamdeck so linux same result the device won,t eject itself after copying UF2 file.

Hey.
If you managed to get the "Mydrive" mounted, this means that the firmware has been uploaded correctly. Also the file you see are expected and are part of the filesystem itself.
If, despite this, you don't see anything on the screen, there are two possibilities:

defective device (unlikely, did you see it working with the original firmware? Or can you upload the original firmware to test?)
the device is not fully compatible with the one I used to create the firmware

Unfortunately there are a lot of different way to wire up LCD to the Pi Pico and if the one you bought is not the same as the one I used to create the firmware, it requires some modification to work.
The best thing here is to obtain the schematics of the connection of your PI watch if available, so we can check the PINs used to connect the LCD and see why it is not working.

Hey.
If you managed to get the "Mydrive" mounted, this means that the firmware has been uploaded correctly. Also the file you see are expected and are part of the filesystem itself. If, despite this, you don't see anything on the screen, there are two possibilities:

defective device (unlikely, did you see it working with the original firmware? Or can you upload the original firmware to test?)
the device is not fully compatible with the one I used to create the firmware

Unfortunately there are a lot of different way to wire up LCD to the Pi Pico and if the one you bought is not the same as the one I used to create the firmware, it requires some modification to work.
The best thing here is to obtain the schematics of the connection of your PI watch if available, so we can check the PINs used to connect the LCD and see why it is not working.

Thanks for the quick reply i didn't expect there would be so much different possibilities for wiring a lcd to a pi pico.
I have added a picture of the labeled pins maybe this could help?

I also do not think its a defective device because i worked when i got it in but it would just not accept any UF2 files so i could not flash anything.
today i will try and to upload the original firmware if i can find it somewhere and hopefully i can use arduino ide to upload it i have no idea how to decompile a UF2 to its raw format or if this is even possible?

Model looks the same as mine.
If you uploaded the firmware through Arduino IDE, you probably re-compiled it. Did you un-commented the following
// Uncomment the following to compile for the RP2040 based TFT round display
// https://www.raspberrypi.com/news/how-to-build-your-own-raspberry-pi-watch/
//#define PIWATCH

Otherwise you are uploading the wrong version. Anyway, the easiest way is to upload the UF2.

Model looks the same as mine.
If you uploaded the firmware through Arduino IDE, you probably re-compiled it. Did you un-commented the following
// Uncomment the following to compile for the RP2040 based TFT round display
// https://www.raspberrypi.com/news/how-to-build-your-own-raspberry-pi-watch/
//#define PIWATCH

Otherwise you are uploading the wrong version. Anyway, the easiest way is to upload the UF2.

If you just un-comment that line, you should not get any errror. Start from the clean source and just un-comment line 24.
Then recompile (assuming you are using Arduino IDE with the proper libraries listed in the repo)

Copied the code from here
https://github.com/cecio/USBvalve/blob/main/USBvalve/USBvalve.ino
And pasted it in Arduino IDE uncommented the piwatch part
still the same error.
could it be that you say to uncomment "//#define PIWATCH"  but in your own code it says "#if defined(PIWATCH)"

you are referring to line 31
Look at line 24. Commens in C are not # but //
So your line 24 should be like this to work:
#define PIWATCH

Damn that did the trick! thanks very much and sorry i think i just can't count :(
Still does not fix the flashing issue but that a whole other problem i have a screen right now!

Great! Happy it worked! :-)
Thanks!


User having a lag issue

log files available #225286

I see nothing about performance problems in the case linked @grethelgahler.
Please clarify.


Do not highlight with multiple words selected

The plugin currently simply selects the first word. However, when I am selecting text, this probably means that I am doing something where highlights do not matter that much anyway, and the extra highlight only adds noise.
Alternatively, the plugin could select both/all words there a selection is, and highlight all occurrences of those two words occurring together, which might also be useful.

The new version I'm about to push actually treats a selection as a different highlighting mode. When a selection is active, the text inside the selection will be used as the regex. In this selection mode, there is no word boundary logic, it's a basic string search. This will allow for your sentence use case, as long as you select the full sentence. You could even select the entire paragraph, if you wanted. It won't work across line boundaries though.

Addressed in 0.0.5

nice idea. two suggestions for this one:

a setting to choose whether in this string-highlight mode the ignore-word-list should still apply or not
by default, the selected string shouldn't receive any styling, since it is a bit redundant with, well, background color a selection already gets?

The selected string styling is also an indicator of whether or not there were any matches found. If your selection is underlined, that means a match was found. You good with not having that?
I assume you'd still want the data attribute containing the text of the selection?

Can you describe your use case for the string select ignore list setting?

The selected string styling is also an indicator of whether or not there were any matches found. If your selection is underlined, that means a match was found. You good with not having that?

Good point. However, a selection always getting additional styling, can also be quite a bit distracting though. not totally sure how to solve this one 🤔

I assume you'd still want the data attribute containing the text of the selection?

sure!

Can you describe your use case for the string select ignore list setting?

Well, by default, the selected-string-highlight should of course ignore the words in the ignore list. However, as opposed to simply having the cursor somewhere, the selection of text is more intentional action by the user. This means that maybe, the user actually does want to search for all occurrences of the word "the" in their text. a setting would allow this kind of flexibility.
(However, I think this is indeed a quite niche use case, so simply having the ignore list always apply would also be fine, I think.)

Addressed in 0.0.6

everything works great!


1.8.2 changelog (#7079)

Backport to master

Codecov Report

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

@@           Coverage Diff            @@
##             master   #7088   +/-   ##
========================================
  Coverage          ?   41.6%
========================================
  Files             ?     443
  Lines             ?   60137
  Branches          ?       0
========================================
  Hits              ?   25021
  Misses            ?   31842
  Partials          ?    3274

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 1831b3b...553be6b. Read the comment docs.


The html contains some ES6 Template String will throw errors.

RT,
When the template html contains some scripts and it contains some Template String like some prefix ${varFoo} some suffix will throw error varFoo not defined.

RT,
When the template html contains some scripts and it contains some Template String like some prefix ${varFoo} some suffix will throw error varFoo not defined.

This this module https://github.com/glebfeklistov/webpack-html-template, may be solved you problem.

thx ~~


remove empty filters recursively

Fixes #667

This should be done in the hideFilter method of the List component


What makes the gan upsampler so effective

I find the results of the gan upsampler to be quite impressive when compared to SD upscaler and real-esrgan. However, in my personal experience, training a SR model with paired lr-hr data in a supervised manner can sometimes make it challenging to generate realistic textures on the hr image, especially when compared to pure generative models. This often results in achieving real-esrgan like outcomes. What, in your opinion, is the main concept that makes the gan upsampler so effective?

@salvadog i guess we'll slowly disentangle what is going on after this is open sourced

@salvadog we can leave this issue open and maybe someone who has been following GAN literature can chime in

@lucidrains Certainly, thank you! In my current work focused on image super resolution or restoration, the top priority is to preserve the faithfulness of the input data rather than solely pursuing visually pleasing outcomes. Although cGAN-based techniques are widely used, their capabilities in terms of generating high-quality images are still somewhat restricted. Considering that diffusion-based methods inevitably modify the input data to some extent, it would be highly advantageous to introduce an upsampler similar to gigagan that can mitigate this issue.

@salvadog yea, i'll get the upsampler finished by mid this month

"The power of GANs in super-resolution lies in their ability to navigate the complex and nuanced landscape of high-resolution images, weaving together the delicate threads of detail and structure to create a tapestry of realistic and stunning resolution."

"The power of GANs in super-resolution lies in their ability to navigate the complex and nuanced landscape of high-resolution images, weaving together the delicate threads of detail and structure to create a tapestry of realistic and stunning resolution."

Interesting. Where does this sentence come from?

@salvadog imo the multi-scale discrimination contributes the most, it somehow teaches the model how to do super resolution. And i feel it provides more supervison than "progressive growing", because the model would always receive gradients from all scales.

Hello, am I not totally lost in here thinking this "multi-scale discriminator" result in the model that could perhaps be considered "scale equivariant"?
I mean sure, in the SG3 it's actually the generator that is equivariant (rotation, translation).
However, in the limitations, discussion, & future directions section of the paper authors state:
"In this work we modified only the generator, but it seems likely that further benefits would be available
by making the discriminator equivariant as well."

@salvadog imo the multi-scale discrimination contributes the most, it somehow teaches the model how to do super resolution. And i feel it provides more supervison than "progressive growing", because the model would always receive gradients from all scales.

yes, I agree with this. perhaps the unet architecture for generator too

I feel like CLIP embeddings are the difference maker when doing the SR task


using tent as spawn

Is your feature request related to a problem? Please describe:
NO
Describe the solution you'd like:
to be able to use a Tent as a mobile re spawn but not have arsenal attached and limit the re spawn to like 10 times
Additional context:
N/A

read #183 , comments from dmorchard.


ENH/API: support coloring Mantel scatters

This pull request introduces a means to color the Mantel scatter plots by a categorical variable, which can help emphasize structure in the plot. An example is attached

Closed and re-opened to trigger CI workflow

That's a much prettier output! Also, the error you're getting confused me for a few minutes:
TypeError: Parameter 'metadata' received 'Metadata' as an argument, which is incompatible with parameter type: MetadataColumn[Categorical]

But it's complaining about a python string "Metadata" which is the default label for beta_correlation (label1). I think if you update that pipeline (probably with keywords) the tests should pass.

Hey @wasade I took a quick look over this PR and re-ran ci and we're still getting the same test failures that @ebolyen mentioned above. Our PR cutoff for the 2022.8 release was technically yesterday, so unless you have something ready to go here to address these failures before EOD we'll have to hold off on this one until the 2022.11 release.

TBH its been so long I don't recall what this really does. I'm on leave for
a few more weeks, it would be okay to close this.
On Fri, Aug 19, 2022, 14:01 Liz Gehret @.***> wrote:

That's a much prettier output! Also, the error you're getting confused me
for a few minutes:
TypeError: Parameter 'metadata' received 'Metadata' as an argument, which is incompatible with parameter type: MetadataColumn[Categorical]
But it's complaining about a python string "Metadata" which is the
default label for beta_correlation (label1). I think if you update that
pipeline (probably with keywords) the tests should pass.
Hey @wasade https://github.com/wasade I took a quick look over this PR
and re-ran ci and we're still getting the same test failures that @ebolyen
https://github.com/ebolyen mentioned above. Our PR cutoff for the
2022.8 release was technically yesterday, so unless you have something
ready to go here to address these failures before EOD we'll have to hold
off on this one until the 2022.11 release.
—
Reply to this email directly, view it on GitHub
https://github.com/qiime2/q2-diversity/pull/318#issuecomment-1221089159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTZMV7LKXKJAJH2OLSSBLVZ7YY5ANCNFSM5G3PX5DA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sounds good @wasade, I'll go ahead and close it! We can always re-open if/when you have bandwidth and want to fix up these test failures. Enjoy your time off! 🙂


[Antora] [PGSQL] Architecture section for postgres doc

If you commit a png also commit the sources that you used for generating it.

MinIO => S3 please it is more generic

If you commit a png also commit the sources that you used for generating it.

yes, I did,
storage_james_postgres.png -> storage_james_postgres.svg

Squash fixups please :)


Shuffle method for the Set object

I added a shuffle method to the set object. I couldn't come up with a very clean way to do this with the methods currently available. I thought you may want to add it to the code base.

I needed to distribute colors randomly over a set of shapes. I could get the number of objects (length) of the set, but not refer to specific items based on their index. Or is there a way to do this?

    var set = paper.set();
    for (var i = 0; i < 10; i++) {
        set.push(paper.circle(20 + i * 20, 20, 10));
    }
    var sorter = [];
    for (i = 0; i < 10; i++) {
        sorter[i] = i;
    }
    sorter.sort(function () {
        return Math.random() - .5;
    })
    for (i = 0; i < 10; i++) {
        set[sorter[i]].attr({fill: Raphael.getColor()});
    }


dylib dependency scan issues when using soundfile / libsndfile

In my project, I'm using the package soundfile. The package requires a local installation of libsndfile. This can be accomplished using e.g. apt install libsndfile or brew install libsndfile.
When importing the package and compiling with nuitka, the following error occurs:
❯ poetry run python -m nuitka --output-dir=build --onefile --macos-create-app-bundle --include-package=dash --include-package-data=dash --include-package=dash_core_components --include-package-data=dash_core_components --include-package=dash_html_components --include-package-data=dash_html_components --include-package=packaging --include-package-data=packaging --include-package=plotly --include-package-data=plotly --include-package=ecgviewer --include-package-data=ecgviewer src/ecgviewer/main.py
The currently activated Python version 3.10.2 is not supported by the project (~3.9).
Trying to find and use a compatible version.
Using python3.9 (3.9.17)
Nuitka-Options: Used command line options: --output-dir=build --onefile --macos-create-app-bundle --include-package=dash
Nuitka-Options: --include-package-data=dash --include-package=dash_core_components --include-package-data=dash_core_components
Nuitka-Options: --include-package=dash_html_components --include-package-data=dash_html_components --include-package=packaging
Nuitka-Options: --include-package-data=packaging --include-package=plotly --include-package-data=plotly --include-package=ecgviewer
Nuitka-Options: --include-package-data=ecgviewer src/ecgviewer/main.py
Nuitka:WARNING: For application bundles, you ought to specify an icon with '--macos-app-icon'.", otherwise a dock icon may not be
Nuitka:WARNING: present.
Nuitka: Starting Python compilation with Nuitka '2.4rc8' on Python '3.9' commercial grade 'not installed'.
Nuitka-Plugins:anti-bloat: Not including '_bisect' automatically in order to avoid bloat, but this may cause: may slow down by using
Nuitka-Plugins:anti-bloat: fallback implementation.
Nuitka-Plugins:anti-bloat: Not including '_json' automatically in order to avoid bloat, but this may cause: may slow down by using
Nuitka-Plugins:anti-bloat: fallback implementation.
Nuitka-Plugins:anti-bloat: Not including 'PIL.ImageQt' automatically in order to avoid bloat, but this may cause: PIL will not be
Nuitka-Plugins:anti-bloat: able to create Qt image objects.
Nuitka-Plugins:matplotlib: Using configuration file or default backend 'macosx'.
Nuitka: Completed Python level compilation and optimization.
Nuitka: Generating source code for C backend compiler.
Nuitka: Running data composer tool for optimal constant value handling.
Nuitka: Running C compilation via Scons.
Nuitka-Scons: Backend C compiler: clang (clang 15.0.0).
Nuitka-Scons: Backend linking program with 15006 files (no progress information available for this stage).
Nuitka-Scons: Compiled 15006 C files using ccache.
Nuitka-Scons: Cached C files (using ccache) with result 'cache hit': 15006
Nuitka-Plugins:dll-files: Found 1 file DLLs from kiwisolver installation.
Nuitka-Plugins:dll-files: Found 1 file DLLs from soundfile installation.
FATAL:     Error, problem with dependency scan of '~/Library/Caches/pypoetry/virtualenvs/ecg-viewer-ZneG1Rn7-py3.9/lib/python3.9/site-packages/_soundfile_data/libsndfile_x86_64.dylib' with '/usr/local/lib/libsndfile.1.dylib' please report the bug.

The same error also occured in a github workflow: https://github.com/Ivorforce/ECG-Viewer/actions/runs/9748479421/job/26903542049
I found related issues:

#2577 also has issues with the soundfile package, but they appear to be orthogonal.
#2580 has similar dependency tracing issues for other packages.

From information discussed in #2580, I put together a command that worked:
❯ export PYTHONPATH=/usr/local/Cellar/libsndfile/1.2.2/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/libogg/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/libvorbis/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/flac/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/opus/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/mpg123/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/opt/lame/lib/:$PYTHONPATH && PYTHONPATH=/usr/local/Cellar/libvorbis/1.3.7/lib/:$PYTHONPATH && poetry run python -m nuitka --output-dir=build --onefile --macos-create-app-bundle --include-package=dash --include-package-data=dash --include-package=dash_core_components --include-package-data=dash_core_components --include-package=dash_html_components --include-package-data=dash_html_components --include-package=packaging --include-package-data=packaging --include-package=plotly --include-package-data=plotly --include-package=ecgviewer --include-package-data=ecgviewer src/ecgviewer/main.py

Note the inclusion of more than just libsndfile into the pythonpath: When running with just PYTHONPATH=/usr/local/Cellar/libsndfile/1.2.2/lib/:$PYTHONPATH, it fails with the same error with subsequent libraries. I tracked them down until arriving at this working command (using otool -L and trial and error compilation).
It may be relevant to note that libsndfile in /user/local/lib is a symlink:
❯ readlink -f /usr/local/lib/libsndfile.1.dylib
/usr/local/Cellar/libsndfile/1.2.2/lib/libsndfile.1.0.37.dylib

Reproducibility
My repository exhibiting this bug is public: https://github.com/Ivorforce/ECG-Viewer
It is already pretty small and should be sufficient. Let me know if a more minimal reproducer is needed.
The produced .app file works on the same computer that built the .app. I have not tested whether it works if libsndfile was not installed on the end machine. I imagine this should not be an issue though considering nuitka packages these dylibs into the binary.
Versions
Note that I am running nuitka from revision 40529e75be5299f7262946b57eb095a502447ee7.
❯ poetry run python -m nuitka --version
2.4rc8
Commercial: None
Python: 3.9.17 (main, Jun 20 2023, 17:20:08)
Flavor: Homebrew Python
Executable: /Users/lukas/Jobs/KUM/ecg-viewer/.venv/bin/python
OS: Darwin
Arch: x86_64
macOSRelease: 14.5
Version C compiler: /usr/bin/clang (clang 15.0.0).

❯ poetry run python -m pip list -v
Package                   Version
------------------------- -----------
blinker                   1.8.2
bottle                    0.12.25
cachelib                  0.9.0
certifi                   2024.7.4
cffi                      1.16.0
charset-normalizer        3.3.2
click                     8.1.7
contourpy                 1.2.1
cycler                    0.12.1
dash                      2.17.1
dash-core-components      2.0.0
dash-html-components      2.0.0
dash-table                5.0.0
ecg-viewer                0.2.0
Flask                     3.0.3
Flask-Caching             2.3.0
fonttools                 4.53.0
idna                      3.7
importlib_metadata        8.0.0
importlib_resources       6.4.0
itsdangerous              2.2.0
Jinja2                    3.1.4
kiwisolver                1.4.5
MarkupSafe                2.1.5
matplotlib                3.9.1
nest-asyncio              1.6.0
Nuitka                    2.4rc8
numpy                     1.26.4
ordered-set               4.1.0
packaging                 24.1
pandas                    2.2.2
pillow                    10.4.0
pip                       23.3.1
plotly                    5.22.0
proxy_tools               0.1.0
pycparser                 2.22
pyobjc-core               10.3.1
pyobjc-framework-Cocoa    10.3.1
pyobjc-framework-Security 10.3.1
pyobjc-framework-WebKit   10.3.1
pyparsing                 3.1.2
python-dateutil           2.9.0.post0
pytz                      2024.1
pywebview                 5.1
requests                  2.32.3
retrying                  1.3.4
scipy                     1.13.1
setproctitle              1.3.3
setuptools                70.2.0
six                       1.16.0
soundfile                 0.12.1
tenacity                  8.4.2
typing_extensions         4.12.2
tzdata                    2024.1
urllib3                   2.2.2
Werkzeug                  3.0.3
wfdb                      4.1.2
zipp                      3.19.2
zstandard                 0.22.0

I made a change that should detect the Homebrew installation path. I am not at all convinced, that /usr/local/lib will be good to add as DLL source path, but that's what it's going to do for Intel systems now.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback on whether this is working is very welcome. Please do not share plans to do it; only confirm or deny that it is working.

Ok great thanks, this seems to have worked (rev 8ffad3f19c9b163d0cbf40a6438f13cf5101bda4)!

Ok, it works now locally, but remotely on the github runner it still fails (also rev https://github.com/Nuitka/Nuitka/commit/8ffad3f19c9b163d0cbf40a6438f13cf5101bda4). I imagine that's because locally I'm using homebrew, but remotely it's using something else (install-python).
Unfortunately, I do not know how to reproduce this other than to run the action remotely. But perhaps the log is already sufficient?
[...]
Nuitka-Scons: Compiled 14988 C files using ccache.
Nuitka-Scons: Cached C files (using ccache) with result 'cache miss': 14988
Nuitka-Plugins:dll-files: Found 1 file DLLs from kiwisolver installation.
Nuitka-Plugins:dll-files: Found 1 file DLLs from soundfile installation.
FATAL:     Error, problem with dependency scan of '~/work/ECG-Viewer/ECG-Viewer/.venv/lib/python3.9/site-packages/_soundfile_data/libsndfile_x86_64.dylib' with '/usr/local/lib/libsndfile.1.dylib' please report the bug.

https://github.com/Ivorforce/ECG-Viewer/actions/runs/9890803939/job/27319848775

Homebrew will be working for DLLs with 2.4 much better, but apparently not with Intel. Can you check out getHomebrewInstallPath and what you get for getSystemPrefixPath and maybe how to tell if this is where it was really installed, as I do not want to hard code paths.

I have run an action to output the requested variables. Here's the script I used to do so, using this workflow.
This is the output on mac:
Executable: /Users/runner/hostedtoolcache/Python/3.9.19/x64/bin/python
Flavour: Unknown
Prefix Path: /Users/runner/hostedtoolcache/Python/3.9.19/x64
Env GH Actions: True
Is GH Action Python: False
Except Homebrew Path ().

Checking the code from PythonFlavors, I'm guessing the culprit is that the GH actions test fails, because the path to the python executable is unexpected.
You can find all 3 runs here: https://github.com/Ivorforce/ECG-Viewer/actions/runs/9907601158/job/27371638455

Ah, github is not Homebrew, so that code doesn't apply to it. I am not at all sure, I want to add support for GitHub Python to be perfect, since we are going to add the Nuitka standalone backend as a target soon enough, for which we would have full control, this would be wasted time potentially.


Implement stops and stations too close validation (GTFS Rule)

Is your feature request related to a problem? Please describe.
Two stations should not be too close or too far from each other. This is a GTFS rule implemented in Google Python validator and featured in Google Type Error as TYPE_STATIONS_TOO_CLOSE, TYPE_STOPS_TOO_CLOSE, TYPE_STOP_TOO_CLOSE_TO_STATION and TYPE_STOP_TOO_FAR_FROM_PARENT_STATION.
Describe the solution you'd like
Actual Google GTFS validator behaviour : verifies if two stops are closer than 2 meters, and if stations are too close (considering the stops). Verifies if two stops are closer than 1000 meters apart, if not they are too far from each other.
Describe alternatives you've considered
Additional context
Line 91, 105, 106 and 116 in Error support priorities https://docs.google.com/spreadsheets/d/1vqe6wq7ctqk1EhYkgtZ0_TbcQ91vccfs2daSjn20BLE/edit#gid=0

cc @timMillet
Same here, these thresholds aren't in the spec or best practices:

https://gtfs.org/best-practices/#stopstxt
https://gtfs.org/best-practices/#stopstxt

There is no mention of this check in the spec or best practices.
That being said, we have heard from users that looking for duplicated stops in a dataset is one of their use cases for using a GTFS validator. They still need to use Google's python validator until this validator provides this type of check.

@bdferris-v2 we could potentially have this as INFO until we reach a consensus on https://github.com/MobilityData/GTFS_Schedule_Best-Practices/pull/50.
This particular check has been brought up a few times by users. Some of them even have a workflow where they only check for potential duplicate stops in a GTFS dataset.


How to get query parameters ? Is this package updated?

Hello Mr. Wu,
Yes, this package was updated. New configuration options were added.
rejectInvalidHostnames (false by default)
enableCORS (useless for now)
rejectInvalidHostnames now allows you to gain tighter security for TLS connections by verifying host/cert combinations. Setting this to true will guard against man in the middle attacks.
These new configuration options are documented at gosf.io.

Regarding getting query parameters, please review the sample project and the documentation.
Documentation: http://gosf.io/#your-first-server
Sample Project: https://github.com/ambelovsky/gosf-sample-app

Regarding getting query parameters, please review the sample project and the documentation.
Documentation: http://gosf.io/#your-first-server
Sample Project: https://github.com/ambelovsky/gosf-sample-app

I want to pass some parameters on socket connection time. like token, but I didn't find anything there. Can you tell me how I can do that ?


OCM-8147 | chore: bump dependency golang.org/x/net and google.golang.org/protobuf

WHAT
Bumps two dependencies due to moderate security issues with current versions

/lgtm


Demo code?

"Great job! I was wondering if there is any demo code available for us to run predictions or inferences on our point cloud?"

All the code are in the readme, please check.


Azure secret list

Allows the AzureKeyVault to expose the names of all available keys, such that they can be iterated.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


revert _make_api interface in da.random

[x] Tests added / passed
[x] Passes pre-commit run --all-files

There were reports of regression in da.random usage - see https://github.com/coiled/benchmarks/pull/790 for an example. This was surprising because some care were taken for backwards compatibility, but there were some changes and one such change was the introduction of _make_api function in dask/array/random.py via c4d35f5515:
def _make_api(attr, state_constructor=None, state_class=None):
    state_constructor = state_constructor or RandomState
    state_class = state_class or RandomState

    def wrapper(*args, **kwargs):
        backend = array_creation_dispatch.backend
        key = (backend, state_constructor.__name__)
        if key not in _cached_states:
            # Cache the default RandomState object for this backend
            _cached_states[key] = state_constructor()
        return getattr(
            _cached_states[key],
            attr,
        )(*args, **kwargs)

    wrapper.__name__ = getattr(state_class, attr).__name__
    wrapper.__doc__ = getattr(state_class, attr).__doc__
    return wrapper

And indeed reverting _make_api seems to make the errors go away. Current working hypothesis is there is a race condition in the wrapper function above in lines 8-10: if key not in dict: dict[key] = "new.rs.object". time of check/time of use. This can result in multiple threads creating their own RandomState instances instead of one global instance which can cause havoc down the line - different bound methods with potentially same id - when we call getattr in _apply_random.
@rjzamora : Does that make sense at all? Was there an overwhelming reason for the introduction of _make_api? And perhaps more importantly, any objections for the revert?
I would also like to take this opportunity not to expose Generator based functions directly in da.random. There are only 2 such functions - integers and multivariate_hypergeometric - and they were recently introduced in version 2023.2.1. It was a mistake and we should have never exposed them. One needs to call default_rng (as documented) to get a Generator instance and use its methods instead. This is a breaking change but, I believe, well worth the cost.
Any comments are more than welcome.
cc: @jrbourbeau

Does that make sense at all? Was there an overwhelming reason for the introduction of _make_api? And perhaps more importantly, any objections for the revert?

_make_api was introduced to account for the fact that the array backend is allowed to alternate between numpy and cudf, and so we cannot assume the default random number generation backend will be consistent after import time. Therefore, we just need to make sure we can do something like the following (I'm sorry if I never added a test like this):
dask.config.set({"array.backend": "numpy"})
da.random...  # Should produce numpy-backed data

dask.config.set({"array.backend": "cupy"})
da.random...  # Should produce cupy-backed data

dask.config.set({"array.backend": "numpy"})
da.random...  # Should produce numpy-backed data again

OK noted. the following should work then. Is there a way to get a list of supported array backends (instead of hardcoding them in line 2):
_cached_states = {}
for _backend in ["numpy", "cupy"]:
    with config.set({"array.backend": _backend}):
        try:
            _cached_states[_backend] = RandomState()
        except ImportError:
            pass

# Many of the RandomState methods are exported as functions in da.random.
# This usage is discouraged, as it is implemented via a global RandomState
# instance which is not advised on two counts:
# 1/ It uses global state, which means results will change as the code changes
# 2/ It uses a RandomState rather than the more modern Generator
# For backward compatible legacy reasons, we cannot change this.
#
# Use da.random.default_rng() to get a Generator based rng and use its
# methods instead.

def _make_api(attr):
    def wrapper(*args, **kwargs):
        backend = array_creation_dispatch.backend
        if backend in _cached_states:
            return getattr(
                _cached_states[backend],
                attr,
            )(*args, **kwargs)
        else:
            raise TypeError(f'Unknown backend type: "{backend}"')

    wrapper.__name__ = getattr(RandomState, attr).__name__
    wrapper.__doc__ = getattr(RandomState, attr).__doc__
    return wrapper

In the _make_api code:
current:
if key in dict:
  return getattr(dict[key], attr)
else:
  raise TypeError

we can of course write this as:
try:
  return getattr(dict[key], attr)
except KeyError:
  raise TypeError

Assuming there is a very high probability of success in the if key in dict check (you dont go around making random backend changes afterall), the second version is faster (about 18% on my laptop) but I did not think it was worth the loss in readability. Let me know if you think otherwise. We might be making billions of such calls and I am not sure if this is the right tradeoff.
Otherwise, I think this can be merged (barring the question about hardcoding the library names above) as it fixes a regression. What do you think?
@rjzamora

Done. Thanks for the comments. Looks good for a merge.


NTRs to OBI for arthropod specimen collection

https://github.com/obi-ontology/obi/issues/1217

Merged in OBI and ready to import after next OBI release.


ACE libraries, .zip packaging, distribution via known addon sites.

First, thank you for keeping this addon alive!
What have you changed in the ACE libraries? Hopefully you are aware of how the ACE libraries load into memory as WoW loads addons. After all addons are loaded, only the most up-to-date version of each ACE library remains in memory and all addons will use that copy.
I would like to see a .zip package with the folder structure similar to almost all the other addons. I'd also like to see this addon distributed via the well known addon sites like Curse (CurseForge), and/or WOWInterface.
I use CurseForge for the addons I maintain and its package manager handles the inclusion of the latest ACE libraries. There are alternative package managers available as well.

I see you are distributing through Tukui so my request for distribution from a known site is met (even though I've never used Tukui).

My understanding of it is that Curse is going the way of the dodo in the near future so I thought Tukui was a safer option for longevity but if there is enough demand I'm happy to put it elsewhere.
I'm not too familiar with ACE this is my first venture into addon development, I'm happy to change the structure to be more consistent with other popular addons. I'll be updating the libs to the latest versions in the next update (sometime today/tomorrow). Which would you recommend as an example of good file structure?
Nothing is changed in ACE outside of this https://github.com/MChambers1992/CopyPasta2/blob/develop/core.lua#L646

These issues mentioned should all be resolved now as of the last 2 commits. Let me know if there are any other issues or if this one isn't properly resolved, otherwise I'll close this one.

As long as the ACE libraries you distribute with your addon are the same as those officially distributed through the official channels, we are good and you can close this issue. The ACE maintainers are pretty good at making sure the libraries are up-to-date with any changes the WoW developers throw at us.
Twitch, Curse, CurseForge are under some sort of cloud and I agree that it currently may not be the best home for an addon. I am the current maintainer/author of Skillet for retail WoW and Skillet-Classic for classic WoW. While both repositories are currently on CurseForge and I use their package manager, there are alternative package managers available. I also upload my packaged .zip files to the WoWInterface site and I find their forums (and their members) are very knowledgeable and helpful (and responsive) with addon related questions / information.

I'll look into WoWInterface as it could be another good option to put the addon on. They are the official distributions of all the shared libraries, updated to the latest versions. I'll close the issue now but please feel free to open another with any other comments if you have any. Thanks!


[Feature request]: option to reset break timers after idle

Version

[X] I'm using version 1.14.0

Existing requests

[X] I've checked Existing requests

Advanced Preferences

[X] I've checked Advanced Preferences

What operating system are you using?
Linux
Operating System Version
6.2.12
Problem description
Currently, "natural breaks" pause the break timer when there is no activity for a while, which is nice. However if I stop working at a random time (not at a break) I frequently get frustrated that right after I start using my computer again, it suggests I take a break. I just took one, I don't need to do it immediately again!
Possible solution
I think a nice solution would be to have another option in the settings, which resets all the break timers after idle time. So after my machine has been idle for the idle duration, and I start using it again, I get the normal duration before getting prompted for a break.
Alternative solutions
I think the solution is proposed is the obvious one, I'm not sure there are any good alternatives really. Currently I just manually reset the breaks if one pops up too quickly, but it's annoying.
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi, what you describe is current behavior: breaks get reset after idle time. You can check (and setup) this behavior a bit: https://github.com/hovancik/stretchly#natural-breaks-inactivity-time-
I would also suggest checking out logs to see what exactly happens and compare to what you expect to happens.

whoops, I am so sorry for taking up your time with this! I think because my DPMS sleep time was shorter than my stretchly idle pause time, I was sometimes coming back from DPMS idle but not stretchly idle and then getting a break right away.
Thanks for all your work on stretchly!!

No worries, glad to help :)

OK, it just happened to me again after I was away from my machine for ~1 hr, so this was for real ;-)
It seems the key issue is that the idle timer doesn't register if I go to my lock screen while I'm away. Here is the relevant excerpt from the log:
[2023-05-05 12:49:50.342] [info]  Stretchly: showing Mini Break notification
[2023-05-05 12:50:00.502] [info]  Stretchly: showing window 1 of 2
[2023-05-05 12:50:00.502] [info]  Stretchly: starting Mini Break
[2023-05-05 12:50:00.518] [info]  Stretchly: showing window 2 of 2
[2023-05-05 12:50:20.505] [info]  Stretchly: finishing Mini Break (shouldPlanNext: true)
[2023-05-05 12:50:20.504] [info]  Stretchly: playing audio/crystal-glass.wav (volume: 1)
[2023-05-05 14:29:53.375] [info]  Stretchly: showing Long Break notification
[2023-05-05 14:30:23.534] [info]  Stretchly: showing window 1 of 2
[2023-05-05 14:30:23.534] [info]  Stretchly: starting Long Break
[2023-05-05 14:30:23.544] [info]  Stretchly: showing window 2 of 2

The 1.5 hr gap in the log is when I was away from my machine with the lock screen on, obviously it is longer than my natural break idle period.
I'm not sure exactly what is preventing idle timer from registering when the machine is locked, but perhaps Stretchly could check for such a gap and reset the breaks?
Some relevant system info: I am running i3wm (X11) and I use LightDM for the display manager/lock screen.
Btw, the natural breaks work great for me if I don't apply the lock screen. It resets the break timers just as I expect it to, in that case.

So there are no log messages between 12:50 and 14:29?
For me, on Debian KDE, breaks are paused when I lock screen:
12:14:28.802 › System: suspend or lock
12:14:28.805 › Stretchly: pausing breaks for 1ms
12:14:28.810 › System: suspend or lock
12:14:28.810 › Stretchly: not pausing for suspendOrLock because paused already
12:36:18.826 › System: resume or unlock
12:36:18.829 › Stretchly: resuming breaks
12:36:18.846 › System: resume or unlock


Backspace triggers hard-selected completion

In Codespaces, backspacing an open bracket { after @ opens a hard-selected completion session, which will consequently commit the second item in the list:

This is likely a Roslyn bug (I think we have to respond to the changes from Amadeusz's recent completion PR to fix this), but not 100% sure.

Fixed in latest release! 💪


bug: Capacitor install cordova plugin salesforce

Bug Report
Ionic version:

[x] 4.7.4
Current behavior:

iOS won't build after installing salesforce Cordova plugin
Expected behavior:

iOS should build successfully and be compatible with salesforce Cordova plugin. If I used Ionic 3 and this Cordova plugin works just fine.
Steps to reproduce:

Run ionic cordova plugin add https://github.com/forcedotcom/SalesforceMobileSDK-CordovaPlugin.git
Run npx cap sync
Run npx cap open ios

Clean and build the project and got below error:

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 5.3.0 (/Users/johnnysitu/.nvm/versions/node/v10.16.0/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.7.4
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.0.0

Cordova:

   Cordova CLI       : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms : none
   Cordova Plugins   : no whitelisted plugins (0 plugins total)

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   Android SDK Tools : 26.1.1 (/Users/johnnysitu/Library/Android/sdk)
   ios-deploy        : 1.9.4
   NodeJS            : v10.16.0 (/Users/johnnysitu/.nvm/versions/node/v10.16.0/bin/node)
   npm               : 6.9.0
   OS                : macOS Mojave
   Xcode             : Xcode 10.3 Build version 10G8

Thanks for the issue. I would recommend contacting the plugin author as Ionic Framework does not manage any of the Cordova plugins. Thanks!


cat instead of echo

ahrasis,
line 54
cat "root" > /etc/incron.allow
should be
echo "root" > /etc/incron.allow

You are correct, it shoule the latter. It fixed now.


feat(helm): initial try at making a helm chart

related to https://github.com/kubernetes-sigs/hierarchical-namespaces/issues/46
This is a rough draft of adding a helm chart. Opening as a draft as it will require changes:

[ ] add a Makefile target that sed's the appVersion and generates hack/templates/hnc-manager.yaml
[ ] adjust cloudbuild.yml to release the helm chart
[ ] someone should check my usage of helm hooks is sensible
[ ] open question -- should hnc-manager.yaml have helm templating forced onto it

/check-cla

Hi @hutchic
Are you still working on this issue?

@SydMusavi I am not. It was mentioned somewhere on an issue that if someone wishes to take over to feel free.

@adrianludwin looks like this has also been dropped. Shall we close?

@adrianludwin looks like this has also been dropped. Shall we close?

I would like to see a Helm chart to install HNC! Any chance someone could pick this up?

The funny thing is, with all the variants we're producing, that just makes
Helm charts more difficult. The real challenge here will be taking the
manifests that are produced by the HNC build process and autogenerating the
charts from them. Gatekeeper's found a way to do this but it would be
nontrivial to adopt it.
I don't want anyone to just build a Helm chart by hand, because that will
quickly become unsupported IMO.
On Tue, Mar 29, 2022 at 5:23 PM Erik Godding Boye @.***>
wrote:

@adrianludwin https://github.com/adrianludwin looks like this has also
been dropped. Shall we close?
I would like to see a Helm chart to install HNC! Any chance someone could
pick this up?
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/hierarchical-namespaces/pull/62#issuecomment-1082386999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZDSVMUQQH5OCLZMRMTVCNYFVANCNFSM5AN6O5FQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm going to close this PR, since with the new variant manifests, it's even more infeasible to follow this approach than it was when there was only one. If someone wants to revive the overall idea that's great, but it needs to be automated and based off of the output of make manifests.
/close


AccessTokenHandler doesn't work with aspnet HttpClientFactory - should not create default inner handler?

Hey, been exploring the new HttpClientFactory stuff but with this code...
var services = new ServiceCollection();
services
    .AddHttpClient("client")
    .AddHttpMessageHandler(() => new AccessTokenHandler(
        "https://demo.identityserver.io/connect/token",
        "client",
        "secret",
        "scope"));
var serviceProvider = services.BuildServiceProvider();
var httpClientFactory = serviceProvider.GetService<IHttpClientFactory>();
httpClientFactory.CreateClient("client");

...I get this exception:
System.InvalidOperationException
  HResult=0x80131509
  Message=The 'InnerHandler' property must be null. 'DelegatingHandler' instances provided to 'HttpMessageHandlerBuilder' must not be reused or cached.
Handler: 'IdentityModel.Client.AccessTokenHandler'
  Source=Microsoft.Extensions.Http
  StackTrace:
   at Microsoft.Extensions.Http.HttpMessageHandlerBuilder.CreateHandlerPipeline(HttpMessageHandler primaryHandler, IEnumerable`1 additionalHandlers)
   at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateHandlerEntry(String name)
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
   at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
   at System.Lazy`1.CreateValue()
   at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateClient(String name)
   at ConsoleApp1.Program.Main(String[] args) in E:\dev\damianh\IdentityModel2\ConsoleApp1\Program.cs:line 27

The reason is because of this check: https://github.com/aspnet/HttpClientFactory/blob/dev/src/Microsoft.Extensions.Http/HttpMessageHandlerBuilder.cs#L78-L80 which, on the face of it, seems reasonable. It also appears that the expected usage of a delegating http message handler is that the InnerHandler is to be set from the outside and is popsicle immutable on first SendAsync.

https://github.com/dotnet/corefx/blob/master/src/System.Net.Http/src/System/Net/Http/DelegatingHandler.cs#L53
https://github.com/dotnet/corefx/blob/master/src/System.Net.Http/src/System/Net/Http/DelegatingHandler.cs#L91-L101

Thus I think AccessTokenHandler should not create a default inner handler here https://github.com/IdentityModel/IdentityModel2/blob/dev/src/IdentityModel/Client/AccessTokenHandler.cs#L82 . Same would also apply to the RefreshTokenHandler here: https://github.com/IdentityModel/IdentityModel2/blob/dev/src/IdentityModel/Client/RefreshTokenHandler.cs#L108
As usual, happy to send a PR removing these if you agree.

Runnable code here: https://github.com/damianh/IdentityModel2/blob/httpclientfactory/ConsoleApp1/Program.cs


Fail to exetute the build script:

Dear Authors,
Thank you for the tool. I am trying to install the tool and run it on my local computer but got an error in the following processes of the 1st step.
For the analytical network backend
$ ./build/astra_analytical/build.sh -c
The error is shown in the screenshot. Would it be possible to provide any help? Thank you.

before this installing protobuf will be helpful

Thank you for the advice. However, protobuf is already installed. I am wondering if there is a requirement for the operation system. Currently, I am using ubuntu 18.04.5 lts（gnu/linux 5.4.72-microsoft-standard-wsl2 x86_64, the linux subsystem from Win10. After I installed cmake and other required libs, I met an error that "cmake 3.15 or higher is required.  you are running version 3.10.2", so I reinstalled the latest cmake which is  3.25.2. Does it matter?

The configuration of my environment is as follows

Have you had the opportunity to follow the instructions in the README file? Did you install all the required packages mentioned in the README? Here's the list for reference:
$ sudo apt-get -y install\
    gcc g++ make cmake\
    libboost-dev libboost-program-options-dev\
    libprotobuf-dev protobuf-compiler\
    python3 python3-pip git
$ sudo pip3 install protobuf==3.6.1 pydot

Thank you for the advice. I am sure I followed the instructions. However, during the previous installation, it reported an error that my cmake version was lower than 3.15, so I updated it to 3.25. I also double-checked other packages and the versions are as follows.

Do you still see the same error? (missing port_def.inc)

Yes. The problem is still there.

I used ubuntu 22.04 wsl and now the tool can run successfully.

Cool, good to know that!

I have the same problem. Could anyone help me solve this on Ubuntu 20.04?


Map_Server flooding on chunks with deleted bucket

Environment info

NooBaa Version: 5.4.0-93d9738
Platform: OpenShift 4.4

Actual behavior
Mappers crash with exceptions and flood the logs with lots of chunks data.
This is due to a known scenario, of deletion of buckets with unfinished (not completed multipart uploads) chunks of data.
This happens in allocations as well, when finding dups and allocations.
We shouldn't do this heavy logging and slow down uploads since it is invalid to replicate and dup from dangling chunks.
The scrubber should isolate working as designed dangling and actual bugs and log only the necessary info.
Expected behavior
We should either fail more gracefully or somehow diagnose the real problem with the expected behavior of the system.
Steps to reproduce

Fail on several multipart uploads to a bucket
Delete the bucket
Take a look for prints with Cannot read property 'tiering' of null in the mappers
@liranmauda
Says that Ceph tests are reproducing it as well.

More information - Screenshots / Logs / Other output
core.log

@jeniawhite As part as our talk, we might want to skip the logic of find_dups and do_allocations in allocation where the chunk is with no bucket property (!chunks[0].bucket)


Features suggestions - Feathub

What do you think to add http://feathub.com to your repo ?
It could be great to suggest and vote for something (and not create an issue)

This may make sense for bigger repos, but for one that is as small as this one, it isn't really necessary. Creating an issue is fine for now.

As you want, no problem ;)


Gdpr banner delay

Delays gdpr banner by four seconds. This has been cross-checked and validated by our legal team .
Resolves: https://jira.corp.adobe.com/browse/MWPW-139363
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/
After: https://gdpr-banner--express--adobecom.hlx.page/express/

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

Comparison is base (04bdf9f) 30.02% compared to head (ec4ff43) 31.65%.
Report is 19 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #471      +/-   ##
==========================================
+ Coverage   30.02%   31.65%   +1.62%
==========================================
  Files         124      125       +1
  Lines       23740    24001     +261
==========================================
+ Hits         7129     7598     +469
+ Misses      16611    16403     -208

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


Add managed module bufbuild/reflect

Mandatory
Where is the source code for the Managed Module?
https://github.com/bufbuild/reflect-proto
Do the proto files declare a package?
Yes, already pushed via buf-push-action to BSR https://buf.build/bufbuild/reflect
Optional
Does this module have other module dependencies/imports?

TBD

Setup complete. Module bufbuild/reflect is now being synced across BSR instances as managed module.


Typescript Support

Summary

Includes some build scripts for Typescript proto generation, borrowed and adapted from @snormore's previous work on the client SDK
Some prototype-y Typescript compilation. Definitely needs some testing to ensure this package is actually importable
Changes all google.protobuf.Timestamp objects to uint64. See this conversation. Technically message timestamps can be negative in Waku. I think we should just validate against that so we can assume all timestamps are positive, but I could switch to sint64 to avoid that validation if we really care.
Setup semantic-release to publish this as a NPM package

Next Steps

Ensure JS code actually runs in different environments

One thing that's pretty cringey about the generated code with protoc-gen-grpc-gateway is that the type for a uint64 is actually a string. Guess that's how they send it over the wire, and there is no runtime transformation of numeric types to strings for you.

Didn't you want to commit the generated code?

One thing that's pretty cringey about the generated code with protoc-gen-grpc-gateway is that the type for a uint64 is actually a string. Guess that's how they send it over the wire, and there is no runtime transformation of numeric types to strings for you.

Uff, that is crappy, anything reasonable alternative that would come out better?

Didn't you want to commit the generated code?

Generally I hate committing generated code. It leads to noisy and hard to read diffs and new places to find merge conflicts.
Go just doesn't leave you much choice in the matter.
But for TS it's both easy and an established pattern to generate everything on demand before you publish.

Uff, that is crappy, any reasonable alternative that would come out better?

Haven't found anything that lets me get better types out of the generator. What I am working on is a more robust ApiClient that wraps the generated code in the SDK, and adds some better types as well as handles things like pagination.

I'll update xmtp-node-go for the new timestamp format.


Improve article 2112.03899

In the end of section 8, the algorithms rendered incorrectly:
{listing*}
[htb]

class StraightThroughOneHotCategorical(torch.distributions.OneHotCategorical):
    def rsample(self, *args, **kws):
        return self.sample(*args, **kws) + self.probs - self.probs.detach()

We seem to have a latexml interpretation error with \cref@currentprefix.

This is connected to #113, can be tested together.


Math Magicians: setup

[ ] Set up a React app
[ ] no linter errors
[ ] Make sure that you used correct  Gitflow

Readme file added


Bump dcos-metrics to 1.0.0-rc4

Bump dcos-metrics to 1.0.0-rc4 to resolve an issue fetching the framework principal.
Issues
N/A
Checklist

[x] Included a test which will fail if code is reverted but test is not
[x] Followed relevant code rules Rules for Packages and Systemd

If this is a component/package update, include

[x] Change Log from last: https://github.com/dcos/dcos-metrics/compare/1.0.0-rc3...1.0.0-rc4
[x] Test Results: https://jenkins.mesosphere.com/service/jenkins/job/public-dcos-metrics/job/public-dcos-metrics-master/72/
[x] Code Coverage: https://jenkins.mesosphere.com/service/jenkins/job/public-dcos-metrics/job/public-dcos-metrics-master/72/cobertura/

Hey @adam-mesos, would you mind taking a look at this PR? This should resolve the issue with the missing framework_principal dimension in dcos-metrics.


Simplify declaration in pom.xml in README

Rather jar should be used (what is a default artifact type).

Coverage remained the same at 90.909% when pulling<PHONE_NUMBER>e17e1c11a99dd15d3aa9460c1acdb0 on szpak:patch-1 into 90d00a5f0a31359b5617842874696329dab11fb7 on http-builder-ng:master.


Update tdr-graphql-client to 0.0.154 in scala-steward-dependencies

About this PR
📦 Updates uk.gov.nationalarchives:tdr-graphql-client from 0.0.144 to 0.0.154
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "tdr-graphql-client" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "tdr-graphql-client" }
}]

labels: library-update, early-semver-major, semver-spec-patch, commit-count:1

Superseded by #31.


Encrypt stream & return stream

Respected @calccrypto ,
Does it supports to encrypt stream(e.g <fstream>) and returns encrypted and seek-able stream ?
Or is  possible to achieve with it ?
Regards.

There is no streaming mechanism in this library. The closest would be using the modes of operations, for fixed amounts of data. I imagine you can wrap their the encryption algorithm or the modes of operation to accomplish what you want.


crypto is deprecated

Blockstack's package.json currently depends on "crypto", which npm indicates is deprecated: https://www.npmjs.com/package/crypto

Thanks for raising this issue @timsu --
I'm testing out simply removing the dependency now (as it's included in nodeJS by default)


Rename flaky-test-attempts to num-flaky-test-attempts

Fix #594

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9c25955). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #595   +/-   ##
=========================================
  Coverage          ?   76.73%
  Complexity        ?      609
=========================================
  Files             ?       81
  Lines             ?     2261
  Branches          ?      324
=========================================
  Hits              ?     1735
  Misses            ?      308
  Partials          ?      218


build(vite): enable minify on build

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#205  👈
#201
#200
#202 : 1 other dependent PR (#198 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lukas-mertens and the rest of your teammates on  Graphite

Merge activity

Oct 15, 4:21 AM EDT: A user merged this pull request with Graphite.


mvkWaitForFences returns VK_SUCCESS with timeout 0

mvkWaitForFences will always return VK_SUCCESS with a timeout of 0 even if the fence isn't ready.
VkResult mvkWaitForFences(uint32_t fenceCount,
						  const VkFence* pFences,
						  VkBool32 waitAll,
						  uint64_t timeout) {

	VkResult rslt = VK_SUCCESS;
	MVKFenceSitter fenceSitter(waitAll);

	for (uint32_t i = 0; i < fenceCount; i++) {
		((MVKFence*)pFences[i])->addSitter(&fenceSitter);
	}

	if ( !fenceSitter.wait(timeout) && timeout > 0 ) {
		rslt = mvkNotifyErrorWithText(VK_TIMEOUT, "Vulkan fence timeout after %llu nanoseconds.", timeout);
	}

	for (uint32_t i = 0; i < fenceCount; i++) {
		((MVKFence*)pFences[i])->removeSitter(&fenceSitter);
	}

	return rslt;
}

&& timeout > 0  means that rslt can never be set to VK_TIMEOUT when timeout is 0.
The vulkan spec states:

If timeout is zero, then vkWaitForFences does not wait, but simply returns the current state of the fences. VK_TIMEOUT will be returned in this case if the condition is not satisfied, even though no actual wait was performed.

Thanks for pointing this out.
This resulted from a typo during recent streamlining of fence handling.
Fixed now in PR #433.
VK_TIMEOUT is returned even on zero wait. And the error is only logged if the timeout is greater than zero.
Please retest with latest code and close this issue if fixed.

yep this works now


Add Astro example

This example is the result of various research, experiments and discussions with the Astro community on the use of css themes with Astro rendering and the known issues of Flash of inAccurate coloR Theme.

Wow. Thank you!


should duplicated media to be hidden in album modal

:tada: This issue has been resolved in version 1.0.0-dev.8 :tada:
The release is available on:

v1.0.0-dev.8
GitHub release
v1.0.0-dev.8

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.1.1 :tada:
The release is available on:

v1.1.1
GitHub release
v1.1.1

Your semantic-release bot :package::rocket:


Bad topology guess in non-orthogonal pbc

Fails to detect topology of a peptide dimer crystal in a non-orthogonal unit cell... it seems some bonds at the cell interface are not detected, e.g., atom 30 is an alpha carbon and should be connected to the nitrogen N and carbon C placed at the opposite PBC image.
Here is the culprit in the Gen format: 5e61--opt--1-dftb.gen.txt

Bond and topology guessing seems to be ok. However, there is some PBC images in the kdtree construction used for connectivity detection, e.g., there should be an image of atom 30 near N and C but it isn't there when debugging.

It turns out that there was an issue when transforming the cutoff (radius) to include adjacent images to fractional coordinates.


How to selectively upgrade dependencies?

Per @techalchemy 's comment on this (now locked) issue:  https://github.com/pypa/pipenv/issues/966#issuecomment-416773434

To those who have been waiting patiently for a fix, please do try the resolver mentioned above— we are eager to see if it meets your needs. It implements proper backtracking and resolution and shouldn’t handle this upgrade strategy

I would love to help test this out, but it's not clear to me from the docs here how to test the workflow where I only want to upgrade dependencies (and their dependencies) that are necessary to the current operation (e.g. upgrading a top-level dependency). Does passa also support --selective-upgrade or --keep-outdated, or is there some other way of achieving similar behavior?

I believe it already does it by default. PinReuseProvider (used by the upgrade command by default) tries to reuse the versions specified in Pipfile.lock, and does not change them if possible. Try it out and see if it works as you’d expect (I can’t be 100% sure whether my understanding of this issue fits yours), and report anything that seems wrong to you.

Interesting. 🤔
That's great to hear, but I was sort of expecting that a resolver used by pipenv would have to also support the upgrade-everything workflow, per discussions on that same issue.
I imagine, though, since you mentioned "by default" a couple times, there's a way to override this to eagerly upgrade everything?

The reminds me, there really needs some write up on how the whole system is designed…


isplanar for Polygon3D

Can have non-planar quadrilaterals. Can check that normals of 2 triangles that partition the quad are facing the same direction

Done


Demo mode on pages

Provide a demo roadmap with a way to interactively modify the descriptor and see the result. Something similar to jsfiddle or jsbin.


egui_extras/Table: Column::auto() too narrow

Describe the bug
I'm building a table using egui_extras::TableBuilder with all auto-sized columns. In the attached repro code, all columns past the first one end up too narrow. In my actual application code, the first column also ends up too narrow but that has a very different layout context. I'm hoping it's all the same problem anyway.
As a result of this, short words end up wrapping that shouldn't - there's plenty of space in the parent panel.
In the screenshotted case, columns 2 and 3 are both too narrow by the same amount, it seems. When I ran this on my Linux laptop it seemed that the third column was narrower again, wrapping as "barbe-cue" instead of "barbecu-e" like you see here.
To Reproduce
Steps to reproduce the behavior:

Build and run https://gist.github.com/barometz/956d70a5902a82b0083646e10d4fca19

Expected behavior
All columns are wide enough to accommodate their cells, at least within the size of the container.

Screenshots

Desktop (please complete the following information):

OS: Windows 10, Fedora 38 with KDE on Wayland
Browser
Version

Additional context

In my own application code, this is partially resolved with ui.spacing_mut().item_spacing.x += 5.0; before building the table, but the first and last columns still get wrapped.
I may pick this up myself over the weekend if no one tells me this is somehow by design.

After some tinkering and an attempt to debug this in egui_extras: the problem goes away when I make it a 4x4 table with a longer word, but then reappears when I make the initial col size 200. Which is odd when the initial column size for auto is documented as having no effect unless you use a wrapping layout (and I checked - cell_layout with wrapping or off makes no difference)

(https://github.com/barometz/egui/blob/extras-table-make-auto-column-wide-enough/examples/tables/src/main.rs)
I'm having a hard time following the column sizing logic, so if anyone has any tips there I'm listening.

Found a workaround: non-wrapping labels! It seems like somewhere in the layout process, the preliminary column width is less than the label needs, so it wraps, at which point the horizontal space it occupies is less and the layout process continues with that number.
Replaced all my labels with non-wrapping ones (egui::Label::new(..).wrap(false)) and now everything looks like it's supposed to.
I'm leaving this issue open because it seems like something that could be fixed at some point, but I'm probably not going to dig into it any further.

Weird - I would expect that when the text wraps, the table row would become higher to include the new line of text - not that it would overdraw the next row.

This is one of the biggest usability issues with my application right now.
And some widgets seem to lack a non-wrapping variant, like SelectableLabel, so I can't use that workaround.

One workaround is to disable wrapping for the ui of the table entirely.
ui.style_mut().wrap = Some(false);
This requires the least amount of changes.


What kind of ornament relates unindexed to indexed Expr?

https://github.com/CoqHott/univalent_parametricity/blob/master/examples/Expr.v

Could be something we can handle now with manual configuration


upgrade rust version

Addresses https://github.com/vega/vl-convert/issues/155

On a local branch it, it ran correctly and created the binary
https://github.com/mgilbir/vl-convert/actions/runs/9539468153/job/26289836616

Awesome, thanks for looking into this!  I approved the tests and will merge when green

I updated the 1.5.0 release page with the linux-aarch64 CLI executable. Thanks again.


Add "Abort Print" function.

There is a pause button. I think a restart print button might be more useful.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
27 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
18 seconds
sanity-inspect-image

✅ Succeeded
20 seconds
deprecated-base-image-check

✅ Succeeded
24 seconds
clair-scan

✅ Succeeded
59 seconds
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
14 seconds
sanity-label-check

✅ Succeeded
15 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary


Android Basics issue:

URL of codelab
https://developer.android.com/codelabs/android-basics-kotlin-write-instrumentation-tests?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fandroid-basics-kotlin-write-instrumentation-tests#2
In which task and step of the codelab can this issue be found?

Select androidTest/java.

Describe the problem
STEP 6 IS WRONG.

PROBABLY BECAUSE IT ALREADY EXISTS.
WHICH CONTAINS "com.eample.tiptime" WHICH CONTAINS "ExampleInstrumentatedTest"
Steps to reproduce?
JUST READ STEP 6
Versions
NOT APPLICABLE
Additional information

The main branch of the starter repository for this codelab does not have the androidTest directory. This codelab expects that you work from that branch as the starting point, in which case this should not be an issue. I was unable to reproduce this.


Excuse, will this add ServiceManager V3 support?

I want to use in zend-framework/zend-mvc:^3.0.

+1

+1

I think there is no maintainers.


Switch to src/ layout

See here for places recommending it:

PyPA discussion, protocol
(Note that “single module” uses official terminology in which “module” means x.is_file() and x.suffix == '.py' and  “package” means x.is_dir() and x.join('__init__.py').is_file().)

Ionel’s blog

Hypermodern Python defers to Hynek’s blog

Furthermore I think:

The symmetry makes sense: ls *(/) will yield src tests docs, package specific names start appearing inside of those
adding {project}/src to PYTHONPATH or doing pip install -e . is safer than adding {project}

(Resolution: Try it out after all PRs touching the code got merged)

Since finding the discussion, I also found that both the Hypermodern Python package guide and Ionel’s blog (i.e. all places that I found that gives reasons for choosing a layout) picked this one.

It may not have been clear due to the hybrid meeting (which we definitely didn't prepare well for), but I think everyone else was against this. I think I'm going to revert this, though we can always add  it if it gets some support.

You said “let’s try it”.

Ah yeah, so what I had meant there (and I think was in response to others) was: "lets try it out for a project before making it a part of the template".

Aah, I see! How far did you get with the stats package yesterday?

The more I read about it the more convinced I become: https://setuptools.pypa.io/en/latest/userguide/package_discovery.html#src-layout

It seems unnecessary and reminded me of Java

That was exactly my feeling, but Hynek makes a few good points.
I think I'm slightly in favor of src now, but the question remains whether it's worth the effort changing something.

whether it's worth the effort changing something

I don’t think I understand what kind of change you’re referring to.
For the code changes it’s mkdir src; git mv '{{cookiecutter.package_name}}' src/
And what other kind of change is there?

I guess it would imply that at some point would want to update all scverse
core packages
On Fri, May 20, 2022, 10:59 Philipp A. @.***> wrote:

whether it's worth the effort changing something
I don’t think I understand what kind of change you’re referring to.
For the code changes it’s mkdir src; git mv
'{{cookiecutter.package_name}}' src/
And what other kind of change is there?
—
Reply to this email directly, view it on GitHub
https://github.com/scverse/cookiecutter-scverse/issues/28#issuecomment-1132655409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVZRV5YTFUST7V5MMBBRUDVK5H55ANCNFSM5U6H45EA
.
You are receiving this because you commented.Message ID:
@.***>

Done in #79


lots of console logs that I can't understand (asked on stackoverflow, no answers yet)

I just upgraded flutter and right after the upgrade I am getting quite a bunch of logs in the console, which don't make sense to me, tried to surf the internet for such sort of errors but apparently there don't seem to be any traces of these kind of errors: For a start, you guys can help me in trying to understand these and how to go about this because it is getting to be quite hard to debug: I must also mention though that the they don't stop the app from running:
I/HwPhoneWindow(25343): updateLayoutParamsColor false mSpecialSet=true, mForcedNavigationBarColor=true, navigationBarColor=ff6451fe, mNavBarShow=true, mIsFloating=false
I/HwPhoneWindow(25343): updateLayoutParamsColor false mSpecialSet=true, mForcedNavigationBarColor=true, navigationBarColor=ff6451fe, mNavBarShow=true, mIsFloating=false
V/InputMethodManager(25343): Reporting focus gain, without startInput
V/InputMethodManager(25343): Reporting focus gain, without startInput

Similarly whenever I get the following when I touch anywhere on the screen:
W/HiTouch_PressGestureDetector(25343): Touch pointer move a lot. The moving distance of X is:2.0, limit is:51The moving distance of Y is:69.0, limit is:51

Below log also just appears am currently not sure of what is triggering this yet:
W/InputMethodManager(25343): startInputReason = 8
W/IInputConnectionWrapper(25343): getExtractedText on inactive InputConnection
W/IInputConnectionWrapper(25343): getTextBeforeCursor on inactive InputConnection
W/IInputConnectionWrapper(25343): requestCursorAnchorInfo on inactive InputConnection
D/ZrHung.AppEyeUiProbe(25343): stop checker.
W/libEGL  (25343): EGLNativeWindowType 0x755f41b010 disconnect failed
D/ViewRootImpl[MainActivity](25343): surface should not be released
D/FlutterView(25343): Detaching from a FlutterEngine:
io.flutter.embedding.engine.FlutterEngine@486f178
W/InputMethodManager(25343): startInputReason = 3
W/IInputConnectionWrapper(25343): getExtractedText on inactive InputConnection
W/IInputConnectionWrapper(25343): getTextBeforeCursor on inactive InputConnection
W/IInputConnectionWrapper(25343): requestCursorAnchorInfo on inactive InputConnection
W/libEGL  (25343): EGLNativeWindowType 0x75660d7010 disconnect failed
V/ActivityThread(25343): Handle window ActivityRecord{d450bd5<EMAIL_ADDRESS>visibility: false
D/ZrHung.AppEyeUiProbe(25343): Current Activity:false
D/ZrHung.AppEyeUiProbe(25343): not watching, wait.
E/        (25343): [ZeroHung]zrhung_get_config: Get config failed for wp[0x0008]

https://stackoverflow.com/questions/60221016/flutter-upgrade-issue-with-lots-of-un-understandable-logs-no-harm-though

Hi @jjoek
could you provide you flutter doctor -v?

@VladyslavBondarenko check it out below: thanks
`[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.2 19C57, locale en-GB)
• Flutter version 1.12.13+hotfix.8 at /Users/jjoek/dev/flutter
• Framework revision 0b8abb4724 (3 days ago), 2020-02-11 11:44:36 -0800
• Engine revision e1e6ced81d
• Dart version 2.7.0
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/jjoek/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3, Build version 11C29
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 42.1.1
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.41.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.8.0
[✓] Connected device (1 available)
• JKM LX1 • DEF4C19A09002052 • android-arm64 • Android 9 (API 28)
• No issues found!`

#50628 because of this issue..
I already downgrade to hotfix.7
run flutter version 1.12.13+hotfix.7
I think flutter should make change log on each version release.. on this pages. https://flutter.dev/docs/development/tools/sdk/releases

@jjoek
Please follow on https://github.com/flutter/flutter/issues/50808
Closing current as duplicate


Request integration for zenkit.com

/case/652422 is requesting an integration with zenkit.com, particularly the ability to Start the Timer based on a specific list item.

ZenKit team reached out in this case as well seeking an update on the progress for the integration

We'd gladly review any pull requests adding an integration with ZenKit 👍

The Zenkit team has informed us that their API has now been released: https://zenkit.com/docs/api/overview/introduction

case reference


Add "Restart and run all" command

Discussed in https://github.com/microsoft/vscode-jupyter/discussions/9217

Originally posted by benlindsay March  3, 2022
In the Jupyter toolbar, there's a Run All button

which looks a lot like the Restart and Run All button in Jupyter Lab

My initial expectation was that those buttons would be analogous, but in VSCode, it does not restart the kernel or clear outputs. Having that button fail to clear outputs is harmful because state from the previous run can be reused. For example, say you have a notebook with 2 cells:
print(foo)

and
foo = 42

If you run just the second cell, then click "Run All", your notebook will not fail even though it should. I can't think of any reason someone should want to rerun a notebook without also restarting the kernel. As for the clearing outputs bit, there is a case to be made for not clearing outputs, because it can be nice to see what was previously there. However, 2 points in favor of clearing outputs are 1) it would match behavior expectations from Jupyter Lab, and 2) when outputs are cleared prior to rerunning, it's much easier to tell where a failure occurred when scrolling through the notebook. Harder to tell when old outputs remain.
I hacked together a solution by adding a multicommand keybinding to keybindings.json, but that really shouldn't be necessary for a critical functionality. My keybinding looks like this:
{
  "key": "alt+r",
  "command": "extension.multiCommand.execute",
  "args": {
    "sequence": [
      "notebook.clearAllCellsOutputs",
      "jupyter.notebookeditor.restartkernel",
      "notebook.execute"
    ]
  },
  "when": "notebookEditable && activeEditor == 'workbench.editor.notebook'"
}

Thanks for all of y'all's work on this project!

After discussion and some analysis of other notebook implementations, it's not clear that just replacing "run all" with "restart and run all" is prudent. Instead, it seemed to be safer to simply add a way to accomplish this behavior in addition to the existing run-all via something like adding a new command in in the jupyter-extension that could be bound to a keystroke.

I have just started using VSCode and I like it a lot, but I agree that this is a problem. I come from using Jupyter Notebooks in an ordinary Google Chrome browser for Python development, so it is quite odd to only have the few buttons available in VSCode.
My workflow when doing R&D is often that I will change some parameters in a Notebook cell, and then re-run everything from that cell and below, so all the experiments are re-run with the new parameters, but there's no need to re-load data-files etc in the cells above.
But sometimes I like to restart the kernel and re-run the entire Notebook, just to make sure that I don't have old variables or function definitions lingering somewhere in the background.
Perhaps an idea is to make the Jupyter run-buttons customizable, so the user can decide which buttons to show in the menu-bar at the top of the Notebook editor?
Thanks!

I support the idea to add a "Restart Kernel and Run all" command or/and button. Also, I think it would be confusing to use the current "Run all" button to perform this behavior.
Why not a customizable "Run all" button's behavior from settings, it is a good idea IMHO.

There haven't been any up votes on this issue hence closing this for now

I'm also interested just forgot to upvote D:

I would also like to see a "restart and run all" button, it is a big part of my workflow

Please do upvote the above issue, thats how we gauge interest in feature requests

@DonJayamanne by upvote, do you mean giving a 👍 to the original post?

I'd like to 👍 for this feature request because I usually get confused if the error/warnings are from a current/previous run. I do end up selecting Clear Outputs of All cells but would appreciate if it was done automatically (or with a single command).

I can't believe this still isn't a feature

As soon as this PR (https://github.com/microsoft/vscode/issues/177722) lands in the mainline of VSCode, you can just add this to keybinding.json.
    {   // Restart and run all cells in Jupyter notebook
        "key": "alt+shift+r",
        "command": "runCommands",
        "args": {
            "commands": [
                "notebook.clearAllCellsOutputs",
                "jupyter.notebookeditor.restartkernel",
                "notebook.execute"
            ]
        },
        "when": "notebookEditorFocused"
    },

Just wanted to update people that this command is now released within VS code insiders via the daily pre-release of the jupyter extension. It will also be available at the next monthly stable release. The command is available via the command palette.
The command name is: Jupyter: Restart Kernel and Run All Cells
The command id is: jupyter.restartkernelandrunallcells

Replacing the "Run All" button is not necessary. It is possible to add another button "Restart and Run All"

To clarify, the current toolbar button for Run All has not been replaced or altered. There is an additional command for Restart and Run All that has been added, and it currently does not have a button (only accessible via the command palette for now).

Thanks for adding this feature - but I really need the menu-button! :-)
It would also be practical if I could configure what exactly it does: I would like it to do a "Clear All" as well, but others may not like that.
Then I would probably remove the "Run All" button from my menu-bar as I don't need it anymore.
Thanks!

FYI restart action name changed to "jupyter.restartkernel"

@nikitakuklev it took me some time to find your comment and figure out why my restart keybinding wasn't working anymore. Is there a place where all the possible commands are listed that I'm not aware of? Seems like an extremely simple and useful thing to have in the Readme...

@atticusmatticus If you use the command palette (ctrl/cmd + shft + p), you can type "keyboard" and then open your keyboard shortcuts in either the UI or a JSON file. On the UI mode you can see where the command is contributed from as well, for example the Restart and Run All command is listed under the Jupyter Source.

To get the actual command name, I looked at the package.json file. Not sure if extensions must declare all commands there, but seems to work for a few common ones.

@nikitakuklev & @atticusmatticus
To get the command id, you can also do the following:

click on the gear, taking you to keyboard shortcuts

Either refer to the search that auto populates into the shortcut search bar, or right click and copy the command id directly to your clipboard


How to decide loss_fn when we build regression models?

I have observed that the loss function KLDivLoss() is utilized in training the surrogate model. However, in the TensorFlow scenario, the loss function employed is categorical_crossentropy. I am now wondering which loss function to utilize when incorporating FastSHAP when we build regression models. Additionally, are there any other significant changes we should consider?

That's a good question, we never tested FastSHAP for regression models but I can explain how I would do it. Recall that the two steps for our approach are 1) training a surrogate model and 2) training the explainer model - I'll describe below how each step needs to be modified.

Surrogate model. For classification models, our goal is to make the surrogate classifier output $\mathbb{E}[f(x) \mid x_S]$, which can be achieved by penalizing the KL divergence between the full-input and randomly masked surrogate predictions. (I'm more familiar with the PyTorch implementation, but it's possible the TensorFlow one uses a cross entropy loss with soft labels - this is the same as the KL divergence up to a constant.) Anyway, for a regression model we can just swap the loss function for a simple MSE loss. This approach to training the surrogate is discussed in this paper that came before ours, and it's the right thing to do for regression tasks because it encourages the surrogate to output $\mathbb{E}[f(x) \mid x_S]$ when the predictions are real-valued ($f(x) \in \mathbb{R}$) rather than probabilities ($f(x) \in [0, 1]$).

Explainer model. For this step, the explainer model is trained with the same weighted least squares loss. One small difference is that instead of outputting estimated attributions for each class, there is just one attribution per feature, so there is no summation in the loss across classes.

Let me know if that makes sense. If you end up writing code to try this out, I'm happy to take a look and see if it matches up with what I described here.

Hi @iancovert
Thank you for sharing your insights on employing FastSHAP for a regression model. Your suggested approach of training a regressor surrogate model and using it as the value function makes sense. However, there is one part of the algorithm that remains ambiguous to me in the regression setting. In the training algorithm for the Explainer, the value function conditioned on all possible labels (categories) is computed and added to the loss for each sample input $x \sim p(x)$:

I am still trying to figure out where this is done in fastshap.py to understand the implications when the surrogate model is modified as you suggested. However, from a fundamental perspective, shouldn't this part of the algorithm also be adapted for regression cases? I can imagine ideas such as replacing the iteration over labels with sampling from the label distribution, but I reckon this might be a heuristic that requires further analysis.

Hi again,
I guess I realized what am I missing after rereading the paper! $v_{x, y}(\mathbf{s})$ is basically the value function for the $y^{th}$ class of the output, not the value function conditioned on both model's input and output. When using a surrogate model as the value function, feeding it with $\mathbf{x}_s$ results in the output vector, and the mentioned iteration is done algebraically. Therefore, in the case of a classic one-dimensional regression surrogate model, it should be all good and nothing needs to be changed!

Hi, yes that’s exactly right! Training FastSHAP for classifiers basically means calculating the squared error loss for each class and then summing, so for regression models with a single output dimension, you just calculate the loss once and no summation is necessary.


Accurate odometry, working field oriented w/ toggle, Pathplanner implemented, working teleop controls

Odometry is working now. The front of the robot is the radio. Path planner works and has accuracy within +- 3 inches or so. Will need to be tuned more on the real bot. Module optimization seems to be working pretty well but it needs to be verified more.

Not a high priority, but the auto path did not run in sim, at least when I tried it (the pose was reset though).

Unfortunately I forgot to try and figure out how to get it working on sim today. I've created an issue for it - #7