When a container is restored, it does not join the service that has selectorContainer that matches the container's label.

Server version - v0.41.0-rc2
Steps to reproduce the problem:
Create a container - c1 with label = "foo:bar"
Create a service with selectorContainer as "foo=bar".
c1 joins this service.
Delete the container. c1 is not part of the service anymore.
Restore c1.
c1 does not join the service.
Expected Behavior:
c1 should join the service once it is restored.

Tested with server version - V0.43.0-rc2
Create a container - c1 with label = "foo:bar"
Create a service with selectorContainer as "foo=bar".
c1 joins this service.
Delete the container. c1 is not part of the service anymore.
Restore c1.
c1  join the service back.


Implement TryFromBytes for Unalign?

Should we impl TryFromBytes for Unalign<T> where T: TryFromBytes? There are a few cases, and many of them are thorny:

For T: Unaligned, we could already support this by delegating to T::is_bit_valid.
For T: !Unaligned (ie, T in the general case without an Unaligned bound), we can't delegate to T::is_bit_valid because it requires an aligned MaybeValid<T>. We could copy the Unalign<T> into the local stack frame in order to ensure its alignment and then call T::is_bit_valid on the local copy, but that could be very expensive.
If we add support for unsized T, then for T: !Unaligned, copying into the local stack frame isn't even an option.

cc @kupiakos

Done in https://github.com/google/zerocopy/pull/817.


Fix the failing clippy CI job, change the CircleCI resource class from 2xlarge to xlarge

In addition, since there was the same breakage related to us using the 2xlarge resource class in CircleCI, I'm adding a commit changing it to xlarge to see if it will suffice.

Ah sorry, I just noticed this! #651 had the same change, apologies!


[1.8] Fix incorrect rounding of resource allocation.

Before:

After:

Checklist

[ ] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

run integration test

mesosphere-ci retest this please

L👀ks good, though I think we should change the rounding to allow at least two decimals in the next iteration.

mesosphere-ci retest this please 🙏


fix: safari tooltip disabled element fixes

SUMMARY:
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[x] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@6d96102). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #309   +/-   ##
=======================================
  Coverage        ?   88.95%
=======================================
  Files           ?      396
  Lines           ?    10710
  Branches        ?     3620
=======================================
  Hits            ?     9527
  Misses          ?     1161
  Partials        ?       22

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


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_b2520a61-17a7-45dd-88b6-7007cf1c6c82/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_b2520a61-17a7-45dd-88b6-7007cf1c6c82.git

Docs Build status updates of commit bb0b79c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bb0b79c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Handful of work

@jbertram can you rebase? I have added a fix to the joram testsuite and another fix.

Rebased.


sched/sched: address performance concerns for sched_lock in non-SMP case

Summary
Address performance concerns from https://github.com/apache/nuttx/pull/7464
Impact
sched_lock/sched_unlock should work as fast as before.
Testing
Pass ostest from https://github.com/apache/nuttx-apps/pull/1711

@davids5 Please take a look if this solution is better

@pkarashchenko
Are there more cases then:

The normal case sched lock prevents preemption from interrupts  because we do not reprioritize while locked.
The case is that a call to change priority while locked must be accepted but delayed. But in fact this can only be the current task that is running and has locked.

Are there more?


support for external links

As this feature was missing, I added a wrapper, as well as a test for it.

@aldanor @mulimoen Friendly ping! This pull request is open for quite a while, but hasn't received any attention yet, so maybe one of you could take a look. The PR is very small and self-contained, and it seems to work without problems (we're using it for a project already). Thanks!

Thanks a lot for the feedback! I added the changelog entry and the link to the HDF5 documentation.


ITE: soc/power: add power management

Add the power state of deep doze. When system enters deep doze, the
clock of CPU and EC can be stopped to reduce power consumption. And
enable the UART Rx WUI before entering deep doze to wake up EC and
CPU.
Tested on it8xxx2_evb board. It will reduce 5.25mA when system enters
deep doze mode.
Signed-off-by: Tim Lin<EMAIL_ADDRESS>

This change is

CONFIG_PM=n
CONFIG_PM_POLICY_APP=n

These are the defaults, correct? I think this can be deleted.

Yes, it's default. Deleted.

dts/riscv/it8xxx2.dtsi, line 35 at r2 (raw file):

			compatible = "zephyr,power-state";
			power-state-name = "standby";
			substate-id = <1>;

substate-id isn't used by ite/common/power.c, so this can be deleted.

Done

soc/riscv/riscv-ite/common/power.c, line 68 at r2 (raw file):
Quoted 12 lines of code…
Normally this code should go into the device driver by defining a pm_action_cb function in the DEVICE_DT_INST_DEFINE macro. However, I see the IT8xxx2 uses the shared ns16550.c driver, which does not provide a power management callback.
You could create driver to handle IT8xxx2 specific UART features. For instance:
  uart1: uart@f02700 {
    compatible = "ns16550";
    /* ... */
  }
  ite_uart1_wrapper: {
    compatible = "uart_it8xxx2";
    /* ... */
  }

The uart_it8xxx2 driver could then define the pm_action_cb and this code could be moved out of this file. You can experiment with this in a separate PR.

Thank you for your information, It is currently under development and will be in another PR.

soc/riscv/riscv-ite/common/power_policy.c, line 12 at r2 (raw file):

#include <pm/policy.h>

__weak struct pm_state_info pm_policy_next_state(int32_t ticks)

This is supposed to be defined by the application. I don't think this belongs in the soc support.

It's removed.

soc/riscv/riscv-ite/it8xxx2/soc.c, line 144 at r2 (raw file):

extern volatile int wait_interrupt_fired;

ALWAYS_INLINE void riscv_idle(enum chip_pll_mode mode, unsigned int key)

ALWAYS_INLINE is only valid for static functions. If you need this to be inline, you will have to move the function body into the header file.

Ok, deleted it.


Memory increase when decoding exception occurs

Restart pipeline cause memory leak .

@szalpal This bug is confirmed by DALI issue, and it will be fixed in next release.


Update number of worker nodes screenshot, remove max

Applies to 1.3 and 1.4/master. Sending to Merlin and Ryan for approval. Thanks.

Thanks @lparis! My understanding is that there's no upper limit to the number of worker nodes unless you're deploying vSphere on NSX-T, in which case the upper limit is 250. Is this correct?

@lparis After discussion with Ryan, Jaime, and others, I've determined the following:
There is an upper limit of 250 for NSX-T, and no upper limit for vSphere w/o NSX-T or any other IaaSes, but we have only tested up to 200 workers.
We'll update the docs to reflect!

