Side staking bot

The new funky thing preventing rug pulls, and not even sure where the name comes from but it's floating around in the team. Issue is for deciding if it needs some visualization with its own numbers on the Pool tab, and if so, how that would look like and then implement. Advantage of visualizing it is that UI helps in understanding what's special with the new pools, explaining it without a wall of text.
The facts:

bot (another Ethereum account) for each pool auto-controls datatoken supply
whenever someone stakes OCEAN into a pool, the pool mints datatokens
just enough datatokens are minted to keep datatoken price stable (constant DT:OCEAN ratio)
whenever OCEAN is unstaked, datatokens are burned
bot will receive half of the liquidity provider's pool shares

That last point might mean we have to calculate/show things like creator/user liquidity differently and not base this all on the user's pool shares.

A reference: Bancor has "single-sided staking". They have different underlying mechanics than Ocean, but the effect is the same: you're a liquidity provider into a pool yet you only have to give tokens on one side.
Here's what it looks like.
https://app.bancor.network/pools/add-liquidity/0xb1CD6e4153B2a390Cf00A6556b0fC1458C4A5533
In app, go to "Earn -> Pools". For a given token (e.g. ETH), see the "+" button. Click it.

After you've clicked, you get the following screen.

Notes:
IMO we should not call it "side staking bot" or "staking bot". We should call it "single sided staking" or "one-sided staking". It's much simpler for the user, because it's less about the "how" (robots? huh?) and more about the "what" (you only have to stake one side).

the contracts call it side staking, which then also populates to ocean.js calling it like that https://github.com/oceanprotocol/contracts/blob/v4main/contracts/pools/ssContracts/SideStaking.sol

the contracts call it "side staking", which then also populates to ocean.js calling it like that

"Side staking" is a terrible label for users; we definitely shouldn't use it in the GUI.
More generally, just because a lower-level library uses a label X, it doesn't mean that it should be propagated to a GUI level. E.g. we don't expose joinswapExternAmountIn(). But you know that:)
There's also a good argument to rename away from "side staking" in the contracts and ocean.js, but I don't want to rock the boat.

yup agree, only dropped for reference cause I wondered in my description where it was coming from. I had so many questions about each of the 3 words :-)

Your Liquidity, Pool Creator Statistics: leave only OCEAN liquidity, pool shares, move % share beside title
opens up space to optionally have new section to visualize the bot
use spot price everywhere, except for Use where we show the consume price
pools are always 50/50 weight, hardcoded in contract

Checkpoint, over in #1057 I reduced the liquidity stats to their essence which is this:

Doing this by simply doubling the fetched ocean liquidity (because of hardcoded 50/50 weight), and ignoring fetched datatoken liquidity for those users.
Think already more clear for liquidity providers although already interesting effect shown. I added 10 OCEAN and right away getting shown I now only have 9.685 OCEAN which I assume is the poolFee kicking in on adding liquidity?

For Your Liquidity:

safer to use chain method to calculate how much baseToken user would get when removing instead of basing it on pool shares fetched from subgraph

we should use this with the pool shares that we get from the subgraph


Spits out error when trying to open a .ogg file

I installed the dependencies and then ran rake install. Then I tried to open a .ogg file, but it ends with an error. The same file opens just fine if this program is ran from source.
Environment: MacOS 12.1, Apple M1 Pro
Error as follows:
Unable to parse auxesia.ogg: /Library/Developer/CommandLineTools/usr/bin/python3: can't open file '/Library/Ruby/Gems/2.6.0/gems/ogg_album_tagger-0.1.2/lib/ogg_album_tagger/../../bin/ogg-json.py': [Errno 2] No such file or directory
auxesia.ogg does not seems to be a valid ogg file.

Hum, it looks like I might have forgotten to add bin/ogg-json.py to the spec.files array in ogg_album_tagger.gemspec, which certainly prevents rake install to install that file.
Could you try that?


Update Composer dependencies (2023-12-20-00-15)

Visual regression test passed!
View the visual regression test report


Column names could be made optional

Could use indices instead (in fact probably should do this for performance sake)

See #319. Not for 1.0.

Instead of making these optional I'm auto-generating Excel-style "letter names" if they are null. Tentatively concluding this is a superior solution. See #331.


Previous post navigation

I am greatly enjoying this, thank you for making it! It reminds me of StumbleUpon in the best of ways.
With SU, your URL changed for each site you visited, so you could press back button if you want to see something in this session again. With kagi it's not, so I wonder if something could be added to the top bar to help you navigate back to previous sites you've read in this session? e.g. a "Prev Post" button or a dropdown next to the URL listing all of the URLs you've opened in this session.

We do not have it by design, to encourage thoughtfullness. This is why you can not press next post for 3 seconds too. Small Web is about slowing down. Feel free to present a differetn case.

I'm not sure I understand why your session history is orthogonal to this design goal? For example, if I read 10 posts and the 10th one is related to the same topic as the first one, I will be pretty unhappy if I can't go back and see exactly what the first one said. Also, opinions change the more distant you get from an experience, it often happens on HN that I will read an article and then an hour later realize I really want to keep a link to it so I go back, find the post, and then favorite it; rarely do I realize as I'm reading something how impactful it's going to be.

I do not disagree, just we found that no history makes people do not rush through this (like they did through stumble upon).
Tip: If you 'appreciate' the post you like, you can later find it through 'appreciated' RSS feed.


Support log source ip to rpc debug log

In the debug log, client IP will be logged

alluxio-bot, merge this please


Add tests for integer and optional primary keys for Swift

For all integer types:

Add tests for adding objects with primary key
Add tests for adding objects with nil primary key
Add tests for retrieve the object with primary key
Add tests for retrieve the object with nil

Realm Objective-C already has integer primary key and null key.
CC @jpsim @mrackwitz

Looks good to me.

👍

:+1:


Invoking sampleApp microphone error

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
Getting various errors related to portaudiomicrophonewrapper when sample app starts, which then fails to initialize
What is the expected behavior?
Sample App should initialize
What behavior are you observing?
ubuntu@ip-172-31-57-71:/vagrant/zenobia$ behave --logging-level=ERROR
2020-08-10 22:31:08.898 [  1] I sdkVersion: 1.13.0
2020-08-10 22:31:08.899 [  1] I SampleApplication:configFiles size: 1
2020-08-10 22:31:08.899 [  1] I SampleApplication:Config file: /vagrant/hybridProxy/build/Integration/AlexaClientSDKConfig.json
2020-08-10 22:31:08.915 [  1] I SampleApplication:Using mock context manager.
.
.
.
2020-08-10 22:31:08.932 [  1] C PortAudioMicrophoneWrapper:Failed to open PortAudio default stream:errorCode=-9985
2020-08-10 22:31:08.932 [  1] C PortAudioMicrophoneWrapper:Failed to initialize PortAudioMicrophoneWrapper
2020-08-10 22:31:08.932 [  1] C SampleApplication:Failed to create PortAudioMicrophoneWrapper!
2020-08-10 22:31:08.934 [  1] C SampleApplication:Failed to initialize SampleApplication
2020-08-10 22:31:09.320 [  1] E SettingManager:removeObserverFailed:reason=invalidSetting,settingIndex=0
Provide the steps to reproduce the issue, if applicable:
On an ubuntu 18.04 EC2 instance, build AVS SDK and run behave
Tell us about your environment:
Ubuntu 18.04 on EC2
What version of the AVS Device SDK are you using?
1.13.0
Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[ X] Other - tell us more:

Tell us about your OS (Type & version):

[ X] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

I have tried various other issues that I saw which recommended building AVS SDK with various flags but can't get around the issue.


Undefined variables doesn't show an error

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

VS Code Version: 1.72.2
OS Version: Windows 11

Steps to Reproduce:

Create simple php file

<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Document</title>

    <?php foreach ($users as $user) : ?>

<?= $user['name'] ?>

    <?php endforeach ?>
    <?php callForNonExistFunction(); ?>

$users Is supposed not defined anywhere but editor doesn't show any errors .

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Update README.md: s/key/publicKey

https://github.com/plaid/react-plaid-link/pull/72#issuecomment-582122347

@pbernasconi It looks like I cannot merge this PR, would you mind merging this PR?


[Aio] Add AsyncIO support for Channelz

This PR adds AsyncIO support for Channelz servicer. However, there are still some test failure due to memory management issue. Posting the PR to see how do test go with our CI.

