Change fullscreen video behavior

The aim of the PR is to fix #539 , while taking into account other devices like foldables.

Please @vitorpamplona I don't know if it's a good idea to include Composable functions in the Quartz module.

Yeah, we should probably keep the utility function in the same package as the videoview

Yeah, we should probably keep the utility function in the same package as the videoview

Got it.

@vitorpamplona The PR is all set and ready.

Thank you @KotlinGeekDev

@KotlinGeekDev we need a fix for portrait videos. The device should not go to landscape on fullscreen if the video is portrait. Many videos in the video tab are like that.

@vitorpamplona does it mean we should just go to fullscreen only when the user willingly changes orientation?

It should go full screen all the time, but only rotate to landscape when the video is in landscape. Otherwise, the portrait video shows in landscape an only uses a tiny space in the middle of the screen

It should go full screen all the time, but only rotate to landscape when the video is in landscape. Otherwise, the portrait video shows in landscape an only uses a tiny space in the middle of the screen

I see. So basically the same behsviour as before?

If the video is portrait, then yes, same behavior as before. If the video is in landscape, then it should change orientation

Got it.

Please @vitorpamplona it therefore means disabling it as you did will enough to fix the issue you brought up, right?

Disabling like that disables the entire PR, right? It won't rotate for landscape videos, when it should.

Disabling like that disables the entire PR, right? It won't rotate for landscape videos, when it should.

Interesting. From my experience testing the effects of disabling, it works even in landscape mode.

My understanding of the original issue is that the author wanted the NewPipe app experience, whereby, on portrait devices, tapping the fullscreen button changes the orientation to landscape, then going fullscreen.
Please @anthony-robin correct me if I am wrong.

Interesting. From my experience testing the effects of disabling, it works even in landscape mode.

But how would it work if it is not calling any function to change orientation? In my device, it now doesn't change any orientation at all.

Interesting. From my experience testing the effects of disabling, it works even in landscape mode.

But how would it work if it is not calling any function to change orientation? In my device, it now doesn't change any orientation at all.

I think that the way the fullscreen experience currently works(the way it worked before my PR) is that the orientation was manually set by the user in the system settings, which affected the app.

@vitorpamplona My understanding of the original issue is that the author wanted the NewPipe app experience, whereby, on portrait devices, tapping the fullscreen button changes the orientation to landscape, then going fullscreen. Please @anthony-robin correct me if I am wrong.

Hi @KotlinGeekDev, the original issue was about a NewPipe experience (that might not be the only app that does this but the one I had in mind when opening the issue) by having an auto fullscreen video mode when I turn my phone in a landscape position on a "Live stream" video page.
Also note that when the landscape fullscreen is turned back to portrait, fullscreen should be disabled back to the original rendering (if it is possible)
I didn't really thought about the portrait mode so I think it is better to keep it in fullscreen portrait with a good video rendering and not a tiny content with lot of black on horizontal sides.
Let me know if you have more questions or if things are not clear enough.
Anyway, thank you for working on the subject :)

Hi @KotlinGeekDev, the original issue was about a NewPipe experience (that might not be the only app that does this but the one I had in mind when opening the issue) by having an auto fullscreen video mode when I turn my phone in a landscape position on a "Live stream" video page.
Also note that when the landscape fullscreen is turned back to portrait, fullscreen should be disabled back to the original rendering (if it is possible)
I didn't really thought about video that are recorded in portrait mode directly so I think it is better to keep it in fullscreen portrait with a good video rendering and not a tiny content with lot of black on horizontal sides.
Let me know if you have more questions or if things are not clear enough. Anyway, thank you for working on the subject :)

Oh @anthony-robin got it! Thanks for making it clearer.


[SPARK-40034][SQL] PathOutputCommitters to support dynamic partitions

What changes were proposed in this pull request?

