14.09.2023 issue-52: insert chat id, token, token alias into database if token is valid

Added dependency for json web token in the pom.xml file
Added validateToken method in the TokenController class
Modified registerToken method to include validation of token in the TokenController class

PR-Codex overview
Detailed summary

Added a new exception class TokenAlreadyExists in the io.blamer.hub.exceptions package.
Created a new class TokenValidated in the io.blamer.hub.pg package, implementing the Tokens interface.
Added dependencies and imports in the TokenValidated class.
Implemented the add method in the TokenValidated class to validate and add a token.
Added a new method validateToken in the TokenValidated class to validate a token.
Modified the TokenController class to use the TokenValidated class as the Tokens implementation.
Updated the registerToken method in the TokenController class to use the new Tokens implementation.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@iColdLight thank you for your Pull Request. I'll assign someone to review it soon.
If this PR solves a todo from the code, please don't forget to remove it.

@h1alexbel please review this Pull Request. Deadline (when it should be merged or closed) is 2023-09-17T19:18:59.313614.
You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough.
Estimation here is 30 minutes, that's how much you will be paid. You will be paid even if this PR gets rejected.

@h1alexbel Don't forget to close this ticket before the deadline (2023-09-17T19:18:59). You are past the first half of the allowed period.

Does it mean that I need to work on creating JWT TokenProvider, JWT TokenFIlter, JWT User, etc.?

@iColdLight we don't need JWTs. We are dealing with simple text token from GitHub

And this token is not JWT, it's a just simple GitHub token, simple string

@h1alexbel Looks like you've missed the task deadline (2023-09-17T19:18:59). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@h1alexbel I couldn't find any assignee for this task. This is either because there are no contributors with role REV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

Hi!
I made new changes. I wonder, if you can see them.
And sorry for 3 commits in a row...
made slight changes in the last two.

@h1alexbel reminder

/codex Could there be a problem here in terms of non-blocking interaction?

@iColdLight take note of this comment

Could there be a problem here in terms of non-blocking interaction?

@l3r8yJ Yes, there could be a problem with non-blocking interaction in the code. The TokenValidation class implements the Tokens interface and overrides the add method. Inside the add method, there are multiple database interactions (validateToken, checkIfTokenExists, and this.tokenInsertion.add) that return Mono objects, indicating asynchronous operations.
However, the code does not utilize reactive operators like flatMap, map, or zip to chain these operations together and ensure non-blocking execution. Instead, it directly calls these methods synchronously, which may lead to blocking behavior and potential performance issues.
To ensure non-blocking interaction, the code should use reactive operators to compose the asynchronous operations and handle the results asynchronously.

I made a change to my TokenValidation class to ensure non-blocking interaction. May you take a look?

@l3r8yJ take a look, please


read_parquet engine parameter: 'fastparquet' vs. default 'auto' argument

Pandas will try pyarrow and default back to fastparquet if pyarrow is unavailable. Is there a specific reason you are specifying engine='fastparquet' because, if not, I think it would be better to leave it as the default 'auto' argument.

No, when I wrote the script originally I was having trouble with pyarrow so changed to fastparquet but I'm perfectly fine changing it to auto.

I'm planning on pushing another update this weekend including an added step to the clean_data() function and a new more comprehensive file for import_schedule() function, I'll include this change in that update.

Sounds good. I'll go ahead and close this.


Add support for datadog span types

This adds support for datadog's built in type.
See https://docs.datadoghq.com/tracing/guide/send_traces_to_agent_by_api/
These correspond to

@xeinherjar thanks for the contribution. This will be released tonight as a part of v0.13.0


Issues with a conflict between the auto-tag workflow and ReleasePlease bot

Persona

Maintainer of this repo
Description

We have an auto-tag GHA workflow that creates a new tag on every push to main. We also have the ReleasePlease bot that does similar things. There is no good understanding at the team level as to how we want to handle automatic releases.
Use Case

Having tags with 2 different formats is confusing to maintainers and downstream automation.
Impact

If this is not addressed we will continue to have 2 competing methods of automatic releases creating tags in 2 different formats.
Completion

An ADR has been written and approved describing how we as a team want our automated release process to work.
The GHA workflows and/or ReleasePlease configuration has been updated to reflect the decision.

Additional Context

none

Great idea to capture this in and ADR. Below are the closest things we have to one currently:

#271 & PR that closed issue

The intent is to migrate all modules to their own repo, which will have independent testing & releases. This repo's purpose is both a comprehensive root module example that consumes those independent modules to help inform how they are intended to be used and validation of functionality across multiple dependencies / upstream modules that update on disparate cadences.


CocoaPods Task fails to find private repos latest update.

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name:
list here (V# not needed):  CocoaPods
Environment

Server - Azure Pipelines

Issue Description
The CocoaPods task with forceRepoUpdate fails to update private repo's. It can't find the latest version. This was not a problem several weeks ago and has only just started to go into error, failing the pipeline.
As this is a private repo I have several tasks that occur before this, first I manually add the private specification. I cache the git credentials securely and manually clone the private repository.
After this I run the normal CocoaPods@0 task to update/install the pods that are in use with the project. This did work without any issues previously but has started to fail to pipeline due to the fact that it can not find the version of the pod in the private repo. No new versions have been published since the pipeline last succeeded.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


Upload 1.20.4 release to CurseForge?

I see that the mod has been updated to 1.20.4, but for whatever reason that one hasn't been uploaded to the CF page yet. Is there an ETA on doing that?
Thanks!

I believe it has, however it was not marked as being for Minecraft 1.20.4. I fixed it, and the file is available here.


Dependency on Git

Looks like the fork of Browserify that this package depends on depends on Git being installed on the system. That's a significant dependency ... any chance it can be done away with? Why is it necessary?

How is it depending on git? Via the GitHub project reference?

And by the way, I hope this is only a temporary issue until browserify merges a number of crucial bug fixes, see here.

He means the node-browserify dep on GitHub versus the usual npm. And yeah, it can't be done away with until https://github.com/substack/node-browserify/pull/1037 and https://github.com/substack/node-browserify/pull/1034 get merged.

Yes. Thanks for the explanation! Hopefully it gets merged soon.


Upgrade OpenDDS and librdkafka

Description

Update OpenDDS to version 3.25 and librdkafka to version 2.2.0
Reviewed .NET project and workflows
Fixed Node 12 deprecation warning
Related Issue

Closed #40
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Forced "squash and merge" due to space limit reached.


Toggle & Turbolinks - turbolinks:load event forces _renderAsyncFunction to fire regardless of toggle

Hey there!
First and foremost, thanks for all the work with render_async, we use it everyday in production!
I was the guy using the toggle feature prior to your release. After updating to 2.1.1, I noticed some strange behavior: when turbolinks is enabled, the _renderAsyncFunction will fire regardless of the toggle configuration.
I believe this little section of code is responsible
  <% if turbolinks %>
  document.addEventListener("turbolinks:load", function (e) {
    e.target.removeEventListener(e.type, arguments.callee);
    _renderAsyncFunction();
  });
  <% elsif !toggle %>
  document.addEventListener("DOMContentLoaded", _renderAsyncFunction);
  <% end %>

I'm sending you a quick PR, let me know if you want me to change anything.
If this isn't the intended behavior, could you please elaborate on your thinking?
Thanks!
edit:
Realized I can probably clarify and describe our use case a bit more.
Assume the following render_async call:
<%= render_async performance_stats_path(campaign), toggle: { selector: "#load-campaign-#{campaign.id}-stats", event: 'click'}, interval: 60000 do %>
  <a id="<%= "load-campaign-#{campaign.id}-stats" %>" class="btn btn-default btn-xs">
    Load Statistics

<% end %>

Basically we render a button as a placeholder and the button contains the toggle selector. With turbolinks off, everything functions as expected: when you click the button, render_async works its magic and loads the partial, replacing the button with some awesome stats for our customers.
When turbolinks config is enabled, the partial is loaded on turbolinks:load regardless of any toggle configuration. It took me a while to actually root this out because I use all of the following:

rails fragment caching
turbolinks (which has its own caching mechanism)
render_async (with caching, toggle, and interval used in different situations)

and I did not want to file an issue prematurely.
We're running my branch in staging now and everything is functioning as expected. We even have a render_async call with caching, toggle, and interval + turbolinks, and it's working well on our initial evaluation.
Once again, thanks!

Hey @preetsethi, thanks for submitting the issue! Explaining everything in details really helped, thank you once again!
Sorry for making this bug, I will try to resolve it ASAP and release a new version that fixes it.

I've just released https://rubygems.org/gems/render_async/versions/2.1.2, let me know if it works for you @preetsethi


Objects Options, but string value

I use vue-multiselect inside a custom component, with an array of objects as options, but the value is just a string. I tried to follow the documentation, but can't find a solution for the problem.
Please see https://jsfiddle.net/91ztus9n/3/
There is the parent component "App" which has just 2 elements:

The "Child" component <my-component>, which is just a wrapper for vue-multiselect
A <pre> tag showing the value.

As you can see, the value of the parent component "App" is "medium" in the beginning. The first problem is, that the selectbox hasn't selected the right option. If you click on the selectbox and select one option (for example "Large Size"), the parent component is updated as expected and the value is set to "large", but the selectbox now says once again "Select option" instead showing "Large Size".
Can you give me a helping hand? What am I doing wrong?

Hey!
This is because Vue-Multiselect is accepts and returns whole objects for Object arrays. This means that if you want to preselect a value, you have to pass the object to Multiselect. There are reasons why it is working this way – most notably the support of dynamic asynchronous options where you wouldn’t be able to show the selected label if it wasn’t already present in the options list (to look up by id).
Of course you can always build a wrapper component like you did, so that you can operate just on the id property instead of passing the whole object.
I made some adjustments to your code that make it work like you described: https://jsfiddle.net/shentao/yo767npo/

Thank you very much for your help. I wasn't sure if this behavior was a "bug or a feature" ;)

Trust me I got this feeling quite often regarding this library and the extensive API... Hopefully 3.0 will make it much smaller and flexible enough for all those fancy features to be implemented on the user side.


[Bug Report]: Publishing of 'latest' versions is affected by cmdlet default behavior change

Describe the bug
When scanning some pipeline logs - I found the following warning: The module "br:adpsxxazacrx001.azurecr.io/bicep/modules/web.sites.slots.hybrid-connection-namespaces.relays:0.1" already exists in registry. Use --force to overwrite the existing module.
It appears the default behavior of the publishing cmdlet changed and we need to append a --force parameter to re-establish the desired behavior.
A small fix, but important.
To reproduce
Run a pipeline that publishes.
Code snippet
No response
Relevant log output
No response

Also related to #3339


Fix Typo

Codecov Report

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

@@           Coverage Diff           @@
##           master     #448   +/-   ##
=======================================
  Coverage   42.27%   42.27%
=======================================
  Files          32       32
  Lines        2278     2278
=======================================
  Hits          963      963
  Misses       1315     1315

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 c0662e8...7343550. Read the comment docs.

Hi, thanks for your contribution!
Does the file in Spanish, Portuguese and Polish also have the same typo?

Hi, thanks for your contribution!
Does the file in Spanish, Portuguese and Polish also have the same typo?

yes Spanish, Portuguese and Polish also have same typo just now noticed.

Hi, thanks for your contribution!
Does the file in Spanish, Portuguese and Polish also have the same typo?

yes Spanish, Portuguese and Polish also have same typo just now noticed.

Would it be possible to include this change in that PR? Please.

Hi, thanks for your contribution!
Does the file in Spanish, Portuguese and Polish also have the same typo?

yes Spanish, Portuguese and Polish also have same typo just now noticed.

Would it be possible to include this change in that PR? Please.

yea sure


fix: Use endpointURL and region in s3_fs

Describe your changes :
Added endpointURL and region_name in s3_utils.py because this was missing in S3FileSystem
and crashed the ingress (because it called the wrong S3 - storage).
This fixes https://github.com/open-metadata/OpenMetadata/issues/10279

Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] All new and existing tests passed.

