Close matching paren enterd in search input

Checklist

[X] Have you tried updating the plugin to the latest version?
[X] Have you checked the Breaking Changes issue?
[X] Have you read through :h nvim-surround to see if there might be any relevant information there?

Neovim Version
: nvim -v
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/local/Cellar/neovim/0.9.1/share/nvim"
Run :checkhealth for more info
Plugin Version
Tagged (Stable)
Minimal Configuration
require("nvim-surround").setup()
Sample Buffer
Empty search input buffer
Keystroke Sequence
/Foo(
Expected behavior
/Foo(
Actual behavior
/Foo()
Additional context
To search for a method I need to enter /Foo( to search for unmatched string

Wrong plugin, you probably have some autopairs-style plugin installed that's autocompleting the end brace.


Is the control group placed first when conducting DEG?

Thank you very much for the impressive package your team has created.
When conducting differential analysis, where should the control group and experimental group be placed?

Hi
The order of control and case data doesn’t matter.
Best
Xin
发自我的 iPhone
在 2024年12月2日，01:55，Fcwttobeonedoctor @.***> 写道：

Thank you very much for the impressive package your team has created.
When conducting differential analysis, where should the control group and experimental group be placed?
—
Reply to this email directly, view it on GitHubhttps://github.com/XZouProjects/scCODE/issues/9, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEGRBRVWBCVUJ7K6ZBRUNXD2DNEQNAVCNFSM6AAAAABSZ5IGUWVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YDSNJZGI2DSNY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

你好 控制和病例数据的顺序无关紧要。 Best Xin 发自我的 iPhone 在 2024年12月2日，01:55，Fcwttobeonedoctor @.> 写道：  非常感谢您的团队创建的令人印象深刻的软件包。 在进行差异分析时，对照组和实验组应该放在哪里？ — 直接回复此电子邮件，在 GitHub< #9 > 上查看，或取消订阅< https://github.com/notifications/unsubscribe-auth/AEGRBRVWBCVUJ7K6ZBRUNXD2DNEQNAVCNFSM6AAAAABSZ5IGUWVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YDSNJZGI2DSNY >。 您收到此邮件是因为您订阅了此主题。消息 ID： @.>

Thank you for your reply. What I mean is that when I use scCODE(A, B), the differentially expressed genes identified are those that are more highly expressed in B than in A, right?


[AUTO-DEVICE] In process early exit device detection

This PR updates the device auto detection to use in process early exit. This hopefully will address some of the windows issues in multi-process based approach while preserving memory without allocating in each during auto mode.

NOTE this can be viewed as a temp fix, or permant one, if we are ok with early exit. Generally I think early exit behavior is useful to iterating over too many drivers and initializing them. It does mean auto mode will pick the first available and not printing out other information
Background issue https://github.com/mlc-ai/mlc-llm/issues/1323


Single LiDAR-Blockage diagnostics

Problem
The LiDAR - blockage is phenomenon when some unwanted objects cling LiDARs and block the emitted/return light.
Blockage situation as out of ODD which might significantly degrade the performance of LiDAR is necessary to be detected.
The blockage might be cause by wide range of objects in the environments, for instance, leaf, plastic bag, dust, snow, ice ...

Suggestion
Blockage can be detected based on the empty return region of LiDAR point cloud or LiDAR depth map with considering the ground and sky range for each LiDAR.

Blank region in ground-range: high possibility of blockage
Blank region is only in sky-range : low possibility of blockage

This issue has been resolved by #461


Dataset problems

Thank you very much for your excellent work. Now I plan to use MRI to transform CT. Do I need labeled MRI and CT data as input items to run the code? Looking forward to your reply.

Yes, you need the labeled MRI and CT.


fix(template): template static build fixed

the static build required data.json is now in appropriate place, along with upgraded template deps,
updated readme
closes #32, closes #33, closes #34

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #35   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          56     58    +2
  Branches        6      6
=====================================
+ Hits           56     58    +2

Impacted Files
Coverage Δ

generators/app/index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update tutorial-yii-integration.md

I found the existing language to be quite confusing. It took me several attempts of several hours each to get it working on my computer. I have several years of Yii 1 development behind me and have now done four or five Yii 2 apps, so I don't think I'm being dense. I think my language is a lot clearer and more specific.
However, note that I could not get it to work without commenting out a line of vendor/yiisoft/yii2/base/Object.php, which can't be a good thing. Hopefully someone will know how to avoid this problem and will be able to further correct my instructions.

After thinking about this overnight, I would amend my directions to recommend that the user copy all the main Yii folders, e.g. controllers, models, etc. into the @app directory, but I can't figure out how to change my commit or add another commit to it.

Checked changes in detail. Won't accept these since there are recommendations which should not be recommended:

Not using Composer.
Using basic template instead of advanced (actually it depends on a project very much).
Storing 1.1 app under webroot.
Commenting out something in the core framework files in the vendor directory.

Thanks Alexander … here are some further thoughts.
On using Composer … the instructions give you the choice of downloading or using composer. I’m used to using composer on my yii2 projects, but not yii1. Maybe this could be clearer? If you are moving from Yii1 to Yii2, here’s how you would use composer in Yii1?
On the basic template versus advanced … again not that clear in the existing docs.
On storing 1.1 under webroot … that is the way the current documentation proposes it be done, is it not? Protected lives inside the webroot. I was just trying to clarify where everything went.
On commenting out some code in the core framework … agreed, that is a non starter. But I could not get the solution proposed in the existing docs to work any other way. Probably just me, but can you confirm that the documented method in fact works?
I’m now doing all my new development with Yii2, which is fabulous. Increasingly I want to get my old Yii1 sites upgrade to Yii2, but it ain’t easy. I’ve decided for now, I’m going to install Yii1 and Yii2 side by side, both working off the same tables. Then I’ll slowly re-create my Y1 classes in Y2, moving more and more of the functionality from Y1 to Y2. That way I can avoid the difficulties of getting the two integrated and make my transition gradually.
I feel there must be many like me struggling with this problem and the documentation could be stronger in explaining the different strategies that one can employ and exactly how to get them to work.
Finally, I just want to express my thanks to the team that wrote Yii1 and Yii2. Using Yii1, and now Yii2, has enabled me to do things that I just could not have done before. They have made me a much better developer, not to mention a lot of money developing really cool projects for clients. I am truly grateful to you and the team for giving us this amazing product.
alec

On Nov 19, 2015, at 6:03 PM, Alexander Makarov<EMAIL_ADDRESS>wrote:
Checked changes in detail. Won't accept these since there are recommendations which should not be recommended:
Not using Composer.
Using basic template instead of advanced (actually it depends on a project very much).
Storing 1.1 app under webroot.
Commenting out something in the core framework files in the vendor directory.
—
Reply to this email directly or view it on GitHub https://github.com/yiisoft/yii2/pull/10123#issuecomment-158227970.


ci: add semver check in CI

I feel I cannot catch all API changes during my review, and it would be a good idea to use a tool in this case 🤪

Okay we did accidentally break something. We will need to bump to 0.13 instead of 0.12.1.

Okay we did accidentally break something. We will need to bump to 0.13 instead of 0.12.1.

I am sorry that I didn't mention a new function parameter explicitly. I wasn't sure how strict the releases in this library are in terms of api compatibility.
Thank you for taking care of this issue!

No worries! I think we also didn't take this seriously for a long time...


Update virtual-machines-linux-mysql-use-opensuse.md

Correct the time stamp to the format of "MM/DD/Year"

Hi @NingKuang, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


1990 bluetooth MAC addresses limits on Android "Unfortunately, your bluetooth has stopped"

Hello there =),
We would like to develop a new app with Estimote SDK on Android.
It is an app that will be used by visitors of a museum to display their position.
The device is kept by the museum, and will scan everyday a lot of bluetooth devices.
In a previous experience (2 years ago),
We had a lot of problem with Estimote SDK due to the 1990 MAC addresses of bluetooth device limitation.
We have tested on JellyBean, KitKat and Lollipop, but the problem was not solved.
Today, we want to use the new Estimote SDK, but we would like to know if this limitation still exists ?
Do you have more information about a bugfix from Google ?
Anybody has tested on newer SDK (Marshmallow or Nougat) ?
For more informations about this problem, an issue has been sent to Google :
https://code.google.com/p/android/issues/detail?id=67272
Hope there is a solution =)

Hello @VincentChann
As far as I know, this problem is solved :)
You can simply try our SDK with your beacons, and check if that meet your requirements.
If you want to make a quick proof-of-concept I suggest you to use one of our examples
Good luck!

I am not sure you understand our problem.
On JellyBean, KitKat and Lollipop, the problem is not solved.
The question is, Does Marshmallow & Nougat solve the problem ?
Do you have some more informations about this problem ?
(This not an SDK problem but an Android OS problem)

Hello @VincentChann
You may take a look at this issue:
https://code.google.com/p/android/issues/detail?id=191831&can=1&q=bluetooth mac 1990&colspec=ID Status Priority Owner Summary Stars Reporter Opened
It seems it is fixed since this September. Is that what you were asking for?


Use docco for legalese

These pages:

https://gratipay.com/about/policies/terms-and-conditions
https://gratipay.com/about/policies/terms-of-service
https://gratipay.com/about/policies/privacy

Need a docco translation to human language (example).

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10978294-use-docco-for-legalese?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

:+1:
Someone should also update gittip to gratipay for TOS;DR: https://github.com/tosdr/tosback2/blob/master/rules/gittip.com.xml

WePay does a pretty good job with this.


Brainslug

Short description

BRAIN SLUG!!! Yea, i fixed some thing, so.... I need to rewrite more and more.....  I'm going to go cry into my pillow and finish pr while you enjoy that text of this pr)
Why we need to add this

New antagonist, more antagonist more problems, more problems, more rdm, so why yes?
Media (Video/Screenshots)

I add it later...
Checks

[ ] I do not require assistance to complete the PR.
[x] Before posting/requesting review of a PR, I have verified that the changes work.
[ ] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
[x] I affirm that my changes are licensed under the Starlight Fork License and grant permission for use in this repository under its conditions.

Changelog

:cl: Rinary, kxvvv

add: Added brainslug.

Yes, I created this pr to show that I'm not slacking these days, but looking at it and how much more I have to write, I can safely say that so far I've done nothing but a “working” version where only a jump out of all abilities works)