Cannot reproduce the error on Linux with 1000 runs:
bazel test --runs_per_test=1000 //src/python/grpcio_tests/tests_aio/channelz:channelz_servicer_test
INFO: Running bazel wrapper (see //tools/bazel for details), bazel version 1.0.0 will be used instead of system-wide bazel installation.
INFO: Writing tracer profile to '/usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/command.profile.gz'
DEBUG: /usr/local/google/home/lidiz/.cache/bazel/_bazel_lidiz/6db67e514c85492694524d2ec993f002/external/bazel_toolchains/rules/rbe_repo/checked_in.bzl:226:13: rbe_msan not using checked in configs; Bazel version 1.0.0 was picked/selected with '["9.0.0", "10.0.0"]' compatible configs but none match the 'env = {"ABI_LIBC_VERSION": "glibc_2.19", "ABI_VERSION": "clang", "BAZEL_COMPILER": "clang", "BAZEL_HOST_SYSTEM": "i686-unknown-linux-gnu", "BAZEL_TARGET_CPU": "k8", "BAZEL_TARGET_LIBC": "glibc_2.19", "BAZEL_TARGET_SYSTEM": "x86_64-unknown-linux-gnu", "CC": "clang", "CC_TOOLCHAIN_NAME": "linux_gnu_x86", "BAZEL_LINKOPTS": "-lc++:-lc++abi:-lm"}', 'config_repos = None',and/or 'create_cc_configs = True' passed as attrs
INFO: Build option --runs_per_test has changed, discarding analysis cache.
INFO: Analyzed target //src/python/grpcio_tests/tests_aio/channelz:channelz_servicer_test (0 packages loaded, 3213 targets configured).
INFO: Found 1 test target...
Target //src/python/grpcio_tests/tests_aio/channelz:channelz_servicer_test up-to-date:
  bazel-bin/src/python/grpcio_tests/tests_aio/channelz/channelz_servicer_test
INFO: Elapsed time: 185.565s, Critical Path: 3.29s
INFO: 2000 processes: 2000 linux-sandbox.
INFO: Build completed successfully, 1001 total actions
//src/python/grpcio_tests/tests_aio/channelz:channelz_servicer_test      PASSED in 3.1s
  Stats over 1000 runs: max = 3.1s, min = 1.6s, avg = 2.1s, dev = 0.2s

INFO: Build completed successfully, 1001 total actions

Known failures: https://github.com/grpc/grpc/issues/22523 https://github.com/grpc/grpc/issues/18111


Running with docker compose won't refresh postee

when running  postee and posteeui via docker compose, the /server path get mounted to a persistent volume.
As the postee binary lives in this directory it won't get updated if you build a new version. Even if your new container contains the new postee binary, the mount of /server at runtime will hide the updated version and the postee version placed in the volume at the first creation of the volume will be started.
The postee binar should live outside of the mount/volume.

thanks - if you'd like to submit a PR we can take a look at it.

can we close this now @poggenpower?

Please reopen if needed.


"Approve claim" adds the claim with no reference

For instance, in https://www.wikidata.org/wiki/Q21536207#revision=333165966 :

if you first approve the (referenced) claim, it is added with no reference (cf. top one);
if you then approve the reference, it is duplicated in a new claim (cf. 2nd one);
the whole claim is still suggested (cf. bottom one, see #94)

I think this is a major source of frustration for users of the tool.

Closing, and leaving https://github.com/google/primarysources/issues/94 open instead.


add map page and add basic layers panel

Signed-off-by: Junqiu Lei<EMAIL_ADDRESS>Description

Update maps route

for maps list: app/mapsDashboards/list
for map create page: app/mapsDashboards/create

Add create map route and map page component
Add basic layers panel
Use Maplibre to show base map for development

Screenshot

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.

The alignment of the Layers and Roadmap in the screenshot seems to be not correct. Please fix that.

Also can we add the UX designer on this PR so that he can take a look at the screen shot only. So that there are no obvious alignment and other issues.

The alignment of the Layers and Roadmap in the screenshot seems to be not correct. Please fix that.

The commits I pushed is for the basic functions, will follow up UX design and update the PR.

Please make a change in the repo so what even when we are raising the PR which gets merged against a feature branch, build check and other checks are run. This will ensure that those checks are not run when we merge the feature branch in the main branch.

Sure, since it's new plugin, currently there seems no check workflow related to this feature branch,  will add some.

Also can we add the UX designer on this PR so that he can take a look at the screen shot only. So that there are no obvious alignment and other issues.

Yes, we need verify on the UI. Currently this PR are more focus on the basic functions, will ask UX designer for review once we have the final UI component as designed.


#38: Adding proxy support for oozie share lib download

Just added proxy support in HttpUtils class. It will allow users to use this framework when they are within corporate network.
Adding a test case as well but have disabled it because public proxy URL might change.
To use proxy, user just have to set any of following environment variable:
HTTP_PROXY, http_proxy, ALL_PROXY, all_proxy
These variables are used by default in LINUX/MAC environment (command line). When running from eclipse, users should set this variable explicitly.

Coverage decreased (-1.3%) to 76.264% when pulling 619fe99764f71eccbabe7c49a12793d0117de884 on rajesh-kumar:master into f9792fed4f036631d4b52abfc75ff207ea330b89 on sakserv:master.

It seems code coverage is not good. I will make functions public and add unit tests.

Coverage increased (+0.1%) to 77.671% when pulling ba44108f6c68bdab8d2dc990641713e5742c721c on rajesh-kumar:master into f9792fed4f036631d4b52abfc75ff207ea330b89 on sakserv:master.


Internal error when run edit function on movies documents in v1.10.2

Describe the bug
Base on this notion guide I call edit documents by function and got internal error.
curl http://localhost:7700/indexes/movies/documents/edit \
  -H 'content-type: application/json' \
  -d '{
    "filter": "id > 3000",
    "function": "doc.title = `✨ ${doc.title.to_upper()} ✨`"
  }'

Expected behavior
Server log:
2024-09-25T07:11:29.563769Z ERROR meilisearch: info=panicked at /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/rhai-1.19.0/src/engine.rs:347:58: called `Option::unwrap()` on a `None` value
2024-09-25T07:11:29.634669Z ERROR meilisearch: info=panicked at /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/rhai-1.19.0/src/engine.rs:347:58: called `Option::unwrap()` on a `None` value
2024-09-25T07:11:29.766880Z ERROR meilisearch: info=panicked at /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/rayon-par-bridge-0.1.0/src/lib.rs:47:5: a scoped thread panicked

Task details
 {
            "uid": 2,
            "indexUid": "movies",
            "status": "failed",
            "type": "documentEdition",
            "canceledBy": null,
            "details": {
                "editedDocuments": 0,
                "deletedDocuments": null,
                "originalFilter": "\"id > 3000\"",
                "context": null,
                "function": "doc.title = `✨ ${doc.title.to_upper()} ✨`"
            },
            "error": {
                "message": "An unexpected crash occurred when processing the task.",
                "code": "internal",
                "type": "internal",
                "link": "https://docs.meilisearch.com/errors#internal"
            },
            "duration": "PT0.391994763S",
            "enqueuedAt": "2024-09-25T07:11:29.371295421Z",
            "startedAt": "2024-09-25T07:11:29.380739658Z",
            "finishedAt": "2024-09-25T07:11:29.772734421Z"
        }

Meilisearch version:
v1.10.2

Hey @Ja7ad, the bug seems to be fixed.
I’ll integrate the fix into the next rc. It would be awesome if you could try it out once it’s released.
Thanks for finding and reporting the issue 🚀


Remove: TeamViewer.TeamViewer version 15.49.3

## Vanity URL - New version (15.50.5) is available in repo

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

URL: https://download.teamviewer.com/download/TeamViewer_Setup.exe
The SSL connection could not be established, see inner exception.
(Automated message - build 763)


Create moodboard layout

Examples in Powerpoint template: pages 21-22
An image grid with no text. 4 columns with either 1 or 2 rows per column.
Question
Will this ever be used by developers?

For now you can just use the available image layouts


[SPARK-28159][ML] Make the transform natively in ml framework to avoid extra conversion

What changes were proposed in this pull request?
Make the transform natively in ml framework to avoid extra conversion.
There are many TODOs in current ml module, like // TODO: Make the transformer natively in ml framework to avoid extra conversion. in ChiSqSelector.
This PR is to make ml algs no longer need to convert ml-vector to mllib-vector in transforms.
Including: LDA/ChiSqSelector/ElementwiseProduct/HashingTF/IDF/Normalizer/PCA/StandardScaler.
How was this patch tested?
existing testsuites

Test build #106872 has started for PR 24963 at commit e9e9c65.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12088/
Test PASSed.

KMeans and BikMeans are left alone, since there are many classes needed to be created on the ml side.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12095/
Test PASSed.

Test build #106879 has started for PR 24963 at commit f34112c.

Test build #106872 has finished for PR 24963 at commit e9e9c65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106872/
Test PASSed.

Test build #106879 has finished for PR 24963 at commit f34112c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106879/
Test PASSed.

Before I review, can you update the JIRA and PR with detail about what you're trying to do here? there's no real info.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12117/
Test PASSed.

Test build #106912 has started for PR 24963 at commit 51ee85c.

Test build #106912 has finished for PR 24963 at commit 51ee85c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106912/
Test PASSed.

@srowen Your method is more reasonable. It is better to maintain only one impl. I will try to add a method with array of double as input, on the .mllib side.

I means on the .mllib side, directly return a udf .ml.vector => double for the call on the .ml side.

Test build #106962 has started for PR 24963 at commit 92d555c.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12164/
Test PASSed.

Test build #106962 has finished for PR 24963 at commit 92d555c.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106962/
Test FAILed.

Test build #106997 has started for PR 24963 at commit 10ba449.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12195/
Test PASSed.

Test build #106997 has finished for PR 24963 at commit 10ba449.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106997/
Test PASSed.

Test build #107058 has started for PR 24963 at commit bd813db.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12251/
Test PASSed.

Test build #107058 has finished for PR 24963 at commit bd813db.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107058/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12271/
Test PASSed.

Test build #107092 has started for PR 24963 at commit 5730ab7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12272/
Test PASSed.

@srowen PCA is an exception, since it use matrix multiplication.

@srowen In this PR, I found that it will be more convenient (like IDF/ElementwiseProduct/StandardScaler) if there is some methods in linalg like:
def mapActive(f: (Int, Double) => Double): Vector
return a new vector whose values are computed by orignial vector and function f,
and f is only applied on active elements.

def updateActive(f: (Int, Double) => Double): Unit
like mapActive but update the values in-place

How do you think about this?

Test build #107092 has finished for PR 24963 at commit 5730ab7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107092/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12274/
Test PASSed.

Test build #107094 has started for PR 24963 at commit d54d073.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12276/
Test PASSed.

Test build #107094 has finished for PR 24963 at commit d54d073.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107094/
Test PASSed.

@srowen Yes, I broken out the impl in .mllib to expose methods for dense and spares (excpet PCA), and call them from .ml to avoid conversion.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12320/
Test PASSed.

Test build #107148 has started for PR 24963 at commit f1314fb.

Test build #107148 has finished for PR 24963 at commit f1314fb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107148/
Test FAILed.

Test build #107153 has started for PR 24963 at commit 096d204.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12326/
Test PASSed.

Test build #107153 has finished for PR 24963 at commit 096d204.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107153/
Test PASSed.

Merged to master


custom url scheme doesn't work in iOS 1.38.1

Submit feature requests to http://www.mattermost.org/feature-requests/. File non-security related bugs here in the following format:
Summary
custom url scheme doesn't work in the newest iOS version 1.38.1
Environment Information

Device Name: iPad Pro 11 (2020)
OS Version: 14.2 or 14.3
Mattermost App Version: 1.38.1
Mattermost Server Version: 5.25.0

Steps to reproduce
configure custom url scheme, download iOS 1.38.1
Expected behavior
trigger another app
Observed behavior
nothing has happened
Possible fixes
the same configuration has worked with version 1.36, after updte doesn't work anymore.

@cafefish Can you share what is the scheme? Also schemes are directly controlled by the OS so I find this to be very curious...
@migbot could this be based on the change made to use Linking.canOpenUrl?

"mpgreader" for launching an internal app


switch to docfx v3

Docs Build status updates of commit 45dfeeb:
:warning: Validation status: warnings

File
Status
Preview URL
Details

java/docs-ref-autogen/com.microsoft.store.partnercenter.IPartnerComponent.getContext.yml
:warning:Warning

Details

java/docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml
:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

java/docs-ref-autogen/com.microsoft.store.partnercenter.IPartnerComponent.getContext.yml

Line 15, Column 3: [Warning-yaml-duplicate-key] Key 'name' is already defined, remove the duplicate key.
Line 16, Column 3: [Warning-yaml-duplicate-key] Key 'nameWithType' is already defined, remove the duplicate key.

java/docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml

Line 11, Column 9: [Warning-duplicate-uid] UID 'com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL' is duplicated in 'docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml(11,9)', 'docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml(16,6)'.
Line 16, Column 6: [Warning-duplicate-uid] UID 'com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL' is duplicated in 'docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml(11,9)', 'docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml(16,6)'.

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

Docs Build status updates of commit faca069:
:warning: Validation status: warnings

File
Status
Preview URL
Details

java/docs-ref-autogen/com.microsoft.store.partnercenter.IPartnerComponent.getContext.yml
:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

java/docs-ref-autogen/com.microsoft.store.partnercenter.AggregatePartnerOperations.ADD_APPLICATION_CREDENTIAL.yml
:white_check_mark:Succeeded

java/docs-ref-autogen/com.microsoft.store.partnercenter.IPartnerComponent.getContext.yml

Line 15, Column 3: [Warning-yaml-duplicate-key] Key 'name' is already defined, remove the duplicate key.
Line 16, Column 3: [Warning-yaml-duplicate-key] Key 'nameWithType' is already defined, remove the duplicate key.

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


Closing search results causes crash

Version and Platform (required):

Binary Ninja Version: 3.0.3337-dev (2022-04-08 19:33:12)
OS: MacOS
OS Version: Monterey 12.3.1

Bug Description:
Binaryninja crashes whenever closing the search results box window.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Go to Edit -> Find (or CMD-F)
Perform a search
Close the result window after the search displays the results

Expected Behavior:
Should be able to close the search results and still remain in the same view in the UI before the search window was opened

I'm not able to replicate this using the exact same BN on the exact same MacOS.
Can you try with:

Another file
Disabling user settings
Disabling plugins

To clarify, you're using the "find all" checkbox?

ah.. yes I forgot to add that this happens with the "find all" option checked. I've tested multiple files with user settings and plugins disabled and as long as the "find all option" is checked binja crashes on my installation.

Thanks. It looks like this bug is happening only on x64 builds on MacOS,  not M1. We've reproduced it and are testing a fix.

Fixed in build 3338

This is why ya'll can have all my money!


Add hanami framework entry to libs/ruby.json

Added entry for the hanami ruby framework.

@rodrigopk  please make sure to run |node verify.js|, this tool will sort libs, check for duplication and update technologies.json, and then push the changes


python-pptx pptx file split can do it? I hope you help answer，thanks！

Hi steve,
I tried using python-pptx to split a single page ppt, but couldn't find a way.
Can it split a single page ppt?
Hope you can help answer, thank you ~

Please post support questions on StackOverflow using the python-pptx tag. You'll need to describe what you're trying to accomplish more carefully. In particular, what is a "single-page ppt"? Do you mean a PowerPoint presentation with a single slide? What would splitting mean?


THEOPHILUS ANUM ADJEI

WS2_M2E2_Submission_Theophilus Anum Adjei.docx

Well done, Theophilus. You clearly grasped the material outlined in Module 2 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


[New game]:

🎮 Game Request
It's a Ludo Game - there's already one - but there's no safe zone in this one. This one can be played by 2, 3 or 4 players, while the other only has a 2-player mode.
Point down the features
You can choose the name and numbers of players.
There is no safe zone,  so you have to be aware of the position of others.
Select program in which you are contributing
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @JPreault !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @JPreault!
We are already having a similar game request in #3018 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @JPreault ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


feat(builtin-gateways): add rules-based policies

You can use the custom test-meshgateway entity to look at a "real enough" and not randomized example.
Changes
chore: mock MeshHTTPRoute and MeshGateway _rules
feat(builtin-gateways): add rules-based policies
Update the built-in gateway detail view to show which rules from MeshHTTPRoutes and MeshTCPRoutes apply to which listeners and to which destination services they resolve.
Move the Dataplanes table into its own tab because selecting a listener in the new UI makes the content beneath it jump around.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>Resolves #2208.
Screenshots

Heads up incase: I just noticed we are probably going to tread on toes here between this one and https://github.com/kumahq/kuma-gui/pull/2247, did you want to try and get 2247 in first?

I’m going to open this up for review now. There are some items left to discuss with design mostly around the topic of information hierarchy in the Routes and Services part of the view. There’s quite a lot of relevant information that needs to be shown. The most tricky aspect is nicely representing multiple rules (i.e. the pair of matches and default) within a route card.

Did I see right that we aren't using nested routes for this? And if not how come we aren't? I guess there's a reason not to?

Did I see right that we aren't using nested routes for this? And if not how come we aren't? I guess there's a reason not to?

That’s a bit of a vague question, but I’m guessing you mean the boundary between the view where a selected listener is in play and the one where there isn’t. I had it like that before and found no benefits to it (rather only complications) and changed it again.

(rather only complications)

And if not how come we aren't? I guess there's a reason not to?

i.e. Could you detail these complications for me?

i.e. Could you detail these complications for me?

It’s just more cumbersome to use a RouterView as the interface for a statically resolved component. You get no prop checking and need a separate route. There’s no need for a separate route here.

Please could you use nested routing for this.
There have been several PRs over the past few weeks moving areas that weren't using nested routing to use nested routing. Doing this fixed several bugs and made several UX improvements (not to mention the DX improvements)

If you can point out what can be improved by using nested routes here, I will consider it. For the time being, I’ll consider this suggestion one of technicality. I’m not interested in changing things around "just because".

The entire project uses nested routing to do this sort of thing, I don't see any reason to do anything different here.
At a general level, it prevents very many negative things as has been seen in moving the places where it wasn't being used to use it.

Compact Name and Value for matches
Add tags for meshServiceSubset (represent the targetRef). Can we design a component for targetRef rendering/linking? --> Potentialy do as a follow up issue/PR
For weight e.g: :wrench::20 can we have a weight icon and a tooltip to say weight. We can ask team to get an icon for this: https://kong.github.io/icons/

Compact Name and Value for matches
Add tags for meshServiceSubset (represent the targetRef). Can we design a component for targetRef rendering/linking? --> Potentialy do as a follow up issue/PR
For weight e.g: 🔧:20 can we have a weight icon and a tooltip to say weight. We can ask team to get an icon for this: https://kong.github.io/icons/
Show a * matcher when matchers is empty (similar to what we do for policies).
Remove Overview and type to just have MeshHTTPRoute or MeshTCPRoute in top left (like we do for listener type).
Needs support for filters. (how do this relate to point 8).
Discuss with design to not show matchers and origin by default (have a toggle for users to see them).
Discuss with design the possibility of having Matches and Services next to each other instead of one after the other.

I’ve done the following now:

Compact Name and Value for matches
Add tags for meshServiceSubset (represent the targetRef). Can we design a component for targetRef rendering/linking? --> Potentialy do as a follow up issue/PR
For weight e.g: 🔧:20 can we have a weight icon and a tooltip to say weight.
Show a * matcher when matchers is empty (similar to what we do for policies).
Remove Overview and type to just have MeshHTTPRoute or MeshTCPRoute in top left (like we do for listener type).

Left to do:

Needs support for filters. (how do this relate to point 8).
Discuss with design to not show matchers and origin by default (have a toggle for users to see them).
Discuss with design the possibility of having Matches and Services next to each other instead of one after the other.

... and the weight icon thing, but that can be done in a follow up, too.

Seems like we've lost the path type now:

Seems like we've lost the path type now:

I’ve fixed this.
I also implemented this now:

6. Needs support for filters. (how do this relate to point 8).

This is a bit of a best effort implementation as there’s quite a bit to handle with filters. We can iterate on this in a follow up.

I created an issue for the two outstanding items that depend on design input we discussed previously: https://github.com/kumahq/kuma-gui/issues/2368
The rest of the feedback was addressed and this PR can be re-reviewed.

Is it just me or am I not seeing any listeners now on the preview site now? No matter which one I click on I get this:

Is it just me or am I not seeing any listeners now on the preview site now? No matter which one I click on I get this:

I can definitely get gateways with listeners entries, but it does seem a little "too rare" for preview purposes. I can see if I can tweak this somehow.

Ok cool, if it's just a mock thing don't worry too much about it

Fix i18n problem (don't i18n keys)
Don't use a pill for type/action use key:value

* For request mirror don't put the number in the service pill. Put it aside as `50%`

I’ve addressed these items now.

@johncowen Please finish reviewing this.

Could we take a look at that?

That looks like an invalid TCP route. It has no matches property in its rules objects and so the _rules API doesn’t have any rules for this.

LGTM from a design POV


implement things.trash for #16

Codecov Report

Merging #65 (8fced32) into main (daf2b31) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
+ Coverage   86.84%   86.99%   +0.15%
==========================================
  Files           2        2
  Lines         342      346       +4
==========================================
+ Hits          297      301       +4
  Misses         45       45

Impacted Files
Coverage Δ

things/api.py
84.67% <100.00%> (+0.25%)
:arrow_up:

things/database.py
88.28% <100.00%> (+0.10%)
:arrow_up:

Continue to review full report at Codecov.

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

@AlexanderWillner
Wonderful job on getting coverage up! ✨
Looks good. Comments mostly around consistency and some API-related.
I'd strongly suggest you add documentation for trashed in things.tasks as well. Maybe just below the deadline parameter?

@AlexanderWillner LGTM once two isort flags are added.

@AlexanderWillner Two clarifications (copied from earlier up):

When we ask for things.trash, are tasks within projects included or omitted?
Would calls involving include_items=True do what we expect for trashed projects?

@AlexanderWillner One more thing. Do you see the # Validation section database.get_tasks? It might be consistent to put trashed in there too.

When we ask for things.trash, are tasks within projects included or omitted?

I'd mimic the behaviour of Things and include them (clicking on Trash shows them as well). Updated api.trash accordingly.

Would calls involving include_items=True do what we expect for trashed projects?

No :/

When we ask for things.trash, are tasks within projects included or omitted?

I'd mimic the behaviour of Things and include them (clicking on Trash shows them as well). Updated api.trash accordingly.

If I click on my local Things app Trash, I see the following:

If a project got deleted that still had open to-dos and headings, then those to-dos and headings are included as part of the project. That is, I can only see those to-dos and headings if I go into the project (⌘→).
If individual headings got deleted from a project, but not the project itself, then I don't see those headings in the Trash.
If individual to-dos got deleted from a project, but not the project itself, then I do see those to-dos listed in the Trash by themselves.

If a project got deleted that still had open to-dos and headings, then those to-dos and headings are included as part of the project. That is, I can only see those to-dos and headings if I go into the project (⌘→).

I believe this is how it's implemented in the API now. At least the first part. For the last part (include_items=True) is an open todo in the code.

If individual headings got deleted from a project, but not the project itself, then I don't see those headings in the Trash.

I've to admin, I didn't test with headings at all.

If individual to-dos got deleted from a project, then I see those to-dos listed in the Trash by themselves. This is also the case, when the project itself is deleted.

I believe this is how it's implemented in the API now.

👌

@AlexanderWillner Well done! ☀


ENH add the yulesimon distribution to scipy.stats

Closes #9183

rvs is very slow for small values of alpha, I guess one needs _ppf

rvs is very slow for small values of alpha, I guess one needs _ppf.
If i understand Wikipedia correctly, you can also generate rvs by first generating an exponential rv W with parameter Alpha and then a geometric rv with parameter exp(-W)

Yeah this is true, I'll need to look into how geometric is parametrized within scipy to make sure this is setup right. According to Devroye there is a faster way. I guess I could put the mixture representation in now and then check the speed of Devroye's Accept-Reject algorithm and open a different PR for that one. I'll do the geometric mixture implementation for _rv.  Is the issue you opened: #9313 related to this comment?

I'll do the geometric mixture implementation for _rv. Is the issue you opened: #9313 related to this comment?

yes, it is, i had a quick look at that approach and unfortunately, the exponential rv can easily take on values > 30 if the rate is high enough and one ends up with very small success rates for stats.geom.
can you check the book "Luc Devroye: Non-Uniform Random Variate Generation", it has a short section on the yule distribution. you can find the book online. Maybe that helps

yeah I've got a digital and physical copies of NURVG. I commented on the other issue for geom, I see the same thing but for different values. Those isses are related to the geometric distribution so they'll have to be fixed in that issue. There are actually 2 issues, 1. The algorithm from Devroye's book is only valid for $\alpha >1$ where the mean exists. That can be implemented but a different algorithm will be required for $0<\alpha<=1$. According to a paper Devroye wrote in 1992 there is an algorithm that works for the small alpha case-and is efficient-but I don't fully understand the algorithm yet. Here is the link: http://luc.devroye.org/Devroye-Digamma+Trigamma-1992.pdf

Second issue is something is flipped somewhere in the algorithm. A large value of $\alpha$ should result in nearly all values being 1s and a couple 2s out of a sample of 100 (say) whereas a small value (less than 1) will not have a finite mean and therefore takes on many arbitrarily large values. I'll need to track down the root of the switch.

Reg 1. Indeed, the YS distribution is a special case of the GHgB3 discussed in the paper you mentioned, which can be sampled by compounding distributions. The notation in p.3 of that paper is unclear to me, it is explained in the paper of sibuya 1979, see eq 6, 7,8 there (see references in the devroye paper, Looks like one can get around using geom). I understand that digamma and trigamma are even more complicated and we don't need that here.
What are actually typical parameters for alpha?

I must've missed the email in my scipy folder that notified me of this update.

What are actually typical parameters for alpha?

For a preferential attachment graph the alpha value is usually somewhere between 2 and 4 (asymptotically around 3). I looked into Devroye's paper in some detail as well, I think the parameterization on wikipedia $\rho = a-1$ where a is the notation used in Devroye's NURVG book.  Confusingly, in the 1992 Devroye paper the yule simon is the special case of GHgB3 where a=b=1 and c>0 corresponds to $\rho$ on the wikipedia notation (a-1) in NURVG book notation. So I think the geometric mixture theoretically works for all values (but is indeed slow for large values. For the rejection algorithm in section 2 of Devroye's 1992 paper if a=1 then the algorithm never rejects. However, you need to have available Poisson(lambda), Exponential(1), and Gamma(c,1) with 0<c<1 r.v. generators that are good for all values of the parameters lambda, 0<c<1. I'm not sure that SCIPY has these specific implementations for the r.v.s. I'm investigating using a discrete hazard rate method from Devroye;s NURVG book (pg 283) if that algorithm is faster I'm push it otherwise I think I'm stuck with the geometric mixture code which is the first distributional identity of Devroye's 1992 paper on page 3 (w/a=b=1).

then you can use directly the method in 6.3 of Devroye's book (here speed does not depend in alpha), no?

Correct, speed does not depend on alpha in method 6.3 in NURVG but does depend on numerical stability of geometric r.v. generator, if the mixture representation is used, or the stability of log1p() function for large values. (6.3 is a restatement of the mixture representation into a single equation. '
Thanks for the suggestion, I'll do 6.3 too and compare with discrete hazard rate method-will be interesting to compare times.

I see a ratio of two exponential rvs in 6.3:
X = -E1 / log(1-Exp(-E2/log(1-a)))
E1 and E2 iid exponential. This avoids the problem of sampling from a geometric rv with parameter
close to zero.

This avoids the problem of sampling from a geometric rv with parameter
close to zero.

Sort of, this is the same algorithm as the mixture representation but all put into a single r.v. generation step. The instability problem in the geometric is the .log() vs .log1p() methods that will need to be handled here.
One thing that isn't clear to me in the unit tests is the broadcasting support tests in the tests/test_discrete_basic.py LIC 92 which singles out Skellam. We need to do something similar here, but it isn't clear to me what the test tests for in this case. Do you understand this test @chrisb83? Any elucidation is appreciated.

One thing that isn't clear to me in the unit tests is the broadcasting support tests in the tests/test_discrete_basic.py LIC 92 which singles out Skellam.

The test still isn't clear to me but it appears to be passing my local build via:
python3 ../../runtests.py -v -s stats
output:
=================================================== 1253 passed, 4 skipped, 378 deselected, 3 xfailed, 1 xpassed in 40.11 seconds ===================================================

rvs looks good, doc and stats need a bit more work

Regarding the test, I don't know the details but the following comment explains a bit:

If shape_only is True, it means the _rvs method of the distribution uses more than one random number to generate a random variate. That means the result of using rvs with broadcasting or # with a nontrivial size will not necessarily be the same as using the numpy.vectorize'd version of rvs(),

So I guess excluding yulesimon is OK, but If you say the test passes, feel free to include it.

So I guess excluding yulesimon is OK, but If you say the test passes, feel free to include it.
What I'm saying is the test only passes if I add the distname to the exclude list with zipf.
FWIW the two output arrays are the same size (so broadcasting is "working") and the output results sets are very close, a the 2 output arrays x and y are the same in every entry except 1 where they differ by 1. My guess is during the broadcast something impacts the random number generators so the results aren't exactly the same. This is mentioned in the comment in the test. Here is the stacktrace from local test run of scipy:
python3 ../../runtests.py -v -s stat
distfunc = <scipy.stats._discrete_distns.yulesimon_gen object at 0x1169a4358>, distname = 'yulesimon', allargs = [array([[11.],
[11.],
[11.]]), array([0., 0.])]
shape = [3, 2], shape_only = False, otype = [<class 'numpy.int64'>]

def check_rvs_broadcast(distfunc, distname, allargs, shape, shape_only, otype):
    np.random.seed(123)
    with suppress_warnings() as sup:
        # frechet_l and frechet_r are deprecated, so all their
        # methods generate DeprecationWarnings.
        sup.filter(category=DeprecationWarning, message=".*frechet_")
        sample = distfunc.rvs(*allargs)
        assert_equal(sample.shape, shape, "%s: rvs failed to broadcast" % distname)
        if not shape_only:
            rvs = np.vectorize(lambda *allargs: distfunc.rvs(*allargs), otypes=otype)
            np.random.seed(123)
            expected = rvs(*allargs)

          assert_allclose(sample, expected, rtol=1e-15)

E               AssertionError:
E               Not equal to tolerance rtol=1e-15, atol=0
E
E               (mismatch 16.66666666666667%)
E                x: array([1, 1, 1, 1, 1, 1])
E                y: array([1, 1, 1, 2, 1, 1])
allargs    = [array([[11.],
[11.],
[11.]]), array([0., 0.])]
distfunc   = <scipy.stats._discrete_distns.yulesimon_gen object at 0x1169a4358>
distname   = 'yulesimon'
expected   = array([[1, 1],
[1, 2],
[1, 1]])
otype      = [<class 'numpy.int64'>]
rvs        = <numpy.lib.function_base.vectorize object at 0x11b1e1358>
sample     = array([[1, 1],
[1, 1],
[1, 1]])
shape      = [3, 2]
shape_only = False
sup        = <numpy.testing._private.utils.suppress_warnings object at 0x11b1e1ba8>
scipy/stats/tests/common_tests.py:306: AssertionError
============================================== 1 failed, 1252 passed, 4 skipped, 378 deselected, 3 xfailed, 1 xpassed in 35.71 seconds ==============================================

the _rvs method of the distribution uses more than one random number to generate a random variate.

This seems to be the case in _rvs implementation; uses 2 exponential(1) r.v.s to generate a single yulesimon, so I'll keep it in exclude list.

Notes on build fails: looks like travis build is the same optimize tests from previous build, either flaky tests or something changed and I need to FF by dev-branch. Still no changes to optimize sub-module codes.
Circle CI builds fail for building the docs, but I successfully built the docs on my dev env on local, via:
python setup.py build_sphinx
open build/sphinx/html/stats.html

I see the issue now:
>>> ys_rv = yulesimon(1.1)
>>> ys_rv.std()
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:952: RuntimeWarning: invalid value encountered in sqrt
  sqrt(alpha - 2) * (alpha + 1)**2 /
inf

but should indicate np.nan

Ok working correctly now;
Python 3.6.6 |Anaconda, Inc.| (default, Jun 28 2018, 11:07:29)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from scipy.stats import yulesimon
>>> ys_rv=yulesimon(1.1)
>>> ys_rv.std()
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:955: RuntimeWarning: invalid value encountered in sqrt
  sqrt(alpha - 2) * (alpha + 1)**2 /
inf
>>> ys_rv=yulesimon(0.5)
>>> ys_rv.std()
nan

I'm still unclear if the std() method is behaving as desired in the 1<alpha<=2 case where the code takes square root of np.inf and throws a runtime warning. If I check the t distribution:
>>> from scipy.stats import t
>>> rv_t=t(1.5)
>>> rv_t.mean()
0.0
>>> rv_t.expect()
0.0
>>> rv_t.std()
inf
>>> rv_t.stats()
(array(0.), array(inf))

with no runtime warning. I'll push what I have now, addressing @chrisb83 comments, but does not address the runtime warning.

i cannot reproduce the runtime warning (yulesimon(1.1).std() returns inf), std should simply return the sqrt of stats(moments='v'). does stats() produce the warning as well?

apart from the minor doc issue (indentation of reference that crashes the build), I think this is complete

i cannot reproduce the runtime warning (yulesimon(1.1).std() returns inf), std should simply return the sqrt of stats(moments='v'). does stats() produce the warning as well?

I'm no longer getting the error on alpha=1.1 either-I'm guessing the additional np.where()s fixed that one. I do still get the error for alpha==1, but not alpha==1.0 so there must be some casting issue somewhere in the np.where() function is my guess.
``

yulesimon(1).std()
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:949: RuntimeWarning: divide by zero encountered in true_divide
mu = np.where(alpha <= 1, np.inf, alpha / (alpha - 1))
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:951: RuntimeWarning: divide by zero encountered in true_divide
alpha**2 / ((alpha - 2.0) * (alpha - 1)**2),
nan
yulesimon(1.0).std()
nan

i cannot reproduce the runtime warning (yulesimon(1.1).std() returns inf), std should simply return the sqrt of stats(moments='v'). does stats() produce the warning as well?

I'm no longer getting the error on alpha=1.1 either-I'm guessing the additional np.where()s fixed that one. I do still get the error for alpha==1, but not alpha==1.0 so there must be some casting issue somewhere in the np.where() function is my guess.
``

yulesimon(1).std()
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:949: RuntimeWarning: divide by zero encountered in true_divide
mu = np.where(alpha <= 1, np.inf, alpha / (alpha - 1))
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:951: RuntimeWarning: divide by zero encountered in true_divide
alpha**2 / ((alpha - 2.0) * (alpha - 1)**2),
nan
yulesimon(1.0).std()
nan

and then it stoppped being an error on my machine.... weird. Must be some delayed updating in the dev environment setup...

>>> yulesimon(3).std()
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:956: RuntimeWarning: divide by zero encountered in true_divide
  (alpha * (alpha - 3)),
/Users/rlucas/scipy-dev/scipy/scipy/stats/_discrete_distns.py:961: RuntimeWarning: divide by zero encountered in true_divide
  (alpha * (alpha - 4) * (alpha - 3)),
1.5
>>> yulesimon(3.0).std()
1.5
>>> yulesimon(3.0).std()
1.5
>>> yulesimon(3).std(

The optimize fails are due to the new NumPy built on the rackcdn. It went up from 1.6.0 to 1.6.3 and something has changed along the way. Still trying to find out what exactly.

Looks like all builds pass w/the new changes except the optimize sub-module unit-tests that ilayn is looking into w/the numpy version update.
Pyflakes is passing PEP8 checks 👍

There is still an error in the first of the py2.7 PYFLAKES=1 PEP8=1 NUMPYSPEC="--upgrade numpy":
scipy/stats/_discrete_distns.py:955:31: E225 missing whitespace around operator
scipy/stats/_discrete_distns.py:960:31: E225 missing whitespace around Operator
I meant to write g1 and g2 as follows, i.e. no linebreak after /:
g1 = np.where(alpha > 3,
  sqrt(alpha - 2) * (alpha + 1)**2 / (alpha * (alpha - 3)),
  np.inf)

There is still an error in the first of the py2.7 PYFLAKES=1 PEP8=1 NUMPYSPEC="--upgrade numpy":
scipy/stats/_discrete_distns.py:955:31: E225 missing whitespace around operator
scipy/stats/_discrete_distns.py:960:31: E225 missing whitespace around Operator
I meant to write g1 and g2 as follows, i.e. no linebreak after /:
g1 = np.where(alpha > 3,
  sqrt(alpha - 2) * (alpha + 1)**2 / (alpha * (alpha - 3)),
  np.inf)

I can make that change for g1 code but not for g2, it will go past the 80 char line limit.

I can make that change for g1 code but not for g2, it will go past the 80 char line limit.

True, but one breach is acceptable in my view since we don't want to spend more time on this :)

You can always place a continuation character backslash \ at the end of the line and continue at the next line.

You can always place a continuation character backslash \ at the end of the line and continue at the next line.

When I did this in the travis build for python 2.7 that checks PEP8 compliance I got an error:
scipy/stats/_discrete_distns.py:956:71: E502 the backslash is redundant between brackets
so I removed the / line continuation and pushed the change.

Yes, so when there is an open bracket the new line doesn't need a backslash,
By the way if you pip install pycodestyle (formerly it was pep8) you can locally run
pycodestyle <filename.py>

and it will run the same test for you hopefully reducing the burden of committing each time and waiting.

Yes, so when there is an open bracket the new line doesn't need a backslash,
By the way if you pip install pycodestyle (formerly it was pep8) you can locally run
pycodestyle <filename.py>

and it will run the same test for you hopefully reducing the burden of committing each time and waiting.

@ilayn Thanks! I've been using pyflakes but I'll add this one to my workflow too. Good tip.

Thanks for your effort, remaining Travis error is unrelated. PR can be merged. Let's wait a few more >days if there is further feedback

@chrisb83 Thank you for your time and assistance with this PR, super helpful. Hopefully, I can find some other code changes/fixes/etc. in the repo to pitch in on too.

@chrisb83 its been a few days and no further comments. Should we merge or do we want another approver beforehand? Would be awesome to get this into the next release.

@ilayn i plan to merge this PR later today, I think your comments have been addressed.

@rlucas7 merged, thanks!


let proxy option allow self-signed certificates for https

I am running an https server locally that uses self-signed certificates. When I include the following in my testem.json, testem fails to start.
"proxies":   {
    "/api": {
      "target": "https://localhost:10443"
    }

The error:
Error: self signed certificate
    at Error (native)
    at TLSSocket.<anonymous> (_tls_wrap.js:929:36)
    at TLSSocket.emit (events.js:104:17)
    at TLSSocket._finishInit (_tls_wrap.js:460:8)

It would be nice to have this as an option to configure, since my local tests do not need production https credentials.

My above configuration is incorrect and testem starts fine when I add "secure": false
"proxies":   {
    "/api": {
      "secure": false,
      "target": "https://localhost:10443"
    }

I think it would be helpful to add some documentation for this, because the above configuration is not especially self-explanatory.  Let me know you would like a PR with some extra verbiage explaining this use case.

Yes, a PR with some additional details around proxy configuration is more then welcome.


Boostrapping Patch command

https://github.com/gunnarmorling/kcctl/issues/15
Notes:

Added API, helper class and patch command.
Tested this locally:

mvn compile quarkus:dev -Dquarkus.args='patch logger com.snowflake.kafka.connector -l INFO' -Dsuspend

Can add more comments and actual logic in next commits.
API for reference: Link

Thanks a lot @sfc-gh-japatel, that's a great start!

Can add more comments and actual logic in next commits.

Looking forward to it.
@aalmiray, for some reason this hasn't triggered a PR build run on GH Actions. Could anything have been broken by the latest release related work by any chance?

Ah, seems it's here where it went missing: https://github.com/gunnarmorling/kcctl/commit/5885d1d7b184bf381d4671b93c3c6050f13e4346#diff-5dbf1a803ecc13ff945a08ed3eb09149a83615e83f15320550af8e3a91976446L21-L23.

Oops. I removed the PR trigger for regular build. My mistake.

No worries. Could you bring it back?

Done https://github.com/gunnarmorling/kcctl/commit/9598e8c9de05f7aed79aa943752e5b4c7818f0f9

Thanks, @aalmiray!

Hey @sfc-gh-japatel, thanks for the update! Could you run mvn license:format and commit the result (currently, the license headers are missing from the new files)? Could you also update the output of kcctl help in the README, showing the new command?

Rebased and applied. Thanks a lot, @sfc-gh-japatel! Note I've split up get loggers and get logger into two commands, so to be consistent with the experience known from _kubetctl. Thanks again!


Test the use of non-root build scripts

Description of changes:
We want to build the cloud extensions in the docker container (similarly to the sample apps, like in https://github.com/aws-robotics/aws-robomaker-sample-application-objecttracker/pull/75 and https://github.com/aws-robotics/aws-robomaker-sample-application-objecttracker/pull/76) as a non-root user to better reflect real-world usage.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Testing complete.


CLI 'VirtualMachineScaleSetsOperations' object has no attribute 'reimage'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az vmss reimage
Errors:
'VirtualMachineScaleSetsOperations' object has no attribute 'reimage'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 659, in execute
    raise ex
cli/core/commands/__init__.py, ln 722, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
...
azure/cli/core/__init__.py, ln 784, in default_command_handler
    return op(**command_args)
cli/command_modules/vm/custom.py, ln 2787, in reimage_vmss
    return sdk_no_wait(no_wait, client.virtual_machine_scale_sets.reimage, resource_group_name, vm_scale_set_name)
AttributeError: 'VirtualMachineScaleSetsOperations' object has no attribute 'reimage'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az vmss reimage --name {} --resource-group {}

Expected Behavior
Environment Summary
Linux-4.15.0-1103-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.17.1
azure-cli-ml 1.19.0

Extensions:
ai-examples 0.2.5
azure-cli-ml 1.19.0

Additional Context

@qwordy is this fixed?

@qwordy is this fixed?

@qqnarwhal Thank you for reporting this issue. It is fixed now. Release date is 01/19/2021.
PR: https://github.com/Azure/azure-cli/pull/16483
Similar issue: https://github.com/Azure/azure-cli/issues/16465

@qqnarwhal Thank you for reporting this issue. It is fixed now. Release date is 01/19/2021.
PR: https://github.com/Azure/azure-cli/pull/16483
Similar issue: https://github.com/Azure/azure-cli/issues/16465


⚠ Bofrost.lu has degraded performance

In b812c85, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1288 ms

Resolved: Bofrost.lu performance has improved in d8cec75 after 13 minutes.


Generating Docs with Namespace results in @see references being incorrect

Please note the two screen shots below: one was generated with package namespace and the other not.
The one without namespace correctly links to other generated classes.

Delivered in <IP_ADDRESS>.


Upgrade ClimaCore

This PR:

Removes the CUDA 5 compat entry (we need CUDA 4 to upgrade ClimaCore)
Updates the ClimaCoreMakie compat
Updates the manifest files

Looks good to me!

bors r+

As @Sbozzolo pointed put on Thursday, we should be bumping the compat requirements for ClimaCore


Warp error message

Factorio version
Version number: 0.18.31
Scenario version
Version number: Whatever's running on S1 15th June 2020
Bug description
After warping the Warp message when you try to create a Warp too close to an existing one gives the warp you travelled from rather than the warp you are too close to.  This is resolved by walking "far enough" away to create a warp then back.
Reproduction method
Steps to reproduce the behavior:

From warp 'B' warp to warp 'A'
Hover over '+' on warp list to create new warp
Look at mouseover text.

Expected behavior
Mouseover text would show warp 'A' too close
Screenshots

Additional context
None

This issue is a result of overly aggressive lag reduction used to avoid api calls.
The code that is used for this will only update the tooltip when you were able to make a warp but now are unable to, seen on line 599.
Either we remove this check, resulting in alot of api calls, or we need a way to check if the closest warp has changed.

Fixed in v6.0.10
See commit


how to build libxgboost.dll by msvc ?

hi, i want install xgboost on windows 10  by vs2015, after read the installation guide, i know i just need build a libxgboost.dll, but I cannot found any  useful messages about build this dll file by mcvs, however, i google the installation way, most of the answer like these, cd to xgboost/windows, or xgboost/xgboost-master, then build ......but there are no windows/xgboost-master folder in xgboost ...

reference  https://github.com/dmlc/xgboost/issues/736, seems windows folder is obsoleted, cmake/mingw is the only way in window, i will try~

tutorial from ibm blog is working, check comments here:
https://github.com/dmlc/xgboost/issues/1049

check this. http://www.thushv.com/uncategorized/i-knew-you-were-trouble-xgboost-windows-a-proper-guide/
Downdrade to V 4.0 stable (not the v4.X) using git commit. Then you will get the old VS projects and the wrapper.

0.4 stable via VS is working, but there is nice feature in master branch: setting learning_rate different for each boosting round...

@gugatr0n1c @thusithaC  thanks for your help~ i installed git for windows and mingw64, added mingw64/bin to system path, open  git bash, exec following  commands
$ git clone --recursive https://github.com/dmlc/xgboost
$ cd xgboost
$ git submodule init
$ git submodule update
$ alias make='mingw32-make'
$ make -j4
however ,make fail for below erros~
g++ -m64 -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -MM -MT build/c_api/c_api_error.o src/c_api/c_api_error.cc >build/c_api/c_api_error.d
g++ -m64 -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -MM -MT build/learner.o src/learner.cc >build/learner.d
g++ -m64 -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -MM -MT build/logging.o src/logging.cc >build/logging.d
g++ -m64 -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -MM -MT build/common/common.o src/common/common.cc >build/common/common.d
g++ -m64 -c -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -c src/common/common.cc -o build/common/common.o
g++ -m64 -c -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -c src/c_api/c_api_error.cc -o build/c_api/c_api_error.o
g++ -m64 -c -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -c src/logging.cc -o build/logging.o
g++ -m64 -c -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -c src/learner.cc -o build/learner.o
In file included from include/xgboost/./base.h:10:0,
from include/xgboost/logging.h:13,
from src/logging.cc:7:
dmlc-core/include/dmlc/omp.h:9:17: fatal error: omp.h: No such file or directory
compilation terminated.
Makefile:97: recipe for target 'build/logging.o' failed
mingw32-make: *** [build/logging.o] Error 1
mingw32-make: *** Waiting for unfinished jobs....
In file included from include/xgboost/./base.h:10:0,
from include/xgboost/logging.h:13,
from src/learner.cc:7:
dmlc-core/include/dmlc/omp.h:9:17: fatal error: omp.h: No such file or directory
compilation terminated.
Makefile:97: recipe for target 'build/learner.o' failed
mingw32-make: *** [build/learner.o] Error 1
dmlc-core/include/dmlc/omp.h is exist in fact~

i try to make in another way, cp make/mingw.mk dmle-core/config.mk, and then cd dmlc-core/ to make -j4 ,but also meet the same problem,

Hi,
Is this the V4.0 or V4.x ? The latter I still cant compile successfully
using mingw64.
One more trick, someone mentioned making the alias 'make' is not enough.
Try going to the mingw/bin folder and making a copy of ming32-make file and
renaming it to make.
Good luck.
On Thu, Mar 31, 2016 at 7:08 PM, leoli2012<EMAIL_ADDRESS>wrote:

i try to make in another way, cp make/mingw.mk dmle-core/config.mk, and
then cd dmlc-core/ to make -j4 ,but also meet the same problem,
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dmlc/xgboost/issues/1063#issuecomment-203880518

--
Thusitha Chandrapala

Hi leoli2021,
I managed to get things running for V4.x.  Yei!!! With mingw64.
One problem was having the mingw path in "/Program files.../". Somehow spaces seems to mess up the make file.
So,

make sure all path names are without spaces
try renaming mingw32-make to make
try getting a fresh copy from git, and those submodule commands correctly. your initial error seems to be because omp.h is missing. I have that file in the correct location. So go check the folder to see if the file is there..

This approach work for me, I spent a lot of hours trying to compile, and this is working on latest xgboost on win7 and win10 (64bit!):
0] recommanding delete all previos instalation of python, mingw64, xgboost :), but you know, it is up to you
1] install git for win (with git bash)
2] install python (I used anaconda, 3.5.1, 64bit - recommanding)
3] as admin update anaconda:
$ conda update conda
$ conda update scikit-learn (not necessary, but I used it next to xgboost)
4] download this as only this version of mingw :) - this is crucial part..
http://sourceforge.net/projects/mingw-w64/files/Toolchains targetting Win64/Personal Builds/mingw-builds/4.9.1/threads-posix/seh/x86_64-4.9.1-release-posix-seh-rt_v3-rev2.7zmake

do not ask why, it just works :)
5] extract to: c://mingw64//
6] in c://mingw64//bin// locate "mingw32-make.exe" and rename to "make.exe"
7] add this "c://mingw64//bin//" to enviroment PATH
8] restart PC
9] open git bash as admin

