segmentation fault (core dumped) error

Hello,
Thanks for the amazing work, I've been trying to run your code with Python 2.7 and TensorFlow 1.5.
However, I have a segmentation fault (core dumped) error that stops the process.
When using docker it only displays "Killed"
There is a similar issue here : https://github.com/tensorflow/tensorflow/issues/2034
But it does not seem to be related to this issue.
root@1234:path/language-style-transfer/code# python -V
Python 2.7.12
root@1234:path/language-style-transfer/code# python -c 'import tensorflow as tf; print(tf.__version__)'
/usr/local/lib/python2.7/dist-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
1.5.0
root@1234:path/language-style-transfer/code# python style_transfer.py --train ../data/yelp/sentiment.train --dev ../data/yelp/sentiment.dev --output ../tmp/sentiment.dev --vocab ../tmp/yelp.vocab --model ../tmp/model
/usr/local/lib/python2.7/dist-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
------------------------------------------------
{   'batch_size': 64,
    'beam': 1,
    'dev': '../data/yelp/sentiment.dev',
    'dim_emb': 100,
    'dim_y': 200,
    'dim_z': 500,
    'dropout_keep_prob': 0.5,
    'embedding': '',
    'filter_sizes': '3,4,5',
    'gamma_decay': 0.5,
    'gamma_init': 1,
    'gamma_min': 0.001,
    'learning_rate': 0.0001,
    'load_model': False,
    'max_epochs': 20,
    'max_seq_length': 20,
    'max_train_size': -1,
    'model': '../tmp/model',
    'n_filters': 128,
    'n_layers': 1,
    'online_testing': False,
    'output': '../tmp/sentiment.dev',
    'rho': 1,
    'steps_per_checkpoint': 1000,
    'test': '',
    'train': '../data/yelp/sentiment.train',
    'vocab': '../tmp/yelp.vocab'}
------------------------------------------------
#sents of training file 0: 179218
#sents of training file 1: 268041
vocabulary size: 9361
2018-01-29 17:55:38.668753: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2
Creating model with fresh parameters.
--------------------epoch 1--------------------
learning_rate: 0.0001   gamma: 1
Killed

Do you have any idea what's happening ?

Can you run classifier.py and language_model.py in this repo? The commands are:
python classifier.py --train ../data/yelp/sentiment.train --dev ../data/yelp/sentiment.dev --vocab ../tmp/yelp.vocab --model ../tmp/model.cls

python language_model.py --train ../data/yelp/sentiment.train.0 --dev ../data/yelp/sentiment.dev.0 --vocab ../tmp/yelp.vocab --model ../tmp/model.lm

Yes both of this commands work perfectly !

Can you run the testing command (with randomly initialized model, but yelp.vocab is required):
python style_transfer.py --test ../data/yelp/sentiment.test --output ../tmp/sentiment.test --vocab ../tmp/yelp.vocab

I haven't encountered this problem before, and the error message doesn't have much information. Are you using the CPU version? That might be a difference, but I'm not sure what's the problem there.

Hello,
I think that's where the problem is coming from.
I am indeed using the CPU version.
This command is not running (or is so slow that I see nothing happening ...).
Is there a way to make it work with the CPU version or is the GPU mandatory ?

Hi,
Unfortunately I think a GPU is necessary to train a model...
I just uploaded a pretrained model (see the readme file and run download_model.sh), you can try it on a small test file (keep a few sentences). I think that should be able to run out.


pipelineSnapshotTesting

vegaprotocol/devops-infra#1384

currently running here: https://jenkins.ops.vega.xyz/job/private/job/playgrounds/job/merlin/
but will create 2 (for devnet and stagnet1) there after merge: https://jenkins.ops.vega.xyz/job/private/job/cd/


bug: Folds not showing symbol on left properly

Neovim version (nvim -v)
0.10.1
Operating system
MacOS
Terminal emulator / GUI
Kitty using tmux
Describe the bug

Good day, did something change related to folds?
Notice in my image above I have a lot of folds and the icon on the left shows up properly, but starting a few days ago, I started to notice that the icons don't show up properly in folds anymore
Notice the image below is the exact same file, but with the issue I'm mentioning, it happens with all my files

If I unfold everything as seen below, the icons do render properly

So its specifically related to folds.

Expected behavior

Notice the image I posted in this already closed issue Change color of Headings #48
Icons were rendering properly back then

Healthcheck output
    2 render-markdown: require("render-markdown.health").check()
    3
    4 render-markdown.nvim [version]
    5 - OK plugin 6.1.4
    6 - OK neovim >= 0.10
    7
    8 render-markdown.nvim [configuration]
    9 - ERROR render-markdown.heading.1: is not a valid key
   10 - ERROR render-markdown.heading.2: is not a valid key
   11 - ERROR render-markdown.heading.3: is not a valid key
   12 - ERROR render-markdown.heading.4: is not a valid key
   13 - ERROR render-markdown.heading.5: is not a valid key
   14 - ERROR render-markdown.heading.6: is not a valid key
   15 - ERROR render-markdown.heading.7: is not a valid key
   16 - ERROR render-markdown.heading.8: is not a valid key
   17 - ERROR render-markdown.heading.9: is not a valid key
   18 - ERROR render-markdown.heading.10: is not a valid key
   19 - ERROR render-markdown.heading.11: is not a valid key
   20 - ERROR render-markdown.heading.12: is not a valid key
   21
   22 render-markdown.nvim [nvim-treesitter]
   23 - OK installed
   24 - OK markdown: parser installed
   25 - OK markdown: highlight enabled
   26 - OK markdown_inline: parser installed
   27 - OK markdown_inline: highlight enabled
   28 - WARNING latex: parser not installed
   29   - ADVICE:
   30     - Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }
   31
   32 render-markdown.nvim [executables]
   33 - WARNING latex2text: not installed
   34   - ADVICE:
   35     - Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }
   36
   37 render-markdown.nvim [conflicts]
   38 - OK headlines: not installed

Plugin configuration

Here's a link to my config file in my dotfiles
Code below:

-- Filename: ~/github/dotfiles-latest/neovim/neobean/lua/plugins/markdown.lua
-- ~/github/dotfiles-latest/neovim/neobean/lua/plugins/markdown.lua

-- https://github.com/MeanderingProgrammer/markdown.nvim
--
-- When I hover over markdown headings, this plugins goes away, so I need to
-- edit the default highlights
-- I tried adding this as an autocommand, in the options.lua
-- file, also in the markdownl.lua file, but the highlights kept being overriden
-- so the inly way is the only way I was able to make it work was loading it
-- after the config.lazy in the init.lua file lamw25wmal

-- Define color variables
-- These are the colors for the eldritch colorscheme
local color1_bg = "#f265b5"
local color2_bg = "#37f499"
local color3_bg = "#04d1f9"
local color4_bg = "#a48cf2"
local color5_bg = "#f1fc79"
local color6_bg = "#f7c67f"
local color_fg = "#323449"
-- local color_sign = "#ebfafa"

return {
  "MeanderingProgrammer/markdown.nvim",
  opts = {
    heading = {
      sign = false,
      icons = { "󰎤 ", "󰎧 ", "󰎪 ", "󰎭 ", "󰎱 ", "󰎳 " },

      -- Heading colors (when not hovered over), extends through the entire line
      vim.cmd(string.format([[highlight Headline1Bg guifg=%s guibg=%s]], color_fg, color1_bg)),
      vim.cmd(string.format([[highlight Headline2Bg guifg=%s guibg=%s]], color_fg, color2_bg)),
      vim.cmd(string.format([[highlight Headline3Bg guifg=%s guibg=%s]], color_fg, color3_bg)),
      vim.cmd(string.format([[highlight Headline4Bg guifg=%s guibg=%s]], color_fg, color4_bg)),
      vim.cmd(string.format([[highlight Headline5Bg guifg=%s guibg=%s]], color_fg, color5_bg)),
      vim.cmd(string.format([[highlight Headline6Bg guifg=%s guibg=%s]], color_fg, color6_bg)),

      -- Highlight for the heading and sign icons (symbol on the left)
      -- I have the sign disabled for now, so this makes no effect
      vim.cmd(string.format([[highlight Headline1Fg cterm=bold gui=bold guifg=%s]], color1_bg)),
      vim.cmd(string.format([[highlight Headline2Fg cterm=bold gui=bold guifg=%s]], color2_bg)),
      vim.cmd(string.format([[highlight Headline3Fg cterm=bold gui=bold guifg=%s]], color3_bg)),
      vim.cmd(string.format([[highlight Headline4Fg cterm=bold gui=bold guifg=%s]], color4_bg)),
      vim.cmd(string.format([[highlight Headline5Fg cterm=bold gui=bold guifg=%s]], color5_bg)),
      vim.cmd(string.format([[highlight Headline6Fg cterm=bold gui=bold guifg=%s]], color6_bg)),

      backgrounds = {
        "Headline1Bg",
        "Headline2Bg",
        "Headline3Bg",
        "Headline4Bg",
        "Headline5Bg",
        "Headline6Bg",
      },
      foregrounds = {
        "Headline1Fg",
        "Headline2Fg",
        "Headline3Fg",
        "Headline4Fg",
        "Headline5Fg",
        "Headline6Fg",
      },
    },
  },
}

Plugin error log

File missing

