Checkstyle XML config file link is broken

Bug Report
Describe the Bug
The link to the Checkstyle XML Document on the styleguide page is broken.
Steps to Reproduce
Steps to reproduce the behavior:

Go to 'https://eclipse-edc.github.io/docs/#/submodule/Connector/styleguide'
Click on 'config written in XML'
See 404 error

Possible Implementation
Remove the "#/" from the link.

thanks for transferring the issue again, I did some investigations and it seems unfixable currently because docsify is not able to handle noCompileLinks and relativePath altogether.
I opened an issue on the docsify repo https://github.com/docsifyjs/docsify/issues/1987

to be honest, I really think that it does not make a lot of sense to have the checkstyle configuration in this repo, as it is getting outdated anyway, I suggest to remove that part and leave the developer to discover that as build stage in every repository

Closed by #159


Flutter fails to launch iOS simulator when Simulator App is running but Device Window is closed

Originally created @ Dart-Code/Dart-Code/issues/3277 and moved her as per the suggestion below from @DanTup

Thanks for the report! This looks like it'll need fixing in Flutter (it reproduces using flutter emulator --launch ios - which is essentially what VS Code is falling) so should be raised at https://github.com/flutter/flutter/.
Thanks!

Problem
Flutter fails to open the iOS Simulator when the device window is closed but the Simulator app is actually running (i.e. shown in the dock like in the image below:)

When the Simulator is in the above state, the following process will run for few minutes before it times out:

To reproduce:

open a flutter project with VS Code in MacOS where the project is for both iOS and Web.
Open the iOS simulator
Close the iOS simulator without quitting it (cmd + w or the 'x' red button).
Wait few seconds until VS code switches to Chrome (web-javascript) at the bottom.
Try to change the device back to iOS
waiting for iOS simulator to connect will run for few minutes before it times out saying that it couldn't launch the device.

Expected Behavior
The device should be shown again.
If this is an issue with the Simulator App itself, please consider one of those alternatives:

provide a way to terminate the process (ie waiting for iOS simulator to connect).
show a message that the simulator app needs to be quitted.

Additional Context
This usually happens when I switch between Web and iOS development and I close the simulator instead of quitting it. Even when I realize the situation, there is no way to terminate waiting for iOS simulator to connect.
Versions:

dart-code extension v3.21.1
VS Code version v1.55.1
Flutter Version:

Flutter 2.0.4 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b1395592de (11 days ago) • 2021-04-01 14:25:01 -0700
Engine • revision 2dce47073a
Tools • Dart 2.12.2

I have a vague memory of this coming up before - may have been this PR but I think it had to be reverted.