10] and now xgboost:
$ git clone --recursive https://github.com/dmlc/xgboost
$ cd xgboost
$ git submodule init
$ git submodule update
$ cd dmlc-core
$ make -j4
$ cd ../rabit
$ make lib/librabit_empty.a -j4
$ cd ..
$ cp make/mingw64.mk config.mk
$ make -j4
11] for python module
cd python-package
python setup.py install
12] launch python and enjoy :)
import xgboost
..

hi,all, i try all the way as you gived, but fail for same error
dmlc-core/include/dmlc/omp.h:9:17: fatal error: omp.h: No such file or directory
however, this omp.h file is existed in fact, open the omp.h and in line7, #include <omp.h>, i do't know why omp.h will include omp.h in some condition...

This means your mingw do now have openmp support.., try to find one with openmp support

@gugatr0n1c  @thusithaC very thanks for your help~ i del  mingw64 from my machine path, and add a new mingw64 for this instruction~then, compile successfully
https://www.ibm.com/developerworks/community/blogs/jfp/entry/Installing_XGBoost_For_Anaconda_on_Windows?lang=en

I tried the methods mentioned above, but always got the following errors. Could anybody give me some help?
**

It's a problem with spaces in the path.  Put everything in paths without
spaces.
On Apr 3, 2016 2:07 PM, "derek1032"<EMAIL_ADDRESS>wrote:

