First README example fails

When running the first README example on a fresh venv, the code fails in two ways.

First it complains that docs is undefined.
If I add something like docs = ["hi there", "bonjour", "buongiorno", "ola"] in the code, I get this. The ValueError makes me believes that it's an internal problem with the lib, not with the way I use it, but I'm not totally sure.

/Users/cpa/Developer/wordsimilarity/.venv/lib/python3.11/site-packages/wordllama/algorithms/kmeans.py:30: RuntimeWarning: invalid value encountered in divide
  probabilities /= probabilities.sum()
Traceback (most recent call last):
  File "/Users/cpa/Developer/wordsimilarity/test.py", line 27, in <module>
    wl.cluster(docs, k=5, max_iterations=100, tolerance=1e-4) # labels using kmeans/kmeans++ init
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/cpa/Developer/wordsimilarity/.venv/lib/python3.11/site-packages/wordllama/inference.py", line 337, in cluster
    cluster_labels, loss = kmeans_clustering(
                           ^^^^^^^^^^^^^^^^^^
  File "/Users/cpa/Developer/wordsimilarity/.venv/lib/python3.11/site-packages/wordllama/algorithms/kmeans.py", line 84, in kmeans_clustering
    distances = compute_distances(embeddings, centroids)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "wordllama/algorithms/kmeans_helpers.pyx", line 16, in wordllama.algorithms.kmeans_helpers.compute_distances
ValueError: Buffer dtype mismatch, expected 'const double' but got 'float'

BTW, awesome project, I always wanted to start a "semantic equality" lib using basically the same methods but never found the time.
For reference, the code snippet I'm talking about:
from wordllama import WordLlama

# Load the default WordLlama model
wl = WordLlama.load()

# Calculate similarity between two sentences
similarity_score = wl.similarity("i went to the car", "i went to the pawn shop")
print(similarity_score)  # Output: 0.06641249096796882

# Rank documents based on their similarity to a query
query = "i went to the car"
candidates = ["i went to the park", "i went to the shop", "i went to the truck", "i went to the vehicle"]
ranked_docs = wl.rank(query, candidates)
print(ranked_docs)
# Output:
# [
#   ('i went to the vehicle', 0.7441646856486314),
#   ('i went to the truck', 0.2832691551894259),
#   ('i went to the shop', 0.19732814982305436),
#   ('i went to the park', 0.15101404519322253)
# ]

# additional inference methods
wl.deduplicate(candidates, threshold=0.8) # fuzzy deduplication
wl.cluster(docs, k=5, max_iterations=100, tolerance=1e-4) # labels using kmeans/kmeans++ init
wl.filter(query, candidates, threshold=0.3) # filter candidates based on query
wl.topk(query, candidates, k=3) # return topk strings based on query

I tried to switch docs in the wl.cluster example to candidates and this did not work either. If I comment that out, all the other examples work.

Thanks for reporting! I'll check it out later today when I have some time to work on it, and update.

This is has been resolved by: https://github.com/dleemiller/WordLlama/pull/11
Thanks again for contributing an issue.


⚠ ClamAV has degraded performance

In f03137f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: ClamAV performance has improved in 29e78cf after 5 minutes.


Expose Tasks to REST

Is your feature request related to a problem? Please describe.
We have to run multiple gorse servers to have multi-tenancy (that we believe is not supported ATM). They run as pods in single kubernetes cluster.
Describe the solution you'd like
In order not to overload our server we would like to "manually coordinate - dispatch" tasks for multiple tenants (multiple Gorse servers) that we find in Tasks dashboard.  This also helps in case that we would like to delay Tasks to "night only" hours -to keep daily CPU for real endusers.
Another alternative could be maybe also, to use CRON expressions instead of "every 24 house". This way we could at least spread CPU intensitive tasks.
This coule be done as REST endpoint similar to existing REST endpoint.
-REST POST endpoint for starting specific task
-REST GET endpoint to receive current progress for specific task (we can use http://..../api/dashboard/tasks)

Good idea. In current design, we could expose an API to start all tasks manually. But it is hard to control individual tasks.

An admin API has been provided in #575, and the detailed documentation will come later.


Validator Confirmation Service Received Notification

Task

Add VALIDATOR_CONFIRMATION_SERVICE_NOTIFICATION support to the app
I included a screenshot of the notification mock below, but I couldn't find the exact text/styling for this specific notification. Check with @kristykjlee for that

WS endpoint:
ws://<IP_ADDRESS>/ws/validator_confirmation_services
{
  "notification_type": "VALIDATOR_CONFIRMATION_SERVICE_NOTIFICATION",
  "payload": {
    "bank_node_identifier": "b1b232503b3db3975524faf98674f22c83f4357c3d946431b8a8568715d7e1d9",
    "validator_confirmation_service": {
      "id": "6c5d1336-4454-454d-96f9-124a9a659cc9",
      "created_date": "2020-10-01T03:39:36.675680Z",
      "modified_date": "2020-10-01T03:39:36.675734Z",
      "end": "2021-06-03T22:15:09.343282Z",
      "start": "2021-06-01T22:15:09.343282Z",
      "validator": "e2a138b0-ebe9-47d2-a146-fb4d9d9ca378"
    }
  }
}

@kristykjlee do we have firma for this issue?

I would love to give this a shot? 😻

@mthomas-io is assigned this ticket and and will be taking charge of it. Don't forget to include your account number onto the ticket so you can get compensated for your efforts.

Okay, let me know if I can help with anything 😊

@mmmols message me on Slack.


Setting the S3 access policy to public

Could you please double check that the S3 access policy has been set up to public, since we are getting the issue like it has not been set up. @aaboyle878 @bosko-m @Ryun1

@Kale98 I have spoke with Sava and it should be sorted now, I had enabled server side encryption which has now been disabled

Thank you @aaboyle878. I will close this one.


Added resolution unit option in tiff method

Added resolution unit in option.param while using tiff method
Added required test cases

Sorry, I should have been clearer, I was thinking of a unit test that verifies the EXIF metadata of the output image (rather than just inspecting internal options), perhaps using an approach similar to the existing metadata tests:
https://github.com/lovell/sharp/blob/1ff84b20b7d9b351786fb06899255cea59aa123c/test/unit/metadata.js#L53-L56

@lovell  All the required change are done on our end. could you review once.

@lovell : Can you please review and approve this PR. We are waiting to use the resolution unit feature with our application. Thanks in advance !!!

@ompal-sisodiya Thanks for the updates. I don't see any verification of output image EXIF metadata in the newly-added tests; am I missing something? (I note you've added to an existing unit test, but that logic is unrelated to this PR, I was merely using it as an example of using exif-reader to parse the output.)

@lovell Thanks for your response. please review the below test for the verification of output image with EXIF metadata.
if this is fine then i will commit the code else you can help me with the  test.
it('TIFF setting resolution unit(cm)', async () => {
    await sharp(fixtures.inputJpgWithExif)
      .tiff({ resolutionUnit: 'inch' })
      .metadata(function (err, metadata) {
        if (err) throw err;
        // EXIF
        assert.strictEqual('object', typeof metadata.exif);
        assert.strictEqual(true, metadata.exif instanceof Buffer);
        const exif = exifReader(metadata.exif);
        assert.strictEqual('object', typeof exif);
        assert.strictEqual('object', typeof exif.image);
        assert.strictEqual('number', typeof exif.image.XResolution);
        assert.strictEqual('number', typeof exif.image.ResolutionUnit);
      });
  });

@ompal-sisodiya Please can you add an assertion on the value of exif.image.ResolutionUnit (as well as the type).

@lovell : Can you please review the PR?

@lovell  Could please review pr?

Thanks again for taking the time to submit a PR, I will review this when I am ready to do so. Coordinated pestering won't make that happen any sooner.

Landed via commit https://github.com/lovell/sharp/commit/f7bed69ffb5a533881faeb02163111deb4ff219a
I also added the (normalised) resolutionUnit property to the response from metadata(), which had the added advantage of making the tests a bit simpler.
This will be in v0.30.0, thanks again for the PR.


Add a workflow to build torchtitan-ubuntu-20.04-clang12 Docker image for CI

Adopt from PyTorch, this workflow will prepare the Docker image torchtitan-ubuntu-20.04-clang12 for the CI.

Base on https://hub.docker.com/layers/nvidia/cuda/12.1.0-cudnn8-runtime-ubuntu20.04/images/sha256-35d5a8eb50ad37fe707a7611a4e20414c5bd2f168adca0cf1700fe2d58411759 to include NVIDIA dependencies.
Install dev-requirements.txt and requirements.txt.  I need to move these files from the top level to .ci/docker directory and create softlinks for them because docker build process will only take a look at .ci/docker.  This is the reason why PyTorch keeps its CI requirements file there.
Install clang or gcc
Install conda (with python 3.11)

torchtitan-ubuntu-20.04-clang12 can then be used as the input for docker-image.

Looking at the CI results from this PR, it looks like it spent 6 minutes in 'calculate docker image' step.  If I look inside there it looks like it's building the image.  I guess that's automated such that it would build a new image automatically if it detected a change in the build scripts (e.g. if requirements.txt got updated), but then it hits the cache and skips the build time normally?

Re-running the job now to observe how it behaves on the second run.

on rerun, i do see the 'docker build' step go down from 6 min to 1 sec, so that's great!

but the docker pull step is taking over 3min, yesterday I was seeing 1m30s roughly, for the pytorch base image you had sent me.  I wonder if this is something related to docker cache- should we expect the pull step to decrease once the same runner is used a second time and its cache is warm?  if so then i think this will be alright.

Looking at the CI results from this PR, it looks like it spent 6 minutes in 'calculate docker image' step. If I look inside there it looks like it's building the image. I guess that's automated such that it would build a new image automatically if it detected a change in the build scripts (e.g. if requirements.txt got updated), but then it hits the cache and skips the build time normally?

Yup, you're right.  The step will build a new image if it needs.  Otherwise, it will download the matching existing images.  Let me retry the job now, and the cached image should be used instead.


[Requirement] In iFrame add any Website and type with non-Unicode characters with Web Font

If source is "self" (like in case of WhatsApp/Facebook) then not possible to access element.


request.is() not work with multipary/form-data boundary include slash

this is not working
multipart/form-data; boundary=b/QeEbFgqK9PCZo4T/eXv7f.T74SHd5MxCZ846AsTz-yNV0xrRR_Zks4fkNMCzJck9ZE8o

this ok
multipart/form-data; boundary=hQ9mmyU9285lg8QhbF7zuDmeuvc8flGel8sSgjHTBinO72dLucgB7oUOC3l6070sPpd1Es

solved!
https://github.com/jshttp/media-typer/issues/5


Fixing all traces of old "remote" against "zone"

This document has been very outdated referencing the word "remote" when we have renamed this to "zone" a while back. This includes a long list of variable names which made all copy&paste examples fail
Summary

Reason

Testing

@svenwal Do you know which version this change was made in? Then we can backport the changes to the relevant versions

Kuma 1.2 / Kong Mesh 1.3 is when we moved the naming from "Remote" to "Zone"


Document read_text / read_blob table functions

https://github.com/duckdb/duckdb/pull/10376

This was a duplicate of #2013


[CT-2702] [Bug] --selector intersection not working as indicated in docs

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
The docs for --selector mentioned here seem to indicate that we can intersect with selectors like below:
--selector shitty_selector_1,crappy_selector_2

But dbt seems to try to interpret this as one single ugly selector returning something like:
dbt run --selector shitty_selector_1,crappy_selector_2
07:32:41  Running with dbt=1.0.6
07:32:51  Found 873 models, 1061 tests, 0 snapshots, 4 analyses, 540 macros, 5 operations, 4 seed files, 265 sources, 0 exposures, 0 metrics
07:32:52  Encountered an error:
Runtime Error
  Could not find selector named shitty_selector_1,crappy_selector_2, expected one of ['x', 'y',... 'shitty_selector_1', 'crappy_selector_2']

Expected Behavior
I would expect it to appropriately intersect the selectors as indicated in the docs
Steps To Reproduce

Try to run variant of --selector shitty_selector_1,crappy_selector_2 in dbt-core == 1.0.6

Relevant log output
dbt run --selector shitty_selector_1,crappy_selector_2
07:32:41  Running with dbt=1.0.6
07:32:51  Found 873 models, 1061 tests, 0 snapshots, 4 analyses, 540 macros, 5 operations, 4 seed files, 265 sources, 0 exposures, 0 metrics
07:32:52  Encountered an error:
Runtime Error
  Could not find selector named shitty_selector_1,crappy_selector_2, expected one of ['x', 'y', 'z', 'shitty_selector_1', 'crappy_selector_2']

Environment
- OS: MacOS Ventura Version 13.2.1
- Python:3.9
- dbt:1.0.6

Which database adapter are you using with dbt?
snowflake
Additional Context
No response

Thanks you for raising this @nib-potato ! And thank you for a link to the docs and examples where it wasn't working as expected.
dbt does not support the intersection or union set operators for selectors with the --selector flag, so I'm going to close this in favor of fixing the docs in https://github.com/dbt-labs/docs.getdbt.com/pull/3535.


bagit/storage ingest: MMO ingest leads to overwritten bags

ingesting a new MMO, i.e. updating a bag currently leads to new versions of the bag which do not contain the other manifestations. (19_century on winslow)

https://api-stage.wellcomecollection.org/storage/v1/bags/digitised/b30499501


treat benchmarks as normal tests

Signed-off-by: tabokie<EMAIL_ADDRESS>Last I checked this behavior is consistent with cargo test. Bench cases that appear in test binary list should be considered a normal test.

Thanks for the PR!
What happens if a benchmark is invoked? Is it run just once or many times?
Note that the most popular benchmarking library criterion.rs doesn't currently work with nextest: https://github.com/bheisler/criterion.rs/issues/562

@sunshowers In the case of cargo test, all #[bench] benches will behave just like normal unit tests (run just once). They will be run even if --tests is passed. Cases managed by criterion.rs will not be run, because they are not contained in the test binary generated by cargo test.

Thanks -- in that case this makes sense. I'll add a note to the release notes explaining that criterion still won't work but at least it won't break due to this.

This is now out as cargo-nextest 0.9.22 (binaries will be out shortly).


[emmet] http-equiv="X-UA-Compatible" in 2019

html:5 macro still includes <meta http-equiv="X-UA-Compatible" content="ie=edge">
According to all the sources, it should be definitely unneeded nowadays.
This has been previously reported in #58796, where issue was locked and limited to collaborators. It was marked as upstream and suggested opening an issue at emmetio/expand-abbreviation (now archived). However https://github.com/emmetio/expand-abbreviation/issues/28 shows that is has already been disabled there.
Why is it still kept in VSCode?

I share the same concern as the OP.
Until the fix (optimistically speaking) is implemented, is there a way to access/update these expansions ourselves?

I believe this meta tag already disabled

Yes, looks like that's true, just checked on a recent version. I'm closing the issue then.


go build does not work

# github.com/felixangell/phi-editor/gui
gui/buffer.go:669:8: undefined: strife.MouseWheelEvent
gui/buffer.go:670:11: event.Y undefined (type strife.StrifeEvent has no field or method Y)
gui/buffer.go:673:11: event.Y undefined (type strife.StrifeEvent has no field or method Y)

Ah my bad, I made some changes to the graphics library. If you try again it should work now @firstrow

Thanks. It also would be nice to have vendoring support.

phi-editor - master ❯ go get -u github.com/felixangell/phi-editor
# github.com/felixangell/phi-editor/gui
gui/buffer.go:85:6: undefined: f
gui/buffer.go:88:4: undefined: f

@firstrow Ah that's a bug from the PR, just fixed it. You might have to merge with master though again

w/r/t vendor I'll look into that, I've never used it before. I presume you mean use that with the strife repository?
Maybe if the editor has a stable release, but I like the fact that everything is using the latest repo so everything kind of has to work type thing.

You know what is weird? Yesterday I was thinking about text editor using GPU to render all the things. And today I've found your lib in go trending. Recently I've found atom/xray, but I do not like the idea of using electron(js/css) for building fast gui applications.

@firstrow hehe, that's a cool coincidence. if i'm honest the only reason I made it is because I love the block cursors and no editors I like (mostly sublime) seem to have them natively supported.

emacs has block cursors.

@firstrow ah yeah that was my goto but I'm not keen on the mac layout personally. and also i can never get the tabs configured correctly - which is another reason I wrote this editor 😄

@firstrow have you managed to get this to build? can i close the issue?

thanks. I got it up and running!

@firstrow Great! thanks for reporting this :)


