Access timeout error stored

Issue
If an access request returns an error other than system.accessDenied, such as system.timeout, the error will be stored in Resgate memory and used to deny access for any subsequent request.
Solution
Only system.accessDenied should be considered a valid error response. All other error responses should be seen as temporary, and should not be persisted.
Recreating the issue

Start book-collection example service
Load example client
Stop example service
Edit a book entry and click OK
After a Timeout error, start service
Edit the same book entry and click OK
Error is still Timeout

Log
$ resgate --config=config.debug.json
08:12:02 [Main] Starting server
08:12:02 [Main] Connecting to messaging system
08:12:02 [Main] Starting HTTP server
08:12:02 [Main] Listening on http://<IP_ADDRESS>:8080
08:12:15 [bia8ljt8smghgb2g5ol0] Connected
08:12:15 [bia8ljt8smghgb2g5ol0] --> {"id":1,"method":"subscribe.library.books"}
08:12:15 [NATS] <== access.library.books: {"token":null,"cid":"bia8ljt8smghgb2g5ol0"}
08:12:15 [NATS] <== get.library.books: {}
08:12:15 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK56F: {"result":{"get":true,"call":"*"}}
08:12:15 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5B0: {"result":{"collection":[{"rid":"library.book.1"},{"rid":"library.book.2"},{"rid":"library.book.3"}]}}
08:12:15 [NATS] <== get.library.book.3: {}
08:12:15 [NATS] <== get.library.book.2: {}
08:12:15 [NATS] <== get.library.book.1: {}
08:12:15 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5PH: {"result":{"model":{"id":3,"title":"Coraline","author":"Neil Gaiman"}}}
08:12:15 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5Fl: {"result":{"model":{"id":2,"title":"Brave New World","author":"Aldous Huxley"}}}
08:12:15 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5KW: {"result":{"model":{"id":1,"title":"Animal Farm","author":"George Orwell"}}}
08:12:15 [bia8ljt8smghgb2g5ol0] <-- {"result":{"models":{"library.book.1":{"author":"George Orwell","id":1,"title":"Animal Farm"},"library.book.2":{"author":"Aldous Huxley","id":2,"title":"Brave New World"},"library.book.3":{"author":"Neil Gaiman","id":3,"title":"Coraline"}},"collections":{"library.books":[{"rid":"library.book.1"},{"rid":"library.book.2"},{"rid":"library.book.3"}]}},"id":1}
08:12:30 [bia8ljt8smghgb2g5ol0] --> {"id":2,"method":"call.library.book.1.set","params":{"title":"Animal Farming","author":"George Orwell"}}
08:12:30 [NATS] <== access.library.book.1: {"token":null,"cid":"bia8ljt8smghgb2g5ol0"}
08:12:32 [NATS] x=> Request timeout
08:12:32 [bia8ljt8smghgb2g5ol0] <-- {"error":{"code":"system.timeout","message":"Request timeout"},"id":2}
08:12:37 [NATS] ==> system.reset: {"resources":["library.>"]}
08:12:37 [NATS] <== get.library.book.3: {}
08:12:37 [NATS] <== get.library.book.1: {}
08:12:37 [NATS] <== get.library.books: {}
08:12:37 [NATS] <== get.library.book.2: {}
08:12:37 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5Yn: {"result":{"model":{"id":3,"title":"Coraline","author":"Neil Gaiman"}}}
08:12:37 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5dY: {"result":{"model":{"id":1,"title":"Animal Farm","author":"George Orwell"}}}
08:12:37 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5iJ: {"result":{"collection":[{"rid":"library.book.1"},{"rid":"library.book.2"},{"rid":"library.book.3"}]}}
08:12:37 [NATS] ==> _INBOX.cerdnYTSkhu6JlTQDbK5n4: {"result":{"model":{"id":2,"title":"Brave New World","author":"Aldous Huxley"}}}
08:12:42 [bia8ljt8smghgb2g5ol0] --> {"id":3,"method":"call.library.book.1.set","params":{"title":"Animal Farming","author":"George Orwell"}}
08:12:42 [bia8ljt8smghgb2g5ol0] <-- {"error":{"code":"system.timeout","message":"Request timeout"},"id":3}
08:12:46 [Main] Stopping server...
08:12:46 [Main] Disconnecting all ws connections...
08:12:46 [bia8ljt8smghgb2g5ol0] Disconnecting - Server is shutting down
08:12:46 [bia8ljt8smghgb2g5ol0] Disconnected: read tcp [::1]:8080->[::1]:54711: use of closed network connection
08:12:46 [Main] All ws connections gracefully closed
08:12:46 [Main] Stopping HTTP server...
08:12:46 [Main] HTTP server gracefully stopped
08:12:46 [Main] Closing the messaging system's client connection...
08:12:46 [Main] Message queue connection closed
08:12:46 [Main] http: Server closed
08:12:46 [Main] Stopping rescache workers...
08:12:46 [Main] rescache stopped
08:12:46 [Main] Service stopped

Fixed in #73


[manuf] add TPM cert generation to personalization binary

This patchset adds generatiinig of TPM certificates during personalization, storing  all three TPM certificates in a dedicated flash INFO page and then hashing the certificates contents on the device after reading them back from flash, and comparing with the hash calculated on the host.

There are some style inconsistencies which slipped through while I was splitting the patch, I'll fix then along with other review comments.

One thing I realized (though it is ok to address in a follow up PR): the HJSON definitions of each TPM cert are almost identical, with the exception of the subject common name. If we made that a variable, then couldn't we just define one TPM HJSON cert template and instantiate the 3 different versions (EK, CIK, CEK)? Or would the variability in the sized of the subject common name's cause an issue? @pamaury

Just a few cleanup suggestions otherwise this looks great @vbendeb ! Thanks for fixing my bugs too :)

Also, is the cert chain validation step on the host side missing? or were you planning on adding that in a follow up PR? @vbendeb

One thing I realized (though it is ok to address in a follow up PR): the HJSON definitions of each TPM cert are almost identical, with the exception of the subject common name. If we made that a variable, then couldn't we just define one TPM HJSON cert template and instantiate the 3 different versions (EK, CIK, CEK)? Or would the variability in the sized of the subject common name's cause an issue? @pamaury

Absolutely, the generator has no problem handling variable sizes, just make sure the variable in the template is defined with the maximum size possible. It would also save some code size so probably nice to have.

One thing I realized (though it is ok to address in a follow up PR): the HJSON definitions of each TPM cert are almost identical, with the exception of the subject common name. If we made that a variable, then couldn't we just define one TPM HJSON cert template and instantiate the 3 different versions (EK, CIK, CEK)? Or would the variability in the sized of the subject common name's cause an issue? @pamaury

Absolutely, the generator has no problem handling variable sizes, just make sure the variable in the template is defined with the maximum size possible. It would also save some code size so probably nice to have. Great idea!

There are in fact two different TPM certificate forms, CEK and CIK certificates do not include some extensions TPM EK has. This could be consolidated, but would require some fine tuning, let's leave it out of this PR.

Also, is the cert chain validation step on the host side missing? or were you planning on adding that in a follow up PR? @vbendeb

yes, I mean to do this in the follow up PR. We also need a check that the device can re-generate the private key when owner FW is runing.

I went through the comments and made modifications, please take another look.

while working on openssl chain validation I fouond a bug in size calculation when hashing certificate contents, PR  updated.


charts/snowplow-iglu-server: Add a new helm hook job to create the dev db postgres user

The postrgres user for dev db is being created using a local-exec, we need to move this to a helm hook job

Completed in https://github.com/snowplow-devops/helm-charts/pull/200


Tuya E14 RGB LED tz3210_ifga63rg_ts0505b not working with ZHA

You can't turn on the light bulb. No RGB, brightness interface.
There is no bulb with ID ifga63rg in the Blackadder database.
I only use ZHA so I can't say if it would work with Z2M.

@rozsap did you open this by mistake?

Sorry, I didn't mean to go here.

I'm deleting it then. No problem.


perf(discv5): boost bootstrap lookups

Fills the three buckets at log2distance furthest from the local node id, at 5 second intervals, at bootstrap. This gives RLPx a better chance of finding peers instantly after bootstrapping.

lgtm
would one second be too noisy? Ideally we would keep the aggressive discovery until we we connect to a peer in our chain, but can be done later on, I'm not even sure if we do that for discv4

this is stats I have so far, from RLPx peering. all started clean (without a known_peers.json file, and new non-default port).
https://github.com/paradigmxyz/reth/pull/7683
reverses start lookup target (start filling buckets at log2distance furthest away from local node first)

on this branch
run 1

run 2

clearly this pr is better for bootstrapping, it makes a successful outgoing RLPx connection within 5 mins. think we only need to do the pulses for 100 counts (with 5 sec sleeps, 100 counts amount to 20 mins, logs show bootstrap count down becomes zero from 14:33:55 to 14:55:25). after that, main doesn't perform better than this branch in making outgoing RLPx connections.
we can expose the bootstrap pulse interval and the countdown in cli in a separate pr, then we can try 1 second like that

hm, we could move the bootstrap pulse out of the loop, and stack them on top of each other without sleep, let me try that

hm, we could move the bootstrap pulse out of the loop, and stack them on top of each other without sleep, let me try that

tried this, it's not good. the queries need some time in-between so that the next query can use the returned peers from the last query, 5 seconds seems good.
thought query_parallelism in discv5 would be handled at a higher level, but it's actually handled inside each query, not among queries. so doesn't work if they aren't executed more or less sequentially.


Investigate Sentry Errors

See if they are critical for launching widely

Make errors not cryptic especially on devices that have IP as non-localhost.


not an error just asking