I tried the methods mentioned above, but always got the following errors.
Could anybody give me some help?
**
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dmlc/xgboost/issues/1063#issuecomment-204881734

Thank you so much for your help!!!! I can use xgboost now! @thusithaC

I might sound dumb.
Why do we need to build the wrapper dll etc on our own? Can't we just share which we have already made?

MSVC is added in #1327

I have a ploblem with a command  make -j4-.  This is the same ploblem, like a derek1032.
When i input make-j4-, it writes: command not found. Help me, please. I can't install xgboost due to this ploblem.


feat: --network-dataplane flag

Fixes #
Description
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

18 of 18 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 97.775%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
36291

Relevant Lines:
37117

💛  - Coveralls


Feature Request: Treat models with same hash as the same when searching

I often rename models, which is why they are quite often repeated when searching
And in general, it would be convenient to search not by model names, but by hash

I don't have an elegant solution for this problem at the moment, but I have implemented advanced search. You can use it to temporarily work around the issue, like this.

see https://github.com/zanllp/sd-webui-infinite-image-browsing/issues/131
I will let you know once I have resolved the issue.


Make a way to record RUPs signed by AH on paper.

For all those still getting done on paper, and the few edge myra cases where we have no option but to mail it to AH, we need a way to capture these.
Down the line some day.

