Automated cherry pick of #1573: add usable filter for network

Cherry pick of #1573 on release/2.11.0.
#1573: add usable filter for network

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@4c9ea9d). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.11.0   #1574   +/-   ##
================================================
  Coverage                  ?   7.66%
================================================
  Files                     ?     411
  Lines                     ?   62949
  Branches                  ?       0
================================================
  Hits                      ?    4828
  Misses                    ?   57711
  Partials                  ?     410

Flag
Coverage Δ

#aFlag
7.66% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/networks.go
0.93% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm
/approve


Getting Type error while running the Implement DRL Algorithms

Describe the bug
when i try to run  below script  i am getting type error  could you please advise ?

Error Message

Desktop (please complete the following information):

OS: [e.g. iOS] MAC OS

did you run FinRL_Ensemble_StockTrading_ICAIF_2020.ipynb in colab?
have you revised anything?

i have the same error.
I am runnning the ipynb in anaconda environment on Windows 10 everything is installed without errors (after 1 day).

did you run FinRL_Ensemble_StockTrading_ICAIF_2020.ipynb in colab? have you revised anything?

I am running in Anaconda .. colab  is getting module issues  they are not getting downloaded.
i have tried Anaconda and my local VSS where i downloaded all the packages all stages working fine only this stage getting the Issue.

pls run it by clicking “open the colab"


🛠 Tooling: Action that removes 'status: waiting for author' label when review is (re-)requested and adds when review submitted

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
This PR comes with a suggested list of labels, stored in script/labels.json. That list includes a status: waiting for author label indicating an issue or PR needs an action taken by the original poster. It's commonly used (by me) when a PR has had changes requested.
However, PR authors who aren't maintainers on this repo typically don't have permissions to remove labels. That means when they re-requests review from me on their PR they won't be able to remove the label... even though it's now out of date - the PR is waiting for a maintainer to review!
Let's add some kind of automation that when someone requests or re-requests review on a PR, the status: waiting for author label is removed automatically!
Additional Info
As with other GitHub Actions automations in this repo, the source code for the action's logic shouldn't live in this repo. It should be in some shared action referenced in this repo's configuration. That way as folks copy this template, they don't need to copy that logic.
I suspect this can work with a combination of https://github.com/marketplace/actions/actions-ecosystem-remove-labels & https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request > the review_requested activity type. But I haven't tried it out and might be wrong. 😄

While we're on the subject, let's use https://github.com/actions-ecosystem/action-add-labels to add the status: waiting for author label when a PR review is submitted.

I think what I'm trying to do is just not possible with public OSS forks right now :upside_down_face:. Same issue as this: https://github.com/actions/labeler/pull/51/files

Note that only pull requests being opened from the same repository can be labeled.  This action will not currently work for pull requests from forks -- like is common in open source projects -- because the token for forked pull request workflows does not have write permissions.

Seems like this article's note is still in effect (I haven't found an option under the repo's Settings to override it) https://github.blog/changelog/2021-04-20-github-actions-control-permissions-for-github_token/#:~:text=GitHub Actions now lets you,token when a job completes.:

Pull requests from public forks are still considered a special case and will receive a read token regardless of these settings.

It's entirely possible I'm wrong / out of date. Someone please correct me if that's the case!

Status update: per https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target:

pull_request_target can run with types: including - review_requested. I've been able to see a PR on a fork having its status: waiting for author label removed
No such equivalent exists that I can find for pull_request_review. Even though the guide says pull_request_review is one of the types that runs under PR events for forked repos, I still get HttpError: Resource not accessible by integration. 😕

At any rate, the review_requested event is more important IMO. Maintainers can be expected to know to add the right label. It's contributors -the ones requesting reviews- who needed assistance with labels.
I'll retarget this issue to just review_requested, mark it as resolved by #632, and file a new one for pull_request_review - submitted.


[charts/redis-ha][BUG] Exporter not deployed

Describe the bug
When "exporter.enabled" is set to "true" it is not deployed.
To Reproduce
Steps to reproduce the behavior:

Enable the exporter in values.yaml

exporter:
  enabled: true
  image: oliver006/redis_exporter
  tag: v1.3.5-arm64

Install the chart

helm install --namespace database redis-ha dandydev/redis-ha -f database/redis-ha-helm-values.yaml

Check the pods

kubectl get -n database pods
kubectl get -A pods | grep redis
kubectl get -A pods | grep exporter

Expected behavior
The exporter gets deployed.
Additional context
I'm running this on K3S on a 4 node Raspberry Pi 4 cluster on Raspbian Buster running the aarch64 kernel.

@slaecker I'll take a look at this, it's a weird one. Because it's working locally for me.
Can you try templating helm template . and share the output (format as best you can please!)

@slaecker
      - name: redis-exporter
        image: "oliver006/redis_exporter:v1.3.2-arm64"
        imagePullPolicy: IfNotPresent
        args:
        env:
          - name: REDIS_ADDR
            value: redis://localhost:6379
        livenessProbe:
          httpGet:
            path: /metrics
            port: 9121
          initialDelaySeconds: 15
          timeoutSeconds: 1
          periodSeconds: 15
        resources:
          {}
        ports:
          - name: exporter-port
            containerPort: 9121
      volumes:
      - name: config
        configMap:
          name: RELEASE-NAME-redis-ha-configmap

It looks like it's deployed. Are you certain the sidecar container isn't created?

@DandyDeveloper I deleted and re-installed the chart now I see the sidecar containers, Prometheus is picking them up and Grafana graphs are filled. Don't know what was wrong before but it was certainly a local issue. Sorry for the confusion and thanks for providing and supporting the chart.


Issue with widgets displaying for Orchard 1.10

The problem is that widget position does not affect their actual displaying in Orchard version 1.10
The test case for this issue is to create two widgets in any zone, then change their order. Widgets are rendered in the order of creation. Attached screenshots represent the issue

Issue was not related to migration to new Orchard version. The cause for it was redefined Zone shape (Zone.cshtml)


feature: query shard on multi table and multi db

support query request like:
select id, drug_res_type_id, base_type from drug_resource where id between 200 and 210;

select id, drug_res_type_id, base_type from drug_resource where id between 200 and 210 order by id desc ;

select id, drug_res_type_id, base_type from drug_resource where id between 200 and 210 order by id desc limit 10, 30;

Codecov Report

Merging #40 (ef718eb) into dev (22d57b1) will decrease coverage by 1.21%.
The diff coverage is 2.71%.

@@            Coverage Diff             @@
##              dev      #40      +/-   ##
==========================================
- Coverage   35.47%   34.26%   -1.22%
==========================================
  Files          40       42       +2
  Lines        4769     4941     +172
==========================================
+ Hits         1692     1693       +1
- Misses       2908     3079     +171
  Partials      169      169

Impacted Files
Coverage Δ

pkg/executor/executor.go
0.00% <0.00%> (ø)

pkg/executor/read_write_splitting.go
0.00% <0.00%> (ø)

pkg/executor/sharding.go
0.00% <0.00%> (ø)

pkg/misc/utils.go
0.78% <0.00%> (ø)

pkg/executor/data_source.go
64.10% <83.33%> (+0.94%)
:arrow_up:

Continue to review full report at Codecov.

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


Minor PR template improvements

Context
The template could do with some small improvements, using https://raw.githubusercontent.com/DFE-Digital/apply-for-teacher-training/main/.github/PULL_REQUEST_TEMPLATE.md as a reference.
Changes proposed in this pull request

Format the comments in each section as actual comments so that they don't appear in the PR if left in
Reword some of the section guidance
Add a prompt to attach the Trello card

Guidance to review
Checklist

[ ] Attach to Trello card
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally

Already merged, but I would've probably preferred a multi line comment instead of the current format.
<!--
- Include a summary of the change.
- Why this particular solution?
- What assumptions have you made?
- Are there any side effects to note?
- If there are UI changes, please include Before and After screenshots.
-->


fix: Add missing telemetry settings

Description
Fix #7990 - telemetry settings were accidentally dropped during publishing

Coverage remained the same at 50.047% when pulling c022bd4261eb47332132c9fd96328c143d926024 on tachou/telemFix into 6b3fab5eac8f9092d154e941fdb77f62d1db3e40 on main.


add interrupt test

Looks like the upload is not resuming on appveyor.

I know it's just a t.Skip test, but it's ready for merge. We'll uncomment the test when we finish the contractor upgrade.


Be compatible with iOS 12 breaking changes

https://github.com/aspnet/Security/issues/1864

Done.


Added script to fetch live cricket scores

Added script for issue #37

thanks for yhe PR :+1:
You may add the idea to the List of Ideas and send another PR :+1:

@gauravkulkarni96 have you tested the pr personaly?


Keras TimeDistributed().  Same input shape produces inconsistent output shape between TF1.15.5 and TF2.11.0

I am updating a model from TF1.15.5 to TF2.11.0.
For TF 1.15.5, Keras == 2.2.4
for TF 2.11.0, Keras == 2.11.0
I am running Python 3.10.8
I am running the model on an AWS p3.2xlarge SageMaker Notebook
In TF == 1.15.5
import keras
### This code snippet prints the shape issue.  The inputs tensor is the output of the prior layer in a Keras function model ###
print("shape of input going into TimeDistributed=",inputs.shape)
conv_2d_layer = keras.layers.Conv2D(filters=1024, kernel_size=(7,7), data_format='channels_last')
outputs = keras.layers.TimeDistributed(conv_2d_layer)(inputs)
print("TimeDistributed output shape =",outputs.shape)
### end of code snippet ###

In TF == 2.11.0
import tensorflow as tf
### This code snippet prints the shape issue.  The inputs tensor is the output of the prior layer in a Keras function model ###
print("shape of input going into TimeDistributed =",inputs.shape)
conv_2d_layer = tf.keras.layers.Conv2D(filters=1024, kernel_size=(7, 7), data_format='channels_last')
outputs = tf.keras.layers.TimeDistributed(conv_2d_layer)(inputs)
print('TimeDistributed output shape =',outputs.shape)
### end of code snippet ###