Hi @osaxma
I've been experiencing this issue lately but I didn't figure exact steps to replicate it. Now I can reproduce it thanks to you
Preview

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.4, on macOS 11.2.3 20D91 darwin-x64, locale
    en-GB)
    • Flutter version 2.0.4 at /Users/tahatesser/Code/flutter_stable
    • Framework revision b1395592de (11 days ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 89.0.4389.114

• No issues found!

[✓] Flutter (Channel master, 2.1.0-13.0.pre.587, on macOS 11.2.3 20D91
    darwin-x64, locale en-GB)
    • Flutter version 2.1.0-13.0.pre.587 at
      /Users/tahatesser/Code/flutter_master
    • Framework revision 0fdc4e8805 (6 hours ago), 2021-04-12 18:50:00 -0700
    • Engine revision 8863afff16
    • Dart version 2.13.0 (build 2.13.0-222.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • iPhone 12 (mobile) • 83060656-28E7-44CD-801E-B11EC3EC89BA • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-x64
      • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)       • chrome                               • web-javascript
      • Google Chrome 89.0.4389.114

• No issues found!

Any workaround for this? For me it's happening all the time (whether the iOS simulator is already running, or started by VSCode)

~Any workaround for this? For me it's happening all the time (whether the iOS simulator is already running, or started by VSCode)~ Nevermind I managed to sort it out

Hey Andrea, how did you resolve this?
My M1Pro is having this issue and it is so frustrating.
I've checked and downloaded all tools, deleted and re-installed Xcode, Preferences are fine; yet it's not connecting.

This seems like a bug, Why would it have so much trouble connecting to a simulator that exists....
I have the same problem on: 3.5 GHz Dual-Core Intel Core i7

I am able to reproduce this issue. There is no way to terminate the waiting for iOS simulator to connect and the only way to launch the iOS simulator is to Quit Simulator and start the simulator again.

any updates in this issue?

I had the same issue on my MacBook pro, M1 Max, trying all variants of starting simulator first, flutter: select device in VS code first, etc. After rebooting it worked fine just doing flutter: select device from VS code


Local Quick Start Guide doesn't work

I am running wandb server on 1 computer and trying the quickstart example code on another computer all on my local network.
On the computer I am trying to run the quickstart guide on I cannot get wandb.init() to work in  Python. The output I get is:
wandb: W&B API key is configured. Use `wandb login --relogin` to force relogin
wandb: Network error (ConnectionError), entering retry loop.
wandb: Network error (ConnectionError), entering retry loop.

I am running wandb login on the command line before running the Python scrip that calls wandb.init()
I am starting wandb server with
wandb server start -e HOST=http://<IP_ADDRESS>:8080
and have tried the login with both
wandb login --host http://<IP_ADDRESS>/
wandb login --host http://<IP_ADDRESS>:8080
and can access the webpage from the computer I am trying to run wandb.init() in Python on.

It just appears to hang and I don't see any logs:
Default host selected: http://<IP_ADDRESS>
Find detailed logs for this test at: /tmp/tmphra6d1qd/wandb

Wandb verify works on the host machine:
Default host selected: http://localhost:8080
Find detailed logs for this test at: /tmp/tmp5pihk34z/wandb
Checking if logged in...................................................✅
Checking signed URL upload..............................................✅
Checking ability to send large payloads through proxy...................✅
Checking requests to base url...........................................❌
Connections are not made over https. SSL required for secure communications.
Checking requests made over signed URLs.................................❌
Signed URL requests not made over https. SSL is required for secure communications.
Checking CORs configuration of the bucket...............................❌
Your object store does not have a valid CORs configuration, you must allow GET and PUT to Origin: http://localhost:8080
Checking wandb package version is up to date............................❌
wandb version is not supported by your local installation. This could cause some issues. If you're having problems try: please run `pip install --upgrade wandb==0.13.9`
Checking logged metrics, saving and downloading a file..................✅
Checking artifact save and download workflows...........................✅

@Inquisitive-ME I notice that on the non-host machine, it is trying to access http://<IP_ADDRESS> but the port is not specified.
I believe you tried logging in with wandb login --host=http://<IP_ADDRESS>:8080 --relogin correct? If so, can you delete ~/.netrc and `~/.config/wandb/settings and then log in again?
Then when you run wandb login --host=http://<IP_ADDRESS>:8080 --relogin it should point to correct port.
Running wandb verify should print Default host selected: http://<IP_ADDRESS>:8080 at that point and I believe this will resolve the issues with the API not connecting as well
Thank you,
Nate

Hi @Inquisitive-ME, I just wanted to follow up and see if that helped or if the API is still not pointing to the correct port?
Thank you,
Nate

I noticed that when I run wandb status the api_key says null
  "api_key": null,
  "base_url": "http://<IP_ADDRESS>:8080",
  "entity": null,
  "git_remote": "origin",
  "ignore_globs": [],
  "project": null,
  "root_dir": null,
  "section": "default"

Could that be an issues?

I found that even if I login on the host machine with the host set to <IP_ADDRESS>:8080 it does not work. wandb verify also hangs. The host has to be set to http://localhost:8080 for it to work

@Inquisitive-ME on the non-host machine you are able to access the UI at <IP_ADDRESS>:8080 in the browser, correct?

@Inquisitive-ME that indicates the machine you're running wandb verify on can't connect to <IP_ADDRESS>.  This is likely due to firewall rules or other networking issues.

Okay, it seems like it's an issue with the computer I was trying to run as host. Are there specific ports that wandb uses. I would guess that a port that wandb uses is in use already?

Hi @Inquisitive-ME, wandb server will use 8080 by default but you can specify which port you would like the server to use by starting the container with the following command:
docker run --rm -d -v wandb:/vol -p <available port you would like to access the app on>:8080 --name wandb-local wandb/local

Hi @Inquisitive-ME, I wanted to follow up and see if you were able to make this work or if you had any other questions?


Variant search - backend

We need a enable the search of variants in the OpenTargets platform search.
Background
It should be possible to search on

variant id (chr_pos_ref_alt)

search on exact ”chr_pos_” then suggest all the variants at that location
when searching for full id it has to be exact match
remember there are long allele strings - will this be an issue for the API?
10^7 variants
also enable searching on synonyms

gnomad notations
rsid
etc.

Data should come from the variant output from gentropy: https://github.com/opentargets/issues/issues/3350
what happens when you search?

type and suggestions happens live suggesting the variant, fixing at the start and only recommending completed terms for exact of this starting string
type + hit return - search results page (variant object should be in the results)

should we return top genes - yes this can be pre-computed and weighted from transcriptConsequences.distance and transcriptConsequences.targetId.

once connected to the target
connect to the disease and drug in the same way as is done for other entities and return those as well.

Tasks

[x] ETL to generate the variant_search opensearch index
[ ] ETL to add variants into the disease and target search indices
[x] configure POS to build the backend including the variant_search index
[x] Enable the search API to search this variant_search index

Acceptance tests
How do we know the task is complete?

When I run the ETL, the variant search index is generated
When I run POS, the variant search index is added to the opensearch backend
When I query the API for the search endpoint, I can search for variants.

@carcruz and FE team feel free to start exploring this dataset from your side (as discussed in the team leads meeting)

disease and target labels should not be added to the variant search index. However, the variants should be added to the target search index.

So it means, when searching for a variant, we'll get targets and diseases suggested, but no variants are suggested if searched for a diseases or targets? I very much agree with this call.

As discussed this week, @gjmcn and @chinmehta will start building a FE feature on top of the current API & while james is on holidays. tagging @prashantuniyal02 for reference in the next two weeks.

May be out of scope for the MVP, but it would be great to have the ability to search by protein coordinates, as well as genomic, as part of the variant synonyms as for coding variants this is how people will recognise/remember them and want to look them up. This may already be covered but couldn't see reference above.
Some examples:
rs121913530: KRASG12C, NP_203524.1:p.Gly12Cys
rs186045772: CFTRF1074L, NP_000483.3:p.Phe1074Leu
The ProtVar team (contact: James Stephenson) has different kinds of mappings for genome > protein which may be useful.

As discussed in Aug 7th...

some BE work is still required, as reported by @jdhayhurst
@gjmcn and @chinmehta starting to think about FE requirements/ideas
it would be good to identify any pending BE requirements early so we can move on with UI implementation


Next release

fix: Updated unloading of account to properly close existing connections - 15 minutes dev time
fix: fixed event based sensors (e.g. current_day_rates) to be situated in the correct domain - 30 minutes dev time
feat(custom): removed deprecated default tariff override sensors - 5 minutes dev time
fix: Removed certain sensors for Octopus Home Pro as data is not available - 5 minutes dev time
feat: fixed issue with duel rate tariffs changing to non DD rates. Toggle is now available in config - 40 mins dev time


Added RAG project

Hello @Andrew-Jang  👋 I added the RAG access control project (permission-aware data filtering). It's open-sourced. Thanks for considering adding it.

btw https://www.raghut.com/ is pretty cool, it looks similar to dev hunt


Issue with nested jinja call and endcall syntax with v0.14.1

Describe the bug
sqlfmt error with jinja syntax for start and endcall that used to parse previously (in v0.13.0)
Similar to issue https://github.com/tconbeer/sqlfmt/issues/335
To Reproduce
Run sql-fmt on the following code from https://github.com/EqualExperts/dbt-unit-testing#different-ways-to-build-mock-values:
{% call dbt_unit_testing.test('customers', 'should sum order values to calculate customer_lifetime_value') %}

  {% call dbt_unit_testing.mock_ref ('stg_customers', {"input_format": "csv"}) %}
    customer_id, first_name, last_name
    1,'',''
  {% endcall %}

  {% call dbt_unit_testing.mock_ref ('stg_orders', {"input_format": "csv"}) %}
    order_id,customer_id,order_date
    1001,1,null
    1002,1,null
  {% endcall %}

  {% call dbt_unit_testing.mock_ref ('stg_payments', {"input_format": "csv"}) %}
    order_id,amount
    1001,10
    1002,10
  {% endcall %}

  {% call dbt_unit_testing.expect({"input_format": "csv"}) %}
    customer_id,customer_lifetime_value
    1,20
  {% endcall %}
{% endcall %}

Expected behavior
Should not fail to parse
Actual behavior
sqlfmt encountered an error: Encountered closing bracket '{% endcall %}' at position xxx, before matching opening bracket. Context: {% endcall %}

Additional context
Version v0.14.0 appears to break this and the fix in v0.14.1 did not resolve it

@AndrewLane this is fixed in the most recent commit. I'll release this as 0.14.2 later today

@AndrewLane this is fixed in the most recent commit. I'll release this as 0.14.2 later today

I tried it out and it works great.  Thanks!


Question about variables

Does this help manage global/local variables? I use Pointshop2 and have a massive item list(Over 7000). When players that have the -d3d9ex argument in their gmod launch cmd they get errors because of a massive table i have but dont get errors when i remove the table even though i have pointshop2 with over 7000 items. From what i can see itemmapping is stored locally on clients. How does pointshop2 do this?
I am a noob please help

Hmmm actually now that i think about it. Since I'm having pointshop2 use the table a generate maybe its not generating fast enough before the ps2 icons are created


feetx needs more checking

the feetx shouldn't have made it this far...
2020-06-19 16:39:37.491 [ERR]  BG: SendRawTransaction error: rejected transaction ff2a3cd1af3ae3790c38e5af8fcb15aea1f98e5f05c5850b2c87b15427ca9c19: transaction has no inputs

I'm unsure what vspd needs to check here. It already checks that the tx has an output paying the correct fee amount to the correct address - doesn't that automatically imply that the tx has inputs?
Does this error indicate a wallet bug? I wouldn't expect wallet to construct (or sign) transactions with outputs but no inputs.


How to draw such a beautiful structure？

    First of all, thank you for sharing the code and your perfect work.
   And I want to know how to draw the structure chart in your paper.Could you give me some recommended tools?Thanks! @ @junting
    Of course, I will also be very grateful if any of the friends here can give me some advice.

@JingangLang You can draw diagrams like this using Google Drawings (drawings.google.com) or Google Slides. Just play around with the cube shape until it looks like a layer.


Check that second parameter of Tonic.add is a kebab-case string (if provided)

the regex is ^[a-z]+(-[a-z]+)*$

Hi,
I’m interested in working on this issue. Could you please assign it to me? This will be my first contribution, and I’m excited to get started. Let me know if there are any specific guidelines or steps I should follow.
Thank you!
@chicoxyzzy


Fix small bug.

Not really a bug.  Not sure I want to make this change either since it will require any user of the library to update their services accordingly.  You can customize the urls in your own project if you want.

Oh, It is small bug in your README.rst document but it isn't important:
Define url in 'Installation' path:
path('api/refresh/', TokenRefreshView.as_view(), name='token_refresh'),
Using in "Usage" path:
http://localhost:8000/api/**token**/refresh/

Ahh, my apologies @quybvs .  Honestly, it's been a while since I've worked really actively on this project so I forgot I didn't include a default url config anywhere.  I didn't realize you were just talking about wording in the README.  Thanks for catching that and yeah, let's make that correction!
Sorry again for misunderstanding you and thanks for being patient.


Nebula link doesn't load from Youtube

The youtube link shows in Nebula, but not the other way around.
Firefox 107.0 (64-bit) X11, Linux (KDE Neon 22.04 jammy)
NebulaEnhance 1.2.0

So... YouTube recently launched quite a big update (channel names changed etc), it seems that broke stuff for me. It's not a Firefox-specific issue, I will investigate this and how to fix it.

The problem was less than I thought, it was simply the channel URL that was changed, which meant I didn't find the symbol. That is fixed now, and even the button design updated to the new style. I'll probably push the new update tomorrow.


Forcekits: too difficult for new users, or useful

After receiving feedback from a new user, Jeremy, on forcekits, I think we should discuss again how useful they are.
-- Passing functions to other functions is somewhat unfamiliar for beginners
-- It is unclear what does a forcekit actually do - maybe better documentation/examples will solve this?
What do others think about forcekits?
Max

To clarify, I was mostly saying they seemed a little inelegant to have, given that there is only one ("polymerChains"). I also think example.py might benefit from having the code currently in forcekits.py explicitly included and commented upon.


No highligh_char on final refresh

Quick fix for #726

I have verified that this PR will fix the issue in https://github.com/kkawakam/rustyline/issues/726
Thank you!


Transactions stopped to be broadcasted

the transactions dont successfully. Pls confirm your Indexer
2 transactions with fail
459f44e157b6e9f379c74156fb60ab375f8afc87cc605b5f2f354f55aadc3223
843f4c899b187800ce979ec36e55149f249ee0b88be455323a82187105188982

Yes, this is a long standing issue. I advise you to use https://www.smartbit.com.au/api PUSH Transaction method for now. I need to take time to fix this issue.

I don't think so, I have not found the root cause yet.
The reset works when the indexer is stuck, but this is not the case here.

tryign to do something about it now.

The server is down. Trying to bring it up.
Updating it brought a bunch of worm's can caused by outdated tooling on the server.

Hello,
Any prediction to correct the problem?
If I want to create my own QBitNinja server will the problem occur?

Hi, this problem is happening again, pls check !

I don't think that is the same issue. You probably did not sent enough fees. What broadcast give you back?

yes, this is giving false positive in all transactions
Em seg, 11 de jun de 2018 às 22:17, Nicolas Dorier<EMAIL_ADDRESS>escreveu:

I don't think that is the same issue. You probably did not sent enough
fees. What broadcast give you back?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MetacoSA/QBitNinja/issues/60#issuecomment-396434917,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJAA7hVhKhYXlD404nxpf8MveeXbPuRlks5t7xajgaJpZM4UAi57
.

the broadcast is not giving error, only a false positive I did the test in 10 addresses with high fee
Enviado do meu iPhone
Em 11 de jun de 2018, à(s) 22:17, Nicolas Dorier<EMAIL_ADDRESS>escreveu:

I don't think that is the same issue. You probably did not sent enough fees. What broadcast give you back?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I will take a look later this week. Sorry I have been busy. As a work around, you can broadcast on https://www.smartbit.com.au/api

I discovered the problem I was giving, the blockchain only allows 26 unconfirmed transactions, now I'm sending bigger fees
thnks

@NicolasDorier could you restart QBitNinja please, because it seems it stopped broadcasting the transactions.
Thank you in advance

@NicolasDorier thx it works now


failed to create pileup

Hi,
I have used longshot before without problems but now it gives me error messages. The cram file is created using a epi2me workflow.
martin@209a4324f86d:/data/longread/Caroline/sample10/ubam$ longshot -A -P --bam sample10.filtered.cram --region chr15 --ref /data/longread/Caroline/T2T-CHM13v2_reference.fasta --out_bam sample10_T2T_longshot.bam --out sample10_longshot.vcf
2022-10-18 11:38:03 Automatically determining max read coverage.
2022-10-18 11:38:03 Estimating mean read coverage...
[E::easy_errno] Libcurl reported error 60 (SSL peer certificate or SSH remote key was not OK)
[W::find_file_url] Failed to open reference "https://www.ebi.ac.uk/ena/cram/md5/914e0733351aaa9a4354ca8efb527401": Input/output error
[E::fai_build3_core] Failed to open the file /longread/Caroline/sample10/ubam/workspace/e3/7b717de1082c541c3103ae0e7dab2b/T2T-CHM13v2_reference.fasta
[E::refs_load_fai] Failed to open reference file '/longread/Caroline/sample10/ubam/workspace/e3/7b717de1082c541c3103ae0e7dab2b/T2T-CHM13v2_reference.fasta'
[E::cram_get_ref] Failed to populate reference for id 14
[E::cram_next_slice] Failure to decode slice
error: Error calculating mean coverage for BAM file.
caused by: Error reading BAM pileup.
caused by: failed to create SAM/BAM/CRAM pileup
Any idea what's going on?

Not the dev, but looks like a cram problem. It can't find your reference FASTA. Can you convert to BAM and retry, that has solved similar problems for me with crams recently.


Android sdk 34 java.lang.RuntimeException:Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created

Description
After upgrading the Android SDK from version 33 to 34, the app crashes on Android 34, although it runs smoothly on Android 33.
the following changes have been made:
buildscript {
ext {
    googlePlayServicesVersion = "+"
    firebaseMessagingVersion = "21.1.0"
    buildToolsVersion = "34.0.0"
    minSdkVersion = 21
    compileSdkVersion = 34
    targetSdkVersion = 34
    androidXAnnotation = "1.1.0"
    androidXBrowser = "1.0.0"
    pdfViewer = "3.1.0-beta.1"
}`
```

  However, upon launching the application, I encountered an issue where the App Crashes immediately on the launch. To investigate this, I pulled a Bug Report, which provided the following insights:

1. Caused by: java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created.
2. Caused by: java.lang.SecurityException: : One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a receiver isn't being registered exclusively for system broadcasts.
3. Caused by: java.lang.reflect.InvocationTargetException.
4. Caused by: android.os.RemoteException: Remote stack trace.

### Steps to reproduce

react-native run-android after running command its crashing

### React Native Version

 0.68.6.

### Affected Platforms

Runtime - Android

### Output of `npx react-native info`

```text
System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i7-12650H
    Memory: 6.42 GB / 15.68 GB
  Binaries:
    Node: 14.20.1 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 6.14.17 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 29, 33, 34
      Build Tools: 28.0.3, 29.0.2, 30.0.2, 30.0.3, 31.0.0, 33.0.0, 34.0.0
      System Images: android-26 | Intel x86_64 Atom, android-28 | Intel x86_64 Atom, android-28 | Google APIs Intel x86 Atom_64, android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom, android-29 | Google Play Intel x86 Atom_64, android-33 | Google APIs Intel x86_64 Atom, android-33 | Google Play Intel x86_64 Atom, android-34 | Intel x86_64 Atom, android-34 | Google APIs ARM 64 v8a, android-34 | Google APIs Intel x86_64 Atom, android-34 | Google Play ARM 64 v8a, android-34 | Google Play Intel x86_64 Atom
      Android NDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: AI-213.7172.25.2113.9014738
    Visual Studio: Not Found
  Languages:
    Java: <IP_ADDRESS> - C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot\bin\javac.EXE
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.1.0 => 18.1.0
    react-native: 0.70.0 => 0.70.0
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found
```

### Stacktrace or Logs

```text
FATAL EXCEPTION:
java.lang.RuntimeException: Unable to create application com.poi.MainApplication: java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created
    at android.app.ActivityThread.handleBindApplication(ActivityThread.java:7716)
    at android.app.ActivityThread.-$$Nest$mhandleBindApplication(Unknown Source:0)
    at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2478)
    at android.os.Handler.dispatchMessage(Handler.java:106)
    at android.os.Looper.loopOnce(Looper.java:230)
    at android.os.Looper.loop(Looper.java:319)
    at android.app.ActivityThread.main(ActivityThread.java:8919)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:578)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)
Caused by: java.lang.RuntimeException: Requested enabled DevSupportManager, but BridgeDevSupportManager class was not found or could not be created
    at
```

### Reproducer

https://github.com/facebook/react-native/issues/new?assignees=&labels=Needs%3A+Triage+%3Amag%3A&projects=&template=bug_report.yml

### Screenshots and Videos

_No response_

I wanted to stick with React Version 0.68.6., as my code runs for both Android and iOS.
In my project, 0.68.6. is compatible for both iOS and Android

I have tried upgrading react native version, but After upgrading react native 0.68 to 0.70 still i am facing issue

On this issue tracker we offer support only for the supported versions of React Native:
https://github.com/reactwg/react-native-releases/blob/main/docs/support.md#what-versions-are-currently-supported
You can seek support for older versions of React Native on StackOverflow


Use logfmt

What is the problem that this PR is trying to fix?
Restructure log output for easier ingestion by travis-honeytail.

Looks awesome! Can you maybe also change logger invocations to logger -t kill-old-containers?


fix indentation for master template

Fixes issue mentioned here

ci check this

would it be possible to backport this into v2.3.0? it breaks the install ( at least on bare-metal ), and apiserver doesn't start.


Declare PY_SSIZE_T_MAX/PY_SSIZE_T_MIN as const

Follow the lead of Cython/Includes/cpython/pyport.pxd to declare
PY_SSIZE_T_MIN and PY_SSIZE_T_MAX as const. This prevents Cython
from trying to override or assign to this "variable" when doing wildcard
imports.
Add a test that shows this behavior in one example.
Fixes https://github.com/cython/cython/issues/5562

Note: This PR morphed from just the test case to a patch that includes a modified version of the test as well.

Thanks


feature request: support override-dependencies for inline scripts

uv: 0.5.2
According to docs inline scripts support [tool.uv] settings but override-dependencies doesn't work for me:
error: TOML parse error at line 7, column 1
  |
7 | [tool.uv]
  | ^^^^^^^^^
unknown field `override-dependencies`

reproducer:
# /// script
# requires-python = ">=3.12"
# dependencies = [
#     "pymilvus",
# ]
#
# [tool.uv]
# override-dependencies = [
#     "milvus-lite; sys_platform == 'never'",
# ]
# ///

This is fixed in v0.5.3, which hasn't gone out yet: https://github.com/astral-sh/uv/pull/9229

Relevant PR is: https://github.com/astral-sh/uv/pull/9162


Error on download courses

anyone getting this same error?

[] : Downloading webpage..
[] : Extracting course information..
[*] : Downloading 'course-course'.
Traceback (most recent call last):
File "F:\udemy_extractor.py", line 67, in _get_csrf_token
return match.group(1)
AttributeError: 'NoneType' object has no attribute 'group'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "udemy-dl.py", line 646, in
main()
File "udemy-dl.py", line 604, in main
udemy.ExtractAndDownload()
File "udemy-dl.py", line 280, in ExtractAndDownload
self.login()
File "udemy-dl.py", line 39, in login
extract_info.login(self.username, self.password)
File "F:\Udemy\udemy\udemy_extractor.py", line 99, in login
csrf_token = self._get_csrf_token()
File "F:\Udemy\udemy\udemy_extractor.py", line 70, in _get_csrf_token
return match.group(1)
AttributeError: 'NoneType' object has no attribute 'group'

And yes, i'm using the latest version =)

Getting the following error:
[code]
[] : Downloading webpage..
[] : Extracting course information..
[] : Downloading 'complete exchange 2013 practical guide'.
[] : Trying to login as (@.)...
[-] Unable to extract login-form form
Traceback (most recent call last):
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy-dl.py", line 924, in
main()
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy-dl.py", line 829, in main
udemy.ExtractAndDownload()
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy-dl.py", line 291, in ExtractAndDownload
self.login()
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy-dl.py", line 40, in login
extract_info.login(self.username, self.password)
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy_extractor.py", line 146, in login
login_form      = self._fill_login_form(login_popup, username, password)
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy_extractor.py", line 97, in _fill_login_form
login_form = self._form_hidden_inputs('login-form', webpage)
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy_extractor.py", line 87, in _form_hidden_inputs
return _hidden_inputs(form)
File "C:\Users*\AppData\Local\Programs\Python\Python36\Scripts\udemy_utils.py", line 80, in _hidden_inputs
html = re.sub(r'<!--(?:(?!', '', html)
File "C:\Users**\AppData\Local\Programs\Python\Python36\lib\re.py", line 191, in sub
return _compile(pattern, flags).sub(repl, string, count)
TypeError: expected string or bytes-like object
[/code]

I have this error - windows 10
[-] Unable to extract login-form form
Traceback (most recent call last):
File "udemy-dl.py", line 936, in
main()
File "udemy-dl.py", line 891, in main
udemy.ExtractAndDownload(quality=res)
File "udemy-dl.py", line 303, in ExtractAndDownload
self.login()
File "udemy-dl.py", line 40, in login
extract_info.login(self.username, self.password)
File "D:\udemy-dl-master\udemy_extractor.py", line 147, in login
login_form      = self._fill_login_form(login_popup, username, password)
File "D:\udemy-dl-master\udemy_extractor.py", line 98, in _fill_login_form
login_form = self._form_hidden_inputs('login-form', webpage)
File "D:\udemy-dl-master\udemy_extractor.py", line 88, in _form_hidden_inputs
return _hidden_inputs(form)
File "D:\udemy-dl-master\udemy_utils.py", line 80, in _hidden_inputs
html = re.sub(r'<!--(?:(?!', '', html)
File "C:\Python27\lib\re.py", line 155, in sub
return _compile(pattern, flags).sub(repl, string, count)
TypeError: expected string or buffer

@tofanelli , @sv000008 , @jonathans96 check #98 , #100


Icons / CSS externally

Since sveltestrap pulls the icons via css import from jsdeliver:
@import url('https://cdn.jsdelivr.net/npm/bootstrap-icons@1.3.0/font/bootstrap-icons.css');
i have two concerns with that:
a) why don't you omit the version, like in
https://cdn.jsdelivr.net/npm/bootstrap-icons/font/bootstrap-icons.css
which always gives you the latest version
do you really think this is too risky?
but
b) is much greater @ least for me/us:
i have privacy concerns with that and cannot build sites with that included.
Do you think it would be possible in any way to use npm "bootstrap-icons" instead? so a local variant?
Maybe just add an option to omit the import statement at install-time so i can always have the latest V. of sveltestrap but must include / import the fonts/svg's myself?
I hope i did make myself clear, it's very early ... ;)