Manual Button was created but has not been made active as we need to have a gatekeeper of sorts.  Thinking that we will put this function in Admin and then determine who gets access to Admin.


Add documentation on how to run the test suite locally

Following the work done in #98 and the increased number of contributors, add documentation in the wiki on how to run the test suite locally. The aim is to allow contributors to run the test suite locally before submitting their pull requests thus speeding up the code review process.
The documentation should include the setup required for Windows and MacOS/Linux and the commands to run the tests. The current configuration for Travis and AppVeyor is a good starting point for writing instructions.

As I recently (re)set by Windows environment to run the test suite locally, I'd like to document here what I've done, hopefully it will be useful for other contributors. It would be good if someone on Windows could check that this is correct. @AirQuick: I am aware that #289 may change some of this but that's a start for some documentation on the wiki.
As I don't own a Windows machine, I dowloaded a Windows virtual machine from the Microsoft website and ran it in VirtualBox. These Windows VM are available for testing and are valid for 90 days. I'm running a Windows 7 32-bit system but I think this setup should be valid on Windows 8 and Windows 10 virtual machine as well as on bare metal.
This is the list of dependencies that need to be downloaded (for the current version used in XSpec, check the relevant version number in appveyor.yaml):

JAVA JDK 8
SAXON HE
Ant
XML Calabash
XML Resolver
Jing
BaseX (optional on Windows, only required on Linux/MacOS)

