Fewer dependencies on kbfsfuse

My use case: I never use the Keybase GUI, I use the keybase command-line tool rarely on Linux, and I never use KBFS.
My problem: Some of the keybase commands (such as keybase chat ls and keybase git create) need a working kbfsfuse running, so I have to run kbfsfuse ~/kbfs in another terminal window first to use these commands. I don't want this. I'd prefer that these commands worked without even having a kbfsfuse binary on my system.
Rationale 1: I don't want services lingering out on my system, not even temporarily, and I don't want the inconvenience of starting and stopping those services manually whenever I need something.
Rationale 2: I don't want to store many large, bloated, unnecessary binaries on my system. kbfsfuse on Linux amd64 is 46 MB. I think that much of its functionality could be integrated to the keybase tool, and it wouldn't become much larger.

It wouldn’t be hard to make a shell script that spins up the kbfsfuse, performs the command, and kills kbfsfuse by storing its PID.
Reworking the keybase binary to do all that would require a large re-write.
I don’t agree it would be worth it.

Thanks for the insights! As a software engineer not familiar with the Keybase code, maybe there is an easy way to implement this: linking the binaries together (renaming the main function in kbfsfuse), and doing a go KbfsFuseMain(...) call when needed. Probably some functionality (e.g. the actual FUSE filesystem) can be omitted, and only the services offered on kbfsd.sock will be kept.
I agree that a shell script which I can write for myself would be helpful. It's just a workaround though, with different tradeoffs (e.g. it still needs the 46 MB of code bloat of kbfsfuse).

@pts: the current logic for keying chat conversations lives in kbfsfuse, for legacy reasons (it was originally implemented for KBFS, and it was faster for us to share the code that way).  And kbfsfuse generally needs to be a separate binary from keybase because it controls the /keybase mountpoint for the file system -- you wouldn't want restarted the keybase service to unmount your directory and render all your files useless.
That said, we do combine them into one binary for the mobile app (which doesn't have a mountpoint), so it's possible.  I don't have a link handy but I think it's in a bind subdirectory in our client codebase.
Also, in the future, we are moving to put chat keying into the keybase binary, so the dependency for the keybase chat commands will go away for new conversations.  keybase git is unlikely to change, since that depends on KBFS file functionality.


[RHIDP-1783] Document how to mount the RBAC policy CSV file using the Operator

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s):

Issue:

https://issues.redhat.com/browse/RHIDP-1783
Link to docs preview:

https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-499/authorization/#_mounting-policy-csv-file-using-the-developer-hub-operator
Reviews:

[ ] SME: @ mention assignee
[ ] QE: @ mention assignee
[ ] Docs review: @ mention assignee
[ ] Additional review: @mention assignee (by writer)

Additional information:

/cc @hmanwani-rh
Please feel free to reassign it to other folks.
It would be great if this could go into 1.3.x..

/cherry-pick release-1.3
(Just so we don't forget)

/cherry-pick release-1.3

@rm3l Could you please manually cherry-pick this PR to the release-1.3 branch?

@rm3l Could you please manually cherry-pick this PR to the release-1.3 branch?

@hmanwani-rh This seems to have already been cherry-picked into release-1.3 and merged by @themr0c - see https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/pull/533


Not able to call pageMenuController(_ pageMenuController: PageMenuController, didSelectMenuItem index: Int, direction: PageMenuNavigationDirection)

I am using this PageMenuController. And i have on table view on did select method i used notification center to trigger this delegate method.But not able to pass the int value:
@objc func overAllSelect(notification: NSNotification) {
   if let val = notification.userInfo?["index"] as? Int {

       pageMenuController(_pageMenuController: PageMenuController, didSelectMenuItem index: val, direction: PageMenuNavigationDirection)
       }
   }

@sathishchinniah pageMenuController(_ pageMenuController: PageMenuController, didSelectMenuItem index: Int, direction: PageMenuNavigationDirection). It's a callback method. I don't recommend you to call this method.
this method is called when you select the menu item.


Change empty result from View Command

if name cannot be found in view command, app now remains in list mode instead of displaying an empty list

Codecov Report

Merging #182 (87602af) into master (bbf1438) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #182      +/-   ##
============================================
+ Coverage     64.79%   64.80%   +0.01%
  Complexity      672      672
============================================
  Files           108      108
  Lines          2505     2506       +1
  Branches        280      280
============================================
+ Hits           1623     1624       +1
  Misses          782      782
  Partials        100      100

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/ViewCommand.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

ur too fast


Bind with React and Preact

Hey there.
First of all, thanks for a great work on this library.
So I created a library to bind Sockette to React and Preact at the same time. I wanted to reduce the hassle of switching the component from the two library. the idea just lands in creating a component factory.
You can check it out here.
I was wondering, is there any chance we can merge it with your version of preact-sockette?
This way both React and Preact users can enjoy using this component. and Yes, I am aware of compat libraries :) I just think creating a factory for components is a better way since it can also be used in other alternatives such as Inferno too.
and probably a good idea that all the components that we create for these libraries, be written as factories.

Hey~! I actually like preact-sockette the way it is, sorry 😕
Setting up aliases for Preact vs React vs Inferno is a one-time configuration that applies globally to a project. AKA, you'd pay more of a cost re-initializing a bunch of factories at runtime, rather than just setting up the proper aliases at build-time.
My component is also basic enough that it already works in any lib that claims to be React-like. I also think that the ws key belongs in context, not state.
You're more than welcome to continue development on your component -- I'm sure others prefer the factory-approach for their projects, too!
Thank you so much though! Happy to see adoption like this 😃


Send - Max privacy flow

Send screens https://app.zeplin.io/project/5c73dc8aa74b4b4085a86a91/dashboard?seid=5f8d8d429ca8de6081c4976a

Offline address - https://zpl.io/bzrDvZl

Max privacy address - https://zpl.io/2jDBenm

Permanent - https://zpl.io/VOez9ZY

One-time use - https://zpl.io/agjMyPQ

Rename "Transaction comment" to "Comment"

Rename "show address" to "more details"

Send to contact. First address type, below contact https://zpl.io/a3LY7Gx

Please add descriptions for all empty fields (address, amount, comment) https://zpl.io/VYey6QJ Please display it from the beginning, not only after tap on field

checked

checked


chore: bdd-spring-1573539442 to 0.0.1

chore: Promote bdd-spring-1573539442 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


refactor: internalization budget module

Description
PR to internalize tendermint/budget module, which was used after import, into this repository to minimize risk and maintenance effort due to differences in sdk and tendermint versions

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[ ] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Don't we need to write upgrade handler logic to migrate old budgets to the new module?


Remove redundant get in calcBaseInGivenOutDT

Tests are failing because of a bug in provider, it's safe to merge


Generator: Update SDK /services/serverupdate

Generated from GitHub run<PHONE_NUMBER>8

there is a newer PR with the same changes https://github.com/stackitcloud/stackit-sdk-go/pull/1018


[Question] Why is the first request always longer than the rest

Hello,
I have written a simple code (nodejs) as below
async function test() {
    console.log(moment().format('D MMM HH:mm:ss.SSS'), "=============>START");
    for (let index = 0; index < LOOP; index++) {
        await ioredis.get("ioredis:string");
    }
    console.log(moment().format('D MMM HH:mm:ss.SSS'), "=============>DONE", found);
}
setInterval(() => {
    test()
}, 1000);

Output:
22 Apr 21:37:13.009 =============>START
22 Apr 21:37:13.050 =============>DONE         (41ms)
22 Apr 21:37:15.011 =============>START
22 Apr 21:37:15.032 =============>DONE         (21ms)
22 Apr 21:37:17.018 =============>START
22 Apr 21:37:17.046 =============>DONE         (28ms)
22 Apr 21:37:19.023 =============>START
22 Apr 21:37:19.048 =============>DONE         (25ms)
I see, the result between LOOP = 1 and LOOP = 100 does not differ much.
And I've noticed that the first request always takes 15-20ms, the following requests are very fast.
Can anyone explain to me why this is happening?
And how to make every request have response time <1ms?
Thanks everyone.

Isn't this about Javascript engine optimizing things while running? In all benchmarks first step always takes biggest time.
Also for the sub millisecond query, did you try caching?


Reanimated 1.3.0 Node values are updated when they shouldn't

Hello there.
As I mentioned in #415, after 1.3.0 upgrade my animation node values are updated even when another node value (in different component) is changed (and no state change is made). Whats worse, even when I push a new screen and then go back, origins screen nodes values are changed. But all these problems disappears when I remove diffClamp method.
It happens only on 1.3.0. When I downgraded back to 1.2.0 the issue dissapears.
Gif - Behvaiour on 1.3.0:

Gif - Behvaiour on 1.2.0 or without using diffClamp method:

I'm enclosing reproducible demo https://github.com/sebinq/react-native-reanimated-bug
Steps to reproduce:

At HomeScreen, move any object just a little bit to any side.
Change Screen using provided button.
Go back to HomeScreen and you will see position is changed!

You can also see debug log in console that translation value was changed.
When you remove diffClamp in 'components/Movable.js' and try to use just withOffset, you can see that everything works fine!
Thank you so much for any help!
react-native 0.61.2
react-native-reanimated: 1.3.0
react-native-gesture-handler: 1.4.1
react-native-screens: 1.0.0-alpha.23
react-navigation-stack: 2.0.0-alpha.23
react-native-redash: 8.2.0

1

I will stick with 1.2.0 for now

I find that problem is with this commit ec1e29104dba9be504226aa9d54345405162ef4f. Without code from this commit,, there is no issue.

Following fragment is causing the issues
  __detach() {
    if (!this._constant) { // This if statement causes the problem
      ReanimatedModule.getValue(
        this.__nodeID,
        val => (this.__nodeConfig.value = val)
      );
    }
    this.__detachAnimation(this._animation);
    super.__detach();
  }

So for now it looks like recycling same instance of InternalAnimatedValue for some basic values from [0, -1, 1, -2, 2] is featured problems. @kmagiera @osdnk guys, what do you think about it?

#415

@ShaMan123 Unfortunately, I'm not able to wrap every single reanimated value in my code with onChange. It would be much better if we could try to solve the issue instead of hacking 😸

Sure. Me neither.
I'll try your suggestion.
I tagged my issue to track yours.

@sebinq You rock!
I can confirm that your patch resolves the issue for my use cases.
I tested the following:
A: causes recursive updates
export default class InternalAnimatedValue extends AnimatedNode {
+    static valueForConstant = _.memoize((number) => new InternalAnimatedValue(number, true));
...
}

