Type Refinement is not working with constants

Tested with flow.org/try: Example
/* @flow */

// Type Refinement is not working with constants:

type TYPE_ONE = 'ONE';
type TYPE_TWO = 'TWO';

const ONE: TYPE_ONE = 'ONE';
const TWO: TYPE_TWO = 'TWO';

const takesOne = (param: TYPE_ONE) => {};

// Does not work when comparing against a constant
const oneOrTwo: TYPE_ONE | TYPE_TWO = ONE;
if (oneOrTwo === ONE) {
  takesOne(oneOrTwo);
}

// Does work when comparing against a literal
if (oneOrTwo === 'ONE') {
  takesOne(oneOrTwo);
}

// Checking against a constant does work when it is in an object
const obj: {key: TYPE_ONE} | {key: TYPE_TWO} = { key: ONE };
if (obj.key === ONE) {
  takesOne(obj.key);
}
// note: {key: TYPE_ONE | TYPE_TWO} does not work

16:   takesOne(oneOrTwo);
               ^ Cannot call `takesOne` with `oneOrTwo` bound to `param` because string literal `TWO` [1] is incompatible with string literal `ONE` [2].
References:
14: const oneOrTwo: TYPE_ONE | TYPE_TWO = ONE;
                               ^ [1]
11: const takesOne = (param: TYPE_ONE) => {};
                             ^ [2]

Duplicate of #6968?

Yes, looks like this is a duplicate.


Enlarged the viewbox and background rectangle of many icons; Fix for #43

... so that the icons would fit nicely inside the background if rx was set to 50%.
Some went to 640 instead of just 600 so that they look even less cramped, especially when the icons are small.

The biggest reason I shrunk the others is because when you shrink the icons (48x48 for example), you may not see the border, or it may only be 1px or so at that size.  Hackernews and StackOverflow are sitting right against the edge. You seem to be ok with them though, so I'll undo the changes to those icons.


Module 3 exercises [Goemeone L. Sewetse]

Please "drag & drop" ex
Land Scape Condition.pdf
ercise 1 pdf file below.

Please "drag & drop" exercise 2 pdf file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 3 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Well done, Goemeone. You clearly grasped the material outlined in Module 3 and successfully applied it to this exercise. Thank you for your participation!


Fix Windows

Fixes #205.
This change appends the file:// protocol to windows paths to the error below
(technically we let pathToFileURL from node:url do the work)
ESBuild: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader.
On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

@yuanqing this is ready (no more commits expected)

I'm getting errors loading CSS Modules but I can't tell if that's related to these changes or it's been broken on Windows.
I'm also working with a forked copy of the UI library so that might be it. Would be helpful for others to verify.
Should be safe to merge in though for people to test, as currently Windows doesn't work at all right now.
ERROR: [plugin: css-modules] CSS file not found: !../css/base.css

I need this, please

@yuanqing Can we please get this PR merged? It's a major blocker in upgrading to the newest version for windows users


File could not be found.

I'm using pipeline.
I tried, explicitly over-riding the font file path. But no luck.
ValueError: The file 'twitter_bootstrap/fonts/glyphicons-halflings-regular.woff2' could not be found with <pipeline.storage.PipelineCachedStorage object at 0xa8e48ec>.

does that file appear in the installed location of dj-tw-bs?

No, I created an empty file with the same name and extension. Error seems
to go away.
Regards,
Saad Bin Shahid | Software Engineer | www.saadbinshahid.com |
On Fri, Apr 3, 2015 at 5:09 AM, Steven Cummings<EMAIL_ADDRESS>wrote:

does that file appear in the installed location of dj-tw-bs?
—
Reply to this email directly or view it on GitHub
https://github.com/estebistec/django-twitter-bootstrap/issues/21#issuecomment-89087782
.

Yes, but surely you don't get the desired icons? That doesn't sound right at all. I need to go check the packaging...

No, I did not get my desired icons. please look into it. Thanks.!
Regards,
Saad Bin Shahid | Software Engineer | www.saadbinshahid.com |
On Sat, Apr 4, 2015 at 6:21 AM, Steven Cummings<EMAIL_ADDRESS>wrote:

Yes, but surely you don't get the desired icons? That doesn't sound right
at all. I need to go check the packaging...
—
Reply to this email directly or view it on GitHub
https://github.com/estebistec/django-twitter-bootstrap/issues/21#issuecomment-89479068
.

Oh, you have woff2 as the file extension. It's just woff.

I know that :) I checked. There is an .woff2 empty file in the repo
@ bootstrap/fonts/glyphicons-halflings-regular.woff2
That's why I added a empty file. The file is also being used in the CSS and
some other places. Hence raising the issue when the PIPELINE try to
pre-process it.
Regards,
Saad Bin Shahid | Software Engineer | www.saadbinshahid.com |
On Sun, Apr 5, 2015 at 3:16 AM, Steven Cummings<EMAIL_ADDRESS>wrote:

Oh, you have woff2 as the file extension. It's just woff.
—
Reply to this email directly or view it on GitHub
https://github.com/estebistec/django-twitter-bootstrap/issues/21#issuecomment-89672323
.

Okay. I see. Looking at the currently supported version of bootstrap: https://github.com/twbs/bootstrap/tree/16dbdbd7a2c6cfa3be4e5dcc52249e577c02c84a/fonts, there is no woff2. It's been added in a recent version. So really this isn't a bug, I just need to release new wrapper packages for latest bootstrap (which will include woff2).
However, I've been meaning to give this repo and package the correct name (see #13). So rather than just release the next versions (one of which would include the woff2), I'd like to close this issue and complete #13 and release subsequent versions.

Great.! Thanks.
Regards,
Saad Bin Shahid | Software Engineer | www.saadbinshahid.com |
On Sun, Apr 5, 2015 at 5:41 AM, Steven Cummings<EMAIL_ADDRESS>wrote:

