Add Replace Query Regex Middlewares

What does this PR do?
Adds 2 new middlewares:

ReplaceQueryRegex: To modify query strings
ReplacePathQueryRegex: To modify paths and query strings

Motivation
This PR carries on the work started with #4000
More

[x] Added/updated tests
[x] Added/updated documentation

Additional notes
Co-authored by @holygeek / @cention-nazri

I am troubled with ReplacePathQueryRegex because we also have ReplacePathRegex.
I wonder if we could merge it with ReplacePathRegex :thinking:

I agree with @ldez, a merge between these middlewares seems to be more appropriate.

I wonder if we could merge it with ReplacePathRegex 🤔

Yes. I will make this happen. It makes sense to me too.
Will aim for:

ReplacePathRegex => Path and query modifier
and
ReplaceQueryRegex => Query only modifier

@ldez @jbdoumenjou I looked into updating this PR, but I ran into the design reason for the difference between ReplacePathRegex and ReplacePathQueryRegex:
ReplacePathRegex modifies the path of the request, but ReplacePathQueryRegex only allows for the regex to match and use parts of the path to modify queries. ReplacePathQueryRegex does not allow for the path to be modified in this manner.
ReplacePathRegex operates on *http.Request.URL.Path, and ReplacePathQueryRegex operates on http.Request.RequestURI, and I'm not sure there is a good way to merge the two without incorporating breaking/design changes.
Thoughts?

Sorry for the delay @dtomcej .
So, as we don't use the same data to apply those middlewares, I agree that it may be confusing to merge them.  Perhaps we need to define two better names to avoid any confusion. WDYT ?

Perhaps we need to define two better names to avoid any confusion

Yessir. I think this would help out immensely.

