disable autoId | Schema Error: Unknown directive "pk"

Describe the bug
I am using the info from post #56 and getting the following Schema Errors:
Schema Errors:
Unknown directive "pk".
GraphQL request (35:13)
34: {
35:     id: ID! @pk

graphQL  Schema:
type Campaign
@model
{
    id: ID! @pk
    ....
}

Additional context
This is a schema that was previously created with just ID!
Thanks

@rfdc The @pk directive is not implemented. There will soon be a new RFC that comments on how to build custom index structures for @model types.

That would be great cos every time I update my schema I need to update the resolvers manually and also document it for future developers
Any idea when it will be implemented? How do I get updates towards it?
thanks @mikeparisstuff

Here is the RFC for custom indexes https://github.com/aws-amplify/amplify-cli/issues/1062. It would be great if you could respond with feedback so we can make sure to address any concerns. You also have the option of overwriting any auto-generated resolver templates with custom templates that will not be overwritten each time you run build. You can read more details on this here https://aws-amplify.github.io/docs/cli/graphql#overwrite-a-resolver-generated-by-the-graphql-transform.

We launched support for custom indexes today. You can find docs for the same out here - https://aws-amplify.github.io/docs/cli/graphql#key
Please let us know if you're still not able to solve your problem through this solution and we'll re-open this issue for you.


[Bug]: repeated "reduce failed" errors reported

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Version
v2.44.2
Database
CockroachDB
Database Version
23.1.13
Describe the problem caused by this bug
About once a second the following is logged in our zitadel container:
time="2024-02-07T14:55:04Z" level=debug msg="trigger failed"
    caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:261"
    error="ID=PROJ-0I91sp Message=Errors.NotFound" instance=242916770875179014
    projection=projections.user_grants4
time="2024-02-07T14:55:04Z" level=error msg="reduce failed" aggregate=usergrant
    caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/statement.go:47"
    error="ID=PROJ-0I91sp Message=Errors.NotFound" instance=242916770875179014
    projection=projections.user_grants4 sequence=1
time="2024-02-07T14:55:04Z" level=info msg="process events failed"
    caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:416"
    error="ID=PROJ-0I91sp Message=Errors.NotFound"
    projection=projections.user_grants4

We recently upgraded from 2.44.1 and, before that, 2.44.0 and 2.41.1
To reproduce
We were importing some users, but I'm not sure why that would cause these errors to fill up the log even after we stopped.  We imported a few hundred users and had to stop when the container began consuming too much CPU.
Screenshots
No response
Expected behavior
No response
Operating System
Docker CE 25.0.1
Ubuntu Jammy (22.04.3)
Relevant Configuration
No response
Additional Context
No response

Found in Discord Just now: Cannot add user grants

From Discord:

I saw this behavior from my system when I upgraded to 2.42. I was able to resolve by rolling back to 2.40 (what I was running before the upgrade). I believe it is related to the “spoiler stuck” issue that the 2.43 rcs appear to be aimed at fixing - if I understand the release notes correctly.

However, as noted above, we're on 2.44.

Rolling back to 2.41.1 eliminated the errors in the log, so the problem is not fixed in 2.44 despite what the Discord comment says.

Oh sorry to hear that, let us investigate what is going on here.
CC @livio-a @adlerhurst

@stebenz can you please have a look at this as well? looks like there's an issue in getting the resourceowner of some part of the user grant


⚠ Bofrost.nl has degraded performance

In 8898c08, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 926 ms

Resolved: Bofrost.nl performance has improved in ceecdb6.


skiring system user?

It has been suggested that the skiring user created by the rpm should be a system user.

Addressed in release 0.4.0.


Problem with boxes

So I'm using Comfy theme and I'm trying to make everything all split and put into there own boxed but I am having trouble with it. Everything else is fine but this bottom part

Asking help on the server would be a better choice imo


CFP: Multicast IPSec Support

Multicast IPSec Support
Implement IPSec support for multicast traffic.
Dependent on #29469

Completed and merged into main-ce
UBS has a custom made image based on v1.15-ce.
Once v1.16-ce is released that branch can be destroyed and they can be moved to the traditional release cadence.


feat(instr-aws-sdk): add instrumentation and tests for AWS S3

Closes: #37

nice.
Are you still intending to work on this to no longer use a hardcoded port?

nice.
Are you still intending to work on this to no longer use a hardcoded port?

Yes and I've added it in my last commit. Thanks for the reminder :)

One final change to make it simpler. Ready for another review @trentm


build: move config.example.json to config.json

Simplify the project's quickstart process for greater convenience.

Forgot that I did this for a reason: make sure to prevent the config.json file from being inadvertently leaked through Git by adding it to the .gitignore file 🤦


How to use it with ssh?

I'm using this image for rancherOS and would like to use ssh for automated deployments but seems like I can't use ssh with this image. Here is the error I see when I try to git clone,
Cloning into 'foo'...
error: cannot run ssh: No such file or directory
fatal: unable to fork

I have updated the image with openssh package installed, could you pull and try again?

Thanks for the update. I have updated your image to latest and tried to clone the repo again, I see a different error.
Cloning into 'foo'...
The authenticity of host 'bitbucket.org (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is SHA256:zzXQOXSRBEiUtuE8AikJYKwbHaxvSc0ojez9YXaGp1A.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added 'bitbucket.org,<IP_ADDRESS>' (RSA) to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

How do I make the image use .ssh directory of the user running that image?


Add SASL support (optional, disabled by default)

Given that it's disabled by default (requires an explicit -S to enable it at runtime) and is also enabled by Debian, it seems sane to include SASL support by default. :smile:
Closes #15 (thanks @ndobromirov!)

Size comparisons:
Before:

debian: ~83MB
alpine: ~5.8MB

After:

debian: ~83MB
alpine: ~6.9MB

Will the documentation be updated to have a guide on:
"How to use the image wiht SASL"?

@ndobromirov it'd be great to have a section about that (although I'm not sure how soon we'll get to it); https://github.com/docker-library/official-images/pull/2754 is where the update is making it's way to being published (didn't realize we hadn't done that yet)
It should be as easy as adding -S to the server startup options, and use something like saslpasswd2 -a memcached -c cacheuser as documented on that page you linked (https://github.com/memcached/memcached/wiki/SASLHowto), but I'm honestly not sure how/where SASL stores that stuff, so I'm not sure if that needs to be run from the image, or against a running instance. :innocent:

@tianon  the package that was added to the debian image, libsasl2-dev, does not include saslpasswd2 command line utility. however, this package does include it: sasl2-bin.
This repo includes the sasl2-bin package and looks like it has a working method of "using the image with SASL" (though i have not tried it yet)


[VATT] Construct and load datasets using DMVR

Hi, @hassanhub. I have some problem with DMVR.
I followed the dmvr example to generate the dataset.
And changed the corresponding code so the code can read the correct path of data.
But when I run the vatt evaluation process, it would raise following error:

tensorflow.python.framework.errors_impl.InvalidArgumentError: 3 root error(s) found.
(0) INVALID_ARGUMENT:  Name: , Key: image/encoded, Index: 0.  Number of values != expected.  values size: 91 but output shape: []
[[{{node ParseSingleSequenceExample/ParseSequenceExample/ParseSequenceExampleV2}}]]
[[MultiDeviceIteratorGetNextFromShard]]
[[RemoteCall]]
[[IteratorGetNext_3]]
[[model/video_module/vit_base/spatio_temporal_embeddings/strided_slice_5/_1]]
(1) INVALID_ARGUMENT:  Name: , Key: image/encoded, Index: 0.  Number of values != expected.  values size: 91 but output shape: []
[[{{node ParseSingleSequenceExample/ParseSequenceExample/ParseSequenceExampleV2}}]]
[[MultiDeviceIteratorGetNextFromShard]]
[[RemoteCall]]
[[IteratorGetNext_3]]
(2) CANCELLED:  Function was cancelled before it was started
0 successful operations.
0 derived errors ignored. [Op:__inference_evaluation_step_21087]
Function call stack:
evaluation_step -> evaluation_step -> evaluation_step

I located the error at line150 in file ./vatt/data/loading.py where the object generate by command factory.make_dataset() cannot be read. The minimize pseudocode is as follow:
dataset =  factory.make_dataset(
              shuffle=self.shuffle,
              num_epochs=self.num_epochs,
              batch_size=per_replica_batch_size,
              padded_batch=False,
              drop_remainder=True,
              keep_key=False,
              override_preprocess_fn=None,
              )
iter_data = iter(dataset)
next(iter_data) // this would trigger the error

Here is the pipeline I used to generate the data file. I write some info in the README.txt.
dmvr_example.tar.gz
I cannot identify whether I miss used the dmvr, or I miss code the vatt data reader.
What should I do?
Thanks

Hi Adam,
Thanks for flagging this. Could you please fork the repo and point me to your commit for the data reader?
I prefer not to open any compressed and/or uploaded files.
Also, having a repo would make it easier for other people to understand the issue and point to it as a reference in future.
Thanks

Hi Adam,
Thanks for flagging this. Could you please fork the repo and point me to your commit for the data reader?
I prefer not to open any compressed and/or uploaded files.
Also, having a repo would make it easier for other people to understand the issue and point to it as a reference in future.
Thanks

I believe I have met the identical problem as @Adam-fei described, though my setting is slightly different. Here is a repo including the code (modified on configs/dataset.py and data/processing.py as described in #917 ) and two data samples. The repo doesn't contain the misc folder and pre-trained model because of the size constrain. It will be great if @hassanhub could also help me to check it. :D
I'm using the "finetune" mode on the generated dmvr tf-record of a different dataset. Currently, I only tried on for video modality with vit_large arch, starting from Large-Base-Small check-point. I used the default setting of "generate_from_file.py" in dmvr repo to generate the data samples in the repo above.
P.S: Thank you guys and happy new year!

I believe I have met the identical problem as @Adam-fei described, though my setting is slightly different. Here is a repo including the code (modified on configs/dataset.py and data/processing.py as described in #917 ) and two data samples. The repo doesn't contain the misc folder and pre-trained model because of the size constrain. It will be great if @hassanhub could also help me to check it. :D
I'm using the "finetune" mode on the generated dmvr tf-record of a different dataset. Currently, I only tried on for video modality with vit_large arch, starting from Large-Base-Small check-point. I used the default setting of "generate_from_file.py" in dmvr repo to generate the data samples in the repo above.
P.S: Thank you guys and happy new year!

Thanks a lot for reporting this.
I have a clarifying question. What fields are you storing in your tfrecord?
Please note that input_feature_name should match the one that is already in the tfrecord fields.
I assume you are storing your visual features under a field which does not match input_feature_name="image/encoded". So, you get INVALID_ARGUMENT: Name: , Key: image/encoded.

Could you also please check if the output of this line is non-zero?

Thanks a lot for reporting this. I have a clarifying question. What fields are you storing in your tfrecord? Please note that input_feature_name should match the one that is already in the tfrecord fields.
I assume you are storing your visual features under a field which does not match input_feature_name="image/encoded". So, you get INVALID_ARGUMENT: Name: , Key: image/encoded.

Thanks for your reply @hassanhub !
I used the generate_from_file.py from dmvr to generate the tfrecord. The key "image/encoded" at there is same as the field name in processing.add_vision() in the dataset factory. I only modified the input_feature_name=** in processing.add_audio() and processing.add_label() in the factory to make them consistent from both sides.
I have reproduced the tfrecord of the dataset to check the output of this line. So far there is no zero-length img_encoded list from the video data I'm using.
The content below is the partial error note I got when running vatt.main. The difference compared to @Adam-fei is that the Name: is <unknown> in my case.
 2022-01-05 17:04:20.749125: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 49 but output shape: []
 2022-01-05 17:04:20.760604: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 76 but output shape: []
 2022-01-05 17:04:20.771905: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 115 but output shape: []

Thanks!

Hi, @hassanhub sorry for the late reply, and big thanks to @adaniefei for offering more informations.
You can find my fork code here in the branch debug.
I'll try to explain my process with the code as clearly as possible.
Besides, you can find my modification for the code in the only commit I pushed
I'm trying to reproduce the result of kinetics400, so I download the following checkpoint file.

To load this model correctly, I modified the ./vatt/configs/video.py file: 14--->20
I generate the tfrecord using dmvr. You can find the pipeline at ./vatt/dmvr/examples/run.sh. I also upload the corresponding kinetics400 videos so you can generate tfrecord through my pipeline if you want.
I put the tfrecord in ./vatt/dmvr/examples/generated_dataset/valid. To read this tfrecord, I modified the code in ./data/datasets/toy_dataset.py, data/datasets/factory.py.
Furthermore, I ensured that the output of this line is non-zero, and the input_feature_name name I used in dmvr is "image/encoded" (you can check at this line)
Besides all above, I also made some changes so the code can run properly:

modified ./configs/dataset.py to meet the config of trained model according to issue917
modified data/processing.py due to the version mismatch of dmvr according to issue917
modified experiments/finetune.py, so the code can load the model I specified in the command line.
downloaded ./misc folder and pre-trained model and put them in the right place.

After the modifications, I run the code using
python -m vatt.main --task=finetune --mode=eval --model_dir=./my_output --model_arch=vit_base --strategy_type=mirrored --override_checkpoint=./vatt/Checkpoint/Kinetics-400_BBS/vision_vatt_finetune_vit_base_k400_modality_specific_pretrain_ckpt-100000
Then I'll get the error message as follows:
I0106 18:39:41.460304<PHONE_NUMBER>00480 video_dataset.py:456] Dataset created successfully
2022-01-06 18:39:55.803064: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 48 but output shape: []
2022-01-06 18:39:55.806709: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 101 but output shape: []
2022-01-06 18:39:55.807336: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 91 but output shape: []
2022-01-06 18:39:55.807725: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 101 but output shape: []
2022-01-06 18:39:55.814186: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 101 but output shape: []

If you need more informations, please tell me
Thank you! :)

It seems that the tfrecord saved by your DMVR's example is not compatible with the parser in the pipeline.
I'll contact the POC in DMVR and get back to you with a solution for storing a compatible tfrecord.
Thanks for your patience. I hope this is the only bottleneck for smoothly running VATT.

Thanks, @Adam-fei for the confirmation, too. :D
And thanks @hassanhub for the reply. Look forward to the new tfrecord version. :D

Are you able to run this example in DMVR's repo?

@hassanhub
Thanks for the reply! I have checked and found the other two generating tfrecord scripts work well following the instruction. Then to run the example, I have to modify the line 48 from
 shards = [f'hmdb51_{subset}_{split}-{i:05d}-of-{num_shards:05d}'

to
 shards = [f'{subset}_{split}-{i:05d}-of-{num_shards-1:05d}'

After this, I used the cmd:
  python linear_mmv_hmdb.py \
  --data_path=/path/to/hmdb_shards \
  --model_name=s3d \
  --hmdb51_split=1 (instead of **split_1**)

and got a similar INVALID_ARGUMENT error:
  ....
  2022-01-24 18:57:42.806099: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 86 but output shape: []
  2022-01-24 18:57:42.816347: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 26 but output shape: []
  2022-01-24 18:57:42.826208: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 35 but output shape: []
  2022-01-24 18:57:42.836729: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at example_parsing_ops.cc:480 : INVALID_ARGUMENT: Name: <unknown>, Key: image/encoded, Index: 0.  Number of values != expected.  values size: 15 but output shape: []
  ....

Also, the script generate_from_file.py generates one less tfrecord file than it should have, For instance, the tfrecord files are named in the format of train_1-xxxxx-of-00059, while the xxxxx here is from 00000 to 00058.
Hope this feedback helps. :)

Thanks, @adaniefei
This is very helpful!
I'll work with DMVR to understand where in the generation things go wrong. Hopefully we can submit patch to fix this soon.

We should have fixed this on the DMVR code: https://github.com/deepmind/dmvr/commit/adf78b7a8edd4bb56c6ce1bffaa5003019761716. Could you retry after that commit? Sorry for this!

Thank you @jalayrac
@adaniefei @Adam-fei please let us know how it goes.

Hi @jalayrac and @hassanhub, I've pulled the latest commit of dmvr, and I'm able to run the hmdb example now. The only change I have to do locally is to upgrade cuDNN from 8.0.5 to 8.1.0.
Thanks, @jalayrac for the latest commit, and @hassanhub later this week I will update about the local VATT running using the latest commit from dmvr. :D Thanks again guys!!!

Hi. A big thank-you to @jalayrac. The hmdb example of dmvr can run successfully.
However, despite using the new version of tfrecord, vatt still output following error:
I0211 14:25:43.123416<PHONE_NUMBER>14144 video_dataset.py:456] Dataset created successfully
2022-02-11 14:25:50.560914: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.560957: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.564367: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.564423: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.565001: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.

tensorflow.python.framework.errors_impl.InvalidArgumentError: 3 root error(s) found.
  (0) INVALID_ARGUMENT:  slice index 0 of dimension 0 out of bounds.
	 [[{{node strided_slice_9}}]]
	 [[MultiDeviceIteratorGetNextFromShard]]
	 [[RemoteCall]]
	 [[IteratorGetNext_3]]
	 [[model/video_module/vit_base/spatio_temporal_embeddings/strided_slice_3/_3]]
  (1) INVALID_ARGUMENT:  slice index 0 of dimension 0 out of bounds.
	 [[{{node strided_slice_9}}]]
	 [[MultiDeviceIteratorGetNextFromShard]]
	 [[RemoteCall]]
	 [[IteratorGetNext_3]]
  (2) CANCELLED:  Function was cancelled before it was started
0 successful operations.
0 derived errors ignored. [Op:__inference_evaluation_step_21087]

Function call stack:
evaluation_step -> evaluation_step -> evaluation_step

What should I do?
@hassanhub
cc @adaniefei

Hi @hassanhub @Adam-fei , sorry for the late update. I have tried latest hmdb51 tfrecords with the following cmd for the fine-tune step, and I got a different error here:
    python -m vatt.main --task=finetune --mode=train --model_dir=pretrain_model\ --model_arch=Vit_Large --strategy_type=mirrored
    ....
    ....
    ValueError: Inputs must be an iterable of at least one Tensor/IndexedSlices with the same dtype and shape.

P.S: I'm using one GPU, so I modified the line 97 using tf.distribute.mirrorstrategy(cross_device_ops=tf.distribute.HierarchicalCopyAllReduce()) based on this post.

Hi. A big thank-you to @jalayrac. The hmdb example of dmvr can run successfully.
However, despite using the new version of tfrecord, vatt still output following error:
I was running finetune/test mode using:
python -m vatt.main --task=finetune --mode=eval --model_dir=./my_output --model_arch=vit_base --strategy_type=mirrored --override_checkpoint=./Checkpoint/Kinetics-400_BBS/vision_vatt_finetune_vit_base_k400_modality_specific_pretrain_ckpt-100000

And following is the error:
I0211 14:25:43.123416<PHONE_NUMBER>14144 video_dataset.py:456] Dataset created successfully
2022-02-11 14:25:50.560914: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.560957: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.564367: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.564423: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
2022-02-11 14:25:50.565001: W tensorflow/core/framework/op_kernel.cc:1745] OP_REQUIRES failed at strided_slice_op.cc:108 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.

