IPGeolocationAPI.com a free geolocation service

IPGeolocationAPI.com provides a JSON API for geolocating website visitors or geolocating IPv4/IPv6 addresses. The returned JSON response contains detailed information about the geolocated country details like ISO 3166 (countries and states/subdivisions ), ISO 4217 (currency), and E.164 (phone numbers).
The service also has a self-configuring "Deploy to Heroku" button so the API can be self-hosted by anyone.
The service has been tested to handle thousands of asynchronous requests and coded in Python. Fully open source and MIT licensed. Github
The service with documentation is available at: IP Geolocation API
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Hi @madisvain , thanks for the PR. The description is a bit long, could you please cut it down to something along the lines of "geocoding based on IP address" ?

@davemachado thank you for the feedback. I have made the description shorter to be in the range of other descriptions.

Thanks!


(FACT-894) Update per-user external facts location

Changes from ~/.facter/facts.d to ~/.puppetlabs/opt/facter/facts.d, as
per the puppet-specification.

Windows changes are blocked by CFACT-272.

CLA signed by all contributors.

Changed to a merge of the cfacter 0.4.x branch, in case there are any more changes later.

Handling merge-up from CFacter#0.4.x to Facter#master as part of CFACT-269.

CLA signed by all contributors.


Re-write conditions to work with kubectl wait

Closes #27

 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.

Will sync with Alex to see if kubectl wait use case is failing with the status interface on master or if I'm not using kubectl wait correctly


How can I save the pcd file of the map generated?

I have a question, how do I create and save maps?
I would appreciate it if you could help me！

Duplicate of #43

save map code ; wechat 394467238


feat(nextra): export compile.tsx

compileMdx is a pretty useful function for me, but unfortunately, it is not exported by Nextra, and I use this function with next-mdx-remote
P.S. also I found a way to remove dummy context.js, data.js, index.js, locales.js and ssg.js files by configuring the exports field in package.json

The exports field is much cleaner, thank you! I've been thinking about the ability to compile remote MDX data in Nextra too, and exporting compile is definitely a required step. However I'm blocked at the page map thing, which is a bit tricky. Anyway I'll merge this first!


Added fping to base.yaml

We use fping in one of our projects. Nice way to ping multiple hosts very fast.
Package sources (I only tested ubuntu) but i think they should be correct.
arch
debian
fedora
gentoo
macports
ubuntu

@HEllRZA on your project do you use macports? Otherwise I might leave it off or prefer the homebrew formula for MacOS dependencies.

I don't need macports.


No implementation found for java.nio.ByteBuffer

I have successfully compiled my project with 'shapedrawer' and generated the apk file.
When run the application it gives error "No implementation found for java.nio.ByteBuffer ".
E/art: No implementation found for java.nio.ByteBuffer com.badlogic.gdx.graphics.g2d.Gdx2DPixmap.newPixmap(long[], int, int, int) (tried Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap and Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap___3JIII)
No matter which variable i use "com.badlogic.gdx.graphics.Pixmap", "com.badlogic.gdx.graphics.Texture" , "com.badlogic.gdx.graphics.g2d.Batch"   it crashes giving the "No implementation found"
How do i resolve this? Please help.

Do you have a reproducible example? I'm not sure this has anything to do with shape drawer.

I just created a new project and compiled with 'shapedrawer' alone. In my main activity  i just added the line below inonCreate()function
Pixmap pixmap = new Pixmap(10, 10, Pixmap.Format.RGBA8888);
I am getting the error when i run the code
E/hapedrawertria: No implementation found for java.nio.ByteBuffer com.badlogic.gdx.graphics.g2d.Gdx2DPixmap.newPixmap(long[], int, int, int) (tried Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap and Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap___3JIII) D/AndroidRuntime: Shutting down VM E/AndroidRuntime: FATAL EXCEPTION: main     Process: com.example.shapedrawertrial, PID: 7193     java.lang.UnsatisfiedLinkError: No implementation found for java.nio.ByteBuffer com.badlogic.gdx.graphics.g2d.Gdx2DPixmap.newPixmap(long[], int, int, int) (tried Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap and Java_com_badlogic_gdx_graphics_g2d_Gdx2DPixmap_newPixmap___3JIII)         at com.badlogic.gdx.graphics.g2d.Gdx2DPixmap.newPixmap(Native Method)         at com.badlogic.gdx.graphics.g2d.Gdx2DPixmap.<init>(Gdx2DPixmap.java:122)         at com.badlogic.gdx.graphics.Pixmap.<init>(Pixmap.java:118)         at com.example.shapedrawertrial.MainActivity.onCreate(MainActivity.java:15)         at android.app.Activity.performCreate(Activity.java:7224)         at android.app.Activity.performCreate(Activity.java:7213)         at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1272)         at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2926)         at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3081)         at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:78)         at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)         at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)         at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1831)         at android.os.Handler.dispatchMessage(Handler.java:106)         at android.os.Looper.loop(Looper.java:201)         at android.app.ActivityThread.main(ActivityThread.java:6823)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:547)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:873)
P.S: i am using android studio 3.5.3

There are no lines that come from ShapeDrawer in your stack trace. I don't know how you've designed your application, but if you aren't using the libGDX application structure, it's going to be a challenge to use a library that's tailor-made for use in libGDX applications. See https://stackoverflow.com/questions/58295075/how-do-i-solve-this-error-libgdx-error-stage-creation#comment102984303_58295075 for a similar error that resulted from not using the libGDX framework but still trying to use something that depended on libGDX.

I think tommyettinger has probably guessed what's going on correctly - this is supposed to be used with libgdx.


Enhance OMIS order search

This adds the ability to:

search for 'reference', 'total_cost', 'contact name', 'company name' using the global search view
search for 'reference', 'total_cost', 'contact name', 'company name' using the specialised search view
filter orders by the 'reference', 'total_cost', 'contact name', 'company name' fields

The way ES is set up during tests has changed as well.
All the ES records are deleted after each test instead of deleting the index itself every time.
This speeds the tests up quite a bit.

Codecov Report

Merging #555 into develop will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #555      +/-   ##
===========================================
- Coverage     96.5%   96.48%   -0.02%
===========================================
  Files          132      132
  Lines         3944     3922      -22
  Branches       360      356       -4
===========================================
- Hits          3806     3784      -22
  Misses          95       95
  Partials        43       43

Impacted Files
Coverage Δ

datahub/search/omis/views.py
100% <ø> (ø)
:arrow_up:

datahub/core/validate_utils.py
92.3% <0%> (-0.55%)
:arrow_down:

datahub/search/interaction/serializers.py
100% <0%> (ø)
:arrow_up:

datahub/search/investment/serializers.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


⚠ Bofrost.fr has degraded performance

In bd89c9f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1352 ms

Resolved: Bofrost.fr performance has improved in b855cb6 after 6 minutes.


coordinate

Hello,
How to get the coordinates of the first touch on the screen ? i see you use PanResponder, maybe with this feature it's possible to get this data ?
Thanks a lot

I got the solution, onStrokeStart :)


503759_ss22f32 lids

None of the lid files will open in Cura, it says they are corrupted or don't contain models. All other files in the directory work normally.

I can't replicate the issue myself.
It seems the file might have been corrupted upon download.
Maybe try downloading the files from either: printables, cults3D or thingiverse.
I am unable to replicate this issue myself.


Updated Biomart exports and added HCOP orthologs as an alternative

Updated Biomart exports
Added HCOP orthologs as an alternative to biomart https://www.genenames.org/tools/hcop/
Version bump from 0.1.3 -> 0.1.4
Little bit of code refactoring

Maybe that's also useful for others how are using this library.

This looks awesome; I will merge, build, and push the new version to PyPI.
Thanks for the improvements!


feat: Use node-fetch for maximized compatibility

Summary by Sourcery
Use node-fetch for HTTP requests in the Veganify API wrapper, refactor error handling, and update the build script to use ES module syntax. Enhance tests with improved error assertions and update Jest configuration for ES module support.
New Features:

Introduce the use of node-fetch for making HTTP requests in the Veganify API wrapper.

Enhancements:

Refactor error handling in API methods to throw standard Error objects with detailed messages.
Update the build script to use ES module syntax and improve file copying logic for non-TypeScript files.

Build:

Modify the build script to use ES module imports and handle file operations with Node.js built-in modules.

Tests:

Refactor tests to use rejects.toThrow for error assertions, improving test readability and reliability.
Update Jest configuration to support ES modules and adjust module name mapping for .js extensions.

@sourcery-ai review

https://github.com/sourcery-ai review

@sourcery-ai review


post_install() isn't run

Recently I added the https://github.com/termux/termux-packages/blob/ab539a2c2fa5b10757193d148841b479bf485b72/packages/pueue/build.sh package to Termux.
post_install() doesn't appear to run when I install this. I don't get :: Running post-transaction hooks....
$ pacman --sync pueue
warning: pueue-3.4.1-0 is up to date -- reinstalling
resolving dependencies...
looking for conflicting packages...

Packages (1) pueue-3.4.1-0

Total Installed Size:  10.30 MiB
Net Upgrade Size:       0.00 MiB

:: Proceed with installation?  My Pacman config is fairly vanilla.

The post_install() function is only run when the package is fully installed on the device, i.e. it will not be run when the package is reinstalled or updated.
To make the post_install() function run after reinstalling or upgrading a package, you must use the post_upgrade() function. Here's an example to help you understand how to implement this in the Termux package: https://github.com/termux-pacman/termux-packages/blob/master/packages/python-pip/build.sh#L56


[Feature][Performance] Improve dimension size of all large images

See below for warning from console.
NG0913: An image with src http://localhost:4200/assets/facebook-logo.png has intrinsic file dimensions much larger than its rendered size. This can negatively impact application loading performance. For more information about addressing or disabling this warning, see https://angular.io/errors/NG0913. Find more at https://angular.io/errors/NG0913

Compare load times before and after changes.


Security fixes

This PR is all the security fixes and improvements I have opened PRs for to upstream: https://github.com/hound-search/hound/pulls/kinghrothgar
Hopefully they all get merged and we can switch to just using the images from the main releases. I will also being doing another PR after this that deals with as many npm packages vulnerabilities as I reasonably can.

Ignore the workflow failures. This image will be manually build and pushed for now (how it was done will be documented). If we have to accept that there won't be upstream merges, then I believe I will start personally maintaining a permanent fork and we can put the images that emit out of that in docker-sync.


Added rule to map 404 errors the 404 page

This could probably be made into a setting so that it could be remapped from the UI but this at least makes the default 404 page that comes with the template more usable.

Thanks!  I manually merged this into 4.6.


Clean up wl_callback resource in ViewBackend::dispatchFrameCallback()