We can have only one middleware with an option like "src/target/criterion/..." to define the request part used. But it seems to be no compliant with the "spirit" of the middleware responsibility.
So, we can factorize the code between this two middlewares and try to find a better name (but I didn't managed to do it). It is just some non blocking comment, so I will accept the PR.

Was the ability of changing the Query part of the URL as part of the modifiers ever implemented?
Thanks

Some notes:

ReplacePathQueryRegex: use the path and the query to modify the query string only, there is no modification of the path.
I continue to think that replacePathRegex, replaceQueryRegex, replacePathQueryRegex can be a single middleware.

I will continue to rewrite the PR, maybe I will open an another PR.

After discussion with the team, we think that the approach is too naive which will produce a high complexity of use.
So we will decline this PR but we keep the idea and for that I created an issue on this subject (#6276).
If you want to participate in the reflection on the subject or explain your use cases, I invite you to express yourself in this issue (#6276).
Thank you for your understanding.


Fail to start docker container after update to version 2.12.0 with error migration failed

Describe the bug
I got this error message
wakapi  | {"time":"2024-10-05T08:56:15.96090864Z","level":"WARN","msg":"failed to parse reverse proxy ip ranges"}
wakapi  | {"time":"2024-10-05T08:56:15.961549236Z","level":"INFO","msg":"disabling data retention policy, keeping data forever"}
wakapi  | {"time":"2024-10-05T08:56:15.961681413Z","level":"WARN","msg":"with sqlite, only a single connection is supported"}
wakapi  | {"time":"2024-10-05T08:56:15.961882919Z","level":"INFO","msg":"Wakapi","version":"2.12.0\n"}
wakapi  | {"time":"2024-10-05T08:56:15.961992182Z","level":"INFO","msg":"starting with database","dialect":"sqlite3"}
wakapi  | {"time":"2024-10-05T08:56:15.965162398Z","level":"INFO","msg":"potentially running migration","name":"20201103-rename_language_mappings_table"}
wakapi  | {"time":"2024-10-05T08:56:15.967797735Z","level":"INFO","msg":"potentially running migration","name":"20201106-migration_cascade_constraints"}
wakapi  | {"time":"2024-10-05T08:56:15.968027674Z","level":"INFO","msg":"not attempting to drop and regenerate constraints on sqlite"}
wakapi  | {"time":"2024-10-05T08:56:15.968103586Z","level":"INFO","msg":"potentially running migration","name":"20210202-fix_cascade_for_alias_user_constraint"}
wakapi  | {"time":"2024-10-05T08:56:15.968189686Z","level":"INFO","msg":"not attempting to drop and regenerate constraints on sqlite"}
wakapi  | {"time":"2024-10-05T08:56:15.968266109Z","level":"INFO","msg":"potentially running migration","name":"20220403-drop_user_project_idx"}
wakapi  | {"time":"2024-10-05T08:56:15.969374507Z","level":"INFO","msg":"no need to migrate","name":"20220403-drop_user_project_idx"}
wakapi  | {"time":"2024-10-05T08:56:15.96947794Z","level":"INFO","msg":"potentially running migration","name":"20220930-drop_heartbeats_entity_idx"}
wakapi  | {"time":"2024-10-05T08:56:15.970888462Z","level":"INFO","msg":"potentially running migration","name":"20221002-fix_summary_id_types"}
wakapi  | {"time":"2024-10-05T08:56:15.971040215Z","level":"INFO","msg":"potentially running migration","name":"20221028-fix_heartbeats_time_user_idx"}
wakapi  | {"time":"2024-10-05T08:56:15.973020971Z","level":"ERROR","msg":"migration failed","error":"attempt to write a readonly database (8)"}

System information
Please provide information on:

Wakapi version: 2.12.0
Operating system

If Docker: ghcr.io/muety/wakapi:latest

Database: SQLite

I found the solution

run as root user

chmod -R 777 data/wakapi.db
UID=${UID} GID=${GID} docker compose up -d

Can you please give instructions on how to reproduce this? Perhaps provide the exact docker run command you're using? When I simply run the official images, Wakapi boots up fine in the container.

Here is my docker compose yaml on Debian 12 64bit
I'm running docker as root user
cat docker-compose.yml
services:
    muety:
        ports:
            - '3000:3000'
        environment:
            - WAKAPI_PASSWORD_SALT='xxxxx'
            - WAKAPI_EXPOSE_METRICS=true
        volumes:
            - './data:/data:rw'
        container_name: wakapi
        image: 'ghcr.io/muety/wakapi:latest'
        restart: always

Did you try one of the solutions proposed by @YC? Did any of these work?


How to set selected drawer item background color to primary?

All the examples show a light grey background but none show the background using primary color
Example:

Requested example:

Also would be nice to have examples with icons, like this

I believe you are just looking for the activated prop

Please post questions on either stack overflow or on the Discord channel.


BIGTOP-3397. Building Spark is failing on CentOS 7 for ARM64 and PPC64.

snappy-java must be upgraded to 1.1.8 to fix build issue on aarm64 and ppc64le.
https://issues.apache.org/jira/browse/BIGTOP-3397

The patch worked for me on CentOS 7 aarch64 (on AWS EC2 A1).

+1, thanks @iwasakims!

Thanks for the patch and merging, @iwasakims , @sekikn


Can you add Printing?

I love this library but can you please add a printing api into it?

Hi thanks for the feedback.
Yes that's coming up in the next couple of versions.
I already have some new functions available for it.
After I deploy the mac apis, printing will be available too.

Version 1.0.0 now has printing.
Please upgrade to the latest package to get access.
You can find the documentation for it here.
https://thearchitect123.github.io/ArtifactsDocProduction/develop/kotlin/multiplatform/kmpessentials/modules/printing/


Remove setavatar command

This PR removes the setavatar command from the core framework.
Closes #23

Good for 2.0 per discussion in #23


Fast Server for tracking

My postal server is behind a firewall
I have configured both, the domain track.xxxx.com and the CNAME click.xxxx.com pointing to track.xxxx.com. I can check with nslookup from the postal server that both are properly refreshed and active in internet. In the postal.yaml file I have

But when I create the tracking domain , the application keeps saying :

Where is my mistake?? Please help me pointing it out.
Best regards

Found the problem, In the postal.yml file should be track.xxxx.com instead of click.xxxx.com


Nuget push to feed will show error "503 (Service Unavailable)"

Nuget push to feed will show error "503 (Service Unavailable)"
Uploading of large nuget packages is still an issue and therefor this way of working is not yet possible for many of us.
See details in my problem report on https://github.com/microsoft/azure-pipelines-tasks/issues/13548#issuecomment-749652324
Would be great if the package can be split in smaller chuncks or if the Azure DevOps team can increase the max. limits

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

ID: 10459944-bcf8-c5fa-d180-9fb4bb3722da
Version Independent ID: 29ae9d16-ed7c-92d8-0300-4fc46c777ddf
Content: Build automation that uses Microsoft-hosted agents and Azure Pipelines - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/hosted-build-automation.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @jorisdg
Microsoft Alias: jorisde

@MZade, thank you for providing this feedback.
@jorisdg, please review and consider updating the content based on this issue.

@MZade, thank you for providing this feedback.
@jorisdg, please review and consider updating the content based on this issue.

This is not a documentation issue. @sericks007 please close the issue.
To not leave the question hanging anyway: we're looking at long-term solutions for the nuget packages. Splitting them up is the obvious choice, however it requires more features to be built into the nuget restore features of our pipeline. That work is being reviewed.
You can upload the nugets from a dev VM you have hosted in Azure (which will be fast).

This is not a documentation issue. @sericks007 please close the issue.
To not leave the question hanging anyway: we're looking at long-term solutions for the nuget packages. Splitting them up is the obvious choice, however it requires more features to be built into the nuget restore features of our pipeline. That work is being reviewed.
You can upload the nugets from a dev VM you have hosted in Azure (which will be fast).


Cannot remove an observer

Hi! when a function with this code is called:
self.publicProfileVC = [[PublicProfileVC alloc]  init];
[self.navController pushViewController:self.publicProfileVC animated:YES];
I'm getting this error:
Uncaught exception: Cannot remove an observer <PublicProfileVC 0x182590f0> for the key path "segmentTopInset" from <PublicProfileVC 0x182590f0> because it is not registered as an observer.
What may be causing this error? thanks!

This is very strange, i need detail info so that i can figure it out.


PointSource enforces flux as float

Related to this Zodiax issue, where the gradients of a float returns None.
dl.PointSource enforces flux to be a float.
"""the dLux source code"""
class PointSource(Source):
    ...
    def __init__(...):
        # Position and Flux
        self.position = np.asarray(position, dtype=float)
        self.flux = float(flux)

        if self.position.shape != (2,):
            raise ValueError("position must be a 1d array of shape (2,).")

This issue was fixed by setting it to a jax array in the same fashion as the position:
"""my alteration of the dLux source code"""
class PointSource(Source):
    ...
    def __init__(...):
        # Position and Flux
        self.position = np.asarray(position, dtype=float)
        self.flux = np.asarray(flux, dtype=float)

        if self.position.shape != (2,):
            raise ValueError("position must be a 1d array of shape (2,).")
        if self.flux.shape != ():
            raise ValueError("flux must be a scalar value.")

I can push this if need be or would it be better just to fix it in Zodiax?

This would be a Zodiax side fix, probably either by pre-processing the leaves into arrays or using a different equinox filter function. Will need to check how it interacts with jax.grad calls though to make a final call


Landing page for developers

Different options will be offered in order for them to verify their apps on zap.store:

signing up for DM reminders of "hey you just made a release, sign your app/release/metadata" by visiting this website or using this CLI tool (active)
if using Github, integrate a Github action that takes either an nsec or connects to a bunker with an auth token (set and forget)
sign a NIP-69 linking their PGP key or APK certificate (set and forget), used in combination with the delegation tags (p/zap)

Splitting into three separate issues:

[ ] signing up for DM reminders of "hey you just made a release, sign your app/release/metadata" by visiting this website or using this CLI tool (active) https://github.com/zapstore/zapstore/issues/5
[ ] if using Github, integrate a Github action that takes either an nsec or connects to a bunker with an auth token (set and forget)
[ ] sign a NIP-69 linking their PGP key or APK certificate (set and forget), used in combination with the delegation tags (p/zap)

https://zap.store/developers
kind of a shit page but gets the job done for now


[BUG] CodeCov throws Forbidden Error

🐛🐛 Bug Report
When we browse files through the CodeCov website to check code coverage for each file separately, the file isn't displayed. Instead, we get a Forbidden screen.
Expected behavior/code
Previously it was possible to check the individual code coverage of the files and see which lines are not covered by the unit tests. This simplified the process of increasing code coverage.

@kristinagrig06

@DebadityaPal, can you please try to log out from codecov, remove codecov access from here, then log back into Codecov and try to access that page. Please let me know if the problems persist.

Thanks a tonne @kristinagrig06! it works now.


The Context Menu text of TextField truncated on Flutter v13.3.5 when using larger text on iOS 16.7

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I have updated the Flutter version recently from v3.7.1 to v3.13.5. I have found that the context menu text has been truncated in the text editor on v3.13.5 but not in v3.7.1 on iOS 16.7.
Step to reproduce:

Enlarge the text on iOS by Settings -> Accessibility -> Display & Text Size -> Larger Text
Using fvm to install Flutter v3.13.5
Run the Flutter Project with TextField in the code.
Copy some text in other app.
Go back to the Flutter app and long press the TextField.
The "Paste" button in the context menu is truncated. (You can find the screenshots in the "Screenshots or Video" section)

Expected results
The text of the context menu should not be truncated even I have enlarged the OS font size.
Actual results
The text of the context menu has been truncated if I have enlarged the OS font size.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
Flutter v3.7.1
Flutter v3.13.5

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.5, on macOS 13.5.2 22G91 darwin-arm64, locale en-HK)
    • Flutter version 3.13.5 on channel stable at /Users/lmshek/fvm/versions/3.13.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 12fccda598 (3 weeks ago), 2023-09-19 13:56:11 -0700
    • Engine revision bd986c5ed2
    • Dart version 3.1.2
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/lmshek/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/lmshek/Library/Android/sdk
    • Java binary at: /Applications/Android [Studio.app/Contents/jre/Contents/Home/bin/java](http://studio.app/Contents/jre/Contents/Home/bin/java)
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google [Chrome.app/Contents/MacOS/Google](http://chrome.app/Contents/MacOS/Google) Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android [Studio.app/Contents](http://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.12+0-b1504.28-7817840)

[✓] VS Code (version 1.70.2)
    • VS Code at /Users/lmshek/Documents/Visual Studio [Code.app/Contents](http://code.app/Contents)
    • Flutter extension version 3.60.0

[✓] Connected device (4 available)
    • Lok Man’s iPhone (mobile) • 00008030-001D18A23E46802E            • ios            • iOS 16.7 20H19
    • iPhone 12 Pro (mobile)    • D60CF662-001D-4588-93D4-98805AA488B7 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)           • macos                                • darwin-arm64   • macOS 13.5.2 22G91 darwin-arm64
    • Chrome (web)              • chrome                               • web-javascript • Google Chrome 117.0.5938.149

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @lmshek. Thanks for filing this. I can reproduce this on the latest master and stable channels as well following the steps highlighted above.

With Normal Text
Native (large Text)
Flutter App (Large Text)

stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.13.7, on macOS 13.0 22A380 darwin-arm64, locale
    en-NG)
    • Flutter version 3.13.7 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2f708eb839 (3 days ago), 2023-10-09 09:58:08 -0500
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • 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
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.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
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.82.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5556                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • iPhone XS (mobile)          • 3CFD1268-0D52-486E-A12E-6E136E972D94 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 117.0.5938.149

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 3.16.0-10.0.pre.72, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.16.0-10.0.pre.72 on channel master at /Users/damilolaalimi/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d479b5e2c6 (72 minutes ago), 2023-10-13 03:35:07 -0400
    • Engine revision b59d779d4f
    • Dart version 3.3.0 (build 3.3.0-20.0.dev)
    • DevTools version 2.28.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • 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 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.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
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.82.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5556                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone XS (mobile)          • 3CFD1268-0D52-486E-A12E-6E136E972D94 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 117.0.5938.149

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I believe this has already been fix by https://github.com/flutter/flutter/pull/133386. The earliest version with the fix is 3.15.0-6.0.pre

Thanks for your input @LongCatIsLooong. Tested this on the latest master channel and I can see this doesn't reproduce there.
Closing as this has been fixed in the latest versions.


Allow representations on @api-return to be optional

When you're doing a DELETE, you usually aren't returning any data from the request, so it doesn't make sense to document it with a representation.
/**
 * Delete a movie.
 *
 * @api-label Delete a movie.
 *
 * @api-uri:private {Movies} /movies/+id
 * @api-uriSegment {/movies/+id} {integer} id Movie ID
 *
 * @api-contentType application/json
 * @api-scope delete
 *
 * @api-return:private {deleted}
 *
 * @api-throws:private {404} \Vimeo\Mill\Examples\Showtimes\Representations\Error If the movie could not be found.
 */

Doing this, we'll also be able to get rid of the (very) confusing noMethod representation config declarations.


Update babel plugin name

Fixes #5
Linting and tests all pass.

@pcguru I was having trouble with switching over so I just left it. I can take a look at it again.

I've further illustrated my trouble with that in this issue on the babel-plugin-root-import repo: https://github.com/entwicklerstube/babel-plugin-root-import/issues/66

That's a breaking change!
IMO, you should either update package.json name as well and republish it, just like babel-root-import did. eslint-import-resolver-babel-plugin-root-import would make sense.
Or you should at least jump an entire version — to 1.0.0 — rather than a subversion like it is now. You don't want to break current app linters who are targeting ^0.0.2

@pcguru It seems that they suggest we keep using the old version. https://github.com/entwicklerstube/babel-plugin-root-import/issues/66#issuecomment-283379803
@zurfyx good point, I will do this.

I'll probably fork and republish if this isn't merged in sometime in the next few weeks.

@adrianmcli is your branch working correctly?  I'd love to get a working version of this bad boy if possible.

@GollyJer yeah it should be working, I've been using it for the past few weeks. Give it a go and let me know.

@adrianmcli Thanks for the help.
I can't get it to work.  Clearly I don't know what I'm doing.
Here's what I tried.
"eslint-import-resolver-babel-root-import": "git://github.com/adrianmcli/eslint-import-resolver-babel-root-import.git#718368a629ef7c181a12a647224823faba5ac5d3"
.eslintrc
settings: {
    'import/resolver': {
      'babel-plugin-root-import': {
        rootPathPrefix: '~',
        rootPathSuffix: './_app',
      },
    },
  },

.babelrc
"plugins": [
    [
      "babel-plugin-root-import", // https://github.com/entwicklerstube/babel-plugin-root-import
      {
        "rootPathPrefix": "~",
        "rootPathSuffix": "./_app"
      }
    ]
  ],

@GollyJer you can see how I use it here: https://github.com/adrianmcli/next-boilerplate

@adrianmcli Working perfectly, thanks!
It looks like others are taking a crack at it too.

@GollyJer haha, maybe it's time that I fork and publish an actual working option.

@adrianmcli yeah, take a look at the version by @rmacklin.

Whoa, wasn't expecting anyone to see my fork, haha. I got that fork working on Thursday and was thinking about publishing it, but then I ended up switching approaches and dropping babel-plugin-root-import for a symlink inside node_modules because I liked the simplicity of that approach (which makes things like eslint and editor import detection "just work"). That said, I'd be happy to combine efforts on an updated version of this plugin.
A few things to note about your (@adrianmcli's) changes:

This line wouldn't handle if someone was using the plugin shorthand that allows you to omit the babel-plugin- prefix (i.e. if their plugins config had root-import rather than babel-plugin-root-import).
Also, this line would need to be updated to maintain support for babel config declared in a package.json.

But another thing I realized is that the getConfigFromBabel function doesn't handle if the plugin is only declared in the env section of your .babelrc. For that reason, one change I was trying on my fork was dropping getConfigFromBabel altogether. Instead, the options would be passed explicitly to the eslint plugin config (which can still be DRY if you're using .eslintrc.js and .babelrc.js with babel 7).

@rmacklin Could you provide a link to details on the symlink approach? I'm migrating a client app that used webpack path resolving to use server-side rendering and I need a server-workable path solution. Relative paths are just a nightmare with our feature-focused directory structure.
Thanks in advance!
Best
Michael

Ooh, some details around symlink, here: https://gist.github.com/branneman/8048520
A bummer that it's a solution that can't quite be committed, though.

@mherodev, a symlink can be committed actually, although I don't do that. Instead, I create the symlink in a postinstall script (and remove it in a preinstall script, so that it doesn't exist during the actual installation step, since that can cause issues).

@rmacklin I don't hate the preinstall/postinstall script approach!
I took the solution available on the linked gist and updated it to use fs.stat over the deprecated fs.exists call:
    "preinstall": "node -e \"var d='node_modules/src',fs=require('fs');fs.stat(d,function(e){!e&&fs.unlinkSync(d)});\"",
    "postinstall": "node -e \"var s='../src',d='node_modules/src',fs=require('fs');fs.stat(d,function(e){e&&fs.symlinkSync(s,d,'dir')});\"",

This seems pretty straightforward and works. Thanks for the insight!

Seems like that'd work 👍 . In my project I just run a bash script rather than doing it in node, since I'm fortunate to not have to worry about cross-platform inconsistencies.

Any plans on this being merged or published elsewhere under a new name?

Anyone wants to be added to the repos'  maintainers? I'm no longer using this plugin.

We're still using. I can take if you don't mind.

@GollyJer added you as a collaborator on GitHub

Can someone merge and push it finally?

@BjornMelgaard I think @GollyJer is your best bet right now.

@GollyJer, you are collaborator now, can you publish this fix))
P.S. @adrianmcli, thanks for boilerplate example, finally got it working)

