Update use-multiple-node-pools.md

when running the original command:
$ az aks nodepool add -g tylloyd-devbox -n v6 --os-sku mariner
az aks nodepool add: 'mariner' is not a valid value for '--os-sku'. Allowed values: Ubuntu, CBLMariner, Windows2019, Windows2022.

Learn Build status updates of commit cfa4fe4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/aks/use-multiple-node-pools.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

@Goedtel,
Can you review the proposed changes?
IMPORTANT: 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

@tyler-lloyd - I just tested and verified that the value 'mariner' is valid for the os-sku argument. I also tested 'CBLmariner' and verified it is also valid, however there is a difference between the node image version used. I'm checking with the team to reconcile this before approving this change or us having to make other changes to the article and other related articles in order to deliver the right information. Thanks for your patience.

@MGoedtel can you share the command you used with just 'mariner'?

@tyler-lloyd - az aks nodepool add --resource-group resourceGroupName --cluster-name aksClusterName --os-sku mariner
I ran this directly in Cloud Shell as test using the sample code, as you can tell. Can you provide more details on your attempt to run the command? Was it local or in Cloud Shell? If local, what version of Azure CLI is installed?

@tyler-lloyd - Could you respond to the questions above? Or should this PR be closed?

Hi @tyler-lloyd, are you still experiencing this issue? If not, I will close the PR. Thanks!


Autoleave option

Please tick all applicable boxes.

[x] I am using Python 3.5 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[ ] The latest master version (1.9.7)
[x] The latest review version

What type of issue are you creating?

[ ] Bug
[x] Feature request
[ ] Question

Just an option for the bot to automatically leave the VC when it's empty, possibly after a certain period of time.

Similar to #647. Will move discussion there.


As part of the swagger AuthN/AuthZ changes:

Add local user model and use bcrypt to serialize passwords
Add roles model
Add 401 and 403 errors to the common errors

:+1:

*** BUILD #321 ***
Test Name: test_node_workflows_del_active
Error Details: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Content-Length': '62', 'X-Powered-By': 'Express', 'Connection': 'keep-alive', 'ETag': 'W/"3e-uVr3eF51jj1og38xQuRGng"', 'Date': 'Fri, 11 Mar 2016 14:34:39 GMT', 'Access-Control-Allow-Origin': '*', 'Content-Type': 'application/json; charset=utf-8'})
HTTP response body: {"message":"NotFoundError: No active workflow found for node"}

Stack Trace: Traceback (most recent call last):
  File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  File "/usr/local/lib/python2.7/dist-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/usr/local/lib/python2.7/dist-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/usr/local/lib/python2.7/dist-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/var/lib/jenkins/jobs/on-core/workspace/RackHD/test/tests/api/v2_0/nodes_tests.py", line 308, in test_node_workflows_del_active
    Api().nodes_del_active_workflow_by_id(identifier=id)
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/apis/api_api.py", line 2673, in nodes_del_active_workflow_by_id
    callback=params.get('callback'))
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/api_client.py", line 322, in call_api
    response_type, auth_settings, callback)
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/api_client.py", line 149, in __call_api
    post_params=post_params, body=body)
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/api_client.py", line 374, in request
    headers=headers)
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/rest.py", line 201, in DELETE
    query_params=query_params)
  File "/usr/local/lib/python2.7/dist-packages/on_http_api2_0/rest.py", line 177, in request
    raise ApiException(http_resp=r)
ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Content-Length': '62', 'X-Powered-By': 'Express', 'Connection': 'keep-alive', 'ETag': 'W/"3e-uVr3eF51jj1og38xQuRGng"', 'Date': 'Fri, 11 Mar 2016 14:34:39 GMT', 'Access-Control-Allow-Origin': '*', 'Content-Type': 'application/json; charset=utf-8'})
HTTP response body: {"message":"NotFoundError: No active workflow found for node"}

test this please

:+1:

+1


Adding ITET Bramante Genga in Pesaro, Italy

@fedegio67 First stage of verification passed, official domain is itbramantegenga.edu.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.itbramantegenga.edu.it/

@philipto that's right. itbramantegenga.edu.it is our official school domain. Let me know if I need to provide further information. Thanks a lot.

@fedegio67 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Image Streamer Refactor

Description

Refactors the 1.3.0 code to support the addition of secondary APIs
Introduces the new ImageStreamer::API300 API
Adds the resource image_streamer_plan_script (OneviewSDK::ImageStreamer::API300::PlanScript)
Specs supporting both ImageStreamer and OneView resources