Typst (https://www.nixhub.io/packages/typst) has two different versioning schemes in its release history. While Typst now uses SemVer (e.g. 0.7.0), earlier this year releases were still tagged by date (e.g. 23-03-28).
Nixhub does not seem to handle this well - the older versioning scheme is treated with higher priority and thus the very outdated version 23-03-28 is chosen as the latest version, while it should be the current 0.7.0.
To illustrate:
devbox add typst

will install typst-23-03-28, while it should install typst-0.7.0.

Thanks for reporting. We have a few packages right now that aren't sorting correctly. We're aiming to get a fix out this week to handle inconsistent version strings better.
Related to #1466.

Search results for typst are now sorted correctly in devbox search:
* typst  (0.8.0, 0.7.0, 0.6.0, 0.5.0, 0.4.0, 0.3.0, 0.2.0, 0.1.0, 0.1, 23-03-28)

Installing typst@latest with devbox now installs version 0.8.0


initial backend task

this merge from branch backend to create base code for project

this merge from branch backend to create base code for project


⚠ Secret Website 3 has degraded performance

In ef9cff8, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3177 ms

Resolved: Secret Website 3 performance has improved in 06968d9.


Timeout errors in the github actions

From time to time, some of the notebook samples test fail with messages like panic: test timed out after 10m0s.
2_Read flow notebook sample (old).txt
Read flow notebook sample (old).zip

This issue might be fixed in https://github.com/fybrik/fybrik/commit/e2d12d826ca8c765c82b5e8cdc72175410266304 which contains a fix for multiple fybrikapps to fight race-condition.

A possible reason might be that port-forwarding of the read service is done too early, before the server running in the pod actually opens the port. When FybrikApplication is ready, it does not necessarily mean that the server is up and running. It only means that the pod is up. Inserting a delay between catching a ready status and port-forwarding would probably solve the issue.


Automated cherry pick of #11155

Cherry pick of #11155 on release-7.4.
/cc  @AshishDhama
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 0dffd0c4f1b0939c4e9a2a64df3b2a8a11009e00


Add RocksDB perf metrics for all components

Signed-off-by: qupeng<EMAIL_ADDRESS>What is changed and how it works?
Add RocksDB perf metrics for all components:

[x] storage
[  ] CDC

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
None

/bench sysbench

/merge


Test against PHP 7.2

With the release of PHP 7.2, would be interesting to test it.

Looks like there are some issues with installing mongodb


Fixed header and drawer for width between 850px and 1024px

MDL version: 1.0.1
Browser: Chrome
Browser version: 44
Operating system: Windows
Operating system version: 7
URL : Codepen of the offical doc "Fixed header and drawer" (http://codepen.io/anon/pen/GJwQJa)
What steps will reproduce the problem:

Reduce the width of the browser between 850px and 1024px

What is the expected result?
The drawer is displayed or the hamburger icon is displayed
and the header bar is "full width"
What happens instead of that?
The drawer is not displayed and the header bar is splitted in two parts

This has been fixed in 1.0.2. If you update your codepen to 1.0.2, it works :)

There is still an issue with the 1.0.2 :
When the width of the browser window is below 1024, and a title is present in the header bar, the burger icon is displayed over the title.
The codepen has been updated : http://codepen.io/anon/pen/GJwQJa


COUNT BSTS COMPETITIVE PROGRAMMING PROBLEM IN C++

🚀 Feature
(Given an integer N, find and return the count of unique Binary search trees (BSTs) are possible with nodes valued from 1 to n.)
Have you read the Contributing Guidelines on Pull Requests?
(Yes)
Motivation
(Output count can be very large, so return the count modulo 10^9+7)
Input Format :
Integer n
Output Format :
Count of BSTs
Contraints :
1<= N <=1000

/assign


func model page

I propose that we:

Adopt @rbrush's changes
Add a link to the ViewDefinition notes to refer to this page

And then this should be the last outstanding change for the v2.0.0 milestone!


Scheduler is now an interface/trait

Final PR in the series to migrate the scheduler to a trait.
This will allow more complex functionality and special configs
for when the scheduler should do additional steps or interact
with workers in a unique way.
closes #140

This change is

cas/scheduler/scheduler.rs line 30 at r3 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…
Sadly both ActionScheduler and WorkerScheduler need it.
ActionScheduler:

CapabilitiesServer needs to get all properties to tell clients about supported properties.
ExecutionServer needs to translate the config Minimum, Exact, Priority into the union version Minimum(u64), Priority(String), Priority(String) versions.

WorkerScheduler:

WorkerApiServer needs to also translate the config->union, to inform the scheduler of the properties the instance supports.

My main issue is how do I proxy that through with a GrpcScheduler.  See https://github.com/chrisstaite-menlo/turbo-cache/commit/de97fb4895b6e0e67b38965b873bfa1c3e549bf8


Flux: Suspend Column

It would be awesome to have a view on current Suspended Flux resources

After #109 The suspended resources show up in the list view only


[Feature]: Possibility to deploy Appsmith on Docker with another port than 443

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
It seems that the Docker image of Appsmith is hardcoded with the 443 SSL port, which is not very convenient when you already have something using the port 443. It could be nice to be able to run Appsmith on other ports too.
Why should this be worked on?

You can always modify the ports in the docker-compose.yml file to use other host machine ports that map to 443 within the container. In the below example, on the host machine Appsmith will occupy ports 8080 (HTTP), 8443 (HTTPS) & 9801 for supervisor & control.
    ports:
      - "8080:80"
      - "8443:443"
      - "9801:9001"

Does this configuration help? Are you running into any issues with custom ports being configured in this manner?

Closing this issue as this should be possible

Small note on suggested change in docker-compose.yml file, if you intend to use HTTPS, please keep the 80's port mapping intact. That is, please leave the 80:80 line as is. The reason is because certbot and Let's Encrypt rely on this port mapping to provision the SSL certificate for your domain. If this port was changed, the SSL certificate provisioning would fail.
However, if you don't intend to use HTTPS, you may freely change 80:80 to any other port mapping like 8080:80 to use port 8080 for example.

Could it be possible to disable HTTPS ? I don't really need a let's encrypt certificate, the host is not public, and the tunnels towards my host are already encrypted.

That is currently not possible, but we just merged a PR where if the certificate provisioning fails, for whatever reason, the server would proceed to start without HTTPS, with just HTTP. The PR is at https://github.com/appsmithorg/appsmith/pull/9546. It should be released soon within this week.


env setup is confusing

We have Env::default but the actual way to create the default env is theme::init. I think it probably should be Env::default, but maybe there's a method in theme that adds the theme items, and Env::default should just call this?

Will give this a shot. Here's what I changed:
// file env.rs
impl Default for Env {
    fn default() -> Self {
        ...
       let env = Env(Arc::new(inner))
            .adding(Env::DEBUG_PAINT, false)
            .adding(Env::DEBUG_WIDGET_ID, false)
            .adding(Env::DEBUG_WIDGET, false);

       crate::theme::init(env)
    }
}

// file theme.rs
pub fn init(env: Env) -> Env {
    env.adding(WINDOW_BACKGROUND_COLOR, Color::rgb8(0x29, 0x29, 0x29))
     ...
    .adding(
        UI_FONT,
        FontDescriptor::new(FontFamily::SYSTEM_UI).with_size(15.0),
    )
}

I think this is a good solution!


[Core][Distributed] use absolute path for library file

https://github.com/vllm-project/vllm/pull/4155 adds integrity check for nccl library by ldd, which causes problem for rocm: ldd requires full path of library. Therefore, we need to find full path of the library.
Should fix https://github.com/vllm-project/vllm/issues/4257 .

Tested in AMD machine, find_library("librccl.so.1") == '/opt/rocm-5.5.1/lib/librccl.so.1' . I think this PR is good to go now.


Add an implementation of evolutionary algorithm

This PR is ready to be merged now. Thanks Sijin!
I have opened a follow-up issue related to the termination criterion: #98


No way to set DetectorParameters members

The rust side of DetectorParameters only exposes getter methods, while setters are not available. It seems the class members are decorated with CV_PROP_RW and the header parser code does not respect the distinction b/w CV_PROP_RW and CV_PROP.
Fixing it once would work on all issues of this kind. Not sure how much time I can dive into the parser code, so I file this bug in case we could have future changes.

Hi, I’ll need to some time to check it out, but in any case, I’m doing a big ground up rewrite of the parser, so please don’t do any changes to gen_rust.py as it will be deprecated in the near future.

That's good news to me. I can't wait to see the API become more rusty as the parser evolves. As I had used this crate in my project, let me list some wishes that about to change.

The opencv::types::* types work like C++ rather than rusty code. The VectorOfXXX is somewhat too verbose. I would rather derive the InputArray trait to built-in types rather that wrapping around C++ types.
For geometry types like Point*, we may take a look at nalgebra types. They have done a good job on it.
If possible, use expression-return pattern instead of passing mut references. For example, change rodrigues(&src, &mut dst, &mut jacobian) to rodrigues(&src) -> (dst, jacobian). I suppose the python bindings already have it.

Thanks for your suggestions! The first iteration of the new parser will not contain any major improvements though, just faster generation of more complete bindings. But at the later stages I will definitely keep your points in mind!

As far as I peek into gen_rust.py, the ClassPropInfo.rw is never used, while the hdr_parser.py has done its good job. That might be the source of this issue.

The unsoundness is fixed in v0.30.0. Please note that this version comes with the totally rewritten binding generator and you will need llvm installed instead of python to be able to correctly regenerate bindings. Please see the changelog and readme for more details.


[Segment Replication] Fix bug of replica shard's translog not purging on index flush.

Signed-off-by: Rishikesh1159<EMAIL_ADDRESS>
Description
This PR fixes bug of replica shard's translog not purging on index flush when segment replication is enabled and fixes related unit tests.
-> Before this fix, we were not updating local checkpoint of safe commit on the replica's translog deletion policy. Updating this on a different incoming generation purges the translog on index flush.
->  This PR updates the testReplicaReceivesGenIncrease() because previously this test was only asserting doc counts on shards, which is not enough to make sure that replicas translog has been purged. So, we update this test to assert on few translog stats to make sure replicas translog has been purged.
-> This PR also updates the testNRTReplicaPromotedAsPrimary() because assert on totalDocs would fail as at that point we will only have newly added additionalDocs in our translog. Previous operations would be commited and purged from translog before the assert.
Issues Resolved
Resolves #4927
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Do we need integ tests for this?

We already have testReplicaReceivesGenIncrease() which is very close/similar to an integ test. Even if we write an integ test we would be asserting on same stats and would be repetition of same existing test. This test testReplicaReceivesGenIncrease() is working fine and catching the bug correctly, so I think we might not need an integ test. But if you have other thoughts/concerns I can definitely add an integ test.

Codecov Report

Merging #4928 (c7ff44c) into main (782dc59) will decrease coverage by 0.60%.
The diff coverage is 57.89%.

@@             Coverage Diff              @@
##               main    #4928      +/-   ##
============================================
- Coverage     71.37%   70.76%   -0.61%
+ Complexity    58338    57841     -497
============================================
  Files          4689     4688       -1
  Lines        277022   276901     -121
  Branches      40315    40299      -16
============================================
- Hits         197718   195947    -1771
- Misses        63304    64698    +1394
- Partials      16000    16256     +256

Impacted Files
Coverage Δ

.../opensearch/gradle/info/GlobalBuildInfoPlugin.java
37.15% <ø> (+0.59%)
:arrow_up:

...ternal/InternalDistributionArchiveCheckPlugin.java
0.00% <0.00%> (ø)

...ternal/InternalDistributionArchiveSetupPlugin.java
0.00% <ø> (ø)

...ain/java/org/opensearch/painless/antlr/Walker.java
85.01% <ø> (+0.18%)
:arrow_up:

...admin/indices/segments/IndicesSegmentResponse.java
73.86% <ø> (+8.86%)
:arrow_up:

...main/java/org/opensearch/bootstrap/JNANatives.java
18.85% <0.00%> (+2.06%)
:arrow_up:

...ava/org/opensearch/cluster/node/DiscoveryNode.java
89.83% <ø> (+3.68%)
:arrow_up:

...main/java/org/opensearch/index/engine/Segment.java
71.42% <ø> (+4.05%)
:arrow_up:

.../opensearch/index/mapper/DocumentMapperParser.java
90.14% <ø> (+7.02%)
:arrow_up:

...org/opensearch/index/query/ExistsQueryBuilder.java
80.64% <ø> (+2.52%)
:arrow_up:

... and 491 more

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


Carina Framework 6.4.40 is not executing on Microsoft Edge Version 88.0.705.56 (Official build) (64-bit)

Steps to Reproduce:

Execute your code via mvn clean install -Dbrowser=edge / MicrosoftEdge
Verify Result

Actual Outcome:
Tests are failed and states Unsupported browser: edge
As discussed with support
Browsers Supported by Carina Framework is Chrome, Firefox, Opera, Safari and IE.
Edge is supported for older version till Version 14 as Microsoft has changed its edge engine.
Expected Outcome:
Scripts should run seamlessly when command fired mvn clean install -Dbrowser=edge
Environment: Win 10
Browser: Microsoft Edge Version 88.0.705.56 (Official build) (64-bit)
Carina Framework Version:6.4.40

I Have tested with normal scripts and its working fine, but not working with Carina Framework

@jmavileth Hello. Strongly recommended to use the 7+ Carina version. I've tested locally your mvn commands with the 88.0.705.74 browser version and both work fine on the latest Carina version. selenium-remote-driver library uses MicrosoftEdge naming to define browser type and we inherited such logic. When you start selenium locally, pay attention to this command:
java -jar -Dwebdriver.edge.driver=./msedgedriver selenium-server-standalone-3.141.59.jar
Dwebdriver capability should be edge instead MicrosoftEdge.


The diffChangeLog command doesn't work for Snowflake functions

Search first

[X] I searched and no similar issues were found

Description
I have two schemas in my Snowflake database. These schemas contain functions. I have the Liquibase license key and use it during the diffChangeLog command generation. And my generation process throws this error:
Caused by: java.lang.NullPointerException: Cannot invoke "String.equalsIgnoreCase(String)" because the return value of "liquibase.structure.core.StoredDatabaseLogic.getName()" is null
	at liquibase.structure.core.StoredDatabaseLogic.equals(StoredDatabaseLogic.java:79)
	at java.base/java.util.HashMap.putVal(Unknown Source)
	at java.base/java.util.HashMap.put(Unknown Source)
	at java.base/java.util.HashSet.add(Unknown Source)
	at liquibase.structure.DatabaseObjectCollection.add(DatabaseObjectCollection.java:83)
	at liquibase.snapshot.DatabaseSnapshot.include(DatabaseSnapshot.java:329)
	at liquibase.snapshot.DatabaseSnapshot.replaceObject(DatabaseSnapshot.java:456)
	at liquibase.snapshot.DatabaseSnapshot.replaceObject(DatabaseSnapshot.java:478)
	at liquibase.snapshot.DatabaseSnapshot.includeNestedObjects(DatabaseSnapshot.java:369)
	at liquibase.snapshot.DatabaseSnapshot.include(DatabaseSnapshot.java:332)
	at liquibase.snapshot.DatabaseSnapshot.init(DatabaseSnapshot.java:106)
	at liquibase.snapshot.DatabaseSnapshot.<init>(DatabaseSnapshot.java:59)
	at liquibase.snapshot.JdbcDatabaseSnapshot.<init>(JdbcDatabaseSnapshot.java:34)
	at liquibase.snapshot.SnapshotGeneratorFactory.createSnapshot(SnapshotGeneratorFactory.java:215)
	at liquibase.snapshot.SnapshotGeneratorFactory.createSnapshot(SnapshotGeneratorFactory.java:188)

As I can see in the code base - only Snowflake Procedures comparing is supported.
You have already all logic for procedures, you can create the copy for functions, The FUNCTIONS  view as very close to PROCEDURES view.
Steps To Reproduce
Create schema TEST1.
And create schema TEST2.
Add simple functions in schema TEST1, like
CREATE FUNCTION pi_udf()
  RETURNS FLOAT
  AS '3.141592654::FLOAT';

CREATE FUNCTION TEST1.simple_table_function()
  RETURNS TABLE (x INTEGER, y INTEGER)
  AS
  $$
    SELECT 1, 2
    UNION ALL
    SELECT 3, 4
  $$
  ;

Enable the diffChangeLog command for these two schemas (use TEST1 as a target).
Error occurs.
Expected/Desired Behavior
Result with one create function query.
Liquibase Version
4.26
Database Vendor & Version
Snowflake
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
Windows
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

Hello @LonwoLonwo ! Thanks for the issue. In fact the Function code is in liquibase-commercial that's why you can't find it. But anyway this is a bug in commercial code that when comparing the objects is generating a null structure for the non-existent function in the destination database - thus generating the null pointer.

@LonwoLonwo This fix is on commercial, but I guess you can't see it: https://github.com/liquibase/liquibase-pro/pull/1638
Once I get an artifact I'll share the link.

Thank you @filipelautert!

Hello @LonwoLonwo ! Are you able to give current master artifacts a try and see if it fixes the issue? https://github.com/liquibase/liquibase/actions/runs/8568825281/artifacts/1388234226 (from https://github.com/liquibase/liquibase/actions/runs/8568825281 run)

Hello @filipelautert
I'm sorry for not getting back to you sooner. I don't know how exactly I can use artifact files.
Maybe I will wait for the next release?

Hello @filipelautert
I'm sorry for not getting back to you sooner. I don't know how exactly I can use artifact files. Maybe I will wait for the next release?

@LonwoLonwo no problems!

@LonwoLonwo I'll close this one as the fix in merged to master. We are planning to release next week so you'll be able to test, but if the problem persists please reopen this one.

Great, thanks!

Hello @filipelautert
I finally tested the latest version with the fix, and now I have another issue. Unfortunately.
liquibase.exception.DatabaseException: Error executing SQL SELECT GET_DDL('function', 'PI_UDF()'): SQL compilation error:
Object 'PI_UDF()' does not exist or not authorized.
	at liquibase.executor.jvm.JdbcExecutor.execute(JdbcExecutor.java:104)
	at liquibase.executor.jvm.JdbcExecutor.query(JdbcExecutor.java:230)
	at liquibase.executor.jvm.JdbcExecutor.query(JdbcExecutor.java:238)
	at liquibase.executor.jvm.JdbcExecutor.queryForObject(JdbcExecutor.java:246)
	at liquibase.executor.jvm.JdbcExecutor.queryForObject(JdbcExecutor.java:261)
	at liquibase.executor.jvm.JdbcExecutor.queryForObject(JdbcExecutor.java:256)
	at com.datical.liquibase.ext.storedlogic.function.SnowflakeFunctionSnapshotGenerator.getBody(SnowflakeFunctionSnapshotGenerator.java:56)
	at com.datical.liquibase.ext.storedlogic.function.SnowflakeFunctionSnapshotGenerator.performSnapshotAndBuildStoredObject(SnowflakeFunctionSnapshotGenerator.java:39)
	at com.datical.liquibase.ext.storedlogic.function.FunctionSnapshotGenerator.performSnapshotAndBuildStoredObject(FunctionSnapshotGenerator.java:115)
	at com.datical.liquibase.ext.storedlogic.AbstractStoredDatabaseLogicSnapshotGenerator.snapshotObject(AbstractStoredDatabaseLogicSnapshotGenerator.java:88)
	at liquibase.snapshot.jvm.JdbcSnapshotGenerator.snapshot(JdbcSnapshotGenerator.java:65)
	at liquibase.snapshot.SnapshotGeneratorChain.snapshot(SnapshotGeneratorChain.java:60)
	at liquibase.snapshot.jvm.JdbcSnapshotGenerator.snapshot(JdbcSnapshotGenerator.java:68)
	at liquibase.snapshot.SnapshotGeneratorChain.snapshot(SnapshotGeneratorChain.java:60)
	at liquibase.snapshot.jvm.JdbcSnapshotGenerator.snapshot(JdbcSnapshotGenerator.java:68)
	at liquibase.snapshot.SnapshotGeneratorChain.snapshot(SnapshotGeneratorChain.java:60)
	at liquibase.snapshot.jvm.JdbcSnapshotGenerator.snapshot(JdbcSnapshotGenerator.java:68)
	at liquibase.snapshot.SnapshotGeneratorChain.snapshot(SnapshotGeneratorChain.java:60)
	at liquibase.snapshot.DatabaseSnapshot.include(DatabaseSnapshot.java:313)
	at liquibase.snapshot.DatabaseSnapshot.replaceObject(DatabaseSnapshot.java:455)
	at liquibase.snapshot.DatabaseSnapshot.replaceObject(DatabaseSnapshot.java:477)
	at liquibase.snapshot.DatabaseSnapshot.includeNestedObjects(DatabaseSnapshot.java:368)
	at liquibase.snapshot.DatabaseSnapshot.include(DatabaseSnapshot.java:331)
	at liquibase.snapshot.DatabaseSnapshot.init(DatabaseSnapshot.java:106)
	at liquibase.snapshot.DatabaseSnapshot.<init>(DatabaseSnapshot.java:59)
	at liquibase.snapshot.JdbcDatabaseSnapshot.<init>(JdbcDatabaseSnapshot.java:37)
	at liquibase.snapshot.SnapshotGeneratorFactory.createSnapshot(SnapshotGeneratorFactory.java:214)
	at liquibase.snapshot.SnapshotGeneratorFactory.createSnapshot(SnapshotGeneratorFactory.java:187)
	at com.dbeaver.ee.scmp.impl.liquibase.LBCompareEngine.createDatabaseSnapshot(LBCompareEngine.java:709)
	at com.dbeaver.ee.scmp.impl.liquibase.LBCompareEngine.lambda$1(LBCompareEngine.java:222)
	at liquibase.Scope.lambda$child$0(Scope.java:191)
	at liquibase.Scope.child(Scope.java:200)
	at liquibase.Scope.child(Scope.java:190)
	at liquibase.Scope.child(Scope.java:169)
	at com.dbeaver.ee.scmp.impl.liquibase.LBCompareEngine.compareObjects(LBCompareEngine.java:221)
	at com.dbeaver.ee.scmp.model.tasks.SCMPChangelogTaskHandler.lambda$0(SCMPChangelogTaskHandler.java:85)
	at org.jkiss.dbeaver.runtime.RunnableContextDelegate.lambda$0(RunnableContextDelegate.java:39)
	at org.eclipse.jface.operation.ModalContext$ModalContextThread.run(ModalContext.java:124)
Caused by: net.snowflake.client.jdbc.SnowflakeSQLException: SQL compilation error:
Object 'PI_UDF()' does not exist or not authorized.
	at net.snowflake.client.jdbc.SnowflakeUtil.checkErrorAndThrowExceptionSub(SnowflakeUtil.java:129)
	at net.snowflake.client.jdbc.SnowflakeUtil.checkErrorAndThrowException(SnowflakeUtil.java:70)
	at net.snowflake.client.core.StmtUtil.pollForOutput(StmtUtil.java:456)
	at net.snowflake.client.core.StmtUtil.execute(StmtUtil.java:366)
	at net.snowflake.client.core.SFStatement.executeHelper(SFStatement.java:464)
	at net.snowflake.client.core.SFStatement.executeQueryInternal(SFStatement.java:192)
	at net.snowflake.client.core.SFStatement.executeQuery(SFStatement.java:129)
	at net.snowflake.client.core.SFStatement.execute(SFStatement.java:742)
	at net.snowflake.client.core.SFStatement.execute(SFStatement.java:652)
	at net.snowflake.client.jdbc.SnowflakeStatementV1.executeQueryInternal(SnowflakeStatementV1.java:251)
	at net.snowflake.client.jdbc.SnowflakeStatementV1.executeQuery(SnowflakeStatementV1.java:136)
	at liquibase.executor.jvm.JdbcExecutor$QueryStatementCallback.doInStatement(JdbcExecutor.java:551)
	at liquibase.executor.jvm.JdbcExecutor.execute(JdbcExecutor.java:83)
	... 37 more

It looks like you do not use the full function name in this query:
SELECT GET_DDL('function', 'PI_UDF()')
It should be
SELECT GET_DDL('function', 'TEST1.PI_UDF()')
or, better
SELECT GET_DDL('function', 'DATABASE_NAME.SCHEMA_NAME.PI_UDF()')
Do I need to create another ticket for this?

It looks like you do not use the full function name in this query: SELECT GET_DDL('function', 'PI_UDF()') It should be SELECT GET_DDL('function', 'TEST1.PI_UDF()') or, better SELECT GET_DDL('function', 'DATABASE_NAME.SCHEMA_NAME.PI_UDF()')
Do I need to create another ticket for this?

Hi @LonwoLonwo - so close :/
We can just reopen this one and I'll go from here, thanks for the stack trace!

@LonwoLonwo this should have been fixed in master already.


fix: [M3-7269] - Display parent email in user menu when no company name is available for restricted parent user

Description 📝
We learned on the backend that company may be null for some users, even if they payByAkamai. (Though awaiting final confirmation that this won't change going forward and can be retroactively applied to parent/child accounts when the relationship made on the backend.) Assuming company can still be null,  we won't always be able to display the company under the helper text "You are currently logged in as:" in the user menu dropdown for parent and proxy users.
This PR falls back on username in those cases, so we are no longer rendering an empty string in the UserMenu dropdown and success toast.
UPDATE (3/4): Marked as Do Not Merge and Requires Changes because:

Product wants us to fall back on email rather than username (pending final confirmation)

We'd need to ensure parent email cannot be updated to null

The child account list in the Switch Account Drawer uses company name; we would also need to replace with email

UPDATE (3/8): Reopening this PR because we found a case where company is not available: when a restricted parent user has the child_account_access grant enabled (and therefore account-switching abilities) but does not have any Billing Access (account_access: null) -- this means that are not authorized to access account/ and retrieve the company.
Changes  🔄

For a proxy user, we fall back on proxy email rather than parent username + child company if we can’t display the child company name.
For a parent user, we fall back on parent email if we can’t display the parent company name.
Updates test coverage in UserMenu.test.tsx to reflect this.

Preview 📷

Before
After

How to test 🧪
Prerequisites

This can be tested with mocks more easily than in the dev environment because you'd need to ask me for provisioned parent/child account access, then log into alpha admin and delete a child account's company name, and make one manual code edit due to an API bug with child_account_access (the grant isn't being returned correctly) to eliminate the final !isChildAccountAccessRestricted check from const canSwitchBetweenParentOrProxyAccount if you want the Switch Account button to show up.

Reproduction steps

Look at the diff and notice how this code before was defaulting to company being an empty string if the conditional was false, and then displaying that empty string in the user menu and tooltip.

Verification steps

Check out this PR and yarn up.
Ensure the parent/child feature flag is on.
With mocks:
Edit the account endpoint here in serverHandlers.ts to return a null company name.  Confirm your user_type is parent.
Open the user menu and confirm the username is visible. Click the Switch Account button and switch into a proxy account by selecting a child.
Edit the profile endpoint in serverHandlers.ts to have a user_type of proxy.
Confirm the success toast includes the username.
Open the user menu and confirm the username is visible.
(Note that it will be the same username ("mock-user") since mocks will use the same profile endpoint.)

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Closing this PR as it has been determined that we do not need a fallback for companyName. Elsewhere on the backend, we were incorrectly retrieving that data from a value that could be optional. This will be changed on the backend to a value that is required, so the companyName will always exist.

For posterity: we reopened this PR after finding an edge case where if a restricted parent account has no Billing Access but has child_account_access, we can’t access the account/ endpoint to grab the company. However, with the enabled grant, they do have account switching access, so we had a blank in the user menu again.
Following guidance from Product, we can fall back on parent email here.
Issue:

Fix:

We received approval on this from Product so I'm going to go ahead and merge this ahead of the demo since it is a bug that otherwise will be visible.


Ingress route configuration prone to misleading

The ingress route configuration in the example is prone to error/misleading.
The current ingress spec:
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: hello-world-ingress
  namespace: ingress-basic
  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/ssl-redirect: "false"
    nginx.ingress.kubernetes.io/rewrite-target: /$2
spec:
  rules:
  - http:
      paths:
      - backend:
          serviceName: aks-helloworld-one
          servicePort: 80
        path: /(.*)
      - backend:
          serviceName: aks-helloworld-two
          servicePort: 80
        path: /hello-world-two(/|$)(.*)

Suggest to change to:
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: hello-world-ingress
  namespace: ingress-basic
  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/ssl-redirect: "false"
    nginx.ingress.kubernetes.io/rewrite-target: /$2
spec:
  rules:
  - http:
      paths:
      - backend:
          serviceName: aks-helloworld-one
          servicePort: 80
        path: /hello-world-one(/|$)(.*)
      - backend:
          serviceName: aks-helloworld-two
          servicePort: 80
        path: /hello-world-two(/|$)(.*)

Reason:

regex matching for nginx should be considered to be first-match prioritized, instead of length prioritized, i.e. "longer" match doesn't necessarily will go to the wanted backend, e.g. if user put /hello-world-two/abcdefg/ the first path rule is also a match. Even this works in certain ingress-controller versions, it is confusing enough.
the rewrite target is $2 however for the first route there's only one regex match group.

Also suggest to add on why is the rewrite-target used and what does it mean, at least point to the nginx or ingress-controller official doc for further reference.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fd0e7d5a-37e9-07dc-f395-9cb5bb580e45
Version Independent ID: 7e4faf70-9724-7e7b-832b-1cd99a974920
Content: Create an ingress controller - Azure Kubernetes Service
Content Source: articles/aks/ingress-basic.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

More context:
https://kubernetes.github.io/ingress-nginx/user-guide/ingress-path-matching/#path-priority

ingress-nginx first orders the paths by descending length before writing them to the NGINX template as location blocks

ingress-nginx doesn't honestly use the yaml path order and directly templating the nginx.conf, instead it sorts by length with desc order. This means the example ordering should work, however it's still confusing enough as the Nginx regex uses a first match policy.

@tdihp
Thanks for the Feedback for improving our documentation! Just so you know, it's also possible to make these changes yourself, if you are interested. Doing so will improve our documentation, and it will add you as a contributor to the document.
In case you're interested, these are the steps to do so:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format - more info: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
Name your file change and provide a description, if necessary
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page

You can find a GIF that demonstrates the end-to-end process for submitting changes here: https://aka.ms/editsqldocs.

Updated the document changes will go live soon.


Enhancement: shortcut to manually open media control overlay

Hi there,
I'm using your software since a couple days and it's really fine.
I'd like to request a feature so that it'd be possibile to open the overlay whenever I want - just so I can check which song is currently playing.
Excuse me if this is already possible, tried to google up for "windows 10 media control overlay shortcut" to no avail.
Thanks

That's sadly not possible because of how the media controls work.


Send All of funds

Hello how to send your entire balance out your account?
Thank you

I don't think it's possible to send out all your balance. If you have experience of using BTC or ETH or any crypto currency, you will know that an account always need some tokens left for paying transaction fees.

I don't think it's possible to send out all your balance. If you have experience of using BTC or ETH or any crypto currency, you will know that an account always need some tokens left for paying transaction fees.

I mean ONE DOT activation cost which is a high cost considering the price of DOT token. Tokens can be completely withdrawn in polkadot.js wallet, is this not possible in Polkawallet?

Any transaction you can do with polkadot.js Chrome extension are supported in Polkawallet. You can describe your issue and upload some screenshot if you find bugs in the App.

Any transaction you can do with polkadot.js Chrome extension are supported in Polkawallet. You can describe your issue and upload some screenshot if you find bugs in the App.

In the image I attached, the keep-alive option is available in polkawallet?

The keep-alive option is available in polkawallet v2.0.0, released yesterday.
And with the keep-alive option keeps the existencial deposit so you can not send out all your balance. The old Polkawallet do not have this option so you can send out most of your balance and kill your account.

The keep-alive option is available in polkawallet v2.0.0, released yesterday.
And the keep-alive option keeps the existencial deposit so you can not send out all your balance. The old Polkawallet do not have this option so you can send out most of your balance and kill your account.

Thank you very much my problem was solved.


v1.2 - Added sequence diagrams and usage functionality of removeTag() and addTag() commands

Pull Request Test Coverage Report for Build 241

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 58.279%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/ui/UiPart.java
0
2
0.0%

Totals

Change from base Build 240:
-0.05%

Covered Lines:
1260

Relevant Lines:
2162

💛  - Coveralls

The sequence diagrams seem to be accurate. Good work!


Ruby 1.8 compatibility

Hi,
My strange bug with bugged triggers deletes was due to bad table names. It is solved by emulating the %3N pattern in Time#strftime for Ruby < 1.9
I've accidentally included @colin's commit from @sofatutor's fork, but since it's also a Ruby 1.8 compatibility issue, I hope it doesn't matter.

Fixed with version 0.2.0.

Thanks


everything-beta: Add version <IP_ADDRESS>2

I know that at the moment, it's the same as the version in extras, but this will change when there's a new beta out.

"persist" is missing, is it intenttional?

/verify

I think it fails because a new release is available.

/verify


Global component properties pollution fixed

declare module "@vue/runtime-core" breaks all other packages that declare global vue properties (e.g. vue-18n). This change fixes it.

What version of router and pinia are installed?

Here is a screenshot from the file pinia version 4.5.0, vue version 3.4.18

Hmm, I updated the pinia version to 2.2.8 and it really is the same as in your screenshot, but in the project the variables are no longer indexed, also when changing to 'vue' in this package, $yandexMetrika stops being indexed in template

Already ready


Vue SSR Transforms lacking documentation

What problem does this feature solve?
The Documentation for writing SSR Transforms for directives, beside a very basic noop example, is nonexistent. I've written an Image Lazy Loading Directive and want to provide an SSR Transform. adding simple properties like src or poster are working flawless. When it comes to adding a style attribute the lack of documentation makes it very hard find a solution. The transform isn't very complex, but i'm struggling to add style='background-image:url()". See the Transform code below
export const vueLazyLoaderSSRTransform = (dir, node, context) =>
{
    if (node.tag === 'img')
    {
        return {
            props: [createObjectProperty('src', dir.exp)]
        }
    }

    if (node.tag === 'video')
    {
        return {
            props: [createObjectProperty('poster', dir.exp)]
        }
    }

    return {
        props: [
            createObjectProperty('style',
                createObjectExpression([
                    createObjectProperty('background-image',
                        createSimpleExpression('url()', true))
                ]))
        ]
    }
}

It would be great to have a more in deepth documentation on SSR Transforms
What does the proposed API look like?
Enhance the documentation in the SSR guide

Can you open an issue at https://github.com/vuejs/docs to keep track of this?

Thanks issue is now here https://github.com/vuejs/docs/issues/1261


CryptoSeals NFT non visible in wallet.avax.network

I transferred one cryptoseals NFT from my metamask wallet to my Contract Chain address in avax.wallet. When I try to add the token to the token list via contract 0x0540E4EE0C5CdBA347C2f0E011ACF8651bB70Eb9 address is displayed as invalid. It's not possible to see (and transfer) an ERC721 token from avax wallet Contract Chain addresses??

Hello, crypto seals is already added so you are probably getting that error. I am able to see and send them just fine.


Marhal/Unmarshal of map with key but no value does not work as expected

What version of protobuf and what language are you using?
Version: (e.g., v1.1.0, 89a0c16f, etc)
v1.26.0
What did you do?
I created a message that contains a map with another message
syntax = "proto3";

package prototest;
option go_package = "prototest";

message Container {
  string a = 1;
  map<string,Content> b = 2;
}

message Content {
  string a = 1;
  string b = 2;
}

Then I create a new object
cj := &Container{
  A: "1",
  B: map[string]*Content{"1": nil},
}

I expect that marshal/unmarshal generates equal objects.
func TestSimpleMarshalUnmarshal(t *testing.T) {
	cj := &Container{
		A: "1",
		B: map[string]*Content{"1": nil},
	}
	assert.Nil(t, cj.B["1"])

	// marshal
	reqBodyBytes, err := proto.Marshal(cj)
	require.NoError(t, err)

	// unmarshal
	out := &Container{}
	err = proto.Unmarshal(reqBodyBytes, out)
	require.NoError(t, err)

	assert.Nil(t, out.B["1"]) // This will fail
}

What did you expect to see?
I expect that this object is identical after marshal/unmarshal.
What did you see instead?
The unmarshaled object contains an empty object in the map.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment?

The protobuf data model has no concept of a nil map value. (The wire encoding does permit representing this concept, but the text encoding does not and no other language's protobuf API provides a way to distinguish between empty and missing map values.)
Since there is no concept of a nil map value in the protobuf data model, the Go protobuf API treats a nil value as an empty message. We also treat a nil value in a slice representing a repeated message field as an empty message.

Understood. I'd like to mention that this is different from the v1 implementation though.

Understood. The change was made deliberately since the v1 behavior is incorrect. See https://github.com/protocolbuffers/protobuf-go/releases/tag/v1.20.0#v1.20-nil-values for details.

I was thinking about this a bit more. The most "annoying" part about this behavior is that proto.Marshal eats  data that is not properly covered/specified in protobuf spec. Personally, I would prefer if the go marshaller would be more strict (maybe via an optional "Strict" option) to fail properly. This would enable users of this implementation to see issues way more early, especially when they are migrating from v1 to v2.

I have a requirement where I need to proto.Unmarshal a map<string, ABVariantEntries> where ABVariantEntries is a struct
I am trying below approaches but none of them work. Kindly let me know what would be the correct way
styleVariants := make(map[string]*style_fetch.ABVariantEntries)
proto.Unmarshal([]byte(value), &styleVariants)// gives `cannot use &styleVariants (value of type *map[string]*style_fetch.ABVariantEntries) as protoiface.MessageV1 value in argument to proto.Unmarshal: *map[string]*style_fetch.ABVariantEntries does not implement protoiface.MessageV1 `

styleVariants := map[string]*style_fetch.ABVariantEntries{}
proto.Unmarshal([]byte(value), &styleVariants)// gives `cannot use &styleVariants (value of type *map[string]*style_fetch.ABVariantEntries) as protoiface.MessageV1 value in argument to proto.Unmarshal: *map[string]*style_fetch.ABVariantEntries does not implement protoiface.MessageV1`

I had to instead define another strict on top of below proto, and use struct StyleVariant
message StyleVariant {
  map<string, ABVariantEntries> style_variants = 1;
}

styleVariants := &style_fetch.StyleVariant{}
proto.Unmarshal([]byte(value), styleVariants)

Is there a way to directly use map<string, ABVariantEntries> in proto.Unmarshal and not define another struct on top of it?
Any help is appreciated

This seems like something that should have been posted to a new issue, rather than a closed one.
There is no way to marshal/unmarshal individual fields. Protobuf is designed to marshal/unmarshal protobuf Messages only.


chore: release main

:robot: I have created a release beep boop
protocol/core: 3.0.0
3.0.0 (2024-03-12)
⚠ BREAKING CHANGES

breaking fix

Bug Fixes

breaking fix (4aa7fa0)
fix some things (f426185)

runtime/tendermint-ssync: 1.1.2
1.1.2 (2024-03-12)
Bug Fixes

fix some things (f426185)

tools/kystrap: 1.4.2
1.4.2 (2024-03-12)
Bug Fixes

fix some things (f426185)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/shifty11/kyve-rdk-fork/releases/tag/runtime/tendermint-ssync%401.1.2 :sunflower:

:robot: Release is at https://github.com/shifty11/kyve-rdk-fork/releases/tag/tools/kystrap%401.4.2 :sunflower:


Training on nuscenes dataset

Hello CaDDN author, thank you for your great work.
I am trying to train the model using nuscenes dataset. So, I have modified the config yaml file but I received an error
 /backbones_3d/ffe/depth_ffe.py", line 51, in forward     images = batch_dict["images"] KeyError: 'images'
It seems like the dataloader extracts batch of data for lidar only, not camera front view.
array(['n008-2018-08-28-16-43-51-0400__LIDAR_TOP__1535489298547291.pcd',
       'n015-2018-10-02-10-50-40+0800__LIDAR_TOP__1538448754047572.pcd',
       'n008-2018-08-28-16-43-51-0400__LIDAR_TOP__1535489299547057.pcd',
       'n015-2018-07-24-11-22-45+0800__LIDAR_TOP__1532402941797653.pcd'],
      dtype='<U62')

Does it need explicit code modification in /CaDDN/tools/train_utils/train_utils.py file to train on nuscenes?
Look forward for any suggestions.
Thank you for your time.

Hello Thom,
may i know how to modify the model config yaml file for nuscenes? i've never done it before. Or where can i get some information about this?
thank you.


[dunfell][gatesgarth][hardknott][honister][kirkstone] {rolling} Upgrade to 2021-11-12 release

https://discourse.ros.org/t/new-packages-for-ros-2-rolling-ridley-2021-11-02/22963

Merged to other branches as well:
0aeab0bc3..7a9a23573  dunfell -> dunfell-next
3911e756c..38afacd48  gatesgarth -> gatesgarth
3911e756c..38afacd48  gatesgarth -> gatesgarth-next
822117285..583ce5e76  hardknott -> hardknott
822117285..583ce5e76  hardknott -> hardknott-next
7479cfe02..d33f8ed10  honister -> honister
7479cfe02..d33f8ed10  honister -> honister-next
7479cfe02..d33f8ed10  master -> master
7479cfe02..d33f8ed10  master -> master-next


Consolidate mount logic

Convert everything except Mount and Export to use unified mounting logic (those two have special conditions which make it more difficult to use the unified functions). Also simplifies the cleanup/unmount logic a bit.

@rhatdan PTAL

One minor question, otherwise LGTM.

@rh-atomic-bot r+

@rh-atomic-bot retry


would it be possible to put her matchups view of the day?

the loading screen before starting the games, which you can edit like the picks and bans you already have, only with the logos and the team's championship score?

above there is an example of a view of matchups / game break

That's a good idea.
I take note ;)


Comment Bar (fix everything)

Needs to be styled and checked with current logic

this might be a duplicate of #277

dupe of #277. closing


PSD. Load failes in Ubuntu 14.04 for Specific File but works in Mac

Error Log:
Traceback (most recent call last):
File "test_psd-tool.py", line 8, in
img = PSDImage.load(filename)
File "/usr/local/lib/python2.7/dist-packages/psd_tools/user_api/psd_image.py", line 239, in load
return cls.from_stream(fp, encoding)
File "/usr/local/lib/python2.7/dist-packages/psd_tools/user_api/psd_image.py", line 247, in from_stream
psd_tools.reader.parse(fp, encoding)
File "/usr/local/lib/python2.7/dist-packages/psd_tools/decoder/decoder.py", line 32, in parse
image_resource_blocks = image_resources.decode(reader_parse_result.image_resource_blocks),
File "/usr/local/lib/python2.7/dist-packages/psd_tools/decoder/image_resources.py", line 60, in decode
return [parse_image_resource(res) for res in image_resource_blocks]
File "/usr/local/lib/python2.7/dist-packages/psd_tools/decoder/image_resources.py", line 70, in parse_image_resource
return resource._replace(data = decoder(resource.data))
File "/usr/local/lib/python2.7/dist-packages/psd_tools/decoder/image_resources.py", line 143, in _decode_icc
return ImageCms.ImageCmsProfile(io.BytesIO(data))
File "/usr/local/lib/python2.7/dist-packages/PIL/ImageCms.py", line 164, in init
self._set(core.profile_frombytes(profile.read()))
File "/usr/local/lib/python2.7/dist-packages/PIL/_util.py", line 27, in getattr
raise self.ex
ImportError: cannot import name _imagingcms
Little-CMS2 is installed and Pillow & PSD_Tools were uninstalled and re-installd multiple times.
This file has Color Profile: sRGB IEC61966-2.1
It works on Mac but does not work on Ubuntu.

Install littleCML


Flutter without Dart: possible using C and ffigen?

Use case
With Flutter 2 now released with support for targeting desktop applications, it has the potential to unseat existing desktop GUI frameworks. To simplify this translation, one could avoid the rewrite—or creating a c2rust equivalent—by supporting C [or C++, I suppose…] as a proper first-level language to target the "Flutter Platform" with.
Proposal
Mirror the existing Dart API as much as possible. Use code-generation to fill in the gaps. Utilise a popular C/C++ build system for the entire process, e.g., CMake. I don't just mean the Engine (which is currently C++) but every component—including widgets—and complex features which are a big selling point to Flutter (like hot reload, trivial cross-compilation, and UX builders).
Maybe for Flutter 3? 🤩

I personally don't think this is feasible for many reasons. Leaving this open for team to take a look.
Cc @Hixie  for overview.

Flutter is very much a framework designed around Dart. It could certainly be ported to other languages, but it would not be ergonomic as is and would need significant changes. For example, Flutter makes heavy use of inheritance to describe parts of its structure below the widget level. Flutter's development mode as depends on the ability to hot reload code, which requires a VM or interpreted environment to be truly stateful. Inheritance would not be as nice to express in a language like C that does not have this functionality built in, and lacking a VM/interpreter its unclear how hot reload could be made to work without some sort of auto serialization/dynamic library shenanigans.
That's not to say that you couldn't take inspiration from the Flutter APIs to design a new framework written in C or the language of your choice, but that isn't an endeavor the Flutter team is going to embark on.

Yeah this is something I would recommend starting as a separate project. It's totally something that should be possible though. It wouldn't even be the first example, I've heard of several efforts to take Flutter and make a version for another language. Sometimes it comes out as a strict port, sometimes it's more "inspired by"; it really depends on the language. For example, porting Flutter to C++ would require thinking about how you would handle the lack of garbage collection in C++. Porting to Kotlin might make you think about whether using classes is idiomatic enough, as opposed to leaning into closures. Porting to ObjectPascal might require you to think carefully about how mixins are used in Flutter. And so on.

Yeah that's what I was thinking to target C and do all those shenanigans. That way the FFI layer to other languages becomes trivial (every language I can think of has an FFI to C… but only some have to C++). One could imagine getting it to the level where the Dart interface to it becomes wholly generated.
But yeah if the Google folk aren't interested in considering it then it'd be a pretty big independent project; as not only would you need API [and ABI?] compatibility, but you'd need to constantly update to support the latest version. Going to park the idea for now, feel free to contact https://offscale.io if you want to work on it (if there's enough interest then let's do it!)


Please add AI-related project topics

In OMB's Implementation Guidance to Federal Agencies Regarding Enterprise Data and Source
Code Inventories they established a set of AI/ML-related keywords and tags for government work.
I think the cool work in this project is AI-related, would you please update the GitHub project properties to add these keywords to your project topics? Specifically, I think you should add "“usg-artificial-intelligence”.
Once you update your project settings, these changes will be automatically picked up by CDC's metadata scraper and included in the code.json file used by GSA to index and include projects in the federal code catalog.
For help on how to update your project, please check out Github's documentation or just send me an email.

Metadata Added!


Load Reporting filter v0

no-op load reporting filter. To be plugged in to census.

@jboeuf I made some changes to https://github.com/grpc/grpc/pull/7637/files#diff-688faa01dd269343d626342a0646b170 Not sure if that's something you'd be happy with, but seemed the right place to do it (and allows testing more paths).

LGTM

test this please

is this still a thing?

Yes. I'm going to merge and go make an offering on the altar of Jenkins to see if this time things pass...

test this please

test this please

Closing. This has gone way too stale. It'll be revisited in the future.


Setup project repository and development environment

Issue: Set Up Project Structure and Dependencies
Description
We need to set up the project structure with separate folders for the Server (NestJS), WebAPP (Astro), and SDK. Additionally, we'll install the necessary dependencies for the Server and pull the forked repository for the WebAPP. The SDK folder will be prepared for future development.
Tasks

Create Project Structure

Set up three folders: Server, WebAPP, and SDK.

Install Server Dependencies

Navigate to the Server folder and install the required dependencies for NestJS:

Mongoose for MongoDB integration
Crypto for cryptographic functionality
Swagger for API documentation

Pull WebAPP Repository

Clone or pull the forked repository for the WebAPP into the WebAPP folder.

Prepare SDK Folder

Create the SDK folder structure and initialize it for future development.

Acceptance Criteria

[ ] Project structure includes separate folders for Server, WebAPP, and SDK.
[ ] Server dependencies (Mongoose, Crypto, Swagger) installed successfully.
[ ] WebAPP repository pulled into the WebAPP folder.
[ ] SDK folder prepared and ready for future development.

Additional Notes

Make sure to follow the folder structure conventions and keep the project organized.
Document any issues encountered during the setup process for reference.

Labels: Setup, Project Structure, Dependencies
Assignees: @teixeped

Issue update:

webapp folder renamed to client to be consistent with server name
crypto won't be installed in nestJs for now as Node as already a crypto library

Additional tasks added:

Add PR and issues templates

Problems during development:

Originally Nx was to be integrated in the monorepo however it could lead to issues when we have non-JS languages in the repo


⚠ SceneTime has degraded performance

In 28d70c2, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3735 ms

Resolved: SceneTime performance has improved in 2db0cc0 after 6 minutes.


Array#(permutation|combination) without block should return self

Sorry, I made a mistake in the commit message ("without block" -> "with block").


Local storage

Milestone - Local Storage
This pull request includes the following changes:

Add formData object created when user keyup every form input
Add formData object to local storage

Hey @OyePriscilla!
I didn't notice that the navbar items doesn't show, that was for a classname change that I do, now it's fixed, thank you so much for your feedback!

Hey @OyePriscilla! I didn't notice that the navbar items doesn't show, that was for a classname change that I do, now it's fixed, thank you so much for your feedback!

@NicolasAndrehh, 👍


Fix crash caused by ozpIwc.log.logMsg when the dev console is open

//        var log = Function.prototype.bind.call(window.console.log, window.console);
//        log.apply(window.console,["["+level.name+"] "].concat(args));

This breaks Firefox if the the Firebug or dev tools console is open.  Fix it.  This seems to work on Firefox:
  window.console.log.apply(window.console,["["+level.name+"] "].concat(args));

IE might not support apply() on the console, see http://tobyho.com/2012/07/27/taking-over-console-log/

Will not work in IE (tested 9,10,11)
window.console.log.apply(window.console, "Do you support me?");

"Object doesn't support property or method 'apply'"


3D floor plan is not showing

Hi,
I followed the YouTube tutorial to install the custom card, but my floor plan is not displayed in the Home Assistant GUI. I got this error:
Source: components/system_log/__init__.py:190
First occurred: 12:29:30 (17 occurrences)
Last logged: 12:37:51

https://my_domain.duckdns.org:8123/hacsfiles/floor3d-card/floor3d-card.js:739:547477 Error
https://my_domain.duckdns.org:8123/hacsfiles/floor3d-card/floor3d-card.js:739:546060 TypeError: undefined is not an object (evaluating 't.states[e.entity].state')
https://local_ip:8123/hacsfiles/floor3d-card/floor3d-card.js:739:547365 Uncaught Error
https://local_ip:8123/hacsfiles/floor3d-card/floor3d-card.js:739:545973 Uncaught TypeError: Cannot read property 'state' of undefined

And this is displayed when I'm configuring the card via GUI:

I attached a copy of my SweetHome floor plan exported like Obj.
Plano 3D-apto.zip
Thank you for your help to resolve this issue.

Hello jjmuriel, please remove spaces from the file name when exporting to obj. Also define at least one entity or remove the entities section from the visual editor. I have to fix these things but for the moment, please accept the work around

Thank you, but is not working. I get this and the same error:

Logger: frontend.js.latest.202106030
Source: components/system_log/__init__.py:190
First occurred: 12:29:30 (35 occurrences)
Last logged: 15:49:36

https://parca.duckdns.org:8123/hacsfiles/floor3d-card/floor3d-card.js:739:547477 Error
https://parca.duckdns.org:8123/hacsfiles/floor3d-card/floor3d-card.js:739:546060 TypeError: undefined is not an object (evaluating 't.states[e.entity].state')
https://<IP_ADDRESS>:8123/hacsfiles/floor3d-card/floor3d-card.js:739:547365 Uncaught Error
https://<IP_ADDRESS>:8123/hacsfiles/floor3d-card/floor3d-card.js:739:545973 Uncaught TypeError: Cannot read property 'state' of undefined```

This time you did not put the trailing slash in the path:
type: custom:floor3d-card
name: TestHome
path: /local/home2/
objfile: Plano3D-apto.obj
mtlfile: Plano_3D-apto.mtl

I know; there are a lot of things to be improved

Sorry, you're wright, now is working. Thank you very much.

Is it working fast for you ? I develop  using an old laptop, your model take 4-5 seconds to load.

I have a MacBook Pro (Retina, 15-inch, Mid 2014) with core I7 gen 4, 16 GB RAM an integrated Intel GPU and takes 2 to 4 seconds charging the model. In my phone (Huawei P40 Pro) takes 4-6 seconds (via Home Assistant mobile app). Once the model was charge the floor plan manipulation is fast in both platforms.

Hello jj, I propose to close the issue, even though I have to fix the  fact that the card visual editor is always  leaving an empty entity which does not work and requires a manual update

As discussed

No problem. Thank you @adizanni.


chore(LICENSE): move to Apache 2.0 per CNCF

Ref https://github.com/getporter/porter/pull/1278

(Note: CI failure addressed in https://github.com/getporter/helm-mixin/pull/75)


Error: Could not find class 'android.support.v7.widget.Toolbar'

It happens at the startup of the app. After that, it works nicely, but this error always shows up.
This is the complete log related to this issue that appears in logcat:
Could not find class 'android.support.v7.widget.Toolbar', referenced from method uk.co.chrisjenx.calligraphy.CalligraphyFactory.isActionBarSubTitle
VFY: unable to resolve check-cast 998 (Landroid/support/v7/widget/Toolbar;) in Luk/co/chrisjenx/calligraphy/CalligraphyFactory;
Could not find class 'android.support.v7.widget.Toolbar', referenced from method uk.co.chrisjenx.calligraphy.CalligraphyFactory.isActionBarTitle
VFY: unable to resolve check-cast 998 (Landroid/support/v7/widget/Toolbar;) in Luk/co/chrisjenx/calligraphy/CalligraphyFactory;
Could not find class 'android.support.v7.widget.Toolbar', referenced from method uk.co.chrisjenx.calligraphy.CalligraphyFactory.parentIsToolbarV7
VFY: unable to resolve instanceof 998 (Landroid/support/v7/widget/Toolbar;) in Luk/co/chrisjenx/calligraphy/CalligraphyFactory;
Could not find class 'android.support.v7.widget.Toolbar', referenced from method uk.co.chrisjenx.calligraphy.CalligraphyFactory.onViewCreatedInternal
VFY: unable to resolve instanceof 998 (Landroid/support/v7/widget/Toolbar;) in Luk/co/chrisjenx/calligraphy/CalligraphyFactory;
Could not find method java.lang.ReflectiveOperationException.printStackTrace, referenced from method uk.co.chrisjenx.calligraphy.ReflectionUtils.invokeMethod

You are not using v7 support library, the Dalvik/ART VM's do this as they can't find that class. Don't worry about it.


feat: add workspaces support

#46  👈
main

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

Join @pevisscher and the rest of your teammates on  Graphite


Passing php data to Js

Hi there,
I was trying to pass data with an exchanger to my js files but it doesn't seem to work.
I am using the latest version of Sage with ES6, router.js, etc.
What would you recommend to achieve such thing? For instance having an exchanger of data available for all js files?
Thanks.

Feel free to ask on discourse.roots.io or stackoverflow or something. This section for posting bug reports or requesting features related to Sage.

@QWp6t is right, btw take a look at wp_localize_script(), like:
wp_localize_script('sage/main.js', 'lang', [
    'current_lang' => ICL_LANGUAGE_CODE,
    'sun' => _x('Sun', 'day of week', 'aba'),
    ...
]);


Average Directional Movement Index

Average Directional Movement Index
Average Directional Movement Index makes use of three indicators to measure both trend direction and its strength.
Two indicators to measure trend direction: Positive Directional Index(+DI) and Negative Directional Index(-DI)
Indicator to measure trend strength: Average Directional Index(ADX)
Indicaton of Strength:

Trend is strong if ADX > 25 and
Trend is Weak if ADX < 20

Indication of Direction:

uptrend if +DI > -DI and
downtrend if +DI < -DI

Allocation Strategy
Average Directional Movement Index is advantageous in the sense that we can use strength to reduce risk in allocations.

if +DI > -DI and ADX > 25 then allocation = max_allocation
if +DI < -DI  and ADX > 25 then allocation = -max_allocation
else: allocation = 0

Reference
https://www.investopedia.com/terms/a/adx.asp

@Cal231995 please do the necessary research.

@Cal231995 Can you do a research on the topic and update the rules described above if necessary?

Yes I can. I'll send it later. But I think this indicator is not available yet on our platform.

What is Average Directional Index?
The Average Directional Index (ADX) introduce by Wells Wilder, is a technical indicator that is used to help measure the overall strength of the trend. This indicator attempts to measure the strength of price movement in positive and negative direction using the DMI+ and DMI- indicators along with the ADX. A trend is considered strong when the value of the Adx is above 25 and considered weak when the value is below 25. A trend is also considered bullish when the DMI+ is above the DMI- and considered bearish when the DMI+ is below the DMI-
Formula :

Allocation Strategy :
The strategy is to identify the trend and the strong of the trend using the values of ADX, DMI+ and DMI-
Bullish Trend : When the DMI+ is above the DMI- and the ADX value is ABOVE 25
Bearish Trend : When the DMI+ is below the DMI- and ADX is below 25
if +DI > -DI and ADX > 25 then allocation = max_allocation
if +DI < -DI and ADX > 25 then allocation = -max_allocation
else: allocation = 0

Definition free of copyright issues


敲碗求更

Should be able to create the first example of word count within LaTeX with
shell escape enabled :-) I am currently working on the SDR experiment due
in one week.
On Sat, Dec 16, 2023 at 23:34 Niurouxing @.***> wrote:

—
Reply to this email directly, view it on GitHub
https://github.com/Teddy-van-Jerry/LaTeX-Awesome/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4L7UM7U6AVBZ7ES5HJ3J3YJW5QBAVCNFSM6AAAAABAXVFSDGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DIOBRGUZTAMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Lambda environment changes on every apply when KMS used

The SLACK_WEBHOOK_URL variable changes on every terraform apply when KMS encryption is used.  This causes aws_lambda_permission.current_version_triggers to require replacement due to the version of the lambda function incrementing.
I'm not sure what to do about this - you can't ignore_changes in modules

Hi @invertigo !
This sounds like a bug but I don't understand why it happens.
Please show the code you use and log output during "terraform apply".

Pretty basic usage.
data "aws_kms_ciphertext" "slack_url" {
  plaintext = var.slack_webhook_url
  key_id    = aws_kms_key.stack_key.arn
}

module "slack_error_notifications" {
  source  = "terraform-aws-modules/notify-slack/aws"
  version = "4.9.0"
  create  = var.slack_alarms

  sns_topic_name       = "${var.environment}-slack-alerts"
  lambda_function_name = "${var.environment}-slack-alerts"

  slack_webhook_url = data.aws_kms_ciphertext.slack_url.ciphertext_blob
  slack_channel     = var.slack_alert_channel
  slack_username    = "${var.environment}-alerts"
  slack_emoji       = var.slack_alert_emoji

  kms_key_arn         = aws_kms_key.stack_key.arn
  create_with_kms_key = true
}

And then here's the terraform plan output after the entire stack has been provisioned and nothing should be changing:
  # module.mds.module.slack_error_notifications.module.lambda.aws_lambda_function.this[0] will be updated in-place
  ~ resource "aws_lambda_function" "this" {
        arn                            = "arn:aws:lambda:us-west-2:12345678910:function:company-staging-slack-alerts"
        filename                       = "builds/8cdc8be7573f183a30e8af9ce59d511795c17dd9fd51299e8d9e827e1f3de602.zip"
        function_name                  = "company-staging-slack-alerts"
        handler                        = "notify_slack.lambda_handler"
        id                             = "company-staging-slack-alerts"
        invoke_arn                     = "arn:aws:apigateway:us-west-2:lambda:path/2015-03-31/functions/arn:aws:lambda:us-west-2:12345678910:function:company-staging-slack-alerts/invocations"
        kms_key_arn                    = "arn:aws:kms:us-west-2:12345678910:key/a92cfa9e-712d-11eb-877b-83c2d3fe191b"
        last_modified                  = "2021-02-17T01:04:08.564+0000"
        layers                         = []
        memory_size                    = 128
        package_type                   = "Zip"
        publish                        = true
      ~ qualified_arn                  = "arn:aws:lambda:us-west-2:12345678910:function:company-staging-slack-alerts:5" -> (known after apply)
        reserved_concurrent_executions = -1
        role                           = "arn:aws:iam::12345678910:role/lambda-company-staging-slack-alerts"
        runtime                        = "python3.8"
        source_code_hash               = "SP923Y4gOpr1DQN9oV+jXPLvNr+SRbqD7IvhrwhdzwI="
        source_code_size               = 1570
        tags                           = {}
        timeout                        = 30
      ~ version                        = "5" -> (known after apply)

      ~ environment {
          ~ variables = {
                "LOG_EVENTS"        = "False"
                "SLACK_CHANNEL"     = "#alerts-nonprod"
                "SLACK_EMOJI"       = ":aws:"
                "SLACK_USERNAME"    = "company-staging-alerts"
              ~ "SLACK_WEBHOOK_URL" = "AQICAHhOH5OVERWRITINGZPARTSjOFETHISUctGu5Soj01/T4QEJ8aZOt6F15U9k8YXx9tfPAAAArzCBrAYJKoZIhvcNAQcGoIGeMIGbAgEAMIGVBgkqTOiRUINBTHEwBASE64ZIAWUDBAEuMBEEDL6mWZ/vpr6R1+H1oQIBEIBoWE5JSFQY6hvL+YsdmYrHdjBVTS3MOhh6Y4/8SEvpf35HeJa34xFfW1P0jOgG8D3UnGkUhAWHB8oqjKL0xTzVXmunbF4xPu/mtsTMcENLcxdzKa0wASsluOoG7KVVjT1O0u8BpRM3xLw=" -> "AQICAHhOH5sCMPQRzK2NOVERWRITINGZBITScHERESTOO1/T4QEwqTyEShWEoyqYRnIGplCDAAAArzCBrAYJKoZIhvcNAQcGoIGeMIGbAgEAMIGVBgkqhkiG9w0BBwEwHgYJYIZIASODBYOUMCANTJDECODEnnC5dc5ragIBEIBoVMsaDMUGk9XgFUxN8gxfIyc/JU2YlAvZvas1Pd1+ChELGmgcu/SSlJETcgyqXWs1519w1fGOjD/8jp5braxDa8bqjfVuLovzsvFWSLCniVRsxHw89IZIOT3d8SMUXNpqYO0Ce/1HvJM="
            }
        }

        tracing_config {
            mode = "PassThrough"
        }
    }

  # module.mds.module.slack_error_notifications.module.lambda.aws_lambda_permission.current_version_triggers["AllowExecutionFromSNS"] must be replaced
-/+ resource "aws_lambda_permission" "current_version_triggers" {
        action        = "lambda:InvokeFunction"
        function_name = "company-staging-slack-alerts"
      ~ id            = "AllowExecutionFromSNS" -> (known after apply)
        principal     = "sns.amazonaws.com"
      ~ qualifier     = "5" -> (known after apply) # forces replacement
        source_arn    = "arn:aws:sns:us-west-2:12345678910:company-staging-slack-alerts"
        statement_id  = "AllowExecutionFromSNS"
    }

Plan: 1 to add, 2 to change, 1 to destroy.

The KMS-encrypted SLACK_WEBHOOK_URL changes every time I run terraform, forcing some new resources.  I dunno how difficult it would be to get a lifecycle hook to ignore changes for that, but it would be welcome.

So not quite a bug, since TF is acting properly given the environment variable changes on every run.  I wonder if it's possible to tell TF not to regenerate the slack_url ciphertext on every run.

Aha: "The KMS ciphertext data source allows you to encrypt plaintext into ciphertext by using an AWS KMS customer master key. The value returned by this data source changes every apply. For a stable ciphertext value, see the aws_kms_ciphertext resource."
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/kms_ciphertext

Exactly :)
Here is an example for this - https://github.com/terraform-aws-modules/terraform-aws-notify-slack/blob/master/examples/cloudwatch-alerts-to-slack/main.tf#L11-L14


Nested classes, preload and const

I'm using a class to aggregate some preloads.
# aggregate.gd
const A = preload("A.gd")
const B = preload("B.gd")

and access the classes through aggregate.A.new(), etc.
Now I have something like this:
# aggregate.gd
const A = preload("A.gd")
const B = preload("B.gd")
const C = B.NestedSomething
const D = B.NestedSomething2

I can still access the classes through aggregate.C.new() etc, but the code editor complains with
Invalid index 'C' in constant expression
But it still seems to work and compile fine. Using var instead of const removes the editor error, but then it breaks runtime and I can't access the classes anymore.
Is this a bug, a feature or how am I supposed to interpret this behaviour?
(Hopefully, it's understandable I suck at markdown)

I fixed the code formatting. I don't really understand what's happening, but it seems that the parser is having some trouble identifying the constant expressions.

@vnen Okay, so I can safely ignore the editor complaint for now.

This won't compile anymore in the current master. With error invalid index 'NestedSomething' in constant expression.
So I guess this isn't relevant anymore?

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

I'll close this as nobody else has been able to reproduce this recently. Please comment (with the issue template filled) if you can still reproduce this.


ValueError: invalid literal for int() with base 16

First of all, thanks for the tool! My team has been successfully using this.
Describe the bug
With Python 3.7.6 and pdfminer 20200124.
Given this file, when running
python pdf2txt.py -V --output_type xml file.pdf

I get an incomplete xml output with a stacktrace
<?xml version="1.0" encoding="utf-8" ?>
<pages>
Traceback (most recent call last):
  File "/usr/local/bin/pdf2txt.py", line 188, in <module>
    sys.exit(main())
  File "/usr/local/bin/pdf2txt.py", line 182, in main
    outfp = extract_text(**vars(A))
  File "/usr/local/bin/pdf2txt.py", line 56, in extract_text
    pdfminer.high_level.extract_text_to_fp(fp, **locals())
  File "/usr/local/lib/python3.7/site-packages/pdfminer/high_level.py", line 85, in extract_text_to_fp
    interpreter.process_page(page)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdfinterp.py", line 895, in process_page
    self.render_contents(page.resources, page.contents, ctm=ctm)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdfinterp.py", line 906, in render_contents
    self.init_resources(resources)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdfinterp.py", line 354, in init_resources
    self.fontmap[fontid] = self.rsrcmgr.get_font(objid, spec)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdfinterp.py", line 187, in get_font
    font = PDFTrueTypeFont(self, spec)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdffont.py", line 620, in __init__
    PDFSimpleFont.__init__(self, descriptor, widths, spec)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/pdffont.py", line 580, in __init__
    self.cid2unicode = EncodingDB.get_encoding(name, diff)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/encodingdb.py", line 108, in get_encoding
    cid2unicode[cid] = name2unicode(x.name)
  File "/usr/local/lib/python3.7/site-packages/pdfminer/encodingdb.py", line 54, in name2unicode
    unicode_digit = int(name_without_u, base=16)
ValueError: invalid literal for int() with base 16: 'bunt'

The file is a pdf with non-embedded fonts.
To help isolate the issue, I was able to reproduce it with this docker setup.
Could this be a bug?
Thanks!

Hi @rcarraretto, thanks for raising this issue!
I can replicate this error. I'll look into this now.

As far as I can see there is an error in your PDF file. More specific, the `` font that is defined in object 369 (after cleaning up with mutools):
369 0 obj
<<
  /Type /Font
  /Subtype /TrueType
  /Name /Fo12S5
  /BaseFont /Ubuntu-Medium
  /FirstChar 0
  /LastChar 125
  /Widths 366 0 R
  /Encoding 367 0 R
  /ToUnicode 368 0 R
  /FontDescriptor 345 0 R
>>
endobj

The encoding of this font has a list of differences wrt the Ubuntu-Medium font.
367 0 obj
<<
  /Type /Encoding
  /Differences [ 32 /uni1F9F /uni1FA0 /uni1FA1 /uni1FA2 /uni1FA3
      /uni1FA4 /uni1FA5 /uni1FA6 /uni1FA7 /uni1FA8 /uni1FA9 /uni1FAA
      /uni1FAB /uni1FAC /uni1FAD /uni1FAE /uni1FAF /uni1FB0 /uni1FB1
      /uni1FB2 /uni1FB3 /uni1FB4 /uni1FB6 /uni1FB7 /uni1FB8 /uni1FB9
      /uni1FBA /uni1FBB /uni1FBC /uni1FBD /uni1FBE /uni1FBF /uni1FC0
      /uni1FC1 /uni1FC2 /uni1FC3 /uni1FC4 /uni1FC6 /uni1FC7 /uni1FC8
      /uni1FC9 /uni1FCA /uni1FCB /uni1FCC /uni1FCD /uni1FCE /uni1FCF
      /uni1FD0 /uni1FD1 /uni1FD2 /uni1FD3 /uni1FD6 /uni1FD7 /uni1FD8
      /uni1FD9 /uni1FDA /uni1FDB /uni1FDD /uni1FDE /uni1FDF /uni1FE0
      /uni1FE1 /uni1FE2 /uni1FE3 /uni1FE4 /uni1FE5 /uni1FE6 /uni1FE7
      /uni1FE8 /uni1FE9 /uni1FEA /uni1FEB /uni1FEC /uni1FED /uni1FEE
      /uni1FEF /uni1FF2 /uni1FF3 /uni1FF4 /uni1FF6 /uni1FF7 /uni1FF8
      /uni1FF9 /uni1FFA /uni1FFB /uni1FFC /uni1FFD /uni1FFE /uni20B9
      /uniE0FF /uniEFFD /ubuntu /uniF0FF /uniF000 ]
>>
endobj

The second last element is /ubuntu and because it starts with /u pdfminer.six tries to interpret it as a unicode. Obviously, this is not possible.
I'm not sure how the /unicode value should be interpret. After replacing it with /.notdef the pdf file is parsed without errors.

A quick fix is to skip characters with exceptions. We already do this for KeyError and I will create a PR where ValueError is added to the list.

Cool, thanks for the fix @pietermarsman!


[mgmt] support release

https://github.com/Azure/sdk-release-request/issues/5022

/azp run js - support - mgmt - ci

/azp run js - support - mgmt - ci


[WIP] Add: iterate empty tuple

Closes #5372. Solves reproducers.  Perhaps it will have sense in some cases.

I thought I saw an issue or PR somewhere to support iterating over heterogenous tuples.
Perhaps it's better to wait for that change rather than to build this hack,

Thank you.
Please do not consider above commits as any kind of "solution". It is more "playing with test system/numba internals".

@1844144 thanks for opening this patch and taking a look at this issue. PR #7750 is attempting to provide a more general solution for iterating over empty containers and is under review, so am going to close this PR. Thanks for your understanding.


[rstream] - inconsistent behaviour with metaStream

Hi Karsten, I've playing with the library again after a bit of a break. I'm trying out the metaStream feature that was added a few months ago. It looks like it adds a lot of flexibility. I've been experimenting with using like this. I know there isn't any examples of usage like this so maybe it's not intended to be used in the is way!
const delayedStream = n => {
  return stream(o => {
    delayed(2 * n, 1000).then(x => {
      o.next(x);
      o.done();
    });
  });
};

const a = stream()
  .subscribe(metaStream(delayedStream));

a.next(99);

So this would generate a stream for each value coming in. Obviously this is just a toy example but it would be useful to define the behaviour of an app calling a backend for instance. Only the latest request would be kept and all others would be dropped. This works actually. Almost :-)
If I do subscription like this:
a.subscribe(trace())

I'll get the transformed value from the delayed stream.
However if I chain a subscription directly I'll get the value that was directly sent into a:
// Chained sub won't use the delayedStream and instead will output value sent in to a
const a = stream()
  .subscribe(metaStream(delayedStream)).subscribe(trace());

I hope this is clear. Let me know if you need any clarification on it. Thanks!

Hey @allforabit - missed you here! :) I will have to dig into this later (by tomorrow) and can't give you a clear explanation right now, but for now wanted to point you to the doc strings for metastream (only in case you haven't see it)...

