bug:

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5
Operating system/version
macOS 14.4.1
Describe the bug
The keymaps to show ignored or hidden files are harcoded to <a-...> (alt) key which is not transparent for Mac users

Steps To Reproduce

got to a directory with hidden and ignored files
search for a file (ff or fF or )
try to find a hidden or a ignored file by pressing [option]-i or[option]-h

Expected Behavior
hidden and/or ignored files should be shown
This can be fixed by changing <a- with <M-
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",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

There are other keys throughout LazyVim as you can see here that use <A-> instead of <M->. Do they work correctly on Mac. I don't have Mac, so I can't test. I'm preparing a PR to change these keymaps to <M-> instead. Could you verify please, so I know if I should change all instances or only the internal Telescope mappings?

 works well, since in my terminal emulators (warp and alacritty) I have "the left key option as goal" configured, a fairly standard option in all Mac terminals.
But it seems  (lower) doesn't behave the same way, I guess the best solution would be to use  (upper) for every case

Could you please try the following in a file in your ~/.config/nvim/lua/plugins/ directory?
return {
	  {
    "dpetka2001/LazyVim",
    branch = "fix/option-keys-mac",
  },
}

And then open Lazy UI and update LazyVim. Then restart and verify if the keymaps work as you expect.

Sorry for the delay.
Did you merge the fix?
I commented out the tweak that I made for those telescope keymaps, and restarted neovim (to be able to reproduce the issue) but it's just working as expected

I'm just a simple user like you. Only the maintainer is able to merge PRs. I haven't even made the PR yet. I asked you to test the branch from the forked LazyVim repo that I have in order to test the keymaps. Is that not needed any more? Everything works as expected at your end?

right now, is working... IDK if there is some cache out there... but I'm currently not able to restart all my tmux sessions since I'm working on three different projects :|
For now, I will leave the config as it is from the factory. Then if I have the same behavior I will use your branch, and let you now if that solves the issue. Thank you!

<A should work perfectly fine on mac with the option key. Whatever wasnt working for you before, is not due to this.

<A should work perfectly fine on mac with the option key. Whatever wasnt working for you before, is not due to this.

Yes, although the problem is related to the <a- (lower) thought

I just discovered that after kill all my tmux sessions and restarting alacritty. I'm able to reproduce the issue.
the missing file that I'm trying to find is hidden (because starts with .) and is ignored by git. So inside the Telescope's picker, I wrote the name ".env.local" but I only got the file ".env.local-example" (that is hidden, but is not ignored by git)
Then I press Option+i but nothing happens
So I added this configuration:
return {
  "nvim-telescope/telescope.nvim",
  opts = function()
    local actions = require("telescope.actions")

    local open_with_trouble = function(...)
      return require("trouble.providers.telescope").open_with_trouble(...)
    end
    local open_selected_with_trouble = function(...)
      return require("trouble.providers.telescope").open_selected_with_trouble(...)
    end
    local find_files_no_ignore = function()
      local action_state = require("telescope.actions.state")
      local line = action_state.get_current_line()
      LazyVim.telescope("find_files", { no_ignore = true, default_text = line })()
    end
    local find_files_with_hidden = function()
      local action_state = require("telescope.actions.state")
      local line = action_state.get_current_line()
      LazyVim.telescope("find_files", { hidden = true, default_text = line })()
    end

    return {
      defaults = {
        prompt_prefix = " ",
        selection_caret = " ",
        -- open files in the first window that is an actual file.
        -- use the current window if no other window is available.
        get_selection_window = function()
          local wins = vim.api.nvim_list_wins()
          table.insert(wins, 1, vim.api.nvim_get_current_win())
          for _, win in ipairs(wins) do
            local buf = vim.api.nvim_win_get_buf(win)
            if vim.bo[buf].buftype == "" then
              return win
            end
          end
          return 0
        end,
        mappings = {
          i = {
            ["<c-t>"] = open_with_trouble,
            ["<M-t>"] = open_selected_with_trouble,
            ["<M-i>"] = find_files_no_ignore,
            ["<M-h>"] = find_files_with_hidden,
            ["<C-Down>"] = actions.cycle_history_next,
            ["<C-Up>"] = actions.cycle_history_prev,
            ["<C-f>"] = actions.preview_scrolling_down,
            ["<C-b>"] = actions.preview_scrolling_up,
          },
          n = {
            ["q"] = actions.close,
          },
        },
        -- configure to use ripgrep
        vimgrep_arguments = {
          "rg",
          "-u",
          "--follow", -- Follow symbolic links
          "--hidden", -- Search for hidden files
          "--no-heading", -- Don't group matches by each file
          "--with-filename", -- Print the file path with the matched lines
          "--line-number", -- Show line numbers
          "--column", -- Show column numbers
          "--smart-case", -- Smart case search

          -- Exclude some patterns from search
          "--glob=!**/.git/*",
          "--glob=!**/.idea/*",
          "--glob=!**/.vscode/*",
          "--glob=!**/build/*",
          "--glob=!**/dist/*",
          "--glob=!**/yarn.lock",
          "--glob=!**/package-lock.json",
        },
      },
      pickers = {
        find_files = {
          hidden = true,
          -- needed to exclude some files & dirs from general search
          -- when not included or specified in .gitignore
          find_command = {
            "rg",
            "-u",
            "--files",
            "--hidden",
            "--glob=!**/.git/*",
            "--glob=!**/.idea/*",
            "--glob=!**/.vscode/*",
            "--glob=!**/build/*",
            "--glob=!**/dist/*",
            "--glob=!**/yarn.lock",
            "--glob=!**/package-lock.json",
          },
        },
      },
    }
  end,
}

(This is not default)
 mappings = {
          i = {
            ["<c-t>"] = open_with_trouble,
            ["<M-t>"] = open_selected_with_trouble,
            ["<M-i>"] = find_files_no_ignore,
            ["<M-h>"] = find_files_with_hidden,
            ["<C-Down>"] = actions.cycle_history_next,
            ["<C-Up>"] = actions.cycle_history_prev,
            ["<C-f>"] = actions.preview_scrolling_down,
            ["<C-b>"] = actions.preview_scrolling_up,
          },
          n = {
            ["q"] = actions.close,
          },
        },
        -- configure to use ripgrep
        vimgrep_arguments = {
          "rg",
          "-u",
          "--follow", -- Follow symbolic links
          "--hidden", -- Search for hidden files
          "--no-heading", -- Don't group matches by each file
          "--with-filename", -- Print the file path with the matched lines
          "--line-number", -- Show line numbers
          "--column", -- Show column numbers
          "--smart-case", -- Smart case search

          -- Exclude some patterns from search
          "--glob=!**/.git/*",
          "--glob=!**/.idea/*",
          "--glob=!**/.vscode/*",
          "--glob=!**/build/*",
          "--glob=!**/dist/*",
          "--glob=!**/yarn.lock",
          "--glob=!**/package-lock.json",
        },
      },
      pickers = {
        find_files = {
          hidden = true,
          -- needed to exclude some files & dirs from general search
          -- when not included or specified in .gitignore
          find_command = {
            "rg",
            "-u",
            "--files",
            "--hidden",
            "--glob=!**/.git/*",
            "--glob=!**/.idea/*",
            "--glob=!**/.vscode/*",
            "--glob=!**/build/*",
            "--glob=!**/dist/*",
            "--glob=!**/yarn.lock",
            "--glob=!**/package-lock.json",
          },
        },
      },

And after restart tmux and alacritty... I can do the same as before, but when I press Option+i Now I see both files ".env.local" and ".env.local-example"

My branch only changed the keys from <a-i> to <M-i>. If that doesn't fix your issue then it's probably an issue with your configuration. You should test with a LazyVim default installation instead of your personal configuration and compare the behavior.

I tested the 3 ways.
My configuration is working!
But this may related to the other arguments that I added
vimgrep_arguments = {
    "rg",
    "-u",
    "--follow", -- Follow symbolic links
    "--hidden", -- Search for hidden files
    "--no-heading", -- Don't group matches by each file
    "--with-filename", -- Print the file path with the matched lines
    "--line-number", -- Show line numbers
    "--column", -- Show column numbers
    "--smart-case", -- Smart case search

    -- Exclude some patterns from search
    "--glob=!**/.git/*",
    "--glob=!**/.idea/*",
    "--glob=!**/.vscode/*",
    "--glob=!**/build/*",
    "--glob=!**/dist/*",
    "--glob=!**/yarn.lock",
    "--glob=!**/package-lock.json",
  },
},
pickers = {
  find_files = {
    hidden = true,
    -- needed to exclude some files & dirs from general search
    -- when not included or specified in .gitignore
    find_command = {
      "rg",
      "-u",
      "--files",
      "--hidden",
      "--glob=!**/.git/*",
      "--glob=!**/.idea/*",
      "--glob=!**/.vscode/*",
      "--glob=!**/build/*",
      "--glob=!**/dist/*",
      "--glob=!**/yarn.lock",
      "--glob=!**/package-lock.json",
    },
  },
},

for the third time,

