bug: the latest commit keys first trigger not work

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the exsiting issues of plugins related to this issue

Neovim version (nvim -v)
0.8.2
Operating system/version
arch 6.1.9
Describe the bug

Steps To Reproduce

update lazy
add config

  {
    "numToStr/Comment.nvim",
    keys = {
      { "gc", mode = { "n", "v", "x" } },
      { "gb", mode = { "n", "v", "x" } },
      {
        "<leader>/",
        function()
          require("Comment.api").toggle.linewise.current()
        end,
        mode = "n",
        desc = "Comment",
      },
      {
        "<leader>/",
        "<Plug>(comment_toggle_linewise_visual)",
        mode = "v",
        desc = "Comment toggle linewise (visual)",
      },
    },
    config = function()
      local pre_hook
      local ts_comment = require("ts_context_commentstring.integrations.comment_nvim")
      pre_hook = ts_comment.create_pre_hook()

      require("Comment").setup({
        ---Add a space b/w comment and the line
        ---@type boolean
        padding = true,

        ---Whether cursor should stay at the
        ---same position. Only works in NORMAL
        ---mode mappings
        sticky = true,

        ---Lines to be ignored while comment/uncomment.
        ---Could be a regex string or a function that returns a regex string.
        ---Example: Use '^$' to ignore empty lines
        ---@type string|function
        ignore = "^$",

        ---Whether to create basic (operator-pending) and extra mappings for NORMAL/VISUAL mode
        ---@type table
        mappings = {
          ---operator-pending mapping
          ---Includes `gcc`, `gcb`, `gc[count]{motion}` and `gb[count]{motion}`
          basic = true,
          ---Extra mapping
          ---Includes `gco`, `gcO`, `gcA`
          extra = true,
        },

        ---LHS of line and block comment toggle mapping in NORMAL/VISUAL mode
        ---@type table
        toggler = {
          ---line-comment toggle
          line = "gcc",
          ---block-comment toggle
          block = "gbc",
        },

        ---LHS of line and block comment operator-mode mapping in NORMAL/VISUAL mode
        ---@type table
        opleader = {
          ---line-comment opfunc mapping
          line = "gc",
          ---block-comment opfunc mapping
          block = "gb",
        },

        ---LHS of extra mappings
        ---@type table
        extra = {
          ---Add comment on the line above
          above = "gcO",
          ---Add comment on the line below
          below = "gco",
          ---Add comment at the end of line
          eol = "gcA",
        },

        ---Pre-hook, called before commenting the line
        ---@type function|nil
        pre_hook = pre_hook,

        ---Post-hook, called after commenting is done
        ---@type function|nil
        post_hook = nil,
      })
    end,
  },

Expected Behavior
expect first trigger key work
Repro

For what key? Can you please add the steps/keys to type that don't seem to work for you?

Also, please provide a minimal repro as instructed in the issue template

repro:
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  {
    "numToStr/Comment.nvim",
    keys = {
      { "gc", mode = { "n", "v", "x" } },
      { "gb", mode = { "n", "v", "x" } },
      {
        "<leader>/",
        function()
          require("Comment.api").toggle.linewise.current()
        end,
        mode = "n",
        desc = "Comment",
      },
      {
        "<leader>/",
        "<Plug>(comment_toggle_linewise_visual)",
        mode = "v",
        desc = "Comment toggle linewise (visual)",
      },
    },
    config = function()
      require("Comment").setup()
    end,
  },
}

require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")

Works on nightly, but not on stable. Will look into it

Sorry for that.
All  keymaps added in keys will not take effect on the first press.

You need to add gcc separately since that keymap is a normal keymap created by comment.nvim

Thanks, work now

I'm facing weird behaviours after ddaffa07156a090383bd32ef88669eea1b22c11a:
I'm posting here because they're kinda related but I can open a new issue if necessary.

repro.lua
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify('./.repro', ':p')

-- set stdpaths to use .repro
for _, name in ipairs { 'config', 'data', 'state', 'cache' } do
vim.env[('XDG_%s_HOME'):format(name:upper())] = root .. '/' .. name
end

-- bootstrap lazy
local lazypath = root .. '/plugins/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
vim.fn.system {
  'git',
  'clone',
  '--filter=blob:none',
  '--single-branch',
  'https://github.com/folke/lazy.nvim.git',
  lazypath,
}
end
vim.opt.runtimepath:prepend(lazypath)

vim.opt.number = true
vim.opt.relativenumber = true
vim.opt.scrolloff = 16

local plugins = {
{
  'folke/tokyonight.nvim',
  lazy = false,
  config = function()
    vim.cmd.colorscheme 'tokyonight'
  end,
},
{
  'nvim-neo-tree/neo-tree.nvim',
  dependencies = {
    { 'MunifTanjim/nui.nvim' },
    { 'nvim-lua/plenary.nvim' },
  },
  cmd = { 'Neotree' },
  keys = {
    {
  '<C-p>',
  '<cmd>Neotree toggle<cr>',
  mode = { 'n' },
    },
  },
  config = function()
    require('neo-tree').setup {}
  end,
},
{
  'akinsho/toggleterm.nvim',
  keys = {
    {
  [[<C-\>]],
  function()
    require('toggleterm').toggle(vim.v.count)
  end,
  mode = { 'n' },
    },

    {
  '<leader>tt',
  function()
    require('toggleterm').toggle_all(true)
  end,
  mode = { 'n' },
    },
  },
  config = function()
    local size = {
  horizontal = 15,
  vertical = vim.o.columns * 0.4,
    }

    require('toggleterm').setup {
  size = function(term)
    return size[term.direction]
  end,
  shade_terminals = false,
    }
  end,
},
{
  'mg979/vim-visual-multi',
  keys = {
    { '<C-n>', mode = { 'n' } },
  },
}
}

require('lazy').setup(plugins, {
root = root .. '/plugins',
defaults = {
  lazy = true,
}
})

toggleterm and neo-tree needs to be pressed twice in order to trigger but vim-visual-multi works regardless.
vice-versa on 4272d2100af2384f4b8aba08aef4a7b9a296bce6
https://user-images.githubusercontent.com/95099980/217541563-726576a7-9d7c-45d0-a26d-aeb1d98d49a8.mov

Awesome! That seems to have fixed it. Do you know what cause the change between master and stable?

Couldn't find the exact cause, but it has something to do with keymaps returning an expression for another keymap that has been created while evaluating the keymap :)
It seems as if on stable, that new keymap is not known.
But I found another way to work around this and make it work.
The new approach (as the one that broke) is much simpler than before, but it's always tricky to make changes to the keys handler due to all the quirks related to Neovim keymaps.


Dual Annealing Algorithm evolution

It is already possible to add a call back in the dual_annealing function to see how the optimization goes. But it is only possible to get minimum and function value at the minimum.
Would it be possible to add more parameters to the callback function, at least the iteration number in order to plot the evolution with respect to optimization time.
It would be really useful to get these informations to be able to plot the path taken  by the algorithm.

Seems like a reasonable request, however the signature of callable isn't easily extensible without breaking backwards compatibility. Unfortunately this is also true for other optimizers - not sure we discussed that before somewhere.
In this case, you could keep track of the iteration number yourself, because there'll be exactly one call to callback each iteration so callback could increment some counter.
@sgubianpm any other thoughts?

Thank you for the prompt answer.
Yes I thought about that but If I want to know if the new energy was accepted or not it would be easier since parameter calibration for this kind of algorithm is quite important.
Also something more important: I think the default parameter 'accept' should be above 1.0 and not negative. Otherwise the algorithm never choses states with a higher energy.

Also something more important: I think the default parameter 'accept' should be above 1.0 and not negative. Otherwise the algorithm never choses states with a higher energy.

This is unrelated to the callback request, and I'm not sure about what the right response is. Could you open a separate issue for this, with an example that shows the problem?

Seems like a reasonable request, however the signature of callable isn't easily extensible without breaking backwards compatibility. Unfortunately this is also true for other optimizers - not sure we discussed that before somewhere.
In this case, you could keep track of the iteration number yourself, because there'll be exactly one call to callback each iteration so callback could increment some counter.
@sgubianpm any other thoughts?

