Pubsub protocol returns empty error when being cancelled

What's happening?
When a pubsub protocol is stopped by cancelling its context, it still returns an empty error.
Expected behavior
When intentionally stopped, the pubsub protocol shouldn't return error at all.
Fix
https://github.com/cloudevents/sdk-go/blob/6ec31e9557787a676442efb2416ebce153f6f8b3/v2/protocol/pubsub/protocol.go#L236
If errs is nil, if should return nil rather than errors.New.

@yolocs can you also check out this PR please? https://github.com/cloudevents/sdk-go/pull/458
I reworked all Close lifecycles to be consistent between them except pubsub :smile:

@slinkydeveloper ack.  Could you help create an issue and assign it me?


Fix for systems that use Python 3 as default, again

This was broken very recently due to you switching to that other Find.cmake module.

Sorry about that.


Depth_image cannot be refreshed properly

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D4345

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win10

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.4.8

Language
python

Segment
others

Issue Description
I had some problems running the code in librealsense/wrappers/python/examples/opencv_viewer_example.py
Question
1、The depth_image does not refresh when its FPS is 30.
config.enable_stream(rs.stream.depth, 640, 480, rs.format.z16, 30)
2、The program reported an error when the depth_image FPS was 10.**
Traceback (most recent call last):
File "F:/Larry/Pro/RealSenseD435/Learn/2opencv_viewer.py", line 46, in
pipeline.start(config) # 以config的配置开始采集数据
RuntimeError: Couldn't resolve requests
The source code is as follows
import pyrealsense2 as rs
import numpy as np
import cv2

pipeline = rs.pipeline()
config = rs.config()

pipeline_wrapper = rs.pipeline_wrapper(pipeline)
pipeline_profile = config.resolve(pipeline_wrapper)
device = pipeline_profile.get_device()
device_product_line = str(device.get_info(rs.camera_info.product_line))

found_rgb = False
for s in device.sensors:
    if s.get_info(rs.camera_info.name) == 'RGB Camera':
        found_rgb = True
        break
if not found_rgb:
    print("The demo requires Depth camera with Color sensor")
    exit(0)

config.enable_stream(rs.stream.depth, 640, 480, rs.format.z16, 30)

if device_product_line == 'L500':
    config.enable_stream(rs.stream.color, 960, 540, rs.format.bgr8, 30)
else:
    config.enable_stream(rs.stream.color, 640, 480, rs.format.bgr8, 30)

pipeline.start(config)

try:
    while True:

        frames = pipeline.wait_for_frames()
        depth_frame = frames.get_depth_frame()
        color_frame = frames.get_color_frame()
        if not depth_frame or not color_frame:
            continue

        depth_image = np.asanyarray(depth_frame.get_data())
        color_image = np.asanyarray(color_frame.get_data())

        depth_colormap = cv2.applyColorMap(cv2.convertScaleAbs(depth_image, alpha=0.03), cv2.COLORMAP_JET)

        depth_colormap_dim = depth_colormap.shape
        color_colormap_dim = color_image.shape

        if depth_colormap_dim != color_colormap_dim:
            resized_color_image = cv2.resize(color_image, dsize=(depth_colormap_dim[1], depth_colormap_dim[0]), interpolation=cv2.INTER_AREA)
            images = np.hstack((resized_color_image, depth_colormap))
        else:
            images = np.hstack((color_image, depth_colormap))

        cv2.namedWindow('RealSense', cv2.WINDOW_AUTOSIZE)
        cv2.imshow('RealSense', images)
        cv2.waitKey(1)

finally:

    pipeline.stop()

Hi @Larry0607  I cannot see an obvious reason in the script why depth would stop updating at 30 FPS, as it is just the same script as the original opencv_viewer.example.py without the comment lines.
In regard to the Couldn't resolve requests error: these occur because a requested stream configuration either was not currently supported at the time of running the program or there was a typing error that made the line unable to be understood by the SDK.  In this partcular case I would recommend using 6 or 15 FPS at 648x480 instead of 10, as 10 is not supported fo that resolution.

Hi @Larry0607 I cannot see an obvious reason in the script why depth would stop updating at 30 FPS, as it is just the same script as the original opencv_viewer.example.py without the comment lines.
In regard to the Couldn't resolve requests error: these occur because a requested stream configuration either was not currently supported at the time of running the program or there was a typing error that made the line unable to be understood by the SDK. In this partcular case I would recommend using 6 or 15 FPS at 648x480 instead of 10, as 10 is not supported for that resolution.

Thank you for your answer！The depth_image can be refreshed when I set 15 FPS at 640×480.However, the depth_image is still static when I set 30 FPS at 640×480.

What is your Windows PC's CPU usage percentage like when depth is set to 30 FPS?  You can find out the usage by using the Ctrl-Alt-Delete keyboard shortcut to access the Task Manager interface and left-clicking on the Performance category.  Then view the CPU usage in real-time whilst your Python program is running.

Hi @Larry0607  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


iter_archive for zip files

In this PR, I added the option to iterate through zipfiles for download_manager.py only.
Next PR will be the same applied to streaming_download_manager.py.
Related issue #3272.

Comments :

There is no .isreg() equivalent in zipfile library to check if file is Regular so I used .is_dir() instead to skip directories.
For now I got streaming_download_manager.py  working for local zip files, but not for urls. I get the following error when I test it on an archive in google drive, so still working on it.  BlockSizeError: Got more bytes so far (>2112) than requested (22)

Tasks :

[x] download_manager.py
[ ] streaming_download_manager.py

And also don't always try streaming with Google Drive - it can have issues because of how Google Drive works (with quotas, restrictions, etc.) and it can indeed cause BlockSizeError.
Feel free to host your test data elsewhere, such as in a dataset repository on https://huggingface.co (see here for a tutorial on how to upload files)


KeyError: 'vd'

KeyErroe:'vd' occurs when I run cliptext_extract_features.py. Can anyone help me fix this

Hello, have you run this script file successfully? I'm having various network issues
OSError: We couldn't connect to 'https://huggingface.co' to load this file, couldn't find it in the cached files and it looks like openai/clip-vit-large-patch14 is not the path to a directory containing a file named config.json.
Checkout your internet connection or see how to run the library in offline mode at 'https://huggingface.co/docs/transformers/installation#offline-mode'.

I didn't encounter this problem. You can check whether the pretrained models are placed in the correct location.

Also please check if you have the right version of the versatile diffusion model as there have been some critical changes in the past weeks which I didn't adapted brain-diffuser to


feat: added pagination and filter for addresses and contacts

[x]  added pagination and filter for addresses and contacts
[x] moved address and contact link and delete customizations to frappe

@hrwX , @DeveloperMetal , @nehasacher143 I need to move the search Contact and Search address field above the listing, can you guys suggest me something?

@MyuddinKhatri What's up with this PR? Also fix Sider issues

@MyuddinKhatri What's up with this PR? Also fix Sider issues

need to change some css here

@hrwX and @sahil28297 changes are done please have a look.

@MyuddinKhatri Fix sider issues.


Swift Playgrounds

Describe the issue
livecontainer crashes during installation process.
Instructions to reproduce
https://armconverter.com/decryptedappstore/us/swift playgrounds
What version of LiveContainer are you using?
3.0.0-release (e5cf20b)
Other
No response

Swift Playgrounds is unsupported. Even if you get it to install, it won’t work at all. It has a lot of private entitlements.
There are SparseRestore-based tools that lets you install it with private entitlements, but your code wont run on newer iOS versions (17.x?)


Refactor spo group set to use util instead of calling other command, Closes #5288

Closes #5288

Thanks for another awesome PR 💪👍. You are on 🔥 today. We will review it ASAP

setting to draft due to the following observations
I will provide more guidance how we should proceed with this PR in near future


Fail to train a GCCN with a code example in the README.md file ?

When I clone the code to my local environment(Windows) and try to run the code example of "Defining 和 training a GCCN", I get some error and I don't know how to fix it out. Can anyone give me some advise?

And I also fail to reproduce experiments in "scripts/topotune/existing_models" with an error "In 'hydra/config': Could not find 'hydra/job_logging/colorlog'" (Is it a problem with my environment configuration?)