I'm on vacation until the 12th. Will merge this asap.

This is now merged and released.
I've never published to NPM.  Is there a simple way to pass that torch or will you need to do it @olalonde?

@GollyJer glad to see this finally merged. You'll need access to publish the package on NPM. @olalonde will likely have to give you permission to do this. If he already did, then just bump the version number (while following SemVer) and do npm publish.
If you've never published anything on NPM before, I highly recommend you publish a test project just so you can see how the process works.
Thanks again for merging!
P.S. For those that want to use this immediately, you can replace the version number in your package.json with a link to this repo.


Remove deprecated value-generation contracts

HHH-18186 - Remove @GeneratorType
HHH-18188 - Remove GenerationTime and its uses

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

Fantastic!!


support changing TLS version in Web Apps

Proposed command:
az webapp update tls -n  -g  -v <1.0 or 1.1 or 1.2>

There is already a webapp update command.  Simply add --tls to that command.

@tjprescott  yes, that is the idea to add --tls  to the existing update command

This needs to release end of  March to early April so setting for milestone 35 that will release April 10th

This is a config setting so will be az webapp config set --min-tls-version


Fixing display name

What does this PR do?
Fixes display name for integrations in order to migrate them to the declarative format.
Motivation
Integrations migration

Your remarks are valid greg, but for the first transition, we have no choice but to set the name as it as internally. Afterwards we will be able to change the name


Bug 1726894: Properly setting "maxWaitTime" environment var.

Image registry expects REGISTRY_OPENSHIFT_REQUESTS_READ_MAXWAITINQUEUE
and REGISTRY_OPENSHIFT_REQUESTS_WRITE_MAXWAITINQUEUE to be of type
time.Duration. This patch renders only the Duration property of a metav1.Duration
struct.

/bugzilla refresh

/bugzilla refresh

/bugzilla refresh

/bugzilla refresh

/cherrypick release-4.1

For reference:
Bug with target as 4.2:
https://bugzilla.redhat.com/show_bug.cgi?id=1726894
Bug with target as 4.1:
https://bugzilla.redhat.com/show_bug.cgi?id=1714168

/assign @adambkaplan @coreydaley @dmage

@ricardomaraschini How about a test that verifies the new behavior, not crashing when those values are set.

@coreydaley yes, you are absolutely right. Implementing it.

@coreydaley I have added tests.

/approve

/lgtm

/retest
infrastructure flake


rstudio 2023.12.1+402 -> 2024.04.0+735

Description of changes
Updated Rstudio to the latest version. Now, two versions of Node are needed, one for building, and one as a runtime dependency. It was enough to create the appropriate directory. I also refer to the commit in question in the default.nix file.
There was also a regression that was introduced that made yaml-cpp not be found, but this was solved this morning with https://github.com/rstudio/rstudio/commit/04de8ca8b83dcc7fee9fd65e6ef58c372489d5ef so I've backported the patch. This will need to be removed for the next release.
Builds and runs on x86 linux. Visual mode for editing markdown documents still loading forever though.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg eval

There's a new release, I'll update the PR to reflect this tomorrow
https://github.com/rstudio/rstudio/releases/tag/v2024.04.1%2B748


Add documentation to explain how the HTTPS certificate is automatically renewed

The "Resource" section should contain more documentation about our own server, and how it's setup. This probably deserves it's own category. One such topic in this category would be the HTTPS certificate used by our website, and how it is renewed automatically.
The documentation for this should explain how it's setup, enough to give some idea where to look in case something goes wrong.

The server setup has been automated using ansible, and is located in a separate repo. Whether to make this repo
I'll close this for now, as it's not clear at this time that anybody wants these scripts to be public or mentioned on the website. At least now they're there for the society executive to reference.


Expose lighter nix expression for deploys

When I deploy with ob deploy push, I can deploy to machines that are quite small.  But if I'm on a small machine and trying to get the cached binary via something like nix-build -A exe, it downloads a huge amount and the machine runs out of RAM.  This isn't a build issue because I'm doing this with a cache that has everything needed.  It would be nice if there was a discoverable attribute in the nix expression to facilitate this kind of pulling of the lightweight nix expression from a cache that has it rather than requiring ob deploy push and all the machine switch / etc that comes with it.
My motivation here was wanting to deploy an obelisk app on a small machine that already was being used for other purposes and therefore was not suitable as an ob deploy push target.

This might be due to import-from-derivation which is used in callCabal2nix and breaks caching pretty badly. I.e. you need to actually download and run cabal (and it's dependencies) only to find out that the thing you're building is already cached. In the meantime, you should be able to run nix-build -A exe somewhere else and do nix-store -r <path> on the server to copy the built result locally.

This might be partially resolved by #368.

Because currently the thing we actually push is not the same as -A exe. It is a full NixOS image. It should be possible to cache those as well.

So part of what I was getting at with this ticket is that I'd like a way to not do the full NixOS image.  I'd like a way to get a nix store path representing everything needed to run the app on an existing (and small) machine without needing to switch that machine's NixOS image.

I think maybe the way to do this would be to expose something from a deployment configuration directory that can be referenced from your machine's nixos configuration.  E.g.: you ob deploy init into /etc/nixos/my-local-deployment, and then in your machine's configuration there's a snippet you can drop in (possibly as a nixos module, or possibly as an expression that you've gotta hook up yourself as a nixos container).

I was thinking something similar to the output of nix-build -A exe that I can use to run the app manually by doing something simple like nix-env -i <storepath> and then running it.  I don't really care about having a super simple way to run it.  I just care that I can run it.  I know I can do this with nix-build -A exe already.  But when I tried to do it on a small deployment machine, that wasn't able to complete even though the results were already cached.  It seems to me like there's got to already be a storepath that does what I want...one that is the app but doesn't require a lot of memory to get from a cache and also isn't a complete NixOS image.  My question is, how do I find out what that storepath is?

In the meantime, you should be able to run nix-build -A exe somewhere else and do nix-store -r <path> on the server to copy the built result locally.

Did you try something along these lines?

Hmm, I thought I tried something like that and it didn't work.  I guess it was probably because of IFD since I  was doing the nix-build on the small deploy machine.  But I tried it again now, this time with nix-store -r <storepath> with the storepath returned by nix-build -A exe and that did work.  This solves my immediate problem, thanks!  Now that this issue is here, the solution is at least a little more discoverable.  I'll leave it up to you all to decide whether/how to make this more available in a user-friendly way.

Great! This is a rather unpleasant workaround though. We'll keep brainstorming ways to avoid this...


Release v0.0.3

when will this release be available in pypi?

I need someone to review the PR, once it's green light it should be released!

Looks good


LOGO

Hello, @abertschi I want to contribute in "ad-free" so i want to design a new logo for "ad-free" and it's free.
I hope you like it. If you like it, I will send you to pull request, editable files/original source and .png format files. If there's something you want to change, I'll do it.
if you want i will send you my design
Contact me on facebook for fast delivery
https://www.facebook.com/waqasanwar68

Sorry, I am not interested.
On 15 May 2018 at 10:41, <waqasanwar94<EMAIL_ADDRESS>wrote:
Hello, @abertschi https://github.com/abertschi I want to contribute in
"ad-free" so i want to design a new logo for "ad-free" and it's free.
I hope you like it. If you like it, I will send you to pull request,
editable files/original source and .png format files. If there's something
you want to change, I'll do it.
if you want i will send you my design
Contact me on facebook for fast delivery
https://www.facebook.com/waqasanwar68
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/abertschi/ad-free/issues/23, or mute the thread
https://github.com/notifications/unsubscribe-auth/AgFJhH2twmAhVctuIGwd9otmvJyqHWf0ks5tyqLNgaJpZM4T_N5O
.


feat(metrics): add labels for metrics in api-middleware

Hi, thanks for this library it helped me a lot.
One thing I was missing was more labels in some metrics in api-metrics.middleware. For now, I coded my own middleware however I think this can be useful for other people as well.

Hi, @chrapek Thanks for this PR! I actually had labels before but found that it actually increase cardinality by a lot. Did you find high increase of metrics in your observability stack? Perhaps you would want to leverage tracing for detailed signals like that?

