Errors when rendering array-like structures (beta6)

Minimal working example:
class MessagesList extends Component {
    @observable messages = ['one', 'two', 'three', 'four', 'five'];
    @computed get messageNodes() {
        return this.messages.map(message =>
{message}
);
    }
    @action changeMessages = () => {
        this.messages = ['six', 'seven', 'eight', 'nine', 'ten'];
    };
    render() {
        return (

unique

                {this.messageNodes}
                <button onClick={this.changeMessages}>Change</button>

        )
    }
}
export default observer(MessagesList)

Clicking the button throws Uncaught Error: Inferno Error: bad input argument called on mount(). Input argument may need normalising.. Changing the structure to
{[
unique
, ...this.messageNodes]}

fixes the issue. Seems that Inferno tries to patch a dom-fragment by an array of objects, which, recording to @trueadm , might be a bug with VFragments

Any chance you could extend this into a JSFiddle? There's a simple base example here https://jsfiddle.net/lukesheard/axw5869e/

Not a problem!
https://jsfiddle.net/mjz6cc57/

With quick look, I think this is user land error. You are mixin Keyd nodes and non keyed nodes, Inferno should give warning in this scenario when not in prod build I think. remove key="unique" form your test and it works, or add keys for all

Here is a fully keyed version: https://jsfiddle.net/y7008e5k/ and a bug persists

You are still mixin keys, see:

unique
  ==> has key
        {this.state.messages.map(message => {message} )}  ==> Dont have key and cannot have its ARRAY
        <button onClick={this.changeMessages}>Change</button>  ==> dont have key

Found real render issue related to this though. Looks like bug in keyed algo:
steps to reproduce here:
https://jsfiddle.net/y7008e5k/1/

keyed algorithm issue with vFragments has been resolved now.
Also here is fixed version with keys of original ticket:
https://jsfiddle.net/mjz6cc57/1/
(note: Only items inside array are keyed) - div and button are siblings of array thus non keyed

commenting this ticket for the sake of documentation:
Since beta14 Inferno will flatten arrays into single array, so above example must have all nodes keyed (button, array items, and unique). Same behavior than with React


bug: [Rider] [Windows] [nightly] Cody 7.7.0 triggers antivirus behavior blocker when attempting to sign in.