add test for assert sample and rtype

This PR attempts to add tests to _assert_sample_and_rtype function defined as
def _assert_sample_and_rtype(
    rp: "RandomVariable", skip_if_none: bool = True
) -> None:

Tests included

return None if (rp is None) and (skip_if_none)
raises error if (rp is None) and (not skip_if_none)
check return type is a tuple and is annotated as such
raises error when return type annotation is missing in sample

Would appreciate feedback on further tests to include.
Related #340

This is a helper function, and I'm unsure how much it is used across the package. Should this be somehow enforced in every model/RV that received RVs?

This is a helper function, and I'm unsure how much it is used across the package.

It seems the usage is split between using _assert_sample_and_rtype and asssert isinstance(some_rv, RandomVariable) . I think it would be good idea to replace assert isinstance(some_rv, RandomVariable) with _assert_sample_and_rtype to check for tuple return inaddition to RV.

Should this be somehow enforced in every model/RV that received RVs?

Deferring the more general question of enforcing this in every model/RV to @damonbayer @dylanhmorris

@sbidari Agree on all points.

Can we keep assert isinstance(some_rv, RandomVariable) but check for the proper return type whenever a  RandomVariable is instantiated?

Right now, we are using assert isinstance(some_rv, RandomVariable) in some codes (eg: in hospitaladmissions.py) and _assert_sample_and_rtype(some_rv) in others (eg: in InfectionsWithFeedback ) as validation when a RandomVariable is instantiated.
We should probably use one of these throughout for uniformity. _assert_sample_and_rtype(some_rv) tests if the input_rv is a RandomVariable and the return type

@sbidari I am suggesting that we separate "checking that something is a random variable" from "checking that the random variable is valid."

@sbidari I am suggesting that we separate "checking that something is a random variable" from "checking that the random variable is valid."

ok! I will create a new issue for extending the use of _assert_sample_and_rtype but not remove assert isinstance(some_rv, RandomVariable)
Marking the tests as ready for review.

@damonbayer I tried following the pattern in other tests and do not feel strongly about the match strings. I think it's probably better not importing additional packages unless necessary, will change to remove importing re.


Create 615 Glenwood Ave Owosso

We are here to give you the best any kind of building any kind of design any kind of homework

Kindly send your message for the best designing you want in life but the signing you want in the whole world we are here to give you the best


Add DocuSign Click connector

Description
Generate the DocuSign Click connector using the official OpenAPI specification.
https://developers.docusign.com/docs/click-api/reference/
Fixes: https://github.com/ballerina-platform/ballerina-extended-library/issues/248
One line release note:

Generate DocuSign Click connector using Open API specification.

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Not tested yet
Test Configuration:

Ballerina Version: SL Beta 6
Operating System: Ubuntu 20.04
Java SDK: 11

Checklist:
Changes to OpenAPI definition

[x] Modified info section of the OpenAPI definition file - Example

description - what the connector is about
x-ballerina-init-description OpenAPI extension - connector initialization details
x-ballerina-display OpenAPI extension - connector name and path to the connector icon

Changes to connector module

[x] Added license header at the top of each .bal file.
[x] Added Package.md as per the guide here
[x] Added Module.md for the module of the  connector as per the guide here
[ ] Added connector icon to the root of the connector. Icon needs to be a png of 200x200 px size with name icon.png
[x] Added Ballerina.toml file with following information. For keywords refer to guide

Verifying connector module

[x] Compiled the connector successfully with the targeted ballerina version.
[ ] Conducted smoke tests on the connector (Recommended when OpenAPI definition is obtain from an unofficial source)

Security checks

[x] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines?
[x] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?

@sachinira CI build fails due to old versions of dependencies. Could you regenerate dependencies using https://github.com/ballerina-platform/ballerina-distribution/suites/4918117688/artifacts/141807451?


join default says butt, appears to be mitre - verify

This is an error - default is miter (not mitre), there is no such thing as a butt join.

fixed with dd54d1e


could not parse config file "./cayley.yml": invalid character 's' looking for beginning of value

Description

Steps to reproduce the issue:

~/downloads/cayley_v0.6.0_linux_amd64$ ./cayley init --config ./cayley.yml
cat ./cayley.yml

store:
  # backend to use
  backend: sql
  # address or path for the database
  address: "mysql://root:gld@db:3306/cayley?sslmode=disable"
  # open database in read-only mode
  read_only: false
  # backend-specific options
  options:
    nosync: false

