Does not exit or display any error when wrong arguments are provided.

I have a tool which should be invoked by mytool --foo, but when I invoke by mytool --foo --bar, it will just run as normal without any warning.
I don't know why, because I remember in previous version, I will get an error and exit, sometimes with suggestions like "Do you mean ....".
It's quite dangerous especially for optional arguments, because any typo will passed the check while the user just think he give correct arguments.

Can you post the source, or a link to the source? It could be some option set that's causing it.

Got it. If I enable the "AllowLeadingHyphen" setting, then check/suggest/exit does not work.

Ah, yeah the docs warn that, that particular setting will silence what would have otherwise been certain errors.
You can still manually validate those values using a Arg::validator, and produce the appropriate clap::Error if needed.
I'm going to close this, but let me know if you need any help with the solution! :smile:

any instruction/docs on my case?

You can still manually validate those values using a Arg::validator, and produce the appropriate clap::Error if needed.

my app settings is constructed by many yaml files. Is validator compatible with yml?
Can I disable "AllowLeadingHyphen", and use some escape to provide negative values, this seems to be the easiest solution?
Is it possible to provide: word with leading hypen allowed only when its previous word is a valid argument name?

@chenhouwu

any instruction/docs on my case?

See:

The docs page for Arg::validator (note, https://docs.rs is more up to date, but I can't link to that page due to firewall settings right now).
The example page for validators

For example, lets say you only want to support numbers, which might be negative (i.e. -20 or 20)
fn is_num(s: String) -> Result<(), String> {
    let maybe_num = s.trim_left_matches('-');
    if let Err(..) = maybe_num.parse::<u64>() {
        return Err(String::from("Not a valid number!"));
    }
    Ok(())
}

let m = App::new("test")
    .arg(Arg::with_name("num")
        .validator(is_num))
    .get_matches();

my app settings is constructed by many yaml files. Is validator compatible with yml?

Kind of. You can add all args in the YAML like normal, but have to add the arg using a validator manaully. i.e.
fn some_function(s: String) -> Result<(), String) {
    Ok(())
}

fn main() {
    let yml = load_yaml!("17_yaml.yml");
    let m = App::from_yaml(yml)
    .arg(Arg::with_name("needs_validator")
        .validator(some_function))
    .get_matches();
}

Can I disable AllowLeadingHyphen, and use some escape to provide negative values, this seems to be the easiest solution?

Yes. But it depends on the shell being used as to which escape codes must be used.

Is it possible to provide: word with leading hypen allowed only when its previous word is a valid argument name?

The best way to do that is with a Arg::validator. But that is in the same position as before :stuck_out_tongue_winking_eye:

Which one do you mean:

I can still use AllowLeadingHyphen, and then use the Arg::validator to validate the value.

Or:

I should disable AllowLeadingHyphen, and then use Arg::validator to enable for specific argument.

But without reading the clap codes, I think the leading hyphen check is a pre-check before the validator, so there is a paradox:

If AllowLeadingHyphen together with validator, then the wrong arguments checking (e.g. the --bar argument which should be presented) is missing.
If without AllowLeadingHyphen together with validator, then the negative value will be rejected even before passed to the validator.

Any misunderstanding?

Kind of. You can add all args in the YAML like normal, but have to add the arg using a validator manaully. i.e.

fn some_function(s: String) -> Result<(), String) {
    Ok(())
}

fn main() {
    let yml = load_yaml!("17_yaml.yml");
    let m = App::from_yaml(yml)
    .arg(Arg::with_name("needs_validator")
        .validator(some_function))
    .get_matches();
}

Do you mean:

You can add all other args in the YAML like normal?

If an arg is already in the YAML, then is it valid to add or update its settings from rust codes?

If an arg is already in the YAML, then is it valid to add or update its settings from rust codes?

That is correct. You can add all other args, besides the ones which need to be validated.

But without reading the clap codes, I think the leading hyphen check is a pre-check before the validator, so there is a paradox

You can't disable AllowLeadingHyphen if you want to accept "invalid" args (i.e. things like -20). But you're also correct in that if you do keep AllowLeadingHyphen you must validate (such as with a validator) all positional args that are part of that command (or subcommand). If the source you're working with is open, I can better answer a more specific and not abstract answer.
Sorry for any confusion :smile:

I have implemented a work around just as question #3:

Can I disable AllowLeadingHyphen, and use some escape to provide negative values, this seems to be the easiest solution?

I use n0.005 in cli to represent -0.005.

But I recommend to implement AllowNegativeNumber, any time the parse see a negative digit number, it will regard it as a number instead of an arg.

Interesting, I hadn't thought of adding AllowNegativeNumber. I'll play around with an idea and see if I can get a working solution. Thanks!

@chenhouwu try out v2.15.0 on crates.io and see if that works for your use case.

it works, great!


Return code handling for requests calls

Not all requests calls are raising errors (specifically RenderErrors) when exiting unsuccessfully.  One example is renderapi.stack.delete_section, but I am suspicious of all get and delete calls.

Russel and I discussed this the other day... plan is as follows.
step 1) make a standard wrapper for get, post, and delete.
step 2) raise RenderError when there is an exception in the http call, or when the http call returns an error code.


[UI Testing] Add gcloud commands to external doc & test UI for JDBCTOGCS template

gcloud commands and REST API to be added for GCSTOJDBC template
UI testing to be done for JDBCTOGCS template

Added commands for GCSTOJDBC template in Google Doc


[excel, outlook, powerpoint, word] (Script Lab) Update Script Lab documentation to omit gist sharing functions

This PR covers some potential upcoming changes to Script Lab.

@ElizabethSamuel-MSFT: This PR is now ready for review.

Tagging @wandyezj for awareness.


Add missing libraries to Alpine container

Hello,
This is a fix for issue #1 .
We just add the missing libraries to the container image.

Oh nice. Thank you!


Fix HuberLoss's backward computation

This PR fixes #1175 .

Modified HuberLoss's backward() so that it will compute correct gradients even when gy is not ones.
Modified its test case so that gy is set to random values.

Since backward() will start to raise an error when gy is None after this PR, some existing code might stop to work.

As the previous behavior is an implementation bug, my opinion is to fix as you did.

LGTM 👍


Feature: Allow user to add and display task in ToDo list

As a user,
I want to add task to ToDo list and display it on my UI.
ACC:

Default task added to database via UI,
UI refreshes and display newly added task,

Blocked by #28

Unblocked

Blocked - mockup needed

Unblocked - mockup added


WIP: Video block new embed

Replacing the semantic ui embed component with a custom one, to allow for more flexibility especially in regards to accessibility. This does mainly address issues raised in #5731.

@sneridagh should this new component be moved to the volto components package or shall it stay in there core package?

@jackahl LGTM, however, I'd put the component as general available component (and, eventually, transfer it to @plone/components, but that's for later, if Volto has to use it right away).


event broadcast

Where is the right place to hook into command processing in order to broadcast resulting events to other systems?

at the moment, i'm return the events in a promise here: https://github.com/yonahforst/easy-source/blob/master/lib/domain/to-class.js#L108 and then broadcasting via SNS in the then

Actually, since I'm using dynamoDB for the eventstorage, I'm just going to subscribe to the dynamostream and broadcast events from there.
I'm gonna leave this open since this library does need a way to hook into events and publish them


Relay pending packets in the relayer loop

Closes: #561
Description

added the new relay_pending_packets() that queries the chains for the unreceived packets and acks, builds and sends transactions. This function uses the exact methods used by the corresponding CLIs. It adds a retry around these.
changed relay_from_events() to perform relaying of pending packets after events subscription but before the first event is processed. The height for queries and client updates must be the height of the first event minus one.
noticed that the signer for msg timeout is wrong in master after my last commit and fixed that. We need to add a timeout test to the CI (#707)
also opened #708 for relayer loop CI

For contributor use:

[x] Updated the Unreleased section of CHANGELOG.md with the issue.
[ ] If applicable: Unit tests written, added test to CI.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

Merging #709 (b2d3d7f) into master (b1b37f5) will increase coverage by 30.2%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #709      +/-   ##
=========================================
+ Coverage    13.6%   43.8%   +30.2%
=========================================
  Files          69     150      +81
  Lines        3752    9821    +6069
  Branches     1374       0    -1374
=========================================
+ Hits          513    4309    +3796
- Misses       2618    5512    +2894
+ Partials      621       0     -621

Impacted Files
Coverage Δ

modules/src/address.rs
100.0% <ø> (ø)

...application/ics20_fungible_token_transfer/error.rs
0.0% <ø> (ø)

...ion/ics20_fungible_token_transfer/msgs/transfer.rs
0.0% <ø> (ø)

modules/src/events.rs
0.0% <ø> (ø)

modules/src/handler.rs
100.0% <ø> (ø)

modules/src/ics02_client/client_def.rs
48.3% <ø> (ø)

modules/src/ics02_client/client_type.rs
79.1% <ø> (+31.5%)
:arrow_up:

modules/src/ics02_client/context.rs
100.0% <ø> (ø)

modules/src/ics02_client/error.rs
100.0% <ø> (ø)

modules/src/ics02_client/events.rs
15.7% <ø> (+15.7%)
:arrow_up:

... and 255 more

Continue to review full report at Codecov.

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


eclipses.plugins: add custom builder

This builder simply adds two convenience functions called installFeatureJar and installPluginJar, respectively. These functions reduce the need of code duplication and makes buildEclipsePlugin correctly install plugin JAR that require unpacking.

@bjornfor, mind having a look at this?

I don't have very strong opinion, so if you want this merged, I can merge it. But you asked for review :-)

Good comments and exactly why I like to have a review :-)
I got the idea to use a custom builder from wanting to introduce the special install shell functions and otherwise have the build be a generic one. But for this simple case I think your solution is much cleaner so I switched to that one. I made the installHelpers variable "file local" since it is of fairly limited use outside buildEclipsePlugin and buildEclipseUpdateSite.