Okay. I see. Looking at the currently supported version of bootstrap:
https://github.com/twbs/bootstrap/tree/16dbdbd7a2c6cfa3be4e5dcc52249e577c02c84a/fonts,
there is no woff2. It's been added in a recent version. So really this
isn't a bug, I just need to release new wrapper packages for latest
bootstrap (which will include woff2).
However, I've been meaning to give this repo and package the correct name
(see #13
https://github.com/estebistec/django-twitter-bootstrap/issues/13). So
rather than just release the next versions (one of which would include the
woff2), I'd like to close this issue and complete #13
https://github.com/estebistec/django-twitter-bootstrap/issues/13 and
release subsequent versions.
—
Reply to this email directly or view it on GitHub
https://github.com/estebistec/django-twitter-bootstrap/issues/21#issuecomment-89694757
.


Additional Subject Data

Increase # of subjects to compare low/medium/high gender uptake

Scraping text off the page is proving to be somewhat difficult (multiple text tags, no consistency in tag name/format/anything really).
In order to have some data, I've taken a semi-manual approach to get some data together whilst continuing to work the problem, so far I've got 30 schools worth of Geography, CompSci and (additional) Drama text from Options pages/booklets.
This took ~1.5 hours, by trying to hit '/curriculum/' pages of school websites, I could be reasonably confident that the website may have a straightforward way of accessing curriculum information.
Note: trying to hit 'y9-options'/'y8-options' may be a good idea, rather than 'curriculum, TBC


warning: unexpected cfg condition name: rust_analyzer

hello
cargo check on latest nightly:
warning: unexpected `cfg` condition name: `rust_analyzer`
   --> game/src/card/components/mod.rs:29:45
    |
29  |               #[derive(Default, Debug, Clone, bon::Builder, Reflect, Serialize, Deserialize)]
    |                                               ^^^^^^^^^^^^
...
101 | / define_components! {
102 | |     struct CardHealth { pub current: u32, pub max: u32 },
103 | |     struct TroopHealth { pub current: u32, pub max: u32 },
104 | |     struct Cost(pub u32),
...   |
114 | |     struct TileExpansion {},
115 | | }
    | |_- in this macro invocation
    |
    = help: expected names are: `clippy`, `debug_assertions`, `doc`, `docsrs`, `doctest`, `feature`, `fmt_debug`, `miri`, `overflow_checks`, `panic`, `proc_macro`, `relocation_model`, `rustfmt`, `sanitize`, `sanitizer_cfi_generalize_pointers`, `sanitizer_cfi_normalize_integers`, `target_abi`, `target_arch`, `target_endian`, `target_env`, `target_family`, `target_feature`, `target_has_atomic`, `target_has_atomic_equal_alignment`, `target_has_atomic_load_store`, `target_os`, `target_pointer_width`, `target_thread_local`, `target_vendor`, `test`, `ub_checks`, `unix`, and `windows`
    = help: consider using a Cargo feature instead
    = help: or consider adding in `Cargo.toml` the `check-cfg` lint config for the lint:
             [lints.rust]
             unexpected_cfgs = { level = "warn", check-cfg = ['cfg(rust_analyzer)'] }
    = help: or consider adding `println!("cargo::rustc-check-cfg=cfg(rust_analyzer)");` to the top of the `build.rs`
    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
    = note: `#[warn(unexpected_cfgs)]` on by default
    = note: this warning originates in the derive macro `bon::Builder` which comes from the expansion of the macro `define_components` (in Nightly builds, run with -Z macro-backtrace for more info)

i think its the flag unexpected_cfgs which is now enable by default
A note for the community from the maintainers
Please vote on this issue by adding a 👍 reaction to help the maintainers with prioritizing it. You may add a comment describing your real use case related to this issue for us to better understand the problem domain.

Thank you for reporting this! The fix for this was released in a new patch 3.0.2


[Feature] Support for PMD Copy/Paste Detection

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I documented the Java code using JavaDoc style.

Motivation and Context

Clones are a threat to the maintainability of software systems. Pieces of code which are used often should be encapsulated and made available to other components by providing well documented interfaces (Don't repeat yourself, Single responsibility principle). With this PR, code clones can be automatically detected and punished by configuring the grading for programming exercises.
Description

Add PMD CPD tool, which automatically extends the build plan if SCA is enabled
Add Copy Paste detection category
Add default configuration for PMD CPD. Only duplication with a minimum size of 70 tokens are reported.

Steps for Testing

Test on TS1 and TS2

Create a programming exercise with SCA enabled
Participate
Submit code with contains duplications (must be at least 70 tokens long, this can be changed in the pom, the BubbleSort algorithm should suffice)
--> Check that the code duplication is detected

Test Coverage

Nothing changed
Screenshots

I added the same method for all 3 classes and got the following issues:

I would expect just to get the 3 issues with the 3 files and not additionally 2 issues with BubbleSort and `Client. Is this intended @kloessst ?

I guess it finds one more duplicate token/line as in the other duplication and creates another issue for this. As the copy paste detection is a blackbox for us, we can't change this.
The reporting of multiple feedback elements for one unique duplication is wanted and approved by @krusche. This allows us to highlight the locations in the code editor and to punish multiple clones of the same code harder.
I will set this PR to the draft state, until it can be tested on both TS1 and TS2.

Is there any reason for the changes in c7b25bc?

Is there any reason for the changes in c7b25bc?

I did some testing, playing around with the options. If you activate ignoreIdentifiers, a lot of tokens e.g. variable names, method names, won't be counted any longer. In turn, the minimumTokens must be reduced significantly, otherwise no duplications will be found. Lowering the minimumTokens too much, increases the risk of false positives.
I tested this with the MergeSort implementation of the template and using ignoreIdentifiers with 70 minimumTokens, copying the whole MergeSort implementation won't be detected. Reducing minimumTokens to 30 detects a if block duplication within the implementation which is a false positive. For 50 minimumTokens nothing is detected.
I try to find a good default configuration but in the end, each instructor is able to play around with these settings as well. Let me know if you found a better solution. Then we can use this as the default.


Improve the selected role displaying view under manually adding multiple users

Describe the issue:

Enable the "invite user to set the password" in carbon.super tenant (need to do this via mgt console in the beta pack. Since the config is not available in new UI)
Login to console. Go to User Management -> Users -> Select Multiple user option
Select few roles

Once a role is selected it is shown as <role name>:<role audience type> This is not a clear representation.

Instead, it would be better to show just the role name in the input field and show other information on a card. When the role audience is an application, the application name is also required to be shown.
Example for showing more details on a card:

Assign users to a role. The user's username is displayed. userstore domain kind of information is available on the card info

https://github.com/wso2/product-is/assets/25483865/9ad0d9bf-0544-48d8-8ec5-585bf6644b27

Suggested Design 01
Discussion in UX Thread: https://chat.google.com/room/AAAALlrrsT0/tZqzeX2oSRo/

Added the Role Name and Role Audience as the title and subtitle of the menu item.
Updated the chip to display Role Name, Role Audience, and App/Org name.

Final Design
Meeting Notes: https://docs.google.com/document/d/1Q5DozDfve5zN7daqnfoISy5TSsh3zs_Io3Taw6i0HI4/edit#heading=h.2b7edbuglwxm

The above roles dropdown will be replaced with the groups down and hence the above roles dropdown changes are not needed. The Roles chip changes are addressed with https://github.com/wso2/identity-apps/pull/4937

Verified in the new pack


User primary key creation - Onboarding vs Login

Build the Onboarding and Login flow for a user to access keys.
The main flow in the user agent to create a primary key during onboarding is as follows:

Vue.js obtains a secret
Vue.js derives a primary key from secret (via hkdf.NewMasterLock()). Note: Primary key is never stored and is not sharable in DID documents.
Vue.js splits the secret into shares.
Vue.js creates a new key store sending one share along with the Create KeyStore rest call to the Key Server.
Vue.js prepares the storage of the second share in SDS for Hub-Auth:

Vue.js instantiates framework with secretLock service using the primary key (and should remove primary key from memory)
Vue.js creates “SDS Vault” keys via the framework
Vue.js creates an SDS Vault via the framework and sets one of the “SDS Vault” keys as the controller
Vue.js creates via the framework an SDS encrypted document wrapping one secret share and setting the recipient to be... the “SDS Vault” key - this encrypted document is the hub-auth share

Vue.js stores the SDS doc ref in hub-auth of the document created in step 5 above.

The issue with this flow is during login, the user agent cannot fetch the primary key half stored in SDS before accessing the keys in the KMS. In order to access the KMS, the user needs to build the primary which depends on the key half in SDS.
We need to provide a way to circumvent this circular root problem.

@Baha-sk

the secret shares are to be stored in SDS vaults by the user agent directly
if the primary key is never persisted anywhere, and the "SDS Vault" keys are encrypted with it, and these "SDS Vault" keys control the SDS vaults, and in these same vaults we're storing the secret's shares, then we have the chicken+egg problem.... how can we unlock the "SDS Vault" keys without having the "secret" on hand.... ?

@Baha-sk

The issue with this flow is during login, the user agent cannot fetch the primary key half stored in SDS before accessing the keys in the KMS. In order to access the KMS, the user needs to build the primary which depends on the key half in SDS.
We need to provide a way to circumvent this circular root problem.

edge-agent has an access_token from hub-auth
edge-agent has a primary key
edge-agent splits primary key into shares
edge-agent asks the key server for an auth key KID (authZ with access_token)
edge-agent creates 1 SDS vault per key share and set the vault's controller to the primary key (did:key) (authZ with access_token)
edge-agent creates 1 document per key share at the SDS vaults and includes the key server's key KID as a recipient (authZ with primary key)
edge-agent updates bootstrap info with key locations at hub-auth (authZ with access_token)
edge-agent purges the primary key from memory
edge-agent inits remote KMS with the key share locations
key server fetches key shares from SDS vaults

New requirements from Troy:
Points I captured from @troyronda :
separation of user data from key server
Key Server NEVER has access to user data - not even encrypted
user device performes encryption ops for user-sensitive data
in all respects: separate user data storage, and key server storage + ops
purvue of Key Server is to handle authentication and key exchange ops
  * implies: Key Server must not ever have user data touching it in ANY form
  * implies: Key server cannot have any access to keys that were used to encrypt user data
only user device EVER touches user data in encrypted or decrypted form
only the user's SDS vault can touch the user data in decrypted mode
user must have control of their SDS vault
bootstrapping component (http-server) only system that can ever see keys related to user data, NOT the user data itself
when user is not currently logged in, none of their keys exist in full-form in any system
when user logs in the keys are reconstructed for a certain amount of time
hub-auth cannot have the full key
  * the only system that can have the key is the user's device and a service that may assist the user's device when bootstrapping
keys split between services that do auth, and user-controlled storage

This has been implemented


CREATE TABLE cannot contain single quotes

Describe the bug
CREATE TABLE cannot contain single quotes
For example,
use
CCJSqlParser parser = CCJSqlParserUtil.newParser(sql);
CreateTable ct = parser.CreateTable();

to parse SQL
CREATE TABLE 'sys_user_info' (
  'user_id' int(11) NOT NULL AUTO_INCREMENT COMMENT '用户编号',
  'user_name' varchar(255) NOT NULL COMMENT '用户名',
  'status' tinyint(1) NOT NULL COMMENT '状态',
  'create_time' datetime NOT NULL COMMENT '创建时间',
  PRIMARY KEY ('user_id')
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COMMENT='用户信息';

an exception will be thrown,The details are as follows:
net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "\'sys_user_info\'" <S_CHAR_LITERAL>
    at line 1, column 14.

Was expecting one of:

    "ACTION"
    "ANY"
    "BYTE"
    "CASCADE"
    "CAST"
    "CHANGE"
    "CHAR"
    "CHARACTER"
    "COLUMN"
    "COLUMNS"
    "COMMENT"
    "COMMIT"
    "CREATE"
    "CYCLE"
    "DESC"
    "DESCRIBE"
    "DISABLE"
    "DIV"
    "DO"
    "DOUBLE"
    "DUPLICATE"
    "ENABLE"
    "END"
    "EXCLUDE"
    "EXTRACT"
    "FALSE"
    "FIRST"
    "FN"
    "FOLLOWING"
    "FORMAT"
    "GROUP"
    "IF"
    "INDEX"
    "INSERT"
    "INTERVAL"
    "ISNULL"
    "KEY"
    "LAST"
    "LEFT"
    "LIMIT"
    "MATERIALIZED"
    "NEXTVAL"
    "NO"
    "NOLOCK"
    "NULLS"
    "OF"
    "OFFSET"
    "ON"
    "OPEN"
    "OPTIMIZE"
    "ORDER"
    "OVER"
    "PARTITION"
    "PATH"
    "PERCENT"
    "PRECISION"
    "PRIMARY"
    "PRIOR"
    "PROCEDURE"
    "PUBLIC"
    "RANGE"
    "READ"
    "REPLACE"
    "RIGHT"
    "ROW"
    "ROWS"
    "SCHEMA"
    "SEPARATOR"
    "SEQUENCE"
    "SESSION"
    "SET"
    "SIBLINGS"
    "SIZE"
    "START"
    "TABLE"
    "TABLES"
    "TEMP"
    "TEMPORARY"
    "TO"
    "TOP"
    "TRUE"
    "TRUNCATE"
    "TYPE"
    "UNSIGNED"
    "VALIDATE"
    "VALUE"
    "VALUES"
    "VIEW"
    "XML"
    "ZONE"
    <K_DATETIMELITERAL>
    <K_DATE_LITERAL>
    <S_IDENTIFIER>
    <S_QUOTED_IDENTIFIER>

Change the SQL to
CREATE TABLE sys_user_info (
  user_id int(11) NOT NULL AUTO_INCREMENT COMMENT '用户编号',
  user_name varchar(255) NOT NULL COMMENT '用户名',
  status tinyint(1) NOT NULL COMMENT '状态',
  create_time datetime NOT NULL COMMENT '创建时间',
  PRIMARY KEY (user_id)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COMMENT='用户信息'

will work fine
To Reproduce
test code
    @Test
    public void parseCreateSql() throws JSQLParserException, ParseException, TemplateException, IOException {
        String sql = "CREATE TABLE 'sys_user_info' (\n" +
                "  'user_id' int(11) NOT NULL AUTO_INCREMENT COMMENT '用户编号',\n" +
                "  'user_name' varchar(255) NOT NULL COMMENT '用户名',\n" +
                "  'status' tinyint(1) NOT NULL COMMENT '状态',\n" +
                "  'create_time' datetime NOT NULL COMMENT '创建时间',\n" +
                "  PRIMARY KEY ('user_id')\n" +
                ") ENGINE=InnoDB DEFAULT CHARSET=utf8 COMMENT='用户信息';";
        CCJSqlParser parser = CCJSqlParserUtil.newParser(sql);
        CreateTable ct = parser.CreateTable();
    }

maven dependency
        <dependency>
            <groupId>com.github.jsqlparser</groupId>
            <artifactId>jsqlparser</artifactId>
            <version>4.0</version>
        </dependency>
        <dependency>
            <groupId>junit</groupId>
            <artifactId>junit</artifactId>
            <version>4.8.2</version>
            <scope>test</scope>
        </dependency>

Expected behavior
Whether you can implement support for single quotes in the statement that creates the table.
System

JSqlParser version 4.0

Greetings!
While I am no developer but just a user of JSQLParser, I wonder whether we should implement support for any exotic RDBMS slang or focus on something like latest SQL standard + support on the "big 3" (Oracle, MS SQL Server and IBM DB2).
Especially when there are real bugs and/or short comings.
In my experience H2 would be a good benchmark. If H2 can't parse the statement, then JSQLParser should not bother.
Apologies for a simple minded question: Why exactly can't you write the CREATE Statement SQL Standard compliant (e. g. using double quotes for names and aliases)?

Greetings!
While I am no developer but just a user of JSQLParser, I wonder whether we should implement support for any exotic RDBMS slang or focus on something like latest SQL standard + support on the "big 3" (Oracle, MS SQL Server and IBM DB2).
Especially when there are real bugs and/or short comings.
In my experience H2 would be a good benchmark. If H2 can't parse the statement, then JSQLParser should not bother.
Apologies for a simple minded question: Why exactly can't you write the CREATE Statement SQL Standard compliant (e. g. using double quotes for names and aliases)?
I see.Thank you for your answer.

Can we close this please? Any objections?


Get-DbaDependency gets stuck in loop when InputObject has a circular reference.

Report
Get-DbaDependency gets stuck in loop when InputObject has a circular reference.
Host used
Name             : Visual Studio Code Host
Version          : 2020.6.0
InstanceId       : 447ac6de-6823-4100-86ee-ae7ba383d496
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : en-US
CurrentUICulture : en-US
PrivateData      : Microsoft.PowerShell.EditorServices.Services.PowerShellContext.EditorServicesPSHost+ConsoleColorProxy
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace
Errors Received
None.  The command is stuck in loop.
Steps to Reproduce
Create a couple inter dependent tables.

Actual Behavior
Command gets stuck with no output.
Environmental information
PowerShell Version        : 7.1.0
dbatools latest installed : 1.0.135
Culture of OS             : en-US

SQL Server:
select @@VERSION
-- Microsoft SQL Server 2016 (SP1-GDR) (KB4505219) - 13.0.4259.0 (X64)   Jun 15 2019 19:20:12   Copyright (c) Microsoft Corporation  Developer Edition (64-bit) on Windows 10 Pro 6.3 <X64> (Build 19042: ) (Hypervisor) ```
select @@LANGUAGE
-- us_english

Thanks @SamWalkerRxSense for the detailed report on this issue.


Analysis for routines for clean up of sent notifications

Description
Currently, no clean up of sent notifications is implemented.
We do not want to clean up to be implemented before we have a system for statistics in place.
We need to know how many notifications were sent on behalf of an application owner to the monthly bill.
Once in place an analysis for how to move forward should be conducted,
In scope
No response
Out of scope
No response
Additional Information
No response
Analysis
No response
Conclusion
No response

Requires a more thorough analysis of what's required of us when it comes to keeping/deleting old data.


[fork] asyncio server: ASSERTION FAILED: next_worker->initialized_cv

What version of gRPC and what language are you using?
Python GRPC
grpcio==1.53.0
What operating system (Linux, Windows,...) and version?
Linux / Docker
python:3.11.3-slim-bullseye
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.11
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
We have a python asyncio enabled grpc server, which delegates some work to a ProcessPoolExecutor.
What did you expect to see?
Server runs without error
What did you see instead?
After a minute or so, requests causes one of the sub processes in the process pool to segfault and die. This renders the process pool dead, and all requests fail after this.
E0411 18:31:05.916260573     152 ev_epoll1_linux.cc:1114]              ASSERTION FAILED: next_worker->initialized_cv
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?
Previous version that worked was 1.51.3
Process pool is created like this:
executor = ProcessPoolExecutor()

And called inside the RPC function like this:
await asyncio.get_running_loop().run_in_executor(self.executor, func, *args)

One work around I found was to create the ProcessPoolExecutor using "spawn" multiprocessing rather than the default "fork". At least on linux the default was fork which I guess was causing issues with the grpc runtime. Spawn creates a whole new process so it would avoid any issues with inheriting memory from the parent process which was serving the GRPC server.
executor = ProcessPoolExecutor(initializer=initializer_func, mp_context=get_context("spawn"))

Confirming that we are encountering the same issue. Upgrading to the latest version (1.63.0) causes GRPC to segault inconsistently.
[359272.565767] event_engine[1857256]: segfault at 0 ip 00007f0f4f043052 sp 00007f0e5fad6a30 error 4 in cygrpc.cpython-311-x86_64-linux-gnu.so[7f0f4edb7000+889000]
[359275.224553] pt_main_thread[1791687]: segfault at 7f0e40044b90 ip 00007f0f4f0504d5 sp 00007f0e613d8738 error 4 in cygrpc.cpython-311-x86_64-linux-gnu.so[7f0f4edb7000+889000]
[359333.882836] grpc_global_tim[1785863]: segfault at 7fde1c00ebd0 ip 00007fde1c00ebd0 sp 00007fde23ffe6f8 error 15

@3asm hi, are you using gevent?

@sreenithi we are not. We did however see that upgrading to the latest version of Python fixes the issue.

Glad to hear it was resolved using latest version!
@thjwhite Can you test if updating to the newest version can solve the problem?


bokehjs failure to load due to content security.

Description:
looks like some bokeh components are failing to load:

Steps to Reproduce:

connect to remote kernel
run base bokeh through output_notebook()

Versions:
Which OS and which version of Hydrogen and Atom are you running?
windows 7
atom 1.26.1
jupyterlab ubuntu server 17.10
python3.6.3
ipykernel 4.8.2
ipython 6.3.1
ipywidgets 7.2.1
bokeh 0.12.15
jupyterlab 0.32.1
Logs:
console output:

entirely possible i'm missing something or this has to be done manually?
https://bokeh.pydata.org/en/latest/docs/installation.html

tried the same test on ubuntu desktop 17.10 direct to local jupyterlab. same results:

p.s. while submitting the above, found the following which resolved the issue in case anyone runs across the same:
https://github.com/bokeh/bokeh/issues/5351


upload image that name's is ads.jpg

Hi everyone !
I am try to upload images has name "ads" but image show blank image as screenshot in jpg or png
1

2

3

@alfuhigi do you happen to have an adblocker enabled in your browser? I suspect the filename may be triggering a block if so. Does this same image render if embedded into some other HTML document not served by Wagtail?

Oops my fault !
thanks @chosak ! , you are right the adblocker is enabled. it detected images as ads


Connection Refused in Mapreduce sample

Hello I am getting the following error when trying to run the Mapreduce sample:
19:47:07,866  WARN t.support.ClassPathXmlApplicationContext: 487 - Exception encountered during context initialization - cancelling refresh attempt
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'runner': Invocation of init method failed; nested exception is org.springframework.scripting.ScriptCompilationException: Could not compile script [class path resource [copy-files.groovy]]: Execution failure; nested exception is javax.script.ScriptException: javax.script.ScriptException: org.springframework.data.hadoop.HadoopException: Cannot test resource /user/gutenberg/input/word/;Call From Gabriels-MacBook-Pro.local/<IP_ADDRESS> to localhost:8020 failed on connection exception: java.net.ConnectException: Connection refused; For more details see:  http://wiki.apache.org/hadoop/ConnectionRefused
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1574)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:539)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:476)
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:303)

