Make misspells with a single correction inline suggestion blocks

Github has a neat feature to suggest code changes on PR lines: https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request
For simple misspells where there is only 1 correction option, it'd be convenient to have them as a suggestion which the reviewer can simply click "Commit suggestion" on.

It seems like this might be a good way to achieve the same thing: https://github.com/marketplace/actions/action-git-diff-suggestions

https://github.com/getsentry/action-git-diff-suggestions

It seems like this might be a good way to achieve the same thing: https://github.com/marketplace/actions/action-git-diff-suggestions

Is there an echo (echo, echo, echo...) in here? :laughing:

My bad -- I saw this today and wanted to make a note of it without re-reading the comments 🙃

In your defence at least they're different URLs!
I keep meaning to try this on the main Codespell repo against the dictionary sort stuff, given I do such a bad job of guessing sorting when submitting entries.

I've re-opened this as while I was joking around, it's still a good idea and something we should try and do (or hopefully GitHub will see sense and finally enable it in due course).
My brief tests on the sorting didn't look too promising unfortunately:
https://github.com/codespell-project/codespell/pull/2253


Broken link in README

The top-level readme has a broken link:
https://github.com/supabase/gotrue-js#docs
There's a 404 at "Using gotrue-js: https://supabase.io/docs/gotrue/client/initializing"

Thanks for reporting this issue!


Change the button color in dark mode

Change the button color in  dark mode to dark green or any color that matches the dark background

Can you please assign this task to me?

can you please assign this to me ?

Done go ahead

Can you please assign this task to me?

@adithyapaib can I change button color to red in dark mode?

I am sorry this is my first time contributing in this type of project. So, can't I contribute in this now?

I have made some changes. Please, checkout the PR.

Hello @adithyapaib please assign me this issue.

Hello @adithyapaib please assign me this issue.

Done

Please assign me

Please assign me

Done

please assingn me this issue

Please Assign me this issue

@adithyapaib can you please assign this task to me?

I have made some changes PLease check.

Can you please assign this issue to me. I can fix this

@adithyapaib I have made some changes. Please check.

Please can you assign this issue to me? I can fix this


adding new icons

@bobbennett looks good to me.  also, time to cut a v1.0.0 release.  I want to remove IE8 specifics builds in v2 and also upgrade some libs.


BGPViewerCore.Models are not deserializing correctly with Newtonsoft.Json in Xamarin.Forms

When trying to deserialize AsnDetailsModel using Newtonsoft.Json the result is always null

Problem was not on BGPViewerCore.Models and neither on Newtonsoft.Json. Issue closed.

Problem was not on BGPViewerCore.Models and neither on Newtonsoft.Json. Issue closed.


Dark mode and CSS cleanup for claude, bing, and chatGPT

This PR:

Forces Chat GPT into dark mode
Does CSS changes to create a dark mode for bing and Claude 2
Deletes some unnecessary UI clutter from Claude and Bing

Much better experience on smaller screens now imo! Please take a look and merge if you agree!

super nice work!


Patch 22

Updated 2 outcomes:

Same sex - user in Sweden - partner other
Opposite sex - user in Sweden - partner other

Closed, as this has been superseded by #2885
Thank you for your help


[C] Fix client image reference counting.

Noticed that image reference count in the C client sometimes had funny values around<PHONE_NUMBER>. Turns out that our macro with -1 literal gets compiled to:
58	    AERON_GET_AND_ADD_INT64(res, var, -1);
   0x000000000000c74b <+39>:	b8 ff ff ff ff	mov    $0xffffffff,%eax
   0x000000000000c750 <+44>:	f0 48 0f c1 45 e8	lock xadd %rax,-0x18(%rbp)

So instead of adding -1, we're adding<PHONE_NUMBER> (0xffffffff).
With this change, it looks like this:
51	    AERON_GET_AND_ADD_INT64(res, var, -1);
   0x000000000000c6c9 <+39>:	48 c7 c0 ff ff ff ff	mov    $0xffffffffffffffff,%rax
   0x000000000000c6d0 <+46>:	f0 48 0f c1 45 e8	lock xadd %rax,-0x18(%rbp)

This is a good spot and we should implement the change.  We should also be more careful with our literals in the C code.  In this case we should also be changing the call site to:
AERON_GET_AND_ADD_INT64(res, var, INT64_C(-1));

I'm wondering we could use _Generic to introduce some type safety here as well.


Respect default ActionCable behaviour for missing connectionUrl

createConsumer accepts optional URL to WebSocket. Otherwise, it tries to read  action_cable_meta_tag in HTML layout HEAD or falls back to the default /cable. This change makes the library to respect this behavior naturally.
Previously, it was possible only by passing () => null function to connectionUrl option only.

Looks good! Thanks for contributing @sponomarev


python310Packages.boxx: 0.10.8 -> 0.10.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.boxx is: Tool-box for efficient build and debug for Scientific Computing and Computer Vision
meta.homepage for python310Packages.boxx is: https://github.com/DIYer22/boxx
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.10.9 with grep in /nix/store/8v641f3jkbjdj0mll1d79sz2zpj65zg5-python3.10-boxx-0.10.9
found 0.10.9 in filename of file in /nix/store/8v641f3jkbjdj0mll1d79sz2zpj65zg5-python3.10-boxx-0.10.9

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.boxx
python310Packages.bpycv
python310Packages.zcs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8v641f3jkbjdj0mll1d79sz2zpj65zg5-python3.10-boxx-0.10.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.boxx https://github.com/r-ryantm/nixpkgs/archive/296282cb0a70d5010099aef59a5456967a87d0ca.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8v641f3jkbjdj0mll1d79sz2zpj65zg5-python3.10-boxx-0.10.9
ls -la /nix/store/8v641f3jkbjdj0mll1d79sz2zpj65zg5-python3.10-boxx-0.10.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

python310Packages.boxx
python310Packages.bpycv
python310Packages.zcs

Maintainer pings
cc @lucasew for testing.

Note for anyone else: the reason this package is in nixpkgs (AFAIK) is because of bpycv
If bpycv passthru test passes, then it's ready to merge


Add contribution guidelines and pull request template

          > We do not allow merge requests into master directly

OK. To make it easier for contributors to know your workflow, I would suggest adding some GitHub contribution guidelines and/or adding a Pull Request template to your repository, then.
Originally posted by @dwatteau in https://github.com/pedrolcl/sonivox/issues/29#issuecomment-2528127227

First step: switched the default branch to 'devel'


Bug: Segmentation fault: WITH / OPTIONAL MATCH / WHERE EXISTS

Kùzu version
nightly
What operating system are you using?
Fedora 40
What happened?
Can be reproduced by:
CREATE NODE TABLE V(id INT, PRIMARY KEY(id));
CREATE REL TABLE links_to(FROM V to V);
CREATE REL TABLE parent(FROM V to V);

CREATE (:V {id: 1}), (:V {id: 2});

WITH 2 as parent_id
OPTIONAL MATCH (v1:V)-[:links_to*0..]->(:V)
WHERE EXISTS {MATCH (v1)-[:parent]->(:V {id: parent_id})}
RETURN v1;

Are there known steps to reproduce?
No response

Hi @sapalli2989, this should be fixed in #4088. Thanks for reporting.


Memory Leak in EF Core?

Hi, My project has a big query with linq queryable, and I have memory leak issues, when I execute my query, memory usage goes up each every request, no GC.Collect helped me. however I used using keyword for my context in every requests.
I uploaded the dump file you can check it in vs 2017 enterprice and see where is the memory leak problems, I uploaded an attached image too.
ef core nuget package version:
3.0.0-preview.19074.3

dump file

full query:
private static MessageContract<List<ShopInfo>> FilterShopInfoes(ShopFilterInfo filterInfo)
        {
            using (ExpertContext context = new ExpertContext(false))
            {
                IQueryable<ShopContractInfo> query = context.ShopInfoes
                    .Include(x => x.FileInfoes)
                    .Include(x => x.CategoryInfo)
                    .AsNoTracking()
                    .AsQueryable().Select(x => new ShopContractInfo()
                    {
                        Rate = x.RateInfoes.Select(y => (double)y.Rate).DefaultIfEmpty(0).Average(y => y),
                        CommentCount = x.CommentInfoes.Count,
                        ProductsVitrinCount = x.ProductInfoes.Count(y => y.IsVitrin),
                        ProductsNotVitrinCount = x.ProductInfoes.Count(y => !y.IsVitrin),
                        Comments = x.CommentInfoes.AsQueryable().SelectPageAsQueryable(0, 10),
                        ShopInfo = x
                    });
                query = query.Where(x => x.ShopInfo.IsEnabled);

                if (filterInfo.UserId.HasValue)
                    query = query.Where(x => x.ShopInfo.UserId == filterInfo.UserId);

                if (filterInfo.StartDateTime != null)
                {
                    DateTime date = filterInfo.StartDateTime.Value.Date;
                    query = query.Where(x => x.ShopInfo.CreatedDateTime >= date);
                }
                if (filterInfo.EndDateTime != null)
                {
                    DateTime date = filterInfo.EndDateTime.Value.Date;
                    query = query.Where(x => x.ShopInfo.CreatedDateTime <= date);
                }
                if (!string.IsNullOrEmpty(filterInfo.Text))
                {
                    query = query.Where(x => x.ShopInfo.Name.Contains(filterInfo.Text));
                }

                if (filterInfo.ShopSaleType.HasValue)
                {
                    query = query.Where(x => x.ShopInfo.SaleType == filterInfo.ShopSaleType || x.ShopInfo.SaleType == ShopSaleType.Both);
                }

                if (filterInfo.CanSend.HasValue)
                {
                    query = query.Where(x => x.ShopInfo.CanSend == filterInfo.CanSend);
                }

                if (filterInfo.CategoryId.HasValue)
                {
                    query = query.Where(x => x.ShopInfo.CategoryId == filterInfo.CategoryId);
                }
                if (!string.IsNullOrEmpty(filterInfo.CityName))
                {
                    query = query.Where(x => x.ShopInfo.CityName == filterInfo.CityName);
                }
                query = query.OrderByDescending(x => x.ShopInfo.CreatedDateTime);

                List<ShopContractInfo> result = null;
                if (filterInfo.Latitude.HasValue && filterInfo.Longitude.HasValue)
                {
                    MessageContract<(double X1, double Y1, double X2, double Y2, double Length)> rect = GetEarthRectangle();
                    Rectangle rectangle = new Rectangle(new Point(rect.Data.X1, rect.Data.Y1), new Point(rect.Data.X2, rect.Data.Y2));
                    int index = RectangleUtil.GetRectIndexByPoint(new Point(filterInfo.Latitude.Value, filterInfo.Longitude.Value), rectangle, rect.Data.Length);

                    int[] indexes = RectangleUtil.GetListOfIndexes(index, rectangle, rect.Data.Length).ToArray();
                    query = query.Where(x => indexes.Contains(index));
                    result = query.ToList();

                    GeoCoordinate sCoord = new GeoCoordinate(filterInfo.Latitude.Value, filterInfo.Longitude.Value);
                    IEnumerable<ShopContractInfo> query2 = result.Where(x => CheckIsNear(x.ShopInfo, sCoord));
                    result = query2.SelectPage(filterInfo.Index, filterInfo.Length);
                }
                else
                {
                    result = query.SelectPage(filterInfo.Index, filterInfo.Length);

                }

                foreach (ShopContractInfo x in result)
                {
                    x.ShopInfo.Rate = x.Rate;
                    x.ShopInfo.CommentCount = x.CommentCount;
                    x.ShopInfo.ProductsVitrinCount = x.ProductsVitrinCount;
                    x.ShopInfo.ProductsNotVitrinCount = x.ProductsNotVitrinCount;
                    x.ShopInfo.CommentInfoes = x.Comments.ToList();
                }

                return result.Select(x => x.ShopInfo).ToList().Success();
            }
        }
        private static bool CheckIsNear(ShopInfo shopInfo, GeoCoordinate sCoord)
        {
            if (!shopInfo.Latitude.HasValue || !shopInfo.Longitude.HasValue)
                return false;
            GeoCoordinate eCoord = new GeoCoordinate(shopInfo.Latitude.Value, shopInfo.Longitude.Value);
            double distance = sCoord.GetDistanceTo(eCoord);
            if (distance <= SettingInfo.GetSettingInteger("NearShopDistance"))
            {
                return true;
            }
            else
                return false;
        }

        public static MessageContract<(double X1, double Y1, double X2, double Y2, double Length)> GetEarthRectangle()
        {
            return (40.011517, 43.570553, 23.613212, 63.823850, 0.0063613231552163).Success();
        }