Ok :-)
But still the end result is about twice as many lines as before (ref. https://github.com/rycee/nixpkgs/commit/40eae632ea3377a9eeb959481ad867c840520aae). I'm not convinced :-)
But more importantly, you say in the commit message:
"These functions [...] makes buildEclipsePlugin correctly install
plugin JAR that require unpacking."
Is that the reason for the increase (bug fix)? Since I'm clearly not seeing what you're seeing, can you try fixing the bug first (in commit 1) and then reduce duplication (in commit 2)?

@rycee: What to do with this PR? Close or merge? (I still don't see the point, but I'm not that into the eclipse expressions as you are.)

Oops, I had completely forgotten about this! I'll close it since I don't have a need for this anymore but might revisit it in the future. Thanks!


ja: Make docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/tutorials/stateful-application/mysql-wordpress-persistent-volume.md
diff between translated and v1.18
https://gist.github.com/b098655f7e7e2409f4d6b6119e78a918
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/assign

/close


Global variables

Is there a way to set a global variable for use in every documentation file included? For example I'd like to have my api endpoint as a variable.

I experienced this as well and i did some looking around.
It looks like middleman had some problem with partials and different engines before, this was fixed in middleman/middleman#1338 and the fix is included since middleman v3.3.7.
I forked slate when it was locked to v3.3.6 so i just had to update middleman and it worked perfectly.

Hey I'm having an issue with this. I have set :host, 'https://example.com/api' in my config.rb and <%= host %> in my .md.erb file but when I try to load the docs, I get a NameError at / undefined local variable or method 'port' error.

@sean-hill not sure if this still works in middleman...try using a different variable than host, and if it still doesn't work, try this? https://middlemanapp.com/advanced/configuration

@lord yeah same issue even with a different name. I found a solution however! If I do config[:host] in my config.rb and then <%= config[:host] %> in my docs, it works perfectly!

:+1:

Is there any possibility that <%= config[:stuff] %> will be replaced by something more easy to write, like <%= stuff %> ?

@musikele We're not going to modify the default behavior, but you could add a custom helper to your Slate installation to at least shorten it to <%= c :stuff %>.

You may even be able to do some method_missing hack to get it down to just <%= stuff %>, but I'm not sure.

We have a scenario where we have 4 different environments ( QA, Test, Stage and Prod ). All these 4 environments have same template and information but different endpoints in the document. Is there a way there would be only one slate document and the document should display the endpoints based on the environment.
For example if I go to mytest.com the endpoint in the doc should show mytest.api.com
If I go to mystage.com the endpoint in the doc should show mystage.api.com
For now we have four different dos for 4 different environments but when we want to update similar information we had to update all the four docs.
Is there a way where we have only one doc and the endpoints in the doc change depending on the environment?
Please let me know if I am not clear. Thanks!

You guys can do  set :api_endpoint, 'https://fluffyendpoint.tld' in config.rb and in .erb files, you can access it via <%= config.api_endpoint %>


Rethink --pre-auto-entitlements (merge entitlement entries from provisioning profile)

Here's what electron-osx-sign does a little differently: Since Electron supports macOS features like Handoff, when app sandbox is enabled, electron-osx-sign (default with --pre-auto-entitlements) will automatically inject com.apple.developer.team-identifier into the entitlements to enable these features... and this entry must be whitelisted by a provisioning profile.
Therefore, electron-osx-sign requires provisioning profiles whenever app sandbox is enabled (applicable to MAS development and distribution). They will be picked up automatically if placed in the current working directory.
🤔 I think the this entitlements automation behavior will likely change in a next major release of electron-osx-sign so the presence of provisioning profile should be what adds in the whitelisted entitlement entries, instead of when app sandbox is enabled.

Originally posted by @sethlu in https://github.com/electron-userland/electron-builder/issues/897#issuecomment-487414713

I get the same issue. Is there any enhancement in the entitlements automation?

@jabetong1 I'm not sure which issue you're encountering, but a solution is discussed in the original comment https://github.com/electron-userland/electron-builder/issues/897#issuecomment-487414713


Neoray does not work with nvim 0.9.2+

nvim 0.9.2 lists as a breaking change:

An adjustment was made to the grid_line event as part of the external UI protocol:
the cells array might now end with a [' ', attr, 0] item with a repeat count of
zero. This is needed by the TUI to disambiguate final spaces from a line from just
clearing the line, which will make a difference when copying text using the terminal
emulators' builtin primary selection support.
External UIs can safely ignore such an empty item and most UIs already handle this fine.
But it could break UIs which has an assert to validate the cell count to be bigger
than zero, or similar.

I tried adding code similar to 468a3f30d8782708acb654c7b350d3ad609c293f, but noticed artifacts, for instance create a vertical split and then add new lines to the right-hand split, and I observed the separator to start getting chars overwritten with empty space.

Thanks for the report. Let me know if the b357fe4 fixes the issue.


/start showing " :x: Failed to initialize worker (websocket connection timeout)! See /help or try again later. "

Ok so , sometime ago i finished configuring the bot and i started it and it worked , i played with my friends for 2 hours and then i stopped the server using /stop now when i am using /start again its giving the following output :x: Failed to initialize worker (websocket connection timeout)! See /help or try again later.

This is what is shows in the logs

hi, is it working now? also btw how did u set up the proxy for your device? could u link me a docker tutorial by any chance?


Update omnibrowser.md

Hi David, I have update the content, please check! thank you so much!

Thanks will review!

@NoopDog and @lelewang111 this looks good to me - just one small change - the mailto: is not required see https://www.markdownguide.org/basic-syntax/#urls-and-email-addresses.
@NoopDog we need to make a ticket to fix the site-map OmniBrowser path.


Proposal: Better command ratelimits

Describe the proposal
The implementation of Command#cooldown is very effective, and it got enhanced thanks to Command#bucket allowed developers to have a better control of them without annoyingly set ratelimits at first command usage and disturb the user.
However, the underlying system feels incomplete and not very performant.

It uses a timeout for every single cooldown, this is separated in multiple commands, so both RAM and CPU usage seems to be affected by this. In practice, the excessive amount of timeouts seems to make other shift even a bit. This could be fixed with a cooldown sweeper using an interval, similar to Schedule's system.

It's not very effective: if a user spams a command, they should be notified only once, however, Klasa will reply as many times as it can. Some developers (me, included) use ratelimits to protect the bot from, not only heavy processing or limit the usage of commands, but also to protect the bot against breaking Discord ratelimits. That is, due to the lack of silent ratelimits.

Finally, there's an extreme side case that has been affecting my bot today: it's cool to put the ratelimits after a successful command run, however, there are commands (such the ones that send files or generates images) that take a while longer. These commands take more to resolve, therefore, they also take longer to fire the finalizers that put the user in cooldown. For example, I have a command that takes ~1 second to process an image and send it to Discord, the command has a cooldown of 30 seconds with a bucket of 1. A user sends 5 messages within that second.
Expected behavior: Run one, ratelimit the second, ignore the other three (with this proposal).
Actual behavior: As cooldowns aren't applied until the first successful command run, the bot does run all 5 commands.
Implementation

Cooldown Sweeper: Not implemented yet.
Silent Cooldown: Not implemented yet.

For your image commands, could you put the rest of the run method in a .then of the promise from fetching or generating your image, and return that promise immediately? That would run the finalizers more quickly.

Well, Ao suggested a slowdown inhibitor in the Discord Server

I suppose a slowdown inhibitor would fix that, too


aztools/Az.Tools.Migration/New-AzUpgradeModulePlan.md - -ToAzVersion wrong info

Prerequisites

[X] Existing Issue: Search the existing issues for this repository. If there is an issue that fits your needs do not file a new one. Subscribe, react, or comment on that issue instead.
[X] Descriptive Title: Write the title for this issue as a short synopsis. If possible, provide context. For example, "Typo in Get-Foo cmdlet" instead of "Typo."
[X] Verify Version: If there is a mismatch between documentation and the behavior on your system, ensure that the version you are using is the same as the documentation. Check this box if they match or the issue you are reporting is not version specific.

Links
https://github.com/MicrosoftDocs/azure-docs-powershell/blob/main/aztools/Az.Tools.Migration/New-AzUpgradeModulePlan.md
Summary
In the docs we say:
"-ToAzVersion
Specifies the Az module version to upgrade to. Currently, only Az version 10.3 is supported."
But on the samples we have -ToAzVersion latest everywhere and the latest version is the 11
https://www.powershellgallery.com/packages/Az/11.0.0
The doc should say something like " only Az version after 10.3 is supported, you can use latest for the latest version"
Details
In the docs we say:
"-ToAzVersion
Specifies the Az module version to upgrade to. Currently, only Az version 10.3 is supported."
But on the samples we have -ToAzVersion latest everywhere and the latest version is the 11
https://www.powershellgallery.com/packages/Az/11.0.0
The doc should say something like " only Az version after 10.3 is supported, you can use latest for the latest version"
Suggested Fix
The doc should say something like " only Az version after 10.3 is supported, you can use latest for the latest version"

@hribeiro-msft
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Adding @VeryEarly


RFC7643: Complex attribute 'addresses' does not declare subAttribute 'display'

Same as #72 but for adresses[].display

Also adresses[].value is not declared in the RFC. I think the discussion in the other issue linked above is the place to discuss more about that, to untangle the MultiValuedComplexAttributes.


Issue template is confusing

As organization template is used by default, the create issue page is really confusing. We could later copy TiKV issue template here.

@zhouqiang-cl it may affect all projects in TiKV.

Fixed in https://github.com/tikv/.github/commit/4855612d69110e5fea09f942bb65ff16124cce1e

Thanks! @hi-rustin


Fix stall on outbound TLS handshake

Summary
I observed outbound TLS handshake is intermittently stall on mater and 9.0.x (almost always on my local box).
When the stall happened, SSLNetVConnection::sslClientHandShakeEvent() was called twice but nothing happened after that until inactive timeout kicks in. These calls come from write_to_net_io().
Logs
ATS Debug
[Jan 19 09:43:42.921] [ET_NET 4] DEBUG: <HttpSM.cc:1773 (state_http_server_open)> (http) [0] [&HttpSM::state_http_server_open, NET_EVENT_OPEN/TS_EVENT_NET_CONNECT]
[Jan 19 09:43:42.923] [ET_NET 4] DEBUG: <Http1ServerSession.cc:81 (new_connection)> (http_ss) [0] session born, netvc 0x94d8b00
[Jan 19 09:43:42.923] [ET_NET 4] DEBUG: <HttpSM.cc:1824 (state_http_server_open)> (http) [0] setting handler for TCP handshake
[Jan 19 09:43:42.924] [ET_NET 4] DEBUG: <SSLNetVConnection.cc:1138 (sslStartHandShake)> (ssl) using SNI name 'localhost' for client handshake
[Jan 19 09:43:42.924] [ET_NET 4] DEBUG: <SSLNetVConnection.cc:1418 (sslClientHandShakeEvent)> (ssl) Initialize outbound connect curHook from NULL
[Jan 19 09:43:42.924] [ET_NET 4] DEBUG: <SSLNetVConnection.cc:1461 (sslClientHandShakeEvent)> (ssl.error) SSL_ERROR_WANT_READ
[Jan 19 09:43:42.924] [ET_NET 4] DEBUG: <SSLNetVConnection.cc:1418 (sslClientHandShakeEvent)> (ssl) Initialize outbound connect curHook from NULL
[Jan 19 09:43:42.924] [ET_NET 4] DEBUG: <SSLNetVConnection.cc:1461 (sslClientHandShakeEvent)> (ssl.error) SSL_ERROR_WANT_READ

Wireshark
29	8.599979	<IP_ADDRESS>	56211	<IP_ADDRESS>	8443	573	Client Hello
31	8.601103	<IP_ADDRESS>	8443	<IP_ADDRESS>	56211	1458	Server Hello, Change Cipher Spec, Application Data, Application Data, Application Data, Application Data

Route Cause
It looks like a2d15151a5c1a69a3826e086ac497a94a4bfa7ea made the read operation for TLS handshake disabled.
-  ua_entry->read_vio  = client_vc->do_io_read(this, 0, buffer_reader->mbuf);
+   //  hold onto enabling read until setup_client_read_request_header
+   ua_entry->read_vio  = client_vc->do_io_read(this, 0, nullptr);

https://github.com/apache/trafficserver/commit/a2d15151a5c1a69a3826e086ac497a94a4bfa7ea#diff-85a75d910dc93751ef5d568fc20fc73b01bfadac58844e88c7b8e68cedda0f0bL580-R584

We don't see the stall if the second SSLNetVConnection::sslClientHandShakeEvent() call from write_to_net_io() success. Definiately, it's race with Server Hello from origin server.

We don't see the stall if the second SSLNetVConnection::sslClientHandShakeEvent() call from write_to_net_io() success. Definiately, it's race with Server Hello from origin server.

Cherry-picked to v9.0.x branch.

Cherry-picked to v9.0.x branch.


Ability to have a customized view of Rounds on the dashboard and submissions tabs

The current listing for rounds displays closed or open rounds only, defaulting to closed. Many managers are actively working rounds that are both open and closed, meaning this default requires extra clicks between closed and open. Also, the rounds listing takes up a lot of space.
Proposed solution:

[ ] Ability to star rounds in both your dashboard and submissions page, open or closed
[ ] Add a new 'Starred' tab in addition to Open and Closed for both dashboard and submission pages
[ ] By default, if starred rounds exist, the starred tab is the one that displays in both dashboard and submission pages
[ ] On both pages, one can minimize or maximize the rounds listing and this will persist (save your personal setting) between sessions, ie if you roll it up and leave it that way, it will stay that way.

@danblah Is this really a priority for the next release? Its not clear to me how much PMs will actually use this in reality? Or am I missing something and most every ticket is getting a priority tag?


Reimplement ranlib in rust

Necessary for cross compiling, as using the native ranlib causes problems when cross compiling between systems using GNU and BSD style ar archives.

For BSD the format of __.SYMDEF (at begin of archive) is:
First length of ranlib structures (8 bytes)
Then pairs of string table offset and offset from begin of archive to start of archive file containing symbol (each pair 8 bytes total)
Finally length of string table (8 bytes)


How do i setParams to 2 tabs at once?

Current Behavior
It can dispatch setParams action for one tab only
Expected Behavior
I want to dispatch / setParams to all tabs
const setParamsAction1 = NavigationActions.setParams({ params: { foo: 'bar' } }, key: 'a' });
const setParamsAction2 = NavigationActions.setParams({ params: { foo: 'bar' } }, key: 'b' });
this.props.navigation.dispatch(setParamsAction1);
this.props.navigation.dispatch(setParamsAction2);

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!


Invert filter

@irinafakotakis Please fix the merge conflict with dev, thanks.

@Keeran10 fixed merge conflict.


Add runtime attribute for locked resources and allow multiple locks

Added runtime attribute support for functionality https://github.com/jenkinsci/lockable-resources-plugin/issues/302

Added attribute to resource Object

Changed logic to return resource if it contains the needed attribute even if it's locked

Changed build reference of Resource to list type List<Run> from Run

Changed locked by externalID of resource to List from String

Adjusted methods to work with lists

[X] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!

[X] Ensure that the pull request title represents the desired changelog entry

[X] Please describe what you did

[X] Link to relevant issues in GitHub or Jira

[X] Link to relevant pull requests, esp. upstream and downstream changes

[X] Ensure you have provided tests - that demonstrates feature works or fixes the issue

This seems to address a similar problem space as #299 (and #20 before it) - but for a somewhat different (or just more specific?) use-case that looks closer to agent/SUT deployment details than lockable resources generally.
Sanity-check: can this goal be implemented on top of that mechanism, to avoid adding vaguely similar fields? Or is it truly different enough?

@jimklimov . I think this PR tries to solve a different problem... The linked PR's - https://github.com/jenkinsci/lockable-resources-plugin/pull/299 (and https://github.com/jenkinsci/lockable-resources-plugin/pull/20 - add "static" attributes to the lockable resources that you can retrieve into env vars inside the lock block, to get specific and unchangeable values that belong to the given resource.
But this PR is about adding "dynamic" attributes to the resource that live until the/any lock is present.
Normally if a lock is present on a resource then all others jobs would need to wait until it's released even if they require basically the same.
So with this change multiple jobs could lock the same resource (if label and attributes are matching) and can run in parallel.
(I described the use case in https://github.com/jenkinsci/lockable-resources-plugin/issues/302)


TLF reset bug fixes

This along with https://github.com/keybase/kbfs-server/pull/64 fixes issues seen when integration testing in the staging environment.
Note: It's not obvious from the diff but when the API server returns unverified keys they aren't tagged as sibling or subkeys so I needed to stop filtering them as such (they all ended up as subkeys w/filtering.)

Looks good to me, just one suggestion, thanks!

Thanks!


Content Security Policy directive error

I have tried a bunch of security policies to connect to our api but always get refused, only on v4.
Refused to connect to 'https://myapi...' because it violates the following Content Security Policy directive: "connect-src 'self' https://myapi.../* 'unsafe-inline' 'unsafe-eval'".
export function setupContentSecurityPolicy(customScheme: string): void {
  session.defaultSession.webRequest.onHeadersReceived((details, callback) => {
    callback({
      responseHeaders: {
        ...details.responseHeaders,
        'Content-Security-Policy': [
          electronIsDev
            ? `default-src ${customScheme}://* https://myapi.../* 'unsafe-inline' devtools://* https://myapi.../* 'unsafe-eval' data: https://myapi.../*; script-src 'self' https://myapi.../* 'unsafe-inline' 'unsafe-eval'; connect-src 'self' https://myapi.../* 'unsafe-inline' 'unsafe-eval';`
            : `default-src ${customScheme}://* 'unsafe-inline' data:`
        ]
      }
    });
  });
}

v4.0.3
Chrome 94
Windows 10
React 17

Seems your building in release mode. Just add your https://myapi.../* to the release line after the ":"
works in my project.

is there a way to dynamically edit the policy? I will open up a new issue.


Remove allocations from the Map iterators

This PR does 2 things in the 2 underlying commits:

relaxes the bounds on the Map trait, applying the Clone and FromIterator in the only site where they are required
removes all the allocations related to iteration via helper wrapper iterators and extra helper methods

This removes all allocations temporarily introduced alongside Cow in the Map.

Why is this approach necessary? Is there no other way to do it upstream?

What does this approach remove in terms of allocations?

Why is this approach necessary?

The first commit is not necessary, but it should be helpful with the persistent storage impl.

Is there no other way to do it upstream?

If you're worried about the number of extra methods, they can be hidden from documentation etc. They would also not be needed if the related object members were not private.

What does this approach remove in terms of allocations?

In the diff you'll notice the removal of same calls to to_owned and collect, which would allocate in the persistent impl; this PR removes them all, making iterations almost as fast as in the in-memory impl (with key deserialization being the only perf penalty).

An additional bit of rationale:

with the storage being either in-memory or persistent and working on end types K and V (as opposed to some enum composed of an object or raw bytes), the former doesn't require deserialization during iteration, and the latter does
while in-memory objects can be iterated over by reference, the deserialized ones can't, as they are deserialized on demand during iteration and don't persist as K and V objects unless passed by value
Cow allows borrowed and owned objects to be handled together (including being iterated over), which is the natural solution here
the initial Cow implementation used some calls to collect and to_owned as a temporary workaround, but this meant that all the affected iterations incurred needless double (cloning + collecting) allocations, which slow them down
this PR removes them both: adding the into_ methods removes the needless clones, while adding the wrapper enums for iteration purposes removes the need for collecting
other than extra code there are no downsides to this solution

Rebased, and added one commit that followed the convention from the recent changes to Transactions.

Rebased.

Closing, as https://github.com/AleoHQ/snarkVM/pull/944 integrated it.


Fully support DISPLAY statement

I welcome you to pull my branch and make the changes you want- I am sure it won’t take you more than 5 minutes, and then I can have my aha moments ;)

I did the changes in your branch.
Previously, you had the OutputStatementNode class implement different statement interfaces (IWriteNode, IDisplayNode).
That would lead to the following confusion/bug:
If you write an analyzer for WRITE statements - lets pretend you want to do a "Don't use WRITE, use DISPLAY instead" diagnostic - then you will register the analyzer to analyze IWriteNodes.
The problem you'd have then, is that the statements that you get might not only be WRITE statements, but also DISPLAY statements, because OutputStatementNode implements both and is constructed for both.
With my change we have some empty classes and interfaces, which is strange and maybe not nice but accomplishes the distinction.
Now only WriteNode  implements IWriteNode  and only DisplayNode  implements IDisplayNode. Their implementation is identical (empty classes in this case, because they both extend OutputStatementNode), but the distinction is important :-)

@MarkusAmshove - looks great, so trick was to replace extends IStatementNode with IOutputStatementNode :)
I've now installed Java 21, and made a comparison. I get some new diagnostics: Need to support the DISPLAY options VERT and HORIZ. Should I implement this in consumeInputOutputOperand similar to TAB_SETTING etc? Or is there a better way? Really, we just don't want them to be seen as Identifiers.

@MarkusAmshove did you test this yet?

I pushed the changes but didn't test anything further, because the PR is still flagged as WIP :)

VERT AS works now :)


Creation fails as FSC package is deprecated

When I ran the installation it failed with that error:

Your request to install package "Financial Services Cloud r216.1.0" was unsuccessful. None of the data or setup information in your salesforce.com organization was affected.
If your install continues to fail, contact Salesforce CRM Support through your normal channels and provide the following information.
Organization: FSC Demo Pack (00D4E0000001XgJ)
User: User User (0054E000004iIsV)
Package: Financial Services Cloud (04t80000000jZjl)
Problem:

Package(03380000000U1CP) AppExchange Package is Deprecated
The AppExchange package has been deprecated and can no longer be installed.
Please try installing a newer version, or contact the package owner directly to resolve.

Please use https://github.com/Vchalem/fsc-demo-pack for latest updates.

Issue is that FSC package version needs to be updated when FSC product team pushes out a new package upgrade.


Constructor function changes for color and regExp

Changed constructor functions to lowercase.
Color -> color
RegExp -> regExp

For terminology, note that these are not constructor functions; they are not called with new.  The spec should probably read like "Colors are constructed with one of the following functions:", not "one of the following constructor functions:"

Code looks good.

OK, I'll update the spec.


[NFC] Scale Back Foundation Dependency

Remove SwiftSyntax's dependency on Foundation - there's nothing in this library that really needs that big a dependency.

@swift-ci test


bug: floats use last used hoverhints highlight

pre hover:

hover event:

post hover:

Could you specify your neovim version?

sure:
$ nvim.exe -V1 --version
NVIM v0.10.0-dev-d667e0e
Build type: RelWithDebInfo
LuaJIT 2.1.1700008891
Compilation: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe /DWIN32 /D_WINDOWS /MD /Zi /O2 /Ob1  -W3 -wd4311 -wd4146 -wd4003 -wd4715 -DUNIT_TESTING -D_CRT_SECURE_NO_WARNINGS -D_CRT_NONSTDC_NO_DEPRECATE -D_WIN32_WINNT=0x0602 -DMSWIN -DINCLUDE_GENERATED_DECLARATIONS -DNVIM_TS_HAS_SET_MAX_START_DEPTH -ID:/a/neovim/neovim/.deps/usr/include/luajit-2.1 -ID:/a/neovim/neovim/.deps/usr/include -ID:/a/neovim/neovim/build/src/nvim/auto -ID:/a/neovim/neovim/build/include -ID:/a/neovim/neovim/build/cmake.config -ID:/a/neovim/neovim/src

   file vimrc di sistema: "$VIM\sysinit.vim"
         $VIM di riserva: "C:/Program Files (x86)/nvim/share/nvim"

Run :checkhealth for more info

Ok, so I tested with the two configs you sent me. The first one doesn't work at all (it's as if the plugin is disabled), at least with only what you sent me. You shouldn't let lazy handle my plugin, my instructions never said you should experiment so much with it. The second one works (not all the time, but still) and doesn't reproduce your issue. Given this, I would assume the issue has something to do with your Neovim version. If you want my plugin to work as intended, you should use Neovim's stable v0.9.2 or v0.9.4 (the ones I'm testing on) and strictly follow my readme instructions (with the exception of adding mousemoveevent to the init property of lazy, which seems like a good idea and I might add it myself). I'm going to leave this issue open for another day and await for some more comments from your end. After that, if we don't reach any kind of solution, I'm going to close it. However, if you manage to reproduce this with the stable versions I just talked about, you are welcome to recreate this issue and I can get to work. As of now, trying to fix this with an unstable version of Neovim is just not a wise option. Thank you.

Hi!
I meet this issue too. It seems to be related to value 0 passed as first parameter to the nvim_set_hl() call at this line.
Nvim documentation for this parameter:
      • {ns_id}  Namespace id for this highlight |nvim_create_namespace()|.
                 Use 0 to set a highlight group globally |:highlight|.
                 Highlights from non-global namespaces are not active by
                 default, use |nvim_set_hl_ns()| or |nvim_win_set_hl_ns()| to
                 activate them.

As specified, using 0 set the FloatBorder highlight group globally. Doing so overrides the existing FloatBorder highlight group and impacts other plugins using this global group.
Could you use a non-global group for Eagle, or just use the existing global group without overriding it?

@gbcreation Hi, I don't have enough time to look into this right now, but if you think changing this value fixes this issue without causing other problems I could add this as a config option.


Unsure if commands are working?

Hello,
I'm pretty new to DevTools protocol, I'm having some issues understanding if what I want to happen is happening. Just wondering if I could get some help:
Below is a small code sample where I am trying to set the userAgent and throttle the CPU.
const chromeLauncher = require('chrome-launcher');
const CDP = require('chrome-remote-interface');

function launchChrome(headless = false) {
  return chromeLauncher.launch({
    chromeFlags: [
      '--disable-gpu',
      headless ? '--headless' : ''
    ]
  });
}

const NEXUS5X_USERAGENT = {
  userAgent: 'Mozilla/5.0 (Linux; Android 6.0.1; Nexus 5 Build/MRA58N) AppleWebKit/537.36' +
    '(KHTML, like Gecko) Chrome/61.0.3116.0 Mobile Safari/537.36'
};

async function init() {
  try {
    const chrome = await launchChrome();
    const client = await CDP({ port: chrome.port });
    const { Network, Emulation, Page } = client;

    Network.requestWillBeSent(params => {
      console.log(params.request.url);
    });

    Network.setUserAgentOverride(NEXUS5X_USERAGENT);
    Emulation.setCPUThrottlingRate({ rate: 4 });

    await Promise.all([Network.enable(), Page.enable()]);
    await Page.navigate({ url: 'http://localhost:3000' });
    await Page.loadEventFired();

    const version = await CDP.Version({ port: chrome.port });
    console.log(version);

    // chrome.kill();
  } catch (err) {
    console.error(err);
  }
}

init();

So when I log the version:
const version = await CDP.Version({ port: chrome.port });

It is never updated, and when I run:
 navigator.userAgent

Within the browser it also has not changed? The CPU throttling has also not been configured.
Am I doing something stupid?
Any help would be awesome, thanks.

Hello! First thing first you should await all the methods returning a Promise in order to ensure the proper order and to catch errors:
await Network.setUserAgentOverride(NEXUS5X_USERAGENT);
await Emulation.setCPUThrottlingRate({ rate: 4 });

The user agent override does work and you can check it with:
await Runtime.evaluate({expression: 'navigator.userAgent'});

(Or even better by navigating to https://httpbin.org/user-agent.)
The override scope is the current session so you will not see the difference using CDP.Version, and for the same reason you don't see it changed when you manually evaluate navigator.userAgent in the console.
Hope it helps!

Hey @cyrus-and this is awesome thanks, the scope point was super useful. I have this working now, one last question - sorry :)
If I have an async function running in my browser, is there anyway to get the Runtime evaluate to await that until it resolves? I assume not because I'm in a different scope, so would the best way to just be keep polling until complete?
Thanks

If I have an async function running in my browser, is there anyway to get the Runtime evaluate to await that until it resolves? I assume not because I'm in a different scope, so would the best way to just be keep polling until complete?

There is! And it's very useful actually (see Runtime.evaluate):
await Runtime.evaluate({
    awaitPromise: true,
    expression: `new Promise((fulfill, reject) => {
        setTimeout(fulfill, 1000);
    })`
});

amazing! Thanks so much, I've got it all working 😄

Great, you're welcome!


Added basic site wide open graph and meta tags

What kind of PR is this?:

/kind chore
What this PR does / why we need it:
Partial fix of #39

Codecov Report

Merging #90 (ca67e8c) into main (4a18ecf) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
+ Coverage   84.30%   84.48%   +0.18%
==========================================
  Files          14       14
  Lines         172      174       +2
  Branches       27       27
==========================================
+ Hits          145      147       +2
  Misses         27       27

Impacted Files
Coverage Δ

src/layouts/Default.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Permission denied error

When I trying to train by myself, I encounter the permission denied error.
This is my log:
2021-04-04 14:14:39,944 - [INFO]:  progress: 0%, start
2021-04-04 14:14:39,944 - [INFO]:  progress: 1%, start train
2021-04-04 14:14:40,788 - [INFO]:  no GPU, will use [CPU]
2021-04-04 14:14:41,598 - [INFO]:  input_shape: (224, 224, 3)
2021-04-04 14:14:41,885 - [INFO]:  load datasets complete, check pass, images num:376, bboxes num:376
2021-04-04 14:14:41,967 - [INFO]:  train, labels:['up', 'mid', 'down']
2021-04-04 14:14:41,967 - [DEBUG]:  train, datasets dir:/work/si_xuan_yi/maix_train/out/datasets/datasets
2021-04-04 14:14:41,968 - [INFO]:  bboxes num: 376, first bbox: [0.30803571 0.13839286]
2021-04-04 14:14:41,997 - [INFO]:  bbox accuracy(IOU): 93.09%
2021-04-04 14:14:41,997 - [INFO]:  bound boxes: (72.000000,28.00),(73.000000,35.00),(67.000000,31.00),(66.000000,38.00),(65.000000,34.00)
2021-04-04 14:14:41,998 - [INFO]:  anchors: [2.25, 0.875, 2.28125, 1.09375, 2.09375, 0.96875, 2.0625, 1.1875, 2.03125, 1.0625]
2021-04-04 14:14:41,998 - [INFO]:  w/h ratios: [1.74, 1.91, 2.09, 2.16, 2.57]
2021-04-04 14:14:42,540 - [INFO]:  train start
2021-04-04 14:14:42,540 - [INFO]:  progress: 1.0%, train start
2021-04-04 14:14:42,556 - [INFO]:  epoch 0 start
2021-04-04 14:15:19,458 - [INFO]:  epoch 0 end: {'loss': 0.22109940648078918, 'val_loss': 0.10796400904655457, 'lr': 1e-04}
2021-04-04 14:15:19,458 - [INFO]:  progress: 49.5%, train epoch end
2021-04-04 14:15:19,529 - [INFO]:  epoch 1 start
2021-04-04 14:15:54,535 - [INFO]:  epoch 1 end: {'loss': 0.044014301151037216, 'val_loss': 0.08665819466114044, 'lr': 1e-04}
2021-04-04 14:15:54,536 - [INFO]:  progress: 98.0%, train epoch end
2021-04-04 14:15:54,610 - [INFO]:  train end
2021-04-04 14:15:54,610 - [INFO]:  progress: 98.0%, train end
2021-04-04 14:16:05,124 - [INFO]:  train ok, now generate report
2021-04-04 14:16:05,124 - [INFO]:  generate report image
2021-04-04 14:16:05,279 - [INFO]:  generate report image end
2021-04-04 14:16:05,279 - [INFO]:  now generate kmodel
2021-04-04 14:16:17,460 - [ERROR]:  failed: TrainFailReason.ERROR_INTERNAL, node error:[Errno 13] Permission denied: '/work/si_xuan_yi/maix_train/instance/../tools/ncc/ncc_v0.1/ncc'

chmod +x /work/si_xuan_yi/maix_train/instance/../tools/ncc/ncc_v0.1/ncc

你好 我也遇到了这个问题 chmod+x 还是出现了权限限制，请问是怎么解决的呢

Hi, I have same problem.
anyone can solved it issue?
I try this command but can't find my need results (have .kmodel model format in out folder)

chmod +x /work/si_xuan_yi/maix_train/instance/../tools/ncc/ncc_v0.1/ncc

please help me 🙏


ActionView::Template::Error (undefined method `edit_query_path'

When saving a query ,I get error like  :
ActionView::Template::Error (undefined method `edit_query_path' for #<#<Class:0x007fa55c348470>:0x007fa564738140>):
    3:     h1
    4:       = query.name
    5:       .pull-right
    6:         = link_to [:edit, query], class: 'btn btn-default btn-sm' do
    7:           i.fa.fa-pencil.fa-pad-r
    8:           | Edit
    9:       .pull-right

actionpack (<IP_ADDRESS>) lib/action_dispatch/routing/polymorphic_routes.rb:216:in `polymorphic_method'
actionpack (<IP_ADDRESS>) lib/action_dispatch/routing/polymorphic_routes.rb:130:in `polymorphic_path'
actionview (<IP_ADDRESS>) lib/action_view/routing_url_for.rb:102:in `url_for'
actionview (<IP_ADDRESS>) lib/action_view/helpers/url_helper.rb:196:in `link_to'

It could not be reproduced by my environment.
Does include edit_query path in result of rake routes?
following log is my application. (Rails<IP_ADDRESS>)
$ be rake routes | grep query
query_executions POST   /q/:query_id/executions(.:format)     adhoq/executions#create
query_execution GET    /q/:query_id/executions/:id(.:format) adhoq/executions#show
edit_query GET    /q/:id/edit(.:format)                 adhoq/queries#edit
query GET    /q/:id(.:format)                      adhoq/queries#show

I've got same error on my API only mode application.
The API-only modes is application wide config.
https://github.com/rails/rails/issues/24264#issuecomment-218218511
It affects to default_actions, edit and new routes are not defined in API only app.
https://github.com/rails/rails/blob/v5.1.1/actionpack/lib/action_dispatch/routing/mapper.rb#L1128,L1134
Here is the reproduce code:
begin
  require "bundler/inline"
rescue LoadError => e
  $stderr.puts "Bundler version 1.10 or later is required. Please update your Bundler"
  raise e
end

gemfile(true) do
  source "https://rubygems.org"
  # Activate the gem you are reporting the issue against.
  gem "rails", "5.1.1"
  gem "adhoq"
  gem "sqlite3"
end

require "rack/test"
require "action_controller/railtie"
require "active_record/railtie"
require "adhoq"

class TestApp < Rails::Application
  config.root = __dir__
  config.session_store :cookie_store, key: "cookie_store_key"
  secrets.secret_token    = "secret_token"
  secrets.secret_key_base = "secret_key_base"

  # NOTE commented out to pass the test
  config.api_only = true

  config.logger = Logger.new($stdout)
  Rails.logger  = config.logger
end

TestApp.initialize!

class ApplicationController < ActionController::Base; end

TestApp.routes.draw do
  mount Adhoq::Engine => '/adhoq'
end

require "minitest/autorun"

ActiveRecord::Schema.define do

  create_table "adhoq_executions", force: :cascade do |t|
    t.integer  "query_id",                            null: false
    t.text     "raw_sql",                             null: false
    t.string   "report_format",                       null: false
    t.string   "status",        default: "requested", null: false
    t.text     "log"
    t.datetime "created_at",                          null: false
    t.datetime "updated_at",                          null: false
  end

  create_table "adhoq_queries", force: :cascade do |t|
    t.string   "name"
    t.string   "description"
    t.text     "query"
    t.datetime "created_at",  null: false
    t.datetime "updated_at",  null: false
  end

  create_table "adhoq_reports", force: :cascade do |t|
    t.integer  "execution_id", null: false
    t.string   "identifier",   null: false
    t.time     "generated_at", null: false
    t.string   "storage",      null: false
    t.datetime "created_at",   null: false
    t.datetime "updated_at",   null: false
  end

  add_index "adhoq_reports", ["execution_id"], name: "index_adhoq_reports_on_execution_id"

  create_table "no_id_tables", id: false, force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "secret_tables", force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

end

# Ensure backward compatibility with Minitest 4
Minitest::Test = MiniTest::Unit::TestCase unless defined?(Minitest::Test)

class BugTest < Minitest::Test
  include Rack::Test::Methods

  def setup
    Adhoq::Query.create!(
      name:        'A query',
      description: 'Simple simple SELECT',
      query:       'SELECT 1'
    )
  end

  def test_returns_success
    get "/adhoq/q"
    assert last_response.ok?
  end

  private
    def app
      Rails.application
    end
end

@wzymaster @hanachin It will fixt. https://github.com/esminc/adhoq/pull/142
I passed hanachin's reproduce code.

#142 merged.

I tried master branch after #142 merged, the edit_path error fixed in my environment! thank you <3


Add metadiff semanticdb tool

This PR adds a tool called metadiff that takes two directories of SemanticDB and produces a diff comparing them. Just plaintext diffing the source of text documents is way too slow, so I make a best effort to speed up the diffing by doing semantic diffs wherever possible. The order diffs can slow it a down a lot, so the --no-orderflag ignores order of symbol information and occurrences. Diffs for symbol information, occurrences, and synthetics can also be turned off with flags.
I am currently actively using this tool, and it seems like it could be useful for the future as part of the SemanticDB toolchain ecosystem. I have not made any tests for this, because I'm not sure how to test it.

We're cleaning up our issue and pull request queue in preparation for 4.0.0-RC1, so I'll be closing this PR. Please feel free to rebase and resubmit once you're ready - I'm still very much in favor of the idea of metadiff.


Create user roles

Create user roles for API user, API administrator and system administrator and document their privileges
Definition of done

[x] Create roles corresponding to our Personas' user roles
[x] Create document describing roles and expected privileges
[ ] Define user roles in code, possibly on Startup
[x] Decide how system administrators will manage user roles
[ ] Add roles field to user administration interface

@JuusoV, check out the following learning resources demonstrating the alanning:roles package:

Adrian Lanning: Authorization with Meteor-Roles -- Devshop 5 Tech Talk

The alanning:roles API documentation indicates there is a [createRole](http://alanning.github.io/meteor-roles/classes/Roles.html#method_createRole) method. We may be able to create the roles on Meteor.startup() using Roles.createRole('role-name') one or more times.

It could also be possible to verify the roles before creating them, to avoid creating duplicates, by using the Roles.getAllRoles() method and checking each returned role for a duplicate.

Also, refer to the Best Practices section of the roles demonstration video, where Adrian Lanning explains conventions for creating roles. This may influence how we implement roles in our project.

Roles.createRole('role-name') works under Meteor.startup(). The created roles shown in the console with Roles.getAllRoles().find().fetch()

@JuusoV please review the similar pattern for avoiding duplicates in the ApiUmbrellaAdmins collection. You will have to take the general pattern and modify it to work with your task.

I got the roles working but I think we should not impelement them in the startup because the roles are only created once. Do you agree?

Where should we implement them?
On 18 June 2015 17:02:00 EEST, Juuso Vallius<EMAIL_ADDRESS>wrote:

I got the roles working but I think we should not impelement them in
the startup because the roles are only created once. Do you agree?

Reply to this email directly or view it on GitHub:
https://github.com/apinf/api-umbrella-dashboard/issues/105#issuecomment-113166154

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@JuusoV consider the Add Roles Collection to Admin Dashboard. This is a simple and flexible recipe to add a 'Roles' button to the admin dashboard, where we can easily manage the Meteor.roles collection.

Thanks for the help and support @brylie.
Task is now done and PR is created.
Estimate: 0


Is it possible to add -h, -v to --help, --version?

I would like to make my app behave consistently and add the short variants ala, -v, --version and -h, --help to trigger the built-in AutoVersion and AutoHelp; is this possible? How would one accomplish that?

I've written #608 to implement this by adding new AutoHelpShortName and AutoVersionShortName settings, which default to false. If you set them to true, then -h will be the shortname for --help and -V will be the shortname for --version. Note that that's uppercase V for consistency with almost all software that I've seen (because -v with a lowercase v is usually the shortname for --verbose).

#608 has been replaced by #686, which has the same feature but has been rebased on top of the current state of the develop branch.


compare with relu

Just updated 4 hours ago, you are really amazing!
Have you tried to make some comparisons with relu on some practical tasks?

In the original paper they have comparisons with other activation functions, but it's very focused on the modeling aspect. I tried image classification on imagenette (small subset of the full imagenet) using a xResnet18 architecture with both ReLu and Siren activations. I found that Siren enables training with higher learning rates, but requires stronger regularization and don't match the accuracy achieved with ReLu (87.4% ReLu vs 83.8% Siren). I will not take these numbers as conclusive, as batchnorm doesn't seem to play nice with Siren (when I tried to add it to the image fitting experiment strange artifacts appeared)

Sample of image fitting using the model with batchnorm, notice the strange patterns that are not present in the original model:


Add support for KitKat (API level 19)

Motivation
Provide workaround for Android bug on Kitkat (api level 19) to grantUriPermission to allow the camera to save photos
Test Plan
Built locally and tested on an Android KitKat device.

@aarontrank I have a device running KitKat and I'm using your fix, but my app continues to crash. :(  I'm assuming that was the behaviour you were experiencing, right?

@rahimrahman can you provide crash logs?

@aarontrank I've looked at the crash log, and it was actually due to OutOfMemoryError
E/AndroidRuntime( 5691): java.lang.OutOfMemoryError

@rahimrahman gotcha.  Sounds like a different issue entirely.

So can this be closed or is it still needed?

@marcshilling still needed to address this issue https://code.google.com/p/android/issues/detail?id=76683 for KitKat


ExportSDF

Would it be possible to export the predicted sdf directly ?

Hi @tiwylli!
You can simply export the volume used for marching cubes.
Go to source/poco_utils.py, line 96 (marching cubes call). Add this line before and you'll get the volume exported as a numpy array:
np.save('your/path/to/volume.npy', volume)

Note that the volume will be a bit different than the mesh because the cleaning is omitted.
Also, the volumes are very inefficient and will be quite large on the hard drive.
Does this solve your issue?

Thank you so much for your answer.
Yes I figured the volume was what I needed. I'm working on a project with input pointclouds and wanted to estimate an SDF, but from what I understand, it's easier to use a mesh as input to estimate an SDF.
Have a great weekend


Add elasticache serverless dimentions and metrics

AWS announced the availability of ElastiCache Serverless offering on their catalog back in Nov 2023 [Amazon ElastiCache Serverless for Redis and Memcached is now available]. This new offering uses a new dimension named clusterId and added a couple of new metrics [Serverless cache metrics]. This PR adds support to this new dimension and these new metrics so that they are available on Grafana CloudWatch data source as well. Here is a sample of the new ElastiCacheProcessingUnits metric:
        {
            "Namespace": "AWS/ElastiCache",
            "MetricName": "ElastiCacheProcessingUnits",
            "Dimensions": [
                {
                    "Name": "clusterId",
                    "Value": "my-redis-cluster"
                }
            ]
        },

Please note that these new metrics are under the same namespace, AWS/ElastiCache as the traditional non-serverless metrics.

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


Set line spacing does not display properly in word

Generate 1.5x line spacing with docx library，but displayed with larger line spacing than in office：

Line spacing generated by ms office：

Found the problem, need to turn off this word settings:


OHIF-332: Users should be able to see all available hotkeys and language settings in one place

This design is a reference design, and not necessarily indicative of which hotkeys should be set, what their labels should be, or what keys they should be bound to.
The modal's title should be "User Preferences"
Should support a "disabled" state that:
Disables all "shortcut fields
Disables the "Save" and "Restore Defaults" buttons
The modal should consist of two sections
General
Contains a dropdown to change the active i18n language for the application
Hotkeys
Contains a two column list of functions and shortcuts
Function names are right aligned
"Setting" a shotcut key combination should reuse the logic from the previous major version of OHIF
Function display names and default shortcut bindings should use the mode configuration's values
The mode's hotkey configuration may require updates
Action buttons
Restore defaults should set all "shortcut" fields to the mode's default values
Cancel (and the model's X to dismiss) should clear the form's temporary values and cause no changes
Save should apply the form's values as the new bindings/settings
PR Checklist

[x] Brief description of changes
[x] Links to any relevant issues
[ ] Required status checks are passing
[ ] User cases if changes impact the user's experience
[ ] @mention a maintainer to request a review

Window level presets aren't in the designs, the designs are on the MGH Jira, could you update your PR to match these please?

The "Function" header should be right aligned, it is currently center aligned. Check the designs.

Note these new shortcut requirements from OHIF-62:

Home: Skip to first image of series in active viewport
End: Skip to last image of series in active viewport

If you turn off "disabled" to test:

Editing a value correctly shows a message that there is a clash, but you can still save it, saving the settings should be disabled in this case. Currently you can save it, and then both say the hotkey when you re-open the menu, but only the most recently changed one works.
The preferences don't persist when you switch from one study to another by navigating the study list. Even if we don't have persistence for now, the settings should persist within a single viewer session if you navigate to the same mode with another study? Do we not care about this as we are disabling it by default for now? cc @swederik  @dannyrb .

@JamesAPetts fixed the disabled state, removed the presets. I didn't touch the persistence inside the mode, I'll wait for their answers since this functionality is currently disabled although it works.

Modals:

Viewer route: About Modal is empty
List route: User Preferences Modal is empty

Design:
I don't think @JamesAPetts realized that your UI was generated from the hotkeys config. The window level preset hotkeys should still be hotkeys, they just shouldn't be user editable or visible in the panel (for the user). I think in an old PR we created a "display name" field for hotkeys. If that's null or undefined, we could filter them out when displaying the fields.

Modals:

Viewer route: About Modal is empty
List route: User Preferences Modal is empty

Design:
I don't think @JamesAPetts realized that your UI was generated from the hotkeys config. The window level preset hotkeys should still be hotkeys, they just shouldn't be user editable or visible in the panel (for the user). I think in an old PR we created a "display name" field for hotkeys. If that's null or undefined, we could filter them out when displaying the fields.

@dannyrb I have added the requested changes. Also, I have extracted the header component so we can reuse it, we had 2 dropdowns with the same logic before.

No hotkey settings are visible from list page


download filename issues with extension if spaces exist (some browsers)

At least on FF and maybe other browsers, a download link from https://lbry.tv/@Deterrence-Dispensed:2/VZ61SkorpionARFCG:d stripes off everything after the first space, so you end up with a file without an extension. Maybe we can just trim spaces.

This should be fixed now!


WIP: feat(live-query): pause polling when the window is inactive

Closes https://github.com/graphprotocol/graph-client/issues/354
Tests are needed.

@schmidsi can you please take a look and see if it solves the issue? (you can try the alpha releases in the comment above)

Wow, that was quick. Works well on my side.

Question: I think we need to document that behaviour or even make it configurable so users could turn it on and off. Like:
plugins:
  - pollingLive:
      defaultInterval: 1000
      defaultPauseOnInactive: true # maybe there is a better variable name

And
query ExampleQuery @live(pauseInInactive: false) {
  transactions(first: 2, orderBy: timestamp, orderDirection: desc) {
    id
  }
}

@ardatan, can we move forward with this? I'd like to write about query optimisations soon and this would be a great feature to shill Graph Client more.


please update your docker build

latest docker build on docker hub is 4.20, which is 4 four month behind.
please update the build!
loving your woks, guys

https://hub.docker.com/r/v2fly/v2fly-core


Medicatins missing

https://github.com/clif-consortium/CLIF/blob/main/mCIDE/clif_medication_admin_continuous_med_categories.csv
Reviewing our current medication list, we've identified that sodium chloride (NaCl) is included (line 65: sodium chlorideothers). However, we've noticed that several other common resuscitation fluids are not present. These include:

Lactated Ringer's solution
Albumin infusion
Plasma-Lyte
Dextrose in water (D5W) ?

@kaveriC update the medication category list

Added the above four meds to the med categories list under fluid_electrolytes med_group


fix: Big Query Edit Form

SUMMARY

With new DatabaseModal we aren't able to edit, users are getting a fatal error due to missing encrypted_extra when building the uri
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@eschutho @AAfghahi @lyndsiWilliams

🏷 2021.21


ENH: Clip T2* values at 100ms to keep consistent histogram axes

Closes #2780.

@mgxd Any thoughts?

Only affects ds210, not waiting for ds005 to finish.


Bug: Fixed Navbar Duplication on All Pages

Description:
This PR fixes the Navbar Duplication Issue on all pages,earlier when hamburger menu was opened on smaller screen and the screen width was increased without closing the hamburger menu,it was not closing on its own,causing duplication of navbar
Issue Mentioned:
Closes #413
Video:
https://github.com/mdazfar2/HelpOps-Hub/assets/72603662/13d34d90-3174-4964-bd52-496a90b8bf54
Additional Context:
N/A

@mdazfar2 kindly review and merge
thankyou

@shreyash3087 Can you show me the difference between before and after?

Yeah one min

https://github.com/mdazfar2/HelpOps-Hub/assets/72603662/4afc5227-c05a-4d8c-a7ef-e1cd07d79cd2
Notice How on the main website if i open the hamburger menu and increase the screen width the navbar doesnt close and it creates two navbar,i fixed it as you can see it automatically closes if screen width is increased,so two navbar doesnt appear on the website

@mdazfar2 if you have any more queries you can ask

@shreyash3087 Could you add some hover effects on the navbar and some buttons to make them look attractive and impressive according to our website's look? If you think the navbar needs it, please create a PR; otherwise, there might be no need. Please see according to your convenience. If needed, add some nice effects on the navbar and buttons like "Star Us," "Sponsor," and more.

@mdazfar2 I can add this animation for the navbar,or if you have any specific effect that you would like me to add,please do tell
https://github.com/mdazfar2/HelpOps-Hub/assets/72603662/b2ee0b88-e988-4059-9b78-8f13c9afbd71

@shreyash3087 maybe it looks good

also try to add some effect on some buttons like sponsor star us

@mdazfar2 yeah sure would think something for the sponser and star us button,in the meantime what do you think of this effect for the footer links-

https://github.com/mdazfar2/HelpOps-Hub/assets/72603662/d559e871-828f-4560-85df-15fa71ea2c4d

@shreyash3087 it's good but what about sponsor button and star us button

@mdazfar2 yes im looking for some good animations,ill inform if i get something good

@shreyash3087 indeed

did you find anything? @shreyash3087

@mdazfar2 Nah nothing looks up to the mark for the sponser button,i think we should leave it for now,ill message you on discord if i get something creative.

@shreyash3087 ok Create pr of everything you have analyzed before raising issue.


Java 11 precommits runs continuously on Apache Beam

[UPDATE]
Current status:

compile with 8
run with 11

Depends on:

#60
#61
#62


Zend\Paginator\Adapter\DbSelect MySQL error

I wrote about this issue. Recent changes in the Zend\Paginator\Adapter\DbSeletst lead to an error in some cases.
Query:
SELECT *, column FROM table
good work, but query:
SELECT COUNT(1) AS c FROM ( SELECT *, column FROM table ) AS original_select
will cause an MySQL error: "Duplicate column name 'property' SQL."
Is normal behavior for code is broken into pieces?..

You can solve this issue by passing a third argument to the join method, an array with an alias for the join table, so for an example, if you have an "id" column on both tables, you can do something like this:
$select->join(array('b' => 'bar'),
                    'a.foo = b.baz',
                    array('b.id' => 'id')
);

@galvao thanks it worked.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Limit tensorflow version to less than 2.0.0.

Thank you for awesome code and resources.
I have noticed that some codes may not support tensorflow2 in this repository.
I propose to limit tensorflow version to less than 2.0.0.
Thank you.

@ysekky
That's a nice point.
Thanks for your nice contribution.


When using ADMIN_APP_PATH, under certain circumstances admin login redirects to homepage

Hi,
I changed my project's admin URL from using a subdomain to using a path. There's a small issue when logging in, where the user gets redirected to the homepage instead of seeing the Twill dashboard page under specific circumstances (see below). The problem does not occur when using a subdomain-based admin URL.
This is somewhat confusing to explain- I'll do my best!
Let's say my env variables are as follows:
ADMIN_APP_URL=myproject.com
ADMIN_APP_PATH=my-admin-panel
Let's say I'm logged out. I visit http://myproject.com/my-admin-panel, it redirects to http://myproject.com/my-admin-panel/login, I see the login form and am able to login correctly (goes to Twill admin page).
However, if I go straight to http://myproject.com/my-admin-panel/login, without the redirection from http://myproject.com/my-admin-panel, and login (eg if I was to logout and then immediately log back in again), I get redirected to my project's homepage at http://myproject.com. However if I then visit http://myproject.com/my-admin-panel I get the Twill admin page, so I am obviously being logged in even though I've not ended up at the right place.
This is a bit confusing for my project's users, so I'd appreciate it if you could have a look at this. As mentioned above, there is no problem when using subdomain-based admin URLs via ADMIN_APP_URL only.
Thanks!
Seb

Hi Seb!
Using the following option in your twill.php configuration file would solve this issue:
 'auth_login_redirect_path' => '/admin',

We could automate this by reading from the ADMIN_APP_PATH when provided, but in the meantime the above should work for you.

Yes, works perfectly now- thanks again!

Please add this to your documentation, I think many people will likely want to test twill on custom ports/sub directories and this is one of the issue that comes up when you do.


qemu: add pipewire support (8.1 feature)

Description of changes
Enable PipeWire audio backend -audiodev pipewire support, available since qemu 8.1 (#251154).
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Caused a infinite recursion in cross builds which use the emulator, fix in https://github.com/NixOS/nixpkgs/pull/253913


EAD integration needs to be able to detect and handle descending X values

The descending X values in graphical curves will currently not work with the EAD integration as it does not know to step through in reverse order. The following code, beginning on line 207 in the PairedData class, should correct the issue:
        public double integrate()
        {
            Boolean descending = false;
            if (Xvals.First() > Xvals.Last()) {
                descending = true;
            }

            double triangle;
            double square;
            double x1 = 1.0;
            if (descending)
            {
                x1 = 0;
            }
            double y1 = 0.0;
            double ead = 0.0;
            for (int i = 0; i < Xvals.Length; i++)
            {
                double xdelta = Xvals[i] - x1;
                square = xdelta * y1;
                triangle = ((xdelta) * (Yvals[i] - y1)) / 2.0;
                ead += square + triangle;
                x1 = Xvals[i];
                y1 = Yvals[i];
            }
            if (descending && x1 != 0)
            {
                ead *= -1;
                double xdelta = x1;
                ead += xdelta * y1;
            } else if (x1 != 1)
            {
                double xdelta = 1 - x1;
                ead += xdelta * y1;
            }
            return ead;
        }

What do we think about converting curve types to all be the same? I feel like simplicity in the computational code (vs. validation logic, construction) is a priority.

@AlexRyanUSACE I like your solution. It is clean, concise, and easy to read. My preference would still be to run all calculations on non-exceeedance probability unless otherwise noted/for good reason. I can see things getting messy handling both measures of probability. I get confused switching back and forth between exceedance and non-exceedance.
The damage frequency function is developed either from an analytical frequency function or a graphical frequency function. The PairedData analytical function is created from BootstrapToPairedData, a method that produces nonexceedance probabilities. The SamplePairedData method of the Graphical object now produces a PairedData object that also consists of nonexceedance probabilities.  I think we're good, then. What do you think?

Yeah, i'm sold on the simplicity of just converting both to non-exceedance. That's a lot cleaner and still provides the same flexibility.

Great! Thank you.


BCF not yet supported

i'm getting following error while running freebayes using cannoli
my command -------
ADAM_MAIN=org.bdgenomics.cannoli.Cannoli adam-submit --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar -- freebayes -freebayes_reference hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa hdfs://master.hdp:8020/opt/small5.adam hdfs://master.hdp:8020/opt/sample.genotypes.adam
the error ------ Exception in thread "main" java.lang.AssertionError: assertion failed: BCF not yet supported

Ah, sorry, the example I gave on gitter is incorrect.
As of the current version of Cannoli, freebayes only supports writing out as VCF.
https://github.com/bigdatagenomics/cannoli/blob/master/src/main/scala/org/bdgenomics/cannoli/Freebayes.scala#L106
$ ADAM_MAIN=org.bdgenomics.cannoli.Cannoli \
  adam-submit \
  --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar \
  -- \
  freebayes \
  -freebayes_reference hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa \
  hdfs://master.hdp:8020/opt/small5.adam \
  hdfs://master.hdp:8020/opt/sample.vcf

okay

i have used the above command you posted, but i'm  getting this error
htsjdk.tribble.TribbleException$InvalidHeader: Your input file has a malformed header: We never saw the required CHROM header line (starting with one #) for the input VCF file

I believe the tribble exception may be misleading. The could not open hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa is likely more relevant.
$ freebayes -f does-not-exist.fa
Please specify a BAM file or files.

$ echo $?
1

when i use --  freebayes -f hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa , shows that
"-f" is not a valid option, but i can find -f in --help (freebayes --help in cannoli), i think -f option not work in freebayes in cannoli.
moreover i never have a bam file , i'm looking in a way that using adam file.

There is no -f in freebayes in cannoli, it should be -freebayes_reference VAL.
This maps to -f when cannoli calls freebayes externally via the pipe API.

Hmmm oke...i tried out again with the following command , but the error is the same!
ADAM_MAIN=org.bdgenomics.cannoli.Cannoli adam-submit  --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar -- freebayes -freebayes_reference hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa   hdfs://master.hdp:8020/opt/small5.adam   hdfs://master.hdp:8020/opt/sample91.vcf
and getting output as two files, that is
0 sized .vcf file and vcf_head file

header file contains header info only. how should i get ride of this issue?

What is the error?

i'm getting the following error
htsjdk.tribble.TribbleException$InvalidHeader: Your input file has a malformed header: We never saw the required CHROM header line (starting with one #) for the input VCF file

Let's start from the beginning, What was the command you input?

i had used this command
 ADAM_MAIN=org.bdgenomics.cannoli.Cannoli adam-submit  --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar -- freebayes -freebayes_reference hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa   hdfs://master.hdp:8020/sample/salim3.adam   hdfs://master.hdp:8020/vcfoutput/sample206.vcf

I'm not sure if a reference on hdfs will work. Could you try using the test reference and see what happens?

Right, the -freebayes_reference argument value hdfs://master.hdp:8020/Data/HumanBase/hg19/hg19.fa is passed to the freebayes executable, which likely doesn't know how to deal with hdfs:// scheme URLs.

okey. i was trying to access reference genome from locally on cluster , not from hdfs.
 ADAM_MAIN=org.bdgenomics.cannoli.Cannoli adam-submit  --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar -- freebayes -freebayes_reference /opt/artificial.fa   hdfs://master.hdp:8020/sample/salim3.adam  -single  hdfs://master.hdp:8020/vcfoutput/sample217.vcf
here

artificial.fa is size of 1.15kb

i got this  reference genome from adam resources and now i could see this error

the job doesn't completes
getting this output when i was abort the job.

@heuermh we should add the ability to use references stored in HDFS. I think this is a separate issue from what's been discussed here, so I've added #50 to track this.

Can you try without the "single" flag?

ya sure

@a1xt06   getting this output when i was trying without the "single" flag

and the command which i used to run
 ADAM_MAIN=org.bdgenomics.cannoli.Cannoli adam-submit  --jars /opt/cannoli/target/cannoli_2.10-0.1-SNAPSHOT.jar -- freebayes -freebayes_reference file:///Data/HumanBase/hg19/hg19.fa  hdfs://master.hdp:8020/sample/salim3.adam   hdfs://master.hdp:8020/vcfoutput/sample229.vcf
also the error is still same as we discussed earlier

@shibuvp Freebayes is complaining that it cannot open your reference file.  could not open file:///Data/HumanBase/hg19/hg19.fa
The error above looks to be due to missing or invalid RecordGroups in the original input BAM file. NoSuchElementException: key not found: NewFile
Perhaps it would be worthwhile to try running freebayes directly on a single node with your data.
$ freebayes -f hg19.fa salim3.bam > salim3.vcf

Then if that succeeds, try running the same via Cannoli.

ok @heuermh i w'll try

and the issue is , i'm trying to using adam file rather than bam , is it possible in freebyes?


feat: enrich request log message

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 96.034%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
863

Relevant Lines:
910

💛  - Coveralls


gem_package resource broken and behaves like chef_gem in Ubuntu 22

Description
In the latest version of Chef Infra Client on Ubuntu 22, the gem_package resource is broken. It behaves the same as chef_gem, i.e., it installs gems into Chef's embedded ruby environment and not the system environment.
Chef Version
Chef Infra Client 18.2.7
Platform Version
Ubuntu 22.04
Replication Case
gem_package 'example' do
  version '1.0.0'
end

After chef converges, verify gem has not been installed in the system ruby environment:
gem list | grep example

Instead, gem is installed in chef's embedded ruby:
# /opt/chef/embedded/bin/gem list | grep example
example (1.0.0)

Client Output
Recipe: example::default
  * gem_package[example] action install
    - install version 1.0.0 of package example
...
Infra Phase complete, 1/5 resources updated in 10 seconds

Additional details
Using execsnoop we can see that under the hood chef is running the following:
# execsnoop | grep example
sh<PHONE_NUMBER>98   0 /bin/sh -c /usr/bin/gem install example -q --no-document -v "1.0.0"
gem<PHONE_NUMBER>98   0 /usr/bin/gem install example -q --no-document -v 1.0.0
ruby<PHONE_NUMBER>98   0 /opt/chef/embedded/bin/ruby /usr/bin/gem install example -q --no-document -v 1.0.0

In the ruby apt package for ubuntu 22, it creates /usr/bin/gem with an env shebang #!/usr/bin/env ruby. In Ubuntu 20, it used a hardcoded shebang #!/usr/bin/ruby2.7.
This new shebang means the returned value of gem env gempath will be different, depending on whether or not it is run from chef:
Chef:
# /opt/chef/embedded/bin/ruby /usr/bin/gem env gempath
/root/.local/share/gem/ruby/3.1.0:/opt/cinc/embedded/lib/ruby/gems/3.1.0

System:
# /usr/bin/ruby /opt/chef/embedded/bin/gem env gempath
/root/.local/share/gem/ruby/3.0.0:/var/lib/gems/3.0.0:/usr/local/lib/ruby/gems/3.0.0:/usr/lib/ruby/gems/3.0.0:/usr/lib/x86_64-linux-gnu/ruby/gems/3.0.0:/usr/share/rubygems-integration/3.0.0:/usr/share/rubygems-integration/all:/usr/lib/x86_64-linux-gnu/rubygems-integration/3.0.0

This is likely the root cause of https://github.com/chef/chef/issues/12879
Stacktrace
No stacktrace; chef believes it installed the gem successfully.

Same behavior on Debian 11 and up.
Here's the minimal cookbook with inspec test to check behaviour.

@drdev and others facing this problem... here's another workaround (just modify for Debian accordingly).
You can monkey patch the gem_package resource by throwing this in <cookbook>/libraries/gem_package_patch.rb
I've only tested on Chef Client 14 and Cinc Infra Client 18, but everything seems to work as expected now.
# Author:: Adam Jacob<EMAIL_ADDRESS># Copyright:: Copyright (c) Chef Software Inc.
# License:: Apache License, Version 2.0
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

class Chef
  class Resource
    class GemPackage < Chef::Resource::Package
      property :gem_binary, String, desired_state: false, default: lazy { default_gem_binary }

      def default_gem_binary
        if node['platform'] == 'ubuntu' && node['platform_version'].to_f >= 22.04 && ::File.exist?('/usr/bin/ruby')
          '/usr/bin/ruby /usr/bin/gem'
        else
          'gem'
        end
      end
    end
  end
end

Hey all,
We chatted about this in the PR Review meeting this week. There's two possible solutions:

We can add code similar to the above to set a default gem_binary in the GemPackage resource (but we'd need to do it differently, more on that below). If we do that then both gem_package and chef_gem will both provide defaults which will render most of the code in rubygems.rb#initialize dead and we should clean it up - it'll be about 3 lines when we're done (the first if). If we do this we should add copious comments to rubygems.rb since it'll be very unclear why it all works when viewing the provider.
We can drop the defaults from both GemPackage and ChefGem resources, and then update rubygems.rb#initialize to do all the logic there. If we go this route, I'd change the flow to be if ChefGem ...  elsif GemPackage ... else raise end to make this all MUCH clearer.

Regardless of which way we go, the way we determine if we can trust /usr/bin/gem needs to be more resilient and future proof. For example, my Debian machine's /usr/bin/gem is an env version. So something along the lines of:
if File.exist?('/usr/bin/gem') && File.open('/usr/bin/gem', &:gets).include?('bin/env')
  '/usr/bin/ruby /usr/bin/gem'
else
  'gem'
end

Finally, we'll want to massively update the tests. We'll want one set of tests with a context of omnibus  one of not omnibus, in each of which we test both chef_gem and gem_package where /usr/bin/gem is and is not the env variety.
Getting back to solution # 1 vs # 2... arguably chef_gem shouldn't HAVE a gem_binary option to it, that defeats the whole purpose. We're currently implicitly making chef_gem chef_gem with a path which seems odd to me. So I'd rather we nuke all the default-calculations from the resources, and make the provider very clear. But it's a mild preference, I'm OK with either.

Drew's code above (https://github.com/chef/chef/issues/13754#issuecomment-1688757588)  is a workaround.
Loading it as usual library in custom client cookbook, it makes it possible to override original GemPackage resource defaults on-the-fly, without changing original chef-client files.
#2 is definitely more preferable, because normally you expect all the logic in provider code, like for all other resources.

have there been any updates on this? Still currently experiencing these pain points

Me too.  Looking at applying the monkey patch.  Thanks for including.

I had to update the monkey patch code to use it in 18.3.0
class Chef
  class Provider
    class Package
      class Rubygems < Chef::Provider::Package
        def gem_binary_path
          if platform?('ubuntu') && node['platform_version'].to_f >= 22.04 && ::File.exist?('/usr/bin/ruby')
            '/usr/bin/ruby /usr/bin/gem'
          else
            'gem'
          end
        end
      end
    end
  end
end


Local CORS

yarn create single-spa
yarn start --port 8500
Go to   http://single-spa-playground.org/playground/instant-test?name=@spa/spaReact&url=8500

You need to configure CORS policy of the webpack development server. Please see https://stackoverflow.com/questions/31602697/webpack-dev-server-cors-issue.


Fix encoding and decoding of MySQL enums in sqlx::Type

fixes #1241

As it turns out, we didn't actually have a comprehensive test covering enum encoding and decoding for MySQL, so it's not clear if it ever actually worked correctly. Because this PR was opened without context and the MySQL internal documentation is garbage, it took writing and running the test to actually understand the situation. (The C API documentation but also lists MYSQL_TYPE_ENUM as corresponding to ENUM fields just above it, so that's not much help.)
The problem is that how MySQL interprets MYSQL_TYPE_ENUM appears to have changed over time. In 5.7, it accepts string inputs with the type MYSQL_TYPE_ENUM and binds them correctly. MySQL 8.4, on the other hand, doesn't like it and throws an error:
1210 (HY000): Incorrect arguments to mysqld_stmt_execute

I suspect it's expecting the integer encoding that's mainly used internally. Or it just doesn't like it altogether; I'm not really sure and I don't want to get into it.
Both 5.7 and 8 return columns with MYSQL_TYPE_STRING and the ENUM flag set, so I'm not really sure what we were thinking when the code was originally written. This is why it's important to have real tests for things.
Your change did result in the correct behavior, with one exception: MySQL doesn't appear to care about the BINARY flag with enums and doesn't set it on returned columns, so MySqlTypeInfo::__enum() wasn't actually being considered compatible with what MySQL was returning. Removing the BINARY flag fixed it.
I've taken the liberty of pushing my changes. Since it's still your PR, you'll get the credit in the CHANGELOG.

@abonander
Thanks for the merge.
Thank you for your dedication and commitment.
I also did a local test and should have shared it with you. Sorry about that.
I love this crate with all my heart.

@abonander
Thanks for the merge and your dedication and commitment.
I also did a local test and should have shared it with you. I am sorry.
I love this library with all my heart.


Add the resolvedData field into the generated schema

Motivation
The resolvedData field  generated for richtext element is not part of the schema definition generated by createTypeNodesSchema() from typeNodesSchema.js.
Proposed solution
The goal of this issue is to add the mentioned field into the generated schema, write test(s), and update documentation.

Seems like the field is there already. It's also in the ReadMe. Is there anything else missing, @makma ?

You are correct @CollierCZ! Closing the issue.


[Bug][Worker] get worker status bug

Purpose of the pull request

We should compare 1 - memoryUsagePercentage and reservedMemory, rather memoryUsagePercentage and reservedMemory.

Brief change log

Verify this pull request

cc @ruanwenjun

The bug is from #13896

Hi, @ruanwenjun , i want to make sure this configuration is worker.max.cpuload.avg, or should be worker.max.cpu.load.avg, if it's wrong either, i can change the doc and log description together.
  # worker max cpuload avg, only higher than the system cpu load average, worker server can be dispatched tasks. default value 1: will use 100% cpu.
  max-cpu-load-avg: 1

Hi, @ruanwenjun , i want to make sure this configuration is worker.max.cpuload.avg, or should be worker.max.cpu.load.avg, if it's wrong either, i can change the doc and log description together.
  # worker max cpuload avg, only higher than the system cpu load average, worker server can be dispatched tasks. default value 1: will use 100% cpu.
  max-cpu-load-avg: 1

max.cpu.load.avg is better, and it should work.

Codecov Report

Merging #13992 (e5b3c9e) into dev (d617620) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

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

@@             Coverage Diff              @@
##                dev   #13992      +/-   ##
============================================
- Coverage     38.89%   38.88%   -0.01%
+ Complexity     4474     4465       -9
============================================
  Files          1164     1164
  Lines         42446    42408      -38
  Branches       4757     4736      -21
============================================
- Hits          16508    16491      -17
+ Misses        24118    24101      -17
+ Partials       1820     1816       -4

Impacted Files
Coverage Δ

...eduler/server/worker/task/WorkerHeartBeatTask.java
80.43% <50.00%> (ø)

... and 31 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add getAlumniUser() function

Feature

[x] Install GitBash if using Windows
[x] Follow README.md of shpe-uci-portal to Install & Run Python web app using Terminal or GitBash
[x] Create a new GitHub branch based on master & call it alumni-filter-functions
[ ] Add a getAlumniUser() in shpe-uci-portal/app/routes/search.py file (similar to getAllUsers() func)
[ ] Ensure that function prints users to the console when called
[ ] Git add, commit, & push any changes to origin/alumni-filter-functions
[ ] Open a Pull-request for a code review anytime

Notes

Reach out @Globins for GitHub Branching clarifications
Reach out @josueal1 for Firebase questions

Shadowee

None

Given our call above sever-side and client-side rendering, I will close this issue and place your efforts into the dynamic front end web development portion @dhsalazar11


Suggestion: Add a priceticker to the main screen

Hello,
just thought about how Raspiblitz would look like with an additionally added price-ticker screen.
For example you can run it like a screensaver, switching between the standard screen and the priceticker.
Would look really nice in my opinion / would be an eyecatcher.
Thought about looking something similar to this: https://steemit.com/cryptocurrency/@cryptotaylor/raspberry-pi-cryptocurrency-ticker
What do you think? Would it be much effort to include it? Individualization of the settings maybe over the main menu?

That woudl be a nice idea, but at the moment we try to reduce dependencies on central information services as much as possible for default features .. and for price information you always need to contact a central service. So this could be a add-on feature that can be switched-on in the future as a contribution - but its not high on my personal prio list. The staus screen might also be update in the future with more menu options (touch, buttons, epaper, ..) at that point price ticker could be someting to add by user choice.

pls close this. Already reponded on other issue


BSD example

BSD example tested on FreeBSD 12.1 and NanoPi NEO board.

@kni ,
Thank you for the example!


[BUG] Azure.Monitor.OpenTelemetry.Exporter EventId duplicate key exception causes LogRecord to be dropped

Library name and version
Azure.Monitor.OpenTelemetry.Exporter 1.3.0
Describe the bug
When a LogRecord has an EventId specified and the LogRecord AttributeData dictionary already contains a key EventId then the LogsHelper.GetMessageAndSetProperties method causes a DuplicateKey exception. Which eventually leads to the whole LogRecord getting dropped.
This problem also occurs with attribute key "EventName".
In my project I am using Serilog. Serilog adds (if set on the log entry) EventId and EventName to the log state which causes it to end up in the LogRecord AttributeData dictionary. The LogsHelper then without checking whether it exists adds EventId again to the AttributeData dictionary causing an exception.
https://github.com/Azure/azure-sdk-for-net/blob/Azure.Monitor.OpenTelemetry.Exporter_1.3.0/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/LogsHelper.cs
Expected behavior
When the attribute dictionary of the LogRecord already contains EventId or EventName then the LogsHelper doesnt try add it again.
Actual behavior
LogsHelper causes a duplicate key exception and the LogRecord is dropped never gets exported.
Reproduction Steps

Environment
No response

https://github.com/Azure/azure-sdk-for-net/blob/Azure.Monitor.OpenTelemetry.Exporter_1.3.0/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/LogsHelper.cs#L112 There are places where we de-dup, but eventid/name are not in that path :(