Oh, that is interesting. My first thought was that I didn't see any increasing cardinality. I will try to validate it today or during the weekend.
I'm new to the metrics and the tracing world, so I'm not sure if I can use tracing in my use case as I just need some graphs in DataDog with a number of errors, etc.

@chrapek What are your thoughts on making those labels optional via the config option? With that I would be totally ok as long it defaults to no label. Otherwise, that would create a breaking change.

ping @chrapek


Delete bin directory

Uploaded as a mistake

Good catch.  I put those in the wrong spot.  Thank you!


Slow build time - Caching issue?

I am experiencing slow (492 sec) build times, and am not clear if caching is enabled.
The readme indicates that "the buildpack caches the R binaries and installed R packages".
When I push to heroku, the R app is detected, which triggers a quite long install. see heroku install.txt (partial) logs for my recent build, 492 seconds. I only have a handful of packages.
I can't tell if this is installing from cache, or downloading fresh each time.
Could someone please confirm if this build time is to be expected, and/or if you can tell if caching is enabled by default/in my case?
Thank you! :)

Hi @jaytaytay
I've taken a look at the log you provided; it appears to be for the first deployment of your application.
The caching logic only takes effect on subsequent deployments, as your packages can only be cached once they have been complied during the first deployment.
Trying making a small change to your code and then re-push to Heroku to see if the speed improves. You can expect to see messages such as "Restoring R packages from cache" in the log output, to confirm that the cache is being used.
I hope this helps.

Closing due to inactivity.


Can spring cloud stream kafka use high level consumers api ?

I found my consumers sent offsets to __consumer_offsets topic , i couldn't find any useful information on here

@wcggit
What version of Spring Cloud Stream Kafka are you using? Unless you have a hard requirement to use Kafka 0.8 I would suggest upgrading to version 1.1.

I think this can be closed now.


Add cosmwasm_std::Coins struct

Closes: #1377
This PR renames coins.rs (which contains the Coin struct) to coin.rs, and adds a new coins.rs which contains the Coins struct.

Thank you! I just merged https://github.com/CosmWasm/cosmwasm/pull/1488 to make the diff easier to read. Could you rebase the work onto current main?


feat: Reaction update event

Summary
Allow new reactions to be actioned on
What is the feature request for?
The core library
The Problem
Allowing the client to wait_for user reaction on some messages, because sometimes a simple message wont cut it.
The Ideal Solution
Just waiting for the event, its probably ws related. Right?
The Current Solution
No response
Additional Context
No response

it does kek i just htought it was reaction_add and it never got brought up when someone mentinoed every event in a pr


[CI] Update ci-cpu to the latest

Include the caffe into the image

cc @zhiics @tmoreau89 @ZihengJiang


[webview_flutter_android][webview_flutter_wkwebview] Adds support to retrieve native WebView

Part of https://github.com/googleads/googleads-mobile-flutter/issues/618
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

cc @jjliu15 @timothyhoang-google as a heads up


Public key error

I am getting the following errors when trying to deploy on Corretto 11 running on 64bit Amazon Linux 2/3.1.4. Any ideas?
eb-engine.log
2021/01/14 22:04:51.043058 [ERROR] An error occurred during execution of command [app-deploy] - [RunAppDeployPostDeployHooks]. Stop running the command. Error: Command .platform/hooks/postdeploy/00_ssl_setup_certbot.sh failed with error exit status 1. Stderr:Repodata is over 2 weeks old. Install yum-cron? Or run: yum makecache fast
Error: No matching Packages to list
http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/68af10e71c8a2c0e3ef1d4d6426c00d014a41d79ebe2e4eb09b113f3db30ca44-updateinfo.xml.bz2: [Errno 12] Timeout on http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/68af10e71c8a2c0e3ef1d4d6426c00d014a41d79ebe2e4eb09b113f3db30ca44-updateinfo.xml.bz2: (28, 'Operation too slow. Less than 1000 bytes/sec transferred the last 5 seconds')
Trying other mirror.
warning: /var/cache/yum/x86_64/2/epel/packages/certbot-1.10.1-1.el7.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 352c64e5: NOKEY

Error downloading packages:
  setools-libs-3.3.8-2.amzn2.0.2.x86_64: [Errno 5] [Errno 12] Cannot allocate memory

eb-hooks.log
Transaction Summary
================================================================================
Install  2 Packages (+26 Dependent packages)

Total download size: 4.4 M
Installed size: 17 M
Downloading packages:
Public key for certbot-1.10.1-1.el7.noarch.rpm is not installed

I got the script to work, but now I am running into this new error.
eb-engine.log
2021/01/16 05:34:48.549888 [ERROR] An error occurred during execution of command [app-deploy] - [RunAppDeployPostDeployHooks]. Stop running the command. Error: Command .platform/hooks/postdeploy/00_ssl_setup_certbot.sh failed with error exit status 1. Stderr:Repodata is over 2 weeks old. Install yum-cron? Or run: yum makecache fast
Error: No matching Packages to list
http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/6d3a401fc683b12ce517bdd8d2c469150605b3aa188ec6ffc871eeb327c6e129-updateinfo.xml.bz2: [Errno 12] Timeout on http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/6d3a401fc683b12ce517bdd8d2c469150605b3aa188ec6ffc871eeb327c6e129-updateinfo.xml.bz2: (28, 'Operation too slow. Less than 1000 bytes/sec transferred the last 5 seconds')
Trying other mirror.
warning: /var/cache/yum/x86_64/2/epel/packages/certbot-1.10.1-1.el7.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 352c64e5: NOKEY
Importing GPG key 0x352C64E5:
 Userid     : "Fedora EPEL (7<EMAIL_ADDRESS> Fingerprint: 91e9 7d7c 4a5e 96f1 7f3e 888f 6a2f aea2 352c 64e5
 Package    : epel-release-7-13.noarch (installed)
 From       : /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator nginx, Installer nginx
Performing the following challenges:
Client with the currently selected authenticator does not support any combination of challenges that will satisfy the CA. You may need to use an authenticator plugin that can do challenges over DNS.
Client with the currently selected authenticator does not support any combination of challenges that will satisfy the CA. You may need to use an authenticator plugin that can do challenges over DNS.

I got the script to work, but now I am running into this new error.
eb-engine.log
2021/01/16 05:34:48.549888 [ERROR] An error occurred during execution of command [app-deploy] - [RunAppDeployPostDeployHooks]. Stop running the command. Error: Command .platform/hooks/postdeploy/00_ssl_setup_certbot.sh failed with error exit status 1. Stderr:Repodata is over 2 weeks old. Install yum-cron? Or run: yum makecache fast
Error: No matching Packages to list
http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/6d3a401fc683b12ce517bdd8d2c469150605b3aa188ec6ffc871eeb327c6e129-updateinfo.xml.bz2: [Errno 12] Timeout on http://csc.mcs.sdsmt.edu/epel/7/x86_64/repodata/6d3a401fc683b12ce517bdd8d2c469150605b3aa188ec6ffc871eeb327c6e129-updateinfo.xml.bz2: (28, 'Operation too slow. Less than 1000 bytes/sec transferred the last 5 seconds')
Trying other mirror.
warning: /var/cache/yum/x86_64/2/epel/packages/certbot-1.10.1-1.el7.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 352c64e5: NOKEY
Importing GPG key 0x352C64E5:
 Userid     : "Fedora EPEL (7<EMAIL_ADDRESS> Fingerprint: 91e9 7d7c 4a5e 96f1 7f3e 888f 6a2f aea2 352c 64e5
 Package    : epel-release-7-13.noarch (installed)
 From       : /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator nginx, Installer nginx
Performing the following challenges:
Client with the currently selected authenticator does not support any combination of challenges that will satisfy the CA. You may need to use an authenticator plugin that can do challenges over DNS.
Client with the currently selected authenticator does not support any combination of challenges that will satisfy the CA. You may need to use an authenticator plugin that can do challenges over DNS.

It looks like that error caused by having a wildcard for subdomains, and it went away once I removed it. However, I am now getting 502 Bad Gateway with 021/01/16 06:02:45 [error] 9838#0: *22 connect() failed (111: Connection refused) while connecting to upstream.

It looks like that error caused by having a wildcard for subdomains, and it went away once I removed it. However, I am now getting 502 Bad Gateway with 021/01/16 06:02:45 [error] 9838#0: *22 connect() failed (111: Connection refused) while connecting to upstream.

It looks like my application was misconfigured and wasn't running. 🤦

It looks like my application was misconfigured and wasn't running. 🤦


Investigate GitHub Actions

This might be a good ressource to set up CI for our project.

CI/CD is configured and working through GitHub Actions


[BUG] crashed VM id not deleted, when I delete it.

Describe the bug

To Reproduce

create a new vm.
Disconnect Network-connection form all Servers to replace Switch.
see that it cannot start anymore one out of 60 VMs after reconnecting.
try to delete the VM. Its marked as delted but it stays ther since nearly 3 days

Expected behavior
The VM doesnt crash and if it does. I can at least delete it.
Environment

Harvester ISO version: 1.1.1

@ne0YT
Did you try to get more information then, about the related POD, VMI, VM crd object. (e.g.  kubectl get virtualmachine -n default/the-name vm-name -oyaml ,  kubectl get virtualmachineinstance -n default/the-name vm-name -oyaml),  or generate a support-bundle file per
https://docs.harvesterhci.io/v1.1/troubleshooting/harvester/#generate-a-support-bundle . thanks.

@w13915984028 I was able to remove it by spaming remove using this tool:
https://github.com/harvester/harvester/issues/3807#issuecomment-1542061551
so sadly no, I didnt collect data on it.

From the descripton, suspect the pending VM deleting may be related to volume detaching/deleting.
Feel free to contact us when it happens again or you recalled something, thanks.


[DataSource][Arrow] Overwrite ParquetFileFormat's reader with Arrow b…

…ased reader
Also, refactor directory tree and add module spark-arrow-datasource-parquet