What is your connection string? Specifically do you have MARS enabled? How much data is on server side? Given the shape of query, there would be a lot of client evaluation.

my connection string is :
Server=.;initial catalog=Expert;User ID=uid;Password=pass
I don't know what is MARS.
data is not large my database is 80MB but users request is comming always,i have more servers just for this server i have this problem.

@Ali-YousefiTelori - Just for info, this is documentation about MARS if you are interested.
By default MARS is off. (looking at your connectionstring that would be case here too). Further since there is collection include we do send multiple queries causing one of the query to buffer. Also there is a lot of client evaluation involved so I would suspect large amount of data is being fetched from server.
There are few things which will improve above scenario in 3.0 release. First we won't be doing so much of client evaluation, hence unexpectedly fetching large data from server would be gone. Further according to #12098, we will use single query for includes. What that means is, we wouldn't need buffering internally. Hence in your case, a no-tracking query would not get/store more data than what is current result row is.

@Ali-YousefiTelori take a look at this #13048 issue, perhaps it's related

@smitpatel  after I activated MARS I think problems gone, my app after 2 days is 80MB in memory. Thank you for your answer

Duplicate of #12098

MARS feature not works too, but its better, i have two reset my server every week because of memory leack problem,


Fix for SqlQuery throwing exception when T is decimal

See issue

Hi @jaspervanderwoude sorry for the 3-month delay, I've been on many subjects at work and completely forgot that. I'll review and integrate this today

Closed by #10


Remove Specialist Publisher from post-data sync run

We're not going to be writing any more migrations for
the old specialist publisher so this can go.

:+1: fftm


ARROW-9285: [C++] Detect unauthorized memory allocations in function kernels

12089

 Quality gate: Failed
20 Issues

 7 Vulnerabilities
 13 Code issues

Details

 Quality gate: Failed
6 Issues

 6 Vulnerabilities

Details


Improving the documentation of enums as discriminators test

@Ocramius was too eager to merge stuff
Related to: https://github.com/doctrine/doctrine2/pull/6141

👍


Docker build broken

Thanks for sharing this cool repo! Excited to try it out.
Following the directions for building in Docker fails with
[+] Building 0.6s (5/12)                                                                                docker:default
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 675B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ros:rolling                                                                                0.3s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => CACHED [1/9] FROM docker.io/library/ros:rolling@sha256:42f7b5ac83c674286dcc2e055226b8fb6deff6f2dcc57e82f58dc7b6ac335f96                           0.0s
 => ERROR [2/9] RUN apt-get install -y     git     python-is-python3     python3-pip                                                                  0.2s
------
 > [2/9] RUN apt-get install -y     git     python-is-python3     python3-pip:
0.181 Reading package lists...
0.191 Building dependency tree...
0.192 Reading state information...
0.193 E: Unable to locate package python-is-python3
0.193 E: Unable to locate package python3-pip
------
Dockerfile:8
--------------------
   7 |     ###### Install package dependencies
   8 | >>> RUN apt-get install -y \
   9 | >>>     git \
  10 | >>>     python-is-python3 \
  11 | >>>     python3-pip
  12 |
--------------------
ERROR: failed to solve: process "/bin/sh -c apt-get install -y     git     python-is-python3     python3-pip" did not complete successfully: exit code: 100

It looks like there might be several problems to address:

Need to run apt-get update
Need to specify PCL as a dependency in package.xml
Need to run rosdep install to populate this dependency in the image

I haven't had time to debug this more fully, but if I'm able I'll put up an MR with any required modifications.

Hey  @playertr! Thanks for your comments.
My mistake in not properly updating the Dockerfile. In the commit bbff036 I just fixed.
Remember to use the Kitti dataset to test this. In the following versions I will leave a configuration file to make the codes more modular.
Stay tuned for the next progress that I will be uploading.

Hey @playertr! Thanks for your comments.
My mistake in not properly updating the Dockerfile. In the commit bbff036 I just fixed.
Remember to use the Kitti dataset to test this. In the following versions I will leave a configuration file to make the codes more modular.
Stay tuned for the next progress that I will be uploading.


Named export 'AuroFormValidation' not found

Please describe the bug
Library has issue with named export and not supporting ESM/SSR development
@aurodesignsystem/auro-input/src/base-input.js:20
import { AuroFormValidation } from<EMAIL_ADDRESS>         ^^^^^^^^^^^^^^^^^^
SyntaxError: Named export 'AuroFormValidation' not found. The requested module<EMAIL_ADDRESS>is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from<EMAIL_ADDRESS>
Reproducing the error
This issue cannot be reproduced on the Auro docsite
Expected behavior
No error log
What browsers are you seeing the problem on?
No response
Additional context
There is an issue with how this is also being imported into auro-input
Exit criteria
This issue can be closed once the default named import issue has been addressed and the library is SSR/ESM compatible.
Once this issue is corrected and released, the following issue can be resolved.

https://github.com/AlaskaAirlines/auro-input/issues/298

It also appears that default settings and issue templates are also out of sync with this repo.


ACCUMULO-4328: Run multiple tablet servers per host

Backport of ACCUMULO-925 to enable pid support.
Modification to TabletServer.java to put the pid into the MDC
Modification to generic_logger to put the pid of the process into the log
Modification to environment to be able to define NUM_TSERVERS per host
Modification to environment to be able to define NUMA parameters per tserver per host
Modification to start/stop scripts to handle multiple tservers per host
To be able to run multiple tservers per host, tserver.port.client must be zero in accumulo-site.xml

An entirely different way to achieve higher utilization is to run multiple tservers on a single node using using containers and mesos.  Not advocating that option over this, just something I thought while trying to think of ways to have multiple log files per tserver on a single node.  This is one option that will give multiple log files, with lots other baggage!   @mikewalch is currently experimenting with this.

I will re-submit this against 1.8


Translate April 17 report into Russian

Hello @nextstrain/russian!
The situation report from 2020-04-17  is ready to be translated to Russian. Sorry it’s a little late this week. The report focuses on the USA. We do have a significant linguistic diversity here, but I’ll let you decide if you want to translate it. In general, I do think it’s worth understanding how the virus spreads in other regions of the world, but I’m not doing the hard work of translation.
Branch to translate on: sit-rep_ru
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_ru/narratives/translations_ncov_sit-rep_ru_2020-04-17.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_ru/ncov/sit-rep/ru/2020-04-17
Twitter post to translate:
“This week’s #COVID19 situation report covers the outbreak in the United States. It is now available in Russian. Thank you @, @, and @ for the rapid translation.”
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your russian is here: https://github.com/orgs/nextstrain/russians/russian
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate Russian version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with a tab, dash, and new line, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Russian; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.
As always a hearty thank you! I am really enjoying working with you all, even if it’s just digitally. Take care!

I can't open the Discussion board following the link in this post. Could someone please send me a working link?

I can't open the Discussion board following the link in this post. Could someone please send me a working link?

I believe this is the right link https://github.com/orgs/nextstrain/teams/russian

That worked, thank you @dtaliun !

@cassiawag I just noticed that the link on how to read phylogenetic trees leads to the English version of the document. I wasn't sure it is replaced with Russian version of the document later, but I figured I'll let you know. Thanks!

@irina95511 Could you please review?

@cassiawag We still need to review the translation, but here is the twit in Russian:

@frogtraveler @dtaliun I can have a look within 4ish hours! Sorry that I was very busy last weekend and have not helped earlier!

@frogtraveler @dtaliun Great job! Again, I did not have much to do.
@cassiawag The reviewing is done, all good!

Published and tweeted: https://nextstrain.org/narratives/ncov/sit-rep/ru/2020-04-17
Thank you all!


Misconception about Identity Synchronization

