Org

Dear Team,
When we are consuming DataverseService Client with D365 instance for CRUD operation, we found internally it is using 2011 WCF services of Dynamics 365 which was depreciated by Microsoft for a long time. I am attaching the screenshot of Application insights which clearly shows the WCF service URL.
Would you please clarify the same?
Thanks,
Nausher
URL: https://xxx.crm.dynamics.com:443/XRMServices/2011/Organization.svc/web?SDKClientVersion=<IP_ADDRESS>43

@naushers Thanks for your question.
We have deprecated and will continue to phase out external use of the WCF endpoint.
This specifically means developers that are trying to integrate with the SOAP contract without using the CrmServiceClient or the Dataverse ServiceClient.
We will at some point block access to it by clients other then CrmServiceClient and Dataverse ServiceClient, but have yet to announce a date on this.
If you'r using the CrmServiceClient or Dataverse ServiceClient you are 'safe' as we will continue to manage the wire protocol to meet the needs of the SDK clients.  As it changes, the clients will be updated to support it.
If you are using the SOAP Endpoint directly, not using our clients, you should transition off direct use of the SOAP Protocol to the WebAPI (OData 4.0 Standard).

Thank you for your update.
Get Outlook for iOShttps://aka.ms/o0ukef

From: MattB @.>
Sent: Thursday, November 17, 2022 6:46 AM
To: microsoft/PowerPlatform-DataverseServiceClient @.>
Cc: Nausher S. Shaikh @.>; Mention @.>
Subject: Re: [microsoft/PowerPlatform-DataverseServiceClient] Org (Issue #337)
Attention: This email was sent from an external source. Do not click links or open attachments unless you recognize the sender.
@naushershttps://urldefense.com/v3/__https://github.com/naushers__;!!LQP3EJY!2EI1UXUHoRyVeRlrAhfYMsETMoxTfBmTVJyAh8X6FSRbw7_M5XqKXvDce4Uz2VAqxd_lnkMXWtlPhGBfTAnyPxVr-za-16Xw$ Thanks for your question.
We have deprecated and will continue to phase out external use of the WCF endpoint.
This specifically means developers that are trying to integrate with the SOAP contract without using the CrmServiceClient or the Dataverse ServiceClient.
We will at some point block access to it by clients other then CrmServiceClient and Dataverse ServiceClient, but have yet to announce a date on this.
If you'r using the CrmServiceClient or Dataverse ServiceClient you are 'safe' as we will continue to manage the wire protocol to meet the needs of the SDK clients. As it changes, the clients will be updated to support it.
If you are using the SOAP Endpoint directly, not using our clients, you should transition off direct use of the SOAP Protocol to the WebAPI (OData 4.0 Standard).
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/microsoft/PowerPlatform-DataverseServiceClient/issues/337*issuecomment-1317915430__;Iw!!LQP3EJY!2EI1UXUHoRyVeRlrAhfYMsETMoxTfBmTVJyAh8X6FSRbw7_M5XqKXvDce4Uz2VAqxd_lnkMXWtlPhGBfTAnyPxVr-xuhbTLD$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4G2NWGZHJO7P6SVMOMZKHLWIWBN5ANCNFSM6AAAAAASB36GKE__;!!LQP3EJY!2EI1UXUHoRyVeRlrAhfYMsETMoxTfBmTVJyAh8X6FSRbw7_M5XqKXvDce4Uz2VAqxd_lnkMXWtlPhGBfTAnyPxVr-_yTikcv$.
You are receiving this because you were mentioned.Message ID: @.***>
Disclaimer: Hope Bancorp, Inc. and its wholly owned subsidiary, Bank of Hope never asks for your account or personal information via email. When emailing us, please do not include any non-public information such as account numbers or Tax Identification Number. This e-mail message is only intended for the person(s) to whom it is addressed. It may contain confidential and proprietary information and is protected from disclosure. If you are not the intended recipient or a person responsible for delivering it to the intended recipient, you are hereby notified that any review, retention, copying or distribution of this communication is prohibited by law. Please reply to the sender that you have received the message in error, and then delete it. In addition, please be aware that any message addressed to our company is subject to archiving and review by persons other than the intended recipient. Although reasonable precautions have been taken to ensure that no viruses or other harm are present, Hope Bancorp, Inc. and Bank of Hope make no warranty or guaranty with respect to, and are not responsible for any loss or damage arising from the receipt or use of this e-mail or attachments. Thank you.


141 ai movement behaviour

Added IMovement Interface for Movement Behaviours
Created Steering Movement Behaviour with Follow working, with flee half working and wander logic on the way
Uses steering data that can be serialized, so it can be used with the meta management module
Created the AISteeringMover that is the Humble Component that uses the Steering Behaviour

Changelog

Added IMovement Interface for Movement Behaviours
Created Steering Movement Behaviour with Follow working, with flee half working and wander logic on the way
Uses steering data that can be serialized, so it can be used with the meta management module
Created the AISteeringMover that is the Humble Component that uses the Steering Behaviour


Change loglevel at runtime

Closes #3369.
Allows changing the log level at runtime through the diagnostics port.

I have one nit, which is not related to this PR. Sending a line that cannot be parsed into Command (that includes an empty line) results in an ERR message, which is not dangerous, but just looks a bit bad :)

That's actually intentional. When using it non-interactively, you will always get back exactly one response indicating the outcome of your operation for every line you sent. I'd rather keep it this way!
The response can be disabled entirely by setting quiet mode.

bors r+

bors retry


Use proper RHEL7 labels in v9.4

Also don't --disablerepo=epel because that command fails if there
is no epel repository configured (default).

ptal

LGTM @hhorak we need to do this for all image, correct?

@mfojtik yes, I hoped this should already be fixed in most of them, but found it's not still for perl, python  and ruby.. (the new versions)
@praiskup could I ask you to use your scripting skills and submit a PRs for that?

[test]

FYI
https://raw.githubusercontent.com/praiskup/temporary-helpers/fcde79895785be9a87eb306aac308d6399652d61/rh-os-label-generator

@praiskup thx, it seems helpful :)

@hhorak @praiskup lets hope we will not need to change labels too often ;-)

As far as we do not need to keep "Release" label in sync with RHEL's dist-git ..

I guess Release label may stay as downstream-only difference.


Fix minimum version tests

I forgot a $

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.19%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
22

Relevant Lines:
84

💛  - Coveralls


Make batch into a drop-down/radio button selection instead of text box

@nate-watson80 completed this


[Ghidra] BinExport should never export multiple functions at the same address

When exporting the busybox binary from busybox_mips_ghidra_binexport.zip, the function _init at address 004028B0 is included twice in the flow_graph field, causing BinDiff to fail:
$ bindiff busybox_g.BinExport busybox_g.BinExport
BinDiff 7 (@376216480, May 27 2021), (c)2004-2011 zynamics GmbH, (c)2011-2021 Google LLC.
Error: AttachFlowGraph: flow graph already attached 004028B0

The root cause is Ghidra not properly handling _init, but the BinExport extension should always output a valid .BinExport.

For reference: BinDiff b/153891410.

FWIW this issue seems specific to MIPS binaries - I had the same problem with MIPS eCos images.
Is there any known work around for this? Thanks in advance

Well, BinExport uses whatever the disassembler gives it. So on one hand, this should really be fixed in Ghidra, either by fixing its analysis/heuristics or manually by cleaning up the disassembly.
A small script that highlights duplicate functions should be easy enough to come up with for this.
On the other hand, like I already said above, BinExport should always produce a valid .BinExport file. So I will fix this in BinExport, too, at some point. But that likely means to just drop one duplicated functions. This may or may not be what you'd want, though, as we might drop the wrong one (and for BinExport there's no way to know which one is good).

I just want to bump this with a little additional information:
I have the same problem with an ARM executable, so this is not restricted to MIPS

Oh, and I should point out that this happens when comparing 2 debug builds of closely related executables. Seems strange for Ghidra to get tripped up on that...

Encountered this same issue on AARCH64 binaries, still an issue on latest version.


cfripper: 1.15.1 -> 1.15.2

Diff: https://github.com/Skyscanner/cfripper/compare/refs/tags/1.15.1...1.15.2
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 272561 run on x86_64-linux 1

2 packages built:

cfripper
cfripper.dist


check, etc. shouldn't be circled

check, plus, close, etc. should just be the icon w/o circle.
check-circle, etc. should be the circled version.
Would need a major version bump for this change.

I agree and am glad if somebody can provide a css hack to remove the circle for a particular icon.


GoFish

"Storport Driver Support Routines" links to GoFish

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

ID: d7f7489b-89e8-1316-c7ad-0855d5f6c299
Version Independent ID: efd328fe-d215-9e1e-9ef9-07dd66c598aa
Content: Storport Miniport Drivers - Windows drivers
Content Source: windows-driver-docs-pr/storage/storport-miniport-drivers.md
Product: windows-hardware
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

Thank, @BenWestbrook. The link now point to the correct topic:
https://docs.microsoft.com/windows-hardware/drivers/storage/storport-driver-support-routines
Merging right now. Should be live shortly.


1.7.4: ChargerPhases > 1 check failing

Checklist