When available, it it recommended to download and extract the full zip files rather than just the .jar file as there are often additional dependencies.
Once all the tools are installed in their own directories, the corresponding environment variables need to be set. To set up system environment variables in Windows, click on the Start menu, search for system environment variables and click on Edit the system environment variables. The System Properties window opens up, select the tabAdvanced, and click on Environment Variables. Click on New and set up the environment variables as in the table below (I provide the values on my system as an example). Note that Path is likely to exist already on the system so select it, click on Edit, go to the end, add a ; to add a new value, and set it.

Variable name
Variable value

JAVA_HOME
C:\Program Files\Java\jdk1.8.0_181

SAXON_CP
C:\saxon\saxon9he.jar

ANT_HOME
C:\apache-ant-1.9.9

Path
%ANT_HOME%\bin;

XMLCALABASH_CP
C:\xmlcalabash\xmlcalabash-1.1.21-97.jar

XML_RESOLVER_CP
C:\xml-commons-resolver-1.2\xml-resolver-1.2.jar

JING_CP
C:\jing\jing.jar

BASEX_CP
C:\basex\BaseX.jar

You can check that the dependencies are installed correctly with the following commands from the Windows command line:
# check values of all environment variables
set
# check Java JDK
java -version
# check Saxon
java -cp %SAXON_CP% net.sf.saxon.Version
# check ant
ant -version
# check XML Calabash
java -cp %XMLCALABASH_CP% com.xmlcalabash.drivers.Main --version
# check XML Resolver
java -cp %XML_RESOLVER_CP% org.apache.xml.resolver.Version
# check Jing
java -jar %JING_CP%
# check BaseX
java -jar %BASEX_CP%