Hi there, yes thanks understood.
This was a MVP to make icons easy basically. The alternatives I see are to:

Make the URL configurable or overridable
Make people include the external CSS and don't import in this library
Make an additional Icon component that does not include styles (without totally confusing people with 2 icon components)
4 Embed the SVG icons directly in this library
Make a separate sveltestrap-icons library

2 & 3 are related solutions
4 is kind of a hassle without making library huge
My concern with 2 is just the "Icons aren't working" issues, and then people suggesting to support 4 in addition.
Would 1 help if we could do it in a clean way?

The external (local) css won't help since you'd need the fonts too. Plus: the css has likely the "wrong" (unwanted) path to the fonts.
I would be willing to spend some time with this, although I'm fairly new to svelte and might not have the best view for this kinda decisions.

The CSS uses relative paths to the font files so you can host yourself too I believe.
I think the alternative is using the svgs directly, but feels like we should start a separate icon package vs combining that many files into sveltestrap bundle

Update on this, I think I'm going to go with option 2 above (Make people include the external CSS and don't import in this library) for next versions of sveltestrap (v4, and v5 for Bootstrap 5).
I don't see any better options given that the version of icons will get stale if I include here.  Thanks for the issue, will update in next week or two.

Hello! Can you please hurry with this?

Hi we'll get v4 out soon, but to be honest this Icon component doesn't do much much.  It just outputs a tag with the icon classname:

So you can easily use the above with your own icon font in the meantime.

Hi @michapixel, @kkarpeev ,
We've published<EMAIL_ADDRESS>if you can please try this and confirm it works for you.  You have to directly install the beta version as it's a prerelease:
npm install<EMAIL_ADDRESS>
Please see the instructions here on Icon usage.
https://github.com/bestguy/sveltestrap/tree/v4#note-on-icons
If this works okay we can publish v4 officially.

Will close with release of v4.0.0, which removes hard-coded CDN.
Usage documentation for Icon is here: https://github.com/bestguy/sveltestrap#note-on-icons


[mkldnn-v1.0] Adopt autograd.record() context to RNNOp

Description
Should fix ISSUE https://github.com/apache/incubator-mxnet/issues/16256
Checklist
Changes

[x] Use ctx.is_train || ctx.need_grad to check if it needs RNNOp to run into forward_training mode.
[x] Nit: indent of a if (ctx_.dev_type == kGPU) branch.
[x] Both cpu and gpu context are adopted.

@pengzhao-intel @TaoLv

@szha Please help to review. Thanks!

Currently, only the CPU context was adopted. @TaoLv @szha
I think the GPU might have the same problem. Need help from @DickJC123 @ptrendx . Thanks.

please rebase the code and the previous ci issue is fixed now.


Automatically detect the saddle point problem in BlockOperator

Hi,
I have a 2x2 block system that is singular in the two diagonal blocks. Therefore, I want to form a different index set of this 2x2 block system based on if the diagonal is 0 or not. I can get the diagonal vectors of two diagonal blocks and thus get the needed index sets. But I am not sure if there is an easy way to get all the sub-matrices and form a new 2x2 system.
I think I want to GetSubMatrix but remain a SparseMatrix and eventually glue things together. Is there an easy way to implement that, at least for a serial case?
Basically, I want to realize -pc_fieldsplit_detect_saddle_point inside mfem. Alternatively, I can use petsc but I am not sure it will work on the block system through the current mfem's petsc interface, which gives MATNEST instead of MATAIJ. Not sure one can call -pc_fieldsplit_detect_saddle_point on a nest matrix whose index set has already been determined through the nest structure.
Qi

close for now. there might be another workaround.


Homepage Wireframe

Issue for Wireframing the homepage

I think a header is enough for the moment, we don't need a landing page, something like this
https://www.figma.com/proto/oOFsQ2ecdyYjnK91yOKPDR/Raid-Guild-Site-Planning?node-id=91%3A269&scaling=min-zoom

I think a header is enough for the moment, we don't need a landing page, something like this
https://www.figma.com/proto/oOFsQ2ecdyYjnK91yOKPDR/Raid-Guild-Site-Planning?node-id=91%3A269&scaling=min-zoom

@ziedtouibi I am happy with this as an example - can you create an issue to have this built with any assets required attached?

Is this picked up by anyone? I cannot see any issue created for it with the assets required. I would like to jump start on it.

@MohitPopli if you create a figma account, then the following URL should let you poke around the designs, and export assets etc...
Use them as placeholder until @ziedtouibi can get back to us :smiley:
https://www.figma.com/file/oOFsQ2ecdyYjnK91yOKPDR/Raid-Guild-Site-Planning?node-id=36%3A0


Rename calc_bc.html to honkalculate.html

Um, did you really just do these renames without updating the HTML files?


Option -n causes DevAudit crash

Steps to reproduce
Run this command:
…/DevAudit//devaudit.exe nuget -n --file …/DevAudit/Examples/v6a6.packages.config

Expected results
It writes all the output, like without -n.
Actual results
After writing some output, it crashes:
--[6/7] [CVE-2019-11358]  Improper Neutralization of Input During Web Page Generation ("Cross-site Scripting")
Runtime error! DevAudit will now terminate.
[ERROR] FATAL UNHANDLED EXCEPTION: System.FormatException: Input string was not in a correct format.
  at System.Text.StringBuilder.AppendFormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x000e7] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at System.String.FormatHelper (System.IFormatProvider provider, System.String format, System.ParamsArray args) [0x00026] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at System.String.Format (System.IFormatProvider provider, System.String format, System.Object[] args) [0x00029] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at System.IO.TextWriter.Write (System.String format, System.Object[] arg) [0x00007] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at System.IO.TextWriter+SyncTextWriter.Write (System.String format, System.Object[] arg) [0x00000] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at (wrapper synchronized) System.IO.TextWriter+SyncTextWriter.Write(string,object[])
  at System.Console.Write (System.String format, System.Object[] arg) [0x00016] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at DevAudit.CommandLine.Program.PrintMessage (System.ConsoleColor color, System.String format, System.Object[] args) [0x00033] in <0fa67779253c47e4b5287cfb02896d82>:0
  at DevAudit.CommandLine.Program.PrintAuditResultMultiLineField (System.ConsoleColor color, System.Int32 indent, System.String field, System.String value) [0x000cd] in <0fa67779253c47e4b5287cfb02896d82>:0
  at DevAudit.CommandLine.Program+<>c__DisplayClass40_0.<PrintPackageSourceAuditResults>b__11 (DevAudit.AuditLibrary.IVulnerability v) [0x00097] in <0fa67779253c47e4b5287cfb02896d82>:0
  at System.Collections.Generic.List`1[T].ForEach (System.Action`1[T] action) [0x0002c] in <d0e12f672b88444ab4b6d9b2ecf20142>:0
  at DevAudit.CommandLine.Program.PrintPackageSourceAuditResults (DevAudit.AuditLibrary.AuditTarget+AuditResult ar, DevAudit.AuditLibrary.AuditTarget+AuditResult& exit) [0x00450] in <0fa67779253c47e4b5287cfb02896d82>:0
  at DevAudit.CommandLine.Program.Main (System.String[] args) [0x01ee0] in <0fa67779253c47e4b5287cfb02896d82>:0

Analysis
I am not 100% sure about the cause, but it looks like the issue is in {} on the output. It seems that it tries to interpret the description string as a formatting string, which fails due to the {}.

Not able to reproduce this, what version of DevAudit are you using and on what platform - Windows or Linux?

Debian 10
Ah, I forgot the DevAudit version: <IP_ADDRESS>

Could you try with this build:
DevAudit_<IP_ADDRESS>.tar.gz
and see if it still happens.

Thank you, it is OK with this version. I was using <IP_ADDRESS>, as it was (and still is) the latest version in releases on Github.

OK we'll make a 3.4 release on GitHub soon. Closing this for now. Thanks for reporting this.

Hey, it's been a little while now and I am still seeing 3.3 as the latest release on the releases page. Some of the code hints at a 3.4, but it isn't there.


[Spike] Weave GitOps UI

As an outcome of this conversation, we want to explore the potentialities of providing Weave GitOps UI to our customers.
TODO

[ ] prepare
[ ] deploy
[ ] test
[ ] demo with customers

Related: https://www.infoq.com/news/2023/01/google-gitops-observability/

