fix: add check for subscription schedule and email in customer sync

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.47%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

billing/customer/service.go
0
1
0.0%

billing/subscription/service.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
10266

Relevant Lines:
108406

💛  - Coveralls


New meshing miniapp: shaper [shaper-dev]

New meshing miniapp, shaper, that can be used to resolve complicated
material interfaces by mesh refinement, e.g. as a tool for initial
mesh generation.
Both conforming and non-conforming refinements are supported in 2D and 3D.
Some examples:

@jakubcerveny and @agrayver, do you mind approving this PR (if you are OK with it)?

We probably don't want this in the distribution, but here is a simple script and modification of the shaper miniapp to use any input image as a material specification function: image2shape.tar.gz.
For example, it can turn this: au.pdf into this:

The resulting mesh than can be used for some quick MFEM simulations.

Shouldn’t the water be blue and the land brown?
From: Tzanio Kolev<EMAIL_ADDRESS>Reply-To: mfem/mfem<EMAIL_ADDRESS>Date: Friday, April 14, 2017 at 1:57 PM
To: mfem/mfem<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [mfem/mfem] New meshing miniapp: shaper [shaper-dev] (#171)
We probably don't want this in the distribution, but here is a simple script and modification of the shaper miniapp to use any input image as a material specification function: image2shape.tar.gzhttps://github.com/mfem/mfem/files/922939/image2shape.tar.gz.
For example, it can turn this: au.pdfhttps://github.com/mfem/mfem/files/922942/au.pdf into this:
[u-2]https://cloud.githubusercontent.com/assets/1278247/25055790/dced2254-2119-11e7-96f3-fc365c18bdc1.png
The resulting mesh than can be used for some quick MFEM simulations.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/mfem/mfem/pull/171#issuecomment-294235283, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMNRnAhxIzqubrTE4yX5g0PcL0Yy9RQwks5rv92ugaJpZM4My135.


The first letter of "русский"

Hello.
Why the first letter of "русский" in the language list is lowercase? It is necessary to make, by analogy with other: "Русский".

thanks!


Cannot use start(context, fragment) with android.support.v4.app.Fragment with version 2.8.0

With version 2.8.0 of this library android.support.v4.app.Fragment is no longer supported. Please add support back for this library. A lot of us are still using this version of fragment.
Thanks

I have same issue

i have the same issue

me too

Same issue, I believe this because they upgraded to AndroidX, and there no support anymore for support.v4.app.fragment

I want to downgrade, what is the latest version that supported android.support.v4.app.Fragment

try: implementation 'com.github.svran:Android-Image-Cropper:<IP_ADDRESS>'
On Mon, 4 Feb 2019 15:58 MuhammedRefaat<EMAIL_ADDRESS>wrote:

Same issue, I believe this because they upgraded to AndroidX, and there no
support anymore for support.v4.app.fragment
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ArthurHub/Android-Image-Cropper/issues/628#issuecomment-460302109,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYEtYE452bDedg_S-zxCFBxtBBS5-uRWks5vKFiigaJpZM4YwK9b
.

try: implementation 'com.github.svran:Android-Image-Cropper:<IP_ADDRESS>'
On Sun, 3 Feb 2019 10:32 erik7667<EMAIL_ADDRESS>wrote:

me too
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ArthurHub/Android-Image-Cropper/issues/628#issuecomment-460040242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYEtYPr5IbV7B1mEY-4-i0eA448Gr81lks5vJrqjgaJpZM4YwK9b
.

try: implementation 'com.github.svran:Android-Image-Cropper:<IP_ADDRESS>'
On Tue, 5 Feb 2019 07:14 Shequad<EMAIL_ADDRESS>wrote:

I want to downgrade, what is the latest version that supported
android.support.v4.app.Fragment
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ArthurHub/Android-Image-Cropper/issues/628#issuecomment-460536136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYEtYJhxFIXenCJV3FLhShcSR6GT0Tj4ks5vKS9TgaJpZM4YwK9b
.

No option in this repo to downgrade?


Expose std support via --print

As discussed on Zulip, this RFC proposes adding a new argument for rustc --print that acts as the single source of truth for whether a target supports std.
Rendered

I don't know what process the compiler team wants to use, but typically in the past they have been using the Major Change Proposal process for new flags and such like this (instead of an RFC).

I don't know what process the compiler team wants to use, but typically in the past they have been using the Major Change Proposal process for new flags and such like this (instead of an RFC).

I think the line is a bit blurry, our documentation says:

It can also be used for small user-facing changes like adding new compiler flags, though in that case we also require an rfcbot fcp to get full approval from the team.

Is this a small user-facing change? Probably? Given that an RFC has been written and it'll be subject to an FCP anyway, I think we might as well keep it as an RFC.

What does "std support" means exactly? You can, for example, make you own crate and make it the std library.
Having an std library shipped is not a property of the compiler, it's a property of the distributed toolchain, similarly to having the LLD linker shipped, for example.
How is compiler going to implement this --print option

Report some hardcoded value set during at toolchain's build time?
Actually start the crate loader and search for the crate named std?

The distribution itself can potentially ship a json file describing its components (rustup already does something like that?), then rustc --print can point to the location of that json file.

From my perspective rustc already treats core/std/others as special when it comes to the sysroot and attributes like #![no_std] or #![no_core]. In addition, these crates are versioned with the compiler and you can't expect to mix and match compiler and std versions, even for different nightlies for the same release version. Hence I think std is already unambigious except for the case where users write their own standard library crates. I can define std more strictly in the RFC to reflect this.
The RFC goes into implementation details - it's going to use an existing field in the Target spec. The output can be changed by using a custom JSON target but if a user's custom std doesn't support a target and the user wants to use a built-in target then I think I agree with your concerns. I'm not sure if people do that or what guarantees the compiler makes for people who write their own std crates.

Having an std library shipped is not a property of the compiler, it's a property of the distributed toolchain, similarly to having the LLD linker shipped, for example.

To add to what @adamgemmell said, this flag isn't that related to whether a std is shipped for a target. With something like -Zbuild-std, you can build a std that isn't shipped, and on some targets that just won't build, or maybe it will but will panic unexpectedly - in that circumstance, we'd like to be able to know "this std isn't going to work" so that we can give the user a helpful message to that effect rather than a potentially confusing build error that they can't easily action.

if a user's custom std doesn't support a target and the user wants to use a built-in target

Looking back at this I'm not sure it's a concern. Whether a target can support std is still a property of the target (i.e. whether it has a suitable OS) and as the RFC says the flag doesn't attempt to differentiate targets with a "broken" std.

I think this is a reasonable addition to the --print command. What do you all think?
@rfcbot fcp merge

I want to second @petrochenkov's questions. In my mind a lot of the motivation reads as a desire to add a mechanism to "ask" before doing. It then again motivates that this is to avoid confusion by the users of -Zbuild-std. It seems to me that this particular problem could be addressed from a direction of "just doing" and then figuring out how to provide personalized diagnostics if the desired action does not succeed. It could be as simple as
// in std/lib.rs
#[cfg(not(supported_platform))]
compile_error!("libstd is not supported for $platform");

or whatever, with huge benefit of such approach being that the support for libstd remains entirely within confines of libstd without affecting the implementation of the compiler in an orthogonal way.


Add application level configuration to enable refresh token expiry extension on refresh token renewal

Is your suggestion related to an experience ? Please describe.
According to issue [1], for browser based applications refresh token expiry time shouldn't be extended upon refresh token renewal. In order to tackle this requirement we already have added an server level config [2]. But ideally this should be configurable for each application independently.
[1] https://github.com/wso2/product-is/issues/8312
[2] https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/1490

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add full_fee_amount column: post-commit review fixes

This is a follow up for the prematurely merged PR #1173. Please, review changes related to Orderbook::add_order and OrderValidator::validate there, and I'll address all comments in this PR.
For now, this PR just changes naming to be consistent.

Codecov Report

Merging #1200 (ea79979) into main (bf17335) will not change coverage.
The diff coverage is n/a.

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

@@            Coverage Diff            @@
##              main     #1200   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            8         8
=========================================
  Hits             8         8


New Flutter Project (application) doesn't generate main.dart

Hi,
I'm using Android Studio Electric Eel | 2022.1.1 Patch 1 (Build #AI-221.6008.13.2211.9514443, built on January 21, 2023) to create my Flutter app. However, new Flutter projects doesn't generate main.dart or lib folder.
I'm using a m1 mac, on MacOS Monterey. Tried reinstalling flutter, android studio but with no results. What can I do to resolve this?
Attached screenshot:

My .zshrc is:

export PATH="/Users/*****/Developer/flutter/bin:$PATH"

My flutter doctor -v is below:

[✓] Flutter (Channel stable, 3.7.1, on macOS 12.5.1 21G83 darwin-arm64
(Rosetta), locale en-SG)
• Flutter version 3.7.1 on channel stable at
/Users/*****/Developer/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 7048ed95a5 (8 days ago), 2023-02-01 09:07:31 -0800
• Engine revision 800594f1f4
• Dart version 2.19.1
• DevTools version 2.20.1
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at /Users/*****/Library/Android/sdk
• Platform android-33-ext4, build-tools 33.0.1
• Java binary at: /Applications/Android
Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
11.0.15+0-b2043.56-8887301)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
11.0.15+0-b2043.56-8887301)
[✓] Connected device (3 available)
• Android SDK built for arm64 (mobile) • emulator-5554 • android-arm64  •
Android 9 (API 28) (emulator)
• macOS (desktop)                      • macos         • darwin-arm64   •
macOS 12.5.1 21G83 darwin-arm64 (Rosetta)
• Chrome (web)                         • chrome        • web-javascript •
Google Chrome 109.0.5414.119
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Hi @Zlearncode, does the recording below capture the steps you use to create your project?
Do you also have this issue if you run flutter create foo?
https://user-images.githubusercontent.com/88313112/217785870-46630573-d33f-40f2-9560-37194ed05cc3.mov

