BulkInsertOrUpdateAsync fails for Postgres when unique index required

First of thank you for all the great work you guys do on this library.
So the issue I've experienced is using the BulkInsertOrUpdateAsync extension method and supplying update properties in the BulkConfig.  When executed the action fails with the below error:
Exception data:
    Severity: ERROR
    SqlState: 42704
    MessageText: index "<indexName>" does not exist
    File: tablecmds.c
    Line: 1299
    Routine: DropErrorMsgNonExistent

I've created the below small program that replicates the issue:
using EFCore.BulkExtensions;
using Microsoft.EntityFrameworkCore;
using System.ComponentModel.DataAnnotations;

namespace EFBulkExtensionsPostgresIssue
{
    internal class Program
    {
        static async Task Main(string[] args)
        {
            var optionsBuilder = new DbContextOptionsBuilder<BloggingContext>();

            var connectionString = Environment.GetEnvironmentVariable("DB_CONNECTION_STRING");
            optionsBuilder
                .UseNpgsql(connectionString);
            using var context = new BloggingContext(optionsBuilder.Options);

            var canConnect = await context.Database.CanConnectAsync();

            if (!canConnect)
            {
                Console.WriteLine("Cannot connect to database");
                return;
            }

            var items = new List<BlogAggregation>();
            Random rnd = new();

            for (var i = 0; i < 20; i++)
            {
                items.Add(new BlogAggregation
                {
                    Id = Guid.NewGuid(),
                    AggregationTypeId = 1,
                    CreatedAt = DateTime.UtcNow,
                    Value = rnd.Next(),
                    AggregationDate = DateTime.UtcNow.AddDays(i * -1)
                });
            }

            var bulkConfig = new BulkConfig
            {
                UpdateByProperties = [nameof(BlogAggregation.AggregationTypeId), nameof(BlogAggregation.AggregationDate)],
                PropertiesToIncludeOnUpdate = [nameof(BlogAggregation.Value), nameof(BlogAggregation.CreatedAt)]
            };

            await context.BulkInsertOrUpdateAsync(items, bulkConfig);
        }
    }

    public class BlogAggregation
    {
        [Key]
        public Guid Id { get; init; }
        public int AggregationTypeId { get; init; }
        public int Value { get; init; }
        public DateTime CreatedAt { get; init; }
        public DateTime AggregationDate { get; init; }
    }

    public class BloggingContext : DbContext
    {
        public DbSet<BlogAggregation> BlogAggregations { get; set; } = null!;

        public BloggingContext(DbContextOptions<BloggingContext> options) : base(options)
        {
        }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.HasDefaultSchema("MySchema");
        }
    }
}

Using this I've diagnosed the issue and the two below are contributing:

index name lengths are limited to 64 chars in Postgres
the schema name is missing when the index is attempted to be deleted

I've also got a solution for this and would love to raise a PR but sharing here first as per the contributing guidelines.

Glad it's useful.
Sure make a PR, add also a test for the issue.

I find it interesting that he still creates a unique index when the table's unique index exists. In the case of very large data volumes, the cost of creating unique indexes is huge. Is it possible to disable the automatic creation of unique indexes?

I found out that it is the following SQL, which does not find the unique index that already exists.
I tried to fix this problem, but I am not familiar with SQL.
SELECT COUNT(distinct c.conname)

                  FROM pg_catalog.pg_namespace nr,

                      pg_catalog.pg_class r,

                      pg_catalog.pg_attribute a,

                      pg_catalog.pg_namespace nc,

                      pg_catalog.pg_constraint c

                  WHERE nr.oid = r.relnamespace

                  AND r.oid = a.attrelid

                  AND nc.oid = c.connamespace

                  AND r.oid =

                      CASE c.contype

                          WHEN 'f'::"char" THEN c.confrelid

                      ELSE c.conrelid

                          END

                      AND (a.attnum = ANY (

                          CASE c.contype

                      WHEN 'f'::"char" THEN c.confkey

                          ELSE c.conkey

                          END))

                      AND NOT a.attisdropped

                      AND (c.contype = ANY (ARRAY ['p'::"char", 'u'::"char"]))

                      AND (r.relkind = ANY (ARRAY ['r'::"char", 'p'::"char"])) AND r.relname = 'supplier_room_rate_analyzes' AND nr.nspname = 'public' AND a.attname IN('SupplierId','HotelCode','RoomId')

@borisdj, do I need to be added as a contributor? I don't have the option to create a branch.

Hi, no, you need to create a Fork, then ones you do a commit, make a PR.

PR merged, thx for contrib.
Will be publish soon with next nuget version.


Update shareable-link.md

We do support peered vnet connection in same subscription and region, but we do not support peered vnets in different subscriptions.

Learn Build status updates of commit acc411f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/bastion/shareable-link.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off

Updated in a different PR. All set! #please-close


⚠ Spryr has degraded performance

In 03dbae8, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6337 ms

Resolved: Spryr performance has improved in f3c513c after 7 minutes.


Provide a confirmation step for Apply

Issue Type: Feature Request
One participant - a SQL MVP - asked for a confirmation for the Apply action.
The concern was that an action applied against a production database could have great and maybe unintended consequences, so a confirmation is warranted.
(Note another participant asked for the ability to rollback an Apply for similar reasons.)
Azure Data Studio version: Azure Data Studio 1.9.0-insider (cecc899949d30fd28f1636cdb65e3a6e17e46a3a, 2019-06-18T07:15:07.629Z)
OS version: Darwin x64 18.6.0

@gengelbms Which feature is this for? Schema Compare?

This one is also a good candidate if it is easy to implement.

Available in the latest builds


[DIGEST] Routing

Route for inherit? #672
Routes does not respect extensions #659
Named routes #547
Conflicting controllers with the same name under different subfolders #445

Closing issue as its moot with tags


chore: update-pre-commit

What does this pull request change?
Why is this pull request needed?
Issues related to this change:

good, the ci failure might be fixed in equinor/dm-core-packages#443

The CI is always failing in this repo now. See for example https://github.com/equinor/create-dm-app/pull/80, where the change is clearly not responsible for the failure. This is due to create-dm-app being in the middle of a rewriting


Rails5 deprecation warning

DEPRECATION WARNING: before_filter is deprecated and will be removed in Rails 5.1. Use before_action instead. (called from included at /Users/.rvm/gems/ruby-2.2.2/gems/impressionist-1.5.1/app/controllers/impressionist_controller.rb:12)

Also have this problem. Any controller I open that contains impressionist actions: i get DEPRECATION WARNING: before_filter is deprecated and will be removed in Rails 5.1.

#241 fixes this, closing.


Importing modules.css styles in ts

Keeps saying styles is declared but never read even after including it in the ts.config file.

Solved


⚠ Mobitopup has degraded performance

In c568c99, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 7859 ms

Resolved: Mobitopup performance has improved in a3eba3c.


Update/Add nvm

Tool information

Tool name: nvm
Add or update? Add
Desired version: latest
Approximate size:
If this is an add request:

Brief description of tool: install/switch between Node.js versions, auto-installs & uses the node version as specified in a project’s .nvmrc file
URL for tool's homepage: https://github.com/nvm-sh/nvm

Virtual environments affected

[ ] macOS 10.14
[x] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

(Already installed one macOS virtual env)
Can this tool be installed during the build?
Yes, but not easily. See https://github.community/t5/GitHub-Actions/How-to-share-shell-profile-between-steps-or-how-to-use-nvm-rvm/td-p/33185
Are you willing to submit a PR?