I already tried the default settings (it doesn't work)
I already tried your fork version (it doesn't work)
and I went back to my settings (it works)

I also think that the problem is not the keymap but it was solved by the arguments that I passed to vimgrep_arguments and pickers

the telescope pickers honor your gitignore, so not seeing that file is to be expected.

the telescope pickers honor your gitignore, so not seeing that file is to be expected.

I understand your point, but I disagree, since this is a configuration file, that contains secrets so is ignored to not be tracked by git, but sometimes (more often than I would like) I have to modify it. It will be useful to be able to search for it, the same way I look for every other file.
But I guess we are getting into a "personal opinion land"
I will use my config.
A possible solution could be to provide a more exposed configuration for the Telescope so users can decide on that kind of detail.
Anyway, in some way it does, since I was able to write this config, so thank you for your time, Lazyvim is awesome.

I understand your point, but I disagree

This is not about having an opinion over some behavior.
I was just stating that this is how ripgrep works by default.
If you don't like the defaults, change it to your liking like you have done.


Help and Version Flags

Add help and version flags

Flag to display help text and version were added as part of release 0.5.


Ignore patch versions of cufft and cusolver in CUDA version checks.

Fixes https://github.com/google/jax/issues/18009

Can CUDA also get patch releases? If yes, should you not be strict there too?


Issue with Equation.sample method when in 1 dimension

Hi Frank, awesome paper and very clean code, thank you for sharing it publicly!
There seems to be an issue with the sample method of the Equation class when considering 1-dimensional equations (eg. line 39 of Equation.py). Unfortunately, scipy's multivariate normal seems to remove the second dimension when self.dim=1, resulting in an indexing error when doing x_sample[:, :, 0] as x_sample will be of shape (num_sample, self.num_time_interval). I am running scipy 1.6.2.

Yes, you are right. Thank you for reporting this issue. I will fix this problem when I finish travel two weeks later.


mem-ruby: Reorder SLC atomic and response actions

Currently the MOESI_AMD_Base-directory transition for system level atomics sends the response message before the atomic is performed. This was likely done because atomics are supposed to return the value of the data before the atomic is performed and by simply ordering the actions this way that was taken care of.
With the new atomic log feature, the atomic values are pulled from the log by the coalescer on the return path. Therefore, these actions can be reordered. In fact, it is now necessary that the atomics be performed before sending the response so that the log is populated and copied by the response action. This should fix #253 .
Change-Id: Ie7e178f93990975367de2cc3e89e5ef9c9069241

Should 1415 and 1424 be changed too?

@abmerop : just to be clear, I meant these lines: https://github.com/gem5/gem5/pull/255/files#diff-ff87ec1b6c3201c2959e60d3e705f4d50a72081e3ccf90f5496c3fd2fa5adb5fL1424, https://github.com/gem5/gem5/pull/255/files#diff-ff87ec1b6c3201c2959e60d3e705f4d50a72081e3ccf90f5496c3fd2fa5adb5fL1415
Both are using responses/WBs in the "old" (now incorrect) order too.


Don't warn about Layout/FirstMethodArgumentLineBreak for single-character method names

Layout/FirstMethodArgumentLineBreak checks for a line break before the first argument in a multi-line method call. But, I argue that there should be exceptions. e.g.
notice =
  t(   # <--- this line break is required by Layout/FirstMethodArgumentLineBreak
    "flash.successful_update_to",
    type: "username",
    to: current_user.display_name)

I often have to break my I18n.translate calls into multiple lines, but the t( having to be on its own line is just a waste of vertical space.
Describe the solution you'd like
Either:

Just always allow single-character methods
Add an option to allow single-character methods
Or, add an option to allow a list of method names, to which I can add t, l, and the like.

Describe alternatives you've considered
Sometimes I'll try to disable Layout/FirstMethodArgumentLineBreak, but it makes for an extra long line (and I try to stick to 80 chars):
notice =
  t("flash.successful_update_to", # rubocop:disable Layout/FirstMethodArgumentLineBreak
    type: "username",
    to: current_user.display_name)

And wrapping the method is just extra noisy:
# rubocop:disable Layout/FirstMethodArgumentLineBreak
notice =
  t("flash.successful_update_to",
    type: "username",
    to: current_user.display_name)
# rubocop:enable Layout/FirstMethodArgumentLineBreak

So I usually just punt and obey the cop:
notice =
  t(
    "flash.successful_update_to",
    type: "username",
    to: current_user.display_name)

Considering cases like the one below, I have doubts about whether it makes sense to treat single-character names as a special case.
t(
  'flash.successful_update_to',
  type: 'username',
  to: current_user.display_name
)

If it is written like the following, it loses the intent of keeping arguments on separate lines within parentheses, which affects the layout:
t('flash.successful_update_to',
  type: 'username',
  to: current_user.display_name
)

This cop is disabled by default, and if intentionally enabled, I tend to think that, regardless of the length of the method name, we should respect consistency with "Checks for a line break before the first argument in a multi-line method call".

I see. I think... I don't use that setting for keeping arguments on separate lines within parentheses. (I'm not sure what it's called.)
Note that Layout/FirstMethodArgumentLineBreak takes no sides on the final parentheses placement. From its documentation:
# bad
method(foo, bar,
  baz)

# good
method(
  foo, bar,
  baz)

Ok, so point taken that Just always allow single-character methods is a bad solution. That's what you're saying, yes?

This is a concern about the design of the Layout/FirstMethodArgumentLineBreak cop (except other cops).
If this were to be implemented, I think it should be a configuration option that is disabled by default. However, I’m not confident that treating single-character methods as special method names makes sense.

IMO rather than adding a new configuration for this specific case, it'd make sense to use AllowedMethods/AllowedPatterns instead to make the cop ignore specified method names.

@dvandersluis That's actually exactly what I was looking for! I didn't realize I could use AllowedMethods on any cop. I guess I thought it'd have to be listed in the "Configurable attributes" section for the cop (which it isn't). Thanks! This does work (and was option number 3 in my list of possible solutions) so I'll close this ticket.

@pdobb Each cop that uses AllowedMethods needs to be updated to do so, and Layout/FirstMethodArgumentLineBreak hasn't been, are you sure that you've fixed your issue? I was suggesting that we could add it to the cop instead of the more specific logic for single method names particularly.

Looking at your config, you're not enabling the cop and it's not enabled by default.

Oh, I see. I typoed the cop name so it broke rubocop and that's why the linter stopped showing the issue. Ok, thanks. Re-opeing :D


added html support on Markdown component

Fixes #1603 .

Changes proposed in this PR:

html support on description field

WIP:

[X] adding html support
[ ] rework content, labels, tooltips

Closing as for comment


Save and load test sets from Parquet files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

16 of 37 (43.24%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 62.109%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qpbenchmark/problem.py
2
3
66.67%

qpbenchmark/problem_list.py
12
32
37.5%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.7%

Covered Lines:
477

Relevant Lines:
768

💛  - Coveralls


Set babel target to node 4 instead of current

Keeping the babel-preset-env target as "current" means that the generated files depend on the node process version. Instead set target to "4" to avoid this issue.
Closes #17.

Yeah, that's true.
I use current  for debugging with devtools, but for the rest of the world... yes.


Hide full search speed setting from public api

Thoughts?

Mostly want it hidden so that I can add and remove it without multiple api changes.
I felt that using pub(crate) might end up being too error prone. Example: accidentally exposing things by writing pub instead of pub(crate).
One alternative would be to save the speed level and seed internal settings inside the encoder, but not inside api exposed structures.

Until I cut a release any API you put move or change is fine and expected to change.
I'll make sure that the non-exhaustive and default() is in place tomorrow.


Update lockfile version & Audit vulnerable and reprecated packages

Why this PR?
#97
The current npm version should be v10. The lockfile version 2 should be used for npm v6 or earlier.

Move to #112


[ISSUE #2071] feat mqtt server

Runnable, smallest mqtt service.
You can use connect and publish.

Codecov Report

Merging #2656 (543cfef) into master (3f0022f) will decrease coverage by 0.62%.
The diff coverage is n/a.

:exclamation: Current head 543cfef differs from pull request most recent head b521074. Consider uploading reports for the commit b521074 to get more accurate results

@@             Coverage Diff              @@
##             master    #2656      +/-   ##
============================================
- Coverage     61.25%   60.63%   -0.63%
+ Complexity     4330     4302      -28
============================================
  Files           660      660
  Lines         18979    18995      +16
  Branches       1681     1684       +3
============================================
- Hits          11626    11518     -108
- Misses         6401     6531     +130
+ Partials        952      946       -6

Impacted Files
Coverage Δ

...n/alibaba/dubbo/cache/AlibabaDubboConfigCache.java
43.75% <0.00%> (-31.25%)
:arrow_down:

...gin/apache/dubbo/cache/ApacheDubboConfigCache.java
43.03% <0.00%> (-29.12%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
61.38% <0.00%> (-19.81%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ache/shenyu/admin/utils/PluginHandleH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
55.55% <0.00%> (-8.74%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
55.31% <0.00%> (-8.52%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

... 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 3f0022f...b521074. Read the comment docs.


Schema editor fixes

Schema editor appears to be wanting a different json format.  Update to the newest schema to fix schema edit functions.

Fixed with latest commit


Create a Slack Command Template

Now that Functions is GA, I wanted to have a go at creating a Slack Command / Function sample application - and I have to say the process was quite counter-intuitive.
I started by creating a GenericWebHook-CSharp - and was pleased to see that under the Integrate tab there is a "WebHook" Mode with a WebHook Type dropdown that includes "Slack" as an option. But if I set up a new Slack command with this Function as an endpoint - invoking it would cause the following error:
{"Message":"The 'token' parameter provided in the HTTP request did not match the expected value."}
If I set the Mode to "Standard" it works as expected.
I just wanted to highlight that the UX was a bit misleading and a bit confusing and I wasted a good amount of time scratching my head and searching for answers.
It might be useful if there were some templates for creating Slack Command integration - or add a Slack Command option (as well as a Slack option - if they are in fact two different types of webhooks)
The simplest C# code for accessing the Slack Command payload is:
#r "Newtonsoft.Json"

using System;
using System.Collections.Specialized;
using System.Net;
using Newtonsoft.Json;

public static async Task<object> Run(HttpRequestMessage req, TraceWriter log)
{
    var command = await req.Content.ReadAsFormDataAsync();

    return req.CreateResponse(HttpStatusCode.OK, new {
        text = command["text"]
    });
}

But you could also a) integrate with the Web API Webhooks SDK or the Slack SDK to allow rich responses using Slack Cards.

According to this ASP.NET webhooks v1 blog post in seems that Functions is already integrated with the web hooks SDK?
https://blogs.msdn.microsoft.com/webdev/2016/12/14/introducing-microsoft-asp-net-webhooks-preview-2/
"In addition to hosting your own WebHook server, ASP.NET WebHooks are part of Azure Functions where you can process WebHooks without hosting or managing your own server!"

 I was trying to find out how it was integrated. The problem might actually be that a Slack Command is different from the standard Slack webhook?
If you create a "GenericWebHook-CSharp" you get the following templated Function:
#r "Newtonsoft.Json"

using System;
using System.Net;
using Newtonsoft.Json;

public static async Task<object> Run(HttpRequestMessage req, TraceWriter log)
{
    log.Info($"Webhook was triggered!");

    string jsonContent = await req.Content.ReadAsStringAsync();
    dynamic data = JsonConvert.DeserializeObject(jsonContent);

    if (data.first == null || data.last == null) {
        return req.CreateResponse(HttpStatusCode.BadRequest, new {
            error = "Please pass first/last properties in the input object"
        });
    }

    return req.CreateResponse(HttpStatusCode.OK, new {
        greeting = $"Hello {data.first} {data.last}!"
    });
}

This does not help at all with a Slack command - if the sample I provided in my original post is how you access the Slack Command Data posted to the end point.
So this was really the point of my issue - there is documentation that says this process "just works", there are options actually within Functions that if you set them, you'd expect them to "just work" - but it doesn't - hence the commend about misleading UX and the code sample of what you'd need to supply to correctly process a Slack Command.
Unless, of course, I'm totally missing something, which is entirely possible, but I did spend a good amount of time searching and reading documentation, and I have written a number of Functions before - so I wanted to highlight that this would not be a great "pit of quality" for less experienced developers.

We do indeed leverage the ASP.NET WebHooks functionality to provide out of the box Slack support to work with Slack WebHooks and commands (that also includes the Slack types supported by the SDK).
Right now, this is really not easily discoverable, so having a template (or a set of them) and/or better documentation on this topic is needed.
We'll have this triaged to see how soon we can address this issue! Thank you for your patience!

Assigning to @christopheranderson to prioritize the value of this as a template. We don't currently have bar for this work.


KAFKA-12926: ConsumerGroupCommand's java.lang.NullPointerException at negative offsets while running kafka-consumer-groups.sh

Jira:: https://issues.apache.org/jira/browse/KAFKA-12926
Instead of setting "null" to negative offsets partition (as in KAFKA-9507), this PR aims to skip those cases from the returned list, because setting them in "null" can cause java.lang.NullPointerExceptions on downstreams methods that tries to access the data on them, because they are expecting an OffsetAndMetadata and they encouter null values.
For example, at ConsumerGroupCommand.scala at core:
  val partitionOffsets = consumerSummary.assignment.topicPartitions.asScala
            .map { topicPartition =>
              topicPartition -> committedOffsets.get(topicPartition).map(_.offset)
            }.toMap

If topicPartition has an negative offset, the committedOffsets.get(topicPartition) is null (as defined on KAFKA-9507), which translates into null.map(_.offset), which will lead to: Error: Executing consumer group command failed due to null
java.lang.NullPointerException
Unit tests added to assert that topics's partitions with an INVALID_OFFSET are not considered on the returned list of the consmer groups's offsets, so the downstream methods receive only valid OffsetAndMetadata information.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

PING @hachikuji @apovzner as I saw you on KAFKA-9507

Regarding tests:

JDK 15 and Scala 2.13:
Successful
[2021-06-11T00:23:47.258Z] 196 actionable tasks: 105 executed, 91 up-to-date

JDK 11 and Scala 2.13
Failed on only what seem's to be a flaky test KAFKA-8940
[2021-06-10T23:38:40.358Z] org.apache.kafka.streams.integration.SmokeTestDriverIntegrationTest > shouldWorkWithRebalance FAILED
[2021-06-10T23:38:40.358Z]     java.lang.AssertionError: verifying tagg

JDK 8 and Scala 2.12:
Failed on non related to change (primaraly on ZooKeeper) tests.
[2021-06-11T00:14:17.629Z] 1205 tests completed, 36 failed, 8 skipped

@IgnacioAcunaF Thanks for the PR! I am a bit on the fence with regarding to modifying the admin api here. Users might expect 'null' in this case.
Did you consider handling this in the command line tool? Also, it would be to add a unit test for the bug.

@dajac Thanks David for your comment.
I thougth of that when started to address this issue, and I came with two approaches
a) Fix locally on the ConsumerGroupCommand
b) Fix upstreams on KafkaAdmin's listConsumerGroupOffsets method
I went by the second approach because thought it could be better for downstreams methods (like the one on ConsumerGroupCommand) to have it globally solved and not delegate the null's case handling to them.
But absolutly taken your point, users might expect 'null' in this case.
I have coded the locally approach on ConsumerGroupCommand also so I am going to update de PR.
Thanks!

@dajac
Just pushed the new commit on the PR.
What do you think?

@dajac first of all, thanks again for guidance.
I could code the unit test, which also made me realize that there was another case which could potentially lead to a NullPointerException:
unassignedPartitions.map { case (tp, offset) => tp -> Some(offset.offset) }
It was the same as for the assigned partitions: if the offset for the partition is a negative integer, it translates to null, so Some(offsets.offset) gets equal to Some(null.offset) and an exception is raised. I approached it with a ternary operator.
Added the fix for it and the unit test in the last commit of the PR

Thanks @dajac for your review and comments.
Just pushed an update with them on consideration. What do you think?
Agree, there could be more potentially cases leading to a null pointer exception in the tool for similar reasons. Will take a look also if I encounter some as a follow-up for the PR.

@dajac Thanks again for comments and support! :)
Updated the PR. I think I improved the unit test. Revert some changes and keep the unit test isolated from the others, but trying to keep the same logic.  Tried to step on all the possible scenarios:

a) Assigned topic partition with no offset data at all  -> Response: None
b) Assigned topic partition with a valid OffsetAndMetadata -> Response: Long
c) Assigned topic partition with an invalidad OffsetAndMetadata (null case) -> Response: None
d) Unassigned topic partition with no offset data at all -> Response: nothing, not on the resulting List
e) Unassigned topic partition with a valid OffsetAndMetadata -> Response: Long
f) Unassigned topic partition with an invalidad OffsetAndMetadata (null case) -> Response: None

