Update create-sso-office-add-ins-aspnet.md

removed reference for Node and express

Docs Build status updates of commit c4fe4ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bbc8fd6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@joelfmrodrigues Thanks for making this updating!


Backward compatibility helpers for ApolloInterceptor

Investigate what's needed to provide helpers for migration to coroutine-based interceptors (from callback-based)

Closing for now as v2->v3 is no longer very relevant nowadays. Also we now have the Java runtime artifact which has callbacks based interceptors


Provide ability to specify  AWSCredentialsProvider

Hi ,
We have a setup wherein we have an on-premise kubernetes cluster and 2 AWS Accounts, Account-A and Account-B. We have a use case where we have an AWS-SQS  in Account-A and a MSK-Cluster in Account-B . All our pods use OIDC federated tokens to access the AWS Resources and WebIdentityTokenCredentialsProvider as credentials provider for accessing AWS-SQS. This is a snippet from an application running in K8s cluster.
AmazonSQSClientBuilder.standard()
                .withRegion(properties.getSigningRegion())
                .withCredentials(new WebIdentityTokenCredentialsProvider())
                .build();

This all works fine because we have following environment variables configured in our pods:
  AWS_ROLE_ARN
  AWS_WEB_IDENTITY_TOKEN_FILE

The same application now needs to connect to MSK cluster hosted in Account-B. I have created a separate role (arn:aws:iam::XXXXXXXX:role/basic/XXXXXXXXX\) in Account-B for our application to allow it to access the MSK Cluster.
So, now instead of 1 role , I have now 2 roles. I can programatically configure the roleArn for the SQS Client like this:
               WebIdentityTokenCredentialsProvider.builder()
                    .roleArn("")
                    .webIdentityTokenFile("")
                    .build();

Is there a way to do something similar with AWS-MSK-IAM-AUTH library? So basically I am looking for a way to configure a different WebIdentityTokenCredentialsProvider for iam-authentication library.

@rampaldheeraj You can configure a KafkaProducer programmatically by passing it the producer's properties including the parameters required foraws-msk-iam-auth, such as security.protocol,sasl.mechanism,sasl.jaas.config and sasl.client.callback.handler.class.
Please see this example of configuring a Kafka Producer programmatically in the tests. Please note that the test uses dummy data which will have to be replaced for your use case.
Also, please look at how an IAM Roles can be specified as an argument.
Please let me know if this helps.

@sayantacC Thanks for the quick response. I have no problem in configuring and running producers with aws-msk-iam-auth library. I just want to know if there's a way to configure WebIdentityTokenCredentialsProvider programmatically. I have explained the situation in my question above.

@rampaldheeraj I apologize for misunderstanding the question.
I am sorry but currently the environment variables are the only way to configure the WebIdentityTokenCredentialsProvider in aws-msk-iam-auth.
Would it be possible for you to configure the SQS client by providing the roleArn and webIdentityTokenFile to point to  the role for Account A? Then the environment variables could point to the role for Account B and could be used by the aws-msk-iam-auth. Is that something that would work for your scenario?

@sayantacC . I can certainly try that. Thanks for the suggestion.

Thanks @sayantacC . This works. But not very convenient. Instead it will be good to have the ability to specify the Credentials Provider. That way it will be explicit and clear in code about which Credentials are being used for which Service.
Is there any way to pass credentials (AWS secret access key and Session Token) to aws-msk-iam-auth ?

@rampaldheeraj You could create a AWS credential profile and pass the awsProfileName as an argument to sasl.jaas.config. Please see this sub-section in the README

I have marked this issue as an enhancement.

@sayantacCm  the awsProfileName approach is a good workaround, but profile files based approach is difficult to work with when a service has to support multi-tenant connectivity to multiple customer's owned MSK instances, in that context using an embedded JaasConfig works out well.
This enhancement request is similar to how kinesis flink connector is supporting ability to override credentials provider when using STS assume role (link) to support connectivity to cross account kinesis streams.
One way we are able to achieve this is by adding two extra properties  providing an OIDC Token file and an assume role arn for sts crednetials provider, see following gist as an example
https://gist.github.com/s-ifti/509ffb6150ab541655313e0889a9a163
Having this feature supported will assist  SAAS developers who need to work with cross account access to MSK using IAM Roles.


fix: allow graph_store.llm to fully override LLM configuration

Description
Allow graph_store.llm to fully override LLM configuration
Previously, graph_store.llm could only override the LLM provider but would still
use config.llm.config for configuration settings. This change allows graph_store.llm
to override both the provider and its configuration settings.
This enables more flexible LLM configuration when needed specifically for graph store operations.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

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


⚠ TiendaEnLinea has degraded performance

In 546fcae, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 5079 ms

Resolved: TiendaEnLinea performance has improved in 8455373.


Upgrade protobuf and remove GH Action googletest-installer

Which issue does this PR close?
Closes #772.
Closes #763.
Rationale for this change
The markusjx/googletest-installer action is not allowed by our CI (see #772).
And it should no longer be necessary since upstream rolled out a fix that makes the installer unnecessary.
What changes are included in this PR?
Upgrade deps, remove the action step.
Are there any user-facing changes?
No

Thank you for fixing this!


add menu localization for change the language

this feature to change languange with menu bar , so we can easily change the language

Is it to change the entire system language setting? I think it would be better if we only change default locale on our app

because sometimes the default language is English and he wants to change
the language to Indonesian and sometimes confused where to set
Pada tanggal Rab, 18 Mar 2020 pukul 20.17 Rizki Maulana <
<EMAIL_ADDRESS>menulis:

Is it to change the entire system language setting? I think it would be
better if we only change default locale on our app
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rizmaulana/kotlin-mvvm-covid19/pull/35#issuecomment-600616974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOS5WXBGJ2D72WD7TCM4ATRIDCXZANCNFSM4LOFHF4Q
.

I think changing system language isn't this app's job, @rizmaulana is right, changing default locale on app looks better

okey good luck


Add Chutes and Ladders

Submitting this PR means I have:

[x] searched the readme file and searched the open pull requests (no duplicates please)
[x] and will follow the desired format
[x] used image links that resolve correctly, are appropriately sized, and are served over HTTPS

Thanks for contributing. Happy Hacktoberfest!


before creating space check if authorization is enabled

Ike Plugins (test-keeper)
Thank you @surajssd for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
Your plugin configuration is stored in the file.

Codecov Report

Merging #2089 into master will decrease coverage by <.01%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##           master    #2089      +/-   ##
==========================================
- Coverage   67.45%   67.45%   -0.01%
==========================================
  Files         158      158
  Lines       14657    14664       +7
==========================================
+ Hits         9887     9891       +4
- Misses       3816     3818       +2
- Partials      954      955       +1

Impacted Files
Coverage Δ

controller/space.go
69.3% <54.54%> (-0.59%)
:arrow_down:

configuration/configuration.go
68.95% <0%> (+0.59%)
:arrow_up:

Continue to review full report at Codecov.

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


5.1.2d Any characters entered in the search box are retained after the button is clicked.

Already done!


add new DragonFly BSD logo, move old logo to dragonfly_old, add drago…

Description
Add new DragonFly BSD ascii, to reflect their new logo as seen here:

Move the original ascii logo to dragonfly_old
Add a small version, dragonfly_small

a preview:

LGTM :+1:


Render items different from selected in Autocomplete tags

It would be nice if Autocomplete could show tags different from selected items. Right now there is renderTags that has selected items and a function to get tag props from the item's index. This is not a very flexible api because it couples items with their indices.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
In our application we built an autocomplete with multi select and items with a tree structure around material-uis Autocomplete. The requirement is when you select a parent, its children are automatically selected and disabled in the list, but only the parent shows up in selected tags.
I managed to filter out the children from selected items and use them in renderTags, but the fact that I had to preserve their indices felt like a hack.
So this might as well be a question, is there a better way for this? If no, maybe the api could instead expose getTagProps based on some other property than index?
Thanks for the help! 🙂

@MargaretKrutikova I'm sorry, I'm not sure to fully understand your pain point. Do you have a reproduction we could look at? :)

@MargaretKrutikova I'm closing for new, feel free to resume the discussion with more details :).


idm_warmepumpe_warmwasserladung_ausschalttemperatur entity no longer available, and idm_warmepumpe_warmwasser_solltemperatur shows wrong value

In previous version there were two entities supported
idm_warmepumpe_warmwasser_solltemperatur and
idm_warmepumpe_warmwasserladung_ausschalttemperatur
Actually they showed the same value. However the ...ausschalttemperatur shows always "Unavailable" and the
idm_warmepumpe_warmwasser_solltemperatur shows a wrong outdated value, I changed long time ago.
Seems somehow broken...
Is there a new entity replacing them?

Hm, my Home Assistant still shows these entities. Not sure what went wrong for you.
BTW Warmwasser-Solltemperatur in Home Assistant should correspond to "Zapftemperatur" in the IDM interface (under "Warmwasser").
Warmwasserladung Ausschalttemperatur and Warmwasserladung Einschalttemperatur should correspond to "Ausschalttemperatur" and "Einschalttemperatur" respectively (under "Zeitprogramm Warmwasser"). The Warmwasserzapftemperatur entity in Home Assistant seems to correspond to "B42 Warmwassertemperatur" only visible in the full sensor list in the IDM interface.
Do you see the values in the list you discovered in #69 (IDs 1030-1034)?

Yes I see them correct in the GLT Monitor
I see
ID 1033 Warmwasserladung Einschalttemperatur 35°C
ID 1034 Warmwasserladung Ausschalttemperatur 40°C
I remember I got the values correct in a previous version of your integration. Somehow it gots corrupted. I struggle as well to delete old not longer entities. I don't know what happened here, and why it is so wired...
What had been the steps to remove unwanted entities?

Yes, Home Assistant makes it really difficult to delete incorrect data, because normally you don't want to loose data...

What had been the steps to remove unwanted entities?