setup-node requires an explicit version of nodejs needed for the job to be specified in the with: property.
Many nodejs projects already have a file in their repo that specifies a desired node version (e.g..nvmrc), so having to repeat the desired node version in .github/workflows/*.yml would seem unnecessary duplication.
If setup-node could detect and use the version specified in an existing .nvmrc then arguably installing nvm would not be necessary.
Additional points:

The macOS virtual env already has nvm preinstalled (but the Linux ones don’t)
CI services such as TravisCI have nvm preinstalled for this very purpose

I'm working with the setup-node owners to determine if https://github.com/actions/setup-node/issues/32 is something they plan on fixing. Leaving this issue open for now to track.

Thanks for the suggestion @scottohara, we're going to add nvm as requested.
@maxim-lobanov , let's do it, let me know if you need any additional assistance.

@scottohara , before we start adding it to the Linux, could you please confirm that it works as expected on macOS images? I am asking because as I know, nvm doesn't work properly on macOS due to NVM initialization way and you have to write initialization trick in each pipeline:
# Delete prefix from system node config
npm config delete prefix
# Need for using `nvm` inside script
. "$HOME/.nvm/nvm.sh"
nvm use $version
nvm alias default $(nvm current)

echo "Node version: $(node -v)"
echo "npm version: $(npm -v)"

# Update `PATH` for the next steps (AzDO specific)
echo '##vso[task.setvariable variable=PATH;]'$PATH
# Update `PATH` for the next steps (GitHub specific)
echo "::set-env name=PATH::$PATH"

This issue comes from the way how to NVM works, it initializes per-session only and we have to use tricks to pass it to the next steps.
@thejoebourneidentity , . nvmrc and .node_version are very popular approaches to specify node. Is it something that we would like to add to setup-node task? We can improve setup-node task to look for these files and use them if they exist

@maxim-lobanov the preinstalled nvm on the macOS image certainly works.
Yes, you need to delete the npm prefix, and to have it pass through to each step you need to specify a login shell (shell: bash --login {0}).
Other than that, it works fine.
runs-on: macOS-latest
steps:
  - name: Checkout repository
    uses: actions/checkout@v1

  - name: Delete npm prefix
    run: npm config delete prefix

  - name: Install dependencies
    shell: bash --login {0}
    run: nvm install && nvm use && npm ci

  - name: Lint and test
    env:
      DEVELOPER_DIR: /Applications/Xcode_11.3.app/Contents/Developer
    shell: bash --login {0}
    run: nvm use && npm test

@maxim-lobanov the preinstalled nvm on the macOS image certainly works.
Yes, you need to delete the npm prefix, and to have it pass through to each step you need to specify a login shell (shell: bash --login {0}).
Other than that, it works fine.
runs-on: macOS-latest
steps:
  - name: Checkout repository
    uses: actions/checkout@v1

  - name: Delete npm prefix
    run: npm config delete prefix

  - name: Install dependencies
    shell: bash --login {0}
    run: nvm install && nvm use && npm ci

  - name: Lint and test
    shell: bash --login {0}
    run: nvm use && npm test

This stopped working few hours ago. Now it uses some generic shell

@Kerizer , no changes from our side for macOS images for the last 2-3 days...

@Kerizer , your screenshot is from ubuntu image. NVM is not installed there.
macOS still works as expected.

@Kerizer , your screenshot is from ubuntu image. NVM is not installed there.
macOS still works as expected.

Still, we used nvm on Ubuntu as following:
      - name: Install nvm
        shell: bash -l {0}
        # In case if we need to use node or nvm - we should only use it from `bash -l {0}` shell.
        run: |
          cd ..
          echo $HOME
          curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.35.3/install.sh | bash
          export NVM_DIR="$HOME/.nvm"
          echo "source $NVM_DIR/nvm.sh --install" >> "$HOME/.bash_profile"
          cd -
          source $HOME/.bash_profile
          nvm install

And it worked for us for month. Starting from today it fails and it looks like macOS and Ubuntu now work in a different ways and are inconsistent.

Thank you for the provided example,
We will take care about installation NVM on Ubuntu.
As for the issue above, no changes related to shell from our side, but we will take a look

@maxim-lobanov we (me and @Kerizer) have been rather internally confused a bit, are you a Github staff memeber or just a core contributor/maintainer of this package?
Our workflows are a bit clusterfucked now, since we relied quite heavily on NVM setup this way (you know, because no other way) and now it would be nice if we could at least point them to some commit where it's not yet broken 🙇
Thanks for quick responses anyway!

Maintainer of virtual-environments repository.
I have found the root cause of your issue. And unfortunately, it caused by the latest Ubuntu image update. In scope of this update, we have added XDG_CONFIG_HOME variable to image. And it looks like NPM installation depends on this variable. So as a result, npm started to be installed to “$HOME/.config/nvm” instead of “$HOME/.nvm”.
Could you please modify script a bit to fix your build? (the single difference is that we define NVM_DIR before NVM installation and invoke mkdir
cd ..
echo $HOME
export NVM_DIR="$HOME/.nvm"
mkdir $NVM_DIR
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.35.3/install.sh | bash
echo "source $NVM_DIR/nvm.sh --install" >> "$HOME/.bash_profile"
cd -
source $HOME/.bash_profile
nvm install

This script will unblock your build.
Sorry for inconvenience and hope adding NVM to the image will improve your experience a bit :)

it caused by the latest Ubuntu image update

Just curious, was it a silent automatic update, or does this package fix the version some place and some commit updated it? I am not sure how to parse this json file in my head, so if you feel like sharing some insides, much appreciated 🙌
https://github.com/actions/virtual-environments/blob/master/images/linux/ubuntu1804.json
If the updates of images are not frozen, would it make sense to freeze them maybe to at least release them alongside updates of action itself?
I remember already voicing a concern somewhere how all updates to all actions by design are released to upstreams like actions/checkout@v2 what can cause overnight failures like this one.
\cc @bryanmacfarlane

Unfortunately, it is not possible for images. All hosted VMs have the same image version.
That json file is just packer template to generate image. We generate image based on this template on weekly basis and update all VMs automatically from our side.
Image updates can be tracked by updating README file or Releases in this repository

So was it a broken release then or a silent update with a nasty side effect?
If it is a release, can you point to the one which broke it, if you have any idea? 🙇

@dentuzhik Let me explain it a bit. We have a user request to add this variable - https://github.com/actions/virtual-environments/issues/491 but unfortunately, that can be done only during the VM initialization process in the scope of self-hosted pool creation, so that we've made changes in the internal scripts to add the variable that's why you couldn't find it at https://github.com/actions/virtual-environments/blob/master/images/linux/ubuntu1804.json. We're going to roll back the changes asap, sorry for the inconvenience.

Thanks for the explanation, much appreciated 🙌
We are not against a breaking migration, if it's really necessary, since we sort of relied on not that much documented feature, but I guess we could make somewhat a "major" release of images where it would be possible to upgrade?

Unfortunately, it is not possible for images. All hosted VMs have the same image version.

I know it was mentioned before, but it would rather make sense to have a way to update images in the backwards incompatible way with providing a path to migrate to them, no?
I mean if we look at version of alpine-linux or ubuntu images in Docker, there's a way to freeze previous version and release a new one which can be broken, without affecting existing CI machines around the globe:
https://hub.docker.com/_/ubuntu/?tab=tags
https://hub.docker.com/_/alpine?tab=tags
If we look at example from CircleCI, they have similar stuff:
https://circleci.com/docs/2.0/executor-intro/#machine

Now after writing all this, I think the problem here which has surfaced, that versioning of images is too tightly coupled to versioning of distributions, which has backfired when you need to have two different versions of the same distro for fixes like https://github.com/actions/virtual-environments/issues/491
I understand that this is oftopic of this issues, and I can help creating a new one, if you guys agree this is a problem

@dentuzhik, Yes, please create the issue for your proposal. As I know, we don't have any plans in this direction at the moment but it would be great to have the issue where we can track popularity of this proposal.

Also note that there is currently a way to freeze the toolsets and image.  Self-hosted runners (on cloud VMs as well).  Yes, much less convenient but at the other end of the spectrum on the control vs convenience spectrum.  Not saying that having a way to pin or do n-1 on images, just highlighting as an option.
Off topic but actions also have a way to pin to a specific version (ref or sha).
So between self-hosted and what you choose to pin to, it is possible to lock down.  One thing you can't lockdown (until actions is in GHES) is the service itself and that moves forward.

Yes, much less convenient but at the other end of the spectrum on the control vs convenience spectrum.

This is indeed a valid point.
But I would assume that the default option which is provided to consumers of this package should be safe enough to avoid accidental breaking.
Therefore there should likely be both a way to push to upstream and let others update (docker actions/checkout@v2:latest tag / master branch) and provide a way to reference to specific LTS version (actions/checkout@v2 in v2 branch) AND have a way to create backwards incompatible changes (actions/checkout@v2:20200319 or actions/checkout@v2.3).
Versioning in a way that everyone is happy is hard.
I have put into my lists to come up with a proposal, at least for this specific case for virtual-environments, let's hope that I get there some time soon, we are quite under crossfires because of COVID-19

Hello, pull request was merged and changes would be added in the next images rollout.

Hello, changes were added. If you have any concerns please feel free to reopen the issue.

shouldn't sth like this:
name: nvm
on:
  push:
jobs:
  nvm:
    runs-on: ubuntu-latest
    steps:
      - name: try out nvm
        run: nvm install

work now?
Getting:
Run nvm install
/home/runner/work/_temp/6f648dd8-690e-4082-9988-421256974eb6.sh: line 1: nvm: command not found
##[error]Process completed with exit code 127.

shouldn't sth like this:
name: nvm
on:
  push:
jobs:
  nvm:
    runs-on: ubuntu-latest
    steps:
      - name: try out nvm
        run: nvm install

work now?
Getting:
Run nvm install
/home/runner/work/_temp/6f648dd8-690e-4082-9988-421256974eb6.sh: line 1: nvm: command not found
##[error]Process completed with exit code 127.

nvm is a shell function, not an executable.
You need to use global shell, like this:
name: nvm
on:
  push:
jobs:
  nvm:
    runs-on: ubuntu-latest
    steps:
      - name: try out nvm
        shell: bash -l {0}
        run: nvm install

A little off topic, but wanted to document somewhere just incase someone else was struggling with getting nvm to work with self hosted runners and very little set up each step. If you're building your own runners images I found this works pretty well.
RUN curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.35.3/install.sh | bash
RUN echo -n 'nvm "$@" && echo $NVM_BIN >> $GITHUB_PATH' >>  ~/.nvm/nvm.sh
RUN sudo cp ~/.nvm/nvm.sh /usr/local/bin/nvm
RUN sudo chmod +x /usr/local/bin/nvm

From here nvm install <version> && nvm use <version> persists and is very speedy.

The version of nvm that seems to be present on the runners atm is v0.39.2 - I published v0.39.3 earlier today. How long should it take before that's updated? I'd have hoped it's largely in realtime.

Since actions/setup-node does not resolve version aliases like lts/* in a timely manner, we're switching our workflows off actions/setup-node and to the built-in nvm, adding the new path to node to the start of the $PATH:
# Use nvm because actions/setup-node does not install latest versions
# https://github.com/actions/setup-node/issues/940
- name: Install latest LTS with nvm
  run: |
    nvm install 'lts/*'
    echo "$(dirname $(which node))" >> $GITHUB_PATH
  shell: bash -l {0}

This allows all further workflow steps to use the new node binary without nvm use:
- run: node --version # v20.16.0 (latest LTS just installed by nvm)
- run: which node # /home/runner/.nvm/versions/node/v20.16.0/bin/node


Comparison with typebox?

The README has a great breakdown of how Zod compares with many similar libraries: https://github.com/colinhacks/zod#comparison. @sinclair/typebox could be added to the comparison; it seems to be comparable with similar syntax, and is widely used.

+1
Seems like a good idea.

November, 2022 from the author of typebox on reddit:

TypeBox does however support the following additional functionality (if you need it)

Strict compliance with the JSON Schema specification.
Currently fastest JIT compiler / validator for JavaScript.
Closer alignment to the TypeScript Type System and Utility Types.
Supports Recursive Type Inference
Supports Conditional Type Mapping
Supports Custom Types with User Defined Inference Rules.
Value API for performing type operations on JavaScript values.
Extensive Test Suite for Ajv compliance
A more low level API to allow you to tune validation pipelines.

The benefit of using TypeBox mostly comes from being aligned to an industry standard specification (JSON Schema). From this you can leverage many frameworks that standardize on the specification (Fastify, OpenAPI, etc), utilize tools to auto generate documentation, generate user interface forms, or simply share types (and validation logic) across multiple systems (useful for micro services). TypeBox also uses the JSON Schema specification as it's Type Reflection Model.
In terms of performance, TypeBox currently ranks 3rd overall benchmarks here, with 1st and 2nd place given to AOT solutions (ones that require TS compiler transforms). TypeBox (like Ajv) provides a JIT compiler solution (for servers) as well as a dynamic type checking solution for applications under code evaluation constraints (for example browsers and certain cloud function providers)
Overall, I think most users of TypeBox just enjoy being closely aligned to JSON Schema specification while still being able to work with a Zod-like API.
Hope that helps

(Not sure what else might have changed since then.)


Favicon seems to still be react app default

Likely just update with Workers logo?

https://docs.google.com/spreadsheets/d/1Fa5e8OniZXCPy3RdMR7B4i5zQpIOzcjhIMh9hfnJ_ZQ/edit#gid=0


Make ARM Runner for AWS

@evshvarov Can you provide more info on this one?

We solved it another way, so, can close now


Server -> Client State Machine Connection

Options

State Machine on server only (maybe)
State Machine on server + client (what we're attempting)
State Machine on client only ❌

Solutions

Store state machine in Hashmap with the key of gameId (in-memory)

Will start the state machine in the initiateGame client event and await the first prompt in which the state machine is stopped?
Whenever any future events occur that require the state machine, it will be referenced from the Hashmap and interpreted.

Keep main state machine on client, use sever to store state in database

Whenever a process that requires the single source of the server (timers, etc.), the client will ask for it and it will be sent by the server

Using a Observer/Subscriber pattern to handle the state machine tracking on the server

https://refactoring.guru/design-patterns/observer

Links

https://kyleshevlin.com/guidelines-for-state-machines-and-xstate
https://developers.redhat.com/articles/2021/11/23/how-design-state-machines-microservices#
https://github.com/statelyai/xstate/discussions/1684
https://github.com/davidkpiano/durable-entities-xstate/tree/main
https://deno.com/blog/build-crud-api-oak-denokv
https://examples.deno.land/http-server-websocket
https://socket.io/docs/v3/rooms/
https://www.programonaut.com/how-to-create-websocket-rooms-without-socket-io/

Options

State Machine on server only (maybe)
State Machine on server + client (what we're attempting)
State Machine on client only ❌

Solutions

Store state machine in Hashmap with the key of gameId (in-memory)
Will start the state machine in the initiateGame client event and await the first prompt in which the state machine is stopped?
Whenever any future events occur that require the state machine, it will be referenced from the Hashmap and interpreted.
Keep main state machine on client, store state in database
Whenever a process that requires the single source of the server (timers, etc.), the client will ask for it and it will be sent by the server

https://github.com/agrattan0820/beeeeeeeep/pull/7 Initially utilizes a state machine on the client and the server acts as the controller of game state and sends messages requiring real-time conditions whenever necessary.

Closing for now since the current connection where the clients send the server their state and it gets persisted in the database seems to work.


[ja] Localize eBPF for Japanese

Describe your changes
Localize eBPF into Japanese
Related issue number or link (ex: resolves #issue-number)
resolves #2804
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

The target page on the deploy preview site:

https://deploy-preview-2935--cncfglossary.netlify.app/ja/ebpf/


Error while deleting quota entry

vol quota status is on and we are trying to delete quota entry.
It gives error as :
Could not evaluate: Executing api call quota-resize volume t_vol failed: "No valid quota rules found in quota policy default for volume t_vol in Vserver data_vserver2. ".
manifest(site.pp)
node 'hnidhi-vsim1' {
}
node 'data_vserver2' {
netapp_quota { '/vol/t_vol/test_qtree':
ensure    => absent,
type      => tree,
volume    => 't_vol',
disklimit => '10G',
}
}

Merged!


datastore remove redundant call

removes redundant call

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1256   +/-   ##
=========================================
  Coverage     67.45%   67.45%
  Complexity     1314     1314
=========================================
  Files           186      186
  Lines          5387     5387
  Branches        547      547
=========================================
  Hits           3634     3634
  Misses         1521     1521
  Partials        232      232

Flag
Coverage Δ
Complexity Δ

#integration
?
?

#unittests
67.45% <0%> (ø)
1314 <0> (ø)
:arrow_down:

Impacted Files
Coverage Δ
Complexity Δ

...oud/gcp/data/datastore/core/DatastoreTemplate.java
92.09% <0%> (ø)
73 <0> (ø)
:arrow_down:

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 95244d6...2046070. Read the comment docs.


Not identifying clang

Hi!
On my system there was no clang file, so I had to make the link clang reffering to clang-3.5.
Is it an idea to change the code, using regex to just identify "clang"?
Best regards

It might be possible to check for both clang or clang-3.5 in the code. I'll investigate.

I've changed the check to clang++. Please see if it works on your machine.

Are you created a new kit with clang compiler by default?

Hi!
I think this is a old issue, I don't even remember what the issue was.
You should probably not follow this.
But thanks :)
Kind regards
Ove M Henriksen
On Sat, Sep 19, 2015 at 6:18 AM, Yusuf (Josè) Luis<EMAIL_ADDRESS>
wrote:

Are you created a new kit with clang compiler by default?
—
Reply to this email directly or view it on GitHub
https://github.com/fire/QtCreatorSourceCodeAccess/issues/6#issuecomment-141618949
.

I can't build and run the Editor with QtCreator, you could do it?
Are you workin in OsX, Windows or Linux?

I don't remember.
I were working in Linux
On Mon, Sep 21, 2015 at 8:06 PM, Yusuf (Josè) Luis<EMAIL_ADDRESS>
wrote:

I can't build and run the Editor with QtCreator, you could do it?
Are you workin in OsX, Windows or Linux?
—
Reply to this email directly or view it on GitHub
https://github.com/fire/QtCreatorSourceCodeAccess/issues/6#issuecomment-142063384
.

cd /usr/bin/
sudo ln -s clang-3.5 clang```

This is probably the command.

For what man?
My problem is in QtCreator with build and run the project, i configured all but doesn't work.


Show capture file names in Mizar GUI

Test: Unit, Manual
Bug: http://b/239695623

https://screenshot.googleplex.com/97mfxhBbRuGzTQR.png


Use absolute path for CARGO_HOME

OK, this should fix https://github.com/rust-lang/cargo/issues/1655.  It worked on my local.  I am not entirely sure how to write a test for this.  Can I get some help and be pointed in the right direction?

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the PR! You should be able to add a test in any of tests/*.rs, and I'd probably poke around the git or registry tests to make sure git repos and registry artifacts are checked out/download into the right locations.

Well, I updated it to address your comments @alexcrichton, but I still can't seem to write a test to replicate the behavior in https://github.com/rust-lang/cargo/issues/1655.

Ah this looks good to me for now anyway, so I'll just send this to bors
@bors: r+ f8b91b577b82dcaa8fef10ff35bfb7be1ddcb8a0

:hourglass: Testing commit f8b91b5 with merge 06dbe65...

And of course, thanks!

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64


Email issue and request

Hi,

When using the "Send email, I only get some information about the certificate, not the detailed information I get when I click on the Copybutton, which incidentally, doesn't copy the QR code2FA Name: Blah
Certificate Name:
Blah
Status: Valid
Expiration Time: 2026-03-10 12:16
Revoke Time: Not Revoked
IP Address: dynamic.pool
Serial: Blah
Registered e-mail<EMAIL_ADDRESS>Registered Country: Blah
Registered State/Region: Blah
Registered City: Blah
Registered Organisation: Blah
Registered Unit: Blah
Two Factor Authorisation (2FA) QR Code:
                  `

Can you add an option to email the configuration to the end user, as well?

Thanks

@reamasesa hello again :)
yes, this is one of the challenge I had during coding this functionality.
I can't figure out how to insert QR code into the mail body the way it will attach as image but not the binary.
Here is the piece of code which currently commented out. It's a good idea to drop JS and do all this on the server side with Go, but it demands time, which Im currently  lacking.
Appreciate community help with this.

I actually had the same request.
Will have a look on this code on the week, probably will found some solution and push pull request to you 😎

it will not be implemented due to limitations of mailto: protocol which does not support any type of attachments.
As alternate way we may use internal SMTP server or 3P server to sending emails, but this will overcomplicate all the solution.

it will not be implemented due to limitations of mailto: protocol which does not support any type of attachments.
As alternate way we may use internal SMTP server or 3P server to sending emails, but this will overcomplicate all the solution.

thanks for replying, I'm not a coder, not sure how to implement this (I have my own SMTP server), any pointers would be greatly appreciated
Thanks


method! examples in README need fn

Per https://github.com/d-unseductable/ruru/commit/09c8819e9dfa4639e4337f9f110e8bbb3e749078
This should prevent other people from hitting the following error message:
error: no rules expected the token `my_method_name`

Hi @malept! I wish cargo test could run README examples too 😄
Thank you!


Surface unordered list style?

Hello!
I was wondering if the jotdown parser could surface the character used for a list item in an unordered list (-, +, and *). Currently, the style is swallowed for ListKind::Unordered:
https://github.com/hellux/jotdown/blob/6e6c28ed5a5944f237e13e4d41a091bf623f30ae/src/lib.rs#L2570-L2572
I understand that the djot specification only has bullet stated for all three, but I think the distinction could help with eg. a custom HTML renderer that uses a different list-style-type.
Thanks!
EDIT: djot.js does include the bullet list style in its AST

I think this is reasonable to have, we could add an extra enum here. It may also be useful for e.g. formatters.

Bullet type exposed in events as of release 0.6.0.


OpenRefine 2.7 does not start on macOS Sierra

I've successfully used OpenRefine for quite some time. After installing the grefine-rdf-extension I started OpenRefine again and it got stuck while trying to reconcile data with DBpedia (just right after having configured the SPARQL end-point - it kept spinning while displaying the working message).
I closed OpenRefine with a Force Quit command, remove the application from my hard disk and re-installed it again with brew.
After starting up, things simply do not progress and all I get when accessing the localhost URL (http://<IP_ADDRESS>:3333/) is the butterfly message

Thanks in advance

I remove it with AppCleaner, and got it started again.


Optimize GET /indexes route

Pull Request
Related issue
Fixes #4694
What does this PR do?

Optimize indexes listings by not processing all indexes and only processing the ones that are used as the result of the pagination
Don't open the index if they are already cached

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

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Superseded by https://github.com/meilisearch/meilisearch/issues/4694#event-15789897476
Closing


Import listens from Foobar2000 playback statistics

This one is a bit more tricky. Currently foobar2000 stores play-count statistics for tracks, although no timestamps.
Timestamps are available with foo_enhanced_playcount though.
Following the same rationale than my other enhancement proposal, it would be great to be able to export/sync Foobar2000 listens to ListenBrainz. i.e. those not already sent. In some cases users may have installed foo_listenbrainz2 at a later point, so many listens would be lost. This could fill the hole retroactively.

Deduplication is required. Comparing the current listens JSON by the user against the local stamps. Since half the track must be listened to, it's more or less trivial to compute a match with JSON timestamp = local timestamp +/- half length  (and the tags).
Some tricks could be added here (like checking the plugin installation time, and only comparing listens before it). But I think it would be best to compare all values to ensure all values are synced on any use-case.
This is thought as a sync option. i.e. will only add new values, and only if they are not already on the server. A one time action for most users.

This only would make sense then if users have foo_enhanced_playcount installed.
This has absolutely no priority for me, but I would accept patches implementing this functionality.


Fix for double clicking on Mocha tests opens external editor.

Issue   #974
Bug
Jumping to line: 0, column: 0 causes JavaScript file to open in external editor.
Fix
Changed default to line: 1, column: 1.

Hi @jcansdale, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

It looks like I did a pull request on my 'master' branch, not realizing this would be a moving target (rather than a snapshot). I'll reissue the pull request on a separate branch that fixes just this issue. I'm familiar with subversion but git is new to me. Thanks for your patience. ;)


Allow to accept sync function for Nullable/unwrapOrElseAsync

We close this once by https://github.com/option-t/option-t/issues/1683#issuecomment-1456004819.
We'll reopen this when we conclude about #1683.
If you have some opinions, please comment to #1683


fix: use pkgId to get relative path (#4957)

Description
solve #4957
Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks for the fix @meixg, merging after testing locally. If you would like to keep collaborating, it would be good to have another PR that adds a test case for this. See https://github.com/vitejs/vite/blob/main/CONTRIBUTING.md#extending-the-test-suite

Thanks for the fix @meixg, merging after testing locally. If you would like to keep collaborating, it would be good to have another PR that adds a test case for this. See https://github.com/vitejs/vite/blob/main/CONTRIBUTING.md#extending-the-test-suite

@patak-js I just added a test case for this PR, see #5549


Made In panel section rendered unsuseable due to different factory levels spamming

As title says. Attached is an image showing said issue

The problem is known, but at the moment not solvable because if i hide these entries, upgrade planner does not work with the mod


SR-IOV systemd configuration improvements

#365 is merged but we have to follow up with following:

[ ] unit tests
[ ] make systemd unit file configurable (e.g. Wants and Before targets should be customizable)
[ ] operator should delete systemd service on uninstall
[ ] other bugfixes if any

@SchSeba feel free to add anything to the list above

closing this one with the great work done by @ykulazhenkov


Added three questions to the react json file

Three questions added to the react json

What is the problem please


Azure Storage Explorer (client tool & portal) | "Folder Statistics" relates to irrelevant paths

Hi Team
There is a bug with the “Folder Statistics” functionality (client tool & portal).
Given the following directories and files (please note that the directory name myfolder is a prefix of the other directories):
rootfolder/
├── myfolder
│   └── myfile.txt
├── myfolder_a
│   └── myfile.txt
├── myfolder_b
│   └── myfile.txt
├── myfolder_c
│   └── myfile.txt
└── myfolder_d
    └── myfile.txt

When standing inside the folder myfolder and using “Folder Statistics” the given result is 5 blobs (instead of 1 blob).
Apparently, the lookup
is done for /path/myfolder* instead of /path/myfolder/*

Duplicate of #1001


Fix clippy const fn warnings in v_frame

Not really sure how these didn't get caught earlier, but this fixes them now.

Never mind, it's because I was on nightly. This will need to be shelved for a while.


Nested JSON gets turned into nested arrays.

A queue message containing nested JSON get the nested properties turned into arrays.
The following queue message:

{"method":"foo","attr":{"templateName":"verify-user","templateContent":[],"detail":{"tags":["one","two"]},"check":false}}

Will output the following:

Node.js queue trigger function processed work item {"method":"foo","attr":[[[]],[[]],[[[[[],[]]]]],[[]]]}

When using the following function:

module.exports = function (context) {
context.log('Node.js queue trigger function processed work item ' + JSON.stringify(context.email))
}

Thanks for reporting - fixed via the attached commit.

Fix is included in this release: https://github.com/Azure/azure-webjobs-sdk-script/releases/tag/v0.0.1-alpha1-10007

Thanks for the work. Really very useful project!


HADOOP-14624. Add GenericTestUtils.DelayAnswer that accept slf4j logger API

https://issues.apache.org/jira/browse/HADOOP-14624 has been fixed. Closing.


danielroca-fixes

According to the blueprint the field importance it's an optional field that takes "HIGH" or "NORMAL", but it actually is an optional field that takes null, and lowercase "normal" or "high", also the behaviour if the field is set to null is not documented. This PR only addresses the casing issue.

@DaniRoca Thanks for fixing.
If "importance" is absent or null, "normal" is assumed. I clarified this in a separate commit.
BTW there is by default no difference between absent fields and null-valued fields.


Revert pointless commit to fix #2320

Revert "use custom object mapper to serialize json in order to avoid
null values"
This reverts commit 3ad759596f321ffb195256e045ec5803c00ee53c.
This fixes issue #2320. The original commit is pointless and dangerous,
as it surprisingly overrides the behavior of SwaggerSerializers.java,
but does essentially the same (but without supporting pretty printing).

I've added a second commit to remove a similar issue for YAML documents, where SwaggerSerializers.java has been ignored, too. Also, I've removed some split-and-then-rejoin-code that has been pointless since ages.

Is there any reason not to merge this?

I see the issue with #2320, and I absolutely understand how this commit fixes it. However, your fix now introduces a regression to #2105
/cc @fehguy


daemon: ignore CILIUM_CUSTOM_CNI_CONF when writing cni configuration

This file was overly-cautious when mimicking cni-install.sh's behavior. It turns out we want to be able to disable cni-install.sh but still have the daemon write the configuration file. This preserves existing behavior in 1.12.
Thus, we need to remove the bit that bails out if the environment variable is set.
Fixes: 12b7b11e1
Signed-off-by: Casey Callendrello<EMAIL_ADDRESS>Fixes a bug where the Helm value `cni.configMap` no longer worked.

/test

Test failure seems like a flake; Cilium was installed correctly but Kafka failed to go ready. One test failed, Cilium was successfully installed many times over the course of this CI run.
cmd: kubectl get pods -o wide --all-namespaces
Exitcode: -1
Err: signal: killed

/test-1.25-4.19

All green except ci-verifier. (is that supposed to run with /test?

/ci-verifier


Error when upgrading packer plugin

When I upgrade the packer plugin using the command packer init -upgrade I always get the following error:
❯ packer init -upgrade build-vm.pkr.hcl
Failed getting the "github.com/veertuinc/veertu-anka" plugin:
1 error occurred:
        * could not get sha256 checksum file for github.com/veertuinc/veertu-anka version 2.3.2. Is the file present on the release and correctly named ? GET https://github.com/veertuinc/packer-plugin-veertu-anka/releases/download/v2.3.2/packer-plugin-veertu-anka_v2.3.2_SHA256SUMS: 404  []

After checking the Release page, I saw that the latest packer release has some difference between the previous one.

Could you maybe change the name of the latest release and remove the x5.0 in the shasums ?

@NorseGaud it seems it had wrongly named the checksum file.

Got it, try now! Sorry about that

Keeping this open because we need a test to catch this in the automation. I'll try to add it in the next release

The checksum files won't have the API version on them.


@conda-forge-admin, please update version

Additional comment:
Upstream protobuf and gcprcio feedstocks have been updated for v1.65.5 so maybe this can be updated as well?
https://github.com/conda-forge/protobuf-feedstock/pull/215
https://github.com/conda-forge/grpc-cpp-feedstock/pull/370

Looks like this was fixed by #51. If a specific version from the "can't be compiled gap" is required, please open a new issue.


Cluster toggle when create new vertex in Studio

http://pasteboard.co/jrO15wo.gif
Cluster: [12, 18]
The default cluster is 12

Hi @StarpTech
if you have 2 or more cluster it is normal
The default cluster selection strategy is round robin
http://orientdb.com/docs/2.0/orientdb.wiki/Cluster-Selection.html

Cool, thanks @maggiolo00 can you also answer this question https://github.com/orientechnologies/orientdb/issues/4116


Issue with Manga-scantrad

Source information
Magan-scan 1.4.35
Source language
"French"
Steps to reproduce

Open the extension to see the List of his manga
enter a manga
click on a chapter
the image is not displayed

Expected behavior
When I click on a chapter, the images must be displayed so that I can read them.
Can you please solve the problem with Cloudflare? Each time we are oblique to open the web view.
It would be nice if we could fix the problem with Cloudflare so that we could go permanently.
Thank you.
Actual behavior
No image is displayed when we open a chapter.
Mihon/Tachiyomi version
Aniyomi <IP_ADDRESS>
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Extension seems to be working fine after passing Cloudflare.
For this issue of "The image couldn't be loaded", check WebView or in your own browser to see if said pages exist on the source's website. If they do exist, go to Mihon Settings > Data and storage, and "Clear chapter cache", then try opening it again after passing Cloudflare. If that doesn't help, come to the Keiyoushi Discord repo-support channel for more help.
As for the Cloudflare "issue", that's the source's way of protecting itself from bots/attacks/spam. Cloudflare and/or the source can change what they block whenever they want, understand when it happens & know how to change the user-agent when that happens (by following the Keiyoushi Troubleshooting Guide). There's really nothing to fix since you/the user can pass Cloudflare & view the pages with no issue. And, passing Cloudflare doesn't mean it'll last indefinitely, some sources are more strict with the protection level they set (where you need to pass Cloudflare on every page, not just initially).

Thanks for your reply. Have a great day.

Okay thanks


Create or Update an Organization Secret has wrong datatype on selected_repository_ids property

What article on docs.github.com is affected?
https://docs.github.com/en/rest/reference/actions#create-or-update-an-organization-secret
What part(s) of the article would you like to see updated?

Under the parameters section for PUT /orgs/{org}/actions/secrets/{secret_name} the selected_repository_ids parameter says the list should be an "array of strings". However, if you send the repository ID as a string value like the following payload:
{
   "encrypted_value":"MyEncryptedString",
   "key_id":"MyKeyId",
   "visibility":"selected",
   "selected_repository_ids":[
      "12345"
   ]
}

The API returns a 422 error with the following content:
{
   "message":"Invalid request.\n\nFor 'items', \"12345\" is not an integer.",
   "documentation_url":"https://docs.github.com/rest/reference/actions#create-or-update-an-organization-secret"
}

If you instead send an array of integers, the API call succeeds.
Based on this information the following line should be changed in the documentation:
selected_repository_ids | array of strings integers
Additional information

@GregBillings Thanks so much for opening an issue! I'll triage this for the team to take a look 👀


NeoButton variant conflicts

seems there are some conflicts between neobutton --k-accent and --rounded
https://github.com/kodadot/nft-gallery/pull/8838#issuecomment-1875331761

closed by #8851


Create TenRulesToBuildACommunity.md

Thanks!

@markcmiller86, is this ready to review?
Also, what is the associated GitHub Issue as per the process:

https://github.com/betterscientificsoftware/betterscientificsoftware.github.io/blob/master/Site/CuratedContentEditorialWorkflow.md

which says:

Authors submit Pull Requests (PR) for completed work

Add GitHub issue #<issue-id> to PR comment (to link PR and Issue).

?

@bartlettroscoe ... I didn't add an issue. I was sent a great article by @junghans, read it in its entirety, and decided on the spot to write a quick CC article for it. And, I am guessing it takes less time to do that and have others decide what to do with it than to first submit an issue, then wait for dialog and up or down votes, then wait for it move through to the ready-for-development state and then remember to write it.

@markcmiller86

I didn't add an issue.

Okay

And, I am guessing it takes less time to do that and have others decide what to do with it than to first submit an issue, then wait for dialog and up or down votes, then wait for it move through to the ready-for-development state and then remember to write it.

If an EB member likes a piece of content (and is not just publishing their own work), then I don't see problem with this.  Do we want less stuff on BSSw?
But we should likely get them to change to process to allow this shortcut workflow.

I've opened a separate issue (#439 ) for the higher-level discussion about the general idea of accepting PRs without topic review, etc.

If this is by @junghans why not add him as a contributor to the article?

It is not by @junghans. I wrote the CC article. @junghans simply brought it to my attention.

We want all articles to acknowledge the contributors.  So either you, or you and Christoph, I think should be listed.  We don't have to be that rigorous about who to include as contributors, as long as it is reasonable.

@markcmiller86  and @bartlettroscoe : Status on this one?

This is ready. I would merge to master myself. However, if I do without necessary metadata block, I fear it will fall through cracks. These are the last steps (6 & 7) in our workflow.
I don't know who should add the metadata block and what it should look like, in particular, key words and categories. Are those listed somewhere?

I added metadata.  I put it in the topic "Strategies for More Effective Teams".  The topics are exactly what you see on the production site.  We also have a categories metadata field, but it is now ignored, since they can be derived from the topics.
I adjusted the contributor links to point to the users' GitHub profiles rather than their entry in the organization team listing.
I merged, checked in preview, and have now marked the article for publication.  It will appear when the production site is next rebuilt. (Thursday at the latest, since we're publishing a blog article then.)


I could see this being configurable, select your LLM.

Thinking ahead, no bad ideas in brainstorming, right? 😂
Look at Click and Typer. Those are great ways to simplify Python Cli's.
I'm putting this more as a discussion, but actionable items would be:

[ ] Choose your reference LLM
[ ] Implement Typer
[ ] Implement Click
[ ] Add tests
[ ] Set up a developer env, Nix, Poetry, Pipenv
[ ] In an ideal world, some declarative model - Nix, Ondir/Pipenv - something
[ ] Set up python packaging to a binary
[ ] Github actions as a CI/CD mechanism to deliver binary releases

@Ceroxylon I know I just hopped in here, so apologies if I'm overstepping.
Those were just the first ideas that came to my mind to build out the tool.

Then rewrite in Rust 😂

Fantastic suggestions, thank you!! It is definitely clunky in its current form so that is a great roadmap to get it to be more polished and easier to develop.

playing around with incorporating click, not sure if it makes it more confusing to use since it isn't a step-by-step walkthrough but certainly looks more professional

Check out Real Python - they have phenomenal content.
If you want Python language specifics - Exercism is basically a free
knowledge trade platform for dev.
On Sun, Apr 14, 2024 at 6:51 PM ceroxylon @.***> wrote:

playing around with incorporating click, not sure if it makes it more
confusing to use since it isn't a step-by-step walkthrough but certainly
looks more professional
—
Reply to this email directly, view it on GitHub
https://github.com/Ceroxylon/konsensis/issues/1#issuecomment-2054210110,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIBBO4IMEK3OYGPJLK63Z3Y5MCATAVCNFSM6AAAAABGGKPENKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJUGIYTAMJRGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Expose NavigationController::IsInitialBlankNavigation

To help detect frame initial state change.
Prereq for fixing https://github.com/brave/browser-laptop/issues/3162
For an example using this, see https://github.com/brave/browser-laptop/compare/master...ayumi:add-is-initial-blank-navigation?expand=1

can you update this? It says brave:master, but the diff looks like electron:master

I've rebased on brave:master fe153b7beef08fcaf63a8882f3e82105d3b677c9 and tested it works with https://github.com/brave/browser-laptop/compare/master...ayumi:add-is-initial-blank-navigation
@bridiver could you take another look?

@bridiver ping on this review, thx

LGTM :smile: You even have it added to the webview, so that we don't have to use the web contents object :+1:


Delay reading and showing Data on UI.

Hi, Thanks for the Awesome plugin so far. I really appreciate your effort.
Am using the plugin to store data after I receive it from the API, The problem is wherever I try to read and display the data, I won't show the Data on UI until I hot reload or reload the App again.
balance = await storage.read(key: Constants.walletbal);    lastName = await storage.read(key: Constants.lname); //btcWallet    btcWalletAdrress = await storage.read(key: Constants.unique_bcwallet);
Wherever I print it on console, It will display null but after refresh or reload it will display the data

Hey @Harbdollar
Please ask this question at StackOverflow.  You can tag the question with flutter-secure-storage

Thanks


Switch to core22 base

Make required changes to enable snap build with core22 base
Relates to: https://github.com/docker-snap/docker-snap/issues/65

I've pushed a change to try and fix the previous workflow failure, but it seems you have to approve each run @xnox

I guess its because the branch is in my own fork

Snap was built at least.
I'm not 100% how to interpret the failure: https://github.com/docker-snap/docker-snap/actions/runs/4409275831/jobs/7725502411#step:10:6
Do you have any thoughts @xnox ?
Worth noting that the snap I built locally ran fine on a UC22 host, tested with hello-world docker image.
I suppose if this was my own CI permission in, I would add some debug to the workflow and run it again.

A core22 snap is not supported on an Ubuntu 20.04 system, right?  If it is, then this failure is worth digging more into, but if it isn't then you probably just want to update this line to ubuntu-22.04 instead:
https://github.com/docker-snap/docker-snap/blob/975f02462d78b74b96da4629dff480d14ec57542/.github/workflows/smoke-test.yml#L14

A core22 snap is not supported on an Ubuntu 20.04 system, right?  If it is, then this failure is worth digging more into, but if it isn't then you probably just want to update this line to ubuntu-22.04 instead:
https://github.com/docker-snap/docker-snap/blob/975f02462d78b74b96da4629dff480d14ec57542/.github/workflows/smoke-test.yml#L14

We could update to 22.04 host base for workflows. But in theory it should work. We have Lxd snap built on any base, and running on any host. Including when Lxd snap was base:core20 and when it moved to base:core22.

Ah, so the failure probably is worth investigating then, because that's even the most basic use case of docker run hello-world failing :sweat_smile:

Ah, so the failure probably is worth investigating then, because that's even the most basic use case of docker run hello-world failing sweat_smile

i think it might be fun things between focal host, core22 confinement, and changes in apparmor_parser --version which tries to parse things, and gets a denial to access /abi/ subdir, that is there, but not allowed to by the docker snapd interface.
This probably works on UC because we don't ship /abi/ directory on UC and nothing tries to access it either. Fun things.
Somewhat wonder if we should try upgrade to core20 first, cause fixing this issue may require upgrades to docker itself, and/or changes to the snapd interfaces.

@xnox Just to join the dots - what was the feedback from interface maintainers and apparmor maintainers ?


Documentation table of contents should not be vendored

Vendored sphinxcontrib.fulltoc can be replaced with an explicitly specified requirements.txt in readthedocs

This is a valid option but Hannes asked for it to be vendored because a
requirements.txt file sets a bad trend for specifying what may be required
for compiling docs.
Either way works.
On Jan 20, 2017 3:13 PM, "Natan Lao"<EMAIL_ADDRESS>wrote:

Vendored sphinxcontrib.fulltoc can be replaced with an explicitly
specified requirements.txt in readthedocs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil/issues/1447, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKPMlJD1hfnpswIGtnHY4lynuNKg1Tl8ks5rUT-KgaJpZM4Lp2VO
.

I cede to The Word of Hannes


Version error

I updated, or at least I think I updated, soundcloud-cli with pip, but sc -v still says it's 0.2.6
$ sc -v
sc 0.2.6

I tried fully uninstalling and reinstalling the program, but nothing changed.

Try pip install --upgrade soundcloud-cli.

I tried it earlier, but I just tried it again:
$ pip install --upgrade soundcloud-cli
Requirement already up-to-date: soundcloud-cli in /usr/local/lib/python2.7/dist-packages
Requirement already up-to-date: soundcloud in /usr/local/lib/python2.7/dist-packages (from soundcloud-cli)
Requirement already up-to-date: requests>=0.14.0 in /usr/local/lib/python2.7/dist-packages (from soundcloud->soundcloud-cli)
Requirement already up-to-date: fudge==1.0.3 in /usr/local/lib/python2.7/dist-packages (from soundcloud->soundcloud-cli)
Requirement already up-to-date: simplejson>=2.0 in /usr/local/lib/python2.7/dist-packages (from soundcloud->soundcloud-cli)
$ sc -v
sc 0.2.6

What does which sc report? Is it pointed to the version you just installed?

$ which sc
/usr/local/bin/sc

I'm not sure how to tell if it's pointed to the version I just installed. How would I check that?

Check if it's a symlink or the file that was just installed:
› ls -l `which sc`
-rwxr-xr-x  1 zk  admin  199 Jul  7 16:32 /usr/local/bin/sc*

Not a symlink, so check contents:
zk@qi ~/work/verus/optimus ‹master 1160254d3 22d›
› cat `which sc`
#!/usr/local/opt/python/bin/python2.7
# EASY-INSTALL-SCRIPT: 'soundcloud-cli==0.2.7','sc'
__requires__ = 'soundcloud-cli==0.2.7'
__import__('pkg_resources').run_script('soundcloud-cli==0.2.7', 'sc')

Everything is good.

I put in those commands, but I'm not sure what it means. cat which sc gave me a slightly different output:
$ ls -l `which sc`
-rwxr-xr-x 1 root root 60 juil.  9 21:51 /usr/local/bin/sc
$ cat `which sc`
#!/usr/bin/python
from soundcloud_cli import cli
cli.main()

You probably have multiple versions installed try to do pip uninstall soundcloud-cli until you see:
› pip uninstall soundcloud-cli
Cannot uninstall requirement soundcloud-cli, not installed

Manually rm the file in /usr/local/bin and install again.

Confirmed fixed!
sc -v
sc 0.2.7


hack,test: Gather the OLM registry resources during e2e runs

Adds a hack script that's responsible for gathering the CatalogSource and the corresponding PackageManifest custom resources that were used to deploy a custom version of Metering.
Updates the test helpers, adding a function that calls that bash script is responsible for setting the correct environment variables based on the user-provided e2e flags. The main_test.go e2e driver will run this gather registry artifacts test helper function when the e2e suite has been instantiated without the --run-dev-setup flag.

/lgtm

/retest


Update Invoker to handle Message objects as input

Now it can accept both string and Message objects as inputs. This change simplifies the process when the input is already a Message object, eliminating the need to convert it again.

Q
A

Breaks BC?
❌

New feature?
✔

Issues
https://github.com/spiral/roadrunner-bridge/issues/90

Summary by CodeRabbit

New Features

Enhanced input handling in the invoke method to support Message objects directly.

Bug Fixes

Fixed string formatting issues with added commas in assertResultType and assertInputType methods.

Tests

Added a new test testInvokeWithInputMessage to ensure proper handling of Message objects.
Improved testInvokeError with additional assertions for exception handling.
Introduced a packMessage utility method for test message preparation.
Updated createMessage method to return Message objects for testing purposes.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (94b5ec7) 76.58% compared to head (00cd2f6) 76.67%.

Files
Patch %
Lines

src/Invoker.php
60.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##                3.x      #25      +/-   ##
============================================
+ Coverage     76.58%   76.67%   +0.09%
- Complexity       94       95       +1
============================================
  Files             8        8
  Lines           252      253       +1
============================================
+ Hits            193      194       +1
  Misses           59       59

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


Make our custom pylint plugin use the AST

pylint plugins are supposed to inspect code without importing it to avoid side effects like importing eventlet.
In some of my manual pylint runs, I ran into a variety of really odd errors and hangs that were ultimately caused by importing the st2 modules from within the pylint plugin.
So, this PR modifies the plugin to use the astroid AST instead of using __import__.
~This is a draft while I compare run times with master branch. I'm curious if this will speed things up at all.~

Fix pylint plugin to rely on ast instead of __import__()
finish pylint api_models plugin so it uses AST only
drop unused import

If this change offers any speedups, the speedups are negligible.
CI master - https://github.com/StackStorm/st2/actions/runs/1140506567
py3.6 - make time 3m 26s
py3.8 - make time 2m 55s
CI unrelated PR - https://github.com/StackStorm/st2/actions/runs/1140762169
py3.6 - make time 3m 14s
py3.8 - make time 3m 35s
CI with pylint AST changes (this PR) - https://github.com/StackStorm/st2/actions/runs/1140785614
py3.6 - make time 3m 10s
py3.8 - make time 3m 0s

Rebased on master. @Kami Is anything left before we can merge this? Could you review (and hopefully approve)?

@cognifloyd How do you run pylint manually? I have not run into issue or have seen the errors you posted above. The changes here looks more complex than before.

Importing code that we are linting is not nice. It breaks the promise of pylint because it is no longer static code analysis, it is reflection.
I have been experimenting running pylint via another program called pants, because it promises to have aggressive caching of results to speed things up. It understands python imports, and so it can track and see that if you change a file b and file a imports file b, then it needs to rerun lint and test steps on both file a and file b, not just the one file you changed. That could really speed up PR CI. Plus, I had one company suggest they would give StackStorm an open source plan to use their remote execution services which would allow us to share cached results between CI and local, so that the tests are that much faster both in CI and locally.
However, because our plugin is using __import__ there is no way to know statically that linting one file will require so many other transient dependencies, not because it uses them, but because of all of the import side effects.

I guess I failed in my goal of justifying this without referencing pants. I guess I'll close this until I'm ready to demonstrate pants.

OK. I'm adding a bunch of comments to this code to explain how it works and what it is doing.


fix(@react-aria/dnd): add support for SSR

Closes
✅ Pull Request Checklist:

[x] Included link to corresponding issue: #3615.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.

📝 Test Instructions:

Directly edited the node_modules with this fix.
🧢 Your Project:
codesandbox

Tests passed now 👍

Hey @Dev-CasperTheGhost if you don't mind updating your branch, we can get it merged in. Thanks!

@reidbarber branch updated :)!


chore: online-store to 0.0.5

chore: Promote online-store to version 0.0.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Update pre commit autoumatic update workflow

Proposed change

Update pre commit autoumatic update workflow:

Add token contents/pull-requests write permissions
Run job for main repository only
Set delete-branch flag to true

Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've added references to all holidays information sources used in this PR
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit command generates no changes
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9230

Relevant Lines:
9230

💛  - Coveralls


Updates machine readable output for odo app

This PR:

Capitalizes the Kind to Application
Uses the full name of Application
Uses ApplicationList instead of List (which is already taken by odo list)

/retest
[odo] Username: Password: Login failed (401 Unauthorized)
[odo] Verify you have provided correct credentials.
[odo]  ✗  challenger chose not to retry the request

Should be fixed by #2164

/hold

Container lease exited with code 1, reason Error

/retest

/un-hold

/hold-cancel

/hold cancel


[Bug]: 'cookie_expiration' and 'cookie_necessary_only_expiration' have no effect

Expected Behavior
In the cookieconsent-init.js file, the options 'cookie_expiration' and 'cookie_necessary_only_expiration' offer the possibility to define custom expiry times of saved cookies. Default time in case of not setting custom time is 182 days.
Current Behavior
The 'cookie_expiration' and 'cookie_necessary_only_expiration' options have no effect and the default expiration time is 7 days, instead of 182 days.
Steps to reproduce
In the cookieconsent-init.js file, enable the 'cookie_expiration' and 'cookie_necessary_only_expiration' options and set the desired expiration times (in days). Clear the cache and save cookies in the consent window, then check the saved cookies for the actual expiration date recorded. In my case, it doesn't matter if I leave the mentioned options commented or if I put any amount of days, the saved cookies always have 7 days of validity.

Proposed fix or additional info.
No response
Version
2.8.9
On which browser do you see the issue?
Firefox, Chrome, Other

Hi @lucasdalpra, I can't reproduce this issue in v2.9.1, both fields work as intended on my end.
Can you provide a live demo that I can inspect (codepen, jsfiddle, stackblitz ...), or at the very least share the full configuration.

Ah, it makes sense if you're using Brave, as it modifies the duration of cookies by default: https://www.cookiestatus.com/.


[JUP]: Expose prometheus collector

Include and expose Prometheus module within the stats module. Functions are not wrapped to take advantage of full prom client functionality

See comments here https://darcs.slack.com/archives/C02PJHDF7QD/p1655415232651579?thread_ts=1655395624.730409&channel=C02PJHDF7QD&message_ts=1655415232.651579

Based on the discussion, we will add the prom-client directly to services otherwise this wrapper would just be duplicate code


Dataflow:  CancelDataflowTransaction method accepts GUID, expects STRING

Wrong data type in method....CancelDataflowTransactionAsync
The method's signature is:
CancelDataflowTransactionAsync(Guid groupId, Guid transactionId, CancellationToken cancellationToken)
However, when querying a Dataflow Transaction, the ID is of type "string" with a DateTime stamp separated by @, a GUID, and an integer separated by a $.
2023-10-03T20:48:31.3705645Z@fbf334ab-55ec-47e1-8ded-705cc5785903$20449390
When reading the Power BI API documentation, the /cancel endpoint takes the string, so the method doesn't accept the right type. TransactionId needs to be changed to a string so the cancel call can be made.
(Even the "WithHttpMessage" version takes a Guid, so it needs to be fixed all the way down the stack.)

Wrong repo :D


Suggestion: Deprecate/remove AnyFunc

A bit of a tangent on the discussion of #89 and #90: I'm not sure how useful AnyFunc is as a whole, nowadays.
I'm skimming through our codebase, and all of the usages of AnyFunc I see are either somewhat wrong, or else looser than they should be - (due to better tuple type support).  The two big categories I'm seeing are:

AnyFunc<T> where () => T would be more appropriate, (or AnyFunc where () => void would be more appropriate)

For example:
function call<T>(func: AnyFunc<T>): T {
    return func()
}

This is a bit dangerous, because:
function loud(x: string) {
    return x.toUpperCase();
}
call(loud); // Compiles, but runtime error

Functions that use AnyFunc that unnecessarily return (...args: any[]) where a more specific set of arguments would be appropriate:

function wrap<T>(func: AnyFunc<T>) {
    return (...args: any[]) => {
         console.log("Called", func.name);
         func(...args);
    };
}

This is better as something like:
function wrap<T, Args extends any[]>(func: (...args: Args) => T) {
    return (...args: Args) => {
         console.log("Called", func.name);
         func(...args);
    };
}

I think the cases where you really don't care what the arguments to a function are is a pretty small niche.
We don't have to remove or deprecate it just because it's niche, but I do think there's a potential source of confusion in having a type that looks more useful than it actually is in practice.

Perhaps AnyFunc could be replaced with some more fine-grained function types?   More things like Predicate<T>
Looking at java.util.function as inspiration,  (When you're looking for nouns, no language has nouns like Java) they've got stuff like:
// Useful
type Supplier<T> = () => T;
type Consumer<T> = (t: T) => void;
type UnaryOperator<T> = (t: T) => T;

// Somewhat less useful
type Func<T, R> = (t: T) => R; // Cound make an arbitrary arity version of this, too
type BiConsumer<T, U> = (t: T, u: U) => void;
type BiPredicate<T, U> = (t: T, u: U) => boolean;
type BiFunc<T, U, R> = (t: T, u: U) => R;

// Everything else seems to be stuff like this, which seems excessive:
type IntConsumer = Consumer<number>

The top three especially seem worth having. (And the Java names are pretty reasonable, IMO, though maybe there are better ones)  Supplier<T> would replace a lot of our current usages of AnyFunc<T>.

As for the other major use-case for AnyFunc I've see, wrapping functions, I'm not sure there's any sort of type that can make that easier.  I tried a type Wrap<F extends Function> = (f: F) => F, but I can't find a way to make that useful without just needing a ugly cast anyway.


[BUG] - HMR: package/preset

Describe the bug
When I was developing, I found that after editing package/preset/src/shortcuts , the newly edited content will not take effect in the browser page.

pnpm run dev
pnpm run example
edit file in package/preset/src/shortcuts
HMR filed

Reproduction
No response
System Info
No response
Used Package Manager
pnpm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a OnuUI issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] The provided reproduction is a minimal reproducible example of the bug.

It seems that the local on my side can trigger the rebuild 😅 .


Need to config log level and schedule control loop frequency in configure file.

When trying to integrate kube-batch with paddle operator, the scheduler log is pretty hard to keep track with, need to config log level and schedule control loop frequency in configure file.

/assign @k82cn

The log level can be changed by "-v"'; we're using glog, so we can not control log by file/struct. Opened PR to handle scheduler period :)


[ImgBot] optimizes images

Hey there. I've just optimized your image. It is the same dimensions and quality as before, only it takes up less space now.
\httpie.png (18.95%))

Coverage remained the same at 93.605% when pulling e759352dad73b2ffc56bc83893bf43827f2bc602 on ImgBotApp:imgbot into dfe6245cd603b26216fd147a3647899148b03973 on jakubroztocil:master.


Dataset Acquision

Hi, thanks for sharing the dataset. I'm not familiar with ROS, can I get access to raw sonar files (e.g. in polar coordinates saved as an image file)?

Hi HikariS97, I am sorry but I will not be able to upload the images right now. I suggest you try a python script like:
https://gist.github.com/wngreene/835cda68ddd9c5416defce876a4d7dd9  from wngreene.
It does require basic ROS install because of libs rosbag, Image, and CvBridge, but it is the easier way to extract the images from the bag file without knowledge of ROS.
When running the script you must specify the image topic as "/son/compressed" to extract the acoustic images on png (zero compress loss).
I hope it can help you,
Cheers


Format/Imports on Save

Either goimports or gofmt should be run every time the file is saved.  Ideally, this is run against the in-memory text prior to writing to file.

Solved with 94aea77e61147aa4b9e155710cc390ce44812689


Thread setting is 2, but only 1 test suite(s) were found. Adjusting configuration accordingly. Weight file not found in path: cypress/parallel-weights.json

Any idea on the below issue?
PS E:\vista\vista> npm run test:parallel

<EMAIL_ADDRESS>test:parallel
cypress-parallel -s 2 --spec "cypress/e2e/vista.cy.js,cypress/e2e/activity.cy.js"

1 test suite(s) found.
Thread setting is 2, but only 1 test suite(s) were found. Adjusting configuration accordingly.
Weight file not found in path: cypress/parallel-weights.json
npm error Missing script: "2"
npm error
npm error To see a list of scripts, run:
npm error   npm run
npm error A complete log of this run can be found in: C:\Users\N A V E E N\AppData\Local\npm-cache_logs\2024-10-15T18_01_59_475Z-debug-0.log
┌─────────────────────────────────────────────────────┬──────────┬──────────┬──────────┬──────────┬──────────┐
│ Spec                                                │ Time     │ Tests    │ Passing  │ Failing  │ Pending  │
├─────────────────────────────────────────────────────┼──────────┼──────────┼──────────┼──────────┼──────────┤
│ Results                                             │ 0s       │ 0        │ 0        │ 0        │ 0        │
└─────────────────────────────────────────────────────┴──────────┴──────────┴──────────┴──────────┴──────────┘
Found test suites does not match results.
Test suites found: 1
Test suite results: 0
Some test suites likely terminated without passing on results, exit with error
The following test suites are missing results: cypress/e2e/vista.cy.js,cypress/e2e/activity.cy.js

@naveen1133
This is not the right repository for issues with cypress-parallel. We deal with cypress-io/github-action issues here.


scene.4ds imports not all objects

Hello. I have an issue. When i'm trying to import scene.4ds file, it gives me this:

How can i fix it?

If you were referring to the city buildings (before the title edit) and other details, these are not part of scene.4DS. Additional items of geometry like most buildings, signs, trees, water, etc. all have their own 4DS models, and scene2.BIN/cache.BIN contain the data that tells the game where to place them. Apologies if I misunderstood and this wasn't what you meant.


add precheck time info

What type of PR is this?
/kind feature

What this PR does / why we need it:
add precheck time info
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

precheck result


twitter does not download videos

The twitter media scraper does not download any videos.

I can confirm. It only scrapes the photos from my account.


Store API (gRPC)

We'll need a daemon that runs a server and a gRPC service that can control multiple stores.
RPC Methods

[x] NewStore
[x] RegisterSchema
[ ] Start
[ ] StartFromAddress
[x] ModelCreate
[ ] ModelHas
[ ] ModelSave
[ ] ModelDelete
[ ] ModelReadTxn
[ ] ModelWriteTxn
[ ] ModelFindByID
[ ] ListenForStateChanges?
[ ] others?

Added some additional methods to the list above


Added a new project Drum Kit Online

Fixes issue #48
IProject Title
Drum Kit Online
Project Description
Free online drum game where you can play drum beats, solos along to your favorite music, by clicking on the screen or with your PC keyboard. The way this website would work that we’ve got a number of keys on the webpage that represents different drums in a typical drum set and when you click on any of those buttons then you’ll get the corresponding sound of the drum. And in addition you can also use the keys on the keyboard to have a sound effect.

Stack:

[x] Html
[x] CSS
[x] JavaScript

@khushi-purwar There was some issue in the previous PR that was causing the merge to have conflicts. I closed that PR and made a fresh one here. This one has no conflicts with the master. Kindly merge it.
Thank You !!


Speed-up facet fetching

We currently have some important issues when we need to gather facet distributions. Indeed fetching the facet distributions involves iterating over a lot of RoaringBitmaps to check that some documents ids are part of them, this involves deserializing and that takes a lot of time.
I propose we associate those bitmaps with xor-filters or bloom filters to speed up those intersections, I am not sure this is the easiest and more performant way to do so, but it's an idea. Those data structures must be freely deserialized (directly read from unaligned raw bytes).

This is no more an issue as we now have implemented level groups.


Options to specify game directory [CR]

I started this branch from my Maven build "fix" but it was rejected so few first commits were reverted when i merged with Master.
Last too commits was due to weird git behaviour. For some reason it thinks that i renamed README.md -> readme.md but the file on filesystem is all uppercase.
I think its not neccessary to describe the change. Just try it yourself and post your feedback.

This is good, thanks! There's a few things that need fixing, but I'll commit those changes myself. It's mostly rearranging a few things and putting some functionality into methods. I've found that looking for cataclysm_tiles.exe doesn't work reliably on Windows either, so I've also set the map editor to look for the tile_config.json file in the gfx folder. Aside from that, this looks great is a nice step forward.


When attempting to install the Catalyze CLI /2.1.1/catalyze_2.1.1_i386.deb onto an Ubuntu 14.10 Utopic machine the Ubuntu Software Center rejects the package installation

When attempting to install the Catalyze CLI /2.1.1/catalyze_2.1.1_i386.deb onto an Ubuntu 14.10 Utopic machine the Ubuntu Software Center rejects the package installation with the following error
"The package doesn't provide a valid Installed-Size control field. See Debian Policy 5.6.20."
Source https://www.debian.org/doc/debian-policy/ch-controlfields.html
5.6.20 Installed-Size
This field appears in the control files of binary packages, and in the Packages files. It gives an estimate of the total amount of disk space required to install the named package. Actual installed size may vary based on block size, file system properties, or actions taken by package maintainer scripts.
The disk space is given as the integer value of the estimated installed size in bytes, divided by 1024 and rounded up.
Screenshot

Hi @ctkdev, this should be fixed with version 3.0.0 and above.

I will install the new CLI and confirm. Thank you!


Count roles API does not work with db2 database

Need to improve count roles API to support all the databases [1]
[1] https://github.com/wso2/carbon-kernel/pull/2350

Tested with DB2 10.5 and 11.5 versions, didn't reproduce the issue, thus closing the issue.


Support lazy loading through DeviceApps.streamInstalledApplications()

Lazy loading through new API streamInstalledApplications
With this PR there's an way to use this package to list all packages individually; how and why was explained on #89.
Take this for instance:
final apps = <Application>[];
final appsStream = DeviceApps.streamInstalledApplications(
  includeAppIcons: true,
  includeSystemApps: true,
);

appsStream.listen(
  (app) {
    apps.add(app);
    setState(() {});
  },
  onDone: () {
    isLoading = false;
    setState(() {});
  },
);

The example above will updated the Flutter UI every time a new package is loaded in the native side, instead of:
// This call takes 5~10 seconds in real devices with large apps data set
// Meanwhile a boring loading spinner should be shown
final apps = await DeviceApps.getInstalledApplications(
  includeAppIcons: true,
  includeSystemApps: true,
);
setState(() {});

getInstalledApplications propose
We can also re-implement the getInstalledApplications API since the Stream<T>s can be converted to Future<T>s (but not vice-versa).
static Future<List<Application>> getInstalledApplications({
  bool includeSystemApps: false,
  bool includeAppIcons: false,
  bool onlyAppsWithLaunchIntent: false,
}) async {
  return streamInstalledApplications().toList(); // Re-use the logic here and convert as Future<T>
}

This PR is a scratch, I would to know what we can change here to improve even more this feature.

Hello, i need this feature to improve my app performance, any updates @g123k @alexrintt  ?

i think this repository is no longer maintained, i am using this feature directly from my fork:
dependencies:
  # ...
  device_apps:
    git:
      url: https://github.com/alexrintt/flutter_plugin_device_apps
      ref: 6051e51fd78778f848eca870ec51a0367b5d2fd9 # use this hash instead 'master' because i can push a new commit in the future maybe, and i don't wanna break anyone else app.

there are no docs for this feature, but you can see how i am using this at:
https://github.com/alexrintt/kanade/blob/f149d373a214d59804b26a50fee703eeb28404ca/lib/stores/device_apps.dart#L187-L214
class Store {
  // ...
  Future<void> loadPackages() async {
    isLoading = true;

    notifyListeners();

    totalPackagesCount = await DeviceApps.getInstalledPackagesCount();

    final appsStream = DeviceApps.streamInstalledApplications(
      includeAppIcons: true,
      includeSystemApps: true,
    );

    appsStream.listen(
      (app) {
        apps.add(app);

        throttle(() {
          if (!isDisposed) {
            notifyListeners();
          }
        });
      },
      onDone: () {
        isLoading = false;
        notifyListeners();
      },
    );
  // ...
}

Thank you, it works !


Add explicit newlines to readme

#46 inadvertently removed some newlines from the README. This adds them back in as <br> tags instead of trailing whitespace.

Coverage remained the same at 100.0% when pulling 027142cf62a3f0a68f4659a612ee782b24c00198 on jeremyruppel:readme into 2c80c43ed6a1257fb25e47918e06c2d9a4d7cd99 on wearefractal:master.


Update Install instructions

From #1

Oops, thanks!
https://github.com/lluchs/evince-typst/commit/72111ca6151af7e9fe0c01f141c041c54c2824eb


Use normal, not scientific encoding for Decimals in Poison

This change shouldn't matter for database saving of JSONs (besides the fact that sometimes the :normal string representation might be a bit bigger in size than the :scientific one) but it makes difference when you want to use Decimal JSON encoding in different contextes.
For example, let's say we have a struct: %{d: Decimal.new("-0.00000001")}
Please tell me which JSON representation is better:
{"d":"-1E-8"}
or:
{"d":"-0.00000001"}
I opt for the second option therefore this PR. There is also an issue with those zeroes staying behind sometimes (for instance Decimal.new("0.00000000")) so maybe we should use some kind of cleanup before parsing? But that would obviously cost performance. What are your thoughts on this?

There is also an issue with those zeroes staying behind sometimes

You can call Decimal.reduce/1 to remove the trailing zeros but keep in mind that you are losing precision information, that may be fine for JSON though.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Fix ios mpp:demo

Make sure xcode doesn't copy irrelevant files

cc: @MatkovIvan Fixed as we discussed.


hideUserInputOnNoneTextInput and mobile problem

Hi,
thanks for the hideUserInputOnNoneTextInput toggle, it is very efficient, especially on mobile.
Still, when you display checkboxes, there is not way to send your answer anymore.
I mean, when you can select 2 in 5 choices, the only way is to press "enter"... something you can't find on mobile ;-)
Any UI design idea to allow to send the answer ?
Thanks,
Philippe

@pvdyck that is a problem :) what do you say @danielfriis ?

That is indeed a problem :) I don't have any great solutions from the top of my head. What about you @pvdyck and @felixnielsen ?

Actually, we are missing a simple button to just send the answer.
Maybe the input box, without the input part would suffice ?
I am really not an UI expert, sorry ...

@pvdyck was thinking the same thing. UI-wise it would look a bit wierd maybe, but let me take a look at it and see if it works.

Also my thinking, but not sure if it would work design wise ... But then again, I think that's the only option. Perhaps have the "send" button float bottom right?

Could these case be applied to other situations ?


Create 0000-01-02-KaminiVaish.md

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request.

created a branch, created a file and made a commit, and opened a pull request.

created a branch, created a file and made a commit, and opened a pull request


Crash when package.json uses wrong type of value for typings, types, or main fields

TypeScript Version:  3.7.2

Search Terms:

charCodeAt
path.charCodeAt is not a function  TSINTERNAL_ERROR
getDirectoryOrExtensionlessFileName
getEncodedRootLength

Code
https://github.com/microsoft/TypeScript/blob/7c14aff09383f3814d7aae1406b5b2707b72b479/src/compiler/moduleSpecifiers.ts#L337
Expected behavior:
To handle cases in which mainFileRelative  resolves to other than a string.
Actual behavior:
Crashes and unexpected bugs under some conditions when mainFileRelative resolves to for example an array. The result is then handled by getEncodedRootLength at:
https://github.com/microsoft/TypeScript/blob/7c14aff09383f3814d7aae1406b5b2707b72b479/src/compiler/path.ts#L117
Which end up calling charCodeAt in something that is not a string:
https://github.com/microsoft/TypeScript/blob/7c14aff09383f3814d7aae1406b5b2707b72b479/src/compiler/path.ts#L119
Failed to compile.

[internal]
TypeScript error in [internal](undefined,undefined):
path.charCodeAt is not a function  TSINTERNAL_ERROR

at: getDirectoryOrExtensionlessFileName
path: /***/***/ts-weird-bug/node_modules/csv-parse/lib/index.d.ts
content of types field: [ "./lib/index.d.ts", "./lib/sync.d.ts" ]