@SuperNascher can you please check py_format and update the PR?

@harshach Done :+1:


Invoking interop jobs automatically

Investigate a way to invoke the rerunning of a test job automatically. This can happen, say, when a ProvPy build on Travis is successful. Then, the provpy-interop-job will automatically be run.
The ProvPy repo page, for example, can include another icon for the status of its interop tests and a developer can be aware of the existence of any interop issue without having to invoke the interop tests manually.

A test job could be rerun automatically via use of the Travis Client (as described in Travis Client). If the ProvPy build succeeds, it could invoke the travis restart command to restart the associated interoperability tests.
I implemented a simple example of this in travis-sender and travis-receiver. travis-receiver has a Travis CI job that just prints a message. travis-sender has a Travis CI job that uses the Travis Client:
language: python

python:
  - 2.7

install:
  - gem install travis

script:
  - echo "Hello"
  - ruby -v
  - travis version --skip-completion-check
  - travis login --skip-completion-check --github-token $GITHUB_TOKEN
  - travis history --skip-completion-check -r mikej888/travis-receiver
  - travis restart --skip-completion-check -r mikej888/travis-receiver
  - travis logout --skip-completion-check

The value of GITHUB_TOKEN being a hidden environment variable configured via the Travis CI user interface. The --skip-completion-check flag is (unfortunately) required otherwise travis pops up an interactive prompt asking  Shell completion not installed. Would you like to install it now? |y|.
Every time this job is run, it triggers a re-run of the last travis-receiver job. See the Travis CI jobs for:

travis-sender
travis-receiver

However, I don't think this approach should be used in practice, since it becomes increasingly complex for third-party developers to set up to run their own tests. They would need to:

Clone ProvPy Git repository on GitHub.
Enable this under Travis CI.
Configure Travis CI for this job with their GitHub API key.
Clone ProvPy interoperability test job Git repository on GitHub.
Enable this under Travis CI.
Update ProvPy .travis.yml with the name of their Travis CI job (e.g. mikej888/provpy-interop-tests)
Update ProvPy interoperability test job .travis.yml with the location of their ProvPy repository.

This introduces a circular dependency between the two repositories. It also makes submitting pull requests back to the canonical ProvPy repository more difficult, as users have to restore the default .travis.yml file before doing so (which they may forget to do).
More generally, I've been thinking about the use of Travis CI to run interoperability tests via Travis Client, and I'm not so sure this is a good idea, as it is somewhat of an abuse of the concept of Travis CI (that is, tests are rerun every time a change is committed). It would be more in the spirit if the interoperability tests for ProvPy, for example, were rerun every time a change to ProvPy was committed (but without the convoluted approach above). This relates to #11.


AWSSDK.Core triggers CWE-798 - Hardcoded credentials

Describe the bug
This is more of an FYI but our organization just turned on AWS Inspector for AWS Lambda and we are getting false positives for the finding CWE-798 - Hardcoded credentials in AWSSDK.Core.
From the finding it looks like its complaining because the string 'password' is contained within the name of a property (EC2InstancePassword).

Expected Behavior
Not trigger CWE-798 - Hardcoded credentials
Current Behavior
Triggers CWE-798 - Hardcoded credentials
Reproduction Steps
Deploy Lambda with AWSSDK.Core and run AWS Inspector against it.
Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.Core
Targeted .NET Platform
.NET 6
Operating System and version
AmazonLinux

Needs review with the team.

P124646133

@skirk-mpr I reviewed this issue with the team. There doesn't appear to be a way to ignore this false positive reported by AWS Inspector. I have opened internal ticket with AWS Inspector team to get their inputs on how to disable such false positives. Would report any updates here as it is available.

@ashishdhingra -- thanks so much, appreciate you circling back regarding this!  Figured there wasn't really something warranting a fix on the SDK side, but either way, wanted to flag it so your team was aware.
Also, somilar thing is also getting flagged with Amazon.Extensions.CognitoAuthenitcation

@skirk-mpr Additionally you might refer Suppressing Amazon Inspector findings with suppression rules on how to create rule suppress such warnings.

Thank you, @ashishdhingra!

@skirk-mpr I would close this issue for now since I have created ticket for the service team. They are working internally on a fix.

@skirk-mpr Based on communication from service team, looks like they have implemented fix to exclude AWS SDK(s) from scanning.


Moved login/check functionality out of init

Having login and checks functionality in init made a few things
difficult. Mainly, the CLI can't properly forsee login errors
when creating the SDK CfdeClient. Logout() for example was difficult
to call from the cli due to Fair Research Login attempting to
steal control from the user in a new login flow. Separating login from
init allows for the CLI to properly check state before a login
flow is initiated.
Having login() be a separate method also allows credentials to be
passed into the client if the user prefers to do their own login
flow insead. This may be required if the user is operating on a portal
and cannot do a native flow, or alternatively is using additional tools
and wants to combine other auth flows into this one.
Lastly, this makes unit testing easier by splitting responsibilities
into smaller siloed functions. I think in general, keeping init
clean allows more flexibility in client usage without necessarily
enforcing auth, and it will make mocking for unit tests much easier.
Fixes after smoke-testing https://github.com/nih-cfde/submission-workflow/issues/4
Going to see if I can sneak in some unit tests, but otherwise should be good to go.

Smoke test looked good, merging...


admc: translate desc to English

No other package in lure-repo has a non-English package description; let's keep things consistent, shall we?

No other package in lure-repo has a non-English package description; let's keep things consistent, shall we?
And you can also respect the language of the authors of the utility) Automated translation in 2022 allows you to get acquainted with the text written in another language without much difficulty.

@xpamych That's not the issue, the description is supposed to be useful for anyone at a glance to tell what the program is for, you can kind of guess what the tool is for in this case because it already contained Active Directory, but if you have to use a translator or look at the upstream project README (which is in English btw) to figure out what it's meant to be the description has utterly failed at its sole purpose.

@JamiKettunen According to your logic, for me as a Russian-speaking person, any description does not meet its purpose, because I have to use a translator in 99% of cases.

I don't believe LURE itself has localization of any kind yet either, but I think cases like the package description should be translated to the user's preferred language locally when displayed by the lure tool according to LC_* settings of the Linux system, it would be tedious work to store translations of the description for each package, though that could be another choice to go forward too (and might be better for the web UI if that gets localization support).

Also, I don't want to offend anyone, but if we operate with numbers, there are unlikely to be more English speakers than native Chinese or even Indian speakers, but this does not mean that we have to do a description for everyone in these languages. So the option when the description will be selected in accordance with the locale is the most correct, albeit time-consuming.

Let's wait on this until localization is properly implemented. I'll close this PR for now.


Wipe possible existing RAID signatures when taking new volume

(still testing...)

Hm, it does not seem to work for me


getAvailablePurchases does not return consistent result

Version of react-native-iap
2.2.0
Version of react-native
0.57.4
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
getAvailablePurchases() should return consistent result
Actual behavior
getAvailablePurchases() does not return correct values if it is called without much interval
Tested environment (Emulator? Real Device?)
Real Device (probably only sandbox...? It is  hard to test in production)
Steps to reproduce the behavior
Call getAvailablePurchases() as follows.
If I call as the follows the result can be very random.(The first time is always correct. I need to wait  for enough time then it returns 7 again)
      purchases = await RNIap.getAvailablePurchases();
      console.log('restorepurchase1', purchases.length);  // 7
      purchases = await RNIap.getAvailablePurchases();
      console.log('restorepurchase2', purchases.length);  // 2
      purchases = await RNIap.getAvailablePurchases();
      console.log('restorepurchase3', purchases.length);  // 1
      purchases = await RNIap.getAvailablePurchases();
      console.log('restorepurchase4', purchases.length);  // 2
      purchases = await RNIap.getAvailablePurchases();
      console.log('restorepurchase5', purchases.length)  ;// 2

We are having a few issues in restoring purchases in ios. We need the help of the community to find out the problem and willing to providing a solution.

I think this is sandbox only issue, it did not seem to occur in production. In that case... this bug does nothing to do with this library?

Closing this as this is sandbox related issue, which is apple's fault.


Changes performed

Blanks removed
Comments added
Couldn't update previous branch was giving errors so forked again
What does this PR do?
Dynamic messages added to console
API and console file updated for cases
Where should the reviewer start?
Console and API file
How should this be manually tested?
Normally use the console
Any background context you want to provide?
Relates Issue 34
What are the relevant tickets?
https://github.com/a-rmz/JEasyCrypto/issues/34

I couldn't update the previous branch because errors related to binaries, git was not allowing me to commit, so forked again and the commit was successful.

Conflicts are arising, not able to push resent changes.


fix: update the reference-architectures

Description
Issue: https://github.com/terraform-ibm-modules/terraform-ibm-cos/issues/580
Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

Since Allen is unavailable this week, I'm going to approve and merge this. If any follow up actions, lets get them into a new issue

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

GitHub release
v8.1.9

Your semantic-release bot :package::rocket:


The project is broken