The architecture itself is just amazing and it shows how migration plans should be. However, I see some misconception about Id synchronization because Active Directory why it's not and should be not one of the candidates to replatform to Azure Active Directory.
I understand that the picture represents synchronization maybe through Azure AD Connect but still does not show how does an enterprise grade solution actually look like. In enterprise-grade environments, you have lots of legacy systems that still relies on kerberos authentication and, for those, Azure Active Directory Solutions is not be the best way.
Using this kind of topology makes Authentication be a SPOF (Single Point Of Failure), which is unacceptable in most of mission critical environment.
What I'd suggest to you guys is updating this architecture using Domain Controller Replica as a solution for Active Directory or even Azure AD Domain Services as a potential candidate. Only this way it's possible to avoid this potential SPOF and garantee a friction free migration to the cloud.
Thanks in advance!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 018a7aef-3117-17c5-8693-7455cefa54d3
Version Independent ID: a6c4e7a0-3cfa-1c4c-bc48-6281c870b035
Content: Enterprise-scale disaster recovery - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/disaster-recovery-enterprise-scale-dr.md
Service: architecture-center
GitHub Login: @adamboeglin
Microsoft Alias: pnp

@cdanieloliveira - would you be willing to offer a PR on the topic describing what you'd like to see and we'd be happy to review it and run it past the AD product team! Closing for now as the article is technically accurate, but that sounds like a great idea for improvements.


Minor updates

I appreciate your repo - these are just minor changes mainly to remove a linting warning.

Hello, thanks for fix the typo!


Add ability to control whether the server starts or not even when single_file_support=true

Language server
N/A
Requested feature
I have denols and vtsls for typescript.
I'm implemented the following root_dir function.
  local function deno_or_node(prefer)
    return function(fname)
      local type = {
        deno = require('lspconfig').util.root_pattern("deno.json")(fname),
        node = require('lspconfig').util.root_pattern("package.json")(fname),
      }
      local other = prefer == 'deno' and 'node' or 'deno'
      if type[prefer] and type[other] then
        local is_other = #vim.split(type[other], '/', { plain = true }) > #vim.split(type[prefer], '/', { plain = true })
        if is_other then
          return
        end
      end
      return type[prefer]
    end
  end

But I have to specify single_file_support=false for this.
It's useful if lspconfig provided a function to determine whether to attach or not.
Other clients which have this feature
No response

need add another condition in this line to determine whether to attach or not
https://github.com/neovim/nvim-lspconfig/blob/63f4c0082f007d2356df4bc335f55e6d414da89c/lua/lspconfig/configs.lua#L125

It's useful if lspconfig provided a function to determine whether to attach or not.

on_attach ?

The on_attach can't be used to determine the server should/shouldn't attach to the buffer.
But My intension is this. I think it's useful if we have should_attach: func... for this situation.

add new field not a good way. in on_attach you can do it like this.
on_attach = function(client, bufnr)
     if not client.config.root_dir then
        local cwd = vim.loop.cwd()
        if some condition here then
            vim.lsp.stop_client(client.id)
        end
    end
end

I'm a bit concerned about calling M._setup_buffer afterward even if I stop the client in on_attach, but the advice seems to meet my needs. thank you!
I think this issue is in a state to be closed if necessary.

yet another way is set the server.auto_start to false then do logic in FileType autocommand callback
tsserver.auto_start = false

