Fix macOS migration nothing-to-upgrade test

When testing whether a macOS deployment target version migration is unnecessary due to already being at the target version, we were previously checking the iOS deployment target version rather than the macOS version. This updates it to test MACOSX_DEPLOYMENT_TARGET instead.
Issue: https://github.com/flutter/flutter/issues/107492
Issue: https://github.com/flutter/flutter/issues/107688
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] All existing and new tests are passing.

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

Looks like the commit is not in stable branch yet
user:~/flutter ‹stable›
$ git pull
remote: Enumerating objects: 130, done.
remote: Counting objects: 100% (123/123), done.
remote: Total 130 (delta 122), reused 122 (delta 122), pack-reused 7
Receiving objects: 100% (130/130), 184.68 KiB | 3.62 MiB/s, done.
Resolving deltas: 100% (122/122), completed with 72 local objects.
From https://github.com/flutter/flutter
ff59250dbe..2703a2bcde  main       -> origin/main
ff59250dbe..2703a2bcde  master     -> origin/master

[new branch]            revert-115009-fix_linux_remapped_capslock_throws -> origin/revert-115009-fix_linux_remapped_capslock_throws
Already up to date.
user:~/flutter ‹stable›
$ git log | grep d33a11c062f655f6d5f1b178cece971b5c409d43

This is available in 3.6.0-0.0.pre, see https://github.com/flutter/flutter/wiki/Where's-my-Commit%3F#finding-the-versions-that-contain-framework-commit-x


Not compatible with PyBrain 0.3.1

PyBrain 0.3 works fine.
Procedure:
Install PyBrain 0.3.1
Run test_pybrainbridge.py
Expected result:
All tests pass
Observed result:
13 tests fail with identical exceptions.
======================================================================
ERROR: testCopyable (__main__.TestNetworkEquivalence)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/nicholas/src/arac/src/python/arac/tests/test_pybrainbridge.py", line 348, in testCopyable
    copied = net.copy()
  File "/home/nicholas/src/arac/src/python/arac/pybrainbridge.py", line 266, in copy
    result.sortModules()
  File "/home/nicholas/src/pybrain/pybrain/structure/networks/recurrent.py", line 137, in sortModules
    super(RecurrentNetworkComponent, self).sortModules()
  File "/home/nicholas/src/arac/src/python/arac/pybrainbridge.py", line 274, in sortModules
    super(_Network, self).sortModules()
  File "/home/nicholas/src/pybrain/pybrain/structure/networks/network.py", line 259, in sortModules
    Module.__init__(self, self.indim, self.outdim, name=self.name)
  File "/home/nicholas/src/pybrain/pybrain/structure/modules/module.py", line 51, in __init__
    self._resetBuffers()
  File "/home/nicholas/src/pybrain/pybrain/structure/networks/network.py", line 263, in _resetBuffers
    super(Network, self)._resetBuffers(length)
  File "/home/nicholas/src/pybrain/pybrain/structure/modules/module.py", line 58, in _resetBuffers
    self.offset = 0
AttributeError: can't set attribute

@o-v-o    I got the following error.
@offset.setter
NameError: name 'offset' is not defined

add  a function to component.cpp
void
Component::set_timestep(int value)
{
    _timestep = value;
    return ;
}

add the function declaration in component.h
then in pybrainbridge.py
add the following :
    @property
    def offset(self):
        return self.proxies.handle(self).timestep()

    @offset.setter
    def offset(self, value):
        self.proxies.handle(self).set_timestep(int(value))

the original  self.proxies(self) does not work.
Then the uncompatibility should be fixed.


UI - Ability to set MTU in cluster yaml/config

UI for https://github.com/rancher/rancher/issues/24096

Is there a reason the MTU value is a string? Wouldn't it make more sense for this value to be an int? Should there be a default value in the schema? Seems like 1500 should be the default.

Hmm, I took it from other types like https://github.com/rancher/types/blob/master/apis/management.cattle.io/v3/rke_types.go#L165 but it seems inconsistent in types as I also see some int types. This can be easily changed (besides some vendoring). As this is an override from default behavior, we don't set a default.

@superseb I think we should change the type if there are no objections. There won't be anything for me to change on the frontend for this so it won't hold up my PRs. But this could prevent any issues from users using the api and sending a number rather than string.

The enhancement is validated in  Rancher v2.3-head baa64d48a with UI v2.3-dev
Check 1: there is a field for network MTU in the cluster creation page

Check 2: create RKE-provisioned clusters with different CNI (Canal, Calico, Weave) and  Network MTU: 4500
Result:

For the cluster with Canal, go to project System to check the Config Map canal-config to see the value for mtu

For the cluster with Calico, go to project System to check the Config Map calico-config to see the value for veth_mtu

For the cluster with Weave, exec into the workload weave-net under the project System  to check environment variables

/home/weave # env | grep  MTU
WEAVE_MTU=4500

Check 3: leave the field blank while creating the cluster with Canal
Results:

the MTU value is not set in the config map, which is the expected default behavior

blocked by https://github.com/rancher/rancher/issues/24647 in master-head

the validation is done and passed in Rancher:mater-head 48a2c4ffc UI vmaster-dev with the same tests in 2.3-head https://github.com/rancher/rancher/issues/24460#issuecomment-566784205


Improve ip_type function.

Currently ip_type is solely based on the existence of '.' or ':' which is terrible.

ip_type function now checks each octet/hexadecatet.  Essentially acts as an IP validation function for IPv4 and IPv6.


locator map jumping from mini tablet view or lower

see video
https://user-images.githubusercontent.com/56318362/199532988-6e43ca98-d4f5-433b-a0fb-c2d3b5c15c45.mov

I cannot see the video and don't know if that is already the issue, but in shortread, the locator map overlaps with the social media download


TFLite Android: Model file will not load. startOffset and declaredLength problems

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: Pixel 2XL, Android studio emulator
TensorFlow installed from (source or binary): Source
TensorFlow version (use command below): 1.9
Python version: 2.7
Bazel version (if compiling from source): 0.11.0
GCC/Compiler version (if compiling from source): 5.4
CUDA/cuDNN version: n/a
GPU model and memory: n/a
Exact command to reproduce: Loading TFLite model in an Android application

Posted as an issue here based on comment from https://stackoverflow.com/questions/51341554/tflite-android-model-file-will-not-load-startoffset-and-declaredlength-problem?noredirect=1#comment90097492_51341554
I'm having issues with loading a TFLite model using the MappedByteBuffer method from the Tensorflow-for-poets-2 TFLite tutorial.
private MappedByteBuffer loadModelFile(Activity activity,String MODEL_FILE) throws IOException {
    AssetFileDescriptor fileDescriptor = activity.getAssets().openFd(MODEL_FILE);
    FileInputStream inputStream = new FileInputStream(fileDescriptor.getFileDescriptor());
    FileChannel fileChannel = inputStream.getChannel();
    long startOffset = fileDescriptor.getStartOffset();
    long declaredLength = fileDescriptor.getDeclaredLength();
    return fileChannel.map(FileChannel.MapMode.READ_ONLY, startOffset, declaredLength);

In particular the model I have converted with the tflite_convert (formerly toco) tools fails when the fileChannel.map is returned. The model is a floating point TFLite mode.
The problem seems to be caused by the startOffset and declaredLength variables. The error I receive in the logcat is
7525-7525/dp.thexor A/libc: Fatal signal 6 (SIGABRT), code -6 in tid 7525 (dp.thexor), pid 7525 (dp.thexor)
If I fix these values to ones from a model that successfully loads then the method successfully returns the fileChannel.map. The values from my model are startOffset = 2846788 declaredLength = 45525464

I know that my TFlite model can successfully be mapped to memory as the tensorflow/contrib/lite/tools/benchmark:benchmark_model is able to benchmark it.
I'd try to load a quantized TFLite model but currently my model contains ops that do not have quantized equivalents (transpose_conv).
What could be causing this?
My .tflite model can be found here: https://github.com/andrewginns/CycleGAN-Tensorflow-PyTorch/releases/download/tf1.7-py3.6.4/float.tflite

Hi @andrewginns Have you tried using your model with the TF Lite demo app? (Trying to find out if this affect only tensorflow for poets or not)
It looks like your model is OK and can run with benchmark_model, for example:
$ bazel run -c opt tensorflow/contrib/lite/tools/benchmark:benchmark_model -- --graph=`realpath ~/Downloads/float.tflite`

Hi @andrehentz yeah I have tried with the TF Lite demo app. I went further and tried with a minimal app that should just load the model file into memory but that throws the same error.
That's what confuses me too. The TF Lite model benchmarks work on both desktop and mobile. I'm also able to view the layers of the model using the html visualiser.

Maybe you can narrow down the crash to a line of native code somewhere? If so, could you share a link?

I tried your model it fails in the model verification:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/lite/java/src/main/native/nativeinterpreterwrapper_jni.cc#L302
Can you share the original model before conversion, I will like to see what is going wrong here.

@shashishekhar
Thanks for looking into it. Here's my model before conversion.
https://github.com/andrewginns/CycleGAN-Tensorflow-PyTorch/releases/download/tf1.7-py3.6.4/frozen-graph.pb

@andrewginns : Quick update, I was able to reproduce the issue, I am working with @miaout17  to check if this is due to any alignment issues.

@shashishekhar Much appreciated

Nagging Assignees @miaout17, @jdduke, @shashishekhar: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@andrewginns : We did a few alignment related fixes and I reconverted the model, I was able to load it without issues, let me know if you are still facing issues. Use tflite_convert for converting the model.
Please reopen if you are still facing the issue.


Is there any way to work with this if the image dimensions are 4000x6000x3 ?

I'm trying to work on a segmentation task where the image dimensions are 4000x6000. Is there any way I can work with them without having to downsample the original images ?
I am not able to initialize any of the 4 models with that input shape.
Any suggestions are welcome.
(I tried with all the four models and the backbone resnet34)

Hi, @andohuman
You can crop your image to smaller ones, and train model on crops.


Port missing sub packages

There are a couple of sub packages that haven't been moved to this repo yet. It is not required to move it completely including code and tests but a start would be to have them bootstrapped into this repo.
If you want to move the whole package, create a new issue so everyone is aware that this is taken care of

Hey @christian-bromann I'd like to pick this one up.
Do you know which sub packages need moving or if you can point me in the right direction?

@fayeezdazn awesome, thanks!
Check out which packages don't have any source code yet (e.g. allure-reporter). Currently highest prio would have the wdio-cucumber-framework.

Okay great, thanks! Shall I create a new ticket to track each package or leave it with this one?

Okay great, thanks! Shall I create a new ticket to track each package or leave it with this one?

Yeah, for each package create a PR (can be just an empty one at the beginning), e.g. https://github.com/webdriverio/v5/pull/61

We moved all packages from the webdriverio org. Done!


kie-osgi tests: use single Karaf instance per test class to speed up …

…the build
I got about 50% improvement when running the tests locally.
@mariofusco, @jiripetrlik, @winklerm WDYT?

+1


[Android 4.4.4] [API 19] Top card disappears when trying to swipe it

Hello,
When testing the library on android Kit Kat 4.4.4, API 19 and when trying to swipe the top card. As soon as you hold it to swipe it disappears, although it is not removed because when you remove your finger the card reappears again ( not swiped more than the threshold)
I tried to debug, yet it seems only appearing on 4.4.4 emulator and devices. On 4.1.1 it does not appear. ( I have seen the closed issue where you mentioned that it works now on API 14, but it is not working on API 19 )
Can you please have a look?
Is this a known issue?
Thank you
Elie

Hello wenchaojiang,
I double checked the issue and it is related to the CardView and not to your CardStack, I tried the same CardView elsewhere and it is not working properly also
Still trying to figure out the root cause of the issue. I asked the question on stackoverflow if you are interested
http://stackoverflow.com/questions/36714540/android-cardview-on-4-4-4-becomes-transparent-when-animated-or-moved
I am closing the issue here since it is not directly related to your library.
Thanks
Elie


Fusion bookmarks module

Description
Please include a summary of the change and which issue is fixed.
List any external dependencies that are required for this change.
Completes: AB#FILL_IN_YOUR_ISSUE_ID
Type of change
Remove options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code.
[ ] I have linked my DevOps task using the AB# tag.
[ ] My code is easy to read.
[ ] Documentation and comments is added where needed.
[ ] My code is covered by tests.
[ ] UX has reviewed relevant UI contributions.

Leg til dette Completes: AB#FILL_IN_YOUR_ISSUE_ID

Please include a summary of the change and which issue is fixed.
List any external dependencies that are required for this change.

Dette ser ut til å trenge noen tester ;)