B: solves the issue

export default class InternalAnimatedValue extends AnimatedNode {
+    static valueForConstant = (number) => new InternalAnimatedValue(number, true);
...
}

It seems that the intention was to memoize constant values to save on allocation but for some reason this causes the opposite behavior.

@ShaMan123 thank you so much for your testing! Anyway, intention to memoize and reuse the  instance of most often created values is just great. I hope that someone with more experience can help and fix the problem 😄

#403 was created to tackle this error I think
Warning: Please report: Excessive number of pending callbacks: 501.
Some pending callbacks that might have leaked by never being called from native code:

Thanks @sebinq for reporting and working on a repro case. It's been really helpful in investigating this very obscure bug. Just submitted a fix and we will likely release it some time soon.
Also many thanks to @ShaMan123 for participating in the discussion and figuring out workarounds 🙌

@kmagiera you don't even know how glad am I to hear this news 🥂 thank you so much!


Webpage not posting the Create_Repo request

Thanks for highlighting it. I have updated the task in HTML webpage which was the issue in post response.
Please do check and let me know it works now. I have also refactored the code in the src directory for easy integration.

Its is working now, thanks for clearing up the issue.


Add Enum to Schema generation

Hello, noticed that the enum logic was setup but not fully implemented so I went ahead and implemented it. I'm not super familiar with this repository, so I don't know how to fully test this but it worked well with the file I used against it (which I can attach, it's a bit large), but here's an example of the output I am now receiving after implementing it.
"assetType": {
"type": "string",
"enum": [
    "EQUITY",
    "OPTION",
    "INDEX",
    "MUTUAL_FUND",
    "CASH_EQUIVALENT",
    "FIXED_INCOME",
    "CURRENCY"
]

generates
class Assettype(Enum):
    Equity = "EQUITY"
    Option = "OPTION"
    Index = "INDEX"
    Mutual_fund = "MUTUAL_FUND"
    Cash_equivalent = "CASH_EQUIVALENT"
    Fixed_income = "FIXED_INCOME"
    Currency = "CURRENCY"

I am concerned about backwards compatibility, as those who have used this library previously may not be able to handle the conversion from str to Enum (since they'd need to do .value. Would you recommend adding a argument to explicitly enable this behavior?
Addresses https://github.com/FebruaryBreeze/json-schema-to-class/issues/19

The flake8 error that's being thrown doesn't make sense to me, but I'll continue looking in this more over the next few days.

Update:
The output now looks like:
Handles if no default is specified in the schema, and also includes a __str__ override to allow backwards compatibility.
No default:
self.session = None
if "session" in values:
    self.session = self.Session(values.get("session"))

class Session(Enum):
    Normal = "NORMAL"
    Am = "AM"
    Pm = "PM"
    Seamless = "SEAMLESS"

    def __str__(self):
        return str(self.value)


Error compiling to wasm index out of bounds when implementing IntoWasmAbi

Describe the Bug
Introducing implementations of WasmDescribe, IntoWasmAbi and FromWasmAbi cause builds to fail with error:
[INFO]: 🎯  Checking for the Wasm target...
[INFO]: 🌀  Compiling to Wasm...
   Compiling test-wasm-bindgen v0.1.0 (~/Projects/Programming/test-wasm-binding)
    Finished release [optimized] target(s) in 0.10s
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', ~/.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-cli-support-0.2.87/src/descriptor.rs:218:15

Steps to Reproduce

Create a simple rust lib crate (I used the hello-world example as a basis)
Introduce a return type that requires an implementation of IntoWasmAbi/FromWasmAbi
Attempt to build
Observe error

see this repo with the code used to minimally produce the bug
Expected Behavior
The build succeeds
Actual Behavior
The build fails with an index out of bounds error
Additional Context
This seems to have popped up from time to time in wasm-bindgen's history with,
seemingly resolved by matching dependency versions (#3441, #2776, among others).
Troubleshooting

Build error only arises when IntoWasmAbi/FromWasmAbi are implemented for a new type.
As far as I can tell, dependency versions are all the same. I've tried many
of the older versions that resolved previous issues to no avail (0.2.78, 0.2.79, 0.2.84).
Attempted reinstalling wasm-pack with wasm-pack --locked to no avail

I'm realizing that perhaps the intention is not to be able to define arbitrary types, but to coerce a type into one of the already provided WasmAbi options. I had sort of hoped that wasm-bindgen would do some magic for simple structs, but I'm not seeing evidence of it in the docs.
I was loosely trying to follow this example, and I'm not sure if I misunderstood the struct here to be an example when instead it is just meant to show a special case.

I'm realizing that perhaps the intention is not to be able to define arbitrary types, but to coerce a type into one of the already provided WasmAbi options.

Pretty much, in your minimal example you didn't add a description to WasmDescribe, so this can't really work. But all of this is subject to the ABI implemented by wasm-bindgen which you can't really expand.


Error: code 8

Everytime I try to deposit Atom from Cosmostation IBC wallet to app.crescent I get this error msg in the picture?
Have tried to remove the Network and connection from the wallet, hard refresh but won't work.

This is core repository that runs Crescent Network blockchain. I suggest you to go ask Cosmostation about this issue. They will be able to give you an answer. I will close this issue as it is related to Cosmostation IBC wallet.

Ok, will. Thanks

From: Queen @.>
Sent: Thursday, October 6, 2022 7:27:55 AM
To: crescent-network/crescent @.>
Cc: MoonBoy @.>; Author @.>
Subject: Re: [crescent-network/crescent] Error: code 8 (Issue #69)
This is core repository that runs Crescent Network blockchain. I suggest you to go ask Cosmostation about this issue. They will be able to give you an answer. I will close this issue as it is related to Cosmostation IBC wallet.
—
Reply to this email directly, view it on GitHubhttps://github.com/crescent-network/crescent/issues/69#issuecomment-1269294080, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS5JJIGEFZSS5DMM52YGP2DWBZIMXANCNFSM6AAAAAAQ3ARG7I.
You are receiving this because you authored the thread.Message ID: @.***>


Fix AAF extact start

Hi,
It's the little correction (here) about a problem with Avid's AAFs.

I will try to get an example AAF with this scenario for a unit test.

Now that #370 is landed, can you rebase onto that to make sure that it passes the tests on the server?  it turns out the AAF unit tests weren't running previously.

Hi,
it's good for the rebase and test.
A little late but I also found an AAF test file.
TEST_1_1PLANS_1PASSE.zip (480ko)

Thanks for your contribution @flavienliger!


Use strict mode in prettier xml plugin

This will make XML files be less pretty, but also less surprising.
Fix #34.

This should be propagate to 14.0 branches. It's better to do it with [ci skip] for avoiding a lot of Travis builds.

This should be propagate to 14.0 branches. It's better to do it with [ci skip] for avoiding a lot of Travis builds.

Yes, waiting for #31 then I'll do the push, with ci skip indeed.


feat: pkg(add go struct tag)

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[X] Y

re https://github.com/KusionStack/KCLVM/issues/198

2. What is the scope of this PR (e.g. component or file name):
kusionstact/kclvm-go/pkg/tools/gen

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[X] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[X] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Thank you for your contribution. LGTM

Oh, the test failed.

Thank you for your contribution. LGTM

Oh, the test failed.
because  the test case can't use "`" inter the  ``, don't test the tag field.
it will return like this pic


Unify how CREATE TABLE is serviced and ensure table is query-ready [JIRA: RIAK-3173]

fix for CREATE TABLE removing the delta between how the request is serviced between HTTP and PB and RPC.
fix for CREATE TABLE, removing the delta between the post-condition of CREATE TABLE (was active bucket-type only) and "query-ready" (is bucket-type active, DDL module compiled and min capability met).

Depends on: https://github.com/basho/riak_core/pull/890
Tested by: https://github.com/basho/riak_test/pull/1254
Relates to: riak_test ts_cluster_riak_shell_regression_log which otherwise exposes a race condition between the CREATE TABLE and subsequent INSERT queries.

I'm not sure this does what's intended, at least not completely.
riak_core_bucket_type:activate returns ok after the claimant twiddles the active bit in CMD, so adding an extra layer of tests to make sure that has propagated makes sense.
However, is_table_supported just checks to make sure the module is compiled and there are no capabilities conflicts.
What we really need, if we want to be careful, is to do an rpc across the cluster to see if everyone agrees that the table is active.

@macintux I agree wrt the code not appearing to do what it expects as a postcondition, but afaik this is due to "action at a distance". The capabilities exchange is what the wait_until_supported/4 is waiting for.
If nothing else, and if my understanding is correct, I will add a comment to wait_until_supported/4.

Unifying create table for all APIs is very good, the other requests are call into riak_kv_ts_api though for writes, single key ops, queries etc.

Apologies, I wasn't very clear.  I meant we should put the create table logic into riak_kv_ts_api because puts, gets, deletes and queries are already shared between APIs from that module.

@andytill you were clear and right, riak_kv_ts_api is the single proper channel to direct these commands and queries through. I've made the change suggested.
I was confused by not finding riak_kv_ts_api to be the single proper channel for all commands over HTTP and PB, but on re-running commands w/ redbug tracing on, idk how I came to that conclusion :face_palm:

+1

I thought this didn't actually necessarily solve the problem (sadly)

+1

I'm not convinced these PRs solve the flappiness that has us concerned, but they're still valuable improvements.

thumbot wait

I’m now thinking that non-obvious changes without a glaring need on release day probably aren’t ideal. Asked Thumbs to wait.

After a good amount of digging through the call flow, this change, assuring the CREATE TABLE postcondition is met in a synchronous manner is necessarily employed on the front-end of servicing the request.
The riak_core metadata exchange is asynchronous, yielding synchronous control flow to the caller upon broadcasting the bucket-type metadata. The alternatives considered follow:

Spawn a a gen_fsm awaiting callbacks following the broadcast for each node to signal that they have reached "ready for query" state.
1.a. Since there is a maximum amount of time, have the gen_fsm have a timeout signal and terminal state, at which point the caller would be notified that the CREATE TABLE did not succeed due to timeout.
Poll from the service.

Both alternatives would work, but polling from the service is more straightforward. The call flow for CREATE TABLE is already complex enough, depending upon a new trigger mechanism sync_notify w/i the metadata exchange, callbacks from riak_core -> riak_kv, and duplication of code that determines whether a table is compiled.
The reason for the deep dive through the riak_core integration is that it was thought that the riak_core integration guaranteed the CREATE TABLE postcondition and did so in a synchronous manner, meaning control flow would not yield to the caller until the postcondition was met. This is not the case, though, as following broadcast to the eager peers (plumtree), the riak_core_claimant returns control flow to the caller. This means that broadcast to the lazy peers through the plumtree or fallback to endemic broadcast (gossip) is fulfilled following the return of control flow to the caller. This is what creates the race condition window.
After analyzing the call flow in full, I believe the solution employed here is the appropriate solution. The race condition window is closed and the code responsible for servicing the CREATE TABLE request was easily modified to perform the checks as well as report back to the client in the event that the postcondition is not met w/i the allowable timeframe.

create jira issue

thumbot retry

+1

Is it the table activation that is async?
I thought the compilation was being checked before activation already: https://github.com/basho/riak_kv/pull/1577

Yes, @andytill the table activation is async, as I commented the metadata exchange uses the plumtree protocol which is async. The compilation is hooked into the metadata listener, so is async.
I assume you intended to link the compilation, so linking here:
Registration of Trigger
https://github.com/basho/riak_kv/blob/riak_ts-develop/src/riak_kv_ts_newtype.erl#L99
Handling of Triggered Event

https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_metadata_evt_sup.erl#L67
https://github.com/basho/riak_kv/blob/riak_ts-develop/src/riak_kv_metadata_store_listener.erl#L53

Ah, I meant to link to the compile check in riak_core_claimant, https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_claimant.erl#L707.

Yes, @andytill, but that check for whether the ddl is compiled is in response to a bucket type status check, see:

https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_bucket_type.erl#L205
https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_claimant.erl#L302
is type compiled, not compile type, https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_claimant.erl#L707
https://github.com/basho/riak_core/blob/riak_ts-develop/src/riak_core_metadata_evt_sup.erl#L78

See https://github.com/basho/riak_kv/pull/1602 for documentation requested.

PR:

https://github.com/basho/riak_kv/pull/1577

[posted via JIRA by Derek Somogyi]

:+1:

+1


add MouseWheelEvent

fix #9071.

:+1:

had a chat with @zhengbli offline, this should be added to addedTypes instead.
something like:
{
        "kind": "typedef",
        "flavor": "Worker",
        "name": "MouseWheelEvent",
        "type": "WheelEvent"
},

@mhegazy Ah, I don't know we can generate types! Thanks.

:+1: Thanks!


Make logging optional

This pull request adds the capability to disable the logging at the start by passing log: false in the options object when creating the viewer. Useful for when a custom logging system is in place or when an app supports disabling logging.

ci failing

reopen if you want to finish


Redundant links to TryFocusAsync

Under See also section, looks like below are redundant entries.
TryFocusAsync(DependencyObject, FocusState)
TryFocusAsync(DependencyObject, FocusState)

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

ID: a1c7085a-dd87-e518-0e5e-6035466bb4b8
Version Independent ID: 9c8df148-d9a3-329a-dd18-2ccd0e851d34
Content: FocusMovementResult Class (Windows.UI.Xaml.Input) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.UI.Xaml.Input/FocusMovementResult.xml
Product: uwp
Technology: winui
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Thanks @turekhe. The topic has been fixed.


Is there a way to know what are the vehicles currently managed by traffic manager and their attributes ?

In particular, in sync mode, I am doing the following to activate a list of generated actors
def activate_actors(spec_actors_list, actors_list, tm):
    for i in range(len(actors_list)):
        perc_speed_diff = float(spec_actors_list[i].speed)
        obj_str = vehicle_types[spec_actors_list[i].model]

        actors_list[i].set_autopilot(True, tm.get_port())
        tm.vehicle_percentage_speed_difference(actors_list[i], perc_speed_diff)
        tm.auto_lane_change(actors_list[i], False)

I wonder after this function is called and the world tick one time, will all the actors be managed by the tm with the newly set parameters (regarding percentage_speed_difference and auto_lane_change)?

They will, yes.


Allow dataset streaming from private a private source when loading a dataset with a dataset loading script

Feature request
Add arguments to the function get_authentication_headers_for_url like custom_endpoint and custom_token in order to add flexibility when downloading files from a private source.
It should also be possible to provide these arguments from the dataset loading script, maybe giving them to the dl_manager
Motivation
It is possible to share a dataset hosted on another platform by writing a dataset loading script. It works perfectly for publicly available resources.
For resources that require authentication, you can provide a download_custom method to the download_manager.
Unfortunately, this function doesn't work with dataset streaming.
A solution so as to allow dataset streaming from private sources would be a more flexible get_authentication_headers_for_url function.
Your contribution
Would you be interested in this improvement ?
If so I could provide a PR. I've got something working locally, but it's not very clean, I'd need some guidance regarding integration.

Hello! It's a google cloud storage, so gs://, but I'm using it with https.
Being able to provide a file system like here would be even easier.
I've found a quite complicated workaround which consists of monkey patching all of the functions in streaming_download_manager.py to use my own get_authentication_headers_for_url .
A support for this use case would be greatly appreciated!
For reference my get_authentication_headers_for_url looks like this:
import os
from typing import Optional, Union

from datasets import config
from huggingface_hub import HfFolder
from gcsfs.credentials import GoogleCredentials

DEFAULT_PROJECT = os.environ.get("GCSFS_DEFAULT_PROJECT", "")
access = "full_control"
gcs_token = os.environ.get("GCS_TOKEN")

def get_authentication_headers_for_url(url: str, use_auth_token: Optional[Union[str, bool]] = None) -> dict:
    """Handle the HF authentication"""
    headers = {}
    if url.startswith(config.HF_ENDPOINT):
        if use_auth_token is False:
            token = None
        elif isinstance(use_auth_token, str):
            token = use_auth_token
        else:
            token = HfFolder.get_token()
    elif url.startswith("https://storage.googleapis.com"):
        credentials = GoogleCredentials(DEFAULT_PROJECT, access, gcs_token)
        credentials.maybe_refresh()
        token = credentials.credentials.token
    else:
        token = None
    if token:
        headers["authorization"] = f"Bearer {token}"
    return headers

I would be a big fan of this feature! @Hubert-Bonisseur if this doesn't become a supported feature, would you mind sharing your code? Thanks!

I would be a big fan of this feature! @Hubert-Bonisseur if this doesn't become a supported feature, would you mind sharing your code? Thanks!

I published it here:
https://github.com/Hubert-Bonisseur/private-dataset-hub
I modified the names of a lot of functions for privacy and I don't have time to test it again so you may get import errors, but you have the code. The custom_load_dataset is the function you are interested in I think.
It relies a lot on patching, if you find a better way to do this, I'd be interested.

Given the amount of patching it does, this is likely to break at one point. I'd encourage you to wait for a proper support in datasets directly if you can wait.


Improve vDVS document home page

We can improve our doc home page as shown below.

/CC @tusharnt

document UI refresh issue (can be taken care in future)


NEW_AMERICAN_STANDARD seems to be broken

Hello,
Thank you for your work on this project!  This is quite nice...  I don't know if this is expected, but for some reason it just does not work...
import pythonbible as bible

reference = bible.NormalizedReference(bible.Book.GENESIS, 1, 1, 1, 31)
verse_ids = bible.convert_reference_to_verse_ids(reference)
for verse_id in verse_ids:
    verse = bible.get_verse_text(verse_id, version=bible.Version.NEW_AMERICAN_STANDARD)
    print(verse)

That throws the following error...
$ python bible.py
Traceback (most recent call last):
  File "/opt/python/virtualenv/py311_test/lib/python3.11/site-packages/pythonbible/bible/bibles.py", line 54, in get_bible
    return BIBLES[version][bible_type]
           ~~~~~~^^^^^^^^^
KeyError: <Version.NEW_AMERICAN_STANDARD: 'NASB'>

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/mpenning/bible.py", line 7, in <module>
    verse = bible.get_verse_text(verse_id, version=bible.Version.NEW_AMERICAN_STANDARD)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/python/virtualenv/py311_test/lib/python3.11/site-packages/pythonbible/formatter.py", line 425, in get_verse_text
    bible = get_bible(version, "plain_text_readers")
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/python/virtualenv/py311_test/lib/python3.11/site-packages/pythonbible/bible/bibles.py", line 56, in get_bible
    raise MissingVerseFileError from error
pythonbible.errors.MissingVerseFileError
(py311_test) mpenning@mudslide:~$

If there is some reason that the verse for that version cannot be retrieved, it's better to make that clear (i.e. You must do foo to support bible.Version.NEW_AMERICAN_STANDARD).  FWIW, you can retrieve quite a bit of the bible for free via biblegateway

The pythonbible does not include the actual text of the New American Standard Bible due to copyright. As far as text that is bundled with the library, we only ever plan to include open source or public domain versions. Currently, we only have ASB and KJV, though we plan to add more. We also plan to make it easier for those who use the pythonbible library and obtain the proper permission from the copyright owner of another version to more easily integrate the pythonbible library with copyrighted versions within their own code, but we do not ever plan to bundle those versions within the library itself.
That said, your suggestion to make it more clear why a verse for a version cannot be retrieved is a very good suggestion, and we will work to make the error message in cases like this clear as to why that text is not included.
Thank you for your feedback and thank you for trying out and using the pythonbible library!

The pythonbible does not include the actual text of the New American Standard Bible due to copyright.

That makes sense... but perhaps you'd consider opening up the API a bit to use more than on-disk references.  For instance, you can use meaningless to retrieve passages assuming you have an internet connection available.
My perfect world is that your API would offer the ability to automatically:

First, retrieve the bible passage from disk
If a disk copy is not available (and the user wants to download from the internet), retrieve the bible from the internet
If internet / biblegateway.com is not available, throw an error

I can build a PR to do this if you'd like...
Thoughts?

To get content the right way from Bible Gateway, we would need to use their API (scraping the content would be a violation of their terms of service). Using the API requires authentication which requires having an account with Bible Gateway. (That's manageable but would require every pythonbible user to have that additional setup of an account if they wanted to use Bible Gateway.) We would also still be subject to the content and copyright restrictions however we obtained the data. Any use of copyrighted material beyond fair use would require obtaining a license from the copyright owner or risk legal issues. Fair use, in this case, being something like:

Material from the Site may be quoted in any form (written, visual, electronic or audio), up to and inclusive of two hundred fifty (250) verses or five hundred (500) words of non-biblical text without express written permission of the Zondervan, providing the verses do not amount to a complete book of the Bible nor do the verses quoted account for twenty-five (25) percent or more of the total text of the work in which they are quoted.

I decided from the very beginning of pythonbible to:

Always strictly honor copyrights.
Avoid, if at all possible, introducing any technical dependencies in the pythonbible library beyond the python standard library.

Either one of those, I think, would keep me from integrating with Bible Gateway. And the final nail would be the possibility that Bible Gateway might consider pythonbible to be a competing service, in which case pythonbible would not be able to access Bible Gateway at all according to their terms of service.
I very much understand why someone would want the functionality you are requesting, and there are python libraries out there for getting content from Bible Gateway (though I would recommend evaluating them to make sure they abide by the terms of service). Maybe a combination of one of those libraries along with pythonbible might provide what you want.


Format error in MemoryLeakDetector.cpp

Hi,
I tried compiling 3.7.2 in a ubuntu g++5.4 and I am seeing an invalid format error in the stated file that goes like : cpputest_3.7.2/src/CppUTest/MemoryLeakDetector.cpp:69:38: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘size_t {aka unsigned int}’ [-Werror=format=]
add("    %04lx: ", currentPos);
Dont know if anyone ran into this particular same problem or not out there.  Can size_t definition be changed here?

Seems current trunk has a cast added to that, which solves this. Closing it.


Merge changes to add Battlemetric Banner to end of MapRotation Embed

Merge in configuration to add the Battlemetrics Banner to the bottom of the Map Rotation Embed.

Hey I am way far behind but I'd like to merge this, do you have an example of a bm_banner_url I could look at? There's an id parameter but I'm not sure which URL you're using despite looking around the BM website @mmonney31


STS VBump with v147

This PR fixes #

I was just about to do it:) you should include more information in your PR about the changes you are bringing in.
these are the changes:

table designer change
query plan code port from SSMS
vbump of dacfx compact level 150 to 160 update.

And importantly the schema compare rollback that @Benjin did.
@ssreerama I assume you are doing this on his behalf or something?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.513%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26942

Relevant Lines:
58872

💛  - Coveralls

Looks good, but make sure you have confirmation from @Benjin that his change should be going in.

@Charles-Gagnon Benjin has reverted his changes in both ADS and STS and he is working on fixing the issue he found last week.
I asked Benjin to revert the changes in STS so that other folks can bring in new version of STS just like this one.

Sorry, I should mention some information in this PR. @Charles-Gagnon , as Alan said, Benjin has reverted the changes that have the ADS SC dependent changes. This PR includes the support of DacFx160 Nuget instead of DacFx_SqlProj(this branch is archived). Thanks.

@ssreerama fyi build for sql projects on linux is broken with this DacFx build https://github.com/microsoft/DacFx/pull/51#issuecomment-963645958
@zijchen already has a CR out with the fix for this in Dacfx, so we'll need to make sure to update the Dacfx nuget used here again before the December release.


Interface needs cleanup and robustness rewrite

To make it simple, we currently consider that any dimension except the first one is a spatial dimension (in TimeData) which make it a pain if you add a dimension. e.g if you solve multiple PDE at once in 2D you have u[t,x,y,p_src] but Devito interpret it as a 3D field and tries to take u.dz2 in the laplacian.
This one should be "easy" as long as we consoder only 2 or 3D problems. If not, I need to reqrite Dimension to seperate spatial, temporal and local ones.

FYI, I had this problem in YASK. In the new version of YASK under development, I'm requiring the user to more explicitly create each dimension as 'step' (i,e., temporal), 'domain' (i.e., spatial), or 'misc' (i.e., local). This allows me to have grids like you have mention here, and mix 3d, 2d, and 1d grids in the same problem, like when using Cerjan arrays for ABC.

if you solve multiple PDE at once in 2D you have u[t,x,y,p_src]

Can you please elaborate? What value is the notation u[t,x,y,p_src] trying to access?

What value is the notation u[t,x,y,p_src] trying to access?

The wavefield for the source number p_src, instead of looping over sources in python you solve all at once

Yes, the new auto-generation of derivative shortcuts should allow this. It's currently ownly restricted to space, but I'm working on time.

PointData is hard-coded as time-dependent, and non-compatible with non-conventional dimensions

Open #374 for the pointdata part, this one can be closed I think wiht #369

Is this still an issue @mlange05  @mloubout

Yes, I need to rewrite the "time free" case for the new devito.


Allow solution generation from Pint

This is related to unit testing in pint but we would have a separate commands that just generates a solution object (and its JSON representation) via something like pint generate-solution or something.

This could potentially be done as a new "package" kind at the manifest level too.
E.g. currently we have "library" and "contract" package kinds, where librarys are collections of items, and contracts compile to a contract <name>.json. Perhaps we could add a "solution" kind that, when built with pint build, generates a solution JSON? Something like:

Package Kind
pint build output in out/<profile>/

library
type-checks

contract
<name>-contract.json, <name>-contract-abi.json

solution
<name>-solution.json

Dup of https://github.com/essential-contributions/pint/issues/598. Closing this one and copying over all the information here.


callPeak question

scTail-callPeak -b $bamFile --gtf $gtfFile --cellbarcode $cellbarcodeFile -f $fastaFile --species human --chromoSize $chromosomeSize -o $outputfile --minCount 50 -p 20
[*] Selected device:  cpu
Traceback (most recent call last):
File "/home01/chenbg/miniconda3/envs/run_scTail/lib/python3.9/site-packages/pandas/core/indexes/base.py", line 3805, in get_loc
return self._engine.get_loc(casted_key)
File "index.pyx", line 167, in pandas._libs.index.IndexEngine.get_loc
File "index.pyx", line 196, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/hashtable_class_helper.pxi", line 7081, in pandas._libs.hashtable.PyObjectHashTable.get_item
File "pandas/_libs/hashtable_class_helper.pxi", line 7089, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'cellbarcode'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home01/chenbg/miniconda3/envs/run_scTail/bin/scTail-callPeak", line 8, in
sys.exit(main())
File "/home01/chenbg/miniconda3/envs/run_scTail/lib/python3.9/site-packages/scTail/bin/callPeak.py", line 142, in main
getPAScluster=GetPAScluster(PASrefpath,generefpath,fasta_file,bam_file,out_dir,n_proc,minCount,maxReadCount,densityFC,InnerDistance,device,chromoSizePath,cellbarcodePath,species)
File "/home01/chenbg/miniconda3/envs/run_scTail/lib/python3.9/site-packages/scTail/utils/getPAS.py", line 31, in init
self.cellbarcode=pd.read_csv(cellbarcodePath,delimiter='\t')['cellbarcode'].values
File "/home01/chenbg/miniconda3/envs/run_scTail/lib/python3.9/site-packages/pandas/core/frame.py", line 4102, in getitem
indexer = self.columns.get_loc(key)
File "/home01/chenbg/miniconda3/envs/run_scTail/lib/python3.9/site-packages/pandas/core/indexes/base.py", line 3812, in get_loc
raise KeyError(key) from err
KeyError: 'cellbarcode'
but I check the barcode.tsv
head -n 5 $cellbarcodeFile
AAACCCAATGAAACACT
AAACCCAATGAAACCAT
AAACCCAATGAAACCCA
AAACCCAATGAAACCCG
AAACCCAATGAAATGAG
what's the matter with my code?
I am looking forward to your reply

Hi,
Thank you very much for your question. I guess that is because you did not add the column name in your barcode file. Could you add the "cellbarcode" as the column name of your barcode file, just looking like the example file in our figshare.
Best regards,
Ruiyan

Oh，thanks very much !!!


Product Roadmap Generator

Title and Issue number
Product Roadmap Generator Added
Issue number #4717
Close #4717
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani Sir kindly please review the pr

@MohitGupta121 sir kindly review the pr


Bug fixes and use local repository option for zbm-builder.sh

Fixes for the zbm-builder.sh script.
Unlike Podman, Docker does not accept . or ./ as arguments for bind volumes at the command line. Modifying zbm-builder.sh to use ${PWD} instead of . works for both Docker and Podman.
A typo in the default config.yaml causes zbm-build.sh to fail, fixed.
Added option to use a local repository as opposed to a remote one.
Currently, zbm-build.sh is hard-coded to pull the required tag from zbm-dev/zfsbootmenu. Adding this command line option allows builds using a local repository tree.

Oops, my apologies @zdykstra. Hopefully this is better? :sweat_smile:

He means squash the commits, which you can do on your end by running
git rebase -i master

This should open up an editor with one line for each commit, starting with pick and followed by the hash. Change the pick on every line except the first to squash, then save and quit the editor. Another editor should open up with all of the commit messages concatenation together; just remove them and write a combined message that makes sense. (The text you have here is fine as the commit message for the squashed commit.) Finally,
git push -f

should force-push the new single-commit branch to your fork and be reflected in this PR.

Thanks and apologies @ahesford, let me know if this is OK. rebase -i was playing up but git reset --soft HEAD~4 followed by git push -f did the trick.

Ready to go, I believe. :+1:

Thank you for the improvements / cleanup!


Implement fmt::String for offset::Local?

When trying to use format() on chrono::Local::now(), I'm getting the following error:
error: type `chrono::datetime::DateTime<chrono::offset::Local>` does not implement any method in scope named `format`

Local::now().to_string() also fails with:
error: type `chrono::datetime::DateTime<chrono::offset::Local>` does not implement any method in scope named `to_string`

I believe this is due to chrono::offset::Local not implementing std::fmt::String, where the other offsets do.

Ah, oops. Local missing String implementation is my fault. I also forgot that DateTime with non-String Offset may still want to format the date and time (and possibly the offset).

Thank you!


Elastic.CommonSchema.NLog - Skip Url-UserName when no value

Improving on #195

:grey_exclamation: Build Aborted

The PR is not allowed to run in the CI yet

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-09-07T19:37:04.676+0000

Duration: 3 min 48 sec

Steps errors
Expand to view the steps failures

Load a resource file from a library

Took 0 min 0 sec . View more details here
Description: approval-list/elastic/ecs-dotnet.yml

Error signal

Took 0 min 0 sec . View more details here
Description: githubApiCall: The REST API call https://api.github.com/orgs/elastic/members/snakefoot return the message : java.lang.Exception: httpRequest: Failure connecting to the service https://api.github.com/orgs/elastic/members/snakefoot : httpRequest: Failure connecting to the service https://api.github.com/orgs/elastic/members/snakefoot : Code: 404Error: {"message":"User does not exist or is not a member of the organization","documentation_url":"https://docs.github.com/rest/reference/orgs#check-organization-membership-for-a-user"}

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Small Spelling Mistake

[Enter feedback here]
URL: https://docs.microsoft.com/en-us/windows/uwp/updates-and-versions/choose-a-uwp-version
Wrong Spelling: This verion of Windows 10 was released in October 2017.
Correction: Change "verion" to "version"

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

ID: fd1ff7f3-d77d-b7bd-8dfa-7f25cb0b5239
Version Independent ID: 71e57b3b-b25f-0299-2cbc-f895a98c685d
Content: Choose a UWP version - UWP applications
Content Source: windows-apps-src/updates-and-versions/choose-a-uwp-version.md
Product: uwp
Technology: deployment
GitHub Login: @hickeys
Microsoft Alias: hickeys

Thanks for the report. I've made this fix; it should be live by the end of the day


Issue 237 - Report evidence regardless of ontology mapping status

Closes #237.

Updated now with quality metrics, if it looks good I'll update the spreadsheet as well.

Pull Request Test Coverage Report for Build 857468260

5 of 20 (25.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 73.06%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

eva_cttv_pipeline/evidence_string_generation/clinvar_to_evidence_strings.py
5
20
25.0%

Files with Coverage Reduction
New Missed Lines
%

eva_cttv_pipeline/evidence_string_generation/clinvar_to_evidence_strings.py
3
47.22%

Totals

Change from base Build 848761943:
-0.4%

Covered Lines:
838

Relevant Lines:
1147

💛  - Coveralls


Regarding public CMS MuOnia Example Analysis python notebook

Hi everyone,
I was wondering if the notebook provided as an example (publicCMSMuonia_exampleAnalysis) was the actual code/workflow run here or at least similar in functionality here and here
Thanks in advanced for any pointers!

hey, yep, all the notebooks are the actual code. this project is no longer maintained, but all notebooks/etc should work, provided the requirements are satisfied.
I would suggest to use something like uproot

hey @vkhristenko,
Thanks a lot for replying back! Those are very cool numbers for the scalability tests 👍
I was wondering about the actual hw where the scalability tests where run. The paper states:

This infrastructure is comprised of almost 1300 cores and 7TB of RAM.

But the slides show three different clusters (1) Hadoop Yarn with 30 nodes with 13TB of RAM, (2) Hadoop Yarn with 65 nodes with 20TB of RAM, (3) Kubernetes 250 cores with 2TB of RAM. So none of these match the ones mentioned in the paper. Would you be able to tell which actual cluster was used? and maybe the actual hw specs of the machines?
Many thanks in advance @vkhristenko !

@renato2099, sorry for the actual hw specs i'm not the right guy to ask
May be @LucaCanali can provide more info in terms of machine specs.

@renato2099 we have been running the data reduction workload at scale on various systems (production and test), also over time the clusters have evolved as new nodes are being added, old machine are decommisioned. For the actual tests reported in the article the number of cores and memory used at the time (on a shared YARN system) are repoted with the performance graphs. YOu can direct mail me if you need more info.

Thanks @vkhristenko and @LucaCanali  for your support!


Fix object memory leaks

Addresses leaks of GitObject instances.  Added test, also tested in GitKraken for stability.

@srajko so it seems that this is causing the freeing of a Commit object to break now on some platforms.


In Ubuntu is not guaranteed that /var/lib/locales/supported.d/local exists

I've run into this problem when trying to provision a docker container from scratch.
It seems that the /var/lib/locales/supported.d/local file is loaded by PAM but PAM is not invoked when using the docker container.
I wonder if we can force the PAM to be invoked?

I have the same problem but on OpenVZ and ubuntu-14.04-x86_64.

I'll see next week if we can find a good awnser  for that issue.
Any help on this problem would be greatly appreciated :)

Hi!
I encountered the same issue on Ubuntu 12.04, but in virtual machine not a container.
My workaround was to touch /var/lib/locales/supported.d/local manually.
@fonsecas72 are you sure that there is a link with PAM?

Maybe we can check is file exists in this ruby_block: https://github.com/redguide/locales/blob/master/providers/default.rb#L80
Chef::Util::FileEdit raise exception if file doesn't exists: http://www.rubydoc.info/gems/chef/Chef/Util/FileEdit#initialize-instance_method [view source]


Unexpected order of fuzzy find results

Version: 1.47.3
Commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
Date: 2020-07-23T13:08:29.692Z (1 wk ago)
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:

Create a folder with files names managed_object.h.template and generated_types.h.template
Open fuzzy find and type gt.h.template
Expect generated_types.h.template to be found first, instead managed_object.h.template is found first

It's unclear whether this is intentional, and I'm not sure how fuzzy find is intended to work, but coming from Sublime Text, the current behaviour is not very intuitive to me. Sublime Text puts generated_types.h.template first.

Does this issue occur when all extensions are disabled?: Yes

/duplicate https://github.com/microsoft/vscode/issues/103889

This is a consequence of our scorer preferring consecutive matches the longer they get. gt.h.template is having more consecutive matches on generated_types.h.template compared to managed_object.h.template.
If you leave out the dots, it will sort properly:


Medium clue - Equip Ring of dueling(8)

You don't need 'ring of dueling(8)' specifically you can use any type of dueling ring however the equip thing doesn't show the green 'X' unless you wear a dueling ring 8
https://i.imgur.com/iwRHqQY.png
Suggestion: change it from 'ring of dueling(8)' to just 'ring of dueling' this would be less confusing and make it show the green 'X' which ever dueling ring you wear. (hopefully that is possible)

I think it is strictly tied to item id, OSB works exactly the same I believe.


chore(sdk): Refactor server testing fixtures

Fixes

Part 5 of a series of PRs for landing W&B -> W&B Import API

https://github.com/wandb/wandb/pull/5780
https://github.com/wandb/wandb/pull/5781
https://github.com/wandb/wandb/pull/5782
https://github.com/wandb/wandb/pull/5783
https://github.com/wandb/wandb/pull/5785
https://github.com/wandb/wandb/pull/5777

Description
What does the PR do?

🤖 Generated by Copilot at 899d0fc
This pull request refactors and improves the code for testing and importing runs from mlflow into wandb. It uses dataclasses, type annotations, and wandb utilities to simplify and standardize the code. It also adds new features and fixes to the sender, interface, and util modules.
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at 899d0fc

Sing, O Muse, of the mighty deeds of wandb
The glorious tool that tracks and logs the runs
Of heroes who train and test their models well
And import their results from mlflow and other sources

/fix-title


feat: upgrade native sdk <IP_ADDRESS>

Update native sdk <IP_ADDRESS> dependencies

This pull request is trigger by bot, you can checkout this branch and update it.

Examples built:

https://nightly.link/littleGnAl/hoe/actions/runs/7219215943/agora_rtc_engine_android_6.2.6-sp.4262_20231215073850.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7219215943/agora_rtc_engine_ios_6.2.6-sp.4262_20231215074459.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7219215943/agora_rtc_engine_macos_6.2.6-sp.4262_20231215073438.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7219215943/agora_rtc_engine_windows_6.2.6-sp.4262_20231215073234.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7219215943/agora_rtc_engine_web_6.2.6-sp.4262_20231215072956.zip

This comment is commented by bot, do not edit it directly


118 5.5 download letters of agreement

Coverage decreased (-1.4%) to 98.617% when pulling 7d530bf2690ea934b903939c621860aa07f1839d on 118_5.5_DownloadLettersOfAgreement into 0a95c56ff26a5c0bb193beee7e04119365df49a5 on dev.

Coverage remained the same at 100.0% when pulling 90463e72f3044b80c8ba64789f2e0736a87a1732 on 118_5.5_DownloadLettersOfAgreement into 0a95c56ff26a5c0bb193beee7e04119365df49a5 on dev.

@Fabian-Sommer It would be cool to have some sample data.

Coverage remained the same at 100.0% when pulling fddbbcbe719dec1fc1bbd0fab7173c803bdc5d55 on 118_5.5_DownloadLettersOfAgreement into 466db106a16882c2beafd87d1361d96dc1e1aa6b on dev.

Coverage remained the same at 100.0% when pulling 0e37ec8528bfd09ed8bda3d3af664a0671c233ef on 118_5.5_DownloadLettersOfAgreement into 466db106a16882c2beafd87d1361d96dc1e1aa6b on dev.

Coverage remained the same at 100.0% when pulling f56e8ff174c9205f2d791d843ec4e83b7b757bb3 on 118_5.5_DownloadLettersOfAgreement into 565ed21ee319a00876b7a6573538d0101939ff7d on dev.

Coverage remained the same at 100.0% when pulling f56e8ff174c9205f2d791d843ec4e83b7b757bb3 on 118_5.5_DownloadLettersOfAgreement into 565ed21ee319a00876b7a6573538d0101939ff7d on dev.

Coverage remained the same at 100.0% when pulling 99c1a42c79500e8ac298ff7832fc37e8a59be8e9 on 118_5.5_DownloadLettersOfAgreement into 02bc4e565db549057f1d698663e9002daf74b663 on dev.


Update Tabs API

🙋 Feature Request
We need to make the Tabs API more flexible to support custom layouts and additional options (e.g. an "Add tab" button). Applications may need to put the tab list in one container and the tab content somewhere else, without being constrained to the default layout. However, they still need to be connected for accessibility purposes. In addition, it's currently more difficult than it needs to be to include add icons to tabs.
💁 Possible Solution
Rather than putting <Item> elements directly inside <Tabs>, and using the title prop to specify the tab content and children to specify the tab panel content, we can instead separate them out. The Tabs component will simply be a state container and communicate to <TabList> and <TabPanels> via a context. The TabList and TabPanels components are both collection components, taking <Item> elements as children, which must have matching key props so that we can show the content for the selected tab. Because the <Item> for the tab and the contents are separate, this makes it easier to include more complex content like icons rather than putting it in the title prop.
<Tabs>
  <TabList>
    <Item>
      <FilesIcon />
      <Text>Files</Text>
    </Item>
  </TabList>
  <TabPanels>
    <Item>
      <FilesPanel />
    </Item>
  </TabPanels>
</Tabs>

By default, <Tabs> should be a flex container in either horizontal or vertical orientation (depending on the orientation prop). However, this API also allows for custom layouts to be built by simply wrapping the <TabList> and <TabPanels> components in any other layout components (e.g. <Flex>). This allows inserting additional content between as well, e.g. an add tab button.
<Tabs>
  <Flex direction="column">
    <Flex direction="row">
      <TabList>
        <Item key="tab1">Tab</Item>
      </TabList>
      <ActionButton>
        <AddIcon />
      </ActionButton>
    </Flex>
    <TabPanels>
      <Item key="tab1">Tab 1 content</Item>
    </TabPanels>
  </Flex>
</Tabs>

API

useTabs should be renamed to useTabList
useTabsState should be renamed to useTabListState
tabPanelProps should be removed from useTabs and added to a new useTabPanel hook

interface TabListProps<T> extends CollectionBase<T>, SingleSelection {}
interface AriaTabListProps<T> extends TabListProps<T>, DOMProps, AriaLabelingProps {
  keyboardActivation?: 'automatic' | 'manual',
  orientation?: Orientation,
  isDisabled?: boolean
}

interface TabListState<T> extends SingleSelectListState<T> {}
declare function useTabListState<T>(props: TabListProps<T>): TabListState<T>;

interface TabListAria {
  tabListProps: HTMLAttributes<HTMLElement>
}

declare function useTabList<T>(props: AriaTabListProps<T>, state: TabListState<T>): TabListAria;

interface TabProps {
  key: Key
}
interface AriaTabProps extends TabProps, DOMProps, AriaLabelingProps {}

interface TabAria {
  tabProps: HTMLAttributes<HTMLElement>
}

declare function useTab<T>(props: AriaTabProps, state: TabListState<T>): TabAria;

interface TabPanelProps {
  key: Key
}
interface AriaTabPanelProps extends TabPanelProps, DOMProps, AriaLabelingProps {}

interface TabPanelAria {
  tabPanelProps: HTMLAttributes<HTMLElement>
}

declare function useTabPanel<T>(props: AriaTabPanelProps, state: TabListState<T>): TabPanelAria;

In this case, the Spectrum API and the Aria/Stately APIs are quite different. Most props should go to the top <Tabs> element which has no Aria equivalent. In addition the <TabPanels> element has no Aria equivalent, just the individual panels. It should be possible to implement this API on top of the ARIA API using context and a bit of prop mapping to send things to the right places.
interface SpectrumTabsProps<T> extends SingleSelection, DOMProps, StyleProps {
  items?: Iterable<T>,
  keyboardActivation?: 'automatic' | 'manual',
  orientation?: Orientation,
  isDisabled?: boolean,
  disabledKeys?: Iterable<Key>,
  isQuiet?: boolean,
  density?: 'compact' | 'regular'
}

interface SpectrumTabListProps<T> extends DOMProps, StyleProps {
  children: CollectionChildren<T>
}

interface SpectrumTabPanelsProps<T> extends DOMProps, StyleProps {
  children: CollectionChildren<T>
}

I believe the <Tabs> component may also need its own state to store the selection, and then use the useTabListState in controlled mode (e.g. passing selectedKeys in from context). The <TabPanels> component will also need to use the useCollection hook to build its own collection so that it can find the correct tab panel to display.
🧢 Your Company/Team
RSP

My initial implementation.
https://github.com/intergalacticspacehighway/react-spectrum/blob/feat/tabs-api-update/packages/%40react-spectrum/tabs/src/Tabs.tsx
(Custom layout requirement (e.g. Flex as children in Tabs) and storybook examples+testcases are working).
This is hacky atm. I didn't find any better way to share state between list and panel (unless we move the entire state to the Tabs)
Also, I am not sure how useful it would be to make TabPanels and TabList as collection components since useCollection already works with <Item /> as children for TabPanels and useTabListState handles the same for TabList.
Also, it would be difficult to build a collection tree like Table as TabList and TabPanels won't always be immediate children.
I might be missing something. Let me know if I should refer to any other component for reference!

Thanks for working on this so quickly @intergalacticspacehighway! I spent some time thinking about this problem and all the alternate solutions I could think of were worse than your current solution. As you said, it's not really possible to move the state to the Tabs because the collection is built based on the children of the TabList. Other options I considered were changing useTabPanel to take the tabListId and selectedKey as props instead of passing in the state, but I don't really like forcing users of React Aria to manage these types of things since they are easy to get wrong. It also makes the API harder to use for the simple case, which is just a single component with both the TabList and TabPanel in one component rather than the more flexible API we are implementing for Spectrum. What you've done is a bit hacky but overall not too bad, so I think it's fine to go ahead with that.

@devongovett true, I'll take a fresh look and see if I can make any improvements. Will generate a PR soon! Thanks 😄

@devongovett what if we do something like below and remove the dependency on state in useTabPanel. I haven't tested this yet. Might have race conditions depending upon order of useTabs and useTabPanels, but using the optional condition should do the trick.
Idea is to have the same ids for a tab that is selected and tabpanel between rerenders 😅
Will test it as I get some time.

@devongovett tabPanel can always have an id like *-tabPanel and the selected tab can always have an id of *-tab-selected. We just need to set them on the mount and preserve them between renders (using the tabIds Weakmap?)
I think something like this should remove the dependency on the state in useTabPanel.

@devongovett One more solution I can think of is to use a useControlledState or a new hook called useSingleSelectState at Tabs component and use useListState in useTabListState instead of useSingleSelectListState.
Downside of this is that it'll change the useTabList ARIA hook's API. 🙁
declare function useTabList(props: AriaTabListProps, state: TabListState): TabListAria;
to
declare function useTabList(props: AriaTabListProps, state: SingleSelectState, listState: TabListState): TabListAria;
declare function useTabPanels(props: AriaTabPanelProps, state: SingleSelectState);


WIP Remove Google Spreadsheet from Repairnator

Fix #481

Pull Request Test Coverage Report for Build 1478

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 52.821%

Totals

Change from base Build 1476:
0.0%

Covered Lines:
2050

Relevant Lines:
3881

💛  - Coveralls


Plans to support controlnet?

Any upcoming plans to support Controlnet integration?

I don't know if the project is still active, but I started adding ControlNet tile model support, cf. #65.


Update spdlog to 1.11.0 and its bundled fmt to latest master

this was not part of the 3.7.4 changes, but the spdlog changelog contained so many bug fixes that I deemed this a good idea to update.
I documented the change of the fmt version bundled into speedlog to the current commit on master via the README rather than diffing the changes into the spdlog patch.
The change from the previous version of the spdlog patch is already included in spdlog v1.11.0
Signed-off-by: Ingmar Sittl<EMAIL_ADDRESS>

Codecov Report

Merging #789 (cee4cc9) into c++14-compliant (5db4aaf) will increase coverage by 0.99%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@                 Coverage Diff                 @@
##           c++14-compliant     #789      +/-   ##
===================================================
+ Coverage            88.25%   89.25%   +0.99%
===================================================
  Files                  243       91     -152
  Lines                10034     6266    -3768
===================================================
- Hits                  8856     5593    -3263
+ Misses                1178      673     -505

Impacted Files
Coverage Δ

framework/include/cppmicroservices/AnyMap.h
71.42% <0.00%> (-28.58%)
:arrow_down:

...rk/include/cppmicroservices/detail/WaitCondition.h
71.42% <0.00%> (-28.58%)
:arrow_down:

...mework/include/cppmicroservices/ShrinkableVector.h
85.71% <0.00%> (-14.29%)
:arrow_down:

framework/src/bundle/BundleResourceBuffer.cpp
67.34% <0.00%> (-13.51%)
:arrow_down:

framework/src/bundle/BundleRegistry.cpp
79.50% <0.00%> (-12.81%)
:arrow_down:

framework/src/bundle/BundleUtils.cpp
80.64% <0.00%> (-12.69%)
:arrow_down:

framework/src/util/Utils.cpp
74.39% <0.00%> (-12.57%)
:arrow_down:

...mework/src/service/ServiceReferenceBasePrivate.cpp
86.45% <0.00%> (-6.78%)
:arrow_down:

framework/src/util/FrameworkPrivate.cpp
70.96% <0.00%> (-5.26%)
:arrow_down:

framework/src/service/ServiceRegistrationBase.cpp
77.31% <0.00%> (-4.55%)
:arrow_down:

... and 227 more


Search on glossary term does not work

Search on glossary terms does not work.
Contract currency is defined in glossary
And has been assign to asset:
Search does not give result:
Nor via as glossary is greyed out
issue glossary.pdf

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

ID: a006b62e-b3a3-18f9-cb26-1fd91b4cf9bd
Version Independent ID: e9ab2116-d1eb-bf34-709a-667582372094
Content: Understand business glossary features in Azure Purview (preview) - Azure Purview
Content Source: articles/purview/concept-business-glossary.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @nayenama
Microsoft Alias: nayenama

@SergeMoreno
Thanks for the feedback! We are currently investigating and will update you shortly.

@SergeMoreno  Thanks for the reaching out! We have assigned the issue to the content author to investigate this and update the document as appropriate.
Hi @nayenama , could you please have a look at this ask and do the helpful needed. Thank you.

I work as consultant with different companies and on different Azure environments.
On the delaware Azure, the Glossary works fine - on the Volvo Financial services Azure environment it does not work.
Kind Regards
From: Kranthi Pakala<EMAIL_ADDRESS>Sent: woensdag 10 februari 2021 0:33
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Moreno, Serge<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Search on glossary term does not work (#70016)
ATTENTION: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@SergeMorenohttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSergeMoreno&data=04|01|serge.moreno%40delaware.pro|d511e69ec7c9459b497b08d8cd5304e6|17b35a1d057c4ac5a15a08758f7a7064|0|0|637485103737621443|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EqwMK%2BOlTwTwqzU56EGv7NYgZ9BrG%2B8YKubYuRiXVVI%3D&reserved=0 Thanks for the reaching out! We have assigned the issue to the content author to investigate this and update the document as appropriate.
Hi @nayenamahttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnayenama&data=04|01|serge.moreno%40delaware.pro|d511e69ec7c9459b497b08d8cd5304e6|17b35a1d057c4ac5a15a08758f7a7064|0|0|637485103737631408|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Ufm1yqLZ0jxpgriQoCP3SZgTBGnuDH1jQEwNn7aD4fI%3D&reserved=0 , could you please have a look at this ask and do the helpful needed. Thank you.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F70016%23issuecomment-776316450&data=04|01|serge.moreno%40delaware.pro|d511e69ec7c9459b497b08d8cd5304e6|17b35a1d057c4ac5a15a08758f7a7064|0|0|637485103737631408|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=a%2FfQIkNJf8rR83hIvH7RpaOsPcpVzCrcmaHBxEYv9Qk%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASWHTDIFTUAYGVN22YKQEZLS6HA2FANCNFSM4XCPRM3Q&data=04|01|serge.moreno%40delaware.pro|d511e69ec7c9459b497b08d8cd5304e6|17b35a1d057c4ac5a15a08758f7a7064|0|0|637485103737641359|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6869db7NGVjQAqx5oQBiTA4IKGO4qdBOmyzDjHyzSQc%3D&reserved=0.

Data assets can be searched based on glossary terms however the search result page does not display schema info to which glossary is tagged.

#please-close


Can Not Install on Windows

I have tried installing using with both Ruby's version 2.2.3 and 2.1.7 with DevKit installed and keep arriving at the following error message. I have done gem install json -v '1.8.3' and that is successful. I have also been able to install your version of Bundler.
Installing the 'vagrant-digitalocean' plugin. This can take a few minutes...
Bundler, the underlying system Vagrant uses to install plugins,
reported an error. The error is shown below. These errors are usually
caused by misconfigured plugin installations or transient network
issues. The error from Bundler is:
An error occurred while installing json (1.8.3), and Bundler cannot continue.
Make sure that gem install json -v '1.8.3' succeeds before bundling.
Warning: this Gemfile contains multiple primary sources. Using source more than once without a block is a security risk, and may result in installing unexpected gems. To resolve this warning, use a block to indicate which gems should come from the secondary source. To upgrade this warning to an error, run bundle config disable_multisource true.Gem::Installer::ExtensionBuildError: ERROR: Failed to build gem native extension.
"C:/Program Files (x86)/Vagrant/embedded/bin/ruby.exe" extconf.rb

creating Makefile
make "DESTDIR="
generating generator-i386-mingw32.def
/bin/sh: -c: line 0: syntax error near unexpected token x86' /bin/sh: -c: line 0: /C/Program\ Files\ (x86)/Vagrant/embedded/bin/ruby -e "puts 'EXPORTS', 'Init_generator'"  > generator-i386-mingw32.def'
make: *** [generator-i386-mingw32.def] Error 2
Gem files will remain installed in C:/Users/Brian/.vagrant.d/gems/gems/json-1.8.3 for inspection.
Results logged to C:/Users/Brian/.vagrant.d/gems/gems/json-1.8.3/ext/json/ext/generator/gem_make.out

Here is a bit about my PATH variable which might help pinpoint which tool is causing the problem:
C:\Users\Brian\AppData\Local\Programs\Ruby21\bin;C:\Ruby22\bin;C:\Users\Brian\AppData\Roaming\npm;C:\Program Files (x86)\Microsoft VS Code\bin;C:\Users\Brian\AppData\Local\Programs\Git\cmd;C:\Users\Brian\AppData\Local\Programs\Git\mingw64\bin;C:\Users\Brian\AppData\Local\Programs\Git\usr\bin;C:\Users\Brian\AppData\Local\Android\android-sdk\platform-tools;C:\Users\Brian\AppData\Local\Programs\Python

As you point out, that's an issue with the json gem on Windows. I'd recommend opening an issue against that project since they likely have a lot more experience getting the gem working on Windows.

Hi.. the gem successfully downloads/installs when I install it directly but when I install this plugin it throws the error. Has anyone gotten this plugin to work on Windows?
If so, do you know anything about their environment settings? What ruby version, what command prompt etc?


Update example of using external CSS

Hey folks and @guidiego!
I've updated next, react and other dependencies to the latest versions.
I also noticed that it does not work as expected. Can anybody help me with debugging? @guidiego?)

 Can I read about it any where?

@GarrisonD next use webpack too make a lot of good stuff, unfortunately, some stuff breaks server-side experience.  I guess that you need  "extensions" in it.

@guidiego before making it work with sass, postcss, etc. i want to make it running with simple css. everything works but only one time when i run it the first time, after that any changes to index.css are ignored and i can't figure out why.

I didnt rember excatly when postcss was added, but I think that this was added to remove this problem @GarrisonD.
@nahue did you remember something about this issue that Garrison report?

@guidiego sorry, but i didn't understand about webpack DEV stuff, can you provide me an example or a link where I can read about it?

@guidiego what do you think about react-starter-kit approach for building universal React apps?


Fix broken import in knex.js

Unless I've misread, I think this link is broken.

Looks like our build system is having some serious problems...

I suppose that this is not a bug. As far as I know this file is not compiled.

Huh? package.json's main property points to knex.js. knex.js exports ./lib/index, which doesn't exist. How is that not broken?

It exists when the lib is babel compiled, check the node_modules files when it's installed from npm

Ah! Thought I was going crazy there.. thanks!


sh.vim [178]: for (( expr1; expr2; expr3 )) not syntax-highlighted correctly

command for (( expr1; expr2; expr3 )) is not syntax-highlighted correctly. And the highlight changes depending on the command location: whether outside or within a compound command.
I'm running vim built from master (lastest as of now: ca4cc018addbeb3ac5d0e05f18847015f91ff814) on Xubuntu 18.04, with ~/.vim/syntax/sh.vim 178 downloaded from Dr Chip's Vim Page
To reproduce it, run:
vim -N -u NONE -i NONE -c 'let b:is_bash=1' -c 'set ft=sh' -c 'syn on' file
where file is this bash script:
for (( i=0; i < 10; i++ )); do
    echo "$i"
done

case $1 in
    foo)    for (( i=0; i < 10; i++ )); do
		echo "case: $i"
	    done
	    ;;
esac

if [[ $1 == bar ]]; then
    for (( i=0; i < 10; i++ )); do
	echo "if: $i"
    done
fi

for word in foo bar; do
    for (( i=0; i < 10; i++ )); do
	echo "if: $i"
    done
done

echo foo | while read -r word; do
    for (( i=0; i < 10; i++ )); do
	echo "if: $i"
    done
done

To give an idea of the results:

Indeed ! And includes @cecamp's latest. There's a still a glitch with numbers inside (( )), and [[ ]].

Please try the latest one from my website. I did another update...
Chip Campbell

On Aug 1, 2018, at 6:42 AM, Paulo Marcel Coelho Aragão<EMAIL_ADDRESS>wrote:
Indeed ! And includes @cecamp's latest. There's a still a glitch with numbers inside (( )), and [[ ]].
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

181 gave the final blow to this issue !
I hope I haven't disturbed your trip again ! Thanks a lot !


How to build openh264 for QNX?

Hello,
I want to compile the openh264 project for my QNX platform. I know openh264 does not support QNX as OS.
Could you please support me on this issue?
Best regards,

#3662


[feature-request]: Add aria-label to "brand logo" on websites to make them more accessible.

What feature would you like to request?
To increase accessibility of our websites, we should see if we can add an aria-label to the brand logos of our websites. This is the logo to the far left of the navbar that e.g., says "Seedcase Project", "Seedcase Design", or "Seedcase Sprout".
Optional: Suggest a solution
No response

I believe we can solve this with the logo-alt: field that we can add in the _quarto.yml file. https://quarto.org/docs/websites/website-navigation.html#top-navigation

Ah, you're right! 🤩  Nice! I'll add it to #258 and rename the PR


Laravel Passport scope on group route does not work

when i use scope in Passport authentication,
i want to add these scopes into api.php routes with different group of route
but i get an error:

Invalid scope(s) provided.

In my AuthServiceProvider.php:
public function boot()
{
    $this->registerPolicies();

    Passport::tokensCan([
        'app' => 'User Application',
        'admin' => 'Admin Application',
    ]);
    Passport::routes();
}

and in my api.php:
Route::prefix('v1')->group(function () {

    Route::middleware(['auth:api', 'scope:app'])->group(function () {
        // Some App Route
    });

    Route::prefix('admin')->group(function () {

        // Some Admin Route without auth

        Route::middleware(['auth:api', 'scope:admin'])->group(function () {
            // Some Admin Route with auth
        });
    });
});

A MissingScopeException is thrown when a request is made by a user without the required scopes (e.g., app or admin). So, the route is correctly protected, but the request is made without those scope(s).

Route::group(['middleware' => ['auth:api', 'scopes:your_scope']], function(){
*your_routes are here
}


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: success
Windows Build State: failure
Commit: 8d5ab28fe882432e96618770f2f392279dc2f5cb
Commit Branch: dev
Commit Author: innerviewer

Auto-closing as build in this branch is successful now. Commit: dfceadc7f6587896691ba923d3a0495f12ccd2df


Dev

Codecov Report

Merging #6 (08adc98) into main (698a855) will increase coverage by 2.39%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   82.09%   84.49%   +2.39%
==========================================
  Files           6        6
  Lines         162      187      +25
==========================================
+ Hits          133      158      +25
  Misses         29       29

Impacted Files
Coverage Δ

src/QuantileMatching.jl
100.00% <ø> (ø)

src/functions.jl
100.00% <100.00%> (ø)


Using Programmatically nuxt-ts

How can I do this in nuxt-ts?
import { Nuxt, Builder } from "nuxt";

This question is available on Nuxt community (#c8586)

like that
const { Nuxt } = require('@nuxt/core');
const { BundleBuilder } = require('@nuxt/webpack')
const { Builder } = require('@nuxt/builder');
const configuration = require("../nuxt.config");
 const nuxt = new Nuxt(configuration);
// Build only in dev mode
 if (process.env.NODE_ENV !== 'production') {
 const builder = new Builder(nuxt, BundleBuilder);
  builder.build();
}


Implement controls for internet radio

This requires adding stop to pyamplipi/pyamplipi/amplipi.py
I opened another ticket for that on that repo.
(https://github.com/brianhealey/hacs_amplipi/tree/main/custom_components/amplipi)/media_player.py
Line 251
Add 'internetradio'
            if self._current_stream is not None and self._current_stream.type in (
                    'internetradio',
                    'spotify',
                    'pandora'
            ):
                return SUPPORT_AMPLIPI_MEDIA

Now that the source that's playing reports the controls  available (in
0.1.8) via info->supported_cmds this logic might now be needed.
On Sat, May 7, 2022, 20:39 hawkeyeP @.***> wrote:

This requires adding stop to pyamplipi
https://github.com/brianhealey/pyamplipi/pyamplipi/amplipi.py
I opened another ticket for that on that repo.
(
https://github.com/brianhealey/hacs_amplipi/tree/main/custom_components/amplipi)/media_player.py
Line 251
Add 'internetradio'
        if self._current_stream is not None and self._current_stream.type in (
                'internetradio',
                'spotify',
                'pandora'
        ):
            return SUPPORT_AMPLIPI_MEDIA

—
Reply to this email directly, view it on GitHub
https://github.com/brianhealey/hacs_amplipi/issues/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZPO6O5M3SLXTDNB6PD4LVI4ELDANCNFSM5VLEPHEQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@linknum23 @hawkeyeP thanks. I updated both the pyamplipi and hacs_amplipi to support the latest firmware, and dynamically determine visibility based upon the values from the amplipi api.


Typing: Excel should be Word

This
A Word add-in interacts with objects in Excel by using the Office JavaScript API, which includes two JavaScript object models:
should read as
A Word add-in interacts with objects in Word by using the Office JavaScript API, which includes two JavaScript object models:
and this
which you can use to check whether a specified requirement set is supported by the Excel application where the add-in is running.
should read as
which you can use to check whether a specified requirement set is supported by the Word application where the add-in is running.

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

ID: 1a8cba22-5199-61d8-c668-a38227ba7e92
Version Independent ID: b01cb6ce-95e9-3682-4d92-7efefa204d55
Content: Word JavaScript object model in Office Add-ins - Office Add-ins
Content Source: docs/word/word-add-ins-core-concepts.md
Product: word
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for pointing that out @iwhp. I'll get those typos corrected soon.

Thanks again @iwhp. These changes should be live later this afternoon.


normally there should be 2 config entries for the payload: max number of entries and max size in bytes for a payload.

Right normally there should be 2 config entries for the payload: max number of entries and max size in bytes for a payload.
Originally posted by @drmingdrmer in https://github.com/datafuselabs/openraft/issues/173#issuecomment-1042539657

@drmingdrmer Remapping my suggestion to use an trait for that, could be more interesting this below?
trait Metrics {
  fn len(&self) -> u64;
}

struct Foo {
  value: String
}

pub enum StateMachineRequest {
  AddFoo { value: Foo },
  BatchAddFo { values_1: Vec<Foo>, values_2: Vec<Foo> },
}

impl Metrics for StateMachineRequest {
  fn len(&self) -> u64 {
    match self {
      StateMachineRequest2::AddFoo { value } => 1,
      StateMachineRequest2::BatchAddFo { values_1, values_2 } => {
        values_1.len() + values_2.len()
      }
    }
  }
}

If Foo does not implement Metrics, openraft could assume that size is 1 always.

I'm thinking about what you need is a batch client_write():
Raft.batch_client_write(reqs: Vec<StateMachineRequest>)

This way, a user does not need to implement the batching mechanism manually.

You are right. Let me fix the entry payload size limit issue first.


How do I update a token that needs a new name?

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[X ] question about how to use this project

Summary
I created a token but the name has to be changed.  How should I make the changes to allow this to occur?  Do I need a maintainer to do a manual merge?

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

One other note - we CANNOT lose the symbol

yes, submit a PR and then link it here

https://github.com/solana-labs/token-list/pull/16759
Thanks @keone.  Please give me an ETA so that I can alert my client.
Thanks!

Hey @keone where are we with this one?  I need to give my client an update.
https://github.com/solana-labs/token-list/pull/16759 again is the link to the PR

Thanks so much @keone for the assist!  How long before the change is reflected on SolScan?


find_object_2d: 0.6.2-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository find_object_2d to 0.6.2-0:

upstream repository: https://github.com/introlab/find-object.git
release repository: https://github.com/introlab/find_object_2d-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.1
previous version for package: null

@matlabbe FYI: If you release from a new branch you will need to bump the minor version number as only one version of the code can exist on the upstream repository for a given tag. Otherwise catkin_prepare_release will fail next time you prepare a release for both kinetic and lunar with the same version number.
(see ros_tutorials for an example of package releasing from a different branch every ROS distro and bumping the version number accordingly)

Thx for the warning. I use the same version of the standalone (no ROS) version of Find-Object to be easier to track which version is actually used in ROS (if I have issues opened on a particular version of Find-Object, standalone or ROS).
I do the same with RTAB-Map, the ros package uses the version of the standalone version to track the issues instead of the ROS distros.
I didn't have problems in the past with bloom doing so though, I'll see the next release if it is too problematic. If it fails, I'll know why!


Separate ingress definition from workload

Adds the ingress definitions (which services to expose at which path) to a separate helm chart. This makes it easier to a) adopt the RI for other workloads and b) to add more services that should be expose (instead of just catalogservice and healthservice).
This also requires that each service is aware of the specific path-prefix, otherwise this requires a lot of rewrite-magic at each reverse proxy stage which in some cases doesn't even properly work at all. Thus the changes on the application code side to add the base path there.
It also enables to expose each swagger UI (when enabled). Currently running here:
https://afe2edd5b-global-fd.azurefd.net/
https://afe2edd5b-global-fd.azurefd.net/catalogservice/swagger
https://afe2edd5b-global-fd.azurefd.net/healthservice/swagger
@heoelri @msimecek thoughts?

I thought about the pros and cons and I'd advocate to not separate the ingress configuration into its own helm chart. I'd instead suggest to put the ingress rule back into each service's helm chart like what we had in the beginning:

gameservice/templates/ingress.yaml
healthservice/templates/ingress.yaml

I think that this is the most reasonable and understandable approach. You're looking for the healthservice's ingress rule? It's in the healthservice helm chart. Packaging together k8s manifests and resources that share the same lifecycle. They're installed, updated and removed together.
- chart
| `- values.yaml
| `- Chart.yaml
| `- templates
|    `- deployment.yaml
|    `- ingress.yaml
|    `- hpa.yaml
|    `- ...

Why have we removed the ingress rule from the healthservice at some point (see ingress.yaml) and moved it into the gameservice helmchart? The issue we were facing is that having an ingress rule in healthservice and gameservice caused cert-manager to request two certs for the same domain name, this caused a lot of conflicts and we often reached Let's Encrypt's rate limits. To work around the issue we've moved the healthservice ingress rule to the gameservice (later catalogservice):
https://github.com/Azure/Mission-Critical-Online/blob/fef64c0242db6799e1197a0beb981abb3876777b/src/app/charts/healthservice/templates/ingress.yaml#L1-L3
https://github.com/Azure/Mission-Critical-Online/blob/fef64c0242db6799e1197a0beb981abb3876777b/src/app/charts/catalogservice/templates/ingress.yaml#L56-L62
This approach has of course its own downsides - the healthservice now requires the catalogservice helm chart to be installed to expose its /health endpoint. This is indeed suboptimal.
That's why I'd suggest to go back to the initial approach - individual ingress rule definition per service / per helm chart. All (or multiple) charts can point to the same k8s secret that contains the TLS cert - and we can use an annotation to let cert-manager only request new certs for one of them.
kubectl get ingress -n workload
NAME                     CLASS   HOSTS                                                ADDRESS         PORTS     AGE
catalogservice-ingress   nginx   afe2e5465-cluster.swedencentral.cloudapp.azure.com   <IP_ADDRESS>   80, 443   10m
healthservice-ingress    nginx   afe2e5465-cluster.swedencentral.cloudapp.azure.com   <IP_ADDRESS>   80, 443   11m

kubectl describe ingress -n workload |grep -A1 TLS
TLS:
  workload-ingress-secret terminates afe2e5465-cluster.swedencentral.cloudapp.azure.com
--
TLS:
  workload-ingress-secret terminates afe2e5465-cluster.swedencentral.cloudapp.azure.com

Only the ingress rule deployed via the chart that has the cert-manager.io/cluster-issuer: "cert-manager-alwayson" annotation set will trigger cert-manager to request a new cert.
https://github.com/Azure/Mission-Critical-Online/blob/4e004f768965e5fc8cc1608e97afaae4070c655a/src/app/charts/catalogservice/values.yaml#L17-L19
This is the same approach other projects with larger numbers of helm charts use. Like for example the TailwindTraders project on GitHub.
See https://github.com/Azure/Mission-Critical-Online/pull/356 how the implementation could look like. And https://dev.azure.com/Mission-Critical/Online/_build/results?buildId=1374&view=results for a successful test.

Couple of additional thoughts why I suggest to keep the Ingress rule in each service's helm chart instead of separating them into an "ingress" helm chart:

When a service changes its path like for example from /api/catalogservice to /api/gameservice.

or

In a newer release an additional path i.e. /api/catalogservice/v2 is introduced that points to a different K8s service.

These changes are done in the per-service helm chart only. A newer version of the chart contains the change and can be rolled back the same way. The chart can be tested and reviewed and does not have dependencies to other (in this case a central ingress helm chart) charts.
When services are split to different clusters like for example not all services are deployed to all stamps/regions or within a given region there are frontend/backend clusters there's no need to then have multiple versions of this "central ingress chart".
Other benefits are that the risk that the "central ingress chart" contains abandoned paths is lower and changes to ingress affect only each individual chart - instead of changes made to a "central chart" that could potentially have an impact to all other services.
Changes to the "central ingress chart" would require testing ALL other charts relying/depending on this chart.

After reading the discussion and looking at the two variants of code, I prefer Heyko's approach to this one. Having the ingress config sitting together with each service means that changes to all ingresses need to be done in multiple places, however it facilitates the idea of common lifecycle and logical grouping.

Having the ingress config sitting together with each service means that changes to all ingresses need to be done in multiple places

what do you mean by this?
My concern with the decentralized approach is that you would need to bring infra-specifics into the applications - or find a different way to keep them out of their ingress definitions. Like the hostname part of the ingress file or additional header validations with front door which we are currently doing. I just doubt this is feasible to tell each app team "please incorporate these parts into your ingress definition"

I share your concerns, we've currently some very specific stuff in our ingress rules.
https://github.com/Azure/Mission-Critical-Online/blob/0a0ed298248a2cbb18f3fbdd34b172df7fa7d6ed/src/app/charts/catalogservice/templates/ingress.yaml#L10-L30
Some of these settings could potentially either go into the global ingress-nginx configuration, though things like force-ssl-redirect and others will probably not work with the currently used http01 certificate resolver (which requires HTTP), or could potentially being dropped like the Front Door Header in the AFDv2 with PL scenario.
To reduce the complexity we should think about ways to simplify these ingress rules / helm charts by moving/removing some of these annotations while still keeping them in each individual per-service helm chart.

My concern with the decentralized approach is that you would need to bring infra-specifics into the applications - or find a different way to keep them out of their ingress definitions. Like the hostname part of the ingress file or additional header validations with front door which we are currently doing. I just doubt this is feasible to tell each app team "please incorporate these parts into your ingress definition"

First of all, I think it's okay for the app team to own their part of the system end to end. Second, no one is preventing the infra team from doing these changes if asking the owning app team is not feasible (due to the sheer number  of teams or whatever other reasons).

@sebader how do we want to proceed with this? Do we want to have another conversation about this?
Or do we proceed with PR #356?
cc: @msimecek