Uses the StreamCapabilities probe in MAPREDUCE-7403 to identify when a
PathOutputCommitter is compatible with dynamic partition overwrite.
This patch has unit tests but not integration tests; really needs
to test the SQL commands through the manifest committer into gcs/abfs,
or at least local fs. That would be possible once hadoop 3.3.5 is out...
Uses the StreamCapabilities probe in MAPREDUCE-7403 to identify when a
PathOutputCommitter is compatible with dynamic partition overwrite.
Why are the changes needed?
Hadoop 3.3.5 adds a new committer in mapreduce-core which works fast and correctly on azure and gcs. (it would also work on hdfs, but its optimised for the cloud stores).
The stores and the committer do meet the requirements of Spark SQL Dynamic Partition Overwrite, so it is OK to for spark to work through it.
Spark does not know this; MAPREDUCE-7403 adds a way for any PathOutputCommitter to declare that they are compatible; the IntermediateManifestCommitter will do so.
(https://github.com/apache/hadoop/pull/4728)
Does this PR introduce any user-facing change?
No.
There is documentation on the feature in the hadoop manifest committer docs.
How was this patch tested?

Unit tests in hadoop-cloud which work with hadoop versions with/without the matching change.
New integration tests in https://github.com/hortonworks-spark/cloud-integration which require spark to be built against hadoop with the manifest committer declaring compatibility

Those new integration tests include

spark sql test derived from spark's own CloudRelationBasicSuite.scala#L212
Dataset tests extended to verify support for/rejection of dynamic partition overwrite AbstractCommitDataframeSuite.scala#L151

Tested against azure cardiff with the manifest committer; s3 london (s3a committers reject dynamic partition overwrites)

this should interest @sunchao and @dongjoon-hyun -know that this doesn't add support to the s3a committers; s3 itself doesn't do the right thing (rename()). Does for abfs and gcs through the manifest committer.

the hadoop side of this change is now merged in.
@attilapiros do you have any time to review this again?

added a new test case and updated the docs. I've not yet rebased it/merged it with your committer work, but the docs shouldn't clash. once this PR is in I will set my local build up to run your tests against s3 london

BTW, when is the ETA for Apache Hadoop 3.3.5, @steveloughran ?

@dongjoon-hyun I'm off on vacation next week; we will fork off the branch the week after.
things i'd like in if anyone has the time

upgraded shaded parquet
the shaded avro PR
get that arm64 docker image working for a release there

Merged to master for Apache Spark 3.4.0.

Thank you, @steveloughran and @attilapiros .

@attilapiros FYI, there is a new PR for this area.

https://github.com/apache/spark/pull/40221

This is reverted from branch-3.4 only while being kept in master branch for Apache Spark 3.5.

https://github.com/apache/spark/commit/57aa3d1ca6e17e4c6b934d74176ea22ca56d60f7


NNI 2021 Jan~Feb Iteration Planning

This is the plan for the iteration in Jan~Feb, it's a 4 weeks iteration.
Release Plan for v2.1

Release manager:  @J-shang
Feature freeze date:  ~2.8~ 2.10
Code freeze and demo date: ~2.22~ 2.24
Release date: 3.1

Let's try to adopt the legend of annotations:

Mark
Description

🏃♂
work in progress

👪
work under review

✋
blocked task

💪
stretch goal for this iteration

🔴
missing issue / pull request reference

🔵
more investigation and survey required

⚫
under discussion within the team

⬛
a large work item, larger than one iteration

Features
nnictl & nni.experiment

[x] P0 - migrate nnicli to python approach @J-shang DL: 1.27 PR: #3334 review: @liuzhe-lz @QuanluZhang
[x] 👪 ~P1~ P0 - ~launch/view/~ stop python approach @J-shang pr: #3396 review: @QuanluZhang @liuzhe-lz @SparkSnail
[x] P1 - Show a warning if user run trial.py without nnictl @liuzhe-lz  PR #3312  review: @ultmaster @J-shang
[x] Nnictl support launch from Python (merge .config & nni.sqlite) @J-shang PR #3309 review: @SparkSnail @liuzhe-lz

Retiarii & NAS

[x] P0 - Improve graph generation and code generation logic (support control flow) @QuanluZhang DL: ~1.29~ 2.5 PR: ~#3330~ #3365 review: @ultmaster @liuzhe-lz

[x] Support member function, and more operators DL: ~1.25~ 1.27
[x] Support shared module DL: 2.1
[x] Test graph convert on different pytorch versions
[ ] Support member variable

[x] P0 - Support APIs ValueChoice  @ultmaster DL: 1.27 PR: #3349

[x] support ValueChoice parameters DL: 2.2 PR: #3382 review: @QuanluZhang @liuzhe-lz

[x] P1 - Refactor Trainer @ultmaster DL: 2.5 PR: #3359
[x] P1 - Support serialization @ultmaster
[x] P1 - Support 2 or 3 simple search algorithms (grid, random, evolution) @ultmaster PR: #3377 review: @QuanluZhang @liuzhe-lz

Model compression

[x] P2 - Make model compression user experience more user-friendly @colorjam DL: 1.29 #3326 review: @QuanluZhang @J-shang
[x] P2 - Documentation improvement (update API reference for model compression in doc, deferred from v2.0) @colorjam DL: 1.29
[x] 👪 Refactor Compression Doc @colorjam #3371 review: @QuanluZhang @J-shang

Training service & NNI manager

[x] P1 - Formally support shared storage for reusable training service @J-shang DL: ~1.22~ 1.27 PR: #3354 review: @SparkSnail @liuzhe-lz
[x] P1 - Hybrid local windows support @SparkSnail DL: 1.25 PR: #3353
[x] P1 - Remove PAIYarn code @SparkSnail DL: 1.22 PR: #3327
[x]  P1 - AML outputs path & python process cannot be killed @J-shang PR #3321 review: @SparkSnail @liuzhe-lz
[x] P1 - show the assigned training service of each trial in hybrid training service (#3261) @SparkSnail  DL: 2.10 PR: #3389 review: @J-shang @liuzhe-lz
[x] P1 - AML scheduling  @SparkSnail DL: ~2.5~ 2.7 PR: #3375 review: @QuanluZhang @J-shang
[x] P2 - Reuse mode envs dir iterate @SparkSnail DL: ~2.5~ 2.7 PR: #3374 review: @liuzhe-lz @J-shang
[x] P2 - GPU metrics collector does not take preCommand in remote training service (#3284) ~@J-shang~ @acured #3367
[x] P3 - Flatten nested JSON in RPC protocol (deferred from v2.0) @liuzhe-lz

WebUI

[x] P1- manage all experiments Support multiple selection for filter status @Lijiaoa DL: 1.29 PR: #3351
[x] Overview page improvement @Lijiaoa

[x] P1 - Update default metric tooltip #2964 DL: 1.22 PR: #3316 review: @ultmaster @liuzhe-lz
[x] P1 - Support show customized tuner name DL: 1.22 PR: #3317
[x] P1 - Support show trial details by clicking the point in trial results graph DL: 1.29 PR: #3352

[x] P1 - copy trial id in table @Lijiaoa PR: #3378 review: @ultmaster @liuzhe-lz
[x] P1 - Hybrid show trial platforms in webui @Lijiaoa (need UX design, depends on #3261)

Pipelines

[x] P1 - Pipeline speedup @ultmaster DL: 1.25 PR: #3313
[x] P1 - pipeline dependence migration DL: ~2.3~ 2.5 @ultmaster  PR: #3364
[x] P1 - fix IT @J-shang PR: #3366

Deferred
nnictl & nni.experiment

[ ] P1 - Improve tuner debuggability, allow debugger tool  @liuzhe-lz
[ ] P1 - write v2 config files for examples @liuzhe-lz PR: #3388 (Depend on pr #3315)
[ ] P2 - Support nnictl tensorboard in Windows @J-shang
[ ] P2 - Mask confidential config fields @liuzhe-lz
[ ] P2 - nnictl auto-completion @liuzhe-lz
[ ] P3 - launch NNI manager in remote container #modelbuilderneeds @liuzhe-lz

Retiarii & NAS

[ ] P1 - Support one system-level optimization: CSE optimization to deduplicate computation @hzhua
[ ] P1 - NAS visualization improvements: refactor graph representation (deferred from v2.0) @98may
[ ] P2 - refactor strategy (strategy keep running when experiment stop)
[ ] P2 - Add high level API for once-for-all related supports @ultmaster
[ ] P3 - Test classic nas tuners on NNI NAS benchmark and report evaluation results (deferred from v2.0) @ultmaster

Model compression

[ ] 👪 P1 - Quantization speedup tool @linbinskn DL: 1.22 PR: ~#3220~ #3356 review: @J-shang @liuzhe-lz @QuanluZhang
[ ] P2 - Pruning model speedup improvement @zheng-ningxin
[ ] P2 - Update TF model compression: support checkpoint (deferred from v2.0) @liuzhe-lz

Training service & NNI manager

[ ] P1 - Native support for config v2 @liuzhe-lz PR: #3315 review: @SparkSnail  @J-shang

[ ] P0 - support view mode

[ ] P2 - Reuse mode remote windows (@acured ask @SparkSnail )
[ ] P2 - Hybrid kubeflow support @SparkSnail
[ ] P2 - Reuse mode envs dir iterate @SparkSnail
[ ] P2 - Documentation improvement @QuanluZhang
[ ] P2 - Add more error message for remote training service (#3259) @QuanluZhang  (fixed upstream, to verify the fix)
[ ] P3 - Unify IPC mechanism for local and others @liuzhe-lz @ultmaster

Hyper-parameter tuning

[ ] P2 - Improve hyperband, PBT with shared storage (dependency on shared storage implementation) @J-shang
[ ] P3 - Formal Advisor API @liuzhe-lz
[ ] P2 - Multi-objective optimization algorithm (deferred from v2.0) @ultmaster
[ ] P4 - FBnetV3 tuning algorithm (deferred from v2.0) @J-shang
[ ] P4 - Tutorial to use tuner directly instead of using nni manager (like hyperopt?) (deferred from v2.0) @liuzhe-lz

WebUI

[ ] Overview page improvement @Lijiaoa

[ ] P2 - Adaptive UI (height) for high resolution, current window for high resolution need improvement @yangou1988 @Lijiaoa
[ ] P3 - Fix redundant margin at the bottom of the page

[ ] P2 - Support tensorboard @Lijiaoa

[ ] UX design DL: 2.3
[ ] backend

[ ] Detail Page improvement

[ ] P1 - Refine search parameters #3024 #3099 DL: 2.21
[ ] P2 - Put the add/remove button in the last column of the header #3099

[ ] P3 - webUI dispatcher log / nnimanager log wrap (#3246)  @Lijiaoa

Pipelines

[ ] P2 - add reuse test @liuzhe-lz
[ ] P3 - Support code coverage check, add this check to IT (deferred from v2.0) @ultmaster

Documentation

[ ] P1 - Refactor readthedocs integration @ultmaster
[ ] P2 - Improve crowdin workflow @ultmaster

Design Discussion Needed

[ ] Design discussion for next steps of manage all experiments, including "actions" for Manage experiments, etc. #3100
[ ] Design discussion for Duration issue #3012
[ ] fail trials counted in total trial nums
[ ] Improve a more typing-friendly experiment id pattern (#3102)
[ ] Avoid reloading data for each trail (#3294) @QuanluZhang
[ ] Experiment tag

Others

[ ] P3 - Bump dependency version (deferred from v2.0) @liuzhe-lz
[ ] P2 - Add the output of subprocess (#3141) @liuzhe-lz
[ ] P3 - Set NNI base URL (aka webui prefix) (#2771)

[x] Check users' requirements @Lijiaoa @yiru
[ ] Web UI front-end  @Lijiaoa
[ ] backend @J-shang
[ ] NNI manager & nnictl back-end  @?

[ ] P4 - AutoML for data augmentation (image and NLP) (research)
[ ] P3 - Improve dev-install on Windows @liuzhe-lz

Backlog

[ ] support IPV6
[ ] update tuner search space

Deferred
nnictl & nni.experiment

[ ] P2 - Support nnictl tensorboard in Windows @J-shang
[ ] P2 - Mask confidential config fields @liuzhe-lz
[ ] P2 - nnictl auto-completion @liuzhe-lz
[ ] P3 - launch NNI manager in remote container #modelbuilderneeds @liuzhe-lz

Retiarii & NAS

[ ] P1 - Support one system-level optimization: CSE optimization to deduplicate computation @hzhua
[ ] P1 - NAS visualization improvements: refactor graph representation (deferred from v2.0) @98may
[ ] P2 - refactor strategy (strategy keep running when experiment stop)
[ ] P2 - Add high level API for once-for-all related supports @ultmaster
[ ] P3 - Test classic nas tuners on NNI NAS benchmark and report evaluation results (deferred from v2.0) @ultmaster

Model compression

[ ] P2 - Pruning model speedup improvement @zheng-ningxin
[ ] P2 - Update TF model compression: support checkpoint (deferred from v2.0) @liuzhe-lz

Training service & NNI manager

[ ] P2 - Hybrid kubeflow support @SparkSnail
[ ] P2 - Documentation improvement @QuanluZhang
[ ] P2 - Add more error message for remote training service (#3259) @QuanluZhang  (fixed upstream, to verify the fix)
[ ] P3 - Unify IPC mechanism for local and others @liuzhe-lz @ultmaster

Hyper-parameter tuning

[ ] P2 - Improve hyperband, PBT with shared storage (dependency on shared storage implementation) @J-shang
[ ] P3 - Formal Advisor API @liuzhe-lz
[ ] P2 - Multi-objective optimization algorithm (deferred from v2.0) @ultmaster
[ ] P4 - FBnetV3 tuning algorithm (deferred from v2.0) @J-shang
[ ] P4 - Tutorial to use tuner directly instead of using nni manager (like hyperopt?) (deferred from v2.0) @liuzhe-lz

WebUI

[ ] Overview page improvement @Lijiaoa

[ ] P2 - Adaptive UI (height) for high resolution, current window for high resolution need improvement @yangou1988 @Lijiaoa
[ ] P3 - Fix redundant margin at the bottom of the page

[ ] P2 - Support tensorboard @Lijiaoa

[ ] UX design DL: 2.3
[ ] backend

[ ] Detail Page improvement

[ ] P1 - Refine search parameters #3024 #3099 DL: 2.21
[ ] P2 - Put the add/remove button in the last column of the header #3099

[ ] P3 - webUI dispatcher log / nnimanager log wrap (#3246)  @Lijiaoa

Pipelines

[ ] P2 - add reuse test @liuzhe-lz
[ ] P3 - Support code coverage check, add this check to IT (deferred from v2.0) @ultmaster

Documentation

[ ] P1 - Refactor readthedocs integration @ultmaster
[ ] P2 - Improve crowdin workflow @ultmaster

Design Discussion Needed

[ ] Design discussion for next steps of manage all experiments, including "actions" for Manage experiments, etc. #3100
[ ] Design discussion for Duration issue #3012
[ ] fail trials counted in total trial nums
[ ] Improve a more typing-friendly experiment id pattern (#3102)
[ ] Avoid reloading data for each trail (#3294) @QuanluZhang
[ ] Experiment tag

Others

[ ] P3 - Bump dependency version (deferred from v2.0) @liuzhe-lz
[ ] P2 - Add the output of subprocess (#3141) @liuzhe-lz
[ ] P3 - Set NNI base URL (aka webui prefix) (#2771)

[x] Check users' requirements @Lijiaoa @yiru
[ ] Web UI front-end  @Lijiaoa
[ ] backend @J-shang
[ ] NNI manager & nnictl back-end  @?

[ ] P4 - AutoML for data augmentation (image and NLP) (research)
[ ] P3 - Improve dev-install on Windows @liuzhe-lz

Backlog

[ ] support IPV6
[ ] update tuner search space

bugbash
nnictl & nni.experiment

[x] P0 - migrate nnicli to python approach @J-shang DL: 1.27 PR: #3334 review: @liuzhe-lz @QuanluZhang
[x] 👪 ~P1~ P0 - ~launch/view/~ stop python approach @J-shang pr: #3396 review: @QuanluZhang @liuzhe-lz @SparkSnail
[x] P1 - Show a warning if user run trial.py without nnictl @liuzhe-lz  PR #3312  review: @ultmaster @J-shang
[x] Nnictl support launch from Python (merge .config & nni.sqlite) @J-shang PR #3309 review: @SparkSnail @liuzhe-lz

Retiarii & NAS

[x] P0 - Improve graph generation and code generation logic (support control flow) @QuanluZhang DL: ~1.29~ 2.5 PR: ~#3330~ #3365 review: @ultmaster @liuzhe-lz

[x] Support member function, and more operators DL: ~1.25~ 1.27
[x] Support shared module DL: 2.1
[x] Test graph convert on different pytorch versions
[ ] Support member variable

[x] P0 - Support APIs ValueChoice  @ultmaster DL: 1.27 PR: #3349

[x] support ValueChoice parameters DL: 2.2 PR: #3382 review: @QuanluZhang @liuzhe-lz

[x] P1 - Refactor Trainer @ultmaster DL: 2.5 PR: #3359
[x] P1 - Support serialization @ultmaster
[x] P1 - Support 2 or 3 simple search algorithms (grid, random, evolution) @ultmaster PR: #3377 review: @QuanluZhang @liuzhe-lz

Model compression

[x] P2 - Make model compression user experience more user-friendly @colorjam DL: 1.29 #3326 review: @QuanluZhang @J-shang
[x] P2 - Documentation improvement (update API reference for model compression in doc, deferred from v2.0) @colorjam DL: 1.29
[x] 👪 Refactor Compression Doc @colorjam #3371 review: @QuanluZhang @J-shang

Training service & NNI manager

[x] P1 - Formally support shared storage for reusable training service @J-shang DL: ~1.22~ 1.27 PR: #3354 review: @SparkSnail @liuzhe-lz
[x] P1 - Hybrid local windows support @SparkSnail DL: 1.25 PR: #3353
[x] P1 - Remove PAIYarn code @SparkSnail DL: 1.22 PR: #3327
[x]  P1 - AML outputs path & python process cannot be killed @J-shang PR #3321 review: @SparkSnail @liuzhe-lz
[x] P1 - show the assigned training service of each trial in hybrid training service (#3261) @SparkSnail  DL: 2.10 PR: #3389 review: @J-shang @liuzhe-lz
[x] P1 - AML scheduling  @SparkSnail DL: ~2.5~ 2.7 PR: #3375 review: @QuanluZhang @J-shang
[x] P2 - Reuse mode envs dir iterate @SparkSnail DL: ~2.5~ 2.7 PR: #3374 review: @liuzhe-lz @J-shang
[x] P2 - GPU metrics collector does not take preCommand in remote training service (#3284) ~@J-shang~ @acured #3367
[x] P3 - Flatten nested JSON in RPC protocol (deferred from v2.0) @liuzhe-lz

WebUI

[x] P1- manage all experiments Support multiple selection for filter status @Lijiaoa DL: 1.29 PR: #3351
[x] Overview page improvement @Lijiaoa

[x] P1 - Update default metric tooltip #2964 DL: 1.22 PR: #3316 review: @ultmaster @liuzhe-lz
[x] P1 - Support show customized tuner name DL: 1.22 PR: #3317
[x] P1 - Support show trial details by clicking the point in trial results graph DL: 1.29 PR: #3352

[x] P1 - copy trial id in table @Lijiaoa PR: #3378 review: @ultmaster @liuzhe-lz
[x] P1 - Hybrid show trial platforms in webui @Lijiaoa (need UX design, depends on #3261)

Pipelines

[x] P1 - Pipeline speedup @ultmaster DL: 1.25 PR: #3313
[x] P1 - pipeline dependence migration DL: ~2.3~ 2.5 @ultmaster  PR: #3364
[x] P1 - fix IT @J-shang PR: #3366

bugbash 2.0
nnictl & nni.experiment

[ ] tuner name display on web UI (launch from python) @kvartet
[ ] update_search_space should work when use start() now @kvartet

Retiarii & NAS

[ ] Support APIs ValueChoice   @QuanluZhang

[ ] support ValueChoice parameters

[ ]  Refactor Trainer @colorjam
[ ] Support 2 or 3 simple search algorithms (grid, random, evolution) @98may

Training service & NNI manager

[ ] Formally support shared storage for reusable training service @SparkSnail
[ ] Hybrid v2 config convert @liuzhe-lz
[ ] AML scheduling  @acured
[ ] GPU metrics collector does not take preCommand in remote training service (#3284) @ultmaster


Upgrade gem version

There is no new version of the gem, so we get old dependencies like rest-client (~> 1.6.7).
Please upgrade to 1.1.3

I'd like to second this and clarify - I think @nasospsa brought this up because there is a known security vulnerability in rest-client versions before 1.7.3.

Not sure if anyone on the Kong team is paying attention to this repo - ping @thibaultcha ? If no update is expect the insecure versions should probably be revoked from RubyGems.


@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/elastic-apm-feedstock#16.

@conda-forge-admin, please add bot automerge


Refactored properties: details page

These changes were requested in my original PR, #1962.
Changes made:

Moved properties to a list view, bound to a grouped data set
Removed dialog header
Removed overview section, as it is unnecessary with a "view all" feature
Support for other languages with property names
Fix issue with text box size
Dialog for failed save

Screenshots:

Push notification for failed save

@winston-de Instead of a push notification, it's probably better to display a dialog.

@winston-de Is this ready for review yet?

@yaichenbaum Yes

@winston-de It looks like there are several merge conflicts, can you solve those?

@yaichenbaum Sure.
I do that by merging with upstream/master in VS, right?

@yaichenbaum Sure.
I do that by merging with upstream/master in VS, right?

Yeah

@winston-de It looks like the resource files just got messed up.

@yaichenbaum Can you please tell me what you mean? I can't seem to figure out what the issue is.

foreach (string line in database)
{
    string[] lineSplit = line.Split(separator);
    string NameResource = lineSplit[0];
    string SectionResource = lineSplit[1];
    string ID = lineSplit[2];
    props.Add(new FileProperty(NameResource, SectionResource, ID));
}

foreach (string line in database)
{
    string[] lineSplit = line.Split(separator);
    string NameResource = lineSplit[0];
    string SectionResource = lineSplit[1];
    string ID = lineSplit[2];
    props.Add(new FileProperty(NameResource, SectionResource, ID));
}

@d2dyno1 Do you want me to move the base list of properties to a separate file?

@d2dyno1 I think this could be accomplished pretty easily with a Json deserializer and a separate Json file.

I don't think we need an extra dialog page. The dialog can be created in codebehind.

@winston-de Can you check, is Details tab working for you while file hash calculation is not complete?

Can you check, is Details tab working for you while file hash calculation is not complete?
@tsvietOK It works on my machine, although it does seem to cause a slight delay.
I'll have access to another device in a little bit, I'll test it on that, too.

@winston-de I testing with 14Gb file, it takes 1 minute to get details properties.

@tsvietOK Oh ok. I tested it with larger files, and I have the same issue. I'll add some code to cancel the task when the user switches tabs.

@winston-de When opening Details tab and pressing Ok button, it shows error dialog, when trying to save changes without an actual chnages.

@winston-de When changing Title field, this change applies to Subject too.

@winston-de What about to remove read only fields if they are empty?

@winston-de For some reason some of the properties are not retrieved.

@tsvietOK

When changing Title field, this change applies to Subject too.

What type of file are you testing on? I am unable to reproduce this issue.

What about to remove read only fields if they are empty?

Good idea.

For some reason some of the properties are not retrieved.

Does this happen with all image files?

What type of file are you testing on? I am unable to reproduce this issue.

Photo(,jpg).

Does this happen with all image files?

Hmmm, now it shows horizontal and vertical resolutions, but without Color space. And yes, for all image files.

@tsvietOK

Photo(,jpg).

I tried it on a jpg file, and I had the issue once, then it stopped happening. I think it might be a bug with the storage API.

Hmmm, now it shows horizontal and vertical resolutions, but without Color space. And yes, for all image files.

I don't see color space either, however I don't think it's an issue with Files. I tried using an online metadata viewer and it didn't show anything.

@winston-de Looks like Color space property is not empty.

Value 1 means it is sRGB: https://www.awaresystems.be/imaging/tiff/tifftags/privateifd/exif/colorspace.html

@tsvietOK Thanks. I think the reason it was showing as null is because it's a Uint16, which doesn't work with the current system. I'll fix that, and also add support for enumerated lists so that it displays the text instead of just the number value.

@winston-de Do you plan to add more supported properties and when? (https://docs.microsoft.com/ru-ru/windows/win32/properties/props)

@tsvietOK I do intend to add some more later this month, however there are a lot of properties and I'm a bit busy. It's also very easy to expand the json file, so if anyone has some they want to add they can.

@tsvietOK I do intend to add some more later this month, however there are a lot of properties and I'm a bit busy. It's also very easy to expand the json file, so if anyone has some they want to add they can.

It is not a problem, just asking.

@winston-de Great work!

@yaichenbaum Thanks :)


feat: Describe api

Codecov Report

Merging #71 (7f57e0f) into main (59e707f) will decrease coverage by 0.89%.
The diff coverage is 78.31%.

@@             Coverage Diff              @@
##               main      #71      +/-   ##
============================================
- Coverage     89.50%   88.60%   -0.90%
- Complexity      343      365      +22
============================================
  Files            47       51       +4
  Lines          1029     1106      +77
  Branches         63       72       +9
============================================
+ Hits            921      980      +59
- Misses           94      112      +18
  Partials         14       14

Impacted Files
Coverage Δ

...om/tigrisdata/db/client/CollectionDescription.java
53.33% <53.33%> (ø)

.../com/tigrisdata/db/client/DatabaseDescription.java
53.33% <53.33%> (ø)

...risdata/db/client/StandardTigrisAsyncDatabase.java
88.33% <75.00%> (-2.24%)
:arrow_down:

...sdata/db/client/StandardTigrisAsyncCollection.java
89.28% <80.00%> (-0.92%)
:arrow_down:

...a/com/tigrisdata/db/client/CollectionMetadata.java
100.00% <100.00%> (ø)

...ava/com/tigrisdata/db/client/DatabaseMetadata.java
100.00% <100.00%> (ø)

...tigrisdata/db/client/StandardTigrisCollection.java
90.12% <100.00%> (+0.93%)
:arrow_up:

...m/tigrisdata/db/client/StandardTigrisDatabase.java
93.10% <100.00%> (+0.37%)
:arrow_up:

...grisdata/db/client/StandardTransactionSession.java
95.00% <100.00%> (+0.12%)
:arrow_up:

...ava/com/tigrisdata/db/client/TigrisJSONSchema.java
82.35% <100.00%> (ø)

... and 1 more

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 59e707f...7f57e0f. Read the comment docs.


New navigation for 1.10 Release

For 1.10 we are changing the taxonomy to product-specific menus.

Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

cc @docker/machine-maintainers

We wrote some feedback inline. PTAL @moxiegirl

@jeanlaurent @dgageot This PR is about changing the menus.  To @dgageot point, we should be using "Machine" in the menus consistently.  I'll update to that.  We do that here to conserve space on the menus and, well, Docker is assumed given the other clues on the website like the logo and so forth.  We do add the Docker back in on the titles for SEO.

@moxiegirl ok for the 3rd party driver. I'll file an issue or write it down in a PR,

Thanks @jeanlaurent!

@jeanlaurent @dgageot updated.  See revised screen capture.

LGTM
We will perform all the submitted changes that did not went through in a separate PR.


Move most coverage options to pyproject.toml

This simplifies pytest calling and also makes most coverage settings tox agnostic, so other tools can load them.

I don't think that piling up this many tool options is a single file is in any way maintainable. It's best to use tool-specific configs.

That counts a personal preference, so it should not be a reason to -1, especially as proposed config is an net improvement over current situation.
The benefit of toml file is that it declutters the project root directory. If we would already had the standalone config, I would not have proposed moving it to toml, but w/o one, i found the move appealing. standalone configs are good for tools that have very verbose config, but for those that you configure once and you are quite unlikely to touch after, is better to avoid create new files in repo root. The list of already existing tools we have there is... self explanatory.

The benefit of toml file is that it declutters the project root directory.

This counts as a personal preference. Wanting to make it harder to discover tool settings is not a good enough reason for this PR to exist 🤷♂

If we would already had the standalone config, I would not have proposed moving it to toml

Funny that it's exactly what happened in other devtools projects — you added duplicate configs to pyproject.toml without reviews despite dedicated config being present.
FWIW I've made a case in the past that synchronizing configs from skeleton repos is substantially easier with standalone files. This point still stands.

I do like the reduced complexity of the tox file.

The right place for moving most of those options is pytest.ini. Even pytest-cov settings should mostly go there. Having the underlying tool config influencing the pytest settings implicitly is arguably not the best idea maintenance-wise.

I'd like to propose an idea here, since it's hard sometimes to look into the future to really know what one config file per tool vs. one for everything that can go it......... let's stay the course with the pyproject.toml file, and when the CI/tox/precommit/GHA stuff is all cleaned up one of use put together a PR that breaks it all out.
The benefit here is everyone on the team will be able to "see" the difference, one vs. many.
The reason I like this PR is because it moves config out of the tox.ini that doesn't need to be there, and moves it into a config file the tool uses, independent of the name of that file.


Renders the entire widget whenever there is a state change

I hope this message finds you well. I recently had the opportunity to explore the impressive package you've developed. I am truly amazed by its simplicity and functionality. However, I have some concerns regarding rendering performance. It appears that the package renders the entire widget whenever there is a state change.

You can put a build around the text element for the count and will be a render boundary!

Also extracting count into another widget would work too

Wrapping the Text widget that displays the counter in a Builder seems to solve the issue.
Builder(builder: (context) {
  print('rebuild counter');
  return Text(
    '${counter.watch(context)}',
    style: Theme.of(context).textTheme.headlineMedium!,
  );
}),

Do we really need a special widget, like Consumer in Provider, so we don't have to bother with wrapping it in Builder?


Add DICOM series conversion to NIfTI

The interface should allow configuring the location of dcm2nii or some other conversion program and using it on a DICOM series when needed for an analysis or requested by a user (there should be some kind of button or configuration that creates and keeps a reference for a NIfTI version).

Currently, dcm2niix is used (program location configuration through .env file) and each series directory is created with a 'DICOM' directory by default. If and when the series is converted to NIfTI, another directory is created with the new appropriate format name.


Update to unity engine breaks previous il2cpp compatibility

When loaded through MelonLoader Windows 64bit repo and targeting the <Duel Masters Play's game, changes to certain function signatures seem to break functionality, log file attached.
I have partial translation. The game only seems capable of reading minor entries such as; card races, menu titles and civilizations (colours in MtG terms), but it's no longer able to read my manual card translations.
Latest.log

It might be hard to believe, but the entirety of the Duel Masters english community quite literally hinges on this very important unity mod. Just praying that the issue can be resolved. We thank you for your time.

UPDATE:  Disregard, we've moved from the MelonLoader version to BepInEx and all seems well for the most part.

The only portion of the game that seems to be undetected is the Card Type field, but only on the card side.

Obviously, this is an incredibly minor thing. We're just glad that we can play our game again. Thankfully, since it's the same mod, it'll take the entire manual card library we've worked on over the years just fine ;_;


Added Pokemon Adventure Game

PR Description 📜
Added Pokemon Adventure Game
Fixes #4972

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in GameZone's README.md
[x] I have added README.md in my Game folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.
[x] I have added my Game to gamesData.json file and it is loading properly on main website as intended.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Vaibhav-kesarwani ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@ishita-43 Please review this PR

@kunjgit Please merge the PR

@kunjgit Please merge the PR

You need to resolve the arising merge conflicts before the merge

@ishita-43 The conflict has been resolved please merge the pull request

@ishita-43 The conflict has been resolved please merge the pull request

The merging process will be completed by the admin with 1-2 days

Thank you  @Vaibhav-kesarwani , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

/assign


Pass another location to the Google Matching Engine

Hi,
I'm using a service account to log from the credential stored in GOOGLE_APPLICATION_CREDENTIALS. I have my index deployed on the location europe-west9.
What should I pass to the matchingEngineconfig to use the europe-west9 location. Should I use the param endpoint or apiEndpoint or location or both ? Should I set the deployedIndexId ? And what should endpoint, apiEndpoint, location or deployedIndexId should look like ?
const index =
    'projects/{projectId}/locations/europe-west9/indexes/{indexId}';
  const indexEndpoint =
    'projects/{projectId}//locations/europe-west9/indexEndpoints/{indexEndpointsId}';

  const matchingEngineconfig = {
    index,
    indexEndpoint,
    apiVersion: 'v1beta1',
    docstore: store,
  };

  const googleVertexVectorStore = new MatchingEngine(
    embedder,
    matchingEngineconfig
  );
  const test = await googleVertexVectorStore.similaritySearch('medical');

I use this as the matchingEngineconfig config :
const index = '{indexId}';
  const indexEndpoint = '{indexEndpoint}';

  const matchingEngineconfig = {
    index,
    indexEndpoint,
    apiVersion: 'v1beta1',
    endpoint: '{endpoint}', // The host to connect to for queries and upserts.
    // apiEndpoint: '{apiEndpoint}', // The API endpoint.
    location: '{location}', // The location.
    deployedIndexId: '{deployedIndexId}', // The id for the "deployed index".
    docstore: store,
  };

But I have this credential error :
{
"status": 401,
   "statusText": "Unauthorized",
   "request": {
   "responseURL": "{responseURL}"
  }
  },
  "status": 401,
  "code": 401,
  "errors": [
   {
    "message": "Invalid Credentials",
   "domain": "global",
   "reason": "authError",
    "location": "Authorization",
    "locationType": "header"
   }
  ],
 "attemptNumber": 1,
"retriesLeft": 6
 }

Which is weird since the credential for the service account is in the env variable : GOOGLE_APPLICATION_CREDENTIALS and I can access the google bucket and google vertex modeles.
What should I explore and how should matchingEngineconfig paramters looks like ?

Hey, I have the same problem with the Matching Engine, I get the error of Invalid Credentials, even though I have a service account connected in my .env that should have all necessary roles.


Please update ikoa version to 1.5.2

I've tried to fork the repo and replace the iKOA with a newer version but after deployed the app, it just refused to download. Could you update the version in your repo please.

Done.


close-as-unplanned doesn't reset to correct text after sending a comment

Description
The button should reset to "Close issue" instead of staying "Close with comment":
https://github.com/refined-github/refined-github/assets/44045911/fd57ae8a-97a6-4448-80d8-d3a21c8ccb80
How to replicate the issue + URL
Test on https://github.com/refined-github/sandbox/issues/73
Extension version
24.6.2
Browser(s) used
Chrome

Yeah I'm confused, this is actually a GitHub bug. I thought it would be trivial to fix but this doesn't work for reasons beyond me:
diff --git a/source/features/close-as-unplanned.tsx b/source/features/close-as-unplanned.tsx
--- a/source/features/close-as-unplanned.tsx	(revision 97f6ef6436253dc8fc4a7282c5e3a9cd4a424142)
+++ b/source/features/close-as-unplanned.tsx	(date<PHONE_NUMBER>238)
@@ -39,9 +39,15 @@
 	$(unplannedCheckbox, button.form!)!.checked = button.id === id;
 }

+function restoreCloseButtonText(): void {
+	const spanText = $('.js-comment-and-button .js-form-action-text')!;
+	spanText.textContent = spanText.getAttribute('data-default-action-text')!;
+}
+
 function init(signal: AbortSignal): void {
 	observe('close-reason-selector .select-menu', update, {signal});
 	delegate('[name="comment_and_close"]', 'click', updateCheckbox, {signal});
+	delegate('.js-new-comment-form', 'comment:success', restoreCloseButtonText, {signal});
 }

 void features.add(import.meta.url, {

I will leave this open but feel free to close if it's not worth investigating further.

Let's see

Let's see

Yeah let's not


JAI GigE camera returns multiple devices with identical properties

Describe the Bug
When using a JAI GigE BM-500GE Camera I get two identical devices with identical entry.
And because they are the same I can't differentiate them and get a subsequent error.
Probably me doing something wrong
To Reproduce
Using the demo program in the examples and the JAI Camera
Expected Behavior
Only one device in List or at least not the exact same
Screenshots
from harvesters.core import Harvester
import numpy as np

h = Harvester()
print(h.add_file('C:/Program Files/JAI/SDK/bin/JaiGevTL.cti'))
print("file",h.files)
h.update()
print(len(h.device_info_list))

print(h.device_info_list[0])
print(h.device_info_list[1])

ia = h.create_image_acquirer(serial_number='U501496', vendor='JAI Ltd., Japan')
print ("ia",ia)

ia.remote_device.node_map.Width.value = 1000
ia.remote_device.node_map.Height.value = 1000
ia.remote_device.node_map.PixelFormat.value = 'Mono8'
ia.start_acquisition()

with ia.fetch_buffer() as buffer:
    component = buffer.payload.components[0]
    _1d = component.data
    print('1D: {0}'.format(_1d))
    _2d = component.data.reshape(component.height, component.width)
    print('2D: {0}'.format(_2d))
    print('AVE: {0}, MIN: {1}, MAX: {2}'.format(np.average(_2d), _2d.min(), _2d.max()))

ia.stop_acquisition()
ia.destroy()
h.reset()

Output
file ['C:/Program Files/JAI/SDK/bin/JaiGevTL.cti']
GenTL producer does not implement IFGetParentTL
GenTL producer does not implement DevGetParentIF
GenTL producer does not implement DSGetParentDev
GenTL producer does not implement DSGetNumBufferParts
GenTL producer does not implement DSGetBufferPartInfo
2
(id_='MAC->00-0C-DF-04-04-EB::JAI Ltd., Japan::BM-500GE', vendor='JAI Ltd., Japan', model='BM-500GE', tl_type='GEV', user_defined_name=None, serial_number='U501496', version=None)
(id_='MAC->00-0C-DF-04-04-EB::JAI Ltd., Japan::BM-500GE', vendor='JAI Ltd., Japan', model='BM-500GE', tl_type='GEV', user_defined_name=None, serial_number='U501496', version=None)
Traceback (most recent call last):
  File "C:/Users/Konstantin/PycharmProjects/CTscanner/CameraCapture.py", line 13, in <module>
    ia = h.create_image_acquirer(serial_number='U501496', vendor='JAI Ltd., Japan')
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 3223, in create_image_acquirer
    raise ValueError(
ValueError: You have two or more candidates. You have to pass one or more keys so that a single candidate is specified.

Process finished with exit code 1

Configuration

OS: Win7 64
Python: 3.8.8
Harvester: 1.2.8
GenTL Producer: JAI
Camera: BM-500GE

h.create_image_acquirer(1)

Seams to work for selection now I have to further debug if it works, but still don't know why there are two and only the second one seams to work.

@TecKonstantin Hi, the list consists of entities observed through the driver software. Especially, GigE Vision GenTL Producers cover both a socket driver and a filter driver so I guess you are seeing the same device twice. I recommend you to check which drivers are activated and enable either one if needed. Best regards, Kazunari.

Thanks that's sound reasonable, I will check.
I get related errors with both of them.
I will test it on a clean system etc. if it is ok I might get back to you if I find something new. Thanks!
Traceback (most recent call last):
  File "C:/Users/Konstantin/PycharmProjects/CTscanner/CameraCapture.py", line 27, in <module>
    buffer = ia.fetch_buffer()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2722, in fetch_buffer
    _buffer = Buffer(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1126, in __init__
    self._payload = self._build_payload(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1261, in _build_payload
    payload = PayloadImage(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1437, in __init__
    self._build_component(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1359, in _build_component
    return Component2DImage(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 837, in __init__
    self._data = self._to_np_array(proxy)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 880, in _to_np_array
    self._buffer.raw_buffer, count=int(nr_bytes),
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4572, in _get_raw_buffer
    return _gentl.Buffer__get_raw_buffer(self)
SystemError: \Objects\listobject.c:248: bad argument to internal function

and
Traceback (most recent call last):
  File "C:/Users/Konstantin/PycharmProjects/CTscanner/CameraCapture.py", line 27, in <module>
    buffer = ia.fetch_buffer()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2722, in fetch_buffer
    _buffer = Buffer(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1126, in __init__
    self._payload = self._build_payload(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1261, in _build_payload
    payload = PayloadImage(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1437, in __init__
    self._build_component(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1359, in _build_component
    return Component2DImage(
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 837, in __init__
    self._data = self._to_np_array(proxy)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 880, in _to_np_array
    self._buffer.raw_buffer, count=int(nr_bytes),
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4572, in _get_raw_buffer
    return _gentl.Buffer__get_raw_buffer(self)
IndexError: list index out of range
Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\weakref.py", line 642, in _exitfunc
    f()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\weakref.py", line 566, in __call__
    return info.func(*info.args, **(info.kwargs or {}))
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1923, in destroy
    self.stop_acquisition()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2887, in stop_acquisition
    self._release_buffers()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2947, in _release_buffers
    _ = data_stream.revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 246, in revoke_buffer
    return self._module.revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4090, in revoke_buffer
    ret = self._revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4095, in _revoke_buffer
    return _gentl.DataStream__revoke_buffer(self, py_obj)
_gentl.ErrorException: GenTL exception: Unspecified runtime exception. (Message from the source: Unknown Error) (ID: -1001)
Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\weakref.py", line 642, in _exitfunc
    f()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\weakref.py", line 566, in __call__
    return info.func(*info.args, **(info.kwargs or {}))
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 3402, in _reset
    ia.destroy()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 1925, in destroy
    self._release_data_streams()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2916, in _release_data_streams
    self._release_buffers()
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 2947, in _release_buffers
    _ = data_stream.revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\harvesters\core.py", line 246, in revoke_buffer
    return self._module.revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4090, in revoke_buffer
    ret = self._revoke_buffer(buffer)
  File "C:\ProgramData\Anaconda3\envs\CTScanner\lib\site-packages\genicam\gentl.py", line 4095, in _revoke_buffer
    return _gentl.DataStream__revoke_buffer(self, py_obj)
_gentl.ErrorException: GenTL exception: Unspecified runtime exception. (Message from the source: Unknown Error) (ID: -1001)

Ok sorry, yes it was the filterer driver, I did not execute with the administrator to disable it. But unfortunately it stops at
buffer = ia.fetch_buffer()

and does not proceed, the same thing happens with your GUI program example.


Modifier -> SwiftUI -> Focus: focusedSceneValue(::)

Doc:
https://developer.apple.com/documentation/swiftui/view/focusedscenevalue(_:_:)-57boz

[ ] Swift implementation
[ ] Elixir implementation

https://developer.apple.com/documentation/swiftui/view/focusedscenevalue(_:_:)-5iugx

[ ] Swift implementation
[ ] Elixir implementation

According to the @supernintendo @carson-katri @shadowfacts we won't support this


remove unused JS component from build

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/3852/

Grunt build looks good:
hkernbach@c9:~/Git/devel$ git checkout "bug-fix/remove-jqconsole"
Branch 'bug-fix/remove-jqconsole' set up to track remote branch 'bug-fix/remove-jqconsole' from 'origin'.
Switched to a new branch 'bug-fix/remove-jqconsole'
hkernbach@c9:~/Git/devel$ cd js/apps/system/_admin/aardvark/APP/
hkernbach@c9:~/Git/devel/js/apps/system/_admin/aardvark/APP$ grunt deploy
Running "sass:dev" (sass) task

Running "eslint:target" (eslint) task

Running "replace:scripts" (replace) task

Running "imagemin:dist" (imagemin) task
Minified 32 images (saved 60.1 kB - 31.8%)

Running "concat:css" (concat) task

Running "concat_in_order:libs" (concat_in_order) task
File "frontend/build/libs.min.js" created.

Running "concat_in_order:default" (concat_in_order) task
File "frontend/build/app.js" created.

Running "concat_in_order:htmlStandalone" (concat_in_order) task
File "frontend/build/index.html" created.

Running "cssmin:dist" (cssmin) task
>> 2 files created. 685.81 kB → 560.34 kB

Running "concurrent:uglifyAll" (concurrent) task

    Running "uglify:libs2" (uglify) task
    >> 1 file created 661.18 kB → 349.5 kB

    Done.

    Running "uglify:default1" (uglify) task
    >> 1 file created 877.43 kB → 477.62 kB

    Done.

Running "htmlmin:dist" (htmlmin) task
Minified 1 files

Running "compress:main" (compress) task
>> Compressed 1 file

Running "compress:standaloneCSS" (compress) task
>> Compressed 4 files.

Running "compress:standaloneJSMinified" (compress) task
>> Compressed 2 files.

Running "compress:standaloneJS" (compress) task
>> Compressed 2 files.

Running "compress:aceJS" (compress) task
>> Compressed 1 file

Running "compress:workerJSON" (compress) task
>> Compressed 1 file

Done.


Hub: Flamethrower - Normalized ray casts for all

Remove the shape-specific ray cast and use normalized ray casts for both entities and avatars
https://hifi-content.s3.amazonaws.com/davidback/development/flamethrower/flamethrowerGitTest.json points to latest script changes here

Tested internally, passes QA


Ports not complete?

I have a database of some 3700 ports worldwide (including those in areas not populated). They have a  lot more information (you can see a template on portinfo.co). I would like to contribute, let me know if there's interest in expanding the ports information to include what I suspect is literally every port in the world.

Definitely interested! This contribution would be appreciated very much :)
What is your source for that data set? Can we include it here without copyright issues?

@raducojocaru Just wanted to check back if you're still willing to and able to contribute this?
Thank you!


Update dependencies

Hello team, while working on https://github.com/conda-forge/napari-spatialdata-feedstock/pull/1
We noticed some updates needed for the dependencies
Imported but not in requirements

cycler
dask-core
geopandas
matplotlib-base
napari-matplotlib
packaging
pandas
scikit-learn
scipy
superqt
vispy

Required but not imported

magicgui
scikit-image
squidpy

Could these be updated for the next release?
Cheers 🚀

Hey! yes will take a look.

Thanks for reporting this! I have locked into this (ripgrep and some filtering via emacs) and I have obtained these lists of packages that are imported respectively in napari_spatialdata, in the tests and in the docs. I will update the requirements to match these lists.
napari_spatialdata package
PIL
anndata
click
cycler
dask
datatree
geopandas
loguru
matplotlib
multiscale_spatial_image
napari
napari_matplotlib
napari_plugin_engine
numba
numpy
packaging
pandas
pytestqt
qtpy
scipy
shapely
sklearn
spatial_image
spatialdata
superqt
vispy
xarray

tests (some packages are already imported above, remove)
anndata
click
dask
logging
loguru
matplotlib
multiscale_spatial_image
napari
numpy
pandas
pytest
qtpy
scipy
skimage
spatialdata

docs (some tests are already used in the main package or test, remove)
matplotlib
napari
scanpy
spatialdata
sphinx
squidpy

The lists above still miss some packages, for instance the ones that are used internally by sphinx when building the docs. And also some packages needs to be renamed since I am listing the import name and not the pip name.
There are some differences with respect to your list, for instance squidpy is only used in the docs, so it should not appear in the conda configuration. Also I am using dask and not dask-core, is this ok?
I will test the new requirements (linked PR) and eventually adjust the requirements there.

I made a release with the update dependencies.

Also I am using dask and not dask-core, is this ok?
Hi sorry I did not reply. dask-core is conda forge package that is more or less equialent to dask on pypi, so yeah all good.
Thanks for updating the deps :)


Update array bug, formatting parameters

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
SPTarget returned 'M' instead of 'RTM'. $AllSPs should always be an array (also if it's just one element). Otherwise $AllSPs[$targetSP] goes wrong.
Approach

Commands to test
See examples
Screenshots

Learning

hey fabulous, thanks, @nvarscar does this fix #4584 ?

thanks a ton! one test round passed, and i submittted a PR to fix the unrelated test failure

This does not fix #4584


doc for sparse multiply

write doc for sparse multiply

this was just a checkbox for https://github.com/stan-dev/stan/issues/1565


BUG:Add Home page option to navbar

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The landing page is not available in navbar.Therefore whenever we leave the page we cannot return to landing page.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

pls assign these issue to me under GSSOC'24

@Anushkabh Can you please assign this issue to me and add gssoc label..?


Will firmware upgrade affect the previously calibrated parameters of the camera?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu20.04 LTS

Kernel Version (Linux Only)
5.10

Platform
rk3588s

SDK Version
2.56.0

Language
C++

Segment
others

Issue Description
Does the calibrated D455 need to be recalibrated after upgrading the firmware through rs-fw-update?
I would like to know if upgrading from firmware <IP_ADDRESS> to firmware <IP_ADDRESS> will affect the calibration parameters.

Hi @diplomatist  No, you do not need to recalibrate a RealSense camera after a firmware update.

Hi! @MartyG-RealSense
I have saved the calibration parameters of the same D455 camera in firmware <IP_ADDRESS> and <IP_ADDRESS>:
The calibration parameters for firmware version <IP_ADDRESS> are as follows:
"intrinsic_left.x.x": "0.493718",
"intrinsic_left.x.y": "0.789467",
"intrinsic_right.x.x": "0.495089",
"intrinsic_right.x.y": "0.79162",
After upgrading firmware version <IP_ADDRESS> to version <IP_ADDRESS>, some calibration parameters are as follows:
"intrinsic_left.x.x": "0.494122",
"intrinsic_left.x.y": "0.790112",
"intrinsic_right.x.x": "0.495493",
"intrinsic_right.x.y": "0.792267",
The above 4 parameters are the only calibration parameter differences among different firmware version. Does the camera need to be recalibrated？

No, you do not need to recalibrate.

No, you do not need to recalibrate.

ok，Thanks for your answer！


Please consider standardizing how EGL external platform modules are discovered

App-container technologies like Flatpak and Steam's Steam Linux Runtime need to understand NVIDIA's proprietary EGL driver finds external platforms, so that they can alter the search path as required for the "shape" of their containers, particularly if they are borrowing graphics drivers from the host system rather than shipping their own.
It would be useful if there was either a spec or an open-source reference implementation for how the external platforms are to be discovered, so that we can make sure that what we're doing is compatible.
The external platform modules seem to be designed to be non-NVIDIA-specific, so it would presumably also be helpful to have a spec that other ICD authors can implement.
My best guess at the spec is that it's the same as GLVND EGL ICDs, but with some names changed:

if $__EGL_EXTERNAL_PLATFORM_CONFIG_FILENAMES is set and we are not setuid, search exactly those JSON manifest files in that order, "most important" first;
else if $__EGL_EXTERNAL_PLATFORM_CONFIG_DIRS is set and we are not setuid, read *.json in those directories, "most important" first, with files earlier in lexicographic (strcmp) order treated as "most important" within each directory;
else search the hard-coded path /etc/egl/egl_external_platform.d:/usr/share/egl/egl_external_platform.d, again looking for *.json in lexicographic order;
each *.json file is in exactly the same format as a GLVND ICD JSON manifest, version 1.0, with a top-level object with keys including file_format_version (a string) and ICD (an object with keys including library_path)
the library_path can either be:

a bare filename with no /, interpreted as a SONAME in the system library search path (this is what's used in practice)
an absolute path, possibly containing special tokens like ${LIB}
a relative path, interpreted as relative to the JSON manifest, and possibly containing special tokens like ${LIB}

Is that guess correct?

The Flatpak developers would also find it useful if the loader searched the XDG basedirs, similar to how Vulkan ICDs are located.


Did not work until I imported the package somewhere

Before vite i used webpack, there the config like the following:
settings: { "import/resolver": { webpack: { config: "webpack.config.js" } } }

Now with this package it is like this:
settings: { "import/resolver": "vite" },

This looks a bit strange to me because it is using some kind of magic string there. Also, the short instructions did not mention that I have to import it somewhere like this:
import eslintViteImportResolver from "eslint-import-resolver-vite";

I did so in the .eslintrc.js
I do not know if other people had the same problem, but maybe put this in the README. And maybe think about the configuration again.

a resolver can be referenced in several different ways. See here for docs.
# .eslintrc.yml
settings:
  # uses 'eslint-import-resolver-foo':
  import/resolver: foo

You don't need to import the resolver, eslint-plugin-import will take the resolver name to do its thing. Did you encounter errors when not importing the module?

Yes, it did not work, even after restarting VS Code. Also, when I remove the import, it stops working and eslint can not resolve any modules.

eslintrc.js with commented out import that does not work
// import eslintViteImportResolver from "eslint-import-resolver-vite";

const eslintConfig = {
    env: { node: true },
    parserOptions: {
        ecmaVersion: 2021,
        sourceType: "module",
    },
    plugins: ["html", "json-format", "ordered-imports"],
    extends: ["airbnb-base", "plugin:ordered-imports/recommended", "prettier"],
    rules: {
        camelcase: 0,
        "eslintno-underscore-dangle": 0,
        "import/no-mutable-exports": 0,
        "import/order": 0,
        "import/prefer-default-export": 0,
        "new-cap": 0,
        "no-await-in-loop": 0,
        "no-console": process.env.NODE_ENV === "production" ? 2 : 0,
        "no-continue": 0,
        "no-debugger": process.env.NODE_ENV === "production" ? 2 : 0,
        "object-curly-newline": [
            2,
            {
                ObjectExpression: { minProperties: 2 },
                ObjectPattern: { minProperties: 2 },
                ImportDeclaration: "never",
                ExportDeclaration: { minProperties: 2 },
            },
        ],
        "ordered-imports/ordered-imports": [
            2,
            {
                "group-ordering": [
                    {
                        name: "root",
                        match: "^@/",
                        order: 20,
                    },
                    {
                        name: "current dir",
                        match: "^\\./",
                        order: 30,
                    },
                    {
                        name: "extermal",
                        match: ".*",
                        order: 10,
                    },
                ],
            },
        ],
    },
    settings: { "import/resolver": "vite" },
};

module.exports = eslintConfig;

vite.js with commented out import that does not work
import path from "path";
import * as vite from "vite";
import { VitePluginNode as nodeVitePlugin } from "vite-plugin-node";

const viteConfig = vite.defineConfig({
    resolve: { alias: { "@": path.resolve(__dirname, "src") } },
    plugins: [
        ...nodeVitePlugin({
            adapter: "koa",
            appPath: "./src/app.js",
            exportName: "app",
        }),
    ],
    server: {
        host: "<IP_ADDRESS>",
        port: 4000,
    },
});

export default viteConfig;

can you try renaming your vite.js to vite.config.js? I created issue #2  to accept different vite config file name/path other than the default one. Currently the plugin assumes eslintrc file and vite.config.js file are in the same directory.

Sorry, yes it is already called vite.config.js 😕

just looked at your vite config again, I don't see the eslint plugin. You need to configure vite to use eslint plugin. Try installing vite-plugin-eslint, and in your vite config, do:
import eslintPlugin from "vite-plugin-eslint";

vite.defineConfig({
    plugins: [
        eslintPlugin()
    ],
});

I'll update the README file with these instructions.

Mhh, didn't get it to work correctly. I created a minimal example: https://github.com/juliushuck/issue-eslint-import-resolver-vite Hope you can help me. It does show errors for the imports in index.jsx until I comment out the first line in eslint.config.js

Thanks for the repo, I see where the problem is. I was using some syntax that is supported by Node version >= 14. When I try Node 12, it failed. When I switch to Node 14, I was able to compile and see your "Hello World" page. I'm going to make some update to support earlier Node versions. In the meantime, can you try updating your Node to >= 14 and see if that works for you?

I'm using Node v16.11.1 Are you using 17 or so?

I was able to compile your sample app using Node 14.

I also tried Node 16, it compiled fine.

Hi, compiling is not the problem, the linting in VS Code does not work until I add that import statement. I also tried the new version for that, still no luck.

If the app compiles fine, that means the plugin is working properly. It's the vscode or its eslint plugin settings you should be looking at, not this plugin. I'm posting my set up for your reference. I'll close this issue since you confirmed your Vite is compiling correctly with this plugin.

For anyone else looking for an answer on this, the root of the problem is the use of an inline require() inside of the exports.resolve function:
https://github.com/pzmosquito/eslint-import-resolver-vite/blob/main/index.js#L25
This is required in order to read the Vite config file and read the alias and namedExport options so you don't have to maintain them in two plases. Unfortunately eslint-module-utils now throws an error when using a dynamic require:
Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Users\l0gicgate\Documents\vite-app\vite.config.js from C:\Users\l0gicgate\Documents\vite-app\node_modules\eslint-import-resolver-vite\index.js not supported.
Instead change the require of vite.config.js in C:\Users\l0gicgate\Documents\vite-app\node_modules\eslint-import-resolver-vite\index.js to a dynamic import() which is available in all CommonJS modules.

If you're looking for a quick solution just use the eslint-import-resolver-alias plugin.
The gotcha is that you will have to maintain your import aliases in two places (vite.config.js and eslint config file).

@l0gicgate Thanks for the detailed explanation. Unfortunately the eslint-plugin-import resolver doesn't support dynamic import. This is also the reason why I have to use namedExport for Vite config as a function since Vite's ResolvedConfig interface is async.


[APR-248] chore: add proper support for sample rate in DSD codec

Context
In #271, we detailed the fact that the DogStatsD codec/source currently don't do anything with the sample rate we parse from DogStatsD payloads. This inevitably leads to inconsistencies when payloads have a non-1.0 sample rate.
Solution
We've done a small amount of refactoring to rearrange the codec to collect the raw values, then the sample rate, and then generate the MetricValues by combining the sample rate with the raw values. We've emulated what the Datadog Agent does, which is:

for counts, we just multiple the value by the reciprocal of the sample rate (e.g., a sample rate of 0.25 and value of 10 leads to 40 based on 10 * (1 / 0.25) -> 10 * 4 -> 40)
for timers, histograms, and distribution, we record the value once if the sample rate wasn't specified, or is 1.0 (current behavior), and if it's not 1.0, we record the value n times, where n is the reciprocal of the sample rate (e.g., a sample rate of 0.25 leads to recording the value 4 times (1 / 0.25 -> 4))

Closes #271.

[!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

#279  👈
#278
main

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

Join @tobz and the rest of your teammates on  Graphite

For posterity: this does make the correctness tests pass now, but they aren't wired into CI yet so there's no record otherwise.


dae file of gladlaks is badly scaled

Ingar, do you suggest we make a new one or just scale the current one down a couple of notches? Is it a lot of work to make a new one? Looks like there is an option to scale the current one (in the xacro file), but I haven't got to try it yet since manually scaling it will take an iterative process to get it right.

The scale I have selected for the gladlaks-model in the .xacro right now is arbitrary and not exact. If we want it to be exact we can:

Do some calculations / measurements and use the scale parameter in the .xacro
or 2) Export using an exporter that supports dimensions. .dae-files are basically XML-files, and if you look in the manta_shell.dae you will see a paramater "dimension = centimeter" (or something similar). This is because manta_shell.dae was exported using Assimp which formats the .dae files differently from the one I used on gladlaks.dae (meshlab).

However, to my knowledge, the .dae files (both for gladlaks and the thrusters) are purely visual, the actual collision geometry of the AUV is a sphere with a center in COM, and the thrusterlayout used in the model shouldn't affect the controller. If we do want it to be perfectly exact I suggest exporting the .dae anew using a exporter that supports dimensions.

Alright, nice point about the collision geometry. Perfection is probably not worth it so I will try to do some scaling first. Apart from the size the model looked sick btw :)

Scaling it x0.01 in xacro file worked out great!


Update List box - Make visible checkbox nested instance in Salt Next/Light/Dark

List box, multi-select secondary Listitem nested instance / property (single select works)

Mon 4 Nov - updated and merged


on change Agent Status

Hi Guys,
I'm integrating amazon connect streams to our CRM. I'm looking for a way that when an agent changes its status i.e. from Available to offline or Available to On Break (Custom agent status) to trigger an ajax to sync the status with our CRM. I'm not sure if I'm doing it right but connect.agent seems now working. I'm using amazon-connect-v1.7.5.js and this is my code below
var containerDiv = document.getElementById("container-div-amazon-connect");
var instanceURL = amazonConnectCCPURL + '/ccp#/';
// initialize the streams api
function amazon_connect_init() {
    // initialize the ccp
    connect.core.initCCP(containerDiv, {
        ccpUrl: instanceURL, // REQUIRED
        loginPopup: true, // optional, defaults to `true`
        loginPopupAutoClose: true, // optional, defaults to `false`
        loginOptions: { // optional, if provided opens login in new window
            autoClose: true, // optional, defaults to `false`
            height: 600, // optional, defaults to 578
            width: 400, // optional, defaults to 433
            top: 0, // optional, defaults to 0
            left: 0 // optional, defaults to 0
        },
        region: awsRegion, // REQUIRED for `CHAT`, optional otherwise
        softphone: { // optional, defaults below apply if not provided
            allowFramedSoftphone: true, // optional, defaults to false
            disableRingtone: false, // optional, defaults to false
            ringtoneUrl: "./ringtone.mp3" // optional, defaults to CCP’s default ringtone if a falsy value is set
        },
        pageOptions: { //optional
            enableAudioDeviceSettings: false, //optional, defaults to 'false'
            enablePhoneTypeSettings: true //optional, defaults to 'true'
        },
        ccpAckTimeout: 5000, //optional, defaults to 3000 (ms)
        ccpSynTimeout: 3000, //optional, defaults to 1000 (ms)
        ccpLoadTimeout: 50000 //optional, defaults to 5000 (ms)
    });

    connect.agent(function(agent) {
        agent.onRefresh(function(agent) {
            alert("onRefresh");
            $.ajax({
                type: "POST",
                url: baseUrl + "connect/ajax-crm-status",
                data: {},
                beforeSend: function() {},
                success: function(data) {}
            });
        });

        agent.onStateChange(function(agentStateChange) {
             alert("onStateChange");
            $.ajax({
                type: "POST",
                url: baseUrl + "connect/ajax-crm-status",
                data: {},
                beforeSend: function() {},
                success: function(data) {}
            });
        });
    });
}

Is it possible or I'm using the wrong API?
Thanks in advance

Hi Guys,
I guess I'm wrong here. The "agent.onStateChange" is not for the agent status i.e. Available, Online or custom break but for when an agent profile is updated. Is there any way we can trigger an event when an agent in CCP change its status i.e. from Available to Break.
Thanks in advance

Hello;
We are using agent event streams for this purpose. It is much more complicated but works :)
https://docs.aws.amazon.com/connect/latest/adminguide/agent-event-streams.html
Thanks

Hi @bbolek ,
Thank you so much for the help, ill check it out.

Hi,
Just want to share what I did on this one. I just follow this one https://docs.aws.amazon.com/connect/latest/adminguide/agent-event-streams-enable.html

In your Amazon Connect Instance enable data streaming. Amazon Connect >> Instance >> Data streaming
Enable data streaming, select Kinesis Stream for Contact Trace Records. (This requires the creation of Kinesis Stream)
Create Lambda. Add trigger Kinesis. make sure the lambda permission is allowing reads to Kinesis.
The following events trigger your lambda (LOGIN, LOGOUT, STATE_CHANGE and HEART_BEAT).
In your lambda you can set an endpoint or script to update something.

Hope this helps. Closing ticket


feat: config driven library linking

What I did

fixes: #43
Fixes: APE-324
How I did it
Allow libraries to be listed in your ape-config.yaml file
How to verify it

Define a library and a contract
Deploy library
Put address in ape-config (See README changes to this PR)
Force re-compile
Tada.

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

see simple example here: https://github.com/unparalleled-js/ape-playground/blob/main/tests/conftest.py

Some of the use-case would be simpler if we merge and use this PR: https://github.com/ApeWorX/ape/pull/1292


doc file updated

Signed-off-by: kevin xu<EMAIL_ADDRESS>

Current coverage is 43.52% (diff: 100%)

Merging #239 into master will decrease coverage by 0.45%

@@             master       #239   diff @@
==========================================
  Files            11         11
  Lines          8990       8990
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3954       3913    -41
- Misses         3993       4024    +31
- Partials       1043       1053    +10

Powered by Codecov. Last update 8e7f736...28abac0


[Past consecutive media designs]  needed in the MP Drive

Ryan,
Can you please drop under the media space > 2020 and past designs the designs used?
We want to show what has been used, either allow for new designers to improve that work or create new ones while we work on 2021 media consecutive pushes.
If you could get this completed by the time Monday's Mktg call meets, that would be amazing to enable the discussion and include others in the work. Thanks!

I will organize these files this week making it easier for other people to use and edit them.


Create Blog “vaccine-hesitancy-and-socioeconomic-variables-in-the-us”

Automatically generated by Netlify CMS

👷 Deploy Preview for koytak processing.

Name
Link

🔨 Latest commit
2d90a36336b500c25910e0a98639d1d94a7d6a69

🔍 Latest deploy log
https://app.netlify.com/sites/koytak/deploys/65373d2a4b61e40008e13676


SAUCE enhancements

Exporting with SAUCE can now be loaded into Pablo
Add SAUCE comments support
Fix null padding -> space padding for some SAUCE recs
Trim spaces from SAUCE fields @ load

Merged with NuSkooler's SAUCE enhancements.


How to use tool_locations

From the source code, it is not clear if tool_locations is actually used in slave.pp and how it copes with spaces in tool names. Wouldn't it be better to use a hash rather than a string? Please clarify.

Never mind, it is in the template.


ci, fix: update path that mediator ci workkflow looks for

On reviewing the latest mediator related pr, I noticed the mediator CI workflow wasn't running. This should fix that.

@xprazak2 Thanks!


Fixing imports for python 3 usage

Use relative imports to ensure proper functionality in py2 and py3

@BeenKim if you have a working py3 setup, can you test to ensure the code (or at least the unit tests) work properly in py3 environment?

on it!


Separate pdf parser into its own module

I would love to be able to just use the pdf parser to check if a PDF file is corrupt or not. And I guess the parser alone could be used for a lot of other cool things by others.

:+1:

For now, you have to include set of the JS files (like we do in unit tests, https://github.com/mozilla/pdf.js/blob/master/test/unit/unit_test.html#L13) and access parser code this way. You don't have to use web worker via display/api.js in this case. Main question is: will this API look like we have now internally or shall we repackage it in something more usable (e.g. in node.js)?

The ultimate node.js implementation would be a streaming parser.

pdf.js is a joy to use! But being able to run it on node.js would make it even better!
I've found this project[1] which has built some sort of wrapper to run pdf.js on the server. But having pdf.js packaged with more consideration to server-environments would be great. :boat:
[1] https://github.com/peerlibrary/meteor-pdf.js/blob/master/package.js#L26

@yurydelendik the API looks great, except for the configuration parts.. currently PDFJS relies on properties attached to global PDFJS for configuration.. in node.js this won't work so well.
i think you can export the parser to something like this:
var pdfParser = require('pdf-parser')({ /*...options..*/ });

in other words.. i think the best way to do this is repackage the parser into something more usable that could be re-used for node.js and the browser (pdf.js)

#6683 might be a solution for this.

thnks! i'll follow the discussion there

Is this still an issue now that we have UMD headers everywhere (see the PR mentioned above)?

I think this issue can be closed, since it was mainly concerned with modularisation.
But there are a few more things that could be done to facilitate server-side use of pdf.js, for example this one: https://github.com/mozilla/pdf.js/issues/7798

Closing as fixed.


New URL allowlist doc for Azure CLI installations

Learn Build status updates of commit 4034e32:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:warning:Warning
View
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-endpoints.md

Line 23, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 47, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 67, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 89, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 106, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 128, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 144, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 162, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 178, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 7dfd860:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a14c763:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit eca7a4e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 9df911b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit d471667:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 56f7612:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 58544ab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-endpoints.md
:white_check_mark:Succeeded
View

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Azure CLI fetches extension index from https://aka.ms/azure-cli-extension-index-v1:
https://github.com/Azure/azure-cli/blob/3feea02888ea67f033f407174a3a7a340158b81a/src/azure-cli-core/azure/cli/core/extension/_index.py#L11
DEFAULT_INDEX_URL = "https://aka.ms/azure-cli-extension-index-v1"

which points to https://azcliextensionsync.blob.core.windows.net/index1/index.json, but it is missing from the list.

@Jiasli, So I'm thinking to add this URL to the !Note section currently at the bottom of the doc, but move the !Note to the top.  Did you prefer a different location?

@jiasli, this is PR https://github.com/MicrosoftDocs/azure-docs-cli/pull/3550


feat!: add private key to libp2p components

Description

Add privateKey to Components and use it where possible
Update pubsub, identify, circuit-relay v2, record envelope sealing to use privateKey instead of peerId

Change checklist

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation if necessary (this includes comments as well)
[ ] I have added tests that prove my fix is effective or that my feature works

BREAKING CHANGE:

IdentifyComponents now includes a privateKey
PubSubComponents now includes a privateKey
CircuitRelayServerComponents now includes a privateKey
RecordEnvelope.seal now has a privateKey as the second param rather than a peerId
RecordEnvelope constructor now takes a publicKey

Is there a way we can do this in stages to prevent it causing libp2p v2?  Perhaps just adding the private key in the first instance, then releasing it, then upgrading noise etc to use it?

Blocked waiting for the next breaking release

Superseded by #2660


Fix for more than 50 fields in Postgres

Postgres does not allow functions that have more that have more than 100 arguments. When using the concat function, this limits comparisons to less than 50 fields.
Using || for concat like the Oracle variant fixes this.

Is there anything I can do to help get this merged.  As I am trying to do a comparison of over 50 fields in postgres and without this patch, it crashes, so I am having to use my own fork.

Relates to https://github.com/datafold/data-diff/issues/397

Is there anything I can do to help get this merged. As I am trying to do a comparison of over 50 fields in postgres and without this patch, it crashes, so I am having to use my own fork.

Thanks for opening this @kindly
If you're up for it adding a test case here would speed things along
There's dev setup info here

@dlawin
I have added a test which will fail if you remove the new code.
I had difficulty reproducing the issue in the tests, and only by using the uuid like the other test could I make it fail.  This is because the error caused by the checksumming/segmenting, and I think that is only triggered in some conditions.

I have added a test which will fail if you remove the new code.
I had difficulty reproducing the issue in the tests, and only by using the uuid like the other test could I make it fail. This is because the error caused by the checksumming/segmenting, and I think that is only triggered in some conditions.

appreciate it @kindly I will take a closer look early next week and get it in a release!


Add function for checking if file is present in container

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>This Pull Request adds function for checking if file exists in container.
I would be glad for any kind of review.

Please add the proposed function to conu/apidefs/container.py so we all can agree on the interface.

I have closed the PR by mistake.

But, based on reading backlog and comments in PR, check will be done otherwise.

I would love if you opened a PR just for the API definition -- we can figure out implementation later. I think that having a method to check presence of files inside container (and image) is crucial and is probably out of scope of the File interface.

jenkins/superCI comment

Status: failure, return code = 1
Result url: https://da.gd/3nj8
Comment: This is testing CI


Undefined variables throw an unhelpful exception

Currently if you assign a value to an undefined var you get Unhandled exception. System.Exception: Invalid identifier "[varname]"
This provides no useful info like file or line number.

There have been a lot of improvements to compile-time error handling. Assigning a value to an undefined var is no longer an unhandled exception and gets you a more useful Error at code.dm:18:4: Unknown identifier "[varname]".


hubstaff: change updateScript url to use new RSS feed

Description of changes
I've got an email
Hello,

I'm the Tech Lead of the Client Desktop team at Hubstaff and saw that you are the maintainers of the hubstaff nixOS package!?

We just got a notification from one of our clients. He patched it for his own use and asked about upstreaming it:

https://github.com/allgreed/nixos-config/commit/7402b23e8285b5094e5b3ca2b3845cb2aaf66063

The thing is we don't support nixOS unfortunately (I'm personally a big fan though!).
Any chance you guys can update that? If you need any more information please just let me know!

We also have a RSS feed with the newest releases (just look for the "enclosure" and sparkle:os="linux"):
https://app.hubstaff.com/appcast.xml

The latest linux release is:
https://app.hubstaff.com/download/7473-standard-linux-1-6-24-release

Thanks,
Artur

and updated script
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Great work, thanks a lot! 🙏🏼


Pick a ciphersuite for the Credential

Since the key used to issue the credential needs to be in the DID document for the issuer, the key needs to be a supported key format for the issuer DID Method.
This issue is blocked by https://github.com/decentralized-identity/interop_project/issues/2

Closing, we will track this in #2.


HierarchyRequestError: Node cannot be inserted at the specified point in the hierarchy toolbar.js:14:0

Please check this error.
I have just install in CE 1.9.1 and this error show in console & debug toolbar not showing.

Something wrong with browsers.
I try on firefox, chrome, ie. Debug panel are not show. When i open developer tool and change to IE 10. It 's suddenly show and seem work great. Could you check it?


RFC: Configurable debugExchange

Hey folks!
We're running URQL on both Server & Client, and we'd like to be able to use pino for server logging. I haven't really spent much time on this yet, as I'd like to understand whether you'd be interested in this, and if this is the right approach. Would love to hear your thoughts.
This is a breaking change, as it changes the signature of debugExchange to be an exchange factory, rather than an exchange.
New API:
interface DebugExchangeOptions {
  // default: console.log
  logFn?: (...args) => void;
  logFormat?: (message, data) => void;
}

type CreateDebugExchange = (options: DebugExchangeOptions) => Exchange;

Thanks! ❤
edit: just to clarify, this is absolutely not mergeable in current shape, this is just to have a discussion around it.
Todo

[ ] Try out in current project
[ ] Rename to logExchange
[ ] Configure logExchange to debugExchange

Yeah I think it might make sense to rename this to logExchange, as it changes its use-case to logging on production, where debug kind of suggests it's for local development (for me anyway). Thoughts?

If we do that, debugExchange could just preconfigure logExchange 🤔

I think that’s a good shout!

So, I can either implement changes as is, or I can develop it in our application and submit it back upstream when we've figured out whether this definitely fits all our needs. If you're not in a rush I'd maybe suggest the latter.

We’re definitely not in a rush 🙌 so do feel free to take your time ✨

🙇

What if we do this:

Create a new logExchange package with the above API
Migrate debugExchange to it using defaults
In the next major version do a breaking change to debugExchange to accept options

@thisismissem im just gonna define it in line in our project for now rather than going through the whole package infra hoopla right now.

@StevenLangbroek I guess we should close this then & create a ticket in our backlog to get back to contributing this unless someone beats us to it?

Closing for now due to inactivity :)


Use map styles in lists and detail views

Codecov Report

Merging #208 (8a051c6) into master (072e126) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8a051c6 differs from pull request most recent head d0e994b. Consider uploading reports for the commit d0e994b to get more accurate results

@@           Coverage Diff           @@
##           master     #208   +/-   ##
=======================================
  Coverage   86.64%   86.64%
=======================================
  Files          27       27
  Lines        2127     2127
=======================================
  Hits         1843     1843
  Misses        284      284

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 072e126...d0e994b. Read the comment docs.


Remove elements and convert to onEvent

We're moving away from

exporting elements as these are PostHog custom encoding of the DOM, which doesn't much make sense outside without our queries/re-encoding. It's also expensive to handle and send.
exportEvents function and buffer - instead we'll either have batch exports (hourly or daily) or webhooks style where each event is sent directly. I'm assuming Rudderstack belongs in the webhooks category, so you get the data immediately.

Note haven't tested this yet - wanted to confirm the changes make sense first and then will set-up local testing.

cc @gauravudia & @utsabc
Does (1) and (2) make sense to you?

For more context we most of the times send a single event at a time anyway per logging data http://metabase-prod-us/question/267-rudderstack-how-many-events-sent-together

As you can see from above we mostly send single events anyway. It will increase the request count < 25%  and we're not their biggest users by far. I have also tagged and not heard back, so I'm assuming it's fine.