Non constexpr operator==

First of all, great work.
This is probably the most thorough and streamlined backport of string_view publicly available, and I thank you for making it such.
I'm using it to replace Google's Abseil implementation, which still has some rough edges, and certainly drags in a lot of baggage for someone who just needs string_view.
One of my biggest motivations for using string_view is to facilitate string keyed associative containers with compile-time lookup, which opens the door to many useful things with zero runtime overhead. However, this requires constexpr comparison to be available...
This string_view delegates operator== to char_traits::compare() which is part of std, and as such is required NOT to be marked constexpr in pre-C++17 compilation. This is despite the compiler being perfectly able to perform constexpr comparisons.
In my case then, there is little value in backporting string_view if it doesn't backport its benefits too.
My workaround solution for this was deriving my own char_traits to implement a constexpr compare(), and then deriving my own string_view, as shown below. Perhaps you can think of a better way to backport constexpr comparison.
Thx
namespace my_own {
    namespace _detail {
        struct char_traits : public std::char_traits<char> {
            static constexpr int compare( const char_type* s1, const char_type* s2, std::size_t count ) {
                ... compiler specific ...
            }
        };
        using plain_string_view = nonstd::basic_string_view<char>;
    }
    class string_view : public nonstd::basic_string_view<char, _detail::char_traits> {
        using Base = nonstd::basic_string_view<char, _detail::char_traits>;
    public:
        using Base::Base;
        // need to provide own implicit conversions with std::string, since it uses the default char_traits
        operator string() const noexcept { return string(data(), size()); }
        string_view(string const & s) noexcept : Base(s.data(), s.size()) {}
        // provide implicit conversions with a string view that uses standard char_traits
        constexpr operator _detail::plain_string_view() const {return {data(), size()};}
        constexpr string_view(_detail::plain_string_view p) : Base(p.data(), p.size()) {}
    };
    inline std::ostream & operator<<(std::ostream & os, string_view sv) { return os << _detail::plain_string_view(sv); }
} // namespace

Hi @mcskatkat  Thanks for your kind words. Also thanks for the interesting suggestion, i'll look into it.

Hi @mcskatkat  can you have a look at above commit?

would that work for you?
should CTraits also provide constexpr eq()?
...

Hi. Thanks for the quick response. This might work but there are two things about it:

You definitely want to use compiler builtin function to implement the comparison itself. It's very easy to get to thousands of comparisons made at compile time and strings may be long. Compilation might take an unnecessarily long time if implemented character by character. Note that compile time code execution is usually way slower than that of equivalent compiled code at run time.
To that end, GCC provides __builtin_memcmp, and AFAIK MSVC and clang implement plain memcmp() calls as intrinsics, and all of them are constexpr. You can look at how Abseil handled this, or how the std lib sources handle it for C++17/20.
BTW the same applies for strlen, which in your code is currently implemented character by character. I think it's best to leave the character by character implementation for cases where the compiler isn't identified / known.

I don't see the benefit of deriving CTraits from Traits if its only use is a single static function that's being called directly by fully qualified name. Such a function could just as well be a protected static member of string_view. The derivation was helpful for me trying to work around this from outside the class without modifying it, and it gave me an extra headache to reconcile the specialized string_view with the classes that use the default traits.

Afterthoughts:

All of the above only applies when the element type is char, however that is by far the most common case.
So I would consider creating a protected static compare() in string_view with a specialization for char that delegates to compiler intrinsics if available. In all other cases fall back to the current element-by-element generic implementation.

looking at:
if ( const int result = CTraits::compare( data(), other.data(), (std::min)( size(), other.size() ) ) )
How would that handle comparison between "abcd" and "abcdef" ? Unless I'm missing something, it would say they're equal.

Similarly, for operator== and operator!= it's probably better to short-circuit if lengths are different, saving the unnecessary memcmp.

Thanks for your notes, it is very helpful to get this steering based on actual use cases, drawing attention to weak spots and offering pointers to possible improvements.
I'll take it further with your suggestions.

I edited my previous comment and removed incorrect items that resulted from me being lazy and only looking at the commit diff, missing the rest of the code.

I have the impression builtins such as memcmp() and __builtin_memcmp(), __builtin_strlen() cannot be used in a constexpr context like with g++ or msvc:
constexpr char hello[] = "hello";
constexpr char world[] = "world";

static_assert( string_view( hello ).compare( string_view( world ) ) == 0, "" );

See e.g. SO question Why does constexpr context make the compiler fail, while w/o it optimizes perfectly?.
On godbolt:

clang (C++11), succeeds
gcc (C++14), fails
msvc (C++14), fails

@mcskatkat I've merged above to master. I'm curious what you think of it, if there's more to it.

Yes, it looks good.
There's a large matrix of compiler-type / compiler-version / C++-standard that needs to be tested, it's hard to tell from looking at the code whether the maze of #if and macro expansions cover it correctly.
I tried this with my code after removing my previous workaround.
It compiles ok, but I'm not getting __builtins (it's using the generic code).
I am using GCC 8 with C++14 and I know that the Abseil code I was using previously had __builtins and worked fine.
I'm also certain that GCC 7 has them and probably earlier versions too, but I'm not sure when they started being constexpr.
BTW I'm quite certain that any compiler that has __builtin_memcmp also has such strlen. So until one that doesn't is encountered, I think it's safe to define one nssv_HAVE_BUILTIN in terms of the other (but keep separate macros).
Unrelated, I think it's better if you merge the two identical, generic length() implementations (constexpr and non-), using macro expansion instead of #if. This will avoid code duplication.

I tried this with my code after removing my previous workaround.
It compiles ok, but I'm not getting __builtins (it's using the generic code).
I am using GCC 8 with C++14 and I know that the Abseil code I was using previously had __builtins and worked fine.

Can you show your try-code?
As far as I can tell, the following does not compile constexpr-ly with Abseil under gcc, or with Abseil under msvc:
constexpr char hello[] = "hello";
constexpr char world[] = "world";

static_assert( absl::string_view( hello ).compare( absl::string_view( world ) ) < 0, "" );

What am i missing?

> ls -l
total 0
> g++ --version
g++ (Ubuntu 8.4.0-1ubuntu1~18.04) 8.4.0
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

> cat | g++ -I$ABSL_PATH -std=gnu++14 -x c++ -c -
#include <absl/strings/string_view.h>

constexpr char hello[] = "hello";
constexpr char world[] = "world";

static_assert( absl::string_view( hello ).compare( absl::string_view( world ) ) < 0, "" );

> ls -l
total 4
-rw-rw-r-- 1 ubuntu ubuntu 1152 Oct  2 22:34 -.o
>

Perhaps you are using an older version of absl. I know for sure that past versions were missing several constexpr annotations.

Here are some excerpts from absl/strings/string_view.h and absl/base/config.h that I'm using (AFAIK they don't really do versioning):
constexpr bool operator==(string_view x, string_view y) noexcept {
  return x.size() == y.size() &&
         (x.empty() ||
          ABSL_INTERNAL_STRING_VIEW_MEMCMP(x.data(), y.data(), x.size()) == 0);
}

#if ABSL_HAVE_BUILTIN(__builtin_memcmp) || \
    (defined(__GNUC__) && !defined(__clang__))
#define ABSL_INTERNAL_STRING_VIEW_MEMCMP __builtin_memcmp
#else  // ABSL_HAVE_BUILTIN(__builtin_memcmp)
#define ABSL_INTERNAL_STRING_VIEW_MEMCMP memcmp
#endif  // ABSL_HAVE_BUILTIN(__builtin_memcmp)

#ifdef __has_builtin
#define ABSL_HAVE_BUILTIN(x) __has_builtin(x)
#else
#define ABSL_HAVE_BUILTIN(x) 0
#endif

This is --version on godbolt (8.3 is the highest 8.x they have):
g++ (Compiler-Explorer-Build) 8.3.0
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE

Seems that they have their own build of gcc. Maybe that's the difference...
BTW I tested with gcc 7.5.0 as well, same results, works on Ubuntu, fails on godbolt.

On Windows: g++ 9.2.0, c++14:
error: '__builtin_strlen(((const char*)(& hello)))' is not a constant expression

Ok, so the problem wasn't the builtins, it's what we've been passing into them.
If you look more closely at the code you sent me and the code I compiled on my own, my char arrays are global while yours are local on the stack.
I have a vague idea why this makes them non constexpr, but it's not really interesting in our context here.
Just move the arrays out of main().

Ok, so the problem wasn't the builtins, it's what we've been passing into them.
...
Also, the pattern of declaring a char array and then constructing a string view from it is not very common or practically useful. Mostly it would be string_view("hello").compare(string_view("world")), which tests ok because the char arrays are implicitly global constants.

Yes :) Thanks a lot!

Great. I will pull this and test when I get back to doing some work.
BTW, this compiles:
int main()
{
    constexpr char const * hello = "hello";
    constexpr char const * world = "world";
    static_assert( absl::string_view( hello ).compare( absl::string_view( world ) ) < 0, "" );
}

So it's really a very specific code form that triggered the problem. I think with local char x[] = "something" the compiler feels compelled to place an actual array on the stack and initialize the string upon entry. Not sure this is required by the language.
I tried returning the compare() result, the code generated is terrible. It's probably a bad idea to have a locally initialized array on the stack.

I agree that my initial constexpr char hello[] = "hello"; should not prevent the application of __buitin_....
Will address that next.

How are you going to address that? I think it's a compiler issue.
In my view, if gcc thinks that constexpr char hello[] = "hello", is not constexpr enough to be fed into constexpr __builtin...() and produce a constexpr, then it should have protested at the declaration, saying that it cannot apply constexpr to hello.
As a programmer, I am disappointed in gcc for quietly letting me declare something constexpr and then refusing to use it as such. I could be naively writing code assuming that hello is constexpr and would resolve comparisons at compile time, while in fact gcc would be silently producing run-time code to do that. It's not reasonable to expect me to static_assert every other line just to verify that things get resolved the way I intended them to be.

How are you going to address that? I think it's a compiler issue.

What I meant is that failing the particular edge case of a function-local constexpr char hello[] = "hello"; etc. as in my example, should not withhold us from using f __buitin_... ().

As per Release 1.5.0, still not getting builtins with GCC due to precompiler symbol mixup:
This is never defined, nor is it found anywhere else.
Neither is this.

Well, that's embarrassing. I'll correct that now.
(How to prevent something like this happening?)