⚠  ~SimpleCov coverage not working for some underworldly reason.~ (If any ChefSpec is executed first than the pure RSpec ones, SimpleCov simply halts and outputs the coverage)
⚠  ~Berkshelf cannot find the compat_resource cookbook in Travis build~ (For some reason it won't work if we call rake using bundle)
Check List

[ ] New functionality includes testing.

[ ] All tests pass ($ rake test).
[ ] New resources and/or actions have are included in the matchers.rb and matchers_spec.

[ ] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in examples (please include helpful comments).

[ ] Changes documented in the CHANGELOG.

@jsmartt Any other suggestions here, or it is ready to merge?
(Notice it is going to be merged on master_i3s and not master)

Sorry @tmiotto , we had a holiday yesterday, so I'm just now getting around to giving it a look. Mainly trying to figure out how the sdk_secondary_api parts are used. Should be done in an hour or so

@jsmartt I improved it, could you take a look?

Also, we need to update the README with all the new I3S stuff. Like the additional ENV var(s) here, client types, etc.

@fgbulsoni @jsmartt Added the last bits. See if I missed something or if you have any further suggestions

^^ 3 things, not 2


BUG: ETL doesnt work for multi page surveys

Because of a different JSON structure , ETL fails for multi page surveys

covered in 1150


add suggested course/pre-req dependencies

Would help in finding the optimal order for taking courses. Idea of where dependencies exist may have a good starting point here: (https://plus.google.com/+RonnyPena1/posts/eUjPAUZQZDa)

Project to collect some data to give people a good understanding of this here: https://goo.gl/forms/WRvti28ZoTgi0Frf2


[Vue warn]: Hydration text content mismatch

Environment
Template framework: dev-master
Node: v16.13.1
Describe the bug

If you use await useFetch directly in a Vue component, you will get the warning above when you press F5 to refresh the page after switching to another language (non-default language).
Example (rewrite your src/pages/index.vue):
<script setup>
const { data } = await useFetch('https://httpstat.us/200')
console.log(data)
</script>

<template>
  <Starter />
</template>

Additional context
No response
Logs
No response

Maybe this problem is not related to this template project, but it is an existing problem.

hi 3.6.0 version now send please re test

Great to see your changes.
But I ran into a problem:

Do you know what is the cause of this bug?

same problem: https://github.com/nuxt-modules/tailwindcss/issues/470


[SPARK-31079][SQL] Logging QueryExecutionMetering in RuleExecutor logger

What changes were proposed in this pull request?
RuleExecutor already support metering for analyzer/optimizer rules. By providing such information in PlanChangeLogger, user can get more information when debugging rule changes .
This PR enhanced PlanChangeLogger to display RuleExecutor metrics. This can be easily done by calling the existing API resetMetrics and dumpTimeSpent, but there might be conflicts if user is also collecting total metrics of a sql job. Thus I introduced QueryExecutionMetrics, as the snapshot of QueryExecutionMetering, to better support this feature.
Information added to PlanChangeLogger
=== Metrics of Executed Rules ===
Total number of runs: 554
Total time: 0.107756568 seconds
Total number of effective runs: 11
Total time of effective runs: 0.047615486 seconds

Why are the changes needed?
Provide better plan change debugging user experience
Does this PR introduce any user-facing change?
Only add more debugging info of planChangeLog, default log level is TRACE.
How was this patch tested?
Update existing tests to verify the new logs

Test build #119506 has started for PR 27846 at commit d0e421f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24235/
Test PASSed.

Test build #119506 has finished for PR 27846 at commit d0e421f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class QueryExecutionMetrics(

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119506/
Test FAILed.

Test build #119509 has started for PR 27846 at commit ce0bedb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24238/
Test PASSed.

Test build #119509 has finished for PR 27846 at commit ce0bedb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119509/
Test FAILed.

retest this please

Test build #119510 has started for PR 27846 at commit ce0bedb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24239/
Test PASSed.

Test build #119510 has finished for PR 27846 at commit ce0bedb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119510/
Test PASSed.

Are these metrics benefit for general users? If these metrics are for developers, how about just showing these metrics via spark.sql.optimizer.planChangeLog.level?

Are these metrics benefit for general users? If these metrics are for developers, how about just showing these metrics via a logger with spark.sql.optimizer.planChangeLog.level?

Yea, logging message should be enough. I'll make the change, thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24268/
Test PASSed.

Test build #119537 has started for PR 27846 at commit 42e40d2.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24269/
Test PASSed.

Test build #119537 has finished for PR 27846 at commit 42e40d2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119537/
Test PASSed.

The improvement looks fine to me.  WDYT? @maryannxue @cloud-fan

Test build #119561 has started for PR 27846 at commit abc864e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24292/
Test PASSed.

retest this please

Test build #119564 has started for PR 27846 at commit abc864e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24295/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24298/
Test PASSed.

Test build #119561 has finished for PR 27846 at commit abc864e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119561/
Test PASSed.

Test build #119564 has finished for PR 27846 at commit abc864e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119564/
Test PASSed.

Test build #119567 has finished for PR 27846 at commit 36bf2e1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119567/
Test PASSed.

thanks, merging to master/3.0!

@cloud-fan @maropu Thanks so much!

late LGTM


How to get data from EdgeX Foundry Performance Report?

Hi team:
In EdgeX Foundry Performance Report

Raw performance data underlying this report is provided here. ( https://docs.google.com/spreadsheets/d/131AVzDQk0Bq8Y3_NePScoP2PPVzwxYBe2fuztG3jetA/edit )

I wonder how to run this Performance test in my project, thanks.

The following notes describe how the performance data described in this report was captured. The services were run with their default configurations and the Virtual Device was allowed to run and produce sensor data in its default manner (generating data every 30 seconds for a variety of mimicked sensors).
The CPU and memory usage data was captured from Docker Engine via docker stats after successful start-up of EdgeX using the release Docker Compose files. Data export testing was run 15 times – that is generating 15 event/readings through Core Data to the Application Services layer via the Virtual Device Service.

Can you give me an example to measure the time to export the data to an endpoint, thanks.

Please see the document here
https://github.com/edgexfoundry/edgex-taf/blob/master/docs/edgex-taf.md
https://github.com/edgexfoundry/edgex-taf/blob/master/docs/run-performance-metrics-collection-on-local.md
You had better run with the released version v1.3.1 = hanoi

Thanks for sharing related document! I still have this Question:

The following notes describe how the performance data described in this report was captured. The services were run with their default configurations and the Virtual Device was allowed to run and produce sensor data in its default manner (generating data every 30 seconds for a variety of mimicked sensors).
The CPU and memory usage data was captured from Docker Engine via docker stats after successful start-up of EdgeX using the release Docker Compose files. Data export testing was run 15 times – that is generating 15 event/readings through Core Data to the Application Services layer via the Virtual Device Service.

Can you give me an example to measure the time to export the data to an endpoint, thanks.

what kind of the example?

please follow the document I provided and run the script, and you can get all the metrics.  the test script is the tool.

Oh, I am sorry, I miss the Statistics from the  detail execution reports.
Thanks for your patience.

no problem


git-lfs missing from container

Hi!
Thanks for this very cool repository, the preprint is very cool, too!
I am a total beginner to using relevant machine learning libraries, but I think the git-lfs is missing from the docker container of hyena-dna. Admittedly, I did a bit of weird stuff with it because I can't execute docker on the HPC. I converted it to Singularity. Nevertheless, I'd expect to be able to call git-lfs from there, too, and it seems to be missing.
Here's what I did:
# build image from existing docker container
singularity build hyena-dna.sif docker://hyenadna/hyena-dna-public:latest

# can not use the hyena-dna inside the container because it tries to write into the same folder, therefore using a local clone, container only holds the dependencies
git clone https://github.com/HazyResearch/hyena-dna.git
cd hyena-dna

# SINGULARITYENV_CUDA_VISIBLE_DEVICES=1 says "use GPU_1" (it will otherwise use GPU_0 by default)
SINGULARITYENV_CUDA_VISIBLE_DEVICES=1 singularity exec --nv ~/images/hyena-dna.sif python -m train wandb=null experiment=hg38/genomic_benchmark_scratch # works like charm!

SINGULARITYENV_CUDA_VISIBLE_DEVICES=1 singularity exec --nv ~/images/hyena-dna.sif python -m huggingface # fails

Error:
Using device: cuda
git: 'lfs' is not a git command. See 'git --help'.

The most similar command is
	log
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/runpy.py", line 185, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/opt/conda/lib/python3.8/runpy.py", line 111, in _get_module_details
    __import__(pkg_name)
  File "/home/hoffk83/images/git/hyena-dna/huggingface.py", line 251, in <module>
    inference_single()
  File "/home/hoffk83/images/git/hyena-dna/huggingface.py", line 209, in inference_single
    model = HyenaDNAPreTrainedModel.from_pretrained(
  File "/home/hoffk83/images/git/hyena-dna/huggingface.py", line 106, in from_pretrained
    config = json.load(open(os.path.join(pretrained_model_name_or_path, 'config.json')))
FileNotFoundError: [Errno 2] No such file or directory: './checkpoints/hyenadna-small-32k-seqlen/config.json'
(base) hoffk83@vision-05:~/images/git/hyena-dna$ ls ./checkpoints/hyenadna-small-32k-seqlen/config.json
ls: cannot access './checkpoints/hyenadna-small-32k-seqlen/config.json': No such file or directory
(base) hoffk83@vision-05:~/images/git/hyena-dna$ ls ./checkpoints/hyenadna-small-32k-seqlen/config.json
ls: cannot access './checkpoints/hyenadna-small-32k-seqlen/config.json': No such file or directory

I think it could be fixed by adding git-lfs to the requirements.txt, and then rebuilding the container.

Hi @KatharinaHoff, thanks so much!  I forgot this dependency, which I believe is just for loading the pretrained weights from Huggingface.  Good catch :)  I've made the change and uploaded a new Docker image (reflected in the readme).  You can now pull with (I removed the 'public' name):
docker pull hyenadna/hyena-dna:latest

Enjoy!

Dear @exnx , thank you so much for trying to fix it. Sadly, the pip install of git-lfs did not fix the problem. Sorry, my bad, I had not tested my first suggestion. I have no figured out how to fix it. Possibly not the most elegant way... but if you append the following to your Dockerfile, then both the docker and the singularity built contain git lfs and models can be loaded from huggingface:
RUN wget https://github.com/git-lfs/git-lfs/releases/download/v3.4.0/git-lfs-linux-amd64-v3.4.0.tar.gz && \
    tar -xvf git-lfs-linux-amd64-v3.4.0.tar.gz && \
    cd git-lfs-3.4.0 && \
    ./install.sh && \
    rm git-lfs-linux-amd64-v3.4.0.tar.gz

This solution, I have tested both with Docker and Singularity. git lfs works.
Maybe you also want to add the Singularity instructions (adapt from my initial post here) to the Readme.md? Just an idea to save other people some time. I tested it with singularity-ce version 3.11.3 , all works well.

Thanks for the update!  I haven't been able to test it out myself, but I'll report back when I do.

I ended making a second Docker image with the Nucleotide Transformer datasets, and weights to reproduce.  This image includes the correct git-lfs dependency.  You can find it here
# pull image
docker pull hyenadna/hyena-dna-nt6:latest

# run container
docker run --gpus all -it -p80:3000 hyenadna/hyena-dna-nt6 /bin/bash

To build the image, I used tips from this thread, which basically just means adding:
RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | sudo bash
RUN sudo apt-get install git-lfs

Eventually I'll add this to the main Dockerfile in the repo, but for now there are 2 Docker images.

Hi @KatharinaHoff and @exnx , thanks a lot for posting directions on how to generate and use a Singularity image of HyenaDNA! I tried as Katharina suggested:
singularity build hyena-dna.sif docker://hyenadna/hyena-dna-public:latest
git clone https://github.com/HazyResearch/hyena-dna.git
cd hyena-dna
SINGULARITYENV_CUDA_VISIBLE_DEVICES=1 singularity exec --nv ~/images/hyena-dna.sif python -m train wandb=null experiment=hg38/genomic_benchmark_scratch

But getting /opt/conda/bin/python: No module named train
Anything missing on my side?
Thanks!

The steps above by Katharina didn't work for me.  Instead I used a different set of commands, which you can find in this image instead.
hyenadna/hyena-dna-nt6:latest

Great, thank you!
As first thing, I tried the nt7 version also available - but the converted
sif image was giving errors. Will try converting nt6 and let you know.
On Tue, Nov 14, 2023, 9:03 PM Eric Nguyen @.***> wrote:

The steps above by Katharina didn't work for me. Instead I used a
different set of commands, which you can find in this image instead.
hyenadna/hyena-dna-nt6:latest
—
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/hyena-dna/issues/14#issuecomment-1811813642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7IT6G3WJBOD6DH6K6FYKTYEREJFAVCNFSM6AAAAAA4GHLP6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJRHAYTGNRUGI
.
You are receiving this because you commented.Message ID:
@.***>

The nt6 image should be using commands from here.  I forgot if nt7 did too or not, might've been testing other things.
But specifically, the commands you want in the Dockerfile are:
RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | bash
RUN apt-get install -y git-lfs

I updated the Dockerfile in this repo, it now includes this command if you build your own image.

Thanks! I re-run with hyena-dna-nt6, getting:
singularity build hyena-dna_nt6.sif docker://hyenadna/hyena-dna-nt6:latest
cd hyena-dna
singularity exec --nv hyena-dna_nt6.sif python -m train wandb=null experiment=hg38/genomic_benchmark_scratch
13:4: not a valid test operator: (
13:4: not a valid test operator: 510.47.03
/usr/bin/python: No module named train

This strange 'not a valid test operator' is the same I was getting with the sif image of hyena-dna-nt7 actually.
Wish I could use Docker - but I am stuck with Singularity on the HPC A100 nodes I have available.

ChatGPT?  ie, check for how to make the docker cmd provided into an equivalent singularity cmd.  Sorry, we don't support singularity on our end, we just don't use it.

eg
apptainer pull docker://hyenadna/hyena-dna-nt7:latest
apptainer exec --nv docker://hyenadna/hyena-dna-nt7:latest /bin/bash

No worries, thank you for all the help. I think I found the culprit - my cloned hyenaDNA folder was on a mounted drive (/mnt/ etc.) that for some reason wasn't accessible by the container image. Now moved everything on my home folder and it seems to work.
If I have further questions I will reach out on Discord. Thanks again!


openssh config

use openssh config to load password from a local file (little bit safer than having code with pwd in it)

solved by using a docker container with published user/password combination xenon/javagat


Create backend endpoints for Roles

Goal

Create CRUD endpoints for the Role model, accessible based on permissions and user
Requirements

4 CRUD endpoints. All of them are only accessible for Developers/Admins:

GET /roles - get all roles. Features pagination
GET /roles/:roleId - get a specific role
PUT /roles/:roleId - modify a specific role. The body contains the new data
DELETE /roles/:roleId - delete a specific user

GET ´/roles/me´ - get the roles of the currently authenticated user
All endpoints have corresponding e2e tests and swagger documentation

Extra notes

The model and repository already exist and are used by the Authentication package

Closing due to not being needed


[Fix] Readme for UI lib

Motivación y Contexto

Se agrega el repo de maven para descargar la Lib de UI de Meli en el Readme para los integradores externos.

Codecov Report

Merging #1226 into hotfix/4.0.2 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             hotfix/4.0.2   #1226   +/-   ##
==============================================
  Coverage            2.58%   2.58%
  Complexity            125     125
==============================================
  Files                 365     365
  Lines               14948   14948
  Branches             1310    1310
==============================================
  Hits                  387     387
  Misses              14536   14536
  Partials               25      25

Impacted Files
Coverage Δ
Complexity Δ

...ndroid/px/configuration/AdvancedConfiguration.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ternal/datasource/MercadoPagoPaymentProcessor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Obtaining the lwjgl demo/samples jar?

I download the early release from lwjgl's website and the getting started pack including DriftFX as I'm interested in getting the HelloDriftFX example to work.
It references "org.lwjgl.demo.opengl.*" but don't see it as part of the downloaded lwjgl jars. It makes reference to GLXGears, and then learn that this makes use of joml, etc
Is there lwjgl samples jar on the website or github that can be downloaded or does all of this have to be built from source? I've already tried to build joml from source using the shipped pom and got a build failure and so a pre-built samples jar would be helpful.

Hey @gmseed!
There is no samples jar available for download. These samples are only available in the LWJGL repository, which you can clone, build and experiment on locally. Assuming you have the required software installed/configured (C/C++ compiler, JDK 8+, Apache Ant), you can build LWJGL with:
ant compile-templates tests

Note that compile-templates is separate because it takes very long. If it succeeds but you get a failure in subsequent tasks, you can skip compile-templates in your next attempts.

Then you can try one of the demos with:
ant demo -Dclass=<main class>

For example, you can run the DriftFX demo with ant demo -Dclass=org.lwjgl.demo.driftfx.HelloDriftFX. When trying new things, I usually also pass a couple of extra JVM parameters like ant demo -Dclass=org.lwjgl.demo.driftfx.HelloDriftFX -Djvmargs="-Dorg.lwjgl.util.Debug=true -Dorg.lwjgl.util.DebugLoader=true".
Hope this helps.


Docker support with HTTPS for local development

How to use it without https in local development?

Why without https?

I don't have a need to use https when doing local dev

Can I just comment the following line?
DockerHelpers.ApplyDockerConfiguration(configuration)


2015 Support?

Which version(s) specifically does your library support for MS Dynamics CRM?
Would be useful to add them to the readme
Thanks.
Jack

Should work with 2011, 2013, and 2015.


Updated README.md for readability

Added 5 H2 sub sections:

Setting up your environment and forking relevant repo
Exec Doc Prerequisites
Exec Doc Writing Requirements
Exec Doc Testing
Publishing an Exec Doc

The Exec Doc Prerequisites section has the 7 Pre-reqs (markdown file, LF, metadata, etc.)
The Exec Doc Writing Requirements section details everting else: Declare environment vars, Declare unique env vars, Add result blocks, Redact PII.
As a part of the refactoring, i moved some Notes to the body of the relevant section.

closing PR. Naman made updates manually due to conflicts.


context.fail! is confusing, behaviour has changed, docs haven't

In Version 3, context.fail! always throws an exception:
    # Raises Interactor::Failure initialized with the Interactor::Context.
    def fail!(context = {})
      modifiable.update(context)
      @failure = true
      raise Failure, self
    end

In version 2, it did not.  The README is still showing the version 2 behaviour, where the caller queries success? after.
What is the current best practice to indicate and respond to failure?  I liked the old idiom of checking context.success? from the controller, after the interactor has returned.  Yet now it seems we must rescue instead - a change that disrupts the structure of the controller methods.
Could we provide a non-throwing 'fail' method in addition to the throwing version?  (I'm currently using a Rails initializer to monkey-patch one in).

Are you using the class method call or the instance method?

Neither.  I was testing internal business-logic methods of my interactor directly - before writing a call that would combine them in sequence.
Reading the source, I discover that call swallows exceptions from fail!, while call! does not.
I'll submit a modification to the README that clarifies this.

Merged #91.


abap2xlsx and PIFs

Hi all,
I installed ABAP2XLSX on a SAP system with development package check “R3ENTERPRISE” .
Executing a package check on ABAP2XLSX produces many errors for the sub-packages, as the sub packages cannot access the objects of the super package.
Is it possible to create an PIF for ABAP2XLSX?
Besides this, there are some other "forbidden" usages like type char7 in ZCL_EXCEL_GRAPH->CREATE_SERIE.
I could solve many package errors by deleting the objects in the DEMO package, but this has to be done after ech . Is it also a possibility to split the demo objects in a second repository?

I created one and tried to a give use access to the DEMO package. But this did not work ( raised an error ). So a PINF cannot solve the visibility in sub-packages.  I assume a reorg of packages is needed. Like a CORE package with DEMOS and NOT_CLOUD package on the same level.

I don't really know the Package Check feature. I'm not against it, I think it's a good idea to organize objects, as long as people who don't use package check can continue pulling abap2xlsx. I am concerned by the changes to package assignment, not sure if that would imply error messages during pull by abapGit, because the "moved" objects are deleted and created again. If you can fork the project and propose a PR, it will be easier for me/us to evaluate all the impacts. I hope that the Package Check is fully supported by the ABAP trial system 7.52 I use (I guess yes). Thank you for your help. 👍
@larshp @gregorwolf @AndreaBorgia-Abo Could you give your opinion please about introducing package check in abap2xlsx? Thank you! 👍

I have (yet) no opinion about it because my knowledge of this feature is similarly lacking.


resolving mobile issues discovered in #12

resolves #12 for the latest version of the component

Codecov Report

Merging #14 into master will decrease coverage by 0.35%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage      80%   79.64%   -0.36%
==========================================
  Files           2        2
  Lines         105      113       +8
==========================================
+ Hits           84       90       +6
- Misses         21       23       +2

Impacted Files
Coverage Δ

src/ReactAddToCalendar.js
68.18% <0%> (ø)
:arrow_up:

src/helpers/index.js
95.74% <75%> (-4.26%)
:arrow_down:

Continue to review full report at Codecov.

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


src/ConnectionSTREAM_UNITE/ConnectionSTREAM_UNITE.cpp:8:10: fatal error: 'ErrorReporting.h' file

Building a 20.10.0 release, got this problem on StreamUnite part. Did not find that header neither in the release nor master sources:
/usr/ports/comms/limesuite/work/LimeSuite-20.10.0/src/ConnectionSTREAM_UNITE/ConnectionSTREAM_UNITE.cpp:8:10: fatal error: 'ErrorReporting.h' file
 not found
#include "ErrorReporting.h"

Any hints welcome :-)

Fix proposition: https://github.com/myriadrf/LimeSuite/pull/337

Fix merged

Fix merged


fix bug and improve spaceship_docker

grep -q "Cannot connect" is fragile. When it talks to the swarm cluster, it can get other error messages because of networking errors

docker info is unused, so docker version is enough.

I second this.
As the docker group on Linux is equivalent to root (source), I always get an error because docker on my system requires root access.
Error message:
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.33/version: dial unix /var/run/docker.sock: connect: permission denied


Update delete.blade.php

Hide delete button for already deleted (soft).

Hmm... I agree, it would be nice to show a "Restore" button if the entry is soft deleted.
I've thought about adding support for this in Backpack before but I think in 99% of use cases you don't even want your admin to know things are soft deleted. Usually I use soft deletes because I don't trust the admin, and want a way back in case he deletes something important. I've dumped my thoughts about it here - https://github.com/Laravel-Backpack/CRUD/issues/1827 let me know what you think about it please.
I think it ok to have this, as it doesn't hurt anyone, I'll merge it now into the 3.6 branch (note I said 3.6 - we've silently upgraded to provide Laravel 5.8 support, I'll send a newsletter in the next few days too).
Thanks, cheers!

Ouch - on second thought, while merging I saw this change makes the list view error for all non-soft-delete entities :-) Which makes a lot of sense, the trashed() method only exists if it has the SoftDeletes trait :-)
In this case, rather than having a method_exists() call and negative conditional on each and every delete button, when this is not actually used 99% of the time, I think we'd rather have a soft_delete button, which copies delete and adds this functionality. I think this will keep things snappy and clear for most people, which don't need this soft deletes hiding, but also provide the functionality. Don't know whether to include this in the core, though... maybe better with a different operation like I mentioned here #1827
Sorry, cheers!


Need a way to set new custom fields on Test Records

In version 21R1 Polarion introduced custom fields on Test Records.  This is a very useful feature and critical to us implementing TFA in Polarion.
Here is the new help on it:
(https://polarion.devel.engineering.redhat.com/polarion/help/index.jsp?tab=links&contextId=com.polarion.xray.web.test_record_custom_fields)
Here is API for Test Record...not sure if it is supported yet..I have posed the question to Siemens

here's some info from Siemens:
There are methods in our API - you can add a property to a test record, get it, and delete it:
ITestRecord.getProperty()
ITestRecord.setProperty()
ITestRecord.removeProperty()
Here's more information about the ITestRecord method: https://almdemo.polarion.com/polarion/sdk/doc/javadoc/com/polarion/alm/tracker/model/ITestRecord.html.

@brian12559

Currently TestRecord has access to some of the elements as stated here (search for complexType name="TestRecord") and I don't see any custom field functionality which is directly possible on TestRecord
If possible can you pls share the steps how it is done via Polarion UI?

@simzacks my understanding is even if we run methods/functions on any object the result will end up in _cls_suds_map and I was not able to figure out where to store the properties of custom fields on TestRecord.
Am I missing anything here?

custom fields are added to a Projects Test Run here:
https://polarion.devel.engineering.redhat.com/polarion/#/project/Polarion/administration/testing/test_record_custom_fields
and they show up here:
https://polarion.devel.engineering.redhat.com/polarion/#/project/Polarion/testrun?id=My Test Run1&tab=records&result=passed
you should be able to use the ID stester2 with password polarion to login and see these.
Let me know if that helps

ack, it's festive week here, will re-look on Monday.
as of now any work_item (testcase, testsuite, requirement etc) object created using Pylero has ability to get and set custom fields need to see how that can be expanded to non work_items

we also set and get custom fields on Test Runs, which are not work items.

That's great, probably I can refer that. Thanks.

The question would be if the wsdl supports them or not.
On Wed, Nov 3, 2021, 14:52 Leela Gangavarapu @.***>
wrote:

That's great, probably I can refer that. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RedHatQE/pylero/issues/41#issuecomment-959002280, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADF6GME5IUSGRJ35RRHDDJ3UKEWAJANCNFSM5HHDZRAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

yes, I don't see wsdl for getting a test record however i could see searchTestRecords and updateTestRecord in here
having said that, the general use case seems to be populating test records as part of test run and so possibility of updating a single test record needs to be looked into.

In the rest of Polarion custom fields are gotten and set through special function calls. If there is no specific function for test record custom fields, I would suspect that they could not be updated through the wsdl

According to Siemens this can be done...
here's some info from Siemens:
There are methods in our API - you can add a property to a test record, get it, and delete it:
ITestRecord.getProperty()
ITestRecord.setProperty()
ITestRecord.removeProperty()
Here's more information about the ITestRecord method: https://almdemo.polarion.com/polarion/sdk/doc/javadoc/com/polarion/alm/tracker/model/ITestRecord.html.

Afaik, it isn't always given that WSDL has one-to-one mapping to internal Java implementations.
We generally query WSDL api to fill python dicts (_cls_suds_map), so not sure whether this featute can be implemented or not.
However, I'll revisit one more time before writing this off.

This is a must have for a few teams that use Polarion.  We plan on implementing Test Failure Analysis using Polarion Test results and we will need the ability to edit these fields.  Please register with Siemens Customer Support and begin a dialog with them around this if you feel that we can not do this on our own. Siemens knows that adding a custom field without providing us API access to it is not very helpful.
https://support.sw.siemens.com/en-US
thanks,
Brian

Being as clear as possible:

Initially we were storing TestRun custom_fields xml file by directly reaching out to polarion backend repo/svnwebclient

Upon request we (@waynesun09) implemented https://github.com/RedHatQE/pylero/pull/30 to use testmanagement.getDefinedTestRunCustomFieldTypes WSDL service for storing TestRun fields

Current request is ability to perform get/set ops on custom fields in TestRecord, for that functionality we need to first get and store the list of custom fields in TestRecord

Here's the problem, there's no WSDL service for getting those custom fields something like testmanagement.getDefinedTestRecordCustomFieldTypes

Woraround would be directly accessing the backend repo/svnwebclient for caching the custom fields in TestRecord, for ex: downloading the file from here

However I don't suggest going this route for 2 reasons, one, it's not possible in polarion.engineering as backend is currently locked down and two, extra parsing of xml file is required

So, unless we reach out to Siemens to provide a WSDL service for getting custom fields defined in Test Record and they implement it, IMHO we can't implement this feature request and pls excuse for the same

pinging @simzacks, @waynesun09 to weigh in their thoughts and correct me if my explanation isn't correct.

Hi,  Any updates on this?  @simzacks any suggestions on this?  Siemens is telling us there is a way to get and set via the WSDL.  if this is not the case we need to open a channel with them to show them we can't do it, so we can get a fix.
thanks, Brian

@brian12559 - the challenge is knowing what the names of the custom fields are so that they can be used natively in pylero.
The other objects that have custom fields have a wsdl function that returns the list of field names.

@simzacks , OK, I think i understand.  So, while not ideal can we we hardcode this until we get a function that can return the list of names.  For the Test Failure Analysis part of this the custom fields we are going to add will be at the global level and will be able to changed at the Project level.  In this case we will know exactly the IDs of the 3-4 custom fields we need for this.  Will this allow us to hack around it?

It could be done,  the challenge is that the repo is now open-source, so we
can't add anything specifically red hat to it.
Another option that we used for test runs before they added a function,
was we downloaded the xml file from the repo and parsed it.
On Tue, Dec 14, 2021, 18:06 brian12559 @.***> wrote:

@simzacks https://github.com/simzacks , OK, I think i understand. So,
while not ideal can we we hardcode this until we get a function that can
return the list of names. For the Test Failure Analysis part of this the
custom fields we are going to add will be at the global level and will be
able to changed at the Project level. In this case we will know exactly the
IDs of the 3-4 custom fields we need for this. Will this allow us to hack
around it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RedHatQE/pylero/issues/41#issuecomment-993694258, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADF6GMHIXT77HE4UUNB23NTUQ5TPFANCNFSM5HHDZRAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@brian12559 @leelavg Please don't post internal urls and sensitive data like credentials here on the open project, I have removed the credential in one comment. While the urls other than almdemo are still an issue here.
I don't think I have more knowledge then @leelavg on how this could be done as detailed in
https://github.com/RedHatQE/pylero/issues/41#issuecomment-973790408
as I checked user could not download the xml the prod polarion instance, @simzacks so it means download and parse it's not gonna work unless special user credentials or privilege been set up somehow which still will not work for general users.
So someone need request Siemens for providing WSDL with TestRecord like:
getDefinedTestRecordCustomFieldTypes

are still an issue here.
Ack, not going to post internal links again.

ACK...fixed mine...my bad


Update Wrangler KV commands in docs to remove ':'

FYI, pushed up some changes to address the conflicts introduced by merging in #15309


Update no-license

qualify "no [actions]" with "without being at risk..."
mention exceptions and limitations, jurisdiction-specific
mention that collaborators don't give you permission, either
remove trivial example copyright notice, suggest adding statement symmetric with for-users section below
suggest exploring contributor agreement so non-licensor has permission from contributors

closes #444 #445

Thank you.


Ryujin Sabotage questline

Report information to David at SSNN not available - Sabotage Ryujin questline

Details
Final portion of the questline broken, waypoint leading to David at SSNN tracks off the map outside of New Atlantis. When you try to go in to SSNN with the questline tracked or untracked, the NPC David is not there. Unable to complete questline. Tried resetting and quitting the game. Even went back to an earlier save to see if it would resolve, but still results in the same outcome.
Game Version
1.7.33
New Game
No
Reproduction Steps
Reload earlier save game prior to this quest point, still resulted in the same outcome, waypoint does not track properly and the NPC is not around to complete the final quest point

Can’t progress through to the last part of the quest for the final rewards

Any update on questline glitches?

You may need to share your save where this is happening as it didn't happen for me.

This is a unique issue I have had happen a few times now with other various quests targeting NPC's randomly showing up at various locations on the map.
All you can do is travel to the marker in a straight line till you find em usually trapped on some terrain or object at that location.
I feel the issue is due to a Spawn bug with the npc's spawning or placing npc's in the incorrect spawn locations.

Never experienced this; supposedly fixed officially in 1.10.31.


[thisav] Fix video extraction, uploader extraction and tests.

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
The thisav extractor has been broken for almost 2 years, this PR attempts to
fix it. I've seen PRs being turned down because they made untested changes to
common.py, while other PRs only changed the regex slightly and hoped for the
best. This PR avoids as much mpd parsing business as possible and extracts only
the relevant bits robustly.

@dstftw Ping for review. Thanks.


[v2] Missing meta tag in head to allow fullscreen webapp on iOS

Adding a shortcut to home page allowed for the app to be launched full screen on v1. v2 is missing this feature.

As a work-around you could first save it to home-page with v1 and then launch the v2 server.

What tags do we need for this, anyway? I think in v1 they were all generated by webpack-pwa-manifest but we don't use that anymore.

I believe this is it:

Thanks!


feat(bundler-vite): dynamically handle plugins and themes for vite (close #585)

Handled in b85b1c3b39e80a8de92a7469381061f75ef33623

I think a filter could still be added

It would fail to resolve in pnpm.


feat: support pandora log to view app/daemon logs

Codecov Report

Merging #20 into beta will decrease coverage by 25.07%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##             beta      #20       +/-   ##
===========================================
- Coverage   78.44%   53.36%   -25.08%
===========================================
  Files         168      156       -12
  Lines        4653     4411      -242
  Branches      553      518       -35
===========================================
- Hits         3650     2354     -1296
- Misses        782     1945     +1163
+ Partials      221      112      -109

Impacted Files
Coverage Δ

packages/pandora/src/daemon/DaemonHandler.ts
0% <0%> (-77.02%)
:arrow_down:

packages/pandora/src/universal/LoggerBroker.ts
51.61% <50%> (-48.39%)
:arrow_down:

packages/pandora/src/application/ClusterSupport.ts
0% <0%> (-100%)
:arrow_down:

packages/pandora/src/monitor/Monitor.ts
0% <0%> (-96.88%)
:arrow_down:

packages/pandora/src/daemon/DaemonBootstrap.ts
0% <0%> (-92%)
:arrow_down:

.../application/ServiceRepresentationChainModifier.ts
0% <0%> (-89.48%)
:arrow_down:

packages/pandora/src/monitor/MonitorManager.ts
0% <0%> (-86.67%)
:arrow_down:

packages/pandora/src/service/ServiceLogger.ts
10% <0%> (-85%)
:arrow_down:

...s/pandora/src/application/WorkerContextAccessor.ts
16.66% <0%> (-83.34%)
:arrow_down:

packages/pandora/src/daemon/spawnWrapper.ts
0% <0%> (-83.34%)
:arrow_down:

... and 44 more

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 dd9b114...facecfd. Read the comment docs.


How to make a custom GET request with relation

I am wondering how I can use restmod to perform a custom GET request on a relation model that I have defined below:
angular
.factory('AccountModel', function() {
    return restmod.model('/account').mix({
       audits: {hasMany: 'AuditModel'}
    });
})
.factory('AuditModel', function() {
    return restmod.model().mix();
})

That allows me to make requests like so:
AccountModel.find(123).audits.$collection(params) // '/accounts/123/audits?params'

But what I am needing to do is a GET request on the following URL:
/accounts/123/audits/templates

I need to be able to get a $collection of templates, too.
I tried nesting relations like so, but it didn't work out, as the template model was undefined
angular
.factory('AccountModel', function() {
    return restmod.model('/account').mix({
       audits: {hasMany: 'AuditModel'}
    });
})
.factory('AuditModel', function() {
    return restmod.model().mix({
       templates: {hasMany: 'TemplatesModel'}
    });
})
.factory('TemplatesModel', function() {
    return restmod.model().mix();
})

Have you seen the hasMany options in the documentation ? below I have copied the snippet from readme. Parameter "path" can be used to alter the relationship link.

HasMany Options
The has many relation provides the following options for you to customize its behaviour:
path: will change the relative path used to fetch/create the records. Ex: { hasMany: 'Part', path: 'pieces' }
inverseOf: adds a property on the child object that points to the parent. Ex: { hasMany: 'Part', inverseOf: 'bike' }.
params: optional query string parameters to be used when fetching collection. Ex: { hasMany: 'Part', params: { foo: 'bar' } }.
hooks: you can also define hasMany relation hooks. Check the hooks advanced documentation for more information.

You can do like below:
angular
.factory('AccountModel', function() {
    return restmod.model('/account').mix({
       audits: {hasMany: 'AuditModel', path: 'audits/templates'}
    });
})
.factory('AuditModel', function() {
    return restmod.model().mix();
})

Thank you @furqanaziz - I have been able to use path to get what I need.


Invalid hostname

Alpine Linux gets it's hostname from /etc/hostname or /etc/conf.d/hostname and sets it via the init script /etc/init.d/hostname
Can you put rc-update add hostname default back? If I add it back and start it manually, i get the right hostname

Thanks @ScrumpyJack, it works :)


Rename Gaussian Attributes to not use Unicode.

For https://github.com/witheve/Eve/issues/739
Once in I'll also post a change notice to the news-group.
Regards,
Cormac.

Thanks!


Blue Shark Soft Toy!

About the PR
Adds a lovable shark plushie in classic blue, pretty pink, and dignified grey.
Screenshots

https://user-images.githubusercontent.com/97707302/196607604-57320f04-0d4e-4142-a785-de8544e96705.mp4
Changelog

:cl:

add: Added Nanotransen Oceanic Apex Predator Plush Figure - Available in blue, pink, and grey!

Added blue shomk to syndie pyjamas bundle, thanks @Mervill good comment 🙌


NPE at at org.knowm.xchart.internal.chartpart.PlotContent_XY.doPaint(PlotContent_XY.java:60)

java.lang.NullPointerException
at org.knowm.xchart.internal.chartpart.PlotContent_XY.doPaint(PlotContent_XY.java:60)
at org.knowm.xchart.internal.chartpart.PlotContent_.paint(PlotContent_.java:51)
at org.knowm.xchart.internal.chartpart.Plot_.paint(Plot_.java:37)
at org.knowm.xchart.internal.chartpart.Plot_AxesChart.paint(Plot_AxesChart.java:35)
at org.knowm.xchart.XYChart.paint(XYChart.java:418)
at org.knowm.xchart.XChartPanel.paintComponent(XChartPanel.java:113)
at java.desktop/javax.swing.JComponent.paint(JComponent.java:1074)
at java.desktop/javax.swing.JComponent.paintToOffscreen(JComponent.java:5255)
at java.desktop/javax.swing.BufferStrategyPaintManager.paint(BufferStrategyPaintManager.java:246)
at java.desktop/javax.swing.RepaintManager.paint(RepaintManager.java:1323)
at java.desktop/javax.swing.JComponent._paintImmediately(JComponent.java:5203)
at java.desktop/javax.swing.JComponent.paintImmediately(JComponent.java:5013)
at java.desktop/javax.swing.RepaintManager$4.run(RepaintManager.java:865)
at java.desktop/javax.swing.RepaintManager$4.run(RepaintManager.java:848)
at java.base/java.security.AccessController.doPrivileged(Native Method)
at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
at java.desktop/javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:848)
at java.desktop/javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:823)
at java.desktop/javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:772)
at java.desktop/javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1890)
at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
at java.base/java.security.AccessController.doPrivileged(Native Method)
at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Can you post code that produces the error? Otherwise there's no way for me to debug and fix the issue...