Weave GitOps UI Summary
Sources and Installation
Here is the Weave GitOps Github repository. Inside, there is a Helm Chart for GitOps Server that provides the UI. The Weave GitOps documentation can be found here. As stated by this documentation here, the GitOps Server may be installed with the help of GitOps CLI, which actually generates the HelmRepository and HelmRelease Flux CRs, a natural choice in the Flux environment.
Configuration
I split the configuration into categories to make it more readable. The last section carries the HelmRelease CR that I used to install the dashboard.
Access
The upstream Helm Chart provides Ingress CR that can be used to provide outside-originated access to the Dashboard. Configuring TLS requires certificate which can be provided by Cert Manager, by annotating the Ingress CR with the cert-manager.io/cluster-issuer: <issuer_name>. Our clusters seem to have at least two issuers configured, the selfsigned-giantswarm and the letsencrypt-giantswarm, and for the tests I've choosen the former one.
Note however, I am not sure of how the issuers get created, nor if their existence is the case for every installation, yet I expect it to be. Both carry the last-applied-configuration annotation that suggests both were created manually, probably as part of the bootstrapping process.
The relevant values.yaml part that must be configured is here, for example:
ingress:
  enabled: true
  className: nginx
  annotations:
    cert-manager.io/cluster-issuer: selfsigned-giantswarm
  hosts:
    - host: gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn
      paths:
        - path: /
          pathType: ImplementationSpecific
  tls:
    - hosts:
        - gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn
      secretName: gitops-tls

Login
As stated in here, there are two ways to login to the Dashboard, either by using a static user account, or by using the OIDC provider. The recommended way is the latter, see:

The recommended method is to integrate with an OIDC provider, as this will let you control permissions for existing users and groups that have already been configured to use OIDC.

The obvious downside of using the static account is, it seems it must be shared accross all the parties that require access for I see no way to configure the Dashboard with multiple of such accounts. And this in turn means there is no way to make the permissions granular, either all parties can do what the static account can do, or none of them.
The static account however seems helpful as a fallback login method for the admin users, in case the IdP provider or ingress are down, for example. For the PoC I configured the OIDC method only.
The login method can be decided with this flag, both methods can be used at the same time. This is the relevant values.yaml part that allows to configure the flags.
For the OIDC provider this part of the values.yaml must be configured, for example:
oidcSecret:
  create: true
  clientID: fakeclient
  clientSecret: fakesecret
  issuerURL: https://dex.g8s.giraffe.pek.aws.k8s.adidas.com.cn
  redirectURL: https://gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn/oauth2/callback

For this to work, Dex or any other IdP must be configured with a new client for a Dashboard, see an example for Dex:
web:
  allowedOrigins:
  - ...
  - https://gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn
...
staticClients:
- id: fakeclient
  redirectURIs:
  - https://gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn/oauth2/callback
  name: fakeclient
  secret: fakesecret

Permissions
For the RBAC configuration the Weave GitOps states the following:

The general recommendation is to use OIDC and a small number of groups that are Weave GitOps can impersonate.
OIDC is the recommended method for managing authentication as it decouples the need to manage user lists from the application, allowing it to be managed via a central system designed for that purpose (i.e. the OIDC provider).

This makes sense, using the OIDC and impersonation relieves the Dashboard from necessity of configuring the RBAC, and instead makes it possible to rely on already established architecture. In such case the only permissions the Dashboard must configure are:

rights to impersonate users and groups, this can be limited by the Kubernetes RBAC resourceNames list.
rights to read both, the oidc-auth and the cluster-user-auth Kubernetes Secrets.
read namespaces.
list CRDs.

Which are minimal. The rest, i.e. what user can see and act upon, is determined by his permissions, and can be configured outside the Dashboard app.
An exception to this is naturally the static admin account. When its creation is being requested, the Dashboard app creates extra roles that grants it extended permissions  to read all the resources in the cluster-scope, and to in addition patch the Flux resources.
As stated by the documentation, see this, it should be further possible to limit which groups and users can be impersonated by the Dashboard, however in my opinion what is stated there is not entirely correct. It would be correct if the Helm Chart was configured differently. Let me elaborate of what I mean.
Currently the document encourages to limit users and groups the application can impersonate, and gives example of this:
rbac:
  impersonationResources: ["groups"]
  impersonationResourceNames:
    - admin
    - dev-team
    - qa-team

However, this configuration alone will result in an error, because impersonating a group requires specifying a user as well, the Kubernetes document, hence the users must also be specified under the impersonationResources.
But when the users resource is added, the Helm template that uses these fields generates the below ClusterRole on installation:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: flux-system-gitops-ui-weave-gitops
rules:
- apiGroups:
  - ""
  resourceNames:
  - admin
  - dev-team
  - qa-team
  resources:
  - groups
  - users
  verbs:
  - impersonate

It looked ok to me at the first glance, but the access didn't work, so I checked that up and it turns out it is a bad idea to put > 1 resources on the resources list and use the resourceNames at the same time. This was not immediately obvious to me, but now when I think of it, it makes total sense. In such scenario, the resourceNames limits apply to all the resources of a given rule, so it actually says:
impersonation to these users - none - and these groups - [admin, dev-team, qa-team] - is allowed

where I thought of it saying:
impersonation to these users - all - and these groups - [admin, dev-team, qa-team] is allowed

To make it work the intended way, the ClusterRole must be changed to:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: flux-system-gitops-ui-weave-gitops
rules:
- apiGroups:
  - ""
  resources:
  - users
  verbs:
  - impersonate
- apiGroups:
  - ""
  resourceNames:
  - admin
  - dev-team
  - qa-team
  resources:
  - groups
  verbs:
  - impersonate

But this is no supported by the upstream Helm Chart as of now. It can easily be done for Management Clusters, for if we decide to deploy it, it will be done with Flux that can patch the resources accordingly. If we are to offer Dashboard as an app, the upstream Helm Chart has to be either fixed there, or extended on our side.
HelmRelease CR
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: gitops-server
  namespace: flux-giantswarm
spec:
  chart:
    spec:
      chart: weave-gitops
      sourceRef:
        kind: HelmRepository
        name: gitops-server
  interval: 1h0m0s
  targetNamespace: flux-system
  values:
    additionalArgs:
      - "--auth-methods=oidc"
    nodeSelector:
      kubernetes.io/hostname: ip-10-0-5-47.cn-north-1.compute.internal
    logLevel: debug
    oidcSecret:
      create: true
      clientID: fakeclient
      clientSecret: fakesecret
      issuerURL: https://dex.g8s.giraffe.pek.aws.k8s.adidas.com.cn
      redirectURL: https://gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn/oauth2/callback
    rbac:
      impersonationResourceNames:
        - 'customer:giantswarm:Employees'
        - 'giantswarm:giantswarm:giantswarm-admins'
        - 'giantswarm-ad:giantswarm-admins'
      impersonationResources:
        - 'groups'
    ingress:
      enabled: true
      className: nginx
      annotations:
        cert-manager.io/cluster-issuer: selfsigned-giantswarm
      hosts:
        - host: gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn
          paths:
            - path: /
              pathType: ImplementationSpecific
      tls:
        - hosts:
            - gitops.g8s.giraffe.pek.aws.k8s.adidas.com.cn
          secretName: gitops-tls
      limits:
        cpu: 200m
        memory: 256Mi
  postRenderers:
    - kustomize:
        patchesJson6902:
          - target:
              version: v1
              group: rbac.authorization.k8s.io
              kind: ClusterRole
              name: flux-system-gitops-ui-weave-gitops
            patch:
              - op: add
                path: /rules/-
                value:
                  apiGroups:
                    - ""
                  resources:
                    - users
                  verbs:
                    - impersonate

Conclusions

Helm Chart is fairly easy and well maintained, with one exception mentioned above. But this should be fairly easy to fix in the upstream, or on our side. To be honest, I'm not sure of why it hasn't been patched already there, since it clearly cannot work the way it is configured now, but well, maybe no one has ever tried to limit the impersonation, or did it with patches like us or by some other means and has never mentioned this in the issues.
the impersonation idea is nice, we already have the well-settled RBAC thanks to RBACOp mostly, so not being force to configure something extra for the Dashboard is huge adventage for me. The only problem with the impersonation is the lack of flexibility, see the next point.
limiting impersonation, what users are encouraged to do when configuring the Dashboard, is not that flexible, what makes it useless in some scenarios. This is because it seems the the application tries to impersonate all the groups that are provided by the token, and fails when some of them are not listed in the resourceNames of ClusterRole. This is not a problem for us, because we already know which groups we are in, in both Github and Azure AD, but I am not sure how limiting process should be conducted for the customers, for I am not sure how many groups the customers tokens carry. For sure we know of some of the groups that are configured in both RBACOp and MCF, but if tokens contain more of them, then the authorization process will fail. But since we do not have many GitOps customers, maybe we could deal with them one-by-one, check of how they login, and if the groups we already know and use are complete set. If not then maybe we could check with them of how many more groups are included in the tokens.
the Flux Runtime part of the Dashboard requires some tricks for displaying flux-giantswarm resources. Namely, the namespace has to be labeld with the app.kubernetes.io/part-of: flux label. Otherwise, only the default namespace of flux-system is displayed, see here. I haven't seen it documented in any way, and this is already a second thing that is documented in a poor manner what makes me think of how many other things are yet to be discovered.
the overall security model is a bit worrying, I mean, I still think the impersonation idea is grate, but lack of flexibility makes me think of what is the right way to configure the application. We already have apps in our cluster that run under elevated privileges, and thus are able to impersonate other subjects, and we are fine with that because customer users cannot abuse these apps. So maybe we shouldn't be worried in this case neither, and instead allow the Dashboard to impersonate any user, without limitations, and only make sure we deploy it into a secured namespace. But I must admit that I don't feel overly confidend that someone may accidently impersonate me and delete the flux Kustomization 😄 On the other hand, if we decide to limit the groups the application can impersonate, then there is still a question opened of how to deploy such application then. For having a single instance of the application with Giantswarm's and customer's groups mixed together in the ClusterRole works more or less like users are still allowed to impersonate any group, aka no limitations. To limit the impersonation when the customer logs in, and do not limit it when Giantswarm employees login, it would require having two instances of the application.
Related to the above, Dex gives possibility to limit the groups that put into the token, I think, so maybe that is the way forward, i.e. limit the groups added to the token to the ones we already use within the cluster.