Received results:

F0418 12:49:01.600949   20236 cayley.go:119] could not parse config file "./cayley.yml": invalid character 's' looking for beginning of value
goroutine 1 [running]:
github.com/cayleygraph/cayley/vendor/github.com/golang/glog.stacks(0xf12500, 0xc400000000, 0x8e, 0xad)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/vendor/github.com/golang/glog/glog.go:769 +0xa5
github.com/cayleygraph/cayley/vendor/github.com/golang/glog.(*loggingT).output(0xef5440, 0xc400000003, 0xc420090180, 0xec1dd1, 0x9, 0x77, 0x0)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/vendor/github.com/golang/glog/glog.go:720 +0x357
github.com/cayleygraph/cayley/vendor/github.com/golang/glog.(*loggingT).printDepth(0xef5440, 0xc400000003, 0x3, 0xc4200efb88, 0x1, 0x1)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/vendor/github.com/golang/glog/glog.go:646 +0x126
github.com/cayleygraph/cayley/vendor/github.com/golang/glog.FatalDepth(0x3, 0xc4200efb88, 0x1, 0x1)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/vendor/github.com/golang/glog/glog.go:1134 +0x5d
github.com/cayleygraph/cayley/clog/glog.Logger.Fatalf(0xb4163b, 0x2, 0xc420186b40, 0x1, 0x1)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/clog/glog/glog.go:25 +0xe3
github.com/cayleygraph/cayley/clog/glog.(*Logger).Fatalf(0xf12790, 0xb4163b, 0x2, 0xc420186b40, 0x1, 0x1)
        <autogenerated>:4 +0x7e
github.com/cayleygraph/cayley/clog.Fatalf(0xb4163b, 0x2, 0xc420186b40, 0x1, 0x1)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/clog/clog.go:65 +0x76
main.configFrom(0x7ffe4a9df566, 0xc, 0x0)
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/cmd/cayley/cayley.go:119 +0x301
main.main()
        /home/barak/src/cayley/src/github.com/cayleygraph/cayley/cmd/cayley/cayley.go:182 +0x207

Expected results:
Output of cayley version or commit hash:
Cayley v0.6.0 built 2016-09-27T14:21:17-07:00

Environment details:
Backend database: (database and version)

What version of Cayley are you using? If it's v0.6, as directory name suggests, you should use docs from v0.6.0 branch. Only v0.7 (master) branch supports yaml configs.

json woks on 0.6.0


Notifications on new emails reappear after removal (notification bar)

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.202
Where did you get the app from?
Google Play
Android version
10
Device model
Samsung S9
Steps to reproduce
Notification on new email(s) appears in the notification bar.
I open the notification bar and see details on all new emails reported by K9.
When I delete the notification (either by swiping away or by pressing the x) it returns as new notification after 1 or 2 seconds.
When I delete the notification a second time it stays gone for good. Although, in some (?) cases the notification will pop up again together with another notification on newer emails at a later time point.
Expected behavior
Dear Sir or Madam,
I am writing at your advice from the K9-Forum. Link to the thread:
https://forum.k9mail.app/t/notifications-on-new-emails-reappear-after-removal/5451
In short: Every email notification appearing in the status bar reappears a second time after I have removed it (mostly swiped away but not only). After its reappearance I usually remove them by swiping them away a second time. And after that they stay gone.
This applies to all my 6 different email accounts I use in K9. Some of them sync via POP3 and some via IMAP.
Expected behavior should be: deleted notifications should stay deleted after their first removal.
Can you help me? I am adding the log file.
Thank you!
Yukusoona
Actual behavior
In short: Every email notification appearing in the status bar reappears a second time after I have removed it (mostly swiped away but not only). After its reappearance I usually remove them by swiping them away a second time. And after that they stay gone.
This applies to all my 6 different email accounts I use in K9. Some of them sync via POP3 and some via IMAP.
Logs
221001-k9mail-logs.txt

same issue here with Samsung Galaxy A51

K-9 Mail 6.202 doesn't contain enough logging to track down notification issues like this. Please install K-9 Mail 6.311 or newer and record a debug log when this is happening. See https://github.com/thundernest/k-9/wiki/LoggingErrors
Please include a description of what you saw when you recorded the log. Example: received three new messages, swiped away the notification for the first message (the notification disappeared), tapped the second notification to open the message (the notification disappeared), went back to the message list inside the app and the two notifications reappeared.

k9mail-logs.txt
Here's a log, I left it running until the error occurred, so the interesting bit will be towards the end (around 2130). I was in some other app and a mail notification appeared; I swiped it away, then the same notification reappeared. I swiped that away too and it didn't come back. Then I went into K-9 and saved the log. IIRC there were other notifications earlier in the day that didn't reappear.
I've redacted the log some, but I have the original in case I redacted too much!

The problem seems to be triggered by your Android version aggressively killing K-9 Mail's process.
When you dismiss a notification, the OS has to (re)start the app to notify it about this event. However, K-9 Mail also restores notifications when the app process is started. It looks like the dismiss event arrives before the app has read the information about notifications from its database. So the dismiss event is ignored because at that point the corresponding message can't be found. Afterwards the app loads information about notifications from the database and creates/restores system notifications based on that data.
We'll have to change the code to make sure the information about active notifications is always loaded before the app handles the dismiss event.
In the meantime you could try to exempt K-9 Mail from being killed all the time. See https://dontkillmyapp.com/