Update inbound-ses-spam-filter to include Lambda ARN

Issue #, if available: N/A
Description of changes: Currently, this filter is pretty useless to deploy as a nested stack. I cannot use the published resource because I cannot tie the Function ARN to my SES ReceiptRuleAction. This change adds an output so that this template is useful for nested deployments.
Description of how you validated changes: No validation
Checklist:

[x] Write/update tests
[x] make pr passes
[x] Update documentation
[ ] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

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

Codecov Report

Merging #1294 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #1294   +/-   ##
=======================================
  Coverage     94.5%   94.5%
=======================================
  Files           72      72
  Lines         3746    3746
  Branches       742     742
=======================================
  Hits          3540    3540
  Misses         104     104
  Partials       102     102

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 b56a09d...736f980. Read the comment docs.

Thank you for contributing this fix!


[Bug]: Path Error

Extension Version
0.1.14
PHP Binary
Local PHP
Operating System
Windows
What happened?
I’m experiencing an issue when using a Windows account with spaces in the username (e.g., test - name). The extension throws an error whenever it tries to execute certain operations. It seems the issue arises because the username or path is not being handled correctly.

[error] "c:\Users[username]" is not recognized as an internal or external command,
an executable program, or a batch file.
Mimimal Code Sample
No response

Possible duplicate of #19 ?

yup, i think it's the same problem, sorry i didn't see that there was already a report about it


feat: add max number of elements to non-prio queue

@arnetheduck how about replacing https://github.com/vacp2p/nim-libp2p/pull/1077/files#diff-abbf987f19a24c9940cd01bde79a1b1e0819f1c3b6d7ba2efa04ad680d022c78R182-R186 by unsubscribePeer?

@arnetheduck how about replacing https://github.com/vacp2p/nim-libp2p/pull/1077/files#diff-abbf987f19a24c9940cd01bde79a1b1e0819f1c3b6d7ba2efa04ad680d022c78R182-R186 by unsubscribePeer?

let's start with peer disconnection - when we reach a stable point with that working, we can consider other strategies (so that we can get a release out).

Codecov Report
Attention: Patch coverage is 82.14286% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 84.84%. Comparing base (c5db35d) to head (8c9a6ae).
Report is 1 commits behind head on unstable.

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable    #1077      +/-   ##
============================================
- Coverage     84.88%   84.84%   -0.04%
============================================
  Files            91       91
  Lines         15416    15423       +7
============================================
  Hits          13086    13086
- Misses         2330     2337       +7

Files
Coverage Δ

libp2p/protocols/pubsub/pubsub.nim
84.29% <100.00%> (ø)

libp2p/protocols/pubsub/gossipsub.nim
86.10% <50.00%> (-0.92%)
:arrow_down:

libp2p/protocols/pubsub/pubsubpeer.nim
87.73% <84.00%> (-0.50%)
:arrow_down:

... and 4 files with indirect coverage changes


[Bug]: Hover effect on the navbar

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hello @ap
The navigation bar appears visually unappealing and cluttered, which negatively impacts the overall design and user experience.
Expected behavior
The goal should be to create a visually appealing and functional navigation system that enhances the overall user experience, rather than detracting from it with an overwhelming or disjointed design.
Add ScreenShots

Kindly assign this issue under GSSOC'24.
Thankyou
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I'm a SSOC'24 contributor
[X] I have starred the repository

@sakshiglaze You are GSSOC Contributor of SSOC Contributor ?

@anuragverma108 I am both. But there is something that you can mark only one then my priority would be GSSOC.
Thankyou.

Can i work on this issue?


FileQueue component

Create a FileQueue component, that takes a list of files, and returns a channel with the appropriate buffer size to hold these files, which can then be assigned to a main in-port.
This in order to enable "sending" all input files into the network before triggering the execution by reading the last outport of it. Otherwise, if not all the filetargets fit into the network, the channel send operation on the inports will block indefinitely, so that the main thread can not proceed to the required reading of the last outport.

Implemented in 0e040e48d9d845ca30b7f8678fa39463d67a996e


fixhavingClause should be substituted the same way as resultExprs

Proposed changes
Issue Number: close #xxx
Problem summary
substituted havingClause in the same way as resultExprs to prevent " HAVING clause not produced by aggregation output" error
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.49 seconds
stream load tsv:          421 seconds loaded<PHONE_NUMBER>9 Bytes, about 169 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230504091336_clickbench_pr_138088.html

run buildall

run buildall

run buildall


Tests hang when running in Jenkins

I have been working to get CI setup for one of my projects that uses Jasmine as the test framework, but it seems to freeze once the testing starts.  I opened an issue with Velocity, but I haven't been able to determine if it's something with Velocity or something with Jasmine.
Is there a way to turn on some debug logging in Jasmine specifically, or some what to tell if it started and completed VS started and hung?  At this point, I just want to determine where the issue is occurring.

So, I stripped things down to their bare minimal:

Created a completely new project
Added sanjo:jasmine, velocity:html-reporter, and velocity:console-reporter to the packages and removed autopublish and insecure
Used the HTML reporter to generate the sample Server Integration test code (Players should play sound)
Started on my Mac with meteor run --test and I get a single run, no errors, and it outputs the tests passed

Now, I move it to my CI server and run it from CLI with the following command:
JASMINE_SINGLE_RUN=1 VELOCITY_DEBUG=1 VELOCITY_DEBUG_MIRROR=1 JASMINE_CLIENT_UNIT=0 meteor --test --release<EMAIL_ADDRESS>--port 9000

And it outputs the following and "hangs"
=> Using<EMAIL_ADDRESS>as requested (overriding Meteor <IP_ADDRESS>)

stream error Network error: ws://localhost:9000/websocket: connect ECONNREFUSED
I20150903-14:34:36.738(-4)? [velocity] adding velocity core
I20150903-14:34:36.743(-4)? [velocity] is in continuous integration mode
I20150903-14:34:37.010(-4)? [velocity] Register framework jasmine-server-integration with regex ^tests/jasmine/server/integration/.+\.(js|es6|jsx|coffee|litcoffee|coffee\.md)$
I20150903-14:34:37.021(-4)? [velocity] Velocity not started. Queueing func
I20150903-14:34:37.021(-4)? [velocity] Register framework jasmine-client-integration with regex ^tests/jasmine/client/integration/.+\.(js|es6|jsx|coffee|litcoffee|coffee\.md)$
I20150903-14:34:37.030(-4)? [velocity] Velocity not started. Queueing func
I20150903-14:34:37.031(-4)? [velocity] Register framework jasmine-server-unit with regex ^tests/jasmine/server/unit/.+\.(js|coffee|litcoffee|coffee\.md)$
I20150903-14:34:37.036(-4)? [velocity] Velocity not started. Queueing func
I20150903-14:34:37.172(-4)? [velocity] Server startup
I20150903-14:34:37.177(-4)? [velocity] app dir /var/lib/jenkins/jobs/WaiveForm (dev)/workspace/WaiveForm_app
I20150903-14:34:37.178(-4)? [velocity] config = {
I20150903-14:34:37.178(-4)?   "jasmine-server-integration": {
I20150903-14:34:37.179(-4)?     "regex": "^tests/jasmine/server/integration/.+\\.(js|es6|jsx|coffee|litcoffee|coffee\\.md)$",
I20150903-14:34:37.179(-4)?     "name": "jasmine-server-integration",
I20150903-14:34:37.180(-4)?     "_regexp": {}
I20150903-14:34:37.180(-4)?   },
I20150903-14:34:37.180(-4)?   "jasmine-client-integration": {
I20150903-14:34:37.181(-4)?     "regex": "^tests/jasmine/client/integration/.+\\.(js|es6|jsx|coffee|litcoffee|coffee\\.md)$",
I20150903-14:34:37.181(-4)?     "name": "jasmine-client-integration",
I20150903-14:34:37.182(-4)?     "_regexp": {}
I20150903-14:34:37.182(-4)?   },
I20150903-14:34:37.183(-4)?   "jasmine-server-unit": {
I20150903-14:34:37.183(-4)?     "regex": "^tests/jasmine/server/unit/.+\\.(js|coffee|litcoffee|coffee\\.md)$",
I20150903-14:34:37.183(-4)?     "name": "jasmine-server-unit",
I20150903-14:34:37.184(-4)?     "_regexp": {}
I20150903-14:34:37.184(-4)?   }
I20150903-14:34:37.184(-4)? }
I20150903-14:34:37.185(-4)? [velocity] resetting the world
I20150903-14:34:37.185(-4)? [velocity] frameworks with disable auto reset: []
I20150903-14:34:37.218(-4)? [velocity] Add paths to watcher [ '/var/lib/jenkins/jobs/WaiveForm (dev)/workspace/WaiveForm_app/tests' ]
I20150903-14:34:37.409(-4)? [velocity] File scan complete, now watching /tests
I20150903-14:34:37.411(-4)? [velocity] Triggering queued startup functions
I20150903-14:34:37.420(-4)? [sanjo:jasmine]: No tests for jasmine-client-integration found. Reporting completed.
I20150903-14:34:37.462(-4)? [sanjo:jasmine]: No tests for jasmine-server-integration found. Reporting completed.

