Support background colour for radial scale point labels

Happy holiday,
I just want to ask for an option to add the background color (a rounded rect with bg color) for labels (individually) in Chartjs. Is there any callback that I can apply for changing this, any help is appreciated with best regards!
This is the demo

To someone who wants to implement this, would need to draw a background rectangle just before https://github.com/chartjs/Chart.js/blob/master/src/scales/scale.radialLinear.js#L278

@etimberg Thank you so much for your feedback.
I am trying to look at this tut http://devlabo.blogspot.jp/2010/03/javascriptcanvas.html as well but not figuring out the way to implement this.
Does it mean that I must modify Chart.bundle.js? I am so confused right now, excuse me.

I think drawing the rectangle should be as simple as fillRect(left, top, width, height) before the line I mentioned above. You could modify Chart.bundle.js or you could modify the source file and then rebuild the output.

@etimberg I have tried but and this is what I got
  // Add a func to draw round rect   function fillRoundRect(ctx, l, t, w, h, r) {     var pi = Math.PI;     ctx.beginPath();     ctx.arc(l + r, t + r, r, - pi, - 0.5 * pi, false);     ctx.arc(l + w - r, t + r, r, - 0.5 * pi, 0, false);     ctx.arc(l + w - r, t + h - r, r, 0, 0.5 * pi, false);     ctx.arc(l + r, t + h - r, r, 0.5 * pi, pi, false);     ctx.closePath();     ctx.fill();   }
// Add to line L277 fillRoundRect(ctx, pointLabelPosition.x - 5, pointLabelPosition.y - 5, 65, 30, 15);

Problems:

Having trouble with the position of each rect, do you guys have any suggestion?
I want to change the color of each rect.
Which is the best way to handle this in a project, I need help to modify the source code and update it in node_modules

I really appreciate if someone goes to test these codes and give me some advice (bow)

Screenshot:

pointLabelPosition is the center of the text, so you would need to adjust the position
var size = scale._pointLabelSizes[i];
fillRoundRect(ctx, pointLabelPosition.x -(size.w / 2), pointLabelPosition.y - (size.h / 2), size.w, size.h, 15)

To change the colour you can do:
ctx.save()
ctx.fillStyle = 'rgba(r, g, b, a)'; // new fill colour here
ctx.restore()

Hi,
Thank you so much for this! I can't, however, get the backgrounds colored, neither in the shape that spideyinf had, nor in the right position. I attach image and code, inserted between these two lines (in v2.7.2, bundle:
adjustPointPositionForLabelHeight(angle, scale._pointLabelSizes[i], pointLabelPosition);

and
fillText(ctx, scale.pointLabels[i] || '', pointLabelPosition, plFont.size);

Here's the code:
function fillRoundRect(ctx, l, t, w, h, r) {
	var pi = Math.PI;
	ctx.beginPath();
	ctx.arc(l + r, t + r, r, - pi, - 0.5 * pi, false);
	ctx.arc(l + w - r, t + r, r, - 0.5 * pi, 0, false);
	ctx.arc(l + w - r, t + h - r, r, 0, 0.5 * pi, false);
	ctx.arc(l + r, t + h - r, r, 0.5 * pi, pi, false);
	ctx.closePath();
	ctx.fill();
}
// Add to L277
ctx.save();
ctx.fillStyle = 'rgba(34, 23, 31, 42)'; // new fill colour here
ctx.restore();
var size = scale._pointLabelSizes[i];
fillRoundRect(ctx, pointLabelPosition.x -(size.w / 2), pointLabelPosition.y - (size.h / 2), size.w, size.h, 15);

And here's the result:

Really grateful for any help.
Vidar

@ninjaklas the issue I think is that in the current code, the pointLabelPosition is not guaranteed to be the left edge of the text.
These two lines get the canvasTextAlign and adjust the height. You should adjust the height and then use the following code based on the alignment

Alignment
Outcome

center
Current code works

left
Need fillRoundRect(ctx, pointLabelPosition.x -size.w, pointLabelPosition.y - (size.h / 2), size.w, size.h, 15);

right
Need fillRoundRect(ctx, pointLabelPosition.x, pointLabelPosition.y - (size.h / 2), size.w, size.h, 15);

Thank you so much @etimberg, exactly what I needed! With some additional tweaks I now got the positioning exactly where I wanted it.

Hi @etimberg, would you happen to have an updated idea/example of how to style the radial scale point labels as rounded rects considering your changes for version 3.x in #8588? I am looking to achieve the same end result as the original poster.

@kyracriteria I don't have updated code, but it should be a lot easier now. You can call chart.scales.r.getPointLabelPosition(index) which will return an object containing left, top, right, bottom representing the edges of the rectangle that is the point label. It should be much easier to draw the background now.


Logging: Actions.py - Remove

Codecov Report

Merging #459 into experimental will increase coverage by 0.04%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           experimental     #459      +/-   ##
================================================
+ Coverage         50.07%   50.12%   +0.04%
================================================
  Files                50       50
  Lines              3193     3196       +3
================================================
+ Hits               1599     1602       +3
  Misses             1594     1594

Impacted Files
Coverage Δ

vent/api/actions.py
51.32% <100%> (+0.38%)
: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 13891de...0d63658. Read the comment docs.


Increase DefaultMaxLength to be aligned with Agora max_tx_set_size

Agora serializes "Value"s which are serialized ConsensusData, represented
as a byte array. To allow 1MB blocks, we should support handling large
"Value"s in serializer as well.

I think we should just set it to 0, as in, unlimited by default, with the ability for applications to set a hard limit, because we have enough evidence that there's no good value here.


Test code bug in training

Thanks for your research!
When I run python scripts/train_rpn_3d.py --config=kitti_3d_base --exp_name base, training is normal, but a bug appear in testing. My torch version is 0.4.1. It looks like a object type error, have you ever met the bug? Thank you!
Epoch:9
acc/fg: 0.950
acc/bg: 0.997
misc/z: 0.641
misc/ry: 0.311
acc/iou: 0.856
loss/ttloss: 0.551
testing
  0%|                                                                                | 0/3769 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "scripts/train_rpn_3d.py", line 323, in <module>
    main(args)
  File "scripts/train_rpn_3d.py", line 288, in main
    iou_3d = test_kitti_3d(dataset_val, rpn_net, conf, results_path, paths.data, writer=writer)
  File "/**/**/model/M3DSSD-master/lib/rpn_util.py", line 1794, in test_kitti_3d
    aboxes = im_detect_3d(im, net, rpn_conf, imobj)
  File "/**/**/model/M3DSSD-master/lib/rpn_util.py", line 1462, in im_detect_3d
    bbox_x3d = bbox_x3d * bbox_stds[0, 4] + bbox_means[0, 4]
RuntimeError: Expected object of type torch.cuda.FloatTensor but found type torch.cuda.DoubleTensor for argument #2 'other'

I just fix the bug.Modify rpn_util.py #1444 1445 1446
anchors = torch.from_numpy(rpn_conf.anchors).cuda().float()
bbox_means = torch.from_numpy(rpn_conf.bbox_means).cuda().float()
bbox_stds = torch.from_numpy(rpn_conf.bbox_stds).cuda().float()

Moreover, #1516 sorted_inds = (-aboxes[:, 4]).argsort(), tensor doesn't support argsort(), should use torch.sort().
In the end, how to use multi-gpu to train?

Thanks for your contribution to fixing the bug. I haven't tested multi-gpu training,  you can modify the code to support this feature.

I just fix the bug.Modify rpn_util.py #1444 1445 1446
anchors = torch.from_numpy(rpn_conf.anchors).cuda().float()
bbox_means = torch.from_numpy(rpn_conf.bbox_means).cuda().float()
bbox_stds = torch.from_numpy(rpn_conf.bbox_stds).cuda().float()

Moreover, #1516 sorted_inds = (-aboxes[:, 4]).argsort(), tensor doesn't support argsort(), should use torch.sort().
In the end, how to use multi-gpu to train?

Hi! To train with multi-gpu, you should modify the code at lib/core/init_training_model to as fllow:
        if 'CUDA_VISIBLE_DEVICES' not in os.environ.keys():
            os.environ['CUDA_VISIBLE_DEVICES'] = '0'
        device_ids = [id for id in range(len(os.environ['CUDA_VISIBLE_DEVICES'].split(',')))]
        network = torch.nn.DataParallel(network, device_ids)
        network.to('cuda')

Then use CUDA_VISIBLE_DEVICES='your gpu device ids' python scripts/train_rpn_3d.py --config=config --exp_name=exp_name to train with multi-gpu


Adding Docker Build related automation as PR checks

If any code in dockerfiles get updated, we should run a validation docker workflow in github actions to ensure the build is success.
#1397 shows some issues in older docker enginer version, we probably need to test the BC as well.
Thanks.

@rishabh6788 @prudhvigodithi @peterzhuamazon @gaiksaya Do we have any automated validation workflows running now?

No we do not. @peterzhuamazon Can you be more clear on this? Do we want to test the docker image building part or see if docker image is causing issues for some plugins?
For 1. where docker image config needs to be tested we can build and test (start a cluster and run basic apis) as integration tests for docker on your part
For 2. If we need to check docker image issues with some plugins, we need to add docker as one of the distribution in the integration testing framework.

This is very old and the current idea of mine is just to have a simple docker build so at least we know there is no syntax issues in the dockerfile before merge.
Thanks.

Adding docker linting:

https://github.com/opensearch-project/opensearch-build/pull/3337


Renabling plugin causes double icons

If you disable the plugin, the folder icons remain.
If you reenable the plugin, the folder shows a double icon on files that have an icon on them.

Thanks for reporting this issue! It should be fixed now, with 1.0.61.


API Docs website: "On This Page" sidebar does not update when navigating

Bug report
Current Behavior

Go to https://formik.org/docs/api/fastfield
Click on <Formik /> under "API Reference"

Expected behavior
The right sidebar updates to show the new page's Table of Contents
(Note: the TOC does fix itself if I Refresh)
Screenshots

2.

Suggested solution(s)
I'm afraid I have not spent the time to see how your website works.
Additional context

Your environment

Software
Version(s)

Formik

React

TypeScript

