HSEARCH-3776 + HSEARCH-3777 Restore advanced configuration options of the Lucene Writer

HSEARCH-3776: Restore advanced configuration options of the Lucene Writer
HSEARCH-3777: Restore configuration of the Similarity in the Lucene backend

Applied your suggestion and merged. Thanks!


About VOT setup issue (#2)

The former issue has been created here: (https://github.com/xl-sr/THOR/issues/6) and has been closed by the repo collaborator.
The original code, which uses the frame number (f) that is not provided to tracker's init function according to VOT official toolkit, is pasted as follows
        # initialize the long term module
        if not f or self._cfg.vanilla:
            self.lt_module = LT_Module(K=self._cfg.K_lt, template_keys=self.template_keys,
                                       lb=self._cfg.lb, lb_type=self._cfg.lb_type,
                                       verbose=self._cfg.verbose, viz=self._cfg.viz)
            self.lt_module.fill(temp)
        else:
            # reinitialize long term only at the beginning of the episode
            self.lt_module.update(temp, div_scale=0)

With this version of code, I successfully reproduced the reported result(EAO=0.416).
{'acc': 0.5818219747855319, 'robustness': 0.2341262408690766, 'eao': 0.4160014947564685, 'mean_fps': 94.913035446227}

However, as I commented the conditional branch which uses f:
        # initialize the long term module
        # if not f or self._cfg.vanilla:
        self.lt_module = LT_Module(K=self._cfg.K_lt, template_keys=self.template_keys,
                                   lb=self._cfg.lb, lb_type=self._cfg.lb_type,
                                   verbose=self._cfg.verbose, viz=self._cfg.viz)
        self.lt_module.fill(temp)
        # else:
            # reinitialize long term only at the beginning of the episode
            # self.lt_module.update(temp, div_scale=0)

, I got this result:
{'acc': 0.5797615533198762, 'robustness': 0.28563401386027343, 'eao': 0.368673347545505, 'mean_fps':<PHONE_NUMBER>1562767}

Honestly, with an EAO of 0.369, the method did bring improvement w.r.t. to SiamRPN baseline(EAO=0.322), but not as much as reported while being removed the conditional branch that uses the frame number information. The latter indicates if an initialization is due to the beginning of new test video or a reinitialization after a drift. This information should not been provided to trackers according to VOT official toolkit.
I wish that a reasonable justification can be provided w.r.t. to this question before this issue is closed. Thanks.

The reason was already given in #6, resetting templates during a sequence destroys the purpose of a long-term module. In the evaluation procedure of VOT, the resetting is done by giving the ground truth position to the tracker for reinitialization, then the tracking is continued. However, it is not stated that the tracker also needs to reset its template. I agree that the current implementation is potentially confusing, in a future commit the current structure will be adapted to make it more conform to the standard VOT API.
Obviously, you cannot just comment out parts of the code and expect it to still reproduce reported results. The hyperparameters for THOR were searched for this setting. If you want to run VOT with a full reset after every drift (which makes the task easier, not harder) you can use the following hyperparameters:
 {
    "tracker": {
      "penalty_k": 0.078722,
      "window_influence": 0.318922,
      "lr": 0.448255
    },
    "THOR": {
      "K_st": 6,
      "K_lt": 4,
      "iou_tresh": 0.91256,
      "lb": 0.841342,
      "tukey_alpha": 0.881996,
      "lb_type": "dynamic",
      "modulate": true,
      "dilation": 10,
      "context_temp": 0.5,
      "viz": false,
      "verbose": false,
      "vanilla": false
    }
}

This yields the following performance:
{'acc': 0.5640699155887349, 'robustness': 0.21071361678216896,
 'eao': 0.4174362534678864, 'mean_fps':  40.30484538793514}

All right thanks for your reply.
However, among the released part of code, I didn't find the code for the hyper-parameter search. Would you mind specify with which technique you performed the hyper-parameter search?


fix(preview): disallow folders with illegal characters

Fix #266

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

GitHub release
v2.0.0

Your semantic-release bot :package::rocket:


WIP: bigquery dataset

Start to create bigquery terratest.
This is for dataset.
Will be looking to extend for tables, and queries.  Lots of testing to do!
Please advise on this initial bit, so I can get standards/style/workflows down.

hi @brucearctor,
Thanks for your PR! In general, I'd suggest contributing methods to Terratest that help with validating the deployed infrastructure. In your specific case it would make sense to keep methods like AssertDatasetExists and AssertDatasetExistsE, but remove anything related to creating and deleting datasets. The reason for this is that these resources exist in Terraform, so you should be invoking Terraform modules/examples to create those resources instead. For more information, please read our Terratest Contribution Guidelines as they go into a lot more detail of what I'm trying to describe. I'll leave this PR open for now and mark it as a work in progress.
Footnote: I can appreciate we have 'create' methods for resources like compute instances and storage buckets, but in general we want to keep these to a minimum. Instead, you could contribute an integration test or reference a separate module such as terraform-google-bigquery that creates the necessary resources.

@robmorgan -- sounds good.  Please bear with me, as I'm seeking clarity and approaches to be consistent within GCP.  Assuming all goes well, I imagine writing a ton of functionality for GCP (esp. around the data services).  So, starting small, and trying to be in line with examples and continue to build in complexity.  The module callout is a reasonable one, so would like to arrive at a plan we're happy for that.  Concrete questions --

Examples!
Ah, so you want additional examples contributed in the repo as well!  I had imagined not cluttering things, since thought anybody competent and writing terraform (that they want to test) would be able to lookup the docs for that terraform (or their own modules based on the core terraform).  But, that's straightforward enough to add.  So, I'll get on that.  Do advise whatever else.  If I understand correctly, you want examples in https://github.com/gruntwork-io/terratest/tree/master/examples/terraform-gcp-example and a test in: https://github.com/gruntwork-io/terratest/tree/master/test/gcp correct?

GCP docs for #developing-terratest
https://terratest.gruntwork.io/docs/community/contributing/#developing-terratest gives examples for AWS.  Should I be updating docs ( https://github.com/gruntwork-io/terratest/blob/0c883d74782fecaf8e5987a760f5aa9e06a04a7f/docs/_docs/04_community/contributing.md ) to include guides to contributing to GCP as well (ex: mention service accounts, gcloud auth activate service-account ... or how to get to run from envvars after gcloud init).  I didn't see reference to this in the repo anywhere?

"Helper modules"?

Certainly the terraform exists for Creating and Deleting Datasets, if not it would make no sense to have something that tries to assert the existence.  But, per the footnote, how are we distinguishing the sort of 'helper' functions (Create/Delete) from the 'core' Assertions?  To me, a dataset approx = a storage bucket.  And a table often would be equivalent to a folder structure within (if I'm using HIVE partitioning, instead of BQ).
Therefore, would we also want to move the create/delete of cloud-storage (that's where I pulled the example/tests from, can also refactor for other things - ex: you mention compute engine) into separate module(s) as well?  Otherwise, I would at least be seeking some sort of rubric for how to distinguish when those functions (Create/Delete) are sensible to include within more 'core', vs not.  Happy to create additional modules like terraform-google-bigquery (if I understand what you're suggesting).  Does that mean I should look to refactor out those equivalent functions for terraform-google-cloud-storage (if that helps with consistency, ex: https://github.com/gruntwork-io/terratest/blob/master/modules/gcp/storage.go#L23).  Where would we want terraform-google-bigquery to exist?  https://github.com/gruntwork-io/terratest/modules/gcp/terraform-google-bigquery/ or elsewhere?  Perhaps I am overthinking it?  I have no reason to actually want the create/delete.  I just added it since was following the storage example.  So, can just as easily remove.  Do advise -

Thanks for bearing with that long message!

Ah, and perhaps the terraform-google-bigquery module you were suggesting, is something like: https://github.com/gruntwork-io/terraform-google-gke  or https://github.com/gruntwork-io/terraform-google-sql .  I hadn't caught those, since was only focused on the terratest repo 🤦 .

@robmorgan : If a repo/module for terraform-google-bigquery is to be like https://github.com/gruntwork-io/terraform-google-sql -- what would the mechanism for contributing that be?  I think you would need to create it, so that I could write a PR to start merging things in.  Or where would you want those contributions ahead of that repo existing?

@brucearctor sorry for the delay, hoping to properly respond soon!

@robmorgan  -- looking for guidance on where you want the modifications/improvements to live, if getting thorough bigquery&more-gcp terra-testing is worth having [ I'm taking the time to focus on some shared/open-source things these days, hence the prompt :-), happy to spend cycles working in these directions ].
I did see on: https://terratest.gruntwork.io/docs/community/contributing/ how helper functions are defined, which largely makes sense.
Creating infrastructure: we try to keep helper functions that create infrastructure (e.g., use the AWS SDK to create an S3 bucket or EC2 instance) to a minimum ...

Hi @brucearctor @robmorgan - is this still going to be looked into?

I left alone since the community/etc here didn't seem to support.  Happy to continue to help address, but would be seeking a welcoming community.  Perhaps here [ or elsewhere ], would be helpful to understand what your aims are - to help ensure developed solutions might actually help/address :-)  Let's also see what terratest team has to say.


Format the repo

This PR formats the repo (pnpm format).
The rationale is that when I create a PR to this repo and want to format the code, I would run pnpm format but because the command fixes all the files in the repo, the PR include will include changes to file I have not updated (that is all the files formatted in this PR).
After the PR, running pnpm format will not add noise to PRs.
Note: I have added a few exclusions in .prettierignore as I don't think it makes sense to format those json files.
/cc @ToriLindsay

rebased.

@vicb Kian approved this so it's good to merge when you're ready.

Thanks @ToriLindsay and @KianNH 🙏

I had a lot of this in https://github.com/cloudflare/cloudflare-docs/pull/17262, but looks like this one was merged first. I've updated my PR to reflect this.


Widgets Matching Tailwind

Is your feature request related to a problem? Please describe.
I am using this plugin in conjunction with the liquid_ui package, but I feel like this could be both.
Describe the solution you'd like
What I get from the liquid UI are the Buttons, Forms, Badges, Expansion Tiles, etc.
Since this package is inspired by tailwind, maybe we could add some of these widgets matching the tailwind style.
I and others who use this plugin will be ready to help I'm sure .

The purpose of this lib is to provide useful extensions or widgets which doesn't exist and also to reduce the repetitive work which needs to be done. Premade widgets that can be easily done with existing widgets are not something we are targeting as of now.


Fix multiple printed "Connected to Tessel" lines from same Tessel. Fixes gh-1214

Signed-off-by: Rick Waldron<EMAIL_ADDRESS>

Coverage increased (+1.02%) to 79.441% when pulling 3e98d1dab9df5e00f0233ea57cf44b5527396bec on 1214 into 208842091d8f80ddfe6910507e3349df43389a55 on master.

Coverage increased (+0.1%) to 78.563% when pulling 3e98d1dab9df5e00f0233ea57cf44b5527396bec on 1214 into 208842091d8f80ddfe6910507e3349df43389a55 on master.

Coverage increased (+0.1%) to 78.533% when pulling 3e98d1dab9df5e00f0233ea57cf44b5527396bec on 1214 into 208842091d8f80ddfe6910507e3349df43389a55 on master.


OneKey RTD Module & OneKey Id Submodule: initial release

🏷 Type of documentation

[x] new feature

📋 Checklist

[x] Related pull requests in prebid.js or server are linked

User Id sub-module: https://github.com/prebid/Prebid.js/pull/8685
RTD module: https://github.com/prebid/Prebid.js/pull/8686

@RomainLofaso Please see patrick's comments.

Hi @RomainLofaso, just a reminder that this is waiting on some changes before we can merge.

Hi,
Thank you for your feedback. I was waiting for clarifications about the new locations before updating. I have just updated it accordingly.

Hi,
Could this brand be merged?
Thanks


Flaky test: TestRunPidHostWithChildIsKillable (on ARM, Power?)

https://jenkins.dockerproject.org/job/Docker-PRs-arm/284/console
16:01:42 FAIL: docker_cli_run_test.go:2999: DockerSuite.TestRunPidHostWithChildIsKillable
16:01:42
16:01:42 docker_cli_run_test.go:3018:
16:01:42     c.Fatal("Kill container timed out")
16:01:42 ... Error: Kill container timed out
16:01:42

Noticed on https://github.com/docker/docker/pull/24835#issuecomment-234055539

looks to be caused by #24648 which also re-vendored in containerd.
@mlaventure looks like the test was failing at one point on that pr, you have any idea what's going on / how you fixed it?

also thanks for opening @thaJeztah 😄

It was failing on my PR because with the changed it introduces all the container processes where not killed.
I'll try to reproduce it on master now that my PR is merged, I thought I had fixed that issue since I couldn't reproduce it on my PR.
The fact that it's on ARM suggest it may be exposing a race I didn't see.


Feature request: Force polling watcher

I am developing a small tool using this code. to run in a container on Linux that watches a mounted volume.
However, I am developing this on a Windows machine, and a folder mounted from Windows into WSL (where the Docker daemon and its containers are hosted) does not support inotify; see https://github.com/microsoft/WSL/issues/4739
I am guessing that the pollwatcher would work correctly here, since I confirmed the relevant file is present in the container, there's just no inotify event. https://github.com/gorakhargosh/watchdog/issues/877 is the same use-case, and the fix there was switching from the inotify implementation to a directory-polling implementation. That project also notes that you must force polling for CIFS filesystems but not which platform/watcher implementation CIFS doesn't work with. (Perhaps it's all of them?) So this might also benefit any users trying to watch CIFS filesystems.
Unless I've overlooked something, I don't see an existing way to do the same override here.
In the meantime, I'm just testing with directories mounted from the WSL drive rather than the Windows drive, so this isn't a blocking issue for me right now.
Perhaps the APIs that take a debug flag for Rust could also take a force_polling flag or something like that to pass into RustNotify's py_new?

Happy to accept a pr if you can get force_polling to work.
No idea how hard it would be.

Should be done in #135, just needs some tests.
Can you let me know if that fixes your problem?

I had a quick burl at getting a build, but it's failing on CI (test run fails after wheel build) so I didn't get a wheel to test. I haven't looked into why yet, I'll have another look on Monday.

Tests are fine, just we don't build the binaries on pr commits because it takes an hour.
You'll need to compile the extension yourself, the error is because you haven't compiled the extension.
Or I could release a bit more CO2 AND and build binaries on all commits - it would help in a few places.

The error came from the project's CI script. I forked your PR and added a commit to run the builds on my branch, and that's where it failed.

Yup, I know what that is, I'll fix it.

Should be fixed on the PR.

I grabbed the watchfiles-0.0.dev0-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64 from the last run of #135 and tried it with force_polling=True, and it indeed picks up changes from the Windows folder mounted into the Linux container. (Without force_polling=True, it still does not see the changes, as expected).
So thank you, my limited testing says the issue is fixed by #135.

Any volunteers for https://github.com/samuelcolvin/watchfiles/issues/187?


Use client.ObjectKeyFromObject() intead of client.ObjectKey{}

Description:
Resolves #

Codecov Report

Merging #47 (1107a7d) into master (49a4186) will increase coverage by 0.60%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
+ Coverage   82.40%   83.01%   +0.60%
==========================================
  Files          12       12
  Lines        1148     1148
==========================================
+ Hits          946      953       +7
+ Misses        150      145       -5
+ Partials       52       50       -2

Impacted Files
Coverage Δ

pkg/api/v1alpha1/pod_resource_injector.go
79.31% <100.00%> (ø)

pkg/controllers/machinenodepool_controller.go
81.66% <0.00%> (+2.33%)
:arrow_up:

Continue to review full report at Codecov.

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


Daemon crashes if first ran as root, then as regular user on Linux

You may get the following exception if you first dotnet run the daemon as root, but then later run it as a regular user:
**Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object.
at Stratis.Bitcoin.Consensus.DBreezeCoinView.**b__3_0() in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/Consensus/CoinViews/DBreezeCoinView.cs:line 32
at Stratis.Bitcoin.DBreezeSingleThreadSession.<>c__DisplayClass10_0.b__0() in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/DBreezeSingleThreadSession.cs:line 127
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
at Stratis.Bitcoin.Consensus.DBreezeCoinView..ctor(Network network, String folder) in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/Consensus/CoinViews/DBreezeCoinView.cs:line 25
at Stratis.Bitcoin.Builder.FullNodeBuilderExtensions.<>c__DisplayClass2_0.b__0(IServiceCollection service) in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/Builder/FullNodeBuilderExtensions.cs:line 48
at Stratis.Bitcoin.Builder.FullNodeBuilder.BuildServices() in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/Builder/FullNodeBuilder.cs:line 153
at Stratis.Bitcoin.Builder.FullNodeBuilder.Build() in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.Bitcoin/Builder/FullNodeBuilder.cs:line 126
at Stratis.BitcoinD.Program.Main(String[] args) in /home/auxon0/sources/StratisBitcoinFullNode/Stratis.BitcoinD/Program.cs:line 24
To workaround this issue, a user may simply use "sudo dotnet run" to launch the daemon.
A better error message explaining that access denied is recommended for a fix, and a documentation update explaining the issue and workaround.

I have made a fix for this ... submitting a PR.

Due to changes in the upstream master branch, I'll be re-jigging the patch before submitting a PR.

Added a check on the data folders to Program.cs.  If access is denied, the following is logged:
crit: Configuration[0]
Access to the path '/home/auxon0/.stratisbitcoin/Main/blocks' was denied.

Fixed.


Change _id with insert op, not update

Similar to #63, this Update function will likely need to be changed:
https://github.com/cloudpipe/cloudpipe/blob/f87bc835490e18914b8dc996a38941615b8a56c5/storage.go#L208

Yup, after #63 is fixed this is the next one on the chopping block.
INFO[7179] Launching a job.                              account=cloudpi jid=2
DEBU[7179] Created the job's container: ok               account=cloudpi jid=2
ERRO[7179] Unable to update the job's start timestamp and container id.  account=cloudpi err=exception: Mod on _id not allowed jid=2

Closing. Upgrading to MongoDB >= 2.6 fixed this.


No matching export in @unhead/vue

Describe the bug
when using command slidev slide.md, there are some errors. One of the errors is like:
[ERROR] No matching export in "../../../../opt/homebrew/lib/node_modules/<EMAIL_ADDRESS>for import "useBodyAttrs"

There are also some similar errors, like "useHtmlAttrs", "useServerBodyAttrs", "useServerHtmlAttrs", "useServerTagBase", "useServerTagLink", "useServerTagMeta", "useServerTagMetaFlat", "useServerTagNoscript", "useServerTagScript", "useServerTagStyle", "useServerTagTitle", "useServerTitleTemplate", "useTagBase", "useTagLink", "useTagMeta", "useTagMetaFlat", "useTagNoscript", "useTagScript", "useTagStyle", "useTagTitle", "useTitleTemplate".
To Reproduce
Steps to reproduce the behavior:

install slidev globally
run slidev slide.md, and slide.md is an empty file
see the errors.

Desktop (please complete the following information):

OS: MacOS 13.4.1
Browser: Chrome
Slidev version: 0.42.7

It's probably because @unhead/vue has updated today, and there are breaking changes in this version.
also see Breaking Changes in Release v1.3.0 · unjs/unhead.

Works again, thanks!


FB-1895: Implement DateTimeFromParts

this uses the tiem.date() function

Well, I have to add limits to the time ranges. For example, dax expects the year range [0 , 9999]. I'll update the code and re request to review.

Looks good to me.


Added a test for the recurrent income interactor with 100% coverage

Hey @xretr0, good work on the tests! They cover a lot of important scenarios. One thought I had after designing the test for my use case's interactor: adding assertions on EnterRecurrentIncomeOutputData fields (like specific error messages) could make it clearer what the interactor is returning in different cases. Also, testing edge cases such as when a user isn't logged in or when the inputData has null or invalid fields could help ensure the interactor handles unexpected inputs. Great job overall!


Managed Identity Wallet Release 23.12 Security Acceptance Criteria

Release Security 23.12
Source in Catena-X Confluence and Expert Contacts here(Source only accessible for Catena-X Consortia members in current transition phase).

[x] Threat Modelling Analysis results
Analysis completed (operations excluded):

List of risks generated or updated, rated & actions defined
Risks accepted or mitigation actions implemented and tested
no high threats acceptable

Artifact Repository:

risk register (decentral on Catena-X confluence)

Prime Contacts:

Security Team: SEC0

[ ] Static Application Security Testing (SAST)

code must be scanned weekly with Veracode tool
medium risks require mitigation statement
high and above not accepted

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Artifact Repository:

Veracode UI
(+ GitHub Action)

Prime Contacts:

Security Team: SEC1

[ ] Dynamic Application Security Testing (DAST)
incl API testing (if applicable)

all findings assessed
high & very high findings mitigated
evidence by re-scan

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Artifact Repository:

INVICTI tool

Prime Contacts:

Security Team: SEC3 SEC4

[ ] Secret scanning
Scan executed centrally by SEC team and ZERO valid findings
Artifact Repository:

Veracode or alternative tool
GitHub Secret Scanning
GitGuardian

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Prime Contact:

Security Team: SEC1

[ ] Software Composition Analysis (SCA)
Dependencies must be scanned with Veracode tool with regards to vulnerability

high and above not accepted
FOSS whitelist policy has to be passed

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Artifact Repository:

Veracode UI
(& GitHub Action)

Prime Contacts:

Security Team: SEC1

[ ] Container Scan conducted
All containers in GitHub Packages must be scanned

High / Critical findings not accepted

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Artifact Repository:

Trivy
via nightly GitHub Action

Prime Contacts:

Security Team: SEC2

[ ] Infrastructure as Code
IaC code must be scanned.

Error findings not accepted

Best Practise:

Confirm relevant repository as early as possible to SEC team to enable regular, automated scans. Evidence required for Gate approval.

Artifact Repository:

KICS or alternative tool
via nightly GitHub Action

Prime Contacts:

Security Team: SEC2

Dynamic Application Security Testing (DAST)
OWASP ZAP scan assessed by @PiotrStys with following result:

Container Scan
daily scan with trivy
Infrastructure as Code
result KICS scan
@scherersebastian could you please confirm? Thanks!

@OSchlienz I no longer perform QG checks

@kelaja Security assessment/threat modeling was performed on 2023-11-21 without any high or above findings.

SCA not passed due to two high findings

Hi,
there are no open GitGuardian findings:

-> Secret scanning approved
Best regards
Daniel

KICS / IaC passed.
Please check container scan findings

kindly check:

[ ] SCA
[ ] SAST
[ ] DAST

@OSchlienz : Hey Oliver,
regarding the DAST scan that you performed using OWASP ZAP, have you added Authentication check during the scan?

Hi @kelaja, SCA is still not passed. High findings aren't mitigated.

@OSchlienz : Hey Oliver, regarding the DAST scan that you performed using OWASP ZAP, have you added Authentication check during the scan?

Hi @RoKrish14 , the scan was performed by @PiotrStys

Hi All,
High findings in SCA are mitigated, so SCA is passed

@PiotrStys kindly comment on SAST and DAST

Hi,
there are still no open GitGuardian findings.
-> Secret scanning passed
Best regards
Daniel

@kelaja :
SAST and SCA: PASS

IaC passed
Container / Trivy passed

@kelaja everything seems to be done. Pls. close


fix: retry when conflict

close #5442
case: removeDir may conflict on child node (dirStat)

The same problem: https://github.com/juicedata/juicefs/actions/runs/12497648861/job/34870503833

passed.


chore(perf): Refactor metrics and update the list of collected metrics

Description
This PR is refactoring kube-burner metrics file.
Metrics are split into several parts:

base OCP metrics file. Taken from: https://github.com/kube-burner/kube-burner-ocp/blob/fd4c2d7c249bb0b066ff05139bd87f8b9daa4bd5/cmd/config/metrics-aggregated.yml
base ACS - OCP capture openshift-.* namespaces, but we need stackrox namespace for our tests.
each component has a dedicated metrics file. They are collected from /metrics endpoints of each component

All sections are combined into a single file that can be used to collect all metrics with kube-burner.
Since metrics are pulled outside of the test template, we need to copy it before running the test. That's why the script is adjusted.
Considerations

I had in past issues with .*_graphql_.* metrics because they had large labels. Full GraphQL query can be long. I would leave these metrics in because they are important. If we notice problems with importing them, we can remove them.
Collector metrics can be large. If we see problems with them, we can remove them.

P.S. I have two small scripts to create queries from /metrics and join everything. If any reviewer finds that important, I can provide them in a separate PR in the utilities directory. For now, I would like to keep this PR as slim as possible.
User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[x] CI results are inspected

How I validated my change
I didn't do much validation. I have compared the difference between new metrics and old ones. I have noticed removals of bleve and rocksdb metrics.

We should add a dedicated ACS metrics file to combine with OCP metrics in our tests. That will keep OCP metrics maintained better.


Remove linkify_type

This supersedes #45, right?

It's a PR to the linkification branch.

Aha. That's hard to notice!


PutObject tagging

Confirm by changing [ ] to [x] below:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Platform/OS/Hardware/Device
Win7/Linux
Describe the question
PutObject supports tagging:

x-amz-tagging
The tag-set for the object. The tag-set must be encoded as URL Query parameters. (For example, "Key1=Value1")

and AWS C++ SDK has related function on PutObjectRequest:
    /**
     *
The tag-set for the object. The tag-set must be encoded as URL Query
     * parameters. (For example, "Key1=Value1")

     */
    inline void SetTagging(const Aws::String& value) { m_taggingHasBeenSet = true; m_tagging = value; }

Is there an easy way to convert given TagSet into a string? For example, I am not really sure how to translate it into a string so that it conforms "URL Query parameters" requirement.

nevermind... seems like percent-encoding every key and value separately works.


Super user (root user) that can access everything

I was thinking about adding a @superuser directive which works just like @user but it creates a root user for every model annotated with it where the value of the password field (or any field in that model annotated with @secretCredential) is the ROOT_SECRET environment variable.
Example:
@superuser
@1 model Admin {
  @1 username: String @primary @publicCredential
  @2 password: String @secretCredential
}

This will create a user model called Admin and you'll be able to login as root if you know the value of ROOT_SECRET environment variable:
mutation {
  Admin {
    login(username: "root", password: "<VALUE OF ROOT_SECRET>")
  }
}

and this mutation will return a JWT that allows you to do everything an Admin can do. Of course you'd have to define the permissions of the Admin role:
role Admin {
  allow ALL Any
}

Note: The Any syntax is not yet implemented but should be avialable soon. Checkout #64

Or maybe we can have this as the default permission for @superuser models, and you can then override it if you want.

A simpler solution would be adding a command like pragma root-token <secret> and this will generate a JWT of this shape when decoded:
{
  "role": "__root__",
  "userId": "__root__"
}

And this can be used to do anything.
This makes sense because if you know the secret with which the app is deployed, you should be able to have access to everything.


kernel: Use k_ticks_t for ticks to avoid conversions

There are several types been used to hold ticks, for example, int, int64_t, int32_t, .... though ticks is supposed to be either uint32_t or int64_t.  Depending on the conversion that is done it can lead to bugs. If we don't use the proper type internally the behavior described in  https://docs.zephyrproject.org/latest/kconfig.html#CONFIG_TIMEOUT_64BIT may not be honored due internal conversions to types with lower precision.

@andyross  can you please check if that makes sense ? There are more things to change to avoid / reduce conversions. If it makes sense I'll proceed with these changes.

Removed the hot path for now. Most changes are concentrated in the PM path now.

@peter-mitsis  can you review it again please ?


Not working

Im using the 0.17.24. I have been able to get some sites linked with yarm, but something happened and for the past few updates its not working at all. I cannot add any new sites.

Hi, @Equitime77, thanks for the interest!
It might be that you're used to the old way of marking ores, with a single click (exactly one ore at a time). The new tool, that doesn't require a crafted item nor a research, is used more like the blueprint tool: drag-select a region containing one or more of the target ores. YARM will still expand the search as much as possible thereafter, as it always has.
I've updated the readme and there is also now a change log that will allow reviewing changes in behavior when they occur. I hope this will help reduce future confusion, and allow you to continue enjoying YARM.
Please reopen this issue if it's still not working for you; if that happens, I'd also appreciate a copy of the save where this occurs, so I can check for mod interactions.


Timeout

After some time the bot crashes and says something about a timeout

"says something about a timeout"
Don't expect help if you don't supply us with actual info about the crash, like a log or printscreen/screenshot of the console

Yeah, sorry closed the cmd to soon, forgot to screenshot. Haven't let it run for that long since. I will provide it as soon as i get it again


Homebrew Formula for LightGBM

Upstream issue: https://github.com/Homebrew/homebrew-core/pull/43244.
@ankane Thanks a lot for your efforts!
Please ping us if any help is needed from our side.
Want to hear opinion of macOS users @jameslamb @Laurae2 @henry0312.

Hey @StrikerRUS, the formula was just accepted, so Mac users can now do:
brew install lightgbm

Homebrew pre-compiles ("bottles") formula, so it installs in a couple seconds :fire:

WOW thank you so much @ankane this is great!
I read the formula and it looks good to me. @StrikerRUS we should consider updating that formula as a part of our release process. I think all we'd need to do is update which archive it points to.

@ankane Awesome! Many thanks to you!
@jameslamb

we should consider updating that formula as a part of our release process.

That's for sure! We should send PR to Homebrew repo right after LightGBM new release.

I think all we'd need to do is update which archive it points to.

@ankane Can you please confirm this? Should we only update url and sha256 fields in the formula?

@StrikerRUS Yeah, unless the build instructions change.

@ankane Got it! Thanks a lot for your kindness!

@ankane BTW, is it possible to add an option --mpi to the formula to install with the MPI support? Maybe for some users it will be useful.
It will require only adding open-mpi dependency and -DUSE_MPI=ON flag for CMake.
https://github.com/microsoft/LightGBM/blob/master/docs/Installation-Guide.rst#apple-clang-2

It looks like options aren't allowed in formula in Homebrew core. From the docs:

Note: options are not allowed in Homebrew/homebrew-core as they are not tested by CI.

https://github.com/Homebrew/brew/blob/master/docs/Formula-Cookbook.md

@ankane Oh, I see! I'm sorry, I'm not very familiar with brew. I asked only because I have brew install gcc --without-multilib command (outdated already) in my head 😃 .


Use useAuth context for useSWR fetcher

Fixes AdminUserManagement page to use UseAuth token instead of fetching from local storage

Partially Resolves #86

These changes will prevent admin users from accessing the page
Need to handle the unauthorised and isLoggedIn states differently. Maybe use auth?.user?.isAdmin for unauthorised and auth?.user != null for isLoggedIn? This would also partially implement the FR in issue #22.

Ah I see, hmm maybe we can close this for now and fix it a bit later on together. Context is we saw that the useAuth wasnt being used so we thought to use it.

Also, noticed that using this implementation introduces a 5-10s delay before the GET request for the list of users is fired.

Hmm that's weird, could it be that the AuthContext is not passing the data fast enough?

Ah I see, hmm maybe we can close this for now and fix it a bit later on together. Context is we saw that the useAuth wasnt being used so we thought to use it.

I see, let's discuss this during the next meeting then, we'll then be able to have a consistent way of doing this. We don't have to close this, but can instead add onto this PR later on.

Hmm that's weird, could it be that the AuthContext is not passing the data fast enough?

Went to investigate a bit and seems like AuthContext is not passing the data as soon as the component renders, it takes 1-2 rendering cycles before the auth value is populated. Seems like the 5-10s delay is a result of the SWR failing initially due to the lack of auth, but retrying 5-10s later.


Get Image Using Website Agent

Hi--I'm trying to extract the first image URL for each item in this feed below using Website Agent.
RSS Feed:
https://www.prnewswire.com/rss/health-latest-news/health-latest-news-list.rss
The image URL is in "item media:content" but I've tried ~20 different coding options and I can't figure it out. Posting an example of what I've tried doesn't seem like it would help as I feel I'm not even close to getting it. Yes, I've seen the Huginn agent examples pages and done google searches, tried Selector Gadget (doesn't work on XML?) and even tried to generate Xpath.
If you're wondering why I'm not using an RSSAgent for this RSS feed, it's because I couldn't use RSSAgent due to the fact that I need to filter out non-English entries. I can't remember exactly why exactly, note that I also have limited coding skills.
HELP ME Huginn Wan Kenobi, e.g. @cantino ! Happy to donate to your efforts for this vital service, just tell me how!
Mary

//item//*[name()="media:content"]/@url

I suggest you ask this question in a different community specialized in XPath query next time.

Thanks, however I can't get this to work--I tried pasting it into both Huginn views, using "toggle view." I really don't know Xpath at all, so if someone can give more explicit instructions, I'd appreciate it.

Pick a website agent, configure it as following:
{
  "expected_update_period_in_days": "2",
  "url": "https://www.prnewswire.com/rss/health-latest-news/health-latest-news-list.rss",
  "type": "html",
  "mode": "on_change",
  "extract": {
    "images": {
      "css": "//item//*[name()=\"media:content\"]",
      "value": "@url"
    }
  }
}

Good luck.

Thanks @Plausible0Bug ! I believe that this outputs every image on the page for all items, however. How do I change it so it only outputs one image per item?

Hi @Plausible0Bug thank you so much!
When you run it, however, it gives an error that is something like "mismatch of number of items." I can send it if you'd like, let me know. I decided to instead run the feed through Inoreader, which exports the first image as an enclosure, which I can then work with.
Thanks again!

You have other selectors capturing other elements and since the number of outputs is different across different selectors huginn cannot match them 1:1 .
For example of you try to capture title as well as images, given some items don't have image, the number of titles captured will be more than number of images captured.
The fix would be setting array: true for the selectors.

Thanks @Plausible0Bug !


Pin disappears on setRangePinsByIndices

My rangebar with constant pin (not the tmeporary one) and only right pin behaves badly when rangebar.setRangePinsByIndices(0, 3); is called. I managed to reproduce it on the sample app. The thing is it only reproduces when setRangePinsByIndices is called after some time like this:
rangebar.postDelayed(new Runnable() {
            @Override
            public void run() {
                rangebar.setRangePinsByIndices(0, 3);
            }
        }, 2000);

It's a simple sample but it behaves the same way when threading with rxjava and probably anything else.
I made a branch demonstrating that on my fork: https://github.com/jkwiecien/material-range-bar/tree/pin-disappear

Did you find a solution?

any solution?

There is a merge request fixing that problem. I did copy parts of that and rewrite the library.

Which merge request did you mention ?


Multi volume imaging example not working on 0.6.0

Running this https://github.com/widgetti/ipyvolume/blob/master/notebooks/multi-volume.ipynb produces an empty volume when both are displayed when using v0.6.0.

Thanks for reporting. Hope to look into this soon!


Not able to uninstall plugin via API call or through WebUI

Environment

Plugin in version 1.15.0
vCMP Guest TMOS <IP_ADDRESS>

Summary
Not able to uninstall f5-telemetry-streaming plugin via API call or through WebUI
Steps To Reproduce
Steps to reproduce the behavior:

Perform the below API Call to uninstall package

 curl -H "Content-Type: application/json" -k -u username:password -X POST -d '{"operation": "UNINSTALL","packageName": "f5-telemetry-1.15.0-4.noarch"}' https://<device_ip>/mgmt/shared/iapp/package-management-tasks

Observed the following response:

{
   "packageName":"f5-telemetry-1.15.0-4.noarch",
   "operation":"UNINSTALL",
   "id":"6d338ae0-09f3-4a53-a51e-202959b85e64",
   "status":"CREATED",
   "userReference":{
      "link":"https://localhost/mgmt/shared/authz/users/<username>"
   },
   "identityReferences":[
      {
         "link":"https://localhost/mgmt/shared/authz/users/<username>"
      }
   ],
   "ownerMachineId":"2f802797-07f9-4dd8-9b0d-b1e18f96f310",
   "generation":1,
   "lastUpdateMicros":1605731577522061,
   "kind":"shared:iapp:package-management-tasks:iapppackagemanagementtaskstate",
   "selfLink":"https://localhost/mgmt/shared/iapp/package-management-tasks/6d338ae0-09f3-4a53-a51e-202959b85e64"
}

Expected Behavior
f5-telemetry should be deleted from /var/config/rest/iapps
Actual Behavior
f5-telemetry folder continues to exist in /var/config/rest/iapps and also in Webui

Below is what i get when i check the status of the uninstall
'''
curl -u username:pass http://localhost/mgmt/shared/iapp/package-management-tasks/6d338ae0-09f3-4a53-a51e-202959b85e64 | jq
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   766  100   766    0     0  26370      0 --:--:-- --:--:-- --:--:-- 27357
{
"packageName": "f5-telemetry-1.15.0-4.noarch",
"operation": "UNINSTALL",
"id": "6d338ae0-09f3-4a53-a51e-202959b85e64",
"status": "FAILED",
"startTime": "2020-11-18T15:32:57.529-0500",
"endTime": "2020-11-18T15:33:57.820-0500",
"errorMessage": "Package f5-telemetry: Cannot remove. Stuck at version 1.15.0 with status DELETING",
"userReference": {
"link": "https://localhost/mgmt/shared/authz/users/aneke"
},
"identityReferences": [
{
"link": "https://localhost/mgmt/shared/authz/users/aneke"
}
],
"ownerMachineId": "2f802797-07f9-4dd8-9b0d-b1e18f96f310",
"generation": 3,
"lastUpdateMicros":<PHONE_NUMBER>820602,
"kind": "shared:iapp:package-management-tasks:iapppackagemanagementtaskstate",
"selfLink": "https://localhost/mgmt/shared/iapp/package-management-tasks/6d338ae0-09f3-4a53-a51e-202959b85e64"
}
'''

This looks like a problem with the iControl LX Extension framework that Telemetry Streaming is built on top of. Do you see similar issues with other extensions?

This looks like a problem with the iControl LX Extension framework that Telemetry Streaming is built on top of. Do you see similar issues with other extensions?

This is the only extension i tried..is there a workaround to delete?

Are you able to delete through the GUI (I know that is not your end goal, but it could be helpful for debugging)? Do you see any errors in /var/log/restnoded/restnoded.log when the uninstall failure occurs?

When uninstall via GUI i see below error

Are you able to delete through the GUI (I know that is not your end goal, but it could be helpful for debugging)? Do you see any errors in /var/log/restnoded/restnoded.log when the uninstall failure occurs?

When uninstall via GUI i see below error

That is definitely a problem with the iControl LX Extension framework that goes beyond what we can attempt to fix in Telemetry Streaming. I suggest opening a case with F5 support.

Case is #1-6856307386 is already open, have not received any support so far, the knowledge required to support components of the F5 Automation Tool chain is severely lacking

.

Thank you for the case number. I will see if I can provide support with some additional information to help them along.

the solution was to run clear-rest-storage which uninstalls all plugins and not just telemetry


reduce duplicate code in test tool

motivation: code cleanup
changes: remove almost identitical code in the two variants of the test method, and replace with a shared implementation that takes an output redirection and either prints or collects the output

@swift-ci please smoke test


v1.16.0

Changed

Update Patron list
Update Last Transit to latest version
Update Pulse-code Modulation to latest version

Added

New piece: "Nakaii"

Verified on staging


Evaluation result far from training's evaluation result

Hi, author~
I have a weird problem: sometimes, my evaluation result is far from my training result.
I cannot understand what's going on.
Only modified parameter setting: batch_size=128, frames=9.
 But I didn't modify the paper's code except for batch_size and frames number.

The bug have been fixed a few weeks ago. You can redownload the repo and pretrained models, and retrain the model.

Got it! thank you so much.
I will redownload your repo and try it again.
I still curious what bug that would cause this issue??
Actually, I am facing the same issue in other paper's code, but still don't know how to fix it.

Maybe the bug occurs when loading the pretrained model (https://github.com/Vegetebird/MHFormer/blob/e48271cc6e7e92c13560295db6ce334a974d8ad9/main.py#L123).
The module name in pretrained model (pre_dict .keys()) and defined model (model_dict.keys()) are different, so it does not load some weights of modules, leading to incorrect results.

Hi, thanks for your prompt reply.
I have redownloaded your repo and retrained again.
The first time I train can get the normal evaluation result.
But when I train with modified a little bit of your model structure, it still got the following result.

2022/06/28 03:38:43 epoch: 1, lr: 0.0010000, loss: 0.0811, p1: 54.36, p2: 42.61
2022/06/28 03:58:35 epoch: 2, lr: 0.0009500, loss: 0.0468, p1: 53.80, p2: 40.96
2022/06/28 04:18:27 epoch: 3, lr: 0.0009025, loss: 0.0414, p1: 51.79, p2: 40.03
2022/06/28 04:38:18 epoch: 4, lr: 0.0008574, loss: 0.0385, p1: 50.77, p2: 39.43
2022/06/28 04:58:10 epoch: 5, lr: 0.0008145, loss: 0.0366, p1: 50.59, p2: 39.05
2022/06/28 05:18:02 epoch: 6, lr: 0.0004073, loss: 0.0338, p1: 49.91, p2: 38.41
2022/06/28 05:37:54 epoch: 7, lr: 0.0003869, loss: 0.0331, p1: 49.47, p2: 38.23
2022/06/28 05:57:45 epoch: 8, lr: 0.0003675, loss: 0.0326, p1: 49.49, p2: 38.25
2022/06/28 06:17:37 epoch: 9, lr: 0.0003492, loss: 0.0321, p1: 49.76, p2: 38.30
2022/06/28 06:37:28 epoch: 10, lr: 0.0003317, loss: 0.0317, p1: 50.06, p2: 38.16
2022/06/28 06:57:20 epoch: 11, lr: 0.0001659, loss: 0.0307, p1: 49.28, p2: 38.06
2022/06/28 07:17:12 epoch: 12, lr: 0.0001576, loss: 0.0305, p1: 50.01, p2: 38.46
2022/06/28 07:37:04 epoch: 13, lr: 0.0001497, loss: 0.0303, p1: 49.58, p2: 38.27
2022/06/28 07:56:56 epoch: 14, lr: 0.0001422, loss: 0.0301, p1: 50.25, p2: 38.50

===Action=== ==p#1 mm== =p#2 mm=
Directions   469.51     208.62
Discussion   441.79     220.50
Eating       401.69     216.48
Greeting     463.40     226.62
Phoning      382.94     218.79
Photo        446.51     230.31
Posing       460.14     218.05
Purchases    489.61     220.97
Sitting      383.07     223.03
SittingDown  385.89     238.07
Smoking      387.83     221.27
Waiting      444.74     211.86
WalkDog      441.39     225.07
Walking      385.67     231.80
WalkTogether 410.36     237.21
Average      426.30     223.24
p1: 426.30, p2: 223.24

Thank you so much!
It solved my problem.


Aarch64 fix. - add stdlib(c)

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like I wasn't able to push to the aarch64_fix branch of dslarm/perl-digest-md5-feedstock. Did you check the "Allow edits from maintainers" box?
NOTE: Our webservices cannot push to PRs from organization accounts or PRs from forks made from organization forks because of GitHub permissions. Please fork the feedstock directly from conda-forge into your personal GitHub account.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12302471751. Examine the logs at this URL for more detail.

@cbrueffer  - this is odd - I've asked the bot to rerender and it thinks it doesn't have permission to add to this PR.
All I need to do is add stdlib(c) which is done, and the PR does have that permission.
Can you check or fix directly.?
@martin-g - fyi


Update robotlib to 2023

Updating the project to 2023 created 400 errors (not kidding) which will need to be taken care of before the robotlib gets copied into the new project


Fixing redirecting OpenGL Apple dev link

https://developer.apple.com/graphicsimaging/opengl/capabilities/index.html link in the doc was upsetting the Jenkins build because it is being redirected to https://developer.apple.com/opengl/capabilities/index.html

Looks fine to me, please merge.

Thanks. Bah, hasn't fixed the build being stroppy mind you. Oh well, I tried.


[OTHER] Modification - Adding X logo & Name instead of Twitter

What would you like to share?
As we all know twitter has been rebranded as X, it would be better if there is logo & name of X instead of twitter.
Screenshot:

Additional information
I'd like to add logo of X, and change the name of twitter to X.

This has been previously raised and will be closed for the reasons mentioned here: https://github.com/EddieHubCommunity/LinkFree/issues/8403


Problems hovering over bar chart after being redrawn

I'm using ChartJS with KnockoutJS and I'm having some problems sorting one of my bar charts.
Here's an example: http://jsfiddle.net/norbiu/aqa8w19d/
After hitting the sort button on top, the bar chart is redrawn correctly, but if I try hovering over the right side of the chart, it jumps back to the original order of the bars. Removing the mouse will return the bars to the corect order.
Is there something I'm doing wrong?

The solution is to destroy the chart before creating a new one.
    // destroy old chart
    element.chart && element.chart.destroy();
    // create new
    var chart = new Chart(ctx)[type](data, options);
    element.chart = chart;


Render Markdown, Mermaid, etc

Great tool! Could this be used to render Markdown, Mermaid, etc too?

Great tool! Could this be used to render Markdown, Mermaid, etc too?

There are a plenty of tools out there that should provide a live preview of Markdown/Mermaid in a browser, so I don't see why not.
You could use something like https://github.com/iamcco/markdown-preview.nvim more specifically, but it would require some extra setup to open a window in Kitty and then use awrit to open the preview URL


Use assertEqual instead of assertEquals for Python 3.11 compatibility.

The deprecated aliases were removed in Python 3.11 in python/cpython#28268

https://github.com/mjumbewu/django-rest-framework-csv/issues/97


Support for multipart request

Hello,
is there a way to send multipart/form-data with Post request using this tool?

Created a super simple way to add multipart/form-data in: https://github.com/braintribehq/plow
Beside maybe some other issues the main drawback is that I added the complete file into memory. For my cases it does not matter but of course this is not a good solution.

Multipart has a complex format, making it difficult to build through command line options. A better approach may be to manually directly build the raw body. Thanks to @Gerry0815 for providing a great reference implementation.


Trouble with composer

When i try run 'php composer.phar selfupdate' i have:
$ php composer.phar selfupdate
PHP Fatal error:  Uncaught exception 'PharException' with message 'phar "/Users/olegstanko/myapp/composer.phar" SHA1 signature could not be verified: broken signature' in /Users/olegstanko/myapp/composer.phar:13
Stack trace:
#0 /Users/olegstanko/myapp/composer.phar(13): Phar::mapPhar('composer.phar')
#1 {main}
thrown in /Users/olegstanko/myapp/composer.phar on line 13
Fatal error: Uncaught exception 'PharException' with message 'phar "/Users/olegstanko/myapp/composer.phar" SHA1 signature could not be verified: broken signature' in /Users/olegstanko/myapp/composer.phar on line 13
PharException: phar "/Users/olegstanko/myapp/composer.phar" SHA1 signature could not be verified: broken signature in /Users/olegstanko/myapp/composer.phar on line 13
Call Stack:
0.0268     227672   1. {main}() /Users/olegstanko/myapp/composer.phar:0
0.0269     227768   2. Phar::mapPhar() /Users/olegstanko/myapp/composer.phar:13
how can i fix that?

I'm sorry, but I don't use and maintain the project anymore. If you manage to fix the issue PR is welcome!


Weight engine should return a UserWarning if any weight variables are delimited sets

If the user accidentally defines a weight variable used in a weight scheme as a delimited set, when they run the engine they will get a ZeroDivisionError: float division by zero error.
Instead, we should introduce a verification step that provides a UserWarning when a scheme is added to an engine and the scheme uses weight target variables that are not either int or single. A non-fatal UserWarning is the better option here because later we will add native support for using delimited set-typed columns for weight targets.
The  error actually occurs here:
    def rakeonvar(self, target):
        #start=1 : This assumes that survey data ALWAYS starts with 1 (eg, sex: male=1, female=2, undisclosed=3)
        #index is the enumerator (1, 2, 3) and weight is the weight value [50.0, 50.0, 0.0]
        for index, weight in enumerate(self.targets[target], start=1):
            df = self.dataframe[self.dataframe[target] == index]  # df is a subset of the dataframe for the current target
            index_array = (self.dataframe[target] == index)
            data = df[self.weight_column_name] * (weight / sum(df[self.weight_column_name]))
            self.dataframe.loc[index_array, self.weight_column_name] = data

At the line data = df[self.weight_column_name] * (weight / sum(df[self.weight_column_name])). A try/except block should be used here to catch a ZeroDivisionError error and prompt the user to check if they are using an incorrectly typed column.

Is this also possible?
Before the algoirthm starts, the _check_targets() method checks the data type of the passed variable(s):
ValueError: Scheme "simple", group "default_name": Variable q2 has unsuitable type for Weighting.
Target variables must be of type integer / single categorical.

Resolved by #212.


Fix compilation issue in tests on nightly

I noticed that a recent update to the nightly toolchain on Travis broke the compilation of tests/graph.rs. The issue is simply a naming clash because node_index was aliased to n, which was also used as a variable in one of the test bodies.

I wanted to not fix this, since it is a bug in the nightly. It's already fixed upstream. It doesn't matter much either way, thanks for dedicating your time to it.

Thanks. Nightly is now fixed, finally, so we don't need to work around it.


Use relative module specifiers in error messages if possible

Fixes #25747
We don't create node module specifiers since within the checker during checking because we don't really have access to the FS to do the fileExists and readFile calls to check if a given possible representation is valid. Naturally, if it seems it's necessary (ie, we dislike ./node_modules/foo/bar in errors enough) we can rig something up to pass them through from the compiler host.

@RyanCavanaugh wanna update your review?


TabNine unable to activate

I am unable to activate TabNine in my VSCode. I am attaching the error pdf file for reference
TabNine2.pdf
.
Tell me if you need any further info.
Thanks!

Are you behind an HTTP proxy?

Are you behind an HTTP proxy?

Yes @zxqfl

Same as #181


colorizeModelLine does not work if you try to use immediately after creating the model

The code bellow using monaco.editor.createModel and  monaco.editor.colorizeModelLine results into a non-colorized line, but the second time we run (on this case with a setTimeout) it works correctly.
monaco-editor version: 0.28
Browser: Chrome
OS: Windows
Playground code that reproduces the issue:
On https://microsoft.github.io/monaco-editor/playground.html try
const code = `/* Some example CSS */`
const model = monaco.editor.createModel(code, 'css');
const element = document.getElementById('root');
element.innerHTML = monaco.editor.colorizeModelLine(model, 1);
window.setTimeout(() => {
    element.innerHTML = monaco.editor.colorizeModelLine(model, 1);
}, 1000)

the same happens if I tokenize right away instead monaco.editor.tokenize(code, 'css')[0][0]);.
the same happens if we create an editor with monaco.editor.create

it appears that we need to listen to some event to guarantee that the editor/model is ready, is that the case?

Yes, the only ones that work really well are colorizeElement or colorize as they return a promise when the language has finished loading. The colorizeModelLine one will colorize the model's line using the current state of things (i.e. as if the file would be displayed in an editor).


Can Falcon-Phase use multiple Hi-C libraries?

I have two kind of Hi-C data, one of which is made by MboI, the other is made by two enzymes (DpnII and HinfI).
Can falcon-phase use both kind of data at the same time? If not, is there any possible way to use both kinds of data?
Thanks,
Changhan

Hi @changhan1110,
As of now, the code is not setup for multiple library normalization. You can merge all the data and specify multiple enzymes, but we have not studied how that affects the results.
Best,
Zev


[REVIEW] Njong392

GitHub Profile Link
https://github.com/Njong392

Looks great! Some suggestions from the live stream https://www.youtube.com/watch?v=UYiuVStI85M

suggest having your name on the left in the name section
contributor graph to have other sections too like code review


UNIQUE DATETIME index on edge not enforced

orientdb-community-2.0.5
CentOS 6.5
Unique indexes on edges on a datetime are not being enforced, and the issue can be replicated using the following test case:
create class test extends V
create property test.test string
create index idx_test_test on test (test) unique

create vertex test content {"test": "test"}
create vertex test content {"test": "test2"}

create class test_edge extends E
create property test_edge.date datetime
create index idx_unique_test_edge on test_edge (date) unique

create edge test_edge from (select from test where test = 'test') to (select from test where test = 'test2') content {"date": "2015-03-24 08:54:49"}
create edge test_edge from (select from test where test = 'test') to (select from test where test = 'test2') content {"date": "2015-03-24 08:54:49"}

In the above example, the two edges are successfully created with the same date, violating the unique index idx_unique_test_edge:
orientdb {db=test3}> select from test_edge

----+-----+---------+-------------------+-----+-----
#   |@RID |@CLASS   |date               |out  |in
----+-----+---------+-------------------+-----+-----
0   |#13:0|test_edge|2015-03-24 08:54:49|#12:0|#12:1
1   |#13:1|test_edge|2015-03-24 08:54:49|#12:0|#12:1
----+-----+---------+-------------------+-----+-----

However, if I create a new edge class using string instead of datetime, the index works correctly:
create class test_edge2 extends E
create property test_edge2.date string
create index idx_unique_test_edge2 on test_edge2 (date) unique

create edge test_edge2 from (select from test where test = 'test') to (select from test where test = 'test2') content {"date": "2015-03-24 08:54:49"}
create edge test_edge2 from (select from test where test = 'test') to (select from test where test = 'test2') content {"date": "2015-03-24 08:54:49"}

The second command results in a duplicate exception:
Error: com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #14:1: found duplicated key 'OCompositeKey{keys=[#12:1, #12:0, 2015-03-24 08:54:49]}' in index 'idx_unique_test_edge2' previously assigned to the record #14:0 RID=#14:0

for some reason index is not used at all (first example). If you try java code, it just works TM
see for example:
select expand(indexes) from metadata:indexmanager
and try to run:
rebuild index *
it will throw exception:
Rebuilding index(es)...
Error: com.orientechnologies.orient.core.index.OIndexException: Error on rebuilding the index for clusters: [test_edge]
Error: com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #14:1: found duplicated key 'Tue Mar 24 08:54:49 CET 2015' in index 'idx_unique_test_edge' previously assigned to the record #14:0 RID=#14:0

fixed closing.


[FEATURE] Add-PnPMicrosoft365GroupToSite Members Parameter

Is your feature request related to a problem? Please describe.
No issues
Describe the solution you'd like
As of now Add-PnPMicrosoft365GroupToSite allows to add group owners with Owner parameter It would be handy to add members along with owners:
Please add Members parameter if possible
Describe alternatives you've considered
The only alternative is using Add-PnPMicrosoft365GroupMember cmdlet once the group is created.
Additional context
PnP.PowerShell 1.7.18

Unfortunately this is not possible to the limitations of the underlying API that we use to groupify sites, so will have to close this issue.
Sorry :(
As you correctly mentioned, please use Add-PnPMicrosoft365GroupMember cmdlet to add members


Callback was already called error

Hi,
After installing and trying to use the plugin, I've got compilation errors.
`throw new Error('Callback was already called.');
^
Error: Callback was already called.
at throwError (/Users/.../node_modules/neo-async/async.js:14:11)
at /Users/.../node_modules/neo-async/async.js:2813:7
at process._tickCallback (internal/process/next_tick.js:61:11)
`
any suggestions?

same for me

how to fix it?


EntityList offset is wrong

For some reason the Entitylist offset is not working when doing entitylist + 0x10 listentry is invalid, and when taking a look at the structure in reclass it doesnt have a order similar to what ive seen before, seems traversing the entitylist is different now, or offset is wrong.

Looks right to me.

Then why is entitylist reading 0? my memory class is completely fine as reading something else is working, tf.

Owner

Nevermind, my driver for some reason didn't catch an Exception in a __try SEH block and crashed my driver, after remapping it it didnt work for some reason, tf is wrong with windows

Add 0x4 to the Entity List value while the dumper doesn't update :)


WIP: Additions to fine-calc

[x] Indexing error
[x] write fine calibration to .dat file
[ ] write round trip tests for loading/saving .dat files

Want to fix the docstring while you're in there?

Yep, np

So there's a little bit of inconsistency in the current read/write functions.
read_fine_cal takes a filename and returns a tuple of the channel names and their fine cal info:

return cal_chs, cal_ch_numbers

write_fine_cal, on the other hand, takes a calibration dict that currently must contain ch_names, locs, and imb_cals as keys. Should we standardize this? Do you care which function we remold? We usually want to get all the channel into one big array asap, so combining locs and imb_cals might make more sense.

Also, okay to put this load/write round trip test in test_maxwell.test_fine_calibration()

read_fine_cal takes a filename and returns a tuple of the channel names and their fine cal info:

I would probably make both deal with the "complete representation", i.e. the dict. Then whatever uses read_fine_cal should take the dict convert to cal_chs and cal_ch_numbers. Make sense?

Also, okay to put this load/write round trip test in test_maxwell.test_fine_calibration()?

It's a bit misleading because it's actually independent of Maxwell filtering. In principle fine cal could be used for other things, e.g. inversion. So I'd make a new file like test_fine_cal.py just for the IO. That way test_maxwell can be dedicated to actually doing MF.

I would probably make both deal with the "complete representation", i.e. the dict. Then whatever uses read_fine_cal should take the dict convert to cal_chs and cal_ch_numbers. Make sense?

Yep, sounds good. The inconsistency is actually in maxwell.py -- sorry for the mistake. I added a checkbox to update that function as soon as this gets merged.

@Eric89GXL, ready for review :)

Nice, thanks @wronk


Build fails on undeclared constant ESP_MAC_WIFI_STA

$ idf.py build

../main/Config.cpp: In member function 'bool Config::Read()':
../main/Config.cpp:39:32: error: 'ESP_MAC_WIFI_STA' was not declared in this scope; did you mean 'ESP_IF_WIFI_STA'?
   39 |     int rc = esp_read_mac(mac, ESP_MAC_WIFI_STA);
      |                                ^~~~~~~~~~~~~~~~
      |                                ESP_IF_WIFI_STA
../main/Config.cpp:39:14: error: 'esp_read_mac' was not declared in this scope
   39 |     int rc = esp_read_mac(mac, ESP_MAC_WIFI_STA);
      |              ^~~~~~~~~~~~
../main/Config.cpp:47:17: error: 'struct mqtt_config_t' has no member named 'client_id'
   47 |     mqtt_config.client_id = mqttId.c_str();
      |                 ^~~~~~~~~
../main/Config.cpp:48:17: error: 'struct mqtt_config_t' has no member named 'uri'
   48 |     mqtt_config.uri = mqtturi.c_str();
      |                 ^~~
../main/Config.cpp:49:17: error: 'struct mqtt_config_t' has no member named 'cert_pem'
   49 |     mqtt_config.cert_pem = trimToNull(mqttServerCert.c_str());
      |                 ^~~~~~~~
../main/Config.cpp:50:17: error: 'struct mqtt_config_t' has no member named 'client_key_pem'
   50 |     mqtt_config.client_key_pem = trimToNull(mqttClientKey.c_str());
      |                 ^~~~~~~~~~~~~~
../main/Config.cpp:51:17: error: 'struct mqtt_config_t' has no member named 'client_cert_pem'
   51 |     mqtt_config.client_cert_pem = trimToNull(mqttClientCert.c_str());
      |                 ^~~~~~~~~~~~~~~

Environment:

Ubuntu 20.04 (amd64)
ESP-IDF v5.1-dev-3066-g236fa5e669

Never mind, downgraded to ESP-IDF v4.4 and it compiled without issues. In case anyone is looking for it, here's how to do it (will remove existing esp-idf version):
cd ~/esp
rm -rf esp-idf
git clone -b v4.4.3 --recursive https://github.com/espressif/esp-idf.git

cd ~/esp/esp-idf
./install.sh esp32


Multiverse respawn point not working /mv setspawn not working

Am So Hi Everyone :D i guess lol. So i Need Help With Multiverse, When I Type In /mv setspawn Then Is Says Spawn Set And The Cords And Then When I Die In That World I Respawn There But When i Die In Like Another One I Respawn Somewhere Else I Have Essentials But Not Essentials Spawn

That's probably because the /mv setspawn only sets the spawn for the current world.
To achieve what you want you have to set the respawnworld to the world you'd like your players to respawn in. You can do that with /mv modify set respawnWorld worldname (instead of worldname the name of the world you want your players to respawn in)

Closing ticket because the solution is posted (by auroasmiles).


Remove "refresh board"

Or at least make it use the correct function, not updateBoard but subscribeBoard.


Update pretty-stacktraces to 0.0.1

Closing this for now, but it would still be great to use a non-snapshot version of pretty-stacktraces.


Adding list of contacts will block the UI

When try to add list of contacts to the Core data , calling addListOfContacts or addContact will block the UI until its done , so If we have a list of 1000 contacts it will block the UI for few seconds,
and when try to add the contacts in background it will crash on managedObjectContext since its in the main context and can't be used in multiple threads.
So I want to be able to add list of contacts without blocking the UI.
Here is my method for adding list of contacts:
-(void)updateContacts:(NSArray *)contacts {
ALContactDBService *dBContactService dBContactService = [[ALContactDBService alloc] init];
[dBContactService purgeAllContact];
[dBContactService addListOfContacts: contacts];
}
Calling this method will block my UI for few second
-(void)getContactList {
//code to get the contact list from the server and save it in myContacts
[self updateContacts:self.myContacts];
}
and when I call it in the background it will crash on managedObjectContext :
-(void)getContactList {
//code to get the contact list from the server and save it in myContacts
dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
[self updateContacts:self.myContacts];
});
}
Maybe I'm adding the contacts in a wrong way, If yes please tell me what I can do to prevent this behavior ?
Applozic SDK version: latest version
Xcode version: 9.2

Hi @ShaymaAAZ we're looking into this. Will update you soon.


WCF and TCP protocol

why exactly you are using TCP protocol for communication and not just standard HTTP ?

I definitely could go with HTTP, but I wanted something that can easily maintain a session connection.
Also since the System is only going to run locally (the house) i didn't see a good reason to use HTTP.

i am gonna close this one, feel free to open again


Update Excel tutorial and Offline Add-in package.json files.

Q
A

Bug fix?
no

New feature?
no

New sample?
no

Related issues?

What's in this Pull Request?
This PR updates a couple packages to account for potential security issues.

@davidchesnut, Thanks for double-checking this. I'm unable to repro this locally. npm run dev-server runs cleanly for me and uploading the manifest to Excel on the web is successful. What error are you seeing?


fix: 🤔 currentColor for headerButtons not working in header context

Version
The bug occurred with the following version of the component library: 1.16.0
Component
The issue arises when using the following component(s): syn-header / syn-icon-button
Description
If syn-icon-button defines its color as currentColor, the color of the parent element should be used. But it isn't.
<syn-header label="My App">
  <nav slot="meta-navigation" style="color: red">
    <syn-icon-button color="currentColor" name="apps" label="Apps"></syn-icon-button>
  </nav>
</syn-header>

Acceptance criteria

[ ] All DoD items successfully completed
[ ] Icon color matches the color of the parent element (in this case red)

This is not connected to the syn-header. It always only works by explicitly setting the collor on the `syn-icon-button´.
I just realized, that this is not documented and there's no example for that over here: https://synergy-design-system.github.io/?path=/story/components-syn-icon-button--colors
So this is more a documentation problem than a fix.

I am not sure, but the currentcolor documentation on MDN implies the color is inherited. Is this maybe a bug in our implementation?

The color is re-set on the host of syn-icon-button. :)

Usage Error, see comments :)


Clearing LocalStorage on each Request

We started using LocalStorage in our app which has lead to some problems with the seoserver, particularly when we decided to change the default values we use in one of our LocalStorage models or when one url changes the values in LocalStorage which then effects another request.
I'd like to setup the seoserver to clear LocalStorage with each request. PhantomJS stores it's LocalStorage data in a file on the filesystem which is configurable via the --local-storage-path command line option (http://phantomjs.org/api/command-line.html).
I'm thinking in lib/seoserver.js:getContent when spawning the new phantomjs process a unique --local-storage-path could be generated and used for each child process, and then removed during the exit event. Starting up the server could also clear out any LocalStorage files that managed to linger around.
Thoughts?

:+1:


Download Doc from Blockchain

When a document has been stored on chain, it should be downloaded from there.

URL.createObjectURL()

done.


Cannot build provided template project

Hi,
The instructions given here: https://github.com/marcoferrer/kroto-plus/tree/master/example-grpc-client-server#quick-start:
git clone https://github.com/marcoferrer/kotlin-coroutines-gRPC-template && \
cd kotlin-coroutines-gRPC-template && \
./gradlew run

produce the following output:
Cloning into 'kotlin-coroutines-gRPC-template'...
remote: Enumerating objects: 57, done.
remote: Counting objects: 100% (57/57), done.
remote: Compressing objects: 100% (37/37), done.
remote: Total 57 (delta 18), reused 44 (delta 9), pack-reused 0
Unpacking objects: 100% (57/57), done.
e: ~/Workspace/kotlin-coroutines-gRPC-template/api/build/generated/source/proto/main/coroutines/io/grpc/examples/helloworld/GreeterCoroutineGrpc.kt: (25, 25): Unresolved reference: Generated
e: ~/Workspace/kotlin-coroutines-gRPC-template/api/build/generated/source/proto/main/coroutines/io/grpc/examples/helloworld/GreeterCoroutineGrpc.kt: (37, 2): Unresolved reference: Generated
> Task :api:compileKotlin FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':api:compileKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s
6 actionable tasks: 6 executed

What it looks like to me is an unresolved reference to javax.annotation.Generated.
Which java version are you using?
I think your issue might be somewhat related to this grpc-issue

If you are using jdk 9+ then you may experience issues with javax.annotation in general. The suggested fix mentioned here is adding the dependency explicitly.
compileOnly "javax.annotation:javax.annotation-api:1.2"

Thanks for pointing this out. Ill be sure to update the templates

Yes I am using a jdk 10.
I'll try the fix. Thanks for updating the template.

I’m going to close out this issue. If you run into anymore issues with running the template project fell free to reopen. On a side note, when I updated the templates I removed a few code smells for ambiguous scope resolution.

OK, thanks @marcoferrer


Do not postdata before project loads

Clicking while the project load throws these errors (though they don't cause issues)

This is because it gets the stage by default, but the stage I guess doesn't exist before project load.

In the upcoming fix, this will only delay detecting mousedown until after the project has loaded


Bold room notifications on room/global promotions

[16:27:41] ~The Immortal: what if promotion messages were bold?? right now they don't stand out at all but promotions are pretty important.

ummmm
I personally don't like this idea... I don't think they need to stand out that much, and some people don't like standing out, anyway.

@TheImmortal

I think it should stand out because it's a big thing. Tournaments and winners are bolded, for example. Someone being promoted is much more important and long-lasting than a tournament. I've seen it first hand that this message is easy to miss. And then like 5 minutes later someone notices the symbol change and say "oh grats". You can't hide a promotion, people notice the symbol change and/or the position on user list change. This message not standing out means it doesn't fulfill its purpose of announcing said promo.

Its purpose isn't announcing the promo, it's just to log it. I think it fulfills that purpose fine.

Global promotions however are much more "significant" @Zarel. If my opinion matters here (just as a user not staff) I think it'd be nice for global promotions (usually only Voice and Driver are done in public rooms anyways) and those are obviously goals or achievements for many people. Seems fair to give them a one announce-esque message of congratulations that really stands out more.
And if it's purpose is simply to log it then that could be done privately - it however, is not. Obviously there was some sort of public information decision at a point and this is no different.

Yes, it's public information, but that's distinct from a public announcement. It's intended to be there for the people who care, but it's not intended to be all in-your-face about it.

I feel pretty neutral on this topic, but I also don't see little to any harm in making it stand out in the room.

The emphasis treadmill will make everything bold and shouty, which is clearly bad. What we need is some sort of criteria for what sort of things we think should be emphasized and why.
I think if a promotion is a big deal, the mod who promoted the user can easily just say everyone congratulate $NEWLY_PROMOTED_USER, if they think it should be a big deal. But personally, I've never thought promotions should be a big deal.

I am going to close this soon, because so far I haven't been convinced.

I would be fine with bold promotions only in Staff room myself (if it's important enough to be marked here, probably only global promotions and room owner promotions would meet the threshold if even those). There should be no reason to bold promotions in public room, and from a point of view of an user, those aren't all that important.
And besides, a staff member can tell chat to congratulate an user in a more personal way than a random message generated by server itself.


Cypress test. Issue 2440.

Motivation and context
Cypress test. Issue #2440 .
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.07%) to 62.727% when pulling 327e34ca3d018bb5fba719b1d193e30ddbd22c9d on dvkruchinin:dkru/cypress-test-issue-2440-pr into 6c60544aab8598a890da05d5ae2fe9a35305e501 on openvinotoolkit:develop.


Imgur Authentication Fails to Redirect to Obsidian

OS: Linux/GNU Debian Bookworm
DE: Gnome
Obsidian Version: 1.1.16
Browser: Firefox 102.10.0esr (with no adblocker of any kind)
I have been trying since yesterday to successfully authenticate, but every time it is unable to set the Obsidian plugin token, thus authentication is not successful even though "obsidian-imgur-plugin" shows as an application in my Imgur account.
Steps to reproduce

Click authenticate in plugin settings
Type in Imgur credentials on the opened Firefox window
Press "Allow," with seemingly nothing happening the first time
Press "Allow" again which opens a new window displaying json data
{"data":{"error":"Unauthorized","request":"\/oauth2\/authorize","method":"POST"},"success":false,"status":403}

Afterwards, the plugin shows as a connected app in my Imgur settings, but the plugin still shows a "Not Authenticated" status.

Hi @Alvin21Bon
I guess this is something related to your local OS setup and not a bug of a plugin. Based on your description I tend to think this can be a problem with your Obsidian installation which did not register a handler for obsidian://... URLs in your OS.
My OS is macOS, I just tried setting Firefox 112.0.1 as my default browser and tried to authenticate Imgur plugin in my Obsidian. It worked out without any problems. There was a single question from Firefox to open the redirected link with Obsidian. I pressed "Allow" and authentication has completed successfully.
I will convert this "issue" to "discussion". Hope you will find a solution for your problem soon. Please share the solution when you will find it, so it can be useful for other Linux users.


changes splash experience

disable spin button
startup status changes from tzarebczan

Comments on button presentation?
Worth deconstructing some feather icons (aperture?) to make a spiral?

Made some small changes so we don't show blocks behind if it's a short status. @seanyesmunt mentioned we need to cancel the timeouts, so he'd take care of that (there's 2 fyi).
There is a scenario where it shows the blocks behind anyway - I think the headers component may not be available right away sometimes, so it enters the blocks behind if statement. Doesn't happen all the time, not super worried about it now.


chore(scripts): add prepare script

This is extremely simple, yet useful (in my opinion) change.

NOTE: If a package being installed through git contains a prepare script, its dependencies and devDependencies will be installed, and the prepare script will be run, before the package is packaged and installed.

Since npm v5 npm executes prepare script when dependency is installed from git. By including prepare script it will allow our users use @vue/test-utils from main (or any other branch) even if release was not published to npm yet
However, this has a downside - after doing install locally (without include @vue/test-utils as dependency) it will be also executed. However, I feel here benefits outweight the downside
WDYT?

I don't have a strong opinion about this one, I don't think I ever used prepare.
@lmiller1990 @freakzlike WDYT?

I don't have a strong opinion about this one, I don't think I ever used prepare.

The great thing is it used automatically and our users will be able to use main branch if needed

I think this is good. Not sure if that will affect the release process

@freakzlike not at all. We have extra build triggered inside pipeline (you can see even in this PR that prepare was called during npm install) but it doesn't affect released package at all

Let's go ahead and try it then


Don't sort contents of a data class / data model

Sometimes, it's useful to specify that you don't want to maintain an ordering on data elements or data classes, but just always view them in alphabetical order.
It would be nice to have a flag that could be set (ideally, by the users in the UI), to 'turn off' ordering.

this should be a UI issue where just pass the query param "sort=label" that will sort the entries alphabetically. We should not be adding additional flags to do anything like this

there is a piece of work on idx and siblings work which i need to do after #241 is sorted. Whilst i do this i will be adding an endpoint to "reset" all idx values for content inside an object.

Would be desirable to allow treating certain specific ModelItem collections as unordered, as otherwise the system won't remember which should be displayed as ordered and the default ordering by index may be arbitrary (in insertion order if creating ModelItems by the UI).
Options to allow treating some ModelItem (element, class, etc.) collections as unordered:

Treat a collection of MIs all having index of 0 as unordered. When adding a new MI to an unordered collection, keep all the indices as 0. Add a method to the parent objects to remove ordering from a collection, e.g. POST .../dataElements/unsort or POST .../dataElements/removeOrder.
Add a flag to the parent objects to specify the default sort field for the collection, to allow storing a custom default sort field for each collection.


SKU-1537 #Changed table rendering and added button to collapse and ex…

#Changed table rendering and added button to collapse and expand rows for compact table view

Hi, I'll accept the request if you disable it by default and putting a setting in Settings.js to enable it, for most people it doesn't make sense to have a table do this.

Hi,
Ok, I’ll do it soon
--
Arseniy Gshyan
From: Tom Barber<EMAIL_ADDRESS>Sent: Thursday, September 29, 2016 3:29 PM
To: OSBI/saiku
Cc: Гшян Арсений Мисакович; Author
Subject: Re: [OSBI/saiku] SKU-1537 #Changed table rendering and added button to collapse and ex… (#574)
Hi, I'll accept the request if you disable it by default and putting a setting in Settings.js to enable it, for most people it doesn't make sense to have a table do this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/OSBI/saiku/pull/574#issuecomment-250452128, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVRF4imQwolGt6XUqdVKTa18v0Wu6dUOks5qu68agaJpZM4KJ3dh.

Hi,
How soon you accept this request?

hey @bftcore I tested your modification, but you need solve the code conflicts in saiku-ui/js/saiku/Settings.js. Please fix it and i'll do the merge ;)


remap bstv

I don't like b,s,t,v. Can I remap bstv? I find you use  make it cannot to realize. Thank you.

You can hook into the User Startified event. E.g.
autocmd User Startified call s:on_startify()

function! s:on_startify()
  nnoremap <buffer> b :echomsg "B!"<cr>
  " ...
endfunction

Oh, I wanna know how to use other key to replace bstv, it looks hard because the bstv use  , thank you.

Oh! I see. Let me change them to autoload functions.

You can now use this for example:
autocmd User Startified call s:on_startify()

function! s:on_startify()
  nnoremap <buffer> x :call startify#set_mark('B')<cr>
endfunction


A faster DomainMatcher implementation

I implemented a faster domainmatcher based on AC automaton, which is about 30% faster than the current implementation in local testing. And it combines the matching of sub domain, plain domain, and full domain routing rule to make it easier to maintain.
The figure below is a comparison of the new and old implementation benchmarks.

Thanks for your work, I will solve the test problem

Thanks for your work, I will solve the test problem

Thanks for you work.

Thanks for you work.


removes use of sc_client::Client from sc_basic_authorship

see #4452

@seunlanlege please merge latest master and fix the conflicts

Why do we merge refactors on code freeze?

feature freeze. During alpha we still accept not-too-involved fixes and changes to the API for improvements. These won't be accepted anymore during beta.

@seunlanlege merge again? this conflicts now – then set the looks-good label and I happily merge it.


Implementation of data generator for employees.csv dataset

Some examples require employees.csv dataset. This dataset is publically available but I am not sure about usage restrictions. Secondly, the public dataset is of fixed size. For making it more useful in variety of examples and tests I've implemented the data generator that mimics original dataset structure but now can be generated of desired size.
For getting-started like examples this dataset can be really small. For scale-out benchmarks the dataset needs to be generated of sufficiently large size to demonstrate scalability effocts.
For instructions how to use the data generator one can type gen_employees_csv.py --help

Hello @samaid! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file generate_data/gen_employees_csv.py:

Line 11:65: W291 trailing whitespace


Issue 2115 - Update spotbugs plugin in query-core module

Make sure you have checked all steps below.
Issue

[x] My PR addresses the following issues and references them in the PR title. For example, "Issue 1234 - My Sleeper
PR"

Resolves https://github.com/gchq/sleeper/issues/2115

Tests

[x] My PR adds the following tests OR does not need testing for this extremely good reason:

Fixed spotbugs issues

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it, or I have linked to a
separate issue for that below.
[x] If I have added, removed, or updated any external dependencies used in the project, I have updated the
NOTICES file to reflect this.

Closing as we'll disable the Spotbugs rule instead, see issue:

https://github.com/gchq/sleeper/issues/2133


Untranslated ability

In Ultima's Bane (Unreal), one of the abilities that the boss uses is in japanese

Most boss auto-attacks are not translated, because their name never shows up in game. It just says 16796 damage in the event log and floating combat text

I could try to distinguish between auto-attacks and named attacks and change the wording in that case, I'll have a look at it.

Fixed in the latest version


Export - Intel Hex - Selected only, option does not export the entire selected range

Describe the bug
When part of the binary is exported, to Intel Hex format, using the Export Program function, with the "Selection only" checkbox ticked, the saved binary does not include all the selected data.
Specifically the last instruction does not seem to be saved.
Note. Exporting to "Binary" seems however to work OK. This only applies to Intel Hex format.
To Reproduce
Select an entire function in the Listing view.
From the File menu, select "Export Program"
From the Format dropdown select "Intel Hex"
Make sure the "Selection Only" checkbox is ticked
Export the file.
Open the file in an editor that can read Intel Hex format, e.g. use Segger J-Flash
Compare the last bytes in the file with the last bytes selected in the disassembly.
Expected behavior
All selected instructions and data should be exported
Environment (please complete the following information):

OS: Windows 7 Pro
Java Version: 11
Ghidra Version: 9.0

Additional context
The disassembly is ARM Cortex Little endian.

FYI. I just updated the bug, because I was in the Listing view not the disassembly view

Thanks for the info @rogerclarkmelbourne. This is happening because we currently restrict output to only 16-byte records; anything left-over is ignored. eg: if you select 36 bytes you'll get two 16-byte records in the output with the remaining 4 left out.
We're looking at making this more flexible so users can configure the record size and/or ensure that all bytes are present in the output.

Thanks
My workaround is to export as a binary, and then use Segger J-Flash to load the binary, where I manually enter the start address. I then resave the binary as Intel Hex using J-Flash.
( there are probably other prograns that could be used for this)
I don’t really understand why you would quantise the size to 16 bytes for intel hex but not for binary.
Perhaps you could remove the quantisation altogether. But I presume there must be a reason for doing it.

PS. When you. Release the source code for Ghidra I would probably change the intel hex export and send a PR

@adamopolous
I just found another bug with the Intel Hex
I only selected 2 bytes, but when I exported the Selection only, I either got nothing or at one time I seemed to get  the whole file.
I'll update my original bug post

The way things are currently implemented I would expect a 2 byte selection to result in nothing every time - not sure how it would get the whole file; if it's doing that then there's certainly something buggy going on. Any more info you can provide on that scenario would be helpful.
But that all might be moot - I'm looking at removing the restriction altogether.


if let binding of a non-optional doesn't raise a compilation error in view builders

Description
Normally, if we bind a non-optional using if let, the compiler will protest (i.e. `Initializer for conditional binding must have Optional type, not 'X').
However, when doing the same in a view builder, the compiler is silent.
Steps to reproduce
Given the following simple view builder:
@resultBuilder
struct StringBuilder {
    static func buildBlock(_ components: String...) -> String {
        String(components.joined(separator: ","))
    }
    static func buildOptional(_ component: String?) -> String {
        component ?? ""
    }
}

struct StringWrapper {
    private let _value: String
    init(@StringBuilder _  builder: () -> String) {
        _value = builder()
    }
}

Trying to bind a non-optional will compile just fine, possibly leading to confusion and unexpected results:
let nonOptional: Int = 0
StringWrapper {
    // compiler should complain
    if let nonOptional = nonOptional {
        String(nonOptional)
    }
}

Expected behavior
The compiler should complain when trying to bind a non-optional in a view builder.
Environment

swift-driver version: 1.62.15 Apple Swift version 5.7.2 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>), Target: arm64-apple-macosx13.0
Xcode 14.2, Build version 14C18
N/A

Can you test with Swift 5.8? We noticed some of these are fixed and warn now

Indeed, this has been fixed in 5.8 as outlined by https://forums.swift.org/t/improved-result-builder-implementation-in-swift-5-8/63192

Yes, it was fixed 🎉. I've just tested. Sorry for the noise, I haven't found a reference mentioning it.

No worries!


chore(bridge-app): remove radix-ui dependencies

ui-components package builds and exports the components, so radix-ui packages don't need to be included the dependencies anymore

Current dependencies on/for this PR:

main

PR #56   👈

This stack of pull requests is managed by Graphite.


Potential Idempotency Issues in the NormalizeDeviceData Function

APIs Involved:
EventHubTrigger, EventHubMeasurementCollector, and IDataNormalizationService
Description:
The NormalizeDeviceData function does not currently handle conditions of unwanted retries. As a result, it may process the same input events multiple times and store duplicate normalized measurements in the output Event Hub. To address this issue, we need to ensure that each input event is processed only once, preventing the insertion of duplicate normalized measurements.
Steps to reproduce:
For each API listed above:

Simulate a failure/error exit after that API but before the function returns.
Run the function for the first time and let it endure that failure/error exit.
Let the function automatically retry with the same input.

Expected behavior:
The NormalizeDeviceData function should process each input event only once and skip any subsequent calls with the same events.
Proposed solution:
Introduce idempotency checks in the NormalizeDeviceData function to ensure that activities are sent only once for each unique message. This can be achieved by:
Though the deviation of the prevalence counter is not a huge problem, such an issue can be resolved by maintaining a LastRequestId field in each table entry and blob entry, which stores the invocation id of the last NormalizeDeviceData function call. The invocation id is constant across Azure Function retries. Before updating a table/blob entry, if the comparison finds item.LastRequestId == FunctionContext.InvocationId, so no more redundant action would be required. Otherwise, if item.LastRequestId != FunctionContext.InvocationId, update the prevalence counter and the LastRequestId field with FunctionContext.InvocationId.
By implementing these changes, the NormalizeDeviceData function will become idempotent and avoid processing the same input events multiple times, ensuring proper usage of the APIs listed above.
Thank you for your contribution to the Github community and I really appreciate your effort in going through this issue.

@NullPointer4096 - Just to clarify this issue, The NormalizeDeviceData function can add duplicate normalized measurements to the Event Hub when an error occurs. Is this correct?
I ask because the data processing has not completed at this point. After this first stage, the duplicate duplicate normalized measurements will be read by the MeasurementCollectionToFhir function and de-duping takes place there. The data output is idempotent at the FHIR Observation, not the normalized measurements.
Is this a performance improvement you are suggesting? Or are you using the normalized measurements for something that requires no duplicates be emitted?

@NullPointer4096,
The solution uses Azure Stream Analytics as a processing step before the MeasurementCollectionToFhir Function is invoked. Azure Stream Analytics reads from the Event Hub and groups normalized measurements by identifiers and sends the normalized measurements to the MeasurementCollectionToFhir Function (via the HttpTrigger).

As designed.  Eventual consistency and replay is expected.


[stable/distributed-tensorflow] Add Dysproz to OWNERS, update owner in Chart.yml

As discussed with @zanhsieh in PR #16436, I add myself to OWNERS of this chart and update Chart.yml. I've mentioned previous owner in OWNERS file in emeritus section.
This PR is based on issue #13869
Signed-off-by: Szymon<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped (Should it be bumped in this situation?)
[x] Variables are documented in the README.md (Owner is not mentioned in README)
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @maorfr

/assign @zanhsieh

/assign @lachie83

@maorfr @lachie83
Would you mind granting me as one of the owners of this chart?

@cheyang ptal

@Dysproz
Check your email if invitation received.

@zanhsieh
I didn't receive any invitation

@maorfr
Would you mind to take a look please? Thank you.
/ok-to-test

i would like to wait for @cheyang  to respond

@maorfr After a month of waiting, can we proceed further or are we still waiting?

/lgtm

/retest

/verify-owners

/lgtm