I cannot unfortunately.. but I think the fix is as easy as checking for null yAxis
  Axis yAxis = chart.getYAxis(series.getYAxisGroup());
  double yMin = yAxis.getMin();

in PlotContent_XY.java
if ( yAxis == nuil )
continue;
just like the isEnabled check before it

It happens in a multithreaded situation where the chart is being constructed, a refresh happens. but the chart isnt all the way complete

@crowlogic @timmolter
This situation occurs when multiple threads are drawing the same Chart. When drawing the axisPair, the prepareForPaint() method will clear the yAxisMap.
private void prepareForPaint() {

    yAxisMap.clear();
    ...
}

When yAxis == null  skip, the Chart drawing will be inaccurate.
You can add synchronization when multiple threads draw the same Chart.

We didn't make this library with multiple threads updating the chart in mind so it's not surprising that this issue is occurring. I don't want to add null checks all over the codebase and chase these errors until the end of time. Maybe someone can post some threadsafe code as @Mr14huashao mentioned and we can direct people towards that. We could even post it in the FAQ.


Update reek api use

Reek's Reek::Examiner class was moved to Reek::Core::Examiner in reek 2.2.0, causing pronto-reek to fail. This PR fixes this.
Note, reek's API for programmatic use hasn't really been defined yet, which is why this change was introduced in a minor release. We're hoping to have a well-defined and stable API in place in the 3.0 release.