Great. Thanks for getting to the bottom of this.
From: ben tarnoff<EMAIL_ADDRESS>Sent: Thursday, February 21, 2019 12:12 PM
To: pivotal-cf/docs-pks<EMAIL_ADDRESS>Cc: Lorenzo Paris<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [pivotal-cf/docs-pks] Update number of worker nodes screenshot, remove max (#564)
@lparishttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flparis&data=02|01|lparis%40vmware.com|4404620d4abe48337fca08d69838c787|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|636863766963861470&sdata=mxQ2eSz9VWhFRlbU1b5r1YrbhX0SiJ7RrMUq1TRQaWc%3D&reserved=0 After discussion with Ryan, Jaime, and others, I've determined the following:
There is an upper limit of 250 for NSX-T, and no upper limit for vSphere w/o NSX-T or any other IaaSes, but we have only tested up to 200 workers.
We'll update the docs to reflect!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpivotal-cf%2Fdocs-pks%2Fpull%2F564%23issuecomment-466148394&data=02|01|lparis%40vmware.com|4404620d4abe48337fca08d69838c787|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|636863766963871475&sdata=WKBgqEWgAo%2B4V3P3hbDYwtQA6ibx14M6IlBWg6djNPc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJV02UUuIriixRj_-3_1IJuLNMrpOo4mks5vPv12gaJpZM4bH4L8&data=02|01|lparis%40vmware.com|4404620d4abe48337fca08d69838c787|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|636863766963881480&sdata=gad62RWphyTPs1yjweLOnDRd8TCa%2FSR3LfJ7k1vGRrM%3D&reserved=0.

The PR works for me.

Thanks @lparis!


[FEATURE] Support Speech OpenAI Audio API

This pull request introduces specific support for the speech synthesis feature of the OpenAI Audio API, directly addressing a part of the feature request #20, which was raised on March 12. These changes enable to leverage OpenAI's speech synthesis capabilities, expanding the suite of audio tools.
Key Features Introduced:

Speech API Integration: Created modules to handle requests to and from the OpenAI Audio API for generating speech. This integration includes handling constructing speech generation requests and processing responses with and without persistance.
Unit test: Include test for various use cases.
Examples and Documentation: Provided detailed examples showing how to utilize the speech synthesis API within our application. Updated the project documentation to include instructions on setting up and using the new speech features.
Error Handling: Implemented comprehensive error handling to address potential issues

@KaisNeffati thank you and sorry for the delay with the review!
@SandraAhlgrimm do you require this PR to move forward with implementing this in LangChain4j? Or do you make your own impl?


pybay 2023

Thanks @CarlFK! Two small requests:

Can you move the video JSON files into a pybay-2023/videos/ directory?
Can you add a pybay-2023/category.json file containing the following:

{
  "title": "PyBay 2023"
}

If you'd prefer, I'm happy to cherry pick these commits into my own PR with the modifications. (I would normally push this change to your branch since the maintainer edits option is checked here, but I don't want to push to a main branch instead of a feature / dedicated branch in that fashion.)

updated
On Sat, Nov 4, 2023 at 12:55 AM Jon Banafato @.***>
wrote:

Thanks @CarlFK https://github.com/CarlFK! Two small requests:

Can you move the video JSON files into a pybay-2023/videos/
directory?
Can you add a pybay-2023/category.json file containing the
following:

{
"title": "PyBay 2023"
}
If you'd prefer, I'm happy to cherry pick these commits into my own PR
with the modifications. (I would normally push this change to your branch
since the maintainer edits option is checked here, but I don't want to push
to a main branch instead of a feature / dedicated branch in that fashion.)
—
Reply to this email directly, view it on GitHub
https://github.com/pyvideo/data/pull/1140#issuecomment-1793353568, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFVOMB3MWAGD6KECUNXR3YCXKGFAVCNFSM6AAAAAA6UKQFNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGM2TGNJWHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Carl K


Nice product!

I am looking for an equivalent to Outline, I have used HedgeDoc a lot but I would like an open source tool more accessible to the public.
I quickly tested your tool and clearly it's great. The interface is clean.
Here are the features that I see to be implemented so that it is easily distributed:

Added translation
Implementation of OAuth authentication

Do you plan to continue development?

@kosssi As far as I know OAuth is supported. Take a look at https://rootspace.app/


Why not delete room or peer if they are useless

type pmap struct {
	sync.RWMutex
	id string
	m  map[string]*Peer
}
type rmap struct {
	sync.RWMutex
	m map[string]*pmap
}

I found the peer and room data in these 2 struct will never be deleted. I think this memory leak is a problem if the service running for a long time. What do you think?

There is no limit the number of elements in map in golang. It does occupy more memory if service running for a long time, but the size of a blank map is small, I think there doesn't have much effect.

There is no limit the number of elements in map in golang. It does occupy more memory if service running for a long time, but the size of a blank map is small, I think there doesn't have much effect.

Sorry, I don't think the effect is small. Below is the info of kraken deployed in my server. There're only 6 active users but 2486 inactive users. The kraken service uses almost 500MB memory. When the service is just started, the memory usage was below 100MB.
{
   "data":{
      "updated_at":"2022-11-26T04:02:16.544191438Z",
      "active_peers":6,
      "closed_peers":2486,
      "active_rooms":4,
      "closed_rooms":119
   },
   "id":"1d870674-9329-44f7-b218-e40db615f52c"
}

There is no limit the number of elements in map in golang. It does occupy more memory if service running for a long time, but the size of a blank map is small, I think there doesn't have much effect.

Sorry, I don't think the effect is small. Below is the info of kraken deployed in my server. There're only 6 active users but 2486 inactive users. The kraken service uses almost 500MB memory. When the service is just started, the memory usage was below 100MB.
The map is not blank also, all Peer struct are not released. But I'm not sure if the memory leak is caused by this.
{
   "data":{
      "updated_at":"2022-11-26T04:02:16.544191438Z",
      "active_peers":6,
      "closed_peers":2486,
      "active_rooms":4,
      "closed_rooms":119
   },
   "id":"1d870674-9329-44f7-b218-e40db615f52c"
}

Thanks, will check and fix it

I used the change in this PR https://github.com/MixinNetwork/kraken/pull/33. The service uses only 40MB memory after running for more than 12 hours. Above screenshot is also captured after the service running about 12 hours.
This change solves the biggest memory leak problems.


v1.4.0

I add a commit here so that we can merge the two change together with two pr.
This simply add the titlehighlight data


Greay border on a white default color

Hi !
We love what you did with the colorPicker but we spotted a little improvement :
When we set up a white default color, the lack of border seems like a empty space and is not well comprehended by everyone.
Could you add a grey border if the default color is white or something like that ?

Thanks for the hard work

@Vaal15 update in version 1.3.3


Edit project name

Feature request: add ability to edit the name of a project from the main menu and/or inside the project

Is this something you're interested in working on @jhea0008 ? I'm happy to add it to the backlog


Google Connector and GitHub Connector have the same ClientID and ClientSecret

Describe the bug
Google Connector and GitHub Connector show the same ClientID and ClientSecret
To Reproduce
Steps to reproduce the behavior:

Click Installed Plugins:

Click on Google Connector:

Click on Github Connector:

Expected behavior
Should show different own ClientID and ClientSecret.
Platform

Device: Desktop
OS: Windows
Browser and version: Chrome

Thanks for your feedback, this is indeed a bug and will be fixed in the next version.


Many TACL builtins and keywords are not highlighted

Not all TACL built-ins are highlighted. For example, #IF and #LOOP are highlighted but #PUSH, #RETURN , #SET, etc. are not. See your test case at https://github.com/knovichikhin/vscode_tal/blob/master/tests/sample.tacl.
Please highlight all TACL built-in commands and built-in variables. These are output by the TACL BUILTINS command.
Also, it be would be helpful if the :UTILS:TACL variables were highlighted. These are output with the TACL command VTREE :UTILS:TACL.
The builtins and commands are also listed in "Summary of Commands and Built-In Functions" in the TACL Reference Manual.
If you don't have access to a NonStop system, I'd be happy to provide these lists.

Current grammar rule assigns function scope (entity.name.function.tacl) to all keywords that start with #.
For example, this is what it looks like in default theme Monokai Dimmed.

Some special functions, such as #if or #loop are assigned a different scope (keyword.control.tacl).
So, if you don't see them highlighted then it's possible your current theme does not recognize scope entity.name.function.tacl. Do you mind checking your theme code? Or if you share your theme I can double check.
As to the second part, it's possible to add them. However, it can be tricky because the tokenizer will not be able to recognize command vs variable vs literal. But most of these should be used as stand alone commands. So, I think the best that can be done is to recognize these commands when are used at the start of a line.
For example:
#output Who Framed Roger Rabbit {simple text}
who {should be recognized as a command}

Current grammar rule assigns function scope (entity.name.function.tacl) to all keywords that start with #.
For example, this is what it looks like in default theme Monokai Dimmed.

Some special functions, such as #if or #loop are assigned a different scope (keyword.control.tacl).
So, if you don't see them highlighted then it's possible your current theme does not recognize scope entity.name.function.tacl. Do you mind checking your theme code? Or if you share your theme I can double check.
As to the second part, it's possible to add them. However, it can be tricky because the tokenizer will not be able to recognize command vs variable vs literal. But most of these should be used as stand alone commands. So, I think the best that can be done is to recognize these commands when are used at the start of a line.
For example:
#output Who Framed Roger Rabbit {simple text}
who {should be recognized as a command}

My theme was set to "Light (Visual Studio)". I switched to "(Light+) (Visual Studio)" and the highlighting appeared for all built-ins. Thanks for the tip.
As for default commands (e.g. "who"), restricting it to the start of a line (ignoring white space), would work for me. But, I can imagine it might cause grief for others. So, it's worth thinking about it. I'll close this issue for now.

My theme was set to "Light (Visual Studio)". I switched to "(Light+) (Visual Studio)" and the highlighting appeared for all built-ins. Thanks for the tip.
As for default commands (e.g. "who"), restricting it to the start of a line (ignoring white space), would work for me. But, I can imagine it might cause grief for others. So, it's worth thinking about it. I'll close this issue for now.

I added basic :UTILS:TACL handling.
In the future more specific command handling can be added tailored to the specific commands.

I added basic :UTILS:TACL handling.
In the future more specific command handling can be added tailored to the specific commands.


Add a workflow task that sends a message to topic in TIBCO

FLPATH-145 https://issues.redhat.com/browse/FLPATH-145

/retest

lgtm

/lgtm

@gciavarrini please review


Adapt readable_msisdn

adapt to also accept country code without '+' (e.g. utils.readable_msisdn("0821234567", '27'))

@gsvr @imsickofmaps please review

:+1:


5.1 error message typo "The following fields where Not found" ("where" -> "were", and capital Not)

fixed


[R4] BDD for List Shopcart Items API

* Write scenario in the .feature file
* Add web tests to web_test.py according to scenario
* Add step tests to shopcarts_test.py according to API returns and error handling

As a [role]
I need [function]
So that [benefit]
Details and Assumptions

[document what you know]

Acceptance Criteria
Given [some context]
When [action taken]
Then [outcome observed]

Hey team! Please add your planning poker estimate with Zenhub @emhao @LillisLuo @nijya @wl2484


server behind proxy doesn't send custom resource pack

I am running 1.21 fabric server with mod polymer
got this error message when trying to connect

Having the same issue here 👍🏻

I don't think this is caused by this mod, have you tried removing fabricproxy?

removing and re adding helped thx


Discord or any other resources (for themes for example?)

Is there a discord for discussion and fast help? If so where?
And is there some custom made themes for one to download to start with?

There isn't, the best way to reach out is here. Keep in mind we do this in our free time as a passion project, so fast help is not something we can promise.
There aren't any fully made custom themes beyond the default theme and a few pointers in the wiki. A full customisation guide is long overdue, but I won't put any work into it until we hear more from Riot about the future with Vanguard.

I get that and I appreciate that you still put in the work.
I just need some snippets so I can ask ChatGPT to help me with the rest. You wouldn't know anybody that did some changes to the themes? Don't have to be fully customized, just some changes.

You can find some customizations in closed issues, for example:
https://github.com/RCVolus/league-prod-toolkit/issues/264
https://github.com/RCVolus/league-prod-toolkit/issues/170
In theory, you can just throw any module webpage at chatgpt and ask it to modify the CSS – your mileage on actually usable CSS may vary.

I'm sorry, this confuses me. I know you don't want to do a full guide on customisation since Vanguard will come but I wanna configurate the appearance for the stream on sunday.
I know in the 3. Customisation Guide you wrote to use "Inspect" to identify CSS classes and/or IDs, but I don't know what to look out for when I press inspect since I'm a beginner. Can you maybe but a screenshot?
What also confuses me is the fact that in the examples you gave me (#264 and #170) and the Guide the call is varying all the time.
In #264 you use .scoreboard { in #160 .module-teams-talk-gfx .vs { and in the Guide you use .rcv-teams-ingame-gfx #score-container { (which is a different color, why is that?) How can I differentiate between them, when to use what, and how to use the inspect?

I managed to figure it out on my own. It's really not that hard. I might consider making a video to help others


How to let multiple independent processes to log into the same place?

I'm running multiple python scripts at the same time, and deliberately want them to log into the same place.
I suppose this is doable as the module supports multiprocessing, but not sure what is the best practice of doing this.

log into the same place
I assume you mean send log into the same logger process. Then that logger process will write the log to its final destination.
Then yes, it is doable.

It includes part parts that involve TCP/UDP logger:
(1) Logger server: the script that starts the TCP server with a predefined port.
(2) Logger clients: other scripts that use TCP/UDP logger and send logs to the server.
For (1), you just start logger_tt with use_multiprocessing=True, port=YOUR_PORT,  and we are done. It will start the logger server usually. No need to really spam any child process in that process.
For (2), you also need to start your scripts with use_multiprocessing=True, port=YOUR_PORT. But this time, you need a way to tell logger_tt not to start the logger server, just replace the log handlers with the TCP handler.
Considering the above scenario, I think we need to add a new param to setup_logging, pass it down to LogConfig, and then into _replace_with_socket_handler method to tell it whether to start the logger server or not.

It includes part parts that involve TCP/UDP logger:
(1) Logger server: the script that starts the TCP server with a predefined port.
(2) Logger clients: other scripts that use TCP/UDP logger and send logs to the server.

I wrote one just like this yesterday. and the trick i used is adding --multiprocessing-fork as the first argv for all the "client", so that they are considered as subprocesses by in_main_process.
It worked so far for me, but may have some side effects if other module also need to check if it's main process.
I think maybe we can add an _server attribute which will defaults to the result of in_main_process but can be overridden by input arguments.


Amplify CLI Sign in with Apple not working

How did you install the Amplify CLI?
curl
If applicable, what version of Node.js are you using?
v18.13.0
Amplify CLI Version
12.2.3
What operating system are you using?
MaC
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
I am trying to integrate a swift app using "Sign in with Apple" and my AWS back-end using Amplify.
I am trying to follow these instructions
URL page where content issue is:
https://docs.amplify.aws/lib/auth/social/q/platform/ios/
The configure auth category is inaccurate:
➜  $ amplify add auth
➜  Peakflow git:(AppleId) ✗ amplify add auth
Using service: Cognito, provided by: awscloudformation

 The current configured provider is Amazon Cognito.

 Do you want to use the default authentication and security configuration? Default configuration with Social Provider (Federation)
 Warning: you will not be able to edit these selections.
 How do you want users to be able to sign in? Username
 Do you want to configure advanced settings? No, I am done.
 What domain name prefix do you want to use? ###
 Enter your redirect signin URI: myapp://
? Do you want to add another redirect signin URI No
 Enter your redirect signout URI: myapp://
? Do you want to add another redirect signout URI No
 Select the social providers you want to configure for your user pool:
✅ Successfully added auth resource ### locally

I selected Sign in with Apple but the options to configure the social provider (as described in the instructions) are not shown and the resulting configuration is not complete (no social providers have been created):

{
    "UserAgent": "aws-amplify-cli/2.0",
    "Version": "1.0",
    "auth": {
        "plugins": {
            "awsCognitoAuthPlugin": {
                "UserAgent": "aws-amplify/cli",
                "Version": "0.1.0",
                "IdentityManager": {
                    "Default": {}
                },
                "CredentialsProvider": {
                    "CognitoIdentity": {
                        "Default": {
                            "PoolId": "eu-west-2:###",
                            "Region": "eu-west-2"
                        }
                    }
                },
                "CognitoUserPool": {
                    "Default": {
                        "PoolId": "eu-west-2###",
                        "AppClientId": "###",
                        "Region": "eu-west-2"
                    }
                },
                "Auth": {
                    "Default": {
                        "OAuth": {
                            "WebDomain": "###.auth.eu-west-2.amazoncognito.com",
                            "AppClientId": "###",
                            "SignInRedirectURI": "myapp://",
                            "SignOutRedirectURI": "myapp://",
                            "Scopes": [
                                "phone",
                                "email",
                                "openid",
                                "profile",
                                "aws.cognito.signin.user.admin"
                            ]
                        },
                        "authenticationFlowType": "USER_SRP_AUTH",
                        "socialProviders": [],  //No details are configured here
                        "usernameAttributes": [],
                        "signupAttributes": [
                            "EMAIL"
                        ],
                        "passwordProtectionSettings": {
                            "passwordPolicyMinLength": 8,
                            "passwordPolicyCharacters": []
                        },
                        "mfaConfiguration": "OFF",
                        "mfaTypes": [
                            "SMS"
                        ],
                        "verificationMechanisms": [
                            "EMAIL"
                        ]
                    }
                }
            }
        }
    }
}

Expected behavior
I expected

To be prompted to enter the Apple service details
For user and identity pools to be created
For the amplify config files to be updated within my swift project

Reproduction steps

amplify add auth
select social sign-in with Apple

Project Identifier
Project Identifier: ddd701997ab3e0fdb790cbfa0d51f2b7
report-1690993820391.zip
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @howardkitto, thank you for reaching out. From the aws-amplify-cli/2.0 I understand this is the amplifyconfiguration.json file. Was the auth resource pushed(amplify push) after the resource was created/updated?
As the attribute should get generated in the amplifyconfiguration.json file once we push.
Additionally, If you would like to verify if the socialProviders has been added currently before a push, we can check the backend-config.json file present under the backend folder.

Yes, this is after amplify push
To be clear: once I select Sign in with Apple as a social provider the script exits and does not give me the opportunity to enter the apple credentials, so it it not surprising that no social providers are created. I suspect that is the location of the bug.

Hey @howardkitto,

once I select Sign in with Apple as a social provider the script exits and does not give me the opportunity to enter the apple credentials

curious on this behaviour as i was unable to reproduce this issue in my environment. It would be great if you could provide us some additional information on the terminal being utilized and a screenshot/video of the behaviour?
Does running amplify update auth and updating the sign in with apple auth flow, have a similar behaviour?

Here is a screen recording:
https://drive.google.com/file/d/15PYPVeZyochQzqMIHhaKejuitAdtszvd/view?usp=sharing
Hopefully you can see that when I select Sing in with Apple there are no further options.
I just confirmed that the same thing happens when I use amplify update auth

@howardkitto thank you for the video, this helped us in diving into the issue.
When selecting the Sign in with Apple, we need to use the Spacebar button for a selection instead of Enter/return.
The flow would show as follows until we hit Enter/return
https://github.com/aws-amplify/amplify-cli/assets/87995712/f4281964-70d5-45f5-ad0c-59d9d537651e

Oh wow I'm such a dufus! Thank you so much

Oh wow - I'm such a dufus. Thank you so much


Crash with 250% or 300% scaling under Wayland

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.89.1 and 1.90.0-insider
OS Version: Ubuntu 24.04

Steps to Reproduce:

Start VSCode using Wayland, in a Wayland session configured with 250% or 300% scaling (I usually run code --ozone-platform=wayland .)
Move the mouse cursor onto the VSCode window. VSCode crashes abruptly.

Other scale factors will probably also cause the issue, I have only verified with 250% and 300%, but this does not happen when configuring the scaling to 200%. It even works to run VSCode on an attached monitor with 200% scaling while the laptop monitor is still configured to 250% scaling.
If I start VSCode with --verbose and give it time to settle before moving the mouse cursor onto its window, I get the following single log line before the process abruptly terminates:

[355646:0528/123537.580840:ERROR:wayland_event_watcher.cc(43)] libwayland: wl_surface@32: error 2: Buffer size (64x64) must be an integer multiple of the buffer_scale (3).

This problem is a recent regression, I have been running VSCode with fractional scaling under Wayland for a long time.

For all I know, it could be that the issue started when I installed Ubuntu 24.04, and wasn't present in 23.10, but I didn't see it immediately since I was using an external monitor with 200% scaling. I have not verified this, but I see from other similar issues that this buffer size mismatch seems to have been handled more leniently by mutter: https://gitlab.gnome.org/GNOME/mutter/-/issues/2428

Hi @maghoff does the issue still occur for you on VS Code 1.96 or later?
If yes, could you attach a crash report following the steps at Creating a Crash Report?

I cannot reproduce this with VSCode 1.96.1, Gnome 47 (Ubuntu 24.10) and 250% fractional scaling.
As a point of curiosity, the reproducability of this bug has been switching back and forth since I initially reported this. Will report again if it should reappear 🙂


Use first item of chain block as input

Just a thought that it might look a little neater if the first item of the chain block could be taken as the input to the chain, in the case where no input is supplied.
This is useful where you're assigning the result of the block to a variable, and have a long function call or variable name as the input.
Compare:
some_output_variable_name = @chain some_long_variable_name begin
    do_something()
	do_something_else(parameter)
    do_other_thing(parameter, _)
end

some_output_variable_name = @chain begin
    some_long_variable_name
    do_something()
	do_something_else(parameter)
    do_other_thing(parameter, _)
end

I thought about this as well, maybe it's a good idea. I wasn't sure at first if it's weird that you then one line is different from the others, but maybe it's so obvious that it's not a problem

added this in 0.4.1

Thanks, been using it with this change and it's really nice :)

Thanks, been using it with this change and it's really nice :)


Relay TXs only if node has a Validator peer

Fixes #1726

Doesn't it just push down the issue ? Shouldn't nodes try to periodically sync tx pools instead ?

Doesn't it just push down the issue ? Shouldn't nodes try to periodically sync tx pools instead ?

That may become too costly, no? Especially, when we reach MB levels of tx pool size. Nodes would at least need to share hashes of all the TXs in their pools. What is important is that we share the TXs that are worth externalizing and that is what consensus already does.
My take on this is that tx pools of FullNodes are not important at all. We could probably just make it a Validator thing IMO.
TXs should probably not be valid forever either, they should have an expiration date/block, so that the wallet knows when to retry.

I have checked to see if Bitcoin has a mechanism to keep node's mempools in sync and AFAICS there is nothing of the sort.
So we are probably better off with the best effort approach we already have but avoid trying to gossip to an empty set of peers.

wouldn't it be easier, if the node that is starting up, and haven't finished the network discovery would just not accept the transaction? Wallets should have retry mechanism anyway, no?

wouldn't it be easier, if the node that is starting up, and haven't finished the network discovery would just not accept the transaction? Wallets should have retry mechanism anyway, no?

It may never be able to complete the discovery tho.

@Geod24 mentioned the existance of INV messages in BCH. https://www.reddit.com/r/Bitcoincash/comments/jko0ty/bchn_mr_746_merged_faster_transaction_relay/
This makes sense for Bitcoin and its forks as they use PoW but we already do this in the nomination round. So our Validator nodes will never suffer from this problem anyways.
Our issue is to somehow get the TX from a FullNode to Validator without it ever getting stuck in the FullNode forever.
What I think will work is to keep the if check this PR adds and make sure that a FullNode always keeps trying maintain a connection to a Validator throughout it's lifetime.


The rxconfig.py seems doesn't work.

Describe the bug
Seems rxconfig.py does't work with me,and Reflex  always works on port 3000.
To Reproduce
I  installed reflex(0.2.3) on my Windows,and  i cloned the reflex-examples to run the twitter example and it works on port 3000. Then i modify the rxconfig.py as follow:
import reflex as rx

config = rx.Config(
    app_name="twitter",
    api_url='http://localhost:5000',
    db_url="sqlite:///reflex.db",
    env=rx.Env.DEV,
)

After excuting reflex run,it works still on port 3000,as the log shows below:
───────────────────────────────────────────────── Starting Reflex App ──────────────────────────────────────────────────
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 3/3 0:00:00
───────────────────────────────────────────── Installing frontend packages ─────────────────────────────────────────────
───────────────────────────────────────────────────── App Running ──────────────────────────────────────────────────────
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 3/3 0:00:00
App running at: http://localhost:3000

I have tried to run  the example todo with the same content of rxconfig.py,and the same result came out as follow:
───────────────────────────────────────────────── Starting Reflex App ──────────────────────────────────────────────────
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 1/1 0:00:00
───────────────────────────────────────────────────── App Running ──────────────────────────────────────────────────────
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 1/1 0:00:00
App running at: http://localhost:3000

Then i  repeat this on my ubuntu server,but still the same result as follow:
# cat rxconfig.py
import reflex as rx

class TestConfig(rx.Config):
    pass

config = TestConfig(
    app_name="Test",
    api_url="http://<IP_ADDRESS>:8000",
    # api_url="localhost:8000",
    #bun_path="/root/.reflex/.bun/bin/bun",
    db_url="sqlite:///reflex.db",
    #env=rx.Env.DEV,
)
# reflex run --env prod
───────────────────────────────────────────────────────────────────── Starting Reflex App ─────────────────────────────────────────────────────────────────────
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 1/1 0:00:00
Creating Production Build:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 9/9 0:00:43
───────────────────────────────────────────────────────────────────────── App Running ─────────────────────────────────────────────────────────────────────────
App running at: http://localhost:3000
Compiling:  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 1/1 0:00:00

Expected behavior
If there anything wrong with my steps,please figure it out.
I expect to host reflex on my own server and i need the right way,thanks.
Specifics (please complete the following information):
Win:

Python Version:Python 3.9.9
Reflex Version:0.2.3
OS: Win11

Ubuntu Server:

Python : 3.8
Reflex Version : 0.2.3
OS: Ubuntu  20.04.6 LTS

More
How can i gracefully stop the reflex task after i have excuted "reflex run"?

To run the frontend on a different port, you should set the frontend_port to the desired port number

as mentioned, the frontend port is controlled by frontend_port config key or reflex run --frontend-port 3333 when running the service.
check out the docs for more information on the configuration knobs: https://reflex.dev/docs/api-reference/config/

How can i gracefully stop the reflex task after i have excuted "reflex run"?

you should be able to send the reflex python process a SIGINT to stop the frontend and backend. if this isn't working, then we need a separate issue (#1515) to investigate that. i have noticed on windows that sometimes two CTRL-C are needed to actually stop the server, but do not reproduce this on mac or linux.
Finally, when running for production, you WILL want to reflex run --env prod, which makes things a lot faster.
I'm going to close this issue for now, but feel free to reopen or open new issues if you encounter further problems.

To run the frontend on a different port, you should set the frontend_port to the desired port number. The api_url is used by frontend to communicate with your backend server(this should typically be the url of your backend server)

Now it works , thanks.

as mentioned, the frontend port is controlled by frontend_port config key or reflex run --frontend-port 3333 when running the service.
check out the docs for more information on the configuration knobs: https://reflex.dev/docs/api-reference/config/

How can i gracefully stop the reflex task after i have excuted "reflex run"?

you should be able to send the reflex python process a SIGINT to stop the frontend and backend. if this isn't working, then we need a separate issue (#1515) to investigate that. i have noticed on windows that sometimes two CTRL-C are needed to actually stop the server, but do not reproduce this on mac or linux.
Finally, when running for production, you WILL want to reflex run --env prod, which makes things a lot faster.
I'm going to close this issue for now, but feel free to reopen or open new issues if you encounter further problems.

It works now ,thanks.


Cannot browse online documentation

It seems the certificate for the server the rustdocs are hosted on as linked by the README has expired some days ago:
> curl -I https://ogham.rustdocs.org/ansi_term
curl: (60) SSL certificate problem: certificate has expired
...

erk

I changed the links to use docs.rs — it doesn't work right now but will soon when it regens the docs. Thanks for pointing it out!


Add implementation of delete operation

@salaboy this is another tiny PR it adds improved handling of delete operation.


build(deps): bump react-native from 0.73.10 to 0.75.4

This PR resolves []
Description
Bumps react-native version from 0.73.10 to 0.75.4.
Removed Patches:

[x] react-native-mapbox-gl/maps
[x] react-native
[x] deprecated-react-native-prop-types dep (included in the react native patch)

Bumped Dependencies:

@gorhom/bottom-sheet
@react-native-async-storage/async-storage
"@react-native-mapbox-gl/maps": "8.5.0" removed in favor of "@rnmapbox/maps": "10.1.31",
@shopify/flash-list
@stripe/stripe-react-native
moti
react-native-reanimated

Removed Dependencies:
Flipper related stuff and react devtools (the react dev tools are still acessible with npx react-devtools

"deprecated-react-native-prop-types": "4.1.0",
"react-devtools-core"
"rn-flipper-async-storage-advanced": "1.0.5",
"redux-flipper": "2.0.2",
"react-native-flipper": "0.265.0",
"react-native-flipper-performance-plugin": "0.4.0",
"react-native-flipper-relay-devtools": "1.0.3",

[!IMPORTANT]
Flipper is decoupled from react native - we will run a survey to see if we need to bring it back or not

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

bump react-native version

Need help with something? Have a look at our docs, or get in touch with us.

ios-8.56.0-2<IP_ADDRESS>
android-8.56.0-1674645923

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 70330d4777df37d06a06062789d34a6d48745e2a

Impressive effort 👏👏👏


vscode.workspace.getWorkspaceFolder does not check if file exists (or API documentation is not clear)

Issue Type: Bug
Hi,
it looks like vscode.workspace.getWorkspaceFolder only checks if the uri matches any of the workspace folders uri. It does not check if the file actually exists in the folder.
For example if I have workspace folder which path C:\ps\build\engine then getWorkspaceFolder('C:\ps\build\engine\doesNotExist.txt') returns this workspace folder.
In other words, following always returns the folder: vscode.workspace.getWorkspaceFolder(vscode.Uri.file(path.join(vscode.workspace.workspaceFolders[0].uri.fsPath, "does not exists.txt")))
It might be that current behaviour is correct. But then, the API documentation is not clear in my opinion. Currently it says:

Returns the workspace folder that contains a given uri.

returns undefined when the given uri doesn't match any workspace folder
returns the input when the given uri is a workspace folder itself

Actual results:
vscode.workspace.getWorkspaceFolder  does not check if file exists.
Expected results:
Either:

vscode.workspace.getWorkspaceFolder checks if file exists - returns only if folder actually contains the specified uri, or
make API documentation more clear about how the function works

System details
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
AMD Ryzen 7 2700X Eight-Core Processor          (16 x 3693)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (5.71GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

jenkinsfile-support
ivo
1.1.0

vscode-typescript-tslint-plugin
ms-
1.2.3

I suggest we tweak the API docs to not break. Up for grab.


Updating from 3.1.1 to 3.5.3 breaks due to the change of SqlSession

MyBatis version
The version after 3.4.0.
Database vendor and version
None
Test case or example project
public class ShardedSqlSessionTemplate implements SqlSession{
	public ShardedSqlSessionTemplate(SqlSessionFactory sqlSessionFactory) {
	}
}

Steps to reproduce
When I try to upgrade mybatis-3 from 3.2.2 to the version after 3.4.0. The above code breaks due to the change of SqlSession. I try to check the release note of 3.4.0. I didn't find it.
Expected result
The code should pass.
Actual result
It throws an error with:
UTD.SeLab.BBI.BugDetection.ShardedSqlSessionTemplate is not abstract and does not override abstract method <T>selectCursor(java.lang.String,java.lang.Object,org.apache.ibatis.session.RowBounds) in org.apache.ibatis.session.SqlSession

@SElab2019
Thanks for your reporting! I've added an explanation for this broking changes on the release note of 3.4.0. For details of this specification changes , please see the #437.

You are welcome. Thanks for confirming. We are trying to propose an approach to detect those BBI issues.


Stuck in a boot loop after macOS Software Update

Summary
This may be a coincidence, but just wanted to report it in case there was a bug. I deployed Escrow Buddy to a system and it showed up as "Configured" within the Jamf EA (Escrow Buddy authdb status). I restarted with no issue, but noticed when I logged back into the system, it prompted (within System Settings) to update macOS to 13.6.4 from 13.5.1. When trying to install the update, it got stuck in a bootloop. Before even getting to the login screen, it prompts to enter a password for the Disk (looks like a FileVault prompt). Upon entering the password, it restarts and back to the same prompt. I've also tried macOS Recovery, including First Aid and reinstall macOS Ventura, but upon Reinstalling macOS Ventura through Recovery, it gives an error: This operation couldn't be completed: (com.apple.BuildInfo.preflight.error error 21.) - At that point I tried SMC/NVRAM Resets, Safe Mode, and Internet Recovery. I still either get the FileVault Prompt to enter my password, which then bootloops, or the preflight error (when trying to reinstall macOS via Recovery).
Steps to Reproduce

Install/configure Escrow buddy
Restarted MacBook
Ran update to latest version of Ventura
Upon restart, boot loop occurs.

Expected Behavior
No boot loop, and only an escrowed Key.
Environment

Escrow Buddy version: [e.g. 1.0.0]
macOS version: [e.g. Ventura 13.5.1]
MacBook Pro (13-inch, 2018)
MDM version: [Jamf Version 11.1.1-t1701704198]

Additional Context
I've added some screenshots of the FileVault prompt and preflight error.

Hi @janzablox - This doesn't seem like a series of symptoms that Escrow Buddy could be responsible for. However just to be thorough, would you mind running through these steps to remove Escrow Buddy from the authorization database on the affected Mac? If the issues persist after the authdb has been reset to its default, I think it's likely the issue has a different cause.

Hi @janzablox - This doesn't seem like a series of symptoms that Escrow Buddy could be responsible for. However just to be thorough, would you mind running through these steps to remove Escrow Buddy from the authorization database on the affected Mac?

Which "Disk" should we be selecting to "Mount" the drive? Because most selections only give the option to "Unmount", not "Mount".

It looks like Macintosh HD is already mounted in your screenshot. You should be OK to proceed to steps 4 and 5, where you delete specific files in Terminal from /Volumes/Macintosh HD/var/db.

I didn't get the desired output within Step 5:
 auth.db
 auth.db-shm
 auth.db-wal

It seems this may be unrelated to Escrow Buddy. I also couldn't reproduce the problem on another machine, so maybe it was a botched macOS upgrade.


[BUG] Query fails with ClassCastException when enabling dynamic partition pruning

my conf is :
spark.sql.optimizer.dynamicPartitionPruning.enabled true
spark.sql.optimizer.dynamicPartitionPruning.reuseBroadcastOnly false
spark.sql.adaptive.enabled true
spark.sql.adaptive.coalescePartitions.enabled true
my bug info is :
dpp&ae_bug_info.txt
what i want to say is that : in cpu , if you set this conf , it will not use dpp, but in gpu ,if i set this conf ,it will occur error as above. so I don't know if this is a bug in rapids.

Pasting the relevant snippet of the error here for easier browsing:
20/10/23 17:54:04 ERROR sql.rapids.GpuFileFormatWriter [pool-2-thread-2]: Aborting job d7683e9e-0fb4-4a66-a4a3-efdf765a2083.
java.lang.ClassCastException: org.apache.spark.sql.catalyst.plans.logical.Aggregate cannot be cast to org.apache.spark.sql.execution.SparkPlan
	at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:238)
	at scala.collection.immutable.List.foreach(List.scala:392)
	at scala.collection.TraversableLike.map(TraversableLike.scala:238)
	at scala.collection.TraversableLike.map$(TraversableLike.scala:231)
	at scala.collection.immutable.List.map(List.scala:298)
	at org.apache.spark.sql.execution.SparkPlanInfo$.fromSparkPlan(SparkPlanInfo.scala:73)
	at org.apache.spark.sql.execution.SparkPlanInfo$.$anonfun$fromSparkPlan$3(SparkPlanInfo.scala:73)
	at scala.collection.TraversableLike.$anonfun$map$1(TraversableLike.scala:238)
[...]

if i set this conf ,it will occur error as above. so I don't know if this is a bug in rapids.

This is definitely a bug.  Queries should not crash with ClassCastException.
@chenrui17 does this happen on every query or for a specific query?  The stacktrace implies this occurred during a particular group by operation, so I'm thinking it may be specific to only some queries.  If possible, it would be great to have full steps to reproduce this.

this is happend when DPP is triggered , and i run the TPC-DS query-7 and query-39a and query-39b ,and it's all triggered the DPP.
my conf is :
#scheduler
spark.scheduler.listenerbus.eventqueue.capacity 800000
spark.scheduler.mode FIFO
spark.locality.wait.process 1ms
spark.locality.wait.node 1ms
#spark.locality.wait.rack 1ms
#wait for cache locality, only for aep worker
#network i/o
spark.rpc.askTimeout 240
spark.rpc.lookupTimeout 240
spark.rpc.message.maxSize 512
spark.rpc.netty.dispatcher.numThreads 128
spark.rpc.numRetries 24
spark.rpc.retry.wait 10s
spark.network.timeout 240s
spark.storage.blockManagerSlaveTimeoutMs 240000
#input & output
#spark.sql.splitCombiner.enabled true
#spark.sql.listChildrenFileStatus.enabled true
spark.sql.files.maxPartitionBytes<PHONE_NUMBER>
#spark.io.compression.codec org.apache.spark.io.LZ4CompressionCodec
spark.sql.legacy.parser.havingWithoutGroupByAsWhere true
spark.sql.sources.partitionOverwriteMode DYNAMIC
#spark.sql.sources.parallelPartitionDiscovery.threshold 4
hive.exec.max.dynamic.partitions 2500
spark.resultGetter.threads 20
#hive metastore
spark.sql.catalogImplementation hive
spark.sql.hive.convertMetastoreOrc false
spark.sql.hive.schema.enable true
spark.sql.hive.filesourcePartitionFileCacheSize<PHONE_NUMBER>
shuffle
spark.memory.fraction    0.75
spark.reducer.maxSizeInFlight	96m
spark.shuffle.sort.bypassMergeThreshold	3000
spark.shuffle.mapOutput.minSizeForBroadcast 5m
spark.shuffle.spill.initialMemoryThreshold 104857600L
spark.shuffle.file.buffer 1m
spark.reducer.maxSizeInFlight 32m
spark.sql.shuffle.partitions 520
spark.sql.broadcastTimeout 1500
scheduler
spark.scheduler.mode FAIR
spark.speculation.quantile 0.9
spark.speculation true
spark.speculation.multiplier 3
spark.speculation.interval 1s
dpp
spark.sql.optimizer.dynamicPartitionPruning.enabled true
default 0.5
#spark.sql.optimizer.dynamicPartitionPruning.fallbackFilterRatio 0.1
spark.sql.optimizer.dynamicPartitionPruning.reuseBroadcastOnly false
ae
spark.sql.adaptive.enabled true
spark.sql.adaptive.coalescePartitions.enabled true
#spark.sql.adaptive.coalescePartitions.initialPartitionNum 520
#spark.sql.adaptive.coalescePartitions.minPartitionNum 520
#spark.sql.adaptive.skewJoin.skewedPartitionThresholdInBytes 1048576
#spark.sql.adaptive.skewJoin.skewedPartitionFactor 5
#spark.sql.adaptive.advisoryPartitionSizeInBytes 67108864
#spark.sql.adaptive.shuffle.targetPostShuffleInputSize 67108864
#spark.sql.adaptive.minNumPostShufflePartitions
#spark.sql.adaptive.maxNumPostShufflePartitions
cbo
spark.sql.cbo.enabled false
#spark.sql.cbo.planStats.enabled true
#spark.sql.cbo.joinReorder.enabled true
#spark.sql.cbo.joinReorder.dp.star.filter true
#spark.sql.cbo.starSchemaDetection true
#spark.sql.autoBroadcastJoinThreshold 500MB
#spark.sql.sortMergeJoinExec.buffer.in.memory.threshold 50000000
#spark.sql.execution.reuseSubquery false
#spark.sql.adaptive.skewJoin.extraShuffle.enabled true
extensions
#spark.sql.extensions io.delta.sql.DeltaSparkSessionExtension
#spark.sql.extensions io.delta.sql.DeltaSparkSessionExtension,org.apache.spark.sql.OapExtensions
#spark.sql.catalog.spark_catalog org.apache.spark.sql.delta.catalog.DeltaCatalog
gpu related
spark.plugins com.nvidia.spark.SQLPlugin
spark.rapids.sql.concurrentGpuTasks 4
spark.rapids.memory.pinnedPool.size 48G
spark.rapids.sql.batchSizeBytes 536870912
spark.rapids.sql.variableFloatAgg.enabled true
spark.sql.optimizer.inSetConversionThreshold 1000
spark.rapids.memory.gpu.pooling.enabled true
spark.rapids.sql.exec.BroadcastNestedLoopJoinExec true
spark.rapids.sql.castFloatToString.enabled true
spark.rapids.sql.expression.Upper true
spark.executor.resource.gpu.amount 1
#spark.rapids.memory.gpu.debug STDERR
spark.rapids.sql.explain true
spark.rapids.sql.hasNans false
spark.task.resource.gpu.amount 0.1249
#spark.rapids.memory.host.spillStorageSize<PHONE_NUMBER>4
spark.executor.resource.gpu.discoveryScript /home/work/chenrui17/getGpusResources.sh
double T4
spark.resources.discoveryPlugin com.nvidia.spark.ExclusiveModeGpuDiscoveryPlugin

Hi @chenrui17 I am trying to reproduce this now. What scale factor was the TPC-DS dataset that you were running against when you saw this error?

Hi @chenrui17 I am trying to reproduce this now. What scale factor was the TPC-DS dataset that you were running against when you saw this error?

sf=10000，but i think sf is nothing to do with this problem . so you can try sf=100 or 1000 ?

@chenrui17 I tried running q7, q39a, and q39b both with the spark-sql-perf version of the benchmark and with the version that we have in this repo (writing to parquet in both cases) and they both ran without error against SF=100 data set that I generated using spark-sql-perf.
Could you provide some more information:

Which Spark version are you using? (I used Spark 3.0.1)
Which plugin version are you using? (I used branch-0.3 as of yesterday)
Can you provide instructions for running the benchmark to reproduce the issue?

sorry, this error is caused by my own code in spark about AQE,  so I close it .


Release in crates

Hello,
could it be possible for you to release the "api" to crates.io?
Thanks for your code!

That would be nice. I haven't had time yet, unfortunately, but feel free to submit a PR!

...feel free to submit a PR

It's something only you can do.  It'll only take a couple min: Publishing on crates.io.

Hi @llambda, just wanted to bring this back up. Like @naturallymitchell said it only takes a couple minutes to publish to crates.io. It's blocking other crates that want to publish because you can't publish crates that depend on crates from git repos.
All you need is cargo login {YOUR_KEY} and cargo publish 😄

Thanks, I will do that, and I was not aware other crates were wanting to publish based off this.

Published, thanks for suggesting it. If you happen to use this software I'd love to hear about your use just out of my own curiosity, feel free to ping me a line on this issue<EMAIL_ADDRESS>or keybase.

I'm using it in my podium project: https://github.com/teovoinea/podium
In the interest of privacy I wanted an offline reverse geocoder for location metadata found in images.

Released

https://crates.io/crates/reverse_geocoder/1.0.1 with existing current API
https://crates.io/crates/reverse_geocoder/2.0.0-alpha.0 with improved API


add generateTaskSummary java lambda function shell

add generateTaskSummary java lambda function shell.
assemble TASK_TABLE and ISSUE_TABLE environment variable in serverless.yml
and simplify the lookup in tasktable.py and issuetable.py.

there is no 3) :) ...


retainContextWhenHidden not working as expected

Testing #105764
Either I am using it wrong or it is not working but I expect if this value is true, the webview just exists and is not recreated when the view is collapsed.

Try adding a console.log in the script run inside the webview. With retainContextWhenHidden disabled, you should see logging when the webview is first shown and when it is expanded
With retainContextWhenHidden true, you should only see logging when the view is recreated (and not on expand). This would happen: on first load, when the webview is disposed, the webview is hidden using the context menu and then shown again

Note that #107842 causes the webview to be disposed more often than it should be, but the main thing to confirm is that simply collapsing and expanding the webview should not destroy the webview when retainContextWhenHidden is set

As discussed: turns out I was setting the value in the wrong place because I misread the documentation and the sample does not include an example.
Potential improvement: Include this option in the sample with the value set to false. include a comment explaining that this is the default, not necessary but included for clarifying how and why to change it to true.

Can we have a separate option to never dispose webview view? (false by default).

Closing this since from our discussion it does sound like the setting is behaving as expected.
Regarding documenting this in the sample: On second thought, I don't think we should include this option. We strongly encourage extensions to use VS Code's default webview behavior, which performs better and is suitable for most cases. An advanced extension user can use this setting if they need to, but with all the performance downsides of retainContextWhenHidden I want to avoid making it an easy options that normal extension authors fallback to instead of making their webviews quick to setup and teardown


A3, A4 submissions

Assignment 3

Certain cookie files had "Set-Cookie" and some had "set-cookie". I only accounted for the lowercase.
Hence I converted the output from the curl request to lowercase and performed the analysis. Even though this changed the other contents of the curl files, as the main objective was to find the types of cookies, I followed this path.
If the case has to be preserved I will have to change the code abit more (which I tried but could not make work).
Assignment 4
Extra credit portion added

Completed A5, so creating a new pull request


Query - can we change build based on GNU autotools or meson system ?

@asoulier - We are using this LC3 codec in PulseAudio and wanted to have either GNU autools based or meson based build. Will that be compatible with android build ?  If not, then I can add "install" target to the exisiting native makefile. Thanks.

@kirankrishnappa-intel I don't see any compatibility with android build (using basel).
There is no problem to add another build system, as long as that t the make is not broken (used for embedded projects).
But before I can accept any pull requests, you have to sign a CLA (https://cla.developers.google.com/clas).
For install target, can you have a look to https://github.com/google/liblc3/compare/main...install-target
If that works for you, I can merge it.

But before I can accept any pull requests, you have to sign a CLA (https://cla.developers.google.com/clas).
@asoulier  - I am already covered under the name - "Intel CLA Submissions". Thanks.

Oh fine, so no problem that you add meson based build or patchs ;)


changing words in Mosaic stats table

This will resolve additional suggestions by Danica at https://github.com/VEuPathDB/web-eda/issues/1384#issuecomment-1460578906
Screenshot is attached after changing:

new screenshot after another change:


Added dovi to color profile 50 (Dolby Vision)

Motivation for changes:
Detailed changes:

Addressed issues:

Fixes # .

Implemented feature requests:

Feathub #XX.

Config usage if relevant (new plugin or updated schema):
paste_config_here

Log and/or tests output (preferably both):
paste output here

To Do:

[ ] Stuff..

This small contribution enables releases tagged with "Dovi" to be matched against color profile 50.


[#28] [Feature] As a user I want to have live passport capturing on the camera

closes #28
What happened 👀

Renamed Image to PassportImage to separate it more clearly from CameraImage
Added new methods to MrzProcessor to handle CameraImage and InputImage
Update sample to process camera input live without requiring image capture
Updated README with samples to perform live image processing for both CameraX and Camera2

Insight 📝
Live camera processing will be an implementation of the calling app, not this library. Instead we can provide methods to handle InputImage (from MLKit) and a new CameraImage class which can wrap data from ImageProxy (from CameraX)
Proof Of Work 📹
https://user-images.githubusercontent.com/53168251/191666368-d1c09eca-3d21-43ea-bee9-b33ecfd2797e.mp4

@ryan-conway Is this our expectation of capturing an image automatically without tapping on the capture button?
https://user-images.githubusercontent.com/28002315/192458475-0130b5a4-7794-4841-9a9a-d80131fa4dc3.mov
Regarding this record ☝  It seems like it captures an image automatically, but when I navigate back from NfcReaderScreen It doesn't trigger captures an image automatically, So I need to tab on capture button instead 🤔


Fix cmyk preview precompiled

What
CMYK pdfs were showing as inverted due to the way colour is handled in the CMYK colour space vs how it is handled in the RGB colour space.
To fix this we transform the colour space to CMYK to match the PDF.
How to review
Upload a CMYK pdf via API and view on the Admin site, you should be able to view the colours without them being inverted.

Screenshots!


Multiple 'No positional arguments allowed'.No  Handling for specific IndyError

Describe the bug
After performing calls to Indy library and trying to send result back to gRPC client,  Miu throws errors, because of not specifying arguments name for protobuf. Also there is no possibility to determine if error is IndyError and send more appropriate  error message.
see below
To Reproduce

run grpc server Miu
2.use this code

	conn, err := grpc.Dial("localhost:50051", grpc.WithInsecure())
	if err != nil {
		panic(err)
	}

	//create  poolServiceClient service

	poolServiceClient := identitylayer.NewPoolServiceClient(conn)
	pcr, err := poolServiceClient.CreatePoolLedgerConfig(context.Background(), &identitylayer.CreatePoolLedgerConfigRequest{ConfigName: poolName, Config: &identitylayer.ConfigCreatePoolLedger{GensisTxn: "/path_to_genesis"}})

	if err != nil {
		panic(err)
	}
	fmt.Printf("Pool creation result %d \n", pcr.GetErrorCode())

Run gRPC client to connect to Miu
Get 'positional argument error'

Expected behavior

get 0 as error Code if pool created successfully.
get 306 error Code if pool has already been created
get Internal error Code in case  other error happenned

Screenshots

Desktop (please complete the following information):

OS: MacOS 10.13.6

Additional context
It's quite common issue in plenty of methods. So PR for IndyError handler and fixing 'no positional arguments allowed' is extremely needed

created a PR for this.
https://github.com/bootstrapsp/miu/pull/17

as suggested PR's merged.


TMSS Engineering College : https://tec.edu.bd

Website URL : https://tec.edu.bd/

@IsrakAhmed Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


error from OUTCAR

Raman off-resonant activity calculator,
using VASP as a back-end.

Contributors: Alexandr Fonari  (Georgia Tech)
              Shannon Stauffer (UT Austin)
MIT License, 2013
URL: http://raman-sc.github.io
Started at: 2019-10-22 13:46

[main]: VASP_RAMAN_RUN='mpirun -np $NP -machinefile $PBS_NODEFILE --mca btl self,sm,openib --bind-to-core $EXEC >> test.out'
[main]: VASP_RAMAN_PARAMS='01_21_2_0.01'
[[4.7828934262654972, 4.7828934262654972, 4.7828934262654972], [4.7828934262654972, 2.0498114683994988, 2.0498114683994988], [2.0498114683994988, 4.7828934262654972, 2.0498114683994988], [2.0498114683994988, 2.0498114683994988, 4.7828934262654972], [0.6832704894664996, 0.6832704894664996, 0.6832704894664996], [0.6832704894664996, 3.416352447332498, 3.416352447332498], [3.416352447332498, 0.6832704894664996, 3.416352447332498], [3.416352447332498, 3.416352447332498, 0.6832704894664996]]
[main]: Mode #1: frequency<PHONE_NUMBER> cm-1; norm:  0.1886973
[main]: File OUTCAR.0001.-1.out not found, preparing displaced POSCAR
[main]: Running VASP...
[main]: File OUTCAR.0001.+1.out not found, preparing displaced POSCAR
[main]: Running VASP...
[main]: ERROR Couldn't find OUTCAR file, exiting...
the above error is shown. The sript is
#!/bin/bash

#PBS -l nodes=1:ppn=16
#PBS -l walltime=300:00:00
#PBS -N USPEXvariation
#PBS -q blade_x
#PBS -j oe
#PBS -V
source /public/software/profile.d/compiler_intel-composer_xe_2016.3.210.sh
source /public/software/profile.d/mpi_openmpi-1.6.5-intel.sh
#The program we want to execute (modify to suit your setup)
EXEC=/public/software/vasp/5.2.12/openmpi/intel/vasp
#setup hostfile
echo "host: "
#cat $PBS_JOBID-$PBS_JOBCOOKIE.hosts
echo "--------Begin-------------------------------------------"
#This jobs working directory
echo Working directory is $PBS_O_WORKDIR
#go to work dir
cd $PBS_O_WORKDIR
echo "Beginning time is: "
echo Time is date
echo Directory is pwd
NP=cat $PBS_NODEFILE|wc -l
echo "Numbers of Processors:  $NP"
echo "---------------------------------------------------"
#running
ulimit -s unlimited  # remove limit on stack size
#export VASP_RAMAN_RUN='time mpirun -np $NP -machinefile $PBS_NODEFILE --mca btl self,sm,openib --bind-to-core $EXEC >> test.out'
export VASP_RAMAN_RUN='mpirun -np $NP -machinefile $PBS_NODEFILE --mca btl self,sm,openib --bind-to-core $EXEC >> test.out'
export VASP_RAMAN_PARAMS='01_21_2_0.01'
python vasp_raman.py > vasp_raman.out
#time mpirun -np $NP -machinefile $PBS_NODEFILE --mca btl self,sm,openib --bind-to-core $EXEC >> test.out
#clean
echo "--------Finish-------------------------------------------"
echo "Finishing time is: "
echo Time is date
echo "--------Finish-------------------------------------------"
Is this problem solve?

have you deal with this issue, I also have this issue.


Change schedule.union not to increase depth of schedule tree

Fix #2818 by changing the definition of Schedule.union() from union of schedules to union of children of schedules so that it does not increase depth of schedule tree.
This PR cares not to lose the performance gain by #2870 (or #2848). That's why _union is newly introduced.
This fix would be a temporary fix until a flat representation of a schedule (mentioned in https://github.com/Qiskit/qiskit-terra/issues/2818#issuecomment-513385315) is implemented.

Thanks, @lcapelluto

keep the implementation out of the docs
This can be said to the change in CHANGELOG.md. Please tweak it if necessary when you resolve conflicts before merge.


feat(shipper): add support for BuildingLink

Proposed change
My building uses BuildingLink to manage the mail room... packages aren't really "delivered" until they've been processed and appear, so adding this means i can look for how many other shippers delivered vs. the number in buildinglink.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update
[x] Adds a new shipper
[ ] Update existing shipper

Updated.

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

Comparison is base (a3cf468) 95.42% compared to head (0f6d2b3) 95.42%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #877   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files           8        8
  Lines        1487     1487
=======================================
  Hits         1419     1419
  Misses         68       68

Files
Coverage Δ

custom_components/mail_and_packages/const.py
100.00% <ø> (ø)

Thanks!


fixed lobby view

Try to make a PR discription and better commits than 'stuff'. Bad.
Code lyk good though. Approved.


[Feature Request] - Access to specific root Drive

the goindex by donwa had this feature in the script to list only a particular drive instead of all the drives in the google account, please make that as an option here too

Modify the code yourself

there is no option to configure that, I have no technical know-how to do this

The simplest way is to create another account, and only share necessary team drive access to it. Or just learn to code

ok this was pretty easy
I just had to remove one letter "s" and I just completed my own feature request.

instead of listing drives I had to list drive

this was easy
I just had to remove the letter "s" and I just completed my own feature request

instead of getting drives I had to list drive


Return a "not found" message or cue when searching for a non-existent user

When you enter a string in the input field that does not correspond to an active community member, nothing happens. This is misleading. The user should get a cue that this is not a malfunction of the software, but a correctly terminated research that yielded no results.

With the autocomplete feature this issue is naturally fixed: if no node label is found, a "No search results." message is shown.


Is vpny_bybit compatible with vnpy 3.3.0?

Hi, I'm just exploring vnpy and try to use this module to connect to Bybit.
If i import it like this in my run.py (on Ubuntu 22.04.1)
from vnpy_bybiy import BybitGateway

I get the error
AtrributeError: BYBIT

Any idea what is wrong/or I do wrong?

u need to use this extra patch module.


pmgr: Fix false positive u8 pmgr ID detection on modern SoCs

The id1 field used on oder SoCs is not constant 0 for all devices on t8122. Instead just compare the id1 fields of the two first devices. Fixes "pmgr: Failed to find parent #36 for IOA0\n" errors in pmgr_init().
Fixes: fb59dc03ca9a ("pmgr: Fix struct for older SoCs")

Cc: @asdfugil does this still work on A7 - A11?

Works on A7-A11, T2, so:
Reviewed-by: Nick Chan<EMAIL_ADDRESS>


Fix issue with === identifying constructor method

Overview
This PR fixes an issue with the Concerns\IdentifiesClassThings@isConstructor method not correctly identifying a constructor.
The issue stems from the underlying Stmt object's attribute name. While $stmt->name will evaluate to a string because of an underlying __toString() method, but that attribute is actually an instance of PhpParser\Node\Identifier. When trying to evaluate $stmt->name === '__construct', this will evaluate to false because it's a class instance.
My proposed fix changes the identity operator to an equality operator, adding type juggling to resolve $stmt->name to a string. An alternate fix is to use $stmt->name->toString(), a public function provided by the Identifier class, and continue using ===.
All tests are passing with this change (using Docker).

@jakebathman I think I would prefer to be explicit (as it is the convention throughout the other linters) by using the ->name access version.
Would you mind adding a test that reproduces the original issue for this in the PR as well? Thanks!

Yes, definitely. I'll make it into an identity === check and add a test for the original misidentification of constructors.


[sqlparser] Replace const Map with static function.

The initialization of the map incurs a high cost at start up (as found out by @gregsexton). Using a function with a pattern match should fix that, and allow more optimizations.

The initialization of the map incurs a high cost at start up
Weird...

LGTM anyway

Nice! Thanks for doing this, I'll test it out.
Yeah the map is constructed at runtime during skip init called from main. Running skdb --init goes from an average of 0.0128 secs to 0.0179 if I checkout before and after the new parser is merged in. So in absolute terms it's not exactly expensive, but this is a 40% regression. It's across the board for all startup. Which really hurts the simulator as it starts a huge number of processes. This map accounts for ~half of the regression. The rest is the const values in SQLPrivacy.

The rest is the const values in SQLPrivacy.

The PredefinedTable singletons? Could look into moving away from Maps there too if that helps startup speeds.


fix(@angular/cli): disable version check when running ng completion commands

Running autocompletion with 14.0.1 installed globally in a 14.0.0 project logs the version warning in a very annoying fashion:
$ ng verYour global Angular CLI version (14.0.1) is greater than your local version (14.0.0). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
sion

This adds an exception for ng completion commands to avoid this edge case. This doesn't include any tests, as I'm not aware of a great way to test version mismatching behavior. I did try it locally and found that it worked as expected. Happy to add a test if someone can point me to the right method / location.

Not sure I fully understand the reasoning here, but updated to use the branch in lib.ts when running ng completion, which I think is what you were going for here @alan-agius4.

Closing in favor of https://github.com/angular/angular-cli/pull/23375


QSP-7 Limit Unlock Attempts

Lockout Screen:

Users will now have 5 chances to enter their password correctly before the extension locks for 5(?) minutes. The attempts reset after the 5 minutes or the next time they enter the correct password.
Files for testing:
Signer-v1.2.0-QSP7-latest

Tested-Pass , with one comments to remove redundant password error warning

Lockout duration is now set to 5 minutes

Tested-passed, 5 min timer works, closing browser and cheating timer also works fine. only redundant password error message still shows

Leaving the error message as is - I don't think it's big problem from a UX perspective the top error bar is consistent with any other error produced by the Signer and the underline error for the text box is pretty common UI/UX.
I agree we could probably do away with the top error bar for the unlock screen but for now we need to get this merged so that Quantstamp can complete their audit.


what is the latest way to generate an abi file

➜  eosio.system git:(master) ✗ eosiocpp -g eosio.system.abi  eosio.system.hpp
1419166ms thread-0   abi_generator.hpp:68          ricardian_contracts  ] Warning, no ricardian clauses found for
1420685ms thread-0   main.cpp:105                  main                 ] 999999 abi_generation_exception: Unable to generate abi
false: types can only be: vector, struct, class or a built-in type. (double)
{"type":"double"}
thread-0  abi_generator.cpp:626 add_type
{}
thread-0  abi_generator.cpp:299 handle_decl

{"decl_location":"/Users/sam/Public/eos/contracts/eosio.system/eosio.system.hpp:84:11"}
thread-0  abi_generator.cpp:44 handle_tagdecl_definition

1420685ms thread-0   main.cpp:105                  main                 ] output: {"version":"eosio::abi/1.0","types":[],"structs":[],"actions":[],"tables":[],"ricardian_clauses":[],"error_messages":[],"abi_extensions":[]}

I am seeing the same issue when I use eosiocpp -n to create scaffolding and then compile, my abi is flat and wen i set contract it works but can call the action as it is not defined in abi
"version": "eosio::abi/1.0",
"types": [],
"structs": [],
"actions": [],
"tables": [],
"ricardian_clauses": [],
"error_messages": [],
"abi_extensions": []

Found the issue - use eosiocpp -g beta1.abi beta1.cpp
use cpp file not recommended hpp file as 2nd param

@PatientDirected   BY cpp file, also issues:
➜  eosio.system git:(master) ✗ eosiocpp -g eosio.system.abi eosio.system.cpp
In file included from /Users/sam/Public/eos/contracts/eosio.system/eosio.system.cpp:4:
/Users/sam/Public/eos/contracts/eosio.system/producer_pay.cpp:3:10: fatal error: 'eosio.token/eosio.token.hpp' file not found
#include <eosio.token/eosio.token.hpp>
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
415302ms thread-0   main.cpp:105                  main                 ] 999999 abi_generation_exception: Unable to generate abi
res:
{}
thread-0  abi_generator.hpp:367 MacroExpands
415302ms thread-0   main.cpp:105                  main                 ] output: {"version":"eosio::abi/1.0","types":[],"structs":[],"actions":[],"tables":[],"ricardian_clauses":[],"error_messages":[],"abi_extensions":[]}
➜  eosio.system git:(master) ✗

