[TASK] Remove deprecated fields after CRD API version bump

What's the task? Please describe.
After adding the migration logic (from old fields to the new fields), all deprecated fields like these can be removed after the version bump.
Need to wait for https://github.com/longhorn/longhorn-manager/pull/1136 merge before starting on this.

Hey team! Please add your planning poker estimate with ZenHub @c3y1huang @shuo-wu @joshimoo @jenting @derekbit @PhanLe1010 @keithalucas

Verified on master-head 20230531

longhorn master-head (ffaa3d2)
longhorn-manager master-head (2a83384)

Result Passed

[x] doc: PR #713

The following deprecated custom resource fields have been removed.

[x] Volume.spec.recurringJob
[x] Volume.spec.baseImage
[x] Replica.spec.baseImage
[x] Replica.spec.dataPath
[x] InstanceManager.spec.engineImage
[x] BackingImage.spec.imageURL
[x] BackingImage.status.diskDownloadProgressMap
[x] BackingImage.status.diskDownloadStateMap
[x] BackingImageManager.status.backingImageFileMap.directory
[x] BackingImageManager.status.backingImageFileMap.downloadProgress
[x] BackingImageManager.status.backingImageFileMap.url


Event search bar is not populated when findings are opened

Loom is here.
This bug makes it impossible to iterate over each event in a finding in the Trace View.

Look at "related events". It should contain the same information previously displayed in the search bar, albeit in a more clear and flexible manner.

@dustinbyrne Yeah I'd seen that, and with the "eyeball" click it's not too bad, but we still lost some control that was there before.


Add Queue stage to test pipelines

Related eclipse/openj9#2372
Signed-off-by: Adam Brousseau<EMAIL_ADDRESS>

Marking as WIP to allow for discussion. @llxia @ShelleyLambert
Previously, we added a queue stage to the OpenJ9 pipelines which helps us visually determine
a) How long a job waits for a machine.
b) Separates queue time from the first 'real" stage of the pipeline.
I think this would be beneficial to have on the test pipelines as well.
See eclipse/openj9#2372 or an example job: https://ci.eclipse.org/openj9/view/Sandbox/job/Adam-Test-sanity.functional-JDK11-linux_x86-64_cmprssptrs/

Thanks @AdamBrousseau - seems like a good enhancement and will more clearly reveal where we can improve our addition and scheduling of machine resources.
Once you are ready to shift this from WIP, we can do final review/merge.

Removed WIP. Updated only the version agnostic files as I understand from Lan that the version specific files will be removed in time.

Also tested with openjdk regression test.

Just noticed this "breaks" the Blue Ocean view by only showing the Queue stage for the build. I'm surprised we hadn't noticed or received complaints before now because we've had this on the Build jobs since July.
Found open issue to add queue time by default
https://issues.jenkins-ci.org/browse/JENKINS-49179
It looks like nested stages was supposed to be resolved by
https://issues.jenkins-ci.org/browse/JENKINS-49050
according to
https://issues.jenkins-ci.org/browse/JENKINS-38442
So maybe its coming soon.
I suppose this isn't a deal breaker as classic view still works and it appears it will eventually work in Blue Ocean.

Hmmm, not ideal but I personally only track blue ocean when doing a release and I've only really got 11.0.1 for hotspot to go and then it's quiet for a bit.

@karianna keep in mind this would only be at the test job level not the overall pipeline level.
ex
Before:
https://ci.adoptopenjdk.net/blue/organizations/jenkins/openjdk11_j9_systemtest_x86-64_linux/detail/openjdk11_j9_systemtest_x86-64_linux/96/pipeline

After:
https://ci.adoptopenjdk.net/blue/organizations/jenkins/openjdk11_j9_systemtest_x86-64_linux/detail/openjdk11_j9_systemtest_x86-64_linux/98/pipeline

Ah OK, it's a bit trickier to debug but in all honesty, I just look for an overall pass/fail.


Prevent chests from losing items.

Fixes  #3067 by preventing the held item component deletion. The mesh is removed when held but otherwise it is untouched. This is a temporary fix while a better solution will be underway.
How to test:
Plonk down the starting chest, you'll see that it now has items inside.
I've tested it thoroughly before judging it as an ok fix, but I might miss something, so a secondary test would be neat to have before merging :)

Hooray Jenkins reported success with all tests good!

Tested it out, found a couple sneaky issues:

Probably the reason the entire entity was previously destroyed is from not knowing what other interesting components could be active - with this change if you switch to a torch then something else the LightComponent from the torch remains active :D
There still seems to be a case where with two regular clients (no headless) player connected together the empty chest can still appear - I saw it in one test as player 2 seeing player 1's placed goodie chest as empty, although surprisingly the inner goodie chest also placed by player 1 was fine

I'm not worried very much about case number 2. The first issue can be resolved simply by adding clientHeldItem.removeComponent(LightComponent.class); as well, but I'm not sure what other sneaky components we might need to look for. Might be a good enough fix till we can rework this better :-)
Dunno what your ideas are for going forward, but the current code does seem awkward. It edits the sort of locally persisted held entity both before and after copying it in fresh from the newly selected held item. Fun things happen if you start messing with it though, like removing the pre-copy block entirely :D

I'd think we'd just want to split that apart further to make it cleaner. Completely separate the held item from the selected item's entity - which is what I thought simply re-copying it would do. But nope that just made other fun things happen. I guess we need a copy but strictly with only the components needed to draw it on the local client - as for other clients in multiplayer maybe they just need a copy synchronized over when it comes time to show other people's held items (which isn't quite yet)
Anyway, you probably have had all those thoughts already. Check out the lighting thing and see if you can find any other creative components. Probably we can merge this with just that one additional component removal added in :-)

Bump. Out of 3 current blockers one got fixed, one has another fix ready, and this is the third, nearly fixed. So close! :-)

Went ahead and applied the minimal LightComponent fix on top and merged this. Couldn't replicate my second issue. So this might be good and properly fixed - at least before any sort of overhaul makes it a moot point anyway :-)
We should keep an eye on any quirky behavior around switching held items in the meantime, especially for unusual items that may have components making some sort of impact while held.


modify ssm editable secrets to allow group of developers to edit variables under certain path

https://github.com/Selleo/terraform-aws-ssm/blob/main/modules/editable-parameters/main.tf

https://github.com/Selleo/terraform-aws-ecr/blob/v0.5.0/modules/repository/main.tf

add permission as iam.tf


Build Failing Randomly

Currently the build is failing randomly. As it seems, all test failures are related to Random* tests. Such tests should be predictable.

Looks like this is the reason why someone deactivated test execution during the build, see https://github.com/m-ender/retina/blob/master/build.sh#L8


Use open sans as font throughout

Already used in webchart objects. Apply to other sections as well.

Test Notes: Load a codebook in CAT and use the inspector in dev tools to confirm that open sans is used throughout (spot check several text elements on the various tabs)

Confirmed that Open Sans is used throughout  the Data Listing elements.


fix: overflowing content bug

Fixes Issue
Closes #732
Changes proposed

I have fixed the responsive bug.

Screenshots

Before
Updated

Note to reviewers

I have properly tested, and I believe this is the sole reason for the bug.
I have checked; other contents are not impacted.
I have verified this by increasing the header text and adding other custom values.

@Anmol-Baranwal i can not see this overflowing issue in linkshub.dev for some reason, could you please tell me what is the screen resolution are having?

@Anmol-Baranwal i can not see this overflowing issue in linkshub.dev for some reason, could you please tell me what is the screen resolution are having?

It only occurs for https://www.linkshub.dev/open-source-blogs, specifically, starting from a width of 616px or less.

I am using the Chrome browser.

Done as per suggestion, this commit fixes a bug where text is being truncated on small devices.

Small Devices
Large Devices

Done as per suggestion, this commit fixes a bug where text is being truncated on small devices.
Small Screen Size	Medium & Higher Screen Size

Thanks, looks perfect to me


TFLite Model Maker changes the order of the outputs for object detection

1. System information

Linux Ubuntu 18.04 as Windows subsystem for linux
TensorFlow installation (pip package or built from source): 2.8.0
TensorFlow Lite Model maker(0.4.2) installation: Installed all the requirements from Github repo, plus from prerequisites of object detection example