ToDos to make it part of GS offering

[ ] setup the Github repository for an app, and configure synchronization with the upstream Helm Chart
[ ] re-tag the GitOps Server image to make it available in our registries
[ ] configure Dex with a new client for GitOps Server, with possibility to limit the groups returned in the token
[ ] add Helm-related Flux resources into the MCF, and decide the overal deployment scenario, like single vs two Dashboard instances
[ ] add extra configuration to make sure flux-giantswarm resources are displayed, and impersonation is handled the right way, if needed


Word Candidate Window has problem when typing Chinese on iPad

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

Execute flutter run on the code sample.
Input Pinyin.

Expected results
The arrow keys and Tab keys can be used correctly.
https://github.com/flutter/flutter/assets/12608693/8c1ab5cb-15d5-4278-bad2-27639026d06e
Actual results

Word can not be selected by arrow keys.
The tab key cannot be used to change the type.

Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: const Center(child: TextField(maxLines: null)),
    );
  }
}

Screenshots or Video

Video demonstration
Simulator Screen Recording - iPad (10th generation) - 2023-09-25 at 15.13.17
https://github.com/flutter/flutter/assets/12608693/942a15de-c1d8-4e28-94a5-3aa4c1d3b68f

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.5, on macOS 13.5.2 22G91 darwin-arm64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] IntelliJ IDEA Community Edition (version 2023.1)
[✓] VS Code (version 1.80.0)
[✓] Connected device (3 available)

Hi @feimenggo. Thanks for filing this. I am able to reproduce this on the latest master and stable channels and this doesn't seem to be specific to iPads. This issue also would only occur with simulators/emulators, as there won't be arrow or tab keys in the case of physical devices.
Is there a reason you'd prefer to use the computer keyboard arrow keys in place of the devices? Can you provide a use case for such an instance?

Thanks for your response @feimenggo. I can reproduce this on the latest master and stable channels.
stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.13.5, on macOS 13.0 22A380 darwin-arm64, locale
    en-NG)
    • Flutter version 3.13.5 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 12fccda598 (6 days 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 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)

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

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 Pro Max (mobile)  • BB55E997-7F31-462D-B3B1-6B177D9B40C7 • 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 116.0.5845.187

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

• No issues found!

[✓] Flutter (Channel master, 3.15.0-7.0.pre.6, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.15.0-7.0.pre.6 on channel master at /Users/damilolaalimi/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d5a8cd3840 (3 hours ago), 2023-09-25 03:12:22 -0400
    • Engine revision e1c1022c2d
    • Dart version 3.2.0 (build 3.2.0-191.0.dev)
    • DevTools version 2.28.0-dev.12

[✓] 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)

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

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 Pro Max (mobile)  • BB55E997-7F31-462D-B3B1-6B177D9B40C7 • 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 116.0.5845.187

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

• No issues found!


added applyRouterMiddleware

I'll add some docs in @knowbody's new docs site thing, but here's the quick version:
// end users put the middleware in the `render` prop
<Router render={applyRouterMiddleware(useFoo())}/>

// a middleware returns an object w/ two methods
const useFoo = () => ({
  // this will wrap `RouterContext` to participate in rendering at the top level of
  // the Router rendering tree
  renderRouterContext: (child, renderProps) => (
    <FooRootContainer>{child}</FooRootContainer>
  ),

  // this will wrap the route components when RouterContext renders them
  renderRouteComponent: (child, routeComponentProps) => (
    <FooContainer>{child}</FooContainer>
  )
})

// typically these things just add something to context
const FooRootContainer = React.createClass({
  propTypes: { children: React.PropTypes.node.isRequired },
  childContextTypes: { foo: React.PropTypes.string },
  getChildContext() { return { foo: FOO_ROOT_CONTAINER_TEXT } },
  render() {
    return this.props.children
  }
})

// so that the route component containers or other components anywhere in the tree
// can access it
const FooContainer = React.createClass({
  propTypes: { children: React.PropTypes.node.isRequired },
  contextTypes: { foo: React.PropTypes.string.isRequired },
  render() {
    const { children, ...props } = this.props
    const fooFromContext = this.context.foo
    return cloneElement(children, { ...props, fooFromContext })
  }
})

What do you think of renaming the callbacks to:

renderRouterContext
renderRouteComponent

They're not constrained to be containers – e.g. useBaz specifically can just have
renderRouteComponent: child => cloneElement(child, { bazInjected })

LGTM.

So, if I had a custom render previously, I would have to inject the enhanced RouterContext, right? Kind of as following:
const RouterContextFactory = applyRouterMiddleware(useFoo());

<Router
  render={renderProps =>
    <Fetcher {...renderProps}>
      {RouterContextFactory(renderProps)}
    </Fetcher>
  }
/>

@ryanflorence thanks for adding this. Do you know if this ended up being documentet anywhere? Could not find anything in the official docs or in the referenced @knowbody docs. Would you want a PR to either?


Fix author

The creator of the theme appears to miscredited.
This theme was created by Bye, as indicated in the omg.lol GitHub repo and on the website itself:
https://github.com/neatnik/omg.lol/pull/205

@Vukkyy will add them immediately. Thanks for it. I was not able to find out who made it initially when making the repo, so had not added them!!

@Vukkyy I think it will take some time for it to get updated on the website. I guess it will occur when @luxonauta adds the next set of new themes and updates to the theme to the site. So maybe within a week or two.


regexp for multiline matches on spaces between words

this breaks some styles, and also looks very silly with short words like A, I etc... can we make multiline an option?

Yes, it should be. Thanks


improve the design of index.html

improve design
--
Add this to your pr message
Title
Number of files/folder added
changes made
I have;

[ ] Read contibuting.md
[ ] Followed all the guidelines
[ ] Linked my tool to the index.html file
[ ] updated readme.md

What kind of changes I can make??

you are free to make any kind of changes until the site looks good. you can add animations, svg, transformation and etc.

I can start working on the home page if anyone else is not working on it.

@shubham2704 Made some changes in the UI.

@shubham2704 Made some changes in the UI.
I have seen your pull request the changes you made are good but,
you should have Read contibuting.md, First.

@ADHIL-MOHAMMED-P-N Thank you for the contribution, it would take some time to merge you PR. As it's in review phase now.💚

@shubham2704 Ok


CQL formatter appending error String in code.

When creating a parameter with syntax error, the CQL parser is appending error String inside the CQL code.
ex:
parameter "test param" Interval<DateTime
returns
parameter "test param" [java.lang.Exception: [1:40]: missing '>' at '<EOF>']

 Interval<DateTime.

Hmm. I am unable to reproduce this. When I run parameter "test param" Interval<DateTime, I am getting the following back:
[java.lang.Exception: [1:40]: missing '>' at '<EOF>']

parameter "test param" Interval<DateTime

Which is the expected behavior. Are you using the most recent version of the formatter?

@c-schuler, @brynrhodes  and @chinbaj, I believe the concern is getting the java error message back, which would then have to be caught and disposed of in an authoring tool before the original CQL that was passed through can be resurfaced to the user. Is there a reason the method is concatenating the error message with the original CQL expression?

@JSRankins: I currently have a listener that detects syntax errors in the CQL and simply returns the error messages along with the original input. The reasoning is because "correctly" formatting syntax errors is hard =). I am open to suggestions on improving that.

Agree with Stan.
@c-schuler , Are you testing against just the one expression or a complete CQL library.
I believe when you try to send in a single expression, the "[java.lang.Exception: ..." is appended at the top.
When you send in the entire CQL library, it gets inserted with the expression.
Either ways, since it comes back as a part of the library, the authoring tool UI shows it back on screen and causes further errors/confusion.

Still unable to reproduce.
Tested:
library test version '0.0.000'
using QDM
valueset "Female Administrative Sex": '2.16.840.1.113883.3.560.100.2'
valueset "TestValueset": '1.1.111'
parameter "test param" Interval<DateTime
context Patient
define "TestDefinition":
	["Encounter, Performed": "TestValueset "]

Returned:
[java.lang.Exception: [10:0]: missing '>' at 'context']
library test version '0.0.000'
using QDM
valueset "Female Administrative Sex": '2.16.840.1.113883.3.560.100.2'
valueset "TestValueset": '1.1.111'
parameter "test param" Interval<DateTime
context Patient
define "TestDefinition":
	["Encounter, Performed": "TestValueset "]

Perhaps you can provide the library so I can test that?
How would you prefer the formatter to handle syntax errors?

The reasoning is because "correctly" formatting syntax errors is hard

Couldn't agree more. :-)
I checked further and the error insertion within the logic seems to be a side effect of the authoring tool code. So that's my bad.
But the larger point is that a library author is sending in a CQL library String and then getting back some additional Java exception messages as a part of the library.

How would you prefer the formatter to handle syntax errors?

Is it possible to have a flag that we can set on the formatter?
Based on the this flag being set/not-set, formatter can prepend/not-prepend syntax errors to the output.

Is it possible to have a flag that we can set on the formatter?

Yeah, I can do that. The output in that case (when syntax errors are encountered) would be the original input without formatting.
@brynrhodes: thoughts?

@jackcmeyer : thoughts

@chinbaj this sounds good to me

