Updating tables

I am not sure if this is possible with the current release but does vpin studio allow you to update the table in pinup popper.
Popper has an update button in game manager that allows you to drag a new table to it and it will update all the files for that table to match. Does your program offer this? I would love to be able to do this remotely.
Thanks

There are several way to update your table and it's data:

You can use the upload button from the toolbar to upload a new VPX file of an existing table. You can also upload new ones or clone existing ones.
Double-clicking a table opens the Popper Data dialog. You can edit the table's metadata there.
Additionally, you can use the "auto-fill" function in the VP-Spreadsheet (VPS) section. When the table has a mapped VPS entry, the VPS data and the VPX files inner metadata is used to auto-fill the table data of Popper. Make sure you have enabled the flag in the UI preferences for this!

I hope this is what you looking for.
I wanted to make a "Getting Started" video about this, but did not find the time yet.

You will also find help on my Discord: https://discord.com/invite/69YqHYd3wD
I'm online quite often and do usually faster respond there.

Ups, seems the link expired, here is the new one: https://discord.gg/AMPwvAZHvy
I'll close this ticket for now. Don't hesitate to write if you miss something or got ideas for improvements.


Unpublish Page

Using django-cms v 2.3.5
I am having issues with my pages still being publicly visible, despite the page status being updated to unpublished.
I have verified the page published attribute updates correctly in the database via both mysql and the python interpreter. When the page is refreshed after updating, the edit page form shows the correct value. However, the page is always visible (for non-logged in users).
Have you got any pointers as to why this might be happening? Is there something I have missed?

I hope you got to the bottom of this. We're no longer developing 2.4, so I am going to close this issue.


Step 3 of Codelab mentions 3 folders in zip file when there are only 2

Step 3 of Codelab mentions 3 folders in zip file when there are only 2:s
washer-start
washer-done

Thank you for finding this. This will be fixed soon.


Doubts

Is pcf console available?
Is there a way to install services in the market place?
If we want to bind to rabbitmq, redis services, how to do that in this case? Is set-env for an application work?

Note: Please label this entry as questions. I don't see an option to do it myself.

Thanks for your interest @pradp

No console is available yet.  This is planned in the future.
There isn't a way to install services from the marketplace.  We're currently working on adding Redis, Mysql and RabbitMQ as available services by default.
I'd recommend using cf create-user-provided-service to connect your app to the instance of Rabbit or Redis you have running on your machine.  That should avoid having to do a manual cf set-env.

More info on cf cups can be found at https://docs.cloudfoundry.org/devguide/services/user-provided.html

@cf-gitbot I tried to follow the issue in Pivotal Tracker but it seems we need some kind of privileges? I got a not allowed error. I was interested in follow the progress of adding some services by default to micropcf.

@davidwadden Thanks for your reply. I need clarification on user provided service for rabbitmq or redis.
How do i create it to make it work automatically with spring boot auto configuration? I have posted in stack overflow too. The issue involves spring boot in specific but still wanted to check with you on this.
http://stackoverflow.com/questions/35065403/how-to-create-a-user-provided-redis-service-which-spring-auto-configuration-clou
This is much needed for my usecase to use micropcf without any code change in my application as well as to keep redis/rabbitmq service outside the VM and use less memory.

I created user provided service for redis and the spring  connector picked it up but there is issue with application connecting to the redis server running in local machine. I will open another issue as it is nothing to do with this one. Thanks for clarifying my doubts.


Add link to full bug check code reference to BSOD troubleshooting guide

Not sure if my changes match the style. But the link should be useful.
Pull request guidance
Thank you for your contribution. The content team is monitoring the pull requests. Once your pull request is ready for review, please type "#sign-off" in a new comment or add a comment such as "PR is ready for review and merge."

@Susko3 The change is live at https://learn.microsoft.com/en-us/troubleshoot/windows-client/performance/stop-error-or-blue-screen-error-troubleshooting#common-windows-stop-errors. This PR will be closed.


issue #3317 resolved. Update required documentation

Now we will be able to add resources and file by using '.' as well as '/'.

This PR seems to lack tests altogether. They need to be added before it can be merged or even more thoroughly reviewed.
Unfortunately this is too late for RF 3.2.

Can you please tell me where I should add test cases. Either in test_importer_util.py or in test_imports.py

Most importantly some acceptance tests are needed.

I close this PR because there has been no activity since 2019. The original issue #3317 is valid so I leave it open. Notice that implementing it probably wouldn't require changes in the code that was changed in this PR.


Output gffs empty if input fasta has bases in lowercase

6/66 assemblies had no genes annotated in the output gff and only the header and sequence-region. However, they had the expected number of genes in the gene presence-absence table.
These were the only 6 fastas where the sequence were in lower case.

Thanks for troubleshooting this! This should be rectified in commit fe65d60.


Issues with LLaMA model and constraints

Hello,
I have encountered an issue when running this code, where I am running my code on the LLaMA model.
sample(temperature=0.8)
   "A list of things not to forget when going to the sea (not travelling, and choose Bathing Suite as the first one): \n"
   "- Sunglasses \n"
   for i in range(4):
      "- [THING] \n"
from
   '/riemann/hugging'
where
   THING in set(["Volleyball Shoes", "Bathing Suite"])

The output ends up being just Bathing, and then continuing to run until it ends with just "Bathing".

As mentioned in the Discord, I had a look. Seems like the HF LlamaTokenizer behaves different from other tokenizers in ‚transformers‘ when it comes to decoding tokens with leading spaces. Will need a fix. I thought this was fixed on their end, but apparently not yet. It will need some special handling from our side then. Sorry about that. Feel free to create a GH issue.

This was fixed in the latest release, e.g. the following code works with the huggyllama/llama-7b model and tokenizer:
sample(temperature=1.8, chunksize=1)
   "A list of things not to forget when going to the sea (not travelling, and choose Bathing Suite as the first one): \n"
   "- Sunglasses \n"
   for i in range(40):
      "- [THING] \n"
from
   'huggyllama/llama-7b'
where
   THING in set(["Volleyball Shoes", "Bathing Suite"])

Note that HF advises to not use outdated llama models from the hub, as some (slow variant) tokenizer implementation still contained buggy behaviour, which can also affect LMQL.


Add my-name to the list of contributors

I have added my name to the list of contributors.
This work is done the contributors.json file in the data folder

Please resolve the conflict

Conflict resolved successfully.


Set better polyfill replaces

This was mentioned here https://github.com/symfony/polyfill/pull/153#discussion_r224391944 so it seems like a good idea to give the right example.

Makes no diff in practice - and this staled :)


My Android app has performance issus, running slowly sometimes or even halts for a while, but iOS one seem normal based on the same code

Do you have any suggestion? Or any attention I should place when developing Android app using Flutter.
flutter --version is as follows:
Flutter 2.5.3 • channel stable • https://mirrors.tuna.tsinghua.edu.cn/git/flutter-sdk.git
Framework • revision 18116933e7 (6 weeks ago) • 2021-10-15 10:46:35 -0700
Engine • revision d3ea636dc5
Tools • Dart 2.14.4

Hi @jims57 there are several reasons why this may happen (slow network requests, app losing frames), you can use the flutter devtool to find the cause https://docs.flutter.dev/perf/rendering/ui-performance. You can also refer to performance best practices documented here https://docs.flutter.dev/perf/rendering/best-practices.
I'll go ahead and close this now as this is not a bug or feature request. If you disagree, please write in the comments and I will reopen it.


No detection result show in rviz when using custom dataset.

Dear author,
Thank you for your great work!
I want to use openpcdet custom dataset to train my own model. It works well and I get the result and the demo.py shows it is correctly detected.

But when I load the model and run it on ros topic, it can't detect any target.
The possible reason maybe cause by the data transferring. In order to using custom dataset form I transfer the point cloud from pcd to bin and then finally transfer it into npy.
And the label format is:
5.54 0.57 0.41 1.08 0.74 1.95 1.57 Pedestrian.
Here is my bag info:
path:        od_1.bag
version:     2.0
duration:    2:56s (176s)
start:       May 10 2024 06:10:04.21<PHONE_NUMBER>.21)
end:         May 10 2024 06:13:00.89<PHONE_NUMBER>.89)
size:        7.0 GB
messages:    8454
compression: none [1766/1766 chunks]
types:       bond/Status                           [eacc84bf5d65b6777d4c50f463dfb9c8]
diagnostic_msgs/DiagnosticArray       [60810da900de1dd6ddd437c3503511da]
dynamic_reconfigure/Config            [958f16a05573709014982821e6822580]
dynamic_reconfigure/ConfigDescription [757ce9d44ba8ddd801bb30bc456f946f]
rosgraph_msgs/Log                     [acffd30cd6b6de30f120938c17c593fb]
sensor_msgs/LaserScan                 [90c7ef2dc6895d81024acba2ac42f369]
sensor_msgs/PointCloud2               [1158d486dd51d683ce2f1be655c3c181]
tf2_msgs/TFMessage                    [94810edda583a504dfda3829e70d7eec]
velodyne_msgs/VelodyneScan            [50804fc9533a0e579e6322c04ae70566]
topics:      /diagnostics                                                  350 msgs    : diagnostic_msgs/DiagnosticArray
/hesai/pandar                                                1765 msgs    : sensor_msgs/PointCloud2
/rosout                                                        23 msgs    : rosgraph_msgs/Log                     (2 connections)
/rosout_agg                                                     5 msgs    : rosgraph_msgs/Log
/scan                                                        1741 msgs    : sensor_msgs/LaserScan
/tf_static                                                      1 msg     : tf2_msgs/TFMessage
/velodyne_nodelet_manager/bond                               1062 msgs    : bond/Status                           (4 connections)
/velodyne_nodelet_manager_driver/parameter_descriptions         1 msg     : dynamic_reconfigure/ConfigDescription
/velodyne_nodelet_manager_driver/parameter_updates              1 msg     : dynamic_reconfigure/Config
/velodyne_nodelet_manager_laserscan/parameter_descriptions      1 msg     : dynamic_reconfigure/ConfigDescription
/velodyne_nodelet_manager_laserscan/parameter_updates           1 msg     : dynamic_reconfigure/Config
/velodyne_nodelet_manager_transform/parameter_descriptions      1 msg     : dynamic_reconfigure/ConfigDescription
/velodyne_nodelet_manager_transform/parameter_updates           1 msg     : dynamic_reconfigure/Config
/velodyne_packets                                            1751 msgs    : velodyne_msgs/VelodyneScan
/velodyne_points                                             1750 msgs    : sensor_msgs/PointCloud2
.
Is there any influence to detection result? Since in the rosbag, the point cloud message is always sensor_msgs/PointCloud2.
I'm very confused and hope you can help me figure out.
Looking forward to your reply.
A lot of thanks!