at: getEncodedRootLength

TypeError: path.charCodeAt is not a function
    at getEncodedRootLength (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:5830:27)
    at isRootedDiskPath (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:5742:16)
    at Object.toPath (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:6163:36)
    at getDirectoryOrExtensionlessFileName (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:99655:49)
    at tryGetModuleNameAsNodeModule (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:99631:70)
    at /***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:99451:88
    at Object.mapDefined (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:704:30)
    at Object.getModuleSpecifiers (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:99451:29)
    at getSpecifierForModuleSymbol (/***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:37232:62)
    at /***/***/ts-weird-bug/node_modules/typescript/lib/typescript.js:37098:39

Playground Link:
A repo that reproduces the issue: https://github.com/juanmait/ts-weird-bug. All the details are in the readme. Unfortunately the issue happen due to a combination of factors. A third party library that ships an invalid value for the field types in the package.json, some trickery made by react-scripts and this typescript bug (?
Related Issues:
https://github.com/microsoft/TypeScript/issues/16048

Why in the Backlog? It is a blocker issue. :-/


unroll small linalg operations for performance

As requested (sort of) by @apaszke and @froystig and someone on Twitter, we should try unrolling small linalg operations into HLO on CPU/GPU, like small dense factorizations.
WDYT @hawkinsp ? Please revise this issue statement as you see fit!