I am running Hadoop version 3 in mac OS X.
$HADOOP_HOME = /usr/local/Cellar/hadoop
$JAVA_HOME = /Library/Java/JavaVirtualMachines/jdk1.8.0_144.jdk/Contents/Home
jps
93667 DataNode
93810 SecondaryNameNode
94004 ResourceManager
94884 Jps
94102 NodeManager
93562 NameNode

Any idea?

Changed my core-site.xml solved the issue.
<configuration>
   <property>
      <name>fs.defaultFS</name>
      <value>hdfs://localhost:8020</value>
   </property>
</configuration>

Now I am facing a problem with:
org.apache.hadoop.ipc.RemoteException(java.io.IOException): File /tmp/hadoop-yarn/staging/gabrielmunumel/.staging/job_1525387543266_0001/job.split could only be replicated to 0 nodes instead of minReplication (=1).  There are 0 datanode(s) running and no node(s) are excluded in this operation.
	at org.apache.hadoop.hdfs.server.blockmanagement.BlockManager.chooseTarget4NewBlock(BlockManager.java:1559)

But this is another issue I guess.

Looks like your cluster has some issues with the datanodes. Also, we have not used the framework with Hadoop 3 and have no plans on evolving it further, see notice at https://projects.spring.io/spring-hadoop/

Realized that. I have downgraded hadoop to version 2.6.
To solved the last issue I have removed my tmp dir and recreated with hdfs namenode -format. It is working now. The only issue is that the job hangs.
I will close the issue because seems not related to spring.


Detail about SURVIVOR simSV

Hi fritz.
I tried to make simulated reads using SURVIVOR simSV
I created SV parameter file as bellow:
$ cat parameter_file
PARAMETER FILE: DO JUST MODIFY THE VALUES AND KEEP THE SPACES!
DUPLICATION_minimum_length: 100
DUPLICATION_maximum_length: 10000
DUPLICATION_number: 3
INDEL_minimum_length: 20
INDEL_maximum_length: 500
INDEL_number: 1
TRANSLOCATION_minimum_length: 1000
TRANSLOCATION_maximum_length: 3000
TRANSLOCATION_number: 2
INVERSION_minimum_length: 600
INVERSION_maximum_length: 800
INVERSION_number: 4
INV_del_minimum_length: 600
INV_del_maximum_length: 800
INV_del_number: 2
INV_dup_minimum_length: 600
INV_dup_maximum_length: 800
INV_dup_number: 2