There is a "Purge entities" service (BE CAREFUL not to delete the wrong entities). If that doesn't help, you may need to edit some JSON files or access the SQLite database. If that's the case, please look for help on the Home Assistant forum. There are already a few posts about deleting entities, maybe one of them is what you need.
The entity list in the settings should also mark any restored entities, which you can delete via the UI and the statistics tab in the Home Assistant developer tools ("Entwicklerwerkzeuge") shows a "Fix issues" ("Problem beheben") button for all entities which issues.
Regarding this integration specifically: AFAIK there should be no need to delete anything, as long as you don't care about the entity IDs. Also the entities in question here were not affected by any updates.

Found the issue, you are right, you not changed that bit of code, but I changed the target value for the "Ausschalttemperatur".
Your integration wants to see this value between 46 min and 53 max. That was the issue, since I lowert the value to 43 and that made it unavailable. Not checked the log. My IDM GUI states, this value can be betwen 35 to 55°C. Therefore I propose you change the file:
sensor_addresses.py in line 660 to 35 and line 661 to 55
I think this are the aligned values.
To be honest I am not a fan of checking again the values from the source. Of course if they are totally rubbish, they should be prevented, but I would open the limits to something imaginable to be valid. So instead of taking the borders from IDM (GUI), which I suspect to change from version to version, just limit it to a reasonable wider range, which would be still possible.
So for hotwater I would allow any value between 5°C to 99°C and leave the rest to the heatpump to limit it further.
Of course this would be differnt for values you support writing. For writing the borders needs to be limited to the spec.

P.S. Still can't delete unwanted entities, but this is nothing hurting me. Is it safe to rename the entities, in case I want?

The min/max issue is fixed in release v0.7.0.

Is it safe to rename the entities, in case I want?

Yes, Home Assistant should be able to keep track of the data via the internal unique IDs.


Test "beautiful docs" GitHub doc display option

Need a dev volunteer to help me build the "beautiful docs" feature in GitHub today ...
https://github.com/beautiful-docs/beautiful-docs  ...
It promises "beautiful" output for our .md GitHub doc files.
GitHub
beautiful-docs/beautiful-docs
beautiful-docs - A beautiful way to read documentation
Example output:
https://raw.githubusercontent.com/beautiful-docs/beautiful-docs/master/docs/screenshot.png (81KB)

@joshhus are there any live examples you can cite? Also, what assistance specifically are you seeking?

Don't know. Javascript.

Happy to assist.

The docs that we want to test this "beautiful docs" display format for are here: https://github.com/hyperledger/fabric/tree/master/docs

@joshhus I don't believe any Javascript is required. The manifest is in JSON which is pretty straight forward. I can help if you get stuck. @JonathanLevi thanks for lending a hand, we do appreciate it!

JavaScript Object Notation is more than I have in my toolbelt right now. Thanks @JonathanLevi !

NP. I just quickly ran bfdocs  and beautiful-docs fails at the moment if the markdown (e.g., README.md) does not have the structure it expects.
For example, https://raw.githubusercontent.com/hyperledger/fabric/master/README.md needs to also have a 'top-level' project name (/an item beginning with a single '#' prior to the '## Incubation Notice')
If you don't mind, I can create a (minor) issue and update these .md files with the top-level titles and/or whatever else is expected. Shouldn't cause any harm anyway, regardless of whether or not we choose to go ahead with beautiful-soup.

@JonathanLevi yes, please! Thanks

Yes @JonathanLevi thanks. Where are you at with this? LMK so I can step in also ... Note the single # text could be "Hyperledger Project" or whatever @christo4ferris would like to see; the same on every page for now. Is the README.md the only failure?

Moving discussion from slack here:
jonathanlevi [2:30 PM]
Hi!
[2:31]
Yes, so it required/requires some changes to the README.md files, etc...
[2:31]
It fails miserably if/when the file is not structures correctly. I opened 2 issues and fixed them.
[2:31]
Waiting for them to get merged.
[2:32]
On that note, do you know a little bit about CSS (styling?)
joshhus [3:05 PM]
.css generally, maybe a little bit. but not specific to this usage - what is your question? ... Also do you have the Issue #s handy for you "BD" fixes - are there Pull Requests for them?
[3:06]
Sounds like to test the BD fixes we need the files committed, okay.
jonathanlevi [3:48 PM]
Yes, they were committed.
[3:49]
We might also need to "prettify" the output of BD to make it nice (hence my question regarding the CSS)
----- May 8th -----
jonathanlevi [12:07 PM]
Hi Josh
[12:07]
Just a s quick one: I don't really like beautiful docs... I'm playing with it (as you can tell in my spare time), and it's OK
[12:08]
Still requires the strict formatting/structure of the ".md" files, and I think that we can do much better than this.
[12:09]
How important is this effort to you? I can put together an example, of how to make these a lot more nice...
[12:11]
Hit me up here to discuss this further - if you have some capacity, we can work on this and make this much nicer.
----- May 9th -----
joshhus [9:11 AM]
Hello Jonathan, because this is an open project, the "best" way to propose any solution is to create an issue and propose the solution to the community. Whether that is "BD" or something else, you are free to contribute your proposals. In this case you can just propose an alternate solution to the "BD" issue. Please describe in detail why BD is not a good solution. Note that we are likely tied to the .md + HTML format no matter what the output though, as a GitHub project.

Jonathan and I did a Google hangout yesterday. We've gotten our .md. to .rst. Now need to go from .rst to Sphinx to try that output. (Comparing Sphinx to Beautiful Docs). Help still wanted!

http://www.sphinx-doc.org/en/stable/

We played around with beautiful-docs  and quickly learned the following:

It is not very forgiving if the markdown structure is not as it expects (e.g., files that didn't begin with H1 caused exceptions and errors, etc)
We are under the impression that it is not actively developed/updated. E.g., as of this writing [Latest commit 84fcf79 on Jan 3, 2015] and unfortunately, even the website the project refers to [Checkout beautifuldocs.com for an example], does not work (anymore)
Even after "fixing" Fabric's ".md" files locally and generating some static htmls, it still seemed to have required some more tweaking and more formatting and styling.

Unless there are strong supporters for this, we will be looking at other frameworks.

@JonathanLevi and @joshhus are looking at Python / Sphinx.

Volunteers always welcome!

Did you consider gitbook?


Fix unified diff view styling

Fixing my own pull request (#584).
I've just noticed (20 minutes to late), the unified code view styling was wrong now. Put unified / split into own css classes and everything is perfect now.
Sorry sorry for that, won't happen again ;)

@mkuhlmann Could you provide the screenshots?

Sure thing,

Unified view before this PR (bugged)
Unified after this
Split view working nicely

@mkuhlmann Good to merge.  LGTM

LGTM


Set up memory-storage-layer module code owners

Signed-off-by: wuyingjun<EMAIL_ADDRESS>What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/cc @Iceber

+1 🎉 Thank you for your contribution


tombstone item when speed scrolling

ENHANCEMENT:
I juste check this article from google: https://developers.google.com/web/updates/2016/07/infinite-scroller
They speak about Tombstones fake item when make crazy scrolling just show a fake row until real. This have sens when we are on mobile, by example on my iphone6 I have data of 300 items, I scroll one or two time down my screen is BLANK until showing next data range.
You can check demo here from Google
PS: This feature will be very appreciate and more UX ready, and make your datatable-component the best on the world ! AWESOME ! :dancers: :dancer:  :yum:
What community says: second it!

@amcdnl can you tell me where your set these empty rows when waiting to renderer on existant tag in dom ?
When I scroll on iphone I just want to stylize with load-icon on each row  until the next data is rendered

Heres the loading code: https://github.com/swimlane/angular2-data-table/blob/master/src/components/body/body.component.ts#L47

@amcdnl oh yes,  this is too important to know, it's neither what I need: I'm looking for that:
I use virtual scroll... and on iphone when I scroll fast the page is blank until other rows appear....What make this blank ? I think there is empty row until renderer ? that I need edit...
I just want add some blurred text when on iphone I scroll fast... that'0s called: tombstone
I just past url above: http://googlechrome.github.io/ui-element-samples/infinite-scroller/ try scroll fast you will see tombstone this is amasing...
facultatif: awesome If you implement this sort of virtual dom

@amcdnl I open this issue 1month ago, can you please confirme me we are talking about same thing with your recent  issue here https://github.com/swimlane/angular2-data-table/issues/249
as well I can close my issue and let yours open if we mean samething I think typically on virtualscroll "tombstone" is very important to avoid EMPTY data when scroll fast on mobile...

Anyone got any references they like? @istiti @gdi2290

@amcdnl  Do you mean how this need to be displayed ? maybe as template ? Actually when i'm on iphone and I scroll very very and very fast with a big object the screen is blank..while waiting next row to be drawn, it will be nice of behind the scene we can show someting will come with empty rows ...

Just visually, any examples of someone doing this you like?

@amcdnl Please check the below for some demos:
https://github.com/bvaughn/react-virtualized

any ETA here ?

This feature is UX advantage, when scrolling too fast with virtual scroll (under iphone for instance) screen is blank white

ETA?


Toradex BSP 5.2

Integration of Toradex BSP 5.2

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix spelling/capitalization of various words

Fix spelling of:

arithmetic
arranged
associated
implementation
primarily
explicitly
menu

Capitalize proper nouns:

Cartesian

LGTM, thanks!


Hooking up Cordova's log events to console.log

Cordova's log events were being ignored, so now we hook them up to console.log to report issues to the end user.

this PR can abandoned now I guess?

Not quite; the other PRs are on top of this one. They'll merge into this, then this will merge into master.


feat(forks,tests): Add EIP-7623

🗒 Description
[WORK IN PROGRESS]
Adds tests for EIP-7623: Increase calldata cost.
Framework changes
Modifies transaction_intrinsic_cost_calculator that is returned by the fork objects to account for the extra intrinsic cost that the EIP adds.
Also a new parameter is added to the function in order to return different values depending on how the function is used:

Return the minimum gas that the transaction requires to be included
Return the gas cost that is deducted before the transaction starts execution

Fix Existing Tests
Some tests were broken because they needed a transaction to start with a deterministic amount of gas and also with a given calldata length.
Solution to most of these issues was to add an access list so the intrinsic gas cost exceeded the new floor data cost.
🔗 Related Issues
None
✅ Checklist

[ ] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[ ] All: Considered updating the online docs in the ./docs/ directory.
[ ] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[ ] Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
[ ] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[ ] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

@reedsa @fselmo


Support TypeScript 1.4 syntax highlighing

+1 ..... ES6 template strings syntax has been added to Typescript, would be nice to see that supported.


VirtualizedLists should never be nested inside plain ScrollViews with the same orientation because it can break windowing and other functionality - use another VirtualizedList-backed container instead.

import React, { useEffect, useState } from 'react';
import { useAppSelector, useAppDispatch } from 'store/hook';
import { SafeAreaView } from 'react-native-safe-area-context';
import { ScrollView, ImageBackground, Image, Modal, TouchableOpacity, Animated, PanResponder, GestureResponderEvent, PanResponderGestureState } from 'react-native';
import { DragSortableView } from 'react-native-drag-sort';
// import { Modal } from '@ant-design/react-native';
import MyStyleSheet from 'components/MyStyleSheet';
import { UIText, UITextInput } from 'components/Text/Texts';
import { Dimensions } from 'react-native';
import { NestableScrollContainer, NestableDraggableFlatList, ScaleDecorator, RenderItemParams } from "react-native-draggable-flatlist"
export const WINDOW_WIDTH = Dimensions.get('window').width;
export const WINDOW_HEIGHT = Dimensions.get('window').height;
const NUM_ITEMS = 10;
function getColor(i: number) {
const multiplier = 255 / (NUM_ITEMS - 1);
const colorVal = i * multiplier;
return rgb(${colorVal}, ${Math.abs(128 - colorVal)}, ${255 - colorVal});
}
type Item = {
key: string;
label: string;
height: number;
width: number;
backgroundColor: string;
};
const initialData: Item[] = [...Array(NUM_ITEMS)].map((d, index) => {
const backgroundColor = getColor(index);
return {
key: item-${index},
label: String(index) + "",
height: 100,
width: 60 + Math.random() * 40,
backgroundColor,
};
});
const Index: React.FC<{ navigation: any }> = ({ navigation }) => {
const [data, setData] = useState(initialData);
const renderItem = ({ item, drag, isActive }: RenderItemParams) => {
console.log(666, item, drag, isActive);
return (

<TouchableOpacity
onLongPress={drag}
disabled={isActive}
style={[
styles.rowItem,
{ backgroundColor: isActive ? "red" : item.backgroundColor },
]}
>
{item.label}

);
};
console.log('data', data);
console.log('renderItem', renderItem);
return (

<NestableDraggableFlatList
data={data}
renderItem={renderItem}
keyExtractor={(item) => item.key}
onDragEnd={({ data }) => setData(data)}
/>

);
};
const styles = MyStyleSheet.create({
rowItem: {
height: 100,
width: 100,
alignItems: "center",
justifyContent: "center",
},
text: {
color: "white",
fontSize: 24,
fontWeight: "bold",
textAlign: "center",
},
});
export default Index;

I have the same issue when using NestableDraggableFlatList

As do I, doesn't seem to matter the arrangement of the nodes.

Your example is not completed.
You don't use NestableScrollContainer to fix this error.
See below:
<ScrollView>
   <NestableScrollContainer>
      <NestableDraggableFlatList
         data={data.products}
         extraData={data.products}
         keyExtractor={(item: any) => `item-${item?.id}`}
         renderItem={(i: any) => renderItem(i)}
      />
   </NestableScrollContainer>
</ScrollView>

@TomCorvus Wrapping  with the  doesn't solve this problem for me

Same here, no matter what I tried with the components reordering etc. the warning won't go away.

The docs say all warnings will be disabled, this is incorrect when using NestableDraggableFlatLists. Please fix this.

Note: When using NestableDraggableFlatLists, all React Native warnings about nested list performance will be disabled.

Update: I give up, I'm temporarily disabling this error until I can migrate to a safer library. It looks like this issue has been around for a while.
Solution based on this thread about LogBox:
// place at top of app.js

if (__DEV__) {
  const ignoreWarns = ["VirtualizedLists should never be nested inside plain ScrollViews"];

  const errorWarn = global.console.error;
  global.console.error = (...arg) => {
    for (const error of ignoreWarns) {
      if (arg[0].startsWith(error)) {
        return;
      }
    }
    errorWarn(...arg);
  };
}

I'm going to check out the following libs until this gets addressed (no disrespect to the author):
react-native-draglist
react-native-sortable-list
react-native-drag-sort

Just a checkpoint post -- I was able to avoid all warnings and get the desired functionality using the setup defined here..

Yo, i faced the same issue and solved it by wrapping NestableScrollContainer with a View

@PaulThiberville can you give an example? By doing this below I still get the error:
 <View>
                <NestableScrollContainer>
                  <NestableDraggableFlatList
                    data={pollChoices}
                    initialNumToRender={pollChoices.length}
                    keyExtractor={(item) => String(item.id)}
                    onDragEnd={({ data }) => setData(data)}
                    onRelease={() => setScrollEnabled(true)}
                    renderItem={renderChoices}
                  />
                </NestableScrollContainer>
              </View>

<View style={{ flex: 1 }}>           <NestableScrollContainer>             <DraggableFlatList               data={socialLinks}               onDragEnd={({ data }) => setSocialLinks(data)}               keyExtractor={item => item.label + item.id}               renderItem={SocialItem}               dragHitSlop={{                 right: parseInt(theme.space.m, 10 * 2) + 50 - width,               }}             />           </NestableScrollContainer>         </View>
This code works for me on android @efstathiosntonas I use NestableDraggableFlatList if i have multiple FlatList

I can't understand what you are discussing here. If you open source code you will see that NestableScrollContainer is using ScrollView and DraggableFlatList is using FlatList (which is actually a wrapper around VirtualizedList).
As a result you will get VirtaulizedList inside ScrollView, which is denied in React Native.
I don't know, how wrapper View will help here. I guess you will still get this error.
This part of library should be rewritten by author or someone else.

I managed to resolve this issue by following this comment on SO: https://stackoverflow.com/a/65949323/2987737
Basically just add any content above and below the draggable list into ListHeaderComponent & ListFooterComponent

I managed to resolve this issue by following this comment on SO: https://stackoverflow.com/a/65949323/2987737
Basically just add any content above and below the draggable list into ListHeaderComponent & ListFooterComponent

This is the right answer for me for this issue and the issues I was having iOS in general - https://github.com/computerjazz/react-native-draggable-flatlist/issues/477#issuecomment-1712396814

// Place this useEffect in the top of the page //

// It will remove the warning message //

 useEffect(() => {
     LogBox.ignoreLogs(['VirtualizedLists should never be nested']);
  }, []);  ```


Add version info to OpenVINO models

Versions of optimum-intel and dependencies can have an impact on performance/accuracy. This PR adds transformers, torch, optimum and optimum-intel dependencies to the openvino_model.xml file (the OpenVINO version is already there). This is really useful when debugging issues, or looking into differences.
Example (<optimum> is added):

@eaidova @AlexKoff88

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Alphabetize package.json

See #10.
This is a draft PR. Its main purpose is to test Leonine diffs of .leo files.

Let's close this PR. It's way out of date.


V12 compatability

any ETA on V12 compatability or the remaster threw you for a loop to get the V12 done

Hi, the PF2E system is still in beta. I will look into V12 compatibility once the system goes out of beta.

Live now, it seems! Hopefully it's just a matter of changing a number.

Updated to version 12. I will slowly start working on other features now.


15863 dividing line approval type

Issue #: /bcgov/entity#15863
Description of changes:

Added v-divider separating line to distinguish between approval type and restoration type.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-create-dev--pr-567-9uqlr2x6.web.app

/gcbrun

Temporary Url for review: https://business-create-dev--pr-567-9uqlr2x6.web.app

There is a difference in spacing when placing divider in parent component.

Previous divider in PR:

current divider in PR:

There is a difference in spacing when placing divider in the parent component that I adjusted. Please let me know if there is a preference.

cc: @yuisotozaki

Current screenshot:

Space underneath Limited Restoration radio button could be reduced, or space above approval type could be increased as shown below (apologies for all the screenshots):

/gcbrun

Temporary Url for review: https://business-create-dev--pr-567-9uqlr2x6.web.app

Ask Yui your layout questions.

/gcbrun

Temporary Url for review: https://business-create-dev--pr-567-9uqlr2x6.web.app

So we're going with this? LGTM


yarn tries to run prepublish script and fails

$ yarn
yarn install v1.2.1
warning You are using Node "9.1.0" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || ^8.0.0"
[1/4] 🔍  Resolving packages...
success Already up-to-date.
$ . ./resources/prepublish.sh
[eval]:1
process.exit(({"remain":[],"cooked":["install"],"original":[]}).original[0].indexOf('pu') === 0)
                                                                            ^

TypeError: Cannot read property 'indexOf' of undefined
    at [eval]:1:77
    at ContextifyScript.Script.runInThisContext (vm.js:50:33)
    at Object.runInThisContext (vm.js:152:38)
    at Object.<anonymous> ([eval]-wrapper:6:22)
    at Module._compile (module.js:641:30)
    at evalScript (bootstrap_node.js:470:27)
    at startup (bootstrap_node.js:167:9)
    at bootstrap_node.js:613:3

   Only Travis CI can publish to NPM.
  Ensure git is left is a good state by backing out any commits and deleting any tags.
  Then read CONTRIBUTING.md to learn how to publish to NPM.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

When running yarn, $npm_config_argv would be an empty array. I've put in a simple fix to work around this issue in a4d9732318a753deec37929cfbfd4a1719507468. Thanks for the report @kevinsimper


Error Creating Swagger doc from json with external references

NSwag.CodeGeneration.CSharp 11.17.1.
Trying to generate a client from https://developers.adp.com/static/Swagger/worker-demographics-v2-swagger.json.  Schemas are all $refs in the same path.  The thrown exception is below.
I could observe using Fiddler that only the main swagger doc was getting downloaded so I thought the issue might be with GetUrlAsync passing the entire url as the documentPath parameter to GetJsonAsync.  I made my own version that took the file portion out of the path before making a call.  That didn't make a difference.
I also tried downgrading NSwag.  When I went down to version 15.4 (after trying 16 and getting the same result) the exception would no longer occur, but the application would block.  I could see on Fiddler the last file downloaded was workerdemographicmeta-v2-schema.json.  This was consistent after three trials, and was always the third schema.json file downloaded and the third referenced in the swagger doc.
I find it peculiar that the exception is complaining about the third referenced schema but doesn't seem to download the ones that come before it in the file, even though that's the observed behaviors of previous versions, and that same file is what seemed to cause the lockup in the previous versions.
Exception
NotSupportedException: Could not resolve the JSON path 'workerdemographics-v2-schema.json' because no document path is available.
Stack Trace
at NJsonSchema.JsonReferenceResolver.d__9.MoveNext()
at NJsonSchema.JsonReferenceResolver.d__4.MoveNext()
at NJsonSchema.JsonSchemaReferenceUtilities.JsonReferenceUpdater.d__5.MoveNext()
at NJsonSchema.Visitors.JsonReferenceVisitorBase.d__6.MoveNext()
//.... a whole lot of these
at NJsonSchema.Visitors.JsonReferenceVisitorBase.d__6.MoveNext()
at NJsonSchema.Visitors.JsonReferenceVisitorBase.d__4.MoveNext()
at NJsonSchema.JsonSchemaReferenceUtilities.JsonReferenceUpdater.d__4.MoveNext()
at NJsonSchema.JsonSchemaReferenceUtilities.d__0.MoveNext()
at NJsonSchema.Infrastructure.JsonSchemaSerialization.d__9`1.MoveNext()
at NSwag.SwaggerDocument.d__53.MoveNext()
at NSwag.SwaggerDocument.d__55.MoveNext()

Try with v11.17.2

I tried with 11.17.2.  It locked up like previous versions.
Interesting thing happened on Fiddler.  When trying to download workerdemographicmeta-v2-schema.json (after downloading 3 files)  the body length was -1 while the application was locked up. The download would finish after I stopped the program.  This was consistent.
I realize I made a mistake in my initial error reporting that on older versions workerdemographicmeta-v2-schema.json would be were the application locked.  Whereas the error in 11.17.1 was workerdemographics-v2-schema.json (no meta in name).  This appears to be corrected.
I should probably open a new ticket, as the error I initially reported appears fixed.

Yes, there was a bug which prevented the loading of referenced schemas... but in your case we may have another problem (maybe because the files are extremely but and we run into a timeout).

Thank you.  I will look into more before I open a new ticket on that issue.


Add a simple subscribe section

I wish to  create a subscribe section just above the footer which would have two fields, name and email. Please assign me this.
@Suchitra-Sahoo

This feature is not required right now.


Allow "$regex" inside "$not" queries on Mongo 4

Fix for https://github.com/davidlatwe/montydb/issues/81
Mongo 4.* now allows "$regex" inside "$not" queries:
 {"$not": {"$regex": "(?i)Hello World"}}

This patch modifies the behavior so that it's allowed in Mongo 4.* but not in Mongo 3.*

Hey @arieltorti , The code looks really good and thanks for the fast contribution. 👍🏼
I have nothing to add, let me know if you are happy to merge and I'll merge it.

Awesome, let's merge it then. Could we do a release including these changes?

Yeah, will make a 2.5.1 release later once merged :)


Fix viewer build warnings

Yeah, the code is a bit questionable. In theory it's there for when we start supporting custom fields 🤷


Add search endpoint for Members, Installs, etc.

While Daniel was working on his "whole building installs" tool, he requested a query pattern we do not currently support. He wanted to provide a search box to his users and have it search across name, phone number, and email address fields automatically (similar to the behavior in the admin UI)
We should probably add a search endpoint to handle this case

Consider doing fuzzy matching as described in #406 on this endpoint as well


geopmread/write requests sporadically fail

Describe the bug
I tried to use geopmwrite to set a control and I expected it to work instead an error was presented.
GEOPM version
3.1
Expected behavior
[bgeltz@node1 ~]$ geopmwrite CPU_UNCORE_FREQUENCY_MIN_CONTROL board 0<PHONE_NUMBER>
[bgeltz@node1 ~]$ echo $?
0

Actual behavior
[bgeltz@node1 ~]$ geopmwrite CPU_UNCORE_FREQUENCY_MIN_CONTROL board 0<PHONE_NUMBER>
Error: cannot write control: <geopm> Invalid argument: PlatformIOImp::write_control(): control name "CPU_UNCORE_FREQUENCY_MIN_CONTROL" not found: at src/PlatformIO.cpp:828

Error messages produced
Upon inspection with a debug build, contention for dbus is observed:
node1 47232: Warning: <geopm> Failed to load SERVICE IOGroup.  GEOPM may not work properly unless an alternate IOGroup plugin is loaded to provide signals/controls required by the Controller and Agent.

node1 47232: The error was: <geopm> Runtime error: SDBus: Failed to call sd-bus function sd_bus_call_method(), error  :-110 name: org.freedesktop.DBus.Error.Timeout: Connection timed out: at src/SDBus.cpp:31

...

node1 47232: Error: cannot write control: <geopm> Invalid argument: PlatformIOImp::write_control(): control name "CPU_UNCORE_FREQUENCY_MIN_CONTROL" not found: at src/PlatformIO.cpp:828

Additional context
It seems we're overwhelming dbus with this request.  It's unclear how much other system traffic over dbus was occurring during the time of the failure.  Reproducing the failure is fairly sporadic but seems to be more likely during large scale jobs (i.e. greater than 2048 nodes).

Work around posted here:
https://raw.githubusercontent.com/cmcantalupo/geopm/refs/heads/batch-write/geopmdpy/geopmbatchwrite.py

The above workaround does not address the dbus calls that occur during the initialization of the ServiceIOGroup/PlatformIO.  Setting the SYSTEMD_BUS_TIMEOUT environment variable to a value of greater than 25 (the default) seems to allow the write requests to proceed as expected.
https://github.com/systemd/systemd/blob/main/docs/ENVIRONMENT.md?plain=1#L86
export SYSTEMD_BUS_TIMEOUT=600

It seems that the dbus timeout was a red herring.  Looking through the syslogs on affected blades, I'm seeing messages like the following:
2024-11-26T12:25:27+00:00 node1 geopmd[147894]: The call io.github.geopm.PlatformGetUserAccess has failed with an exception:
2024-11-26T12:25:27+00:00 node1 geopmd[147894]: Traceback (most recent call last):
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:   File "/usr/lib/python3.6/site-packages/geopmdpy/system_files.py", line 1232, in get_user_access
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:     user_groups = self._get_user_groups(user)
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:   File "/usr/lib/python3.6/site-packages/geopmdpy/system_files.py", line 1063, in _get_user_groups
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:     return [grp.getgrgid(gid).gr_name for gid in all_gid]
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:   File "/usr/lib/python3.6/site-packages/geopmdpy/system_files.py", line 1063, in <listcomp>
2024-11-26T12:25:27+00:00 node1 geopmd[147894]:     return [grp.getgrgid(gid).gr_name for gid in all_gid]
2024-11-26T12:25:27+00:00 node1 geopmd[147894]: KeyError: 'getgrgid(): gid not found: XXXX'

These groups are provided via LDAP in this case, so the current theory is that something is interfering with our ability to query for these groups when LDAP is in use.
The sequence as it's understood now:

User issues a geopmwrite.
Server-side geopmd begins servicing that request.  When it attempts to enumerate the groups associated with the user, the above error occurs.  Code: https://github.com/geopm/geopm/blob/release-v3.1/geopmdpy/geopmdpy/system_files.py#L1060-L1063
When the error occurs, geopmd crashes and is restarted by systemd.
The original geopmwrite request is never completed since geopmd dies, and eventually the client-side code hits a dbus timeout.

Try to derive work around that does not require an updated version of the GEOPM service install.


Unable to login to OctoEverywhere via Google

iPhone 12 Pro
iOS 15.1.1
Adding an OctoEverywhere printer and trying to login via Google results in
"Error 403: disallowed_useragent"
https://developers.google.com/identity/protocols/oauth2/javascript-implicit-flow#authorization-errors-disallowed-useragent

Yup, same for me. Can’t login and use the app because of that

Yup, same for me. Can’t login and use the app because of that

Hey guys,
IIRC, there is a workaround to this problem that was shared by OctoEverywhere developer. I will take a look at this error again but so far I was not able to figure out how to fix it. In the meantime, use the workaround (sorry I do not remember it - maybe it was just using regular login with your email but not sure which password - OctoEverywhere developer can help with this).
Regards,
Gaston

Hey guys,
IIRC, there is a workaround to this problem that was shared by OctoEverywhere developer. I will take a look at this error again but so far I was not able to figure out how to fix it. In the meantime, use the workaround (sorry I do not remember it - maybe it was just using regular login with your email but not sure which password - OctoEverywhere developer can help with this).
Regards,
Gaston

I believe I was able to get around it by doing a password reset for my
email which allowed me to login directly to the site
On Sat, Jan 1, 2022 at 1:02 PM Gaston Dombiak @.***>
wrote:

Hey guys,
IIRC, there is a workaround to this problem that was shared by
OctoEverywhere developer. I will take a look at this error again but so far
I was not able to figure out how to fix it. In the meantime, use the
workaround (sorry I do not remember it - maybe it was just using regular
login with your email but not sure which password - OctoEverywhere
developer can help with this).
Regards,
Gaston
—
Reply to this email directly, view it on GitHub
https://github.com/gdombiak/OctoPod/issues/545#issuecomment-1003593938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOT74P5PSRSK6AHQO7H5GTUT46URANCNFSM5J2U6HOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe I was able to get around it by doing a password reset for my
email which allowed me to login directly to the site
On Sat, Jan 1, 2022 at 1:02 PM Gaston Dombiak @.***>
wrote:

Hey guys,
IIRC, there is a workaround to this problem that was shared by
OctoEverywhere developer. I will take a look at this error again but so far
I was not able to figure out how to fix it. In the meantime, use the
workaround (sorry I do not remember it - maybe it was just using regular
login with your email but not sure which password - OctoEverywhere
developer can help with this).
Regards,
Gaston
—
Reply to this email directly, view it on GitHub
https://github.com/gdombiak/OctoPod/issues/545#issuecomment-1003593938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOT74P5PSRSK6AHQO7H5GTUT46URANCNFSM5J2U6HOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Unable to restart test debugger

Type: Bug

Behaviour
Expected vs. Actual
When debugging unittests, I'd like to be able to restart the debugger.
Steps to reproduce:

Create a file named test.py with the following content:

#!/usr/bin/env python

import unittest

class Test(unittest.TestCase):
    def test_me(self):
        pass
        self.assertTrue(True)

if __name__ == "__main__":
    unittest.main()

Run Python: Configure Tests to use unittest at . with test*.py.
Create a breakpoint in line 9.
Debug the test either using the Testing tab or right click the icon on line.
Restart the test.

Seems like #19030 and #17636 could be related.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.10.4
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Pyenv
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Pylance"

formatting
• provider: "black"

testing
• unittestArgs: "<placeholder>"
• unittestEnabled: true

Extension version: 2022.20.2
VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:59:02.252Z)
OS version: Windows_NT x64 10.0.19043
Modes:
Sandboxed: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
63.78GB (48.12GB free)