@yankunsam
Just edit file "eosio.token.cpp",
replace #include <eosio.token/eosio.token.hpp> with  #include <eosio.token.hpp>.

@taokayan can you provide explanation why is it not working with that contract?
What about for a contract that has the code from eosio.token?

@taokayan Sorry for the long time, I write and compile a contract as dice. It works now.

@yankunsam Could you explain why eosiocpp -g <abiFile> <cppFile> doesnt work with the eosio.token.cpp file? I'm encountering the same problem.


Restructure the mess out of the repo root

Currently the directory tree looks like mess that is inconvenient to navigate. We need less files within the repo root and more of directory hierarchy.

Thank you for bringing it up. I would have to agree that it isn't very well structured and even moves the README render below the main github page. We will evaluate how to best proceed.


Fixes #8 Unnecessary whitespace when removing classes that do not match those defined in stylesheet

See test case. There is a case when removing unused class name produces unnecessary whitespace. This fixes issue #8

Coverage remained the same at 100.0% when pulling 04d5a889894066a91676f043a5813cfcd2fbbf22 on mir3z:master into abb201590ae6d1dbbc68c59ac7948ddd1cd1a424 on simonlc:master.

This is great, thanks so much for your contribution!


pickAll usecases?

So I stumbled on pickAll because lodash's pickAll is equivalent to your pick.  In R.pickAll I was surprised to see the inclusion of the undefined properties.  What utility does that give?  I do know delete obj.prop is technically not the same as obj.prop = undefined, but I've yet to see a case where the difference carries semantic value.  As far as any readable program is concerned, an object with a property that's undefined should behave as it's not there in the first place.
My suggestion would be to rename pickAll to pickWithDefault or something of the sort, where the api allows the developer to specify a placeholder for when a property doesn't exist.  In my mind null makes more sense than undefined as a default.  The user may want an empty string, a zero, or some other magical value instead. (irrelevant)