One typical case of this feature is to make ArrowDataSource work together with Delta Lake. Some PoC codes: https://github.com/zhztheplayer/delta/commit/81249fce7f45f44783c991ca577dd04b1d5b34ab


Use QMailAccount directly and drop libaccounts.

I'm proposing to drop completely AccountsCache and directly use QMailAccount(id). Technically, after the Sailifsh OS patch is applied in QMF, this will call QMailStorePrivate::getEmailAccount() from qmailstore_p.cpp which will do the Accounts::Account::fromId() previously existing in accountscache.cpp. It also checks the service availablity for email and that it is enabled. I'm not completely sure though, but I think that an Accounts::Manager is caching the list of accounts and their properties. So I don't see the point in caching the data here in the plugin also. Looking at the log of accountscache.cpp file, I didn't see any particular message mentioning performance penalties. Besides, QMF code is full of QMailAccount(id) already, even this same plugin, see for instance MailStoreObserver::transmitFailed().
I'm currently testing it.

Notifications on reception are working ok. I was not able to get a send failure to see if the failed banner is displayed though…


Julia: vectorized calls

Currently the generated functions don't allow vectorized argument calls. (see below example).
A very quick fix would be for _unpack_var to return a special type of array view which would implement vectorized views by default.
@spencerlyon2, I we discussed that when you where in London. Can you remember me which where the options ?
For the record, here is the type of error message one currently gets:
type dummytype
end

eqs = [:(1/exp(a)+b)]
args = [(:a,0)]
parms = [:b]

using Dolang

println( make_method(Der{1}, dummytype, eqs, args, parms) )

code = make_method(Der{1}, dummytype, eqs, args, parms, funname=:fun)
f = eval(code)

a = [1.0]
b = [0.0]

f(Der{1},dummytype(),a,b)      # works
f(Der{1},dummytype(),a'',b'')   # doesn't work (a and b are list of points)

Here is the output:
MethodError: Cannot `convert` an object of type Array{Float64,1} to an object of type Float64 This may have arisen from a call to the constructor Float64(...), since type constructors fall back to convert methods.  in fun(::Type{Dolang.Der{1}}, ::dummytype, ::Array{Float64,2}, ::Array{Float64,2}) at ./<missing>:0  in include_string(::String, ::String) at ./loading.jl:441  in eval(::Module, ::Any) at ./boot.jl:234  in (::Atom.##60#63)() at /home/pablo/.julia/v0.5/Atom/src/eval.jl:39  in withpath(::Atom.##60#63, ::Void) at /home/pablo/.julia/v0.5/CodeTools/src/utils.jl:30  in withpath(::Function, ::Void) at /home/pablo/.julia/v0.5/Atom/src/eval.jl:45  in macro expansion at /home/pablo/.julia/v0.5/Atom/src/eval.jl:107 [inlined]  in (::Atom.##59#62)() at ./task.jl:60

I actually think the new fun.(args...) syntax introduce in Julia 0.5 (but available on 0.4 via the @compat macro from Compat.jl) should allow us to do this. Check out the "Vectorized function calls" section here
Given that I believe this to be the way forward,  I think we can do away completely with the _unpack_var function and just assume that all arguments are 1d arrays. It was originally there to extract either a scalar if the first argument (to _unpack_var) was a 1d array or a 1d array if the first argument was a 2d array.

Actually, I think I misunderstood something.
the fun.(...) syntax is meant to take functions defined over scalars and let them work over arrays.
We have functions defined over 1d arrays and we're asking them to work over >1d arrays. I think we need a bit more brainstorming for this...

Plus I actually like the unpack - var  function  as we can use it to pass
nothing
On Fri, Nov 4, 2016, 00:40 Spencer Lyon<EMAIL_ADDRESS>wrote:

Actually, I think I misunderstood something.
the fun.(...) syntax is meant to take functions defined over scalars and
let them work over arrays.
We have functions defined over 1d arrays and we're asking them to work
over >1d arrays. I think we need a bit more brainstorming for this...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EconForge/Dolang/issues/11#issuecomment-258315579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ5KfM-JGjeGvkPOE52q6tH1dJ5a2Rsks5q6n7vgaJpZM4KpBnI
.


Introduces importOrderBuiltinModulesToTop for Node Builtins

This is a cherry-pick of a PR from @nickhudkins to the upstream project: https://github.com/trivago/prettier-plugin-sort-imports/pull/150, adjusted slightly to work within the rule of this project to not sort beyond side-effect import groups.
@blutorange what do you think of this?  I'm reluctant to add another option, but if we changed this without an option it would be a breaking change.  I don't have a strong opinion one way or the other, but personally I do like sorting builtins to the top, so it would be nice to have some way to do it.

The commit itself looks good, it's only a very small change.
@IanVS Have you checked if this works for prefixed modules such as import {...} from "node:fs" or import * as fs from "node:fs/promises"?
Other than that, my very personal opinion is that a prettier plugin should be opinionated in the same way prettier is and have as few options as possible. But this plugin doesn't really follow that philosophy. I took a quick look at https://www.npmjs.com/package/@ianvs/prettier-plugin-sort-imports and I'd say there are enough downloads we can't just introduce a breaking change.
In the long term we should decide if we want to tend towards "many options for configurability" or "few options with an opinionated sort order" -- we could remove options in a major release. What are you thoughts about this? I also wonder what users are thinking in this regard.
But in short, if we add this without a new major release, it needs to be an option that defaults to false. And a major release imo only makes sense if we decided to set a new goal for this plugin and start removing other option.

Thanks @blutorange.  I've added some more tests as you suggested.
I agree with you about not making breaking changes outside of majors.  It annoys me that prettier itself makes breaking changes in minors, and I don't want to follow that pattern.
I think one thing we should consider is that this plugin is changing the actual AST, rather than only formatting like prettier does.  So, there's a chance we do need to allow a few more options than a purely-formatting tool would.  But where it makes sense, I think we should avoid options.  I'd like to make this behavior the default in the next major version, and maybe even remove the option entirely, unless we hear that it causes big problems for folks.
I think this is ready for another review, when you have time.  Thanks again!

I'd like to make this behavior the default in the next major version, and maybe even remove the option entirely, unless we hear that it causes big problems for folks.

Semver only works if people actually stick to it, so yes, sounds like a good idea to me
I also don't really like the importOrder option -- it requires me to manually add the dependency I use. At the least I'd prefer a sensible default behavior when no option is set that might perhaps group all imports with the same scope automatically. But that's outside the scope of this issue.

I'm going to merge this so I can rebase https://github.com/IanVS/prettier-plugin-sort-imports/pull/12 on main.  I think I got an approval above, but if you want any changes, @blutorange, let me know and I'll open another PR.


Attrib & Attdef

Hi. I did it for myself, but if it will be useful to you, I can try to make the necessary corrections for the review.
add classes Attdef, Attrib
add methods addAttrib, addAttdef

Yes please fix it

I fixed it. Thanks for the review.


Not explicit test in step 12 of Penguin project of new RWD

Describe the Issue
The description says: Use the margin property to horizontally center the .penguin element, and set the margin-top to 75px.
Maybe we should explicitly say in the description that the test requires that margin-bottom: auto or change the test.
Affected Page
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/14-responsive-web-design-22/learn-css-transforms-by-building-a-penguin/619699c10a0f6e11591d73c4.md
Your code
The code
margin: 75px auto 0;
or
margin-top: 75px;
margin-left: auto;
margin-right: auto;

does not pass because the test requires that also margin-bottom is set to auto. So the accepted code is margin: 75px auto auto;.
Expected behavior
From reading the description I understand that the requirements involve margin-left and margin-right (value of auto) and margin-top (value of 75px). I didn't expect to need also margin-bottom: auto.
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

had a hard time figuring this out. thanks young king


Add 'use strict'; in template to support es6

I use let a lot in my projects and they don't work without 'use strict' on top.

Hey, this is good, the only issue is that you've edited the generated file but forgot the source CS file in src. Please update accordingly and I'll get it published.

My bad. Will do it in a moment.

Updated. Please have a look.

Hi @alyyousuf7 I'm sorry for the delay, I've definitely haven't paid much attention to this module in the last couple of years:(.

Merged. Will be included in the next release.

Published as<EMAIL_ADDRESS>


unit-tests layout for modules

Fixes #182
Adding tests layout for modules
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM 👍 . We might consider also adding some linting and formatting with tools like black and isort

LGTM 👍 . We might consider also adding some linting and formatting with tools like black and isort

That is what fix.sh does...

LGTM 👍 . We might consider also adding some linting and formatting with tools like black and isort

That is what fix.sh does...

Great! Although the name fix.sh doesn't tell me that


Tell echolocation when nitrates fail

Reports ERROR to queue when BKG fails, or when no data is found after a day.
Reports COMPLETE to queue when exit due to no pointing data.
Any other failure modes missing that I  can catch to keep the queue clean and notify us of problems?

What about for slews?

Oh i do that too, just forgot to say. See line 1336 in do_manage in this PR.
It reports "complete" if no pointing data.

Reports COMPLETE to queue when exit due to no pointing data.
did you mean something else?

Thanks! Just saw that in the code too. Looks good to me.
It will still miss the slews/no data that NITRATES misses where there's no usable data for the signal window (+/- 20s), but data available to do the bkg. But that's probably something that should be fixed inside of NITRATES first. Right now do_full_rates will just fail, then the split rate jobs will be submitted but also fail right away when there's no time seeds file.

Feel free to merge if you're not making more edits

Thanks @jjd330

It will still miss the slews/no data that NITRATES misses where there's no usable data for the signal window (+/- 20s), but data available to do the bkg. But that's probably something that should be fixed inside of NITRATES first. Right now do_full_rates will just fail, then the split rate jobs will be submitted but also fail right away when there's no time seeds file.

Can you open an issue for this?


test(create): add basic integration test for app with all features

fixes: #941
closes: #970
What this PR Does:
tests that the following command
node -r @babel/register packages/create/src/create.js \
  --destinationPath ./output \
  --type scaffold \
  --scaffoldType app \
  --features linting testing demoing building \
  --buildingType rollup \
  --scaffoldFilesFor testing demoing building \
  --tagName scaffold-app \
  --writeToDisk true \
  --installDependencies false

in fact creates the desired output files.
Each output file's content is checked against the snapshot.
What this PR Doesn't Do:

test any other commands
test the wizard UI
test the keyboard input
provide an automated way to update the snapshots
provide clear guidance or convention for naming snapshots based on input/test case

I like this. We are committing the whole generated output as a snapshot, but I think it's a nice way to keep a grasp on what the generator does and that our changes don't mess it up.

I think this looks really nice :)
I wonder if it would be possible to run lint/test on the output files? It has happened in the past tht the generator runs fine and scaffolds some files, but that there was a mistake in the files (wrong import, introduced  a linting problem, etc)
if we can somehow run linting and tests we can be really confident that we didnt break anything