Yes, I like that solution, but I would want to verify that whether or not the error messages are included in the output, the errors are still available to the caller as structured exceptions, right?

I would want to verify that whether or not the error messages are included in the output, the errors are still available to the caller as structured exceptions, right?

It would be nice If it could be done.

Thanks for everyone's input. After discussion with Bryn, we believe the best solution is to return a structure like so:
Result(Errors[], input/output)
When there are errors in the CQL a list of errors will be returned with the original input. When no errors are encountered an empty error list will be returned with the formatted output.
If there aren't any objections, I will move forward with this implementation.

Seems fine with me. Thanks a lot for making this change,

Addressed in v.1.2.13-SNAPSHOT


[DOC] document download URL for core plugins

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
in most cases the core plugins can just be installed by running opensearch-plugin install ${pluginname}. however, if you're offline you need to have access to the ZIP file of the plugin and accordingly need to download it first.
for plugins hosted in other repositories this is not an issue, as they're (a) published as a github release and (b) usually published to maven central.
currently, the download URL for core plugins is not documented, thus it's not possible to just download them.
Version: n/a (all of them)
What other resources are available?
as discussed on slack with @reta & @saratvemulapalli who can provide the necessary input here

@Naarcha-AWS: please don't assign this to me - I don't know the URL to be documented 😅

@Naarcha-AWS please assign it to me, thank you!

i guess it's always https://artifacts.opensearch.org/releases/plugins/${PLUGIN}/${OS_VERSION}/${PLUGIN}-${OS_VERSION}.zip?

@rursprung I am waiting till https://github.com/opensearch-project/opensearch-build/issues/5188 is resolved so we would ensure no changes in the SNAPSHOT repo coordinates.


Compability for .NET Core 3.0 (preview 6)

In accordance with https://github.com/aspnet/AspNetCore/issues/3756 and https://github.com/aspnet/AspNetCore/issues/3612 this pull requests creates compability for .NET Core 3.0.
This PR contains the breaking change that the library has to target .NET Core 3.0.
Primary changes in Discord.OAuth2.csproj and DiscordHandler.cs.
Changes in Discord.OAuth2.targets are optional, eventually to be replaced later.

This PR should be the solution for #11.


The rest of Tera

This PR is split into commits, that's by far the best way to review it

Commit 1: Builds
Before

After

Commit 2: Errors
This commit looks larger than it actually is since I had to update the call site of every ctry! and cexpect! invocation
Before

After
Error message is now the page description

Commit 3: Crate Details
Before

After
The builds page is now always displayed

Commit 4: Source
Before

After
Now there's a few more icons used

Commit 5: Rustdoc
Before

After

Commit 6: Release Activity
Before

After

Commit 7: Build Queue
Before

After

cc @antifuchs
Closes #777, #740 and #786

I looked into trying to unify the details dropdown of crate/details.html and rustdoc/navigation.html but I'm nervous to try since navigation.html has lots of extra CSS classes, and I think that's a bit out of scope for this PR. I'd be happy to open an issue after this PR lands and do it as a follow-up, but it's quite a lot to do for one PR


Fix query timeouts when refreshing feature metrics

Context
The feature metrics dashboard is updated hourly by a background job that runs a number of DB queries in order to populate the dashboard. One (at least one) such query is frequently timing out - it's just too slow. This PR attempts to optimise that query.
Changes proposed in this pull request

[x] Add a GIN index to the audits.audited_changes column because this is the column on which we need to define the most important query condition.
[x] Add a redundant condition to the query that checks for the existence the work_history_completed key in the audited_changes column. This should take advantage of the index added above. There is already a condition that checks the value of the work_history_completed key but it doesn't look like it's possible to use an index for this.