Hi Jane,
Could you print out few points in demo.py and also in inference.py where ROS callback to check if the point data is correct?
self.points[:,0] += move_lidar_center  And you can try to comment it out first as it looks like move all point x-axis a little bit.

Based on the pull request, I think maybe solved. should we close it? @Jane-QinJ

Yes, we can close it, thank you for your help again!
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: @.*** @.> 代表 Qingwen Zhang @.>
发送时间: 星期三, 八月 14, 2024 1:24 上午
收件人: OpenPCDet_ros @.>
抄送:<PHONE_NUMBER> @.>; mention @.***>
主题: Re: [Kin-Zhang/OpenPCDet_ros] No detection result show in rviz whenusing custom dataset. (Issue #15)
Based on the pull request, I think maybe solved. should we close it? @Jane-QinJhttps://github.com/Jane-QinJ
―
Reply to this email directly, view it on GitHubhttps://github.com/Kin-Zhang/OpenPCDet_ros/issues/15#issuecomment-2286646765, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKB5TVGRKSYZFGHPCO4JBJLZRIXKJAVCNFSM6AAAAABLII5OQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGY2DMNZWGU.
You are receiving this because you were mentioned.Message ID: @.***>


Optimize image size

Describe the enhancement
Various sub projects (eg: managed-serviceaccount) build separate binary for agent and manager instead of one binary with sub commands. This results in docker image having 2x the size than it should be.
Please consider producing one binary from a repo to reduce image size. This will speed up the release process. This will also simplify supporting arm64 arch.

/assign @zhujian7

Fix PR for the managed service account: https://github.com/open-cluster-management-io/managed-serviceaccount/pull/121

this PR is merged. Close this one. We can create a separated issue for another sub project
/close


Added Firecrawl tools to docs

Complementing the PR on https://github.com/joaomdmoura/crewAI-tools/pull/34

@joaomdmoura realized that this didn't get merged. Some people were having issue finding the docs for it. Could we get this in? Thank you :)

@joaomdmoura could we get this in? Thank you!


Coverage is confusing in watch mode

These two screens say it all:

It's the same code without any changes. It's just that the first time no tests ran (because none changed since last commit), and thus coverage reported 0%.
Should we disable the "lazy run" logic when coverage is requested?

This is why I repeatedly said that using coverage in watch mode isn't very useful. I think it would be smarter to allow coverage only when you run all tests in this mode but not sure how to best signal it.

I definitely agree the behavior is initially confusing, but I think coverage + watch mode does enable a pretty nice workflow after you realize it's only reporting on what ran last. I keep thinking how nice it would be to pair Jest's watch mode with some kind of coverage server that could reload the html report when it changes. No more alt tab and refresh, just write tests and watch the red code change to green code.
One thing that probably should be disabled for coverage + watch is the coverage threshold error though.

Is it possible to display only the coverage of the changed/affected files? That would be really great to see as I'm writing tests. Unsure how possible that is.

Similar question as above, would it be possible to perhaps configure a parameter that controls/overrides collectCoverageFrom so that you can generate coverage for only part of an app?

@moaazsidat Tangentially, today I discovered that, in order to see coverage information when watching a pattern, I had to provide a similar pattern again via --collectCoverageFrom=[similar pattern].
It'd be great if there were some option to show coverage  just for the watched patterns!


[FEATURE] User watch notification settings

This is a superset of #15742.
In GitHub a user can configure the extent they would like to be notified in watched repos. Gitea should add the option at least for "Participating and @mentions", "All Activity", and "Ignore".

The default setting should be configurable as a user setting, repo setting, org setting, and/or config setting.

Maybe duplicate with https://github.com/go-gitea/gitea/issues/11120.


Add oeyoews site link

Thanks @oeyoews – you also need to add the missing comma before your new entry.

Thanks @oeyoews


Different "Name"-column width for both teams in Details

If you open the Details menu for any match, the  "Name"-column width for each Team is set by the longest name in each team. This makes the data comparison between players of different teams kind of awkward as all the rows behind "Name" are shifted from team to team.
Fix: Calculate and set the column width for "Name" from the longest name among all players.

A new major version has been released, closing the issue 🧹


I would call it a letter..

Hi!
It’s not really an issue, just that messaging was disabled in Github a while ago..
I am (among others) a Hololens developer. I got excited when I read your Android-Tensorflow retraining tutorial because I am working on a project that would make possible to learn (and recognize) deaf sign language using the Hololens.
When I read your tutorial, I was thinking, I could just teach the model all the signs, and just pass data (screenshot-recognized label) between the Hololens and a running Android app. The Hololens part is all good, but I hit a little bump while following your tutorial, hence the letter. As a couple others also mentioned, sometimes the directory structure after the build process seems to be different than the one you anticipated. (no android\Assets directory) I tried to grab the assets manually (there is a zip file which contains the model file+labels), but you used a different naming pattern for your model, meaning you also had to modify the internal wiring, to get the connections right.. Anyway, do you have an idea what could’ve gone wrong? It would be quite interesting to see a Hololens recognizing sign language.. I would like to make it work. :)
Best,
Imre

That sounds like a cool project. :-) I'm not sure what went wrong for you, but you could take another look at my tutorial as I've been making some updates?  It seems like the most common problem is that people forget to change between their Docker container and their local machine (I've tried to make it more obvious when this is necessary).  You could also watch the TensorFlow For Poets tutorial to get ideas.

Hi,
I managed to solve it in a different way, though I used some of your ideas. In the end it got more close to the corporate life, I recognize different tools, engines, etc.
You need a slightly different strategy to recognize sign language, as you first need to crop out (/other strategy) the hand from the image. If not, the difference between the sign-classes won't be significantly big.
God job with the tutorial, the ideas helped a lot!


CUDNN_STATUS_ALLOC_FAILED

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no, it's on examples/tutorial/mnist,  mnist_deep.py
Bazel version: N/A
Mobile device: N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): windows 10 x64 latest build
TensorFlow installed from (source or binary): binary prebuilt
TensorFlow version (use command below): 1.11.0rc1
Python version: 3.6.6
CUDA/cuDNN version: 9.0/7.2.1
GPU model and memory: GTX1050Ti GDDR5 4GB
Exact command to reproduce:

python mnist_deep.py
E tensorflow/stream_executor/cuda/cuda_dnn.cc:353] Could not create cudnn handle: CUDNN_STATUS_ALLOC_FAILED

other codes are fine-working as before

no problem with my laptop, which has geforce mx150 GDDR5 2GB

all this happens after I installed geforce driver 411.63
Source code / logs
PS D:\VSWorkspace\tensorflow\tensorflow\examples\tutorials\mnist> python .\mnist_deep.py
WARNING:tensorflow:From .\mnist_deep.py:130: read_data_sets (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use alternatives such as official/mnist/dataset.py from tensorflow/models.
WARNING:tensorflow:From C:\Anaconda3\lib\site-packages\tensorflow\contrib\learn\python\learn\datasets\mnist.py:260: maybe_download (from tensorflow.contrib.learn.python.learn.datasets.base) is deprecated and will be removed in a future versi
on.
Instructions for updating:
Please write your own downloading logic.
WARNING:tensorflow:From C:\Anaconda3\lib\site-packages\tensorflow\contrib\learn\python\learn\datasets\mnist.py:262: extract_images (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future vers
ion.
Instructions for updating:
Please use tf.data to implement this functionality.
Extracting /tmp/tensorflow/mnist/input_data\train-images-idx3-ubyte.gz
WARNING:tensorflow:From C:\Anaconda3\lib\site-packages\tensorflow\contrib\learn\python\learn\datasets\mnist.py:267: extract_labels (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future vers
ion.
Instructions for updating:
Please use tf.data to implement this functionality.
Extracting /tmp/tensorflow/mnist/input_data\train-labels-idx1-ubyte.gz
Extracting /tmp/tensorflow/mnist/input_data\t10k-images-idx3-ubyte.gz
Extracting /tmp/tensorflow/mnist/input_data\t10k-labels-idx1-ubyte.gz
WARNING:tensorflow:From C:\Anaconda3\lib\site-packages\tensorflow\contrib\learn\python\learn\datasets\mnist.py:290: DataSet.init (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future ve
rsion.
Instructions for updating:
Please use alternatives such as official/mnist/dataset.py from tensorflow/models.
Saving graph to: C:\Users\alanp\AppData\Local\Temp\tmpxpinc32v
2018-09-20 22:10:32.455130: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2018-09-20 22:10:32.641187: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1411] Found device 0 with properties:
name: GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.4175
pciBusID: 0000:01:00.0
totalMemory: 4.00GiB freeMemory: 3.30GiB
2018-09-20 22:10:32.659090: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1490] Adding visible gpu devices: 0
2018-09-20 22:10:33.556406: I tensorflow/core/common_runtime/gpu/gpu_device.cc:971] Device interconnect StreamExecutor with strength 1 edge matrix:
2018-09-20 22:10:33.565294: I tensorflow/core/common_runtime/gpu/gpu_device.cc:977]      0
2018-09-20 22:10:33.571150: I tensorflow/core/common_runtime/gpu/gpu_device.cc:990] 0:   N
2018-09-20 22:10:33.578487: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3013 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1050 Ti, pci bus
id: 0000:01:00.0, compute capability: 6.1)
2018-09-20 22:10:36.376573: E tensorflow/stream_executor/cuda/cuda_dnn.cc:353] Could not create cudnn handle: CUDNN_STATUS_ALLOC_FAILED

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Bazel version
Mobile device