Hi @JieCoa ! Thanks for reaching out and pointing this out. Actually, the scripts to reproduce TopoTune got outdated after the last changes that we applied to the library.
Our last PR (https://github.com/geometric-intelligence/TopoBenchmark/pull/124) addresses this, and now the scripts are compatible with the current code. For instance, you could run
python -m topobenchmark  model=cell/topotune  model.tune_gnn=GCN  model.backbone.GNN.num_layers=1  model.backbone.neighborhoods=\[1-up_incidence-0,1-up_adjacency-1,1-down_incidence-2\]  dataset=graph/MUTAG  optimizer.parameters.lr=0.001  model.feature_encoder.out_channels=128  model.backbone.layers=2  model.readout.readout_name=PropagateSignalDown  model.feature_encoder.proj_dropout=0.25  dataset.dataloader_params.batch_size=32  transforms.graph2cell_lifting.max_cell_length=10  dataset.split_params.data_seed=0,3,5,7,9  trainer.max_epochs=500  trainer.min_epochs=50  trainer.check_val_every_n_epoch=5  callbacks.early_stopping.patience=10
Please let us know if you still have some issues reproducing the experiments: probably then it would be related to an environment configuration, and we could take a closer look.

Thanks for your reply, but I still have some questions about the model.backbone.neighborhoods  configuration. I used the script you provided with --multirun to test. And I got an unexpected error in the follow image.

Then I test the script in "Defining and training a GCCN".
python -m topobenchmark  dataset=graph/PROTEINS  dataset.split_params.data_seed=1  model=cell/topotune  model.tune_gnn=GCN  model.backbone.GNN.num_layers=2  model.backbone.neighborhoods=\[1-up_laplacian-0,1-down_incidence-2\]  model.backbone.layers=4  model.feature_encoder.out_channels=32  model.feature_encoder.proj_dropout=0.3  model.readout.readout_name=PropagateSignalDown  logger.wandb.project=TopoTune_cell  trainer.max_epochs=1000  callbacks.early_stopping.patience=50
A different but somewhat similar error to the one mentioned above occurred.

I thought the error was related to the \ symbol in model.backbone.neighborhoods=\[1-up_laplacian-0,1-down_incidence-2\] , so I removed all the \ and then the code ran successfully.
python -m topobenchmark  dataset=graph/PROTEINS  dataset.split_params.data_seed=1  model=cell/topotune  model.tune_gnn=GCN  model.backbone.GNN.num_layers=2  model.backbone.neighborhoods=[1-up_laplacian-0,1-down_incidence-2]  model.backbone.layers=4  model.feature_encoder.out_channels=32  model.feature_encoder.proj_dropout=0.3  model.readout.readout_name=PropagateSignalDown  logger.wandb.project=TopoTune_cell  trainer.max_epochs=1000  callbacks.early_stopping.patience=50

I don't know what the situation is like in your area, but this is my current situation. If I have misunderstood, I hope to get your reply.


Can't use enum (TypeScript) as type identifier

I want to use enum for defining action types，such as
enum ActionTypes {
  ADD,
  DELETE
}

const add = createAction(ActionTypes.ADD, item => item)
const reducer = handleActions({
  [ActionTypes.ADD](state, item) {
    return [...state, item]
  }
}, initialState)

After compile, ActionTypes.ADD will transform to number 0.
But the key of handlers will become string '0'.
Because '0' !== 0, the demo above will not work.

what about
const ActionTypes = {
  ADD: 'ADD',
  DELETE: 'DELETE'
}

@alexgorbatchev
👍 Good idea


Is it currently possible to synchronize more than one variable

I am trying to replace my Node Red with ioBroker. Hence the question, is it currently possible to synchronize more than one variable between iobroker and Twincat?
I only found the input for a variable.

?

Oh sorry for the late response ... i don't get a notification from Github ... Yes for sure its possible ... The input is not for a Variable ... in TwinCat 3 it's for a Global Variable Table and in TwinCat 2 its for a global Struct

The Adapter syncronises all Variables automatic in the ioBroker (Complete Tree) but only when all Variables inside are supported ... See readme

So only one variable that contains a structure can be registered.
.VarStruct1
.VarStruct2
it does not work.

Yes that is true

What do you think if I convert the configuration to tab and install a tab in which you can specify more than one variable table?
The currently maintained variable could be transferred to the first line. I would implement it and make a merge request.
It could look like this:

That's a good idea ... It's would be a good feature and a good improvement for the Adapter to use more than one Struct/V-Table

I would be happy when u implement this ... Looking forward for your merge request

Hi :-) i work with your adapter and for me it would be great, if the adapter get this update.
I have a lot variables i have to read from existing PLCs, and i dont want to change software in every PLC and integrate a interface vartable.
it would be great, if i can define the needed variables in iobroker, without touching the plc

@PLCHome ... Are u working on that? Or is this issue obsolete?

Close because stale


dusk is looking for a new maintainer

https://github.com/C2SM/spack-c2sm/blob/03bf4aa20ef791b4a565a707441382584a4bf8f6/repos/c2sm/packages/dusk/package.py#L15

@cosunae is this package still needed?

yes, we should retire this, is not developed anymore.
thanks


PINURLSessionManager Crash on iOS 9

EXC_BAD_ACCESS KERN_INVALID_ADDRESS on
PINURLSessionManager.m line 76
-[PINURLSessionManager URLSession:task:didReceiveChallenge:completionHandler:]
here is the stack trace
Thread : com.apple.main-thread
0  libsystem_kernel.dylib<PHONE_NUMBER> mach_msg_trap + 8
1  libsystem_kernel.dylib<PHONE_NUMBER> mach_msg + 72
2  CoreFoundation<PHONE_NUMBER> __CFRunLoopServiceMachPort + 196
3  CoreFoundation<PHONE_NUMBER> __CFRunLoopRun + 1032
4  CoreFoundation<PHONE_NUMBER> CFRunLoopRunSpecific + 384
5  GraphicsServices<PHONE_NUMBER> GSEventRunModal + 180
6  UIKit<PHONE_NUMBER> UIApplicationMain + 204
7  App<PHONE_NUMBER> main (main.m:16)
8  libdyld.dylib<PHONE_NUMBER> start + 4

Hmm, that stack trace doesn't give us much to work with. Are you sure that's the crashing thread?
Can you reproduce the issue?

I can't reproduce the issue, I'm just seeing the crashes in Crashlytics. I've attached another stack trace if it helps? The issue mainly affects iOS 9.1.0 users
Crashed: NSOperationQueue 0x145bc5b0 :: NSOperation 0x15c09a90 (QOS: LEGACY)
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000000000030

Thread : Crashed: NSOperationQueue 0x145bc5b0 :: NSOperation 0x15c09a90 (QOS: LEGACY)
0  libdispatch.dylib              943204326 dispatch_async$VARIANT$mp + 65
1  libsystem_blocks.dylib         943577435 _Block_copy_internal + 298
2  AppName                        4161421 -[PINURLSessionManager URLSession:task:didReceiveChallenge:completionHandler:] (PINURLSessionManager.m:76)
3  CFNetwork                      639487947 __68-[NSURLSession delegate_task:didReceiveChallenge:completionHandler:]_block_invoke + 38
4  Foundation                     661201801 __NSBLOCKOPERATION_IS_CALLING_OUT_TO_A_BLOCK__ + 8
5  Foundation                     660553075 -[NSBlockOperation main] + 146
6  Foundation                     660496409 -[__NSOperationInternal _start:] + 768
7  Foundation                     661210689 __NSOQSchedule_f + 192
8  libdispatch.dylib              943212523 _dispatch_queue_drain$VARIANT$mp + 1758
9  libdispatch.dylib              943210025 _dispatch_queue_invoke$VARIANT$mp + 284
10 libdispatch.dylib              943220005 _dispatch_root_queue_drain + 396
11 libdispatch.dylib              943219605 _dispatch_worker_thread3 + 96
12 libsystem_pthread.dylib        944970537 _pthread_wqthread + 1024
13 libsystem_pthread.dylib        944969496 start_wqthread + 8

This is being tracked in #92


Title: gRPC instead of REST?

The title says REST and gRPC but REST is only mentioned in the intro and the rest (pardon the pun) is gRPC. I think the title could better reflect this.

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

ID: dc86015f-c6c7-5fdc-ae0f-c69353afcb34
Version Independent ID: 491d7006-ae8d-fb74-8b29-7e4b5031fcae
Content: REST and gRPC
Content Source: docs/architecture/cloud-native/rest-grpc.md
Product: dotnet-architecture
Technology: cloud-native
GitHub Login: @robvet
Microsoft Alias: nanil