The model I am trying to convert (MaskRCNN), has been utilized extensively by others in TF1.15.  Is this a bug of TF2.11.0?  If not could you explain in the documentation why these are different and/or how to achieve the correct output shape.  The correct output shape should be that shown for TF1.15.5.

I think I have found the issue, but do not know how to resolve yet:
The Class TimeDistributed(Wrapper) in Keras 2.11.0 performs the the 'if input_shape[0]:' of the def call(), the Class TimeDistributed(Wrapper) in Keras 2.2.4 performs the 'else:' of the def call().
The reason for this, is that the backend function keras.backend.int_shape(input) has changed.  In Keras 2.2.4 it is looking for an attribute of the input called ._keras_shape.  This attribute exists in Keras 2.2.4 and the function returns input._keras_shape which = (None, 200, 7, 7, 256), triggering the "else" inside the call of TimeDistributed().  This attribute does not exist in Keras 2.11.0 and keras.backend.int_shape(input) returns input.shape which = (2, None, 7, 7, 256), triggering the 'if' of TimeDistributed().

Root Issue Identified:
The root issue is caused by a change in the keras backend function int_shape(input).  In Keras 2.2.4, if the input to int_shape(input) has an Attribute called "_keras_shape", the return is input._keras_shape, else the return is input.get_shape().as_list().   These two return shapes do not match in Keras 2.2.4 for this model.    In Keras 2.2.4, the call() of TimeDistributed(Wrapper) has two optional pathways to create the output shape and the selection of that pathway depends on the return of int_shape(input).  The "if" pathway uses an rnn-based implementation which produces (2, None, 1,1,1024), the "else" pathway uses a reshape implementation which produces (None, 200, 1, 1, 1024) for the MaskRCNN model.
In Keras 2.11.0 the backend function int_shape(input), no longer looks for the attribute "_keras_shape", I also ran a check to see if the input to int_shape(input) even has the attribute "_keras_shape" -  It did not.  So I suspect the same rnn pathway is being run, instead of the the reshape implementation in Keras 2.11.0 TimeDistributed(Wrapper).

@ctmckee Thank you for the update!
Yes you were right. The back-end function int_shape(input), no longer looks for the attribute "_keras_shape".
For an example please check the following code snippet and refer to this guide as well. This is a guide for migration from TF v1.x to 2.x . Please let us know if it helps?
inputs = tf.keras.Input(shape=(10, 128, 128, 3))
conv_2d_layer = tf.keras.layers.Conv2D(64, (3, 3))
outputs = tf.keras.layers.TimeDistributed(conv_2d_layer)(inputs)
outputs.shape

Thank you!


AttributeError: 'list' object has no attribute 'keys' when attemping to run the sample with Python 3.7.0 and all dependencies listed in the readme with exact versions.

C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo>python -m robot testPerfectoDevices-Demo.robot
testPerfectoDevices-Demo
[ WARN ] Keyword 'Capture Page Screenshot' could not be run on failure: No application is open
Demo test on iphone                                                   | FAIL |
AttributeError: 'list' object has no attribute 'keys'
testPerfectoDevices-Demo                                              | FAIL |
1 test, 0 passed, 1 failed
Output:  C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\output.xml
Log:     C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\log.html
Report:  C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\report.html

Hi Chris,
Can you please send me the following three log files of your failed TC so I
can investigate?
Output: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo*output.xml*
Log: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo**log.html
Report: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo*report.html*
Thanks.
Jack
On Fri, Nov 26, 2021 at 9:36 AM Chris Hamid @.***>
wrote:

C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo>python -m robot
testPerfectoDevices-Demo.robot testPerfectoDevices-Demo [ WARN ] Keyword
'Capture Page Screenshot' could not be run on failure: No application is
open
Demo test on iphone | FAIL |
AttributeError: 'list' object has no attribute 'keys' testPerfectoDevices-Demo
| FAIL |
1 test, 0 passed, 1 failed
Output: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\output.xml
Log: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\log.html
Report: C:\Users\CHamid\Downloads\Robotframework_Perfecto_demo\report.html
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PerfectoMobileSA/Robotframework_Perfecto_demo/issues/3,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZTZ4EITGXYVYA2VTC5HDLUN6LPNANCNFSM5I2ZSYSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Notify properties

Hi, could you add the notify flag to  value selected selectedItem selectedItems and selectedValues properties so we can use data binding ?

Merged, thanks alot!


EmailAuth cannot handle actions after successful login

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
https://github.com/supabase/auth-ui/blob/main/packages/react/src/components/Auth/interfaces/EmailAuth.tsx#L78-L83
To Reproduce

login with email and password
The login was successful, and the user information was returned, but the action could not continue

Expected behavior
Can handle the action after login
Screenshots

System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Add any other context about the problem here.

Hi @jokimina, Are you using a framework with auth-ui?

I can confirm this. I am using next.js as framework.
When using email/pw login no redirect  to the given redirectTo happens after clicking the Sign In Button. The Login is successful btw, after reloading the page i am logged in.

any workarround?


KAFKA-16026: Send Poll event to the background thread

related to KAFKA-15818
This is a bug in the AsyncKafkaConsumer poll loop that it does not send an event to the network thread to acknowledge user poll. This causes a few issues:

Autocommit won't work without user setting the timer
the member will just leave the group after rebalance timeout and never able to rejoin.

In this PR, a few subtle changes are made to address this issue

Hook up poll event to the AsyncKafkaConsumer#poll. It is only fired once per invocation
Upon entering staled state, we need to reset HeartbeatState otherwise we will get an invalid request
We will clear and current assignment and remove all assigned partitions once the heartbeat is sent.  See changes in onHeartbeatRequestSent

TODO:

We will need to implement the callback invocation logic.  We didn't do it for this PR is because we want to limit to scope of the changes
I think FencedException and UnknownMemberIDException: we will also need to reset the HeartbeatState. See KAFKA-16034

Test testEnsurePollEventSentOnConsumerPoll() fails deterministically in all CI builds. That means that something is wrong either with the test you added or with the production code that is tested. Please ensure that at least the tests you added or modified or are affected by your change are successful by running them locally. Otherwise, we review code that obviously needs changes and we end up cycling over the PR more than needed.

hi @cadonna - Thanks for putting time into the review.  I've addressed your comments.

@cadonna @dajac @AndrewJSchofield  - Thanks a lot for the reviews.  I've made changes accordingly.

hmm @cadonna - seems like there are some build stability issue.  my other PR also failed after 8hr (it is completely unrelated to the consumer refactor) PR-15023

Don't worry! We will take care of restarting the builds.

@dajac - thanks for merging the changes.


[DOC]: Deploy Random Forest Model on the Triton Server

How would you describe the priority of this documentation request
High
Describe the future/missing documentation
One team wanted to take simple use Random Forest and to use it in Morpheus (was very complicates, for XGBoost we have convertor to bst but not for Random Forest)
Where have you looked?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open documentation issues and have found no duplicates for this bug report

Now Random Forest Classifier model can be converted to treelite model and deploy to Triton Server.


Build fails on Xcode 10

I was just trying to get my app building with Xcode 10 and the AudioKit (4.3) build is failing with Carthage.
I'm running Xcode 10 Beta 5.
If there's anything else I can try or more information you need, let me know.
*** xcodebuild output can be found in /var/folders/3g/yfpp2r4907jggy7zvnln7k280000gn/T/carthage-xcodebuild.xr2BqI.log
*** Building scheme "AudioKitUI" in AudioKit For iOS.xcodeproj
Build Failed
	Task failed with exit code 65:
	/usr/bin/xcrun xcodebuild -project /Users/alex/Developer/lifx-ios/Carthage/Checkouts/AudioKit/AudioKit/iOS/AudioKit\ For\ iOS.xcodeproj -scheme AudioKitUI -configuration Release -derivedDataPath /Users/alex/Library/Caches/org.carthage.CarthageKit/DerivedData/10.0_10L221o/AudioKit/3919bf3bc149fb56b82c6e4a1002a940ba763b3f -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/3g/yfpp2r4907jggy7zvnln7k280000gn/T/AudioKit SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Users/alex/Developer/lifx-ios/Carthage/Checkouts/AudioKit)

This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/3g/yfpp2r4907jggy7zvnln7k280000gn/T/carthage-xcodebuild.xr2BqI.log

carthage-xcodebuild.xr2BqI.log

We don't support beta Xcode or Carthage so I'm not sure where to begin to help you out. :)

When are you planning on supporting Xcode 10? I understand it's still in beta, but we're on beta 5 at this point, it's not too far of it being released. And regarding Carthage, I've been using it with Carthage for over a year without problem. CocoaPods doesn't build either.

The day it comes out we usually scramble to make a release.

Happy to hear so... but I need to test in advance under  iOS12 devices...so.. I have to use (in may case..) Xcode Version 10.0 beta 2 with cmd lien tools (10L177m)
Only a couple of notes:

I downloaded Audiokit 4.3 from "https://github.com/audiokit/AudioKit/releases" :
I tried to add new frameworks to my Xcode 10 project but I got: Module compiled with Swift 4.1 cannot be imported in Swift 4.1.50.
I downloaded "AudioKit-iOS-4.3" from the same page, of course your samples do to compile, same error.
I downloaded  "Source code (zip)" (same page). examples do not link as frameworks are missing, so I built them from scratch after reading README.
I run form terminal:

a)
PLATFORMS="iOS" ./build_frameworks.sh
b) at run I got:
warning: file type '::com.apple.instruments.instrdst' is based on missing file type 'default::com.apple.package
c) after some time I got:
** BUILD FAILED **
The following build commands failed:
CompileSwift normal x86_64
CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
d) I opened Xcode project "AudioKit For iOS.xcodeproj" and tried to build both "AudioKit" and "AudioKitUI"  from here:
(I got a LOT of warning "...'let' properties are implicitly 'final'; use 'public' instead of 'open'")
I can get 4 frameworks inside "Products", 2 for device and 2 for simulator.
I did get frameworks but once imported in "HelloWorld" project I get:
".....Frameworks/AudioKitUI.framework/Modules/module.modulemap:9:12: Header 'AudioKitUI-Swift.h' not found"

Hey guys, I just got a Mojave/iOS12 test machine so I will be working on this.  I made a few commits last night to develop that allowed to me to compile on Xcode 10.  I didn't get many errors, but maybe I was compiling for iOS11, I'll check later today.

happy to hear so.
Anyway I succeded in creating  an iOS  project from scratch for ios12 using Xcode 10.
I added "AudioKit For iOS.xcodeproj" as  a subproject.
I got a LOT of warnings (mainly Static declarations are implicitly 'final'; use 'public' instead of 'open'), but works.

Weird, on Xcode10. I am only getting Umbrella header issues showing up, nothing about final/public. I'm using Xcode 10 beta 5 how about you?

On 12 Aug 2018, at 21:51, Aurelius Prochazka<EMAIL_ADDRESS>wrote:
Weird, on Xcode10. I am only getting Umbrella header issues showing up, nothing about final/public. I'm using Xcode 10 beta 5 how about you?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/AudioKit/AudioKit/issues/1460#issuecomment-412367367, or mute the thread https://github.com/notifications/unsubscribe-auth/AD3GJ5AF5OCNezEYYcyE8cRtwpcP_kGWks5uQIc9gaJpZM4VzSEr.

ing. Conti G. Enrico
<EMAIL_ADDRESS>Ordine Ingegneri di Monza al n. A894
PIVA<PHONE_NUMBER>6
via Andrea Doria, 9/A
20090 MONZA MI ITALY
0039-039-324723
www.ingconti.com

So are you saying you're not seeing any compilation issues with Xcode 10? Are you using latest master or develop branch?

I only really work on the develop branch, and yes that Is what I am saying.

Great, I'll try it out, thank you.

BTW, I'm also on Mojave, so let me know if you are on High Sierra and still having issues with Xcode10.


Does not compile in OS X 10.5

Hey,
Enjoy2 is not working in OS x 10.5. If I compile the build, it simply crashes. Any idea guys?

What are the errors you are getting in the log and Xcode

Hey @sdrib I opened XCode, tried to compile, got a bunch of errors (35, minors), Enjoy2 got launched but there wasn't any change with my right anlaog stick problem, I'm trying to put a dead zone at the beginning of the right joystick cause it moves even when it is set...


AeternoBlade [PCSE00628]

App summary

App name: AeternoBlade
App serial: PCSE00628
App version: 1.00

Vita3K summary

Version: v0.1.8
Build number: 3158
Commit hash: https://github.com/vita3k/vita3k/commit/60a68325
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Sam
OS: Windows 10
CPU: AMD 5500u
GPU: AMD Radeon Graphics
RAM: 6 GB

Issues
Have music and joystick have response, but screen just yellow colour only .
Screenshots

Log

Recommended labels

Intro
B?
C?

Vita3K summary

Version: v0.1.8
Build number: 3209
Commit hash: https://github.com/vita3k/vita3k/commit/22ea4f03
CPU backend: Dynarmic
GPU backend: Vulkan

App summary

App name: AeternoBlade
App serial: PCSE00628
App version: 1.00

Test environment summary
Tested by: Sam
OS: Windows 10
CPU: AMD 5500u
GPU: AMD Radeon Graphics
RAM: 6 GB
Issues
Yellow screen when start the game, press start button and press O button
many time can enter the game,
Screenshots

Log
vita3k.log
Recommended labels
https://github.com/Vita3K/compatibility/labels/Ingame -
https://github.com/Vita3K/compatibility/labels/crash
https://github.com/Vita3K/compatibility/labels/screen flashing

@Zangetsu38 Goes in-game now, change label on issue

Vita3K summary
Vita3K Build: 3502
Version: v0.1.9
Commit hash: https://github.com/Vita3K/Vita3K/commit/4ac8a416
CPU backend: Dynarmic
Test environment summary
Tested by: UsualVII
OS: Windows 11
CPU: Intel i5-11600k
GPU: RTX 3060 Ti 8GB
RAM: 32GB @ 3600MHz
API & INFO

Tested both with OGL & Vulkan
The Audio is working

Issue:

Doesn't reach Menu

Recommended labels

Nothing

#Logs
vita3k.log
Screenshot


Corepack and Yarn isolation in an nvm-managed environment

My system has only two versions of Node.js installed and managed by nvm: the default is 20, which I use for almost all of my projects; the other one is 16. There is no other version of Node.js on this system, regardless of how it is installed. Both only have Corepack and NPM installed globally, each at the latest supported version.
The issue I come to discuss is that Corepack does not activate Yarn in isolation, i.e., every time a certain version of Yarn is activated in one of the Node.js versions, it is also activated in the other version. Therefore, I cannot, for example, use the latest version of Yarn on Node.js 20, as it is not compatible with 16.
Is there a way to isolate Yarn or this is the way Corepack works?
I use nvm with Z shell in a macOS v14.5.

That seems like the way corepack works - if it's not designed to do per-shell tool selection, then it won't work well with nvm's per-shell design.
You may want to file a corepack issue about it - it's still experimental.

Okay, I'll do that. Thanks!


Docs: order of lifespan ctx exit and on_shutdown hooks

Summary
From a quick glance at the docs, it doesn't appear that we explicitly document the order that __aexit__ methods on the lifespan context managers are called, relative to the registered on_shutdown hooks. This ordering might be important, especially that it prevents an on_shutdown hook from interacting with a resource that is managed by the lifespan context, e.g., a db engine, or redis connection or similar.
Take this app, for ex:
app = Litestar(lifespan=[ctx_a, ctx_b], on_shutdown=[hook_a, hook_b])

On shutdown, the order of calls is:

ctx_b.__aexit__()
ctx_a.__aexit__()
hook_a
hook_b

So, on shutdown, context managers are exited in reverse order (which is natural behavior of the exit stack), and before shutdown hooks. Then, shutdown hooks are called in registered order.
We should also add tests to lock this behavior in, as FWICT (checked here: https://github.com/litestar-org/litestar/blob/main/tests/unit/test_asgi_router.py), we have tests that they are all called on shutdown, but nothing that tests for explicit order.

Funding

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified

I assume the docs would go here?
I can add a paragraph to the Startup/Shutdown section and then give the tests a try.

That would be great @orgarten, cheers!

I assume the docs would go here?

Yep, around that area. You could add an admonition to each of the sections about lifespan context, and shutdown hooks that demonstrates their place in the sequence, or add a new section following that talks about the sequencing.. or something else that you think is better!
We probably should include the calls to on_startup hooks and entering the lifespan contexts as well.
Shall I assign the issue to you?

Yes, go ahead an assign me @peterschutt. It's probably gonna take a couple of days though.
I will see what works best in my opinion. Once the content is there, the format should be a quick change anyway.

Ok, I added a section inside the documentation of the app regarding the order.
I would implement the tests insider tests/unit/test_app.py unless there are any objections with a suggestion of a better place :)


Update 2024-08-22-newcomers-meeting.md

Attended newcomers meeting
Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit d89c988383c2adc78271a3ca78eab06cbcb73330 at: https://66c7419624832e2a110d25d7--meshery-docs-preview.netlify.app


Allow canonicalize_address/humanize_address to return an error string

Closes #433.
The focus of this is to get the error communication between VM and contract right. This hopefully is the last ABI breaking change before 1.0.
It turns out that FfiError::Other is too general to pass to the contract and we probably need a more specific error type for that. In this test we see a case that should not be reported back to the contract but is expressed as FfiError::Other:
    #[test]
    fn errors_in_imports_are_unwrapped_from_wasmer_errors() {
        // set up an instance that will experience an error in an import
        let error_message = "Api failed intentionally";
        let mut instance = mock_instance_with_failing_api(&CONTRACT, &[], error_message);
        let init_result = call_init::<_, _, _, serde_json::Value>(
            &mut instance,
            &mock_env(&MockApi::new(MOCK_CONTRACT_ADDR.len()), "someone", &[]),
            b"{\"verifier\": \"some1\", \"beneficiary\": \"some2\"}",
        );

        // in this case we get a `VmError::FfiError` rather than a `VmError::RuntimeErr` because the conversion
        // from wasmer `RuntimeError` to `VmError` unwraps errors that happen in WASM imports.
        match init_result.unwrap_err() {
            VmError::FfiErr {
                source: FfiError::Other { error, .. },
            } if error == error_message => {}
            other => panic!("unexpected error: {:?}", other),
        }
    }

Ideas welcome, but improving the error source should be handles separately.

I extracted the remaining questions to #445


andrey #395 dynamic robots.txt

Obviously, you can not see how it will look like on a master branch until you merge

Deploy preview for landscape ready!
Built with commit 0921d1a7ddec665822a67538249ef9398afe04ee
https://deploy-preview-403--landscape.netlify.com


Crown hosting is no longer legacy

This will need to be merged at the same time as @superroz's PR here:

[ ] https://github.com/alphagov/digitalmarketplace-buyer-frontend/pull/474

👍  once the content change goes in. (is there anything holding it back?)

Merging this because @superroz's is going in.


Implement a perunnode application in cmd/perunnode

Description

Implement cmd/perunnode executable package

The package is implemented using spf13/cobra library.

Added run, version and help commands in the perunnode app.

The run command can take either