Also modified a little bit the getPartitionOffsets to adapt it to being able to call directly to collectConsumerAssignment, as suggested.
PD: As I posted on the comments earlier I encountered that the sibling test, testAdminRequestsForDescribeOffsets, lacks the validation for assigned topic partitions (it is currently only doing the validation against unassigned topic partitions, basically because there is not an topic's assigment to the testing consumer group at its initialization). Solved at the new test testAdminRequestsForDescribeNegativeOffsets, and I think that I could complement the former one.
What do you think? Is it worth to open a new PR to approach that separatly?

@dajac Updated the PR. Passed directly the param to collectConsumerAssignment as suggested, which makes the code cleaner. Also added a commit to fix an error I encountered on ARM compilation from the new unit test (from mockito).

Hi @dajac, thanks again for your comments.
I've updated the PR with them in consideration.
What do you think?
best regards

Hi @dajac,
thanks again for review and support.
Made some changes to the PR taking them into account.

Some compileTestScala Error arised. Looking on it.

Encounter and error on compileTestScala, so change
endOffsets.view.filterKeys(assignedTopicPartitions.contains).toMap.asJava
to
endOffsets.filter{ case (tp, _) => assignedTopicPartitions.contains(tp) }.asJava

Task :core:compileTestScala
[Error] /mnt/c/Users/IgnacioAndresAcunaFr/dev/IgnacioAcunaF/kafka/core/src/test/scala/unit/kafka/admin/ConsumerGroupServiceTest.scala:117: value filterKeys is not a member of scala.collection.IterableView[(org.apache.kafka.common.TopicPartition, org.apache.kafka.common.KafkaFuture[org.apache.kafka.clients.admin.ListOffsetsResult.ListOffsetsResultInfo]),scala.collection.immutable.Map[org.apache.kafka.common.TopicPartition,org.apache.kafka.common.KafkaFuture[org.apache.kafka.clients.admin.ListOffsetsResult.ListOffsetsResultInfo]]]
[Error] /mnt/c/Users/IgnacioAndresAcunaFr/dev/IgnacioAcunaF/kafka/core/src/test/scala/unit/kafka/admin/ConsumerGroupServiceTest.scala:121: value filterKeys is not a member of scala.collection.IterableView[(org.apache.kafka.common.TopicPartition, org.apache.kafka.common.KafkaFuture[org.apache.kafka.clients.admin.ListOffsetsResult.ListOffsetsResultInfo]),scala.collection.immutable.Map[org.apache.kafka.common.TopicPartition,org.apache.kafka.common.KafkaFuture[org.apache.kafka.clients.admin.ListOffsetsResult.ListOffsetsResultInfo]]]

Thanks @dajac for review and comments!
Updated the PR taking them into account.

Thanks @dajac for review and comments.
Learnt a lot :)


Fix starburst enteprise tests

Problem
Test suite for SEP is often failing because the robot tries to deploy the CustomResource before the CustomResourceDefinition is available in the cluster.
Solution
Wait for the CRD to be available before proceeding with the rest of the test case.

verified on rhods-ci-pr-test/2017 job


Quest: Bugs in the System [Progress Error]

Step 7: Interact with the statue (enters you into battle with a gigalith)
Step 8: Capture the Gigalith
Issue: While in battle with the Gigalith, if you /endbattle, the quest does not get set back a step and leaves you to capture the Gigalith without allowing you to enter battle with it again.

If this is part of a bigger pattern of tickets with all 'abnormal end' issues, please test and confirm, and link all tickets together. Close all doubled tickets. Keep one master ticket up, and ping me so I can assign it to a dev.

If this is part of a bigger pattern of tickets with all 'abnormal end' issues, please test and confirm, and link all tickets together. Close all doubled tickets. Keep one master ticket up, and ping me so I can assign it to a dev.

Closing in favour of master ticket. Quest battle loop is working as expected, warp trigger will be fixed.

Closing in favour of master ticket. Quest battle loop is working as expected, warp trigger will be fixed.


Update 69.4-steering-evolution.md

Adding my clientside Java version of the steering behaviours coding challenge

Looks great! We should really write a unit test to check that jekyll builds correctly, but I'm throwing caution to the wind, thank you for sharing!

I'm flattered you like my work! i can't seem to access the webpage it's
supposed to be on within your website. i Hope i didnt break anything with
the pull request . I can look up all of your evolutionary steering
behaviour videos except the 4th wich is the one i contributed to. Cheers!
2018-02-09 22:00 GMT+01:00 Daniel Shiffman<EMAIL_ADDRESS>
Looks great! We should really write a unit test to check that jekyll
builds correctly, but I'm throwing caution to the wind, thank you for
sharing!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CodingTrain/Rainbow-Code/pull/421#issuecomment-364565035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG5RBL7s_CInvnWp2Mybo51SvnIGb1k2ks5tTLHWgaJpZM4SAPVN
.