We simply clone it, run a flutter pub get and run. This version is receiving error for Android X and other stuff, and when you look for how to migrate the project its like building a rocket, im like 2 hours trying to create new Android X and copy files, or trying to migrate the currently project, but nothing works, it envolves a lot of build.gradle file updates, and confusing stuff.

I found a pull request where the guy just edit 2 files. And the documentation and other stackoverflow links that i read were point other solutions


PHP inside comment loses syntax highlighting

First - thanks for making this extension! It's really nice to get some highlighting in the comments.
When using this extension, PHP inside the comments lose highlighting and is highlighted like a string.
With extension active:

Without extension active:

Hello, please can you try the latest release (0.04). This should now support PHP syntax highlighting within the comment.

Thanks! It's perfect!


D1.7 Profile Updates to Running H3 Crawls

Add new functionality to Target Instance to allow updating of profile for a running H3 crawl. Functionality would be available via the Profile tab.
Might need to investigate which parts of the profile can be updated once a crawl has started.

Closed by PR #48


Added Mpt-Instruct-DotNet-S

Artificial Intelligence/Mpt-Instruct-DotNet-S - https://github.com/NethermindEth/Mpt-Instruct-DotNet-S
LLM that can generate and explain C# code (and its C# wrapper to run on CPU with 5GB ram+, with Console and Blazor sample projects)

Mod notes: Merged because AI needs more awesome content


Fix filename for "components/_button.scss" in "main.scss" file

Title.

Nice catch, thank you very much. :)


TEAMNADO-7820 adding rpms signature scan to tekton pipeline

adding rpms signature scan to pull request and push yamls
No testing needed

/retest

/retest

/retest

/retest

/retest


Country code is not working

Hey support,
I do have an issue with the country code.
No matter which country code I choose, the SMS is always sent on the country code +33 even though it is not the one I chose.
I'm using the latest version of FirebaseUI.
private void authSMS() {         List<AuthUI.IdpConfig> providers = Collections.singletonList(new AuthUI.IdpConfig.PhoneBuilder().build());         startActivityForResult(                 AuthUI.getInstance()                         .createSignInIntentBuilder()                         .setAvailableProviders(providers)                         .setTheme(R.style.FirebaseUI)                         .build(), RC_SIGN_IN);     }
I'm using this code to create the Instance.
Can you please help me ?

Hi, thanks for filing this issue! We are unable to promise any timeline for this, but if others also have this issue, adding a +1 on this issue can help us prioritize adding this to the roadmap.
(Googler-only internal tracking bug: b/217765697)

Hey @lisajian,
Do you have any news regarding this issue ?

Hi, @Sosotess93 most likely this is a duplicate for https://github.com/firebase/FirebaseUI-Android/issues/2005
As far as I understand it can be fixed with one line, they even have a PR with fix.
But for some reason it has been going on for 4 months and no one cares about to merge this PR and release a fixed version.

This is fixed in #2025 and is available in the 8.0.1 release.


docs: removed deprecated svgSrc from README

fixes #2690

Thank you! I was just about to submit this same PR :)


Records in push_subscriptions automatically disappear

Hi there,
Thanks for the awesome library and demo.
I have been trying out this in localhost and it works pretty well.
I set up a Artisan command to send notification (to test) like this:
Artisan::command('web-push', function () {
    $user = \App\Model\User::find(1);
    $user->notify(new \App\Notifications\HelloWorldNotification());
});

After run artisan web-push, the notification appears. But when I click on the notification, it opens the URL (that works as expected), but then the record in push_subscriptions table gets removed automatically.
I dont have any code to remove that at all, and I can 100% reproduce this by:

click "enable push notification"
see a new record stored in push_subscriptions table
hit artisan web-push
see new notification(*)
click on the notification
it opens localhost(**)
hit artisan web-push
now the record in push_subscriptions gets removed and no notification is showed

Notes:

(*): if I dont click the notification, and keep hitting artisan web-push, I can see new notifications are showing up
(**): after localhost is opened, I checked database and still can see record in push_subscriptions

My service worker JS is copied from https://github.com/cretueusebiu/laravel-web-push-demo/blob/master/public/sw.js and the demo is copied from https://github.com/cretueusebiu/laravel-web-push-demo/blob/master/resources/js/components/NotificationsDemo.vue, so nothing is changed.
P/S: Not sure if this is relevant, but I console.log('subscription', subscription) in https://github.com/cretueusebiu/laravel-web-push-demo/blob/master/resources/js/components/NotificationsDemo.vue#L74 and it always return null, even after I have enabled the push notification (record is stored in database successfully).
Thanks,

Also found this in logs:

Notification failed to sent for subscription https://fcm.googleapis.com/fcm/send/...: Client error: POST https://fcm.googleapis.com/fcm/send/... resulted in a 410 Gone response:
push subscription has unsubscribed or expired.


Communication between Micro-ROS Agent (@RPi 4B) and Client (@esp32-s3) is not established