@thepassle I'm going to look into running the linter on the output in tests
only reason i can think of not to would be that npm i will be expensive in terms of CI time. we could hack around that by treating the ACTUAL test output as a package in the monorepo, for purposes of linting

@thepassle
https://github.com/open-wc/open-wc/blob/91321f43ed939aef70a6282f4f24265b8906078c/packages/create/test/integration.test.js#L121-L126

So nice 😄 Can we run tests on the output too?

output ?
ACTUAL_PATH is the output, so this runs eslint programmatically on the generated files

AFAICT it's the same as cd scaffold-app && npx eslint

? Not sure what you mean with your last comment

edited. I mean this test does lint the generator output

🤦♂ oh you mean TESTS
uhh yeah let me check

so, these tests run in a node context.. i suppose i could spin karma up in a child process

discussing it on slack, we agreed it's best to leave this at simple tests for now. later on we'll implement proper e2e tests that run, install, and test output in a separate ci process


[BUG] Unable to select filters

Extension largely works as intended on iOS, using Orion Browser having installed the Firefox version if this extension, but I cannot click on the filters while the extension is active. Without the extension this does work.
Not too sure if this is an issue because I am using it from my phone

Well filters should work for both the chrome and firefox versions. But because I don't own an iOS device I can't actively target/debug for this platform.

I can't toggle any of the filters on firefox. I can mark objects on the map but I can't interact with the filters on the left in any way. Game I tried it on was Elden Ring.

Well tested it on firefox-desktop-120.0b9, And it still seems to work for me.
Do the filters work occasionally or non at all?
It would also be helpful to provide:

errors/warnings
platform (linux, windows, ...)
browser version
if applicable steps to reproduce the error.
other information 'if you think it would be helpful, to much information is better than none'.

Errors for FMG are prefixed.

Errors for mapgenie will have map.js as src.

I'm running Firefox 119.0.1 (64bit) on Windows 11
When I open the site it refreshes like 4-5 times by itself or something, during which I'm able to turn filters on/off. Only after it stops I can't interact with it anymore.
I have turned off all my browser add-ons just in case but it didn't help either.
Errors/warnings I don't see any on the site itself, I'm not sure if the error from the console will help much.
https://puu.sh/JUzIo/02dec08d40.mp4

The refreshing is because the extension can't communicate with the injected script.
One case I know this can happen is when permissions are not enabled for the extension.
And can't inject the content script to the page.
Just to make sure can you check if permissions are enabled for the extension.
Step 6 in firefox install guide.
Firefox installation guide
For now I maybe work on something that will try to detect this and warn the user instead of refreshing the page.

One thing for sure is that I will remove the auto reload and instead change it to a warning/error that the extension couldn't find the window.store object.
And requests the user to reload the page manually.

@BasedOnPixels It could be helpful for me if you could check this for me.

Go to this Url in Firefox about:debugging#/runtime/this-firefox.
And check if the background script is running under the fmg extension.

If not go to a map page elden ring for example. And check if to switches from stopped -> running
Also check if the extension has any errors

It shows a warning because of a invalid manifest value already working on fixing that.

The background script is stopped on the about:debugging page. If I load it in as a temporary add-on, i can reload it and it is running at first but stops seemingly after a while.
The regular and the dev build didn't have any errors.
On the mapgenie.io page it tells me that window.store is already defined and asks me to reload the page.

When loading the dev build you can click on the inspect button and it will open a dev tools window, go to the console tab.
And open a mapgenie map and if everything is right it should print a block request, redirect request and a allow request.

On the mapgenie.io page it tells me that window.store is already defined and asks me to reload the page.

By mapgenie.io page do you mean the homepage?

Because that should not be happening. If so open the dev tools on the homepage you can do this by pressing f12.
And type window.store in the console.

It should return undefined.

Oh no, i meant generally on mapgenie.io, as in the Elden Ring map, sorry for not being clear about it. On the main page it doesn't show the error and window.store returns undefined.
The console on the dev add-on doesn't say more than "Background script loaded".

Ok so there is the problem the backgrounds script is unable to block the script hence why window.store is already defined.
Now to find out why its unable to block.

I just figured it out.
I had to enable "Access your data for all websites".
I thought turning on data for mapgenie.io was enough due to it only requiring access to the site?

Ok good, But does it fix the filters?

Maybe I can finally switch over to declarativeNetRequest for Firefox. Which I already use for the chrome version.

It does fix the problem, I can set filters and they all save and load when I refresh the page. It seems to be completely working now after giving it full permission to all sites.

Well just implemented declarativeNetRequest for the Firefox version, which doesn't need any extra permissions as far as I know.
As for the OP case its probably the same issue. So I will mark this as solved for now. It can always be reopened.
Solution:

I just figured it out.
I had to enable "Access your data for all websites". I thought turning on data for mapgenie.io was enough due to it only requiring access to the site?

I tried the new release and I get the new ‘window.store is already defined’ error pop-up, but I assume this is because the browser I am using on my phone does not allow to set the optional permissions.
I think a fix for this could be to have these permissions set as required instead of optional, but that’s probably not a desirable solution.

I think a fix for this could be to have these permissions set as required instead of optional, but that’s probably not a desirable solution.

Its a manifest v3 issue which makes the content script injection optional on Firefox.
Well I found out what breaks the filter, The map.js script is loaded twice the default one because my background is unable to block it, and one after my extension is done with the setup face.
So one easy fix is to check if the store is already defined and not load the map a second time.
But some features may not work.

Pro maps (tarkov)
Mock user (not so important just login with your account)

A second solution is add a second Firefox version for manifest v2 which I don't think has the option permissions by default.

Its possible that this issue is fixed by disabling the declarative net request option in the browser action.
If it still does not work reopen this thread or post a new issue.


Rework cdba to support controlling device via external program

Here are the changes that I have here to allow controlling devices through the external script.

@andersson gracious ping

Rebased on top of current master branch.
@andersson any hope to hear response from you?

The last patch fixes the external support or fixes in all cases ? If it fixes all cases it should got in a separate or right ?

It is a generic fix, but I stumbled upon it only with the "external" driver, since other drivers do not output a lot.

Is it possible to combine this with the qcomlt debug board? For the phones I have a Pi Pico exposing a compatible serial control interface, but for USB VBUS I need to use PPPS control...

@calebccff here is (an excerpt) of the script that I have here for the relays and PPPS hub:
#!/bin/sh

#set -x
set -e

if [ "$#" != 3 ]
then
	echo "Usage: $0 [board] [command] [on/off]"
	exit 1
fi

echo "Called $1 $2 $3"

case "$1" in
	"rb5-2" )
		HUBLOC="10-1"
		HUBPORT="1"
		LAURENT="REL,1"
		;;
	"db820c-1" )
		HUBLOC="10-1"
		HUBPORT="2"
		LAURENT="REL,2"
		;;
	"rb1-1" )
		HUBLOC="10-1"
		HUBPORT="3"
		LAURENT="REL,3"
		;;

	* )
		echo "Unknown board $1"
		exit 2
esac

uhubctl() {
	sudo uhubctl -l $HUBLOC -p $HUBPORT -a $1
	if [ "$1" = "off" ] ; then
		sudo udevadm trigger --action=remove /sys/bus/usb/devices/${HUBLOC}.${HUBPORT}/ || true
	fi
}

laurent() {
	curl "http://laurent.lan/cmd.cgi?psw=$LAURENTPW&cmd=$LAURENT,$1"
}

#echo "$2-$3" >> /tmp/cdba-log

case "$2-$3" in
	"power-off" )
		uhubctl off
		#sleep 1
		laurent 0
		#sleep 1
		;;
	"power-on" )
		uhubctl off
		#sleep 1
		laurent 0
		#sleep 1
		laurent 1
		;;
	"usb-on" )
		uhubctl on
		;;
	"usb-off" )
		uhubctl off
		sleep 1
		;;
	"key-power-on" | "key-power-off" )
		# ignore
		;;
	*)
		echo "Unknown command $2-$3"
		exit 2
		;;
esac

I hooked up my pico to control the enable pin of the buck converter I have as a battery replacement, this makes all the button wrangling unecessary as I can just erase the boot partition and the devices always boot straight to fastboot mode.
There is maybe some value in this PR for someone else, but given the amount of finesse necessary to get this to work reliably, I'm not gonna stick with it... See #33


QUESTION: Subset expression matrix from custom genes

Hi,
I have asked this question before in Scanpy, but I wasn't sure I made it clear. So I'm giving it a try again:
Say I have the PBMC 3K dataset, and after clustering and DEG in Scanpy, I have 120 genes specific for cluster 1 and 80 genes specific for cluster 3. Now, what I would like is to take these 200 genes and export them as an expression matrix (csv or tab), where rownames are the gene symbols (i.e: IFNG, IL10, etc) and colnames are the cell ids (barcodes) present within each cluster.
How do I get this from an h5ad file?
Thanks!