However, I ssh port tunnel out to my CI server, and low and behold the app server is running!  It's not hung, it's actually ran the app.  On top of that, it is ignoring the tests in the /test folder.  There are also no logs at all in the .meteor/local/log folder.

The tests must be in the ./tests/ folder, not ./test/. Is that the case? The debug log should say that it found some tests. But it doesn't in your log.

The issue seemed to be with Velocity.  If you notice my path to the app had a parenthesis in it:
/var/lib/jenkins/jobs/WaiveForm (dev)/workspace/WaiveForm_app/tests

It appears that Velocity cannot handle the parans in the path.


do not log requests from kubernetes liveness/readiness probes for python/ruby apps

This PR took WAY longer than it needed to. Basically it looks to see if we $KUBERNETES_SERVICE_HOST is present, and if so, disable logging from any requests coming from Kubernetes that is identified with the user agent Go-http-client/1.1.
I wish we could be more specific on the incoming requests, but it doesn't seem like that's possible.

@bacongobbler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Event cannot be both direct and bubbling

Description
After changing node version from 14.17.1 to 16.13.0 I've got error:
 ERROR  Invariant Violation: Event cannot be both direct and bubbling: sid

This error is located at:
    in RNCSafeAreaProvider (created by SafeAreaProvider)
    in SafeAreaProvider (created by Application)
    in Application
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in travelee(RootComponent), js engine: hermes

When I render simplest tree
return (
  <View>
    <Text>test</Text>
  </View>
);

I've got similar error, but with RCTView
 ERROR  Invariant Violation: Event cannot be both direct and bubbling: sid

This error is located at:
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in LogBox(RootComponent), js engine: hermes

I've init new project and moved all sources there.
After installing dependencies app is bootstrapped fine.
But when I clone project from git, I still have the same error.
I tried to clean cache of yarn, metro, watchman and delete Pods.
Rollback to node 14.17.1 didn't help.
Version
0.66.3
Output of react-native info
System:
    OS: macOS 12.0.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 1.65 GB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    Yarn: 1.22.17 - ~/.nvm/versions/node/v16.13.0/bin/yarn
    npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
    Android SDK:
      API Levels: 29, 30
      Build Tools: 30.0.2, 30.0.3
      System Images: android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>42882
    Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.9 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.66.3 => 0.66.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

Clone project https://github.com/mihasia20008/travelee-mobile
Install dependencies and pods
Bootstrap app with yarn ios

Snack, code example, screenshot, or link to a repository
No response

The error was due to effector/babel-plugin.
After removing this plugin, it works.


Need support with conversion of a correspondence table to rdf (an example of direct correspondence using skos vocab)

I tried to develop a direct correspondence between YSTAFDB and Exiobase flows and activities using skos vocabulary. I chose to do it only for phosphorus flows (as these flows are the only ones available for 2011 which the data in Exiobase corresponds to).  I have made a selected choice of flows which can be broadly corresponded to the flows and activities in the supply and use tables (which is the only part of exiobase converted to RDF currently).
correspondence_Phosphorusflows.xlsx

Hi
we do not have yet a general procedure, but I think it would make sense to go from a (csv) table with 3 columns

subject
predicate
object

phosphate rock mining (PRM) refining
skos:narrower
Mining of chemical ...

...
...
...

and transform the labels into their corresponding URIs. This "transformed" table can be combined with a (datapackage) file of metadata of the table (provenance or whatever)... and then have  function in the library that creates a turtle file based on both. I am not sure why there are more than 3 columns in the correspondence example tab.

In YSTAFDB, the flow data is given with the activity types it is related to. So each row is a flow that consists of the activity the flow is an outputOf and inputOf. Hence there are more than 3 columns. If I turn this into the three column data forum, i think the information is not the same.

If the flow-object names in YSTAFDB are not unique, i.e. if the same name is used for different flow-objects depending on which activity it comes from, then this should be fixed before converting to RDF. e.g. by creating "flow-object from activity". However, this should probably best be done in cooperation with the YSTAFDB folks.


Where are we supposed to file bug reports related to DacFX?

UserVoice is a black hole of "if it's not upvoted it's ignored".
https://github.com/microsoft/DACExtensions/issues has not received Microsoft employee feedback in over 3 years.
Where should these issues be directed?

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

ID: 121d051e-9962-0554-b11d-ba22c1802ff7
Version Independent ID: 3fc25a21-a2b1-8d06-2ba2-373112e004ef
Content: DacFx and SqlPackage release notes - SQL Server
Content Source: docs/tools/release-notes-sqlpackage.md
Product: sql
GitHub Login: @pensivebrian
Microsoft Alias: broneill

@IanKemp -- Ian, thank you for your question.  Please consider adding your issue here:

SQL Server Feedback

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.
cc @pensivebrian for awareness

@IanKemp - You can also use https://developercommunity.visualstudio.com and file the issue against SSDT. The DacFx team monitors this queue for issues that are DacFx specific.


User is seeing a backend message after trying to edit the submission that has closed or closing state

Problem description
Backend request is visible for the users instead of user friendly message.
URL of the page
https://staging.getodk.cloud/v1/projects/79/forms/multi warning/submissions/uuid%3A92604e98-6ea7-416a-9361-82fab90f3cde/edit
Steps to reproduce the problem

Have at least one form in the project;
Send Submission for a form;
Navigate to the Form Access tab and change status of the form for Closing or Closed;
Go back to Submission page and click on the Edit button

Screenshot

Expected behavior
User friendly message should be visible
Central version shown in version.txt
106f533a128d43bb71ff2366bf2d07ff23082b38 (v2023.5.1-2-g106f533)
+00d84096788bbf31bd4ca903449848e8fd177159 client (v2023.5.0-34-g00d84096)
+e610ed1f41c1b04777c1779500c92d4cfea17ed0 server (v2023.5.0-30-ge610ed1f)
Browser version
Firefox 123.0.1
Around when did you see the problem (in UTC)?
10:10 AM UTC

We're hoping to fix the root cause at some point so that users are able to edit submissions to closed or closing forms. There's been related discussion at getodk/central-backend#469. However, it hasn't been an easy fix so far, and I don't think we plan to address it in the very near future. In the meantime, I agree that it'd be nice to prevent the user from seeing this message from Backend. How about we disable the submission edit button if the form is closed or closing and show an informative tooltip? That's what we do for the form preview button, which faces the same restriction.

This issue happens because of how Enketo communicates with Central. Once we replace Enketo with Web Forms, this won't be an issue anymore. After discussing this issue with @lognaturel, I think we should hold off on making this change, as we would have to revert or modify it once we further integrate Web Forms into Central. I'm going to go ahead and close the issue for now, but we can reopen it if we hear about more users being affected by it. Thank you for filing the issue, @srujner!


Upload Validator: modify CMYK validator to resolve a Promise with an array of warnings.

lib/uploadValidator
    .CMYKWarning
      ✔ returns a promise with an array containing a warning when validating a CMYK image
      ✔ returns a promise with an empty array when validating a RGB image
      ✔ appends warning to optionally passed in array of existing warnings

Addresses: https://github.com/adobe-community/issues/issues/14395

👍

👍


Stuck on Encrypting step

No Error just stuck on Encrypting when trying to join using my proxy

Hi, @Satanaelcode
Have you tried removing BungeeGuard from the proxy and backend servers to make sure it is causing the problem?


Issues how to fix

listening on port :7777
######## there's no ongoing at the moment => getongoinggameID
node:internal/process/promises:289
triggerUncaughtException(err, true /* fromPromise */);
^
[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "Not connected to server.".] {
code: 'ERR_UNHANDLED_REJECTION'
}
Node.js v19.8.1

Just comment the line 153 // getongoinggameID();


test: unixise the paths on the test (NFC)

This converts the path to use Unix path separators.  This is required to
ensure that the SOURCE_DIR substitution works properly in the case
that the path uses mixed path separators.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test


Usage with WebCrypto instead of node's crypto

I'm wanting to make requests to Flickr's api in a Cloudflare Worker, but it provides WebCrypto rather than node's crypto implementation.
This turns out to be causing a real headache especially as the WebCrypto API is asynchronous.
Any pointers gratefully received!

Neat! So there are two main parts to our OAuth stuff: there's the OAuth class, which implements the OAuth signature algorithm, and the OAuth superagent plugin, which uses the OAuth class to sign the request. I would probably start out by making your own copies of both of those files and edit them to be async/use webcrypto, then use your plugin instead. Let us know how it turns out!