and, I run this command:
SURVIVOR simSV ~/human_reference_genome/GRCh38_p12_chr22.fa parameter_file 0.1 0 simulated

This script takes more than 2 days for generating SVs.
Can you explain how SURVIVOR simSV works and why it takes so long time?
I imaged that just modifying reference sequence doesn't take so long time.
Sorry for bothering you.
Thanks.
harazono

The main reason is that it tries to place SVs in an non overlapping way, which might get tricky for such a short chromosome.
Second reason is that you requested 2 translation which cannot be simulated with only one Chr.
I thought I put in some warnings, but it seems that they did not catch this. I will try to improve this.
Thanks for letting me know
Fritz

I have just checked the code and changed it so it gives a warning much sooner.
I hope that helps.
Fritz

I reduced number of SVs and set 0 for TRA.
It looks works well.
Thanks a lot !
harazono


Getting Err value on provider.get_contents().unwrap()

The (WindowsClipboardContext) clipboard provider, on calling get_contents(), returns an Err value with code 0 and a message saying "The operation completed successfully." This makes it impossible to unwrap() the contents and use them although the clipboard provider appears to have successfully captured the clipboard.
Windows 10, Rust 2018 edition.

I believe this happens when the clipboard is empty.


{j|$variable|j} interpolation not type safe.

I have an opaque type that {j|url/$myOpaqueType/path|j} is perfectly happy compiling. If I change my syntax to "url/"++myOpaqueType++"/path" it won't compile which is what I would expect from the interpolation syntax as well.
I suppose I'm fine with safe coercions like string_of_int or string_of_float because those have an expected behavior but I certainly want the compiler to yell at me for using a custom type in an interpolation without first converting it to some string safe type first.

This issue belongs in the BuckleScript repository. There's no interpolation syntax outside of BuckleScript.

Ah my B!


typings/browser/definitions/xlsx/xlsx.d.ts(250,17): error TS2304: Cannot find name 'object'

Hi,
In my workplace, we use npm, jspm and TypeScript to develop in our comprehensive angularjs application.
After installing xlsx, and adding the types to the file "typings.json", I run the command "npm i", but I got the error "typings/browser/definitions/xlsx/xlsx.d.ts(250,17): error TS2304: Cannot find name 'object'"
Looking at the code in node_modules\xlsx\types\index.d.ts, I see that the Workbook interface has a optional property "Custprops?" at line 247 of type "object".... Apparently, that is not a valid type. After I change it to "Object", the command "npm i" gives no error.
Now, that would not be a great deal if I run my application locally, but in a professional environment, in each stage of deployment, "npm i" is run on each build in each stage. So, in other than local environments, where I cannot change "object" to "Object", this will be a problem.
Can you fix that please ? Or is there any other way to solve this problem ?
Thanks !
Erwin Bauwens
developer
Belgium

we are using typescript 2.0.6... unfortunately, company policy prohibits me to share the project...

tried using typescript 2.2.0 and that works fine.... but other dependencies in project work only with 2.0.6...
Is there anything I can do to use the typings of XLSX with this earlier version ?
thanks

We'd accept a PR that changes object to any for Custprops

Changed the type def and credited 66a9cc79ca18ea5979bdd4287950c40e01d983a0 to you


Fix ansible lint errors and upgrade ansible-lint to 24.2.0

Problem
Linting fails
Solution
Fix syntax
Tests
Passed: 0 failure(s), 0 warning(s) on 382 files. Last profile that met the validation criteria was 'production'.

@KimForss this fixes the linting errors (for now)


Maintain major release tags

This will maintain branches release-v1 (and in the future release-v2) that always point to the latest patch release of the same major version.
For every v1.x.y tag, it will force push to a new or existing release-v1 tag, and update the release assets for a "Latest v1 release" Github release to be the same cli assets as the latest v1.x.y tag.
Will also work automatically when we start new tags with 2.x.y (it will then create a release-v2 tag and a "Latest v2 release".
This allows customers and our actions to always download assets from the latest CLI release within a specific major release, using:
curl -L https://github.com/debricked/cli/releases/download/release-v1/cli_linux_x86_64.tar.gz | tar -xz debricked

Amazing work, thanks!


Adds functionality to publish the tsdf layer as well upon service call

Can one of the admins verify this patch?

add to whitelist


Fixed various typos

While translating strings, I found some typos in Orchard localized strings. This PR is to correct those typos.

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


Use caching to speed up response time of workflow

Currently you have to wait a little bit to get a response from bitbucket, even for the same query. It would be valuable to have a configurable cache lifetime, which could be implemented easily by adding the maxAge option here: https://github.com/nicklayb/alfred-bitbucket/blob/master/utils.js#L3
It's also important that this cache lifetime be configurable (possibly an option to clear?) as I could have it cached for days at a time as my repos do not update often

@jonicious has add the feature using an env configurable delay. Hope it matches your need.


154 get deposit to work

Changes have been merged as part of other PRs. @fenekku if you see that something is wrong, please open it again :)


ok idk what happened to the last PR

Moving the version number + Formatting main.menu #1
Results in this, which is, in my opinion, the best place to put the version number

Also dealt with the horrible respawn formatting in main.menu :)
oh and also contains the bumped version numbers commit from earlier EDIT: that was breaking things so i removed it

this should work now finally

yeah looks great!


Optimize proxy to pod|service mapping computation and usage

Please describe the Improvement and/or Feature Request

The current code is pretty suboptimal because:

We compute proxy->service and proxy->pod mappings repeatedly on every update to proxies, even when there has been no change to those mappings (pod or service updates in the cluster).
Repeated cache access on thread-safe structures results in contention, which could be avoided.

I was thinking we could do the following:
When a service/pod event is received, don't generate a proxy broadcast from the broker, but compute proxy-service && pod-service mappings, and trigger updates to relevant proxies. As a start, we can broadcast the update and optimize on filtering proxies as a next step
With the above optimization, there would be no need to look up all pods/services in the mesh for proxy config generation when there are no pod/service changes in the cluster, significantly resulting in performance in scale environments.
Scope (please mark with X where applicable)

Performance          [ ]
Envoy Control Plane        [X]

Possible use cases
Performance optimization for scale environments.

tl;dr I'm not 100% convinced (the way I've implemented) this results in any significant performance gains.
I have a working version of these changes on a branch here: https://github.com/openservicemesh/osm/compare/main...nojnhuh:svcreg. (In case I update that branch, it's currently pointing to https://github.com/openservicemesh/osm/commit/f2643c6228be113e35acf897f4ccfdf120e1ab63).
To summarize how this works, a new background goroutine is spun up to listen for events related to Kubernetes Endpoints resources. When an event comes through, a local cache mapping proxies to services is updated. Queries to this cache when XDS responses are generated are then simply sync.RWMutex-protected map lookups. For context, currently when any XDS handler needs to know which services are associated with a proxy, the k8s.Controller's local caches are traversed to rebuild that mapping from scratch each time.
My changes do not address triggering updates for individual proxies based on each event and instead update all proxies for any Endpoints event seen. I hoped that we would only need to update the proxies associated with a service when its corresponding endpoints were updated, but I did run into one case where we would also need to update any proxy allowed to communicate with a service whose endpoints were updated, which would require much more bookkeeping. The issue I saw was, roughly:

mysql Endpoints created with no ports listed
bookwarehouse proxy connects and gets initial config. mysql cluster is not programmed because the target port on the corresponding MeshService is 0.
mysql Endpoints updated with mysql Pod, only an update for the mysql proxy is triggered
No broadcasts are triggered, so bookwarehouse config remains stale (no mysql cluster).

To test how these changes compare with the existing behavior, I ran both the main branch and my changes through two different scenarios. All tests were run on an AKS cluster with 6 Standard_DS2_v2 nodes in the context of the bookstore demo with each service scaled up to 10 replicas. The main branch for my tests was at 8dccabbf98d4187d7072b2145b1e280d45303f6d. Each test installs the demo, runs some operation repeatedly, runs a CPU profile for 15 minutes, takes a memory heap profile, then takes a Prometheus snapshot.
The first scenario constantly churns Endpoints as fast as possible:
while true; do kubectl delete ep -n bookstore --all; done

This is essentially the worst-case scenario for the async mapper where it does as much work as possible.
The second scenario constantly updates a label on a traffic target:
while true; do
  kubectl label traffictarget -n bookstore --overwrite bookbuyer-access-bookstore-v1 churn=$(date +%S)
  sleep 2 # osm-controller coalesces proxy broadcasts triggered less than 2 seconds apart
done

The particular operation isn't important here, just that it triggers a proxy update broadcast and that it doesn't add/update/delete any Endpoints. This is the best-case scenario for the async mapper, where it will build the cache once and never need to update it.
The results are summarized below:

scenario
log level
branch
Total CPU%
% of total CPU% spent mapping proxies
Memory heap

endpoint-updates
disabled
main
2.40%
2.31%
128.24MB

endpoint-updates
disabled
svcreg
3.16%
0.21%
109.83MB

endpoint-updates
trace
main
3.93%
2.63%
87.83MB

endpoint-updates
trace
svcreg
5.27%
6.96%
116.98MB

traffictarget-label
disabled
main
4.56%
4.97%
25.34MB

traffictarget-label
disabled
svcreg
6.16%
0.036%
25.8MB

traffictarget-label
trace
main
9.29%
5.59%
24.12MB

traffictarget-label
trace
svcreg
8.78%
0.90%
22.08MB

Some notes on the above table I find surprising:

The "% of total CPU% spent mapping proxies" for the endpoint-updates scenario on svcreg branch is much higher for trace logging than disabled logging. In the profile, I can see that the time spent logging does not come close to making up that whole difference.
For the traffictarget-label tests with disabled logging, the "Total CPU%" for the svcreg branch is slightly higher than the main branch despite the "% of total CPU spent mapping proxies" being almost 0.

The full data and extra scripts I used:  mapping.tar.gz
In that tarball, each of the endpoint-updates and traffictarget-label directories contain the following:

snapshots/: Prometheus snapshots that can be imported to another Prometheus instance (https://suraj.io/post/how-to-backup-and-restore-prometheus/)
*.pb.gz: pprof profiling output named for the image tag, log level, and type (cpu/heap). Inspect with go tool pprof.

Full steps to gather the same data:

Build and push OSM images off of the main branch. I tagged mine latest.
Build and push OSM images from my branch linked above. I tagged these svcreg.
Select a scenario to run at the bottom of scenario.yaml from the above tarball.
Run profile.sh from the above tarball. (This will take about 75-90 minutes)

I didn't try to make this portable at all, so you might need tweaks to get this to run.

snapshots/ and *.pb.gz end up in ~/Desktop. Make sure to move these before running more scenarios.

Overall, considering the profiling results are inconclusive at best, I'm not sure it makes sense to add these changes, at least the way I've implemented them.


readme and changelog edits

@jichangjichang  please check lisa's modificaiton.

@LisaDelaney  Should we apply this to rocm 6.0 branch as well?

@LisaDelaney Should we apply this to rocm 6.0 branch as well?

It would be nice if we could...but not a tragedy if we can't.  :)

@LisaDelaney  Review process is done. Please merge if you don't have other modification.


Enable stricter Gradle plugin validation

Pull Request Details
Improves cacheability of plugin tasks.
Description

Enable stricter Gradle plugin validations
Remove unnecessary property annotations from extension
Add missing property annotations to tasks

Related Issue
Addresses one of the points raised in https://github.com/JetBrains/gradle-intellij-plugin/issues/1134, specifically:

The properties should be annotated correctly
https://docs.gradle.org/current/userguide/more_about_tasks.html#table:incremental_build_annotations

Closes #1258
Motivation and Context
This improves the state of Gradle incremental builds and also identifies tasks that are missing annotations that declare whether they're cacheable or not. Both of these improve build performance.
Changing the configuration of the validatePlugins task to use stricter checks will ensure this problem doesn't occur again as new tasks and properties are added.
How Has This Been Tested
Ran ./gradlew validatePlugins. I haven't done testing of the tasks though. The annotations need to be reviewed by someone who has detailed knowledge of each task before merging.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have included my change in the CHANGELOG.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thank you very much for taking care of it, Matthew!
For tasks you've marked with UntrackedTask that download additional resources – I'll extract CLI tools downloading to separated tasks (as they may or may not be executed when the latest value is used as a CLI version), so the actual task will be possible to get cached if CLI is already available.


Stopify compilation bug involving boxing

Vague directions to reproduce:

Create a class that uses super
Apply the Babel transformation to eliminate the class keyword
Apply stopify and it will break. It seems to create ".box.box.prototype".

Can you give the code of the source program you used? I tried making a simple test case and haven't reproduced this.

You have access to ocelot-ide.org. Can you try your example there? Ocelot does babel-class-desugaring then Stopify.


Fix cpp stdlib include error

While trying to compile with your toolchain using C++ I ran into an error:
ERROR: /work/dbenusovich/sw/fw/app/arm_test/BUILD:6:10: Compiling fw/app/arm_test/main.cc failed: (Exit 1): arm-none-eabi-gcc failed: error executing CppCompile command (from target //fw/app/arm_test:binary) external/toolchains_arm_gnu~~arm_toolchain~arm_none_eabi_linux_x86_64/bin/arm-none-eabi-gcc -MD -MF bazel-out/k8-fastbuild-ST-bf8fd271169b/bin/fw/app/arm_test/_objs/binary/main.d ... (remaining 40 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
In file included from fw/app/arm_test/main.cc:3:
external/toolchains_arm_gnu~~arm_toolchain~arm_none_eabi_linux_x86_64/arm-none-eabi/include/c++/13.2.1/cstdlib:79:15: fatal error: stdlib.h: No such file or directory
   79 | #include_next <stdlib.h>
      |               ^~~~~~~~~~
compilation terminated.
Use --verbose_failures to see the command lines of failed build steps.

This is how I was including the toolchain:
bazel_dep(name = "toolchains_arm_gnu", version = "1.0.2")
arm_toolchain = use_extension("@toolchains_arm_gnu//:extensions.bzl", "arm_toolchain")

arm_toolchain.arm_none_eabi()
use_repo(arm_toolchain, "arm_none_eabi")
register_toolchains(
    "@sw//fw/toolchain/arm:all",
)

With a custom toolchain of:
# Cortex-M7 toolchain
arm_none_eabi_toolchain(
    name = "cortex_m7_toolchain",
    copts = [
        "-mcpu=cortex-m7",
        "-mthumb",
        "-mfloat-abi=hard",
        "-mfpu=fpv5-d16",
    ],
    linkopts = [
        "-mcpu=cortex-m7",
        "-mthumb",
        "-mfloat-abi=hard",
        "-mfpu=fpv5-d16",
        "-nostartfiles",
    ],
    target_compatible_with = [
        "@platforms//os:none",
        "//fw/toolchain/arm:cortex_m7_cpu",
    ]
)

Very similar to the examples you provided :) Thanks for making those!
@mark64 pointed out that the order of includes was incorrect. I am not sure if this will break something for other users so feel free to decline this change. However, this does compile for C++ now.

Cool, we faced the same issue. @hexdae Do you have a plan, or how can we support in releasing this? :)

Hey @FaBrand, this change should be released in the latest version on the registry


Multiple USDC assets showing up in asset search

Overview
When searching for "USDC" in asset search (trade component for example), 2 other flavors of USDC show up first (USDC wormhole token), see screenshot attached.
The real USDC should be displayed first to not confuse the user.
And maybe the 2 other tokens should not be called simply USDC but something like USDC (Wormhole).

References and additional details
Link to the 2 others tokens on etherscan :
https://etherscan.io/token/0x7cd167b101d2808cfd2c45d17b2e7ea9f46b74b6
https://etherscan.io/token/0x566957ef80f9fd5526cd2bef8be67035c0b81130
Acceptance Criteria
At least the real USDC should show up first in the list.
Ideally the assets should have different displayed name.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
4 hours

Related to https://github.com/shapeshift/web/issues/2101 - same domain, we should tackle these in sequence to make sure we have a maintainable logic that works with both.
Agreed about the tagging part, at the very least we should be able to override these assets.
We already have some asset labeling logic for the specific of X on Y chain e.g:

X on Avalanche: https://github.com/shapeshift/lib/commit/ee13ac8ab46c2bfa07d30be1dceeac13b13cb3b0#diff-b7ef478822cbb04bdaa2867afc02f32c57fdd32f520da5979da8acf5ac5e73aaR45
X on Osmosis: https://github.com/shapeshift/lib/commit/947e654fb7cbce1f1c2635295737571042357509#diff-092ab88ac8582f8a698f1f1c3c49a8d78ff853a3daa7668fa2813a299569eebbR57

We could iterate on this logic and arbitrarily override asset names, in a similar fashion as how we override descriptions


add namespace option in CloudConfig

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes: kubermatic/kubermatic#9108
Special notes for your reviewer:
Optional Release Note:

NONE

Signed-off-by: Sankalp Rangare<EMAIL_ADDRESS>

/retest

/retest

/lgtm

/retest

/retest

/retest


How to test  some pipeline steps without mock ?

Desired Behavior
I have simple  class method wich I want to  test :
static def getAllProperties(script,configDir,appId ) {
   def envProperties = script.readProperties file:  "${configDir}/env.properties"
    def props =  script.readProperties   default: envProperties ,file: "${configDir}/${appId}/env.properties"
    props
  }

I'm using  JenkinsPipelineSpecification to write tests in my project but in this case, readProperties are mocked and test will not detect that I have typo in parameter default. Also, the compiler will not fail. Is it possible to set up a test to fail or somehow configure to really call readProperties step ? Mocking  using
1 * getPipelineMock("readProperties")(default:["x":"valueX"],file:"./appId/env.properties") >>  ["y" : "valueY"] doesn't help as there is same typo as in the method implementation
Benefits

Detecting such typos during a test is much more efficient than deploy to Jenkins and test manually

I am not quite sure I understand the question. It sounds like you're asking:
Assuming

a function takes in a map of arguments
One of the arguments' names is default

e.g. getAllProperties(default: ["x": "valueX")
Then
I would like a unit-test failure if my code calls this function with a typo, e.g.
getAllProperties(deflat: ["x": "valueX")
Is that correct?

Yes, that's what I need.  There is  method readProperties   ( from the "Pipeline Utility Steps plugin")  which expects "defaults" on input in my example, but I used  "default". The test passes because I also used  "default" in the getPipelineMock. I would like to find these types of errors/typos before deploying to Jenkins.


Get consumer group rebalance event signal

In the case of some middleware, manual commits with Kafka happen re-consume, for example.
Kafka -> application -> cache -> consume -> commitoffset
In this case, a new client joining the group will occur with an offset not committed. Making the new client re-consume once,When cache become empty I can call AllowRebalance, but this has no effect until the rebalance happens and maybe occur error(message lost).

Although we can use OnPartitionRevoke, in this function you cannot send signals to other goroutine (or channels, since there are only three param).

Not sure I understand the first comment fully -- are you saying that BlockRebalanceOnPoll and AllowRebalance does not prevent the double-processing issue you're running into?
Also, what are you looking for in OnPartitionRevoke? In OnPartitionRevoke, can pass a method that has the state you need, similar to this: https://github.com/twmb/franz-go/blob/master/examples/goroutine_per_partition_consuming/autocommit_marks/main.go#L127 ?

tks, I missed example of  goroutine_per_partition_consuming.


Email Verification

Currently no email verification is being done which can lead to spammy accounts been created. A verification link should be sent to the email before signing which the user must click to verify the account.
Pls assign me this issue under hacktoberfest

Assigned ! @tanish35


Fix Unit Tests

Describe the issue
Unit Tests are broken.
Is this a bug in the codebase(not a usage issue) that needs to be fixed?
Yes
What files need to be fixed? (Please list them all)
make check
then loop at src/test-suite.log for the files that need to be fixed.
What needs to be done?
Change the unit tests to use GoldCoin information (such as addresses and private keys)
Why?
Ensure that the GoldCoin upgrade does not have bugs.  This may help with other issues.

This code was added to miner_tests.cpp to mine 100 blocks and report the nonce values to update the blockinfo data structure.
        if(!CheckProofOfWork(pblock->GetPoWHash(), pblock->nBits, chainparams.GetConsensus()))
        {
            if (true) {
                // This will figure out a valid hash and Nonce if you're
                // creating a different genesis block:
                arith_uint256 hashTarget;
                hashTarget.SetCompact(pblock->nBits);
                while (UintToArith256(pblock->GetPoWHash()) > hashTarget)
                {
                    ++pblock->nNonce;
                }
            }
            char S[64];
            sprintf(S, "%d.  old=0x%08x : new=0x%08x", i, blockinfo[i].nonce, pblock->nNonce);
            BOOST_CHECK_MESSAGE(pblock->nNonce==blockinfo[i].nonce, S);
        }
        std::shared_ptr<const CBlock> shared_pblock = std::make_shared<const CBlock>(*pblock);
        BOOST_CHECK(ProcessNewBlock(NULL, chainparams, shared_pblock, true, NULL));
        pblock->hashPrevBlock = pblock->GetHash();
    }

From my testing and that of the travis system, this appears to be complete.  Any other changes to the code may break these tests and cause them to fail.  They should always be run before submitting before making commits.

Unit tests were fixed.


conflicting references

Imaging:
I have 2 test cases. Both of them use a library in one repo. One requires the library in master branch, the second requires the library in the e.g. deprecated branch.
Tmt now rejects to run with message
Library 'XY' using ref 'deprecated' conflicts with already fetched library using ref 'master'.

Tmt should be able to split such conflicting cases into e.g. a virtual plans so both cases can be executed. In the worst case there would be as many virtual plans as test cases. However the results should be aggregated under the original plan.
Simply adding one more directory under plans/<name> could work. By default there would be plans/default/0 the conflicting would split to plans/default/1, ...

I was thinking, if the following two changes would be incorporated:

rlImport does work with an PATH-like environment variable specifying (prioritized) directories which should be used for library lookup
during the discovery (?) step, fmf would track which libraries where imported where (conflicting library would be downloaded to a different location) so for each test (or library) a complete directory stack would be tracked and during the test execution the environment variable above environment variable would be populated with the respective directory stack.

Would that work?

I was thinking, if the following two changes would be incorporated:

rlImport does work with an PATH-like environment variable specifying (prioritized) directories which should be used for library lookup

This could work for libraries being in he path is exaclt expected sub-paths (component/{,Library/}lib_name). An ability to search using upwards traversal is not applied to BEAKERLIB_LIBRARY_PATH. That could be changed, though.

during the discovery (?) step, fmf would track which libraries where imported where (conflicting library would be downloaded to a different location) so for each test (or library) a complete directory stack would be tracked and during the test execution the environment variable above environment variable would be populated with the respective directory stack.

This could be done e.g. by symlinked structure for each test. Basically each test would have it's own workdir with all the relevant dependencies. This way it would not be even necessary to utilize the BEAKERLIB_LIBRARY_PATH.

Would that work?


No matching version found for<EMAIL_ADDRESS>

I've been getting this issue recently,
 ERR_PNPM_NO_MATCHING_VERSION  No matching version found for<EMAIL_ADDRESS>
This error happened while installing the dependencies of<EMAIL_ADDRESS>
The latest release of @llamaindex/openai is "0.1.2".

If you need the full list of all 1 published versions run "$ pnpm view @llamaindex/openai versions".

I'm not sure which package is importing llamaindex but seems the 0.1.1 doesnt exist in npmjs anymore.

We had the same error in our package where we use llamaindex 0.5.20. It requires @llamaindex/openai 0.1.1. We were unable to install the package after the version 0.1.1 was removed from npm.
bumping llamaindex 0.6.0 fixed the issue since it depends on @llamaindex/openai 0.1.2. https://github.com/upstash/rag-chat/pull/72
Still, the older versions of @llamaindex/openai shouldn't have been removed from npm

@CahidArda sorry for the trouble, the version wasn't removed - the problem was that it was never published as it was accidentally marked as private package 😄


parseContent() code blocks undesigned

see this image u can know doesn't designed code block
how can i fix this problem?

Hi, this is fixed in v1.3.1


New Gcloud plugin search path for ArchLinux

Hi @ianchesal, I have added the path for ArchLinux, the SDK is located on /opt/google-cloud-sdk.
Cheers.

👍 thank you!

Thanks!


Policy is not saved

Bug description
After running enforcer.savePolicy() and restart the service. Previous adding policy is not saved
Details
I setup the enforcer to use prisma-adapter
export const getEnforcer = async () => {
  const a = await PrismaAdapter.newAdapter()
  const e = await casbin.newEnforcer(`${__dirname}/rbac_model.conf`, a)
  e.enableAutoSave(true)

  return e
}

and add role for user
await enforcer.addRoleForUser(newUser.id, 'user')

but the data didn't save to prisma as when I restart the server and check userRoles it returns empty array of roles
const roles = await enforcer.getRolesForUser(userId)

even when I do manual save
await enforcer.savePolicy()

@nodece @zhmushan can anyone take this issue?

Hi @hsluoyz,
thank you for quick reply
Just realise it's problem coming from my casbin setup :)
So I will just close the issue
sorry for the trouble


Send an HTTP request doesn't send the custom headers added

Describe the bug
Send an HTTP request doesn't send the custom headers added
Version
2.1.0
Browser

[ ] Electron distribution

OS

[ ] Windows

To Reproduce
Steps to reproduce the behavior:

Go to 'Send an HTTP request'
Click on 'add headers'
give values for headers name and value
on http request trace in emulator i find headers are not sent in the request headers and the request fails due to this.

Expected behavior
Added headers should pass in the request headers.
Screenshots

Hey @SSanjeevi ,
I think there is an issue with the SDK that is not logging the custom headers in the HttpRequest trace, because I don't see my custom headers in the Emulator either when trying to reproduce this.
However, if you use a network traffic monitor like Fiddler, you can see that the outgoing request does in fact have the custom headers attached:


Update dependency pathspec to v0.10.3

This PR contains the following updates:

Package
Update
Change

pathspec
patch
==0.10.2 -> ==0.10.3

Release Notes

cpburnz/python-pathspec
v0.10.3
Compare Source
New features:

Added utility function pathspec.util.append_dir_sep() to aid in distinguishing between directories and files on the file-system. See Issue #65_.

Bug fixes:

Issue #66/Pull #67: Package not marked as py.typed.
Issue #68_: Exports are considered private.
Issue #70/Pull #71: 'Self' string literal type is Unknown in pyright.

Improvements:

Issue #65_: Checking directories via match_file() does not work on Path objects.

.. _Issue #65: https://github.com/cpburnz/python-pathspec/issues/65
.. _Issue #66: https://github.com/cpburnz/python-pathspec/issues/66
.. _Pull #67: https://github.com/cpburnz/python-pathspec/pull/67
.. _Issue #68: https://github.com/cpburnz/python-pathspec/issues/68
.. _Issue #70: https://github.com/cpburnz/python-pathspec/issues/70
.. _Pull #71: https://github.com/cpburnz/python-pathspec/pull/71

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


currentProject andcurrentStack to OIDC claims sent b Pulumi.

Goal is to ensure only specific stacks can use an environment and get cloud credentials.

https://github.com/pulumi/pulumi-cloud-requests/issues/400


Fix htmlhint variable names

Thought: It might be nice to also have a function that sets ale_... variables, to further avoid these errors but also just because it's nicer when you don't have to specify the variable name twice, like you do when using let ... = get(...).

That's my bad. That looks good to me.
I considered adding a function for setting variables, but it seems like overkill. I tend to avoid hiding logic in small functions until I'm positive it's an improvement. I'll keep considering it.


\mathstrut is not implemented

Seems it should be something like <mphantom style="width:0">(</mphantom>, i.e., an object that has the height of a paren but no width?

temml does this:
  <mpadded width="0px">
    <mphantom>
      <mo form="prefix" stretchy="false" lspace="0em" rspace="0em">(</mo>
    </mphantom>
  </mpadded>


nomad

adds nomad-based deploy implementation

Hey, @adamdossa, there was one final issue remaining around WS urls are all valid: https://github.com/PolymeshAssociation/polymesh-apps/blob/develop/packages/apps-config/src/endpoints/index.spec.ts#L18
I've renamed RPC_ENDPOINT -> RPC_HOSTNAME to reflect the change from a URI to a hostname, so that the templated value works correctly while also maintaining the existing tests/other functionality.
Otherwise, during testing, the application sets a localStorage.settings.apiUrl value, which isn't cleared when loading the application without cache (despite it being set correctly at run-time), so it was a bit of a headache to track down why the value wasn't being successfully updated - quickest fix if you have a bad value cached is to localStorage.clear() in the dev tools' console. This won't affect end-users, but during testing it's definitely a bother.

This is live @ https://staging-app.polymesh.dev/


Citation (rehype-citation) does not work

Describe the bug
I have added the following text to the .mdx file: Check citations.[@Nash1950]
The .bib file remains untouched and it contains Nash1950:
@article{Nash1950,
  title={Equilibrium points in n-person games},
  author={Nash, John},
  journal={Proceedings of the national academy of sciences},
  volume={36},
  number={1},
  pages={48--49},
  year={1950},
  publisher={USA}
}

Citation does not work. Rendered markdown does not recognize it:

To Reproduce

See the branch code: https://github.com/dariusz-wozniak/dariuszwozniak.net-v2/pull/8
And deployed version: https://dariuszwozniak-net-v2-git-feat-6c2f69-dariusz-wozniaks-projects.vercel.app/blog/2023-11-02-hello-world

Expected behavior
Citation should work.
System Info (if dev / build issue):

OS: (Vercel one)
Node version: 18.x

Browser Info (if display / formatting issue):
Any
Additional context
No

You will need to add the location of the bib file to the frontmatter i.e. bibliography: references-data.bib like https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/12b4ed236fd75d59f2941b651d83d180f611c92e/data/blog/new-features-in-v1.mdx#L9C1-L9C34.
Or if you prefer to use one BibTex file for all posts, you can also add it to the options in the plugin.

@timlrx : works like a charm, thank you!


Task cannot be resolved

@mkaring System.Threading.Tasks.Task cannot not be resolved(system.runtime <IP_ADDRESS>)
.net core 2.1

You need to add all the assemblies of .NET Core as external references to your ConfuserEx project for the .NET Core 2.1 obfuscation to work.


Add copy_cpointer to String class to copy fixed number of bytes.

The exisiting copy_cstring method could be used for either
null-terminated or fixed size strings. The string was considered
null-terminated when the given length was 0, leading to surprising
behaviour when copying a 0 sized string.
Instead the method is split in two, copy_cstring for null-terminated
and copy_cpointer for fixed size, following the existing naming
convention used by other methods.

I'm not sure why this Appveyor build failed.
Also, this may need to be rebased and added question marks now that explicit partial calls has been merged.

I think this should have had a CHANGELOG entry. Can you add one @plietar?


TooManyFunctions description could be improved

TooManyFunctions - 11/10 - [VehicleRenderer] at car2go/src/main/java/com/foo/barui/renderer/VehicleRenderer.kt:8:1
It's not really clear which of the many threshold properties I'll need to use here.

What do you mean with which of the many threshold properties I'll need to use here?
If a class has too many functions the signature and position would change. This tells me that the file has too many functions ^^.

In this case the range is pointing to the class though and not the file.

Hm can you provide the class to me via email or something if it is closed source and I can check?

This is the file. I replaced the names with some arbitrary ones.

Looking at the TooManyFunctions rule it reports it's issues with detailed descriptions. These currently don't get printed to the CLI output, but are included in the HTML and XML reports.
See: https://github.com/arturbosch/detekt/blob/master/detekt-rules/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/complexity/TooManyFunctions.kt#L68 which clearly mentions that an interface has too many functions. That in turn makes the THRESHOLD_IN_INTERFACES parameter the obvious candidate.
I'm not sure about putting the config param in the issue message as that would be a bit along the lines of: "To fix this codesmell simply turn up the thresholds".

Oh okay, got it. Then I'll next time look at the xml / html report.

We could think about adding the message to the CLI output as well. But that might make the CLI output very noisy.


Can not find  class   io.netty.util.collection.LongObjectHashMap   in   4.1.8.final

In the  version   4.1.8.final
module   codec-redis
class     FixedRedisMessagePool
already  import  the  follow  class
import io.netty.util.collection.LongObjectHashMap;
import io.netty.util.collection.LongObjectMap;

it  Prompt error message is as follows ：Unused import statemen

but  I   can  not find  the class  in  the whole  project.
In  the   4.1.6  final  and   4.1.7  final  version
I  find  these  classes   in  the   io.netty.util.collection    package

In  the   4.1.8  final  version
the  whole  project  is  changed.   I   find    the  io.netty.util   package  is   in   common       module，
But   I   can not  find  in  io.netty.util.collection   package ，in   the  whole  project  ,   it   also  didn't  find  it  .
I   thought  it  may  forget  to  copy  the    io.netty.util.collection   package  in  the module  common .
only  my   thougnt,   help  me  @sprsquish @adamv @mheath @ngocdaothanh    ,   thank  you
In addition， I    am   from   China,     bad   English  ,  Please wait

@zhisheng17 its generated during build. run mvn clean compile in the common module.

thanks！

朋友 mvn clean我使用了  还是没用  求教

@Quantriple did you call compile as well?

@johnou   thx  mvn clean  then install          perfect

@Quantriple  like with you , but i find the way : using high version maven , i change my idea maven setting , using the 3.5.3 version , someone tell me , you can try it.
In addition， I am from China, bad English ......

@YangZhuoNJ   嗯   已经解决了  不过还是 谢谢你  都是国人 我就  不说 蹩脚的英文了

cd common;
mvn clean compile;

Reason in the common module common/src/main/templates/io/netty/util/collection directory.
The .template file.

在我这为什么不生效， Failure to find io.netty:netty-dev-tools:jar:4.1.26.Final-SNAPSHOT in https://oss.sonatype.org/content/repositories/snapshots was cached in the local repository，这个错有人遇到过没

早上试了一个小时了，这个答案对我没用，要么报错Failure to find io.netty:netty-dev-tools:jar:4.1.26.Final-SNAPSHOT in https://oss.sonatype.org/content/repositories/snapshots was cached in the local repository，要么不报错，但是问题也不解决，就是这么神奇。


Sprite Button Maker

Description
This extension changes your sprites depending on weather you hover over them or press them.
Easy event conditions: find out weather a sprite is clicked or being pressed.
The extension is fully customizable!
Define how much the button moves when clicked, weather it changes animation when clicked, if it responds only to Left mouse clicks or other Mouse clicks etc. ...
And you can change all settings inside the event sheet!
Features include:
-Super easy and intuitive: install the behavior and you already have a usable button!
-Setting draggable clicks on/off
-setting hover/pressed animations
-setting how much the button moves up/down when hovering/pressing
-deactivate all button events
-deactivate button animations while the button keeps responding to clicks
How to use the extension
You install the behavior on a sprite, and you can immediately set the parameters.
You can change all parameters inside the event sheet as well.
Checklist

[X] I've followed all of the best practices.
[X] I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.
[X] I am aware that the extension may be updated by anyone, and do not need my explicit consent to do so.

What tier of review do you aim for your extension?
Reviewed
Example file
sampleproject.zip
Extension file
extension.zip

Closing as duplicate of #540


the step of using the pretrained model to get HR image

thanks for the code, but how can i use the provided pretrained model to transfer a LR to HR? In another word , what is the step to use SR3 in this project? [config file ] means what in this project? if someone can instruct the step , i will appreciate that.

Hi, Did you solve it?


initialSlide should apply the correct class in the thumbs component

This is a (multiple allowed):

[x] bug
[ ] enhancement
[ ] feature-discussion (RFC)

Using the latest swiper version..
when applying the initialSlide option on a thumbs component that is synced to a thumbs gallery, then the class swiper-slide-active is in fact added, but in essence the swiper-slide-thumb-active should be applied to make it work properly.
The class swiper-slide-active is only added when option centeredSlides:true is set for the thumbs component.

The same problem for me

@nolimits4web would you be able to publish 4.5.2 and 4.5.3 on npm? I can't upgrade to >5 because I need IE11 support, but 4.5.1 has the bug described in this ticket, so I'd like to try 4.5.2 or 4.53.


fix: submit button not clickable

Added the pointer: '' to trailing icon as documented in https://ui.nuxt.com/components/input#trailing

Thank you!


..

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
Do you want to request a feature or report a bug?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Please tell us about your environment:
Angular version: 2.0.0-rc.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

@DaniloEmpire this does not look like an issue/question, please submit another one with the description. Thanks!

how can i do that?
Main Menu
->sub Menu 1
->Sub Main menu 1
->add
->delete
->edit
->Sub Main menu 1
->add
->delete
->edit
->sub Menu 1
when i create menu
1st step and 2nd step  work good
but when i add menu 3rd step with child this time my code doesn't work.
if i create this type menu in my application what can i do that?
can any one explain?


Support for pluggable bindings

Some weeks ago I wrote basic Universal Windows Platform bindings for Noble. These bindings are available at https://github.com/hgwood/winble. They are still pretty raw.
While developing those bindings I found out there is no easy way to make Noble use bindings that it is not aware of. Basically you have to edit noble.js. I think it would be useful if Noble users (especially bindings developers) could optionally load any bindings they want and make Noble use them without editing its source.
This PR is an attempt to bring this feature to Noble without breaking the API. As of yet it is just a quick sketch of what I'm thinking of and I haven't tested it. Hopefully, from there, we can start a discussion.

@hgwood cool!
@monteslu has done something similar in PR #238 to split out the binding resolver: https://github.com/sandeepmistry/noble/pull/238/files#diff-a3c59ae9a2f115e917fefe6609846672R1
noble has been using environment vars to control non-API flags, so another option would be to specific the binding via env var, so something like this:
process.env.NOBLE_BINDING = 'blah';
var noble = require('noble');

or command line
NOBLE_BINDING=blah node <file>.js

Thoughts? I'm ok when the require('noble/with-bindings') you suggested.

The with-binding solution is more flexible because it allows to pass an arbitrary object as bindings and that is useful when developing those, whereas with the environnement variable the bindings have to be in an installed module. The environment variable is still a useful possibility though, and I can add support for it.

Oh, wait. I guess NOBLE_BINDING could be set to something like ./my-bindings and that would work also when developing bindings. If you find this technique more in line with how noble works I'm OK with using it. It has the advantage that the code change required is a lot less significant. I'll update the PR later.

@hgwood your feedback from https://github.com/sandeepmistry/noble/pull/366#issuecomment-206202139 has convinced me to go with your original patch :) even though it is a slightly bigger code change. I'll merge later this week, I also want to split the "guessBindings" part out. As well, as start an advance usage section in the read or separate markdown file.

OK then. Ask me if you need any help.

@hgwood thanks!
Done, I've merged with the changes I mentioned in https://github.com/sandeepmistry/noble/pull/366#issuecomment-206634022, see https://github.com/sandeepmistry/noble/commit/e0b48b58f875979d536629063df29bba5544ed11 and https://github.com/sandeepmistry/noble/commit/c1e7a065f24fa0530bde41a1efab8b4917eb2965.
You can try out via npm install sandeepmistry/noble prior to the next publish to npm.

Great!


Struct/Models will now validate entries using the requires DSL

Allow Structs/Models to enforce any requirements described through the requires DSL:

Adds processing of requirements in the overloaded validate function of Model.

Fixes #152
Signed-off-by: Josep M. Blanquer<EMAIL_ADDRESS>

LGTM.


Add animations on scroll

🛠 Fixes Issue
Closes #149
✔ Check List

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

@ingressive4good @dabsclement
This is ready to merge


No information about licence

Some information about the licence of this project would be usefull.

https://github.com/benlau/quickflux/blob/master/LICENSE


Make Layer Editing Tools Contextual

Right now all of the layer editing tools are up in the toolbar.  This makes them rather awkwards as there is a process of "activating" and "deactivating" layers from the catalog and then needing to access the editing tools in the toolbar.
I propose the following:

Remove the Layer Tools from the toolbar.
Allow users to select which layer controls they wish to have displayed ONLY on editable layers.
If CONFIGURATION.services.disable_others is set then selecting a layer tool will disable the toolbar until work on the layer has finished.
Change layer "activation" to a process of "unlocking" and "locking" for editing purposes.  Ensure there is a warning for unsaved changes when the layer is "locked" that changes would not necessarily have been saved.

Negative consequences:

It removes some of the "free roaming" nature of GeoMOOSE.
It will make some of the editing tools a bit less obvious as they will be changed to small (though CSS stylable) icons under the layer instead of larger, more readable menu items).

Positive consequences:

I think it makes the workflow more clear to users.
It allows administrators to configure which and how they want layers to work.
The workflow is more inline with Desktop GIS systems.  There is widespread debate as to whether or not this is a benefit.  Given GeoMOOSE's target audience as of late (replacement for desktop GIS) then I think it is still a reasonable target goal.
It allows us to remove the goofy drop-down menu which just burys functionality anyway.

The layer tools have previously served two different purposes:

Freestyle on screen drawing and annotation resulting in in-browser vectors that show up in printing and are as temporary as the browser session.
WFS-T editing tools used on screen to send data to some serverside datastore through WFS-T.

Is this proposal to remove the first function of freestyle in-browser vectors and make the WFS-T editing tools more specific to individual WFS-T capable layers?  Separating these two functions would be great.  Making the WFS-T tools more layer specific would be an improvement too.  I think that there is still a substantial role for the freestyle on screen markup.
For the "activation" "unlocking" and 'locking" we might want to stay close to the language and icons of established analogies/workflows.  Here is a QGIS screenshot.
 "Pencils" and 3.5 floppies seem to be the editing icon language.
Buried drop down functionality is fine for rarely accessed and related tools.

The "just draw anywhere on the Map" aspect of the GeoMOOSE drawing tools has been a bit of a lie.
Really, what has happened is that the "drawing and markup" layer has been activated by default in the last few releases.  This can be problematic if someone is using the popups or using the vector editing capabilities because you can get (as @klassenjs likes to call it), "Tool Confusion."  It's hard to determine where and when a particular tool is valid or active -- including "draw polygon" or "draw point."
There are also issues now that if a vector layer should only be points, lines, or polygons that a less useful error is generated.  Instead, under this modal, only the "draw polygon" or "draw point" tool will be available when that layer is "editable."
I'm all for harvesting similar noun representations from QGIS and others.

All,
What about setting them up in a right click menu.  You can still leave
the on/off toggle layer dependent this way, and there seems like there
might be some other things that could be added to a right click menu
over time.
bobb
On 03/23/2015 10:32 AM, theduckylittle wrote:

Right now all of the layer editing tools are up in the toolbar. This
makes them rather awkwards as there is a process of "activating" and
"deactivating" layers from the catalog and then needing to access the
editing tools in the toolbar.
I propose the following:

Remove the Layer Tools from the toolbar.
Allow users to select which layer controls they wish to have
displayed ONLY on editable layers.
If CONFIGURATION.services.disable_others is set then selecting a
layer tool will disable the toolbar until work on the layer has finished.
Change layer "activation" to a process of "unlocking" and "locking"
for editing purposes. Ensure there is a warning for unsaved changes
when the layer is "locked" that changes would not necessarily have
been saved.

Negative consequences:

It removes some of the "free roaming" nature of GeoMOOSE.
It will make some of the editing tools a bit less obvious as they
will be changed to small (though CSS stylable) icons under the layer
instead of larger, more readable menu items).

Positive consequences:

I think it makes the workflow more clear to users.
It allows administrators to configure which and how they want
layers to work.
The workflow is more inline with Desktop GIS systems. There is
widespread debate as to whether or not this is a benefit. Given
GeoMOOSE's target audience as of late (replacement for desktop GIS)
then I think it is still a reasonable target goal.
It allows us to remove the goofy drop-down menu which just burys
functionality anyway.

—
Reply to this email directly or view it on GitHub
https://github.com/geomoose/geomoose/issues/74.

No right click menus. Ever. They are the devils spawn of evil. Lots of reasons, here are some poignant ones:

Not everyone uses a mouse, much less one with two buttons.
The browser already wants to do useful things with that menu.
Right click menus hide important functionality from the user. If it is throw away or developer stuff, that's one thing, if its core usability, that should be exposed in an obvious fashion.

I didn't catch some of the nuances of this initially.  Here is an attempt to summarize and clarify this.
The way that the layer tools currently is that there are 0 or more WFS-T layers in the catalog and there is also a default, hidden, and activated layer that these tools automatically work on.  If you have 0 WFS-T layers then everything is simple and that is the end of the story.  If you have more than 0 WFS-T layers, then you have to activate (only one layer at a time) each layer you want these tools to work on.
This proposes to move the default, hidden, and activated layer to a visible layer in the catalog.  This will make it obvious that there is a layer that the non-WFS-T function of these tools work on.  It will also allow the toggling of visibility of this layer.  Also, the tools will move away from the top toolbar and be integrated into the layer (I think).  There will also be an optional configuration to disable the toolbar while using these tools (i.e. eliminate possibility of "tool confusion").
Both of the following functionalities would continue:

Freestyle on screen drawing and annotation resulting in in-browser vectors that show up in printing and are as temporary as the browser session (this would also have a corresponding layer in the catalog).
WFS-T editing tools used on screen to send data to some serverside datastore through WFS-T.

As a part of this ticket I discovered a few things:

We need more icons.
We are using CSS sprites and I need to update the sprite system slightly.
We were not properly attributing the Silk icons to famfamfam.
I found GeoSilk but had to patch together a different download location from the original announcement 5-years ago.

Revision master@7d4ba199a99ed209971f44e8d8e55aa3e46d0cbc attempts to right the licensing and attribution wrongs.  I'm going to take a whack at updating the sprite generation now.

I never fully understood why there was some duplication of icons in the all.png included with the demo. For one of our projects, I went in and swapped out some of the duplicates for other famfam items that were more usefult for the editing options.
http://www.coastalatlas.net/estuarymaps/images/all.png

:) Good news! It's better now. Less duplication.  I've got one more icon I want to clean up but then this is getting fairly ready for prime time.