bro why did you removed nsfw generates ?
its shows flowers. When i write nsfw prompt
please bring it back. :(

bro why did you removed nsfw generates ? its shows flowers. When i write nsfw prompt
please bring it back. :(

Hi ,
Thank you for reaching out and sharing your feedback regarding the removal of the NSFW generates feature. I appreciate your input and understand your concern.
The decision to remove the NSFW feature was made in order to align the package with certain guidelines and policies, and to ensure a wider range of appropriate use cases for our users. However, I understand that this feature may be important to some users.
While I am unable to directly add the NSFW generates feature back into the package, I encourage you to explore customizations or extensions that can be implemented on top of the existing package to meet your specific requirements. This will allow you to tailor the functionality according to your needs while maintaining the core package's guidelines.
If you need any assistance or have any further questions, please feel free to reach out. I'm here to help.
I m the package creator not the server side
Thank you once again for your feedback.
Best regards,
dj-yacine

So, now we can't generate nsfw, right?


Update RepeaterModule.java

修复console后台刷新节点，导致订阅器重复问题

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


Add support for multiple mouse button events

Once the left mouse button is clicked, you cannot check for the right mouse button until that touch is finished.
See: https://groups.google.com/forum/#!topic/kivy-users/dlgPYB4Aui8
Minimal example:
from kivy.app import App
from kivy.uix.widget import Widget

class MainWindow(Widget):
    def on_touch_down(self, touch):
        # Once the left mousebutton is pressed, this does not fire for the
        # right mouse button until the left is release
        print(touch.button, "is being pressed")

    def on_touch_move(self, touch):
        print("touch moving: {0}".format(touch.button))

    def on_touch_up(self, touch):
        print(touch.button, "was released")

class TestingApp(App):
    def build(self):
        return MainWindow()

if __name__ == "__main__":
     TestingApp().run()

Following the thread from the mailing list, it seems there is another issue here, too.  Which is that 'on_mouse_XXX' events on the Window are not being delivered, either.  Ever.  Which may have something to do with why on_touch_up events are being missed.

Ran the code on Windows 11, Kivy 2.2.1.
Far worse than described. Right button releases are ignored. Pressing right button can make left button release instead.

Far worse than described. Right button releases are ignored. Pressing right button can make left button release instead.

@Julian-O This is also related to #5898 These issues all seem to be cause by the implementation of the MouseMotionEventProvider(), https://github.com/kivy/kivy/blob/master/kivy/input/providers/mouse.py
These look like a deceptively easy set of fixes to the on_mouse_press, on_mouse_release, and on_mouse_motion methods.  Are there tests associated with this file?

@ElliotGarbus: I can't comment meaningfully on the technical side here. I don't know what an EventManager is, and I don't understand the cost of adding new MotionEvents. (That said, I do understand combinatorial explosion - is there a limit to the number of distinct buttons a mouse can have?)
Are you proposing that this prototype become part of the API, or more a proof-of-concept that users can build there own?
If the latter, I would like to see it documented (e.g. added as an example) because the current behaviour is astonishing (in the "Principle of Least Astonishment" sense.)

@Julian-O The EventManager is documented here: https://kivy.org/doc/stable/api-kivy.eventmanager.html#module-kivy.eventmanager
My intent is that it is a proof of concept so that users can create their own.

Understood. I vote make it an example, and add a warning to Widget's docstring.

@Julian-O Update:  The raw mouse events are not propagated thorough the EventManager system.  What I thought would be a clever fix, will not work.  I have not yet given up...


Cargo.toml:40-42: Find way to spread clippy config We...

The puzzle 74-414ed44d from #74 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/d4e60e8b6bb1e33603dfd42353931d6a198ff0d9/server/Cargo.toml#L40-L42
The puzzle was created by @h1alexbel on 24-Jul-24.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thanks for detailed and disciplined report.

I see you're talking to me, but I can't reply since I'm not a chat bot.

The puzzle 74-414ed44d has disappeared from the source code, that's why I closed this issue.


Allow any Texture type to be assigned as NextIndicator icon

Allow .tres assignment to NextIndicator so things like AnimatedTexture can be used and any other texture time.
Also made the TextureRect conditional checking more clear and consistent too.

Thanks. LGTM!


UW MAIZSIM

Here is a collection of changes cherry-picked from what we've put into our version of MAIZSIM since last merge happened years ago.

~Leaf elongation fix~ (already merged)
Stomatal conductance default change
LightEnv replacement
macOS update (Intel Fortran 2020 on Xcode 10.3)
Linux compatibility fix

Note that some soil files had to be touched since they caused a problem when Jennifer was trying to set it up on Linux machine where file names are case-sensitive.
I didn't have chance to go through testing on Windows/Linux myself yet, so we may likely expect some more commits later.

I had to replace a tab in that line in MNGMB.FOR with two space since that caused error on the next line where GNU Fortran wanted comment symbol ! properly aligned with the previous line. Maybe should have replaced the tab with more space characters to get proper indentation. I missed it was in DO loop.


The Lieutenant never terminate, this will cause problems when running on a cluster.

This will cause problems on clusters in the sense that your job will not finish when it's done but always be killed at the end of your walltime.
One solution would be to modify the Soldiers so that it always send their PID with the request and modify the handle_control function in the Lieutenant so it takes another argument (workerID).
This way then the user implement a Lieutenant they will have the opportunity to keep track of the worker and shutdown when there is no more active worker.
We should also modify the LSTM example to consider this.

Addressed by #18


Test ambient enroll/unenroll dynamically

Fixes https://github.com/istio/istio/issues/48386
Depends on https://github.com/istio/istio/pull/53063
Relates to https://github.com/istio/istio/issues/53064

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
c7e349567b4031d902deef9476fe90b1b99e44e4
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


OMS link on DevOps checklist redirects to page not reflecting the links intention

Hi

Microsoft Operation Management Suite also provides centralized monitoring and management for cloud or hybrid solutions.

Links to https://www.microsoft.com/cloud-platform/operations-management-suite which redirects to https://azure.microsoft.com/en-gb/services/virtual-machines/secure-well-managed-iaas/, I don't think the new content at the link reflects what was intended from the original link.
Possibly it should link here instead: https://azure.microsoft.com/en-gb/services/monitor/ and be renamed Azure Monitor

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

ID: ab2bdcdc-80e6-c47a-dae6-0f1bc44b765b
Version Independent ID: aec4e510-ca48-54ba-beda-82e6ac06bf11
Content: DevOps checklist - Azure Design Review Framework
Content Source: docs/checklist/dev-ops.md
Service: architecture-center
Sub-service: well-architected
GitHub Login: @dragon119
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 390619

Thanks for opening this issue. This has been updated and should publish today.


Markdown not rendering headings

Bug in the docs where Markdown is rendering headings.

This was already fixed in #1450 by @victorhaggqvist and should be part of the next release, I guess.

Closing per previous comment.


WRP-8078:Button: Add roundBorder prop making both sides of button fully round

Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[x] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
As a result of a discussion with the GUI team, it was decided to provide a boolean prop to have both the left and right sides of the button completely round. Therefore, a roundBorder boolean prop is added to achieve that.
Resolution
Use

--button-height: $height
border-radius: calc(--button-height / 2).

Use CSS custom property --button-height and LESS property as variable $height. CSS custom property --button-height is overridden wherever CSS property height is defined.
Additional Considerations
add sampler/qa-sampler and screenshot tests for roundBorder prop

sampler

add roundBorder prop in controls

qa-sampler

add roundBorder variants in each <Section>s of KitchenSink

screenshot tests

add roundBorder variants for the following

size
backgroundOpacity
icon
large text size

Links
WRP-8078
Comments
Enact-DCO-1.0-Signed-off-by: Hoeun Ryu<EMAIL_ADDRESS>

A fix-up patch has been pushed to address the comments.


class com.sun.xml.ws.spi.ProviderImpl cannot be cast to class jakarta.xml.ws.spi.Provider

Team,
Very similar to https://github.com/jakartaee/jax-ws-api/issues/90
I am getting class com.sun.xml.ws.spi.ProviderImpl cannot be cast to class jakarta.xml.ws.spi.Provider
I am using
Java 21. Spring boot 3.3
Gradle 8.7
My deps include
implementation group: 'com.sun.xml.ws', name: 'jaxws-rt', version: '2.3.1'

Kindly advise on the combination of jars to be used?
I have spent couple of days on this issue now.
Any leads are much appreciated.
Some debug screenshots

Hi @hariraogotit could you fix this problem?
How did fixed if you fixed!

@ThiarllesMaster - I got it working by adding implementation 'com.sun.xml.ws:rt:4.0.1'
Specifically https://stackoverflow.com/a/78487339/4068218 post helped me.


python312Packages.aiorwlock: 1.4.0 -> 1.5.0

Diff: https://github.com/aio-libs/aiorwlock/compare/refs/tags/v1.4.0...v1.5.0
Changelog: https://github.com/aio-libs/aiorwlock/releases/tag/v1.5.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359784

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.aiorwlock
python311Packages.aiorwlock.dist
python312Packages.aiorwlock
python312Packages.aiorwlock.dist


Does FileStreamResponse in ControllerBase gets disposed?

Trying to browse through the code but cannot find relevant place.
When returning FileStreamResult from controller, does the stream get disposed or should we make sure we call HttpContext.Response.RegisterForDispose(stream)?
thanks

@jbagga can you take a look and see what MVC does here?

To dispose of it indirectly, use a language construct such as using (in C#)

https://msdn.microsoft.com/en-us/library/system.io.stream(v=vs.110).aspx#Remarks
The using block on the FileResultExecutorBase disposes of the stream.

@jbagga ah, so it does get disposed? I guess we can close the issue?

Yeah we can! @codefactorydevelopment let us know if you have any further questions.


[LPTOPCI-657] Add Option to Skip Reporting of Subsequent Failing Steps if a Previous Step Fails That Those Steps Depend On

This PR adds the optional group value to the configuration.
group
A dictionary object that is used to define the group of rules a specific rule belongs to and the priority of the rule within that group.
This value is useful for when you have one or more steps that are dependent on other steps. If multiple steps are members of the same group,
and they all fail because one of the steps failed, firewatch will look for the highest priority failure and only report on that failure.
The dictionary object should include a string value for "name" and an integer value for "priority". For example: "group": {"name": "some-group-name", "priority": 1}
Example:
In this scenario, we have three steps: step-1, step-2, and step-3. If step-1 fails, it will cause step-2 and step-3 to also fail and if step-2 fails step-3 will also fail.
Because these steps are all dependent on each other, it makes sense to group them together to avoid multiple Jira issues being created for the same failure.
[
    {"step": "step-1", "failure_type": "all", "classification": "Misc.", "jira_project": "TEST", "group": {"name": "some-group", "priority": 1}},
    {"step": "step-2", "failure_type": "all", "classification": "Misc.", "jira_project": "TEST", "group": {"name": "some-group", "priority": 2}},
    {"step": "step-3", "failure_type": "all", "classification": "Misc.", "jira_project": "TEST", "group": {"name": "some-group", "priority": 3}}
]

Using the example configuration above:

If step-1 fails causing step-2 and step-3 to fail, only the rule for step-1 will be reported because it has the highest priority.
If step-2 fails causing step-3 to fail, only the rule for step-2 will be reported because it has the highest priority.
If step-3 fails, only the rule for step-3 will be reported.

/verified

/test image-images

/verified

/lgtm


Speed test data usage.

Hey everyone, didn't see a label for discussion but this isn't really an issue/problem.
I'm just wondering how much data these speedtests usually use.
Asking because I've used over 75% (975GB) of my data limit (1.2TB) in 10 days!!
My speedtest-tracker setup is choose the best server and run a test every 10 minutes.
My results come back usually around 400Mbps.
I've had it set up for a couple of weeks so I'm wondering if that's what used up all that data!
Thanks for any convo/discussion!

The app uses ookla's official speedtest-cli client to run tests. With default settings, it should use around 1GB of data for download test and 1GB for upload test.
I wouldn't recommend to let speedtest-cli find the best server, try to find the best servers for your location and stick to the most reliabe servers (I use 3 servers and specify their IDs in the options).
Also, I run tests every 1h.

OK by my calculations, I too was using 1GB per download test!! My uploads is much smaller though.
Wow I was using 6+ GB per hour for the last couple of weeks!! :<
Ha oh well... I love the app, just gotta calm down on it ha

Yeah speedtest cli is pretty data heavy, unfortunately a product of their services


Fix Event Links

Jesse found why event links didn't work

Nothing like a good old fashion Friday night pull request!


pull request for e2e test at 11/20/2016 05:56:07

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix drop events between two logstash instances

This PR introduce the bulk send feature of the ruby client and add a
local buffer inside this plugin. Theses changes make the plugin closer
to the LSF behavior.

@jsvd Can you test this? you need latest version of https://github.com/elastic/ruby-lumberjack/pull/9

@ph for clarification, this PR requires https://github.com/elastic/ruby-lumberjack/pull/9 be released first, right?

@jordansissel yes, https://github.com/elastic/ruby-lumberjack/pull/9 add support for bulk send which this PR need

code review done

@jordansissel Can I get a LGTM for this one? I'll squash after.

@ph will review shortly <3

Code + Test code looks good, will test shortly.

Can't build the gem:
ERROR:  While executing gem ... (Gem::InvalidSpecificationException)
    duplicate dependency on stud (>= 0, development), (>= 0) use:
    add_runtime_dependency 'stud', '>= 0', '>= 0'

Confirmed tests passing for me

Tested perf is 2x (eyeballing it with pv -War) with this patch (3k/sec -> 6k/sec). We can probably improve further but this is a lovely start :)


Answer dependencies for dynamic options

What is the context of this PR?
This PR clears any dynamic answers that have a dependency on another question.
How to review
Using test_dynamic_radio_options_from_checkbox and test_dynamic_answer_options_function_driven_with_static_options answer the first 2 questions (2nd one being generated from dynamic_options) and then go back and change the 1st question, check that the answer(s) for the dynamic_options have been removed from the store.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

If I change the first answer to a single checkbox, shouldn't it skip the subsequent ones because there's no point choosing from a list of one? It does take me through those with just one answer.

If I change the first answer to a single checkbox, shouldn't it skip the subsequent ones because there's no point choosing from a list of one? It does take me through those with just one answer.

Just for visibility, single answer shouldn't happen for real, as they would be routed around based on the count of checkboxes selected. It just hasn't been done for the test schema, that's all.


Items appearing in item list, but not capable of being searched for. (difficult to reproduce)

Hi,
I have this storage system which previously worked fine, however, for some unknown reason, it is now broken.
There are some items where even if I know they are in the system (as they are displayed when not searching), if I attempt to search for their name, I cannot find them.
Here is a demonstration video: https://youtu.be/W_rdta_dnfk
Also, this bug is rather annoying as I cannot reproduce it outside of my specific storage system. If I place an inventory connector beside some chests for testing, then when searching using the crafting terminal, I'm able to find all the items. It might be positional ? idk.
Also, there is nothing in the logs indicating why such a thing would be occurring.
Here are my logs (also contains mods list): https://mclo.gs/QY5BrwS
version: 1.6.5
mc version: 1.20.1
other mods: see logs
mod loader: fabric

Update: I have since restarted my game, and the bug is no longer occurring and I cannot reproduce. Leaving this issue open, as it is still a bug.
Last night I had tried relaunching the game several times, but it did not work. I'm not sure what has changed.

This is happening to my personal storage system. I have 64 nether chests from the nether chested mod connected directly to my storage system and a bunch of satellite farms connected in with inventory cable connectors. That system has this search bug, but I also have a separate storage system that's only connected to the 8 or so farm chests for my friends to use and that doesn't have the bug at all. mc version 1.19.2 version 1.5.9

I identified the issue previously, but forgor to update this issue:
if you have a mod that's called smth along the lines of "lazy language loader" or "language reloaded", then it may be lazy loading the lang files, which causes this issue.


Create a live demo site

Here is one I made but I won't be updating it and make no promises it will continue to exist. username and password are demo
http://bsc-wagtail-demo.herokuapp.com/
If someone wants access to the heroku app to maintain it let me know.

This is currently being addressed in https://github.com/torchbox/wagtaildemo/pull/44.
I'd like to create a little app that launches instances of wagtaildemo on a Docker server so people can play around with it without needing to sign up for Heroku. That's something I'll probably have a go at on a weekend sometime though.

I think the same, is a good idea.

http://demo.wagtail.io

:+1:


Include ballerina tool commands

Purpose

Include ballerina version update notice
Include ballerina self-update command
Improve existing command codes

Codecov Report

Merging #18377 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18377   +/-   ##
=======================================
  Coverage   14.86%   14.86%
=======================================
  Files          48       48
  Lines        1265     1265
  Branches      201      201
=======================================
  Hits          188      188
  Misses       1063     1063
  Partials       14       14

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 6f60b73...c452db0. Read the comment docs.


Batch 1 Story 3 Travel 2

Vertical - Travel
Design - https://www.figma.com/file/majSgyyvvfncHgLZHFKy8e/Stories---Template-Batch-1?node-id=7423%3A5946
Asset Folder - https://drive.google.com/drive/folders/16Y5cDdI9ixFyKj7pxiKLrAwoHMQgy9R7

@divyarajmasani can you commit these 2 stickers for me please
https://drive.google.com/drive/folders/1ixsBiN53-f0HYiZTbENB-eEnUPwL1yeC

@ganeshkerkar , I've committed the stickers. let me know if you don't see it on the dev site.


Development

Revisited quickstart.
Automated link checks.
Doctests for all module docstrings.

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

Project coverage is 97.24%. Comparing base (6dcd891) to head (57ed379).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage   97.24%   97.24%
=======================================
  Files          20       20
  Lines        1815     1815
=======================================
  Hits         1765     1765
  Misses         50       50

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1103 ms

Resolved: Bofrost.lu performance has improved in 3c69934.


[READ] Before reporting about your issue...

Hi, thank you for using the cordova-googlemaps-plugin.
If you come here, you may have some issues.
I would like to help you to solve your issues and improve this plugin as much as possible.
But in order to that, I need also your help as much as possible.
take a screen capture or video record
Please take a screen capture or take a video to describe your issue.
I need to know what is wrong.
For example, crash your app, please take a video when your app crash.
save the logs
If you are familiar with Android Studio or Xcode, please save the error logs, and provide it.
The 80% of issues, I can understand what happens when I see the logs.
Please provide all logs, not a few lines.
provide your app code
The most important thing of debugging is reproduce your issue.
In order to that, please share your whole project files.
This is the fastest and valuable resource for debugging.
I know you don't want to publish your source code.
So that's why please share your code using Bitbucket.
Bitbucket provides private git repository service as free.
You can share your project files with only me.
If you are not familiar with git repository, please watch this video.
https://www.youtube.com/watch?v=BtEvnE79jxY
Need NDA? I can do it.
Sometimes, your company might prohibit that except contracting NDA.
If your company require NDA, ask me. I can agree it.
Basically, I'm not interested in your app or source code. So I don't share your source code with another person.
Don't send your code as zip file or text file.
Please provide your project files.

Thank you for your help.

Hey man, you can create templates for issues and pull requests. This can guide users in a more concisely way than this issue.
See: https://github.com/blog/2111-issue-and-pull-request-templates

Thank you for letting me know. I will read it!

This page has been moved to https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/articles/before_post_your_issue/README.md


Change Jython to include the word Python to make it easier to find

Not all users who are familiar with Python know that Jython is the name for the version that runs in the JVM.  We should change the name in the UI to make it easier for people to identify.

I'm going to use the wording from July 29 version that I created in b296c29937b20bf3cf7f12b25fc94284615a11d1as you can see in the history above.  I don't see the point of leaving Google Refine references lying around.


GENOMEWIDE.READ.COUNT does not equal the sum of REGION.READ.COUNT

Hello Bryce and Graham,
I notice that the sum of REGION.READ.COUNT is less than GENOMEWIDE.READ.COUNT. Shouldn't they be equal? If not, why? Sorry if I am missing something obvious!
From reading extract_haplotype_read_counts.py, GENOMEWIDE.READ.COUNT is obtained directly from the h5 file, so naively I would assume that some of  the reads does not overlap linked regions, therefore GENOMEWIDE.READ.COUNT is greater than the sum of REGION.READ.COUNT.
More importantly, if these two quantities are not equal, how should I calculate GENOMEWIDE.READ.COUNT after permuting REGION.READ.COUNTS (for test calibration)? Thanks!

Hi Bosh,
You are correct that the GENOME.WIDE.READ.COUNT is the total number of genome-wide filtered mapped reads, not simply the number of reads in the linked regions being tested. This is intentional, and the value is used as a scaling coefficient for each individual's read counts.
The way we do the shuffling is to shuffle the genotypes across individuals while preserving the counts. This way the region read counts are kept appropriately paired with the total read counts. The combined_test.py script has a --shuffle option that will do this for you.
Graham


scripts: gen_syscalls: Use explicit encoding when reading headers

In Python, if open() doesn't specify "encoding" parameter,
locale.getpreferredencoding(False) will be used as the default,
as explained in
https://docs.python.org/3/library/functions.html#open ,
which may differ from system to system. So, explicitly specify
"encoding" param in open() call.
Also, fix a typo in a comment.
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

Fix for https://github.com/zephyrproject-rtos/zephyr/issues/4131.
I contemplated an idea to make more changes like this, and grepped thru scripts/*.py. What stopped me is that everything else seems to run in more varying context. Take for example DTS parsing code. One may imagine that a Windows user may write an override file, and then if they put non-ascii chars there, it may very well be not UTF-8, but single-byte encoding. So, I decided to refrain from that, and fix the script which causes an issue, which is designed to read in-tree header files which are known to be utf-8.


Models refactoring, add stella, BGE, Linq

I created the SentenceTransformerWrapper class to remove code duplication and make it easier to add new models. I also added the task_type parameter to the model, as it is needed for some instructions.
Added models Stella 400M, 1.5B, Linq-Embed-Mistral and bge-multilingual-gemma2.

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

Adding datasets checklist

Reason for dataset addition: ...

[ ] I have run the following models on the task (adding the results to the pr). These can be run using the mteb -m {model_name} -t {task_name} command.

[ ] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[ ] intfloat/multilingual-e5-small

[ ] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[ ] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[ ] I have filled out the metadata object in the dataset file (find documentation on it here).
[ ] Run tests locally to make sure nothing is broken using make test.
[ ] Run the formatter to format the code using make lint.

Adding a model checklist

[x] I have filled out the ModelMeta object to the extent possible
[x] I have ensured that my model can be loaded using

[x] mteb.get_model(model_name, revision_id) and
[x] mteb.get_model_meta(model_name, revision_id)

[x] I have tested the implementation works on a representative set of tasks.

I think that this PR very similar to https://github.com/embeddings-benchmark/mteb/pull/1221

Thanks @AlexeyVatolin, sorry for the slow reply on this - as @Samoed says it is very similar to #1221 (though I think they are compatible). Had hoped to implement some of these changes in SentenceTransformers. Will finish off #1221 as most of the debates are already there, but I will get back to this PR after

Will close this PR for now. Seems like it has gone stale. Will be very happy to review an updated version of it if needed.


Merge Two Binary Search Tree

Hey!! I'm solving this problem with c++. Can you Assign this problem to me. It is Leetcode Hard Problem.

sure go for it😄

Hii, I want to do this problem in CPP. Can you assign this issue to me?

Hii, I want to solve this problem in c++. Can you assign this issue to me?

Sure @manvika-10 go for it✨

Hi! I want to solve this problem in python. Can you please assign me this?

Sure go for it @ShrijanaSapkota

can i solve this?

Sure go for it @adithya27082002

i also want to contribute here

Sure go for it 👍

hello , I want to contribute
can you assign me this problem?

Yeah go for it!!


Add ability to run control in MACS2

Currently macs2 is running with only the sample input; controls are not being used
Ask: add the ability run controls with peaks
To do:

[ ] Add param to allow for the addition of the control

feature branch for macs2 control addition:

documentation about control here: https://hbctraining.github.io/Intro-to-ChIPseq/lessons/05_peak_calling_macs.html

complete with PR https://github.com/CCBR/CARLISLE/pull/34


GetAxieBriefList auction value?

Hey, thanks for the repository! I'm trying to use the GetAxieBriefList endpoint using python3 requests library. I do receive the list with the specified search criteria but in the auction key field the value is always 'Null'. I was able to generate the authentication token and attach it to the header but nothing seems to change. Am I missing something?

my mistake, On the request payload I was replacing <> as a string 'null', but I've just realized you are supposed to use python <> keyword. This solved the issue.


FakeMetaCommand::registerClassAutoloadExceptions() not compatible anymore

Describe the bug
Running psalm causes following error:
❯ vendor/bin/psalm
Scanning files...
PHP Fatal error:  Declaration of Psalm\LaravelPlugin\FakeMetaCommand::registerClassAutoloadExceptions() must be compatible with Barryvdh\LaravelIdeHelper\Console\MetaCommand::registerClassAutoloadExceptions(): callable in /Projects/MY_PROJECT/vendor/psalm/plugin-laravel/src/FakeMetaCommand.php on line 11

   Symfony\Component\ErrorHandler\Error\FatalError

  Declaration of Psalm\LaravelPlugin\FakeMetaCommand::registerClassAutoloadExceptions() must be compatible with Barryvdh\LaravelIdeHelper\Console\MetaCommand::registerClassAutoloadExceptions(): callable

  at vendor/psalm/plugin-laravel/src/FakeMetaCommand.php:11
      7▕ {
      8▕     /**
      9▕      * @return void
     10▕      */
  ➜  11▕     protected function registerClassAutoloadExceptions()
     12▕     {
     13▕         // by default, the ide-helper throws exceptions when it cannot find a class. However it does not unregister that
     14▕         // autoloader when it is done, and we certainly do not want to throw exceptions when we are simply checking if
     15▕         // a certain class exists. We are instead changing this to be a noop.

   Whoops\Exception\ErrorException

  Declaration of Psalm\LaravelPlugin\FakeMetaCommand::registerClassAutoloadExceptions() must be compatible with Barryvdh\LaravelIdeHelper\Console\MetaCommand::registerClassAutoloadExceptions(): callable

  at vendor/psalm/plugin-laravel/src/FakeMetaCommand.php:11
      7▕ {
      8▕     /**
      9▕      * @return void
     10▕      */
  ➜  11▕     protected function registerClassAutoloadExceptions()
     12▕     {
     13▕         // by default, the ide-helper throws exceptions when it cannot find a class. However it does not unregister that
     14▕         // autoloader when it is done, and we certainly do not want to throw exceptions when we are simply checking if
     15▕         // a certain class exists. We are instead changing this to be a noop.

      +1 vendor frames
  2   [internal]:0
      Whoops\Run::handleShutdown()

Impacted Versions
barryvdh/laravel-ide-helper           v2.10.0   Laravel IDE Helper, generates correct PHPDocs for all Facade classes, to improve auto-completion.
fruitcake/laravel-cors                v2.0.4    Adds CORS (Cross-Origin Resource Sharing) headers support in your Laravel application
laravel/framework                     v8.49.2   The Laravel Framework.
laravel/sail                          v1.8.4    Docker files for running a basic Laravel application.
laravel/tinker                        v2.6.1    Powerful REPL for the Laravel framework.
psalm/plugin-laravel                  v1.4.6    A Laravel plugin for Psalm
spatie/laravel-ray                    1.23.0    Easily debug Laravel apps
vimeo/psalm                           4.8.1     A static analysis tool for finding errors in PHP applications

Additional context
Apparently the signature of the parent class changed and now returns a callable

2.12.0 of ide-helper is not supported due to a breaking change on their end. you need to downgrade it https://github.com/psalm/psalm-plugin-laravel/blob/master/composer.json#L19

@mr-feek Is there an issue in one project or the other for resolving the break and supporting future versions?


Fix typos

Found via codespell -q 3 .

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:
@crbelaus Let me know if you like to add typos check as job in GitHub Actions.

That would be awesome @kianmeng !


Align tag between section

Hi,
Have you plan to align the tags between sections
Example
in the section 9 the tags are:

level1-domaincontroller
level1-memberserver

in the section 1 the tags are

level1
level2

I guess you did it for Windows 2016
Thanks and for good work 👍

Hey @Julieeeen,
I just wanted to give you the heads up I'm working on getting the tags synced up, I didn't realize they were like that in the rest of the role. They really should be the level1-domaincontroller/memberserver so I'm going through now and making the adjustments.
Sorry it took a minute but we have been working a bunch of updates to the roles. I should have a release at some point this week that will sync the tags and also address your other issue you raised. Keep an eye for these closing and the new release!
Also thank you for raising the issues, it really makes these so much better.
George


buffer-aggregate.qmd is not reproducible

I'm getting this error when trying to compile it:
quarto preview /home/robin/github/nptscot/networkmerge/buffer-aggregate.qmd --no-browser --no-watch-inputs

Starting python3 kernel...Done

Executing 'buffer-aggregate.ipynb'
  Cell 1/8...Done
  Cell 2/8...Done
  Cell 3/8...Done
  Cell 4/8...Done
  Cell 5/8...Done
  Cell 6/8...ERROR:

An error occurred while executing the following cell:
------------------
gdf_output = gpd.read_file("data/gdf_road_simplified_updated.geojson")
gdf_output.head()
gdf_output[['value_sum']].mean()
gdf_output_projected = gdf_output.to_crs('EPSG:27700')
# calculate the length of each segment:
gdf_output_projected['length'] = gdf_output_projected['geometry'].length
------------------

---------------------------------------------------------------------------
CPLE_OpenFailedError                      Traceback (most recent call last)
File fiona/ogrext.pyx:136, in fiona.ogrext.gdal_open_vector()

File fiona/_err.pyx:291, in fiona._err.exc_wrap_pointer()

CPLE_OpenFailedError: data/gdf_road_simplified_updated.geojson: No such file or directory

During handling of the above exception, another exception occurred:

DriverError                               Traceback (most recent call last)
Cell In[4], line 1
----> 1 gdf_output = gpd.read_file("data/gdf_road_simplified_updated.geojson")
      2 gdf_output.head()
      3 gdf_output[['value_sum']].mean()

File ~/.local/lib/python3.10/site-packages/geopandas/io/file.py:259, in _read_file(filename, bbox, mask, rows, engine, **kwargs)
    256     path_or_bytes = filename
    258 if engine == "fiona":
--> 259     return _read_file_fiona(
    260         path_or_bytes, from_bytes, bbox=bbox, mask=mask, rows=rows, **kwargs
    261     )
    262 elif engine == "pyogrio":
    263     return _read_file_pyogrio(
    264         path_or_bytes, bbox=bbox, mask=mask, rows=rows, **kwargs
    265     )

File ~/.local/lib/python3.10/site-packages/geopandas/io/file.py:303, in _read_file_fiona(path_or_bytes, from_bytes, bbox, mask, rows, where, **kwargs)
    300     reader = fiona.open
    302 with fiona_env():
--> 303     with reader(path_or_bytes, **kwargs) as features:
    304         crs = features.crs_wkt
    305         # attempt to get EPSG code

File ~/.local/lib/python3.10/site-packages/fiona/env.py:457, in ensure_env_with_credentials.<locals>.wrapper(*args, **kwds)
    454     session = DummySession()
    456 with env_ctor(session=session):
--> 457     return f(*args, **kwds)

File ~/.local/lib/python3.10/site-packages/fiona/__init__.py:336, in open(fp, mode, driver, schema, crs, encoding, layer, vfs, enabled_drivers, crs_wkt, allow_unsupported_drivers, **kwargs)
    333     path = parse_path(fp)
    335 if mode in ("a", "r"):
--> 336     colxn = Collection(
    337         path,
    338         mode,
    339         driver=driver,
    340         encoding=encoding,
    341         layer=layer,
    342         enabled_drivers=enabled_drivers,
    343         allow_unsupported_drivers=allow_unsupported_drivers,
    344         **kwargs
    345     )
    346 elif mode == "w":
    347     colxn = Collection(
    348         path,
    349         mode,
   (...)
    358         **kwargs
    359     )

File ~/.local/lib/python3.10/site-packages/fiona/collection.py:243, in Collection.__init__(self, path, mode, driver, schema, crs, encoding, layer, vsi, archive, enabled_drivers, crs_wkt, ignore_fields, ignore_geometry, include_fields, wkt_version, allow_unsupported_drivers, **kwargs)
    241 if self.mode == "r":
    242     self.session = Session()
--> 243     self.session.start(self, **kwargs)
    244 elif self.mode in ("a", "w"):
    245     self.session = WritingSession()

File fiona/ogrext.pyx:588, in fiona.ogrext.Session.start()

File fiona/ogrext.pyx:143, in fiona.ogrext.gdal_open_vector()

DriverError: data/gdf_road_simplified_updated.geojson: No such file or directory
DriverError: data/gdf_road_simplified_updated.geojson: No such file or directory

Can anyone else reproduce this?

Hi Robin,
I was trying to run the code, however, the code was added to use Reticulate for using Python alongside R is disappeared.
I tried to add lines:
library(reticulate)
use_python(.Renviron)

but not working, can you help me to fix this?

What is the error message you see?
Regarding .Renviron, this is not correct:
use_python(.Renviron)

As documented in https://github.com/nptscot/networkmerge/issues/33 after adding the .Renviron file with the correct installation path, it should work. You do not need to explicitly read-in the file.

Hi Robin,
I have saved gdf_road_simplified_updated.geojson to data folder, now the issue is solved.

Great job @wangzhao0217. Note: you can close issues with commit messages, e.g. as follows:
git add data/ # add the new file
git commit -m 'Add new data, close #38' # close the issue after pushing the change
git push


feat: add datadog_trace 1.5.0 release

update datadog_trace with release 1.5.0

:tada: This PR is included in version 1.400.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Markdown Previewer App

Requirements for the app:-

User can enter Github flavored markdown into a text area
User can see the resulting HTML in another container/box by pressing on a button

Requirements for the README.md file:-

Concepts used
Documentation and resources used
Future Aspects of the app

Note:- Add the app to the Android Projects section

Hello! I would like to take this issue :)

@Pinkolik You can start with this project.

Ok. Thanks I have started working on this.


[Help][Documentation Request] In class InvalidScoreLogitsProcessor, what's the meaning of tonken_id 5?

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
My model constantly rolling out token id 5, but I can't find any explanation of what token id 5 means.
And btw, what does image token mean? I can't find doc anywhere.
Expected Behavior
No response
Steps To Reproduce

go to file modeling_chatglm.py
at top of the file
you will find the class there.

Environment
Not relevant.

Anything else?
No response

Oh! The image token will be used for multi-modal I guess!!

It is the space token. We use the space token when the output contains nan value.

Thank you.


MVP-300: Automated type docs

🎯 Summary

MVP-300

Scripts that run on push of source code. If the typescript types change, they will be updated where they are referenced in the docs.
Scripts can be run locally to test. Change the content of the typescript code blocks and run the script to watch it update them.
🔰 Checklist

[x] I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.

I have performed a self-review of my code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation where required.
I have tested my changes to the best of my ability.
I have consulted with the team if introducing a new dependency.
My changes generate no new warnings.

hitting the following error when trying to run the script locally:
Error reading the declaration file: [Error: ENOENT: no such file or directory, open 'build/bundle.d.ts'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: 'build/bundle.d.ts'
}

Possibly a mac vs. pc pathing issue

hitting the following error when trying to run the script locally:
Error reading the declaration file: [Error: ENOENT: no such file or directory, open 'build/bundle.d.ts'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: 'build/bundle.d.ts'
}

Possibly a mac vs. pc pathing issue

Do you have the build directory? If not run npm run build


Split stopify/stopify-continuations into separate compiler/runtime packages

For browser-consumption, we webpack runtime/compiler package entrypoints to reduce the bloat of including stopify in a webpage. In the serverless context, people typically install npm dependencies and deploy a zip of all of node_modules and the function's source code.
Right now, installing stopify or stopify-continuations as a package dependency pulls in all of babel and it's many packages. Programs that consume just the runtime part of stopify and stopify-continuations therefore pull in over 10 MB of unnecessary dependencies (assuming the runtime doesn't need the compiler, which isn't the case if you want to support eval).
I propose we separate the compiler and runtime portions of stopify and stopify-continuations into distinct yarn workspaces, so we can deploy and install them as separate npm packages. This way consumers of just the runtime portion don't suffer large bloat in node_modules, causing slow deploy times/container init times in serverless contexts.
I'm happy to do this work. I already have some local work fixing dependencies in the various package.json files for each workspace.
Open to feedback. @arjunguha , is this going to conflict with any of the things you're doing?

Sounds reasonable.
On Thu, Aug 2, 2018 at 10:12 Sam Baxter<EMAIL_ADDRESS>wrote:

For browser-consumption, we webpack runtime/compiler package entrypoints
to reduce the bloat of including stopify in a webpage. In the serverless
context, people typically install npm dependencies and deploy a zip of all
of node_modules and the function's source code.
Right now, installing stopify or stopify-continuations as a package
dependency pulls in all of babel and it's many packages. Programs that
consume just the runtime part of stopify and stopify-continuations
therefore pull in over 10 MB of unnecessary dependencies (assuming the
runtime doesn't need the compiler, which isn't the case if you want to
support eval).
I propose we separate the compiler and runtime portions of stopify and
stopify-continuations into distinct yarn workspaces, so we can deploy and
install them as separate npm packages. This way consumers of just the
runtime portion don't suffer large bloat in node_modules, causing slow
deploy times/container init times in serverless contexts.
I'm happy to do this work. I already have some local work fixing
dependencies in the various package.json files for each workspace.
Open to feedback. @arjunguha https://github.com/arjunguha , is this
going to conflict with any of the things you're doing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plasma-umass/Stopify/issues/394, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOYcd2c6g8CzwWCpH8aYtdG6t5IFz3ks5uMwi3gaJpZM4VsXbn
.

likely not. Other things are priority before leaving, but it will be one of the first things I work on after picking it back up.

OK. After you return, I want to discuss merging this branch too:
https://github.com/plasma-umass/Stopify/commits/220
It involves changes to stopify-continuations that may make sense to merge
before your runtime surgery. I am definitely not going to merge anything
until classes start.
On Thu, Aug 2, 2018 at 11:03 AM Sam Baxter<EMAIL_ADDRESS>wrote:

likely not. Other things are priority before leaving, but it will be one
of the first things I work on after picking it back up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plasma-umass/Stopify/issues/394#issuecomment-409957844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOYWPpOMbRnHwlwlBHqg-5iubrnmSkks5uMxTNgaJpZM4VsXbn
.

Done


Cleanup header includes

The main issue addressed by this PR are circular dependencies in header files. Many of them include umrella headers (<soter/soter.h>, <themis/themis.h>) which creates a circular dependency in other headers included from umrella headers. This prevents individual headers from compiled and analyzed separate by the tools. It also introduces hidden dependency on the inclusion order. To resolve issue avoid using umbrella headers in library code: include specific headers where necessary. Umbrella headers should be used only for the user code (and tests).
While we're here, do some cosmetic improvement as well:

unify include guard style to be <PROJECT>_<HEADER>_H everywhere
sort included headers alphabetically
group headers in the meaningful groups
use proper relative paths in headers
consistently use quotes "soter/soter_rsa_key.h" for internal inclusions
consistently use angle brackets <soter/soter.h> for public inclusions

These changes are inspired and rationalized by Google C++ Style Guide.

Please do not change header structure! We spent a lot of time to make
sure that some wrappers (like iOS one) export correct header structure
(especially in ObjC++). Moreover I don’t see any real benefits from
changing folders layout except “for the sake of style” one — we don’t
expect many users to use core library as is, so beauty won’t be noticed,
but compatibility problems might arise.

Please do not merge this PR until I confirm it’s working for iOS —
@ilammy ping me when you’re done, I’ll fetch branch to check.

On Wed, Feb 27, 2019 at 00:15 Alexei Lozovsky<EMAIL_ADDRESS>wrote:

@ilammy commented on this pull request.
In src/soter/boringssl/soter_ecdsa_common.h
https://github.com/cossacklabs/themis/pull/392#discussion_r260512463:

#include <soter/soter_ec_key.h>
-#include "soter_engine.h"
+#include <soter/soter_error.h>
+#include <soter/boringssl/soter_engine.h>

Mmm... I think you're right, it's not public. I initially assumed all
headers to be public and thus use only angled brackets there (and quotes in
*.c files). I'll review the headers and will tweak the quotes.
Now when I think of it... It's kinda hard to distinguish public headers
from private ones. I've seen quite a few libraries put their public headers
into a separate top-level directory include. For example:
include
themis
secure_cell.h
themis.h
...
src
themis
message.h
portable_endian.h
secure_session_utils.h
...
How do you feel about moving to this layout in the next PR?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/cossacklabs/themis/pull/392#discussion_r260512463,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACvp4NXx1aO-n2n4nr148l-zA90_r7Z6ks5vRbIBgaJpZM4bSlfa
.

@vixentael, understood. Thank you for the warning.

@vixentael you can try checking the iOS wrapper now.
I've managed to see the tests pass locally and it seems to have used the correct source code for that, but I'm not really sure. Please check.
Now that I've interacted with iOS wrapper a bit I completely agree with you to not touch that fine-tuned header magic unless absolutely necessary. Though, it seems that we export too many headers for iOS, but I'm no expert, maybe we really need to do that because of reasons.

@ilammy i've checked iOS wrapper – it's working fine. If @Lagovas doesn't have any more comments – let's merge!
The reasoning behind my warning (lots of details inside)
It's not a magic of any kind, it's more a fine-tuned Header Search Paths.
In podspec we precisely define what Themis core files and iOS wrapper files should be exported, what path should be preserved and what are public headers:
# don't use as independent target
so.subspec 'core' do |ss|
    ss.source_files = "src/themis/*.{h,c}", "src/soter/*.{c,h}", "src/soter/ed25519/*.{c,h}", "src/soter/openssl/*.{c,h}"
    ss.header_mappings_dir = "src"
    ss.header_dir = 'src'
    ss.preserve_paths = "src/themis/*.h", "src/soter/*.h", "src/soter/ed25519/*.h", "src/soter/openssl/*.h"
    ss.public_header_files = "src/themis/*.h", "src/soter/*.h", "src/soter/ed25519/*.h", "src/soter/openssl/*.h"
end

# don't use as independent target
so.subspec 'objcwrapper' do |ss|
    ss.header_mappings_dir = 'src/wrappers/themis/Obj-C/objcthemis'
    ss.source_files = "src/wrappers/themis/Obj-C/objcthemis/*.{m,h}"
    ss.public_header_files = 'src/wrappers/themis/Obj-C/objcthemis/*.h'
    ss.header_dir = 'objcthemis'
    ss.dependency 'themis/themis-openssl/core'
end

This is how these headers in Themis pod.xcconfig looks like:
HEADER_SEARCH_PATHS = $(inherited) "${PODS_ROOT}/Headers/Private" "${PODS_ROOT}/Headers/Private/themis" "${PODS_ROOT}/Headers/Public" "${PODS_ROOT}/Headers/Public/GRKOpenSSLFramework" "${PODS_ROOT}/Headers/Public/themis" "${PODS_ROOT}/themis/src" "${PODS_ROOT}/themis/src/wrappers/themis/Obj-C"

This is how they are transformed (notice a "flat" structure, without any folders):


Cosign fix, bundle update

Signed-off-by: jonahjon<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Don't require desktop development workload for Windows

After latest master version upgrade I have started seeing:
Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        Windows 10 SDK

It seems the check now relies on "Desktop development with C++". If it is not installed the flutter won't build even when one has all the required components (previously it built just fine), this started ocurring after recent flutter upgrade. After installing the whole Desktop development package the issue disappeared (MSVC was already installed). I don't think it's good idea to force everyone to install everything since on systems with smaller disk space it can waste precious space.
Reproduction

Install required VS components (without installing Desktop development with C++)
Run flutter doctor (or build)

Fixing title to accurately describe what's being requested; this isn't a failure to detect the toolchain component. It's correctly determining that you don't have the workload that the documentation indicates you need.

It seems the check now relies on "Desktop development with C++".

This is correct.

If it is not installed the flutter won't build even when one has all the required components (previously it built just fine)

The instructions have always said to install that workload; the difference is that we're now actually checking. "Required components" is a somewhat ambiguous term here since it could mean either what our checks require, or what would technically be enough to successfully build. By the former definition, you don't have all the required components. The distinction is important because I'm almost certain there were components that weren't checked before, but were actually needed in order to build and run (e.g., the redistributables that are needed by the runtime).

I don't think it's good idea to force everyone to install everything since on systems with smaller disk space it can waste precious space.

That workload is a small fraction of everything in the VS installer; its list of required components is quite short.
We've had more issues and questions about correctly setting up VS than probably any other single topic across all of the desktop platforms; people have made instructional videos just showing people how to find and check the correct checkboxes. The recent change was specifically made to simplify the setup process to address this longstanding issue. Micro-optimizing for size (especially given that the recommended instructions already said to do this, so in practice will not affect the resulting install size for the vast majority of users) is much less important than ease of use.
If someone wants to figure out exactly the set of necessary components and a robust, future-proof (i.e., not re-introducing the SDK version problem) approach for checking for exactly those components, and it doesn't increase the complexity of the basic instructions, I will review it, but it's not something I expect the Flutter team to ever work on. (And if someone does do that, please include an actual number for how much space is saved, so that can be weighed against the added code complexity.)

Ok from what you described I can see why it would be preferable to just simply check for the workload instead of each component specifically and taking into account future-proofness, it really would be a waste of time to work on this.
Previously I had custom setup from previous work with C++ so I never noticed the message in earlier builds since I had all the components it checked for and everything worked.
Installing the workload took roughly 1gb of extra space.

Installing the workload took roughly 1gb of extra space.

Interesting, that's much larger than I would have guessed. I wonder which components are the bulk of that.

I inspected it again and I really don't understand what could've taken so much space. I noticed that around 300-400MB can be reduced by unchecking all the non-essential optional components. But I really couldn't find what would take the extra space. It is possible that something extra was installed that I couldn't find though.
From what I could see, the workload should not contain much extra than what is needed by flutter.
After going through all this in more detail, if you agree I think this issue can be closed, since there is not much value in this and the 1gb - the non-essential optionals could have simply been a space approximation error.

Thanks for the follow-up information. Closing for now, but if someone does ever want to work on this and can find reasonable ways to reduce the footprint, we can re-open.


[docs] Fixed typo in Components/Modal

react-transitin-group is supposed to be react-transition-group

[x] I have followed (at least) the PR section of the contributing guide.

Resolves #17703

@lzhuor Thanks!


Support for caddy v2

Amazing project! I've been struggling to find an up-to-date dashboard for a while, and yours worked out of the box :)
Thanks!
Side request: maybe, if you find it useful, you can integrate caddy as a target. I would do it myself, but I can't figure it out.
Should be pretty straightforward for a dev, since caddy supports metrics with Prometheus.
https://caddyserver.com/docs/metrics
I tried it myself, but I get lost and all I get is empty tables.

Hi, thanks for your message. The purpose of this stack is to monitor the Raspberry OS and the dockers running on it. I didn't know Caddy, it doesn't seem to meet the expectations of this observability project.
Sorry but I won't spend time on it, but if you need help or if I guide you, don't hesitate.


Bump [compat] MLJBase="^0.8.1"

To bring MLJLinearModels into synch with the other repos.

done 782c1d9


Chart example using chartkick

This adds a simple way of view basic stats for a variety of models.

Adds dashboard/overview view
Adds StatsPresenter object

TODO:

[ ] Enable from/to date filtering
[ ] Figure out how to make this error resilient the current implementation is, by design, very eager to crash

Screenshot

@pbl thoughts?

@pbl just updated this PR, take a look when you have time :)


Link to support page

Rather than have a paragraph of text in the left-hand nav we should keep it simple and just link to our support page. This is also consistent with the nav in the upper menu Help > Developer-to-Developer Support

This seems like something I'd want to run by @ChristieQ in a flight.

Thanks,@rgbkrk! According to heat map data, the only items in the left nav that get any clicks at all are the page navigation (i.e., languages in the SDKs section of the site and product names in the API docs section). Based on that data, we expect that changes to this section of the page will have little to no impact. Our plan is to move all of this content in the upcoming redesign, based on the results of user testing.
TL;DR: We can make the change, but chances are good that it will have little impact on CX.

I'm really just looking for consistency in how we direct users to support.


[Issue-2799] Error parsing token balance for frozen asset on Asset Hub

Related issue(s)

#2799

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

Fix error with show frozen balance

Describe the solution you'd like

Fix error with show frozen balance

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2801--sw-web-runner.netlify.app


Rust API and index improvements

This is a major Rust implementation update:

Simplify high-level API
Extract processing API with GeoJSON and SVG implementations into separate crate https://crates.io/crates/geozero
Implement stream_search for FileReader and HttpReader
Return Error types instead of unwrap
Implement remaining property types

Detailed commits are in https://github.com/pka/flatgeobuf/commits/geozero

Looks great, I'll merge.


Govee intergration keeps breaking

The hacss govee tile keeps saying that it is unavailable, after restarting either home assistant or reloading the tile, the problem is fixed for an uncertain amount of time before breaking again. I do not believe it to be the API key because I have never had to change the API key.

Same here. The integration just stops working and need a restart of HA to get it working again.

same for me also.

same here.  even a restart doesn't work today

This is the problem.  Can we reduce the number of requests to be less than 10,000 limit ?
Logger: custom_components.govee
Source: custom_components/govee/init.py:66
Integration: Govee (documentation, issues)
First occurred: 7:17:57 AM (1 occurrences)
Last logged: 7:17:57 AM
Could not connect to Govee API: API: API-Error 429: {"message":"rate limited! the limit is 10000 requests every 24 hours. X-RateLimit-Reset in Http Response Header shows the reset time."}

Similar/same as @ecard0 issue here:
Logger: custom_components.govee
Source: custom_components/govee/init.py:66
Integration: Govee (documentation, issues)
First occurred: 5:21:48 PM (1 occurrences)
Last logged: 5:21:48 PM
Could not connect to Govee API: API: API-Error 429: {"message":"rate limited! the limit is 10000 requests every 24 hours. X-RateLimit-Reset in Http Response Header shows the reset time."}

Same here , is there a fix ??
Could not connect to Govee API: API: API-Error 429: {"message":"rate limited! the limit is 10000 requests every 24 hours. X-RateLimit-Reset in Http Response Header shows the reset time."}

deleted this integration and installed this local control https://github.com/wez/govee-lan-hass
works like a charm and supper fast

deleted this integration and installed this local control https://github.com/wez/govee-lan-hass
works like a charm and supper fast

This is a great option, thanks for sharing!

deleted this integration and installed this local control https://github.com/wez/govee-lan-hass
works like a charm and supper fast

This is great for devices that have LAN capability, but the majority of mine do not. Any other fixes/workarounds out there?

Also curious, only half of my Govee lights support LAN control.

@ebotty , @Stormster - the govee-lan-hass integration seems perfectly capable of controlling Govee lights via the API. It just prefers LAN control where available.
I just tested it with H6057 and H6047 (neither have the LAN control option right now), and they worked fine. Well, until I started getting "connection lost" messages - but my guess is that this is because of very aggressive rate limits on the Govee API (which would also explain why LAN control would be preferred :) ).


Colcon build error, is this foxy supported?

When I tried to run colcon build inside the /gazebo_ros2_control, I received error like this
bwuk@robots:~/gazebo_ros2_control$ source /opt/ros/foxy/setup.bash
bwuk@robots:~/gazebo_ros2_control$ colcon build --merge-install
Starting >>> gazebo_ros2_control
--- stderr: gazebo_ros2_control
CMake Error at CMakeLists.txt:6 (find_package):
  By not providing "Findcontroller_manager.cmake" in CMAKE_MODULE_PATH this
  project has asked CMake to find a package configuration file provided by
  "controller_manager", but CMake did not find one.

  Could not find a package configuration file provided by
  "controller_manager" with any of the following names:

    controller_managerConfig.cmake
    controller_manager-config.cmake

  Add the installation prefix of "controller_manager" to CMAKE_PREFIX_PATH or
  set "controller_manager_DIR" to a directory containing one of the above
  files.  If "controller_manager" provides a separate development package or
  SDK, be sure it has been installed.

---
Failed   <<< gazebo_ros2_control [2.13s, exited with code 1]

Summary: 0 packages finished [2.41s]
  1 package failed: gazebo_ros2_control
  1 package had stderr output: gazebo_ros2_control
  1 package not processed

Is this supported on Foxy?
What did I miss with this one?

I just realized, is there any way for me to test in local instead of docker?

Found the solution:
What I did was
go in the main directory such as gazebo_ros2_control
Then run this line
       source /opt/ros/foxy/setup.bash
        rosdep update
        rosdep install --from-paths ./ -i -y --rosdistro foxy --ignore-src
        colcon build --packages-up-to gazebo_ros2_control_demos
        colcon test --event-handlers console_direct+ --packages-select gazebo_ros2_control gazebo_ros2_control_demos

Following this from the CI
https://github.com/ros-simulation/gazebo_ros2_control/blob/master/.github/workflows/ci.yaml#L36
Closing the ticket since this solved my issue.


Portal Side of functionality is not working, the control does not render

This documentation needs an update e.g.  the CORS part requires to mention that the URL is of the CDS/Dynamics instance, and also the web resource control for adx/annotation does not render or load properly on the portal page when configured, the functionality is broken.

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

ID: 7cdd598f-2269-05e7-25e8-f014bef0ff6e
Version Independent ID: 8b9996ad-708b-f5dc-3df2-d0d7e8b1a8bf
Content: Add azure storage web resource to a form - Power Apps
Content Source: powerapps-docs/maker/portals/add-web-resource.md
Service: powerapps
GitHub Login: @tapanm-MSFT
Microsoft Alias: tapanm

Thank you for reaching out. Content is being updated. Please allow some time to reflect the changes. For issues with the functionality, please reach out with a support service request for further investigation.


Add PodDisruptionBudgets

See https://github.com/Azure/Mission-Critical-Online/pull/257 for more details.

Not tested in e2e, yet.


Add subflow meta data edit pane

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

Proposed changes
This is part of the npm subflow module work - #2690.
This is targeted against the sf-module branch that #2690 adds. I've done this to help with reviewing the discrete parts of the subflow module work.
This PR adds an edit pane to subflows to set the module meta data to the subflow.
All of these meta-data properties are stored under a meta property of the node.

There is basic validation on the module and version inputs to ensure they conform to the npm module requirements.
The License field is a typedInput where they can select from common licenses, or select the 'other' option that allows a custom entry to be added.

Is this "meta" data concept specific to the subflow nodes - or could it potentially extend to other nodes like function, and group ? Presumably this becomes a reserved property name also (to be avoided by other nodes).

As it stands, this is only related to the subflow node type - it isn't a generic property that any node could have, so no need to be reserved.
Yes other nodes may gain a meta property in the future - but their use of that property would be specific to their node type. That's distinct from other common properties that the editor uses regardless of the node type.


[BUG 🐛] division by zero error in estimate_foreground_ml

i am getting division by zero errors in  estimate_foreground_ml()
what i tried:

pymatting 1.1.1 and 1.1.3
making sure both the image and the mask are not uniform (i've seen the error when both have min_val=0 and max_val=1)
default parameters and different variations

the environment is google colab. also sometime this (or something else in pymatting) causes the colab itself to crash and disconnect.

Hi @eyaler, thank you for opening this issue! Could you kindly provide code, the image and the alpha matte that yield the division-by-zero-error? That would be very helpful. Thanks!

The only place where I could imagine that division by zero errors could occur is when dividing by the determinant here:
https://github.com/pymatting/pymatting/blob/e5b1e86338ccf36a68904a9ace3eb4ddbc7fb29a/pymatting_aot/estimate_foreground_ml.py#L137
A positive regularization value should in theory ensure that the determinant is positive: https://www.wolframalpha.com/input/?i=((a^2+%2B+d)++%28%281+-+a%29%5E2+%2B+d%29+-+%28%281+-+a%29++a%29%5E2%29+%3E+0%2C+d+%3E+0
That might not happen when the regularization parameter of estimate_fb_ml is regularization <= 0 or gradient_weight < 0, but those values are not sensible and not the default, so I doubt that this is the reason.
Either way, I agree with @tuelwer that we need a minimal reproducible example to debug this. (See test case template: https://github.com/pymatting/pymatting/blob/master/.github/ISSUE_TEMPLATE/bug_report.md)

i was running this with the defaults (reg=1e-5, weight=1).
anyways to reproduce: https://colab.research.google.com/drive/1MXfC3vlU6u2muNG0WaB1p6HeB-5q2ReL
sometimes it works, sometimes crashs, sometimes divide by zero. try a few times.

The alpha mask should be of shape (h, w) instead of (h, w, 1).
Replace
mask=imageio.imread('mask.png')[:,:,:1]/255

with
mask=imageio.imread('mask.png')[:,:,0].copy()/255

and it should work fine.
Apparently, Numba simply ignores the shape specified at https://github.com/pymatting/pymatting/blob/e5b1e86338ccf36a68904a9ace3eb4ddbc7fb29a/pymatting_aot/estimate_foreground_ml.py#L170 and computes garbage instead of throwing an error:
https://github.com/numba/numba/issues/2757
I think this issue should be fixed in the Numba library.

i can confirm this solves my issue. until numba fixes it, this could easily be solved by doing alpha.squeeze() or add a warning and mention this in the docs, instead of the current:

alpha: numpy.ndarray
    Input alpha matte shape :math:`h \\times  w \\times 1`

which is just evil...

Sorry about that. This was definitely a mistake in the documentation. Should be fixed now.
We could of course do all kinds of checks to see whether the passed arguments are correct and convert them to a more suitable format if necessary, but there is an infinite number of ways to get it wrong, so I do not think that this doable. Just a few examples:

None arguments where not allowed
incorrect domain
incorrect object type (e.g. someone might pass a pytorch or tensorflow tensor)
incorrect numpy dtype (uint8 should probably be divided by 255, but what if the matte just happens to be binary? If the data type is uint16, it might still be in range [0, 255] because some image loaders just happen to do it like that. Or maybe it is in range [0, 32767] because it was actually int16 and not uint16)
fortran or C data order (this definitely has to be solved by Numba)
incorrect shape ([:, :, :1] could be converted to [:, :], but what about [:, :, :3]? Mean or median over channels? Pick a specific channel? Convert to HSL or CIELAB color space and pick L? What if someone passes a batch of images of size 1? ([:1, :, :]) What the size of all dimensions is 1? How to tell those cases apart?)

What I am trying to say is, if we start to fix incorrect input, there will be no end to it.
The updated documentation (thanks for the tip by the way) will have to do for now.

Type hints might also be a good additional measure to issues like this. Unfortunately, NumPy does not yet support shapes, only dtype.
I'll leave these issues here so it is easier to follow the development of type hinting for shapes:

https://github.com/numpy/numpy/issues/16544
https://github.com/numpy/numpy/issues/7370

hi, I'm facing the same issue . [ZeroDivisionError: division by zero]
step 1 :
Reading image and binary mask then generating trimap:
img = cv2.imread("blend/ILSVRC2012_test_00000250.jpg",cv2.IMREAD_COLOR)
img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)
mask = cv2.imread("blend/ILSVRC2012_test_00000250.png", cv2.IMREAD_GRAYSCALE)
mask = mask.astype("uint8")
t, msk = cv2.threshold(mask, 127,255, cv2.THRESH_BINARY | cv2.THRESH_OTSU)

trimap function:
def generate_trimap(mask,eroision_iter=6,dilate_iter=4):
    mask[mask==1] = 255
    d_kernel = np.ones((3,3))
    erode  = cv2.erode(mask,d_kernel,iterations=eroision_iter)
    dilate = cv2.dilate(mask,d_kernel,iterations=dilate_iter)
    unknown1 = cv2.bitwise_xor(erode,mask)
    unknown2 = cv2.bitwise_xor(dilate,mask)
    unknowns = cv2.add(unknown1,unknown2)
    unknowns[unknowns==255]=127
    trimap = cv2.add(mask,unknowns)
    labels = trimap.copy()
    labels[trimap==127]=1 #unknown
    labels[trimap==255]=2 #foreground
    return labels

generating trimap:
result = generate_trimap(msk)
step 2:
preprocess the image & mask then fed it to pymatting library:
rr = result.copy()
rr[rr == 2] = 255
rr[rr == 1] = 127
rt = rr / 255
img1 = img.astype('float64')
alpha = pymatting.estimate_alpha_cf(img1,rt, laplacian_kwargs={"epsilon": 1e-6},cg_kwargs={"maxiter":2000})

now, estimate_alpha_cf is throwing the error of "ZeroDivisionError: division by zero"
am I missing something?

I think this is a different issue because the error happens in estimate_alpha_cf instead of estimate_foreground_ml.
Anyway, could you create a "minimal, reproducible example" with complete code and images so we can reproduce the issue?

@shamindraparui Thank you, I can reproduce the issue.
PyMatting expects that all images are in the range [0, 1], but your img is in the range [0, 255]. Simply dividing it by 255 will fix it:
img = img / 255.0

However, the results are not great because ILSVRC2012_test_00000250.jpg is JPEG-compressed a lot. The compression artifacts will confuse the algorithm.
The artifacts can be reduced a tiny bit by using more erosion, but that will remove part of the beak of the bird. It is better to use a few steps of skeletonization in this case, but the result is still not perfect.
You might be better off using the original mask you had for that image. What did you use to compute it?

from PIL import Image
import numpy as np
import matplotlib.pyplot as plt
import cv2
import pymatting
import scipy.ndimage

img = cv2.imread("blend/ILSVRC2012_test_00000250.jpg",cv2.IMREAD_COLOR)
img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)

img = img / 255.0

mask = cv2.imread("blend/ILSVRC2012_test_00000250.png", cv2.IMREAD_GRAYSCALE)

is_fg = (mask > 255 * 0.9).astype(np.uint8)
is_bg = (mask < 255 * 0.1).astype(np.uint8)

def incorrect_but_good_enough_skeletonization_step(mask):
    keep = np.ones(2**9, dtype=np.bool8)
    keep[[23, 27, 31, 54, 55, 63, 89, 91, 95, 216, 217, 219, 308, 310, 311,
        432, 436, 438, 464, 472, 473, 496, 500, 504]] = 0
    padded_mask = np.pad(mask, [(1, 1), (1, 1)], mode="edge")
    windows = np.lib.stride_tricks.sliding_window_view(padded_mask, (3, 3))
    kernel = (2**np.arange(9)).reshape(3, 3)
    return mask * keep[(windows * kernel).sum(axis=(2, 3))]

for _ in range(3):
    is_fg = incorrect_but_good_enough_skeletonization_step(is_fg)
for _ in range(5):
    is_bg = incorrect_but_good_enough_skeletonization_step(is_bg)

trimap = 0.5 + 0.5 * is_fg - 0.5 * is_bg

alpha_cf = pymatting.estimate_alpha_cf(img,trimap,laplacian_kwargs={"epsilon": 1e-2})
alpha_lkm = pymatting.estimate_alpha_lkm(img,trimap,laplacian_kwargs={"radius": 5})
alpha_knn = pymatting.estimate_alpha_knn(img,trimap)
alpha_original = mask / 255.0

def blend(alpha):
    foreground = pymatting.estimate_foreground_ml(img, alpha)
    new_background = 0.5 * np.ones_like(foreground)
    blended = pymatting.blend(foreground, new_background, alpha)
    return blended

composite_cf = blend(alpha_cf)
composite_lkm = blend(alpha_lkm)
composite_knn = blend(alpha_knn)
composite_original = blend(alpha_original)

plt.figure(figsize=(20, 10))

for i, (title, image) in enumerate([
    ("Alpha CF", alpha_cf),
    ("Alpha LKM", alpha_lkm),
    ("Alpha KNN", alpha_knn),
    ("Alpha original", alpha_original),

    ("Trimap", trimap),

    ("Composite CF", composite_cf),
    ("Composite LKM", composite_lkm),
    ("Composite KNN", composite_knn),
    ("Composite original", composite_original),
], 1):
    plt.subplot(2, 5, i)
    plt.title(title)
    plt.imshow(image, cmap='gray')
    plt.axis('off')

plt.tight_layout()
plt.savefig("plot.png")
plt.show()

@99991
Thank you for the solution. The original mask that I have shared with you, is part of DUTS dataset for Salient Object Detection.
I am trying to compute the salient part of a given image as mask. The algorithm that I am using is U2Net. It is predicting a mask (not so good) which needs to be binarized, trimapped and alpha matted for final overlaying/segmenting the original image.

@shamindraparui Nice, I am looking forward to the results! For animals, there is also https://github.com/JizhiziLi/GFM and it seems to generalize to other prominent image objects to some degree. Maybe it will be helpful for your project.
I have added some sanity checks for the input image and trimap. I think this will make it less confusing to provide the input in the expected format. I'll upload it to PyPI later. Until then, it can be installed from source.


Add aarch64-none-elf 0.1 support

This series of patches adds support for the aarch64-none-elf triple.
These patches introduce this support for the 0.1 version; but not yet for the HEAD version.
I thought it's going to be easier to first get the 0.1 version reviewed; the changes for the HEAD version should then be very similar and easy to review.
As this support should probably be considered experimental at the moment, I'm not sure if any documentation should be updated at all?

A few questions and requests:
1. Does the build succeed after each commit? I.e. shouldn't the "Fix cmake configuration issues" and "set CMAKE_SYSTEM_NAME to 'Generic'" go before "Add (experimental) aarch64-none-elf support"?

I wasn't sure if keeping each build succeeding was important in this project or not.
I'll follow your guidance here and verify builds keep on succeeding after each commit for the next version of this patch series.

2. Does this patch series break the HEAD build? If it does please disable AArch64 when building the HEAD version.

It does. Maybe the best way to fix this is for every commit that changes the .patch files (which are version-specific), to change both the the -0.1 and -HEAD versions in the same commit?

3. Please add armv8a-64 to the "Supported architectures" sections in README.md and mention that the support is experimental.

Will do, thanks for the guidance.

I wasn't sure if keeping each build succeeding was important in this project or not.

We already had at least one bug where git bisect turned out to be very helpful, so I think it is important.

It does. Maybe the best way to fix this is for every commit that changes the .patch files (which are version-specific), to change both the the -0.1 and -HEAD versions in the same commit?

That would be ideal, but if you don't want to spend time on creating patches for the HEAD version right now, disabling AArch64 for HEAD is also fine.

I've now updated the patch series, addressing all previous comments.

I wasn't sure if keeping each build succeeding was important in this project or not.

We already had at least one bug where git bisect turned out to be very helpful, so I think it is important.

I tested that both the 0.1 and HEAD version builds work correctly after each commit.

It does. Maybe the best way to fix this is for every commit that changes the .patch files (which are version-specific), to change both the the -0.1 and -HEAD versions in the same commit?

That would be ideal, but if you don't want to spend time on creating patches for the HEAD version right now, disabling AArch64 for HEAD is also fine.

Oh, and I also added aarch64, armv8.0 to the "Supported architectures" sections in README.md and mentioned that the support is experimental.


logs from spawned processes dropped

I'm using the sudospawner with great effect! One issue I'm having is that logs from the spawned processes are not making it back to the jupyterhub process. I tried piping stderr back to the parent process, but that doesn't appear to help.

Yes, there was a problem where leaving stderr open prevented the mediator from exiting properly, so it's closed. This is the main source of debugging sudospawning being a nightmare. If there's a way to leave stderr open without reintroducing the problem, that would solve a lot of problems.

Here's a hack that collects process output somewhere.


container open error message.

If occurrence exception when container make, catch error and show message
with readium-sdk/Platform/Android/jni/epub3.cpp
https://github.com/readium/readium-sdk/pull/214

Thank you for your work, but can you please target this pull request at develop and not master?  We don't normally merge PRs into master.


A better notation for limit

It is my first time opening an issue.
Here is the zulip topic
Is there a better notation for the limit in mathlib ?
like lim x → -∞ ,  (∫ (t : ℝ) in Iic x, f t) = 0 instead of  Tendsto (fun x ↦ ∫ (t : ℝ) in Iic x, f t) atBot (𝓝 0)
It would be great if someone could add this notation to mathlib.

I understand making notation for this is quite difficult due to the sheer generality. But really I'd be satisfied with basically anything using lim or →; I think it'd do wonders for readability.

At what level of generality should we try to get that notation?
In CompleteLattices we have notations for sSup and sInf limits.
I'm not sure whether there are notations for limits in CategoryTheory.
I was wondering if the notation you are proposing now is actually split up into two parts.
A part lim (x : Filter X) (f : X → Y) where X needs to be of some filter type and f is in some way continuous,
and a part where you find notation for a particular filter, like x → ∞ to replace (x : Filter X).


To be checked that the patient receives the questionnaire invitation

Participant does not receive a notification when a new questionnaire is available (but the questionnaire shows in their agenda correctly)

@cosmintuca a patient that is In Treatment should also receive the questionnaire and the updates

all good now


When left/right keyboard key strokes inside HTML text input field, content-editable etc., previous/next page should NOT be triggered (instead: text cursor move)

To reproduce the bug, use the following EPUB created by @JayPanoz (Jiminy Panoz):
https://github.com/JayPanoz/readium-2/blob/master/misc/touch-handling/touchHandling.epub
Unlike Readium-1 generic iframe-based implementation where keyboard interaction inside EPUB HTML Content Documents must be forwarded explicitly to the parent context (top-level web frame) in order for pagination previous/next actions to be triggered (despite the lack of keyboard focus inside the application context), here with Electron WebViews the top-level BrowserWindow seems to "steal" keyboard events regardless of the currently keyboard-focused web "frame" (Electron WebView are actually objects, not conventional iframes).
To be investigated further...

https://github.com/edrlab/readium-desktop/blob/develop/src/renderer/components/ReaderApp.tsx#L230
https://github.com/edrlab/readium-desktop/blob/2384c542b5a0bfb379a4274a694f18cf99fcdfd3/src/renderer/components/ReaderApp.tsx#L230

https://github.com/edrlab/r2-testapp-js/blob/develop/src/electron/renderer/index.ts#L737
https://github.com/edrlab/r2-testapp-js/blob/15ad9100d8e22cf35886e5fed78c453aad887ee6/src/electron/renderer/index.ts#L737-L751

This bug will affect any in-page BrowserWindow keyboard interaction (not just EPUB HTML documents) that utilizes left/right arrow keys, such as a horizontal slider widget in the application's UI, or even just a scroll view for horizontally overflowing text (e.g. table of contents label).

Note that the keydown event is not listened-to with useCapture, yet the Electron WebView object and text input controls within it do not seem to exhibit the preventDefault() or stopPropagation() behaviour that we normally experience with iframes (to be verified, I am just brainstorming from the top of my head here).
More general-purpose information here:
https://github.com/readium/readium-2/blob/master/interactions/dom-events.md
...and here:
https://github.com/readium/readium-2/blob/master/misc/RS-touchHandling-overview.md

To fix this, we could:

add logic on the application side to determine when to ignore the left/right arrow single-key keyboard shortcuts (still on keydown so that the event auto-repeats => fast page turn actions, or on keyup so it does not).
add logic in the "navigator" under the hood, so that keyboard events in "special" HTML constructs do not get captured by the application BrowserWindow host.
(these two are not mutually-exclusive, and a full solution might actually require combining both approaches)

TODO: debug / trace the keydown event payload in the Electron-Chromium console / web inspector, in order to assert whether there are differences depending on whether the keyboard focus is in the BrowserWindow top-level frame (app HTML DOM) or inside the Electron WebView object (EPUB HTML DOM). Then, also test when keyboard focus is inside text field input HTML element, contenteditable, etc.

Note that this is relevant in both reflowable publications (especially educational content that contains quiz inputs and other kinds of interactive material), as well as fixed-layout pre-paginated publications (typically, children's books with gamified reading activities, requiring user input).

This was fixed (see above comment)


reducing use of from form of import

Codecov Report
Base: 78.03% // Head: 78.04% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (24079b5) compared to base (5859c33).
Patch coverage: 86.12% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #624   +/-   ##
=======================================
  Coverage   78.03%   78.04%
=======================================
  Files         178      178
  Lines       30599    30602    +3
=======================================
+ Hits        23879    23883    +4
+ Misses       6720     6719    -1

Impacted Files
Coverage Δ

resqpy/grid/_pillars.py
86.74% <ø> (ø)

resqpy/grid/_pixel_maps.py
6.66% <ø> (ø)

resqpy/grid/_transmissibility.py
61.63% <ø> (ø)

resqpy/multiprocessing/multiprocessing.py
19.78% <ø> (ø)

resqpy/multiprocessing/wrappers/blocked_well_mp.py
0.00% <0.00%> (ø)

resqpy/olio/ab_toolbox.py
77.14% <ø> (-0.64%)
:arrow_down:

resqpy/olio/box_utilities.py
84.24% <ø> (-0.10%)
:arrow_down:

resqpy/olio/class_dict.py
0.00% <ø> (ø)

resqpy/olio/consolidation.py
79.20% <ø> (-0.21%)
:arrow_down:

resqpy/olio/factors.py
97.05% <ø> (-0.09%)
:arrow_down:

... and 121 more

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

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


AMAZING WORK!

I just wanted to say:
Amazing work! exactly what I was looking for! <3

@mkellerman  i may transition this to use https://github.com/keldenl/gpt-llama.cpp backend instead! stay tuned (since there hasn't been much movement around dalai)

Have you thought of being model agnostic?
Have the model available as an api, and your extension point to an endpoint?
See: https://github.com/go-skynet/llama-cli

that's exactly what gpt-llama.cpp is! i'll migrate fleece at some point to point towards openai api and make it easily work well with gpt-llama.cpp