a config file or
a config file with a few flags to override the config file or
complete config in flags by specifying all flags
(if all flags are specified, configfile will be ignored.)

The version command prints version of perunnode, go-perun. Added a
Makefile to set the version using linker flags during build.
help command the message autogenerated by the library.

Added a ListenAndServe function is [api/grpc] to start the grpc
channel API server and use it in the executable package.

Category

Implementation Task
Relevant issue

Relates to #98. (2/3) PRs.
Testing

Steps to run the tests

To build test the building of binary and proper version is assigned, run make command from project root. The perunnode binary should be created in the root folder.
Run ./perunnode help to see the help output.
Run the ./perunnode version to the the version of software in this format.

<version:tag if it is present, else short form of commit id" Git revision: <commit-if of current version> (go-perun version: <go-perun version as in go.mod file>)

To test if the run command starts the node with a usable configuration:
a. Start the ganache-cli node using below command:

ganache-cli -b 1 --account="0x1fedd636dbc7e8d41a0622a2040b86fea8842cef9d4aa4c582aad00465b7acff,100000000000000000000" --account="0xb0309c60b4622d3071fad3e16c2ce4d0b1e7758316c187754f4dd0cfb44ceb33,100000000000000000000

b. In another terminal, from the root directory of the project, start the perunnode using below command:
./perunnode run --loglevel "debug" --logfile "" --chainurl "ws://<IP_ADDRESS>:8545" --adjudicator "0x9daEdAcb21dce86Af8604Ba1A1D7F9BFE55ddd63" --asset "0x5992089d61cE79B6CF90506F70DD42B8E42FB21d" --chainconntimeout "10s" --onchaintxtimeout "10s" --responsetimeout "10s"

c. Comment out line no 85 in api/grpc/server_integ_test.go containing this function call "StartServer()". Save the file and run the test from root folder of the project using below command:
go test -tags=integration -count=1 -cover ./api/grpc -v

The test should complete without any errors by connecting the running perunnode instead of starting its own. In the terminal where perunnode was started, the logs would be printed at debug level.
d. Stop the perunnode and ganache-cli by pressing Ctrl+C.
Checklist

[x] Name is added to the NOTICE file, if it is not present already.
[x] Changes are rebased onto the target branch.

This should be closed after #120.


Got exception when start up

I'm runing kubernetes1.2 on centos7
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] done.
[services.d] starting services
level=info msg="Starting go-dnsmasq server 0.9.8 ..."
level=info msg="Search domains in use: [default.svc.cluster.local. svc.cluster.local. cluster.local.]"
level=info msg="Ready for queries on tcp://<IP_ADDRESS>:53 - Nameservers: [<IP_ADDRESS>:53 <IP_ADDRESS>:53]"
level=info msg="Ready for queries on udp://<IP_ADDRESS>:53 - Nameservers: [<IP_ADDRESS>:53 <IP_ADDRESS>:53]"
[services.d] done.
[2016-03-28 09:24:55,025][INFO ][node                     ] [Saint Anna] version[2.2.1], pid[171], build[d045fc2/2016-03-09T09:38:54Z]
[2016-03-28 09:24:55,026][INFO ][node                     ] [Saint Anna] initializing ...
[2016-03-28 09:24:55,495][INFO ][plugins                  ] [Saint Anna] modules [lang-expression, lang-groovy], plugins [cloud-kubernetes], sites []
[2016-03-28 09:24:55,513][INFO ][env                      ] [Saint Anna] using [1] data paths, mounts [[/data (/dev/mapper/rhel-root)]], net usable_space [19.4gb], net total_space [40gb], spins? [possibly], types [xfs]
[2016-03-28 09:24:55,513][INFO ][env                      ] [Saint Anna] heap size [1007.3mb], compressed ordinary object pointers [true]
[2016-03-28 09:24:57,340][INFO ][node                     ] [Saint Anna] initialized
[2016-03-28 09:24:57,340][INFO ][node                     ] [Saint Anna] starting ...
Exception in thread "main" java.lang.IllegalArgumentException: No up-and-running site-local (private) addresses found, got [name:lo (lo), name:eth0 (eth0)]
at org.elasticsearch.common.network.NetworkUtils.getSiteLocalAddresses(NetworkUtils.java:186)
at org.elasticsearch.common.network.NetworkService.resolveInternal(NetworkService.java:233)
at org.elasticsearch.common.network.NetworkService.resolveInetAddresses(NetworkService.java:209)
at org.elasticsearch.common.network.NetworkService.resolveBindHostAddresses(NetworkService.java:122)
at org.elasticsearch.transport.netty.NettyTransport.bindServerBootstrap(NettyTransport.java:424)
at org.elasticsearch.transport.netty.NettyTransport.doStart(NettyTransport.java:321)
at org.elasticsearch.common.component.AbstractLifecycleComponent.start(AbstractLifecycleComponent.java:68)
at org.elasticsearch.transport.TransportService.doStart(TransportService.java:170)
at org.elasticsearch.common.component.AbstractLifecycleComponent.start(AbstractLifecycleComponent.java:68)
at org.elasticsearch.node.Node.start(Node.java:252)
at org.elasticsearch.bootstrap.Bootstrap.start(Bootstrap.java:221)
at org.elasticsearch.bootstrap.Bootstrap.init(Bootstrap.java:287)
at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:35)
Refer to the log for complete error details.
[2016-03-28 09:24:57,421][INFO ][node                     ] [Saint Anna] stopping ...
[2016-03-28 09:24:57,433][INFO ][node                     ] [Saint Anna] stopped
[2016-03-28 09:24:57,433][INFO ][node                     ] [Saint Anna] closing ...
[2016-03-28 09:24:57,441][INFO ][node                     ] [Saint Anna] closed
/run.sh exited 1
level=info msg="Application exit requested by signal: terminated"
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] syncing disks.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

No up-and-running site-local (private) addresses found, got [name:lo (lo), name:eth0 (eth0)]

You can try and change your own version of elasticsearch.yml and change the network.host flag.

Thanks!  I check the offical es image and use "network.host:<IP_ADDRESS>" this fix the problem

With pull request #6, network.host can be easily changed, without building a new image.


added a mode parameter to select ascii mode for a serial modbus slave

Thank you for your effort. I'll merge the commit in a different branch as I'm unable to lint it directly.

Released in 0.40.0.


How can I make it move vertically?

Hello,
How can I make the paging cards vertical instead of moving horizontal when swiping.
This is in the demo.

@JayBone16 you can't do that. Expanding collection designed only for horizontal scroll.


Parsing XML Example

Is it possible to just get a complete example of parsing XML for a specific key using the Facade?
Currently I'm able to convert the whole XML structure into an array using
$xml = Parser::xml($response);
however is there a way to specific pull keys? Using the documentation it would be something like
Parser::get('key', 'default value');
but I don't know how to tie it all together.
Thanks

Please read through the other most recent issues.  This topic has come up a lot in the last week or so.

Yeah documentation SUCKS.
cannot figure out how to use get function.

Basically, Parser::get(), along with all the other helper methods, can only look at the HTTP request body. If the data you want to parse isn't in there, you can't use the helpers. They cannot be used on variables.

OH.... now it's more logic. I think you should add this info near function description or in documentation.

Agreed.
(Note: this explanation had already been given in other, older issues.)


(Cosmetic) Give the ability to modify distance between top panel and where grid view starts.

Feature Description
As the title suggests, I'd like to see that distance decreased somewhat to give more space to work with in Grid View.
Perhaps the addition of a setting that users themselves can fiddle with to modify that distance to their liking?
In any case, thanks!
Screenshots

I agree with this one, I love everything about this theme but the weird space under the top bar is my one little annoyance with it

Done in https://github.com/darklinkpower/Stardust/commit/983fca0ae72671a1d0ca75df13a64ccf6a89511c

Just to clarify, the image posted is incorrect since you can see that not all the area is the spacing of the top panel. Some of that area is created by the library group expander

Thanks for implementing this nice, albeit small, QOL improvement :)
And yes, my image was incorrect, I was more trying to convey the general gist of my request.


Add patch #199 from Propel 1.6

https://github.com/propelorm/Propel/pull/199
/cc @martinj

Merged :)


Layout Index Routes Don't Have Proper Id

Describe the bug
Follow up of #1150.
Something in this changes caused the route id's to get a bit strange. Also causing confusion in dev tools
File Tree:

Router Dev Tools:

Your Example Website or App
N/A
Steps to Reproduce the Bug or Issue
Create project as referenced above
Expected behavior
I'm not exactly sure what the route Id should be but im pretty sure the layout should come before the second /auth
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Firefox
Version: 1.19.2

Additional context
No response

cc @SeanCassiere

I've fixed this locally. Will do some more testing and open a PR if all works as expected.

@brandonryan this should be fixed now.


DNS service support in windows environment

refer to https://github.com/rancher/rancher/issues/10450
Use <IP_ADDRESS> as DNS nameserver in windows environment

@yasker

@orangedeng integration tests are failing, thats a bit of a red flag
@vincent99 @janeczku could you review this PR?

Seems mostly sane to me (without running it or anything...).
Shouldn't windows default to fallbackRecurse instead of empty?

Changed

@orangedeng Correct me if i am wrong, but it looks like the Windows build will always use Google Public DNS as recursing nameservers since it doesn't look up the hosts DNS configuration. What about cases of air-gapped deployments or where on-premise nameservers need to be used for other reasons?
Other than that LGTM 👍

refer to https://github.com/rancher/per-host-subnet/pull/3/commits/42b015937aa2e236c4acd4d5b63ac151d6d37d83
I have already set the default dns server into the network. So We can just use get-dnsclientserveraddress as our global recurse dns address.

@orangedeng i don't have a Windows desktop so i can't comment on the powershell stuff. Please see my remarks for the bash/Go code.
@deniseschannon