What utility does that give?

This may sound strange, but I find this line of thinking dangerous. A focus on convenience has led to many strange API decisions in the JavaScript world, such as functions which take three arguments, all of which are optional and which may be provided in any order!
What you're suggesting is perfectly reasonable, but there's a price to pay for the convenience of skipping each map entry whose value is undefined: information loss.
Current behaviour:
> R.pick(['a', 'b', 'c'], {a: null, b: undefined})
{a: null, b: undefined}

Proposed behaviour:
> R.pick(['a', 'b', 'c'], {a: null, b: undefined})
{a: null}

I've yet to see a case where the difference carries semantic value

In the vast majority of cases there is no difference. Even if there is no difference in 99.99% of cases, I still don't feel comfortable having R.pick destroy information. If for some reason a caller does consider {a: null, b: undefined} and {a: null} to be meaningfully different values, she could no longer use R.pick if we were to make the proposed change. On the other hand, a caller who wants to treat these values as equivalent can simply filter the map:
> R.reject(R.equals(undefined), {a: null, b: undefined})
{a: null}

> R.pick(['a', 'b', 'c'], R.reject(R.equals(undefined), {a: null, b: undefined}))
{a: null}

It's worth noting that R.pick is currently guilty of destroying information: from the return value we cannot determine the "set" of keys provided. We would need the Maybe type to preserve all the pertinent information:
> R.pick(['a', 'b', 'c'], {a: null, b: undefined})
{a: Just(null), b: Just(undefined), c: Nothing}