tensorflow.python.framework.errors_impl.InvalidArgumentError: 3 root error(s) found.
  (0) INVALID_ARGUMENT:  slice index 0 of dimension 0 out of bounds.
	 [[{{node strided_slice_9}}]]
	 [[MultiDeviceIteratorGetNextFromShard]]
	 [[RemoteCall]]
	 [[IteratorGetNext_3]]
	 [[model/video_module/vit_base/spatio_temporal_embeddings/strided_slice_3/_3]]
  (1) INVALID_ARGUMENT:  slice index 0 of dimension 0 out of bounds.
	 [[{{node strided_slice_9}}]]
	 [[MultiDeviceIteratorGetNextFromShard]]
	 [[RemoteCall]]
	 [[IteratorGetNext_3]]
  (2) CANCELLED:  Function was cancelled before it was started
0 successful operations.
0 derived errors ignored. [Op:__inference_evaluation_step_21087]

Function call stack:
evaluation_step -> evaluation_step -> evaluation_step

What should I do? @hassanhub
cc @adaniefei

I have the same problem, is the problem solved?

@G-Apple1 Hi, got the same error but I can surpass it when we use tf.funtion() decorator.

@sainivedh19pt may I ask where you put your tf.function() decorator? I couldn't solve the above error when trying to use my own dataset which is transformed into tf_record format as dmvr example describes.

