Emails are displayed in LHN instead of names

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

Action Performed:

Go to https://staging.new.expensify.com
Log in with phone number (Account A)
Find a user you like to chat and send a message (Account B)/ or you can request money.
Log in with Account B and observe who sent you message

Expected Result:
First and Last name should be displayed in LHN
Actual Result:
Emails are displayed in LHN and after opening the conversation the First and Last name are displayed.
Workaround:
Unknown
Platform:
Where is this issue confirmed?

[x] Web
[ ] iOS
[x] Android
[ ] Desktop App
[x] Mobile Web

Version Number: 1.0.80-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @MonilBhavsar (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I have been able to reproduce this issue from time to time. Here's some extra information.

We don't have solid reproduction steps but I have noticed this happens after having a session open after a long time.
All conversations in LHN are affected by this.
If you open the conversation, the correct name and last name is displayed in header and in LHN

I will try to get this on video and post it here.

Got it on video 🎉
https://user-images.githubusercontent.com/44479856/126868358-9acd6b53-fbcd-499b-9bfa-24c04475a7b9.mp4

Will look at it today

Investigating it, I saw this in production too

AFAIK, users' details are fetched from a different API. And on the app start, we don't wait for the response to come. But LHN is not rerendered when the response is received. I suggest we add the user details for the LHN list in the new API which Marc added.

@parasharrajat the issue is that prevProps are different for options.participantsList. Once I add the condition here to compare previous and next props for participantsList. It renders the page again and it works!
What do you think?

@MonilBhavsar Yeah. If we are updating the options.participantList after the participantsDetails is updated in Onyx, then yes this is what we need to cause re-render. But we have to check that it should not cause a flash on the first load on mobile screens.

I'm unable to reproduce this on Android, so can't make sure if it flickers when I apply this patch.

I am not properly able to reproduce this issue. So can't make sure if it flickers by applying the patch. I think we should apply it and see.
@marcaaron what should we do here

Looking the logs in that video and I'm seeing that offlineMessageRetry warning that was discussed briefly here https://expensify.slack.com/archives/C01GTK53T8Q/p1627565830443000

AFAIK, users' details are fetched from a different API. And on the app start, we don't wait for the response to come. But LHN is not rerendered when the response is received.

That's correct. However, I think maybe in this case the console errors might suggest a network issue that prevented the data from being fetched.

I suggest we add the user details for the LHN list in the new API which Marc added.

We could look into what that would take, but I think in most cases this should happen and also very quickly (about 100ms call for me on production).
I'm not 100% sure if we need to prioritize this right now - but I like the solution of eliminating the extra logic here and fixing it in the back end.
https://github.com/Expensify/App/blob/d26de0563905d6b5fb4a8d2a6696030ba15a9678/src/libs/actions/PersonalDetails.js#L134-L188
I noticed there is a second call to mergeCollection() in the API callback so by returning all data at once we can maybe avoid merging reports twice (which may have perf implications - but honestly not too sure).

Looking the logs in that video and I'm seeing that offlineMessageRetry warning that was discussed briefly here expensify.slack.com/archives/C01GTK53T8Q/p1627565830443000

I only see this error during hot-reload. I believe it's not an issue. It only happens when the browser cancels the inflight request on page reload and etc.

I only see this error during hot-reload. I believe it's not an issue. It only happens when the browser cancels the inflight request on page reload and etc.

Our QA testers are not building from dev (i.e. no hot reload) yet the console logs in the video show a similar message. But I agree it could be unrelated - not sure!

Bumping it to weekly, Feel free to change it back to Daily, if it's on a top priority

No update

I've been able to constantly reproduce this on Web on 2 of my accounts lately

Hmm, when did you last send message to this account, was it frequent or some days/weeks back?

Frequent since it's my testing account. I used it several times yesterday

I got back to this today, but still hard time reproducing it. @isagoico, can you please share in which version you're able to reproduce this and if you see any errors in console.

I'm currently able to reproduce this consistently after log in in version 1.0.93-0. I checked console and no errors were displayed
https://user-images.githubusercontent.com/44479856/132029700-478937ed-17f0-46f7-ae5c-b71b1ec3274f.mp4

I found the issue, working on it.

@kavimuru @isagoico can you please test this again, It's resolved.

I was unable to reproduce the issue at first log in but it had this behaviour sometimes. I will keep an 👁 out today for this issue and will post here if I experience it again.

@isagoico were you able to see this again?

Closing this out now


BUG - MAE method doesn't like pd.Series when using dimension preservation

Hi folks, not sure if this is a bug but i thought i'd drop it in here anyway...
I am trying to do simple MAE between two pandas Series data using scores.continuous.mae, which the docstrings tells me is kosher, and trying to preserve dimensions. When i do this i get an AttributeError: 'Series' object has no attribute 'dims'
trace:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/tmp/ipykernel_2250677/1926751918.py in ?()
----> 1 scores.continuous.mae(loaded_site_forecast.data.siteforecast_air_temperature, loaded_site_forecast.data.observations_temperature_at_screen_level, preserve_dims='all')

~/mambaforge/envs/site/lib/python3.11/site-packages/scores/continuous.py in ?(fcst, obs, reduce_dims, preserve_dims, weights)
    141     ae = abs(error)
    142     ae = scores.functions.apply_weights(ae, weights)
    143
    144     if preserve_dims is not None or reduce_dims is not None:
--> 145         reduce_dims = scores.utils.gather_dimensions(fcst.dims, obs.dims, reduce_dims, preserve_dims)
    146
    147     if reduce_dims is not None:
    148         _ae = ae.mean(dim=reduce_dims)

~/mambaforge/envs/site/lib/python3.11/site-packages/pandas/core/generic.py in ?(self, name)
   6198             and name not in self._accessors
   6199             and self._info_axis._can_hold_identifiers_and_holds_name(name)
   6200         ):
   6201             return self[name]
-> 6202         return object.__getattribute__(self, name)

AttributeError: 'Series' object has no attribute 'dims'

I expect that this is purely that no-one has ever run a pd.Series through this pathway before and so the .dim hasn't been an issue...

Here is a snippet to reproduce this issue:
import pandas as pd
import scores
xda_fcst = scores.sample_data.simple_forecast()
xda_obs = scores.sample_data.simple_observations()
print(scores.continuous.mae(xda_fcst, xda_obs, preserve_dims='all'))
fcst = pd.Series(scores.sample_data.simple_forecast())
obs = pd.Series(scores.sample_data.simple_observations())
print(scores.continuous.mae(fcst, obs))
print(scores.continuous.mae(fcst, obs, preserve_dims='all'))

Support for this will likely need to be pushed to further down the roadmap for a variety of reasons, mostly to do with complexity of supporting heterogenous data types in an elegant way. There are options which can be considered, but none of them have yet emerged as a clearly appropriate way forward. Pandas support remains on the roadmap for the medium term, but it is more urgent to complete the current tranche of work associated with implementation of required scores and preparation for particular use cases.

There is now an explicit scores.pandas API which should be used and is supported. This should resolve this use case, please take a look and see if it meets your needs.


The sku was not found. Please fetch products first by calling getItems

I have this error for some users. Looks like an edge case.
For 99% of the users works without an issue.
The items are Consumable and I don't use getItems
Environment:

react-native-iap:  12.5.1,
react-native: 0.70.6
Platforms (iOS, Android, emulator, simulator, device): Android (Samsung S20, A13 and more)

To Reproduce
Steps to reproduce the behavior:
I use this code
try {
	const remoteProducts = await getProducts({ skus: skuIds });
        this.setState({ remoteProducts });
} catch (error) {
}

and
try {
	await requestPurchase(Platform.OS === 'ios' ? { sku: skuId} : { skus: [skuId] });
} catch (error) {
}

I have the "skuIds" and "skuId" all the time, I check them first.

@MariusCatanoiu since you include two different blocks of code. could it be that there's a race condition in your code?

@MariusCatanoiu since you include two different blocks of code. could it be that there's a race condition in your code?

I don't see this case. The second cod is running only if the remoteProducts flag is changing status in the state

Are you calling endConnection at some point?

Are you calling endConnection at some point?

Of course. In componentWillUnmount, and initConnection in componentDidMount.
On app resume I call again initConnection. I have a method with a listener on AppState when you put the app in background and return. From time to time the connection is lost, and I have to call it again

Same issue.