<EMAIL_ADDRESS>[24/08/15]
❯ ll
Permissions Size User     Group Date Modified Name
.rw-r--r--  2.1k linkarzu staff  9 Aug 05:31  conform.log
.rw-r--r--   11k linkarzu staff 14 Aug 18:06  file_frecency.bin
drwxr-xr-x     - linkarzu staff 10 Jul 19:59  lazy
.rw-r--r--   540 linkarzu staff 25 Jul 11:39  log
.rw-r--r--   17k linkarzu staff 14 Aug 09:01  lsp.log
.rw-r--r--    75 linkarzu staff 14 Aug 09:01  mason-registry-update
.rw-r--r--  122k linkarzu staff 15 Aug 05:23  mason.log
.rw-r--r--  6.5k linkarzu staff 30 Jul 16:45  noice.log
drwxr-xr-x     - linkarzu staff 10 Aug 05:58  sessions
drwx------     - linkarzu staff 15 Aug 05:23  shada
drwxr-xr-x     - linkarzu staff 15 Aug 05:23  swap
drwxr-xr-x     - linkarzu staff 14 Aug 18:24  undo
drwxr-xr-x     - linkarzu staff 25 Jul 10:56  view

Confirmations

[X] I have provided markdown text for any screenshots used in my description & understand that my issue will be closed if I have not

Additional information
No response

Not sure. But it feels like only the symbols that are near the cursor render and not all the symbols in the file. I may just be talking nonsense though

Found the issue.
There was a change about 2 weeks ago to only parse the visible range of a buffer to improve performance: https://github.com/MeanderingProgrammer/render-markdown.nvim/commit/c7a20552b83c2abad92ac5e52feb7fe3b929f0a7.
The visible range calculation doesn't take into account folds. So content in a fold gets parsed while leaving visible content outside a fold unparsed. This explains the symbols near the cursor still rendering sometimes, would just depend on the size of the individual folds.
I'll see if I can find the folds and adjust the range based on them. If not I'll add a configurable buffer for the range that you could set to a really high value, would work, just not an ideal solution.

Wonderful! Appreciate you taking the time to take a look
Yeah, around 2 weeks ago sounds about right.
I'll test in a bit and report back

Should be fixed by: https://github.com/MeanderingProgrammer/render-markdown.nvim/commit/cd0a5ad8c77c3754d02437048bc0bb604a2fe268
LMK if it works, performance should hopefully seem unaffected, but large folds will likely cause problems.

Yep, just updated and its back to normal, really appreciate it! (see image below for reference)

Sweet, thanks for bringing this up! I don't use folds so never came across this.
LMK if you find any other issues or have ideas!
One minor aside, I would suggest moving the highlight creation out of opts. It's leading to the health check errors, the plugin essentially ignores them so they're not harming things but add a bit of noise.
Something like so:

return {
  "MeanderingProgrammer/render-markdown.nvim",
  init = function()
    local color1_bg = "#f265b5"
    local color2_bg = "#37f499"
    local color3_bg = "#04d1f9"
    local color4_bg = "#a48cf2"
    local color5_bg = "#f1fc79"
    local color6_bg = "#f7c67f"
    local color_fg = "#323449"

    -- Heading colors (when not hovered over), extends through the entire line
    vim.cmd(string.format([[highlight Headline1Bg guifg=%s guibg=%s]], color_fg, color1_bg))
    vim.cmd(string.format([[highlight Headline2Bg guifg=%s guibg=%s]], color_fg, color2_bg))
    vim.cmd(string.format([[highlight Headline3Bg guifg=%s guibg=%s]], color_fg, color3_bg))
    vim.cmd(string.format([[highlight Headline4Bg guifg=%s guibg=%s]], color_fg, color4_bg))
    vim.cmd(string.format([[highlight Headline5Bg guifg=%s guibg=%s]], color_fg, color5_bg))
    vim.cmd(string.format([[highlight Headline6Bg guifg=%s guibg=%s]], color_fg, color6_bg))

    -- Highlight for the heading and sign icons (symbol on the left)
    -- I have the sign disabled for now, so this makes no effect
    vim.cmd(string.format([[highlight Headline1Fg cterm=bold gui=bold guifg=%s]], color1_bg))
    vim.cmd(string.format([[highlight Headline2Fg cterm=bold gui=bold guifg=%s]], color2_bg))
    vim.cmd(string.format([[highlight Headline3Fg cterm=bold gui=bold guifg=%s]], color3_bg))
    vim.cmd(string.format([[highlight Headline4Fg cterm=bold gui=bold guifg=%s]], color4_bg))
    vim.cmd(string.format([[highlight Headline5Fg cterm=bold gui=bold guifg=%s]], color5_bg))
    vim.cmd(string.format([[highlight Headline6Fg cterm=bold gui=bold guifg=%s]], color6_bg))
  end,
  opts = {
    heading = {
      sign = false,
      icons = { "󰎤 ", "󰎧 ", "󰎪 ", "󰎭 ", "󰎱 ", "󰎳 " },
      backgrounds = {
        "Headline1Bg",
        "Headline2Bg",
        "Headline3Bg",
        "Headline4Bg",
        "Headline5Bg",
        "Headline6Bg",
      },
      foregrounds = {
        "Headline1Fg",
        "Headline2Fg",
        "Headline3Fg",
        "Headline4Fg",
        "Headline5Fg",
        "Headline6Fg",
      },
    },
  },
}

So that's what was causing the noise, applied what you suggested and now its gone, thank you!
    2 render-markdown: require("render-markdown.health").check()
    3
    4 render-markdown.nvim [version]
    5 - OK plugin 6.1.7
    6 - OK neovim >= 0.10
    7
    8 render-markdown.nvim [configuration]
    9 - OK valid
   10
   11 render-markdown.nvim [nvim-treesitter]
   12 - OK installed
   13 - OK markdown: parser installed
   14 - OK markdown: highlight enabled
   15 - OK markdown_inline: parser installed
   16 - OK markdown_inline: highlight enabled
   17 - WARNING latex: parser not installed
   18   - ADVICE:
   19     - Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }
   20
   21 render-markdown.nvim [executables]
   22 - WARNING latex2text: not installed
   23   - ADVICE:
   24     - Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }
   25
   26 render-markdown.nvim [conflicts]
   27 - OK headlines: not installed
   28 - OK obsidian: not installed
   29


Is tooltip dynamic? Can it be rendered on change?

I am incrementing the value on select(on change), this change doesn't reflect in the tooltip until I refresh. Is it possible to render or refresh the tooltip on change?

When there is a single slider on page then
  $scope.$on("slideEnded", function() {       $timeout(function() {         $scope.$broadcast('rzSliderForceRender');       });   });
^this works, but when in ng-repeat how to force refresh?

It's working, Sorry for the trouble.
And thanks for making such a good directive


Switch from global api.Scheme to local Service Catalog Scheme

The sample-apiserver (API server example) is not using api.Scheme at least since 1.7 (see apiserver.go), while Service Catalog still registers all the types in the global client-go/api.Scheme.
While it worked for k8s 1.7 (client-go 4.0), the client-go/pkg/api package was completely removed from the client-go repo in k8s 1.8 (client-go 5.0). Some of this code now lives in a separate k8s.io/api repo, but there is no global api.Scheme there (looks like it was completely dropped).
I see two possible ways of fixing this issue before switching to 1.8:

Follow the pattern used in sample-apiserver: declare our own Scheme and used it where needed instead of api.Scheme
Technically we could use the kubernetes global Scheme instead of client-go (it's still there and seems to have the same types/methods). While it could be less work for us, I would not do this, as it contradicts the pattern used by sample-apiserver, and it looks like a quick hack which will lead to more serious issues in the future (as we diverge from the recommended path of changes performed in the main k8s repo over the last several months).

Note that while I only mentioned api.Scheme, there are other objects used from the same package, so it applies to using the entire k8s.io/client-go/pkg/api package in general (Scheme, Registry, Codecs etc). api.Scheme seems to be the central part there though, so I would start with deciding on the changes of using it first.


Wierd behavior when running in arm64

When upscaling images or GIFs in an arm based machine, the function finishes almost instantly, without errors and without generating the upscaled file.
const waifu2x = require("waifu2x")

(async () => {
    await waifu2x.upscaleImage("./image.png", "./upscaled.png", {
        upscaler: "waifu2x"
    })
})()

Code that i used to test, tested with all default upscalers, waifu2x, real-esrgan and real-cugan.
Any leads on what could cause this strange behavior?

On macOS you have to run the chmod777 function to set execution permissions on the binaries. You may also have to go in and right click -> open each, because they are unverified they will refuse to run the first time.

On macOS you have to run the chmod777 function to set execution permissions on the binaries. You may also have to go in and right click -> open each, because they are unverified they will refuse to run the first time.

Im using linux with an Ampere arm cpu, even after modifying the script to run the chmod777 function, it doenst work or error.
I also did further testing, Windows 11 x86 with gpu runs fine, Linux x86 with gpu runs fine, Linux x86 without gpu works for real_cugan and waifu2x (real_esrgan doenst seem to support cpu mode), Linux arm64 does not work with any

After compiling real-cugan from source, and changing the binary, real-cugan upscaler now works as intended.
Should i look into the source code and see if i can make the library throw an error when it finds an incompatible binary?

The provided binaries are compiled for x64 linux and will not run on arm64 without translation software. You will need to use the linux equivalent of Rosetta 2, quick search found me this: https://box86.org/
You could also compile each of the programs from source. You can open a pull request to add them but this project is approaching the maximum allowed npm package size and I don't know if they will fit.
https://github.com/nihui/waifu2x-ncnn-vulkan
https://github.com/xinntao/Real-ESRGAN-ncnn-vulkan
https://github.com/nihui/realcugan-ncnn-vulkan


Use directories

Hi, this PR causes preferences to use the directories crate instead of the app_dirs crate, which is retired.

Would it be possible to merge this @andybarron ?
Forking would be a solution but not the solution !

Would be nice to see this merged :-).


Reimplement match as a call

Should resolve #1219, but we will have to JIT compile this kind of match

Merge now, because 6 greens is really rare


Comparison to serverless + aws-scala-sbt