Hi again,
I was able to build micro-ROS client for esp32-s3 using custom transport / UART (int32_publisher_custom_transport).
And I was able to build ROS2 humble and micro-ROS agent (https://github.com/micro-ROS/micro_ros_setup?tab=readme-ov-file#building-micro-ros-agent) on a RaspberryPi 4B. My ROS2 setup and micro-ROS agent are working as expected together with a micro-ROS client running on a STM32. Now I am trying to get this to work with an esp32-s3.
There seems to be a problem establishing the connection.
RCCHECK(rclc_support_init(&support, 0, NULL, &allocator));
I am sure the correct UART port has been selected. When I monitor the UART port on the other side on the RPi, I see that the client is trying to establish a connection. The client sends data (31 bytes) 10 times.
My settings in colcon.meta are as follows:
        "microxrcedds_client": {             "cmake-args": [                 "-DUCLIENT_PIC=OFF",                 "-DUCLIENT_PROFILE_DISCOVERY=OFF",                 "-DUCLIENT_PROFILE_UDP=OFF",                 "-DUCLIENT_PROFILE_CUSTOM_TRANSPORT=ON",                 "-DUCLIENT_PROFILE_SERIAL=OFF",                 "-DUCLIENT_PROFILE_TCP=OFF",                 "-DUCLIENT_MIN_HEARTBEAT_TIME_INTERVAL=1"             ]         },         "rmw_microxrcedds": {             "cmake-args": [                 "-DRMW_UXRCE_XML_BUFFER_LENGTH=400",                 "-DRMW_UXRCE_TRANSPORT=custom",                 "-DRMW_UXRCE_MAX_NODES=1",                 "-DRMW_UXRCE_MAX_PUBLISHERS=2",                 "-DRMW_UXRCE_MAX_SUBSCRIPTIONS=2",                 "-DRMW_UXRCE_MAX_SERVICES=1",                 "-DRMW_UXRCE_MAX_CLIENTS=1",                 "-DRMW_UXRCE_MAX_HISTORY=1"             ]         },
Any help is welcome.

When I use
	while(rmw_uros_ping_agent(timeout_ms, attemps) != RMW_RET_OK) 	{ 		printf("Please, start your micro-ROS Agent first\n"); 	}
the client is sending the following frames:

The micro-ROS agent is running but nothing happens.

problem solved. I changed UART source_clk to UART_SCLK_XTAL.


feat: rely on default ktlint version provided by spotless (1.0.0)

@soulcramer & @kazaky can you both confirm if you agree on this configuration?


search bar missing on mobile device?

Hi,
I use search a lot but it is not showing on mobile device. Default safari isn't showing, tried installing Firefox but no avail too. Any idea on how to fix it? No problem when i access it through desktop browsers. My device is ipad..

I don't think anyone ever optimized the UI for mobile browsers.

By removing visible-lg in _navigation.html.erb actually makes it show in small screen again 😄

That's a good catch! Do you mind sending a pull request with the change?


Handle restarted containers and add --include/--exclude command line flags

@mgmorcos This PR looks good. Are you able to merge this, and publish it to NPM?

@program247365 I do not have write access to this repository

I believe @tobico or @zendesk have to be the ones to accept this PR.


Downloading some episodes cannot be played

When I download episode 6, it can't be played after the download is complete. But I downloaded episode 5 again and it played fine.
./crunchy archive -l ja-JP https://beta.crunchyroll.com/series/G5PHNM403/the-dawn-of-the-witch[S01E06] -v -g 5
Download process:
[debug] Entering season 1, episode 6 with 0 additional formats
[debug] Set up signal catcher
[info] Downloading episode `Knowledge of Ignorance` to `Knowledge of Ignorance.mkv` (merging audio for additional formats)
[info]  Episode: S01E06
[info]  Audio: ja-JP
[info]  Subtitle:
[info]  Resolution: 1920x1080px
[info]  FPS: 23.97

[debug] Merging segments
[debug] Downloaded 'ja-JP' video
[debug] Downloaded 'ar-SA' subtitles
[debug] Downloaded 'en-US' subtitles
[debug] Downloaded 'fr-FR' subtitles
[debug] Downloaded 'de-DE' subtitles
[debug] Downloaded 'it-IT' subtitles
[debug] Downloaded 'pt-BR' subtitles
[debug] Downloaded 'ru-RU' subtitles
[debug] Downloaded 'es-419' subtitles
[debug] Downloaded 'es-ES' subtitles
[debug] FFmpeg merge command: ffmpeg -y -i "/tmp/Knowledge of Ignorance.mkv_ja-JP_video_4228809968.ts" -i "/tmp/Knowledge of Ignorance.mkv_ar-SA_subtitle_2246873797.ass" -i "/tmp/Knowledge of Ignorance.mkv_en-US_subtitle_96231048.ass" -i "/tmp/Knowledge of Ignorance.mkv_fr-FR_subtitle_3549463093.ass" -i "/tmp/Knowledge of Ignorance.mkv_de-DE_subtitle_3645851011.ass" -i "/tmp/Knowledge of Ignorance.mkv_it-IT_subtitle_2836418908.ass" -i "/tmp/Knowledge of Ignorance.mkv_pt-BR_subtitle_2467827141.ass" -i "/tmp/Knowledge of Ignorance.mkv_ru-RU_subtitle_3717983405.ass" -i "/tmp/Knowledge of Ignorance.mkv_es-419_subtitle_1758587536.ass" -i "/tmp/Knowledge of Ignorance.mkv_es-ES_subtitle_3931097704.ass" -map 0 -map 1 -map 2 -map 3 -map 4 -map 5 -map 6 -map 7 -map 8 -map 9 -metadata:s:v:0 language="ja-JP" -metadata:s:v:0 title="Japanese" -metadata:s:a:0 language="ja-JP" -metadata:s:a:0 title="Japanese" -metadata:s:s:0 language="ar-SA" -metadata:s:s:0 title="Arabic" -metadata:s:s:1 language="en-US" -metadata:s:s:1 title="English (US)" -metadata:s:s:2 language="fr-FR" -metadata:s:s:2 title="French" -metadata:s:s:3 language="de-DE" -metadata:s:s:3 title="German" -metadata:s:s:4 language="it-IT" -metadata:s:s:4 title="Italian" -metadata:s:s:5 language="pt-BR" -metadata:s:s:5 title="Portuguese (Brazil)" -metadata:s:s:6 language="ru-RU" -metadata:s:s:6 title="Russian" -metadata:s:s:7 language="es-419" -metadata:s:s:7 title="Spanish (Latin America)" -metadata:s:s:8 language="es-ES" -metadata:s:s:8 title="Spanish (Spain)" -disposition:s:0 0 -c copy -f matroska /tmp/3676206230
[debug] Download finished
[debug] Stopped signal catcher

And the file that cannot be played normally contains a German subtitle with an abnormal time，When I remove this subtitle file and remix the stream, the video can be played normally.
ID                                       : 6
Format                                   : ASS
Codec ID                                 : S_TEXT/ASS
Codec ID/Info                            : Advanced Sub Station Alpha
Duration                                 : 3 时 59 分
Compression mode                         : 无损
Title                                    : German
Language                                 : German (DE)
Default                                  : 否
Forced                                   : 否

This is the 5th episode, it can be played normally after it is downloaded, but the 6th episode cannot.
./crunchy archive -l ja-JP https://beta.crunchyroll.com/series/G5PHNM403/the-dawn-of-the-witch[S01E05] -v -g 5
[debug] Entering season 1, episode 5 with 0 additional formats
[debug] Set up signal catcher
[info] Downloading episode `I Decide Who's Worthy` to `I Decide Who's Worthy.mkv` (merging audio for additional formats)
[info]  Episode: S01E05
[info]  Audio: ja-JP
[info]  Subtitle:
[info]  Resolution: 1920x1080px
[info]  FPS: 23.97

[debug] Merging segments
[debug] Downloaded 'ja-JP' video
[debug] Downloaded 'ar-SA' subtitles
[debug] Downloaded 'en-US' subtitles
[debug] Downloaded 'fr-FR' subtitles
[debug] Downloaded 'de-DE' subtitles
[debug] Downloaded 'it-IT' subtitles
[debug] Downloaded 'pt-BR' subtitles
[debug] Downloaded 'ru-RU' subtitles
[debug] Downloaded 'es-419' subtitles
[debug] Downloaded 'es-ES' subtitles
[debug] FFmpeg merge command: ffmpeg -y -i "/tmp/I Decide Who's Worthy.mkv_ja-JP_video_3629755215.ts" -i "/tmp/I Decide Who's Worthy.mkv_ar-SA_subtitle_594230327.ass" -i "/tmp/I Decide Who's Worthy.mkv_en-US_subtitle_775737117.ass" -i "/tmp/I Decide Who's Worthy.mkv_fr-FR_subtitle_2237748135.ass" -i "/tmp/I Decide Who's Worthy.mkv_de-DE_subtitle_2171227641.ass" -i "/tmp/I Decide Who's Worthy.mkv_it-IT_subtitle_1034428868.ass" -i "/tmp/I Decide Who's Worthy.mkv_pt-BR_subtitle_1369825909.ass" -i "/tmp/I Decide Who's Worthy.mkv_ru-RU_subtitle_2177413224.ass" -i "/tmp/I Decide Who's Worthy.mkv_es-419_subtitle_1392795710.ass" -i "/tmp/I Decide Who's Worthy.mkv_es-ES_subtitle_3336197141.ass" -map 0 -map 1 -map 2 -map 3 -map 4 -map 5 -map 6 -map 7 -map 8 -map 9 -metadata:s:v:0 language="ja-JP" -metadata:s:v:0 title="Japanese" -metadata:s:a:0 language="ja-JP" -metadata:s:a:0 title="Japanese" -metadata:s:s:0 language="ar-SA" -metadata:s:s:0 title="Arabic" -metadata:s:s:1 language="en-US" -metadata:s:s:1 title="English (US)" -metadata:s:s:2 language="fr-FR" -metadata:s:s:2 title="French" -metadata:s:s:3 language="de-DE" -metadata:s:s:3 title="German" -metadata:s:s:4 language="it-IT" -metadata:s:s:4 title="Italian" -metadata:s:s:5 language="pt-BR" -metadata:s:s:5 title="Portuguese (Brazil)" -metadata:s:s:6 language="ru-RU" -metadata:s:s:6 title="Russian" -metadata:s:s:7 language="es-419" -metadata:s:s:7 title="Spanish (Latin America)" -metadata:s:s:8 language="es-ES" -metadata:s:s:8 title="Spanish (Spain)" -disposition:s:0 0 -c copy -f matroska /tmp/458289425
[debug] Download finished
[debug] Stopped signal catcher

./crunchy download -r best https://beta.crunchyroll.com/series/G5PHNM403/the-dawn-of-the-witch[S01E06] -g 5
I tried downloading the .ts problem and it was no problem. Seems to be a problem converting to .mkv

I tested again when I use:
./crunchy download -o mkv -r best https://beta.crunchyroll.com/series/G5PHNM403/the-dawn-of-the-witch[S01E06] -g 5 -v
At this time, the downloaded mkv file can be played normally.

That's a weird issue, normally this shouldn't happen. Maybe there was a issue while downloading the subtitle file which caused this.

The problem reappeared when downloading "Case Closed (Detective Conan)"
Remix and remove all subtitles and the problem goes away
Just give feedback on the problems in use

For me, the episode 6 video somehow extends to 4 hours length, but only around 24 minutes (the actual length the episode) can be played, after this my video player (vlc / mpv) stops the video entirely. If I remove the German subtitle the video has the normal 24 minutes length. Mysterious.

In the .ass file which contains the German subtitles, one entry has the end-time of around 4 hours. This is the cause of the extended video length. Once again a design decision I can not understand ¯\(ツ)/¯.

@ltsj which video player are you using? When I download it on Windows 10 it can be played normally with the built-in video player, at least until the addressed subtitle bug occurs

I use the latest version of PotPlayer

Fixed in release v2.3.1


Testing BasicVSR Pre-trained

I have downloaded the pre-trained model path of BasicVSR from the given link and tried to test it but I am getting the error shown below:
**The model and loaded state dict do not match exactly
missing keys in source state_dict: step_counter**

That's normal, If I am not mistaken, only the model is saved on the website's checkpoint. If it is not present, it is set to 0.
step_counter is added at train time to count training step. Afterwards, if you resume the training, iteration numbers will start at the value of step_counter.

What am I supposed to do to test the Pre-trained BasicVSR model?
Please let me know.

You can just ignore this warning while running the following command:
python demo/restoration_video_demo.py ./configs/restorers/real_basicvsr/realbasicvsr_c64b20_1x30x8_lr5e-5_150k_reds.py https://download.openmmlab.com/mmediting/restorers/real_basicvsr/realbasicvsr_c64b20_1x30x8_lr5e-5_150k_reds_20211104-52f77c2c.pth data/<path to data folder or video> ./<output folder or video>

Yes, @gauone is right. You can ignore this warning. This does not affect the result.

What am I supposed to do to test the Pre-trained BasicVSR model?
Please let me know.

Thank you!


Implement a Robust Solution for LocalStorage

Troughout the Venus-Simulator a LocalStorage is used for storing of settings. Most of these settings are the exact same mentioned in #54 .
At first we used the node internal node-Localstorage. This lead to some issues as it needs a folder path where it writes to. Because of the sandboxed nature of VSCode and differences between Operating Systems this lead/can lead to issues, see #50 .
Right now we use a MockStorage which stores the settings in memory. This is a somewhat bad hack.
If possible we should eliminate the dependicies from localstorage and use the internal extension-settings from VSCode to store the Venus-Settings. This would be the cleanest way IMO and would also solve #54

The Mockstorage Solution proved to be working fine.
I found out that the Venus Simulator saves Crash Reports in LocalStorage. It would be nice to restore that functionality.
However one of the advantages of the Mock Solution was that you don't need to use a folder at all and can open files directly. I also wanna keep that functionality.
The Solution is i think:

Initialize with mockStorage
Check if we are in a vscode-folder or workspace
If yes then we use the builtin vscode storage to save in a logs or scratch folder
If not then we keep using mockStorage


material-select not showing predefined value from ng-model

    html:
    <select ng-model="editUserCtrl.state" material-select>
       <option value="active">Active</option>
       <option value="locked">Locked</option>
    </select>

    JS:
    vm.state = "active"

Result - when I open the page active option is not selected.

See this: http://stackoverflow.com/questions/17606936/angularjs-dot-in-ng-model
You are storing a property directly on your controller, Angular doesn't handle that well with nested scopes.
It works for me when storing the state as an object.

Ok I got what I was wrong. It was not really stored as a property directly. I had it stored in vm.model.state. However, on controller init I made I request to backend to fetch the model. So before the request was done, vm.model.state was undefined. Because of that a third, blank option was prepended to select and the black option was treated as active and locked was mistreated as active. That's a bug in my opinion. However, adding watch to select solves the issue.

adding watch should only fix it, if the actual options were changed. If it is only the value in ng-model that was changed, then watch shouldn't make a difference.
If you look at this relevant code from the script:
$timeout(initSelect);
if (attrs.ngModel) {
    scope.$watch(attrs.ngModel, initSelect);
}
if ("watch" in attrs) {
    scope.$watch(function () {
        return element[0].innerHTML;
    }, function (oldVal, newVal) {
        if (oldVal !== newVal) {
            $timeout(initSelect);
        }
    });
}

You'll see that watch shouldn't do anything, except if the options change.
It might look further into it later, because it sounds weird.

If you have some standalone code that demonstrates the issue (you can just use setTimeout() instead of a server call), then it would be nice if you send it my way.

There u go:
http://jsfiddle.net/Lvc0u55v/5129/

That is perfect, thanks.

Turns out you were right, adding watch was the right thing to do.
Having the state as "undefined" in the beginning adds an extra "ghost" option (that appears in the html), that disappears when the state is set to an actual value.
This is the reason the watch actually made a difference, because the innerHTML is actually different.
I might add a note about it in the documentation.

Or instead of mentioning it, I could just fix it.
If you include the above commit, you should no longer need to add the watch attribute.


Improve build pipeline

So far I've also managed to eliminate these dependencies:
"@google-cloud/translate": "^1.1.0",
"@google-cloud/vision": "^0.12.0",
"agentkeepalive": "~2.0.3",
"babel-preset-babili": "0.0.9",
"babel-preset-latest": "^6.16.0",
"bower": "^1.8.4",
"jscs": "^2.10.1",
"jshint": "^2.9.1",
"limiter": "^1.1.3",
"request": "^2.85.0",
"yargs": "^3.32.0"

I'm worried about some of the eliminations of packages - how have you tested that it was safe to remove them?
Afaict some are used:

@google-cloud/* and limiter in https://github.com/CopenhagenCityArchives/kbh-billeder/blob/master/webapplication/lib/services/google-apis.js
agentkeepalive in https://github.com/CopenhagenCityArchives/kbh-billeder/blob/master/webapplication/lib/services/elasticsearch.js and https://github.com/CopenhagenCityArchives/kbh-billeder/blob/master/webapplication/controllers/image__proxy.js
babel-preset-babili is commented out in gulp-js.js for beta and production builds, so probably ok to remove
babel-preset-latest is used in gulp-js.js for beta and production builds (not development) for support of older browsers
bower was only used through gulp-bower, so the bower lib is probably fine to remove
request is used in a bunch of places https://github.com/CopenhagenCityArchives/kbh-billeder/search?q=require+request
yargs is used in https://github.com/CopenhagenCityArchives/kbh-billeder/blob/081c5abf35bdcc6433f00343fb66fb9559c01582/webapplication/indexing/initialize/mode.js to read command line arguments, e.g. when running npm run index all it sets the mode to "all".

Some of these files may be called or required in late, and the requires of the system are still potentially somewhat unpredictable, so I would basically urge caution when removing libs :-)

Yup - I noticed many of the removed dependencies were required in multiple places. There are two case:

Dependency is installed through another dependency. That is true for request. The question is whether we are fine with having such peer dependencies, or whether we should be explicit?
Dead code.

I figured that since we are doing a full test after the restructuring, we will catch the unpredictability.

I would definitely prefer to have all directly used dependencies in package.json so we avoid unpredictability of removing some unrelated package and breaking things. Especially seeing as it doesn't increase the size of our dependencies (as it is already included) - all removing the line in package.json gives us is a false sense of accomplishment imo :smile:
Dead code is real, but also we are unlike to hit all edge cases of the system... so I would still prefer to be cautious for now. That's my gut feel about it. Otherwise we should do a full cleanup and try to remove all dead code, which would be good eventually, but is a big task in its own right.


Update plugin-bitbucket-pullrequest-builder.yml

Description
added new developer, see https://github.com/jenkinsci/bitbucket-pullrequest-builder-plugin/commit/77d917dac2d5d52c8756b2e068da93d3af75713d
Permissions pull request checklist
Always
https://github.com/jenkinsci/bitbucket-pullrequest-builder-plugin
When adding new uploaders (this includes newly created permissions files)
@nishio-dens
Unfortunately, I can't log in to the repo.jenkins-ci.org with my jenkins account

Unfortunately, I can't log in to the repo.jenkins-ci.org with my jenkins account

Note that there's a problem right now with the reverse proxy in front of Artifactory; try jenkinsci.artifactoryonline.com instead.

As usual in such cases, we will give existing committers like @nishio-dens some time to reject this request -- or approve (just by posting a comment here), in which case this change will be merged more quickly.

@daniel-beck many thanks for your suggestion! yes, i can log in to the jenkinsci.artifactoryonline.com
This PR can be merged ....

@damovsky Please see my second comment for how we're handling upload permission grants to new committers.

I'm in touch with  @nishio-dens via emails...I'll ask him to approve this. Thanks ;-)

Hello @daniel-beck and @damovsky,
I approve this PR, @damovsky  is a new commiter of bitbucket-pullrequest-builder-plugin.
Thanks


moved source type under sources tab

closes: https://issues.redhat.com/browse/COST-1740

Codecov Report

Merging #2060 (06b584c) into master (e5ab532) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #2060    +/-   ##
========================================
  Coverage   70.68%   70.68%
========================================
  Files         404      404
  Lines        7347     7347
  Branches     1573     1441   -132
========================================
  Hits         5193     5193
- Misses       2128     2130     +2
+ Partials       26       24     -2

Impacted Files
Coverage Δ

src/components/charts/costChart/costChart.tsx
9.37% <0.00%> (ø)

...omponents/charts/dailyCostChart/dailyCostChart.tsx
7.89% <0.00%> (ø)

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 e5ab532...06b584c. Read the comment docs.


Qt interface

Hi -
Are there any objections to looking at a possible Qt interface?

Yes, basically you need to warp the Qt graphical backend to implement the interfaces declared in the file src/graphic/graphic.h (see the demo app in src/platform/cairo/graphic_cairo.cpp), and you can check the demo app src/samples/gtkmm_main.cpp to see how it works.

Thanks very much. I've attempted to translate the graphics interface. I'm now trying to make a demo app to see if it works...

That is awesome! Make me know if you have any problems. 😄

The good news is that I'm getting an almost pixel-perfect copy of the Gtk output using my Qt backend for almost all of the samples. Unfortunately here are my current problems:

I have an issue with the "\Omega" character. This is being displayed as a "«", unlike the other Greek letters. The code being passed in the wstring is 171, as the source implies. I can only imagine that Qt is doing something special with this character. I'll have to investigate in its internals... It's also not working with \bigtriangleup, \bigtriangledown, \rightleftharpoons  (\dashleftarrow and \dashrightarrow don't work in the GTK version). These are also broken

The wstrings passed to drawText have a zero character at the end. I have to strip these off as Qt renders them as a dotted box. Are they there for a reason?

There are lots of deprecation warnings on building saying warning: dynamic exception specifications are deprecated in C++11 (gcc 9.3).

The ASCII test has some non printable characters in the GTK version. What are they testing? "". Qt doesn't render these, but GTK does.

In the example with "\begin{align}" it looks a bit strange in the GTK version:

This looks quite different to the Qt version:

(note also the \Omega problem here). Is the GTK version correct? This section also seems to move right when I click the "rendering" button a second time (maybe something isn't initialised), and the page is very wide.

Do you mind you create a PR include your Qt implementation to this repository? I'm trying to test these commands and to fix these issues.

The PR was merged. The \Omega command is likely related to the TTF font file that cannot be rendered correctly on Qt graphical backend. Actually, I have met this problem on Android, modify the cmaptable (character to glyph map) finally solved this problem, but that is not a long-term solution.
I'm planning to transfer from plain ASCII TeX to Unicode-math font that can use these much modern OTF fonts (see issue #21), so this issue could be solved.


docs/upgrading-openshift: document password_hash field in 4.13

It's currently a step in the spec stabilization checklist.  However, #289 tried adding commented-out entries for the upcoming stabilization, and that might be worth trying in the future, for the same reason that we've switched to maintaining release notes as we go.


Fix regression in Helm diff

Fixes #68
extracted from #88

Please rename PR (should include what this PR is doing, "Show diff for non-existent releases" or sth along those lines)


Port from master to RB-2.1 - Remove Python wheel workflow running on every pull request (#1503)

Signed-off-by: Rémi Achard<EMAIL_ADDRESS>Co-authored-by: Patrick Hodoul<EMAIL_ADDRESS>

No this is a cherry-pick from master, did you expect something would change ?


after entering to TH state on QM after the vision, player body have sufficient forse

Not a big deal, it's resetting after entering the ship body

not causing any problems


[ICON REQUEST]

Describe the icon you'd like to be added
Dashboard icon
Sample icon
ic_baseline-space-dashboard

Hi @Believer13 this icon has been added to the library.


Allow alpha in shadow color

Fixes #15540

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15541/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15541/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15541/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15541/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15541/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15541/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15541/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15541/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15541/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15541/merge/testResults/webgpuplaywright/index.html


Execute npm run dev doesn't run the server

I found and issue, when I run the command: npm run dev it exit:
C:\passkeys-demo>npm run dev

><EMAIL_ADDRESS>dev
> set NODE_ENV=localhost node server.mjs

C:\passkeys-demo>

This happens because I was running on Windows.
To solve it I need to update the package.json file like that:
"dev": "set NODE_ENV=localhost && node server.mjs",

FYI - in case someone will had this issue.


[oc-id] Upgrade rails from 4.0.0 to 6.0.3

Updated rails version 4.0.0 to 6.0.3

Upgraded oc-id to rails_6.
Fixed specs failure.
Fixed asset paths.

Issues Resolved
#2047
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Running a build at: https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/1430#ea5c37be-73ae-46b1-9ffc-00a6452697cd

Closing this PR. I'll create new PR from new branch.


Update implicit_differentiation.qmd and related_rates.qmd

implicit_differentiation.qmd: sorry, I made a mistake yesterday and delete a -3 at wrong place. Now I fixed it.
related_rates.qmd: some typos and I wonder if the last Example is unfinished?

This one lost a lot of text somewhere in the editing cycle. I had more questions and maybe more examples. Let me go back and find it. Thanks for pointing this out! I'll merge this and try an resurrect the rest.


[FEATURE] Catch exception by type in pyhton

Is your feature request related to a problem? Please describe.
I cannot catch exceptions by type in python, python's generic RuntimeError is thrown instead.
Describe the solution you'd like
I would like to be able to catch exceptions by the types defined in the C++ RDK, like this:
import flexivrdk

robot_ip = '<IP_ADDRESS>
local_ip = '<IP_ADDRESS>

try:
    flexivrdk.robot(robot_ip, local_ip)
except flexivrdk.InitException as e:
    # do some stuff
except flexivrdk.CompatibilityException:
    exit(1)
except flexivrdk.CommException:
    # do some other stuff

Describe alternatives you've considered
I thought about parsing the exception's message string, but that required access to the source code to make sure all cases are covered, but I assume that is not possible
Additional context
I am using python 3.10 on both x86_64 as well as arm64 machines
I installed the corresponding .so libraries in /usr/local/lib/python3.10/site-packages/ on both machines

@skyrimax You can do this in RDK v1.4, which will be released by the end of this week. The mapping between RDK exceptions and Python exceptions can be found here.

@skyrimax Yes, you will have to upgrade to Flexiv Software Package v3.6. You can contact your sales rep and request the upgrade.

Ok, will do, thank you

Hello,
Sorry to reopen this thread. We have update our Rizon 10 to use RDK 1.4 and I noticed that most standard C++ exceptions thrown by the RDK collapse into RuntimeError, so I still have to distinguish them by string comparisson. I was wondering if that was taken into account.

@skyrimax Could you provide an example where programmatically accessing the thrown exception is a must-have?


New algorithm: EK1 with additional updates on second derivative information

Updates the state with information not only on the first, but also on the second derivative! Can be (asymptotically) as cheap as the EK1 when jac_quality=1 or jac_quality=2. But right now, only the more expensive jac_quality=3 seems really reliable - but it requires an additional jacobian.

Codecov Report

Merging #80 (22cd48c) into main (2491ff7) will decrease coverage by 0.07%.
The diff coverage is 87.93%.

@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   87.68%   87.61%   -0.08%
==========================================
  Files          25       25
  Lines         942      993      +51
==========================================
+ Hits          826      870      +44
- Misses        116      123       +7

Impacted Files
Coverage Δ

src/ProbNumDiffEq.jl
66.66% <ø> (ø)

src/perform_step.jl
91.79% <85.41%> (-2.33%)
:arrow_down:

src/algorithms.jl
100.00% <100.00%> (ø)

src/caches.jl
100.00% <100.00%> (ø)

src/initialization/taylormode.jl
100.00% <100.00%> (ø)

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 2491ff7...22cd48c. Read the comment docs.


Master

I suggest two independent patches:
First one allows building on a platform without gSOAP as long as we don't try to build the server.
Second one prevents a crash when an (end-user) certificate has an empty subject DN. Even though empty subject DNs are not created by normal CAs, VOMS should protect itself against it.
It now fails (but doesn't crash) with an error "Cannot discover CA name or DN from user's certificate".

Hi Mischa,
thanks for the PR. I'll check them out.

Reopened against branch develop-2.1.X, closing this one.


Unable to edit meta tags on CMS pages

When I create a new CMS page, I am able to configure the SEO Storm attributes 1 or 2 times. After that, the attributes no longer get saved.
I change some attributes and click "save", but the attributes do not get saved, and after refreshing the page, the attributes are back to their old values.
I am using October CMS v2.1.15 and Initbiz.SeoStorm v3.0.0 from the Marketplace.

Hey @tomaszstrojny
I'm seeing similar issues. I can't add any seo meta to CMS pages, unless it's a newly created page. Any page that has been closed in the Editor will no longer take seo meta via the inspector.
In the file system if a CMS page has not had any seo meta added before closing the page in the Editor, no seo_options attributes are able to be added to the page.
I haven't tried it yet, but I assume I will be able to add all required seo_options attributes via my text editor.
I am also unable to render any seo meta for my custom models, but I'm guessing this is due to the CMS page not storing seo_options attributes.
Hope this helps, cheers

Hi @tomaszstrojny
See js errors when I open SEO Storm inspector on a CMS page
bundle-min.js?v=c3461273:1 Error: Inspector data schema error. /0/description should be string     at Sub.created (3805e8b8b5c8b00c76f51348ed013692-1634483568?vc3461273:7729)     at invokeWithErrorHandling (bundle-min.js?v=c3461273:1)     at callHook (bundle-min.js?v=c3461273:1)     at Sub.Vue._init (bundle-min.js?v=c3461273:1)     at new Sub (bundle-min.js?v=c3461273:1)     at createComponentInstanceForVnode (bundle-min.js?v=c3461273:1)     at init (bundle-min.js?v=c3461273:1)     at createComponent (bundle-min.js?v=c3461273:1)     at createElm (bundle-min.js?v=c3461273:1)     at createChildren (bundle-min.js?v=c3461273:1)     at createElm (bundle-min.js?v=c3461273:1)     at createChildren (bundle-min.js?v=c3461273:1)     at createElm (bundle-min.js?v=c3461273:1)     at createChildren (bundle-min.js?v=c3461273:1)     at createElm (bundle-min.js?v=c3461273:1)     at createChildren (bundle-min.js?v=c3461273:1)
Cheers

I have this same issue, V.3.0.1 , the fields of the seo storm component on the pages are not saving my changes. when I refresh the fields are empty.


PIZ11-fix-route-to-report-bug

Ticket link:
PIZ11
Description:
When I’m viewing the order details I’m unable to go to the Report section, the report option is not visible.
Requirements
Display the report option when viewing an order detail
Screenshot/video:

Before:

After:

:100:


RuntimeError: The size of tensor a (1024) must match the size of tensor b (2048) at non-singleton dimension 3

What should I make sure for the masking to happen,
energy = energy.masked_fill(mask == 0, -1e10)
RuntimeError: The size of tensor a (1024) must match the size of tensor b (2048) at non-singleton dimension 3
These are the below parameters,
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
INPUT_DIM = 1024
OUTPUT_DIM = 1024
HID_DIM = 256
ENC_LAYERS = 3
DEC_LAYERS = 3
ENC_HEADS = 8
DEC_HEADS = 8
ENC_PF_DIM = 512
DEC_PF_DIM = 512
ENC_DROPOUT = 0.1
DEC_DROPOUT = 0.1
SRC_PAD_IDX = 0
TRG_PAD_IDX = 0

shapes of Energy and mask..

        print("**********************************")
        print(mask.shape)
        print("**********************************")
        print(energy.shape)
        **********************************
        torch.Size([2, 1, 1, 1024])     # mask.shape
        **********************************
       torch.Size([1, 8, 2048, 2048]) # energy.shape

original code works.


Check PMD ruleset after qa-eclipse-plugin update

After qa-eclipse-plugin's PMD version was updated, check if the PMD ruleset actually works.

I created a PR for PMD 6 support in the qa-eclipse-plugin. The ruleset works for PMD 6.
Close this issue, after the PR is eventually merged.

Done.


Fix Travis builds

Trying to find the cause for the failing Travis builds

Abandoned. Fixed in #520.


It skips binary files from the cache

And node_modules has a lot in ./node_modules/.bin/ which it neatly skips unfortunate, because it is not a regular file.

Is it a symlink? It's not possible to bundle symlinks into a tarball.

@sethvargo I solved it in #10 😄

See https://github.com/sethvargo/gcs-cacher/pull/10#issuecomment-1585173079


[BUG] Expected property name or '}' in JSON at position 1

Describe the bug
When running a chatflow using ollamaembeddings and Redis VectorStore i am receiving this error :
SyntaxError: Expected property name or '}' in JSON at position 1
at JSON.parse ()
at similaritySearchVectorWithScore (~\Flowise\packages\components\dist\nodes\vectorstores\Redis\Redis.js:276:44)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async vectorStore.similaritySearchVectorWithScore (~\Flowise\packages\components\dist\nodes\vectorstores\Redis\Redis.js:214:20)
at async RedisVectorStore.similaritySearch (~\Flowise\node_modules.pnpm@langchain+core@0.2.14_langchain@0.2.3_@aws-sdk+client-s3@3.529.1_@aws-sdk+credential-provide_iy4pkl2ze2vpuxlmnivkyhhz5q\node_modules@langchain\core\dist\vectorstores.cjs:108:25)
To Reproduce
Build a chatflow with "Recursive Character Text Splitter", Read from a folder, pass the document to a Redis vector store
Use OllamaEmbeddings and use ChatOllama and Convesational Retrieval QA Chain
Expected behavior
The pexected behavior is after upserting the files, when communicating with the Chatflow i want to receive results.

does it works with other embeddings? or does ollama embedding work with other vector store

@HenryHengZJ actually when i changed the "top k" value of the Redis retriever from 1000 to 100, it didn't fail.
Noting that the same "top k" value is used with another embedded data, and it worked.
Could it be the amount of embedded data was not enough?

I am guessing there might be some sort of rate limit from Redis? or from the embeddings side


Remove Hashable to use Into

Reference

Should be fairly straightforward given that
impl<T, U> Into<U> for T where U: From<T>

and
impl<'a, T: Clone> From<Vec<T>> for Cow<'a, [T]>

Since there are no appropriate From instance for plain u8, I changed the tests to work with values of type Vec<u8> (which is what merkle_signing will use anyway). It works out of the box for Vec<u8> and String.

Now that I think about it, I should have checked what happens with slices (and thus kept that commit in its own branch). Anyway, feel free to re-open if needed.

LGTM 👍

Just rebased proof on top of that.

Cool, just don't miss my comment
https://github.com/SpinResearch/merkle.rs/pull/1#pullrequestreview-8638872

Was just reading it :)


Advanced paste doesn't work with simple example

Microsoft PowerToys version
0.81.0
Installation method
WinGet
Running as admin
None
Area(s) with issue?
Advanced Paste
Steps to reproduce

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

For me the whole generated json makes no sense. It should be an object not an array with arrays.

It seems to be 100% important to have correct formatted XML code with correct tag endings.
Log_2024-05-26.txt


Issue to import this project in MounRiver Studio v1.80

Hi,
https://stm32world.com/wiki/CH32_Oscillator
Today When I try your project (I have now osciloscope hanmatec dos1102) I have issue in the start
How can I solve

https://github.com/RT-Thread/rt-thread/issues/6463

https://github.com/RT-Thread/rt-thread/issues/6463#issuecomment-1270238827


If any let method is invoked from before(:all), all let values are shared across all examples

If you invoke any let-defined method from a before(:all) block, all let values will be shared across all examples. A sample. This case is not specified one way or the other in the documentation, but the observed behavior is a bit surprising. I think a less surprising behavior would be for the lets to be memoized separately for before(:all).
This behavior occurs in all RSpec 2.x versions (I tested all the way back to 2.0.1 and up to 2.8.0.rc1), but I can find no mention of it in the mailing list or via google, so I guess that it doesn't come up too often.

Agreed the current behavior is confusing, and that referencing one let-defined method should not impact the others. There are two ways to go with this: memoize lets in before(:all) separately, as you suggest, or decouple them entirely. i.e. let always memoizes values per example regardless of whether they are referenced in before(:all). Personally, the latter feels cleaner and less confusing to me, both as a user and a maintainer of the underlying code. WDTY?

I agree, David.  If people want a helper method that memoizes a value in before(:all) they can always do:
def my_helper
  @my_helper ||= calculate_my_helper
end

before(:all) do
  my_helper # force it to initialize
end

I agree with @dchelimsky
@rsutphin - fantastic bug report (the gist), btw

Not sure about the internals as I just started looking at the rspec-core  code. Is this still an open issue that needs to be fixed by ensuring that let always memoizes values per example?

Personally, I think we should get rid of before(:all). It provides much more confusion and complexity than value.

I've used before(:all) before to perform expensive setup with integration tests (in the case I'm thinking of it was testing stuff acting on a dynamically defined db schema, which only needed to be done before all the tests as it was performed by independent code) and whilst this was only to support legacy code that I didn't write.... it did help massively...
I doubt I'm alone in that, so while from a personal 'design' perspective I'd like to see it go I wouldn't remove it unless it's a significant benefit in terms of maintenance overhead for rspec?


cost based CTE

inline cte only use one time
cost based cte
predict pushdown in cte is handled now

still need to perfect:

FROM can handle alias tabel name, so I don't  emit eliminate FROM under CTEProducer and CTESelect.
CTEAnchor Node and CTESelect Node  do nothing and they should be eliminated too.
subquery in CTE is not well handled due to some thing in Bind. so q58 and q83 in tpcds can't run well.

1. InitCteInfo

every SQLstatment has a member CteInfo, and each has a CteInfoEntries. For convenience, all of the members are declared as PUBLIC.
1.1 See cte as "select * from cteQuery", so it's a SelectStmt. and we can bind and create plan for it.
when meet with cte as (cteQuery),  create a plan for it , see CteSelectStmt 's constructor.
    public class CteSelectStmt : SelectStmt
    {
        private CteExpr originCte_;
        public int CteId() => originCte_.cteId_;
        // we have to constract a CteSelection
        // select * from cte
        public CteSelectStmt(CteExpr originCte, CTEQueryRef from, List<CteExpr> previousCtes) : base(
                    new List<Expr>() { new SelStar(originCte.cteName_) },
                    new List<TableRef>() { from },
                    null, null, null,
                    previousCtes, null, null, null, $"select * from {originCte.cteName_}")
        {
            originCte_ = originCte;
        }
    }

1.2 add previous cte to current cte's context
in InitCteInfo, bind and create plan for each CTE, and add previous CTEs into current cte’s context and ctes_, so if current cte refer to previous ctes, it can find them.
List<CteExpr> previousCtes = cteInfo_.GetAllCteExprs();

// prevoious ctes have smaller cteId
Debug.Assert(previousCtes.All(x => x.cteId_ < curCte.cteId_));

SelectStmt cieStmt = new CteSelectStmt(curCte, from, previousCtes);

1.3 add cte to subquery's context so subqueries can find ctes
2. how to delete unused CTE
2.1 Example
It seems easy to do so, if a cte is not referred, it should be delete, but the paper put forward a extreme stupid situation. see:
sql1:
"with cte0 as (select * from a),cte1 as (select * from cte0) select * from b where b.b1 = 2";

cte0 has been referred by cte1, but cte1 is never used. here we use a stack to remove cte0.
TU.ExecuteSQL(sql, "2,3,4,5", out phyplan, option);
var answer = @"PhysicScanTable b (actual rows=1)
                Output: b.b1[0],b.b2[1],b.b3[2],b.b4[3]
                Filter: b.b1[0]=2";
TU.PlanAssertEqual(answer, phyplan);

2.2 Algorithms
Algorithm findUnusedCTE
INPUT: cteStack
OUTPUT: cte will be marked used or not.
step1 	findCteConsumer(rootPlan,&cteStack)
step2
        for subq in subqueres:
            findCteConsumer(subq,&cteStack)
step3   while not cteStack.empty():
            cteid = cteStack.pop()
            findCteConsumer(GetCteInfoEntryByCteId(cteid).plan_,&cteStack)

findCteConsumer will find CteConsumer from LogicNode recursively, push and mark cteInfoEntry as used.
For instance, in sql1, step1 will not push cte1 into cteStack,  step2 will not either. so the cte1 will not be marked used. so the cte0 will not be marked used too.
2.3 Confirm the Algorithm and Explan the example
For sql1, cte1 will not be marked used, so it will not push into the cteStack, so cte1 is not mark used. so they are all deleted.
3. Logic Plan

3.1 LogicCteAnchor
3.1.1 where Cte Anchor show up.
the LogicCteAnchor will be add at the root of a plan, such as (b) .
3.1.2 where is the code
the realize code is bellow in cteToAnchor
            for (int i = 0; i < cteIsUsed.Count; i++)
            {
                var lca = new LogicCteAnchor(cteInfo_.GetCteInfoEntryByCteId(cteIsUsed[i].cteId_));
                lca.children_.Add(root);
                root = lca;
            }
            return root;

3.2 LogicCteProducer
3.1.1 where Cte Producer show up.
the plan produced for select * fom (cteQuey) is the plan for CTEProducer.
the LogicCteProdicer will be produced through RuleTrans
3.1.2 where is the code
the child of the LogicCteProducer is the select * from cteQuery , see section 1.1.
        public LogicCteProducer(CteInfoEntry cteInfoEntry)
        {
            cteInfoEntry_ = cteInfoEntry;
            children_.Add(cteInfoEntry.plan_);
            // CTEProducer has only one child
            Debug.Assert(children_.Count() == 1);
        }

3.3 LogicCteConsumer
cteQueryRef will be transform as LogicCteConsumer.
3.4 Logic Sequence
3.4.1 where it show up
RuleTrans CteAnchor2CteProd will  produce a LogicSequenceNode, and it has the same MemoSignature  with CTEAnchor for it.
4. Cost-Based Optimized CTE
4.1 how to produce a memo as this figure below

the LogicCteProducer will be add through RulesTrans. when we meet a LogicCTEAnchor, it will producer a LogicSequence  Node, and the LogicSequence will add CTEProducer Node as its child.
when meet CTEConsumer, it will producer LogicCTESelect and has the same Soginature with CTEConsumer. it will also create nodes from cteplan. To avoid Group7 produce  the same Signature  with Group6, the Signature have to consider  if it is  belong to a child of cte consumer,  cteId and cteRefId.

group
is child of cte consumer
cteid
cteRefId
Signature

Group6
false
1
-

Group7
true
1
1

Group8
true
1
2

by this , cteid = 2 and cteRefid = 1 will has the same signature with cteid = 1 and cteRefid=2 will has the same Signature. To avoid this ,  see cteRefId as string.

group
is child of cte consumer
cteid
cteRefId
Signature

Group6
false
1
-

Group7
true
1
ref1

Group8
true
1
ref2

4.2  select plan

Figure 4.1
4.2.1 how to avoid  plan(a)
A LogicCteSequence  which is a binary operator will bind to a cte, so it has its cteid  and will generate requirement (p 0) for left child and (c 0) for right child. And a LogicCteAnchor will generate no requirement for its child.
requirement for CTE  will always pass to children and never disappear.  The Group Member of a Group will satisfied the requirement only when it's cteSpec is the sub set of the CteReq
a LogicCteAnchor will generate no requirement for its child, when meet consumer0, consumer0 will seen as [(c 0)], it is not the subset of []. So the plan will not pass.

4.2.2  c and d are valid
In fact, c and d should be avoid. But now we don't support transform property from left to right child, they are not avoid.
But they will be optimized out by cost.
cost-based select will not choose plan like (c) and (d)


exclude napari-tomoslice and napari-subboxer

Add 2 plugins to exclusion list, at the request of the plugin owner:

[ ] napari-tomoslice
[ ] napari-subboxer

@neuromusic Just to clarify, what visibility value will be set to napari-tomoslice and napari-subboxer respectively? Right now, there are three distinct values in excluded_plugins.json: hidden, disabled, invalid

hidden: they won't show up in a search on the main page but will still have a plugin page
disabled: won't show up completely
invalid: the plugin data cannot be processed and is therefore excluded by the hub

thx @potating-potato
@klai95: we want disabled here

hey @klai95, it looks like these have been disabled... can you confirm and close out this issue?

hey @klai95, it looks like these have been disabled... can you confirm and close out this issue?

@neuromusic Where are you seeing that napari-tomoslice and napari-subboxer have been disabled? excluded_plugins.json on S3 currently do not have these plugins

@klai95 they aren't on the hub anymore.
perhaps @alisterburt got tired of waiting and pulled them down by another means :D

@klai95 they aren't on the hub anymore.
perhaps @alisterburt got tired of waiting and pulled them down by another means :D

@neuromusic From my conversation with @DragaDoncila, it seems like the process should be done by the plugins themselves. Since it seems like the two plugins aren't on the hub anymore, can I go ahead and close the issue?


add an action to export the server cert and ca cert

Enhancement Proposal
when communicating with traefik from uncharmed applications it can be handy to obtain the certificates to enable TLS.
ATM the only way to do that for a cloud admin is to obtain the cert is via manually running openssl commands. This would be nicer API:
juju run traefik/0 get-certificates
similar actions would have to be implemented in the grafana agent charms

I would not say it is a common pattern per se, but it would not be uncommon either. In instances where the certificate is not issued by a public authority, there is a need to configure the clients with the issuing certificate authority.
In manual operations, this would either be by getting the certificates configured on the gateway and installing them on the client's trust store, or get the issuing certificates from the central private authority and installing them in the trust store.
The most secure way would be using an out of band mechanism; meaning that the client gets the certificates from the certificate authority itself, and the gateway gets its certificate signed by that same authority.
Regarding this proposal more specifically, I think it can make sense. It let's a user do something they could already do in a nicer way.
I would suggest that in most cases, our documentation pushes to have the client obtain the issuing CA from the signing authority itself.

Since it is not considered a best practice, we should be only getting the CA from the provider not the gateway. Closing the issue.


Add RestoreFromBackup command to vtctl

@enisoc @michael-berlin
Executing the command vttablet will completely delete the existing data and
then replace it from the latest backup. The sequence of actions is basically
the same as vttablet has already used to setup a freshly started replica.
Also mysqlctl interface is extended with ReinitConfig function. It's used by
vttablet to change server_id of the replica before restoring it from backup.
That's necessary to avoid a possible situation when restored replica skips
transactions in the replication stream which have the same server_id.
The new RestoreFromBackup command will be used later in the implementation of
an automated pivot.
This change doesn't add any tests for the new functionality because I didn't
find a place where I could add these tests. Please tell me if there is actually
a place that I missed. Meanwhile I tested this manually on the example
Kubernetes setup.

This change is

The right place to test it would probably be in test/backup.py, where we already test the restore from scratch of a tablet.

As Alain said, you should add a test in backup.py. One thing that's awkward is that the desired before and after states for RestoreFromBackup are the same, so it may be challenging to design a test that will actually fail if you simply comment out the RestoreFromBackup command.

Reviewed 38 of 38 files at r1.
Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

go/vt/tabletmanager/action_agent.go, line 261 [r1] (raw file):

			// restoreFromBackup wil just be a regular action
			// (same as if it was triggered remotely)
			agent.actionMutex.Lock()

We like to avoid this pattern of manual locking like this: MakeRestoreData() into a wrapper that acquires the lock and calls restoreDataLocked() which does the work. Then in places that already hold the lock, they can call restoreDataLocked() directly.

go/vt/tabletmanager/restore.go, line 90 [r1] (raw file):

	// let's update our internal state (start query service and other things)
	if err := agent.refreshTablet(ctx, "after backup"); err != nil {

"after restore from backup"

go/vt/tabletmanager/rpc_backup.go, line 74 [r1] (raw file):

// Should be called under RPCWrapLockAction.
func (agent *ActionAgent) RestoreFromBackup(ctx context.Context, logger logutil.Logger) error {
	tablet, err := agent.TopoServer.GetTablet(ctx, agent.TabletAlias)

Is there a reason you prefer to re-fetch from topo, rather than use agent.Tablet()? If so, please add a comment explaining what scenario the re-fetch guards against.

go/vt/tabletmanager/rpc_backup.go, line 86 [r1] (raw file):

	// now we can run restore
	err = agent.RestoreData(ctx, l, true /* deleteBeforeRestore */)

