Build Failing with the following error - can't find any fixes online

Yep, here too...
Just npm install and react-native link react-native-svg
Here is the error
SvgViewManager.java:31: error: <anonymous com.horcrux.svg.SvgViewManager$1> is not abstract and does not override abstract method measure(YogaNodeAPI,float,YogaMeasureMode,float,YogaMeasureMode) in YogaMeasureFunction
    private static final YogaMeasureFunction MEASURE_FUNCTION = new YogaMeasureFunction() {
                                                                                ^
/SvgViewManager.java:32: error: method does not override or implement a method from a supertype
        @Override
        ^
2 errors
:react-native-svg:compileReleaseJavaWithJavac FAILED

Can anyone help?

exact same issue here. please help.

I am also getting this error

+1 please help

I finally fix my issue.
I use RN 0.44, and update package.json into "react-native-svg": "react-native-community/react-native-svg#5.1.8",

+1

@pewh Thanks, this fix worked for me.

I'm having the same issue on v5.3.2 with react-native 0.44.
/Users/tastycode/workspace/countable-mobile/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/SvgViewManager.java:31: error: <anonymous com.horcrux.svg.SvgViewManager$1> is not abstract and does not override abstract method measure(YogaNodeAPI,float,YogaMeasureMode,float,YogaMeasureMode) in YogaMeasureFunction
    private static final YogaMeasureFunction MEASURE_FUNCTION = new YogaMeasureFunction() {
                                                                                ^
/Users/tastycode/workspace/countable-mobile/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/SvgViewManager.java:32: error: method does not override or implement a method from a supertype
        @Override
        ^```

@pewh solution doesn't work for me because react-native-svg isn't compatible with new changes in react-native 0.44, until v5.3.0 https://github.com/react-native-community/react-native-svg/issues/416#issuecomment-317899666. So locking to `5.1.8` is not an option.

Also verified this is happening in 5.3.0.

Reversing this commit seems to make the error go away: https://github.com/react-native-community/react-native-svg/commit/e125a28caa78934a928cc0f232da8bf399834a9d

Seems like an old version compatibility issue. Closing...


Update milvus-sdk-java to 2.4.3

About this PR
📦 Updates io.milvus:milvus-sdk-java from 2.4.1 to 2.4.3
📜 GitHub Release Notes - Changelog - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.milvus", artifactId = "milvus-sdk-java" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.milvus", artifactId = "milvus-sdk-java" }
}]

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

Superseded by #349.


Roll Engine from 92f4dfb0484f to 4cd3ab8d8e3d (6 revisions)

https://github.com/flutter/engine/compare/92f4dfb0484f...4cd3ab8d8e3d
2021-04-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from dl1JUKVuj... to 0Db2pEb0U... (flutter/engine#25467)
2021-04-07<EMAIL_ADDRESS>Roll Dart SDK from<PHONE_NUMBER>b0 to e778310fcdcc (1 revision) (flutter/engine#25466)
2021-04-07<EMAIL_ADDRESS>Roll Skia from d986f476ae68 to 0195bec798a9 (1 revision) (flutter/engine#25463)
2021-04-06<EMAIL_ADDRESS>Revert "Reland: Call Dart plugin registrant if available (#25375)" (flutter/engine#25462)
2021-04-06<EMAIL_ADDRESS>Roll Skia from 601abfacc736 to d986f476ae68 (5 revisions) (flutter/engine#25461)
2021-04-06<EMAIL_ADDRESS>Roll Dart SDK from b9471aadaa1a to<PHONE_NUMBER>b0 (1 revision) (flutter/engine#25459)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from dl1JUKVujkyS to 0Db2pEb0Ur2h
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


How to use avatar or logo field with form builder

How to use avatar or logo field with form builder

Please help

I closed it by mistake. The package can’t handle images.


Not recognizing monitoring plan

Describe The Bug:
When I add credentials to access my SimpliSafe account is says I don't have a matching plan and I have the Self Monitoring plan.

To Reproduce:
Restart Homebridge, error happens in log.
Expected behavior:
The Plugin sees my account has a plan associated with it and allows me to use my SS devices with the bridge
Environment:

Plugin version: 9.1.10
Node.js Version: v16.14.0
NPM Version:
Homebridge Version: 1.4.0
Operating System: Ubuntu

Homebridge Config:
{
"bridge": {
"name": "Homebridge BA29",
"username": "0E:44:23:D6:BA:29",
"port": 53303,
"pin": "031-45-154"
},
"accessories": [],
"platforms": [
{
"name": "Config",
"port": 8581,
"auth": "form",
"theme": "auto",
"tempUnits": "f",
"lang": "auto",
"platform": "config"
},
{
"name": "Home Alarm",
"cameras": true,
"subscriptionId": "395923",
"sensorRefresh": 15,
"persistAccessories": true,
"resetSimpliSafeId": false,
"cameraOptions": {
"enableHwaccelRpi": false
},
"platform": "homebridge-simplisafe3.SimpliSafe 3"
}
]
}
Screenshots:
Not Applicable
Logs:

Link to logs
[homebridge.log.txt](https://github.com/homebridge-simplisafe3/homebridge-simplisafe3/files/8228350/homebridge.log.txt)

Hey @bigchrisatx , from your logs it looks like youre not running the newest version, is that intentional?
[3/10/2022, 7:43:46 PM] Loaded plugin<EMAIL_ADDRESS>Newest version is 1.9.10, perhaps reinstall?

What happens if you remove the account number from your config?

Account number removed, same result.

Would you feel comfortable editing the code a little to tell me what happens? Specifically, add this.log(data); to line 259 of simplisafe.js and restart then send output please.
Thanks

I added "this.log(data);" to the following location on the .js file.

Attached is the log out put after restarting.
homebridge.log.txt

Sorry thats not the right spot. You should be editing the file in .../node_modules/homebridge-simplisafe3/simplisafe.js (the location of your node_modules depends on your install process). Here's a screenshot of those lines so you can see

Here is the output:
subscriptions: [
{
uid: 4709237,
sid: 5034415,
sStatus: 7,
activated: 0,
planSku: 'SSSLFCAM',
planName: 'Self Monitoring with Camera Recording',
price: 9.99,
currency: 'USD',
country: 'US',
expires:<PHONE_NUMBER>,
canceled: 0,
extraTime: 0,
creditCard: [Object],
time: 2628000,
paymentProfileId: '2153105216',
upgradeStatus: 0,
features: [Object],
status: [Object],
subscriptionFeatures: [Object],
dispatcher: 'cops',
dcid: 0,
activationCode: '',
systemVersion: 20,
orderProductId: 0,
location: [Object],
pinUnlocked: true,
pinUnlockedBy: 'pin',
billDate:<PHONE_NUMBER>,
billInterval: 2628000,
autoActivation: null
}

I think I see the issue. BTW, I suspect that your account number is wrong and if you put in the right one it would work, but still this seems like the plugin should handle this better. I'll release a fix in a few, please tell me if it works for you.
Thanks for your patience

That worked, thanks!


Document undocumented table test helpers

This PR adds all the remaining test helpers from packages/tables/.stubs.php to the documentation.
In the Stub file, I moved the two label assertions next to teach other like all the other paired assertions.
This follows up on the work in #6262

Thank you!


warehouse: OIDC beta ACLs

WIP.
See #12965.

Just to summarize: this adds a new principal (group:oidc_beta).
Users are added to the group:oidc_beta principal under one of two conditions:

They're a superuser (User.is_superuser)
They've been explicitly added to the OIDC beta (User.has_oidc_beta_access)

Anybody in the group:oidc_beta principal has the new (temporary) OIDC permissions:

manage:user:oidc: view and manage "pending" OIDC publishers;
manage:project:oidc: view project-level OIDC publishers;
manage:project:oidc:modify: manage project-level OIDC publishers;

The last two permissions are separate so that users who aren't in the OIDC beta but have access to projects with OIDC publishers can see but not modify those publishers.

Just remembered another thing I need to do here: the admin panel should allow Warehouse's admins to enable/disable OIDC beta status for a user.

This should be good to go -- I've added the User feature flag plus an admin control for it, along with checks to each OIDC view (+ menu renderers).


Future-proof CI (getting the version)

📝 Summary

Upgraded Github CI with future-proof versions (previous method is going to be deprecated
Added more mev-boost flags to the release guide

✅ I have run these commands

[x] make lint
[x] make test-race
[x] go mod tidy

Codecov Report

Merging #406 (9d2e27f) into main (dd999ff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #406   +/-   ##
=======================================
  Coverage   66.15%   66.15%
=======================================
  Files           7        7
  Lines         904      904
=======================================
  Hits          598      598
  Misses        271      271
  Partials       35       35

Flag
Coverage Δ

unittests
66.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Code Editor and Record Actions

We finally enabled and got the  automate ribbon in excel for web enabled however both Record Actions and Code Editor sub windows are not loading/are blank. Is the some additional setting than we need to install/enable?
Additional info: The help sub window does populate correctly, so not sure.

Hi @LielanC,
Thanks for raising this issue. If the Automate tab is appearing, I'm assuming most of the requirements listed here are met, but are third-party cookies enabled for your browser? Also, which browser are you using?

Thanks Alex, that is correct we have met the requirements.
Currently due to group policies, they have disabled
"Allow sites to save and read cookie data"
"Block third party cookies"
on Chrome.
So it might be an option/ maybe another MS solution required? Will provide feedback if org allows it.

Would the site to be added be
https://appsforoffice.microsoft.com/
?

Thanks for the information @LielanC. @peronald, can you confirm if that's the site that needs to be allowed for Office Scripts cookies?

Hey @LielanC, you can find the site you need to add by opening a new workbook in Excel on the web and taking the part of the url before the first "/ ". So, for example, if the url looks like the below:
"https://contoso-my.sharepoint.com/:x:/r/personal/..."
you should add the "https://contoso-my.sharepoint.com/" as the site that can have third party cookies enabled.
If that doesn't work, could you provide the session id of the Excel on the web session? To get the session id, in the Excel workbook, go to File > About and then you should see the id there.


Give EPA/GSA users a helpful link to the login docs when they try to reset their cloud.gov password

In order for cloud.gov users to have a smooth experience logging into their cloud.gov accounts from both the web UI and CLI (including if they've forgotten a bit about how to use it): if they try to reset their password, the automatic error message should give them a helpful link to our docs that explain how to use SSO in the UI and CLI (https://docs.cloud.gov/getting-started/accounts/ and/or https://docs.cloud.gov/getting-started/setup/).
Right now, when people at GSA/EPA try to reset their password, the error message says

Your account credentials for login.cloud.gov are managed by an external service. Please contact your administrator for password recovery requests.

This message is confusing. What it should really say is more like this (just sample language; I'd want to rework this a little more when we actually implement it):

Hi! We received a password reset request for your cloud.gov account. Your cloud.gov account access is handled by your agency's account system, so there is no cloud.gov-specific password that you can reset. To log into the cloud.gov web interface or command line interface, use your agency credentials by using these instructions and commands: https://docs.cloud.gov/getting-started/accounts/

The default message comes from a Cloud Foundry template: https://github.com/18F/cg-uaa/blob/master/server/src/main/resources/templates/mail/reset_password_unavailable.html
Info about changing this from this Slack message:

We have overridden UAA templates for login pages, etc. on our fork via a hack to check the web/cg directory since there’s no native functionality to overload the templates as it stands. Seems like we need to do the same for mail.

And:

Would have to look at the commit history to see where [we] did the hack to look in the cg directory. Probably easy to extend/replicate.

Is there a reason we can't modify the template directly? Can someone tell me where the template is being overwritten right now?

Modifying the templates in situ will make it harder to merge upstream changes, particularly as UAA is expected to grow its own template overriding method at some point. We want to keep our UAA fork as mergeable as possible in case of security issues fixed upstream.
So just copy a template you want to change into the cg/ subdirectory and do your mod there.


Handlers get overwritten

Currently the iffe resource loader does overrife the handler options. This should be resolved!

@SheepFromHeaven Is this still an issue?


form-control-success is required to display the validation icon

In the documentation , it mentions that

To use, add .has-warning, .has-danger, or .has-success to the parent element. Any .col-form-label, .form-control, or custom form element will receive the validation styles.

But, if we only add .has-success to the form-group div, the form-control-success will still need to be manually applied to the input textbox to show the validation icon.
Have I misunderstand the documentation?

It is not mandatory to provide "form-control-success" on the input fields because adding .has-warning, .has-danger, or .has-success to parent element provides style like border color to the input field where as form-control-success adds a background-image like a check.
Finally, both the classes are provided to add validation styles to the input elements but need not to be together.
.has-success .form-control-success {
background-image: url(data:image/svg+xml;charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' v… 1.4 3.4-3.8c.6-.63 1.6-.27 1.2.7l-4 4.6c-.43.5-.8.4-1.1.1z'/%3E%3C/svg%3E);
}
.has-success .form-control {
border-color: #5cb85c;
}
.form-control-danger, .form-control-success, .form-control-warning {
padding-right: 2.25rem;
background-repeat: no-repeat;
background-position: center right .5625rem;
background-size: 1.125rem 1.125rem;
}

Revisiting validation states in #22762.


New variables and ability to load images randomly from a custom directory not only game images.

It would be great if there is a variable that contains the name of the system for a selected game in the detailed and grid views to be able to load a logo of the system to which the selected game belongs.
for example: game.system.name and game.system.fullName
Other great feature that would be great is the ability to load randomly images from a custom directory not only game images. For example:
`

<!-- Load a system logo based on the selected game -->
<image>

    <path>./assets/${game.system.name}.png</path>

</image>

<!-- Load images randomly from a specific directory -->
<image>

    <path>./assets/banners/${system.theme}/{random}</path>

</image>

`
Thanks!!!

I agree with the system logo thing per game in gamelist, it can be a very good resource, especially in collection lists.

There are dynamic variables now.


api: Add unreadMsgs in initial snapshot

Our own data structure in PerAccountStore will be constructed from this value, and it will probably have quite a different shape; for example, we're likely to want to collapse the distinction between 1:1 and group DMs.
But now we're validating this part of the /register response and making it available to feed into those new internal data structures.
Related: #253

Thanks! Revision pushed.

Thanks! Looks good; merging.


Support run on multiple solutions

Sometimes whole project has different solutions (to speedup development). In this case, solution cop should be run on the all of them (of course, rules are the same, however each solution is checked independently).
Nice to have: parallel check, instead of sequental check.

How much time will you win from parallel checks in your case?

@Litee , I'd like to have one script for all solutions. For my opinion it is better to run all in parallel which is a minor improvement for virtualized environment (when VM host uses HDD, which is virtualized and shared between separate VMs, moreover sometimes different agents works on the same computer, and IO is a bottle neck)


Android Basics: change-app-theme

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-change-app-theme?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-change-app-theme#8
In which task and step of the codelab can this issue be found?
9 Challenge Task
Describe the problem
MaterialThemeBuilder is not in the given GitHub repo. Also the provided link for the MaterialThemeBuilder README.md leads
to a 404 GitHub page.
Steps to reproduce?

Go to "9 Challenge Task"
Click on "MaterialThemeBuilder/README.md"
See error...

Duplicate of #241


Change Colour Schema of web application

Modify the web application to have better set of colour schema for the web application

Hi @DiptoChakrabarty, can I work on this issue once Hacktoberfest starts ?

Sure @BALaka-18  please go ahead

Hey @DiptoChakrabarty do you have any specific color scheme in mind ? Like, you want certain shade(s) of color more prevalent in your web app ? Or can it be any color scheme ?

@BALaka-18  I do not have any specific schema in my mind , I would just like it to be more professional looking

@BALaka-18  I do not have any specific schema in my mind , I would just like it to be more professional looking

@DiptoChakrabarty there's a bug in the backend, do I fix this in this issue itself, or do I have to create a new Issue for that ?

Please create a new issue


Rsync doesn't work if .bashrc invokes machine report

I can transfer files server -> local fine. But when sending files to a remote server with .machine_report.sh in .bashrc file, I get this error. Removing machine report from loading restores functionality of rsync.

I don't think there's a way to fix this inside the script itself. Rsync runs .bashrc when connecting to another host and it doesn't play well when there's anything in there that produces output. Here's more info on that.
The best way to solve this is to add this line at the top of .bashrc:
# If not running interactively, don't do anything
[[ $- != *i* ]] && return

Some distros do this already by default and it also solves problems with scp and sftp.

I have changed my stance on this issue back to "shouldn't be our problem". Adding that line of code to the script has some weird effects and it causes inconsistencies and when I tried to make it work just when it's sourced in .bashrc it started complaining about bad substitution from sourcing /etc/os-release and printing the info.
Adding this to the script itself introduces lots of complexity that shouldn't be there in the first place, since this issue only pops up specifically when the script is used inside .bashrc and trying to use rsync/scp/sftp without the proper safeguards most distros already provide by default. I think instead of fixing this with the script, we provide the documentation and a simple explanation why this line of code needs to be at the top of the user's .bashrc:
# If not running interactively, don't do anything
[[ $- != *i* ]] && return

I have modified the README.md in my fork to reflect this.

Thanks! Added your suggestion to the installation instructions.


Flaky test TestGRPCResolverRoundRobin (again)

This is a repeat of #2085 and #2674.
TestGRPCResolverRoundRobin fails 100% of the time on my Mac (macOS Monterey 12.2.1) when run using time go test -tags=memory_storage_integration ./pkg/discovery/....
The test is flaky in two ways.
I was able to cure both problems, but I don't understand my fixes, and thus am not confident enough to create a PR around them (yet).
Rarely the test fails with a panic on addrs[p.Addr.String()] = struct{}{}.  This is likely because p.Addr is set with grpc.Peer(), which populates the peer field after the RPC completes based on the Context.  We are using a non-blocking Client, and re-using the same Context for each connection.  Something is racing in there.  Wrapping the statement with if p.Addr != nil { removes the occasional panic, but I am unsure if it is safe to use grpc.Peer() at all if work might be done on another thread.
I noticed makeSureConnectionsUp() has the comment // 3000 * 10ms = 30s but the test fails in under 2s.  Adding time.Sleep(10 * time.Millisecond) before continue to make the comment true fixes the test.  (The problem also goes away if I raise 3000 to 2000.)

@tarhphilomina feel free

I am so sorry , I can't reproduce this error.  I just realized the error occurs only on Mac , I work with ubuntu. i am sorry. I won't be able to handle this issue.

@tarhphilomina no worries. fwiw I don't think the test failure is really dependent on Mac vs Linux, from the description (and the fact that adding sleep fixes it) it sounds like a race condition, perhaps more rare on some architectures. I could not reproduce it with 100 runs
$ go test -count 100 ./pkg/discovery/...
ok  	github.com/jaegertracing/jaeger/pkg/discovery	0.162s
ok  	github.com/jaegertracing/jaeger/pkg/discovery/grpcresolver	2.283s

Also, -tags=memory_storage_integration has no bearing on the test, this build tag is not used in ./pkg/discovery/....
Overall, this is a pretty old ticket and I do not recall this test failing too recently, it may have been fixed or the race went away from upgrading grpc. I am closing it.


FIX: avoid deprecation warnings of y_train_mean

Hot fix for #462

LGTM. Waiting for the bots.

Looks good, thanks :)

Codecov Report

Merging #464 into master will decrease coverage by 0.02%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master     #464      +/-   ##
==========================================
- Coverage   86.42%   86.39%   -0.03%
==========================================
  Files          22       22
  Lines        1584     1588       +4
==========================================
+ Hits         1369     1372       +3
- Misses        215      216       +1

Impacted Files
Coverage Δ

skopt/learning/gaussian_process/gpr.py
80.61% <80%> (-0.24%)
:arrow_down:

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 0d60dd7...32cf492. Read the comment docs.


feat: add resolve_types to Resolver trait

Closes #44

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


Updated documentation for Consumers to prefer KafkaConsumer instead

Coverage remained the same at 89.27% when pulling aa19d71f9804f431199f4f68087ad4a53b0cbab9 on sontek:update_consumer_docs into 3ab9a42fcf43466f233068585f47154fbd34760b on mumrah:master.

Coverage remained the same at 89.27% when pulling aa19d71f9804f431199f4f68087ad4a53b0cbab9 on sontek:update_consumer_docs into 3ab9a42fcf43466f233068585f47154fbd34760b on mumrah:master.


Json output for forge script and forge debug

Component
Forge
Describe the feature you would like
The --json flag for forge create is very useful for piping in bash scripts.
I propose something similar for forge script and forge debug.
The --json flag for these two commands would replace all stdout with json output.
Potential keys include gasUsed, logs, and return.
Currently, forge script will print return values in the following format:
== Return ==
0: uint256 1
1: bytes32 0x0000000000000000000000000000000000000000000000000000000000000007

If the return values are named, it looks like
== Return ==
a: uint256 1
b: bytes32 0x0000000000000000000000000000000000000000000000000000000000000007

I would suggest the return value to contain a nested json with, in the case of unnamed return values
{
  "0": {
    "type": "uint256",
    "value": "1"
  },
  "1": {
    "type": "bytes32",
    "value": "0x0000000000000000000000000000000000000000000000000000000000000007"
  }
}

and in the case of named return values
{
  "a": {
    "type": "uint256",
    "value": "1"
  },
  "b": {
    "type": "bytes32",
    "value": "0x0000000000000000000000000000000000000000000000000000000000000007"
  }
}

The logs value would just be a single string with all the logs, similar to what is printed to stdout now.
Additional context
No response

Just a note here: Until Forge is stabilized, these flags will always inherently be unstable and prone to change

closed by #1812


Flight Flag Definition not honoring MemberOfSecurityGroup filter entry

Hi, Feature Flight Management Team,
In one of our feature flights, the security group operator is not honoring all the MemberOfSecurityGroup filter entries. This seems to be happening intermittently. Details sent via private message.
Please advise.

RCA
A day-0 bug in the caching framework caused the issue.


opt Avoid create_column inside cast_column function

Proposed changes
In the cast function, we don't need to initialize the result column.

run buildall

run buildall

add testcases ?


[PRED-1785] Use t.StrBool() for config booleans

ConfigParser doesn't understand types, and everything it reads has str
type out of box. That means all booleans we read a failed to go through
Trafaret validation, because it expects booleans. Fortunately, Trafaret
provides so called StrBool() type that expects string as input, and
produce boolean as output. So let's use it in order to do not crash
reading boolean options from config.
PRED-1785

Coverage remained the same at 82.364% when pulling 4ae513cf3898106cfe683525e85a6b0e1e2e3d49 on ikalnytskyi/booleans into b94dd8a2818b9e1528fb07673da6b3b25d2da318 on master.


key-words are ignored as labels

Hi
There's this guide Moonforth to build a Forth for the DCPU-16. You can run the code interactively but it's not as nice as using your debugger. Thus I used the code from the first example and everything was working great.
But then I noticed that the ADD-command wasn't executed. I compared the generated object code from both assemblers and lo and behold: your assembler appears to ignore this label. I imagined that the reason could be that ADD is a key-word for the assembler and changed the label to ADDER. With this workaround everything works fine.
I'm not familiar with the specification of the DCPU-16 and thus it might be correct as it's currently implemented. On the other hand I assume that there shouldn't be a problem using a key-word as a label.
I've included the aforementioned code below. However I assume there is a minor bug in the original. A comma is missing in this assignment instruction:
:LOOP_XTO
  SET PC, LOOP_XTO

Kind regards
:START SET PC, MAIN ; JUMP MAIN

:NEXT
:WORD_FROM_CODE
  SET J, [I]   ; WORD-R <- *CODE-R
:NEXT_WORD
  ADD I, 0x01  ; CODE-R++
:JUMP_INTO_XTO
  SET PC, [J]  ; JUMP *WORD-R

:ENTER
:R_PUSH
  SET [Z], I   ; *RET-R <- CODE-R (PUSH PART 1)
  ADD Z, 0x01  ; RET-R++ (PUSH PART 2)
:WORD_AFTER_XTO
  SET I, J     ; CODE-R <- WORD-R
  ADD I, 0x01  ; CODE-R++
  SET PC, NEXT ; JUMP NEXT

:EXIT
  DAT EXIT_XTO
:EXIT_XTO
:R_POP
  SUB Z, 0x01  ; RET-R-- (POP PART 1)
  SET I, [Z]   ; CODE-R <- *RET-R (POP PART 2)
  SET PC, NEXT ; JUMP NEXT

:DOUBLE
  DAT ENTER DUP ADD EXIT

:DUP
  DAT DUP_XTO
:DUP_XTO
  SET PUSH, PEEK ; STACK DUP
  SET PC, NEXT   ; JUMP NEXT

:ADD
  DAT ADD_XTO
:ADD_XTO
  SET X, POP   ; STACK POP
  ADD PEEK, X  ; ADD X to top of stack
  SET PC, NEXT ; JUMP NEXT

:LOOP
  DAT LOOP_XTO
:LOOP_XTO
  SET PC LOOP_XTO

:CODE_1 DAT DOUBLE
:CODE_2 DAT DUP
:CODE_3 DAT LOOP

:MAIN
  SET I, CODE_1  ; CODE-R <- CODE_1
  SET PUSH, 0x05 ; STACK PUSH 0x05
  SET PC, NEXT   ; JUMP NEXT

Thanks for this, I'll have a look.  At first blush it appears that the tokenizer is attempting to tokenize reserved words before labels, which is incorrect.  I'll try to get to the bottom of it...

After looking at this a bit I found that this is issue is due to a bug in my tokenizer.  The problem is that the tokenizer will match commands and reserved words before label references.  However, fixing this would require quite a big refactoring effort in the tokenizer code to convert it from a single pass algorithm to a double pass algorithm that first identifies / matches labels before all other tokens.  Considering that this issue has a relatively straightforward workaround I'm going to defer fixing until a later time.
I did modify the code to identify this particular condition (a non-supported label is referenced in a DAT block) and added some error handling to let the developer know.  So now at least it won't fail silently.


chore(deps): upgrade jenkins-x/jx3-pipeline-catalog to version 3.10.85

from: https://github.com/jenkins-x/jx

/retest


chore(package.json): update package.json

TODO:

[x] npm run build
[x] npm run build:examples

right now it fails on npm run build
> rimraf build/* && tsc -d --module ES6 --outDir build/es6 && tsc -d && rimraf build/examples

node_modules/cyclejs-utils/lib/es6/index.d.ts(2,27): error TS2307: Cannot find module 'cycle-onionify'.
src/modalify.ts(100,51): error TS2556: Expected 1-2 arguments, but got a minimum of 0.

i've updated cyclejs-utils to latest.. but now  [this line(https://github.com/feliciousx-open-source/cyclejs-modal/blob/86f0d0c7ee1f55310b994d7a4c32d9b2edc34e69/src/modalify.ts#L100) is giving me a typescript error on mergeSinks
src/modalify.ts(100,51): error TS2556: Expected 1-2 arguments, but got a minimum of 0.

How do we fix this? @jvanbruegge
it's because mergeSinks needs at least 1 arguments ( I think? ) but our fold for modalStack$ starts with an empty array that gets spread out.. so 0 argument.

I had a breaking chang in mergeSinks it not takes an array as argument, forgot to update here

So, I'm home again.

hmm, what should I do to help?

Just replace modalStack$.map<Sinks>(arr => mergeSinks(...arr)) with modalStack$.map<Sinks>(mergeSinks)

super sorry for the late response. fell sick over the weekend.
I've updated the branch with the latest fixes, it also broke at

nvm, fixed that too

Do the examples work again now?

yes the examples work again!

Thanks! Will publish now

Published it as 5.0.0, because of the major version bump in cyclejs-utils, just to be save


Changes to support Bankex, Finnifty, Midcap for Order Slicing and SL Orders

Also added a minor fix for margin calculation.

Removed those two files


Bumping h3ron-sys

Hi,
H3 recently merged the following PR: https://github.com/uber/h3/pull/749
I've updated h3o with the new values, but since I've some test running against H3 through h3ron-sys, they are now failing because I'm comparing against an old version.
Would it be possible to bump the commit targeted by h3ron-sys and release a new version?

Hi, sure - understandable. It also has been a while since the last release here.
The last H3 release is v4.1.0, which I am currently using in the main branch. The PR you linked is not yet available in a release by H3 itself yet (https://github.com/uber/h3/compare/v4.1.0...master), so I am a bit reluctant to create a new h3ron-h3-sys release based on a unreleased upstream version.
In case you really need this urgently, we can do a release of course ;)

No hurry, this can wait 🙂

In the end, I've wrote my own tests that doesn't rely on H3 (which I should do anyway, it nice to have a differential test suite but I should also have my own independent one) by reimplementing the computation myself 🙂

Alright, so this has been decoupled now - which is great.
I will keep checking uber/h3 every once in a while for new releases.


Touch event

Hey, great project.!Could you please let me know could I somehow register click/pointermove/swipe event on object or screen?

Currently I only have click events added.
Select the object in question that you want to make into a button. Then click the "AR.js" menu item, and select "Make Button." You can then change the URL that the button links to in the inspector when the object is selected. NOTE: buttons don't work the best on mobile for some reason. You kinda need to tap the object many time in quick succession, but on the computer using the mouse it always gets it on just one click.
This is currently the only thing I have included in the plugin, but there are more things that are indeed possible. AR.js uses Aframe to handle 3d objects and interactions and such. If you want to dig around in the html and try adding something else, you can check out aframe's documentation. https://aframe.io/docs/1.0.0/introduction/interactions-and-controllers.html


Remove Basil from GitLab plugins

Description
I originally adopted these plugins in September and October 2021 to deliver JENKINS-66304 and JENKINS-66303. Since then, I have modernized these plugins and delivered a few releases, including several bug fixes and new features. I did some major work on updating the dependency trees for proper dynamic linking, including creating the jersey2-api plugin, updating jackson2-api, and modernizing gitlab-api. I feel that now is a good time for me to step away from these plugins, having left them in a better state than I found them. I have added the adopt-this-plugin label to each plugin.
https://github.com/jenkinsci/gitlab-plugin
https://github.com/jenkinsci/gitlab-oauth-plugin
Submitter checklist for adding or changing permissions

Always

[ ] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Many thanks!


Add draft config format example

Discuss! ;)

A general comment is that I believe we should use different configs for different settings - one for development, one for testing and one for production (or have the same for prod and test). This would ease the development, while at the same time provide good/sane values for production. One such example is logging - during development, it is more likely one would use files and/or stdout, rather than logstash. Not to mention that constructs like <%= @logstash_host %> require puppet :8ball:

A general comment is that I believe we should use different configs for different settings

Agreed. Supervisord already supports the -c flag to load a specific config instead of the default.

Idea here is that there can be multiple services running in each worker.

I'd rather advocate service separation and put each of them in its own process. One simple case: if there is an uncaught exception in a service, the worker is killed, disrupting the others. In general, service separation would also probably be the way they are actually run in production.

I'd rather advocate service separation and put each of them in its own process.

We would not use this in production for the reasons you mention, but somebody on a small VM might be interested in running several services in a single process to save memory.


error: use of undeclared identifier 'PROCESS_CREATION_MITIGATION_POLICY2_RESTRICT_INDIRECT_BRANCH_PREDICTION_ALWAYS_ON'

Build fails on Windows 10 with error
 error: use of undeclared identifier 'PROCESS_CREATION_MITIGATION_POLICY2_RESTRICT_INDIRECT_BRANCH_PREDICTION_ALWAYS_ON
Console output and logs
Full console output available here.
Environment details
ungoogled-chromium version: 72.0.3626.122
Configuration bundle: windows
Packaging type: windows
OS and version for building: Windows 10 (64-bit)
Compiler toolchain: Visual Studio Community 2017 C++ tools
Installed sub-components:
Visual C++ core desktop features
Just-In-Time debugger
VC++ 2017 version 15.9 v14.16 latest v141 tools
C++ profiling tools
Windows 10 SDK (10.0.17763.0)
Visual C++ tools for CMake
Visual C++ ATL for x86 and x64
Test Adapter for Boost.Test
Test Adapter for Google Test
Visual C++ MFC for x86 and x64
Building and packaging steps
Install Visual Studio Community 2017 with the above sub-components
Enable Debugging Tools For Windows for Windows 10 SDK
Install Python 2.7.15, Python 3.7.1 and 7-Zip
Run the command pip install pypiwin32
Set PYTHON user environment variable pointing to Python 2.7.15
Run the commands below in ungoogled-chromium Git repository in cmd.exe as administrator:
mkdir build\src
py get_package.py windows build\src\ungoogled_packaging
cd build\src
py ungoogled_packaging\build.py

@mymortal did you take a look at https://github.com/Eloston/ungoogled-chromium/issues/675 ?

@mymortal你看了＃675吗？

Yes, I have seen it, and reference
https://github.com/Eloston/ungoogled-chromium/issues/639#issuecomment-466478108
Methods

@mymortal it only works up to 71.0.3578.98-2. You can't build anything newer than that.

@mymortal it only works up to 71.0.3578.98-2. You can't build anything newer than that.

thank you


"model" argument not handled

The plugin always downloads the en-us model even if model is set with a different language.

Fixed by https://github.com/OpenVoiceOS/ovos-tts-server/commit/290cdbee3f72d3e72cf4b2f90f9eed47267368fa


Why is there still no progress for Provider

Why is there still no structure for Provider?
Writing functions on a view is a bad approach in Flutter. There is no fast method like notifyListeners on the controller.
A page is created and state update is done on the viewModel connected to it. This is the right approach. In 2022, I see that an “issue” was opened on this topic, but nothing was added. I like your package very much, I even made a 1+ hour long demo on YouTube. You can get inspired by another package called Stacked. It has a View - ViewModel structure and has nice methods.

Hi @ruwiss,
Sorry if I've misunderstood but there is a structure to Providers in Nylo, heres the docs https://nylo.dev/docs/6.x/providers
Re. Writing functions on a view is a bad approach in Flutter
You can organize your project to use MVC, Nylo supports controllers where you can add methods related to your view in there.
Lastly, on the issue you've spotted, I'll take a look and investigate 🙂
Thanks for your support and if you have any ideas to contribute to the framework, feel free to send a PR

That's not the provider I'm talking about, I'm talking about ChangeNotifier.
https://pub.dev/packages/provider
Why can't I update state via controller?
Why do I need to specify the state name every time?
State name does not work, I need to specify the page path

Hi @ruwiss,
That package has nothing to do with Nylo but if you want to use it, you can 🙂 flutter pub add provider
If you have ideas to improve how state management is handled, feel free to send a PR
Re. Why can't I update state via controller?
I've mentioned in my last message: "Lastly, on the issue you've spotted, I'll take a look and investigate 🙂"

My English is bad so I didn't understand your last words sorry.
If I specify it as HomePage.path it works. However, if I specify it as state, it does not work.
When I change the stateName via HomePage, it still doesn't work.
If this problem is solved, there will be no need to use additional packages.
I'm also trying to understand the problem

Hi @ruwiss,
I've just tried this today and it appears to work as expected.
Can we arrange a quick Google Meet call to see what's happening?
You can send me a message on IG https://www.instagram.com/agordn

@ruwiss,
No worries, maybe we can still connect on Google meet and translate via AI 😅
Oh and the VS Code extension looks amazing!! Wow, I will definitely try this and maybe add it to the nylo.dev docs.
I'll send you a message on Instagram.

😄😄 Thank you
Instagram: https://www.instagram.com/ruwis.io/

As I understood The question is about:

separate View-Controllel logics
reactive controller which contains logics and has ability o change variable in View from controller
It would be nice to have these features in future releases.


[stable/datadog] Create system-probe.yaml in main agent container when sysprobe is enabled

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

No
What this PR does / why we need it:
Ensures we can read/use the system-probe config in the main container agent when the system probe is enabled to run. Allows the main agent container access to the config options of the system probe container which is currently necessary to prevent a log line, but will eventually be used by other checks.
Which issue this PR fixes
Currently will log a message that we cannot find the appropriate configuration for the system probe when running the main container agent.
Special notes for your reviewer:
The system probe (by default) uses the configmap to create its yaml file, and is the supplied/expected method in all of our documentation.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

This chart is deprecated now: https://github.com/helm/charts/pull/23384


Adding use of Vectors STL

Description of Change

Checklist

[x] Added description of change
[ ] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Sorry, but this does not look like an algorithm. Closing.


support nested memoize (2nd)

#4 was suboptimal and didn't work for the case I wanted to make work.
Instead of trackMemo, this propagates affected to parent memoize functions even if it's returning a cached value.

https://codesandbox.io/s/vanilla-typescript-forked-d83b8?file=/src/index.ts
@theKashey This is what I wanted and looks ok.
// will look again
console.log(
  getItemA({ state: prevState, id: 3 }) === getItemA({ state, id: 3 })
);

👍 , this was a little issue for memoize-state, as nested memoization was doing parent one a little unstable - "affected" keys were changing every time.
I was considering something like touchAffected, but in my case, the underlying memoization was reselect based.

During memoize-state development I added a shallBePure helper, which just runs memoization twice to check that function is pure. Just to found that state access is a part of function pureness.
This touched moment is a missing piece. Thank's @dai-shi, I'll make my stuff a little bit touched as well.

I was just creating a redux example, and found unexpected behavior. Maybe a bug... 🤔

https://codesandbox.io/s/proxy-memoize-demo-forked-fi5ni?file=/src/App.tsx
I finally made this work. I suppose this is the best we can do with proxy and weakmap combination.


Support for vue3

Is there a way to use the library with vue3?

any news?

I created a Vue 3 version of this package at https://github.com/stellR42/vue3-pdf-app. Feedback/contributions welcome.


Have you tested PointMLP on semantic segmentation datasets?

Hi @ma-xu
Have you tested PointMLP on semantic segmentation datasets? For example, S3DIS dataset.
Thanks.

@whuhxb Hi, we didn't tested it in theis version.  We will add the semantic segmentation task iin our extension work. Thanks.

@whuhxb Issue closed since no further discussions. Feel free to re-open it if necessary.


How can I update AuthenticityToken automatically?

For security, I use session["csrf"] = Rack::Protection::AuthenticityToken.random_token to update token after post reponse  avoid quickly send double request.

Sorry for the late reply. This repository has not been used. Please open an issue on https://github.com/sinatra/sinatra


Andrii: Headless CMS, 3 weeks

Week 1
I Need Help With:

I have some problem with the exercises as in Postman for me was more clear and easy to do it than in the script. Plus for some of them i'm not really sure what exactly i should use sort, filter etc.....

What went well?

Understanding what a content management system is, and why people use it
Perform a simple http call via postman to retrieve a piece of information

What went less well?

The connection inside of CMS is kind confusing but i think with some time it will be more clear

Lessons Learned

How to sort, filter, populate data in Postman and how to convert than in JS file

Sunday Prep Work

Still doing exercises and studying suggested material

I have some problem with the exercises as in Postman for me was more clear and easy to do it than in the script.

Postman is just there to test an API, but you will sadly have to write the code yourself so it can be used in the frontend.

Plus for some of them i'm not really sure what exactly i should use sort, filter etc.....

well it depends if you want to sort (ascending or descending, based on price or alphabetical order, etc.) or if you want to filter the database and only included a subset of the items in the result

Week 2
I Need Help With:

I understand relationship in Strapi but at some point i have issues with that. Specially when need to create users. As well i don't know maybe i missed that during last class but i don't know how should i connected my Starpi collection with HTML(simply create file in public folder or it should be done in other way ). And same problem about qs.js i can use qs.js which we have in our study lenses or it should be created with Strapi

What went well?

I finished chat collection but i have some doubts but generally it was good.

What went less well?

At some point i feel like i get lost. I have to much gabs in this module. Was trying to find nice tutorial to understand better but all what i find is already using  React so was really confused as i don't know yet React and it wasn't really helpful for me and i feel like waist this week as have so many time but couldn't do to much

Sunday Prep Work

Trying to understand how i can create chat app

understand relationship in Strapi but at some point i have issues with that. Specially when need to create users.

yes, it is a bit confusing. I think we have to differentiate between the users who can control the Strapi UI and users who use the JS application. I will look into this a bit more.

how should i connected my Starpi collection with HTML(simply create file in public folder or it should be done in other way )

well these are 2 different topics, not sure which one you ask about. one is to use the backend created by Strapi in the frontend code, by fetching data from it (if run locally, it is served on localhost:1337), the other question is hosting the frontend app within the Strapi installation, in the public folder. in this case, opening localhost:1337 (or the URL where the app is hosted) in the browser will automatically display the contents of the public folder

Was trying to find nice tutorial to understand better but all what i find is already using  React so was really confused as i don't know yet React and it wasn't really helpful for me and i feel like waist this week as have so many time but couldn't do to much

yes, just follow until they get to React, that is where we are now..creating the database, the relationships and fetching data using Postman and simple JS queries.

Week 3
What was not clear, where did you get stuck?

For me this module was super interesting but at the same time little bit hard. Everything was clear but its kind my personal issue. i understand all parts separately but getting problems when i need to put everything together. I had small issue with server but thanks to @danielhalasz i fix its fast with npm rebuild.

What was clear, what did you master?

How to create collection in Strapi, install my project and run server. As well filtering, sorting and populating data from server.

Where can you still use some help?

I am still little confused with the relationship between Strapi collections. It's not so bad but sometimes i have doubt which one i should chose. But probably i can help out with Postman. For me its wasn't difficult and it's quite useful and helpful app.

Where can you help others moving forwards?

In this module be honest i asked more for help than help for others. One of this people @MMikhailova so would like to say huge thank you for her)

The Course
What can there be more of?
I wish this module could be more longer as its super exciting
What can there be less of?
All was perfect
What material were most helpful (from HYF or elsewhere)?
This time i just used suggested material and watched record video from our class
Any suggestions for future classes?
Just want to say huge THANK YOU such amazing module)


Refactor to use new PassBuilder for LLVM > 4.

Using the new PassBuilder is impractical for LLVM 3.9
(so it remains for now behind a preprocessor ifdef),
but it will pave the way to fixing some other problems
with LLVM > 4, including #2371.

This shouldn't affect anything for LLVM 3.9 - the refactor only applies to LLVM > 4.

The CI failures are related to this gcc bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51322
(referred to me by https://bugs.llvm.org/show_bug.cgi?id=33161)
When compiled with clang, the CI failures are resolved.
So clang works but gcc < 5.1.0 doesn't.... ugh.

I'm giving up on this :disappointed:.
We'll have to fix our LLVM >= 4 issues without using the new PassBuilder.


[Document Intelligence] Prepare release for Beta Nov 2023

API change check
API changes are not detected in this pull request.


Only run coverage with inlining on

Use a separate buildbot for noinline tests
Fix #55
I realized that the test list is from CoverageBase.jl so this requires https://github.com/JuliaCI/CoverageBase.jl/pull/19
@staticfloat

can we make this run not on the ubuntu slave? there are too many nightly jobs on that builder and it gets backed up

I've just been copying the previous config file I touched, where should I run this then?

centos 7 maybe? that only has the one job right now

Sure, is this the right way to do it?

I just realized that doing this will not run the tests with inliing off.

@tkelman @staticfloat do we actually still want to have a buildbot running noinline tests? I think it won't really be testing sth that we want to keep working (now that we don't need inlining off for coverage). I might be useful as a Fuzz test but probably not more than that.
If we still want to have no-inline tests, https://github.com/JuliaLang/julia/pull/18680 should provide an much easier way.

I think it's worth identifying and tracking fragile tests, yes

In that case https://github.com/JuliaLang/julia/pull/18681....

It's trivial to add a second Ubuntu buildbot if that's what we need.  Then tasks assigned to an Ubuntu builder will be load-balanced accordingly.

we may want to do that anyway, but we have the centos worker and it's a bit underutilized

Rebased and updated the script to use the env added in https://github.com/JuliaLang/julia/pull/18680. This still launches the test process manually so that the compile test on master process also has inlining off.


Allow multiple models in sqlalchemy

See issue #24

Hey @AndiZeta, any chance you could squash your commits into 1?
I'm planning on merging this soon, but rebasing the 10 commits to get to the current state of master is kind of a pain 😅


[IDE][Utils] Add an IDE stream utility.

This provides a stream utility for writing to a underlying string buffer multiple string pieces and retrieve them later as StringRef when the underlying buffer is stable.

@swift-ci please smoke test

@swift-ci smoke test

@swift-ci smoke test


Reduce performance overhead of ResourceLeakDetector

Motiviation:
The ResourceLeakDetector helps to detect and troubleshoot resource leaks and is often used even in production enviroments with a low level. Because of this its import that we try to keep the overhead as low as overhead. Most of the times no leak is detected (as all is correctly handled) so we should keep the overhead for this case as low as possible.
Modifications:

Only call getStackTrace() if a leak is reported as it is a very expensive native call. Also handle the filtering and creating of the String in a lazy fashion
Remove the need to mantain a Queue to store the last access records
Add benchmark

Result:
Huge decrease of performance overhead.
Before the patch:
Benchmark                                           (recordTimes)   Mode  Cnt     Score     Error  Units
ResourceLeakDetectorRecordBenchmark.record                      8  thrpt   20  4358.367 ± 116.419  ops/s
ResourceLeakDetectorRecordBenchmark.record                     16  thrpt   20  2306.027 ±  55.044  ops/s
ResourceLeakDetectorRecordBenchmark.recordWithHint              8  thrpt   20  4220.979 ± 114.046  ops/s
ResourceLeakDetectorRecordBenchmark.recordWithHint             16  thrpt   20  2250.734 ±  55.352  ops/s

With this patch:
Benchmark                                           (recordTimes)   Mode  Cnt      Score      Error  Units
ResourceLeakDetectorRecordBenchmark.record                      8  thrpt   20  71398.957 ± 2695.925  ops/s
ResourceLeakDetectorRecordBenchmark.record                     16  thrpt   20  38643.963 ± 1446.694  ops/s
ResourceLeakDetectorRecordBenchmark.recordWithHint              8  thrpt   20  71677.882 ± 2923.622  ops/s
ResourceLeakDetectorRecordBenchmark.recordWithHint             16  thrpt   20  38660.176 ± 1467.732  ops/s

@carl-mastrangelo PTAL... After I had a look at your change (https://github.com/netty/netty/commit/b32cd26a961c71cc5ad0e9f88fa8c15fbfd89930) I noticed we can do even better 👍

@carl-mastrangelo PTAL again

Another number to share is that running mvn -Pleak clean package in the handler module before this patch took over 10 mins on my machine... With this patch it takes 2 minutes ;)

Cherry-picked into 4.1 (3c8c7fc7e9) and 4.0<PHONE_NUMBER>).


ci: save space

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

Project coverage is 69.89%. Comparing base (cfcc3cd) to head (0405b83).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   69.89%   69.89%
=======================================
  Files         117      117
  Lines       17958    17976   +18
  Branches    17958    17976   +18
=======================================
+ Hits        12551    12564   +13
- Misses       5204     5210    +6
+ Partials      203      202    -1

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


Gathering contributor list for ApplicationSet controller v0.1.0 release announcement

:arrow_right:  Please let me know ASAP if you (or someone you know) contributed to ApplicationSet code/design doc/planning/reviews/inception and are not on the below list (I really don't want to miss anyone :smile: !).  :arrow_left:
I'm working on a draft of the release announcement, and I'd like to make sure that everyone that has contributed to ApplicationSet is shouted out (:tada: :tada: :tada:) for their contributions to the ApplicationSet project over the last year.
I've gathered an initial list of folks from GitHub contributions+reviews+issues:

Name (from GitHub page)
GitHub ID

Omer Kahani
@OmerKahani

Devan Goodwin
@dgoodwin

Michael Goodness
@mgoodness

Alexander Matyushentsev
@alexmt

Jonathan West
@jgwest

Matteo Ruina
@maruina

Jesse Suen
@jessesuen

Xianlu Bird
@xianlubird

William Tam
@wtam2018

Ratnadeep Debnath
@rtnpro

John Pitman
@jopit

Shoubhik Bose
@sbose78

Alex Sharov
@kvendingoldo

Omer Levi Hevroni
@omerlh

If any of the above folks have any requests on how they they are credited, please reach out!
The list of names and GitHub IDs will be used in:

The release notes doc page on this repo
Blog post on https://blog.argoproj.io/
GitHub Release text (the text you see alongside a release in the GH Release tab)

Feel free to reply to this issue, or on Argo Slack (@jgwest or on #argo-cd-appset).

@jgwest my name is Alex Sharov :)

Thanks Alex, I've updated your name :+1:


proposal: Postgres Operator Upgrade

This enhancement proposes a method of upgrading the version of Postgres on the Red Hat Kubernetes Operator from version 10 to version 14.

Should we give users the option of deleting the backup? How long should it persist?

Will this process be identical for the Clair DB?


migrations multiple databases

To use Phinx migrations for multiple database would have to add each corresponding directive to the tenant database with the other credentials in the file phinx.yml right?
So to run would do: vendor/bin/phinx migrate -e tenant_02 so to the others right?

You can also have a script wrapping phinx and pass the path to the configuration file you want o use per tenant


performance optimize

performance optimize
code reformat

Could you please do this for MAKECMP as well -- that's where most of the time is spent at the moment.


[gdal] linux build does not get dependencies from vcpkg

Is your feature request related to a problem? Please describe.
The linux vcpkg build of gdal does not get its dependencies from vcpkg.  The configure command needs to refer to the vcpkg install path so it finds its dependencies there. For example, xerces-c or tiff. They are found in /usr not in the vcpkg install path.

I believe #6752 is somewhat blocking this effort in that making vcpkg symlink debug/include is better than an invasive patch for gdal's ./configure script and m4 macros to distinguish between the various vcpkg paths.
Not familiar with the vcpkg source though to try adding support for this.

I am also currently running into this issue. If I have netcdf/gdal installed from the arch repositories, then gdal will attempt to compile and link against them, and then I run into compatibility issues because arch linux includes newer versions.

Yes, these problems with gdal have made me give up using vcpkg with linux. I have found vcpkg works great on windows, but on linux where it is rather difficult to exclude pre-installed packages in /usr, I am encountering inconsistencies with packages that sometimes get dependencies from vcpkg and sometimes from what has been apt-get installed.

This issue should be fixed now.

Please reopen this issue if it still exist.


🐛 fix: add codspeed features

fix

修复 codspeed features

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.06%) to 71.569%

Files with Coverage Reduction
New Missed Lines
%

server/rfuses_device_local/src/notify_loop.rs
2
81.94%

Totals

Change from base Build<PHONE_NUMBER>1:
0.06%

Covered Lines:
1236

Relevant Lines:
1727

💛  - Coveralls


Code Coverage is Always 0%

Hi,
I'm using Echo framework and unable to get the code coverage from go test command (with options of course ).
This test is not in the same package from its functional package. I.e : the main functional package is handlers but package fot test is tests.
Is it this package is currently unable to produce code coverage, or actually I'm doing wrong on my test ?
This is sample of my project structure :
Myapp\
    Handlers\
        user.go
    Router\
        router.go
    Tests\
        user_test.go

This is the sample of user_test.go :
package tests

import (
	"net/http"
	"net/http/httptest"
	"testing"

	"myapp/router"

	"github.com/gavv/httpexpect"
)

func TestGetUsers(t *testing.T) {

	RebuildData()

	// This is actually the instance of Echo router
	handler := router.New()

	// run server using httptest
	server := httptest.NewServer(handler)

	defer server.Close()

	// create httpexpect instance
	e := httpexpect.New(t, server.URL)

	// without query string
	obj := e.GET("/users").
		Expect().
		Status(http.StatusOK).JSON().Object()
	obj.Keys().Contains("data")
	obj.Value("data").Array().Element(0).Object()

	// with rp & p set
	obj = e.GET("/users").WithQuery("rp", "2").WithQuery("p", "2").
		Expect().
		Status(http.StatusOK).JSON().Object()
	obj.Keys().Contains("data")
	obj.Value("data").Array().Element(0).Object().Value("id").Equal(3)

	// with filter name
	obj = e.GET("/users").WithQuery("name", "iman").
		Expect().
		Status(http.StatusOK).JSON().Object()
	obj.Keys().Contains("data")
	obj.Value("data").Array().Element(0).Object().Value("name").Equal("iman")
}

This is the command to produce coverage
> go test tests/*_test.go -cover -coverprofile=coverage.out

Sorry, its not related with this repo but the Go it self.
Coverage cannot be reported within Go 1.10,  you need to upgrade to 1.11 or down grade to 1.9.
https://github.com/golang/go/issues/25093


Add support for JBoss EAP

closes elastic/apm-agent-java#163

Codecov Report

Merging #481 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #481      +/-   ##
============================================
- Coverage      64.4%   64.26%   -0.15%
  Complexity       68       68
============================================
  Files           164      164
  Lines          6189     6195       +6
  Branches        709      712       +3
============================================
- Hits           3986     3981       -5
- Misses         1957     1969      +12
+ Partials        246      245       -1

Impacted Files
Coverage Δ
Complexity Δ

...tic/apm/agent/servlet/helper/ApmAsyncListener.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...co/elastic/apm/agent/report/ApmServerReporter.java
54% <0%> (-5%)
0% <0%> (ø)

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 1003529...243f106. Read the comment docs.

Hi, thanks for implementing this feature. I was wondering do you have any estimations that when we can have a new release including these commits? 😄

We’re planning to release this week. You can watch this repo for “releases only” to get notified about the release.


As a pool UI user, I want to see the accumulated volume / APY / fees

Problem
What problem are we solving and why does it matter, ser?
Show 24H volume per pool.
Proposed Solution
Thoughts on solution design. We should do  to solve this problem.

[ ] Show 24H volume per pool

Acceptance Criteria
From an end tester and/or user perspective, the below criteria must be met to consider this done:

[ ] Shows 24H volume per pool (getYieldStatsForDay)

This data need to come from the subgraph -> cartographer. I believe @liu is already working on stable swap subgraph. Let's spec this out once we have that task completed.

#2641 has getYieldData implementation done which has been released in latest version. we need to implement this on UI level now. Moving this under, Coinhippo's task.

@pjenviri @nrsirapop any issues with this one? it is higher priority

@alexwhte @sanchaymittal, still getting errors on getYieldData.

This impacts Volume and APR
@pjenviri can we please show 'TBD' in UI for volume and fees until we get these metrics completed?

updated it to show TBD.

back to connext for getYieldData impacting these metrics

Let's make sure the stats are all converted to the right $ amount. getYieldData returns fees, liquidity, and volume in 1e18 precision.

These amounts are in usd? getYieldData api returns in ETH. I think we have to convert ETH to USD.

@pjenviri we need a double check on this conversion, the decimals are 1e18 returned from all getYieldData stats

@just-a-node , We display the data returned from yieldData. They are already converted.

Awesome, confirmed with swap volumes from subgraph. TY


Updated README.md

like @AnandBaburajan  mentioned it should be how to contribute rather than setup in the readme
rest of it looks great!
also you can add badges and other cool stuff on your readme.


duplicate layer downloading

@tianon I was doing some work with the library/golang image this morning, and noticed this happening
The key is that both of the huge layers in progress, are actually the same id.  Why is Docker downloading the exact same layer in parallel?  Seems very inefficient.  Should this be happening?

If you can repro please ping here and we can try to repro and re-open.
Thanks!


rustdoc-json: Add rustdoc_json::Builder::verbose(bool)

My use case was that I got SIGILL (see https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/SIGILL.20in.20build-script-build.20with.20nightly-2023-08-25)
I got SIGILL when building rustdoc JSON. I wanted to see the command to see if I could reproduce the SIGILL with just the rustdoc command.
So I had one use for it. And I'm thinking one use probably means more uses in the future?

@Emilgardis I don't want to merge this if you are against it, so I'm curious on if you could elaborate your concerns a bit? Is it the API surface you are concerned about? Or something else?


expm implementation for dense matrices

This will implement the equivalent to scipy.linalg.expm for dense matrices.
Implemetationss worth taking into account, both TensorFlow and PyTorch already have implementations. The former's relies on scaling and squaring Padé's aproximation while the latter uses an optimized polynomial taylor sum approximation.
There is also this implementation, which is also based on Pade's aproximation.
And this reference from the implementation on Mathematica, which hints at some optimizations for hermitian matrices.
This will tackle #282, and help with https://github.com/qutip/qutip-cupy/issues/40 .

Thank you for working on this! I think cupyx.scipy.linalg.expm is the right place to put this function as numpy.linalg.expm does not exist 😄
https://docs.scipy.org/doc/scipy/reference/generated/scipy.linalg.expm.html
SciPy's implementation supports both dense and sparse but it's ok to start with the dense-only implementation.

Thank you for working on this! I think cupyx.scipy.linalg.expm is the right place to put this function as numpy.linalg.expm does not exist
https://docs.scipy.org/doc/scipy/reference/generated/scipy.linalg.expm.html
SciPy's implementation supports both dense and sparse but it's ok to start with the dense-only implementation.

Thank you for the catch , I am editing the description and changing the locationof the files.

Hi @MrRobot2211 Any chance you can push your WIP so that I can ask someone to take over and help move it pass the finish line? As you know this is an important API for quantum computing and quantum physics, so let's not drop the ball 🙂 Thanks!

Kindly ping @MrRobot2211 😃

So I think this repo (https://github.com/Lezcano/expm) was how PyTorch used to implement expm. It implements both GPU-friendly approaches (see its README and the linked papers). We could port it to CuPy and see how the performance goes, and decide if we need any dedicated CUDA kernel. Could be a good mini project 🙂

https://github.com/cupy/cupy/pull/7660 has an implementation based on Pade 13/13 approximation of Higham'2005

Thanks, @ev-br, it's very encouraging to know! I suggest we close this PR due to inactivity. Once #7660 is in, we can revisit expm() anytime if the Taylor-expansion based approach is needed.

If/when this PR is closed, I'd suggest to open an enhancement issue with links to other implementations, especially Lezcano's one (https://github.com/cupy/cupy/pull/5688#issuecomment-1552067121). As a general purpose expm for potentially large matrices, it is probably way more efficient than gh-7660's version. And as a yet further enhancement, https://github.com/higham/expmv has a Taylor-expansion based expm times vector algorithm.

@takagi shall we close this PR and follow up in #7769?

Yes!


Enable dontWrap on radios

Will resolve #523

  Theres no option to prevent radios from wrapping themselves in a fieldset


Get coordinates while drawing

Hey, is there a way to get coordinates of the line while drawing ? or when finished ? Or maybe get the object ?
My goal is to use it with sockets.
Thanks.

I am also been looking for this feature, please. Shapes, texts and some features have one but free drawing doesnt have, seems kind of odd.

var addObjectList = imageEditor._graphics._canvas._objects;
iterate over addObjectList, we can get all the added object present on image and also drawing object with all co-ordinates


Add "Africa" tag and capitalize “Lesson Development” and "Hackathon" tags

Adding some tags for tagathon #73

I see, I've updated the tags. Hopefully this is better. Also I think I have fixed the typographic quotation marks to regular quotation marks for the tags with quotes.

Many thanks @ceesu! I took a short break, but I'm now at my desk and will be merging this shortly. We really appreciate all your input during our Tagathon week.


Added a CodeTour to the project

Category

[ ] Bug Fix
[x] New Feature

What's in this Pull Request?
Added a CodeTour to the solution. If you like this, we'll add more CodeTours for every web part, and one for deployment.

@eoverfield,
@hugoabernier @PopWarner  and I spend some time setting up our initial code tour for the Starter Kit tonight. We have plans of expanding this out to include tours of individual spfx solution. But this is the first set up, exploring the solution root level folders.

Awesome! Thank you very much. Tours of each individual part would be outstanding and very useful for the community. Thanks, and please add whatever you can.

Do you have a snippet of text we could add to the root readme to help guide new devs to this extension? etc?

Yes. I can make an update to the readme to include this information!


Open files context no more available in JetBrains

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS:
- Continue version: 0.0.83
- IDE version: IntelliJ 2024.3.1 Ultimate
- Model:
- config.json:

Description
I have added "open" to context providers. It works correctly in VSCode. It doesn't appear at all among context providers in IntelliJ.
To reproduce
In IntelliJ, Just type "@" and you will NOT see "open files" among context providers.
Log output
No response

Can you share your config.json?

{   "models": [     {       "title": "Bedrock: Claude 3.5 Sonnet",       "provider": "bedrock",       "model": "anthropic.claude-3-5-sonnet-20240620-v1:0",       "region": "eu-central-1",       "profile": "bedrock-continue",       "completionOptions": {         "maxTokens": 4096       }     },     {       "title": "Azure: GPT-4o",       "provider": "azure",       "model": "gpt-4o-2024-08-06-continue",       "apiBase": "https://openai-sviluppo.openai.azure.com",       "engine": "gpt-4o-2024-08-06-continue",       "apiVersion": "2023-03-15-preview",       "apiType": "azure",       "apiKey": "",       "completionOptions": {         "contextLength": 16000,         "maxTokens": 3000       }     },     {       "title": "Gemini Pro 002",       "provider": "gemini",       "model": "gemini-1.5-pro-002",       "apiKey": "",       "completionOptions": {         "maxTokens": 4096       },       "contextLength": 2000000     },     {       "title": "Gemini Pro Exp 1206",       "provider": "gemini",       "model": "gemini-exp-1206",       "apiKey": "",       "completionOptions": {         "maxTokens": 4096       },       "contextLength": 2000000     },     {       "title": "Gemini 2.0 Flash Exp",       "provider": "gemini",       "model": "gemini-2.0-flash-exp",       "apiKey": "AIzaSyBA0nukevtbIjNgv49xDnJj4P3QcHaDGfA",       "completionOptions": {         "maxTokens": 4096       },       "contextLength": 2000000     },     {       "title": "Gemini Flash 002",       "provider": "gemini",       "model": "gemini-1.5-flash-002",       "apiKey": "",       "completionOptions": {         "maxTokens": 4096       }     },     {       "title": "Deepseek",       "provider": "deepseek",       "model": "deepseek-coder",       "apiKey": "sk-"     },     {       "model": "gpt-4o",       "title": "GPT-4o",       "apiKey": "sk-proj-",       "completionOptions": {         "maxTokens": 4096       },       "provider": "openai"     },     {       "model": "o1-mini-2024-09-12",       "title": "o1-mini",       "apiKey": "sk-proj-",       "completionOptions": {         "maxTokens": 4096       },       "provider": "openai"     }, //    { //      "title": "Codestral", //      "provider": "mistral", //      "model": "codestral-latest", //      "apiKey": "AB", //      "completionOptions": { //        "maxTokens": 4096 //      } //    },    {      "title": "claude-3.5-sonnet-1022",      "provider": "anthropic",      "model": "claude-3-5-sonnet-20241022",      "apiKey": "sk-ant-api03-",      "completionOptions": {        "maxTokens": 4096      }    }   ],   "slashCommands": [     {       "name": "edit",       "description": "Edit selected code"     },     {       "name": "comment",       "description": "Write comments for the selected code"     },     {       "name": "share",       "description": "Export this session as markdown"     },     {       "name": "cmd",       "description": "Generate a shell command"     },     {       "name": "onboard",       "description": "Familiarize yourself with the codebase"     }   ],   "customCommands": [     {       "name": "test",       "prompt": "Write a comprehensive set of unit tests, using a PyTest class. It should setup, run tests that check for correctness including important edge cases, and teardown. Ensure that the tests are complete. Give the tests just as chat output, don't edit any file. Prefer literal values instead of computed ones. Do this for {{{ input }}}",       "description": "Write unit tests for "     },     {       "name": "check",       "description": "Check for mistakes in my code",       "prompt": "{{{ input }}}\n\nPlease read the highlighted code and check for any mistakes. You should look for the following, and be extremely vigilant:\n- Syntax errors\n- Logic errors\n- Security vulnerabilities\n- Performance issues\n- Anything else that looks wrong\n\nOnce you find an error, please explain it as clearly as possible, but without using extra words. For example, instead of saying 'I think there is a syntax error on line 5', you should say 'Syntax error on line 5'. Give your answer as one bullet point per mistake found."     },     {       "name": "code",       "description": "Write code as patch",       "prompt": "{{{ input }}} Output only code as a patch. Write nicely refactored code, avoiding repetitions as most as possible. Ensure the code you write is working, reason step by step if necessary."     }   ],   "contextProviders": [     {       "name": "code",       "params": {}     },     {       "name": "docs",       "params": {}     },     {       "name": "url"     },     {       "name": "codebase",       "params": {         "nRetrieve": 70,         "nFinal": 30,         "useReranking": true       }     },     {       "name": "tree"     },     {       "name": "open",       "params": {}     }   ],   "embeddingsProvider": {     "provider": "openai",     "model": "voyage-code-2",     "apiBase": "https://api.voyageai.com/v1/",     "apiKey": "pa-"   },   "reranker": {     "name": "free-trial"   },   // "tabAutocompleteModel": {   //   "title": "Deepseek",   //   "provider": "deepseek",   //   "model": "deepseek-coder",   //   "apiKey": "sk-"   // },   "tabAutocompleteModel": {     "title": "Codestral",     "provider": "mistral",     "model": "codestral-latest",     "apiKey": ""   },   "tabAutocompleteOptions": {     "maxPromptTokens": 4096,     "debounceDelay": 1000,     "maxSuffixPercentage": 0.4,     "useOtherFiles": true   },   "allowAnonymousTelemetry": false,   "docs": [],   "ui": {     "displayRawMarkdown": false   } }

Hmm, looks like I'm experiencing the same behavior. Going to look into this shortly!

Hmm, looks like I'm experiencing the same behavior. Going to look into this shortly!

I would like to add that I am also facing the same problem

Hmm, looks like I'm experiencing the same behavior. Going to look into this shortly!

I would like to add that I am also facing the same problem

me too :-(
only files and prompt files are shown as available

This appears to have been resolved in 0.0.84
That version is currently in EAP but we plan to release later this week. Please follow up if anyone is still having issues after installing 0.0.84 👍


Borderlands 3 ~ 900p Fullscreen resolution issue

Hi,
First, The game does run brilliantly... Sadly though, when fullscreen I'm unable to use 1600x900 resolution. I'll bullet point what happens.
~ when trying to apply this resolution, my physical screen (not the game) displays the message "Unsupported Signal"

~ my screen defaults internally to 1080p but displays the game in 900p with black border padding
~ when try to swap to another resolution, the game won't let me. I have to set the game to windowed (with borders) then, go back to fullscreen in order to get the game to go into another resolution
900p does work on this screen, I use it on many other titles. Also, I have attached the D3D logs below.
Software information
~ Lutris patched wine - lutris-wf-nofshack-1.45-x86_64
~ Custom Media Foundation Platform patches to the wine prefix (https://www.thelinuxexp.com/post/borderlands-3-on-linux-it-kinda-works-how-to-step-by-step-guide)
~ Borderlands 3, very low gfx preset
System information

GPU: GTX 870m
Driver: 430.40
Wine version: 4.15 (lutris-wf-nofshack-1.45-x86_64)
DXVK version: 1.4.3

Apitrace file(s)

Put a link here

Log files

d3d11.log:
Borderlands3_d3d11.log

dxgi.log:
Borderlands3_dxgi.log

Does this work if you use wined3d?

I also noticed you aren't using FSHack, does anything change if you enable/use that?

Please don't use fshack as it might mask the problem. I really want to know if this is my bug or not.

Fair enough

I'm very sorry but, I can't test right now. (also, I'm not sure fs hack is)

so, I won't be able to test it today due to denuvo ~ https://support.codefusion.technology/b3_da2dsa23/?e=88500006&l=english
All I did was change some audio hardware options and now I'm getting the error you receive when it thinks you're trying to run to the game on a new pc.

I've done some testing...
Wined3d ~ Plays Borderlands 3 intro videos but, crashes whilst trying to get to the main menu.
DXVK (install --without-dxgi) ~ Borks the to prefix preventing the Epic Game Store from launching so, I can't start Borderlands...
Returning to the normal DXVK 1.4.3 install restores the functionality I had at the beginning.

Can you try setting dxgi to builtin in your Wine prefix only for Borderlands3.exe? That way it shouldn't break EGS.
Just to make double-sure, since quite a few users aren't aware that this functionality exists, this is what it would look like:

I've tried this and the outcome is the same.

OK, it's the same outcome. Maybe it's a Borderlands issue?
Also, I forgot to mention 720p does work.

Since you're on a laptop, are both displays active in some way?
I kind of suspect that this might be the good old wine multi-monitor issue (there's a reason why DXVK only supports one monitor...); it would report the same display modes for both your displays and obviously they won't be correct for one of them.

The Laptop display is disabled in the /etc/X11/nvidia-xorg.conf.d/10-monitor.conf to prevent any performance degradation from screen mirroring so, that screen isn't visible to the rest of the system. (Also, the screen is broken)

@Niroc If you are still around and able can you test if this is still an issue?


Manage IAM users and access to discfg configs?

Should discfg manage AWS IAM? ...And ultimately, access to the DynamoDB table?
It would be very convenient to create a config and then assign additional access. But how common is this need? Wouldn't someone just use AWS management console for this? Is it really that convenient?
Does there need to be a way to manage IAM users now through discfg? Set new users up? Get the keys (presumably to give to someone else) and then allow/disallow access?
I don't know just yet...But it's something that can certainly be done. If this feature is added, it'll certainly be later on in the roadmap.

I think since I'm using Apex now to build out the Lambdas that then use Discfg and since it uses Terraform...It makes more sense to leave this out for the foreseen future. People can simply adjust the Terraform configuration to manage access.


Testbranch

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

0 out of 2 committers have signed the CLA.

[ ] Reanne Lee
[ ] leeminli

Reanne Lee seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.
Have you signed the CLA already but the status is still pending? Recheck it.


refactor: Move transition handling to shared chart popover component

Description
Improve code reuse between chart instances and apply the same transition treatment to all popovers used there
Related links, issue #, if available: n/a
How has this been tested?

PR build
Locally, ensured transition works on all chart instances

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.

Yes, this is not a functional change, only for a better code reuse


Inconsistent generation of authentication_id

If you subclass DefaultAuthenticationKeyGenerator and override the method extractKey(...) and make it return a debuggable key like below:
Map<String, String> values = new LinkedHashMap<String, String>();
OAuth2Request authorizationRequest = authentication.getOAuth2Request();
if (!authentication.isClientOnly()) {
   values.put(USERNAME, authentication.getName());
}
values.put(CLIENT_ID, authorizationRequest.getClientId());
if (authorizationRequest.getScope() != null) {
  values.put(SCOPE, OAuth2Utils.formatParameterList(authorizationRequest.getScope()));
}
return values.toString();

You will see the scopes are some times out of order so the authentication_id will have a different checksum value.
The is caused by the BaseRequest.getScope() always returning a HashSet. From my perspective the proper fix is to always sort the scopes when generating the id so regardless what kind of collection we are given the results would remain the same.

Thanks for the report @born2snipe


Save profiles when running startLocal

Right now, profiles are thrown away when running startLocal. With this change, profiles will be written out to files.

Should this PR be closed or submitted?

I think I'll close this. If startLocal does save profiles, it should probably be only when the user specifies this.


Intellivision.json - 7z files for freeintv

The freeintv core info for retroarch doesn't state it, but 7z files work with it. I can run them with no problem.
Can't get them to list in Daijishou though.
It's not like I saved much space so I just uncompressed them to get them to list. But 7z should be accepted.

It's not the core that has 7z support, but RetroArch itself, no matter what core.

Eh, okay. Regardless, 7z works in RetroArch with the freeintv core, so the games should list in Dai as well.

Well, I guess zip and 7z could be extensions for all platforms with non-disc-based roms. Actually, I never understood why there's even the need to narrow down extensions for each platform.

I've been going through platforms the past few days and cleaning/fixing some missed file extensions. Just haven't pushed the changes yet. Adding 7z will be apart of it.
@EgoBizarro the reason why you'll want to narrow down file extensions is because not every file can actually be launched or might have problems launching.
For instance CD based platforms that use bin/cue. Sure some emulators can launch just the bin but for games that are multitrack you'll have a problem with audio. So what we do is omit the bin file from being looked at entirely. This will make the user load the cue file thus preventing the possible problem.

But wouldn't it be a lot less work not to include but to exclude file types, e.g. bin in disc-based platforms. I can't think of many other examples like that.

Scummvm, Arcade, PC, & few others would be a pain. Plus regex is already used twice to include or exclude, user can edit either if they desire. One is global while the other is per player
Global:

Daijishou > Settings > Inspect platform > Accepted filename regex

Player:

Daijishou > Settings > Inspect player > Accepted filename regex

You can use this in ways where one player might not have as many supported file extensions as another. Sounds like a headache at first but it works.

Updated the platforms to include both zip & 7z in 1b76a88
Please update your platforms:

Daijishou > Settings > Library > Update platforms automatically

Or

Download > Select platform

There's still more platforms I have to clean up. Closing this issue for now


Getting unknown property: craft\models\UserGroup::uid

Hitting this error when going to create an entry where the user group field is attached. The problematic code appears to be on line 146 of UserGroupFieldField.php. Stacktrace attached below.
stacktrace.txt
Craft version: 3.0.35
PHP: 7.0.32
Database driver & version: MySQL 5.7.23

I thought I had already required 3.1 in composer. It's basically because 3.1 started using uid's for everything, and I wanted to support that without adding extra code to check which versions we're using.
I'll constrain this Craft 3.1 and have release out soon.


[FLINK-8596][CLI] Also catch NoClassDefFoundErrors

What is the purpose of the change
This PR modifies the CliFrontend to also catch NoClassDefFoundErrors when loading the YARN CLI, as otherwise the entire CLI would crash if hadoop wasn't on the classpath.
Verifying this change
This change added tests an end-to-end test that is run on travis.

+1 to merge

E2E test isn't fully working yet, debugging...

There's also a ClassNotFoundException in the logs, which is not ignored.

merging.


Update Cargo Resolver Version

Update cargo resolver to version 2 to avoid unwanted feature activation.
The package sdl2-sys has features bundled and static-link for target
linux activated, since the cargo resolver version 1 aggregates all
possible features. The newer version avoids this behaviour.
See also:
https://doc.rust-lang.org/cargo/reference/resolver.html#feature-resolver-version-2

This has been done in the opengl branch, but I will merge this for those still on main

Thanks a lot!


feat: add missing transitions to ContractNegotiation state machines

What this PR changes/adds
Add missing transitions to ContractNegotiation state machine, in particular:
AGREED to VERIFIED
and
VERIFIED to FINALIZED
this will happen only with a protocol different from ids-multipart because it does not support the new messages.
Why it does that
dataspace protocol implementation
Further notes

completed the RetryProcess refactor, now we have a component that can handle retry over a StatefulEntity easily configurable and controllable
this PR misses event dispatching, I'd like to have #2593 merged first. Related issue will come.
Added finalized (on consumer) and verified (on provider) methods on the manager interfaces, they are currently not used but they will be soon.
updated the state diagram as it contained a mistake about the consumer acceptance part (that's not implemented yet)

Linked Issue(s)
Closes #2595
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and Etiquette for pull requests for details)

E.g., currently ContractRejectionHandler directly calls the negtiation managers. I can leave it like that for the moment, but I thought we could "fix" this in the context of these clean-ups :)

LGTM. One question (because I came across this during protocol implementation). Why is ContractNegotiationServiceImpl only using the interfaces of ConsumerContractNegotiationManager? If I should only use ContractNegotiationServiceImpl to forward requests to the state machine, more methods have to be added. Can you take care? @ndr-brt

Yes, in the TransferProcessService I already added a notifyStarted method that means "the counter part started the transfer", we could add similar method to the ContractNegotiationService. I will provide an issue/PR for this.

Codecov Report
Patch coverage: 72.49% and project coverage change: +0.31 :tada:

Comparison is base (7adb2cb) 64.24% compared to head (ce8385e) 64.56%.

: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             @@
##             main    #2601      +/-   ##
==========================================
+ Coverage   64.24%   64.56%   +0.31%
==========================================
  Files         886      890       +4
  Lines       18173    18248      +75
  Branches     1086     1087       +1
==========================================
+ Hits        11676    11782     +106
+ Misses       6058     6032      -26
+ Partials      439      434       -5

Impacted Files
Coverage Δ

.../statemachine/retry/EntityRetryProcessFactory.java
0.00% <0.00%> (ø)

.../edc/connector/transfer/TransferCoreExtension.java
0.00% <0.00%> (ø)

...greement/ContractAgreementVerificationMessage.java
0.00% <0.00%> (ø)

...pes/agreement/ContractNegotiationEventMessage.java
0.00% <0.00%> (ø)

...act/spi/types/negotiation/ContractNegotiation.java
0.00% <0.00%> (ø)

.../edc/connector/transfer/spi/types/DataRequest.java
25.45% <ø> (ø)

.../spi/types/protocol/TransferCompletionMessage.java
0.00% <ø> (ø)

...fer/spi/types/protocol/TransferRequestMessage.java
0.00% <0.00%> (ø)

...nsfer/spi/types/protocol/TransferStartMessage.java
0.00% <ø> (ø)

...spi/types/protocol/TransferTerminationMessage.java
0.00% <ø> (ø)

... and 12 more

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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create linter baseline file for azure-sdk-for-go.

Create linter baseline file for azure-sdk-for-go. See parent issue for full details.
Parent Issue: azure/azure-sdk-tools#7122

Linter pipeline enabled and the baseline error file was added in this PR


Not working with dynamic generated object

It dosn't work with dynamic generated object
var input ='<input type="text" name="text1" maxlength="20/>"';
var $input = $(input);
$input.maxlength(); //this dosn't work
$(p).parent().append($input); //p is the div element

Please submit a fiddle if this is still an issue.


Add relabeling setting to add label only to response count metric

Dynamic relabeling can potentially result in a lot of time series being collected, especially related to the response and upstreams time histograms.
This change is intended to allow collecting more detailed information on response count without the side effect of adding more dimensions to the other metrics.
I hope this is something that you may find useful. Basically my first time writing any Go code, so you are very welcome to propose improvements to the code.

No, not stale. This is still on my to-do list, @stempler -- sorry about the wait.

Sorry for the lack of attention. I'll give this PR another look shortly.

Finally got around to have a detailed look at this -- merged :tada: (I pushed a few minor nitpicks on top of your branch myself). :+1: Thanks for the contribution and again sorry for the wait.

Finally got around to have a detailed look at this -- merged :tada: (I pushed a few minor nitpicks on top of your branch myself). :+1: Thanks for the contribution and again sorry for the wait.