Hey again, so I figured it out and it's actually very simple:
The return value for a.subscribe(b) is the value (or a possibly wrapped version of) b, not a. Likewise, for a chain like x = a.subscribe(b).subscribe(c), the x will be the result of the last subscribe() (i.e. c or a wrapped version of it).
In other words, the confusion here comes from the misinterpretation of your first variation:
const a = stream().subscribe(metaStream(delayedStream));

Here a is not the original seed stream, but already the metaStream instance. The example still seemingly works, because 1) using next() you can inject values at any point in the processing pipeline (as long as you have a handle to it) and 2) there're no other processing steps between stream() and the metaStream sub...
OTOH, here the issue becomes more obvious:
const a = stream().transform(map((x)=>x*100)).subscribe(metaStream(delayedStream));

a.next(99);
// 198 (still)

a instanceof MetaStream
// true

So I guess, you already can see the solution :) - just split the call chain and assign the right parts to variable(s):
const a = stream();
a.subscribe(metaStream(delayedStream)).subscribe(trace());

a.next(99);
// 198 (after delay)

Makes sense?

Doh! Yes of course, thanks very much for the quick and helpful response :-)

Pleasure & good luck!

@allforabit Forgot to mention earlier, you might find this helpful too: https://github.com/thi-ng/umbrella/wiki/rstream-recipes