@echo5india, @rgommers
The callback is not working the same way for each global optimisers (but is consistant for local minimisers). If I am not mistaken,  for minimize function, as @rgommers mentioned, it is called at the end of each iteration (as well as for global optimshgo). For other global optimisers, basin_hopping and dual_annealing, callback will be called for a new minimum found (with 3d argument that as a different meaning), differential_evolution has a different signature.
So, maybe there would a need for aligning all signatures with more flexibility on the arguments (adding optional args or kargs?)

There's certainly scope for an improved callback system which, amongst other things, could return an intermediate result. THis intermediate result would contain iteration number, current best solution/energy, etc. However, it's difficult to do without breaking back compatibility, or increasing complexity. At the moment the best way to achieve what you want to do (and arguably cleaner as well) it to create something along these lines:
class Callback(object):
    def __init__(self, fun):
        self.x_history = []
        self.fun = fun

    def __call__(self, x):
        self.x_history.append(x)
        # unfortunately this calls the user function again
        self.fun_history = self.fun(x)


[Feature] sc-29090 Add elapsed-milli field for table in the run.json

Signed-off-by: Kent Huang<EMAIL_ADDRESS>
PR checklist

[x] Ensure you have added or ran the appropriate tests for your PR.
[x] DCO signed

What type of PR is this?
Feature
What this PR does / why we need it:

Add elaspsed-milli & profile_duration on table level in the run.json file

Which issue(s) this PR fixes:
sc-29090
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@kentwelcome
Approved. Feel free to merge after conflict resolved.


clarifications to the English in explanations

@Tranquilled: Thanks again for your contribution! I'm closing this PR for now, since it's been a little while, but please feel free to reopen it.


the frame resolution when evaluating on TAP-Vid

Hi, thanks for your excellent open-source repositories.
I'd like to confirm that when testing on tap-vid, all the data pre-processing processes such as extracting optical flow, dino features, etc. are done on 256x256 video frames. Is that right?

yes, everything is done on 256x256 video frames.


Status Page not Red despite having increased risk in second space

Avoid duplicates

[ x] Bug is not mentioned in the FAQ
[x] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[ x] Bug is not already reported in another issue (that I've seen)

Technical details

Device name: Poco X3 NFC 128GB
Android version: 11
App version: 2.15.1

Describe the bug

My smartphone has a "second space" mode, where I can add a separate space for eg work (kinda like a second account, but still different).
In the second space, the Exposure logging doesnt work. ("Covid-19 Exposure Notifications System is not available on your ....").
However, thats not the bug.
The Problem is, that I still use the CWA to log in into Restaurants.
Then, sometimes I get an notification from the CWA.
If i go to the CWA I can see from the Journal Tab, that I have increased Risk based on my Restaurant visit.
In my "Status" page I can see also, that I have an increased Risk. However, the is not red how it should be and as it is with my partners phone.
I suspect, that this problem arises because I dont have exposure logging active.
However, Im also not quite sure if it is intended behaviour.
Steps to reproduce the issue

Expected behaviour
I expected the Status Page to turn Red as I have an increased risk status.

Possible Fix

Additional context
This could cause more problems, because in some Bundesländer you can make an PCR with a "red CWA app warning".
In my case, it is not completely clear if i am elidigble to make an PCR test or not.

See also: https://github.com/corona-warn-app/cwa-documentation/issues/773

ok, I didn't find that. I guess my Issue can be closed then. Thanks!


Fix keyboard staying as email when switching between default and email

Summary
Right now, when we change the keyboardType on android between between default and email, the value keyboard type stays as email (specially noticeable with the key next to the spacebar, that changes between the comma (,) to the at sign (@)).
This is because the mask we are using when updating the input is only taking into account the class, and not the flags nor the variations.
We don't apply all masks because it may interfere with flags assigned by other props, like multiline or secure text entry. Therefore, we have created our own mask, taking into account all the variations and flags that the keyboardType prop may set. This may be hard to maintain, since whenever we add any other keyboard type, we have to take these flags into mind.
The error I was trying to fix was in particular regarding going back and forward from email, but this fix may solve other similar issues with other keyboard styles.
Changelog
[Android] [Fixed] - Fix a bug where the keyboard, once set as email, won't change back to default.
Test Plan
In order to test this PR, any test code with a TextInput, and a way to change the value of the keyboardType should work. We should be able to see how the keyboard changes to the correct type without staying, for example, on the email state.

Some tests are failing, but looking at the CI seem unrelated to the PR. Not sure if you want me to proceed in any particular way.

Some tests are failing, but looking at the CI seem unrelated to the PR. Not sure if you want me to proceed in any particular way.

Could you rebase this commit past https://github.com/facebook/react-native/pull/33930? The test failures you are seeing should be fixed then.

@makovkastar Merged and passing the tests


Screen Rendering Issues

System Information: ZX Spectrum Next
Raspberry Pi Accelerator Present: Yes
RTC: Yes
Wifi: Yes
Memory: 2MB
Firmware: 1.2.9b
Core: 3.01.5
Graphics Mode: 7/60Hz/scan*2
Display Connection Type: HDMI
Display: Samsung T24E390
MicroSD Card: Default one issued with pack
Other information: MicroSD connected via extension cable.  ActiveConsult control pad connected.
Actions to reproduce error:

Format MicroSD with SD Association SD Card Formatter
Copy contents of of sn-complete-v.1.3.2.zip to SD card
Create apps folder on root of MicroSD
Copy CalmComander.nex to apps folder
Boot ZX Spectrum Next, holding down D
Choose screen mode of 7/60Hz/scan*2
Clock through first boot display
Choose Browser
Chose Apps->CalmComander.nex

The screen is rendered incorrectly.  The menu is missing on the display:

Workaround: Set the screen mode to 7/50Hz/scan*2.  This will render the screen correctly, although some displays may not allow this refresh rate and make this workaround unusable.

Hi, but this is problem monitor, right?
maybe I can switch views to another view. I try it and write you and send test file, ok?

Each monitor can have different "overscan" area. There's .displayedge tool in distro so the user can set the visible frame in each mode and store that in /sys/env.cfg file, then any other tool can hypothetically read the config, and adjust display output, i.e. you need to write CC in such way, that you can display at 80x32 or 76x24 (or 80x42 with the 8x6px tiles :) ) ...
... but that's not 1-2 line change, especially if you don't plan for it from start, so I guess for the moment 50Hz mode + good display is the answer.

I'm sure any decent Next SW should support this and read the displayedge config, but the "calm commander" total development time is still counted rather in days, so I guess this kind of problems are still acceptable... maybe after few years when it will be in "2.0" and the display will be still hard 80x32, I will raise my eyebrow onto Shrek, wtf is he thinking... :)

Now I'm working on disk operations and other functions, ... then I'll try to solve this problem - if I can simulate it on my monitor. ;)

Now I'm working on disk operations and other functions, ... then I'll try to solve this problem - if I can simulate it on my monitor. ;)

You should be able to simulate a vastly under-scanned area on your display via the displayedge config and get it working on your setup that way, then I can test on mine @60Hz.


Declare dependency on copyFlutterAssetsTask in bundleAarTask

Fixes https://github.com/flutter/flutter/issues/129471.
Does so by doing a very similar thing that is done by https://github.com/flutter/flutter/pull/121958, but for a task that is specific to the add to app build process.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@bartekpacia I'm not able to request you as a reviewer for some reason, but your input is always welcome on gradle related PRs!

Thanks for pinging me @gmackall!
This LGTM.
Workarounds like this aren't perfect (see #123934 for more details) but I can image there are many much more important things to do:)

Agreed, the pattern of "find out there is a problem with a previously unknown task" -> "guess the task name and add some logic to resolve" is an unfortunate one. Thanks for documenting those issues!


Individual element commands.

With store_var we should store an element and apply individual commands on them.

#52 In process feature will be covered for all functions.


jenkinsfiles: Don't display nulls in current build display name

Signed-off-by: Tom Payne<EMAIL_ADDRESS>

Note; this will have to be backported if we want it to apply where the jobs are currently displaying null values (which is older stable branches).


Ext not opening.