IDE Information
7.7.0-nightly
Rider 2024.3.2
Windows 11
Bug Description
Clicking on the authorize Cody to sign in an account triggers antivirus block and disables the action (can't sign in, not launched via browser)
Antivirus (Emsisoft (Bitdefender engine) windows 11.
Behaviour Blocker: "suspicious behaviour "Exploit" triggered.
Workaround:
Disabling the antivirus will allow the authentication request to work successfully.
Additional context
No response

Hey @jafin
Thank you for reporting this issue. I will redirect this report directly to the team.


request: disable image export

GLTF2 exporter
writes images to directory.
I am missing an option to disable this slow feature.

There's another use for this, too:  Sometimes the source images are JPG files or pre-existing PNG files that the author doesn't want to decode and re-encode.  This is especially true for images that have been connected to the custom Cycles node, as those images must already exist on disk and don't need to be rewritten.
It's especially bad when this exporter reads in a JPG, decodes it, and then re-encodes as PNG for either re-writing to disk or embedding in GLB.  The result has the disadvantages of both JPG artifacts and PNG's large size in the same file.
It would be nice to have the option to keep (in the case of glTF) or embed (in the case of GLB) the pre-existing image files verbatim.

Also talked about in #124.

Fix in progress: https://github.com/KhronosGroup/glTF-Blender-Exporter/pull/181


GetAccountStatementFile: Resource pdf statement file not found using the spring profile

Description

A PDF has been provided by the customer and configured on the sandbox, this file is loaded from the classpath and the path to find the pdf file is defined by the spring configuration profile name.

When the tpp call the endpoint aisp/accounts/${accountId}/statements/${statementId}/file to get the statement resource file the implementation attempt to search the file from the classpath account.statements.${profile}using the value from the profiles, the file lives in account.statements.lbg in the classpath but the profile lbg doesn't exist anymore, the current profiles are spring.profiles.active=k8s,lbg-prod,lbg-prod-green and the resource never be found by the service aspsp- StatementPDFService because the proper profile to find the resource should be lbg

References
Main issue: 23
Zendesk Issue: 59248
Date Raised: 2021-03-23
Environment: New cluster uplifted to TREX release
Analysis

aspsp - StatementPDFService

Tasks

[ ] Improve the strategy to get the statement resource file to avoid get the resource using the spring profiles.

Solution
Improvements on aspsp - StatementPDFService to consume the property value rs-store.statement.resource.file.path from config service that have set the path value where lives the statement resource file to avoid use the spring profiles.
The default value for the statement resource file is account/statements/default/fr-statement.pdf that lives in rs-store classpath, this file is a simple pdf with Forgerock logo and few text lines to identify it as a default sandbox statement pdf file.
To set a specific statement resource file for a customer will be necessary follow the below steps:

1: Add the pdf file in reference-implementation/aspsp/aspsp-rs/rs-store/resources/unfiltered/account/statements/${customer-folder}/${statement-pdf-file}
2: Override the default values in spring config properties for that customer: rs-store:
   # the path to find the statement pdf file issue: https://github.com/OpenBankingToolkit/openbanking-toolkit/issues/28
   # default value in aspsp/rs-store/StatementPDFService -> account/statements/default/fr-statement.pdf
   statement:
     resource:
       file:
         path:  account/statements/${customer-folder}/${statement-pdf-file}

Solution
To protect the statement resources we store these resources on a bucket in GCP and now the  aspsp - StatementPDFService implements the access to the GCP bucket to get the statement resource.
To configure the access to the statement resource follow the below steps:

1: Add the statement resource file to the bucket ob-sandbox-storage
2: Add the below fields in the spring configuration yaml file gcp-storage:
   project-id: ${projectId}
     statements:
       bucket-name: ob-sandbox-storage
       resource: ${resource-path-file}


Update to NuGet 3.0

Do we want to update to NuGet 3.0 before the release? If for nothing else, at least to support semver 2..

Does NuGet 3 finally support semver 2?
When you say update to Nuget 3, do you mean in the version of the library we use for execution of the nuproj MSBuild tasks? Or do you mean support for project.json (which certainly merits a review).

IIRC yes. Semver 2 support should be in NuGet.Packaging.
I mean both. First update current functionality with latest NuGet packages - will probably involve some changes to MSBuild tasks and tests, and then include support for project.json.


feat: initial TTS support

Describe the change
Adds very basic support for create_speech endpoint.
See also https://platform.openai.com/docs/api-reference/audio/createSpeech and https://platform.openai.com/docs/guides/text-to-speech.
Describe your solution
This is an extremely minimal approach that returns the http body (io.ReadCloser). An additional method that saves the result directly to a file is likely to be useful. For streaming, having the raw chunks is most helpful. The test includes an example for saving to file.
I've added the relevant constants, and the basic API should work, could use more tests for sure.
Tests
I likely don't have time to write the necessary tests. I've added a quick and dirty integration test that succeeds - it synthesizes some text and saves to mp3. Listening to the mp3 confirms that the audio is correct.
It would be useful for somebody to write the rest of the mock methods and add a test mp3 file to return.
Additional context
Saving to a particular file format is trivial, but streaming particular forms of audio is tricky and I want to leave that to the end user. Most implementations will require additional format-specific libraries, no need for that in this library.

I feel like the error check that codecov is complaining should be covered by the code here?

I feel like the error check that codecov is complaining should be covered by the code here?

Thanks for the PR! I am giving out a shot.
Regarding your codecov question
speech.go#L50-L51

Added lines #L50 - L51 were not covered by tests

then the computer is correct.
Your test returns an error from the server request handler response, err = c.sendRequestRaw(req);
whereas the coverage complains about req, err := c.newRequest(ctx, http.MethodPost, c.fullURL("/audio/speech", request.Model)
I am new to this codebase - it's awesome though - so unsure if even should be tested.

hi @nguyendc-systran -- think this should be good to review!


Question: What's the correct way to fetch a yet un-available piece of state?

Let say we have a few groups:
Example:
[{
  "_id" : "5af37f93b0eb083c13a4c2d0",
  "name" : "Group A",
  "desc" : "Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book.",
  "isArchived" : false
}, {
  "_id" : "5af37fa2b0eb083c13a4c2dd",
  "name" : "Group B",
  "desc" : "It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged.",
  "isArchived" : false
}, {
  "_id" : "5af37fa9b0eb083c13a4c2e1",
  "name" : "Group C",
  "desc" : "It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.",
  "isArchived" : true
}]

and then we have members in each group:
[{
  "_id": "5af37f93b0eb083c13a4c2d0",
  "name": "Group A",
  "desc": "",
  "isArchived": false,
  "members": [{
    "firstName": "Daniel",
    "lastName": "Cordaro",
    "email"<EMAIL_ADDRESS>    "status": true,
  }, {
    "firstName": "Anne",
    "lastName": "Esperitou",
    "email"<EMAIL_ADDRESS>    "status": false
  }, {
    "firstName": "Christina",
    "lastName": "Bradely",
    "email"<EMAIL_ADDRESS>    "status": true
  }]
}, {
  "_id": "5af37fa2b0eb083c13a4c2dd",
  "name": "Group B",
  "desc": "",
  "isArchived": false,
  "members": [{
    "firstName": "Justin",
    "lastName": "Milano",
    "email"<EMAIL_ADDRESS>    "status": true
  }, {
    "firstName": "Joseph",
    "lastName": "Seed",
    "email"<EMAIL_ADDRESS>    "status": true
  }]
}, {
  "_id": "5af37fa9b0eb083c13a4c2e1",
  "name": "Group C",
  "desc": "",
  "isArchived": true,
  "members": [{
    "firstName": "Jacki",
    "lastName": "Wong",
    "email"<EMAIL_ADDRESS>    "status": true
  }, {
    "firstName": "William",
    "lastName": "Smith",
    "email"<EMAIL_ADDRESS>    "status": false
  }]
}]

Now we have a route for listing just the groups:
/groups

Here we simply fetch the groups data from the database and set it in the store in groups node. As in the first JSON object.
Then we have another route for listing all the members grouped by group name*:
/members

As in the second JSON object.
Now my question is, if the user have already visited the /groups route and then visits /members route then I already have the group data to process the members data using reselect.

[ ] 1. - But what to do in case the user directly visits the /members route first?
[ ] 2. - Is it right way to fetch the group data from members?
[ ] 3. - And if I do fetch it, then how to set the group data on group node instead of members node in current store?

This is how I'm currently mapping it:
const mapStateToProps = createStructuredSelector({
  groups: makeSelectGroups(),
  currentUser: makeSelectCurrentUser(),
  user: makeSelectUser(),
  loading: makeSelectLoading(),
  error: makeSelectError(),
});

function mapDispatchToProps(dispatch) {
  return {
    dispatch,
    fetchCurrentUser: (evt) => dispatch(currentUser()),
  };
}

const withConnect = connect(mapStateToProps, mapDispatchToProps);

const withReducer = injectReducer({ key: 'groups', reducer });
const withSaga = injectSaga({ key: 'groups', saga });

export default compose(
  withReducer,
  withSaga,
  withConnect,
)(Groups);

Store

Not answering the question, sorry.
Its interesting is this a library you're using for reducers and sagas injection  - https://github.com/GuillaumeCisco/redux-sagas-injector ?

This is actually bootstrapped from react-boilerplate.
You can find the injector code in this repository:
https://github.com/react-boilerplate/react-boilerplate/tree/master/app/utils


BUG: Fixing Text Attribute of "About Us" Section.

The font color of "About Us" box text is light-colored that blends with the background hampering User Experience.

I want to change the text-color in order to make the test easily visible.
@sunny0625 Kindly assign me this issue along with GSSOC labels.
I am a GSSOC' 24 contributor.

@sunny0625 I have made the pull request. Kindly look into it and merge.
Also, please assign the level label.


Support filtered $count operations inside $filter & $orderby

Update pinejs-client-core from 6.9.6 to 6.12.0
Change-type: minor

Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!


Posts in the stream dont show that they are pinned, when you click on them they do

When you click on them they do show the pin in the details


Cannot run tests with active Unc0ver jailbreak

Describe the bug
I have created a dummy Xcode project with Unit tests. Before jailbreaking with the Unc0ver jailbreak the tests are installed and ran just fine.
After jailbreaking I get the following error in Xcode when starting the tests:

The test runner encountered an error (Failed to establish communication with the test runner.
(Underlying error: Unable to connect to test manager on 00008030-001A15D83488802E.
(Underlying error: Could not connect to the device.)))

To Reproduce
Steps to reproduce the behavior:

Create a dummy Xcode project
Run tests on the iPhone

works with not jailbroken device
fails with jailbroken device with the error above

Screenshots
See above.
Device (please complete the following information):

iOS Version: 13.5
iOS Device: iPhone 11 Pro
unc0ver Version: 5.0.1

Place an "x" between the brackets if true:

[x] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present on the latest version of unc0ver

Did anyone ever tried to run tests on a jailbroken device?

Same here.

Same here.

I almost thought I‘m the only one having this problem. Hopefully someone can help.

I've been running into this on iOS 12 and 13 as well

@ppamorim @JohnCoates have you found out anything new? Is there a workaround or do you know of any other channels we could get help?

@Cryptiiiic I see you're very active in the other issues. May I ask you if you have any clue what could be wrong that the tests do not run on jailbroken devices? Have you ever tried to do that?

I've just made another try with a dummy test project. This time I got a different error message when starting the tests
Could not launch “testprojectTests”

Sonova Mobile iPhone Xr #4 has denied the launch request.
Internal launch error: process launch failed: The operation couldn’t be completed.
Unable to launch com.dummy.testproject because it has an invalid code signature,
inadequate entitlements or its profile has not been explicitly trusted by the user.

I'm not sure if this kind of error is related to the initial one, but it might help finding the route cause 🤔
Every input is still very much appreciated 👍

This is the dummy project (zip) I use.

Bro.  Just stop.  You are looking for help in the wrong place. Please close this ridiculous thread.

@Hamlock-maneuverr what's the right place for such problems then? And again, running the tests fail only when JB is active.

The jailbreak works correctly yes?

What I can say is that it runs through succesfully and Cydia and tweaks work.
Nevertheless I hope to get some helpfull input from one of the experts to at least understand better why this happens.
I run tests on older phones with Yalu Jailbreak and newet phones up to iPhone X with Checkra1n without any issue.

@ppamorim @JohnCoates no update from your side yet?

I tried to explain to you. You are looking for an answer that does not exist here. You should try twitter. If you are having problems with your tweaks and jailbreak.  then you are in the right place.

Not sure why you are giving me a thumbs down. I'm not the author of said jailbreak. And your issue doesn't exactly apply to this forum. Should I give you a thumbs down for no real reason as well?

I tried to explain to you. You are looking for an answer that does not exist here. You should try twitter. If you are having problems with your tweaks and jailbreak. then you are in the right place.

What community on Twitter would you ask such questions?
I still think my issue is not completely wrong on this GitHub page. I just do a Jailbreak with Unc0ver, install Cydia and after that I cannot run any tests on the device anymore, which works perfectly without the jailbreak. So from my perspective it's a problem caused by Unc0ver and I guess here are the Unc0ver experts which know what could cause the issue behind the scenes and might know a workaround or even have an idea how to fix it.

Not sure why you are giving me a thumbs down. I'm not the author of said jailbreak. And your issue doesn't exactly apply to this forum. Should I give you a thumbs down for no real reason as well?

What do you think Github issues are used for, exactly?
This is a side effect caused by the jailbreak. Bugs related to the jailbreak should be posted here. Other jailbreaks such as Checkra1n do not exhibit this issue.
Something about the jailbreak or its bootstrap is causing the test runner service to crash when used with Unc0ver.
Twitter is not the right forum for that, this is.

See the problem is you both took "you should try twitter" as an insult of some sort. I was genuine when I said that.  As you can clearly see.  The author simply does not post here or even read this. He does however, read twitter and answer questions on that platform. So I suggest you take this to twitter, as you will most likely get a response their.
See this is why people need to stop just assuming others are attacking them in some way in a forum... RELAX and read the content for what its worth.  Not how it invokes an emotional response in you.  Because most likely this was not the intention of the poster.  Anyway.  I hope you take my advice. That is, If you actually want a response to your original question.

You've been more productive in this single post than in your previous four.  You started by calling the thread ridiculous and told us to stop. Funny you don't see that as a baseless attack.
I understand your reasoning, but it's misguided at best. You're kind of missing the point of open source software if you're only seeking help from the creator. I don't see anyone tagging him in this. We're opening it up to the community instead of relying on one very busy person being able to help.
I just figured you were a troll since you have no contributions and all I've seen you do is be negative. Either way this is the right place to ask for help, and I think you need to stop stalking this thread. Your suggestion is noted, but you are not needed or wanted here anymore.

All you have seen me do is be negative?  Your the child going around marking down peoples posts (who are trying to help you). I waited well over a month watching you guys get no response.      No I think I have done WAY more then I needed to.      Now I'm being negative.  Thank goodness I don't give 2 fucks about you.

Still think he reads this?   Or anyone who could help you with your weird issue? Twitter's looking a lot more promising now, isn't it?
But you can keep waiting if you like.  Gotta stick to your guns so to speak. (no matter how ridiculous it may be)

All you have seen me do is be negative? Your the child going around marking down peoples posts (who are trying to help you). I waited well over a month watching you guys get no response.   No I think I have done WAY more then I needed to.    Now I'm being negative. Thank goodness I don't give 2 fucks about you.

All you have seen me do is be negative? Your the child going around marking down peoples posts (who are trying to help you). I waited well over a month watching you guys get no response.   No I think I have done WAY more then I needed to.    Now I'm being negative. Thank goodness I don't give 2 fucks about you.

@Hamlock-maneuverr why are you mad?


Trending category added

🛠 Fixes Issue: #85

📝 Additional Notes
Discord username

Link to any external resources

📷 Screenshots

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.
[x] I am a participant of GSSOC23

Have you implemented the functionality to display trending videos or just added a category?

I have just added a category for now. Please tell me if I need to implement its functionality, I can do that.

@kartikayasija please add functionality as well to the trending category.

@Fenrir-04 The trending functionality is implemented, but the API you have used is not fetching correct data, It only fetches the data of youtube search not the category. This is the problem with every Category.
I think we should use official youtube API for this project.
Only these endpoints are available with this API

Yes @kartikayasija please work on Issue #89 first and then implement trending category.

Sure @KOMALDEWNANI

hey @kartikayasija you have some conflicts in constants.js file, please solve them asap


Should ignore PSModulePath

The agent shouldn't store this as powershell has a standard process to append the system PSModulePath with the use module folders at startup. Having the agent store this prevents that.
See #3228

This LGTM and fixes an issue that @simonsabin lost multiple hours to debugging.

Without knowing that the agent captures environment variables as capabilities and then overwrites them when the agent runs, its really hard to triage issues that occur. Even in knowing that there is nothing in the logs that identify that environment variables are being overwritten by the agent.
The general understanding is that variables are set in the pipeline. or on the machine,
The special behaviour of PSModulePath adds to complexity.


Option to remove borders

It would be nice a option and shortcut to remove borders.

Would that be useful if I add it to the actual website, or as a config that will compile the website borderless?


Storage syscall test

Found and fixed a bug in the storage:
The storage write function tried to read before writing the key, giving an error.
depends on #89

Codecov Report

Merging #121 (9f3c37c) into main (0873847) will increase coverage by 0.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
+ Coverage   68.53%   68.77%   +0.23%
==========================================
  Files          32       32
  Lines        1815     1819       +4
==========================================
+ Hits         1244     1251       +7
+ Misses        571      568       -3

Impacted Files
Coverage Δ

...rc/core/syscalls/business_logic_syscall_handler.rs
57.61% <ø> (+1.22%)
:arrow_up:

src/starknet_runner/runner.rs
85.91% <ø> (ø)

src/utils.rs
50.00% <100.00%> (+2.94%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

See comments

done!

Added some tests, but due to various bugs in the state they wont pass right now, so they are ignored to get this merged.


fix: index_statistics may miss index_type

Before this, we try to fetch the index_type from stats of index, but many indices don't put index_type in it.

Also, this is a breaking change to the API. Although, I don't think many ppl relies on this field.

I changed title to reflect the breaking change

Codecov Report
Attention: Patch coverage is 33.33333% with 22 lines in your changes missing coverage. Please review.

Project coverage is 79.29%. Comparing base (304f590) to head (dc4ad5e).

Files
Patch %
Lines

rust/lance/src/index/vector/ivf.rs
18.75%
13 Missing :warning:

rust/lance/src/index/vector/ivf/v2.rs
37.50%
5 Missing :warning:

rust/lance-index/src/lib.rs
42.85%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2716      +/-   ##
==========================================
- Coverage   79.32%   79.29%   -0.03%
==========================================
  Files         227      227
  Lines       66771    66792      +21
  Branches    66771    66792      +21
==========================================
- Hits        52963    52962       -1
- Misses      10706    10730      +24
+ Partials     3102     3100       -2

Flag
Coverage Δ

unittests
79.29% <33.33%> (-0.03%)
:arrow_down:

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

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


chore(deploy): Add some clarrification for deploy packlist generation

Closes https://github.com/pnpm/pnpm/issues//7286
The packlist generation used by pnpm deploy is significantly more complex than the docs currently suggest which resulted in a few lost hours for me figuring out why my dist directory was missing (it was .gitignored).
I believe the rules come from https://www.npmjs.com/package/npm-packlist or at least mimic the rules from that package but I couldn't follow the src in the time I had available to be honest.
Instead I made a test repo to check the behaviour described in this PR, it's over here if you want to check my work https://github.com/antiBaconMachine/pnpm-deploy-packlist-examples/tree/main.
If someone could confirm that deploy does indeed use npm-packlist then it might be better to just link straight to that more complete documentation, because there are almost certainly other cases I haven't covered here.
Thanks.

A few hours of my time wasted, I hit this same problem.
The documentation says

By default, all the files of the project are copied during deployment. The project's package.json may contain a "files" field to list the files and directories that should be copied.

No, .gitignore is respected, and if you assumed build outputs that are listed in your .gitignore would be packed, they aren't.

I don't think gitignore should be taken into account. We should fix this.

Thanks for merging this 🎉

I don't think gitignore should be taken into account. We should fix this.

Yeah agreed a .gitignore is for git not random other tools.
I did spend an hour looking into the code but it was clear it was going to take me more time then I had available to learn the codebase so I came over here to fix the docs as a compromise. If someone can point me at the interface point where deploy uses npm-packlist I can probably do something.


System don't send ngrok link after restarting

When system reboots on #36 restart op command and script run at startup it don't send ngrok link.
here the the  for the issue

Issue is working fine in latest PR.


feature/login

Awesome! Thank you, Julia! Just pulled it down to test and it looks great. I added you as a member to the swimmy org as well so you can create your own feature branches. I'll make sure to add that step to the onboarding process as well. Appreciate your help!


feat(go): add GetSecuredApiKeyRemainingValidity helper, fix GenerateSecuredApiKey encoding

🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-1898
Changes included:

Add GetSecuredApiKeyRemainingValidity helper
Add tests for GetSecuredApiKeyRemainingValidity and GenerateSecuredApiKey

this code looks familiar ahah

yup it's basically glueing the existing implem with our design ahah


perf(consensus): Make the proposer run setProposal immediately

Component of #3130 . This PR makes the proposer, upon building a block, directly update its local Proposal setting, rather than going through the internal message queue. This is progressing towards the goal, of minimizing the time between "proposer has locally built a block", and "proposer is ready to gossip block parts".

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

Hrmm, I clearly don't understand something about how the WAL works. I can't tell if its just the crashing WAL tests, or the real WAL replay that has an issue here. I suspect its the crashing WAL tests?
If thats not true, only thing I can think of is that the WAL writes need to have the time in ascending order?

Ah I understand the problem. Its that with this change, we first write to WAL
{[Proposal Proposal{1/0 ({hash}:1:C028B329B066, -1) C2DDC7EDA7A0 @ 2024-05-28T07:13:04.786963728Z}]  2024-05-28 07:13:04.804440718 +0000 UTC}

Then
{1 0 RoundStepPropose}

Whereas before it went RoundStepPropose then the proposal (and blockparts)


community: VectorStore integration for SAP HANA Cloud Vector Engine

Description:
This PR adds a VectorStore integration for SAP HANA Cloud Vector Engine, which is an upcoming feature in the SAP HANA Cloud database (https://blogs.sap.com/2023/11/02/sap-hana-clouds-vector-engine-announcement/).
This typescript version is as close as Python version integration (https://github.com/langchain-ai/langchain/pull/16514)

Issue: N/A

Dependencies: SAP HANA Node.js Client

Twitter handle: @sapopensource

Implementation of the integration:
libs/langchain-community/src/vectorstores/hanavector.ts
Unit tests:
libs/langchain-community/src/vectorstores/tests/hanavector.test.ts
Integration tests:
libs/langchain-community/src/vectorstores/tests/hanavector.int.test.ts
Examples:
examples/src/indexes/vector_stores/hana_vector/
Documentation:
docs/core_docs/docs/integrations/vectorstores/hanavector.mdx
Integration tests and examples were run successfully.
Access credentials for the execution of the integration tests can be provided to the maintainers.

Hi @cinqisap, thanks so much for this!
The SAP client dep is quite large (186MB!) and I'm worried it'll slow down our CI. I see that it's only being used as a type in the connector itself - is there any lighter variant?

Yeah, it adds a full minute to our yarn install.
If there is no lighter version, what we can do is split it out into a partner package that is versioned separately, and skip building it in CI.
The other option I see is to inline the examples/ in the docs, comment out the test cases, and define some other interface for hanaClient.Connection or even type it as any so that the client isn't actually a direct dep.
What do you prefer?

hi, @jacoblee93, thanks for the suggestion!
I pushed a new commit to address this issue. My solution is to add support for another HANA Node.js driver hdb which is only about 1MB and thus remove the large @sap/hana-client as a direct dev package.  Since the hdb did not originally support typescript, I added another declaration file to avoid errors during building time.
Now the client type is defined as any and can represent both drivers.
Looking forward to hearing more if the current solution is not as desired.

Yes that's a great solution! Thank you, I'll look shortly.

Ah, sorry missed the part about the declaration. Will fix.

Thank you!


Build improvements

I wasn't happy with the contortions @HughMacdonald was having to go through to avoid GCC's spurious "error: assuming signed overflow does not occur" problem in #1517. In this PR I've taken an alternative approach, downgrading that particular warning back to normal warning status (after it was promoted to error status by -Werror). I'm convinced that the code triggering the warnings is perfectly reasonable, and I've also verified that the warnings are not issued in GCC 4.8 anyway. I've also added GCC 4.8 to the Travis test setup for good measure, so we now also test with the correct compiler version for VFXPlatform 2016.

I'm absolutely fine with not contorting. Once this is merged, I'll rebase my branch and remove the contortions.

I've updated this to include testing with -std=c++11 in addition to the old tests, and fix a bug which came up as a result.


Local variable should be with camelCase.

#Fixed name of variable

Is this project actively being worked on as I want to contribute to it. If yes, can you please assign a good first issue to me in this project? I am a junior, completing my Bachelor's degree in CS and this will be my first contribution.


[Feature request] Add import from clipboard feature

As FreeOTP+ was unable to recognize a QR code (see #277), I had to do this with Camera app, then copy the recognized URI to clipboard, create a text file with text editor, paste the URI to file, save it, and only after that I was able to import it in FreeOTP+. Please provide an ability to import URI/JSON directly from clipboard.

In some later Android versions, if you select and copy the URI, it has the option to open in FreeOTP+. But yea, import from clipboard is a viable option indeed.


I want to use a separate version number for each flavor. how can I do it

Questions and bugs
If you need help with the use of the library or you just want to request new features, please use the Discussions section of the repository. Issues opened as questions will be automatically closed.


MAINT: Fixed two unused imports in scipy/misc.

Reference issue
Further work on gh-11529
What does this implement/fix?
To reduce Pyflakes warnings, remove two(2) unused imports.

LGTM, merged. Thanks @pvanmulbregt


Rotation animation not playing as expected

I am trying to animate a spinning windmill. 2 key frames: one at Euler X 0° and one at Euler X 360°. It does a full rotation in blender as expected, but upon export the animation doesn't move because 0° apparently equals 360°. If I use a value of 359°, the animation moves by -1° instead of +359°. I didn't have this issue with an older version of the exporter, before multiple animations were implemented.

I can't reproduce this with a simple spinning animation, can you share a .blend file? You can drag .ZIP files into github comments.

I have the same issue. I tried to spin an object the whole way around from 0° to 360°. The same thing happened for me, the object didn't rotate at all. After finding this issue I tried using 3 keyframes, at 0°, 180° and 359° (Assuming 360=0 I thought that using 359 might be better). This time the animation actually showed, but now it turns until 180° and then reverses direction, going back to 1° instead of continuing to 359°. I also tried 0°, 180°, 270° which produce the same issue. I couldn't get the object turn past 180°.
I looked at the GLTF model in both the BabylonJS viewer and Don's.
rotation-example.zip

Thank you for posting an example. I'm sorry for abandoning this issue. What worked for me was to use the graph editor's Sample Keyframes command.

Thank you for posting an example. I'm sorry for abandoning this issue. What worked for me was to use the graph editor's Sample Keyframes command.

Moving this to the new repository (https://github.com/KhronosGroup/glTF-Blender-IO) where it still seems to be an issue. Sampling keyframes will work around the issue in the meantime.


fail early if readinto attribute does not exist when trying to load file

Proposed changes
Fixes a small bug when attempting to load a file from python missing the readinto func. Now it catches early and avoids attempting to cast to PyFileReader
The following would throw an error of unknown attribute as readinto does not exist in text mode
import mlx.core as mx
mx.load(open("./temp.npy"))

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Nice! Though I still find it confusing that it fails for that case as the error message is now:
ValueError: [load] Input must be a file-like object, string, or pathlib.Path

But open(...) is a file like object is it not?
CC @jagrit06 is the loader supposed to support that case?

open() by default opens in text mode creating a TextIOWrapper which returns text,
While open(..., "rb") will return BufferedReader which returns bytes


Set requirements for users to vote

I'd like the ability to set requirements in order for users to vote.
This would prevent competitors or random new users from voting.

How would you see this working with our current api approach?

Would just need to pass in a Boolean for if the user is eligible for voting & pass in a closure (ineligibleHandler?) for if they try to vote & aren't allowed to.
I have the RoadmapView inside of a RoadmapContainerView that I created & I'd create a viewModel for that which would have a method that calculates if the user is eligible for voting & it'd also handle presenting a toast message if they try to vote & haven't met the requirements yet.
This does add complexity to the Roadmap library, but I feel like a lot of devs would like this functionality.

Hm, wouldn’t the more simple approach be to not show the roadmap to people who are not allowed to vote? Or would you want to show it to them in read only mode?

Read-only mode could work

Maybe we should leverage SwiftUI for this with the disabled(true) modifier and respect that?
Or maybe: @JamesSedlacek can you try to set RoadmapView(configuration ...).allowsHitTesting(false) and see if that works for you?

@AvdLee neither of these solve the problem.
Both completed remove the ability to scroll & disabled makes the vote button greyed-out.
I think applying allowsHitTesting to the vote button could work?
& pass into the configuration a Boolean property for if read-only mode is enabled?
With a default value of false, of course
isReadOnlyModeEnabled: Bool = false

@JamesSedlacek too bad! In that case, I think we could add a isVotingEnabled property to the configuration object. Would that work?

Yeah that would work!

https://github.com/AvdLee/Roadmap/pull/45
Made a first pass at it

Fixed with https://github.com/AvdLee/Roadmap/pull/45


Reload register resource

After waiting for register to complete, reload it so that it has the
access and secret key available.
Addresses #1369

beat to the punch


Save users on invoices and quotes in a linking table

Create a proper linking table to save users on quotes and invoices instead of just serializing the ids into a field

Coverage decreased (-2.3%) to 27.843% when pulling 6cff30193d60909ee6e7c6fb1f2d8c42b772be41 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage decreased (-2.3%) to 27.859% when pulling 4f927d07a9a7ffa291e3fd06e763a0fab9e75c02 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage increased (+0.09%) to 30.262% when pulling 4f927d07a9a7ffa291e3fd06e763a0fab9e75c02 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage increased (+0.09%) to 30.262% when pulling 4f927d07a9a7ffa291e3fd06e763a0fab9e75c02 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage increased (+0.09%) to 30.262% when pulling af155202bc359de4324d5de5c2b58d75987af4c4 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage increased (+0.09%) to 30.262% when pulling 01e5e0a0ba648001a807562f4cf8d7659a204620 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.

Coverage increased (+0.09%) to 30.262% when pulling c201350e40fe138d27cb6d41c45f902086834e71 on pierredup:users-link into e410cb040a8569f37e1de76fe9c3d258cea7fe50 on CSBill:master.


Harden url fetcher

Sending https://google.com\x0f would crash some lounge instances due to some obscurities in node.
This PR does following:

Strip IRC formatters from messages before searching for urls
Encode urls to prevent crashing
Set timeout and max redirects

Looks good to me, tested working. :+1:
I agree with @astorije here that 3 redirects and 3 seconds is a bit low. It's somewhat easy to blow the 3 redirects. Short url + access to invalid area (= redirect to home page of the site) + redirect to regional page, we've reached 3. Some sites can take over 3 seconds to load when crowded (Reddit/imgur are easy examples) too, so 5/10000 sounds reasonable without being abusive.

Do keep in mind that timeout does not kick in while content is loading
On Sun, 27 Mar 2016, 02:23 Max-P<EMAIL_ADDRESS>wrote:

Looks good to me, tested working. [image: :+1:]
I agree with @astorije https://github.com/astorije here that 3
redirects and 3 seconds is a bit low. It's somewhat easy to blow the 3
redirects. Short url + access to invalid area (= redirect to home page of
the site) + redirect to regional page, we've reached 3. Some sites can take
over 3 seconds to load when crowded (Reddit/imgur are easy examples) too,
so 5/10000 sounds reasonable without being abusive.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/thelounge/lounge/pull/219#issuecomment-201958848

So that's a timeout for establishing the connection? I'm fine with that then.

@xPaw, it makes sense re: title/content, but I often notice that long loading times happen even before the page starts getting in. I'm fine with 5 or 10 seconds then, still wondering if 3 is not a bit too short in some situations.
What about the redirects? @maxpoulin64 has a good point.

It should be, but needs to be looked in more closely to know for sure.
On Sun, 27 Mar 2016, 02:31 Max-P<EMAIL_ADDRESS>wrote:

So that's a timeout for establishing the connection? I'm fine with that
then.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/thelounge/lounge/pull/219#issuecomment-201960797

I've tweaked it to 5/5000

:+1:, thanks for this @xPaw!


Toast notifications redesign

It is time to redesign our notification system. For now, let's start with the most used notifs - warning, error, and info
It will enable reporting issues directly from the app when the user encounters one.

i will add more and more in the future

Design

updated in figma file: toast notif page: https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=9705-14201&mode=design&t=tTxaqMd4D1AUbd4M-11

Warnings
old one:

new one:

Changes

different background
different copy (after i specify it)
close button
duration indicator
paragraph text - 14px
heading - 16px regular
option to report issue

Duration and placement

placement in the top right corner like we already have

lets put the duration to 7 seconds

this bar will shrink as the time is getting past

when you hover over the toast, the timer will stop

Other PBJs

Dark mode

New color pair
we already had this one:

but the light mode was bit weird and is not longterm solution imo
here is the new one:

End

lastly, I would like to slowly rewrite the most used error and warning codes to more human-friendly language. Would anyone be able to help me here? Are there any stats which toast pops up the most

thank you!

rel:

https://github.com/kodadot/nft-gallery/issues/9038


Occasional wrong output on stdout

How to reproduce:
$ cat test
abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc abc test lol
$ replacer abc toast
Patching: test
-- ...  ...
++ ...  ...

Expected: I should see something between the '...'

Thanks for the bug report !


Focusing on a specific task does not log the task timings

Tested Version - v1.1.8

When we focus on task on Timer panel and start the pomodoro timer, upon completion it updates/increases the pomodoro count for the task. But it does not log the time pomodoro timing.

Logs appear on the log file only if we start the pomodoro timer without selecting any task to focus.

It would be great to have the logs appearing on the log file, plus updating/increasing the pomodoro count when we focus on a task.

P.S: I have tested with Simple, Verbose and Custom log format. Does not work in all three formats.

Hi, @GanapathyRaman, sorry about this bug
The bug has been fixed in the latest update. Please let me know if it's resolved.

Thanks @eatgrass for swiftly fixing the bug.
I did a quick test on v1.1.19. It seems to work fine. Pomodoro count increases and logs with start & end time appear in the log file.
But one quick suggestion: Is it possible to append the logs below the task rather than appending the new log at the end of the file?
Example, let say we have two task Task-1 and Task-2:
[ ] Task-1
[ ] Task-2
If I focus on Task-1, and the timer ends, log appear at the end of the file.
[ ] Task-1
[ ] Task-2

🍅 (pomodoro::WORK) (duration:: 1m) (begin:: 2024-01-17 15:14) - (end:: 2024-01-17 15:15)

Now I have to manually move the log below Task-1 so that if I run dataview query, logs can be counted as part of a particular task.
After manual movement:
Example:
[ ] Task-1
- 🍅 (pomodoro::WORK) (duration:: 1m) (begin:: 2024-01-17 15:14) - (end:: 2024-01-17 15:15)
[ ] Task-2
Can this be done automatically? Thanks in Advance!

Yes, here is another issue also mentioned about this #21
And I've tried to implement this feature, appending the log right under the task as a sublist, which means I need to write the log with correct indentation, but the indentation might vary depending on the user's setting, so... I just got stuck in retrieving the correct indentation

Ah I see the challenge now.
But what if we extend the log format (for focused tasks) to have the task name after (pomodoro::WORK). Then we can write the dataviewjs query to sum the duration based on task name. Other (un-focused tasks) can be categorized into General/Miscellaneous. Doing this way probably we might not have to worry about the indentation.
What's your thought on this idea?

Sure, a custom log format can be used to extend the log as needed, and here's an example for reference: https://github.com/eatgrass/obsidian-pomodoro-timer?tab=readme-ov-file#custom-log-template-optional.

Yes, here is another issue also mentioned about this #21
And I've tried to implement this feature, appending the log right under the task as a sublist, which means I need to write the log with correct indentation, but the indentation might vary depending on the user's setting, so... I just got stuck in retrieving the correct indentation

What if you make a setting where we can pick our tab size and then either use that or strip the white space up to the start of the task and add our tabsize to that?


Update Set-SPOTenant.md

Modified the default value for the ShowEveryoneClaim as described here that the setting has changed since March 2018 https://docs.microsoft.com/en-us/office365/troubleshoot/access-management/grant-everyone-claim-to-external-users

Docs Build status updates of commit 6c9106a:
:clock10: Full build: the average full build time is 17 min(s) 34 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 6c9106a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

@Kateyanne Please merge this PR. Thanks.

Docs Build status updates of commit b192fea:
:clock10: Full build: the average full build time is 17 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Fix SMT logging in the arrays encoding

[ ] ~Tests added for any new code~
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] ~Documentation added for any new functionality~
[x] Entry added to UNRELEASED.md for any new functionality

Closes #1594. This PR fixes a logging error and adds a missing constant declaration, both in Z3SolverContext. See the linked issue for details.

Codecov Report

Merging #1604 (3b6feff) into unstable (f0feb05) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           unstable    #1604   +/-   ##
=========================================
  Coverage     74.97%   74.98%
=========================================
  Files           361      361
  Lines         11626    11627    +1
  Branches        536      524   -12
=========================================
+ Hits           8717     8718    +1
  Misses         2909     2909

Impacted Files
Coverage Δ

...rsyte/apalache/tla/bmcmt/smt/Z3SolverContext.scala
80.50% <100.00%> (+0.04%)
:arrow_up:

.../forsyte/apalache/tla/pp/ConstSimplifierBase.scala
95.29% <0.00%> (-1.18%)
:arrow_down:

...he/io/annotations/parser/CommentPreprocessor.scala
96.70% <0.00%> (+1.09%)
: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 f0feb05...3b6feff. Read the comment docs.


Improve code generality to cover for ops without air::AsyncOpInterface

Do you have any examples (tests) of code that is now covered after this improvement?

Do you have any examples (tests) of code that is now covered after this improvement?

Yes it is part of a series of bugfixes to enable a 2k-2k-2k E2E AIR GEMM example. I can add a new test to show this.

Do you have any examples (tests) of code that is now covered after this improvement?

Test added.


Untrusted Hash found

Couldn't find the following hash on this list:
_ "Update der Firmware...
Hash: fd4e3fc1 566c9016 f0218ad9
5bca6994 39d0b509 99fd5aa6
ae848a97 699f73c3"_
Version: 1.0.29, updated via Bluetooth.
I've tried to find it here:
https://jadefw.blockstream.com/bin/jade/index.json
Is my hash trustworthy? Do I have to check the hash maybe somewhere else?
How to reproduce:
Brand new Jade, updated via iOS AppStore Green App, this Friday (12th of April 2024).
Green Version: 4.0.26

If your Jade has a 'rocker', rather than a 'true wheel', it is a Jade revision 1.1.  (Only a relatively small number of 'true wheel' units were made).
So you should probably look in: https://jadefw.blockstream.com/bin/jade1.1/index.json
Specifically, here - full fiel upload (ie. not a delta/patch), updating to 1.0.29 fw, variant including Bluetooth drivers.
  "stable": {
    "full": [
      {
        "filename": "1.0.29_noradio_1118208_fw.bin",
        "version": "1.0.29",
        "config": "noradio",
        "fwsize": 1118208,
        "cmphash": "b0b06a3b280361d4a00d7037579c30ff0935e97595e22155ca7e6823a7341a2c",
        "fwhash": "aee4feff3ea1ee591ac1ab3f283ac828fcedd38d53f69676254cd23c206ef9be"
      },
      {
        "filename": "1.0.29_ble_1445888_fw.bin",
        "version": "1.0.29",
        "config": "ble",
        "fwsize": 1445888,
        "cmphash": "fd4e3fc1566c9016f0218ad95bca699439d0b50999fd5aa6ae848a97699f73c3",
        "fwhash": "673054922e0e2b6e31680fa1317c4245ae5b941f5c552564798457e455dca62f"
      }
    ],

I don't see an issue here - pls close if you agree.  Many thanks.  J.


MKOB: Wire connection doesn't use station name specified

When connecting to a wire MKOB (python) uses "AC Les - Seattle, WA" even though a different station name was specified.
Steps to replicate:

Enter a station name in the text box (not labeled, but defaults to "AC Les - Seattle, WA" when MKOB is started)
Select a wire using the 'Wire No.' spinner (used '10' for the test)
Connect
On a different machine run MorseKOB 2.5 (the Windows application)
Connect to the same wire (using a station name different from the one specified in 1. and not "AC Les - Seattle, WA"
Monitor the upper-right text box (un-labeled 'stations connected' list)
Notice that the station list (eventually) displays "AC Les - Seattle, WA" as one of the two stations rather than the station name specified in 1.

I have an initial (quick) change that resolves this. I will fix a few things up and submit this change, but would rather this be handled with the introduction of a 'Context' object that holds the values for wire and station and 'etc...'.
[Windows MKOB-2.5]

Well, this is awkward. I worked on issue #71 (save and restore settings) over the weekend. Among other things, these changes address the present issue (#53, which unfortunately I had forgotten about). I pushed a commit of my changes late yesterday, but wanted to do a bit more testing today before creating a pull request.
I'm going to need some help untangling the mess I've gotten myself into. It may be time for a Zoom session.

I will pull your branch and take a look. Did you move the station name to Internet.connect (as I did) or did you address it by creating a new Internet object each time with the station name?

Did you move the station name to Internet.connect (as I did) or did you address it by creating a new Internet object each time with the station name?

Neither. I'm ashamed to admit I updated the class attribute Internet.officeID through a direct assignment. It crossed my mind that I could/should have created a new method to do this assignment within the class, but my intent on this go-around was to get MKOB working without making any changes to the core pykob modules. I knew this would all change once we agree on how to handle config settings.
I did use the pykob.config module to fetch and save the settings. I'm sure I did this in a clumsy way, but it worked for my purposes and I realize the code needs a seriously cleanup and it's all subject to change. At the risk of causing you to recoil in revulsion, here's the code I added to kobactions:
def doOfficeID(event=None):
    global myInternet
    kc.OfficeID = kw.varOfficeID.get()
    config.set_station(kc.OfficeID)
    config.save_config()
    myInternet.officeID = kc.OfficeID.encode(encoding='ascii')

By the way, MorseKOB 2.5 users sometimes change their station ID while connected to the server. I do this myself on occasion. There are several reasons for wanting to do this. A good example is when you want to change your status to indicate that you're temporarily unavailable.
I'm sorry if I wasted your time while you're so busy with other projects.

I think we can add a 'set' method on Internet without changing the rest of your code. We can close this issue with your change and I'll back mine out.
I'll pull your branch and test it out. I can check adding a 'set' method and make sure that works. I can then push that change (on your branch) and we can merge it to master.


Issue with vulkan and related packages

This is the content of /nix/store/s1wrbh5lprvifmdj1l0qmjgzg8pxc85h-vulkan-loader-<IP_ADDRESS>-dev/lib/pkgconfig/vulkan.pc on my system:
prefix=/nix/store/gj752368s9jdplvszcq3p37jfk6h4w88-vulkan-loader-<IP_ADDRESS>
exec_prefix=/nix/store/gj752368s9jdplvszcq3p37jfk6h4w88-vulkan-loader-<IP_ADDRESS>
libdir=${exec_prefix}//nix/store/gj752368s9jdplvszcq3p37jfk6h4w88-vulkan-loader-<IP_ADDRESS>/lib
includedir=/nix/store/s1wrbh5lprvifmdj1l0qmjgzg8pxc85h-vulkan-loader-<IP_ADDRESS>-dev/include

Name: Vulkan-Loader
Description: Vulkan Loader
Version: 1.1.114
Libs: -L${libdir} -lvulkan
Libs.private:  -lstdc++ -lm -lgcc_s -lgcc -lc -lgcc_s -lgcc
Cflags: -I${includedir}

It has includedir set to /nix/store/s1wrbh5lprvifmdj1l0qmjgzg8pxc85h-vulkan-loader-<IP_ADDRESS>-dev/include, which does not exist. This path should point to the include directory of vulkan-headers package.
Also, the glfw3 package contain headers that include headers from vulkan-headers package. Should the pc file for glfw3 add vulkan-headers to the include path?
Metadata

system: "x86_64-linux"
host os: Linux 4.19.80, NixOS, 19.09.976.c75de8bc12c (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
channels(saga): "nixos-19.09-19.09.2070.b9cb3b2fb2f"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Ping @Ralith (Vulkan maintainer), I'm seeing -Wmissing-include-dirs when building anything depending on vulkan-loader using pkgconfig.

Thanks for the ping! Drafted https://github.com/NixOS/nixpkgs/pull/82425. Please confirm that this solves the problem.


Remove unecessary custom C++11 logic

Users can/should just set CMAKE_CXX_STANDARD. This option just complicated and overlapped native CMake features.

@martiert

I've demonstrated backwards compatibility in the cmake_coverage build.

Coverage decreased (-0.0004%) to 98.514% when pulling c81c93d21afecf05756d4c7bc82135c7e6e3a236 on thetic:despecialize into a521a90d35ba267089f6bb1fd28c754f9f4a04b9 on cpputest:master.


Document How To Restrict @Listeners Scope

The docs say

Note that the @Listeners annotation will apply to your entire suite file, just as if you had specified it in a testng.xml file. If you want to restrict its scope (for example, only running on the current class), the code in your listener could first check the test method that's about to run and decide what to do then.

Can you put some example code of doing this in the docs so that I can copy and paste it :+1:

@opticyclic
Take a look here : https://github.com/cbeust/testng/issues/973#issuecomment-182685020
I just now added a sample that shows you how to do this on your own.

@juherr
I have raised a PR for addressing this https://github.com/testng-team/testng-team.github.io/pull/1

:+1:


Add Rust guidelines for subclients

@JeffreyRichter, @RickWinter do we want to codify this based on this thread?

Yes, I think we can codify this. As we learn more, we may have to change it (but I doubt it).

👍🏻 I was just thinking of looking for this and making the PR myself :)


WIP: [E2E] windows UX automation

Add UX feature E2E automated testing for windows platform

Can one of the admins verify this patch?

@adrianriobo which changes are needed on the tray to make this work properly?

@gbraad now the code is in on the tray I will open the proper PR with notify icon and menu items attributes to identify. But I will close this PR as this implementation was goautoit based and now I will use the gowinx library instead.

Re implementation based on https://github.com/adrianriobo/gowinx


⚠ Thumbnails API Endpoint has degraded performance

In 9c001e8, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1270 ms

Resolved: Thumbnails API Endpoint performance has improved in 26d7b7a after 20 minutes.


Addional search path for WinMerge

Hi
When I install WinMerge using winget (winget install WinMerge.WinMerge) then the installation path is "C:\Users\%username%\AppData\Local\Programs\WinMerge\WinMergeU.exe" which is different than the hardcoded one. Is it possible to handle that use case?

yep will fix when i get back to a computer

this is deployed. i also deployed a new copy of Verify, so u should not need an explicit diffegine reference

Excellent works correctly. Thank you very much!


[Bugs] AMF is not requesting the retransmission of the Registration Request message

Describe the bug
AMF is not requesting the retransmission of the Registration Request message after security context has been established.
To Reproduce
Steps to reproduce the behavior:

Basic configuration
Real gNB and UE, or simulator
Run standard registration procedure

Expected behavior
In the Security Mode Command message, in Additional 5G Security Information in NAS part, the AMF shall request the retransmission of the original NAS message (Registration Request).
Reference: 3GPP 24.501
Section <IP_ADDRESS>.2 Initial registration initiation, it is specified that: "If the UE does not have a valid 5G NAS security context, the UE shall send the REGISTRATION REQUEST message without including the NAS message container IE. The UE shall include the entire REGISTRATION REQUEST message (i.e. containing cleartext IEs and non-cleartext IEs, if any) in the NAS message container IE that is sent as part of the SECURITY MODE COMPLETE message as described in subclauses 4.4.6 and <IP_ADDRESS>."
Section <IP_ADDRESS> NAS security mode control initiation by the network, it is specified that: "If, during an ongoing registration procedure, the AMF is initiating a SECURITY MODE COMMAND (i.e. after receiving the REGISTRATION REQUEST message, but before sending a response to that message) and: a) the REGISTRATION REQUEST message does not successfully pass the integrity check at the AMF; or b) the AMF can not decipher the value part of the NAS message container IE in the REGISTRATION REQUEST message; the AMF shall include the Additional 5G security information IE with the RINMR bit set to "Retransmission of the initial NAS message requested" in the SECURITY MODE COMMAND message requesting the UE to send the entire REGISTRATION REQUEST message in the SECURITY MODE COMPLETE message as described in 3GPP TS 33.501 [24]."
Screenshots
Correct Behaviour

free5gc's AMF

Environment (please complete the following information):

free5GC Version: 3.0.4
OS: [e.g. Ubuntu 200.04 Server]
Kernel version: 5.4.0-53-generic
go version: go1.14.4 linux/amd64

PCAP File
free5gc_Missing_2nd_Registration_Request.zip

TS 24.501 Section <IP_ADDRESS> said that: AMF shall set RINMR bit when the REGISTRATION REQUEST message does not successfully pass the integrity check at the AMF.
But in initial registration procedure, the UE & AMF have not establish any NAS security context, so the integrity check didn't be triggered and the condition does not successfully pass the integrity check is not fulfilled (because integrity protection is not enable at this moment). As a result, AMF would not set RINMR bit.
TS 24.501 Section <IP_ADDRESS> said that: If, prior to receiving the SECURITY MODE COMMAND message, the UE without a valid 5GS NAS security context had sent a REGISTRATION REQUEST message the UE shall include the entire REGISTRATION REQUEST message in the NAS message container IE of the SECURITY MODE COMPLETE message as described in subclause 4.4.6.
So in initial registration, we think that the UE should include the entire registration request inside the security mode complete which is described at section <IP_ADDRESS>. And RINMR bit is used in the other case like mobility registration procedure, service request procedure, etc.

In all the examples I have (with simulators and/or real cases), the AMF sets the RINMR bit for instructing the UE to re-send the REGISTRATION REQUEST. In my understanding, the UE doesn't send it if not explicitly requested by AMF.

Section <IP_ADDRESS> NAS security mode control initiation by the network, it is specified that: "If, during an ongoing registration procedure, the AMF is initiating a SECURITY MODE COMMAND (i.e. after receiving the REGISTRATION REQUEST message, but before sending a response to that message) and: a) the REGISTRATION REQUEST message does not successfully pass the integrity check at the AMF; or b) the AMF can not decipher the value part of the NAS message container IE in the REGISTRATION REQUEST message; the AMF shall include the Additional 5G security information IE with the RINMR bit set to "Retransmission of the initial NAS message requested" in the SECURITY MODE COMMAND message requesting the UE to send the entire REGISTRATION REQUEST message in the SECURITY MODE COMPLETE message as described in 3GPP TS 33.501 [24]."

It seems that the two conditions for letting AMF set the RINMR bit to 1 won't be satisfied.

The REGISTRATION REQUEST message does not successfully pass the integrity check at the AMF.

Since the REGISTRATION REQUEST is a plain NAS message, AMF won't do an integrity check on it.

The AMF can not decipher the value part of the NAS message container IE in the REGISTRATION REQUEST message.

The REGISTRATION REQUEST even doesn't contain the NAS message container.

Therefore, It seems quite reasonable that the AMF sets the RINMR bit to 0 in the SECURITY MODE COMMAND message.


customElement.define's 2nd parameter is a constructor

Previously it was a call signature, but this is not correct. I'm not sure if HTML - Custom Elements.widl is wrong or whether our interpretation of it is wrong. A call signature does not match actual usage or the semantics as implemented by Chrome, however.

@saschanaz I'm going to merge this since I'm pretty sure it's right based on usage from MDN's example repo, but if you have comments, please do chime in.


[help]How to keep an item in bin?

I want to keep an item in bin.
if there are 64 items in a bin,right-click will get all 64 items, how to keep an item in bin,so get 63 items,then i
can use the bin to store the same item next time,because there are more bins at my auto factory.

You can shift left click to get 1.  Unfortunately, I don't know of a way to lock bins to a particular item.  That would be rather handy and I would suggest making that a feature request if it isn't already possible.

depending on the item transport being used is there an item filter available to only allow that item into bin?


[BULK] Fix 21 broken links in AspNetCore.Docs as of 6/19/2023

For User Story https://dev.azure.com/msft-skilling/Content/_workitems/edit/98992: VALIDATION: AspNetDocs/AspNetCore
Addresses all broken links reported for AspNetCore.Docs on the Broken Link Dashboard https://aka.ms/brokenlinkdashboard as of 6/19/2023 (21 links)

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/blazor-server-ef-core.md
ASP.NET Core Blazor Server with Entity Framework Core (EF Core)

aspnetcore/blazor/components/js-spa-frameworks.md
Use Razor components in JavaScript apps and SPA frameworks

aspnetcore/data/ef-rp/migrations.md
Part 4, Razor Pages with EF Core migrations in ASP.NET Core

aspnetcore/migration/inc/remote-authentication.md
aspnetcore/migration/inc/remote-authentication

aspnetcore/mvc/controllers/dependency-injection.md
Dependency injection into controllers in ASP.NET Core

aspnetcore/razor-pages/sdk.md
ASP.NET Core Razor SDK

aspnetcore/security/enforcing-ssl.md
Enforce HTTPS in ASP.NET Core

aspnetcore/security/samesite.md
aspnetcore/security/samesite

@Rick-Anderson @mcleblanc - This PR is ready for your review and merge. It fixes all broken links reported for AspNetCore.Docs on the Broken Link Dashboard at https://aka.ms/brokenlinkdashboard as of 6/19/2023 (21 links). (Note that all of these broken links were in earlier versions, 1.0-6.0.) Thanks! This should clear all the validation and broken link issues for User Story 98992.


Fix #2881 Avoid propagating server environment

Fixes #2881
Dependencies

[ ] os-lib fix for propagateEnv https://github.com/com-lihaoyi/os-lib/pull/238

Now I don't have idea how to test this. Do we have integration tests where the client is used to call run on a module?

Superseded by https://github.com/com-lihaoyi/mill/pull/3437


factoryFor should have an options argument

According to https://emberjs.com/api/classes/Container.html#method_factoryFor the factoryFor method takes a 2nd argument, "options".

CI failure is related to changes on Ember canary that we need to deal with (nothing wrong in this PR).

I made https://github.com/rwjblue/ember-factory-for-polyfill/issues/12 to track that issue.

ember-factory-for-polyfill 1.1.2 published 🎉


Support annotations for versioning behavior

What was changed

This PR supports providing a default versioning behavior in Worker Options, and a programmatic API to override these defaults:
For example:
	workflow.SetVersioningBehavior(ctx, workflow.VersioningBehaviorPinned)

called  during the first workflow task sets the versioning mode for that workflow to Pinned, eliminating the need of patching.
And
	w := worker.New(c, ts.taskQueueName, worker.Options{
		DeploymentOptions: worker.DeploymentOptions{
                            BuildID:                   "1.0",
		            UseBuildIDForVersioning:   true,
		            DeploymentName:            "deploy-test1",
		            DefaultVersioningBehavior: workflow.VersioningBehaviorAutoUpgrade,
              }
	})

sets the default versioning behavior to AutoUpgrade. Note that DeploymentName, UseBuildIDForVersioning and BuildID need to be set to enable versioning-3.
If versioning-3 is on, but no default or programmatic versioning behavior has been set for a workflow, the server will fail its first workflow task.
Why?

A hint that a workflow is short-running, and therefore, does not need to be moved to the new Build ID on the next task, eliminates the need of patching for many workflow types.
Checklist

Closes
#1663
How was this tested:
Integration test that uses versioning-2 and a grpc interceptor to show that versioning behaviors are propagated with responses.
The dependency on versioning-2 is just to set a default Build ID.

I think the latest commit addresses most of the issues raised, let me know if it is ready to merge...

@antlai-temporal - based on thread above, I wonder if this needs to be on worker.RegisterOptions too if that is the most common way this should be set (at the workflow type level).

@antlai-temporal - based on thread above, I wonder if this needs to be on worker.RegisterOptions too if that is the most common way this should be set (at the workflow type level).

We could have a type-level default in RegisterOptions that can be overridden by the programmatic API. That would be more consistent with the Java annotation+programmatic override I was thinking on. The Getter will probably return just the programmatic to be consistent that we can change worker defaults without patching.
I'm trying to unblock testing for the server team, so I may do that in a separate PR if that's ok...

I'm trying to unblock testing for the server team, so I may do that in a separate PR if that's ok...

I think it's probably good (will defer to @Quinn-With-Two-Ns) but probably want to create an issue to track the work. I do think we should be encouraging that way to set worker-type level value and this should be a lesser-used in-workflow override.

In the comment I'm encouraging to set it in the first task to avoid the first case...

This is why we should be encouraging the workflow register options approach over the inline approach (can even change the name of the inline one to something less wieldy, e.g. SetVersioningBehaviorOverride).

Yeah OK merging into the branch, +1 to everything Chad said.
Once we are further along I would also like to see documentation as to how this workflow API is supposed to interact with versioning info set externally described here https://github.com/temporalio/sdk-go/pull/1692#discussion_r1821431989.

Yeah OK merging into the branch, +1 to everything Chad said.
Once we are further along I would also like to see documentation as to how this workflow API is supposed to interact with versioning info set externally described here #1692 (comment).

@cretz @Quinn-With-Two-Ns
I added the issue https://github.com/temporalio/sdk-go/issues/1698 . Merging as it is to unblock server team, but will add another PR before merging to main.
We will clarify the interactions with the external workflow property setting (after discussing today with @carlydf they seem orthogonal). The keyword "override" is used by this external mechanism, I will stick to default vs non-default to avoid confusion. The getter will only return what it was set (as it does right now).


Different fonts for links

I think it would be really nice to be able to set different fonts for links and hashtags, or at least to be able to bold them.
I thought this already existed because there were so many branches working on it but I couldn't find in the README how to actually do it. If it does already exist could someone let me know how they've used it please?

Ok, nvm I found out how. It's kind of unintuitive though:

self.msg.highlightFontName = "Arial-BoldMT"
self.msg.highlightFontSize = 14


[SPARK-39866][SQL] Memory leak when closing a session of Spark ThriftServer

What changes were proposed in this pull request?
Fixed memory leak caused by not clearing the FileStatusCache created by calling FileStatusCache.getOrCreate when close a SparkSession of Spark Thrift Server.
Why are the changes needed?
When create many sessions in Spark Thrift Server and do querys in each session, the fileStatus of files related will be cached.
But these filestatus will not be cleared when these sessions are closed, the memory leak causes Driver OOM.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Tested by the steps in SPARK-39866.

ok to test

jenkins, test this please

reset this please

jenkins, test this please

jenkins, test this please

no jenkins now :)

Can one of the admins verify this patch?

Different sessions can share this cache. Maybe you can set spark.sql.metadataCacheTTLSeconds to a positive value to workaround this issue?

@wangyum IIUC different sessions' file status cache is separated, but they share memory, so clean file status when closing session asap is necessary: https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala#L157

jenkins, test this please

no jenkins now :)

thanks for reminding :)  @Resol1992 maybe you need to check your github action settings?

jenkins, test this please

no jenkins now :)

@LuciferYang thanks for reminding :)
@Resol1992 maybe you need to check your github action settings?

This is a very good fix when using multiple connections to query the same table and then closing the connections, I don't think relying on TTL is enough.

This is a very good fix when using multiple connections to query the same table and then closing the connections, I don't think relying on TTL is enough.

Different sessions can share this cache. Maybe you can set spark.sql.metadataCacheTTLSeconds to a positive value to workaround this issue?

@wangyum Thanks for your advise. I have tried to workaroud this with setting spark.sql.metadataCacheTTLSeconds = 10, but it does not work, the fileStatus objects still exist after sparkSession is closed.
In fact,

Can one of the admins verify this patch?

also cc @yaooqinn @cloud-fan , could you also take a look?


Change caret size to reflect different modes[Feature]

Describe the feature you'd like
Sometimes it is helpful to know which mode one is in. For example, it would be great if the caret (insertion point) changes to block style like in the terminal to reflect that you entered normal mode just like in Vim, and then returns to the normal thin mode in insert mode. It would be helpful also to have a key binding to trigger the status indicator to reveal the current mode (just like the one that appears when switching modes). This is helpful especially for the GUI Modes.
Thanks and appreciate your work.
Related projects (Optional)
A similar project that implements Vim using AutoHotKey (vim_ahk), has the caret width feature as an option.

Thanks for the request.
The Binding that bring up the current mode is easy to implement. However, there are two concerns with implementing a block-style caret.
Firstly, vim_ahk only dynamically changes Settings > Ease of Access > Cursor & pointer > Cursor thickness on Windows 10, which has no effect on applications that have their own rendering (e.g. Chrome, Edge). Therefore, the support is likely to be uneven. Secondly, if it goes further than vim_ahk and changes the thickness that matches the size of the text, the character width detection will be a major problem.
Once I make a prototype, I will reply again.
Thanks

Instead of changing Caret (cursor), would an easier solution be to switch application icon design in system tray?
Like "Vi" where the "i" is overlaid in front of the "V" and is a prominent bold colour to make it stand out from the "V". Some for other modes "Vgn", "Vgv", and so on...

Instead of changing Caret (cursor), would an easier solution be to switch application icon design in system tray?
Like "Vi" where the "i" is overlaid in front of the "V" and is a prominent bold colour to make it stand out from the "V". Some for other modes "Vgn", "Vgv", and so on...

I am not very motivated to dynamically change the system tray icon like vim_ahk. This is because some people have grouped system icons or hidden the taskbar to make the icons invisible. But it might be a good idea to implement it as an option.
By the way, if you make the value of the cmd_fadeout option large enough, you can make the virtual command line not disappear.
For example
set cmd_fadeout = 3600 " one hour visible

Today, I released v4.2.0-rc1 as the preview version. It supports block-style carets. However, it was difficult to detect the character width dynamically, so we implemented a width-fixed block style and a pseudo block style with selection instead.  Please give it a try.

Hi,
Thanks for you hard work. I really appreciate your efforts.
Unfortunately, I upgraded to Windows 11, and apparently win-vind doesn't work well in it unfortunately. I hope to test the feature once Windows 11 gets supported. Meanwhile, I hope someone with the ability tests the feature.
Thanks

I haven't tried Windows 11 yet. I'll do my best to support Windows 11 as soon as possible.
Thanks again for your feature suggestions.

Hi,
I am sorry, I just tried win-vind again, and it really does work. I really don't know why it didn't work when I tried it previously. I am going to try the feature. Sorry again for the mistake. Thanks for letting me know.

I'm glad it works. Thanks again!

Just tried the feature, it works really well! Thanks, and good luck!


Documentation Matrix that details what functionality is available by Data Source

User Story
As a carbon aware developer, I know what functionality/routes are available given the different data sources and any special considerations when switching between the two
Design Goals

Clearly note which routes are supported by each data source
Note which data sources may be best for particular situations
Note special considerations one data source may have over the other

Acceptance Criteria

[ ] New readme added to docs folder

PR: https://github.com/Green-Software-Foundation/carbon-aware-sdk/pull/216

PR merged


Add change stage event

What feature do you want to get added on the base engine?
Stage change and the offset of thé caracther
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

this has been asked many times, no


Plant Interactor Cake Duplication

Industrial Foregoing Version: 1.12.2 1.9
TeslaCoreLib Version: 1.12.2 1.0.15
When a vanilla cake is placed in the world and is within the working range of an Industrial Foregoing Plant Interactor, it will drop a cake item into the world without destroying the cake. Cakes can be generated quickly and indefinitely as long as the plant interactor has power.
To recreate:
Place plant interactor and connect to power source.
Place cake in working area of plant interactor.

:wave: Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


CCL Ops: all-gather ops refactor

Request to refactor existing all-gather interface under single-api. Currently we have:
ttnn.all_gather()
ttnn.line_all_gather()

It would be great if this could be defined under a single-interface that specifies all-gather topology: {ring, line}

Agree. The underlying all gather program creation already operates this way (the same code works with line and ring) so we just need to thread the appropriate information through from the mesh tensor according.

Agree. The underlying all gather program creation already operates this way (the same code works with line and ring) so we just need to thread the appropriate information through from the mesh tensor according.

We can also infer {ring, line} if user didn't specify / override.

We can also infer {ring, line} if user didn't specify / override.

Right - I didn't explicitly mention it but this is what I was alluding to. Personally, I think we should do the decision making at mesh tensor level but maybe I'm overlooking something that makes it more useful at op level (under mesh tensor).

This refractor has been added to main, can we close this issue?


implement (functional) responsive brain image search engine

currently implemented in index.js but does not output anything (no bot errors tho)

note: index.js and main.js currently both exist but only main.js will remain after this bug is resolved in index.js (externalized solution)

note: index.js and main.js currently both exist but only main.js will remain after this bug is resolved in index.js (externalized solution)


Feature Request: Defined access to GraphData

As mentioned in https://github.com/FHell/NetworkDynamics.jl/issues/54#issuecomment-723142323 I'd like to propose a clearly defined set of accessors to get the underlying data of a graph. I think this is a major need for #49. Since it defines a new set of functions for the end users this interface should be designed very carefully though.
From my understanding the main accessors are
# get the 'view' for the i-th vertex
function get_vertex(gd::GraphData, v_idx) :: AbstractVector{elE}

# get the 'view' for the i-th edge
function get_edge(gd::GraphData, e_idx) :: AbstractVector{elE}

# get the 'view' to the src vertex of the i-th edge
function get_src_vertex(gd::GraphData, e_idx) :: AbstractVector{elV}

# get the 'view' to the dst vertex of the i-th edge
function get_dst_vertex(gd::GraphData, e_idx) :: AbstractVector{elV}

# get all the 'views' to the outgoing edges of the i-th vertex
function get_out_edges(gd::GraphData, v_idx) :: AbstractVector{AbstractVector{elE}}

# get all the 'views' to the incomming edges of the i-th vertex
function get_in_edges(gd::GraphData, v_idx) :: AbstractVector{AbstractVector{elE}}

These are the ones used in the core loop. Replacing this would already help decoupling the NetworkStructures with the rest of the package, for example one could change to Julias build-in views without changing another file. It also simplifies testing of different implementations.
These accessors could be overloaded, for example to get multiple edges at once. One could also think of other useful accesors, for example to get access based on the symbols. Right now, this is done via
function view_v(gd::GraphData, gs::GraphStruct, sym="")

which uses the GraphStruct. I think methods like this should be available for the user without even knowing about the existence of GraphStruct. Once the GraphDataBuffer is in place one should consider why GraphStruct and GraphData are even separate types since they strongly depend on each other.
further thoughts:

this is somewhat related to #52 because it would solve the uncertainty about the right way to access the data
strongly related to #35, one has to think about how to keep the static values in the GraphData up to date with changes to the non-static part (i.e. don't recalculate static edges if the vertex data hasn't changed).
right now a lot of the stuff in NetworkStructures serves the purpose of caching (i.e. all possible queries like the outgoing edges for every vertex are precomputed during setup). Once one allows access to graph data based on symbols in callbacks it is not possible to compute all of the 'answers' during setup. Maybe one could look into a more automatic approach of Memoization based on the arguments of each accessor function.

Inside ND.jl there was a last function which depended on a specific field of GraphData.
function (sef::StaticEdgeFunction)(x, p, t)
    gd = sef.nd_ODE_Static(x, p, t, GetGD)

    (gd.e_s_v, gd.e_d_v)
end

I got rid of it by introducing allocations. Would be good to know whether this function is used by PD.jl in a way which can't be done via the new accessors.

Okay it is used at least in

States.jl (lines 244, 256)
PowerGridSolutions.jl (lines 146, 158, 175, 180)

at least these are the ones which can be found easily. I think it would be now problem to change the calls there. They don't need the full gd.e_s_v which is an Array of Arrays. Instead they are only interested in  gd.e_s_v[i] for a specific vertex i which can be achieved with get_in_edges and get_out_edges.
So since 0.5 will be breaking anyway it might be okay to just get rid of the function?


python error

import pyttsx3
import speech_recognition as sr
engine = pyttsx3.init("sapi5")
voices = engine.getProperty("voices")
text to speech
def speak(audio):
engine.say(audio)
print(audio)
engine.runAndWait()
def take_command():
rec = sr.Recognizer()
with sr.Microphone() as source:
print("listening")
rec.pause_threshold = 1
audio = rec.listen(source)
try:
    print("RECOGNIZING...")
    words = rec.recognize_google_cloud(audio,language="en-in")
    print(f"user said: {words}")

except Exception as exp:
    speak("say that again please...")
    return "none"
return words

if name == "main":
take_command()
#speak("MY NAME IS JARVIS I AM HERE TO ASSIST U")

(How do you make the issue happen? Does it happen every time you try it?) every time i run this code
(Make sure to go into as much detail as needed to reproduce the issue. Posting your code here can help us resolve the problem much faster!)
(If there are any files, like audio recordings, don't forget to include them.)

Expected behaviour
it should be listening my voice command
Actual behaviour
'NoneType' object has no attribute 'close'
File "C:\Users\atif\Documents\PYTHON\JARVIS.py", line 20, in take_command
audio = rec.listen(source)
^^^^^^^^^^^^^^^^^^
AssertionError: Audio source must be entered before listening, see documentation for AudioSource; are you using source outside of a with statement?
During handling of the above exception, another exception occurred:
(If the library threw an exception, paste the full stack trace here)

System information
(Delete all the statements that don't apply.)
My system is . (For example, "Ubuntu 16.04 LTS x64", "Windows 10 x64", or "macOS Sierra".)
My Python version is . (You can check this by running python -V.)
My Pip version is . (You can check this by running pip -V.)
My SpeechRecognition library version is . (You can check this by running python -c "import speech_recognition as sr;print(sr.__version__)".)
My PyAudio library version is  / I don't have PyAudio installed. (You can check this by running python -c "import pyaudio as p;print(p.__version__)".)
My microphones are: (You can check this by running python -c "import speech_recognition as sr;print(sr.Microphone.list_microphone_names())".)
My working microphones are: (You can check this by running python -c "import speech_recognition as sr;print(sr.Microphone.list_working_microphones())".)
I installed PocketSphinx from . (For example, from the Debian repositories, from Homebrew, or from the source code.)

please help me with this

It states that the source is used outside of the with scope. Are you sure that the try/except block and the rec.listen(source) are inside the "with" block? Its just unclear because of the formatting on github :0

It states that the source is used outside of the with scope. Are you sure that the try/except block and the rec.listen(source) are inside the "with" block? Its just unclear because of the formatting on github :0
I have it like this and it works.
`import pyttsx3
import speech_recognition as sr
engine = pyttsx3.init("sapi5")
voices = engine.getProperty("voices")
def speak(audio):
engine.say(audio)
print(audio)
engine.runAndWait()
def take_command():
rec = sr.Recognizer()
with sr.Microphone() as source:
    print("listening")
    rec.pause_threshold = 1
    audio = rec.listen(source)

    try:
        print("RECOGNIZING...")
        words = rec.recognize_google(audio,language="en-in")
        print(f"user said: {words}")

    except Exception as exp:
        speak("say that again please...")
        return "none"
    return words

if name == "main":
take_command()`

@JeziahAnon Thanks. I added python syntax highlight
@SAAD0206 Please check JeziahAnon's reply


Deselect table rows after event

I have a situation where the user must select only 1 row among 2 tables.
Ex .: once a row from table 1 is selected, the row from table 2 must be deselected.
Is there a deselect event or could it be created?

I solved the situation by creating a new state + the "rowsSelected" option. I don't know if it's the best option, but it worked.
const [rowsSelected, setRowsSelected] = useState({ section: 0, row: {}, indexRow: -1 });
rowsSelected: [rowsSelected.section === 1 ? rowsSelected.indexRow : -1],


refactor(meta): deprecate branched sst

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Remove the code of branched ssts.
Currently it is used in

branched sst stat
check whether is valid to move a table id from a compaction group to another compaction group
check whether a compaction task is expired

For 1, we changed to rebuild the full branched sst every time we calculate the stat
For 2, we currently don't have code to move to an existing compaction group, and branched sst related code only exists in cases for existing compaction group, so we can safely remove the usage.
For 3, we change to validate whether the input sst still exists for a reported compact task.
After changing 1, 2 and 3, we don't have any usage on branched sst any more, and we can deprecate it.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

branched sst also tells the ref_count of an object store object.
https://github.com/risingwavelabs/risingwave/blob/7df946504594a294e66f471f5e3300e30cc2450a/src/meta/src/hummock/manager/mod.rs#L3339


Action aborted due to fatal error.

When run command: overlay -v -l /media/root-ro -u /media/root-rw/overlay -f diff
I have got an error: "Action aborted due to fatal error." that points "File %s can not be read for content." (for example /etc/ssh/sshd_config file)
System Description:
debian buster
selinux enabled & permissive mode
relabelling service active

I made a pull request to show the exact error if this is still an issue to you (2021!) #29.


Does this package automatically collect events?

Hello! I was wondering if your package support this: https://support.google.com/firebase/answer/6317485 without doing anything with it or otherwise I have to log all events manually. So for example if I want to log first_open or app_update, does this package do it automatically? Or have I to log it manually an put this on my app?
Thanks!

it logs

What does this mean? Does it log stuff automatically, and if so which things?

app level events like first_open or app_update are automatically logged by firebase SDK. please check their documentation about what they log default.
for other events like view or transactions, you need to log manually.


is A128CBC-HS256 not available currently?

I found the closed issue (#82) in 2019 for using A128CBC-HS256.
But I think A128CBC-HS256 is deprecated now.
Is there any way to use A128CBC-HS256 with JOSESwift?

Hey, it is available since https://github.com/airsidemobile/JOSESwift/releases/tag/1.8.0 was released. You can use it like any other supported JWE algorithm.


Using RDK/imx6 backend on an iMX8 with mouse/touchscreen support

Hi all, I'm currently testing the meta-webkit layer on the Yocto 3.0 (zeus) branch by compiling framebuffer images for a i.MX8-based board. I'm using the cog launcher and the wpebackend-rdk/imx6 backend, and I can browse pages perfectly fine out of the box without tweaking anything.
However, I'm unable to use the mouse or touchscreen to interact with the webpages. I have seen that the RDK/rpi backend code has additional code to support the touch and/or cursor, but this code isn't present in the imx6 flavor of the backend.
Is touch/mouse support available in any way in the imx6 flavor of the RDK backend? Or is it something that needs to be added by hand?

We don't maintain the RDK backend here, can you file an issue in this tracker? https://github.com/WebPlatformForEmbedded/WPEBackend-rdk

Will do, I will close this issue then


DiffSinger: support different dictionaries for Chinese, Japanese and Cantonese

DiffSinger voicebanks can use different dictionaries for Chinese (dsdict-zh.yaml), Japanese (dsdict-ja.yaml) and Cantonese (dsdict-zh-yue.yaml). For example, DiffSinger Chinese phonemizer tries to load dsdict-zh.yaml first. If dsdict-zh.yaml isn't found, it will load dsdict.yaml as before. This will make it easier to develop multilingual voicebanks.

This change doesn't affect existing voicebanks.

Add DiffSinger Japanese phonemizer (DIFFS JA). Currently the only difference between DIFFS JA and DIFFS is that DIFFS JA tries to load dsdict-ja.yaml before loading dsdict.yaml.
Moved diffsinger phonemizer code to a subfolder.

conflict


Hello! its levi :D

What is your question?
the html port isnt being updated

That's because a change was made to make html5 more stable, as multiple sites uses the web framework. In the future there will be a way to host a dev build site yourself using heroku tho.

Ah i see


Attempt to resolve linting issues in #43

Attempting to resolve linting issues found in CI process during PowerShell/xDnsServer#43.

This change is

@mrala,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

Closing, just wanted to test this PR.

Reopening to run tests.

@mrala,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Closing in favor of #43


add ICluster.isLocalMember(ClusterMember member) to validate if a given

add ICluster.isLocalMember(ClusterMember member) to validate if a given member is local member
improve the examples

+1