FEATURE jQuery Dayjs - Color changing time blocks.

Add code to apply the past, present, or future class to each time block by comparing the id to the current hour.

DONE! initialized variable to capture the current hour of the day with day js. then create a loop that check each time blocks hour by parseinting their respective id, splitting the id at the hyphen into an array, then accessing the 2nd element in that array. then compare that integervalue with the currenthour value to add classes to them according to time of day dynamically.

added a wrapper around all the code as well as instructed. altered the code for this issue so that the color changing is a named function that will be called to activate every hour instead. looks more organized


I'm a student of Tsinghua University. When I ran the example of MINST, I encountered a problem involving urlib.error. I sincerely asked for the reason. Thank you very much

Short summary about the issue/question:
when i use nni/examples/trials/mnist-tfv1/config.yml
Brief what process you are following:

Instructions for updating:
Please use urllib or similar directly.
Traceback (most recent call last):
File "/usr/lib/python3.5/urllib/request.py", line 1254, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/lib/python3.5/http/client.py", line 1122, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python3.5/http/client.py", line 1167, in _send_request
self.endheaders(body)
File "/usr/lib/python3.5/http/client.py", line 1118, in endheaders
self._send_output(message_body)
File "/usr/lib/python3.5/http/client.py", line 944, in _send_output
self.send(msg)
File "/usr/lib/python3.5/http/client.py", line 887, in send
self.connect()
File "/usr/lib/python3.5/http/client.py", line 1276, in connect
server_hostname=server_hostname)
File "/usr/lib/python3.5/ssl.py", line 377, in wrap_socket
_context=self)
File "/usr/lib/python3.5/ssl.py", line 752, in init
self.do_handshake()
File "/usr/lib/python3.5/ssl.py", line 988, in do_handshake
self._sslobj.do_handshake()
File "/usr/lib/python3.5/ssl.py", line 633, in do_handshake
self._sslobj.do_handshake()
ConnectionResetError: [Errno 104] Connection reset by peer
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "mnist.py", line 235, in
main(params)
File "mnist.py", line 161, in main
mnist = download_mnist_retry(params['data_dir'])
File "mnist.py", line 151, in download_mnist_retry
return input_data.read_data_sets(data_dir, one_hot=True)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 272, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py", line 260, in read_data_sets
source_url + TRAIN_IMAGES)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 272, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 252, in maybe_download
temp_file_name, _ = urlretrieve_with_retry(source_url)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 272, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 205, in wrapped_fn
return fn(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 233, in urlretrieve_with_retry
return urllib.request.urlretrieve(url, filename)
File "/usr/lib/python3.5/urllib/request.py", line 188, in urlretrieve
with contextlib.closing(urlopen(url, data)) as fp:
File "/usr/lib/python3.5/urllib/request.py", line 163, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.5/urllib/request.py", line 466, in open
response = self._open(req, data)
File "/usr/lib/python3.5/urllib/request.py", line 484, in _open
'_open', req)
File "/usr/lib/python3.5/urllib/request.py", line 444, in _call_chain
result = func(*args)
File "/usr/lib/python3.5/urllib/request.py", line 1297, in https_open
context=self._context, check_hostname=self._check_hostname)
File "/usr/lib/python3.5/urllib/request.py", line 1256, in do_open
raise URLError(err)
urllib.error.URLError: <urlopen error [Errno 104] Connection reset by peer>
How to reproduce it:
I  work fllow this:
https://nni.readthedocs.io/zh/latest/Tutorial/InstallationLinux.html
nni/examples/trials/mnist-tfv1/config.yml

nni Environment:

nni version:
nni mode(local|pai|remote):
OS:
python version:
is conda or virtualenv used?:
is running in docker?:

need to update document(yes/no):
Anything else we need to know:
Log message:

nnimanager.log and dispatcher.log :

nnictl stdout and stderr :

Try again later or use a proxy. Most datasets are hosted on amazon s3 as I know, which is unstable to connect from China.

@xianglei3 - as suggested by @liuzhe-lz , it might be a dataset connection problem. Had you try the connection in other method or try another dataset insstead?

I’m closing this issue as it has no updates from customer for 3 months, please feel free to reopen if you are still seeing it an active issue.


Add reminder functionality to the autonag bot

For several purposes, it would be good to be able to have a bot come back and place a reminder on a bug.  This can be used for sec-approval to land a test or to come back and remove a preference after letting it ride the trains.
Checklist

[ ] Type annotations added to new functions - No.  I'm not sure these are needed?
[ ] Docs added to functions touched in main classes - n/a
[ ] Dry-run produced the expected results - No.  I'm not sure how to do a dry run for it.  I think I'll need some assistance.

Quick comment, haven't had time to look at the code yet.

For several purposes, it would be good to be able to have a bot come back and place a reminder on a bug. This can be used for sec-approval to land a test

We have https://github.com/mozilla/relman-auto-nag/issues/978 for that which we plan to implement soon. Were you thinking of this "reminder" functionality as a replacement or in addition to that?

or to come back and remove a preference after letting it ride the trains.

We are thinking of defining a process for expiring test variants (https://docs.google.com/document/d/1U17fEx9D1lRjJMfjNrGPjjbQCOy1qQjjuj9gufsXW9c/edit), which we will likely enforce with autonag. We were thinking of doing something similar in the future about preferences. Might be worth formalizing this instead of relying on a generic reminder functionality to ensure people don't forget to add reminders :)
That said, this generic reminder functionality is a neat idea!

I believe this patch would be useful and I'm OK if this supersede the work in #978. Tom tells me that the in-testsuite? flag is used inconsistently and there are bugs with this flag that do not even have a test attached.
Our main requirement is that we can get people to remember to check in tests, this here would support that use case.

The main drawback is that it becomes very hard to perform statistical analyses if it gets used for multiple things (stupid example: if you want to know how many sec bugs had tests to land). IDK if we will ever need it, but you never know what you might need in the future so it could be good to make it potentially possible.
What if we define a sub-whiteboard to specify what the reminder is about? E.g. "[reminder-test DATE]" or "[reminder-pref DATE]".

I believe this patch would be useful and I'm OK if this supersede the work in #978. Tom tells me that the in-testsuite? flag is used inconsistently and there are bugs with this flag that do not even have a test attached.
Our main requirement is that we can get people to remember to check in tests, this here would support that use case.

 This way we totally circumvent the problem and people don't have to remember to add reminders or keywords or flags.

I believe this patch would be useful and I'm OK if this supersede the work in #978. Tom tells me that the in-testsuite? flag is used inconsistently and there are bugs with this flag that do not even have a test attached.
Our main requirement is that we can get people to remember to check in tests, this here would support that use case.

 This way we totally circumvent the problem and people don't have to remember to add reminders or keywords or flags.

Yes. if that would take care of the backlog.   The output indicates that this is working now.

@suhaibmujahid okay, think I've addressed all those.

Looks good to me, we should just add a check so only people with editbugs (or canconfirm) privileges can set reminders up. You can add groups to the include_fields when you retrieve users, and then for each user check if there is a group which is "editbugs" (or "canconfirm").

Looks good to me, we should just add a check so only people with editbugs (or canconfirm) privileges can set reminders up. You can add groups to the include_fields when you retrieve users, and then for each user check if there is a group which is "editbugs" (or "canconfirm").

This is done.


[feature request] Simple drawing tools

It would be nice for the DM (maybe even the players) to be able to draw (freehand) on the map.
This would mean that players and the DM can track where they are and where the monsters are.
Without this, the tool is just limited to showing what the players can see.

I like the idea, currently the architecture is made for Push from DM and Pull from the Players.
I do not see anytime soon the other direction.

I think implementing simple drawing capability for the DM should be easy to implement. Just an additional canvas layer  must be added.

Oh I must of read it wrong. I thought he wanted to go from Player to DM. Not DM to Player.

In my own games, players have suggested a "token" (ideally adjustable in size) that could be dragged around the map, with support for multiple tokens to represent players and monsters. I will create a separate issue for that.

Tokens are now implemented, let's discuss the feature of drawing tools!

@n1ru4l You making a photoshop online version =) I like it.


BREAKING CHANGE: Making settings more precise

Introducing BREAKING CHANGEs to the cloudwash:

Introducing sections in settings for auth, criteria and exception. The auth should focus on authentication credentials for connection with cloud provider, the criteria should focus on criteria to delete the resource also the criteria could be defined per resource type and finally, the exception is the exception for resource type to skip the resource deletion and that is also per resource type based.
Introducing conf directory to define the configuration for dedicated cloud provider cleanup. To make use of them, the template files in conf directory should remove template extention and make them just the yaml file.
e.g azure.yaml.template file should be renamed to azure.yaml for cloudwash to read the credentials from conf/azure.yaml for azure resources cleanup.
EC2 is renamed to AWS in the swach command as well in settings file, so now users has to rename their command and settings. The aws cleanup will have more than EC2 cleanup support , hence this change.
Some settings has been renamed for better understanding of user.
Disc, Nic and Public IP settings could now be set to false for deletion if they are not assigned to any VM in settings files using UNASSIGNED setting value to False.
The stable version dockerfile is no more supported/removed due to conflicts between versions and settings file changes, so instead the latest(points to the master/latest) and version-based images should be available in the form of tags on quay. The latest tag is same as latest version tag in quay.
The OCP build config only supports for cloudwash DEV (or master) for now, could be improved in future for quay image-based as a stable version.
The docker image comes with both settings.yaml and conf directory YAMLs giving higher priority to settings.yaml always.
Fixed skipped_vms formatting to show the actual list than the skipped_vms string.

@jyejare there is no new quay latest image for 1.0.0 version. Is there any issue with that?

@franvila Yeah it failed in building image automatically in GHA https://github.com/RedHatQE/cloudwash/actions/runs/3128512035/jobs/5076533645
Looking into it now !


Fix off-by-one error when loading

With:
[DTA]
...
total_interval = 240
...