Process Argv
--crash-reporter-id 3515b736-1f7c-4358-b41b-322311caf5fa

Screen Reader
no

VM
19%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2591)

Memory (System)
31.24GB (29.28GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2591)

Memory (System)
31.24GB (29.28GB free)

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848:30577860
nodejswelcome1:30587005
282f8724:30602487
f6dab269:30613381
fim-prod:30623723
a9j8j154:30646983
vscsbc:30628655

Also relevant, I consistently get this error in the Remote Extension Host logs:
2023-01-17 12:36:36.226 [error] TypeError: Cannot read properties of undefined (reading 'toString')
	at c.onCloseSocket (/home/zjw/.vscode-server/extensions/ms-python.python-2022.20.2/out/client/extension.js:2:738410)
	at Socket.<anonymous> (/home/zjw/.vscode-server/extensions/ms-python.python-2022.20.2/out/client/extension.js:2:737743)
	at Socket.emit (node:events:526:28)
	at TCP.<anonymous> (node:net:687:12)

When hitting restart, I see this get logged to Python:
DAP Server launched with command: /home/zjw/.pyenv/versions/3.10.4/bin/python /home/zjw/.vscode-server/extensions/ms-python.python-2022.20.2/pythonFiles/lib/python/debugpy/adapter

Hello! We are currently working on a fix for this- I apologies for the lack of response while we have been doing so. Thanks

