Cursor jumps to first target in operator-pending mode for char movements

Discussed in https://github.com/folke/flash.nvim/discussions/186
I initially opened this as a discussion, but I do think this is a bug, unless I'm missing something

Originally posted by TxHawks July 26, 2023
When I use char-mode movements in operator-pending mode, e.g., df<char>, the cursor will jump to the first instance of <char> and will then carry out the operator-pending action - in this case, delete.
So if I have my cursor at the beginning of the following line:
A random line of text

and then hit dte, the cursor will to the n in line, and delete from there to the label I pick or the first one if I just hit enter.
Is this a bug or do I have something misconfigured?
This is my config:
{
  incremental = true,
  jump = {
    -- Disabling this was the first thing I tried. Didn't help
    autojump = false,
  },
  label = {
    current = false,
  },
  modes = {
    char = {
      jump_labels = true,
      label = { exclude = "abcdehijklrxw" },
    },
  },
  search = {
    incremental = true,
  },
},

I am having the same issue. As-is, flash is "breaking" the functionality of ct<character>. I think the fix would be to disable flash's "char mode" when in operator pending mode. I tried the following, but it did not work:
opts = {
    modes = {
        char = {
            config = function(opts)
                if vim.fn.mode(true):find('o') then
                    vim.notify('Started flash char jump from operator pending mode')
                    opts.enabled = false
                    opts.autohide = true
                else
                    vim.notify('Started flash char jump from normal mode')
                end
            end,
        },
    },
},

In normal mode, the notify calls are printing correctly when I press t vs. ct. However, the behavior is the same. Flash takes over and jumps to the next letter after ct rather than doing the default vim behavior of deleting up to that character and entering insert mode.

autohide is what you want, but it only work when jump_labels is false. So, you have to disable jump_labels too in operator-pending mode.
{
	label = {
		after = false,
		before = true,
	},
	modes = {
		char = {
			highlight = { backdrop = false },
			config = function(opts)
				opts.autohide = vim.fn.mode(true):find("no")
				-- Show jump labels only in operator-pending mode,
				-- and not using a count or when recording/executing registers
				opts.jump_labels = not vim.fn.mode(true):find("o")
					and vim.v.count == 0
					and vim.fn.reg_executing() == ""
					and vim.fn.reg_recording() == ""
			end,
		},
	},
}

That's great! I only wish there was a way to avoid momentary blink of the labels after pressing char in operator-pending mode.

@mrbeardad It still doesn't work. I have had jump_labels set to false the whole time. ct<character> still just jumps forward to that character and doesn't allow the regular c operator to function properly.

@nullromo What is your config?

You can see my config here. The relevant part is copied below. jump_labels is false by default.
opts = {
    label = {
        -- only use lowercase letters for jump labels
        uppercase = false,
        -- use <CR> as the first label
        current = false,
        -- position the labels at the beginning of the search words
        after = false,
        before = true,
    },
    highlight = {
        backdrop = false,
    },
    modes = {
        search = {
            enabled = false,
        },
        char = {
            autohide = false,
            search = {
                wrap = true,
            },
            highlight = {
                backdrop = false,
                matches = false,
                groups = {
                    label = "FlashMatch",
                },
            },
            char_actions = function(motion)
                return {
                    [motion:lower()] = 'right',
                    [motion:upper()] = 'left',
                }
            end,
        },
    },
    remote_op = {
        motion = true,
    },
},

opts = {
    label = {
        -- only use lowercase letters for jump labels
        uppercase = false,
        -- use <CR> as the first label
        current = false,
        -- position the labels at the beginning of the search words
        after = false,
        before = true,
    },
    highlight = {
        backdrop = false,
    },
    modes = {
        search = {
            enabled = false,
        },
        char = {
-            autohide = false,
+            autohide = true,
            search = {
                wrap = true,
            },
            highlight = {
                backdrop = false,
                matches = false,
                groups = {
                    label = "FlashMatch",
                },
            },
            char_actions = function(motion)
                return {
                    [motion:lower()] = 'right',
                    [motion:upper()] = 'left',
                }
            end,
        },
    },
    remote_op = {
        motion = true,
    },
},

When autohide is true, the flash functionality of f and t turns off. The point is to be able to press, for example, fcffffff... to cycle through occurrences of the letter c with them all highlighted instead of pressing fcfcfcfcfc with no highlighting.

It doesn't even work anyway. With autohide set to true, I still can't ct<character>. It has the same behavior as setting autohide to false.

Maybe it's something else in my config that's causing it to fail, so I will try to use a minimal setup to check.

I followed the steps below and confirmed that this is reproducible with a miniaml setup.

touch test.lua
Write the following contents to the file:

local lazypath = vim.fn.stdpath('data') .. '/lazy/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    'git',
    'clone',
    '--filter=blob:none',
    'https://github.com/folke/lazy.nvim.git',
    '--branch=stable',
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)
local plugins = {
    {
        'folke/flash.nvim',
        --
        -- rest of the flash config stuff here ...
        --
    }
}
require('lazy').setup(plugins, {})

vim -u test.lua test.lua
ct<character>

This confirms that it's not some other part of my vim config that's causing the problem.

opts = {
  label = {
    -- only use lowercase letters for jump labels
    uppercase = false,
    -- use <CR> as the first label
    current = false,
    -- position the labels at the beginning of the search words
    after = false,
    before = true,
  },
  highlight = {
    backdrop = false,
  },
  modes = {
    search = {
      enabled = false,
    },
    char = {
-     autohide = false,
+     config = function(opts)
+       -- autohide flash when in operator-pending mode
+       opts.autohide = vim.fn.mode(true):find("no")
+     end,
      search = {
        wrap = true,
      },
      highlight = {
        backdrop = false,
        matches = false,
        groups = {
          label = "FlashMatch",
        },
      },
      char_actions = function(motion)
        return {
          [motion:lower()] = "right",
          [motion:upper()] = "left",
        }
      end,
    },
  },
-   remote_op = {
-     motion = true,
-   },
}

For detail on remote_op, refer to the readme

Removing remote_op = { motion = true } made it work. I tried setting motion to true and false back and forth and I couldn't figure out what that setting even does. But with the default motion = false, ct<character> works correctly 🎉

I'm also having trouble with this. Adding this config entry does not fix the problem:
   remote_op = {
     motion = false,
   },

My current config (with character motion disabled):
vim.opt.incsearch = true
vim.opt.hlsearch = false

require('flash').setup({
   search = {
      incremental = true,
   },
   modes = {
      char = {
         -- https://github.com/folke/flash.nvim/issues/204
         enabled = false,
      },
   },
})

function FlashSearch()
   require('flash').jump({ pattern = vim.fn.expand("<cword>"), mode = "search" })
end

vim.api.nvim_set_keymap('n', '#',
   [[:lua FlashSearch()<CR>]],
   { noremap = true, silent = true })

vim.api.nvim_set_keymap('n', '?',
   [[:lua require("flash").treesitter()<CR>]],
   { noremap = true, silent = true })

I cannot currently find a way to go from this config to a config where character motions work. I have tried:
require('flash').setup({
   search = {
      incremental = true,
   },
   modes = {
      char = {
         enabled = true,
         jump_labels = false,
         autohide = true,
      },
   },
   remote_op = {
      motion = false,
   },
})

or:
      char = {
         enabled = true,
         jump_labels = false,
         autohide = true,
      },

or:
      char = {
         enabled = true,
         config = function(opts)
            opts.autohide = vim.fn.mode(true):find("no")
            -- Show jump labels not in operator-pending mode
            opts.jump_labels = not vim.fn.mode(true):find("o")
                and vim.v.count == 0
                and vim.fn.reg_executing() == ""
                and vim.fn.reg_recording() == ""
         end,
      },

or:
      char = {
         jump_labels = false,
         autohide = true,
         highlight = { backdrop = false },
         config = function(opts)
            opts.autohide = vim.fn.mode(true):find("no")
            -- Show jump labels not in operator-pending mode
            opts.jump_labels = not vim.fn.mode(true):find("o")
                and vim.v.count == 0
                and vim.fn.reg_executing() == ""
                and vim.fn.reg_recording() == ""
         end,
      },

