Implement  #2099，添加apollo-client的身份认证功能及accesskey管理功能

What's the purpose of this PR

添加apollo-configservice对apollo-client的身份认证
添加apollo-portal的accesskey管理功能

Which issue(s) this PR fixes:
Implement  #2099
Brief changelog
apollo-client

在客户端添加配置apollo.accesskey.secret，用来做访问apollo对密钥
用secret对请求签名，在客户端访问configservice对请求添加请求头Authorization,Timestamp

apollo-configservice

添加filter过滤器，解析header，对客户端请求进行身份认证，如果非法请求则返回401
对于旧版项目没有配置secret对，默认跳过。等添加了secret才会进行身份认证
ApolloConfigDB添加新表AccessKey

CREATE TABLE `AccessKey` (
  `Id` int(10) unsigned NOT NULL AUTO_INCREMENT COMMENT '自增主键',
  `AppId` varchar(500) NOT NULL DEFAULT 'default' COMMENT 'AppID',
  `Secret` varchar(128) NOT NULL DEFAULT '' COMMENT 'Secret',
  `IsEnabled` bit(1) NOT NULL DEFAULT b'0' COMMENT '1: enabled, 0: disabled',
  `IsDeleted` bit(1) NOT NULL DEFAULT b'0' COMMENT '1: deleted, 0: normal',
  `DataChange_CreatedBy` varchar(32) NOT NULL DEFAULT 'default' COMMENT '创建人邮箱前缀',
  `DataChange_CreatedTime` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP COMMENT '创建时间',
  `DataChange_LastModifiedBy` varchar(32) NOT NULL DEFAULT '' COMMENT '最后修改人邮箱前缀',
  `DataChange_LastTime` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP COMMENT '最后修改时间',
  PRIMARY KEY (`Id`),
  KEY `AppId` (`AppId`(191)),
  KEY `DataChange_LastTime` (`DataChange_LastTime`)
) ENGINE=InnoDB AUTO_INCREMENT=1 DEFAULT CHARSET=utf8mb4 COMMENT='访问密钥';

apollo-portal & apollo-adminservice

感谢 @huangchenhai-666 提供Portal页面支持

添加密钥管理功能
支持多个密钥（最多5个），用于密钥更替时兼容
已创建的项目需要添加权限。可以用insert select清洗数据

INSERT INTO `Permission` (`PermissionType`, `TargetId`, `IsDeleted`, `DataChange_CreatedBy`, `DataChange_LastModifiedBy`) VALUES ( 'ManageAppMaster', 'demo', 0, 'apollo', 'apollo');

Featrue Preview
项目管理员可以看到管理密钥对菜单

为每个项目配置密钥，每个环境密钥不同。选择环境添加后自动生成，默认是禁用的

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.

Codecov Report

Merging #2828 into master will decrease coverage by 0.63%.
The diff coverage is 27.64%.

@@             Coverage Diff              @@
##             master    #2828      +/-   ##
============================================
- Coverage     50.46%   49.83%   -0.64%
- Complexity     2084     2115      +31
============================================
  Files           414      425      +11
  Lines         12668    13034     +366
  Branches       1295     1333      +38
============================================
+ Hits           6393     6495     +102
- Misses         5821     6074     +253
- Partials        454      465      +11

Impacted Files
Coverage Δ
Complexity Δ

...ring/boot/ApolloApplicationContextInitializer.java
93.61% <ø> (ø)
15 <0> (ø)
:arrow_down:

...trip/framework/apollo/common/dto/AccessKeyDTO.java
0% <0%> (ø)
0 <0> (?)

...rk/foundation/internals/provider/NullProvider.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...framework/apollo/core/signature/HmacSha1Utils.java
0% <0%> (ø)
0 <0> (?)

...rip/framework/apollo/core/signature/Signature.java
0% <0%> (ø)
0 <0> (?)

...k/apollo/portal/component/PermissionValidator.java
15.9% <0%> (-0.76%)
1 <0> (ø)

.../defaultimpl/DefaultRoleInitializationService.java
87.36% <100%> (ø)
17 <0> (ø)
:arrow_down:

...va/com/ctrip/framework/apollo/util/ConfigUtil.java
94.61% <100%> (+0.04%)
50 <1> (+1)
:arrow_up:

.../configservice/ConfigServiceAutoConfiguration.java
93.33% <100%> (+1.66%)
5 <1> (+1)
:arrow_up:

...framework/apollo/biz/service/AccessKeyService.java
12.12% <12.12%> (ø)
1 <1> (?)

... and 25 more

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 9c75d90...6df8130. Read the comment docs.

Coverage decreased (-0.7%) to 53.399% when pulling 6df81306a84a0a36f1db404265195896673396bb on 360jinrong:feature/accesskey into 9c75d908f85829f9e60eb272256e610a8a1f3366 on ctripcorp:master.

@nobodyiam Hi, Song. Thank you for your review and for some advice. I have resolved some problems that you mentioned above, and this PR has some changes as follows:

Use isAppAdmin replace hasManageAccessKeyPermission, then we just need to create AccessKey table  with the upgrade, no more DML to fix historical data.
Add icon and some tips for AccessKey Management in the portal, it is more beautiful than before.
RenameAccessFilter toClientAuthenticationFilter.
Add some unit tests for core modules.

Please take a look.
Preview

@nobodyiam I have fixed the problems above, the changes are as follow:

Revert some files that have nothing changes.
Add new unit test case about AccessKeySecret in RemoteConfigRepositoryTest and RemoteConfigLongPollServiceTest
Squash git commits. There are 2 commits in this PR.

@nisiyong
I have tested in my local environment and it looked great!
However, it seems it will take up to 1 minute to take effect after I enable/disable the access key. It may be a problem in some situations.
Can we change the implementation of com.ctrip.framework.apollo.configservice.service.AccessKeyServiceWithCache#scanNewAccessKeys to make it faster?

However, it seems it will take up to 1 minute to take effect after I enable/disable the access key. It may be a problem in some situations.

Yes, the cache is refreshed every minute by default. In my opinion, I don't think it is a problem because we don't change the secret often. Maybe you can tell me more details about this in some situations.

Can we change the implementation of com.ctrip.framework.apollo.configservice.service.AccessKeyServiceWithCache#scanNewAccessKeys to make it faster?

Do you mean to change the default rebuildInterval? I think I can set it 10 seconds by default.

@nisiyong
Suppose the user enable the secret and just to find the applications printing error logs (because of no secret), so he decides to disable the secret temporarily. In this situation, he has to wait for up to 1 minute to see the effect, which I think will confuse the user.
What I suggest is to change the implementation, e.g. find the new/updated access key by DataChange_LastTime, I think this can solve the problem.

What I suggest is to change the implementation, e.g. find the new/updated access key by DataChange_LastTime, I think this can solve the problem.

@nobodyiam
I have considered this implementation before, but the deletion of the access key will still take effect one minute by default. It is indeed better to use DataChange_LastTime to find the new/updated access key every second by default.
Okay, I will change the implementation this week.

Codecov Report

Merging #2828 into master will increase coverage by 8.53%.
The diff coverage is 52.23%.

@@             Coverage Diff              @@
##             master    #2828      +/-   ##
============================================
+ Coverage      50.4%   58.94%   +8.53%
+ Complexity     2083     1219     -864
============================================
  Files           414      201     -213
  Lines         12672     6084    -6588
  Branches       1297      649     -648
============================================
- Hits           6387     3586    -2801
+ Misses         5831     2227    -3604
+ Partials        454      271     -183

Impacted Files
Coverage Δ
Complexity Δ

...ring/boot/ApolloApplicationContextInitializer.java
93.61% <ø> (ø)
15 <0> (ø)
:arrow_down:

...m/ctrip/framework/apollo/biz/config/BizConfig.java
27.77% <0%> (-3.48%)
8 <0> (ø)

...trip/framework/apollo/common/dto/AccessKeyDTO.java
0% <0%> (ø)
0 <0> (?)

...rk/foundation/internals/provider/NullProvider.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../apollo/internals/RemoteConfigLongPollService.java
80.72% <100%> (+0.47%)
29 <0> (+1)
:arrow_up:

...ework/apollo/internals/RemoteConfigRepository.java
87.42% <100%> (+0.32%)
23 <0> (+1)
:arrow_up:

...va/com/ctrip/framework/apollo/util/ConfigUtil.java
94.61% <100%> (+0.04%)
50 <1> (+1)
:arrow_up:

.../ctrip/framework/apollo/util/http/HttpRequest.java
86.66% <100%> (+3.33%)
7 <2> (+2)
:arrow_up:

...com/ctrip/framework/apollo/util/http/HttpUtil.java
75% <20%> (-5%)
8 <0> (ø)

...framework/apollo/biz/service/AccessKeyService.java
42.42% <42.42%> (ø)
3 <3> (?)

... and 233 more

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 99a6f5d...205f18a. Read the comment docs.

@nobodyiam
I have already solved all the problems you mentioned. If there are other problems, I will be happy to solve them.


chore(main): release 1.2.0-beta

:robot: I have created a release beep boop
1.2.0-beta (2023-07-26)
Features