Thanks! :bow:
Released as 0.4.2 :smile:


grouping by @Api value

all our controllers have annotations like this one:
@Api(value = "auth", description = " ")
in the old implementation of swagger-springmvc the operations where grouped by the value (here auth). e.g. having multiple controllers with the same @Api annotation where showing up in the same section in the UI.
Now since I'm migrating to swagger 2.0, all controllers are listed separately and the value of the API annotation seems to be ignored.
Is there anything I can do to get the old behavior back?

@imod The resource grouping is no longer available. It used to be that way for swagger 1.2.
Since swagger 2.0 uses tags to facilitate grouping, this feature is no longer supported (hence the deprecation). Keep in mind that while swagger-ui might not respect tags, it will in the future.

is there any documentation how to use tags with springfox? So I could at least start grouping them already, so it will work with a future version automatically?

@imod Details available in this issue. Essentially
tags can be defined via @Api at the controller level and @ApiOperation annotation at the operation level. The tags on each operation are a union of tags on the Controller and the handler method. Swagger-ui does not support tags yet, but when you use the swagger editor you can use it to group by tags as u see fit

@dilipkrish grouping by tags is still not available - right?


bug: in case a final / remain in push URL, the push will fail

Simon, I don't understand that issue. Could you elaborate on it for me, please?