Require(...) is not a function.

replace:
var oauth2 = require('simple-oauth2')(credentials)
with:
var oauth2 = require('simple-oauth2', credentials);


Use user provided schema or default to public.

I had issue introspecting a specified schema and then figured out that my selection wasn't even passed to introspect method.
I removed TODO since CLI always asks for schema, so maybe there should be no fallback to public or schemas[0] which was public in my case.

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

Thanks for the PR!


Adding Options Window to Menu

In OS X I can't seem to open the options window, nor can I figure out how to add it to a menu. I've tried so many ways but nothing seems to work. Not even the shortcut. I know you don't use OS X so this isn't really a problem with the program. I just would like to be able to access the settings ini via the GUI since a lot of the editable settings are not included in the settings ini.

First, I just saw that you added the OS X compilation steps to the wiki, and I'd like to thank you for that. I admit I neither posses a mac or have any background on using it, so I couldn't do it, thank you. :smile:
If I understand the issue correctly, the menu simply doesn't show up in the top bar and Ctrl+P / Cmd+P doesn't open the options either?
Maybe some compilation flags have turned on Qt::AA_DontUseNativeMenuBar. I forced the use of the native menu bar on the latest commit on the develop branch ().
But it's strange that you don't see the menu at all since Qt should fall back to using a menu inside the window itself if this property is disabled.
In the meanwhile, here's a settings.ini which should contain most (if not all) settings that are usually editable through the options menu.
http://www.mediafire.com/view/4batdefksa2q16p/settings.ini

Oh, thank you for that settings file. that's excellent! So the menus are actually showing up, but the specific menu item to get to the Options Window does not appear to be anywhere. OS X's default Preferences seems to be paired with toggling the Options widget in the MainWindow rather than the OptionsWindow. I see in mainwindow.ui that actionOption is there (which I think is what I am needing), but it doesn't seem to show up under the "Outils" drop down. I will keep playing with the mainwindow.ui now that I have a better idea where to look.

I figured it out! I changed
<string>Options</string> to <string>Test</string> and it showed up. Apparently OS X must not allow the use of Options as a menu item.

To go one step further, I set the menuRole of Options to PreferencesRole which makes it the default preferences menu item accessible with the system wide shortcut ⌘,.

Ah I see. I'll rename it to settings or something equivalent then, as well as setting it as the preference role, thank you very much.

I'll see about sending one a little later.

So as it turns out, it's not actually OS X that bans the use of Options as a name. It's actually QT trying to help! http://doc.qt.io/qt-5/qmenubar.html If it is named any of these: "config, options, setup, settings or preferences", then it will automatically be made the default program Preferences. This is why the Options widget is defaulting to the Preferences menu item and why my options menu item is disappearing. Because once it found the Options widget, then it was just throwing away anything else it found in favor of the widget. Since I don't know how to prioritize that, I'll just rename the widget too if I can.


Support loading multiple yamls in a file

Description
Used YAML.v2 library  that supports reading multi-resource YAML files
Support Multi resource YAML files. The standalone component loader loads the YAML from the components directory. If the YAML has multi resources separated by ---, only the first component is loaded. The other components are skipped.
Using the decoder method, the standalone component loader unmarshals all the resources in YAML.
For e.g
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: statestore
spec:
  type: state.couchbase
  metadata:
  - name: couchbaseURL
    value: http://localhost:8091
  - name: username
    value: "TESTUSER"
  - name: password
    value: "PASSWORD"
  - name: bucketName
    value: dapr-state
---
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: receive-event
spec:
  type: bindings.aws.kinesis
  metadata:
  - name: region
    value: us-east-1
  - name: accessKey
    value: AWS_ACCESS_KEY # replace
  - name: secretKey
    value: AWS_SECRET_KEY #replace
  - name: streamName
    value: STREAM_NAME # Kinesis stream name
  - name: mode
    value: shared # shared - Shared throughput or extended - Extended fanout

Previous:
INFO[0000] standalone mode configured                    app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] dapr id: node-subscriber                      app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] mTLS is disabled. Skipping certificate request and tls validation  app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] found component statestore (state.couchbase)  app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] application protocol: http. waiting on port 3000  app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge

After Change:
INFO[0000] standalone mode configured                    app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] dapr id: node-subscriber                      app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] mTLS is disabled. Skipping certificate request and tls validation  app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] found component  (bindings.aws.kinesis)       app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] found component  (state.couchbase)            app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge
INFO[0000] application protocol: http. waiting on port 3000  app_id=node-subscriber instance=CEI-840.local scope=dapr.runtime type=log ver=edge

Issue reference
PR will close: #1298
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation
[ ] Specification has been updated
[x] Provided sample for the feature

Notice that the Google yaml package now caused the name of the component to not be serialized..
You can see it in the log examples above.

Notice that the yaml.v2 package now caused the name of the component to not be deserialized.
You can see it in the log examples above.

The YAML v2 is NOT accepting json tag in a struct.
Couple of approaches we can take here.

Add yaml tags along with json tag in the components struct.
Split the YAML file into multiple resources using the separator and process them individually using the ghodss library.

After looking into the metadata, I see it includes metadata from k8s.io/apimachinery/pkg/apis/meta/v1, which is json tagged. In this case, I'll drop using gopkg.in/yaml.v2 and keep the previous ghodss library for json tag support.
Instead, I will add the split YAML method using separator and pass the parsed individual yaml file to unmarshal.

@amanbha @yaron2 Can you please take a look at this pr again ?


Update CI to be n'sync with template repo

Brings in:

pr assignment
codeowners (actual group name)
standardizes some naming of jobs/workflows

Motivation

Future Work

Current dependencies on/for this PR:

main

PR #19

PR #20   👈

This comment was auto-generated by Graphite.

⏳ This pull request is set to merge as part of a Graphite merge job
Stack job ID: 6Sp7X9qSlxH9DNn07QXA.
See details on graphite.dev


Center the image

This will center the image in the middle of the page. I find it looks a lot better like this.

Thanks for the PR! Unfortunately these changes break the displaying of some images, like this one. It's meant to appear like this

but the top gets cut off


Add: Documentation on ADFS errors.

Let me know when the PR is no longer a work-in-progress and I'll consider it for merging.

@markstos This documentation is more complete. Tell me what you think.
Cheers,

I'm rejecting this PR because the advice is generally bad.

Checking the timing is an valuable security feature, yet you recommend disabling it completely
There a number of potentially valid Name ID formats as seen here, but rather than recommend finding the correct stricter format to use, you recommend setting the "don't care" format.

This advice may solve problems in short term, but there may be better options including make sure the clocks are aligned and the correct NameID format is known.

Hi @markstos,
Thanks for your review. I understand your comment and do agree with you. TBH, this table served as a reference to debug the connection with my providers.
The settings can be quite complex, and starting with a 'simple' config allowed me to enable the restrictions one by one.
The message is not clear in my PR, I definitely do not advise disabling those in production.
Would you like me to rephrase? Do you think this table adds value to the documentation?
Cheers,

@cadesalaberry I think I'm going to pass on this for now.
Personally, I keep another document of private/internal SAML documentation that details how we troubleshoot specific errors that come up in our own context.
Our wiki could be a better place to work on collaborative documentation:
https://github.com/bergie/passport-saml/wiki


Windows LMDB out of disk space error

On mwc-wallet Azure build, after updating some libraries, LMDB frequently runs out of space on the Azure tests. need to investigate why this is happening. When running locally the tests appear to pass on Windows.

@bayk We haven't run into this for a while. Maybe we should close this now?

@yyangli   Windows tests are disabled.  We should fix it and enable the tests. Let's keep it.

The rood problem is how rust tests are implemented and Windows FS works.  Because of rust tests a lot of resources are leaking and Windows just can't handle that. It doesn't make sense to change bunch of code because of that.


How to use with prerender from preact-cli

How can I use this with the prerendering ability of preact-cli
I've tried using .rewind() in the prerender.js of preact-cli's source but that doesn't work

@DannyAziz I think I'm having the same issue, were you able to fix it?

@acangiani I wasn't really able to fix this, I ended up writing a python script (You could definitely do this in node) that would edit the HTML pages that got spat out.
Not the best way but it kinda works

@DannyAziz thanks!

Any progress yet???

Hi @Faisal-Manzer, I finally ended up rewriting all the server with razzle.


update readme.md: add docs executable permission to ./make.sh

Hi @99702 - thanks for contributing, but I don't quite see why this change would be needed. Could you please help explain in which scenario make.sh would not have executable permission?

Hi @d99kris  , in new linux system after cloning , we often get .sh files with permission -rw-r--r--. or not executable.
In order to run we need to provide chmod +x executable permission to file, I had same problem faced, so I thought it would be nice to have command right in docs.

Ideally I'd like to keep the basic build instructions very brief and simple, so if this only affects a small minority of users, I would maybe prefer to add an FAQ entry instead. But before doing even that, I'd be curious to know in what actual environment one would encounter this. Would you mind sharing distro name (can be retrieved with grep PRETTY_NAME /etc/os-release) and filesystem type (can get with df -Th / - assuming no dedicated home partition)?

Oh, Its okay.
Here is my system information:
df -Th /
Filesystem     Type  Size  Used Avail Use% Mounted on
/dev/nvme0n1p3 xfs   237G   59G  178G  25% /

grep PRETTY_NAME /etc/os-release
PRETTY_NAME="Fedora Linux 40 (Xfce)"

Thank you.


Add worker_version to queued Job

This can be useful when e.g. killing a zombie job because right now we assume that the current worker version is the one the worker was using when the job started.
Then we can properly kill zombie jobs that come from an older worker version.
See https://github.com/huggingface/datasets-server/pull/827#discussion_r1114260946

should we keep it open, @lhoestq?

I think it's not high priority for now


Local EAS Android build gets stuck on Task :app:createBundleReleaseJsAndAssets