typescript templates (694ced9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/appblocks-hub/BB-CLI/releases/tag/v1.2.0-beta :sunflower:


VIH-6259 Added unit tests

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/VIH-6259
Change description

Added unit tests

I am expecting the coverage to be close to 100% once integration tests are completed in the next stage

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No


Does not work with older versions of node

It works fine under the latest version of node (4.2.2 here), but when testing on v0.10.36 (the version Amazon's AWS Lambda runs), it fails with Promise is not defined.
I'd suggest requiring a minimal version of node, so npm warns on install and to mention it in the README or to make it compatible with older versions of node.
I'd be happy to stick a warning in the README and create a pull request if you want me to.

Alternatively, I fixed it it now by creating a Promise using bluebird: global.Promise = require('bluebird');

@niklasR  - Faced the same problem today. Found out that in one of the files i wasn't requiring bluebird.
Added Promise = require('bluebird'); and now it works in lamdba.

Warning added to the README ! Thanks !


Chose which tables are created on  app init

how i can turn off auto create table user + animal models
edit: Please add the description of your issue on the description, not in the title

Hello @caohothang
you can either manually list each entity you want to register on your database here
https://github.com/tsmean/tsmean/blob/master/backend/src/database/database.providers.ts#L23
or remove [name].entity.ts files and tyeorm will not recognize them.

for more information about entities and typeorm related configuration on the connection you can check this link http://typeorm.io/#/connection

@AngelMunoz  thanks for your support i will try it.


[SC-889] Deploy on Base network

Codecov Report
Patch and project coverage have no change.

Comparison is base (874f564) 78.31% compared to head (222f9f4) 78.31%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   78.31%   78.31%
=======================================
  Files          26       26
  Lines         738      738
  Branches      167      167
=======================================
  Hits          578      578
  Misses        160      160

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


Bugs on submission form

Describe the bug
I've fount out a couple of bugs in the submission form.
To Reproduce
Bug #1

start a new submission
edit the title
save the submission form
edit again the title by cancelling all the text and then point the form focus to another form's field
edit again the title by entering a new text
save the submission form, you should get an error notification

Bug #2

start a new submission
add an empty new item in a repeatable field by clicking on the Add more button
delete the new item added by clicking on the trash icon
save the submission form, you should get an error notification

Expected behavior
In case of bug#1 the form should properly save the change. In case of bug #2 no changes to save should be detected

@tdonohue the estimation is about 2 hours and I can open the new PR to resolve them

@atarix83 : Yes, these sound like issues to fix immediately.  Thanks, I'll add the 2 hr estimate & move to the 7.0 board for your team to work on.


fixes issue 781

Current coverage is 11.39% (diff: 0.00%)

No coverage report found for development at 55fb90e.

Powered by Codecov. Last update 55fb90e...f5b66d5

@the-dagger yeah sure. dont know why i am getting "menu_tracks.xml and menu_sessiondetail.xml file changed but not tracked" .

@the-dagger have made the changes. u can have a look now.


   Use -n X instead.

I still can't build on OSX because of this.

Thanks for the reminder, should be fixed now (I don't have a non GNU environment to test in, so please confirm this).

Yes. It works now. Thanks.


mtl: Implement fill_buffer

I suppose this is WIP, because I'm still unable to get dEQP-VK.api.fill_and_update_buffer.suballocation.fill_buffer_whole to pass with this change. It works fine with the quad example in the cases I've tested.
Need to keep investigating... maybe something unrelated to this change causes the failure. (turned out to be an issue with the CTS using 0 bytes during invalidate memory range)
(Also need to fix Vulkan/dx12 because of the signature change to support VK_WHOLE_SIZE) Vulkan now sends the correct values, DX12 still expects to fill the entire range (as it did before)
For context, this uses a simple compute shader to write the uint32 value into the Metal buffer, because Metal has no way to do this natively (the native fillBuffer only allows uint8). There is probably lots of room for optimization here (although the optimal amount of elements to write per thread is device-dependent).
PR checklist:

[X] make succeeds (on *nix)
[X] make reftests succeeds
[X] tested examples with the following backends: Metal

We should assume 0 as wd do everywhere else (iirc, there was a resolve_range helper)

On May 7, 2018, at 21:55, Josh Groves<EMAIL_ADDRESS>wrote:
@grovesNL commented on this pull request.
In src/backend/metal/src/command.rs:

       .unwrap();

   let pso = pipes

       .get_fill_buffer()

       .to_owned();

   // TODO: Reuse buffer allocation

   let fill = self.shared.device.new_buffer_with_data(

       &data as *const u32 as *const _,

       mem::size_of::<u32>() as _,

       metal::MTLResourceOptions::StorageModeShared,

   );

   inner.retained_buffers.push(fill.clone());

   let start = *range.start().unwrap();

@kvark Should we assume 0 (i.e. .. would mean fill buffer) or simply expect here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Still need to fix the threads/threadgroups
I'm not sure whether pipeline barriers are required in the function where fill_buffer is actually called, maybe somebody could clarify?
Currently the tests are a bit flaky on Metal when we perform fill_buffer for a subset of the buffer. The buffer somehow isn't always initialized to zero, so we end up with i.e. [72, 65, 76, 80, 255, 0, 0, 0], where we only care about the 255, 0, 0, 0 due to the fill. Need to investigate further
It seems like my Vulkan driver doesn't round down to the nearest smaller multiple and writes parts of the u32 value to the remainder, so that test is commented out for now too

The buffer somehow isn't always initialized to zero

Why do you expect the buffer to initialize to zero?

@kvark Regardless I think we just need another Expectation variant that matches a subset of a buffer.
I was slightly surprised because most of the MTLBuffer creations will initialize to zero (i.e. new MTLBuffer object with a new zero-filled memory allocation) or some data you provide. But it doesn't state this for buffers allocated from MTLHeap for example, so it seems fine.

@kvark Updated my commit, please take a look. I'm still not sure about the pipeline barriers. Are both these states needed?

I was slightly surprised because most of the MTLBuffer creations will initialize to zero

Perhaps, but warden framework is backend agnostic.

Updated to address comments except for the single buffer and partial write

@kvark Rebase was a bit ugly because the changes were similar 😆 everything should be ok though. PTAL

warning: spurious network error (2 tries remaining): [6] Couldn't resolve host name (Could not resolve host: crates.io)

Took the opportunity to fix the last WORD_ALIGNMENT 😃
bors r=kvark

Bors r=grovesNL

On May 10, 2018, at 00:01, bors[bot]<EMAIL_ADDRESS>wrote:
🔒 Permission denied
Existing reviewers: click here to make grovesNL a reviewer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Bors r-

On May 10, 2018, at 00:01, bors[bot]<EMAIL_ADDRESS>wrote:
🔒 Permission denied
Existing reviewers: click here to make grovesNL a reviewer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Bors r+

On May 10, 2018, at 00:01, bors[bot]<EMAIL_ADDRESS>wrote:
🔒 Permission denied
Existing reviewers: click here to make grovesNL a reviewer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[BUG] Azure Key Vault Secret client library SecretClient.getSecret call hangs if processed by more than 11 threads

Describe the bug
com.azure.security.keyvault.secrets.SecretClient.getSecret hangs if processed by more than 11 threads in parallel.
Exception or Stack Trace
N/A
To Reproduce
Call getSecret from more than 11 threads. For simplicity, list.parallelStream can be used.
Code Snippet
package azure_keyvault_secret;

import com.azure.core.credential.TokenCredential;
import com.azure.identity.ClientSecretCredentialBuilder;
import com.azure.security.keyvault.secrets.SecretClient;
import com.azure.security.keyvault.secrets.SecretClientBuilder;

import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.util.List;
import java.util.Objects;
import java.util.stream.Collectors;

public class App {
    public static void main(String[] args) throws IOException {
        final String clientId = args[0];
        final String clientSecret = args[1];
        final String tenantId = args[2];
        final String vaultName = args[3];
        final String vaultUrl = String.format("https://%s.vault.azure.net", vaultName);
        // read secret names from a file
        final List<String> secretNames = Files.readAllLines(Path.of(args[4]));

        final TokenCredential tokenCredential =
                new ClientSecretCredentialBuilder()
                        .clientId(clientId)
                        .clientSecret(clientSecret)
                        .tenantId(tenantId)
                        .build();

        final SecretClient secretClient = new SecretClientBuilder()
                .vaultUrl(vaultUrl)
                .credential(tokenCredential)
                .buildClient();

        // 11 secrets are fetched without any issues, >= 12 makes app hang and no secret is returned...
        final int secretsToFetch = 11;
        final List<String> subList = secretNames.subList(0, secretsToFetch);
        System.out.printf("Fetching %d secrets ...", subList.size());

        final List<String> secrets = subList.parallelStream()
                .map(secretName -> secretClient.getSecret(secretName).getValue())
                .filter(Objects::nonNull)
                .collect(Collectors.toList());

        System.out.println("Secrets fetched:" + secrets.size());
    }
}

Expected behavior
getSecret should work for more than 11 threads in parallel. There shouldn't be a limit on how many times it can be called in parallel.
Screenshots
N/A
Setup (please complete the following information):

OS: macOS Catalina 10.15.7
IDE : IntelliJ
Version of the Library used:

    implementation 'com.azure:azure-security-keyvault-secrets:4.2.3'
    implementation 'com.azure:azure-identity:1.2.0'

Additional context
N/A
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Let me preface this by explaining a bit about the authentication flow in the Azure Key Vault SDK: for any request to make to the service, we first make a naive unauthenticated call to obtain a challenge that will then be used in a subsequent authenticated call. This flow uses the Azure Identity library internally, which by default uses the common ForkJoinPool to obtain authentication details to add to calls.
parallelStream uses whichever ForkJoinPool it was created in and, in the case of the example you shared, uses the same pool as the Identity library.
Given all this, it is possible to have parallelStream use up all the available threads in the common pool, making first the initial unauthenticated requests and not leaving any available thread for the authentication flow to start at all. In my computer, for example, the limit to this is 8 threads since I only have 8 logical cores, so I was only able to get 7 secrets simultaneously but nothing more.
There is a few ways to get around this:

Change the thread pool Azure Identity will use by specifying which ExecutorService to use:
final TokenCredential tokenCredential =
    new ClientSecretCredentialBuilder()
        .clientId(clientId)
        .clientSecret(clientSecret)
        .tenantId(tenantId)
        .executorService(Executors.newCachedThreadPool()) // Here
        .build();

Change the thread pool the parallel stream will use. Although it appears this is not officially supported by the language[1][2].

Use an async client. The Azure SDK async clients use Project Reactor, which uses its own thread pool. Here's a snippet showing how this could end up looking like:
AtomicInteger secretsCounter = new AtomicInteger();

Flux.fromIterable(subList)
    .flatMap(secretAsyncClient::getSecret)
    .doOnNext(secret -> {
        System.out.println(secret.getName() + ":" + secret.getValue());
        secretsCounter.getAndIncrement();
    })
    .blockLast();

System.out.println("Secrets fetched: " + secretsCounter);

All in all, I think the least disruptive and most stable solution is probably number 1.
I will be closing this issue for the time being but please feel free to re-open it if you found out that what I suggested was not sufficient or incorrect.
Finally, I want to give a quick shoutout to @srnagar for helping me debug this issue.

@vcolin7 Thank you for the update, I'll modify our code as per suggested point # 1.

@usmansaleem  do you know how we can get this secrets from AKV to ConfigMap or Application YAML file !


Feature Request - Add Tag display & selection

Feature Description
Similar to how layers are displayed and can be changed, tags would be nice too.
Reason for feature
QHierarchy could do that and it was quite useful

Happy to add this - can get this in soon! 👍

thank you, your work is greatly appreciated!

This has been implemented!
Honestly, it never had the focus because bigger underlying systems, but it was essentially the same logic as the layers being displayed, so thats on me 😭
I've implemented it so they can be displayed above each other, or horizontally in the hierarchy. It looks a little cramped when above one another, but keeps things cleaner.
if you have any suggestions let me know! 🙏


BigCache implementation does not return an error of type *store.NotFound when the key is not found

I'm trying to check if a key exists in the cache. Since there is not a CacheInterface.Exists, I have to rely on CacheInterface.Get to perform the check. The Redis implementation correctly returns *store.NotFound when the key is not found, but the BigCache one does not.
Steps for Reproduction

Set up BigCache:

bigcacheClient, _ := bigcache.New(ctx, bigcache.DefaultConfig(5 * time.Minute))
bigcacheStore := bigcache_store.NewBigcache(bigcacheClient)
cacheManager := cache.New[[]byte](bigcacheStore)

Check the error type:

_, err := bigCacheCache.Get(ctx, "key")
_, ok := err.(*store.NotFound)
fmt.Printf("BigCache Type is *store.NotFound: %v\n", ok)

See that is not of type *store.NotFound
Perform the previous steps with Redis and see how the error is of type *store.NotFound

Expected behavior:
Returned error is of type *store.NotFound
Actual behavior:
Returned error is of not of type *store.NotFound
Platforms:
macOS and dockerized Linux from scratch
Versions:
gocache v4.1.3 and v4.1.4
go 1.21
BigCache store v4.1.2 and v4.2.0
BigCache v3.1.0
Redis store v4.2.0
Redis client v9.0.5
Redis server 7.0.12

This seems like a trivial fix, just need to handle bigcache.ErrEntryNotFound in Get method. However, many existing library usages may be dependent on current behavior which returns error directly. So fixing this may cause issues for some users.

This seems like a trivial fix, just need to handle bigcache.ErrEntryNotFound in Get method. However, many existing library usages may be dependent on current behavior which returns error directly. So fixing this may cause issues for some users.

The two errors could be joined so that errors.Is would be true for both


Some of the icons download has error - no content in the file

Describe the bug
Some of the icons download has issues opening. I only tested with "dollar" and "dollar-bill". But "Unknown" and "Home" are behaving correctly.
When the file is open in the Chrome browser, it shows "error on line 1 at column 1: Document is empty".
How to reproduce
Go to https://clarity.design/icons; search wit"
Steps to reproduce the behavior:

Go to '.https://clarity.design/icons.'
Search for  'dollar'
Downloads the icons
Open the icons in the web browser

Expected behavior
Users should be able to see the icons.
Versions
App

Angular: [e.g. 6]
Node: [e.g. 8.10.0]
Clarity: [e.g. 0.12.5]

Device:

Type: MacBook
OS: macOS Mojave
Browser: Chrome
Version:  72.0.3626.121 (Official Build) (64-bit)

Additional notes
Thank you. :-)

@Shijir
I looked into this and it appears that the Commerce shapes are not available to download and are also not in the ZIP file of all archives. Could you get this corrected for the release this week?

For some reason, I missed this set in the clarity-assets repo, just need to update the latest build into icons/commerce


Implement chunking mechanism for loading synapses in dryrun mode

Context
Up until now for the purposes of counting in the dry run mode, all synapses were loaded at the same time. This started to create some issues when moving to very large circuit where the execution would consistently run out of memory.
This MR tries to fix this issue by chunking the loading in blocks of 100 cells at a time in order not to spend the whole memory.
Scope
The changes are relatively small and include the adding of a small generator for chunking ranges in the util section, and a small rework of the loading synapses block to load cells in predefined ranges.
Testing
No new functionality so no new tests are necessary.
Review

[x] PR description is complete
[x] Coding style (imports, function length, New functions, classes or files) are good
[x] Unit/Scientific test added
[x] Updated Readme, in-code, developer documentation

This approach is a good start. We can just maybe limit the number of buckets to 10 and then apply that change which samples from all metypes


Add callbacks to commit

PR for issue #55

Add several structs to save information from commit importing logic.
Add callback enum to execute code that propagates the commit in case the information gathered is good.

(will merge after CI)


Support for other versions?

Do you have plans for supporting other versions?
341, 381, 383, 430
Thanks for making this project. Also, I'm curious what was your motivation for making it?

Hello @joelatgrayv. Thank you for your interest in dedex.
I actually have plans to support other versions, in particular later versions than the currently supported (3.8.2 and 4.1).
Note that the consortium announced the "sunset" plan of some versions of the standard, as visible here: https://kb.ddex.net/about-ddex-standards/what's-new/

On 24th February 2023, DDEX announced that the following standards will be “sun-set” from 2025 onwards:
All versions of the Electronic Release Notification Message Suite standard (ERN) that have the main version number of 3 and the version 4.0 of that standard;

In practice, many still use versions 3.8.*. This sunset policy may encourage companies to upgrade their versions.

Also, I'm curious what was your motivation for making it?

As a software developer in the music industry, I had to code DDEX formatters or parsers multiple times.  It might seems "easy" to code an XML parser, and even fun; that's without considering the complexity and the possibilities of DDEX standards... It's harder than it looks.
Do you have any particular need? I encourage you to share to experience around the library/standard.
Have a good day.

Nice, thanks!
Would you be wiling to accept a PR for 341? Also, do you have any tips for adding that support for it?

Sure, be my guest.
I suggest to start with one (or more) DDEX 341 examples that you add in the test suite; then write tests that parse this file and gets some data.
Don't commit private data in the XML file, like sender/recipients names and party IDs.
You'll also need to find the 341 XSD file to generate the PHP classes (I explain it in the official documentation).
Please read the README and CONTRIBUTE files and try to push a first version of your PR. We can then discuss in the PR.


UI change - added a services column in the host table view with filter

Implemented a new column in the host table view, to see all detected services.
Also include a regex filter input on top of the table.
This is greatly increase ease of managing and filtering many hosts.
Signed-off-by: Suen Chun Hui<EMAIL_ADDRESS>

push


502 Bad Gateway

When I try to open the homepage https://cyber-dojo.org/creator/home I get 502 Bad Gateway error.
I don't know if this is the correct repository to post this problem.

Don't know what changed, but it is working now.


Issues 34 36 38

Resolution of issue #34, issue #36, issue #38.

Real readme.md created.  Licensing information regularized: copyright notice added to every file; notices.md file added.  Big/Little endian conditional compilation preprocessor symbol and related removed.


Net Worth Toolkit Report crashes for individual accounts

#Tookit Reports
System Settings (Required)
ToolKit Version: 1.3.0
Operating System (Mac/Windows/Etc): Windows 10
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: Version 61.0.3163.100 (Official Build) (64-bit)

Other Extensions:

Adblock Plus 1.13.4
Used on over 100 million devices, Adblock Plus is the world's most popular ad blocker.
Details  Options

Allow in incognito

Allow access to file URLs

Enabled

Add to Amazon Wish List <IP_ADDRESS>
Add products from any website to your Amazon Wish List
Details  Options

Allow in incognito

Enabled

Bounce! 0.4
NodeMCU/Esp8266 Visual Programming Language.
Details

Allow in incognito

Enabled
Not from Chrome Web Store.

Change Colors 2.144
Change pages styling according to the user's preferences.
Details  Options

Allow in incognito

Enabled

Chrome Connectivity Diagnostics 1.2.1
Details

Allow in incognito

Enabled

Chrome Remote Desktop 62.0.3202.40
Access other computers or allow another user to access your computer securely over the Internet.
Details

Allow in incognito

Allow access to file URLs

Enabled

Docs 0.10
Create and edit documents
Details

Allow in incognito

Enabled

Dropbox for Gmail 1.1.5
Send and preview Dropbox files and links without leaving your Gmail window.
Details

Allow in incognito

Allow access to file URLs

Enabled

Evernote Web Clipper 6.12.3
Use the Evernote extension to save things you see on the web into your Evernote account.
Details  Options

Allow in incognito

Allow access to file URLs

Enabled

Flashcontrol 9.4
Block Flash from loading automatically and allow content only on selected sites.
Details  Options

Allow in incognito

Enabled

h264ify 1.0.6
Makes YouTube stream H.264 videos instead of VP8/VP9 videos
Details  Options

Allow in incognito

Enabled

Markdown Viewer 3.1
Markdown Viewer
Details  Options

Allow in incognito

Allow access to file URLs

Enabled

RoboForm Password Manager <IP_ADDRESS>
RoboForm, the #1 ranked Password Manager makes your life easier by remembering passwords and logging you into websites automatically
Details  Options

Allow in incognito

Allow access to file URLs

Enabled

Stylist 2.1.0
Chrome Stylist
Details  Options

Allow in incognito

Enabled

Toolkit for YNAB 1.3.0
UI customisations and tweaks for the web version of YNAB.
Details  Options

Allow in incognito

Enabled

None newly added

Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export

[{"key":"AccountsDisplayDensity","value":"0"},{"key":"AccountsEmphasizedOutflows","value":true},{"key":"AccountsStripedRows","value":true},{"key":"ActivityTransactionLink","value":true},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":true},{"key":"BetterScrollbars","value":"0"},{"key":"BudgetBalanceToZero","value":true},{"key":"BudgetQuickSwitch","value":false},{"key":"CategoryActivityCopy","value":false},{"key":"ChangeEnterBehavior","value":false},{"key":"CheckNumbers","value":true},{"key":"ClearSelection","value":false},{"key":"ColourBlindMode","value":true},{"key":"CustomFlagNames","value":true},{"key":"DisplayTargetGoalAmount","value":"2"},{"key":"EditAccountButton","value":"2"},{"key":"GoalWarningColor","value":false},{"key":"HideAgeOfMoney","value":false},{"key":"HideReferralBanner","value":true},{"key":"ImportNotification","value":"0"},{"key":"PrintingImprovements","value":false},{"key":"PrivacyMode","value":"0"},{"key":"RemovePositiveHighlight","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"RunningBalance","value":"1"},{"key":"ShowCategoryBalance","value":true},{"key":"SplitKeyboardShortcut","value":false},{"key":"SquareNegativeMode","value":true},{"key":"StealingFromFuture","value":false},{"key":"TargetBalanceWarning","value":true},{"key":"accountsClearSelection","value":false},{"key":"activityTransactionLink","value":true},{"key":"budgetBalanceToZero","value":false},{"key":"budgetProgressBars","value":"both"},{"key":"budgetQuickSwitch","value":false},{"key":"calendarFirstDay","value":"0"},{"key":"categoryActivityPopupWidth","value":"2"},{"key":"checkCreditBalances","value":true},{"key":"collapseExpandBudgetGroups","value":true},{"key":"collapseSideMenu","value":true},{"key":"compactIncomeVsExpense","value":false},{"key":"currentMonthIndicator","value":true},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"0"},{"key":"enableRetroCalculator","value":true},{"key":"enlargeCategoriesDropdown","value":true},{"key":"goalIndicator","value":true},{"key":"googleFontsSelector","value":"0"},{"key":"hideAccountBalancesType","value":"0"},{"key":"hideHelp","value":true},{"key":"highlightNegativesNegative","value":true},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":true},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"navDisplayDensity","value":"0"},{"key":"pacing","value":"3"},{"key":"popupCalculator","value":true},{"key":"reconciledTextColor","value":"0"},{"key":"removeZeroCategories","value":false},{"key":"reports","value":true},{"key":"resizeInspector","value":true},{"key":"rightClickToEdit","value":true},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":false},{"key":"spareChange","value":false},{"key":"splitKeyboardShortcut","value":false},{"key":"splitTransactionAutoAdjust","value":false},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":false},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"2"},{"key":"warnOnQuickBudget","value":false}]