The remote repo should be exactly on the form:
https://github.com/user/repo
A remote repo like: https://github.com/user/repo/ or https://github.com/user/repo.git can led to a bug when pushing.


EllipseEditorObservable Disposing

Disposing of EllipseEditorObservable throws an error
Uncaught TypeError: Cannot read property 'dispose' of undefined .
Go to https://stackblitz.com/edit/angular-cesium-ellipse-editor-example?embed=1&file=src/app/ellipse-editor-example/ellipse-eidtor-example.component.ts, create an ellipse a then click 'Stop Edit Ellipse' to reproduce.

Looks like it is crashing here:
https://github.com/articodeltd/angular-cesium/blob/2d451bdd328fcf81af7ff994847a8fffde17c96c/projects/angular-cesium/src/lib/angular-cesium-widgets/services/entity-editors/ellipses-editor/ellipses-editor.service.ts#L398
The addSecondRadiusRegistration is undefined on line 390 and then the observables map gets polluted with the undefined element.
Hopefully with that info you can dig a little deeper into the problem.  If you need further help I can look into it more next week.

@boyanstanchev where you able to find a solution to this?  I can still assist further if you would like me to.

@anotherhale Thanks for the info, how can I test and debug this? I've forked and cloned the repository. It's my first time contributing to a project.

I think I found where the undefined is coming from https://github.com/articodeltd/angular-cesium/blob/2d451bdd328fcf81af7ff994847a8fffde17c96c/projects/angular-cesium/src/lib/angular-cesium-widgets/services/entity-editors/ellipses-editor/ellipses-editor.service.ts#L390. addSecondRadiusRegistration is always undefined and we have if statement checking that, so maybe const observables = [pointDragRegistration] will fix that?


dataset and models的下载链接失效了

您好，关于工程Readme中对于dataset and models的下载链接失效了是吗，还是换成了新的链接这里没有更新呢，烦请指导如何下载dataset and models

I've double-checked, and the download link appears to be valid. Have you configured the correct proxy settings?


Prefer MISSION_ITEM_INT over MISSION_ITEM

Previously I was using raw mavlink to communicate with a PX4 controller and it would request and accept the MISSION_ITEM_INT messages for mission uploads. I've now switched to using MAVSDK and the same PX4 controller will no longer request or accept MISSION_ITEM_INT, just MISSION_ITEM (using the pass-through plugin). Is something being set to cause this? Is there a configuration of MAVSDK I can set to allow use of MISSION_ITEM_INT?

What version of PX4 are you using?


Add session tags

Role sessions can now be tagged: https://docs.aws.amazon.com/IAM/latest/UserGuide/id_session-tags.html

A role's trust policy needs to grant permission to call sts:AssumeRole, so we probably want:

To make tags optional (probably on a tag-by-tag basis)
Allow this to be configured at a server-wide level by administrators