You should now be able to run the XSpec test suites on your machine. Go to the xspec directory on the branch you want to test, navigate to the directory test, and run the test suites :
xspec-bat.cmd
run-xspec-tests.cmd
cd end-to-end
run-e2e-test.cmd

Updated the Linux/MacOS section (diff). I guess this suffices for now.

@xspec/xspec
I restructured the Wiki. Now the page is not sectioned by OS but by the steps.


#20 - Restrict number of queries per minute(s)

Implement a configurable amount of queries a user can make in a configurable interval. (defaults to 3 queries per 1 minute). This branch is based on #21 - so better first merge the other one before considering merging this one.
Close #20

Hi,
simply returning a 429 status code is not enough, since a client does not know when it's OK to issue a new request that is about to succeed (regarding the rate limit).
That's the reason why something like this happens: medizininformatik-initiative/feasibility-gui#48 (implementation without retry functionality - the user won't know what to do).
Thus, sending an additional retry-after header back to the client helps to negotiate for retry intervals. This would also allow the GUI to display an appropriate message, a spinner or whatever.


spelling mistake corrected

csv has been written in place of cvs. corrected the spelling mistake.

Pull Request Test Coverage Report for Build 316

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 94.396%

Files with Coverage Reduction
New Missed Lines
%

autokeras/net_transformer.py
1
95.31%

Totals

Change from base Build 309:
-0.07%

Covered Lines:
1398

Relevant Lines:
1481

💛  - Coveralls

The classifier.md is automatically generated from the docstrings in the code.
You may modify the docstrings instead.
Thank you for the contribution.


Verify contracts on Etherscan

We're adding a step verifying the migrated contracts on Etherscan. We're
using etherscan-verify plugin. In order for the plugin not to pick up
wrong files for verification we have to remove contracts of the
dependent projects stored in the node_modules.
We're also adding a step which executes in case of failures during the
contracts verifications. The step adds a warning annotation to the
GitHub page with the workflow run details.
Some info regarding configuration of annotations.json and
github-action-create-annotations action:

Values of paths, start_line, end_line, start_column and
end_column don't seem to have much effect on how the annotation is
displayed for workflows triggered by the workflow_dispatch event
(they matter more for workflows triggered by pull_request, but we
only annotate for workflow_dispatch).
We were getting errors when end_line was configured to different
value than start_line.
The value of the check-name parameter of the action needs to be the
same as the name of the job we want to annotate. Otherwise the
annotation may get assigned to the random job (even in different
workflow).

Refs:
https://github.com/keep-network/keep-core/pull/2598
https://github.com/keep-network/keep-ecdsa/pull/913
https://github.com/keep-network/tbtc/pull/834
https://github.com/keep-network/tbtc-v2/pull/52

Workflow run proving successful contracts verification: https://github.com/keep-network/coverage-pools/runs/3633195118?check_suite_focus=true
Workflow run showing annotations added in case of verification failure: https://github.com/keep-network/tbtc-v2/actions/runs/1245161578 (in tbtc-v2 project, same implementation of the annotation action)

Initial implementation executed etherscan verification in the same job
as migration and utilized annotations to inform about failures in the
verification process.
We decided to move the etherscan verification to a separate job,
executed after the job which migrates contracts. This way the failure
of verification will not block anything and will be easily visible as a red
status dot.
To have the verification in a separate job, we need to pass the migrated
contracts between the jobs. I considered also passing all needed
node-modules subfolders, to get rid of the yarn install step, but
the number of needed modules was big, which would make maintaining
the workflow file troublesome.
I've tested the new solution on seperate branch, in following workflow:
https://github.com/keep-network/coverage-pools/runs/3729852018?check_suite_focus=true.


Parser bug when using "and" and "or", or equality operators in return statements with return types.

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Windows 10
Issue description
The exact error I receive is:
Parser bug (please report): Trying to check compatibility of unset value type
The error occurs whenever I have a func with a strict return type defined using ->, and when I use the and or or operators, or an equality operator (==, !=, etc.) in a return statement.
The parser seems to be choking on the operators themselves. The issue only seems to occur when only the first (left-hand) operand is present.
Steps to reproduce
Step-by-step guide