Might be related, but working with Jax on fairly dimensional data (classic filtering algorithms), I have noticed poor performance compared to direct C++ CPU implementation in Eigen (and even worse on GPUs). I'll try to provide some better benchmarks to see if it is related :)


port 879 fix on to develop

Have the fix of #879 in develop to include in 4.0.4. We will find a permanant solution by 4.0.5 through #881. And we will remove the temporary solution once we have the permanant fix in place.

Test are running fine.


Required property 'url' is missing

Hello,
I've been banging my head against the wall trying to sort this out. I've followed the instructions as closely as I could. I'm trying to deploy a node application to heroku. I've even tried creating bare bones node projects within my NX repo, but so far no luck. I always end up with this error message when I run the build target:
Required property 'url' is missing
Any direction would be greatly appreciated.

Hey,
Sorry for you!
Could you show me how you configured your Nx project(s)?
Even better if you have a repository that reproduces the issue.

Thanks for the fast reply. I tried two different projects. One was my ongoing express based one. The config I used was:
`
"deploy": {
"executor": "@getlarge/nx-heroku:deploy",
"options": {
    "appNamePrefix": "server-test",
    "buildPacks": [
      "heroku/nodejs",
      "heroku-community/multi-procfile"
    ],
    "variables": { },
    "useForce": true,
    "debug": true
  }
},

`
I also tried it with a generated regular node app, and got the same issue. For now I ended up going a different route so I could get it to deploy but I will try to do it again some time this week. I'd really prefer to use this lib to get it running. I can't share my work repo, but I can perhaps make a stripped down version to further test.