Browser
Google Chrome Version 88.0.4324.150 (Official Build) (x86_64)

npm/Yarn

Operating System
MacOS 10.15.7

Seems like the useEffect hook that regenerates the headings isn’t being re-rendered on client side route change. Passing down the current route/path to this useEffect dependency array should cause it to rerender

Ran into this myself. Was confused for a while, as the links (obviously) goes nowhere.


Feature update for intrin_dot_sample.cpp

Summary
Include a short summary of the request. Sections below provide guidance on
what factors are considered important for a feature request.
README says that "They provide access to instructions that cannot be generated using the standard constructs of the C and C++ languages, and allow code to leverage performance enhancing features unique to specific processors. "
May we use some SIMD compiler options to achieve vectorization  ?
Problem statement
A comparison between the performance of using intrinsics and the performance of using SIMD compiler options
Preferred solution
Add a solution to the problem where the size is not a multiple of 8. Currently, it is 24.
Add a solution, if possible, to adding some SIMD compiler options to achieve vectorization
Thanks

@ethanhirsch I think you are the author of this sample.

@ethanhirsch is working on this


Fix typo in security note of ACI spec

In the security note located in "Image Archives" section of App Container Image spec there was a typo mentioning compression when it should say encryption.

LGTM


[0.15.2] Debian build problem with flit

Hi,
I just started with upgrading the poliastro package on Debian (from 0.14.0). One change here is the use of flit which is new for me and causes a build failure for the moment: when running flit in the root dir, I get the error message
$ flit build
Config error: [tool.flit.metadata] section is required

This is also what our automated build does. Do I miss/misunderstand something here? How would I start building?

It looks like poliastro has started using PEP 621 metadata (the [project] table), which can be read by flit_core 3.2+. I'm guessing the version in Debian is older than that.
I don't think there's any easy workaround. If you really need to, you could patch the pyproject.toml file back to the previous format (basically revert this commit). But now that PEP 621 has given us a standard way to specify this metadata, I would imagine that it's going to get adopted by other packages pretty rapidly.

@takluyve if what you say abut adoption of PEP621 is true, that will most likely result in noise. Debian 11 which was released last Summer has flit 3.0.0. Are there backwards-incompatible changes with 3.2.x that would prevent working on pushing that to Debian stable release?

There are.  Flit 3.3 is available in Debian stable backports now.

Also, for Debian packages, we've integrated use of flit into dh-python.  All you need to do now is package it as you traditionally have for a Debian python package and add flit to build-depends.  It will use the pyproject.toml information automatically.  I'm not sure if the newer PEP 621 stuff works on stable.  It does work on testing/unstable.

Closing this out, since @kitterma seems to have made progress on this, on Debian’s side! 🎉


Fix tintColor android issue

The missing default prop of titleColor leads to error on Android OS when the tintColor is used.
This PR solves the #36 and #29 issues.
Until this PR is merged, a workaround could be the use of both titleColor and tintColor in the object props.

You need to send all the parameters as android class file try to get tint color and title color prop and if its null then there is no default values set to parse.

Exactly! This is what this PR resolves. In the documentation there are default values available. Default value means that the prop definition is optional.

@mzachariadis for the Effort and PR.
I have merged the PR.
Thank
</  Pranav >


Remove dependency on swift-system

This dependency is unused and just unnecessarily complicates things.

Hoping I found all remnants of swift-system here. I tested swift build and bootstrap locally. There may be clients of SwiftPM in the toolchain that also need updates to their CMake-based builds.

@tomerd from our offline discussion today, it sounds like you don't want us to do this because we anticipate to eventually use this dependency? But we can do it for TSC, is that a correct summary?

@neonichu correct

OK, so let's close this PR.


deps: Bump envoy.code.check -> 0.2.1

and update extensions_schema.yaml
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Currently the descriptions for the extensions metadata are hardcoded in protodoc and also added as comments in the extensions schema config
This extends the extensions schema config to include the descriptions. The hardcoded values will be removed from protodoc subsequently and it can use the configured values.

prefer in future PRs that changes which don't have an obvious connection

there are 2 changes here:

data schema
parser of ^^

the changes were inseparable and related to nothing other than each other


functionality to monitor process with minimal configuration

I made several changes to:
a) create a jar with all its dependencies in 1 jar (with agent classifier)
b) added a default configuration inside the jar.
c) added some configurations to be applied 1 by one until one solves the name of the process.
d) replaced InputSoure with Document so we can read configuration files from files as before or from input streams.
Added a logging.properties that only reports errors in the logs. Using a property we could make that the agent read other configuration file so we can make logging more verbose.
The idea is that once the agent is installed, it could server several processes without configuration. Other scenario is to have a central configuration somewhere. This is useful for cluster monitoring like amazon, google, etc.

Thanks for this contribution, your efforts are really appreciated.
One initial thing that sticks out when I look at this is the logging stuff.  I've been very tempted to remove logging from this project for a few reasons:

as it is loaded before the main class, the preferred logger may not be ready yet
different people prefer different loggers
we had some problems with JBoss and initialization order, I'm a bit worried that if we initialize logging too early, then similar problems could come back

What is your feeling about this?
We can still have logging in unit tests of course.

I believe we should replace java.util.logging classes used in the agent for
our simple implementation that logs to system.out, a file or does not log
at all.
I would remove it, but it is useful for debugging.
Pablo
On Mon, Apr 20, 2015 at 10:14 AM, Daniel Pocock<EMAIL_ADDRESS>wrote:

Thanks for this contribution, your efforts are really appreciated.
One initial thing that sticks out when I look at this is the logging
stuff. I've been very tempted to remove logging from this project for a few
reasons:

as it is loaded before the main class, the preferred logger may not
be ready yet
different people prefer different loggers
we had some problems with JBoss and initialization order, I'm a bit
worried that if we initialize logging too early, then similar problems
could come back

What is your feeling about this?
We can still have logging in unit tests of course.
—
Reply to this email directly or view it on GitHub
https://github.com/ganglia/jmxetric/pull/35#issuecomment-94512672.


Reference a "raw" generic class directly. [phab:D1604]