Updated.
@janeczku I don't know why I changed the ./test/integration/test_helper.bash to set the new binary path but drone still fail.

@orangedeng can you use the https://github.com/rancher/build-tools/blob/master/bin/tag-and-build.sh to sanity check the output? This is the script we use to release micro-service. Just make sure everything works properly by using that script.

unfortunately, some tests will always fail behind the GFW.

@StrongMonkey I run the script and everything for linux is fine. We will need to have a Windows Host to build the windows image. I am not sure how they plan to do this. So in the package script, I skip packaging Windows image when there is not a Windows host ip set to WINDOWS_DOCKER_HOST.

@orangedeng I think it is probably fine as long as it doesn't break the linux. We can have some extra release scripts for windows.


Removing yarnrc from build process

This gets rid of the .yarnrc file from the project and goes back to a symlink during the build. Fixes #243

This is replaced by PR #252


kdeFrameworks: 5.62 -> 5.64; kdeApplications: 19.08.2 -> 19.08.3

Motivation for this change
Update KDE Framework and applications
#71232 was already updating the framework but is blocked by the new Plasma update.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

ran nixosTests.plasma5 successfully

@grahamcofborg test plasma5

Could you have a look at the failing akonadi build on staging-next?
https://hydra.nixos.org/build/108066998
If you know of a fix, please open a PR to staging-next

Opened #75533


[ZOrder] Fast approach of interleaving bits

Description
to resolve https://github.com/delta-io/delta/pull/1149#discussion_r881782878
Improve interleave bits perf, the basic idea is from http://graphics.stanford.edu/~seander/bithacks.html#InterleaveTableObvious
How was this patch tested?

Existing UTs.
Does this PR introduce any user-facing changes?

No.
Benchmark
Before this PR:
[info] OpenJDK 64-Bit Server VM 1.8.0_292-8u292-b10-0ubuntu1~18.04-b10 on Linux 5.4.0-110-generic
[info] Intel(R) Xeon(R) CPU E5-2660 v4@ 2.00GHz
[info] 1000000 rows interleave bits benchmark:   Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
[info] ------------------------------------------------------------------------------------------------------------------------
[info] 1 int columns benchmark                             590            612          27          1.7         590.0       1.0X
[info] 2 int columns benchmark                             858            862           8          1.2         857.8       0.7X
[info] 3 int columns benchmark                            1102           1121          29          0.9        1102.5       0.5X
[info] 4 int columns benchmark                            1266           1291          23          0.8        1265.7       0.5X

After this PR:
[info] OpenJDK 64-Bit Server VM 1.8.0_292-8u292-b10-0ubuntu1~18.04-b10 on Linux 5.4.0-110-generic
[info] Intel(R) Xeon(R) CPU E5-2660 v4@ 2.00GHz
[info] 1000000 rows interleave bits benchmark:   Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
[info] ------------------------------------------------------------------------------------------------------------------------
[info] 1 int columns benchmark                             454            473          21          2.2         454.1       1.0X
[info] 2 int columns benchmark                             633            646          12          1.6         633.0       0.7X
[info] 3 int columns benchmark                             821            824           4          1.2         820.9       0.6X
[info] 4 int columns benchmark                             966            987          30          1.0         965.6       0.5X

cc and thank the Apache Kyuubi interleave bits author @ulysses-you

@vkorukanti @scottsand-db

@cfmcgrady This is absolutely great addition. I took a very quick cursory look, not in the details of the new logic, but overall PR. My major concern is that this is a lot of new code with very custom logic for handling up to some N columns in a more optimized way, but there isnt a lot of unit tests to really ensure the test coverage of the correctness in these cases. The baseline default case has a simple logic and therefore simple to test. All of these new code paths need sufficient testing. However, it also hard to explicitly conclusively test every combination that have been handled specifically.

Put this new code path behind a flag in DeltaSQLConf so that we can easily turn if off if there are concerns.

For test coverage, lets try to do randomized testing. Generate a sequence of random int, run it through the default and fast path, and make sure that the result is the same.

Make sure that the randomization is based on a randomly picked seed that is printed in the logs, so that if there is an error, we can use the logged seed to reproduce the exact input and output.
Make the number of iterations configurable in the code so that we can run it manually like 1B times (will take a while), but when we check it in, we do it like less iterations that will take say < 10 seconds.

what do you think?

Put this new code path behind a flag in DeltaSQLConf so that we can easily turn if off if there are concerns.

Added a new configuration fast.interleaveBits.enabled, which is enabled by default.

For test coverage, lets try to do randomized testing. Generate a sequence of random int, run it through the default and fast path, and make sure that the result is the same.

Added new randomized testing, which runs 100,000,000 random interleaved bit test cases.
[info] InterleaveBitsSuite:
[info] - 0 inputs (85 milliseconds)
[info] - 1 input (304 milliseconds)
[info] - 2 inputs (26 milliseconds)
[info] - 3 inputs (30 milliseconds)
[info] - nulls (221 milliseconds)
[info] - consistency (253 milliseconds)
[info] - supported types (11 milliseconds)
[info] - randomization interleave bits (2 minutes, 29 seconds)

Put this new code path behind a flag in DeltaSQLConf so that we can easily turn if off if there are concerns.

Added a new configuration spark.databricks.delta.fast.interleaveBits.enabled, which is enabled by default.

@cfmcgrady, The config is actually disabled by default, is that expected?
cc @tdas


Uninstalling app retain locksmith data

Hi
when I am uninstalling app from device, app saved data doesn't get deleted. Any idea?

Keychain data is not necessary removed when app uninstalled.
https://forums.developer.apple.com/message/75464#75464


Assertion error in JapaneseTokenizer backtrace

I found this bug when searching for solution from Lucene for the last issue I reported. I find it is reproduceable in lucenenet even the original report is for lucene 8.x. The descreptions are from  LUCENE-10059.
In a rare case an AssertionException will be thrown in the backtrace step of JapaneseTokenizer.
If there is a text span of length 1024 (determined by MAX_BACKTRACE_GAP) where the regular backtrace is not called, a forced backtrace will be applied. If the partially best path at this point happens to end at the last pos, and since there is always a final backtrace applied at the end, the final backtrace will try to backtrace from and to the same position, causing an AssertionError in RollingCharBuffer.get() when it tries to generate an empty buffer.
It can be reproduced by adding some code in TestJapaneseTokenizer:
public void TestEmptyBacktrace()  {
            String text = "";

            // since the max backtrace gap ({@link JapaneseTokenizer#MAX_BACKTRACE_GAP)
            // is set to 1024, we want the first 1023 characters to generate multiple paths
            // so that the regular backtrace is not executed.
            for (int i = 0; i < 1023; i++) {
                text += "あ";
            }

            // and the last 2 characters to be a valid word so that they
            // will end-up together
            text += "手紙";

            IList<String> outputs = new List<String>();
            for (int i = 0; i < 511; i++) {
                outputs.Add("ああ");
            }
            outputs.Add("あ");
            outputs.Add("手紙");

            AssertAnalyzesTo(analyzer, text, outputs.ToArray());
 }

This can be fixed by stop backtrace when the from and to pos are the same. I will create a PR that port the Lucene patch.
The PR may break the parity with Lucene 4.8 and may not get accepted. But I decide to create it in case that someone meet the same problem.
BTW, I find several other Lucene bugs exist in Lucene.net. I've done most of work that port the patches and will create PRs just for reference.

Sorry for created a PR containing irrelevant file changes. Please see the latest one.


(maint) Avoid transient error due to empty password

Previously the test created a user without a password which sometimes
fails in appveyor due to password complexity requirements. Set a
password that includes lower, upper, numerics and punctuation.