Hello! We have just finished our testing rewrite and are beginning the roll out to users. If you are able, it would be very helpful to know if your issue still exists on the rewrite, I have tried this myself with the rewrite and it has worked! To try it yourself, you need to be on the most recent release of the Python extension and add this setting to your users settings.json "python.experiments.optInto": ["pythonTestAdapter"].
You can confirm you have the rewrite enabled by setting  "python.analysis.logLevel": "Trace", in your user settings then check for Experiment 'pythonTestAdapter' is active in your python logs. (might need a reload and to set vscode log levels to trace too via the command palette)
Let me know if the rewrite fixes your issue. Thanks!

Hey @eleanorjboyd, this seems to have resolved the issue after I got onto the pre-release version and did a reload + cleared python cache. Thanks!

Great! Thanks for testing it out- happy coding!


fix ron & json serialization of large floats

Currently for JSON and RON, nanoserde serializes large floats as integers when possible, and during deserialization first reads them as integers and later converts to a float. This leads to a bug when handling large numbers. For example, 1e30 is serialized as "1000000000000000000000000000000", which attempts to be deserialized as a u64 and errors with a positive overflow.
This PR changes float serialization to use the debug trait rather than the display trait, resulting in the representation "1e30" which is deserialized directly to a float.

Thanks for doing this, looks like a win all the way around


fix: time limitation set (seconds)

Codecov Report

Merging #59 (555c5b4) into master (7a0761b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   21.96%   21.96%
=======================================
  Files          21       21
  Lines         724      724
  Branches      118      118
=======================================
  Hits          159      159
  Misses        565      565

Impacted Files
Coverage Δ

src/components/widgets/card/CardRoomTime.vue
6.66% <0.00%> (ø)

Continue to review full report at Codecov.

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

@all-contributors please add @simonrousseau  for review


Retention based Data replication

ReplicationDataRetentionCategory specify whether the copy is "Sync", "limited snapshots copy", "limited daily partitions copy" or "limited hourly partitions copy"
ReplicationDataValidPathPicker will use GlobModTimeDatasetVersionFinder or DateTimeDatasetVersionFinder to find the valid paths to copy from source to target

There are FindBugs warnings:

Execution failed for task ':gobblin-data-management:findbugsMain'

There seem to be unit test failures in ReplicationConfigurationTest

Please make sure that unit tests pass. Other than that, LGTM

fixed the unit testing


FaceEncoding class do not have properties to record: 'the image where its from'

Thanks you and agree with you that do not expose the constructor of FaceEncoding.
here is another consideration please， the FaceEncoding class do not have properties to record: 'the image where its from'， 'the rectangle where its from', 'the Entity where its related in real world' .  so when we done a 1:n or m:n FacesCompare，it is really hard to related back the picture， the rectangle(we may draw a box on that face)， the Entity(we should display the name or other info below face box) with the FaceEncoding.  if we can have these properties inside the FaceEncoding class.  the above tasks should be very easy.  and I think these properties just free for get set，convient to use, do not slow down the Ecluid Distance computation.
Originally posted by @timiil in https://github.com/takuya-takeuchi/FaceRecognitionDotNet/issues/106#issuecomment-643159519

@timiil
I understand what you hope.
But FRDN is porting of face_recognition.
Your requirements will occur large difference between FRDN and face_recognition.
FRDN should keep be simple and can catch up update of face_recognition anytime.
For example, head pose esitmation and age/gender estimation are form of plugin.
Because this function is not in face_recogniton. I keep low coupling of class design.
So your requirements should be achieved by upper layer of applicaiton.

@timiil
I understand what you hope.
But FRDN is porting of face_recognition.
Your requirements will occur large difference between FRDN and face_recognition.
FRDN should keep be simple and can catch up update of face_recognition anytime.
For example, head pose esitmation and age/gender estimation are form of plugin.
Because this function is not in face_recogniton. I keep low coupling of class design.
So your requirements should be achieved by upper layer of applicaiton.

thanks for your reply,  i think may be another way is : dot not sealed the FaceEncoding class, so the application layer can create a new class with  'full gear porperties' which inherited from FaceEncoding.

@timiil
From view point of my experience of development of AFIS, iris and face recogtnion system, feature data class (this case: FaceEncoding) must not have many data.
We create one entity class and it hold encoding data.
class FeatureData
{
    byte[] feature;
    int featureType;
}

class Finger
{
   int fingerPosition;
   FeatureData feature;
}

class PersonData
{
   Finger[] fingers;
   string name;
}

FaceEncoding class represents face encoding data. Why does FaceEncoding class represent person name or origin of image? It's not responsiblity of FaceEncoding.
It is not task of FaceRecognition but task of  business application like domain logic.
If I remove sealed modifier of FaceEncoding class, someone derive FaceEncoding class.
It means that it is hard to modify FaceEncoding class.
My modification of FaceEncoding class may occurs any problem on derived FaceEncoding class.
In this case, someone requires me to revert modification or fix problem.
Yes, it is easy to reject someone's requirements but bad aftertast.
In conclusion, I can not remove sealed modifier of FaceEncoding class.
Thanks a lot.


Broken link to PoSH Documentation

Current Behavior:
404 error when access https://rubrikinc.gitbook.io/rubrik-sdk-for-powershell/. Needs to be https://rubrik.gitbook.io/rubrik-sdk-for-powershell/.
Context:
Need to update quick start guide.

PR merged, closing issue


Missing local.php file in the latest Mautic release version (45_mautic-449)

Hi Everyone,
The most recent version of Mautic (i.e., 4.4.9 45_mautic-449.zipapp_config) does not extract local.php, or app/config/local.php. How can we change the database settings in the current version to reflect the host, username, password, and MySQL database name?
The WinRAR extractor version 6.21 is shown in the following screenshot.

Regards,
Team

Hi there, please post your issue over at https://forum.mautic.org/c/support as Github is for reporting bugs with Mautic whereas the forums are for getting help if you encounter problems :)


Undervolt: Make timer optional

Motivation for this change
The undervolt timer applies the undervolt every 30s which spams the journal. IIRC this was done because the undervolt would not survive certain events (reboot, resume).
I have been running these patches for ~3 months and the undervolt holds perfectly on my Broadwell convertible. I don't know if he tested them long-term but they also worked on my friend's Coffelake laptop.
More test results welcome.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

hmm, this could break the setup for some people. Maybe add a notice to https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2009.xml ?

Sure, I've never written docbook before but I'm sure I can manage.
How can I build the release notes to verify they look correct?

Oh sorry, I missed the notification.
In the root of nixpkgs you can run nix-build nixos/release.nix -A manual.x86_64-linux this outputs a path to the nix-store. in the path you will find the html under /share/doc/nixos/index.html

Bump


Updated quandl bundle to use latest version of Quandl's API.

Coverage increased (+0.006%) to 87.208% when pulling e2cdc2a62ff14e4710b8a64d45104530094f8be9 on quandl_bundle_update into 658d10607ed79c65f1d8d3d8861d371bc5e47539 on master.

Coverage increased (+0.006%) to 87.208% when pulling 442c8edfb518bc7c569009f95c4d99868eb63c41 on quandl_bundle_update into 658d10607ed79c65f1d8d3d8861d371bc5e47539 on master.

Coverage increased (+0.1%) to 87.319% when pulling 93729e922e024335677a174632576d72685c051a on quandl_bundle_update into 658d10607ed79c65f1d8d3d8861d371bc5e47539 on master.

Oh, also could you add commit prefixes to your commit messages?

Coverage increased (+0.02%) to 87.341% when pulling ccfa692e1c67d1cc483e9b7c8eae93aa44f93c4a on quandl_bundle_update into 28a41a54240d63a8e3180c6dacf543851067bbdd on master.