Thanks for sharing the options that confirms what i suspected. Until now I always declared the webhook option with a url property. Now i suppose that when it is missing it leads to the error you encountered. I’ll try to push and release a fix ASAP.

related to https://github.com/nrwl/nx/issues/23153

Awesome @getlarge I'll check that out in the next few days. Thanks for the quick fix!


:Sparkles: Check Cluster.Spec.Pause and Pause Annotation in BMC & BMM reconcile

This PR checks Cluster.Spec.Pause and cluster.x-k8s.io/paused Annotation in BareMetalCluster & BareMetalMachine reconcile.
Tests are added.

/hold

/cc @maelk
/cc @Jaakko-Os
/cc @jan-est
/cc @Xenwar

/test-v1a3-integration

Can you please open this PR against master ? I will remove the v1alpha3 branch as soon as Jaakko's commit gets in metal3-dev-env to change the branch tracked

I don't see any problems with this one.


remove store explicitly.

/cc @qiuyesuifeng
We support gc dead store, but do we need to support removing a store explicitly?

Yes, we should also have this issue in #102

not same,


[MOB-21475] Adding repo pattern

…for propositionsDisplay track event

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thank you @siddique-adobe. Good progress; I have a few high-level comments; I'll leave it to Spoorthi to comment on syntax :)

We don't want to support batching tap interactions yet.
I would support passing optional propositions into trackDisplayInteraction if customers batch/cache the propositions themselves. This way, they won't need to call ..addPropositions first.