This has been in the demo for a while now.  I've tested it this morning.  A longer term task would be adding some usability tutorials.  And I have an enhancement ticket to file for this.


fix(cmp): don't enable snippets source without nvim-snippets

Make it easier, so if users decide to disable nvim-snippets, they don't have to remove the snippets source manually in their config.

Thanks!


fix(simple_planning_simulator): fix bugprone-branch-clone

Description
This is a fix based on clang-tidy bugprone-branch-clone error.
The second error was suppressed due to the complexity of the condition.
Conditions can be summarized if necessary.
/home/emb4/autoware/autoware/src/universe/autoware.universe/simulator/simple_planning_simulator/src/simple_planning_simulator/simple_planning_simulator_core.cpp:588:49: error: repeated branch in conditional chain [bugprone-branch-clone,-warnings-as-errors]
      if constexpr (std::is_same_v<T, Control>) {
                                                ^
/home/emb4/autoware/autoware/src/universe/autoware.universe/simulator/simple_planning_simulator/src/simple_planning_simulator/simple_planning_simulator_core.cpp:590:8: note: end of the original
      } else if constexpr (std::is_same_v<T, ActuationCommandStamped>) {
       ^
/home/emb4/autoware/autoware/src/universe/autoware.universe/simulator/simple_planning_simulator/src/simple_planning_simulator/simple_planning_simulator_core.cpp:590:72: note: clone 1 starts here
      } else if constexpr (std::is_same_v<T, ActuationCommandStamped>) {
                                                                       ^
/home/emb4/autoware/autoware/src/universe/autoware.universe/simulator/simple_planning_simulator/src/simple_planning_simulator/simple_planning_simulator_core.cpp:643:63: error: repeated branch in conditional chain [bugprone-branch-clone,-warnings-as-errors]
    vehicle_model_type_ == VehicleModelType::DELAY_STEER_ACC) {
                                                              ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@soblin @maxime-clem @TakaHoribe @xtk8532704 @tkimura4 @zulfaqar-azmi-t4
Please confirm the PR. Thank you very much.


Heatmap Yaxis offset labels don't work

Bug report
HEATMAP: yaxis --> labels --> offsetY doesn't work for Heatmap with large amount of rows.

I would want to see the labels aligned with their respective rows
What is happening instead?
with a big amount of rows the labels start to lose the alignment
What error message are you getting?
none

It seems to me this doesn't work at all. I'm using it on Angular11 and the yaxis.labels.offsetY option does not do anything in my heatmap. I was able to work around this, tho, with the style.cssClass option and a global style with translateY(3px). I guess this was necessary because for Angular Material's Roboto font the labels were not correctly aligned by themself.

It is still reproduced.

@EduardChernyak I will reopen his issue since you reported this is still and issue for you.

It seems like the offset is still present (reproduction link in op). Its growing bigger as the series getting larger.
I tried to fix it here pr #4429, now ticks and labels consistent no matter how many rows there are in total.
However i'm slightly concerned about labels positions
Not that they used to be perfectly aligned - more like their baseline was ~2px lower than corresponding tick (quite noticeable with larger font), and now it's more like the baseline matches the tick perfectly, rendering labels ~2px higher.

nevermind, looks like parsing style attribute did the trick


[Feature]  Making FAQ more engaging and attractive.

Description
i want to make FAQ section more attractive by adding features like hover effect attractive font and much more.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done

Please assign this issue to me i would like to work on it #731

@swaraj-das I have raised PR regarding this issue please check


nvdiffrast: init at 0.3.0

...currently not precompiling the CUDA modules -> cpp_extension builds them at runtime; suprisingly, without running into any permission errors; at least on NixOS

Already in master https://github.com/SomeoneSerge/pkgs/commit/e00ec0476b92c94178ecff214f72cbabd57b62c3, also follow updates in https://github.com/SomeoneSerge/nvdiffrast/pull/1 later


Feature request: Improve the 404 page

I was discussing the 404 problem with @chenopis and realized that one of the best things we can do is give users a way to report 404s. I've seen this done before and I have some good ideas how we can make the 404 page more useful in general, too. I'm interested in working up a POC for this, but I won't "cookie-lick" this in case anyone else wants to take it on.
Broad ideas:

Give a big button people can click to generate a GH issue for a 404 in such a way that it captures the referring page and the page the user intended to get to. The resulting GH issue should have a normalized searchable title, so a kind soul can fix lots of 404s in a single PR.
Provide the website-wide search field plus maybe a "Did you mean" if we can figure out a way to do a fuzzy search based on the URL they were trying to hit.

Implementation will probably need to touch Hugo and maybe Javascript.
cc/ @chenopis

/kind feature