Hey Jeremy.
Excellent feedback. We are making the change.
Would love any feedback from you.
Rob Vettor  | Principal Cloud-Native Application Architect
| ASD IP Architect
| CNCF (Cloud-Native Computing Foundation) Speaker Bureauhttps://www.cncf.io/speaker/robvet/
| Developer Advisory Services for Azure
<EMAIL_ADDRESS>[Description<EMAIL_ADDRESS>214-707-0584
[Description<EMAIL_ADDRESS> robvet<EMAIL_ADDRESS><EMAIL_ADDRESS>www.thinkingincloudnative.comhttps://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.thinkingincloudnative.com%2F&data=02|01|robvet%40microsoft.com|a0ad6989676c41a1c02208d7659ff0d4|72f988bf86f141af91ab2d7cd011db47|1|0|637089609919099893&sdata=Ih3VA8PbdEeYpmeYsH5jWzaoZX6Ys5Rwtpw3UlpdIVA%3D&reserved=0
Co-Author of…
[cid:image005.png@01D5FEE3.6C6F9160]https://docs.microsoft.com/en-us/dotnet/architecture/cloud-native/
From: Jeremy Likness<EMAIL_ADDRESS>Sent: Friday, March 20, 2020 4:47 PM
To: dotnet/docs<EMAIL_ADDRESS>Cc: Robert Vettor<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [dotnet/docs] Title: gRPC instead of REST? (#17532)
The title says REST and gRPC but REST is only mentioned in the intro and the rest (pardon the pun) is gRPC. I think the title could better reflect this.

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

ID: dc86015f-c6c7-5fdc-ae0f-c69353afcb34
Version Independent ID: 491d7006-ae8d-fb74-8b29-7e4b5031fcae
Content: REST and gRPChttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdotnet%2Farchitecture%2Fcloud-native%2Frest-grpc&data=02|01|robvet%40microsoft.com|58bd6ebd6374464a625108d7cd182f38|72f988bf86f141af91ab2d7cd011db47|1|0|637203376070454689&sdata=6kGOUSAsF%2BkSxluNdeeGhfe7AyAxfYqItDusoyzFNP4%3D&reserved=0
Content Source: docs/architecture/cloud-native/rest-grpc.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fblob%2Fmaster%2Fdocs%2Farchitecture%2Fcloud-native%2Frest-grpc.md&data=02|01|robvet%40microsoft.com|58bd6ebd6374464a625108d7cd182f38|72f988bf86f141af91ab2d7cd011db47|1|0|637203376070454689&sdata=YQgpR5qT2CfMpKB7nDlWsKsrfRQs1BwWKpj927PxivI%3D&reserved=0
Product: dotnet-architecture
Technology: cloud-native
GitHub Login: @robvethttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frobvet&data=02|01|robvet%40microsoft.com|58bd6ebd6374464a625108d7cd182f38|72f988bf86f141af91ab2d7cd011db47|1|0|637203376070464691&sdata=CPmrEpxYVx7CgBporb%2B6IuDOGqMEaouOMV%2BfjfgQi0Q%3D&reserved=0
Microsoft Alias: nanil

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fissues%2F17532&data=02|01|robvet%40microsoft.com|58bd6ebd6374464a625108d7cd182f38|72f988bf86f141af91ab2d7cd011db47|1|0|637203376070464691&sdata=Bz7%2Fvfbot8GHcTM45%2Bvuv3vwQCuAjq6klk30D8MP9uk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABFIQYG5QTI4E2PMU3XQEN3RIPP4JANCNFSM4LQVDMDA&data=02|01|robvet%40microsoft.com|58bd6ebd6374464a625108d7cd182f38|72f988bf86f141af91ab2d7cd011db47|1|0|637203376070464691&sdata=9jEzosnfe5H7tCp4nreI%2FQ48oqpRGxLilKhABwEzGBg%3D&reserved=0.


[icons] Refresh material icons

[x] I have followed (at least) the PR section of the contributing guide.

Google has updated quite a few of their Icons. More than a few actually.
This refresh will put @material-ui/icons in sync with https://material.io/resources/icons/.

@colemars Thanks for working on it.
There are 235 icons deleted which makes this a massive breaking change. (Some may have been renamed, but looking through index.js I don't see many possibilities.
CI is failing because icons used in the docs examples can't be found, so we would have to pick a suitable substitute for those.

@mbrookes yeah I saw that. Doesn't look like any were renamed - comparing against material.io it seems google has just removed some icons.
One solution to get the new icons in there would be to just keep all the deleted icons in there with the new/updated ones?


Fix issue with bundles dir volume

The bundles dir used to be automatically made a volume if BINDDIR was
not set.
The PKGCACHE_* stuff was added recently which gets added to
DOCKER_MOUNT. This breaks the case where someone wants to use the
package cache (DOCKER_INCREMENTAL_BINARY is enabled) but has not
specified a bind dir but needs the bundles dir made a volume.
Moves the package cache handling to after the bundles dir is made a
volume.
Before this patch, if you have DOCKER_GRAPHDRIVER set to anything
other than vfs, the tests in the daemon suite (and swarm suite, and
registry suite) will fail to run since they cannot startup a daemon
(except for speicifc cases where  over  is possible,
like devmapper).
Signed-off-by: Brian Goff<EMAIL_ADDRESS>

Error is from a flakey test

LGTM

LGTM 🐸
janky failure is unrelated and a known flakey test, merging :wink:


Custom field action arguments fail validation

This fails in both UIs
To reproduce:

load mockdoc.adm
change any index value
observe the backend server log output:

ref: processActionArguments -n mapping-serializer
https://github.com/atlasmap/atlasmap/blob/master/ui/src/app/lib/atlasmap-data-mapper/utils/mapping-serializer.ts#L344
javax.ws.rs.WebApplicationException: HTTP 400 Bad Request 	at io.atlasmap.service.AtlasService.fromJson(AtlasService.java:1022) ~[atlas-service-1.44.0-SNAPSHOT.jar:1.44.0-SNAPSHOT] 	at io.atlasmap.service.AtlasService.validateMappingRequest(AtlasService.java:607) ~[atlas-service-1.44.0-SNAPSHOT.jar:1.44.0-SNAPSHOT]
Caused by: com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "paulsParam" (class io.atlasmap.v2.CustomAction), not marked as ignorable (5 known properties: "inputFieldType", "name", "methodName", "className", "outputFieldType"])  at [Source: UNKNOWN; line: -1, column: -1] (through reference chain: io.atlasmap.v2.AtlasMapping["mappings"]->io.atlasmap.v2.Mappings["mapping"]->java.util.ArrayList[1]->io.atlasmap.v2.Mapping["inputFieldGroup"]->io.atlasmap.v2.FieldGroup["field"]->java.util.ArrayList[0]->io.atlasmap.json.v2.JsonField["actions"]->io.atlasmap.v2.CustomAction["paulsParam"])

Fixed by https://github.com/atlasmap/atlasmap/commit/e9f72cf3f03c39cee8ee57db50dc9e199988af31

@pleacu is this fixed?

available in 1.44.0-M1 - I'll look at i1932 as soon as I can

Need to reverify this in conjunction with i1932

Hey @rkorytkowski @igarashitm - Sending to both Rafal and Tomo as both may have some insights.
I need some help defining custom field actions from the UI.  I'm using:
https://github.com/atlasmap/atlasmap/tree/master/lib/service/src/test/resources/upload/io/atlasmap/service/my/MyFieldActions.java
https://github.com/atlasmap/atlasmap/tree/master/lib/service/src/test/resources/upload/io/atlasmap/service/my/MyFieldActionsModel.java
to build a local MyFieldActions.jar.  I have a simple one-to-two mapping and then import this jar.  What's interesting about  MyFieldActions.jar is that the custom field action takes one argument.  My question is - what should the requesting JSON from the UI look like?
Option 1:
In this example - I lead with the parameter and generate an @type containing the custom class:
{"AtlasMapping":{"jsonType":"io.atlasmap.v2.AtlasMapping","dataSource":[{"jsonType":"io.atlasmap.json.v2.JsonDataSource","id":"s","uri":"atlas:json:s","dataSourceType":"SOURCE"},{"jsonType":"io.atlasmap.json.v2.JsonDataSource","id":"t","uri":"atlas:json:t","dataSourceType":"TARGET","template":null}],"mappings":{"mapping":[{"jsonType":"io.atlasmap.v2.Mapping","id":"mapping.698435","inputField":[{"jsonType":"io.atlasmap.json.v2.JsonField","name":"address","path":"/address","fieldType":"STRING","docId":"s","userCreated":false,
"actions":[{"delimiter":" ","@type":"Split"}]}],
"outputField":[{"jsonType":"io.atlasmap.json.v2.JsonField","name":"city","path":"/city","fieldType":"STRING","docId":"t","userCreated":false,"index":0,
Look here!! *****************************
"actions":[{"param":"car","@type":"io.atlasmap.service.my.MyFieldActionsModel","name":"io.atlasmap.service.my.MyFieldActionsModel","className":"io.atlasmap.service.my.MyFieldActionsModel","methodName":"myCustomFieldAction"}]},
{"jsonType":"io.atlasmap.json.v2.JsonField","name":"country","path":"/country","fieldType":"STRING","docId":"t","userCreated":false,"index":1}]}]},"name":"UI.0","lookupTables":{"lookupTable":[]},"constants":{"constant":[]},"properties":{"property":[]}}}
This results in the following error:
Caused by: com.fasterxml.jackson.databind.JsonMappingException: Invalid type id: io.atlasmap.service.my.MyFieldActionsModel (through reference chain: io.atlasmap.v2.AtlasMapping["mappings"]->io.atlasmap.v2.Mappings["mapping"]->java.util.ArrayList[0]->io.atlasmap.v2.Mapping["outputField"]->java.util.ArrayList[0]->io.atlasmap.json.v2.JsonField["actions"])
Option 2:
In this example I use CustomActio as the type but there's no place to put the parameter.
{"AtlasMapping":{"jsonType":"io.atlasmap.v2.AtlasMapping","dataSource":[{"jsonType":"io.atlasmap.json.v2.JsonDataSource","id":"s","uri":"atlas:json:s","dataSourceType":"SOURCE"},{"jsonType":"io.atlasmap.json.v2.JsonDataSource","id":"t","uri":"atlas:json:t","dataSourceType":"TARGET","template":null}],"mappings":{"mapping":[{"jsonType":"io.atlasmap.v2.Mapping","id":"mapping.698435","inputField":[{"jsonType":"io.atlasmap.json.v2.JsonField","name":"address","path":"/address","fieldType":"STRING","docId":"s","userCreated":false,
"actions":[{"delimiter":" ","@type":"Split"}]}],
"outputField":[{"jsonType":"io.atlasmap.json.v2.JsonField","name":"city","path":"/city","fieldType":"STRING","docId":"t","userCreated":false,"index":0,
Look here!! *****************************
"actions":[{"param":"car","@type":"CustomAction","name":"io.atlasmap.service.my.MyFieldActionsModel","className":"io.atlasmap.service.my.MyFieldActionsModel","methodName":"myCustomFieldAction"}]},
{"jsonType":"io.atlasmap.json.v2.JsonField","name":"country","path":"/country","fieldType":"STRING","docId":"t","userCreated":false,"index":1}]}]},"name":"UI.0","lookupTables":{"lookupTable":[]},"constants":{"constant":[]},"properties":{"property":[]}}}
This results in the following error:
Caused by: com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "param" (class io.atlasmap.v2.CustomAction), not marked as ignorable (5 known properties: "inputFieldType", "name", "methodName", "className", "outputFieldType"])
Note: there is no error generated if I don't pass the user parameter.
Q:  Should I use "@type":"io.atlasmap.service.my.MyFieldActionsModel" and if so what's wrong with the type id or should I use "@type":"CustomAction","name" and if so where does the parameter get passed?  Both appear to be supported.

IIRC option1 is the way to go
https://github.com/atlasmap/atlasmap/blob/master/lib/itests/reference-mappings/src/test/resources/javaToJava/atlasmapping-collection-fieldaction2.json#L27-L28
The error message sounds like you have id as a child of /mappings/mapping[0]/outputField[0]/actions? did you check mapping definition in the real payload UI is sending? should be visible via chrome devtool network tab.

@igarashitm - this is the real payload that's failing from the network tab:
actions: [{param: "c", @type: "io.atlasmap.service.my.MyFieldActionsModel"}]   0: {param: "c", @type: "io.atlasmap.service.my.MyFieldActionsModel"}     param: "c"     @type: "io.atlasmap.service.my.MyFieldActionsModel"
Error:
Caused by: java.lang.IllegalArgumentException: Invalid type id: io.atlasmap.service.my.MyFieldActionsModel
Looking at your example and looking at MyFieldActions.java, MyFieldActionsModel.java I also tried these two:
'{"@type":"io.atlasmap.service.my.MyFieldActions$MyFieldActionsModel","param":"car"}'
and
'{"@type":"io.atlasmap.service.my.MyFieldActions$io.atlasmap.service.my.MyFieldActions.MyFieldActionsModel","param":"car"}'
They generate the same backend error ( Invalid type id )

I'll look into later

Here's my latest attempt - similar to the example:
Source:

"actions":[{"@type":"io.atlasmap.service.my.MyFieldActions$Concur","separator":"+"}]},
Caused by: com.fasterxml.jackson.databind.JsonMappingException: Invalid type id: io.atlasmap.service.my.MyFieldActions$Concur (through reference chain: io.atlasmap.v2.AtlasMapping["mappings"]->io.atlasmap.v2.Mappings["mapping"]->java.util.ArrayList[0]->io.atlasmap.v2.Mapping["outputField"]->java.util.ArrayList[0]->io.atlasmap.json.v2.JsonField["actions"])


Add fit=fill feature for resize

When resizing an image which is smaller than the dimensions you want to
resize it to in one or both directions, the fit=fill option allows you
to specify a fill color (when combined with the Background manipulator)
to fill the extra canvas space.
This is accomplished using a query like ?w=300&fit=fill&bg=000 to resize
the image to 300 pixels wide while filling any new canvas space with
black.

Nice work Jeff, thanks for adding this!

:+1:


camera movement - right click & drag to circle around player and mousewhel to zoom in and out

still got the zooming to do @ddaydd  :)


https://registry-1.docker.io/v2/: net/http: request canceled

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Same problem with docker installation using Kitematic

Same Error here:

Step to reproduce:

Windows 10 Enterprise
Docker running in Program Files\Docker
Kitematic "Installed" in Program Files\Docker\Kitematic
Version of Kitematic: v1.8.8