I assume the last interval _current_inter returned should be 240. In 85b1f88c64b08dd79d9e9845d60e1bc3b0a1c521 the loading will run for 241 intervals and return _current_inter as 241, which does not seem quite right. So commit da193f53fd429a5bb3e00b79071e52529805aba8 fixes this.
Qiling @psychogeekir, I do not use the DUE/DSO module, so I am not sure how this would affect them. Please review this change.
Besides, I also clean up the loading function a bit:

As an initial step to drop SNAP, use more primitive data types.
Use C++ stream IO more. Here I flush the output often by using std::endl because I think for verbose mode it makes a bit more sense.

Thanks for pointing that out. In the original code, the final returned interval is always total_interval + 1. Although this does not affect the DNL, I always thought that this place can be improved but have yet not taken any action. But this modification may affect multiple places since I wrote quite some checks based on it, so it may take me some time to do some tests before merging it.

Actually the commit 85b1f88c did introduce a regression -- test case test_3link failed at this line: https://github.com/maccmu/macposts/blob/42381486399ec428301ced256c8730f5b6105571/tests/test_dta.py#L50, and that was how I discovered this regression.
Anyway, commit 85b1f88c increased the number of intervals by one, FWIW.

I fixed the off-by-one issue and it turns out that it did not require many modifications.


Computer vision failing to return a result

I am not sure if this is normal but all my images do get get a caption because result.description is returning null in the ResizeImage function.
What I would like to know is if this is probably a mistake on my part or just computer vision is not reliable?

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

ID: b1acd3de-471c-d954-af1a-62f7fc440370
Version Independent ID: 84549c67-6733-e861-6ec9-aab9f248a406
Content: Build a serverless web app in Azure
Content Source: AzureInteractive/tutorial-static-website-serverless-api-with-database.yml
Service: multiple
GitHub Login: @ggailey777
Microsoft Alias: glenga

@noobling Thanks for your feedback! We will investigate and update as appropriate.

@noobling Are you following any specific Microsoft documentation on vision API as I do not see mention of vision API in the current document you have created this issue.

@SaurabhSharma-MSFT The documentation in question is here https://docs.microsoft.com/en-gb/azure/functions/tutorial-static-website-serverless-api-with-database?tutorial-step=5 More specifically the exact code is here https://raw.githubusercontent.com/Azure-Samples/functions-first-serverless-web-application/master/csharp/ResizeImage/run-module5.csx

@noobling I have followed the tutorial and I do not face any issue with the result.description.

@noobling We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


✨ Making StrictlyNegativeInt.Companion.create* factory functions available on Java

📝 Description
We would like to make the StrictlyNegativeInt.Companion.create* factory functions available for Java users.
This means that their compatibility should be tested and their documentation should indicate their usage from Java code.
🔗 Dependencies
This issue is blocked by the following ones:

[x] #347
[x] #434
[ ] #598

✅ Checklist

[ ] Make the create(Number) function available for Java users, test its compatibility, update the public API binaries and update the unreleased changelog.
[ ] Make the create(Number, () -> Any) function available for Java users, test its compatibility, update the public API binaries and update the unreleased changelog.
[ ] Make the createOrNull(Number) function available for Java users, test its compatibility, update the public API binaries and update the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

Another issue for stabilizing these functions will be created while resolving #619.


User views all scheduled migration plans

need to verify here if we will show Scheduled Migrations "in it's own list" -

we may need to say given this plan, is it scheduled
or we may need to say give me all the scheduled plans

@blomquisg - is there an API functional requirement for this?  Or do we think "this comes for free 💸" once we have scheduled migrations

@vconzola please add mockups for viewing scheduled migration plans

is there a filter for this? List View?  Column for schedule time? what would the format be?
is there a card for scheduled migrations?  or are they lumped into "not running"

depends on #369

depends on #369

@serenamarie125  I don't see a way to filter the list view for a schedule column that is present vs. not present, so I was thinking we could sort on that column by date. Also, scheduled plans are lumped into the Not Started bucket.  Mockups (WIP) here.

for the sake of 🖼

@vconzola @serenamarie125 gonna repeat back a summary of this ask, looking for yay/nay for if its close to accurate

This work is asking for a toolbar to be added to each different state of the migration plans lists.
This toolbar will enable us to sort by each column of each list.
Given we want to view only scheduled migrations, this toolbar will enable us to filter values of each of the lists

@AllenBW I was thinking the toolbar would only be for the list view states - Not Started and Completed. Also, per the notes in the mockups, "Options for the sort are Name, Infrastructure Mapping, and Scheduled Date/Time. This should at least pop scheduled plans to the top of the list." However, if it's easier to sort by each column in the lists, go for it.
Regarding viewing only scheduled migrations, I don't know how we can do that since we don't have a scheduled/not scheduled column to filter on. That's why I suggested the sort. My hope is that if we sort on the Scheduled Date/Time column plans with a schedule will appear first and those without a schedule (column contents are blank) will appear last. Is that the way ascending sort works on a field that has dates and times?
Let me know if this answers your questions and/or if you have other/better ideas.

@vconzola thanks for the quick reply! Yep that answers my questions, enough to start movin on this :heart: 🤗

@priley86 consider it done! yeah agreeeeed it would be wise not to write yasf (yet another sort function)
@vconzola need yer 👁 on this ss 👇 about the flavor of what you were thinking?  need to ask because... we'll be removing a few pf stylings from the toolbar, backgrounds, borders, box shadows...

@priley86 OH oh oh wait... are we gonna add pagination to this this?

We aren't doing pagination on the backend, so I'm assuming that it would still apply.

@AllenBW If we do pagination - probably want to do it across the board as a larger effort? I don't know which APIs do/don't support it so have to defer to you and @AparnaKarve . I don't think @vconzola has asked us to prioritize it yet.

@priley86 everything that has an api (should) support limit and offset so for service templates, It might look a little like this:
GET http://localhost:8080/api/service_templates/?filter%5B%5D=type%3D%27ServiceTemplateTransformationPlan%27&filter%5B%5D=deleted_on%3Dnil&expand=resources&attributes=name,description,miq_requests,options,created_at&sort_by=updated_at&sort_order=desc&limit=5&offset=2

@vconzola present mocks don't mention pagination, is that something we want to add here?

@AllenBW yes, just realized we did not clarify about server vs client side pagination. We did client side pagination elsewhere, but if we decide server side is possible, then by all means. It’s probably worth re-evaluating which UIs we could server side page if that is an option.

It’s probably worth re-evaluating which UIs we could server side page if that is an option.

@priley86 Agree.
Implementing Server side pagination could be an appreciable effort and would only make sense if we expect some of our lists to be really huge.

🤷♂ gonna sound real cold here, but.....
Pagination wasn't mentioned in mock, not something to be done in this scope of work.
Just need quick 👁 on the ss above, to confirm its what @vconzola wants.  Reason for this ask is because it modifies a few in place PF designs (toolbar things).

@AllenBW - i think we decided in the sprint planning meeting to defer pagination for now. The idea being - we'd like to revisit that in the future after @vconzola rearchitects some more of the Overview UI. Let's keep it simple for now ;)


[FEATURE] Add PageSize option to more cmdlets

Is your feature request related to a problem? Please describe.
I would like to have -PageSize as an option for the following cmdlets: Get-PnPFileInFolder, Get-PnPFolderItem, and Get-PnPFolderInFolder.  Right now, PageSize is not supported.
I have a large Sharepoint library with > 4 million files.  Right now I have to use Get-PnPListItem, which loads the entire site.  This is incredibly slow and memory intensive.  I tried using Get-PnPListItem -List "Documents" -FolderServerRelativeURL "/Sites/Url/", but  it doesn't seem to work when PageSize is specified.
Describe the solution you'd like
I would like to be able to specify PageSize for the aforementioned cmdlets so that they can be used on large folders and subfolders.
Describe alternatives you've considered
I've considered buying 256GB of RAM and carving out 2-3 weeks to let the script work, but there has to be a better way.

To avoid loading the whole list/library in memory, use the -ScriptBlock parameter of Get-PnPListItem

The script block to run after every page request.

It accepts script block like { Param($items)  } where $items contains as many items as specified in -PageSize .
I was able to process 1.5 M files in a library this way without any RAM issues.

To avoid loading the whole list/library in memory, use the -ScriptBlock parameter of Get-PnPListItem

The script block to run after every page request.

It accepts script block like { Param($items) } where $items contains as many items as specified in -PageSize . I was able to process 1.5 M files in a library this way without any RAM issues.

Thanks.  I didn't think of using ScriptBlock - I will try that.  I do think that adding PageSize to more cmdlets would be helpful as scriptblocks are a bit kludgy, especially if you need to reference other variables.

hi @sg1888 , this has been fixed/added in the latest nightly builds.
Closing this.


fix:github.com/istio/istio/issues/40767

Signed-off-by: cobb-tx<EMAIL_ADDRESS>Please provide a description of this PR:
fix：https://github.com/istio/istio/issues/40767

Hi @cobb-tx. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Does not support this


introduce unnamed slot behaviour

This will allow us to insert contents anywhere in the parent template without ugly slot name magic in the child templates.

👍


crypto/sha256: provide a way to efficiently hash multiple byte slices