Wallet still works with dapps but I can not open wallet on its own. It spins and does whats shown in the pic. Brave and ext are up to date.

@cryptoPaulie It sounds like your query would be best dealt with by the support team.
Please see the link below to our dedicated support line:
Support :Cardano Live Support
Note: Click on the live chat icon at the bottom corner of the page to start a conversation

Have you able to resolved your issues? @cryptoPaulie

BEWARE !!! SCAM !!!

Same!!

same

im done the issue,


Test data should not have emails and should use openfga/openfga as a refrence repo

Users in tests such as here should drop the @auth0.com email (generally we should avoid using emails in test data)
Repositories in tests can be switched to repository:openfga/openfga?

@rhamzeh - I can take this chore/issue 👍

Thanks for taking this on @dblclik!


Add "ctl" command to coordinator-client

Make it easier to interact with the ceremony in an ad hoc way. E.g.,
./coordinator-client ctl --path /foo --method get

Include an "http-auth" command to generate Authorization header to facilitate
using other tools. E.g.,
curl -H "Authorization: $(./coordinator-client http-auth --path /foo --method GET)" \
  -X GET \
  http://localhost/foo

I describe a more interesting example usage in https://github.com/celo-org/snark-setup-coordinator/pull/21


Dynamic assertions + permission issue

The documentation about dynamic assertion says:

In certain situations simply checking a permission key for access may not be enough. For example, assume two users, Foo and Bar, both have article.edit permission. What's to stop Bar from editing Foo's articles? The answer is dynamic assertions which allow you to specify extra runtime credentials that must pass for access to be granted.

From this description I would expect that the "extra runtime credentials" will be checked in parallel with the credentials but it seems that the actual implementation just bypass it.
According to the use case described before, I provided here a unit test to prove it. Basically, if Foo doesn't have the permission on article.edit and if the dymanic assertion is true the result is true (but should be false).
To fix this behaviour, in my opinion the dynamic assertion should be matched in AND with the permission. I provided a gist here.

@bakura10, @spiffyjr what do you think about this issue? /cc @weierophinney

I wrote the text below to zf-devteam on 21 Nov 2016 and @ezimuel replied. Unfortunately, I haven't had time to open a proper issue. I can now also provide a pull request, see #21.
There are two mentions of dynamic assertions in the documentation:

The answer is dynamic assertions which allow you to specify
extra runtime credentials that must pass for access to be granted.

https://github.com/zendframework/zend-permissions-rbac/blob/develop/doc/book/intro.md#dynamic-assertions
And:

If $assert is provided and either an AssertInterface instance or callable,
it will be queried before checking against the given role.

https://github.com/zendframework/zend-permissions-rbac/blob/develop/doc/book/methods.md#zendpermissionsrbacrbac
This conveys the impression, that somehow, the assertion and the
permission is checked. However, the implementation immediately returns
the result of the assertion without checking any roles for the
permission.
Why?
Actually, the $role and $permission parameters could be null, if the
$assert parameter is provided. The documentation mentions "extra
runtime credentials"  and "queried before checking", so the actual
behaviour is very counterintuitive.

Hi,
Zend permissions Rbac is a bit broken and I'm not sure it's maintained anymore.
All the efforts has been put in ZfcRbac for some years now: https://github.com/ZF-Commons/zfc-rbac

@bakura10 what do you mean with "is a bit broken"? The component is still maintained by us, Zend Framework Team.
Do you have something so say about this issue? If not, thanks anyway.