2. Code
As described in example
3. Changes the output order of object detection model.
As per the tflite model maker, the model generated by tflite model maker can be directly used in Task Library.  As per Model compatibility requirements, out put order is already defined. But in tflite model maker case, it changes the output order, in general it should be  location, classes, scores and number of detection, but here it is, score, location, number of detection, classes. Here is the output of generated tflite model meta data from tflite model maker.
{
  "name": "ObjectDetector",
  "description": "Identify which of a known set of objects might be present and provide information about their positions within the given image or a video stream.",
  "subgraph_metadata": [
    {
      "input_tensor_metadata": [
        {
          "name": "image",
          "description": "Input image to be detected.",
          "content": {
            "content_properties_type": "ImageProperties",
            "content_properties": {
              "color_space": "RGB"
            }
          },
          "process_units": [
            {
              "options_type": "NormalizationOptions",
              "options": {
                "mean": [
                  127.0
                ],
                "std": [
                  128.0
                ]
              }
            }
          ],
          "stats": {
            "max": [
              255.0
            ],
            "min": [
              0.0
            ]
          }
        }
      ],
      "output_tensor_metadata": [
        {
          "name": "score",
          "description": "The scores of the detected boxes.",
          "content": {
            "content_properties_type": "FeatureProperties",
            "content_properties": {
            },
            "range": {
              "min": 2,
              "max": 2
            }
          },
          "stats": {
          }
        },
        {
          "name": "location",
          "description": "The locations of the detected boxes.",
          "content": {
            "content_properties_type": "BoundingBoxProperties",
            "content_properties": {
              "index": [
                1,
                0,
                3,
                2
              ],
              "type": "BOUNDARIES"
            },
            "range": {
              "min": 2,
              "max": 2
            }
          },
          "stats": {
          }
        },
        {
          "name": "number of detections",
          "description": "The number of the detected boxes.",
          "content": {
            "content_properties_type": "FeatureProperties",
            "content_properties": {
            }
          },
          "stats": {
          }
        },
        {
          "name": "category",
          "description": "The categories of the detected boxes.",
          "content": {
            "content_properties_type": "FeatureProperties",
            "content_properties": {
            },
            "range": {
              "min": 2,
              "max": 2
            }
          },
          "stats": {
          },
          "associated_files": [
            {
              "name": "labelmap.txt",
              "description": "Labels for categories that the model can recognize.",
              "type": "TENSOR_VALUE_LABELS"
            }
          ]
        }
      ],
      "output_tensor_groups": [
        {
          "name": "detection_result",
          "tensor_names": [
            "location",
            "category",
            "score"
          ]
        }
      ]
    }
  ],
  "min_parser_version": "1.2.0"
}

How we can fix the issue with changed output order by TFLite Model Maker.

Hi @krunalvaghani !
Thanks for sharing your observation on changing order of outputs in metadata .
Workaround for this issue is to declare the sequence of outputs explicitly using metadata writer.
@sachinprasadhs !
Could you look at this bug.
Thank you!

By exploring tflite-model-maker, tensorflow lite, core issue is in tflite model generations using method from tf.lite.TfLiteConverter. It is some how not considering the name of the tensor.
For example, output layers of tensorflow model are,
{'output_1': TensorSpec(shape=(None, 100), dtype=tf.float32, name='output_1'),
 'output_2': TensorSpec(shape=(None, 100), dtype=tf.float32, name='output_2'),
 'output_0': TensorSpec(shape=(None, 100, 4), dtype=tf.float32, name='output_0'),
 'output_3': TensorSpec(shape=(None,), dtype=tf.int32, name='output_3')}

TfLiteConverter is not considering the name of tensor, to sort the output order. It converts on random order of output. It also happens with latest stable version TF 2.10.0