vim.api.nvim_create_autocmd('FileType', {
     pattern = foo,
     callback = function()
          if logic then
              require('lspconfig.configs')['tsserver'].launch()
          end
     end

Nice! Should we add the above as a hint to the readme / :help lspconfig ?

I think require('lspconfig.configs')['%SERVER%'].launch() is ideal solution.
Thank you again!

It may be a documentation issue, but I'm a little concerned that launch actually means not only starting the server, but also attaching to the current buffer.

it will attach current buffer to this client. in this way start client or not determine what  the your logic condition is
lspconfig.tssserver.setup({
       -- config this lspconfig won't set the FileType event to start server you need start server manually
        auto_start = false
})

vim.api.nvim_create_autocmd('FileType', {
    pattern = 'typescript'
    callback = function()
        local cwd = vim.loop.cwd()
        if vim.fn.filereadable(cwd ..'/deno.json') == 0 then
             require('lspconfig.configs')['tsserver'].launch()
        end
    end

I tried this approach but it didn't work.
It seems that once nvim-lspconfig attaches to a particular file type, it automatically attaches the next time you open a buffer of the same file type.
And in that case, checks such as root_dir are still performed, so deno/vtsls isolation does not work well.
It seems to me that there should be a setting like should_attach.
Google Translation
thank you for the advice! I'll explore this.

vim.api.nvim_create_autocmd('FileType', {
    pattern = 'typescript',
    callback = function(opt)
        local cwd = vim.loop.cwd()
        local client
        for _, item in ipairs(vim.lsp.get_active_clients(opt.buf)) do
          if item.name == 'tsserver' then
            client = item
            break
          end
        end
        local condition = vim.fn.filereadable(cwd ..'/deno.json') == 0
        if client and condition then
          vim.lsp.buf_attach_client(opt.buf, client)
        elseif not client and condition then
          require('lspconfig.configs')['tsserver'].launch()
        end
    end
  })

In my investigation, the lspconfig adds the autocmd at https://github.com/neovim/nvim-lspconfig/blob/master/lua/lspconfig/configs.lua#L104.
So tsserver will be attached on all of new typescript buffer.
I think the current lspconfig doesn't have an ideal solution for this situation.
I will consider this problem and lspconfig.
Along the way, I may choose not to use lspconfig.
First of all, I'd like to see if lspconfig could be improved.

IMO, the autocmd still be added in L104.
If we specify autostart=false, root_dir = nil, the autocmd wont added.
But root_dir=nil was specified, workspaceFolder wont be initialized correctly.

nope what i mean is add a patch to check create this event BufReadPost  or not .if autostart to false lspconfig shouldn't create any event.

+1 to the should_attach idea. I think I came to the same conclusion as hrsh7th prior to finding this issue. You can be in a state where both LSPs should be running in your editor, but you still want to micromanage their attachments to different buffers. Lspconfig provides lots of tools for controlling language server start/stop, but not attach/detach.
Using on_attach could work if it were possible to call vim.lsp.buf_detach_client() here, but neovim doesn't actually mark the client as attached to the buffer until after on_attach and LspAttach have been fired. This Reddit comment describes using vim.schedule() or vim.defer_fn() to work around the limitation, but this would be much cleaner if you had fine-grained control over which buffers lspconfig tries to attach to in the first place.

@NReilingh Bump


Figure out Testing Infrastructure

Find some sort of way to run tests, and ensure that things actually work!
Maybe create an in-memory MongoDB (as per @anandani4136 's suggestion and go from there)? Maybe Jest?

https://github.com/nodkz/mongodb-memory-server Is what we should use, and it will allow us to run tests in parallel. We can also use jest.config.js to globally define the beforeAll and afterAll required so it's define in one central place.
Jest is fairly easy to set up
Example Code
// tests/your-module.test.ts
import mongoose from 'mongoose';
import { MongoMemoryServer } from 'mongodb-memory-server';
import YourModel from '../src/your-module';

let mongoServer: MongoMemoryServer;

beforeAll(async () => {
  mongoServer = new MongoMemoryServer();
  const mongoUri = await mongoServer.getUri();
  mongoose.connect(mongoUri, {
    useNewUrlParser: true,
    useUnifiedTopology: true,
  });
});

afterAll(async () => {
  await mongoose.disconnect();
  await mongoServer.stop();
});

describe('YourModel', () => {
  it('should save a document', async () => {
    const doc = new YourModel({ name: 'Test' });
    await doc.save();
    const foundDoc = await YourModel.findOne({ name: 'Test' });
    expect(foundDoc).toBeTruthy();
  });
});


RT-2.5: IS-IS Hello-Padding Globally disabled

Deprecated in version 1.5 of the testplan. Hence closing the issue.


Console is broken for non-primitive types

That is because "TypedArrayPrototypeSymbolToStringTag" is not defined.

Fixed by https://github.com/denoland/deno_core/pull/572


Issue #2723: ListItem when disabled className is not returned

Added in className to _createDisabledElement in list-item.jsx so a class can still be assigned even if the list item is disabled.

Sorry, my this is my first pull request this is related to #2723

@mikedklein That's a good start :+1:.

@oliviertassinari ok adding {...other} fixed all the issues in your comments as well as offering more extensibility for a disabled ListItem Component.

@oliviertassinari so I am passing the the right other variable now but I am not sure the name I am using in the createDisabledElement function is what you would want https://github.com/callemall/material-ui/pull/2726/files#diff-8bb668c32f1e189eb250efe330c28070R251 .

@mikedklein Looks like there is an issue with your rebase:
-1,248
+1,039

@oliviertassinari ok I think I got it. I used the same naming convention that was used on the _pushElement Method here. Let me know if this naming convention is proper, and if the ...other variable from render was passed properly.

@mikedklein I haven't tested it yet, but that seems almost good :+1:.
(If you have time and If you want to prevent regression, adding a unit test is welcomed)

@oliviertassinari awesome, definitely should I do that in a separate PR? Also, do you have an example of a good one I should follow?

definitely, I do that in a separate PR?

I'm happy to hear it :smile:! Yes, that would be great to be in this PR. So that we can rely on a green travis task.

do you have an example of a good one I should follow

Have a look at https://github.com/callemall/material-ui/blob/master/test/checkbox-spec.js#L17.

@oliviertassinari not sure if these tests are going in the direction that you were thinking. One thing I just realized is testing for types is probably a waste since we should be using propType in most components.

I think that there is not point to use listItem.props in the test expect. You are testing the property that are given to the component. That's not usefull. It would be more intereseting to so how this property in used by the render method of the ListItem.

Totally agree. After thinking about it some more I realized I wasn't really doing anything different then React was with it's proptypes and realized the rendered component was "actually" getting tested. Do you have any pointers on debugging the test so I can inspect the element I am grabbing and try to get at the properties on that element?

Do you have any pointers on debugging the test so I can inspect the element I am grabbing

Have a look at the method provided by
import TestUtils from 'react-addons-test-utils';

Awesome thanks, I was able to get it I think. It looks like classList is not behaving properly in phantomJS as
expect(itemDiv.classList[0] === testClass).to.be(true);

works in chrome but in phantom I am getting
TypeError: '[object Object]' is not a function (evaluating 'expect(itemDiv.classList[0] === testClass).to.be(true)')
        at /var/folders/6d/wg781t7s5xq23g6s2f7d8qdh0000gn/T/ef6525c76d6b744a9b051d848604b73c.browserify:42046 <- test/lists/list-item-spec.js:23:4

I think this might be the issue. https://github.com/ariya/phantomjs/issues/12782 . There are couple workarounds in there but they don't look like good solutions for what we are trying to do. Any ideas? I can still get the className property of the object maybe we will have to use that for now?

@oliviertassinari ok so I have redone the tests and figured out the class problem. I still want to write the tests for displaying an icon, avatar, etc in a list-item. But I wanted to get some feedback before I went too far to make sure I am on the right track.

ok so I have redone the tests and figured out the class problem.

Awesome :+1:.

I still want to write the tests for displaying an icon, avatar, etc in a list-item.

This can be done in another PR. Let's stick to the classname here.

Great! Just let me know if anything needs to be fixed and I will get on it!

Can you squash down the commit, so we keep the history clean?
Aside from that, I think we can merge the PR.

Alright all squashed into one commit from me related to this PR. :+1:

@mikedklein Thanks :tada: !


rename function if it's a reserved word

so, for whatever reason I was working with a node module that had a NODE_SET_METHOD(obj, "delete", deleteFn);
as a result thenify would not be able to export that function, ever -- just because of it's name. no workarounds
it's pretty unlikely that anyone will run into this -- but here it is just in case.. I'm convinced the only way to set a function name to a reserved word, is with C++
cheers

is there a module for this? not really down with having a list of words in this module :P

don't worry about it. this case can only happen if the function name is set in C++. it can never happen in plain javascript, so this is a silly PR.
I ended up just copy and pasting the right into that PR, as there's no reason to slow down everyone's promisification just because of a super-rare edge case.
cheers
ref: https://github.com/nodegit/nodegit/issues/615


Testnet deployment followup

Closes: #102
Description

Updates deployment script addressing issues during Testnet deployment
Updates flow.json aliases
Adds Testnet deployment addresses to README

For contributor use:

[x] Targeted PR against main branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the standards mentioned here.
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

cc @bthaile @nialexsan for changes related to Flow Port

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

Project coverage is 86.33%. Comparing base (8623710) to head (0c2899f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   86.33%   86.33%
=======================================
  Files          17       17
  Lines         922      922
=======================================
  Hits          796      796
  Misses        126      126

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


"Manage classic deployments"

What exactly does it mean to "manage classic deployments"? This is said to be the reason for adding an RBAC Owner as a Co-administrator, but in my understanding RBAC Owners would have access to all operations in the ClassicCompute and other Classic providers. What ability is gained by being co-admin?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f5624f3e-5fbf-6f0f-93cf-a4d5aa93ccf8
Version Independent ID: 9ca6d22b-e684-4e65-cab3-c272189a3ddc
Content: Add or change Azure admin subscription roles
Content Source: articles/billing/billing-add-change-azure-subscription-administrator.md
Service: billing
GitHub Login: @genlin
Microsoft Alias: banders

@genlin can you share insight on this pls?

reassigning to @genlin as a doc-enhancement request to clarify documentaiton.

@genlin appears to be off until the 13th, is there another contact that can comment?

following up to see who else I can engage. we may have to wait for @genlin to return.

@dcbrown16, The RBAC permission can manage all resources (including classic) by using Azure portal. You will need to add RBAC owner as co-administrator if you still use the Azure Service Management PowerShell Module(old PowerShell module for classic resources) to manage the resource.

Thanks @genlin! I know ASM is slowly fading out, but can we make that info clear in the doc?

@dcbrown Sure, I will update the article. Thank you for the feedback!

@dcbrown16， I have submitted a pull request to update the article. I will process to close the issue when the pull request is merged. Feel free to reopen this issue if there is any additional comment about this problem. Thanks again for helping to improve our content experience.

@genlin on a completely different case I'm researching, I just heard that, for example, SQL DB audit log failure notifications are sent to subscription admins and co-admins. This seems like a "hidden" feature of the co-admin setting. Is there any way we could expose more info about that?

@dcbrown16 , As far as I know, RBAC owner can also receive the notifications from "Alerts (classic)" service. I suggest you contact our support for this detail.

I think it's more of an issue with SQL DB's notification implementation.
I'll leave the feedback here in case anyone comes across this. Thanks!
https://feedback.azure.com/forums/217321-sql-database/suggestions/32425666-allow-non-admins-and-non-co-admins-to-recieve-noti


Allowing configurable bind addresses.

Just seemed like a convenient option to be able to bind to arbitrary addresses. I might have overlooked some meaningful reason why this shouldn't be an option.
Thanks!

Nevermind - realized that this is now an option in the develop branch, so I think this is a moot request. Sorry bout that...


net: Implement kqueue support

Huge thanks to @mrdefacto who provided most of the code for the patch, I only fixed a few things. Seems to be working fine on macOS Mojave 10.14.6 and FreeBSD 12.2 and results in a ~2x lower load average comparing to select (on FreeBSD).

@PastaPastaPasta It doesn't complicate code that much really, we just plug it in the same way we did with previously - all the breaking code changes were made when we were preparing for poll/epoll implementations. Plus most of the changes here are isolated via USE_KQUEUE. Changes in the SocketEventsMode enum and in the getnetworkinfo rpc help string are the only that were not wrapped in USE_KQUEUE but that's how we do it for all other modes too. So it should be harmless for other systems and you can always opt-out to other modes on systems with kquque support (select for macOS and select or poll for FreeBSD) in case there is an issue with our implementation or with kqueue itself.


[Function added] Sharing settings

I hope I can save the pinned node settings and share them with others.

implemented in 2c08bb7


Flutter build process should not write to the sdk location, so flutter can be installed as a system level sdk on linux.

I have two users on my linux machine for development.
Currently I must install two instances of the flutter sdk for it to work smoothly. It would be nice if I only had to install one sdk folder at a root location. Then, only for sdk updates I would need root access.
The current issue is that running build commands writes files to the flutter sdk folder. For example I get the following error after I built a flutter project as one user, and then tried to build a project as another user.
Flutter failed to write to a file at "/usr/local/flutter-sdk/bin/cache/libimobiledevice.stamp". The flutter tool cannot access the file.
Please ensure that the SDK and/or project is installed in a location that has read/write permissions for the current user.

Ideally, generated files related to builds should be placed in a temporary .flutter folder in the home folder, so the sdk folder does not have user written files.

This is a duplicate of https://github.com/flutter/flutter/issues/44526


Content for Walkthrough cards should be dynamically loaded.

Code needs to be updated to dynamically update the cards based on the available walkthrough to the user.

@finp Is this still an issue?


missing util/conv4d.py?

trying out demo.py, getting an error:
Traceback (most recent call last):
  File "demo.py", line 8, in <module>
    from model.cnn_geometric_model import CNNGeometric, TwoStageCNNGeometric
  File "/Users/tkramer/projects/weakalign/model/cnn_geometric_model.py", line 9, in <module>
    from util.conv4d import Conv4d, Linear4d
ModuleNotFoundError: No module named 'util.conv4d'

Thanks for sharing the good. I'm getting the same error

Thank you for sharing! Can you add & recommit the file? Thanks!

Hi, thanks for reporting this.
I have deleted the references to this legacy file.
I don't have a computer to try the code now, but it should work :)


fix the vulnerability of atomicity in async commit

To fix https://github.com/pingcap/tidb/issues/33641

For a pessimistic async commit txn, set its TTL to be at least (max commit ts - start ts)
When an assertion fails, stash it. If the commit phase failed, ignore it. If the commit success, return the stashed error.

@sticnarf @MyonKeminta PTAL


Issue with importing task

Hey everyone,
After exporting a task and re-importing it, I am getting the following error:
"KeyError: 'reviews'\n".
Can anyone explain what is going on?
I am on the latest stable release.
Cheers

@lschaupp , did use CVAT v1.7.0 for export and CVAT v2.0.0 for import? If so, it will not work. You can try to modify the backup manually to remove unsupported keys.


Make liquidpy faster to use

The loading phase alone is super slow (seconds). May be bad imports.
https://github.com/pwwang/liquidpy

Started exploring the issue with the original maintainer: https://github.com/pwwang/liquidpy/issues/34

This was fixed by the original maintainers of liquidpy.


bug: We can see history with same password multiple times

As you can see in this image we can copy same password again and again and it increase same password as a new password in the history table.

Thanks for informing we try to resolve it


SL Review: Delay in final validation

Background:
In TEST when you submit the form for final validation there is a very long delay before the action is processed and the validation message is displayed
Steps to Reproduce:
Actual Behavior:

Expected Behaviour:
the "Final submission" confirmation message would display in a reasonable time similar to the "submit for validation" process
Environment:
TEST

@Stella-Archer
This is ready for testing on TEST.
A new loading indicator was added when forms are updating. For testing check the followings:

Senior Leadership form

Open a submitted Senior Leadership form under Submitted Forms and check the loading when clicking on Resubmit for validation and Final submission

Cold/Flu

Open a submitted Cold/Flu form and check out loading when clicking on Resubmit

Mat/Pat

Same thing for a submitted Mat/Pat when clicking on Resubmit under submitted forms and Validated  under Review and Process Form

Telework

Same thing with Telework under Review and Process Form when clicking Submit decision

The loader works great, thank you!
Just a snapshot of the new loader -

PO approved - tested


[Docs] Missing Spring Boot compatibility guide for Spring Cloud 2021 serial in https://spring.io/projects/spring-cloud

In document https://spring.io/projects/spring-cloud#adding-spring-cloud-to-an-existing-spring-boot-application 's Table 1. Release train Spring Boot compatibility section, there is no guiding for Spring Cloud 2021 serial is compatable with which version of Spring Boot like other Spring Cloud serials:

Ps: I found the compatiable Spring Boot version later in https://docs.spring.io/spring-cloud/docs/2021.0.0/reference/html/ .
Reference:

The previous issue I submitted at spring-projects/spring-boot which redirected me to here

Should be fixed now, thanks


#5440 fix(web): include missing JDBC properties in catalog creation payload for Apache Iceberg

Why are the changes needed?
These changes are needed to address a bug in the Gravitino UI that prevents users from successfully creating a catalog with type: relational and provider: Apache Iceberg. The missing properties cause a 400 error on the backend, blocking catalog creation.
Fix: #5440
Does this PR introduce any user-facing change?
Yes, this PR allows users to create catalogs with type: relational and provider: Apache Iceberg without encountering a 400 error due to missing JDBC properties. It resolves the issue of incomplete payloads and improves catalog creation reliability.
How was this patch tested?

Manual Testing:

Opened the "Create Catalog" dialog in the Gravitino UI.
Set type to relational, provider to Apache Iceberg, and configured catalog-backend as jdbc with appropriate jdbc-driver, jdbc-user, and jdbc-password values.
Clicked "Create" and verified successful catalog creation without errors.

Steps to Test:
Follow the reproduction steps outlined in the issue report to confirm that the changes address the problem.

Edge Cases:
Tested with catalog-backend set to jdbc and verified correct behavior.

@LauraXia123 would you please help to review this PR?

@niyushabaghayi hi, the latest commit have no issue about #5440 , let me know if you have any problems after retesting!

Thanks, @LauraXia123! Could you please point me to the specific issue and related PR that resolved this problem? I tried searching but couldn't find it, and I'm still experiencing the issue on the latest version (Gravitino 0.6.1-incubating). Any guidance would be greatly appreciated!

Fixed on (Gravitino 0.7.0-incubating), you can retry it on main branch or tag (v0.7.0-incubating-rc2).


Edit user fields via the admin panel?

Would be nice to Edit user fields via the admin panel

Dupe of #892


Mobile Menu Remains Open After User Interaction

Description:
The mobile menu in the Header component is not closing automatically after a user clicks on a menu item. This creates a suboptimal user experience as users have to manually close the menu after each interaction.]
Steps to reproduce:
Open the application on a mobile device or use responsive mode in browser dev tools
Click on the hamburger icon to open the mobile menu
Click on any menu item
Expected behavior:
The mobile menu should close automatically after a user clicks on a menu item.
Current behavior:
The mobile menu remains open after a user clicks on a menu item, requiring manual closure.
Possible solution:
Modify the navbarToggleHandler function to be called when a user clicks on a menu item. This can be done by adding an onClick event to each menu item that calls navbarToggleHandler.
Affected component: Header.tsx
Additional notes:
This issue affects the mobile user experience and should be prioritized for a smoother navigation flow.
Consider also adding an overlay that closes the menu when clicked outside the menu area for additional usability.

Fix:
To resolve the issue of the mobile menu not closing after user interaction, we need to call the navbarToggleHandler function when menu items are clicked. Here are the key changes:

Add onClick={navbarToggleHandler} to the logo Link component:

  {/* ... */}

Add onClick={navbarToggleHandler} to all menu item Links:

  {menuItem.title}

For submenu items, modify the onClick handler:
 {
    handleSubmenu(index);
    navbarToggleHandler();
  }}
  className={/* ... */}
>
  {submenuItem.title}

For the mobile "Sign Out" button, include the navbarToggleHandler:
<button
onClick={() => {
signOut({ callbackUrl: "/", redirect: true });
navbarToggleHandler();
}}
className="ud-menu-scroll flex py-2 text-base text-dark group-hover:text-primary dark:text-white dark:group-hover:text-primary lg:inline-flex lg:px-0 lg:py-6"

Sign Out

These changes ensure that the mobile menu will close after any interaction, including clicking on menu items, submenu items, the logo, or the sign-out button.


Enum without associated value, causes derive macro to hit "entered unreachable code" error

steps to reproduce:
Attempt to use enum like this one:
#[derive(Serialize, Deserialize, OpgModel)]
#[serde(tag = "req_type", content = "content")]
pub enum Repro {
    TypeA(),
    TybeB
}

Error:
error: proc-macro derive panicked
  --> src/data.rs:64:34
   |
64 | #[derive(Serialize, Deserialize, OpgModel)]
   |                                  ^^^^^^^^
   |
   = help: message: internal error: entered unreachable code

error: aborting due to previous error

error: could not compile `...`

Just encountered this error, using this awesome! crate. Haven't yet had time to dig deeper.

Thanks for the report, will be fixed soon!


Sorting horizontally sometimes updates the wrong items

I just setup ui-sortable and i'm experiencing some funky issues with horizontal sorting. I was wondering if bugs still exist around horizontal sorting? I tried floating:true and axis: x... however they didn't do much if anything.
Thanks!

Does the horizontal list example from README work for you? Any chance you
can share an example with the use case that troubles you?
On Thu, Nov 5, 2015, 02:01 Josh Bedo<EMAIL_ADDRESS>wrote:

I just setup ui-sortable and i'm experiencing some funky issues with
horizontal sorting. I was wondering if bugs still exist around horizontal
sorting? I tried floating:true and axis: x... however they didn't do much
if anything.
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/issues/402.

--
Thodoris Greasidis
Computer, Networking &
Communications Engineer

The Horizontal List example links to a Vertical List example. Yeah i'll put something together today.

@thgreasi here is a video of what i'm seeing - http://screencast.com/t/Rjiilubss
Basically i can't sort the order because other non-selected items that are hovered jump to a new location... either that or too many items are in the drop zone.

With bootstrap its a little less jumpy but the resorted order makes no sense - http://screencast.com/t/t4Bxjzg9M

So i ended up fixing the issue. I realized i had orderBy in my view which was causing issues.. after i moved that to the controller it worked fine. The other issue was that i called update on the update method but that should be done on the stop method. So changing those two things solved the problems.

I think that there is a button that toggles the layout to horizontal, isn't
it?
Anyway I'm glad you found your way through the docs / trial & error...
:+1: for being one of those who apply their filters in their controllers!
On Thu, Nov 5, 2015, 18:20 Josh Bedo<EMAIL_ADDRESS>wrote:

Closed #402 https://github.com/angular-ui/ui-sortable/issues/402.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-sortable/issues/402#event-455907496.

--
Thodoris Greasidis
Computer, Networking &
Communications Engineer


Rewrite CIF3 plugin to convert STIX-2 Indicators

:notebook_with_decorative_cover: Description

Following up on the STIX-2 rewrite of Threat Bus: this PR updates the CIF3 plugin.

Convert STIX-2 Indicators to CIF3 Indicators

:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/threatbus, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

Code review should be enough as this plugin has seen little attention and there are no unit tests, to begin with.
If you want to test interactively:

Follow the instructions in the CIF3 plugin's readme to start a CIF3 container and copy the API TOKEN to your Threat Bus config.yaml
Send Indicators via Threat Bus and confirm they make their way into CIF3 (e.g., see the logs)

I tested this with a local installation of CIF 3 and indicators updates are forwarded to CIF without errors.


Remove bdd test (cucumber)

This never got traction within the team and can be removed in the spirit of simplification.

Coverage remained the same at 64.693% when pulling ae79e588a8af147ddfdb4477b51d9c454980a4f9 on clean/remove-bdd-test into d912fb3b6220c15596d56d28fb75841279a58a2d on master.


Are GPG signatures available for source tar.gz files?

Hi there.  Is there is supposed to be a .asc file as a signature for the source releases?  I'm poking around at the v3.1.2 release and there are detached signatures for the binary releases but not for the source.
Apologies if I've missed something obvious here.

Currently we don't sign zipped source files.
Should we? /cc @xiang90 @heyitsanthony

Probably a good idea; some users prefer to build from source for auditing purposes.

Ok I will start signing the zipped source files as well from the next release.

Fantastic - thanks very much!

Fixed. Closing.

Just added tweaks to the Debian etcd package that successfully verifies the source signature when downloading. Thanks again!


Should we use system fonts?

https://css-tricks.com/snippets/css/system-font-stack/

Tradeoff: consistency of app on different platforms vs. consistency with other apps on a particular platform.
TL;DR: do we want to look same on all platforms or a bit different on each one?

That's exactly the question. Personally, I would love to use the system font for consistency. I would look a bit more native. But that's personal.

I'm also leaning towards being closer to native looks than to being internally consistent

System fonts are used on macOS as the app looks more integrated. There is a fallback on OpenSans for Linux/Win as the fonts sometimes looks weird.
Closing it for now, feel free to continue the discussion.


Request to update pipeline utility steps plugin to 1.5.1

Plugin: pipeline-utility-steps
Version: 1.5.1
https://plugins.jenkins.io/pipeline-utility-steps
The version of the plugin in the current Jenkins image is 1.1.5 and is missing useful utility steps like readJSON/writeJSON, readYaml/writeYaml

@gabemontero

seems reasonable to me.


leveldb: add effective disk read and write statistics

Hello,
We need to measure the amount of data read from and written to the file storage in order to optimize the database usage. Here is a previous related issue: https://github.com/syndtr/goleveldb/issues/113
IOStats contains the cumulative number of read and written bytes, and every read and write operations should update its state.
Does this approach cover all the disk read and write?

I think this would be much easier if implemented as Storage wrapped. Also it will guarantee to cover all disk R/W.

Thank you for your quick reply, and also apologies for being too direct. I'm a newcomer to the open source community :)
I tried to implement a Storage wrapper, as you adviced. Can you please take a look whether it's appropriate? https://github.com/syndtr/goleveldb/pull/207
Thanks,
Misi

Not a problem at all. Thanks for the contribution :)


how to call AttentionWrapper?

I am trying to write a simple seq2seq model with attention. But  it gets the following error:
attn_cell = tf.contrib.seq2seq.AttentionWrapper(
AttributeError: 'module' object has no attribute 'AttentionWrapper'

How should I call AttentionWrapper?
Here is my code:
    T=1000
    N=100
    input = tf.placeholder(tf.float32, shape=(N, T, 512), name="input_matrix")
    seq_lengths = tf.placeholder(tf.int32, shape=(N), name="input_lengths")

    cell= MultiRNNCell([DeviceWrapper(ResidualWrapper(LSTMCell(num_units=512)),device='/gpu:%d' %(i+1)) for i in range(2)])
    encoder_outputs, encoder_final_state = tf.nn.dynamic_rnn(cell,input, parallel_iterations=32, swap_memory=True, dtype=tf.float32)

    # Attention Mechanisms. Bahdanau is additive style attention
    attn_mech = tf.contrib.seq2seq.BahdanauAttention(
        num_units = 100, # depth of query mechanism
        memory = encoder_outputs, # hidden states to attend (output of RNN)
        #memory_sequence_length= T,#tf.sequence_mask(seq_lengths, T), # masks false memories
        normalize=False, # normalize energy term
        name='BahdanauAttention')

    cell_out= MultiRNNCell([DeviceWrapper(ResidualWrapper(LSTMCell(num_units=512)),device='/gpu:%d' %(i+1)) for i in range(2)])
        # Attention Wrapper: adds the attention mechanism to the cell

    # Attention Wrapper: adds the attention mechanism to the cell
    attn_cell = tf.contrib.seq2seq.AttentionWrapper(
        cell = cell,# Instance of RNNCell
        attention_mechanism = attn_mech, # Instance of AttentionMechanism
        attention_size = 100, # Int, depth of attention (output) tensor
        attention_history=False, # whether to store history in final output
        name="attention_wrapper")

    # TrainingHelper does no sampling, only uses inputs
    helper = tf.contrib.seq2seq.TrainingHelper(
        inputs = x, # decoder inputs
        sequence_length = seq_len_dec, # decoder input length
        name = "decoder_training_helper")

    # Decoder setup
    decoder = tf.contrib.seq2seq.BasicDecoder(
              cell = attn_cell,
              helper = helper, # A Helper instance
              initial_state = encoder_final_state, # initial state of decoder
              output_layer = None) # instance of tf.layers.Layer, like Dense

    # Perform dynamic decoding with decoder object
    outputs, final_state = tf.contrib.seq2seq.dynamic_decode(decoder)

You are likely running a version of tensorflow that is too old. Please upgrade and let us know.

Thank you for your response. But I am using tf 1.1.0 and still get this error.

Are you running from HEAD?

I think it's still called DynamicAttentionWrapper in TF 1.1.

Correct, as @ales-t suggested, in version 1.1 there is no AttentionWrapper:
https://github.com/tensorflow/tensorflow/blob/r1.1/tensorflow/contrib/seq2seq/__init__.py#L39

I'm struggling to figure out how AttentionWrapper interacts with MultiRNNCell in TensorFlow 1.2. Could anyone explain or link me to an example? @gunan @ales-t @drpngx
Also, for anyone else coming along, TensorFlow 1.2 replaces DynamicAttentionWrapper with AttentionWrapper.

In 1.3 also its AttentionWrapper instead of DynamicAttentionWrapper

I am also struggling with below error:
AttributeError: module 'tensorflow.contrib.seq2seq' has no attribute 'DynamicAttentionWrapper'
I'm using tensorflow 1.8.0

@rajendraarora16 even iam facing the same issue can anyone help ?

I am facing the same issue... Please someone help to rectify it. I also install Tensorflow addons, yet its not working

I'm experiencing the same issue. I've tried tf.contrib.seq2seq.AttentionWrapper and tf.contrib.seq2seq.DynamicAttentionWrapper and both gave the same error everyone above had.


Re-scans takes forever

Somehow I never managed to finish a Re-scan of the steam library it takes forever everytime without any error but I had to close the app.
Sometimes new games are detected sometimes not.

Would it be possible to select specifically which Drives/libraries to search for new games instead of running for everything everytime?
Also, would it be possible to create a default setting to be added as always as the default values instead of having to set every game on it's own. That way I can see all at once and only modify values on specific games.

anyway besides those details, great app !!!

Can you take a look in %LOCALAPPDATA%\openvr_fsr_app and see any *Exception*.log files? If not than it is IO bound by your hard drives. The scan module is using almost all available CPUs/threads and should report it's progress in the menu bar.
I'm thinking about creating a [  ] Scan Steam Libraries On/Off switch. That way you could simply add directories yourself that you are interested in.

Yes I found some exception*.log files, what should I do?
That idea about selecting which Steam Libraries to scan is great, maybe also being able to refresh only one game could be possible?
For example right now I have EverSlaught pointed to the wrong directory by the app, and I cannot refresh that directory information because the app never finishes the re-scans.
Keep it up!


Issue in redirecting to WordCount application

Extremely sorry for referring different page for my issue #37467. The issue is on this page
If you click on

fabric:/WordCount application
It will not redirect to any application

Please find location of this line in below screenshot

I hope I am able to clear where this issue exactly is now @MicahMcKittrick-MSFT
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ed92f9d3-f918-fde4-34e8-e38369e5f6c7
Version Independent ID: 795688ed-1d8d-1c08-c679-022d20daaffc
Content: How to view Azure Service Fabric entities' aggregated health
Content Source: articles/service-fabric/service-fabric-view-entities-aggregated-health.md
Service: service-fabric
GitHub Login: @oanapl
Microsoft Alias: oanapl

@RJK23 Thanks for the feedback. We are investigating and we will update you shortly.

@oanapl  @peterpogorski The link in the document (fabric:/WordCount application) redirects to aka.ms and the points to http://servicefabricsdkstorage.blob.core.windows.net/publicrelease/WordCountV1.sfpkg
That storage account is no longer available.
Do we have that Wordcount app uploaded some where so that we can replace the correct link?

Found the package here https://github.com/smartpcr/ServiceFabric/blob/master/WordCount/WordCountV1.sfpkg
But Cant point to some random repo and we are not sure about the integrity of that content

@msfussell #22998 Similar issue where the same storage account is mentioned

@oanapl please take care of this issue

I don’t work in Service Fabric team anymore, please work with Mark Fussell to find the right owner.
From: jakaruna-MSFT<EMAIL_ADDRESS>Sent: Monday, August 26, 2019 9:23 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Oana Platon<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Issue in redirecting to WordCount application (#37593)
@oanaplhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Foanapl&data=02|01|Oana.Platon%40microsoft.com|334e181c8f524a0b10c208d72aa63a97|72f988bf86f141af91ab2d7cd011db47|1|0|637024765744879223&sdata=cGBedi8DuFKuSKgyefy8kUBs2FIQoz%2Bt4cDMmwTiJ1k%3D&reserved=0 please take care of this issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F37593%3Femail_source%3Dnotifications%26email_token%3DACVWMFVXJ2RJ3Q25NSFJZPTQGSTZZA5CNFSM4IORDIR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5GNYAY%23issuecomment-525130755&data=02|01|Oana.Platon%40microsoft.com|334e181c8f524a0b10c208d72aa63a97|72f988bf86f141af91ab2d7cd011db47|1|0|637024765744879223&sdata=1tF%2FgsMgsNAcZrPHLlRfw0QIRElb9kFk54YU7q1CH88%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACVWMFTYADSW3QHF2KAYAGLQGSTZZANCNFSM4IORDIRQ&data=02|01|Oana.Platon%40microsoft.com|334e181c8f524a0b10c208d72aa63a97|72f988bf86f141af91ab2d7cd011db47|1|0|637024765744889228&sdata=Wld0TmI1%2BLapbA26hlW1DwwvwD068oxRq5yIO8%2BCZzo%3D&reserved=0.

Thanks @oanapl
@msfussell  Can you suggest the new owner for this document

Hi @RJK23, thanks for bringing this to our attention. I've rehosted the sample elsewhere and changed the document appropriately.
Nick
#please-close


Add note for onAdd's second param

Related to https://github.com/colyseus/schema/issues/147
Currently, this second param is undocumented, I had to dig through github issues to find the solution for my problem.
This note might not be quite right, it assumes the TypeScript implementation. Do other langs behave the same, do they have the same 2nd param?
The fn signature should probably be mentioned in some way or another. And I think it should be mentioned that listen() can have this same problem it seems (causing a listen -> onAdd nesting to cause onAdd to still trigger multiple times if the data existed before the user joins the room).

Thank you so much for this @francislavoie 🎉
I added the signature as you suggested and described the default behaviour of second argument ("triggerAll" )


QUESTION Keeping User_ID not null

Is there's a way to no change to NULL the USER_ID when they logout, im trying to get a record of everyone who used the project, for control purpuses.

add "logout" and "login" pages to the 'do_not_track_paths' in tracker config file.
like this:
    'do_not_track_paths' => [         'api/*',         'logout',         'login',     ],


python312Packages.keyutils: drop

The upstream repository has been archived.
part of https://github.com/NixOS/nixpkgs/issues/346385

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This is still used in udiskie

Should we switch to https://github.com/lilatomic/turkeyutils?

I'm not too sure. Seems to be pretty unknown at 2 stars, and a quick look at the commit log shows it's only really adding things rather than fixing any security issues or other bugs. I would also question how maintained it really is, given the last commit is over a year ago


Upgrading Twitter Bootstrap to v.2.0.0

Twitter Bootstrap has been upgraded to v2.0.0 which was a major release so I thought it would be a good idea to go ahead and upgrade everything for roots.
Here is a list of files and brief comments of what was changed.
roots-actions.php:  added responsive css framework file
roots-options.php: changed default css classes for bootstrap v2.0.0
roots-scripts.php: updated less to v1.2.1, changed the load order and
added new scripts for bootstrap v2.0.0
Added all of the new css, js, less asset files and updated all of the
previous ones for bootstrap v2.0.0

thank you!


CARGO_INCREMENTAL: Cargo should only compile crates incrementally that can be changed by the user

Compiling a crate in incremental mode with an empty cache is often slower than compiling the crate non-incrementally. Also, the generated binary is likely to exhibit worse runtime performance and the IC cache for the crate takes up space on disk. At the same time, having a non-incrementally compiled dependency does not have any detrimental effects on incremental compilation as long as that dependency does not change. For these reasons it makes sense to always compile things coming from crates.io or other external sources in non-incremental mode.

Sounds reasonable to me!
Such a change would likely go here and have an extra check for path sources. Users can only modify path sources, all others should be frozen in Cargo's cache somewhere.


Functional fixes to AWS stacks guide

Signed-off-by: Muvaffak Onus<EMAIL_ADDRESS>
Description of your changes

Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[x] Updated any relevant documentation, examples, or release notes.
[x] Updated the RBAC permissions in clusterrole.yaml to include any new types.

@jbw976 this is ready to merge


Release v1.0.15

Part of release process.

tag update https://github.com/spacemeshos/go-spacemesh/releases/tag/v1.0.17


When will Access Control APIs be added?

Is your feature request related to a problem? Please describe.
No.
Describe the solution you would like
As https://www.databricks.com/blog/open-sourcing-unity-catalog says, Access Control APIs is in the roadmap, so when will it be added?
Additional context
None.

We should have support for access control in V0.2, end of August. I'll post some details in #229 soon

We should have support for access control in V0.2, end of August. I'll post some details in #229 soon

Will it have authorization API on objects for role or user like this: https://docs.databricks.com/en/data-governance/unity-catalog/index.html#granting-and-revoking-access-to-database-objects-and-other-securable-objects-in-unity-catalog ?


update link

This point to EBI binderhub
Tested this am.
We could offer both links if you prefer
cc @YojanaGadiya

@jburel Was trying to look at the link, it seems to no be broken. Please take a look when you get time this week.

@YojanaGadiya I have tested it and it seems that there is a problem
I will contact the team to see what's going on

@jburel any update on this?

@YojanaGadiya Thanks for re-activating the discussion. I will look at it

Closing the PR since the link is still not working. We can reopen it once the testing is completed.


Approximate nearest neighbors support

As the ANN plugin is out, it would be great to have an index and field for it

Would you be up for creating the PR?

Transferring this issue here as opensearch-dsl-py is being merged into opensearch-py as part of https://github.com/opensearch-project/opensearch-py/issues/194


Plugins :Allow Hyphens in Short URLs， Does not work

Hi,
After opening Allow Hyphens in Short URLs, open a short URL with a forward slash and a 404 error will occur.
Thank you!

YOURLS version: v1.7.3
Plugins enabled: Allow Hyphens in Short URLs、Mass Remove Links、QR Code Short URLS、Random Keywords
PHP version: PHP-7.0
MySQL version: 5.7

I'm pretty confident that after all these years and all the dozens of thousands YOURLS installs running this plugin, it actually works. (I'm assuming that instead of "open a short URL with a forward slash" you meant "with a hyphen")
If you can reproduce the problem, something else is at cause here, and you'll probably have to pin point the problem with various tests. The server error log might help. Feel free to comment on with more reports so maybe we'll be able to help

The above may be incorrectly described. I enabled the plugin "Allow Hyphens in Short URLs". The short link contains a hyphen. Opening it will cause a 404 error. I tested it on another server, and the same phenomenon. Can you tell me how to deal with it, I need your assistance very much, thank you!

See http://ozh.in/i-think-it-works and then my previous answer


Documentation at http://uavcan.org/ issue

From:
http://uavcan.org/Implementations/Libuavcan/Platforms/Linux/#uavcan_monitor
Section:
uavcan_dynamic_node_id_server
This application implements a dynamic node ID allocation server. Start without arguments to see usage info.
Usage info is crap, this is much better:
uavcan_dynamic_node_id_server 10 can0 -c1 -s .
where:
10 - node ID of the server
can0 - name of the interface
-c1 - cluster size is 1 (no replication, no redundancy)
-s . - keep the persistent state in the current directory
however it's better to make the -s argument absolute, e.g. ~ (your home dir)

Usage info says exactly that but in a less verbose way.


electron: 1.8.1 -> 1.7.9

Motivation for this change
Downgrade Electron to latest stable version. See #32398, #32741, #32765.
Things done

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

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

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

I can't reproduce that bug with Rambox. /cc @gnidorah

Why not keep both?
Like https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/interpreters/php/default.nix#L334 or https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/libraries/openssl/default.nix#L109

electron is currently used only in 2 derivations (rambox, nix-tour). Maintaining one Electron branch is troublesome enough, given it's a binary derivation with 4-5 target platforms. Also their beta branch apparently can contain critical bugs like this one.
Those facts combined make me think that it's more trouble than it's worth.

BTW, I have a source-based Electron in the works, but it will take a while because their build system implies internet access, which makes it hard to get working, all dependencies are pinned/vendored, and builds take a very long time.

I meant, the version pinned for rambox and another one following the upstream.
Otherwise, someone could break rambox again by upgrading electron
On 12/17/17, Yegor Timoshenko<EMAIL_ADDRESS>wrote:

electron is currently used only in 2 derivations (rambox, nix-tour).
Maintaining one Electron branch is troublesome enough, given it's a binary
derivation with 4-5 target platforms. Also their beta branch apparently can
contain critical bugs like this one.
Those facts combined make me think that it's more trouble than it's worth.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/32768#issuecomment-352281926

This pull request follows the upstream. I don't believe that pre-release versions should be packaged by default.
Rambox is just an HTML page and some JavaScript. It's a highly sandboxed environment, and if anything breaks to the point of a failed assert in that environment, release is probably buggy.

@yegortimoshenko

I can't reproduce that bug in Rambox with Electron 1.7.9

Same for me. Thank you!


Fix potential message lost in routine load

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

No issues proposed for now.
Problem Summary(Required) ：

The return value of KafkaConsumerPipe::append_data was not processed, message could get lost if we failed to append.

 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.

 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.

@chen9t
Thanks for your contribution! And could you please sign the CLA ?

 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.

@chen9t Thanks for your contribution! And could you please sign the CLA ?

CLA signed, thank you!

run starrocks_clang_format


Some idiomatic english corrections.

Some minor corrections to make the english more idiomatic - it was the easiest way to help!

Thanks a lot!


iPhone touch issue fixed

https://github.com/xuopled/react-simple-tooltip/issues/26

Thank you. Just published v2.6.1


Add time operations

Adds some useful time operations to ScmDataFrame

@lewisjared I'm not super happy with how this is implemented. In particular, passing in the keyword style is pretty yuck. If you can see a better way to do it (underneath the hood it's just a groupby operation so maybe we should refactor to that?) I'd be happy to change. This should be enough to get the conversation started and the tests sorted hopefully.


Fixed #29534 -- Made dbshell use rlwrap on Oracle if available.

Ticket 29534.

buildbot, test on oracle.

@timgraham Thanks for the review :+1:. I updated PR.

buildbot, test on oracle.


About sound effect

I know sound effects have several types
Through my research, I found that its type can be defined by these parameters.

AAA type
The volume of AAA type SFX will decay very quickly with the distance between the target and the camera getting farther and farther.
The attenuation of ABA type SFX is relatively small.
The BBA type does not decay at all.
I didn't study the impact on sound effects when the third is B, such as AAB? BBB? Abb, and there seems to be another type of BCA, which I didn't study.

Usually, if I want to modify the type of SFX, I must use WinHex and textools, and then repackage the modified SFX and VFX.
So is it possible to add parameters that can directly modify SFX type in vfxeeditor?

huh, that's really interesting. Adding a sound effect replacement feature is definitely possible, but would be a long-term goal. I'll look into it

I came across this git repository while searching for an SCD editor because I made a similar observation to this issue topic while looking through a comparison view between a sound file pulled from Shadowbringers and a sound file pulled from Endwalker.
I hate the way that Endwalker changed combat skill sound effects to be basically silent when other players are casting them and have been looking for a way to reverse that.
So, I just wanted to add a "+1" to show support for a tool that would let me edit these sound files more easily than scouring through them with a hex editor.
It has been very difficult to find information about this because it seems that the average player didn't even notice this was changed. For example, I like being able to hear if my group WHM is casting Holy so I know if I need to move out of AOE or if enemies will be stunned. I should be able to listen to what is going on around me and look at my feet placement instead of the party window to see what the WHM is doing. Endwalker made other group members' spells almost completely silent. I cannot even hear a warrior do their 1-2-3 combo whilst standing right next to them.

@Unknown-Zombie  you can get around this by going into each action TBM file finding all the sounds and adding them into the timelines of the .avfx files. Any sound in a modified .avfx file when used by another player will be heard by you (but not them obviously). It would require a lot of work to do it for every skill, but that is one way to accomplish this.
Also as to the topic of sound fall off, I think sounds in .avfx files are effected by binders. and there are some other things you can do as well, like stack two of the same exact sounds on two timeline object emitters. it has the effect of making the sound louder for most sound effects. Some sound effects that have very fast gradations of tones or pitches might occasionally glitch out when you hear them (but its not a bad glitch, its sorta like hearing an echo of the same sound or a deeper 'in a tunnel' version of the sound).


Mbt build command getting stuck on a windows machine

We currently have the issue that our mbt build is sometimes getting stuck on a windows machine. This seems to only happen while packaging a module build result into the temporary mta_build_tmp folder. For me, it gets stuck e. g. while building a module like the following. Unfortunately, the error is not reproducible in a stable manner and only occurs from time to time.
Module Definition:
modules:
- name: rm-db
  type: nodejs
  path: db
  ...

with the following package.json
{
	"name": "rm-db-deployer",
	"dependencies": {
		"@sap/hdi-dynamic-deploy": "1.7.2"
	},
	"engines": {
		"node": "^10.0.0"
	},
	"scripts": {
		"postinstall": "node collectArtifacts",
		"start": "node node_modules/@sap/hdi-dynamic-deploy/"
	}
}

Here is the relevant part from the build log. I hade to remove some parts. Notice that there are no dots behind the packaging command for the rm-db module. This is why we think the build is getting stuck at that point. I manually aborted the build after a few minutes with no additional log output. Windows Task Manager did not show any CPU utilization related to the mbt build at that point.
> mbt build --mtar <HIDDEN>

[2020-11-13 14:04:19]  INFO Cloud MTA Build Tool version 1.0.16
[2020-11-13 14:04:19]  INFO generating the "Makefile_20201113140419.mta" file...
[2020-11-13 14:04:19]  INFO done
[2020-11-13 14:04:19]  INFO executing the "make -f Makefile_20201113140419.mta p=cf mtar=<HIDDEN> strict=true mode=" command...
[2020-11-13 14:04:19] DEBUG the executable file is at "<HIDDEN>"
[2020-11-13 14:04:20]  INFO validating the MTA project
[2020-11-13 14:04:20]  INFO running the "before-all" build...
[2020-11-13 14:04:20]  INFO executing the "npm install --production" command...
[2020-11-13 14:04:20] DEBUG the executable file is at "C:\Program Files\nodejs\npm.cmd"

<<< a (large) cds build is executed >>>

[cds] - build completed in: 30018.092ms

[2020-11-13 14:05:02]  INFO validating the MTA project
[2020-11-13 14:05:02]  INFO building the "rm-db" module...
[2020-11-13 14:05:02]  INFO executing the "npm install --production" command...
[2020-11-13 14:05:02] DEBUG the executable file is at "C:\Program Files\nodejs\npm.cmd"

<<< some other native artifacts are copied over to the db folder during npm postinstall >>>

.[2020-11-13 14:05:07]  INFO the build results of the "rm-db" module will be packaged and saved in the "<HIDDEN>_mta_build_tmp\rm-db" folder

The db folder contains about 4000 files and has a size of ~230MB
Version info:

MBT Version: 1.0.16
OS Version: Windows 10 version 1909

Please let me know if further information is required. Thanks for your help!
Best Regards,
Florian

We are observing the same issue. The mtb build command is getting stuck when trying to build the following module:
- name: mkv-api-broker
  type: nodejs
  path: broker

The build starts, but never returns after the last output [2020-11-15 16:46:54]  INFO ... below:
[2020-11-15 16:46:51]  INFO validating the MTA project
[2020-11-15 16:46:51]  INFO building the "mkv-api-broker" module...
[2020-11-15 16:46:51]  INFO executing the "npm install --production" command...

up to date in 0.786s

[2020-11-15 16:46:54]  INFO the build results of the "mkv-api-broker" module will be packaged and saved in the "C:\SAPDevelop\c4u-edom-mkv\.c4u-edom-mkv_mta_build_tmp\mkv-api-broker" folder

We have noticed, that mtb version 1.0.15 successfully completes the job, while mbt version 1.0.16 causes the problem.

Steps to reproduce the issue:

clone mkv github repo
execute mbt build -t .
the affected module is mkv-api-broker (move the module at the front of the mta.yaml to avoid waiting)

The problem occurs almost every time on my windows pc.
Other colleagues observe it sporadically.

I have the same problem building an MTA application. When generating final zip packages MBT tool just stop working, no log information about the problem (zip file is 0 bytes). Before, it just happened to me 1 of each 10 times, now it happen every time and I can't build my project.

@jofunkillo can you provide your project?

Sorry, I can't share for privacy reasons.
As mentioned above, I downloaded the MBT version 1.0.15 and the whole project compiled correctly now.
My project is having 7 UI applications, 4 fiori freestyle applications and 3 fiori elements apps, on top of a CAP service. It takes a while to build it.
With latest MBT version, the build process was randomly stopping in the generation of the zip files. I tried to delete the existing files before the build process to see if that could be the reason (file locks) but sometimes it didn't work.

Hi,
Got exactly the same issue than @jofunkillo
When generating final zip packages MBT tool just stop working, no log information about the problem but zip is not 0 size. It looks like an issue with McAfee as my CPU goes to 100%.
Everything was stuck but mbt was running because when i want to remove the zip file it says that mbt.exe lock the file.
Goes back to version 1.0.15 solve the issue.
make -v
GNU Make 4.3
Built for Windows32
Copyright (C) 1988-2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Cloud MTA Build Tool version 1.0.15
|:------------------ | ----------- |
| Node.js            | v12.19.0    |
| @sap/cds           | 4.4.5       |
| @sap/cds-runtime   | 2.7.5       |
| @sap/cds-compiler  | 1.46.6      |
| @sap/cds-dk        | 3.3.2       |
Thanks

Hello colleagues,
I have the same issue, i.e. I can't build my project https://github.wdf.sap.corp/GRC-DPP/questionnaire anymore with mbt 1.016
Process stucks when creating data.zip file. Sometimes the first file can be created and process stucks on the second file, as can be seen from the screen shot.  MBT tool just stops working w/o any log information about the problem.
Thanks, Thomas

Issue was reproduced and we're investigating.

Hello colleagues,
just did an update to mbt version 1.0.17 and ran the build process multiple times. It came to a successfull end every time.
Thanks, Thomas

Issue solved by version 1.0.17.


ValueError: Could not find object: <package_name>

I have installed mkapi via pip.
I am trying to use page mode as per the documentation.
My mkdocs.yml has
# multiple lines not here for brevity
nav:
  - API: mkapi/api/wappdriver
plugins:
  - mkapi:
        src_dirs: ['wappdriver']

Here wappdriver is the name of my package.
The directory structure is  like:
- mkdocs.yml
- setup.py
- wappdriver
   - __init__.py
   - module.py
- docs
    - index.md

When I run mkdocs serve
I get the following error
INFO    -  Building documentation...
Traceback (most recent call last):
  File "/home/linuxbrew/.linuxbrew/bin/mkdocs", line 8, in <module>
    sys.exit(cli())
  File<EMAIL_ADDRESS>line 829, in __call__
    return self.main(*args, **kwargs)
  File<EMAIL_ADDRESS>line 782, in main
    rv = self.invoke(ctx)
  File<EMAIL_ADDRESS>line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File<EMAIL_ADDRESS>line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File<EMAIL_ADDRESS>line 610, in invoke
    return callback(*args, **kwargs)
  File<EMAIL_ADDRESS>line 133, in serve_command
    serve.serve(
  File<EMAIL_ADDRESS>line 141, in serve
    config = builder()
  File<EMAIL_ADDRESS>line 136, in builder
    build(config, live_server=live_server, dirty=dirty)
  File<EMAIL_ADDRESS>line 236, in build
    config = config['plugins'].run_event('config', config)
  File<EMAIL_ADDRESS>line 94, in run_event
    result = method(item, **kwargs)
  File<EMAIL_ADDRESS>line 50, in on_config
    config, self.abs_api_paths = mkapi.plugins.api.create_nav(
  File<EMAIL_ADDRESS>line 23, in create_nav
    page[key], abs_api_paths_ = collect(
  File<EMAIL_ADDRESS>line 47, in collect
    module = get_module(package_path)
  File<EMAIL_ADDRESS>line 96, in get_module
    obj = get_object(name)
  File<EMAIL_ADDRESS>line 39, in get_object
    raise ValueError(f"Could not find object: {name}")
ValueError: Could not find object: wappdriver

Is my way of usage of page mode incorrect ? Could you please guide if so.
Many thanks for reading. Your help is most appreciated.

I'm having the same problem on readthedocs.org - does anyone have an idea how to resolve this? I'm assuming it's something with having correct paths set, but after trying all the options I could think of it is the same. Any help appreciated!

Over a year after my comment, I think I have figured out the issue.
The issue on Read the Docs was caused by the fact that my library was not installed, and their builder doesn't access the local packages; so the library in case needs to be installed alongside any other build dependencies. My original .readthedocs.yaml had this configuration:
python:
   install:
   - requirements: docs/requirements.txt

This installed the dependencies, but not the actual library. Since pip can now find the dependencies in pyproject.toml, I have changed the configuration to:
python:
   install:
   - method: pip
     path: .
     extra_requirements:
       - docs

In this version, the docs is a set of dependencies defined in [project.optional-dependencies] of pyproject.toml; and the library gets installed along with them.
Another cause of the above error is when the path is incorrectly set in mkdocs.yaml. I would dare to guess that @aahnik's example above doesn't need the src_dirs: ['wappdriver'] in its configuration, and that including it makes mkapi to look for the code in wappdriver.wappdriver; but I might be wrong.


NTR: chlorthion

Not sure how this escaped the mass CHEBIfication of the insecticides, but it remains (with just two data points from 1976 using it...)
MIRO:10000211 | chlorthion
Doesn't appear to be in CHEBI.  Could we have a EUPATH term please?  No need to submit to CHEBI as we don't expect more data.
More info here
https://pubchem.ncbi.nlm.nih.gov/compound/Chlorthion
Looks like nasty stuff (though probably all insecticides are)

I can make a new term under 'organophosphate insecticide' using the pubchem info.

As with the new organism terms I added, I didn't give this term a definition, but I made a note that I added the term to VEuPath because it is not in ChEBI. However, it inherits the insecticide role from its parent.

Let's submit to ChEBI.

Let's submit to ChEBI.

I have made the submission to ChEBi. They have automatically assigned ChEBI identifier CHEBI:167184 for chlorthion. ChEBI curator will now review the submission and make final decision.

I have made the submission to ChEBi. They have automatically assigned ChEBI identifier CHEBI:167184 for chlorthion. ChEBI curator will now review the submission and make final decision.

This ChEBI term has now been curated: https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:167184

This ChEBI term has now been curated: https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:167184

Chris, I have now assigned this issue back to you as the new term is now available in ChEBI.

Chris, I have now assigned this issue back to you as the new term is now available in ChEBI.

John please import in next release.


PR:25564 New Room - No possibility to create New Room, New Room option is upset in FAB menu

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to https://25564.pr-testing.expensify.com/
Login with any Expensifail account
Make sure that you have an existing workspace or create a new one
Click on FAB button

Expected Result:
There is a possibility to create new room in the App in FAB menu or in any other way
Actual Result:
There is no new room option and there is NO other way to create a new room in the app
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.64-0 PR:25564
Reproducible in staging?: No
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/2d25cab5-c4a3-478d-8566-e0d5ee1c0efc
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Link PR https://github.com/Expensify/App/pull/25564


KAFKA-12539: Refactor KafkaRaftCllient handleVoteRequest to reduce cyclomatic complexity

More detailed description of your change

Add grantVote to EpochState
Move the if-else in KafkaRaftCllient.handleVoteRequest to EpochState
Add unit-test for grantVote

Summary of testing strategy (including rationale)
Unit Test
Committer Checklist (excluded from commit message)

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

@hachikuji , Thank you, hava addressed your comments, and will take a look at KAFKA-12607.


Update README as in General Grants repo

 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.

Hi @alxs, thanks for your delivery. Could you please sign the CLA? ;)

@alxs Could you double check the links?

I fixed them in w3f/Grant-Milestone-Delivery#109, just double-checked and everything works.