context-canceled.
Why ? Are windows problems or blockers ?


Test casess added for ResourceGrp and ResourceID does not exist

Can one of the admins verify this patch?

ok to test


Add Travis CI Integration

Add a minimal Travis CI integration (run unit tests).

We will integrate soon the Travis support with some other features. Thanks @neokeld for your pull request.


Maybe it doesn't make sense to instal_db before a custom my.cnf is installed

Extending this image with a new my.cnf, means that the mysql_install_db needs to be repeated (depending on the changes to the conf.)
Should we be installing in this image?  Should we accept a base image that does not itself produce a "running container"?
For example, this image used to extend alpine-mariadb, but now just steals most of the Dockerfile, instead if re-creating the db.

I don't see any reason why we could not have this image as a base image that does not itself produce a "running container". So let's drop the mysql_install_db from this image?

@iler I will submit a PR


Added 'L' and NextRunTime

https://github.com/guycipher/gochrono

Thank you for your implementation. I’m gonna add these features in next release version.

It will be closed because of this issue is duplicated #12


Document docstring requirements with examples

Updates contribution guide with details on docstring requirements.
Change Summary

Reorganize "Documentation style" contribution guidelines with details on docstring style and linting.
Provide examples of correct docstrings
Add ignore_init_summary: true and merge_init_into_class: true configuration. This will enable correct documentation of class vs instance attributes -- instance attributes should be documented in __init__, and mkdocstrings will pull this into the class documentation. See https://mkdocstrings.github.io/python/usage/configuration/docstrings/#merge_init_into_class for background.

The merge_init_into_class: true configuration change does not affect API documentation currently, but enables documenting instance attributes correctly in future PRs.
Related issue number
Closes DOC-4
Closes DOC-5
Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Please review


Coverity 1216302,1022033,1022032,1022031: State While Revalidate