After calling wl_callback_send_done(), the callback wl_resource object
must be destroyed. This effectively avoids leaking these objects.

Cherry picked to the wpebackend-fdo-1.0 branch as commit bf9de05174f27b6dfad1ed1353534643bd9000fe.


Cannot find normal binary files on the MAC of Apple m1 chip

Downloading node-libpng prebuilt binary form "https://github.com/Prior99/node-libpng/releases/download/0.2.20/node-libpng-darwin-arm64-93.node"
Noe supported node-libpng

I can't add support for this platform as I do not own the hardware.
Apple makes it notoriously complicated to build software for their hardware.
I would welcome a pull request that adds support for automated builds for this machine.

Automated builds not enough. I have built library on my  M1 mac and not all the tests passed and have no idea how to fix them
 PASS  src/__tests__/test-xy.ts
 PASS  src/colors/__tests__/test-index.ts
 PASS  src/colors/__tests__/test-rgba.ts
 PASS  src/colors/__tests__/test-gray-scale.ts
 PASS  src/colors/__tests__/test-palette.ts
 PASS  src/__tests__/test-rect.ts
dyld[47348]: missing symbol called
 PASS  src/colors/__tests__/test-gray-scale-alpha.ts
 PASS  src/__tests__/test-is-png.ts
 PASS  src/__tests__/test-encode.ts
 PASS  src/colors/__tests__/test-rgb.ts
dyld[47347]: missing symbol called

 RUNS  src/__tests__/test-png-image.ts
 RUNS  src/__tests__/test-decode.ts

Test Suites: 10 passed, 10 of 12 total
Tests:       151 passed, 151 total
Snapshots:   21 passed, 21 total

I'm running into the same issue every time I run yarn install to install dependencies for Redocly. Mac M1 2021, running Ventura 13.3.1, Node 18.13.0.
Any chance of getting this fixed? As far as I know, Apple isn't putting out any new Macbooks with Intel chips, which means that anyone using a newer Mac with Apple silicon who is working on a Node project that requires this package is going to have a bad time.
I see that the reason this hasn't been addressed is due to lack of Apple hardware. I tried to build on my machine just now, but of course the install.js is throwing me an error that there's no supported version of this library for Node 18.13.0 on Darwin (arm64). If there's any way I might be able to help, I'm all ears, as I'm sure that I'm going to run into more projects that are affected.


Game state / settings aren't being saved on share page player

Repro:

open: https://makecode.com/_L0hMMp43RAam
notice the score going up
if you restart the game, the score starts from zero whereas it should start from the previous score

If you open the game in the editor, it works. The score persists across runs.
If you load on meowbit, it also works.

another bug (from @riknoll): falsiness check before printing input

@darzu extension bugs needs to be filed in the extension repo. This is not a product bug.

Can you transfer to the right repo?

Or the bug is in the platform?

The extension is http://github.com/Microsoft/pxt-settings-blocks, but this should repro with hi score as well / any other settings that are saved (I think brightness and sound when set in the menu)? I think issue is that we persist it settings in the editor as a hidden file in the project and so it wouldn't actually save that in the share page itself

@abchatra , this is a platform regression. There's nothing you can do in the settings extension to fix this.

The code on the share page must have regressed at some point. This happens outside the iframe, so like Daryl said this fix needs to go into pxt-arcade.


The results of sink/transformer/windowed under outputs_*/ folders are all the same

Hi,
The results you gave here are problematic. Under the same directory, the results of 3 different attention types are identical.

Hello!
They indeed start equivalent for the first 1024 tokens, but they differ after that. This is because the windowed and attention_sinks approaches in the benchmarks use a window size of 1024, before which the three approaches are identical. See for example token 4000 for evidence that the results are not identical:

attention_sinks at 4k
windowed at 4k
transformers at 4k

Also, these figures in the README are direct plots of these .csv files:

As you can see, they're not identical.
I hope that clears it up!

Tom Aarsen

Oh my mistake, thank you for your reply. Plese feel free to delete this issue!

No worries!


Fixed #32975 -- Fixed admin system check for inlines with foreign keys to proxy models.

https://code.djangoproject.com/ticket/32975

@felixxm thanks for the review. Added your improvements to the PR.

@taliraj Thanks :+1: Welcome aboard :boat:


Add evaluation for GermanQuAD

https://www.deepset.ai/germanquad
Dataset: https://huggingface.co/datasets/deepset/germanquad

I think Konstantin worked on this already. See https://github.com/OpenGPTX/lm-evaluation-harness/tree/germanquad

done and merged into master https://github.com/OpenGPTX/lm-evaluation-harness/pull/56


Mid-paragraph push/pull spans not working

Hanging punctuation no longer works mid paragraph:

Captured on Chrome Version 78.0.3904.70 (Official Build) (64-bit)
Here's the breaking change to fix this courtesy of MB:
.pull-double,
.push-double,
.pull-single,
.push-single {
display: inline-block
}

FWIW my proposed fix is to put inline-block on the push classes, but not the pulls.

Thanks @mbutterick, I now updated the documentation.

Updated grunt-typeset to v0.0.8