Build/Submit details page URL
No response
Summary
When I run the command eas build --platform android --local --profile staging --non-interactive, the build gets stuck on Task :app:createBundleReleaseJsAndAssets, specifically on [RUN_GRADLEW] info Done copying assets.
When I cancel it, an error message is printed that states Error: package.json does not exist in /tmp/ubuntu/eas-build-local-nodejs/cbdd028b-bc78-4a31-ba86-fe6e97ae1b88/build/ui.
This happens on my computer as well as my teammate. We've tried a mix of EAS cli versions but the issue remains. Local builds used to work but suddenly stopped.
The eas.json config for the staging profile is:
"staging": {
      "env": {
          "ENV_1": "x",
          "ENV_2": "y"
      }

One thing we noted was that if the configuration had the below code, it would not hang and would create the apk file:
"developmentClient": true
Managed or bare?
Bare
Environment
expo-env-info 1.0.5 environment info:
    System:
      OS: Linux 5.19 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
      Shell: 5.1.16 - /bin/bash
    Binaries:
      Node: 18.14.0 - ~/.nvm/versions/node/v18.14.0/bin/node
      Yarn: 1.22.19 - ~/.nvm/versions/node/v18.14.0/bin/yarn
      npm: 9.3.1 - ~/.nvm/versions/node/v18.14.0/bin/npm
    SDKs:
      Android SDK:
        API Levels: 29, 30, 31, 32, 33
        Build Tools: 30.0.2, 30.0.3, 31.0.0, 33.0.0
        System Images: android-26 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom, android-31 | Google APIs Intel x86_64 Atom
    npmPackages:
      expo: ~48.0.11 => 48.0.11
      react: 18.2.0 => 18.2.0
      react-dom: 18.2.0 => 18.2.0
      react-native: 0.71.6 => 0.71.6
      react-native-web: ~0.18.10 => 0.18.12
    npmGlobalPackages:
      eas-cli: 3.8.1
      expo-cli: 6.3.6
    Expo Workflow: bare

Error output
[RUN_GRADLEW] Error: Gradle build failed with unknown error. See logs for the "Run gradlew" phase for more information.
[ON_BUILD_ERROR_HOOK]
Error: package.json does not exist in /tmp/ubuntu/eas-build-local-nodejs/cbdd028b-bc78-4a31-ba86-fe6e97ae1b88/build/ui
    at readPackageJson (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/utils/project.js:52:15)
    at runHookIfPresent (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/utils/hooks.js:25:55)
    at /home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/common.js:21:52
    at BuildContext.runBuildPhase (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/context.js:83:34)
    at runBuilderWithHooksAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/common.js:20:23)
    at async Object.androidBuilder (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/android.js:15:12)
    at async buildAndroidAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/android.js:62:12)
    at async buildAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/build.js:39:29)
    at async main (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/main.js:16:9)
[ON_BUILD_COMPLETE_HOOK]
Error: package.json does not exist in /tmp/ubuntu/eas-build-local-nodejs/cbdd028b-bc78-4a31-ba86-fe6e97ae1b88/build/ui
    at readPackageJson (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/utils/project.js:52:15)
    at runHookIfPresent (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/utils/hooks.js:25:55)
    at /home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/common.js:27:52
    at BuildContext.runBuildPhase (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/context.js:83:34)
    at runBuilderWithHooksAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/common.js:26:23)
    at async Object.androidBuilder (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/@expo/build-tools/dist/builders/android.js:15:12)
    at async buildAndroidAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/android.js:62:12)
    at async buildAsync (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/build.js:39:29)
    at async main (/home/ubuntu/.npm/_npx/ac2fe9123ebf2e7d/node_modules/eas-cli-local-build-plugin/dist/main.js:16:9)

Build failed