@siriobalmelli I started using clever-f again instead of 'char' mode in flash.nvim and that's exactly the behavior I need.
  {
    'rhysd/clever-f.vim',
    event = 'VeryLazy',
    config = function()
      vim.g.clever_f_smart_case = 1
    end,
  },
  {
    "folke/flash.nvim",
    event = "VeryLazy",
    opts = {
      modes = {
        char = {
          enabled = false,
        },
      },
    },

Has not been addressed, so is not stale.

Bump

Bump


Custom path for websocket URL

Hello,
Can you please help me how do I create custom path of WS url's. I have 10 script which I need to run via WebSocket so instead opening 10 port on our firewall I just wants to add single port and launch specific script via custom URL say url:'http://localhost:8080/task-1/', url:'http://localhost:8080/task-2/' and so on

Put them all to same folder (or put small bash scripts that call your scripts in same folder) and use that with websocket as  --dir /path/to/folder   every file inside folder becomes URL ws://host.name.tld/yourfilename (+ .extension if you have one) ... You can use symlinks too.
To get closer to your question, you can have --dir with "task-1" script and "task-2" scripts in it.. then you can use URLs you wrote above... Final slash should not be a problem.

Thank you so much for your response, I will try in a while and update you with the status.


Fixed regex for "-"

/claim #234
Before, it unclustered a "-" followed by letters into different options:
-asd -> -a -s -d
but - was transformed into the void string.
Now it only unclusters when - is followed by something more. Also, -- has always been accepted.

I'll work on that. To solve this case, the whole parsing of options must be changed. It would solve other examples that produce ambiguous output:
import zio.cli._
import zio._

object A extends ZIOAppDefault {
  private val opts = Options.text("abc").map(t => s"abc: $t")
  private val op = Options.text("c").map(t => s"c: $t")
  private val args = Args.text("args").*
  val command = List("--abc", "-c", "-c", "a", "b")

  def cliApp: CliApp[Any with ZIOAppArgs with Scope, Any, Any] =
    CliApp.make(
      "test",
      "0.1.0",
      HelpDoc.Span.text("Reproduce -- issue"),
      command = Command("test", opts.optional ++ op.optional, args)
    ) { (opts, args) =>
      ZIO.debug(s"opts: $opts") *>
      ZIO.debug(s"args: $args")
    }

  def cliApp2: CliApp[Any with ZIOAppArgs with Scope, Any, Any] =
  	CliApp.make(
      "test",
      "0.1.0",
      HelpDoc.Span.text("Reproduce -- issue"),
      command = Command("test", op.optional ++ opts.optional, args)
    ) { (opts, args) =>
      ZIO.debug(s"opts: $opts") *>
      ZIO.debug(s"args: $args")
    }

  val run = cliApp.run(command) *> cliApp2.run(command)
}

produces
opts: (Some(abc: -c),Some(c: a))
args: List(b)
opts: (Some(c: -c),Some(abc: a))
args: List(b)

The output should be the same.

I have refactor the algorithm for validating Options. Now, instead of iterating over Single or KeyValueMap options and finding the first text that fits, it iterates over the arguments
introduced by the user and finds the first option that fits. If there are no options with same name, there would be only one way
to parse the arguments.
This way, if the arguments are -a -b -b c where -a and -b being text option, it always parses
-a    ->    -b
-b    ->    c

Before it changed depending if the Options were a ++ b or b ++ a. If b was before, then it fitted b, so it made
-a    ->    c
-b    ->    -b

I imagine that there might be some similar bugs related to position and ambiguity that should be solved by this.

@pablf Fantastic work! Can you fix conflicts and then we can merge?

@jdegoes The algorithm modifies the behaviour of BuiltInOptions. If there is another parameter, it will give some error. I think this is the appropriate behaviour for the Script, but maybe it isn't for the wizard and the help.
If the command is
val command = Command("command", Options.text("opt"))

command --help would trigger the Help while  command --opt --help or command --opt asdkjh --help would throw an error.
We can:

Make --help and --wizard "protected" and check if they appear anywhere.
--help and --wizard must appear alone so nothing interrupts the parser.

Let me know your thoughts.

I believe there should be a way to represent --help as an option value. i.e. command -a --help with option a equals to string --help

@pablf

Make --help and --wizard "protected" and check if they appear anywhere.

I think this makes a lot of sense as it seems more user-friendly to me. These are very special options and supporting their use even in the presence of other 'junk' could be useful.
Note that --help can appear with subcommands (subcommand help pages) so it's important to preserve parsing command structure even in the presence of --help.
(I am not sure what a user expects if they do not type --wizard alone command --wizard, but the simplest plan to ignore other arguments is probably ok.)

Then, I think a logical behaviour would be:

First, built-in options are matched "cleanly", so if there is some junk term this fails.
If 1 fails, normal options and args are matched.
If 2 fails, then help or wizard are triggered if they appear somewhere, even with junk terms.

If there is no inconvenient, I would make the appearance of 3 and the order between 1 and 2 configurable in CliConfig. @senia-psm In this way, if the command is command <text> we can give value --help to <text> if that would be a reasonable input for that CLI (although it would not be default behaviour). command -a --help would still proccess option a with value --help.

command <text> is not an issue. We can always use command -- --help for args.

Right, thanks! Then we don't switch order of 1 and 2.

@jdegoes Now, we have the behaviour in the last comments. Issue #222 still exists, but its fix shouldn't be very different with or without this PR.

@pablf Sounds great. Can you fix the last conflicts? Thank you!

@jdegoes Ready to merge now!

Thank you! 🙏


Improving Readme.md

Description of  the change
Mistakes found in README.md
Domain of Docs

[x] Readme
[ ] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[ ] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[ ] Solidity , Python3 , Web3.py
[ ] Other , feel free to mention below

Are you contributing under any open-source program ?
IWOC

what mistakes did u encountered with?

These are the few mistakes that can be solved to make it more prettier.

Is this issue is fixed ? @Vikash-8090-Yadav

start working on this @A91y


Adding batch 2 data

The pipeline is slightly hacky (the recipe would fix this) but the profiles are now harmonized to the same pycytominer pipeline as batch 1 data. I add profiles, metadata, and cell counts in this PR.

I have not yet checked if the pipeline processed all plates as described in https://github.com/broadinstitute/lincs-cell-painting/issues/57#issuecomment-757392154

let's do this in three pull requests

Merge batch 2 profiles
Merge #48
Add batch 2 consensus signatures in a separate PR

@shntnu - will you be able to get to #48 this week?


mysql driver expects that you can create an new database on connection

when connecting with the mysql driver, the default behavior is to create a database on the fly.
re: https://github.com/StarRocks/dbt-starrocks/issues/26

alternatives are:

Using the starrocks toolkit container to create the database.
retry statement in your code.


Account for historical/changing gym data

Use an extra column in the gyms sheet to indicate status

Moved to ...
Renamed to ...
Removed

The following gyms are case examples that should be accounted for:

Woodlands Admiral Garden (removed)
Super Tree (moved/renamed to The Wit Ijo of Garden By The Bay)

Only 16 Nov data is valid for it
old coordinates are 103.864352,1.281377


Source of cheatsheet is missing

Followup to #663.
We are missing the source of the cheatsheet PDF.
When adding it, we should also

add a build target to update the PDF
fix the name of the record domain operator field to the correct fieldNames (noted by @lasarojc)

The source is not missing but it is available in a proprietary format: https://github.com/informalsystems/quint/blob/main/doc/quint-cheatsheet.key

How does this get updated?

It gets updated via "Save as..." in the keynote. Not nice, but it was the fastest way of producing a nice cheatsheet I could think of.

Given that we rarely update the cheatsheet, exploring ways for automatically exporting keynote to pdf is not worth our time


Fix of duplicated opt image on FS after uploading, deletion of opt image from FS and DB

Fix for duplicated raw opt image upload
Optical image deletion (Deletion from DB)

@Ren22 next time please make sure a pull request passes the tests


Open IoT SDK port

Open IoT SDK IP review

is it OK to commit symlinks to the repo? does chip have symlinks anywhere already?

is it OK to commit symlinks to the repo? does chip have symlinks anywhere already?
Yes, they love symbolic links :)


Update upsampling operator

The existing Upsampling operator supports only one integer scale.
It would be useful to have this as a list of floats - a different scale per dimension.
This is required with ONNX export/import: https://github.com/onnx/onnx/blob/master/docs/Operators.md#Upsample
Other requirements related to upsampling operator:
Downscaling: https://github.com/apache/incubator-mxnet/issues/10674#issuecomment-384938962
Float scale: https://github.com/apache/incubator-mxnet/issues/9679#issue-293791580

@mxnet-label-bot [Feature request, Operator]
@apeforest @haojin2 @samskalicky @Roshrini

FYI. Changes currently in review on ONNX https://github.com/onnx/onnx/pull/1467


Internal Server Error thrown when base_price < 9

While using instamojo-py to create an offer and base_price was lesser than 9 an internal server error was thrown.
{"message": "Internal Server Error", "success": false}

However if compulsory fields like title and currency are missed out or invalid, proper errors are reported in the response. If base_price is missing the right error is shown; however if base_price < 9 then an internal server error is thrown.
One way to solve this is to add this constraint in the API endpoint documentation.

Closing, as this is no longer valid.


Issue with  installation of kubernetes in centos

This is a Bug Report

Problem: Issue with  installation of kubernetes in centos
Proposed Solution: kubernetes installation
Page to Update:
https://kubernetes.io/...

 1.17.3

yum install docker-ce
Last metadata expiration check: 0:01:41 ago on Tue 25 Feb 2020 02:11:51 PM UTC.
Error:
Problem: package docker-ce-3:19.03.6-3.el7.x86_64 requires containerd.io >= 1.2.2-3, but none of the providers can be installed

cannot install the best candidate for the job
package containerd.io-1.2.10-3.2.el7.x86_64 is excluded
package containerd.io-1.2.2-3.3.el7.x86_64 is excluded
package containerd.io-1.2.2-3.el7.x86_64 is excluded
package containerd.io-1.2.4-3.1.el7.x86_64 is excluded
package containerd.io-1.2.5-3.1.el7.x86_64 is excluded
package containerd.io-1.2.6-3.3.el7.x86_64 is excluded
package containerd.io-1.2.11-3.2.el7.x86_64 is excluded
package containerd.io-1.2.12-3.1.el7.x86_64 is excluded
package containerd.io-1.2.6-3.2.el7.x86_64 is excluded
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

facing error for installing docker-ce let me know if you need more information

Installation issue got fixed

cat < /etc/yum.repos.d/kubernetes.repo
[kubernetes]
name=Kubernetes
baseurl=https://packages.cloud.google.com/yum/repos/kubernetes-el7-x86_64
enabled=1
gpgcheck=1
repo_gpgcheck=1
gpgkey=https://packages.cloud.google.com/yum/doc/yum-key.gpg https://packages.cloud.google.com/yum/doc/rpm-package-key.gpg
EOF
yum-config-manager  --add-repo https://download.docker.com/linux/centos/docker-ce.repo
yum install docker-ce docker-ce-cli
yum install kubeadm -y

kubectl get nodes
NAME                            STATUS     ROLES    AGE   VERSION
ip-10-164-20-104.ec2.internal   NotReady   master   35s   v1.17.3

kubectl get nodes
NAME STATUS ROLES AGE VERSION
ip-10-164-20-104.ec2.internal NotReady master 35s v1.17.3
Node is not getting ready

35s

it might take longer for the node to get ready.

@rajendraprasad9 it looks like you wanted help with Kubernetes. Have I got that right?
If you're looking for Kubernetes help, you can ask using websites such as Stack Overflow.
You can also post your question on the Kubernetes Slack or the Discuss Kubernetes forum.

kubernetes  v1.17.3 version is unstable some issue with node to getting ready after assign a network also

kubernetes v1.17.3 version is unstable some issue with node to getting ready after assign a network also

try moving to a newer release - 1.17.5. i know for sure we had issues with .0 and .1.
if you find issues around that please log a kubernetes/kubernetes ticket.
also for generic support questions the support channels are the right place.
thanks
/close


Follow up action items on xkcd module 🤖

These are meant to be easy issues so newcomers are welcome to contribute!
Please comment below with the issue that you'd like to take up.

[ ] Add caching to API calls
[ ] Use a library for xkcd interaction
[ ] Add tests? (need to explore)
[ ] Add xkcd feature to Telegram bot
[ ] Extract out common logic?
[ ] Add status check
[ ] Logs / Analytics?
[ ] Explore quick access to common queries
[ ] Explore quick access to check out another comic!
[ ] Update README.md showing a demo of $xkcd in action

Join the conversation on Discord: https://discord.gg/MsZ9N9n 👈
We'll assign some tasks to everyone. 😇