This is an expected behavior of TFLite converter. However, you can leverage signatures (https://www.tensorflow.org/lite/guide/signatures) to control your inputs/outputs for both training and inference.

If the above suggestion does not work for you, could you please check the workaround mentioned here to reorder the output using layer name.
https://github.com/tensorflow/tensorflow/issues/57043#issuecomment-1318333182

If the above suggestion does not work for you, could you please check the workaround mentioned here to reorder the output using layer name. #57043 (comment)

I have tried as it states in comment by giving output names to the output layers (link). That is not working.

@krunalvaghani , Could you try the comment here https://github.com/tensorflow/tensorflow/issues/47927#issuecomment-1499781808 to try with the tf.keras.layers.Identity to retain the output name.

@krunalvaghani
I have trained a yolov5 model and modified it to get 4 ouput arrays when converted to tflite
im getting 4 arrays loc,class,score,num_detections
but i have a issue that im getting box upon box issue
so while converting pt to tflite i have included nms to resolve the issue.......issue is gone now but output order got jumbled now im getting loc,score,class,num_detections
My need is a ready made tflite which gives loc,class,score,num_det ...where i need to upload to a camera which takes in this order
so please guide to change the order of tflite
now i hv location,score,class,num_detections
but i need location,class,score,num_detections


"." character in input name prevents error message from clearing

Possible solution to add to the list of dirty char.
var elmInputName = elmName.replace(/[.|&;$%@"<>()+,[]{}]/g, '');
Great package btw!

I don't understand your issue, can you be a bit more specific of your problem and can you make a Plunker to test with.

http://plnkr.co/edit/ArMzNELn9vKkWH9SErWQ
If the name attribute of the input field contains a DOT(.) char, the validation error messages will not be purged on input change.
Try clicking submit, or editing the input a few times and you will see.


Any plan to add a dashboard UI like apisix-dashboard?

Is there any plan to add a dashboard UI like https://github.com/apache/apisix-dashboard?

This is a feature of the enterprise edition, which is not open-sourced.

This is a feature of the enterprise edition, which is not open-sourced.

Cool, thanks for you reply.


ISPN-12353 REST allow backup restore progress to be submitted and polled

https://issues.redhat.com/browse/ISPN-12353
https://issues.redhat.com/browse/ISPN-12381

looking

This now depends on https://github.com/infinispan/infinispan/pull/8752 which makes it so that templates are replicated across the cluster and persisted in the globalstate.


Allow borrowing without underscore in pattern matches.

rdar://126305009

Please test with following pull request:
https://github.com/apple/swift/pull/72995
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift/pull/72995
@swift-ci Please test

https://github.com/apple/swift/pull/72995
@swift-ci Please test Windows


fix(userpreviewmodal): fix #2495 - change Ip into IP

fix #2495

@BronzW Can you tell me what to do? Because I'm unaware of the error

@BronzW Can you tell me what to do? Because I'm unaware of the error

your scope for the commit is wrong, it can only be one of these. so fix simply run:
git commit --amend -m "fix(kibbeh): fix #2495 - change Ip into IP "


Data splits for all datasets

Hi,
While you make the 'train_val_split.npz' files available, can you specify what the data splits are for all the datasets (Flickr, CHI3D and Hi4D)?
Thank you!

Hi laubravo,
you can download the train/val/test splits from here
The Flickr Fits data for val/test should be available later today.
Best wishes,
Lea

These are the GT 3D joints from Hi4D and CHI3D projected onto the image plane to get the people in contact and then solve the correspondence with BEV and OpenPose/ViTPose keypoints. I pushed the two scripts:
CHI3D
Hi4D


SAK-31705 Added 'Search Users' facility to site-info.

Added Search input and clear search in the chef_site-siteInfo-list.vm.
If search word is in session state then Participant List is filtered for
the search term in each Participant's displayName and displayId.
Corrected the position of search box in the participant's table.
Have added new variable to STATE_SITE_PARTICIPANT_LIST in SiteAction class
to save the original site members list. Search is case insensitive.

This is how it looks in 11.

Minor comments in-line; otherwise LGTM

@ouit0408 Are you able to fix the comments on this one?

it is in my to-do list, will fix.

implemented review comments and PR is updated.

Thanks.


Too many levels of symbolic links

I get the following error if I run this container:
Check environment configuration...
Configure Toran Proxy...
cp: failed to access '/data/toran-proxy/toran': Too many levels of symbolic links

cp: failed to access '/data/toran-proxy/toran/config.yml': Too many levels of symbolic links

sed: can't read /data/toran-proxy/toran/config.yml: Too many levels of symbolic links
sed: can't read /data/toran-proxy/toran/config.yml: Too many levels of symbolic links
mkdir: cannot create directory '/data/toran-proxy/toran': Too many levels of symbolic links
cp: failed to access '/data/toran-proxy/toran/composer/auth.json': Too many levels of symbolic links

Installing Token Github...
sed: can't read /data/toran-proxy/toran/composer/auth.json: Too many levels of symbolic links
Installing Cron...
ln: failed to create symbolic link '/var/www/app/logs/downloads.private.log': File exists

Loading permissions...
Configure PHP...
Loading Nginx vhosts...
Loading HTTPS Certificates...
Using provided HTTPS Certificates...
ln: failed to create symbolic link '/usr/local/share/ca-certificates/toran-proxy/certs': File exists

find: File system loop detected; `/usr/local/share/ca-certificates/toran-proxy/certs' is part of the same file system loop as `/usr/local/share/ca-certificates/toran-proxy'.
Updating certificates in /etc/ssl/certs... 0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d....done.
Starting Toran Proxy...
2016-10-21 16:21:19,065 CRIT Set uid to user 0

I use this command to start the docker container:
docker create --name="toran-proxy" --net="bridge" -e "TORAN_CRON_TIMER"="minutes" -e "TORAN_TOKEN_GITHUB"="xxxxxxxxxxxxxxxxxxxxxxxxxxx" -e "TORAN_HTTPS"="true" -e "TORAN_HTTPS_PORT"="444" -e "TORAN_HOST"="<IP_ADDRESS>" -p 444:444/tcp -v "/mnt/user/appdata/toran-proxy":"/data/toran-proxy":rw cedvan/toran-proxy

I use Unraid so maybe the command is a little bit different then normal.
Thank you

@SolidRhino Are you using the newest version? This bug has been corrected in the past.

Hi @SolidRhino,
On what OS is start the toran-proxy container ?

I closed, reopen if necessary @SolidRhino


Fir 30153 support use engine in node sdk

Added support for use engine statements to node sdk

updated implementation based on the doc: https://docs.google.com/document/d/1f0kdPCPbhCrmMTK-sfqAkqEkonT7xyNK-teJmXE_6p0
added unit and integration tests
updated CI

Integration tests: https://github.com/firebolt-db/firebolt-node-sdk/actions/runs/8155414683


Implement AddParent() method

this would resolved #33

@chirino Thanks for the pull request. I will check this tomorrow

Do you want the cleanup function to be called for each container separately?

Yes.  I think the typical usage will have child containers having a shorter lifespan than parents.  So I’m not sure we can do much to safeguard proper cleanup. It’s up to the user to clean up parent containers once they are not being used by children.

@chirino It looks good. I plan to merge this into the master today and test it a bit on my apps before release

Is it ready for a release yet?

@chirino Yes, sorry for delay. I will prepare release today


added hover card effect

Closes #29
Changed 2 files- index.js and styles.module.css
Added hover card effect on Home Page features
https://github.com/subhadipbhowmik/30-Days-Of-CPP/assets/98379619/434882cc-4868-4840-962d-f280e0779f9d
Kindly verify and merge.

Thanks for your contribution and for raising the PR.. You did great  job.


Cilium function extraction test (list of functions can't be extracted)

commented_OPENED_cilium_bpf_overlay.c:

handle_ipv6
tail_handle_ipv6
handle_ipv4
tail_handle_ipv4

commented_OPENED_cilium_cilium-probe-kernel-hz.c:

main (we are not sure if this is a problem but since there are many main functions, probably still need a main for a specific file to see how it uses those functions?)

commented_OPENED_cilium_sockops_bpf_redir.c:

bpf_redir_proxy

commented_OPENED_cilium_sockops_bpf_sockops.c:

bpf_sock_ops_ipv4
bpf_sock_ops_ipv6
bpf_sockmap

other failing functions

sock4_connect

cqsearch is now working for the above functions..

@OwenZhang001122 can you ensure that you reinstall codequery ... by doing:

git pull inside the codequery folder
cd to build directory
run ninja
run ninja install
If you remember from our last session, these steps are necessary as docker build is not working in your environment.

closing as the issue doesnt occur in our test env. please reopen if it persists


Tutorial Page btp-app-kyma-deploy-application.md Issue. PROD

Tutorial issue found: https://github.com/sap-tutorials/Tutorials/blob/master/tutorials/btp-app-kyma-deploy-application/btp-app-kyma-deploy-application.md contains invalid tags. Even though your tutorial was created, the invalid tags listed below were disregarded. Please double-check the following tags:

software-product>sap-btp-kyma-runtime

Affected server: PROD

fixed yesterday


Add "Copy URI" for resources with URI properties in the sidebar

We currently support Copy Name, Copy ID, (for Kafka clusters) Copy Bootstrap Server(s), and (for CCloud orgs) Copy Organization ID, but having a Copy URI action can also help users who may require the values for setting up project templates and/or other clients.
Currently this only applies to the following resource types:

CCloud+local Schema Registry
local Kafka cluster

Done with https://github.com/confluentinc/vscode/pull/463


Allow dozer to be managed directly by the camel-transform component

Instead of allowing the DozerTypeConverterLoader bean to manage Dozer, allow the user to pass a Dozer config through a transform endpoint URI.  Dozer would be managed locally in the component.

https://github.com/fabric8io/data-mapper/pull/94

pushed


LocalDate.ofEpochDay may return a wrong result

LocalDate.ofEpochDay(x) sometimes returns a wrong or illegal result instead of throwing an exception for large values of x. For instance, LocalDate.ofEpochDay(9223371671611556645L) returns a date with a negative value for d.getDayOfMonth() instead of throwing a DateTimeException.

I too have experienced this issues on my android app but it seems to be limited to only certain models. This happens only on Moto g 3rd Gen

I've had this issue for a few weeks in my Android app. It seems to only happen on Motorola devices running Android 6.0: http://stackoverflow.com/questions/35043000/motorola-devices-org-threeten-bp-datetimeexception-when-parsing-a-date-in-thre

For what it's worth, I can reproduce this bug on both Java SE 7 and 8 (64 bit edition, Mac OS X), so this bug is not specific to Android.

We're hitting this on Android; haven't yet identified a particular input value that causes the crash.


update readme with instructions for BLEU-1 to BLEU-4

@sgrvinod added the details in FAQs.

Merged, thanks. I made some minor edits.


Fixed 'unlimited archiving' references

Docs Build status updates of commit 871f265:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Enable-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add index.html

Documentation
what did I learn so far ? And what am I doing ?
I have created a pull request to add the index.html which is the add-index branch created by the BOT!😁
Where are we ?
we are in the im/intro-html (Public repo) created by the bot. It taught me that its not enough to just create html pages we need to serve them somewhere. In order to serve you website on the internet you need to deploy your website to the web hosts or (github pages) . These host are connect to the internet.
By changing the settings on in github pages and set the dropdown to main as the publishing source.
https://imshayan.github.io/intro-html/
what is index.html so important ?
whenever the github pages tries to serve your webpage it looks for file called index.hml and if it does not gets it it renders the readme.MD file as default

what is happening ?
we are merging the changes from add-index branch into the main branch and then deleting the branch add-index this branch had the changes in index.html files we need to render something else on our currently served webpage on gitHUB pages


[website] Refactor on existing Tutorials part structure

Description
According to issue (#1081), the main structure of the website examples part needs to be refactored. Currently this pull request deals with part of this issue. This pull request refactors all the existing files in the examples part of the website. All existing structure of the examples part of the website are modified accordingly. For new features about adding new files to the examples part of the website, they will be handled in future pull requests.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

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

cc @dmlc/gluon-nlp-team

Job PR-1154/6 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/6/index.html

Job PR-1154/7 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/7/index.html

After a pilot experiment and research, I have found that I could only add up to 3 cards in a cards container in the rst file.
Would you mind changing the structure a bit to limit to 3 cards or less; Or do you want me to do some more research to find out how to add more cards?
Please refer to the difference before and after commit 62f7eb0 to find out how it fails on four cards.
@eric-haibin-lin
Thanks for your reply!

@StrayBird-ATSH what did you mean by it failed? Are you referring to this error msg?
[2020-02-13T11:55:37.743Z]

[2020-02-13T11:55:37.743Z] Exception occurred:

[2020-02-13T11:55:37.743Z]   File "/var/lib/jenkins/workspace/gluon-nlp-cpu-py3-master/conda/cpu/py3-master/lib/python3.5/site-packages/mxtheme/card.py", line 25, in run

[2020-02-13T11:55:37.743Z]     cid = nodes.make_id("card-{}".format(options['title']))

[2020-02-13T11:55:37.743Z] KeyError: 'title'

[2020-02-13T11:55:37.743Z] The full traceback has been saved in /tmp/sphinx-err-obhz33o5.log, if you want to report the issue to the developers.

[2020-02-13T11:55:37.743Z] Please also report this if it was a user error, so that a better error message can be provided next time.

[2020-02-13T11:55:37.743Z] A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!

[2020-02-13T11:55:38.303Z] Makefile:57: recipe for target 'html' failed

[2020-02-13T11:55:38.303Z] make[1]: *** [html] Error 2

[2020-02-13T11:55:38.303Z] make[1]: Leaving directory '/var/lib/jenkins/workspace/gluon-nlp-cpu-py3-master/docs'

[2020-02-13T11:55:38.303Z] Makefile:61: recipe for target 'docs' failed

[2020-02-13T11:55:38.303Z] make: *** [docs] Error 2

http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/7/examples/word_embedding/index.html
Looks like there's some rendering issue

Yes. I was referring to that error messages. That error prevents the entire website from being built. Even though it didn't mention what on earth caused the error, I gradually found out that it was the 4 cards causing it fail. It will compile successfully on 3 cards.

http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/7/examples/word_embedding/index.html
Looks like there's some rendering issue

Thanks for pointing out this. This seems to be caused by a minor missing space problem. I have already fixed it in the latest commit. For now you can preview the website of the 6th commit in this pull request.
Thanks!

@szha are u aware of any limitation on the number of cards for the theme??

@szha are u aware of any limitation on the number of cards for the theme??

Sorry... It seems that it is the missing indentation among one of the four cards that prevents the cards from compilation. Now I finds it... The cause was so trivial... Now hopefully this problem can be solved.
Perhaps if I didn't put the original three cards on and let the indentation-caused rendering problem come out, I would never realize it was the indentation causing this problem.

Job PR-1154/8 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/8/index.html

Job PR-1154/9 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/9/index.html

I think now this pull request should be ready for reviewing and merging.
Please give any advice if my refactor doesn't meet what you wish.

Yes. For the API notes, I plan to deal with them in the next step so that in these pull request, all the changes are concerning existing examples files, for more simplicity.
I will change the .md files accordingly. Thanks for your remind.

Job PR-1154/10 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1154/10/index.html

@eric-haibin-lin For your review.
Thanks!

Good job. Look forward to the community page and data/vocab API notes


How do I tell if fiber finished?

@bjouhier  Probably dumb question.  As I understand it, each 'run' call will run until the code returns, or until a 'yield' occurs - my question is how do I tell the difference between these two cases?  Of course, I may not have understood correctly.  Thanks!

Yes, that's right. I usually track that with a special variable. For example the yielded variable in https://github.com/Sage/streamlinejs/blob/master/lib/fibers/runtime.js#L175-213

@bjouhier That makes sense!  Thanks for getting back to me so fast!  Regards, P.


GDWeave Being flagged as virus with suspicious files

Good afternoon
I downloaded GDWeave through the github and windows immediately flagged it as a PUP.
I ran the winmm through virus total and found some suspicous results

What is "Sexinminecraft.zip"

https://github.com/NotNite/GDWeave/issues/25


GEODE-8364: Change log level at runtime

Do you have a use case for this?  I don't suppose it's really harming anything, but I'm not sure I understand why/how you'd want to change this midstream, and it could cause problems down the road for things like log analysis tools.  We've also been planning for a long time to completely replace our logging mechanism with an off-the-shelf solution like spdlog or log4cpp, and this would add a new logging feature that we'd have to replicate with any such change.

It will be useful for us to have this feature in order to get more info when there is a problem with the client. If we detect an issue in the client, we can activate debug log to get more info about what is happening. This will help a lot our engineers to troubleshoot and report problems.

@pivotal-jbarrett could you please check if the changes are ok for you? Thanks


Many links on a wrapped line in the terminal causes performance issues

When mousing over the terminal:

/verified


Initial symcrypt + scossl commit

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary
Adding SymCrypt and the Symcrypt Engine for OpenSSL as packages
Change Log

added SymCrypt package
added SCOSSL package

Does this affect the toolchain?
No
Test Methodology

Pipeline build id: xxxx

Why have the libsymcrypt.so binary in the scossl spec? scossl should depend on symcrypt and link against the symcrypt binary in /usr/lib

Closing this PR.  Symcrypt is now available on Mariner 2.0.


error(apple_pie): An error occured handling request: 'BrokenPipe'

Unfortunately didn't get a stack trace but this showed up in my logs recently, thought I'd make a tracking issue

Thanks for reporting!
This occurs when we try to write a response to the client, but the client has already closed its connection. I should probably catch this error and close the connection when this happens.


Spelling tools swift inspect

This fixes some misspellings in Swift tools/swift/inspect, it is split per https://github.com/apple/swift/pull/42421#issuecomment-1101092698

@swift-ci smoke test and merge

@swift-ci smoke test macOS


Request: Stop Task & Parallel Execution Feature

I was wondering if it would be possible to implement a "stop task" feature. Currently, when executing a task, such as making an API request or running a process, it seems necessary to start a completely new task if you want to discontinue or halt the previous one. It would be helpful to have the ability to stop an ongoing task before moving on to another.
Additionally, I would like to know if there is a way to execute multiple commands simultaneously. For example, if "Claude" is testing the frontend and the process is set to run in the background, it would be great to also initiate a backend process concurrently. After both tasks are completed, it would be useful to analyze and compare the output from both operations in a unified manner. This would enhance workflow efficiency by allowing multiple processes to run in parallel without manual task switching.

I am working on a way to cancel API requests, but you can now resume tasks + run commands in the background which streams its output to claude as he works in the foreground. If you have any other ideas pls feel free to add them here so people can vote on them: https://github.com/saoudrizwan/claude-dev/discussions/categories/ideas


The table is now displayed.

Reformatted demo.json to be a valid json file.

@raywo  Hi Ray! Wow!! Thank you so much for sending me this PR!!
I've just been testing it (in a roundabout way) and it look great!
However, I already did some changes to my own code before finding your PR, so I will not merge this one, but instead just make a new push from my side with most of your edits. I know this is not the ideal/right way to do it, but I find it just simpler than merging, pulling and re-doing...
In addition, I noticed it was rather hard to see what changes you made since you reformatted all the code from using TABs to spaces, thus everything get highlighted! That is just fine, and I had forgotten since I'm on remote that my nano was not using spaces. I'll fix this as well.
BTW. The JSON as shown in the file is "raw" and is what I get from the API, so the proper formatting has to be done in the module. I'll have a look at that next.
Thanks again for taking a serious look at this!  I was just about to give it up.  :champagne:  :+1:


Deletion and re-deploy does runs into errors

I still see a deletion problem: I deleted 3 Functions on ACA at 8:16 but was not able to do a fresh deploy at 8:18

{\"error\":{\"code\":\"Conflict\",\"message\":\"Request specified that resource '/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx/resourcegroups/kw-m2-rg/providers/Microsoft.Web/sites/kw-m2acafrecvexp' is new, but resource already exists.
This may be due to a pending delete operation, try again later.\"}}

Waiting some more minutes then brings up a different error:

{"code":"DeploymentFailed","target":"/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx/resourceGroups/kw-m2-rg/providers/Microsoft.Resources/deployments/acafBase-Distributor","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"InternalServerError","message":"There was an unexpected InternalServerError.  Please try again late
r.  x-ms-correlation-request-id: 0c897159-bb41-43e4-845e-5c7c7042e13a"}

How long is one is supposed to wait? And if this is to be expected, it should be mentioned in docs.

issue is fixed now


Jamal: Watch rolexreplicasstore.uk.com

Jamal requests the watch of the watch_keyword rolexreplicasstore.uk.com. See the Metasmoke search here and the Stack Exchange search here.

!!/approve

Coverage remained the same at 63.155% when pulling c470f689accc9fbb3cad46165ad37ac421495778 on auto-blacklist-1512784849.0626802 into 736a92cb57f384b7530113f748b6bf910fac9a65 on master.


Add linkedin social icon to footer and content pages

https://www.linkedin.com/company/participedia

I also just realized that the links in the footers are share links - do you think the footer icons should direct people to our actual social pages rather than be share links @samjianga?
Share links make sense on content pages.
*Not huge priority, but when you get around to it.

Done in ##378

@andreadelrio linkedin icon still needs to be added to the sidebar below fb and twitter on content pages. I just looked at a case and dont see it yet.
Also, the linkedin icon on the footer looks less sharp on my screen compared to the FB and twitter icons. Is it just me? Thanks!


Update create-maps-data-transformation-visual-studio-code.md

Removed the limitation for Windows operating systems only. I'm running the Data Mapper on MacOS.

@ecfan
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @msitsoftware, I checked with the product team, and they've responded that currently only Windows is supported.
If this changes, we'll update. Thanks again!
#please-close


Tests for out JS.

We should make some tests for our JS. Not for angular's but the one i made outside to filter data etc.

Karma - enviroment
Jasmine - test framework

http://gruntjs.com/


[Feature Request] Add ability to listen to identity changes

I think the documentation incorrectly suggests that there is an ability to call analytics.user().id() or analytics.user().anonymousId(), but user() returns a promise.
Knowing this, I'm looking for a way to listen to identity changes for me to keep tools in sync.
I essentially want to do something like this:
useEffect(() => {
  Sentry.setUser({ id: segmentUserId });
  LogRocket.identify({ id: segmentUserId });
}, [segmentUserId]);

Where segmentUserId is either the automatically applied user.anonymousId or user.id (if set).

this.emit(
'identify',
ctx.event.userId,
ctx.event.traits,
ctx.event.options
)
Hey @kylemh, we have an event emitter.
analytics.on('identify' (userId) => ... )
PS:
Yep, you've noticed a little weirdness: the docs use the snippet API (which use the raw Analytics instance), and there is a slight API difference between that and the API when using the regular npm library -- in the latter, everything is a promise.
You can always get around this by doing:
analytics.instance?.user().anonymousId()

@silesky I was looking for this feature/API as well. Below is the solution I came up with that I think/hope? will be accurate. Would be great if there was an explicit built-in API for this.
@kylemh would love to see your solution if you can share.
// This hook returns the current Segment userId and anonymousId and _should_
// automatically update whenever those ids are set or change. Unfortunately
// Segment doesn't offer an API for this so we approximate it by checking for
// changes after each render (using the asynchronous analytics.user() API) and
// after each call to 'identify' or 'reset'.
export function useSegmentIds() {
  const [userId, setUserId] = useState(analytics.instance?.user().id())
  const [anonymousId, setAnonymousId] = useState(analytics.instance?.user().anonymousId())

  useEffect(() => {
    let cancelled = false

    const updateIds = () => {
      if (cancelled) {
        return
      }

      const newUserId = analytics.instance?.user().id()
      if (userId !== newUserId) {
        setUserId(newUserId)
      }

      const newAnonymousId = analytics.instance?.user().anonymousId()
      if (anonymousId !== newAnonymousId) {
        setAnonymousId(newAnonymousId)
      }
    }

    analytics.user().then(updateIds)
    analytics.on('identify', updateIds)
    analytics.on('reset', updateIds)

    return () => {
      cancelled = true
      analytics.off('identify', updateIds)
      analytics.off('reset', updateIds)
    }
  })

  return [userId, anonymousId]
}

I'm using a Next.js app, so I tied into the route change emitter:
type AnalyticsUser = { anonymousId: string; id?: string; traits?: Partial<Traits> };

export const RenderlessAnalyticsComponent = () => {
  const { events: routerEvents } = useRouter();
  const [analyticsUser, setAnalyticsUser] = useState<AnalyticsUser | undefined>(undefined);

  useEffect(() => {
    // If the user has a legit ID, use it; otherwise, fallback to anonymous ID.
    if (analyticsUser?.id) {
      LogRocket.identify(analyticsUser.id, { ...analyticsUser.traits });
      Sentry.setUser({ id: analyticsUser.id, email: analyticsUser.traits?.email ?? undefined });
    } else {
      Sentry.setUser({ id: analyticsUser?.anonymousId });
    }
  }, [analyticsUser]);

  useEffect(() => {
    routerEvents.on('routeChangeComplete', () => {
      analytics.user().then(user => {
        setAnalyticsUserId({
          id: user.id() ?? undefined,
          anonymousId: user.anonymousId()!,
          traits: user.traits()
        });
      });
    });
  }, [routerEvents]);

  return null;
};


Can't add Autoload: res:// is an invalid path. File does not exist.

Hello,
I've tried to install the plugin by:

Downloading everything from github
unpacking the zip
copying the whole folder to the "addons" directory of my project
going to the plugins section in my project settings and activating the plugin

Then I get the message
Can't add Autoload:
res:// is an invalid path. File does not exist.

Also, the plugin does not seem to work then.
How can I fix this?
Thank you!

Please install the plugin through the Asset Library built into the editor. Also please note that this plugin only supports Godot 4.x.


Update dep5

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Thank you @ajinkyapatil8190 for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.


It's slow when the file is large

I tested this on a 2000 line ts file and it took 10 seconds to expand all the symbols.

You can use this https://github.com/simrat39/symbols-outline.nvim/pull/223


Scala 3.0.0

Broken on Scala 3, but so is the main branch.
Missing:

[x] blaze (https://github.com/http4s/blaze/pull/531)
[x] case-insensitive (https://github.com/typelevel/case-insensitive/pull/153)
[x] vault (https://github.com/typelevel/vault/pull/234)
[x] unique (https://github.com/typelevel/unique/pull/191)
[x] log4s (https://github.com/Log4s/log4s/pull/76)
[x] keypool (https://github.com/typelevel/keypool/pull/211)
[x] jawn-fs2 (https://github.com/typelevel/jawn-fs2/pull/204)

Some nonsense about this:
[error] (update) sbt.librarymanagement.ResolveException: Error downloading org.scala-lang:scala3-library_3.0.0:3.0.0
[error]   Not found
[error]   Not found
[error]   not found: /home/ross/.ivy2/local/org.scala-lang/scala3-library_3.0.0/3.0.0/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/org/scala-lang/scala3-library_3.0.0/3.0.0/scala3-library_3.0.0-3.0.0.pom
[error]   not found: https://jcenter.bintray.com/org/scala-lang/scala3-library_3.0.0/3.0.0/scala3-library_3.0.0-3.0.0.pom

We're blocked on a final release without a Circe release.

This org.scala-lang/scala3-library_3.0.0/3.0.0 should be org.scala-lang/scala3-library_3/3.0.0. It might be because of some plugin used by http4s.
I don't think you need sbt-dotty anymore. Could you try sbt 1.5.2 instead? Or let me see if I can help anything after work today.

PRs up for case-insensitive and vault.
I hit the same issue where something pulls in an old version of sbt-dotty which makes resolution fail.

PR for blaze is up, but it's missing a log4s dependency.

@aeons I think it will be released soon.

https://github.com/Log4s/log4s/pull/75
https://github.com/Log4s/log4s/pull/76

Yes, I linked to that from the blaze PR :)
There might be an issue with the build pipeline though, since the RC Ross made yesterday is not on maven central.

The PR I made for vault was for the CE3 branch. Now there's one for the CE2 branch as well: https://github.com/typelevel/vault/pull/234
That depends on a release of unique though: https://github.com/typelevel/unique/pull/191

There should be PRs up for all dependencies now.

All that's missing now are releases of vault and blaze.

I think vault is available now but I get this when I sbt compile on my local machine.
[error] Symbol 'type org.http4s.jawn.JawnInstances' is missing from the classpath.
[error] This symbol is required by 'package org.http4s.jawn.package'.
[error] Make sure that type JawnInstances is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'package.class' was compiled against an incompatible version of org.http4s.jawn.
[error] one error found
[error] stack trace is suppressed; run last blaze-core / update for the full output
[error] (jawn / Compile / compileIncremental) Compilation failed
[error] (blaze-core / update) sbt.librarymanagement.ResolveException: Error downloading org.http4s:blaze-http_2.13:0.15.0

jawn is weirdness from package object inheritance, and a clean should fix that.
I'll check on the blaze release.

https://github.com/http4s/blaze/issues/505 ate blaze despite an attempted fix.  I'm just publishing it manually from the tag.

@rossabaker Thanks! Yeah I also just found that blaze-http 0.15.0 is not released.

Some nonsense about this:
[error] (update) sbt.librarymanagement.ResolveException: Error downloading org.scala-lang:scala3-library_3.0.0:3.0.0
[error]   Not found
[error]   Not found
[error]   not found: /home/ross/.ivy2/local/org.scala-lang/scala3-library_3.0.0/3.0.0/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/org/scala-lang/scala3-library_3.0.0/3.0.0/scala3-library_3.0.0-3.0.0.pom
[error]   not found: https://jcenter.bintray.com/org/scala-lang/scala3-library_3.0.0/3.0.0/scala3-library_3.0.0-3.0.0.pom

We're blocked on a final release without a Circe release.

@rossabaker Just FYI, I think I found the reason why this happened.
When I do show scalaBinaryVersion, I get 3.0.0 but it should be 3. I found one of the plugins used in http4s overrides it.
I think It's dotty.tools.sbtplugin.DottyPlugin used by SpiewakPlugin. I can see the DottyPlugin was dropped from sbt-spiewak but I don't think the new one was released yet.

Yeah, that's the issue. I created https://github.com/djspiewak/sbt-spiewak/issues/55

It now fails on the unusedCompileDependenciesTest with a lot of false positives.
[warn] scalatags >>> The following libraries are declared in libraryDependencies but are not needed for compilation:
[warn]  - "com.lihaoyi" % "scalatags" % "0.9.4"
[warn] boopickle >>> The following libraries are declared in libraryDependencies but are not needed for compilation:
[warn]  - "io.suzaku" % "boopickle" % "1.3.3"
[warn] twirl >>> The following libraries are declared in libraryDependencies but are not needed for compilation:
[warn]  - "com.typesafe.play" % "twirl-api" % "1.5.1"
[warn] play-json >>> The following libraries are declared in libraryDependencies but are not needed for compilation:
[warn]  - "com.typesafe.play" % "play-json" % "2.10.0-RC2"
[error] (play-json / unusedCompileDependenciesTest) Failing the build because unused dependencies were found
[error] (twirl / unusedCompileDependenciesTest) Failing the build because unused dependencies were found
[error] (scalatags / unusedCompileDependenciesTest) Failing the build because unused dependencies were found
[error] (boopickle / unusedCompileDependenciesTest) Failing the build because unused dependencies were found

It doesn't look like GitHub Actions is functioning (again 😔).

Yeah, I have tried canceling and restarting a couple of times.

Oh, the unused compile dependencies are the things we use withDottyCompat for so we can compile them until there's a proper release.  I'll change it to opt out on the Scala 3 build.  That check has saved us from bugs a few times, and I'd really like to keep it.

Yeah, it is alteret disabled on scala 3. The errors showed on 2.12 and 2.13.

Oh.  Well, nuts.  Hopefully it's only a temporary thing, then.

The build failures now are the moved blaze client stuff, so I guess this is good to go.
@rossabaker

@rossabaker I'm not sure if it's actually an issue but I got this when I tested this branch locally. #4848


More improvements to typeahead

Better styling and user experience using the typeahead.

There are far more conflicts than I anticipated. If you need any guidance on what changes are the correct ones, I'd be happy to help in the merge.


[fix](auto bucket) Fix auto bucket regression case occasional fail

Proposed changes
Issue Number: close #xxx

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

run buildall


Barriers are ignored

There are many route-blocking barriers like gate, lift_gate, cycle_barrier or bollard in the OSM data. Graphhopper does not evaluate them. This will lead to rouiting through the obstacles.
Most of these barriers are tagged on nodes. These nodes are always pillar nodes.
What would be the proper way to introduce such blocks into the graph?

unjoin the way at the blocked node and add it as two disconnected ways?
split the way in 3 and replace the blocked node with a very short edge which is marked with neither FORWARD nor BACKWARD? (Is it possible to block an edge this way?)
Something completely different is required?

If it works, I would suggest 2. as the block can be different for foot, bike and car.

In the util folder, opening up FootFlagEncoder, there are a lot of references to "safehighwayTags" and "intended/restrictive values". In what directory are these methods defined in their functionality?


[BEAM-4059] Bump Dataflow ValidatesRunner Timeout to 5h

R: @kennknowles @swegner

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Run ./gradlew build to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

run seed job

LGTM


Collection model attribute

The model attribute of a collection expects a constructor, as opposed to the models attribute, which is an array of model instances. I personally find this syntax confusing, and would prefer one where the constructor attribute is uppercased (or if both formats are supported). I take it there may be a very good reason for this syntax, which I would find helpful to know. Any thoughts?

See https://github.com/jashkenas/backbone/issues/1765. I personally hate the lowercased name... :sigh:


[DOCU-2635] Product name variables

Adding product name variables for mesh, gateway, and KIC. They're currently matching what we have on docs.konghq.com here: https://github.com/Kong/docs.konghq.com/blob/main/jekyll.yml#L174-L184
I only added those three variables for now as the only ones that should appear in Kuma.
To use a variable in content, use the tag {{site.<some_name>}}. For example, {{site.mesh_product_name}}.
https://konghq.atlassian.net/browse/DOCU-2635

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending?


feat: merge upstream 1.0.0

Description

Closes #64
Types of changes

[ ] Bug fix (changes which fixes an issue)
[ ] New feature (changes which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[ ] I followed the contributing guidelines.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] The PR title and commits are followed conventional commit form.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 4 committers have signed the CLA.:white_check_mark: shiki-tak:x: webmaster128:x: Deployer:x: ethanfreyDeployer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Could you modify commit titles only "fix"? (it. is ok to use force push)

Could you modify commit titles only "fix"? (it. is ok to use force push)

Oops...I had forgotten to put together my commits, thx.


Several problems with string references, object data and few more

https://easyupload.io/v9c691
This is Werewolf transylvania map.
Here's the original map, the deprotected map used by the tool, and the deprotected map by me (the one with "good" word). Along with additional files for comparison.
Strings on .wts related to quests title and description are being wiped out, yet in war3map.j references those that no longer exist
Major problem at the moment is neutral passive units become units owned by player 16, this is most likely due to the patch version which the map was saved in. So in really it should be Player 28 (27 in JASS) instead.
Unit object data has some flaws, incorrect values such as unit types n00R, H01E, H01V, H01D, H01X having "none" movement type instead of "foot" as it should be. Other things like RGB colours and such. Ability object data is also a bit screwed.

Thanks for the bug report!
Your notes are extremely detailed, which makes it very easy for me to troubleshoot, thanks!
I think most of these were caused by a major bug in version <IP_ADDRESS>. It was working better in previous versions. I've removed that build & fixed it in the latest version (<IP_ADDRESS>). Could you please re-test & update this with the list of remaining bugs?

Everything works perfectly except one thing. Compiler errors, which, as you mentioned has to be solved manually. These ones were "Function redeclared" errors such as Unit000042_DropItems, Doodad000465_DropItems, Doodad000467_DropItems, etc. Looks like there are more DoodadXXXX_DropItems than UnitXXXX_DropItems.
I exported the whole map script on this new deprotected version: https://easyupload.io/tl9bh0

I missed a problem on the latest deprotected version I made: https://easyupload.io/gem3ty
Adrenaline Rush ability, 'A02G' has a buff "Unknown (B000)"
While on my "good" version which was sent before, shows the ability actually having a buff Panic, 'B000'.


integral_constant does not model Euclidean Ring

consider this:
BOOST_HANA_CONSTANT_CHECK(hana::uint_c<1u << 31> * hana::uint_c<2> == hana::uint_c<0>);
(p.s.: i love you and your library (nohomo))

Hmm, actually the problem here is that overflow is defined to wrap for unsigned integral types. Because if you say for example hana::int_c<1 << 31> * hana::int_c<2>, then you get a compile-time error saying the result is not representable in an int.

It would be very unfortunate if we were unable to provide % and / for unsigned integral types... But at the same time the laws are not respected as-is, which is not acceptable. Do you have a resolution in mind?


Deploy Opendata instance @ SSL

It could be used for AGC https://github.com/iris-hep/analysis-grand-challenge

Already done as a part of https://github.com/iris-hep/analysis-grand-challenge/issues/2


[3.6] fix gauge assignment operators

Scope & Purpose
(Please describe the changes in this PR for reviewers - mandatory)

[X] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[X] :book: CHANGELOG entry made

Backports:

[X] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[X] This change is a trivial rework / code cleanup without any test coverage.
[X] The behavior in this PR was manually tested
[X] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/12816/
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12979/

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13266/

Closed in favor of https://github.com/arangodb/arangodb/pull/13184 according to https://github.com/arangodb/arangodb/pull/13184#pullrequestreview-548547271.


Set custom licenses

See #135 for background

@AndreyKozlov1984 - everything works as expected, thank you so much! Could you please add the last custom license, CC-BY-SA-4.0 ? FYI - info is in https://spdx.org/licenses/CC-BY-SA-4.0.html . Thank you!

Just change a license field to any value you want!
Sent from my iPhone

On 23 Mar 2022, at 20:36, Maurizio Pillitu @.***> wrote:
pushing landscape.yml
File Changes  (1 file)

All fixed, thanks so much @AndreyKozlov1984 !


Update SDK to support sending requests using API keys

Updates our standard network interface with a method to set the api key.
Updates underlying network config to store and retrieve the API key.
Updates GSN client relay sending to include API key in auth header
Updates README for steps involved in new API key process.
Updates Example app to set the API key
Introduces a private config file not included in git where we can
store the API key for each devs local environment

I've only updated the mumbai config to point at the saas for now. Feels like we should wait to change where the prod config points until everything is properly deployed and well provisioned.

Looks like the typescript compiler is unhappy because of a missing private_config.ts file, which makes sense since it doesn't exist. Looks like the CI config isn't on main yet though, so I can't easily update it as part of this PR.

🤦 realized I was just missing the codemagic file in my editor for some reason. Updated CI to do the necessary config setup, which should get this branch building clean again.

@DonGambas realized I hadn't updated the example expo app, so made another commit to make sure that is also updated to relay transactions through the saas using an API key.


Background is being ignored on a HCanvas

The background property is being ignored when is set on an HCanvas.
Example:
import hype.*;

HCanvas hCanvas;
HRect rect;

void setup() {
  size(500, 500);
  H.init(this).background(#00ff00); // this is green

  hCanvas = new HCanvas().background(#ff0000); // this is red
  H.add(hCanvas);

  hCanvas.add( rect = new HRect(100) ).fill(#ff44bb).rotate(45).loc(width/2, height/2);
  H.drawStage();
  noLoop();
}

Screenshot

Expected behaviour
Background should be red, as is the background color set on the HCanvas.
Related
Also, if you init H without background color the sketch will have a blueish bg. i.e. H.init(this);
Info
OS: Mac 10.14.5 (also in Win10)
P: 3.5.3
Hype: 2.1.0

Regarding the related issue with init H without a background, this is expected behaviour. HYPE has a default background color of #ECF2F5, so that is not a bug. I'll discuss with Josh if we want to change the default BG color, but really it would only be white or black if we do that.
As for the canvas background not setting, that does appear to be a bug. Internally it looks as if background is setting a fill (which is weird). I'll need to look into it further.

how did u get it working ?


Get stuck on the "Find free port on remote running" msg

Describe the bug
"Find free port on remote running"
To Reproduce
Steps to reproduce the behavior:

Install the plugin
Setup a docker container at port 35222 (ssh)
Try to connect to the remote server
Get stuck on the "Find free port on remote running" msg

Expected behavior
Go forward in the setup process
Screenshots

System Info

Your local OS: Ubuntu
Local Neovim version: v0.8.3
Remote host OS (if applicable): Ubuntu
Remote Neovim version (if applicable): v0.8.3

Thanks for reporting this! Let me have a look.
While I do that, if you could possibly do the following and share results, it would make my task quite easy.

Reproduce the issue using the minimal init file. Start your neovim instance with nvim -u init.lua (init.lua being the above file).
If you are able to reproduce the issue, share the output of the :RemoteLog command. This would give me more insight into what's happening behind the scenes.
The init.lua file creates a .tests directory where it stores all the configuration. Please delete it after you are done with your reproduction. Thanks!

Closing due to no response. Was unable to reproduce this.


Command for moving nodes between namespaces

Added command for moving nodes between namespaces (see #362)

[x] read the CONTRIBUTING guidelines
[x] raised a GitHub issue or discussed it on the projects chat beforehand
[x] added unit tests
[x] added integration tests
[ ] updated documentation if needed
[x] updated CHANGELOG.md

Alright, integration tests were lifesavers. Turns out that a machine's namespace wasn't updating properly.


loop iteration code incorrect

-- This model tests for the presence of a bug first observed on commit
-- fa71679f33aec0cceafb7528d24c03e3b2d315f3. The limits used for unpacking the
-- handle related to the loop variable were incorrect, causing loops like the
-- one in the following model to turn into no-ops. If this bug has been
-- reintroduced, this model will deadlock.

var
  x: 0 .. 10;

startstate begin
  x := 0;
end;

rule x = 0 ==> begin
  for y := 5 to 9 do
    x := x + 1;
  end;
end;

rule x = 5 ==> begin
  x := 0;
end;

invariant x = 0 | x = 5;

This model actually triggers an assertion failure with --debug.


Module 7 Lab Intergration and behavior fixes

Naive date sent by the UWP client to the web API
UWP filter out flights where both destination and origin are the same.
Some cleanup
FlightsManager failed to connect to Azure App Service due to missing TLS protocols.
Instructions were made a bit clearer

This should fix #92 along with some other undocumented issues.


Does not work for computed

I am working on a complex price estimator.  I have multiple sections each with multiple rows that can be dynamically added and removed, each with multiple components.  It is completely data-driven, which is neither here nor there, but is a requirement.  When any of the component inputs change, the row price, section price, and total price all must be updated.  Further complicating the matter, the components are interdependent.  Meaning, one component of a row may effect the price of 1 or more components in the row.  I originally had this working by sending in the row to each of the individual components in order to find the related components that effected the price of another component.  I found your module and thought it would be great for decoupling the components.  However, due to the interdependencies, the price and other values require ko.computed values.  When I use postbox, the row price does not update when a computed value changes.  However, changing a ko.observable value does trigger the computed function to run. The bottom line is that the built in knockout binding works as long as I don't try to bind a postbox subscription to a computed value.  When the computed function accesses a computed function bound to a subscribable, the computed function does not trigger.  It only triggers when an observable value changes.  That is why I think postbox does not work correctly with computed values.  It seems to override, rather than augment the built-in KO functionality.

I was just playing around and found something interesting.  I was actually using pureComputed, and used ko.postbox.subscribe within the function.  When I change it to computed, I get errors because the subscribe function does not have the error yet.  I should also note that I am using v3.4.0 with ko.options.deferUpdates = true; set.

I think I have a workaround for this, but it would require subscribing to multiple topics.  Is that possible?


Adapt uki install to new rootfsbase

If we know immucore is mounting the livecd under rootfsbase like dracut does, we can do size calculation and prepare the spec much earlier in the process
Part of: https://github.com/kairos-io/kairos/issues/2222

If we know immucore is mounting the livecd under rootfsbase like dracut does

I don't see any changes related to this in this PR. Did this happen in another PR (the mounting)?

If we know immucore is mounting the livecd under rootfsbase like dracut does

I don't see any changes related to this in this PR. Did this happen in another PR (the mounting)?

Never mind, found it: https://github.com/kairos-io/immucore/pull/224

If we know immucore is mounting the livecd under rootfsbase like dracut does

I don't see any changes related to this in this PR. Did this happen in another PR (the mounting)?

Never mind, found it: kairos-io/immucore#224

yep, sorry I updated the card at https://github.com/kairos-io/kairos/issues/2222 but didnt link this to it directly

Manually tested and it works as expected.


path name in VS Code terminal was capitalized

I don't know to reproduce this case,
however, it is clear to capitalize specific path in new opened terminal.
As below, real path is not "Source" but "source".
This was caused many case warnings.

/question
Could you specify how it is related to VSCode? If you run same command in cmd do you see same error?
This is output from build tool (most likely Webpack) that warns about possible wrong case in import path. See https://stackoverflow.com/questions/47534267/webpack-there-are-multiple-modules-with-names-that-only-differ-in-casing-but


Brave Ad-Blocker Blocks Default Umami Analytics Script URL – Need Custom Host Support

The default Umami script URL (https://analytics.umami.is/script.js, https://cloud.umami.is/script.js) is blocked by browsers with built-in ad-blockers, such as Brave. This prevents the Umami analytics script from loading and tracking analytics data.
To Reproduce
Steps to reproduce the behavior:

Go to a site using the Pliny Umami component with the default src set to https://cloud.umami.is/script.js.
Open the site in the Brave browser with the integrated ad-blocker enabled.
Inspect the console/network tab to observe that the script fails to load.

Expected Behavior
The Umami script should load successfully, and analytics should be tracked even when using browsers with ad-blockers enabled.
System Info (if dev / build issue):

OS: macOS
Node version: v18.20.4
Npm version: 10.9.0
Yarn version: 3.6.1

Browser Info (Brave:

Device: Desktop
Browser: Brave (with integrated ad-blocker enabled which comes as default)
Version: 1.70.123

Additional Context
To bypass ad-blockers, hosting the Umami script on a custom domain is a solution. However, specifying a custom data-host-url attribute is necessary for directing analytics data to the correct endpoint when the script is hosted on a different server. Additionally, the Umami component would benefit from handling other data-* attributes dynamically to support current and future Umami script configurations.

will submit PR in short


Failed to start container with privileged as true on peer pod

On peer pod, If we need create a container with privileged as true under securityContext, it will failed.
For example, the pod definition:
apiVersion: v1
kind: Pod
metadata:
  name: nginx
  labels:
    app: nginx
spec:
  runtimeClassName: kata
  containers:
    - name: iks-vpc-block-node-driver
      env:
        - name: KUBE_NODE_NAME
          valueFrom:
            fieldRef:
              fieldPath: spec.nodeName
      envFrom:
        - configMapRef:
            name: ibm-vpc-block-csi-configmap
      image: gcr.io/k8s-staging-cloud-provider-ibm/ibm-vpc-block-csi-driver:master
      imagePullPolicy: Always
      securityContext:
          privileged: true
          runAsNonRoot: false
          runAsUser: 0
      ports:
        - containerPort: 9808
          name: healthz
          protocol: TCP
      volumeMounts:
        - mountPath: /etc/storage_ibmc
          name: customer-auth
          readOnly: true
        - mountPath: /csi
          name: plugin-dir
        - mountPath: /dev
          name: device-dir
        - mountPath: /etc/udev
          name: etcudevpath
        - mountPath: /run/udev
          name: runudevpath
        - mountPath: /lib/udev
          name: libudevpath
        - mountPath: /sys
          name: syspath
  serviceAccountName: ibm-vpc-block-node-sa
  volumes:
    - name: customer-auth
      secret:
        secretName: storage-secret-store
    - hostPath:
        path: /var/lib/kubelet/csi-plugins/vpc.block.csi.ibm.io/
        type: DirectoryOrCreate
      name: plugin-dir
    - hostPath:
        path: /dev
        type: Directory
      name: device-dir
    - hostPath:
        path: /etc/udev
        type: Directory
      name: etcudevpath
    - hostPath:
        path: /run/udev
        type: Directory
      name: runudevpath
    - hostPath:
        path: /lib/udev
        type: Directory
      name: libudevpath
    - hostPath:
        path: /sys
        type: Directory
      name: syspath

Found following error in kata runtime logs:
# journalctl -t kata -l --no-pager
Jul 21 07:28:41 cdlleili-new-worker kata[552606]: time="2022-07-21T07:28:41.416924329Z" level=error msg="HotplugAddDevice: not implemented"

That's because remote.HotplugAddDevice is called, and it panics with error "not implemented"
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/virtcontainers/remote.go#L185
func (rh *remoteHypervisor) HotplugAddDevice(ctx context.Context, devInfo interface{}, devType DeviceType) (interface{}, error) {
	panic(notImplemented("HotplugAddDevice"))
}

If privileged is true, kubelet will add all devices on host to the list of linux.devices in the container OCI spec.
kata runtime need handle the devices in the list. It calls Container.attachDevices to attach devices from host to VM:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/virtcontainers/container.go#L1339
It depends on VM hypervisor to attach the device, so finally will call hypervisor.HotplugAddDevice implemented by hypervisor.
For remote hypervisor, I think there is no need and no way to attach the devices from worker VM to peer pod VM.
So remote.HotplugAddDevice is not implemented.
But I think we don't need to panic here. The caller just cares if it returns error. For example:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/pkg/device/drivers/block.go#L115
if err = devReceiver.HotplugAddDevice(ctx, device, config.DeviceBlock); err != nil {
	return err
}

So remote.HotplugAddDevice can just return nil for err, I think
func (rh *remoteHypervisor) HotplugAddDevice(ctx context.Context, devInfo interface{}, devType DeviceType) (interface{}, error) {
	return "HotplugAddDevice is not implemented", nil
}

After above fix, we still failed to create container with privileged as true.
Found following error in kata runtime logs:
Jul 21 08:02:01 cdlleili-new-worker kata[556338]: time="2022-07-21T08:02:01.672463557Z" level=error msg="createContainer failed" error="rpc error: code = Internal desc = Timeout after 3s waiting for uevent ScsiBlockMatcher { search: \"/0:0:0:0/block/\" }" name=containerd-shim-v2 pid=556338 sandbox=ffa9ada8081b0442ac0066c7868e52ee8882f5a908e05f6b0fbac58d92f8c6d7 source=virtcontainers subsystem=kata_agent

The error is returned by kata-agent
When creating container, kata runtime will create grpc.Device according to linux.devices in OCI spec, and append the devices to CreateContainerRequest.Devices list:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/virtcontainers/kata_agent.go#L1244
In kata agent, it will call add_devices to handle CreateContainerRequest.Devices list, to match the real device inside the VM, and update the devices listed in the OCI spec if necessary:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/agent/src/rpc.rs#L214
// Some devices need some extra processing (the ones invoked with
// --device for instance), and that's what this call is doing. It
// updates the devices listed in the OCI spec, so that they actually
// match real devices inside the VM. This step is necessary since we
// cannot predict everything from the caller.
add_devices(&req.devices.to_vec(), &mut oci, &self.sandbox).await?;

The problem is the block device type. When appending the device to CreateContainerRequest.Devices list, kata runtime will set block device type and other info according to HypervisorConfig.BlockDeviceDriver:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/virtcontainers/kata_agent.go#L1039
switch c.sandbox.config.HypervisorConfig.BlockDeviceDriver {
	case config.VirtioMmio:
		kataDevice.Type = kataMmioBlkDevType
		kataDevice.Id = d.VirtPath
		kataDevice.VmPath = d.VirtPath
	case config.VirtioBlockCCW:
		kataDevice.Type = kataBlkCCWDevType
		kataDevice.Id = d.DevNo
	case config.VirtioBlock:
		kataDevice.Type = kataBlkDevType
		kataDevice.Id = d.PCIPath.String()
		kataDevice.VmPath = d.VirtPath
	case config.VirtioSCSI:
		kataDevice.Type = kataSCSIDevType
		kataDevice.Id = d.SCSIAddr
	case config.Nvdimm:
		kataDevice.Type = kataNvdimmDevType
		kataDevice.VmPath = fmt.Sprintf("/dev/pmem%s", d.NvdimmID)
	}

For remote hypervisor, it will use the default value defaultBlockDriver:
https://github.com/yoheiueda/kata-containers/blob/CCv0-peerpod/src/runtime/virtcontainers/hypervisor.go#L63
defaultBlockDriver = config.VirtioSCSI

So the block device type is virtio-scsi. kata agent will try to find the device on peer pod VM based on the type and device info, but failed.
I tried other block device type, such as virtio-blk, still failed on peer pod VM:
 "container create failed" container=b55dcceb7473206e5d2a243be3c519a044e378089c60cddc3d7455cd4a763aa9 error="rpc error: code = Internal desc = invalid ID and VM path for device type: \"blk\" container_path: \"/dev/loop0\""

Maybe we don't need kata agent to update linux.devices list in OCI spec on peer pod VM. There is no valid value for remote.BlockDeviceDriver, we can set it as "dummy", and don't append the device without type to the list of CreateContainerRequest.Devices.

@yoheiueda , I created a PR https://github.com/yoheiueda/kata-containers/pull/2 for this issue.
Can you help to review it? And how could I merge the PR to CCv0-peerpod branch?

@genjuro214 thank you very much for the investigation.
I'll manually include your changes when I update the branch.
BTW, I'd like you to investigate possibility of another solution.
We can manipulate CreateContainer requests in cloud-api-adaptor like this one https://github.com/confidential-containers/cloud-api-adaptor/commit/506b6fa31cd04f5fd17f2708b56a100f6f9ca3c6.
https://github.com/confidential-containers/cloud-api-adaptor/blob/631daaeab09d6b956babedde335b2197cca7ae26/pkg/adaptor/proxy/service.go#L386-L389
Can we simplify necessary changes in the kata shim, if we can manipulate the device list in a CreateContainer request?

Great investigation @genjuro214 and will surely help folks when faced with similar situations debugging peer-pods issues.
@yoheiueda the fix proposed by @genjuro214 in the kata shim is primarily isolated to the remote hypervisor type so I think it should be ok. Imho, we can be selective in using the agent proxy for api call manipulation.
I'm not sure about the kata-agent changes whether it's specific to remote hypervisor or generic - https://github.com/yoheiueda/kata-containers/pull/2/files#diff-6fb7f551035031ed29ad100c61f1a7e92b0fa15d2f67e4c7ea7ebf3b715583f4

OK. I have updated the branch to include this change along with the other changes based on review comments.
@genjuro214 could you please check it?

@genjuro214 is this issue still relevant or can be closed?
If it's still relevant to what release do you think we should map it to (mid-November, end-December, mid-February etc...)?

Yes, the PR has been merged, I'm OK to close this issue now.


An error during setup

I have installed the packages listed in the Readme file and the version exactly equals to the version listed in the Readme file, but when I run the command "python setup.py develop", the error below occured:

I don't know why this error occured. I have no idea to solve this problem.
I guess, maybe some packages that I installed mismatch my cuda version? My cuda version is 10.1 .  Can you tell me how can I fix this problem and setup successfully?

Oh，thanks for your reply in your busy schedule. After 2 days of the occurance of the issue I find that the problem is exactly about  the gcc version that you mentioned. When I changed the gcc version to 9 (I used 5 before), the problem solved.
Thank you again! I'll close this issue.


Remove sql edge container option from sql projects publish

This removes the SQL Edge container as an option from sql project publish since it is being retired: https://techcommunity.microsoft.com/t5/sql-server-blog/azure-sql-edge-update-september-2023/ba-p/3930827
Before:
ADS:

vscode - this step is removed in this PR:

After:
ADS publish dialog - removed base image dropdown:

Quickpick used in vscode extension - removed step where the base image is chosen since there's only one option now:

Just like this option has been in VSCode, can something similar be done for ADS as well, so that the customer doesn't get confused on why is the dropdown presented when there is no other option to chose from. Thoughts?

Just like this option has been in VSCode, can something similar be done for ADS as well, so that the customer doesn't get confused on why is the dropdown presented when there is no other option to chose from. Thoughts?

I wasn't sure at first if it leaving the dropdown with one option would help give context on the container base image in the dialog, but for the quickpick it seemed silly to have the extra step. It makes sense to have it removed in both for consistency though, so removed the dropdown in the latest commit and updated the description.