I'm just exploring scala lambdas, and I was starting to use the serverless project with the aws-scala-sbt plugin. It works for a single toy function, but I don't have a good picture of how cleanly I can keep my source structured as I add more functionality. (And their local invoke is currently broken which would add a fairly annoying overhead to testing...)
Then I came across your project, which looks like an interesting alternative. I saw your comment about new docs coming soon, and I wondered whether you could perhaps include a quick list of pros and cons in comparison to serverless. In terms of getting started, they provide serverless create-template which gets you up and running with a sample very easily. Looking at your project it's not clear to me how to get started since I'm not familiar with SAM. Will you be including a simple setup example, and do you consider this project ready for widespread use yet?
Thanks.

Hi there. Thanks for your interest. I would not consider myself an expert
in SAM, and therefore the scope of the library is really small: it just
helps you write somewhat cleaner lambda functions in Scala (compared to
just using the java lib from AWS) and takes care of (de)serialization of
input and output.
The library is totally agnostic of the way you organize the code, build and
deploy. So, sorry, this project will not help you with the complexity of
SAM and issues with testing, but it can help reducing boilerplate you’ll
need to write for each function. Does this make sense?
You are right, I should make it clear in the readme, thanks for posting the
issue.
On Wed, 7 Mar 2018 at 21:32, thunderkid<EMAIL_ADDRESS>wrote:

I'm just exploring scala lambdas, and I was starting to use the serverless
project with the aws-scala-sbt plugin. It works for a single toy
function, but I don't have a good picture of how cleanly I can keep my
source structured as I add more functionality. (And their local invoke is
currently broken which would add a fairly annoying overhead to testing...)
Then I came across your project, which looks like an interesting
alternative. I saw your comment about new docs coming soon, and I wondered
whether you could perhaps include a quick list of pros and cons in
comparison to serverless. In terms of getting started, they provide serverless
create-template which gets you up and running with a sample very easily.
Looking at your project it's not clear to me how to get started since I'm
not familiar with SAM. Will you be including a simple setup example, and do
you consider this project ready for widespread use yet?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mkotsur/aws-lambda-scala/issues/13, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAKDKrZpAI5k0PNE96lINxf2MVzPkAgyks5tcEO8gaJpZM4ShHEZ
.

Makes sense, thanks.
 (Two gives a better sense of how much DRY could be achieved.) If it's a good fit, users would have the option to learn more SAM themselves as needed, but they'd have a starting point to build from.

https://medium.com/@mkotsur/this-is-why-you-should-consider-using-aws-lambda-scala-6b3ea841f8b0
Would you say that this post answers your questions? I'll add link to it to the README.
Providing a good starting example would be nice, but it's a bit out of scope for this library. Serverless provides an example, and I'm just explaining how aws-lambda-scala can help you write more pure Scala.

Sorry about the delay. Yes thanks, that answers the question I was asking. I didn't understand that it could be run in addition to serverless - I thought required SAM instead. It looks very useful and I shall now try to implement it...

Great, thanks for the feedback.

Yes, version 0.0.11 is basically the same one as 0.0.13, which is the latest one. There's been a bit of confusion: 0.0.12 has been accidentally released only for Scala 2.11, but starting from 0.0.13 both versions of Scala will be supported.


Fixed #25655: Dropped support for GEOS < 3.3.

Apart the minor issue with prepared.py, the patch is ready.

Oh, and just a small nit-pick: could you separate the ticket number from the message by -- instead of :? (Fixed #25655 -- ...).

Merged in b78226cd3d, thanks!


Support non-pure Grid.Column valueProviders

Usually the value providers passed into grid.addColumn(vp) are pure functions, and such use-case is correctly supported by Karibu-Testing.
However, it's possible to write the following code which works with Vaadin but fails with Karibu:
var grid = new Grid<Person>();
grid.addColumn(person -> { person.setCalculated("Foo"); return person.getCalculated(); });
grid.addSelectionChangeListener(e -> assertEquals("Foo", e.getItem().getCalculated());
grid.setItems(new Person());
_select(grid, _get(grid, 0));

The Karibu code will fail in _select(grid, _get(0)); - the selection listener's assertEquals() will fail. The problem is that _get(grid, 0) will not run the column value providers, causing the person.calculated property to not to be filled in.

One of the solutions would be for _get() to take advantage of the DataCommunicator's cache (it caches all visible objects and generates a key for them; the key is then used by client-side when a row is clicked etc). This could however break existing tests which rely on getting fresh values from the data provider? However, it could be possible for DataProvider._get() to return fresh stuff while Grid._get() could return cached stuff, passed through value providers?
Another solution would be to add a function to Karibu which would run value providers on a bean, therefore shifting the responsibility to the test writer... Not sure about this.

Yup, the KeyMapper is not populated. Perhaps _get() could populate the KeyMapper manually, then reuse the returned beans?

The KeyMapper caching solution has been implemented in Karibu-Testing 1.3.18. It's a potentially breaking change, but everything happens under the hood so hopefully it won't break anything. If it does, please open a new ticket.


do not copy columns

Codecov Report

Merging #42 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   85.51%   85.48%   -0.03%
==========================================
  Files           5        5
  Lines        1049     1047       -2
==========================================
- Hits          897      895       -2
  Misses        152      152

Impacted Files
Coverage Δ

src/TableReader.jl
91.58% <100%> (ø)
:arrow_up:

src/download.jl
66.66% <0%> (-13.34%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix wrong console cursour position with long input text

Also changed cursor from "|" symbol to rectangle as it's less likely to be affected by screen resolution - rectangle takes centre as drawing position.

Made the commit message more descriptive

I thought text margin would be different if ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME is called multiple times for one text label drawing, but looks like it's not the case. For this reason, I don't think there's no need to call GetTextLength recursively.
In this snippet, width2 and width3 will match and the feed will show Text widths are the same!.
Function.Call(Hash.SET_TEXT_FONT, 0);
Function.Call(Hash.SET_TEXT_SCALE, 0.35f, 0.35f);
Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "ABCDEF");
var width1 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

Function.Call(Hash.SET_TEXT_FONT, 0);
Function.Call(Hash.SET_TEXT_SCALE, 0.35f, 0.35f);
Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "ABC");
Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "DEF");
var width2 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

if (width1 == width2)
{
  GTA.UI.Notification.Show("Text widths are the same!");
}
else
{
  GTA.UI.Notification.Show("Text widths are not the same!");
}

Also note that END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT returns very small value like 0.001 if no characters are pushed, which is different from this snippet:
float GetTextMargin()
{
  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.35f, 0.35f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "A"); // any letter is allowed, even if font file does not contain the glyph for that letter
  var len1 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.35f, 0.35f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "AA");
  var len2 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  return len1 - (len2 - len1); // [Margin][A] - [A] = [Margin]
}

In this snippet, width2 and width3 will match and the feed will show Text widths are the same!.

I'm not sure i understand what you mean?
That's the case for strings less than ~99 bytes. PushLongString, by default already split the string into chunks and push them separately, it works well except for END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, that's why I needed to begin a new text command to measure the length of the remaining string.(as suggested in #1133)
But every command takes into account the little invisible space - referred to as margin, at the start of the text label (multiple calls to ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME doesn't increase the margin as they're seen as single command), so i have to calculate the length of that space first and substracts it in subsequent recursive calls.

Looks like that's even the case for strings more than 99 bytes, making I said wrong about text margin. Even in this example, AboutEqual call will return true.
static void TestMethod()
{
  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.30f, 0.30f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, new string('a', 99));
  var widthSplitted1 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.30f, 0.30f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, new string('a', 3));
  var widthSplitted2 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.30f, 0.30f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, new string('a', 99));
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, new string('a', 3));
  var widthNoSplitted = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  // AboutEqual guarantees the 2 float expressions have almost the same value (the small difference is caused by rounding error)
  if (AboutEqual(widthNoSplitted, (widthSplitted1 + widthSplitted2 - (margin))))
  {
    GTA.UI.Notification.Show("Text widths are the same!");
  }
  else
  {
    GTA.UI.Notification.Show("Text widths are not the same!");
  }
}

static float GetMarginLength()
{
  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.30f, 0.30f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "a");

  var width1 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  Function.Call(Hash.SET_TEXT_FONT, 0);
  Function.Call(Hash.SET_TEXT_SCALE, 0.35f, 0.35f);
  Function.Call(Hash.BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, "CELL_EMAIL_BCON");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "");
  Function.Call(Hash.ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME, "aa");

  var width12 = Function.Call<float>(Hash.END_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT, true);

  return width1 - (width12 - width1);
}

static float CalcFloatEpsilon()
{
  float eps = 1.0f;
  while (1.0f + eps / 2.0f > 1.0f)
  {
    eps /= 2.0f;
  }
  return eps;
}

public static bool AboutEqual(float x, float y)
{
  return Math.Abs(x - y) <= CalcFloatEpsilon();
}

Maybe take maxLengthUtf8 as a parameter and make the default 99, and change it to 67 when measuring text length?

I guess we could add maxLengthUtf8 as a parameter to GetTextLength and make the default 99. As I tested, BEGIN_TEXT_COMMAND_GET_SCREEN_WIDTH_OF_DISPLAY_TEXT takes up to 98 bytes (99 bytes including the terminator character) per ADD_TEXT_COMPONENT_SUBSTRING_PLAYER_NAME call.


Release 9.22 has version 9.21

Apparently new released version 9.22 has set library version to 9.21, see:
https://github.com/ladislav-zezula/StormLib/blob/v9.22/CMakeLists.txt#L316

Fixed in commit 008a48ed998dedcae4a6c7646c8f8562eb8af906.


Revert "BUG: SetScale() and SetSkew() did not recompute offset: ScaleSkewVersorT"

Reverts InsightSoftwareConsortium/ITK#1523, to preserve existing transform files the transform numerics and number of parameters must not change.

Python configuration failure is unrelated.


fix: clean start

This should be covered with tests

We also need to create an issue in vue-router repo I think...


Fix requirement listing extras misses space

Codecov Report