Steps to Reproduce the Behavior

Select Toolkit Reports
Select Net Worth
Select Dec 16 to Oct 17
Select individual account

Expected Behavior
Graph showing Net Worth for account by month.
Actual Behavior
Received message: An error occurred in Toolkit for YNAB (a browser extension you installed to enhance YNAB). Click here to report the issue to the YNAB Toolkit project.
Proper behavior for "All Accounts" and "All Budget Accounts". Fails for "All Tracking Accounts", as well as individual accounts

@woodnboats thanks for the bug report. I reformatted your issue to use the standard layout with one exception. I put the Other Extensions info in an expandable section to reduce the amount of scrolling necessary to get to the reproduction steps.

Thanks, David. I had a little trouble figuring out how to use the report
template. Should I just have entered  my text without removing the prompts?
Kirk
On Mon, Oct 23, 2017 at 2:38 PM, David Baldon<EMAIL_ADDRESS>wrote:

@woodnboats https://github.com/woodnboats thanks for the bug report. I
reformatted your issue to use the standard layout with one exception. I put
the Other Extensions info in an expandable section to reduce the amount of
scrolling necessary to get to the reproduction steps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/1008#issuecomment-338789121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfNLAogoJseF2xrILkow12ozj4xNKr7Tks5svPkvgaJpZM4QDcev
.