LGTM!

I tried to use the new version of quandl.py, unfortunаtely, it went out of memory on 4GB ram amazon instance when reading df from downloaded cvs file. On 8GB instance everything worked ok. Cаn it be optimized? Mаybe by loаding csv into df in pаrts.

Thanks @VitalyErmilov - I opened #2022 to track it.

Dask provides a similar API to Pandas and supports datasets larger than can fit into RAM.
From https://dask.pydata.org/en/latest/dataframe.html :

A Dask DataFrame is a large parallel dataframe composed of many smaller Pandas dataframes, split along the index. These pandas dataframes may live on disk for larger-than-memory computing on a single machine, or on many different machines in a cluster.

Docs: https://dask.pydata.org/en/latest/
Src: https://github.com/dask/dask
Pypi: https://pypi.org/project/dask/
CondaPkg: dask

@VitalyErmilov can you please test the latest master branch on your 4GB instance? and maybe post your findings in https://github.com/quantopian/zipline/issues/2022


How do I encode to a writer in 0.4?

There used to be write_hex that encoded straight into Writer, but now it's gone and I'm not sure what's the most idiomatic way to replace it without temporarily allocation.

Uhh yeah since 0.4 hex uses "from-iterator" types for encoding result and thus doesn't support to encode directly to Writer any more :x
This "ability" will most surely come back in later versions but for now yeah there's sadly no way without allocating beforehand :(
I'll tag this issue as "bug" or something in order to have it somewhere for the "1.0" milestone


Open Arizona

Open Arizona is a portal of open-access titles from the University of Arizona Press. We are adding new content to the site monthly. Recently, we added titles in anthropology, archaeology, border studies, and Latin American studies. Please check back frequently to see our latest offerings.

https://open.uapress.arizona.edu/
Collection of 124 contemporary books published under CC BY-NC-ND 4.0

NC-ND means "non-commerical, no derivatives" and is not suitable for this project.


End case when puppeteer  sometimes load certain element, and sometimes don't

Steps to reproduce
Tell us about your environment:

Puppeteer version<EMAIL_ADDRESS>Platform / OS version: Ubuntu 18.04.2 LTS
Node.js version: v12.9.1

What steps will reproduce the problem?
If you try the following simple example, you will find it is working without a problem:
    const puppeteer = require('puppeteer');

    (async() => {

      const browser = await puppeteer.launch();
      const page = await browser.newPage();
      //await page.setViewport({width:800, height:800});

      const response = await page.goto(`https://finance.yahoo.com/chart/TSLA`, {waitUntil: 'networkidle2'});

      let selector = `div#chart-toolbar ul > li`
      page.waitForSelector(selector)
      selector = `div#chart-toolbar ul > li:nth-child( 7 ) > button` // > span > span
      page.click(selector)

      const inputElement = await page.$('div#fin-chartiq')
      await inputElement.screenshot({path: 'yahoo-finance.png'})
      await browser.close();

    })();

However, if you take off the comment and expose the await page.setViewport, you will find that it will error out with:
UnhandledPromiseRejectionWarning: Error: No node found for selector: div#chart-toolbar ul > li:nth-child( 7 ) > button
Why such huge different?
What is the expected result?
The behavior is the same, with or without page.setViewport.
What happens instead?
The behavior is different.

@suntong try with headless: false and see if that makes it more consistent. I'm having similar issues and I think headless is a factor.
https://github.com/puppeteer/puppeteer#default-runtime-settings

Oh yes @phawxby, I forgot to mention that with headless: false, the problem would go away.
So, yes indeed, headless is a factor.

ping.
problem still exist

Hm, I believe you need await before waitForSelector and click. Also, a different viewport might have a different DOM (determined by the website) so you need to make sure the selector is valid. I don't think there is a bug here but feel free re-open the issue with a more recent repro (the currently mentioned selectors don't seem to exist on the page anymore).


'new' label template and style on overall score

Template for the new label:

The setting of the flag needs a bit thought, the I did implement it in one way, but I'd rather do it differently.

@sethladd – thoughts?
@isoos – guessing we'll add the isNew argument later...

guessing we'll add the isNew argument later

yeah, that's the plan

love it, let's try it!


sos:font size for merged cells

thanks a lot for this powerful package.
I run code
table.cell(i, j).paragraphs[0].runs[0].font.size = Pt(9)
but the merged cell font is still Pt(10)
can anyone help, emrgercy

solved, but it was pretty slow now


MT-Zip 2.0.1 update

Simply updated the method names, some simple error reporting, added missing arguments with the value of None.
New features have been added in the 2.0.0 update, if you would like to add support for them in this library, this PR can be made a draft PR and we can discuss on how to implement this.
I am not a C/C++ programmer, so I don't know how optional arguments or error reporting should be done.

Thanks, I will write optional arguments.


Include player(s) position in combat log

Love the plugin! I think it'd be really cool if the player's position was included in the combat log, to further enable analysing one's combat performance. For example in something like Corrupted Gauntlet, the ability to review positioning could help a lot in finding lost ticks. (and it'd enable the possibility of a replay, just like in warcraftlogs)

Hi,
This is something I'd probably like to add eventually, Though there's a few other things I want to get to first and the Runelite approval process can be kind of slow.

As a fellow wow nerd, I love this idea and the direction of your plugin Eric. it certainly provides a lot of fun programming challenges. If you're interested in contributors, reach out!

Hi @Scotho ,
I'm happy to take contributions for both the plugin and the website. If you're interested in contributing you can join the discord here.
https://discord.com/invite/ZydwX7AJEd

Position tracking has been added in https://github.com/SuperNerdEric/combat-logger/commit/14c88c731c9544b49ff37089f84b73009af83a3a, but it won't go live until RuneLite merges https://github.com/runelite/plugin-hub/pull/6903


Complete the annotation of FindLibElf.cmake

I think the whole file is copied from bcc

I don't think that we use it. It'd be better to remove the test completely.

I don't think that we use it. It'd be better to remove the test completely.

OK


engage with Floodgate

good first meeting, strong interest in joining our rounds, now need to substantiate with the usual financial projections


tdigest segfaults with PG10 on amd64

tdigest segfaults with PG10 on amd64 (x86_64):
https://github.com/df7cb/tdigest/runs/4719521616?check_suite_focus=true

Thanks for the report - indeed, the PG10 seems to work differently when combining results of partial aggregates. Should be fixed by 745b58ddc2d7b282a93193162c1a5c69a476e361.


Improved: ContentWrapper empty string result breaks simple FTL null check and default syntax  (OFBIZ-10194)

Improved: ContentWrapper empty string result breaks simple FTL null check and default syntax (OFBIZ-10194)

This looks good to me an is field tested in custom projects for some time now.
Any objections to merge?


fix: don't error for missing baseConfig, when makeServer is provided

This PR attempts a fix for https://github.com/miragejs/ember-cli-mirage/issues/2212.
The bug arises when an app defines an export function makeServer(), but doesn't also define an export default function ().
According to the Mirage docs, this use case is supported. In practice, it is not.

Typically, the /mirage/config.js file contains a single default export which is a function... You can now opt in to ... exporting a single named function called makeServer instead.

https://github.com/miragejs/ember-cli-mirage/blob/f0ce745cfbe8f10a0b7ae7b2c32aa1c46e0e63e0/tests/dummy/app/pods/docs/advanced/server-configuration/template.md
The change in this PR was made naively, editing the specific conditional causing the error to be thrown, without a full understanding of code being changed. Nonetheless, this fixed my own issues and the Mirage development server ran without issue.

I have created a PR https://github.com/miragejs/ember-cli-mirage/pull/2215 that in addition to addressing this concern, will also allow what was the makeServer function to actually be a default exported function. This should help with the embroider issues.
If you agree, I would close this PR in favor of that one.
Sorry for the churn, if I only knew then when I know now :)

Thanks so much! ❤


Bump embedded Python to address vulnerability

What does this PR do?
Bump embedded Python to 3.11.8

Motivation
Vulerability: CVE-2023-5678 on Windows because the embedded Python ships its own version of OpenSSL. For simplicity/consistency bump embedded Python for linux and macos too.
Jira cards:
https://datadoghq.atlassian.net/browse/VULN-5166
https://datadoghq.atlassian.net/browse/AI-3778

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Add visible window setting

Would be great to set initial visibility of window with a new window setting that gets passed to https://docs.rs/winit/0.24.0/winit/window/struct.WindowBuilder.html#method.with_visible.
Related to #822 and implementing a sys tray for our application. We want to have the option to launch it to the tray. We have a workaround but we get a very brief "flash" of the iced window before it gets hidden.

I'll open a PR for this


Add configuration options for chat history plugin

This is my first time contributing, but I'd love to keep adding features! This pull request fixes issue #2418 by adding a configuration menu for the Chat History plugin. I'm not 100% sure of every situation where each of the ChatMessageType enums are fired, so I'd love some insight on that.

The solution for this would be just not filter game chat and not add 90 toggles

@deathbeam The "GAME" message type is already allowed through the filter, though, but there's other message types that go through the game tab (server messages, examine messages, "filtered" messages) that are not saved. I personally wouldn't want most of these messages to be saved in the chat history without an option to turn them off, but I can remove or condense the configuration if that's what you think would be best.

Just a heads up, both RuneLite formatted and actual chat messages contain <col, as the way to colour chat messages is to add a <col={hexcolour}> before the message


Add NPM metadata to the create-plugin template

Hey, I just made a Pull Request!

Subsequent to #3585 and #3624, update the create-plugin template to properly add the NPM meta data.
Obviously, if a user were to want to use a plugin locally or seprately from the app at whole, they would modify this metadata, so I figured it was better to template it than not and have the users change it if they need to, than leave it empty.  Happy w/ alternate opinions on it. In future, could even have the cli ask a question about the URL if not defaulted, that sort of thing. For now, fired a patch to the cli for it.

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)

Ah right, I see what you mean!  Meaning in my custom scaffolded app from say npx @backstage/create-app; got it.  Will take a peek!

As I noodle this out, I think for now I'll close this one and move on. 👍


Full schema sync functionality