Reproducible demo or steps to reproduce from a blank project
Using a basic eas.json config for staging with just two environment variables:
"staging": {
      "env": {
          "ENV_1": "x",
          "ENV_2": "y"
      }

run:
eas build --platform android --local --profile staging --non-interactive
and it gets stuck on:
[RUN_GRADLEW] info Done copying assets

@modelorona , were you able to get the issue fixed?

Affecting me too since I switched the build host from OSX to Ubuntu.
Love the fact that you just closed the ticket within 24 hours without comment.

TL;DR: you've been bitten by this: https://github.com/expo/expo/pull/24952
Add this to the bottom of your package.json:
"resolutions": {
"@expo/cli": "^0.15.0"
}
It will stop the hang and give you a nice dump of the actual problem (which in my case turned out to be a plugin which was looking for package.json in the lib folder instead of the module root)

We have a similar issue but mine was because of stripAnsi
> Task :app:createBundleReleaseJsAndAssets FAILED
/home/expo/workingdir/build/apps/higala-pay/node_modules/metro/src/lib/reporting.js:15
const stripAnsi = require("strip-ansi");
                  ^

Package Versions
"expo": "49.0.22",
"react": "18.2.0",
"react-native": "0.72.10",
"metro": "0.76.7",
"metro-config": "^0.80.8",
"metro-react-native-babel-transformer": "^0.77.0",
"metro-resolver": "0.76.7",
I solved this by adding this in package.json. If you are using a monorepo(NX console), add it also in the package.json
  "resolutions": {
    "wrap-ansi": "7.0.0",
    "string-width": "4.1.0"
  }


Use a nonexistant hostname

- What I did
Sped up two tests from 30s to <1s
- How I did it
Changed the hostname in the tests
- How to verify it
Run the tests, and they fail more quickly.
- Description for the changelog

Use a nonexistant hostname for the "failing server" instead of one that does resolve (e.g. example.com) which I'm assuming was just a coincidence.
- A picture of a cute animal (not mandatory but encouraged)

Before, the TCP handshake had to time out (approx 30s) before
this test completed. If you use a hostname that doesn't resolve,
then it fails faster.
Signed-off-by: Christy Perez<EMAIL_ADDRESS>

There is actually an rfc for this https://tools.ietf.org/html/rfc2606 and you are supposed to use .invalid for this use case.

Oh! I knew about example.com (which can be used to actually test an existing domain), but TIL about the .invalid.
@clnperez can you change it to an .invalid domain?

Updated. Thanks @justincormack. I didn't know about that one. That might be the shortest RFC I've ever seen, btw. :laughing:

LGTM

ping @justincormack PTAL :+1:

LGTM


[FEATURE REQ] Storage.Queues change internal modifier to public for in-memory tests

Library or service name.
Microsoft.Azure.WebJobs.Extensions.Storage.Queues
Is your feature request related to a problem? Please describe.
I wanted to create an integration test, that emulates storage in memory as it was in the previous webjobs sdk.
Old webjobs sdk test implementation, that can be faked:
https://github.com/Azure/azure-webjobs-sdk/blob/18ddad07187d94f6e890c6ebe1c02ba758d1f490/test/Microsoft.Azure.Webjobs.Extensions.Storage.UnitTests/FakeStorageAccountProvider.cs#L26
    public class FakeStorageAccount : StorageAccount
    {
        private FakeStorage.FakeAccount _account2 = new FakeStorage.FakeAccount();

        public override CloudQueueClient CreateCloudQueueClient()
        {
            return _account2.CreateCloudQueueClient();
        }
        public override CloudBlobClient CreateCloudBlobClient()
        {
            return _account2.CreateCloudBlobClient();
        }

        public override CloudTableClient CreateCloudTableClient()
        {
            return _account2.CreateCloudTableClient();
        }

        public override string Name => _account2.Name;
        public override bool IsDevelopmentStorageAccount() { return true; }
    }

https://github.com/Azure/azure-webjobs-sdk/blob/18ddad07187d94f6e890c6ebe1c02ba758d1f490/test/FakeStorage/Queue/FakeQueueClient.cs
public class FakeQueueClient : CloudQueueClient
    {
...
        public override CloudQueue GetQueueReference(string queueName)
        {
            return new FakeQueue(this, queueName);
        }
...

This is possible because the classes are public and can be used in different projects and not just in local assemblies.
New azure sdk implementation uses internal everywhere:
I identified, that the main class, that i could fake is QueueServiceClientProvider. And if i could fake the class, then i could pass a fake QueueServiceClient, that can create a fake QueueClient, that internally would use ConcurrentQueue in order to run tests only in memory and without an azure storage emulator.
    internal class QueueServiceClientProvider : StorageClientProvider<QueueServiceClient, QueueClientOptions>
    {
        public QueueServiceClientProvider(IConfiguration configuration, AzureComponentFactory componentFactory, AzureEventSourceLogForwarder logForwarder)
            : base(configuration, componentFactory, logForwarder) {}

        protected override QueueServiceClient CreateClientFromConnectionString(string connectionString, QueueClientOptions options)
        {
            return new QueueServiceClient(connectionString, options);
        }

        protected override QueueServiceClient CreateClientFromTokenCredential(Uri endpointUri, TokenCredential tokenCredential, QueueClientOptions options)
        {
            return new QueueServiceClient(endpointUri, tokenCredential, options);
        }
    }

Possible solution:
Can we turn some classes to public in order to be able to fake some types or do you suggest some other solution how i could test webjobs in memory and without azure storage emulator?

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi @maris-jurgenbergs, I'd recommend using a mocking framework like MOQ.
using Azure;
using Azure.Core;
using Azure.Storage.Queues;
using Azure.Storage.Queues.Models;
using Moq;
using NUnit.Framework;
using System;
using System.Collections.Generic;
using System.Diagnostics.CodeAnalysis;
using System.IO;
using System.Threading.Tasks;

namespace ConsoleApp8
{
    class Program
    {
        static async Task Main(string[] args)
        {
            // Arrange
            Mock<QueueClient> queueClient = new Mock<QueueClient>();

            QueueMessage[] queueMessages = new QueueMessage[1];
            queueMessages[0] = QueuesModelFactory.QueueMessage("messageId", "popReceipt", "messageText", 0);

            queueClient.Setup(queueClient => queueClient.ReceiveMessagesAsync()).Returns(Task.FromResult(Response.FromValue(queueMessages, new MockResponse())));

            // Act
            Response<QueueMessage[]> returnedResponse = await queueClient.Object.ReceiveMessagesAsync();

            // Assert
            Assert.AreEqual("messageText", returnedResponse.Value[0].MessageText);
            queueClient.Verify(queueClient => queueClient.ReceiveMessagesAsync());
        }
    }

    class MockResponse : Response
    {
        public override int Status => throw new NotImplementedException();

        public override string ReasonPhrase => throw new NotImplementedException();

        public override Stream ContentStream { get => throw new NotImplementedException(); set => throw new NotImplementedException(); }
        public override string ClientRequestId { get => throw new NotImplementedException(); set => throw new NotImplementedException(); }

        public override void Dispose()
        {
            throw new NotImplementedException();
        }

        protected override bool ContainsHeader(string name)
        {
            throw new NotImplementedException();
        }

        protected override IEnumerable<HttpHeader> EnumerateHeaders()
        {
            throw new NotImplementedException();
        }

        protected override bool TryGetHeader(string name, [NotNullWhen(true)] out string value)
        {
            throw new NotImplementedException();
        }

        protected override bool TryGetHeaderValues(string name, [NotNullWhen(true)] out IEnumerable<string> values)
        {
            throw new NotImplementedException();
        }
    }
}

Yeah Mock is a good scenario, when I would want to have just a unit test to test something specific.
What I wanted was a integration test, that would create the host, the services and the storage specific functionality.
If I could replace one part that was responsible for saving the data in storage with in-memory functionality, then I could create a integration test, that is lightweight compared to a solution, if I would have to use a storage emulator dependency (which means configuring a build agent, that has an emulator configured and up to date).
I encountered, where a problem happens with WebJobs not starting properly and the problem was that after a library update one of the dependent libraries was not fully updated. Everything compiles but I got a runtime error later.
With a unit test I might not get that error, because mocks don't use the whole original code.
Trying to have an automated test that checks if WebJobs are fully running.


Sometimes a card got a background that belongs to other card

Hello @RamotionDev, thanks for this awesome library. Very simple to use and  the result is very smoth and cool.
I got a problem with the cards, sometimes a card got a background that belongs to other card.
VIDEO
https://streamable.com/43zbj
As you can see, the first time I enter the city list everything is fine.
But when I enter again the same list (no changes are made in background): City 2 and City 3 exchanged their backgrounds.
ISSUE FLOW

Enter expanding collection activity. Everything OK.
Go back to your main menu.
Re-enter again the expanding collection activity.
Error:
Card nº 1 got the background from card nº 3.
Card nº 3 got the background from card nº 5.
etc ....

TEST
I've checked both setMainBackgroundResource() and setHeadBackgroundResource() and they are being called properly.
In fact, my activity, card/comments classes and layouts are the ones from:
https://github.com/Ramotion/expanding-collection-android/tree/master/expanding-collection-full-example/src/main
HARDWARE
Issue happens on the 2 mobiles I tested:

Nexus 5  (android 6.0.1)
Xiaomi MiA1 (android 8.1.0)

Thanks for all!

Hello, @t0mjoad. If you really made an absolute copy of example app, then you probably also copied getDataset() method from ExampleDataSet class. You should remove Collections.shuffle(dataset);
from your getDataSet() method.

Hello @golovin47, thanks for your answer. I was able to solve it!
In fact I forked your project and customized some things (lazy loading of images from URLs and corrected a pair of minor bugs). Don't know if you guys accept pull request but I can do it if you want.
Thanks!


embedding question

HI, I got a question about the embedding part in your seq2seq pytorch tutorial. You created two separate embedding matrix for encoder and decoder. Now I want there two matrices are the same. I mean I want the single embedding matrix updated both in the encoder part and decoder part.
Do you have any suggestions? Thanks.

a simple way as follow：
self.decoder.embedding.weight = self.encoder.embedding.weight


Various cleanups prior to 1.1.0 release

Add support for RGBA images (e.g. PNG)
Add Swagger example response

glad to see flake is running on some models


zstash Globus functionality has become overly cumbersome

Request criteria

[X] I searched the zstash GitHub Discussions to find a similar question and didn't find it.
[X] I searched the zstash documentation.
[X] This issue does not match the other templates (i.e., it is not a bug report, documentation request, feature request, or a question.)

Issue description
Using zstash with Globus is now much more cumbersome than it used to be; to the point that it may become questionable whether integrating Globus actually adds value to zstash.
Here is the workflow I have to use:
(1) Login to Globus web interface and activate end points
(2) Delete existing globus cfg file
rm ~/.globus-native-apps.cfg
(3) Start interactive zstash test transfer, something like
zstash create -v --non-blocking --hpss=globus://nersc/home/g/golaz/E3SMv3.LR/tmp tmp
During the the interactive test run, zstash will ask to copy and paste authorization code twice. zstash runs but files are not transferred.
(4) Start a second interactive zstash transfer
zstash create -v --non-blocking --hpss=globus://nersc/home/g/golaz/E3SMv3.LR/tmp tmp
This one should complete without issue or any prompt.
(5) Start long transfer (in a screen session to keep it running)
zstash create -v --non-blocking --hpss=globus://nersc/home/g/golaz/E3SMv3.LR/${EXP} . 2>&1 | tee zstash/zstash_create_${stamp}.log
Note that this transfer is limited to 48 hours due to Globus token expiration. Given the relatively slow transfer speeds between chrysalis and NERSC HPSS (~100 MB/s), 48 hours is often insufficient for a large simulation.
(6) Using Globus web interface, manually transfer zstash files that were not transferred due to token expiration.
(7) Repeat steps (2) to (4) above. Restart zstash archiving that stopped:
zstash update -v --non-blocking --hpss=globus://nersc/home/g/golaz/E3SMv3.LR/${EXP} 2>&1 | tee zstash/zstash_create_${stamp}.log
I should note that in the not so distant past, only steps (1) and (5) were needed. It is also possible that I'm doing something wrong, in which case I'd like to know what.

Related to https://github.com/E3SM-Project/zstash/issues/338


Clean up GPU branch [10]

point estimate: 10 points
feature: #67
The GPU branch github.com/bcumming/nestmc-proto/tree/gpu needs to be merged with master. There are issues that make this difficult

its "diff" is overwhelming relative to the master
the quality of the code in this branch needs to be improved (types, comments, design, documentation)
more work is required to get it ready for final merge: namely validation of the gpu results

This task aims to target steps 1 and 2 in a gpu branch of the main repository. Once the state of the branch is satisfactory, the validation in step 3 will be addressed in a separate project task.
Acceptance criteria

all tests pass and validation pass for the CPU back end
the CPU back end has unchanged performance relative to current master
gpu version of tests and miniapp compiles and runs without crashing
@halfflat approves the design and implementation of the cpu and gpu implementations

Steps

address @halfflat's comments in #45 [3]
add changes since #45 to the PR [4]
address @halfflat's comments on step (2) [3]

Looks good to me!

Finished the estimates. I went through your first code review and just made a quick tally of the small and large fixes, to come up with 6 points for me to make fixes, comment on the review, and then make further changes based on follow up review.

Step 1 has been finished, and step 2 mostly finished. The backend implementation hasn't been fully polished, but I think that its structure is quite clear and fit for purpose.

Step 2 has been finished, the PR #45 is ready for review.

PR #45 merged!


[Follow up from Steam] Is there something wrong, or the is the plugin outdated?

Anyways, thanks for all the help in the Archi group and here is my final question everything up and working for the past day! Domain ready, everything perfect, just wanna say thanks and I got one last question. How would I enable this plugin? I keep getting a error and wont show up in plugins. Here's the log https://pastebin.com/JhsCq0wS Plugin I'm trying to install is ASF-Achievement-Manager by Ryzhehvost (you which i didnt know)

Yes, this plugin is outdated and does not work with latest ASF. I plan to update it when I will have some time, but there is no estimated date.

Yes, this plugin is outdated and does not work with latest ASF. I plan to update it when I will have some time, but there is no estimated date.

closed with #12


Unable to configure Navidrome as Subsonic source

Describe the bug
Attempting to configure Navidrome as a subsonic API source, but logs show that auth failed. Not sure if the subsonic API that Navidrome uses (v1.16.1) is compatible with multi-scrobbler, or if I've just not set it up correctly.
To Reproduce
Steps to reproduce the behavior:

setup navidrome in docker (v.0.49.3)
Configured the subsonic.json to point to navidrome instance
restart multi-scrobbler docker container
See error

Expected behavior
Navidrome succcessfully authenticates as a source in multi-scrobbler
Logs
multi-scrobbler    | 2023-05-24T16:46:07.131334315Z 2023-05-24T10:46:07-06:00 debug   : [Sources] (Navidrome) Constructing subsonic source
multi-scrobbler    | 2023-05-24T16:46:07.131861158Z 2023-05-24T10:46:07-06:00 debug   : [Sources] Attempting subsonic (Navidrome) initialization...
multi-scrobbler    | 2023-05-24T16:46:07.138446153Z 2023-05-24T10:46:07-06:00 info    : [Sources] [Subsonic - Navidrome] Subsonic Connection: ok
multi-scrobbler    | 2023-05-24T16:46:07.139103041Z 2023-05-24T10:46:07-06:00 info    : [Sources] subsonic (Navidrome) source initialized
multi-scrobbler    | 2023-05-24T16:46:07.139352562Z 2023-05-24T10:46:07-06:00 debug   : [Sources] Checking subsonic (Navidrome) source auth...
multi-scrobbler    | 2023-05-24T16:46:07.142008990Z 2023-05-24T10:46:07-06:00 warn    : [Sources] [Subsonic - Navidrome] Request failed but retries (0) less than max (1), retrying request after 1.5 seconds...
multi-scrobbler    | 2023-05-24T16:46:08.646301039Z 2023-05-24T10:46:08-06:00 error   : [Sources] [Subsonic - Navidrome] API Call failed: Server Response => undefined {"response":"404 page not found\n","status":404}
multi-scrobbler    | 2023-05-24T16:46:08.647011058Z 2023-05-24T10:46:08-06:00 warn    : [Sources] subsonic (Navidrome) source auth failed.
multi-scrobbler    | 2023-05-24T16:46:10.154650567Z 2023-05-24T10:46:10-06:00 error   : [Sources] [Subsonic - Navidrome] Cannot start polling because source is not authenticated correctly.

Versions (please complete the following information):
Provide version information for any related sources/clients.

multi-scrobbler: :latest on docker
navidrome: 0.49.3

Additional context
Can provide info on other media sources/clients as needed.

Nevermind, I edited the URL to remove the trailing "/" so that it's just "url": "http://<ip address>:4533" in the json and that seems to have done the trick.


Simple example of event handler?

Very usefull widget. Can you provide example on event handler,triggered when user select item from dropdown?
Thank you!

An onSelect option was added:
var single = $('#singleInput').materialize_autocomplete({
  multiple: {
    enable: false
  },
  dropdown: {
    el: '#singleDropdown'
  },
  onSelect: function (item) {
    console.log(item.text + ' was selected');
  }
});


Added Policy Group 7 and changed Policy 1.1 to a SHOULD in Google Chat Baseline

🗣 Description
Changed GWS.CHAT.1.1v0.1 from a SHOULD to a SHALL.
Fixes #175
💭 Motivation and context
Google Product Update/RFC

🧪 Testing

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] If applicable, All future TODOs are captured in issues, which are referenced in the PR description.
[x] The relevant issues PR resolves are linked preferably via closing keywords.
[x] All relevant type-of-change labels have been added.
[x] I have read and agree to the CONTRIBUTING.md document.
[ ] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge Checklist

[ ] This PR has been smoke tested to ensure main is in a functional state when this PR is merged.
[ ] Squash all commits into one PR level commit using the Squash and merge button.

✅ Post-merge Checklist

[ ] Delete the branch to clean up.
[ ] Close issues resolved by this PR if the closing keywords did not activate.

Wait to merge until TTP Mappings Added

@LaurenBassett The mappings are complete. Once all other reviews are done, this PR can be merged.

@buidav Added to TOC. Ready for review.

Baseline changes look good. I will start making the code changes.


Speed up fetching group members by listing members concurrently

Is your feature request related to a problem? Please describe.
When querying all googledirectory_group_member using the next query, the members are not queried in parallel, making it slow:
        select
            g.customer_id as customer_id,
            g.id as group_id,
            g.name as group_name,
            m.id as member_id,
            m.type as member_type,
            m.email as member_email,
            m.status as member_status
        from googledirectory.googledirectory_group as g
        join googledirectory.googledirectory_group_member as m on m.group_id = g.id
        where g.customer_id = 'my_customer_id'

Describe the solution you'd like
The preferred solution executes the googledirectory_group_member.list-methods in parallel, using a configurable concurrency to reach the Admin SDK rate limit of 2400 req/min.
Describe alternatives you've considered
Additional context
In the plugin log, the group_member-calls are listed sequentially:
2024-04-15 14:57:50.951 UTC [INFO]  steampipe-plugin-googledirectory.plugin: [INFO]<PHONE_NUMBER>432: Plugin execute table name: googledirectory_group quals: customer_id &{=} string_value:"...",<PHONE_NUMBER>432)
...
2024-04-15 14:57:53.681 UTC [INFO]  steampipe-plugin-googledirectory.plugin: [INFO]<PHONE_NUMBER>235: Plugin execute table name: googledirectory_group_member quals: group_id &{=} string_value:"...",<PHONE_NUMBER>235)
2024-04-15 14:57:53.963 UTC [INFO]  steampipe-plugin-googledirectory.plugin: [INFO]<PHONE_NUMBER>115: Plugin execute table name: googledirectory_group_member quals: group_id &{=} string_value:"...",<PHONE_NUMBER>115)
2024-04-15 14:57:54.294 UTC [INFO]  steampipe-plugin-googledirectory.plugin: [INFO]<PHONE_NUMBER>677: Plugin execute table name: googledirectory_group_member quals: group_id &{=} string_value:"...",<PHONE_NUMBER>677)
2024-04-15 14:57:54.575 UTC [INFO]  steampipe-plugin-googledirectory.plugin: [INFO]<PHONE_NUMBER>546: Plugin execute table name: googledirectory_group_member quals: group_id &{=} string_value:"...",<PHONE_NUMBER>546)
...