It's also worth noting that many other Ramda functions destroy information: R.nth, R.slice, and the functions derived from these come to mind. The difference in this case, though, is that the function can preserve information (about the values) without the use of the Maybe type.
I'm interested in whether others consider preservation of information an important design principle for a general-purpose library such as Ramda.

Ehm - I'm referring to pickAll.  I removed an irrelevant portion of my original post.
const obj = { a: 'this is a' };

R.pickAll(['a', 'b'], obj); // -> { a: 'this is a', b: undefined }

// vs pickAllWithDefault

R.pickAllWithDefault(['a', 'b'], 0, obj); // -> { a: 'this is a', b: 0 }
R.pickAllWithDefault(['a', 'b'], null, obj); // -> { a: 'this is a', b: null }

Ehm - I'm referring to pickAll.

Oh! After using and contributing to Ramda for more than two years I had no idea that R.pickAll exists! I thought by "pickAll" you meant the Lodash function of that name, or the Ramda equivalent. I'm sorry about the confusion.
I now really like this API:
> R.pick(['a', 'b', 'c'], {a: 42})
{a: Just(42), b: Nothing, c: Nothing}

> R.map(S.fromMaybe(0), R.pick(['a', 'b', 'c'], {a: 42}))
{a: 42, b: 0, c: 0}