Before index:
bat_apply_development=# explain analyze SELECT "audits"."auditable_id"
FROM "audits"
WHERE "audits"."auditable_type" = 'ApplicationForm'
  AND (audited_changes ? 'work_history_completed')
  AND (audited_changes#>>'{work_history_completed, 1}' = 'true')
  AND (created_at BETWEEN '2021-3-31' AND '2021-4-30');
┌───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│                                                                QUERY PLAN                                                                 │
├───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Bitmap Heap Scan on audits  (cost=9.14..204.28 rows=1 width=4) (actual time=0.849..1.897 rows=25 loops=1)                                 │
│   Recheck Cond: ((auditable_type)::text = 'ApplicationForm'::text)                                                                        │
│   Filter: ((audited_changes ? 'work_history_completed'::text) AND (created_at >= '2021-03-31 00:00:00'::timestamp without time zone) AND …│
│…(created_at <= '2021-04-30 00:00:00'::timestamp without time zone) AND ((audited_changes #>> '{work_history_completed,1}'::text[]) = 'tru…│
│…e'::text))                                                                                │
│   Rows Removed by Filter: 90                                                                                │
│   Heap Blocks: exact=80                                                                                │
│   ->  Bitmap Index Scan on index_audits_on_auditable_type_and_id  (cost=0.00..9.13 rows=114 width=0) (actual time=0.095..0.095 rows=115 l…│
│…oops=1)                                                                                │
│         Index Cond: ((auditable_type)::text = 'ApplicationForm'::text)                                                                    │
│ Planning Time: 0.738 ms                                                                                │
│ Execution Time: 2.039 ms                                                                                │
└───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
(9 rows)

After index:
bat_apply_development=# explain analyze SELECT "audits"."auditable_id"
FROM "audits"
WHERE "audits"."auditable_type" = 'ApplicationForm'
  AND (audited_changes ? 'work_history_completed')
  AND (audited_changes#>>'{work_history_completed, 1}' = 'true')
  AND (created_at BETWEEN '2021-3-31' AND '2021-4-30');
┌───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│                                                                QUERY PLAN                                                                 │
├───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Bitmap Heap Scan on audits  (cost=25.67..29.70 rows=1 width=4) (actual time=0.504..1.645 rows=25 loops=1)                                 │
│   Recheck Cond: (((auditable_type)::text = 'ApplicationForm'::text) AND (audited_changes ? 'work_history_completed'::text))               │
│   Filter: ((created_at >= '2021-03-31 00:00:00'::timestamp without time zone) AND (created_at <= '2021-04-30 00:00:00'::timestamp without…│
│… time zone) AND ((audited_changes #>> '{work_history_completed,1}'::text[]) = 'true'::text))                                              │
│   Rows Removed by Filter: 53                                                                                │
│   Heap Blocks: exact=70                                                                                │
│   ->  BitmapAnd  (cost=25.67..25.67 rows=1 width=0) (actual time=0.095..0.098 rows=0 loops=1)                                             │
│         ->  Bitmap Index Scan on index_audits_on_auditable_type_and_id  (cost=0.00..9.14 rows=115 width=0) (actual time=0.050..0.050 rows…│
│…=115 loops=1)                                                                                │
│               Index Cond: ((auditable_type)::text = 'ApplicationForm'::text)                                                              │
│         ->  Bitmap Index Scan on index_audits_on_audited_changes  (cost=0.00..16.28 rows=37 width=0) (actual time=0.039..0.040 rows=78 lo…│
│…ops=1)                                                                                │
│               Index Cond: (audited_changes ? 'work_history_completed'::text)                                                              │
│ Planning Time: 2.284 ms                                                                                │
│ Execution Time: 1.683 ms                                                                                │
└───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
(12 rows)

The amount of data in the test here isn't anywhere near enough to measure performance but we can see that the index does get utilised by the query - so it should be faster.
Guidance to review

Does the approach make sense?
How expensive is the index going to be to maintain? Especially wrt to the impact on write performance.
I'm also concerned about the time it will take to create the index. (how long the migration will take).
How can we test this before deployment to production?
Note that there is a code change and migration in the same PR here. I think that's reasonable here because the migration only adds an index - so the code change doesn't depend on it to run, though the (theoretical) performance gain does depend on the migration and code change being in place.

Link to Trello card
https://trello.com/c/70B0qwOa/3395-investigate-query-timeouts-when-updating-feature-metrics
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[x] New environment variables have been added to the Azure config

How expensive is the index going to be to maintain? Especially wrt to the impact on write performance.
I think this might be an issue due to how different each audit change is between objects, and different models/field, but this is only theory not sure in practice

I'm also concerned about the time it will take to create the index. (how long the migration will take).

How can we test this before deployment to production?
Vignesh created a db dump for us from prod last time, which we could test against

Although this seems ok, my concern still remains around the change we saw between Azure/PaaS where now we even have a significantly smaller table size but we are not utilising any optimised queries through the planner. I ran some index queries to check if we are hitting indexes as we should, and it doesn't seem we are (for audits). I wonder if we are missing something, a setting/autovacuum/reindex or just the query planner itself not optimised between the migration that we could help the issue. Happy to discuss suggestions

I've done some testing on this with a more realistic data set (local database that had generate_test_data for a few hours leading to about 600K audit records).
The results for the index in this PR are not great. It appeared to make a significant and consistent different when tested on a small dataset reducing query times from ~90ms to ~55ms. However with the larger dataset the timings were around 500-600ms with and without the index.
There are a lot of different kinds of index we can create on jsonb column. I tried a couple of others, this one makes the most difference reducing query times by an order of magnitude:
CREATE INDEX hash_test ON audits USING HASH ((audited_changes->'work_history_completed'));

This only works with the -> and = operators. So the query changes to:
SELECT count("audits"."auditable_id")
FROM "audits"
WHERE (audited_changes->'work_history_completed' = '[false, true]')

leading to this query plan:
┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│                                                           QUERY PLAN                                                           │
├────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Aggregate  (cost=10549.81..10549.82 rows=1 width=8) (actual time=33.224..33.226 rows=1 loops=1)                                │
│   ->  Bitmap Heap Scan on audits  (cost=81.27..10541.66 rows=3260 width=4) (actual time=9.320..31.731 rows=17215 loops=1)      │
│         Recheck Cond: ((audited_changes -> 'work_history_completed'::text) = '[false, true]'::jsonb)                           │
│         Heap Blocks: exact=16223                                                                                │
│         ->  Bitmap Index Scan on hash_test  (cost=0.00..80.45 rows=3260 width=0) (actual time=6.736..6.736 rows=17215 loops=1) │
│               Index Cond: ((audited_changes -> 'work_history_completed'::text) = '[false, true]'::jsonb)                       │
│ Planning Time: 0.121 ms                                                                                │
│ Execution Time: 33.270 ms                                                                                │
└────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
(8 rows)

Time: 33.668 ms

Parking this one for now following meeting because we are going to try some database config changes first.


Support node version 6.x, as it still in Maintenance LTS duration

What problem does this feature solve?
Hi, i don't know whether this request is in your roadmap. I think this is necessary, because there are still user under node 6.x. You can use babel to compile another cli file for lower version user.
What does the proposed API look like?
See what parcel do link

What prevents you from upgrading Node to 8.x?

Honestly, this is probably not going to happen, unless there are a substantial group of users who have strong reasons on why they cannot upgrade Node to 8.x, which is now the active LTS.

@yyx990803 Thank for your replay.
We have a packaged server for building static files used by multiple departments, there is no independent environment or container to do this thing, so we can not actively upgrade the globally node version. I tried something like using repo nave to build a virtual environment. But normal role has no authority to change global variable.
At last, if this is not in your roadmap, it's fine. Thank for your work!


Trouble with Model.publishCreate({},req)  and not working in sails  v0.11.4

Hi
I'm Just experiencing a problem with the sails Model.publishCreate()  method . I just did all the thing that is mentioned in sails documentation but didn't get any success with Model.publishCreate().
Is there something missing in sails documentation or Is something I'm missing in my following code.
=================
Controller code (TestController.js)
module.exports = {
create: function(req, res){
if(req.isSocket){
Test
.create(req.body)
.exec(function(err, created){
if(err) return res.sreverError(err);
if(created){
console.log("Created Record", created);
Test.publishCreate(created, req) ;
return res.ok("Record Created successfully");
}
});
}
}
};
============================
Model Code (Test.js)
module.exports = {
attributes: {
//id: {
//  type : 'string',
//  unique: true
//},
name : {
type : 'string'
},
className : {
type : 'string'
},
rollNo : {
type : 'string'
}
},
afterCreate: function(test, next){
next();
}
};
================================
Client Side code
$(document).ready(function(){
$('#saveRecord').click(function(){
  var data = {
    name : $('#name').val(),
    className: $('#className').val(),
    rollNo: $('#rollNo').val(),
  };

  io.socket.post('/create', data, function(event){
      console.log('post event', event);
      $('#name').val('');
      $('#className').val('');
      $('#rollNo').val('');
  });
});

});
io.socket.get('/test', function(resData) {
console.log(resData);
});
io.socket.on('connect', function(){
console.log("Socket Connected successfully!!!!");
});
io.socket.on('test', function (event){
console.log("server Event", event);
});
Is there any thing I'm missing in the above code ?
Thanks

@mssramthakur Nothing immediately jumps out, but I'll take a closer look. btw, would you mind editing your issue to make it a bit easier to read the code snippets?  (just need to wrap them in triple backticks)  Thanks!

@mssramthakur Your front-end code to /create should be /test/create if you want to use blueprint routes. If you wanted to use /create add a custom route in /config/routes.js :
module.exports.routes = {
  '/create': 'TestController.create';
};

@irlnathan thanks!

@irlnathan Thanks for your suggestion , I didn't want to use blueprint routes , I just want to make io.socket.on() listner working .


Level 100 mod extends health bar off screen of party pokemon

:)

This issue was also present during battles. It was a result of HPNormal not being updated when Pokemon statistics were reevaluated, only HP was. The width of the health bar was calculated as a ratio of HP to HPNormal, but since HPNormal still thought the Pokemon was a lower level the width was out of bounds.


Navigation.RemovePage on page with SVG causes error

If you use Navigation.RemovePage on a page that is using an SVG it throws the error Object reference not set to an instance of an object. Anyone have any ideas how to fix this or if it is a package issue?

I think that my pull request solves this too. https://github.com/paulpatarinski/Xamarin.Forms.Plugins/pull/34

@JoshuaNovak919 this should be resolved in >= <IP_ADDRESS>. @erikarenhill thanks for the PR. I have implemented the custom renderer fix. I will close the PR since there is no need for the Forms upgrade yet

Thanks @paulpatarinski , the form upgrades were pushed to the wrong branch, sorry. Intended to only have the fix in the pull request.


Init Readme

Set up the readme outlining the project.

Readme, docs, and placeholder unit tests are now initialized.


Bug due to changes in Entur component

Due to changes in the Entur component the following lines of code should be changed
const time = moment(state.attributes['due_at']).format('HH:mm');
const human = moment(state.attributes['due_at']).fromNow();
...
const next_time = moment(state.attributes['next_due_at']).format('HH:mm');
const next_human = moment(state.attributes['next_due_at']).fromNow();

To
const time = state.attributes['due_at'];
const human = moment(state.attributes['due_at'], "HH:mm").fromNow();
...
const next_time = state.attributes['next_due_at'];
const next_human = moment(state.attributes['next_due_at'], "HH:mm").fromNow();

Thanks for looking into it. I just pushed these changes to develop branch. Could you please test and verify?


Warning: Received false for a non-boolean attribute className

Keep getting the following warning in the console log when using GraphiQL:
index.js:170 Warning: Received `false` for a non-boolean attribute `className`.

If you want to write it to the DOM, pass a string instead: className="false" or className={value.toString()}.

If you used to conditionally omit it with className={condition && value}, pass className={condition ? value : undefined} instead.
    in p (created by HistoryQuery)
    in HistoryQuery (created by QueryHistory)
    in div (created by QueryHistory)
    in div (created by QueryHistory)
    in QueryHistory (created by GraphiQL)
    in div (created by GraphiQL)
    in div (created by GraphiQL)
    in GraphiQL
warningWithoutStack @ index.js:170

This is my application entry point:
import '@babel/polyfill';
import 'raf/polyfill';

import React from 'react';
import ReactDOM from 'react-dom';
import GraphiQL from 'graphiql';
import axios from 'axios';
import 'graphiql/graphiql.css';
import '../Content/graphiql.css';

function graphQLFetcher(graphQLParams) {
  return axios({
    method: 'POST',
    url: './api/graphql',
    data: graphQLParams
  }).then(resp => resp.data);
}

ReactDOM.render(<GraphiQL fetcher={graphQLFetcher} />, document.getElementById('app'));

The page renders ok and GraphiQL seems to work as normal but the warning is always there. The page itself contains nothing except the 'app' div hosting React and GraphiQL.

I suspect it's coming from this line:         className={this.state.editable && 'editable'} in HistoryQuery.js.


Enviroment configs

Whats the best (or at least reliable) way to have multiple enviroments in application?
most node packages solving this problems relaying on process.env which is not accesible :(
at very least we need a way to pass variables from console, right?

You could use AsyncStorage and separate your keys based on your different environments.

?dev=true/false in the packager's URL sets DEV to true/false in the JS.

DEV is good to determine development and production environment, but what if you have multiple production environments (stage, test, a/b testing etc)?
For now we stick with exporting environment variables from xcode via native module constants export. (probably will open source it soon).
But I do think react-native will be better with kind of rails config solution (yml/json config files for environments, selection via command-line arguments). Every app will need this as soon as it's gets out of sandbox so this could be considered "core feature".

As a suggestion Environment Variable Configuration should be in the docs, didn't know ?dev=true/false in the packager sets DEV to true/false. Thanks!

+1 for the docs - didn't know that little trick either. Thx

@romanonthego We are currently trying to deal with that issue, did you ever find a more practical way to pass environment variables?


appendBitmapText error handling

adds some error handling for the appendBitmapText block

adds the promise as an argument appendCommands and rejects it if an error is caught
main reason for adding this was sc-14192

Unsure of exact reason for error at the moment, this will at least help avoid a crash

On my end, I was able to print without the warning, so I guess it's okay
btw, for a little help testing, this could help to avoid building the package locally, so just need to add this line to the package.json
"@popmenu/star-print": "https://github.com/popmenu/react-native-star-prnt.git#matthewparavati/nullPointerException-try-catch",

Yeah, should've added that. And apparently all you need is "@popmenu/star-print": "popmenu/react-native-star-prnt#[branch name]" and it will pull from the correct github branch

Approving for now, but still need to test

Tested, this is good to go


Updated Readme

Description

The changes made in this pull request should be briefly described.
Added a header for GSSOC and added a table of contents

Resolves: [#62 ]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[ ] Have you adhered to the repository's defined coding convention rules?
[ ] Have you updated the 'documentation.md' file with the method/function documentation?
[ ] Have you sent a message along with the result or response?
[ ] Have you used the try-catch technique?
[ ] Has the method/class been added to the documentation (md file)?

Screenshots

Uploading a screenshot illustrating the approach you developed for validation is required.

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

@CS251000 Why this color change?

Please fix

@hereisSwapnil these color changes were not done by me it was like this when i started working
I only added the heder of gssoc and the table of contents

@CS251000 Please fix this also

@hereisSwapnil what do i fix in this? I dont know how it looked earlier??


Creates DnsService interface and DnsOptions

Comments implemented. Ready for another pass.

Included forgotten files. It should be good now.

All ready for another pass.

LGTM, please remove the static Field static import in Dns.java before you merge.