Write the header for a function
Give it any strict return type. bool is the most viable since the affected operators return bools, but you can use any return type. The bug occurs regardless of whether or not the code is valid.
In the function body, add a return statement.
Put any primitive or a variant name after return.
Add either the or or and operator.
Wait a moment for the parser to notice the operator. As soon as it does, the error message will appear in the output.
Hold down the control key. The error will be output repeatedly, as long as you hold the key, and as long as the script editor is focused (you may need to place your mouse cursor over some text for this to work).

Specific examples
Using the editor's default GDScript editor, type this:
func my_func() -> bool:
	return true or

Give the editor a moment to process after you type the "r" in or. You can substitute or with and for the same result.
Equality comparisons also cause the error
func foo() -> bool:
	return true !=

All of the affected operators, as I've been able to find so far, are:

and
or
==
!=
<=
>=

You can also substitute true for a variant or any other primitive, the result is the same.
func foo() -> int:
	return "something" and

func bar() -> RandomNumberGenerator:
	var a = "something"
	return a and

The following code does not cause the error to occur. It simply causes an "Expected end of statement after return" error in the editor, as expected.
func foo() -> bool:
	return and true

Typing anything after the operator does not cause the error. The following code is perfectly fine as it appears, but will cause the error as it is being typed-out.
func foo() -> bool:
	return true or false

Typing a whitespace character (space or indent) after the operator will not cause the error to appear immediately. It will cause the error if you hold down the control key, assuming you haven't typed anything else after the whitespace.
The operators is and as do not appear to cause this error. Most arithmetic operators (+-/*%) also seem fine, as do the non-equal inequality operators (< and >) I do not know about any other operators at this time.
Minimal reproduction project
func my_func() -> bool:
	return true or

Doesn't seem to happen in v4.2.dev6.official [57a6813bb]

Reproducible in master. You need to type something, but not save the script.


Add tests to make sure the only plausible reason a build may fail is due to user mistakes

As an extension of #19, tests can now be added using pytest that make sure a build only plausably fails due to missing information from a user/app-developer.

@CasperWA Is this issue resolved by merge #48 ?

@CasperWA Is this issue resolved by merge #48 ?

I guess. Looking over the touched files from a year and half back together with yours just now the tests cover very different grounds. One being for the build of the website and yours being for the actual changes a user may introduce in this repository.
Hence, since I don't really have the overview of the whole repository, I couldn't say whether these two test suites together encompass everything for this repository. But again, I would guess.

I honestly am not 100% sure how to interpret this issue and since you created it I thought you would know. It appears that there are tests for the website build and for the schema validation now. What else would you like to test?

My analysis of this would be the following then: If the tests for the schemas cover checking any changes to the JSON files performed by developers in future PRs, then we should have covered everything. Since the website tests will make sure the value of the passed URLs are valid.
I.e., if you are confident of the first, then feel free to close this issue on this basis, since the original issue will indeed have been fixed :smiley: :muscle:

I noticed that the apps_meta.json as it is generated right is not valid according to our schema. That is because apps_meta is in fact not just a combination of the apps.json and categories.json files, but includes additional fields. Since this went unnoticed I will add tests that will both cover the published data and schemas as well as the generate apps_meta.json file. Then we can close this issue.

I noticed that the apps_meta.json as it is generated right is not valid according to our schema. That is because apps_meta is in fact not just a combination of the apps.json and categories.json files, but includes additional fields. Since this went unnoticed I will add tests that will both cover the published data and schemas as well as the generate apps_meta.json file. Then we can close this issue.


compile

Recently I noticed that building myClock on Linux results more available heap than the same exact build on MacOS or WIndows. I'm still investigating, but the result is not displaying options on web page due to insufficient heap. We end up with 2492 more bytes available on Linux with same exact code and libraries.
Linux
Sketch uses 403424 bytes (38%) of program storage space. Maximum is 1044464 bytes.
Global variables use 62804 bytes (76%) of dynamic memory, leaving 19116 bytes for local variables. Maximum is 81920 bytes.

MacOS
Sketch uses 378036 bytes (36%) of program storage space. Maximum is 1044464 bytes.
Global variables use 65296 bytes (79%) of dynamic memory, leaving 16624 bytes for local variables. Maximum is 81920 bytes.

I'm going to look at further optimizing memory usage before 1.0 release.

I will keep the myClock.ino.bin in this repository updated and tested as necessary.

I think this is the issue.

That was not the issue (if you use my build.sh script it automatically disables Assert messages).
It was, in fact, the WiFiManager library. The default master branch is a memory hog, but the 'development' branch, that I was using on my Linux system, has many fixes and more efficient memory usage.
I updated the install and update scripts to automatically switch to development branch on WiFiManager. If you run 'update.sh' twice, once to update itself, and the second time it will switch the branch.


MINOR: Improve KRaftMigrationZkWriter test coverage

This patch adds complete test coverage for the snapshot methods in KRaftMigrationZkWriter.

I'm going to try something out for triggering builds...

retest this

retest this, please

Shall we close this since we are removing all the ZKMigration stuff 👀


test_gre_profile:test_vxlan_profile in test_network.py KeyError

Title: test_gre_profile:test_vxlan_profile in test_network.py:125 KeyError
Attachments:
Details:

    @log_test_call
    def test_vxlan_profile(setup):
        te = setup
        name = 'test_vxlan_profile'
        profile = network_helper.create_vxlan_multipoint_profile(te.mgmt_root, name,
                                                                 te.partition)
        add_resource_teardown(te.request, profile)
        params = {'params': {'$filter': 'partition eq %s' % te.partition}}
        vc = te.mgmt_root.tm.net.tunnels.vxlans
        profiles = vc.get_collection(requests_params=params)
        profile_names = (r.name for r in profiles)
        assert(name in profile_names)
        p = te.mgmt_root.tm.net.tunnels.vxlans.vxlan
        p.load(name=name, partition=te.partition)
        payload = NetworkHelper.vxlan_multipoint_profile_defaults
        for k in payload.keys():
            if k == 'partition':
                continue
>           assert(p.__dict__[k] == profile.__dict__[k])
E           KeyError: 'floodingType'

test_network.py:125: KeyError

Associated md5sums:

221a2db7c3577bceab793f30d3229632

Impacted tests:

openstack_agent_mitaka_12.1.1-overcloud.87.consoleText.test_vxlan_profile
openstack_agent_newton_12.1.1-overcloud.64.consoleText.test_vxlan_profile

OpenStack Release

mitaka
newton

Description

Details: For bugs, fill out the template below.

Agent Version

stable/newton - d884f8e70e84a07d5d65161df71c270cc847aefa
mitaka - 6d48f81c945da2e783f9c74a4c353dd52adee9b4

Operating System
Nightly

Additional sum: 221a2db7c3577bceab793f30d3229632

@dflanigan, this is an actual implementation error, not test error.  The scenario expectation has changed for a given set of input that is not time dependent here.
Neutronless tests (agent tests) are time irrelevant as they are in serial and has nothing that is in parallel due to the fact that the agent is only talking to the BIG-IP with no intermediate dependencies.

Additional sums from today:
md5sum:cc03f4d3440de6eb9ac082e3f2dde16f

openstack_agent_mitaka_12.1.2-overcloud.5.consoleText

Type: failures
Test: test_vxlan_profile

openstack_agent_newton_12.1.2-overcloud.5.consoleText

Type: failures
Test: test_vxlan_profile

openstack_agent_liberty_12.1.2-overcloud.5.consoleText

Type: failures
Test: test_vxlan_profile

Fixed by: 0cdebc5be80dc4ba8759672f6ee808123c590e5e


plan for npm publishing?

Now that this has been officially forked, is there a plan to publish updates on npm?  Is there a way to gain collaborator access on npm to publish under the same package name?

Agreed!

Right now it is looking like we cannot publish under the same name on NPM.  One option is to publish under the namespace @joltup/react-native-fetch-blob, another is to rename the project to something new.
Any ideas/opinions on which option would be better?

we are now published to npm as rn-fetch-blob, this should stay consistent with how the classes are defined throughout the code as RNFetchBlob


code review

chore: format files with gofumpt
chore: use "any"
fix: error returned by library should be exported
chore: remove unused receivers

let's merge them like this.


JSON field cannot be handled. It might be undefined or disabled by other fields.

Hey,
when trying to create a very simple client-ssl template we're receiving this issue. This is not a terraform provider issue, as it also happens when trying to interact with axapi. Anything changed here?
resource "thunder_slb_template_client_ssl" "client-test-cert" {
  name    = "client-test-tf"
  certificate_list {
    cert                = "Let_s_Encrypt.crt"
    chain_cert          = "Sub-CA-Lets_Encrypt_Authority_R10"
}
}

Error:
│ Error: axapi failure:JSON:err=Failed to handle json field "chain-cert". JSON field cannot be handled. It might be undefined or disabled by other fields.

aXAPI version: 3.0 on 5.2.1-P10,

Apparently key is now required, which is not in the docs.

https://github.com/a10networks/terraform-provider-thunder/blob/530353f5c2d58891cdbfe94880af4efff63596a4/thunder/resource_slb_template_client_ssl.go#L170C33-L170C48
should probably be Required: true now


Document technical specs of running app

I installed a few different versions of Node.js before finding a version that supported String.prototype.startsWith.  Having a tech spec doc would be helpful for working within this repo.

I am just spoiled by babel adding polyfills ;) . We should remove node 4 dependencies to make it more accessible.

Going to close this as we've not seen other problems here.


Feature Request: read files from an existing workspace and request AI to make changes

This will be very useful when we have made modifications to the workspace generated by AI.
We can then ask the AI to continue modifying the code, just like a PM.

I proposed it here: https://github.com/AntonOsika/gpt-engineer/issues/79
I think I can work on this, would be awesome

I proposed it here: #79
I think I can work on this, would be awesome

Thanks Pablo. Maybe we can do it together :)

I've created an adjacent issue about getting ChatGPT to finish code #119

Solved 🚀