@tensorflowbutler  updated

@zheng-xq @zhangyaobit @alextp Could you comment or reassign?

@azaks2 do you know if there is a known issue with this driver version?

I would upgrade to cuDNN 7.3 (7.2 is no longer supported on the NVIDIA site and we are going to downgrade to 7.1.4 for our builds but anything over 7.1 will work).  The 411 driver is really new and likely more aligned with CUDA 10 but I have no data points even on Linux at this time as to combinations I know work.
My advice (with no data points I am sorry):

Upgrade to cuDNN 7.3
Consider downgrading to an older Device Driver like 396 but I realized that is really hard on windows as it tries to keep you on the latest possible version.  I would likely do this as a last resort.

@tfboyd
with cudnn 7.3, same
but thank you anyway, I'd rather stick with ubuntu this moment and leave this pc for a while

For people searching the keyword CUDNN_STATUS_ALLOC_FAILED
and finding this issue. You may check if there is another process allocating the gpu. In my case i got this error on terminal after previously running tensorflow from a jupyter notebook


generic article: nextflow PBS cluster config

As discussed, we need a concise article on how to configure Nextflow for PBS Pro (potentially including some local specifics)

@sci-kai
It is not really obvious to me where I am supposed to put the information specified in this section:
Configuration file for HILBERT and local runs

Is that config priority level 3 (further up in the article) ...
 for adapting to Infrastructure or modifying processes. (3.)

... so into a run.config file?
Also, one shortcoming that I make out in the official docs as well, the syntax looks like a JSON document to me, is it?
If so, it should be made explicit, and does that mean I can name the file run-config.json?
P.S.: any estimate how long it will take to deposit a HILBERT-compatible profile here: github.com/nf-core/configs?


Confirm all FAPI R/W OP w/ MTLS conformance tests are passed by the locally built keycloak

Confirm all FAPI R/W OP w/ MTLS conformance tests are passed by the locally build keycloak that the following issues are resolved (not yet sent as PRs).

KEYCLOAK-15236
KEYCLOAK-14195
KEYCLOAK-14204
KEYCLOAK-14205

clarified by https://github.com/jsoss-sig/keycloak-fapi/issues/41

Confirmed that all FAPI R/W OP w/ MTLS conformance tests are passed.
The detailed results can be found on FAPI-SIG Activities Update #3 in JPN


Move code zen to the title menu

When we enter code zen, we don't keep the current position of the page. Therefore, having the code zen button anywhere on the page can be misleading, as soon as we click it, we are "relocated" in the page.
Therefore, I suggest to move "Code zen" to the title menu. Then the next question is: where do we put the button to exit/toggle outputs? One option is to have a small selector at the top (think the search bar on chrome):
                              | [] Show outputs     X Exit code zen |
                              |------------------------------------ |

       > Evaluate
      |---------------------------------------------------|
      |                                                   |
      |                                                   |
      |                                                   |
      |---------------------------------------------------|

We also need to decide if it means we should remove the shortcut too.
Alternatively we try to no relocate so much when entering code zen.

Therefore, I suggest to move "Code zen" to the title menu.

Like this?

Precisely, although I would put it at the top :)

All right, so is this right?
[before enter code zen]

[after enter code zen]

I think the toggle and exit code zen should be outside of the menu and rather at the top of the page. But I would wait for @jonatanklosko's opinion first.

@ByeongUkChoi thanks, I think in this case it's best to just improve the relocation :) Fixed in #1157.


Support leader election for controller and webhook

Signed-off-by: Ruofeng Lei<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does
Fix: #1960
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
For dataset, fluidapp, webhook and all runtimes controller:

add the leader-election-namespace flag to set the namespace leader election resource will be created, the default is fluid-system.
update LeaderElectionID by the component name.

PS. The webhook component is ok to add leader election directly because the webhook service is recognized as the non-leaderelection Runnables by controller-runtime so that all webhook Pod will start the webhook service in the multi-replica scenario.
PPS. There are still some RBAC issue with the leader election, I'll update the helm chart template in next PR.

@abowloflrf Thanks very much for the great contribution!


added dependency check

as requested in issue #42  i have added a dependency check to show the user a friendly message reminding them to update node

@mohammad-aliB - it looks like the tests failed on your PR and also I'm not sure I see the dependenciesMet function.  So I'm going to accept PR #45 instead, but if you'd like to take a shot at another PR, please let me know on the Github Issue, and I'll reserve and assign it for you.

@GantMan id like to take another go at it, and ive already made a pull request

@mohammad-aliB - your solution looks good now, but it has a typo now.  I'm just going to merge #45 -
Let me know if you start another ticket so that no one else completes the same task.  I'm even willing to help you find something easy if you'd like!  Especially considering your existing efforts.

yeah i spelt "dependencies" wrong, my bad. please let me know if you've got any relatively easy tasks for me to tackle though.

@mohammad-aliB - I highly recommend you take a look at #38
Basically you'd remove a for loop, and add a way to grab indexes other than first.

@mohammad-aliB - you can probably knock out #48 in < 5 minutes.


Add email to Identifier and new layer for API access

Thanks for PR! What kind of new functionality does it bring?

Thank you @vasilcovsky !
The PR adds the possibility of sending-receiving messages to/from a Team and at the same time adds email info to the Identifier. We needed it for developing the MS Teams version of AccessBot
Thanks to your contribution, we made it work with very few changes!
We should update the PR during the week and update you here :-)

Thanks again for your contribution to the community. Looking forward to continue the alliance. Keep the good work!


deeplung

您好，有幸读到您发表的集肺结节检测和诊断于一体的文章。我在复现Deeplung中，检测部分代码，训练验证正常跑通，进入测试代码就报内存溢出。测试输入大小是，torch.size(18,1,208,208,208]这个有问题吗？希望您能给与帮助，非常感谢您。

https://github.com/wentaozhu/DeepLung/issues/37

Please read all these reported issues. I am pretty sure there are some solutions for the exactly same issue.
Including this repository https://github.com/uci-cbcl/DeepLung.git


Support progressive delivery for ECS apps using only ECS components

What would you like to be added:
Support ECS apps progressive delivery using only ECS components (without ALB configuration)
Why is this needed:
Currently, to perform progressive delivery for ECS applications, we need users to prepare ALB targetgroups. For some usecases, users' tasks may not directly accept/handle traffic from the ALB but via other tasks/components which leads to the inability to perform progressive delivery strategies.

An idea for this is using 2 services that are configured using the same service registry id, one for the primary tasks and one for the canary tasks. Then tasks of those services will be treated as the same service to other components of users system. 💡

@khanhtc1202
Hi, I have a question.

For some usecases, users' tasks may not directly accept/handle traffic from the ALB but via other tasks/components

Could you give me some examples of such cases?
(Directly calling ECS APIs or IP addresses or something? From where?)
I understand the current flow of canary deployment for ECS with ALB and target groups,
but I'd like to clarify what problem to solve.

I understand the current flow of canary deployment for ECS with ALB and target groups,
but I'd like to clarify what problem to solve.

Thank you for the explanation.
I'll investigate the Service Discovery feature of ECS.
https://docs.aws.amazon.com/ja_jp/AmazonECS/latest/developerguide/service-discovery.html

I share my current situation of consideration.
I think this case assumes currently using ECS Service Discovery, not ECS Service Connect or App Mesh.
There can be three ways which solve the problem, though I'm not sure all of them are possible yet.

Enhance ECS Service Discovery - manipulate Route53??
Use ECS Service Connect - a young feature.
Use App Mesh  - Possible but complicated.

Besides, I'm wondering which is simpler, option-1 or option-2, if both are possible.