Merging #277 (ac1190d) into main (9c3242e) will decrease coverage by 0.84%.
The diff coverage is 92.27%.

@@            Coverage Diff             @@
##             main     #277      +/-   ##
==========================================
- Coverage   91.30%   90.45%   -0.85%
==========================================
  Files          17       16       -1
  Lines        1576     1394     -182
==========================================
- Hits         1439     1261     -178
+ Misses        137      133       -4

Flag
Coverage Δ

unittests
90.45% <92.27%> (-0.85%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

setup.py
0.00% <0.00%> (ø)

grayskull/license/discovery.py
95.62% <44.44%> (-1.52%)
:arrow_down:

grayskull/base/github.py
73.58% <73.58%> (ø)

grayskull/strategy/abstract_strategy.py
83.33% <83.33%> (ø)

grayskull/utils.py
89.18% <85.18%> (-10.82%)
:arrow_down:

grayskull/base/pkg_info.py
85.71% <87.50%> (+0.52%)
:arrow_up:

grayskull/strategy/py_base.py
94.58% <94.58%> (ø)

grayskull/__main__.py
97.29% <96.29%> (-0.86%)
:arrow_down:

grayskull/config.py
98.07% <98.07%> (ø)

grayskull/strategy/pypi.py
98.66% <98.66%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


fix: check that last_non_whitespace_character is not an empty string

Fixes the problem where JSON Schema could end up with leading commas.  The problem was that last_non_whitespace_character is set to an empty string if there has not been a non whitespace character, but the check that increments num_items didn't handle that case.
Dealign with the empty string is a bit unsatisfying, but I didn't feel comfortable with all of the implications of refactoring it other than to add the check.
Resolves #99

Thank you for finding this!
last_non_whitespace_character should never be an empty string once the parsing encountered one non whitespace character, so I solved it in the correct way:
https://github.com/noamgat/lm-format-enforcer/commit/5461294f5cb6ee5eae677c75358f4b83353ee4db


Education endpoint needed

SubTask of User Story 09

Design adjustment:
Model 'Many to One' relationship on 'occupation' changed to 'One to One'
Description changed to JSONField
The API returns at least 2 different results for the description.  It is quicker to change this field to a JSON Field to take in both styles as well as the additional information that the API returns in the entry.  This will be reviewed later to determine if a more efficient method can be implemented.


Attaching to the next WebServiceClient fails with "Invalid debug adapter"

1. Describe the bug
I try to attach to the next WebServiceClient. If I trigger the webservice call, the debugger just stops and there is a log entry in the "extension host" window in vscode.
2. To Reproduce
Steps to reproduce the behavior:
Have a workspace file with the following launch/configurations settings:
{
  "name": "Attach: sandbox-test (WebService)",
  "type": "al",
  "request": "attach",
  "tenant": "<tenant-id>",
  "environmentType": "Sandbox",
  "environmentName": "sandbox-test",
  "breakOnError": "All",
  "breakOnRecordWrite": "None",
  "breakOnNext": "WebServiceClient",
  "userId": "USER ID"
}

Run: "AL: Debug without publishing"
3. Expected behavior
Debugging should work
4. Actual behavior
The following log entry occurs in vscode, as soon as the webservice call is triggered.
2024-06-13 17:20:20.593 [error] Error: Invalid debug adapter
    at l.p (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1769:95)
    at l.$acceptDAExit (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1769:35)
    at y.S (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1658:14598)
    at y.Q (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1658:14364)
    at y.M (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1658:13397)
    at y.L (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1658:12453)
    at u.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1658:10928)
    at h.y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:89:659)
    at h.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:89:876)
    at r.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:615:14848)
    at U.onmessage (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1673:8154)

5. Versions:

AL Language: Version: 13.0.1027618
Visual Studio Code: 1.90.0
Business Central: Version: DE Business Central 24.1, (Platform 24.0.20440.0 + Application 24.1.18927.19282)
List of Visual Studio Code extensions that you have installed: Tested with only AL extension enabled.

Internal work item: AB#538498

I have no repro on this? On the other hand it looks like a vscode issue? Have you tried with the experimental vscode? Whether that also reproes?
I have seen traces of similar issues on teh vscode branch:
https://github.com/microsoft/vscode/issues/196948


warcio doesn't verify digests on read

I was experimenting with injecting digests from my crawler, so that digests aren't computed twice, and noticed that records with a bad WARC-Payload-Digest don't raise an exception on read. No code for it, so I suppose this is a feature request.
The check should be disable-able, and "warcio index" and recompress ought to have a command line flag to ignore digest errors.
Lacking this feature, I don't think that warcio currently has any test to ensure that it's correctly computing digests.

I coded this up by hacking LimitReader to be configurable by recordloader.py to test digests. If I turn it on for all of the existing tests, the only ones that fail are the truncated warc (expected) and the block_digest revisit record in example.warc (which might be your bug, not mine... I'll figure it out)
Because of your enthusiasm for streaming, the digest check will only fire if the user reads to the end of the record. So be it.
I have a few style questions:

Should I raise ArchiveLoadFailed if the digest check fails? ArchiveDigestFailed? ValueError?

Should the digest check default off? I didn't check perf, but it ought to be a lot cheaper than decompressing.


Remove corresponding author field from unpublished study page in Search UI

The corresponding author field currently displays the first author instead, so should be temporarily removed to avoid displaying incorrect information.

e.g. on this page: https://www.ebi.ac.uk/gwas/studies/GCST90267408

This is waiting for release


Equivalent for importing png/jpg

🚀 Feature Proposal
Being able to import .png|.jpg as ReactJS components directly
Motivation
We found this module convenient with https://react-styleguidist.js.org/ to distribute to our people a static assets bundler with html documentation.
We now have all documentation in HTML for every of our svg quite quickly thanks to your module.
This is how we configure :
    module: {
      rules: [
        {
          test: /\.svg$/,
          use: ['@svgr/webpack'],
        }
      ],
    },

and
    {
      "name": "Svg components",
      "components": "static/**/*.svg"
    }

This is how a markdown can import a svg quickly :
```js
<LogoBlack />
```

Example
This is how we'd like to import all our images:
    {
      "name": "Images",
      "components": "static/**/*.{png|jpg}"
    },
    {
      "name": "Svg components",
      "components": "static/**/*.svg"
    }

And in markdown, we can use logoPurple.{png|svg}
```js
<LogoPurple />
```

Pitch
It simply does not, but I have not found an equivalent for doing the same, and without experience, the closest people who could achieve that are people from this module.
My experience with babel and transformation is not big, I am more a user of webpack and babel.

JPG / PNG is another thing, completely different. There is a lot of plugins that are already doing it.
For an example, you have lqip-loader that does a great job!
SVGR is focused on SVG, for now we don't want to extend it to PNG / JPG.

@neoziro thanks for the lqip-loader link.
What I am looking for is something that would turn the JPG/PNG/GIF into a valid React component.
If you have an idea how I could start. Thanks in advance.

There is a lot of way to load an image... but the simplest one is , you don't need a converter ;)

neoziro, I'd like to avoid to write that and have directly . Lazy level 10
Even if it's sounds dumb, do you know how to do :)?

I think you could write a simple webpack loader, take inspiration from the svgr one.

That's what I thought, and also why I ask you this. I've read the svgr/webpack plugin and I don't see how the convert file is transforming the export.
Where is the export build? This could lead to a potential plugin that would rock!
We currently use this with svgr and react-styleguidist, and we'd like to make one of those for images. Thanks if you agree to help me get through this.

I am sorry, I can't help you doing it. I suggest you to read writing a webpack loader, I am sure you will find a way to implement it!

I went through this already and I found it was not possible. Just wantede to confirm. But thanks anyway.


Support Standalone Authentication

Dashboard should be able to sign in use standalone credentials, which is stored in etcd, so that dashboard can auth users even if there is no TiDB.

Related: https://github.com/pingcap-incubator/tidb-dashboard/issues/88


Light Theme for <NTooltip />

This function solves the problem (这个功能解决的问题)
Unified Equipment
Expected API (期望的 API)
datatable Support for ellipsis light theme

ellipsis calls the tooltip variables, you can look at the tooltip variables, please refer to https://www.naiveui.com/en-US/light/docs/customize-theme#Use-the-peers-vars

Please describe your scene.

The overall style of the project is bright color, suddenly a dark color, do not match

All themes in the current project are in bright colors. If they are dark, they will not fit

Tooltip is black backgrounded by design in light theme.

I may add the feature. Not sure for now.

Ok, thank you

yes thank you

I've managed to fix this by adding a :theme-overrides attribute to the n-config-provider with a tooltip colour value set to white.
<n-config-provider
  :theme="lightTheme"
  :theme-overrides="{
    Tooltip: {
      color: 'white'
    }
  }"
>
</n-config-provider>


 The winner has to be determined probabilistically.

Resolved with the new quantum chess engine


Version 1.3 will not Consume from Broker v 2.4 of Kafka. It is broken...

Description
Just run basic Consume example against inter.broker.protocol.version=2.4-IV1 it it just HANGS.  It works from my laptop with kafkat and Conduktor.IO.  Also works with Java Spring Boot.  But dotnet core 3.1 with version 1.3 is a non-starter.  We are moving to Kafka fast.  I need this to work, since everyone else seems to have abandoned their Kafka packages in favor of yours.  Our broker server is currently http://[IP Address]:19092 for our POC, incase that makes a difference.  I am also running this on Windows 10 laptop.
How to reproduce
Run basic Consume in Readme.
Checklist
Please provide the following information:

[ x] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[x ] Confluent.Kafka nuget version.
[x] Apache Kafka version.
[x] Client configuration.
[x] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[x] Critical issue.

Please reproduce this issue with ConsumerConfig Debug: "consumer,broker" and provide the logs.

closing due to inactivity


Bubble charts - ability to have names for the bubbles