[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
Not yet seeing the stream entities appear in Home Assistant
Looks like the ChargerPhases > 1 check is failing.
Debug logs
2024-03-15 17:23:55.878 DEBUG (MainThread) [teslemetry_stream] event {"data": {"Soc": "80.732"}, "createdAt": "2024-03-15T07:22:44.429441182Z", "vin": "5YJ3F7EC5LF698201", "timestamp":<PHONE_NUMBER>429}
2024-03-15 17:23:55.879 DEBUG (MainThread) [teslemetry_stream] event {"data": {"ChargeCurrentRequest": "16"}, "createdAt": "2024-03-15T07:22:54.429487543Z", "vin": "5YJ3F7EC5LF698201", "timestamp":<PHONE_NUMBER>429}
2024-03-15 17:23:55.879 DEBUG (MainThread) [teslemetry_stream] event {"data": {"Soc": "80.913"}, "createdAt": "2024-03-15T07:23:04.429417483Z", "vin": "5YJ3F7EC5LF698201", "timestamp":<PHONE_NUMBER>429}
2024-03-15 17:23:55.879 DEBUG (MainThread) [teslemetry_stream] event {"data": {"ChargerPhases": "2", "ChargeAmps": "16", "ChargeLimitSoc": "100"}, "createdAt": "2024-03-15T07:23:24.429636642Z", "vin": "5YJ3F7EC5LF698201", "timestamp":<PHONE_NUMBER>429}
2024-03-15 17:23:55.879 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/teslemetry_stream/__init__.py", line 193, in listen
    listener(event)
  File "/config/custom_components/teslemetry/entity.py", line 179, in _handle_stream_update
    self._async_value_from_stream(value)
  File "/config/custom_components/teslemetry/binary_sensor.py", line 270, in _async_value_from_stream
    self._attr_is_on = self.entity_description.is_on(value)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/teslemetry/binary_sensor.py", line 64, in <lambda>
    is_on=lambda x: x > 1,
                    ^^^^^
TypeError: '>' not supported between instances of 'str' and 'int'

This has been fixed for the next release.


Behavior of "cf services" and "cf service" is inconsistent with regard to permissions

Issue
Behavior of "cf services" and "cf service" is inconsistent with regard to permissions
Context
An organization manager without any space role assigned in the target space can use cf services to list all service instances of the space. However, when using cf service to show the service instance info for such a service, an error message is displayed that states that the service instance was not found.
Steps to Reproduce
Assign the organization manager role to a user for an organization. Choose a space (or create one) but don't assign a space role to this user. Create a user-provided service instance in that org / space.
Login with the user above
cf login
List all services instances
cf services
Show service instance info for the user-provided service
cf service <name of user-provided service>
Expected result
As the user-provided service instance is listed in the output of cf services, I would expect that the user can see the service instance info for the user-provided service instance as well.
Current result
cf service <name of user-provided service> returns an error.
Possible Fix
cf services shows only service instances of spaces for which a space role is assigned to the user.
Example
> cf services
Getting services in org Multi Region Team / space dev as<EMAIL_ADDRESS>OK

name             service         plan   bound apps   last operation
logstash-drain   user-provided

> cf service logstash-drain
FAILED
Service instance logstash-drain not found

Hi @neckert
Thanks for filing the issue. However, we are not able to reproduce the problem using the steps you have provided. User with OrgManager role and no Space role can see the service with both commands cf services and cf service.
We are testing this on API version 2.55.0 (cf target to see version). Can you provide further steps to reproduce the problem?
Thanks
@simonleung8

Hi @neckert,
Checking back in with you. Can you provide us with more reproduction steps?
Thanks,
@utako and @simonleung8, CF CAPI Team

Hi @utako and @simonleung8 ,
I found the issue on an older version.
I can confirm that the problem is solved on 2.54.0.
Thanks
@neckert


zsh: operation not permitted: flutter

1.charged the terminal to zsh and made new zprofile through touch
2.then i used vim .zprofile, .zshrc and added this variable- export PATH="$PATH:/Users/username/flutter/bin"
Now when i run flutter i get this error "zsh: operation not permitted: flutter"

added full disk access for my terminal

Closing as duplicate of https://github.com/flutter/flutter/issues/65101


Implement Compact Score Bar according to design

The puzzle 1-1c865966 from #1 has to be resolved:
https://github.com/codeforthailand/election-live/blob/07762ca9a76792d7a6a37b927c0ac6c676f7dc34/src/components/CompactScoreBar.js#L6-L6
The puzzle was created by Thai Pangsakulyanont on 19-Mar-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

From design:


CAMEL-12848 - handle differently on windows for fa

iling test case

guys, please wait before merging.


Show Chrome bookmarks bar at top

I have my bookmarks bar in Chrome hidden by default, but it does show on Chrome's default new tab page. Could you make it show the same way on your page?
I know there is a bookmarks card, but I have a lot of sub menus and the default bookmarks bar allows you to quickly navigate folders by holding the left mouse button down.
Perhaps you could toggle the chrome setting when the page comes into view instead of replicating the bookmarks bar.
Environment

Version used:
Browser Name and version: Chrome
Operating System and version: Windows 10

Chrome does not give the possibility to do it and I do not want to recreate it, sorry, I leave the issue open if there are changes


Flex project fail to build since version 0.18.0

Hi, my flex project build succesfully on version 0.18.0, but fail to build on newer versions. After 10 min, I receive a java heap space.
Visual Studio Code 1.36.1
Apache Flex 4.16.1

Can you share the exact error message that you see?

And are you on Windows or macOS?

Hi, i'm on Windows 7.
I receive the message:

Executing task in folder Flex: C:\Program Files (x86)\Common Files\Oracle\Java\javapath\java.exe -jar c:\Users\Valter.vscode\extensions\bowlerhatllc.vscode-nextgenas-0.21.0\bin\asconfigc.jar --sdk c:\Dev\Flex\4.16.1 --debug=true --project c:\Dev\SVN\MKSaude\Flex\asconfig.json <

Loading configuration file C:\Dev\Flex\4.16.1\frameworks\flex-config.xml
Error: Java heap space
The terminal process terminated with exit code: 1
Terminal will be reused by tasks, press any key to close it.
Flex.zip
Configuration files atacched.
Thanks.

In your previous IDE, do you remember if you ever had to specify some kind of option to increase the Java heap space, like -Xmx1024M or something like that?

Yes, in the Flash Builder, I changed the FlashBuilder.ini with this:
-Xms512m
-Xmx1024m
-XX:+UseParallelGC
-XX:MaxPermSize=256m
-XX:PermSize=64m
Em qui, 8 de ago de 2019 às 12:35, Josh Tynjala<EMAIL_ADDRESS>escreveu:

In your previous IDE, do you remember if you ever had to specify some kind
of option to increase the Java heap space, like -Xmx1024M or something
like that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BowlerHatLLC/vscode-as3mxml/issues/397?email_source=notifications&email_token=AM3H253W7MCB7YM5BCBJJGTQDQ4NPA5CNFSM4IKJNC72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD34ALLQ#issuecomment-519570862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM3H252HHMDHHYJ2G4LB2WDQDQ4NPANCNFSM4IKJNC7Q
.

Thank you. It seems like I need to add a setting to the VSCode extension to allow you to customize it there too.

Any time. In version 0.18 build without errors and very fast.
Em qui, 8 de ago de 2019 14:45, Josh Tynjala<EMAIL_ADDRESS>escreveu:

Thank you. It seems like I need to add a setting to the VSCode extension
to allow you to customize it there too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BowlerHatLLC/vscode-as3mxml/issues/397?email_source=notifications&email_token=AM3H254HNP2AT74W6SXE5SLQDRLS7A5CNFSM4IKJNC72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD34MJGA#issuecomment-519619736,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM3H254E3MCSPXBCSSS4ANLQDRLS7ANCNFSM4IKJNC7Q
.

In the next update, a new as3mxml.asconfigc.jvmargs setting will be available. You'll be able to specify extra arguments for the JVM, like the ones you used in FlashBuilder.ini. In particular, -Xmx1024m will increase the max heap size, which should fix your build.

I just published v0.22.0, which restores the old memory behavior. As mentioned in my previous comment, you can now also pass in custom values, if the JVM's defaults don't work either.


redis transport downgrade implementation

redis transport with downgrade implementation

Details? If you're just asking that a Redis transport be added, that's a nice request but not one that I'd be willing to take on at this point.


Private package error

I want to try this package. But is it private package or deleted?
When i try to pnpm add drizzle-postgis, below error occurs.
 ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/drizzle-postgis: Not Found - 404

This error happened while installing a direct dependency of C:\code\orgs\hhyg\hilchin

drizzle-postgis is not in the npm registry, or you have no permission to fetch it.

An authorization header was used: Bearer npm_[hidden]

@toumku sorry about this, I haven't gotten around to actually publishing the package yet 🤦
I'll try to get to it soon, in the meantime, feel free to copy the *.ts files from https://github.com/Schmavery/drizzle-postgis/tree/main/src into your project?
Guess I was a little too optimistic when writing the README 😆

Ok this is now published. Please try again! :)

by the way It worth mentioning that npm supports direct links from github to be installed as packages
I guess this will work :
npm install Schmavery/drizzle-postgis

did it work for you? wasn't sure if the typescript would be handled properly


Bridge unit tests

This PR adds unit tests to the bridge server. This is in preparation for the release of bridgev0.0.32.
Most of the tests are from the old repo.
There are also some fixes that were included

Assets in config are now represented as protocols.Asset types.
Validation has been added to the authorize handler so it only authorises assets that are in the config file.
The payment listener and tx submitter now updates the database with the correct status if there was an error

These are changes that I had missed when converting to the new SDK or that were not moved at all.. For example validating asset details for the /authorize endpoint was not moved here from the old repo.
These are only now visible now that we have tests.


Paste support on website (ex: github,com)

When you paste Gyazo's URL it convert to markdown style.
For example, you paste https://gyazo.com/a9c009b7e9ba6d000604b05ff00684c7 on github's issue page, then pasted text will be [![https://gyazo.com/a9c009b7e9ba6d000604b05ff00684c7](https://gyazo.com/a9c009b7e9ba6d000604b05ff00684c7/raw)](https://gyazo.com/a9c009b7e9ba6d000604b05ff00684c7)
You can enable/disable on option page

prior art: https://github.com/uiureo/pastem


Exploring flaky tests: more logs and several improvements

Added additional check to Listener::bind to handle the case when socket creation is failed
Added more logs to debug flaky http_server_test.multiple_client_with_requests_to_multithreaded_server test
Improved https test and fixed warning

Codecov Report

Merging #716 into master will decrease coverage by 0.01%.
The diff coverage is 75%.

@@            Coverage Diff            @@
##           master    #716      +/-   ##
=========================================
- Coverage   67.51%   67.5%   -0.02%
=========================================
  Files          50      50
  Lines        7229    7231       +2
=========================================
  Hits         4881    4881
- Misses       2348    2350       +2

Impacted Files
Coverage Δ

src/server/listener.cc
67.28% <75%> (-0.13%)
:arrow_down:

include/pistache/async.h
88.94% <0%> (-0.26%)
:arrow_down:

Continue to review full report at Codecov.

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


Add 'run' to instruction in README for using nodemon

The current line about nodemon in the README is:
run: npm watch:dev to start nodemon in local dev enviornment.
When we used it, it was not successful at starting the local server.
However,
npm run watch:dev
was successful at starting the local server.
We suggest adding 'run' to that line of instructions.

Fixed by commit 74bcc67e


ODBC connection

I need to connect to Marxmeier Eloquence Database which has only odbc connection
is it possible?

DBeaver has ODBC driver, you could try to use it.
Note: only 64-bit drivers are supported.
I cannot say any details about Marxmeier Eloquence  though.


Mute On/Off event