@woodnboats the only text that should be removed is the [paste settings here] stuff. Most folks also don't include the other extensions info which is generally ok.

Hey @woodnboats the fix for this should be in our current release. If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!

Still have the problem, Josh. I checked to make sure the current versions of both Toolkit and YNAB are installed, but when I try to view Net Worth of anything other than All Accounts or All Budget Accounts (such as viewing just my Savings Account or All Tracking Accounts, I get the message:
Uh-oh!
An error occurred in Toolkit for YNAB (a browser extension you installed to enhance YNAB). Click here to report the issue to the YNAB Toolkit project.
Not a terribly big deal, but it would be nice to have it working.
Thanks for the Team's good work, and your attention.
Kirk

@woodnboats okay, I re-ran the test case on my old budget and I was able to reproduce it there, I think I found a fix as well -- by change do you know if you're selecting an account which doesn't have any data for the selected date range? That's the issue I've reproduced and have a fix for in our next release.

Hey @woodnboats the fix for this should be in our current release. If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!

YNAB no longer crashes when I try to see the net worth for a single
account,but no graph is shown either. Perhaps I am trying to do something
not designed.I get graphs with All Accounts and All Budgeting Accounts, but
not All Tracking Accounts, nor any individual account. Is that how it is
supposed to work?
Please forgive my ignorance,
Kirk
On Thu, Mar 15, 2018 at 10:08 PM, Josh Madewell<EMAIL_ADDRESS>wrote:

Closed #1008
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/1008.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/1008#event-1524753129,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfNLApAaSdxX9gdDpvELodUXSUpjCRfLks5tezqkgaJpZM4QDcev
.

@woodnboats My best guess is that you're trying to look at your net worth of an account in a date range where there are no transactions on that account. If you do a search on the account you're filtering for the same date range do you see any transactions?


Task are not queries correctly (assuming due or scheduled date detection)

What happened?
Hello there!
I have put my tasks into several notes to keep the tasks roughly together for the same project / topic.
However, in my dashboard, I want to query only the tasks for this week, unfortunately this is kind of unreliable since a few weeks (I recognized this behavior a few weeks ago - not sure if it was present before).
Example task:
- [ ] Test entry? ⌛ 2023-09-07
Based on the query:
TASK
WHERE contains(text, "⛳") OR
      (scheduled <= date(today) AND scheduled != null) OR
      (due <= date(eow) AND due != null) OR
      (start <= date(today) AND start != null)
WHERE !completed

it should show up in the list, but doesn't. Now if I go back to my task, delete the date and type it in with the exact same characters, there is a good chance for it to show up:
- [ ] Test entry? ⌛ 2023-09-07
I just noticed (while pasting in here), the hour glasses are not the same. In Obsidian, they look identical, but when pasting here, they are different.
Maybe a few more symbols needs to be added for detecting dates?
hourglass not working: ⌛
hourglass working: ⌛
- [ ] Test entry not working via dataview tasks: ⌛ 2023-09-07
- [ ] Test entry WORKING with dataview tasks: ⌛ 2023-09-07

The same did happen to me with the calendar symbols, but I'm not able to reproduce it right now.
Please let me know if you need any additional information.
Kind regards,
Loradim
DQL
TASK
WHERE contains(text, "⛳") OR
      (scheduled <= date(today) AND scheduled != null) OR
      (due <= date(eow) AND due != null) OR
      (start <= date(today) AND start != null)
WHERE !completed

JS
No response
Dataview Version
0.5.56
Obsidian Version
Version 1.4.5 (Installer 1.1.16)
OS
MacOS

Addition: I'm using the Emoji Toolbar plugin to access the calendar and hour glass "characters".
Version 0.4.1

Ah emoji parsing. See relevant comment: https://github.com/blacksmithgu/obsidian-dataview/issues/1502#issuecomment-1299590495

Looking a bit more into it, I can say:

both different hour glasses look the same in Obsidian and on this website via iPhone as well os Chrome on macOS
they look both different in macOS Texteditor (though another "different look") and on this website via Safari browser on macOS
in Swift Playgrounds and Xcode they look identical, but string comparison considers them not equal

Gotta love Unicode
Anyway, this is very frustrating from a user point of view (though I'm sure it's frustrating from a developer point of view as well), as I can't be sure my tasks will pop up as expected (and I was overdue on at least two items due to this).
Is there any way to fix this or any workaround you can suggest?
Kind regards,
Loradim

I kind of found out, how I managed to produce the „wrong“ symbols… when working on my mobile phone, I made tasks including scheduled or due dates as well, using the iOS keyboard and their emoji search.
An emoji used this way would not get recognized in Dataview.
Maybe this is helpful in any way.
Kind regards,
Loradim


No static method AlertDialog - missing dependency? Material3 issue?

Here line 84 is where ComposeCalendar is called.
It's confusing - I can create an AlertDialog on its own, but I get this error when using ComposeCalendar.
The above error is when I paste your sample code into a material design 3 project; it seemed to work when I pasted it into a material design 2 project.

It might be an issue with a mismatched version of the Compose Material 3 library since the one used here is still in alpha. Try 1.0.0-alpha14 and see if that resolves the issue. Otherwise, I've made a fork that drops the AlertDialog usage if you want to give that a try. https://github.com/sproctor/ComposeCalendar/tree/date_input I'll upload it to maven central later today.

This should've been fixed in v1.1.0 now.


Solution does not build as UAssetGUI is referenced in solution file but not included in repo

Describe the bug
A clean clone of this repository does not build successfully as the solution file references the UAssetGUI project, which is in its own repo and not referenced here. Is this supposed to be included as a submodule?
To Reproduce
Steps to reproduce the behavior:

Clone this repository
Run dotnet build (or attempt to build the solution with any other tool)
The build will fail as the UAssetGUI project is not present

Expected behavior
Ideally, this repo should be able to be built, or the extra steps documented. If UAssetGUI was added as a submodule, anyone could just git submodule update --init  and get the right version of UAssetGUI immediately present in their environment.
Desktop:

OS: any, but tested on Windows 10
Version or Commit: 1a43738

I have rearranged the project structure such that you have the option to build UAssetAPI with a solution that does not include UAssetGUI (which may be useful if you are not using Visual Studio). Please see the revised build instructions in the README.md file if needed.


Update Playground with the latest package changes

coverage: 100.0%. remained the same
when pulling 2e35d158ca49cd64c68ab507d7b9610e16b4dd99 on update_playground
into 4d8c34ad5d9d8a58ef96b341d7f1372c8eac614c on master.


Added Codeforces problem

added button codeforces B question

@kingjuno please don't change the default Pull Request Template.Could you please send another PR with the automatically generated PR template and complete the checkboxes in order for your PR to reviewed.


NewGitCommit

This was my new Pull request

Thank you @aman566 for the PR!
Since we want to keep the mistakes for other participants to work on, we wouldn't be merging this PR. Hope the practice was good nevertheless :)


Vagrant with SSL

Trying to get Revel to run in Vagrant using HTTPS and SSL. I am unable to bind to port 443 and wanted to know if I have any other options for this? I am currently stuck as HTTPS is required =(

You can proxy it through apache and mod_proxy

Tried that already. Nothing seems to work. I can forward port 80 to 9000 and http works fine. When I use https its as if it isnt even getting there.

Is your vhost like ?:
<VirtualHost *:443>
...

With http I do not even need Apache though, so you're saying I will need apache? That seems a little counter-intuitive

Generally for most sites you want to use a server to serve the static content of the site. Also with apache you will need to enable ssl using a2enmod ssl

I do not want to use Apache. Also this is for a dev environment only and I would like to be able to specify an https port like I can with http. Is this not possible with Revel? If not is there a specific reason? I would very much like to never install apache or Nginx if I can help it.

I should reiterate I am on MacOS using Vagrant running Debian. It seems like 443 is not being forwarded properly or at all. I have to forward 443 to a DIFFERENT port other than 443 because Vagrant wont forward a port < 1024

ok nevermind I was able to get this dev oauth server working without https. it is fine since it is just my dev env


secret_key problem

Traceback (most recent call last):
File "/Users/apple/hack/muxi/muxi/muxi.py", line 396, in call
return self.wsgi_app(environ, start_response)
File "/Users/apple/hack/muxi/muxi/muxi.py", line 384, in wsgi_app
with ActiveRequestContext(self, environ):
File "/Users/apple/hack/muxi/muxi/muxi.py", line 207, in enter
_request_ctx_stack.push(_RequestContext(self.app, self.environ))
File "/Users/apple/hack/muxi/muxi/muxi.py", line 95, in init
self.session = app.open_session(self.request)
File "/Users/apple/hack/muxi/muxi/muxi.py", line 317, in open_session
secret_key=key
File "/Library/Python/2.7/site-packages/werkzeug/contrib/securecookie.py", line 298, in load_cookie
data = request.cookies.get(key)
File "/Library/Python/2.7/site-packages/werkzeug/local.py", line 338, in getattr
return getattr(self._get_current_object(), name)
File "/Library/Python/2.7/site-packages/werkzeug/local.py", line 297, in _get_current_object
return self.__local()
File "/Users/apple/hack/muxi/muxi/muxi.py", line 53, in
request = LocalProxy(lambda: _request_ctx_stack.top.request) #|
AttributeError: 'NoneType' object has no attribute 'request'

secret_key 有毒呀！！

request.cookies => {}
This may be the problem!!!!

but, if key is not exist, request.cookies.get(key) will return None not error .....

so, the problem is here: global request in :8ball: func:open_session

果然是这个...(hahaha, 做好了用处还是很大大的.)

Yes, If I change it to test request, it work!
from werkzeug.test import create_environ
from werkzeug.wrappers import Request

environ = create_environ()
Request(environ)


Added Layout synthesizing files for ALIGN

Layout using ALIGN

Hi,
Could you please check the Layout/ALIGN/setup_files/bandgap.sp, 2 important nodes seem to be missing, vref , the output and en, the enable input. Also, I have replaced the diodes with diode connected parasitic BJTs, which are realisable in the CMOS process.
You could find the modified subckt here: https://github.com/sherylcorina/avsdbgp_3v3/blob/master/Documentation/Beta/Subcircuit/bandgap.cir


It's not apply activeColor in text

With:
• react-native 0.59.8
• react-navigation 3.11.1
The activeColor property no apply in the Tab text (Android).
Regards,

Same problem here. What I found is that the if/else condition I believe is wrong: https://github.com/react-navigation/material-bottom-tabs/blob/master/src/navigators/createMaterialBottomTabNavigator.js#L44
For now @erbud you can use: activeTintColorLight and inactiveTintColorLight
If any moderator can confirm my assumption I could work in a fix PR.
Hope it helps

All right. Thx bro!

I was struggling with color apply on my Icon.
activeTinitColor, inactiveTintColor works. Thx!


Improve Error classifier logic

Originally posted by @tozka in https://github.com/vmware/versatile-data-kit/pull/577#discussion_r758794539
I think we are missing test cases for error classifying logic :
Possible stack traces (last one is assumed to be the exception)
1. vdk.internal
2. job.module
3. vdk.internal (job_input.execute_query for example raises exception)

Should be platform error, right? That's already covered by PLATFORM_ERROR_STACKTRACE test
1. vdk.internal
2. external library (sqlalchemy, pandas)
3. job.module (called by the external library)

This is covered by USER_ERROR_STACKTRACE
1. vdk.internal
2. job.module
3. vdk.internal (job_input)
4. external_library

Should be platform error . We should add a test. I suspect it will pass.
1. vdk.internal
2. job.module
3. vdk.internal (job_input)
4. job.module

This is mainly possible in send_xx_for_ingestion as they pass (user provided) iterator which can fail  .
Should be user error.  I suspect the detection won't work.
1. vdk.internal
2. job.module
3. vdk.interanl (job_input.ingest)
4. external_library_provided_by_job

Should be user error similar to above but the issue here is harder to detect because we do not know if external_library_provided_by_job is provided by job or by vdk.
Can you think of other scenarios?
Originally posted by @tozka in https://github.com/vmware/versatile-data-kit/pull/577#discussion_r758794539

Triaged. Closed as stale.


perf: Faster singleton SiblingSubgraph construction

Wraps the topoConvexChecker used by SiblingSubgraph in a OnceCell so we can delay initialization until it's needed.
This lets us avoid traversing the graph (and computing a topological order) when creating sibling subgraphs with zero or one nodes.
This partially helps with #1652.
See benchmark results:
$ critcmp before after -f '.*subgraph'
group                      after                                  before
-----                      -----                                  ------
singleton_subgraph/10      1.00      3.0±0.08µs        ? ?/sec    2.71      8.1±0.18µs        ? ?/sec
singleton_subgraph/100     1.00      4.8±0.07µs        ? ?/sec    9.81     47.4±1.16µs        ? ?/sec
singleton_subgraph/1000    1.00     23.2±1.86µs        ? ?/sec    18.94  439.3±17.04µs        ? ?/sec
multinode_subgraph/10      1.01     17.9±0.25µs        ? ?/sec    1.00     17.7±0.29µs        ? ?/sec
multinode_subgraph/100     1.01    170.0±3.72µs        ? ?/sec    1.00    168.5±3.04µs        ? ?/sec
multinode_subgraph/1000    1.02      2.4±0.02ms        ? ?/sec    1.00      2.3±0.04ms        ? ?/sec

singleton_subgraph creates a single-node subgraph in a region with around k * 3 nodes.
multinode_subgraph creates a subgraph with 1/3rd of the nodes for the same region.
This PR is quite noisy since it's adding those two new benchmarks, and tidying up the files in the process.
drive-by: Add bench = false for all the targets in the workspace. Otherwise, the auto-added test harness threw errors when passing criterion flags to cargo bench.

Uh, looks like I introduced a breaking change -.-
(TopoConvexChecker is no longer Sync now that it uses an OnceCell internally...)


Correct the srclang attribute in the example for Displaying video text tracks in the Video and Audio Content article

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

Exactly here, in the example use of WebVTT subtitles, the <track> element has the srclang attribute set to "en", but in the results, the subtitles are in Spanish. The srclang attribute should be set to Spanish.

@SnakeWhisperer that's a good point; I've updated the en instances in the <track> element to es.


Crash Report

Hi - my PhantomJS instance crashed with this crash report (please rename extension to .dmp). Can you please help?

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.


fixed typo, duplicates and added new colleges.

Good job @JotiradityaB
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

https://www.clueless.tech/profile/admin/102620336485783408752
@nikhil25803
Thanks a lott.. it was a great experience for first timers like me

https://www.clueless.tech/profile/admin/102620336485783408752
@nikhil25803
Thanks a lott to the community.. it was a great experience for first timers


fix: minor code nit

Pretty useless functionally but still manual implementation of a transpose, is redundant. Hope no one minds this. :P

LGTM!


BUG: Fix crystalball.entropy(2, 3) == array(-inf)

See #13602

Reference issue
Closes #13602
What does this implement/fix?
Sometimes _lib._utils._lazywhere(cond, arrays, ...) is called with the shape of cond not the same as the broadcast shape of arrays, and the np.extract fails. This change should be safe.
Additional information
There is no error message, maybe because it happens it happens inside of a numerical integration routine and the error is ignored. _lazywhere is called like:
 _lazywhere(x > -beta, (x, beta, m), f=rhs, f2=lhs)

x> -beta results in cond having a shape of np.broadcast(x, beta), which is not necessarily the same as the shape of np.broadcast(x, beta, m).

I don't think this routine is ever right. For example,  cb.entropy(.3, 2) produces a integration warning:
/usr/lib64/python3.9/site-packages/scipy/stats/_distn_infrastructure.py:2444: IntegrationWarning: The algorithm does not converge.  Roundoff error is detected
in the extrapolation table.  It is assumed that the requested tolerance
cannot be achieved, and that the returned result (if full_output = 1) is
the best which can be obtained.
h = integrate.quad(integ, _a, _b)[0]
array(-24455349.92208048)

Merged in PR [#13611]


More descriptive error in findParent for Epic

LGTM
Related to #11 @merrickjeff would having these error message have helped you?

Yes, I think the updated error message would have led us in the right direction. Thanks


Can't get download links :(

I test this package with Laravel, so my code was:
Route::get('/', function () {
	$downloader = new Irazasyed\VideoDownloader\Factory('facebook');
	$videoInfo = $downloader->getVideoInfo('1562054907145592');
	dd($videoInfo);
});

I got this error:

(1/1) ErrorException
Undefined offset: 1
in Facebook.php (line 25)

I open Facebook.php file and try to change this line:
$id = $matches[1];
to
$id = $matches[0];
then i got this result :

array:6 [▼
"title" => "Facebook - Today is Friends Day! To celebrate, we invited..."
"description" => false
"owner" => false
"created_time" => "1486220403"
"hd_link" => false
"sd_link" => false
]

did this package still working ??
any solution please
and thanks in advance

I fix it by changing the regex to => '/'.$key.':"([^"]+)"/'
:smile:

I fix it by changing the regex to => '/'.$key.':"([^"]+)"/'

Same issue, thanks a lot!


[thunder] Use the locale module to display distance

This PR changes the default units setting to use the locale module, based on a suggestion by @halemmerich. This will break the units if anyone is already using the app and has changed the settings but it's still a new app, which is unlikely to be widely used, and it's even more unlikely that anyone has updated the settings, so hopefully that's ok!

LGTM, I don't think the default change is a worry :)


Huge memory consumption during opening big files in Google Drive

Hi
I tried to open a big .avi file (~400MB) from Google Drive and the Mounter ate a lot of memory during this operation (~1700MB). I'd even was not able to view this video file. Here is the screenshot of the Task Manager:

Hi @psxvoid,
a file size of ~400MB sounds a bit excessive - actually the largest size I've been able to transfer in any reasonable amount of time was 250MB.
Nevertheless you've touched onto something here. Due to the requirement of determining the unencrypted content size of cloud files during read operations and the fact that some cloud gateways return non-seekable Streams, DokanCloudFS resorts to buffering the content of encrypted files. This is likely the reason for the substantial memory usage that you observed.
I will try to see if I can avoid the present buffering behavior in a future version of DokanCloudFS.


underscore in script filename, convert run arguments to flags, +

Some polishing, most importantly:

use an underscore in script filename
use a NOTICE file for licensing information
convert run arguments to flags

Haha no worries, we can wait until Wednesday and until everyone had the chance to take a look at the PR.


Popups not rendering on demand

I am unable to get Popups to show correctly when using the DeckGL onClick handler. See this snippet below... I will work on putting this into demo repo soon.
//Assumes it is wrapped in a MapContainer
export const Issue: React.FC = memo(() => {
  const { layers } = useLayers();

  const [selected, setSelected] = useState<
    { feature: GeoJSON.Feature; position: [number, number] } | undefined
  >(undefined);

  return (
      <Pane
        name={"Analysis Pane"}
        style={{
          zIndex: 400,
        }}
      >
        <LeafletDeck
          layers={layers}
          onClick={(info) => {
            setSelected(
              info.object && info.coordinate
                ? {
                    feature: info.object,
                    position: [info.coordinate[1], info.coordinate[0]],
                  }
                : undefined,
            );
          }}
        />
      </Pane>
      <Pane
        name={"Popup Pane"}
        style={{
          zIndex: 500,
        }}
      >
        {selected && <IssuePopup selected={selected} />}
      </Pane>
    </div>
  );
});

const IssuePopup: React.FC<{
  selected: { feature: GeoJSON.Feature; position: [number, number] };
}> = ({ selected: object }) => {
  const ref = useRef<L.Popup>(null);
  useEffect(() => {
    if (ref.current) {
      ref.current.openPopup();
    }
  }, [ref, object]);
  //FIXME: OPEN THE POPUP IMMEDIATELY WHEN RENDERED
  return (
    <Popup position={object.position} ref={ref}>
      Hello... NOT SHOWING
    </Popup>
  );
};

Hi @charlieforward9, thanks for investigating. This package has been deprecated in favour of @deck.gl-community/leaflet. Is there anything you can contribute there?


[util] Fix enum description in register table

Allow to not close the cell.
This makes it possible for the following structures to be included in
the same element.
Also remove an additional closing tag.
Currently it might look like this:

With the fix:

I am not sure if that is the best looking solution, but it fixes the broken end of the tables.

@rswarbrick I just saw your PR #5283 At first glance our PRs will not conflict? But would be nice to have your input here.

Hi Tobias! Thanks very much for noticing this bug in my code, and for suggesting a fix. I'm not a huge fan of the "maybe close the ; maybe not" logic though, so I took a closer look. I'm just open #5308 with what I think is a slightly cleaner fix. Would you mind taking a look?

Thanks Rupert for providing a better fix.
I was actually thinking about that my solution is not very nice, like what happens if something else then enum will be appended, it will just get messier. Sporadic thought was that there should be a table builder library, but that seemed like an overkill.


Some SSO-SAML improvements

[x] All password related fields should be invisble for users with a saml_id

[x] Password field in user details (edit)
[x] Password reset in "my account"
[x] Set password as superadmin
[x] Bulk update for password (three-dot-menu - "password reset to the default ones"; "Generate new passwords"), if possible to detect if all selected users have a saml_id.

[x] Show saml_id in account/participant list and detail pages
[x] Simplify login page if SAML is enabled

[x] The SAML login button should be more present (first element)
[x] The login with username an password should be wrapped in a collapsible box (collapsed by default)

[ ] The SAML config fields should get more descriptive labels and some helptext.

I refrained from showing the saml_id in the participant list since that view doesn't show the username separately either, but it is visible in account list and -detail, as well as in the participant detail


Bump MSRV to 1.53.0 and re-enable macos MSRV check

To solve the issue mentioned https://github.com/rust-lang/cc-rs/pull/889#issuecomment-1788887737

Oops, it looks like 1.53 still has problem with linking on latest macos.
Will revert that change.

Hm, do you know which version fixes it?

Hm, do you know which version fixes it?

Probably 1.56 or 1.58, had the same issue in https://github.com/cargo-bins/jobslot , however even 1.56 might be too new for some users (@dot-asm ).
Given that there's only one line of cfg!(target_os = "macos") which does not conditionally pull in new code but instead evalutes to a boolean, I think it's fine not checking on macos.


Fixing GDFA: Check that no new points are added

Fixing #1829
Also removed unused import.

Added unittest for the patch using examples from #1829

Thanks @alvations


Pipeline Details page: Settings

Mockups
Triggers

Described in #145

Specifications

In Pipeline Details page, clicking on Pipeline Settings link, will open the Pipeline Settings popup described in #145
The Pipeline Settings popup will have the same functionality described in #145, #218 and #219
Clicking Done button in Pipeline Settings popup, during create pipeline process will open the Pipeline Details screen. In the rest of the scenarios, clicking Done button will close the Settings popup

This is working as expected

@lcc1289 There are some bugs here:

Working partially as expected

Working as expected


[Currency Bot] Bumped patch/minor dependencies

Tada!

closing this for now, will reconsider after v4 release.


Failed to build DeepMatch with Programmer Studio <IP_ADDRESS>-preview

Hi,
Recently, we want to run DeepMatch on our Netronome Agilio SmartNIC, but we failed.
When we build the project of deepmatch according to READEME, the Programmer Studio reports us several errors like "struct 'pktio_meta' has no field 'xxx'", e.g., "plugin.c(302): error :  struct 'pktio_meta' has no field 'pkt_buf'".
We are really new to the development of Netronome SmartNICs and not able to figure out the root cause of this compilation error. Do you know the reason of that?

Hi, The README states "we used version <IP_ADDRESS> build 3241" of the SDK. Please grep your Netronome SDK source to find that structure and see how it is defined -- does it include the pkt_buf field (it may be in a file called pktio.h)?

Hi, Joel. Thanks for your reply. Acutally, I have checked the declaration of 'struct pktio_meta' in the pktio.h, and there are reallly no such fields. In my SDK (version <IP_ADDRESS>), 'struct pktio_meta" is defined as follow.
struct pktio_meta {
    union {
        struct {
            struct nbi_meta_pkt_info p_nbi;     /**< Std NBI-formatted metadata
                                                  *  accessor #defines below */

            unsigned int p_ctm_size:2;          /**< CTM buffer encoded size */
            unsigned int p_orig_len:14;         /**< Original RX pkt length */
            unsigned int p_offset:16;           /**< Offset of data in buffer*/

            unsigned int p_src:16;              /**< Source port */
            unsigned int p_dst:16;              /**< Destination port */

            unsigned int p_seq:16;              /**< Sequence number */
            unsigned int p_ro_ctx:5;            /**< Reorder context mapped
                                                  *  from NBI or NFD seqr */
            unsigned int p_is_gro_seq:1;        /**< Is pkt to be reordered
                                                  *  with GRO? */
            unsigned int p_rx_ipv4_csum_ok:1;   /**< Set if IP csum was ok */
            unsigned int p_rx_ipv4_csum_present:1;/**< Set if IP csum present*/
            unsigned int p_rx_l4_csum_ok:1;     /**< Set if L4 csum was ok */
            unsigned int p_rx_l4_csum_present:1;/**< Set if L4 csum present */
            unsigned int p_rx_l4_tcp:1;         /**< Set if L4 is TCP */
            unsigned int p_rx_mac_matched:1;    /**< Dst MAC match port MAC? */
#ifdef MAC_EGRESS_PREPEND_ENABLE
            /* The CSUM Tx flags serves two purposes :
             * 1. For packets received from the host those flags cache the
             *    Tx CSUM flags extracted from the NFD descriptor and are
             *    later used in the Tx CSUM offload performed by the ME.
             *    When this stage is done those bits are cleared.
             * 2. For packets being sent out on the wire those flags are used
             *    to setup the MAC CSUM offload processing.
             */
            unsigned int p_tunnel:1;            /**< A tunnel packet */
            unsigned int resv4:1;               /**< Reserved */
            unsigned int p_tx_l3_csum:1;        /**< Req L3 csum TX offload */
            unsigned int p_tx_l4_csum:1;        /**< Req L4 csum TX offload */
#else
            unsigned int p_tunnel:1;            /**< A tunnel packet */
            unsigned int resv4:3;               /**< Reserved */
#endif

            union {
#ifdef PKTIO_LSO_ENABLED                        /**< LSO if host packet */
                struct {
                    unsigned int p_tx_lso:1;    /**< Pkt was PCIe LSO pkt */
                    unsigned int p_tx_lso_end:1;/**< Pkt is last in series*/
                    unsigned int resv2:6;       /**< Reserved */
                    unsigned int p_tx_lso_seq:8;/**< Pkt num in LSO sequence */
                    unsigned int p_tx_mss:14;   /**< MSS of TCP */
                    unsigned int resv1:2;       /**< Reserved */
                };
#endif
#if NBI_PKT_PREPEND_BYTES >= 8                  /**< timestamp if NBI packet */
                unsigned int p_timestamp:32;    /**< MAC timestamp */
#endif
            };

#ifdef PKTIO_VLAN_ENABLED
            unsigned int p_vlan:16;             /**< VLAN id for RX and TX VLAN
                                                  *  offloading */
            unsigned int resv3:16;              /**< Reserved */
#endif
        };

        uint32_t __raw[PKTIO_NBI_META_LW];
    };
};

It seems that Netronome has changed their some APIs in the latest SDK?

Yes, the API changed. Someone else tried installing with <IP_ADDRESS>-preview and ran into the same compiler error. They reverted to <IP_ADDRESS> and the install worked. Are you able to contact Netronome to obtain the compatible SDK version?
https://help.netronome.com/support/home

Yes, I have submitted a ticket to apply for an older stable version. However, Netronome has not replied to me...

Hi, Joel. Netronome has still not replied to me for my request for an older SDK. Can you send me a compatible SDK version?

Hi, Joel. Netronome has still not replied to me for my request for an older SDK. Can you send me a compatible SDK version?

Hi, Joel. We can only install the <IP_ADDRESS> version of SDK. However, when we compile DeepMatch with <IP_ADDRESS>, a new error "error :  No LM for spilling. (-Qspill=3). compilation stopped" appeared ... Do you learn about this compilation error?

Hi, Joel. We can only install the <IP_ADDRESS> version of SDK. However, when we compile DeepMatch with <IP_ADDRESS>, a new error "error :  No LM for spilling. (-Qspill=3). compilation stopped" appeared ... Do you learn about this compilation error?

Has Netronome responded about the SDK? The new error could be due to your build settings? You need reduced threads mode and to turn off/reduce features that are not needed, like debugging and multicast groups.

Has Netronome responded about the SDK? The new error could be due to your build settings? You need reduced threads mode and to turn off/reduce features that are not needed, like debugging and multicast groups.

Note that Netronome has made SDK 6.0.3 available upon request. You must send them an explicit request for it.


'kafka_enabled' parameter is ignored for Eventhub Namespaces

Package Name: azure-mgmt-eventhub==10.1.0
Package Version: 10.1.0
Operating System: Debian bullseye
Python Version: 3.10

Describe the bug
Trying to create an Eventhub Namespace with kafka_enabled=False creates an Eventhub Namspace where kafka is enabled.
To Reproduce
from azure.mgmt.eventhub import EventHubManagementClient
from azure.mgmt.eventhub.models import EHNamespace, Sku
from azure.identity import DefaultAzureCredential

params = EHNamespace(location="<location>", sku=Sku(name="Standard"), kafka_enabled=False)

with EventHubManagementClient(credential=DefaultAzureCredential(), subscription_id="<subscription-id") as client:
      r = client.namespaces.begin_create_or_update("my-resource-group", "my-eventhub-ns", params)
      r.result()

Expected behavior
Trying to create an Eventhub Namespace with kafka_enabled=False creates an Eventhub Namspace where kafka is disabled.
Additional context
I als tried the v2022_01_01_preview API version but it did not word either.

Thanks for the feedback, we’ll investigate asap.

Hi @philipp-sontag-by
According to my investigation, this issue is a service problem.
Because the SDK is generated by swagger, the SDK is responsible for processing the quantity returned by the service and displaying it.
I suggest you go to the rest and resubmit your issue

Sure thanks for the help


feat(BUX-754): PATCH endpoint for unconfirmed contact

Pull Request Checklist

[x] 📖 I created my PR using provided  : CODE_STANDARDS
[x] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
[x] 🏠 I tested my changes locally.
[ ] ✅ I have provided tests for my changes.
[x] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[x] 💾 PR was issued based on the Github or Jira issue.

Codecov Report
Attention: Patch coverage is 41.17647% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 53.14%. Comparing base (43cef0c) to head (98d8d65).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #552      +/-   ##
==========================================
- Coverage   53.17%   53.14%   -0.03%
==========================================
  Files         194      195       +1
  Lines       10896    10930      +34
==========================================
+ Hits         5794     5809      +15
- Misses       4553     4569      +16
- Partials      549      552       +3

Flag
Coverage Δ

unittests
53.14% <41.17%> (-0.03%)
:arrow_down:

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

Files
Coverage Δ

actions/contacts/routes.go
100.00% <100.00%> (ø)

engine/model_contact.go
81.53% <100.00%> (+0.73%)
:arrow_up:

engine/action_contact.go
63.63% <57.14%> (+0.06%)
:arrow_up:

actions/contacts/unconfirm.go
0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

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

testing


Secure the WPS-T User Interface

Pending group discussion: secure the WPS-T UI such that if a user uses a browser to access the UI:
o if the user is not authenticated, they will be redirected to the Cognito authentication page
o if the user is authenticated, they will be abe to access the UI
This will involve adding Javascript code to the WPS-T UI.

There is no WPS-T UI - is this ticket perhaps about using the Jupyter Notebook with authentication, passing credentials to the WPS-T API?


Broken Search

Search just hangs - any simple search on readthedocs just hangs: http://docs.readthedocs.org/en/latest/search.html?q=version&check_keywords=yes&area=default#
This is also happening in my docs project (docs.smartthings.com).

I'm seeing this too, it appears to be broken on all the sites I've tried. I think it is related to a JavaScript error, I'm seeing this error:
Uncaught TypeError: Cannot read property 'waypoint' of null

That happens here: https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/core/static/core/js/readthedocs-doc-embed.js#L242

Yes, I see a recent commit merged in changes that should avoid that error, but looks like RTD hasn't been rebuilt with that change?

Possibly, I'm not sure. I had a look at the recent changes and I couldn't see anything obviously related but I don't know the code well at all.

It looks like this commit would fix it: https://github.com/rtfd/readthedocs.org/commit/2cc53f31eed576106856080e57a3752fd8e0a50f
Specifically, the change to readthedocs-doc-embed.js. I don't know how RTD is built, so not sure why we're still seeing this issue. Broken search for this many days, on RTD itself and most sites using it... is not good!

If that does indeed fix it, I'd guess a deploy is needed. Given that PyCon.us just happened, I expect much of the team have been busy.

Yes, we're a bit behind after PyCon, sorry for not chiming in on this sooner. A fix is deployed for search, I'm working on a more permanent change this morning.

Much appreciated, thanks very much!

Calling this resolved, the underlying cause was the above changeset merged with https://github.com/rtfd/readthedocs.org/commit/8313d3108c101798593268228506003bf40e2aa5

@agjohnson just for clarification, should search work again already or does the fix still need to be deployed? currently, http://docs.readthedocs.org/en/latest/search.html?q=version&check_keywords=yes&area=default# does not seem to work yet. thanks!

@rened That link works for me. Perhaps a caching issue?

ha, you are right - didn't occur to me that that could be an issue. thx!


Run Playwright with SST

Is it possible to run tests created with Playwright to run it in an SST environment?
I have seen example for vitest for v2, but I'm using v3 and and I would prefer e2e tests.
Thank you!

You should be able to run pnpm sst shell -- <your playwright command> and it will inject links for you.

Thank you @arpadgabor!
Looks like the sst dev (live) server is not launched when running sst shell

I was able to get it working by running sst dev separately and the running npx sst shell -- playwright test
However, did anyone tried running a similar setup in CI/CD? Github Actions is not really designed to run parallel commands.

You should probably just run Playwright on a deployed environment in CI tbh

Makes sense, thank you!


npm ERR! Error: EACCES: permission denied error during azure/webapps-deploy@v2

Hi,
We use this workflow for our Azure deployment, and suddenly, the deployments started failing with this permissions error:
npm ERR! code EACCES
npm ERR! syscall access
npm ERR! path /opt/nodejs/14.19.1/lib/node_modules/npm/node_modules/abbrev
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, access '/opt/nodejs/14.19.1/lib/node_modules/npm/node_modules/abbrev'
npm ERR!  [Error: EACCES: permission denied, access '/opt/nodejs/14.19.1/lib/node_modules/npm/node_modules/abbrev'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/opt/nodejs/14.19.1/lib/node_modules/npm/node_modules/abbrev'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

This was working perfectly fine a couple weeks back, but suddenly errors out with this error whenever we try to deploy. Happy to share anything I can to get this fixed. It seems like an odd error to be getting during deployment, especially because npm install has already been run as a pre-step to this (successfully).
This is a particularly critical failure for us, as it's preventing us from deploying a bug fix in production.
This is our full github actions workflow yml that is failing:
# Docs for the Azure Web Apps Deploy action: https://github.com/Azure/webapps-deploy
# More GitHub Actions for Azure: https://github.com/Azure/actions

name: Build and Deploy

on:
  pull_request:
    branches:
      - production
    types: [closed]
  workflow_dispatch:

jobs:
  build:
    if: github.event.pull_request.merged == true
    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v2

    - name: Set up Node.js version
      uses: actions/setup-node@v1
      with:
        node-version: '14.x'

    - name: npm install, build, and test
      run: |
        npm install
        npm run compile
        npm run test --if-present

    - name: Compress files as zip
      run: tar cf archive.tar .

    - name: Upload artifact for deployment job
      uses: actions/upload-artifact@v2
      with:
        name: node-app
        path: archive.tar

  deploy:
    runs-on: ubuntu-latest
    needs: build
    environment:
      name: 'production'
      url: ${{ steps.deploy-to-webapp.outputs.webapp-url }}

    steps:
    - name: Download artifact from build job
      uses: actions/download-artifact@v2
      with:
        name: node-app

    - name: Unzip compressed files
      run: tar xf archive.tar

    - name: 'Deploy to Azure Web App'
      id: deploy-to-webapp
      uses: azure/webapps-deploy@v2
      with:
        app-name: 'edify-safetymojo-api-sandbox'
        slot-name: 'production'
        publish-profile: ${{ <removed secret> }}
        package: .

@elleklinton can please you check if the WEBSITE_RUN_FROM_PACKAGE is set to 1? Please make sure the app setting: SCM_DO_BUILD_DURING_DEPLOYMENT is not present?

@elleklinton Were you able to resolve the issue by changing the app settings?

Closing this, do create new issue or re-open if the issue persists.


Add slash commands to bot

Add slash command functionality to the bot.

discmoji/bot.py

Add a new attribute _all_slash_cmds to store all registered slash commands.
Add a new method slash_command to register slash commands.
Add a new method register_slash_commands to register all slash commands with Discord.
Modify the connect method to call register_slash_commands.

discmoji/command.py

Add a new class SlashCommand to represent a slash command.
Add a new method error to the SlashCommand class to handle errors.

discmoji/context.py

Add a new method send_slash_message to the Invoked class to send messages for slash commands.

hmm.. i'll think about uploading this into optimization branch.

(TY to @AlmostDemoPy) This pr needs improvement because it has stuff that clogs the main bot class with more command stuff, instead there should probably be a manager for the classes.

My opinion is the same as @almostDemoPy's... this should be in a separate place.


Passing parameters to a flow in powerapps

I have a flow that sends image data, filename, and two values to update meta data columns in a sharepoint document folder.
 The question I require answer to is where do I find the parameter list for the flow?
You document uses this example "FlowInApp.Run(TextInput1.Text)" and the word parameters is not found on the page.
My code view is here but there is no place within powerapps where I can see the required parameters that it is counting and telling me I am supplying too many. What is the answer?
{
"kind": "PowerApp",
"inputs": {
"schema": {
"type": "object",
"properties": {
"Createfile_FileName": {
"type": "string",
"description": "Name of the file.",
"isPartial": false
},
"Updatefileproperties_shipping_list_number": {
"type": "number"
},
"Updatefileproperties_line_number": {
"type": "number"
}
},
"required": [
"Createfile_FileName",
"Updatefileproperties_shipping_list_number",
"Updatefileproperties_line_number"
]
}
},
"metadata": {
"operationMetadataId": "c48e30d6-3163-4441-a359-a59298e2b156"
}
}

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

ID: bb0f4c87-49bd-5cbd-f336-d1cae937f23c
Version Independent ID: bb0f4c87-49bd-5cbd-f336-d1cae937f23c
Content: Use Power Automate pane - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/working-with-flows.md
Service: powerapps
Sub-service: canvas-maker
GitHub Login: @TashasEv
Microsoft Alias: tashas

Thanks for your comment, @shinyion. This issue has been assigned to the doc author for review.


'Report an Imagery Problem' link broken

The link for 'Report an Imagery Problem' in the background panel of the Preview.ideditor.com doesn't link to an existing page
https://github.com/openstreetmap/iD/blob/master/FAQ.md#how-can-i-report-an-issue-with-background-imagery

This has already been fixed in the develop branch.


Docs: Add missing NPM script section to Vue guide

It seems that this necessary setup step didn't get copy-pasted over from the React guide (probably because it isn't a standalone section in the React guide).

Thank you very much @JanCVanB ! I will deploy a new docs version now!

Deployed!
https://storybook.js.org/basics/guide-vue/#create-the-npm-script

Thanks @ndelangen!


Ability to change colors of various items in editor

The blue background, the meter lines, the trajectories (and selected trajectories), and the spectrogram and melograph.

started work on a branch to explore at least the spectrogram side of this.


retryWhen operator

I want to implement back-off strategy delay and need retryWhen but rxcpp doesn't have retryWhen.
any idea ?

Hi!
I would try using switch_on_error() | repeat()
Switch on error can return emtpy() to retry immediately and timer() to retry later and error() to stop with the latest exception.

Hi!! Just to complement the suggetions of @solvexyz I'm new using this awesome library, I did a sample how to implement the fix:
observable code ...
.on_error_resume_next([&](std::exception_ptr ep) {
            printf("Wait 2 seconds ------------------------\n");
            return rxcpp::observable<>::timer(std::chrono::milliseconds(2000))
                .flat_map([&](int) {
                    printf("Returning seconds ------------------------\n");
                    return rxcpp::observable<>::error<int>(std::runtime_error("Error from source"));
            });
        })
.retry()
....

leaving this here for posterity
template<typename ObservableT, typename DurationT>
auto retryWhen(ObservableT o, std::chrono::duration<DurationT> time_between_retries, int max_retries) {
    return o.on_error_resume_next([=](std::exception_ptr /*unused*/) {
                return rxcpp::observable<>::timer(time_between_retries)
                        .flat_map([](int) {
                            return rxcpp::observable<>::error<typename ObservableT::value_type>(
                                    std::runtime_error("Error from source"));
                        });
            })
            .retry(max_retries);
}


Support exporting jar

Add 'getMainMethod' command in jdtls.ext, which will return all the existing main methods of the workspace.

Hi, this seams great.
Can you briefly explain how this exporting to a jar will work from a user perspective?

Any ETA on this? Don't mean to push, I think you guys are doing an amazing
job. It's just that I can't wait to try this out :)
Also a question. How will manifest files be provided?
Op do 23 jul. 2020 om 09:22 schreef Sheng Chen<EMAIL_ADDRESS>
@jdneo commented on this pull request.
In src/exportJarFileCommand.ts
https://github.com/microsoft/vscode-java-dependency/pull/271#discussion_r459254840
:

+import { CancellationToken, commands, Extension, extensions, ProgressLocation,

    QuickInputButtons, QuickPick, QuickPickItem, Uri, window, workspace } from "vscode";

+import { buildWorkspace } from "./build";
+import { isStandardServerReady } from "./extension";
+import { Jdtls } from "./java/jdtls";
+import { INodeData } from "./java/nodeData";
+import { WorkspaceNode } from "./views/workspaceNode";
+
+enum ExportSteps {

ResolveProject = "RESOLVEPROJECT",
ResolveMainMethod = "RESOLVEMAINMETHOD",
GenerateJar = "GENERATEJAR",
Finish = "FINISH",
+}

+export async function createJarFile(node?: INodeData) {
I observed some UX issues here:

The status report in the status bar looks strange to me. Because at
this point, there is no pending requests actually, but it showing the
spinning icon.
[image: image]
https://user-images.githubusercontent.com/6193897/88260397-e0936a80-ccf6-11ea-9c05-ae70d6e22254.png
2.
When you are in the middle of the command process, if you click export
jar again, it will compile the workspace, which seems not correct
3.
When I try to export jar for spring pet clinic, it takes very long
time to resolve the main method.(actually it never ends) why?
4.
No need to report error when user canceled
[image: image]
https://user-images.githubusercontent.com/6193897/88260740-952d8c00-ccf7-11ea-9e5a-c2be3e783a5a.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-java-dependency/pull/271#pullrequestreview-453882603,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFUKWSY34VAK6W6BMWCLYTR47QKLANCNFSM4OFL7SEA
.

Any ETA on this? Don't mean to push, I think you guys are doing an amazing job. It's just that I can't wait to try this out :) Also a question. How will manifest files be provided? Op do 23 jul. 2020 om 09:22 schreef Sheng Chen<EMAIL_ADDRESS>
We have to do further adjusting and debugging, you can also see a lot of comments here. I think this PR may be merged in the next week. For your question on manifest, we may export a jar with a manifest including the main-class property with its value that is chosen by the user.


chore: Update docker dep to fix CVE-2024-41110

Updates github.com/docker/docker to address CVE-2024-41110 targeting Scanner 2.33.6 and Stackrox 4.4.5
Expecting scanner version 2.33.6 to contain this change and be used in this backport: https://github.com/stackrox/stackrox/pull/12422

Also, you may have to rebase? I actually just cherry-picked https://github.com/stackrox/scanner/commit/d4b374d5d30dc4249c27ea6619c5b28a1df45c35 into this branch, and it modified the go.mod/go.sum

LGTM, but something to think about: I'm pretty confident we are not affected by this. I think this just affects the docker engine runtime, which we aren't running/using.

Agreed, I'm sure some scanners would report it regardless, this would at least eliminate some future false positives / headaches.

Also, you may have to rebase? I actually just cherry-picked https://github.com/stackrox/scanner/commit/d4b374d5d30dc4249c27ea6619c5b28a1df45c35 into this branch, and it modified the go.mod/go.sum

ACK - right now seems OK

Test failure unrelated to change - tests were fixed in this PR https://github.com/stackrox/scanner/pull/1567 (has not been backported to this branch)


Prevent unplugin-vue-components from removing declarations

Clear and concise description of the problem
The issue that I have is, (which could possibly be considered a bug):
Components are removed often from the dts file. It's not because they are being removed from the app, but unplugin-vue-components removes them from the dts, until they are stumbled on in dev. When building it gets each component and puts them in dts, which is great, however, when running dev, for some reason, these components get removed from the dts file.
This would normally not be an issue, however, the components.d.ts file says: "// We suggest you to commit this file into source control".
So, the big issue is, this file seemingly is always changing, even when it really shouldn't be changing. Leading to me always needing to commit a file that doesn't need to be changed in the first place.
Suggested solution
The above could be considered a "bug", but one form of solution could be an option that does not remove dts entries...
So perhaps
// vite.config.ts
plugins: [
  Components({
    noDtsAutoRemove: true // default false
  })
]

When that new option is turned on, it will not remove entries from the components.d.ts file.
To add more complexity, the option could be modified to only remove/shake off old components during build. But the option name I came up with would be a bit unusual if it did this. So, in this alternative, during build it will remove unused components, but during dev it will only add components.
Alternative
Most likely what I intend to do is to remove the components.d.ts from source control entirely. It doesn't serve a huge amount of use to the team using the project... any issues are fixed when another team member simply runs their dev server.
Alternatively, it would be equally nice if when using a resolver, that it just added all possible component registers to the dts, regardless if they are used or not in the app. This could be preferable, but doesn't solve the root cause of the issue above. I personally would enjoy this second alternative as a standalone feature. This would be similar to the existing "types" option, but without the "names" property for specific type imports, instead it just imports everything... Though, this seems "difficult" to implement, considering the vast options of component libraries, perhaps somehow built-in resolvers could support this, though they would likely need an update
Additional context
It should be noted, I am on Vue 2.7, using the Vite plugin, and am using the VuetifyResolver. My main issue is that -- even though the components are USED IN THE APP, they are being removed in the dts file during dev.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

please show me your plugin version

i dont see the  unplugin-vue-components delete dts, and dts alawys increases.

This is a major issue for me as well. Removing it from source control is also probably our likely solution. Curiously, the comment in the file that recommended committing it to source control seems to have been removed. Perhaps that guidance was removed for this exact issue?

The vite dev server deletes and recreates the components.d.ts file content multiple times during startup and recompilation. It seems that removing the file from source control is the only solution


Add functionality equivalent for "cjs-module-lexer"

https://github.com/nodejs/cjs-module-lexer
Basically we need a function, that given a CJS module source will return a list of exports and reexports.
We need this functionality for https://github.com/denoland/deno/issues/12648 and https://github.com/denoland/deno/issues/13200

Implemented in #55


Artifactory Nuget build task not setting build.name and build.number properties on push

Is your feature request related to a problem? Please describe.
Not sure if this is a feature request or an actual bug.  Issue is that pushed artifacts need build.name and build.number properties set for proper repo path for artifacts to be available from artifactory build screen.
Describe the solution you'd like to see
Would like to see build.name and build.number property fields set on artifacts in addition to the nuget.* properties that are being set when Capture Build is selected on 'Artifactory Nuget' task set to push.
Describe alternatives you've considered
Currently only alternative would be to manually add these properties or add it via api calls after the push task.
Additional context
Add any other context or screenshots about the feature request here.

Shows artifact properties screen of what is currently being set.  Missing build.name and build.number.

Shows build, published modules, artifact screen where repo path is not set.

Hi @tobmat ,
Thank you for reporting this issue.
Bamboo Artifactory plugin version 3.3.0 was just released and includes a fix.
We'd appreciate your feedback for it.
Thank you for your patience.

@RobiNino Thanks for the update.  We are in the process of upgrading now but I encountered another issue that I posted here.  Once that is resolved I can validate this one and give you feedback.  Thanks!
https://github.com/jfrog/bamboo-artifactory-plugin/issues/200

@RobiNino build.name and build.number propertied look good.  I will close this issue.


Fix arithmetic typo

fix arithmetic typo

FYI Line 32 below the table explains why it is not a typo but an intentional example.

FYI Line 32 below the table explains why it is not a typo but an intentional example.

Oh, I've missed that, thank you!


Added support for LSTM layers as well as Embedding layers.

Added support for LSTM layers as well as Embedding layers.
Also included a couple of activations: sigmoid, hard-sigmoid and tanh.
Some additional code to operate tensors included.
Full python export and test generators included for given layers and activations.
I also set some methods of KerasModel to virtual in order to ease Mocking during testing in my projects.

Thanks for the PR! I refactored your modifications to Tensor a little, hope you don't mind..

Thanks for accepting it, your lib was really useful for me. And about modifications to tensor, I don't mind, you are in charge of keeping the style.  ;)

hey @javierfvargas I noticed if you compile with -DDEBUG the LSTM unit tests fail, it looks like KerasLayerLSTM::Apply produces a 2 dimensional tensor whereas the unit test is expecting a 1D tensor.

Let me check it on Monday.

@moof2k I have it fixed now.  It was a mistake on how the tensor is build when the LSTM doesn't  have to return sequences.  But I also found similar problem on the embedding layer.   Fixed an tests passing with -DDEBUG
What should we do now? Another pull request?

yes please :)


Too many sockets in TIME_WAIT can cause a crash

Thanks for a great project! Stumbled into a problem with the access_point example, commit  a32d109, no modifications.
If I repeatedly connect via telnet, the free heap shrinks with ~120 bytes for each connection. Crash when reaching zero. Haven't looked into it much yet, guessing something in lwip is leaking. Or perhaps it is even expected..?

Thanks! Hope you find it useful.
I think I've seen this with TCP sockets being stuck in CLOSE_WAIT inside LWIP. If you leave it a little while (I think maybe 60 seconds) then you should find the heap usage goes back to normal.
However, it shouldn't crash in any case! It might be related to #76, in which case merging the feature/tests branch on top of current master may fix it (there's a fix for #76 in that branch.)

There's also come code in the LWIP stack that's supposed to prune TIME_WAIT sockets if memory gets tight:
https://github.com/SuperHouse/esp-lwip/blob/3cf8d514bd76e6ef77e6fa514d0ec6d96da7fd9a/src/core/tcp.c#L1288
Maybe that threshold value needs to be higher? Although again it shouldn't outright crash on OOM I would hope (but that's probably the malloc-never-fails issue causing it to allocate outside of accessible RAM.)

Confirmed - after 60 secs the memory is released. And yes, the crash is most certainly a side-effect of #76.
However, the pruning does not seem to work. Anyhow, for me this is not critical. It might be a security aspect for some if it is this easy to crash it just by connecting to it x times within a minute. I'll try this test out with the #76 fix when I can. Closing for now - thanks for the explanation.

Thanks @pellepi for confirming.
I'm going to leave this open though, as I think it's a serious enough problem we should find a fix.

As of the latest and greatest (with #76 in), this is not reproducible any longer.
The heap repeatedly shrinks, but will stop shrinking at ~8kB - the pruning seems to work. Zombie connections are still reclaimed after 60 secs. Safe to close I'd gather.
Good work!

Closing this as I've awlso confirmed it's fixed since #76.


2fa dont work

After update i cant get acces to my accounts and cant look from them.
In config
"UseAsfAsMobileAuthenticator": true,
But when i write !2fa it writes
That bot doesn't have ASF 2FA enabled!
I used prev version, but it didnt solve the problem =(

Please review the guidelines for contributing to this repository.

https://github.com/JustArchi/ArchiSteamFarm/blob/master/CONTRIBUTING.md

If my issue is not meeting contributing guidelines specified above, especially if it's a question or technical issue, please close my issue immediately - I won't complain.

In addition to that, UseAsfAsMobileAuthenticator was removed so long ago that I don't even remember in which version to be exact. Read Escrow part to learn how to use ASF 2FA.