Hi @Zlearncode, does the recording below capture the steps you use to create your project?
Do you also have this issue if you run flutter create foo?
Screen.Recording.2023-02-09.at.10.22.11.mov

Yes, the recording captured the steps I did, except that mine only showed external libraries and scratches and consoles.
Running "flutter create foo" in terminal works fine. I don't know why my new flutter project doesn't generate all the other files and folders.


Tasks/Terminal: Run Selected Text in Terminal won't launch terminal with background task in progress

Code 1.14.2 and the insiders 7-24
Windows 7
Steps to reproduce:

Use background or watch task such as build VS Code
select file with whatever shell commands, select line
Attempt to use Run Selected Text
no new terminal or previously open terminal will be used, command is ignored
It does not matter if the task is currently executing or idle

Interesting find. I don't think it's worth moving on this as the tasks will eventually be moved to their own panel which will solve this. In the meantime it's kind of as designed as task terminals are regular terminal instances that can receive text just like non task terminals.


cmake 3.25.2 doesn't work on ARM MacBook Pros

Describe the bug
I have brew, asdf via brew, and the cmake asdf plugin installed. I used the plugin to install cmake 3.25.2 on my ARM MacBook Pro. When I then try to run cmake in a terminal I get a GUI based popup from MacOS saying it is corrupted. A co-worker also with a ARM MacBook Pro has the same issue. I then tested installing brew, asdf via brew, and the cmake asdf plugin on a Intel MacBook Pro. I then installed cmake 3.25.2, and running cmake in a terminal works as expected. Finally a co-worker on an ARM MacBook Pro installed cmake via brew and that installed 3.26.0, which worked as expected.
Steps to reproduce

Use ARM MacBook Pro running MacOS 13.2.1(Ventura)
Install brew
Install asdf via brew
Install cmake asdf plugin
Install cmake 3.25.2 via the plugin
Run cmake in a terminal

Expected behavior
Usage output

No crash, program runs, and the programs help/usage output is displayed.

GUI based popup from MacOS saying it is corrupted
I had the same issue on my side but it is not that the binary is corrupted in the end but the fact that the provided binaries are not officially accepted/signed and therefore you need to open "System Setting"->"Privacy & Security" and allow that binary to be executed in the future.

Facing the same issue here.
Does anybody know if there is a way to fix the issue in an automated way instead of manually click the Perform Anyway button in the Privacy & Security settings?


Use files to my other project

Hi,
Can i use cmake files from this project inside my project that is a build tool too?
Thanks.

You mean whether you're allowed to do this? Sure!
Whether it technically works, I don't know :)


Updated to work with azure cli 2.0

As of Azure CLI 2.0.68, the --password parameter to create a service principal with a user-defined password is no longer supported.

Hi, @Buzzardo, could you please close this PR because it's outdated?


Issue with Faraday dependency

I am seeing a lot of issues in my project in relation to Faraday depends - e.g. bit.ly depends on 0.7.5, oauth2 depends on 0.7.5.

My fork fixes this by moving dependency to faraday 0.7.5, and faraday-middleware to 0.8.0


绝对路径的图片显示是不是不支持?

有这个选项, 但试来试去还是没用.
缺少这个功能最大的不方便就是在移动文档的时候. 相对图片路径根本没什么用, 极大的限制了功能

There is this option, but it is still useless to try and try.
The biggest inconvenience of lack of this function is when moving documents. Relative image path is useless at all, which greatly limits the function

Which file is being affected? Could you please provide more details & maybe a sample file could be helpful to troubleshoot your issue, since this extension deals with many file types like .html, .docx, etc...


Better structure virtualenv installation instructions.

@johngian really quick r?

r+

wow, that was fast! :beers:

Coverage increased (+0.01%) to 91.472% when pulling 414fd855dff43d770a14a96bffe38b3697a36215 on akatsoulas:small-fix-doc into feff31678f9c534bf73bfe7003b5087a987900fc on mozilla:master.


Consider adding a depreciation warning for Dark Sky weather API for 2021

With the recent acquisition of Dark Sky by Apple: https://www.bbc.com/news/technology-52115095
The news seems to indicate that the Dark Sky API will be discontinued and no longer accepts new signups.

Dark Sky's API - the software that transmits information between the app and its users - will be available to customers until 2021. But the company said it will no longer sign up new subscribers.

Should there be a warning or a depreciation note for that service?

I'd suggest removing it altogether since they're not accepting new users.


GH-4826: Fix StackOverflow in ContextAwareConnection

GitHub issue resolved: #4826
Briefly describe the changes proposed in this PR:
Cast ConvertingIteration to Iteration so calling super.add(...) does not end in a StackOverflowError.

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] I've squashed my commits where necessary
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

Thanks for this fix @domkun I think it's related to the default interface method implementation.
For the next major version of RDF4J we are simplifying the Iteration and CloseableIteration interfaces by removing the Iteration interface and making everything use just the CloseableIteration interface. This makes me think that this bug hopefully doesn't exist in RDF4J 5.0.0-M2 (our latests milestone build).
Any chance you could test 5.0.0-M2 to see if the bug still exists or not. If it's not present in 5.0.0-M2 then I'm fine with merging your PR as is, without a need for a test.

Hi @hmottestad , you are right, the bug does not exist in version 5.0.0-M2, just checked it. Running the reproducer from the issue against 5.0.0-M2 works without a problem.


Wishlist view

Codecov Report

Merging #81 (4cc6578) into master (a5f4e61) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #81   +/-   ##
=======================================
  Coverage   95.72%   95.72%
=======================================
  Files           7        7
  Lines         444      444
=======================================
  Hits          425      425
  Misses         19       19

Impacted Files
Coverage Δ

service/routes.py
96.75% <100.00%> (ø)

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


Solver Results

So from my understanding, after the solver completes a solve, it writes the results to a file.  This file is then interpreted by some SolverResult class.
So my question is, is there any way to save these files and then re-import them into the SolverResult GUI to view the already solved scenario without re-running the sim?
Also where are these result files saved?  Are they saved somewhere in the test/ directory?
Thanks for all your work.

Actually your understanding is wrong.
After the solver completes a solve, it didn't write the result to file, at least it's not what happened in the gui version. After solving, the solver keeps the result in memory and access them through memory. Currently sadly there is no way to re-import the result to the gui, but I will put this to schedule.
ps: This java solver will be abandoned soon，because new c++ version's gui is under development https://github.com/bupticybee/TexasSolver , I will try implement re-import in c++ version.
Thank you for your feedbacks.

Understood. Excited about the c++ version! What are you writing the GUI in? QT?

Closing because this issue is solved.


Python 3 support

Hello guys. Awesome tool. I hit some problems with UTF-8 filenames in mountpoints. I saw in fusepy this issue is fixed with python 3 support.
Any plans for Python 3 support soon?

+1 I'm interested too.

+1
Python 2 is rapidly approaching official EOL. Is there a list of what needs to be done to accomplish this? I am happy to contribute changes to make it happen.

+1


Deprecated su_owner parameter referenced in README.md

Affected Puppet, Ruby, OS and module versions/distributions

Module version: 3.4.0

What are you seeing
README.md still refers to deprecated su_owner parameter. If used as documented manifest compilation will fail:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Logrotate::Rule[somerule]: has no parameter named 'su_owner'

What behaviour did you expect instead
README.md should contain correct information.

it's also referenced in a comment in rule.pp


Enable clients to obtain environment reference to be able to track progress

Similar to how Console creates environment showing progress to user:

we need to enable clients (like Vagrant plugin) to be able to track the environment creation progress.
For this we need to add support to client REST API

Will be able to test it after vagrant plugin release


API documentation for 0.11.2003.2506.

This PR includes documentation changes from the 0.11.2003.2506 release of the Quantum Development Kit.

Docs Build status updates of commit 225a118:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/qsharp/Microsoft.Quantum.Arrays.yml
:white_check_mark:Succeeded
View

api/qsharp/microsoft.quantum.arrays.all.yml
:white_check_mark:Succeeded
View

api/qsharp/microsoft.quantum.arrays.equala.yml
:white_check_mark:Succeeded
View

api/qsharp/microsoft.quantum.arrays.filtered.yml
:white_check_mark:Succeeded
View

api/qsharp/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Todo

Move to Net6.0 :white_check_mark:

Add nullable checks :white_check_mark:

Add GitVersion :white_check_mark:

Add NuGet package definitions to project files :white_check_mark:

Update namespaces to NaGet :white_check_mark:

Add Github Sourcelink :white_check_mark:

Add file-scoped namespaces :white_check_mark:

Add standard code style

Double license all files

Update readme

Adjust build and so on to known practices

Inform other users of the added features

Add global usings

Rework docs

Rework samples

Rework tests

Adjust frontend from Razor to Blazor?

Add all contributors plugin

Update NuGet packages

Update Todos

Fix names "BaGet"

Add Serilog as Logger

Bring this whole thing to work properly again

Fix all namespaces to match the folders

Development stopped.


new feature toggle for PBAv3

Addition of new feature toggle which will be used as part of implementing PBAv3
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

All good for me. Get a second review and it can be merged. Nothing to test here as it's functional to other features.


Include BeamLocation as a parameter in beamTo (or create a new builder)