Thoughts on interesting tags:

Teamcity server root URL
Build ID
Project ID
Parent project ID
Triggered by
VCS number (e.g. git commit sha)
Information about the build agent used
Whether it's a "personal build"

We could configure tags using parameters matching the pattern teamcityAwsRolePluginTag.$tagName = "%some.value%
Then it could be configured by administrators at the root project level, etc. Is it possible to block parameters from being maliciously overridden? 🤔

param("awsrole.tags.teamcity:vcsNumber", "%build.vcs.number%")
param("awsrole.tags.teamcity:vcsUrl", "%vcsroot.url%")
param("awsrole.tags.teamcity:buildId", "%teamcity.build.id%")
param("awsrole.tags.teamcity:triggeredBy", "%teamcity.build.triggeredBy%")
param("awsrole.tags.teamcity:buildTypeId", "%system.teamcity.buildType.id%")
param("awsrole.tags.teamcity:projectId", "%teamcity.project.id%")


API erroneously requiring manufacturer when submitting inventory through /api/dcim/inventory-items/

Environment

Python version:  2.7.15rc1
NetBox version:  2.4.2

Steps to Reproduce
POST a json body without manufacturer to /api/dcim/inventory-items/, e.g.:
$ curl -d '{"device": 2171, "name": "RAID Controller", "description": "LSI Logic MegaRAID"}' netbox-s2/api/dcim/inventory-items/ -H 'authorization: Token 00deadbeef00' -H 'Content-Type: application/json; version=2.4;' -sv

This seems similar to the issue reported in #2305

Expected Behavior
The item should be created successfully with a null manufacturer
Sending the same payload to netbox 2.3 works fine -- a 201 created response comes back
Creating an inventory item through the UI at /dcim/devices//inventory/
accepts items without manufacturer just fine (and when queried via the API, they appear with a null manufacturer)

Observed Behavior
 HTTP/1.1 400 Bad Request
{"manufacturer":["This field is required."]}

Similar issue to #2335, where UniqueTogetherValidator is forcing manufacturer as a required field. Adding default=None to the field relaxes this requirement.


[Notion] Change GC complexity

Description
Change the big O complexity of the GC notion_pages fetching algorithm from O(N^2) to O(N).

Risk

Deploy Plan

We could but they are not problematic for now.
On Thu, May 16, 2024 at 5:32 PM Henry Fontanier @.***>
wrote:

@.**** approved this pull request.
Looks great but I think we want to do the same below where we have the
exact same patter
In connectors/src/connectors/notion/temporal/activities.ts
https://github.com/dust-tt/dust/pull/5102#discussion_r1603607432:

offset = 0;

let offset = 0;

I think we want the exact same for databases below no ?
—
Reply to this email directly, view it on GitHub
https://github.com/dust-tt/dust/pull/5102#pullrequestreview-2061176421,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACXUNJ25CXZDW7HVDS6X6LZCTGQZAVCNFSM6AAAAABH2OYC5GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANRRGE3TMNBSGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

And I would like to see how this behaves before moving forward. WDYT?
On Thu, May 16, 2024 at 5:39 PM Aric Lasry @.***> wrote:

We could but they are not problematic for now.
On Thu, May 16, 2024 at 5:32 PM Henry Fontanier @.***>
wrote:

@.**** approved this pull request.
Looks great but I think we want to do the same below where we have the
exact same patter
In connectors/src/connectors/notion/temporal/activities.ts
https://github.com/dust-tt/dust/pull/5102#discussion_r1603607432:

offset = 0;

let offset = 0;

I think we want the exact same for databases below no ?
—
Reply to this email directly, view it on GitHub
https://github.com/dust-tt/dust/pull/5102#pullrequestreview-2061176421,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACXUNJ25CXZDW7HVDS6X6LZCTGQZAVCNFSM6AAAAABH2OYC5GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANRRGE3TMNBSGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Good for me @lasryaric


Fix warning about shadowed arg1

I got this warning when installing rom-core-5.1.2:
Installing darkfish documentation for rom-core-5.1.2
(eval):9: warning: key :arg1 is duplicated and overwritten on line 9
(eval):9: warning: key :arg1 is duplicated and overwritten on line 9

I believe it is related to this line.

thanks!

That was super fast! Thanks!


v0.8.5

fixes https://github.com/firebase/firebase-jobdispatcher-android/issues/157
fixes https://github.com/firebase/firebase-jobdispatcher-android/issues/161

Looks like 0.8.5 isn´t published to jcenter already?

When can we expect this? I'm urgently waiting for this release!


[BUG]: stripe.confirmAcssDebitPayment is not a function

What happened?
I have installed the latest stripe-js v1.54.1, upon trying to implement one of the functions in your docs: https://stripe.com/docs/js/payment_intents/confirm_acss_debit_payment It says it is not a function?
Environment
No response
Reproduction
No response

hey @taylorchance  -- can you share your code, include the way you initialize Stripe.js and call that method?

