Fix to Pagination position is retained between searches #2159

References
Add references/links to any related issues or PRs. These may include:

Fixes #2159

Description
I made a little fix, now when search submit is trigger, it must return to first page. I think this is the correct behaviour for search components.
Instructions for Reviewers

Open https://demo7.dspace.org/search (in this case, the /search route)
Go to the last results page
Change the query to test → bound to have fewer results than everything
"Your search returned no results"
The pagination UI is not shown, the only way to find out you're on a page that doesn't exist is through the URL: it will still have e.g. spc.page=170

List of changes in this PR:

First, I added firstPage Query Param to force navigating to first Page after submit
Second, I changed test's of Search-Component-Form in order to expect the correct behaviour of the search.

Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

Thanks @hugo-escire. I can reproduce the bug as described and this patch fixes it.


Adding plots to app.py

Team,
I have created an app.py file. Please try to run it locally on your machines.
Also, send your charts as soon as possible. We need to host this app as well.
I will try to host this version, just to set up the workflow.

I've updated the map accordingly regarding issue #10. Could you please take a look and if you think the new initial chart is now better, add it to your app.py? Thanks.

@singh-karanpal do you think it's necessary to put the map and associated line chart to a separate page within the dashboard if this is possible (like adding a navigation bar or table of contents)? Having four graphs on the same page might be a little confusing for the users. Thanks.

@jasmineqyj Nice point! However, I feel all of these charts are associated and represent interlinked information. Users should be able to get all information by looking at just a single screen of the dashboard rather than going to and fro between multiple tabs. I think that might be a little annoying for users in this particular use case.

The app layout is finalized and deployed so closing this issue


Update v3 docs

[x] Is this related to documentation?

@xtreamwayz just only one note, we should probably update also description in composer.json, now we have there: PSR-7 Middleware Microframework based on Stratigility and any single word about PSR-15 ;-)

…and the description of this repository!
/cc @weierophinney

…and the description of this repository!

Same for the skeleton repo.

…and the description of this repository!

Same for the skeleton repo.

... and stratigility ?

Thanks, @xtreamwayz!


.Net Template Internal Feed Changes

Related Issue
Fixes #3985
Description

Added internal nuget package feed support.
Added nuget.config for AEL MyGet source.
Updated Versioning mechanics for CI build

How Verified

ran the unit tests for regression
tested nuget feed with wpf sample app

Microsoft Reviewers: Open in CodeFlow

awesome -- glad you figured it out :)

hey @nesalang, should this be  merged into @dclaux 's refactor branch ? If so then let's just do that ?

@shalinijoshi19, no we can't do that. if this is merged to the refactor branch, we can't build for .Net daily feed.

In reply to: 624837281 [](ancestors = 624837281)

@jwoo-msft should this be updated to work with RC bits of AEL now instead of nightlies for stability?

@shalinijoshi19, no we can't do that. if this is merged to the refactor branch, we can't build for .Net daily feed.
In reply to: 624837281 [](ancestors = 624837281)

Could you help me understand why and also how are we tracking closure for this? Will this be merged directly into master? My worry is there are too many different topic branches for this same feature and we might miss something?

This is branch is going to be merged to man .NET branch. The main changes for this branch is at commit #5. I think the other changes shown here is due to rebasing to .NET main branch. I run the diff between two branches, and the diff was the same as in commit #5.

In reply to: 625773776 [](ancestors = 625773776)

yes, that's updated in .NET main branch.
thank you.

In reply to: 625773448 [](ancestors = 625773448)


Fix small typo in error messages

Noticed two small typos while running Overmind, figured I'd help!

Thanks!


Changelog 3.26: minor edits/additions related to point clouds

add entry about classified renderer improvements for point clouds
move an entry about following 2d renderer by 3d renderer for point clouds to the "Point Clouds" category where it better fits

Thanks Martin!


Update 'on' functions typings

Changed the typings of the 'on' functions so there is no type error when someone passes more than 5 dependencies.
I also changed two things:

prev now returns T | undefined on the single dependency version and [...T] | [] on the multiple dependencies version.
prevResults can be undefined

I think it reflects better what the js code does, but can be a breaking change for ts users.

Looks good to me. I will trust you on this. Thanks.

Looks good to me. I will trust you on this. Thanks.


Migration from vertex_ai.preview to vertex_ai.generative_model

Summary and motivation
Current integration of VertexAIGemini uses vertex_ai.preview package which has some experimental features. We should migrate to a more stable vertetx_ai.generative_model package.
Checklist
### Tasks
- [ ] The changes are merged in the `main` branch (Code + Docstrings)
- [ ] New package version declares the breaking change
- [ ] The package has been released on PyPI
- [ ] Docs at https://docs.haystack.deepset.ai/ were updated
- [ ] Integration tile on https://github.com/deepset-ai/haystack-integrations was updated
- [ ] Notebooks on https://github.com/deepset-ai/haystack-cookbook were updated (if needed)
- [ ] Tutorials on https://github.com/deepset-ai/haystack-tutorials were updated (if needed)
- [ ] Articles on https://github.com/deepset-ai/haystack-home/tree/main/content were updated (if needed)

Look out for breaks of the serialization


W 13702334 review develop an integration tutorial cp duke

simplifying develop tutorial based off of

Simone and I went over changes on 8/8 and decided to merge and address later if additional changes are needed.


server(ticdc): server panic on print log

What problem does this PR solve?

Issue Number: close #7120
What is changed and how it works?
always new the capture info pointer.
Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/merge

/cherry-pick release-6.3


Logo design contribution

Hallo
I am Md Masud Rana
Great to know about this projects but this projects have no logo.I am a graphics designer wants to make a logo for this projects.Its totally free for open source projects.
Thank you

Hi Md Masud Rana,
Thank you. I have updated the wiki to introduce the current implemented features and show some screenshots. The main functions of the 1st stage are roughly finished. I plan to submit this project to apple app store and Google play store. I hope everything will be OK. Before that, a logo and possible better UI should benefit this project a lot.
Actually, I'm also looking for some graphic designers' help locally and not sure if I can get one or not. I very welcome any contribution and very happy someone wanna to help. If you do not mind some possible situation (e.g. not match my visual preference....), making a logo is OK and we can see the results and discuss if adapting it or not.
Again, I appreciate your enthusiasm very much.

I'm going to close this since I have applied some logos from a free website


Persist source descriptions to database

Describe the feature
Currently description fields are persisted to the appropriate relation (table/column) in the target database. Would it be a good feature to extend that to sources?
The current use case I have is that data is streamed from Kafka in to BigQuery. We then create a source in DBT pointing at that streamed table and have DBT staging models that refine the data and marts models to consume the staging models.
It would be nice if the description added to the source YAML file could then both feed the DBT documentation and be persisted to BigQuery
Describe alternatives you've considered
An option I have considered would be to create a script that parses the manifest.json to extract resource_type = source and cal the BigQuery API to update the description
Additional context
This particular request is for BigQuery, but I imagine would be applicable to other target databases that allow documentation to be added
Who will this benefit?
This feature will be used from a data discovery point of view. It will allow our analytics team to understand information about the table without inferring from the table name/content

hey @whittid4 - this is a neat idea, and I totally buy the reasoning, but I don't think this is something we're going to want dbt to do. dbt doesn't really touch source data, and I have concerns about baking something like this directly into dbt. For one: when would this code run? I don't think it should happen in a dbt run, and I don't know that there is really an appropriate subcommand for something like this today.
Instead, what do you think about making a macro that sets these descriptions? This macro could inspects the sources defined in a project, then execute some queries to add table and column-level comments to source tables.
I can picture that working with:
dbt run-operation dbt.persist_source_descriptions

The persist_source_descriptions macro might look something like:
-- pseudo-code
{% macro persist_source_descriptions() %}

for source in graph.sources:
  persist_relation_description(source)
  persist_column_descriptions(source)

{% endmacro %}

We could either ship with dbt, or we could include it in a package like dbt-utils.
@jtcohen6 curious to hear what you think too
Thanks for opening this issue @whittid4!

@drewbanin thanks for the response and your comment make sense.
I didn't realise how powerful the dbt graph context was. I will take a look and see if I can get a macro going. I get something working maybe I could contribute it to dbt-utils

Totally agree with the reasoning here. I see this as a run-operation, rather than an embedded action within dbt run. I think it's powerful, compelling, and we want people to really know what they're doing when they're running any statements against source data tables.


Pin rustfmt's version so it doesn't break the build

Hopefully. :)
Ran a cargo clean and ran crategen to verify that output did not change.
Blocking #710

This made it past the crategen point in CI where rustfmt actually gets used, and passed on all of stable, beta and nightly. Merging to unblock.


PAPP-23589 - Add md5 support for non-fips phantom instances