Anyway, I'm using 6.400 now, and I've turned off as much of the power management as I could find (it's in at least two places) but the issue is still there—should it have gone away?

No. The issue is still open. The bug hasn't been fixed yet.

Thank you very much for looking into this problem. If you see a chance to fix this problem and there is a prospect of fixation in the future, I for myself am going to "endure" this behaviour and eagerly wait for a future update with a fix.
Thanks a lot!


🏷 Add tags to project frontmatter

@agoose77 @jmunroe as we were talking through this today in the pythia meetup, this brings tags up to the top level to support some of the work in Pythia, especially around galleries. This is necessary to support some of the work in collecting centralized resources.
Previously we could add tags to pages, but not projects. This opens up the possibility to tag a project and is more in line with how pythia, scipy-proceedings, and curvenote are dealing with projects that have this metadata at the project level.

Yes, tags should be similar to how title works (they exist on both project and page, but are not inherited).
Will document keywords, and add descriptions.


Problems with cxf-client and adyen wsdl: Could not create web service client for interface

I'm developing a grails plugin for payment with adyen.com (called: "adyenplugin").
This grails plugin has some services i use in my mainproject (called: "mainproject"). My grails version is 2.2.0 and cxf-client 1.5.3.
I created a binary plugin of the adyenplugin and placed the library (jar) into the mainprojects lib folder. I also added the cxf-client config i also have in adyenplugin to the mainprojects config. The generated java files from wsdl2java (generated via cxf-client plugin) are placed in the adyenplugin (also packed in the jar).
After starting the mainproject with "run-app" sometimes everything works good (also the payment part where the cxf-client is used) but often the app crashes and doesn't start. It's not only on my computer, it's on all the computers i've tested. Then i do some clean, and restart the app. Often it works then, but very often it doesn't work again.
This is the exception i get after starting the mainproject with "run-app":
https://gist.github.com/PatrickHuetter/74dae3c2b92d219b62d5#file-gistfile1-sh
I also tested to copy the generated java files to the src/java folder of my mainproject, but same problem here.
My config (username and password edited):
cxf {
    client {
        recurringClient {
            wsdl = "https://pal-test.adyen.com/pal/Recurring.wsdl"
            wsdlArgs = [
                '-autoNameResolution',
                '-frontend',
                'jaxws21'
            ]
            bindingFile = "src/wsdl/recurringBindings.xml"
            clientInterface = "com.adyen.services.recurring.RecurringPortType"
            requestContext = [
                (javax.xml.ws.BindingProvider.USERNAME_PROPERTY<EMAIL_ADDRESS>                (javax.xml.ws.BindingProvider.PASSWORD_PROPERTY): "mpasswd"]
            serviceEndpointAddress = "https://pal-test.adyen.com/pal/servlet/soap/Recurring"
        }

        paymentClient {
            wsdl = "https://pal-test.adyen.com/pal/Payment.wsdl"
            wsdlArgs = [
                '-autoNameResolution',
                '-frontend',
                'jaxws21'
            ]
            bindingFile = "src/wsdl/recurringBindings.xml"
            clientInterface = "com.adyen.services.payment.PaymentPortType"
            requestContext = [
                (javax.xml.ws.BindingProvider.USERNAME_PROPERTY<EMAIL_ADDRESS>                (javax.xml.ws.BindingProvider.PASSWORD_PROPERTY): "mpasswd"]
            serviceEndpointAddress = "https://pal-test.adyen.com/pal/servlet/soap/Payment"
        }
    }

recurringBindings.xml:
<jaxb:bindings version="2.1"
xmlns:jaxb="http://java.sun.com/xml/ns/jaxb"
xmlns:xjc="http://java.sun.com/xml/ns/jaxb/xjc"
xmlns:xs="http://www.w3.org/2001/XMLSchema">
   <jaxb:globalBindings generateElementProperty="false"/>
</jaxb:bindings>

Do you know any solution for that? I can't work really efficiently if the application doesn't start always.

I have this issue (still stuck on a very old version of Grails - 2.2.1). From debugging the code, I believe the issue is that the cls.cast fails because the cls comes from a different class loader to Thread.currentThread().getContextClassLoader(). Probably the code in the catch block that uses cls.getClassLoader() would work ok, but unfortunately the exception is only thrown after that on the cast line.
Oddly, it does seem to work sometimes, if you clean and rebuild a few times. I don't know if there's a weird race condition of some kind that causes issues with the class loaders.


Ways of loading exchanges past data.

Hello,
Example task: load all last 5 minutes trades for poloniex exchange.
Question: Poloniex has about 40 currency pairs. So, if i want to download trades for all pairs, than i should create 40 HTTP requests? And same for others exchanges? Or, may be i miss something? Is there more efficient way to grab past markets data?
Thanks!

If they don't have an API for that, then no, there is no easier way. You'll have to look up their API docs and if a batch download is available, it can be implemented in XChange.

@timmolter thanks for answer, i close the issue.


Backported various performance-related options to SMSTCPIP and other fixes

Pull Request checklist
Note: these are not necessarily requirements

[ ] I reformatted the code with clang-format
[x] I checked to make sure my submission worked
[x] I am the author of submission or have permission from the original author
[ ] Requires update of the PS2SDK
[ ] Requires update of the gsKit
[ ] Others (please specify below)

Pull Request description

Special thanks to @rickgaiser and ps2netbox for some ideas.


Warn older Chrome users to upgrade to use WebAuthn

@webknjaz reported:
@brainwane I've tried that on Test PyPI.
So I have a TOTP set up. I clicked on Add 2FA with security key.
It prompted me to enter a Key name which I did (Yubikey Neo).
STR
After that, clicking Provision key does nothing visually. So I've opened DevTools.
I can see a successful GET request to https://test.pypi.org/manage/account/webauthn-provision/options with some JSON payload in the response. It looks legit, contains my user data and a challenge.
After clicking more times on that button, each of them produces an exception being logged to the JS console.
The same happenes on prod PyPI, in incognito mode, with browser extensions disabled.

  Would it be possible to try with a more recent release of Chrome @webknjaz?

It'll take some time for me to get it, but from what I see it should be a bug on JS side unrelated to a browser.

[2] bind-modal-keys.js:43 Uncaught (in promise) DOMException: The operation either timed out or was not allowed. See: https://w3c.github.io/webauthn/#sec-assertion-privacy.

specifically this log line makes it appear that it's not an issue with JS served by warehouse, but rather on the browser side, as the attestation never completed.
WebAuthn requires browser implementations to "play nicely", and in this case it appears the browser did not ever respond.
Was there a pop over requesting access to the token?

Right. No pop-up. I've just tried on a PixelBook (Chrome 75, ChromeOS==fancy Gentoo) and a pop-up appears there.
Interestingly, I'm able to use my Yubikey with GitHub which probably means that they work around the standard...
/me navigates to the Google Chrome changelog for more details

This suggests that they've added some webauthn support in Chrome 67: https://developers.google.com/web/updates/2018/05/webauthn

So it seems like the better support for Webauthn was introduced around January. It probably makes sense to put a big red warning next to the 2FA setting if the browser is old...

So what's the minimum Chrome version that properly supports WebAuthn, including PublicKeyCredential and any other components we need? Clearly Version 69 didn't, even though there was some WebAuthn support as of Version 67.

https://webauthn.guide/ states this:

As of January 2019, WebAuthn is supported on Chrome, Firefox, and Edge, with upcoming support on Safari.

So probably matching release dates should give proper versions.

There was a stable release of Chrome 72 in Jan 2019. If some wants to, they can investigate from Google's Chrome Releases blog:
https://chromereleases.googleblog.com/2019/01/

I think this may hit non-Chrome users as well.

They might have changed it since I last checked, but I believe GitHub doesn't actually use WebAuthn for security keys yet -- I believe they're still on U2F via u2f.js. That would explain why they aren't affected.
Chrome 74 on macOS works for me, as does FF latest on macOS. At the risk of speculating idly, maybe it's a CTAP issue with Linux + Chrome? I vaguely remember having to add udev rules when I added my security keys on Linux, so maybe there's other mis-interacting components there. I'll do some testing on my Linux machines this evening.

Nah, that's def on the browser side ;)

Google Accounts uses webauthn for login, but not registration.

Just confirmed that it works on Chrom(e|ium) 75 (Ubuntu 18.04).
Chrome 67 was released in May 2018 and WebAuthn Level 1 wasn't formally released until March of this year, so it's entirely possible that we're seeing either the effects of a spec change or just a buggy early implementation. Perusing through the chromium issue tracker, I think the latter is a safe bet.
Either way, it'd be interesting to see what exactly is failing. @webknjaz, would you be able to run a local deployment and insert some catches + logging?

Do you mean that I'd need to run it with a front-end build in dev mode with src-maps?
I don't have warehouse locally, but if needed I can find some time and set up the dev env. It may be faster for you to run some Ubuntu VM from vagrant locally with an old Chrome installed there...

Hey, sorry for the late response. You shouldn't need the source maps, it should be sufficient to just pepper some .catch(...) calls on the promises and see what the actual exception is.

Can I just stick some breakpoints in DevTools then?

Yep, that should also work.

Ah, yes. Nothing should be different in prod.

That's better! Because I cannot find time to spin up everything locally right now :)
I thought that maybe it's minimized there.

Hm... It's minified and my browser doesn't pick up srcmaps for some reason.

This should be resolved, there was a misconfiguration on test.pypi.org that was not present on pypi.org.
Can you attempt to reproduce and update us if it is still not working?

Hm... Setting breakpoints in https://github.com/pypa/warehouse/blob/0d9c726/warehouse/static/js/warehouse/utils/bind-modal-keys.js#L43 doesn't work for me.

I was able to set some breakpoints but no success figuring out where the exception happens. Only that it's in some promise...

So I set up a local instance. I commented out the whole bind-modal-keys.js but the exception logged in the console still points to that file, to one of the commented lines.
I think that maybe srcmaps aren't generated properly...

I have a feeling that some promise hasn't been awaited.

I believe that one of the problems is that func(csrfToken) @  https://github.com/pypa/warehouse/blob/0d9c726153394cf6aca178c9015c467707fbaa2e/warehouse/static/js/warehouse/utils/webauthn.js#L47 isn't awaited. But there's more.

After doing some debugging with @webknjaz, I noticed that I can reproduce this exception in modern Chrome as well by simply clicking "Provision key" and then canceling the in-browser modal that appears, so it seems like there's two issues here:

we aren't correctly detecting incompatible versions of Chrome
we have this exception happening in all versions of Chrome when the modal fails or is cancelled

After further debugging, I've found out that if (!window.PublicKeyCredential) { feature detection check is invalid. This function is present in my version of Chrome, that's why I don't get any error messages displayed (the DOM node stays hidden).
https://github.com/pypa/warehouse/blob/1d8e0fa/warehouse/static/js/warehouse/utils/webauthn.js#L163

So I've tried demo.yubico.com/u2f and demo.yubico.com/webauthn .
The first one works and the second works partially. There's an "Internal authenticator" test which doesn't work.

pics

So does Warehouse only implement this internal authenticator type support?

I've noticed there's u2f.getApiVersion() method available. Here's the version in my browser:
> u2f.getApiVersion(console.log)
{js_api_version: 1.1}

If modern browsers have a different one — maybe that's it?

Also, this article https://www.ubisecure.com/api/fido-webauthn-api/ suggests using PublicKeyCredential.isUserVerifyingPlatformAuthenticatorAvailable().then() for the feature discovery:

The isUserVerifyingPlatformAuthententicator method can be used to detect if a platform authenticator that implements user verification exists on the user’s device. This allows the application to improve user experience by only displaying options in the user interface that are sure to exist.

Here's the output for me:
> PublicKeyCredential.isUserVerifyingPlatformAuthenticatorAvailable().then(console.log)
Promise {<pending>}
false

I probably nailed it. Needs testing tho. On the platforms, I don't have. #6264.

I probably nailed it.

Wrong guess...

@webknjaz Can you attempt to repro on master (which should also be live)? #6305 may have fixed this.

@woodruffw So I've tried it out locally. Nothing happens on the first click. But when I click [ Set up security device ] for the second time, A request is already pending. error message appears. When I click more, every subsequent click adds another instance of that error message.

Contractors on the OTF-funded work need to stop work on the security features in order to ensure we complete the accessibility and internationalization work by the end of the month. Therefore, while this is necessary to get us out of beta for this feature https://github.com/pypa/warehouse/issues/5661#issuecomment-515802931 , I'm removing it from the milestone.

@woodruffw did you mean to also close this?

Yes! Thanks @webknjaz.


create my imagine map

Hey I want to create my world map in leaflet. There is no real borders or countries. I want to create my counties. Actually my imagine world. Can ı do. Can ı change borders or something in leaflet. Actually I am asking for is it sensible using leaflet for this work. Thanks for answer :)

Hi, great to hear that you find Leaflet useful!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet, please use gis.stackexchange.com or stackoverflow.
If you are really sure that this is a bug in leaflet, or a feature request, please change the wording to make it look like a bug report.

Do have a look at https://www.opengeofiction.net/#map=6/40.007/-2.488 and https://leafletjs.com/examples/crs-simple/crs-simple.html , though.


added formatted error messages to exeption

Hey there,
I added extra info to the error thrown when failOnError or failOnWarning are on.
Let me know if that works for you.
Thanks

Emmitter don't do its job before?

The emitter doesn't get the chance to print the emitted error because the error in this loader is thrown before webpack prints any errors.
The use case here is for production builds that need to fail on error.  Unfortunately, that only prevents the bundle from being created, but it doesn't throw an error (it only logs the error to the console), therefore my build script thinks that the bundle it was successfully created.
Turning failOnError works, throws the error and stops the build script, but the execution of Webpack is stopped too soon and the error is just Module failed because of a eslint error without extra info.

Thanks for your explanation. Make sense. Releasing as a patch :)


Update http4s-blaze-server, http4s-circe, ... to 0.21.28

Updates

org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.1 to 0.21.28.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

dependency:v0_21@org.http4s:http4s-scalafix:0.21.5

Documentation:

https://github.com/http4s/http4s/releases/tag/v0.21.5

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch, scalafix-migrations

Superseded by #332.


Readme fixes

Just a few tweaks to the WordPress.org README.

LG. Making an exception to the one commit rule at my discretion. Thanks for this!


Remove notional-finance subgraph monitor alert

@dhruv-chauhan Do you mind blacklisting this error for notional finance for now? It is getting a little noisy and we are aware of it. Thanks!
https://discord.com/channels/953684103012683796/1046922440846098492/1123297452498370703
Protocol Errors on notional-finance
ethereum: https://subgraphs.messari.io/subgraph?endpoint=messari/notional-finance-ethereum&tab=protocol
Field: cumulativeTotalRevenueUSD
Value: $1,738,706.84 != $0.00 + $1,390,965.47
Description: lendingProtocol entity has a cumulativeTotalRevenueUSD that is not within $1000 of the value of cumulativeProtocolSideRevenueUSD + cumulativeSupplySideRevenueUSD

@dhruv-chauhan sorry nvm, going to just mark it as dev instead!


Update Azure AD references to Microsoft Entra ID under Microsoft login

Is your suggestion a test requirement/change or improvement of the test framework? Please describe.
Azure Active Directory is now Microsoft Entra ID. https://www.microsoft.com/en-us/security/business/identity-access/microsoft-entra-id
There are product UIs, URLs and documentation that still refer to Azure AD.
https://wso2.com/asgardeo/docs/guides/authentication/social-login/add-microsoft-login/
https://is.docs.wso2.com/en/next/guides/authentication/social-login/add-microsoft-login/

Describe the solution you would prefer
Update the UIs, URLs and Product documentation and screenshots to update the necessary references to Entra ID.
Optional Fields
Additional context

Related Issues:

UI and URL changes associated with this issue are done. A new issue has been created to track the documentation changes related to this. Please refer to https://github.com/wso2/product-is/issues/19329 for details.


Deprecate the module

[ ] Deprecate on NPM to link to https://content.nuxtjs.org
[ ] Add note on README
[ ] Clean issues
[ ] Deprecate and Archive https://github.com/nuxt-community/nuxtent-template
[ ] Archive the repository

@alidlorenzo can you confirm me that I can do this?

Sure; great work on nuxt/content, btw

Thank you @alidlorenzo!


Bake node-adjusted textures?

I like to work with simple black-white pattern textures, which I can apply to roughness/specular/transmission or even base color by using Blender's nodes. This gives me flexibility and very fast iteration times with lookdev by editing node parameters rather than phaffing around in something like photoshop.
When exporting to GLTF it's frustrating that the original texture is exported, rather than the modified texture which Eevee sees.
Is it possible to query blender for a "baked" version of this texture and include that instead? The particular case that's got me today is a simple color Invert on a B&W Non-Color texture to adjust transmission amount.

In this example I'm able to use the same texture with different settings - saving me tonnes of time. I had expected the GLTF to end up with two textures - one of which had been fed through the Invert node.
Apologies if this has already been requested.

Closing this ticket, as there is no plan to implement bake into this addon.
This is a huge task, that should be done on Blender side, or by another addon


(Do Not Merge) test e2e setup

/hold

/ok-to-test

/ok-to-test

/hold cancel

/assign @ddutta

A followup PR is to be created to add detailed testing functionalities.

/lgtm

/approve

/approve


feat: Avoid crossing label lines + Prioritize longests texts

Set correct type hint for ta.allocate (using Optional would be better, but at least i set the correct tuple size :wink:)
Avoid label lines to cross
Change text allocation order by starting with largest text boxes

Before

After

Very nice, thanks!

Good catch, and I agree Optional would be better, but only if applied on all type hints 😉
Great addition!
I think I disagree with adding this one. By default the texts are prioritized in the order they are passed to allocate(), so you can order them by longest, shortest or any other prio you want, so this option feels redundant, no?

Merged and uploaded v1.1.3 to pip


[FEATURE] and  incompatible with Flutter_html package

Hi guys,
You made a beautiful package, I love this so much cause I can edit my html by flutter on my devices.
But I got a issue recently, when I export my html and read it by another package (flutter_html), the font element doesn't support cause they just support html5 . I have to change it to span element with style color and font-size to work normally.
I don't know if you have a function to export to html5 to support this case.

Hi, thanks for your kind words, glad you like the package!
Unfortunately I can't change it to html5, since I use the document.execCommand Javascript which is deprecated. Unfortunately there is still no real replacement for that so most developers still use that deprecated API. When changing font color and font size it uses the older tag.
There is a feature request for <font> support in flutter_html linked above, I made my case there to add support for the tag, hopefully the maintainers agree.

PR created at https://github.com/Sub6Resources/flutter_html/pull/694


Add coverage to sync http api.

This PR:

adds coverage for HttpClient.sendSync in HttpClientTests
brings @SyncAsyncTest extension from experimental branch to aid sync/async API pair testing.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core-test


Study: Foldable / displaced comments & synopses

MaweJS scenes are made of different types of paragraphs:

Text exported to draft: <p>, and` paragraphs. These are fine as they are now.
Text not exported: <synopsis>, <comment>, <fill> and <tags> paragraphs.

So far, I have mostly used Synopsis to put a "bookmark" to the middle of the scene. Now, when struggling one of my stories, I am sketching an outline editor, which is meant to create synopses. This would mean, that the amount and length of synopsis text will grow.
Current implementation: Currently, comments and synopses are just plain paragraphs (<p> elements). They work fine, because that makes them editable in the editor itself. There is no need to click around to change them.
The drawback of current implementation is that you tend to keep comments and synopses quite short and scarce, so that they don't disturb the flow of the text. Earlier I have tried out many different styling of these blocks, but so far I have found best to keep them as un-intrusive as possible while stille being able to easily notice.
To encourage writers - that is, me - to write longer synopses and keep more comments around, we could try out some techniques to shorten longer blocks.
Margin notes: One of my old ideas have been, that comments and synopses could be pushed to "margin", that is, a floating box outside the text itself. They could also be pulled back to text somehow.
That is bit difficult to implement with SlateJS and React. The implementations I have investigated for this, generally work so that they have their own database for margin comments, and both comments and anchors are rendered separately. This may be fine, true, but this will also mean, that you can't edit the comment / synopsis together with the text, but you need to separately click your cursor to comment/synopsis box.
Folded notes: I have thought a "light-weight" solution for this. Now that editor has folding, it could be possible to use folding feature to compress synopses and comments to single like (disable word-wrap, give fixed max-width and give ellips, just like in index). This could help a bit.
This is not necessarily even difficult. We'd need to make p.comment.folded style, apply that style to comment block when rendering, and add some code to folding to recognize comment & synopsis blocks as foldable.

Tried it out shortly: comments, synopses and missing are truncated to oneliners when folded.
Folding needs more work, so that folded blocks are protected from cursor movement. When a block is truncated to oneliner (in CSS), SlateJS does not understand how it should move around.

One possible idea:
Tables? What if we make certain elements as 2-column tables? That way you could have some sort of table heading in the first column, and content in second column.
This could be possible implementation of so called metablock:

#223

Metablock is meant to collect information together, so that it can be attached to scenes and chapters for various purposes.

There are now synopsis and comment (notes) scenes. These are foldable, so this particular problem is partially solved.


Androidx Android Support Library

I'm trying to get cordova-diagnostic-plugin and mauron85/cordova-plugin-background-geolocation to get installed in the same App, but i get alot of compatibility issues.
Are there any plans to support androix:compat?

Also using AndroidX, subscribing in for a resolution


Labeling of Transactions, Actions, and Requests

Goal
We want to label transactions, actions, and requests. So we can group these better. See https://training.xceptance.com/qa-and-test/050-performance-rating-system-devtest-leeds-2019.html#/14 for an example of categories (homepage, catalog, cart, account, checkout, place order).

These labels have to be part of the testreport XML and later HTML.
For performance reasons, this labeling has to happen when all data was collected because it does not merge or split data, just "stamps" it.
A label can be any string, such as homepage, #test, F17, Foo Bar or even an emulated double label, such as "homepage, B2C" (w/o the quotes of course). Barely any limits on what characters are possible.

Non-Goal
We would rather not have the full freedom yet to grab data from the evaluation and move it into the label. This is a  very static approach.
Examples

Transactions: transactional, registered
Actions: catalog, cart, checkout, homepage
Requests: catalog, cart, checkout, homepage

Non-Goal
We would rather not add multiple labels at the moment, so we don't think tags here.
Requirements
Report
All label information has to be part of the output XML so the reporting can later on incorporate it as well as the upcoming evaluation module.
The transaction, action, and the request table in the section of the report will get a new column "Label" (next to name) which supports sorting and filtering similarly to "Name".
One should be able to filter one column and later the other. We imagine two input fields here so it is clear where you filter, and we don't introduce any special syntax for the filters by that (this is lower prio and can be moved into a future release).
When there is no label attached, the cell or column stays empty.
Definition
The test suite gets new properties for the labeling. We will use a similar concept to merge and Apdex rules.
com.xceptance.xlt.reportgenerator.label.1.matching.name = .*Homepage$
com.xceptance.xlt.reportgenerator.label.1.matching.name.exclude = ^Goto.*
#com.xceptance.xlt.reportgenerator.label.1.matching.label = ordering
#com.xceptance.xlt.reportgenerator.label.1.matching.label.exclude = bathtub.*
com.xceptance.xlt.reportgenerator.label.1.label = Homepage
com.xceptance.xlt.reportgenerator.label.1.stopOnMatch = false

Matching: We match against the name and label only. One or the other has to be given, if both have to be given, we run an AND operation across all matching types.
Exclude: gives us the ability to specify things to negate against in case our regexp cannot cover that.
stopOnMatch: Allows us to stop match rule processing for that name if needed, this will simplify rule setup
Label: The label to apply. This is just a string, so anything is permitted here.

Rules are processed in order (1, 2, ...) and you can have holes in the list (similar to mergerules) and you don't have to start with 1 or have an upper limit (well.. Integer.MAX is the limit, kind of).
When
Labeling takes place after merge rule processing, so labeling can efficiently cache many evaluations and does not have to perform it again. Ensure that this is per thread to avoid contention (similar to merge rules).

Labeling rules are similar to merge rules, so applied the same mechanism here. Also reused the rule configuration syntax.


发现这个模型对于数量好像没有概念，比如希望进行限定字数的生成时，不符合预期

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
1
Expected Behavior
No response
Steps To Reproduce
1
Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA Support (`python -c "import torch; print(torch.cuda.is_available())"`) :

Anything else?
No response

chatgpt3.5偶尔也会这样，gpt4会好一些，感觉目前的大模型对一些outdomain constraint还不够好


Add the Owner property to BufferHandle

This would stop having to carry around the buffer with the pinned handle. This is already carried around by the Handle anyway, so it seems a waste to have to have "BufferHandle" + "Buffer" be passed through when calling functions.

But now callers can get OwnedBuffers from Buffer<T>. This is not good. The idea behind the architecture is that OwnedBuffer<T> is never shared around and instead Buffer<T> is shared.

Would mean this could go away https://github.com/dotnet/corefxlab/blob/master/src/System.IO.Pipelines.Networking.Windows.RIO/Internal/RioThread.cs#L94-L135

So here is the scenario (psuedo code)
So I want to avoid pinning for each of the NativeCalls but instead take the hit once (imagine there is more than 1 native call). I could wrap the pointer into a span, but then I need to fix the span in each call as well. So I want to pass the handle, that means either

fixing a span that contains a pointer that is already on a fixed object is very close to free? If so forget this I don't care
I have to pass both around, which just seems very ugly
There is a size put on the bufferHandle?
There is an owner property on the bufferHandle
I have to pass both around which is pretty ugly

async Task UpperMethod()
{
    var pooledBuffer = FromSomePool();
    var bufferHandle = pooledBuffer.Pin();
    do(true)
    {
            var workNotDone = SomeNativeWorkThingThatWillNeedToPinBuffer(pooledBuffer);
            if(workNotDone)
            {
                 await SomeThingToGetMoreWork(pooledBuffer);
            }
            else
            {
                 break;
             }
     }
     bufferHandle.Release();
     //return buffer
}

@dotnet-bot test this please

But now callers can get OwnedBuffers from Buffer. This is not good. The idea behind the architecture is that OwnedBuffer is never shared around and instead Buffer is shared.

@Drawaes, can you address @KrzysztofCwalina's point?

I guess my view on it is, if you are pinning stuff and handing around a pin, it's got a pointer on it anyway, so I don't see it as an added risk that you could get access to the owned buffer....?

As long as its not directly the type e.g. OwnedBuffer; or a suggestion for people to as cast it e.g. object
Can it be non browsable/non-intellisense? (or DangerousOwner...)

@Drawaes, is this change still relevant? If so, can you please open an issue in corefx for API review? Ty.
Closing this PR given the type has moved to corefx.
cc @benaadams, @KrzysztofCwalina


I wonder pretrained model  YOLO

When I install using ./download_yolo.sh , I have yolo weight.
Is weight trained all COCO training set? ( 90 class)

Yes, the downloaded yolo models were trained with COCO dataset.  It outputs 80 classes.  You could refer to this file: darknet/data/coco.names

Thank you!!!!


Copilot not showing when there is active alert

@okgrow/react-native-copilot 2.4.1
react-native: 0.56.0
react-native-navigation: 2.0.2641 (this is likely not relevant, just in case)
native-base: 2.8.1(this is likely not relevant, just in case)
Xcode 10.1
iOS simulator iPhone6 - 12.1
When there is an active alert on screen (usual RN alert, not 3rd party). this.props.start() doesn't initiate showing steps, when no active alerts it works as expected.

Does this happen when you start the walkthrough while an alert is being shown? Or when you're in the middle of a walkthrough and you make an alert, it vanishes the walkthrough?

The first scenario, when I start the walkthrough and an alert is already being shown.


Balena Sound keeps waiting for supervisor

i am running Balena sound on a raspberry pi 3A+, but i can't connect via anyting. everyting keeps disconnecting after a few seconds. no airplay, spotify connect or bluetooth.
i am running all services but every service keeps waiting for the sound-supervisor. even when i kill the sound-supervisor they are still waiting for a ip-adress that doesn't exist in my network.
i would like to use it for spotify connect but i have been reading that spotify is blocking that service.
this is what i got
audio  Waiting for sound supervisor to start at <IP_ADDRESS>:80
multiroom-client  Waiting for sound supervisor to start at <IP_ADDRESS>:80
Restarting service 'sound-supervisor sha256:6d607fcb4205bdbcd29d12576d576a7cdd180c830674efaa190074ccba782049'
sound-supervisor  standard_init_linux.go:219: exec user process caused: exec format error
Service exited 'sound-supervisor sha256:6d607fcb4205bdbcd29d12576d576a7cdd180c830674efaa190074ccba782049'
multiroom-client  Waiting for sound supervisor to start at <IP_ADDRESS>:80
multiroom-server  Waiting for sound supervisor to start at <IP_ADDRESS>:80
audio  Waiting for sound supervisor to start at <IP_ADDRESS>:80
multiroom-client  Waiting for sound supervisor to start at <IP_ADDRESS>:80
anyone any suggestions?

same with me

same thing

I am getting the same error. The Container keeps restarting.
Error response from daemon: Container 1908008c48508d8c38b6390d0dbe074735f97a849372da190cd8b699cc197fc1 is restarting, wait until the container is running

same for me

Same issue here

I have the same with latest balena-sound update
root@balena:~# balena logs sound-supervisor_6454329_2559361_5fce214260f3dc89830b60cdf1b3c600
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error
standard_init_linux.go:219: exec user process caused: exec format error

I got the same issue with a Raspberry Pi 3B

I found this solution
modify in core/sound-supervisor/Dockerfile.template
FROM balenalib/%%BALENA_ARCH%%-node:14 as build
WORKDIR /usr/src
COPY core/sound-supervisor/package*.json ./
RUN npm install
#dev-cmd-live=npm run livepush
COPY core/sound-supervisor/src ./src
COPY core/sound-supervisor/tsconfig.json .
RUN npm run build
COPY VERSION .
#FROM node:14-alpine3.14 (replaced by following line tho get proper architecture
FROM balenalib/%%BALENA_ARCH%%-alpine-node:14
WORKDIR /usr/src
COPY core/sound-supervisor/package*.json ./
COPY --from=build /usr/src/build ./build
COPY VERSION .
RUN npm install --production
CMD [ "npm", "start" ]

For anyone not quite technical enough for GitHub like me, I've created some instructions for applying the fix above over on the other issue comments thread #637

A fix for this has been merged.


Domain::createSubscriber fails if executed on another thread [5625]

Hi there,
I'm getting an exception trying to call Domain::createSubscriber.
In our system we use a discovery message sent on a wildcard partition. The discovery message contains a string with a specific partition name.
Upon receiving the discovery message, I want to create a new subscriber, subscribing to another message with the given partition.
Now, when I receive the discovery message, and I then try to create a new subscriber inside the discovery receiving thread, the thread exits without an exception at all.
Is this expected behaviour? Do I have to create all the subscribers inside the main application thread?
If I bypass the discovery and create my specific subscriber inside the main app thread it works as expected.
Regards,
Dominik

Hi @codewithpassion
Let's see if I understand what you mean. So you are trying to create another subscriber in the first subscriber's listener function. Or are you creating it in a new thread in there? And then, what's happening? Are you getting an exception or the thread exists well?
I've tried to make an example code and what I got is a deadlock, because the first subscriber stops listening because it's waiting for the new subscriber.
Can you post some code so I can understand better what you want to do?

Hi @AlvaroMolinillo thanks for your response.
Yes, I'm trying to create a new Subscriber inside the first subscribers listener function.
I've got this in a bigger piece of code that I cannot post like that but I'll recreate it and post it.
As I'm travelling at the moment, this will take me a few days.
Thanks
Dom

Hi @codewithpassion
Any advancement with your issue?
If you can provide us the example code we will review it, but as @AlvaroGonzalezMolinillo said, a deadlock is expected.

Closing this, as the question was answered. Feel free to reopen if still an issue.


KEP-1672: update docs for EndpointSliceTerminatingCondition

Update docs for EndpointSliceTerminatingCondition
https://github.com/kubernetes/enhancements/issues/1672

This looks ready for tech review.

/assign @robscott

Thanks @andrewsykim!
/lgtm

Hi. This now needs to target main
/lgtm cancel


Could export UIBezierPath to SVG path?

Could export UIBezierPath to SVG path?

As this is currently only a macOS application I assume you are talking about NSBezierPath. I think CoreGraphics does not provide a native api for doing so but there are libraries doing that: https://github.com/mro/MROGeometry/
Still, in ReformGraphics we have a custom struct for representing shapes: https://github.com/laszlokorte/reform-swift/blob/master/ReformGraphics/ReformGraphics/Shape.swift#L11-L27
This would be straightforward to serialize into SVG.


Doc Feedback:

Location : https://apim.docs.wso2.com/en/3.2.0/install-and-setup/setup/distributed-deployment/configure-apim-analytics/active-active/
While trying to create an HA configuration we had to piece together directions from this other page: https://apim.docs.wso2.com/en/3.2.0/learn/analytics/configuring-apim-analytics
This lead to lots of guessing and a so far, unsuccessful installation.
Please provide complete directions for creating an this configuration.

@bhathiya & @tgtshanika , can you please assign an SME to look into this please.

@mushthaq33 & @samgnaniah , please look into this issue based on Bhathiya's reply to the email in this regard.

Internal tracking reference A723


python310Packages.pontos: 23.1.3 -> 23.2.0

Diff: https://github.com/greenbone/pontos/compare/refs/tags/v23.1.3...v23.2.0
Changelog: https://github.com/greenbone/pontos/releases/tag/v23.2.0
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.pontos
python311Packages.pontos


Fix #82 :: Space_Grotesk font-family & Some update ✅

[ ] Fix #82
[x] Update Telegram link

Just forgot to add the fonts in MobileMenu.tsx

Yes, I forgot about the mobile menu. I'll fix that quickly!

@NDOY3M4N also fixing #97


Make operation_name public

See #253.

Codecov Report

Merging #265 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #265   +/-   ##
=======================================
  Coverage   87.53%   87.53%
=======================================
  Files          97       97
  Lines       14200    14200
=======================================
  Hits        12430    12430
  Misses       1770     1770

Impacted Files
Coverage Δ

juniper/src/http/mod.rs
0% <0%> (ø)
: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 bd455c9...0ac8ba0. Read the comment docs.

Where can I add an integration test? Is there a collection of them already that exists, or should I create a juniper/tests directory alongside src in that crate?

I think the tests directory is the way to go. You'll need to use GraphQLRequest::new() and then call operation_name(). If it is private (that is, we undo this change), the integration test should then fail to compile.
Let me know if you run into any issues!

@cgm616 Thank you very much! As soon as CI goes green I will merge.

🎉


Implement Aycock-Horspool precomputation of nullable token sets

from Aycock & Horspool "Practical Earley Parsing", The Computer Journal, Vol. 45, No. 6, 2002

Marpa continues to use the ideas of Aycock & Horspool on using a rewrite to deal with nullable symbols and rules. And in fact, Marpa uses the rewrite to deal with other issues.

Hi @jeffreykegler. What issues are you talking about? What advantage does converting to NNF get you? Thanks!

@blob8108 The set of issues that I do rewrites for is evolving.   At this point the main use, other than for nullables, is for statements which extend BNF -- sequence rules, and precedence rules.  These are implemented via a rewrite into BNF.


⚠ ClamAV has degraded performance

In 7e29c1e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 52 ms

Resolved: ClamAV performance has improved in 468c6c8.


[FEATURE] Make OrderBook API's order_type an optional input field

Is your feature request related to a problem? Please describe.

I may want to get both buy and sell OrderBook in one API request.
Currently, order_type is a required field, but I think it could be made an optional field.

Describe the solution you'd like

Make order_type an optional field in the OrderBook API.
If nothing is set, all OrderBooks will be returned.
Add order_type to the response item.

Since this functionality is not considered that necessary, I would deem it unnecessary to support.


Crud operations runs

Describe the pull request
A util method for CRUD operations.
Validation
Create
Adds a new run to the db when the user saves their run by pressing the following icon:

Code:
  FirestoreCRUDUtil firestoreCRUD = new FirestoreCRUDUtil();
  firestoreCRUD.createEntry("runs", run);

Results:

Read
Will get a specific entry.
Finds an entry by documentId (which is also the track.getUuid() value)
Code:
       Map<String, Object> updateRun = new HashMap<>();
       updateRun.put("id",127);
       updateRun.put("ascent",1);
       firestoreCRUD.updateEntry("runs", updateRun);

Update
Will modify a specific entry.
Finds an entry by documentId (which is also the track.getUuid() value)
Code:
        Map<String, Object> getRun = new HashMap<>();
        getRun.put("id",123);
        firestoreCRUD.getEntry("runs", getRun);

Results:

Results:

Delete
Will delete a specific entry.
Finds an entry by documentId (which is also the track.getUuid() value)
Code:
       Map<String, Object> deleteRun = new HashMap<>();
       deleteRun.put("id",134);
       firestoreCRUD.deleteEntry("runs", deleteRun);

Results:
134 isn't present anymore

Link to the the issue
(If available): The link to the issue that this pull request solves.
License agreement
By opening this pull request, you are providing your contribution under the Apache License 2.0 (see LICENSE.md).
Note: new dependencies/libraries
Please refrain from introducing new libraries without consulting the team.

Very good. approved on my end.

I think we should aim to use the JSON type from the gson google lib, not a Map<String, Object>
This is simply bc they are not the same type, for example this would be a valid map but invalid json:
{
"person": Person
}
I know the documentation shows this, but imo its not an ideal use of strongly typed features
Essentially we have POJO Models that are already used internally as DTOs and for the SQLite persistence, and for the noSQL external database we know we need JSON, hence the serialization. If FireStore requires the Map<String, Object> I would personally opt for the adapter pattern to simply convert our generinc noSQL JSON type to the Map but only specifically for FireStore
I know it sounds nit picky, but the current implementation does not use my serialization at all, we need to construct the map manually every time, its not ideal
I would use static constants for the table names, not strings, and same thing for your Log TAGS in the crud util
now you have for example "runs" passed as argument to your functions, imagine what would happen if the "runs" files is renamed to "tracks" for example, we'd need to update a ton of different places
I would create a constants/FireStoreFiles or whatever and just export a final String RUN_TABLE = "runs"
Typically we want to code to interfaces, not classes
FirestoreCRUDUtil firestoreCRUD = new FirestoreCRUDUtil();
This is generally a bad idea, you should make an abstraction using polymorphism
ExternalStorage firestoreCRUD = new FirestoreCRUDUtil();
Missing docs for the class (awesome for the methods! but its nice to have docs for the class too)
Its a good practice in Java to make everything  final unless you plan on changing it, including parameters received in functions
Not necessarily here, but a note that we'll need good error handling at the top level whenever we call these methods!


Fix date test bug on Windows

had a localisation issue with creating dates for testing since they are created in powershell using "MM/DD/YYYY" format and my machine is localised to "DD/MM/YYY" so keeping them ISO8601 will solve this i.e. "YYYY-MM-DD"

Codecov Report

Merging #438 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #438   +/-   ##
=======================================
  Coverage   78.54%   78.54%
=======================================
  Files          34       34
  Lines        3360     3360
=======================================
  Hits         2639     2639
  Misses        721      721

Impacted Files
Coverage Δ

src/sort.rs
97.97% <ø> (ø)

src/meta/date.rs
93.25% <100.00%> (ø)

tests/integration.rs
100.00% <100.00%> (ø)

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 3537575...bc74f9e. Read the comment docs.


warninng to default disabled service on startup

My void linux created this down file by default disabling the kmonad service (created with xmksv) and took me some days to note this, so I want to warn newcomers to not spend so much time on this as I did.

Thanks!


feat: limit concurrent access to some parts of the api resources

This change is

This should be also documented on OpenAPI level!

This should be also documented on OpenAPI level!

Thanks for noticing it! Fixed in #2387


Veracode issue in GDTCCTNanopbHelpers.m - Unchecked Return Value

Environment
Xcode version: 11.3.1
Firebase SDK version: 6.26.0
Firebase Component: GoogleDataTransport (6.2.1)
Installation method: CocoaPods
Description:
Ignoring a method's return value can cause the program to overlook unexpected states and conditions.
Relevant Code:
.../GDTCCTNanopbHelpers.m 44
.../GDTCCTNanopbHelpers.m 112

@YessineM Thank you for filing the issues. Could you please provide a bit more information on your concern. As far as I can see the return values are validated and used at the specified lines:

.../GDTCCTNanopbHelpers.m 44
.../GDTCCTNanopbHelpers.m 112


Added Controller function for delete comment

fixes #128

@Shubham-1105 this pr contains changes to frontend files. Please check if you committed changes by mistake.
@salil-naik Please review the frontend files if there are good to go.

I had worked on this files for previous issue,in this pr i had only worked on the backend controller file

please resolve the conflicts @Shubham-1105


AUTOMATED_CONVERSION_PRESET does not handle space around "="

"AUTOMATED_CONVERSION_PRESET=Custom/Preset" works properly but "AUTOMATED_CONVERSION_PRESET = Custom/Preset" does not work as intended. The only difference being spaces before and after the space. Using docker compose for reference.

Can you share your docker compose file ?  Environment variables are defined like this:
    environment:
      AUTOMATED_CONVERSION_PRESET: "Custom/Preset"

services
handbrake:
container_name: handbrake
image: jlesage/handbrake
restart: unless-stopped
ports:

5800:5800
volumes:
${Docker}/Handbrake/Config:/config
${Docker}/Downloads/Downloaded:/watch
${Docker}/Downloads/Processed:/output
environment:
TZ = ${TZ}
AUTOMATED_CONVERSION_KEEP_SOURCE = 0
AUTOMATED_CONVERSION_PRESET=General/HighQ

Either "var=value" (no blanks) or "var: value" (one blank after ":"), see https://docs.docker.com/compose/compose-file/compose-file-v3/#environment

Closing this issue.  Please re-open if needed.


Can't manage implements after latest PF2 update

As of Pathfinder 2e version 5.11.0, I'm getting the following error when trying to Exploit vulnerability using the macro:
foundry.js:68326 You do not have implements selected. Go to the inventory tab on your character sheet, click manage implements, and assign items to your implements.
When I go to inventory to "manage implements", I can open the manage implements window, but dragging items into it does nothing.

I'll work on this when I get a chance. I'm working a ton of overtime right now at work. I think I'll have some time this weekend.

I'll work on this when I get a chance. I'm working a ton of overtime right now at work. I think I'll have some time this weekend.

Update: PF2 version 5.11.1 fixed the broken drag and drop functionality and that fixed it! However, it no longer marks the implement with a symbol on the character sheet, so you can't see which ones you have set as implements.

Okay, glad that 5.11.1 at least mostly fixed it. For the time being, if you click the manage implements button you should still be able to see what you have selected as implements. I understand it's not the most desirable way to see it, but it should help a little.

Okay, glad that 5.11.1 at least mostly fixed it. For the time being, if you click the manage implements button you should still be able to see what you have selected as implements. I understand it's not the most desirable way to see it, but it should help a little.

Trust me, I’m just glad that it’s working. I have a player who uses this all the time. I’m not worried about the symbol, just wanted you to know. They update that system so often that something always breaks somewhere.


Status bar:

Append to existing status text:

[ ] {Shortcut} for docs. (When docs are available for that identifier / when it's a token)

@abmirayo It seems we won't be able to resolve which keybinding is currently associated with our command, in the case the user changed it, so it'll be great if could have some kind of fallback strategy

@abe33
Yeah, makes sense. I think fallback should be Docs available.


fix: Replace string-template with url-template

Fixes #1099
Path parameters are incorrectly encoded. This fix uses a different module to parse the path parameters, using the same RFC6570 format as the API Discovery Documents uses

[x] npm test succeeds
[ ] Pull request has been squashed into 1 commit

1 commit with actual changes, 1 commit with regenerated files

[ ] I did NOT manually make changes to anything in apis/

I actually did! src/apis/videointelligence/v1beta1.ts is generated with wrong formatting

[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate JSDoc comments were updated in source code (if applicable)
[x] Appropriate changes to readme/docs are included in PR

CLA fixed


Not able to clear mui chips for Autocomplete with multiple select option

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
I have 2 Autcomplete fields added. On Changing the value in field 1: Type, I am resetting the options for field 2: Subtype based on the value selected in Type. I can able to see that the options getting updated. But the mui-chips selected is not getting cleared/reset.

Expected Behavior 🤔
I am expecting the mui-chips in Subtype field to be cleared whenever options are getting updated for Autocomplete.

Steps to Reproduce 🕹
Steps:

Select value as Banking from the Type field.
Now select Options from Subtype field like Corporate Banking, Private Banking,..
Now Change the value selected in Type field from Banking to Central Bank.
Now click on the options available for Subtype field. Options will be updated with new set of values, but the values selected were not getting reset.

codesandbox link:
https://codesandbox.io/s/autocomplete-multiple-wkumx
Let me know, if I am doing it in wrongway.
Thanks,
Prem Kumar

Tech
Version

Material-UI
4.11.0

React
16.13.1

Browser
Google Chrome

@gpremkumar You need to control the second combo box if you want this behavior: https://codesandbox.io/s/autocomplete-multiple-forked-onu5n.

Thanks for the quick response @oliviertassinari :+1:


Unable to uninstall angular cli

Angular checks for updates instead of uninstalling.
Area
devkit
Versions
node version: v8.11.2
npm version: 5.6.0
Windows: 10
Angular cli version: 6.0.8
Repro steps
I tried simply running:
npm uninstall -g @angular-cli
then I read I had to run:
npm cache clean --force
npm uninstall -g @angular-cli
The log given by the failure
The output is:

up to date in 0.055s

Desired functionality
I want to uninstall the angular-cli no update it.

This seems like a problem with your node/npm and not with Angular CLI.
Please have a look at the fixing npm permissions page, common errors page, npm issue tracker, or open a new issue in that issue tracker if the problem you are experiencing isn't known.

On Windows 7 the npm uninstall does not work. The solution I found is to 1. uninstall node.js, 2. delete the npm and npm-cache directories, 3 reinstall node js and 4 install the cli version you need. Note: ng -v will still show the prior version of the cli if it exists in a nod-modules directory in the path where you call it. it.
In my case the directories to delete were as followed:
C:\Users\userid\AppData\Roaming\npm-cache
C:\Users\userid\AppData\Roaming\npm
To install specific version of the Angular CLI:
npm install -g<EMAIL_ADDRESS>Finally to have ng -v report the correct version. I deleted a node-modules directory which contained the previous Angular cli version (in my case was 6.1)
Scott

I am getting this erro while installing Angualr
C:>npm install -g @angular/cli
[..................] | loadIdealTree:loadAllDepsIntoIdealTree: sill install loadIdealTree[..................] / fetchMetadata: sill removeObsoleteDep removing<EMAIL_ADDRESS>from the tree as its been replaced by a newer version or is no longer [..................] - fetchMetadata: sill removeObsoleteDep removing<EMAIL_ADDRESS>from the tree as its been replaced by[..................] | fetchMetadata: sill removeObsoleteDep removing<EMAIL_ADDRESS>from the tree as its been replaced by a n[..................] - fetch[        ..........] | extract:run-async: http fetch GET 200 https://registry.npmjs.org/promise-retry/-/promise-retry-1.1.1.tg[        ..........] | extract:run-asynpm ERR! path<EMAIL_ADDRESS>npm ERR! code EPERM
npm ERR! errno -4048
npm ERR! syscall unlink
npm ERR! Error: EPERM: operation not permitted, unlink<EMAIL_ADDRESS>npm ERR!  { [Error: EPERM: operation not permitted, unlink<EMAIL_ADDRESS>npm ERR!   cause:
npm ERR!    { Error: EPERM: operation not permitted, unlink<EMAIL_ADDRESS>npm ERR!      errno: -4048,
npm ERR!      code: 'EPERM',
npm ERR!      syscall: 'unlink',
npm ERR!      path:
npm ERR<EMAIL_ADDRESS>},
npm ERR!   stack:
npm ERR!    "Error: EPERM: operation not permitted, unlink<EMAIL_ADDRESS>npm ERR!   errno: -4048,
npm ERR!   code: 'EPERM',
npm ERR!   syscall: 'unlink',
npm ERR!   path:
npm ERR<EMAIL_ADDRESS>}
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It's possible that the file was already in use (by a text editor or antivirus),
npm ERR! or that you lack permissions to access it.
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      C:\Users\GANESH\AppData\Roaming\npm-cache_logs\2019-06-11T04_35_21_488Z-debug.log


Sum, Mean Aggregations support for Prometheus exporter

Fixes #215

Awesome, thank you for the review @rakyll!


script fails to update bsd.sp kernel

I have MP system and the script only copies the bsd.mp -> /bsd and bsd.rd -> /bsd.rd.
Problem with this is that normal upgrade using bsd.rd copies the bsd kernel to /bsd.sp which this script doesn't do. Just got burned by this as I had old bsd.sp kernel with new userland which causes issues.
Simple fix should be to add following to update_kernel function:
cp bsd /bsd.sp || FAIL=1

@zmyrgel is this a machine that does MP, but you are intentionally running a SP kernel? Is the output of sysctl -n hw.ncpufound > 1?

Yes, this is MP machine (Atom box). Recent changes to the em driver started to cause system freezes and I wanted to test if the problem exists with SP kernel as well. And also if the system would function with SP kernel so long as the MP-issues would get sorted out.
The problem was that the SP kernel wasn't as up-to-date as rest of the system which caused then different panics.
I don't think that there's any problem copying the SP kernel as /bsd.sp. This would allow MP-systems to boot SP kernel when needed. SP systems would have two kernels, /bsd and /bsd.sp.

I keep going back and forth between adding bsd.mp and bsd.sp kernels (along side the regular /bsd).. and adding a flag to just force using the sp kernel. Leaning more towards the flag..

I suggest copying both bsd.mp and bsd.sp to / and then copying the relevant kernel to /bsd. Not only does this allow a user to easily determine which kernel /bsd is (ls -l /bsd* and compare file sizes) but offers the most flexibility. It's also relatively similar to what the BSD installer does:
	if [[ -f /mnt/bsd.mp ]] && ((NCPU > 1)); then
		echo "Multiprocessor machine; using bsd.mp instead of bsd."
		mv /mnt/bsd /mnt/bsd.sp 2>/dev/null
		mv /mnt/bsd.mp /mnt/bsd
	fi

Thanks for the feedback! I also added a bit to backup the .sp kernel.


Sort values in AutoComplete if values are numerical

This solves the problem of the prepended zeros. Values inside AutoComplete are sorted based on their id if numerical values are expected
fixes #471

The docs are a bit unclear which flags exactly need to be true for this to work, but with the current flags it seems to do its job, i.e.,
localSorting={onlyNumbers}
sortByMatchedKeywords={true}
localFiltering={onlyNumbers}

you are right btw, the non-numerical autocomplete seems to fail now :-/ I don't understand why though

I addressed the comments, should work now! the strange ()=>func seems to be necessary for it to work.

I addressed the comments, should work now! the strange ()=>func seems to be necessary for it to work.

Ah, I see, you are right.


Clipboard replaced using "x" or "cw"

VIM will put into clipboard a "dd" or cut selection for pasting again later. With Atom if I even press "x" or "cw" my clipboard is replaced with that single character. This means that there is never a way to copy from another application except for insert new. Cut characters, start insert, switch apps, copy, then back to atom to paste.
Could argue that this is normal behaviour and expected - maybe it is - but it is not the same as VI/VIM.
However, I think this bug is more related due to the fact that ATOM shares the real clipboard. So in VI/VIM if you "x" or "cw" or "dd" you use the VI buffer, and cut/paste from either GUI VIM or Command Line still has the operating system clipboard. ie. they are separate. ATOM has just one - the operating system clip board.

+1 to have a seperate buffer for VI mode buffer and system clipboard.

Have you tried the "Use Clipboard As Default Register" setting?
In VIM and in Atom, I often use xp to switch two characters, e.g. in a typo like "foten" for "often".

In vim a delete within a line (x, dw, 2dw, d$, etc.) does not update the numbered registers but instead goes into the "- register (also called the "small register"). Maybe this (applied to the clipboard) should be an option if you have chosen "Use Clipboard As Default Register"...  Thus line-oriented deletes or changes would go into the clipboard but intra-line deletes or changes would not. Presumably any use of yank implies the desire to put it in a register and so yw and y$ should probably update the clipboard.
Another alternative would be to not update the clipboard until the end of the change command, so c2w^v<ESC> would change the next 2 words to whatever was in the clipboard and then replace the clipboard with those 2 words afterwards.
Another alternative is to train people in the use of the "_ register -- if you don't want your deleted/changed text to be recorded in a register then prepend it with "_. So if I wanted to replace the next 2 words with the clipboard I could type _2c^v<ESC>.

+1 cw should not copy what is removed into the clipboard, this is not how existing gvim works

Don't update register on change or substitute is the setting I had to enable for this to stop happening. Use clipboard as default register was checked by default I believe, and not doing it for me.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Add tests and QA analysis

The changes in this PR add covering tests for the code, and integrate QA tools so that a baseline of quality can be assured. To do this, the route handler code, that was originally stored in closures, was refactored to standalone callable classes. From there, small refactorings were also made to make the most of the new approach. Finally, the code was cleaned up with help from php-cs-fixer and Psalm.

Hey @pkamp3, thanks for approving the PR some time back. Could you merge it, as I don't have sufficient permissions to do so?


Update bindings and fix GUI to make standalone copies of tasks

Fixes issue #1268
Brief summary of changes
Make local copies of IKTask(s) for GUI use so they don't get garbage collected using copy constructors (instead of using clone)
Testing I've completed
Built GUI and couldn't cause the issue reported by Antoine to reproduce.
CHANGELOG.md (choose one)

no need to update because internal bugfix

@antoinefalisse if you can test the artifact for a fix to the scaletool exception that would be great. Thank you


soroban-token-spec: Expose full token contract spec

What
Expose the full token contract spec
Closes https://github.com/stellar/rs-soroban-env/issues/535
Why
See https://github.com/stellar/rs-soroban-env/issues/535 and https://github.com/stellar/soroban-cli/pull/214#issuecomment-1273738006
Known limitations
As noted by a TODO, the spec is not generated automatically, which could cause the spec to fall out of sync with the contract.
I tried to extend contractimpl to do so but I haven't written Rust procmacros before so I gave up before spending too much time on it.
It's worth noting that the contract itself is already manually kept on sync, so it's probably not much worse to do the same with the spec.

Since the library uses no_std it seems we cannot use a Vec<u8> either. Thus, I have resorted to use concat_arrays to return a static array.
This may be overkill though. I am open for suggestions.

I ended up pushing my solution after yours, but they are practically the same.
I would say it's pretty ugly to maintain, but maybe less so than a macro.
If we need the same for other contracts though, we should probably use a macro instead.

If we need the same for other contracts though, we should probably use a macro instead.

The challenge with a macro, and why the concat-arrays crate uses transmute, is that a macro is just a token processor and it can't know the length of each of the sub-components, that can only be determined by the compiler.
Once trait fns are supported in const fns (https://github.com/rust-lang/rust/issues/67792 https://github.com/rust-lang/rust/issues/57563), this should all get much much better.
Or even if https://github.com/rust-lang/rust/issues/57349 was implemented would clean this up significantly.

The challenge with a macro, and why the concat-arrays crate uses transmute, is that a macro is just a token processor and it can't know the length of each of the sub-components, that can only be determined by the compiler.

You get a compile-time error though, so I don't see it as a big problem.


add documentation hosted in pages

fixes #21

As an idea: Can we link docs in our pkg.go.dev page as well? Most of the time this is where people look for initial function/method level documentation so linking to more context from there would make sense imo. For instance the changes to the README will already be reflected there so the general link will be available. My idea would be to extend that onto the text markers you created if that is possible. Otherwise good work adding more docs!

I am not sure if I understand what you mean. Posting a link to https://pkg.go.dev is fine by me, although I have no clue how.

My idea would be to extend that onto the text markers you created if that is possible.

Sorry but you completely lost me here. extend that onto the text markers?

Otherwise good work adding more docs!

Thanks! I am still working on documenting how the docker containers with the fake docker compose work.

I suppose yes. I'm also not 100% sure right now which package doc gets rendered where but that looks about right. In addition the README will also be rendered so as you're putting the link there it will already link to the main docs page from the README here and the one at go.dev

Added the comment. I hope it is sufficient.