Hello @brendanm-stripe, I am using a vue3 application, I import as such:
import '@stripe/stripe-js'
import { useStripe, StripeElement } from 'vue-use-stripe'
I make an API request to const intent = https://api.stripe.com/v1/setup_intents?customer=${user.value.uid}
and then pass try to run:
        payment_method: {
          billing_details: {
            name: 'Marys Chicken',
            email<EMAIL_ADDRESS>          },
        },
      });```

Ah, I'm not super familiar with Vue or vue-use-stripe. The above doesn't show how stripe is initialized. Are you using loadStripe from @stripe/stripe-js?
I suspect that your stripe instance is not being persisted. At the point there you call ``, are you able to create an elements group?
var elements = stripe.elements({
  clientSecret: 'CLIENT_SECRET_FROM_SI_OR_PR',
});

https://stripe.com/docs/js/elements_object/create


Zone Telechargement shows an SSL error on every Feed

Describe the bug
Zone Telechargement bridge does not work because of an SSL error :
Young Sheldon - Saison 5 Qualité WEB-DL 720p | VOSTFR - Zone Telechargement was unable to receive or process the remote website's content!
Error message: Unexpected response from upstream. cUrl error: SSL certificate problem: unable to get local issuer certificate (60) PHP error:
Query string: action=display&bridge=ZoneTelechargement&context=Suivre+la+publication+des+%C3%A9pisodes+d%27une+s%C3%A9rie+en+cours+de+diffusion&url=telecharger-serie%2F82405-telecharger-young-sheldon-saison-5-saison-5-web-dl-720p-vostfr.html&filter=both&format=Atom
Version: dev.2021-04-25
To Reproduce
Steps to reproduce the behavior:
Create any Feed with Zone Telechargement bridge
Expected behavior
Shows the feed content
Screenshots
Additional context
I'll try to fix this issue in the next days !

curl "https://zone-telechargement.stream/" -I
curl: (60) SSL certificate problem: unable to get local issuer certificate

https://decoder.link/sslchecker/zone-telechargement.stream/443
I believe they are missing a third cert in their cert chain. Might be related to prior issues with expiration of letsencrypt root ca.
Some solutions here: https://github.com/RSS-Bridge/rss-bridge/issues/2397

I had something similar in Anidex and did the following in #1693:

Disable certificate validation before performing requests:

https://github.com/RSS-Bridge/rss-bridge/blob/master/bridges/AnidexBridge.php#L139-L142
$opt[CURLOPT_SSL_VERIFYHOST] = 0;
$opt[CURLOPT_SSL_VERIFYPEER] = 0;

This is essentially the same thing as @dvikan mentioned above.

Intentionally mark bridge as insecure by setting URL to http:// instead of HTTPS

https://github.com/RSS-Bridge/rss-bridge/blob/master/bridges/AnidexBridge.php#L6
const URI = 'http://website.com/'; // mark bridge as insecure because certificate validation is disabled

You may manually change back to HTTPS dynamically by adding something like self::URI = str_replace('http://', 'https://', self:URI); before performing requests. The point is to let the user know that the requests will be insecure, because certificate validation is disabled. This is not really a big deal for content scrapping since nothing sensitive is done (no credentials...) but still better to let the user know about it.

Hello !
Thanks for your help :)
I was offline for a several days, so I did not have time to search a solution !
Those old URLs are missing some intermediate certificate, and therefore cURL is complaining about the SSL chain being incomplete (and I think, they won't be fixed, because no one should us them anymore !).
I think I found a solution without bypassing SSL verification, but I must do additional testing, and maybe I could do only one HTTP request, instead of two !
Stay tuned !

It's even better if you can fix it without bypassing 👍

The PR #2426 should fix this issue :)
(without bypassing SSL verification :sweat_smile:)


refinements: removed from individual cases

fixes #645

Getting rid of the refinements in the Case classes is good 👍 and I think what you have here is mergable.
I'm not sure continuing to use the function with refinements the way we are here is a good idea. Underscoring a string and underscoring an Integer are two fundamentally different operations performed for different reasons, yet they are called using the same function.
Perhaps two different methods is the right way to go, but naming them based on why rather than what.
@kotp had some thoughts about this (that I can't find to link to right now.)

I agree that string underscore and integer underscore are very different operations, and I'm in favor of creating the literal(number) and underscore(string) methods.

I'm in favor of creating the literal(number) and underscore(string) methods.

Is this something you want to do as part of this PR?

@Insti sure, no problem! I can make the change right now actually.

@Insti I think this is  good to, but let me know if you want any more changes.

Looks good, thanks @ajwann


Keyboard Stops

Microsoft PowerToys version
0.53.1
Running as admin

[X] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
No steps. Random issue.
✔ Expected Behavior
Keyboard functions
❌ Actual Behavior
Laptop keyboard, USB Keyboard, and Bluetooth keyboards all stop responding. Only resolution is to reboot.
Other Software
No response

Sorry to hear. Strange, though. How did you determine Keyboard Manager to be the cause, exactly? Do your keyboards keep working when the module - or all of PowerToys - is not running?

Yes, if I do NOT use PowerToys, no issues with the keyboards at all. If I'm
using Power Toys, then the Keyboards have issues and will randomly stop
responding....and I did neglect that I do use Logitech Gaming Software, so
maybe a conflict somewhere?
On Tue, Jan 18, 2022 at 12:21 PM Jay @.***> wrote:

Sorry to hear. Strange, though. How did you determine Keyboard Manager to
be the cause, exactly? Do your keyboards keep working when the module - or
all of PowerToys - is not running?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/15612#issuecomment-1015639247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXBF2CDRGDHFLWGEILLLAX3UWWOSBANCNFSM5MHIB7KA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Suddenly experienced something like this too. The keyboard input was reduced to around one keystroke per second. Issue goes away once powertoys is restarted.

Could you attach a full bug report file?
Right-click on the tray icon and click Report Bug. Then drag-and-drop the generated file into the GitHub comment to upload it. Thanks!

Same issue. Keyboard stops working randomly. Nothing to determine what causes it.

Able to Recreate the Issue
When I switch my left Alt key and the Windows key this is when the issue of the keyboard randomly not working occurred. Hope this helps to narrow down the issue.

@Lthrboytom what version of PowerToys are you using and what is your computer's date & time?
In this zip I see "v.51.1" and "log_2021-12-16"

I'm sorry, this issue was several months ago. I no longer have the laptop
and have stopped using PowerToys.
On Fri, Aug 19, 2022 at 7:29 AM Jay @.***> wrote:

@Lthrboytom https://github.com/Lthrboytom what version of PowerToys are
you using and what is your computer's date & time?
In this zip I see "v.51.1" and "log_2021-12-16"
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/15612#issuecomment-1220562532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXBF2CADZY4RJT4X23E33LDVZ5VYRANCNFSM5MHIB7KA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okay thanks for answering


[Feature Request]: Technology Trends Analysis (Analyze shifts in technology preferences over time)

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Analyze shifts in technology preferences over time Explore LanguageWorkedWith, DatabaseWorkedWith, PlatformWorkedWith, FrameworkWorkedWith to identify popular technologies and their usage patterns.
Analyze shifts in technology preferences over time (LanguageDesireNextYear, DatabaseDesireNextYear, etc.).
source dataset ->survey_results_public_2018.csv
Use Case
Exploring LanguageWorkedWith, DatabaseWorkedWith, PlatformWorkedWith, and FrameworkWorkedWith helps identify popular technologies and usage patterns, enabling organizations to stay current with industry trends and make informed technology investments.
Benefits
Analyzing shifts in technology preferences over time (LanguageDesireNextYear, DatabaseDesireNextYear, etc.) allows companies to anticipate future demands, align skill development with emerging trends, and maintain a competitive edge.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Can you assign it to me? @sanjay-kv
Full name: Manish Rana
GitHub profile link: https://github.com/ranamanish674zu
Email ID<EMAIL_ADDRESS>Approach for this : python & machine learning
What is your participant role?: GSSoC-2024 contributor
Can you add the label for GSSoC, i want to work on it
Thanks.

I'm eager to contribute! Could you please assign me an issue to work on?
Thank you!


QC/log files  report NAS/TAS/IC still retained after filtering for papers processed through canto

This is a QC check
Redundant IC/NAS/TAS are filtered. This means that we do not always  delete old annotations in Artemis using this evidence code. However, we would like to know if, when we have annotated a paper in Canto, any Artemis annotations with these evidence  codes remain.
This could indicate one of 2 things
i) the old annotation was incorrect and needs deleting
ii) the  new annotation is missing something which was captured by the old annotation
If these were reported we could distinguish and fix appropriately
Original comment by: ValWood

I don't think we do filter redundant IC, need to re-check what is filtered... we  should suppress these if there is an EXP annotation (NAS, IC,TAS)

I usually delete them, but obviously this takes time...

closing because  we don't really need to do this...


ci: fix nightly build

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
as title
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Refer to a related PR or issue link (optional)

as title

I remember that v4 has an unstable download issue before. Has it been fixed upstream? Perhaps you can add the link for the change triggering this "fix".


Parse Vensim negative value associativity properly

In vensim, -2^2 will yield 4. In python, -2**2 will yield -4. We need to address signed values and put them in parentheses such that the first case gets translated to vensim as (-2)**2.

Behaves same as vensim, actually - test was mistaken.


add pods

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master      #5   +/-   ##
======================================
  Coverage    91.6%   91.6%
======================================
  Files          17      17
  Lines         441     441
======================================
  Hits          404     404
  Misses         37      37

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 351e835...b08e9a4. Read the comment docs.


Not running on windows

Hi,
I have had no issue running it on Linux (at least starting the software, it couldn't hook onto the game for some reasons even if I added amonguscapture's bindings which are supposedly compatible to 2020.12.9) but when running on windows I have no issue compiling it but no window shows up, and nothing is logged in the powershell prompt, both in debug and release build
I have installed both dotnet core 3.1 and 5.0 but it seems to compile with 5.0

I recommend using the official release for Windows, as stated in the readme.
if you're not seeing a window, make sure you've installed the GTK+ runtime linked in the comments.

I recommend using the official release for Windows, as stated in the readme.
if you're not seeing a window, make sure you've installed the GTK+ runtime linked in the comments.

Standard version is too much bloated UI-wise and I don't want to have to change it that much, I quite like the one from this repo, plus GTK is better than WPF / windows official things
Pretty sure GTK+ runtime is set up correctly so I don't know what's wrong there :/
Thanks anyway..

Standard version is too much bloated UI-wise and I don't want to have to change it that much, I quite like the one from this repo, plus GTK is better than WPF / windows official things
Pretty sure GTK+ runtime is set up correctly so I don't know what's wrong there :/
Thanks anyway..

This issue will be closed, as it is related to v1.0, which is no longer being worked on.


actions: bump node 20.11.1

Applied mkpr on .github/workflows/*.yml
sed s/20.11.0/20.11.1/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
54

Relevant Lines:
56

💛  - Coveralls


Fatal error: UnsafeRawBufferPointer has a nil start and nonzero count

After latest update by Carthage have this error on Xcode 11.3.
Failing while parsing answer with route from server.
In December there were no errors with routing.

👋 @gralod95
We're sorry to hear that you're having problems. Would you be able to provide us with a little bit more information?

What version are you using? If not on a version (say, on master), what's the SHA of your checkout?
What are your failing waypoints? (Typically this is a pair of coordinates, representing the origin and destination)
Could you share a Stack-Trace with us?

Thanks,
-Jerrad


Feature proposal: Allow greater options during initial container creation

As many have noticed, it's sometimes difficult to get a container running when that container requires special commands to be passed or doesn't have the 'latest' tag.
I would like to propose an enhancement to the current layout that would open an advance window allowing the user to set all initial parameters prior to the container being created.
The user flow would be as follow:
Search for image - > Click more - > Click advance options - > Create container with options
Instead of the current:
Search for image - > Create container with defaults -> Select container -> Change settings -> Re-create container with new settings
If you feel this would be a helpful change, then add a +1 or comment. Thanks :D

After reviewing this with @jeffdm it feels natural to add this step after the download.
The flow would be:
Search for image - > Click create container to initiate the download -> Download prompts the user with defaults from the image defaults -> Create container based on said defaults

+1.
But I'm not sure about the flow. Is is really good to prompt every time the user to confirm the command? I'm thinking more like a new button like the create that exists right now to alter the default behaviour. What do you think?

We are not going forward with this topic. Any new ideas about that?

Just need to come up with the proper user flow. I didn't see a ton of feedback here, so it's perhaps not needed?

:+1:
The fact that the container auto-starts on create is frustrating since I don't have the option to specify ports or other options with the create button. I think that the flow @FrenchBen proposed is valid as a first go round.

+1

+1

Progress?

I am interested in this issue for GSoC. @FrenchBen I will write a proposal for it.


[Feature Proposal] Option to generate utilities from screens object only

Right now I'm using something like this:
screens: {
  xs: '0',
  sm: '640px',
  md: '768px',
  lg: '1024px',
  xl: '1280px'
}

The xs key with 0 value allows us to sort breakpoint classes in alphabetical order:
xs:bg-red-100 xs:text-red-600 sm:bg-green-100 sm:text-green-600 md:bg-blue-100 md:text-blue-600
Another example:
screens: {
  b0: '0',
  b1: '640px',
  b2: '768px',
  b3: '1024px',
  b4: '1280px'
}

This allows us to sort all classes in alphabetical order.
I can remove unwanted classes (without breakpoints) using purgecss, but I would like to be able to disable them altogether (so that you cannot use breakpointless classes in development).
I know it looks verbose to write so many xs but some people prefer consistency over brevity.

Hey! Interesting idea — I haven't heard this request from anyone else so I'm going to say no as an official feature right now, but you can actually do this in user-land!
To do it you need to write a very simple PostCSS plugin, which you can inline directly into your PostCSS config file:
// postcss.config.js
module.exports = {
  plugins: [
    require('tailwindcss'),
    function(css) {
      let removing = false

      css.each(node => {
        if (node.type === 'comment' && node.toString() === '/* postcss start remove */') {
          removing = true
        } else if (node.type === 'comment' && node.toString() === '/* postcss end remove */') {
          removing = false
        }

        if (removing) {
          node.remove()
        }
      })
    },
    require('autoprefixer'),
  ],
}

What this plugin does is let you delimit chunks of a CSS file to automatically be removed using comments.
So if you do this in your CSS file, you'll get the results you want:
@tailwind base;
@tailwind components;

/* postcss start remove */
@tailwind utilities;
/* postcss end remove */

@tailwind screens;

Hope that helps!


Test against PHP 8.3

Change Log

Added

Fixed

Changed

Test against PHP 8.3

Updated

Removed

Deprecated

Security

Description

Closing this for now as there are few other things that needs to be changed first, it all can come in a one bigger batch.


Search on the documentation page doesn't work

Description of the modifications
Search on the documentation page doesn't work

Useful links and references
No response

Sure, thanks!


In my code this is not working. I can not visualize the ellipses once I start using the subscribers data for the diameter dimension

https://github.com/CodingTrain/website/blob/381e42ad49d842759e909aebeae44cf369be05a7/CodingChallenges/CC_109_subscriber_map/P5/sketch.js#L47

Duplicate of #1692


Master clue in pyramid plunder checks for wrong item

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error
the master clue in pyramid plunder checks for "Menaphite purple robe" it should be "Menaphite purple kilt"
Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Windows, Ubuntu, macOS]
RuneLite version: [e.g 1.4.6]
Launcher version: [e.g 1.5.2]

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

It doesn't check the wrong item, it just doesn't check for all combinations of menaphite robes. I've opened #4494 to fix this.


DIODE does not transform in latest master

Transformations broke since the merge of the serialization cleanup

How to reproduce?

/path/to/dace/scripts/diode
Then open gemm.py, then click mapreducefusion

Issue was fixed, but now exposed other issues


:warning: Make all ScorecardResult format options pointers

What kind of change does this PR introduce?
breaking change

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
Some of the AsX functions take pointers (AsProbe), some take structs (AsString, AsJSON2)
What is the new behavior (if this is a feature change)?**
All three AsX functions which have option structs, take pointers.

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
One item from #4048
Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Just curious, what's the motivation to make all of the functions return pointers instead of all return values?

From my own experience, being able to pass nil is slightly shorter when I just want to use the defaults. But I took a look at what some of our other libraries do, and they both use pointers for options too, so I figured that was better.
For example the go-gitlab often uses pointers: https://pkg.go.dev/github.com/xanzy/go-gitlab#ProjectsService.GetProject
As does our go-github library: https://pkg.go.dev/github.com/google/go-github/github#ChecksService.ListCheckRunsForRef


suppert "text/html" in script tag

This PR fixes #77977

https://github.com/microsoft/vscode/issues/77977#issuecomment-784110969


Macro crashes when polymorphic default constructor arguments types are mismatched

Hi,
I've recently discovered a curious fact about default values in Scala. In the presence of type parameters, the default value can be a specific type. For example:
case class Foo[A](value: A = 3)

Although value is a generic type, it's perfectly fine to assign an Int as its default value. Typechecking is deferred to the actual instantiation of Foo.
This causes problems for Magnolia...
Invoking the Magnolia macro when there is a mismatch between the default argument's type and the actual expected type causes the macro to crash with an exception.
Given the following minimal set up:
import magnolia.{CaseClass, Magnolia}
import scala.language.experimental.macros

trait Dummy[A]

object Dummy {
  implicit val dummyForString: Dummy[String] = new Dummy[String] {}

  type Typeclass[A] = Dummy[A]

  def combine[A](caseClass: CaseClass[Typeclass, A]): Typeclass[A] =
    new Dummy[A] {}

  implicit def gen[A]: Typeclass[A] = macro Magnolia.gen[A]
}

Compiling this:
case class Foo[A](value: A = 3)

object DummyTest {
  Dummy.gen[Foo[String]]
}

Crashes with:
[error] test/DummyTest.scala:6:12: exception during macro expansion:
[error] scala.reflect.macros.TypecheckException: type mismatch;
[error]  found   : Int
[error]  required: String
[error]         at scala.reflect.macros.contexts.Typers.$anonfun$typecheck$3(Typers.scala:44)
[error]         at scala.reflect.macros.contexts.Typers.$anonfun$typecheck$2(Typers.scala:38)
[error]         at scala.reflect.macros.contexts.Typers.doTypecheck$1(Typers.scala:37)
[error]         at scala.reflect.macros.contexts.Typers.$anonfun$typecheck$7(Typers.scala:50)
[error]         at scala.reflect.internal.Trees.wrappingIntoTerm(Trees.scala:1756)
[error]         at scala.reflect.internal.Trees.wrappingIntoTerm$(Trees.scala:1753)
[error]         at scala.reflect.internal.SymbolTable.wrappingIntoTerm(SymbolTable.scala:28)
[error]         at scala.reflect.macros.contexts.Typers.typecheck(Typers.scala:50)
[error]         at scala.reflect.macros.contexts.Typers.typecheck$(Typers.scala:32)
[error]         at scala.reflect.macros.contexts.Context.typecheck(Context.scala:18)
[error]         at scala.reflect.macros.contexts.Context.typecheck(Context.scala:18)
[error]         at magnolia.Magnolia$.$anonfun$gen$58(magnolia.scala:376)
[error]         ...
[error]   Dummy.gen[Foo[String]]
[error]            ^
[error] one error found

The culprit is an invocation of an intentional typecheck during macro expansion here:
val dummyArgs = params.filterNot(_.isParamWithDefault).map(p => q"${p.name} = $x")
// `A <: Nothing` to avoid dead code warnings
val dummy = c.typecheck(q"def $x[$A <: $scalaPkg.Nothing]($x: $A) = new $genericType(..$dummyArgs)")

The second line throws an exception and crashes the macro.
Ideally, I think that in such a mismatch, it should be considered that the default value is absent.
Will you be open to accept a patch to fix this issue? If so, pointers as to how to tackle it will be greatly appreciated (the mechanics of the code quoted above are a bit opaque to me).
Thanks

Hi @ncreep a fix would certainly be accepted.
I don't have any pointers at the moment how it would look like.
I can explain the mechanics of the code above.
We create a constructor call new Foo(a = x, b = x, c = x) for all parameters (a, b, c) that don't have a default value. x here is a dummy with type Nothing so we can substitute it for anything. Well it's a bit more complicated as you see to avoid dead code warnings.
After we typecheck this constructor call, the compiler inserts the default arguments for the remaining parameters - and we can extract them from the resulting tree. These gymnastics are necessary because we haven't found a better way to access the default arguments.

The problem is that you can't check that the types don't match before obtaining the default value and you can't obtain the default value before typechecking. And typechecking fails because the types don't match. So perhaps the best we can do is to catch the error and provide a better error message.

Well, I've seen other approaches to getting a handle on the default arguments, that do it one-by-one. For example, this. Is there any reason to think that it can't work out for Magnolia as well?
As for just improving the error message for this case. In my specific use-case, it'll suffice to just ignore the default arguments rather than erroring-out (maybe emit a warning). Is this an acceptable approach, or hiding errors this way will be too confusing for users?

I've seen other approaches to getting a handle on the default arguments, that do it one-by-one. For example, this. Is there any reason to think that it can't work out for Magnolia as well?

companion is buggy. We used to do it this way but look at the horrible hack we had:
https://github.com/propensive/magnolia/pull/219/files#diff-d10f395adc80ba75bb7634617a7c0e7a
I thought of another way though - walk the owner chain of the case class and the current owner chain until you find a common owner. This will let you construct a reference to the companion object without having the symbol. And then you will have to compare the types.

As for just improving the error message for this case. In my specific use-case, it'll suffice to just ignore the default arguments rather than erroring-out (maybe emit a warning). Is this an acceptable approach, or hiding errors this way will be too confusing for users?

I think that's acceptable with a warning. 👍

I think I have an approach that circumvents the problem, while not doing anything too clever beyond what's already there.
Instead of creating a list of dummy arguments only for the non-default parameters, I'll create a list of dummies standing in for all arguments. Then, for each default parameter at index i, omit i from the dummies list. Run the current code on this new list to extract the one default argument there.
This way, I'm checking one argument at a time, if it compiles fine, if not I omit it and move on (maybe with a warning).
The price I'm paying is the possibility of being quadratic in the number of arguments. Not sure whether it matters, but if it does, it's possible to keep the old code for the "happy path" and only apply the logic above when the old approach fails.
I have a messy sketch of this approach, and it seems to be okay. Will try to open a pull request in the coming days.

The price I'm paying is the possibility of being quadratic in the number of arguments as well as more typechecking runs. Not sure whether it matters, but if it does, it's possible to keep the old code for the "happy path" and only apply the logic above when the old approach fails.

Yep that would be too expensive. We really try to keep the compile times down in Magnolia.
It's a major goal of the project.

You beat me to it...
I guess your way is better anyways. Thanks for the quick turnaround.
Is there a snapshot release that  I can depend on with the changes from #250?

I'm not sure if we publish snapshots


Not ignoring .map files

I have *.map in my .gitignore, but the ag command is showing results that are in map files as well (which are very very big and make it impossible to scroll over on the terminal window). I wonder if it is because the map files are nested more than 1 level deep.
Does anyone have any suggestion for what I could do to have ag ignore any .map files by default?

Does it successfully ignore files of other types or with other patterns?

Yes, it seems to be.

It works fine for me :
$ git init
Initialized empty Git repository in /tmp/tst-git
$ echo '*.map' > .gitignore
$ echo "foobar" > myfile.map
$ ag foobar # returns nothing
$ rm .gitignore
$ ag foobar
myfile.map
1:foobar

@netei can you try nesting your .map file within a couple directories deep?

@netei Try naming your map file foo.js.map. That seems to be the problem for me.

Still not able to reproduce :
echo "foobar" >> deep/nested/path/foo.js.map
ag foobar # Returns no results

Do you have ag version 0.31.0? Other than that, I am not sure what else would be the cause for this.

Right, with 0.31.0, the issue can be reproduced.
This issue seems to be a duplicate of https://github.com/ggreer/the_silver_searcher/issues/755


Add library documentation

Adds full documentation for Faktory Worker.
Tasks

[x] Update the README with

Getting started
Contributing
Developing

[x] Add a docs page about configuring Faktory Worker
[ ] Document all public modules, functions and types
[x] Bump version to 1.0.0

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #72   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          26     26
  Lines         346    346
=====================================
  Hits          346    346

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 0a88a5b...9a0c45f. Read the comment docs.

Thanks for the typo finds. I have since installed a spellchecker!