@t-kikuc If (2) and (3) are not promising, let's focus on (1). But I don't really understand why we need to manipulate Route53 for that 🤔
AFAIK we can create a copy of the current primary service A (call A') with desiredCount value set to the number of the canary variant's workloads/tasks and use the same serviceRegistryID with primary service A.
In that case, the CloudMap service would ensure that traffic to serviceRegistryID will be routed to both A and A' and we can use that to perform progressive delivery approach, wdyt?

@khanhtc1202
AFAIK we can realize Blue/Green deployment without difficult configurations of Route53,
but Canary --with strict percentage control-- might be difficult.
For instance, when service A has 2 tasks, how can we route traffic to A' for 20%?
Or strictness is not required?
So I'm trying to find out how to use the Weighted Routing of Cloud Map.
(ECS Service Discovery uses Cloud Map in the back, and Cloud Map uses Route53 in the back.)

@t-kikuc in canary deployment strategy pattern, we don't need strict set how many percent route to primary/canary variant, just keep same input route and ensure the number of variants fit percentage, traffic would be forwarding in required percent if the number of requests is enough 👀

@khanhtc1202
Here's the PR of RFC for this issue, including designs:
https://github.com/pipe-cd/pipecd/pull/4676

Thanks @t-kikuc, merged your rfc PR. Let's start implement this feature as your plan 😄

@khanhtc1202
Thank you for your reviewing!
I'll begin to code👍

https://github.com/pipe-cd/pipecd/pull/4690
I found it's so difficult to support Blue/Green for ECS with Service Discovery
because Service Discovery does not support flexible traffic control.

TODO: update feature status page

the flow of canary for Service Discovery is as below:

We will keep working on this by https://github.com/pipe-cd/pipecd/discussions/4709 and close this issue 👍


Add support for timeouts to dsync

Codecov Report

Merging #49 into master will increase coverage by 0.53%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #49      +/-   ##
=========================================
+ Coverage   85.77%   86.3%   +0.53%
=========================================
  Files           3       3
  Lines         232     241       +9
=========================================
+ Hits          199     208       +9
  Misses         27      27
  Partials        6       6

Impacted Files
Coverage Δ

drwmutex.go
85.87% <100%> (+0.75%)
:arrow_up:

Continue to review full report at Codecov.

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

@balamurugana depending on the nature of the time out (eg. encountering a long lock on an object versus not getting read or write quorum), we may actually want to use different time out values (under discussion), so pushing the time out down to newRetryTimerWithJitter would actually make this harder.


chore: Minor usability updates

Codecov Report
Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 97.82%. Comparing base (2c22f42) to head (ca33b68).

Files
Patch %
Lines

src/harfile/__init__.py
83.33%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
- Coverage   99.27%   97.82%   -1.45%
==========================================
  Files           3        3
  Lines         274      276       +2
  Branches       57       56       -1
==========================================
- Hits          272      270       -2
- Misses          2        3       +1
- Partials        0        3       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Specify the default schema --defaultSchemaName=<schema>

Introduce the main feature, support for --defaultSchemaName=<schema> (docs).
Additionally,  test containers have been integrated into bq-liquibase for testing purposes. Please, decide if database containers are useful and to be committed into master.
Ref: #22

Build failure cause is absence of a new bq-jdbc feature allowing to pick the last child of a data source factory in case of linear hierarchy, implemented in PR#71.


Use function types and lambda expressions in Kotlin: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-function-types-and-lambda?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-function-types-and-lambda#5
In which task and step of the codelab can this issue be found?
6. Use the repeat() function, the last step
Describe the problem
Hello. I'm not sure if there's a problem, but I'm pretty confused.
On the last step of [6. Use the repeat() function] it's offered to move the call of the treatFunction() into the repeat() function's lambda expression:
repeat(4) {
treatFunction()
}
This will result in the following output:
5 quarters
Have a treat!
Have a treat!
Have a treat!
Have a treat!
No treats!             // the result of trickFunction() call, it's not interesting right now
At the same time one call of the treatFunction() results in the next output:
5 quarters
Have a treat!
Why only "Have a treat!" is repeated? Looks like 'extraTreat' lamda expression is lost after the 1st repeat() iteration. Why so?
Steps to reproduce?

Go to... Kotlin playground on the link: https://pl.kotl.in/2S46SnuP3
Click on... Run button in the top right corner
See error... The result is:
5 quarters
Have a treat!
Have a treat!
Have a treat!
Have a treat!
No treats!

My expectation the result is to be:
5 quarters
Have a treat!
5 quarters
Have a treat!
5 quarters
Have a treat!
5 quarters
Have a treat!
No treats!
Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.
Please follow the link on Kotlin playground: https://pl.kotl.in/2S46SnuP3

Sorry, I've seen that I duplicated the issue [#99] only after I'd posted mine.


Add Seattle, please!

Lorena Gonzalez (Council President<EMAIL_ADDRESS>Lisa Herbold (Councilmember<EMAIL_ADDRESS>Debora Juarez (Councilmember<EMAIL_ADDRESS>Andrew J. Lewis (Councilmember<EMAIL_ADDRESS>Tammy J. Morales (Councilmember<EMAIL_ADDRESS>Teresa Mosqueda (Councilmember<EMAIL_ADDRESS>Alex Pederson (Councilmember<EMAIL_ADDRESS>Kshama Sawant (Councilmember<EMAIL_ADDRESS>Dan Strauss (Councilmember<EMAIL_ADDRESS>

Thanks for providing the recipients @cstrouse but we will need the email template and subject as well. Closing this out for now, but feel free to reopen once this is ready!


can't run mediaval minecraft curseforge server

i'm trying to start a mediaval minecraft but if i use USE_MODPACK_START_SCRIPT=false don't pull the modpack server, if don't use it i have the make sure is a ftb server error

What modpack exactly are you trying and I'll see if I can find any workarounds?

https://www.curseforge.com/minecraft/modpacks/medieval-minecraft-modpack
I use this and didn't work
version: "2"
services:
minecraft-server:
image: itzg/minecraft-server:java8
container_name: minecraft1
network_mode: host
environment:
- PUID=1000
- PGID=1000
- EULA=TRUE
- VERSION=1.16.5
- TYPE=CURSEFORGE
- CF_SERVER_MOD=https://media.forgecdn.net/files/3518/284/Medieval+Minecraft+Server+Pack+%5bFORGE%5d+v37.zip
- MEMORY=4G
- ANNOUNCE_PLAYER_ACHIEVEMENTS=true
- ALLOW_FLIGHT=TRUE
ports:
- 25565:25565
volumes:
- /srv/dev-disk-by-uuid-f56a2030-b0f5-4e5b-9d30-18fe84f81052/mi/Minecraft:/data
restart: unless-stopped

I also had the same issue, are there any solutions or workarounds yet?

I ran into a similar issue when trying to use USE_MODPACK_START_SCRIPT which is that it looks like the modpack cannot be a web URL when using USE_MODPACK_START_SCRIPT=false. It seems that USE_MODPACK_START_SCRIPT=false limits the container to only local modpacks

A workaround for really misbehaving modpacks is to use TYPE=FORGE and the GENERIC_PACK variable; however, for the time being GENERIC_PACK  only supports remote files via http.

so I tried to use the workaround and the server started, but i wasnt able to join. I think the server didnt import the mods right?
version: '3.8'
services:
mc:
image: itzg/minecraft-server:${IMAGE_TAG:-latest}
ports:
- 25565:25565
volumes:
- ./medievaldata:/data
- ./modpacks:/modpacks:ro
environment:
EULA: "true"
TYPE: FORGE
GNERIC_PACKS: https://media.forgecdn.net/files/3797/938/Medieval+Minecraft+Server+Pack+[FORGE]+v11.zip
restart: unless-stopped
volumes:
data: {}

Starting curseforgeserver_mc_1 ... done
Attaching to curseforgeserver_mc_1
mc_1  | [init] Running as uid=1000 gid=1000 with /data as 'drwxrwxr-x 9 1000 1000 4096 May 24 11:46 /data'
mc_1  | [init] Resolved version given LATEST into 1.18.2 and major version 1.18
mc_1  | [init] Resolving type given FORGE
mc_1  | [init] Checking Forge version information.
mc_1  | [init] server.properties already created, skipping
mc_1  | [init] Checking for JSON files.
mc_1  | [init] Setting initial memory to 1G and max to 1G
mc_1  | [init] Using Forge supplied run.sh script...
mc_1  | 2022-05-24 17:40:53,030 main WARN Advanced terminal features are not available in this environment
mc_1  | [17:40:53] [main/INFO] [cp.mo.mo.Launcher/MODLAUNCHER]: ModLauncher running: args [--launchTarget, forgeserver, --fml.forgeVersion, 40.1.0, --fml.mcVersion, 1.18.2, --fml.forgeGroup, net.minecraftforge, --fml.mcpVersion, 20220404.173914]
mc_1  | [17:40:53] [main/INFO] [cp.mo.mo.Launcher/MODLAUNCHER]: ModLauncher 9.1.3+9.1.3+main.9b69c82a starting: java version 17.0.3 by Eclipse Adoptium
mc_1  | [17:40:53] [main/INFO] [mixin/]: SpongePowered MIXIN Subsystem Version=0.8.5 Source=union:/data/libraries/org/spongepowered/mixin/0.8.5/mixin-0.8.5.jar%2314!/ Service=ModLauncher Env=SERVER
mc_1  | [17:40:56] [main/INFO] [cp.mo.mo.LaunchServiceHandler/MODLAUNCHER]: Launching target 'forgeserver' with arguments []
mc_1  | [17:41:04] [modloading-worker-0/INFO] [ne.mi.co.ForgeMod/FORGEMOD]: Forge mod loading, version 40.1.0, for MC 1.18.2 with MCP 20220404.173914
mc_1  | [17:41:04] [modloading-worker-0/INFO] [ne.mi.co.MinecraftForge/FORGE]: MinecraftForge v40.1.0 Initialized
mc_1  | [17:41:05] [Forge Version Check/INFO] [ne.mi.fm.VersionChecker/]: [forge] Starting version check at https://files.minecraftforge.net/net/minecraftforge/forge/promotions_slim.json
mc_1  | [17:41:06] [Forge Version Check/INFO] [ne.mi.fm.VersionChecker/]: [forge] Found status: UP_TO_DATE Current: 40.1.0 Target: null
mc_1  | [17:41:07] [main/INFO] [mojang/YggdrasilAuthenticationService]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
mc_1  | [17:41:09] [main/WARN] [minecraft/VanillaPackResources]: Assets URL 'union:/data/libraries/net/minecraft/server/1.18.2-20220404.173914/server-1.18.2-20220404.173914-srg.jar%2339!/assets/.mcassetsroot' uses unexpected schema
mc_1  | [17:41:09] [main/WARN] [minecraft/VanillaPackResources]: Assets URL 'union:/data/libraries/net/minecraft/server/1.18.2-20220404.173914/server-1.18.2-20220404.173914-srg.jar%2339!/data/.mcassetsroot' uses unexpected schema
mc_1  | [17:41:13] [main/WARN] [minecraft/Commands]: Ambiguity between arguments [teleport, location] and [teleport, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
mc_1  | [17:41:13] [main/WARN] [minecraft/Commands]: Ambiguity between arguments [teleport, location] and [teleport, targets] with inputs: [0.1 -0.5 .9, 0 0 0]
mc_1  | [17:41:13] [main/WARN] [minecraft/Commands]: Ambiguity between arguments [teleport, destination] and [teleport, targets] with inputs: [Player, 0123, @e, dd12be42-52a9-4a91-a8a1-11c01849e498]
mc_1  | [17:41:13] [main/WARN] [minecraft/Commands]: Ambiguity between arguments [teleport, targets] and [teleport, destination] with inputs: [Player, 0123, dd12be42-52a9-4a91-a8a1-11c01849e498]
mc_1  | [17:41:13] [main/WARN] [minecraft/Commands]: Ambiguity between arguments [teleport, targets, location] and [teleport, targets, destination] with inputs: [0.1 -0.5 .9, 0 0 0]
mc_1  | [17:41:15] [Worker-Main-11/INFO] [minecraft/RecipeManager]: Loaded 7 recipes
mc_1  | [17:41:17] [Worker-Main-11/INFO] [minecraft/AdvancementList]: Loaded 1141 advancements
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/DedicatedServer]: Starting minecraft server version 1.18.2
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/DedicatedServer]: Loading properties
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/DedicatedServer]: Default game type: SURVIVAL
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/MinecraftServer]: Generating keypair
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/DedicatedServer]: Starting Minecraft server on *:25565
mc_1  | [17:41:19] [Server thread/INFO] [minecraft/ServerConnectionListener]: Using epoll channel type
mc_1  | [17:41:20] [Server thread/INFO] [minecraft/DedicatedServer]: Preparing level "world"
mc_1  | [17:41:20] [Server thread/INFO] [minecraft/MinecraftServer]: Preparing start region for dimension minecraft:overworld
mc_1  | [17:41:22] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:22] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:22] [Worker-Main-9/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:22] [Worker-Main-8/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:23] [Worker-Main-10/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:23] [Worker-Main-9/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:23] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:24] [Worker-Main-12/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:24] [Worker-Main-14/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:25] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:25] [Worker-Main-9/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:26] [Worker-Main-8/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 0%
mc_1  | [17:41:26] [Worker-Main-10/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 1%
mc_1  | [17:41:27] [Worker-Main-8/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 1%
mc_1  | [17:41:27] [Worker-Main-9/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 1%
mc_1  | [17:41:28] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 6%
mc_1  | [17:41:28] [Worker-Main-10/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 15%
mc_1  | [17:41:29] [Worker-Main-12/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 23%
mc_1  | [17:41:29] [Worker-Main-13/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 52%
mc_1  | [17:41:30] [Worker-Main-10/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 69%
mc_1  | [17:41:30] [Worker-Main-10/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 69%
mc_1  | [17:41:31] [Worker-Main-8/INFO] [minecraft/LoggerChunkProgressListener]: Preparing spawn area: 95%
mc_1  | [17:41:31] [Server thread/INFO] [minecraft/LoggerChunkProgressListener]: Time elapsed: 10680 ms
mc_1  | [17:41:31] [Server thread/INFO] [minecraft/DedicatedServer]: Done (11.254s)! For help, type "help"
mc_1  | [17:41:31] [Server thread/INFO] [minecraft/DedicatedServer]: Starting remote control listener
happythepills@compaq-mc:~/curseforgeserver$
mc_1  | [17:41:31] [Server thread/INFO] [minecraft/RconThread]: RCON running on <IP_ADDRESS>:25575
mc_1  | [17:41:31] [Server thread/INFO] [ne.mi.se.pe.PermissionAPI/]: Successfully initialized permission handler forge:default_handler
^CGracefully stopping... (press Ctrl+C again to force)
Stopping curseforgeserver_mc_1 ...


fix: add new line before share image URL

Fixes an issue where the title and URL were not separated when sharing the image (without alt text).
For example, it would be shared as Helium Reservehttps://xkcd.com/2766/ instead of Helium Reserve https://xkcd.com/2766/.
This pull request fixes this by prepending a new line to the URL.

Thanks, much appreciated!


Fix emit.maxListenrs warning, update dependencies, expand tests

I have made a number of changes to the module which I would like to request be merged into the main module. These changes include

Remove emmitter.maxListener warnings. Some of the funciton in the module were adding error listeners, but never removing them. Initially, I added code to remove them, but then realised most of them were not needed, so just removed them.

Upgraded ssh2 dependency from 0.5.5 to 0.6.1

Enhanced error messages to provide more context and to be more consistent

Updated versions of test modules (mocha, chai) as the ones being used had high and critical security vulnerabilities. Also added the chai-as-promised module to make some of the tests a bit cleaner.

re-factored and added a number of tests to help ensure tests actually test what they should and provide better feedback on failure. All tests are now passing.

I also added the dotenv module to manage sftp credentials. My testing was against an openSSH sftp server. Details of server, user, password and URL are placed in a .env file rather than hard coded into test file.
I also tried to make some of the code more consistent e.g. some method returned reject/resolve and others did not, some resolved with a message, others did not.  May have got some things wrong, but at least it should be consistently wrong!
Update: Added new 'exists' method and re-factored mkdir/rmdir to address issue #88

Extended this pull request to also include a new function 'exists' and re-factoring of mkdir and rmdir to address issues raised in issue #88

pertty good.
publish new version 2.4.0


[ART-1528 partial] add new cli command create candidate

I tried this afternoon it works with my personal access token
https://github.com/openshift/cincinnati-graph-data/pull/55
So currently we need one github access_token and one slack_api_token and ssh-keys

@vfreex Sounds good to me, I am not opinionated on name picking, so request-cincy-edge if no objections since I think currently we are not going to add any drawback PR features contrast with add
Also as you guys can see here for this command to work I need at least github access token, ssh-pub and priv key any instructions on How could I get those or way to workaround?
@sosiouxme @thiagoalessio

Relates to https://github.com/openshift/aos-cd-jobs/pull/2126

@sosiouxme @vfreex @thiagoalessio I closed this since https://github.com/openshift/aos-cd-jobs/pull/2153 has a better way to do it.


Build behind proxy

This enables running the script behind a corporate proxy

@aateeque is this something that should be added here: https://github.com/cake-build/resources

@gep13 thanks for pointing in the right direction. I'll close it here and open it there


feat: adds depth linter

Adds a linter to get the maximum depth of go functions, which allows developer to know wheter it's worth splitting a function into smaller one

Codecov Report

Merging #36 into feature-3.0 will increase coverage by 2.4%.
The diff coverage is 64.38%.

@@              Coverage Diff               @@
##           feature-3.0      #36     +/-   ##
==============================================
+ Coverage        59.91%   62.31%   +2.4%
==============================================
  Files                6        6
  Lines              893      966     +73
==============================================
+ Hits               535      602     +67
- Misses             304      312      +8
+ Partials            54       52      -2

Impacted Files
Coverage Δ

engine/engine.go
58.44% <48.93%> (-0.99%)
:arrow_down:

tools/report2html.go
61.98% <92.3%> (+9.73%)
:arrow_up:

tools/processbar/processbar.go
64.15% <0%> (+3.77%)
:arrow_up:

Continue to review full report at Codecov.

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

I see the code where you submit the code is not standardized, recommend you use go fmt, go vet and go lint format your code.


Change RealtimeIndexTask to use AppenderatorDriver

@gianm This is a follow up PR to the discussion we had on #4774
This PR changes the RealtimeIndexTask to use the AppenderatorDriver instead of the RealtimePlumber.  This allows the realtime indexing to behave similar to the kafka indexing service and ingest events from arbitrary time periods without concerns for windowing.
Implementation notes:

I added a maxRowsPerSegment configuration to RealtimeTuningConfig.  I defaulted this to 5 million, the same value that is used in IndexTuningConfig

When maxRowsPerSegment is reached for any segments, all segments are published and handed off in the background.  This isn't necessarily ideal because we may end up pushing small segments if there is one large one that reached maxRowsPerSegment
The API of AppenderatorDriver didn't support publishing a single segment so I opted to keep this behavior for now.  I suppose this is also the way that the KIS behaves so we are consistent there.

The above behavior means that segments are not merged and pushed on a time-based interval as they were in RealtimePlumber.

The changes I made to TaskLifecyleTest are a little hacky but I wasn't sure of the best way to refactor that test.  Suggestions welcome

Hi @kevinconaway, would you let me know when you're available for finishing this PR? I'm asking because https://github.com/druid-io/druid/pull/5297 is a blocker issue for 0.12.0 which will cause conflicts with this PR. I think it would be easier for me to resolve the conflicts because I reviewed this PR, so I'm waiting for this issue to be merged.

I will submit the changes tomorrow, is that OK?

Yes, it is. Thanks!

I've made some readability improvements.  I also combined the publish + handoff in to a single future per your request.
Please let me know what else I can change.
Thanks!

I've updated the PR to add a new RealtimeAppenderatorTuningConfig class instead of reusing RealtimeTuningConfig.

@jihoonson Not sure why this particular build failed, it seems unrelated

@kevinconaway just restarted the failed job.

@jihoonson Is this particular test known to be flaky?
Are you able to get the test output off the build nodes?

Looks like it failed here and here too

@kevinconaway yeah, looks like flaky. The failed task passes in my mac.
Raised https://github.com/druid-io/druid/issues/2373

OK thanks.  What are the next steps for this PR?

I think it's ready for merging. @himanshug do you have further comments?

@jihoonson I'm sorry but I haven't really looked at the patch. Given that it is a new task now that can be tested independently in prod and wouldn't impact existing users. I'm fine with your approval on the PR. Feel free to merge.

@himanshug thanks. I'll merge shortly.

@kevinconaway raised #5355. Please take a look.

@jihoonson Thoughts on backporting this to 0.12?

@kevinconaway basically, I think we don't have to backport this to 0.12 because this is a new feature rather than improving an existing one. However, I think it would be fine if this doesn't cause a lot of conflicts for backporting. More people can test this new task type earlier instead of waiting for a next release or backporting this by themselves. @gianm @jon-wei what do you think?

basically, I think we don't have to backport this to 0.12 because this is a new feature rather than improving an existing one.

The reason I ask is that I'd like to get off our fork and start using this feature when 0.12 is released, rather than waiting for 0.13

@kevinconaway I'm sorry to say, but we usually backports only bug fixes, so I would say it's difficult to make an exception for this PR without a reasonable reason. We already created 0.12.0-rc3 (https://groups.google.com/forum/#!topic/druid-development/DZx-opThc50) which will be released soon unless another bug is found.

No worries, thanks for following up

Thanks for understanding!

@kevinconaway are you a Tranquility user? I'm curious whether you've looked into modifying Tranquility to make use of this appenderator-based realtime task

@dclim We're not using Tranquility.  I don't know Scala well enough to confidently hack on it.  The last time I looked at it, it wasn't up to date with even Druid 0.10 but that was some time ago

@kevinconaway cool, thank you for the contribution!


Sdxl Q8 gguf

I use a lot sdxl for inpainting and upscaling. I wanted to convert my custom sdxl model to Q8 gguf. There  is a very slight drop from fp16. I searched for a guide but could not find one. Can you help ?

https://github.com/city96/ComfyUI-GGUF/tree/main/tools
That's the main guide, or you can use my Windows GUI which is easier to use:
https://github.com/rainlizard/EasyQuantizationGUI/releases

Wow, i will for sure use your GUI instead. Thank you for sharing. I had tried coding a quantization script with O1, but this is easier. I hope it's self contained.

dude maybe put this GUI tool in the main README cause I went through the tools way and it is a lot of work and couldn't get it to work, thanks a lot !


Return error when given a non-existent exact version of a system chart

Issue: related to https://github.com/rancher/rancher/issues/40159
Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

QA Testing Considerations

Regressions Considerations

Now that this is returning an error, you will need to update: https://github.com/rancher/rancher/blob/c1175854f8e3a96b7fc39dc8110300ced1f6e186/tests/v2/codecoverage/package/Dockerfile as well, otherwise that probably won't be able to find a valid version.


Fixes #282 made sure that begin transaction is called before taken a wri…

…te/read lock in case of lightweight handler in HdfsVariables

Jenkins retest this please

Jenkins retest this please

Jenkins retest this please.


Floating bar charts

I'm using Chart.js for plotting temperatures. Each plotted measure point is average of n measurements per hour/day/week/month/year. So I'm collecting data each minute, but plot only averages per different time periods. However, I would like to add also minimums and maximums per each measurement, because that would be a lot more useful than plotting just averages. Unfortunately I didn't find any kind support for plotting floating bars (or just plain vertical lines); all the bars are always started from zero.
I have attached similar kind of graph that I have in my mind (only addition to that is, that I want to plot also the average as continuous line too).

I think this is different enough from a regular bar chart that it involves a new chart type. I think you could create an extension that derives from a bar chart and the overrides the updateElement method so that the base property of the bar model is set to the value you want.
I think this would made a good extension to Chart.js but I think it's too specific to include in the main library.

Thanks for the comment and the tip! At the updateElement there is a line base: reset ? scaleBase : me.calculateBarBase(me.index, index),; do you meant that if I simply set desired value to the base variable instead of calling calculateBarBase(), I get the results I'm looking for? So all I have to do, is create an extension, implement updateElement method there with one extra parameter and then use the parameter value instead of calculateBarBase()?

It's almost that simple, yes 😄
However, one thing to keep in mind is that the base value is a raw pixel coordinate. To translate from a data coordinate to a pixel coordinate you need to use the axis to translate the value. In calculateBarBase that's done here so you should be able to do something very similar for your case.
Hope this helps!

@mahead In case you are working around the bar implementation, note that it has been fully refactored in #4044!

Thanks a lot for your help! I ended up modifying the Bar graph directly, because all the code was there already and basically all I had to do was to fake the bar class to think I was always having stacked bars. And add support for plotting range instead on single value. There are still couple of bugs left (bars don't affect the y-axle scale at all for some reason, tool tips always show NaN as a range, bottom border line missing on bars), but overall I'm very happy that I was able to modify the code for supporting my own purposes. :)
Here is live example with hour-by-hour humidity data (both outside & inside).

I'm happy to share the modifications if someone is interested, but I don't think there is much general use for those as it's probably not very often useful to have multiple bars on same x-axle point etc. And of course current implementation is mostly useless because it overwrites original bar functionality.

@mahead glad that was easy to do :)
I think for the tooltip issue you can get around it by changing the tooltip callback (though I'm not sure why the data would be NaN).
For the axes, I assume you are referring to the limits? Those should be updated for the data in each dataset but that won't work too well for ranges.

@etimberg I took a look at the tooltip, but didn't understand where the actual value is set. I believe the issue happens because I have modified the data format, this way:
{ type: 'bar', data: [{min: 60, max: 80},{min: 48, max: 84},{min: 40, max: 82},{min: 64, max: 88},{min: 60, max: 90},{min: 80, max: 86}], label: 'Piha min/max', backgroundColor: 'rgba(75, 192, 192, .2)', borderColor: 'rgba(75, 192, 192, .2)', borderWidth: 1, }
Because of this, I modified all the references to Dataset.Data[] => Dataset.Data[].Max at controller.bar.js . To make tooltips work, should something be fixed at another file? This is how it looks currently:

Regarding axes, you assume correctly. If I disable all the regular lines so that only these range bars are left, the scale is changed from -1...1, even though actual data values are greater:

But this is really not an issue, because other data lines help the scale to be correct. That tooltip would be only actual thing I would like to fix, since it's useful to hover mouse over the bar to check what the exact range is.

Changing the data format would do it. The default label is generated by https://github.com/chartjs/Chart.js/blob/master/src/core/core.tooltip.js#L72-L80
You can just override that in your config
options: {
  tooltips: {
    callbacks: {
      label: function(tooltipItem, data) {
        var label = data.datasets[tooltipItem.datasetIndex].label || '';

        if (label) {
          label += ': ';
        }
        label += data.datasets[tooltipItem.datasetIndex].data[tooltipItem.index].max.toString();
        return label;
      }
    }
  }
}

@etimberg Thank you again, that indeed did the trick. :+1: (Though I also added the min value to the tooltip.)
When I originally noticed Chart.js doesn't support range bars, I tried to find alternative charting libraries. None of them pleased me (or they were commercial and prices were $300+), so I came back hoping you could implement the feature some day. I must say I'm more than happy that you could point me to do it by myself, and what I looked the code, it looked very clean and well structured; at least before my modifications. Keep up the good work!

Thanks @mahead! 😄  I think your chart would make a nice extension. If you publish it as it's own type eventually I'm happy to link to it from the docs

@mahead would you be able to share your implementation? I'm curious if you had to build new axes types or just a new controller?

@benmccann Sure, I could share the implementation. However, in it's current form it's an ugly hack that replaces the default bar graph. So, while it's very useful for me, there is no way it could be considered as general solution. So for now I'm not willing to contribute it anywhere, since I think I should make it properly first (and because of lack of time, I can't give any estimates).
Basically, as I see it, it's new type of controller. It is very similar to current stacked bar graph, just so that the base is taken from configured coordinates. I also changed the bar width to constant, because I don't want to place bars next to each other on single x point, but stack them and let them overlap. So far only issue I have, is that y-axle don't scale based on this modified bar type values. But it doesn't bother me, because I can use predefined values for my axles.

I've created a candlestick chart - https://github.com/chartjs/chartjs-chart-financial. You could probably create a floating bar chart using the candlestick chart by setting the high & low within the open & close.

@mahead Hello, I'm currently working on a similar feature but for a horizontal bar chart instead. I managed to shift all bars with a constant value but would like to fill a specific one directly through the datasets (like your example). Where do I have to look to do this? Thanks.

Does this vaguely relate to #3574 ? I'm looking for a solution similar to what @antoinecorne is mentioning, I want a normal horizontal bar chart, but I want to set where each bar starts and ends. I don't need all the date-stuff in 3574.
What you have made @mahead looks great, basically exactly what I want, just vertically. It would be great to see your solution, even though you call it a hack.

Hi. I've modify the Bar Chart to support the floated bar by adding "minData" parameter into the bar chart itself. You can have a look HERE
There are still problem with lower border of the floated bar.

@WasinTh Didn't know about minData, haven't seen it in documentation either. I think this might be exactly what I'm looking for. Thanks!

@eirikb It's not in the official release of ChartJS. I just modified their code and recompile it. If the community accept my approach, I'm willing to request pull on this modification.

@WasinTh Ah I was a bit quick on the trigger. I didn't see any custom code in the fiddle, but now I noticed that the external resource is actually pointing to your version. Nevertheless, I think minData would suite me just fine, so I think it's a good idea.

It seems that I don't have time to properly implement my modifications, so I'm throwing it all here. Please feel free to take a look and take any ideas I might have had there. (I also added the Chart.js-master.zip that I had taken, so it's easier to diff only my modifications.)
chart.js.tar.zip
Chart.js-master.zip

And how can I set the minimum value for each data element?
For example:
 data: [
{min: 60}, // or simply 60 - when starts from zero
{min: 48}

Those. It is possible for each dataset to specify from which value it should be shown (in the case of a stacked bar chart, so that the value does not start from zero). Is this possible?

@lex111 If you are using Chart.js version that I tweaked, you can define datasets this way:
data: [ { min:-13.5, max:7.6},  {min: -4.0, max:19.0 } ]
That will create two bars with ranges of -13,5 ... 7,6 and -4,0 ... 19,0.

@mahead Tried, but does not work incorrectly with stacked groups bar charts. All data must be represented by the object (eg { min:-13.5, max:7.6})? Can you show the demo, please?

@lex111 Sure, here is demopage.html as zipped attachment. You need to have Chart.bundle.min.js in same directory with the html file, then it will work (obviously you will need my version of the Chart.bundle.min.js, see earlier attachment).
Sorry for the Finnish language, I grabbed the file directly from the live environment.
demopage.zip

@lex111 Sure, here is demopage.html as zipped attachment. You need to have Chart.bundle.min.js in same directory with the html file, then it will work (obviously you will need my version of the Chart.bundle.min.js, see earlier attachment).
Sorry for the Finnish language, I grabbed the file directly from the live environment.
demopage.zip

I fiddled around a bit to and created a new chart type derived from bar based on version 2.6.0.
http://jsfiddle.net/3fepvcte/
I hope this helps!

Can someone plz suggest me, how to plot xaxis values with dates [like 13/10/2017 format]

@Meenu-Hexaware you need to use the time scale. If you have more questions about this, let's please discuss elsewhere as the question is a bit off-topic on this thread

@sclaflin Thanks a lot. It indeed helped a lot!!! Although there were some issues with tooltips but I fixed it anyway ;)

Implemented by #6056 and will be released in v2.9.


Fixes setting deploy all asset checkbox to true for AE standard apps

fixes #1651

reverts the previous approach and just sets the UI field to true by default

no problem :)


I tried to install gsi via the adb shell command for non root method and it says installation failed. can you help fix my problem

There is no way to debug installation with adb operation mode, install Shizuku, start it, and grant Shizuku permissions to DSU Sideloader, after that, you should be running as Shizuku operation mode, now, try installing a GSI again.


Build ELF parse at compile time using EverParse

Description
Invoke the Everparse generated validation prior to parsing the ELF file.
Testing
CI/CD
Documentation
NO
Resolves: #834

@shankarseal when you get a chance, could you please create a CMakeLists.txt file to build the spec.lib? I was unable to get cmake to build it correctly.


[✨] Allow for multiple keyed menus with useContent

Is your feature request related to a problem?
Currently it seems like it's only possible to define a single menu inside of menu.md. This makes it frustrating to account for the possibility of multiple menus, such as a header menu and a footer menu (which will probably contain a basic sitemap).
Describe the solution you'd like
There are a couple good ways I can think of handling this. Keep in mind that I have done zero digging into the code to see how things are structured, so take these with a grain of salt.

Allow for multiple menus to be defined in menu.md using top level headings. For example:# Header

## [Home](/)

## [About](/about)

# Footer

<!-- Other footer specific menu items -->

Allow for multiple menu.md files the same way we allow for multiple layouts, and allow useContent to accept an identifier as a parameter. So useContent() would use menu.md, while useContent('footer') would use menu-footer.md.

Describe alternatives you've considered
The obvious one is to just hard code the menu into the component, use a json file, or use an exported javascript object from another file. This is how I will probably handle the problem for now, but I wanted to bring this idea up for discussion anyway.
Additional context
No response

We moved this issue to qwik-evolution repo to create a RFC discussion for this.
Here is our Qwik RFC process thanks.


Why not export get_data_per_hour() and others?

The class LinkyClient() defines a method _get_data_per_hour(self, start_date, end_date).
https://github.com/Pirionfr/pyLinky/blob/master/pylinky/client.py#L103
Why not rename it without the leading '_' so that it is no more considered as internal only?
This would be useful and could prevent forks like domoticz_linky to diverge from pyLinky to provide such a method.
https://github.com/empierre/domoticz_linky/blob/master/linky.py#L80
It would be better if domoticz_linky could just use pyLinky module as-is.

if you need for domoticz you can do a pull request, and I will create a new version on pypi

I would also need an equivalent of export_hours_values_json_format() and other methods
https://github.com/empierre/domoticz_linky/blob/master/linky_json.py#L81
One major problem is that your code is licensed under Apache v2.0 and domoticz_linky uses GNU GPL v3. Maybe you don't want to include GNU GPL v3 code and move the whole project to this licence?

I think we can dev an json export.
what format you need ?

Do you plan to move from Apache 2.0 to GNU GPL v3?

Problem fixed in my fork https://github.com/LudovicRousseau/pyLinky
I do not propose a Pull Request because I changed the licence to reuse GNU GPL code from https://github.com/empierre/domoticz_linky
If you change the licence I will create a PR so you can merge my changes.

+1 for a merge
For the licence, I guess as this lib is inspired by a GPLv3 lib, staying with GPLv3 make sense

go head make the pull request

PR available
https://github.com/Pirionfr/pyLinky/pull/13

We actually exported a function to get any interval of time at any scale. Maybe this answers this issue ?

New PR proposed in https://github.com/Pirionfr/pyLinky/pull/18

merge available in v0.3.0


autobind classes and methods

Hi. I was working on this myself in an experimental project so thought id contribute it as I came across your repo which you've already published.

That's pretty cool, thanks! Are you ok if I add you as a collaborator to the repo and on npm?

That would be good thanks

Added on github, what's you npm username?

domarmstrong

OK, added there too.


Requires cgo ?

Can the library only build with cgo?
$ export CGO_ENABLED=0
$ go get github.com/ThalesIgnite/crypto11
# github.com/ThalesIgnite/crypto11
../testgopath/src/github.com/ThalesIgnite/crypto11/crypto11.go:119:9: undefined: pkcs11.ObjectHandle

whereas if I set CGO_ENABLED=1 it works.
Is the CGO dependency within this project or one of the dependencies?

Seems like the dependency https://github.com/miekg/pkcs11 requires CGO.
https://github.com/miekg/pkcs11/issues/24


How do you add controls?

I can run new Application().Execute(new Form());, but how do you add controls to the Form?

As stated in the "Get Started" section, you have to derive from Form to add your components. Form is an abstract class, so you cannot create an instance from it without derivation. This library was designed to be as design-equal to WinForms as possible. There you also derive from a class Form and have a method in it (normally InitializeComponent()) which adds the whole tree of components you want.
Form has a property Content of type Component. So you can only add one single component to a form. Adding multiple components to a form, requires you to create one of the layout components.
Example:
`
class MainForm : Form
{
public MainForm()
{
InitializeComponent();
}
private void InitializeComponent()
{
Content = new Button { Caption = "Test" };
/* -or- */
Content = new StackLayout();  // adding multiple components to the layout
}
}
`
Followed by the application line in your main function:
new Application().Execute(new MainForm());

I also updated the READ.ME to hopefully be more clear about the starting process.


integration tests

Moves the test_examples.sh and lint_examples.sh to Rust integration tests.
This is indeed much nicer and makes it possible to actually test the Http and Tcp ones, for which I introduced a bug in my last changes that was now caught 🤦🏻

Ok classic "works on my machine" but not CI... I'll try and dig in. I think I may need to rework some of this

@dicej ok after much wrestling with windows, this should be ready. All example running and linting is now happening in Rust tests, and with that I was able to get much better coverage.

Good points. I was so focused on just getting the readme examples to work, I neglected this (at some point I thought of pulling in wasmtime as a lib, but pressed on).
I can do a round two when I get a chance to clean this up and be a little more maintainable. It would also have the benefit of running the nc example on Windows.


Add example of E2E testing

Motivation
Discussion: https://github.com/ipetkov/crane/issues/366
Checklist

[ ] added tests to verify new behavior
[x] added an example template or updated an existing one
[ ] updated examples/README.md

I went ahead and added PostgreSQL to the example, just to hammer the point home that you can spin up anything together in the test.

I didn't squash so it's easy to drop postgres if necessary


Enhance Authentication with JWT and Refresh Token Mechanism #264

closes #264
Description
This PR introduces a secure authentication mechanism using JWTs for access tokens and refresh tokens, replacing the current session-based authentication. It enhances security, enables frontend access to user data, and ensures token lifecycle management.
Motivation
The current authentication system has the following limitations:

Stolen Cookies Cannot Be Rejected: Once stolen, cookies can be perpetually renewed, preventing proper user logout.
Limited Frontend Access: User information is encrypted within the session cookie, making it inaccessible to the frontend.

To address these issues, JWTs for access tokens and refresh tokens are introduced.
Changes Implemented
1. JWT-Based Access Tokens

User Data: User information is stored in a JWT, signed with the backend's private key.
Accessibility: Stored in a frontend-accessible cookie.
Expiration: Tokens expire after 5 minutes to limit their lifespan.

2. Refresh Tokens

Purpose: Allows secure renewal of access tokens without requiring a full re-login.
Storage: Stored as an HTTP-only cookie for added security.
Validation: Refresh tokens are recorded in the database to ensure integrity.
Expiration: Longer expiration period (e.g., several days).

3. Token Renewal Workflow

When the access token expires, the system verifies the refresh token:
If valid:
*Issues a new access token.
*Invalidates the old refresh token and generates a new one.

Overall, this is excellent work! The code is well-structured and the functionality is clearly implemented. Great job on adding a robust JWT-based authentication mechanism. Before merging, there are a few suggestions and potential adjustments to consider that could further enhance the maintainability, security, and readability of the code. Then the code should be ready to merge!


Release

Release new configuration changes

Codecov Report
Patch coverage has no change and project coverage change: -0.19 :warning:

Comparison is base (b4a1521) 54.85% compared to head (ec9c823) 54.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #426      +/-   ##
==========================================
- Coverage   54.85%   54.66%   -0.19%
==========================================
  Files          24       24
  Lines        1586     1586
==========================================
- Hits          870      867       -3
- Misses        651      653       +2
- Partials       65       66       +1

Flag
Coverage Δ

unittests
54.66% <ø> (-0.19%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Pin pip to a specific version

The latest version of pip and pip-tools creates absolute paths instead of relative paths when compiling requirements files. Pin pip to version 24.3, which should keep the relative paths.

See issue here: https://github.com/jazzband/pip-tools/issues/2131


Drop variant name field

Currently ProductVariant.name field is not particularly useful. Moreover, it causes some confusion in code when it comes to variant representation. and should be dropped. In my opinion, we should drop this field and use variant attributes as a name. We already do it, but our model allows variants without attributes which doesn't make much sense and we should eventually change it, but for now we could use SKU as fallback.

Related issue: #1846

One problem with creating variant name from its attributes is that we need to prefetch them in advance to get their names. Maybe a better idea would be to keep the name field and update it accordingly when attributes change? What do you think?

For reference, Shopify seems to set the variant title on change and it returns Black / 10 in the API (notice how it does not contain the product title).

I think it's a good idea that would save us from:

problems with prefetching the attributes
confusion about how to represent the variant

In that case we lack one feature that Shopify offers: their attributes have a defined order.

Why is this a problem?

It's only a problem if you have preference of either "Black / 10" or "10 / Black".


Smoke test using DirectX windowless software driver

Implement software windowless mode into native engine's DX11 driver
Add run step to AppVeyor to verify game engine boots and runs without error.

This smoke test will verify at a minimum that resources required to boot the engine are present and loadable.  Additional tests can be added later.
Additional Tests  (stretch goals)

Log scrubbing for warnings or errors?
Verify some output using ImageMagick or such

This stretch goal list is not meant to be part of this task.  It should be better defined and converted into its own tracking issue after the software windowless mode and Appveyor integration are complete.

Submitted PR #111 to provide DX11 windowless operation.


Add bot intent

I noticed this assistant is missing a "are you a bot" intent. So I figured adding one.

I can also add a test if folks prefer it.


Fix Client model mapping in UpdateClient

Hi,
Thanks for the project and the contribution, super stable and complete.
I'm using this as part of my OCP GitOps installation; see here for reference.
The secret and rootUrl are in inversed order, as per as the model defined here, resulting in a wrongly configured client-secret down the road.
I haven't added any test, nor find that many tests in that area to validate model mapping, but as I faced this issue today, I figure I'll at least submit the PR to track it.
Let me know if you wish I add tests surrounding this.
Cheers

Hey adetalhouet,
thanks for finding and fixing it :+1:  i don't see the necessity to test this separaretly. I'll merge it and release it as soon as possible.
<3

Released with  https://github.com/mayope/keycloakmigration/releases/tag/0.2.22


test: Skip K8sPolicy on GKE and 4.19

Running K8sPolicies on those CI jobs is not expected to increase coverage, so let's disable to reduce cost.

test-me-please

test-me-please

This was fairly effective (~27% reduction) so marking for backports to v1.8 and v1.9.


postfix/error program

These paterns doesn't work for postfix/error program
tags:	  	_grokparsefailure, _grok_postfix_program_nomatch
message: 2DC5552A34<EMAIL_ADDRESS>relay=none, delay=63495, delays=63350/144/0/0, dsn=4.4.2, status=deferred (delivery temporarily suspended: lost connection with mta7.am0.yahoodns.net[<IP_ADDRESS>] while sending RCPT TO)

Hi, thanks for reporting
Can you supply the complete log message, including syslog date and process name etc?
And do you have more examples from the postfix/error process?

It's working after your last commit. Thank you!


Fix ruby-build to work with LLVM and the ruby versions which support it (e.g. 1.9.3-p125)

Unfortunately #130 and 45d4277 didn't fix the LLVM ruby issue, at least as far as my system with a clean install of Lion and the command-line tools installed.
CC needs to be set for the build to succeed, but require_gcc can't be used to do so currently as it filters out llvm versions of gcc. I added a variable to skip that test. My bash is pretty weak though so I'll be happy to hear of better options.

Oops - issued the pull request from a comparison page, which apparently issues it to the comparison target. :-P


Discovery: Vocabulizer current state & develop next steps

Background & context
One of our FY25 key results is to connect ReportStream to third party services, including a vocabulary service. This ticket captures the work of initial discovery of the current state of Vocab services, including the Vocabulizer project. From there, we will assess ReportStream's connection to vocabulary services and develop next steps for FY25Q2.

FY25 Key Result: Connect to three third-party standards/services (such as VOCAB, QHIN/TEFCA, RCKMS) in production

Working links

Running notes: 3rd party/senders doc

Acceptance criteria

[x] Current state of RCKMS captured in notes doc
[x] Initial recommendations for next steps developed
[x] Current state & recommendations reviewed with Product & Eng

Approved by Product. To be shared at PO sync. Closing ticket.


Uncaught TypeError: Cannot read property 'substr' of undefined

Uncaught TypeError: Cannot read property 'substr' of undefined
Atom Version: 0.155.0
System: Mac OS X 10.10.1
Thrown From: atom-compile-coffee package, v1.3.0
Steps To Reproduce

...
...

Stack Trace
At /Users/bjebb/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:48

TypeError: Cannot read property 'substr' of undefined
  at atomCompileCoffee (/Users/bjebb/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:48:23)
  at atom-workspace.<anonymous> (/Users/bjebb/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:68:48)
  at atom-workspace.handler (/Applications/Atom.app/Contents/Resources/app/src/space-pen-extensions.js:115:32)
  at atom-workspace.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app/node_modules/space-pen/vendor/jquery.js:4681:9)
  at atom-workspace.elemData.handle (/Applications/Atom.app/Contents/Resources/app/node_modules/space-pen/vendor/jquery.js:4359:46)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app/src/command-registry.js:243:29)
  at /Applications/Atom.app/Contents/Resources/app/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:549:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:391:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:167:20)

/cc @atom/core

Happens to me too
Steps to reproduce

Create a new file
Set as html
Write some html
Open Html Live Preview (cmd+shift+p atom HTML preview: Toggle)
Save (cmd+s)

I wasn't using atom-compile-coffee for this page (or should I enable it only when I need it?). HTML did not contain any javascript. I couldn't save any other file after having this issue.
Atom Version: 0.177.0
System: Mac OS X 10.10.2
Thrown From: atom-compile-coffee package, v1.3.0
Stack Trace
Uncaught TypeError: Cannot read property 'substr' of undefined
At /Users/gableroux/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:48

TypeError: Cannot read property 'substr' of undefined
  at atomCompileCoffee (/Users/gableroux/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:48:23)
  at atom-workspace.<anonymous> (/Users/gableroux/.atom/packages/atom-compile-coffee/lib/atom-compile-coffee.coffee:68:48)
  at atom-workspace.handler (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/src/space-pen-extensions.js:115:32)
  at atom-workspace.jQuery.event.dispatch (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/node_modules/space-pen/vendor/jquery.js:4681:9)
  at atom-workspace.elemData.handle (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/node_modules/space-pen/vendor/jquery.js:4359:46)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/src/command-registry.js:243:29)
  at /opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/src/command-registry.js:3:61
  at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:558:16)
  at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:396:22)
  at HTMLDocument.module.exports.WindowEventHandler.onKeydown (/opt/homebrew-cask/Caskroom/atom/latest/Atom.app/Contents/Resources/app/src/window-event-handler.js:170:20)

Commands
     -0:04.2 core:paste (atom-text-editor.editor.is-focused)
     -0:03.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "metrics",
      "timecop",
      "vim-mode",
      "symfony-snippets"
    ]
  }
}

Installed Packages
# User
ScssBundle, v0.4.0
angularjs, v0.1.0
ascii-codes, v0.3.3
atom-compile-coffee, v1.3.0
atom-django, v0.2.0
autocomplete-plus, v2.0.10
build, v0.23.3
coffee-lint, v0.7.3
coffee-refactor, v0.6.2
color-picker, v1.4.4
css-snippets, v0.5.0
editorconfig, v0.3.0
grunt-runner, v0.6.0
js-refactor, v0.3.0
language-blade, v0.7.0
language-erlang, v2.0.0
language-gitignore, v0.1.1
language-ini, v1.3.0
language-python-django-templates, v0.1.0
laravel, v0.4.2
laravel-facades, v1.0.0
linter, v0.11.1
linter-pylint, v0.2.0
linter-scss-lint, v0.0.11
linter-xmllint, v0.0.5
minimap, v4.0.2
minimap-git-diff, v3.1.1
refactor, v0.4.1
scope-inspector, v0.2.1
ssh-config, v0.10.0
travis-ci-status, v0.11.2

# Dev
No dev packages

/cc @atom/core

Same problem here

I encountered this as well. Easy to reproduce.

Cmd+N (File>New File)
Cmd+S (File>Save)

The workaround is Shift+Cmd+S (File>Save as), although ideally Save should behave like Save As if it's a new file.


IGNITE-14383 No need to update metastorage if cache statistics disabled

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

@zstan What if

Statistics enabled.
We create cache group.
Disable statistic.
Remove cache group.

It seems this scenario will lead to a situation when we don't clear meta storage properly.
Can we check the presence of metal storage keys before trying to remove them?
And what is the issue with removing unexisting keys in the first place?