Ah very nice thanks, I hadn't seen that.


Interdependent symlinked packages don't resolve

Describe the bug
Interdependent symlinked packages don't resolve as expected (i.e., they are not optimized during development and they don't resolve at all).
Reproduction
Not sure how to reproduce this without some manual work done locally, but here's how I achieved the simplest reproduction:
Part 1: Create interdependent symlinked packages
First, create two new local folders, foo and bar.
In the foo folder, add the following package.json:
{
  "name": "@vite-symlink-test/foo",
  "version": "1.0.0",
  "main": "index.js",
  "module": "index.js"
}

And the index.js of foo is:
// @vite-symlink-test/foo/index.js
export const foo = 'foo'

Run npm link in the foo directory to create the symlink.
bar is nearly identical, except that it depends on foo.
{
  "name": "@vite-symlink-test/bar",
  "version": "1.0.0",
  "main": "index.js",
  "module": "index.js"
}

And the entry file:
// @vite-symlink-test/bar/index.js
import { foo } from '@vite-symlink-test/foo'

export const bar = `${foo}bar`

Again, run npm link to symlink the package.
Note that I scoped both packages under @vite-symlink-test to avoid namespace collisions.
Part 2: Init new vite app, link the packages, and use them.
Make a new Vite app and link the packages:
npm init vite-app vite-symlink-test
cd vite-symlink-test
npm link @vite-symlink-test/foo @vite-symlink-test/bar