…instances
Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[X] Testing of all the changes has been performed (for bug fixes / features)
[X] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[X] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[X] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[X] Verify all checks are passing.
[X] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[X] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[ ] New App
[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Currently we generate SDIs in SHA256 format when creating artifacts on ingestion.

What is the new behavior? (OPTIONAL)

For non-fips Phantom instances, we will generate SDIs in MD5 format so we do not create duplicate artifacts if the artifact already existed prior.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Integration Test: Pending an IMAP asset configured for FIPS so test connectivity passes.
Static Test: parse 1.17 is a BSD license which is approved.
https://pypi.org/project/parse/1.17.0/

looks like the parse package doesn't correctly report its license on pypi. can you add it to https://cd.splunkdev.com/phantom/app_release/-/blob/master/data/approved_modules.json since it uses BSD which is approved?

looks like the parse package doesn't correctly report its license on pypi. can you add it to https://cd.splunkdev.com/phantom/app_release/-/blob/master/data/approved_modules.json since it uses BSD which is approved?

https://cd.splunkdev.com/phantom/app_release/-/merge_requests/1719


[Uglifyjs Error] The dist file contains ES6 error in version 7.6.0 / 7.5.0

Describe the bug
When using uglifyjs to minify the size of the bundle, it failed becaulse the dist of react-dnd includes ES6 syntax.
I find the problem occurs both in version 7.5.0 and 7.6.0. However, in 7.4.0, there is no problem.
I checked that the compilation option "target: es5" does exist in tsconfig.json so I have no idea why it seems not work in 7.5.0 and 7.6.0 ...
To Reproduce
Steps to reproduce the behavior:

npm run build
Check the dist/ReactDnD.js and search for ES6 syntax, such as let / const /  () =>

Expected behavior
No ES6 syntax in the dist file
Screenshots
In ReactDnD.js:

Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 7.6.0, and also in 7.5.0

Additional context
No

in Webpack configuration I have to add
	 resolve: { 		extensions: [".tsx", ".ts", ".js", ".json"], 		alias: {  	 	 	"dnd-core": require.resolve("dnd-core/lib/cjs") 		} 	}
and then it can resolve to ES5 version
But this is an annoying fix.

in Webpack configuration I have to add
	resolve: { extensions: [".tsx", ".ts", ".js", ".json"], alias: { "dnd-core": require.resolve("dnd-core/lib/cjs") } }
and then it can resolve to ES5 version
But this is an annoying fix.

thanks! it's kind of annoying but saves the day!
hope there will be more elegant workaround

I'm facing the same problem.

Oh gross - if you can cut a PR of the webpack conf in react-dnd that would be great. If not I'll try to make some time next week.
In the future I'd like to do away with the UMD builds and just publish esm output. I'd love to hear community opinions on that.

Facing same issue

Oh gross - if you can cut a PR of the webpack conf in react-dnd that would be great. If not I'll try to make some time next week.
In the future I'd like to do away with the UMD builds and just publish esm output. I'd love to hear community opinions on that.

the issue we are having is webpack try to bundle esm module instead of cjs since the moment you added "module" to package.json. And our site doesn't run on IE anymore. Using a transpiler like babel is not feasible as our code is typescript based and adding it adds huge overhead in compile time :(

Hi @darthtrevino the PR only fixes problem in your example but not problem from other project using react-dnd. Doesn't seems to be a real fix to the real issue don't you agree?

How so? We corrected the UMD builds of react-dnd and react-dnd-html5-backend

no the problem is not with UMD build. it's with "module" added to package.json and therefore webpack is bundling ESM version, which doesn't run on IE. This was added in https://github.com/react-dnd/react-dnd/commit/ead0bca30dc8dfebf65e3bb9c1ed02d2352beb2f#diff-d5468382cf3875957a18e720907ffe82 which to me is a breaking change. It should be done in 8.0.0 instead.

I see, that's a good point - that probably should have gone in the major version cut. At this point I think it would probably do more hard than good to revert though.

@darthtrevino  Can you be more specific what harm it could do ;)
We are building library with DnD capability, and with many customer using it. This changes cause all project on customer side failing to run on IE. I would really appreciate if you are ok with reverting it :)
I can submit a PR if you agree.

Before we cut a 7.x branch for legacy maintenance, can you try out
react-dnd-cjs?
On Mon, Jun 17, 2019, 9:59 PM BinhPV<EMAIL_ADDRESS>wrote:

@darthtrevino https://github.com/darthtrevino Can you be more specific
what harm it could do ;)
We are building library with DnD capability, and with many customer using
it. This changes cause all project on customer side failing to run on IE. I
would really appreciate if you are ok with reverting it :)
I can submit a PR if you agree.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-dnd/react-dnd/issues/1387?email_source=notifications&email_token=AAA3XCBR3B5DJMND5UEH4VLP3BTRXA5CNFSM4HXWVCXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX5F3GY#issuecomment-502947227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA3XCDUEZHEDEJAJBFJR4TP3BTRXANCNFSM4HXWVCXA
.

@darthtrevino this require change in our customer code base, to import from new package so it's doesn't really solve the problem.

I see - I suppose we can cut a patch 7.x version for this. It would be in a
new v7 branch though
On Tue, Jun 18, 2019, 2:48 AM BinhPV<EMAIL_ADDRESS>wrote:

@darthtrevino https://github.com/darthtrevino this require change in
our customer code base, to import from new package so it's doesn't really
solve the problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-dnd/react-dnd/issues/1387?email_source=notifications&email_token=AAA3XCF66GJO5KHK24YB4Z3P3CVQJA5CNFSM4HXWVCXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX52XVY#issuecomment-503032791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA3XCGGTYAN4BRSU6UPQMTP3CVQJANCNFSM4HXWVCXA
.

@darthtrevino appreciate all the work you're doing on this. just wanted to chime in and say that it would be ideal if the latest version of the packages available for 7.x weren't broken for IE
this caused a huge headache for us and it's been especially frustrating because of the sub-dependency issue with dnd-core resolving to 7.6.0 even when the parent packages are locked at 7.4.5 (see https://github.com/react-dnd/react-dnd/issues/1418)

I'll try to make some time tonight to cut a new 7.x patch version that reverts module support.

https://github.com/react-dnd/react-dnd/releases/tag/v7.7.0

Same problem exists with v8.0.3


how to make cabot running with latest python requests version ?

hy,
we are working several cabot plugins (hopefully some of them will be available publicly on github).
for one of our module, we uses ovh python client : https://github.com/ovh/python-ovh
this module needs requests 2.7.0 , but cabot uses requests 0.14.2.
we have tried latest requests version but failed because of some changes in resp.json handling.
how can it be possible to make cabot perfectly running with latest requests version ?
Thanks for your help,
Regards

All that needs to be done is:

Update requests version in setup.py
Grep through the code to replace all instances of resp.json with resp.json()

We're not using anything fancy in requests so I think that should be OK. Comments welcome

It seems to be working, needs a little more test to be sure everything is ok :)
i'll keep you in touch with a nice (or so i hope) PR, when i'm sure everything works right

I actually fixed this in f587076 - if there are any other changes please send a PR...


Dose't support asp.net mvc

I have a project with asp.net MVC, not core and i have this error
Response Content-Type not supported: [application/json; charset=UTF-8]

A Flutter SignalR Client for ASP.NET Core, so it's not compatible with asp.net

when can be compatible 😅or what can I do

either write your own plugin, or port your project from asp.net to asp.net core

oh thanks

@AymanMubark , I had the same problem - my web api solution was standard .net framework... but I needed SignalR, so I ported/converted my solution to .net core. Yes, there are a few things that were broken, but if you go thru the issues one by one, it was not that bad.


Support for Bare metal/single task/thread

Multi task environment for simple applications like beacon/eddystone is kind of overkill and increases ROM/RAM requirements.
Also porting the lib to some new OS becomes challenging when time is scarce.
I would love to have an option to run the entire host stack in single task/thread (similar to LWIP in NO_SYS mode).
Here is the pseudo code that I am after.
int main(){
  AppInit();                                  // initialize system
  NimbleHostInit(conf, transport, callbacks); // initialize nimble host

  while (true) {
    AppTask();    // handle all system operations here
    NimbleTask(); // handle all nimble host operations here like read from/write to transport
  }
  return 0;
}

Currently we don't have any plans for bare metal support in NimBLE host, but I don't see a reason to block this if anyone is willing to work on that.
BTW  upcoming Nimble 1.5 release have much improved ram/rom usage when unused BLE roles are disabled.
Eg this is for advertiser sample app (beacon):
FLASH   RAM
9990    1518 @apache-mynewt-nimble_nimble_controller.a
3372     710 @apache-mynewt-nimble_nimble_drivers_nrf51.a
8522    1153 @apache-mynewt-nimble_nimble_host.a
80      32 @apache-mynewt-nimble_nimble_host_services_gap.a
599    5596 @apache-mynewt-nimble_nimble_transport.a
And you should be able to tune it further down as this is using stack's defaults for transport buffers...


Wrong Warp or something like that in sample game

Describe the bug
It is possible to warp to random scenes in the sample game.
To Reproduce
Steps to reproduce the behavior:

Start the game and enter a state where you can use the quest menu
Push on right or left and don't stop pushing
Spam a and start until you see the menu vanishing and reapearing without seeing eaven just one frame with the player and the scene where you should be
Exit the menu and see that you are in another random scene in the upper right corner.

Expected behavior
The menu should just close and open and not warp you arround.
Platform (please complete the following information):

OS: Windows 10
App version: 1.1.0