> R.filter(S.isJust, R.pick(['a', 'b', 'c'], {a: 42}))
{a: Just(42)}

> R.map(S.toNullable, R.filter(S.isJust, R.pick(['a', 'b', 'c'], {a: 42})))
{a: 42}

It's flexible, and avoids the need for two "pick" functions. It's out of scope for Ramda, though.

R.pickAllWithDefault(['a', 'b'], 0, obj); // -> { a: 'this is a', b: 0 }
R.pickAllWithDefault(['a', 'b'], null, obj); // -> { a: 'this is a', b: null }

I like the idea of replacing R.pickAll with R.pickAllWithDefault. Anyone wanting the current functionality could simply use R.pickAllWithDefault(undefined). I think the default value should be the first argument, though, and I like the name you originally suggested: pickWithDefault.

Yeah my original post was worded poorly - sorry for that.

I'm not familiar with FP enough to translate this into js
R.pick(['a', 'b', 'c'], {a: 42}
// -> {a: Just(42), b: Nothing, c: Nothing}

// does this translate to
R.pick(['a', 'b', 'c'], {a: 42}
// -> {a: 42, b: undefined, c: undefined}
// ?

The only reason I ask is because I want your reasoning for why b and c should even be returned.  R.pickAll is only used in the ramda codebase in the 'project' function fyi, which I also think shouldn't be returning properties set to undefined.
Seems we're in agreement on the rest - and I didn't think about argument order, you're most likely right.

I'm not familiar with FP enough to translate your example into js

Here's one possible implementation using Sanctuary:
//    pick :: Array String -> StrMap a -> StrMap (Maybe a)
const pick = R.curry((keys, obj) =>
  S.reduce(memo => key => R.assoc(key,
                                  R.has(key, obj) ? S.Just(obj[key]) : S.Nothing(),
                                  memo),
           {},
           keys)
);

I like the idea of replacing pickAll with pickWithDefault, and simply noting that those who want the somewhat odd current behavior of pickAll (was it done to match Codd's laws for project -- I can't recall?) need to use pickWithDefault(undefined).

If I don't find time this weekend, I will definitely make time next to get a PR.  Thanks for the feedback.


.asar files missing

hello, I just installed betterdiscord because I got a new computer, but its saying I have missing asar files. I have been looking at some articles but none have helped me. i found a screenshot showing a bunch of files in resources, but i have no files in resources. please help me

@Jiiks

resources.zip
you need this

it has the folders that you're missing


Issue with Icingaweb 2 Graphite plugin

I added graphite plugin to Icinga web2 and enable it ,but now when i am trying to visualize graphs i get following page error attached image
Any idea?
Please advice
Thanks

That is just an iframe and what you see there is http://www.graphite.com. You need to configure the plugin to use your own graphite url. See /etc/icingaweb/modules/graphite/config.in and change base_url.

Thanks for the answer , but i don't see any config.ini or such under /etc/icingaweb/modules/graphite/
I re-downloaded the plugin to make sure and there is no such file, can you please instruct me what should be the structure of the file , so i can create it manually?
Thanks

What is the source (src) for the iframe?

Hi ,
I don't see the iframe , i just see that when i click on time frame
10 minutes 	1 hour 	12 hours 	1 day 	1 week 	1 month 	1 year
i get the i mage in a div instead of img , but when i right click and Open link in new TAB  i get img time frame correctly

Same problem here.
I have to servers with Icinga2, Icingaweb2 + Graphite
On the first server:
Ubuntu 14.04.2 LTS (GNU/Linux 2.6.32-37-pve x86_64)
Graphite is running on an extra subdomain.
Second server:
Ubuntu 14.04.2 LTS (GNU/Linux 3.16.0-38-generic x86_64)
Graphite and Icingaweb running on the same domain.

as a "dirty" hf replace:  /usr/share/icingaweb2/modules/graphite/application/views/scripts/index/index.phtml
with:

showOnlyCloseButton() ?>

url) ?>&from=-10min">10 minutes
url) ?>&from=-1h">1 hour
url) ?>&from=-12h" active>12 hours
url) ?>&from=-1d">1 day
url) ?>&from=-1w">1 week
url) ?>&from=-1mon">1 month
url) ?>&from=-1y">1 year