@mcskatkat Made another 'attempt' (#38)...

Works for me with Release 1.5.1. From my perspective, I think the issue can be closed.
I appreciate your diligent work on this 🙏

You're welcome, also thanks for you insistence, help and patience :)


[Info] How do you do static extraction?

Hi, awesome work! I'm writing a design system for my company and I'm evaluating some css-in-js options. Emotion has deprecated extractStatic. I saw you have a script to generate the extraction of css in a separate file but I have doubt: how do you remove the css in the component to avoid repetition?
Thanks in advance

@eatsjobs One point of clarification -- static CSS extraction is only meant to be used in non-React apps (i.e. HTML/CSS-only web pages). For React apps you obviously don't need it because it'll be injected for you. So you should not be in any scenario where you have duplicate CSS from the component because you only use extracted CSS when you cannot use the React components. Does that help?

Hey @eatsjobs I'll close the issue since the question seems to be answered now. Please feel free to reopen if you have any other questions!


train erro

Why does it get stuck during training without giving any error messages?

The same problem in gameformer Interaction Prediction，but training suscessully in fameformer Open-loop Planning .

I am not sure what's the problem. My guess is that a deadlock occurs in the data loading phase, so I would like to suggest reducing the number of worker threads in the DataLoader.

Although the above issue hasn't been resolved, I managed to complete the model training on another device using a Docker environment. One issue is that Docker can't visualize in real-time. Is it possible to save the visualization data as videos or images?

See #10.


Additional color schemes/color options

Current color scheme is analogous to a 'light' theme, which might not be preferable for all users. Would improve readability and user experience to implement additional themes (ie light & dark themes) for a variety of user preferences. Themes should be tracked via configuration file and be easy to adjust from the application using a form (settings component with form to control user color preferences)

[ ] Create settings component with form for adjusting user color preferences.
[ ] Get preference data from form and export to configuration file, use to assign color values to components in CSS.
[ ] Add a quick-toggle dark/light mode button to the application header for easy access.

https://github.com/toomeyDev/AngularTodo/commit/6bace1310911d966b8956b9230e99537b3cb5f82 added settings component to handle settings access and adjustment, still requires form to handle adjustment of settings on user side.


Update dependancies to run better on a Mac in 2024

I'm having some problems running the server locally on a 2020 macbook with an m1 chip because of outdated dependancies. I can clarify more in a follow up PR.

Closing as resolved.


${''} in new line will invalidate the whole css

See the following example:
import css from './pp.js'
export default () => <div className="pp">
	<style jsx>{css}</style>
</div>)

pp.js
export default `
	.pp {
		background-color: red;
		width: 100px;
		height: 100px;
	}
        ${''} /* no matter what is put, even nothing like this one, it will cause the whole style-jsx invalid*/
`

The stranger thing is the following is valid
export default `
	.pp {
		background-color: red;
		width: 100px;
		height: 100px;

		border: blue ${'13px'} solid  /* this one works*/
	}
`

By the way, I am using next.js 3 beta11 with syled-jsx 1.0.4.

I am fixing this in #227

fixed in #227


ImportError: No module named misc.logging

Dear
When I am trying to run the code It gives me this Error
No module named misc.logging
How can I solve this

The module is here. Please run the code according to the readme.

It seems that You don't add a description on the readme file only the results and requirements.

Please refer to README. It tells you how to run.

[mnabih@moissan Co]$ scripts/run_tasnet.sh
-bash: scripts/run_tasnet.sh: Permission denied

Please try to use bash scripts/run_tasnet.sh or add excutabel permission to scripts/run_tasnet.sh (like chmod +x scripts/run_tasnet.sh).
I think this is basic skills for Python and Shell and shouldn't be discussed here.


Render Document meta element higher up

Description
This ensure that if you set a <meta charSet="utf-8" /> (actually, any meta tag) in your Document, it will be rendered higher up, hence be valid and observed by browsers (charset must be in the first 1024 bytes).
https://spectrum.chat/react-static/general/hello-are-there-any-known-bugs-around-unicode-in-react-static~0ae1abd4-c64e-4f2c-8e74-63bf70479528
Changes/Tasks

[x] Render children meta element above the rest

Motivation and Context
Fix the issue described here: https://spectrum.chat/react-static/general/hello-are-there-any-known-bugs-around-unicode-in-react-static~0ae1abd4-c64e-4f2c-8e74-63bf70479528
Types of changes