Summary
I think it would be useful to have the ability to add names to the bubbles. It is helpful to identify outliers. This is what it could look like:

API Changes
I think adding a 4th (optional) parameter with the name would work.
Instead of [5, 10, 50] it could be  [5, 10, 50, 'Bubble Name']
Intended Use Case
Identifying outliers mostly, and individual datapoints.

Thanks @junedchhipa
I didn't know about the meta property. I was currently getting the data from the initial config.
tooltip.x = {
        formatter: function (val, {series, seriesIndex, dataPointIndex, w}) {
            const name = w.globals.initialConfig.series[seriesIndex].data[dataPointIndex][3];
            return name + ' (' + val + '%)';
        }
    }


Solutions workbook

Required items, please complete
Change(s):
Added the Tool/SolutionData folder and added the PowerShell to create the CSV files, a sample CSV file, a sample PowerBI file, and the workbook
Reason for Change(s):
First push
Version Updated:
N/A
Testing Completed:
Testing completed in MS Sentinel instance
Checked that the validations are passing and have addressed any issues that are present:
N/A

Hi @garybushey, could you please update your branch with master? Thanks

Hi @garybushey, please update your branch with master? Thanks

Hi @garybushey, could you please update your branch with master? Thanks

Hi @garybushey, please update your branch with master? Thanks

Hi @garybushey,
few check are stuck so please update your branch from master. Thanks

Hi @garybushey,
As we are unable to figure out why checks are stuck, we would like you to take the latest master and push it. Thanks

Hi @garybushey, please update your branch with master? Thanks

Hi @garybushey, please update on above. Thanks

Hi @garybushey,
Please work on above comment.

Hi @garybushey,
As we are unable to figure out why checks are stuck, we would like you to take the latest master and push it. Thanks

We wanted to check on the status of PR #9188. PR is pending for more than 30 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Since we have not received a response in the last 7 days, we are closing your PR #9188 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation.


1.1 Implicit Methods Middleware are not available anymore

After yesterday merge now on master (1.1-dev) implicit methods middleware are no longer avaibale.
There is documentation on develop (2.0-dev) which says:

Starting with version 1.1, Expressive offers middleware for implicitly supporting HEAD and OPTIONS requests.
https://github.com/zendframework/zend-expressive/blob/develop/doc/book/features/middleware/implicit-methods-middleware.md

Is it possible to backport them also to 1.1 version or documentation of 2.0 will be updated and this feature is gonna be only in version 2?
Please see #413 (base PR with this functionality).

We could backport them; however, they are currently written to target http-middleware 0.4.1, which means they'd need to be rewritten to work with Expressive 1.1 — which is why I didn't do it in the first place.
I'd rather save the feature for 2.0, and have 1.1 focus on deprecations and preliminary support for pipelines. As such, I'm marking this as a documentation fix, and will flag it for the 2.0.0 milestone.

@weierophinney I've tried it before, and here is PR if you change your mind and want backport this feature https://github.com/zendframework/zend-expressive/pull/439
I've just cherry pick your first commits with this functionality.

Just my 2 cents: Don't backport too much. I think it makes thing much more difficult and it delays a 1.1 release even more. Use 1.1 for the current merged features and deprecations, release it and leave all new features for 2.0.

Decided not to do this for the 1.1.0 release, for reasons stated in #439 and here.

@weierophinney we need then documentation fix, please see:
https://github.com/zendframework/zend-expressive/blob/develop/doc/book/features/middleware/implicit-methods-middleware.md, this is why you labeled this issue ;)


Not all links on page detected

Describe the bug
Hello,
When I scan the page https://audit.zugang4all.de/ with crawl enabled, I only get the report for the / and /contact page.
All links on the top are ignored.
Here is my config.

/// <reference types="unlighthouse" />
import { defineConfig } from 'unlighthouse'

export default defineConfig({
  site: 'https://audit.zugang4all.de/',
  debug: true,
  cache: false, // disable caching of results
  scanner: {
    maxRoutes: 500, // max number of routes to scan
    samples: 2, // number of samples to take
    crawler: true, // enable the crawler
    robotsTxt: false, // don't respect robots.txt
    sitemap: false, // don't respect sitemap.xml,
    // exclude all pdfs and amp pages
    exclude: [
      '/.*?pdf',
      '.*/amp',
      /* 'en-*',*/
    ],
    // use desktop to scan
    device: 'desktop',
    // enable the throttling mode, good for getting more consistent results
    throttle: false,
  },
  chrome: {
    useSystem: false, // use the bundled chrome,
  },
  puppeteerOptions: {
    headless: true, // headless works best
  }
})

Reproduction
No response
System / Nuxt Info
Win 11, bundled chromium, node 18

I have a similar issue. Looks like unlighthouse does not find certain links. What do do?

I have a similar issue. Looks like unlighthouse does not find certain links. What do do?

I probably use something else (eg playwright and lighthouse by myself.)
Most of the reported performance metrics are way of / to low in comparison with page speed insights or running lighthoues by myself

Seems like something on your end is blocking Unlighthouse's crawler. I don't think there's anything I can do on my end sorry.


UNABLE TO SUBMIT LIMIT ORDERS - Failed to submit your order/Swap failed- PLEASE HELP I CAN'T TRADE

Good day, I have been unable to submit limit orders via the order book or via merged quotes for almost all of the ETH pair I've been trying since around January 21st around 11 PM EST.
Each time I try to submit an order, I get the following messages :

However, for some reasons, it seems that if I try to route an order via the bot, it works and get submitted, but when I try to do it manually, it gets rejected each time. Order routed via bot :

It seems like I can't trade any ETH-PAIR. I've been able to submit orders on USDC/USDT and DAI/USDT for some reasons.
I have enough ETH available and it is not frozen. I have tried a dozens pair in both the legacy UI and AMM I can't seems to figure out why it won't let me place some limit orders and again it will place it when routed via Label Order Maker so enough ETH is available..
Please help,
Kind regards,

Seemslike there's a maximum amount of orders that can be placed, idk.


feat: add etimo-client and break out security project

Codecov Report

Merging #48 (1bf2624) into develop (c04fad0) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop     #48      +/-   ##
==========================================
- Coverage     8.88%   8.50%   -0.39%
==========================================
  Files          143     149       +6
  Lines         2566    2682     +116
==========================================
  Hits           228     228
- Misses        2338    2454     +116

Flag
Coverage Δ