IdField property not working in Node data-annotation

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
A specific property is not recognized as the ID while trying to implement the Node interface using the IdField in the data-annotation.
Background: I'm trying to use the query: node(id: "HashedSerial")  in a model that doesn't have a property call Id nor ExampleModelId
    [ExtendObjectType(typeof(ExampleModel))]
    [Node(IdField = nameof(ExampleModel.Serial))]
    public class ExampleModelNode
    {
        [NodeResolver]
        [BindMember(nameof(ExampleModel.Serial))]
        public async Task<ExampleModel?> GetExampleModelBySerialAsync(
            long serial,
            ExampleModelBySerialDataLoader exampleModelBySerial,
            CancellationToken cancellationToken)
            => await exampleModelBySerial.LoadAsync(serial, cancellationToken);

The application compiles but in execution time it throws the following error in the output console:
The specified id field Serial does not exist on ...ExampleModelNode.

It seems that is trying to find the Serial property in the ExampleModelNode class and not in the ExampleModel class.
I tried to add this property and bind it to the model but it doesn't work too:
    [ExtendObjectType(typeof(ExampleModel))]
    [Node(IdField = nameof(ExampleModel.Serial))]
    public class ExampleModelNode
    {
        //[BindMember(nameof(ExampleModel.Serial))] with this it doesn't work too
        public string? Serial { get; set; }

        [NodeResolver]
        [BindMember(nameof(ExampleModel.Serial))]
        public async Task<ExampleModel?> GetExampleModelBySerialAsync(
            long serial,
            ExampleModelBySerialDataLoader exampleModelBySerial,
            CancellationToken cancellationToken)
            => await exampleModelBySerial.LoadAsync(serial, cancellationToken);

The error is not appearing anymore in the console but the query result always have the Serial with value zero.
After one comment in the slack channel, I also tried to create a property call Id inside the ExampleModelNode class:
    [ExtendObjectType(typeof(ExampleModel))]
    [Node(IdField = nameof(ExampleModelNode.Id))]
    public class ExampleModelNode
    {
        [BindMember(nameof(ExampleModel.Serial))]
        public string? Id { get; set; } // Also tried:  public long Id { get; set; }

        [NodeResolver]
        [BindMember(nameof(ExampleModel.Serial))] // Also tried with/without this data annotation
        public async Task<ExampleModel?> GetExampleModelBySerialAsync(
            long serial,
            ExampleModelBySerialDataLoader exampleModelBySerial,
            CancellationToken cancellationToken)
            => await exampleModelBySerial.LoadAsync(serial, cancellationToken);

I'm getting the same result as before.
It is possible that the IdField or BindMember annotation are not working as expected?
Steps to reproduce

Create a model with a property called Serial.
Create the Node for this model in any of the ways I described before.
Create the Resolver in any of the ways I described before.
Verify that you get the explained behavior for each scenario.

Query to run
query  {
  exampleModelBySerial(serial: 1) {
    serial
    id
 }
}

Expected Result
{
  "data": {
    "exampleModelBySerial": {
      "serial": 1,
      "id": "<hashedSerial>",
 }
}

Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
12.0.1

Just to get this right, your model contains a string property called Serial, which should be rewritten as the ID field, but it should still be called serial in the schema, right?
Also I'm not sure what you are attempting with the BindMember on the NodeResovler, what is your goal here?
I would expect it to work like the following:
[ExtendObjectType(typeof(ExampleModel))]
[Node(IdField = nameof(ExampleModel.Serial))]
public class ExampleModelNode
{
    [NodeResolver]
    public async Task<ExampleModel?> GetExampleModelBySerialAsync(
        long serial,
        ExampleModelBySerialDataLoader exampleModelBySerial,
        CancellationToken cancellationToken)
        => await exampleModelBySerial.LoadAsync(serial, cancellationToken);
}

Hello Tobias, thanks you for replying also in the slack channel too.
Just to get this right, your model contains a string property called Serial, which should be rewritten as the ID field, but it should still be called serial in the schema, right?
Yes, this is what I'm trying to do.
Also I'm not sure what you are attempting with the BindMember on the NodeResovler, what is your goal here?
I was thinking that this could bind my property with the specific field that I need. Also, it was a suggestion that someone tries here: https://hotchocolategraphql.slack.com/archives/CD9TNKT8T/p1637272494256700
I would expect it to work like the following:
I tested it as the way you are saying and I'm getting this error in the console (the same I reported before):
The specified id field Serial does not exist on ...ExampleModelNode.

I've updated my post from above. I was able to confirm the issue. Could you please confirm that it still represents what you are trying to do?

The user should be able to query this like the following:
{
  id # ID!
  serial #Long!
}

Yes, this is exactly what I need.

I think we should definitely make this work using the IdField, but if you require an immediate solution, you can use the following workaround:
[Node]
[ExtendObjectType(typeof(Model))]
public class ModelNode
{
    [ID]
    public long Id([Parent] Model model) => model.Serial;

    [NodeResolver]
    public Model GetModelAsync(long id)
    {
        // ...
    }
}

Thanks for this information, it is working if I run the query requesting the Id and Serial with this fix.
It is possible that with this workaround the node() query doesn't work?
query nodeExampleModel{
  node(id: "<hashedId>"){
    id
    ... on ExampleModel {
      id
    }
  }
}

I get the following error:
{
  "errors": [
    {
      "message": "There was no argument with the name `serial` found on the field `node`.",
    }
}

It should be working, I tested the workaround before. Could you post your Node definition again?

Model Class:
public class ExampleModel
{
    public long Serial { get; set; }
}

Node Class with fix:
   [Node]
   [ExtendObjectType(typeof(ExampleModel))]
   public class ExampleModelNode
   {
       [ID]
       public long Id([Parent] ExampleModel model) => model.Serial;

       [NodeResolver]
       public async Task<ExampleModel?> GetExampleModelBySerialAsync(
           long serial,
           ExampleModelBySerialDataLoader exampleModelBySerial,
           CancellationToken cancellationToken)
           => await exampleModelBySerial.LoadAsync(serial, cancellationToken);

The issue appears to be that the parameter to the GetExampleModelBySerialAsync method is called serial and not id. It works with id, but not with something else like serial. I'm raising a different issue for this.

I stumbled into this issue and it seems to be scoped to the Data-annotation approach, which forces you to update your model to use Id as the property name.
Code-first approach works fine and lets you use the name you want for the property within the model


Bluetooth: Host: Merge bsim test builds to reduce CI load

For most tests we spend more time building them than actually running them. It would be better if they could all be built in a single build.
Issue spawned from: https://github.com/zephyrproject-rtos/zephyr/pull/79501#discussion_r1790165096
Some work has been drafted on how to merge builds that originally required differing configurations: https://github.com/zephyrproject-rtos/zephyr/pull/77122

Also, look if some code can be shared with babblekit or reused.

For most tests we spend more time building them than actually running them. It would be better if they could all be built in a single build.
Issue spawned from: #79501 (comment)
Some work has been drafted on how to merge builds that originally required differing configurations: #77122

Thanks @alwa-nordic . Note that at least tests of the same area which share configuration (like the one that triggered that comment) should be easy to build in one image (as we do for many other tests).


Mycroft skills page: what is installed?

Describe the bug
When going to https://market.mycroft.ai/skills there is no clear indication of what skills are available to install nor what skills are installed.
To Reproduce
Steps to reproduce the behavior:

Go to Configuration page: marketplace
View list of skills
Be puzzled as to which one is available.
go to https://home.mycroft.ai/skills actually see the ones that are installed.
find a skill named privacy that is installed. try to find information about what it does
go to marketplace don't  find what is installed.
go to skills see that the mark II app is installed.
Go to market place find what it does. it can chance faceplate brightness. Does not seem to work. So not installed? Or am I saying the command wrong?

Expected behavior
A legend explaining what green tick is what the blue mycroft icon means.  And if needed an indication to know what app is installed. 3-6 steps above
Explanation of what skill does on the home section or a link to the marketplace to be able to find it out. 6-7 steps above.
Environment (please complete the following information):

Device type: MARK2
OS: DIMKUM
Mycroft-core version: 21.2.1
Other versions: N/A

Additional context

Hey there, thanks for posting this. It certainly isn't intuitive right now as the Skills Marketplace isn't being used in any way by the Dinkum image.
Unfortunately I don't have a simple solution for you - but it is something we want to address.

As mentioned in mattermost. dinkum does not interact with market place but the skills market still marks some skills as installed. In the market place image posted it shows the blue logo even though I only have a mark2 device.


Pop-up closing... freezes on admin interface

This has been happening for a long time. When you try to add an author in a pub page, the pop-up modal does not auto-close

In /static/admin/js/SelectBox.js:56
Uncaught TypeError: Cannot read properties of undefined (reading 'push')
    at Object.add_to_cache (/static/admin/js/SelectBox.js:56)
    at dismissAddRelatedObjectPopup (/static/admin/js/admin/RelatedObjectLookups.js:74)
    at popup_response.js:13

I think this is now fixed... I'm not seeing it anymore at least.


Redo with Docusaurus

Redo the docs with Docusaurus, it looks nicer in my opinion. I was already done with this, and I added git and set the origin to this repository, but then I accidentally reset my local environment which removed all Docusaurus files and replaced them with the files from this repo.

Scrap that idea, I'm going to use Nextra instead.


Event callback signature in docs

cy.on("afterExpand", function(node) { }) should be cy.on("afterExpand", function(event) { var node = this; ...  })

It would be better, thanks but there is a minor issue.
Now, cy.nodes().on("afterExpand", function(event) { var node = this; ... }) works properly but cy.on("afterExpand", function(event) { var node = this; ... }) this value is not a node. It is a scope including _private & one of my variables.
I am using node.trigger("afterExpand") to emit event. I also tried cy.trigger.call(node, "afterExpand) but result was the same.

Typo: cy.on("afterExpand", function(event) { var cy = this; ... })
If you want the node, you have to bind directly to the node or use a delegate selector.
I've confirmed this working fine in the console:
cy.on('foo', 'node', function(e){ console.log( e, this ) });
Core {_private: Object}
cy.nodes()[0].trigger('foo')
VM488:1 Event {originalEvent: Object, type: "foo", cy: Core, cyTarget: Element, cyPosition: undefined…} Element {0: Element, length: 1, _private: Object}
Element {0: Element, length: 1, _private: Object}

After the commit above, it works now as expected.


Windows OpenGL FullScreen artifacts with popups and wrong mouse hittests

Describe the bug
We use Avalonia in Windows OpenGL Mode. The reason is we use ActiVIZ.NET to integrate VTK as an avalonia control (no support for angle :().
Switching to fullscreen or even to a none styled window with manually setting the size of the window to the desktop produce the same artifacts and flickering. In addition hit tests behave wrongly when I click an element on the popup controls on the main window react.
We need the fullscreen feature to run the application in a medical environment on a windows laptop and therefore are very open for help and advice.
To Reproduce
Open attached application and open the combobox. On Open and close the window procudes some flickering:
FullScreenError.zip
Expected behavior
When the application is fullscreen, the app should not produce flickering, artifacts and hit detection should work.
Avalonia version
11.0.10
OS
Windows
Additional context
Graphics Card: Intel Iris XE Graphics
Operating System Details: Windows 10.0.19045

Note that the entire WGL mode is more or less experimental and is not really supported.
You can try adding glFinish call though.

@kekekeks where should I add this call?

Once you've finished rendering the frame

it seems that it works with avalonia 11.1.2 fine. It used to work also well for PCs with nvidia graphics cards. we can close this issue.


BUG: Website Logo Not Redirecting to Home Page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There is an issue where clicking on the logo of the website does not redirect the user to the home page. This behavior is not consistent with standard website navigation practices and may cause confusion for users trying to return to the home page quickly.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Not required


Binning Summation

Can you clarify this binning mode summation?
Should you use torch.index_add_ ? Otherwise there is no difference bteween binningmode SUM and ADD
https://github.com/lava-nc/lava-dl/blob/185edfe9fb035e13c24eec063a4971ccb0956d3a/src/lava/lib/dl/slayer/io.py#L234C1-L241C38

Hi @pbonazzi I am guessing you are meaning to say SUM mode and OR mode. Yes the current implementation seems to be a bug. Would you like to make a correction PR?

Yes, I can do it over the weekend.


Using script instead of iframe approach gives 404 error

Using the following markup:
<!DOCTYPE html>

    <title>VRView Test</title>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />

Test

I get this error in console:
GET http://storage.googleapis.com/vrview/2.0/vrview.min.js 404 (Not Found)
Uncaught ReferenceError: VRView is not defined

Also if I check the script directly in the browser I get this:

@ron666 Use this path https://storage.googleapis.com/vrview/2.0/build/vrview.min.js it should work

@P-Henoch I received this errors when I tried to load http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4 for testing purposes and it didn't load it:

What can be the cause of these ones?

@ron666 The video must be loaded over HTTPS protocol. If you have access to the server you can use Let's Encrypt, it's free. Or try to use another video loaded over HTTPS, like the one in the example.

@P-Henoch Thanks that worked with the video in the example. As a last question: if the video is just a regular MP4 (like https://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4) and not a 360 video, the player displays an error, or it will play it? Because I think that's what happened with the other 2 errors above, but I wanted to confirm this with you.

And also why in iOS is not working the example? Please refer to https://github.com/googlevr/vrview/issues/77#issuecomment-262323703 for more details

@ron666 If the video is a regular MP4, I guess it will display those two errors. It have to be a 360° video or an adaptive streaming manifest file (.mpd or .m3u8). Look at the parameters for the VR View.
For the error on iOS, I have no idea why it's not working. The video and the VR View code seems to be hosted on the same server, so it should work. Wait an answer from @borismus

i still can not run this thing..i keep getting the same error and i cant load my own mp4 files


couldn't replicate results.

followed instructions to a T. wired signal to GP18 and postive to 3.3v. gnd to gnd. i even took apart a n64 controller to check that everything was wired right. drag and dropped the file into the Pico (the basic one)
not sure if i went wrong anywhere.

Did you put a pull up resistor between 3.3v and GP18?

You were right. soldered a 1k pull up resistor between GP18 and 3.3v and it worked no problem

That's great! Glad I was able to help!


CI - Run python 3.6 unit test using Terra 0.19 from Pypi

Summary
This solution will remain in the main branch for now and will go to the stable/0.3 branch when created. It will be removed from the main branch when the PR #121  gets merged after the release 0.3.0.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 82.735%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
714

Relevant Lines:
863

💛  - Coveralls


linuxPackages.perf fails to build on unstable

$ nixos-version
17.09.git.0011f90 (Hummingbird)

possibly due to changes to default compiler warnings or something, because it seems to be a -Werror problem
util/probe-event.c: In function 'post_process_module_probe_trace_events':
util/probe-event.c:669:9: error: implicit declaration of function 'post_process_probe_trace_point' [-Werror=implicit-function-declaration]
   ret = post_process_probe_trace_point(&tevs[i].point,
         ^
util/probe-event.c:669:3: error: nested extern declaration of 'post_process_probe_trace_point' [-Werror=nested-externs]
   ret = post_process_probe_trace_point(&tevs[i].point,
   ^
cc1: all warnings being treated as errors
mv: cannot stat 'util/.probe-event.o.tmp': No such file or directory

This seems to be an upstream problem in linux-4.9.36: https://lkml.org/lkml/2017/7/5/537

Fixed by 954c66983df3.


blocker "each" raise an exception in the android webview

android 5.x webview raise an exception when compile "each" blocker, that show the javascript code like
function push(expr) { if (!(expr instanceof Literal)) { expr = this.source.wrap(expr); } this.inlineStack.push(expr); return expr; }

i met the same.

i change the method to handle tpl , use the regexp in a simple function wriiten by myself.

@kuangpeng 能说下详细吗？

@heke2929  自己写个简易的函数，使用正则匹配参数并用值替换
var exp = new RegExp("{{" +da + "}}","gi"); tpl_html = tpl_html.replace(exp, data[da]);
这只是简单的处理，没有什么条件判断等表达式，能否适用还得看具体需求。
如果不满足需求可以选择换个模版框架。

@kuangpeng 首先感谢您的答复。我先按照您的方式尝试一下。

This bug doesn't provide enough information to reproduce it locally. I'm going to close it. Please reopen if you can provide a template that reproduces the failure you saw.

@lawnsea I faced this issue a minute ago. I can't see a concrete pattern to this behavior. However I have noticed that it might be occurring while I am compiling multiple handlebar templates with large json arrays simultaneously. It occurs frequently, but not always. Hope this helps.


Work higher-order hardware objects into the compressor

The compressor currently has no support at all for hardware objects of order greater than 1. Augmenting the compressor to deal with higher-order hardware objects is a tall ask, but I think it will ultimately result in a more maintainable code base: parts of the logic of the compressor are quite kludgy (and, accordingly, hard-to-follow) and read more like band-aids than like something well-reasoned. I think that accommodating higher-order hardware objects will necessarily entail sorting all of these kludges out.

@notmgsk Operations native to qubit triads. For instance, if you had a native CCNOT.

See also #118

beware, PR adventurer: the slothful vector formations here https://github.com/rigetti/quilc/pull/210#discussion_r296949684 might bite you


Order of table inserts incorrect

OK so I have a database with some PK -> FK constraints and my workflow is:

Load the db schema in postgres
Load some fixtures in postgres
Init from DB
Push to mergin cloud
Pull to QGIS Desktop (using a separate working dir to the one db sync uses)
Make some edits (add a plant type, add a vegetation point) in QGIS
Sync from QGIS Desktop to mergin cloud
Use mergin-db-sync to try to pull the changes back from cloud to my pg database

This process fails because it tries to insert the vegetation point record before it has created the needed plant type record.
Here is the log of my process:
# Just a clean up in case we had the db already
# NOTE: Also remove the GPKG from the mergin cloud instance and commit the change
PGSERVICE=osgs.kartoza.com psql -c "drop database farming;" postgres
# Grab the schema
wget -O farming.sql https://raw.githubusercontent.com/kartoza/smallfarming-gis/main/sql/farming.sql
# Load the schema
PGSERVICE=osgs.kartoza.com psql -c "create database farming;" postgres
PGSERVICE=osgs.kartoza.com psql -f farming.sql farming
# Grab the fixtures
wget -O fixtures.sql https://raw.githubusercontent.com/kartoza/smallfarming-gis/main/sql/fixtures.sql
# Load the fixtures
PGSERVICE=osgs.kartoza.com psql -f fixtures.sql farming
# Verify db state (list schemas, list tables)
PGSERVICE=osgs.kartoza.com psql -c "\dn" farming
PGSERVICE=osgs.kartoza.com psql -c "\d" farming
# This is mergin-db-sync's folder, it will get recreated in the line after
rm -rf /home/timlinux/gisdata/MerginSyncProjects/farming/
python dbsync.py init-from-db
rm -rf ~/Syncthing/QGISProjects/MerginMapsProjects/farming
# Now do steps 5,6,7 in my QGIS Desktop, checking out the project to the folder above
python dbsync.py pull

Here is the log from the last (step 8 above) pull command:
 timlinux  crest  ../mergin-db-sync  master □  python dbsync.py pull
== Starting Mergin Maps DB Sync version 1.1.2 ==
Logging in to Mergin Maps...
Pulling...
Processing Mergin Maps project 'timlinux/farming'
DB Changes:
electricity_line_condition_type    0    2    0
electricity_line_type    0    5    0
month                   0   12    0
plant_growth_activity_type    0    6    0
plant_usage             0    3    0
water_line_type         0    2    0
water_point_type        0    4    0
water_polygon_type      0    4    0
water_source            0    3    0
Pulled new version from Mergin Maps: v7
Mergin Maps Changes:
plant_type              1    0    0
vegetation_point        2    0    0
Applying new version [WITH rebase]
Debug: rebase info (base2their / old)
TABLE plant_type
  inserted 1,
  deleted  --none --
  updated  --none --
TABLE vegetation_point
  inserted 1,2,
  deleted  --none --
  updated  --none --

Debug: mapping
--none --

Debug: No conflicts present
Error: rebase-db failed!
GEODIFF: Error: postgres cmd error: ERROR:  insert or update on table "vegetation_point" violates foreign key constraint "vegetation_point_plant_type_uuid_fkey"
DETAIL:  Key (plant_type_uuid)=(0b25842b-ec57-479b-993f-5a64b0ba6347) is not present in table "plant_type".

SQL:
INSERT INTO "public"."vegetation_point" ("id", "uuid", "last_update", "last_update_by", "notes", "image", "estimated_crown_radius_m", "estimated_planting_year", "estimated_height_m", "geometry", "plant_type_uuid") VALUES (2, 'fce59771-f33a-4311-ae2a-35c7cc1e4224', '2023-03-28T23:16:07.248'<EMAIL_ADDRESS>NULL, NULL, NULL, NULL, NULL, ST_GeomFromWKB('\x0101000000EC68098D0ACF1DC0396D2117CBB44340', 4326), '0b25842b-ec57-479b-993f-5a64b0ba6347')
Error: Unable to perform GEODIFF_applyChangeset modified2final

Error: geodiff failed!
['geodiff', 'rebase-db', '--driver', 'postgres', 'host=osgs.kartoza.com dbname=farming user=docker password=eiGooGhooYe3unieChei sslmode=require', '--skip-tables', 'spatial_ref_sys', 'mergin_base_do_not_touch', 'public', '/tmp/farming-dbsync-pull-base2their', '/tmp/farming-dbsync-pull-conflicts']

Ordering of table inserts/updates/deletes to avoid this would be probably very tricky, fortunately there is an alternative solution: constraints such as foreign keys can be deferred, so they are not evaluated immediately after each insert/update/delete, but only at the end of the transaction.
There are three valid options for constraints:

NOT DEFERRABLE - if it can not be deferred at all (the default)
DEFERRABLE INITIALLY IMMEDIATE - it can be deferred, but it has to be explicitly enabled in transaction
DEFERRABLE INITIALLY DEFERRED - it can be deferred, and it is set as deferred initially in transactions

So the SQL creating tables can be modified from this:
    	infrastructure_type_uuid UUID NOT NULL REFERENCES infrastructure_type(uuid)

to something like this:
    	infrastructure_type_uuid UUID NOT NULL REFERENCES infrastructure_type(uuid) DEFERRABLE INITIALLY DEFERRED

Then the problem should go away. Foreign key checks are still enabled, but only evaluated at the end of transaction when everything should be in order.
Things worth improving in geodiff / db-sync on this matter:

detect if there are any foreign keys that are not deferrable
support also DEFERRABLE INITIALLY IMMEDIATE by running SET CONSTRAINTS ALL DEFERRED; at the start of a transaction

Useful reference: https://emmer.dev/blog/deferrable-constraints-in-postgresql/

Ah this is super helpful advice, thanks @wonder-sk !


Adsk Contrib - Remove 'expat' symbols

Signed-off-by: Patrick Hodoul<EMAIL_ADDRESS>The pull request removes the 'yaml' symbols from the library to avoid clashes with hosts (i.e. an app or another library) when it also uses the yaml with a different version. On macOS and linux (tested with ubuntu) only the OpenColorIO symbols are now present.

This worked on my macOS build. Did you saw this -hidden-l flag? I wonder if there is a way to use that easily through CMake as it seems like a cleaner solution to the problem.

@remia the solution is a lot less cryptic so, the code uses it now.

For information about my previous comment, I was not clear how this would work with expat being already linked in the target_link_libraries call but this appears to take precedence and work fine which is good news.


feat: Added DSN

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?
Added a DSN tool to facilitate DSN character concatenation
User Case Description
dsn := gorm.DSN{
	Host: "<IP_ADDRESS>",
	Port: 3306,
	User: "root",
	Pass: "password",
	DB:   "gorm",
	Options: map[string]string{
		"charset":   "utf8mb4",
		"parseTime": "True",
		"loc":       "Local",
	},
}

gorm.Open(mysql.Open(dsn.String()), &gorm.Config{})

Should be added to the: https://github.com/go-gorm/mysql

I don't think it's necessary to add this feature
https://github.com/go-sql-driver/mysql (https://pkg.go.dev/github.com/go-sql-driver/mysql?utm_source=godoc#Config.FormatDSN)


Added unit tests and some fixes.

Fixed order by to work when the property doesn't have column attribute; added MSSQL and Postgre compatibility with SetOrderBy and SetLimit.
With this, #114 can be closed

@phnx47 IDK Why security/snyk is failed, should proceed with this merge?

Problem with MySql Auth in AppVeyor
https://ci.appveyor.com/project/phnx47/microorm-dapper-repositories/build/tests

Oh, i've changed the password. I run these tests locally.

@pvictorlv When you continue working, you should write "WIP: " in subject

Sorry, the SQLite support was supposed to be in another PR.

Okay, added AppVeyor pass

Use
       connString = "";
}```

Bro, the connection was right... check it again

@pvictorlv ok, and last thing about DefineConstants - Trace, I commented line

Sorry, didn't understand your question.

@pvictorlv  Why need this in csproj? I wrote comment before
  <PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|net461|AnyCPU'">
    <DefineConstants>TRACE</DefineConstants>
  </PropertyGroup>

Sorry, this seems to be an VS Bug. I've removed it.


adding "@vue/compiler-sfc": "3.2.0-beta.7" to devDependencies gives "Invalid VNODE type: Symbol(Fragment)"

Version
3.2.0-beta.7
Reproduction link
https://github.com/mariusa/vue32-error
Steps to reproduce
npm init vite@latest my-vue-app --template vue
cd my-vue-app
# add "@vue/compiler-sfc": "3.2.0-beta.7" to devDependencies, required for refSugar. Change vue version to "3.2.0-beta.7"
npm i
npm i bootstrap bootstrap-vue-3 --save

main.ts
import { createApp } from 'vue'
import App from './App.vue'
import BootstrapVue3 from 'bootstrap-vue-3';

import 'bootstrap/dist/css/bootstrap.css'

const app = createApp(App);
app.use(BootstrapVue3);
app.mount('#app')

App.vue
<template>
  <img alt="Vue logo" src="./assets/logo.png" />
  <HelloWorld msg="Hello Vue 3   Vite" />
  <b-card>test</b-card>
</template>

<script setup>
import HelloWorld from './components/HelloWorld.vue'

// This starter template is using Vue 3 experimental <script setup> SFCs
// Check out https://github.com/vuejs/rfcs/blob/master/active-rfcs/0040-script-setup.md
</script>

What is expected?
have the app work and render 'test'
What is actually happening?
[Vue warn]: Invalid VNode type: Symbol(Fragment) (symbol)
at
at

Tried to use refSugar with 3.2.0

Your reproduction is not minimal, please read and follow https://new-issue.vuejs.org/?repo=vuejs/vue-next#why-repro when reporting a bug. Most bugs should be reproducible with the SFC Playground.

I've spent 2h to narrow down the cause. The vite starter repro has only 1 library, bootstrap-vue-3
It works fine with vue 3.1.x
Only when changing to vue 3.2.0-beta.7  package.json, the error appears:
{
    "name": "my-vue-app",
    "version": "0.0.0",
    "scripts": {
        "dev": "vite",
        "build": "vite build",
        "serve": "vite preview"
    },
    "dependencies": {
        "bootstrap-vue-3": "0.0.1-alpha.2",
        "vue": "3.2.0-beta.7"
    },
    "devDependencies": {
        "@vitejs/plugin-vue": "^1.3.0",
        "@vue/compiler-sfc": "3.2.0-beta.7",
        "vite": "^2.4.4"
    }
}

Your problem is with bootstrap-vue-3. have a look at it in your node_modules folder. it has its own node_modules subfolder, containing Vue 3.1.5
The reason for this is that this package declares Vue as a dependency (which should rather be a optional peerDependency) combined with the fact that 3.2-beta isn't the latest version - it's no the next tag.
This would have to be resolved in that package.

Thank you, Thorsten, appreciate your time for finding the cause!


Ingress network should not be attachable

Ingress network is a special network used only to expose
ports. For this reason the network cannot be explicitly
attached during service create or service update
Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>(cherry picked from commit 610e2684f478eb63c7373fc28e60b50addd09684)

test issue:
ok  	github.com/docker/swarmkit/manager/state/raft	69.720s	coverage: 69.0% of statements
time="2018-02-27T21:48:44Z" level=warning msg="no certificate expiration specified, using default" cluster.id=sj4jwoht1624qwhudtp7ytenl method="(*Server).UpdateRootCA"
--- FAIL: TestCanRemoveMember (23.98s)
	Error Trace:	cluster_test.go:353
	Error:		Received unexpected error context deadline exceeded

	Error Trace:	cluster_test.go:355
	Error:		Expected nil, but got: &api.RaftMember{RaftID:0x7847527aa88fb68e, NodeID:"n6g8i2vwcd5l7axh831y7tgux", Addr:"<IP_ADDRESS>:43813", Status:api.RaftMemberStatus{Leader:false, Reachability:1, Message:""}}

	Error Trace:	cluster_test.go:356
	Error:		Not equal: 3 (expected)
			        != 2 (actual)

FAIL

@anshulpundir can you restart the CI?

This time:
time="2018-03-01T22:08:24Z" level=info msg="Failed to dial <IP_ADDRESS>:49499: context canceled; please retry." module=grpc
time="2018-03-01T22:08:24Z" level=info msg="Failed to dial <IP_ADDRESS>:49499: context canceled; please retry." module=grpc
time="2018-03-01T22:08:24Z" level=info msg="Failed to dial <IP_ADDRESS>:49499: context canceled; please retry." module=grpc
time="2018-03-01T22:08:24Z" level=info msg="Failed to dial <IP_ADDRESS>:49499: context canceled; please retry." module=grpc
--- FAIL: TestManager (0.14s)
	Error Trace:	manager_test.go:93
	Error:		Not equal: "node not registered" (expected)
			        != "dispatcher is stopped" (actual)

@nishanttotla today the only restriction that we have on the network is the ingress network. It's possible that in the future we require additional check on the spec regarding networks.
Not sure which kind of side effects are you referring to. can you elaborate on that?

@fcrisciani I just meant that in the future, if there would be a case for having a separate check for isIngressNetwork in the future by breaking up validateNetworks, but on second thoughts, I don't really think it's that important.
Also, tests pass now.


WebUI: Add Update & Recovery of old node/backup on Setup

I think this is one of the important things to add for the future - that users get assisted thru the update process and get offered to download the lnd/cl-rescue backup at one point. And during Setup a rescue-file can be used to recover an old node.
@cstenglein how do we wanna start the design/frontend part of this? Should I create some mockup first we then can discuss?

I think it could be solved in one screen for the backup & one to download on recover.
Not sure about where in the flow it should be placed though.

Not sure about where in the flow it should be placed though.

np. will prepare a mockup once we will pickup this issue for v1.8.1

Sent with GitHawk

Sent with GitHawk

We will push this to v1.10 because if time constraints.

Is there an idea / mockup how to integrate it?
And is there an API endpoint to download a backup @fusion44?

Is there an idea / mockup how to integrate it?

Not yet - would it make sense to plan/do this together with the general redesign of the dashboard?


Wizard filter step - disable proceeding if filter results in zero tables

In the 'create connector' wizard, if the user applies table/schema filter which results in zero tables - seems like we should not allow user to proceed.  Is that right @gunnarmorling ?

Good question; I think we should allow to proceed actually. It'd be a valid situation for a user to first set up a connector and only then create a table whose name actually matches that filter.

thanks for the reply, we are allowing to proceed with the current implementation. we can close this


[OpenAPI] Improvement for how enum array type parameters are represented in generated client

Description:

For the parameter 'expand with above schema. The type is generated as follows,
public type A_SalesOrderExpandOptions (
    "*"|
    "to_BillingPlan"|
    "to_Item"|
    "to_Partner"|
    "to_PaymentPlanItemDetails"|
    "to_PrecedingProcFlowDoc"|
    "to_PricingElement"|
    "to_RelatedObject"|
    "to_SubsequentProcFlowDoc"|
    "to_Text")[];

The A_SalesOrderExpandOptions is a array of a union of strings.
When this is seen in the API docs this gives the illusion that this parameter is a single value rather than an array.

Shall we improve this by representing the type as union of strings only. The parameter will be of A_SalesOrderExpandOptions[]?

Link to the openapi specification, https://github.com/niveathika/module-ballerinax-s4hana.sales/blob/main/docs/spec/API_SALES_ORDER_SRV/API_SALES_ORDER_SRV.json#L1

With the new improvement to the client signature, this issue can be resolved. Therefore, we are closing this with the support of https://github.com/ballerina-platform/ballerina-library/issues/6475issues/6475issues/6475issues/6475


Fine tune output messages

Rephrase messages trying to be more user friendly
Print errors at the end to avoid interleaving with dots "."
Print an "all is good" message if the analysis found no problems
Add a way to force the output of all errors to stderr (disable the "omit error" feature; intended for development-only)

:+1: LGTM


GitHub.dev blocker - merge and PR disabled

Several folks have observed a problem with the steps of the GitHub.dev workshop https://github.com/microsoft/workshop-library/tree/main/learn-github -
We are unable to complete the merging of the branch and raising a PR to our main branch using GitHub.dev. The error seems to indicate that there is no upstream branch, which is wrong.
This was captured by our student ambassador tester:

@joyceerhl Do you have thoughts on this?

Investigating... This message is displayed by GHPRI: https://github.com/microsoft/vscode-pull-request-github/blob/48bb6db679042486573e8ab592e489c548815442/src/github/createPRViewProvider.ts#L315
cc @alexr00

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/3309

Sure! I'll test it and let you know asap.

Closing! nice :)


Use test data for integration tests

Currently the integration tests populate the database with the same data as the production version. Instead of this they should use their own testing data.

Added test scripts for the DB data


Fix/large module size

I noticed that the example geojson files were ~28MB in size, which seems a little hefty for users to pull down when they npm install.
I'm not sure if my approach with the parser example works for you, let me know what you think. Was the intention of that file to just demonstrate how one might go about creating geojson with start-end properties? Or were you using it for development?
Anyway, just an idea that I thought might help the module size a bit.

Thanks you for the PRs and issue report!
I don't think I'll be merging this one, though.
border-parse.js is specifically for borders.json -- it's not really a "standard" format (properties in GeoJSON is arbitrary data). It's partly for development (at least, originally) and partly as a demo of how one might process a GeoJSON file that's not exactly in the right format.
borders.json was included because the generation of it requires extra software, and I'd like to avoid that need.
I agree the json data is not necessary for use in production, but rather than just deleting it here, it could be put into a .npmignore. It'll still be included if they pull the entire repo, but deleting it wouldn't help with that anyway (it's still in the git history and the gh-pages branch).

.npmignore added in 3178ed64f0819684648465eb3f041f7855dc9749

Oh, yeah, that's a much better idea @skeate :+1:
I'm keen to contribute to your project if you're interested. Is there a preferred way for us to discuss ideas before I go down the path of implementing unwanted changes? Perhaps a Gitter chat room? Or just open an issue for it? My preference would be Gitter.

I've started a Gitter chatroom!


Resolve bug in resource details filtering - iso6

Description

Short description here *

closes Add ticket reference here
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Code is clear and sufficiently documented
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

was it required to be fixed on ISO-6?

Didn't ask, I assumed that as this error is there we should get rid of it, but I'll ask. won't hurt :)

Update, Arnaud confirmed to remove bug from both versions

Processing this pull request

Merged into branches iso6 in 90be8c84860c694292e2ab6b17dada2b7a83cc23


Delayed Mobile Transitions

Hi Julian,
I have a problem with my animations on https://layonthebeech.github.io/slidingpuzzle/. On desktop they work totally fine but on mobile there is a definite delay when I hit the buttons. When I had this code on codepen and was using velocity, there wasn't a delay at all so i'm pretty confused as to what is causing it. I made sure to force MobileHA to true. Here is some example code for reference. I am using iphone 6 latest chrome browser and latest version of velocity.
$('#' + String(twoDArray[tileIndex[0]][i])).velocity({
            translateX: tileCoordinates[tileIndex[0]][i][0] + tileDimensions[gameType]
          }, {
            delay:false,
            duration:1,
            mobileHA:true
          }, 5)

Both the delay and mobileHA options are redundant there, as there's no delay by default, and the mobileHA is also on by default (you'd need to manually turn it off) - mobileHA also adds a transform value the same as your translateX - so does nothing anyway.
There is a tiny delay between starting the animation and the first frame called after then (though as it's between browser rendering frames shouldn't ever be noticeable unless the browser is running slowly) - but you can use force-feeding to see if it helps.
The most likely cause is simply the iPhone pushing stuff off to the GPU, and being somewhat laggy about it - sadly Apple have really screwed up where Safari is concerned in terms of not following standards and ignoring fixes that have been in the root Chromium codebase for (in some cases) literally years.
Final thing you can do is to add a will-change:transform; to your css for the element - it basically warns the browser that this value will change and it needs to be prepared. Support is there in Chromium based browsers and Firefox, but it's usefulness is relatively unknown so far.

Thanks for the reply! I guess my question is do you know why codepen tells the browser that makes it so there is no delay at all? If I can just figure that out, I know the ability is there.
Pat
On Nov 27, 2016, 9:45 AM -0600, Rycochet<EMAIL_ADDRESS>wrote:

Both the delay and mobileHA options are redundant there, as there's no delay by default, and the mobileHA is also on by default (you'd need to manually turn it off) - mobileHA also adds a transform value the same as your translateX - so does nothing anyway.
There is a tiny delay between starting the animation and the first frame called after then (though as it's between browser rendering frames shouldn't ever be noticeable unless the browser is running slowly) - but you can use force-feeding (http://velocityjs.org/#forcefeeding) to see if it helps.
The most likely cause is simply the iPhone pushing stuff off to the GPU, and being somewhat laggy about it - sadly Apple have really screwed up where Safari is concerned in terms of not following standards and ignoring fixes that have been in the root Chromium codebase for (in some cases) literally years.
Final thing you can do is to add a will-change:transform; to your css for the element - it basically warns the browser that this value will change and it needs to be prepared. Support is there in Chromium based browsers and Firefox, but it's usefulness is relatively unknown so far.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/julianshapiro/velocity/issues/724#issuecomment-263129172), or mute the thread (https://github.com/notifications/unsubscribe-auth/AGA7YDZfHyFaTcDlGrgFS1mHSfJ8gmq9ks5rCaWTgaJpZM4K9A3Z).

Not sure what you mean - Codepen is pretty much just putting your code into an iframe, and leaving it to it - so once written it's not really got anything to do with being on Codepen itself. Have you tried it as a stand-alone page outside Codepen to see if it still happens?

Yea so these two pages are pretty much the same code but if you tried them on mobile, the transitions in the codepen aren't delayed.
https://layonthebeech.github.io/slidingpuzzle/.
http://codepen.io/layonthebeech/pen/JbJZoY

If anyone cares I didn't know about this https://www.sitepoint.com/5-ways-prevent-300ms-click-delay-mobile-devices/

Doh - I completely forgot about that, too used to bypassing it to begin with :-/

Hahaha no worries. Glad i found a solution! Hope this helps any fellow
users. http://imgs.xkcd.com/comics/wisdom_of_the_ancients.png
On Wed, Nov 30, 2016 at 1:58 AM, Rycochet<EMAIL_ADDRESS>wrote:

Doh - I completely forgot about that, too used to bypassing it to begin
with :-/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/julianshapiro/velocity/issues/724#issuecomment-263807464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGA7YB6UVg9DA5foYd7IgPLR1mG5LExPks5rDSzBgaJpZM4K9A3Z
.

--
Patrick Beecher


Update Fitbit sensor (variable battery icons and formatted names/values)

Description:

Add variable icon for battery status
Clean up formatting for resource names and values

Related issue (if applicable): N/A
Pull request in home-assistant.github.io with documentation (if applicable): N/A
Example entry for configuration.yaml (if applicable):
N/A

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Would be nice to include an upgrade of the requirement.

@fabaff Are you referring to upgrade the client requirement from 0.2.3 to 0.3.0?

@MartinHjelmare @pvizeli @fabaff Thanks for the comments, everything should be addressed now.

@MartinHjelmare Good suggestion on battery level dict, should be ready to merge now.


⚠ Bofrost.de has degraded performance

In 3886d8f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2047 ms

Resolved: Bofrost.de performance has improved in 9cbb832 after 13 minutes.


Convert recursively directly in the output directory

The "_converted" behavior can't be used with -u in sync scenarios.
I'm currently syncing epubs on my Kobo but I'd like to have the original epubs only on the computer and have all kepubs on my Kobo. If I mount the Kobo and run kepubify with my modification, I can use the -u flag to sync between my library and the Kobo's root directory (unidirectional sync, that's ok, it's what I want).
This might be a very personal use-case scenario, but I thought I'd contribute the patch anyway. Maybe we can discuss this scenario and use another approach (e.g. another command line flag?)

I'm not quite sure I understand exactly what you want. What I currently use with my kobo (triggered by udev) is: kepubify --update --output=$(findmnt LABEL=KOBOeReader -uno TARGET)/kepubify /home/patrick/Documents/Books/; seriesmeta. This puts the books in kobo/kepubify/Books_converted.

(I've repushed because of something I left out of the commit)

This puts the books in kobo/kepubify/Books_converted

I'd like to put the books directly on the kobo root folder (that is, the partition I see from my pc when I mount it).

@daniele-athome I see what you mean. The reason why I left it out is to reduce the risk of conflicts, but I can probably add an option to allow putting it in the root.

This will probably be implemented in kepubify v3 along with the redo of path handling.


Natively animated ShapeSource for Point and LineString

Description
This adds two new components, AnimatedPointSource and AnimatedLineSource.
Currently, animating a point or a line on the map requires sending messages over the bridge many times per second. This is inefficient even when sending small updates (hence why Reanimated exists), but animating especially a long line involves transferring a very large array repeatedly, where potentially most of the coordinates are identical from frame to frame.
These components minimize the messages (both size and frequency) that need to be sent from the React Native app. With AnimatedPointSource, you can specify a duration and then simply send coordinate updates, and the source will animate to them. With AnimatedLineSource, you can send the entire line string once, and then just update the start ~and end~ (not implemented yet) distances, and the source will animate the line ends accordingly.
Note: I considered augmenting ShapeSource instead, but I decided that it would potentially overwhelm that component and make it hard to maintain.
Checklist

[x] I have tested this on a device/simulator for each compatible OS
[x] I updated the documentation with running yarn generate in the root folder
[x] I added/updated a sample - if a new feature was implemented (/example)

Videos
https://github.com/rnmapbox/maps/assets/2423291/3dfb20b5-51f9-4020-b600-ff55ad74fe21
https://github.com/rnmapbox/maps/assets/2423291/5352a0e1-3780-4280-b297-aeb7e3838475
https://github.com/rnmapbox/maps/assets/2423291/89f5cdfc-5387-4b01-862c-29fa22aae9f5
https://github.com/rnmapbox/maps/assets/2423291/a44d0b19-7763-45a1-980f-83ce6e8843ca

Sorry it took so long to get to this.
I generally agree about the value of a single component. I had a few concerns which led me to make separate components:

Possible disagreement about what it means to animate a line - the particular use case here is animating the beginning and end, but keeping the line string constant. But I could also imagine somebody wanting to actually morph one line into another, which this PR doesn't support, and which would call for an incompatible implementation.
Having one component support animations for some data types but not others. For example, what would the implementation be for animating a clusterable FeatureCollection?
If we were to abstract ShapeSource to handle animation, I think the component and TS glue would need a refactor, particularly in the interface, to prevent it from getting really confusing and enabling nonsensical property combinations. I could see something like

type ShapeSourceBaseProps = {
  id: string;
  children?: React.ReactElement | React.ReactElement[];
};

type LineSourceProps = {
  readonly type: "LineString";
  shape: GeoJSON.LineString;
  startOffset?: number;
  endOffset?: number;
  animationDuration?: number;
};

type PointSourceProps = {
  readonly type: "Point";
  shape: GeoJSON.Point;
  snapIfDistanceIsGreaterThan?: number;
  animationDuration?: number;
};

type ShapeSourceProps = ShapeSourceBaseProps &
  (LineSourceProps | PointSourceProps);

That way, these would be valid:
<ShapeSource
  type={'LineString'}
  id={'my-line-source'}
  shape={...}
  startOffset={250}
  endOffset={0}
  animationDuration={1000}
>
  ...
</ShapeSource>

<ShapeSource
  type={'Point'}
  id={'my-point-source'}
  shape={...}
  snapIfDistanceIsGreaterThan={300}
  animationDuration={1000}
>
  ...
</ShapeSource>

but these would correctly throw errors:
<ShapeSource
  type={'Point'}
  id={'my-point-source'}
  shape={...}
  startOffset={250} // property `startOffset` does not exist
  animationDuration={1000}
>
  ...
</ShapeSource>

<ShapeSource
  type={'PointFeatureCollection'}
  id={'my-points-feature-collection-source'}
  shape={...}
  animationDuration={1000} // property `animationDuration` does not exist
>
  ...
</ShapeSource>

Another possibility is going with a generic:
type LineSourceProps = {
  shape: GeoJSON.LineString;
  startOffset?: number;
  endOffset?: number;
  animationDuration?: number;
};

type PointSourceProps = {
  shape: GeoJSON.Point;
  snapIfDistanceIsGreaterThan?: number;
  animationDuration?: number;
};

type ShapeSourceProps<T = LineSourceProps | PointSourceProps> = {
  id: string;
  shapeProps: T;
  children?: React.ReactElement | React.ReactElement[];
};

to be used like:
<ShapeSource<LineSourceProps>
  id={'my-line-source'}
  shapeProps={
    shape: ...,
    startOffset: 250,
    endOffset: 0,
  }
  animationDuration={1000}
>
  ...
</ShapeSource>

but this wouldn't allow the same runtime type inspection that the first method would give us.
Thoughts?

What about introducing a parent component, if ShapeSource is placed inside of on AnimatedShape containers then shape property would not be allowed.
<AnimatedLineShape
  animationDuration={...}
  ...
>
   <ShapeSource id={...} ...>
</AnimatedLineShape>

You can probably tell I don't have a lot of time to work on this, so I'm realizing it would be smart to try to find a relatively simple solution.
If you prefer to not have new native components, what about just adding extra props to ShapeSource, like animationDuration and snapIfDistanceIsGreaterThan? Then e.g. RCTMGLShapeSource can conditionally add and remove specialized subcomponents, but the native <--> JS interface would still be only ShapeSource.
What do you think? I'll try to respond a little more quickly this time :)

@mfazekas This is now working on both platforms with no new native files - everything is inside of ShapeSource. You can see up-to-date examples in the description above.
Thoughts?

@naftalibeder thanks much for working on this.
I've made a POC implementation on the structure described earlier:
<AnimatedLineShape
  animationDuration={...}
  shape={...}
  startOffset={...}
  endOffset={...}
  ...
>
   <ShapeSource id={...} ...>
</AnimatedLineShape>

See:
https://github.com/rnmapbox/maps/pull/3012
I think it offers the following advantages:

we can have multiple animator classes as shapes can be animated multiple ways
animation logic and shape source logic is separated (this simplifies both usage/docs and implementation)
does support future structure when a path animator have one  shape source for the enterie line, while the shape source for the end of line

The POC needs to implemented on Android and the real animation functionality implemented in your PR moved, what do you think?

@mfazekas Interesting idea. It does have the benefit of keeping the animation component specific to the type of animation needed (coordinates, line offset, etc.)
I have a few concerns, mostly around complexity:

I feel like it makes the JS interface more confusing. In my opinion, animating something shouldn’t feel like a completely separate action from positioning it - you should just be able to say “and move it smoothly” and have it do that. I worry about needing to nest source components in a specific way (when already the nested source -> layer system can be confusing enough).

I think adding more native components makes this harder to maintain. For future devs, understanding the native ShapeSource implementation as a kind of linear switchboard will be easier.

To minimize complexity in the native implementations, we can simply refactor out utilities, something like animateTo(value, duration, fps). This would dramatically shrink those files.
To simplify the JS interface (i.e. not allow invalid prop combinations), it would be simple to create composed components like LineStringShapeSource, which would return ShapeSource with only the appropriate values set.
In short, I think we should make animation feel more like an optional setting, instead of a different kind of functionality. Type restrictions would be better handled with composition.
What’s your feeling about all this?

@mfazekas Interesting idea. It does have the benefit of keeping the animation component specific to the type of animation needed (coordinates, line offset, etc.)
I have a few concerns, mostly around complexity:

I feel like it makes the JS interface more confusing. In my opinion, animating something shouldn’t feel like a completely separate action from positioning it - you should just be able to say “and move it smoothly” and have it do that. I worry about needing to nest source components in a specific way (when already the nested source -> layer system can be confusing enough).
I think adding more native components makes this harder to maintain. For future devs, understanding the native ShapeSource implementation as a kind of linear switchboard will be easier.

To minimize complexity in the native implementations, we can simply refactor out utilities, something like animateTo(value, duration, fps). This would dramatically shrink those files.
To simplify the JS interface (i.e. not allow invalid prop combinations), it would be simple to create composed components like LineStringShapeSource, which would return ShapeSource with only the appropriate values set.
In short, I think we should make animation feel more like an optional setting, instead of a different kind of functionality. Type restrictions would be better handled with composition.
What’s your feeling about all this?

I do not want to add animation to the interface of ShapeSource. There can be many animations possible, there could be a rotation animation, a translation animation, route animation etc. For me animation is a different kind of functionality, and we should be having it outside ShapeSource both in terms of implementation and the interface.
Also an animation might be source for multiple ShapSource. Like the same route animation can output the route as a LineString to one ShapeSource, but also the end point as a Point to another ShapeSource.
It's a valid point that embedding can be constraining as it might force some layer ordering. Another alternative would be similar to the way react-native and reanimated handle animations.
const shape = useMemo(() => new TransitionShapeAnimator(originalShape), []);
useEffect(() => { shape.moveBy(x,y) }, []);
...
<MapView>
   <ShapeSource shape={shape} ... />
</MapView>

After thinking about it more, I generally agree with you. Of the two options you laid out, I prefer the second one, as I think it's easier to understand and use.
So, for something specific, like animating the offsets of a line, what do you think about this example usage:
const animatedShape = useMemo(() => {
  return new ShapeTransitionLineOffsets({
    shape: routeLineString,
    initialStartOffset: 0,
    initialEndOffset: 0,
  });
}, []);

useEffect(() => {
  animatedShape.current.setShape(routeLineString);
}, [routeLineString]);

useEffect(() => {
  animatedShape.current.changeStartOffset(startOffset, { duration: 800 });
}, [startOffset]);

The only concern I have is that it requires some boilerplate. A more concise (and React-y) alternative might be to ditch the class instance idea in favor of a pure interface:
const animatedShape = useMemo((): ShapeTransitionLineOffsets => {
  return {
    shape: routeLineString,
    startOffset,
    endOffset: 0,
    duration: 800
  };
}, [routeLineString, startOffset]);

Thanks, I'd prefer the first one. Iideally in longer term we could have method like stopAnimation to stop an animation and return the last state. I don't see that's possible with the second option.
Instead of new ShapeTransitionLineOffsets we could do a createShapeTransitionOffsets

@mfazekas Would you have any objection to invoking functions on the component ref, like
  const sourceAnimator = useRef<ShapeSourceAnimator>(null);

  useEffect(() => {
    sourceAnimator.current.setLineStartOffset(10, { duration: 2000 });
    // Could also do something like:
    // sourceAnimator.current.animate({ type: 'lineOffsets', startOffset: 10, duration: 2000 });
  }, []);

  return (
    <ShapeSource ref={sourceAnimator}>
      {...}
    </ShapeSource>
  );

Pros:

It matches the interface we have for animations/transitions on MapView, Camera, etc.
It's a lot simpler than creating an animator class, which requires setting up a global manager instance and dealing with maintaining references to all nodes instances.
It still allows for ref.stopAnimation(), etc.

Cons:

Honestly, not sure I see any ¯\(ツ)/¯

@mfazekas Would you have any objection to invoking functions on the component ref, like
  const sourceAnimator = useRef<ShapeSourceAnimator>(null);

  useEffect(() => {
    sourceAnimator.current.setLineStartOffset(10, { duration: 2000 });
    // Could also do something like:
    // sourceAnimator.current.animate({ type: 'lineOffsets', startOffset: 10, duration: 2000 });
  }, []);

  return (
    <ShapeSource ref={sourceAnimator}>
      {...}
    </ShapeSource>
  );

Pros:

It matches the interface we have for animations/transitions on MapView, Camera, etc.
It's a lot simpler than creating an animator class, which requires setting up a global manager instance and dealing with maintaining references to all nodes instances.
It still allows for ref.stopAnimation(), etc.

Cons:

Honestly, not sure I see any ¯_(ツ)_/¯

I don't like this a shape animation still would be still strongly coupled with shapeSource both in terms of interface and in implementation.
Also the example of having an animator animating the route then having 2 shape sources connected to that animator, one getting the route itself (LineString) other just the endpoint (Car symbol for example) would not be possible this way.
I've updated my POC #3012. The interface there is:
animatedRoute = new RouteShapeAnimator():
() => {
   animatedRoutes.animate(...);
}

...
<ShapeSource shape={animatedRoute}>
...
</ShapeSource>

Yes the it's a bit more complex implementation, but adding different animations are much easier both in terms of implementation and interface.

I do like your update to the POC. I guess I don’t mind the added complexity of the native singleton tracking animators, if the added benefit is worth the trade-off - still not sure though.
Could you post an example of how two ShapeSources could use different parts of the same animator? Would an animator have getter methods like getLineSlice(start, end)?

I do like your update to the POC. I guess I don’t mind the added complexity of the native singleton tracking animators, if the added benefit is worth the trade-off - still not sure though.
Could you post an example of how two ShapeSources could use different parts of the same animator’s shape? Would an animator have getter methods like getLineSlice(start, end)?

Something like this:
let routeAnimator = useMemo(() =>  new RouteAnimator(route), []);
let routeEndAnimator = useMemo(() => new RouteEndAnimator(routeAnimator));

return (
  <ShapeSource shape={routeAnimator}>
     <LineLayer ... />
  </ShapeSource>
  <ShapeSource shape={rounteEndAnimator}>
     <CircleLayer .../>
     <SymbolLayer .../>
  </ShapeSource>
);

But instead of new RouteEndAnimator(routeAnimator) we could also implement as  routeAnimator.getEndpointAnimator()

@mfazekas I was wondering if there was any possibility of integrating something like what's in this PR right now - potentially with experimental tags on the animation properties? I haven't had the free time to refactor this into the ShapeAnimator architecture, and I don't just want to leave this PR open indefinitely.
if you're not comfortable with that, I can close this PR and pull the functionality out into some custom components that I'll maintain in my own fork. That would be OK with me, although of course it's preferable to be fully synced with upstream. Please let me know what you think.

@naftalibeder I'm sorry, I do like the feature but I don't want to add it in current form, thanks for the understanding.
If I have time I'll try to work on the shape animators, so adding this is easier


scd2html: init at 1.0.0

Description of changes
Add scd2html - generates HTML from scdoc source files.
Home page:- https://git.sr.ht/~bitfehler/scd2html

Things done

Built on platform(s)

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

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

1 package built:

scd2html

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2779

Thanks for the reviews @pbsds & @onemoresuza, all fixed up.

[x] package path fits guidelines
[x] package name fits guidelines
[x] package version fits guidelines
[x] package build on x86_64-linux
[x] executables tested on x86_64-linux
[x] meta.description is set and fits guidelines
[x] meta.license fits upstream license
[x] meta.platforms is set
[x] meta.maintainers is set
[x] build time only dependencies are declared in nativeBuildInputs
[x] source is fetched using the appropriate function
[x] the list of phases is not overridden
[x] when a phase (like installPhase) is overridden it starts with runHook preInstall and ends with runHook postInstall.
[x] patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
[x] patches that are remotely available are fetched rather than vendored

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

1 package built:

scd2html


Problem compiling PhantomJS using Docker

Which version of PhantomJS are you using? master
What steps will reproduce the problem?

Get a clean checkout of phantomjs
Run docker run -v $PWD:/src debian:wheezy /src/deploy/docker-build.sh
Wait
It will eventually fail with this output:

Compiling PhantomJS...
Submodule '3rdparty-win' (https://github.com/Vitallium/phantomjs-3rdparty-win.git) registered for path 'src/qt/3rdparty'
Submodule 'qtbase' (https://github.com/Vitallium/qtbase.git) registered for path 'src/qt/qtbase'
Submodule 'qtwebkit' (https://github.com/Vitallium/qtwebkit.git) registered for path 'src/qt/qtwebkit'
Usage: git submodule [--quiet] add [-b branch] [-f|--force] [--reference <repository>] [--] <repository> [<path>]
   or: git submodule [--quiet] status [--cached] [--recursive] [--] [<path>...]
   or: git submodule [--quiet] init [--] [<path>...]
   or: git submodule [--quiet] update [--init] [-N|--no-fetch] [-f|--force] [--rebase] [--reference <repository>] [--merge] [--recursive] [--] [<path>...]
   or: git submodule [--quiet] summary [--cached|--files] [--summary-limit <n>] [commit] [--] [<path>...]
   or: git submodule [--quiet] foreach [--recursive] <command>
   or: git submodule [--quiet] sync [--] [<path>...]

ERROR: Failed to build PhantomJS! Initial update of git submodules failed.
Executing in /root/build/src: git submodule init
Executing in /root/build/src: git submodule update --remote

Which operating system are you using?
OS X, using Docker 1.9.1

The problem seems to be this commit: cde33dc6fbae8c5d8950ee9b5336c8c66912dc64
If I build against 74ed9404c2ac70a100d6b9a666633b4e02568e94 everything works as expected

Compilation of master without commit cde33dc6fbae8c5d8950ee9b5336c8c66912dc64 also works as expected.

This was originally to fix #14082. Any comments @mepard?

Thanks, reverting cde33dc6fbae8c5d8950ee9b5336c8c66912dc64 let me build 5232c9dff98115600d1b1491e538e01c0de2d650
Needed the DPI setting.
Prerequisites: git and docker.
Deploy instructions from here: https://github.com/ariya/phantomjs/tree/master/deploy

git clone https://github.com/ariya/phantomjs.git
cd phantomjs
git checkout 5232c9dff98115600d1b1491e538e01c0de2d650
git revert -n cde33dc6fbae8c5d8950ee9b5336c8c66912dc64
docker run -v $PWD:/src tubia/debian:wheezy /src/deploy/docker-build.sh


Avoid some allocations

I've got some bigger improvements coming as part of the first step toward #228, but this is just gathering some low-hanging binary-compatible fruit.
Before:
Benchmark                          Mode  Cnt      Score     Error  Units
DecodingBenchmark.decodeFoosC     thrpt   40   4382.738 ±  57.071  ops/s
DecodingBenchmark.decodeIntsC     thrpt   40  21273.053 ± 500.874  ops/s

After:
Benchmark                          Mode  Cnt      Score     Error  Units
DecodingBenchmark.decodeFoosC     thrpt   40   4772.316 ±  26.420  ops/s
DecodingBenchmark.decodeIntsC     thrpt   40  23965.881 ± 190.893  ops/s

Current coverage is 81.75% (diff: 87.50%)

Merging #457 into master will increase coverage by 0.20%

@@             master       #457   diff @@
==========================================
  Files            80         80
  Lines          2135       2132     -3
  Methods        2043       2033    -10
  Messages          0          0
  Branches         92         99     +7
==========================================
+ Hits           1741       1743     +2
+ Misses          394        389     -5
  Partials          0          0

Powered by Codecov. Last update 80eed79...8c911de


🚀 - Add Card Holder Name for InputCardGrouped

Description
Some countries regulations or payment solutions APIs require cardHolderName. In this case, a field is required for the card holder name and the component is rendered unusable.

I don't think there's anything to be done here to warrant a whole new component, you can just add autofill="cc-name" input to the same form and it should work.
Example for v4
<tui-textfield>
  <label tuiLabel>Card holder</label>
  <input
    tuiTextfield
    formControlName="holder"
    autocomplete="cc-name"
  />
</tui-textfield>

https://stackblitz.com/edit/angular-scw1un


There are probably some bugs relating to normal people being able to use @everyone

Fix with message.cleanContent instead of message.content

Fixed in d9ba9912f2673590bc27f799a790711842a5c9bf


Allow passing in timeout/retry settings to Net::HTTP

When fetching remote XML files from arbitrary URLs, you might want to configure different values for timeouts/retries to avoid allowing users to DoS you via intentionally slow endpoints.  The Net::HTTP defaults are 60 seconds plus 1 retry, which could easily deplete resources if intentionally exploited.

@pitbulk this looks reasonable to merge.

@johnnyshields  I think its ok, but I gonna prefer request_options to be the last paramter of parse_remote and parse_remote_to_array methods, rather than include them before the options parameter.
@tjschuck can you do the change?

@pitbulk Pushed!
Thanks @pitbulk and @johnnyshields

Sorry to bikeshed this, I misread @pitbulk's earlier comment. I was thinking we would make these just part of options. But I guess it's fine this way too.

@johnnyshields That's fine with me too.  I honestly can't remember why I did it this way...
If you want me to push it into options, just let me know and I can do that shortly.

@johnnyshields I think it's ok to have both options differentiated. As we are introducing now all related to the request, I'm not against to leave its own place. Wonder if there is a way to inject them in the request rather than initialize each parameter one by one.
 http.open_timeout = request_options[:open_timeout] if request_options[:open_timeout]
 http.read_timeout = request_options[:read_timeout] if request_options[:read_timeout]
 http.max_retries = request_options[:max_retries] if request_options[:max_retries]

Using one options arg avoids human errors like the one I just caught. I strongly prefer it.

Also this should go on branch V2.0.0 where we force new ruby version

@tjschuck I'll take this one over.

@johnnyshields Go nuts!  Thanks.

Replaced by #709


terra-core heroku master branch deployment failures

Issue Description
Whenever a branch is merged into master, a deployment on heroku is triggered. This has recently started to fail consistently whenever we merge a branch in to master.
Interestingly, all of our PR deployments through heroku do not fail, it is only the ones generated from merge to master that fail.
As far as I can tell, it may be related to how we are specifying node versions the project intends to use and how heroku determines what node version to use, heroku is installing node 11.15.0
This is output from the heroku build:
Installing binaries
engines.node (package.json):  >=8.9.2 <12
engines.npm (package.json):   unspecified (use default)

Resolving node version >=8.9.2 <12...
Downloading and installing node 11.15.0...
Using default npm version: 6.7.0

I believe there are issues with node 11, and thats the version heroku is resolving to use for this deployment.
There is also an error around multiple versions of disclosure-manager:

       ERROR in terra-disclosure-manager
npm ERR<EMAIL_ADDRESS>compile:prod: `webpack --config webpack.config.js -p`
npm ERR! Exit status 2
npm ERR!
npm ERR!         Multiple versions of terra-disclosure-manager found:
 Failed at the<EMAIL_ADDRESS>compile:prod script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
           4.17.0 ./~/terra-application/~/terra-disclosure-manager
           4.9.0 ./~/terra-disclosure-manager

npm ERR!  Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>heroku-postbuild script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /tmp/npmcache.AZddu/_logs/2019-08-21T14_42_40_489Z-debug.log

Additional Context / Screenshots
This doesn't really impact much that we do, though I wanted to document it in-case anyone was interested/had time to investigate.

Talked about updating engine to node >=8.9.2 <11


Change default root to /var/lib/rancher/convoy

Per @ibuildthecloud, we should use the /var/lib/rancher dir on the host. Important for rancherOS.
Disclaimer: I've done no validation of this change. Just made the change and pushed.

@ibuildthecloud I need to understand why it's "important to rancherOS"...

@yasker RancherOS does not persist /var by default, only /var/lib/rancher.  So /var/lib/convoy is lost after reboot.

The problem is all the old version would have configure file at /var/lib/convoy, so either we got to help user to upgrade, or specify configure file location for RancherOS.

Honestly this doesn't matter a huge deal.  In every place we use Convoy in Rancher we change the path to /var/lib/rancher/convoy.  It would have been nice if that was the default path, but I don't know if it's worth the effort now.


Bug/debug why test fail

Bug/debug why test fail

Closing this as this pr was for debugging purpose


[SDPA-1762] Adds RplFieldset component, adds grouped fields stories.

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-1762
Changed

Adds RplFieldset component to encapsulate fieldset wrapper and grid.
Adds example story for using grouped fields and nested object for address component (see form story)

Screenshots

@tim-yao - The documentation for using form groups is within the forms story, best to lead by example.  I think the best place for implementation documentation will be on the nuxt side as this is where there will need to be changes (see https://github.com/dpc-sdp/vic-gov-au/pull/450)
We should consider a way to add custom text to the Readme, as is anything added will be removed after the docs are regenerated.


Fixed various issues related to authorization

Fixed panicing middleware.
Fixed broken authorization.
+Small renaming for readability

Coverage decreased (-0.07%) to 46.952% when pulling 06b2b17068f735d325fc1bb32f47840711889042 on egopher:fixes into 2fa6391f4440fb0cefca7ab8fece540533c48cdd on supergiant:master.


[UPDATE] Remove deprecated code from hydrography sample

Description

Remove deprecated code from hydrography sample.
Use property to control progress indicator visibility
Modularize the sample
Moved the adding on the progress indicator to the StackPane (I found it was not being seen because it was under the map)

Checklist:

[ ] Set target branch to main (current release) or v.next (next release)
[ ] Request a reviewer
[ ] Assign a reviewer
[ ] Tag reviewer in comment

Assigning to @jenmerritt but really superseded by other work.

@jenmerritt modularized this - ready for another review.


add label name function

I think we can custom label name in attach action

Hey, you can use
->recordSelect(function (Select $select) {
    return $select
        ->label('New label here');
})

Ah,, i am sorry, thank you for your answer


Person Name

Hello,
i hve the E340 Doorbell, in the ntities i see the person name entity, but no names apperas, also the person detection doesn't work? In the Eufy Mobile App it works fine.
Could somebody help?
Best regards

https://github.com/fuatakgun/eufy_security/issues/1037


TLB Misses

To obtain measurements that are consistent with an "idealized execution", you map all memory addresses to the same cache set. While this eliminates cache misses, it does not prevent TLB misses.
An example is the 4881c4380800005b5d415c415d415e415f benchmark ("add rsp,0x838; pop rbx; pop rbp; pop r12; pop r13; pop r14; pop r15"). The difference between your measurements/Ithemal (~6 cycles) and IACA (3 cycles) is due to TLB misses.

Thanks for filing this issue. You are correct. How did you detect the TLB misses? I tried using (PERF_COUNT_HW_CACHE_DTLB) | (PERF_COUNT_HW_CACHE_OP_READ << 8) | (PERF_COUNT_HW_CACHE_RESULT_MISS << 16) but got a counter of 0.

I think the counter you mention measures only L2 TLB misses. I used the DTLB_LOAD_MISSES.STLB_HIT counter which counts loads that miss in the L1 TLB but hit in the L2 TLB.


ucsd_blacklight.ucsd_blacklight mapper validation results

spatial / coverage / subject: map geographic_tesim to spatial only (this is currently getting mapped 3 times, to coverage/subject/spatial. Please do not map geographic_tesim to coverage. The current mapping to subject is okay.)

This isn't happening in the mapper as it turns out. There was mapping configuration for coverage, but removing it didn't resolve this because map_couch_to_solr_doc() method in mapper is copying the spatial field to coverage:
                'coverage': filter_blank_values(record.get('spatial')),
Should I remove that line?

@lthurston Lucas, following on our brief discussion of the UCSD mapper yesterday -- we'd like to accept the new Rikolti mapper as-is, where it's taking data from geographic_tesim and mapping it to spatial (which is also then subsequently getting copied to coverage from a subsequent map_couch_to_solr_doc() function). We did some further digging into the legacy code, and it turns out that the Calisphere front-end only displays one of those data values (not both) in the front end -- and it's going to be messy to try to refactor the map_couch_to_solr_doc() function and/or front-end at this time. It's something we can come back to once we cut over to Rikolti / after this year.
Going that route, we can close out this PR: Remove coverage field mapping in blacklight #622

See: https://github.com/ucldc/rikolti/issues/620#issuecomment-1802659599
This mapper is accepted! @amywieliczka @barbarahui


Fix UX Guide Image Sizing

Some of the images are too big/sizing is not consistent throughout the page