migrating examples to react 18

This will need to be rebased. I'd like to see it include all of the examples as well.

This will need to be rebased. I'd like to see it include all of the examples as well.

I will migrating all examples soon.

This will need to be rebased. I'd like to see it include all of the examples as well.

I will migrating all examples soon.
rebased and completed migration.
https://github.com/TanStack/table/pull/4061


Machine pool dropdowns do not have disabled styling

Existing machine pool fields will be disabled on edit. In dark mode, some dropdowns do not appear to be disabled.

Related issue: https://github.com/rancher/dashboard/issues/3553
Master PR: https://github.com/rancher/dashboard/pull/3775
v2.6-head PR: https://github.com/rancher/dashboard/pull/3776

Tested on Rancher v2.6-head version 0784199 and master-head version 2922af5 with the following steps:
Deploy an RKEv2 cluster.
Edit the cluster.
Result:
Machine pool fields have disabled styling on cluster edit. Dark mode:


About analyzing the performance of the Launching process.

It seems that is is not convenient to use Detox Instruments to analyze performance of the App Launching process. Because when the app shows in the "Select an app to profile" window, the app has launched.

That is correct. Live profiling is currently not suitable for launch analysis. To profile launch, use the DTXProfiler API to start a recording somewhere very early (like +load or __attribute__((constructor))) and stop recording somewhere late. You can then pull the recording from the device using the app management tools.

I don't really have good ideas how to do this, as iOS is a very closed system. If you have ideas, please share.

I've implemented app launch profiling. It will be out in the next version of Detox Instruments and profiler. You will have the option to start an app launch profiling from the UI, and then see the results in the UI once the configurable time has passed. Thanks


[Draft] test data gen draft

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
All Promptflow Contribution checklist:

[ ] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Keep the the code in the private branch and do not merge the code to main until we get conclusion.


useAuth is sometimes the latest user data and sometimes the decoded token

Link to reproduction
will add
To Reproduce

Set saveToJWT to a field in the users collection (with auth enabled)
Print the return value of useAuth()
Notice everything is fine
Update the currently logged in user in the auth collection
Print the return value of useAuth()
It's outdated
Request /api/users/refresh-token
Still outdated

Describe the Bug
It's not clear why useAuth() returns the outdated token instead of the most recent user document.
Based on the code below, it was meant to return the user document at some point in the past. setUser and setToken both update the same state.
https://github.com/payloadcms/payload/blob/3677cf688d0e456c42068b4eab0086e64407d938/src/admin/components/utilities/Auth/index.tsx#L109-L113
Payload Version
1.8.4

I've seen this before and come to the same conclusion. useAuth should have the full user object insetead of being reassigned to the decoded JWT user object.

I would suggest something like this:
const user = useUser()
const { user, token, decoded } = useAuth()

Potentially even removing user from useAuth as it has a different lifecycle than auth.

Hey @thgh take a look at #2813 and let me know what you think! As I wrote in the PR, we're considering this a feature request because the current behavior is the intended one, where refreshing the token does exactly what it's named after, it only refreshes the expiration date of the given token.

Reviewed!
If the changes to src/admin/components/utilities/Auth/index.tsx make it in Payload, that would be great! That means users see the UI after a page refresh.
Refreshing the full token is lower prio as that will only resolve the short blink when the token and user object are out of sync. Ideally, the useAuth hook detects mismatches between user object and token and refreshes when needed.


Installation metrics has long timeout and blocks Nautobot startup

Environment

Nautobot version: 1.6.0
Python version: 3.8
Database platform, version: PostgresQL 14
Middleware(s): -

Steps to Reproduce

Start Nautobot with installation metrics enabled (default) in an environment where Nautobot Cloud can't be reached

Expected Behavior
Nautobot starts up as quickly as usual, with either asynchronous communication to Nautobot cloud or an aggressive timeout.
Observed Behavior
Nautobot hangs for at least 5 minutes, I just disabled the setting at that point.

Hey team! Please add your planning poker estimate with Zenhub @glennmatthews @gsnider2195 @HanlinMiao @jathanism @timizuoebideri1

Is the plan to make this asynchronous or just add a timeout?

@gsnider2195 I would think make it asynchronous because it would entirely get around any concerns with timeouts but if it's not possible or would require an entire rewrite, a significantly shorter timeout would be fine.

FYI in certain environments we are also seeing Nautobot fail the post_upgrade because the SSL cert at nautobot.cloud... I suspect in these environments the organization proxies ssl traffic or for whatever reason doesn't trust the Let's Encrypt CAs.  In any case I feel like these errors should be non-blocking and silently ignored, maybe just a warning


grafana-image-renderer: 2.1.0 -> 3.0.0

Motivation for this change
ChangeLog: https://github.com/grafana/grafana-image-renderer/releases/tag/v3.0.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126125 at 3bc85d5c run on x86_64-linux 1

1 package built successfully:

grafana-image-renderer

2 suggestions:

warning: build-tools-in-build-inputs
yarn is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

warning: build-tools-in-build-inputs
rsync is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^


bump x/net module to v0.24.0

What type of PR is this?
cleanup
What this PR does / why we need it?
Bump go x/net to fix security vulnerabilities
Which Jira/Github issue(s) this PR fixes?
https://issues.redhat.com/browse/OSD-22066
Pre-checks (if applicable):

[x] Tested latest changes against a cluster
[x] Ran make generate command locally to validate code changes
[ ] Included documentation changes with PR

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

Project coverage is 48.08%. Comparing base (965301a) to head (0b65552).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   48.08%   48.08%
=======================================
  Files          20       20
  Lines        1512     1512
=======================================
  Hits          727      727
  Misses        745      745
  Partials       40       40

/lgtm

/lgtm

/approved

/approve

/hold

Raised another PR to bump x/net only.
https://github.com/openshift/ocm-agent/pull/102

closing this PR in favour of https://github.com/openshift/ocm-agent/pull/102


Hovering over maximize button appears different when window is focused

Describe the bug
When the window is not focused, hovering over the maximize button looks as expected.
Here is a close-up of all 3 buttons being hovered at the same time (edited together):

When the window is focused however, hovering over the maximize button produces a smaller background area:

To Reproduce
I imagine this behavior is the same for every window.
Expected behavior
No response
Avalonia version
11.1.0-rc2
OS
Windows
Additional context
No response

Not having the best track records when it comes to issues on this repo.
I believe this is not Avalonia's fault. The Windows explorer has this same problem.


encoded content of cell (type: str) to avoid hashlib error

Hi @serge-sans-paille
As described during the pythran tutorial, pull request to fix the cell (type: str) encoding bug.
Best,
Florian

It's a good place to add a test case about this. Can you do that or do you prefer I take care of it?

Hi Serge,
Unfortunately I don't know how to do that. Would you be so kind and add it
yourself?
Thanks.
Best regards
Florian
On Thu, Jul 6, 2017, 20:29 serge-sans-paille<EMAIL_ADDRESS>wrote:

It's a good place to add a test case about this. Can you do that or do you
prefer I take care of it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/serge-sans-paille/pythran/pull/692#issuecomment-313480675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQFXQVi_dJnzQ-MyJgcbyzYQcbnUoEDks5sLSeXgaJpZM4OOJ3d
.

ok, let's wait until 90804e04c349d8c6ac94c0a094c6daa28b7cb95f gets merged, then you can update the .travis.yml file to also test ipython on Python3. keep in touch!

Ok, I'll wait until its merged and will try to insert a unit test following the one you already have under Python 2, but for Python 3, as mentioned in your .travis.yml:
- PYTHON_VERSION=2 TESTCASE=pythran/tests/test_ipython.py
Thanks.

On Fri, Jul 07, 2017 at 12:39:05PM +0000, Florian LB wrote:

Ok, I'll wait until its merged and will try to insert a unit test following the one you already have under Python 2, but for Python 3, as mentioned in your .travis.yml:
- PYTHON_VERSION=2 TESTCASE=pythran/tests/test_ipython.py

Yeah, that's actually just a matter of removing PYTHON_VERSION=2 from
this line :-)

@flothesof you can rebase on master and update the trabvis file!

I just rebased and added the test for travis under Python 3. Let's see how it goes.

Actually, I tried to run local tests with python setup.py test but ran into an error. The trace is:
============================================================ test session starts ============================================================
platform darwin -- Python 3.6.1, pytest-3.1.2, py-1.4.34, pluggy-0.4.0
rootdir: /Users/kappamaki/workspace/pythran, inifile: setup.cfg
plugins: pep8-1.0.6
collected 2498 items / 1 errors

================================================================== ERRORS ===================================================================
_____________________________________________ ERROR collecting pythran/tests/test_exception.py ______________________________________________
pythran/tests/test_exception.py:299: in <module>
    args = exception_args[exception]
E   KeyError: 'BlockingIOError'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
========================================================== 1 error in 7.12 seconds ==========================================================

What should I do?

python setup.py test only works for python2 :-) only a subset of tests are compatible with python2, it's hard coded in the travis files... Travis looks happy with your change, so am I


Implement the rest APIs of DefaultLogger

Summary:
ClientLogger is providing logging strategy to public use. We have a DefaultLogger which is used when client logger is not able to find SLF4J binding.
The DefaultLogger is an impl of SLF4J. There are bunch of APIs are inherited from SLF4J logger but not implemented yet.
Task:
Fill in the implementation of the DefaultLogger APIs.

For API is*Enabled() (e.g. isDebugEnabled()), we can check env variable to determine whether log is able to print.
For APIs related to trace, it is a low priority speaking to others.

Question: Is there already a class DefaultLogger or does it need to be entirely created? If so, will it need to be similar to the implementation of ClientLogger?

@fdmcneill2019 Thank you for your interest in this issue. @sima-zhu has worked through the implementation and there's an open PR for this - #7298
Please feel free to look for other open issues that might interest you.

This will be done in #7298.


Array of nans when running scglue.models.fit_SCGLUE

This is what I'm seeing. I made sure to get rid of all NaNs where I could find them but I think it may be a problem with how I configured the dataset.

I have the same problem, do you solve it?

Hi, you can try to use gradient_clip in the optimization process or adjust the batch size to avoid the problem. This should be caused by numerical problems.


Loosen rails dependencies a bit

See https://github.com/roidrage/lograge/pull/181#commitcomment-18116867

Thanks! This makes more sense.


Upgrade Yarn before installing dependencies, if needed

Fixes issue #1648.

Did yarn change what running yarn check does?

What version of yarn should people being using for dev?

@kevinbarabash Yarn speeds up the install process by skipping file copy if their size and modified time is equal. However, npm publish script removed and locked timestamps down to specific date: https://github.com/npm/npm/commit/114d518c75732c42acbef3acab36ba1d0fd724e2 and https://github.com/npm/npm/commit/58d2aa58d5f9c4db49f57a5f33952b3106778669. It's causing problems with old Yarn caching.

Please hold off merging this for a while.


Bump Spanner emulator version

@bpg130, I think you were having running tests because of an outdated Spanner emulator version here.
See https://pantheon.corp.google.com/storage/browser/cloud-spanner-emulator/releases for more details.

This definitely helped, thanks!


need logdumps from all the devices

fyi @aboynejames i've started to think & work on the interface to our database layer, so i would like to take a good look at some real data captures from any/all devices that we intend to support.
Adding newer devices as they come to market should be a trivial task (so long as they can be uniquely identified).

I have uploaded a short and long log files from the amiigo wearable  https://github.com/Dsensor/exampledata/tree/master/amiigolog
The parser in the sensorLIbrary is build for this data only at present.
Hope to have the angel sensor  this week or next.
Other to get  Fitbit jawbone microsoft band misfit etc. etc.

What is missing is the interpretation of this data into e.g. pulse rate, blood oxygen levels etc. in Dsensor speak, Dmaps - scientific computations that extract meaning from source sensor data.

For the initial release of our 3D visualisation client i expect i can fake this data (in the particle physics update function), at least to get a working prototype which can be refined in a future iteration, as more accurate input data becomes available, thanks for the example logs @aboynejames


pull request for e2e test at 11/17/2016 00:56:05

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


PR cleanup

This PR contains 3 small improvements or code cleanups:

Fall through compile warning: attribute ((fallthrough)) was introduced in GCC 7. To maintain backward compatibility and clear the fall through warning for both Clang and GCC, you can use the /* fall through */
main.cpp - since conf is calloc-ed at the beginning it contains only zeroes, so we don't have to check if it is null or even free null pointer.
while/if read_msg - if first read doesn't read all the available data in the buffers then epoll raises event again, so the control will return to this code until everything is read. If we use while then reading of all file descriptors will be blocked until everything is read which might theoretically be indefinite.

Why did you close it?

Sorry - I made a mistake!

Please update to solve the conflicts

Please reabase do not merge master branch back

Rebased.

@MatejFranceskin still not the proper way. The end result needs to be clean, not with fixup commits. We can't have known broken commits that are later fixed up. Please take a look on interactive rebase and fix it the proper way.

I squashed the last 2 commit. It looks better now.

@lucasdemarchi I rebased it again.

I think it's pretty common to have more than 1 message. Not sure I like the change to process one at time round-tripping to the kernel on the epoll wait.
If this is in fact a problem, maybe we should have a max time we spend processing data for each fd?

@MatejFranceskin this would need at least a rebase. Are you still interested in merging this?

Closing for inactivity. Feel free to open a new one if this is desired.


Update readme to suggest building with yarn

Every now and then there will be issues by people who try to build with NPM. The build works with yarn.

While at it I'm also removing the link to this Slack team, since it's no longer being maintained


[RAC-6209] RACADM based Get BIOS Config implementation with WSMAN API

Background
As a developer of RACADM replacement work squad, based on the preceding investigation work (  RAC-6170 DONE  ), I could start to port RACADM based Get/Set BIOS configuration implementation with WSMAN API.
Details
TODO
(1) Based on the gap analysis spreadsheet created from   RAC-6170 DONE
(2) Finish the feature code to replace RACADM code with WSMAN API
(3) Finish unit test code
(4) Pass code review and get code merged
Reviewers
@anhou @nortonluo @lanchongyizu

BUILD on-tasks #252 : FAILURE

test this please


Support for Hibernate 6

Hibernate has released a CR for their 6th generation. While trying to upgrade (or at least support it in) Spring Data JPA, our tests fail with a NullPointerException coming from JPAUtil.setConstants(…):
[ERROR] Failures:
[ERROR]   QuerydslJpaPredicateExecutorUnitTests.findByFluentPredicateOneValue:369
Expecting actual throwable to be an instance of:
  org.springframework.dao.IncorrectResultSizeDataAccessException
but was:
  java.lang.NullPointerException: Cannot invoke "org.hibernate.query.BindableType.getBindableJavaType()" because "this.anticipatedType" is null
	at org.hibernate.query.spi.AbstractQueryParameter.getParameterType(AbstractQueryParameter.java:61)
	at com.querydsl.jpa.impl.JPAUtil.setConstants(JPAUtil.java:53)
	at com.querydsl.jpa.impl.AbstractJPAQuery.createQuery(AbstractJPAQuery.java:133)
	...(79 remaining lines not displayed - this can be changed with Assertions.setMaxStackTraceElementsDisplayed)
[ERROR]   QuerydslJpaPredicateExecutorUnitTests.findOneWithPredicateThrowsExceptionForNonUniqueResults:329
Expecting actual throwable to be an instance of:
  org.springframework.dao.IncorrectResultSizeDataAccessException
but was:
  java.lang.NullPointerException: Cannot invoke "org.hibernate.query.BindableType.getBindableJavaType()" because "this.anticipatedType" is null
	at org.hibernate.query.spi.AbstractQueryParameter.getParameterType(AbstractQueryParameter.java:61)
	at com.querydsl.jpa.impl.JPAUtil.setConstants(JPAUtil.java:53)
	at com.querydsl.jpa.impl.AbstractJPAQuery.createQuery(AbstractJPAQuery.java:133)
	...(76 remaining lines not displayed - this can be changed with Assertions.setMaxStackTraceElementsDisplayed)

Actually, it looks like the NPE is caused by Hibernate in the first place.

There's a reproducer to be found here (plain Querydsl + Hibernate 6). It looks like the issue only appears when certain types of predicates are involved. contains triggers the issue, eq does not. I still think it's a bug in Hibernate 6 as it works just fine on 5.6.5. That said, I think you need to bring this up with Hibernate as I cannot argue on behalf of Querydsl.

It looks like the issue has been resolved in Hibernate 6.0 CR2.

Just to note, The final version of Hibernate 6.0 is released.
https://in.relation.to/2022/03/31/orm-60-final/


Added Captcha from DreckSoft plus changes in delivery options

Thanks to DreckSoft, Added changes for captchas.
Plus correcting some delivery options.

Not sure if it's your pull request or if ebay kleinanzeigen changed something again but neither the date_updated nor the id gets updated in the JSON. At least date_updated should not depend on the webpage.

Updated with issue #35 + #37


generate patch files if a patch file doesn't already exist

Since we're leaning more into encouraging people to use the patch.py to modify the generated code, we want to . This will also clean up a little bit of our generated code, because we can assume that a patch.py file exists. If a patch.py file already exists, we don't do anything and keep the code the same. However, if there is no patch.py file in the generated output folder, we want to create a patch.py file (for both sync and async). This patch.py file should also have basic instructions on how to write to this patch.py file, so users have an idea when they first start out about what to do

I would suggest by default this file has no executable code, some come documentation about what this file is about, and some example of commented code that you could just uncomment to start with the patch
Example
# This file is a patch file that allows to manually update the generated code behavior. Changes in this file will NOT be overwritten and will be kept from generation to generation
# yadayadayada more yadayadayada
#
# Uncomment this code for a patch_sdk
# def patch_sdk():
#     """Use this code to change something."""
#     import this
#     this.something = True
#
# class BasicOporationCustomization:
#      def something()

Hi @iscai-msft, got it and I will do it asap.


Exposing removeFromWorkingSet() API seems error prone

A few sprints ago we changed removeFromWorkingSet() from a private method to a public one, to support #1197 (see 59e40a9).
It seems easy to misread this as the API to call when you want the same effect as the user clicking the "X" button in the working set.  However, it's actually a low-level model API that doesn't handle prompting for unsaved changes, etc.  (The FILE_CLOSE command would be the right thing to call).
I propose we change removeFromWorkingSet() back to private, and have the DocumentCommandHandlers error case call notifyFileDeleted() instead (which is more semantically correct anyway, and is already a public API).

@peterflynn can we close this? The API has been deprecated and removed with SplitView


jf audit and maven parameters

Is your feature request related to a problem? Please describe.
Currently, the jf cli doesn't have the capability to add maven custom parameters during a jf audit --mvn command.
This if frustrating because for specific projects like talend project, we often pass parameters for maven commands.
Describe the solution you'd like to see
jf audit --mvn has the capability to get additional maven goals.
The mvn.go in the jfrog-cli-core  have already a set of goals and parameters ("-B", "compile", "test-compile"). It should be possible to pass an additional array of strings containing the custom parameters.
Describe alternatives you've considered
Additional context

I agree this is very frustrating, I would like to pass commands like a local m2 repository folder and no transfer progress

It would be nice if the jf command line respected MAVEN_ARGS environment variable


Browser refresh on canvas creation

I am using chartjs with react, and when i create the chart it looks like the browser is refreshing.
The props are ready when the line chart component mounts, which means that there is no need to update the data or call .update(). So the problem is not related to re-creating charts, should have destroyed the chart, etc ...
So this is what happens:
The render function only render the canvas:
<canvas id={this.id} ref="myChart" width="200" height="100"></canvas>
When the component mounts:
componentDidMount() {   this.chart = new Chart(this.refs.myChart, {/* options here */}) }
And now, when the canvas is created it looks like the browser is being refreshed. Obviously, it's not actually being refreshed (because the react state is not reset or anything), but it just looks like it. And, even though that may not be much of a problem if the state is maintained and everything, it looks buggy in the browser.
I am not sure if this i a bug or not, but there is something weird going on with my browser(s) (Chrome, Safari, Firefox) when creating charts. When creating a vanilla canvas like the one below, I don't experience this buggy behavior.
var c = document.getElementById("myCanvas");         var ctx = c.getContext("2d");         ctx.moveTo(0, 0);         ctx.lineTo(200, 100);         ctx.stroke();
And yes, that one is also created in reacts componentDidMount function.
Because I can get this stuff to work with a plain canvas, and it appears to be buggy, I hope this post is accepted here, because it is probably more of a chart.js issue than a React/browser issue.

@stianlp do you have a jsfiddle or other test case that reproduces the issue?