Using go1.9.
I'm trying to implement an optimized version of HMAC-SHA-256 digest computation that avoids allocation.
Current code is available in [this github repository]](https://github.com/chmike/hmacsha256)
The function signature I provide is the following:
func Digest(buf []byte, key []byte, data ...[]byte) []byte

The MAC is computed over the list of data slices given as argument.
Unfortunately, I'm not able to use the hardware implementation of SHA, so I'm stuck.
"internal/cpu" is not accessible.
I could use the  sha256.Sum256(data []byte) [Size]byte function, but it accept only one data slice.
With a small backward compatible change to this function signature, it would be possible to compute an SHA over multiple slices without needing an allocation with sha256.New.
hmac requires to compute a hash over at least two data blocks.
If there is another solution to use the hardware or assembly optimized sha256 block function, I'm a buyer.
PS: as explained in the end of the readme of the github repository, my goal is to use this for secure cookie which has to be verified for nearly every request. Avoiding these basically useless allocations, could reduce the pressure on the GC.

Unfortunately we can not make that change, as it would violate the Go 1 compatibility guarantee (while existing calls would still compile, other uses of the function might not).
The sha256.Sum256 function is quite small and simple.  I recommend just making a copy of it that is modified to serve your needs.
Closing.

Oh, sorry, I failed to notice that digest is private.
I will reopen.  We can't do what you suggest, but perhaps we can do something.

Given this snippet:
var b1, b2, b3 []byte
var h [sha256.Size]byte

d := sha256.New()
d.Write(b1)
d.Write(b2)
d.Write(b3)
d.Sum(h[:0])

Even though New returns an interface with a *digest, it seems to me that the compiler should be able to figure out the following:

The concrete type of d is really just *digest instead of hash.Hash
Thus, the call to Sum should not cause h to escape. The input to digest.Sum does escape to the output, but in this case, it is entirely ignored.

Thank you for reopening the issue.
Another and simpler solution would be to add a new function accepting a varargs.
Sums256(data ...[]byte) [Size]byte

This would be perfectly backward compatible, and it would be easy to understand what it does.

As @ianlancetaylor, this is not backwards compatible as it changes the signature of the function.

@dsnet @chmike's last suggestion was a new function (Sums256, not Sum256).
I agree that normal hash.Hash usage should be optimized by the compiler here rather than adding new API.

Dup of #19361 and/or #21070 ?

@bsadfitz #21070 is about Sum but it address a different issue which I agree with. AppendSum would make  clear that it uses the append pattern. It took me some time to figure it out. For #19361, I don't see the link with this issue.
A compiler optimization would allow to fix the problem for all hashes in one hit. But I have no idea of how much effort it would require.   After thinking about it, I agree that adding functions would a false good idea.
PS: For the secure cookie, I think I can get around the problem by concatenating slices to hash. This should be possible by arranging the data differently with a minimum of data copy. So for my point of view, the possibility to hash multiple slices by avoiding a heap allocation is now "good to have" instead of "must  have". Beside, hashing one block is faster than hashing multiple block fragment. So I hope the concatenation cost will be compensated by that benefit.

@chmike
As far as I can see you should be able to use a sha256 instance within your HMAC struct:
type hmac struct {
    ipad, opad [sha256.Size]byte
    digest hash.Hash
}

and use Hash.Reset. This should avoid the allocations as far as I can see...
Nevertheless Go can do the compiler optimization but for this specific use case I don't think that it's required.

Thanks for the suggestion. This is what I did. Code is available at github.com/chmike/securecookie.
I also rewrote the ctr ciphering computation to avoid the ctr allocation for the same reason. The aes block cipher is allocated once.
My secure cookie get and set need only 3 allocations and they are all outside of the value encoding and decoding.  As a comparison, the gorilla secure cookie get/set require respectively 37 and 39 allocations. While there is a factor 10 difference in number of allocations, the speed difference is only around 3.
Of course the gorilla secure cookie code is much more readable because it use the New functions and don't use arrays to avoid heap allocations.

The problem here is that sha256.New returns a hash.Hash. If it returned a *digest then the compiler would have an easier time inlining the whole thing and then seeing that nothing escapes and removing all the allocations. There was a recent CL that does the first step of devirtualizing this kind of thing. I think we should probably let the compiler optimizations continue to the point where the obvious code just doesn't allocate. That's better than adding new API that won't be (hopefully) necessary in a few releases. It's fine of course if you want to fork sha256 for your use.


.flaskenv should respect use_celery config

The current .flaskenv file is generating celery-related config even when
the use_celery option is false.

Thanks for this PR, I'm merging it


Duplicit declaration of 'null'

Hi guys.
We have input object called IntFilter which contains filed named 'null'.
null : Bool -> IntFilter -> IntFilter
null newArg_ inputObj_ =
    GraphQL.Engine.addField
        "null"
        "Boolean"
        (Json.Encode.bool newArg_)
        inputObj_

In generated code is this name already used, so it doesn't compile.
null :
    { eq : IntFilter -> IntFilter
    , gt : IntFilter -> IntFilter
    , gte : IntFilter -> IntFilter
    , in_ : IntFilter -> IntFilter
    , lt : IntFilter -> IntFilter
    , lte : IntFilter -> IntFilter
    , neq : IntFilter -> IntFilter
    , nin : IntFilter -> IntFilter
    , null : IntFilter -> IntFilter
    }
null =

Can I suggest renaming this function to something, which can't collide? For example null_ or.. I don't know. You know better than me, how to call that thing.

Good call, that makes sense!


Revert "feat: migrate base/dds.dart to null-safety"

Reverts flutter/flutter#88015
See https://github.com/dart-lang/sdk/issues/46934

/cc @wanglikun7342

Can't get the full package to null safety fast enough.  Thanks for tracking it down, @knopp!

Thanks! There are multiple call sites where startDartDevelopmentService is called with nullable ddsPort argument without proper fallback (?? 0).


New package: GEE v0.1.0

Registering package: GEE
Repository: https://github.com/kshedden/GEE.jl
Created by: @kshedden
Version: v0.1.0
Commit: f2aab1a626634fdc874f82bf0416161ed800d3bd
Git reference: HEAD
Description: Generalized Estimating Equations (GEE) in Julia

This is not a widely knoen acronym.
See point 1 here;
https://pkgdocs.julialang.org/v1/creating-packages/#Package-naming-guidelines

GeneralizedEstimatingEquations seems like a doable package name. Maybe a tiny bit long, but definitely ok.

Ah, never mind, I see you already renamed to EstimatingEquationsRegression, which is also fine.

This can be closed, I renamed and the new package is here: https://github.com/JuliaRegistries/General/pull/84478


Add Alternate modes for positioning

Implement positioning logic for center, flex-end and space-between.

Fixed in 99216ec4066eca62f48605995622c03beedb71cc


[Android] Sending custom object through SendPort makes app crash - Release only

I've been using this library for a while now, but the last Version created a new bug for me.
I've mannaged to find a workaround in the way i'm using the library to avoid that crash, but I though I'd open an issue, because this is still a bug worth fixing.
A bit of context :
Platform : Android (no bug in IOS), reproduced in Android 10, 12 & 13
Phone : Reproduced on Pixel 6A, crosscall X4, Samsung A12
Note that this issue is happening ONLY in RELEASE, in debug I have no issue.
The issue :
To communicate between the Forground Service and the App, I'm using SendPort.send et Received port, like in the exemple.
When I use these port to send Data like Int, String or Map I have no issue.
BUT I was using them also to send a simple custom Object that I wrote, which is basically a simple class with different string and int attribute. And when I send this object via the SendPort, the App Crash.
It uses to work fine with the previous version of the library, and it is still working fine in Debug.
With a lot of time and print, I have made sure that the crash is only related to the object being sent through the send port.
Indeed :

If I just send a String -> No crash
If I send my CustomObject be with nothing on the side of the received port except for a print -> crash

In the Documentation about sendPorts, I saw the following :
Sends an asynchronous message through this send port, to its corresponding ReceivePort.
The transitive object graph of message can contain the following objects:
Null
bool
int
double
String
List or Map (whose elements are any of these)
TransferableTypedData
SendPort
Capability
If the sender and receiver isolate share the same code (e.g. isolates created via Isolate.spawn), the transitive object graph of message can contain any object,

So what I guess is that the isolates in this library are now handled diffrently and do not "share the same code", and that's why sending my custom object makes the app crash, but if I send something else, it does not.
Maybe this is because of the new "pragma" we need to add ?
My Workaround
Instead of sending my CustomObject, I created a method who transform it into a Map, with it's Attributes as Elements of this map.
Something like :
Map _map = Map();
_map ["Attribute1"] = CustomObject.Attribute1 ;
_map ["Attribute2"] = CustomObject.Attribute2 ;
_map ["Attribute3"] = CustomObject.Attribute3 ;

And then I send this Map instead of My Object. Then when I received this map in the Received Port, I can build my Object Again.
With this Workaround my code works fine, but I think it would be nice to have the SendPort to work again like it use to.
Does anyone notice the same thing ? Does anyone know how to fix it ?
How to Reproduce the Issue :
1- Create a custom class with some random attributes :
class CustomObject{
int attribute1 ;
String attribute2 ;

CustomObject(attribute1, attribute2);
}

2- In your Foreground task somewhere, send it through the SendPort :
sendPort?.send(CustomObject(1,"test"));

and eventually add in your ReceivedPort a methode to handle t hat data or just to print something when any data is received.
3- Notice your app crash

any update ?

On my side, I am still using the workaround I mentionned.

I have the same issue .Thanks you.
you helped me saved a few day on this problem

this lib has serious issues with the release mode. eventually, your workaround does not fit into my use case. I hope flutter will add its own background service workaround to exit the UI only App SDK box.

using map DS to send custom objects to prevent crash. it worked for me.

Thanks you solved my issues.

I confirm that sending a map solves the issue. But, it is still a bug... or would need to be documented...


fix WriteBytes itemCount

Hello. I think WriteBytes itemCount maybe uint16(len(b)/2), not uint16(len(b)).

Hi, agree that in common case it should be /2 but can you add a test or some to discuss the behavior on concrete example?


Azure Devops Security Group from AAD documentation

Unclear documentation on az devops security group create from AAD group
Duplicate of https://github.com/Azure/azure-cli-extensions/issues/1287
Suggest adding the information from https://github.com/Azure/azure-devops-cli-extension/issues/667 to the documentation

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

ID: 5348dfb5-d361-4857-1459-6d7f3cbf5d6c
Version Independent ID: 50319cc5-540d-6689-e5e7-748281d7d9ee
Content: az devops security group
Content Source: latest/docs-ref-autogen/ext/azure-devops/devops/security/group.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Devops


Suggested fix for issue #560

Summary
Resolves issue (bug) #560
With this fix, the following issue in CVAT will also be resolved and does not need further changes: https://github.com/openvinotoolkit/cvat/issues/3940
How to test

This was tested manually through the CVAT tool:

Create a new project and upload a video
Open the task
Click Actions -> upload annotations -> MOT 1.1
Select zip folder with MOT annotation format (including seqinfo.ini file) and click 'upload'

With this PR, importing should succeed
Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues

Could you please add a test using seqinfo file in https://github.com/openvinotoolkit/datumaro/blob/develop/tests/test_mot_format.py? Probably, with an example directory in https://github.com/openvinotoolkit/datumaro/tree/develop/tests/assets/mot_dataset.

Could you please add a test using seqinfo file in https://github.com/openvinotoolkit/datumaro/blob/develop/tests/test_mot_format.py? Probably, with an example directory in https://github.com/openvinotoolkit/datumaro/tree/develop/tests/assets/mot_dataset.

Done. I added a new assets folder without an img1 directory and with a seqinfo.ini file. I appended a test case in test_mot_format.py that checks if the import works and that all imported fields are as expected. The test fails if I remove my change to mot_format.py.

2. Please add a line in changelog.

Done.

Thank you for the contribution!


[docs] Adds extensions to docs web page

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Base: 72.08% // Head: 73.44% // Increases project coverage by +1.35% :tada:

Coverage data is based on head (3dcc2c8) compared to base (bb5073f).
Patch coverage: 73.86% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2284      +/-   ##
============================================
+ Coverage     72.08%   73.44%   +1.35%
- Complexity     5126     7240    +2114
============================================
  Files           473      774     +301
  Lines         21970    39646   +17676
  Branches       2351     3429    +1078
============================================
+ Hits          15838    29118   +13280
- Misses         4925     8959    +4034
- Partials       1207     1569     +362

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
59.21% <0.00%> (ø)

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

... and 713 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Develop a new chess activity

This issue is a challenge for students to the GSoC 2020. Every student interested to be candidate to GSoC 2020 Sugarizer projects must complete this task.

Develop a new chess activity for Sugarizer.

Detailed features expected:

[ ] The activity should be playable against computer or against another user on the network (using presence).
[ ] The activity should display the list of move during play.
[ ] The UI of the activity should be simple, self-explanatory and respect Sugar standards.
[ ] The activity should be correctly integrated into Sugarizer and use buddy colors when possible.
[ ] The activity should be integrated with Journal (save its context in the Journal).
[ ] The activity should use localization.
[ ] The activity should include a tutorial to explain the UI and an interactive tutorial to explain how each pieces move on a chessboard.
[ ] The activity should be responsive (work on any screen size) and should include a fullscreen button.
[ ] The activity should work on any browser (Chrome, Firefox and Safari) and in Electron.
[ ] The activity should not depend of an Internet connection (use local content only).
[ ] The activity should work with mouse, touch and possibly keyboard.
[ ] Every content/library used in the activity should rely on a Free Libre Open Source license.

Technically speaking, the activity could be written using pure JavaScript or using Vue.js inline (as in the Calligra activity). JavaScript ES6 syntax should be avoided.
The activity should be based on Chessboard JS for the UI  and  P4wn for the engine.

I've two doubts:

"pawn promotes to" and "computer level" feature should be there or we can skip them using their default value?
Computer playing for both sides should be allowed?

Pawn should promote on reaching the end like in normal chess game .
No I don't think we need computer vs computer . There should be computer vs Player or Player vs Player .

can we use libraries other than P4wn for the engine?

You could if it match the rule above. Specifically: it work on any browser, it's open source, it not depends from a server side, it don't use ES6 syntax and feature (Service Worker, ...).

@llaske I have submitted a PR #640 Please review....

Actually I was developing this activity and when I opened sugarizer I noticed this error.

As I haven't changed any internal file I don't know what's going wrong.

@Swayamsvk  could you elaborate more as to what exactly you did and if maybe possible share the code

Is this due to internet connection.
I am having this error in the terminal.

Is this due to internet connection.
No, sugarizer works offline.
Is this working fine in firefox with about:config as false?

It works fine on firefox. But showing problem on chrome.

@Swayamsvk I suggest you to clean your browser cache

@Swayamsvk try closing all tabs on your browser and then just running index.html from terminal

Thanks a lot @llaske it helped.

Few questions,

what if the user plays the game vs computer for a while and then shares the activity, what is the expected functionality here? Should the game continue in multiplayer from the same state or should it reset?
if one user exits from multiplayer mode, should the game reset or the other user should be able to continue it with the computer?

Yes the game will continue from same state . Although there will be reset button to restart game
Yes if the number of users reaches one , you should be playing against computer .

@ashish0910 thanks for the reply, I have another question related to the multiplayer mode, what happens when two people have already joined the activity and another person attempts to join it? Should the third person be not allowed to join? or maybe he could join but not do anything, just watch the game.

Yes the third person should be allowed to join the game as spectator . But if one of the user leave the game then the spectator can take place of the player

Thanks to all participants.
BTW we've decided that the winner of the challenge is https://github.com/llaske/sugarizer/pull/702
Anyone could propose improvements on the activity.

Fixed in https://github.com/llaske/sugarizer/pull/702


Docker 1 click deploy to digital ocean

Closed as completed. https://github.com/TransformerOptimus/SuperAGI/pull/1206


Math block display wrong

Have you checked that your issue isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] Yes, I have checked that this issue isn't already filed.