unittests
8.50% <0.00%> (-0.39%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../Etimo.Id.Api/Controllers/ApplicationController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/AuditLogController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/OAuthController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/RoleController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/ScopeController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/UserController.cs
52.63% <ø> (ø)

src/Etimo.Id.Api/Startup.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdBaseClient.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdBootstrapper.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdClient.cs
0.00% <0.00%> (ø)

... and 27 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #48 (1bf2624) into develop (c04fad0) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop     #48      +/-   ##
==========================================
- Coverage     8.88%   8.50%   -0.39%
==========================================
  Files          143     149       +6
  Lines         2566    2682     +116
==========================================
  Hits           228     228
- Misses        2338    2454     +116

Flag
Coverage Δ

unittests
8.50% <0.00%> (-0.39%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../Etimo.Id.Api/Controllers/ApplicationController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/AuditLogController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/OAuthController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/RoleController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/ScopeController.cs
0.00% <ø> (ø)

src/Etimo.Id.Api/Controllers/UserController.cs
52.63% <ø> (ø)

src/Etimo.Id.Api/Startup.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdBaseClient.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdBootstrapper.cs
0.00% <0.00%> (ø)

src/Etimo.Id.Client/EtimoIdClient.cs
0.00% <0.00%> (ø)

... and 27 more

Continue to review full report at Codecov.

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


[REQUEST] Implement Unified File Extraction

Is your feature request related to a problem? Please describe.
Given the work that is landing / landed in #338, we are in the following state:

A UnifiedFileExtractor that is agnostic to the source of the files and the file reading methodology by introducing ReadableFile and FileSource concepts.
An S3Extractor, FileExtractor, and RemoteFileExtractor that are all subclasses of UnifiedFileExtractor that have no behavior changes aside from a specialized from_file_data method that consumes arguments.  These are effectively here for backwards compatibility.
No way to retrieve files from multiple different sources. I.e ingesting from multiple buckets, or a mix of remote and local files (for example)

Describe the solution you'd like
It seems like a win-win from both a user perspective and code maintenance perspective if we were to do the following:

Drop S3Extractor, FileExtractor, and RemoteFileExtractor in favor of directly utilizing the UnifiedFileExtractor class.
Change the FileSource concept to be pluggable like other aspects of nodestream.
Modify the UnifiedFileExtractor class to work a little something like this:

- implementation: nodestream.pipeline.extractors:UnifiedFileExtractor
  arguments:
      sources:
         - type: s3
            bucket: my-s3-bucket
            # ... Any other arguments
        - type: local
           globs:
              - foo/bar/baz.json

This would obviously constitute a breaking change when we drop S3Extractor,  FileExtractor, and RemoteFileExtractor.  However, it does seem that the rest can be implemented without a breaking change which is awesome.

This has been closed via #338


vaultwarden: fix the build for rustc 1.56

Motivation for this change
This fixes https://github.com/NixOS/nixpkgs/issues/146215
rustc 1.55 was ignoring vaultwarden's rust-version = 1.57, causing
the build to succeed. rustc 1.56 was erroring out because it does not
ignore the minimum rust-version.
Patch out the rust-version because we do not really need 1.57;
rustc 1.56 with RUSTC_BOOTSTRAP=1 produces a working vaultwarden.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

cc maintainer @msteen
It seems to build, but I still need to confirm that vaultwarden is working properly.

I have now tested this on my x86_64 NixOS 21.11-pre server, and vaultwarden seems to function properly, as before.


rbenv created wrong shim for a gem: command is named for the gem, not the gem's bin/* contents

For the xcode-install gem, the provided binaries are xcversion and 🎉:
% gem contents xcode-install
…
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/bin/xcversion
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/bin/🎉
…

But rbenv created an xcode-install command:
% ○ where xcode-install                                                                                /Users/cbowns/.rbenv/shims/xcode-install

which looks for a missing bin/xcode-install command:
% ○ xcode-install --help                                                                                2015-12-09 [10:51:58]
/Users/cbowns/.rbenv/versions/2.2.1/bin/xcode-install:23:in `load': cannot load such file -- /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/bin/xcode-install (LoadError)
	from /Users/cbowns/.rbenv/versions/2.2.1/bin/xcode-install:23:in `<main>'

(Discovered via https://github.com/neonichu/xcode-install/issues/106)
The shims:
% ○ cat /Users/cbowns/.rbenv/shims/xcode-install                                                                                2015-12-09 [10:53:39]
#!/usr/bin/env bash
set -e
[ -n "$RBENV_DEBUG" ] && set -x

program="${0##*/}"
if [ "$program" = "ruby" ]; then
  for arg; do
    case "$arg" in
    -e* | -- ) break ;;
    */* )
      if [ -f "$arg" ]; then
        export RBENV_DIR="${arg%/*}"
        break
      fi
      ;;
    esac
  done
fi

export RBENV_ROOT="/Users/cbowns/.rbenv"
exec "/usr/local/Cellar/rbenv/0.4.0/libexec/rbenv" exec "$program" "$@"

cbowns at Atlas in ~ ☆
% ○ cat /Users/cbowns/.rbenv/versions/2.2.1/bin/xcode-install                                                                                2015-12-09 [10:53:40]
#!/usr/bin/env ruby
#
# This file was generated by RubyGems.
#
# The application 'xcode-install' is installed as part of a gem, and
# this file is here to facilitate running it.
#

require 'rubygems'

version = ">= 0"

if ARGV.first
  str = ARGV.first
  str = str.dup.force_encoding("BINARY") if str.respond_to? :force_encoding
  if str =~ /\A_(.*)_\z/ and Gem::Version.correct?($1) then
    version = $1
    ARGV.shift
  end
end

gem 'xcode-install', version
load Gem.bin_path('xcode-install', 'xcode-install', version)

The gem's full contents:
% ○ gem contents xcode-install                                                                                2015-12-09 [10:48:41]
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/Gemfile
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/LICENSE.txt
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/README.md
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/Rakefile
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/bin/xcversion
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/bin/🎉
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/cleanup.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/cli.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/command.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/install.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/installed.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/list.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/select.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/selected.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/simulators.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/uninstall.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/update.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/lib/xcode/install/version.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/cli_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/curl_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150414.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150427.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150508.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150601.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150608.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150624.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/devcenter/xcode-20150909.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/hdiutil.plist
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/xcode.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/xcode_63.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/fixtures/yolo.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/install_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/installed_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/json_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/list_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/prerelease_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/spec/spec_helper.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.3/xcode-install.gemspec

This is being addressed in RubyGems. https://github.com/rubygems/rubygems/pull/1407

Yup, not an rbenv issue at all


Better odbc-bridge executable select

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

00411_long_accurate_number_comparison:                                   FAIL - Timeout!

Do you want to implement the same solution for clickhouse-clang and clickhouse-lld?


Feat/ Venue homepage: add decision tabs after post decision stage

Fixes #1602
Should we hide the 'Your Consoles' and 'Recent Acitivity' tabs after the post decision is run?

Works well with https://github.com/openreview/openreview-web/pull/1383

Should we hide the 'Your Consoles' and 'Recent Acitivity' tabs after the post decision is run?

I would keep "Your Consoles" because the camera ready revision could still be active.


Launchpad filtering

It's behaving slightly weirdly. For instance, in the screenshot below, Scratch shows up as what I'm selecting but nothing appears.

ah, it just needs an empty item to default to. It starts on Scratch, thus you can not select scratch

done... added to universe as well. Verify!

It still feels slightly weird because I think it should go back to filter when you select something. For instance,

Closed in favor of #39


Upgrade to breeze-client 2.0

Can this be upgraded to work along with breeze-client 2.0.
We need getEntityGraph to work with breeze-client 2.0 and Angular 8.
Thank you !

We use enableSaveQueuing, could that be upgraded as well?

I'm working on these now.

Added both getEntityGraph and enableSaveQueuing in breeze-client 2.0.1.
Please see the Release Notes for usage, because enabling these mixins is different now.
The big problem was that Production builds were tree-shaking the mixins away, so they had to be changed to prevent that.

Awesome. Now I can get rid of my custom getEntityGraph package and start using the breeze one again :)


Does not support unicode strings

Tested against https://raw.githubusercontent.com/akrylysov/python-json-benchmark/master/data/twitter.json
Traceback (most recent call last):
  File "json-algorithm/verifier.py", line 227, in <module>
    main()
  File "json-algorithm/verifier.py", line 83, in main
    state = parse_ch(cat, ch, stack, state, ds, ss, es, chart)
  File "json-algorithm/verifier.py", line 114, in parse_ch
    do_action(action, ch, ds, ss, es)
  File "json-algorithm/verifier.py", line 142, in do_action
    val = u"".join(ss)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe5 in position 0: ordinal not in range(128)

Hi @odeits, That is a gorgeous file!
TL;DR: The fix is to iterate fd.read().decode('utf-8') rather than fd.read()
I replicated your experiment quickly, including this error. To understand what's going on, here's a small explanation of the error.
First of all, put a print ss before the line 142 that crashes, run it again.
And you got the same error. Now, ASCII alphabet tends to be 7-bit codes. Python has places where it checks for this. When it hits one you get this kind of UnicodeDecodeError with "Ordinal not in range(128)". The problem is explained in the error message.
print (0xe5 not in range(128))

Those are utf-8 character points. To get python understand them and treat them as individual characters, you need to decode your utf-8 input file.

@cheery Thank you, this resolved the issue. Did you want to add an example of decoding a utf-8 file to the README or provide a test case?

@odeits I can give the verifier that functionality.
Going to look on the pretty printing during the weekend. I'll add the file decoding example during that, perhaps.
Btw. Are you going to write a blog post on benchmarking this?

I don't write a blog. I was just curios and messing around with it. I can
tell you that under pypy it has much better performance but under regular
cPython it is quite slow.
On Fri, Aug 26, 2016 at 3:22 AM Henri Tuhola<EMAIL_ADDRESS>wrote:

@odeits https://github.com/odeits I can give the verifier that
functionality.
Going to look on the pretty printing during the weekend. I'll add the file
decoding example during that, perhaps.
Btw. Are you going to write a blog post on benchmarking this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cheery/json-algorithm/issues/1#issuecomment-242649765,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA__srtZUJONYioPoUN8CsjKktpgR4u0ks5qjpQwgaJpZM4Jtffl
.


tabBarPosition is not working

[ ] I have searched existing issues
[ ] I am using the latest react native tab navigator version

Steps to Reproduce

Expected Behavior

Actual Behavior

I am equally having the same issue with the latest version of react native


feat(lanelet2_map_preprocessor): port lanelet2_map_preprocessor from ROS1 to ROS2

Related Issue(required)

part of
https://github.com/autowarefoundation/autoware.universe/issues/457
Description(required)

This ports lanelet2_map_preprocessor package into ROS2.
Review Procedure(required)
Run the following launch files and make sure that it works:

ros2 launch lanelet2_map_preprocessor transform_maps.launch.xml llt_map_path:=<input_lanelet2_map> pcd_map_path:=<input pointcloud_map> llt_output_path:=<output_lanelet2_map> pcd_output_path:=<output_pointcloud_map> x:=<translation value for x>
ros2 launch lanelet2_map_preprocessor fix_z_value_by_pcd.launch.xml  llt_map_path:=<input_lanelet2_map> pcd_map_path:= llt_output_path:=<output_lanelet2_map>`

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Read commit-guidelines
[ ] Assign PR to reviewer

If you are adding new package following items are required:

[ ] Documentation with description of the package is available
[ ] A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@kenji-miyake closed by #847?

@wep21 Oh, I didn't notice this. I'll rebase it and check the diff.

I'll merge this for Rolling/Humble cleanup.
If there are some problems left, please open a new pull request. :pray:


Add Czech translation

This PR adds a Czech translation and also fixes some minor translation issues in all other translations.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #169   +/-   ##
=======================================
  Coverage   86.17%   86.17%
=======================================
  Files           7        7
  Lines         123      123
  Branches       17       17
=======================================
  Hits          106      106
  Misses         17       17

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 8d2abb8...3a6c17e. Read the comment docs.

Many thanks for the contribution! ;]


Scenario Runner: watchdog timeout error

I am trying to execute a scenario using scenario_runner , however when i run it i am getting this error at random frames:
Watchdog exception - Timeout of 11.0 seconds occurred
I found out that the server gets stuck after calling .tick() from scenario_manager.py:
CarlaDataProvider.get_world().tick(10)
As you can see i've given timeout parameter as 10 seconds and yet i'm seeing this error.
I've integrated my AD client in external_control.py in the run_step() function. I hope the details are enough.
I am using carla version 0.9.10 and also tried with 0.9.11 but the issue still exists.
System configuration: Intel i7-9th gen, RTX2080 8gb, 32gb RAM, OS: Ubuntu 18.04
Note: The timeout is random, sometimes the execution completes successfully without this timeout error.
Synchronous mode step is 0.01hz (10ms)

Hey @NikhilKalghatgi, if the CARLA tick is causing a timeout, then the problem is server related. What happens to the server when that timeout happens. Is it frozen, shutdown, or anything of the sorts? If that is the case, as you said, the client will get stuck when trying to tick it.

@glopezdiest  Thank you for the response. Yeah the server gets frozen, wanted to understand why that happens. Is there any way to avoid it ?

You said you were using CARLA 0.9.10 and 0.9.11 but what version of ScenarioRunner are you using? What commands are you trying to execute exactly?

I am scenarioRunner version 0.9.11 . I am running scenario_runner.py with open scenario parser.

@glopezdiest Hi, on further digging I found that there's a deadlock in traffic manager tick in TrafficManagerLocal.cpp.
step_end_trigger.wait(lock, this { return step_end.load(); });
The above line is where its getting stuck.

Hey @NikhilKalghatgi. thanks for the input. I actually havent't seen that issue, which seems weird, but we'll investigate it regardless. tagging @jackbart94 too, he'll know more about the TM

Hey @NikhilKalghatgi. thanks for the input. I actually havent't seen that issue, which seems weird, but we'll investigate it regardless. tagging @jackbart94 too, he'll know more about the TM. Just to recap, you are using CARLA and SR 0.9.11, and are using the python scenario_runner.py --openscenario command?

Yup @glopezdiest that's right.


Use latest rustfmt-preview [ECR-1158]

See: https://jira.bf.local/browse/ECR-1158
Definition of Done

[ ] The continuous integration build passes

I think yes, we can fix any PR easily.


Stop panicking if a connection has multiple failures

Motivation
Currently, Zebra panics if a connection has multiple failures.
In #1610, we describe a complicated refactor that would stop Zebra having multiple failures on the same connection. But this refactor could introduce other bugs.
There's a much simpler solution that avoids the panics.
Suggested Solution

[ ] close both the inbound and outbound channels in fail_with

[ ] check how those closed channels are eventually handled (they should not panic, debug logs are ok)

[ ] log a warning, with a bug report link, or a request to report the bug

Related Work
We'll remove some kinds of connection errors in #2107
This ticket obsoletes #1610

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @jvff @oxarbitrage @teor2345 @upbqdn


Show the default model when listing all available models.

Show the current default model in the list of models in the list printed by llm models

Closing this in favor of #688. Exact same changes but the PR is made from my main branch. Somehow that fixes the failure in the run-cog GH action.


Seems to be a problem with This mod & IR

Error message or bug description
Im running a few mods, however when i try to add Loader Redulux, Industrial Revolution throws an error
I'm not 100% sure it's even your mod creating the problem, however it only happens when trying to add Loader Redulux
To Reproduce
seems the 2 mods work fine by there lonesome, but on added it to my mods it goops out / kills IR

Included is
the mod list im using,
A custom save 'BEFORE' adding Loader Redulux
An image of the error
As well as the Log file

Loader Redux only adds loader recipes to existing base technologies.
If other mods change the technology tree without porting ALL recipes in said technology that's hardly a fault in Loader Redux.


PARQUET-1882: Buffered Reads should allow for 0 length

Also, there doessn't semm like a great reason to crash when non-zero.

CC @pitrou

Looks good to me


Planes only update after window resize

Hi, I think I'm having a problem similar to issue #8, where the planes are not loading into the correct positions until there is a window resize. You mentioned in a comment that it was likely a CSS issue, "the curtains container CSS is loaded after the canvas has been appended."
What is your recommended fix for this?
Codesandbox: https://codesandbox.io/s/curtainsjs-experiment-ic51k4?file=/src/App.js

Hi @noelsner, did you find any solution for this problem and would like to share it?


Avoid direct navigation to a new page after inputting data into a testset.

We aim to ensure that a user doesn't lose the data they've entered into a new or existing testset.
To facilitate this, we intend to display a modal prompting the user to confirm whether they want to save the entered data or not.
If the user chooses to save, we'll store the data and redirect them to their intended destination

Here is an example of the issue:
Here I modify the testset but I forget to save my modification. I navigate somewhere else and my modification is lost.
https://github.com/Agenta-AI/agenta/assets/4510758/ae792d43-4a7b-4359-af69-0e9277261296

fixed in https://github.com/Agenta-AI/agenta/pull/264


ArchLinux - Failed to install

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

yay -S slides

Expected behavior
Slides is installed
Desktop (please complete the following information):

OS: Arch
Version Kernel: 6.1.14-1-lts

Additional context
Logs:
romarin ~ λ yay -S slides
:: Checking for conflicts...
:: Checking for inner conflicts...
[Aur:1]  slides-0.9.0-1

  1 slides                           (Build Files Exist)
...
:: (1/1) Parsing SRCINFO: slides
==> Making package: slides 0.9.0-1 (sáb 04 mar 2023 07:17:28)
==> Retrieving sources...
  -> Updating slides git repo...
==> Validating source files with sha256sums...
    slides ... Skipped
==> Making package: slides 0.9.0-1 (sáb 04 mar 2023 07:17:29)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Updating slides git repo...
==> Validating source files with sha256sums...
    slides ... Skipped
==> Removing existing $srcdir/ directory...
==> Extracting sources...
  -> Creating working copy of slides git repo...
Cloning into 'slides'...
done.
Switched to a new branch 'makepkg'
==> Sources are ready.
==> Making package: slides 0.9.0-1 (sáb 04 mar 2023 07:17:30)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> WARNING: Using existing $srcdir/ tree
==> Removing existing $pkgdir/ directory...
==> Starting build()...
go: clean -testcache cannot be used with package arguments
==> ERROR: A failure occurred in build().
    Aborting...
 -> error making: slides

Hey @rlopzc I packaged this on the AUR; I'll take a look at this.

Try it again; looks like the go cli changed what args are valid there.
I just updated the slides package, working on fixing slides-git:

https://aur.archlinux.org/cgit/aur.git/commit/?h=slides&id=abdf11e531b172a66207e1fb4e39791e221013e9

Ref:

https://github.com/golang/go/issues/53725
https://github.com/evanw/esbuild/commit/4e53c2783b460f79b01e3d7ea9455a97a69eb269

Ok, slides and slides-git are both fixed. slides-bin should be unaffected since it doesn't actually build the software.
Can you confirm you're able to install slides now with 0.9.0-2 on the AUR?

It is fixed!
I opened the slides commmand, is working, but the graphs look like this:

How can I fix them?

Awesome!
Afraid I'm not going to able to be much help with the graph issue since I don't really know what it's supposed to look like. A new issue should help find somebody who does know.


Entrez/Ensembl gene claims discrepancy

The v5 gene grouper provides considerably fewer gene claims from ensembl and ncbi/entrez-gene (on the order of an ~80% dropoff). We should check this out.

@jsstevenson per meeting, I think we should be good to go ahead and close this issue as solved?

Agreed


Apply log provider for specific log levels

Is your feature request related to a problem? Please describe.
Currently it is not possible to register a log provider only for a specific log level.
Example what NOT works:

Registering provider 1 for warning messages
Registering provider 2 for error messages

Describe the solution you'd like
Possibility to add log providers associated with a specific log level.
Additional context
Suggestion from the presentation in the company.

Implemented with PR #139


Feature: allow the overflow property to be given by the user

Proposed Feature or Bug
This would be similar to the display property (which is block by default, but can be overriden by the user using the options).
In practice it would useful to be able to specify overflow: hidden to avoid having that temporary scrollbar while the element is being shown (which causes a unpleasant jump in the text at the beginning / end of the animation).

For anyone else who runs into the problem of temporary scrollbar, until/unless this gets merged, it can be fixed (cross-browser) by this handy CSS snippet on whatever the element you are sliding
scrollbar-width: none;

&::-webkit-scrollbar {
  display: none;
}

I second this request.

Actually never mind, current code already checks the overflow status on the sliding element. I don't really understand why the default is auto but it doesn't really matter to me.

Yeah, why is overflow: auto the default anyway?
overflow-y: hidden !important is another crude fix.

Yeah, why is overflow: auto the default anyway?
overflow-y: hidden !important is another crude fix.

Yes, that is also good, assuming you do not have any potentially overflowing elements inside the collapsed block (various JS selectboxes, Tippy, etc)

The overflow value is now configurable in v2.3.0. Thanks for the patience on this!
https://github.com/alexmacarthur/slide-element/releases/tag/v2.3.0


Add regex filter to audit

I have a large number of legacy passwords that share a common formula that I'd like to clean up and bring able to use a regex during audit would help me track all of these passwords down.  I think this might be a fairly common problem for a lot of people who are just getting started using a password manager and are importing all of their legacy account accesses.

Sure that sounds useful and easy to do.
Could you provide a mock invocation so I better understand how you'd like to use the feature?

I was thinking something like this:
gopass audit --regex ".*_jeff_the_camel[1-9]{2}"
gopass audit -r ".*_jeff_the_camel[1-9]{2}"

Actually this can be done with grep already:
gopass grep --regexp ".*_jeff_the_camel[1-9]{2}"
Let me know if that works for you.

That works perfectly!  I'm currently running the following version
gopass 1.10.1-git+HEAD go1.13.15 linux amd64

Is the grep command added in the help pages in the current version? Unexpectedly, I can't seem to get v1.12.1 to build due to several os level packages that are claimed to not exist.
For example
# github.com/gopasspw/gopass/pkg/tempfile
pkg/tempfile/file.go:24:13: undefined: os.MkdirTemp
# github.com/gopasspw/gopass/internal/cache
internal/cache/disk.go:47:14: undefined: os.ReadFile
internal/cache/disk.go:57:9: undefined: os.WriteFile
# github.com/gopasspw/gopass/pkg/gopass/secrets/secparse
pkg/gopass/secrets/secparse/mime.go:45:33: hdr.Values undefined (type textproto.MIMEHeader has no field or method Values)
# github.com/gopasspw/gopass/internal/config
internal/config/io.go:69:14: undefined: os.ReadFile
internal/config/io.go:191:12: undefined: os.WriteFile
# github.com/gopasspw/gopass/pkg/otp
pkg/otp/otp.go:65:12: undefined: os.WriteFile
# github.com/gopasspw/gopass/internal/backend/storage/fs
internal/backend/storage/fs/fsck.go:141:13: undefined: os.ReadDir
internal/backend/storage/fs/store.go:42:9: undefined: os.ReadFile
internal/backend/storage/fs/store.go:58:9: undefined: os.WriteFile
# github.com/gopasspw/gopass/pkg/clipboard
pkg/clipboard/clipboard_others.go:42:18: undefined: os.ReadFile

That's probably because you don't have Go v1. 16

Currently we don't show the grep subcommand in the gopass help output.
Not sure why, we should change that.


Windows Defender removed PicoTorrent

After the 0.25 upgrade., Windows Defender removed PicoTorrent

Odd, I've never seen this... Is there any way to allow it to exist?

You need to send information about a false positive to Microsoft: https://www.microsoft.com/en-us/wdsi/filesubmission

@vktr Further down in his pic there should be an option to "take action" and one of the action is to allow it

Could be related to this? https://torrentfreak.com/utorrent-continues-to-be-flagged-as-severe-threat-and-its-not-alone-210318/


Add a missing 'to'

@GauthamBanasandra : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks.


Chromecast button not acivated

Have you read the Tutorials?
yes
Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
3.3.0
Please ask your question
I have integrated shaka-player in the android app using a web view.
when I try to use Chromecast the button doesn't appear on the UI.
I have set the Chromecast ID in the config and also linked script cast_sender.js in the index.html?
It works when I try from the browser but not from the android app web-view.
Can you please help me with this? thanks

Did you have time to see my question, please?

This could be a deficiency of WebView.  The Cast SDK communicates through private APIs to Chrome, which does the actual UDP and/or HTTP communication with the Chromecast devices on the network.  It is possible that WebView doesn't expose those APIs, whatever they are.  I don't know for sure, but this seems like the most likely cause.

Listing all Cast devices on the network would have to be a function of the Cast SDK.  We don't have any direct way to do that in the player.  Full Cast SDK docs are here: https://developers.google.com/cast/docs/reference/web_sender
@JulianDomingo, I haven't found a method to list receivers on the network, but I am still learning my way around the SDK.  Is there a method for that, or is that restricted to the browser UI for selecting a receiver?

Yes, it would be great to have this function.
I am also on my learning the SDK to have this possibility.
If we have something, we can discuss it here.
Thanks

As far as I can tell, device discovery is all handled inside the browser in C++, and isn't surfaced through JavaScript.  Sorry!
Since we don't have control over either of your issues in the player (WebView support for cast APIs, JS API for device discovery), I think we should close this issue.

Can you provide me the source where you find that the device discovery is all handled inside the browser in C++ please?
I found some sources that discover the Chromecast receiver
Get the list of ChromeCast devices (there is a 10 second delay for finding the devices on the local network): java -jar caster.jar -l
https://github.com/NoRulesJustFeels/Caster
https://github.com/NoRulesJustFeels/DIAL
And
For the WebView will it be possible to add an external cast button in the android app and cast the video from the shaka player?
What do you think about it?

Can you provide me the source where you find that the device discovery is all handled inside the browser in C++ please?

There is no API in the Cast SDK to expose that information in JS.  When you use the Cast SDK to start a session, the list of devices is shown in Chrome's native UI, so it must be handled internal to the browser in C++.

I found some sources that discover the Chromecast receiver
https://github.com/NoRulesJustFeels/Caster

Absolutely.  People have reverse-engineered the Cast protocol and can do mdns discovery from other environments.  Our testing tools for Chromecast rely on a node module called castv2 that can do this.  I meant that in a web app, the only way to do MDNS is through the browser, and the browser doesn't expose that capability to JavaScript.

For the WebView will it be possible to add an external cast button in the android app and cast the video from the shaka player?

I have no idea.  My guess is yes, but I wouldn't know where to start.  There is probably some way to do this in Java, and to expose that capability to JavaScript in some custom way through your app's WebView.  I don't have any Android app development experience, so this is just a guess.

Okay, thanks for the explanations :)


ERROR in Stack.ts - TS2564: Property 'parent' has no initializer and is not definitely assigned in the constructor.

Per the reports of this error from @SoutiRini, the<EMAIL_ADDRESS>version appears to have an error that was previously masked but has appeared due to an update to the typescript package which allowed make deeper type inferences for class inheritance that involves a super class declaring and managing instances of parent. When attempting to compile, the tsc compiler gives the following error messages:
webpack --config webpack.config.js --progress --colors
Version: webpack 3.11.0

ERROR in /Users/Rini/Documents/Synectic/synectic/src/core/Stack.ts
[tsl] ERROR in /Users/Rini/Documents/Synectic/synectic/src/core/Stack.ts(8,10)
TS2564: Property 'parent' has no initializer and is not definitely assigned in the constructor.

ERROR in /Users/Rini/Documents/Synectic/synectic/src/core/Stack.ts
[tsl] ERROR in /Users/Rini/Documents/Synectic/synectic/src/core/Stack.ts(9,10)
TS2564: Property 'children' has no initializer and is not definitely assigned in the constructor.

The following screenshots demonstrate these errors:

Pulled latest commit < c18e0a9 > from branch API_architecture. The bug is resolved and Syntectic runs without problems.


(maint) Run package resource acceptance in CI - 5.5.x branch

CLA signed by all contributors.

This has been tested and is ready to merge.
Before this, the RPM multiversion test only ran on EL 7 (because that's the only platform where multiple kernel versions were available in the package manager). For this purpose a local repo was created, with noarch packages, and now the test runs on all suitable EL platforms.

passing tests run with
bundle exec rake ci:test:git  HOSTS=debian8-64a-redhat8-64a SHA=maint/audit-high-5.5.x RUNTIME_BRANCH=5.5.x FORK=GabrielNagy TESTS='tests/provider/package/dnfmodule_ensure_versionable.rb,tests/provider/package/dnfmodule_manages_flavors.rb,tests/provider/package/dpkg_ensure_held_package_should_preserve_version.rb,tests/provider/package/dpkg_hold_true_package_is_latest.rb,tests/provider/package/dpkg_hold_true_should_preserve_version.rb,tests/provider/package/rpm_ensure_install_multiversion_package.rb'


code issues fixed

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/OfficeDocs-Exchange-Test-pr.fr-fr, working directory: W:\wwq1-s, target branch: code-headers, target branch commit: 8f9ac8fd0880f0880f6c6b9a53808f4a4df4bdf6, source branch: 2c33366d867fb8878bfa182ea9b333d5/code-headers, source branch commit: .

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.

Conflicts.  Asked to close this in favor of new PR.


[FEATURE] Numeric input in sliders

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
A numeric input box that reflects the actual value for each slider to type in the exact desired value.
At the moment the sliders are a little too "reactive"
Describe alternatives you've considered
Additional context

Yup, although low priority for now.
Perhaps good first issue for new contributors.


Fixed: Unchecked call to Long.valueOf(String) may cause an NFE

Hello Aoki,
I've found that if the Uri passed into CropImageView.load(Uri) looks like content://com.android.providers.downloads.documents/document/raw%3A%2Fstorage%2Femulated%2F0%2FDownload%2Fsome_file, then a NumberFormatException will be thrown, which is caused by passing the unchecked raw:/storage/emulated/0/Download/some_file to Long.valueOf(String).
I've added an if-else statement in class com.isseiaoki.simplecropview.util.Util to fix this problem.
Anyway, SimpleCropView is a great work. It has saved me a lot of time on handling image cropping things.
Cheers
Makito

Sorry for late response.
Thank you for your contribute!
I will release new version soon!


Fix TLS tunneling

This fixes some of autest failures that are tracked on https://github.com/apache/trafficserver/issues/10999.
I'm not sure if it was broken on master/10.0 or it has been broken since 9.x. The 9.2.x branch has less tests, but autest passes even with BoringSSL on the branch.

Cherry-picked to v10.0.x


Guidelines for lyrics module

From andrew.hankinson on September 08, 2011 10:59:29
Placeholder issue for lyrics module guidelines
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=22

From andrew.hankinson on September 08, 2011 07:59:47
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on May 14, 2012 10:24:21
Draft completed.
Status: Started

From<EMAIL_ADDRESS>on May 24, 2012 07:30:05
Status: Needs-Review

From<EMAIL_ADDRESS>on September 17, 2012 12:39:54
Status: Fixed


⚠ ClamAV has degraded performance

In 044bd3d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 389 ms

Resolved: ClamAV performance has improved in 137a82f.


Outsource data channels

Disband tight interaction between the task and WebRTC data channels
DataChannel instances are no longer handled by the task. The application has to take care of managing data channels itself. If the application wants to leverage the established SaltyRTC session keys, it can use a DataChannelCryptoContext instance to encrypt and decrypt data.
Furthermore, if the application wants to hand over the signalling channel to a dedicated data channel, that data channel must also be managed by the application itself while the SignalingTransport will take care of the crypto and the chunking. Concretely, the application can request a ISignalingTransportLink instance which contains all necessary information to create a data channel for the purpose of handing over the signalling transport. The application then has to provide an implementation of a ISignalingTransportHandler towards the task. As a rule of thumb, the ISignalingTransportLink handles all incoming messages and events forward by the application while the ISignalingTransportHandler handles all outgoing messages and events forwarded by the task.
Resolves #1
Resolves #26
To do:

[x] Define new API
[x] Update tests
[x] Add new tests
[x] Update readme/docs

API documentation
Testing
Coding guidelines, ...

[x] Ensure compatibility with legacy v0

Update the demo for that purpose: saltyrtc/saltyrtc-demo#6

@lgrahl what's the status on this?

Leave it open for now. I was in the middle of writing the benchmark demo in the SaltyRTC demo project before merging this. Need to continue on that soonish.

The only blocker here is the discussion regarding the guide in the readme.