I just tested it(you don't actually have to press a direction button), It's indeed weird. Looks like it could have something to do with the Warping of backgrounds in combination with the previous location and scene being stored with Scene: Restore Previous From Stack
It's one way of getting to the space dog 😝

Weird! Looks like all you need to do to reproduce this is press Start while the Menu scene is fading out which causes it to return back to the Menu scene but when you leave your X/Y position has been reset and the scene index is one higher than it was previously.
Reminds me of the Screen Warp Glitch in Link's Awakening! https://zelda.gamepedia.com/Glitches_in_Link's_Awakening#Screen_Warp_Glitch

Should be fixed now with fd109b203d83407bdb3318cc426185321859abff


focusOnInvalid invalid

Is there an existing issue for this?

[X] I have searched the existing issues

Package/Plugin version
8.0.0
Platforms

[X] Android
[X] iOS
[ ] Linux
[ ] MacOS
[ ] Web
[ ] Windows

Flutter doctor
[✓] Flutter (Channel stable, 3.7.9, on macOS 13.5 22G74 darwin-arm64, locale zh-Hans-CN)
• Flutter version 3.7.9 on channel stable at /Users/star/flutter/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 62bd79521d (8 months ago), 2023-03-30 10:59:36 -0700
• Engine revision ec975089ac
• Dart version 2.19.6
• DevTools version 2.20.1
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at /Users/star/Library/Android/sdk
• Platform android-33, build-tools 33.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.12.0
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.1)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 74.0.4
• Dart plugin version 231.8770.15
[✓] VS Code (version 1.84.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.76.0
[✓] Connected device (3 available)
• iPhone 14 (mobile) • D6A01FD8-C25C-4676-B720-A1DEEBD16167 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
• macOS (desktop)    • macos                                • darwin-arm64   • macOS 13.5 22G74 darwin-arm64
• Chrome (web)       • chrome                               • web-javascript • Google Chrome 119.0.6045.159
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
Minimal code example
_formKey.currentState!.validate(focusOnInvalid: false)

Current Behavior
~
Expected Behavior
~
Steps To Reproduce
In FormBuilderState,  method validate will call FormBuilderState()._formKey.currentState.validate(). but no argument focusOnInvalid
currentState is FormState.
And in  FormState, validate will call FormFieldState.validate(), and no argument focusOnInvalid.
Now, FormBuilderFieldState extends FormFieldState, and override validate, and added agument focusOnInvalid.
so, FormBuilderState does not pass argument focusOnInvalid into FormBuilderFieldState.
Aditional information
No response

We have a solution but so ugly.
FormBuilder(
  key: _formKey,
  onChanged: () {
  buttonEnabled.value =
    _formKey.currentState!.fields.values.fold(false, (previousValue, element) => previousValue && element.validate(focusOnInvalid: false));
  },
  ...
)


fix typo for AndroidInstallation

I found typo in AndroidInstallation.md. This PR is fix for that.

thanks @KensakuKOMATSU


Fix usage of active_submission_index.fetch_add

From https://doc.rust-lang.org/std/sync/atomic/struct.AtomicU64.html#method.fetch_add:

Adds to the current value, returning the previous value.

Thus, receiving the new active submission index after incrementing it requires adding 1 to the return value locally as well.
For comparison, the other uses of fetch_add in wgpu (unrelated to the current changes) treat the atomic as a "next id" variable instead of an "active id", thus not requiring to increment the return value of fetch_add:
https://github.com/gfx-rs/wgpu/blob/dd64343f942ff519f980a2adc02fa60f3a7b72a3/wgpu/src/backend/web.rs#L29
https://github.com/gfx-rs/wgpu/blob/dd64343f942ff519f980a2adc02fa60f3a7b72a3/wgpu-hal/src/gles/device.rs#L1094

Very good catch!


feat(pipeline): enable usage of template languages within base

Changes included in PR:

Enable the user to consume template languages (golang & starlark) within the .vela.yml without templates
Replace Render function with RenderStep and RenderBuild
Update folder structure of testdata
Update .golangci configuration to remove deprecated linters

Co-authored-by: @kneal

Codecov appears to be failing due to the ongoing checksum issue discussed within https://github.com/codecov/codecov-action/issues/437.


Implement the current resin count

There seems to be a project using the Chinese API which has access to the current resin yaomeng0722/genshin_task-resin-expedition_alert
It uses an endpoint that doesn't seem to have an overseas equivalent
https://api-takumi.mihoyo.com/game_record/app/genshin/api/dailyNote
Figuring out if there's an overseas version and implementing it could be very beneficial.

Oh my, when I created this issue it hasn't existed yet. Mihoyo is working hard for once. I'll try to look into it now.

sure did, no such call showed up.

I've been testing the api, but abyss and activities endpoints return the same invalid request error...


minor: fix some typo.

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

fix some typo.
Are there any user-facing changes?

None

Codecov Report

Merging #3453 (79773ec) into master (c5c1dae) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3453   +/-   ##
=======================================
  Coverage   85.68%   85.68%
=======================================
  Files         298      298
  Lines       54645    54645
=======================================
+ Hits        46820    46822    +2
+ Misses       7825     7823    -2

Impacted Files
Coverage Δ

...ore/src/physical_optimizer/aggregate_statistics.rs
100.00% <ø> (ø)

...afusion/core/src/physical_optimizer/repartition.rs
100.00% <ø> (ø)

datafusion/core/src/physical_plan/stream.rs
66.66% <ø> (ø)

datafusion/core/src/test_util.rs
98.18% <ø> (ø)

datafusion/expr/src/expr_visitor.rs
56.84% <ø> (ø)

datafusion/core/src/dataframe.rs
89.21% <100.00%> (ø)

datafusion/core/src/physical_plan/metrics/value.rs
86.93% <0.00%> (-0.51%)
:arrow_down:

datafusion/common/src/scalar.rs
85.12% <0.00%> (+0.06%)
:arrow_up:

datafusion/expr/src/logical_plan/plan.rs
77.35% <0.00%> (+0.16%)
:arrow_up:

datafusion/expr/src/window_frame.rs
93.27% <0.00%> (+0.84%)
:arrow_up:

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


Label Contours

Is there any way to get the contours of the labels?

Never mind, I got it to work with the polygons.

Sorry for the delay, but glad to hear :)


Hl/incremental review update

Type
enhancement, bug_fix

Description

Added support for incremental PR reviews in markdown conversion, including conditional headers and notes for commits since the last review.
Fixed the initialization of file_set for handling incremental commits in GitHub provider.
Enhanced PR reviewer tool to skip incremental reviews when there are no new files since the last review, including publishing a comment about the skip.

Changes walkthrough
Relevant filesEnhancement

utils.pySupport for Incremental PR Reviews in Markdown Conversion

pr_agent/algo/utils.py
Added incremental_review parameter to convert_to_markdown function.  Conditional markdown headers for incremental PR reviews.  Include a note for commits since the previous PR-Agent review in incremental reviews.

+6/-2

pr_reviewer.pyEnhancements and Fixes for Incremental PR Reviews

pr_agent/tools/pr_reviewer.py
Skip incremental review if no new files since last review.  Publish a comment when incremental review is skipped.  Pass incremental_review_markdown_text to convert_to_markdown.

+15/-10

Bug_fix

github_provider.pyInitialize File Set for Incremental Commits

pr_agent/git_providers/github_provider.py
Initialized file_set dictionary for incremental commits.  Removed redundant initialization of file_set.

+2/-1

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/desribe

/describe


Make the color scheme available for vim

Description 💡

I don't use vim (yet), but I'd like to make sevastolink available for vim;
Can someone help on this?

Any progress?

Any progress?

@brokenshardsofdeath not even started hahahaha would you like to work on it? ✨

@brokenshardsofdeath not even started hahahaha would you like to work on it? ✨

@paulopacitti if I knew anything about coding I would've created the windows version of the Sevastolink theme and then the whole OS.

@paulopacitti if I knew anything about coding I would've created the windows version of the Sevastolink theme and then the whole OS.


fix(List links bullet point styling)

This PR fixes a styling bug where link list items with a newline would display the bullet point at the bottom, instead of at the top, of the list item.

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1208311653825044

@aaschlote afaict removing this doesn't affect other StandaloneLinks in the app, but lmk if you see a problem with it

@aaschlote afaict removing this doesn't affect other StandaloneLinks in the app, but lmk if you see a problem with it

I added this inline-block because of this ticket. Can you check if it doesn't break anything mentioned in the ticket?


[iOS] About Feature: Update layout with Figma

Overview
About feature's layout is not current design. you would update it.

Goal

🙋♂🙋♂

@matsuurayuki1219
Thank you for your interest.
Sorry, the current "About" screen layout seems to be Goal's, so please check other issues 🙇🏼


How to show a bitmap

I have to show 3D and 2D objects. The example works for 3D..How can I draw a bitmap at a certain x,y,z ?

https://github.com/viromedia/virocore/issues/115
Moved this issue over to ViroCore github as this is for ViroReact. Answer is provide there. Closing this issue.


how to input  numerial value to mxnet?

i want do some regression task under mxnet, but my data is  libsvm format,  like this (1:10,2:33,5:8.5,9:22)... i read example on mxnet, but seems mxnet  is prepare for image, would anybody tell me how to input these data to mxnet model??

is mxnet.io.CSVIter or mxnet.io.NDarray or mxnet.io.DataIter?

Yes, you could use mxnet.io.NDArrayIter. First load your data to a numpy array (or mxnet ndarray). Then feed it to a mxnet.io.NDArrayIter. You could then use it with your model in mxnet.

thanks ~


Allow user to set leaf position for PathVar

Description

Currently, for PathVar, the location information is encoded by path: Vec<Boolean> and leaf_is_right_child: Boolean (chaining those two get the big endian encoding of index of leaf).
There are indeed many circumstances where we also have constraints on leaf position (take IOP verifier's query as an example). For now, path and leaf_is_right_child are just variables created from Boolean::new_variable, and user do not have access to such those variables. In this commit, user will be able to:

get the leaf index variable in PathVar and constrain them
replace the leaf index variable in PathVar with their own variable (possibly already with some constraints)

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

[x] Targeted PR against correct branch (main)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[x] Updated relevant documentation in the code
[x] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

Leaf position is also used in the Sapling circuit.

@weikengchen Does code look good? If so we can merge it

The changes look good, modulo the two comments. Sorry for the delay in reviewing!


SourceMap.lookup_line panics when position is not found

Describe the bug
I'm using SWC as a library in a dead code analysis tool. I noticed  that swc_common::source_map::SourceMap.lookup_line panics when the given byte position cannot be found from the source map, for example when using SourceMap::default() as a placeholder. I assume this is unintentional, as the function returns a Result, and there is some error handling code in lookup_line_with. The issue seems to be in lookup_source_file, which explicitly panics when the source file is not found.
Input code
fn create_span_source(&self, span: Span) -> ModuleSourceAndLine {
    let line = self
        .source_map
        .0
        // This panics at the moment
        .lookup_line(span.lo())
        .map(|source_and_line| source_and_line.line)
        .unwrap_or(0);

    ModuleSourceAndLine::new(self.root_relative_path.clone(), line)
}

Config
No response
Playground link
No response
Expected behavior
I would expect lookup_source_file to do the necessary error handling so that lookup_line never throws under any regular circumstances.
Version
swc_common = "0.14.6"
Additional context
No response

using SourceMap::default()

This is the problematic part.
All code of swc assumes that SourceMap has correct data.

That definitely makes sense as an implementation detail, but the API is still a bit misleading. The documentation for the function says:
/// If the relevant source_file is empty, we don't return a line number.

which I guess is technically true, but in my mind not returning a line number does not mean that the library panics. Perhaps we should just update the documentation to cover the case when the function panics?

Yeah, I think it's good change.
I'll close this while adding docs about design of SourceMap

Ran into this problem when mapping a Span to line/col in a plugin. One of the nodes had a position of 0 which triggers this panic. Would it be better to have the interface return Option instead of panic? For backwards compatibility purposes, a new try_lookup_char_pos could be added that is safer? Either way, this interface is problematic and scary.
pub fn lookup_source_file_in(
        files: &[Lrc<SourceFile>],
        pos: BytePos,
    ) -> Option<Lrc<SourceFile>> {
        if pos.is_dummy() {
            return None;
        }

Related: https://github.com/swc-project/swc/issues/5535 (Not the same issue, but related with error handling)
I agree hard panic is not something desirable as there's no way consumers to workaround / ignore failures.


Update DeveloperGuide.md

Codecov Report
Base: 61.87% // Head: 61.87% // No change to project coverage :thumbsup:

Coverage data is based on head (d020b56) compared to base (7272575).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #221   +/-   ##
=========================================
  Coverage     61.87%   61.87%
  Complexity      637      637
=========================================
  Files           112      112
  Lines          2371     2371
  Branches        314      314
=========================================
  Hits           1467     1467
  Misses          775      775
  Partials        129      129

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Any plans for react native web or expo web?

Would like to try this if it works for web too.

You can easily create a .native.js version based on this lib and a .web.js for web application.
What is relevant for me is the ability to import this into expo without ejecting.

You can easily create a .native.js version based on this lib and a .web.js for web application.
What is relevant for me is the ability to import this into expo without ejecting.

I am in the same boat here, I'd love to use this lib in a managed expo project.


Rack rename do not work

Scenario/Intent
When trying to rename a rack, ansible task remain unchanged and nothing is done
Environment Details
"git clone" then "docker container build" on November 19th

Module Version: v5.9.0
Ansible Version: 2.7.7
OneView Appliance Version: DCS CP 5.50
OneView Client API Version: 800
Python Version: 3.6.12
python-hpOneView SDK Version: 5.5.0
Platform: Debian 10.6

Steps to Reproduce
rename.yml
- hosts: all
  vars:
    config: "{{ playbook_dir }}/oneview_config.json"
    rack_name: "LL0-N-08X"

  tasks:
    - name: Rename the default rack
      oneview_rack:
        config: "{{ config }}"
        state: present
        data:
          name: '2AB100LMNB'
          newName: "{{ rack_name }}"

    - debug: var=rack['name']

# ansible-playbook rename.yml
Expected Result
Rename of the rack to newName when it exists with (old) name
TASK [Rename the default rack]
changed: [localhost]

TASK [debug]
ok: [localhost] => {
    "rack['name']": "LL0-N-08"
}

Actual Result
Nothing done
TASK [Rename the default rack]
ok: [localhost]

TASK [debug]
ok: [localhost] => {
    "rack['name']": "2AB100LMNB"
}

My comments
library/module_utils/oneview.py:dict_merge(dict1, dict2) do not work as expected
It makes a deep merge of 2 dictionaries and returns the merged dictionary.
Values in dict2 will overwrite same key values of dict1.

merged_data = dict_merge(self.data, self.current_resource)
Common keys in current_resource will overwrite what we want to change in data.
We have to permute.

2.dict_merge() modifies dict1 passed to it.
After permutation because of point 1 above, original current_resource is now modified.
When _update() compare current and merged data, it doesn't detect any change.
We have to work on a copy of dict1 before modify it to keep original unchanged.

rename task is not idempotent.
When no rack with (old) name exists but there is a rack with newName, it will create a new rack with (old) name.
We have to check first if rack with newName exists.

Thanks for the  sharing the fix. We will review it and the update the pull request.


3.0.0

Coverage remained the same at 98.199% when pulling c7b90094a2c9aca6fd158c207962fe71ab786c61 on release_v3.0.0 into 088494a526b2b2d971e14c05d545ba95f752f218 on master.


Fix #1132: Add debug target.

What is the purpose of this change? What does it change?
Adds a debug target to Makefile.
Was the change discussed in an issue?
fixes #1132
How to test changes?
make debug then try to attach a debugger to the running odo instance. Without this fix, you'd get an error similar to: could not attach to pid 90236: decoding dwarf section info at offset 0x0: too short

:white_check_mark: odo build 792 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/fe05864591 by @metacosm)

:white_check_mark: odo build 802 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/865d18c23a by @metacosm)

Tests pass plus the change shouldn't even impact anything.

Merging this based on 3 :heavy_check_mark:  and @kadel's concern is being addressed :smile:


Docs - revise bg colour & wagtail on hover styles

background teal colour  to match latest Wagtail admin styles

small on hover animation (similar to the wagtail admin one but more basic)

I did a testdrive and this is a nice change!


Godot fails to load on macOS / "Unable to initialize Video driver"

Godot version: b7dc08fcf6117bc13b0b1210119e51b4c51f5086 (~master)

OS/device including version:

MacBook Pro (13-inch, 2019, Two Thunderbolt 3 ports)
macOS 10.15.5 (19F101)
MoltenVK from brew install molten-vk
Intel Iris Plus Graphics 645 1536 MB graphics
Metal: Supported, feature set macOS GPUFamily2 v1

Issue description:

Compiled Godot and ran with the following:
$ scons platform=osx arch=x86_64 --jobs=(sysctl -n hw.logicalcpu)
$ cp -r misc/dist/osx_tools.app ./Godot.app
$ mkdir -p Godot.app/Contents/MacOS
$ cp bin/godot.osx.tools.x86_64 Godot.app/Contents/MacOS/Godot
$ chmod +x Godot.app/Contents/MacOS/Godot
$ open ./Godot.app

At that point, I was given the following error dialog:
Unable to initialize Video driver
Your video card driver does not support any of the supported Metal versions.

Steps to reproduce:
(See above)
Minimal reproduction project:

(N/A)
Relevant PRs:

https://github.com/godotengine/godot/pull/40354 (Adds the above error message)

I think you missed a step to bundle the Vulkan loader, see the end of this section: https://docs.godotengine.org/en/latest/development/compiling/compiling_for_osx.html#compiling

MoltenVK from brew install molten-vk

homebrew moten-vk package seems to have wrong path (./libMoltenVK.dylib) in it's ICD config (/usr/local/share/vulkan/icd.d/MoltenVK_icd.json), it should be either absolute path or relative to the .json location.
Or just copy .dylib to the Godot.app/Contents/Frameworks/.

@akien-mga
cp /usr/local/lib/libMoltenVK.dylib Godot.app/Contents/Frameworks/libMoltenVK.dylib

And it works! Thank you, I did miss that step.

@bruvzg I think you're right, modifying MoltenVK_icd.json to be the following:
{
    "file_format_version" : "1.0.0",
    "ICD": {
       // or "../../../lib/libMoltenVK.dylib",
        "library_path": "/usr/local/lib/libMoltenVK.dylib",
        "api_version" : "1.0.0"
    }
}

also fixes the fixes the issue and allows Godot to initialize.

It would be good to report the issue to homebrew maintainers so that they fix the package.

I am trying to run master branch. Following the instructions on the official documentation (https://docs.godotengine.org/en/latest/development/compiling/compiling_for_osx.html#compiling) I was able to get the editor running.
I would like to set it up so I can easily run/debug the executable from the godot/bin directory but how do I do this?
It seems when running from a .app somehow the executable knows to load Resources/vulkan/icd.d/MoltenVK_icd.json which then points it to the Frameworks/libMoltenVK.dylib.


Unable to move, not paralyzed character

Hi,
as reported two weeks ago to Cattletech it is possible to have a character that is unable to act. It isn't the first time that my cleric gets stuck randomly.
Here's a save game related to it. Sorry but no log :( .
http://www.filedropper.com/save816unabletomovepriest
Test in r816. Just load the save and notice that the priest (3rd char, [2], can't move, nor cast spells).
p.s. posting so the github issue exists, as well as say that it wasn't solved pre-816

I just re-reproed on 831, after a fight (where I used prayer, then magic circle against evil), then heal on my barbarian and cure light wounds (lv5 spell; not sure on the order for those 2 spells) on 4 players after the fight, then I can't recall being able to move nor cast...
TemplePlus.log_Stuck.txt
I re-opened the save a couple of times, so I doubt the very end of the .txt is useful. I also noticed late (I believe) the stuck-scenario.


App split

Separation of features that do not fit on Nano S.

Will be part of Conway PR, e.g. https://github.com/LedgerHQ/app-cardano/pull/67


Is this doc still relevant?

https://github.com/zalando/api-first-hand/blob/master/docs/REFERENCES.md
Asking because the main README has this:
If not, we can delete it.

The REFERENCES is in-depth explanation how we work with json references in the plugin's source code. This is supposed to be useful for plugin developers as opposed to the plugin uses, for which is the section in the README

Thanks, @slavaschmidt! This sentence is a little confusing: "A $ref element of the specification is allowed to contain a name of file as its part." As its part of what, exactly? And "a name of file" means ... can we be a bit more specific?

Also this sentence: "It is also possible to reference a definition in one specification from another specification." [rewrite as?] => "You can reference a definition in one specification across other specifications."

👍

This sentence is a little confusing: "A $ref element of the specification is allowed to contain a name of file as its part." As its part of what, exactly? And "a name of file" means ... can we be a bit more specific?

It basically means, you can use filename to reference part of another specification.

Does this make sense to you, @slavaschmidt?

You can use a filename to reference part of another specification with the $ref element. You can split a single specification into multiple files (as shown in cross_spec_references.yaml). You can also reference a definition in one specification across other specifications.

Totally. For example one can create domain_model.yaml and then reference it from any other API specification

Awesome, thank you @slavaschmidt!


[CP] AGP 8.0 support for integration tests

issue_link
https://github.com/flutter/flutter/issues/125181
Commit Hash
3a6c4eb
Target
beta
pr_link
https://github.com/flutter/flutter/pull/125686
Impacted Users
Android devs using the recommended AGP version by stable Android Studio
Impact Description
https://github.com/flutter/flutter/issues/125181#issuecomment-1525894244
AGP 8.0 requires a ecosystem wide change. All dependencies must set namespace in build.gradle. This change adds namespace as a required parameter in a backwards compatible way.
Strong confidence existing tests will cover ensuring that AGP users less than 8.0 will continue to work. Cannot add a test for AGP 8 because it required java 17 to execute and there are no java 17 tests on stable branch. See https://github.com/flutter/flutter/issues/125325
Workaround
Users can either not update AGP or not have integration tests.
Risk
medium
Test Coverage
yes
Validation Steps
Attempt to build any integration test that uses AGP 8.0.

@stuartmorgan and @camsim99 could you take a look at this cherry-pick?  Need some more experts to take a look.

LGTM. We shipped the same change in all of our 1P plugins this morning.


in test_dependencies.py, the test case test_patch_request_body_parameter_producers_from_post are failed.

The patch has image_url in the body request. and post has a response of "image_url" and "url". The correct dependencies is image mapping to url. but now it is mapping from image_url to image_url. Need more investigating.


Add notes to Removing-IFluidRouter.md for aliased dataStores and sub-requesting

AB#6251

Can you please add a description that goes into the commit message as well. The work item there is internal.


Realm: DMA Engine Sparsity Gather/Scatter Optimization

Realm should automatically infer when it is profitable to do gathers for bulk transfers of data and then scatters at the destination for sparse data movement. This is often more efficient than moving the sparse data in pieces as hardware is optimized for bulk data movement. The Realm path planner will have to figure this out and allocate the needed intermediate buffers for performing this optimization.

@amaleewilson Is doing the initial investigations into how to determine when it is good to perform this optimization for various kinds of transfers.

@lightsighter it would be great if we could identify/create a small app or two that uses this pattern in a performance-sensitive way to provide the necessary apples for comparison purposes.  Got any suggestions?
Theoretically, the implementation of this transformation should be straightforward in the DMA system.  The open question is the heuristic for when to use it.

Adding the S3D tag. :) Ghost cell exchanges in the Y and Z dimensions should gather, bulk copy, and then scattter. Pennant should do the same thing but for unstructured, although it's unclear if there is as much of a performance win.

We have concrete evidence that Soleil-X encounters this now as well. Ghost cell exchanges become very slow once we have to partition in the (fastest changing) X dimension (Fortran order for Regent) and we get copies that look like this:
120ms to move 2MB of data
63ms to move 7KB
102ms to move 715KB
This is affecting weak scaling of Soleil-X application performance as the DMA channels become the bottleneck on execution once we have to partition on the X dimension.

Can we do a run for just 2-3 iterations with -level dma=2,inst=1 and put the logs somewhere I can look at them (they'll probably be too big to attach here).

@manopapad See above ^^^^^

4 node run (no ghost cell exchanges on x): http://sapling.stanford.edu/~manolis/742031/
8 node run (ghost cell exchanges on all dimensions): http://sapling.stanford.edu/~manolis/742032/

The specific change requested here (local packing/unpacking of sparse data) was added with commit 7d12a8ed.  Other issues are tracking other improvements to scatter/gather (unstructured, affine, or both).

Do we need a separate issue for tracking packing/unpacking being done on GPUs or do you feel like other current issues handle that sufficiently?

Issue #621 is the umbrella issue for using kernels rather than cuMemcpy* to do GPU data movement.

What's the priority of this issue among the list of outstanding work items for Realm? Is there any context/data which is missing in order to make a decision with respect to this issue?
I would be interested to pick this up if the priority is appropriate in our roadmap

Ah, didn't notice that this has been done


Allow Dependency injection in $table->query()

this allows the use of dependency injection in table's query method
public static function table(Table $table): Table
{
    return $table->columns([/*...*/])->query(fn ($query, $data, Component $livewire) => $query)
}

my use-case:
I have a ListRecords with a modal action to configure a frame_quote_price and vat
I can show the final price with a TextColumn
TextColumn::make('sell_price_with_frame')->prefix('€')->formatStateUsing(function (Product $record, Component $livewire) {
	return round(($record->sell_price_eur + $livewire->frame_quote_price) * (1 + ($livewire->vat / 100)), 2);
}),

but I also  want to be able to filter for max price considering frame_quote_price and vat
now it's not possible to access $livewire from the ->query method's callback

yes this is about the query method for filters


Update the term "mobile app" to "web app" in snapshot testing documentation

Change the mobile app to web app in documentation

Summary

Test plan

It can be used for react native as well, tho. Not sure this is better/more accurate>


Resolving Assertion Encryption issue.

Fixed Assertion Encryption.

No effective change in this PR.

Assertion Encrytption failed saying "org.apache.commons.collections.CollectionUtils" can't be found. Dynamic import wasn't there in the pom by that time. But later it was added. So, just refactored the code here.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_dc2e62e4-6058-4a3e-8796-5440a31e454f/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_dc2e62e4-6058-4a3e-8796-5440a31e454f.git

Docs Build status updates of commit 72a2027:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd1, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Pro icons (@hugeicons/react) don't tree-shake and increase bundle size by a lot

Hi!
I've recently purchased the Pro version of hugeicons (@hugeicons/react) and using it in a NextJS app. And I've noticed a huge increase in bundle size because of it.
In particular our minified+gzipped bundle went from 400kb to 600kb, basically a 50% increase just because of the icons we use.
So I went through the rabbit hole, and apparently it's because every icon included in the app comes with all variants, automatically.
Ex: I want to use ArrowRight01Icon, hugeicons includes all stroke, solid, twotone, bulk variants, even though they're never used.
Can you do anything about it? This is a deal breaker for us (and I'm sure a lot of other projects).

Hi.
We are working on this, and will fix it very soon.

Such a good news to hear! Any ETA?

@alea-git We have published<EMAIL_ADDRESS>that supports tree-shaking for icons and variants, you can check it and let us know if you face any issues.
You can now import icons that include all variant as:
Home01Icon
Or you can import single variant like:
Home01IconBulkRounded
Home01IconDuotoneRounded
Home01IconSolidRounded
Home01IconSolidSharp
Home01IconSolidStandard
Home01IconStrokeRounded
Home01IconStrokeSharp
Home01IconStrokeStandard
Home01IconTwotoneRounded


Wrong path to the endpoint

The path to Get All Stream Tags endpoint is wrong, - it should be tags/streams instead of streams/tags, please see the doc https://dev.twitch.tv/docs/api/reference#get-all-stream-tags

Fixed in 4.3.2, thanks!


Updated and improved the data

I rescanned modrinth and put the data i got here
This pull request closes issue #76

The merge should be dalayed, i discovered that Umimcj doesn't scan data properly now :/ i need to remake it


Prioritize gun mods

avatar and ncps do not change gameplay much. With unloaded gun mods you can seem to die for no reason which is very irritating especially with gamemodes like deathmatch.

What exactly is being requested here?

What exactly is being requested here?

make gun mods download over all other mods (have priority)

I will not be doing this, I cannot detect what type of spawnable a spawnable is when it's being downloaded, plus mods can contain more than just guns or avatars, they can contain multiple types of mods in one pallet


multi-gpu with joblib runs sequentially

Description
When using joblib (threading) for multi-gpu computation (see minimal example below), we are observing sequential evaluation for the first execution, while the subsequent evaluations are performed in paralllel, see nsight-system output below. We observe this behaviour in multiple hardware setups. Does cupy requires something else to run asynchronously?

To Reproduce
import joblib
import numpy as np
import cupy as cp

def apply_device(state, index, nqubits, device):
     with cp.cuda.Device(device):
         # transfer from host to device
         piece = cp.asarray(state[index])
     # transfer back from device to host
     state[index] = piece.get()
     # delete vector from device memory
     del(piece)

def main():
     # initialize state on host
     nqubits = 28
     state = np.zeros((2, 2 ** (nqubits - 1)), dtype='complex128')
     state[0, 0] = 1

     pool = joblib.Parallel(n_jobs=2, prefer="threads", verbose=1)
     for _ in range(5):
         pool(joblib.delayed(apply_device)(state, index, nqubits, device)
              for index, device in [(0, 0), (1, 1)])

if __name__ == "__main__":
     main()

Installation
Wheel (pip install cupy-***)
Environment
OS                           : Linux-5.16.2-zen1-1-zen-x86_64-with-glibc2.33
Python Version               : 3.10.2
CuPy Version                 : 10.0.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.21.5
SciPy Version                : 1.7.3
Cython Build Version         : 0.29.24
Cython Runtime Version       : 0.29.26
CUDA Root                    : /opt/cuda
nvcc PATH                    : /opt/cuda/bin/nvcc
CUDA Build Version           : 11050
CUDA Driver Version          : 11050
CUDA Runtime Version         : 11050
cuBLAS Version               : (available)
cuFFT Version                : 10600
cuRAND Version               : 10207
cuSOLVER Version             : (11, 3, 2)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 5)
Thrust Version               : 101301
CUB Build Version            : 101301
Jitify Build Version         : 60e9e72
cuDNN Build Version          : 8300
cuDNN Version                : 8300
NCCL Build Version           : 21104
NCCL Runtime Version         : 21104
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : NVIDIA RTX A6000
Device 0 Compute Capability  : 86
Device 0 PCI Bus ID          : 0000:68:00.0
Device 1 Name                : NVIDIA TITAN V
Device 1 Compute Capability  : 70
Device 1 PCI Bus ID          : 0000:1A:00.0

Additional Information
No response

I guess this is a context creation overhead? CUDA contexts are created (by CUDA, not CuPy) when using a device for the first time. (A context is needed for each device)

you can try the persistence daemon to improve a bit the response time, but in general, this is not easily avoidable and depends on the application. For example, you can try to run a very simple operation in both GPUs at the beginning to try to hide this, but the overhead will remain.
https://docs.nvidia.com/deploy/driver-persistence/index.html#persistence-daemon

@emcastillo thanks for the suggestion. I have introduced a simple preprocessing operation following your suggestion (see code below). Please find the respective nsight report and joblib logging below. I confirm that the initial overhead is reduced (shifted to the context_init call)  for the operation, however I am observing 2 issues:

despite the context pre-initialization (i.e. vertical lines around 0.8s), the first call to apply_device does not run in parallel, seems like there are blocking operations when moving to another kernel.
the time distance between the last memcopy in the first apply_device call (at ~4.1s) and the next call (at ~5s) is larger than the distance between subsequent calls.

import joblib
import numpy as np
import cupy as cp

def apply_device(state, index, nqubits, device):
    with cp.cuda.Device(device):
        # transfer from host to device
        piece = cp.asarray(state[index])
    # transfer back from device to host
    state[index] = piece.get()
    # delete vector from device memory
    del(piece)

def context_init(device):
    with cp.cuda.Device(device):
        c = cp.ones(10, dtype='complex128')

def main():
    nqubits = 28
    state = np.zeros((4, 2 ** (nqubits - 1)), dtype='complex128')
    state[0, 0] = 1

    pool = joblib.Parallel(n_jobs=2, prefer="threads", verbose=1)
    pool(joblib.delayed(context_init)(device) for device in [0, 1])

    for i in range(5):
        pool(joblib.delayed(apply_device)(state, index, nqubits, device)
             for index, device in [(0, 0), (1, 1)])

if __name__ == "__main__":
    main()

[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    2.2s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    2.2s finished
[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    4.0s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    4.0s finished
[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s finished
[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s finished
[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s finished
[Parallel(n_jobs=2)]: Using backend ThreadingBackend with 2 concurrent workers.
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s remaining:    0.0s
[Parallel(n_jobs=2)]: Done   2 out of   2 | elapsed:    1.4s finished


Should we always add the calling module to the CLI

With the current structure in #12 it is possible to use a CLI defined in for example roc-web-react but not use anything that it provides. There is no real point to this since we should aim for that all Roc packages have their own CLI that can be used. So the question is if we should always add the module that the user uses for the CLI or if this behaviour should be ok.
This also mean that a user could say that the no extensions should be used meaning that the CLI will be pointless. In this case should we add the module itself, give information about that there is no extensions active or do nothing.
We need to think about how we can solve this in the best way before releasing v1.0.

Closing issue. We will fix this (and related potential for confusion) with the merge of roc and roc-config. All packages will have a dependency on that module and hence use the same cli. As discussed around this issue in person, we must aim to avoid confusion around these things and so far this is the best way to go about it without losing flexibility.


Feature Request: Custom inputs for Commands

First off, thank you for creating this! Already see how helpful this will be.
I would to request a feature that would allow us to customize what the commands that we input are, for instance, I would like to be able to change the configurations like so:

/action —→ "/a "
/remove —→ "/x "
/bookmarks —→ "/b "
/history —→ "/h "
/tabs —→ "/t "

Where the space at the end would be the confirmation of the action / command meant to activate. Similar to something like Alfred & how you can customize those Workflow prompts.
Thanks for the consideration & thank you again for the awesome tool!

I might have been too quick to jump, as after testing it out I do believe the auto-complete feature that currently exists does a sufficient enough job of fulfilling this request.
While it's not exactly a custom key input for the Commands, it is more than enough for what the current starting letter options are.

It's true - autocomplete works, but it's not something that's easy to figure out on the first try. @alyssaxuu great product - thanks for the effort you've put in!


How do I recreate layer from geojson?

I have the geojson data saved from layer that user has drawn and I want to re-create it.
I was able to create the layer properly but the style is missing(ie. layer is there but is invisible).
What is the proper way to create the layer with the PM styling?
            let layer;
            let style = new L.PM.Draw().options.templineStyle; // not good
            switch (foo.shape) {
              case 'Circle': layer = L.circle(foo.geo.geometry.coordinates, { ...style, radius: area.radius}); break;
              case 'Polygon': layer = L.polygon(foo.geo.geometry.coordinates, style); break;
              default: L.geoJson(foo.geo, style); break;
            }
            layer.addTo(bar);

you should do it the other way around. Don't try to put leaflet.pm's style on other layers in your app.
Give leaflet.pm or the created layers in the pm:create callback your own style. And then use it accross your application.


Good work so far

Documentation required , as it describes the parts of codebase clearly

Thanks @codeWithHyder, Please would you like to say more details?

your work is very great 🥇
MY SUGGESTIONS , each part should have given documentation  eg;  addtask,deletion part, clearAll .. , so that if any new trainee wants to go through your code may not feel hectic to understand your codebase

Hello @codeWithHyder, I will implement your suggestions on my project.
Thanks for giving me such a best advice.


[WIP] Display campus ID in username menu

MUMPFL-152: As a MyUW Madison user, I would like to be able to easily find out what my campus id number is, so that I don't have to hunt for it every time I need it.
In this PR:

Get the wiscEduStudentID attribute from Shibboleth and display it if available
Get user displayName from Shibboleth session instead of /portal (any possibility this could negatively impact non-UW users/developers?)
Update username menu to show these attributes and link to my-profile app
Show username menu on mobile

Note: I'm not sure how best to go about linking to "Local Emergency Contacts" app -- looking for suggestions. Also looking for suggestions about how to conditionally display this profile link (hide it when user wouldn't have access to the profile app).
Screenshots

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Yeah, displayName is Madison specific, there's a Wisconsin shib attribute for that as well. So we'll probably need to grab that if displayName doesn't exist?

Hi All,
We were instructed by the registrar's office to not display the student
campus ID in the Academic Navigator.
Was this change from the RO?
Mike
On Jul 12, 2017 2:19 PM, "David Witter"<EMAIL_ADDRESS>wrote:
MUMPFL-152 https://jira.doit.wisc.edu/jira/browse/MUMPFL-152
In this PR:

Get the wiscEduStudentID attribute from Shibboleth and display it if
available
Get user displayName from Shibboleth session instead of /portal
Update username menu to show these attributes and link to "Profile"

Note: I'm not sure how best to go about linking to "Local Emergency
Contacts" app -- looking for suggestions. Also looking for suggestions
about how to conditionally display this profile link (hide it when user
wouldn't have access to the profile app).
Screenshot
[image: screen shot 2017-07-12 at 11 03 47 am]
https://user-images.githubusercontent.com/5818702/28135813-26c03b44-670d-11e7-8f67-37d1054d6344.png
Contributor License Agreement adherence:

This Contribution is under the terms of Individual Contributor License
Agreements https://www.apereo.org/licensing/agreements (and also
Corporate Contributor License Agreements to the extent applicable)
appearing in the Apereo CLA roster
http://licensing.apereo.org/completed-clas.

You can view, comment on, or merge this pull request online at:
https://github.com/UW-Madison-DoIT/uw-frame/pull/470
Commit Summary

Get username menu attributes from Shib and display campus ID

File Changes

M uw-frame-components/css/buckyless/md-generated.less
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-0 (27)
M uw-frame-components/js/app-config.js
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-1 (2)
M uw-frame-components/portal/main/controllers.js
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-2 (51)
M uw-frame-components/portal/main/partials/username.html
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-3 (17)
M uw-frame-components/portal/main/services.js
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-4 (62)
M uw-frame-components/portal/timeout/controllers.js
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-5 (8)
M uw-frame-components/portal/timeout/services.js
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-6 (26)
M uw-frame-components/staticFeeds/Shibboleth.sso/Session.json
https://github.com/UW-Madison-DoIT/uw-frame/pull/470/files#diff-7 (18)

Patch Links:

https://github.com/UW-Madison-DoIT/uw-frame/pull/470.patch
https://github.com/UW-Madison-DoIT/uw-frame/pull/470.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UW-Madison-DoIT/uw-frame/pull/470, or mute the thread
https://github.com/notifications/unsubscribe-auth/AANM5xtVK469S06DpUF_ksvuIDFQFanKks5sNRxXgaJpZM4OWFVn
.

@thevoiceofzeke , @mrfarnham :
Let's ensure display of student ID is behind a (server-side) configuration flag so that we don't accidentally ship this to production in Wisconsin, and then let's separate the discussion about developing the uPortal-home product feature (capable of displaying user ID number) from the Wisconsin-local consideration of whether and how to use this feature.
At the least this looks like a beautiful (really, @thevoiceofzeke , I think it looks great!) product improvement that we may not yet be ready to realize locally in MyUW.


Include demo GIF

Git history in https://github.com/amethyst/space-menace/pull/52 got messy, reopening here.

You mean a branch on my fork?

@szunami yes. It would be great if you can do that.

I'm happy to do that, but I'm a bit confused what the benefit would be.


Fix damaged nextcloud installation

(daftness alert)
I tried to mount a 2nd disk volume at /home/user-data/owncloud and assumed all would be fine...
Of course I copied to content of the original /home/user-data/owncloud first and then copied it back, but nextcloud says a error has occured.
Running the nextcloud.sh doesn't help, it complains about many things.  Also tried running mailinabox again, but no joy.
How can I reinstall nextcloud without losing the rest of the configured services and email?

I tried running the script again...
root@productive:~# curl -L https://power-mailinabox.net/setup.sh | sudo bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  3220  100  3220    0     0   5476      0 --:--:-- --:--:-- --:--:-- 10454
Updating Mail-in-a-Box to v60.2 . . .
remote: Enumerating objects: 113, done.
remote: Counting objects: 100% (100/100), done.
remote: Compressing objects: 100% (26/26), done.
remote: Total 39 (delta 34), reused 17 (delta 13), pack-reused 0
Unpacking objects: 100% (39/39), 12.20 KiB | 416.00 KiB/s, done.
From https://github.com/ddavness/power-mailinabox
 * [new tag]         v60.2      -> v60.2

Primary Hostname: vm1.productive.co.za
Public IP Address: <IP_ADDRESS>
Private IP Address: <IP_ADDRESS>
Mail-in-a-Box Version:  v60.2

Updating system packages...
Installing system packages...
Initializing system random number generator...
Firewall is active and enabled on system startup
Installing nsd (DNS server)...
Installing Postfix (SMTP server)...
Installing Dovecot (IMAP server)...
doveconf: Warning: NOTE: You can get a new clean config file with: doveconf -Pn > dovecot-new.conf
doveconf: Warning: Obsolete setting in /etc/dovecot/conf.d/10-ssl.conf:86: ssl_dh_parameters_length is no longer needed
Installing OpenDKIM/OpenDMARC...
Installing SpamAssassin...
Installing Nginx (web server)...
Installing Roundcube (webmail)...
Updating database schema<PHONE_NUMBER>)... [OK]
Updating database schema<PHONE_NUMBER>)... [OK]
Installing Nextcloud (contacts/calendar)...
PHP Notice:  Undefined index: version in Command line code on line 1
Upgrading Nextcloud --- backing up existing installation, configuration, and database to directory to /home/user-data/owncloud-backup/2022-10-20-16:50:37...

Upgrading to Nextcloud version 24.0.6

Nextcloud is not installed - only a limited number of commands are available

  Command "upgrade" is not defined.

If I could reinstall nextcloud, the upgrade command may work...   How can I do that?

There were a few issues being reported for a while, namely:
https://github.com/mail-in-a-box/mailinabox/issues/2120
https://github.com/mail-in-a-box/mailinabox/issues/2004
https://github.com/mail-in-a-box/mailinabox/issues/1885
This is related to a step on nextcloud configuration that should not be touching that directory, yet they attempt to do it anyway.
The problem with this comes up because when upgrading versions we need to unlock said directory first (which we don't).


Median time is 45 minutes behind?

Is that on purpose?

Yes, that is expected. The median time is the BItcoin network time. It is the median timestamp of the last 11 blocks. This means that it will be ~1 hour behind and update every 10 minutes or so.


GitHub action to rerun a step upon failure with specific exit-codes is doing nothing

To try to mitigate the intermittent CI failures, I wrote this small GitHub composite action which should retry a step when one of the retry_error_codes is hit. In the case of pytest runs, the action is used here:
      - name: Run pytest
        uses: ./.github/actions/retry-step
        with:
          command: pytest mne_lsl --cov=mne_lsl --cov-report=xml --cov-config=pyproject.toml -s
        env:
          MNE_LSL_LOG_LEVEL: DEBUG

Thus, the exit codes on which a retry should be triggered are the default ones, 134 and 139 (Python Fatal Error, segmentation fault).
Yet, it's clearly not working. c.f. this CI run. It's actually doing nothing, with the runner being terminated prematurely.

@larsoner When able, could you have a look at the composite action? Maybe you spot why this short bash script is not rerunning the command 😭

You could for example look at
$ set -o | grep errexit
errexit        	off

If it's on for you then the eval commands will exit as soon as any command hits an error (like set -e has been). So it's possible you need to do something like:
          set +e
          (eval $command)
          set -e

To be explicit, either way at the top of your command I would do something like set -eo pipefail so that if any other commands (other than the one you want to retry) in your script fail it does exit.


have another version in javascript?

Is it possible to provide an version in javascript? or could be used in web with wasm?

It can be used with WASM, but we don't have emscripten bindings for this project yet.

It can be used with WASM, but we don't have emscripten bindings for this project yet.

Looking forward to porting to javascript :)


Add support for CycloneDX version 1.5

Add support for CycloneDX version 1.5

Introduced in release 0.4.0


Mute unconfirmed test case of a dictionary

Fix #14

Codecov Report

Merging #15 into master will increase coverage by 1.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   82.89%   84.04%   +1.15%
==========================================
  Files           4        4
  Lines         152      163      +11
==========================================
+ Hits          126      137      +11
  Misses         17       17
  Partials        9        9

Impacted Files
Coverage Δ

opengraph.go
79.06% <100.00%> (+1.85%)
:arrow_up:

tag_meta.go
84.61% <100.00%> (+1.28%)
: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 2c56ab0...a6ad2d4. Read the comment docs.


once being called twice?

Hi there, I am having an issue where after calling emit in my main.ts file, The on reciever in ui.tsx is being invoked twice or several times.
i am running emit after I call an async function to upload an image:
        uploadImage(prompt, currentSelection, apiKey).then(packData => {
          console.log("Promise resolved for upload image!");
          emit<Base64Creator>("BASE_64_IMG_CREATE", packData)

        })
        .catch( error => {
          console.log("Error encountered during uploadImage: ", error);
        })

on('BASE_64_IMG_CREATE', async function (packData: any) {
  console.log("prompt: ", packData)
  setLoading(true);
  const body = {
      prompt: packData.promptStr,
      init_image: packData.base64img,
      seed: Math.floor(Math.random() * 10000000),
      prompt_strength: 0.8,
      num_outputs: 1,
      num_inference_steps: 50,
      width: 512,
      height: 512,
  }
  ...

as you can see the "prompt: " console log is being print twice, I know the method where uploadImage is called is only resolved once due to there being one console log of the "promised resolved" line

Even if i change it to on to once for the Base64ImgCreator Handler it still is invoked multiple times, heres one example where it is executed 4 times and i am not sure why

You need to wrap the call to once or on within a useEffect when in a Preact (or React) component


[wip] copy resources

first attempt at copying resources

This fails with e.g.
./www/7sgl.pdb

ERROR: NotFound: No such file or directory (os error 2)

NotFound: No such file or directory (os error 2)
    at Object.opSync (deno:core/01_core.js:172:12)
    at Object.realPathSync (deno:runtime/js/30_fs.js:139:17)
    at copyResourceFile (file:///home/jannik/sw/quarto-cli/src/project/project-resources.ts:66:13)
    at copyFile (file:///home/jannik/sw/quarto-cli/src/command/render/pandoc-dependencies-html.ts:204:9)
    at file:///home/jannik/sw/quarto-cli/src/command/render/pandoc-dependencies-html.ts:258:50
    at Array.forEach (<anonymous>)
    at processHtmlDependencies (file:///home/jannik/sw/quarto-cli/src/command/render/pandoc-dependencies-html.ts:258:28)
    at readAndInjectDependencies (file:///home/jannik/sw/quarto-cli/src/command/render/pandoc-dependencies-html.ts:70:5)
    at file:///home/jannik/sw/quarto-cli/src/command/render/pandoc.ts:1050:17
    at withTiming (file:///home/jannik/sw/quarto-cli/src/core/timing.ts:37:20)


add gRPC mTLS authentication

add mTLS authentication to gRPC client and server and expose in the configuration

added in #15 but on underlying tcp connection instead


five_end incorrect output

It seems that five_end() is returning the interval start regardless of the strand.  three_end() seems to be better, except that the intervals have the same start and end rather than differing by one as the documentation suggests.  I'm using version 0.0.98.
>>> gr = pr.from_dict({'Chromosome': ['chr1', 'chr1'], 'Start': [3, 5], 'End': [9, 7], 'Strand': ["+", "-"]})
>>> gr
+--------------+-----------+-----------+--------------+
| Chromosome   |     Start |       End | Strand       |
| (category)   |   (int32) |   (int32) | (category)   |
|--------------+-----------+-----------+--------------|
| chr1         |         3 |         9 | +            |
| chr1         |         5 |         7 | -            |
+--------------+-----------+-----------+--------------+
Stranded PyRanges object has 2 rows and 4 columns from 1 chromosomes.
For printing, the PyRanges was sorted on Chromosome and Strand.
>>> gr.five_end()
+--------------+-----------+-----------+--------------+
| Chromosome   |     Start |       End | Strand       |
| (category)   |   (int32) |   (int32) | (category)   |
|--------------+-----------+-----------+--------------|
| chr1         |         3 |         3 | +            |
| chr1         |         5 |         5 | -            |
+--------------+-----------+-----------+--------------+
Stranded PyRanges object has 2 rows and 4 columns from 1 chromosomes.
For printing, the PyRanges was sorted on Chromosome and Strand.
>>> gr.three_end()
+--------------+-----------+-----------+--------------+
| Chromosome   |     Start |       End | Strand       |
| (category)   |   (int32) |   (int32) | (category)   |
|--------------+-----------+-----------+--------------|
| chr1         |         9 |         9 | +            |
| chr1         |         5 |         5 | -            |
+--------------+-----------+-----------+--------------+
Stranded PyRanges object has 2 rows and 4 columns from 1 chromosomes.
For printing, the PyRanges was sorted on Chromosome and Strand.

I think this was recently fixed. Do you mind updating and trying again? If not I’ll fix it tomorrow.
thanks for reporting. Appreciate it!

I broke it, did not fix it. 9-9 is a zero-length interval. Will fix now.

I've fixed it now, but need to run all tests before I push :)
Thanks! So embarrassing.

Should be fixed in 0.0.104 which is out now: https://github.com/biocore-ntnu/pyranges/pull/210/files

Thanks for the quick fix, and for the library more generally.  It's been quite useful for me!


Dataset cache is duplicated WORLD_SIZE times

In DDP mode each gpu runned in own process and don't have common cache

I don't really want to put a lot of effort into this task because a user can simply turn off caching in case of memory issues.


Update autofill to 7.2.0

Task/Issue URL:
Autofill Release: https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/7.2.0
Description
Updates Autofill to version 7.2.0.
Autofill 7.2.0 release notes
What's Changed

Fix Windows UI line bug by @greyivy in https://github.com/duckduckgo/duckduckgo-autofill/pull/325
Remove separator above Duck Address whenever it's first item by @alistairjcbrown in https://github.com/duckduckgo/duckduckgo-autofill/pull/329
macOS support for in-context signup by @alistairjcbrown in https://github.com/duckduckgo/duckduckgo-autofill/pull/317
Other minor fixes by @GioSensation in https://github.com/duckduckgo/duckduckgo-autofill/pull/328

Full Changelog: https://github.com/duckduckgo/duckduckgo-autofill/compare/7.1.0...7.2.0
Steps to test
This release has been tested during autofill development. For smoke test steps see this task.

Closing in favour of: https://github.com/duckduckgo/duckduckgo-privacy-extension/pull/2031


Add support for user-specified PWD from the command line.

When the $PWD is not available inside the container, singularity currently defaults to starting the process inside the user's $HOME.
It's not a bad strategy, particularly for interactive use.  However, when invoked from other tools (such as a batch system), it would be useful to be able to specify this directly.

I had the one comment, but aside from that, looks good!


taking over this project

Hello Passy,
I read your deprecation note in the readme. I need to get this project working for a client, can I take over this project?
Thank you

That's what the fork button is for

coolio, thanks


In Chrome, text in modal dialogs is not translated until the dialog is clicked

Overview
When a modal dialog is shown in Chrome, the text is displayed in English even if a different language is selected. When browsing with Firefox, the text is initially shown in English but is translated after a short delay.
Expected Behavior
When a modal dialog is opened in Chrome, the language of the text in that dialog matches the language selected on the header bar.
Actual Behavior
When a modal dialog is opened in chrome, the text in that dialog is displayed in English. Clicking on whitespace in the dialog triggers the translation.
Steps to Reproduce

Browse the login page in a non-English language (http://localhost:6543/auth/login#googtrans(en|da))
Click the password reset link

Demo

Additional context
We are using a client-side Google translation library
https://github.com/open-apparel-registry/open-apparel-registry/blob/7352adcb210557e7db99cfd0683e53b957305090/src/app/public/index.html#L71-L76

I just tested this out in Chrome and Firefox and the dialogs are now translated immediately after they appear rather than after interaction. There's still a small flicker when they are rendered in English before having the text adjusted into the target translation language, but I think this is an unavoidable consequence of how React rendering interacts with Google Translate.
I believe this got fixed by https://github.com/open-apparel-registry/open-apparel-registry/pull/322


Error cannot unmarshal number into Go struct field when attempting to use data "meraki_devices"

upstream: https://github.com/cisco-open/terraform-provider-meraki/issues/130

fixed in newest release


A qlog file formated with json-seq

I have got  a qlog file（it‘s JSON-SEQ format），So I want to convert JSON-SEQ to JSON. Because website's qlog example supported JSON.
How can I slove this problem, any tools or function ?

Hey @nixwl,
qvis should just support the JSON-SEQ format as well, so there should be no need to convert to JSON (as such, there are also no existing tools for this (though it should be easy enough to write one, it's quite simple)).
Can you share an example file and the implementation you're using maybe? If you're actually getting errors loading the file into qvis, which errors?

hi， I can import  JSON-SEQ  format qlog file to website，but I found  that:

This qlog file record client download file form a QUIC file server（the QUIC server uses TQUIC）
This is the client  qlog info:

and client qlog file is here:    qlog_c.txt
I don't know if my qlog file has any problems, but  the connection may be established,  and file download function had well worked

I take that issue to TQUIC developer few days before. And they make a scipt to covert JSON-SEQ to JSON, So I try it recently.
This is the result:(converted qlog file is here: qlog_c.qlog.json)

So it turns out that the JSON-SEQ file you use isn't actually proper JSON-SEQ as per the qlog spec :)
It just uses a \n newline character to separate the records, but that's not enough: they also need to be separated by the special RS (record separator, 0x1E) character (see also https://www.ietf.org/archive/id/draft-ietf-quic-qlog-main-schema-07.html#format-json-seq).
Since they don't do that, qvis can't properly parse the file and shows an error in the JavaScript console (I agree it's confusing that it does show a green popup in the webpage UI, pretending that it did load correctly ;)).
So, the solution here is to ask the TQUIC maintainers to either output proper JSON-SEQ as per the spec, or for you to use their script to convert to JSON, which indeed does seem to work as expected.
(I will keep this issue open to remind myself to one day properly show a red popup with an error message if this error occurs again, rather than people having to open the JavaScript console in the developer tools).


Update media-services-sspk.md

Link was broken

Hi @xpouyat - This pull request was opened in the public repo. It looks like you are the article author. Would you make this content update in the private repo instead? Thank you! We'll close this PR.

Sure


feat: add sf-org-open commands (+ open current file)

I added sf org open with the option to open the org, as well as the current file (e.g. flow)
This does not perform a lot of validation, though, so it's the user's responsibility to make sure they can use sf org --source-file ... properly.
Happy to hear some thoughts!

@xixiaofinland  I'm having some issues with IDE, so i have to apply a bunch of smaller commits.
Changes should be there now.
Please let me know if I missed anything, and thanks for suggestions!

@ognjen-vuceljic
I added a commit after the PR approval to tweak some small things.
One question, how does sf org open --source-file really work? Does it work only for pages? I read sf org open --help but still not fully get it.
If it works oddly/in unexpected behavior, I might consider disabling the sO default key.

@xixiaofinland  In theory, the sf org open --source-file should open the current file that we specify.
Since it works well for flows, and it might be a useful feature, can we modify the list of defaults in config.lua to include Flow metadata type?
As it stands right now, the sO hotkey won't be run when we are in Flow, since this metadata type is not added to defaults. It works from APEX code, but the command itself will open a Lightning App Builder instead of APEX class, which is a sf issue I guess.
So far I noticed that the main use case for --source-file is opening a Flow directly via sf command

can we modify the list of defaults in config.lua to include Flow metadata type?

It's part of the default configuration, thus meant to be adjusted by users according to each's individual needs.
Is flow a xml file? you can add it into your setup like below.
require('sf').setup({
   // other own settings you already have;
   hotkeys_in_filetypes = {
    "apex", "sosl", "soql", "javascript", "html", "xml"
  },
})


Google Data names uses wrong charset

Seems that Google documents filenames are not retrieved and displayed in uft-8 as stated in the meta charset attribute.

Should have stated that this is using OpenRefine 2.6-beta.1 on Arch Linux.


Create and print out attributes to the profile window

Add an NFT to your wallet with certain attributes (e.g., "federal," "military," or "space").
Display the attributes as normal text to the "attribute"/"organization" tab in the profile window.

No longer doing "federal" "military" and "space" attributes. To keep things simple, we will only have dataset-access and algorith-access

This issue overlaps with issue #29, so we will merge this issue into #29


Error: unknown flag: --live-addr

Describe the bug
Pod external-secrets can't start successfully.
kubectl get pod -A shows:
NAMESPACE          NAME                                               READY   STATUS             RESTARTS      AGE
external-secrets   external-secrets-6766d86d7-gthzm                   0/1     CrashLoopBackOff   4 (72s ago)   3m4s
external-secrets   external-secrets-cert-controller-76cdcfdcc-dv67k   1/1     Running            1 (86s ago)   3m4s
external-secrets   external-secrets-webhook-57f877cb54-6664c          1/1     Running            1 (86s ago)   3m4s
kube-system        coredns-5dd5756b68-fbxzv                           1/1     Running            1 (81s ago)   4m18s
kube-system        etcd-minikube                                      1/1     Running            1 (86s ago)   4m31s
kube-system        kube-apiserver-minikube                            1/1     Running            1 (76s ago)   4m31s
kube-system        kube-controller-manager-minikube                   1/1     Running            1 (86s ago)   4m31s
kube-system        kube-proxy-hz6sl                                   1/1     Running            1 (86s ago)   4m18s
kube-system        kube-scheduler-minikube                            1/1     Running            1 (86s ago)   4m31s
kube-system        storage-provisioner                                1/1     Running            3 (73s ago)   4m30s

kubectl logs external-secrets-6766d86d7-gthzm -n external-secrets shows:
Error: unknown flag: --live-addr
Usage:
  external-secrets [flags]
  external-secrets [command]
...
...

After I revert the this pr, everything goes well.
To Reproduce
Steps to reproduce the behavior:

Pull the latest repo
Follow the doc to install the Helm chart locally

Expected behavior
Pod external-secrets  starts successfully.
Screenshots

Additional context
Add any other context about the problem here.

@fdberlking f*ck :DD I have to revert the commit, or otherwise we'll need to fix this. :(

https://github.com/external-secrets/external-secrets/pull/3131 Introduced the new flag. But the tests passed. Apparently we did it incorrectly. Or I don't know.

It's most likely because the chart is downloading the version that doesn't have that flag? It's really weird.

I'm going to revert said Pull Request than try to investigate what's wrong with it. https://github.com/external-secrets/external-secrets/pull/3213

#3131 Introduced the new flag. But the tests passed. Apparently we did it incorrectly. Or I don't know.

Exactly 😄 I really do not understand why the problem comes up after I was running through all of this tests to make it actually available 😞

#3131 Introduced the new flag. But the tests passed. Apparently we did it incorrectly. Or I don't know.

Exactly 😄 I really do not understand why the problem comes up after I was running through all of this tests to make it actually available 😞

The revert will take a while to go through if you fix it before that, I'm happy to approve it. :D

I would expect so too. I have no idea what's wrong. 🤷

It's most likely because the chart is downloading the version that doesn't have that flag? It's really weird.

I'm a new to kubernetes and it's my first time to run this project. I'm a little worried that I made any mistakes. Can you reproduce it on your machine?

I'm a new to kubernetes and it's my first time to run this project. I'm a little worried that I made any mistakes. Can you reproduce it on your machine?

Don't worry, you didn't. The linked PR is at fault somehow. But we have to figure out how. :D I'm reverting that commit from main then it should work.

@Hayden-Chang Can you please try again now? The PR was reverted.

@Hayden-Chang Can you please try again now? The PR was reverted.

It works well now. Thank you~

Nice, thanks for reporting.

Nice, thanks for reporting.

Thus, could we close this one (and I come up with something to solve the other, flag related issue)?

Nice, thanks for reporting.

Thus, could we close this one (and I come up with something to solve the other, flag related issue)?

Yes. Let me do it


Angela - Add new routes and serialization

Changes Made:

Added routes for Employee profile, HR profile, Renumeration, Renumeration Description, Experience.

Testing:

Verified the features on localhost.

Additional Notes:

No conflicts were encountered during the merge.

Looks good, merging