[ ] Refactoring/add tests (refactoring or adding test which isn't a fix or add a feature)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the documentation accordingly
[ ] I have updated the CHANGELOG with a summary of my changes
[x] My changes have tests around them

Perfect


region: fix get resource details panic

这个 PR 实现什么功能/修复什么问题:

是否需要 backport 到之前的 release 分支:

3.1
/area region
/cc @wanyaoqi

Codecov Report

Merging #6481 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6481      +/-   ##
==========================================
- Coverage    7.17%    7.17%   -0.01%
==========================================
  Files         646      646
  Lines      108928   108928
==========================================
- Hits         7819     7817       -2
- Misses     100509   100510       +1
- Partials      600      601       +1

Flag
Coverage Δ

#aFlag
7.17% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
66.17% <0.00%> (-2.95%)
:arrow_down:

pkg/cloudcommon/validators/validators_cert.go
39.77% <0.00%> (ø)

Continue to review full report at Codecov.

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


New Feature: Add restrict css to the spider by UI

Add the ability to select specific divs and tags to be added to the spider's linkextractor restrict css tool like annoation UI. This feature add a great usability to users who doesn't know regex to add allow and deny rules and also will incredibly increase the performance of the spider in certain cases.

I am asking whether this feature is on your plan to implement ?


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.9 to before v4.14.257",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=7f9664525f9cb507de9198a395a111371413f230",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=550658a2d61e4eaf522c8ebc7fad76dc376bfb45"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "7f9664525f9cb507de9198a395a111371413f230",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "550658a2d61e4eaf522c8ebc7fad76dc376bfb45",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net: qlogic: qlcnic: Fix a NULL pointer dereference in qlcnic_83xx_add_rings()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.257 by commit 550658a2d61e4eaf522c8ebc7fad76dc376bfb45, it was introduced in version v3.9 by commit 7f9664525f9cb507de9198a395a111371413f230. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002395


get_node_type isn't working

What happened?
When calling get_node_type(), the operation always return Standard_D3_v2.
From my initial investigation, it seems like there's no API call made to the provider endpoint in order to retrieve the list of supported node types. As I use a proxy (squid-cache), my network traffic is funneled there. If I run a curl command to the databricks endpoint, then I'm able to retrieve a JSON of supported node types. However, when I run the pulumi code provided, I don't see any traffic to the endpoint at all.
Looking at the upstream code, if the list/json is empty, then Standard_D3_v2 via defaultSmallestNodeType()
The problem seem to occur with both Azure and AWS.
Steps to reproduce

Create a new Pulumi app (Python) using the code below
Update subscription_id (L9), tenant_id (L10)
Comment out node_type = pulumi_databricks(... (L63-76)
Comment out pulumi.export("node_type", ... (L79)
Run pulumi up
Once deployed, Go to the Azure portal > Resource Groups > case-2022-db and click 'Launch Workspace`
Then, in the databricks UI, top right click on your email address > User Settings > Generate Token
Update db_token (L11)
Enable commented code from step 3 and 4
Run pulumi up
Observe the node_type value as a stack output

I've commented some code for you to fiddle with (L65-71) but regardless of the values, the same result is shown.
Verbose logs attached below out.txt
pulumi up --yes --skip-preview --verbose 9 --logtostderr 2> out.txt

Expected Behavior

The node type returned changes based on the parameters provided to select the correct node type.

Actual Behavior

With the code provided on Azure, Standard_D3_v2 is always returned.

Output of pulumi about
CLI
Version      3.46.1
Go Version   go1.19.2
Go Compiler  gc

Plugins
NAME          VERSION
azure-native  1.86.0
databricks    1.4.0
python        unknown

Host
OS       debian
Version  11.5
Arch     x86_64

This project is written in python: executable='/home/aureq/work/customers/mbrdna/case-2022/venv/bin/python3' version='3.9.15
'

Current Stack: case-2022

TYPE                                  URN
pulumi:pulumi:Stack                   urn:pulumi:case-2022::zendesk::pulumi:pulumi:Stack::zendesk-case-2022
pulumi:providers:azure-native         urn:pulumi:case-2022::zendesk::pulumi:providers:azure-native::case-2022-az-provider
azure-native:resources:ResourceGroup  urn:pulumi:case-2022::zendesk::azure-native:resources:ResourceGroup::case-2022-rg
azure-native:databricks:Workspace     urn:pulumi:case-2022::zendesk::azure-native:databricks:Workspace::case-2022-db
pulumi:providers:databricks           urn:pulumi:case-2022::zendesk::pulumi:providers:databricks::case-2022-db-db-provider

Found no pending operations associated with case-2022

Backend
Name           pulumi.com
URL            https://app.pulumi.com/aureq
User           aureq
Organizations  aureq, team-ce, menfin, menfin-team, demo, pulumi

Dependencies:
NAME                 VERSION
pip                  22.3.1
pulumi-azure-native  1.86.0
pulumi-databricks    1.4.0
setuptools           65.5.1
wheel                0.38.4

Additional context
Here is the Pulumi app.
"""An Azure RM Python Pulumi program"""
import os
from enum import Enum
import pulumi
import pulumi_azure_native
import pulumi_databricks

# change me (x3)
subscription_id = "12345678-1234-1234-1234-1234567890ab"
tenant_id = "09876543-6789-6789-6789-ab0987654321"
db_token = "dapixxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-x"

class DatabricksNodeCategory(str, Enum):
    GENERAL_PURPOSE = "General Purpose"
    GENERAL_PURPOSE_HDD = "General Purpose (HDD)"
    COMPUTE_OPTIMIZED = "Compute Optimized"
    MEMORY_OPTIMIZED = "Memory Optimized"
    MEMORY_OPTIMIZED_REMOTE_HDD = "Memory Optimized (Remote HDD)"
    STORAGE_OPTIMIZED = "Storage Optimized"
    GPU_ACCELERATED = "GPU Accelerated"

base_name = "case-2022"
rg_name = f'{base_name}-rg'
location = "West Europe"
dbname = f'{base_name}-db'

az_provider = pulumi_azure_native.Provider(
    resource_name=f"{base_name}-az-provider",
    subscription_id=subscription_id,
    tenant_id=tenant_id
)

resource_group = pulumi_azure_native.resources.ResourceGroup(resource_name=rg_name,
    location=location,
    resource_group_name=rg_name,
    opts=pulumi.ResourceOptions(
        provider=az_provider
    )
)

workspace = pulumi_azure_native.databricks.Workspace(dbname,
    location=location,
    managed_resource_group_id=f"/subscriptions/{subscription_id}/resourceGroups/{rg_name}-dbmanaged",
    resource_group_name=rg_name,
    workspace_name=dbname,
    opts=pulumi.ResourceOptions(
        depends_on=resource_group,
        provider=az_provider
    )
)

db_provider = pulumi_databricks.Provider (
    resource_name=f"{dbname}-db-provider",
    auth_type="token",
    token=db_token,
    azure_workspace_resource_id=workspace.workspace_id,
    host=workspace.workspace_url,
    http_timeout_seconds=90,
    azure_tenant_id=tenant_id,
)

node_type = pulumi_databricks.get_node_type(
    min_cores=32,
    # local_disk=True,
    # gb_per_core=1,
    # min_gpus=1,
    # category=DatabricksNodeCategory.GENERAL_PURPOSE,
    # min_memory_gb=56,
    # min_cores=16,
    # local_disk=1,
    opts=pulumi.InvokeOptions(
        provider=db_provider,
        parent=db_provider
    )
)

pulumi.export("url", workspace.workspace_url)
pulumi.export("node_type", node_type.id)

Here is the cURL command I used to get the JSON node_types.zip. The correct base host is obtained via workspace.workspace_url.
curl -X GET -H 'Authorization: Bearer <datasbricksToken>' 'https://adb-XXXXXXXXXXXXXXXX.XX.azuredatabricks.net/api/2.0/clusters/list-node-types' | jq . > node_types.json

Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for the detailed report!
Checked that both Python and Go (as sanity check) SDKs invoke the provider's databricks:index/getNodeType:getNodeType function. Digging further.

This comment in the TF provider, just before the part you linked to, is suspicious:
func (a ClustersAPI) GetSmallestNodeType(r NodeTypeRequest) string {
	list, _ := a.ListNodeTypes()
	// error is explicitly ingored here, because Azure returns
	// apparently too big of a JSON for Go to parse

The call to ListNodeTypes could fail for any number of reasons. I might have to step through in a debugger.

ok, so by the sound of it, there's a data source that's not functional if I read your comment correctly, right?

Turns out the data source is functional, just sparsely documented and it doesn't surface errors you would need to see.
I finally found time to set up (very well documented!) repro and step through in the debugger. What happens is that the query to list node types fails in the setup phase before it's sent (hence you don't see it in the proxy).
"cannot configure token auth. Attributes used: auth_type, azure_tenant_id, azure_workspace_resource_id, host, token. Please check https://registry.terraform.io/providers/databricks/databricks/latest/docs#authentication for details"

That's why we don't swallow errors as in the // error is explicitly ingored here above...
After a bit more experimentation, it turns out it's a misunderstanding in how the provider is configured in your program.
    auth_type="token",
    token=db_token,

The token property is meant for token-based authentication to the cloud provider, not to Databricks.
Simply removing those two, and setting databricks:token and databricks:host instead as per our docs, makes the authentication part succeed for me (after prior az login).
The API request then fails server-side with "400 Invalid resource ID" but that's probably easily fixable.
I hope that helps!

@thomas11 this doesn't seem to work for me unfortunately...
Here's a snippet of the code I changed
# [...]
db_provider = pulumi_databricks.Provider (
    resource_name=f"{dbname}-db-provider",
    # auth_type="token",
    # token=db_token,
    azure_workspace_resource_id=workspace.workspace_id,
    host=workspace.workspace_url,
    http_timeout_seconds=90,
    azure_tenant_id=tenant_id,
)

node_type = pulumi_databricks.get_node_type(
    min_cores=16,
    # local_disk=True,
    # gb_per_core=1,
    # min_gpus=1,
    category=DatabricksNodeCategory.GENERAL_PURPOSE_HDD,
    # min_memory_gb=56,
    # min_cores=16,
    # local_disk=1,
    opts=pulumi.InvokeOptions(
        provider=db_provider,
        parent=db_provider
    )
)
# [...]

config:
  azure-native:location: West Europe
  databricks:host: adb-5758842155562043.3.azuredatabricks.net
  databricks:token:
    secure: AAABAC14nm1wwRvqG77MbMfbf70So3HutDQfFpzu/DPRZxHxt38EsnH+n0ZQ1+rCck2HEEjmyt/Y/BhPoOA/8K5bE/gcTQ==

I1121 11:12:22.444219   19052 eventsink.go:59] Using Azure CLI authentication with AAD tokens
I1121 11:12:22.444276   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>Using Azure CLI authentication with AAD tokens<{%reset%}>)
I1121 11:12:22.985069   19052 eventsink.go:59] Refreshed OAuth token for https://management.core.windows.net/ from Azure CLI, which expires on 2022-11-21 12:26:15.000000
I1121 11:12:22.985124   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>Refreshed OAuth token for https://management.core.windows.net/ from Azure CLI, which expires on 2022-11-21 12:26:15.000000<{%reset%}>)
I1121 11:12:23.561706   19052 eventsink.go:59] Refreshed OAuth token for 2ff814a6-3304-4ab8-85cb-cd0e6f879c1d from Azure CLI, which expires on 2022-11-21 11:39:58.000000
I1121 11:12:23.561778   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>Refreshed OAuth token for 2ff814a6-3304-4ab8-85cb-cd0e6f879c1d from Azure CLI, which expires on 2022-11-21 11:39:58.000000<{%reset%}>)
I1121 11:12:23.562174   19052 eventsink.go:59] Configured azure-cli auth: host=https://adb-5758842155562043.3.azuredatabricks.net, azure_workspace_resource_id=5758842155562043, azure_tenant_id=706143bc-e1d4-4593-aee2-c9dc60ab9be7
I1121 11:12:23.562212   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>Configured azure-cli auth: host=https://adb-5758842155562043.3.azuredatabricks.net, azure_workspace_resource_id=5758842155562043, azure_tenant_id=706143bc-e1d4-4593-aee2-c9dc60ab9be7<{%reset%}>)
I1121 11:12:23.562675   19052 eventsink.go:59] GET /api/2.0/clusters/list-node-types
I1121 11:12:23.562710   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>GET /api/2.0/clusters/list-node-types <{%reset%}>)
I1121 11:12:24.678530   19052 eventsink.go:59] 400 Bad Request [non-JSON document of 286 bytes]
I1121 11:12:24.678594   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>400 Bad Request [non-JSON document of 286 bytes]<{%reset%}>)
I1121 11:12:24.679185   19052 eventsink.go:59] /api/2.0/clusters/list-node-types:400 - Invalid resource ID https://docs.databricks.com/dev-tools/api/latest/clusters.html#list-node-types
I1121 11:12:24.679245   19052 eventsink.go:62] eventSink::Debug(<{%reset%}>/api/2.0/clusters/list-node-types:400 - Invalid resource ID https://docs.databricks.com/dev-tools/api/latest/clusters.html#list-node-types<{%reset%}>)
I1121 11:12:24.679944   19052 rpc.go:274] Unmarshaling property for RPC[Provider[databricks, 0xc0009ddc20].Invoke(databricks:index/getNodeType:getNodeType).returns]: category={General Purpose}
I1121 11:12:24.679987   19052 rpc.go:274] Unmarshaling property for RPC[Provider[databricks, 0xc0009ddc20].Invoke(databricks:index/getNodeType:getNodeType).returns]: id={Standard_D3_v2}
I1121 11:12:24.680004   19052 rpc.go:274] Unmarshaling property for RPC[Provider[databricks, 0xc0009ddc20].Invoke(databricks:index/getNodeType:getNodeType).returns]: minCores={16}
I1121 11:12:24.680032   19052 provider_plugin.go:1357] Provider[databricks, 0xc0009ddc20].Invoke(databricks:index/getNodeType:getNodeType) success (#ret=3,#failures=0) success

And from your comment, I have a few questions.

What are the differences as to why it's working for you and not me? (happy to work together when you're back on Monday)
How to configure the provider programmatically? (ie, not via env nor Pulumi.<stack>.yaml? (I could trick using os.environ but it doesn't feel clean. In the logs above, seems to be using AAD token though.
If the provider isn't set correctly, why no error is shown? because that adds a lot of confusion to the user.
setting DATABRICKS_HOST and DATABRICKS_TOKEN results in an error More than one authorization method configured: azure and token but not when databricks:host and databricks:token are set in the YAML which is also confusing.

Thank you.

Seems like it's working for you, no? Note that I said above

The API request then fails server-side with "400 Invalid resource ID" but that's probably easily fixable.

Like for you. You should debug why that is, but we're a step further.
As to how to configure a provider programmatically, you can do this with explicit providers.

Going to close this as stale since there have been quite a few releases of the provider since, and it seems like this may not be an issue with the provider itself – please reopen if any further issues/ questions!


$convert between XML and JSON is inconsistent

Describe the bug
When using the $convert operation the transformation of a resource turns out to be inconsistent.
Environment
4.7.1
To Reproduce
Steps to reproduce the behavior - XML to JSON:

Take the following resource as XML and pass it on to the $convert operation:<CodeSystem xmlns="http://hl7.org/fhir">
    <name value="test-codesystem"/>
    <title value="Test Code System"/>
    <status value="active"/>
    <date value="2016-11-24"/>
    <description value="XXX

XXX"/>
    <content value="complete"/>
    <concept>

        <display value="TEST CODE"/>
    </concept>
</CodeSystem>

The resulting JSON representation of the resource will be:{
    "resourceType": "CodeSystem",
    "name": "test-codesystem",
    "title": "Test Code System",
    "status": "active",
    "date": "2016-11-24",
    "description": "XXX\n\nXXX",
    "content": "complete",
    "concept": [
        {
            "code": "0",
            "display": "TEST CODE"
        }
    ]
}

Steps to reproduce the behavior - JSON to XML:

Take the following resource as JSON and pass it on to the $convert operation:{
    "resourceType": "CodeSystem",
    "name": "test-codesystem",
    "title": "Test Code System",
    "status": "active",
    "date": "2016-11-24",
    "description": "XXX\n\nXXX",
    "content": "complete",
    "concept": [
        {
            "code": "0",
            "display": "TEST CODE"
        }
    ]
}

The resulting XML representation of the resource will be:<CodeSystem xmlns="http://hl7.org/fhir">
    <name value="test-codesystem"/>
    <title value="Test Code System"/>
    <status value="active"/>
    <date value="2016-11-24"/>
    <description value="XXX

XXX"/>
    <content value="complete"/>
    <concept>

        <display value="TEST CODE"/>
    </concept>
</CodeSystem>

Expected behavior
When converting from JSON to XML the \n should be transformed to
. Then it would be possible to convert between XML and JSON consistently.

https://www.w3.org/TR/REC-xml/#AVNormalize has me agreeing this could be an issue.
Based on my reading of that, an XML parser would parse
<description value="XXX

XXX"/>

to an attribute value of
XXX

XXX

whereas it would parse
<description value="XXX

XXX"/>

to an attribute value of
XXX XXX

We don't do any special $convert logic, we just read the resource and then write it back out using XMLStreamWriter.writeAttribute.
Unfortunately, I don't see any supported configuration for preserving the newlines.
Its basically the same issue that is reported here:
https://stackoverflow.com/questions/8331364/how-to-preserve-whitespace-in-attributes-when-using-xmlstreamwriter
I'd lean toward moving this to the icebox and reporting a related issue upstream. How important is this one to you?

I added a test at https://github.com/LinuxForHealth/FHIR/pull/3816/files#diff-8ec2e5b4061497aaf09998b194bb8e294ab785f38c512b5d64119ed4940e7463R23-R38 to demonstrate the root of the issue which has nothing to do with JSON and everything to do with XML.


Extension to Video Datasets

How do we extend x-decoder to video datasets? In the appendix, it is mentioned that the model generalizes to generic segmentation and referring segmentation on videos.

Thanks for your question, the evaluated video dataset is simply evaluated frame by frame. For referring segmentation, the referring phrase is a natural tracking id, for segmentation we didn't apply any tracking.


Created Sinhala language translation of call-the-create-api.md file #491

cc: @jayair
This is my first pull request. Please have a look 😁. Thank you

Thank you!

It's my pleasure 😊


Please provide an example using sequelize v5 and TypeScript.

In particular, how to organize the models when using sequelize v5 and TypeScript?
I wonder can we remove this setup file: https://github.com/sequelize/express-example/blob/master/models/index.js and organize the models file just using export and import, and combine these models using some method.
The setup file executes immediately when importing it, or import some modules depend on it. Sometimes, It will affect the unit test when importing some modules into test files.
E.g.
models/index.ts:
import {User} from './user';
import {Role} from './Role';
import {Project} from './Project';

const models = combineModels([User, Role, Project])
// ...
export {models};

@papb I made a example using sequelize v5 and TypeScript. But I met an issue, it blocks me one week. Can you help me solve this issue? I also submit a post on stackoverflow, but still has no working answer.
I am thinking if the organization of models is correct or not.
Here is the post on SO: https://stackoverflow.com/questions/60054969/belongs-to-many-associations-add-foreign-keys-to-source-model-sequelizedatabase
Here is the example repo: https://github.com/mrdulin/node-sequelize-examples/tree/master/src/db
Thanks!

I figured it out.

Hi @mrdulin sorry I couldn't give you more attention, I have too little time hehe. Glad you figured it out. Can you please share it? Or update your github repository? When I have time I would like to take a look and incorporate it as an example for this repository, of course giving all credits to you.  I passed a wrong modelName(Category) to the Language model. After fixing it, the integration tests passed. I already updated the examples in the repo.

Hi @mrdulin, we have another issue (#104) also requesting a TypeScript example, so I will close this one as a duplicate for now. The current JS example has been updated to use v6 and should work with v5 as well.
Hopefully I will add a TS example in the near future. Thank you again!


Adding PDF output with Hyperlinks

->This modification gives  the user and option for the type of output file.  For PDF input is 1 and for .txt
input is 2. Users can now directly click on the link of the pdf to open the article instead of manually
copy pasting the URL.
->The  encode('latin-1', 'ignore')  code eliminates any character which is not included in the ASCII code.
This is essential for writing the text in .pdf format.  Next the decode('latin-1') part again decodes the
rectified string into byte-form as the FPDF.cel() command takes input in byte-form in Python 3.
->The text color is in black and the link color is in blue in the .pdf output.
->In the line f= open("output.txt","a+", encoding='utf8') encoding='utf8' is required because the
default encoding may vary according to the environment. Attaching a screenshot from StackOverflow
regarding the matter.
Link ->https://stackoverflow.com/questions/42070668/python-3-default-encoding-cp1252

Can you update relevant sections of readme and possibly a screenshot with it?

Updated the Readme.
Solves #1 #3

Hey @KanishkaHalder1771, can you add a relevant ss for the pdf file after the ss of the text file?

Sorry, couldn't understand what you wanted. Can you explain what you mean by 'ss'

My bad. I meant screenshots @KanishkaHalder1771

Added the screenshot in README.md and file is located in /images/output_pdf.PNG


namespace step missing for production-recommended-secure-setup

Can we please add a step to create a namespace and setting the namespace for the k8s context on production-recommended-secure-setup page?

kubectl config set-context --current --namespace confluent
https://docs.confluent.io/operator/current/co-quickstart.html#step-1-create-a-namespace

Another suggestion would be to add the --namespace <namespace> to the commands used on secure setup page.

This has been addressed with PR #30


Inaccurate information in Tasks documentation

In the tasks documentation page:
https://code.visualstudio.com/docs/editor/tasks
The page states:
"Select the Tasks: Configure Task Runner command and you will see a list of task runner templates. Select Others to create a task which runs an external command."
but the templates are ONLY presented after clicking on "Configure task runner" if there is NO tasks.json file.  If you already have a tasks.json file the instruction is confusing as the templates don't appear.

This issue was moved to Microsoft/vscode-docs#1052


[FEATURE] Softer validation of corpora workload parameters for vectorsearch benchmark

Is your feature request related to a problem? Please describe
Currently each workload corpus requires a target index parameter when there are multiple indices. However, the vector search bulk ingest workload operation (bulk-vector-data-set) does not use this target index when ingesting data. Instead, users specify the target index as a parameter in the custom-bulk ingest operation in their test procedure.
I'm opening this issue because the target-index parameter is required at workload validation time despite it being unnecessary for vector search workloads. As a result the VS workload.json must contain unused parameters.
Describe the solution you'd like
One solution is to make the target-index corpora parameter optional at validation time. Perhaps it's also possible to enforce that either the parameter is specified in the corpora in workloads.json or that bulk-vector-data-set is used in a test procedure.
Describe alternatives you've considered
I don't have all the context for why there are two ways of specifying the target index for ingesting data but I believe it's due to vector datasets being in hdf5 format and the normal bulk operation requiring json documents.
Additional context
There is another issue stemming from VS ingestion being different than normal ingestion — Issue 317 in the workloads repo requests a VS feature that's available in non-VS workloads. I think the lack of feature parity is due to the vector-bulk operation being different from the normal bulk operation.

Thanks for bringing this up @finnroblin. I agree, it'd be convenient not to specify unused parameters. Of the two approaches suggested, I prefer the second because target-index is still a requirement for workloads other than Vector Search.
One way we can circumvent this pain point is to rearrange test procedure creation to be before corpora in loader.py
# Switch these in loader.py so that test_procedures is created first.
  corpora = self._create_corpora(self._r(workload_specification, "corpora", mandatory=False, default_value=[]),
                                 indices, data_streams)
  test_procedures = self._create_test_procedures(workload_specification)

We can pass in test_procedures object to self._create_corpora() so that we have access to the operations / schedule and check if bulk-vector-data-set operation exists in the list. If so, target-index is not necessary.
If you (or anyone else) think of any cleaner approaches, feel free to propose and implement them.


Missing fields

Hi Josemmo,
Love this package! It seems that there are still some fields missing.
E.g. AccountingCostCode is a field that can be placed in the invoice or invoice lines.
Is it possible to add these fields, or is it perhaps possible to add these using a general function?
Something like
public function setCustomValues(string $fieldName, string $fieldValue) {
        $this->{$fieldName} = $fieldValue;
        return $this;
    }

Hello @ThisIsMyFavouriteJar,
What field are you referring to? I cannot find any field with that name in the PEPPOL specification or in EN 16931.
If it's cbc:AccountingCost, that's already implemented.

Closed due to inactivity.


feat(hmr): modify serve.host print the new urls

Description
The default configuration of vite.config does not expose the network url. When I modify the configuration to host: true, the server automatically updated, but the new url is not printed, so I added this.

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the Commit Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

#6075 is touching a similar topic as this, and the change there within restartServer() is a bit more robust. Perhaps we can close this and move the host handling to #6075 as well to prevent merge conflicts.

Closing this in favor of #6075, but thanks for your effort on this too! 🙂


ENH: Add one more derivative-free optimization method

Is your feature request related to a problem? Please describe.
General derivative-free optimization. Currently, the only options are

Nelder-Mead
Powell
COBYLA

Describe the solution you'd like.
The paper http://dx.doi.org/10.1007/s10898-012-9951-y from 2011 conducts an extensive benchmark study of 22 derivative-free optimization algorithms (see also http://infinity77.net/go_2021/thebenchmarks.html from 2021 and http://www.optimization-online.org/DB_FILE/2008/01/1883.pdf from 2008). They conclude

The solvers TOMLAB/MULTIMIN, TOMLAB/GLC-CLUSTER, MCS and TOMLAB/LGO, on average, provide the best solutions among all the solvers tested. However, computational results show that there is no single solver whose performance dominates that of all others. In addition, all solvers provided the best solution possible for at least some of the test problems. Although no subset of solvers suffices to solve all problems, our results suggest that the combination of the commercial TOMLAB/MULTI-MIN and TOMLAB/GLCCLUSTER with the free MCS and SNOBFIT is sufficient to provide the best results in most cases. Problem dimensionality and nonsmoothness were found to rapidly increase the complexity of the search and decrease performance for all solvers. Finally, from a starting point close to a solution, TOMLAB/OQNLP, NEWUOA and TOMLAB/MULTI-MIN showed the fastest convergence towards the solution. Missing bounds on the variables are found to affect significantly the performance of all solvers, particularly the stochastic ones.

Therefore, I propose to consider implementing MCS (Matlab code, license unclear), which, unfortunately, might be a lot of work.
Describe alternatives you've considered.
https://github.com/scipy/scipy/issues/1477
Additional context (e.g. screenshots, GIFs)
No response

You may be interested in gh-14300.

@mdhaber That's great.
@infinity77 Thanks to @mdhaber's hints, I landed on your great benchmark page and also saw that you have implemented MCS in python according to http://infinity77.net/go_2021/mcs.html#mcs. Would you be interested in sharing your inplementation with scipy in case the core devs give green light for inclusion?

Another algorithm which ranked high in the benchmark was biteopt. A Python wrapper is available under MIT license. Might be easier to integrate than MCS which looks very complicated from the paper and the Matlab code unfortunately .

Therefore, I propose to consider implementing MCS (Matlab code, license unclear), which, unfortunately, might be a lot of work.

@mnasini opened gh-15155 with the start of an implementation. Does this make sense before clarifying the status of the license? Or is it a reimplementation from scratch without looking at the Matlab code?

@rgommers sorry for the late answer, i'm quite busy right now :) .
For the box definition and the initialisation step, i'm performing a porting from julia to python from this repository https://github.com/timholy/MultilevelCoordinateSearch.jl. which is licensed under a mit 'expat' licence.
For the split part, instead i'm writing a fresh implementation from the original paper (without looking at the matlab code).

With #14300 merged, there is a further solver. So I close this issue.

For future reference for anyone else who stumbles upon this thread.
Actually, scipy has more derivative-free solvers when default options are overriden. dual_annealing can run without local search (no_local_search=False) and differential_evolution runs without derivatives if polish=False is set. Both are very capable global optimizers.
It seems like it takes some experience and digging through the documentation to properly use `scipy.optimize. Maybe this can be seen as another data point hinting towards a necessary documentation upgrade?

fwiw, NEWUOA is available in NLopt

x-ref gh-1477 gh-18118


Support invalid partToExtract for parse_url

Closes #11659
In parse_url we used to fallback to cpu if the partToExtract was not valid. However, the behaviour of cpu is just to always return null, we can do that too easily.
But in customer case #11659, it did run on gpu because GpuOverrides thought lowercase path was valid, which is not, so it went to a defensive branch which threw an exception.
This pr supports invalid partToExtract in parse_url running on gpu, and fixes the bug in it's override.

build


Add YAML generation comand line scripts

Checklist

[X] Used a fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Cannot set attribute on  element

Hi, thanks for the great work on this crate! Am I right in assuming there's no support for attributes on the <html> element?
It is customary in HTML5 to always use a language attribute on the <html> element to declare the default language of the text in the page. I can't see any apparent way to do this in Maud, since the <html> is implicit.

I just now realized I simply need to do
html! {
    (DOCTYPE)
    html class="no-js" lang="" {
        head {
            …
        }
        body {
            …
        }
    }
}

Closing as PEBKAC. Sorry about the noise.


Numeric Data Type Definitions

At the annual meeting, we agreed to revise the current Representation System into a smaller and more manageable list of types.   I'm attaching a draft proposal for the Numeric Types going forward.

NumericDTDs.xlsx

Updated List to include Yield Volume and Alphabetic Codes
NumericDTDs.pdf

Rerouting for confirmation that upon moving away from required SI base units, we no longer expect scientific notation.


Add note about accessing real files in pytest

Should give pytest users a way to load test resources that does fit with
the style that pytest is normally used.

Here are the updated docs.


can the debug panel change it's position?

It always conceal some file tab.

I means this panel. Can I move to other place?

Yeah it's a duplicate, as the smart bot says.


Segmentation fault

when run the script “python tools/train.py configs/faster_rcnn_r50_fpn_1x.py --gpus 1 --work_dir logs --validate ”,meet the problem
2018-10-13 23:54:59,086 - INFO - workflow: [('train', 1)], max: 12 epochs
Segmentation fault

Such problems are hard to identify remotely... It may be related to the environment or hardwares and we cannot reproduce it. Maybe you need to debug on your own by printing more information.

Updates: Installing PyTorch with conda and compiling cuda extensions with gcc 4.8 will cause the segmentation fault. Either compiling PyTorch from source or using gcc 5 to build ops solves the problem.
@yanxp You may check this and see if it works.

@hellock Thanks. It works now.

Updates: Installing PyTorch with conda && compiling cuda extensions with gcc 4.8 will cause the segmentation fault. Either compiling PyTorch from source or using gcc 5 to build ops solves the problem.
@yanxp You may check this and see if it works.

Hi! @hellock
I install gcc 5.5.0 from source in my own folder (ubuntu 14.04), and I reinstall mmcv and mmdet. While running "python tools/test.py", but also got "[ ] 0/5000, elapsed: 0s, ETA:bash: line 1: 6150 Segmentation fault (core dumped)". So how to solve this problem? Must I reinstall pytorch for source?

Did you run "python setup.py install" again ?

I have the same segfault issue

You can check whether the segmentation fault is caused by custom operators (RoIAlign, etc.). Usually such problems are related to the compiling environment and you may try compiling pytorch from source. Another attempt can be switching to pytorch 1.0.

I installed pytorch1.0, gcc4.8.5 also have error with segmentation fault. My environment is  in conda env, when I use "conda install -c serge-sans-paille gcc_49" to change version, it also not work. And have "error: command 'gcc' failed with exit status 1" when I run 'python setup.py develop'. Please tell me how to  do next.

Updates: Installing PyTorch with conda && compiling cuda extensions with gcc 4.8 will cause the segmentation fault. Either compiling PyTorch from source or using gcc 5 to build ops solves the problem.
@yanxp You may check this and see if it works.

Thanks for your advice , I have solve the segmentation falut after update gcc-4.8.5 to gcc-5.3.0

Hello, I meet the same error. But I want to ask, when I upgrade gcc to 5.3.0, do I need to rebuild the mmdet again. Thanks


The gem prevents the app to start with Rails 6 beta 1

I am testing my app with the newly release Rails 6 beta 1 and it is breaking when deploying with Honeycomb:
ActiveRecord::AdapterNotSpecified: ActiveRecord::AdapterNotSpecified
  ./shared/bundle/ruby/2.6.0/gems/activerecord-6.0.0.beta1/lib/active_record/connection_adapters/connection_specification.rb:142:in `resolve'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-honeycomb-0.4.0/lib/active_record/honeycomb.rb:30:in `resolve_config'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-honeycomb-0.4.0/lib/active_record/honeycomb.rb:7:in `munge_config'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-honeycomb-0.4.0/lib/activerecord-honeycomb/auto_install.rb:20:in `block (2 levels) in auto_install!'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-import-0.28.1/lib/activerecord-import/import.rb:242:in `establish_connection_with_activerecord_import'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-6.0.0.beta1/lib/active_record/railtie.rb:193:in `block (2 levels) in <class:Railtie>'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:72:in `class_eval'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:72:in `block in execute_hook'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:62:in `with_execution_control'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:67:in `execute_hook'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:43:in `block in on_load'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:42:in `each'
  ./shared/bundle/ruby/2.6.0/gems/activesupport-6.0.0.beta1/lib/active_support/lazy_load_hooks.rb:42:in `on_load'
  ./shared/bundle/ruby/2.6.0/gems/activerecord-6.0.0.beta1/lib/active_record/railtie.rb:191:in `block in <class:Railtie>'
  ./shared/bundle/ruby/2.6.0/gems/railties-6.0.0.beta1/lib/rails/initializable.rb:32:in `instance_exec'
  ./shared/bundle/ruby/2.6.0/gems/railties-6.0.0.beta1/lib/rails/initializable.rb:32:in `run'
  ./shared/bundle/ruby/2.6.0/gems/railties-6.0.0.beta1/lib/rails/initializable.rb:61:in `block in run_initializers'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:228:in `block in tsort_each'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:350:in `block (2 levels) in each_strongly_connected_component'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:431:in `each_strongly_connected_component_from'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:349:in `block in each_strongly_connected_component'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:347:in `each'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:347:in `call'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:347:in `each_strongly_connected_component'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:226:in `tsort_each'
  /usr/local/rbenv/versions/2.6.0/lib/ruby/2.6.0/tsort.rb:205:in `tsort_each'
  ./shared/bundle/ruby/2.6.0/gems/railties-6.0.0.beta1/lib/rails/initializable.rb:60:in `run_initializers'
  ./shared/bundle/ruby/2.6.0/gems/railties-6.0.0.beta1/lib/rails/application.rb:361:in `initialize!'
  ./releases/20190126144941/config/environment.rb:7:in `<top (required)>'
  config.ru:9:in `require_relative'
  config.ru:9:in `block in <main>'
  ./shared/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/builder.rb:55:in `instance_eval'
  ./shared/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/builder.rb:55:in `initialize'
  config.ru:in `new'
  config.ru:in `<main>'
  ./shared/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/builder.rb:49:in `eval'
  ./shared/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/builder.rb:49:in `new_from_string'
  ./shared/bundle/ruby/2.6.0/gems/rack-2.0.6/lib/rack/builder.rb:40:in `parse_file'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/configuration.rb:318:in `load_rackup'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/configuration.rb:243:in `app'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/runner.rb:145:in `load_and_bind'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/single.rb:96:in `run'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/launcher.rb:184:in `run'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/lib/puma/cli.rb:78:in `run'
  ./shared/bundle/ruby/2.6.0/gems/puma-3.12.0/bin/puma:10:in `<top (required)>'
  ./shared/bundle/ruby/2.6.0/bin/puma:23:in `load'
  ./shared/bundle/ruby/2.6.0/bin/puma:23:in `<top (required)>'

When disabling the beeline in config.ru the app works fine.
I am not sure whats happening here. While upgrading I noticed that when loading YAML config (like database.yml) Rails is now using a hash with symbols as keys, where it before accepted strings in getters, maybe it is related?

FYI, Rails 6 is in release candidate now. (This blocked me from trialing Honeycomb on a Rails 6 app.)

@jeremy @renchap hey, thanks for the report, and sorry for the delayed response. We are working on a longer-term architectural solution that changes the way the beeline is configured that I believe will fix this, but I will also see if we can look into whether there's a quick fix that would unblock using us with a Rails 6 app sooner.

@renchap @Jeremy v1.0.0 has been released now which should support rails 6!


Resizing images keeping aspect-ratio?

Hi everybody! Its possible to define Jquery croppie when resizable function is available, to keep the aspect-ratio... the relation between Width/Height? something like this 2:1 .e.g if I resize my image with 500x250 this it's good! but I dont want to change with the square and generate an image 400x250... but also 400x200? or 100x50? Keeping the scale 2:1 when I resize the image using the "square"?
Now I can make this:

Is it possible to keep relation Width/Height when I resize? Or how Can I update my code to allow this functionality?
Thanks!
Ani

related with : https://github.com/Foliotek/Croppie/issues/253
and no solution yet.


fix: remove golib dep

Thank you for contributing to vc!
Please add a summary of your change
fix: remove golib dep
Does your change fix a particular issue?
Fixes #(issue)
n/a
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

Codecov Report

Merging #13 (74d1756) into main (d29662d) will increase coverage by 0.11%.
The diff coverage is 62.50%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   82.13%   82.25%   +0.11%
==========================================
  Files           4        4
  Lines         599      603       +4
==========================================
+ Hits          492      496       +4
  Misses         75       75
  Partials       32       32

Flag
Coverage Δ

unittests
82.25% <62.50%> (+0.11%)
:arrow_up:

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

Files Changed
Coverage Δ

constraints.go
72.84% <0.00%> (ø)

compare.go
88.97% <71.42%> (+0.35%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Correction for raw affiliation Department of Chemical and Biomolecular Engineering, Yonsei University, 50 Yonsei-ro, Seodaemun-gu, Seoul 03722, Korea; Korea; Seoul 03722; UMI Building Blocks for Future Electronics; Yonsei University

Correction needed for raw affiliation Department of Chemical and Biomolecular Engineering, Yonsei University, 50 Yonsei-ro, Seodaemun-gu, Seoul 03722, Korea; Korea; Seoul 03722; UMI Building Blocks for Future Electronics; Yonsei University
raw_affiliation_name: Department of Chemical and Biomolecular Engineering, Yonsei University, 50 Yonsei-ro, Seodaemun-gu, Seoul 03722, Korea; Korea; Seoul 03722; UMI Building Blocks for Future Electronics; Yonsei University
new_rors: 00pnt8b91;02sh10804;01wjejq96;049tcsg76;01w108f05
previous_rors: 00pnt8b91;02sh10804;01wjejq96;049tcsg76
works_examples: W2947217855
contact: b616bde23b53d1e46637cb7a71678dc5:09465efb5593f72f983e493d3b @ sorbonne-universite.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


handle invoke by command link

@v-bbrady is working on the welcome page and he needs the ability to open Extension details using command link, currently the command only accept string parameter which doesn't work with command link, command link expects all parameters to be inside an JSON object.
usage example to open agent extension page.
href="command:extension.open?%7B%22id%22%3A%22microsoft.agent%22%7D"

@anthonydresser do you prefer to create our own command and invoke the vscode one to minimize chance of the merge conflict ?

It will be more sustainable to just write our own command that calls this command.


Replace macOS default "delete" key to "backspace"

Is your feature request related to a problem? Please describe.
The default macOS keyboard (Magic Keyboard) doesn't have "delete" key.
I can understand that many people use third-party keyboards on Apple computers but if Ghidra wants to conform to mac application design it should be changed to backspace (even better, cmd+backspace). The backspace key is treated as the default "delete" key across macOS applications.
Currently, we can't assign anything to backspace key as it's treated as "remove key binding" key. Maybe just add button  to UI to clear key binding and allow users to assign any key they want to anything. The same goes for enter key, which we could map to "navigate forward".

The "delete" key should just be fn + delete.  I am able to bind to this.  It sounds like you are just asking to be able to bind the backspace key, regardless of platform.  Correct?

Yes, I'm familiar with fn - delete key but "unlocking" backspace to binding would (in my opinion) benefit Ghidra overall.


Attempted import error: 'hasOwnProperty' is not exported from './internal' (imported as '_').

🐛 Bug report
After upgrading at 2.11.0 I get this error:
Attempted import error: 'hasOwnProperty' is not exported from './internal' (imported as '_'). from the file ./node_modules/fp-ts/es6/struct.js

Software
Version(s)

fp-ts
2.11.0

TypeScript
4.3.5

Having the same issue. Looking into node_modules there is indeed not a single export in internal.d.ts

Should be fixed by https://github.com/gcanti/fp-ts/pull/1545


feat: Tag & Tag group components with filter and read-only variants

Summary

Tag component  with read-only and filtered variants.
Tag Group with show more / less option

ADO Story or GitHub Issue Link
https://dev.azure.com/Kyndryl/Shidoka - Bridge Design System/_workitems/edit/1581008
Resolves #78
Checklist

[x] Used Conventional Commit messages as outlined in the contributing guide.

[x] Noted breaking changes (if any).

[x] Documented/updated all props, events, slots, parts, etc with JSDoc.

[x] Ran the analyze command to update Storybook docs.

[x] Added/updated Stories with controls to cover all variants.
[x] Added/updated a basic unit test, at least, for each component.
[x] Added/updated the Figma link for the Story's Design tab.
[x] Added any new component exports to the src/index.ts file

The Docs page is only showing the ArgsTable for the Tag component. You'll need to manually create a Docs.mdx with ArgsTables for both components, or use the subcomponents approach.

The Docs page is only showing the ArgsTable for the Tag component. You'll need to manually create a Docs.mdx with ArgsTables for both components, or use the subcomponents approach.

Done. Added Docs.mdx

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing functions

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

Comparison is base (b412f69) 59.14% compared to head (8af7570) 59.31%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   59.14%   59.31%   +0.17%
==========================================
  Files          36       36
  Lines        2100     2104       +4
==========================================
+ Hits         1242     1248       +6
+ Misses        858      856       -2

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


feat: uitransform pointerFilter property

Added PointerFilterMode in UiTransform.

Related PRs:

https://github.com/decentraland/js-sdk-toolchain/pull/792
https://github.com/decentraland/unity-renderer/pull/5819

I'll merge after I get approval in the other 2 related PRs.


[FIX]: GitHub link in Contributing.md file is incorrect

This issue is regarding the incorrectly mentioned GitHub link in the Contributing.md file.
Screenshot:

Congratulations, @DurgaPrasad-R! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@DurgaPrasad-R thanks for addressing the error but its not something you will be getting a gssoc level for, its really a very small issue and perhaps don't require a pr to be raised.


Bug: Library does not return errors for receive

cashu-ts tries to receive multiple tokens from multiple mints at once and returns tokens with errors in receive. I think this is not a great approach because the implementor does not receive the error.
I am also not sure if the treatment of the counter for deterministic derivation still would work as intended. The counter should depend on the keyset which implies that a function using it should only create outputs of a single keyset, otherwise that function would need multiple counters.
I would suggest that the library should only receive tokens from a single mint with receive and multiple mints should be handled by an additional function. That means receive can return the error to the implementor. We should also check other functions that do not return errors and make them return it.
Any thoughts on this @gandlafbtc?
I'm c/p the relevant code here to make it easier to reason about:
	/**
	 * Receive an encoded or raw Cashu token
	 * @param {(string|Token)} token - Cashu token
	 * @param preference optional preference for splitting proofs into specific amounts
	 * @param counter? optionally set counter to derive secret deterministically. CashuWallet class must be initialized with seed phrase to take effect
	 * @param pubkey? optionally locks ecash to pubkey. Will not be deterministic, even if counter is set!
	 * @param privkey? will create a signature on the @param token secrets if set
	 * @returns New token with newly created proofs, token entries that had errors
	 */
	async receive(
		token: string | Token,
		options?: {
			preference?: Array<AmountPreference>;
			counter?: number;
			pubkey?: string;
			privkey?: string;
		}
	): Promise<ReceiveResponse> {
		let decodedToken: Array<TokenEntry>;
		if (typeof token === 'string') {
			decodedToken = cleanToken(getDecodedToken(token)).token;
		} else {
			decodedToken = token.token;
		}
		const tokenEntries: Array<TokenEntry> = [];
		const tokenEntriesWithError: Array<TokenEntry> = [];
		for (const tokenEntry of decodedToken) {
			if (!tokenEntry?.proofs?.length) {
				continue;
			}
			try {
				const { proofs, proofsWithError } = await this.receiveTokenEntry(tokenEntry, {
					preference: options?.preference,
					counter: options?.counter,
					pubkey: options?.pubkey,
					privkey: options?.privkey
				});
				if (proofsWithError?.length) {
					tokenEntriesWithError.push(tokenEntry);
					continue;
				}
				tokenEntries.push({ mint: tokenEntry.mint, proofs: [...proofs] });
			} catch (error) {
				console.error(error);
				tokenEntriesWithError.push(tokenEntry);
			}
		}
		return {
			token: { token: tokenEntries },
			tokensWithErrors: tokenEntriesWithError.length ? { token: tokenEntriesWithError } : undefined
		};
	}

@callebtc looking at the I don't think the receive function not returning errors for tokens that failed to be redeemed is a result/ consequence of the receive function being able to accept tokens from different mints and as such doesn't in itself hinder errors with failed tokens being returned. I'm not particularly sure if all errors would be useful but if we wanted to return errors here we could have them returned from the  receiveTokenEntry function being called in receive.
Secondly regarding the counter I think changing it's application might need fixes across multiple functions as it seems to be consistently used the same way across multiple functions in the CashuWallet class. So personally I think that might be a lot of work.
let me know your thoughts on this @callebtc @gandlafbtc

I think we should aim to simplify the datastructure of tokens to not allow multiple tokens inside a token.
Reasons:

less complexity and confusion
we can disregard this change, and return errors properly
if we must, we can achieve the same by chaining tokens and processing them sequentially

#86 has some additional discussion on this topic

@gandlafbtc as per the issue closed on #86 this issue can be closed out too, correct?

The api has been simplified and now either throws an error or returns the successfully swapped proofs. I think this can be closed @gandlafbtc


RedisKeyConf optional Key

because, error upgrading from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>

I think it's optional

Use RedisConf instead, if you don't need Key.

You should use redis.RedisConf by Config

Use RedisConf instead, if you don't need Key.

I have same trouble too, but the RedisKeyConf is in the part of RpcServerConf, how can I change it to RedisConf?

And you cannot use Redis as a name

I get it, thanks.


Object editor styling broken on optimized production build

Hello,
I've been using this library as a crux piece for a tool i've been working on.
When running in my dev electron environment everything works exactly as expected; however, upon packaging my application using react-scripts build to create a production build it seems the styling is completely broken.

If your project is publicly available, I'll try to repro directly - otherwise let's put together a minimal repro of this issue.

Hi @MadJayQ
I'm unable to repro this on the latest version, 1.1.0. Try downloading the latest version and see if there's still an issue


[IDEA] Adding social media links of the community

Description of the idea
It would be great if we add social media links of the community to the readme file so that people can find out easily
Add (multiple) screenshot links (if applicable)
No response
Add any other context about the idea here
No response
Code of Conduct

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

I would like to work on this

I would like to work on this

I would like to work on this

Assigning you this issue, you can use the image shields for the links :) https://shields.io/

I would like to work on this

Assigning you this issue, you can use the image shields for the links :) https://shields.io/

I would like to work on this

Assigning you this issue, you can use the image shields for the links :) https://shields.io/

I will use the same image links used in the web3community readme as shields don't have all the social media badges.

I would like to work on this

Assigning you this issue, you can use the image shields for the links :) https://shields.io/

I will use the same image links used in the web3community readme as shields don't have all the social media badges.


ngx-card does not work on angular 6.

Hi,
I have install in my angular 6 application. But its show Two errors.
Uncaught TypeError: Cannot read property 'Directive' of undefined
at Object. (ngx-card.umd.js:111)
at webpack_require (ngx-card.umd.js:30)
at Object. (ngx-card.umd.js:205)
at webpack_require (ngx-card.umd.js:30)
at Object.idCounter (ngx-card.umd.js:337)
at webpack_require (ngx-card.umd.js:30)
at module.exports (ngx-card.umd.js:76)
at ngx-card.umd.js:79
at webpackUniversalModuleDefinition (ngx-card.umd.js:9)
at ngx-card.umd.js:10
And second issue is:-
appComponent.html:13 ERROR ReferenceError: Card is not defined
at NgxCard.push../node_modules/ngx-card/card.js.NgxCard.ngAfterViewInit (card.js:41)
at callProviderLifecycles (core.js:9567)
at callElementProvidersLifecycles (core.js:9541)
at callLifecycleHooksChildrenFirst (core.js:9531)
at checkAndUpdateView (core.js:10467)
at callViewAction (core.js:10699)
at execComponentViewsAction (core.js:10641)
at checkAndUpdateView (core.js:10464)
at callViewAction (core.js:10699)
at execEmbeddedViewsAction (core.js:10662)

Same here.

GlobalErrorHandlerService.handleError:  ReferenceError: Card is not defined
    at NgxCard.push../node_modules/ngx-card/card.js.NgxCard.ngAfterViewInit (card.js:41)
    at callProviderLifecycles (core.js:18854)
    at callElementProvidersLifecycles (core.js:18828)
    at callLifecycleHooksChildrenFirst (core.js:18818)
    at checkAndUpdateView (core.js:19754)
    at callViewAction (core.js:19986)
    at execComponentViewsAction (core.js:19928)
    at checkAndUpdateView (core.js:19751)
    at callViewAction (core.js:19986)
    at execComponentViewsAction (core.js:19928)
    at checkAndUpdateView (core.js:19751)
    at callViewAction (core.js:19986)
    at execEmbeddedViewsAction (core.js:19949)
    at checkAndUpdateView (core.js:19746)
    at callViewAction (core.js:19986)
    at execComponentViewsAction (core.js:19928)

Any chance someone figured out how to fix this? Thanks

Any chance someone figured out how to fix this? Currently on Angular 8. Thanks

I added  in index.html
its works

I am on Angular 10. I keep getting the same error  as Card not defined
I am clueless in finding it out. Can anyone, please explain what's going wrong.
Example Stackblitz: https://stackblitz.com/edit/ngx-card-example-project

After making user card is installed via npm npm install --save card
In angular.json I added a reference to the node_modules script
....
"architect": {
        "build": {
                ...
                "options": {
                    ...
                    "scripts": [
                        "./node_modules/card/dist/card.js"
                     ]
....

this got rid of that error message for me

I need help with this error message.
This likely means that the library (ngx-card/ngx-card) which declares CardModule has not been processed correctly by ngcc, or is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

I am experiencing the same error too


Restore old DRS copy_batch API; separate copy_batch with manifest

This is "chained" on top of #369 in order to avoid merge conflicts. I will rebase on master once the other PR lands.

@DailyDreaming the problem is a little more complex than that, but I appreciate you pointing out the error. I'm working on it though, and I'll ping you when I'm ready for your review.


Move unit tests to new repo

Port all unit tests to @ngrx/router. Blocked by #1

Tests that need to be added:

[ ] component-renderer.ts
[ ] guard.ts
[ ] location.ts
[ ] match-pattern.ts
[ ] match-route.ts
[ ] middleware.ts
[ ] redirect.ts
[ ] route-params.ts
[ ] route-set.ts
[ ] route-view.ts
[ ] route.ts
[ ] util.ts


Validation messages for missing Structure Inventory Shapefile and required attributes in the first page of the Import Structure Inventory panel

When moving to the next page of the Import Structure Inventory panel, the following messages are expected when required files are missing. Currently, the priority of which fields to populate seems to be out of order. The following cases were done while filling out the required attributes with the First Floor Elevation selected.
Current behavior after clicking the "Next" button:

When the Structure Inventory Shapefile is missing, the user is informed that the First Floor Elevation Value is missing.

Expected: The user should be informed that the Structure Inventory Shapefile should be selected.

When the Structure Inventory Shapefile is missing and the Structure ID is selected, the user is informed that the First Floor Elevation Value is missing.

Expected: The user should be informed that the Occupancy Type should be selected.

When the Structure Inventory Shapefile is selected and the First Floor Elevation Value is selected, the user is informed that the Structure ID is then required.

The expected priority when informing the user that required files and attributes are missing should have a flow from top to bottom.

Structure Inventory Shapefile

a First Floor Elevation
2.a.i Structure ID
2.a.ii Occupancy Type
2.a.iii First Floor Elevation Value
2.a.iv Structure Value

2.b Ground Elevation and Foundation Height
2.b.i Ground Elevation Source - Terrain File
2.b.i.1 Structure ID
2.b.i.2 Occupancy Type
2.b.i.3 Foundation Height
2.b.i.4 Structure Value
2.b.ii Ground Elevation Source - Structures File
2.b.ii.1 Structure ID
2.b.ii.2 Occupancy Type
2.b.ii.3 Foundation Height
2.b.ii.4 Ground Elevation Value
2.b.ii.5 Structure Value

I did a total refactor of the validation system. I now display the total list of all the errors in the error message in order from top to bottom. I don't have a check for a missing file. If there isn't a file then the selections will be empty and that will be displayed to the user. This refactor was big enough that you should check all the validation stuff again.

Tested and verified fixed in HEC-FDA-2.0-Snapshot 12_22_21


fix path gc should consider tablet migration

Proposed changes
Issue Number: close #xxx
Background:

Migration will create new tablet in different DataDir, the old tablet will be moved to TabletManager::_shutdown_tablets.
The migration task won't copy data in stale rowsets to new tablet, so after migration, the new tablet don't contains stale rowsets of old tablet
The path GC process will check every path, to make sure if it's an useless tablet, or an useless rowset. If it is, will remove data of these tablets/rowsets

The issue:

When path GC got a stale rowset path from the data dir of old tablet, it extract the tablet id and rowset id
Then it check if the tablet id exists in TabletManager, and the answer is YES!
It got the tablet instance, which is the new tablet, then it check if the stale rowset id from the old tablet path exists in the new tablet instance, and got the answer NO.
The path GC process treat the rowset as an useless rowset, since it can't find anyone holds reference to it, then delete  the data of this stale rowset.
But some query may still holds reference to this stale rowset, the deletion will cause query failure.

Solution:

The lifecycle of all rowsets in a shutdown tablet, should be related with the lifecycle of this tablet
We need to differentiate the old tablet and the new one created by migration task, while performing path GC.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Phonons

Adapt the phonopy workflow from VASP for FHI-aims

All things fixed and added seekpath requirement for one the phonon steps

Ready to merge?


There are no macros in sys::stat to determine file type from st_mode

Can't find any analogues to these macros:
S_ISREG()
S_ISDIR()
S_ISCHR()
S_ISBLK()
S_ISFIFO()
S_ISLNK()
S_ISSOCK()

This is not particularly clear in the documentation, but S_IFMT is the thing to bitwise AND with to get only the bits that specify the file type.  Those macros that are missing from the Rust wrapper are defined like:
    #define 	S_ISBLK(m)   (((m) & S_IFMT) == S_IFBLK)


'Run Code Inspector' dumps on offline repositories

Abapgit version 1.72
An invalid cast between ZCL_ABAPGIT_REPO_OFFLINE and ZCL_ABAPGIT_REPO_ONLINE occurs.

Full dump here:
MOVE_CAST_ERROR_20180723_101807.zip

@FreHu can you try again with latest build?


Refactor!: normalize environment names according to the engine's dialect

We previously treated environment names as case-insensitive, lowercasing them regardless of the target engine.
This was problematic for engines like Snowflake, because it essentially forced users to use quotes in queries that referenced tables within these environments. So, for example instead of FOO_DEV, we'd create FOO_dev.
This PR aims to address this issue by normalizing the environment names according to the rules of the targeted engine's dialect. So, for the above example users will now be able to query FOO_DEV.table without using quotes for the schema part.
This PR should not affect existing environments, which is achieved by setting their normalize_name column to FALSE. Hence that column will only be TRUE for new environments.

I plan to wrap this up soon. Remaining items on my end are:

[ ] Normalize default schema name (relevant discussion)
[ ] Add tests

This is ready for another review. I added a simple test that demonstrates how the default schema & environment name get normalized, but I haven't tested that old environment names won't be normalized (though I did test against snowflake and it seems to work fine). Any suggestions on how we should do this, if needed?


Added service container for Selenium standalone with Chrome

Resolves #356
Added Selenium using the official Selenium docker images from https://github.com/SeleniumHQ/docker-selenium
The Selenium server is served on port 4444

Can you update the README file?

@appleboy Updated, thanks

@edmundluong thanks for your contribution!

@edmundluong thanks for your contribution!

@edmundluong Hello. Do you use it for codeception?
I am trying to configure it for use acceptance tests. It requires selenium web driver or phantomJS.
But the problem is website url which unavailable inside selenium container.

Hi, im trying to integrate browser testing with dusk and selenium, when I run php artisan dusk , all my test are passing but I cant see the browser being loaded or open?
Steps: (scripts below are excuted inside workspace - docker-compose exec workspace bash)
1-docker-compose up -d apache2 selenium
2-added/setup dusk (php artisan dusk:install)
3-created browser test
4-php artisan dusk
->My sample regression does succeed but I can't see the browser being tested.
--visited the selenium url: http://localhost:4444/wd/hub
--I can only see a session menu
What am I missing here? Thanks.


feat: state sync on epoch change (see #154)

Description
At each epoch change, current validator node must synchronize state from VNs in the same committee, with missing state.
Motivation and Context
Address issue #154.
How Has This Been Tested?

It also addresses issue #180.

utAck
Not sure if it works but we can fix up any issues as we find them

On second thoughts, this needs to happen once scanning is complete - currently, it will try sync for each epoch while the VN is processing the blockchain.
I suggest we change the epoch manager to only call update_epoch on the final block height that the scanner scans

ACK
Tested and the node starts and syncs the base layer - thanks :rocket:


apps in group is not opening

apps in group is not opening

apps in group is not opening

pretty much this , hope we will get an answer

I have 15 groups with anywhere from 2 to 18 apps in them, and every single one works.
Probably need more information than just "they're not opening" to be able to figure out what is going on.


Render the area of resource

Render the area of resource in header section when the detail of
resource is rendered.

@tomijarvi I had a small chat with @juhakujala and I decided to have square with m2 as icon.


Fix typo in deprecation-guide.md at v1.16

This makes the wording consistent with the rest of the file.

@manueljacob Thanks you for your contribution.
Could you please sign the CLA before the PR can be reviewed.
You can follow the steps documented here: CLA Steps

/easycla

Hi @manueljacob Could you please address the suggestion made by dipesh for CLA and sign it.
Also is there any other change you want to make in this page.
As per our contribution guide it is recommended to try finding more improvements in the page.

/retest

/close
I am closing this PR because the author, @manueljacob, hasn't signed the CLA or responded to feedback for over 2 weeks.

@manueljacob  , I'm closing this PR since the CLA has not been signed.
When you have signed the CLA, you can reopen this PR with /reopen in a comment
/close

@manueljacob , I'm closing this PR since the CLA has not been signed. When you have signed the CLA, you can reopen this PR with /reopen in a comment /close

Thanks for looking into this.


ban bypass

is there any way i can bypass a ban on tf2? how can i join the community server again?

no

alt

alt

my ip is banned, alts wont work. would appriciate help so i can hack on skial again

no

thanks for a very helpful answer

alt

my ip is banned, alts wont work. would appriciate help so i can hack on skial again

vpn

You can't

ik this is locked but protip for everyone, dont cheat on tf2 community servers.


[R][Docs] Version list is not updated

Describe the bug, including details regarding any error messages, version, and platform.
https://arrow.apache.org/docs/r/versions.html does not seem updated.
It shows:

<IP_ADDRESS>00 (dev)
<IP_ADDRESS> (release)
<IP_ADDRESS>
10.0.1
9.0.0
8.0.0
7.0.0
6.0.1
5.0.0
4.0.1
3.0.0
2.0.0
1.0.1

But the release version is 13.0.0 now.
Component(s)
Documentation, R

i want to work on this @eitsupi

@Divyansh200102 Glad to hear that.
I think you can assign this issue for you via commenting "take" on this issue.
https://github.com/apache/arrow/blob/c5bce96ba626ad2558e10acddf8ef19ea06940e8/CONTRIBUTING.md?plain=1#L22-L28

take

in which file exactly do you want me to make the changes?? @eitsupi

take

i think the website you just mentioned  mentions the older version of the docs for the users. @eitsupi
r_pkgdown.yml:
352    components:
353:     older_versions: "Older versions of these docs"
354

Thanks for repoting this @eitsupi!
No need to do anything here, @Divyansh200102  - there is currently a PR open to fix this on the arrow-site repo: https://github.com/apache/arrow-site/pull/413
The version list is updated via a script which is run during the release; we should check if it ends up being run successfully or not during the release, and if not, look into that.


Doesn't seem to build on travis from the metadeps switch.

See https://travis-ci.org/servo/rust-azure/builds/192097103 (https://github.com/servo/rust-azure/pull/253).
Concretely, the error is "Requested 'x11 >= 1.6' but version of X11 is <IP_ADDRESS>". This used to work, so I guess it's safe to downgrade the minimum required version?

That seemed to help, thanks! :)


Targets of the wrong type are immune to Synchronoise

PS! currently displays "But it failed!" in this case, which is wrong according to a YouTube video I watched recently (sorry I've mislaid it but I'll comment if I find it again.)

Ah, right, you need to know whether the move can miss. (onImmunity is only for status immunity).

Meritt kindly researched for me that it can only miss semi-invulnerable targets, otherwise targets of the wrong type are simply immune as if they were Dark type.
Unfortunately the sim as currently implemented doesn't let the move itself check for immunity between the semi-invulnerable check and the accuracy check; the TryHit event that Abilities use does happen in between, but the TryHit that the move itself sees happens after accuracy. On the other hand, the problem with the Try event is that it happens before the semi-invulnerable check. Currently my best idea is to disable the accuracy check for immune targets in the Try event, so that the semi-invulnerable check will process as normal before the TryHit event flags up the immunity.


analytics: respect env var to disable analytics, allow us to send metrics in the same session where someone opts in

Hello @landism, @nicks,
Please review the following commits I made in branch maiamcc/analytics-clean-up:
ae94c7367b90cb2de095b0097f41ab5295e7b5be (2019-05-10 17:31:40 -0400)
analytics: respect env var to disable analytics, allow us to send metrics in the same session where someone opts in
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

Unit test?

are there particular parts you're concerned about? this code seems like a lot more trouble to test than it's worth b/c it involves spoofing out user input and files in specific directories...

@nicks bump -- if I refactor some stuff for testability i want to make sure i hit the cases you're concerned about!

...but also, are tests on this code a good investment atm? this codepath is gonna change soon, but also the way we invoke it in CI means that that env var will always be set, so that's not a super useful thing to test...?

hmmmm...this bug seems likely to regress, because changing the order of calls can re-introduce it.

What bug?

if the codepath is going to change soon, that seems like an even stronger reason to invest in tests now? if you're not sure how to test it effectively, let's talk about that! I would probably factor out a struct with the os streams as properties

Nick and I talked, if this codepath is gonna change soon it doesn't seem worth it to make this change, closing this PR.


Add favicon, .webmanifest, OG and meta-tags to new site

Favicon

[ ] Configure with 2022 recommendations

[ ] Put svg icon to /client/assets
[ ] Add Vite favicon plugin

[ ] Check webmanifest (Lighthouse PWA)

SEO, Social

[ ] Open-graph description
[ ] Open-graph image — 1200x630px
[ ] Meta-tag description

I found favicon plugins not very useful. The guidance is very easy to do manually once. It is faster than dealing with plugin config.
Adding plugin for this one-time task will only increase node_modules and slow down build time.


New-DbaDatabase: Fix issues with generated filenames

Type of Change

[x] Bug fix (non-breaking change, fixes #7981 )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[ ] Documentation
[ ] Build system

Fixes for the linked issue.
But this will also effect the filenames when used without advanced parameters, as now the "special" filenames are created always:

I'm sorry, I'm confused. Does this deviate from Microsoft's default naming standard? Or does it better align with it?

Let me share two screenshot.
This one is done with the current version of dbatools:

Database "tsql" is for reference. Database dba1 does not use any "advanced" parameter, database "dba2" uses an advanced parameter to run the advanded section of the code. Here you see the different names for both logical and physical filenames.
This one is from the current branch in this pull request:

Same commands, same output for the first two. But the last one now follows the same rules like in "tsql".

hell yeah! thank you 💯 💯 💯


Killing a mob with no loots throws an error

TypeError: Types[mob.mob].loots is not iterable
    at EventEmitter.<anonymous> (file:///Users/sceat/dev/aresrpg/aresrpg/src/mobs/loot.js:14:55)
    at EventEmitter.emit (node:events:394:28)
    at file:///Users/sceat/dev/aresrpg/aresrpg/src/mobs/damage.js:61:22
    at HAsyncIterator.reduce (/Users/sceat/dev/aresrpg/aresrpg/node_modules/iterator-helper/dist/async-iterator/HAsyncIterator.js:208:25)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

like killing a npc #339

see #369


react dependency consistent with peerDependencies

If you're not using react ^0.14.0, current setup causes npm to require react twice. Example:

@abpai Can you fix this build.
react-dom still requires react version 0.14 at this moment which causes a conflict.

Closing in favour of https://github.com/Sitebase/react-avatar/pull/44


make some presets

hey!
really like the program, but could you please add some presets for the triggers...
a side note, a doc on what each option would also be very helpful..

i'll try making some presets, but documentation will be hard, I would have to just test inputs and see what happens.

ok.. Thanks!


Update duet to <IP_ADDRESS>

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Thank you for the contribution. It was merged directly as 9896cecd6aff15f95a115b7e917ddbd532865010. Your contribution is still credited to you.


Magic NPC's

Is there a way to make NPC's from the citizens plugin cast spells to attack, like using the sentinel trait but with magic instead of normal weapons?

No, I'm afraid not. I can keep this open as a feature request if you like.
Your best bet in the meantime is to use Magic Mobs (see wiki) or MythicMobs, though the last time I tried that their command skill didn't seem to work anymore, so maybe not.

Yes that'd be good thank you :)

Sentinel NPCs can cast spells in the current dev build, slated to be released as 7.5.3
.. Didn't quite make it in under a year, but better late then never, amirite?


Unsupported format version: 0.0

I get .Unsupported format version: 0.0 when trying to run sourcelink test on my assemblies. I've tried using the tool, and the msbuild task (which seems to just be running the tool), and regardless I get the same output. I've made sure that the debug type is set to embedded, and sourcelink outputs information about my repository during build:
       SourceLinkCreate:
         git rev-parse --show-toplevel
         /Users/alxandr/hub/YoloDev.Expecto.TestSdk
         git config --get remote.origin.url
         git@github.com:YoloDev/YoloDev.Expecto.TestSdk.git
         git rev-parse HEAD
         e1d4ae52a6696302de7ccf448b9a578ddee1c1fa

So at this point I'm unsure about how to debug anymore. I get zero stack-trace, just this one line telling me that the format of something or another is wrong.

You could try using portable pdb files just to see if that works. Run sourcelink test on the pdb files.
You could run dotnet-sourcelink Program from Visual Studio and step through it. It is a pretty simple code base.

You may wish to try updating the System.Reflection.Metadata version used. And you may end up setting a breakpoint in DebugReaderProvider where the error is probably happening.

portable seems to work. Using dotnet cli on OSX and F#.

Now it works on my mac, it works on travis (it seems), but it fails on AppVeyor. https://ci.appveyor.com/project/YoloDev/yolodev-expecto-testsdk
Would love some help trying to figure this out.

@Alxandr, it looks like you've go this sorted. I also released 2.6.0 today that may help.

Well, no. I had to not use portable, so I would still consider that as a bug. That being said, I can check if 2.6.0 solves it.

dotnet sourcelink test will only work on portable pdb files or assemblies with embedded pdb files.

Ah, right. I remember now. It was the embedded ones that didn't work. I'll try to test if it's been fixed over the weekend.