Add functionality to the SDK to submit a full json schema and the SDK will try to update/synchronize it with the ExH backend.

Take a look at the current schema-sync-tool for input

Still necessary since this is in exh-cli now?

Indeed, the Extra Horizon CLI should be used for this.
Docs: https://docs.extrahorizon.com/cli


Do not mutate settings object

I have an issue because of mutations in my project: https://github.com/styleguidist/react-styleguidist/issues/348

@probablyup Could you please review this PR?

Sorry, I've been absolutely swamped at work. I'll make time for this over the weekend. Could you tweak the PR to not use object.assign and instead just do spread applies?

Sure, will do that ;-)

This was merged separately in #97. Thanks for your contribution, and sorry for the delay!


Clicking home button doesn't trigger onChangeDateTime

Clicking the home button to go to today does not trigger onChangeDateTime.

Hi. I also encountered this problem.
Button logic:
one click (musedown) - only focus on the current date, but does not change input value
Double click - change input value and close the window.
I also expect changes in one click... )


Handling null exit codes

I've been having trouble with grunt watch and nodemon. I'm starting these with:
concurrently \"grunt watch\" \"nodemon\"

When I press CTRL+C, nodemon exits with 0 but grunt watch exits with null (for whatever reason) which results in concurrently returning 1.
I've read through #110 where the solution to something like this was to add --success:
concurrently --success last \"grunt watch\" \"nodemon\"

I'd expect this to always take the exit code from nodemon but from my experience it takes the exit code from whichever process happened to exit last:
iMac:project me$ npm run dev

><EMAIL_ADDRESS>dev /Users/me/repos/project
> concurrently --success last "grunt watch" "nodemon"

[nodemon] 1.15.0
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node dist/index.js`
Running "watch" task
Waiting...
^Cnodemon exited with code 0
grunt watch exited with code null
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: `concurrently --success last "grunt watch" "nodemon"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/me/.npm/_logs/2018-02-19T15_18_01_383Z-debug.log
iMac:project me$ npm run dev

><EMAIL_ADDRESS>dev /Users/me/repos/project
> concurrently --success last "grunt watch" "nodemon"

[nodemon] 1.15.0
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: *.*
[nodemon] starting `node dist/index.js`
Running "watch" task
Waiting...
2018-02-19T15:18:14.888Z ams:server Listening on 3000
^Cgrunt watch exited with code null
nodemon exited with code 0

I think I've found the cause of this: https://github.com/kimmobrunfeldt/concurrently/blob/cd5d53017152bfdf6a64dffb923c00b952c86835/src/main.js#L341
Instead of just pushing exit codes into this array, if they were added using childInfo.index...
exitCodes[childInfo.index] = exitCode;

...then they would be in the same order as the commands that are passed to concurrently.
Would this be worth adding?

@Zakini Did you find a work around for this?


build errors in both backend projects

Hi,
I face several build errors after just cloning the main branch and doing a nuget restore.
Though all packages and dependencies loaded successfully, there remain several errors.
Any idea how to fix this?
Any help appreciated.

I'm not an expert when it comes to the backend Projects but it seems like something went wrong with the Nuget packages. As far as I can remember you don't have to rebuild any packages. To reproduce your issue I cloned the project in Visual Studio und just hit play for the Registry and AAS Generator. For me it worked right away. Try deleting all Nugets and install them again (Version 1.0.0). Check if .dotnet is installed in the right version. If that not helps, you could also try to run the Basyx Dotnet Examples: https://github.com/eclipse-basyx/basyx-dotnet-examples or the Basyx Java Examples: https://github.com/eclipse-basyx/basyx-java-examples
As long as you start a registry and an AAS, the GUI can visualize that AAS.
Hope this helps :)

Thanks for your help.
I finally got it to work by not only clearing all by nuget caches.
I also had to reset my nuget feeds and just re-added nuget.org as source. https://learn.microsoft.com/en-us/nuget/resources/nuget-faq#i-don-t-see-nuget-org-in-my-list-of-repositories--how-do-i-get-it-back-
Maybe other nuget sources got in the way there.


Strange code formatting in 6.11's exercises

It should probably look like this:
class Monoid m <= Action m a where
  act :: m -> a -> a

However it comes out like this:
class  Monoid   m   <=   Action   m   a   where
    act   ::   m   ->   a   ->   a

Similarly with the other two examples.
Browser: Firefox 50.0, macOS Sierra

Potential solutions to the issue:

there seems to be some hidden characters / blank spaces in the markdown (view line 576, 578 in image)

Code sample only seem to fail when with indented sections that use X>
I can't seem to find X> in the Markua specs which is what LeanPub uses.

Looks like a CSS bug. If I remove the padding and margin from things with the pre code selector, it looks fine.
cc @leanpub Not sure how to file a bug report ...

I've filed a bug report on the LeanPub google group, so I'll close this.


Fix changing a pointer to be computed when there is a subtype

This was a regression introduced by #6542, which moved the deletion of
the pointer to happen after processing subcommands. It turns out
that that was correct, but in order for it to work right around
inheritance, we need to make sure that the propagation of alters into
child types occurs as "caused" actions, so that they happen after all
the processing for the parent objects themselves. (Including the
actual deletion of a pointer when it is changed to computed.)
Also, make sure that constraints get propagated when this change
happens.  That change then required a knock-on change to expression
propagation to avoid messing up error messages in that case.
Fixes #6542

This was a regression introduced by #6542

You meant some other PR here, not the reported issue?

Sorry, introduced by #5411

What are "caused" actions in delta commands?

What are "caused" actions in delta commands?

They run after the command is finished


Ungroup Command

Please update your user guide to gn for ungroup in the example command section.

Also mentioned in #71.


mozSystem parameter for window.XMLHttpRequest on FirefoxOS

To be allowed to do Cross-Domain requests in FirefoxOS you need the permission, and window.XMLHttpRequest has to be passed a constructor parameter like this:
new window.XMLHttpRequest({mozSystem: true});

See https://developer.mozilla.org/en-US/docs/DOM/XMLHttpRequest#XMLHttpRequest()
I currently patch Angular in the creatHttpBackend function to get this working, would be nice to not have to do that.

+1

Yay!


Search completion broken for amazon uk

Seems to be working for all other domains. Here is the error I get for amazon.co.uk:
[16:52:56.616] Amazon Suggest[Script Filter] Queuing argument 'ne'
[16:52:56.934] Amazon Suggest[Script Filter] Script with argv 'ne' finished
[16:52:56.942] ERROR: Amazon Suggest[Script Filter] Code 1: https://completion.amazon.uk/api/2017/suggestions?limit=11&suggestion-type=KEYWORD&alias=aps&mid=undefined&prefix=ne

/Users/Dave/Library/Caches/com.runningwithcrayons.Alfred/Workflow Scripts/637DCEC5-CD2D-4B84-AB36-6CA6442D5B14: execution error: Error: SyntaxError: JSON Parse error: Unexpected EOF (-2700)

Fixed in 2023.3, just released. Thank you for the report.


Improve style

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

Project coverage is 100.00%. Comparing base (e3fd213) to head (a9095a8).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #87   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           39        39
  Branches         4         4
=========================================
  Hits            39        39

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


fix FST Like query benchmark to remove SQL parsing from the measurement

I noticed that the FST Like benchmark results vary a lot:
Benchmark                                                 Mode  Cnt  Score   Error  Units
BenchmarkNativeAndLuceneBasedLike.testLuceneBasedFSTLike  avgt   25  3.127 ± 0.578   s/op
BenchmarkNativeAndLuceneBasedLike.testNativeBasedFSTLike  avgt   25  3.440 ± 0.555   s/op

When running these benchmarks with -prof jfr to figure out what they were actually measuring I noticed a couple of things:

Most of the time is spent parsing SQL creating the query context

Because the benchmark doesn't properly segregate the FST types or clean up after itself properly (it uses a testng AfterClass annotation which JMH doesn't know anything about), the testNativeBasedFSTLike sometimes measures the Lucene implementation, but when it does measure the Native implementation, the SQL parsing frames are much narrower relative to the construction of the filter operator.

After a couple of changes to use proper JMH lifecycle and to factor out SQL parsing into the setup, most of the benchmark time is spent in construction of the filter operator:
Lucene:

Native:

The results are more stable and tell a different story, which should help drive future improvement in this space:
Benchmark                                (_fstType)  (_intBaseValue)  (_numRows)                                                                 (_query)  Mode  Cnt    Score    Error  Units
BenchmarkNativeAndLuceneBasedLike.query      LUCENE             1000     2500000  SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE '%domain%'  avgt   25   65.626 ±  1.454  us/op
BenchmarkNativeAndLuceneBasedLike.query      NATIVE             1000     2500000  SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE '%domain%'  avgt   25  232.908 ± 17.302  us/op

I've added a parameter to vary the number of blocks iterated over so this benchmark doesn't just measure planning.

 If so, we should revisit the native FST solution and make its performance at least on par with Lucene FST

Thanks for the tag, I will take a look tomorrow

Added a query to match a prefix - Lucene winning for prefix queries too:
Benchmark                                (_fstType)  (_intBaseValue)  (_numBlocks)  (_numRows)                                                                    (_query)  Mode  Cnt    Score    Error  Units
BenchmarkNativeAndLuceneBasedLike.query      LUCENE             1000<PHONE_NUMBER>     SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE '%domain%'  avgt    5  140.027 ± 18.472  us/op
BenchmarkNativeAndLuceneBasedLike.query      LUCENE             1000<PHONE_NUMBER>  SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE 'www.domain%'  avgt    5  177.205 ± 15.342  us/op
BenchmarkNativeAndLuceneBasedLike.query      NATIVE             1000<PHONE_NUMBER>     SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE '%domain%'  avgt    5  304.332 ± 92.048  us/op
BenchmarkNativeAndLuceneBasedLike.query      NATIVE             1000<PHONE_NUMBER>  SELECT INT_COL, URL_COL FROM MyTable WHERE DOMAIN_NAMES LIKE 'www.domain%'  avgt    5  290.019 ± 57.574  us/op

Codecov Report