Bug description
Math Display Error
How to reproduce the bug

Error messages and logs
The error message you got, with the full traceback if available. Please paste it between these triple backticks.

What operating system are you using?
Mac
Where are you seeing the problem on?
Running locally with Docker (docker compose)
More info
\begin{equation}
\mathbf{H}=\phi\left(\mathbf{X} \mathbf{W}{\mathrm{xh}}+\mathbf{b}\mathrm{h}\right)
\end{equation}
this is not being displayed properly why?

I solved this in html it interpret as  for the underscore, it maybe a bug
solved by _


itest: fix payment test error message

Minor bug fix to payments test to include correct object in test fail message.
Pull Request Checklist

[x] All changes are Go version 1.15 compliant
[x] Your PR passes all CI checks. If a check cannot be passed for a justifiable reason, that reason must be stated in the commit message and PR description.
[X] If this is your first time contributing, we recommend you read the Code Contribution Guidelines

[x] The code being submitted is commented according to Code Documentation and Commenting
[x] Commits have a logical structure according to Ideal Git Commit Structure

[x] For new code: Code is accompanied by tests which exercise both the positive and negative (error paths) conditions (if applicable)
[x] For bug fixes: If possible, code is accompanied by new tests which trigger the bug being fixed to prevent regressions
[x] Any new logging statements use an appropriate subsystem and logging level
[x] For code and documentation: lines are wrapped at 80 characters (the tab character should be counted as 8 characters, not 4, as some IDEs do per default)

Very easy review! Anyone willing to take @guggero @Roasbeef @alexbosworth ?

Thank you @guggero , I have updated the release notes and included in the PR

Thanks! Though it seems now you've included a merge commit in your PR. Can you rebase it please to get rid of that?

Thanks @guggero ... did rebase, resolved conflict, and I think I've cleaned it up ... let me know if I am mistaken, appreciate your help.


⚠ ClamAV has degraded performance

In aa44ca3, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 563ddef.


[TASK] Integrate back-end with Shielded Tables in Hubble Stats

Implement functionality to dynamically fetch and display the data for the Shielded Tables component on Hubble Stats.

[ ] Shielded Asset Table

[ ] Shielded Pool Table

also utilizing onchaindata like #1445
get tvl and deposit data from vanchor-client lib
update logic to handle suported chains


Configure MySQL integration for log testing

What does this PR do?

Update the MySQL E2E setup (dd_environment fixture) so that logs collection is enabled and configured without extra manual setup.
TODO:

[x] Configure the error log (only general and slow query logs are configured for now).
[x] Update mariadb.yaml / mariadb.conf.
[ ] Resolve failing tests.
[ ] mysql8.yaml is missing changes from #5186 - need to fix this and then update mysql8.yaml here as well.

Motivation