url) ?>&from=-1d" style="height: 700px; width: 800px" frameborder="no">

need to replace replace relative path to absolute like here: https://github.com/kain64/icingaweb2-module-graphite/commit/5a8125200aabf8f111c7bf7ad3657540253d9d5d.
/usr/share/icingaweb2/modules/graphite/application/views/scripts/index/index.phtml
it's work for me. but I'm nub in PHP : )

Thanks for taking a look @kain64 however the path is absolute already. Also thanks for introducing me to the term "nub", i intend to take that as my own..

Duh looking at it again thanks to @kain64 's insight i think i see the problem now. @tbaror and @mipapo please use an absolute url for the configuration setting "base_url" and see if that helps.

I came across both issues found here.
the first was fixed by adding a config.ini in /etc/icingaweb2/modules/graphite (note this not the /usr/share/icingaweb2/modules path) and ensuring the base_url was a full path (http://yourhost.yourdomain.com/render?). Mine is something like:
[graphite]
metric_prefix = icinga
base_url = http://graphite.host.domain.com/render?
legacy_mode = true
service_name_template = "$host.name$.$service.name$"
host_name_template = "$host.name$"
graphite_args_template = "&target=$target$&source=0&width=300&height=120&hideAxes=true&lineWidth=2&hideLegend=true&colorList=049BAF"
graphite_large_args_template = "&target=alias(color($target$_warn,'yellow'),'warning')&target=alias(color($target$_crit,'red'),'critical')&target=$target$&source=0&width=800&height=700&colorList=049BAF&lineMode=connected"
the second issue - large graph png being incorrectly rendered as gibberish - i fixed by making sure both my icingaweb2 and graphite instances were configured as vhosts (i have them running on the same phyiscal machine), as the issue appeared when icingaweb2 was not running as a vhost using the default icingaweb2 config. Equally you then need to update your icingaweb2-module-graphite's config.ini so the base_url points to the vhost address (like my example above)
hope this helps someone.

@tbaror I have your same problem: I added graphite plugin to Icinga web2 and enable it ,but now when i am trying to visualize graphs i get following page error attached image.
Can you help me what is the solution
Thank you a lot


Pipelining

Well, it's time.
Only a problem: it doesn't work anymore (it executes fine until a certain point, then something happens)
Commit message because I want to go to bed, I will split it tomorrow :D

Codecov Report
Patch coverage: 61.33% and project coverage change: -2.21 :warning:

Comparison is base (a65dfe2) 58.55% compared to head (b8a6ec0) 56.35%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #198      +/-   ##
==========================================
- Coverage   58.55%   56.35%   -2.21%
==========================================
  Files          40       40
  Lines        2874     2864      -10
==========================================
- Hits         1683     1614      -69
- Misses       1191     1250      +59

Impacted Files
Coverage Δ

emu/src/cpu/arm/instructions.rs
75.81% <ø> (-1.31%)
:arrow_down:

emu/src/cpu/arm/mode.rs
20.00% <ø> (-57.15%)
:arrow_down:

emu/src/cpu/thumb/alu_instructions.rs
76.66% <ø> (ø)

emu/src/cpu/thumb/instruction.rs
88.23% <ø> (-1.07%)
:arrow_down:

emu/src/cpu/thumb/mode.rs
13.88% <ø> (-83.34%)
:arrow_down:

emu/src/cpu/arm7tdmi.rs
69.14% <51.21%> (-3.78%)
:arrow_down:

emu/src/cpu/arm/operations.rs
85.91% <64.70%> (+0.56%)
:arrow_up:

emu/src/cpu/thumb/operations.rs
86.39% <82.35%> (+0.78%)
:arrow_up:

... and 2 files with indirect coverage changes

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

Fixed, it was a missed flush :D


Generate lenses with Myriad

Since https://github.com/MoiraeSoftware/myriad/pull/46 Myriad supports the generation of lenses.
I really wanted to try that, so I added manually build .nupkg files to try it out and implement it for Andromeda.
If there is a release on nuget, I will replace that with a normal reference in paket.dependencies.

Done in a2942f40672b7706d7e8df91d51cea21fda7f13a