Merging #8097 (af0c74d) into master (3f11902) will decrease coverage by 56.99%.
The diff coverage is 2.47%.

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

@@              Coverage Diff              @@
##             master    #8097       +/-   ##
=============================================
- Coverage     71.21%   14.21%   -57.00%
+ Complexity     4263       81     -4182
=============================================
  Files          1607     1572       -35
  Lines         83409    81919     -1490
  Branches      12458    12313      -145
=============================================
- Hits          59397    11644    -47753
- Misses        19972    69415    +49443
+ Partials       4040      860     -3180

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
14.21% <2.47%> (-0.03%)
:arrow_down:

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 3f11902...8df9f57. Read the comment docs.

@atris Have you got a chance to look at this PR?

I'm going to go ahead and merge this.

I'm going to go ahead and merge this.

Thanks for this, @richardstartin . I have been swamped at work so couldn't take a look. I will investigate and update on your findings.


Deselect when switching tools

Selection currently remains when switching off select/reshape tool, which can have unintended side effects (especially since selection mode is now kind of hard to see, since the handles are invisible)

Selection isn't cleared when i selected the pen tool. Which means when i tried to select a color to draw with, it changed the color of the previous shape - Paul


Add DNS-SOA record.

Bit different, this one returns a hash with the fields. This is a
present from the Ruby resolver class.

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

thanks for the work!

thanks :)


Create OWNERS file for sdk section

Hi all adding an OWNERS file to the SDK section as suggested by @8bitmp3  in https://github.com/kubeflow/website/pull/2214
Currently proposing as approvers:

Bobgy
IronPan
neuromage
Ark-kun
numerology
I can add myself as a reviewer but would prefer someone else with more knowledge on the sdk than me.

Thanks @alfsuse For reviews, feel free to to add me. And for backup doc reviews - consider @RFMVasconcelos

cc @Bobgy @IronPan @neuromage @Ark-kun @numerology - you were mentioned in @alfsuse 's PR 👍

Thanks for doing this!
/lgtm

Updated reviewers list with 8bitmp3 and @RFMVasconcelos.
I think we should be good now.

Thank you!
/lgtm
/approve


Most recently released redis modules

RedisGraph 2.8.19
RedisBloom 2.2.8
RediSearch 2.4.14
RedisTimeSeries 1.6.17
RedisAI 1.2.7
RedisGears 1.2.5
RedisJSON 2.2.0

Hi @chayim,
Thank you for the PR.
However, these JSON files are meant to be used by the command generator (./hack/cmds/gen.go) and unfortunately the generator does not read the new redis command.json format (ex type: pure-token). So this PR does not work currently.
Instead of supporting the new format now, I suggest we maintain these json files manually until all Redis Modules switch their command.json to the new format as well. And based on my experience, their command.json sometimes differ from the real behavior and require manual fix anyway.
I also checked your diff line by line. It seems that the only module need to be updated is RedisGears? Others look like equivalent with the current version.

@rueian If you can help me understand the generator, I'm happy to poke it a bit.
RedisBloom has changes in return types, as well as arguments - at least for the TDIGEST series of functions. Similarly, there are upcoming command changes in RedisTimeSeries 1.8, and RediSearch 2.6.

Hi @chayim,
After verifying TDIGEST is the only series of functions that need to be regenerated, I have updated and merged your changes. Commands of RedisTimeSeries 1.8 and RediSearch 2.6 are also covered in the latest commit.
In terms of the generator, it is just a script that generates the command builder. The implementation of the generator is not important though. It may be even better if we could rewrite a new one for the new redis command.json format.


RC_CMD_ERROR_UNKNOWN: unkown error

my ymir service is running in a virtual machine， when I train the model, it prompts an error

model_group_23 > TrainModel Detail

Model Name
model_group_23 V2
mAP
0.00%

Task State

Current State
Invalid

Failure

Error Reason
RC_CMD_ERROR_UNKNOWN: unkown error

my evn:
1、vm
2、centos 7
3、Docker version 20.10.16
4、docker-compose version 1.27.2
5、YMIR config=> SERVER_RUNTIME=runc

check the executor log under ymir-workplace/sandbox/TaskTypeTraining//

check the executor log under ymir-workplace/sandbox/TaskTypeTraining//

thanks, the log content is as follows :
t0000001000005256d3f1653449534<PHONE_NUMBER>.268425	1.00	4	169999	[Errno 13] Permission denied: 'nvidia-docker'
cmd exception: Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/mir/tools/command_run_in_out.py", line 95, in wrapper
ret = f(mir_root=mir_root, src_revs=src_revs, dst_rev=dst_rev, work_dir=work_dir, *args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/mir/commands/training.py", line 356, in run_with_args
_run_train_cmd(cmd, out_log_path=os.path.join(work_dir_out, mir_settings.EXECUTOR_OUTLOG_NAME))
File "/usr/local/lib/python3.8/dist-packages/mir/commands/training.py", line 93, in _run_train_cmd
subprocess.run(cmd, check=True, stdout=f, stderr=f, text=True)
File "/usr/lib/python3.8/subprocess.py", line 493, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.8/subprocess.py", line 858, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/lib/python3.8/subprocess.py", line 1704, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
PermissionError: [Errno 13] Permission denied: 'nvidia-docker'

possible issue:
1.nivida-docker was not properly installed, try "nvidia-docker" and fix any issue promoted.
2.run "bash ymir.sh" with sudo (which is not recommended): solution https://www.linuxlinks.com/getting-started-docker-run-docker-without-sudo/

possible issue: 1.nivida-docker was not properly installed, try "nvidia-docker" and fix any issue promoted. 2.run "bash ymir.sh" with sudo (which is not recommended): solution https://www.linuxlinks.com/getting-started-docker-run-docker-without-sudo/

thanks, the problem fixed


update to clang-7

Description:
LLVM 7 is released. Consider update the build tool chain to clang-7.
cc @PiotrSikora

I have this done and pending in my local tree already, but I'm waiting until 7.0 packages are published on apt.llvm.org (right now it still installs pre-release snapshots for clang-7 llvm-7 lldb-7) before opening a PR.
Feel free to assign this to me, thanks.


[FEATURE REQ] Support for api version 2023-10-31-preview

Library name
Azure.AI.FormRecognizer
Please describe the feature.
Support for latest (preview) api version that is 2023-10-31-preview which adds support for Office documents with the Layout model.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.


when fetching dependent packages in the same base repo, revision is not respected

Happened to me with kubernetes, I did a
govendor fetch k8s.io/kubernetes/pkg/kubectl/cmd@01e79b88e1fd589861f92f0d7e2f8b236988a441
But many of the other k8s.90/kubernetes dependencies that were added used the most recent revision

Thank you. I need to have fetch default to the specified revision. Right now you need to use cmd/...@1e7 to specify all the packages.

This should now be fixed on master HEAD.


Support custom glyphs for text and actions

Need to spec this out.

Other than an image payload within the card, wouldn't this require the ability to use custom font files?  How do you practically allow that, other than using a web-hosted and cached file.
By default, HTML Card renderers can handle this with @fontface - but for Android, iOS, UWP?

It will have to be part of the API we give developers using the SDKs, and the custom glyphs will only work in an app that registers support for it. For example Skype could register a custom glyph font in all of the platforms, and they would tell their bot developers what glyphs are available to use.

@matthidinger  So there will be an ability for the host app to "provide" a custom font with glyphs in, and call on them within the payload.  Not restricted to the MDL2 font icons, or platform glyph fonts.

Toasts would find this useful. "Quick Pair" settings team uses Seoge MDL2 font icons for 95% of their images, so if they want to use a toast, they have to first create images out of them which is impractical and wastes storage space.

Did this get anywhere?  Is there a way to to use glyphs easily in adaptive cards?

Dupe of #3053
@VMey we are working on a story to make this possible, hopefully able to share more early next year!


Implement hole punching

I would like to help implement hole punching for understanding i think we need stun server to know peer public ip and port. Client will send udp packing to server with port and adding it public ip and address to DHT So that others can connect directly to it.
1)The two peers both open a UDP socket bound to a random local port
The two peers both contact a server on the internet. This server responds and tells them what IP and port their packet was received from. Since the server is on the Internet the IP/port seen by the server is the external IP and port
2)The two peers exchange information about the external IPs and ports through some mechanism.
The two peers start trying to send UDP traffic to each other.
3)Hopefully the external IP and port used by the NAT for the traffic to the server are the same as the external IP and port used for the traffic sent towards the other peer. So once both peers have sent packets out through their NATs the incoming packets match the mappings and peer to peer communication is established.
Note:
This strategy will work reliablly with full cone, restricted cone or restricted port cone NATs. It will work most of the time with port-preservative symmetric NATs. It will not work with randomising symmetric NATs.
Sun can you provide some guidance for this ???

That's cool. And I have some suggestions:

In our network, there are nodes with the role of relay (with public IP), and they can do the stun server work.
Now, NAT is automatically performed in the node, so when a node is connected, the NAT address of the node will be stored, which will help, when two nodes want to hole punching. And the relay (stun) node will has two functions: one is to tell the connector their external IP, and the other is to help the connector actively send data to the target node.

@sunhuachuang  do we have message for sharing DHT Tablet yet? Relay node should have DHT record for alice and bob can   alice ask relay node to share the DHT records?

@sunhuachuang

Yep, We had the DHT share functions, https://github.com/CympleTech/chamomile/blob/main/src/server.rs#L285 and https://github.com/CympleTech/chamomile/blob/main/src/session.rs#L847


ISPN-7379 AffinityRpcTest random failures

https://issues.jboss.org/browse/ISPN-7379

Need more details @gustavonalle, trying to open HSEARCH-2522 I get "This issue can't be viewed"

@danberindei https://hibernate.atlassian.net/browse/HSEARCH-2522

@gustavonalle Aha, the problem was I was looking for it in issues.jboss.org :)

Ops, hold on, I left a System out in the test....

Ready to go now

Cool, thx @gustavonalle


Add PrettyPrint option

This time json write use only single string format. We have to implement option to use pretty print format.

Done in 1.1


Delete task after a certain number of retries

Should not retry a task for more than 3 times. If it failed for more than one time, it probably fails because of bad config.

Done.