This should fix this error we sometimes see in Appveyor:
 User update failed: (in OLE method `SetInfo': )
           OLE error code:800708C5 in Active Directory
             The password does not meet the password policy requirements. Check the minimum password length, password complexity and password history requirements.
           HRESULT error code:0x80020009
             Exception occurred.

CLA signed by all contributors.


chore(package): update yarn.lock

coverage: 97.236%. remained the same
when pulling 3c545233c432cf53e1e936e5db5a6932b4b0b018 on update-yarn-lock
into 998af09972112c39dbb38c5c5f675241326367e6 on master.


not work with @angular/core": "^6.0.3

Hi @marcodevecchi , other obligations have kept me from keeping this up to date.
I have a branch I am working on that will update it to the latest version of Angular that should be out this weekend.
Thanks for understanding.

Any update on this? We are currently on Angular 7.2.7 and getting an error when trying to use this package.

Thanks for the response @yasiryc. I have a branch that is almost complete in updating it to the latest version of Angular.  There are just a few small things to fix (making it more customizable, better integration in to the page itself that should be added in this update).  Keep posted in the next few days!

I am working with angular version 8 and it emits an error when the project is compiled.
ERROR in ./node_modules/ngx-ssrs-reportviewer/fesm2015/ngx-ssrs-reportviewer.js
Module not found: Error: Can't resolve '@angular/elements' in 'C:\XXXXX\FUNC-668\SOLUCIONAPP\FRONTEND\XXXX.XXXX.Web.Pagos\node_modules\ngx-ssrs-reportviewer\fesm2015'.


stack exchange/overflow buttons

I created a buttons for Stack Exchange and Stack Overflow.

:+1: +1


geogebra: 5-0-745-0 -> 5-0-783-0, geogebra6: 6-0-745-0 -> 6-0-783-0

Description of changes
This pull request updates both Geogebra and Geogebra6 to their latest releases. The URL for the Geogebra package on MacOS now includes a "-x64" postfix. Since Electron version 14 is considered insecure, the Electron dependency of Geogebra6 has been updated to the latest version. I have only tested these changes on Linux x86-64, so I would appreciate it if someone could test them on Darwin.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

cc maintainers: @sikmir @imsofi @voidIess

Works for me on darwin x86_64.


[AutoPR storage/resource-manager] [Bug fix] Remove read-only constraint for management policy

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6551

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/6382


Add Tree Route Guard

Travels up the route tree to find a route that is configured for (Tree) Route Guard and linked to roles

The CI timed out. Please rerun the tests to let me see, if there are still issues left.


Add java solution of 91. Decode Ways

Hi,
I would like to add java solution for the puzzle of 91. Decode Ways
Can pls assign this issue to me for Hacktoberfest? Thanks!!

sure!
why not


Change selection color to work for blue backgrounds

Fixes #40797
Changes the selection color on elements with blue backgrounds to a lighter shade of the same blue (#7DA1FF) to maintain palette / aesthetics

@dipesh-rawat this should work, can you review this?

@kbhawkey @reylejano

/unassign @kbhawkey
/assign @sftim
could you review this? sorry, not sure who's active haha

/area web-development
/unassign
This isn't a high priority change; anyone is welcome to review.


Cleanup Fiori

This repo has gone through many iterations and ideas. I believe I have a firm grasp on what this repo should be now ;p I think ;p Cleanup time.

Resolved in https://github.com/mdn/mdn-fiori/pull/107


Restructure lib directory to organise source files

Is your feature request related to a problem? Please describe.
Although the files in lib are correctly named, it would help having them organised into sub-directories that aggregate related code. This feature would reduce mental effort in locating desired source files.
Describe the solution you'd like
For example, files related with the abstract syntax (e.g., intOp.ml or floatOp.ml) should be place in a directory titled syntax.
Describe alternatives you've considered
Alternatives have yet to be considered.
Additional context
No additional context.

Proposed directory structure:

lib/encoding.ml - Directly export modules we want to expose and documentation
lib/syntax - Files related with the abstract syntax
lib/semantics - Files related with the abstract syntax's semantics


Explicitly implement isValidX

The implementation in the base class will not work for rationals.
x.toLong == x.toInt does not guarantee that the value is within int range
for values that can become larger than a long.
Fixes #385
However, there should be some tests to make sure this does not happen again. Also, SafeLong seems to suffer from similar issues which will have to be tracked separately.

Looks great. Thanks!


Fix more Flakiness of JVMPauseMonitorTest

Instead of giving leeway on assertEquals, just check that the total time paused now exceeds the base sleep threshold.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10998/
Test PASSed.

alluxio-bot, merge this please


BottomNavigationBar items become white when more than 3 items are present

Steps to Reproduce
When adding more than 3 items in a BottomNavigationBar, all items turn white, and are unreadable on the light-gray background. Also, the active item is also white, and not the provided fixedColor. A minimal code sample can be found here
Also see the following images:

Logs
N/A
Flutter Doctor
[✓] Flutter (on Mac OS X 10.13.1 17B1003, locale nl-NL, channel alpha)
    • Flutter at /Users/dirk/Developer/flutter/sdk
    • Framework revision 8f65fec5f5 (5 days ago), 2017-12-12 09:50:14 -0800
    • Engine revision edaecdc8b8
    • Tools Dart version 1.25.0-dev.11.0
    • Engine Dart version 2.0.0-edge.d8ae797298c3a6cf8dc9f4558707bd2672224d3e

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.0)
    • Android SDK at /Users/dirk/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-26, build-tools 26.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[-] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    ✗ libimobiledevice and ideviceinstaller are not installed. To install, run:
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    ✗ ios-deploy not installed. To install:
        brew install ios-deploy
    • CocoaPods version 1.2.0

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.3)
    • Flutter plugin version 20.0.3
    • Dart plugin version 173.3727.108

[✓] Connected devices
    • iPhone X • 9500EA8E-1292-4223-BFFF-1B2134D5C47C • ios • iOS 11.2 (simulator)

/cc @xster

cc @HansMuller, I vaguely remember seen an older bug about this

when use BottomNavigationBarType.fixed that ok ，but when use BottomNavigationBarType.shifting that wrong

I too have this problem.  anyone know how to fix this?

same problem to me

I have added this to our list of things to look at later this year. We would be happy to review patches if anyone has the opportunity to look at the code and figure out what's going on before we get to it, though.

When more than 3 BottomNavigationBar items are provided the type, if unspecified, changes to BottomNavigationBarType.shifting per https://docs.flutter.io/flutter/material/BottomNavigationBar/BottomNavigationBar.html. This bit of information should be highlighted in the class's doc. It's easy to overlook where it is (I overlooked it).
When the BottomNavigationBar's type is BottomNavigationBarType.shifting the items text and icons are rendered in white, via DefaultTextStyle and IconTheme. It's assumed that theirBottomNavigationBarItem.backgroundColor will be specified as a contrasting color. This is obviously confusing.
The overall idea with shifting type bottom navigation bars is that each item will have a different background color (that contrasts with white), since that color will become the color of the entire navigation bar, when the item is selected.
The doc for BottomNavigationBar, and NavigationBarItem needs to be improved.

@HansMuller  Thanks, that worked!

@HansMuller, kudos!

bottomNavigationBar: BottomNavigationBar(
          type: BottomNavigationBarType.fixed,

Am I the only one that thinks that it shouldn't change behavior based on the number of tabs automatically?

thank you 👍

What do I do, if I don't want the icons to be white, yet it should have the behavior of shifting?

Why don't just make the default type .fixed?

I know this issue has been closed but it still feels like a bug even though the documentation has been nicely updated.
I'd suggest making fixed the default or fixing the behavior of BottomNavigationBarType.shifting with four items.

bottomNavigationBar: BottomNavigationBar(
type: BottomNavigationBarType.fixed,

bottomNavigationBar: BottomNavigationBar(         type: BottomNavigationBarType.fixed,

I provided the " backgroundColor" agrument for each "BottomNavigationBarItem" and the problem is solved.

Eu consegui resolver o problema inserindo a propriedade showUnselectedLabels: true, no BottomNavigationBar
return new BottomNavigationBar(
  showUnselectedLabels: true,

  items: <BottomNavigationBarItem>[...

what is we don't want the behavior to be shifting if we have more than three items. Is there any work around for it ?

add  showUnselectedLabels:true to BottomNavigationBar
add backgroundColor: Colors.blue to BottomNavigationBarItem
Then all shown

¿Soy el único que piensa que no debería cambiar el comportamiento en función del número de pestañas automáticamente?
No me parece intuitivo esperar un comportamiento diferente.

I think the same

[Solved] Flutter BottomNavigationBar not working with more than three items


Server State is "File not Found." Server does not appear to be running.

RapidScada version 5.8.4
Windows 10 Home Version 64-bit 1909 18353.1556
I was trying to do the RapidScada First Run Tutorial.  I was able to load the project DemoProject.en-GB. At 1:15 it advises to check the status of the services. In the video, the SCADA server details are displayed under Stats > Server State. But in my case Stats > Server state and Server Log both say "File not found."
What I've Tried:

Before installation steps from this video
Installation steps from this video
Uninstall RapidScada, delete SCADA folder and contents, reinstall RapidScada
Manually starting the server by running SCADA/ScadaServer/svc_start.bat. I get the following output:

"System error 5 has occurred. Access is denied.

Steps from similar topic "Unable to login: Server is Unavailable"

TCPView - Checked for other apps using port 10000. No apps are using port 10000, not even Rapid Scada.
Hercules - Used TCP client and attempted to connect to Module IP:<IP_ADDRESS> Port:10000. Received TCP connection error :10061

What partially fixed the issue:

If I run SCADA\ScadaServer\svc_start.bat as Administrator, it succeeds. Hercules, TCPView and RapidScada Administrator all show connection to the SCADA server. The only odd thing is that in RapidScada Administrator Server > Stats, it seems to lose connection for a beat every now and then, showing "No data" for a second and then going right back to State:normal.

Did I fix this the proper way? Is there something else I need to do to stabilize the server state? Is there a way that the server can start up by itself without me manually running svc_start.bat as Administrator?

Hello,

If I run SCADA\ScadaServer\svc_start.bat as Administrator, it succeeds.
Run services.msc and check if ScadaServerService, ScadaCommService and ScadaAgentService are configured to start automatically.  Likely, svc_start.bat was needed only for the 1st start.

The only odd thing is that in RapidScada Administrator Server > Stats, it seems to lose connection for a beat every now and then, showing "No data" for a second and then going right back to State:normal
The Agent service reads status files, for example, ScadaServer.txt, parses the service status from this file and provides to the Administrator app. If there is no errors in log files, there is nothing to fix.


Excel tool with macros blocked by security / ASR rules - how to fix?

I have an Excel tool (xlsm file) I'm spreading to users, they download the file from our site and after that can use it. For the ASR documentation I'm lacking an insight into the effects on Excel files with macros. In a nutshell:

the tool has its own menu bar and a bunch of macros that e.g. do calculations, pull in data from an API, etc.
I use Unviewable to hide the code from the users
I sign the VBA project with a certificate

Last month, an user reported having issues running the tool and passed it on to his IT department. They came back with several ASR rules they had to switch off to make the tool work (all macros etc). They were reluctant to do so because of "general virus threats". These were the 3 rules they had to switch off in Attack surface reduction (ASR):

Block Win32 API calls from Office macro
Block execution of potentially obfuscated scripts (js/vbs/ps)
Block JavaScript or VBScript from launching downloaded executable content
Reference page: https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/attack-surface-reduction-rules-reference?view=o365-worldwide

So my question 1 is: I think the obfuscated scripts line has to do with Unviewable, but have no idea what can cause the other 2 items in that list. Does anyone have an idea? Maybe some references in my project? And the second question: as I signed my tool with a certificate, wouldn't that for that IT department be a fix - just allow macros that are signed (and add my certificate to the trusted publishers in Excel)?
Cross post from:
https://stackoverflow.com/questions/75005280/tool-with-macros-blocked-by-security-asr-rules-how-to-fix

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

ID: 71a17626-d89b-a06f-9cc6-308b80a84acc
Version Independent ID: 71a17626-d89b-a06f-9cc6-308b80a84acc
Content: Attack surface reduction rules reference
Content Source: microsoft-365/security/defender-endpoint/attack-surface-reduction-rules-reference.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @jweston-1
Microsoft Alias: dansimp

HI @msbemba , thanks for your reply. One part of question is basically on the edge of 2 products: the Defender & MsOffice/VBA. I am looking for info on what macros would trigger the "Win32 API calls", that is documentation I expect to find with the Defender, not with MS Office. The same goes for the other options Defender offers - I would like to have some practical examples that I, as a VBA developer, can use. Currently, the page is only written for admins, but the people whose work it impacts (developers) are left in the dark. So I hope you can add some practical examples per option?
Thanks

@krijnsent  I will check with the author to see if there can be some practical examples that can be added per option.
Thanks
Sri

Emailed author  -2/1/23

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Return message if empty post

Summary:
Related issue: closes #194
Suggested reviewer(s):
Reviewer checklist:

[ ] The headers of all files contain a reference to the repository license
[ ] 100% test coverage of new code - meaning:

[ ] The overall test coverage increased or remained the same as before
[ ] Every function is accompanied with a test suite
[ ] Tests are both positive (testing that the function work as intended with valid data) and negative (testing that the function behaves as expected with invalid data, e.g., that correct exceptions are thrown)
[ ] Functions with optional arguments have separate tests for all options

[ ] Examples are supported by doctests
[ ] All tests are passing
[ ] All names (e.g., files, classes, functions, variables) are explicit
[ ] Documentation (as docstrings) is complete and understandable

The checklist is based on the S-ENDA conventions and definition of done (see General Conventions). The above points are not necessarily relevant to all contributions. In that case, please add a short explanation to help the reviewer.

I suggest to change line 142 in app.py to msg += f"\n Rejected persistent file : {file_uuid}.xml \n "


P2P: Feature-gate libp2p's mdns & remove wasm-timer

Description of change
This PR contains two Wasm-related changes in the stronghold-p2p crate.

Remove wasm-timer and add instant. This is required for proper Wasm support because libp2p itself has moved away from wasm-timer.
Feature-gate libp2p's mdns feature, which is not supported on Wasm targets.

Type of change
Choose a type of change, and delete any options that are not relevant.

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
No new functionality was added, so only existing tests were run.
Change checklist
Add an x to the boxes that are relevant to your changes, and delete any items that are not.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

I can only hope that you are doing this in order to have a thin stronghold client in the browser, without any signing or other cryptographic powers. Otherwise, this is a clear violation of the underlying philosophy of never putting a stronghold in the browser. Can you please advise to your planned usage of this feature?

Can you please advise to your planned usage of this feature?

This PR re-enables compilation of stronghold-p2p for Wasm. This is required for the identity actor we're building, which is supposed to be able to run in node.js applications, as an example. We have not fully decided on whether an actor will eventually also run in a browser, but since it would need to authenticate to another actor before being able to do any cryptographic operations in that scenario, that does not have the same security implications as running a full stronghold in a browser. A similar argument goes for a "remote stronghold" (i.e. a thin client that connects to a stronghold via stronghold-p2p) that we may want to run in a browser.
We definitely do not plan to run full strongholds in browsers.

The test in behaviour.rs also needs to be changed

I've added mdns as a feature of libp2p in the dev-dependencies so feature-gating should not be required in tests then, right?

The test in behaviour.rs also needs to be changed

I've added mdns as a feature of libp2p in the dev-dependencies so feature-gating should not be required in tests then, right?

You can specify what features are active when running a test, so we also need conditional compilation within the test. Even if mdns is included as dev-dependency, running cargo test --no-default-features fails.

We definitely do not plan to run full strongholds in browsers.

Understood. However if you make it feasible, someone will build it. Similar to gadget attacks, we also never shipped with functionality to read out a secret, you are potentially opening up venues for misuse and I think you would do well to make a MASSIVE disclaimer. Also, I am pretty sure this will trigger the requirement for an entire audit, since NEON bindings are not the same as WASM inclusion and the memory space of V8 + WASM means you have a plethora if new attack vectors such as electron and server-based applications to have reviewed.

Understood. However if you make it feasible, someone will build it. Similar to gadget attacks, we also never shipped with functionality to read out a secret, you are potentially opening up venues for misuse and I think you would do well to make a MASSIVE disclaimer. Also, I am pretty sure this will trigger the requirement for an entire audit, since NEON bindings are not the same as WASM inclusion and the memory space of V8 + WASM means you have a plethora if new attack vectors such as electron and server-based applications to have reviewed.

@nothingismagick maybe there is some misunderstanding here. This PR only targets the Stronghold-P2p crate (which does not have any Stronghold-related logic in it), not Stronghold itself.
Or am I missing something?

You can specify what features are active when running a test, so we also need conditional compilation within the test.

Got it, I feature-gated it.

ok ok - sorry. you guys know my paranoia with anything surrounding WASM. sorry for the bruhaha :)

If you look at the description at https://noiseexplorer.com/patterns/XX/ - there is one comment that is the same in every stage:
"Assuming the corresponding private keys are secure, this authentication cannot be forged."
If you are using WASM in the browser, how do you guarantee that the private keys are secure?

Thanks for the input @nothingismagick, we will discuss this some more before going ahead with the PR.

thanks @nothingismagick for your input!
The more I think about it, the more I agree, that this might not be a good idea. There is the (still experimental) credentials api, that give access to an agent's password manager. However, the password manager can be easily compromised, thus storing access credentials to a running Stronghold instance in the background would break the security model.
I would like to add some thoughts on how an extension might interact with Stronghold without compromising security, and give the flexibility of a browser extension:

Since it's a browser extension, user input must be allowed to be required. the integrating software, that most probably runs as a background service ( eg. daemon ) could issue one time tokens provided signed by a private key. The private key's hash could be shortly made visible during the creation inside the extension. A request is send to the background service, which in turn also requires confirmation by the user. A one time token is being issued for the resources requested. This token could only be used once, after that It would be invalid.
Since browser would have access to hardware tokens that support user interaction (eg. solokeys or yubikey), the private key could be stored inside the hw token, requesting user interaction each time a certain action is required inside the background service. This type of interaction with a hw token could also be part of Stronghold itself. The communication with the background service would only be token based.

Having said that, there might be still a window when secrets can be intercepted when issuing requests from the extension. We surely have to discuss this.


Update custom-visual-develop-tutorial.md

fixed typo

@mihart : Thanks for your contribution! The author, @sranins, has been notified to review your proposed change.


feat: option to configure session duration in shellable

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: 934dcdae967cd22c627e4824b6323bac44a4f721
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Closing as this is not a viable solution


Export runs that fail completely not reported in UI correctly

Confirming this. I used a corrupted preset file and export wouldn't start. Just got a spinning wheel.

Thanks Thomas. Fixed this on export detail page. Think the error you reported has to do with #95 which I've also fixed. These fixes will appear in the next release.. not sure when that is. You can review these changes at http://hot.geoweb.io/


Rs selective sync

PR for review only

Opened in error/habit against basho repo.

What repo are you making these PRs against these days? I was/am "watching" the commits to this repo and I'd like to track y'alls upstream repo too.
Thanks.


Look into Transitions export error

Transitions throws an error when pulling the full theme from magma (not just overriding certain colors)

Relates to #851


Display Active Projects

AirTable  view
https://airtable.com/shrJwZqwa8KIahHR6

I have integrated 2 Airtable views of projects, under active development and live projects. Debating about providing a Kanban view of projects additionally or instead of the current views.

Include all 3 views in page for review purposes. Easy to remove any view we don't wish to have.

After review at Hack Night. Provide 3 separate breakouts: Discovery, Active Development, Live


[BUG]:  Incorrect Schema Generated when introspecting extisting pg database

What version of drizzle-orm are you using?
0.30.0
What version of drizzle-kit are you using?
0.20.14
Describe the Bug
Context
I am using neon's PostgreSQL db, which is connected to my Directus instance.
I want to query directly from db using drizzle, so that i don't have to deploy Directus,
Currently i am running Directus locally on docker with neon db.
What i have done

Generated schema from existing db using drizzle-kit introspect:pg

Error
In the generated schema and SQL file

Default value is causing error

--> statement-breakpoint
CREATE TABLE IF NOT EXISTS "directus_roles" (
	"id" uuid PRIMARY KEY NOT NULL,
	"name" varchar(100) NOT NULL,
	"icon" varchar(30) DEFAULT 'supervised_user_circle'::character varying NOT NULL,
	"description" text,
	"ip_access" text,
	"enforce_tfa" boolean DEFAULT false NOT NULL,
	"admin_access" boolean DEFAULT false NOT NULL,
	"app_access" boolean DEFAULT true NOT NULL
);

Parsing error: ',' expected. 22:71:70 eslint_d
',' expected. 22:72:70 typescript1005

export const directusRoles = pgTable("directus_roles", {
	id: uuid("id").primaryKey().notNull(),
	name: varchar("name", { length: 100 }).notNull(),
	icon: varchar("icon", { length: 30 }).default('supervised_user_circle'::character varying).notNull(),
	description: text("description"),
	ipAccess: text("ip_access"),
	enforceTfa: boolean("enforce_tfa").default(false).notNull(),
	adminAccess: boolean("admin_access").default(false).notNull(),
	appAccess: boolean("app_access").default(true).notNull(),
});

I have around 50 tables now, and schema will be changing as the app grows
and this issue is almost in every table.
And Every time i generate schema it generates wrong code.
Expected behavior
No response
Environment & setup
linux 6.7.8.arch1-1
Hyprland
EndeavourOS

I am facing same issue. Is the any fix?
Currently I am having to search and remove all ::character varying from schema.ts file.

This should be fixed in<EMAIL_ADDRESS>If you still encounter this issue, please reopen the ticket


[atspi][clippy] fixes (some) clippy noise

Fix match on bool and fix an unneeded reference.
Fix cast from i32 to usize using 'try_from' - breaks AccessibleExts signatures. Guidance needed!
Silence Clippy on missing backticks in documentation. Not sure whether the backticks are appropriate here.

IMO most (if not all) of these doc comments should be replaced eventually. The only information they give is that the code was auto-generated. It's not helpful at all to anyone just wanting to write an AT.

I completely agree. Obviously, most of these docs were auto-generated themselves and it does not have detailed descriptions of very many things.
I do want to have really, really good documentation for release 1.0 of atspi. This is just on the back-burner for me right now. Contributions are very, very much appreciated!

IMO most (if not all) of these doc comments should be replaced eventually. The only information they give is that the code was auto-generated. It's not helpful at all to anyone just wanting to write an AT.

True. I will drop the silly 'backticks commit' as these comments are not helpful anyway.

I do want to have really, really good documentation for release 1.0 of atspi. This is just on the back-burner for me right now. Contributions are very, very much appreciated!

Fully agreed.
The AT-SPI2 documentation landscape is fragmented. Bits and pieces scattered all over the web. It would be good if we would not repeat that. I will commit to this goal as well.


Add RNTester box sizing example and e2e test round 2

Differential Revision: D63844898

landed in https://github.com/facebook/react-native/commit/3f1e381ded2d819ac4810902485998dd2aed0c71


Google Maps API based address autocomplete

Closing as we agreed to push this code to through the backend.


100% GPU usage when acceleration is on

Describe the bug
While GPU Acceleration is enabled it will max out the GPU.
Desktop:

OS: macOS
Ferdi Version <IP_ADDRESS>6e say so. -->

Debug information: https://debug.getferdi.com/2f89a847-7282-4e9c-a919-85f7963327fa
Additional context
Only active service was slack

Same -- my laptop sounds like it's about to take flight.
Possibly a coincidence, but I switched to Ferdi because I was having a similar issue with Franz.

Closing due to inactivity.


oz rpc for dynamic proxy

the problem:
Applications such as Ricochet and Onionshare will need to be able to receive their onion service connections from little-t tor which sits on the host outside of oz. The application will create it's listener but it will be listening inside the oz jail cell and therefore will never receive the connection from the tor process unless we setup a proxy rule for that particular port. We however cannot know in advance which port to proxy because applications like Ricochet typically decide this at run time.
one possible solution
We've been planning to deploy roflcoptor as the tor control port filter daemon. Oz profiles for applications like Ricochet will be configured to proxy the tor control port to roflcoptor running outside of oz on the host. This much I have tested: https://github.com/subgraph/oz/pull/65 the Oz tcp-to-unix socket proxy. oz-init creates a TCP listener inside oz that listens on <IP_ADDRESS> port 9051 which is the default tor control port that these applications use. Outside of oz it proxies to a specific unix domain socket that roflcoptor provisions and listens on with the filter policy for that specific application.
This however isn't enough. Onionshare can for example create it's onion service... however it specifies a port for the tor process to connect and forward to when it receives an onion service connection. The listener for these onion services are in the oz sandbox and therefore will not be able to receive the connection from the tor process unless we specifically configure oz to proxy those connections!
We must add an RPC interface to add and delete proxy rules. Roflcoptor can then use this RPC interface when it receives either an ADD_ONION or DEL_ONION command to create the necessary proxy rules to allow tor to actually connect to the sandboxed application's onion service listener. Initially we can do this with an oz TCP proxy but eventually it would be nice to configured unix domain sockets all the way down.

Everything for this is now in master and in the 0.1.0 tag! Thanks for the awesome work @david415 :+1: :100: :tada:


TYPO: Changed 'you' to 'your'

Issue Link :link:

Goals :soccer:

Implementation Details :construction:

Testing Details :mag:

Thanks @jacoblukewood! 🍻


press disable drag?

I want to use the press event on my app, but then I cant drag/scroll down the page if I happen to hit the photos expecting a press event. Is this the expected behavior? Am I missing something?

With out a test case its hard to say much but it sounds like you are using something thats setting touch-action value to none. press only needs manipulation. what other hammer events are you using?

@arschmitz i will come up with a plunkr trying to replicate this issue. Its worth saying that I am on a Ionic2 platform.

@arschmitz there is no need. You can check my issue at http://hammerjs.github.io/.
Interestingly it does not happen on the press event, but you can check it on the Rotate and Pinch/Zoom examples. You cannot scroll down the when hitting them, which its a bummer on a photo based website.
Tested on iPhone 6 and iPhone6 plus

@cescoferraro thats exactly what i was talking about it should not scroll on those examples. please see http://hammerjs.github.io/touch-action/ and https://www.w3.org/TR/pointerevents/#the-touch-action-css-property
Closing as not a bug.

@arschmitz is there a workaround this? setting the element touch-action to manipulation does not solve the issue

@cescoferraro are you setting the css touch action or doing it in your hammer instance the css is not supported in many browsers you must set it in your hammer instance like in the docs page i linked

i am setting the css.
I am not creating a Hammer instance. I am just using the event as an ANgular2 event like
<img  template="ngFor #item of allPhotosTWO " src="{{item.url}}" (press)="photoswipe()" alt="">

@cescoferraro im not an angular user but basicly the default hammer setup wont work you need to customize the hammer instance the css won't be enough.

i will try to create a component that initialize Hammer on its own initialization.
Would that be it?
var hammertime = Hammer(element).on("press", function(event) {
        alert('hello!');
    })
hammertime.({ cssProps: {touch-action: manipulation} });

the new component solved the issue right away!
thanks for your time @arschmitz
hammer rocks

@cescoferraro awesome glad you like!


Support zero rate and below one message/second rate

Is your feature request related to a problem? Please describe.
Running omq amqp-amqp --publisher-uri $TARGET --consumer-uri $TARGET -y 1 -x 1 --rate 0 results in the following:

 Log with "panic"
time=2024-05-06T17:25:55.988Z level=INFO msg="consumer started" protocol=amqp-1.0 consumerId=1 terminus=/topic/omq
time=2024-05-06T17:25:55.988Z level=INFO msg="Waiting for all publishers and consumers to complete"
time=2024-05-06T17:25:56.227Z level=INFO msg="publisher started" protocol=AMQP-1.0 publisherId=1 rate=0 destination=/topic/omq
panic: non-positive interval for NewTicker

goroutine 28 [running]:
time.NewTicker(0xc0000a40d0?)
	/usr/local/go/src/time/tick.go:22 +0xe5
github.com/rabbitmq/omq/pkg/amqp10_client.Amqp10Publisher.StartRateLimited({0x1, 0xc00019e0f0, 0xc00047c000, {0xa37a71, 0xa}, {{0x7fffbf1e2c89, 0x64}, {0x7fffbf1e2cfd, 0x64}, 0x1, ...}, ...}, ...)
	/home/ubuntu/go/pkg/mod/github.com/rabbitmq/omq@v0.4.6-0.20240506131709-8bad78b9f771/pkg/amqp10_client/publisher.go:106 +0x1b9
github.com/rabbitmq/omq/pkg/amqp10_client.Amqp10Publisher.Start({0x1, 0xc00019e0f0, 0xc00047c000, {0xa37a71, 0xa}, {{0x7fffbf1e2c89, 0x64}, {0x7fffbf1e2cfd, 0x64}, 0x1, ...}, ...}, ...)
	/home/ubuntu/go/pkg/mod/github.com/rabbitmq/omq@v0.4.6-0.20240506131709-8bad78b9f771/pkg/amqp10_client/publisher.go:86 +0x12b
github.com/rabbitmq/omq/cmd.start.func3()
	/home/ubuntu/go/pkg/mod/github.com/rabbitmq/omq@v0.4.6-0.20240506131709-8bad78b9f771/cmd/root.go:282 +0x18f
created by github.com/rabbitmq/omq/cmd.start in goroutine 1
	/home/ubuntu/go/pkg/mod/github.com/rabbitmq/omq@v0.4.6-0.20240506131709-8bad78b9f771/cmd/root.go:275 +0x3b0

Setting the rate to a low number as as 0.01 is not accepted as input.
Describe the solution you'd like
Support zero rate and below 1 messages per second.
Describe alternatives you've considered
No response
Additional context
Both of them (zero-rate, and below 1/s) are useful for many clients with few messages scenarios ("IoT").

I have a local branch for changing adding this but don't have access to push a new branch.
Is it possible to get access?

@annieblomgren - please see GitHub's documentation for creating a pull request:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request
Thank you for your contribution!

version 0.5.0 supports --rate 0. It just starts an idle publisher (if you don't want a publisher, use -x 0)


Suggestions for "Platform and tools: a kitbag for our journey"

"Don't forget to define the SPLUNK_HOME environment variable on your machine."

"Installing this version of the Splunk Event Generator is easy:"

Not sure my suggestions is appropriative, thanks!

Yep, you're right :) I'll get those changes made.
I definitely appreciate these edits! As you go through the guidance, please don't hesitate to reach out to me with questions or suggestions like this.

These changes are also live now. Thanks again!

Thanks! It's updated, closing.


TFRS - Credit transfer incorrectly categorized in TFRS #2916

#2916
Current code doesn't consider the edge case scenario where the difference is exactly 6 months, but ignores any extra days. Added fix to handle edge case scenarios.

Is there a data fix required with this ticket?

Is there a data fix required with this ticket?

@AlexZorkin , thanks for the review, yes a data fix is be required. Also can you please help on merging of the code, seems like it's being blocked and expecting to have 6 approving reviews.