I have intergrade saving volume value with localStorage.... Working fine!
On volumechange event. When user change volume with dragging slider...It's fine.
But user click into speaker icon (on/off mute) this event not called.

Thanks for pointing this. I'll check this and keep you posted

When I clicked the speaker icon I can see the volumechange event coming through using this code:
new MediaElementPlayer(mediaElements[i], {
	success: function (media) {
		media.addEventListener('volumechange', function(e) {
			console.log(e);
		});
	}
});

Any updates on this?

Closing this issue since event is being fired on muting media

Sory @rafa8626 . I have busy over week.
When click mute icon. getVolume always return 1. It seems changed after event called.
media.addEventListener('volumechange', function(e){
		alert(media.getVolume());
    });

The volume never changes when you mute the player: it simply changes the muted property, to avoid losing the original volume

So, how to store muted status?

Check media.muted or media.getMuted() when volumechange happens

That way you can store it locally


Generate static html pages

After #65 it will be possible to pre-generate html pages to avoid tiny lag before page rendering.
https://github.com/markdalgleish/static-site-generator-webpack-plugin

This will be incorporated in #64


Build depends on qulice

objectionary/eo-intellij-plugin#22

@yegor256 please merge

Codecov Report

Merging #24 (7c577c3) into master (b6256c2) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage     11.46%   11.46%
  Complexity       13       13
=========================================
  Files            11       11
  Lines           157      157
  Branches          4        4
=========================================
  Hits             18       18
  Misses          134      134
  Partials          5        5

Impacted Files
Coverage Δ

...java/org/eolang/jetbrains/EoColorSettingsPage.java
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yasamprom @yegor256 Oops, I failed. You can see the full log here (spent 15s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 1069567
+ echo '2.0-SNAPSHOT BUILD'
2.0-SNAPSHOT BUILD
+ date
Tue 16 Aug 2022 03:10:25 PM CEST
+ uptime
 15:10:25 up 48 days, 17:11,  0 users,  load average: 0.06, 0.01, 0.00
+ cat
com.rultor.spi.Profile: There is no 'merge' section in .rultor.yml
	at com.rultor.agents.req.StartsRequest.docker(StartsRequest.java:350)
	at com.rultor.agents.req.StartsRequest.vars(StartsRequest.java:255)
	at com.rultor.agents.req.StartsRequest.script(StartsRequest.java:148)
	at com.rultor.agents.req.StartsRequest.process(StartsRequest.java:109)
	at com.rultor.agents.AbstractAgent.execute(AbstractAgent.java:70)
	at com.rultor.spi.Agent.execute(Agent.java:84)
	at com.rultor.Routine.process(Routine.java:190)
	at com.rultor.Routine.safe_aroundBody0(Routine.java:170)
	at com.rultor.Routine.run(Routine.java:1)
	at org.aspectj.runtime.reflect.JoinPointImpl.proceed(JoinPointImpl.java:179)
	at com.jcabi.aspects.aj.MethodInterrupter.wrap(MethodInterrupter.java:108)
	at com.rultor.Routine.safe(Routine.java:165)
	at com.rultor.Routine.run(Routine.java:133)
	at com.jcabi.log.VerboseRunnable.run(VerboseRunnable.java:190)
	at com.jcabi.aspects.aj.MethodScheduler.lambda./run.sh(MethodScheduler.java:194)
	at java.base/java.util.concurrent.Executors.call(Executors.java:539)
	at java.base/java.util.concurrent.FutureTask.runAndReset(FutureTask.java:305)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor.run(ScheduledThreadPoolExecutor.java:305)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor.run(ThreadPoolExecutor.java:635)
	at com.jcabi.log.VerboseThreads.run(VerboseThreads.java:222)
	at java.base/java.lang.Thread.run(Thread.java:833)

+ exit -1
'cid' file is absent, most probably the Docker container wasn't started correctly

@yegor256 I forgot to add merge section in .rultor.yml. Let's try one more time

@yasamprom it won't work like this. .rultor.yml is used from master, not from your branch

@yasamprom I updated .rultor.yml in master: 78b056b

@yegor256 check please

@yasamprom pushed to master

@yegor256 ready to merge

@yegor256

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 4min)


Update UploadField.php to allow configurable GridFieldPaginator

Changed $config->addComponent(new GridFieldPaginator(8)); to allow it to be configured from the config.yml

Closing as per https://github.com/silverstripe/silverstripe-framework/issues/5007#issuecomment-179753935


bazel: Add pre-compiled clang header poc

This adds a custom rule for producing clang pre-compiled headers. The
hope is that by using these we reduce significant duplication in some
larger headers that get recompiled by many dependees.
This works by fetching the normal C++ configuration from bazel's
crosstool, and adding custom flags to compile our pch in the right mode.
This currently has a few limitations / rough edges:

It only supports clang
There's no way with the CcInfo provider to propagate compiler flags
up the dependency tree. Because of this we instead add the required
-include-pch flag to all macros.
Pch files are timestamp sensitive, this means if you add a include to
the pch, build, then remove it and rebuild, it will currently fail.
LLVM master has a flag to ignore this timestamp, but it isn't
currently available in the release version of Xcode on macOS.

Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

Do we have any data re: how this improve the overall build performance locally?

Do we have any data re: how this improve the overall build performance locally?

I'd be willing to do some tests around that. @keith is this in a good enough state for perf testing yet? Is this clang-only, or will it work with gcc?

It would definitely be great if someone more familiar with these issues could test some configurations and see how it looks.
I could look into making it work with gcc as well, off the top of my head I'm not sure how much work that would be, but it might be best if we could prove it out with clang first assuming the performance characteristics would be the same?
I think this is ready for some testing, but definitely still in the alpha state since I haven't tried many configurations (and CI makes it look like there's an issue on non clang or maybe non macOS builds already).

I tried this and get error ERROR: /Users/ggreenway/dev/envoy/upstream/source/extensions/quic_listeners/quiche/platform/BUILD:95:17: C++ compilation of rule '//source/extensions/quic_listeners/quiche/platform:quic_platform_logging_impl_lib' failed (Exit 1)
Also, I had to make the visibility of the test pch public to fix an bazel error in quic.

What target were you building? And what platform?

linux: bazel build --config=clang-pch //source/...

I'll try to repro in docker

Pushed fixes for both of those, sorry for the trouble

This version builds, but everytime I rebuild (with no changes since the previous build), it rebuilds v8, which takes forever, and makes any timing measurements useless. Executing genrule @com_googlesource_chromium_v8//:build;

Interesting. Based on bazel query I see no connections between that target and the pch targets. Would you expect their to be one? Does this query return anything in your configuration?
bazel query<EMAIL_ADDRESS>//source/common/common:common_pch)'

I removed no-cache here for now, I'm not sure I would expect that to help, but it's worth a try

$ bazel query<EMAIL_ADDRESS>//source/common/common:common_pch)'
INFO: Empty results
Loading: 5 packages loaded

Hrm. I wouldn't expect this to cause that then, but you don't see it without it?

Hrm. I wouldn't expect this to cause that then, but you don't see it without it?

Yep, I double-checked that going back to master it does nothing on rebuild when nothing has changed. No idea why this change causes v8 to rebuild.

Thanks for testing. I'll debug early next week
On Fri, Oct 30, 2020 at 15:53 Greg Greenway<EMAIL_ADDRESS>wrote:

Hrm. I wouldn't expect this to cause that then, but you don't see it
without it?
Yep, I double-checked that going back to master it does nothing on rebuild
when nothing has changed. No idea why this change causes v8 to rebuild.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/13788#issuecomment-719834397,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACFJ3R6ZV4AKSE5A7OAVHDSNM7YBANCNFSM4TBSJHHQ
.

--
Keith Smiley

So I actually see this without this change, but only when sandboxing is disabled. It seems like the issue is when the build artifacts are left around the output is not hermetic. Specifically for example some files like the .ninja_deps files are included (which I assume contains timestamps):
@@ -7598,7 +7598,7 @@
 inputs {
   path: "external/com_googlesource_chromium_v8/wee8/out/wee8/.ninja_deps"
   digest {
-    hash: "a378893aa515feadcf3ce240040f076e17778f607507fd348a9e94d9093d8960"
+    hash: "6c83d26de24732e0cf89c8c882968f4d65feae9ccd5a91ea22d5db2110c3597f"
     size_bytes: 1199916
     hash_function_name: "SHA-256"
   }
@@ -7606,8 +7606,8 @@
 inputs {
   path: "external/com_googlesource_chromium_v8/wee8/out/wee8/.ninja_log"
   digest {
-    hash: "4f54dc5aa0dbb71d836b63a9a430ccca3fcdbc7b9123c6d25c2dde5c06d8fdd1"
-    size_bytes: 119886
+    hash: "6c7a83ad170f7331a3c3b149b94ede9aacce2cc1dd0f45f2f2cf9d5f54caaa84"
+    size_bytes: 119928
     hash_function_name: "SHA-256"
   }
 }

I'll debug this some more.

Fix for that is here: https://github.com/envoyproxy/envoy/pull/13866

@ggreenway mind taking a pass at this again?

I got some results. To test, I first did a full build, then timed how long it took to build //source/exe/... after making a comment change in buffer.h. Results were consistent down to +- 1 second across multiple runs.
PCH took 7:17 to rebuild.
Baseline (non-PCH) took 8:12.
So that's a pretty good gain, considering we haven't even added a lot of stuff to the PCH yet. I think the next step is to add some of the protobuf generated headers to the PCH and see if we can get more gains.

I suppose one more sanity test is to benchmark without-pch, and without sandboxing, to make sure the improvement isn't due to the sandboxing. Do you know a bazel invocation that will do that off the top of your head?

You can delete line 51 from the .bazelrc from this change to do that. Or copy the spawn strategy from line 50 and an exclude this new config from the command line

7:53 to rebuild with --spawn_strategy=local

I still think the biggest bang for the buck we will get is the mocks. Have you tried that yet?

I'm trying, but bazel is unhappy. I tried adding a mock header to the test pch, and to do that, I had to add the target as a dependency (to access the header), but that makes a cycle in the dependencies. @keith any idea how to resolve?
ERROR: /Users/ggreenway/dev/envoy/upstream/test/BUILD:41:18: in pch rule //test:test_pch: cycle in dependency graph:
    //test/test_common:contention_lib
.-> //test:test_pch
|   //test:test_pch_libs
|   //test/mocks/access_log:access_log_mocks
`-- //test:test_pch
This cycle occurred because of a configuration option
ERROR: Analysis of target '//test/test_common:contention_lib' failed; build aborted

envoy_pch_library(
    name = "test_pch",
    testonly = True,
    external_deps = [
        "googletest",
    ],
    includes = [
        "gmock/gmock.h",
        "gtest/gtest.h",
        "test/mocks/access_log/mocks.h",
    ],
    visibility = ["//visibility:public"],
    deps = [
        "//test/mocks/access_log:access_log_mocks",
    ],
)

My assumption (without being at my computer) is that the mocks use the bazel macro where I implicitly added the pch as a dep, leading to that cycle. There's some UX question here of how we want to deal with that. For a quick workaround (assuming the dependency tree doesn't go deeper) you could exclude the pch in the macro if the name matches the target you're trying to add. I can work on a long term improvement here though.

I won't have time to fix this up until next week

Sorry I didn't get to this this week, I'm out for the holiday next week but it's on my list for afterwards. In the meantime if anyone has ideas on the UX question above they'd be appreciated!

Still not sure entirely on what we should do for this case, but for now I've made all mocks not depend on the pch, so the pch can therefore depend on all / any mocks. I've added the one from your previous cycle error as an initial dependency for testing, lmk what you think!

Finally got back to this today. Had to apply the following changes to get unblocked at various places:
diff --git a/.bazelrc b/.bazelrc
index 1d343e3303..fecc15e790 100644
--- a/.bazelrc
+++ b/.bazelrc
@@ -47,6 +47,7 @@ build:clang --action_env=BAZEL_COMPILER=clang
 build:clang --linkopt=-fuse-ld=lld

 # Flags for Clang + PCH
+build:clang-pch --config=clang
 build:clang-pch --spawn_strategy=local
 build:clang-pch --define=ENVOY_CLANG_PCH=1

diff --git a/bazel/envoy_test.bzl b/bazel/envoy_test.bzl
index d84cc1eeb3..2cbea6e16f 100644
--- a/bazel/envoy_test.bzl
+++ b/bazel/envoy_test.bzl
@@ -35,15 +35,17 @@ def _envoy_cc_test_infrastructure_library(
         **kargs):
     # Add implicit tcmalloc external dependency(if available) in order to enable CPU and heap profiling in tests.
     deps += tcmalloc_external_deps(repository)
-    pch_deps = []
+    pch_deps = [envoy_external_dep_path("googletest")]
+    pch_copts = []
     if not exclude_pch_dep:
         pch_deps = [repository + "//test:test_pch"]
+        pch_copts = envoy_pch_copts(repository, "//test:test_pch")
     cc_library(
         name = name,
         srcs = srcs,
         hdrs = hdrs,
         data = data,
-        copts = envoy_copts(repository, test = True) + copts + envoy_pch_copts(repository, "//test:test_pch"),
+        copts = envoy_copts(repository, test = True) + copts + pch_copts,
         testonly = 1,
         deps = deps + [envoy_external_dep_path(dep) for dep in external_deps] + pch_deps,
         tags = tags,
@@ -181,7 +183,7 @@ def envoy_cc_test(
         linkopts = _envoy_test_linkopts(),
         linkstatic = envoy_linkstatic(),
         malloc = tcmalloc_external_dep(repository),
-        deps = envoy_stdlib_deps() + deps + [envoy_external_dep_path(dep) for dep in external_deps] + [
+        deps = envoy_stdlib_deps() + deps + [envoy_external_dep_path(dep) for dep in external_deps + ["googletest"]] + [
             repository + "//test:test_pch",
             repository + "//test:main",
             repository + "//test/test_common:test_version_linkstamp",
@@ -228,6 +230,7 @@ def envoy_cc_test_library(
         copts,
         visibility = ["//visibility:public"],
         alwayslink = alwayslink,
+        exclude_pch_dep = True,
         **kargs
     )

@@ -294,7 +297,9 @@ def envoy_py_test_binary(

 # Envoy C++ mock targets should be specified with this function.
 def envoy_cc_mock(name, **kargs):
-    envoy_cc_test_library(name = name, exclude_pch_dep = True, **kargs)
+    envoy_cc_test_library(name = name,
+    #exclude_pch_dep = True,
+    **kargs)

 # Envoy shell tests that need to be included in coverage run should be specified with this function.
 def envoy_sh_test(
diff --git a/test/BUILD b/test/BUILD
index 270a56ff44..15d57ff3e6 100644
--- a/test/BUILD
+++ b/test/BUILD
@@ -43,13 +43,33 @@ envoy_pch_library(
     testonly = True,
     external_deps = [
         "googletest",
+        "spdlog",
     ],
     includes = [
         "gmock/gmock.h",
         "gtest/gtest.h",
+        "spdlog/sinks/android_sink.h",
+        "spdlog/spdlog.h",
+        "test/mocks/access_log/mocks.h",
+        "test/mocks/network/mocks.h",
+        "test/mocks/http/mocks.h",
+        "test/mocks/runtime/mocks.h",
+        "test/mocks/server/factory_context.h",
+        "test/mocks/common.h",
+        "test/mocks/event/mocks.h",
+        "test/mocks/stats/mocks.h",
+        "test/mocks/server/instance.h",
     ],
     visibility = ["//visibility:public"],
     deps = [
         "//test/mocks/access_log:access_log_mocks",
+        "//test/mocks/network:network_mocks",
+        "//test/mocks/http:http_mocks",
+        "//test/mocks/runtime:runtime_mocks",
+        "//test/mocks/server:factory_context_mocks",
+        "//test/mocks:common_lib",
+        "//test/mocks/event:event_mocks",
+        "//test/mocks/stats:stats_mocks",
+        "//test/mocks/server:instance_mocks",
     ],
 )
diff --git a/test/mocks/http/conn_pool.h b/test/mocks/http/conn_pool.h
index fcfb5e090c..cdca565449 100644
--- a/test/mocks/http/conn_pool.h
+++ b/test/mocks/http/conn_pool.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <memory>

 #include "envoy/http/conn_pool.h"
diff --git a/test/mocks/network/BUILD b/test/mocks/network/BUILD
index 95f9fdb989..1a541d1076 100644
--- a/test/mocks/network/BUILD
+++ b/test/mocks/network/BUILD
@@ -59,7 +59,6 @@ envoy_cc_mock(
         "//source/common/network:utility_lib",
         "//source/common/stats:isolated_store_lib",
         "//test/mocks/event:event_mocks",
-        "//test/mocks/stream_info:stream_info_mocks",
         "//test/test_common:printers_lib",
         "@envoy_api//envoy/config/core/v3:pkg_cc_proto",
     ],
diff --git a/test/mocks/network/mocks.h b/test/mocks/network/mocks.h
index 27f9782cdf..e94065595e 100644
--- a/test/mocks/network/mocks.h
+++ b/test/mocks/network/mocks.h
@@ -20,7 +20,6 @@

 #include "test/mocks/event/mocks.h"
 #include "test/mocks/network/connection.h"
-#include "test/mocks/stream_info/mocks.h"
 #include "test/test_common/printers.h"

 #include "gmock/gmock.h"

Then I hit something unexpected when I added that missing #pragma once:
[0 / 3] [Prepa] BazelWorkspaceStatusAction stable-status.txt
ERROR: /source/test/server/admin/BUILD:25:14: C++ compilation of rule '//test/server/admin:admin_test' failed (Exit 1)
fatal error: file '/build/.cache/bazel/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/test/mocks/http/conn_pool.h' has been modified since the precompiled header 'bazel-out/k8-fastbuild/bin/test/test_pch.pch' was built: size changed
note: please rebuild precompiled header 'bazel-out/k8-fastbuild/bin/test/test_pch.pch'
1 error generated.

So it seems like some dependency is messed up, because I would have expected a rebuild of the test_pch before anything else that depends on it.

I think I figured out how to fix it. Add this patch:
diff --git a/bazel/pch.bzl b/bazel/pch.bzl
index dca1530c0a..cc30a6b770 100644
--- a/bazel/pch.bzl
+++ b/bazel/pch.bzl
@@ -63,13 +63,17 @@ def _pch(ctx):
         variables = cc_compile_variables,
     )

+    deps = []
+    for i, d in enumerate(ctx.attr.deps):
+         deps.append(d[CcInfo].compilation_context.headers)
     ctx.actions.run(
         executable = cc_compiler_path,
         arguments = command_line,
         env = env,
         inputs = depset(
             items = [generated_header_file],
-            transitive = [cc_toolchain.all_files],
+            transitive = [cc_toolchain.all_files] + deps,
         ),
         outputs = [pch_file],
     )

But someone who understands bazel better should double-check that this is right.

Timing results (all use fastbuild config):
rebuild most of envoy and tests, but not external deps
change include/buffer/buffer.h, build //test/...
pch:
0.05user 0.07system 20:36.72elapsed 0%CPU (0avgtext+0avgdata 8708maxresident)k
0.16user 0.04system 20:37.28elapsed 0%CPU (0avgtext+0avgdata 8744maxresident)k
non-pch:
0.12user 0.09system 22:12.36elapsed 0%CPU (0avgtext+0avgdata 8716maxresident)k
0.09user 0.09system 22:13.18elapsed 0%CPU (0avgtext+0avgdata 8756maxresident)k
rebuild single slow test target
change protocol_integration_test.cc, build //test/...
pch:
0.00user 0.00system 0:32.10elapsed 0%CPU (0avgtext+0avgdata 8704maxresident)k
0.00user 0.01system 0:31.60elapsed 0%CPU (0avgtext+0avgdata 8756maxresident)k
non-pch:
0.00user 0.00system 0:41.20elapsed 0%CPU (0avgtext+0avgdata 8644maxresident)k
0.00user 0.01system 0:40.56elapsed 0%CPU (0avgtext+0avgdata 8632maxresident)k
Tests run with the following pch:
envoy_pch_library(
    name = "test_pch",
    testonly = True,
    external_deps = [
        "googletest",
        "spdlog",
    ],
    includes = [
        "gmock/gmock.h",
        "gtest/gtest.h",
        "test/mocks/access_log/mocks.h",
        "test/mocks/network/mocks.h",
        "test/mocks/http/mocks.h",
        "test/mocks/runtime/mocks.h",
        "test/mocks/server/factory_context.h",
        "test/mocks/common.h",
        "test/mocks/event/mocks.h",
        "test/mocks/stats/mocks.h",
        "test/mocks/server/instance.h",

        "spdlog/sinks/android_sink.h",
        "spdlog/spdlog.h",
        "envoy/config/route/v3/route_components.pb.h",
        "envoy/config/core/v3/config_source.pb.h",
        "envoy/extensions/filters/network/http_connection_manager/v3/http_connection_manager.pb.h",
        "envoy/service/discovery/v3/discovery.pb.h",
        "envoy/config/cluster/v3/cluster.pb.h",
        "envoy/config/bootstrap/v3/bootstrap.pb.h",
        "envoy/config/core/v3/base.pb.h",
    ],
    visibility = ["//visibility:public"],
    deps = [
        "//test/mocks/access_log:access_log_mocks",
        "//test/mocks/network:network_mocks",
        "//test/mocks/http:http_mocks",
        "//test/mocks/runtime:runtime_mocks",
        "//test/mocks/server:factory_context_mocks",
        "//test/mocks:common_lib",
        "//test/mocks/event:event_mocks",
        "//test/mocks/stats:stats_mocks",
        "//test/mocks/server:instance_mocks",

        "@envoy_api//envoy/config/route/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/core/v3:pkg_cc_proto",
        "@envoy_api//envoy/extensions/filters/network/http_connection_manager/v3:pkg_cc_proto",
        "@envoy_api//envoy/service/discovery/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/cluster/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/bootstrap/v3:pkg_cc_proto",
    ],
)

envoy_pch_library(
    name = "common_pch",
    external_deps = [
        "spdlog",
    ],
    includes = [
        "spdlog/sinks/android_sink.h",
        "spdlog/spdlog.h",
        "envoy/config/route/v3/route_components.pb.h",
        "envoy/config/core/v3/config_source.pb.h",
        "envoy/extensions/filters/network/http_connection_manager/v3/http_connection_manager.pb.h",
        "envoy/service/discovery/v3/discovery.pb.h",
        "envoy/config/cluster/v3/cluster.pb.h",
        "envoy/config/bootstrap/v3/bootstrap.pb.h",
        "envoy/config/core/v3/base.pb.h",
    ],
    visibility = ["//visibility:public"],
    deps = [
        "@envoy_api//envoy/config/route/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/core/v3:pkg_cc_proto",
        "@envoy_api//envoy/extensions/filters/network/http_connection_manager/v3:pkg_cc_proto",
        "@envoy_api//envoy/service/discovery/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/cluster/v3:pkg_cc_proto",
        "@envoy_api//envoy/config/bootstrap/v3:pkg_cc_proto",
    ],
)

The includes protos and mocks were chosen by counting the number of times a #include line for protos mocks occurred and sorting by frequency, and taking the top several hits.

Pretty nice improvement, especially for the single test case. Nice!

@keith Do you think you'll be able to package this up so we can merge it?

@keith Do you think you'll be able to package this up so we can merge it?

Updated with some of your fixes here, thanks for those. I did have to skip the BUILD file formatter because it believes the pch deps are unused, I'll probably have to look at that, lets see what CI says here though.

Updated with some of your fixes here, thanks for those. I did have to skip the BUILD file formatter because it believes the pch deps are unused, I'll probably have to look at that, lets see what CI says here though.

Also FYI I couldn't take the --config=clang addition since that isn't compatible with macOS

Also FYI I couldn't take the --config=clang addition since that isn't compatible with macOS

hopefully fixed the CI issue when the pch is disabled

hopefully fixed the CI issue when the pch is disabled

I'm suspicious that this change is responsible for the new failures. I'll rebase and triage monday

I'm suspicious that this change is responsible for the new failures. I'll rebase and triage monday

/retest

/retest

i need to rebase but also hoping @lizan can get a chance to look at this soon

looks like this is failing DCO?
/wait

ugh, I edited that in the github UI, thanks fixed

/retest

@lizan when you get a chance can you re-review?

Thanks everyone!


Fix confusion about TreeItem.add_button ambiguity between id and index

A TreeItem-button has an id and an index, which can be distinct.
fix #70423 which shows that this can be confusing.
This PR documents the distinction between id and index more clearly.

We've discussed it with a production team a bit, and it's probably not worth doing the renames at this time for something like this. It's poor, but what can we do, we need to release at some point and it's beta.
But I would still try to improve the descriptions of all relevant methods, and also improve the parameter names. Namely, idx is such a lazy thing, when we can easily write index and be clear, causing less visual confusion with id. So if you wish to work on this, I think a small overhaul of all button methods is in order. But no renames for the methods themselves, I don't think.

Sounds like a good idea. I will update my PR.

Thanks!


[release-v1.4] Add pause/resume lifecyecle to ordered consumer (#2438)

Cherry-pick of https://github.com/knative-sandbox/eventing-kafka-broker/commit/a370abc70df0dd788b926adfb587111169c2dc14

/assign @aliok @matzew

/retest

/retest

/retest


feat(Pipeline): factorize() a pipeline into its possibilities

What does this implement/fix? Explain your changes.
This PR implements a way to quickly factorize a pipeline with choices into individual pipelines where those choices are instead fixed (see image below for example).
The default behavior is to assume we want to create a new child for every child in a Choice and that to create this new pipeline, we just mutate the node and set that as it's only child.
if isinstance(node, Choice):
    possible_choices = [node.mutate(nodes=(child,)) for child in node.children]

There are also avenues for dependancy injection (as there are for things like builders and searchspace parsers). This is because we may allow for custom Node types in the future.
Namely the algorithm works on two assumptions:

There is a predicate factor_by: Callable[[Node], bool] to say if we should split which replaces the isinstance in the snippet above
We update a Node to only refer to a single child with assign_child, this replaces the node.mutate(nodes=(child,)) in the snippet above.

if factor_by(node):
    possible_choices = [assign_child(node.copy(), child) for child in children]

Minimal Example / How should this PR be tested?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@LennartPurucker, I've added the features you requested, namely you can now specify a min_depth and max_depth at which factoring will take place. The default behaviour is still to factor everything though.
There's also a test which shows how you can factor by a specific node name which I'll include in the documentation.


Text color properties doesn't work.

Hi, JohnCoates.
When I try to change color for text, but doesn’t work.
This plugin not supported text color?
Can you check this issue. Thanks.
Sketch version 3.8.2 (Latest)
OS El Capitan 10.11.4 (Latest)
//Sass
[type="text"] {
color:#f00;
}

I'm also seeing this issue. Not only does color not update in the Sketch document, it prevents the entire style block from functioning. Example:
[type="text"] {
    color: #ff0000;
    text-transform: uppercase;
}

I don't see the text-transform: uppercase; applied until I delete the color line.
Here's what I'm seeing in the Console:
6/3/16 4:28:09.359 PM Sketch[74268]: rules: {
    "background-color" = "rgb(255, 0, 0)";
    "border-bottom-left-radius" = 10px;
    "border-bottom-right-radius" = 10px;
    "border-top-left-radius" = 10px;
    "border-top-right-radius" = 10px;
    color = "rgb(255, 0, 0)";
    display = block;
    "font-family" = Gotham;
    "font-size" = 20px;
    height = 24px;
    left = 0px;
    offsetLeft = 0;
    offsetParent = "<DOMHTMLElement [LAYER]: 0x11bf9e780 ''>";
    offsetTop = 0;
    "overflow-x" = auto;
    "overflow-y" = auto;
    position = absolute;
    top = 0px;
    width = 163px;
}
6/3/16 4:28:09.359 PM Sketch[74268]: -[MSTextLayer markLayerDirtyOfType:]: unrecognized selector sent to instance 0x7fe8b6d49470
6/3/16 4:28:09.359 PM Sketch[74268]: *** WebKit discarded an uncaught exception in the webView:didFinishLoadForFrame: delegate: <NSInvalidArgumentException> -[MSTextLayer markLayerDirtyOfType:]: unrecognized selector sent to instance 0x7fe8b6d49470
6/3/16 4:28:09.827 PM WindowServer[14470]: send_datagram_available_ping: pid 384 failed to act on a ping it dequeued before timing out.
6/3/16 4:28:13.000 PM syslogd[49]: ASL Sender Statistics

Any resolution?

Ditto. It also prevents many of the example projects from functioning. E.G. the keyboard project won't update styles until you comment out the text color property on line 48 of its Less file. This is a major blocker for the plugin's intended functionality.


allow log_level as option in settings.ini

Closes #272

Do not forget to add this new setting in docs/user-guide/get-started/settings-definition.md


Fix the Blazor about dialog

Have to do some gritty work to be able to show a proper version in the
about dialog. The generation of the proper assembly attributes is part
of the new-style csproj, so we don't get to use it for free.

@rynowak @danroth27 I wasn't sure whether this should be in release/0.1.0, but it looks like it, so I have cherry-picked it over there for inclusion in 0.1.0. If this was incorrect please let me know and we can rebuild.

Yes, this should be in 0.1.0.

I thought I had already done this 😲


Prevent mobile map rotate

May not be possible without also disabling zoom https://github.com/mapbox/mapbox-gl-js/issues/3371


[WIP] Dev/things

Description of Change
This seems to make it better.

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


FileBeat 5044 Port Problem

i write configure logstash:
i add input:
input {
	beats{
		port => 5044
	}
}

and then add port in docker-compose.yml:
logstash:
    build:
      context: logstash/
    volumes:
      - ./logstash/config/logstash.yml:/usr/share/logstash/config/logstash.yml:ro
      - ./logstash/pipeline:/usr/share/logstash/pipeline
    ports:
      - "5000:5000"
       - "5044:5044"
    environment:
      LS_JAVA_OPTS: "-Xmx256m -Xms256m"
    networks:
      - elk
    depends_on:
      - elasticsearch

so i am use command: systemctl  start filebeat.service
and filebeat -e -c filebeat.yml
FileBeat Error:
2017/12/06 12:04:23.453887 crawler.go:44: INFO Loading Prospectors: 1
2017/12/06 12:04:23.454878 registrar.go:150: INFO Starting Registrar
2017/12/06 12:04:23.496609 prospector.go:103: INFO Starting prospector of type: log; id:<PHONE_NUMBER>067760114
2017/12/06 12:04:23.504035 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/EntrustTest.log
2017/12/06 12:04:23.506145 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/FlowMB.log
2017/12/06 12:04:23.598994 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/Master.log
2017/12/06 12:04:23.600621 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/guankeapi.log
2017/12/06 12:04:23.601833 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/B2B.YYJZT.log
2017/12/06 12:04:23.604193 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/BankPayment.log
2017/12/06 12:04:23.605806 harvester.go:207: INFO Harvester started for file: /home/ubuntu/logs/EntrustDistribution.log
2017/12/06 12:04:23.821100 crawler.go:78: INFO Loading and starting Prospectors completed. Enabled prospectors: 1
2017/12/06 12:04:23.821622 reload.go:128: INFO Config reloader started
2017/12/06 12:04:23.829547 reload.go:220: INFO Loading of config files completed.
2017/12/06 12:04:25.382005 output.go:74: ERR Failed to connect: dial tcp <IP_ADDRESS>:5044: **getsockopt: connection refused**
2017/12/06 12:04:27.383202 output.go:74: ERR Failed to connect: dial tcp <IP_ADDRESS>:5044: **getsockopt: connection refused**
2017/12/06 12:04:31.383973 output.go:74: ERR Failed to connect: dial tcp <IP_ADDRESS>:5044: **getsockopt: connection refused**
2017/12/06 12:04:39.385356 output.go:74: ERR Failed to connect: dial tcp <IP_ADDRESS>:5044: **getsockopt: connection refused**

so i use excuted  command :netstat -anp|grep 5044.
and then just like this:
tcp6       0      0 :::5044                 :::*                    LISTEN      5399/docker-proxy

so what should i do? would you help me?

The indentation of your port 5044 definition is off in the docker-compose.yml, that could be a reason. Could you provide the logs of your Logstash container?


Convert existing implementation to a non-default advanced mode

To reduce complexity with using AEM Vite it will be good to provide the existing implementation as an advanced mode that is opt-in while providing the same level of capability as v2.

Closing as this functionality will be handled by https://www.aemvite.dev/guide/front-end/vite-plugin/


Keyboard Shortcut in Mac

What is the best way to add keyboard shortcut in Mac Lightroom CC? Thanks. I tried adding the standard way for any app Show Focus Point but it wasn't working.

It worked. Just have to give three spaces at the beginning of the menu title.


Upgrade parser to newest version.

https://github.com/pingcap/tidb/tree/master/parser

confirm v6.1.5 version?


Feature/develop v1.0.0

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@582bd63). Click here to learn what that means.
The diff coverage is 97.94%.

@@            Coverage Diff            @@
##             master      #49   +/-   ##
=========================================
  Coverage          ?   97.31%
=========================================
  Files             ?       59
  Lines             ?     2162
  Branches          ?      168
=========================================
  Hits              ?     2104
  Misses            ?       58
  Partials          ?        0

Impacted Files
Coverage Δ

...tructure/repository/ChainedVendorListRepository.js
100% <100%> (ø)

src/cmp/domain/vendor_consents/VendorConsents.js
91.83% <100%> (ø)

...mp/domain/vendor_consents/vendorConsentsCreated.js
100% <100%> (ø)

src/cmp/domain/event_bus/DomainEventBus.js
100% <100%> (ø)

src/globalstorage/application/globalStorage.js
100% <100%> (ø)

...lstorage/application/service/WriteCookieUseCase.js
100% <100%> (ø)

src/cmp/domain/consent/consentValidation.js
100% <100%> (ø)

src/cmp/domain/vendor_consents/whitelistFilter.js
100% <100%> (ø)

src/cmp/infrastructure/configuration/defaults.js
100% <100%> (ø)

...frastructure/repository/IframeConsentRepository.js
100% <100%> (ø)

... and 49 more

Continue to review full report at Codecov.

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


ng-ztree rendering performance

What problem does this feature solve?
render ng-ztree more fast with big data;
When we use more than 1500 one-time nodes in  ng-zree. Its rendering is very worse. At least 10s from my observation.
What does the proposed API look like?
[online demo]  https://stackblitz.com/edit/antd-multi-nodes?file=src/app/app.component.ts

组件套组件，无限迭代产生的。


feat!: Replace socket.io-client

Not done, but throwing up a draft to show some work in progress/create a place to discuss. Proposed solution to #90
"Live" Node compatibility, outside of unit tests, still need to be tested.
Will have to be a major version release - since this also exposes the actual client there has always been a possibility to directly interact with it, and any methods on the previous one that is not implemented in the new one will break. But that should be a fairly uncommon thing.

Looks like our tests don't like ws being a peer dependency. Any way to work around that @jimmycallin?

Looks like our tests don't like ws being a peer dependency. Any way to work around that @jimmycallin?

add it as a devDependency as well!

Looks like our tests don't like ws being a peer dependency. Any way to work around that @jimmycallin?

add it as a devDependency as well!

Done.
Worth noting is that the ws dependency just being there have no impact on the compiled size - it doesn't get automatically added to the final bundle even when it's in the "normal" dependencies. But this is neater.

@jimmycallin - didn't really expect a full code review yet. Greatly appreciated. If you feel I should split it into parts for easier reviewing, just say the word.

@jimmycallin - something I would appreciate help with is if I could do any improvements to the WebSocket-mocking for the tests. Tried adding mock-socket, but it didn't play nice with msw.

We love seeing the progress on this! Truly awesome stuff! We so hope it'll make it into production!

I'd say it's looking increasingly more likely, now that I've started writing a test suite and at least in the browser seem to be at approximate feature parity. I need to polish the reconnect feature a little bit, write a few more tests and verify it actually working as it should in Node. Barring catastrophic stops on the way I'm now looking to get this in.

That is amazing news! Yaaaay!!

Considering this ready for review. Starting some integration tests with it myself now. If this is too large to review well, I can try splitting it into smaller pieces.

It's getting close. It's currently a bit too eager with reconnecting and that will need to be fixed before final testing can begin.

Great to hear! Thanks for the regular updates! Ftrack is an amazing company in terms of this kind of stuff! State of the art! 😍

Is the too eager connecting really preventing release though? I mean, technically, it wasn't even working before - right?

That part did work previously and needs to be fixed before this goes forward. It's an issue that affects the use of the event hub in the ftrack application, and therefore we can't have any major regressions there. It shouldn't be a major issue to fix - I just have to find time for it.  But once that is done I'll try to get a pre-release of 2.0 out.

That sounds great. Is there anything we can do to help?

Maybe I can at least help resolve the conflicts, but I think that'd require write access.

Didn't find quite the time to spend on this that I had hoped, and my Friday afternoon brain didn't quite get me all the way there, but the reconnect behavior is better now. If you've got time @ffMathy it would be very helpful if you could check the test reconnect method exponentially increase delay for every attempt and see if the error for that is in the test or in the code.

I think I've fixed the reconnect behavior now, but didn't have time to properly test it. I'll also have to look at the code again later to see if it needs to be cleaned up.

That's awesome stuff! I'll throw an additional review at it ASAP.

LGTM! 😍

Loving the progress so far! We're cheering for you in my team. Keep up the good work! ❤

Yaaaay!


Any way to display battery percentage while on lockscreen?

Feature Request
When I open the lid i dont want to unlock the laptop to see if it is charged fully.
Implementation
Like a little bar on the side/top
Have you planned working on this feature?
No

Do the work yourself bum, dont recommend things that (arent) as important.

Sorry, i didnt thought that I would be this controversial.
i asked for this feature cause who knows of anyone needs it and i have no idea how to write this. I mean i dont even know that i could do this… sorry. I didn’t wanted to upset you. But im totally beginner in all the linux and open source stuff…

Nah man im just playing around. As for now, there isnt a way to display battery percentage while on lockscreen. I also would like to apologize for my trolling actions, but may you not think negatively on the Linux community (and overall the open source community) due to my foolish actions.

Its okay. No worries, just from you i wont think that a whole community is bad or trash or i dont know. Its okay.

@Lolis4TheWin I'm sorry this happened to you and I wasn't able to react earlier on this! This person is not affiliated with the project at all and I've taken necessary actions for this behavior and removed the offensive comment.
For your original request, I assume it would be better to report the request to i3lock-color as we are only a shell-script wrapper generating the required (composed) images especially for multi-screens, caching them and forwarding options. As the image given to i3lock-color is static we can't really include dynamic elements without hacks which I personally would like to avoid. But until you are satisfied with the answer or would like the request to stay open here as well, I will leave it open :)

Nah man im just playing around. As for now, there isnt a way to display battery percentage while on lockscreen. I also would like to apologize for my trolling actions, but may you not think negatively on the Linux community (and overall the open source community) due to my foolish actions.

The only reason I do not entirely ban you is your apology - if something like this will happen again, I will block you on all projects I'm affiliated/in charge. Everything else is up to GitHub which I've reported this incident.

I fully understand SebTM, thank you for reporting me to Github (and also giving me another chance). I will also like to apologize to you (and the Betterlockscreen project) for my behaviour. For those reading this and my orginal comment, I do not represent them in anyway shape or form, all disrespect shall be held against me and not to anyone affiliated with said project. I am sorry for my actions

@Lolis4TheWin I'm sorry this happened to you and I wasn't able to react earlier on this! This person is not affiliated with the project at all and I've taken necessary actions for this behavior and removed the offensive comment.
For your original request, I assume it would be better to report the request to i3lock-color as we are only a shell-script wrapper generating the required (composed) images especially for multi-screens, caching them and forwarding options. As the image given to i3lock-color is static we can't really include dynamic elements without hacks which I personally would like to avoid. But until you are satisfied with the answer or would like the request to stay open here as well, I will leave it open :)

Thank you! I think i will go to i3-lock with this. Thanks all of you for your time and help. I think this request could be closed, i have got my answers. Have a great day.

@Lolis4TheWin You are welcome - thanks for letting me know and have nice holiday and Christmas (if applicable) :)
Please note we require the usage of i3lock-color


fix typo error and added prims algorithm

Please give this repo a star and follow me for follow back 😊

Thank you for merging my pull request . I have given this repo a star ⭐ and also started following you don't forget to follow me back 😀😀


standard.engine => This setting does not support per-language configuration.

What version of this package are you using?
Version: v2.1.0

What problem do you want to solve?
This setting is only supported as a top level setting: "standard.engine": "ts-standard"

What do you think is the correct solution to this problem?
Will be good to support this setting per language so I can globally set standard for js and ts-standard for ts
  "[javascript]": {
    "standard.engine": "standard",
    "editor.defaultFormatter": "standard.vscode-standard"
  },
  "[javascriptreact]": {
    "standard.engine": "standard",
    "editor.defaultFormatter": "standard.vscode-standard"
  },
  "[typescript]": {
    "standard.engine": "ts-standard",
    "editor.defaultFormatter": "standard.vscode-standard"
  },
  "[typescriptreact]": {
    "standard.engine": "ts-standard",
    "editor.defaultFormatter": "standard.vscode-standard"
  },

Are you willing to submit a pull request to implement this change?
I don't have enough experience working on vs code extensions, I can help if you tell me what to do

Hey! :wave: @LuisEnMarroquin
Thanks for your report!
If you are in a project using both JavaScript and TypeScript files, you should only use ts-standard and not standard.
So we don't need to specify the engine to use per language.
Please close the issue, if it solves your problem.


fix number_format parameter count

fix #75

thanks for your contribution. I release a new version now.


docs: add keyfield to the Table examples

[ ] I have followed (at least) the PR section of the contributing guide.
@90milesbridge/tigger

Pull Request Test Coverage Report for Build 3037

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.979%

Totals

Change from base Build 3031:
0.0%

Covered Lines:
2666

Relevant Lines:
3181

💛  - Coveralls


Performance issues?

I've been using my laptop for some coding and I keep running into massive performance issues with atom-typescript (I've tried disabling it, and the problems disappeared).
I can be typing and Atom's editor will completely freeze/hang for up to 20 seconds, sometimes even longer.  During that time, CPU use will spike and one of the Atom processes will take up close to 1GB of RAM until both suddenly level off again and Atom responds again like nothing ever happened.  This doesn't seem to affect system UI elements like the menu, although closing Atom during this time also doesn't work.
I'm not sure why this is so, but these extended freezes can happen every minute or every few minutes and it's getting to the point where it's just excrutiating to get any work done - so I'd love some insight or see if something could be done there.
Things worth pointing out (off the top of my head)

These lock-ups haven't happened on my desktop machine
This has happened even on tiny files (ie a handful of lines; although I'll have other files open too)
My "pending request" always skyrockets, it's easily at 400 to 500

Developer Tools screenshot:

http://i.imgur.com/LtFly5b.png

Laptop details:

Windows 10 Home 64-bit
Intel Core i7-2670QM @ 2.2 GHz
8GB RAM

I get this as well. Right now it has been frozen for over five minutes. I can confirm that disabling this package prevents this from happening. My "pending requests" seems to max out at 300. My CPU utilization doesn't spike as far as I can tell, but there is an Atom background process that is taking up almost a Gig. This is happening on my desktop rig that has a Skylake i5 and 16GB of RAM.


[Pricing Page. Admin cabinet. Gear. Add location] New city doesn't appear on the drop-down list of the specified field 'Місто' on the page 'Тарифи' after creating new location

Environment: Google Chrome Версія 107.0.5304.88 (64-розрядна версія); Windows 10 Pro.
Reproducible: always.
Preconditions

The user should be authorized as SuperAdmin
Go to https://www.testgreencity.ga/GreenCityClient/#/ubs-admin/tariffs
Main UBS menu - page 'Тарифи' should be opened.

Steps to reproduce

Click on 'Gear' - "Додати" - "Локація"
Fill out the field 'Область' by valid data
Start typing and choose the city in drop down list that is related to the region, chosen in the field 'Місто'
Tab on the link 'Додати місто'
Press the bttn 'Додати'
Check if the new city added to  the drop-down list of the specified field 'Місто' on the page 'Тарифи'

Actual result
New city doesn't appear on the drop-down list of the specified field 'Місто' on the page 'Тарифи'
Expected result
New city should appear on the drop-down list of the specified field 'Місто' on the page 'Тарифи'
User story and test case links
E.g.: "User story #3859
Test case"
Test case"
Labels to be added
"Bug", Priority ("pri: high"), Severity ("severity: minor"), Type ("UI, "Functional").

Not reproduce

Bug is fixed


ISIC

https://www.isic-archive.com/#!/topWithHeader/wideContentTop/main
Veronica Rotemberg will fill out the form or follow-up here on the repo about adding this resource.

ISIC Archive was added Jan 2023 with commit c6cdffb


BidAdapter: Connatix Bid Adapter

Documentation for the new Connatix bid adapter.
🏷 Type of documentation

[x] new bid adapter
[ ] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> https://github.com/prebid/Prebid.js/pull/10186
[x] For new adapters check submitting your adapter docs

Preview URL: https://deploy-preview-4713--prebid-docs-preview.netlify.app/dev-docs/bidders/connatix.html

@alex-calauz-cnx - sorry, but there are linter errors for no-hard-tabs - please resolve or grant us permission to your fork to resolve for you.

Hi @bretg , I fixed the linter errors. Thanks

Hello @bretg , @muuki88 . The Connatix bid adapter was released yesterday, but the documentation (aka .md from this PR) appears to be missing from https://docs.prebid.org/dev-docs/bidders.html.
Can we merge this PR as well, so that publishers that want to integrate our adapter have the necessary documentation?
Thanks

Thanks for the ping @alex-calauz-cnx
Docs are merged by the docs team, which may have a slight delay.


Some suggestions to Application part

Hi @AlekseiPolkovnikov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Aleksei Polkovnikov). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@AlekseiPolkovnikov Good suggestions, thanks. I'll incorporate these into our master doc (which actually lives in the Azure publishing repo)


Support sql module in usage logging.

Codecov Report

Merging #518 into master will decrease coverage by 2.85%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
- Coverage   93.64%   90.79%   -2.86%
==========================================
  Files          31       31
  Lines        4188     4192       +4
==========================================
- Hits         3922     3806     -116
- Misses        266      386     +120

Impacted Files
Coverage Δ

databricks/koalas/sql.py
94.79% <100%> (+0.05%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
23.85% <33.33%> (-73.32%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/conftest.py
87.87% <0%> (-9.1%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

databricks/koalas/frame.py
92.15% <0%> (-1.86%)
:arrow_down:

databricks/koalas/namespace.py
88.23% <0%> (-1.69%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks! merging.


Initial data should send only one write request

At the moment, we send multiple write requests with each one create event. Change this. See #58

Solved by #65


Snapshot dont gets updated .

I have a problem with my snapshot . Im using a streambuilder on one page and the snapshot in another but the snapshot dont gets updated . I cannot figure out why . This is first where I declare the stream

class Comments2N extends StatefulWidget {
  final snapshot;
  final String id;
  final VoidCallback onTaps;

  final int index;
  final FocusNode myFocusNode;

  const Comments2N(
      {Key key,
      this.snapshot,
      this.comment,
      this.id,
      this.index,
      @required this.onTaps,
      this.myFocusNode})
      : super(key: key);
  @override
  _Comments2NState createState() => _Comments2NState();
}

class _Comments2NState extends State<Comments2N> {
  String currentcomment;
  Stream stream;
  Stream stream2;

  @override
  void initState() {
    super.initState();
    stream = FirebaseFirestore.instance
        .collection('meinprofilsettings')
        .doc(widget.snapshot.data.docs[widget.index].data()["uid"])
        .snapshots();
    stream2 = FirebaseFirestore.instance
        .collection('videos')
        .doc(widget.id)
        .collection('comments')
        .doc(
          widget.comment.data()['id'],
        )
        .collection("commentlikes")
        .doc(uid)
        .snapshots();
  }

  likecomment(String id, String timestamp) async {
 -----
    }
  }

  @override
  Widget build(BuildContext context) {
    return StreamBuilder(
        stream: stream,
        builder: (context, _snapshot) {
          if (_snapshot.connectionState == ConnectionState.waiting) {
            return SizedBox.shrink();
          } else {
            return StreamBuilder(
                stream: stream2,
                builder: (context, snapshot) {

                  if (snapshot.connectionState == ConnectionState.waiting) {
                    return SizedBox.shrink();
                  } else {
                    return
                 CommentsWidget(

                      ----
                          snapshot: snapshot,

                          liking: () {

                            likecomment(widget.comment.data()['id'],
                                DateTime.now().toString());

                          },
                        ));
                  }
                });

And then using the snapshot in the commentswidget
class CommentsWidget extends StatefulWidget {
  final snapshot;
  final VoidCallback liking;

  const CommentsWidget(

    @required this.liking,
    @required this.snapshot,
  });

  @override
  _CommentsWidgetState createState() => _CommentsWidgetState();
}

class _CommentsWidgetState extends State<CommentsWidget> {

  @override
  Widget build(BuildContext context) {
    return Column(children: [
      Container(
          child: ListTile(
        leading: GestureDetector(
-----
trailing: InkWell(
          onTap: () {
            widget.liking();

          },
          child: widget.snapshot.data.exists
              ? Icon(
                  Icons.star,
                  size: 25,
                  color: Colors.yellow,
                )
              : Icon(
                  Icons.star_border_outlined,
                  size: 25,
                ),
        ),

So as you can I call the callback to liked a post when user press the star icon. And that works fine but displaying the star in yellow if the user liked the video does not work correctly . This is only working when I leave the page and come back then I get the correct data . And thats the only case for what im using the snapshot . Hope anyone can help if you need more information please leave a comment
[✓] Flutter (Channel stable, 2.0.6, on macOS 11.3.1 20E241 darwin-x64, locale
    de-DE)
    • Flutter version 2.0.6 at /Users/yavuzatmaca/flutter
    • Framework revision 1d9032c7e1 (3 weeks ago), 2021-04-29 17:37:58 -0700
    • Engine revision 05e680e202
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/yavuzatmaca/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✗] Chrome - develop for the web (Cannot find Chrome executable at
    /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.56.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.22.0

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

Heres the pubsec

environment:
  sdk: ">=2.7.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  cloud_firestore: ^1.0.0

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.2
  firebase_auth: ^1.0.1
  cloud_functions: ^1.0.0
  firebase_core_web: ^1.0.0
  provider: ^5.0.0
  flutter_svg: ^0.19.3
  flutter_spinkit: ^5.0.0
  auto_size_text: ^2.1.0
  firebase_core: ^1.0.0
  get_it: ^6.0.0
  flutter_staggered_grid_view: ^0.3.4
  image_picker: ^0.7.4
  image_cropper: ^1.4.0
  firebase_storage: ^8.0.6
  http: ^0.13.0
  random_string: ^2.1.0
  timeago: ^3.0.2
  video_player: ^2.1.0
  video_compress: ^3.0.0
  flutter_rating_bar: ^4.0.0
  intl: ^0.17.0
  get: ^4.1.4
  audioplayers: ^0.18.3
  share: ^2.0.1
  get_storage: ^2.0.2
  dropdown_search: ^0.5.0
  dropdownfield: ^1.0.3
  firebase_analytics: ^8.0.1
  visibility_detector: ^0.2.0
  flutter_share: ^2.0.0
  url_launcher: ^6.0.3
  flutter_datetime_picker: ^1.5.1
  dismissible_page: ^0.6.1
  flutter_switch: ^0.3.1
  rflutter_alert: ^2.0.2
  shimmer: ^2.0.0
  firebase_messaging: ^9.1.4

Hi @userdontknow
From what I can see, the issue is related to a FlutterFire plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,  providing your flutter doctor -v, your pubspec.yaml, a minimal reproducible code sample that does not use 3rd party plugins, your complete flutter run -v logs running the minimal code sample
and I will reopen it.
Thank you

@TahaTesser  Hey I think I found the problem its not what you said. The problem is that the stream dont get rebuild and I cannot figure out why I think I have to add something like a callback or check . Do you know what I need ? Maybe it is because I call it in initstate ?And thats calling it just one time ?

@TahaTesser And I updated my post with the flutter doctor v  and pubsec

Hi @userdontknow
Comment below providing  a minimal reproducible code sample that does not use 3rd party plugins, your complete flutter run -v logs running the minimal code sample
and I will reopen it.
Thank you


Fix HorizontalExecution merging transformations

prevent OTF to merge if intermediate is used later or is signature parameter
prevent GraphMerging if iteration spaces don't match and any output is API
remove GreedyMerging, since not used anywhere and above issue can't be fixed there easily
also applies OTF repeatedly, which can increase the number of valid matches.

bors try

bors try

bors try


Add ExHashRing namespace all modules

Both this package and libring export a HashRing module without
any top-level namespace. This diff adds a top-level namespace of
ExHashRing to avoid the conflict.
Fixes #3

@scohen I did bump the version in mix.exs. Is there something else I'm missing?

Derp... LGTM; r+


add "rounded" formatter for paren surrounded string

I am not married to the "rounded" formatter and surround2 names.  I could even change surround to take an optional second argument.

I'm a bit disinclined to add this formatter since it doesn't do something you can't do pretty straightforwardly with the existing commands and adds a new global prefix. Currently you'd have to say 'args say in parentheses', with this you'd instead say 'rounded in parentheses'. Do you think the slightly shorter prefix is worthwhile?

I'll let you take a look into the chaining.
I did some archaeology, and it turns out nobody's added a new formatter since knausj_talon was created back in early 2020 :D. I view adding new commands (particularly global ones that add a new prefix) as a tradeoff between usefulness and discoverability/misrecognitions/prefix takeover. You can see another tradeoff case here. I probably wouldn't have had several of the formatters (e.g. padded, slasher, dunder).
We haven't discussed whether others feel the same way though to my knowledge. When we do have these conversations on general topics we're hoping to put the results here: https://github.com/knausj85/knausj_talon/blob/main/CONTRIBUTING.md .
In any case if you think this addition is significantly valuable vs the alternative we can put it in.

FWIW there was a recent thread where someone had trouble using the formatters to insert StringValue, since string is also a formatter that can be chained.

I'm back from traveling and food poisoning, and I still have to do research so I can understand the pros and cons of surround-based-formatters vs empty/inside commands.
Existing surround-based-formatters:

padded, spaces
string, single quotes
dub string, double quotes

Existing1 empty/inside commands:
empty dub string: user.insert_between('"', '"')
empty escaped (dub string | dub quotes): user.insert_between('\\"', '\\"')
empty string: user.insert_between("'", "'")
empty escaped string: user.insert_between("\\'", "\\'")
(inside parens | args): user.insert_between("(", ")")
inside (squares | square brackets | list): user.insert_between("[", "]")
inside (bracket | braces): user.insert_between("{", "}")
inside percent: user.insert_between("%", "%")
inside (quotes | string): user.insert_between("'", "'")
inside (double quotes | dub quotes): user.insert_between('"', '"')
inside (graves | back ticks): user.insert_between("`", "`")

Empty/inside commands are far more numerous than surround formatters.  So, internal consistency is in favor of inside commands. (Though really the empty commands should use the inside terminology.)
For simple usage, we have "rounded some words here" vs "inside parens say some words here".  The inside way is more verbose.  The formatter way uses up valuable first-two-syllables recognition space.
For usage that is equivalent to multiple formatters, we have "rounded dub string kebab some words here" vs "inside parens inside dub quotes kebab some words here" to get ("some-words-here").  Both seem to stack well.  The inside way is more verbose.  The inside way is more friendly to doing things and pieces rather than racing against the timeout clock so much.
I am very open to the idea that I am missing some trickier consequence of formatter vs inside command, but as far as I can tell, the inside command way is fine other than being a bit more verbose and requiring some getting used to.  Perhaps this issue should be closed and I should make other issues for...

Getting rid of or changing empty commands. Ex: "empty string" and "inside string" do the same thing.  Probably add "string" as alternative for "quotes" in "insert dub quotes" command.
Change "string" formatter to "sig string" formatter because it screws up formatters when you want to format something like "StringValue" (from "hammer string value").

The above proposal to close out this PR and make a couple of issues sounds reasonable. Just an FYI we have a fairly long deprecation period when removing commands: https://github.com/talonhub/community/blob/main/core/deprecations.py#L6 .

I just thought of another difference between inside commands and surround formatters.  Inside commands are more awkward to continue typing stuff afterwards because you have to move over the surrounding characters.
If you say "rounded dub string hello there over, space, say apple banana", you get ("hello there") apple banana.  If you want the same thing with inside commands, you would say "inside parens, inside dub quotes, say hello there over, go right second, space, say apple banana".

Perhaps an even better way is a series of "after" commands.  They are like the inside commands, but leave your cursor after stuff.
ex: after parens [<user.text>] [over]: '({text or ""}) '
One problem with the above is that it is inconsistent with the inside commands.  To get (stuff) more words, you do "inside parens, say stuff, tail, say more words" and using my example command would be "after parens stuff over, say more words".

I previously forgot to bring up a big advantage of formatters: there are a lot of commands that use formatters, like formatting selected text, reformatting recently inserted text, formatting a homophone choice, and applying a format in cursorless on a target.
Adding another formatter gives you all of that other functionality for free.  If you add an after/inside command, you only get that exact functionality.  To achieve functionality parity with a formatter, you need the following flavors:

inside
after
surround selection
surround recently inserted text
surround homophone choice
surround cursorless target

Maybe we should have a <user.surround> capture so that we can have generic inside/after/etc <user.surround>: ... commands rather than each surround character set needing more commands.
Also, if we don't like that each formatter takes up precious beginning-syllables-of-command space, then the commands should start with form <user.formatters> rather that starting with the formatter.

Those are a few extra features you get with formatters for sure. I think I can see the rounded formatter being more useful for prose than programming.
In prose you'd probably use them like say I can jump tall buildings over args say in my dreams over go right dot. Versus say I can jump tall buildings over rounded in my dreams over dot. So that is better. Do you imagine you'd want to go back and parenthesise some text you'd spelt out previously using e.g. the reformatter?
In programming you probably want to stay within the parentheses in order to put together multiple arguments. So the args form would probably be more useful. You might want to use a cursorless target to wrap arithmetic operators to deal with precedence issues (e.g. 1 + 2 * 3 vs (1 + 2) * 3), but I think it already has that built in: https://www.cursorless.org/docs/#wraprewrap


Patch version is not correct

I have a solution and last hotfix is 2.119.16. GitInfo thinks that current version is 2.119.14. Where is tag and merge commint for hotfix/2.119.16. How can I find a reason of this calculation mistake?

run a detailed build and you'll see all the information that's being calculated by GitInfo and other targets.
To make the output less verbose, you can just run the GitVersion target, as follows:
msbuild [myproject] /t:GitVersion /v:detailed

I strongly suggest you try out the MSBuild Structured Logger to have a clearer view than a looong log output. You can pass the exact same parameters as above to the tool and it will yield a very nice and even searchable tree of the process.

Closing as we got no further info to diagnose.
Please reopen if you follow the suggested steps and you still see weird behavior.
Thanks


Noisy Pointcloud after launching the stereo_node.launch

Hi,
I just got a new OAK D 2 days ago. I've been tinkering with the camera. I noticed that the camera's giving very noisy pointcloud. I could find a solution online. I've uploaded a bag file in case you want to take a look at the pointcloud and the images.
Kindly look into this.
Thanks,
Harshal

Hi,
Joined the discord channel after posting this issue. I'll be trying out the lastest commit on depthai-core. Will let you know if the performance improves.

Is this ROS1 or ROS2 ?.

I'm using ROS1.
Performance improved significantly after installing the latest version of depthai-core. There's still some noise, but I think that can be removed by filtering.

Some noise is still there since it is raw depth. We are working on some post processing which will be included in upcoming releases

Thanks for letting me know. Looking forward towards newer releases 😄
We can close this issue if you want.

Thank you.


fix node-youtube-dl for windows

As title, in order to make sure I can make Kaku's windows build, I need to rewrite some old codes of node-youtube-dl.
https://github.com/EragonJ/node-youtube-dl/commits/master

Windows build still does not work. Problem with the constants and API keys?

@Mimerme sorry, didn't notice your message. If you are going to report bug, please file another one in issue tracker with detailed information including OS version, Kaku version and error messages, thanks.


Introduce REST-API

As discussed with Victor in today's meeting, i am proceeding with merging this branch.


update npm publish workspaces

What:
The Host repo has a GitHub Action to automatically publish package to npm registry after a release version created.
After splitting the packages, we have to update the npm publish command line to work well with the monorepo structure.
Why:
Using workflow can easily maintain the package publishing and save developers time.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This won't try to publish the root, right?


feat(CX-2382): refresh My Collection when submitting an artwork via SWA flow

The type of this PR is: Feature

This PR resolves CX-2382
Description
Added refreshMyCollection method to my collection when submitting artworks with SWA. When refreshing I am checking GlobalSore, if the artwork is present, I display it. It is done, because after submission artworks are now available immediately. Similar to what we have when editing an artwork in My Collection.
iOS, Upload of one artwork
https://user-images.githubusercontent.com/36167539/155393951-01e90569-b916-431f-a072-34879d7c3519.mp4
iOS (the video is too long, including only screenshot), the look of My Collection after uploading two artworks in a row

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[ ] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] 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

added refresh to my collection when submitting artworks with SWA -daria

iOS user-facing changes

Android user-facing changes

Dev changes

Please, feel free to suggest changes, I feel like it looks a bit messy
I'm not sure about resetting session state, in UseEffest it is triggered only when the application reopens, as I understand