Hi all,
we are facing the same situation on Android, calling getProducts passing an array of skus will result in missing products.
I could explain it a little better, adding some screenshot.
Calling getProducts(skus:['test002','test003]) will result in adding only 'test002' in products, ignoring 'test003'
Calling useIAP().products give us only 'test002'. See screenshot below.

So when calling requestPurchase({skus: [_sku]}) where _sku is 'test003', we get the error 'The sku was not found. Please fetch products first by calling getItems' because actually 'test003' is missing in products.
While on IOS everything seems to be fine, Calling getProducts(skus:['test002','test003]) will result in adding both 'test002' and 'test003' in products and so, calling requestPurchase({skus: [_sku]}) where _sku is 'test003', result in a successful request purchase (and we can also finish it by calling finishTransaction({...})
Calling useIAP().products give us 'test002' and 'test003'. See screenshot below.

Android: tested on Samsung Galaxy Tab S6
IOS: tested on Ipad Air 6th generation
Hope this help in debugging.
@andresesfm let me know if I can give you further details

@davide-desio-eleva  getProducts(skus:['test002','test003]) has to be getProducts({ skus:['test002','test003] }).
I'm missing something?

@davide-desio-eleva getProducts(skus:['test002','test003]) has to be getProducts({ skus:['test002','test003] }). I'm missing something?

Hi @MariusCatanoiu, we are passing an object as you point out calling getProducts({ skus:['test002','test003] }).
Sorry for the mistake, I've edited my previous post.

Other info that might be helpful.
Everything refers only to Android.
We have iap updated to 12.7.2.

We've added a 'test004' product in google play console, and suddenly calling getProducts({ skus:['test004'] }), it returns the product 'test004'
- We've then added a 'test005' product in google play console, but calling getProducts({ skus:['test005'] }), it has returned an empty array
- After a couple of hour we call again getProducts({ skus:['test005'] }), and the 'test005' appears magically into products
Then we add another product 'test006' in google play console, and calling getProducts({ skus:['test006'] }), it return instantly the product 'test006'
- Then we add another product 'test007' in google play console, but calling getProducts({ skus:['test007'] }), it has returned an empty array

We are retrying calling getProducts({ skus:['test007'] }) to see if it magically appears after a while.
All products were always active, and consumables

It seems that there is a 'period' before the in-app-purchase is available even if it is signed as active in google play console.
@andresesfm could this be true in your expirience?
This probably explain why this is happening only on android platform while IOS is ok.

Thank you for the clarification @davide-desio-eleva . It seems to be a known issue (expected behavior?) on the Google Side: https://stackoverflow.com/questions/16320381/propagation-delay-when-changing-in-app-product-price-description-in-google-pla

Same issue.
It seems to me that the problem is that I am caching the offerToken, and for some reasons they are different every time? What's the point of that? That is particularly annoying since getSubscriptions fails more than half the time for some reason, with init failing with the error message :"Billing is unavailable. This may be a problem with your device, or the Play Store may be down"

@Pingou I had no idea the offerToken is different every time. I'm curious as to why you need to cache it?
As for getSubscriptions failing it could be that the device's connection drops or that you are calling it too often

Too often per user

Are you calling endConnection at some point?

Of course. In componentWillUnmount, and initConnection in componentDidMount. On app resume I call again initConnection. I have a method with a listener on AppState when you put the app in background and return. From time to time the connection is lost, and I have to call it again

@MariusCatanoiu I think you might be doing this too often which causes race conditions. The recommendation is to init/end the connection at the top level of your app

Thank you for the clarification @davide-desio-eleva . It seems to be a known issue (expected behavior?) on the Google Side: https://stackoverflow.com/questions/16320381/propagation-delay-when-changing-in-app-product-price-description-in-google-pla

Well, reference is about a propagation delay when changing in app product info (like price or description) and this could be based on cached information on google side.
But we are creating completely new products.
Those should be available instantly on google side and thus calling the getProducts() method of react-native-iap.
Is this method getting them from the local play store cache or directly via an API?

@davide-desio-eleva
We are not caching the response as described above. Also note that the reference is for an old version of Billing services. Here is another reference: https://stackoverflow.com/questions/11068686/this-version-of-the-application-is-not-configured-for-billing-through-google-pla

Thank you all for your comments. Please reopen if any other issues

Thank you for your clarification.
Good to know that nothing is cached as described above.
Still the reference has no clue with this issue: it says Google take a while to process applications, but those are totally other situation than this issue.
We are adding an in-app-purchase on Google Play Console, and before seeing it with getProducts() sometimes we should wait seconds, sometime we should wait hours, sometimes it takes a day.
We could assume this is an intended behavior, still without any specific reference.
We'll test it in as much condition as possible, and update here for future reference is anyone is experiencing the same issue.

@davide-desio-eleva Would you have any clue for it. I got exactly same issue with.

You should wait until it appears: I can confirm that "We are adding an in-app-purchase on Google Play Console, and before seeing it with getProducts() sometimes we should wait seconds, sometime we should wait hours, sometimes it takes a day." and there is anything you can do but wait.


Add <uses-permission android:name="android.permission.DETECT_SCREEN_CAPTURE" /> to implementation steps

As stated here: https://developer.android.com/about/versions/14/features/screenshot-detection#implementation  devices running Android 14 need this permission to be able to detect screen recordings.
Would be great if this could be added to the implementation steps of this package!
Cheers.

@nicklbaert
Thanks a lot!
You saved me👍


Remove-DbaAgentAlert - new command

Type of Change

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

Purpose

Expanding functionality.
Approach

Similar to Remove-DbaDbView, Remove-DbaDbMailProfile, Remove-DbaDbMailAccount
Commands to test

Remove-DbaAgentAlert -SqlInstance localhost, localhost\namedinstance

Remove-DbaAgentAlert -SqlInstance localhost -Alert MyDatabaseAlert

Get-DbaAgentAlert -SqlInstance SRV1 | Out-GridView -Title 'Select SQL Agent alert(s) to drop' -OutputMode Multiple | Remove-DbaAgentAlert

Screenshots

Learning

Gorgeous, i love it! Thank you 🙇🏼

@potatoqualitee hey Chrissy, was it added to the module? I don't see it
Get-command -Module dbatools -name 'Remove-DbaAgentAlert*'
Neither here https://docs.dbatools.io/Remove-DbaAgentAlert

my apologies! just fixed it. will do a release shortly.


Add support to Godwoken Testnet

Website
https://startwithnervos.com/godwoken
ChainId
Network Name: Godwoken Testnet v1
RPC URL: https://godwoken-testnet-v1.ckbapp.dev/
Block Explorers: https://nervos.gitbook.io/layer-2-evm/ecosystem/block-explorers
Chain ID: 71401
Additional information
Before you can deploy contract you need to create an account on Layer 2:
L2 Godwoken Testnet Account Generator: https://testnet.bridge.godwoken.io/ or https://dev.ckb.tools/create-layer2-account

Hey, thanks for the update. It was delayed because I was looking into the account generation part. I will get back to it this week,

Factory creation tx: https://gw-explorer.nervosdao.community/tx/0xd6ec7457ec684b64b7c5934f30ddefe1d076af72f8658e2d65cdc263fd8c2d18/internal-transactions
Factory:
https://gw-explorer.nervosdao.community/address/0x914d7Fec6aaC8cd542e72Bca78B30650d45643d7/transactions


Will hide helpfull error messages during next.js

Description
Due to checks for NODE_ENV error messages are hidden during next build, making it impossible to debug.
Next.js 13.2.
(My errors are also only happening during build, not using next dev but I am still analyzing this)
Mandatory reproduction URL (CodeSandbox or GitHub repository)
Sorry, next build does not work on CodeSandbox
Reproduction description
Steps to reproduce:

Open any nextjs project
Do not (!) load message.jsons
run next build
Notice the error:

n [Error]: MISSING_MESSAGE

Notice that it is not stating which file is
(Also there is a weird "n" but maybe that is from the production minification)
Expected behaviour
Error message should tell me which message path and namespace could not be found.

Hey,
yes, this is done on purpose to reduce the bundle size.
Usage questions should generally be directed to StackOverflow as mentioned in the issue wizard, I hope you understand!
Maybe this is the issue you're seeing: https://github.com/amannn/next-intl/issues/87


Addition of 'bulletPosition' prop to help position the bullets.

A prop 'bulletPosition' can be passed into the Carousel to position the bullets in the carousel.

This should be called bulletsStyle

Would like to see this merged if possible, not sure what the best naming convention for the prop is but it is definitely valuable functionality.

@CodyBrouwers feel free to use bulletsContainerStyle but in most cases you'll need to overwrite defaults: https://github.com/phil-r/react-native-looped-carousel/blob/master/index.js#L430

Great, thanks @phil-r!


JavaScript FFI test fails with Error: Cannot find module 'bn.js'

When running the presumbit.sh script, the test js_test fails with error:
10/11 Test #10: js_test ..........................***Failed    0.35 sec

><EMAIL_ADDRESS>test /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js
> mocha

Error: Cannot find module 'bn.js'
Require stack:
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/crypto.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/test/crypto_test.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/node_modules/mocha/lib/esm-utils.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/node_modules/mocha/lib/mocha.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/node_modules/mocha/lib/cli/one-and-dones.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/node_modules/mocha/lib/cli/options.js
- /home/dev/repos/starkware/crypto-cpp/src/starkware/crypto/ffi/js/node_modules/mocha/bin/mocha

Presumably it's due to the bn package being installed instead of bn.js.

pragma once
#if PLATFORM_USE_APPLE_LLVM_ATOMIC_CMPXCHG_128_PATCH
//
//  Patch for Apple LLVM version 8.x.x (clang-800.0.38 - clang-900.0.37) intrinsic 128-bit __atomic_compare_exchange implementation (debug, using opt level -O0).
//  Note that this patch is only in effect on tvOS/iOS AArch64 debug builds for Apple LLVM version 8.x.x. Arm32 verified working without patch.
//
//  Problem:
//   For the above builds, the __atomic_compare_exchange asm expasion used SUBS/SBCS to compare the pair of "obj" and "expected" values.
//   SUBS/SBCS does not provide sufficient NZCV flags for comparing two 64-bit values.
//   The result is erraneous comparison of "obj" and "expected".


Unable to compile Assimp so that I can debug problem with importing in LumixEngine.

c:\development\lumixengine_3rdparty\3rdparty\assimp\code\version.cpp(67): fatal error C1083: Cannot open include file: 'revision.h': No such file or directory
Any information would be greatly appreciated.

I will try to fix this but in the meantime you can create revision.h in assimp's include directory looking like this:
#ifndef ASSIMP_REVISION_H_INC
#define ASSIMP_REVISION_H_INC

#define GitVersion 0x0
#define GitBranch "master"

#endif // ASSIMP_REVISION_H_INC

By the way you do not need to build assimp yourself to debug it, there is already a prebuilt library with debug information. Assimp has a lot of issues and I am considering to get rid of it and use fbx sdk, it supports far less formats but at least they 99% work.

Wow, so awesome! Thank you so much!
On Tue, Apr 19, 2016 at 1:55 PM, Mikulas Florek<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/nem0/LumixEngine_3rdparty/issues/1 via
4bb0a77
https://github.com/nem0/LumixEngine_3rdparty/commit/4bb0a776685454d738f2e2b14a99d8b2759a9ab7
.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nem0/LumixEngine_3rdparty/issues/1#event-633506988


GH-360 Select action names from drop-down list

Fixes #360

Applied to main (=2.1) and branch 2.0.


add responsive parameter and set to true

Responsive waveform on resize and device orientation change set to true by default
parameter responsive default true, can be set to false to disable responsive waveform.
todo

create onResize method
add event handler
this._onResize = this.onResize.bind(this)

Can you place this in a separate method, not add it to init?

I would define onResize as a regular method, and add this._onResize = this.onResize.bind(this) for the event handler.

@entonbiba does this means that code snippets to resize the waveform on window resize event like you shew me here https://github.com/katspaugh/wavesurfer.js/issues/1067#issuecomment-297217766 will not required anymore ?

@X-Raym yes

@katspaugh I'll update this code on the weekend

@entonbiba Any news from this feature ?
Also, does this feature also adjust the cursor potion in the waveform to match the new waveform ?
Thanks !

@X-Raym This PR was merged added to v2 some time ago. It currently does not support correct handling of position, this should have been fixed in #1210 but since that PR will probably not be merged this needs to be fixed in a seperate PR.

@mspae
ok ! this feature was intersting, I hope it will be bump soon.

@entonbiba Can we merge into es5  or close? Since #1219 this functionality is present in v2.

I'm closing this. Please comment/reopen if anybody disagrees.


Possible race condition when a user force stops a pipeline

Bug description
The following unit test reproduces this issue: https://github.com/ConduitIO/conduit/pull/1797/commits/309833b672d1da8d270ad09e7304655813f9d80c.
Steps to reproduce
https://github.com/ConduitIO/conduit/pull/1797/commits/309833b672d1da8d270ad09e7304655813f9d80c
Version
v0.11.0

Closing this one. Duplicate of https://github.com/ConduitIO/conduit/issues/1891


Grafana 500 error installing oci-datasource plugin

When installing the oci-datasource plugin, an api call is made to load the plugin metadata:
https://grafana.com/api/plugins/oci-datasource
The response includes an invalid url for the repository, which results in an error when attempting to fetch release/1.0.2
https://grafana.com/api/plugins/oci-datasource/versions/1.0.2/download

1 for issue

{
"code": "Internal",
"message": "Couldn't load https://api.github.com/repos/grafana/oci-datasource/zipball/575707096aecfd6ad416630b0654d73fcd1f4d0b: 404 {"message":"Not Found","documentation_url":"https://developer.github.com/v3/repos/contents/#get-archive-link\"}"
}

I'm also getting the same issue, can anyone have any idea on it.

What worked for me were the steps (in my case Linux) described in the github doc readme's. In other words, I did not use the "grafana method" but used "wget":
Download the OCI Grafana Plugin:
wget https://github.com/oracle/oci-grafana-plugin/releases/download/V1.0.4/plugin.tar.
Create a directory and install the plugin: mkdir oci && tar -C oci -xvf plugin.tar and then remove the tarball: rm plugin.tar.

Thanks for the info.


Table 3 looks broken for fractional units

I'm looking at Table 3:

"fractional" was supposed to be an internal change, but based on this table, it looks like we're accepting it from the public options bag.
Also, why is "fractional" the default for microseconds and nanoseconds but not milliseconds?
@ben-allen @FrankYFTang

Note: the spec says

For each row of Table 3 ...
...
d. Let valueList be the Values value of the current row.
...
f. Let unitOptions be ? GetDurationUnitOptions(unit, options, style, valueList, digitalBase, prevStyle).

which calls GetOption with the value list.

Fixed by https://github.com/tc39/proposal-intl-duration-format/pull/197


Restores or adds the "Nothing to see here" cell item

[x] Following
[x] Starred
[x] Discover
[x] Notifications

Merging master into 1.20 makes much of this obsolete - redoing.


infozip (unzip and zip) site down

Infozip site appears to be down.
Might be worthwhile to mirror binaries with CDN service (e.g. bintray) during CI and use that for default bucket?

Moved them to my public keybase directory.

And fixed my unneeded pre_jnstall, too!


chore(engine): refactor pipeline outcome processing

Description
Refactor and simplify the processing of pipeline outcome.
Remove the anti-pattern of returning Option<Result<(), _>> where the only use case for returning Some(Ok(())) was to indicate that the pipeline has reached the max block.
Gets rid of match result statements and uses residual instead.

Preemptive hive run, because why not
https://github.com/paradigmxyz/reth/actions/runs/8718483460

hive results are unchanged


Corrects a broken "What's New in AD FS" link

Continuation of 2ca09f4bae1510a0ba0e91f5cc5dd661d50db132

This change is correct, so I'm going to merge now. Thanks!


Feat: npm i from Github

Include dist dir into repository.
Install plugin including the latest changes from Github.
npm install<EMAIL_ADDRESS>npx cap sync

I would consider including dist a bad practice because someone might forget to build and it just feels wrong.

Neither Capacitors core plugins have the dist folder checked in nor any community plugin I checked.


VXQUERY-209, 207 - Parallel index creation and access and index centralization

This pull request contains fixes for several indexing issues.

VXQuery 207/ 197 - Index Centralization
VXQuery 209 - Parallel index creation and access
VXQuery 220 - Show index query

And fixing following bug.
VXQuery 235 - Exception when executing Aggregation queries.

This also includes the indexing user documentation.
Please review.

Do we have tests for parallel indexing (for 1, 2 and 4 partitions)?

I think any major issue with this PR can be fixed later. Lets just create issues for any of these issues.
For example:
Can you create an issue for updating indexing to support the "source" variable (as I understand it uses absolute paths instead of relative)?

Yes it uses absolute path. I'll create a Jira issue for it.

This code has been merged. PR can be closed.


Rho update tic overwrites runtime tic

Hey guys,
I think there is an issue with the timings. I noticed this because as I was playing around, I sometimes got the same number of iterations (1500ish) but saw a runtime that was ten times as low. I am pretty sure the problem is in osqp.c, in the call to osqp_tic on line 1258 (for updating rho) overwriting the call on 321 (keeping track of runtime). Can you check this and fix it if necessary? Thanks.
Best,
Ben

I think this also explains issue #115 Timings don't seem to be monotonic.

Indeed, this issue is related to https://github.com/oxfordcontrol/osqp/issues/115 that should be fixed in this commit in develop branch. From which language are you using OSQP?

Hey, I was using Matlab, which pull from the master branch. I just checked out the develop branch of osqp_sources, and I can confirm this fix works. Thanks.


Increment version to 0.17.0

Summary of the Pull Request
Increase the version number

We no longer have to change this version number in several places, right?


Upgrade Gradle Enterprise Gradle plugin to 3.3.4

Oh, I didn't release the merge was to the release branch. Was that intentional, given that 6.5 is already out?

Oh, I didn't release the merge was to the release branch. Was that intentional, given that 6.5 is already out?

Yes as this is a patch release I thought it can go out in a Gradle 6.5 patch release if there is one, as master is Gradle 6.6. work correct ?


Create a CONTRIBUTING.md file

Add a CONTRIBUTING.md file with contribution guidelines this repository.

I can create the contributing.md file.

You can tell me if any changes are required.


[Docs] Document the drivers that are supported for minikube.

Current State
Currently there is no note regarding which drivers are supported for minikube.
Desired State
There should be a note regarding which drivers are supported for minikube.
Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation
Layer5 Community Slack

Thank you, @tushar912 👍

@tushar912 given our Slack conversation, I assume that you would like to be assigned this issue, is that right?

Sure I am working. I would make PR ASAP.

@tushar912 is this something that you would still like to complete?


When click close button with form select multiple => turn off my modal

Hi guys!
I have problem when use multiple select in video.
How to fix this?
Thanks!
https://youtu.be/_WCbZByDMWY

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


feat(drag-drop): indicate in dropped event whether item was dropped outside of container

Adds an extra flag on the CdkDragDrop event that indicates whether the user's pointer was over the container when they dropped an item.
Fixes #14136.

@crisbeto API gold needs to be updated.


Add run_constrained pennylane-lightning <0.0a0.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

I'm getting errors such as
error: no member named 'aligned_alloc' in the global namespace

in OSX. @leofang you have not seen this before by any chance?

Yes, I saw that in the previous CI run too. That's why I was wondering if rerendering (which could update the CI infra) would fix it, but it's not. I  don't think the package constraint would trigger a compiler error. I seem to recall that someone recently mentioned an osx compiler issue somewhere...  I am poking around.

It's better to keep the linter happy, otherwise it'd show up every time we update the recipe.

Let's try using a newer OS X SDK to build. Could you add a new file recipes/conda_build_config.yaml and add the the following content? (As suggested in the Gitter chat: https://matrix.to/#/!SOyumkgPRWoXfQYIFH:matrix.org/$1635405605328azMuC:gitter.im?via=matrix.org&via=gitter.im&via=cadair.com)
MACOSX_SDK_VERSION:        # [osx]
  - "10.14"                # [osx and x86_64]

(OS X Arm64 support is a different story, we'll get to that separately...)

@conda-forge-admin, please rerender

Linux is back. I'll try to check the OS X build issue locally, it's a bit confusing to me that the same build env was used in #16 (same SDK, same clang) without problem.

I'm a bit lost here, thanks @leofang . Maybe @BastianZim would have an idea what's going on?

@conda-forge-admin, please rerender

I think between when #16 was merged and now, only the CI runner image was changed. But from the change log I couldn't tell if the system headers were touched. Anyway, I confirmed in #18 that using macOS SDK 10.15 would work. (Building osx packages could be quite annoying sometimes... 😅)
As promised, let's get M1/M2 supported in a separate PR.

Thanks again for your help @leofang .

Hi @leofang , I'm still experiencing the PL version issue you reported here. Is it the point of your PR?

Yes, that's the hope... Normally conda would look for packages with a higher build number, so if we rebuild we shouldn't need to fix the repodata. But in this case I feel that the multiple "buggy" (under-constrained) packages give conda a lot of headache to resolve it properly (mamba does the right job). So fixing the repodata at least on one side should hopefully push us out of this trap.

I am wondering if we should also send a repodata patch for pennylane... Conda hju
$ conda create -n my_pny_env python=3.10 pennylane
...
  pennylane          conda-forge/noarch::pennylane-0.23.0-pyhd8ed1ab_0
  pennylane-lightni~ conda-forge/linux-64::pennylane-lightning-0.24.0-py310h3839ac3_0
...

We can certainly add a version constraint like I mentioned elsewhere:
$ conda create -n my_pny_env python=3.10 "pennylane>=0.29"
...
  pennylane          conda-forge/linux-64::pennylane-0.29.0-py310hff52083_1
  pennylane-core     conda-forge/linux-64::pennylane-core-0.29.0-py310hff52083_1
  pennylane-lightni~ conda-forge/linux-64::pennylane-lightning-core-0.29.0-py310h4f277b2_1
...

but it's kinda ugly...

We can certainly add a version constraint like I mentioned https://github.com/conda-forge/pennylane-lightning-gpu-feedstock/issues/1#issuecomment-1496089447:

Yes, at least what definitely should work, works. But I agree the default should be the latest version.

As promised, let's get M1/M2 supported in a separate PR.

So I submitted https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/4302, and after merging the bot sent #20 for osx-arm64 and #19 for aarch64/ppc64le.

@leofang Would you like to be a member as well since you're the resident CUDA expert. :)

Hi @BastianZim , would you have some time to look at my Kokkos-kernels PR? After it's merged, we could move on to adding the Kokkos and Kokkos-kernels dependencies back into the build as we discussed.

@leofang Would you like to be a member as well since you're the resident CUDA expert. :)

This feedstock does not need any CUDA expert 😄 But it seems I better add myself to fix #20...


Election Day Reminder Request Submitted.

Change the contents of the prompt from #180 to say
Thanks!  We'll email | text | email and text you on election day! depending on what information the user provided.

Fixed in #191


Cannot set left offset value more when reference width

Hello. I'm trying change popper position with offset.fn
I send PR in bootstrap dropdown menu https://github.com/twbs/bootstrap/pull/24208 for use offsets.fn function and use it
const dropdownMegaToggle = $('.dropdown-toggle--mega');
dropdownMegaToggle.dropdown({
    offsetFn: function(data) {
        let popper = data.popper;
        popper.position = 'fixed';
        let buttonOffset = dropdownMegaToggle.offset();
        popper.top = buttonOffset.top + 60;
        popper.left = 1000;
        return data
    }
});

But my offset positioning not work. I can set any negative value for offset left, but can set more when reference.width (it's 90px)

Thank you for your issue.
Unfortunately I can't answer to all the reported issues without a little help from you.
Please, make sure to follow the issues template when you submit a new issue, in this way I will already have all the informations needed to understand the problem and help you with it.
I'm closing this issue for now, but feel free to open a new one making sure to follow all the instructions!
Thank you for your understanding.


Move to StatefulSets

I'm working on moving the example deployments to Stateful Sets instead of Replication Controllers. This should simplify the setup and provide more stability. From my initial testing, it seems like we still need a sidecar to do the configuration though.
Right now, the Makefile creates the individual volumes, a RC, and a Service per MongoDB Replica, along with a "master" service for all the MongoDB pods.
Under the new system, we can use a Volume Provisioner to dynamically create disks for a TON of platforms out of the box. Then we use a single Stateful Set and a single headless service. Each MongoDB replica will have its own DNS address and volume created automatically.
As an added bonus, using a Volume Provisioner means there is no longer a need for different steps for different cloud providers, etc.
Fortunately, I think this can be done with no code changes to the actual sidecar. The current method the sidecar uses to find the other members of the Replica Set should still work.
I'm working on a PR, but wanted to run it by the community first for comments.

@thesandlord That's great thanks for that PR ! If you need it, i can add the AzureDisk example :) i'll try that StatefulSet right now, and give you my feedback then.

@thesandlord Ok, so i tested with a Kubernetes StorageClass using AzureDisks, it's working perfectly ! My 3 Pods are coming up one after another and my PVCs are created and mounted.
I face one problem, witch is not related to the StatefulSet but more to the sidecar Container, let me explain :

If you delete de StatefulSet the PVCs are still provisioned on the cluster and that's great !
By the way when i redeploy my StatefulSet the pods are retrieving their PersistentVolumes and they found that the Mongo replicaSet is already configured from the previous StatefulSet on /data/db.

2017-01-06T12:58:24.232522253Z 2017-01-06T12:58:24.232+0000 W REPL     [replExecDBWorker-1] Locally stored replica set configuration does not have a valid entry for the current node; waiting for reconfig or remote heartbeat; Got "NodeNotFound: No host described in new configuration 5 for replica set rs0 maps to this node" while validating { _id: "rs0", version: 5, protocolVersion: 1, members: [ { _id: 0, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 }, { _id: 1, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 }, { _id: 2, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 } ], settings: { chainingAllowed: true, heartbeatIntervalMillis: 2000, heartbeatTimeoutSecs: 10, electionTimeoutMillis: 10000, catchUpTimeoutMillis: 2000, getLastErrorModes: {}, getLastErrorDefaults: { w: 1, wtimeout: 0 }, replicaSetId: ObjectId('586f85383bd8fcea442c5ba9') } }
2017-01-06T12:58:24.232546053Z 2017-01-06T12:58:24.232+0000 I REPL     [replExecDBWorker-1] New replica set config in use: { _id: "rs0", version: 5, protocolVersion: 1, members: [ { _id: 0, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 }, { _id: 1, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 }, { _id: 2, host: "<IP_ADDRESS>:27017", arbiterOnly: false, buildIndexes: true, hidden: false, priority: 1.0, tags: {}, slaveDelay: 0, votes: 1 } ], settings: { chainingAllowed: true, heartbeatIntervalMillis: 2000, heartbeatTimeoutSecs: 10, electionTimeoutMillis: 10000, catchUpTimeoutMillis: 2000, getLastErrorModes: {}, getLastErrorDefaults: { w: 1, wtimeout: 0 }, replicaSetId: ObjectId('586f85383bd8fcea442c5ba9') } }
2017-01-06T12:58:24.232552153Z 2017-01-06T12:58:24.232+0000 I REPL     [replExecDBWorker-1] This node is not a member of the config
2017-01-06T12:58:24.232555853Z 2017-01-06T12:58:24.232+0000 I REPL     [replExecDBWorker-1] transition to REMOVED
2017-01-06T12:58:24.232558753Z 2017-01-06T12:58:24.232+0000 I NETWORK  [replExecDBWorker-1] legacy transport layer closing all connections

That's resulting in 3 POD with SECONDARY State with no Master elected, it's seems normal since the Sidecar is not deleting Mongo ReplicaSet config, if one was already configured previously...
I'll try to find a way to maybe handle that case on the SideCar because with a StatefulSet it will be common to update or reschedule Pods...

Yeah I'm running into a similar problem. Looks like a bug in the sidecar. Interestingly, if you kill the pods slowly (i.e one at a time), things seem to work fine. The container spins back up and rejoins the set. It only seems fails if you kill a lot at once (or all of them at once).

So here is the error I'm getting:
{
	"state" : 10,
	"stateStr" : "REMOVED",
	"uptime" : 151,
	"optime" : {
		"ts" : Timestamp(1483730812, 1),
		"t" : NumberLong(2)
	},
	"optimeDate" : ISODate("2017-01-06T19:26:52Z"),
	"ok" : 0,
	"errmsg" : "Our replica set config is invalid or we are not a member of it",
	"code" : 93,
	"codeName" : "InvalidReplicaSetConfig"
}

In the sidecar code, worker.js looks like it is checking for error code 94 only, not 93.
Going to change it to both and see what happens

@thesandlord That should be a solution ! Waiting for your feedback.

Unfortunately there is a assumption in the "notInReplicaSet" function that if there is a replica set created, MongoDB will eventually fix itself. In this case, all the nodes are broken but still in the set, so we need to force a reconfigure.
Because there is no primary, I'm thinking the easiest way is to just add the newly spawned pods to the replica set, and have MongoDB do another election. There is already code to clean up replica set members without pods, so there should be no issue there.
If I'm understanding how this works correctly, this should work.

It kind of worked... the nodes are no longer stuck in the OTHER status. Two become secondary and one becomes primary as expected. The problem is the secondary nodes don't connect to the primary nodes, they are still connected to the three non-existent ones...
Not sure what the solution is, open to ideas!

Got it to work! I was running the reconfigure on all the nodes which was a mistake. Changed it to only run on a newly elected primary, and have the primary add the secondary nodes as well. Looks like it recovers without issue!
In any case, this is probably a fringe scenario because having all the nodes go down at once is pretty unlikely.

Thanks for all the work on this, good job guys. I'll close this issue once @theobolo has given it a test on Azure.
On that note, an azure provisioner yaml file as @thesandlord mentions would be welcome 😄

@thesandlord @cvallance So i tested the case when we redeploy the StatefulSet on the same persistentVolumes and that working well !
The new pods are running the dry election and a new master is elected, the DB Data wasn't lost and my DB is not corrupted, so far it's working fine !

Awesome, thanks guys. Closing this issue now.


ERC20/VRC20 token transaction history doesn't stay between app launches

In the wallet page, you can see a list of transactions if you click on VLX tokens. But after the app is closed and reopen, ERC20 transaction history's gone.

@symblox ,
I'm not clear about this issue, could you please take some screenshots?


Add externalTrafficPolicy setting to services

This PR adds an option in the chart crowdsec to define the externalTrafficPolicy for agent and lapi services.
This setting is useful for preserving source IP when using services of type LoadBalancer and NodePort.

Hi @keliansb, thanks for the contribution.
You need to bump the Chart version to generate automatically the new chart release including this feature.

Hi @he2ss, the Chart version has been updated.

@he2ss is everything good in this pull request/commit? I would like to make a new pull request to add an option in the chart to mount extra volumes, and I wanted to know if I was respecting your requirements/best practices before doing that.

Hi @keliansb, all is good, thanks.
Don't hesitate to add another PR.


wip: new(libs): replace elfutils/libelf with elftoolchain/libelf

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build
/area libscap-engine-modern-bpf

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
I would like to propose replacing elfutils/libelf with elftoolchain's libelf. The latter is an implementation of the libelf spec which is BSD licensed and therefore can be statically linked in Falco and its dependencies. The current situation has a few main disadvantages:

Having a dynamically linked only libelf prevents us from having a static build which we used to have with musl
It adds a dependency for our users, making the use of a binary package more complex and potentially failing
It adds additional issues for clients of the Falco libs (not just Falco) because their build process is more complex in turn
It may create additional issues if we try to switch to a different dependency management solution (e.g. vcpkg)

The proposed solution is based on Elftoolchain which I've adapted to work with libbpf and Falco libs.
So, how do we build this library? Upstream, the library is obtained via svn and built with BSD Make (which is different from GNU Make) and m4. Since none of those are currently dependencies for Falco or libs, I would propose NOT to require falco libs client to install those. So, I've generated the m4 files (hoping that they're not arch dependent, if they are I'll find a solution) and wrote CMake files just for libelf. In addition, some patches are in order (I don't know yet if they can be upstreamed but we may explore that). There is a README file that explains these steps in detail.
As you can see the PR is huge because it includes everything in the libs repo. We can discuss one of the following:

leave it as-is (functional, but bloated repo)
create another repo for our patched elftoolchain and download it from here (a bit more cumbersome, makes it harder to import patches because they'd require a release)
remove everything from libs which is not required for libelf (less bloated repo, but would be harder to update)

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
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 [LucaGuerra]

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

692c9e2 temp: debug msg for ci

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Hey @LucaGuerra! Thanks for putting this together, I just have some thoughts and questions I'd like to share with you.
My main concern is with maintenance of this new vendored dependency, while languages like Go and Rust have integrated tools to do this kind of thing (see go mod vendor and cargo vendor), C/C++ and/or CMake, do not. This means the dependency will need to be manually updated by following the steps described in the added README. While they are not terribly complicated, it is additional work that needs to be accounted for, but we might also be able to put some automation behind this process as well, so not terribly concerning. That is, until we get to the custom patches, this does seem like a bad idea, since updating the dependency will require re-applying the patches and that can be painful (speaking from experience here), if we can upstream the changes as you mentioned, all the better, but if we can't, a random commit on the main branch of the repo is really easy to miss/lose, so I would recommend keeping the patches on a separate diff file, or some other form of backup, just in case.
Another concern with this approach is how do we track what version of elfutils we are using? I haven't looked super deep into the PR, but I've not seen any mentions of a specific version being used. While this might not be terribly concerning now, it would be nice to have a way to know this in the future for multiple reasons (checking for CVEs, bugfixes, etc..).
Finally, for the proposed approaches:

leave it as-is (functional, but bloated repo)

I don't like this one, that is all I have to say 🤣

create another repo for our patched elftoolchain and download it from here (a bit more cumbersome, makes it harder to import patches because they'd require a release)

This one is my favorite. Correct me if I'm wrong, but since we are not planning to redistribute this library, the release could just be a tag on the new repo, right? We don't need to do anything fancier than that. Worst case, I guess we need to build the archive file for it and publish it as a github package?

remove everything from libs which is not required for libelf (less bloated repo, but would be harder to update)

This is even more work to keep track of changes, I don't like it either.
Anyways, sorry about the mile long comment! 😅

Closing in favor of #2175


Headphones are not working after long sleep

After a night sleep(close the lid and without charging) and replug the headphone, I can hear about a half second the sound and it stops working (and if I replug, I can hear a half-second sound again).
Reboot fix this, but it's annoying to do this every day(and so what's the point to sleep....)
I have double checked that ALCPlug fix is installed(did a reinstall to make sure), I can see it on /Library/LaunchAgents

Well, after a bit of debugging, I found out that Boom 3D was causing this(like #104 ) so after that, it should work fine.


Monomethylated Histidine Fixes

There exists a problem where UniProt's ptmlist points to an incorrect entry because the correct entry did not exist in PSI-MOD.  This is described in detail in #20.  This PR closes #20.

@rfellers Please take a look at this.  The issue was that UniProt's PTMList did not have a correct PSI-MOD entry to point to.  Once this is created, I will contact UniProt and have them update their PTMList.

thanks @pmt706! Yes, this would address #20 nicely.


Where can I get the C VERSION of go-lang source code?

Starting with Go 1.5 the compiler and runtime were entirely written in Go.
You can find the source code for 1.4 (which requires a C compiler to build) here: https://github.com/golang/go/tree/release-branch.go1.4/

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For asking questions, see:
Stack Overflow with questions tagged "go" or "golang"
The Go Forum, a web-based forum
Gophers Discord "Get live support and talk with other gophers on the Go Discord"
Gophers Slack, use the invite app for access. The #general channel is a good starting point.
Go Community on Hashnode with questions and posts tagged with "go"
The golang-nuts mailing list
Subreddit for Go
IRC channel #go-nuts on Libera


Added sessionStorage support

I was anticipating the merge of pull/40. However, since it hasn't been merged yet, I thought the PR might have been discontinued. Given that others are also interested in this feature, I decided to implement it myself.
Previously, I created a PR but I noticed a few issues when integrating it into my own project. As a result, I decided to close that PR and have now raised this one with the necessary changes to ensure it works properly.
I hope we can get this feature merged soon, as it would be quite useful. If there are any further changes needed, please do let me know, as I am more than happy to assist with further contributions.

Hi @Anon0x19 the only issue we faced is on the NextJs when the component is rendering on the server side! We are working on it, and we will be closing this Request with this week without any delay

Hi @Anon0x19 the only issue we faced is on the NextJs when the component is rendering on the server side! We are working on it, and we will be closing this Request with this week without any delay

Thank you for the quick response @sushinpv! This fix should work on NextJs. I just tested it using nextjs-react-secure-storage-example.
The only change I made was to modify the import statement from import secureLocalStorage from "react-secure-storage"; to import {secureLocalStorage} from "react-secure-storage";. Additionally, I used yarn link and yarn link react-secure-storage with the branch in this PR

Thanks a lot @Anon0x19 , Let me go through all the test cases and if everything looks good we will be able to release this new version within this week

Thanks a lot @Anon0x19 , Let me go through all the test cases and if everything looks good we will be able to release this new version within this week

Awesome! Thank you again for getting back to me so quickly. You were right about the issue with certain Next.js versions. I tested it using 14.2.5 and 13.4.13 and it seems to work on both without issues now.
By the way, for react, I had to change the usage from i.e secureLocalStorage.setItem("number", 12); to secureLocalStorage?.setItem("number", 12); to avoid errors.
Furthermore, it seems the issue arose from new secureStorage(localStorage), but I've now changed it to typeof window !== 'undefined' && typeof window.localStorage !== 'undefined' ? new secureStorage(window.localStorage) : null;

This will be breaking change for the existing users, and integrations, I'll make the necessary changes and planning to close this in next two days


typo in installation script

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e57d5d5). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #215   +/-   ##
=========================================
  Coverage          ?   75.07%
=========================================
  Files             ?      126
  Lines             ?    13430
  Branches          ?        0
=========================================
  Hits              ?    10083
  Misses            ?     3347
  Partials          ?        0


Terminal cursor shape or color not correctly restored when closing a terminal

Cursor shape or colors is not correctly restored when killing a terminal.
This is unrelated to #2938 which is also about cursor color.
Steps to reproduce:

start vim with 3 terminal windows:

$ vim --clean -c term -c term -c term

in the 1st (top) terminal, change the cursor color to red:

$ echo -ne "\033]12;red\007"

in the 2nd (middle) terminal, set the cursor color to green:

$ echo -ne "\033]12;green\007"

in the 3rd (bottom) terminal, set the cursor color to green:

$ echo -ne "\033]12;blue\007"

Observe that the cursor color is properly restored when
switching from one terminal to another (by clicking
in each terminal window for example). Good so far.

Now go in the 2nd terminal (with green cursor) and
press CTRL-D to close that terminal.

Observe that the cursor is now in the terminal where we
had set cursor color to blue, but the cursor color is actually
green (bug!).
The color cursor gets properly restored to blue when
switching to the other terminal (the one with red cursor)
and then switching back to the other terminal where cursor
was meant to blue.
It's not specific about cursor color.  The same bug
happens when different terminals have different cursor shape
or blinking attribute.
I can reproduce the bug with vim-8.1.10 at least in:

xfce4-terminal <IP_ADDRESS>
xterm  (XTerm(330))

Interestingly, the bug does not happen with gvim (GTK3) i.e.
if I start vim with:
$ vim -f -g -c term -c term -c term

Dominique wrote:

Cursor shape or colors is not correctly restored when killing a terminal.
This is unrelated to #2938 which is also about cursor color.
Steps to reproduce:

start vim with 3 terminal windows:

$ vim --clean -c term -c term -c term

in the 1st (top) terminal, change the cursor color to red:

$ echo -ne "\033]12;red\007"

in the 2nd (middle) terminal, set the cursor color to green:

$ echo -ne "\033]12;green\007"

in the 3rd (bottom) terminal, set the cursor color to green:

$ echo -ne "\033]12;blue\007"

Observe that the cursor color is properly restored when
switching from one terminal to another (by clicking
in each terminal window for example). Good so far.

Now go in the 2nd terminal (with green cursor) and
press CTRL-D to close that terminal.

Observe that the cursor is now in the terminal where we
had set cursor color to blue, but the cursor color is actually
green (bug!).
The color cursor gets properly restored to blue when
switching to the other terminal (the one with red cursor)
and then switching back to the other terminal where cursor
was meant to blue.
It's not specific about cursor color.  The same bug
happens when different terminals have different cursor shape
or blinking attribute.
I can reproduce the bug with vim-8.1.10 at least in:

xfce4-terminal <IP_ADDRESS>
xterm  (XTerm(330))

Interestingly, the bug does not happen with gvim (GTK3) i.e.
if I start vim with:
$ vim -f -g -c term -c term -c term

I can reproduce it.  Interesting...  I had trouble stepping through this
with the terminal debugger, since it also suffers from this problem.
Turns out that when switching buffers while inside terminal_loop() the
in_terminal_loop variable gets the wrong value.  It should bail out of
the loop in that case.
--
ARTHUR:    Be quiet!

ARTHUR:    I am your king!
OLD WOMAN: Well, I didn't vote for you.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Empty string does not fail schema validation with JRuby like it does with MRI

With Nokogiri 1.5.5 on JRuby validating an empty document does not result in an error, while with MRI it does.
If you try the below snippet in MRI and JRuby (XML schema taken from #642, perhaps there is some relation between the issues -- atleast for JRuby):
require 'nokogiri'

xsd_data = <<-XSD
<?xml version="1.0" encoding="UTF-8"?>
<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema"
  targetNamespace="http://www.example.org/contactExample"
  xmlns:tns="http://www.example.org/contactExample"
  elementFormDefault="qualified">
  <xs:element name="Contacts"></xs:element>
</xs:schema>
XSD

xsd = Nokogiri::XML::Schema.new(xsd_data)

empty_xml = Nokogiri::XML("")

xsd.validate(empty_xml)
#=> MRI: [#<Nokogiri::XML::SyntaxError: The document has no document element.>]
#=> JRuby: []

xsd.valid?(empty_xml)
#=> MRI: false
#=> JRuby: true

I've create #2641 which includes a pending (on JRuby) test that captures the issue.
I looked into this and I wasn't able to find the root cause -- but I traced enough to verify that javax.xml.validation.Validator isn't calling the SchemaErrorHandler at all in this case. It's not clear to me if that's a bug in the implementation or not, but I do know that I need help from someone who can dig deeper into the Java internals.

PR #3260 will fix this.


Update logo of Reliably on OAuth flow

Specificallly here https://github.com/reliablyhq/cli/blob/2884128d43f9b66ec3839d6c4aead5d69664b9b0/cmd/reliably/auth/auth_with_provider.go#L280
Change from

to

cc @mperrien


Error in the code

Check on https://discord.gg/wtGEjwGsd

I'm not joining some random discord server. Post the details here.


Vscode tailwindcss intellisense not working with epic stack

The Tailwindcss intellisense is not working for the epic stack repository.  I suspect the problem is the tailwind config file, currently our config file is written in typescript whereas the extension supports config file in  javascript. In case I change the config file to javascript the intellisense starts working.

Sounds like a bug in the tailwind extension. Could you open an issue over there?

Opened the issue in tailwindcss intellisense: https://github.com/tailwindlabs/tailwindcss-intellisense/issues/813


How to detect objects Real Time

Description
Pytorch is amazing,thank you for the work you are doing.
I'm trying Object detection example from tutorial , Can you give an example of real time use "onFrame" of CAMERA?
Website
https://pytorch.org/live/docs/tutorials/get-started/

Hey @dongdv95,
Take a look at the following example: ObjectDetectionExample.tsx. In this example, you can learn how to perform object detection with DETR. The problem is that it's a transformer so it's not meant to work on real-time because it has too many parameters.
You can use a "lighter" model (with less parameters) like the Faster R-CNN (for the moment, you may experiment some issues if you try to use YOLO, that's why I recommend you that model). You can learn how to import this custom model in the following tutorial: PyTorch Live: Prepare Custom Model.
To import the model in the make_models.py file you can use torchvision:
torchvision.models.detection.faster_rcnn

You can see more details about this model in the following link: torchvision faster_rcnn
Finally, to adapt the example so it works in real-time you can follow the Image Classification tutorial which covers how to perform image classification in real-time.
Just to summarise, if you only want to see an example of a real-time use of "onFrame" you can just take a look at the Image Classification tutorial and you will see. But I also wanted to show you an example that implements the drawing of the bounding boxes and how you can import a custom model as I think that it will speed up your creation of the app.
Best of luck!

Hey @dongdv95,
Take a look at the following example: ObjectDetectionExample.tsx. In this example, you can learn how to perform object detection with DETR. The problem is that it's a transformer so it's not meant to work on real-time because it has too many parameters.
You can use a "lighter" model (with less parameters) like the Faster R-CNN (for the moment, you may experiment some issues if you try to use YOLO, that's why I recommend you that model). You can learn how to import this custom model in the following tutorial: PyTorch Live: Prepare Custom Model.
To import the model in the make_models.py file you can use torchvision:
torchvision.models.detection.faster_rcnn

You can see more details about this model in the following link: torchvision faster_rcnn
Finally, to adapt the example so it works in real-time you can follow the Image Classification tutorial which covers how to perform image classification in real-time.
Just to summarise, if you only want to see an example of a real-time use of "onFrame" you can just take a look at the Image Classification tutorial and you will see. But I also wanted to show you an example that implements the drawing of the bounding boxes and how you can import a custom model as I think that it will speed up your creation of the app.
Best of luck!

Thanks for ans

@dongdv95 when do you get this error? When trying to run the model in a PyTorch Live app?

@dongdv95 when do you get this error? When trying to run the model in a PyTorch Live app?

Hi @clarksandholtz, When I make model  (model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)) , I get this error , I'm not sure my  live.spec.json is correct.

@dongdv95 if this is happening when you are making the model in Python, then the live.spec.json isn't being used at all. The live.spec.json is only used by PyTorch Live in mobile apps. If you remove all of the code that references the live.spec.json you still get the same error.
Looks like there is an issue with the tracing, but I'm not sure what it is.

Thanks @clarksandholtz , I test with another model
( torchvision.models.detection.ssd300_vgg16(pretrained=True) ) and
(FasterRCNN(backbone= torchvision.models.mobilenet_v2(pretrained=True).features)), I make model success
but when run inference on Android with error
Error: Following ops cannot be found. Check fburl.com/missing_ops for the fix.{torchvision::nms, } ()
Exception raised from print_unsupported_ops_and_throw at ../torch/csrc/jit/mobile/parse_operators.cpp:65 (most recent call first):
(no backtrace available)

I’m grateful for your assistance

Closing this issue out since the packer / unpacker approach was removed with 0.2.2. Check out the PyTorch-inspired API in the README as an alternative that avoids these issues: https://github.com/facebookresearch/playtorch/tree/main/react-native-pytorch-core#example-usage
As for the unsupported operation, I'd try it again. We have updated the versions of PyTorch and Torchvision used in the PlayTorch app (side note: we have rebranded from PyTorch Live to PlayTorch), and what you need may now be supported. Here is a starting-off point for using the PlayTorch app: https://playtorch.dev/docs/tutorials/get-started/

Hello @aaronespasa,  Where can i find example of a real-time use of "onFrame"?
This Image Classification tutorial link is unavailable and I can't find real-time example at image classification example link of playtorch tutorial.


Fix RESTEASY-2526 by using asyncutil lib for async loops

Proper fix for #2342, as described there.
This PR introduces a new (tiny) dependency that is required to properly implement loops using CompletionStage. I'm not sure if it's best to add a jboss modules for it, or include it in the jaxrs-all module.
Need guidance from @asoldano about this.

Thanks a lot @FroMage . Yes, we need a new module, to be declared private. I'm reviewing the PR soon.


fix: breaking docs CI

SUMMARY
The CI checks are failing in master for changes which relate to the documentation. This PR fixes the lint issues.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
CI.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

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

@@            Coverage Diff             @@
##           master   #11156      +/-   ##
==========================================
- Coverage   65.84%   62.21%   -3.64%
==========================================
  Files         827      436     -391
  Lines       39051    14560   -24491
  Branches     3673     3673
==========================================
- Hits        25715     9059   -16656
+ Misses      13229     5320    -7909
- Partials      107      181      +74

Flag
Coverage Δ

#cypress
?

#javascript
62.21% <ø> (ø)

#python
?

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

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

Actually there seems to be another issue related to https://github.com/gatsbyjs/gatsby/issues/26941. @rusackas et al. would you be able to look into this?

Yeah that one drove me crazy. gatsby-plugin-mdx doesn't like the ASF header/comment on docs/README.md, and RAT (ASF-license scanner/validator) doesn't want to ignore that file (looks like a bug where it doesn't want to .rat-exclude a README file). After going down the rabbit hole unsuccessfully I decided that the easy hack forward is to rename README.md to README.txt.
I'm having other issues with Fossa going MiA my PR that does that (amongst other things)
https://github.com/apache/incubator-superset/pull/11154

I managed to merge my PR that addresses the issues mentioned here.


Fixed #31529 -- Added support for serialization of pathlib.Path/PurePath and os.PathLike in migrations.

ticket-31529

Rebased targeting 3.2.

@pope1ni Thanks :+1:
I squashed commits, rebased from master, and simplified test_serialize_path_like() with os.scandir() and DirEntry.

Perfect 🙂 I had trying to manually instantiate an instance of os.DirEntry stuck in my head which isn't possible. Didn't think to just actually use os.scandir()!

Merged in 074844e9479a9ee1420685b36189aa2881289993.


mdTextareaAutosize doesn't set amount of rows to one in Firefox

Similar #4852
Bug, feature request, or proposal:
Bug
What is the expected behavior?
Textarea should be resized to a single line.
What is the current behavior?
Resize doesn't shrink textarea to a single line, stops at minimum of two lines.
What are the steps to reproduce?
Plunker template: https://plnkr.co/edit/BGAwyS?p=preview
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 4.2.6
Material: 2.0.0-beta.8
OS: Win10
Browser: Mozilla Firefox 54.0.1
Is there anything else we should know?
In Chrome everything works fine, bug only in Firefox.

I tried to reproduce it with Firefox 54.0.1 but I was not able to see the issue.
Is there anything specifically I need to do, in order to see the issue?

I don't now how it possible, but neither i now have it.


Does the utility work if Redshift audit logging is enabled to CloudWatch?

One of my customer is planning to use this utility ? They have the audit logging enabled to CloudWatch .  Do they have to change audit logging to S3 to use this utility ?

They need to enable audit logging in their Redshift cluster, when ch logs
queries automatically in S3. The CW audit logging you mentioned might be
enabled for CloudTrail, which is different than redshift audit logging
On Thu, Aug 25, 2022, 6:13 AM akshayar @.***> wrote:

One of my customer is planning to use this utility ? They have the audit
logging enabled to CloudWatch . Do they have to change audit logging to S3
to use this utility ?
—
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/amazon-redshift-config-compare/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADYJN6BE2IFVL7BXLOC7Z2DV25BM7ANCNFSM57SP6GUQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

They have enabled https://docs.aws.amazon.com/redshift/latest/mgmt/db-auditing.html#db-auditing-cloudwatch-provisioned . The user activity logs go to CW and not S3.
Is there a workaround for them or they have to enable https://docs.aws.amazon.com/redshift/latest/mgmt/db-auditing.html#db-auditing-manage-log-files.


How to work with static files ?

Hello,
I am setting up wpack.io in a new Wordpress Starter with Bedrock & Timber. I've extended the Webpack config to add a first plugin : SVG Spritemap Plugin. At the moment everything is perfect and fast :)
Now the next step is to handle images / static files, but i dont know in which way to go ^^ and i found nothing in the documentation :/
I would like to view static files during development and have a compression when compiling to production.
I was thinking about a static folder in src/theme (or maybe better at the theme root) and extend Webpack config to copy+compress it in dist/ folder when compiling for production. You think it is good or there is a better way ?
EDIT: hum, no it's not a good solution because it would be necessary to change the path to /dist in all templates 😅
Well i guess, i'm going to keep all the static files in a static folder at the root of the theme and compress files inside when compiling to production..
Thanks :)

Hello,
When you import a static file like
import svgURL from '../static/mystuff.svg'

It is automatically copied to the /dist during build. So you don't have to change paths or copy anything. You just need to make sure you have proper loader for the type of file you are importing.

Ok great ! And what i have to write in HTML files when i want to display an image ?

Hello,
Sorry for not being clear. The above is what you'd do in a JS file. For HTML or PHP, you have rely on server side logic to get the static asset. This library is for managing front-end things only.

@swashata ? Makes no sense. I'd like some help on this one as well; "How can one fetch a static asset, like an image, in the HTML?"

This seems to do the trick: https://github.com/swashata/wpackio-enqueue/issues/4


friends not working on Windows

When I try to rebuild-leveldown on windows, it have to replace $(../../version.js) with the actual version, and then it gets stuck when trying to build with msbuild.

same error!

Sorry, looks like #153 is causing some issues. May need to revert the last pull request -- I'll look into it.

This should be resolved now.

good


breaking: bump version 5.0.0-dev

Motivation and Context
Bump for Major Release

Will turn green after #626 is merged in.. But this PR will be merged in first.


refactor(Avatar): Remove deprecated shadow and refresh API

Overview

Adds new sizes and responsiveness.
Removes the shadow prop entirely as new Testimonials don't use it.
Adds layout system props.
Removes extraneous markup.
Adds new stories for sizing

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.78-alpha.cd1ae6.0@codecademy/gamut-labs@19.26.7-alpha.cd1ae6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6141e2ef5015b6294ae8f7b9--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.78-alpha.1de87e.0@codecademy/gamut-labs@19.26.7-alpha.1de87e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61435a9958950c1be74afbcc--gamut-preview.netlify.appDeploy Logs


kvm/bridge: Allow Link Local Cidr (cloud0 interface) to be configured

Description
There are certain scenarios where the <IP_ADDRESS>/16 subnet is used for different
purposes then CloudStack on a hypervisor.
Once of such scenarios is a BGP+EVPN+VXLAN setup using BGP Unnumbered where the
<IP_ADDRESS> address is used by Frr/Zebra BGP routing to send traffic to the
neighboring router.
The following settings can be changed in the agent.properties (default values added):
control.cidr=<IP_ADDRESS>/16
Make sure the global setting 'control.cidr' matches the values defined in the agent.propeties!
In the future the mgmt server can send this parameter to a KVM Agent on startup, but at the moment
this framework is not in place and thus these values can't be send to the Agent in a proper manner.
Fixes: #3488
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?
Ran local test on our cloud environment and verified cloud0 now has a different address.

root@hv-138-a05-23:~# ip addr show cloud0
58: cloud0:  mtu 1500 qdisc noqueue state UP group default qlen 1000
    link/ether fe:00:a9:fe:f0:c9 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/20 scope global cloud0
       valid_lft forever preferred_lft forever
    inet6 fe80::8c3c:82ff:fe5a:77c/64 scope link
       valid_lft forever preferred_lft forever
root@hv-138-a05-23:~#

In addition, also tested and verified: systemvms do get their ips from within the control.cidr and they are fully functional.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-141

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-172)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 29838 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3500-t172-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_04_rvpc_internallb_haproxy_stats_on_all_interfaces
Error
201.73
test_internal_lb.py

@wido cc @GabrielBrascher - can you review the oustanding comments. Tests LGTM.
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-168

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Left some remark, some changes I'll need to re-review, meanwhile I'll wait for regression tests to come back.

Trillian test result (tid-191)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35446 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3500-t191-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_05_rvpc_multi_tiers
Failure
417.48
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
446.76
test_vpc_redundant.py

I think I addressed the comments above. Some seem to be a coding style thing but not really a technical change.
Let me know if this PR is LGTM

@wido can you comment on the IvsVifDriver removal?

@rhtyd I did? The file is not referenced anywhere. I also think that it was a typo when the file was created.

@wido nope, that's not a typo it's a driver for the indigo virtual switch introduced in https://github.com/apache/cloudstack/pull/151 and it's not used anywhere because this is something one configures for the libvirt.vif.driver= in the agent.properties file. Kindly revert that change.

@rhtyd I see! The file is back with a few modifications.
@nvazquez I also took another look at your comments and they should be adressed now.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✖centos7 ✔debian. JID-179

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-183

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-216)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39097 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3500-t216-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_05_rvpc_multi_tiers
Failure
433.07
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
462.51
test_vpc_redundant.py


updateStatus function typo

function updateStatus(message) {
    var statusInfo = $('#status');
    statusInfo.innerHTML += message + "
";
}

should be:
function updateStatus(message) {
    var statusInfo = $('#status');
    statusInfo[0].innerHTML += message + "
";
}

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

ID: a5f7a698-efa2-3584-6ea0-85fa3718718f
Version Independent ID: cef6669b-bfc0-840a-ea4e-ebc2ae5cc58a
Content: Get the whole document from an add-in for Word - Office Add-ins
Content Source: docs/word/get-the-whole-document-from-an-add-in-for-word.md
Product: word
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for reporting this @georgesfarah4.
@ElizabethSamuel-MSFT, can you please investigate and update the documentation as needed? Thanks!

@georgesfarah4 Thanks for letting us know about this. The fix should be published soon.
Thanks.


[Suggestion] Multiselect - add query from datasource option

Other than adding value selections manually, add options to get the list from a query.
For example:
"SELECT city FROM Country"
and will return list of city as the selections in thr multi-select.

We've also looked at this. One way to do this is to reference a dashboard variable (like $dash_var) that does this query, assign it to a JS object, and then append it to the form elements (elements in JS) of the dropdown like https://volkovlabs.io/plugins/volkovlabs-form-panel/dynamic/.
The issue only issue I have with this is that is that you can only feed in dashboard variables after the first initial request. It would be nice if this could happen on dashboard load.

Update: You can use onChange(elements) where you've modified you elements to map to the Grafana dashboard vars and they will appear initially.

@henjoe @jacksongoode Thank you for the feature request. It's possible to do it with custom code as described in the documentation: https://volkovlabs.io/plugins/volkovlabs-form-panel/dynamic/
We plan to implement the UI options in the upcoming release, similar to the Initial Request using Query.

For those who needs more examples (like me), this code loads list of options from a data source:
const hosts_ds = data.series.find((serie) => serie.refId === "A");
if (hosts_ds?.fields.length) {
  const hosts =
    hosts_ds.fields.find((f) => f.name === "hosts").values.buffer ||
    hosts_ds.fields.find((f) => f.name === "hosts").values;

  elements = [{
    id: 'active', title: 'Host', type: 'select',
    options: hosts.map((host) => {
      return { id: '', value: host, label: host, type: 'string' }
    })
  }];

  onOptionsChange({ ...options, elements });
}

How the code is connected to the data source: https://youtu.be/RSVH1bSBNl8?t=53

@Radrik5 Thank you for sharing the example until we implement this feature.


Promote NotBlankString's deprecations

Description
Promote deprecations declared in io.github.kotools.types.string.NotBlankString.
Checklist

[x] Implement.
[x] Test.
[x] Update Work in progress section in changelog.

Done.


Add Federated Credentials support for GitHub Action (Azure/static-web-apps-deploy)

Is your feature request related to a problem? Please describe.
When using static-web-apps-deploy i cannot use the more secure federated credential for deployment authentication (as supported by deployment targets such as Azure App Service), but have to use azure_static_web_apps_api_token instead, which is less secure for obviouse reasons.
Describe the solution you'd like
support the federated credentials via az login - similar to Azure/webapps-deploy
Describe alternatives you've considered
I was considering using a workarround as described here: https://www.cloudwithchris.com/blog/using-oidc-github-actions-azure-swa/  however piping secret tokens into GITHUB_ENV is propably even less secure than storing them in the Github secrets store.
Additional context

Hi @MichaMican, I'm noting the feature request and we'll consider this in future plannings. Thanks for the feedback!


[Docs] NS_ENUM exported to JS, used as method arguments for native modules, must use RCT_ENUM_CONVERTER in RCTConvert class extension

I think this section of the docs needs to be further expanded to explain the use of RCT_ENUM_CONVERTER.
In v0.5.0, I was able to export NS_ENUM defined constants, and use them as method arguments. Example code below.
Enum Definition:
    typedef NS_ENUM(NSInteger, MyEnumType)
    {
        kEnumOne,
        kEnumTwo,
        kEnumThree,
    };

Constants:
    - (NSDictionary *)constantsToExport
    {
        return @{@"OnError": @{ @"enumOne" : @(kEnumOne),
                                @"enumTwo"  : @(kEnumTwo),
                                @"enumThree"   : @(kEnumThree)}
                      }
    };

Method Definition:
RCT_EXPORT_METHOD(doSomethingWithEnum:(MyEnumType)enum
                           completion:(RCTResponseSenderBlock)callback)

In v0.7.0, this failed with the following log:
Unknown argument type 'MyEnumType' in method -[MyNativeModule
doSomethingWithEnum:completion:]. Extend RCTConvert to support this type.

I was only able to puzzle the solution out after reading through the internal React Native code for quite some time. (Create a class extension for RCTConvert, use RCT_ENUM_CONVERTER to implement a converter method.)
    @implementation RCTConvert (MyEnumExtension)
    RCT_ENUM_CONVERTER(MyEnumType, (@{ @"enumOne" : @(kEnumOne),
                                       @"enumTwo"  : @(kEnumTwo),
                                       @"enumThree"   : @(kEnumThree)),
                       kEnumOne, integerValue)
    @end

@MattFoley - thanks for the detailed description! If you can write up the docs for this and submit a PR I'll be happy to review, just ping me on it!

Will do, thanks @brentvatne.


Add option for adjusting upstream target weighting via OAS

My client wants to have the ability to adjust the target weighting on the "upstream targets" via the OpenAPI Spec (when converted to the Kong Deck YAML format).
This will allow them to easily do canary releases by deploying a 'green' backend, and forwarding some of the old 'blue' backend traffic to it. Once they are satisfied with the results, they will remove the 'blue' backend target from the OAS.
It defaults to "100.00" or just "100" as is normal for the Kong defaults.

Sorry I see this is failing tests, that's because I had lax lerna on my computer! I will fix that shortly.


" errors" when processing output stream

Every so often I'll get something like this in my output stream when doing async processing
{"event":{"event_type":"progress","delta":" errors","logprobs":null,"stop_reason":null}}

here's the code reading and printing the response, it's a modified client:
        # Process the response from the server
        async for log in client.chat_completion(request):
            # print(f"Log structure: {log}\n{type(log)}\n")
            # print(f"\nLog structure: {log}\n{type(log)}\n{log.event}\n{type(log.event)}\n")
            if isinstance(log, ChatCompletionResponseStreamChunk):
                # print(f"RESPONSE STREAM CHUNK")
                if isinstance(log.event, ChatCompletionResponseEvent):
                    # print("\tRESPONSE EVENT")
                    if isinstance(log.event.event_type, ChatCompletionResponseEventType):
                        if log.event.event_type == ChatCompletionResponseEventType.start:
                            cprint(f"\n", "white", flush=True)
                        elif log.event.event_type == ChatCompletionResponseEventType.complete:
                            _stop_reason = log.event.stop_reason
                            cprint(f"\nCOMPLETED::Stop Reason::[{_stop_reason}]\n", "red", flush=True)
                            break
                        elif log.event.event_type == ChatCompletionResponseEventType.progress:
                            if ("errors" in log.event.delta and  log.event.logprobs == None and log.event.stop_reason == None):
                                break
                            else:
                                cprint(log.event.delta, "cyan", end='', flush=True)
                        else:
                            print(f"""
                            Log structure: {log}
                            {type(log)}
                            {log.event}
                            {type(log.event)}
                            ---------------------------------------------------------------------
                            print(f"\t\tEVENT TYPE             ::[{log.event.event_type}]")
                            print(f"\t\tEVENT [type] TYPE      ::[{type(log.event.event_type)}]")
                            print(f"\t\tEVENT TYPE DELTA       ::[{log.event.delta}]")
                            print(f"\t\tEVENT TYPE LOGPROBS    ::[{log.event.logprobs}]")
                            print(f"\t\tEVENT TYPE STOP_REASON ::[{log.event.stop_reason}]")
                            ---------------------------------------------------------------------
                            UNHANDLED ChatCompletionResponseEventType::[{log.event.event_type}]
                                """)

why do I get the errors every so often and my attempts to catch them and skip seems to fail.
Any suggestions?

Another update, it seems those messages are just sent by the model and needs to be parsed.
Can I get pointed to a reference on how to parse those out and how to better process them?
Any documentation on what are the purpose of those values when parsing the response?
<|begin_of_text|><|start_header_id|>system<|end_header_id|>

Cutting Knowledge Date: December 2023
Today Date: 23 July 2024

You are a helpful assistant<|eot_id|><|start_header_id|>user<|end_header_id|>
lets discuss Ni Electrochemical synthesis and redetermination of Ni(acac)x(H20)2.H20
<|start_header_id|>assistant<|end_header_id|>

Ni Electrochemical synthesis and redetermination of Ni(acac)x(H[2]
O)[2]
.H[2]
O is a topic in inorganic chemistry, specifically focusing on the electrochemical synthesis and structural analysis of a nickel complex.

To provide a better understanding, let's break down the key components:

[1]
. **Ni(acac)x(H[2]
O)[2]
.H[2]
O**: This is the chemical formula for the nickel complex being studied. 'acac' stands for acetylacetonate, a common ligand in coordination chemistry. The 'x' in the formula indicates that the complex has a variable number of acetylacetonate ligands. The 'H[2]
O' at the end represents two water molecules.

[2]
. **Electrochemical synthesis**: This is a method of synthesizing compounds using an electric current. In this case, the synthesis of Ni(acac)x(H[2]
O)[2]
.H[2]
O likely involves the reduction of nickel ions at an electrode, resulting in the formation of the complex.

[3]
. **Redetermination**: This refers to the process of re-determining the structure of a compound using advanced analytical techniques, such as X-ray crystallography or NMR spectroscopy. In this case, the researchers likely used these techniques to re-determine the structure of Ni(acac)x(H[2]
O)[2]
.H[2]
O.

Some possible questions or topics related to this subject include:

- What are the conditions for the electrochemical synthesis of Ni(acac)x(H[2]
O)[2]
.H[2]
O?
- What is the structure of Ni(acac)x(H[2]
O)[2]
.H[2]
O, and how does it differ from previously reported structures?
- What are the properties of Ni(acac)x(H[2]
O)[2]
.H[2]
O, such as its magnetic properties or reactivity?

If you have any specific questions or topics you'd like to discuss, I'd be happy to help.
COMPLETED::Stop Reason::[StopReason.end_of_turn]

We offer a client SDK with event logging lib to help with parsing of the output streams. You can find an example using client SDK here: https://github.com/meta-llama/llama-stack-apps/blob/main/examples/inference/client.py. This repo also contains example references on helping with parsing the output response.
For reference on parsing the output streams: you can check out the client SDK EventLogger for inference, for agents.


Search Tool

Currently the search tool has some bugs.
For example:

TAB doesn't work for the latest results
the ::after text in the search bar isn't always helpful / able to execute
the latest results are only shown the forst time the page loads and not if no pattern (no text in search bar) exists

Feel free to comment on other bugs of the search tool below this issue.

"the latest results are only shown the forst time the page loads and not if no pattern (no text in search bar) exists" -
is now possible - stupid mistake (shit happens...)


does Functional Model support "string" dtype for end-to-end pipelines?

As mentioned keras-team/keras-core#843, I was trying further modify the mlm-bert pipeline for keras-core with backend-agnostic support. The last part of the example tries to create an end-to-end pipeline with raw texts as the input to the model. Mentioned as follows:
def get_end_to_end(model):
    inputs_string = keras.Input(shape=(1,), dtype="string")
    indices = vectorize_layer(inputs_string)
    outputs = model(indices)
    end_to_end_model = keras.Model(inputs_string, outputs, name="end_to_end_model")
    optimizer = keras.optimizers.Adam(learning_rate=config.LR)
    end_to_end_model.compile(
        optimizer=optimizer, loss="binary_crossentropy", metrics=["accuracy"]
    )
    return end_to_end_model

end_to_end_classification_model = get_end_to_end(classifer_model)
end_to_end_classification_model.evaluate(test_raw_classifier_ds)

But while executing with jax backend, it throws the following error:
ValueError                                Traceback (most recent call last)
Cell In[10], line 14
     10     return end_to_end_model
     13 end_to_end_classification_model = get_end_to_end(classifer_model)
---> 14 end_to_end_classification_model.evaluate(test_raw_classifier_ds)

File [~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/keras_core/src/utils/traceback_utils.py:123](https://file+.vscode-resource.vscode-cdn.net/Users/mrutyunjaybiswal/Documents/oss/tfjax/keras-core-examples/keras-core-port-nb/nlp/~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/keras_core/src/utils/traceback_utils.py:123), in filter_traceback..error_handler(*args, **kwargs)
    120     filtered_tb = _process_traceback_frames(e.__traceback__)
    121     # To get the full stack trace, call:
    122     # `keras_core.config.disable_traceback_filtering()`
--> 123     raise e.with_traceback(filtered_tb) from None
    124 finally:
    125     del filtered_tb

File [~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/tree/__init__.py:435](https://file+.vscode-resource.vscode-cdn.net/Users/mrutyunjaybiswal/Documents/oss/tfjax/keras-core-examples/keras-core-port-nb/nlp/~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/tree/__init__.py:435), in map_structure(func, *structures, **kwargs)
    432 for other in structures[1:]:
    433   assert_same_structure(structures[0], other, check_types=check_types)
    434 return unflatten_as(structures[0],
--> 435                     [func(*args) for args in zip(*map(flatten, structures))])

File [~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/tree/__init__.py:435](https://file+.vscode-resource.vscode-cdn.net/Users/mrutyunjaybiswal/Documents/oss/tfjax/keras-core-examples/keras-core-port-nb/nlp/~/Documents/oss/tfjax/keras-core/kscore/lib/python3.10/site-packages/tree/__init__.py:435), in (.0)
    432 for other in structures[1:]:
    433   assert_same_structure(structures[0], other, check_types=check_types)
    434 return unflatten_as(structures[0],
--> 435                     [func(*args) for args in zip(*map(flatten, structures))])

ValueError: Invalid dtype: object

Is having the input layer of dtype `string' might cause the problem here?

@Mrutyunjay01, Could you please provide sample reproducible code, the tutorial you have mentioned has other error to solve before we get to the reported issue part.

Apologies for late response. I faced the issue when tried to port BERT MLM to backend agnostic. Been a while, let me refine the port code once, and see if the issue persists still.
cc: @sachinprasadhs

Update:
The issue no longer persists in keras 3.3. So, we can close this. Currently I am trying to port BERT MLM to backend agnostic, will report if I come across any such issues.

Reopening the issue as faced in the draft PR mentioned.