Fix for "Electron"

Hi, angular-moment does not load in Electron "renderer" environment. It is because
module and module.exports exists in Electron environment, but the followed code does not
angularMoment(require('angular'), require('moment'));
module.exports = 'angularMoment';
code bellow works for me

Thanks for the PR!
I believe that commit 517f42f should solve this. Can you please check and confirm?

Nope, does not work on Electron. (typeof require === 'function') is evaluated as True in Electron renderer environment, so the call falls into module.exports = angularMoment(require('angular'), require('moment')) which does not
My PR is based on the issue where is solved how to detect Electron environment, and it works quite well

Got it! Can you please update the PR for the latest code version?

rebased on the latest changes (release 1.0.0)

Coverage increased (+0.03%) to 93.243% when pulling dccdbbcea6421def5c000c6c88f53affc103d5ad on gantrior:master into 448e472efb22bd51bdc210cf80c7d296b63e0aaa on urish:master.

Thanks!


Add eslint-plugin-testing-library

Is your proposal related to a problem?
People don't always know the best way to use testing library
Describe the solution you'd like
Include eslint-plugin-testing-library by default and add a lint script.
Describe alternatives you've considered
Have people include it themselves. Discoverability is an issue.

Whoops 😅
https://github.com/facebook/create-react-app/issues/8959


Screen / view multiple stations at once

There was a feature request from Hermione to visualize images and predictions for multiple stations simultaneously to speed screening and eliminate redundancy when radar domains overlap.
This is a potentially big feature depending on how it can be implemented and would likely also require changes to roost-system.

I don't think we're going to do this


Add Rust version

fix #100
I've set the 𝗥 emoji for now feel free to propose others changes.

Here is a screenshot with the Rust plugin enabled:

@charlesvdv I see too many spaces before R. Let me explain:
I use two spaces before other symbols, because they're usually emojis. Emojis take 2 characters and it looks ugly:

If R takes only one character, there's a reason to use only one space.
I'm wondering about using single space after symbols. In case if terminal emulator displays them properly, users will have only one space (as it should be), instead of two spaces. In case if terminal emulator sticks characters together (as on screenshot above), users won't have space between version and symbol (that's acceptable, I think).

I don't understand from your last message if I need to remove or leave the extra space 😄 For now, I've removed it. Here is a screenshot with the updated themes.

Thanks for contributing!
Published as v2.1.0!


Create integration test for local setup

This should test the following:

Adding an agent after the cluster has run for a while
Removing an agent while the cluster is running
Adding the agent back after it has been removed (will the cache reset itself correctly?)
Moving an agent from one concentrator to another one.
Removing the auctioneer and adding it back with a different market basis (will the whole cluster reconfigure itself?)
Removing the concentrator and adding it back (will the cluster reconnect everything correctly)

These tests could be performed in different cluster configurations:

x agents with 1 auctioneer
x agents with 1 concentrator and 1 auctioneer
x agents to each of y concentrators and 1 auctioneer
x agents to each of y concentrators and 2 auctioneers (are the different clusters really independant?)

It looks like these tests are fine in the development_cgi branch. We'll pull those changes later.


Fix detect ocp

@sudhirverma Now it also works with kubectl downloaded via extension, locally build and run works fine. Will give a try to deploying as well.

LGTM.


It takes long to "fetching rates.."

When I call curcon it will start fetching rates and it takes around 5 seconds to show the data. Using BRL as BASE, node -v 17.3.1.
I have a M1 Pro Macbook, restarted multiple times and still takes long to show data.

I am experiencing the same delay here, also on a Mac. Tested with other Node versions (via NVM).


Redirect URL after checkout/{orderId}/token

Using Umbraco 13.1, with Umbraco.Commerce.PaymentProviders.Stripe 13.1.0, trying to do a headless checkout workflow with Hosted option (just redirect to payment), following the documentation, everything goes fine until the call /umbraco/commerce/storefront/api/v1/checkout/{orderId}/token, and redirect to the payUrl, the payURL is something like:
https://domain.com/umbraco/commerce/storefront/api/v1/checkout/2a0bfa0e-26e0-4e1a-ae38-0190746bcec4/pay/aa5973cd-cdaf-4d80-9770-9561909aa2de
Redirecting to the URL gives a broken HTML and it doesn't work. I already tried the same thing using the https://github.com/umbraco/Umbraco.Headless.Demo, it also has the same problem using the redirect option on UMBRACOCOMMERCECHECKOUT_MODE.
the HTML returned:
<!DOCTYPE html>
<html>

<head>
    <meta charset="utf-8">
    <title>Umbraco Commerce - Redirecting...</title>
    <script
        src="https://js.stripe.com/v3/"><script>
                var stripe = Stripe('pk_test_xxxxxxxxxxxx');
                window.handleStripeCheckout = function (e) {
                    e.preventDefault();
                stripe.redirectToCheckout({
                    sessionId: 'cs_test_xxxxxxxxxx'
        }).then(function (result) {
                    // If `redirectToCheckout` fails due to a browser or network
                    // error, display the localized error message to your customer
                    // using `result.error.message`.
                });
                return false;
    }
                <script></head><body><form action="https://domain.com/umbraco/commerce/payment/continue/stripe-checkout/2a0bfa0e-26e0-4e1a-ae38-0190746bcec4/ORDER-01645-61742-GPQPJ/b905d5b42dc26929a36594a6126879fb7a817297" method="post" id="form6235abd338fe4bbca11d98466d5ecb3a"></form><script>window.onload = function(event) { var frm = document.getElementById("form6235abd338fe4bbca11d98466d5ecb3a"); if (frm.hasAttribute("onsubmit")) {frm.dispatchEvent(new Event("submit", { cancelable: true })); } else {frm.submit(); } }</script>
    </body>

</html
```>

Closing this issue as it’s already been raised here https://github.com/umbraco/Umbraco.Commerce.Issues/issues/532
Closing this one as it’s more an issue in the core Umbraco Commerce. Please follow the other issue for updates.


Intersection param against bbox

I was looking for a "contains" parameter, cause i strictly require catalogs contains my AOI.
bbox parameter (I whould like it in geojson format) seems the only one(by documentation) to follow this but:

bbox: Only features that have a geometry that intersects the bounding box are selected. The bounding box is provided as four or six numbers, depending on whether the coordinate reference system includes a vertical axis

So, it's not my case.
Intersects parameter seems specifically for an intersection...(so the same as bbox but supports geojson)

intersects** - Provide a GeoJSON Feature string or the name of a GeoJSON file containing a single Feature that is a Polygon of an AOI to be searched.

There is a way to search for contains?
Thanks

I'd also be interested in a new "contains" parameter. With intersects, the AOI doesn't have to fit into a single scene which is not ideal in some cases.


Return lastrowid on insert_returning method

There was a bug that the insert_returning method successfully inserted but returned None.
Since asyncmy and PyMySQL set lastrowid in the Cursor instance at the time of insert, it has been modified to return it.
https://github.com/long2ice/asyncmy/blob/bb7fdd109fb04a365211eece55460d8ff9ea8112/asyncmy/cursors.pyx#L382

Thank you for the pull request! This is the first ever pull I have received from somebody I don't work with 🥇

@kamiazya I think you might have misunderstood the nature of insert_returning. This is a feature that is not implemented in MySQL, but is present in most other RDBMS.
For reference, other RDBMS implementations:
PostgreSQL: https://www.postgresql.org/docs/9.4/dml-returning.html
MS SQL: https://docs.microsoft.com/en-us/sql/t-sql/queries/output-clause-transact-sql?redirectedfrom=MSDN&view=sql-server-ver15
SQLIte: https://www.sqlite.org/lang_returning.html
MariaDB: https://mariadb.com/kb/en/insertreturning/
Because MySQL does not support this functionality, there are a few options:

Raise a meaningful error like the python NotImplementedError.
Return RowId as you have suggested
Parse the INSERT returning statement and run the returning part as a select using LAST_INSERT_ID to filter to the correct row.

Of these, 2 is not really the expected behavior of an insert returning and 3 is outside of the scope of both aiosql and aiosql-mysql so I think item 1 is the best choice.


Generate proper bits of random value for LongDecimalType tests.

This PR is to fix https://github.com/facebookincubator/velox/issues/6317. The explanation of the fix is in the issue.

@Yuhta Can you please review this PR? Thanks!

Looks like this PR is already delivered to facebookincubator:main

Merged in #6275


Performance: Android app

Find a way to optimize FPS on the mobile app

Optimized physics floor collision

Won't fix - no design for such purpose


Core: Use ThreadLocalRandom for PoissonInjection

We'd lose the ability to set the seed, but we'd remove contention.
The problem is we wouldn't have a predictable total number of users, which would impact the Controller (termination) and the DataWriters (progress).

Reverting, causes too many issues.


Add Websocket testing

Description
Add websocket testing. It closely follows how Uvicorn testing works, but the app is automatically pointed towards the project's.
I've added some comments on how to do things (e.g. authentication on websocket.connect using JWT decoding. A little unsafe, but it should be fine if it's only used for that specific use-case. I don't think it needs any other warning than don't use it for desktop, however the package is called DRF SimpleJWT.).
Rationale
All projects need testing!
Use case(s) / visualization(s)
I said I wouldn't put EXTREMELY new things on cookiecutter django, but I thought it would be great to have people at least take a look at a proven design which Django is pushing towards anyways.

I'm not exactly sure what the error is...? I ran tox with no issue, but I am using python 3.7... Hm There's an apparent import error for conftest.py

Closing in favor of a new approach


buildkite-agent: 3.85.1 -> 3.86.0

Diff: https://github.com/buildkite/agent/compare/v3.85.1...v3.86.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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

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

1 package built:

buildkite-agent


Overloading propertynames

It would be convenient to overload Base.propertynames for DICOM.DICOMData. It makes interactive work with it easier by exposing properties in REPL tab-completion. I've prepared the following implementation and I can make a PR if you're interested:
function Base.propertynames(dcm::DICOM.DICOMData)
    basic_properties = invoke(propertynames, Tuple{Any}, dcm)
    dcm_keys = keys(dcm.meta)
    pnames = Symbol[basic_properties...]
    for (k, v) in DICOM.fieldname_dict
        if v in dcm_keys
            push!(pnames, k)
        end
    end
    return pnames
end

Now I can get a nice list:
julia> dcm_data.
AccessionNumber                FileMetaInformationGroupLength  InstanceCreationDate            OtherPatientIDs                 PositionReferenceIndicator      SeriesNumber                    StudyTime
BitsAllocated                  FileMetaInformationVersion      InstanceCreationTime            PatientBirthDate                ReferencedRTPlanSequence        SliceThickness                  TissueHeterogeneityCorrection
BitsStored                     FrameIncrementPointer           InstanceNumber                  PatientID                       ReferringPhysicianName          SoftwareVersions                TransferSyntaxUID
Columns                        FrameOfReferenceUID             Manufacturer                    PatientName                     Rows                            SpecificCharacterSet            endian
DeviceSerialNumber             GridFrameOffsetVector           ManufacturerModelName           PatientSex                      SOPClassUID                     StationName                     isexplicit
DoseGridScaling                HighBit                         MediaStorageSOPClassUID         PhotometricInterpretation       SOPInstanceUID                  StudyDate                       meta
DoseSummationType              ImageOrientationPatient         MediaStorageSOPInstanceUID      PixelData                       SamplesPerPixel                 StudyDescription                vr
DoseType                       ImagePositionPatient            Modality                        PixelRepresentation             SeriesDescription               StudyID
DoseUnits                      ImplementationClassUID          NumberOfFrames                  PixelSpacing                    SeriesInstanceUID               StudyInstanceUID

That's brilliant!
It would definitely be a nice a improvement, so a PR is welcome.


python39Packages.click: 8.0.3 -> 8.1.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

This might just be a failure in the test suite, after all it's using a private function

Please let's do this via another python-updates run. The black regression is fixed in the version on staging and typer will need a patch. There are probably others.

Merged into my local working tree for python-unstable. Thank you.


info update Maas

Description

As already mentioned in #ISSUE_NUMBER, this PR tackles:

...
...
...

In particular, the ...
Checklist

[ ] I followed the indications in the CONTRIBUTING
[ ] The documentation related to the proposed change has been updated accordingly (also comments in code).
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Ready for review! :rocket:

Fixes

Fixes #

@sebbalex quando vuoi/puoi smarmella pure
grazie


Container View getting pushed down if UINavigationBar + Translucent?

I have a UIViewController with a Container View. This Container View contains a UITableViewController.
The UIViewController is embedded in a UINavigationController. I configure the NavigationBar by changing the tint color, the bar tint color and the title text attributes, but when I try to change the translucency I have a problem
UINavigationBar.appearance().isTranslucent = false
When translucent = YES
+---------------------------------+
|                                 |
|     UINavigationBar             |
|                                 |
+---------------------------------+
|                                 |
|                                 |
|                                 |
|                                 |
|                                 |
|    UITableViewController        |
|    in a Contained View          |
|                                 |
|                                 |
|                                 |
|                                 |
|                                 |
|                                 |
+---------------------------------+

when translucent = NO
+---------------------------------+
|                                 |
|     UINavigationBar             |
|                                 |
+---------------------------------+
|                                 |
|blank space created on top of view
|                                 |
+---------------------------------+
|                                 |
|                                 |
|                                 |
|                                 |
|                                 |
|                                 |
|     UITableViewController       |
|     in a Contained View         |
|                                 |
+---------------------------------+

I have the same issue too.
With this setting UINavigationBar.appearance().isTranslucent = false I get the following:

With this setting UINavigationBar.appearance().isTranslucent = true  I get this:

+1

I think the problem is in SJUtil::getTopSpacing() that does not take in consideration isTranslucent.

Here is what I did to solve my problem.
For the navigation bar I added the following if statement
 if !(navigationController?.navigationBar.isOpaque)! {            topSpacing += (navigationController?.navigationBar.bounds.height)!                 }
For the tab bar I modified the following if statement
if !tabBarController.tabBar.isHidden && !tabBarController.tabBar.isOpaque {                 bottomSpacing += tabBarController.tabBar.bounds.size.height             }

still m facing same issue
Help?


Add support for subsequent software updates or patches (e.g., Cisco SMUs, Palo Alto hotfix, etc.)

As ...
Nelly - Network Engineer
I want ...
To be able to denote a software version as a prerequisite to other software versions.
So that ...
I can properly identify and model not only the most up to date software version that is required to be on a device, but also denote any prerequisite versions that must be previously installed to function properly.
I know this is done when...

I am able to specify a software version as a prerequisite of another version
I can properly model out the various relationships between a base image and 1 or more subsequent patches that need to be installed

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[X] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Apps (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
This would probably require the Software model to change to be nestable.
External Dependencies
I don't think so.

I created this issue based on this closed issue from the device lifecycle repo: https://github.com/nautobot/nautobot-app-device-lifecycle-mgmt/issues/173


Update Ethereum to version 0.1.1

Notable changes:

Fixed an issue preventing the sealer nodes from starting in EE DC/OS in Permissive mode.
Reduced resource requirements for boot and ethstats nodes
Improved usage documentation

Changes since v0.1.0:
$ git shortlog v0.1.0..HEAD
Drew Kerrigan (2):
      Use Scheduler VIP instead of master.mesos
      Reduce resource requirements, limit geth cache, improve documentation

@vishnu2kmohan @ryadav88 I found a problem preventing this framework from running in EE with permissive security, please restart tests if you get a chance. Thanks!

Triggered tests. @drewkerrigan could you add a diff from the last package update?

@ryadav88 Sure, I created this PR with 2 commits, the second one showing only the changes from the first version. The diff is visible here: https://github.com/mesosphere/universe/pull/2111/commits/dca85f5efc5c295ea82ac1fb0f05c2e029551c5c


Unable to prevent reportUndefinedVariable false positives

Environment data

Language Server version: 2021.4.2 (pyright 3888d715)
OS and version: macOS 11.3
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.9 from MacPorts

Expected behaviour
I am getting "ctx" is not defined Pylance(reportUndefinedVariable) errors from code files where ctx is a symbol that's injected by the runtime environment (MeVisLab) for which these files are written.
I would have expected a way to suppress these false positives, much like with other tools: For instance, I have additional-builtins in my .pylintrc, and I have an environment variable PYFLAKES_BUILTINS set up.
Furthermore, I would have expected this configuration to be discoverable. It may also be a good idea to document that configuration for other tools like pylint or pyflakes does not apply, but that pylance is a standalone tool that does not make use of these. (Since some python linting tools call others, it was not obvious enough to me that pylance is not just some efficient integration of existing tools for VS Code.)
Actual behaviour

The measures I took to make pylint and pyflakes happy do not seem to apply to pylance, but I get a lot of false positives (with apparent side-effects for variable renaming, for instance?).
clicking on the reportUndefinedVariable link does not lead to instructions how to configure additional builtins.
I cannot find a way to configure these by searching for "pylance" in the VS Code configuration.
TROUBLESHOOTING.md does not help, either.

Logs

Background analysis message: setFileOpened
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
[FG] parsing: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (9ms)
[FG] parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/builtins.pyi [fs read 0ms] (103ms)
[FG] binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/builtins.pyi (115ms)
[FG] binding: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (1ms)
Background analysis message: getSemanticTokens full
[BG(1)] getSemanticTokens full at /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py ...
[BG(1)]   parsing: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (16ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/builtins.pyi [fs read 0ms] (130ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/builtins.pyi (60ms)
[BG(1)]   binding: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (4ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/typing.pyi [fs read 1ms] (104ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/typing.pyi (38ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/re.pyi [fs read 0ms] (8ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/re.pyi (11ms)
[BG(1)]   parsing: /Users/hmeine/Developer/minimal_mevis/mevis.py [fs read 0ms] (4ms)
[BG(1)]   binding: /Users/hmeine/Developer/minimal_mevis/mevis.py (1ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/abc.pyi [fs read 0ms] (1ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/abc.pyi (0ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/enum.pyi [fs read 1ms] (9ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/enum.pyi (1ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/sys.pyi [fs read 0ms] (31ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/sys.pyi (2ms)
[BG(1)]   parsing: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/types.pyi [fs read 0ms] (11ms)
[BG(1)]   binding: /Users/hmeine/.vscode/extensions/ms-python.vscode-pylance-2021.4.2/dist/typeshed-fallback/stdlib/types.pyi (2ms)
[BG(1)] getSemanticTokens full at /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (750ms)
Background analysis message: getSemanticTokens range
[BG(1)] getSemanticTokens range 0:0 - 116:68 at /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (11ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: analyze
[BG(1)] analyzing: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py ...
[BG(1)]   checking: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (7ms)
[BG(1)] analyzing: /Users/hmeine/Developer/MeVisLab/Standard/Modules/Macros/Helpers/RunPythonScript.py (7ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: resumeAnalysis
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange

Pylance doesn't support adding random variables to the builtins (and I didn't know pylint/pyflakes have configuration for this).
Can you do something like this to notify type checkers (like pylance) that the additional variables exist?
from typing import TYPE_CHECKING

if TYPE_CHECKING:
    ctx: SomeType = ...

Yes, I also agree that this injection is not nice.  (It's also only exactly this one symbol ctx for the local context, so it's not much in terms of pollution. Other functionality is just imported.)
TL;DR: 1 is no option here, even if I would prefer it otherwise.
Since I like the "reportUndefinedVariable" functionality a lot, I'll have to investigate option 3, even though it is a hack.

If you're investigating option 3, I'm going to close this issue as I don't think that this is waiting for user response anymore. Please ping if you think this should be reopened.

Yes, makes sense.
For the record, I wrote a script to automate locating and modifying builtins.pyi which I run after each VS Code upgrade.


PPM SUm problem

I have strange problem witch graupner MX-20 and GR-16 receiver
when I switch receiver to SUMO 08 ch it only see first eight channels, but if I switch to SUMO 16 it don't see any signal from any channels, any ideas?
regards Michal

Very likely a different timing than expected. Have you seen this doc-segement:
https://github.com/wernerdaehn/CC3D-CableCam-Controller/blob/master/docs/Step 3b - Receiver Input SumPPM.md#settings
"The debug command $d p was added to see the current timings. The output might look like..."


add a client.NewShell(shellID) method so we can create a Shell for a given shellID

I've a use case where I want to be able to create a Shell for a pre-existing shell ID so that I can delete it. The client is private in the Shell struct so I can't just create a Shell directly - so it'd be nice to add a little helper method to make a new Shell from a shellID.
A PR is on its way shortly.
BTW if you're interested, we need this for kansible:
https://github.com/fabric8io/kansible
Many thanks for winrm BTW! Its a lovely piece of work!

PR now merged!


mvn build of command line OWL tools on OSX files with missing file - cycle.omn

From<EMAIL_ADDRESS>on December 17, 2013 11:12:12
<b>What steps will reproduce the problem?

Downloaded read-only set of sources
In appropriate directory issued command: mvn clean package
<b>3.

<b>What is the expected output? What do you see instead?

Build fails during OWLTOOLS-core section on failed test. Test error:

Failed tests:   testTransitiveCycle2(owltools.util.MinimalModelGeneratorTest): Requested resource does not exists: src/test/resources/cycle.omn

Verified that MinimalModelGeneratorTest.java references cycle.omn as resource. Verified that cycle.omn file does not exist in the download

<b>What version of the product are you using? On what operating system?

OSX version 10.8.5

<b>Please provide any additional information below.

Original issue: http://code.google.com/p/owltools/issues/detail?id=84

From<EMAIL_ADDRESS>on December 17, 2013 11:16:06
This issue was closed by revision r1822 .
Status: Fixed


Added XmlTransform Aliases for XSL/XML transformations

This PR propsed new aliases for transforming xml data via xml stylesheet.
This is done either via passing xml & xsl as string and getting transformed string back, or passing path to xml, xsl & result.
Example usage with paths:
XmlTransform("breakfast_menu.xsl", "breakfast_menu.xml", "breakfast_menu.htm");

Example usage with strings:
var xls = System.IO.File.ReadAllText("breakfast_menu.xsl", Encoding.UTF8);
var xml = System.IO.File.ReadAllText("breakfast_menu.xml", Encoding.UTF8);
var html = XmlTransform(xls, xml);
Information(html);

To both above methods there's an overload where you can pass settings to change generated XML output, example usage:
XmlTransform("breakfast_menu.xsl", "breakfast_menu.xml", "breakfast_menu.htm", new XmlTransformationSettings
    {
        OmitXmlDeclaration = false,
        Encoding = Encoding.Unicode
    });

Looks good to me.


Portuguese translations for timeline module

Need Portuguese translations for the timeline module, either by someone here or through Crowdin (former would probably be faster). Since our translations.json was split into separate files for each language, you can find Portuguese in pt-PT.json
The timeline's at the bottom.
Thanks in advance!

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Done, courtesy of @puzzledbytheweb


How to solve question steps by steps using mathsteps

Welcome
How to solve question steps by steps using mathsteps
whats is the steps to enter parameter in classes of this library please is necessary ??

Can you elaborate it a bit further ?

Hi @OmarTawashi I'm not sure what you're asking - could you rephrase?
The screenshot there is from mathpapa and not from our own library

can i use mathsteps to solve polynomials and calculus problems if true how

Hi @salumsinare,
Glad you asked! Currently mathsteps only supports arithmetic and basic algebra. This applies to polynomials also. You can also factor and simplify polynomials, but that's about as far as it gets. However, we are working towards implementing more complex operations and calculus is definitely on our todo list. Make sure to join the mathsteps gitter discussion to learn more about our progress!

no to calculus, yes (most of the time) to polynomials
if you give me an example of a problem? and I'll let you know how you can use mathsteps with it

@evykassirer
Here is the problem i tried to solve
x^2+3x+2=0
so far it gives me x^2+3x=-2

ah! yes that requires factoring - factoring quadratics is actually all coded up and ready to go but not merged or deployed yet.
Do you need it soon for something or were you just playing around with it? If you'd like to be able to use that functionality soon, I can prioritize getting it ready :)

@salumsinare it looks like only certain forms of quadratics are handled at the moment, see https://github.com/socraticorg/mathsteps/blob/master/lib/ChangeTypes.js#L179-L183.

#148 added some coverage on top of that too, though didn't actually add anymore changetypes

@evykassirer  i don't need it that soon but i was just working on a project that requires such functionalities plus calculus

is there a way to detect the nature of an expression/equation based on mathematical topic or concept and the general method/formula to solve it (if it exist) before actually solving it
if i enter to solve  x+2=0 it gives something like linear equation
and if i do   2x+3<0 it gives inequality

Yup, all we would have to do is scan the expression for those specific relation nodes (<, =, >). From that we can determine whether it is an expression, equation, inequality, etc.

can mathsteps generate graphs based on an expression or there are any other alternative to achieve this

That's outside the scope of mathsteps but you can search the web for graphing libraries - a bunch exist

I think we can close this! (outside the scope of project)


poudriere image: Remove comments when passing contents of package lis…

…t file to "pkg add".
Sponsored by: Modirum

Will submit a better fix for this in a while.


Fix bug with stale state in http request factory

If we don't re-use an existing factory, the label doesn't change and all
the snippets get generated with the same identifier.
Also remove explicit version from wiremock test

Oops, that was merged a while back already.


Fix artist list comma color

Thanks for leaving in a layup issue to get me started.
This should work but I wasn't able to run the node server locally because I wasn't sure what should go in the config = require('./config') file as I couldn't find any checked in example files. If you could point me in the right direction to set that up so I can run the server next time that would be great.

Fixes #203

Great work, thanks!
A template for the server config for can be found here to get you started: https://github.com/pdaddyo/soundbounce/issues/228#issuecomment-93723898

Sweet! Thanks @andrewthecoder
Maybe that sample config should be put in the repository as src/server/config.js.example... what do you think?

Yeah agreed it should be added to repo, thanks for fix too.


You don't have an extension for debugging AutoLISP.

Issue Type: Bug
Updated VSCode to 1.56.2 from 1.55.2 and AutoLISP extension stopped working. Now I cannot attach to AutoCAD 2021 and debug autolisp code.
An error message states: "You don't have an extension for debugging AutoLISP" Should we find a AutoLISP extension in the market place?
However extension is already installed and previous version 1.55.2, was working before update.
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 x 4200)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
63.95GB (49.17GB free)

Process Argv
--open-url --crash-reporter-id 4a117aef-66c6-475a-be97-0d7c883b40a0 -- vscode://autodesk.autolispext/?pid=18940

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

autolispext
Aut
1.4.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt943:30300582
vspre833:30267464
pythonptprofiler:30281270
vshan820:30294714
vscoreces:30290705
pythondataviewer:30285071
vscus158:30286553
vscgsv2ct:30301613
bridgeflightcf:30302070
vscod805:30301674

Dup of https://github.com/microsoft/vscode/issues/123881#issuecomment-842134812

@serhanbakir
Please follow the steps in the comments here to work around it for now:
https://github.com/Autodesk-AutoCAD/AutoLispExt/issues/143


How to completely remove this program from windows 10

this program completely broke everything and i can't connect to anything please help me remove since i su*k at using computer

I'm probably a bit late, but just reset your DNS server settings and everything should be okay


[Rendering] Accessibility: Screen reader cannot read "alternate text" from image with selection action (Action.OpenUrl)

Target Platforms
NodeJS
SDK Version
2.9.0
Application Name
Web Chat
Problem Description
When adding an image element with selection action of type Action.OpenUrl, the "alternate text" from the image element cannot be read by the screen reader.
Rendering the attached card through AC 2.9.0, the DOM tree will be the following. Note the convoluted  with role="link".

When reading using a screen reader (Windows Narrator + Edge), it will say: "link this is title". And it will not read the alt text "this is alt text". Even when we press CAPS + DOWN to dig into the content of the link, the alt text will not be narrated.
In comparison, when implementing an image link in HTML with the following DOM tree:

The screen reader will narrate:

"Link image this is title"
Then, press CAPS + DOWN to dig into the content of the link
"This is alt text"

(Related to https://github.com/microsoft/BotFramework-WebChat/issues/4025)
Screenshots
The card is rendered as follow.

Card JSON
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "Image",
            "style": "Person",
            "size": "Small",
            "url": "https://www.bing.com/th?id=AMMS_538ac5fe8754dd7df7589f49378971dc",
            "altText": "This is alt text",
            "selectAction": {
                "type": "Action.OpenUrl",
                "title": "This is title",
                "url": "https://github.com/"
            }
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.3"
}

Sample Code Language
No response
Sample Code
No response

Hi team - please be aware that this is a Severity 2 Accessibility bug and needs to follow compliance guidelines


Added Bson-Kotlin Array Codec

Adds Kotlin array support to the bson-kotlin library
JAVA-5122

Thanks @vbabanin the when block looks much better


Console errors during mouse-over event for component with no corresponding App

Version
6.1.4
Browser and OS info
Windows 10 / Chrome 101.0.4951.54
Steps to reproduce
The following is happening on my Laravel website that has Vue used on one or two pages.  (I haven't used the Vue DevTools in a few months but I never saw this behavior before on my website when I last used it.  My website's use of Vue has not changed in that time.)

Navigate to a page using Vue
Open DevTools to the Vue tab and note the presence of an App (in the apps list) and a mounted Vue component (under Root in the components list)
Click on the component and each child to view that component's details
(leave DevTools open and) Navigate to a different page that does not have a Vue component.
In DevTools on Vue tab, the component from step 2 is still listed under Root but the app is no longer in the apps list.
Click on (or mouse over) the component under Root.

At this point, Javascript errors are logged in the console from different points in the backend.js file.  (detailed in the 'What is actually happening?' section of this issue)

Close and re-open the Chrome DevTools to the Vue tab and now the app list and the component list are empty.  No more errors are encountered (until repeating from step 1)

What is expected?
No errors
What is actually happening?
Errors seen:

[Bridge] Error in listener for event b:component:mouse-over with args -- backend.js:7028
TypeError: Cannot read properties of null (reading 'instanceMap') -- backend.js:7029

Hi, I have the same problem, how can I solve it?

Same here

Got same problem.
Is any solution for it?


Update time tracking on casting geomancy

Related problem
When using mobile, or especially when using the Kronos anima patch, the time tracking plugin loses it's value to keep track of your farming patches.
The solution I'd like
Casting the Lunar spell Geomancy retrieves most of the usefull data for the time tracking plugin. It would be great it the plugin could update it's times when you cast Geomancy.
Alternatives I've considered
Without it, a player has to either remember all possibly relevant patches, make a note of them, or repeatedly cast geomancy.

Duplicate of #7350


Incorrect test vector

000 40 41 42 43 44 45 46 47 48 49 4a fb 4c 4d 4e 4f  @ABCDEFGHIJKLMNO
             Should be 4b and not fb^^^

Should be 4b and not fb.

Thanks! 👍
This was fixed in https://github.com/bikeshedders/xchacha-rfc/commit/95bd86c0d943caaa73f4f64f49a76a07fcfd0046
I included a verification script in https://github.com/bikeshedders/xchacha-rfc/commit/25eba62263bffbab229a8da1d19505e86f1da307 as well.


Possible issue when GPT API key does not have access to GPT 4

Hi,
This method looks like a legacy.
Unfortunately with my API key the "model": "gpt-3.5-turbo" does not work.

I had to change the Model to the following below...
Source : https://platform.openai.com/docs/api-reference/completions/create
   '{
"model": "gpt-3.5-turbo-instruct",
"prompt": "Say this is a test",
"max_tokens": 7,
"temperature": 0,
"stream": true
 }'

As I have been using my API key in my own project works fine with the following request.
The recommended request for GPT 3.5
https://platform.openai.com/docs/api-reference/chat/create
   "model": "gpt-3.5-turbo",
"messages": [
  {
    "role": "system",
    "content": "You are a helpful assistant."
  },
  {
    "role": "user",
    "content": "Hello!"
  }
],
"stream": true

So the default model should be "model": "gpt-3.5-turbo-instruct"  Can someone with GPT4 access test this? Thanks

Thanks! I was getting the same error with gpt-3.5-turbo, but gpt-3.5-turbo-instruct worked. gpt-4, as I have, is working too.

Thanks for testing.

Hello, after receiving the build from the program, when I ask a question to the artificial intelligence, my program crashes. What should be done?

Hello, after receiving the build from the program, when I ask a question to the artificial intelligence, my program crashes. What should be done?

Check what error message you have in your logcat.

The internet must be on, otherwise the software will not work and it will
kick me out. I turn on the internet, I ask a question, it gives this error
Samsung S24 U
On Tue, 26 Mar 2024, 16:17 Peter Szollar, @.***> wrote:

Hello, after receiving the build from the program, when I ask a question
to the artificial intelligence, my program crashes. What should be done?
Check what error message you have in your logcat.
—
Reply to this email directly, view it on GitHub
https://github.com/lambiengcode/compose-chatgpt-kotlin-android-chatbot/issues/17#issuecomment-2020336780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGX66KHNIQ4RS5OFHP7P7B3Y2FN3JAVCNFSM6AAAAAA5OUC53WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGMZTMNZYGA
.
You are receiving this because you commented.Message ID:
<lambiengcode/compose-chatgpt-kotlin-android-chatbot/issues/17/2020336780@
github.com>

to be fair I have not done anything on this repo for a long time. So I have no idea what the owner of the repo has changed and too busy with my work to look into it. As far as I remember if you do not have the api key accessing to GPT4 or 3 it could through an error like this. Sorry

@sollarp  Thank you for your support. @mehrdadpasiyann We neglected this project for quite a while, because of some factors, I think it's small enough for a reference project. In addition, I see that there are still success messages on the firebase database, some are fail because wrong api key. If you use firebase in this repo, I think just a reason is wrong api key and if you change your own firebase, let's check indexes tab on firebase, you need create index for this project.

Thank you
Samsung S24 U
On Tue, 26 Mar 2024, 17:25 Kai Dao, @.***> wrote:

@sollarp https://github.com/sollarp Thank you for your support.
@mehrdadpasiyann https://github.com/mehrdadpasiyann We neglected this
project for quite a while, because of some factors, I think it's small
enough for a reference project. In addition, I see that there are still
success messages on the firebase database, some are fail because wrong api
key. If you use firebase in this repo, I think just a reason is wrong api
key and if you change your own firebase, let's check indexes tab on
firebase, you need create index for this project.
—
Reply to this email directly, view it on GitHub
https://github.com/lambiengcode/compose-chatgpt-kotlin-android-chatbot/issues/17#issuecomment-2020496521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGX66KEXWDVB72GTOS6NJ7DY2F46TAVCNFSM6AAAAAA5OUC53WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGQ4TMNJSGE
.
You are receiving this because you were mentioned.Message ID:
<lambiengcode/compose-chatgpt-kotlin-android-chatbot/issues/17/2020496521@
github.com>


Some findings aren't displayed when using paging.

Bug description
When we view all findings on one page (page size=all), it displays all findings. However, if paging is used to display findings (default mode is 25 items per page), it does not display some findings and instead shows some findings repeatedly.
For example, in picture 1, we can view all the critical and high findings of a product on one page (page size=all). Items marked in red are not displayed when we use paging.

However, if paging is used to display findings, it does not display some findings and instead shows some findings repeatedly. For example, pictures 2 and 3 show that some of the findings are repeated (similar colors show similar findings) and the findings that are marked in red in picture 1 aren’t displayed on the two pages (page size=25).

Unable to read the screenshots, can you upload high res screenshots?

I uploaded screenshots again.

What's on page 3?

This also happens on pages 2 and 3. As we can see in the images below, some of the findings on page 2 and 3 are similar.

@manuel-sommer I sent a sample file :all findings are shown in sheet1, sheet2 contains findings when using paging (page size=25) and in sheet3 we can see missed findings that are not displayed in sheet2, in other words these findings are not displayed when using paging. (in sheet 2 the repeated findings are marked with color)
findings-bug.xlsx

I would rather like to have a scan file which can be uploaded to reproduce this behavior.

@manuel-sommer this an exported file that you can use it in defectdojo to reproduce it.
findings.xlsx

I have the same issue when I see my findings on one page vs multiple pages (Aqua Scan). Is this bug fixed or should I open a new issue?


[BUG] New wallet bottom bar overlaps on iOS [0.10.7]

Describe the bug
New wallet bottom bar  (0.10.7) overlaps slightly with wallet information page.
To Reproduce
Steps to reproduce the behavior:

Open LNbits wallet on iPhone
Scroll the wallet info page to the bottom
New Wallet bottom bar overlaps slightly

Expected behavior
When scrolling to the bottom of the page there should be additional padding so that all information is visible.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

LNbits version: [0.10.7]
Database sqlite
iPhone 14 Pro
Safari

Additional context
Add any other context about the problem here.

Issue seems a bit more severe since the Extensions buttons can’t be clicked anymore.

this was solved by https://github.com/lnbits/lnbits/pull/1740


Update check_extra791 remediation verbiage

Context
Noticed the remediation verbiage could use cleanup.
Description
Very minor change to update remediation verbiage from "stronger as possible" to "as strong as  possible"
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Very minor change to update remediation verbiage from "stronger as possible" to "as strong as possible"


Code generator does not create some imports

Describe the bug
Declarations in the safeds package or any subpackage are always available. If explicit imports are missing, however, no imports are created in the Python code. This also leads to failures in the runner: "name 'Table' is not defined".
To Reproduce
Safe-DS:
package test

pipeline myPipeline {
    val table = Table.fromColumns([]);
}

Python:
# Pipelines --------------------------------------------------------------------

def myPipeline():
    table = Table.from_columns([])

Expected behavior
The following code should be created:
# Imports ----------------------------------------------------------------------

from safeds.data.tabular.containers import Table

# Pipelines --------------------------------------------------------------------

def myPipeline():
    table = Table.from_columns([])

Screenshots (optional)
No response
Additional Context (optional)
No response

:tada: This issue has been resolved in version 0.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.10.0
GitHub release

Your semantic-release bot :package::rocket:


fix:[PIPE-18556]: Add limitation of allowed values with workaround for runtime inputs in list fields

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 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.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66ceda671f71780ef8141b52--harness-developer.netlify.app


[Improvement] Add people who made significant contribution to contribitors table

What would you like to be improved?
We are planning to donate our Gravitino project to the ASF, part of the process is to make sure everything is correct IP-wise. Usually, this means contributors signing Individual Contributor License Agreements (ICLAs). We want to make this a bit easier for people who have contributed to our project.
How should we improve?
@coolderli @xloya @caican00 @charliecheng630 @hiirrxnn @zivali @xiaozcy @NIRMALraki @Lanznx @Yangxuhao123 @danhuawang @TEOTEO520 @noidname01 @laiyousin @xnge
(don't worry if your name is not listed as some other contributors have been contacted via email)
If you could go to:
https://github.com/datastrato/gravitino/blob/main/MAINTAINERS.md
And create a PR adding your name, GitHub username and company (if applicable) to the Contributors table.

@coolderli @xloya @caican00 @charliecheng630 @hiirrxnn @zivali @xiaozcy @NIRMALraki @Lanznx @Yangxuhao123 @danhuawang @TEOTEO520 @noidname01 @laiyousin @xnge can you please add your names to the contributor's table. If you do not do this we will need to ask you to sign an ICAL. Filling in an ICLA is a lot more work, as you need to print it off, personally sign it, scan it back in and send it back.


filenotfounderror

I have created all of the directories below the cache ,but when i run python demo.py why is it displayed no such file or directory:'/home/miao/CornerNet-Lite/core/../cache/nnet/CornerNet_Saccade_500000.pkl'

You will need to also download the pretrained model for CornerNet-Saccade and put it under cache/nnet/CornerNet_Saccade/


NeoVim terminal closes a window

Hi,
I'm currently using NeoVim in conjunction with pytest. When I set the test strategy to be "neovim", the terminal opens just fine. However, it takes over the active split and, upon test completion, the terminal and the split get closed.  Is there a way to direct the terminal to open in a new split?
Thanks!

Recently the neoterm project emerged, which is a wrapper around Neovim's :terminal feature. The support was added just a few days ago to vim-test (let test#strategy = "neoterm"). As I understood, Neoterm allows some customization of how Terminal is opened, so maybe you could take a look to see if it suits your needs.

In general, it's outside of vim-test's scope to implement customization of strategies. Strategy plugins should provide their own customization and configuration, and should expose a simple interface for vim-test to use.

No worries. Thanks!

@janko-m how would someone create a custom strategy for vim-test? Specifically, when using something like Vundle which uses version control to manage plugins (so you don't want to edit the strategies.vim file)? How do we create a custom strategy given that it appears to be using test#strategy#<name> namespacing?

Thank you for noticing, this indeed is something I haven't given any thought. Try to put the strategy function inside of ~/.vim/autoload/test/strategy.vim, hopefully Vim will autoload your file as well as mine. But I should definitely make it so that you can put the strategy inside of .vimrc.

@janko-m Thanks for the quick response! I initially tried that, and sadly it didn't seem to work. Maybe double check before you do any work to re-factor (again, I am an absolute novice when it comes to vimscript/how vim plugins load), but it'd definitely be idea if we could put custom functions in our .vimrc.

@rschmukler I wrote a blog post about overriding vim-test strategies. Hope it helps.

@zenbro Thanks for discovering that! I tried the same thing before, but with defining a new strategy, and I get an error that I have to define it in the appropriate script file, but when overriding an existing strategy it works.
But I think it still isn't an ideal solution for @rschmukler, because he wants to add a new custom strategy, and I still didn't solve #28.

@zenbro thanks, super useful!

@janko-m why don't we just update the neovim strategy to use @zenbro's solution?

Good idea, for some reason it hasn't come to my mind even though I've read the blog post, I updated it now.

Now it works great, except for the following:
if you have 2 or more windows open and you run your tests from a window that isn't leftmost, the focus/cursor switches to the left most window


Semi-Automated Source-Cluster Detection

Whenever several people with Corona-Warn-App are close to each other, the app should log such moments.
A timestamp is sufficient.
No storage of personal or GPS data necessary.
In case of a positive test or risk warning: Time stamp goes to the health authority.
This would help to help affected persons to find out source clusters.
It allows the health department to ask the following question during the follow-up:
"Where were you last Monday afternoon? The app has recorded that you were among a lot of other people back then."
Details and discussion about this idea here (in german):
https://twitter.com/Florian83616025/status/1316734257059037185

Internal Tracking ID: EXPOSUREAPP-3331

@fstandhartinger
The exact timestamp of an Encounter is not given to CWA by the ENF at the moment, so this isn't really possible to introduce with the currently used way the App is programmed.

Thats sad, but thanks for the quick answer.
If the CWA would query the latest state from ENF lets say 4 times a day, would it then be capable of storing an information like this:
User has been in high exposure situation with multiple people on Monday afternoon?
That might be sufficient to help the health authorities with backward contract tracing (which is arguable worth more than forward contact tracing when it comes to lower infection spread).

@fstandhartinger

If the CWA would query the latest state from ENF lets say 4 times a day, would it then be capable of storing an information like this:
User has been in high exposure situation with multiple people on Monday afternoon?

Unfortunately no: all of this is handled within the Google/Apple Exposure Notification Framework (ENF) and not exposed to CWA:

If an encounter is found (by ENF) only the day of the encounter will be exposed, not the exact time (see here for more details on the response CWA gets from ENF)
The number of non-positive contacts is not exposed to CWA by ENF: https://github.com/corona-warn-app/cwa-wishlist/issues/124#issuecomment-660121125

Hello everyone,
I've mirrored the feature request to Jira. Even if it is not possible to implement right now, the cluster detection mechanism can be discussed in this thread. Official responses on the issue will also be posted here.
Regards,
CH

Corona-Warn-App Open Source Team

Hello everyone,
What is the current status of the important request?
I see a lot of open points in cwa-wishlist here, but I think the cluster detection should be a great help from a medical point of view. The clusters appear to be the biggest driver of the pandemic. If it is technically possible, why is this not simply implemented?

@kyklos as stated above: the implementation proposed in OP is unfortunately not technically possible within the current ENF system. One thing which could actually be implement though is a type of "venue registration" which would allow users to register if they visit high risk / high density events. See e.g. #138 for such a proposal.

@daimpi Thank you for your comments. What is not technically possible within the current ENF system? A venue registration is a other implementation feature and If cluster detection is an idea, why not use even if it means adjustments to the ENF system.

@kyklos

What is not technically possible within the current ENF system?

Querying information on non-positive encounters
Even for positive encounters querying exact timestamps is not possible, the info exposed by ENF is coarsened to a one day time frame.

I know CWA currently only queries ExposureSummary but why not using ExposureInformation?

This wouldn't help. Same points (1) and (2) apply there. Even ENF v.1.5+ with ExposureWindow mode is still restricted by those.

[…] why not use even if it means adjustments to the ENF system.

This is something Google/Apple would have to implement as they're in control of the ENF but maybe the CWA-Team could forward this request to them 🙂.

@kyklos

What is not technically possible within the current ENF system?

Querying information on non-positive encounters
Even for positive encounters querying exact timestamps is not possible, the info exposed by ENF is coarsened to a one day time frame.

I know CWA currently only queries ExposureSummary but why not using ExposureInformation?

This wouldn't help. Same points (1) and (2) apply there. Even ENF v.1.5+ with ExposureWindow mode is still restricted by those.

[…] why not use even if it means adjustments to the ENF system.

This is something Google/Apple would have to implement as they're in control of the ENF but maybe the CWA-Team could forward this request to them 🙂.

@daimpi Thank you pointing out.  It seems to me that ENF is restricted. When ENF is coarsened to a one day time frame why not create an additional database on the devices, insert needed data und use them to build such or other feature? A control of the ENF would be also great so forward a request Google/Apple.

@kyklos

When ENF is coarsened to a one day time frame why not create an additional database on the devices, insert needed data und use them to build such or other features?

You mean in the sense that CWA itself would record BLE tokens aka RPIs? CWA cannot do this because this is not allowed for ENF apps by Google/Apple. See this thread for further discussion on this topic.

Yet another idea:
CWA could just record how many other bluetooth devices are in range.
Lets say once every 5 minutes.
Then store that together with a timestamp.
Completely without using ENF, just using the normal bluetooth features of the smartphone.
Using this information it should still be possible to detect and show moments in the past weeks, where the user was obviously exposed to more devices (i.e. very likely more human contacts) than usual.
Would that be possible?
______________________ some additional info here ________________________
Why is it so important?
We have to be aware:
Backward contact tracing (source cluster detection) is be far more efficient in containing virus spread than forward contact tracing. This is because of the high dispersion of the virus.
Explanation:
We statistically know that 4 in 5 infected people are not infectious. They can not pass on the virus, because their virus load is not high enough in their upper respiratoral tract.
The whole virus spread happens, because 1 in 5 infected people can pass on the virus to others, these are the (super-)spreaders we need to detect.
In forward tracing a lot of energy is spent to identify the people the current case could have infected. But knowing that 4 of 5 don't pass on the virus means that in 80% of the cases this work is actually useless.
In backward tracing (source cluster detection) we try to find the event in which our current case caught the virus. It is pretty sure that in this event we find the 1 in 5 person that actually was infectious.
And quite possibly this event may have led to a couple of other infections that developed in parallel to our case, which all are very likely in the same stage themselfes now, 1 in 5 on them likely currently infectious.
Thus source cluster detection is far more efficient, because we are not spending our energy on situations that are only 20% relevant for the virus spead, but instead focus on situations where we can expect close to 100% relevance.

@fstandhartinger

Would that be possible?

No I don't think so:

Apple and Google (mainly Apple) have strict guidelines how Bluetooth can be used in the background if it's scanning for other devices (see the France Corona App which doesn't use the ENF, it has to be opened in the foreground to be active...)
The RPIs (Bluetooth Beacons your Handy records and sends) change every 15-20 minutes, so after this time there are only "new" Devices and you can not know if this is really a new device or just a device which changed it RPI...

@fstandhartinger unfortunately @Ein-Tim is correct afaik: CWA itself is not allowed to record BT signals see this thread for further discussion on this topic.
@Ein-Tim

The RPIs (Bluetooth Beacons your Handy records and sends) change every 15-20 minutes, so after this time there are only "new" Devices and you can not know if this is really a new device or just a device which changed it's RPI...

This is correct but probably wouldn't pose a problem if you just want to find out how many ENF devices are approximately near you at a given point in time. If you have a short recording window (say a few sec) the number of unique RPIs recorded in a given window will (in expectation) be very close to the number of ppl with ENF around.
@fstandhartinger
Regarding backward contact tracing / automated cluster detection: I agree that this is a very good and important idea from an epidemiological perspective (see also this paper via @pdehaye who has probably much more to say on this topic) but I think Google/Apple would have to implement such a functionality directly into ENF for it to work properly.

Thanks for the feedback! Is there any way you (or myself) can create a feature request for the Exposure Notification API?
I searched, but didn't find the forum where I could do that - maybe it's access is restricted to developers of the local warn apps?
Best regards Florian

As I didn't find a better way to contact google/apple on this I create a generic support ticket here:
https://support.google.com/android/thread/80101430?hl=en
Anyhow, if you know the correct place to post this feature request, please give me a hint. Thank you

@fstandhartinger

As I didn't find a better way to contact google/apple on this I create a generic support ticket here:
support.google.com/android/thread/80101430?hl=en

Upvoted 👍
Other than that there is the Google exposure-notifications-internals repo: https://github.com/google/exposure-notifications-internals/issues but the devs there suggested (in a different context) to address requests via the respective national ENF app teams who have a dedicated channel:

Hi,
Thanks for your question. Unfortunately this is not a great place to report or discuss such issues. The best advice I can provide is that these issues should be shared with the Public Health Authority/App developer in your region. Those partners. can then share that feedback with Google via their DevRel contacts.

Dear CWA devs, please use your DevRel channel to directly push this topic with Apple/Google.
Please be aware this may be a key change to the effectiveness of pandamic containment strategy.
Its cruical.
This is imporrtant. I can't stress this enough!
It's having huge potential to save lives!
Thank you

@fstandhartinger
There is also an e-mail address for Google
<EMAIL_ADDRESS>however I wouldn't expect too much from that e-mail address. I did once get a reply from them after I reported a bug that they had fixed but not yet released.
The app is published by the Robert Koch Institut however I haven't seen any direct involvement here from them, so I suspect you would need to get into direct contact with RKI for any major strategic addition to the CWA functionality. I may of course be wrong about this; I'm just a voluntary community member.

The app is published by the Robert Koch Institut however I haven't seen any direct involvement here from them

Me neither. They don’t even report usage statistics in their daily situation reports.
Very disappointing.

@alanrick

They don’t even report usage statistics in their daily situation reports.

The CWA app statistics are handled separately and they are reported on the RKI website weekly under Übersicht zu aktuellen und früherer Zahlen und Fakten zur Corona-Warn-App (ab 12.10.2020).
RKI could certainly do a better job about integrating the app reports into their other reports and site pages. The English language documents and German documents are inconsistent in the way they link into the app reports, so if you don't know your way around, you may not even find them on the RKI site.
These reports are however also manually copied into the App Blog.
The usage statistics from yesterday Nov 13, 2020 are on the RKI site here.

I know the statistics are available. It just appears that while the RKI is happy to remain the owner of this prestigious project it does not have an interest in its success and doesn't regard it as operationally significant.

Could we please have a clarification about this wishlist item and #240 & #355.
Is a cluster detection in any form possible with the currently used GAEN-Framework?

@dsarkar Is it the exact method that @fstandhartinger proposed at the start of this thread? If not, is there a user-story that we can read that describes the "Cluster Event" capability.

@alanrick Not that method described in the OP, which is actually with the current ENF impossible to implement as discussed by the community above. It is a QR code based check-in functionality. You will find PR in the iOS and Android repository named "Event Registration"

Thanks @dsarkar.
I guess I'd have to interpret the code in the Pull Request to figure it out for sure.  But would it be fair to say this is not semi-automated, but a manual cluster detection similar to the CrowdNotifier?
And unlike the Luca App, darfichrein... there is no additional connection to the local health center and only anonymous data is exchanged so it is not used for tracing.
I ask to avoid confusion about whether this complements Luca App/darfichrein, or whether it replaces them.

Just to add, the App mentions the site www.coronawarn.app/en/Screenshots as a means to discover what's coming in future releases, but, sadly, the "cluster event" isn't described.

@alanrick
Please see https://github.com/corona-warn-app/cwa-documentation/issues/547

Closing this issue as the related internal ticket is set to obsolete


Publish to Maven Central

Also fix the organization name.

https://issues.sonatype.org/browse/MVNCENTRAL-1375


When the OAuth token expires, refresh it, don't error

Code fix for https://github.com/wellcometrust/platform/issues/3809

:+1: Looks good!

💃
@helenst If you install v1.2.1 at some point, you'll pick up this fix.
I also added some instructions for the next time somebody wants to do a release.


Chat Prefixes are not Displaying at ALL in Chat

I have setup chat prefixes for every group, but it is not showing. The permissions seem to be working correctly, so I must be doing something right at least. For example, the chatmod group has the prefix &6[Chat-Mod] using the node prefix.3.&6[Chat-Mod]. However, when a chat-mod says something in the chat, it just has the default of <playerName>.  I am running the latest LuckPerms version for Nukkit built from source code. Build was successful. Is there another dependency that I should fulfill in order for chat prefixes to work (I don't think so)?

Read this: https://github.com/lucko/LuckPerms/wiki/Prefixes,-Suffixes-&-Meta#displaying-prefixes-and-suffixes

@TheKiller65YT Ah...I see. Is SVault supported? I couldn't find a Nukkit build for Vault. I have the same question for the chat plugins (EssentialsXChat, VaultChatFormatter, etc.) Thank you!

As long as prefix show in the command output /lp user USER info, luckperms is working. For nukkit chat plugin you can probably search up on their website I suppose.
https://cloudburstmc.org/resources/categories/nukkit-plugins.1/

@benwoo1110 It shows up with /lp user USER info, but not in chat. I suppose I'll have to got find some plugins then.


[TRACKER] New commands to add to the editor command palette

Godot version
master
System information
Any OS
Issue description
Godot now has a Command Palette. But only all the commands are not added to the palette.
Below are some commands that are currently in the Command Palette. Feel free to report which Commands(mostly shortcuts) you think should be added to the command palette.
Editor

[ ] editor/reload_current_project
[ ] editor/reopen_closed_scene
[ ] editor/unlock_selected_nodes
[ ] editor/ungroup_selected_nodes

Script Editor

[ ] script_editor/new
[ ] script_editor/open
[ ] script_editor/reset_zoom
[ ] script_edior/next_script
[ ] script_ediot/find_in_files
[ ] script_editor/previous_script

Steps to reproduce
look for Editor Settings > Shortcuts that need to be added into the command palette.
Minimal reproduction project
No response

I'd like to have a switch between the editor modes (2d/3d/script), open scene and start game and start current scene in there. Thanks!

Any chance to have a "Close All Open (Scripts/Scenes)" feature?

As a plugin dev I'd love to have all of them in the palette list (or as much as possible) and be able to run them from a plugin.
This would be a great feature, now unfortunately most of commands cannot be called from the plugin (without some ugly workarounds).

Export targets could be accessible in here. Ie Export . Open editor settings, open project settings.

in a mono build, I’d like a command that just Builds (compiles) the code without running the game if there’s no such shortcut already. This helps me with compilation error checking when I don’t have intellisense enabled or something like that.
That’d be fantastic. Thank you!

in a mono build, I’d like a command that just Builds (compiles) the code without running the game if there’s no such shortcut already. This helps me with compilation error checking when I don’t have intellisense enabled or something like that.
That’d be fantastic. Thank you!

Is this what the Build button does in the top-right corner of the editor?

Yes. Is there a keyboard shortcut for that? Or for now is that button the only way of compiling?

Yes. Is there a keyboard shortcut for that? Or for now is that button the only way of compiling?

According to the source code, there isn't any shortcut for it yet: https://github.com/godotengine/godot/blob/e95fa21b455a5e2c512fe6e054f86df1f01e5bd3/modules/mono/editor/GodotTools/GodotTools/GodotSharpEditor.cs#L420-L427
Please open a proposal for this on the Godot proposals repository.

from "editor" tab, tried to select must have shortcuts for palette
close scene
distraction free mode
open 2D editor
open 3D editor
open script editor
editor settings
export
quit
toggle fullscreen
new inherited scene
play
play custom
play custom scene
project settings
reload saved scene
save all scenes
save scene
save scene as
stop
take screenshot
visible collision shapes
visible navigation

i'm wondering. command-palette must support checkboxes/flags, for example: visible collision shape and navigation must have visual feedback for usability.

i'm wondering. command-palette must support checkboxes/flags, for example: visible collision shape and navigation must have visual feedback for usability.

That would be great. I know this feature from IntelliJ and it's pretty awesome.
Thinking about it, changing the theme would help for switching between dark and light mode. But there's no flags toggle currently for that, so this would expand command palette features to include select boxes as well.
That might be a too big feature set for the first version maybe.

For now, the command palette contains only 2 entries for the Script Text Editor:

Find...
Replace...

Is there any reason to not have all the menus in the script editor (File, Edit, Search, ...) like in vs-code? Commands like Move Up, Upper Case, ... would be nice to have.


Change API Key to match Java

Change API_KEY in gradle.properties to apiKey to match Java tutorials
Description
Java samples repo uses an API key in the gradle.properties in <userhome>/.gradle, just as Kotlin samples repo does. They do not, however, use the same property name or value.
The Java samples' property looks like this:
apiKey = <actual 100-character API Key here>

whereas Kotlin looks  ike this:
API_KEY = "<actual 100-character API Key here>"

Our new hire on the SDK docs team, Steve Rozic, found it quite confusing to end up  with gradle.properties file like this:
apiKey = <actual 100-character API Key here>
API_KEY = "<actual 100-character API Key here>"

He though it was an error and that one should be deleted. Of course, that would break the samples repo (and tutorials verification project) for the API Key that one deleted. Since the doc team covers for each other's vacations and other absences, it's important that any colleague can fairly easily fill in during those times, even in they are not completely familiar with the specific SDK.
To achieve that goal, I created this PR. It contains small changes in the build.gradle file of each tutorial app.
Links and Data

Java samples repo: https://github.com/Esri/arcgis-maps-sdk-java-samples

What To Review
This PR changes the following.

Changes this code:buildConfigField("String", "API_KEY", API_KEY)

to this:buildConfigField("String", "API_KEY", "\"$apiKey\"")

Updates the README to refer to apiKey instead of API_KEY.

Please evaluate these proposed changes, verify that they do not break the samples, and let me know if this is something you would contemplate doing. The PR works fine for me.

@shubham7109 Please consider this PR a proposal for your cosideration and Kaushik's too.
Thanks

Thanks, Mark for the helpful suggestion here. This would make cross-development with Java and Android less of a hassle. In the rare case, this is a requirement, the "fix" here could be helpful. But that being said I could also see this causing disruptions in automatic workflows and a manual edit of every user's global gradle.properties file when migrating from the Runtime SDK. After discussing this issue with Rama, I feel it would be best to look into prioritizing this in the future with all SDK platforms in mind regarding the usage of API keys.
I'll use this PR as a reference as instructions for any request to combine the API keys to match Java 👍


Exception not handled in train.py when --val_set is not provided

https://github.com/cjlin1/simpleNN/blob/ad58af428dd16f80e095fd2ac8a5adaedab81f9c/Python/train.py#L305-L308
If --val_set is not provided, val_data will be set to None, which will cause an exception like:
TypeError: 'NoneType' object is not subscriptable

This case is handled in the gradient_trainer function.

Hi,
Thanks for your interest in trying our work. The exception is now handled. For your information, if no validation set is provided during the training phase, we will store the model in the last iteration for future prediction.
Best,
Pengrui


test(dtslint): add timeout

Description:
This PR adds dtslint tests for timeout.
Related issue (if exists): #4093

Pull Request Test Coverage Report for Build 7339

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 96.967%

Files with Coverage Reduction
New Missed Lines
%

src/internal/observable/fromPromise.ts
1
87.5%

src/internal/observable/fromIterable.ts
2
95.24%

src/internal/observable/fromObservable.ts
2
83.33%

Totals

Change from base Build 7318:
0.2%

Covered Lines:
5786

Relevant Lines:
5967

💛  - Coveralls


Add overrides for LoadMany to support List as opposed to T[].

I vote extension methods as well. I'm going to say again that I'm really not enthusiastic about these going into our core.

@jeremydmiller I agree, that this is something that probably our user could write on their own (eg. by adding the extensions methods on their codebase), but if that helps the migration for the RavenDB users (that I assume were our initial userbase) then I think that it's fine. I think that if it's in the different namespace than the original interface then it shouldn't be a huge maintenance burden.

I echo the same thought as @jeremydmiller  and @oskardudycz . We could possibly look at adding a docs section to outline these so that users porting code base from RavenDB to Marten.
@jeremydmiller @oskardudycz should we close this PR or wait to hear from @apereiratl?

I'd propose to wait a few days and see if @apereiratl still would like to come up with the extension methods solution.

Sorry I've been a bit busy but will convert them to extension methods and resend the patch.

@apereiratl there is no rush on that. We just wanted to know if we may expect the updates in the nearest future 👍

@apereiratl we are planning to release v3.12.0 on 8th, is it something which you could look to squeeze in?

@jeremydmiller @mysticmind I checked and unfortunately, it won't work with extension methods. Params methods will still take precedence and IEnumerable or IList will be used.
I think that it's fine to add such methods, as it's a common error of wrongly passing IEnumerable - especially in Append events function.
Same case for https://github.com/JasperFx/marten/pull/1485 provided by @trullock.
After consideration, I'm fine with that (however our interface will be even bigger). Probably suggestion made by @trullock to make the IEnumerable default method and params extension method would also work, but such a switch can be made in v4.
I can add necessary changes.
Thoughts?

I don't think this adds a huge amount of value to worry about what's an extension method and what's not. I say just add the overloads to LoadMany() and call it a day.
I do want to reevaluate IDocumentSession in v4, but there I'm more worried about moving the plain text searching out of the root interface.

I was also referring to other places where we have params [] eg. Store, Append events etc.
I think that having a smaller interface and implementation and overloads as Extension methods might make it easier to maintain and less prone for breaking changes.
So if there are no votes against I’ll da overloads and we’ll evaluate removing some of them in V4

@oskardudycz Gotcha, so just that. It's a breaking binary change, so you have to leave the changes to thin it down for v4

@jeremydmiller sounds good to nr 👍

@jeremydmiller @mysticmind @jokokko I cherrypicked @trullock changes, aligned both implementations and added other overloads eg. to Insert, Update and added for EventStore.
Could you verify if you're fine with that?


Problem working from postgis to geopandas to trajectory

I'm trying to use moving pandas to process GPS data stored on a PostGIS database.
I'm connecting to PostGIS with psycopg2
I load a table to a geodataframe with geopandas.
I loaded the geodataframe to trajectories and if I print the collection I get
TrajectoryCollection with 34 trajectories so it seems to be ok
I get the first trajectory with my_traj = traj_collection.trajectories[1]
Problem 1)
If I print (my_traj) I get an error
AttributeError: 'DataFrameGroupBy' object has no attribute 'geometry'
On PostGIS my geometry column is named geom, I tried to rename the geom to geometry on the geodataframe prior to creating the TrajectoryCollection, It seems to work but then I get two geomemetry fields, geom and geometry
If I print the trajectory again, I get this error.
AttributeError: 'numpy.int64' object has no attribute 'to_pydatetime'
Problem 2)
If I try to store speed with
traj_collection.add_speed(overwrite=True)
I get an error message, the last line is
AttributeError: 'float' object has no attribute 'total_seconds'
Could you help me to solve this problem?
Thanks!

Hi Jose. Please include your code to reproduce both issues. I'll then give it a try. In both cases, it seems like the GeoDataFrame columns are not what MovingPandas expects. Please also provide the schema of your PostGIS table and a small data sample exported to CSV which I can import into my local database.

Hello I found the solution, you were right the GeoDataFrame columns weren't right,
I changed the sql to get the data, instead of import geom column and changing the name of the column in geodataframe, now I selected the geom column as geometry.
Also imported the datetime column as t, instead of fecha (date in Spanish)
sql_puntos = "SELECT loc_gps.geom AS geometry, loc_gps.fecha AS t , id_barco_interno, loc_gps.id_dispositivo_gps FROM loc_gps WHERE fecha >  now() - interval '1 day'  ORDER BY loc_gps.id desc;"
Also, I wasn't defining the index properly.
Now it seems to run properly.
Thanks for the advice!

Great to hear that it works now.

Hi @anitagraser,
I have the same problem but If I try to rename the geom column to geometry as @bigcarto did I got an error in GeoPandas
single_animal = "select geom, gps_data_animals_id, animals_id, acquisition_time as t from main.gps_data_animals where gps_data_animals.animals_id=10 and geom is not null;"
animal_df = gpd.GeoDataFrame.from_postgis(single_animal, con)
traj = mpd.Trajectory(animal_df, 1)
traj.get_geom_column_name()
  'geom'
traj.get_bbox()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-7-76341803138b> in <module>
----> 1 traj.get_bbox()

~/gitfem/euromammals/venv/lib/python3.9/site-packages/movingpandas-0.7rc1-py3.9.egg/movingpandas/trajectory.py in get_bbox(self)
    345             Bounding box values (minx, miny, maxx, maxy)
    346         """
--> 347         return self.to_linestring().bounds  # (minx, miny, maxx, maxy)
    348
    349     def get_start_time(self):

~/gitfem/euromammals/venv/lib/python3.9/site-packages/movingpandas-0.7rc1-py3.9.egg/movingpandas/trajectory.py in to_linestring(self)
    249         """
    250         try:
--> 251             return point_gdf_to_linestring(self.df)
    252         except RuntimeError:
    253             raise RuntimeError("Cannot generate LineString")

~/gitfem/euromammals/venv/lib/python3.9/site-packages/movingpandas-0.7rc1-py3.9.egg/movingpandas/trajectory.py in point_gdf_to_linestring(df)
    788     """
    789     if len(df) > 1:
--> 790         return df.groupby([True] * len(df)).geometry.apply(
    791             lambda x: LineString(x.tolist())).values[0]
    792     else:

~/gitfem/euromammals/venv/lib/python3.9/site-packages/pandas/core/groupby/groupby.py in __getattr__(self, attr)
    750             return self[attr]
    751
--> 752         raise AttributeError(
    753             f"'{type(self).__name__}' object has no attribute '{attr}'"
    754         )

AttributeError: 'DataFrameGroupBy' object has no attribute 'geometry'

when I try to rename geom in geometry I get error on GeoDataFrame
single_animal = "select geom as geometry, gps_data_animals_id, animals_id, acquisition_time as t from main.gps_data_animals where gps_data_animals.animals_id=10 and geom is not null;"
animal_df = gpd.GeoDataFrame.from_postgis(single_animal, con)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-8-2a9a8c0b83ba> in <module>
      1 single_animal = "select geom as geometry, gps_data_animals_id, animals_id, acquisition_time as t from main.gps_data_animals where gps_data_animals.animals_id=10 and geom is not null;"
----> 2 animal_df = gpd.GeoDataFrame.from_postgis(single_animal, con)

~/gitfem/euromammals/venv/lib/python3.9/site-packages/geopandas/geodataframe.py in from_postgis(cls, sql, con, geom_col, crs, index_col, coerce_float, parse_dates, params, chunksize)
    657         """
    658
--> 659         df = geopandas.io.sql._read_postgis(
    660             sql,
    661             con,

~/gitfem/euromammals/venv/lib/python3.9/site-packages/geopandas/io/sql.py in _read_postgis(sql, con, geom_col, crs, index_col, coerce_float, parse_dates, params, chunksize)
    161             chunksize=chunksize,
    162         )
--> 163         return _df_to_geodf(df, geom_col=geom_col, crs=crs)
    164
    165     else:

~/gitfem/euromammals/venv/lib/python3.9/site-packages/geopandas/io/sql.py in _df_to_geodf(df, geom_col, crs)
     63
     64     if geom_col not in df:
---> 65         raise ValueError("Query missing geometry column '{}'".format(geom_col))
     66
     67     geoms = df[geom_col].dropna()

ValueError: Query missing geometry column 'geom'

However with geom column I'm able to plot the date with plot() and hvplot() functions

I cannot use geom as geometry because I get the error reported before. Probably I should rename the column names in the table but I cannot modify it

I also have AttributeError: 'numpy.int64' object has no attribute 'to_pydatetime' error when I run get_start_time or get_end_time.
It seems to me that traj.df.index return the index number and min() is returning the first index, I have the t column
AttributeError                            Traceback (most recent call last)
<ipython-input-23-233768a2683f> in <module>
----> 1 traj.get_start_time()

~/gitfem/euromammals/venv/lib/python3.9/site-packages/movingpandas-0.7rc1-py3.9.egg/movingpandas/trajectory.py in get_start_time(self)
    356             Trajectory start time
    357         """
--> 358         return self.df.index.min().to_datetime()
    359
    360     def get_end_time(self):

AttributeError: 'numpy.int64' object has no attribute 'to_pydatetime'

Ah "It seems to work but then I get two geomemetry fields, geom and geometry" ... so gpd.GeoDataFrame.from_postgis() does some weird things.

I have the t column

Great! Now just set the index of the GeoDataFrame
animal_df.set_index('t', inplace=True)

Probably I should rename the column names in the table but I cannot modify it

The GeoDataFrame geometry column can be renamed:
animal_df.rename(columns={'geom':'geometry'}, inplace=True)

@lucadelu did you get it to work now?

@anitagraser yes, renaming geom column and set t as index works!
Thanks a lot

Renaming geom in geometry brakes geodataframe in Trajectory and  hvplot doesn't work any more (but get_bbox work properly).
Applying your patch everything works fine (just to rename index, but now a error message appear and it help a lot)

Ok, please open a new ticket if you encounter any more issues with the development version.


Unit tests

Fixes #83
@lukevp  I think this is a start, please check it out so a good pattern can be established.

You're welcome @lukevp


Add ability to add custom expiration to links

Allow expires_at argument to LoginToken.encode and handle the same in LoginToken.decode.
The default configuration of Devise.passwordless_login_within is respected, when expires_at is nil.
Addresses: #18
This is similar to #19 but less open.

@bvsatyaram  good job!
@abevoelker tested that in local and in our staging app, looks good

Thank you very much for your contribution! The option to provide an :expires_at key for token encoding[^1] has been implemented for the 1.0 release, and I've credited you in the CHANGELOG. :tada: :bowing_man:
[^1]: This logic now lives in the SignedGlobalIDTokenizer::encode and MessageEncryptorTokenizer::encode methods, now that the tokenization process is configurable


♻ refactoring: moving from okp4 to axone

Purpose
This PR is part of the initiative to migrate repositories from the OKP4 organization to the Axone Protocol organization on GitHub.
Scope

[x] configure Upptime for Axone
[x] update CI
[x] create GH_PAT secret on repository (once migrated)
[ ] configure DNS for status.axone.xyz to point to the GitHub page

Summary by CodeRabbit

New Features

Updated project URLs and ownership details to reflect the transition from OKP4 to Axone Protocol.

Documentation

Revised README.md to align with the Axone Protocol, updating URLs, badges, and references.

Chores

Modified GitHub workflow tokens and project URLs to match the new Axone Protocol settings.
Updated ownership, site URLs, themes, logos, and footer links in configuration files to reflect the Axone Protocol branding.
Changed copyright holder in the LICENSE file to "Axone.xyz."

CI will require the GH_TOKEN (https://upptime.js.org/docs/get-started#add-repository-secrets), which will be created once the repository is moved to the axone-protocol organization.

OK great, it seems that the upptime bot has generated the status of the various servers, so we're ready to merge. I'll do the DNS redirection next to point to the new Axone status page.


chore: update vendored pyproject-metadata

[x] Make sure it works
[x] Support PEP 639
[ ] Support validation
[x] Support multierrors

Note-to-self: should have marked the Fedora license as Apache-2.0 AND MIT:
https://github.com/scikit-build/scikit-build-core/blob/f0ae31922ff802b179fce3327a9c93ff65934a23/.distro/python-scikit-build-core.spec#L16
If we devendor it in Fedora we can reduce it. Probably the pyproject.toml should also be adapted, but I'm not sure how well PyPI supports dual licensing?

PEP 639 covers dual licensing. :) Not sure when it will be common enough to use, though.

I'm thinking what to do in this case and if we should be de-vendoring on the Fedora side. Would adding a check against importlib.metadata.version be ok with you?

@henryiii I've added db4160bbcc2d8381ce5d43d9fc40ca6f7d091f15 that clarifies and tracks the vendoring of pyproject-metadata. Feel free to move this commit when refactoring

FWIW, I have permission from the original copyright holders to relicense under the same license as packaging.

Packaging is out. So this is unblocked, will finish it up soon.


[#issue] Fix PMD issue

Codecov Report

Merging #3849 into master will decrease coverage by <.01%.
The diff coverage is 84%.

@@            Coverage Diff             @@
##           master    #3849      +/-   ##
==========================================
- Coverage   40.94%   40.94%   -0.01%
==========================================
  Files        2186     2186
  Lines       71055    71063       +8
  Branches     9763     9763
==========================================
+ Hits        29095    29098       +3
- Misses      39200    39203       +3
- Partials     2760     2762       +2

Impacted Files
Coverage Δ

.../navercorp/pinpoint/rpc/cluster/ClusterOption.java
51.42% <ø> (ø)
:arrow_up:

...ofiler/sender/planer/SpanStreamSendDataPlaner.java
0% <0%> (ø)
:arrow_up:

...point/profiler/instrument/ASMClassNodeAdapter.java
71.12% <0%> (ø)
:arrow_up:

...avercorp/pinpoint/bootstrap/util/NetworkUtils.java
58.82% <0%> (ø)
:arrow_up:

...r/zookeeper/ZookeeperClusterDataManagerHelper.java
78.18% <100%> (ø)
:arrow_up:

.../profiler/sender/SpanStreamSendDataSerializer.java
84.37% <100%> (ø)
:arrow_up:

...ent/classreading/DefaultInternalClassMetadata.java
64.44% <100%> (+3.46%)
:arrow_up:

...ument/classreading/DefaultSimpleClassMetadata.java
92.59% <88.88%> (+1.28%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
80.59% <0%> (-2.99%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


merge from template-sync/template-arlac77-github

.github/pr_labeler.yml

chore(labels): merge from template .github/pr_labeler.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
54

Relevant Lines:
56

💛  - Coveralls


Test CI; continue with correctness section of appendix

CI was running exceptionally slow, but that was just a temporary issue on the GH backend.  Closing.


Feat/add node modules, add node css/sass scripts

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR closes the discussed issue of outsourcing the fonts/font-face declarations and separate them in an own CSS file for

a, better understanding and transparency
b, performance since these assets can be used over all components

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[X] Build (npm run build) was run locally and any changes were pushed
[X] Lint (npm run lint) has passed locally and any fixes were made for failures

Changes
Changes in this pull request:
Node modules added to provide CSS fonts output target

fs-extra
clean-css-cli
sass

Node tasks added and reordered

build:css
clean
css:minify
css:sass
enhanced the existing build task## Preflight Checklist

Updated Stencil configuration

Updated the Stencil configuration so a new CSS font-face declaration file gets generated, minified and added to the collection output target dist/collection/fonts which is part of the npm distribution package
Refactored CSS
updated .gitignore to exclude css dir

Browsers
I tested the build on the following browsers:

[X] Firefox Desktop
[X] Chrome Desktop
[X] Safari Desktop
[ ] Firefox Mobile
[ ] Chrome Mobile
[ ] Safari Mobile
[ ] Opera
[X] Brave

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

@feerglas What would you suggest, how I should adapt the stencil:start script "start:stencil": "stencil build --prod --watch --docs", with the new nesting of the build scripts? Somehow we need to pass in the the --watch property ... either in the existing scripts so we don't duplicate or on the otherhand add
    "start:css": "npm run css:sass && npm run css:minify",
    "start:stencil": "npm-run-all --sequential clean start:css start:stencil:itself",
    "start:stencil:itself": "stencil build --prod --docs --watch",

then we might need also start/watch scripts for css:minify and css:sass
What do you think?

Codecov Report

Merging #833 (8ff003e) into master (0a2131b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #833   +/-   ##
=======================================
  Coverage   45.00%   45.00%
=======================================
  Files          13       13
  Lines         571      571
  Branches      135      135
=======================================
  Hits          257      257
  Misses        291      291
  Partials       23       23

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 0a2131b...8ff003e. Read the comment docs.