Stale While Revalidate Coverity bug fixes
CID 1216302 (#1 of 1): Unchecked return value (CHECKED_RETURN)
CID 1022033 (#1 of 1): Unchecked return value (CHECKED_RETURN)
CID 1022032 (#1 of 1): Unchecked return value (CHECKED_RETURN)
CID 1022031 (#1 of 1): Unchecked return value (CHECKED_RETURN)

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/526/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/536/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/523/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/652/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/2219/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/780/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/2115/

[approve ci]

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/541/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/528/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/2224/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/657/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/531/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/2120/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/785/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/532/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/542/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/529/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/658/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/2121/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/786/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2225/

clang format failed! https://ci.trafficserver.apache.org/job/clang-format-github/538/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/551/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/536/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/2128/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/665/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/2232/

clang-analyzer build failed! https://ci.trafficserver.apache.org/job/clang-analyzer-github/794/


Publish your library to npm

Hey,
First of all, great project.
I've been working on a library to display panels of emojies using your data.
As for now, using this library as peer-dependency (because in the future I'll make another libraries like emoji-input and such) and don't won't to include \ pack your data twice.
Peer-dependencies to github pages is not usual, better take it to npm.
Please consider

Similar to issue #21, the problem is that npm does not support packages that contain a ton of binary image data; even just the 64px sheets come out to 17MB

A package also already exists on npm called emoji-data, which includes this data file as a vendored dependency: https://www.npmjs.com/package/emoji-data
You can use this repo as a dependency by including the git URL in your own package.json

npm dependencies are just more clean and you can exclude some files (only the result spritesheets and the data is needed actually) from the registry, and speedup the installation process for my own library, same goes for bower.
Would really love seeing that!

Also your repo has a package json, and even though I installed your repo from github, it still errors (here's my build log) on package.json validation.

i'm not sure about the error you're seeing; sorry
i see you switched your dependency to download from github and that seems like a reasonable interim solution. i've been advised by npm to put the images as a tarball somewhere and have the package download these, rather than including them into the package itself, but that will require a bunch of work

Exactly, that what I did in my own project, though you can create another repo which will download the images on installation.
Again, that's work.
who knows I might find the time to do it myself

After PR #48 we're pretty close - just need to get the namespace now :+1:

Hey, how is it going?

it's stalled trying to figure out how to not break things for existing users of the emoji-data package. i guess a new name for the npm package makes sense. any suggestions?

Yeah from experience you'll have to get a new name.
emoji-db, emoji-source, emoji-datasource

done! https://www.npmjs.com/package/emoji-datasource

Great! testing usage right now.
You should update your README.md


Is the Company Communicator ready for the domain changes announced by Microsoft?

It's about the following post:
https://devblogs.microsoft.com/microsoft365dev/action-required-ensure-your-microsoft-teams-apps-are-ready-for-upcoming-domain-changes/
The post states that the following actions must be taken:

Upgrade Teams JS library to version 2.19 or higher.
Update your x-frame-options/CSP headers to allow for the new domains. To ensure the app keeps working across existing and future Microsoft 365 hosts, please ensure to allow “*.cloud.microsoft”.

Where should these actions be carried out in the Company Communicator solution, or how can I check if they have already been performed in an existing deployment?

Hi @neidantaslopez ,
Thanks for bringing it to our notice. We are already looking into this on priority. We will keep you posted on this.

Hi @neidantaslopez ,
We tested within new domain already. Currently this domain change has no impact on CC. We will let you know in future if there is any impact or any updates required

Is there no impact on any version of the CC or just the current version? How can I check this in my environment?

We have checked on the current version of CC.  To validate, change the URL to teams.cloud.microsoft.com and test the application


Fix automerge checks

Fixes #4305.

This will still fail beyond 100?

Yes, hopefully we won't have 100 checks, but yes, technically all the pageable queries should use paging in our automerge bot.


Backport 1325 to 2.2

Issue #, if available: https://github.com/opensearch-project/security-analytics/issues/718
Description of changes:
Manually resolves backport of #1325
CheckList:

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

CI failures look unrelated
* What went wrong:
Could not determine the dependencies of task ':alerting:integTest'.
> Server returned HTTP response code: 403 for URL: https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.2.2/6720/linux/x64/tar/builds/opensearch/plugins/opensearch-notifications-<IP_ADDRESS>.zip


TS-5087: Updates the AL2 license on all .po files

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1256/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1151/ for details.

:+1: You po' soul for having to track all those down. Looks good to me for merging if our translators agree.


Add issue templates and security policy

Labels automatically added via issue templates have been adjusted from their original name to match the names from cilium/cilium.

Multicluster test failed on #399 (though on no-policies/pod-to-service/curl-0, not allow-all/pod-to-service/curl-2): https://github.com/cilium/cilium-cli/runs/3547587561?check_suite_focus=true
This PR  improves issue reporting for contributors and users, thus merging.


Map connorshea to my full name

This was bugging me :P

Apparently we're doing this on GitLab.com now, nevermind!

Yep, thanks!


broken content-type serving files in sub directories

I'm using static-files with oak and I'm getting broken content-type headers for files in sub-directories:
$ curl -D - 'http://localhost:8000/test.js'
HTTP/1.1 200 OK
accept-ranges: bytes
content-type: application/javascript; charset=utf-8
etag: W/"10-1639076608438"
last-modified: Thu, 09 Dec 2021 19:03:28 GMT
content-length: 10
date: Thu, 09 Dec 2021 19:03:42 GMT

alert("h")

$ curl -D - 'http://localhost:8000/js/test.js'
HTTP/1.1 200 OK
accept-ranges: bytes
content-type: \js/test.js
etag: W/"10-1639076638723"
last-modified: Thu, 09 Dec 2021 19:03:58 GMT
content-length: 10
date: Thu, 09 Dec 2021 19:04:08 GMT

alert("h")

thanks for creating this issue. I will investigate your case first

@retog fixes in new version 1.1.3


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Zefau/ioBroker.nello

[ ] Icon must be in the following path: https://raw.githubusercontent.com/Zefau/ioBroker.nello/master/
[ ] Meta URL(latest) must be equal to https://raw.githubusercontent.com/Zefau/ioBroker.nello/master/io-package.json
[ ] Icon(stable) must be in the following path: https://raw.githubusercontent.com/Zefau/ioBroker.nello/master/
[ ] Meta URL(stable)  must be equal to https://raw.githubusercontent.com/Zefau/ioBroker.nello/master/io-package.json

Thanks,
your automatic adapter checker.

Thanks for fix #15

Thanks for fix #15


Add mini.files example on how to show git status as an extmark

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.files
Description
Hi!
This is less of a feature request and more of an inquiry.
The help page is very clear on git integration being out of scope, however it does hint that it's achievable by setting extmarks on the MiniFilesBufferUpdate event.
I hope some code snippet of this gets added to the wiki or the examples section in the help file.

Thanks for the suggestion!
The problem here is actually computing status of each path, adding extmarks would be somewhat straightforward (needs only the proper combination of options). In turn, getting information about Git status for every path and in a way that is at least somewhat efficient (at least non-blocking and not loading 100% CPU) is not something that can be expressed in a concise and usable manner. So I don't think it will be added.
Closing as not planned.

(There is planned 'mini.git' module for which I already have approximate scope and I also don't really see a concise way of utilizing it for this kind of task).


Regression in string concatenation with the item access

When trying to construct a new value by using a string variable plus suffix enclosed with [], the framework throws an error:

Variable '${the_var}' is string, not list or dictionary, and thus accessing item '1' from it is not possible.

In my code I use this for constructing xpath locators - either having a common root that's shared across a number of locators, or to append a predicate.
An example:
*** Variables ***
# the first type of error, constructing a variable by concatenating a preexisting one with a suffix
# error in the initialization
${LOC HOME CHART BOX INFO}                      xpath=//div/div/table[@class="c3-tooltip"]//tr
${LOC HOME CHART BOX DATE PLACEHOLDER}          ${LOC HOME CHART BOX INFO}[1]/th[text()="PLACEHOLDERselenium"]       # the first type of error, in the Variables section

# the second one - on this value a predicate is to be appended
${CLIENT DDL MENU OPTIONS}          xpath=//nav[contains(@class,"topnavbar")]//div[@class="Select-menu"]/div

*** Test Cases ***
A test case
    Log To Console      ${CLIENT DDL MENU OPTIONS}[1]     # the actual keyword doesn't matter (it's usually "Click Element")

Running this sample produces the following errors:

Error in file 'the_file.robot': Setting variable '${LOC_SEARCH_RESUTLS_TABLE_COLUMN_NAME_PLACEHOLDER}' failed: Variable '${LOC_SEARCH_RESULTS_TABLE_HEADING}' is string, not list or dictionary, and thus accessing item 'text()="PLACEHOLDERselenium" or .//*[text()="PLACEHOLDERselenium"' from it is not possible.

and

Variable '${CLIENT DDL MENU OPTIONS}' is string, not list or dictionary, and thus accessing item '1' from it is not possible.

This is a regression from 3.0.4, where this automatic concatenation syntax worked without issues. I have literally hundreds of constructs like this, on which I'm currently looking for a stable workaround (probably something involving xpath's self axis).

A solution would be to exclude during parsing string-like variables from item access (or, to limit it to dict and list derivatives); there won't be functionality loss - even though a code like "string 1"[1] is perfectly fine python idiom, Robot Framework doesn't support it (as obvious from the error messages), so the current state will be preserved.

This is caused by #2601 and explained in the release notes under backwards-incompatible changes:
https://github.com/robotframework/robotframework/blob/master/doc/releasenotes/rf-3.1.rst#id113

After re-reading I noticed you proposed limiting item access to lists and dicts. That, and just excluding access with strings, can be considered if there turn out to be a lot of problems caused by this. Escaping with \ is easy, but obviously somewhat annoying if you need to do it in many places.
I'll reopen this issue and we can discuss what to do.

Thanks for the workaround suggestion, I actually haven't thought of using escaping (\). Checked it, and the produced string is what I expect; that will take care of most of the occurrences of this issue.
Another workaround is to use whitespace - e.g. ${CLIENT DDL MENU OPTIONS} [1], which as it turns out is insignificant in this locations for xpath (I'll have to use this one in more complex situations, like concatenating two variables  - ${var_a}${var_b}, where the escape character is not usable). As my situation is a corner case (hopefully), I'll workaround it on my end as a change will not come immediately.

To summarize, for anyone that might face the same issue - the workarounds are either:

escape the [ character, e.g. ${variable}\[1]
for xpath, put an extra whitespace character b/n the variable and the [, it's of no difference for it - ${variable} [1]

As for excluding access in strings, here's a naive solution (disclaimer: haven't checked it fully for side effects, nor ran the framework's cases) - in variables/replacer.py, _get_variable():
138a139,142
>         # item access is not supported for string variables, return the string as-is
>         if is_string(variable):
>             return variable
>

And, looking at the code, doing the reverse - actually allowing item or slices access to string variables is trivial. Here's a sample:
https://github.com/tminakov/robotframework/commit/bc7b14ae04c52bf46338b003e7bb06e894c14b82
@pekkaklarck  if you find this valuable, and won't cause confusion to the users, we might work an adding it?

Could you clarify why would you need to use a space for escaping? Depending in the context it mat work, but you shouldn't need to use it.
Supporting item access with strings is definitely possible and could be useful. Don't want to so such changes in a minor version, so it would need to wait until RF 3.2, through. If we consider that, the current behavior that item access syntax with strings is an explicit error is actually a good thing.

I'm also hitting this issue and many test cases where I set XPaths started to fail when upgrading to RF 3.1.
IMO, this warrants a minor version fix (ex. 3.1.1) as I'm pretty sure this will break many other people using XPaths as well. Doing ${X}=    Set Variable    ${Xpath}[${i}] is pretty common.
Reverting to 3.0.4 until this is dealt with.  If the official solution is to escape, I'll update the tests then.

You are @tminakov right that ${var1}${var2} doesn't initiate item access even if ${var2} would look like [something]. Similarly it isn't considered named argument syntax even if ${var2} would contain =value. Such special syntax must always be used literally in test case level.
As I wrote earlier, supporting item access with strings is something that we definitely can consider in RF 3.2. We could actually try it with all values to make it possible to use this syntax with custom objects implementing __getitem__.

We were @joaonc aware of backwards-incompatibility problems when designing the new functionality involving this syntax. If you go through the discussion in #2601, we in the end considered the benefits bigger that problems. We didn't realize that syntax like ${xpath}[${predicate}] is so common, though.
I still think that the new item access syntax itself is useful. How to handle item access with strings is something we still can discuss. Personally I like it that this syntax always mean item access and escaping is needed if you want to use the literal syntax. That makes understanding the data easier, and also allows adding the actual item access syntax to strings later if we consider it a good idea.
The above said, if there are lot of reports about problems with the new syntax, we can consider changing it so that ${var}[whatever] isn't item access if ${var} is a string. There would need to be quite a bit more such reports, though. My current thinking is that escaping is easy enough that changes aren't needed. In other words, that's the current "official solution".

We also detect this problem:
xpath=(//body/${headingTag}[text()='${text}'])
And fixed using Catenate KW:
${heading xpath}=    Catenate    SEPARATOR=    //body/${headingTag}    [text()='${text}']

I had this problem after upgrading. Apparently it's mostly a matter of coding style; the only tests broken in our codebase were written by me.
Escaping is trivial and the affected rows are easy to find. No need for changes imo.

It seems backwards compatibility problems caused by #2601 aren't severe enough, especially when compared with benefits, that the functionality should be changed.
I also submitted #3182 about supporting ${var}[item] syntax with all objects, including strings.


What to do if we have over 20+ people?

Have a group of 3-6 people work in the one big table in the upper area.


Support for Lambda integration

The README mentions aws-sam-local uses Lambda proxy integration by default.  Is there a way to tell aws-sam-local to use Lambda integration instead?

No, SAM defaults to proxy integration when you define the APIs. We are keeping the behavior consistent here with SAM Local.

I switched to lambda proxy integration.  Lambda integration not really needed.

With SAM, it is possible to define your Api configuration using AWS::Serverless::Api. When you then define x-amazon-apigateway-integration, you can choose to have type "aws" or "aws_proxy". Shouldn't this "aws" type option be supported by sam-local? Then it would make sense to support lambda integration.

I agree with @alexiaallanic-myob lambda integration should also be supported.

Will this never be supported? If not that means I can only use the proxy integration with SAM.
Otherwise whats the point if I can't test locally the same as the deployed version?


[state management] Update docs around transactional operations

Signed-off-by: Hannah Hunter<EMAIL_ADDRESS>Description
state_api.md

Clarify wording around table for transactional operation fields
Add supported transactional operations
Link to state mgmt overview explanation of bulk vs transactional operations

state-management-overview.md

Clarify wording around bulk vs transactional operations

Issue reference
PR will close: #1980

I did a direct update to the branch, since this was a little easier than explaining, mostly to the table to make sure it was clear that this was a request associated with an operation. PTAL


Update   ssm update

I updated the SSM UPDATE function to check repo and only update as needed.
I also made a big change to the console output messages.

oh, added Onset


Not working with older IE browsers versions 5 & 7

I tried it with older versions of IE explorer and found it doesn't work with older IE browser 5 & 7 the Debugger through an error:
 var bLazy = new Blazy();
'Element' is undefined

How to fix?
Thank you

@jodyshop
Not sure if you are serious or simply trolling, but in case.. the last stable release of MSIE 5 was 18 years ago.
MSIE7 reached it's end of life (finally) in 2016, with Microsoft no longer supporting it.
Requesting support for any JS library to support these browsers is useless, no (sane) developer will spend another minute of their precious time on these browsers.

@jodyshop Blazy works in IE7+. No support for IE 5 and 6. I hope you're not serious about that 😄


JSON/PropertyList coders userInfo should require Sendable values

This requires that the userInfo values are Sendable since they are stored in Sendable types. The @preconcurrency attribute ensures that this is not an ABI breaking change or a source breaking change for projects using the Swift 5 language mode.

@swift-ci please test

Marking as draft at the moment while we wait on this one - we need to wait for a compiler fix to ensure this isn't source breaking to clients

@swift-ci please test


switching to github docs as per #301

Addressing #301

Pull Request Test Coverage Report for Build 1706

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.854%

Totals

Change from base Build 1669:
0.0%

Covered Lines:
1354

Relevant Lines:
1368

💛  - Coveralls

I would suggest removing everything in doc/_build or simply removing this folder.

Great! Thanks!
You might also need to copy .nojekyll to docs/

jeez... I am the king of easy mistakes today... LOL

It is easy to miss. no worries.


Create cqu_explainForRemovingFromBlacklisting

Hi Philipto:
I just heard from Prof. Liu Ji that Chongqing University was blacklisted by Jetbrains because the school issues the email addresses not only to students. Though I fully understand that it is the discretion of the JetBrains to decide which institution should be blacklisted, but I will really appreciate if you can take my following words into consideration
First, I think the school only assigns the email addresses to students, faculties and employees. No one except the above three categories can be assigned the email address. I am pretty sure about that. It is also stated in the school policy here, though it is written in Chinese, but I think you can translate using google translators anyway.
http://wlaq.cqu.edu.cn/info/1004/1078.htm
Second, I think it is the common practice of universities not to annul the alumni email addresses after they graduated. I achieved the bachelors degree in Chongqing University and then an advanced degree in New York University, both universities don't void the students email after they graduate. As far as I know, most prestigious universities in the United States or in China don't void the alumni's email neither. After students graduate, they will be employed and the educational pack will not be applicable to them automaticallysince it is not designed for commercial use.
In addition, as one of the most prestigious universities in China, which is listed as the Top 30 in China and QS 300-400 in the world, I think the security of the email will be assured, and in no way will this promotion be abused. If you have already found the abuse here, it will be really nice of you to submit the evidence to the IT department.
Thanks again for your time and consideration, I fully understand whether or not to cancel the blacklist is at the sole discretion of your company, but it will benefit students a lot if you can de-blacklist the cqu.edu.cn domain.
Best Regards,
Richard
Institution/School Name
Chongqing University
Instiution/School Website
www.cqu.edu.cn
Email Users
Please place an x between the square brackets if yes

[ ] Students
[ ] Academic/Teaching Staff
[x ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

As I have checked just now, the university has recently launched a new security system that requires its members to change the password regularly when logging in Email account， so I think there is no security risk anymore. Meanwhile, if their is anything we can do to remove from the blacklist, I will be really happy to inform the relevant departments in CQU.
Thanks so much in advance

Also, I would be really grateful to coordinate with the IT department to solve the abuse problem(if their is any), thanks so much in advance

Here is the policy of the school, and
第三条 邮件帐号的申请、开通、注销
1．教工用户首次由信息与网络管理中心组织各部门统一填报，今后凭本人有效证件在信息与网络管理中心免费申请。一个教工只能申请一个邮箱帐号。
2．校内单位（部门）用户由单位（部门）开具证明到信息与网络管理中心免费办理。
<EMAIL_ADDRESS>4．教工邮箱帐号原则上可以永久使用，用户可以到信息与网络管理中心申请注销帐号。教工邮箱帐号两年不登陆、不使用则系统自动注销，但本人随时可以重新申请。学生邮箱帐号在其毕业后保留三个月的时间，然后由系统统一注销。


List and double-check the checksums, please.

Found through the Romhacking.net, where you list checksums.
You rather ambiguously list separate "File" checksums alongside "ROM" checksums.
Please do also clarify the distinction as snes9x gives me the exact same checksum in its rom infodump as I calculate from the file on disk.
The two files I tried matched the "ROM" checksums for 1.0 and 1.1 you list.
However, after trying numerous combinations of source files and patches you provide, not a single resulting file matches any of the expected resulting checksums.
I'm using Lunar IPS, which worked just fine just recently for other roms.
I suspect that the checksums listed on Romhacking.net are erroneous. (Possibly out of date.)
And there's still the confusion in regards to your distinction of "File" vs "ROM".
Please, introduce some much-needed clarity by listing valid checksums in the README.

The file vs ROM thing is RHDN's doing, not mine.
Here's their autogenerated output for scanning a headered 1.0 rom:
Database match: Fire Emblem - Monshou no Nazo (Japan)
Database: No-Intro: Super Nintendo Entertainment System (v. 20210222-050638)
File SHA-1: B1C82729077A48993621DC1D294A7759557A90CD
File CRC32: 23B7227
ROM SHA-1: 3EE8CBE6638DA64F1D91DF9B81B1478498A5999F
ROM CRC32: 25D214F4

Presumably the File checksums include the header and the ROM fields do not.
Is this a major issue? It's been 2 years since I last updated the checksums and I haven't seen any issues from anyone. If you apply the right patch and the game runs, there's nothing too worry about.
RHDN only requires checksums for unmodified roms. And those haven't changed.

If it's really a concern, here are md5's for the most recent patch:
3a6bc7804fe8842ebdebfd1ed6e1fc09  v1.0_headered.smc
4d6ea7343a552ee17557861d861262d5  v1.0_unheadered.sfc
ef044f99ffd16c2ae8f981a19ad57570  v1.1_headered.smc
eb2c31971709221382590caa278f6052  v1.1_unheadered.sfc

Thank you, this is rather helpful!
Patched an unheadered 1.1 rom, the resulting checksum matches and it finally runs well.
To clarify, the first couple of times I tried to apply the patch resulted in files that didn't match the hashes and while one would not play at all, the other would be all jumbled up and unplayable as well.
That's when I decided to just run through all combinations until one matches the resulting hash listed on Romhacking.net, since that would ultimately determine that everything worked.
However, none of the resulting files matched the hash there and the ones I tried running wouldn't play either.
Really seems now like it's out of date and probably in need of editing.
For sake of posterity, if anyone falls down the same rabbit hole as me, here are more checksums for unheadered rom files:
(Your headers may vary, resulting in wildly different hashing, so best strip them during patching for uniformity.)
1.0
CRC32:  35EA82D3
MD4:    8FD57490684ED8B3C281ED56BE3647BF
MD5:    4D6EA7343A552EE17557861D861262D5
SHA1:   34FF9D724B005A422518C6B7CF40B37D5AF8B379
SHA256: A18D4AAD9A6A771B5C4DA85E415DCA070C991BE58AD3F03CAE1C9C770F780338

1.1
CRC32:  F97DEEB0
MD4:    995501E4F52F0D6B5E9885FFA586685C
MD5:    EB2C31971709221382590CAA278F6052
SHA1:   C0200C357FED3E8B32A4A6A84DECC99722F853F7
SHA256: D9DB5D5CFBB13E387EBDA9199CB2E854AE02D6C27EAF2B295E5D3B0AA7B9023C

Another point of confusion was that this line from your readme…

No 1.1 transition patch was included because the Quirino 1.1 patch converts the ROM to 1.0.

…coupled with there only being a single resulting hash listed on Romhacking.net, gave me a false speculative impression that no matter which route I take — I should be arriving at the same exact file and the 1.1 revision changes somehow aren't relevant anymore. (I take partial blame for an overreaching interpretation.)
Thank you for your work on the patch and your assistance with checksums.


Docs don't actually say what messages are for

I like to be able to write code from the documentation without experimenting to figure out what something might mean first. These docs are pretty good, but the repeated statement "optional description message msg" doesn't tell me anything about this message. Is the message only conditionally shown? What is that condition -- test passing or failing?
I realize that I will have my answer with a simple experiment, but it would be helpful to say something somewhere about these messages.

Okay, so I did the test. The message is always shown. So it's a description of the tested condition. More helpful language might be:

optional description of tested condition msg
optional description of assertion msg

Saying "optional description of test" might ambiguously refer to the containing call to test() and not just the particular assertion.
Also, a "message" seems like a report of news. This isn't really news so much as a description, so desc might be a more helpful parameter name. That may explain why I assumed the message would only show under one of the conditions.
Okay, now back to more significant things...

Typically the "message" with every test framework is a human-readable description of the test being performed, and is often called "message" or "description" interchangeably.
The fact that that is a common pattern doesn't mean the docs can't be improved - PRs welcome!

I posted a pull request. I noticed that a number of methods are using the msg parameter as a message to report a condition, so I decided to leave the parameter name unchanged. The consistent name should help make it clear that there is a string description associated with each assertion. Besides, it's best to leave the names consistent with the code as well.
I also clarified the throws() assertion, which otherwise doesn't say how it's used. I noticed that when expected is neither a function nor a regex, the assertion always passes. Is that a bug? It would be nice to help protect the programmer from herself when possible.
I could not make sense of doesNotThrow(). The code doesn't seem right to me either. I didn't attempt to clarify this assertion.

The pull request failed for code failing to execute? The only thing that changed was a few lines in readme.

I have found a bug in the .only mechanism and would like to offer a fix, but my master branch now has this readme file that won't pass tests. git/github forever confuse me.

@jtlapp typically PRs are best done from a branch, not from your local master - in that case you'll want to make a branch off of master~1 - ie, off of the remote's master, not the local master.

Okay thanks. Does that mean my master branch should only ever mirror the source repository master branch? Would I only ever pull into my master from the source repository, never merging with local branches? (Thanks for this aside helping me with mechanics.)

And now magically my original pull request is passing. Someone reran it.

Yes, that's basically how it would work. Also yes, I reran it - that particular test is flaky.

Okay, thank you. Is it safe for me to pull your merge into my master, or is that going to muck things up? (I have no confidence in this. We once mucked things up pretty bad thinking we were merely merging to get everyone on the same revision.)

Generally you want to always hard reset your fork's master to match the source's master.

Okay, as shown on this SO? http://stackoverflow.com/questions/1628088/reset-local-repository-branch-to-be-just-like-remote-repository-head

Sure.

Pull request https://github.com/substack/tape/pull/298 took care of this.


Breadcrumb unusable with multiple ui-views

When you have multiple ui-views, the ncyBreadcrumb.parent function gets executed for each of the views.
This is not correct, you should be able to specify for what view/controller/scope to get the parent and label executed against.
How can you do that?

+1 It's popped up here too

+1 I think this would solve my current problem of the label not being parsed properly because of the different views.

I just had a look at this to try and solve it for myself.
I'm guessing the ideal place to check the mainView would be when getting the $lastViewScope on $viewContentLoaded but I can't see any way tosee what view a scope belongs to.
Any ideas?

+1, I don't have multiple views, but I define a single view in the state's views config object.
I'm not able to read the scope variable in the label.

+1

+1

Just did a pull request to solve this issue. #93

+1
Any chance to see this pull request merged?

I thought this issue has been automatically closed by 934c5523208a9615d7cfa3abcb397bbe131332ac but I hasn't :smile:

I reopen the issue to remind myself I must mention the ncyBreadcrumbIgnore in docs (a new entry "FAQ" I think)

Can you explain where i should use ncyBreadcrumbIgnore i'm a bit lost ..

`angular.module('app', ['ui.router', 'ncy-angular-breadcrumb'])
.config(['$stateProvider', '$urlRouterProvider', function($stateProvider, $urlRouterProvider) {
$stateProvider
.state('A', {
abstract: true,
url: '/a',
views: {
'a@': {
template: 'View A',
controller: 'ACtrl'
}
}
})
.state('A.B', {
url: '/b',
views: {
'b@': {
template: 'View B',
controller: 'BCtrl'
}
},
ncyBreadcrumb: {
label: 'State {{tripleB}}'
}
});
$urlRouterProvider.otherwise('/a/b');

}])
.controller('ACtrl', function($scope) {
$scope.ncyBreadcrumbIgnore = true;
console.log('a');

}).controller('BCtrl', function($scope) {
console.log('b');
$scope.tripleB = 'BBB';
})
.run(function($rootScope, $location) {
$rootScope.$location = $location;
});`


Update evaluation loops

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

src/gravitorch/loops/evaluation/vanilla.py
ø

src/gravitorch/datasources/base.py
100.00%

src/gravitorch/loops/evaluation/accelerate.py
100.00%

src/gravitorch/loops/evaluation/basic.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


add 3d patch merging layer

Codecov Report

Merging #806 (0e40bf8) into main (c998703) will increase coverage by 0.04%.
The diff coverage is 90.90%.

@@            Coverage Diff             @@
##             main     #806      +/-   ##
==========================================
+ Coverage   70.80%   70.84%   +0.04%
==========================================
  Files         169      170       +1
  Lines        7723     7745      +22
  Branches     1267     1269       +2
==========================================
+ Hits         5468     5487      +19
- Misses       1880     1883       +3
  Partials      375      375

Impacted Files
Coverage Δ

towhee/models/layers/patch_merging3d.py
90.90% <90.90%> (ø)

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

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 c998703...0e40bf8. Read the comment docs.


Bug: label isn't shown when form is used in cards

when use form inputs in cards , the labels aren't shown

please include a code example


looking for an alternative to null ls

null ls will be archived, so looking for an alternative is necessary.
Could it be efm server again?
What will go on with the C++ linter and formater?

The alternative could be https://github.com/creativenull/efmls-configs-nvim
So to resolve this issue is the implementation of this plugin


How to validate the same password for password and confirmation fields?

Hi,
Can you you show how to validate the password and confirmation fields? I do have
private var passwordConfirmation = mutableStateOf("")

Validators.Custom("Password given must be the same") { password ->
                        password.toString() == passwordConfirmation.value
                    }

And I do have the above in the password TextField and I am trying to capture the confirmation in

TextFieldState(
                name = "password_confirmation",
                validators = listOf(Validators.Required(),
                    Validators.Min(8, "Password must be 8 to 16 characters"),
                    Validators.Max(16,"Password must be 8 to 16 characters"),
                    Validators.Custom("Password given must be the same") { confirmation ->
                        passwordConfirmation.value = confirmation.toString()
                        true
                    }
                )
            )

But this does not work. Thanks for your help.

Hello @wpoosanguansit 👋 ,
I assume you have 2 textfields, one for the Password and another for ConfirmPassword, and each is passed as a field in your FormState instance. In the FormBuilder Library any input field state such as the TextFieldState has a property value which enables you to access the value of a given input field. Therefore, one does not need to create another variable to store value of an input field.
Building up on this, for your case you can create the custom function that accesses the value in the 'password' TextFieldState and compares it to the value in 'confirm_password' TextFieldState . We don't need to add true when checking otherwise the result will always be true despite the conditions not matching. Note this has to be away from the instatiation of the formstate to avoid compilation errors. The code looks like this:
val formState: FormState<BaseState<*>> = FormState(
        fields = listOf(
            TextFieldState(
                name = "password",
                validators = listOf(
                    Validators.Min(8, "Password must be 8 to 16 characters"),
                    Validators.Max(16, "Password must be 8 to 16 characters"),
                    Validators.Required(),
                ),
            ),
            TextFieldState(
                name = "confirm_password",
                validators = listOf(
                    Validators.Min(8, "Password must be 8 to 16 characters"),
                    Validators.Max(16, "Password must be 8 to 16 characters"),
                    Validators.Custom("Password given must be the same") { confirmation ->
                        passwordMatch(confirmation.toString())
                    },
                    Validators.Required(),
                ),
            ))
)

private fun passwordMatch(confirmationPass:String): Boolean {
        return confirmationPass == formState.fields.find { it.name == "password"}?.value.toString()
 }

Also, the order of the validators matters, for better UX, consider passing the Validators.Required() as the last validator.


Table creation, CREATE/EDIT Vehicle from UI  and reflecting the changes in the UI

Hi @Stewart86
While Setting up this project to work on another issue, this issue has been found and resolved.
Issue:

VehicleGen.py: Generating the Simulated values but not creating the table entries in the database.
Vehicle model class(model.py): Create/Edit Functionality was not working.
Reason: The custom creates a method that was written in the model class instead of using the database create method.
View.py: After Add/Edit and delete the view was not updating.

All the above issue is resolved in the below link.
https://github.com/rajrohan/roadtaxTracker

Along with these changes, Multiple selection #18 has also been implemented.
Please find all the changes in the new pull request. #27


Initial load issues

There seems to be an issue on first load with the cache on the latest version, user object isn't always loaded even though it is fetched via a promise.

Resolved in https://github.com/jakke-korpelainen/newtab/commit/edb1130aee546249bbb94af1f30d071da662d301


fix [BUG]The IP and process pid of the master and worker in the monitoring center are not displayed

What is the purpose of the pull request
bug fix#2735

store process id in zk.
resolve host and port through zk path.
modify ui key.

Verify this pull request

Manually verified the change by testing locally.

Codecov Report

Merging #2770 into dev-1.3.0 will increase coverage by 0.98%.
The diff coverage is 35.09%.

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

The problem of e2e failure has been resolved, please pull the latest code of dev-1.3.0 and submit the code again

@tlhhup
is IP field can be removed?

sorry, i miss some code.


Run apt-get clean

Running apt-get clean at the end of the RUN commands would make the image a little bit smaller.

Any reason to not do this?

Could you please clarify?


feat: Added the hello world program in rust language.

Please, add Rust in available languages liste.

Please, add Rust in available languages liste.

Updated

Good job


Questions about names for query parameter values

I'm finding some of the examples for serializing query parameters a little confusing in the 3.0.2 spec.
Given this portion of the examples:

style
explode
empty
string
array
object

form
false
color=
color=blue
color=blue,black,brown
color=R,100,G,200,B,150

form
true
color=
color=blue
color=blue&color=black&color=brown
R=100&G=200&B=150

spaceDelimited
false
n/a
n/a
blue%20black%20brown
R%20100%20G%20200%20B%20150

pipeDelimited
false
n/a
n/a
blue|black|brown
R|100|G|200

deepObject
true
n/a
n/a
n/a
color[R]=100&color[G]=200&color[B]=150

For form, I see we are always listing a parameter name (color, or R, G and B, as appropriate).
For deepObject, I see we also have parameter names (color[R], color[G], color[B])

Questions:

spaceDelimited and pipeDelimited however don't have names.  I'd expect these should actually be:

color=blue%20black%20brown
color=R%20100%20G%20200%20B%20150
color=blue|black|brown
color=R|100|G|200|G|150

This part of the OpenAPI Specification is inconsistent - the Style Values section defines the spaceDelimited and pipeDelimited behavior for arrays only and not for objects, but the table still shows object examples. This was previously discussed in https://github.com/OAI/OpenAPI-Specification/issues/1508:

dekryptic commented
2. Also, under the "Style Values" table, should the types for simple be primitive, array, object and the types for spaceDelimited and pipeDelimited be array, object? You have examples for these styles and types in the next section.

darrelmiller commented
2. For simple, primitive is the same as an array of one. For object the behavior is undefined. I think the behaviour should be the same for pipeDelimited and spaceDelmited.
darrelmiller commented
2. I said undefined, but I sort of lied. OAS doesn't explicitly define what to do with objects that are not exploded, but 6570 does and that is what the example shows. See https://tools.ietf.org/html/rfc6570#section-3.2.2 I'm not really sure we want to specify that behavior because it really isn't very useful.

@hkosova I agree with you about the inconsistency.
Do you know what's the right way to deal with this?  That is, it seems like it's unfortunate that the spec has problems in it, especially if there's some resolution about it in the issues here. It would be beneficial to all if these were eventually addressed.  I see they haven't been addressed in the 3.1.0-dev branch, so I could, for example, submit a PR with my best interpretation of the conclusions in the threads.

#3737 is a duplicate of this, which I did not realize, but now it has more discussion and several PRs attached to it.  Closing this in favor of that issue - the examples will be fixed in 3.0.4, 3.1.1, and 3.2.0, as tracked through #3737.


RSI with 3 months of data?

I'm feeding the RSI with 3 months of data, but it's only graphing the last two months. How do I make the RSI handle 3 months? I looked at the rsi plot code and I don't see anything limiting the months there.

Most indicators with a period need to be primed before they start emitting anything useful. That's probably what you're seeing. ie rsi needs 14 periods of moving average otherwise the calculation won't be very helpful.
If you want 3 complete months of rsi, you will need 3 months + rsi period of data.


[HOPS-676] Rewrite the UsersGroups cache

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-676

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Yes, it removes the flushUser/Group RPCs and instead adds two RPCs for adding and removing users or groups

Other information:
depends on https://github.com/hopshadoop/hops-metadata-dal/pull/133 and https://github.com/hopshadoop/hops-metadata-dal-impl-ndb/pull/165

Can one of the admins verify this patch?

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.

can you check this test: https://bbc1.sics.se/jenkins/view/pull/job/hops-pull/117/io.hops$hadoop-common/testReport/org.apache.hadoop.security/TestUserGroupInformation/testLogin/

@berthoug fixed. It was failing because i added code to bypass the Hadoop's GroupsCache when HopsGroupsWithFallback is used. They were updating the metrics while getting the groups inside their cache, which i didn't take into consideration when bypassing the cache. The fix is that i call the same method to ensure that we update the metrics in both cases where the cache is used or not.


WIP: Support for async BuildModel implementation

@mortenbock You should have .ConfigureAwait(false) on every task you await, since you don't know if any SynchronizationContext/TaskScheduler is active in the environment you are running in.


Ingestion server hangs after cleaning

Description

I've seen this issue a bunch as I've been running tests locally. It seems like the ingestion server hangs immediately after finishing the cleaning:
integration_ingestion_server_1  | 2021-12-17 23:49:59,173 INFO cleanup.py:306 - Cleaned all records in 0.4112739562988281 seconds

No other logs occur after this and the integration tests fail due to timeout. This seems super transient and is frequently resolved in future runs.
Reproduction

Run just recreate && just init or just ing-testlocal
Get lucky

Additional context

Resolution

[ ] 🙋 I would be interested in resolving this bug.

I attempted to tackle this in #412 - I'm going to close this but we can re-open it if it reappears


Balder/update attributes

@havardpe PR

[ERROR] configs_related_to_reference_fields_are_derived(com.yahoo.searchdefinition.derived.ReferenceFieldsTestCase)  Time elapsed: 0.021 s  <<< FAILURE!

Yes, java test needs an update due to new field in attributes.def.
It is building and will come soon. The c++ part should be fine though.

All comments have been addressed. Merging.

@geirst FYI


feature: deduplicate InPredicate.

Proposed changes
Issue Number: close #xxx
Problem summary
deduplicate InPredicate.
tpcds-q27
(s_state IN (
     'TN'
   , 'TN'
   , 'TN'
   , 'TN'
   , 'TN'
   , 'TN')

Checklist(Required)

[x] Does it affect the original behavior
[x] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] 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

run buildall


Patch 1, added pages

I found that editing an item, and creating a "patch" was a way to clone and edit the repo.

Very nice; Thanks!


Proposed HTTP API Changes

Up until this point the Kapacitor HTTP API has been "implemented" rather than "designed". This PR presents an API "design" in the form of documentation, in preparation for locking down the API to a more supportable long lived API.
Currently the existing API has some inconsistencies and odd usage patterns. This doc attempts to unify and simplify using the API. It makes the API more RESTful and consistent with itself.
For the most part only small changes where made to the existing API. For example when query parameters vs JSON structures vs path elements are used is clear and consistent. Parameter names and top level keys in JSON structures all use lower case names.
This represents  a breaking change to the API. It is intended that once we settle on this API design that it will not need any further changes before it becomes a stable locked down 1.0 version.
This PR only documents the proposed behavior of the API but does not implement any of the changes. Once accepted another PR will finalize the implementation.
Some notable usage changes from the existing workflow.

Task pattern matching is only supported on the list task requests. To enable/disable multiple tasks matching a pattern one must first list matching tasks and then make the appropriate request for each returned task. This simplifies using the API since each task has a unique path and only modified via that path. If this is not going to work for some's use case please comment and we can work on a consistent solution, but I defaulted for simplicity in this case.
No more plural path names like /tasks or /recordings since the lack of a child path indicates you want all to interact with all entities.

@yosiat @goller @arussellsaw How do you like this proposed change?

:+1: will this also be hosted on the influxdata kapacitor docs page?

@arussellsaw Yes

@nathanielc this looks great! added in-line comment about the recordings.
By the way - "/debug/vars" will stays? or it's removed?

By the way - "/debug/vars" will stays? or it's removed?

It will stay.

@goller @yosiat @arussellsaw I have updated the API doc per the many good suggestions.
In general the changes are:

Use POST for new creations, Use PATCH for modification. Only tasks can be modified.
Use id for everything so that words like name are not implicitly associated with tasks etc. IDs for everything can be client specified or generated randomly server side.
The three entities are tasks, recordings and replays. Each has its basic POST,GET, GET (list) endpoints.
All list requests can be paginated.
Both replay, and recordings will return progress information and never block.

This represent a lot of work but I like the consistency and flexibility it is going to provide. And most of the work is duplicated across entities so it should be too bad if done right (fingers crossed).

I don't know if this is related to this pull request, but it will be nice if Kapacitor will expose (or have in the repo) a swagger file, then you can generate clients for all languages from go (https://goswagger.io/) to c#

@yosiat  @goller  I have added a swagger file for the API. It probably has a few typos at this points since I haven't tested it but the general structure is all there.

Merging this for now, implementation coming that makes a few small changes.


Garbled when I use rasax to add Chinese templates

Rasa version:
1.1.6
Rasa X version (if used & relevant):
0.19.5
Python version:
3.6.8
Operating system (windows, osx, ...):
windows
Issue:
Garbled when I use rasax to add Chinese templates,Unable to start when I restart rasax, prompt coding problem
Error (including full traceback):
Traceback (most recent call last):
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\xuexi\Anaconda3\envs\rasax\Scripts\rasa.exe\__main__.py", line 9, in <module>
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\__main__.py", line 76, in main
    cmdline_arguments.func(cmdline_arguments)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\cli\x.py", line 291, in rasa_x
    run_locally(args)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\cli\x.py", line 315, in run_locally
    _validate_rasa_x_start(args, project_path)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\cli\x.py", line 262, in _validate_rasa_x_start
    _validate_domain(os.path.join(project_path, DEFAULT_DOMAIN_PATH))
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\cli\x.py", line 275, in _validate_domain
    Domain.load(domain_path)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\core\domain.py", line 72, in load
    other = cls.from_path(path, skill_imports)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\core\domain.py", line 86, in from_path
    domain = cls.from_file(path)
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\core\domain.py", line 99, in from_file
    return cls.from_yaml(rasa.utils.io.read_file(path))
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\site-packages\rasa\utils\io.py", line 130, in read_file
    return f.read()
  File "c:\users\xuexi\anaconda3\envs\rasax\lib\codecs.py", line 321, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc4 in position 707: invalid continuation byte

Command or request that led to error:
rasa x

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for raising this issue, @Saladdin will get back to you about it soon.

same here.

出现同样的问题了

nobody care this bug. no any response.

We are working on a fix for the issue. Will let you know once a patch is released 👍

thanks for help!

change setting in requirements.txt maybe solve it.
keras==2.2.4, numpy==1.16.5

this should be fixed in 0.20.3


This fixes issue-174 and issue-175 by outputting the proper write fun…

…ctions even if the reset-value is missing.
I'm currently working with my Arduino Due and try to get things working. I wrote a quick hello world-application that blinks. All the registers that were needed work with this patch. I did not do much further testing and I'm relatively new to Rust.
I tested with the sam3x8e-SVD-file from the Atmel homepage.
The changes are based on another older pull request, which cannot be applied to the current repository and seem to work fine.

Is there anything I can do to help with the review?

This is great! Unspecified reset values are valid in SVD but unforuntantely have made the whole device fail to compile when used with svd2rust.
Any reason this is not a "no-brainer"?

Any reason this is not a "no-brainer"?

Well, a test case exploiting/using that would be great.
Other than said, it looks good to me, but in order to approve it I'd need to somehow verify the effect and correctness of this change myself for which I haven't found the time yet.

I made a PR on posborne/svd-cmsis to add some SVD files where this is the case to enable basic automated testing against them. https://github.com/posborne/cmsis-svd/pull/67
I've added a file that contains definitions that doesn't compile unless using the code from this PR. Hope that can be of some convinience.
ATSAMV71Q21B.svd.txt

I took a quick look at the code, it seems like you are silently degrading write to be the same as modify when a reset value is not specified?
in general:

Write starts with the reset value, and applies any values you change to the reset value
Modify starts with the current value, and applies any values you change to the current value

If this is correct, is there a reason you want to use write instead of just modify? Personally, I am not a fan of silently changing the behavior, as this would be hard to catch for most users of a -pac crate, and would make the behavior of -pac crates inconsistent (write does one thing on some platforms, and another on other platforms). They would have to look at the code to see what has been generated.
Please let me know if I misunderstood your intent, or what the code actually does.

I'm trying to get the CI working on my system. Therefore sorry for the "force-push". I'll create another branch and do my testing there.

Semantically, this PR is a breaking change. The svd2rust documentation specifies: "the write method writes some value to the register using a single, volatile STR instruction.". This PR changes write into a RMW operation in some cases; there's no precedent for the svd2rust API having different semantics based on the content of the SVD file (we only ever add / remove API based on the information available) and I'd rather keep it that way ("no suprises and backward compatible when improving the SVD file").
I have mentioned before (either on this issue tracker or IRC) that when resetValue info is missing there should be some "write" method (not exactly with that name)  that does a single volatile write and whose argument forces you to specify all the (non-reserved) bit fields of the register (there are a few ways to do that, but a struct would be the easiest).

That makes sense. I try to come up with a new function (maybe something like reset?). It will need some time, I think I have to become more familiar with the whole tool.

I think we can close this PR and if I (or someone else) comes up with a better solution, we open a new one.

In the short term, would it be acceptable to remove everything inside the "else" block? This would mean that there would not be a write method for registers with unknown reset value.
Alternatively, what about not compiling anything for the register, issue a warning and moving along with the file?
Anything would be better than the current solution where the whole file fails to compile if the reset value is undefined.

Most of the registers of my SVD don't have a reset value, which means I can't write any registers. I'm not sure how that's better.

Ping from triage: @ammoniak Do you still want to close this PR or consider the change from @kjetilkjeka.
@therealprof @jamesmunns is the @kjetilkjeka  proposition valuable for sdv2rust?

Most of the registers in my SVD file miss the reset value, so "ignoring" those, won't help at all.

So I suggest that you close this PR.


Microsoft.PowerShell.Archive Cmdlets should work on typical linux compression formats

PowerShell 5 now supports native (de)compression of zip files. It would be great if this could be extended to include:

.tar
.gz
.lzma / .7z
.bz2

Ideally, this would function recursively so formats like .tar.gz get decompressed or created.
This would make PowerShell better at handling archives from multiple systems.

+1

Great suggestion!
Per #1979 we are working on consuming the Archive module from it's own repo https://github.com/PowerShell/Microsoft.PowerShell.Archive
Can you, please, open an issue there?

Opened #2743 to decouple it

Thx @vors!


fatal error: 'bits/libc-header-start.h' file not found - on ide.CS50.com

Hello. I learn CS50 course and have one problem with compiling some program on web-server ide.CS50.com
I think, that I'm trying to compile 32-bit program on 64-bit machine.
I'm trying every solution that I can found:

Added  -m32 in every line of make file.
Checked tab-symbols in every line of make file.
Tryed to install "sudo apt-get install gcc-multilib g++-multilib",  "sudo apt-get install gcc-multilib" and "sudo apt install libc6-dev-i386" but saw the massage with "Failed to fetch".

Please, help.

How is this related to the repository?


Missing license information

Hello! Please can you add a license to the project. Thanks.

Hi there! Updated. Thank you.

Thanks!


Automating data uploads

**Feature Request
**Low priority
**For discussion post-AGU
I wanted to record these thoughts while they were fresh, but I realize that everyone is busy with the AGU release. So, let's discuss after AGU is over.
Background
Review this resource: Troch, P., R. Dwivedi, A. A. M. Neto, T. Liu, T. Roy, R. Valdés-Pineda, M. Durcik, S. Arciniega-Esparza, J. A. Breña-Naranjo (2017). Data for the catchment-scale groundwater recharge and vegetation water use efficiency estimation, HydroShare, [http://www.hydroshare.org/resource/99d5c1a238134ea6b8b767a65f440cb7]
The publisher used a generic resource to combine hundreds of shape files into a zip file, thousands of time series into zip files (both MOPEX.ZIP and evap.zip) and two excel spreadsheets that add properties to the MOPEX basin beyond what the original  MOPEX data did.
This approach makes sense from the publishers point of view--he could easily create this resource, it was self-contained, and could be issued a DOI (which is his objective). However, it doesn't take advantage of any of the Hydroshare features. But to take advantage of them would require the user to upload thousands of time series, hundreds of shape files, etc. manually. This is not feasible.
Furthermore, consider a data user downloading the resource. Arguably, it is more convenient to download these files as zip files and unzip them, and open them locally than downloading all of these shapefiles etc. individually. Perhaps the bagit structure would solve this problem (but I don't think it allows a hierarchy of files that a user-created zip file would).
We are spending a lot of effort to define metadata profiles and apps to access data resources, but this publisher can't take advantage of all that effort. How can we solve that problem?
A few automation tools could be helpful

n-tuple time series support. A user has a spreadsheet of multiple data columns all with a single time column--e.g., a set of samples with multiple solutes analyzed or, like the MOPEX data, daily time series with 5 different data values (precip,et, discharge, min temp, max temp). The user should be able to submit this time series to create n time series. Internal storage could be as SQL Lite file or other format. Users should be able to download this set of time series with a single click--possible as a csv file with an additional metadata file (explaining the measurements, for example).
Multiple shape file uploads. Here, I don't really know the options, but if I have a Arcproject containing multiple layers, how can I upload that entire project to hydroshare without having to manually upload each shape file. How can they be grouped for convenient download?

We need to think about this both on the upload side and the download side. What processing is done locally at either end and what processing is done by Hydroshare?
Hope this is specific enough to be useful.

Closing, automated uploads are available through the API

Closing, automated uploads are available through the API