This should be a pretty easy addition - I'd make my own PR but don't wanna mess up the tests etc.
In my use case, I have a loading screen that redirects the user to the requested BeamLocation once the check is satisfied (as it's a Stream).
  /// If guard [check] returns false, build a location to be beamed to.
  ///
  /// This has precedence over [showPage].
  BeamLocation Function(BuildContext context, BeamLocation location)? beamTo;

class BeamerRouterDelegate extends RouterDelegate<BeamLocation>
    with ChangeNotifier, PopNavigatorRouterDelegateMixin<BeamLocation> {
...
  @override
  Widget build(BuildContext context) {
    final BeamGuard? guard = _guardCheck(context, _currentLocation);
    if (guard?.beamTo != null) {
      beamTo(guard!.beamTo!(context, _currentLocation));
    }
...

Duplicate of https://github.com/slovnicki/beamer/issues/115


General: Add data export for exercise participations and communication data

This PR contains approved database changes and adds a new property artemis.data-export-path
Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I followed the coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

This is the first step implementing a data export for users of Artemis
Description
In this PR I added the functionality to export personal data of exercise participations, submissions, and results
and  a basic user interface on the client side.
In the future this will become a scheduled nightly operation because it can potentially be a resource-consuming operation.
This PR adds the export for all exercise submissions and for everything related to communication (messages, posts, reactions)
Steps for Testing

Prerequisites:

1 Admin

Log in to Artemis
Participate in exercises of different types.
Write some messages, thread replies, react to posts
Navigate to the privacy statement
Click on the data export button
Click on the request data export button (wait till the creation is finished)
Download the export and check that the information about your previously generated participations, submissions, results and the messages is in there

Review Progress

Performance Review

[x] I confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[x] I confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[x] Code Review 1
[x] Code Review 2

Manual Tests

[x] Test 1
[x] Test 2

Test Coverage

Client

Class/File
Line Coverage
Confirmation (assert/expect)

data-export.component.ts
100%
✔

data-export.service.ts
100%
✔

privacy.component.ts
95%
✔

data-export.model.ts
100%
✔

delete-button.directive.ts
97.29%
✔

delete-dialog.component.ts
91.17%
✔

delete-dialog.model.ts
100%
✔

delete-dialog.service.ts
100%
✔

Server

Class/File
Line Coverage
Confirmation (assert/expect)
Comment

BaseExercise.java
93%
✔

DataExport.java
100%
✔

DataExportState.java
100%
✔

User.java
90%
✔

CourseRepository.java
81%
✔
no changes that could be covered

DataExportRepository.java
100%
✔

ExerciseRepository.java
64%
✔
no changes that could be covered directly

QuizQuestionRepository.java
N/A%
✔
no changes that could be covered directly

UserRepository.java
92%
✔
no changes that could be covered directly

AnswerPostRepository.java
87%
✔
no changes that could be covered directly

PostRepository.java
92%
✔
no changes that could be covered directly

ReactionRepository.java
57%
✔
no changes that could be covered directly

AuthorizationCheckService.java
91%
✔

DataExportService.java
96%
✔

DragAndDropQuizAnswerConversionService.java
98%
✔

ProgrammingExerciseExportService.java
85%
✔
my changes are fully covered

DataExportResource.java
100%
✔

Screenshots

Codecov Report

Merging #6500 (dda8c18) into develop (cf74c01) will decrease coverage by 4.40%.
The diff coverage is 90.53%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6500      +/-   ##
=============================================
- Coverage      84.94%   80.55%   -4.40%
- Complexity     13321    13435     +114
=============================================
  Files           1093     2370    +1277
  Lines          42898    91154   +48256
  Branches        4439    13445    +9006
=============================================
+ Hits           36440    73429   +36989
- Misses          4007     9746    +5739
- Partials        2451     7979    +5528

Impacted Files
Coverage Δ

...m/in/www1/artemis/repository/CourseRepository.java
92.68% <ø> (ø)

...in/www1/artemis/repository/ExerciseRepository.java
87.50% <ø> (ø)

...artemis/repository/metis/AnswerPostRepository.java
60.00% <ø> (ø)

.../www1/artemis/repository/metis/PostRepository.java
92.30% <ø> (ø)

...1/artemis/repository/metis/ReactionRepository.java
100.00% <ø> (ø)

.../main/java/de/tum/in/www1/artemis/domain/User.java
87.38% <25.00%> (-2.34%)
:arrow_down:

...va/de/tum/in/www1/artemis/service/FileService.java
78.55% <50.00%> (-0.27%)
:arrow_down:

...ww1/artemis/service/AuthorizationCheckService.java
74.64% <69.23%> (-0.55%)
:arrow_down:

...pp/shared/delete-dialog/delete-button.directive.ts
89.18% <83.33%> (ø)

...pp/shared/delete-dialog/delete-dialog.component.ts
82.35% <85.71%> (ø)

... and 17 more

... and 1315 files with indirect coverage changes

Flag
Coverage Δ

client
76.55% <96.66%> (?)

server
84.98% <89.70%> (+0.03%)
:arrow_up:

Continue to review full report in Codecov by Sentry.

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

Some feedback for a follow-up PR:

The text in this screenshot is way too big. Please use the default fonts

The resource should be responsible for input validation and authorization checks, not the service. So please invoke authorization checks from DataExportResource and not from DataExportService
The actual authorization check method is only used in one place and there is no need to put this method in the AuthorizationCheckService, so please move it into DataExportResource where the check should happen


Replace mock data

Fix #148
Test URLs:

Before: https://main--helix-sportsmagazine--headwirecom.hlx.page
After: https://replace-mock-data--helix-sportsmagazine--headwirecom.hlx.page

Can you create another branch with the same code but non blocking index retrieval ie no await call to fetch the content to compare performance ?

For blocks that are dynamic ie they get their content from the index, I propose in general to use classes to help authors define which content they would like to show like we did for carousels:

WDYT ?


proposal: x/tools/cmd/goimports: ability to exclude specific directory, like vendor

Currently as i know, goimports dont have an option to exclude specific directory so i am excluding vendor directory on my personal projects like below:
for item in $(find . -type f -name '*.go' -not -path './vendor/*'); do goimports -l -w $item; done
it is a little bit uggly i guess, so i and i think lots of gophers like me will be glad if that option is supported by goimports.

CC @golang/tools-team

This seems like a duplicate of #30058, #37724, and others. Is there a difference I am missing?

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

@rsc why it is likely declined brother? what kind of negative consequences that tiny feature could have?

@bilalcaliskan See the issues mentioned above as duplicates.

I ve missed the duplicated issues so sorry for trouble. Closing that duplicate right now. Cheers!

This proposal has been declined as retracted.
— rsc for the proposal review group

I believe we need some way of ignoring .git directory or any workaround for this.
mau@mba-776:lucidgarage$ goimports -w .
.git/logs/refs/remotes/origin/feature/IOTTDO-2934-rename-dashboard.go-to-fhm_event_detection.go:1:63: expected ';', found 2
.git/logs/refs/remotes/origin/feature/IOTTDO-2934-rename-dashboard.go-to-fhm_event_detection.go:1:133: illegal character U+0040 '@'
.git/logs/refs/remotes/origin/feature/IOTTDO-2934-rename-dashboard.go-to-fhm_event_detection.go:2:2: expected '}', found 'EOF'
.git/refs/remotes/origin/feature/IOTTDO-2934-rename-dashboard.go-to-fhm_event_detection.go:1:23: expected ';', found cde7bb5fd5415b7f190e192b2c26ac2ffe6d883
.git/refs/remotes/origin/feature/IOTTDO-2934-rename-dashboard.go-to-fhm_event_detection.go:2:2: expected '}', found 'EOF'


az storage account keys renew appears to require --resource-group

Describe the bug
Attempting to run is command without specifying the optional --resource-group fails.
This command indicates that --resource-group is optional as per the documentation here: https://docs.microsoft.com/en-us/cli/azure/storage/account/keys?view=azure-cli-latest#az-storage-account-keys-renew
Command Name
az storage account keys renew
Errors:
The command failed with an unexpected error. Here is the traceback:
No value for given attribute
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 663, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 697, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 333, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/storage/operations/account.py", line 27, in regenerate_key
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/core/tracing/decorator.py", line 73, in wrapper_use_tracer
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/mgmt/storage/v2022_05_01/operations/_storage_accounts_operations.py", line 1386, in regenerate_key
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/mgmt/storage/v2022_05_01/operations/_storage_accounts_operations.py", line 362, in build_regenerate_key_request
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msrest/serialization.py", line 652, in url
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msrest/serialization.py", line 760, in serialize_data
ValueError: No value for given attribute

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az storage account keys renew --account-name {REDACTED} --key secondary

Its unclear if this happens with the --key primary argument (as this was not tested) but is reproducible with --key secondary for sure.
Expected Behavior
I would expect the secondary storage key to be rotated for the given account name.
Environment Summary
Windows-10-10.0.19044-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.40.0

Extensions:
azure-devops 0.25.0

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context
When the --resource-group parameter is provided this command works as expected and is a work around for now.

@evelyn-ys for awareness

@aolszowka Thank you for reaching out, we are looking into it.

@aolszowka If you would like to run the command without specifying the resource group set the config as shown below:

az config set defaults.group=myRGName

az storage account keys renew --account-name MyStorageAccountName --key primary
az storage account keys renew --account-name MyStorageAccountName --key secondary

Output:
[
{
"creationTime": null,
"keyName": "key1",
"permissions": "FULL",
"value": "JF1bMEJ0IfVVp/BT48zUG********kC4vDugc0HXuzaw=="
},
{
"creationTime": "2022-10-18T07:13:22.377876+00:00",
"keyName": "key2",
"permissions": "FULL",
"value": "8qGE49QRU04N4YB+AStwN/Ugg=="
}
]
Please let us know if that helps.

Hey @navba-MSFT that's great but that means that the documentation is still incorrect.
--resource-group is not optional, unless you have defined a default. That's fine, but should probably be called out in the documentation.
For now I am specifying the --resource-group with every command going forward.


Page does not describe how to cleanup old configuration

This page does not provide enough information to make the following messages in pipeline summary actionable:

This classic pipeline contains customized retention rules. Azure DevOps will stop supporting per-pipeline retention rules in an upcoming release. We strongly suggest you consider moving this pipeline to the project-level retention feature by removing the custom rules.

How should we "remove the custom rules"❔ I can't find anything that seems relevant in the pipeline settings.

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

ID: 43a285f9-738d-9f76-4de0-a4c4ecf6f5f5
Version Independent ID: 9581a043-4873-bf0a-138f-85f6a61ae950
Content: Retention policies for builds, releases, and test - Azure Pipelines
Content Source: docs/pipelines/policies/retention.md
Product: devops
Technology: devops-cicd-policies
GitHub Login: @juliakm
Microsoft Alias: rabououn

@dougbu Thanks for your question. While we're looking into the current documentation, consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow


Can't compile ch5/min_sysinfo

Pulled the latest code and run "make", receive the errors below. What fixes it is adding function prototypes on top.

--- Building : KDIR=/lib/modules/6.10.10-lkp-dbg/build ARCH= CROSS_COMPILE= ccflags-y="-UDEBUG -DDYNAMIC_DEBUG_MODULE" MYDEBUG=n DBG_STRIP=n ---
gcc (Debian 12.2.0-14) 12.2.0

make -C /lib/modules/6.10.10-lkp-dbg/build M=/home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo modules
make[1]: Entering directory '/home/user/data/kernels/linux-6.10.10'
  CC [M]  /home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.o
/home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.c:40:5: error: no previous prototype for ‘my_snprintf_lkp’ [-Werror=missing-prototypes]
   40 | int my_snprintf_lkp(char *buf, size_t maxsize, const char *fmt, ...)
      |     ^~~~~~~~~~~~~~~
/home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.c:68:6: error: no previous prototype for ‘show_sizeof’ [-Werror=missing-prototypes]
   68 | void show_sizeof(void)
      |      ^~~~~~~~~~~
/home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.c:84:6: error: no previous prototype for ‘llkd_sysinfo2’ [-Werror=missing-prototypes]
   84 | void llkd_sysinfo2(void)
      |      ^~~~~~~~~~~~~
/home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.c:157:6: error: no previous prototype for ‘llkd_sysinfo’ [-Werror=missing-prototypes]
  157 | void llkd_sysinfo(void)
      |      ^~~~~~~~~~~~
cc1: all warnings being treated as errors
make[3]: *** [scripts/Makefile.build:244: /home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo/min_sysinfo.o] Error 1
make[2]: *** [/home/user/data/kernels/linux-6.10.10/Makefile:1934: /home/user/data/Linux-Kernel-Programming_2E/ch5/min_sysinfo] Error 2
make[1]: *** [Makefile:240: __sub-make] Error 2
make[1]: Leaving directory '/home/user/data/kernels/linux-6.10.10'
make: *** [Makefile:131: all] Error 2

Thanks for your report!
Are you compiling with -Werror i.e. treat warnings as errors?  ..as that's what's happening here:
cc1: all warnings being treated as errors
In any case, it's as these functions (deliberately) aren't marked as static so that we can export and use them in other modules... to fix the warning/error, i'll declare the prototypes; pl pull again and retry, thanks.

Thanks. I used both the lkm script and make by itself - without any local modifications. I can confirm min_sysinfo now compiles, but the same issue exists in modstacking.

Ok.. can you provide your platform details pl: distro version, gcc version, anything else that may be relevant.. thanks

I have a plain vanilla debian 12 installation with kernel updated to 6.10.10:
OS: Debian GNU/Linux 12 (bookworm) x86_64
gcc: gcc (Debian 12.2.0-14) 12.2.0
kernel: 6.10.10
make: GNU Make 4.3

I find that these remain compiler warnings on my platforms (i even tried on a very recent 6.11 kernel on x86 Ubuntu), of the form:
core_lkm.c:43:6: warning: no previous prototype for ‘llkd_sysinfo2’ [-Wmissing-prototypes]
It doesn't throw an error as you seem to be getting; am unsure why this is... (perhaps Debian forces the -Werror?).
In any case, I've declared the func prototypes & added a commit - with credit to you.. Thank you


jsonschema should have tests

Can we have a test? What happens when this is out of the sync with the YAML struct definition?

Originally posted by @AkihiroSuda in https://github.com/lima-vm/lima/pull/2306#discussion_r1807391382

Now the generation of the schema will fail, if the result doesn't match the default yaml
e.g.
- at '/mountTypesUnsupported': got null, want array

(if removing the "nullable", like when it was added)


WIP: Avoid making WMI calls to find the windows version on each symlink creation

We can gather this information via ChefUtils which abstracts the Ohai data. That way we only make these WMI calls once.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

Looks like there are some unhappy unit tests.

Sadly this won't work because we use this all over cheffs and that won't have node data.


fully implimented night mode using css classes. i also used the dom m…

…ethod toggle to add the night mode class to various tags that will be affected by the night mode change
Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Wells - Awesome job implementing a fully functional theme toggle!
Improves -

Colour scheme - Use https://tailwindcss.com/docs/customizing-colors to find colours for the dark mode.
Merge conflicts - Please resolve merge conflicts and push again.

cc: @Reggie-Amedee-Tech

great work @Reggie-Amedee-Tech !


The books are now hosted on sre.google instead of landing.google.com

This PR fixes references to the domain as well as updating a couple of paths that has changed

Thanks a lot @danquah @asciigirl 💯

Thanks a lot @danquah @asciigirl 💯


Changes to properly support Mutexes on NET6.0

Global Mutexes were being created without insufficient rights for a 2nd process running on same machine to use them
This fix addresses that by using same methodology applied to NETFRAMEWORK but to NET6.0

Is there any plan on merging this PR?

Any update? What's the symptom without this fix, is there an particular error raised?
Looking for solutions to access denied errors on shared mode access, but I'm not seeing it on the mutex but on the journal file (and only one process is writing, the other opens database in read mode).

I would also like to see some repro code before merging this.

@JKamsker I can't provide a repro, but I can provide the scenario.
Environment: Windows 10/11
Version: 5.0.12
2 writer processes from different users.
1 of the writers is also a reader.
This happened sometimes during our tests, but our customers encountered it all the time, forcing us to release a patch disabling some functionalities.
We replaced it with SQLite to restore those functionalities.

@inbarbarkai Oh i remember now, i had a similar problem with named pipes and different users.
Ok thank you for the clarification! I will try to get to it asap :)

Similar to above, have not reproduced the issues we see, but have had them from customers, particularly this kind of error.
System.IO.IOException: The process cannot access the file 'C:\\XXX\\somedatabase.db' because it is being used by another process.
This occurs either on .db or the log, and the processes all open the database in shared mode. Have tried long and short lived instances of LiteDatabase also (I believe in shared mode it is internally opening and closing connections on each operation so likely makes little difference, unlike direct mode).
We've done extensive diagnostics and cannot see what the other process is. Some customers show signs of things like Microsoft Defender for Endpoints (the corporate version of Defender) access files but looks more like it detecting file access not actually opening and locking files. Other customers don't have anything like this though.
Any solution needs to be proven fix to roll out to customers though as already got frustrated customers, hence repo is very desirable.
Only alternative have thought about is to move to a single process for database operations and some communication between them. Getting into building a database server territory though and already considering supporting MongoDB as an alternative to LiteDB.

I am not sure about the added dependencies.
about the framework change: we are currently internally thinking about following the official microsoft support matrix.

Hi all - Is there any update regarding following the Microsoft support matrix? I'd love to add .NET 8 support if that's allowed. I was able to get this working locally for my  .NET 8 applications using the code from this related issue, but I would be happy to help get this merged if I can.


Improper resolution of .. components in symlinks

I believe I've run across a bug in wasmtime's handling of symlink targets which include "..". I've reproduced this using the Go program below which calls path_open on a path and reports the result. It's not the simplest test program, but it avoids dependencies and I have a Go compiler handy.
Simple case, no bug:
$ echo foo > file
$ ln -s file link
$ GOOS=wasip1 GOARCH=wasm go run main.go link
errno 0

The bug shows up when resolving a link with a target of "..":
$ echo foo > target
$ mkdir b
$ (cd b && ln -s .. up)
$ cat b/up/target
foo
$ GOOS=wasip1 GOARCH=wasm go run main.go b/up
errno 0
$ GOOS=wasip1 GOARCH=wasm go run main.go b/up/target
Not a directory

I'm not sure exactly what is going awry, but the "not a directory" error is clearly wrong. Note that we can open b/up (resolves to . as expected). The problem seems to be specific to links that end in ..: if we change the link target to ../a, everything works as expected:
$ mkdir a
$ echo bar > a/target
$ mkdir b
$ (cd b && ln -s ../a upa)
$ cat b/upa/target
bar
$ GOOS=wasip1 GOARCH=wasm go run main.go b/upa/target
errno 0

Test program:
package main

import (
        "fmt"
        "os"
        "syscall"
        "unsafe"
)

//go:wasmimport wasi_snapshot_preview1 path_open
//go:noescape
func path_open(rootFD int32, dirflags uint32, path unsafe.Pointer, pathLen uint32, oflags uint32, fsRightsBase uint64, fsRightsInheriting uint64, fsFlags uint32, fd unsafe.Pointer) uint32

const LOOKUP_SYMLINK_FOLLOW = 1

func main() {
        f, err := os.Open(".")
        if err != nil {
                panic(err)
        }

        filename := os.Args[1]
        var fd int32
        errno := path_open(
                int32(f.Fd()),
                syscall.LOOKUP_SYMLINK_FOLLOW,
                unsafe.Pointer(unsafe.StringData(filename)),
                uint32(len(filename)),
                0,
                syscall.RIGHT_FD_READ,
                syscall.RIGHT_FD_READ,
                0,
                unsafe.Pointer(&fd),
        )
        fmt.Println(syscall.Errno(errno))
}

Thanks for the report! Would you be able to gist exactly what wasmtime command is being used? You're using go run but it's not clear to me what exactly wasmtime is doing. Locally I can't seem to reproduce this with what I believe is an equivalent Rust program:
$ cat foo.rs
fn main() {
    println!(
        "{:?}",
        std::fs::File::open(std::env::args().nth(1).unwrap())
    );
}
$ rustc foo.rs --target wasm32-wasip1
$ mkdir a
$ echo bar > a/target
$ mkdir b
$ (cd b && ln -s ../a upa)
$ cat b/upa/target
bar
$ wasmtime --dir . foo.wasm b/upa/target
Ok(File { fd: 4 })
$ ls
a  b  foo.rs  foo.wasm
$ rm -rf a b
$ echo foo > target
$ mkdir b
$ (cd b && ln -s .. up)
$ cat b/up/target
foo
$ wasmtime --dir . foo.wasm b/up
Ok(File { fd: 4 })
$ wasmtime --dir . foo.wasm b/up/target
Ok(File { fd: 4 })
$ wasmtime --dir . foo.wasm nonexistent
Err(Os { code: 44, kind: NotFound, message: "No such file or directory" })

That may mean that the difference boils down to what Rust and Go are doing, and so the next step here would be to narrow in on that and see what's going on. If you're able to upload the Go-generated wasm binary here that would also help too

The wasmtime command is from here: https://go.googlesource.com/go/+/refs/tags/go1.23.0/misc/wasm/go_wasip1_wasm_exec
wasmtime run --dir=/ --env PWD="$PWD" --env PATH="$PATH" -W max-wasm-stack=1048576 main.wasm b/up/target

Wasm binary I'm using:
main.wasm.gz

Hm I actually can't seem to reproduce with the binary that you provided. I get:
$ wasmtime run --dir=/ --env PWD="$PWD" --env PATH="$PATH" -W max-wasm-stack=1048576 main.wasm b/up/target
errno 0

which I think means that it's working?
I noticed above sometimes the symlink was called "up" and sometimes it was "upa". I mixed the two up and thought Rust was working and Go wasn't but then when I fixed the name they both worked. Mind double-checking on your end too?

Setup:
$ echo foo > target
$ mkdir b
$ (cd b && ln -s .. up)
$ cat b/up/target
foo

And running:
$ wasmtime --version
wasmtime 25.0.0 (0b195ef5d 2024-09-20)
$ wasmtime run --dir=/ --env PWD="$PWD" --env PATH="$PATH" -W max-wasm-stack=1048576 main.wasm b/up/target
Not a directory

This is on macOS. I had thought I'd observed this on macOS and Linux, but I just rechecked and I can't reproduce on Linux. So perhaps macOS-specific?
Or possibly even Linux version specific, if wasmtime uses openat2 with RESOLVE_BENEATH when available.

Aha the plot thickens! I was indeed testing on Linux. I'll work next week on digging more into macOS, as I suspect you're right this is macOS-specific behavior.

I can reproduce the problem using cap-std directly with this example:
use std::io::Read;

fn main() {
    let path = "b/up/target";
    let dir = cap_std::fs::Dir::open_ambient_dir(".", cap_std::ambient_authority()).unwrap();
    let mut file = dir.open(path).unwrap();
    let mut s = String::new();
    file.read_to_string(&mut s).unwrap();
    eprintln!("read: {:?}", s);
}

If I modify cap-std to avoid using openat2, then I can reproduce the "Not a directory" error even on Linux. I'll investigate further.

I've now submitted #9307 which should fix this issue.


Pass ExpectedProps to PolymorphicComponentProps (v6)

Passes ExpectedProps to PolymorphicComponent as last argument to fix type inference with react-docgen and probably other tools. There is probably some problem with the inference from ActualComponent. Keeps inference from styled(Component) notation.

@DavidReinberger @joshjg Let's retest/revisit this after #3809 is merged.

@agriffis I can see the PR is merged 🥳 I've been poking around some more and just can't get the hang of it 😮💨... Nevertheless as much as I understand the problem is that PolymorhicComponentProps does not correctly return all the props.
Whenever I either add the ActualProps to the generic call or remove keyof ActualComponentProps from the Omit within PolymorphicComponentProps it starts working 🤔...
Since I'm bit lost in this whole problem I won't be rebasing the PR and can be closed.

@DavidReinberger It's not easy to get the hang of it. I'm holding on by my fingernails.
You can go ahead and close this. We'll track the issue in https://github.com/styled-components/styled-components/issues/3813
I use Storybook too and I'm quite interested to see this fixed.


Remove tailwind

As we switched over to Radix and Stitches a while ago we should clean up the codebase and get rid of Tailwind, which we're still using in a few leftover areas.
We can break up this task like this:

[ ] Change EditTokenForm to use Stitches
[ ] Change PushDialog to use Stitches
[x] Change TokenListing to use Stitches
[ ] Change Modal to use Stitches
[ ] Change CreateBranchModal to use Stitches
[ ] Change ADOForm to use Stitches
[ ] Change BitBucketForm to use Stitches
[ ] Change GitForm to use Stitches
[ ] Change JSONBinForm to use Stitches
[x] Change AnnotationBuilder to use Stitches
[ ] Introduce ErrorMessage component as lots of old tailwind classes are used for that one

There's some larger areas that need refactoring as these heavily depend on Tailwind and could be refactored:

[x] TokenButton
[x] ColorPicker
[x] .content and .content-dark scroll areas

What was the main reason for making the switch? Just out of curiosity?

Mainly the huge benefits of composability that Stitches brings. Compare creating variants with tailwind (adding classes) to that of using Stitches: https://stitches.dev/docs/variants
we made the switch a while ago but there's been some lingering remnants of tailwind code that I put off changing until now.


Godot no longer detecing gamepad input after initialize steam

Describe the bug
initialize steam causes gamepad-inputs to no longer be detected in any way (by _input() or otherwise)
code similar as in tutorial:
func initialize_steam() -> void:
var initialize_response: Dictionary = Steam.steamInitEx(true, SteamAppId)
print("Did Steam initialize?: %s " % initialize_response)
if initialize_response['status'] > 0:
print("Failed to initialize Steam, shutting down: %s" % initialize_response)
get_tree().quit()
is_on_steam_deck= Steam.isSteamRunningOnSteamDeck()
is_online= Steam.loggedOn()
is_owned= Steam.isSubscribed()
steam_id= Steam.getSteamID()
steam_username= Steam.getPersonaName()
steam_init=true
if is_owned == false:
print("User does not own this game")
get_tree().quit()
after calling this function gamepad inputs will no longer be detected by godot
To Reproduce
new project
new scene that just does:
_input(event):
print(event)
call previous initialize_steam() function
after that, gamepad input will not be detected
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
OS: WIN 10
GPU: 4080 super (newest driver)
CPU: AMD Ryzen 7 7800X3D 8-Core Processor
MOBO: Gigabyte B650 Gaming X AX (newest BIOS)
SSD: WD_Black SN850X
PSU: 1000W
Version of Godot:
Godot v4.2.2.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4080 SUPER (NVIDIA; <IP_ADDRESS>86) - AMD Ryzen 7 7800X3D 8-Core Processor (16 Threads)
Version of GodotSteam:
using the newest gdextension addon
Additional context
again only after initialize_steam(), before that everything is detected.
maybe this is expected? I don't know.
Thanks in advance
:)

just found the old issue with steam input where i have to disable steaminput in the spacewar game settings. still a little weird.

Hey there!  Yes, I was gonna say this comes up every so often and is a pain.  Steam loves to hijack the inputs and for a reasonable reason: providing controller support to games.
That being said I don't often have an issue with it so surely there is another solution.  I will try to dig up some more info to add to the docs and this issue.


Writing a post in a RTL language

Checklist

[x] I have read the tutorials and know the correct effect of the functional design.
[x] There are no similar question on existing issues (including closed ones).
[x] I searched the Internet for related problems, but still couldn't solve it.
[x] My question is based on the latest code of master branch.

Description

Hi.
Is there any way that I can write in a RTL language? Or a way I can change the alignment of a post without making a mess with something like code blocks and etc. ?
(I tried to change the post layout and I've added a "style=text-align: right" to the 'div' tag. but it ruined the whole post(including code blocks and some parts of the content that are in a LTR language(English).
Regards,
SeedPuller

Thanks for response.
Yes.
Look at this:

As you can see, the code block is LTR while the main text is RTL.
I tried to change the post Layout like this:

        {% if page.image.src %}

        {% endif %}

        {{ content }}

But the result was not satisfying...

How can i explicitly set the code block (or anything else) direction?

Hi.
I fixed this issue by changing the : _includes/refactor-content.html
{% if _content contains '
' %}
  {% assign _content = _content
    | replace: '

<code', '
<code'
    | replace: '

', '
'
  %}
{% endif %}


add support for Forgejo

Container images including this PR are available for testing at:

https://codeberg.org/forgejo-contrib/-/packages/container/woodpecker-forgejo-server/versions
https://codeberg.org/forgejo-contrib/-/packages/container/woodpecker-forgejo-agent/versions
https://codeberg.org/forgejo-contrib/-/packages/container/woodpecker-forgejo-cli/versions

This is a followup of a previous PR that was submitted at an early stage of development. The implementation of this Forgejo driver has matured since and is now substantially different. I hope you will consider its inclusion in Woodpecker and I stand ready to address any review comments you may have.

@earl-warren TLDR: please just update tje docs

Gitea's and Forgejo's APIs are the same (and will be the same as it's a soft fork).

This is not true: new API will be added, for instance to support semantic versioning. Forgejo API will be a superset of the Gitea API as long as it is a soft fork. It means that the Gitea ecosystem will work with Forgejo. But not the other way around: there is no commitment from Gitea to be compatible with Forgejo.
Here is another example: Forgejo adds a new webhook type "forgejo". This is not going to be supported by Gitea.
Does that clarify why this pull request is proposed?

These is value in using additional Forgejo API endpoints to improve the driver and I'm sure you'll agree eventually.

@earl-warren asap forgejo do break with gitea api  a pull to add forgejo dedicated will be accepted until then it's just another implementation to maintain :/

I'd rather maintain the current Forgejo driver in Woodpecker than in a Woodpecker fork but it is not for me to decide. It's only marginally problematic on my end though.

@earl-warren please point out the value of your implementation over the currently used one?

I pushed an updated version of the Forgejo driver. I understand you see no value in it as long as the Forgejo server does not introduce a breaking change that would make it unusable with the Gitea driver and I respect that. My intention is to keep you informed, not to convince you. I hope this is fine and I would not be offended if you are not interested. I would however appreciate if you do not characterize my work as being redundant or useless.

Looks like it is going to fix  #1505 ...
If I'm allowed I will cherry-pick parts of this pull ...

That's a lot of negative feedback and I'm not sure what to do with it. It is surprising to read that people seem to be angry at me for letting them know of what I'm doing. I'd like to emphasize, again, that I'm completely fine if people are not interested in what I'm doing or how I'm doing it.
Could you just please refrain from characterizing my work as useless, duplicate, bad engineering etc.?

well some of your work is de-facto to duplicate code ...
about "bad engineering", I personaly did not compare it to the existing gitea driver so I cant rate it in any meaningfull way atm
about useless, well the idea to do it in the first place might be, but it's done and now it should be compared.
But as wat i saw is to implement the client without a sdk directly into woodpecker ... I'm rather going with a good, well used sdk, maintained separately if possible (go-sdk of gitea in that case).
tldr: I personally never called your work bad in any way just that it redo what's already done( possible )

There seems to be a misunderstanding about my intent, let me clarify.
I'm doing work to improve Woodpecker for my own benefit as well as to address specific challenges in the context of the Forgejo project.
To behave like a good Free Software citizen, I communicate with the upstream project (on GitHub despite how much I dislike using it). I also make an effort to comply with what I understand of the best practice (not using panic() but returning err in functions is one example) and publish my work under a compatible license (non copyleft in this case).
It is then up to the maintainers to choose how to react. The feedback I got has been abundantly clear: not interested. And this is fine. I think that will change in the future but in the meantime there is no need to repeat. Maybe just leave this PR evolve and come back later?

->  #1733


Running migration SyntaxError: Unexpected token 'export'

Hi,
I created a migration script using the automation via CLI, but then whern I try to apply it using:
datocms migrations:run --destination=main --api-token=xxxxxxI get the error:
Running migration "1670316600_version3.ts"... !
SyntaxError: Unexpected token 'export'
The generated migration script is like this:
import { Client, SimpleSchemaTypes } from '@datocms/cli/lib/cma-client-node';

export default async function (client: Client) {
....
}

Running with macOS Ventura, nodejs v18.2.0
any ideas please?

do you have a tsconfig.json? can you share it?

Sure, here it is, and I forgot to mention, the project is using React 18 and Next.js 13 with the /pages folder:
{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "allowJs": true,
    "skipLibCheck": true,
    "strict": false,
    "forceConsistentCasingInFileNames": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve",
    "baseUrl": ".",
    "paths": {
      "@/blocks/*": [
        "blocks/*"
      ],
      "@/components/*": [
        "components/*"
      ],
      "@/lib/*": [
        "lib/*"
      ],
      "@/queries/*": [
        "queries/*"
      ],
      "@/styles/*": [
        "styles/*"
      ],
      "@/types/*": [
        "types/*"
      ]
    },
    "incremental": true
  },
  "include": [
    "next-env.d.ts",
    "**/*.ts",
    "**/*.tsx"
  ],
  "exclude": [
    "node_modules",
    "test/*",
    "migrations/*"
  ]
}

@marcello-palmitessa can you please try to temporarly replace the content of your tsconfig with the following, and running the migrations again?
{
  "compilerOptions": {
    "declaration": true,
    "importHelpers": true,
    "module": "commonjs",
    "outDir": "lib",
    "rootDir": "src",
    "strict": true,
    "target": "es2019"
  }
}

With that tsconfig I'm getting this error now:
Error: POST https://site-api.datocms.com/plugins: 422 Unprocessable Entity
[
{
"id": "631444",
"type": "api_error",
"attributes": {
"code": "INVALID_FIELD",
"details": {
"field": "name",
"code": "VALIDATION_UNIQUENESS",
"record_type": "Plugin"
}
}
}
]
at /Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+rest-client-utils@1.1.12/node_modules/@datocms/rest-client-utils/src/request.ts:294:19
at step (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+rest-client-utils@1.1.12/node_modules/@datocms/rest-client-utils/dist/cjs/request.js:44:23)
at Object.next (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+rest-client-utils@1.1.12/node_modules/@datocms/rest-client-utils/dist/cjs/request.js:25:53)
at fulfilled (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+rest-client-utils@1.1.12/node_modules/@datocms/rest-client-utils/dist/cjs/request.js:16:58)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
Caused By:
Error
at Client.request (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+cma-client@1.1.12/node_modules/@datocms/cma-client/src/generated/Client.ts:138:21)
at Plugin.rawCreate (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+cma-client@1.1.12/node_modules/@datocms/cma-client/src/generated/resources/Plugin.ts:52:24)
at Plugin.create (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+cma-client@1.1.12/node_modules/@datocms/cma-client/src/generated/resources/Plugin.ts:18:17)
at default_1 (/Users/EPALMIMGB/projects/tangity-website/migrations/1670344958_updateToVersion3.ts:13:46)
at Command.runMigrationScript (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+cli@1.0.20_@types+node@18.11.11/node_modules/@datocms/cli/lib/commands/migrations/run.js:117:23)
at async Command.run (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@datocms+cli@1.0.20_@types+node@18.11.11/node_modules/@datocms/cli/lib/commands/migrations/run.js:84:13)
at async Command._run (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@oclif+core@1.20.4/node_modules/@oclif/core/lib/command.js:80:22)
at async Config.runCommand (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@oclif+core@1.20.4/node_modules/@oclif/core/lib/config/config.js:272:25)
at async Object.run (/Users/EPALMIMGB/projects/tangity-website/node_modules/.pnpm/@oclif+core@1.20.4/node_modules/@oclif/core/lib/main.js:74:5)

thanks @stefanoverna I think I found the issue, in the generated script a plugin was first declared and then deleted, bu thte deletion is an error, the plugin is needed:
console.log('Create private plugin "Scribble Editor"');
newPlugins['48468'] = await client.plugins.create({
name: 'Scribble Editor',
url: 'https://tangity-scribble-editor.netlify.app/',
description: 'Scribble Editor',
package_name: null,
package_version: null,
permissions: []
});
await client.plugins.update(newPlugins['48468'], { parameters: { migratedFromLegacyPlugin: true } });
console.log('Delete plugin "Scribble Editor"');
await client.plugins.destroy('116252');

cool! we should have fixed this bug in the latest CLI version :)


Add support for CIs that do gitlab

Took a look through the docs:

[x] BuildKite does it with not real changes
[x] Jenkins requires some setup, so I sent them to the official docs
[x] TeamCity can but just with GitHub it's very much a "good luck to you"
[x] Buildasaur ( for Xcode ) doesn't support it, so for now, it's skipped

          1 Error

  :no_entry_sign:
  Please include a CHANGELOG entry.

You can find it at CHANGELOG.md.

  Generated by :no_entry_sign: danger


Fix approx derivative types

Reference issue
This is a proposed solution to #18772
What does this implement/fix?
The approcimated gradient of a function will have the same output type as the function itself. In particular, a complex-valued function with real argument(s) will have a derivative/gradient that is also a complex-valued function of real argument(s).
Additional information
I have not managed to run the tests locally due to apparently conflicting dependencies in the environment.yml.

Thank you @KeithWM. Can you post the issue about the environment?

It's going to take a bit of convincing to merge support for complex functions to approx_derivative.
Changes are made to a private function that clearly states it's designed for real functions, not complex. Approx_derivative is a key part of optimise infra supporting minimisers and least_squares, so to reorganise it for complex derivatives is risky.
I think if we're going to have complex derivative support it should be in functionality specifically designed for it.

It's going to take a bit of convincing to merge support for complex functions to approx_derivative. Changes are made to a private function that clearly states it's designed for real functions, not complex. Approx_derivative is a key part of optimise infra supporting minimisers and least_squares, so to reorganise it for complex derivatives is risky.
I think if we're going to have complex derivative support it should be in functionality specifically designed for it.

I see your point. And this change should certainly not be made lightly. I have the suspicion though that whatever way you look at it, it makes more sente to put the difference between two function calls into a matrix that has the same element type as what the function returns. I'm not that familiar with the different types that can be inside a matrix, but I can imagine other types that are broader than float yielding similar issues. For example arbitrary precision numbers.
I feel compelled to admit that my view of how types should be handled is heavily influenced by my familiarity with and love of the Julia language. This might go against the zen of Python. I will refrain from any further comment :-D.

I will refrain from any further comment

Not at all. It's good to get a direction of what's needed, we just need to find how to implement things properly.

Related: statsmodels numdiff includes complex step derivatives which requires complex returns even though the derivative is only for a real function.
However, whether complex step derivatives are correctly computed depends on the user's function. (in statsmodels there is no check for this, the CS derivative might raise an exception, e.g. with scipy norm.cdf or return incorrect results, e.g. if there is an np.abs in the computation)

Not at all. It's good to get a direction of what's needed, we just need to find how to implement things properly.

I didn't mean that I wouldn't comment at all any more, just the I wouldn't start a whole monologue about the virtues of Julia's approach to typing.

Related: statsmodels numdiff includes complex step derivatives which requires complex returns even though the derivative is only for a real function.
However, whether complex step derivatives are correctly computed depends on the user's function. (in statsmodels there is no check for this, the CS derivative might raise an exception, e.g. with scipy norm.cdf or return incorrect results, e.g. if there is an np.abs in the computation)

I had also used the 'cs' option in scipy.optimize.least_squares, but I seemed to suffer from some function there that was, as you say, not capable of working with complex inputs.
How should I interpret the status of the checks?


Air Purifier 2 (zhimi.airpurifier.mc2) has no fan speed control.

Hello there!
The only issue with this device is the absence of fan speed control, the other plugin I've used before this one did have this feature, and it is highly desired.
Would you kindly fix this for the users of this device?

Hi,
this is not an issue! The zhimi.airpurifier.mc2 is an older device which uses the old miio protocol and not the new miot protocol. There is an emulation available to control the device using the miot protocol over the mi cloud but because of that some functionality is not available, that included the fan speed control. There is nothing that can be done with the miot protocol.


Creating AtlasTextures in Godot 4.0 beta 4 causes re-imports every time we reload, and engine runs slower

Godot version
4.0 beta 4
System information
Windows 11
Issue description
I've started using AtlasTextures on all my sprites for the massive VRAM savings.
There are three issues -:

Everytime I open the project now, I see errors like this that eventually go away:

There are reimports every time I open the project

The engine seems to be running on a different setting now. It is generally slower, and specially if I alt tab, it takes it a few seconds to wake up. This happens any time I change the window, and otherwise when navigating the filesystem.

The VRAM savings are good, but slowed development is bad. Thanks for your help.
Steps to reproduce

Use atlastextures in project

Minimal reproduction project
No response

Every time I alt tab and come back, CPU usage shoots up to 30%.

@LakshayaG73 Please upload a minimal reproduction project to make this easier to troubleshoot.
We can't do anything here without a MRP, so the issue will have to be closed in 7 days otherwise.

To create a MRP for this, I will need to a lot of textures, and create atlases. I will do that if needed.
The issue is very evident. Every time I alt-tab, or minimize the godot window, or even play the scene, I am reloading all the textures that point to sprite atlases:

This is happening every time I alt tab! This cannot be normal behavior.

MRP.zip
The project above is small but experiences all the issues as described above even with a single atlas texture. Seems like the feature is broken.
You will experience huge delays when alt tabbing and opening the project.

I can see that this is coming from:

Could it be because this cache mode has been set incorrectly? Shouldn't these resources be cached somewhere?

The cache mode could be set this way due to 'load from file' being called, because the image is not being loaded 'OK'.
Please let me know if this is the right path?

Doing some digging it looks like the hash for TextureAtlas is broken, not sure exactly what's going on but on reload the md5 hash for the atlas file is changed, this would naturally cause the editor to reimport it, and thus force the target file to be reimported, on each entering of the editor, will see if I can find where this occurs and what is going on exactly with that

Ignore all that, it turns out it was way simpler:
Files included from a group are not given a UUID on import, possibly an oversight from past versions I don't know, however import files without one are always queued for import, therefore these atlas files are always reimported, should be straight forward to solve will take a look

Thank you for your efforts, much appreciated

There seems to have been a second issue as well where under some condition (like opening the editor) the target file would be reimported on its own, and then the group would be reimported, which would more or less double the import time under those conditions, so adding a tentative check to prevent files that are groups from being imported twice

Also, for the future I recommend you not include the .godot folder in your MRPs, it can contain information about your computer that could be a security risk, not a big deal in this case but a good practice to keep in mind

Also, for the future I recommend you not include the .godot folder in your MRPs, it can contain information about your computer that could be a security risk, not a big deal in this case but a good practice to keep in mind

Is that actually the case? Please don't spread this kind of information without verifying it beforehand (and reporting it in a dedicated issue if it turns out to be the case).

Oh I'm sorry I thought I had seen that being pointed out before! And considering the issues with files in git repositories

I had understood that file paths external to the res:// filespace was an issue, and there is the reference to the executable in there somewhere

Is that actually the case?

@Calinou I've briefly checked the files from the MRP and this can be found in the .godot\editor\project_metadata.cfg:
[editor_metadata]

executable_path="C:/Users/User/Downloads/Godot_v4.0-beta4_win64.exe/Godot_v4.0-beta4_win64.exe"

The metadata storage in question was added by https://github.com/godotengine/godot/commit/2f151068b5333c133d1a9dffe124bb532ba0a25b, likely for use by the Godot Rider plugin.

I am under the impression that .godot content including project_metadata.cfg is not supposed to be added to the vcs. If so, then what is the issue with the username?

I am under the impression that .godot content including project_metadata.cfg is not supposed to be added to the vcs. If so, then what is the issue with the username?

It's not supposed to be committed to version control, but some people still include it in ZIP archives they create to upload as minimal reproduction projects here (or on other places).
Including the username is problematic for people who use their real name as their OS username and wish to keep their online identity private. Not everyone can change their OS username (e.g. on corporate-owned machines). Even when it's possible, it requires recreating a new and moving everything, which is a time-consuming process.

We discussed it in https://github.com/godotengine/godot/issues/29674 (I managed to find it by searching for "privacy")

Sorry to derail this conversation a bit...
Regarding this issue. The PR from @AThousandShips works great.
There is one more thing that I've noticed. Which may be by design, but I'm not sure. I've noticed that the width of the atlastexture doesn't go below 2048:

Can't help but wonder about the extra dead space here as well. Is this done on purpose?

There is one more thing that I've noticed. Which may be by design, but I'm not sure. I've noticed that the width of the atlastexture doesn't go below 2048:

@LakshayaG73 You mean it doesn't go above 2048? That's indeed the default value which is currently used for the max width by the atlas packer:
https://github.com/godotengine/godot/blob/256069eaf00be2340259f896695014d92b1e22ed/editor/editor_atlas_packer.h#L71

Can't help but wonder about the extra dead space here as well. Is this done on purpose?

Is there an extra dead space though? Looking at the preview you've shown it seems like it's just zoomed out and the preview's transparent background is rendered way bigger then it should (not just behind the actual image) as this rectangle has approximately 2048 / 18556 aspect ratio:

whoops. you're right. thanks for the clarification!

There is one more issue here it seems.
If I use a Mesh to pack these textures together, like I've done in the image above, and try to add the frames to a spriteframes, some frames show up as empty. The reason is that some frames are simply not added to the mesh. This does not occur if you select 'region' when creating the AtlasTexture.
Mesh2D, created with the same frames:

Region, created with the same frames:

The same frames are being skipped every time. Perhaps they are being considered duplicates?

The same frames are being skipped every time. Perhaps they are being considered duplicates?

@LakshayaG73 No idea. But for sure it seems like a seperate problem. So please open a new issue and provide all needed details in there (and an MRP so it could be investigated properly).

https://github.com/godotengine/godot/issues/68350

Could you please remove the "needs testing" label from this @Calinou as there is a PR out. Thank you.


fix: summary panel nrql query issues (Fixes #10)

fixes summary panel disappearing act by replacing the NrqlQuery component with NerdGraphQuery (also utilizes NerdGraphError for convenient error handling)

Additional Info: Repeated fetches with the same Nrql query using the NrqlQuery component fail to return data when NrqlQuery.FORMAT_TYPE.RAW is specified. I did a test with the SINCE clause continuously updating vs remaining static to determine this is an issue. It seems something in the service itself is not responding with data, so we may need to look into this more in case it impacts other Nerdpacks. Switching NrqlQuery out for NerdGraphQuery addressed the issue.

Tidying up styling so summary/charts don't jump as much when different processes are selected. Also fixed some spacing issues by updating Stack props.
Updated Icon.SIZE_TYPE to comply with warning from dev tools console.
Added nr1.json to root of project

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

Looks like all the changes are present in the files now. Not sure why the 4th commit didn't come through in the conversation but ¯_(ツ)_/¯

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


ModuleNotFoundError: No module named 'dlimp'

That module is nowhere to be found

Hi @tekntrash,
Have you completed the setup instructions in the README here? https://github.com/openvla/openvla?tab=readme-ov-file#installation
The pip install -e . step in the openvla base directory should download dlimp as it is listed as a requirement in pyproject.toml.
-Moo Jin


Jodd Http content type is multipart/form-data Chinese error

If the content type is multipart / form-data, Received Chinese parameters is '?????'

Hey @rmbzlib can you please provide the example? So far we didn't have any issue with encodings, i.e. it was read correctly. Which methods do you use for getting the content?

HttpRequest.post("http://requestb.in/1lxouwe1")
                .form(
                        "zh_cn", "中文 中文 中文",
                        "file", new File("/Users/qufenqi-xun/Desktop/中文中文中文.txt"))
                .send().bodyText();

http://requestb.in/1lxouwe1?inspect

@rmbzlib As per HTML5 spec, default encoding for multipart/form-data is UTF-8. This default applies as you have specified a different one (typically in the Content-Type header or part header). My best guess is that your file is actually encoded in GB-2312.

@slandelle file and parameters encoding is UTF-8,if use Postman not the problem,I have set the encoding.
HttpRequest.post("http://requestb.in/1lxouwe1")
                .form(
                        "zh_cn", "中文 中文 中文",
                        "file", new File("/Users/qufenqi-xun/Desktop/中文中文中文.txt"))
                .charset("UTF-8")
                .send().bodyText();

It's settled
update 3.7.1
thanks
@igorspasic @slandelle


GIF not working

avatarView?.loadImage(gifUrl)

Hey, @iostyle. Sorry for the late reply. Which module do you use? Coil or Glide?

Hey, @iostyle. Sorry for the late reply. Which module do you use? Coil or Glide?

Coil


feat(fee): add n_events field to TransactionExecutor

This change is

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (fbea143) 69.44% compared to head (4bea666) 69.40%.

Files
Patch %
Lines

...ates/native_blockifier/src/transaction_executor.rs
20.00%
4 Missing :warning:

crates/blockifier/src/state/cached_state.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@               Coverage Diff                @@
##           main-v0.13.1    #1516      +/-   ##
================================================
- Coverage         69.44%   69.40%   -0.05%
================================================
  Files                59       59
  Lines              8127     8134       +7
  Branches           8127     8134       +7
================================================
+ Hits               5644     5645       +1
- Misses             2024     2030       +6
  Partials            459      459

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


Nix has stale serpl version

Currently nixpkgs has serpl version 0.1.34, latest version is 0.3.3 (at the time of opening this issue).

$ serpl --version
serpl 0.3.3-VERGEN_IDEMPOTENT_OUTPUT (1980-01-01)
Authors: Yassine Bridi<EMAIL_ADDRESS>Config directory: /Users/matt/.config/serpl
This seems to be current on the latest nix packages.


How to get calculated value from a formula

getValue or getData gives for example "=B8*C8"  but I would like the calculated value of have an option wich one.

If you initiate the spreadsheet with copyCompatibility:true, it will behave export, copy, with values not formulas.

I'm having the same problem, used copyCompatibility:true but still getData() giving the formula string

var tab = jexcel(document.getElementById('spreadsheet'), {
    data:[[1,2,3,'=C1+10'],[1,2,3,4],[1,2,3,4]],
    copyCompatibility:true,
});
console.log(tab.getData()[0][3])

This prints =C1+10 not 13

Please, update from the master repository to use the getData. To get one cell only you can use:
tab.getValue('D1', true);

But now with the updated version I can't use the corner drag to autofill/copy formulas

Yes, you can. Please take a look on the examples.
https://bossanova.uk/jexcel

Bear in mind to update jsuites as well.

Okay, I used this https://github.com/paulhodel/jexcel/blob/master/dist and https://github.com/paulhodel/jsuites/blob/master/dist files and used this
var tab = jexcel(document.getElementById('spreadsheet'), {     data:[[1,2,3,'=C1+10'],[1,2,35,4],[1,2,12,4]],     copyCompatibility:true, });
But now if I drag down from the corner of last column of the first row it fills as 14 and 15 instead of 45 and 22

So, I kept copyCompatibility:false and used myTable.records.map(x=>x.map(y=>y.innerHTML)) which gives the data for now.

So, I kept copyCompatibility:false and used myTable.records.map(x=>x.map(y=>y.innerHTML)) which gives the data for now.

it works. thanks for the solution


[FEATURE REQ] Ability to consume HttpClientOptions in XHttpClientBuilders.

When user wants to configure http client options they have two ways to achieve it. One is via HttpClientOptions passed to service builder which are then applied to implicit http client. The other option is to build HttpClient and pass that to builder. However the later requires specific knowledge about particular HttpClient implementation as the builders don't offer uniform API. We should consider adding ability to pass HttpClientOptions to each of http client builders.

Closing as this is consumable through the HttpClient provider APIs which is where this is more usable. The HttpClient builders allow for finer control than HttpClientOptions allows, given they are specific to the given implementation rather than being generalized.


[FEATURE] Publish OpenSearch Connector on the Tableau Exchange

It'd be great if users could access the OpenSearch Tableau connector directly from the Tableau application and listed on Tableau Exchange.

@dai-chen, would jdbc label be more appropriate? The connector now uses JDBC not ODBC.

Sorry, I wasn't aware of that. Re-label as JDBC issue.


Compiler should convert binary string to byte array

As mentioned in https://github.com/CityOfZion/neo-python/pull/438  I believe neo-boa needs some improvements, and there's an opportunity here.
Currently, both codes generate the same opcodes:
OWNER=b'031a6c6fbbdf02ca351745fa86b9ba5a9452d785ac4f7fc2b7548ca2a46c4fcf4a'
OWNER="031a6c6fbbdf02ca351745fa86b9ba5a9452d785ac4f7fc2b7548ca2a46c4fcf4a"

The opcode is: PUSHBYTES66<PHONE_NUMBER>63366662626466303263613335313734356661383662396261356139343532643738356163346637666332623735343863613261343663346663663461 ### "031a6c6fbbdf02ca351745fa86b9ba5a9452d785ac4f7fc2b7548ca2a46c4fcf4a"
In other words, it assumes the input is a string composed of ascii chars. It would be nice if the binary input was treated like: OWNER=b'\x03\x1alo\xbb\xdf\x02\xca5\x17E\xfa\x86\xb9\xbaZ\x94R\xd7\x85\xacO\x7f\xc2\xb7T\x8c\xa2\xa4lO\xcfJ'
(which is the output of binascii.unhexlify(OWNER), if OWNER is binary string)
I don't know if this information is available (binary string vs pure string) for byteplay, etc, but this is possibly a transparent solution for the user.

I believe @canesin will also support this, if possible to be implemented.

@metachris or @localhuman .. the idea is to remove the extension from the VM and replace it for a helper function into the compiler, this way support can be kept but behavior of the generated AVM is the same.

It doesn't make very much sense.  a binary string, eg b'abmq' is not necessarily one that an be unhexlified ( especially a binary string with an odd amount of characters ).  If we arbitrarily unhex binary strings that can be unhexxed to something like b'\x01\xaf etc, then we will be introducing a non-transparent api.
The main thing I have thought about in regards to this is that the VM should have opcodes ( or there should be interop methods) to hex and unhex a string or string of bytes.

Not sure this has a strong enough use-case to warrant a backwards incompatible change. Would break applications that used this part before.
Perhaps a helper the other way around, to easily convert your bytearray into the current standard format?

Thats true, not all bitstrings can be unhexlified, too bad, that would be beautiful :) anyway, keeping compatibility is also quite important, so a helper method HexToBytes will do the job. But still its better to accept a bitstring perhaps or both (string and bitstring) since its a helper it doesnt.matter on the avm. Thanks for the attention.

on that line hex, unhex could be a compiler level tool while it is not opcodes or interop ? this way we can actually start something: 1st as helpers -> interop -> opcode .. depending on the usage/popularity.

https://github.com/neo-project/neo/issues/263

Guys, you know in C# there's already a helper to do it... something like: const byte[] b = "abcdabcd".HexToBytes();
The generated opcodes are just abcdabcd, and could solve at least the problem above... but the discussion is nice to have something in lower levels for Neo, it opens more possibilities to build a safer smart contract code.

That would be a helper method for string literals, but it wouldn't work for actual variables.  Because of that I think it would be confusing/ misleading.
So mybytes = "abadad".HexToBytes() would work but mybytes = myvar.HexToBytes() would not


about page error

When I check the result _site, it doesn't included js folder.
check it plz
I solved this problem how did copied it in _site folder

@bluebamus There is a js folder in assets - see https://github.com/jekyller/jasper2/tree/gh-pages
Are you referring to some other folder?


Ignore idea and vscode files.

Maven use src/main/java default, I use src/main/include followed you.

src/main/include and src/main/c are for c++ code by nar-maven-plugin, you should use src/main/java for java code


feat: use Dockerhub API to poll for digest change

Use Dockerhub API (docs) instead of checking manifest from Docker registry (docs)
so each poll does not count as an image pull. This is needed as we got warning from
docker support of exceeding docker rate limits.
Mock Dockerhub server in tests.

Updated note on using DinD for integration test: https://community.codefresh.io/t/mounting-docker-socket-in-composition-step/615/2
Short answer: not possible with SaaS codefresh


docs/readme: clarify CUDA requirement for examples

@homu r+

:pushpin: Commit 0bf239d has been approved by hobofan

:zap: Test exempted - status


[BUG] Scanner Keeps getting Stuck :/

⬇⬇ Fill in the forms below where you can answer/applies to you. ⬇⬇
What happed? What is seems to be a bug?
Please describe what seems to be an issue:

For Characters, the Scanner gets lost and ends up opening the "Change Avatar Menu"
For Weapons and Artifacts, the Scanner crashes after a few Weapons/Artifacts
For Character Ascention mats, seems to work fine
For Materials, skips to the bottom, scans a few materials in reverse then stops

For more Details look here:
https://www.youtube.com/watch?v=FPwfD44hxvA
How to reproduce
Please describe how can WE reproduce the issue. Feel free to add your log folder (by zipping and dropping here) to speed up our processes:
Click "Scan Genshin"
Expected behavior
Please describe what you expect from Inventory Kamera or output files:
Info about your device

Device Operating System : Windows 11
Genshin Impact Version : OSRELWin2.6.0_R6708157_S7320343_D631353
Inventory Kamera Version : V1.2.5
Screen resolution affected : 1920x1080 Windowed

Checklist before sending issue
Check the tickbox by replacing [ ] with [x] (in editor) or tick it after leaving the Text Editor:

[x] I am using the latest version of Inventory Kamera or/and have clicked Update Lookup Tables to update the database.
[x] I am using English as a game language
[x] I am using the 16:9 or 16:10 window screen and leave the screen in the Paimon menu before starting the application.
[x] I tried setting the delay to maximum and still experiencing issues.
[x] I have correctly set my character and inventory keys in Inventory Kamera
[x] I do NOT have any image filtering softwares enabled for Genshin (HDR, Image Sharpening, etc.)

If you cannot do any of the above, please describe why.
Additional Notes
Add any other context about the problem and log files/report here.
logging.zip

Yup, can confirm restarting the Game fixed everything. If you want to test IK on Ultrawide, i can give you feedback/screenshots for UI Layout etc


imagepolicy in portieris namespace / if portieris in docker.io

When the portieris image is in docker.io the policy becomes

name: /portieris:0.6.0
policy: null

with a leading / so will fail to match, template issue.

I think portieris:0.6.0 or docker.io/portieris:0.6.0 should work?