Here
https://jsfiddle.net/stianlp/caumphff/6/
When clicking toggle chart, you'll see that the browser does a quick "refresh"

I added a background color to the button when it is showing the charts. Also added a few more charts, so it hopefully will spend a few more ms creating all chart.
What happens when the browser "refresh" is that it takes a while before the button turns red. This is what I refer to as buggy behavior.

I'm pretty sure the brower loading is caused by the injected <iframe> when the chart is created. Starting version 2.4, you will be able to prevent the iframe creation with responsive: false (but you will need to specify canvas sizes). Later (probably version 2.5), the iframe should be replaced by divs, so this "refresh" artifact should disappear.
Also, don't forget to call chart.destroy() before unmonting/destroying your component: it's required to cleanup your DOM (such as this extra iframe).

Alright, so that's it. It's always good to know what is going on.
I guess can live with the browser loading for now.
Thanks for all answers!

Closing as resolved since v2.4 will take care of this

I'm still seeing this behavior in 2.6.0.  Were the iframes ever replaced with divs, or do we still need to set responsive: false to avoid this?

@SlowburnAZ iframes are replaced in the current master build which will be released as 2.7

@etimberg Any idea when that'll be released?


[WIP] Bulbs video carousel

http://onion.local/special/lawn-and-garden/video/3943
@shawncook @spra85 @kand

Coverage remained the same at 86.06% when pulling 1175a6e22cacc3d15bcd99d7b5583abe487b4e06 on bulbs-video-carousel into 4e2b196565a0697c5c3b2a45bb9c4ab7b00c8c40 on master.


New App: Melbourne Buses

Description
This is a Bus Transit app for Melbourne, Australia.

It retrieves real-time data from the official bus network API in Melbourne (PTV - Public Transport Victoria) and displays it the user.
Specifically the app displays; Bus Stop, Departure Route and Departure Time in minutes
The app uses location to find the nearest bus stop.

Screenshot

I have read the CLA Document and I hereby sign the CLA


Add a Dockercompose extend file to allow for easy code editting in Development

Please submit your pull requests to the 'dev' branch.
When submitting a pull request, please make sure you have completed the following checklist:

[ ] Your code is flake8 compliant (DefectDojo's code isn't currently flake8 compliant, but we're trying to correct that.)
[ *] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Add applicable tests to the unit tests.

I Added a docker-compose.override.yml file that enables contributors easily make changes to code right from their host system and this changes get reflected automatically inside the container.
The Idea around this is just mounting the code directory on host machine directly to docker uwsgi container.

@propersam I ended up changing this around a bit so I opened a new PR and just merged it. This did get us in the right direction. Thx!

happy to be of help in my own little way. :smiley: :+1:


[WIP] PredictionIO

This is "halfway" there

What's missing?

If you don't intend to work on this anymore it might be better to close it.

Looks like prediction.io is dying, their blog and docs site is down. Let's just close this.


Any alternative way to view the data in the local datastore

Since we have to use gcd-rpc as our local emulator and it doesn't support an admin console, is there some other sort of plugin/view/ready code or alternative method to view the data inside of the datastore?

This issue was moved to GoogleCloudPlatform/google-cloud-datastore#116


Include holiday schedule as feature

On Holidays, energy consumption should be low for buildings such as offices but might be higher for churches for example

I'd take this with care.
Some kernels show that holidays are not relevant.
Maybe it is a possibility to combine it with something like is_weekend?

The commit b75bb29 closed the wrong issue.


Footer Color

Describe the bug
Color of the footer in contact us page is same.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.sack.edu.lk'
Click on 'Contact us' button
Scroll down to 'footer'
There is no difference in color of footer and body of contact us page.

Expected behavior
Slightly change the color of the footer a little bit so that it seems to be different.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser : firefox
Version : 93

Are you working on it?
Yes, I am working on it, please assign it to me.

Shall I work on this?

Okay, Please go ahed!

Thanks


Accessing Group Vars for ansible Inventory

Hi Team,
I see a bug or challenge may be to access a group vars/items in nornir script when we use ansible as a inventory plugin.
Below thing works :-
nr.inventory.hosts['<host_name>'].items()
Below one Doesn't works :-
nr.inventory.groups['<group_name>'].items()
Versions of package used :-
netmiko==3.4.0
nornir==3.1.1
nornir-ansible==2020.9.26
nornir-napalm==0.1.2
nornir-scrapli==2021.1.30
nornir-utils==0.1.2
Python 3.9.5
Any help would be appreciated. Thanks

Ref :- https://github.com/nornir-automation/nornir/issues/701

that said, even if this issue was opened there, provided details are not enough to assist you. at the minimum, there needs to be minimal reproduction which demonstrates the problem together with versions of python/nornir/plugin.

If you think there is a bug and want to open a PR I'll review, but without a reproducible example/setup I'm going to close this. Feel free to reopen with a reproducible example and I'll try to take a look at some point.

I have already added a example which works and which doesn't works

Config.yaml file
inventory:
plugin: AnsibleInventory
options:
hostsfile: "inventory/inventory.yaml"
runner:
plugin: threaded
options:
num_workers: 10
defaults.yaml
platform: "ios"
connection_options:
scrapli:
extras:
auth_strict_key: False


[feature] external ember app [#PREP-87]

external ember app & configuration to support local development via simple reverse proxy
h/t @abought

Should this be targeted at the feature/osf-ember branch instead?
Documenting changes required for the companion ember app

Check out this OSF branch
In website/settings/local.py, specify EXTERNAL_EMBER_BASEURL = '/preprints/'
Start your ember server (on port 4200).
Go to `localhost:5000/preprints/' to see the preprints page.

Evaluation checklist

[x] Static assets should be served correctly (the proxy handles image, css, and js requests correctly)
[x] The app responds to the nested URL (eg /preprints/ ) with the top level route for the application.
[ ] Doesn't use any deprecated settings (eg baseURL)

Other notes
Proxying to a static server makes sense for ember preprints, which has a requirement to stay as separate as possible. But unlike serving the contents of a static build directory, this method means the ember server must be running to view those pages, even if they are not being actively developed.
This could be inconvenient for regular OSF development, for example- I'm not sure the proxy solution is what we want to use for everything. For example, the current OSF build process (eg webpack) lets us use prebuilt static assets, which is one less thing to keep running.


IntelliSense missing value when trying to define RunTrigger in Insert/Delete/Modify

When writing calls to Insert/Delete/Modify triggers there is no value TRUE given by IntelliSense. Instead the only suggestion is call to tprocedure snippet:

But it exists in suggestions when calling IntelliSense directly with Ctrl+Space.

Looking into this as part of #510


TINKERPOP-2005 Reject multiple final responses in AbstractEvalOpProcessor

Add isFinalResponse() getter to ResponseStatusCode
Introduce ResponseHandlerContext to allow tracking the final response
status per request message.
Update AbstractOpProcessor, AbstractEvalOpProcessor and related classes
to write response messages through ResponseHandlerContext methods as
opposed to ChannelHandlerContext methods.

https://github.com/apache/tinkerpop/pull/899


Allow dynamic configuration based on annotations/labels (feature request)

In CI/CD cases is common to create several deploys using git user or branch name for demo/testing proposes. To avoid interact directly with voyager, any service can use annotations for basic voyager configuration, so a foo-staging-svc can use:
ingress.appscode.com/ingress-name: foo
ingress.appscode.com/ingress-opts: {mode: http, port: 80, host: staging.foo.com}

A similar approach is used by traefik in docker swarm mode.

ref: https://docs.traefik.io/user-guide/swarm-mode/#deploy-your-apps

I'm not sure about something like this. It seems to open up the potential for privilege escalation, and would allow a user to create an ingress despite only having authorization to create a service.
In your CI/CD pipeline, why could you not create an Ingress resource alongside your Service? The whole point of an Ingress is to decouple applications from interacting with the ingress controller - so I don't understand To avoid interact directly with voyager - perhaps I am just not familiar with how voyager operates 😄

Hey @munnerz , great to see you here!
I agree with your point about privilege escalation. But the issue here is that Voyager creates a separate LB (haproxy pods) for each Ingress object. This can be problematic if you have a lot of services. ref: https://github.com/kubernetes/ingress-nginx/issues/1539
Currently users have to update the Ingress YAML to expose additional services from an Ingress.

Dup #637


Manjula Mishra [DS-Unit-4-Sprint-1-Tree-Ensembles]

Well done on getting your Kaggle competition score all the way to the upper .70s! You are officially the No.1 in our team! 🏆Fantastic work!
T.


Feature/add include vector field in show json

@JackyKoh
Can you be sure to squash the commits for this PR?

@boba-and-beer consistency, include_vector_fields -> include_vector


‼ Crash on iOS 16 for 100% users – signal SIGABRT

#11	0x000000010066a680 in AudioParser.determineIfMoreDataNeedsToBeParsed(index:) at AudioParser.swift:218

stella(637,0x1701eb000) malloc: Heap corruption detected, free list is damaged at 0x281eb6080
*** Incorrect guard value: 0
stella(637,0x1701eb000) malloc: *** set a breakpoint in malloc_error_break to debug
stella(637,0x1701eb000) malloc: Heap corruption detected, free list is damaged at 0x281eb6080
*** Incorrect guard value: 0
(lldb)

How we will solve this ?

How we will solve this I am getting the same error?

+1

@Sethi1998 I decided to switch to another library

@snowtema when is this happening? I'm using this library too, on iOS 16 iPhone 13 Pro, and it's working fine for me

Crash In here
SwiftAudioPlayer
DirectorThreadSafeClosures.swift - Line 49
didUpdate + 49

Crash in here
SwiftAudioPlayer
DirectorThreadSafeClosures.swift - Line 49
didUpdate + 49

same issue + 1

A lot of crashes only on iOS 16 in this library. Looks like Apple broke something in iOS16.

@snowtema Have you solved it? 😂

@Sethi1998 I decided to switch to another library

Which Library ?

@snowtema when is this happening? I'm using this library too, on iOS 16 iPhone 13 Pro, and it's working fine for me

It is crashing in iPhone 11 and 12 pro I tested in both the devices in iOS 16 version

both

I found that it works fine in the simulator，crashes in real devices(IOS16)

both

I found that it works fine in the simulator，crashes in real devices(IOS16)

yes its working in simulator have anyone solved it I got the error that the buffer Is nil when we are calling it in method

@snowtema Have you solved it? 😂

Yes @jiangdi0924, I switched to ModernAVPlayer, which works fine 😂

FYI, @jiangdi0924 @Sethi1998, ModernAVPlayer doesn't have a queue functionality from the box

@snowtema if you want to give this a try
https://github.com/tanhakabir/SwiftAudioPlayer/pull/175

pod 'SwiftAudioPlayer' , :git => 'https://github.com/Kuama-IT/SwiftAudioPlayer.git', :branch => 'fix-ios-16-crash'
solve the problem

pod 'SwiftAudioPlayer' , :git => 'https://github.com/Kuama-IT/SwiftAudioPlayer.git', :branch => 'fix-ios-16-crash'
solve the problem

Thanks working now

@snowtema if you want to give this a try #175   after a few days of testing, I found sometime  audio duration not correctly

Will this ever be fixed? 🙏🏼

This is still borked I take it?

+1


RFP - lychee

Checklist

[x] The package I am requesting does not already exist on https://community.chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://github.com/lycheeverse/lychee
Direct download URL for the software / installer : https://github.com/lycheeverse/lychee/releases/download/v0.10.3/lychee-v0.10.3-windows-x86_64.exe
Software summary / short description:  ⚡ Fast, async, stream-based link checker written in Rust. Finds broken URLs and mail addresses inside Markdown, HTML, reStructuredText, websites and more!

Package added. It is in moderation queue: https://community.chocolatey.org/packages/lychee

The package is approved.


FIX: update ct/otel and etcd

This somehow requires some fun with replace rules (perhaps a better go
mod whisperer can find a more elegant solution).
This would allow us to update opa and get rid of the two vulns on <0.40.0.

closing in favor of #2054


Refactor how transitions are resolved

This is my refactoring I have started few weeks ago. Getting slowly back to it but thought it might be worth pushing this out (even though this is completely broken right now) as it should both motivate me to move this forward and also get initial thoughts on this.
The point of this refactor is to fix a few problems with SCXML-compatibility and also allow implementing new, minor, features from the SCXML spec - I expect this would resolve a bunch of reported issues.

I believe this work has largely been done already in the next branch.


[speculative] Convert the RSpec idiom to a generic shared context.

The idea is to package the generic shared context in chef-provisioning so any improvements will propagate automatically to any driver using it.
I'm iterating on the code using chef-provisioning-vagrant because it's easier: if this is successful, the code will eventually go into chef-provisioning.

Closing in favor of https://github.com/chef/chef-provisioning/pull/408.


Update corpus_analysis & basic nb

In allofplos_basics notebook, fix import statements and update function names to latest versions. Also update samples.corpus_analysis to import correctly from plos_corpus.

I'm trusting the tests and the fact that this looks pretty solid, but I haven't tested any of the functionality changes.


Add CA Certificates Buildpacks to Java Native Image

Should merge after https://github.com/paketo-buildpacks/java-native-image/pull/99 is released.
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.

The PR blocking this documentation update has been merged for a few months now. Are these docs changes still desired/relevant?
Please rebase on main (with the newly reorganized Java docs) if they are :)

Yes, I totally forgot about this. Sorry. I'll rebase and update this.

@fg-j - All set.


test: refactor pipe tests to AVA

Will update when I verify all tests are copied

Ready for review


Request to apply to Quest

Hello,
I really love this VRSL and it works perfect for PC. Now, Audiolink supports Quest, so I hope VRSL can support Quest as well. I have tried with Quest, and it seems that it is working fine when I saw through the VR camera... I don't see the lights through my bare eyes, but I can see the lights through the camera. I believe it is almost their.
Thanks,

Sorry, I didn't realize that there was a discord server. I would like to close this issue and ask for the request at discord.


Moving my packaging repos to Chum

Hello, I am  looking into moving some of the software I merely package and not maintain/write here, with the goals:

to ease collaborative maintainance.
where chum packages are currently in tar+spec format, they will br tar_gitted
(And I want to clean out my github/gitlab/openrepos collections.)

For this I ask for either of these repos be created here.
Feel free to not do it for any of those that you think should not be here - I don't want to cause unnecessary work or clutter.

Package
currently in chum/testing
current packaging
proposed repo name here

AtomicParsley
yes
https://gitlab.com/nephros/openrepos-atomicparsley
atomicparsley

ImageMagick
yes
https://gitlab.com/nephros/harbour-imagemagick
ImageMagick

aria2
yes
as tarball+spec on chum
aria2

detox
yes
as tarball+spec on chum
detox

dstat/dool
yes
https://gitlab.com/nephros/harbour-dstat
dool

hardlink
yes
as tarball+spec on chum
hardlink

inotify-tools
yes
as tarball+spec on chum
inotify-tools

keychain
yes
https://gitlab.com/nephros/openrepos-keychain
keychain

mytraceroute
yes
https://gitlab.com/nephros/openrepos-mtr
mtr

ncdu
yes
as tarball+spec on chum
ncdu

nethogs
yes
https://gitlab.com/nephros/openrepos-nethogs
nethogs

nload
yes
https://gitlab.com/nephros/openrepos-nload
nload

nocache
yes
as tarball+spec on chum
nocache

parallel
yes
as tarball+spec on chum
parallel

smem
no
http://gitlab.com/nephros/openrepos-smem
smem

sysstat
yes
as tarball+spec on chum
sysstat

xtail
yes
as tarball+spec on chum
xtail

lnav
no
as tarball+spec on my OBS
lnav

Notes on relocating the packaging repo from elsewhere:
Some of my packaging repos have quite a messy git history. As it is not important for most things, I plan to not clone but start afresh here. Unless there is a clever way to clone one branch into a fresh git repo? My obs-specific branches tend to be clean enough.

That is a decent number of repos. I don't see any reason for not adding them here. I will look into how I can create them via cmd line

I you find out let me know ;) I haven't managed to do that using the official gitlab cli tool so far.

Looks like it is possible. Requested in https://github.com/cli/cli/issues/2807. Solution seems to mix in a script gh with its extension https://github.com/mislav/gh-repo-collab
Let's see if it works ...

Seems to be simple enough:
gh repo-collab list sailfishos-chum/atomicparsley
gh repo-collab add sailfishos-chum/atomicparsley nephros --permission admin

Now just will make short script and make all the repos

... and done! Please accept the invitations for joining as admin. Good luck with moving the projects!

Nice! Thanks for everything!


key of rowMap

I would like to use a different key to rowMap, today is "key", in my case I using id, had any props to set a key name to get rowMap?

I think you are looking for a keyExtractor. If you pass this prop to the SwipeListView you don't need to worry about having an explicit key for each object in your data:
https://reactnative.dev/docs/flatlist#keyextractor

Like @jemise111 said, just use keyExtractor if you would like, See how i used in my app:
<SwipeListView
            useFlatList={true}
            useNativeDriver={false}
            data={data}
            renderItem={({item, index}) =>
                <View
                    <ButtonList
                        onPress={onclick()} name={item.name} time={item.key} foods={item.foods} />
                </View>
            }
            keyExtractor={(item, index) => item.key}
            disableLeftSwipe
            leftOpenValue={Dimensions.get('window').width}
            onSwipeValueChange={onSwipeValueChange}
            renderHiddenItem={ (item, index) => (
                <View style={s.rowBack}>
                    <EvilIcons name="trash" size={30} color={Config.primaryColor.color}  />
                    <Text style={{color:'white'}}>Deletado!</Text>
                </View>
            )}
        />

My data looks something like:
caloriesFood:[
        { foods: [id: "0", {foodName:"Maça", kcal:52, icon:"apple", count:3, grams:100}] },
       { foods: [id: "1", {foodName:"Bacon", icon:"bacon", kcal:541, unit:"unit", grams:"100"}] }
    ]

Use this:
<SwipeListView>
    keyExtractor={(item, index) => item.id}
    another props...
</SwipeListView>

id = unique key of ur object
My data looks something like:
rows:[
    {id: 'abc',userName:'test'},
    {id: 'abd',userName:'test1'},
]

The above solutions should definitely be working, if not please re-open, thanks!

Let's see if this reopens ... I've got an interesting case where keyExtractor information does not seem to reach the rowMap...

No, not reopened.  I'll post a new issue and reference this one


:rotating_light: ACTION REQUIRED: Confirm Repository Status for Migration by Dec 20 :rotating_light:

Hi BCDevOps users,
The Developer Experience (DevX) team is moving code from the current BCDevOps GitHub organization to the new bcgov GitHub Enterprise organization, which is protected by single sign-on (SSO). To prepare for this migration, we need you to confirm the status of your repositories.
Here’s what you need to do:
Respond to this issue by December 20, 2024, to confirm if each of your repositories is:

Active and ready for migration to bcgov, OR
No longer needed and can be archived.

:bell: ACTION REQUIRED: Leave a comment on this issue with your repository status.
If we don’t hear back by December 20, we’ll proceed with archiving your repositories.
Questions? Email<EMAIL_ADDRESS>for help.
Thank you for your cooperation!
The Developer Experience Team, Digital Office

Hi BCDevOps users,
This is a reminder to confirm the status of your repositories by December 20, 2024, as part of the migration to the bcgov GitHub Enterprise organization.
Please leave a comment on this issue to let us know if your repositories are:

Active and ready for migration, OR
No longer needed and can be archived.

If we don’t receive a response by the deadline, we’ll proceed with archiving your repositories.
Need help? Email<EMAIL_ADDRESS>Thank you for your attention!
The Developer Experience Team, Digital Office

Hi BCDevOps users,
:rotating_light: Final call! This is your last chance to confirm the status of your repositories as part of the migration to the bcgov GitHub Enterprise organization.
Leave a comment on this issue by December 20, 2024, to confirm whether your repositories are:

Active and ready for migration, OR
No longer needed and can be archived.

If we don’t hear from you by the deadline, we’ll proceed with archiving your repositories.
Questions? Email<EMAIL_ADDRESS>Thank you for your prompt response!
The Developer Experience Team, Digital Office


Feat: Implement the functions in the increase_order_utils library.

Description
Implement the process_order function in the increase_order_utils library.
File: src/order/increase_order_utils.cairo.
Function signature:
fn process_order(params: ExecuteOrderParams);

Implement the validate_oracle_block_numbers function in the increase_order_utils library.
File: src/order/increase_order_utils.cairo.
Function signature:
fn validate_oracle_block_numbers(
    min_oracle_block_numbers: Array<u128>,
    max_oracle_block_numbers: Array<u128>,
    order_type: Order,
    order_updated_at_block: u128);

Acceptance criteria

Function is implemented and well commented.
Associated unit tests (nominal and failure) are implemented and passing.

Resources

processOrder function in GMX repo
validateOracleBlockNumbers function in GMX repo

Hello, may I take this?

Sure ! Assigned

following issue required for this issue:

#269
#331
#296
SwapUtils

following functions that require to finish other issues  :

processOrder function in GMX repo
validateOracleBlockNumbers function in GMX repo

Hello, may I take this?

Hello, can you please pull from main please cause the signatures just got implemented

Hello, can you please pull from main please cause the signatures just got implemented

Gotcha will do 🤙

Hi @ftupas ! Do you have any news on your issue ?

Hey there, sorry haven't given an update. I managed to implement validate_oracle_block_numbers but still working on process_order. Will try to create the pr today or tomorrow


Colors are wrong

Hi,
When I choose colors, the result is not good, all displaying colors cells are more dark, than the original color chosen.
For example a yellow this one, FCB700, become totally orange in cells.

Hello, hex that you attached really darker. On right - is system yellow. Can you explain more? Did you set .systemYellow and see orange color? or which color did you set?


XS✔ ◾ Fix devworkspace operator namespace

❌ Add a description.

@hussnain612 Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-151-129f8f5f


Add type tests, update docs

Mostly independent from #1, but copies the type changes over and adds extra documentation.
Not sure on the return type documentation in the readme.

Thanks :)


Provide Linux-style ERR_PTR/PTR_ERR/IS_ERR macros

Linux has macros which enable functions returning pointers to indicate specific error codes in their return value rather than just returning NULL. It'd be nice to have similar functionality for Zephyr.

It might be hard to do this since we do not virtualize memory, the Linux implementation assumes the range ((uint32_t)-MAX_ERRNO)-0xFFFFFFFF is never a valid memory address it seems.
There might be other approaches, such as stating that any returned pointer that isn't aligned to 4 bytes is a flag indicating that the real error value should be fetched from some other variable (thinking out loud here)

Can errno be used instead?  This is an idiom that I've seen used in the wild: return errno = ESOMETHING, NULL;.

Can errno be used instead? This is an idiom that I've seen used in the wild: return errno = ESOMETHING, NULL;.

In some cases it could be an option. However it's not multi-thread safe, so it can't always be used. In general I'm not sure we should encourage errno usage because of its vulnerability to multi-threaded (or thread + ISR) race conditions.

I'm not a fan of errno either unless we can migrate it to thread-specific storage.
Memory protection makes dealing with it a lot more complicated.

Yeah. That makes sense -- also, we don't seem to use errno that much in Zephyr anyway; we mostly use the error codes.
The idea to use unaligned pointers to signal errors has potential.  There's no need to have another errno-like variable to store the error, as it can be encoded in the pointer itself if it's not aligned. With a 4-byte alignment, only 2 bits dictate the alignment, so we have 30 bits for error codes;  that's more than plenty enough to encode errno codes.
In a way, this isn't much different than a function returning a negative errno on failure and 0 on success (sign bit being set to signal error, etc), with the exception that masking bits is more expensive than just checking for sign.
Just have to be careful to not use this for pointer types that do not have 4-byte alignments (char *, short *, etc).  In general, for returning a struct, this should be fine (unless it's packed and its sizeof() is smaller than 4); maybe there's a way to static_assert() that these macros will only be used with appropriately-sized types?

I'll close this, since after the discussion in #22684 we decided not to pursue these Linux-style APIs.


canOpenURL: failed for URL

On iOS platform :ios, '12.0' it fails to open mail app with the below logs:
2022-10-25 21:29:39.714746+0000 Runner[21078:1264847] flutter: The Dart VM service is listening on http://<IP_ADDRESS>:64543/0RdSJNgK4r8=/ 2022-10-25 21:31:01.858828+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "x-dispatch://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.859949+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "readdle-spark://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.860793+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "airmail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.861616+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "ms-outlook://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.862326+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "ymail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.862942+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "fastmail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.863467+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "superhuman://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)" 2022-10-25 21:31:01.863888+0000 Runner[21078:1264517] -canOpenURL: failed for URL: "protonmail://" - error: "This app is not allowed to query for scheme protonmail"
I have already added the URL schemes in my app's Info.plist file
<key>LSApplicationQueriesSchemes</key> <array>     <string>googlegmail</string>     <string>x-dispatch</string>     <string>readdle-spark</string>     <string>airmail</string>     <string>ms-outlook</string>     <string>ymail</string>     <string>fastmail</string>     <string>superhuman</string> </array>
Also, the documentation stated that They need to be added to both your app's Info.plist and in the source of this library. Is the later part responsible for this failure, if yes then why the example app doesn't have the url schemes added to the source of this library.
Please, I would appreciate your help on this, thanks

I am also getting the same exact error
[+3311 ms] -canOpenURL: failed for URL: "x-dispatch://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[        ] -canOpenURL: failed for URL: "readdle-spark://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[   +2 ms] -canOpenURL: failed for URL: "airmail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[   +1 ms] -canOpenURL: failed for URL: "ms-outlook://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[   +2 ms] -canOpenURL: failed for URL: "fastmail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[   +1 ms] -canOpenURL: failed for URL: "superhuman://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[   +4 ms] -canOpenURL: failed for URL: "protonmail://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"

Hi
I am facing same issue with iOS with device version 16!
Has anyone have any solution for same?

@esentis @rxPrachi @king-kloy This is not the error, this is just log that indicates that email app can't be opened because it is not installed on device. So, it's fine.

@eli1stark yeah I realised that's the case, cheers


Queries on an Accumulo Store may skip over data that has the same vertex/source/dest and timestamp

There is a 'limitation' in Accumulo where you can't have 'many' rows with the same Key. When you scan over the rows, the scanner reads the data in batches. After the first batch Accumulo tries to work out where to restart the scan and will skip over rows that have a key equal to one already read. To fix this we will set the timestamp to a random number for non aggregated groups. Aggregated elements work already as the elements with the same key are aggregated together, so there are never duplicate rows with the same key.

The AccumuloStoreConstants.DEFAULT_TIMESTAMP constant has been deleted. If you have not configured the timestamp value then it will now be set to a random number (previously it was set to 1).
This will mean if you are using AddElementsFromHdfs and not setting a timestamp then the Reducer will not reduce anything. A separate issue will fix this later.

Merged into develop.

Merged into develop.


Add a ShedLock provider for GCP Spanner

Is your feature request related to a problem? Please describe.
My team is planning to migrate our Spring based services, which make use of ShedLock, to use Spanner databases using the spring-cloud-gcp-data-spanner module provided by Google. Platform choices mean that we are unable to use JDBC and are therefore unable to use the JDBC template provider, which appears to be the only provider that is capable of supporting Spanner at this time.
Describe the solution you'd like
We'd like to use a provider that is compatible with and dependent upon the Spring Cloud GCP Data Spanner module without the need to add any JDBC dependencies and configuration.
Describe alternatives you've considered
We've considered using only the JDBC template as a database connection but this is not the preferred approach of the client and prevents us from using Spanner to maximum effect. We've also looked at using JDBC just for ShedLock and a different  Spring Cloud GCP Data Spanner connection for everything else, but this has its own performance impact.
Additional context
The spring-cloud-gcp-data-spanner module is provided by Google as part of the spring-cloud-gcp project: https://github.com/spring-cloud/spring-cloud-gcp

Hi, thanks for feedback. Unfortunattely I do not have capacity to implement it, but I accept pull-requests 😈  It's quite easy to impement a LockProvider, just mimic some of the existing providers that is similar to the Data Spanner.

Hi Lukas, thanks for the rapid feedback. We will look into the possibility of doing this and get back to you.

We've completed an implementation but it hasn't been fully tested against spanner yet.

Hi @lukas-krecan I've created a PR for a Spanner implementation.
Feel free to take a look.

To use spanner with ShedLock.

Define your shedlock table structure as you desire

<column name="processed_date" type="timestamp"/>             <column name="lock_until" type="timestamp"/>             <column name="locked_at" type="timestamp"/>             <column name="locked_by" type="VARCHAR(255)"/>             <column name="name" type="VARCHAR(255)">                 <constraints nullable="false" primaryKey="true"/>             </column>

Create your Spanner Datasource config

`@Configuration
@EnableTransactionManagement
@EnableJpaRepositories(basePackages = { "your_packages_here"       },
entityManagerFactoryRef = "spannerEntityManagerFactory",
transactionManagerRef = "spannerPlatformTransactionManager")
public class SpannerDataSourceConfig {
@Value("${spring.datasource.url}")
private String datasourceUrl;
//... other variable
@Value("${spring.liquibase.change-log}")
private String liquidBaseChangeLog;

@Bean("spannerDataSource")
public DataSource spannerDatasource() {
    var dataSource = new JdbcDataSource();
    dataSource.setUrl(datasourceUrl);
    return dataSource;
}
//... other beans

@Bean(name = "spannerLiquibase")
public SpringLiquibase spannerLiquibase(){
    var liquidBase = new SpringLiquibase();
    liquidBase.setDataSource(spannerDatasource());
    liquidBase.setChangeLog(liquidBaseChangeLog);
    return liquidBase;
}`

Define your ShedLockConfig

`@RequiredArgsConstructor

public class ShedLockConfig {
private final SpannerDataSourceConfig spannerDataSourceConfig;

@Bean
public LockProvider lockProvider() {
    return new JdbcTemplateLockProvider(
            JdbcTemplateLockProvider.Configuration.builder()
                    .withTableName("your_shed_lock_table_name")
                    .withJdbcTemplate(new JdbcTemplate(spannerDataSourceConfig.spannerDatasource()))
                     //DO NOT INCLUDE THE dbWithTime since it does not work with spanner timestamp
                    .build()
    );
}

}`


Optimise calls to noise

Noise for terrain height only needs to be called once for each x, z position rather then ran for each individual block

Included In: https://github.com/FrederoxDev/Cosmos-Genesis/commit/d6018b875e56480df11eee11bc28d17bf20bac84


p2p connection optimizations [Tracking issue]

[ ] #2951
[ ] #2954
[ ] #2956
[ ] #2958
[ ] #2988

Not part of, but related: https://github.com/cometbft/cometbft/issues/2955

Not part, but related: #2685


Return groups that match search results

Search for a group based on different queries, then return the results.

You can now search for groups by posting to /groupsearch


[CIT-551] Make the job pass (temporarily) on job level as well

Proposed changes
CIT-551 Make the job pass (temporarily) on job level as well
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactorings that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://cumulocity.atlassian.net/browse/CIT-551
Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Codecov Report

Merging #408 (e35744f) into main (13fc9df) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
- Coverage   64.06%   63.98%   -0.08%
==========================================
  Files         106      106
  Lines        6809     6820      +11
==========================================
+ Hits         4362     4364       +2
- Misses       2447     2456       +9

Impacted Files
Coverage Δ

tedge/src/cli/connect/command.rs
0.00% <0.00%> (ø)

sm/plugin_sm/src/plugin_manager.rs
53.01% <0.00%> (+2.40%)
:arrow_up:

Continue to review full report at Codecov.

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


Bug Report - Enable DDoS Std policy does not work with managed VNets

Describe the bug
Certain PaaS services - we tested Databricks and AKS - manage underlying network in a specific way. In my customer environment, we have the following policy enabled (comes as part of Contoso reference implementation):

Policy name: Virtual networks should be protected by Azure DDoS Protection Standard
Policy definition ID: /providers/Microsoft.Authorization/policyDefinitions/94de2ad3-e0c1-4caf-ad78-5d47bbc83d3d

This is a 'Modify' policy, but it doesn't work for PaaS services mentioned above. Their VNets are then listed as non-compliant and there seems to be no way to fix it.
Steps to reproduce

Deploy Azure Databricks service in your subscription, where this policy has been assigned (in the RI, it is assigned on {prefix}-landingzones MG).
Check the compliance in the Azure Policy - Compliance blade.

Screenshots

Thanks for raising awareness of this @pazdedav.
Do you get any error messages when trying to remediate the policy or enable DDoS on the VNET manually? (share any correlation IDs 👍)

Any updates @pazdedav?
We were able to test on a databricks VNET and it worked

I got write access to the environment, so I could reproduce the error and provide more input / info, @jtracey93 . I will update this issue with more details 👍.

Hello @jtracey93
I was able to reproduce an error (about "deny assignment") when trying to remediate that non-compliant VNet:

It is important to note, we are testing a scenario with Databricks-provided VNet and not "Bring Your Own VNet", where I assume we have more control.

Thanks @pazdedav, this actually looks like its because its a locked VNET rather than a policy issue. Have you raised or the customer raised a support ticket to ask if possible? Could be a good idea

No worries and thanks for yours to @pazdedav,
I think for now whilst we work on if this could be supported on managed VNETs with engineering we would advise to create an exemption for the resources or scopes, as required so the compliance report is reflected correctly.

Thanks for the guidance @jtracey93. Much appreciated.

Trigger ADO Sync 1

Trigger ADO Sync 2


The auto-translated subtitles of many videos can‘t be downloaded

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=7_Wzo_LE_ws', u'-f', u'bestvideo+bestaudio/best[ext=mp4]/best', u'--no-check-certificate', u'-o', u'%(title)s.%(ext)s', u'--write-sub', u'--write-auto-sub', u'--sub-format', u'best', u'--sub-lang', u'zh-Hans,zh-TW,en', u'--embed-subs', u'--embed-thumbnail', u'--merge-output-format', u'mp4']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 2.7.18 (CPython) - Linux-5.4.0-88-generic-x86_64-with-Ubuntu-20.04-focal
[debug] exe versions: ffmpeg 4.4, ffprobe 4.4
[debug] Proxy map: {}
[youtube] 7_Wzo_LE_ws: Downloading webpage
WARNING: zh-Hans subtitles not available for 7_Wzo_LE_ws
WARNING: zh-TW subtitles not available for 7_Wzo_LE_ws
[info] Writing video subtitles to: Landscape Building (Organic Formed) Archicad Tutorial.en.vtt
[youtube] 7_Wzo_LE_ws: Downloading thumbnail ...
[youtube] 7_Wzo_LE_ws: Writing thumbnail to: Landscape Building (Organic Formed) Archicad Tutorial.jpg
[debug] Invoking downloader on u'https://r1---sn-a5mekn6d.googlevideo.com/videoplayback?expire=1634063346&ei=kn9lYaTeM-2PsfIP9sWIqAM&ip=<IP_ADDRESS>&id=o-AAS8dl0EoUIw6ndkN-n-j5643WIpfjGgOxllsnmIk7Ah&itag=137&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=Xp&mm=31%2C29&mn=sn-a5mekn6d%2Csn-a5mlrn7r&ms=au%2Crdu&mv=u&mvi=1&pl=22&vprv=1&mime=video%2Fmp4&ns=MpGMRqTxrJ4pfA6io4K4wWoG&gir=yes&clen=145751804&dur=1132.130&lmt=1633995860859946&mt=1634041330&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=WEB&txp=5316224&n=hxH5guA_qAXfh5iPqZJ&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIgD52JFES43nVBLbthEujHjufF3oGHZp_oz5PW6Hz25RQCIQC6oV3disJhLbTgq-Hau7AnrRKCdtGmO5gOGDnJEcZ7JA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl&lsig=AG3C_xAwRAIgT8URPUOlDy0hYreMn9-NpUFa0PwJhcBKoyiX-dZiBmECIFQpuP_pVV6yOiQXe_D0tL7ko7FaYSkTPnJje95RsQ5U'
[download] Destination: Landscape Building (Organic Formed) Archicad Tutorial.f137.mp4

Description

I found that the auto-translated subtitles of many videos can‘t be downloaded
For example, the following video
https://www.youtube.com/watch?v=7_Wzo_LE_ws
I tried to download the automatically translated Simplified Chinese subtitles, but yt-dlp intelligently recognizes the subtitles native to the video.
This problem exists on many videos, especially if the video has subtitles uploaded by the author.

See also https://github.com/ytdl-org/youtube-dl/issues/30667#issuecomment-1048703724.


add component "PropertyFieldSitePicker"

Q
A

Bug fix?
[ ]

New feature?
[ x ]

New sample?
[ ]

What's in this Pull Request?
A new control named "PropertyFieldSitePicker". This control allows the user to select one or multiple sites from the SharePoint Search API. Similar to the SitePicker in the official "News"-WebPart.

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

Hi @nsoeth,
Thank you for the great addition to the library!
Could you please also update documentation to add details about the new control?
The documentation is located in the same repo in docs\documentation\docs folder.

Hi, I will do it. Might take some time though since I hurt my wrist yesterday. But I will do it asap.

Hi @AJIXuMuK ,
Managed to write the docs (once mkdocs is set up, it is quite fun writing docs) and also fixed a minor "inconvenience" in the process. Hope this is ok.

Thank you @nsoeth for the update!


typo: Double word "then"

@stevewhims  : Sorting out permissions to merge these types of changes myself. But in the meantime, PR passed checklist review. Please review for technical accuracy, etc., and approve/merge if appropriate.


the console command api:routes doesn't work anymore.

As of 0.10.*, it complaints the isProtected() method is missing.

Good pickup, cheers. Will need to tweak the command slightly to account for
the recent changes.
On Tue, 21 Jul 2015 11:56 Chuck JS. Oh<EMAIL_ADDRESS>wrote:

As of 0.10.*, it complaints the isProtected() method is missing.
—
Reply to this email directly or view it on GitHub
https://github.com/dingo/api/issues/530.

I have created a pull request to close this issue by adding the method Api\Routing\Route::isProtected()

This method determine if the route is protected.

I have made a pull request on #538

538 was merged however I I'll be amending it once I'm home so that the
protected column is still included.
On Sat, 25 Jul 2015 01:29 Robert Kummer<EMAIL_ADDRESS>wrote:

I have made a pull request on #538 https://github.com/dingo/api/pull/538
—
Reply to this email directly or view it on GitHub
https://github.com/dingo/api/issues/530#issuecomment-124559125.

Have made some amendments to the command. Protection is now displayed and you can filter routes based on their protection, e.g., php artisan api:routes --protected will only show protected routes.


Add PR templates

I suggest using the experimental GitHub Discussions for questions and suggestions, leaving the issues solely to organize and document the development process.
Thus, instead of using issue templates, we could link the according discussions forums directly as depicted below.

It might still make sense to add a bug report template. What do you think?

Sounds good

Comments are displaced and belong to #7 instead.


SvgVisualElement.Bounds always returning "0"

Description
Trying to get the Bounds for a "SvgTextBase" in v2.4.1 and before would return valid data. Starting with v2.4.2 (and up to and including v3.0.102), Bounds always seems to return "0" for everything. (Height, Width, X, and Y.)
If this is expected, that's fine, but would be curious then how to get sizes of Elements before rendering.
Example data
internal static RectangleF CompileObject(SvgVisualElement obj) {
		SvgDocument doc = new SvgDocument();
		doc.ID = "test";
		doc.Overflow = SvgOverflow.Visible;
		doc.Ppi = Vars.SIZE_DPI;
		doc.ViewBox = new SvgViewBox(0, 0, 5, 5);
		doc.Width = new SvgUnit(SvgUnitType.User, 5);
		doc.Height = new SvgUnit(SvgUnitType.User, 5);
		doc.Children.Add(obj);

		//And get the bounds.
		RectangleF retValue = obj.Bounds;

		return retValue;
}

In v2.4.1, The above code, when SvgVisualElement being a SvgText object, will return proper H and W, X and Y values. (Latter is unimportant here, as it's relative to the temporary document breated above.) The object needs to be added to the Document before these values are populated.
in v2.4.2 (and above, including v3.0.102), the same above code will always return Bounds of Height=0, and Width=0.
Used Versions
v2.4.1, v2.4.2+, .NET 4.8.

@foxmichaels
As far as I checked, I can get Width and Height values other than 0.
Please tell me test code that can be reproduced.

Sorry for the delay. Through together a solution that reproduces the issue. If I did something wrong (entirely possible), let me know. It threw me because of different results between v2.4.1 and v2.4.2+.
ClassLibrary1.zip

This difference is due to #414.
(I think you have too many changes to understand...)
If you make the following changes, you will get expected behavior.

before

tspan.Nodes.Add(new SvgContentNode { Content = "LINE TEXT" });

after

tspan.Text = "LINE TEXT";

Okay - yup. Getting rid of the SvgContentNode usage fixed that. Fantastic to know. I started this project back on v2.0, and never needed to touch it until recently. A LOT of thing changes since then.

(I think you have too many changes to understand...)

If you see anyhting else that looks wrong please let me know. Otherwise, good - it works as expected.