Created by benjyw at 2017-05-03 02:27:23:
A previous change (#1860)
made all references to, e.g., java.util.List, refer to its abs node
instead of the class node itself.  This makes sense when the reference
in question is in the context of a type application, e.g., List<String>.
But, for compatibility with pre-generics Java, the language supports use
of generic types as raw, non-generic types.
See https://docs.oracle.com/javase/specs/jls/se8/html/jls-4.html#jls-4.8.
These raw types (e.g., List) are not themselves generic.
So this change makes references to raw types reference the type itself,
and not the abs node.

benjyw wrote at 2017-05-03 02:29:47:
Two questions as part of the review:

Does this seem sensible to you? It aligns with my understanding of raw types (particularly the fact that they are supposed to behave the same as pre-generics Java), and with what ends up in the classfile.

Can you think of other cases of "generic context", apart from typeapply and import?

Thanks!

benjyw wrote at 2017-05-03 21:23:28:
PS I could be convinced that this change is misguided or unnecessary.

salguarnieri wrote at 2017-05-03 22:11:09:
This seems reasonable to me. Is it possible to add a test for the import statement?
It feels like there should be some relationship between NGType and GType.

benjyw wrote at 2017-05-04 19:33:50:
There actually is a test for the import type - it's the imports in kythe/javatests/com/google/devtools/kythe/analyzers/java/testdata/pkg/Import.java.
In fact, it was that test failing that prompted me to add the check for JCImport.
But now that I think of it, I'm leaning against the import counting as "generic context". I only added that to get the test to pass, really, without much thought. But it seems very arbitrary, and probably not really what people looking at a graph will expect.
I'll remove the JCImport check and fix the test, let me know what you think. Thanks!

benjyw wrote at 2017-05-04 19:34:13:

Make the import point to the raw type as well.

salguarnieri wrote at 2017-05-08 19:45:44:
Sounds good to me, zarko, can you take a second pass over this?

zarko accepted this revision at 2017-05-08 20:44:51

benjyw wrote at 2017-05-18 17:57:50:
Thanks for the review - can someone land this?

danielmoy-google wrote at 2017-05-18 18:07:01:
Sure I can land, sec

danielmoy-google closed via commit 659654b2868249ad4e874691f66c7aecca0a9d7d at 2017-05-18 18:07:47

benjyw wrote at 2017-05-18 18:54:51:
Thanks!


update ORT to eb34b0dd11

This fixes https://github.com/oss-review-toolkit/ort-ci-github-action/issues/17

This fixes oss-review-toolkit/ort-ci-github-action#17

Are you certain? I currently don't see how changing the ORT version that is used to perform the checks in this config repo changes the ORT version being used as part of the GitHub Action.

My bad, sorry. I was playing around with the config and accidentally opened this PR here instead of in my fork.


Could not find generator 'haml:application_layout'.

Followed the installation guide for haml-rails.
$ bundle exec rails generate haml:application_layout convert
Running via Spring preloader in process 17003
Could not find generator 'haml:application_layout'. Maybe you meant 'application_record', 'integration_test' or 'migration'
Run `rails generate --help` for more options.

I'm facing similar issues for my rails 3 app.

Any solution for this..


Support for boolean flags without explicit values

Description
This PR adds support for hybrid boolean flags which applies to large and short boolean flag variants.
Implicit example:
static-web-server --root public/ --compression -z

Explicit example (note the are no spaces in between):
static-web-server --root public/ --compression=false -z

Note also that this PR migrates to the latest clap v4.
Related Issue

Resolves #209 (post enhancement of #211)
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

@mac-chaffee It would be great a test from your side.

tested with --security-headers and --security-headers=false and it worked

tested with --security-headers and --security-headers=false and it worked

I also will send another PR to fix --security-headers not set in config.


[6467] Changing a placement adds a new one instead

Context
Changing a placement
Changes proposed in this pull request
Amended form to be explicit with regards to new or edit of a stash placement
Fixed placement numbering to be consistent with stash and saved items
Amended ordering to be consistent
Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Bug party 6: 'changing' a placement adds a new one instead


Add in filters

Medium term project, add in optional filters to manage larger datasets, starting with time and geography, but eventually integrating arbitrary filters.

I'm going to be doing a fair few changes going forward, which will split off into new sub-issues, in order to bring this forwards.
In order for users to meaningfully constuct their own filter - they are going to need a way to query the overall structure of the dataset. dataset_dimensions is the first part of this. I'm just putting together a new function that will return all the unique values of a dimension. Once that's complete, the next step is going to be a function that will return the dimensions and unique values of a dataset as a single function.


First basic unit test for LDAP auth

Description
Add basic unit test (with mocking ldap) for ldapAuth function.
Fixes # (issue)
Type of change
Add basic unit test
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

Testing steps
Run go test ./...

/test


FLEX-1972 ~ Moves authentication code from Shared to webservice-client

All is well.

All is well.


Bug 1985852: Set CPU and memory requests for webhook

The webhook is not currently deployed (see https://github.com/openshift/vmware-vsphere-csi-driver-operator/pull/34), but once we enabled it, it  should not run as BestEffort qosClass.
CC @openshift/storage

/bugzilla refresh

/test all

/retest

/lgtm


Wallet extension not opened on browser

The documentation suggests to run the docs locally in order to try out the contract. But, selecting either Nami or Ethernl, after running it locally, doesn't trigger the wallets to open.

Weird, which version of Atlas are you using? Is it the latest, 0.4.0?
Also, unfortunately, the docs are outdated and need to be updated in sync with recent Atlas changes. This is prioritised on our end.

Hi @sourabhxyz, thanks for the response.
Yes, I'm using the release version v0.4.0... Looking forward to a fix.
Thanks, once again.

Thanks for letting me know @Fiftyw3bs, I'll take a look on Monday 👍

@Fiftyw3bs wallet popup is suppose to happen when you initiate an operation. For instance, for operation, "Enter the address to put Reference Script at", does wallet popup happen when you enter address and click on "Submit"? Note that wallet logic is available here.

HI @sourabhxyz, it turns out I was submitting the wrong form.


Incorrect 'module' Field and Missing 'types' Field in 'exports' Causing TypeScript Errors in ES Environment

Describe the bug
When trying to import the 'vue3-google-signin' library in an ES environment, I encountered a type error indicating the absence of a declaration file for the module. The detailed error message is as follows:
Could not find a declaration file for module 'vue3-google-signin'. '/Users/junolee/Documents/mine/google-news-rss-pusher/node_modules/.pnpm/vue3-google-signin<EMAIL_ADDRESS>implicitly has an 'any' type.
  There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The 'vue3-google-signin' library may need to update its package.json or typings. ts(7016)

To Reproduce
Steps to reproduce the behavior:

Set up a project with the 'vue3-google-signin' package.
Attempt to import the package in an ES module environment.
Observe the TypeScript error related to type declarations.

Reproducible Example
Currently, I do not have a reproducible example on platforms like StackBlitz or CodeSandbox. I will update this section if I am able to create one.
Expected behaviour
The expected behavior is for the module to be imported without any type errors, with TypeScript being able to find and use the declaration files correctly.
Additional context
To resolve this issue, the types field should be added under "exports" > "." in the package.json. Additionally, the module field is incorrectly pointing to a CJS module. It should instead specify the entry point for an ES module, typically using a .mjs or .js extension.
Contribution Proposal
I am considering contributing to the 'vue3-google-signin' library to address this issue. I would like to propose corrections to the package.json file, particularly adding the types field under "exports" > ".", and correcting the module field to accurately point to an ES module entry point. I wanted to inquire if contributing these changes would be appropriate.

Hi @zuhno thanks for reporting the issue. you can create a pr if you like to contribute to the project

@kasvith I am interested in contributing, so I have opened a PR(https://github.com/wavezync/vue3-google-signin/pull/59). Please review it :)

Closed by #59

@coverrt I have noted your comments as additional context in the issue(https://github.com/wavezync/vue3-google-signin/issues/60). Thank you for reminding me again.

Hi @zuhno can we get this fix on a separate PR?

@kasvith Hi
I have submitted the PR(https://github.com/wavezync/vue3-google-signin/pull/62) for version 1.3.4. This is because version 1.3.3 has already been released.

this should be fixed by now


ppx_irmin: add irmin-type as a runtime dependency

This will cause Dune to automatically add irmin-type as a dependency wherever the PPX is used.

No longer needed thanks to the Repr extraction.


Write stderr to Logstash

Create a logstashWriter that implements io.Writer.
Then one could simply
log.SetOutput(logstashWriter)

and have logstash persist the errors, while simply using golang's native log commands Print, Fatal,...
log.Print("This message will be sent through logstash!")

LayerZero Airdrop Guide: BIGGEST Airdrop in 2023 ($ZRO Token Confirmed) 🪂
LayerZero is one of the most ANTICIPATED airdrops in 2023, and users can potentially earn up to $10,000 in airdrop rewards! This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Claim Now on Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Share on Social Media:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

Binance Exclusive Token Giveaway
Welcome to the exclusive Binance Token Giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this incredible opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Seize the moment and indulge in your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share this Token Drop

2. Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected, go to Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

4. Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

5. Share the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1500 in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @narr, @JerryCastanos, @Abeautifulsnow, @xdef, @oudommeas, @tucq88, @ilya-bobyr, @smsimone, @salgua, @nxm, @rvbansal, @JulioCVaz, @s3nt3, @yesh0907, @mrmign, @dimorphic, @hcxiong, @aleclarson, @SuperSine, @tmpsn, @mylittlegh, @MuratGundes, @xz-dev, @0xdariel, @kvnyu24, @futuretech6, @robbiewain, @CryptGodSon, @loveyandex, @sandyz1000, @senthil28, @andreoav, @DINGYIOK, @praveenperera, @fatalbar, @chandraprakash, @hanhungsoo, @liushenr, @butonium, @hytalo-bassi, @tyearly, @yezz123, @Shaunmak1214, @aoudiamoncef, @Tienisto, @DeemanOne, @taha-lyousfi, @vonlooten, @coldnight, @a17

@tim-tepia Reported due to sharing of phishing links.

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @Sandalots, @KiRist-code, @khoaguin, @Hafanul, @Makerosophy, @aloptrbl, @andreadito, @doglex, @heyiamraghu, @jTanG0506, @ManWithA1000Names, @StricklandF, @aeksco, @basicsharp, @ixiejun, @crosserclaws, @rorosan, @jrepp, @Dadiselef20, @mohsenusr, @daviddao, @akulich, @jessb321, @Harry-Hopkinson, @JuanDelgadillo, @Evalir, @akashchakravarti, @suite, @saravask, @sadeghbarati


Package request: vimix-cursor-theme

Project description
This is an x-cursor theme inspired by Materia design and based on capitaine-cursors.
Metadata

homepage URL: https://github.com/vinceliuice/Vimix-cursors
source URL: https://github.com/vinceliuice/Vimix-cursors
license: GPL-3.0
platforms: all

Closing, added in #255963


fix: Fixes table editable cell styles and simplifies it

Description
The PR refactors table body cell styles with the focus on editable cells. That is needed for the follow-up task to move editable cell style from the cell element to the cell content instead. This would allow having both row expand toggle and editable cell trigger in a single cell (in 1st column).
How has this been tested?

Screenshot tests
Manual testing on the cell-permutations page

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Found 2 issues with manual testing:

can not open select dropdown

https://github.com/user-attachments/assets/6682b553-b18c-44bd-89eb-2d8266ff114a

the input position seems a bit off

@georgylobko, @YueyingLu
I added a new commit with the following changes:

Reverted changes to negative vertical space. That is because in classic+compact the focus outline can be still cut off if the button is rendered inside content and the rows only have text content. The vertical paddings we use for table cell are under 2px whilst the focus outline offset for buttons is 2px - hence we need to do a trick so that the focus outline can fit.
Reverted changes to visible overflow in editable cells. When a cell is editing the overflow is allowed so that components with dropdown like Select can be displayed correctly.

I will re-run the screenshot tests for these changes, too.


Combine istioctl upgrade and help should offer same args (e.g. --set as manifest apply

Should these be aligned?  apply lets me set things, upgrade does not.
  Need test cases for apply removing components.

These now offer the exact same flags, closing


主流水线缓存配置key移除go版本区分

由于GitHub默认的缓存容量只有10GB，如果缓存过多，则会被清理而导致缓存失效，无法获取缓存的加速效果。

Codecov Report
Base: 79.56% // Head: 79.60% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (9c4e1f5) compared to base (ad737de).
 Consider uploading reports for the commit 0d7675f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2463      +/-   ##
==========================================
+ Coverage   79.56%   79.60%   +0.03%
==========================================
  Files         599      599
  Lines       49777    49777
==========================================
+ Hits        39605    39624      +19
+ Misses       8180     8162      -18
+ Partials     1992     1991       -1

Flag
Coverage Δ

go-1.15-386
79.63% <ø> (+0.03%)
:arrow_up:

go-1.15-amd64
79.63% <ø> (+0.01%)
:arrow_up:

go-1.16-386
79.60% <ø> (-0.01%)
:arrow_down:

go-1.16-amd64
79.63% <ø> (+0.04%)
:arrow_up:

go-1.17-386
79.65% <ø> (+0.03%)
:arrow_up:

go-1.17-amd64
79.60% <ø> (+0.02%)
:arrow_up:

go-1.18-386
79.56% <ø> (+<0.01%)
:arrow_up:

go-1.18-amd64
79.55% <ø> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

contrib/registry/zookeeper/zookeeper_watcher.go
51.11% <0.00%> (+2.22%)
:arrow_up:

os/gfpool/gfpool_file.go
68.57% <0.00%> (+2.85%)
:arrow_up:

os/gfsnotify/gfsnotify_watcher_loop.go
84.87% <0.00%> (+6.72%)
:arrow_up:

os/grpool/grpool_supervisor.go
100.00% <0.00%> (+66.66%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix typo in "unexceptional" docs

The word "unexceptional" was misspelled and the description of usage for the :unexceptional-status was missing not.

Thanks for fixing this!


Error: Additional properties not allowed: enabled' on property error_page

Current example will trigger the following error on version 2.8:
Payload validation error: 'Additional properties not allowed: enabled' on property error_page (Tenant error page customization).
To resolve this, we removed the "enabled" on the error_page.json.

Thanks for reporting this issue!
API2 doesn't support the enabled property which you can see here: https://auth0.com/docs/api/management/v2#!/Tenants/patch_settings
Our docs were recently updated to reflect this.
Thanks!

Thanks for updating the documentation!  I think it would be helpful to remove the "enabled" attribute or even the error_page.js in this repo so that users would not see a warning.


Refactoring Cluster to own a HostSet rather than implementing HostSet

Description:
Refactoring Cluster to own a HostSet rather than implementing HostSet
This is prep-work for #1354 - logically a cluster will have a primary HostSet and a failover HostSet and it can't very well do that with an inheritance relationship.
Alternatives considered
Having HostSetImpl have a failover_ set.  and a healthy_failover set.  And then failover_by_locality_ and healthy_failover_by_locality_ and updater functions for both and no, really, we should probably have two host sets.
Risk Level: Low to Medium
Fairly large refactor but there's no functional change expected, and the main code change is to how ClusterImplBase gets its stats updated.
Testing:
Refactor: simply ran existing tests.
Verified that without the changes to the ClusterImplBase constructor that tests failed (stats are checked in at least load_stats_integration_test)

FYI long convo happening offline as to whether this is the right approach or just do https://github.com/envoyproxy/envoy/issues/1929.

@alyssawilk do you mind summarizing where we ended up on this to make sure this is OK with the larger community? I think this will end up solving #1929 which is great.

The summary would be
"we're going to have N priorities rather than n=2 ('primary' and 'failover') in this PR" which is probably a little too tl;dr.  Summing up more of the conversation:
This requires a change to the EDS config to remove the concept of failover endpoints, since that naturally scoped to n=2.  The current proposed solution is to stick a "priority" field in LocalityLbEndpoints, to allow for arbitrary priority groups as requested in #1929.
There was a bit of debate over that - in code it makes sense to structure this as a tree with the first level being divided by priority, and the second level by locality, so @hutch and I leaned towards making the tree structure evident in the config, but @mattklein123 had a compelling argument that many users don't care about this sort of feature, so keeping the config simpler would be optimal for a larger set.  Given the structure of the code is orthogonal from the structure of the config the current plan is to stick with the simpler config.
Meanwhile in code, it's clear (to the parties involved at least =P) that we'll want the tree structure.    It doesn't much matter if we change how hosts are arranged in the HostSetImpl (the host set has PriorityGroups which contain the current hostPerLocality) or if we add priorities above the HostSetImpl (We have a HostAndPrioritiesSet which maps priority to HostSetImpl which continues to own hostsPerLocality).
The load balancer will have to change to be priority-aware (first pass will be doing the horribly-naive thing docced in EDS config and hard-failing from priority A to priority B iff all endpoints in A are down, better algorithms with gentle failover to follow).  The lb-updates will likely have to be priority aware.  Stats (such as "how many healthy endpoints do I have" should be priority-aware since "how many healthy endpoints do I have in my primary cluster" is generally more useful than "how many do I have across all priority groups").  Given that I'm currently inclined to stick the new level above the current HostSet Impl as I think the refactor will be easier that way.
In the long run we'll likely want to be able to update parts of the tree independently (changes to one Locality would ideally not affect all other localities across all other priorities) and we'll definitely need the bulk of the work to be done in the one thread rather than duplicated in all of the workers.  The first step will be the code restructure (which shouldn't make things more expensive as everyone will still only be creating a tree with the first level having one node) and the rest will follow as folks need optimized code.
I'm going to start in on this as I think the overall plan is sound but I encourage folks who are interested to weigh in early and often.  As Matt said this should address both #1929 and #1354 (anyone who wants primaries and fail-overs can limit themselves to 2 priority levels) but if there's other features the design points don't address in the long run, better to hash them out sooner rather than later.

Thanks for the detailed summary @alyssawilk!

Drive by comments:
Circuit breakers apply at cluster or priority or locality level or host level? I think it’s host level as it exists today
This is strictly a fallback thingy (use next prio only if former fails for some definition of failure - panic threshold gets complicated as matt said). Are we going to remember the state of a priority group for fast fallback for subsequent connections ? And then track that something higher prio is available again and try that for subsequent requests? If we cache, then aggregating per thread host prio into global view becomes tricky.
If we had true weighted load balancing (not the optimized one), can we achieve the priority effect by auto adjusting host weights ? Like lightly loaded or healthy hosts bubble to the top.
What’s the rationale for priority->locality->host over locality -> priority-> host ?
How does this affect the subset load balancer impl that has concept of default cluster? Is that what you referred to as existing impl?

@alyssawilk should we keep this PR open to preserve history or move it to an issue?
@rshriram circuit breakers (https://github.com/envoyproxy/data-plane-api/blob/master/api/cds.proto#L41) operate at a cluster + "priority" level today. However, the priority here is different to the priority being proposed by @alyssawilk (https://github.com/envoyproxy/data-plane-api/blob/master/api/base.proto#L77). This priority is closer to QoS than to endpoint fallback selection, which is what the current proposal is about.
I think we prefer priority->locality->host over locality -> priority-> host because this gives you greater control over directing traffic across localities in scenarios where you have close to total failure of the set of primary endpoints.

I was planning on doing the update in this branch but it'll be next week at best.  I'll close for now and reopen when I have something worth looking at


⚠ Carbubu has degraded performance

In cabd323, Carbubu (https://carbubu.fr) experienced degraded performance:

HTTP code: 200
Response time: 6042 ms

Resolved: Carbubu performance has improved in 2cc84a2 after 6 minutes.


Add ADOPTED to g7:enumset-NAME-TYPE

Since adoption is a concept, which is already used in several parts of the GEDCOM specification, it should also be available within the name type.
Therefore, ADOPTED should be added as an enum value to g7:enumset-NAME-TYPE

Further possible proposals:
DIVORCED (Name after a divorce)
ESTATE (House name, farm name, name after moving into or marrying into a house/farm)
RELIGIOUS ( Religious name, name adopted after joining a religious order)
UNIFIED (Unified spelling for a family name)
VARIANT (Different spelling for a name, also spellings based on other languages such as Latin, French)

In the sandbox branch, PR #349 proposed adding ADOPTED to g7:enumset-NAME-TYPE.


Questions about the success rate of experimental results

In Table 7 of the paper, I have several questions to consult you, which are not consistent with the experimental results I have obtained.
Q1: For the code you provided, I did about 2 experiments on average, but the results of these two times are not as good as those provided in your table. May I ask whether the results in the table mean that you conducted separate debugging for each task?
Q2: Similarly, I found that for the success rate, that is, the data calculated in the table, is it the average success rate in the 3000epoch, or the highest success rate, or the success rate evaluated after the last training for the 2800epoch?
Q3: Similarly, I also found that the results may not always be integers, and what does the '+-' symbol in your table mean?

Thank you for your feedback.
A1: Our experimental results are the average S5 results of the same hyperparameters under three different seeds, without any fine-tuning for a specific task. If your experimental results differ significantly, there may be some problems with the parameters or simulation environment.
A2: The S5 result follows the DP3 work, which means that there were 15 evaluations during the model training process, and we chose the highest five to take the average.
A3: The success rates in the table have been rounded to the nearest integer. The '+-' in the table represents the standard deviation.


os/mac/version: remove Yosemite.

As of 3.5.0, it's no longer possible to run Homebrew there.

Review period will end on 2022-06-06 at 18:21:07 UTC.

Review period skipped due to critical label.

Las time we provided a deprecation period: https://github.com/Homebrew/brew/commit/b58fa4ebb1e6b918d8556cba629cb55b79d423f5#diff-fdfe1bae33efb16bb42dfacca5d41a0ef63d56719345539b63ac05e29b07cd7f

I think the depends_on deprecation part at least can be merged before 3.5.0.
I've already removed all depends_on Yosemite references in Homebrew/cask(-versions) since fixing them did not require breaking Yosemite support (depends_on macos: ">= :yosemite was effectively no-op). I've left everything else that involved actually removing Yosemite support until after 3.5.0, but none of them are to do with depends_on.

https://github.com/Homebrew/homebrew-cask/pull/124927
https://github.com/Homebrew/homebrew-cask/pull/124928
https://github.com/Homebrew/homebrew-cask-versions/pull/14044

Forgot about cask-drivers: https://github.com/Homebrew/homebrew-cask-drivers/pull/2865

Makes sense, thanks for the changes @Bo98!


Unable to attach to CoreCLR. Unknown Error: 0x80131c3c

.net6 version is used

unable to debug the project in LINUX MINT OS
kernel version-5.15
INTEL CPU i7
16gb ram
USING VSCODE VERSION-1.72.2
Installed .net6 by
sudo apt-get install dotnet6
Just created the console app and try to debug the hello world code.
BUILD WORKS
CODE RUNIG FINE
ON DEBUGGING  IS SHOWING
Unable to attach to CoreCLR. Unknown Error: 0x80131c3c
Following is my launch and task files
### LAUNCH.JSON
{
    "version": "0.2.0",
    "configurations": [

        {
            // Use IntelliSense to find out which attributes exist for C# debugging
            // Use hover for the description of the existing attributes
            // For further information visit https://github.com/OmniSharp/omnisharp-vscode/blob/master/debugger-launchjson.md
            "name": ".NET Core Launch (console)",
            "type": "coreclr",
            "request": "launch",
            "preLaunchTask": "build",
            // If you have changed target frameworks, make sure to update the program path.
            "program": "${workspaceFolder}/bin/Debug/net6.0/gini-projects.dll",
            "args": [],
            "cwd": "${workspaceFolder}",
            // For more information about the 'console' field, see https://aka.ms/VSCode-CS-LaunchJson-Console
            "console": "internalConsole",
            "stopAtEntry": false,
            "targetArchitecture": "x86_64"
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach",
            "targetArchitecture": "x86_64"

        }
    ]
}

### TASK.JSON

{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "build",
            "command": "dotnet",
            "type": "process",
            "args": [
                "build",
                "${workspaceFolder}/gini-projects.csproj",
                "/property:GenerateFullPaths=true",
                "/consoleloggerparameters:NoSummary"
            ],
            "problemMatcher": "$msCompile"
        },
        {
            "label": "publish",
            "command": "dotnet",
            "type": "process",
            "args": [
                "publish",
                "${workspaceFolder}/gini-projects.csproj",
                "/property:GenerateFullPaths=true",
                "/consoleloggerparameters:NoSummary"
            ],
            "problemMatcher": "$msCompile"
        },
        {
            "label": "watch",
            "command": "dotnet",
            "type": "process",
            "args": [
                "watch",
                "run",
                "--project",
                "${workspaceFolder}/gini-projects.csproj"
            ],
            "problemMatcher": "$msCompile"

        }
    ]
}

That error code is CORDBG_E_DEBUG_COMPONENT_MISSING, which means that the .NET's debugging services dlls (libmscordbi.so and libmscordaccore.so) couldn't be found (or loaded).
Are you publishing a self-contained app? Or is your app using the shared framework? If you haven't modified your project file to do something special, it will default to the shared framework.
Do you have those .sos? Here is where they are on my Ubuntu 22.04 system:
/usr/lib/dotnet/dotnet6-6.0.109/shared/Microsoft.NETCore.App/6.0.9/libmscordaccore.so
/usr/lib/dotnet/dotnet6-6.0.109/shared/Microsoft.NETCore.App/6.0.9/libmscordbi.so

/usr/lib/dotnet/dotnet6-6.0.109/shared/Microsoft.NETCore.App/6.0.9/libmscordaccore.so
/usr/lib/dotnet/dotnet6-6.0.109/shared/Microsoft.NETCore.App/6.0.9/libmscordbi.so

@gregg-miskelly  NO i have found no shared libs in those directories.

Issue fixed by uninstalling prev installed dotnet6 with package manager and  following steps :

Installing manually tarbal latest 6.0 and extracting to a path .
Setting the path value which  containing dotnet bin   to DOTNET_ROOT env variable in ~/.bashrc
restarting vscode fixed  that issue.

I have the same error scince dotnet 6 up to current dotnet 7. I don't know where it tries to find these libraries so I don't know where to put them.
I tried to create /usr/lib/dotnet/dotnet7-7.0.0/shared/Microsoft.NETCore.App/7.0.0/libmscordbi.so
files and all of these, but omnisharp path for searching these libraries is mysterious to me. It just cannot find them whatever I do


tabular data/ noisy instances

Hi,
thanks for sharing your implementation. I have two questions about it:

Does it also work on tabular data?
Is it possible to identify the noisy instances (return the noisy IDs or the clean set)?

Thanks!

Hi,

It should be able to work on tabular data. There is no constraint on the types of input data it works on.
We did not test such an idea. But you may start by looking into the loss values for individual samples. Higher values may reflect that they could be potentially noisy.

I see, thank you! Last question: your code is not tailored to the few datasets you test, right? It seems that only dataset.py contains special datasets and hard-coded parameters.

There are some hard-coded parameters in the dataset.py for asymmetrical noise. For noise generations on your dataset, you may need to change them.

thank you!


Use json to build translate form body

Hi @Avimitin, wondering if you could comment on this approach to sending a json body. To make it work I did the following

derive Serialize for TranslateRequester
change field text in translate requester to have type Vec<String>
fix the serde "rename" attribute for Lang (otherwise SCREAMING-KEBAB case wants to hyphenate every letter)

fixes #25

Thx for your contribution! This looks great! I don't have time for refactoring this library for a while and you help me a lot.


is-sql Assertion not working

Describe the bug
When using the followin sql assert something fails.
To Reproduce

Add the following assertion to your test suit: ]

- description: "Specific SQL Assertion"
  vars:
    custom_instructions: |
      Can you give me all entries ?
    database: file://sql/data_table.sql
  assert:
    # Ensure output is a valid SQL query
    - type: is-sql
      value:
        databaseType: "MySQL"
        allowedTables:
          - select::null::data_table

Run npx promptfoo@latest eval -c promptfooconfig.yaml --no-cache --output=output.json
Look with view in the results
See error
`
SQL validation failed: authority = 'select::null::data' is required in table whiteList to execute SQL = 'SELECT "data" ..
Expected behavior
This assertion passes.

Promptfoo version:

Always using the latest version
Additional context
I have a proper ddl.sql meaning a proper CREATE TABLE data_table.
I seem to had this working for other application but now it fails and I can't figure out why.

Related to #1128

Hi @SysOverdrive,
Thanks for reporting this issue and for being such a power user of promptfoo! I'm sorry we haven't been able to address it yet, but it's on our backlog.
@YingjiaLiu99 wrote the original feature. @YingjiaLiu99 would you mind taking a look at this and #1128 if you have some time?
Thanks again!

Hi @SysOverdrive,
Thanks for reporting this issue and sorry for the late reply.
I just replied to #1128 and I hope it helps and provides some insight.
As for this issue, it seems that the result LLM query is trying to access data, which doesn't match data_table. This could be due to the following reasons:
If data is a different table from data_table, you'll need to add it to the allowedTables like this:
- select::null::data_table

Could you please provide the full SQL query statement and any other relevant details? That would be very helpful for me to investigate the issue.
Thanks again for your report, and I appreciate your patience while we work through this!


Add  defintion of table value functions

Is your feature request related to a problem? Please describe.
Unfortunately, the definitions of table value functions are currently missing
Describe the solution you'd like
inlcude table value functions (both inline and multi-statement)
Describe alternatives you've considered
none

Hey @ozz-project, thanks for the issue! I don't have a ton of time to do feature work on this right now, but if you want to open a PR I am happy to work through it with you!

I have unfortunately fixed it in a local repro. So I can't send it as a PR. Therefore as a screenshot. There are only a few characters in the sp_doc.sql


no one .ino appears

after executing miduino it just tells that:
This is a midi file
Single Track Midi File
Number of tracks = 1
Time division = 480
Successfuly Read Midi Header
Reading track
Track length = 5920
Meta Event
Length of meta event : 16
Meta Event
Length of meta event : 16
but I can't find .ino file

Hi Roman,
It looks like the program is only reading the first two events in the midi
file and then stopping for some reason. Would you be able to send me the
midi file in question and I can take a look and maybe find the bug with my
code?
Regards,
Jack
On Jul 18, 2015 8:15 AM, "Roman Отряскин"<EMAIL_ADDRESS>wrote:

after executing miduino it just tells that:
This is a midi file
Single Track Midi File
Number of tracks = 1
Time division = 480
Successfuly Read Midi Header
Reading track
Track length = 5920
Meta Event
Length of meta event : 16
Meta Event
Length of meta event : 16
but I can't find .ino file
—
Reply to this email directly or view it on GitHub
https://github.com/mycroftHo/miduino/issues/1.

the same going with any of midi files. That one exactly is 20tx fox: https://freemidi.org/download-658 , also tryed on https://freemidi.org/download-1490-evita-movie-themes

OK I'll have a look and see what's going wrong. Will hopefully have a patch
to push soon.
On Jul 18, 2015 2:24 PM, "Roman Отряскин"<EMAIL_ADDRESS>wrote:

the same going with any of midi files. That one exactly is 20tx fox:
https://freemidi.org/download-658 , also tryed on
https://freemidi.org/download-1490-evita-movie-themes
—
Reply to this email directly or view it on GitHub
https://github.com/mycroftHo/miduino/issues/1#issuecomment-122597844.

My apologies for the delay in responding fully to this. This bug has now been fixed.


[bug]: remove_strings: it does not work

Check the bug has not already been reported

[X] I could not find the bug in existing issues

Edit function_name and the short error description in title above

[X] I wrote the correct function name and a short error description in the title above

What happened and what did you expect?
The function remove_strings does not work with some strings :
When used to remove brackets, it does not do it.
To be precise, here is what happens :
select bigfunctions.eu.remove_strings('test_-test test()', ['_', '-',' ','(',')']) as cleaned_string
will result in :
testtesttest()
And
select bigfunctions.eu.remove_strings('test_-test test()', ['(',')','_', '-',' ']) as cleaned_string
will result in :
test_-test test()

Congrats @shivam221098.
Great first contribution. 👏


Wood CN ratios max needs to be higher

Several spp have Wood CN ratios that exceed the max of 600.

I modified the code, Rob incorporated the change into the latest installer.  Done.


Drop databases after unit tests if the database instance is not running in docker

Hey, I just made a Pull Request!
This PR extends backend-test-utils to drop databases after tests has run if the database instance is not running in docker. This is to make sure that we don't inadvertently fill the database instance with bunch of test databases. It is assumed that if you give a connection string to external database then the database is more "permanent" that the one that has been launched in a container just for the tests.
Relates to the discussion here: #21619
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Gonna hand this over to you @freben as I know we discussed this before, think you have more context on this :)

Note the build errors about using const and running prettier on changed files (you may want to configure your editor to auto-format on save).

Requesting re-review @freben . I could not put ALL code in the TestDatabase class but now DatabaseManager remains unchanged.
I made the suggested improvements:

moved drop database functionality to mysql and postgresql drivers
removed all modifications from DatabaseManager
replaced boolean-flag with callback-style hook to easily separate container-based and envvar-based databases from each other.

not stale, waiting for review :/

not stale, just waiting for review :/


Icon request: Artisan as Laravel icon

There is Laravel icon for *.blade.php files, we can use it for Laravel artisan file. Maybe other color but the same shape.

@Grawl Since artisan is a cli script wouldn't it be better to use "run script" icon? Also, Laravel has a new/updated logo which will be released with v6 soon. See https://laravel-news.com/laravel-v6-announcement

run script is ok i think


[fix][broker] Added newline to end of standalone.conf

Motivation
Without a new line at the end of the file, running apply-config-from-env-with-prefix.py with a new property appends directly to the file so you end up with something like managedLedgerMaxUnackedRangesToPersistInZooKeeper=-1myotherproperty=value and startup fails
Modifications
Added a new line at end of file
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API: ( no)
The schema: ( no
The default values of configurations: ( no)
The wire protocol: ( no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: ( no)

Documentation
Need to update docs?

[X] no-need-doc
simple conf file change

that is missing from apply-config-from-env-with-prefix.py

Oh, gotcha. Do you mind creating a PR to fix it there too? :)

Oh, gotcha. Do you mind creating a PR to fix it there too? :)

No probs, created -> https://github.com/apache/pulsar/pull/15596


config.json file

I did everything that stood and got this, help? homebridge wont start because its a problem reading my config.json file
*** WARNING *** The program 'node' uses the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see http://0pointer.de/avahi-compat?s=libdns_sd&e=node
*** WARNING *** The program 'node' called 'DNSServiceRegister()' which is not supported (or only supported partially) in the Apple Bonjour compatibility layer of Avahi.
*** WARNING *** Please fix your application to use the native API of Avahi!
*** WARNING *** For more information see http://0pointer.de/avahi-compat?s=libdns_sd&e=node&f=DNSServiceRegister
[2018-4-24 19:17:51] Loaded plugin: homebridge-config-ui
[2018-4-24 19:17:52] Registering platform 'homebridge-config-ui.config'
[2018-4-24 19:17:52] ---
[2018-4-24 19:17:52] Loaded plugin: homebridge-magichome
homebridge API version: 2.2
[2018-4-24 19:17:52] Registering accessory 'homebridge-magichome.MagicHome'
[2018-4-24 19:17:52] ---
[2018-4-24 19:17:52] There was a problem reading your config.json file.
[2018-4-24 19:17:52] Please try pasting your config.json file here to validate it: http://jsonlint.com
[2018-4-24 19:17:52]
/usr/local/lib/node_modules/homebridge/lib/server.js:207
throw err;
^
SyntaxError: Unexpected token : in JSON at position 13
at JSON.parse ()
at Server._loadConfig (/usr/local/lib/node_modules/homebridge/lib/server.js:201:19)
at new Server (/usr/local/lib/node_modules/homebridge/lib/server.js:57:38)
at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:26:16)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (module.js:569:30)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:503:32)
at tryModuleLoad (module.js:466:12)
at Function.Module._load (module.js:458:3)

https://jsonlint.com/

I tried them, but they did not help. so yeah:(


Re-enable backend test

Uses qibo 0.2.11 which includes the qulacs deps bug.

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

Project coverage is 88.88%. Comparing base (d1fee5e) to head (808ab05).
Report is 72 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   89.09%   88.88%   -0.21%
==========================================
  Files           3        3
  Lines          55       81      +26
==========================================
+ Hits           49       72      +23
- Misses          6        9       +3

Flag
Coverage Δ

unittests
88.88% <ø> (-0.21%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add Spanish translation

Hello there,
First time committing a pull request, so hopefully everything is in order.
I've done the Spanish translation and localization. I used the names found in the official ESP books that I have available. I tried to use the exact terminology so automatic matching should work (I tested with umbral sight and it's working).
There is one case where I did not change the name of the feature ("Hollow One" in "/detection modes/divine-sense.js") as it's from a book who does not have an official ESP localization and I did not dare to use non official terminology.
Thanks for this module and for all your work.

Thank you for your contribution!
Among other smaller fixes, I made some changes to capitalization: some translations only capitalized the first letter, others nouns, verbs, and adjectives too. Now only the first letter is capitalized, but I'm not sure if this the proper capitalization. Feel free to correct this. Perhaps it was correct already, and I just messed it up.
I added "Aquel que está vacío" for the translation of "Hollow One", which is from the official translation of the MTG card "Hollow One". Let me know if that makes sense.

Hi, always happy to help!
In relation to the capitalization issue, that is a confusing one. In Spanish we don't usually capitalize every first letter of a word, only those that initiates the sentence as in your fix.
In the localized books those are all over the place they capitalize the first letter of a complete word of some features, spells or titles, but in other places it follows the correct grammar rules for Spanish, but your normalization is the correct way to write in Spanish in general. I just followed the book's styling to avoid any issues.
"Hollow One" as "Aquel que está vacío" it's a perfect fit. I completely forgot that Theros was a plane from MGT

Okay, thanks! Let's keep the capitalization then that only capitalized the first letter of the first word. This only affects the translations of the modes in the detection modes tab of the token config anyway. The automated detection of features/spell effects is not case-sensitive. So anyone that wants to can us different capitalization for the feature/spell names.

https://github.com/dev7355608/vision-5e/releases/tag/v1.12.0


CastDevices disappear after some hours, usually on the next day.

Describe the bug
CastDevices disappear after some hours, usually on the next day.
To Reproduce
Just wait for a long time, usually next day.
Expected behavior
CastDevice should always be shown.
Logfiles
[2021-02-24 11:23:58] #### Start preuninst... ####
[2021-02-24 11:24:08] #### Done preuninst ####
[2021-02-24 11:24:08] #### Start postuninst... ####
[11:24:08] User "airconnect" found - deleting it
[11:24:12] "airconnect" user folder "/var/services/homes/airconnect" not found - nothing deleted
[2021-02-24 11:24:15] #### Done postuninst ####
[11:25:45] start-stop-status - Creating "airconnect" user.
[11:25:46] Starting AirConnect ...
[11:25:46] Starting airupnp on <IP_ADDRESS>:49154
[11:25:46.555610] main:1415 Starting airupnp version: v<IP_ADDRESS> (Jan 14 2021 @ 22:18:02)
[11:25:46.555727] main:1423 no config file, using defaults
[11:25:46.557413] Start:1127 Binding to <IP_ADDRESS>:49154
[11:25:46] Starting aircast on <IP_ADDRESS>
[11:25:46.563478] main:966 Starting aircast version: v<IP_ADDRESS> (Jan 14 2021 @ 22:17:01)
[11:25:46.563597] main:974 no config file, using defaults
[11:25:46.563942] Start:727 Binding to <IP_ADDRESS>
[11:25:46] AirConnect successfully started on ip "<IP_ADDRESS>"!
[11:25:46.612670] AddCastDevice:664 [0xfb3ff0]: adding renderer (Chromecast Audio)
[11:25:46.612754] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[11:25:46.613676] AddCastDevice:664 [0xfb4988]: adding renderer (Black Nest Mini)
[11:25:46.613731] AddCastDevice:670 [0xfb4988]: creating MAC fb4992
[11:25:49.027110] rtsp_thread:356 got RTSP connection 12
[11:25:49.027518] read_line:1219 disconnected on the other end 12
[11:25:49.027576] rtsp_thread:371 RTSP close 12
[11:25:49.028382] rtsp_thread:356 got RTSP connection 9
[11:25:49.028614] read_line:1219 disconnected on the other end 9
[11:25:49.028665] rtsp_thread:371 RTSP close 9
[11:25:52.040972] rtsp_thread:356 got RTSP connection 9
[11:25:52.041166] read_line:1219 disconnected on the other end 9
[11:25:52.041217] rtsp_thread:371 RTSP close 9
[11:25:52.041886] rtsp_thread:356 got RTSP connection 12
[11:25:52.042108] read_line:1219 disconnected on the other end 12
[11:25:52.042163] rtsp_thread:371 RTSP close 12
[22:06:14.743961] CheckAndLock:259 device is NULL
[22:06:14.744124] CheckAndLock:259 device is NULL
[22:06:14.744259] CheckAndLock:259 device is NULL
[22:06:14.744304] CheckAndLock:259 device is NULL
[22:07:08.833721] CheckAndLock:259 device is NULL
[22:07:08.833835] CheckAndLock:259 device is NULL
[22:07:08.833938] CheckAndLock:259 device is NULL
[22:07:08.834080] CheckAndLock:259 device is NULL
[22:08:08.413748] CheckAndLock:259 device is NULL
[22:08:08.413824] CheckAndLock:259 device is NULL
[22:08:08.413917] CheckAndLock:259 device is NULL
[22:08:08.414073] CheckAndLock:259 device is NULL
[04:33:04.941891] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Chromecast Audio+) 0
[04:33:04.942297] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[04:33:09.051530] rtsp_thread:356 got RTSP connection 12
[04:33:09.051733] read_line:1219 disconnected on the other end 12
[04:33:09.051799] rtsp_thread:371 RTSP close 12
[04:33:47.393761] AddCastDevice:664 [0xfb3ff0]: adding renderer (Chromecast Audio)
[04:33:47.393859] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[04:33:47.393997] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Chromecast Audio+) 1
[04:33:47.394305] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[04:33:51.004528] rtsp_thread:356 got RTSP connection 8
[04:33:51.004761] read_line:1219 disconnected on the other end 8
[04:33:51.004831] rtsp_thread:371 RTSP close 8
[08:58:52.195839] mDNSsearchCallback:535 [0xfb4988]: removing renderer (Black Nest Mini+) 0
[08:58:52.196572] DeleteCastDevice:512 [0xfb4988]: Cast device stopped
[08:59:28.247738] AddCastDevice:664 [0xfb3ff0]: adding renderer (Black Nest Mini)
[08:59:28.247856] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[08:59:28.248001] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Black Nest Mini+) 1
[08:59:28.248510] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[10:36:49.739628] AddCastDevice:664 [0xfb3ff0]: adding renderer (Black Nest Mini)
[10:36:49.739837] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[10:36:49.740093] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Black Nest Mini+) 1
[10:36:49.740700] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[11:03:52.810809] AddCastDevice:664 [0xfb3ff0]: adding renderer (Black Nest Mini)
[11:03:52.810912] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[11:03:52.811050] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Black Nest Mini+) 1
[11:03:52.811314] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[11:03:55.673754] AddCastDevice:664 [0xfb3ff0]: adding renderer (Black Nest Mini)
[11:03:55.673836] AddCastDevice:670 [0xfb3ff0]: creating MAC fb3ffa
[11:03:55.673968] mDNSsearchCallback:535 [0xfb3ff0]: removing renderer (Black Nest Mini+) 1
[11:03:55.674626] DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped
[12:32:42.733069] CheckAndLock:259 device is NULL
[12:32:42.733176] CheckAndLock:259 device is NULL
[12:32:42.733292] CheckAndLock:259 device is NULL
[12:32:42.733435] CheckAndLock:259 device is NULL

Hardware (Device on which you are running AirConnect-Synology):

Synology NAS DS918+, Asus RT-AC86U
DSM 6.2.3-25426 Update 3

Package (which package from the releases page have you downloaded):

package name: AirConnect-x86-64-<IP_ADDRESS>-20210119.spk

Additional context
I have two devices, one Nest Mini and one Chromecast Audio. I found if I do not use both of them, all will be disappeared on the next day. If I keep using Nest Mini, I find it is still there on the next day, but Chromecast Audio is disappeared.
I have to stop the spa in NAS and start it again, and both will work for around one day.
I tried enable multicast routing in Asus router, it won't help at all. These are the settings:
Use DHCP routes: Microsoft (there is RFC3442 option to choose)
Enable multicast routing: Enable
Default IGMP version: IGMP v3 (have v1 and v2 to choose)
Enable Fast Leave: Disable
UDP Proxy (Udpxy): [empty]

I tried to restart Chromecast Audio, but I still can't see its Airplay counterpart.

Hi @eos1d3
Thanks for the detailed description of your problem and the information, that always helps a lot to narrow down the problem more precisely.
This looks like a multicast problem. IGMP Snooping / Multicast Routing is a little bit difficult to integrate with Airplay.
In my local environment i am using IGMP version 2, with version 3 i had similar problems as you.
In general you do not need to enable multicast for this purpose if you have all your devices in the same network subnet.
There exists a known bug with your router so that sometimes cast/upnp devices will disappear, for more information please take a look at this chapter in the readme

When players disappear regularly, it might be that your router is filtering out multicast packets. For example and testing, for a Asus AC-RT68U, you have to login by ssh and run echo 0 > /sys/class/net/br0/bridge/multicast_snooping but it does not stay after a reboot.

Maybe there exist new information about this problem for your router, maybe you can find more in google.
btw. in one of the next version you can decide during the installation if you want to install only aircast or airupnp or if you want to install both. I just don't have enough time to finish it at the moment.
René

Thanks! At the very beginning I did not enable multicast at all. But after seeing the problem, I enabled multicast settings but still get the same problem. I will try to use IGMP2 instead.
My all devices are in the same subnet. So that is strange to see the problem. And my problem is not the same as others. Both cast devices can last many hours (usually more than 10 hours). This happens repeatedly at least 3 times/3 days.
I am running Merlin firmware for Asus router, if nothing works I will try the multicast_snooping  command with auto-start script in the router.
I will report back later.

Hi, the same problem exists when using IGMP2.
When I tried echo 0 > /sys/class/net/br0/bridge/multicast_snooping but I got the error:
-sh: can't create /sys/class/net/br0/bridge/multicast_snooping: Permission denied
I am using the same and the only Asus router admin account. There is no root account. Any idea for this?

When I tried echo 0 > /sys/class/net/br0/bridge/multicast_snooping but I got the error:

Have you tried with sudo?
sudo echo 0 > /sys/class/net/br0/bridge/multicast_snooping
br0 should be your network interface (bridge).

Right, there is no sudo. I login Synology NAS again with account name, now it still shows
$cat /sys/class/net/br0/bridge/multicast_snooping
cat: /sys/class/net/br0/bridge/multicast_snooping: No such file or directory

Anyway, I did find something. I stopped the SPK and I run aircast-win on my Windows 10 machine in the same network, it has been more than 2 days and cast devices are still there. So original AirConnect on Windows platform does not have this issue.
I have two cast devices, Nest Mini and Chromecast Audio, I see only one entry of this after two days in AirConnect log file.
[03:12:02.169] mDNSsearchCallback:527 [022B8BF8]: removing renderer (Chromecast Audio+) 36483040
And unlike SPK version, there is NO DeleteCastDevice:512 [0xfb3ff0]: Cast device stopped after receiving mDNSsearchCallback.

It looks like some mDNS announces are not relayed. These mDNS packages have a time to live (ttl) configured but i don't know long the timeout is set for.
Do you have any managed switches in your environment? Or any differences in the connection from your windows pc and your NAS to your router? (Windows PC -> XX -> Router -> IPhone | NAS -> XX -> Router -> IPhone).
I also had a miss configuration in my environment setup in the past (Repeater in combination with a missing multicast configuration on my managed switches).
I think it would be best if you inform the developer @philippe44 of AirConnect directly (open an issue), so that he can check it again. not that there is a bug in the aircast package.
You could also try to raise the logging level to sdebug instead of the normal command line execution.
Maybe he can then also identify your problem more precisely if you upload the logfile.
/volume1/@appstore/AirConnect/aircast -b [synology device local ip] -l 1000:2000 -x<EMAIL_ADDRESS>-z -f "/var/log/airconnect.log" -d all=sdebug

I will keep testing for a while. Original AirConnect is not problem free. It won't disappear, but when I try to connect it, it just has no response.
I have a managed switch with VLAN support. But I will try router setting first as I just find it has Enable IGMP Snooping option in Wireless setting, not in Lan setting.  It was enabled before, so shall I  disable it to see any difference?

The problem may be caused by AirCast <IP_ADDRESS>. See https://github.com/philippe44/AirConnect/issues/330
I am running aircast-x86-64  <IP_ADDRESS> directly on my DS918+ without using SPK. It has been running more than 20 hours and everything is OK.
I will run it for some more days in order to confirm it is caused by <IP_ADDRESS>.

Hi, I am still running AirCast <IP_ADDRESS> (not SPK version) for 2 days in my Synology NAS. Both cast devices are still working. So it is very likely the problem is caused by <IP_ADDRESS>.

@eos1d3 Thanks for the debugging and clarification!
You are right, the problem was introduced in <IP_ADDRESS>, for more information please check this issue: https://github.com/philippe44/AirConnect/issues/330
I have update the spk package to the newest version <IP_ADDRESS>, you can find the release here: https://github.com/eizedev/AirConnect-Synology/releases/tag/<IP_ADDRESS>-20210308
Please give a short feedback if it works.

<IP_ADDRESS> fixes the issue after further test. Thanks!


Update IDE0040 options

Just removing a note about a potential future C# language feature which is in the past.
In this timeline at least.

This one is for you @PureKrome <3

@ msdocs-peeps .. @davidwengie beat me to the PR.
he also knows i get triggered by default implementations, so i was still shuddering/crying/in-a-state-of-shock-for-60-mins ... and he snuck this PR in during my meltdown.


Should the apache_httpd importer address all 15 MITRE CVE ID JSON File Format 4.0 comparators?

Based on my exploration of the .json files we're currently fetching, the apache_httpd advisories use 5 of the 15 comparators listed in the MITRE 4.0 DRAFT - CVE ID JSON File Format 4.0 version_affected table.  https://github.com/CVEProject/cve-schema/blob/master/schema/v4.0/DRAFT-JSON-file-format-v4.md#version_affected .
Our importer currently addresses only the 5 comparators currently used.  Do we need to address the other 10 as well?

Implement the ones you need or fail for now, unless the implementation is trivial.


Bump fauxton, docs for 3.1.1 release

This can't be released until https://github.com/apache/couchdb-documentation/pull/586 and https://github.com/apache/couchdb-fauxton/pull/1292 are merged and both repos are tagged/released correctly. In Fauxton's case this involves a grunt couchdb run, see https://github.com/apache/couchdb-fauxton/pull/1289

All done, this is ready to go.


i can't even

8ro I literally struggle with setting this thing up!!!!!!!
Instructions unclear, npm installed minecraft

you can
just do what instructions say

removed modclean


Update CLI help text to reflect removal of rebuild command

AIUI 873fc5c0db4876d9e1f347fa6cbd2a3a1933df69 removed the rebuild alias for build

Hi! Sorry to say I already have a fix in the pipeline (in a big commit due later today or tomorrow), as a part of a larger group of changes, so I'll have to turn it down, but thanks for the PR in any case!

No prob, thanks!


remove extra component in TimeSeriesChart panel

As a follow-up to the discussion here, this PR is a refactor of our TimeSeriesChart panel to remove the unnecessary container component.
Previously, since GraphQueryRunner passed data via context, but the chart needs it as a prop--I had to add an extra TimeSeriesChartContainer.tsx component as a workaround. This wasn't necessary in StatChart and GaugeChart since those panels only support one query.  Since then in #601, the TimeSeriesQueryRunner approach was removed in favor of react-query's useQueries (which was possible after PR #556)

Closing in favor of upcoming PR that also includes legend updates


Driver's Status in Championship.

In a championship ... Can a driver verify his situation? Eg. Accept,Reject and Skins.
Could it be a query button here?

Could the answer be with these fields? Only "Create", "Name", "Car/Skins" and "Status". ( No Guid, No mail)

this is fixed by #533


Investigate alternative ways to get the OS information on windows without relying on registry keys

Today, we use the registry to fetch the OS info.
https://github.com/Azure/azure-sdk-for-cpp/blob/76870c44144cc426f0a0c4d3104eeb915ac002b6/sdk/core/azure-core/src/http/telemetry_policy.cpp#L35-L42
Consider using an alternative like GetVersion (investigate other approaches that might be better suited):
https://docs.microsoft.com/en-us/windows/win32/api/sysinfoapi/nf-sysinfoapi-getversion
That said, we need to figure out how to handle differences between 8.1 and 10.
One approach could be to use the version helpers and specify the version strings explicitly (but we lose some fidelity of info):
https://docs.microsoft.com/en-us/windows/win32/sysinfo/version-helper-apis
With this approach, we could remove:
https://github.com/Azure/azure-sdk-for-cpp/blob/76870c44144cc426f0a0c4d3104eeb915ac002b6/sdk/core/azure-core/inc/azure/core/internal/hkeyholder.hpp#L31
cc @JeffreyRichter, @RickWinter, @antkmsft

We get a much richer set of information about the OS using the registry.  We could change this in the future as the telemetry is an implementation detail.


Jump to next problem always works.

Currently when I press the default keyboard shortcut of F8 to jump to next problem, it only does so if I have initially opened the problems view, and jumped to the first problem. The an only then, will F8 begin jumping to other problems.
The desired behaviour would be, no matter what state the problems panel or editor windows are in, if there is a problem, F8 will jump to it.

It works for me even if problems view is not opened.
Can you please provide steps to repro?

It seems like I have to close the current editor as well. Jump to first problem, close the editor window so no editor tabs are open, the hit F8. Nothing happens for me. If these steps do not reproduce it foryou, I will have to dig deeper.

F8 is supported only when you have an editor opened.


Adds back default go-licenses symlink

Issue #, if available:
Description of changes:
Add back default go-licenses symlink
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold

/lgtm
/approve


[FEATURE] More video embed types (Loom & Wistia)

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
At the moment, it is only possible to upload YouTube or Vimeo videos, Since Yoopta would be part of other software products, it would be interesting I think to be able to add Wistia videos and Loom videos by adding Loom or Wistia urls.

Proposed Solution
For example to detect wistia & loom urls, we can use this regex:
wistia: [%r{(?:https?://)?(?:www.)?wistia.com/medias/(.+)}]
loom: [%r{(?:https?://)?(?:www.)?loom.com/share/(.+)}]
The Wistia embed would look like this:

The Loom embed would look like this:

Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to the above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
I would implement the proposed solution by adding new embed providers, I might be a bit slow though while I get used to Typescript.

🚀 🚀 🚀

added in v4.9.3


Have a servidor?

help i hace. Servidor but a i donar hace reís un my red and hack ing my dns

@alexaguilar89 Can you add more information about the problem you're having with docs?


Revert "Specify LCM community for the video logger via environment viariable."

Reverts RobotLocomotion/director#141

hang on, if we want to make this more flexible then there is a little more work required.  I think everything does work correctly without modifying files, i may have just been a little confused when i typed some instructions on the other github issue.

What was your preferred setting of VIDEO_LCM_URL?

I don't know yet. We'll have a URL for the robot machine when we set it up but I was using just my localhost/default url for this for now, i.e.
VIDEO_LCM_URL=""

I pulled it out into environment variable so that I can set it in the pmd file (Same as the other URL set in the video_capture.pmd).

VIDEO_LCM_URL is supposed to be private to the video capture and playback tools.  It was selected to be different than any other URL, because we don't want gigabytes of GoPro video LCM traffic to be in the robot lcm community.
The URL selected for VIDEO_LCM_URL is hard coded in that script and in the pmd file.  I'm happy to make it an environment variable instead, but then we should change it everywhere, and make it mandatory.  But I'm not sure it needs to be user settable, it's just an implementation detail of the video capture tools.
I think these are the issues:

the pmd launches videoLogServer.py with LCM_DEFAULT_URL set to the same thing as VIDEO_LCM_URL.  That LCM_DEFAULT_URL that the videoLogServer.py tool is launched with is the url that it will use to listen for EST_ROBOT_STATE for playback syncing.  So as it is hard coded in the pmd, you must playback robot logs using VIDEO_LCM_URL for syncing to work, otherwise change the pmd, perhaps it should use the "review" url I mentioned yesterday.

the videoLogPlayer.py is supposed to be launched with LCM_DEFAULT_URL set to VIDEO_LCM_URL, which is the url everything else is launched with in the pmd.  The videoLogPlayer communicates with videoLogServer over this URL..

Of course, documentation is lacking, sorry about that.  You are the first person to attempt to use these tools outside our lab, where we only have one computer equipped with a capture device, and everything was designed to just work on that computer and then no more time was spent to make these tools general purpose.

the videoLogPlayer.py is supposed to be launched with LCM_DEFAULT_URL set to VIDEO_LCM_URL
This I didn't know but it makes sense now.
I got confused by the videoLogPlayer.py, because I started it without setting LCM_DEFAULT_URL. In which case I had to change the VIDEO_LCM_URL for  videoLogServer.py.

I'll run it again for my own sanity.
Where do you usually launch the videoLogPlayer.py? With the director or with the video logger?

On our setup, videoLogPlayer.py is launched using a gnome task bar launcher.  It is launched using directorPython and you should set LCM_DEFALUT_URL to VIDEO_LCM_URL ( udpm://<IP_ADDRESS>:7650?ttl=1 )

I just confirmed the tools are working without modifications, so I'm going to merge this revert.
Everything works as long as you start videoPlayer.py on the correct LCM url.
I think it would be beneficial to remove the hardcoded lcm url as you suggested and replace it with an environment variable.  In this case, I think the environment variable should be mandatory, and used in both videoLogServer.py and videoPlayer.py.


Fixed Bricks read only issue

I got an issue Bricks is read-only  because bricks is const at line 176 declared so it must be let

Please have a look

Interesting I thought I had merged something similar. Will merge, thanks!


print

maybe is a simple thing but I don't see it.
python print doesn't shows as output
I see there is a: from py4godot.utils.print_tools import print
but looks like it needs some kind of object as parameter.
Is there any way to print logs to console?
thanks
jorge

Hey,
thank you for submitting this issue. It seems like importing print from print_tools is broken. But you should be able to use print without importing it from print_tools. This code should print hello to the console:
# file: node3d.py
from py4godot.classes import gdclass
from py4godot.classes.Node3D.Node3D import Node3D
@gdclass
class node3d(Node3D):
	def _ready(self) -> None:
		print("hello")

I hope, this helps.

Hi,
Thankyou for your answer. You are right simple print() works
I am using python in godot to include some AI tools. When I imported pytorch hub I got this error:
    sys.stderr.write(f"Using cache found in {repo_dir}\n")
    ^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'write'

I've solved it patching sys.stderr and stdout as I guessed it was not available under godot.
stream = io.StringIO()
sys.stdout = stream
sys.stderr = stream

this obviously broke print().
Looks like sys.stdout was available after all and didn't need patch
thanks for the help


Option for VMSS single placement group and overprovisioning

Added option for single placement group in the config file

@JonShelley this looks good to me. in addition to the config.json file can you also add documentation in the readme for it as well as adding an entry in the changelog.md ?

@JonShelley this looks good to me. in addition to the config.json file can you also add documentation in the readme for it as well as adding an entry in the changelog.md ?

I have made the requested changes. Please review and approve if changes look good.


Expose proxies by default

Exposes all proxies by default when no id is specified via: Controller.1.proxies

@nxtum / @sebaszm there is probably a bit of a bigger story here. Lifetime :-) There was no addref in the past there is no addref now on the list that we passout, but Ithink it is a bit ticky (a larger flaw) The proxies are reference counted but as we pass them out here we do not increase the reference count which could be "scary"... Lets discuss tomorrow in the Sync up call..


[Q] are there really no other transcodes on odysee.com?

I'm not familiar with how this tool or odysee.com works.
Do odysee really not transcode videos to other resolutions or is it just a yt-dlp related limitation that you can't choose other formats? - when I run "yt-dlp -F https://odysee.com/@iamRucka:0/Like-a-Cop-Small:3" i get the following:
[lbry] @iamRucka#0/Like-a-Cop-Small#3: Downloading stream JSON metadata
[lbry] 3dd033ae623974565bf3900334bcd7828845683a: Downloading streaming url JSON metadata
[lbry] @iamRucka#0/Like-a-Cop-Small#3: Downloading streaming redirect url info
ERROR: No video formats found

youtube-dl reports it as format code 0
which there is no fix yet.

lbry should have 1 format. yt-dlp not showing it in -F is a bug

Do odysee really not transcode videos to other resolutions

Not afaik

I assume the question has been answered


Round progress a number larger than 100%

Hello,
I am running MPOS in Litecoin with testnet=1 for testing purposes, the round progress gives me very
large number, around 678057% i am wondering what's wrong this number should be less than or equal 100%, when u checked the code i found it calculated from this part of code
// Round progress
$iEstShares = $statistics->getEstimatedShares($dDifficulty);
if ($iEstShares > 0 && $aRoundShares['valid'] > 0) {
$dEstPercent = round(100 / $iEstShares * $aRoundShares['valid'], 2);
} else {
$dEstPercent = 0;
}
in dashboard.inc.php
is that normal or i need to fine tune anything, i am actually confused with the round progress number, but i have noticed that it's always less than 0 in running pools with MPOS

It's just an estimated based on block difficulty and number of shares found. Please keep in mind that for the latest Bitcoin RPC spec from Bitcoin 0.16 you may need to switch to the development branch or MPOS won't work properly/at all. This could result in no blocks being discovered by MPOS.
Please also check that your enabled your crons as described in the quick start guide or nothing will work and you rounds will just keep growing.


Login when MFA is enabled

When MFA is enabled, login will return:
{"mfa_type":"sms","mfa_required":true}
And then I get a text message on my phone.
I presume some OTHER URL is needed to take the SMS code.
it would be great to document this (although I'll just turn MFA off for now).

#14 Try this?

I'd rather not turn multi-factor on so if @tolidano tests and verifies the docs are correct, I'll accept pr #14.

I'll turn my MFA back on and modify my stuff to see if this works and report back here, before Sunday.

I can confirm this works as described. The token is returned when the mfa_code parameter is set to the code sent by SMS to the phone number on the account. Please merge that in. Thanks for figuring that out.

Thank you both!


chore: add github action for pub publish

Status
READY
Description
Getting an action in for publishing to pub.dev
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[x] ✅ Build configuration change
[ ] 📝 Documentation
[x] 🗑 Chore

Confirmed that pub.dev configuration is setup correctly as well.


CLOUDSTACK-10157: Wrong notification while migration.

Steps to Reproduce:

Create a cluster with three hosts, two of them with GPU cards NVIDIA GRID K1 and GRID K2 and one normal host.
Deploy a VM with GRID K2 vGPU.
Stop the VM and try starting it on a different host. (Expect it to fail as there is only one host supporting that particular vGPU type)
The log shows it as failed operation, VM remains in the stopped state, yet the notification shows "Start Instance" as successful and no error message displays.

Root Cause:
Earlier, it was failing with ArrayIndexOutOfBoundsException, when the list is empty and accessing the first element.
The error was only observed in Log, but was not showing in UI as it was not throwing any exception.
Hence the API call was in turn successful.
Solution:
Added the empty check before sending device details.
Which says either the required GPU device is not available or out of capacity.

Thanks, @rafaelweingartner !!

LGTM, simple NPE check.

Given our current regression testing system does not have GPU capabilties, I'll use travis test results as verification of the build/code, two reviews/lgtms and merge this PR.