@ybkim95 Sorry I may not be much of help as its been a long time

excuse me ,can you help me convert the raw videos and text to TFRecords,i have downloaded the two datasets

how can i convert them? thank you

Hi,
I'm currently attempting to fine-tune VATT for the MELD dataset following the instructions provided by @Adam-fei and @adaniefei. Unfortunately, I'm encountering the same issue with the error message:
2023-12-11 22:26:34.523213: W tensorflow/core/framework/op_kernel.cc:1839] OP_REQUIRES failed at strided_slice_op.cc:117 : INVALID_ARGUMENT: slice index 0 of dimension 0 out of bounds.
I've tried adjusting the batch size and ensuring the input CSV is consistent, but so far, no solution has worked.
Does anyone have an idea on how to solve this problem with VATT?

Hello
I am a beginner.
Can you please explain how to make the tfrecord file that I have produced or the file related to the KINETICS400 or HOWTO100M dataset to the factory file?
I always face this problem:
File "/home/zahra/Desktop/google-research/vatt/data/dataloaders.py", line 133, in init
factory_class = ds_fctr.get_ds_factory(
File "/home/zahra/Desktop/google-research/vatt/data/datasets/factory.py", line 39, in get_ds_factory
ds = DS_TO_FACTORY[dataset_name]
KeyError: 'HOWTO100M'
File "/home/zahra/Desktop/google-research/vatt/data/dataloaders.py", line 560, in init
factory_class = ds_fctr.get_ds_factory(
File "/home/zahra/Desktop/google-research/vatt/data/datasets/factory.py", line 39, in get_ds_factory
ds = DS_TO_FACTORY[dataset_name]
KeyError: 'KINETICS400'
I don't even know what is meant by checkpoint data and index?
Please explain to me with an example.
Thank you.


Use generator option on multiple diectories

Would it be possible to select multiple directories and execute the selected generator option on each one individually? It would save a lot of mouseclicks when taking care of lots of separate features in a project.
Sascha

Hi @saschaernst!
It is something I have kept in mind for a while. I think it may take me some time to do it and I currently do not have such. Nonetheless, I try to do it or at least to find an acceptable solution.
As of now, I can only close the issue as I can't tell when this will be done! Sorry for the inconvenience but I will keep you noticed 😉


Clarify term "dataverse" vs. "dataverse collection"

Do I understand it correctly, there is

dataverse - the repository software
dataverse collection - A folder-like data structure made to group data sets and (optionally) other dataverse collections.

The key point is that there is only one datastructure dataverse collection and the current readme just uses a short form dataverse when it really means dataverse collection. Is this correct?
In the dataverse docs both terms (dataverse and dataverse collection) are used for apparently the same thing, too. Using consistent language would be helpful for a precise understanding.

Also related here commit https://github.com/nfdi4cat/repo4cat/commit/cd39d772b65f7f7a4ec11b7c5c6a5ab7cc43debd.


Make 'git gui' not block other git commands

There are a few possible solutions here:

Windows creates 'git-gui.exe' which uses 'git update-index', instead of 'git gui' which uses 'git-update-index.exe'.  'git-gui.exe' or equivalent doesn't appear to be build for Mac.
Make 'git-update-index.exe' use command hooks
Make 'git gui' use 'git update-index' instead of 'git-update-index.exe'

However we should follow what we've done on Windows and instruct users to run "git-gui.exe".
It looks like git-gui is installed in /usr/local/git/libexec/git-core/git-gui on Mac.  We'd need to make it path accessible on Mac for parity. @dscho.
Note this work has not been prioritized.

FWIW the git-gui.exe in Git for Windows exists mainly so that we have a neat target for the Start Menu item "Git GUI"...


Creation of Environment fails with Message status "uuid: can't be blank"

Describe the bug
Issue with creating Environment in Project. Creation of Environment always fails with Message status "uuid: can't be blank"
Development or production environment
Docker image: clivern/lynx:0.11.8
TF Provider version: Clivern/lynx 0.3.0
Additional context
Return error "uuid: can't be blank" for Web UI and Terraform provider
TF Output:
lynx_environment.prod: Creating...
╷
│ Error: Client Error
│
│   with lynx_environment.prod,
│   on main.tf line 16, in resource "lynx_environment" "prod":
│   16: resource "lynx_environment" "prod" {
│
│ Unable to create environment, got error: status: 400, body: {"errorMessage":"uuid: can't be blank"}

@ankek please bump your version to v0.11.9 and check again. I was doing some refactoring in a separate brach and a change slipped into master and broke it.

@ankek Please feel free to reopen the ticket if you still face the issue.

All works good. Thank you for fixing it


Update dep

Coverage remained the same at 100.0% when pulling 67e2703827792428afd27e37163b54a5ce6f1928 on update-dep into 65bedf9d2448b26adbc4bf0823563dde7b856c09 on master.

Coverage remained the same at 100.0% when pulling 67e2703827792428afd27e37163b54a5ce6f1928 on update-dep into 65bedf9d2448b26adbc4bf0823563dde7b856c09 on master.


how to transfer ckpt to pb or saved model?

as the title shows, how we do this?
tensorlfow has lots of saved format, and version issues.
there is no such document,about how we can do this.
i search the similar issues, you all said these question maybe suitalbe for stackoverflow.
but none of you have answered the question.
maybe this is an document issue or not.
thank you very much if you can give me the instructions

Have you tried using freeze_graph.py.
https://github.com/tensorflow/tensorflow/blob/236a77594fec644e0676ffe9144ff047cd7142fd/tensorflow/python/tools/freeze_graph.py#L26

thanks ,i will try.. it seems to be able to transfer ckpt to frozen_graph as pb.
what about the solutions for transfering to saved_model?

See TensorFlow Saved Model https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/saved_model/README.md
Feel free to reopen if have any further problems. Thanks!


Add HaUI - Trường Đại học Công Nghiệp Hà Nội

Add Hanoi University of Industry at: https://dhcnhn.vn/

Here is the proof that Hanoi University of Industry has 2 emails address:

https://khoadien.haui.edu.vn/vn

@denlink
Kindly help on this.
Thanks.

@davidpaf-MSFT Thank you for the clarification. Pull request merged.


feat: Added ability to connect to RDS via secret.

Added DB_SECRET, GRPHILE_ENDPOINT, and DATABASE_ENDPOINT env variables. This allows CDK to give secret and allow connection to RDS.

Added fix to workers to connect to correct connection string for function schema in process-event

Looks good, thanks Chris!


Ourea in the List?

Hello i Love this Mod and there are so Many Characters to Play whit^^ I found a Character Called Ourea on Wiki she should be Part off Frozen Wilds
But i cant find her in the Tool ( I have all Characters shown)  Is she there or is she not Playable?
Thanks

Sometimes the internal names don't match up with the characters, I couldn't find a good way to link the two.
Ourea should be 'DLC Prophet'

thank you its correct i dont know why but i like Ourea xd totaly Fun to play as her


Content / UserStorage APIs

Consoles often require very specific mounting for file access and save data. I think there should be 2 utility classes that can be used to simplify this. Obviously if the user doesn't care about non-desktop portability they can just ignore these:
The Content class lets the user enumerate and read files from a Content directory. I think the App should create a default one. On Desktop platforms this would find files beside the .exe location, for example new Content("Assets") would work over files in the Assets directory adjacent to the YourGame.exe.
public class Content
{
	public Content(string directory);

	public IEnumerator<string> EnumerateFiles(string relativePath, string searchPattern, bool recursive);
	public IEnumerator<string> EnumerateDirectories(string relativePath, string searchPattern, bool recursive);

	public bool FileExists(string relativePath);
	public bool DirectoryExists(string relativePath);
	public Stream OpenRead(string relativePath);
}

User Storage is simpler and should be a singleton... it just reads / writes data to user files and can see if they exist. On Desktop this would use App.UserPath. I don't think there's any need to enumerate files here.
public class UserStorage
{
	public static bool Exists(string path);
	public static byte[] Read(string path);
	public static void Write(string path, byte[] data);
}

I think there is some utility in being able to list user storage files. Specifically, in relation to game saves. Many games rely on enumeration of some some shared save location that allows players to drop in saves as needed without having to modify some shared save directory file.
Additionally, being able to use streams (OpenRead, OpenWrite) would be beneficial here for larger/more complex save files.
All this is dependent on whether consoles even allow enumeration/streams/support for big enough files to warrant them, which I wouldn't know or be able to find out.
Also, is this going to be incorporated with all of the current api methods that load files? Or would a user be expected to get a stream using this new api and use it with existing ones (I recommend the latter).
Finally, if this is a purely optional component of Foster, maybe keeping it in its own namespace (Foster.Framework.Storage) would be beneficial, seeing as storage methodologies/implementations can be highly opinionated and vary greatly depending on project requirements.

I think there is some utility in being able to list user storage files.
Yeah, I think you're right. In general I just do a hacky thing where I check "does 'fileN' exists", incrementing N until none are found. But it'd be better to actually return a list of existing files.

Additionally, being able to use streams (OpenRead, OpenWrite) would be beneficial here for larger/more complex save files.

Yeah, I agree with this in general. My concern is that it may make the Platform side more complicated because it would need to better handle mounting/opening/closing around the stream - but if the Stream is a custom class we make that handles it, then it should be fine.

Also, is this going to be incorporated with all of the current api methods that load files?
I don't want to change any methods that load files as long as they take a Stream. I agree don't want weird Content-specific load methods - I think they'd be confusing and redundant.

Finally, if this is a purely optional component of Foster, maybe keeping it in its own namespace
Definitely agree. My intention with this is just to have some way to handle content/user data generically across platforms, but like other things I don't think it should be mandatory and if someone wants to write their own better one for their use cases it should be easy to ignore the default one.

I think the current implementation is "good enough" for now. If/once actual porting to non-desktop applications happens new issues can be created.


Zend\Form\Form::getInputFilter() issue with Collection Element

Hi !
I'm working on a project and I think that the method getInputFilter() of the class Zend\Form\Form must be rewrited.
If we have an Element of type Collection where we can add some new values and we want to get the input filter before to set any data from POST to the form, we have an issue because when we populate values with data POST after that we have getting the input filter of form, the validators, filters & co was not initialized with the new values of the collection... So, we can't validate theses new values and can't get filtered values.
I have not any time for now to propose a correction about that but if you can give me some reflexions about that, you are welcome :)

This looks to be a duplicate of #5243, which appears to be fixed.  You can pull the InputFilter instance before populating the form and the same instance will be updated accordingly when bind or setData is called on the form.
Is this still an issue for you, @YannickDa ?


Build ENS namespacing into the CLI as an experiment for this approach

Input for this experiment:

Dealing with prefix_chainID_tokenID table names is cumbersome.
Some kind of namespace for a group of tables sounds very attractive.
This namespacing should be resolvable on- and off-chain.
The validator should not need to handle resolving namespace -> table mapping, as this would introduce another source of truth.
The validator is not a good place to store the mapping, as this would make it unavailable off-chain (unless duplicated).
Building a unique namespace management system is complex and comes with a number of important considerations:

Name expirations are important to ensure no permanent lock-in of a given name.
Interop with existing tools is important, but requires a lot of traction to make happen.

ENS is the most widely adopted name system in the Ethereum ecosystem.
ENS plans to launch on L2 but are currently not on any.
Whatever we do here should be done slowly and with great consideration / testing because it would be very hard to back peddle.

The experiment
Determine if ENS domains (top-level or subdomains) can provide a "good enough" mapping from a "name" to a set of tables. We can do this without touching the SDK... just trying to get a feel for it with as little work as possible.
Consider an app that needs tables for inventory, customers, and sales.
Creating the tables
Assume this hypothetical user has a .tablelandrc file with their private key and other needed params set (chain, etc), and that they own the foo.eth ENS domain.
> tableland create --ns myapp.foo.eth "id integer primary key, item text, count int" inventory
Namespace 'myapp.foo.eth' not found. Would you like to create it? N/y
> y
Namespace 'myapp.foo.eth' created
Table 'inventory.myapp.foo.eth' created.

What's going on here?

Using etherjs / ennjs, the CLI creates the subdomain myapp under foo.eth
CLI creates the table as usual (nothing special here)
Using etherjs / ennjs, the CLI set's the content field of the subdomain to be a JSON key value store of prefix -> full table name, e.g.,

{
    "inventory": "<chainID>_<tokenID>"
}

If the user didn't own foo.eth, the CLI could ask if they want to register it and do it for them (if it's available).
This same flow could also work for top-level domain names, but this would mean every "namespace" is mapped 1:1 with an ENS domain, which generally be a more expensive way of managing namespace. Nevertheless, it should still be doable if somebody wants to do that.

Create the other tables
> tableland create --ns myapp.foo.eth "id integer primary key, name text" customers
Table 'customers.myapp.foo.eth' created.

> tableland create --ns myapp.foo.eth "id integer primary key, customer_id int, inventory_id int, count int" sales
Table 'sales.myapp.foo.eth' created.

What's going on here?

The namespace already exists, so no need to prompt about creating it.
The content field of myapp.foo.eth is now:

{
    "inventory": "<chainID>_<tokenID>",
    "customers": "<chainID>_<tokenID>",
    "sales": "<chainID>_<tokenID>"
}

Writing to the tables in the namespace
From the CLI
> INSERT INTO inventory.myapp.foo.ens (...) VALUES (...);
{txn info}

What's going on here?

Using etherjs, the CLI resolves the name after pruning the first part of the "namespace", ie, myapp.foo.ens and reads it's content, which is our JSON mapping
The CLI then finds the real table name that is being touched and proceeds as normal.

From solidity
It's possible to resolve an ENS domain on-chain, but I haven't looked into whether or not you can extract the content field. If you can, then this works in much the same way as from the CLI. We could have some helper methods to that do the resolving under the hood.
Reading from the tables in the namespace
> SELECT * from customers.myapp.foo.ens;
{rows}

What's going on here?

Using etherjs, the CLI resolves the name after pruning the first part of the "namespace", ie, myapp.foo.ens and reads it's content, which is our JSON mapping
The CLI then finds the real table name that is being touched and proceeds as normal.

Downsides

This approach handles the namepacing at the client layer, meaning that queries in URLs that hit a gateway can't have namespace references. Unless we want to get ENS resolving into the gateway, which may not be a bad idea.

Other considerations

This approach could be extended to support other popular name services like Unstoppable Domains

here's a pretty good and simple ens schema example that arose from playing with it quite a bit this weekend.
https://app.ens.domains/name/tictactoe.rockefeller.eth/details
tictactoe.rockefeller.eth has 3 text fields
xyz.tableland.players: temp1_80001_541
xyz.tableland.results: temp1_80001_540
xyz.tableland.disputes: events_80001_436

so, when connected to this ens, just use players, results, and disputes for short names for the tables. matching the json setup more or less.
the key bits:

works the same as above but follows the patterns of the text record name space. for example, com.reddit in the defaults.
this is fully readable on chain since you know every key will map to a single string. versus parsing an arbitrary json payload, which would be really hairy. just look for any keys that start withxyz.tableland and you're in business.
use subdomains as the namespace since they work really well for that. it also means you can use all the other fields to document your app metadata when helpful.
no other tools that support ens things need to know something about our json payloads.

super, very cool that text fields can be arbitrary. nice exploration! sounds promising.


some journeys skipped

There seems to be a bug where some journey sometimes get skipped.
I have repeatedly run this in psql
update journeys set disabled=FALSE where id>=5428 and id<=5471 ;
and the resulting journeys are enabled run once and then disabled (because there type is test_routing)
As per the table below, in some runs all 44 are run but in several some are skipped – not always the same ones each time.  Could you look into this?

Run
N runs / 44
ones missiong

21208
44

21209
44

21210
44

21211
44

21212
41
5453,  5454,  5455

21213
40
5462,5463, 5464 , 5465

21214
44

21215
43
5464

I noticed it most acutely in this set of multiple short journeys, made me wonder if there could be some sort of maximum requests per minute timeout.

I don't see this:
select run_id, count(*) from journey_runs where run_id between 21208 and 21215 group by run_id;

 run_id | count
--------+-------
  21208 |    44
  21209 |    44
  21210 |    44
  21211 |    44
  21212 |    44
  21213 |    44
  21214 |    44
  21215 |    44

Oh right sorry I see – those journeys are there I just couldn't see them in the export I did. That is because the csv files were sorted by column A and that was weirdly formatted so was not listing everything strictly chronologically


build fail error with bibliography

I tried to set up my own publications with bib file and etc, however saw that it did not work and the github build failed related to the bibliography tag.
https://github.com/kenkyusha/kenkyusha.github.io/actions/runs/4018037216/jobs/6903150436#step:4:36488
I ended up completely removing the jekyll scholar and bib functionality in the following commits and got it building:
https://github.com/kenkyusha/kenkyusha.github.io/commit/6af9017b00751f7b8696aa12bd5d0579111b3ed1
https://github.com/kenkyusha/kenkyusha.github.io/commit/e6c0a43469bcd24d09573d3fbf77178f55a3cd64
https://github.com/kenkyusha/kenkyusha.github.io/commit/303f1c4fa981bad32626b0ffb994bb43d9973a00
https://github.com/kenkyusha/kenkyusha.github.io/commit/3549bec3cee2402814fd99cb0f310114248bf6fa
https://github.com/kenkyusha/kenkyusha.github.io/commit/1a9a0f87eb4a97d71db3b149e645d1ce43bc0c39
Anyone else had this error or suggestions how to work with it?
Locally the bib and publication thing just ran fine.

I had exactly the same problem as you last week.
As stated in CLAMS and many issues, the error Unknown tag 'bibliography' inevitably occurs in automatic deployment, due to the conflict between the security setting of Github and plugins: jekyll/scholar.
It seems that manual deployment is the only solution:
$ ./bin/deploy

Use the master branch for the source code and deploys the webpage to gh-pages.

I remember I removed a bunch of the bibliography stuff that had dependencies

If your website still use the plugin plugins: jekyll/scholar, its automatic deployment would be reject by Github.

I had exactly the same problem as you last week. As stated in CLAMS and many issues, the error Unknown tag 'bibliography' inevitably occurs in automatic deployment, due to the conflict between the security setting of Github and plugins: jekyll/scholar.
It seems that manual deployment is the only solution:
$ ./bin/deploy

Use the master branch for the source code and deploys the webpage to gh-pages.

as he mentioned, read the blog post, it explains quite well.
Myself, I will not do it but, closing the topic for now.

The batch file works only on linux, how to deal with this on Windows?


we've got issues

grrrrrr

Oh no! 🙈

Hey @GitGud44 - random question - are you getting notifications when a response to your issue comes in? Trying to see if students are notified when I respond. Thanks so much!

Hey @GitGud44 - random question - are you getting notifications when a response to your issue comes in? Trying to see if students are notified when I respond. Thanks so much!

Hey! I did get an email, but only when you tagged me


WARNING : Refusing to register transform "MatrixOffsetTransformBase_float_3_3" again!

WARNING message
WARNING: In /usr/include/ITK-4.10/itkTransformFactoryBase.h, line 81
TransformFactoryBase (0x5619fe2fac30): Refusing to register transform "MatrixOffsetTransformBase_float_3_3" again!
when I was trying the ANTs example in
https://github.com/qureai/Multi-Atlas-Segmentation
What's more, I get similar warning message when I run antsApplyTransforms with 2 transformations in my own script, while the output seems to be right.
In my script, I try to apply transformations to an altas label
antsApplyTransforms
-d 2 --float 0
-i ${SEG} -r ${FIXED} -o ${WSEG} -n Linear
-t ${TRANS}1BSpline.txt
-t ${TRANS}0GenericAffine.mat
Is it an ANTs bug?
Environment
x86_64 computer running Debian testing with latest ANTs version( build from source code, no superbuild), and ITK4.10 from debian mirror.

I haven't seen this before but you might want to let ants use its super
build this may be related to some sort of mixing of versions or something
like that .... anyway it looks like a harmless warning
On Tue, Feb 28, 2017 at 10:50 PM Kai Xuan<EMAIL_ADDRESS>wrote:

WARNING message
WARNING: In /usr/include/ITK-4.10/itkTransformFactoryBase.h, line 81
TransformFactoryBase (0x5619fe2fac30): Refusing to register transform
"MatrixOffsetTransformBase_float_3_3" again!
when I was trying the ANTs example in
https://github.com/qureai/Multi-Atlas-Segmentation
What's more, I get similar warning message when I run antsApplyTransforms
with 2 transformations in my own script.
In my script, I try to apply transformations to an altas label
antsApplyTransforms
-d 2 --float 0
-i ${SEG} -r ${FIXED} -o ${WSEG} -n Linear
-t ${TRANS}1BSpline.txt
-t ${TRANS}0GenericAffine.mat
Is it an ANTs bug?
Environment
x86_64 computer running Debian testing with latest ANTs version( build
from source code, no superbuild), and ITK4.10 from debian mirror.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/420, or mute the thread
https://github.com/notifications/unsubscribe-auth/AATyfs57uGTvCH_lsUDwT738SyS5ppEPks5rhOrogaJpZM4MPMDc
.

--
brian

thank you very much and I will try superbuild soon.
Due to poor internet condition here, downloading the whole ITK repository
from github is very time-consuming( more than 24 hours)
What's more, ANTs is a very convenient and powerful tool and it does help
me a lot.
thanks.
On Thu, Mar 2, 2017 at 3:43 AM, stnava<EMAIL_ADDRESS>wrote:

I haven't seen this before but you might want to let ants use its super
build this may be related to some sort of mixing of versions or something
like that .... anyway it looks like a harmless warning
On Tue, Feb 28, 2017 at 10:50 PM Kai Xuan<EMAIL_ADDRESS>wrote:

WARNING message
WARNING: In /usr/include/ITK-4.10/itkTransformFactoryBase.h, line 81
TransformFactoryBase (0x5619fe2fac30): Refusing to register transform
"MatrixOffsetTransformBase_float_3_3" again!
when I was trying the ANTs example in
https://github.com/qureai/Multi-Atlas-Segmentation
What's more, I get similar warning message when I run antsApplyTransforms
with 2 transformations in my own script.
In my script, I try to apply transformations to an altas label
antsApplyTransforms
-d 2 --float 0
-i ${SEG} -r ${FIXED} -o ${WSEG} -n Linear
-t ${TRANS}1BSpline.txt
-t ${TRANS}0GenericAffine.mat
Is it an ANTs bug?
Environment
x86_64 computer running Debian testing with latest ANTs version( build
from source code, no superbuild), and ITK4.10 from debian mirror.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/420, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AATyfs57uGTvCH_
lsUDwT738SyS5ppEPks5rhOrogaJpZM4MPMDc>
.

--
brian
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/420#issuecomment-283447667, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQ3ch62Tasv_RiwzxcAiU88HHJcFnZ2nks5rhcpsgaJpZM4MPMDc
.

No warning message anymore after compilation of ANTs with superbuild enabled.
Problem Solved.


migrate from jbuilder to dune

closes #265
most of this was just renaming files and adjusting conventions for dune.
the only exercise left out was https://github.com/exercism/ocaml/tree/master/exercises/palindrome-products due to reliance on ppx_deriving see https://github.com/ocaml-ppx/ppx_deriving/pull/170

By convention .merlin files seem to be ignored https://github.com/github/gitignore/blob/master/OCaml.gitignore#L23 . From my understanding they are auto generated - e.g. in my case with the dune switch the files included absolute paths relevant only to machine.
but if in doubt let someone comment with more experience in that domain.

If you still get autocomplete, etc, in your IDE with the new files then I'm happy with it.
I'm without PC at the moment, but I'll also be able to check in a couple of weeks time.

Still all works fine in vim.

Good stuff... I'll merge.
Thanks again for your help.


Change Theme/Style

Is there a way to change theme/style?

zurnet,
I had this same question so I looked into how I might accomplish it.  Here's what I came up with.
It looks like ember-code-snippet includes its own highlight-style.css file.  This means it gets included in the project's generated vendor.css file.  Keep that in mind...
I found this project, ember-cli-node-assets, which lets you include assets from node_modules of your project.  At first I tried "importing" a stylesheet from the node_modules/highlight.js/styles/ like this:
    nodeAssets: {
      'highlight.js': {
        import: ['styles/monokai.css']
      }
    }

However, that includes the styles in vendor.css and the ember-code-snippet styles override the highlight.js styles, which were included in vendor.css first.  So I went the "public" direction:
    nodeAssets: {
      'highlight.js': {
        public: ['styles/monokai.css']
      }
    }

(There are many stylesheet files included with highlight.js.)

This puts the monokai.css file into dist/assets/styles/.  I then used a link tag in the HTML head section after vendor.css like this:
    <link rel="stylesheet" href="{{rootURL}}assets/vendor.css">
    <link rel="stylesheet" href="{{rootURL}}assets/styles/monokai.css">

It would be really nice if ember-code-snippet could provide a configuration option to let you include stylesheet files from highlight.js, but this technique does work.
tr


replace absurdly large numbers with error messages on insufficient liquidity

Thanks for filing this, @rolandpo. What threshold would you use to determine that it's insufficient?

I'm not sure about the exact number, but I think it would still be valuable to have as many options available to show liquidity levels, so even something like below 10% would be enough to get rid of these numbers overlapping

I'm not sure about the exact number, but I think it would still be valuable to have as many options available to show liquidity levels, so even something like below 10% would be enough to get rid of these numbers overlapping

This might be what you mean by 10%, but I was thinking that we should refrain from showing the numbers if your effective price is worse than $1.10. The only problem I can think of is that you can trigger a terrible price like that just by entering very low numbers (e.g. "1"). We wouldn't want to prevent those transactions since some people will do this to submit test transactions and kick the tires.
We actually already have a "slippage too high" state, which we could extend to include these high-price cases. In fact, I would probably just change that message to be "Price too high" since it's not exactly slippage in these cases.
Just for fun, here's the worst case resulting from low numbers (not as bad as your screenshot):

Maybe we could exclude routes where Est. received differs from input amount by more than 10% rather than Effective Price. That would be pretty much same thing for larger amounts, but wouldn't affect smaller test transactions

Maybe we could exclude routes where Est. received differs from input amount by more than 10% rather than Effective Price. That would be pretty much same thing for larger amounts, but wouldn't affect smaller test transactions

Smart 🧠


[FEAT]: Create Issue forms and make the issue options more visually appealing

Describe the Feature you want to end

Currently, the issue template for this repo seems to be basic also it is less visually appealing.👇

Changes I will make

Expected behavior

Screenshots (optional)

Additional context (optional )
Will add issue forms to the respective issues to make it easier for the contributor to raise it.
Here is an example of how a Bug Issue Form will look like 👇

kindly assign me this issue under GSSOC24

Hey @VanshKing30 can you add the GSSOC24 label and apt level to this issue.


Stash cannot be "ticked" or deleted from interface

Hello,
It seems there is a bug with the Uchiwa interface version 0.15.0 with sensu 0.15.1-1.
On the /#/stashes URL, I cannot tick nor delete stashes anymore.
I tested both on Chromimum 51.0.2704.79 and Firefox 46.0.1. Is it something you observe as well?
Kind regards,

Hi @xlr-8,
Thank you for reporting this issue. It's indeed something I was able to reproduce, I will have a patch ready as soon as possible.
Thanks


Existing jobs fail if there is a dangling job in the database

Output below. I am not sure how I ended up in this state, but I used to have a jobPublishLocation and I don't have it anymore. However, this failure was happening when I was trying to run a different jobPublishPost.
After uninstalling/installing my app the error went away.
02-13 13:53:36.555 10746 21171 W ReactNativeJS: Possible Unhandled Promise Rejection (id: 0):
02-13 13:53:36.555 10746 21171 W ReactNativeJS: Error: Job jobPublishLocation does not have a worker assigned to it.
02-13 13:53:36.555 10746 21171 W ReactNativeJS: getConcurrency@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:86548:26
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81631:67
02-13 13:53:36.555 10746 21171 W ReactNativeJS: write@[native code]
02-13 13:53:36.555 10746 21171 W ReactNativeJS: getConcurrentJobs$@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81619:33
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCatch@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14069:23
02-13 13:53:36.555 10746 21171 W ReactNativeJS: invoke@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14242:32
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14112:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCatch@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14069:23
02-13 13:53:36.555 10746 21171 W ReactNativeJS: invoke@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14145:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14173:19
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCallTwo@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13407:9
02-13 13:53:36.555 10746 21171 W ReactNativeJS: doResolve@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13571:25
02-13 13:53:36.555 10746 21171 W ReactNativeJS: Promise@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13430:14
02-13 13:53:36.555 10746 21171 W ReactNativeJS: callInvokeWithMethodAndArg@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14172:29
02-13 13:53:36.555 10746 21171 W ReactNativeJS: enqueue@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14177:157
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14112:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: async@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14193:69
02-13 13:53:36.555 10746 21171 W ReactNativeJS: getConcurrentJobs@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81614:40
02-13 13:53:36.555 10746 21171 W ReactNativeJS: start$@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81512:71
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCatch@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14069:23
02-13 13:53:36.555 10746 21171 W ReactNativeJS: invoke@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14242:32
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14112:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCatch@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14069:23
02-13 13:53:36.555 10746 21171 W ReactNativeJS: invoke@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14145:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14173:19
02-13 13:53:36.555 10746 21171 W ReactNativeJS: tryCallTwo@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13407:9
02-13 13:53:36.555 10746 21171 W ReactNativeJS: doResolve@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13571:25
02-13 13:53:36.555 10746 21171 W ReactNativeJS: Promise@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:13430:14
02-13 13:53:36.555 10746 21171 W ReactNativeJS: callInvokeWithMethodAndArg@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14172:29
02-13 13:53:36.555 10746 21171 W ReactNativeJS: enqueue@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14177:157
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14112:30
02-13 13:53:36.555 10746 21171 W ReactNativeJS: async@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:14193:69
02-13 13:53:36.555 10746 21171 W ReactNativeJS: start@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81478:40
02-13 13:53:36.555 10746 21171 W ReactNativeJS: createJob@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81468:21
02-13 13:53:36.555 10746 21171 W ReactNativeJS: http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81331:26
02-13 13:53:36.555 10746 21171 W ReactNativeJS: getQueue@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81311:9
02-13 13:53:36.555 10746 21171 W ReactNativeJS: add@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:81330:17
02-13 13:53:36.555 10746 21171 W ReactNativeJS: startPublishJob@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:70845:44
02-13 13:53:36.555 10746 21171 W ReactNativeJS: touchableHandlePress@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:37523:47
02-13 13:53:36.555 10746 21171 W ReactNativeJS: touchableHandlePress@[native code]
02-13 13:53:36.555 10746 21171 W ReactNativeJS: _performSideEffectsForTransition@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:36807:36
02-13 13:53:36.555 10746 21171 W ReactNativeJS: _performSideEffectsForTransition@[native code]
02-13 13:53:36.555 10746 21171 W ReactNativeJS: _receiveSignal@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:36740:46
02-13 13:53:36.555 10746 21171 W ReactNativeJS: _receiveSignal@[native code]
02-13 13:53:36.555 10746 21171 W ReactNativeJS: touchableHandleResponderRelease@http://localhost:8081/index.delta?platform=android&dev=true&minify=false:36625:26

If you have old jobs you can flush them much easier than doing a full reinstall by simply calling
queue.flushQueue(); // flush all jobs.

or in your case,
queue.flushQueue('jobPublishLocation'); // only flush jobPublishLocation jobs.

Link to code doc: https://github.com/billmalarky/react-native-queue/blob/master/Models/Queue.js#L381
I'm going to close this, as the intended functionality is to throw error and fail if a job is pulled off the queue that has no worker assigned to it. I really don't want this type of thing to silently fail since that opens the door to some pretty critical bugs.

the issue wasn't that I pulled a job that had no worker assigned, the issue
was that I pulled a job that had a worker assigned and still saw the
error for a different job. I agree that the former scenario should not fail
silently, but to crash an app because there is any dangling job in the DB
might be too aggressive in a production environment (when our app gets
there).
On Feb 14, 2018 12:19 PM, "Reid Mayo"<EMAIL_ADDRESS>wrote:

If you have old jobs you can flush them much easier than doing a full
reinstall by simply calling
queue.flushQueue(); // flush all jobs.
or in your case,
queue.flushQueue('jobPublishLocation'); // only flush jobPublishLocation jobs.
Link to code doc: https://github.com/billmalarky/react-native-
queue/blob/master/Models/Queue.js#L381
I'm going to close this, as the intended functionality is to throw error
and fail if a job is pulled off the queue that has no worker assigned to
it. I really don't want this type of thing to silently fail since that
opens the door to some pretty critical bugs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/billmalarky/react-native-queue/issues/7#issuecomment-365732673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGk7qkzoYZDJ-0vxFSP2YoK92B_0BcQks5tUz_rgaJpZM4SEj26
.

thanks for the debugging tips!

Hmm, I'm definitely seeing your point. Perhaps it should console.error() instead of throw fatal error.
That way it will redbox in dev environment, which would serve to alert engineering and force them to fix the issue, but silently fail in production.
Or, even better, what are your thoughts on catching all errors that occur in the library, and then emitting them off the queue object? This is what the popular node kue library does. It forces best practices in production (ie doing something with bugs/errors not silently ignoring), but still allows for silent failure in prod.
https://github.com/Automattic/kue#error-handling

wow, this is a great idea (kue error handling). Especially if I can distinguish between different error types: job failures, db errors, library errors, etc. That way, I can dispatch relevant redux actions to update the UI. I have been thinking about how to notify users if a job failed after all retry attempts, I wonder if this can help address that...

Just fyi, I looked into this again and with the current logic if a job is pulled off the queue and doesn't have a worker assigned to it it really does break the queue logic so I'm going to keep that fatal thrown error (recall we had discussed using console.error or console.warn to silently error in prod).

the issue wasn't that I pulled a job that had no worker assigned, the issue
was that I pulled a job that had a worker assigned and still saw the
error for a different job.

When you started up the queue it pulled the "next" job off the top (depending on how your ordered it via priority etc), the "next job" at some point was an old job that you no longer were actually using and thus had no worker assigned to it, but it was still in the realm database. This is why flushing the queue resolved the issue entirely.
I've investigated building the queue error event logic, but currently it will provide little value as the only place that throws errors that are uncaught are places where an "error" situation truly is breaking. I think event logic will be valuable at some point in the future but I'm trying to prioritize development and I think what you say next is higher priority:

I have been thinking about how to notify users if a job failed after all retry attempts, I wonder if this can help address that

^ This seems like functionality that is needed more than the event logic for now.
Seems like the best solution for this type of functionality is allowing an "onFailure" handler function to be supplied as a worker option. This would be a function that is called with the job id and job data when a job failed after all retry attempts. For that matter, it would probably make sense to have a series of optional lifecycle handlers like "onSuccess", "onFailure", "onComplete" so you can easily hook logic in wherever you want during the job lifecycle.
I imagine it would look something like this in practice:
queue.addWorker('job-name-here', (id, payload) => { console.log(id); }, {

  // Set max number of jobs for this worker to process concurrently.
  // Defaults to 1.
  concurrency: 5,

  onSuccess: function(id, payload) {
    console.log('job ' + id + ' finished processing successfully'); // Sure code that you would put here could be simply placed at the end of the standard "worker" handler, but breaking it out into an onSuccess handler is a better place for logic unrelated to the actual job process itself, like code that updates UI by dispatching a redux action or something
  },
  onFailure: function(id, payload) {
    console.log('job ' + id + ' failed'); // Not sure if this should fire with every failed attempt or only if all attempts fail and the failure attempt limit is hit. Maybe it makes sense to have an "onFail" hook that fires on each fail and an "onFinalFailure" or something that fires when the job has failed enough times to reach the max attempts limit.
  },
 onComplete: function(id, payload) {
    console.log('job ' + id + '  complete'); // This fires when a job is marked as complete after it either succeeds or fails enough times to hit max attempts limit. IE this is similar to the [.finally()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/finally) promise method logic.
  },

});

Thank you so much for looking into this. onComplete sounds like a great
idea, along with the other callbacks. Taking it in the call to addWorker
should work very well I think.

Akshay

On Wed, Mar 7, 2018 at 9:29 AM, Reid Mayo<EMAIL_ADDRESS>wrote:

Just fyi, I looked into this again and with the current logic if a job is
pulled off the queue and doesn't have a worker assigned to it it really
does break the queue logic so I'm going to keep that fatal thrown error
(recall we had discussed using console.error or console.warn to silently
error in prod).
the issue wasn't that I pulled a job that had no worker assigned, the issue
was that I pulled a job that had a worker assigned and still saw the
error for a different job.
When you started up the queue it pulled the "next" job off the top
(depending on how your ordered it via priority etc), the "next job" at some
point was an old job that you no longer were actually using and thus had no
worker assigned to it, but it was still in the realm database. This is why
flushing the queue resolved the issue entirely.
I've investigated building the queue error event logic, but currently it
will provide little value as the only place that throws errors that are
uncaught are places where an "error" situation truly is breaking. I think
event logic will be valuable at some point in the future but I'm trying to
prioritize development and I think what you say next is higher priority:
I have been thinking about how to notify users if a job failed after all
retry attempts, I wonder if this can help address that
^ This seems like functionality that is needed more than the event logic
for now.
Seems like the best solution for this type of functionality is allowing an
"onFailure" handler function to be supplied as a worker option
https://github.com/billmalarky/react-native-queue#worker-options. This
would be a function that is called with the job id and job data when a job
failed after all retry attempts. For that matter, it would probably make
sense to have a series of optional lifecycle handlers like "onSuccess",
"onFailure", "onComplete" so you can easily hook logic in wherever you want
during the job lifecycle.
I imagine it would look something like this in practice:
queue.addWorker('job-name-here', (id, payload) => { console.log(id); }, {
// Set max number of jobs for this worker to process concurrently.
// Defaults to 1.
concurrency: 5,
onSuccess: function(id, payload) {
console.log('job ' + id + ' finished processing successfully'); // Sure code that you would put here could be simply placed at the end of the standard "worker" handler, but breaking it out into an onSuccess handler is a better place for logic unrelated to the actual job process itself, like code that updates UI by dispatching a redux action or something
},
onFailure: function(id, payload) {
console.log('job ' + id + ' failed'); // Not sure if this should fire with every failed attempt or only if all attempts fail and the failure attempt limit is hit. Maybe it makes sense to have an "onFail" hook that fires on each fail and an "onFinalFailure" or something that fires when the job has failed enough times to reach the max attempts limit.
},
onComplete: function(id, payload) {
console.log('job ' + id + '  complete'); // This fires when a job is marked as complete after it either succeeds or fails enough times to hit max attempts limit. IE this is similar to the .finally() promise method logic.
},
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/billmalarky/react-native-queue/issues/7#issuecomment-371216565,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGk7o08XAJOvO03xmWCV83EBZ_-1kbNks5tcBjogaJpZM4SEj26
.

Daku,
Thanks for your feedback.
The job lifecycle callback functionality has now been added via this PR:
https://github.com/billmalarky/react-native-queue/pull/14
Functionality now exists in react-native-queue version 1.2.0, and the new functionality has been published to npm.
For more info on the new functionality, take a look here:
https://github.com/billmalarky/react-native-queue#options-and-job-lifecycle-callbacks

Thank you, this is great!

i have face the same issue and resolved it after spending 3 days hitting my head on the table. There was very silly bug in my worker id. I created workers manually but I create my job dynamically, there was a space in my worker id; which i saved in my local storage.
Example :
queue.addWorker("create-user", aync ({ id, payload }) => { //code goes here } ); // it is fine
problem was here
this.state.queue.createJob(jobId); // here jobId was culprit because it was coming from other source and had the space in the ending :-(
jobId = 'create-user   '; // it was something like that. So double check your id


Remove recent peer dependency bumps

Explanation

In a previous release, some packages received new versions, and corresponding entries in dependencies, devDependencies, and peerDependencies were bumped to match. However, for packages that only received minor or patch bumps, it was not necessary to bump the peer dependencies. (These bumps may have been required by a rule that either the release tool or the core repo enforces.) Reverting these changes avoids having to create major releases for the packages in which these peer dependencies were bumped.
References

Changelog

(Updated in PR)
Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

I opted to keep all of these peer dependency bumps in https://github.com/MetaMask/core/pull/4342, so this PR is now moot.


fix(copy): 修复rowdata可能为null的情况

👀 PR includes

✨ Feature

[ ] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[x] Solve the issue and close #0

🔧 Chore

[ ] Test case
[ ] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
用户传入的数据可能是null，所以这里做一下兼容
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.19.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Closes #62 advs template add interpolation functions

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md under the header # admiralpeds (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[ ] Build admiralpeds site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the Reference page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

I am currently unable to run pkgdown::build_site() with the default value NULL defined in derive_interp_records()

I am currently unable to run pkgdown::build_site() with the default value NULL defined in derive_interp_records()

I finally could run pkgdown::build_site()but do I need to call library(admiral) in addition to dplyr for the below error?

I finally could run pkgdown::build_site()but do I need to call library(admiral) in addition to dplyr for the below error?

sure, add library(admiral, warn.conflicts = FALSE) and see if that resolves

@zdz2101 , @rossfarrugia : I am still facing the error with by_vars since using assert_vars() in the example.
Could you please suggest any solution ? I have unsuccessfully worked around the whole afternoon.

Thanks.
All other comments are implemented. Please let me know if there are further comments.

@zdz2101 , @rossfarrugia : I am still facing the error with by_vars since using assert_vars() in the example. Could you please suggest any solution ? I have unsuccessfully worked around the whole afternoon.

@bundfussr can you take a look at this PR please and the error when we run pkgdown::build_site(). It's something to do with the assertions as by_vars is allowed to be NULL. Appreciate you sanity checking as you'd probably find the issue a million times quicker than me.

@zdz2101 , @rossfarrugia : I am still facing the error with by_vars since using assert_vars() in the example. Could you please suggest any solution ? I have unsuccessfully worked around the whole afternoon.

@bundfussr can you take a look at this PR please and the error when we run pkgdown::build_site(). It's something to do with the assertions as by_vars is allowed to be NULL. Appreciate you sanity checking as you'd probably find the issue a million times quicker than me.

Adding library(rlang) at the beginning should solve the error when we run pkgdown::build_site().

By the way, superseded dplyr functions like group_by_at() and do() are used in the code. I wonder if this should be avoided.

And to be in line with the admiral messaging cli functions should be used for messages, e.g., cli_abort() instead of stop().

@Fanny-Gautier appreciate if you can look into Stefan's 2 comments above and undo my bad suggestions above - sorry 😞 and then hopefully we can get this one merged!

Could you please let me know how to handle the below "error" ?
When running styler, it set back my lines to 6 spaces, while Lintr requires 21 spaces. So when it fits to Lintr this makes a styler error, and when it fits to styler, it makes a Lintr error.

Could you please let me know how to handle the below "error" ? When running styler, it set back my lines to 6 spaces, while Linter requires 21 spaces. So when it fits to Linter this makes a styler error, and when it fits to styler, it makes a Linter error.

Yes, this is a known issue. Please add indentation_linter=NULL, to the .lintr file. This should solve the issue.

@Fanny-Gautier i ran through the ADVS template and the results look good. i also checked test coverage using covr and your new function is 100% covered. so i'll approve/merge this now so that i can merge into my open PR.
@bundfussr if you spot any further code improvements for this one please can you make an issue for us and we can tackle separately to this PR.


Sokol nuklear - issue with rendering nk_image

Hi Hi,
I've got an issue while trying to build a nuklear gui with sokol app, sokol nuklear and sokol gfx. Not sure if this is a bug or expected/intended behaviour, so I'd like to describe it and my solution as best as I can and would like to hear some comments or advise:
My Platform is Ubuntu 20.4, I build with fips and config sapp-linux-ninja-debug ( from the sokol examples).
So in my GUI I would like to render some 2d images as ng_image, for now just static images but goal is to render fast changing images (not every frame but likely changing at 10-20Hz).
So in sokol app init() I create a global nk_image with:
sg_image img = sg_make_image(...);
image = nk_image_id(img.id);

and in frame() I add it to the GUI with
...
nk_layout_row_dynamic(ctx, 400, 1);
nk_image(ctx, image);
...

No matter how my image looks like I always got this result (The image is supposed to be displayed under the buttons):

I tried to figure out where this image comes from and also tried a lot of workarounds in my code with no sucess.
Then I started stepping true the actual rendering code with the debugger and figured out that in sokol_nuklear.h in snk_render() in the nk_draw_foreach loop for every cmd  sg_apply_bindings is called with the texture _snuklear.img , which is the font atlas in the global sokol nuklear state object. This texture is bound even when the cmd brings it's own texture handle which is the case for cmd converted from nk_image
Here's a screenshot of the situation from my debugger (cmd->texture.id != _snuklear.img.id):

Changing sg_apply_bindings to bind the texture from the cmd in case it differs from _snuklear.img and is not zero solves my problem and the image is rendered and displayed correctly.
So what do you think? Is this a bug or am I missing something? This are my first tries on 3d rendering backends so I actually still quite often have the feeling I don't know exactly what I am doing. ;-)
Best
Andreas

Ok this should be fixed now. I also wrote a minimal sample/test here: https://github.com/floooh/sokol-samples/blob/master/sapp/nuklear-images-sapp.c
..sample webpage on the way as soon as CI is green.

Thanks for the fast fix and confirmation, it's much appreciated!


Windows: Fix arch detection via VCTOOLSINSTALLDIR if not first in PATH

Trying to run invoke scons on Windows with VCTOOLSINSTALLDIR set (as is the case in the default VS / GDK command prompts) resulted in the following error for me:
KeyError: 'bin':
  ...
  File "C:\git\godot\.\platform\windows\detect.py", line 144:
    return msvc_target_aliases[first_path_arch]

Turns out we have this piece of code in the Windows platform detect.py, trying to determine the arch:
https://github.com/godotengine/godot/blob/6b281c0c07b07f2142b1fc8a6b3158091a9b124c/platform/windows/detect.py#L141-L146
Here is what this tries to do:

It checks if VCTOOLSINSTALLDIR is set
(which for me has a value like C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\).
if so, it appends BIN\HOST to that value and searches for it in PATH and if found, saves the starting index of that in PATH.
It should find a value like C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\bin\Hostx64\x64 in PATH; it now tries to extract the arch bit in ...\Hostx64\... (x64).

However, there are 2 errors here:

Line 145 does not actually use the found index in the PATH. os.getenv("PATH").split(";")[0] just gets the first element of PATH, then .rsplit("\\", 1)[-1] cuts parts of. For me, this resulted in a value of bin instead of x64, as VCTOOLSINSTALLDIR was not the first in PATH. Changing it to os.getenv("PATH")[host_path_index:]... to actually start at the found index fixes this and resolves the error.
There is a missing guard to check whether the found arch actually exists in the dict before returning it. All other similar places in the file have such guards. So, I added one here with if first_path_arch in msvc_target_aliases.keys(), which would at least have prevented this error.

This seems to have been introduced in https://github.com/godotengine/godot/pull/64921.

Thanks!


Update for babel 6?

Can you update this for the babel 6 workflow?

Sure. Give me some time, i'm making dinner right now...
On Jan 15, 2016 4:02 PM, "Corey Frang"<EMAIL_ADDRESS>wrote:

Can you update this for the babel 6 workflow?
—
Reply to this email directly or view it on GitHub
https://github.com/anvk/babel-npm-module-example/issues/2.

Was trying to set this up myself:
[16:03:07] Failed to load external module babel-core/register
[16:03:07] Failed to load external module babel/register
[redacted]/gulpfile.babel.js:1
(function (exports, require, module, __filename, __dirname) { import gulp from
                                                              ^^^^^^
SyntaxError: Unexpected reserved word
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Liftoff.handleArguments (/usr/local/lib/node_modules/gulp/bin/gulp.js:116:3)
    at Liftoff.<anonymous> (/usr/local/lib/node_modules/gulp/node_modules/liftoff/index.js:192:16)
    at module.exports (/usr/local/lib/node_modules/gulp/node_modules/liftoff/node_modules/flagged-respawn/index.js:17:3)

Not quite sure what I'm missing.

Oh hey - found it with npm instal --save-dev babel-core

Resolved in 0.0.6. Please give it a try and let me know if it works for you.


Invalid map data in MoveToMapPokemon task

Expected Behavior
Read valid map data
Actual Behavior
I'm using latest dev branch, bot is actually giving this error
[15:07:20] Moving towards pokestop San Palacio - 0.05km
[15:07:20] Arrived at fort.
[15:07:20] Failure! Map data was not valid
[15:07:23] Discarded 1x Potion (maximum 0).

Your config.json (remove your credentials and any other private info)
{
    "auth_service": "google",
    "username": "XXXXX",
    "password": "XXXXX",

    "location": "45.464274, 9.189034",

    "gmapkey": " XXXXXX",
    "tasks": [
        {
            "type": "HandleSoftBan"
        },
        {
            "type": "CollectLevelUpReward"
        },
        {
            "type": "IncubateEggs",
            "config": {
                "longer_eggs_first": true
            }
        },
        {
            "type": "TransferPokemon"
        },
        {
            "type": "EvolvePokemon",
            "config": {
                "evolve_all": "Pidgey,Caterpie,Rattata,Weedle,Zubat,Goldeen,Krabby,Abra,Paras,Psyduck,Staryu,Spearow,Tentacool,Ekans",
                "first_evolve_by": "cp",
                "evolve_above_cp": 10,
                "evolve_above_iv": 0.1,
                "logic": "and",
                "evolve_speed": 20,
                "use_lucky_egg": false
            }
        },
        {
            "type": "RecycleItems",
            "config": {
                "item_filter": {
                    "Pokeball":     {"keep": 80},
                    "Greatball":    {"keep": 50},
                    "Ultraball":    {"keep": 50},
                    "Potion":       {"keep": 0},
                    "Super Potion": {"keep": 0},
                    "Hyper Potion": {"keep": 0},
                    "Revive":       {"keep": 0},
                    "Razz Berry":   {"keep": 40}
                }
            }
        },
        {
            "type": "CatchVisiblePokemon"
        },
        {
            "type": "CatchLuredPokemon"
        },
        {
            "type": "SpinFort"
        },
        {
            "type": "MoveToFort",
            "config": {
                "lure_attraction": true,
                "lure_max_distance": 2000
            }
        },
        {
            "type": "MoveToMapPokemon",
            "config": {
                "address": "http://localhost:5000",
                "max_distance": 500,
                "min_time": 60,
                "min_ball": 10,
                "map_path": "raw_data",
                "prioritize_vips": true,
                "snipe": true,
                "snipe_high_prio_only": true,
                "snipe_high_prio_threshold": 400,
                "update_map": false,
                "mode": "priority",
                "catch": {
                    ....
                }
            }
        }
    ],

Steps to Reproduce
With some difficulties I installed pokemongo map (pokeminer now) and was now I can see the map at localhost:5000. But then I get that error in the bot
Other Information
OS: OS X El Capitan
Git Commit: 67fe00bcfc04fb6ddd2c78ca208aad4e4cbf2523
Python Version: Python 2.7.11

@iruy are you still having this issue with the latest version?

i get a similar error too
config
{
{
"type": "MoveToMapPokemon",
"config": {
"address": "http://localhost:5000",
"max_distance": 500,
"min_time": 60,
"min_ball": 50,
"prioritize_vips": true,
"snipe": false,
"snipe_high_prio_only": false,
"snipe_high_prio_threshold": 200,
"update_map": false,
"mode": "priority",
"map_path": "raw_data",
"catch": {
i get the error message
[14:56:46] Moving towards pokestop G Ross Lord Park Service Yard  - 0.14km
[14:56:48] Failure! Could not get Pokemon data from PokemonGo-Map: http://localhost:5000. Is it running?
[14:56:48] Moving towards pokestop G. Ross Park Sports Pavilion - 0.15km
[14:56:50] Failure! Could not get Pokemon data from PokemonGo-Map: http://localhost:5000. Is it running?
[14:56:50] Moving towards pokestop G Ross Lord Park Service Yard  - 0.14km
[14:56:51] Failure! Could not get Pokemon data from PokemonGo-Map: http://localhost:5000. Is it running?
[14:56:51] Moving towards pokestop G. Ross Park Sports Pavilion - 0.14km
[14:56:54] Failure! Could not get Pokemon data from PokemonGo-Map: http://localhost:5000. Is it running?
[14:56:54] Moving towards pokestop G Ross Lord Park Service Yard  - 0.14km
[14:56:56] Failure! Could not get Pokemon data from PokemonGo-Map: http://localhost:5000. Is it running?

@trademarrk that's not the same issue, are you running PokemonGo-Map?


first setup: bad video quality, distortion, packet loss, headtracking laggy

hi. ive just established the connection Ogo-pc and i get problems as mentioned above:
loss rate 30%, ca. 20 packets/s,total delay >400ms
ive already increased the buffer to 2MB but no change. also tried different codec etc, no change (after closing all apps/restarting).
has anyone used this method with the go for iracing etc?
any help is greatly appreciated. ^^
PS: used this guide: https://www.windowscentral.com/how-play-steamvr-games-oculus-go

It is perhaps an issue of Wi-Fi.
If you are using 2.4GHz Wi-Fi, try 5GHz.
And lowering bitrate may mitigate the problem.

pleasure meeting you ^^ thanks for this great app. changing to 5 GHz worked. what other wifi settings do you recommend, eg. bandwidth, mode (n,a,ac), bitrate ,multicast rate?
whats the difference between the codecs?
go supports 2560 right? so i should be able to increase the qulity? what bitrate?i got a 500Mbit connection.
higher buffer size increases latency? thx

Use AC, it's the newer and faster standard.
H.265 compared to H.264 codec provides about double the image quality at the same bit rate.
Try finding the sweet spot between image quality (bit rate) and latency.


Updated qmlui catalan and spanish translations

Updated qmlui catalan and spanish translations.

coverage: 28.069% (+0.002%) from 28.067%
when pulling 9381b8e24141671f976abe7d303dca72cdaa06e9 on sbenejam:master
into 2c52fef7466a0f8a84259d1208b16f5276c82521 on mcallegari:master.

Lightning fast, thanks! :+1:


update apiVersion

Fixes APIVersion for RouteTable now to reflect newer provider versions.
Fixes #49
Signed-off-by: Steven Borrelli<EMAIL_ADDRESS>Tested by creating a network using examples/network.yaml.
$ kubectl get network,compositenetwork,routetable
NAME                                            READY   CONNECTION-SECRET   AGE
network.aws.platformref.crossplane.io/network   True                        112s

NAME                                                           READY   COMPOSITION                                       AGE
compositenetwork.aws.platformref.crossplane.io/network-7qgms   True    compositenetworks.aws.platformref.crossplane.io   112s

NAME                                                   READY   SYNCED   ID                      VPC                     AGE
routetable.ec2.aws.crossplane.io/network-7qgms-fkr4p   True    True     rtb-03845ac6d12449e2e   vpc-08750e7a040c90f33   110s

@stevendborrelli I can merge this PR and cut a new release for this reference platform, and get it published to all our dev/staging/prod environments.  this is still a manual process, so I have captured an issue to automate it in https://github.com/upbound/solution-eng/issues/5.
Do you want to shadow while I do this so you can learn the process better? :)


One UsersDetailsEntity cannot be cast to another UsersDetailsEntity

My parent project contains two separate SpringBoot authorization services that refer to the entity objects of the same project, both of which inherit UsersDetails. Report errors. How to specify that authorization service uses that UsersDetails object

entities.users.UserEntity cannot be cast to entities.role.UserAdminEntity

So, the error message looks like it is specific to your project - for some reason, the application is trying to cast one of your custom domain objects to another.
This would probably be better for StackOverflow as you get access to the entire community. Would you please post your question there? It would probably also be helpful to include your existing security configuration and even a sample that demonstrates the problem.

Also, feel free to link your SO question here to the ticket.


Auto-update and Sync profile has stopped working

Bug report
Your Environment

Netflix add-on version: 1.2.2
Operating system version/name: Windows 7
Device model:

Used Operating system:

[ ] Android
[ ] iOS
[ ] Linux
[ ] OSX
[ ] Raspberry-Pi
[x] Windows

Describe the bug
Auto-updating and synchronization has stopped working.  TV shows have to be manually updated and it is no longer possible to export movies to a local library

Expected behavior
Adding movies and/or TV shows to the Netflix "My List" should result in them being synced to a local library automatically.

Actual behavior
Adding movies and/or TV to the Netflix "My List" results in no change to the local library after a day, a week, or a month (so far).

Steps to reproduce the behavior

Add content to the Netflix "My List"
Observe that it's never atuo-synced to your local library

Possible fix

Debug log

The debug log can be found from this link:
https://pastebin.com/7Kg3VhKD
Additional context or screenshots (if appropriate)
Installation

[x] I'm using other Netflix Repo
[ ] I'm using a different source

Other information

Screenshots

the log is useless without netflix debug verbose

I think I found the problem anyway
try this and let me know:
plugin.video.netflix_1.2.2_20200512_t1.zip

Sorry about the log-level, I'll make note of that for future tickets.
Prior to me installing the update, it looks like a movie was synced to my library, but still missing new TV episodes.  I've installed the update in the zip you attached, set my update time to 5pm EDT, and have turned on verbose logging, I'll update the ticket with the results later.

The issue seems to have been fixed with the update, both movies and shows that I've added to My List started showing up in my local library again.  Here is a combo of my kodi.old.log and kodi.log, the former should show an auto-sync of My List to the library and the latter should show a manual update.

I'm actually still seeing weird issues, there's a show called "The Hollow" that just had it's second season come out.  I can see the second season in the Netflix lists, but all of the second season episodes are refusing to export.  Here's a verbose log file of my starting up Kodi, navigating to the relevant show in Netflix, and attempting to manually export new episodes.

i think was a cache issue,
i can not reproduce to my local pc
can you try?
plugin.video.netflix_1.2.2_20200514_t2.zip

That appears to have fixed the issue, the second season episodes were exported