[Bug]: grpc 1.62.1 imports googleapis as bzlmod, but also does http_archive on it.

What happened?
I've got this to happen on my project, using the latest gRPC (1.62.1). I've noticed that some library was linked twice. Normally this would work, but this .dll links with /WHOLEARCHIVE, treating every .o (.obj) file from the .a (.lib) file as if it was linked on .o file level.
And I've noticed that somehow googleapis gets linked twice - once from the bzlmod, the other from the http_archive coming of grpc. That surprise me, as https://github.com/bazelbuild/bazel-central-registry/blob/main/modules/grpc/1.62.1/patches/add_module_bazel.patch#L31 specifically does not mention it during the use_repo - but still it gets in https://github.com/grpc/grpc/blob/master/bazel/grpc_deps.bzl#L394 and it's not in the diff here - e.g. https://github.com/bazelbuild/bazel-central-registry/blob/main/modules/grpc/1.62.1/patches/adopt_bzlmod.patch#L508 (you can see that it's still present)
I've made an additional patch for my project soft fork - https://github.com/malkia/opentelemetry-cpp/blob/main/bazel/grpc_deps_remove_googleapis.patch -> gets referenced here https://github.com/malkia/opentelemetry-cpp/blob/main/MODULE.bazel#L22
But maybe a new 1.62.1.bcr.1 (?) version might be good to have with it.
cc: @keith (and Big thanks for working on that!!!)
Version
Development (host) and target OS/architectures:
Windows
Output of bazel --version:
7.1.1
Version of relevant rules from the WORKSPACE or MODULE.bazel file:
It's in the link above
Language(s) and/or frameworks involved:
C++
How to reproduce
Hard to tell, but probably not needed, but I can add these steps to show it.

Any other information?
No response

++ we should add a new release that only has the 1. originally the problem was since these have a circular dependency they had to be added before they could use each other directly

Latest versions of grpc no longer bring in googleapis via a module extension (see https://github.com/bazelbuild/bazel-central-registry/blob/d770320fe8c51a0c301b6d027f69962b202f5d5e/modules/grpc/1.65.0/patches/adopt_bzlmod.patch#L225)


Strict null checks

Few bugs discovered:

Notifications do not load in associated tab despite notifications existing (reproducible in mocks).
In mocks, marketplace tab will not load.
When a service is uninstalled, it is quickly removed from the service list page, but if I navigate to that service again in the marketplace, it says it is in "Removing" state. Same if I click the "View Service" button from the marketplace page. This is fixed by a page refresh. Reproducible live, and might be reproducible in mocks, but blocked by #2.

Remaining bug: When switch marketplaces, the header and marketplace data do not get updated unless I refresh the page


Heroku app

Hi all,
As I've reached the limit of opening the heroku app, can someone host the R version?
We can walkthrouth the steps again in the lab.
Thanks
Thomas

let me deploy and grant you all the access for - https://worldcovid-dashr.herokuapp.com/

wait; I believe each time we make the change and we would have to re-deploy. So I guess we don't need to deploy until milestone 4


Downloads from Sentinel Fail

Hi,
I'm looking to download a dataset of satellite imagery using the Farmvibes API platform. I seem to have failing downloads. When I run run.reason, it seems to be a key error, any insights into this issue?
failed getting secret with key pc-sub-key from secret store kubernetes: secrets “pc-sub-key” not found
I am currently trying to use the public api key.

Hi, @shenoy10. Thank you for using FarmVibes.AI.
It seems that you have not added your PlanetaryComputer API Key to the cluster, or you have added it with a different key name (and the workflow is expecting "pc-sub-key"). You can add a new secret by following the instructions in our documentation page.
An alternative would be to pass your key as a workflow parameter directly when calling client.run() method.
Could you provide a code snippet of how you are calling your workflow and which workflow you are trying to run?

Solved the issue. Added the proper key.


[Bug]: Dramatic performance drop with ESM since v14.0.0

Bug description
Steps to reproduce the problem:

Set package type as module, install puppeteer v13.7.0 dependency
import puppeteer from 'puppeteer';
Run some script and measure performance
Update puppeteer to v14.0.0 and repeat step 3

For convenience, I created a https://github.com/feesler/puppeteer-perf package to test this issue.
git clone https://github.com/feesler/puppeteer-perf.git
cd puppeteer-perf
npm install
npm test
npm install<EMAIL_ADDRESS>npm test

Script will automatically download Chromium revision 982053 to discard possible difference in browser performance.
Temporary solution to fix performance for ESM packages:
- import puppeteer from 'puppeteer';
+ import puppeteer from 'puppeteer/lib/cjs/puppeteer/puppeteer.js';

Puppeteer version
14.0.0 - 15.3.1
Node.js version
16.15.1 - 18.2.0
npm version
8.1.4
What operating system are you seeing the problem on?
Windows
Relevant log output
Node: v16.15.1
Puppeteer: 13.7.0
Browser: HeadlessChrome/101.0.4950.0
Duration: 77.40569996833801

Node: v16.15.1
Puppeteer: 14.0.0
Browser: HeadlessChrome/101.0.4950.0
Duration:<PHONE_NUMBER>523163

Node: v16.15.1
Puppeteer: 15.3.1
Browser: HeadlessChrome/101.0.4950.0
Duration:<PHONE_NUMBER>687195

Node: v18.2.0
Puppeteer: 13.7.0
Browser: HeadlessChrome/101.0.4950.0
Duration: 72.06540012359619

Node: v18.2.0
Puppeteer: 14.0.0
Browser: HeadlessChrome/101.0.4950.0
Duration:<PHONE_NUMBER>951508

Node: v18.2.0
Puppeteer: 15.3.1
Browser: HeadlessChrome/101.0.4950.0
Duration:<PHONE_NUMBER>256134

I am able to reproduce. It looks like dynamic imports are much worse performing with the ESM loader compared to the CJS loader. I suggest we cache the dynamic imports on our side.

Caching seems to help and the numbers go back down for me: https://github.com/puppeteer/puppeteer/pull/8652 Could anyone confirm?

Checked patch on my environment. Still 30% slower than v13.7.0, but it way more better than last results.
Thanks!

On my machine, it's almost identical with Node 18:
❯ npm test

><EMAIL_ADDRESS>test
> node index.js

Node: v18.4.0
Puppeteer: 13.7.0
Browser: HeadlessChrome/101.0.4950.0
Duration: 47.12399911880493

~/pptr-test/puppeteer-perf main*
❯ npm test

><EMAIL_ADDRESS>test
> node index.js

Node: v18.4.0
Puppeteer: 13.7.0
Browser: HeadlessChrome/101.0.4950.0
Duration: 45.374149322509766

~/pptr-test/puppeteer-perf main*
❯ npm test

><EMAIL_ADDRESS>test
> node index.js

Node: v18.4.0
Puppeteer: 15.3.2
Browser: HeadlessChrome/101.0.4950.0
Duration: 47.71191596984863

~/pptr-test/puppeteer-perf main* 21s
❯ npm test

><EMAIL_ADDRESS>test
> node index.js

Node: v18.4.0
Puppeteer: 15.3.2
Browser: HeadlessChrome/101.0.4950.0
Duration: 42.071983337402344

~/pptr-test/puppeteer-perf main*
❯ npm test

><EMAIL_ADDRESS>test
> node index.js

Node: v18.4.0
Puppeteer: 15.3.2
Browser: HeadlessChrome/101.0.4950.0
Duration: 49.50673818588257


Horvathd swift3

@DavidHorvath Could you please modify the Readme to state that it support swift3?

Does this work well with swift 3.1 as well?


fix(test): Increase time out for process baseline delete test

Description
This change increase the time out for process baseline test. The value is in align with what is used in RiskTest.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

https://redhat-internal.slack.com/archives/C02BE6W562Z/p1701992038443509


Reclient issues requests with duplicate Authorization headers

As in the title, outgoing requests from reclient include the Authorization header twice (with the exact same value)
On paper this doesn't sound too bad but nginx enforces there is only a single authorization header.  This means that folks using nginx as tls termination in front of their RBE cluster (hi, that's me) can't use reclient with authentication.
My configuration was mostly reverse engineered as the docs are a bit all over the place but I think I pieced it all together correctly.
// reproxy.cfg
async_reproxy_termination=true
cas_concurrency=1000
# compression_threshold=0
deps_cache_max_mb=256
depsscanner_address=exec:///Users/samuel/projects/electron/electron/src/buildtools/reclient/scandeps_server
fail_early_min_action_count=4000
fail_early_min_fallback_ratio=0.5
fast_log_collection=true
log_format=text
max_concurrent_requests_per_conn=50
max_concurrent_streams_per_conn=50
min_grpc_connections=50
use_batches=false
use_unified_uploads=true

I'm then also configuring two options via environment variables:
RBE_service: 'my.rbe.com:443',
RBE_experimental_credentials_helper: 'my/creds/helper',

The credential helper I'm using at the moment is just a binary that echos hardcoded json so nothing funky going on there.
Included below is a screenshot from a mitm proxy (burpsuite) showing the 400 response and the duplicate auth header

I tracked down the "duplicate header injection" part to this core code in grpc-go 😅 And I "fixed" the issue locally by building a version of reproxy with these lines removed.  The comment in that file indicates this behavior is intentional so I'm not sure what the path forward is here.
Maybe fixing it so that use_rpc_credentials actually works for credential helper backed requests 🤔 At the moment that only impacts a subset of authentication strategies based on their impl in remote-apis-sdk.  Either that or patching grpc-go to never set duplicate Authorization headers specifically to avoid this issue.  I'm not familiar enough with go or this ecosystem at large to actually offer a change in the right place but if someone has an answer I'm happy to write the code in the place I'm pointed to 😄

Just a heads up: the experimental_credentials_helper flag might change in the next few months (and hence the experimental_ prefix). The way we envision credentials helper to actually work is that it will directly return the headers to be included and reproxy will include only those headers with the request (https://github.com/bazelbuild/reclient/discussions/16). However, it will still use PerRPCCreds so the issue will remain.
That aside, thanks for tracking the issue down - that was some great debugging 😄
The comment indicates that these will be included if both PerRPCCreds are included as part of CallOptions and DialOptions. I have a patch to the remote-apis-sdks where if I remove the PerRPCCreds to clientopts, I no longer see duplicate headers. Can you test and see if this works for you in avoiding the redundant header?

@MarshallOfSound
Separate question: Burpsuite actually didn't work for me, I got an error like:
I0105 17:17:06.645384 4042768 capabilities.go:68] call failed with err=rpc error: code = Internal desc = server closed the stream without sending trailers, retrying.

Is there a specific extension I should be using to make it work with gRPC?

The way we envision credentials helper to actually work is that it will directly return the headers to be included and reproxy will include only those headers with the request

Yup I've been following that issue and any progress on the credential helper work, our current helper returns both the current format and the proposed headers format at the same time so it'll keep working when the switchover is made 😅

Can you test and see if this works for you in avoiding the redundant header?

I actually swapped out the nginx ingress portion of my RBE setup, I can spin it back up again quite quickly to test though this evening.

Separate question: Burpsuite actually didn't work for me, I got an error like:

Ah no, unfortunately burpsuite doesn't successfully proxy GRPC requests 🙃 It just proxies enough of the initial HTTP/2 request to diagnose this issue, I couldn't find a very nice GRPC-capable proxy and burpsuite worked just barely to figure this out so I didn't really look harder 😆

I actually swapped out the nginx ingress portion of my RBE setup, I can spin it back up again quite quickly to test though this evening.

Confirmed this appears to fix it 👍

Ah no, unfortunately burpsuite doesn't successfully proxy GRPC requests 🙃 It just proxies enough of the initial HTTP/2 request to diagnose this issue, I couldn't find a very nice GRPC-capable proxy and burpsuite worked just barely to figure this out so I didn't really look harder 😆

Got it. I was able to setup and make mitmproxy CLI to work (didn't try intercepting requests, but it atleast logs them which gave me sufficient info to confirm the fix).

Confirmed this appears to fix it 👍

thanks for confirming! I'll close this out now, you should see the SDK version bumped in reclient soon.


How to remove OOV and infer text directly  to sql

While running run_sparc_editsql.sh script on my added data I am facing the following issue.
read_data_json data/sparc/train_no_value.json 3034
read_data_json data/sparc/dev_no_value.json 423
continue
OOV! Doc_name
continue

and I am not able to infer a direct query to SQL.
Any suggestions thanks.

@Akshaysharma29 You could try our codebase to support predicting SQL for dialogues on your custom data.

While running run_sparc_editsql.sh script on my added data I am facing the following issue.
read_data_json data/sparc/train_no_value.json 3034
read_data_json data/sparc/dev_no_value.json 423
continue
OOV! Doc_name
continue

and I am not able to infer a direct query to SQL. Any suggestions thanks.

@Akshaysharma29 I am currently facing the same problem, did you manage to solve it?


Question: How to update props based on fetch status

Hello, firstly, I wanted to thank you for the library!
Also, I have a question concerning updating props based on some fetch call status.
Is it somehow possible for isLoading to be updated based on what happens inside of the handleDelete function?
Would like to avoid declaring the modal like this:    <Modal id="my-antd-modal" isLoading={isLoading} handleDelete={handleDelete} />
const [isLoading, setIsLoading] = useState(false)
const modal = useModal(DeleteModal);

const handleDelete = () => {
  setIsLoading(true)
  api.delete("url").then(() => {}).finally(() => setIsLoading(false))
};

modal.show({
  handleDelete,
  isLoading
});

export default NiceModal.create(({ isLoading, handleDelete }) => {
  const modal = useModal();
  return (
    <Modal visible={modal.visible}>
        Are you sure you want to delete?
      <Button onClick={handleDelete} disabled={isLoading}>Confirm</Button>
    </Modal>
  );
});

I had this same question a while back. You can checkout the answers here: https://github.com/eBay/nice-modal-react/issues/2


Demo pages not using HTTPS

Hello,
As stated in your own README:

Make sure you're calling Geolocation APIs (such as geolocator.locate() and geolocator.watch()) from a secure origin (i.e. an HTTPS page). In Chrome 50, Geolocation API is removed from unsecured origins. Other browsers are expected to follow.

Since the demo is not using HTTPS, the demo doesn't ask for location in Google Chrome.
Please put the demo pages under HTTPS, its free using Let's Encrypt now!

Thanks. Using CloudFlare now.
Pls empty you cache if you still get the old site.


add ml-commons release notes for 2.4

Signed-off-by: Bhavana Ramaram<EMAIL_ADDRESS>Description
add ml-commons release notes for 2.4
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #538 (aca899b) into 2.x (7766646) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x     #538   +/-   ##
=========================================
  Coverage     79.97%   79.97%
  Complexity      930      930
=========================================
  Files            92       92
  Lines          3521     3521
  Branches        323      323
=========================================
  Hits           2816     2816
  Misses          583      583
  Partials        122      122

Flag
Coverage Δ

ml-commons
79.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


3 panel group-listing dropdown filter does not apply

In the 3-panel review interface, the dropdown in left-hand side does not apply filter.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to test instance
Click on 'External' in dropdown
Select 'Accepted'
Instead of seeing just Accepted applications, you see ALL applications - Accepted, External Review, etc.
Expected behavior

When user clicks on Accepted, only applications that have that status should appear.
Priority

Medium (causes annoyance, but also fundamental usability issue)

Affected roles

Staff
Reviewers
?

Screenshots

Additional information
The dropdown codeblock is:

<form class="form form__select"><select aria-label="Jump to listing group"><option value="draft">Draft</option><option value="screening">Screening</option><option value="more-information-required">More information required</option><option value="dismissed">Dismissed</option><option value="accepted">Accepted</option></select></form>

This is not so mush a filter as a quick jump menu.

Hmm, that doesn't seem to work as it doesn't "jump" anywhere either!

For my selecting "Accepted" scrolls the left column so "Accepted" section is at the top.

Hmm, OK so maybe there's a bug.
I'm using:
Firefox Developer 82.0b9 (64-bit)
Mac OS 10.15.6

I see the issue in firefox browser, Will fix this


[AMORO-2855] Paimon version bump to 0.8.0

Why are the changes needed?

Close #2855.
Brief change log

-paimon version bump to 0.8.0
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

👍


Add PPU Viewer

An older version of tetanes had a basic PPU viewer allowing you to see the Nametables, CHR patterns, and palettes with the ability to change the scanline in which data was read allowing a user to see when an IRQ happened that switched nametables. This functionality was removed when the UI was overhauled to use egui.
Desired initial MVP features:

[x] Nametable viewer showing all 4 Nametables in a 2x2 grid along with the current Mirroring mode and tile grid lines. Hovering over a tile should show it's details such as the address in memory. Should also include a scanline/cycle slider
[x] CHR viewer showing the left and right pattern tables
[x] Sprite (OAM) viewer showing which sprites are loaded into OAM/Secondary OAM
[x] Palette Viewer showing the current palette with options to change palette colors

Inspiration: https://www.mesen.ca/docs/debugging/ppuviewer.html

Completed in #339


Can't send event update despite being admin

I am an admin of the parent Collective, but when I try to send an update on an event it says I can't because I'm not an admin? I am logged in.

event: https://opencollective.com/ehf-wellington-meet-up-pizza-at-mevo-068b2b5f

I thought it might be because the event was past, so I tried creating a new event on staging - same issue. Can't create an Update.

Probably introduced in https://github.com/opencollective/opencollective-frontend/pull/6537

Fix deployed


Slicing

Changed World_Trace slicing utilities
fixed a bug when finish=0 in get_at_timestamp_range
return_by_reference changed default to False, cause with True dragons
return_by_reference refactored to copy_by_reference
Added equivalent World_QSR_Trace slicing utilities, closes #165

Can be tested with the scripts in the dbg folder.

Can I merge if it passes jenkins?

Since it passes jenkins and does not break the core of the lib, and given that none has used these before as they didn't exist I will merge this PR.


Publish updated SpeakEasy

Glad to see the changes from #21 and #33 were merged in! However, the npm version has not yet been bumped from 1.0.3 that was published three years ago. It'd be great to see a 1.0.4 :smile:

Thanks @connor4312. Bumped version in d27a8a20f9bd874f5d4ecc07dcadb50ce574a5a0 and published to npm. We're also working on a 2.0 in #44 that will incorporate many of those changes.

Great, thanks!


h2_census_binary_metadata_nosec_test.exe fails under Windows, Release

https://grpc-testing.appspot.com/job/gRPC_master/13753/config=opt,language=c,platform=windows/consoleFull

Seen again in 13875


Issues

Traceback (most recent call last):
File "requests\models.py", line 971, in json
File "json_init_.py", line 346, in loads
File "json\decoder.py", line 337, in decode
File "json\decoder.py", line 355, in raw_decode
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "<obf\luna_platinum_patch.py>", line 2, in
File "", line 48, in
File "", line 48, in
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 493, in exec_module
File "<variables_platinum.py>", line 2, in
File "", line 66, in
File "", line 66, in
File "requests\models.py", line 975, in json
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
[21628] Failed to execute script 'luna_platinum_patch' due to unhandled exception!

ask got for help about this error

gpt*


Create bower.json

Hey, @uqee PR is updated

Oh, I forgot: please, also bump a version in the package.json and we're good to go.

updated


Dependency isolation in components

I think we should come up with a good design for dependency isolation.
The current model is not very extensible - each DB and model framework we support gets flat-added to the core dependency set.
This does not scale:

as the number of dependencies approaches infinity, the probability of requirement clashes approaches one
the user is already now forced to install a huge set of dependencies

We should, instead aim to:

minimize core dependencies
ensure the dependencies for any given user's environment are handled smartly.

I think the later we address this, the more this will grow into tech debt.

closing because duplicate of #562


Middleware: Update or Stop Server Error

The following error happened when attempting to either Stop or Reload an EAP server, with MIQ running in OpenShift:
Connection refused - connect(2) for "localhost" port 8080 [middleware_server/button]
Will add MIQ log info.

@miq-bot add_label providers/hawkular.

@mattmahoneyrh yes, logs will be very helpful.
How new is the build that this was discovered on?
There were some issues that sound related with defaulting to localhost in the gem, but they were recently fixed and the fix entered miq.
cc @pilhuhn

@miq-bot add_label providers/hawkular

Retested on more recent MIQ running in OS, but did not repro the error.
I am fine with closing this issue, and reopening if the error is later reproduced.

Retested on more recent MIQ running in OS, but did not repro the error.
I am fine with closing this issue, and reopening if the error is later reproduced.

yes, please close


Stop at first colon when parsing docker-compose.yml volumes #69

See #69 for more details.

Oooh, good catch. Love the gif too :)
The fix looks good, but one request: could you add an automated test for this? See test/docker-osx-dev.bats. Thanks!

Whoa, I learned you can write unit tests for bash scripts - awesome!
Here is the failing test case I added before applying my patch:
 ✗ configure_paths_to_sync correctly reads paths with access modifier
   (from function `assert_equal' in file test/test_helper.bash, line 29,
    in test file test/docker-osx-dev.bats, line 142)
     `assert_equal "/host" "$PATHS_TO_SYNC"' failed
   expected: /host
   actual:   /host:/guest

Awesome, thank you!


News dragging

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #260
Test URLs:

Before: https://main--icicidirect--aemsites.hlx.live/research/equity
After: https://news-dragging--icicidirect--aemsites.hlx.live/research/equity

Few observations,

Hide the horizontal scroll bar, visible when the user is scrolling manually
When the user has scrolled to a position, stop the auto-scroll action, this leads to scrolled items getting changed.

Please see why PSI check is failing. Rest lgtm.

re-run cleared PSI

Few observations,

Hide the horizontal scroll bar, visible when the user is scrolling manually
When the user has scrolled to a position, stop the auto-scroll action, this leads to scrolled items getting changed.

fixed


upgrade Prometheus version to latest main

Upgrade Prometheus version to latest main to unblock upgrading prometheus version in Thanos main repo.

Looks like we might need to bump the go version.

Something seems wrong for this version upgrade. I don't think it is an issue on us. Maybe upstream changed something?
--- FAIL: FuzzEnginePromQLSmithInstantQuery (0.88s)
    --- FAIL: FuzzEnginePromQLSmithInstantQuery (0.23s)
        enginefuzz_test.go:449: load 30s
            			http_requests_total{pod="nginx-1", route="/"} 600.00+10.00x40
            			http_requests_total{pod="nginx-2", route="/"}  8+28.00x40
        enginefuzz_test.go:451: increase({__name__="http_requests_total",route="/"} offset 1m7s[1h:1m] offset -3m33s)
        enginefuzz_test.go:453: case 71 error mismatch.
            new result: {pod="nginx-1", route="/"} => 55.666666666666664 @[44000]
            {pod="nginx-2", route="/"} =><PHONE_NUMBER>6666667 @[44000]
            old result: {pod="nginx-1", route="/"} => 55.666666666666664 @[44000]
            {pod="nginx-2", route="/"} =><PHONE_NUMBER>6666667 @[44000]
        enginefuzz_test.go:458: failed 1 test cases

Probably something around rate and increase changed in upstream?

https://github.com/prometheus/prometheus/pull/13725 maybe this one?


Add reordering options to phpunit.xml too

Desired functionality

add the reverse/random reordering flags to collections of tests (TestSuite, TestCase, directory, mulitple files) as defined in phpunit.xml
provide do not reorder option
finer grained control: make configurable at every level; e.g.: randomize TestSuites, but not TestCases

Current functionality
None, the feature is only available via command line arguments.
Open issues

prevention of reordering can be enforced by explicitly stating dependencies

Closed by https://github.com/sebastianbergmann/phpunit/commit/7bbe522411917130fa59da8bc68a465e68b1df5f and https://github.com/sebastianbergmann/phpunit/commit/3c4456b490f56edb0e59bd72ce3472172a610285


Update knex and nest packages to latest versions

This solves the following:

If you update your knex to the latest version, you will get an error because @nestjsplus/knex is not using latest knex version
You can't run this on newer node version (v14+)

cc @johnbiundo

This will solve the issue https://github.com/knex/knex/issues/3831. This library is practically unusable now.


Localise tabs

Hi,
Following the https://github.com/volkanceylan/serenity-guide/blob/master/tutorials/movies/listing_movies_in_person_dialog.md
How is this possible to localise TAB titles?
In this case - Person, Movies.

Within the followin lines:

Person

Movies

I have not tested this but try the following:
replace the Static text (Person and Movies) with the following:

Q.Text("Dialogs.Tabs.Persons")

Q.Text("Dialogs.Tabs.Movies")

And then add in the file: /scripts/site/Texts/site.texts.invariant.json the following entries:
"Dialogs.Tabs.Person" : "Persons",
"Dialogs.Tabs.Movies" : "Movies"

Now find/create such json files for your desired language.
E.G. for german there is a file called site.texts.de.json.
Add here the same Tokens but with translated text:
"Dialogs.Tabs.Person" : "Personen",
"Dialogs.Tabs.Movies" : "Filme"

Make a full build.
Now if you Change the user's language to german the Tabs should appear in german.
Hope this helps.
With Kind regards,
John

Thanks @JohnRanger

This approach won't work, because Q.Text("Dialogs.Tabs.Persons") in plain HTML is just a text.
But I found workaround.
In html I added ID to the span

Person

And, in PersonDialog.ts added this
document.getElementById("Db.Default.Person").innerText = Q.text("Db.Default.Person.EntitySingular");
Not sure this is an elegant solution, but it works like I need

Look in customer dialog template


'GoogleReverse' object has no attribute 'client'

Hey,
Issue with 1.21 version:
import geocoder
results = geocoder.google(
    location=[1.1, 1.1],
    method='reverse',
)

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-9d35f5ef892c> in <module>()
      1 results = geocoder.google(
      2     location=[1.1, 1.1],
----> 3     method='reverse',
      4 )

**/lib/python3.5/site-packages/geocoder/api.py in google(location, **kwargs)
    184         > elevation
    185     """
--> 186     return get(location, provider='google', **kwargs)
    187
    188

**/lib/python3.5/site-packages/geocoder/api.py in get(location, **kwargs)
    150         if method not in options[provider]:
    151             raise ValueError("Invalid method")
--> 152     return options[provider][method](location, **kwargs)
    153
    154

**/lib/python3.5/site-packages/geocoder/google_reverse.py in __init__(self, location, **kwargs)
     35             'client': kwargs.get('client', google_client)
     36         }
---> 37         if self.client and self.client_secret:
     38             self.params['client'] = self.client
     39             self._encode_params()

AttributeError: 'GoogleReverse' object has no attribute 'client'

Also, it would be helpful if you would keep this repo up to date with all the releases :)
Thanks

👍
I just installed and used the CLI for the first time, and got exactly the same AttributeError.

@yardensachs  @Zearin Sorry about that... last commit broke some things.
This latest patch 1.21.2 should of fixed the above issues.
Really need those Travis-CI tests back up and running.

Thank you for the fix!  The CLI will really help with a project I have at work. :)


Remove error modal and replace with Toast notifications

https://www.npmjs.com/package/ng2-toastr

This is complete.


Fix #1071: support enums

This adds support for Enum and IntEnum classes.

Note the Python 2.7 tests are currently failing because of this issue:
https://bitbucket.org/ambv/singledispatch/issues/8/inconsistent-hierarchy-with-enum

Current coverage is 87.58%

Merging #1829 into master will not affect coverage as of c9a2219

@@            master   #1829   diff @@
======================================
  Files          283     287     +4
  Stmts        50785   51018   +233
  Branches      5270    5292    +22
  Methods
======================================
+ Hit          44481   44683   +202
- Partial        898     900     +2
- Missed        5406    5435    +29

Review entire Coverage Diff as of c9a2219

Powered by Codecov. Updated on successful CI builds.

Hi, sorry for commenting on this old issue, but I am having problems with the documentation section of this merged PR.
It is really hard to understand what "both enum.Enum and enum.IntEnum subclasses are supported" without any example or reference in the documentation.
The intuitive way, of @jitclass-ing a Python Enum throws an AttributeError in my code base. Apparently I am not the only one, see this stack overflow question for a more complete example.
Are there any limitations/preconditions to be met? This should really go in the user documentation.

@muxator thanks for raising this issue! Since this is an old PR here, I would recommend getting a discussion going on https://numba.discourse.group/ to figure out what works and what doesn't and then submitting a fresh PR with any necessary modifications to the Numba documentation.

For example, this works:
import numba
from enum import Enum

class MyEnum(Enum):
    Red = 123
    Blue = 456

@numba.njit
def check_isblue(e):
    assert e == MyEnum.Blue

check_isblue(MyEnum.Blue)

See the file numba/tests/test_enums.py introduced by this pull request.


Update to version 67

This issue tracks the upgrade progress to Chromium 67, subsequent revisions, and related discussion.
Here are the steps necessary towards the first release in the upgrade process:

[x] Update common and other abstract base bundles
[ ] Update other base bundles and associated packaging types:

[ ] Arch Linux
[x] Debian and Ubuntu
[x] macOS
[x] openSUSE
[x] Portable Linux
[x] Windows

NOTE: A base bundle is considered "updated" once its current configuration has been added/removed/changed to produce working builds (i.e. runs without severe regressions) on the new version. It does not always consider new Chromium features.
After the initial release is published, this issue will remain open for status updates and discussions relating to new revisions or the version as a whole. It will be closed once a new Specific problems should still be posted in a separate issue.
Miscellaneous TODO:

[x] build target 'widevinecdmadapter' was removed - References to it and the resulting library need to be removed from the packaging scripts
[x] binary pruning and domain substitution lists need to be updated when the stable version is released

1b9bc64c465543101f7944aee7e8a52087d40344 can build Debian stretch packages that can't play any media. I need to refresh Portable Linux patches and try building.

Status update: Portable Linux seems to be fully functional as of fb6a3a450508ba633914aa6a2924aeb42a620e64 with a build available here

The OpenSUSE build is also having the issue where media cannot be played.

hey, I am building for macOS atm and this line https://chromium.googlesource.com/chromium/src/base/+/master/mac/sdk_forward_declarations.h#288
does not work on 10.11
error: expected ';' after top level declarator
typedef NSString* VNImageOption NS_STRING_ENUM;

Got to replace it with typedef NSString* VNImageOption;
Should be fine on newer systems though.
I am hesitant to creating a patch for this since it looks like an upstream problem and it affects only people on older systems.

Does media playback with the faulty builds produce any logs in the console or on the chrome:gpu, chrome:media-internals pages?

@xsmile I didn't save the logs, but both stderr and chrome://media-internals said something about FFmpegDemuxer not being able to open a context (I tried looking it up, and it's a generic failure trying to load FFmpeg). When I tried opus audio (e.g. YouTube or opus audio files), it just says the audio decoder wasn't able to initialize, or something to that effect. chrome://gpu didn't have anything IIRC, and toggling VA-API acceleration has no effect.
I suspect this may be a FFmpeg incompatibility (not too sure about opus though). I'm just going to wait and see if Debian will push a patch to fix this or not, because I haven't seen anyone else building 67 with old FFmpeg yet.

@Eloston please merge it only after someone on 10.13 tries it first.

@tectiv3 , @Eloston
macOS 10.13.5 - latest build  - 67.0.3396.62-1 causes high CPU usage, details below.

yeah, I noticed this as well. Sometimes one of the tabs starts hogging CPU.

Debian Stretch binaries for 67.0.3396.87-1 have been uploaded for testing.

Building version 67.0.3396.87-1 on Windows seems to fail. Actually the build process doesn't even start.
This is the output I get when running build.bat:
https://paste.debian.net/plain/1030109
Also note that on same setup the build process starts normally for version 66.0.3359.181-1, so I'm guessing this is something inherent to the latest update.

@tectiv3 // @Eloston
Latest release seems to be working fine. I put the macOS binaries here 67.0.3396.87-1

@nixballs no random high CPU usage? It happens once or twice a day for me on my build.

@tectiv3 nope, none so far. @bjrnio also mentioned .79 was supposed to fix the high CPU. Using around 4~5 tabs and CPU is hovering around 5-10%.

It wasn't about tabs count, some random tab was consuming 100% CPU. Happened to me only couple of times in those two weeks. Good to know .79 address that!

Status update: Windows support has been updated. A build is available here. Please test and report any bugs; I will tag a new release after several days if there aren't any major issues.

Latest macOS build (.87) - no issues in four days.

No issues so far on the Windows build mentioned above.

Been using 67 on Windows last few days. No issues so far. Thanks @Eloston, @Wyse-

Hello,
I'm trying to compile this revision (The master)
https://github.com/Eloston/ungoogled-chromium/commit/7c9d189ff0c4ba33d923bc67f47998d4fb6460a2
This version possess one more "commit" to enhance the tag version
https://github.com/Eloston/ungoogled-chromium/tree/67.0.3396.87-2
All is going well until it do not find "kDisableWebRtcHWDecoding"
This var is only available if ENABLE_WEBRTC and OS_CHROMEOS are enable inside "content\public\common\content_switches.cc"
ENABLE_WEBRTC flag seems to be part of media_buildflags inside "media\BUILD.gn" file.
So the build seems to miss a flag, maybe ENABLE_WEBRTC  or OS_CHROMEOS, how can I solve the compilation part ?
Or did I not install the good SDK
My softwares are:
Visual studio Community 2017 - 15.5.27130.2010
Windows Software Development Kit - Windows 10.0.15063.468 is install among others.
Thx to looking at my problem and point me in the good direction.
Project is great !
 -I../.. -Igen -I../../third_party/libwebp/src -I../../third_party/khronos -I../../gpu -I../../third_party/wtl/include -I../../third_party/libyuv/include -I../../third_party/protobuf/src -I../../third_party/ced/src -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../skia/config -I../../skia/ext -I../../third_party/skia/include/c -I../../third_party/skia/include/config -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/encode -I../../third_party/skia/include/gpu -I../../third_party/skia/include/images -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pdf -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/src/gpu -I../../third_party/skia/src/sksl -I../../third_party/libwebm/source -I../../third_party/protobuf/src -Igen/protoc_out -I../../third_party/leveldatabase -I../../third_party/leveldatabase/src -I../../third_party/leveldatabase/src/include -I../../third_party/webrtc_overrides -I../../testing/gtest/include -I../../third_party/webrtc -I../../third_party/webrtc_overrides -I../../third_party/webrtc -Igen/third_party/metrics_proto -I../../third_party/boringssl/src/include -I../../third_party/blink -Igen/third_party/blink -I../../v8/include -Igen/v8/include -I../../third_party/angle/src/common/third_party/base -Igen/angle -I../../third_party/brotli/include -I../../third_party/re2/src -I../../third_party/zlib -I../../third_party/mesa/src/include -I../../third_party/expat/files/lib /utf-8 /X -fcolor-diagnostics -Xclang -mllvm -Xclang -instcombine-lower-dbg-declare=0 /Gy /FS /bigobj /d2FastFail /Zc:sizedDealloc- -fmsc-version=1911 -m64 /W4 -Wimplicit-fallthrough -Wthread-safety /wd4091 /wd4127 /wd4251 /wd4275 /wd4312 /wd4324 /wd4351 /wd4355 /wd4503 /wd4589 /wd4611 /wd4100 /wd4121 /wd4244 /wd4505 /wd4510 /wd4512 /wd4610 /wd4838 /wd4995 /wd4996 /wd4456 /wd4457 /wd4458 /wd4459 /wd4702 -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -Wno-unused-lambda-capture -Wno-user-defined-warnings -Wno-enum-compare-switch -Wno-null-pointer-arithmetic /O1 /Ob2 /Oy- /Zc:inline /Gw /Oi /MT -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -Wshadow /wd4267 /TP /wd4577 /GR- /c gen/content/browser/browser_jumbo_11.cc /Foobj/content/browser/browser/browser_jumbo_11.obj /Fd"obj/content/browser/browser_cc.pdb"
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
In file included from gen/content/browser/browser_jumbo_11.cc:51:
.\../../content/browser/renderer_host/render_process_host_impl.cc(2850,15):  error: no member named 'kDisableWebRtcHWDecoding' in namespace 'switches'
    switches::kDisableWebRtcHWDecoding,
    ~~~~~~~~~~^
.\../../content/browser/renderer_host/render_process_host_impl.cc(2851,15):  error: no member named 'kDisableWebRtcHWEncoding' in namespace 'switches'; did you mean 'kDisableWebRtcEncryption'?
    switches::kDisableWebRtcHWEncoding,
    ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
              kDisableWebRtcEncryption
../..\content/public/common/content_switches.h(251,34):  note: 'kDisableWebRtcEncryption' declared here
CONTENT_EXPORT extern const char kDisableWebRtcEncryption[];
                                 ^
In file included from gen/content/browser/browser_jumbo_11.cc:51:
.\../../content/browser/renderer_host/render_process_host_impl.cc(2886,34):  error: no matching function for call to 'ArraySizeHelper'
                                 arraysize(kSwitchNames));
                                 ^~~~~~~~~~~~~~~~~~~~~~~
../..\base/macros.h(54,34):  note: expanded from macro 'arraysize'
#define arraysize(array) (sizeof(ArraySizeHelper(array)))
                                 ^~~~~~~~~~~~~~~
../..\base/macros.h(53,40):  note: candidate template ignored: could not match 'T [N]' against 'const char *const []'
template <typename T, size_t N> char (&ArraySizeHelper(T (&array)[N]))[N];
                                       ^
3 errors generated.
[17511/21319] CXX obj/content/browser/browser/browser_jumbo_10.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17512/21319] CXX obj/content/browser/browser/browser_jumbo_9.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17513/21319] CXX obj/content/browser/browser/browser_jumbo_6.obj
FAILED: obj/content/browser/browser/browser_jumbo_6.obj
../../third_party/llvm-build/Release+Asserts/bin/clang-cl.exe /nologo /showIncludes  "-imsvcC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.12.25827\ATLMFC\include" "-imsvcC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.12.25827\include" "-imsvcC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\ucrt" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\shared" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\um" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.15063.0\winrt" "-imsvcC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.12.25827\ATLMFC\include" "-imsvcC:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.12.25827\include" "-imsvcC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.16299.0\ucrt" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.16299.0\shared" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.16299.0\um" "-imsvcC:\Program Files (x86)\Windows Kits\10\include\10.0.16299.0\winrt" -DENABLE_SCREEN_CAPTURE=1 -D__ATLHOST_H__ -DV8_DEPRECATION_WARNINGS -DUSE_AURA=1 -DNO_TCMALLOC -DOFFICIAL_BUILD -DCHROMIUM_BUILD -D"CR_CLANG_REVISION=\"328716-2\"" -D_HAS_EXCEPTIONS=0 -D__STD_C -D_CRT_RAND_S -D_CRT_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_DEPRECATE -DDEPRECATEDENUMERATOR(x)=[[deprecated(x)]] -D_ATL_NO_OPENGL -D_WINDOWS -DCERT_CHAIN_PARA_HAS_EXTRA_FIELDS -DPSAPI_VERSION=1 -DWIN32 -D_SECURE_ATL -D_USING_V110_SDK71_ -DWINAPI_FAMILY=WINAPI_FAMILY_DESKTOP_APP -DWIN32_LEAN_AND_MEAN -DNOMINMAX -D_UNICODE -DUNICODE -DNTDDI_VERSION=0x0A000002 -D_WIN32_WINNT=0x0A00 -DWINVER=0x0A00 -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -DCONTENT_IMPLEMENTATION -DV8_USE_EXTERNAL_STARTUP_DATA -DWEBP_EXTERN=extern -DUSE_EGL -DGOOGLE_PROTOBUF_NO_RTTI -DGOOGLE_PROTOBUF_NO_STATIC_INITIALIZER -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DU_STATIC_IMPLEMENTATION -DICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_FILE -DUCHAR_TYPE=wchar_t -DSK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS -DSK_HAS_PNG_LIBRARY -DSK_HAS_WEBP_LIBRARY -DSK_HAS_JPEG_LIBRARY -DSK_SUPPORT_GPU=1 -DGR_GL_FUNCTION_TYPE=__stdcall -DLEVELDB_PLATFORM_CHROMIUM=1 -DDeleteFile=DeleteFileW -DOS_WIN -DWEBRTC_NON_STATIC_TRACE_EVENT_HANDLERS=0 -DGTEST_RELATIVE_PATH -DWEBRTC_CHROMIUM_BUILD -DWEBRTC_WIN -D_CRT_SECURE_NO_WARNINGS -DMESA_EGL_NO_X11_HEADERS -DXML_STATIC -DNO_MAIN_THREAD_WRAPPING -DFLAC__NO_DLL -I. -I../.. -Igen -I../../third_party/libwebp/src -I../../third_party/khronos -I../../gpu -I../../third_party/wtl/include -I../../third_party/libyuv/include -I../../third_party/protobuf/src -I../../third_party/ced/src -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../skia/config -I../../skia/ext -I../../third_party/skia/include/c -I../../third_party/skia/include/config -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/encode -I../../third_party/skia/include/gpu -I../../third_party/skia/include/images -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pdf -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/src/gpu -I../../third_party/skia/src/sksl -I../../third_party/libwebm/source -I../../third_party/protobuf/src -Igen/protoc_out -I../../third_party/leveldatabase -I../../third_party/leveldatabase/src -I../../third_party/leveldatabase/src/include -I../../third_party/webrtc_overrides -I../../testing/gtest/include -I../../third_party/webrtc -I../../third_party/webrtc_overrides -I../../third_party/webrtc -Igen/third_party/metrics_proto -I../../third_party/boringssl/src/include -I../../third_party/blink -Igen/third_party/blink -I../../v8/include -Igen/v8/include -I../../third_party/angle/src/common/third_party/base -Igen/angle -I../../third_party/brotli/include -I../../third_party/re2/src -I../../third_party/zlib -I../../third_party/mesa/src/include -I../../third_party/expat/files/lib /utf-8 /X -fcolor-diagnostics -Xclang -mllvm -Xclang -instcombine-lower-dbg-declare=0 /Gy /FS /bigobj /d2FastFail /Zc:sizedDealloc- -fmsc-version=1911 -m64 /W4 -Wimplicit-fallthrough -Wthread-safety /wd4091 /wd4127 /wd4251 /wd4275 /wd4312 /wd4324 /wd4351 /wd4355 /wd4503 /wd4589 /wd4611 /wd4100 /wd4121 /wd4244 /wd4505 /wd4510 /wd4512 /wd4610 /wd4838 /wd4995 /wd4996 /wd4456 /wd4457 /wd4458 /wd4459 /wd4702 -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -Wno-unused-lambda-capture -Wno-user-defined-warnings -Wno-enum-compare-switch -Wno-null-pointer-arithmetic /O1 /Ob2 /Oy- /Zc:inline /Gw /Oi /MT -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -Wshadow /wd4267 /TP /wd4577 /GR- /c gen/content/browser/browser_jumbo_6.cc /Foobj/content/browser/browser/browser_jumbo_6.obj /Fd"obj/content/browser/browser_cc.pdb"
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
In file included from gen/content/browser/browser_jumbo_6.cc:43:
.\../../content/browser/gpu/gpu_process_host.cc(138,15):  error: no member named 'kDisableWebRtcHWEncoding' in namespace 'switches'; did you mean 'kDisableWebRtcEncryption'?
    switches::kDisableWebRtcHWEncoding,
    ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~
              kDisableWebRtcEncryption
../..\content/public/common/content_switches.h(251,34):  note: 'kDisableWebRtcEncryption' declared here
CONTENT_EXPORT extern const char kDisableWebRtcEncryption[];
                                 ^
1 error generated.
[17514/21319] CXX obj/content/browser/browser/browser_jumbo_15.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17515/21319] CXX obj/content/browser/browser/browser_jumbo_8.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17516/21319] CXX obj/content/browser/browser/browser_jumbo_7.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17517/21319] CXX obj/content/browser/browser/browser_jumbo_5.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17518/21319] CXX obj/content/browser/browser/browser_jumbo_4.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
[17519/21319] CXX obj/content/browser/browser/browser_jumbo_12.obj
clang-cl.exe: warning: argument unused during compilation: '/X' [-Wunused-command-line-argument]
ninja: build stopped: subcommand failed.

Cannot buil linux_portable on Ubuntu 16.04
./out/Default/gn gen out/Default --args="host_cpu=\"x64\" blink_symbol_level=0 clang_use_chrome_plugins=false enable_ac3_eac3_audio_demuxing=true enable_google_now=false enable_hangout_services_extension=false enable_hevc_demuxing=true enable_iterator_debugging=false enable_mdns=false enable_mse_mpeg2ts_stream_parser=true enable_nacl=false enable_nacl_nonsfi=false enable_one_click_signin=false enable_reading_list=false enable_remoting=false enable_reporting=false enable_service_discovery=false enable_swiftshader=false enable_widevine=true exclude_unwind_tables=true fatal_linker_warnings=false ffmpeg_branding=\"Chrome\" fieldtrial_testing_like_official_build=true google_api_key=\"\" google_default_client_id=\"\" google_default_client_secret=\"\" is_clang=true is_debug=false is_official_build=true optimize_webui=false proprietary_codecs=true safe_browsing_mode=0 symbol_level=0 treat_warnings_as_errors=false use_gnome_keyring=false use_jumbo_build=true use_official_google_api_keys=false use_ozone=false use_sysroot=false use_unofficial_version_number=false custom_toolchain=\"//build/toolchain/linux/unbundle:default\" host_toolchain=\"//build/toolchain/linux/unbundle:default\" linux_use_bundled_binutils=false optimize_for_size=false use_gtk3=true use_kerberos=false use_lld=true use_vaapi=true" --fail-on-unused-args
Done. Made 8488 targets from 1535 files in 6893ms
ninja -j1 -C out/Default chrome chrome_sandbox content_shell chromedriver
ninja: Entering directory `out/Default'
ninja: error: '../../third_party/test_fonts/test_fonts/Arimo-Bold.ttf', needed by 'test_fonts/Arimo-Bold.ttf', missing and no known rule to make it
debian/rules:120: recipe for target 'override_dh_auto_build-arch' failed
make[1]: *** [override_dh_auto_build-arch] Error 1
make[1]: Leaving directory '/home/manuel/ungoogled-chromium/buildspace/tree'
debian/rules:105: recipe for target 'binary' failed
make: *** [binary] Error 2
dpkg-buildpackage: error: fakeroot debian/rules binary gave error exit status 2

Build of the master branch on Windows is indeed failing. I pretty much got the same error as @Ti-R, here it is:
https://paste.debian.net/plain/1031758

I have moved 67.0.3396.87-2 to prerelease status for now. To encourage more consistent testing, I am looking to switching my development model to include "prereleases" (same meaning as a GitHub prerelease). There will be no changes to the tagging scheme. I will work out the specifics of this system and document it somewhere.
@Ti-R @Wyse- e9ce6ca67ed84d68a2716e7cca8ba55d3feca963 should reverse the breaking changes I made.
@dubvulture I think it should be possible to just disable the dependency on the testing fonts.

@dubvulture @Eloston test_fonts can most probably be skipped using the patch at  https://github.com/Eloston/ungoogled-chromium/commit/ba903b2e93036ba366ca7be5f9e90d39b839bb2e#diff-31eef377f22cab09ac4b8eabe94dd6b2 which was removed a while ago.

@Eloston, @Ti-R build successful on Windows (commit e9ce6ca67ed84d68a2716e7cca8ba55d3feca963 of develop branch). Compiled binaries available here.

@seppiofish I've never heard of this issue. Do you know what file(s) are growing in size?

@Eloston yes, a file named BrowserMetrics-spare.pma of 8MB is created and then a folder named BrowserMetrics, in which new files BrowsersMetrics-<hex_string>.pma of 8Mb of size each are constantly created growing the profile size indefinitely.

@seppiofish I can confirm the issue. My profile directory contains about 10GB consisting of many 8MB sized BrowserMetrics-*.pma files. A new file seems to be created each time a new browser session is started.
Starting with version 67 inox-patchset/9000-disable-metrics.patch was changed to use Chromium's new DummyHistogram class to prevent histogram recording and apparently this has some unwanted side-effects. I will look into it for the next stable release in two weeks.
In the meantime you should disable write access to the BrowserMetrics directory to prevent .pma files from being written. I'm not sure how psd will react to this though.

@Eloston, @Wyse- build successful on Windows also for me.
Took me a while to come back.
Got some problems some days agos with

The procedure entry point _o_memset could not be located in the dynamic link library api-mswin-crt-private-l1-1-0.dll

Due to some of my Windows Software Development Kit missing "Debugging Tools for Windows" option
All is ok now.
I compile this version

Move VA-API patch back into Linux bundles
e9ce6ca67ed84d68a2716e7cca8ba55d3feca963

@Wyse- If you believe the Windows binaries are good, you can submit them to the contributor binaries site (ungoogled-chromium-binaries)

@Wyse- Hmm, you brought up a good point about master and develop branches. It's unnecessary to have two branches like this with the new release model that I've developed. I'll have to consolidate them.
I also don't really have a system to receive feedback for testing, to promote development binaries to release binaries.
Some documentation about the new release system is here (I will need to organize it a bit better):

https://github.com/ungoogled-software/ungoogled-chromium-binaries#changing-version-statuses
https://ungoogled-software.github.io/ungoogled-chromium-wiki/statuses

Closing due to development on version 68.


Added body/note is automatically selected

To test:

[ ] After page load, create a body. Ensure it is selected and that info pane is accurate.
[ ] After page load, create a note. Ensure it is selected and that info pane is accurate.
[ ] Create a few more notes/bodies in random order.

Closes #202

I am getting an error when adding a body.

@benjic Whoops, need to make a new engine version

Alright if you reinstall engine (should be version 0.2.3) that error should not appear. I'll work on the conflicts.


Added ref to CreateOrUpdatePrivateZone in /examples

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Hi @markjbrown
Your PR is doing two things:

Adding the ref to example CreateOrUpdatePrivateZone
Adding many VirtualNetworkGateway example to Network 2018-02-01

About the first, it seems there is some validation issues:
https://travis-ci.org/Azure/azure-rest-api-specs/jobs/407643168
About the second, why adding in 2018-02-01 knowing that 2018-04-01 is out? Adding @MikhailTryakhov for guidance.

@lmazuel err, there is a bunch of changes in here that I did not intend to submit in this PR. I'm going to pull this back and resubmit.
Apologies for the crappy PR.

@markjbrown no problem :). Thank you.


AWS Lambda: global state for database connection caching.

Is there a reason why the connection to AWS Lambda must be rebuilt each time as per the tegola lambda readme? I am trying to speed up a web app, and I believe this is a bottleneck. Is there a way the connection can be implemented in order to keep it open?

@paul-pries-weatherlogics could you give me a bit more context? Are you talking about the connection between the web app and tegola lambda or tegola lambda and the database?

The tegola_lambda README says that the connection to the database must be rebuilt on each request. (No connection pooling). It's my understanding that if a connection is opened outside of the handler, it remains warm in an AWS lambda container and does not need to be rebuilt, thereby speeding up API calls.
Is there a way to implement the connection outside of the handler in this way, or is there something in the design of the lambda function that prevents this from being done?

Hmm, do you have some documentation you can point to about the connection being persisted outside the handler? I would like to look into this a bit more.
Regarding the documentation, if the lambda function is still warm the connection pool may be maintained by the lambda runtime. If the lambda function goes cold it will absolutely be killed. One of the slowest parts of using tegola lambda is when it's used against a database within a VPC because an elastic network interface (ENI) needs to be dynamically created between the lambda function and the VPC. The only way around this that I know of is to move your database instance outside the VPC. I don't know if this applies to your setup but may.
Regarding web app performance, what types of caching have you implemented so far?

https://docs.aws.amazon.com/lambda/latest/dg/running-lambda-code.html
This is the main bit that was I was referring to, where it says that any declarations made outside of the handler function can be maintained and reused, including connections.
https://www.jeremydaly.com/reuse-database-connections-aws-lambda/
This post goes into a bit of detail about how to implement this.
The guy who actually wrote the web app is pretty sure the database is in a VPC, so I'll look into how or if I can change that.
The only caching being used currently is Amazon API caching.

@paul-pries-weatherlogics very interesting. Some additional articles that are applicable:

AWS Lambda Execution Context
AWS Lambda Function Handler in Go: Using Global State

This could be a great optimization and from my brief research is something the codebase is positioned to implement.
For immediate performance, I would suggest implementing tegola's S3 tile cache so the tiles don't need to be regenerated on every request.

I'm going to re-open this one as it's something I would like to implement eventually.

@paul-pries-weatherlogics I just pushed up some code to the branch issue-609 that addresses this issue. If you have some time, it would be great to get your review/testing to see if the change is performing as desired.

@paul-pries-weatherlogics this is now in the v0.11.x branch.

Hi guys, this article can help you to reuse db connection in lambda environment.
https://muhammadusmanalibaloch.medium.com/aws-lambda-reuse-database-connection-and-save-around-200ms-response-time-2c50a445f464

@musmanalibaloch yep, this was implemented in the PR #670


Website | Gallery: Update Basic Grouped Bar Chart Legend

Quick update for the legend to only capitalize the first letter. (Issue)

Thanks for fixing this @lee00678!
I've a small thought: since it is one of our official examples, I find it a little "dirty" to use n.charAt(0).toUpperCase() + n.slice(1) there. In the real word use case, I would define a function to do this, or used text-transform: capitalize in CSS.
I don't think we want to overcomplicate this example by adding unnecessary CSS. We can probably add a "capitalize" function to data.ts or simply add a labels map there: { republican: 'Republican', ... }. Let me know what you think.
P.S. We usually have a practice of adding screenshots and videos to our PRs to simplify the review process. That would be great if you could do it too.

Thanks for fixing this @lee00678!
I've a small thought: since it is one of our official examples, I find it a little "dirty" to use n.charAt(0).toUpperCase() + n.slice(1) there. In the real word use case, I would define a function to do this, or used text-transform: capitalize in CSS.
I don't think we want to overcomplicate this example by adding unnecessary CSS. We can probably add a "capitalize" function to data.ts or simply add a labels map there: { republican: 'Republican', ... }. Let me know what you think.

Agree with not adding CSS to complicate the example. I like the idea of adding a capitalize function in data.ts. I think it's a common use case and would be nice to provide an small example for that? I will update. Thanks for the feedback!

P.S. We usually have a practice of adding screenshots and videos to our PRs to simplify the review process. That would be great if you could do it too.

Gotcha.

Yep, that works! Thanks for addressing the comments.


Build sbt diet

Trims 200+ lines off build.sbt in an attempt to make it easier to maintain.
 48 files changed, 115 insertions(+), 340 deletions(-)

Most project boilerplate is now configured via circeCrossModule and circeModule.
Make several cross projects pure: core, generic, numbers, refined, scodec
Improve management of projects by declaring several groups and use that to define aliases etc.

There are several modules which doesn't show up in the scaladoc, e.g. parser and java8 because they only expose a package object. One way to fix this is to move the implicits and defs into a trait.

Current coverage is 81.87% (diff: 100%)

Merging #474 into master will decrease coverage by 0.13%

@@             master       #474   diff @@
==========================================
  Files            80         80
  Lines          2157       2157
  Methods        2053       2053
  Messages          0          0
  Branches        104        104
==========================================
- Hits           1769       1766     -3
- Misses          388        391     +3
  Partials          0          0

Powered by Codecov. Last update 0c1ee23...be04de0

@travisbrown OK, reverted the change to make more projects pure as well as the workaround to remove sources from certain macro projects for 2.10.

Thanks again, @jonas! I want to run through a few more checks (publishing locally and looking at artifacts, etc.) this morning, but I'll try to get this merged today.

Okay, I've just spent some time poking at this, and it looks perfect except for one tiny thing—we're no longer publishing real Javadoc artifacts for circe-shapes on 2.10 even though we can (and used to). I'm happy to fix that in a follow-up PR, @jonas, unless you want to take care of it here.

OK, fixed it in the additional commit instead of rebasing.

@jonas Ah, great, thanks! Will merge when green.

Lean sbt builds FTW. Jonas is a legend.


Help needed

Hello,
I wanted to ask if you can add a function to edit a date by adding a small calendar from which the date can be chosen.
I have tried a few ways to implement it but it did not work.
Also, I wanted to ask if it's possible for errors to mention what generated them(ex: error when saving Lastname , error when saving id)

Example with eternicode/bootstrap-datepicker and markcell/jQuery-Alertiny:
$('table').Tabledit({
  url: 'example.php',
  columns: {
    identifier: [0, 'id'],
    editable: [[1, 'date'], [2, 'name']]
  },
  onSuccess: function(data) {
    // Example of response from PHP script:
    // json_encode(array('type' => 'error', 'message' => 'Example.'));
    if (data.type && data.message) {
      $('.alert-container').Alertiny(data.type, data.message);
    }
  },
  onDraw: function() {
    // Select all inputs of second column and apply datepicker each of them
    $('table tr td:nth-child(2) input').each(function() {
      $(this).datepicker({
        format: 'yyyy-mm-dd',
        todayHighlight: true
      });
    });
  }
});

Thanks, this works great.

Can you please help me with an example for the validation of another fields?
I need a min-max number validation and something for just alphanumerical chars.
I tried to use https://github.com/jzaefferer/jquery-validation but i think i didn't use it right as i get no validation, no errors.

Hello I was trying to do the same thing, I need to modify dates using Jquery Tabledit and with your example i was able to add the datepicker in the table when click to edit, but when i send the info to the PHP to be storaged in the database even when it says success it does not safe the info, how did you manage to Storage the date in the Data Base?
Hope you can help me and i really appreciate it.
regards!

@gabmen1711 make sure your DB table is set up properly(ex: Datetime) and that you send it in the proper format.
You might need to change the format(ex: DD-MM-YYYY) or remove the seconds and so on.
Test using echo $date-from-datepicler to see how it's sent and then compare to what your DB expects.

Thank you very much for your help!
One more thing when I click the Edit Button it popups automaticly the Calendar, as I have 3 different dates inputs it popups the 3 at the same time, do you know if its possible to prevent them to auto popups when i click edit or if its possible to not highlight the inputs when i click edit i believe that is why they popup automaticly!
Thanks again for the help

@gabmen1711: comment line 237
$input.focus();

Could someone share jsfiddle for date picker. I have included bootstrap-datepicker3.css ,bootstrap-datepicker3.css.map and altertiny js in my application. But I am getting javascript error as undefined function near..$(this).datepicker({
I am new to front end development so kind of struggling with this. Any help is appreciated.

Its working !!

Still cant remove the DatePicker AutoPopup. Waiting for answers thanks

hello ,
i try to use datepicker in my Code but not working with me ,
please if any one  can share working code
thanks a lot

Thanks, this works great.

please can you share full code of page , i try use same your code but its not working with me
thanks a lot

When I put in the onDraw code everything on my page becomes un-editable. Any idea what I'm missing?

Hi, from this example
https://trac.edgewall.org/ticket/12228
This for the autocomplete:
$(this).prop("autocomplete", "off");
This is the code
$('#tblrevop tr td:nth-child(5) input').each(function() {
$(this).datepicker({
format: 'mm/dd/yyyy',
todayBtn: true,
todayHighlight: true
});
$(this).prop("autocomplete", "off");
});


Replace Travis and update deps

To ensure that cobbler is compatible and stable we need a good CI and updated deps.

I don't care about Windows for now. Thus merging to add a new release.

I don't care about Windows for now. Thus merging to add a new release.


puller(ticdc): fix renameTables ddl job

What problem does this PR solve?

Issue Number: close #9476
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test all

/retest

/retest

/retest

/retest


Additional placeholders, including for other users and offline players

Description
This would come in 2 parts:

Introduce a new wave of placeholders that returns data from other users. This is kind of like the "parseother" PAPI expansion, except this would be done via internal luckperms placeholders only. For instance: %luckperms_{otherusername}_meta_key%. This would return the meta_key for the specified player. Another example that would benefit us greatly: %luckperms_{otherusername}_in_group_{group}%. We are a decently sized server (roughly 150-250 players a day ish) and we have a lot of custom commands and content that we've been developing. This would help tremendously because it lets us customize user experience with the custom commands
These new placeholders would also work on offline players! For instance, if I ran was a player and was running a custom command to find someone's favorite color(which we, as an example, store in meta data), currently the other other player would have to be online. But if I wanted to run something like "/favcolor Lucko" when Lucko was not online then ideally, our custom command would return %luckperms_Lucko_meta_favcolor%. It would be built in a way that simply took the input, and checked it against the database and return the meta value. If not set it would return blank. If player not found, it would return blank.

This is doable for plugin-side internal placeholders since a lot of those placeholders are run against plugin data itself and not against the target player. This is not doable for papi expansions such as the above mentioned "parseother" because all parseother does is run the placeholder on the other online player without alerting them and return the value it finds, it has no way to run it against the plugin's data instead of the player itself.
Proposed behaviour
Participating luckperms placeholders would allow for an optional {username} input as the second variable in the placeholder string. So instead of saying %luckperms_meta_key% we could say %luckperms_{Lucko}_meta_key% which would return Lucko's key instead of my own. The reason we use {brackets}  is to treat it like an internal placeholder (since some usernames have underscores in them). The reason it's the second variable in the placeholder is for ease of developing and checking input.

Hey, this is an interesting idea.
Unfortunately, it's not possible to easily implement placeholders for offline players because loading their data requires I/O - and can potentially happen on the main server thread.
It's the same reason why Vault queries for offline players do not happen by default.
If PlaceholderAPI ever introduced the idea of an async placeholder - this would be possible - but until then, unfortunately it's not.
Thanks for the suggestion nonetheless :)


ReImplement KeyNameChecker using CRuby's new API

CRuby v2.5 will be able to call KeyError#receiver and KeyError#key.
I reimplement KeyNameChecker using it.
We'll be able to support suggesting Hash#fetch_values, Kernel#sprintf and ENV#[] also ;)
refs:

https://bugs.ruby-lang.org/issues/12063
https://github.com/yuki24/did_you_mean/pull/71
https://github.com/yuki24/did_you_mean/issues/89

Thank you for reviewing.
I fixed and rebased on this branch as a first-class feature 🎉

Thanks!


Add License

It would be good to ensure this is included.
Any thoughts re which variant to use here?

MIT


Pipenv doesn't use pyenv and installs separate version of Python when installing with Homebrew

Issue description
I am currently running MacOS Sierra with pyenv that contains versions of Python 2.7, 3.5, 3.6, and 3.7. When I installed Pipenv, I expected it to use pyenv to install a version of Python that could be isolated. Instead, it installed another version of Python 3.7 from Homebrew. While this may not be that much of a problem, it will mess up the Python paths.
Expected result
Pipenv does not need to install Python from Homebrew, but use pyenv for an isolated python environment.
Actual result
Pipenv installs another version of Python from Homebrew for an isolated environment.
Steps to replicate
Install pyenv from Homebrew: brew install pyenv
Install a couple of versions of Python from pyenv: pyenv install 3.7.0 3.6.5 3.5.5
Install Pipenv from Homebrew: brew install pipenv

$ pipenv --support
Pipenv version: '2018.7.1'
Pipenv location: '/usr/local/Cellar/pipenv/2018.7.1/libexec/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/Cellar/pipenv/2018.7.1/libexec/bin/python3.7'
Other Python installations in PATH:

2.6: /usr/bin/python2.6

2.6: /usr/bin/python2.6

2.7: /Users/SrinuL/.pyenv/shims/python2.7

2.7: /Users/SrinuL/.pyenv/shims/python2.7

2.7: /usr/bin/python2.7

3.6: /Users/SrinuL/.pyenv/shims/python3.6

3.7: /usr/local/bin/python3.7m

3.7: /usr/local/bin/python3.7

2.7.10: /Users/SrinuL/.pyenv/shims/python

2.7.10: /usr/bin/python

None: /Users/SrinuL/.pyenv/shims/python2

3.7.0: /Users/SrinuL/.pyenv/shims/python3

3.7.0: /usr/local/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.0',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '16.7.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 16.7.0: Thu Jan 11 22:59:40 PST '
                     '2018; root:xnu-3789.73.8~1/RELEASE_X86_64',
 'python_full_version': '3.7.0',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

PATH
TERM_PROGRAM
NVM_CD_FLAGS
ANDROID_HOME
PYENV_ROOT
TERM
SHELL
TMPDIR
Apple_PubSub_Socket_Render
TERM_PROGRAM_VERSION
TERM_SESSION_ID
ZSH
NVM_DIR
USER
SSH_AUTH_SOCK
__CF_USER_TEXT_ENCODING
PYENV_VIRTUALENV_INIT
PAGER
LSCOLORS
_
PWD
JAVA_HOME
DEFAULT_USER
LANG
ITERM_PROFILE
XPC_FLAGS
XPC_SERVICE_NAME
PYENV_SHELL
SHLVL
HOME
COLORFGBG
GOROOT
ITERM_SESSION_ID
LESS
LOGNAME
LC_CTYPE
NVM_BIN
DISPLAY
COLORTERM
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /usr/local/Cellar/pipenv/2018.7.1/libexec/tools:/Users/SrinuL/.nvm/versions/node/v8.1.3/bin:/usr/local/Cellar/pyenv-virtualenv/1.1.3/shims:/Users/SrinuL/.pyenv/shims:/Users/SrinuL/.pyenv/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Library/TeX/texbin:/Users/SrinuL/Library/Android/sdk/tools:/Users/SrinuL/Library/Android/sdk/platform-tools:/usr/local/opt/go/libexec/bin
SHELL: /bin/zsh
LANG: en_US.UTF-8
PWD: /Users/SrinuL

"but it uses the Python it is installed to by default if you don’t supply one."

What do you mean by this? When I run brew install pipenv, the first thing that happens is this:
==> Installing dependencies for pipenv: python
==> Installing pipenv dependency: python
Warning: Building python from source:

==> Downloading https://www.python.org/ftp/python/3.7.0/Python-3.7.0.tar.xz
...

It then proceeds with a lengthy compilation of Python, which I already have (2.7 and 3.7).

Okay so I misunderstood you. I thought you were talking about Pipenv installing a new Pyth, but what you are referring is Homebrew not installing Pipenv with an existing Python. This is not Pipenv’s problem, and we can’t do anything even if we want to (also, we don’t). Talk to Homebrew maintainers if you think this is wrong (also, this is not).

Pipenv can be installed via $ pip install. It is documented here.
$ which pip
/Users/MyUser/.pyenv/shims/pip
$ pip install pipenv
$ pipenv --version
pipenv, version 2018.10.13


DRILL-8123: Revise scan limit pushdown

DRILL-8123: Revise scan limit pushdown
Description
Recent work added a push down of the limit into a scan. The work had a few holes, one of which was plugged by the recent update of EVF to manage the limit. Another hole is that the physical plan uses a value of 0 to indicate no limit, but 0 is a perfectly valid limit, it means "no data, only schema." The field name is "maxRecords", but should be "limit" to indicate the purpose.
Documentation
N/A
Testing
Reran unit tests.

Thank you for reworking these PRs @paul-rogers.  It was only the timestamp fixes that I really felt should receive separate review, but these incremental PRs are great.  Should I close the monolithic #2419, or switch it to draft, so that it's clear we should not merge that one?
Anecdote: I had to look at a LIMIT 0 query against Parquet yesterday and saw that the Parquet reader reads a single batch of data anyway, even though I don't believe there's any need for it to.  All "limit 0 shortcut" config opts that I could see were enabled.  I know the present-day Parquet readers are their own animals entirely, entirely unlike EVF style readers, but I thought this anecdote might be of interest since the same LIMIT 0 case is being addressed here.

@jnturton, you an mark the big one as a draft. It will still contain the EVF V2 roll-out (only), all the other stuff will be done in other PRs. Actually, I think that one is the next one up after the two today are merged. Then, the two readers converted to EVF V2 will follow in additional PRs. The stuff you don't like, the date/time fixes, I'm omitting. The result is that I can't test on my machine, so I'm letting GitHub run the tests: it must operate in the UTC time zone.


On the README, list that this is legal.

I couldn't believe that VA allows applications submitted by email.  The state's form/instructions downplays this (while stressing all the walkin/usps info), though it mentions in passing.
It would be great if the README noted whatever laws or court decisions that legitimize this application process.

A fine idea that I only just now got around to acting on. :) Thanks, @jvanasco!

Thanks!  My friend Debra runs LongDistanceVoter.org so I've seen a lot of political tech stuff over the years.  I was really worried this was illegal (and applications would be thrown out) until someone pointed out who was behind this project.

This project only exists because the state OK'd it. :)

Jonathan,
Thanks for the good suggestion, and pleased to e-meet you. I'm a fan of
Debra and LDV too.
Waldo and company did in fact do the legal homework, but you're right in
pointing out that it's important for that info to go in the repo!
Cheers,
John Sebes
Jonathan Vanasco wrote:

Thanks! My friend Debra runs LongDistanceVoter.org so I've seen a lot of
political tech stuff over the years. I was really worried this was
illegal (and applications would be thrown out) until someone pointed out
who was behind this project.
—
Reply to this email directly or view it on GitHub
https://github.com/TrustTheVote-Project/horatio-server/issues/60#issuecomment-156316216.

Sorry, I should have been more clear.  I've long known that this project was legit -- I was primarily worried about someone in another state saying "wow this is awesome!", then forking it for a state where it's not legal.


[iOS] Tracks stop playing after pausing for ~ 2 minutes

Configuration
Run react-native info in your project and share the content:
System:
    OS: macOS Mojave 10.14.6
    CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
    Memory: 22.91 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
    Yarn: 1.21.1 - ~/.yarn/bin/yarn
    npm: 6.10.2 - ~/.nvm/versions/node/v10.16.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.0, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 23, 25, 26, 27, 28, 29
      Build Tools: 27.0.3, 28.0.2, 28.0.3, 29.0.0, 29.0.2
      System Images: android-19 | Google APIs Intel x86 Atom, android-21 | Google APIs ARM EABI v7a, android-21 | Google APIs Intel x86 Atom, android-21 | Google APIs Intel x86 Atom_64, android-22 | Google APIs Intel x86 Atom, android-23 | Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
    Xcode: 11.0/11A420a - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5

What react-native-track-player version are you using?: v2
Issue

On iOS I've noticed that if an audio track is paused for longer than ~2 minutes (sometimes 3, rarely earlier) and then played again the playstate never changes from buffering.
I tried to debug through the Swift code to the best of my abilities, but the issue appeared to happen immediately following these logs:
2020-02-26 17:56:49.762763-0500 BibleIs[39384:2804056] [] nw_endpoint_flow_copy_multipath_subflow_counts Called on non-Multipath connection
2020-02-26 17:56:50.116227-0500 BibleIs[39384:2804129] [] nw_endpoint_flow_copy_multipath_subflow_counts Called on non-Multipath connection
2020-02-26 17:56:50.122304-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.122419-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.122533-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.124472-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.124591-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.124674-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.125503-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.125606-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0
2020-02-26 17:56:50.125684-0500 BibleIs[39384:2804336]  AudioObjectRemovePropertyListener: no object with given ID 0

If I refresh my app and hit play again (just calling TrackPlayer.play) then the track begins playing. Pause, wait 2 minutes, and we're back to unplayable. It's almost as if the track player is wiping its own queue?

This issue is happening only when you use hls stream files. It is working fine for me with mp3 files.

@ds8k you mentioned the possibility of this being a simulator bug.  Are you able to confirm it happens on real devices?
Are you able to provide a sample URL for testing this behavior?


aa

Please use your own private repositories for testing.

Please use your own private repositories for testing.


Cannot generate global parameters node using Java annotations

I have a bunch of JAX-RS 2.0 interfaces that define my API. I'm using Swagger 2.0 annotations to produce an OpenAPI 3.0 spec.
Many of the operations in the API share certain query and header parameters (things like language, client version, etc). It would be incredibly excessive and a nightmare to maintain if we just copied and pasted these shared parameters onto each operation.
Ideally, we'd like to blanket apply all of these parameters to our operations, but there doesn't seem to be support for that in OpenAPI 3.0. The orthodox way to do this seems to be to use $ref to refer to parameters defined at the global API level.
There doesn't seem to be any way to produce that global parameters node using just Swagger annotations, though. Is this a gap in functionality, or am I missing how to do this?
As an implementation, I'd expect to see a parameters field in the @OpenAPIDefinition annotation.

Looking at the OpenAPI v3.0 spec it might be better, or even easier, to add a @Components annotation for use inside @OpenAPIDefinition. Then inside @Components any number of @Parameter or @Schema or anything else as mentioned here: https://swagger.io/specification/#componentsObject

Your route sounds like a pretty clean way to handle this for now.
I struggled to find a simple way of converting @Parameter annotations into Parameter models and couldn't find a way to hide the "/dud" endpoint but merging YAML spec would solve both this issues.
Later on I found out that @Parameter(in = ParameterIn.QUERY, name="limit") doesn't 'imply' @QueryParam("limit") and so still had to include those in the method signature:
public List<Pet> getPets(@Parameter(ref = "limit") @QueryParam("limit") int limit, @Parameter(ref = "offset") @QueryParam("offset") int offset) {

So not quite perfectly minimal but certainly better than lots of copy&paste.

We still need this. The workarounds are a pain. Please add an @Components annotation or some equivalent.

+1 on the addition of @Components

+1 on the addition of @Components


[Coder] When creating a new class in the playground, if a new package and tag are entered, the package name is incorrect

In a fresh image, open a playground and enter a new class name and new. The define class action pops up. Define a new package and a tag. Save. The class gets created in package-tag, with tag empty

Linux, 281

It works now:


Stop program when a listener thread stops

Currently if a listener either directly calls exit or triggers a when statement that does so, then only the listener stops.
However from the dreamberd semantics (and my functionality needs) it looks like it should instead exit the whole program.
So now it is regularly checked if any listener has stopped. Under normal circumstances this should only happen when it hits an exit.

Agreed, this would be better behavior


SNOW-742344 Allow python version >=3.8 in internal recipe

Description
This helps us building internal snapshot packages for regression tests, and gets us prepared for stored proc testing when server supports py39
Testing
N/A
Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-742344

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
This helps us building internal snapshot packages for regression tests, and gets us prepared for stored proc testing when server supports py39

Codecov Report

Merging #690 (29fe798) into main (5df6846) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #690   +/-   ##
=======================================
  Coverage   98.74%   98.74%
=======================================
  Files          49       49
  Lines        7978     7978
  Branches     1352     1352
=======================================
  Hits         7878     7878
  Misses         40       40
  Partials       60       60

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


Replace URL constructor calls with URI::toUrl

As pointed out Java 20 will deprecate URL public constructors.

In Java SE 1.4 a new class, java.net.URI, has been added to mitigate some of the shortcoming of java.net.URL. Conversion methods to create an URL from an URI were also added. However, it was left up to the developers to use java.net.URI, or not.
This RFE proposes to deprecate all public constructors of java.net.URL, to provide a stronger warning that these constructors can be misused. To construct a URL, using URI::toURL should be preferred.

Looks like a good candidate for the new rewrite-templating method of recipe declaration, as there's no library needed for the replacement, only a new import.

@timtebeek Should this instead be moved to openrewrite/rewrite-migrate-java?
This particular recipe should indeed be possible to implement using the rewrite-templating annotation processor. But as it will require a Java 20 JDK on the classpath, I am not sure we can do that.


WIP: Static Linux builds against Musl [Do Not Merge]

This intends to tackle #255.

I've tried a few things, i'm giving up for this release.

Eventually we could try bootstrapping rustc against Musl


[BUG REPORT] Unable to run RVC

Describe the bug
When attempting to run RVC on an input file, in this case, a sub 10 second .wav file, a bunch of traceback errors are thrown in the console and I'm not entirely sure what the issue would be.
To Reproduce
Steps to reproduce the behavior:
After loading RVC model, try to run the model on an input.
Expected behavior
Supposed to process the audio as expected, but throws a bunch of traceback errors instead.
Screenshots
Attached a .txt w/ the error (it's 140 lines.)
audio-webui error log.txt

Send a screenshot of your settings. Also, did you wait for the file to finish uploading? (this is indicated by being able to preview the audio in the editor)

I'm going to close this issue due to inactivity.


docker error

can not start docker on windows.
using comand:
docker run -it seccubus/seccubus web or
docker run -it seccubus/seccubus full
docker run -it seccubus/seccubus -full
does not work..

exec --fulldocker run -it seccubus/seccubus web
/entrypoint.sh: line 298: exec: --: invalid option
exec: usage: exec [-cl] [-a name] [command [arguments ...]] [redirection ...]

or

exec web
/entrypoint.sh: line 298: exec: web: not found

please fix.

@vulnz ,
The way to configure Seccubus is via environment variables, not by running commands.
In order to run a full stack seccubus on port 8443 you can do the following:
docker run -p 8443:433 seccubus

If you want to just run the web stack, you do the following:
docker run -p8443:443 -e STACK=web seccubus

Thank you. You are great. ;)
but it does not work.
`docker run -p 1443:443 seccubus/seccubus

STACK=full
DBHOST=<IP_ADDRESS>
DBPORT=3306
DBNAME=seccubus
DBUSER=seccubus
DBPASS=seccubus
TLS=yes
TZ=UTC
[[ -e /usr/share/zoneinfo/UTC ]]
rm -f /etc/localtime
ln -s /usr/share/zoneinfo/UTC /etc/localtime
[[ '' == \s\c\a\n ]]
[[ full != \f\u\l\l ]]
[[ yes == \y\e\s ]]
[[ full != \c\r\o\n ]]
[[ full != \p\e\r\l ]]
[[ -e /opt/seccubus/data/seccubus.pem ]]
[[ ! -z '' ]]
openssl genrsa -des3 -passout pass:x12345 -out /opt/seccubus/data/seccubus.pass.key 4096
Generating RSA private key, 4096 bit long modulus
.................................................................................................................................................................................++
............................++
e is 65537 (0x010001)
openssl rsa -passin pass:x12345 -in /opt/seccubus/data/seccubus.pass.key -out /opt/seccubus/data/seccubus.key
writing RSA key
rm /opt/seccubus/data/seccubus.pass.key
openssl req -new -key /opt/seccubus/data/seccubus.key -out /opt/seccubus/data/seccubus.csr -subj /CN=Seccubus
openssl x509 -req -days 365 -in /opt/seccubus/data/seccubus.csr -signkey /opt/seccubus/data/seccubus.key -out /opt/seccubus/data/seccubus.pem
Signature ok
subject=CN = Seccubus
Getting Private key
rm /opt/seccubus/data/seccubus.csr
TLSCERT=/opt/seccubus/data/seccubus.pem
TLSKEY=/opt/seccubus/data/seccubus.key
PORT=443
[[ full == \f\r\o\n\t ]]
[[ full == \a\p\i ]]
[[ full != \f\r\o\n\t ]]
mkdir -p /opt/seccubus/.ssh
chmod 700 /opt/seccubus/.ssh
echo ''
export SSHKEY1=
SSHKEY1=
echo ''
export SSHKEY2=
SSHKEY2=
echo ''
export SSHKEY3=
SSHKEY3=
echo ''
export SSHKEY4=
SSHKEY4=
echo ''
export SSHKEY5=
SSHKEY5=
echo ''
export SSHKEY6=
SSHKEY6=
echo ''
export SSHKEY7=
SSHKEY7=
echo ''
export SSHKEY8=
SSHKEY8=
echo ''
export SSHKEY9=
SSHKEY9=
chown -R seccubus:seccubus /opt/seccubus/.ssh
chmod 600 /opt/seccubus/.ssh/SSHKEY1 /opt/seccubus/.ssh/SSHKEY2 /opt/seccubus/.ssh/SSHKEY3 /opt/seccubus/.ssh/SSHKEY4 /opt/seccubus/.ssh/SSHKEY5 /opt/seccubus/.ssh/SSHKEY6 /opt/seccubus/.ssh/SSHKEY7 /opt/seccubus/.ssh/SSHKEY8 /opt/seccubus/.ssh/SSHKEY9
[[ -z '' ]]
++ cat /opt/seccubus/etc/SESSION_KEY
SESSION_KEY=XpJNnrGgJFCBY76vcc099VKqJxWyj5uG
cat
[[ full == \f\u\l\l ]]
[[ ! -z '' ]]
sed -i.bak 's#baseUrl(){return"/api/"}#baseUrl(){return"../api/"}#' /opt/seccubus/public/seccubus/production.js
cd /opt/seccubus
PERL5LIB=/opt/seccubus
hypnotoad seccubus.pl
[[ <IP_ADDRESS> == \1\2\7.\0.\0.\1 ]]
[[ 3306 == \3\3\0\6 ]]
DBDIR=/var/lib/mysql
[[ -e /opt/seccubus/data/db ]]
/etc/init.d/mysql start
[[ ! -d /var/lib/mysql/seccubus ]]
[[ full == \c\r\o\n ]]
[[ full == \f\u\l\l ]]
CRON_MAIL_TO=
[[ ! -z '' ]]
/mkcron
/etc/init.d/rsyslog start
/etc/init.d/cron start
[[ full != \f\r\o\n\t ]]
cd /opt/nikto
git pull --depth=1
cd /opt/testssl.sh
git pull --depth=1
From https://github.com/drwetter/testssl.sh
785e949...ca71420 2.9dev     -> origin/2.9dev  (forced update)
fatal: refusing to merge unrelated histories
cd
case $1 in
[[ full == \f\r\o\n\t ]]
[[ full == \f\u\l\l ]]
grep -v 0x794c7630
tail -f /var/log/seccubus/production.log`

@vulnz what did you expect that would happen and what happened?
What happens if you point a browser to https://<IP_ADDRESS>:1443?

Amost ;) My mistake was - i used http, not https.

What is the password?

It is 'GiveMeVulns!'

Thank you. Are are awesome!

Hey @seccubus, unrelated but it will be really helpful if you could update this password in the README itself. Thanks!

Fixed in last PR


makefile: clean up output, add linking for building from source

The purpose of this pull request is to make things easier when building from source/cloning the repo
Added:

symlinks for binary and man page included with running make or make install
config file ./examples/yabairc is copied to ~/.yabairc
removal of binary, man pages, .lock and .socket symlinks when running make clean
suppressed command output (didn't suppress clang)
messages that indicate what is happening (i.e. "Removing man page...")
make and make install both invoke the clean rule first, so I added testing to see if the aforementioned files exist before attempting to remove them

(if these changes are added, the readme would also have to be updated for building from source. I didn't add that here but could help w that if needed)
I considered adding variables to the makefile for a few things, so if it's requested that those are added I can do so. Any other feedback/suggestions for this are appreciated.

I don't thing these changes are necessary. Removing the lock / socket / symlinks file simply to have them recreated after a new build is not necessary.
Automatically creating symlinks and copying of the config is not something I want, because formulaes for package managers handle this their own way - and the directory is not always going to be /usr/local/..

ok - got it, thanks for your response! I guess I made this more for my own convenience w/o thinking of the impact on the project as a whole. I'll look for other ways to contribute!


Dark Horizon fails to launch after first run

Dark Horizon will happily run the first time, but the second time you try to launch the program, it gets stuck showing simply 'Loading Dark Horizon...'.

Curiously that screenshot is also transparent. Do we not pre-fill our main canvas?

I wonder if we're corrupting the settings file somehow?

Starting program with UID3 10005007
Stop hook called, exiting interpreter with error(KStopErr)
Interrupted!
-999 (KStopErr)
DARKHORIZON\READCFG:00005C12: 7B [IntToLong]
DARKHORIZON\STARTUP:000014FC: 53 [RunProcedure] 0x0FFF (name="READCFG" nargs=0)
stack traceback:
	[C]: in local 'opsync'
	runtime.lua:863: in function <runtime.lua:847>
	[C]: in function 'xpcall'
	runtime.lua:981: in method 'pcallProc'
	runtime.lua:1250: in function 'runtime.runOpo'

I wonder if we're corrupting the settings file somehow?

IOREAD() is supposed to return KErrEof when trying to read beyond the end of a file, and it isn't; turns out Dark Horizon relies on that to terminate its ini file parsing.