Why isn't this answer (https://github.com/theislab/scanpy/issues/262#issuecomment-447613930) working for you?
Obviously, you'd need to subset the anndata object before:
import scanpy.api as sc
adata = sc.datasets.pbmc68k_reduced()
# this writes everything, the data matrix and annotations end up in separate files
adata[:, ['HES4', 'TNFRSF4', 'SSU72']].copy().write_csvs('./test/', skip_data=False)
# this writes only the data matrix
adata[:, ['HES4', 'TNFRSF4', 'SSU72']].to_df().to_csv('./matrix.csv')

Hope this helps!
PS: Please, next time reference your previous issue. Also, provide a small example such as the one above.


[ST-Engine][Api] Replace SeaTunnelContext with JobContext and remove singleton pattern

Purpose of this pull request

Since the new engine can run multiple jobs at the same time, the SeaTunnelContext of the previous singleton pattern mode is not applicable and will cause bugs, because each job needs to hold a SeaTunnelContext.
so replace SeaTunnelContext with JobContext and remove singleton pattern.
Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Hi, could you submit this change to dev branch first?

Hi, could you submit this change to dev branch first?

yes i am doing this


Vec: Simpler creation from slice

Right now creating a Vec with fixed data can be a bit verbose...
let mut data = Vec::<u8, consts::U256>::new();
data.extend_from_slice(b"Hello").unwrap();

Would you accept a PR that adds a constructor function?
let data = Vec::<u8, consts::U256>::from_slice(b"Hello").unwrap();

Besides requiring less code, it also allows to drop the mut.

As a practical example, a refactoring in the ATAT crate that changed some types from String to Vec contains code like this:


insert fail on distributed mode

After running stably for 3 months, suddenly there were a lot of following exceptions, seems node1 cannot accept insert operation, but update/query is ok. and insert is ok on node2.
Now we disabled distributed mode.
Thanks for any idea.
version: 2.1.7
"executionMode": "synchronous"
data size:  vert 2500k, edge 2500k