I mean that the implementation was quite bad, inefficient and unflexible. I'm not sure many people are using this directly (as it is only a base implementation but require a higher level library like ZfcRbac). But ZfcRbac has not been based on this from many years now, but from another library I've developped in the past whose goal was to replaced this implementation.
I don't know how this library work unfortunately as I never really used it once I've found out it was too limited :(.

@bakura10 I tried to be professional in my previous comment but evidently you missed the point. If you don't have something to say related to the issue please don't say nothing. You cannot just say that sucks and suggest to move to a different project, especially if maintained by you, not really diplomatic. Even because your project started from this one. Don't forget where your bread's buttered, @bakura.

Admin note: deleted unrelated comments so we can keep discussion focused on the reported issue.

@weierophinney Note: I haven't been doing much PHP for over a year but, if I recall correctly, the implementation by @ezimuel is actually correct. Dynamic assertion is a horrible name, but I digress. The result of the permission check AND the assertion should be valid if the assertion is provided.

I'm trying to use zend-permissions-rbac in my project, but found this problem, too. Hope this will be fixed soon.

@spiffyjr thanks for your feedback. I'm going to merge #21.


Trouble using Nan::NewBuffer

Hi, I apologize if this ends up being a duplicate or dumb issue, but I haven't been able to find the info. I'm looking for.
I'm trying to create a binding for a C library, though I really only need one function from it.  The function I've written in my .cc file should return a node Buffer off the output from the C function,  however I can't seem to get Nan::NewBuffer to work.
In the .cc file I have:
unsigned int len;
unsigned int len;
......
if (!RSA_sign(NID_sha1, token, node::Buffer::Length(args[1]), sig, &len, rsa)) {
    Nan::ThrowTypeError("failed to sign token");
    return;
}
args.GetReturnValue().Set(Nan::MaybeLocal<v8::Object> (Nan::NewBuffer((const char*)sig, len)));

node-gyp erros on the Nan::NewBuffer call with:
CXX(target) Release/obj.target/binding/binding/binding.o
../binding/binding.cc:63:58: error: no matching function for call to
      'NewBuffer'
  args.GetReturnValue().Set(Nan::MaybeLocal<v8::Object> (Nan::NewBuffer(...
                                                         ^~~~~~~~~~~~~~

Along with some other errors.
Any help would be appreciated :smiley:

NewBuffer() returns a MaybeLocal<v8::Object> that you have to unwrap first with .ToLocal() or .ToLocalChecked().

unsigned int len;
unsigned int len;
......
if (!RSA_sign(NID_sha1, token, node::Buffer::Length(args[1]), sig, &len, rsa)) {
    return Nan::ThrowTypeError("failed to sign token");
}
args.GetReturnValue().Set(Nan::NewBuffer(reinterpret_cast<const char*>(sig), len).ToLocalChecked());

Note that NewBuffer takes ownership of the pointer and will call free on it when garbage collection occurs.

Thanks for the quick responses!
I am still getting the `no matching function for call to 'NewBuffer' error with both of your suggestions.

Then something is badly wrong in your code or environment. Can't tell from what you posted.

Ah, now I see what it is.  Use CopyBuffer or a non-const buffer.

What other information do we need to figure out what is wrong?  I think it may be related to the fact that I'm using a module from npm which make use of<EMAIL_ADDRESS>I am running node v0.12, and<EMAIL_ADDRESS>Here is the entire .cc file:
#include <stdio.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>

#include <node.h>
#include <v8.h>
#include <nan.h>

#define RSA_verify RSA_verify_mincrypt
#include "rsa.h"
#undef RSA_verify

#include <openssl/evp.h>
#include <openssl/objects.h>
#include <openssl/pem.h>
#include <openssl/rsa.h>
#include <openssl/sha.h>

#include "binding.h"
#include <errno.h>
#include <string.h>
#define MAX_PAYLOAD 4096

void Sign(const Nan::FunctionCallbackInfo<v8::Value>& args) {

  if (args.Length() < 2) {
    Nan::ThrowTypeError("Wrong number of arguments");
    return;
  }

  FILE *fd;
  unsigned int len;
  const char* key_path;
  const unsigned char* token;

  key_path = (const char*)node::Buffer::Data(args[0]);
  token = (unsigned char*)node::Buffer::Data(args[1]);

  unsigned char sig[MAX_PAYLOAD];

  RSA *rsa = RSA_new();
  fd = fopen(key_path, "r");
  if (!fd) {
    Nan::ThrowTypeError("fopen() failed");
    return;
  }
  if (!PEM_read_RSAPrivateKey(fd, &rsa, NULL, NULL)) {
    fclose(fd);
    RSA_free(rsa);
    Nan::ThrowTypeError("couldn't read private key");
    return;
  }
  if (!RSA_sign(NID_sha1, token, node::Buffer::Length(args[1]), sig, &len, rsa)) {
    Nan::ThrowTypeError("failed to sign token");
  }

  args.GetReturnValue().Set(Nan::NewBuffer(reinterpret_cast<const char*>(sig), len).ToLocalChecked());
}

void Init(v8::Local<v8::Object> exports) {
  exports->Set(Nan::New("sign").ToLocalChecked(),
              Nan::New<v8::FunctionTemplate>(Sign)->GetFunction());
}

NODE_MODULE(binding, Init)

Yes, you want CopyBuffer instead of NewBuffer, since you allocated sig on the stack and not using a method (like malloc()) that requires using free().

Perfect, got it working.  Thanks! :smiley:


other summs?

Can you add other summoner spells too please?

sorry I'm afraid not cyz I'm busy recently. Also I'm no longer playing league, so...
But if anyone is interested in this feature, please feel free to create a pull request, I'll approve it asap!
Thanks


Route not work in production server with Rails

I use ruby on rails in development I have a trouble with route can't find views then solved with gem "angular-rails-templates". It's woking well. Since I deploy application I got empty page no error both server(log) and browser(inspector). If anyone used to work with rails and angular I posted the other information in gem "angular-rails-templates" repository here https://github.com/pitr/angular-rails-templates/issues/155 and Help me find out please
My version of UI-Router is: "0.2.18" and In my bower I put "angular-ui-router": "~0.2"

Update: I found actually problem, A real problem. if some state have resolve function it will be freeze ui. I don't know why because I never met before. I working with resolve many time only error form server will make resolve and ui will freeze. I ran "rails s -e production" is work normal but on real server not working


Add twoslash examples to the interfaces page

Goes through adding twoslash to the code samples.

So, how do you do these? First clone the repo, use node 13
git clone https://github.com/microsoft/TypeScript-website
cd TypeScript-website
yarn install
yarn bootstrap
code .
yarn start

Wait for the site to boot up, this might take a bit of time
Pick a page from the handbook
Jump to the first ts code sample and add the word "twoslash" after - go back to the CLI, did it complete? Do this and repeat.
If you can't figure something out, you don't need to translate them - you can ask in a PR or skip it
twoslash docs are here

Been wanting to dig into TS for a while now. Mind if I pick this one up?

Sure, grab any handbook page! Feel free to comment in here to claim the page

I'd like to take Advanced Types 🙂


PCA fails for large dataset

I adapted the pipeline for use with a scRNAseq dataset with ~50000 cells. The input for PCA is the normalized expression matrix.
! Disclaimer: I edited the pipeline, but the PCA is at the very beginning, so this should be the same. However, I may have messed something up. !
 for hours and then fails. It worked fine for a dataset of 500 cells.
Here is an example of a log file:
Building DAG of jobs...
Your conda installation is not configured to use strict channel priorities. This is however crucial for having robust and correct environments (for details, see https://conda-forge.org/docs/user/tipsandtricks.html). Please consider to configure strict priorities by executing 'conda config --set channel_priority strict'.
Using shell: /usr/bin/bash
Provided cores: 2
Rules claiming more threads will be scaled down.
Provided resources: disk_mb=18545, disk_mib=17686
Select jobs to execute...

[Wed Oct 18 08:59:38 2023]
rule pca:
    input: /data/path/normalized_expr_data.csv, /data/path/obs_metadata.csv
    output: /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_object.pickle, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_data.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_data_small.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_loadings.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_loadings_small.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_var.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_axes.csv
    log: logs/rules/PCA_sample_name_default_2.log
    jobid: 0
    reason: Missing output files: /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_var.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_loadings_small.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_loadings.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_axes.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_data_small.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_data.csv, /results/path/unsupervised_analysis/sample_name/PCA/PCA_default_2_object.pickle
    wildcards: sample=sample_name, parameters=default_2
    threads: 2
    resources: mem_mb=400000, disk_mb=18545, disk_mib=17686, tmpdir=/tmp

python -c "from __future__ import print_function; import sys, json; print(json.dumps([sys.version_info.major, sys.version_info.minor]))"
Activating conda environment: /path/to/conda/env/snakemake_conda/40431c23e3640492480b1b2b0c8d33df_
python /path/to/pipeline/.snakemake/scripts/tmp83o38501.pca.py
Activating conda environment: /path/to/conda/env/snakemake_conda/40431c23e3640492480b1b2b0c8d33df_
slurmstepd: error: *** JOB 4197138 ON d015 CANCELLED AT 2023-10-18T09:49:43 ***

The scheduler info doesn't point to an OOM issue.

has been resolved by itself.


How can I use a wav file that is different from the target music as my melody condition, when i fine-tune musicgen-melody?

Now I know that when I fine-tune musicgen-melody, the form of the dataset is the same as the form of the other text2music models, that is, the target audio and the condition audio to be converted to the chromatogram are the same. So I would like to ask how to make them different? I gave it a try but found the code hard to modify. I would appreciate any help.

@Apple-jun I'm currently working on a project where I need to use vocal audio as the input and generate instrumental music as the output using the musicgen-melody model. I understand that typically the target audio and condition audio are the same, but I need them to be different in my case. Any progress regarding this matter?


DCS sequences split across multiple "packets" can be corrupted by conpty

Windows Terminal version
1.20.10822.0
Windows build number
10.0.19045.4291
Other Software
No response
Steps to reproduce
Run the following python script:
import sys
import time

sys.stdout.write('\033[37;40m\033[2J')
sys.stdout.write('\033P2$p0;2;50;0;0/')
sys.stdout.flush()
time.sleep(0.1)
sys.stdout.write('0;2;0;0;0\033\\')

Expected Behavior
It should clear the screen and briefly make the background color red. It works as expected in OpenConsole.
Actual Behavior
In Windows Terminal, the background is left permanently red, and you can see the second half of the DCS sequence output on the screen: 0;2;0;0;0.
Looking at the debug tap, it appears that conpty is inserting an SGR reset sequence in the middle of the DCS stream (where the script sleeps for a bit). This seems similar to issue #16079, but that test case is now working correctly on the main branch, while this is still broken.
I haven't had a chance to git bisect it, so I'm not sure if it's a regression, but I wouldn't be surprised if it was always broken in some way (although possibly not obviously so). There may also be an element of timing involved, but I can reproduce the problem consistently with the script above.

BTW coincidentally I noticed something weird today and I'm not sure if it's related to the issues you've found. Try running this in a new pwsh tab (or WSL with the ` replaced):
"a`e[100m`r`nb`e[m`r`nc"

Then continue executing this until the terminal starts scrolling. Once it does, entire rows are given the attribute at a time.

Once it does, entire rows are given the attribute at a time.

This is expected behavior. When the screen scrolls, the newly revealed line is meant to be filled with the active color attributes by default. You can control that behavior with DECECM (Erase Color Mode). For example, in pwsh you would need to do this first: "`e[?117h".

I figured out the problem here. When performing a DCS sequence passthrough, the first thing we do is call Renderer::TriggerFlush to make sure any pending VT engine paints are output before starting the actual DCS sequence. However, if there is already a pending paint waiting on the console lock in the render thread, that's still going to be executed when the lock is next free, even if there's nothing left for it to do.
So what's happening in this case is the initial part of the DCS gets passed through, then the console lock is released and the VT render engine proceeds with its paint. At this point there's nothing left for it paint, but that doesn't stop it from resetting the attributes in the "Prep Colors" step of _PaintFrameForEngine. That SGR sequence then ends up in the middle of the ongoing DCS sequence, resulting in an early termination.
As a quick hack fix, I made the Renderer::TriggerFlush method toggle the console lock after calling _PaintFrame. That gives the render thread a chance to grab the lock and flush its pending paint, but this feels like a very flaky solution. I think it would probably be better if the VT render engine could just avoid sending an SGR reset when there is no real need for it. But I don't know the code well enough to know if that's a workable solution.

After further investigation into why the VT render engine is sending that SGR reset, I've discovered that that is technically a bug.
In theory, the VtEngine::StartPaint method already has a check to see whether there's something to do. See here:
https://github.com/microsoft/terminal/blob/378b6594bd94cf3b27f4e309a11efe25d83de0d9/src/renderer/vt/paint.cpp#L32-L36
If not, that method returns S_FALSE, which should indicate to the renderer that we can skip this frame entirely. However, the derived XtermEngine class ignores the S_FALSE return value, so that information is lost. See here:
https://github.com/microsoft/terminal/blob/378b6594bd94cf3b27f4e309a11efe25d83de0d9/src/renderer/vt/XtermEngine.cpp#L38-L40
The other problem is that the somethingToDo calculation always evaluates to true, because the _cursorMoved flag is always set to true. This is the result of the PR #15500 changes described in issue #17150 - we're now calling InvalidateCursor on every paint, which makes the VT engine think the cursor is always moving.
In theory that could be fixed by checking whether the cursor position in the InvalidateCursor call is different from the current position, if it weren't for the fact that the InvalidateCursor call isn't actually passed the current position - it's passed the previous position. But that should be resolved by my suggested fix for #17150.
In short, I think we could fix this bug with the following:

Issue #17150 would need to be fixed first.
We would need to check for a change in position when setting the _cursorMoved flag in VtEngine::InvalidateCursor.
The XtermEngine::StartPaint method would need to check for S_FALSE being returned from the parent class.

This way we wouldn't need to mess with the console lock, so it seems like a safer approach. There might be edge cases that I'm missing though.

After trying this out, I found there's an extra step required. When the XtermEngine::StartPaint method returns S_FALSE, it also needs to reset the _noFlushOnEnd flag, otherwise the next frame may not flush when it is supposed to.


[ACM-6056][ACM-6057][ACM-6058][ACM-6060] Remove name label from dropdown & split what's changed dashboard for policy/cluster

Jira issues:

ACM-6056 - Remove the name from managedcluster label dropdown

Removes the name label from the drop down list in the Global Hub - Cluster Group Compliancy Overview and Global Hub - Offending Clusters dashboard.

ACM-6057 - The standard,category and control filter should pass to next view when drilling down to offending policy/clusters

Update the data links in the Global Hub - Cluster Group Compliancy Overview and Global Hub - Policy Group Compliancy Overview dashboards to pass the standard, category, and control variable values in the URL.

ACM-6058 - should able to filter the policy/clusters based on the regional hub name

Add the regional hub name filter to the Global Hub - What's Changed / Clusters and Global Hub - What's Changed / Policies dashboard.

ACM-6060 - split the what change dashboard to show policy and cluster separate

Creates individual what's changed dashboard for cluster/policy.

PR is still WIP.
/hold

since we are close to the code freeze date, I would like to merge this PR firstly. so that @ldpliu can continue to work based on this PR.
Let me address the comments or question in separated PRs.

/unhold


PageTitel not changing

Hi, i tested several (default) Template and PageTitel in Browser is not really changing.
Is that correct? Why can i insert manually Page Titles in Backend, if this wont affect Frontend, even in default Template?
Is this a BUG?

Hi Alorson,
Could you please explain a bit more on what you where trying to do? Are you changing the page title under current page > page title, general > menu, on the page itself? Where do you expect to see the title change?
If you change the title under current page > page title, you will see the title change in the title bar of the tab, and in the meta tag for search engines.
If you change the title under general > menu, you will see the title change in the menu, and in the url of the page (iirc).
If you change the title on the page itself, you will only see it change on the page itself.
Also, could you maybe send some screenshots of what you are doing and what results you get vs what you would expect? Some version info (installed plugins, version of WonderCMS and version of php) would be handy as well.

I believe @StephanStanisic answers is sufficient in explaining the page title vs website title.
TLDR:
Page title -> Current page title (shown only on specific pages).
Website title -> Actual website title (shown globally).
@alorson, please feel free to re-open this issue or continue commenting for additional support.


deploy: update templates to v3.9.0

This commit updates templates new
v3.9.0 release.
depends-on: https://github.com/ceph/ceph-csi/pull/3937

@mergifyio queue

/retest all

/retest all

all runs had the same error,
lost connections to centos machines .
+ ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no<EMAIL_ADDRESS>./system-status.sh

ssh: connect to host n27-31-0.pool.ci.centos.org port 22: Connection refused

script returned exit code 255

@mergifyio requeue

Restarted since all ci tests were stuck before
https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/blue/organizations/jenkins/mini-e2e-helm_k8s-1.26/detail/mini-e2e-helm_k8s-1.26/409/pipeline/6

@Rakshith-R i think there is a CI outage going on.

/retest ci/centos/upgrade-tests-cephfs

/retest ci/centos/upgrade-tests-cephfs

/retest all

/retest ci/centos/k8s-e2e-external-storage/1.25

/retest ci/centos/upgrade-tests-cephfs

/test ci/centos/mini-e2e-helm

/test ci/centos/mini-e2e

/test ci/centos/upgrade-tests

/retest ci/centos/k8s-e2e-external-storage

@mergifyio queue

/retest ci/centos/k8s-e2e-external-storage/1.23

/retest ci/centos/k8s-e2e-external-storage/1.23

The run was not found in the link
https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/job/k8s-e2e-external-storage-1.23/3226/display/redirect


Issue #49 Resolved (Footer)

Fixes Issue

Closes #49

Changes proposed
Social Icons in the footer are now accurate and according to the design of the website.
Redirection links are given to the social icons, and they all are in working state following the target blank property.
Footer Size is modified according to the official design of the website.
Alignment of text and sponsor card fixed.
Sponsor Card size is modified according to the official design of the website.
Design of Sponsor Card is also modified and its now accurate and according to the design o the website.
Font Size is modified according to the design of the website.

Screenshots
Previous Footer:

Current Footer :

Note to reviewers
cc- @AvineshTripathi  @verma-kunal @SuperAayush

Previously SVG format images were used for social icons, but they were not working properly as for all social icons same and only one image was rendering which was in the front line of the code.
So for fixing that issue I have now used React Icons instead of SVG files.

Hey @Jai-Solania-29 any updates

Hey @AvineshTripathi actually I'm having my exams these days so I'm little busy right now. I was thinking may be you can merge this PR for now, and I'll open a different issue for responsiveness of the footer so that someone else can work on that if interested, Otherwise I'll start working on this issue in next couple of days.

All the best @Jai-Solania-29 for exams and thanks for notifying
LGTM
P.S. i ll create an issue for it after merge

Thank you @AvineshTripathi , I have my last exam tomorrow. So once you create an issue I’ll start working on that.

Hey @AvineshTripathi any updates ?

Hey @Jai-Solania-29  sorry for the delay, can you create issue and start working or i should create issue?

Hey @AvineshTripathi I’ll create an issue and will start working on that , but that is only possible when my PR gets merged. I mean the actual footer will reflect after my PR gets merged, and only then I can open an issue for making the footer responsive, and will start working on that .

LGTM with a reminder to myself the package.lock.json file has updated, in case tree gets disturbed after merge
cc @verma-kunal

can you make the positioning relative and not absolute

I believe this change hasn't been done yet! @Jai-Solania-29 can you pls check this one :)

can you make the positioning relative and not absolute

I believe this change hasn't been done yet! @Jai-Solania-29 can you pls check this one :)

Hey @verma-kunal I have pushed the requested changes, Kindly check and merge.
__
  Summary: :hammer_and_wrench: This code needs a fix

Thanks for contributing @Jai-Solania-29 ✨


[Doc] add explanations for setting bookie rack name

Add docs for https://github.com/apache/pulsar/pull/13683
This doc should be added to the code file and shown on the pulsar-admin website automatically.
However, pulsar-admin bookies are not rendered successfully. I've reported the issue and @urfreespace is investigating it.
Consequently:

I add the docs to the CLI tool page first. If pulsar-admin bookies can be shown on the pulsar-admin website, I'll add the explanations to the code file and delete the explanations on the CLI tool page.
Though there are many other commands and flags, I only add explanations for the set-bookie-rack command and rack flag since this kind of content should be generated automatically rather than updating manually. Adding them here is to show an important reminder to users.

Preview looks good:

@hangc0276 @Huanli-Meng @momo-jun could you please help review? Thanks


Project import unable to import Git-based project properly and manual enablement is always required for new projects

Description of the bug:
There is a misalignment with Bazel plugin and IJ platform when .ijwb baseDir for the project denying git enablement automatically.
There are no roots detected in this case

Even when we alter vcs.root.detector.ignore.pattern to not ignore .ijwb anymore, in case of .ijwb presence in .gitignore it is failing "test" for eligibility here

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Try to auto import any git-based project for Bazel.
Easy to reproduce when you importing project for the first time.
After setting vcs.root.detector.ignore.pattern to the value that not disqualifying .ijwb I was able to reproduce it for .ijwb specified in .gitignore but suspect that unless
Which Intellij IDE are you using? Please provide the specific version.
2023.2
What programming languages and tools are you using? Please provide specific versions.
No response
What Bazel plugin version are you using?
2<IP_ADDRESS>.1-api-version-232
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

Does this issue only happen on IntelliJ 2023.2 or is it reproducible in older versions?

I think this is not new and was like this before. We haven't tried this flow and I have some doubts that this has changed recently...


updated HIE bios

Forgot this one. And also, this signs the previous PR (#1658)

Hm, if HaRE didn't work before, how did it get past CI if the CI builds hie?

Timeline:

test dirs broken (committed)
updated HIE to newest version where HaRe fails (not committed)
fixed HaRe and now test dirs work (not committed in previous PR)
^ intermediate message missing.

Huh, github approvals don't get reset after you make changes.

That's right - you can re-request a review, but it would be a bit tedious if they all reset.
The CI statuses do rerun, of course.


Bookbag deployment not available

Even since  https://github.com/redhat-gpe/agnosticv/pull/6680  bookbag ist not available in my development deployment.

It looks like boogbag is not deployed on the workshop cluster. It is deployed on the infra cluster.
I found the boogbag urls in the yaml at demo.redhat.com:

Done


Merged observables not showing properly in html

RxJS version:
5.5.8
Code to reproduce:
*.ts:
    let temp_observable2: Observable<any[]>;
    let temp_observable3: Observable<any[]>;

    //Firestore collection with one document that contains one attribute:date
    temp_observable2 =
      this.db.collection(
        'testing_merge/'
      ).valueChanges()

    //Firestore collection with one document that contains one attribute:date
    temp_observable3 =
      this.db.collection(
        'testing_merge_2/'
      ).valueChanges()

    this.eventObservables = Observable.merge(temp_observable3, temp_observable2)
    this.eventObservables.subscribe(items =>{
      for(let item of items){
        console.log(item.date)
      }
    })

*.html:
<ion-content>
  <ion-list>
    <ion-item *ngFor="let event of eventObservables | async">

static text

{{event.date}}

    </ion-item>
  </ion-list>
</ion-content>

Expected behavior:
Expect that the html has the same two dates printed that the console.log statement printed.
Actual behavior:
The html view only gets output of one of the merged observables (whichever was last in the list of merging). Meanwhile the subscribe call with the console.log has expected behavior and prints out both dates.
Additional information:

If the console log is logging as expected, RxJS is doing what you'd expect. You're going to want to file this against Angular or Firebase. My gut tells me, though, that this has something to do with the Observables returned by Firebase. If you add a shareReplay(1) to those temp_observables, it might wor


[📑 Docs]: Fix the relative paths of index.md & migration.md files.

What Dev Docs changes are you proposing?
In the index.md & migration.md  there are many places where relative paths are wrongly placed(after the migration to monorepo) , fix the relative paths for this two particular file.
Example
In the https://github.com/asyncapi/generator/blob/6f60d50f3e4544d7356e8b1c8dc40380dfb87ece/apps/generator/docs/index.md?plain=1#L6
the AsyncAPI Document relative path is not correct, it should be asycapi-document.md instead of generator/asyncapi-document
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I would love to work on the issue please assign me

We dont assign issues, just make a PR solving the issue

@Gmin2 Raised a PR. Have a look please.

@Gmin2 but the paths should be valid only for asyncapi.com, where docs are rendered, not here in docs

@Gmin2 but the paths should be valid only for asyncapi.com, where docs are rendered, not here in docs

yes @derberg

not sure what links you have in mind exactly, can you share?

Hi @Gmin2 , I have fixed all links in index.js but i am unable to find the file migration.md , can you please send me path for that file.

@Gmin2 but the paths should be valid only for asyncapi.com, where docs are rendered, not here in docs

@derberg  we should close this issue


react-hot-loader version 4.8.1 no longer exists

package-lock.json needs an update because<EMAIL_ADDRESS>does not exist

Thanks for reporting this :)
I will look into it.

fixed by https://github.com/wonism/gatsby-advanced-blog/commit/3a4a61f0f62f3c1e3e6035800f4535297aa34359


Please add a few missing Arabic Characters from Unicode 9

Hi Georg,
while working on one of our Arabic fonts, we realized that U+08BB, U+08BC and U+08BD were not shaping as expected (tested with Glyphs 2.4.2 build 1004). I've added them to a local GlyphData.xml file and they work again as intended.  Can you please add the following to the main GlyphData.xml file?
Thanks!

GlyphData-addition-for-Unicode-9-Aracbic-characters.txt

Added it. Thanks


rescale cell profile matrices

A cell profile matrix with an excessively low scale, e.g. with max values of 1e-3, can produce convergence failures. Add an up-front step to rescale cell profile matrices to the same expression level used by safeTME.

done.


Feature-94: Client Bug Fixes

Summary of changes:

Revised hardcoded values related to referencing pid and cid refs file paths to use a class variable
Fixed hashstore client issue when a format_id argument is not supplied to use the default format_id based on the hashstore config file.

Note - this branch name is incorrect. It should have been bug-94-hashstore-client.


JFXPasswordField and JFXTextField issue

I'am using intellij idea and windows 10 18363
Scenebuilder 8.5,
Jfoenix 8 and
Java version 1.8
When after build my project and used JFXPasswordField and JFXTextField im getting this error  "Error invoking method, failed to launch jvm" from exe file.
But if i dont use JFXPasswordField and JFXTextField, i can open .jar after build artifacts.
tried java -jar myapp.jar

Not sure, but I don't think this issue is related to JFoenix.
I think it's related to how you are building the application / java version.


spfx doctor command do not support 1.15

v5.4.0.
Despite "added support for upgrading SPFx projects to v1.15.0" the doctor wasn't affected by this update.

I'm willing to work on this one.

Awesome! All yours @milanholemans and I appreciate your help 👏


Nothing happens

When I try start it nothing happens. Windows 10 64bit.
No error message.

Please try to download and install the current 64bit Visual C++ 2019 redistributable.

Yep. It fixed it.


TeX package for symbol macro \textrevcommaabove

In a bib-file I'm using a greek letter (Ἱ), which is transformed by biber --output_safechars to
\textrevcommaabove{Ι}
I can't find which package to load to enable this command. I read (ftp://ftp.dante.de/tex-archive/biblio/biber/documentation/utf8-macro-map.html) that I can find it in the file loaded by texdoc symbols -- but I can't find the command in that file. Do you know where to look? Otherwise I could ask on of the mailing lists.

(Or is there a way to define \textrevcommaabove myself?)

There is a definition for \textrevcommaabove in l8uenc.def but this is an internal encoding not meant for printing. Imho the option --output_safechars`  is not a so good idea if you are using such chars but for an on-time-shot you can define the command e.g. like this (for pdflatex, with lualatex it would be different):
\documentclass{article}
\usepackage[utf8]{inputenc}
\usepackage[LGR,T1]{fontenc}
\newcommand\textrevcommaabove[1]{{\fontencoding{LGR}\selectfont\accdasia#1}}
\begin{document}
\textrevcommaabove{I}
\end{document}

Thank you for your reply! It solves the \textrevcommaabove-problem, but there seem to be more.
Is there a better way than doing it with --output_safechars? For this project I'm probably bound to pdflatex, but I have to use utf-8 greek. All fine so far, the only problem is the bibliography ...
(If you prefer, I can start a discussion on the tex-d-l list, instead of this one here.)

I probably wouldn't use --output_safechars but use normal utf8 export but mark up the greek in the bib correctly so that pdflatex then can handle it. But here is imho not the place to ask about this. Better use a sensible usenet list or tex.sx.

As I always had weird problems with the utf8 output, I was using --output_safechars. Now I tried without it, and after some changes in the bib files (regarding turkish .i) it's all fine w/o --output_safechars. So thank you for your help!


First pass at a Reflux.js solution

Here's my first attempt at a solution with Reflux.js. I ended up going with a design where the UI requests and releases "cursors" which tell the store/action level what it should keep loaded, and what it should throw away and/or abort.
The way React can request and release cursors in the component lifecycle methods ends up being very elegant IMO. The part where it tracks the requested cursors to do the right thing gets a little hairier, and the stores and actions end up quite coupled in the end. I'd love to hear other thoughts on solving this.

It doesn't look to me like you cancel the requests when Obi-Wan is at one of the planets. Perhaps I'm missing it?

Hi @pindia I get error
XMLHttpRequest cannot load file:///Users/amed/Hobby/flux-challenge/submissions/pindia/app.jsx. Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, https, chrome-extension-resource.

in the browser when opening your index.html. Can you add build & run instructions?

@winstonewert Ah interesting, I missed that requirement. Will look into how that can fit in.
@staltz Will do, looks like I was relying on my IDE's automatic "serve current directory on localhost" feature and it doesn't work over file://

@staltz Added run instructions, let me know if you still have trouble

Thanks @pindia, but there are still some requirements to pass. For instance

The list cannot be scrolled down if the last known sith has no apprentice. The list cannot be scrolled up if the first known sith has no master.

If I click 3 times quickly on scroll up, I get an empty list that simply cannot load any data anymore.

@staltz Ah interesting, that requirement is a little ambiguous (if the "last known Sith" is loading, what happens?). I chose to allow it because in the demo GIFs, it seemed that the scroll buttons did not gray out while the last known Sith was loading, only when the last known Sith completed loading (Of course, we don't know whether it's going to be the last one until it's done loading). If preventing scrolling during loading is the desired behavior, it's a very simple change (a boolean on line 124), but does this interpretation sound right to you?

If preventing scrolling during loading is the desired behavior, it's a very simple change (a boolean on line 124), but does this interpretation sound right to you?

That's what @winstonewert did in his implementation, but in my implementation I allow scrolling even during loading. I can merge in your submission, but the more interesting question is: how much more complicated would it be to allow scrolling during loading without making it possible to "break" the app (meaning: possibility to take it to an bad state with no recovery).

how much more complicated would it be to allow scrolling during loading without making it possible to "break" the app

Interesting suggestion. I think I pulled it off by replacing the hardcoded Boolean with reading the load state of the Jedi +- 3 away from the loading one (so a minimum of 1 loaded Jedi is always on the page).
Also hacked in stopping scrolling when the current planet matches, though not nearly as elegantly as @winstonewert with the React-like diffing approach. Don't have time now but in the future will look at how that approach might fit with mine.

Yes, it works now

Alternately, for a more challenging interpretation, you could allow all the sith to scroll off the page. Then you'd have to request all the sith that were scrolled off one-by-one to figure out which sith you are displaying.


Added -webkit-transform to css transform operations.

-webkit-transform is needed for older browsers/webkits that do not support the native css3 transform. This is especially needed for Cordova apps which target older webkits on iOS and Android.

+1


Extension types loaded via entry points are registered too late

If an extension project uses entry points and the first function to be JIT-compiled takes that project's custom types as arguments, Numba will not recognize the custom types (error message is "non-precise type pyobject").
If the custom types are used in the second function to be compiled (i.e. start by compiling a dummy, numba.njit(lambda x: x + 1)(123)), then the custom types are correctly recognized (no "non-precise type pyobject" error).
If the custom types are registered with
@numba.extending.typeof_impl.register(custom_python_type)
def typeof(val, c):
    return custom_numba_type()

before the entry point gets triggered (i.e. before compiling a dummy), then the custom types are correctly recognized.
I see that the extension project used as an entry point example, scipy-numba, doesn't define any custom types (only functions), so it hasn't encountered this issue.
I can still put most of my type and lowering definitions behind the entry point, since numba.extending.typeof_impl.register only has to reference the custom types inside a function definition, which is delayed. However, the register decorator itself requires Numba to be loaded, so I can't escape the Numba load-time in processes where Numba is not actually used. (I'm avoiding any time involved in constructing my custom types, but it's more the LLVM-load that I was trying to avoid.)
Since this involves setting up entry points, I don't see how to make a minimal reproducer that's really "minimal," but I'm willing to be a guinea pig for proposed fixes.

Thanks for reporting this, without investigating I can guess that this is as described just from knowing where the entry points are loaded in Numba. I think it perhaps may be helpful to discuss some use cases and then try and work out how to accommodate them?

You could say that Python lets you have three levels of dependency on another project:

Strict load-time dependency: when your project is imported, it explicitly imports the other.
Conditional load-time dependency: when your project is imported, it imports the other if the other exists on the user's system.
Runtime dependency: your project might import the other if certain functions are called or certain submodules are imported by the user.

A project that has connectors to many others shouldn't depend on them all strictly, since that would require users to install everything before doing anything, even though they might only use a few of them. Conditional load-time dependencies are a little better because they don't require the user to install things, but they do have a startup time cost. (For example, Pandas adds 1.5 seconds to startup time on a Chromebook—the slowest plausible user system.) Runtime dependencies are best because then users only pay for what they use: both in installation effort and in startup time cost.
Decorating a function with numba.extending.typeof_impl.register can't be a runtime dependency because I have to import numba to use it. I had thought that the entry point mechanism would allow it to move from conditional load-time to runtime, but this type registration is the only step that prevents that.
Numba adds 1.0 seconds to startup time on a Chromebook. It's not terrible, but I just thought that this was the reason for using the entry points.

Is there any chance of this being addressed by 0.50?
The issue is that even with an entry point, things are resolved in this order:

User defines a function decorated with @nb.njit.
User creates an Awkward Array.
User passes the Awkward Array to the JIT-compiled function.
Numba identifies the arguments to the function and holds that information somewhere.
While loading Numba machinery, entry points get triggered, including Awkward's, which registers Awkward Arrays as Numba extensions.
Attempting to compile the function, Numba uses the old information about the function arguments, which was made before the entry point was triggered, so it doesn't recognize the Awkward Arrays as Numba extensions.
Compilation fails with a type error.

The second time the function is invoked, the Awkward Arrays are recognized as arguments and there is no type error. It's because the entry point had already been triggered before reaching the second step 4, so its information about the arguments isn't stale anymore. This inconsistency looks like a bug.
For the time being, I've resolved the order dependence by explicitly importing Numba in Awkward (if it exists on the system), which means that I really don't need to have an entry point. In fact, I don't see how entry points are useful until this gets fixed.
The reason I'd like to only import Numba if Numba gets used is in part because of the startup time (1.0 seconds on a slow computer), but also because not all users of Awkward use Numba, and the explicit dependence would cause Awkward to be non-importable if Numba is non-importable for some reason. This can happen if the user incorrectly installs Numba—a common occurrence among my users, since they usually install several versions of ROOT (not through pip or conda), which comes with a custom LLVM build, and it can happen that symbols clash. (Incidentally, this also prevents using ROOT and Julia together in any case, not just bad cases.)
I've been advocating Numba to users who wouldn't ordinarily use it, but if they get hung up in the installation and give up with a broken Numba, they'll also have a broken Awkward. I recognize that the real problem is that broken installations are possible, but like unhappy families, each situation is different.
If Numba 0.50 fixes this issue, I'd make it Awkward's minimum version. (Right now, the minimum version is 0.49 because of the API changes.)
I don't know where to look to solve the issue, but I can review the PR.

I've taken a bit of a look at this and can't reproduce with a dummy extension package I've created. The package has a dummy type and a nonsense overload of np.linalg.norm for that dummy type. I then have a test script which creates one of these types and passes it into a jit function that calls the norm, seemed to work, so I think I'm missing something. Do you have a way to reproduce? Having that would likely speed up finding a fix. Thanks in advance.

I'm working on a minimal reproducer now. I remember that the issue only affects new types, not new functions or new overloads of existing functions. That's why it worked fine for scumba (numba-scipy now?), because the latter is a collection of functions, not types.

Here is a minimally reproducing example:
dummy-numba-entry-points.zip
dummy-numba-entry-points
├── dummy
│   ├── define_type.py
│   ├── __init__.py
│   └── register_numba.py
├── setup.py
└── test_dummy.py

Enter this directory and run
pip install .

From another directory (to avoid confusion with the dummy subdirectory in it), run test_dummy.py. For me, the output looks like this:
% python dummy-numba-entry-points/test_dummy.py
decorating pass_through as a numba.njit function
calling that function with an Interval as an argument
registering the dummy module with Numba
defining IntervalType, its boxers, etc.
it failed!
calling that function with an Interval as an argument (again)
the function was successfully executed!

(and then clean up with pip uninstall dummy). The reason for the pip-install is to have the entry point in setup.py registered with Numba.
What we see in the output is that the new type failed to be passed as an argument into the Numbafied function the first time but succeeded the second time. This is because the registration does get triggered; it's too late for the first function invocation but not too late for the second. A user who doesn't recognize that pattern would be inclined to think that it's an intermittent bug, which looks bad.
At one point, I tried this with overloading functions, rather than types, and the issue only presents itself when overloading types. I think that's because function overloads are encountered in the body of the jitted function, but types have to be recognized when the input arguments are examined. If that input argument examination could be moved a little later or performed a second time if any modules had to be loaded with entry points (potentially introducing new types to handle), then that would fix the issue.
Thanks!

Brilliant, thanks for that, I can reproduce, will take a look.

I think I've worked it out. The dispatcher object call, which is made from C code, calls into numba.core.dispatcher._DispatcherBase::_compile_for_args, and this is where the types of the args is computed, immediately after this numba.core.dispatcher._DispatcherBase::compile is called, which triggers the extension loading, by which time it's too late.

https://github.com/numba/numba/pull/5717 is a patch to try and fix, @jpivarski if you have some time to test it against e.g. Awkward it would be much appreciated, many thanks.


[Feature Request]

Is your feature request related to a problem? Please describe.
< A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] >
Describe the solution you'd like
< A clear and concise description of what you want to happen. >
Describe alternatives you've considered
< A clear and concise description of any alternative solutions or features you've considered. >
Additional context
< Add any other context or screenshots about the feature request here. >
Hello, since last update, I'm no longer able to use the micro menu buttons. When I mouse over them the tooltip appears like usual, but nothing happen when I click them. The only one that is working is the clock that lead to the calendar.
Thx so much for all yuour hard and amazing work .. hope u can fix it soon

This is not a future request at all, please choose the Bug Report Template next time. Or just join my discord, which would be the easiest and fastes support.
You need to change the Bug Fix in: Information


how to draw figure 2 in the paper?

Hi, thx for your great work. Can you share the method to draw the figure 2 in your paper?

Thank you for your response. “The table in the lower right corner was added later in latex.”, can you share the latex code?

\begin{wrapfigure}{r}{0.5\textwidth}
\includegraphics[width=0.99\linewidth]{figure/param_ap.pdf}
\vspace{-39.0mm}\hspace{30.2mm}
\resizebox{0.26\columnwidth}{!}{\tablestyle{2pt}{1}
	\renewcommand\arraystretch{0.55}
	\input{table/fig2.tex}
}
\vspace{9mm}
\caption{
\textbf{Object detection performance on COCO val2017 using Mask R-CNN.}
We see that the proposed ViT-Adapter brings significant improvements to plain ViTs.
$^\bigstar$~indecates using multi-modal pre-trained ViT from~\citep{zhu2021uni}.
Backbones pre-trained on ImageNet-22K are marked with $^\dagger$, otherwise ImageNet-1K.
}
\vspace{-3mm}
\label{fig:param_ap}

\end{wrapfigure}

\begin{tabular}[b]{l|c|c}
	% \whline
	\renewcommand{\arraystretch}{0.1}
	Method & \#Param & AP$\rm ^b$ \\
	\hline
	PVTv2-B1   & 33.7M  & 44.9 \\
	ViT-T      & 26.1M  & 40.2 \\
	\rowcolor{gray!20}
	ViT-Adapter-T (ours)            & 28.1M  & 46.0 \\
	\hline
	PVTv2-B2   & 45.0M  & 47.8 \\
	Swin-T     & 47.8M  & 46.0 \\
	ViT-S & 43.8M  & 44.0 \\
	\rowcolor{gray!20}
	ViT-Adapter-S (ours)            & 47.8M  & 48.2 \\
	\hline
	Swin-B     & 107.1M & 48.6 \\
	ViT-B      & 113.6M & 45.8 \\
	\rowcolor{gray!20}
	ViT-Adapter-B (ours)            & 120.2M & 49.6 \\
	\rowcolor{yellow!15}
	ViT-Adapter-B$^\bigstar$(ours)   & 120.2M & 51.2 \\
	\hline
	ViT-L$^\dagger$      & 337.3M & 48.8 \\
	\rowcolor{gray!20}
	ViT-Adapter-L$^\dagger$ (ours)            & 347.9M & 52.1 \\
\end{tabular}

Thank you very much!


@types/react-table and react-table versions for 6.x don't match

Can you add the types defintions for the 6.x version to @types/react-table. The version is 6.11.4.

Try npm i<EMAIL_ADDRESS>

Types are not being maintained here anymore. You'll have to install them manually from here on out.


Issue with k8s.io/docs/tutorials/stateful-application/cassandra/

This is a Bug Report

Problem:
By following the tutorial, I tried to use cqlsh. It shows the following error:
>  kubectl exec -it cassandra-0 cqlsh
> No appropriate python interpreter found.
> command terminated with exit code 1

Proposed Solution:
I replaced the docker image to latest one in docker hub and removed Readiness probe in stateful yaml file.
Page to Update:
https://kubernetes.io/docs/tutorials/stateful-application/cassandra/

Thanks for the feedback
/language en

/priority backlog

/priority backlog


Use remote git executeable?

Hi, is it possible to use a remote git executeable?
I've tried telling the plugin to use the following script as my git executeable, but it's not working (and it was anyways a longshot):
#!/path/to/python/3.6

import subprocess
import sys

#  git command, add quotes for multiple words
cmd = ' '.join([f"'el'" if ' ' in el else el for el in sys.argv[1:]])
#  Construct full command line call-string
callStr = f'ssh<EMAIL_ADDRESS>"cd /project/folder && git {cmd}"'
#  Run the command
call = subprocess.Popen(callStr, shell=True)
call.wait()
call.terminate()

It works when I do a simple python36 myscript.py status, so guess it should also work when called from ST?
I need to use the remote git, since it's using a hook called nbstripout, in which some paths are hard coded to the server file system.

Never mind, figured out a workaround, so that I don't need this.
Closing this issue.


Keep database files history

Hi,
I'm using Dropbox as storage provider and payed attention that though Dropbox keeps deleted files for one month, but no such history is available for KeeEnywhere data file.
Probably file is not deleted, but rewritten.
Can you change KeeAnywhere behavior to allow keeping history (may be create new temporary file, delete old one and then rename)? Google Drive also has thrash bin, so such approach can be used for it too.
Or, may be, rename old file to name consisting time stamp before creation of new, and keep configurable number of files.
Thanks.

Hi @vovodroid,
you are right, KeeAnywhere does not delete a file, it simply overwrites it. And this is the expected behavior. The reasons:

I don't want to litter "Deleted Files"
Dropbox keeps a file history and if I delete a file and recreate it then it's simply a new file with no history.

What do you want to achieve with keeping "deleted copies"? Maybe you want to restore to an older maybe accidently overwritten version. And especially for this Dropbox keeps file hostory. Right-Click a file in Dropbox, the context menu opens. Choose "Earlier Versions" and a screen appears with a history of every single "save".
Does this help you?

Or, may be, rename old file to name consisting time stamp before creation of new, and keep configurable number of files.

I already have a similar function in mind: a kind of "backup" enhancement which keeps a defined number of copies.
Regards,
Daniel

Dropbox keeps file history

Ahh, you are right, great!