Import and use the bar package in your app:
// App.vue
<template>
  <img alt="Vue logo" src="./assets/logo.png" />
  <HelloWorld msg="Hello Vue 3.0 + Vite" />
</template>

<script>
import { bar } from '@vite-symlink-test/bar'
import HelloWorld from './components/HelloWorld.vue'

export default {
  name: 'App',
  components: {
    HelloWorld
  },
  setup() {
    console.log(bar)
  }
}
</script>

Run npm run dev and visit localhost:3000.
On my system, I get the following error:
[vite] Failed to resolve module import "@vite-symlink-test/foo". (imported by /<EMAIL_ADDRESS>
However, I can still import foo directly and use it normally.
I consistently see this error when a symlinked package imports another symlinked package. I'm unsure if this is a Rollup issue, or more specifically Vite, but I know that the same technique of symlinking interdependent packages does not cause problems for me in Nuxt (Webpack-based).
EDIT: Today I followed all the same steps listed above, except that I symlinked, imported, and successfully used the bar package in a fresh Vue CLI project + vue-cli-plugin-vue-next instead of Vite. It worked perfectly, and Vue CLI is also Webpack-based, which increases the likelihood of this being a Rollup or Vite-specific bug.
Relevant dependencies from the Vue CLI app:
"dependencies": {
    "core-js": "^3.6.5",
    "vue": "^3.0.0-beta.1"
  },
  "devDependencies": {
    "@vue/cli-plugin-babel": "~4.4.0",
    "@vue/cli-service": "~4.4.0",
    "@vue/compiler-sfc": "^3.0.0-beta.1",
    "babel-eslint": "^10.1.0",
    "eslint": "^6.7.2",
    "eslint-plugin-vue": "^7.0.0-alpha.0",
    "vue-cli-plugin-vue-next": "~0.1.3"
  },

System Info

required vite version: 1.0.0-beta.3
required Operating System: MacOS 10.13.6
required Node version: 12.13.0
Optional:

npm/yarn version 6.14.4
Installed vue version (from yarn.lock or package-lock.json) 3.0.0-beta.15
Installed @vue/compiler-sfc version 3.0.0-beta.15

Logs (Optional if provided reproduction)
As mentioned above:
[vite] Failed to resolve module import "@vite-simlink-test/foo". (imported by /<EMAIL_ADDRESS>

I tested it but not reproduction it with 1.0.0-beta.11.Can you upgrade vite have a try?
If you also meet it, can you provider a git repo with this? (Maye be I missed sth.)

I upgraded to Node 14.5.0 and vite 1.0.0-beta.11 but I'm still seeing the same issue.
I created a repo to make this a little easier:
git clone https://github.com/AlexVipond/vite-symlink-test.git

Once you have the repo cloned, you'll still need to set up the symlinks locally:
# Starting from top level of the repo
cd symlinks/foo && npm link
cd ../bar && npm link

And then you'll need to link those to the vite app. I put in a script to make it easier:
# Starting from top level of the repo
cd vite-app
npm run link
npm run dev

When I do this and I visit in the browser, it triggers an error
[vite] Failed to resolve module import "@vite-symlink-test/foo". (imported by /<EMAIL_ADDRESS>
But if you go into App.vue in the vite-app directory and comment out the bar stuff (lines 3, 8, and 17), it works fine.

You forgot link foo inside bar.You need run npm link @vite-symlink-test/foo inside bar dir and that work for you.

Linking foo inside bar fixes it, thanks! Weird that Webpack is still able to resolve it without that step


added US ANSI layout with TUX key

Here is the US ANSI layout variant for the InfinityBook Pro 14 Gen6

Here is the custom SUPER key variant I would like to have engraved.


Added nl_NL translation

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a new translation

Description
Added the Dutch translations by @nfrelink
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage remained the same at 38.349% when pulling c3b506e7cae199c0edcf27e40e17ae7ab44ab45a on cyian-1756:nl_NLTranslation into 7eeb6b8378354f770552bea59573be5f6b1cb436 on RipMeApp:master.


Adding tests for Go 1.6

Despite the highest minor version of Go being the only supported version, I think it is important to include tests for the current and most recent minors.

Might as well add master to that list. It would be good to know if something upcoming breaks the clients.

@lukebakken should we allow failures on master?

Yep

:shipit:

Oh wait, I meant the exact opposite ... let's not allow failures so we know when things break.