As part of validating MySQL slow log support (see https://github.com/DataDog/logs-backend/pull/6293), I needed to send MySQL logs to staging, which required extra manual configuration and changes.
This PR addresses the "test on staging" use case, but these changes can also be helpful in other situations.
Additional Notes

Logs won't be collected nor sent to the logs intake by default (what we want in most cases), despite the configuration added here. Indeed, log collection on the Agent is off by default: we need to explicitly pass -e DD_LOGS_ENABLED=true to ddev env start for the Agent to actually collect logs.
Information on how to do test logs integrations on Staging in general is being gathered here.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Tests are failing on 5.6/5.7 (dd_environment can't connect to the MySQL server). Opened https://github.com/DataDog/integrations-core/pull/5222 to reduce the size of changes introduced here, hoping this helps pinpoint the issue.


API design of cnb_runtime causes clippy::unnecessary_wraps lint warnings

If a buildpack that's enabled the pedantic clippy::unnecessary_wraps lint has a detect handler that never returns an error (for example since it uses std::fs::Path::exists() due to try_exists() being nightly-only), then a lint warning is reported that cannot be resolved other than allowing it:
pub fn detect(context: GenericDetectContext) -> Result<DetectOutcome, BuildpackError> {
    let procfile_path = context.app_dir.join("Procfile");

    if procfile_path.exists() {
        let buildplan = BuildPlan::new();
        Ok(DetectOutcome::Pass(buildplan))
    } else {
        Ok(DetectOutcome::Fail)
    }
}

$ cargo clippy
warning: this function's return value is unnecessarily wrapped by `Result`
  --> src/detect.rs:5:1
   |
5  | / pub fn detect(context: GenericDetectContext) -> Result<DetectOutcome, BuildpackError> {
6  | |     let procfile_path = context.app_dir.join("Procfile");
7  | |
8  | |     if procfile_path.exists() {
...  |
13 | |     }
14 | | }
   | |_^
   |
note: the lint level is defined here
  --> src/main.rs:5:9
   |
5  | #![warn(clippy::pedantic)]
   |         ^^^^^^^^^^^^^^^^
   = note: `#[warn(clippy::unnecessary_wraps)]` implied by `#[warn(clippy::pedantic)]`
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_wraps
help: remove `Result` from the return type...
   |
5  | pub fn detect(context: GenericDetectContext) -> libcnb::DetectOutcome {
   |                                                 ^^^^^^^^^^^^^^^^^^^^^
help: ...and then change returning expressions
   |
10 |         DetectOutcome::Pass(buildplan)
11 |     } else {
12 |         DetectOutcome::Fail
   |

It seems like perhaps we should be using Traits here to make the API with cnb_runtime more explicit, and so Clippy can tell we're implementing a known interface?

This has been addressed with #129.


Missing 2nd Argument

ErrorException in CascadeDeleteListener.php line 18:
Missing argument 2 for Askedio\SoftCascade\Listeners\CascadeDeleteListener::handle()
I'm getting the error above when deleting a model.   In debugging, I've removed all the items in the $softCascade array, so I don't think it's a Model code issue, but who knows.
I'm Laravel 5.3.  Need more info?

See #42
To use with Laravel 5.3 yo need to use Askedio/laravel5-soft-cascade version 1.8.

That worked perfectly sorry for the duplicated error.  I didn't see how to review closed issues, but now I do.  Thanks for the uber quick response!

You're welcome and thanks!!


"Printing to Screen" contains a funny character

At https://github.com/phil-opp/blog_os/blob/gh-pages/printing-to-screen.html#L250,
Should be repr(C), but appears in HTML as repr©

Thanks for reporting! I've migrated to a new static site generator today, which uses the blackfriday markdown renderer. It seems like it has a "feature" to automatically convert (C) to ©. I'll take a closer look tomorrow…

Note to myself:
https://gohugo.io/overview/configuration#configure-blackfriday-rendering
and
https://github.com/russross/blackfriday/blob/b43df972fb5fdf3af8d2e90f38a69d374fe26dd0/smartypants.go#L152
seem relevant.


Cannot use 'className' as a prop in components

Describe the bug
A common pattern when using TailwindCSS is to accept a className prop and merge it inside a component with existing styles to allow controlling the styles of a reusable component.
Using this library with the nativewind/babel plugin makes me unable to add a prop named className to my components: no value is passed to the component unless I rename the prop to something else.
To Reproduce
Steps to reproduce the behavior:

Create a component that uses NativeWind
Give it a prop named className
Try to use the value of className inside the component

Reproducible example:
Example component:
import { View, Text } from "react-native";

export default function MyComponent({ className }: { className: string }) {
  return (
    <View>
      <Text>Your classname is: {className}</Text>
    </View>
  );
}

<MyComponent className="bg-red-400"/>

👆 This will not render className, unless we rename that prop to something else.
Expected behavior
The value passed to className should be accessible, but NativeWind seems to prevent that from happening.
Additional context
Renaming the prop to tw also doesn't work.

Upon reading the docs, it seems this is a known limitation of the library.
https://www.nativewind.dev/guides/custom-components#passing-styles-to-components


feature request: adding an option to disable dial on creating client

When we initialize a redis client with go-redis, it does not return any error. I think this is a very good feature. Imagine I use redis in my web service only as a cache. If it goes down, it is still ok and I can use the main repository for fetching my data. If it goes up during the service runtime, I will use that cache again. But this package returns nil and error if it cannot connect to redis on initialization and I don't want to interrupt my service start up because my redis server is out of reach.

Hi @mehdipourfar,
If redis servers go down after the initialization, rueidis will try reconnection automatically and can work as you want.
rueidis has a good reason to return an error at initialization, allowing users to verify whether their config for establishing a Redis connection is correct or not. I generally recommend users to fail fast and fix fast.
Indeed, this approach will interrupt service startup if their config is correct but Redis server are not ready. However, there is another purpose of initialization which I think is the true blocker to your request: choosing the correct client implementation.
Currently, rueidis will connect to the server to check if it is a Redis cluster. If yes, then the initialization returns a cluster client implementation, otherwise returns a standalone client implementation.
If we let this behavior be configurable, then we can return a concrete client implementation instead of a nil even when the initialization has failed.

Thanks for your reply. What happens if we try to not Dial in newSentinelClient and newSingleClient?

You will have no chance to know whether your config is wrong or not before sending your first redis request.

I think we should keep the Dail() even if we make the auto-detection configurable. Users can just ignore the error returned by the initialization if they don't care about it.

That would solve my problem if we don't return nil as client.

Hey @rueian, can we clarify the description and expected change of this task for a potential implementation?

Hi @erdemtuna,
The goal is to make the rueidis.NewClient return a concrete client implementation even if there is an error occurs. However, which concrete client implementation to use is currently queried from an online redis instance. This is the first blocker that we are unable to return a concrete implementation. We need a method to let users specify which implementation to use explicitly.
Since we already have the option ForceSingleClient, a potential implementation starts from adding a new one ForceClusterClient:

When ForceClusterClient is true, let rueidis.NewClient return a clusterClient even if there is an error occurs.
When ForceSingleClient is true, let rueidis.NewClient return a singleClient even if there is an error occurs.
When Sentinel.MasterSet is set, let rueidis.NewClient return a sentinelClient even if there is an error occurs.
Make sure all the above clients can work after redis is back.

Thanks @rueian, I will work on the issue and communicate wherever necessary.

Hi @rueian, I came up with these edits as a basic try however, I cannot make them work in the following scenario:

Create a client when the cluster is down.
Start the cluster.
Run a redis command with the client.

I seem to be stuck as of now. If someone could guide me, I would appreciate it. Otherwise, I should stop working on the issue.
https://github.com/redis/rueidis/commit/5053565ef72de657d0d945bf33e19805ec553d06

@erdemtuna Would you like to continue on this otherwise I can pick it up
// @rueian


change raw.github.com to raw.githubusercontent.com

because that's how you get raw files from github now.

Thanks!


Add some DNS debugging and random load balancing

Troubleshooting here some server-side connection drops. This adds some debug information on the IPs that are being used. Also randomizes (probably unnecessary) the IP address used.

Closing this one as we don't need it any longer.


Set Up Your SAP Cloud Platform Integration Tenant

Tutorial URL: https://developers.sap.com/tutorials/cp-starter-integration-cpi-onboard-subscribe.html
Issue : In trial US region
-In Step 4  Provision SAP Cloud Platform Integration Tenant : on 4th instruction screenshot is showing Provision button, but in application there is no Provision button instead there is "Activate" button.

Hi Mounika,
Thanks for the update. its already in our road map and we are working towards adopting the screenshots of these tutorials to the latest one.
You can see these changes in the coming days.
I hope it will suffice your concern.
Regards,
Vikram Kulkarni

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


empty initial conditions

The commit 0348a71f1cac9babe82cf23ae5a10c4345ebe14f (PR https://github.com/VulcanClimateModeling/fv3config/pull/104) seems to have removed support for
initial_conditions: []
when making the run directory. The error is:
Traceback (most recent call last):
File "venv/bin/write_run_directory", line 8, in
sys.exit(write_run_directory())
File "venv/lib/python3.7/site-packages/fv3config/cli.py", line 49, in write_run_directory
fv3config.write_run_directory(config, args.rundir)
File venv/lib/python3.7/site-packages/fv3config/config/rundir.py", line 17, in write_run_directory
current_date = get_current_date(config)
File "venv/lib/python3.7/site-packages/fv3config/config/derive.py", line 64, in get_current_date
coupler_res_filename = os.path.join(config["initial_conditions"], "coupler.res")
File "/usr/local/opt/python/bin/../Frameworks/Python.framework/Versions/3.7/lib/python3.7/posixpath.py", line 80, in join
a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not list
make: *** [setup_rundir] Error 1
It does work to make the run directory with
initial_conditions: ""
So the choice is to add back support for [], or update the repos currently using older versions of fv3config that contain config files with empty initial conditions specified.

This will be resolved by #107


Question about Model & add/get States

I have made a model for my entity which implements ModelInterface and ModelStateInterface
My model :
<?php
namespace myProject\myBundle\WorkFlow\Model;

use Lexik\Bundle\WorkflowBundle\Model\ModelInterface;
use Lexik\Bundle\WorkflowBundle\Model\ModelStateInterface;
use Lexik\Bundle\WorkflowBundle\Entity\ModelState;
use myProject\myBundle\Entity\myEntity;

class myModel implements ModelInterface, ModelStateInterface
{
    ...
    public function addState(ModelState $modelState)
    {
        $this->myEntity->setStates[] = $modelState;
    }

    public function getStates()
    {
        return $this->myEntity->getStates;
    }
}

Entity side added parameter (getter and setters by doctrine) :
    /**
     * @ORM\Column(nullable=true, name="States", type="array")
     */
    protected $states = array();

    /**
     * @ORM\Column(nullable=true, name="Status", type="smallint")
     */
    private $Status;

In my controller i do :
$em = $this->getDoctrine()->getManager();
$em->getConnection()->beginTransaction();
try {
    // persist 1st my entity to get the entity id
    $em->persist($myEntity);
    $em->flush();

    // next create modelstate and update status in my entity
    $myEntityModel = new MyModel($myEntity);
    $processHandler = $this->get('lexik_workflow.handler.myprocess');
    $ModelState = $processHandler->start($myEntityModel)
    $em->flush();
    $em->getConnection()->commit();

} catch (WorkflowException $e) {
    $em->getConnection()->rollback();
    $em->close();
}

I have to implement addStates and getStates into the model or directly into the entity ?

Is it the best way in my controller to try / catch and reverse if WorkFlowException ?

My entity is ALWAYS saved in DB with States => empty array, why ?

    // This code doesn't work :(
    $myEntityModel->addState($ModelState)

Why there is an objects option into getStates, we have to do it from the entity model, no ?

Sorry for my bad english !

Hi, you have to implement ModelStateInterface on the same class that implements ModelInterface. If you want to get states for a given model don't forget to call
$container->get('lexik_workflow.model_storage')->setStates($model); this will fill the object with its states, then you can call $model->getStates(). This is used to read states from th DB and display them, you don't need to map the $states property on your Doctrine model.

Thanks for you'r answer, solved my problem +1: