More 32 core Linux VM capacity

Today, I noticed jobs on Engine CI getting cancelled due to queuing while waiting for 32 core Linux machines. Currently, it looks like we have 49 32 core VMs available assigned as follows:
prod: 15
staging: 4
try: 30
total: 49
Would it be possible to increase this to 70 VMs allocated like:
prod: 25
staging: 5
try: 40
total: 70
So 21 more 32 core VMs.
cc @yusuf-goog

F.Y.I.: there are 20 new 32-core targets enabled in try and 26 new 32-core targets enabled in prod in these weeks (from packages repo). They seem to start affecting the queue time now.
/cc @stuartmorgan I somehow remember you mention something to improve, like sharding? Also wondering how many more 32-core targets we are expecting on the way?

Added in: go/cirgs/6152279

@keyonghan I just increased the capacity. @stuartmorgan can you please take a look and see if we can optimize capacity usage. Do you want me to create a issue tracker for that?

@stuartmorgan I somehow remember you mention something to improve, like sharding?

I won't be able to evaluate the sharding of the android_platform_test task until the blockers for moving Android unit and integration tests from Cirrus to LUCI are resolved.

Also wondering how many more 32-core targets we are expecting on the way?

No new ones are planned (modulo the above), but runtime on the 12(?) android_platform_test tasks will increase when unit and integration tests migrate.

@stuartmorgan can you please take a look and see if we can optimize capacity usage

As I mentioned earlier, I would need someone to point me to tools for understanding the machine utilization. For Cirrus it was very easy to figure out when we needed to move tasks between light and heavy machine configs because every run has CPU and memory utilization graphs (example). What's the equivalent for LUCI that would let me evaluate the current setup?

Thanks @stuartmorgan for the clarification. Considering we had only <10 targets needing 32-core bots before, I am not sure if the proposed capacity increase here is enough. I will need more data to confirm. Before that, let's land the capacity CL to unblock the engine tree. At the same time, let's keep monitoring a bit.

New hosts are deployed

FYI I ran some timing tests over the weekend, and I don't see any evidence that the use of 32-core machines in flutter/packages is actually making a meaningful difference in run times, so I am converting all of the flutter/packages configurations to 8-core.

FYI I ran some timing tests over the weekend, and I don't see any evidence that the use of 32-core machines in flutter/packages is actually making a meaningful difference in run times, so I am converting all of the flutter/packages configurations to 8-core.

Thanks for the heads up @stuartmorgan . This is good to know, and surely will help with the 32-core queue time.


Object stick together when using cannonjs as physic engine

For example, when try to switch to use CannonJS in this playground
https://www.babylonjs-playground.com/#8PQK71
e.g. add this line to playground
scene.enablePhysics(null, new BABYLON.CannonJSPlugin());
All object sticker together once start

@kelvinkoko Thanks for finding this. It looks like cannon was ignoring the connectedPivot parameter you were using.


Implement fit content width in TextEdit

TextEdit already has scroll_fit_content_height, but there is no equivalent for width. This PR adds scroll_fit_content_width with the same behavior for width. When both are enabled, there are no scrollbars. I have tested that this works correctly in The Mirror and in a minimal project.

Is it working with line wrapping enabled?

@bruvzg Yes. When wrap mode is set to Boundary it wraps instead of getting wider, behaving the same as if this setting is disabled. Note that these settings are intended to be mutually exclusive, having both enabled is not useful.

Thanks!


Profdir with profiles.yml

Hi @mwhitaker
I am using your profdir branch in a ci. When providing a DBT_TOKEN, the sed command targets a datab.yml file. IMO this should be the standard profiles.yml file.

Hi there,
Many thanks for this PR. I don't remember why I created the profdir branch, so am curious why you use that branch. Is there any feature in there that is different from the main/master branch?
Other than that I don't mind merging your PR since it's not an official branch.

I use the profdir branch as the main branch of this action requires the profiles.yml file to exist alongside the dbt_project.yml file. In my project, I have the following setup:
├── .ci
│   ├── profiles.yml
├── .github
│   ├── workflows
│   │   ├── dbt_action.yml
├── packages
│   ├── dbt
│   │   ├── dbt_project.yml
│   │   ├── models

This gives me and other developers a clear picture that we have a specific profiles.yml file for ci that is not to be used elsewhere.
The difference between main and profdir is that in profdir there is an additional input of dbt_profile_folder, which I specify as dbt_profile_folder: "../../.ci" to account for my project structure.

OK. I merged the PR.
For some reason, perhaps for testing specifically Databricks, I used a different profile yml, but I agree that it makes more sense to use profiles.yml.
Thanks for the PR!
Michael


#142 - updated podfile

As per #142

Updated podfile to cocoapods v 1.5.3

Job #143 is now in scope, role is REV


Custom File Name For Download

I am looking for a way to add a file name to the downloaded file. This is for URLs such as http://someurl.com/getFile?id=123

I'm faced the same need


feat(tutorial): complete step 5 - Rochester

Here is the deployed URL:  http://carbon-tutorial-drstanic.mybluemix.net

Looks good. Marking step 5 complete!


C# code being displayed as Smalltalk in language statistics

https://github.com/Iwuh/IwuhSelfbot You can see here that although all the code in my repo is C#, Github says that it's 33.5% Smalltalk.

This is the same bug as #353:
bundle exec linguist ../IwuhSelfbot/IwuhSelfbot/src/IwuhSelfbot/Commands/Services/EmojiService.cs
/linguist/lib/linguist/blob_helper.rb:281:in `split': invalid byte sequence in UTF-8 (ArgumentError)
	from /linguist3/lib/linguist/blob_helper.rb:281:in `lines'
	from /linguist3/lib/linguist/blob_helper.rb:298:in `loc'
	from /linguist3/bin/linguist:59:in `<top (required)>'
	from /var/lib/gems/2.3.0/bin/linguist:23:in `load'
	from /var/lib/gems/2.3.0/bin/linguist:23:in `<main>'

You can use Linguist overrides to fix it locally.

Going to close this as a dupe of #353. Thanks for the report!


added cask for STAMP (4.2.15)

STAMP allows users to migrate from one music streaming
service to another easily.

Thank you for the contribution. It was merged directly as ab055b7bca72eb8441b666225379e47a0065380f to keep commit history cleaner. Your contribution is still credited to you.


Thin Region Selector

Implementing a thinner Region Selector.
Lots of work to do!
To Do

[x] Basic UI
[x] fix captured Region
[x] fix Snap to Window
[x] Resizing using border
[x] Lower opactiy of controls when not mouse over
[x] move size boxes below main controls on small region size
[x] remove Hide during Recording option.
[x] low opacity of size boxes during recording.
[ ] low opacity animation.

First Look

@Godly75
Please check this build for new Region selector: https://ci.appveyor.com/api/buildjobs/twah80tvxo04f9yb/artifacts/Captura-Portable.zip

way way beter !!!! :)
I love it ! This app has become so much more valuable the last days !
It's gonna be my nr1 app for screenshots.

Suggestions to the video region recording:

maybe add the magnifier to the video region selection also?
add the screen size pixels notification the same as in the screenshots (instead of the right bottom corner)
add big countdown notification on the delayed start

maybe add the magnifier to the video region selection also?

No, this can't be done because while the magnifier is shown, user cannot work with other apps.

add the screen size pixels notification the same as in the screenshots (instead of the right bottom corner)

It's not a notification but those text boxes can be used to changed size. I think they should not be put between the region where user might want to do his work.

add big countdown notification on the delayed start

Not big, but I added countdown in place of Duration on delayed start.
The Start Delay setting has changed to Pre-Start Countdown and is now specified in seconds.
It will be shown on both Main UI and Region selector.
Please test: https://ci.appveyor.com/api/buildjobs/ux6hwho7h3njfl8d/artifacts/Captura-Portable.zip

Works perfect ! I was thinking to see it big in the middle of the screen, but this works also :)


off-by-one reading in pbc_wmessage_string() line 268,303 ?

I don't think so, but fix it is better :)


added a unit test for logging encoding issues

PR Checklist

[ ] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated

Description of changes

Codecov Report

Merging #106 (1086f25) into main (bc0c967) will decrease coverage by 1.1%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main    #106     +/-   ##
=======================================
- Coverage   94.5%   93.3%   -1.2%
=======================================
  Files         30      30
  Lines       1132    1145     +13
=======================================
- Hits        1070    1069      -1
- Misses        62      76     +14

Impacted Files
Coverage Δ

tests/test_import_parser.py
72.5% <100.0%> (-27.5%)
:arrow_down:

deptry/import_parser.py
96.7% <0.0%> (+3.2%)
:arrow_up:

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


a bug in browser env with @toast-ui/editor-plugin-code-syntax-highlight

Describe the bug
I just want to decorate it but not too be familiar with it.
when I try to introduce the plugin local with tag

@www159
Try to use CDN link in your case~!
Please refer this lilnk


Custom ExperienceMaker

I have a use case where I'd like to use a custom ExperienceMaker class instead of either of the provided ones. As far as I can tell, there isn't currently a way to configure the experience maker class used by PPOTrainer / ActorPPOTrainer.
Would you be open to a PR that makes this configurable?
And if yes, would the best way of doing this be to put it into the DeepspeedStrategy?

I think ExperienceMaker  has nothing to do with DeepSpeed
I would like to know what kind of ExperienceMaker  maker you need first

The alternative would be to e.g. pass the experience maker class as an optional argument the the Trainer constructor, but in the Ray case that would require passing it through several other objects as well, i.e. more changes.

I think it's probably better this way
Welcome to have related MR contributions~

OK! I'll test this on my side for a bit to make sure it covers all the use cases I have in mind. I'll open a PR in a couple of weeks.


Enhancement Request: Specific entries in the LogBook for Scheduled events/Automation.

I think it would be useful to have entries in the logbook that note specifically when a scheduled event has taken place.  It would be nice to know when something happened as a result of Automation or the Scheduler component rather than a person initiating the action.
Maybe a highlight, italicized, a system icon or just additional verbiage in the log.
I think this is also a great way to help new users gain additional insight into the workings of Home Assistant without having to dive too deep into the actual logs (which could prove technically challenging to some)

I think this https://github.com/balloob/home-assistant/issues/338 could also be a solution for this.

Fixed in dd71e4fdd1129a492339b54587c4410f3414004f


Confusing item in documentation

Thanks for taking the time to build this.
Just wanted to point out something in the documentation that is confusing.
The docs say:

Add the $clipboard magic property to your project by importing the package before Alpine.js.

But the example right below it shows Alpine being imported first.

Looking at this again, perhaps the order of the import statements doesn't matter. What matters is that Alpine.plugin(Clipboard) is called before window.Alpine. If that's it then I apologize for the confusion.


output image shape from export_for_gaussian_splatting.py is not same as the original image

I use the COLMAP GerrardHall Dataset to test the script.
Original Image

Exported Result

Hi, image is changed because of the undistortion process.
Alignment of cameras in Metashape includes precise intrinsics/extrinsics camera parameters calibration. But because of export into colmap format - those parameters should be converted into format supported by colmap, and this requires proper images transformation too.
So this is an expected behavior.


Allow user to choose from multiple templates

Status
Currently @FredKSchott has created the structure to allow for multiple templates.
Each template belongs in its own subdirectory under assets/ (see app-preact). The template then requires a config.json file with any details that are required for this template, such as npm scripts, and most importantly, install commands so that the create-pika-app wrapper can install and prepare the project for the user.
Todo

Allow the user to select a template e37730098cfd0d5dca26d44d1f6729109ef71be9
When they are executing create-pika-app with only the project name, we could show a cli select menu  where it asks the user which available template they'd like to use. I think it makes sense to offer the template selection as a cli argument as well so that you can run this without any further user input. We could list the available template options either in --help and/or in the README.

Integrate packages from Pika CDN?
So im not positive how possible this would even be, but what if you could pass a pika cdn URL as the template argument and it would prepare that project for you? Would this be helpful? How much easier would this be than simply npm install https://cdn.pika.dev/package ?
Can we deliver any value on top of installing the package via this wrapper?

Integrate more templates :)

@FredKSchott - I made an issue out of this so we can continue the discussion here. That way we dont have to keep flipping back to a closed PR.

@thiagoabreu I noticed you have a @pikapkg/web example project listed on their examples.md. Do you mind if I use that project here as an initial second template to test selection, etc.?

@thiagoabreu I noticed you have a Vue @pikapkg/web example project listed on their examples.md page. Do you mind if I use that project here as a second template to test selection, etc.?

No problem. :+1:

@thiagoabreu updated and integrated now :)

We now have preact & vue, safe to close?

Can we add a lit-html/lit-element template, too? I got a similar, but more minimal, setup here

@ndom91 We now have preact & vue, safe to close?

Yeah lets close this one. If people want to add more, they can open a new issue specific to their template.
@thepassle - can you please open a new issue regarding your lit-element template?
thanks!


Implementation Issue for SuperMixins: VM

What: Issue for implementation for VM
ETA: 10/15 in Dart Repo.
For: SuperMixins Implementation and strawman documentation
Meta Issue: This is subsumed under the Super Mixins Meta Issue in the Language Repo
Comments: Feel free to break this further into further tasks and issues, or use as a meta issue for all the tasks in the implementation plan

CFE is using the current framework available in the VM for all the new SuperMixins support, there is no additional work required in the VM for this.


Let's put some real code here

Wordpress server stack
@blackJack1982 Maybe we could include our docker-compose and try to deploy it with github actions to different target

[ ] dev
[ ] staging
[ ] prod

What do you think ?

Ok, but maybe let's see what's assigned to follow the tutorial.


[Epic] iOS: XCUITest integration

Motivation
Currently, there are several gaps in Detox iOS we would like to eliminate, especially the non-support of these two:

Multiple apps flows (tests that involves multiple applications).
Real devices (running tests on a physical device).

There are other features we would like to implement and support but we are limited to doing so, infrastructure-wise:

Web-view interactions.
Unmocked push notifications flows.
System dialog interactions.

Currently, in Detox, the entire interaction with the app under test (or, AUT), is done from the JS side, communicating with the Detox framework, in which resides inside the app, aware of all the resources (DetoxSync for instance), and triggers UI events from within. This way, Detox is limited to interact only with the AUT.
Note that, running apps on physical devices, and specifically UI tests, will require code-signing the application with signing certificates issued by Apple.
Also, on iOS Simulators, Detox framework is injected dynamically into the application (using the DYLD_INSERT_LIBRARIES environment variable). For physical devices, inserting dynamic code to the application will require to code-sign the dylib with the same signing certificate of the running application (Apple Code Signing).
Since in many cases our users do not use code-signing on their applications on their local environment, we would like to keep this step optional (unless running on real devices).
XCUITest
XCUITest is a user interface testing framework launched by Apple in 2015, the XCUITest framework rests upon two core technologies: the XCTest framework and Accessibility.
The UI testing in XCUITest is based on the semantic data about the UI that is exposed as "Accessibility", it works by locating the UI elements using queries, sending UI events to these objects and providing a rich API enabling us to examine the elements' attributes and state, and comparing them against an expected state.
The transition to accessibility-based operations, will allow Detox to interact with multiple apps, iOS SpringBoard, push notifications, system dialogs and other system elements all managed from the test target process, without having to inject any dynamic code injected into the AUT. We should also be able to operate on apps without the additional phase of inserting dynamic code.
Phase 1 (Reimplement Detox iOS)

[x] POC: executing XCUITest operations from a WebSocket.
[x] Integration: reimplement Detox iOS simulator driver with XCUITest.
[x] #3346
[x] #3367
[x] #3380
[x] #3429
[x] #3448
[x] #3447
[x] #3449
[x] #3463
[ ] #3465
[x] #3466
[ ] #3467
[x] #3468
[ ] #3481
[ ] #3830

Next Phases

[ ] Multiple apps support for iOS.
[ ] Support unmocked push notifications flows for iOS.
[ ] Implement iOS real device driver (see https://github.com/wix/Detox/issues/95).
[ ] Web-view support for iOS (see https://github.com/wix/Detox/issues/3030).
[ ] Interact with iOS system alerts and dialogs (fixes https://github.com/wix/Detox/issues/3017, https://github.com/wix/Detox/issues/3445).

This is an amazing summary. Very well articulated. I consider this the new feature Epic, then.

how we looking on this? I saw we now have android web view support. so close to bailing on appium...

Are you guys planning to use EarlGrey 2 for achieving this?
This is such an important feature. There's a huge enterprise opportunity with the adoption of Ionic. Being able to interact with a Webview would be key for Ionic apps :)

@guzmanoj this would allow for web-views testing, as well, in due time!

Are you guys planning to use EarlGrey 2 for achieving this? This is such an important feature. There's a huge enterprise opportunity with the adoption of Ionic. Being able to interact with a Webview would be key for Ionic apps :)

No, we don't have any plans to use EarlGrey 2 for that.
I did not get to check ionic at all, this sounds very interesting. Thanks :+1:

our app's login module is inside a web view which works for android but i'm eagerly waiting for detox team to release the same support in webview for ios as well

@asafkorem
Shall you need any help for testing the feature on an Ionic app (webview based) just ping me!
I'm really looking forward to introduce Detox as our official E2E testing framework for mobile apps ;)

Sounds good, thanks @guzmanoj, that would be great if we could use Detox on ionic apps :+1:

Hi!
What's the progress on this? I see most of the Phase 1 issues are closed but no PRs ale linked.

It's actually in the works. Is there anything specific you're planning on integrating once this is done?

It's actually in the works.

Great! Is it open-source?

Is there anything specific you're planning on doing once it is done?

I'm developing a new UI testing framework for Flutter and was wondering how you guys solved the problems mentioned in this issue :)

Yes @bartekpacia, and  I'd be happy to share my knowledge and learn more about your work on this project. As far as I know, Flutter doesn't use the standard UI hierarchy and elements of native platforms, but please correct me if I'm mistaken. If you're interested, we can continue this conversation on Discord, my username is Asaf Korem#7718. 🙂

Hi @asafkorem, thanks for your will to help :) Fortunately, we've already managed to implement this feature. See https://github.com/leancodepl/patrol/issues/661 for more.

looking forward to seeing new features and implementation to support about mentioned limitations. it's amazing to support and believe that most of us face these issues. I'm willing to join test once it release. hope Detox become the best UI testing framework for cross-platform and native apps

looking forward to seeing new features and implementation to support about mentioned limitations. it's amazing to support and believe that most of us face these issues. I'm willing to join test once it release. hope Detox become the best UI testing framework for cross-platform and native apps

Thanks @aaronpliu. You are more than welcome to start testing out our XCUITest (i.e. Detox v21) RC, today!

Any updates on the previous comment from @chrisgradl ?

I'm also interested to know if Detox can handle nested elements that have a depth of more than 60 (in regards to @chrisgradl's comment, as this has been an issue for a very long time in Appium and we are currently unable to test our iOS app due to this issue with Appium.
Considering making the swap to Detox but I'd like to know if we will run into this same issue using Detox.

@chrisgradl @matiaswastaken we were not aware of this issue but that's not to say we'd be affected by it. We will check (@asafkorem wdyt?). Do you have a use case where you could give the Detox 21 rc a try and see why it breaks? In any case, I do wonder what app has views nested all the way down to a depth of 60, tbh.

@d4vidi If I remember correctly, the common cause of this is using the React Navigation UI libraries, which apparently use a lot of nested elements.
I'm not sure why the deep nesting is necessary, but it is a popular library so there are probably a lot of Detox users that also use React Navigation.
This bug is actually the main reason I switched from Appium to Detox. I would guess that this change may break tests for a lot of people.

@d4vidi  Hi,
The issue is reported in here with sample application code to reproduce it.
I'm also interested in if the issue is reproduced with Detox or not.

@d4vidi Hi, The issue is reported in here with sample application code to reproduce it. I'm also interested in if the issue is reproduced with Detox or not.

Thanks. We will look into it once the upcoming release of Detox 21 fully materializes 🙏🏻

For those waiting for the release of this version specifically for WebView testing capabilities, it's important to highlight that WebView-testing APIs are already available on both platforms, independent of XCUITest.
Comprehensive information can be found in the documentation: API, guide.

This is no longer an epic, since we decided to do the integration in smaller steps. Initial XCUITest integration is now part of iOS: Support System Dialogs epic.

Closed due to https://github.com/wix/Detox/issues/3208#issuecomment-2029624077


Create lib folder with Romi-specific support

This PR refactors the Romi lab code so that instead of making things look like a Kobuki, everything is specific to the Romi. Chapter 7 of the lab manual has been updated accordingly.

Looks good to me. :)


[HLS] Batch resolving subtitles' playlist on start

Have you read the Tutorials? yes
Have you read the FAQ and checked for duplicate open issues? yes
What version of Shaka Player are you using? 3.0.1
I've met strange to me behavior when player start to download all available subtitles' playlists from master's playlist on load, any way to get rid of it? Such behavior makes player freeze on start.

Hello @interlark , could you please follow our bug report template, and provide us with more details to reproduce your issue? Thank you!
https://github.com/google/shaka-player/issues/new?assignees=&labels=&template=bug_report.md&title=

This is working as intended for the moment.  We need all the metadata loaded during startup.  See #1936.

Ok, no lazy-load for manifests. Got it, thank you.


Codecov Report
Base: 36.28% // Head: 36.41% // Increases project coverage by +0.12% :tada:

Coverage data is based on head (1e06300) compared to base (4596997).
Patch coverage: 82.35% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
+ Coverage   36.28%   36.41%   +0.12%
==========================================
  Files          46       46
  Lines        2588     2598      +10
==========================================
+ Hits          939      946       +7
- Misses       1539     1541       +2
- Partials      110      111       +1

Impacted Files
Coverage Δ

pkg/terraform/terraform.go
80.15% <82.35%> (-0.88%)
:arrow_down:

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.


Add a switch to disable localization

Would be nice to have a switch to disable translation of EpubCheck messages and show original english messages only.
Current workaround with setthing java opts:
java -Duser.lang=en -Duser.country=US -jar epubcheck.jar test.epub

Better:
java -jar epubcheck.jar --disable-translate test.epub

I think implementing a --lang <lang> option as suggested in #498 would be a better solution.
Closing this in favor of #498


Admin Activity chart pop-out dialog does not display on first render

So this results from "reusing" the same chartmodel to show the chart in two places.  Clever, and you can see why an application developer would want to do that, but chart model is not really prepared to do that, because it stores a reference to its actual high charts model in state.
We are a bit inconsistent throughout the API about our expectations regarding whether multiple component
You could certainly imagine a different implementation of chart/chart model that stored the highcharts object in a local component specific model but right now the object is on the main app model, and indeed documented as a public property which applications can read from.

Greg -- took an initial look at this and have some ideas.  Let's discuss

Closed with #3019


Release for scala 2.11

Any chance to release it for scala 2.11?
Yeah, I know 2.12 is modern. But I cannot easily upgrade to 2.12 cause of a lot of dependencies.

The same for us. We work with spark and we are forced to stick to 2.11. Would be possible to release the last version for 2.11 ? Thank you

:+1:


Have you extended the vocabulary size of the tokenizer?

Nice work on your repo. I'm curious to see whether you managed to achieve your results in Chinese and multilingual models with or without resizing the vocabulary size of the tokenizer.
Have you extended the vocabulary size of the llama to support the multilanguage training?
Please let me know.

Hi @mohataher,
Thanks! We did not extend the vocabulary size for llama.
Best,
Zhihong


PR Merge Score

Hey Tony, I've got good news and bad news. The good news is that everything is compiling and running. The bad news is that there is some bug in your scoring that is allowing the MergeScore to be a negative! The function's range is [0,1]. Im not sure where the negatives are coming from just off the top  of my head, but you will need to investigate and resolve this problem before this feature can be marked as complete and merged into dev!

I had set some of the returns as negative when error checking so I knew where they were coming from. I forgot to reset them to zero. The return was also set to be a percentage. I changed that back to being less than or equal to one. Should be all good now. If you run into any more bugs, let me know!


Method coder displays wrong method when opened

Seen in GT v0.8.2360:

There are two method coders for signatureBuilder, the lower one being the correct one. The upper one showed rewriteSystemBuilder when it was closed. Upon opening, it switched to displaying signatureBuilder as well. Closing it makes it revert to the correct method:

This looks like the same issue I was having

It should be better now. We should reopen the issue if it appears again


Add ability to dynamically hide menu / nav items

Hi there!
Loving what you're doing with the v3 branch. We have a use-case where we want to dynamically hide / show certain pages (we have some docs behind auth).
It isn't possible to dynamically do checks as to whether these pages are shown or not, so I've added a config option that takes a function, that allows you to check if the menu item should be rendered or not (both for the Navbar and Sidebar)
It is used like so in your theme.config.tsx:
export default {
  menuItems: {
    visibilityCheck: (item: PageItem | Item) => {
      if (item.route === '/hidden-page') {
        return false;
      }

      return true;
    }
  }
}

I thought a better route would be supporting _meta.tsx with a dynamic visibility type, but that would require the meta file supporting items as functions and I'm not sure how to do that 😄
Feedback appreciated!

please create an issue first


Fix typo in the .red class text

From REDY to READY.
It creeps me out when I see mistakes all over the place. I dunno, its just me by the way.

Its on purpose.

@aaron7pm Ah, I get it now. I didn't get it in the first place but now I know it. :smile:


Support multiple orientations

Measuring the window at load time doesn't work when the orientation changes and width becomes height. Luckily RN now supports percentage based dimensions so we mostly don't have to use measurement at all.

Currently the Modal component isn't working as expected. That is due to a missing option "supportedOrientations" in the Modal component, as seen here


avoid descheduling when ReplicaSet=1

fix #786
Compared with setting PodDisruptionBudget for every RC/RS, easy to avoid evicting pods with replicasSet=1.

@jiangxiaobin96  Excuse me, how is it going now?

@jiangxiaobin96 Excuse me, how is it going now?

Let me solve the conflict and rebase.

Because this issue has been create some time ago, it has large difference with master branch. I will close this issue first and reconstruct this issue based on new branch.

Because this issue has been create some time ago, it has large difference with master branch. I will close this issue first and reconstruct this issue based on new branch.

How is it going now?

Because this issue has been create some time ago, it has large difference with master branch. I will close this issue first and reconstruct this issue based on new branch.

How is it going now?

testing, this week can finish

reconstruct  in #1152

/reopen
keeping this open until the PR merges


convert all scss font-szie to rem from px

Describe the changes you have made in this PR -
Changed all font-size: xx px to rem

Coverage remained the same at 84.266% when pulling 8682acd080798627a3584e3324391f12d8acb280 on abstrekt:css into 4b67855b21c0a7aaa7e105a2f50ced3ad2d6b3ce on CircuitVerse:master.

but they are static, it's about the user more than the device i believe, if someone has browser font size to high or low , having rem will take that change, pixel will remain the same no matter the browser settings

I thought px won't be affected by the browser font size settings, but it seems to work fine, nevermind @nitin10s, thanks for the review.


Feat: Improvements on French translation

Improved the general and ingredients translation (translation.json and ingredients.json).
Added french translation for cocktails instructions (instructions.json).
Hope you'll like it. I love your app btw <3

Thank you for the PR. Great work 👏


mobile navbar fixed to top on scroll

Make the mobile navigation bar stay on the top when scrolling.

This is more of a user preference. I'll keep this here for people to reference, but I like the non-fixed look on mobile to allow for a nice clean reading experience.


All tests failing but celeri runs fine locally???

I added a few lines (118, 119, and 120) to celeri to add a few more fields to the dictionary meshes after it reads in data from the .json file that describes the meshes.  I get the error below reporting a failure related to the assignement of one of these variables...but...the entire thing runs just fine locally on my laptop?  I'm pretty confused. Thoughts @tbenthompson?
=================================== FAILURES ===================================
_____________________________ test_global_closure ______________________________
celeri.py:2184: in test_global_closure
command, segment, block, meshes, station, mogi, sar = celeri.read_data(
celeri.py:118: in read_data
meshes[i].smoothing_weight = mesh_param[i]["smoothing_weight"]
E   KeyError: 'smoothing_weight'
=========================== short test summary info ============================
FAILED celeri.py::test_global_closure - KeyError: 'smoothing_weight'
========================= 1 failed, 2 passed in 17.71s =========================

I got exactly the same error when running locally. When you tested locally, how were you running the code? To exactly replicate the CI system, you need to run pytest celeri.py celeri_closure.py (see this file for a definition of exactly what the CI runs): https://github.com/brendanjmeade/celeri/blob/37c8f8cd1aed1fcd412fccdcc86bfd44da1a1130/.github/workflows/test.yml#L28
I just created https://github.com/brendanjmeade/celeri/pull/41 to fix the issue!

Running pytest locally will help me a lot.  Thanks a lot not only for the fix but more importantly for your patience in teaching me how to listen and learn from the testing results!

That helps a lot.  Good to be learning!


Add sprint report 35

@ChrisBr here is the part about the request review view, probably for the bug section:
Fix request review view for role requests
When reviewing a request of type set_bugowner and add_role there was some information missing in the request review page, such as the user or group for which we were requesting the role, and in some cases, there was no information at all. That made really complicate the request role revision. How should I know if I want to accept a request if I do not know who will get the role? :confused: This have been fixed and now all the information is shown. We have even made clearer the difference between add_role bugowner and set_bugowner: :tada:

@Ana06 updated


Batch selection for tables

This function solves the problem (这个功能解决的问题)
It would allow selecting more than one element at once
Expected API (期望的 API)
It would be cool if when we press the Shift key, we can select multiples elements in the table.
We can take a look to the way Ant Design did it

This is the current behavior in Naive-UI

I know the feature. Some of antd table's features are implemented by me 😄.
We will do it.

Haha😄 @07akioni that's even better, I didn't know you're part of the Ant Design team.

Closing this since this was fixed in #1026


chore: add target to build arm64 preflight and use it in vagrant

I can add it to vagrant, but they doesn't work on darwin 😓

I can add it to vagrant, but they doesn't work on darwin 😓

You mean it doesn't work in a QEMU VM on darwin?

You mean it doesn't work in a QEMU VM on darwin?

it doesn't work outside QEMU 😅


Feature/multicast delegate

added multicast delegate for match delivery
tests improvements
updated readme
cleaned example (now I’m going to bring it back to correct API)

PR fixes added.


Fix the Broken How To Guides Links in API Docs

Purpose
This is to fix the 2 broken links of the How-To Guides linked from API Docs in [1] and [2].

[1] https://ballerina.io/v1-2/learn/api-docs/ballerina/websub/index.html
[2] https://ballerina.io/v1-2/learn/api-docs/ballerina/observe/index.html
Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22123 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #22123   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 0f0b160...ad3858c. Read the comment docs.

Fixes are sent to the "ballerina-1.2.x" branch via https://github.com/ballerina-platform/ballerina-lang/pull/22133.


refactor(workflow): separate workflow with filter for backend testing

🍰 Pullrequest
Separate the test workflow for backend.

[x] add file filter
[x] transfer all jobs from general test.yml to separate workflow file test_backend.yml
[x] use  the file change check job in test_backend.yml
[x] use file change check results as condition for job runs
[ ] adapt check requirements for master branch protection (@ulfgebhardt)
- set the workflow jobs Detect File Changes - Backend and Unit Tests - Backend as required
- remove Unit tests - Backend from the list of required checks

Issues

relates #2664

This PR is obsolete and can be closed.
All issues where covered in #2894 and #2962.


broken link

this link is broken in this page
https://github.com/Azure-Samples/media-services-dotnet-functions-integration/blob/master/media-functions-for-logic-app/LiveStreamAnalysis.md - receiving a 404

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

ID: 6358a2ad-74ae-3ec0-4039-9835c6b975e0
Version Independent ID: 8b136f7e-581f-9b49-7a05-d547a3447bf7
Content: Live stream analysis using Azure Video Indexer - Azure Video Indexer
Content Source: articles/azure-video-indexer/live-stream-analysis.md
Service: azure-video-indexer
GitHub Login: @Juliako
Microsoft Alias: Juliako

@vikramhn
Thanks for your feedback! We will investigate and update as appropriate.
I'm going to assign this to the document author so they can take a look at it accordingly

Hi, the linked worked for me, the supporting doc article has been removed. #please-close


Remove the "required" flag of the "--servers" parameter when creating a new AppGW address-pool

Description
Outline the issue here:
Could you please remove the "required" flag of the "--servers" parameter when creating a new AppGW address-pool (az network application-gateway address-pool create)?
When I try to create a new AppGW backend address pool to associate to a VMSS I need to create a new one with some value in the --servers parameter, then I need to clean the backendAddresses property (AppGW) to allow the VMSS association.
Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: apt-get
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here:
azure-cli (2.0.18)
acr (2.0.12)
acs (2.0.16)
appservice (0.1.17)
backup (1.0.1)
batch (3.1.4)
billing (0.1.5)
cdn (0.0.9)
cloud (2.0.8)
cognitiveservices (0.1.8)
command-modules-nspkg (2.0.1)
component (2.0.7)
configure (2.0.11)
consumption (0.1.5)
container (0.1.11)
core (2.0.17)
cosmosdb (0.1.13)
dla (0.0.12)
dls (0.0.14)
eventgrid (0.1.4)
extension (0.0.3)
feedback (2.0.6)
find (0.2.7)
interactive (0.3.10)
iot (0.1.12)
keyvault (2.0.11)
lab (0.0.11)
monitor (0.0.10)
network (2.0.15)
nspkg (3.0.1)
profile (2.0.13)
rdbms (0.0.7)
redis (0.2.9)
resource (2.0.15)
role (2.0.12)
servicefabric (0.0.4)
sql (2.0.12)
storage (2.0.16)
vm (2.0.15)
OS Version: What OS and version are you using?
Answer here: Windows 10
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: Bash on Windows

Hi @matiasma could you email me at alias trpresco. I would like to better understand your scenario as part of a larger look into how we can improve manipulating app gateways with VMSS.


Introduce FileSystemInterface, make helper create instance and call its methods

[x] Research existing implementations
[x] Introduce FileSystemInterface
[x] Create PHP implementation
[ ] Create dummy implementation, write tests
[ ] Make helper create instance and call its methods

Maybe we need simplest possible wrapper around "core" FS functions.
createDirectory = mkdir
writeFile = file_put_contents
existsFile = file_exists
...
etc
This way it will be easy to implement and fake.

The package is done: https://github.com/yiisoft/yii-filesystem/pull/1, need additional changes as listed in issue description.


Avoid adding transformation action to rule

The t action is being evaluated in rules and it's just a noop
2023/04/03 11:06:20 [DEBUG] Evaluating action tx_id="gFfpqMtpTnykAkSecSV" action="t"
2023/04/03 11:06:20 [DEBUG] Evaluating action tx_id="gFfpqMtpTnykAkSecSV" action="t"
2023/04/03 11:06:20 [DEBUG] Evaluating action tx_id="gFfpqMtpTnykAkSecSV" action="t"
2023/04/03 11:06:20 [DEBUG] Evaluating action tx_id="gFfpqMtpTnykAkSecSV" action="t"
2023/04/03 11:06:20 [DEBUG] Evaluating action tx_id="gFfpqMtpTnykAkSecSV" action="t"

It is a waste of resource and we should avoid adding the action to the rule if possible. Not mandatory though

IIUC, and I know @M4tteoP has dug into this and maybe more on top of it, disruptive and flow actions also have special semantics such as there being only one, which make it seem like they should be captured directly rather than as part of the action list. If that makes sense, it could be a cleanup that happens together with removing t from the list.

Wanna take a stab @jptosso ?


Travis builds are failing on Snyk test

@lirantal I'm not familiar with Snyk. Can you take a look at this?

@mleanos no worries, I'll work it out.


Blocker: user without permission 'users.can_see_name' could not log in

Without this permission all request to /rest/users/ result in 403 Forbidden. But the client needs to load the operator, so the request /rest/users/user/43 (my user has id 43) should not result in a 403 but all other requests (e.g. for user with id 1: /rest/users/user/1) should be forbidden. In fact that the client could not load the operator, I cannot log in.
Solution: Do not protect request to the own data.

I think this is fixed, isn't it?


TP 01 Pokemon swift finit

Given that I could not run the code I don't know if it does what it's meant to do.
It compiles without any problems (docker build .)

Why can't you run the code?

because wercker never worked on my computer
so all I could do was compile it, and it did

You can also change your Dockerfile and add swift build && swift test to run the tests that check some of your code.


[BE] Asset scale 2 to asset scale 9

[ ] Convert transactions with different asset scales by using Rafiki rate service (leveraging quote)
[ ] All transactions should take virtual balance into account

Changes required by #1357

Closed. See details here -  https://github.com/interledger/testnet/issues/1321#issuecomment-2323915072


Newlines and colors in NPC usernames

I can't seem to insert colors or newlines in NPC usernames.

You can only have one line. If you want more lines then you need to use a hologram.
This plugin is using MiniMessages for colors and formatting:

https://docs.advntr.dev/minimessage/format.html
https://webui.advntr.dev/

I also have a Hologram plugin and I will make it comatible with the NPC plugin, so that you can "link" a hologram with a NPC

Here you can test the feature to link a hologram with a NPC if you want
https://github.com/OliverSchlueter/FancyHolograms/pull/4


TF-M build failed tfm_s.axf

I'm trying to use golioth with dfu and after some warning caused by golioth I fixed it. But now on building phase I get this error that I don't know how to fix. I need support cause I'm in an hurry with my startup.
output.txt

Those errors look like an issue with "libc" (toolchain component) and not directly related to Golioth SDK. One thing that is concerns me is the "gnuarmemb" toolchain that you use. I'm not sure that is still supported with Zephyr/NCS. Is there a reason why you don't use Zephyr SDK toolchain (https://github.com/zephyrproject-rtos/sdk-ng)?


Setup com.unity.burst as a package dependency

Currently, the com.unity.burst package isn't actually configured as a dependency of the package.json file.
That means that it won't automatically add it when you install it, the user would have to do so manually.

Fixed in 23ce6ac46


Fixes #85

XMl view now shows XML for resultnodes if available

Closing this one, as a new (identical) one has been made on nunit-gui


Feature/add optional class names to hexen

Hi Hannu,
Please see the attached PR.  I've updated the HexShape class so that if you provide a className in the Hex properties it will carry that through and apply it as an additional className within the g element of the SVG.
I manually constructed my board as follows
constructBoard() {
  return [
    new Hex(0,  1, -1, {className: 'edge'}),
    new Hex(0,  2, -2, {className: 'edge'}),
    new Hex(1,  0, -1, {className: 'outer'}),
    new Hex(1,  1, -2, {className: 'edge'}),
    new Hex(1,  2, -3, {className: 'outer'}),
    new Hex(2, -1, -1, {className: 'inner'}),
    new Hex(2,  0, -2, {className: 'outer'}),
    new Hex(2,  1, -3, {className: 'outer'}),
    new Hex(2,  2, -4, {className: 'inner'}),
    new Hex(3, -1, -2, {className: 'inner'}),
    new Hex(3,  0, -3, {className: 'outer'}),
    new Hex(3,  1, -4, {className: 'inner'}),
    new Hex(4, -2, -2, {className: 'start'}),
    new Hex(4, -1, -3, {className: 'inner'}),
    new Hex(4,  0, -4, {className: 'inner'}),
    new Hex(4,  1, -5, {className: 'start'}),
    new Hex(5, -2, -3, {className: 'start'}),
    new Hex(5, -1, -4, {className: 'inner'}),
    new Hex(5,  0, -5, {className: 'start'}),
    new Hex(6, -2, -4, {className: 'centre'}),
    new Hex(6, -1, -5, {className: 'centre'}),
    new Hex(7, -3, -4, {className: 'start'}),
    new Hex(7, -2, -5, {className: 'inner'}),
    new Hex(7, -1, -6, {className: 'start'}),
    new Hex(8, -4, -4, {className: 'start'}),
    new Hex(8, -3, -5, {className: 'inner'}),
    new Hex(8, -2, -6, {className: 'inner'}),
    new Hex(8, -1, -7, {className: 'start'}),
    new Hex(9, -4, -5, {className: 'inner'}),
    new Hex(9, -3, -6, {className: 'outer'}),
    new Hex(9, -2, -7, {className: 'inner'}),
    new Hex(10, -5, -5, {className: 'inner'}),
    new Hex(10, -4, -6, {className: 'outer'}),
    new Hex(10, -3, -7, {className: 'outer'}),
    new Hex(10, -2, -8, {className: 'inner'}),
    new Hex(11, -5, -6, {className: 'outer'}),
    new Hex(11, -4, -7, {className: 'edge'}),
    new Hex(11, -3, -8, {className: 'outer'}),
    new Hex(12, -5, -7, {className: 'edge'}),
    new Hex(12, -4, -8, {className: 'edge'})
  ]
}

and using the Sass as follows:
svg {
  g {
    fill: #00AFB0;
    fill-opacity: 0.6;
    &:hover {
      fill-opacity: 1;
      text {
        fill-opacity: 1;
        fill: #fff;
      }
    }
    polygon {
      stroke: #3F51B5;
      stroke-width: 0.2;
      transition: fill-opacity .2s;
    }
    text {
      font-family: monospace;
      font-size: 0.15em;
      fill: #000;
      fill-opacity: 0.9;
      transition: fill-opacity .2s;
    }
    &.edge {
      fill: #ede2fe;
    }
    &.outer {
      fill: #eafec9;
    }
    &.inner {
      fill: #bbfee2;
    }
    &.start {
      fill: #80b281;
    }
    &.centre {
      fill: #a5ffe5;
    }
  }
  path {
    fill: none;
    stroke: hsl(60, 20%, 70%);
    stroke-width: 0.4em;
    stroke-opacity: 0.3;
    stroke-linecap: round;
    stroke-linejoin: round;
  }
}

with the following result.

I also fixed a typo in the README and took the liberty of tightening up the use of some vars, lets, and consts

Update: I figured out the scaling issue I was having and have updated this PR to also support the optional supplying of a viewBox property to the HexGrid component.

Hey!
Awesome job man! Looks great, and I like the code style. Looks very much something that I would write. All the vars and stuff looks like from time when I was still getting into ES6 :D
Do you think classnames dependency is necessary? I guess it doesn't add too much size to the library but considering if it's used only for that, maybe just use something more manual for now. Thoughts?

Hey Hannu,
The ClassNames lib is tiny and worth including I feel. If you really don't want it I'm okay to remove it however.
Cheers
Dave
Sent from my iPhone

On 13 Feb 2017, at 4:36 pm, Hannu Kärkkäinen<EMAIL_ADDRESS>wrote:
Hey!
Awesome job man! Looks great, and I like the code style. Looks very much something that I would write. All the vars and stuff looks like from time when I was still getting into ES6 :D
Do you think classnames dependency is necessary? I guess it doesn't add too much size to the library but considering if it's used only for that, maybe just use something more manual for now. Thoughts?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That's true, can probably use it for other purposes later on as well. I'll go ahead and merge this, thanks again!
Was there something else you were still working on or having problems with?

Awesome.  Let me know when the new version hits npm — I may do some more PRs as I find stuff I need for my game. Thanks for accepting the PR

There's a new version now in NPM, including your changes and some other internal refactoring. Also one more example in the repo


enable raft in coordinator

This merely enables raft replication of coordinator fsm.
The subsequent PRs will add support for the below

Client discovery of coordinator leader
Concurrent map for coordinator.

@varks Please also take care of conflicts.

@renga86 @supriya-premkumar @alex8937 guys take a look if any of you are online.


Ordered list number clipping

PHP Version: 5.6
MySQL Version: 5.6
Expected Behaviour: Ordered list items are expected to show in full to user
Actual Behaviour: Numbers greater than 10 have some slight clipping to the left

I'll admit, I got a little carried away with the squiggles :)

Thank you for finding this issue. It is now fixed as of release v0.12.2.


[weak-refs] UAF in cli-generated JS shims when dealing with by-value receivers

Describe the Bug
When the --weak-refs feature is enabled, the JS shim code that wasmbindgen-cli generates (out_dir/libname_bg.js file) is incorrect and leads to UAFs (e.g., double free) when a decorated [wasm_bindgen] method uses a by-value receiver.
The very short technical description of the bug is that in the JS glue wrapping such a method is missing a call to ThatClassFinalization.unregister(this);.
Steps to Reproduce

//! src/lib.rs
use ::wasm_bindgen::prelude::*;

#[wasm_bindgen]
pub struct Foo { /* … */ }

#[wasm_bindgen]
impl Foo {
    #[wasm_bindgen(constructor)]
    pub fn new(/* … */) -> Self {
        Self { /* … */ }
    }

    pub fn identity(self) -> Self {
        self
    }
}

# Cargo.toml
[lib]
crate-type = ["cdylib"]

[package]
name = "example"
version = "0.0.0"
edition = "2018"

[dependencies]
wasm-bindgen = "0.2.70"

cargo build --target wasm32-unknown-unknown
wasm-bindgen --out-dir out_dir --out-refs ./target/wasm32-unknown-unknown/debug/example.wasm

The generated out_dir/example_bg.js will then contain, among other things:
const FooFinalization = new FinalizationRegistry(ptr => wasm.__wbg_foo_free(ptr));
/**
*/
export class Foo {
    static __wrap(ptr) {
        const obj = Object.create(Foo.prototype);
        obj.ptr = ptr;
        FooFinalization.register(obj, obj.ptr, obj);
        return obj;
    }

    free() {
        const ptr = this.ptr;
        this.ptr = 0;
        FooFinalization.unregister(this);
        wasm.__wbg_foo_free(ptr);
    }
    /**
    */
    constructor() {
        var ret = wasm.foo_new();
        return Foo.__wrap(ret);
    }
    /**
    * @returns {Foo}
    */
    identity() {
        var ptr = this.ptr;
        this.ptr = 0;
        var ret = wasm.foo_identity(ptr);
        return Foo.__wrap(ret);
    }
}

Now, somewhere within JS code interacting with this wasm library, do:
let foo = new Foo();
let foo2 = foo.identity();

Expected behavior
Everything is fine
Actual behavior
Some use-after-free or double-free error occurs, such as a borrow_mut() error, once garbage collection disposes of both foo and foo2.
Screenshot of a debugged situation
The following is not exactly a screenshot of the minimal repro, but rather, of a personal debugging session that was conducted to identify the cause of the borrow_mut error. In that debugging session, all WasmRefCells were monkey-patched to log whenever a borrow occurred or one ended, with a global ever incrementing "uid" to distinguish between each WasmRefCell instance:

As you can see, we have a wbg_…_free called twice on the same instance #4.

Explanation
When foo is GC-ed, wasm.__wbg_foo_free(ptr) will be called by the Finalization registry, but that ptr was actually given in an owned fashion to  wasm.foo_identity(ptr), which deallocates that ptr when it does Box::from_raw:

Code for wasm.foo_identity
/// `wasm.foo_identity`
pub extern "C" fn __wasm_bindgen_generated_Foo_identity(
    me: u32,
) -> <Foo as wasm_bindgen::convert::ReturnWasmAbi>::Abi {
    let _ret = {
        // let me = unsafe { *Box::from_raw(me) }; /* frees `me` */
        let me = unsafe { <Foo as wasm_bindgen::convert::FromWasmAbi>::from_abi(me) };
        me.identity()
    };
    <Foo as wasm_bindgen::convert::ReturnWasmAbi>::return_abi(_ret)
}

Solution
Do the same thing that the glue for JS free currently does (in that regard, free() is not that special, it could simply be the glue generated for a dummy drop-ping method, such as impl Foo { pub fn free (self) {} }.
Fix implemented in #2448

This fix https://github.com/rustwasm/wasm-bindgen/pull/2448 only fix self consumed case not the parameter's. For example if you have
#[wasm-bindgen]
pub fn test(foo: A){...}

foo will used after free when --weak-refs config is on.
Should we reopen this issue or file another one?


Update HeightmapExtensions.cs

Added some more useful Heightmap extensions, e.g., like making a texture or mesh from the heightmap

Is this a duplicate PR? If not, could you please redo it? You'll need to merge our current master (where I've already merged your previous PR) and then create a new PR.
For changes to just one file, ideally, it should be limited to one PR at a time.
However, if you're making changes to multiple different files, then it's okay to create multiple PRs.
Thank you.

It was duplicate, sorry. We should be good,

Super, once again, thank you.
Closing, as this is a duplicate to https://github.com/stride3d/stride-community-toolkit/pull/69.


test: mark attestation_types::verify::sample test only

Signed-off-by: Harald Hoyer<EMAIL_ADDRESS>

ping


refactor(project_detail_right.html): changing post commit hook url to…

closes #1535

Thanks :)

:+1:


Property class should be dropdown

Only a few classes are used to classify properties in Wikidata. It would be useful to have a dropdown for them rather than an auto completing free text input.

Done by #72


⚠ Bofrost.de has degraded performance

In a4e4c84, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 885 ms

Resolved: Bofrost.de performance has improved in 05abe9e.


tpnam vs mkstemp

When I compile the tests, I get a warning that tpnam is dangerous and mkstemp should be used instead.

This may not be fixable due to the restrictions of the library.


Dangling USB File Descriptor if Program is Not Terminated Properly

Summary
We are seeing dangling USB file descriptors when using the USBTransport if the program is not terminated properly. We believe this is because the USB resources are not disposed if .close() is not called on the transport when the program is terminated abnormally (say canceled by a test runner):
        async def close(self):
            self.source.close()
            self.sink.close()
            await self.source.terminate()
            await self.sink.terminate()
            self.device.releaseInterface(self.interface)
-->         self.device.close()
            self.context.close()

            # Wait for the thread to terminate
            await self.event_loop_done

Error
ResourceWarning: unclosed <socket.socket fd=20, family=1, type=1, proto=0>

Workaround
Currently, we have our code setup to explicitly call transport.device.close() our consuming classes's destructor, __del__. Would it be possible to put that into USBTransport's destructor, so that if .close() is not called, the USB resource is still freed?

All transport classes, including UsbTransport implement an async context manager, which you should use in order to guarantee that the resources will be free'd. Using the async context manager is preferred to explicitly calling close().
A typical use for a transport goes like this:
async with open_transport(...):
   ...

When the code in the async with block ends, the transport will be closed.


ld: unknown option: -soname clang: error: linker command failed with exit code 1 (use -v to see invocation)

get an error on mac pro during installation of PHP-CPP
sudo make
ld: unknown option: -soname
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [libphpcpp.so.2.0.0] Error 1

Just replace
-Wl,-soname,libphpcpp.so.$(SONAME)

with
-Wl,-install_name,libphpcpp.so.$(SONAME)

in Makefile.

@sjinks  thanks this solved my error also!!

I think this is solved with PR #391


configuring the render pipeline and graphics

switching to the universal render pipeline (URP) and adjusting some graphics

done


ItemList.unselect() should be ItemList.deselect()

Godot version:
3.1.1

Issue description:
ItemList.unselect() is a very strange name. ItemList.deselect() is easier to find.
We have all these deselects:

And only this unselect:

Good point. I've added it to the list in #16863.

Maybe even rename deselect_all to match this:


Push manifest doesn't include custom chunks

Description:
Push manifest does not contain chunks created by using webpack's dynamic imports (import()) most likely by this line.
TBH I have no idea how to fix this or even if we should fix this.
Obviously we don't want to include chunks used by specific routes on every route. Maybe looking into webpack dependency graph and including them based on their parent (e.g. if it was bundle.js or route-*.js will do.
Steps to reproduce:

preact create app
cd app
echo "console.log('bar')" > ./src/foo.js
echo "import('./foo.js')" >> ./src/index.js
preact build --no-prerender
cat ./build/push-manifest.json

Generated chunk isn't there (should be called 2.chunk.*.js)

TBH I have no idea how to fix

It happends because preact-cli only includes chunks that starts with route- https://github.com/developit/preact-cli/blob/60ba2ae81f539978309539bdf6a77ad25435ccdd/src/lib/push-manifest.js#L7

The dep graph could be a really cool way of doing this.  Right now we have no way to know that those chunks correspond to a given URL.


chore: revert back to Mocha 4

Mocha 5+ dropped support for IE 8 and IE 9. This leads to failing
BrowserStack tests, so reverting back to version 4 until Karma decides
to drop support for the mentioned browsers.

The failed tests look like flakes... Will re-trigger the build.


Relay Server Minor Issue (Cron job may skip)

Over the weekend, monitoring identified one situation the AWS_UPLOADER cron job skipped once.
By design, it is auto-recoverable: the next cron job will first take care the previous one's workload and then the current.
Still, worth verifying:

[ ] The Data is intact
[ ] The cron job skipping is just one off (not due to some systemic issue. )

FYI @ian-r-rose @ZhenyuZhu-Caltrans

Can this issue be closed or moved to a sprint?

Closing as resolved, thanks @pingpingxiu-DOT-ca-gov


aio -h command does not exist

Expected Behaviour
https://developer.adobe.com/runtime/docs/guides/tools/cli_install/ and https://developer.adobe.com/runtime/docs/guides/getting-started/setup/ mention use of aio -h
aio -h should return help documentation
Actual Behaviour
aio -h not found
Reproduce Scenario (including but not limited to)
Steps to Reproduce
aio -h
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

the 2 prs resolve this issue, please merge them and close this

I don't have write access so cannot merge the PR

Yeah, sorry @alexdkaminski I should have been more clear. The message was intended for someone who does have write access ( I don't either )
Thank you for your commits!

Done, thanks!


No getSignUpUrl import?

Hi,
The doc mentions getSignInUrl and getSignUpUrl, but it seems that getSignUpUrl is not available to import.
Is it normal?

This has just been released, install version v.0.5.0 and you should see getSignUpUrl.

I got the v0.5.0 and it seems the module is broken:
Cannot find module '@workos-inc/authkit-nextjs' or its corresponding type declarations.

Sorry about that, our new github action was missing a step. Version v0.5.1 should work correctly.

You most likely don't have WORKOS_REDIRECT_URI set in your environment variables. See the readme for further instructions.


tembo-operator: Add support for Azure Backup and Restore

Add support for Azure backup and restore in tembo-operator. Pulling conductor changes into a separate PR.
Note: There is a failing conductor test that will pass once we merge the conductor changes into this branch.

We need to add this spec value to the Cluster resource for Azure deployments:
  inheritedMetadata:
    labels:
      azure.workload.identity/use: "true"

I'll follow up with this shortly.


add(examples) scripting example

add a scripting example
move the standalone app to test apps

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1561

Relevant Lines:
2855

💛  - Coveralls


Support filter/2 in WindowAPI

Excellent. Can you please add a basic test here? https://github.com/elixir-ecto/ecto/blob/master/test/ecto/query/builder/windows_test.exs - please make sure the tree at is failing before the fix and happy holidays!

@josevalim I don't know what test should be in test/ecto/query/builder/windows_test.exs file because I can't find similar tests there, so I added it to test/ecto/query/builder_test.exs

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


please make the lowest sponsoring tier as low as $2

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Description
i want to support you, but asking for at least $5/month is too much for me, can you please make the lowest tier to be $2/month?

didn’t even notice when you’ve done this but thank you @hrsh7th!


👉 Visualise Live Objects

👉 Is your feature request related to a problem? Please describe.
This task is related to User Story #177
It is important for users to quickly and easily see which WELLS / WELLBORES and LOGS which are currently live streaming so they can get an overview and work on operations with live data.
Describe the solution you'd like
A visual indicator next to the objects to indicate they are currently live streaming, similar to what is seen in the picture below.
When a log object is live streaming, the relevant wellbore and well should also be indicated in the same way.

Describe alternatives you've considered
Currently no alternatives, but open to suggestions
Additional context

Will discuss with UI designer asap, will notify you.


publish<EMAIL_ADDRESS>

Requested by: @chadwhitacre
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[x] docker[release]
[x] docker[latest]
[x] pypi
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

Reviewed craft publish code, I believe I can reaccept this ticket, craft publish will run again, it will find the existing branch, the branch has red CI (right? why did this not block it before? are the failed jobs non-required? regardless, I restarted the failed jobs (yay this)) ... with green CI craft publish will proceed with only unfinished targets per checkboxes, i.e., GitHub. Should be fine? 🤞

From internal conversation FE tests are known flakey, maybe we made them optional? I've canceled the test runs and am going to reaccept here.

Dope.

Confirmed behaved as expected. 👍
https://github.com/getsentry/publish/runs/5561413621?check_suite_focus=true

Also confirmed that https://github.com/getsentry/craft/pull/343 saved our 🥓 again, there was a draft release.


potentially harmful notation

p as argument of dynamics function overlaps with p as the first subset of state vector
https://github.com/JinraeKim/FlightSims.jl/blob/8ae93b0030ac73d8c36fbcc0f44d91760f86cf95/src/environments/integrated_environments/backstepping_position_controller_static_allocator_multicopter_env.jl#L30-L38

Thanks :)
I'll fix it.


Custom key for files

Is there a way to pass a custom key to addFileToFormData?
https://github.com/danialfarid/ng-file-upload/blob/master/src/upload.js#L194
I would like to be able to set it based off of the file index like so:
addFileToFormData(formData, config.file[i], 'upload[files]['+i+']');

You can have:
Upload.upload({...
  file: {'upload[files][` + i + ']': files[i]},
  ...
})

@danialfarid Thanks!


The Frank!Doc may not build without access to our Nexus

I tried to build the Frank!Doc on my NN laptop. I am doing without a settings.xml file. The build failed with the following message:
[ERROR] Failed to execute goal on project frank-doc: Could not resolve dependencies for project org.ibissource:frank-doc:jar:1.1-SNAPSHOT: Could not find artifact com.sun:tools:jar:1.8 in central (https://repo.maven.apache.org/maven2) -> [Help 1]

We have to investigate this issue, because WeAreFrank! is an open-source company. Anyone should be able to build the Frank!Doc.
I have checked that  our Nexus holds artifact com.sun.tools:jar:1.8. As a solution, we may document how to add our Nexus to Maven's settings.xml file before building the Frank!Doc. We may also fix the build by adding our Nexus repository to the Frank!Doc pom.xml.

Solved by #166


draft: Add support for OpenCover format based on opencover plugin

This is on on-going process to implement the opencover  format to coverage-model
Only need to convert the parser from https://github.com/jenkinsci/opencover-plugin/blob/main/src/main/java/io/jenkins/plugins/opencover/OpenCoverReportAdapter.java#L37
Originally raised here on the deprecated code-coverage-api-plugin
Testing done

### Submitter checklist
- [ ] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [ ] Ensure that the pull request title represents the desired changelog entry
- [ ] Please describe what you did
- [ ] Link to relevant issues in GitHub or Jira
- [ ] Link to relevant pull requests, esp. upstream and downstream changes
- [ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Let me know if you need some help here. I created a similar PR for a new parser of JUnit results, see https://github.com/jenkinsci/coverage-model/pull/53 for details.

@uhafner Thanks probably just missing time. Also I'm not very familiar yet with the model

Closing this PR. Will open a new clean one in the next few days.
Only need to finish method complexity parsing and enrich test coverage.


Do not attempt to start the server when usedPortAction is ignore and isPortTaken is true

Summary
I have a dev server already running and used usedPortAction: 'ignore' but the server is still attempted to be started despite the port showing as being taken. This patch puts a fork in the code to not attempt to start the server if ignore is used and isPortTaken is true.
Test plan
Start a server on a given port. Then attempt to use the same port in jest-dev-server config using usedPortAction: 'ignore'. Without the patch, the code will throw an EADDRINUSE error. This is because it will still attempt to run the given command. With the patch, it will simply run the tests.
Workaround
For now, I'm just trapping the EADDRINUSE error for my command (in this specific case it is an express.js server):
  command: `node -e "server = require('./server');server.listen(${config.get('ports.server')}).on('error', (err) => { if (err.code !== 'EADDRINUSE') throw new Error(err) })"

Hello @brandonaaron, thanks for this, I think it is great but it adds a lot of complexity in the code. @xiaoyuhen do you have ideas to refactor it?

😢I can't think of a better solution, maybe and some comments will be easy to understand.

@xiaoyuhen OK then, we can merge it.


feat: reimplement read-action-input in bash instead of node

This PR reimplements the utility function read-action-input in bash instead of node

For v3, the file will be replaced in:

https://github.com/reproducible-containers/buildkit-cache-dance/pull/25


Investigate/implement better permissions system

Slash commands come with their own granular permissions system. At some point it might be worth implementing.
https://discordjs.guide/interactions/slash-command-permissions.html

Can you elaborate what you mean here?


Execute transactions on DEX with solana-py

Is it possible to execute swaps using solana-py? If yes, can someone please point me to an example of a working code?

Which dex? Might look into using https://github.com/kevinheavey/anchorpy

@michaelhly thanks for the reference! I was looking into Serum, for that I found pyserum (https://github.com/serum-community/pyserum) but there it's not possible to execute swaps. Hence I'm looking for an alternative.
I believe anchorpy could be a candidate, since I can interact directly with the contract, however if there was a wrapper for that it would be a dream.


[docs] Enhance container selection in docker dump

The problem with the previous query was, that it sometimes selected multiple containers, which make the command respond with a hard to understand error message.
With this regular expression only the container is selected with the exact name.

Codecov Report

Merging #14292 (5d0cf17) into master (d989247) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14292      +/-   ##
==========================================
- Coverage   41.78%   41.78%   -0.01%
==========================================
  Files         743      743
  Lines       79464    79464
==========================================
- Hits        33207    33206       -1
- Misses      40792    40794       +2
+ Partials     5465     5464       -1

Impacted Files
Coverage Δ

modules/git/tree_nogogit.go
60.86% <0.00%> (-8.70%)
:arrow_down:

modules/charset/charset.go
68.53% <0.00%> (-4.50%)
:arrow_down:

modules/git/repo_language_stats_nogogit.go
57.44% <0.00%> (-4.26%)
:arrow_down:

modules/indexer/stats/db.go
56.00% <0.00%> (-4.00%)
:arrow_down:

models/unit.go
46.57% <0.00%> (-2.74%)
:arrow_down:

services/pull/pull.go
43.06% <0.00%> (+0.49%)
:arrow_up:

modules/queue/workerpool.go
60.00% <0.00%> (+1.22%)
:arrow_up:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

modules/git/utils.go
80.55% <0.00%> (+2.77%)
:arrow_up:

modules/git/repo_base_nogogit.go
72.72% <0.00%> (+9.09%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #14292 (5d0cf17) into master (d989247) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #14292      +/-   ##
==========================================
- Coverage   41.78%   41.78%   -0.01%
==========================================
  Files         743      743
  Lines       79464    79464
==========================================
- Hits        33207    33206       -1
- Misses      40792    40794       +2
+ Partials     5465     5464       -1

Impacted Files
Coverage Δ

modules/git/tree_nogogit.go
60.86% <0.00%> (-8.70%)
:arrow_down:

modules/charset/charset.go
68.53% <0.00%> (-4.50%)
:arrow_down:

modules/git/repo_language_stats_nogogit.go
57.44% <0.00%> (-4.26%)
:arrow_down:

modules/indexer/stats/db.go
56.00% <0.00%> (-4.00%)
:arrow_down:

models/unit.go
46.57% <0.00%> (-2.74%)
:arrow_down:

services/pull/pull.go
43.06% <0.00%> (+0.49%)
:arrow_up:

modules/queue/workerpool.go
60.00% <0.00%> (+1.22%)
:arrow_up:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

modules/git/utils.go
80.55% <0.00%> (+2.77%)
:arrow_up:

modules/git/repo_base_nogogit.go
72.72% <0.00%> (+9.09%)
:arrow_up:

Continue to review full report at Codecov.

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

@openscript Any update?

@lunny I'll check it today and give some feedback :)

@lunny I can verify that it works without the quotes. This is I usually do my backups:

Create a new backup  docker exec -u git -it -w /tmp $(docker ps -qf name=^/gitea$) bash -c '/app/gitea/gitea dump -c /data/gitea/conf/app.ini -f gitea-backup.zip'
Copy backup  docker cp $(docker ps -qf name=^/gitea$):/tmp/gitea-backup.zip .
Rename backup file  mv gitea-backup.zip gitea-backup-$(date "+%Y%m%d-%H%M%S").zip

Maybe this section in the documentation can be enhanced further. When no file name for the dump is specified, a timestamp will be used. So you need to go into the container to find the actual file name.

Without quotes it is shell dependent. Bash works fine but zsh definitely will not.

I think @delvh 's suggestion is pretty good. So, approve and move on?


run R_ProcessEvents on Windows

it should fix #104

Yes, that fixes things on my end! Thanks!

I am a bit surprised that no one else has reported this issue until now...perhaps only a handful of people are using RCall on Windows.

Coverage decreased (-1.5%) to 72.919% when pulling 342e756f776e70f117292f6a8c25110ae0c283bf on fix#104 into 5dcd2923641d2295571524c30665184509f218b0 on master.

Well, to be honest, I'd be surprised if there are more than a few users overall, the lack of a "how to use this package" like documentation makes it almost impossible to get started with the package...

The failure is not relevant to the PR.

@randy3k or @simonbyrne could you tag a new release that incorporates this fix? Thanks!

I think it makes sense to release a patch for this. But I would like to wait and see @simonbyrne has anything to add to the next release.

Not soon, go ahead and tag it.
On 29 Apr 2016 7:27 pm, "Randy Lai"<EMAIL_ADDRESS>wrote:

I think it makes sense to release a patch for this. But I would like to
wait and see @simonbyrne https://github.com/simonbyrne has anything to
add to the next release.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/JuliaStats/RCall.jl/pull/106#issuecomment-215839517

@davidanthoff
Will release it when #107 is confirmed to be fixed.


Provide a vcpkg argument to list the dependency tree of a package

It would be useful to be able to get the full dependency tree for a package, in either a "flat" string list (such as brew does with brew deps [package_name]) or a "tree" view (such as brew does with brew deps --tree [package_name]).

Did you try vcpkg depend-info?
It even has a --dot option.

Please use ./vcpkg depend-info ${PORT_NAME} to show its dependency tree.

Thanks a lot, I don't know why I didn't figure it out while using the help.

The depend-info shows either forward or backward dependencies. I expected the command to provide only dependencies the ${PORT_NAME} relies on (for instance, vcpkg depend-info grpc shoud list protobuf, abseil and so on, but not the ports that depend on grpc). I need a command to get the ${PORT_NAME} dependencies list, not the list of the ports depending on ${PORT_NAME}

for instance, vcpkg depend-info grpc shoud list protobuf, abseil and so on, but not the ports that depend on grpc

I don't see a single port which depends on grpc:
$ ./vcpkg depend-info grpc
abseil:
c-ares:
openssl:
protobuf:
re2:
upb:
zlib:
grpc: abseil, c-ares, openssl, protobuf, re2, upb, zlib

I used the grpc port as an example of the feature I expect. I could have used boost-detail for instance.
"depend-info" on boost-filesystem provides the dependencies in reverse order, with each sub-dependencies having its own dependency line and with some dependencies appearing twice or more (see boost-vcpkg-helpers below) :
./vcpkg depend-info boost-detail boost-uninstall:  boost-vcpkg-helpers: boost-uninstall boost-compatibility: boost-vcpkg-helpers boost-config: boost-compatibility, boost-vcpkg-helpers boost-preprocessor: boost-vcpkg-helpers boost-detail: boost-compatibility, boost-config, boost-preprocessor, boost-vcpkg-helpers
I expect to have the full flat dependency list with each dependency appearing only once for the package i.e. (an argument -flat to depend-info should be a good option) :
$ ./vcpkg depend-info -flat boost-detail boost-compatibility, boost-config, boost-preprocessor, boost-vcpkg-helpers, boost-uninstall

Not a -flat option, but:
$ ./vcpkg depend-info --max-recurse=0 boost-detail
boost-detail: boost-compatibility, boost-config, boost-preprocessor, boost-vcpkg-helpers

Also useful:
$./vcpkg install --dry-run boost-detail
Computing installation plan...
The following packages will be built and installed:
  * boost-compatibility[core]:x64-mingw-dynamic -> 1.75.0
  * boost-config[core]:x64-mingw-dynamic -> 1.75.0
    boost-detail[core]:x64-mingw-dynamic -> 1.75.0
  * boost-preprocessor[core]:x64-mingw-dynamic -> 1.75.0
  * boost-uninstall[core]:x64-mingw-dynamic -> 1.75.0
  * boost-vcpkg-helpers[core]:x64-mingw-dynamic -> 7#2
Additional packages (*) will be modified to complete this operation.

$./vcpkg install --dry-run boost-detail

Or rather:
./vcpkg install --dry-run --x-install-root=/tmp/empty boost-detail

This issue should be fixed in latest release.


ISIS-1364: Declare and implement persistIfNotAlready in RepositorySer…

…vice and RepositoryServiceDefault

Oscar and I have been discussing on<EMAIL_ADDRESS>think that this isn't required, because RepositoryService#persist(...) has these semantics already.  Oscar will update the docs to make this clearer, and (I believe) just close this PR.


Feature: Supports fine-grained (option-level) declarative configuration (overrideConfig)

Currently, overrideConfig is a global option, which means that I either enable it for all options or disable it for all options. However, there are some special options in KDE, or rather, they are not options but the state of the software, which may only apply to specific machine and not to other machines (if we use home-manager or plasma-manager, I believe in many cases we would like our configurations to be transferable between multiple different devices). Examples of such "options" include:
"kdeglobals"."KScreen"."ScreenScaleFactors" = "HDMI-1\x3d1.5;DP-1\x3d1.5;HDMI-2\x3d1.5;DP-2\x3d1.5;HDMI-3\x3d1.5;";，It appears that this option enumerates all display ports for the device, but when I apply these configurations on different devices, these options are definitely no longer applicable. At this point, I hope to specify these options individually as variable state, while maintaining the declarative nature of the overall configuration.
One possible implementation: Introduce a special value, such as keepState, where specifying "kdeglobals"."KScreen"."ScreenScaleFactors" = "keepState" will cause the plasma-manager to retain its previous state. This keepState might simply be something like { _special = "keepState"; }.

I believe this is already supported. You should for example be be able to use:
"kdeglobals"."KScreen"."ScreenScaleFactors".persistent = true;

Is that what you're looking for?

Is that what you're looking for?

Thank you, this looks promising,  so where can I find the related document/manual ?

There's no manual at the moment (we probably should add one), but to see the options which are supported for the files module take a look at advancedSettingsType. For other cases I would suggest looking at the examples directory, but I don't think there are any examples for persistence there yet (I can add one though).


Run Tye with Dapr

I have a query regarding the usage of Tye with Dapr. Can both be used to together, because to run Tye, first we have to run Dapr runtime in the background for service discovery to work. So how can we debug application with Tye, if we use it with Dapr?
Any thoughts or suggestions on the same, would be great.

When working with Dapr and Tye. When your run your applications using tye run --dashboard you can see the sidecars that are also spun up by Tye for you. Take a look at this guide here.
Simply invoke your sidecars using the DaprClient via either the DaprInvokeMethodAsync or by creating a DaprClient based on the appId of the microservice you need to access.
For Example let's say you have two microservices: app1 and app2. In order to invoke a call from app1 to app2 you can do something similar.
var client = DaprClient.InvokeHttpClient("app1");
var requestData = await client.GetAsync('endpoint/path');

If you spin up the Zipkin extension with tye, and make an invocation request to one of your microservices using the Service-to-service methodology. You will see Zipkin will log the route call. When inspecting the route call you will notice it will use the invoke method endpoint to access your Dapr sidecar.
By using the Microsoft.Tye.Extensions.Configuration package for Service Discovery. Tye will help you find the ports to your Sidecars based on the AppId you gave your service in the Tye.yaml configuration file.
The documentation is pretty extensive and well documented.


Fiat-to-Crypto Onramp/Offramp

Everything to do with integrating with onramps which provide access to fiat-to-crypto tranfers, eg Wyre and Simplex

Closing as issue is outdated


Change command recommended for building .wasm files for deployment

What
Change command recommended for building .wasm files for deployment
Why
...

Preview is available here:http://soroban-docs-pr476.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr476.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr476.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr476.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr476.previews.kube001.services.stellar-ops.com/


0/1 Knapsack problem - Subset Sum Problem

Description
I want to add a 0/1 Knapsack application problem using the top-down approach in the Dynamic Programming Folder
Problem:
Subset Sum Problem
Problem Description:
In this problem, we will be given a set of non-negative integers and a value sum. We have to determine if there is a subset of the given set with a sum equal to the given sum.
I will solve this using dynamic programming.
Example:
Input: set[] = {3, 30, 4, 15, 5, 2}, sum = 9
Output: True
There is a subset (4, 5) with sum 9.
Input: set[] = {3, 34, 4, 11, 5, 2}, sum = 30
Output: False
There is no subset that add up to 30
Domain
CPP
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

I want to work on this issue as a part of GSSOC'22

Kindly just work on a single question, as per the rules I cant allow you to do all of them in a single issue so please update the title and I will assign it to you.

Kindly just work on a single question, as per the rules I cant allow you to do all of them in a single issue so please update the title and I will assign it to you.

okay I have made the changes

I have added a description


Add<EMAIL_ADDRESS>as dependency on npm

It'd be really nice if you could document that version 0.6.5 and greater require react-dom v0.14.0. This way if you're using React v0.13.0 you know to use <= 0.6.4.

It'd also be nice if future breaking changes like this didn't happen in a minor release version bump.
Thanks for everything you've done with the library! I don't mean to sound unappreciative, just wanted to provide feedback :smile:

Hey, my bad - you're totally right. I sadly can't fix it (I can't republish on npm), but thanks for telling me that I messed up semver - I'll add a line to the readme!

Actually, I believe that the current solution is okay - I unpublished 0.6.5 on npm, meaning that ^0.7 needs React-Dom, while ^0.6 doesn't. Again, thanks for letting me know!

Excellent! Thank you


Included status information of costs (active / inactive)

This PR includes the cost status as proposed in #720.
As mentioned in previous pull request, the idea behind of this is to have a global mechanism allocation.
Additionally, this PR includes unittest code for cost sum.
Note the changes of this PR are still back compatible with previous code.

The additional commits include:

The concept of nr, nc and ni total for global allocation
Cost sum data doesn't inherit from cost data abstract.
Multiple contact / impulse datas don't inherit from contact / data data abstract.
Unittest code for nr, nc and ni total.
Allowed to register copy of cost sum, multiple contact and impulse classes (Python bindings)


Cannot find the commits in an open PR to a private repo

Extension version:
VSCode Version: 1.56.0
OS: Windows

Steps to Reproduce:

Goto the extension
Click on the All Open option to open all the open PRs
Click on the expand/collapse button
All commits should show up, but its empty

Currently the commits do not show up in the tree until you have checked out the pull request. There's an existing feature request to add the commits here as well (https://github.com/microsoft/vscode-pull-request-github/issues/2201), so I'm going to close this as a duplicate. Thanks!


Added internal support for Invoke-AzRestMethod in Test-AdfLinkedService

Closes #144

Please let me know if there are any changes needed before merging this. E.g. argument naming or logic of the arguments.


Fix bug #76 where embedded resources were not used

Fixes bug #76 which caused the embedded resources to be ignored

Re-opened in #79 and fixed the LogListDataSourceFactoryTest


hide the flatpickr

Can we hide flatpickr for  specific condition ?

Sorry, can you be more specific?
If you're looking to programmatically close the calendar, you should be able to do something like this using flatpickr's close method:
<div>
    <Flatpickr ref={this.fp} />
    <button type="button" onClick={() => { this.fp.flatpickr.close(); }}>Close</button>
</div>


Facebook and Twitter login fail if [Parse setApplicationId:clientKey:] is called multiple times

Our application has a legitimate use for changing Parse apps while running.  In Parse 1.8, this worked without a problem.
However, when we tried upgrading to Parse 1.10, Facebook and Twitter login is failing.  The call to [PFFacebookUtils logInInBackgroundWithReadPermissions:block:]is called back with user and error both nil.  Internally, there is an exception that is caught but not propagated:
No registered authentication delegate found for `facebook` authentication type.
Register a delegate first via PFUser.registerAuthenticationDelegate(delegate, forAuthType:)

I see in https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/issues/159 that apparently our use case is not supported.  Please consider making this a supported use case and fixing this issue.

Hey @jimspeth, unfortunately the behavior is by design undefined when calling Parse.+setApplicationId:clientKey: multiple times in a single application run.
I will mark this issue as discussion/enhancement, as I can see small narrow set of apps that could benefit from it and we might support it later.

@nlutsenko How about just exposing a +[Parse reset] method that just resets all state?  It seems like this would be fairly straight-forward.
I've been able to work around this in our code by calling these private methods:
    [Parse _clearCurrentManager];
    [PFAnonymousUtils _clearAuthenticationProvider];
    [PFFacebookUtils _setAuthenticationProvider:nil];
    [PFTwitterUtils _setAuthenticationProvider:nil];

Anything I'm missing resetting?

Implemented in #874


fix start dkv server and cli not work

fix no main class for starting dkv-server and dkv-client.
fix inconsistent between startup command and documentation.

Thanks for your efforts.
Could you also please add this as a CI job? so that we can know it in a broken PR.

It seems that the CI hangs at starting dkv server.
I guess the reason is that /scripts/install_distkv.sh will hold the shell.
Maybe it's better to use nohup to start dkv server to avoid this issue.
Thanks.

Hi @meijies ,
Could you accept the suggestions I left above? And then we can merge this PR ASAP.

@jovany-wang  OK

@meijies  Thanks for your efforts.
It will be merged once CI passes.


Dataframe resampling by groups

The examples that I see for resample seem to only involve pd.Series, although the method is technically defined for a pd.DataFrame.
Can it be used to do the type of more complex resampling described here? Can I perhaps do this combining resample with transform having it work on one-column at a time as in the examples with the series?

@hayd is should you something more complex than .resample actually grouping with a frequency based grouper (essentially a resample) and a regular type grouper.
.resample works on DataFrames as well.


Search ARPayment by Reference Number

Hi,
How do I search payments by reference number? I entered RefId and Reference Number, but the query fails. Here is my code:
            InListString query = new InListString()
            {
                Field = "CUSTOMERID",
                ValuesList = customerIds,
            };

            ReadByQuery readByQuery = new ReadByQuery()
            {
                ObjectName = "ARPayment",
                Fields = { "*" },
                Query = query
            };

When I create payments, I assign a reference number:
            ArPaymentCreate record = new ArPaymentCreate("unittest")
            {
                CustomerId = doc.CUSTVENDID,
                UndepositedFundsGlAccountNo = account.ACCOUNTNO,
                ReferenceNumber = payment.ReferenceNumber,
                TransactionPaymentAmount = payment.PaymentAmount,
                ReceivedDate = payment.PaymentDate,
                PaymentMethod = payment.CreditCardName,
                ApplyToTransactions = new List<ArPaymentItem>()
                 {
                      new ArPaymentItem()
                      {
                           AmountToApply = payment.PaymentAmount,
                           ApplyToRecordId = int.Parse(doc.PRRECORDKEY)
                      }
                 }
            };

I don't really want to come right out and tell you because I want you to figure this out on your own.  Try making a payment with the today's date and the reference number<PHONE_NUMBER>.  Then in the readByQuery ARPAYMENT response, look for the payment with today's date and that reference number.  Here's a screenshot hint -- https://i.imgur.com/uQ5kpQ5.png
If you aren't already using postman to test some of your requests, take a look here - https://developer.intacct.com/web-services/your-first-api-calls/


encoding isn't working correctly for swagger docs

We're not handling "==" correctly in the UI docs
It does function properly, just reads weird.

This is being done by the swagger editor. Will have update the swagger editor code for this.

Migrated item to MsAzure Dev Ops


Add linting to watch task

As far as I know, it's not possible to integrate linting into watchify. Linting was integrated into karma which gets started by the watch task anyway. Unfortunately this causes senseless linting errors, which should probably be part of another ticket.


F2 renaming not working across all python files

Steps to Reproduce:
file1.py
class tempclass(object):
def __ init__(self):
self.var1 = 0
in file2.py
I imported file1.py and called var1.
If I rename var1 in file1 using F2, it is not changing in file2.py
version used: v2024.6.0
Vs code version: 1.89

Hi,
I had python.analysis.include = [“./**.py”]
When I removed this, it worked properly.


Add new logos to beta-logos package

Acknowledgements
https://github.com/api3dao/tasks/issues/445
https://github.com/api3dao/tasks/issues/479
https://github.com/nodaryio/utilities/pull/20
To-Do
Add the following logos;
Commodity:

[x] NG

Crypto:

[x] PYTH
[x] WOO
[x] stETH

LSD:

[x] ETHx
[x] MATICx
[x] MATIC Exchange Rate
[x] stMATIC

Stock:

[x] BABA
[x] PYPL
[x] QQQ

https://api3dao.github.io/logos/

Symbol Name
Logo

BABA

ETHx

MATICx

PYTH

WOO

MATIC Exchange Rate

NG

QQQ

PYPL


Typo in Get-BrowserData.ps1

There's a typo in Get-BrowserData.ps1, in the function Get-ChromeBookmarks.  The Json object created to hold the Chrome bookmarks lists the browser as 'Firefox', when it should be 'Chrome'.

Thanks!


djangocms-text-ckeditor 3.6.0 has requirement html5lib<0.99999999

Divio Please Update the requirement of djangocms-text-ckeditor.
from  HTML5lib <0.99999999 to html5lib 1.0.1
because I can't run other library.
xhtml2pdf can't run without HTML5lib 1.0.1

Perhaps more importantly, this fails the checks using the Safety database:
html5lib                   │ 0.9999999 │ <0.99999999              │ 35693    │
╞══════════════════════════════════════════════════════════════════════════════╡
│ The serializer in html5lib before 0.99999999 might allow remote attackers to │
│  conduct cross-site scripting (XSS) attacks by leveraging mishandling of the │
│  < (less than) character in attribute values.                                │
╞══════════════════════════════════════════════════════════════════════════════╡
│ html5lib                   │ 0.9999999 │ <0.99999999              │ 35694    │
╞══════════════════════════════════════════════════════════════════════════════╡
│ The serializer in html5lib before 0.99999999 might allow remote attackers to │
│  conduct cross-site scripting (XSS) attacks by leveraging mishandling of spe │
│ cial characters in attribute values, a different vulnerability than CVE-2016 │
│ -9909.                                                                       │
╞══════════════════════════════════════════════════════════════════════════════╡
│ html5lib                   │ 0.9999999 │ <0.99999999              │ 25846    │
╞══════════════════════════════════════════════════════════════════════════════╡
│ html5lib before 0.99999999 is vulnerable to a XSS attack. Upgrading avoids t │
│ he XSS bug potentially caused by serializer allowing attribute values to be  │
│ escaped out of in old browser versions, changing the quote_attr_values optio │
│ n on serializer to take one of three values, "always" (the old True value),  │
│ "legacy" (the new option,  and the new default), and "spec" (the old False v │

See https://pypi.org/project/safety/

Note that this is already fixed in trunk - see #403 and #464, etc.
I.e. even though it would be good get a new Pypi release you can fix it now by installing github, e.g. in your requirements file:
git+https://github.com/divio/djangocms-text-ckeditor.git@b2c9bd9dbe9c3bdf0464953845e8933b05ae125b#egg

Personally, I think that this is a duplicate of #403 and can be closed.

The newest version use html5lib>=0.999999999


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.11.39 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Configure coordinate scale values independently for x and z

Currently coordinate scale can be configured differently between the y axis and the xz plane due to the existence of height scale, however coordinate scale still affects both the x and the z axis at once. I'd suggest making it possible to configure the coordinate scale values for the x and z axis independently of each other so the overflow point can differ depending on each axis.

closing as the notice on this mod's readme has announced discontinuation for a while now - feel free to reopen in future if work starts again