Caused by: com.orientechnologies.orient.server.distributed.ODistributedException: Error on executing distributed request (id=5022941 from=node1 task=record_create(#11:-1 v.0) user=#5:0) against database 'orion.[orion_v]' to nodes [node1, node2]
at com.orientechnologies.orient.server.hazelcast.OHazelcastDistributedDatabase.send2Nodes(OHazelcastDistributedDatabase.java:189)
at com.orientechnologies.orient.server.hazelcast.OHazelcastPlugin.sendRequest(OHazelcastPlugin.java:360)
at com.orientechnologies.orient.server.distributed.ODistributedStorage.createRecord(ODistributedStorage.java:547)
at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.executeSaveRecord(ODatabaseDocumentTx.java:1999)
at com.orientechnologies.orient.core.tx.OTransactionNoTx.saveRecord(OTransactionNoTx.java:159)
at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(ODatabaseDocumentTx.java:2568)
at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(ODatabaseDocumentTx.java:2409)
at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(ODatabaseDocumentTx.java:121)
at com.orientechnologies.orient.server.network.protocol.binary.OBinaryNetworkProtocolAbstract.createRecord(OBinaryNetworkProtocolAbstract.java:367)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.createRecord(ONetworkProtocolBinary.java:1569)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.executeRequest(ONetworkProtocolBinary.java:365)
at com.orientechnologies.orient.server.network.protocol.binary.OBinaryNetworkProtocolAbstract.execute(OBinaryNetworkProtocolAbstract.java:223)
at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)
Caused by: com.orientechnologies.orient.server.distributed.ODistributedException: Quorum 2 not reached for request (id=5022941 from=node1 task=record_create(#11:-1 v.0) user=#5:0). Elapsed=6ms Servers in timeout/conflict are:

node2: #11:2660136 v.1
Received: {node1=#11:2660169 v.1, node2=#11:2660136 v.1}
at com.orientechnologies.orient.server.distributed.ODistributedResponseManager.manageConflicts(ODistributedResponseManager.java:570)
at com.orientechnologies.orient.server.distributed.ODistributedResponseManager.getFinalResponse(ODistributedResponseManager.java:339)
at com.orientechnologies.orient.server.hazelcast.OHazelcastDistributedDatabase.waitForResponse(OHazelcastDistributedDatabase.java:423)
at com.orientechnologies.orient.server.hazelcast.OHazelcastDistributedDatabase.send2Nodes(OHazelcastDistributedDatabase.java:186)
... 12 more

first we tried to restart the db, but the exception still appear several minutes after restart. then restarted it as standalone server, and it disappeared.

logs before exception:

2016-05-01 20:09:03:219 INFO  [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] processors=2, physical.memory.total=3.7G, physical.memory.free=106.2M, swap.space.total=7.8G, swap.space.free=7.8G, heap.memory.used=1.5G, heap.memory.free=426.2M, heap.memory.total=1.9G, heap.memory.max=1.9G, heap.memory.used/total=78.54%, heap.memory.used/max=78.54%, minor.gc.count=72210, minor.gc.time=3056529ms, major.gc.count=521, major.gc.time=556340ms, load.process=1.00%, load.system=1.00%, load.systemAverage=0.01, thread.count=98, thread.peakCount=154, cluster.timeDiff=0, event.q.size=0, executor.q.async.size=0, executor.q.client.size=0, executor.q.query.size=0, executor.q.scheduled.size=0, executor.q.io.size=0, executor.q.system.size=0, executor.q.operation.size=0, executor.q.priorityOperation.size=0, executor.q.response.size=0, operations.remote.size=3, operations.running.size=0, operations.pending.invocations.count=3, operations.pending.invocations.percentage=0.00%, proxy.count=18, clientEndpoint.count=0, connection.active.count=1, client.connection.count=0, connection.count=1 [HealthMonitor]
2016-05-01 20:09:23:506 WARNI [node1] detected 1 node(s) in timeout or in conflict and quorum (2) has not been reached, rolling back changes for request (id=5015367 from=node1 task=record_create(#11:-1 v.0) user=#5:0) [ODistributedResponseManager]
2016-05-01 20:09:23:506 WARNI [node1] Quorum 2 not reached for request (id=5015367 from=node1 task=record_create(#11:-1 v.0) user=#5:0). Elapsed=6ms Servers in timeout/conflict are:
node2: com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record orion_v{pk:150001346477,pk_type:3}: found duplicated key '150001346477' in index 'idx_orion_v_pk' previously assigned to the record #11:2658222 RID=#11:2658222
Received: {node1=#11:2658285 v.1, node2=com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record orion_v{pk:150001346477,pk_type:3}: found duplicated key '150001346477' in index 'idx_orion_v_pk' previously assigned to the record #11:2658222 RID=#11:2658222} [ODistributedResponseManager]
2016-05-01 20:09:23:506 WARNI [node1] sending undo message (record_delete(#11:2658285 delayed=false)) for request (id=5015367 from=node1 task=record_create(#11:-1 v.0) user=#5:0) to server node1 [ODistributedResponseManager]
2016-05-01 20:09:25:116 WARNI [node1] detected 1 node(s) in timeout or in conflict and quorum (2) has not been reached, rolling back changes for request (id=5015370 from=node1 task=record_create(#11:-1 v.0) user=#5:0) [ODistributedResponseManager]

I noticed that for insert, normal responses of send2Nodes should have same rid for all nodes, right?

responses:{node1=#11:2675912 v.1, node2=#11:2675912 v.1}

but in our case have different  rid which causes the exception. (all insertion failed after that point)

Received: {node1=#11:2660169 v.1, node2=#11:2660136 v.1}

what causes the inconsistence, thanks for any idea.

This exception:
node2=com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record orion_v{pk:150001346477,pk_type:3}: found duplicated key '150001346477' in index 'idx_orion_v_pk' previously assigned to the record #11:2658222 RID=#11:2658222} [ODistributedResponseManager]

Is the cause. We had an issue on indexes, so I guess you have 2 records for the same key. Could you please resolve the duplication and upgrade to last 2.1.x? @tglman is this fix in v2.1.x?

Hi,
Yes all the fix based on cleanup on exception are in 2.1.x.
Regards

thanks for quick reply, currently we use unique index to avoid duplication, what do you mean by resolve the duplication? do we need to check duplication before insert, and delete existing duplication manually?
thanks.

Hi @ray3888,
Well this is actually what does oriendb in case of notTx usage, the point is that some of this beavior was fixed in a hotfix more recent of 2.1.7 so we are suggesting to update to the last hotfix(currently 2.1.16).
Regards

thanks for the advice.  we will upgrade soon.
Regards

upgraded to 2.1.16 two days ago, enabled distributed mode, and changed from noTx to Tx just in case,  works well up to now.

hi @ray3888,
Great, can we close this now ?

sure


[profiler] Skip i386 skip condition

See #10698 for some context, this test wasn't actually being skipped on i386
cc @tkonolige
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Oh yeah, I forgot to remove that skipif. The test should work on i386. Can you remove the skipif instead.

discussed a bit offline, the test does run on i386 but consistently takes a long time so we're keeping disabled until that gets debugged


fix: add explicit module names

pylint triggered:
pylint: Command line or configuration file:1: UserWarning: Specifying exception names in the overgeneral-exceptions option without module name is deprecated and support for it will be removed in pylint 3.0. Use fully qualified name (maybe 'builtins.StandardError' ?) instead.
pylint: Command line or configuration file:1: UserWarning: Specifying exception names in the overgeneral-exceptions option without module name is deprecated and support for it will be removed in pylint 3.0. Use fully qualified name (maybe 'builtins.Exception' ?) instead.
pylint: Command line or configuration file:1: UserWarning: Specifying exception names in the overgeneral-exceptions option without module name is deprecated and support for it will be removed in pylint 3.0. Use fully qualified name (maybe 'builtins.BaseException' ?) instead.
Suggestion works out.
These style guides are copies of Google's internal style guides to
assist developers working on Google owned and originated open source
projects. Changes should be made to the internal style guide first and
only then copied here.
Unsolicited pull requests will not be merged and are usually closed
without comment. If a PR points out a simple mistake — a typo, a broken
link, etc. — then the correction can be made internally and copied here
through the usual process.
Substantive changes to the style rules and suggested new rules should
not be submitted as a PR in this repository. Material changes must be
proposed, discussed, and approved on the internal forums first.

thanks!


Update import after user renamed on github

Looks like user cenkalti renamed his account to cenk. Github
automatically redirects but it's better to update the import path.

Thanks @erikdubbelboer, ill merge this as part of the next release 😄


Add someshbhandare

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Please merge this pull request to the main branch

As per our merging process, we require approvals from at least two maintainers before proceeding. Since we currently have only one approval, kindly wait for another maintainer to review and provide their approval for this PR.
Reference: https://eddiehubcommunity.github.io/support/#/moderators/github?id=when-to-merge-prs


Drop the distinction between Create/Update.

There was a bunch of boilerplate to differentiate between Create/Update, which wasn't particularly useful.  This drops it and streamlines a bunch of the logic.
I expect this will be the first cleanup of a number to follow...
/assign @dlorenc

Nice cleanup!


support for billing project locations

Based on an offline discussion, we may end up moving forward with this in the future, but it won't happen immediately so we're closing this PR for now.


[FlexGrid] not updating layout when printing

hi guys, so before v1.0 when we had Grid, we were responsive by using media queries and that meant our printed content were laid out correctly respective to the paper width.
but with FlexGrid, depending on the width of the browser when the user clicks print, the layout stays the same in the print preview (ie. based on browser width, not paper width). This is causing our 2 columns of content to show up side by side on the print instead of stacking like it should and was using width of letter paper.
The best solution i can come up with is in the print css, override flex-basis and max-width for both columns explicitly, and add in appropriate margin bottom. Is there a better solution? thanks
This issue spawned from my question in @tds-support channel on slack. Just logging it here so it's up for discussion.

Hi @lxingtelus, as of<EMAIL_ADDRESS>the grid is back to using media queries for responsive behaviour. Based on your description, I think that should account for this issue without having to override any CSS.
Are you able to confirm?

We have been able to confirm that layouts are preserved when printing and using FlexGrid.


remember IP addresses

like a drop down menu which displays the last used IP addresses?

Done in this commit.


chore(package): bump version typescript 2.0.3

Close pull fix https://github.com/driftyco/ionic-conference-app/commit/ea2f5ddb9d37cb7590f1458ee7053a2ef770ff91#diff-b9cfc7f2cdf78a7f4b91a753d10865a2


[SHIRO-671] Add support for javax.annotation.security.RolesAllowed, PermitAll, and DenyAll

I'm not sure this should be part of Shiro, as exposing theSubject as a java.security.Principal in  web and jaxrs modules should allow this functionality to work without needing code in Shiro (so fewer bits for us to support).
That said, this would allow for a portable solution between containers.
Thoughts and ideas welcome.  If anyone finds this useful, please leave a note

@bdemers I think it could be usefull for migrate existing application to Shiro without source code modification :)
:+1:

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/54/

@fpapon 👍 I'll take another look at it for the next minor release

@bdemers great! I created a Jira for this:
https://issues.apache.org/jira/browse/SHIRO-671
Let me know if can help ;)

@bdemers I think we can close this one because it's already managed by https://github.com/apache/shiro/pull/185
Thoughts?

Yes, please close this :)

Superseded by #714


Re-rendering works not correctly?

https://jsfiddle.net/es8behkn/4/
I want, load many JSON files, but I can't render them. What do I do wrong?

When you call .render on an element, it goes back to its original state as defined in the HTML. Try wrapping them in separate elements, like so: https://jsfiddle.net/edu1gp3a/
Alternatively, you could first do all AJAX requests, and then combine the data in 1 object and render it all at once.

Yep, that works. But if i use $('body').render(data); (or <html>? I want rerender whole html file ...
May be wrappers can help, but wrappers in wrappers etc ... Not good.

https://jsfiddle.net/8uau1qo7/1/
I can use async.js, but i think that also bad practice.


added new jurassic theme artwork

I have attached a new "Jurassic Park" themed artwork.

 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.


Configure defaults for IAVL cache size

cref https://github.com/osmosis-labs/cosmos-sdk/pull/59
We used to change this directly in the SDK, @marbar3778 got this into a nice config param, so we need to configure that to be what we had before prior to next release!
See: https://github.com/osmosis-labs/osmosis/blob/main/cmd/osmosisd/cmd/root.go#L98-L101 for where we override it

https://github.com/osmosis-labs/osmosis/tree/faddat/iavl-cache
I am thinking like a 1gb cache by default, and then allowing that to be overridden w/ a flag like this from terra:
https://github.com/terra-money/core/blob/9ef67cf1c441bc51937d6285e9cdc5f90cd3b3dd/cmd/terrad/root.go#L242

I think setting it to 100mb is fine for now. Lets not forget this software is meant to be run by many people on smaller machines not only people who can afford bigger machines. Validators should manually configure their node

That's the big-big hope sir:
https://twitter.com/gadikian/status/1488244560442519553
Many tiny computers.


Print run duartion

It would be great if Flank prints the duration of the entire execution at the end. The intermediate steps duration breakdown is really great! But it would be really nice if, at the end of everything, Flank could print the combined duration.

We potentially see 3 different possible runs to print and they can be explained as follows:

Entire execution of the Flank run where it is timed from the start of Flank Execution and until it shuts down.

Entire execution of the interaction with Google Firebase CLI, i.e. The time from when the tests execute till when they shutdown.

Total cost duration, a flank estimate of the time that is billed for (Ultimately the correct source of truth will be the final billing cost)

Can be calculated by simply summing up the intemediatry steps.

Might need some work to seperate out where exactly the CLI is interacted with

Is a tricky one and needs to be discussed if we should approach it (Due to possibility of being incorrect and legal issues)

@jan-gogo WDYT?


in mac os x only one tab opens

I can't open any new tab in Mac os x elCapitan.
setting opens but I can't open one more terminal tab.

Please include a screenshot

I have this issue too.
Here is the StackTrace from the DevTools.
ERROR Error: Uncaught (in promise): Error: spawn ENOTDIR
Error: spawn ENOTDIR
    at exports._errnoException (util.js:1022) [angular]
    at ChildProcess.spawn (internal/child_process.js:313) [angular]
    at Object.exports.spawn (child_process.js:399) [angular]
    at ScreenPersistenceProvider.<anonymous> (persistenceProviders.ts:105) [angular]
    at Generator.next (<anonymous>) [angular]
    at fulfilled (colorPicker.component.scss:8) [angular]
    at Object.onInvoke (ng_zone.ts:261) [angular]
    at Object.onInvokeTask (ng_zone.ts:253) [angular]
    at exports._errnoException (util.js:1022) [angular]
    at ChildProcess.spawn (internal/child_process.js:313) [angular]
    at Object.exports.spawn (child_process.js:399) [angular]
    at ScreenPersistenceProvider.<anonymous> (persistenceProviders.ts:105) [angular]
    at Generator.next (<anonymous>) [angular]
    at fulfilled (colorPicker.component.scss:8) [angular]
    at Object.onInvoke (ng_zone.ts:261) [angular]
    at Object.onInvokeTask (ng_zone.ts:253) [angular]
    at rejected (/Volumes/Terminus 1.0.0-alpha.16/Terminus.app/Contents/Resources/builtin-plugins/terminus-terminal/…:33290) [angular]
    at Object.onInvoke (ng_zone.ts:261) [angular]
    at Object.onInvokeTask (ng_zone.ts:253) [angular]
defaultErrorLogger @ errors.ts:42
ErrorHandler.handleError @ error_handler.ts:69
next @ application_ref.ts:366
schedulerFn @ event_emitter.ts:96
SafeSubscriber.__tryOrUnsub @ Subscriber.ts:252
SafeSubscriber.next @ Subscriber.ts:204
Subscriber._next @ Subscriber.ts:135
Subscriber.next @ Subscriber.ts:95
Subject.next @ Subject.ts:61
EventEmitter.emit @ event_emitter.ts:78
NgZone.triggerError @ ng_zone.ts:87
onHandleError @ ng_zone.ts:283
ZoneDelegate.handleError @ /Volumes/Terminus 1.0.0-alpha.16/Terminus.app/Contents/Resources/app.asar/node_modules/zone.js/dist…:369
Zone.runGuarded @ /Volumes/Terminus 1.0.0-alpha.16/Terminus.app/Contents/Resources/app.asar/node_modules/zone.js/dist…:141
_loop_1 @ /Volumes/Terminus 1.0.0-alpha.16/Terminus.app/Contents/Resources/app.asar/node_modules/zone.js/dist…:604
drainMicroTaskQueue @ /Volumes/Terminus 1.0.0-alpha.16/Terminus.app/Contents/Resources/app.asar/node_modules/zone.js/dist…:613

Can confirm i am also getting this issue

Please post the output of lsof -p 12345 -Fn, where 12345 is your shell's PID

For me it's
p99225
n/Users/olivier
n/bin/bash
n/usr/lib/dyld
n/private/var/db/dyld/dyld_shared_cache_x86_64h
n/dev/ttys007
n/dev/ttys007
n/dev/ttys007
n/dev/ttys007

Thanks

I Just found something.
If you focus the setting tab, you can open a new shell tab.
But you can't if you are focusing a shell tab.
Hope this helps.


Stepper not showing

I have the following markdown inside deck.mdx:
# Page one

---

# Page two

---

# Page three

As you can see it renders the slides but it doesn't render the current step at the bottom. Is this a bug or a setting I have to enable manually?

In v2, the dot indicator at the bottom was removed. You can add your own custom Provider to show UI like that – or if you want to wrap a custom Provider into a theme, I'd accept a PR

I'll close this as it's not an actual bug then =), thanks for the reply!


Fix react-hot-loader/webpack lookup

Coverage remained the same at 93.252% when pulling 74e69d94e650a28a4aaec43757791eeffc2f79b0 on 55-fix-loader into 9054afb15e5213da63f57d1d7910afd40947a033 on master.