And, consequently, the Admin SDK API monitoring lists 1 to 2 req/second.

Hello @laurensknoll,
Thank you for bringing this issue to our attention.
I've reviewed the current code and noted that when querying the googledirectory_group_member table, the group_id parameter is essential for calling the List Members API.
I've implemented some updates in the issue-88 branch to enhance how members are listed concurrently:
Updated Table Behavior in issue-88:

The googledirectory_group table now acts as the parent for googledirectory_group_member.
The group_id column has been made optional.
The Group ID can be derived from the parent table.
It is now optional to specify group_id when querying googledirectory_group_member, allowing for simpler queries like select * from googledirectory_group_member.

I have modified the query plan to make it faster.
The behavior of the query(mentioned by you in the issue description) will remain the same as we are making the API call by passing the Group ID.
WITH GroupDetails AS (
    SELECT
        id AS group_id,
        customer_id,
        name AS group_name
    FROM
        googledirectory.googledirectory_group
    WHERE
        customer_id = 'my_customer_id'
),
MemberDetails AS (
    SELECT
        id AS member_id,
        group_id,
        type AS member_type,
        email AS member_email,
        status AS member_status
    FROM
        googledirectory.googledirectory_group_member
)
SELECT
    g.customer_id,
    g.group_id,
    g.group_name,
    m.member_id,
    m.member_type,
    m.member_email,
    m.member_status
FROM
    GroupDetails g
    JOIN MemberDetails m ON m.group_id = g.group_id;

Please try pulling the changes from issue-88, build the plugin locally, and run the above query. Your feedback would be greatly appreciated.
Steps to build the plugin locally with the changes from issue-88:

Clone the repository: git clone https://github.com/turbot/steampipe-plugin-googledirectory.git
Change to the directory: cd steampipe-plugin-googledirectory
Check out the branch: git checkout issue-88
Build the plugin: make
Execute the query mentioned above.

Looking forward to your feedback. Thanks again!

Hello, @laurensknoll, Did you get an opportunity to give it a try?

Hi @ParthaI , Thanks for preparing the improvement. I've run both version to check performance. However, more interesting, the improvement branch returns different results. This is not expected, because the WHERE direct_member_count > 0 should filter out all the groups without members.
Query
query "google_group_members" {
    sql = <<-EOQ
        select
            googledirectory_group.customer_id as customer_id,
            googledirectory_group.id as group_id,
            googledirectory_group.name as group_name,
            googledirectory_group_member.id as member_id,
            googledirectory_group_member.type as member_type,
            googledirectory_group_member.email as member_email,
            googledirectory_group_member.status as member_status
        from googledirectory.googledirectory_group
        left join googledirectory.googledirectory_group_member
            on googledirectory_group_member.group_id = googledirectory_group.id
        where googledirectory_group.customer_id = $1
          and googledirectory_group.direct_members_count > 0
        limit 200
    EOQ

    param "customer_id" {
        description = "$1, Google Workspace customer ID."
    }
}

Performance
Old:
time steampipe query 'monitoring.query.google_group_members(customer_id => “…”)’
steampipe query   7.70s user 1.98s system 12% cpu 1:16.63 total

New:
time steampipe query 'monitoring.query.google_group_members(customer_id => “…”)’
steampipe query   18.63s user 3.55s system 14% cpu 2:29.76 total

Results
Old:
+-------------+-----------------+-------------------------+-----------------------+-------------+
| customer_id | group_id        | group_name              | member_id             | member_type |
+-------------+-----------------+-------------------------+-----------------------+-------------+
| ...   | 00lnxb... | group1 | 10365.. | USER        |
| ...   | 00lnxb... | group1 | 11481.. | USER        |
| ...   | 00lnxb... | group1 | 11315.. | USER        |
| ...   | 00ihv6... | group2 | 041mg.. | GROUP       |
| ...   | 02szc7... | group3 | 01gf8.. | GROUP       |
| ...   | 02p2cs... | group4 | 040ew.. | GROUP       |

New:
+-------------+-----------------+-------------------------+-----------+-------------+--------------+---------------+
| customer_id | group_id        | group_name              | member_id | member_type | member_email | member_status |
+-------------+-----------------+-------------------------+-----------+-------------+--------------+---------------+
| ...   | 00ln.. | group1  | <null>    | <null>      | <null>       | <null>        |
| ...   | 02p2.. | group2  | <null>    | <null>      | <null>       | <null>        |
| ...   | 04an.. | group3  | <null>    | <null>      | <null>       | <null>        |
| ...   | 01x0.. | group4  | <null>    | <null>      | <null>       | <null>        |
| ...   | 02sz.. | group5  | <null>    | <null>      | <null>       | <null>        |
| ...   | 049x.. | group6  | <null>    | <null>      | <null>       | <null>        |
| ...   | 01mr.. | group7  | <null>    | <null>      | <null>       | <null>        |
| ...   | 00ih.. | group8  | <null>    | <null>      | <null>       | <null>        |
| ...   | 025b.. | group9  | <null>    | <null>      | <null>       | <null>        |
| ...   | 049x.. | group10 | <null>    | <null>      | <null>       | <null>        |
| ...   | 02lw.. | group11 | <null>    | <null>      | <null>       | <null>        |

@laurensknoll Thank you for testing the PR branch. I've committed another update to resolve the output result discrepancies for the specified query. In the enhancement branch, we have structured the table to enable parallel API calls by configuring the ParentHydrate within the table definition.
The table will retrieve all groups in your workspace, then it will execute parallel API calls for each group to fetch the group members.
Could you please pull the latest changes from the issue-88 branch, build the plugin locally, and then execute the query? Let us know if you encounter any discrepancies in data between the old and new changes.
Thanks!

Hi @ParthaI , thanks for the update.
When running the query, many 404's are returned by the API. These seem to be caused by the ParentHydrate query for the group (this API requires either customer_id or domain to be configured). Can you take a look at that using the additional evidence below.
BTW In terms of concurrency, the number of requests seems to stick at ~4 req/sec.  Thanks!

Here’s a more understandable version of the sentence:

Hello @laurensknoll,
I revisited the issue and tried a few different approaches. However, the changes in the issue-88 branch seem to be the best solution for maximizing parallel API calls, as we discussed earlier here.
There’s a configuration at the Steampipe workspace level that allows you to increase the number of parallel API calls. You can do this by adjusting the max_parallel value in your ~/.steampipe/config/workspaces.spc file.
For example:
workspace "all_options" {
      max_parallel = 15
}

For more details about configuring workspaces, you can refer to the documentation here: Steampipe Workspace Configuration.
Please try pulling the latest changes from the issue-88 branch, rebuild the plugin, and let us know if it improves performance.
Thanks!

Hello @laurensknoll ,
We haven’t received a response in a while, so we’re going to close this issue for now. If this is still a concern or if you have additional details to share, please don’t hesitate to reopen the issue, and we’ll be happy to assist you further.
Thank you for your understanding, and we appreciate your contributions!!


Compress Failed on Windows

Command Line Error:
Incorrect item in listfile.
Check charset encoding and -scs switch.
C:\Users\XXXX\AppData\Local\Temp\tmp-XXXXXXXXXXXXXXXX.tmp

While extract and extractTarGz works well.

This issue occurs when we use tmp to create a temporary file and use it to spawn process without discarding the file descriptor. Should be fixed in 5ef54b1.


test(Row): adds commented out test back

Description
this PR adds the commented out test back for Row.
References
LUI-709
Testing
clone branch
yarn run test Row
all Row tests should past with no skipped test.
Automation
Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

I was looking at this before, yes that test is passing but may be we can rewrite that test looks like that test is written based on old code.


Brimhaven agility arena arrows, and timer, go missing

Type
Incorrect behavior
Operating System
Windows
OS Version
Win 10
Bug description
Running around Brimhaven agility arena the arrows, and occasionally RuneLite's timer, for the target pillar go missing.
This seems to be triggered randomly when I interact with other software (f.e. discord, Chrome) while doing the activity, and runeLite loses being the focus point. Most surefire way to trigger this seems to be crossing the balancing rope and interacting with other software during the animation.
This also most often happens when the game loads during obstacles.
Apparently also using RuneLite's "take screenshot" button triggers this as well.
Seemingly toggling 117 HD, or Low Detail, either on or off has no effect on this.
Screenshots or videos

RuneLite version
Runelite Version <IP_ADDRESS>
Launcher version 2.4.5
Logs
No response

duplicate of #6938


EditableText can't be implemented for user-defined types

Someone pointed out in #1781 that EditableText will eventually be removed. However, for the time being I would like to use it and migrate my code if necessary.
The AppState in my application so far looks like this:
#[derive(Clone, Data, Lens)]
struct AppState {
    github_user: GithubUser,
    #[data(ignore)]
    manually_maintained_time: ManuallyMaintainedTime,
}

where GithubUser is defined like this:
#[derive(Clone, Data)]
pub(crate) enum GithubUser {
    Unknown,
    SigningIn(SignInState),
    SignedIn(String),
}

where in turn SignInState is defined like this:
#[derive(Clone, Data)]
struct SignInState {
    github_username: String,
    // off-topic: maybe Instant should implement Data. I'm still new to Druid so Idk
    #[data(ignore)]
    initiated_at: Instant,
}

As you can notice, any instance of enum can be used as editable text. For example, a GithubUser::Unknown can be turned into GithubUser::SigningIn which has a String field.
So I wanted to implement druid::text::EditableText for GithubUser, yet I figured out that I can't do this because druid::text::StringCursor doesn't have a public constructor.
Implementation of druid::text::EditableText for user-defined types requires a public constructor for  druid::text::StringCursor because required method EditableText::cursor generally requires returning Some(StringCursor<'a>).
Is there a rationale behind lack of public constructor for druid::text::StringCursor or should I make a PR with an implementation?

In this particular case, I can relay the implementation of all these methods to the implementation of String. Closed


standalone package for src/providers/

I asked this from the Discord a while ago, I'd like to create an issue to track this:
in order to minimize dependencies using the web3modal project, would it be possible to have standalone package that include code from src/providers?
Thank you for the works.

definitely in the roadmap 👍 I will keep Discord posted on major updates

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.


Feature request: ENS integration

Do you want to request a feature or report a bug?
Request a feature.
What is the current behavior?
I have a list of transactions with "from" and "to" address fields. When I render them in my app, I fetch the ENS domain after the UI loads, so there is a glitch for the time it takes to fetch the data.
What is the expected behavior?
Ideally, I could add fields for the ENS domains (if any) for "from" and "to". I know about the ENS subgraph, but I'm not sure if I can query that data from my subgraph mappings.

You can't query that data from the subgraph right now but we're working on a feature called subgraph composition right now that will allow your subgraph to reference entities in other subgraphs
This is the tracking issue: https://github.com/graphprotocol/graph-node/issues/1202

This is now possible :rocket: https://github.com/graphprotocol/ens-rainbow.

Cc @razgraf and @gavriliumircea - this might be useful in V2 Interfaces.


Fix certification failure for Commercetools Connect

Move c8 to devDependencies in package.json as it uses vulnerable dependencies
Exclude integration test and e2e test from npm run test command for Commercetools Connect, and execute npm run test-ci in Github Action
Use mock config in notification module unit test
Update documentation

You don't need to run Integration and e2e tests completely in the main branch?
I think still there are other users using this repo. May you could consider ignoring only those tests which are failing at the moment. We can fix them later(Try to create a issue for our reference).

I misunderstood the changes, Yes the code looks good for me.

You don't need to run Integration and e2e tests completely in the main branch?
I think still there are other users using this repo. May you could consider ignoring only those tests which are failing at the moment. We can fix them later(Try to create a issue for our reference).

The change in this PR would not affect the integration test and e2e test in CI/CD. Since Commercetools Connect does not get the adyen config from env-var during build, that's why we add more npm commands (npm run test is now dedicated for Commercetools Connect).


how to get current month

I am using this directive,i want to fetch current month,year in my js.
Also if user clicks on next month then i want to get the month and year value in js to execute my custom logic.
I have added
ui-calendar="uiConfig.calendar" calendar="myCalendar1" class="span8 calendar" ng-model="eventSources"
but myCalendar1 is not available in my controller scope.

$scope.calendarConfig = {
calendar: {
allDaySlot: false,
timezone: 'local',
editable: true,
lang: 'de',
eventOrder: "-title",
height: 650,
header: {
left:   'title',
center: 'Custom text',
right:  'today prev,next'
},
eventClick: $scope.eventClick,
eventResizeStop: $scope.alertResize,
eventDragStop: $scope.alertDrag,
eventRender: $scope.eventRender,
dayClick: $scope.dayClick
}
};

I tried to get uiCalendarConfig.calendars.medi_calendar but getting error uiCalendarConfig not configured.


Upgrade to logstash 1.5.0 rc3

Upgrades logstash from 1.5.0-rc2 to logstash 1.5.0-rc3, which:

Has performance improvements
Allows plugins to be installed from local gem files (meaning tha the logstash package compilation should no longer need internet access; and is marginally faster)

@dpb587 - could you review this please.
As best I can tell there are no breaking changes here - the logsearch-for-cityindex test pass with no changes.

Looks fine. I think you should deploy it to meta before merging.
Reminds me we still need to figure out plugin support.

I'll get this deployed to meta shortly.
Concerning plugin support, I think we copy the way we allow adding plugins to elasticsearch by listing them in your manifest.  I wish there was a way we could install plugins at compile time rather than deploy time; but I can't think of one.

I think this was successfully deployed and safe to merge, right?


Plugin doesn't run for project references

Much like tslint and (to an extent) ts-loader, it seems this project doesn't yet fully support Typescript>=3.0.0 style project references. If I introduce an error into a dependent project, fork-ts-checker-webpack-plugin doesn't find the error.

If you'd like to have a crack at adding support you'd be most welcome! cc @andrewbranch - author of project reference support in ts-loader in case he has any thoughts 🌻♥

I believe that, like TypeStrong/ts-loader#851, this ultimately requires the new Typescript APIs yet to be implemented and tracked by Microsoft/Typescript#28640.
I may look into making such a contribution once the APIs are available, to learn them for my own needs; but I don't think it's worthwhile to do so before then.

The new API was added in https://github.com/microsoft/TypeScript/pull/31432 and has been released in Typescript v3.6 so this should be unblocked now. Unfortunately, I don't know enough about how this plugin or the typescript API works to really tackle this myself yet.

Is there any updates on this? Would like to be able to use this & ts-loader with projectReferences.

Currently, I'm focusing on refactoring and I hope I will release an alpha version in the next month. It's a huge task and I have limited time to work on it :)

Hey all,
The amazing @sheetalkamat is busy improving the  project references support in ts-loader in https://github.com/TypeStrong/ts-loader/pull/1076
Part of the discussion is around how transpileOnly should work. Feel free to be part of the discussion 🌻🥰

@MLoughry , @noahm , @sikloidz , @Ivaylo-Lafchiev
Please try fork-ts-checker-webpack-plugin@alpha - I've published a new version which supports project references 🚀
I will close this issue to clean-up the backlog :)


Manual/Introduction link inconsistencies lead to 404 page

When on the Manual's "Introduction" page -- coming from the home page's topbar link -- clicking on any of the links in the Feature highlights section redirects users to a 404 page.
For example, clicking on web platform functionality redirects to https://deno.land/runtime/web_platform_apis.md (a 404), rather than https://deno.land/manual/runtime/web_platform_apis (the correct one).
Notably, the issue occurs only when starting from https://deno.land/manual but works fine when starting from https://deno.land/manual/introduction. For further information, the first link is what we get when you click "Manual" from the home page's links, and the second is what we get if we click "Introduction" in the table of contents sidebar when already on the Manual page.
I've tested on Safari, Chrome, and Firefox.
Seems to be similar to #2128, but the links specified there seem to have been fixed. Also, I think the main issue here is the inconsistency between https://deno.land/manual and https://deno.land/manual/introduction.

Forgot to mention in original post, but I am currently looking into the issue to hopefully make my first contribution :)


Add unit tests for tls authority logic

Adds unit tests and intruces a few minor changes to improve logging and make the test work as expected. Should make the logs a lot more insightful.
Kind
/kind cleanup
Release Note
NONE

/remove-kind bug

/kind cleanup

/lgtm

/approve