Here's an example where you can call restoreDataLocked() since you're assumed to be wrapped by RPCWrapLockAction.

go/vt/tabletserver/testutils_test.go, line 91 [r1] (raw file):

		"",
		"",
		mysqlctl.NewMycnf(11111, 6802),

It used to be that some places relied on the MySQL ServerID being the same as the Tablet UID. I fixed those and set two different values here as a regression test. Can you add that back by calling NewMycnf() as a separate line, and then setting cnf.ServerID = 22222?

go/vt/vtctl/backup.go, line 31 [r1] (raw file):

		"Removes a backup for the BackupStorage."})

	addCommand("Tablets", command{

Please run the doc/vtctl_go_reference.py script to regenerate docs.

Comments from Reviewable

Thanks! I've added the test now. PTAL.

Review status: 25 of 40 files reviewed at latest revision, 6 unresolved discussions.

go/vt/tabletmanager/action_agent.go, line 261 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> We like to avoid this pattern of manual locking like this: Make`RestoreData()` into a wrapper that acquires the lock and calls `restoreDataLocked()` which does the work. Then in places that already hold the lock, they can call `restoreDataLocked()` directly.

Done.

go/vt/tabletmanager/restore.go, line 90 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> `"after restore from backup"`

Done.

go/vt/tabletmanager/rpc_backup.go, line 74 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Is there a reason you prefer to re-fetch from topo, rather than use `agent.Tablet()`? If so, please add a comment explaining what scenario the re-fetch guards against.

No reason. This was just copied from the Backup() function above. Should I change this or keep consistent?

go/vt/tabletmanager/rpc_backup.go, line 86 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Here's an example where you can call `restoreDataLocked()` since you're assumed to be wrapped by `RPCWrapLockAction`.

Done.

go/vt/tabletserver/testutils_test.go, line 91 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> It used to be that some places relied on the MySQL ServerID being the same as the Tablet UID. I fixed those and set two different values here as a regression test. Can you add that back by calling `NewMycnf()` as a separate line, and then setting `cnf.ServerID = 22222`?

Done.

go/vt/vtctl/backup.go, line 31 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Please run the `doc/vtctl_go_reference.py` script to regenerate docs.

Done.

Comments from Reviewable

Reviewed 14 of 15 files at r2, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

go/vt/tabletmanager/rpc_backup.go, line 74 [r1] (raw file):
Previously, pivanof (Pavel Ivanov) wrote…
> No reason. This was just copied from the Backup() function above. Should I change this or keep consistent?

I see. If it's copied from Backup, let's just keep it consistent for now.

test/backup.py, line 288 [r3] (raw file):

    self._restore_old_master_test(_restore_using_kill)

  def test_in_place_restore_vtctl(self):

I don't think we need to test individual commands through vtctl. We expect it to be the same result as through vtctlclient->vtctld. If there is potential for vtctl to differ from vtctlclient->vtctld, those cases should be targeted in something like vtctld_test.py.

Comments from Reviewable

Review status: 39 of 40 files reviewed at latest revision, 1 unresolved discussion.

test/backup.py, line 288 [r3] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> I don't think we need to test individual commands through vtctl. We expect it to be the same result as through vtctlclient->vtctld. If there is potential for vtctl to differ from vtctlclient->vtctld, those cases should be targeted in something like vtctld_test.py.

I removed the extra test.

Comments from Reviewable


Added support for ida9.0 beta

IDA 8.x does not support get_type_by_tid, and IDA 9.0 does not support ida_struct. I don't know of a way to support both simultaneously.

File : core.py line 185, the api changed cause ida can not compare arm to "ARM".
latest 9.0, the arm be changed to "ARM". this patch fix arm64 machine ASM not work.
        arch_name = ida_ida.inf_get_procname()
        print(f"当前加载的CPU类型为: {arch_name}")

        if arch_name == "metapc":
            assembler = AsmX86()
        elif arch_name.startswith("arm") or arch_name.startswith("ARM"):
            assembler = AsmARM()```

File : core.py line 185, the api changed cause ida can not compare arm to "ARM".
latest 9.0, the arm be changed to "ARM". this patch fix arm64 machine ASM not work.
        arch_name = ida_ida.inf_get_procname()
        print(f"当前加载的CPU类型为: {arch_name}")

        if arch_name == "metapc":
            assembler = AsmX86()
        elif arch_name.startswith("arm") or arch_name.startswith("ARM"):
            assembler = AsmARM()

Thank you, I didn't notice that.

Traceback (most recent call last):
File "C:\Program Files/IDA Professional 9.0/plugins\patching\core.py", line 128, in load
self._init_assembler()
File "C:\Program Files/IDA Professional 9.0/plugins\patching\core.py", line 175, in _init_assembler
assembler = AsmARM()
TypeError: AsmARM.init() missing 1 required positional argument: 'inf'

change line 665 in asm.py to if ida_ida.inf_is_be():

 change line 665 in asm.py to if ida_ida.inf_is_be():

Thank you for reminding me. In these two modifications, I replaced ida_struct and ida_typeinf with idc to ensure compatibility from version 7.6 to 9.0 beta.

Thanks!


Rare chance of a nether portal at the end of a cave

Adding, Removing, or Changing
Adding
Type of Modification
Other
What are you trying to modify
Add a nether cave at the end of some caves
Minecraft Version
1.17
Iris Version
1.8.2
Checklist

[X] I am using an unmodified version of the overworld. (If you modified the overworld but know the content suggestion is valid)
[X] I am using the latest version of the overworld OR I have made sure this feature was not already implemented in the latest version.

need to implement the "cap" system in the the jar


config with AUTH_REQUIRED = False

I'm having issues trying to get alerta-webui working against an alerta-api that has no authorization configured.
The error console in the browser shows 404s trying to get to urls like:
https://alerta-webui.XX.XXXX.com/undefined/environments?status=open
Note the 'undefined'.  That's where it seems things are setup wrong.
I'm obviously doing something wrong here.  I've tried a config.js with just an endpoint defined, nothing else.  I've also tried with 'provider' as basic, but I think that's wrong.

Interestingly, it appears that my modification of config.js wasn't being picked up.
If I hard code my config into /web.js, then it starts doing the proper thing.

Are you trying to deploy Alerta webUI to Heroku? The Heroku Procfile is the only place where web.js is used.

Yes.  It appears that it's not picking up the config.js.

If you're deploying to Heroku you should make use of the Config Vars to set the Alerta API endpoint for the web UI.
Example


Application log monitoring

I have my application generating 10 types of log files and I have it running on 100s of servers. I want to monitor these log files. at the same time, i want to find "ERRORS/EXCEPTIONS etc." which gets printed in these log files and raise alert. Is there a exporter which can handle this?? Infact handle multiple log files!!! I have tries grok_exporter but it can handle only 1 at a time and I do not want to start so many exporters!!!

You can look at https://github.com/google/mtail too.
I'm closing it for now. If you have further questions, please use our user mailing list, which you can also search.


how to update java

Description

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):

gotta be in error


ExtensionSnapshotFailedCOM error is failing my VM back up everyday.

Hi
We have a production server running in Windows 2016. This is a legacy application.
I scheduled a VM back every day at 7 AM up using Azure Backup. The backup was successful for the first two days. It started failing from the last 2 days.
Below are the error codes and messages from the console.
Error Code | ExtensionSnapshotFailedCOM
Error Message | Snapshot operation failed due to COM+ error
Recommended Action | Please restart windows service "COM+ System Application" (COMSysApp). If the issue persists,, restart the VM.
I have followed the documentation and restarted COM+ System Application, and restarted the VM. I was able to do a backup buy manually yesterday. Today morning I have the same error again.
Any suggestions or help will be appreciated.

@briojoel Thanks for the comment. Follow the suggestions from the troubleshooting guide and see if that helps resolve this issue. If not, you may reach out to support by creating a technical support request for further investigation into this issue.

@briojoel Closing this issue for now. Please do not hesitate to reply if you've any concerns in this matter and we shall continue the conversation.


Update version: zxch3n.PomodoroLogger version 0.6.3

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

@vedantmgoyal2009 - Looks like your manifest keys aren't sorted like they used to be. Also, this PR doesn't make any changes to the actual content of the manifest other than re-ordering the keys


add 'level_names' for 'GELFRabbitHandler'

Another attempt to fix the missing 'level_names' argument. Without that field in the message payload, there is no way to display ro filter by 'level_name' field in Graylog.
The test script is also updated.

Codecov Report

Merging #88 into master will increase coverage by 7.62%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   51.48%   59.11%   +7.62%
==========================================
  Files           3        3
  Lines         202      203       +1
==========================================
+ Hits          104      120      +16
+ Misses         98       83      -15

Impacted Files
Coverage Δ

graypy/rabbitmq.py
35.38% <0%> (+27.57%)
:arrow_up:

graypy/__init__.py
71.42% <0%> (-28.58%)
: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 436c0f7...8bf0b54. Read the comment docs.


emacsPackages.idris2-mode: updateScript

Description of changes
https://github.com/NixOS/nixpkgs/issues/278925
https://github.com/NixOS/nixpkgs/pull/329005

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@wuyoli

I suspect wuyoli is inactive...

@jian-lin I added another commit transferring the maintenance of this package to me, since @wuyoli does not seem active.


Migrate Access API docs

We need to render this page: https://docs.onflow.org/http-api/ - it uses redoc. Is this possible? How is this going to look?

relevant links:

https://raw.githubusercontent.com/onflow/flow/master/docs/content/http-api.mdx
https://github.com/onflow/flow/blob/master/docs/plugins/gatsby-theme-flow/src/components/http-api.js