Swagger API for Azure CDN WebApplicationFirewallPolicy resource

This PR includes the following changes
Adds

Added the swagger api spec for CdnWebApplicationFirewallPolicy which is a new (public preview) arm resource under the Microsoft.Cdn provider
Added the relevant examples

Edits

To integrate this with Azure CDN, a new property "webApplicationFirewallPolicyLink" is added to the "Endpoint" sub resource in cdn.json arm resource.  This new property is the means by which a "CdnWebApplicationFirewallPolicy" can be references in a cdn "Endpoint" resource.
All the examples related to the "Endpoint" resource in cdn.json were edited to include this new property
Readme.md, a new package was added for this new change

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6183
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6183/commits/52a697e26d4650fb6302c657519aec14c5033c29

@veronicagg I am working on the LintDiff failures. However the Model Validation failures are unrelated to my change. It looks like they have always existed for cdn.json. I have only added a single property to cdn.json. The crux of my change is adding a new resource cdnwebapplicationpolicies.json.
How do I suppress the unrelated Model Validation failures? Ideally there should be something similar to LintDiff failures that detects what new failures were introduced.
We did the swagger review with a couple of folks from the arm api review board, should we schedule one with you as well to help you understand the changes better?

@veronicagg do you need anything else from my side for this pull request? All the validations are passing except ModelValidations which are old unrelated errors in cdn.json which have always been there. I have just added a single property to cdn.json and there are no failures related to that. I updated the pull request description to explain that better.

startbuild

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6183
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6183/commits/904da172ce557aa99fb564b33bc18e37f7baf65c

@veronicagg I have sent an email to the two reviewers from the ARM api board (David Justice and Marcin Jastrzebski) to see if they can sign off. You're on that thread as well.

@sushantsingh do you have any questions on the job failures? looks like some file paths are incorrect, see https://dev.azure.com/azure-sdk/public/_build/results?buildId=30680&view=logs&jobId=95dbed5b-845d-57ba-33d3-f6c5e69afcf4&taskId=44f1510e-a822-56b0-2e0b-241e0a387c6a&lineStart=74&lineEnd=75&colStart=1&colEnd=1

@sushantsingh I'm changing roles, so @amarzavery will take over this review. thanks!

@sushantsingh Could you please add old files first (new iteration) and then paste new content (another iteration) so that I can see the difference.. it is not practical to comb through these huge files to figure out what changed. thanks.

Hi @devigned  do you have any other pending concerns?

@sushantsingh please resolve the build issue. I know it is from a previous example, but it's not a good reason to keep propagating incorrect examples. Thank you!

@devigned Sure I can try to fix the build issue but I have no idea what it means (I did try my best to reduce the error count due to existing issues by fixing everything I could understand).
Can you please help me understand what "message: Cannot read property 'readUInt8' of null" means? Please expedite this if possible, our release date of June 15th.
Validating "examples" and "x-ms-examples" in  specification/cdn/resource-manager/Microsoft.Cdn/preview/2019-06-15-preview/cdn.json:
error :
operationId: CustomDomains_EnableCustomHttps
scenario: CustomDomains_EnableCustomHttpsUsingYourOwnCertificate
source: request
responseCode: ALL
severity: 0
code: INTERNAL_ERROR
details:
message: Cannot read property 'readUInt8' of null
code: INTERNAL_ERROR

@KrisBash are you good for preview. @sushantsingh and I chatted and we have commitment to add a subscription level get, improved documentation for models / operations, and to start a conversation about migrating cache duration to an ISO duration rather than string for the stable release.

Hi folk, this needs to be merged soon or we risk not meeting the preview release date of Jun 15th. Please let me know if anything is needed from our end to expedite things.
@KrisBash @amarzavery @devigned


add 'mark as read and unread'

Describe your suggested feature
Add the title says add in the chapter selection a button to set one chapter as read or unread when is need for
Can be useful to keep track or reread
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.12.
[X] I will fill out all of the requested information in this form.

Nvm it wasn't able for me in some novels till I re open the app maybe a bug


Add a GitHub Action that runs the build script

This is a just a quick experiment with GitHub Actions.
I realize that this particular action is not useful at all. See #7852.

This check succeeded on my fork so I'm going to merge it.

FYI, there's going to be a snag with Actions after November 13. They won't work until some time later (hopefully not too long after).

I'm curious to know more about that but good to know anyway. We are planning to use GitHub Actions more in the future but we'll hold off for now.

The tldr is it's just a contract thing we need to renew and it'll take a bit

Interesting. Thanks for sharing.


[KOGITO-9973] Add SonataFlow.Status.Services field and improve workflow properties ConfigMap reconciliation

Description of the change:
This PR adds supporting services to the SonataFlow status. With this change, we'll start using the SonataFlow.Status.Services when generating workflow properties ConfigMaps. This is to ensure SonataFlow.Status.Services matches & controls what is set in the managed application.properties ConfigMap.
This PR also ensures that all supporting services changes, whether cluster-wide or local to the workflow, are reflected via ConfigMap reconciliation... which automatically triggers the workflow deployment to rollout and use the new properties.
An example of the new status fields -
apiVersion: sonataflow.org/v1alpha08
kind: SonataFlow
metadata:
  name: workflow
  namespace: <namespace>
spec:
  ...
status:
  ...
  services:
    dataIndexRef:
      url: 'http://example-data-index-service.<namespace>'
    jobServiceRef:
      url: 'http://example-jobs-service.<namespace>'

Motivation for the change:
To reflect which supporting services are being used by each workflow so the user has a clearer understanding.
To ensure supporting services settings/changes are reconciled to workflows cluster-wide. This is critical. For example, we don't want workflows attempting to communicate against services which have been disabled or deleted. The workflows should always reflect the current state of the supporting services.
Checklist

[x] Add or Modify a unit test for your change
[x] Have you verified that tall the tests are passing?

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

i need to modify the e2e tests before we merge to account for this change.

ok, e2e tests have been added and should be working

@wmedvede @jakubschwan can you please review? @tchughesiv will have to jump into another task.


Flask issue

Hi, when building in docker on Mac, I get the following error.
I have installed Python3 and PIP via homebrew.
Also note that I haven't generated a vault seed as it was optional in the install instructions. (maybe that is the main root cause)
Collecting flask-user==0.6.17
Downloading Flask-User-0.6.17.tar.gz (62 kB)
ERROR: Command errored out with exit status 1:
command: /opt/conda/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/setup.py'"'"'; file='"'"'/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-h23y36ni
cwd: /tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/
Complete output (13 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/setup.py", line 5, in
from flask_user import version as flask_user_version
File "/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/flask_user/init.py", line 13, in
from . import forms
File "/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/flask_user/forms.py", line 71, in
class AddEmailForm(FlaskForm):
File "/tmp/pip-install-u1q79rq3/flask-user_dae01a91109d481f9b47d3f4148660ac/flask_user/forms.py", line 74, in AddEmailForm
validators.Email(_('Invalid Email')),
File "/opt/conda/lib/python3.8/site-packages/wtforms/validators.py", line 332, in init
raise Exception("Install 'email_validator' for email validation support.")
Exception: Install 'email_validator' for email validation support.
----------------------------------------
WARNING: Discarding https://files.pythonhosted.org/packages/64/67/1310c6062e114fb7619602d14e76fbcea5f338d6214c2cc2011f492f4e9e/Flask-User-0.6.17.tar.gz#sha256=2bdf0021810f69f9ee11c7586ce970519d8dd8ac46df3573f59f8d3ec352f4ac (from https://pypi.org/simple/flask-user/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement flask-user==0.6.17
ERROR: No matching distribution found for flask-user==0.6.17
ERROR: Service 'flask' failed to build : The command '/bin/sh -c pip install -r /back-end/requirements.txt' returned a non-zero code: 1

Updated the requirements file locally to flask-user==0.6.21 (that is the latest available in the 0.6.* versions) and it seems to go further when running docker-compose build.

And further Python version issues...
Solving environment: ...working... failed with initial frozen solve. Retrying with flexible solve.
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed
UnsatisfiableError: The following specifications were found
to be incompatible with the existing python installation in your environment:
Specifications:

pyomo.extras -> python[version='2.7.|3.5.|3.6.|>=2.7,<2.8.0a0|>=3.6,<3.7.0a0|>=3.7,<3.8.0a0|3.4.']

Your python: python=3.8
If python is on the left-most side of the chain, that's the version you've asked for.
When python appears to the right, that indicates that the thing on the left is somehow
not available for the python version you are constrained to. Note that conda will not
change your python version to a different minor version unless you explicitly specify
that.
The following specifications were found to be incompatible with each other:
Output in format: Requested package -> Available versions
Package libstdcxx-ng conflicts for:
python=3.8 -> libffi[version='>=3.2.1,<3.3.0a0'] -> libstdcxx-ng[version='>=4.9|>=7.2.0']
python=3.8 -> libstdcxx-ng[version='>=7.3.0|>=7.5.0|>=9.3.0']
Package libgcc-ng conflicts for:
python=3.8 -> zlib[version='>=1.2.11,<1.3.0a0'] -> libgcc-ng[version='>=4.9|>=7.2.0']
python=3.8 -> libgcc-ng[version='>=7.3.0|>=7.5.0|>=9.3.0']
Package python_abi conflicts for:
pyomo -> python_abi[version='2.7.|3.6|3.6.|3.7.|3.8.|3.9.|3.7',build='_cp27mu|_cp36m|_cp38|_cp37m|_cp39|_pypy37_pp73|_pypy36_pp73']
pyomo.extras -> dill -> python_abi[version='2.7.|3.6.|3.6|3.8.|3.7.|3.9.|3.7',build='_cp27mu|_cp36m|_cp38|_pypy36_pp73|_cp37m|_cp39|_pypy37_pp73']
Package zlib conflicts for:
pyomo.extras -> matplotlib -> zlib[version='1.2.|1.2.11|>=1.2.11,<1.3.0a0|1.2.8|1.2.11.']
ipopt -> scotch[version='>=6.0.8,<<IP_ADDRESS>a0'] -> zlib[version='1.2.|>=1.2.11,<1.3.0a0']
python=3.8 -> zlib[version='>=1.2.11,<1.3.0a0']
pyomo -> python[version='>=3.7,<3.8.0a0'] -> zlib[version='1.2.|1.2.11|>=1.2.11,<1.3.0a0|1.2.8|1.2.11.*']
Package _openmp_mutex conflicts for:
ipopt -> libgcc-ng[version='>=9.3.0'] -> _openmp_mutex[version='>=4.5']
pyomo -> libgcc-ng[version='>=9.3.0'] -> _openmp_mutex[version='>=4.5']
python=3.8 -> libgcc-ng[version='>=9.3.0'] -> _openmp_mutex[version='>=4.5']
Package libgfortran4 conflicts for:
ipopt -> libgfortran4[version='>=7.5.0']
ipopt -> libgfortran-ng -> libgfortran4=7.5.0
Package openblas conflicts for:
ipopt -> libblas[version='>=3.8.0,<4.0a0'] -> openblas[version='0.2.18|0.2.18.|0.3.5.|>=0.3.6,<<IP_ADDRESS>a0']
ipopt -> openblas[version='0.2.19|0.2.19.|0.2.20|0.2.20.|>=0.2.20,<<IP_ADDRESS>a0|>=0.3.3,<<IP_ADDRESS>a0']
Package libblas conflicts for:
ipopt -> liblapack[version='>=3.8.0,<4.0a0'] -> libblas[version='3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0.|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.8.0|3.9.0|3.9.0|3.9.0|3.9.0|3.9.0.',build='4_openblas|4_mkl|5_openblas|5_mkl|7_openblas|7_mkl|8_openblas|8_mkl|9_openblas|10_mkl|11_openblas|14_mkl|15_openblas|16_openblas|16_mkl|17_openblas|20_mkl|21_mkl|0_openblas|2_openblas|4_mkl|5_mkl|6_mkl|7_mkl|8_mkl|8_openblas|7_openblas|6_openblas|5_openblas|4_openblas|3_openblas|1_openblas|19_mkl|18_mkl|15_mkl|14_openblas|13_openblas|13_mkl|12_mkl|12_openblas|11_mkl|10_openblas|9_mkl|6_mkl|6_openblas']
ipopt -> libblas[version='>=3.8.0,<4.0a0']
Package blas conflicts for:
ipopt -> openblas[version='>=0.3.3,<<IP_ADDRESS>a0'] -> blas==1.0=openblas
ipopt -> blas==1.1=openblas
Package libgfortran-ng conflicts for:
ipopt -> libgfortran-ng[version='>=7,<8.0a0']
ipopt -> ampl-mp[version='>=3.1.0,<3.2.0a0'] -> libgfortran-ng[version='>=4.9']
Package libgfortran5 conflicts for:
ipopt -> libgfortran5[version='>=9.3.0']
ipopt -> libgfortran-ng -> libgfortran5[version='8.4.0.|9.3.0.']
Package libopenblas conflicts for:
pyomo.extras -> numpy -> libopenblas[version='>=0.2.20,<<IP_ADDRESS>a0|>=0.3.2,<<IP_ADDRESS>a0|>=0.3.3,<1.0a0']
ipopt -> libblas[version='>=3.8.0,<4.0a0'] -> libopenblas[version='0.3.3|0.3.3|0.3.3|>=0.3.10,<<IP_ADDRESS>a0|>=0.3.12,<<IP_ADDRESS>a0|>=0.3.12,<1.0a0|>=0.3.10,<1.0a0|>=0.3.9,<<IP_ADDRESS>a0|>=0.3.9,<1.0a0|>=0.3.8,<<IP_ADDRESS>a0|>=0.3.8,<1.0a0|>=0.3.7,<<IP_ADDRESS>a0|>=0.3.7,<1.0a0|>=0.3.6,<<IP_ADDRESS>a0|>=0.3.6,<1.0a0|>=0.2.20,<<IP_ADDRESS>a0',build='h5a2b251_1|h5a2b251_3|h5a2b251_2']
ERROR: Service 'trader' failed to build : The command '/bin/sh -c conda install -c conda-forge pyomo pyomo.extras ipopt' returned a non-zero code: 1

To solve the Flask issue I edited the back/back-end/requirements.txt by replacing "flask-user==0.6.17" with "just flask-user" and it cleared up all the first errors.
Then I got the same error you did: Specifications:
pyomo.extras -> python[version='2.7.|3.5.|3.6.|>=2.7,<2.8.0a0|>=3.6,<3.7.0a0|>=3.7,<3.8.0a0|3.4.']

Your python: python=3.8
I haven't resolved that one yet, but I have a Peppermint Linux virtual machine with python version 3.7.3 and conda 4.7.10 so I'm going to try and build it on there since that conda uses python 3.6-3.7. Hopefully that will get it done!
Did you get yours resolved?

I have solved it by editing back/trade-engine/Dockerfile
Change the 2 first lines to:
FROM continuumio/miniconda3:4.7.12
RUN apt-get update --allow-releaseinfo-change && apt-get install -y build-essential redis-tools
Explanation:

First line (the line begining with FROM) pinned a version using python 3.7
Second line (the line begining with RUN) added the option --allow-releaseinfo-change since that image was debian stable when it was launched, and now its oldstable. That switch prevents said condition to throw an error.

After those 2 changes, the build process finishes with no errors.

Hi javimurcia, the first line works but the second line throws and error. Any ideas? Here's what I got:
=> [ 1/13] FROM docker.io/continuumio/miniconda3                                                           0.0s
=> ERROR [ 2/13] RUN apt-get update && apt-get install -y build-essential redis-tools default-libmysqlcli  2.0s

[ 2/13] RUN apt-get update && apt-get install -y build-essential redis-tools default-libmysqlclient-dev:
#5 0.763 Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
#5 0.763 Get:2 http://security.debian.org/debian-security buster/updates InRelease [65.4 kB]
#5 0.995 Get:3 http://deb.debian.org/debian buster-updates InRelease [51.9 kB]
#5 1.294 Reading package lists...
#5 1.964 E: Repository 'http://security.debian.org/debian-security buster/updates InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
#5 1.964 E: Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
#5 1.964 E: Repository 'http://deb.debian.org/debian buster-updates InRelease' changed its 'Suite' value from 'stable-updates' to 'oldstable-updates'

executor failed running [/bin/sh -c apt-get update && apt-get install -y build-essential redis-tools default-libmysqlclient-dev]: exit code: 100
ERROR: Service 'flask' failed to build : Build failed

I have solved it by editing back/trade-engine/Dockerfile
Change the 2 first lines to: FROM continuumio/miniconda3:4.7.12 RUN apt-get update --allow-releaseinfo-change && apt-get install -y build-essential redis-tools
Explanation:
* First line (the line begining with FROM) pinned a version using python 3.7

* Second line (the line begining with RUN) added the option --allow-releaseinfo-change since that image was debian stable when it was launched, and now its oldstable. That switch prevents said condition to throw an error.

After those 2 changes, the build process finishes with no errors.

Hi Javimurcia, the first line worked but the second line failed. Do you have any ideas? Here's what I got:
=> [ 1/13] FROM docker.io/continuumio/miniconda3                                                           0.0s
=> ERROR [ 2/13] RUN apt-get update && apt-get install -y build-essential redis-tools default-libmysqlcli  2.0s

[ 2/13] RUN apt-get update && apt-get install -y build-essential redis-tools default-libmysqlclient-dev:
#5 0.763 Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
#5 0.763 Get:2 http://security.debian.org/debian-security buster/updates InRelease [65.4 kB]
#5 0.995 Get:3 http://deb.debian.org/debian buster-updates InRelease [51.9 kB]
#5 1.294 Reading package lists...
#5 1.964 E: Repository 'http://security.debian.org/debian-security buster/updates InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
#5 1.964 E: Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Suite' value from 'stable' to 'oldstable'
#5 1.964 E: Repository 'http://deb.debian.org/debian buster-updates InRelease' changed its 'Suite' value from 'stable-updates' to 'oldstable-updates'

executor failed running [/bin/sh -c apt-get update && apt-get install -y build-essential redis-tools default-libmysqlclient-dev]: exit code: 100
ERROR: Service 'flask' failed to build : Build failed


Alarmo validates whether sensors are clear within the exit delay period

When leaving the apartment, I expect to be able to arm the system even if there is motion on the sensors or the magnet sensors are open. The exit delay would start, and once I'm out all sensors would be cleared and alarm would arm. What happens now, is that I get a message that alarm can't be armed due to open door. Need to try the same with the motion sensors - I expect the same would happen.

Not a valid bug report, closing.
Please check the documentation regardng sensor configuration options.


Added Minimum spanning tree

Added minimum spanning tree using prim's algo in DP section.

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added Minimum spanning tree using Prim's Algorithm in DP section.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

@HarshCasper plz review my PR


No module named turtlebot3_autorace_camera.cfg

After I did the both camera calibration and intrinsic calibration, I went ahead to extrinsic calibration. I got an error after roslaunch turtlebot3_autorace_camera turtlebot3_autorace_extrinsic_camera_calibration.launch From what I read on the console it is a Python error showing the turtlebot3_autorace_camera.cfg cannot be imported.

Hi, @GToidZ :)
I'm sorry for late reply.
Could you try to follow command?
sudo chmod +x turtlebot3_autorace_camera.cfg
Thanks,
Gilbert.

@kijongGil Thanks for the reply, but I have already fixed it by reinstalling the ROSCore; and I apologized that I forgot to close the issue.

That's ok @GToidZ. If you have any other questions, feel free to ask. Thanks.


Individualizing logger.traces on server

@clebertsuconic I see what you are trying to achieve here.  But, the JIT compiler would likely remove any string creation and method calls after it realises that the branch is never reached.  Do you have any benchmark/test you are using to see if this makes any form of performance increase?  I think it's worth investigating before we start polluting the code with all these, if(isTrace) statements.

@mtaylor  I haven't changed anything in that regard...
the if (isTrace) is something we have done since the ancient days... all I did was to individualize the loggers:
if you called log.trace("a" + "b" + integerSomething");
the string will be concatenated.. sent down to log.trace to only be reject in case it was not traced.
In some cases we do quite a lot of traces.. at a very down level.. so that would definitely be relevant.
It is really costly... but as I said.. I didn't change anything.. we were already checking if (isTrace) some way or the other here.

@clebertsuconic I see you're not actually adding isTrace here.  Just refactoring what is already there.  I made the assumption due to our previous conversation.  I'll merge this, I just wonder if this pattern is still relevant with cool stuff the JIT compiler can now do.  It might be worth testing/benchmarking before wrapping all our log statements.  I'll merge this as is.  Thanks.

@mtaylor  I have done some changes after talking to jamezp


Fronius Wattpilot: re-add

Follow-up to https://github.com/evcc-io/evcc/pull/15213
/cc @mabunixda would you continue supporting this implementation?

Yes i gonna add the logging to evcc asap


Fixed tone for secondaryContainer for dark theme

The tone for secondaryContainer is different according to the design kit.

Fixed internally.


sRGB banding on Oculus runtime

Description
It appears as if something in the rendering surface presentation causes banding issues on dark scenes. My primary suspicion is that this has something to do with Linear/sRGB surface conversion, or possibly Oculus's color space extension.

Platform / Environment
Headset is Rift S (Windows?), doesn't appear in the Simulator

Additionally, this looks correct on Quest standalone, but does appear when using Quest+Link.


Shivansh2407 patch 20

Fixes #930
Describe the changes you have made in this PR -
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.279% when pulling 84a4740e852ae989bf013484d3c98be3abfd9733 on Shivansh2407-patch-20 into 840061f248780d383113cf13351d42e9b47de4fc on master.


Create a search form group-1

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in Form groups/form-groups-search-form-group1.html.
If you need to use any icon please use it from Hero Icons
Good luck.

!assign

not assign you? @its-kundan Ha ha.

@rupsha014 Can I work on this?

not assign you? @its-kundan Ha ha.

bro... its bot command for assign !!
I guess you are noob,

@Rajdip019 Assign this to me please! Done with the other 2

Okay go for it. @myGlitches

@Rajdip019 thanks!

Done!


Interop Datatransfermanager shown for a short time, almost 1 second Win10 21H2

Describe the bug
Using the Datatransfermanager on UWP works like a charm, on WinUI3 the Datatransfermanager is only shown for a short time, almost 1 second. This happens after migrating from Xamarin to .NET7.
I have implemeted the code from your documentation, but it only shows the share dialog for almost 1 or 2 seconds
This is the
example project. Please use the Buttons Native Share Dialog With Provider or Native Share Dialog Without Provider. The other Button is for a Maui Essentials Bug.
Do I need any capabilities for the  Datatransfermanager ?
Steps to reproduce the bug

Implement Datatransfermanager Interop
Nuget Package Windows App SDK  v1.3.230724000
Show Datatransfermanager

Expected behavior
The Datatransfermanager should be shown and not closed after 1 second.
Screenshots
No response
NuGet package version
None
Windows version
Windows 10 (21H2): Build 19044
Additional context
No response

It works for me in WinUI 3 (Windows 10 22H2) when I do initializations at beginning, like in a sample I had uploaded  (MainWindow.xaml.cs)

I tested my test sample as Unpackaged, x86 or x64, Debug or Release, but I cannot reproduce this problem
It just can be slow sometimes to display the GUI :

I also tested your example, but it didn't work:
https://github.com/microsoft/microsoft-ui-xaml/assets/44831004/65e0e83b-3dcb-4e88-8347-046c60061e5c

Did you test the MS WPF sample : https://github.com/microsoft/Windows-classic-samples/tree/main/Samples/ShareSource ?


Add scalastyle checker

This simply adds the scalastyle checker as an option.  It's working on my setup.

Thanks!


Start maximized

Do it is possible to start scrcpy maximized in Windows, without to have to click che window button?
the help have a tiny error:

--window-height value
    Set the initial window width.
    Default is 0 (automatic).

No, but why not fullscreen instead of maximized?

I would to switch other app from taskbar.

I have the same problem, how is the feature going on? @rom1v

Well, I found a workaround like this:
scrcpy --window-borderless --window-height=1080 --window-width=1880

This makes scrcpy start with a borderless  maximized  window without covering the taskbar.
My taskbar is on the right of the screen with 40px width, so I set the window width to 1920 - 40 = 1880.
This solves the problem but not perfect, because we have to do some calculate.

After I open a app the workaround fail.
The windows resize to app size.


Exporting png with a $ in a text label

Hey All,
I am using Plotly and Kaleido to export a png. The legend text labels begin with dollar signs such as '$50k' but this is special character (or something) and causes the text to become smaller font, truncated and italicized. If I use an escape character such as '\$50k' the text label stays '\$50k'. If I do r'$50k' it works when I output to html but not png (converts to smaller font, etc). Any help on getting around this would be much appreciated!
Thanks,
James

Hi @jtamerius,
It sounds like Latex rendering mode is somehow being triggered. But this should only happen if the first and last characters are $.
import plotly.io as pio
import plotly.graph_objects as go
pio.templates.default = None

pio.show(
    go.Figure(data=go.Scatter(y=[1, 3, 2], name="$Hello", showlegend=True)),
    renderer="png", engine="kaleido"
)

pio.show(
    go.Figure(data=go.Scatter(y=[1, 3, 2], name="$Hello$", showlegend=True)),
    renderer="png", engine="kaleido"
)

If you only have a leading $, and you're seeing an issue, could you provide an example that reproduces it?
Regardless, a workaround is to use the HTML code for the dollar symbol (see http://cactus.io/resources/toolbox/html-currency-symbol-codes)
pio.show(
    go.Figure(data=go.Scatter(y=[1, 3, 2], name="$Hello$", showlegend=True)),
    renderer="png", engine="kaleido"
)

You caught me. I didn't provide the entire label in my post because I didn't think it mattered. My bad. The entire label is '$50k-$100k'. The solution you have provided works!


add outline support

When opening in markdown-editor, the outline sidebar is disabled, showed "The active editor cannot provide outline information.".
Will you add this feature?
Thanks.

outline works here, I just have to reopen the outline pane everytime

@TheOneValen are you sure it works?  Maybe you're talking about the built-in outline that show up to the left but inside of the text editor:

I believe that @ijesonchen is talking about the VScode outline panel that works with all types of files (not just markdown):

Like OP, I would prefer to re-use the built-in Outline panel if possible.

Dup of #5


Shortcut tweaks

#7

[ ] Parse shortcut using commands API to correct symbols (e.g ⌘). Do this because on FireFox, the command API returns the shortcut as "Command+K" instead of "⌘+K"
[ ] Short parsed shortcut on hover over the search field
[ ] Update description and documentation


Recipe update for v0.1.2

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Updating to allow use of PL v1.6.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[Feature request] Allow to pass arguments to browser

As far as I tried with different values of RTV_BROWSER it is not possible to pass arguments when rtv opens a browser for viewing a link e.g. "chromium-browser --new-window".
Would be nice to have more control https://github.com/michael-lazar/rtv/issues/455 goes into the same direction.

Thank you for the input, I sure understand your reasoning for using the package.
I thought this could work as I saw it as example for window, but it didn't on unix
webbrowser.get("/usr/bin/chromium-browser --incognito").open_new("www.google.com"
FileNotFoundError: [Errno 2] No such file or directory

I tried RTV_BROWSER='firefox -P reddit' so that a specific profile that includes the reddit extension could be used, but the arguments broke it.


v2 release date

Hello there!
I've been searching forever for a good scrollbar that works as it should in all kinds of devices! Thanks for this!
Do you have any plans on the release of v2? I can't make the beta version to work :/

Hi!
I'm just waiting for some feedbacks and people around me to test it and have it working successfully before actually releasing it (and maybe write some proper tests as well). So there is no specific date yet.
But what's your issue with the v2 beta?

..the source code is not defined as a usable plugin. Or am I doing something wrong?

Sorry I was away from my laptop for the last 2 weeks.
How do you use the script? Do you import it as a <script> tag or do you import it with a module loader (like in Webpack or Require js environment?).
@scooterlord no on V1 it's not possible.

Hello, thanks for getting back to me.
I tried using it in a

hey @scooterlord check the README there: https://github.com/Grsmto/simplebar/tree/v2.0.0-beta.3#installation
there is documentation on how to use the V2 with script tag. Sorry this is not very clear from the homepage I'll try to clarify.

Well, I just saw the link to the source code and never saw v2 github page. So In other words, you just use the script found online right<EMAIL_ADDRESS>Is there any roadmap about finalizing this? We are about to release our project and have to either stick to v1 or advance to v2. Is is easy to clarify what is v2 missing at this point?

If it's working fine for you, you can you the V2. The only reason it's on "release candidate" state and not release yet is to warn people that they might find bugs and things not working as expected.
As the plugin is used and works for most of the usages, I will release it as stable (but it won't be any major change anyway, so it's fine to use the RC now!).

Cool, thanks for sharing!


Use proc macro to define Config

This PR implements a proc macro for defining Config struct, which replaces the current define_config macro.
The ultimate goal is to define the config like the following:
#[rustfmt_config]
pub struct Config {
    /// A maximum width of each line.
    ///
    /// - Default value: 100
    /// - Possible values: any positive integers
    /// - Stable: Yes
    ///
    /// See also err_on_line_overflow.
    ///
    /// ## Example
    ///
    /// ### Input
    ///
    /// ```rust
    /// // An example code.
    /// ```
    ///
    /// ### 100 (default)
    /// ```rust
    /// // An example code formatted with max_width = 100.
    /// ```
    ///
    /// ### 80 (default)
    /// ```rust
    /// // An example code formatted with max_width = 80.
    /// ```
    #[rustfmt::config(default = 100, stable = "1.0.0")]
    max_width: usize,

    // ... other configs
}

and let the config proc macro generate Configurations.md and test code under test/source/configs/ and tests/target/configs/. This PR is the first step toward the goal.
Other minor benefits include the following:

a better interface for setters
internal config options are now private

Since the Config is part of the public API of rustfmt-as-a-lib, and this PR changes its interface, we should merge this as a part of the preparation for 2.0 release. I am creating a PR at the moment, however, for early feedback.

I will try this again after 2.0 release :man_shrugging:


Odd query results

I'm seeing odd result, like that there is one patent issued in 2012.  Another user in the patentsview forum is reporting no results from a query that used to return results.  https://community.patentsview.org/forum/7/topic/156#comment-186

@mustberuss Thanks for reporting this, we were able to fix this earlier today. Please verify if your queries are working now and let us know if you continue to see incorrect api results.

Looks good, thanks!


Regression: Missing character: There is no ^^@ in font

The following example
\documentclass{article}
\usepackage{fontspec}
\usepackage{unicode-math}
\setmathfont{xits-math.otf}
\setmathfont[range={\rightarrow}]{Latin Modern Math}
\begin{document}
$\left(\right.$
\end{document}

triggers the following error with XeLaTeX:
Missing character: There is no ^^@ in font LatinModernMath-Regular/OT:script=ma

The problem appears for any range specification and for many fonts replacing Latin Modern Math.
The bug appeared after upgrading to Ubuntu 16.04 and still present in unicode-math 2015/09/24 v0.8c. It causes documents which used to compile not to compile anymore (notably with lyx, which stops the compilation when such errors are found).

Thanks, this is obviously a pretty serious bug. I’ll do my best to get a fix out ASAP.

I found a workaround for now, which is to have a working font in the last call to setmathfont, e.g.
\documentclass{article}
\usepackage{fontspec}
\usepackage{unicode-math}
\setmathfont{xits-math.otf}
\setmathfont[range={\rightarrow}]{Latin Modern Math}
\setmathfont[range={\dummysymbol}]{xits-math.otf}
\begin{document}
$\left(\right.$
\end{document}

where \dummysymbol is a symbol I want in XITS anyway

I noticed this has to do with the empty delimiter. If you write $\left.\right.$ then you have the error twice.

This seems much related to http://tex.stackexchange.com/questions/154001/ and the fact that \setmathfont changes the delcode of the period.
The following test file
\documentclass{article}
\usepackage{unicode-math}

\showthe\Udelcodenum`.

\setmathfont{XITS Math}

\showthe\Udelcodenum`.

\setmathfont[range=\int]{Latin Modern Math}

\showthe\Udelcodenum`.

\stop

shows, with XeLaTeX,
> 0.
l.4 \showthe\Udelcodenum`.

?
><PHONE_NUMBER>.
l.8 \showthe\Udelcodenum`.

?
><PHONE_NUMBER>.
l.12 \showthe\Udelcodenum`.

and, with LuaLaTeX,
> 0.
l.4 \showthe\Udelcodenum`.

?
(load luc: /usr/local/texlive/2016/texmf-var/luatex-cache/generic/fonts/otl/xit
s-math.luc)(load luc: /usr/local/texlive/2016/texmf-var/luatex-cache/generic/fo
nts/otl/xits-mathbold.luc)
> 0.
l.8 \showthe\Udelcodenum`.

?
(load luc: /usr/local/texlive/2016/texmf-var/luatex-cache/generic/fonts/otl/lat
inmodern-math.luc)
> 4194304.
l.12 \showthe\Udelcodenum`.

Possible manifestation of the same bug in xelatex + elsarticle + “Asana Math” + \right. = ↯ :
\documentclass{elsarticle}
\usepackage{unicode-math}
\setmathfont[range={\setminus}]{Asana Math}
\begin{document}
\(\left( \right.\)
\end{document}


MIT relicense

Re-licenses the project under the MIT license. Will need permission from all contributors before this can be merged.
@DoctorMacc ✓
@endevrr
@AJ-Ferguson ✓
@rtm516
@D3ATHBRINGER13
@Arktisfox
@bundabrg ✓
@davchoo ✓
@jasoryeh
@OnlyBMan ✓
All contributors will need to agree with this license being put on the project. If you do, please state so below.

I agree.

I agree 😄

I agree

I agree.

agreed

I agree

I agree

I agree

I agree

I agree

I agree 💯

Thanks!


Release v0.3.0-beta1

After we merge #78, I propose we do a new release so projects that use cnab-to-oci can cleanly use the newly added features.
Is there anything else we would want in this release?
I'd very much propose we cut the release sooner rather than later, and just release again if we have other features.

Closed with latest release v0.3.0-beta1 👍


Custom Option Renderer style paramater description

Added style parameter to Custom Option Renderer parameters table. Styles must be applied to the rendered option for correct displaying in the dropdown. Lack of styles leads to the dropdown "shaking" effect during scrolling - options are auto-added and then insta-auto-removed from the dropdown.

Whoops, sorry for missing this until now! Thanks for the PR! 😁


Update subandono.toml

Previous PR:
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-11/subandono.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-12/subandono.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-13/subandono.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-14/subandono.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-15_legacy/subandono.toml
Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present
[x] The email, discord, elements telegram, and twitter fields are present and valid

Successfully validated transactions.toml file for commit hash d1d3e80


[util] Ignore proxies when calling Nova Metadata

What this PR does / why we need it:
Prior to this change, the call to the Nova Metadata service used HTTP proxies as directed by the environment variables HTTP_PROXY, HTTPS_PROXY and NO_PROXY (or the lowercase versions thereof).
However, the call to the Metadata well-known IP must originate from the requesting server directly to be correctly served.
With this change, calls to the Nova Metadata service ignore proxy settings.
Which issue this PR fixes(if applicable):
fixes #2217
Release note:
NONE

/ok-to-test

/test openstack-cloud-csi-manila-sanity-test

/test  openstack-cloud-csi-manila-e2e-test

seems the test fail is valid  ? maybe @pierreprinetti you can check whether we need update ?
if it's not related, I can override this test
 --- FAIL: TestDriver (0.00s)
    sanity_test.go:51: failed to initialize CSI Manila driver: failed to initialize proxied CSI driver: connecting to unix:///fake-fwd-endpoint endpoint failed: failed to build resolver: passthrough: received empty target in Build()

seems the test fail is valid ? maybe @pierreprinetti you can check whether we need update ? if it's not related, I can override this test
 --- FAIL: TestDriver (0.00s)
    sanity_test.go:51: failed to initialize CSI Manila driver: failed to initialize proxied CSI driver: connecting to unix:///fake-fwd-endpoint endpoint failed: failed to build resolver: passthrough: received empty target in Build()

@jichenjc That test hasn't passed since 2nd March: https://prow.k8s.io/job-history/gs/kubernetes-jenkins/pr-logs/directory/openstack-cloud-csi-manila-sanity-test

/override openstack-cloud-csi-manila-sanity-test
/override openstack-cloud-csi-manila-e2e-test
ok, I didn't pay attention (maybe we didn't trigger the manila test mostly) on this
thanks for the reminder, not sure @gman0 you still work on this area, FYI

/approve

@jichenjc thanks, I'm away for a week, but I'll have a look when I get back.


updated well drill cost correlations

Updated well drilling cost correlations to match GETEM correlations based on Sandia Report: https://www.nrel.gov/docs/fy23osti/82771.pdf

@bpulluta thanks for submitting!
First step is to get Actions (automated tests) passing, see test failure https://github.com/NREL/GEOPHIRES-X/actions/runs/8380691581/job/22950491702?pr=165#step:5:1221

Yes, once all 16 correlations are incorporated, I would say yes that is a good idea to switch the default to Intermediate 1 - large diameter - vertical

Hi @bpulluta, just checking in - I'm assuming you're working on addressing the test failures and filing another revision of this PR to fix, but let me know if you would like any further guidance or info in the meantime.

Note: this PR, once approved/merged will probably be sufficient to resolve https://github.com/NREL/GEOPHIRES-X/issues/129 (pending evaluation of whether there are any aspects of that issue that are not addressed in this PR)

ew changes to example results to ensure they are as expected for the cost correlation updates, then update this PR with updated example outputs with passing tests.

@softwareengineerprogrammer, I updated the cost correlations. Previously, I thought the previous geophires implementation was using ideal correlations but actually it was using baseline correlations. That's why we saw the cost decrease when we updated it to the new correlations. All tests are now passing correctly.
Additionally, i added a function to test_csv to help track where in the csv there were differences. Nothing major but it helped me to identify the lines that needed revision.

Awaiting updated example outputs and passing Actions per #165 (review)

@softwareengineerprogrammer @kfbeckers , I updated the cost correlations. Previously, I thought the previous geophires implementation was using ideal correlations but actually it was using baseline correlations. That's why we saw the cost decrease when we updated it to the new correlations. As such, I fixed the implementation to call baseline correlations for the first 4 correlations. All tests are now passing correctly.
Additionally, i added a function to test_csv to help track where in the csv there were differences. Nothing major but it helped me to identify the lines that needed revision.

Note from offline discussion with @kfbeckers: We will file a follow-up issue to change the default correlation to intermediate 1 (no action required on @bpulluta's part)


Add function to throw out old visual before re-rendering int node vis

Closes #311
Removing the visual elements and redrawing is the fastest path to what we want. We should check that we're not missing any data because of a weird enter/exit issue.
Let me know what you think @gotdairyya. We could also try to approach this in a smarter way, using ids and the proper d3 enter exit merge, but it might be more work than it's worth

Not sure why your code doesn't work. There are 2 main issues
(1) it does not update the gridlines:
This is switching from 2 hops to 3 hops

(2) The color scale doesn't reset
This is switching from 3 hops to 2 hop

those edits fixed most of the issues!
two things coming up though:
(1) I'm still not seeing the correct color scale though when I go from 3 hop to 2 hop
(2) in going from a 2 hop query to a 1 hop query (so the int node table disappears) and then back to a 2 hop this happens:

Looks good with 3503454 and fc27a71


Friend online/offline status is not updated

I found that when you add a friend user and click into the profile of that user, the status will always be "online" even though the friend user is logged out.

Updated with cron every 15 minutes, we need to think about making it shorter for release


sudo mode

Any way to have a "sudo" mode to save files with read-only access? (Would only work when the user is in the sudoers file anyway)

This is not supported by the package. The underlying library used for ssh connections, ssh2, seems to support this in exec mode, but remote-edit uses the sftp functionality. Unfortunately I do not have the opportunity to add it, but you're more than welcome to submit a PR.


load_root_certificates()

Hello,
thats not realy an issue but could you provide an example for load_root_certificates()?
BR
Lukas

Hey Lukas, first you should download all root certificates from the "Apple Root Certificates" section on the Apple PKI website. If you store all certificates in an /apple-root-certificates/ folder, your function could look something like this:
/**
 * @return string[]
 * @throws Exception
 */
function loadRootCertificates(): array
{
    $rootCertificates = [];
    $certificatesDir = __DIR__ . "/apple-root-certificates";
    if (($files = scandir($certificatesDir)) !== false) {
        foreach ($files as $file) {
            if (str_ends_with($file, ".cer")
                && ($content = file_get_contents($certificatesDir . "/" . $file)) !== false
            ) {
                $rootCertificates[] = $content;
            }
        }
    }

    if (empty($rootCertificates)) {
        throw new Exception("No Apple root certificates found.");
    }

    return $rootCertificates;
}

Make sure to provide the correct directory containing the certificates, and implement proper error handling.


Introduce ok() as a counterpart of fail()

Consider the following test:
    run sendWithoutMintTest = {
        init.then(send(minter, "bob", 5))
            .fail()
    }

Although this test is expected to fail, we cannot guarantee that it fails only after calling send(...). We have to make sure that init goes through, which may be non-trivial to show, as init is non-deterministic. We have to add ok() that complements fail() in tests.

Or maybe we introduce expect(expectedResult) to avoid proliferation of new operators.
/cc @shonfeder, @thpani, @bugarela, @Kukovec, wdyt?

I don't think I really understand the proposal. Maybe we can discuss tomorrow of you could show an example of how this would be used.

We have to add ok() that complements fail() in tests.

I get the technical challenge here, but wouldn't this be rather verbose?

This is probably subsumed by #1252


Large string truncated without error

In trying to do some performance testing I noted that when trying to build a large string it gets silently truncated to zero bytes:
var txt = 'xy';
for (var i = 0; i < 20; i++) {
    print(i, txt.length);
    txt = txt + txt;
}
print(txt.length);

This should print:
0 2
1 4
2 8
3 16
4 32
5 64
6 128
7 256
8 512
9 1024
10 2048
11 4096
12 8192
13 16384
14 32768
15 65536
16 131072
17 262144
18 524288
19 1048576
2097152
2097152

But prints:
0 2
1 4
2 8
3 16
4 32
5 64
6 128
7 256
8 512
9 1024
10 2048
11 4096
12 8192
13 16384
14 32768
15 0
16 0
17 0
18 0
19 0
0

This is probably caused by an internal limitation on string length? What was confusing and is a bit dangerous is that the result is silently truncated rather than triggering a RangeError.
What seems to happen is an overflow in the length field because the result is not always zero length. For example:
x = 'x'; while (x.length < 65530) x += 'y';
print(x.length);
x += 'dummydummy';
print(x.length);
print(x);

This prints:
65530
4
xyyy

So that the contents are from the beginning of the original string before appending.
Assuming this is a simple 16-bit overflow, the string append processing should check for overflow and cause a RangeError if there would be an overflow. Without the error the behavior is a bit dangerous; in performance testing it caused the test to run but with incorrect inputs, for instance.

Thank you for the report, I will fix this in due course. There was only an assert for this.

Since 1275 is accepted, we can close this bug.

Just to be clear, the 16-bit string length limitation is quite understandable for low memory environments, but the silent length wrapping is dangerous because a script can proceed silently with incorrect data. Typically engines throw e.g. a RangeError when encountering internal limits and I think that would be appropriate here.


Use Go 1.13 in Vagrantfile (testing)

(Testing something here, that's why I create a PR for a 5 year old project...)

Here is a test comment

Another comment


windows server 2016

hi,
i have the problem with dhcp log, i am using the sidecar collector and i dont know where there might be a problem.
i am using patch C:\Windows\Sysnative\dhcp\DhcpSrvLog-Tue.log for testing.
this is the log in nxlog:

my sidecar configuration:

nxlog configuration:

sidecar configuration in graylog:

Any idea of why please ?
Thanks.

Yes, try replacing sysnative by system32 or syswow64 in the path

thanks, it work.
now i have problem with parse these logs, i know it is off topic, but i need parse any value of message to fields in grayling. Is there any options to do that automatic or i must use extractor in graylog to extract value from message ? i mean something like this https://pastebin.com/4pjFmqRG

Well the content pack includes the extractors for graylog. So that should parse everything into nice searchable fields like you want.
Note however that I did not have the opportunity to test on Win2016 so if the DHCP log format changed, it might not parse correctly. If that's the case, you're more than welcome to fork my work and adjust for 2016 :)

Side note, I'll make an 2016 version once I get to it but I can't reasonably provide an ETA on that :(

ok, there is the log from graylog and and I think it's the same as on Win 2012 R2.
and in this case, nothing was parsed, expect ID_Description and time. :(

Can you post a line from the actual DHCP log for comparison?

ok, there is the log with a few others records.

Please post them as plain text so I can compare easily :)
But at first glance there see to be more fields...

31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,VM-STUD01.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,VM-ZAM01.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,9002
31,10/24/17,17:22:14,DNS Update Failed,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,9002
11,10/24/17,18:19:46,Renew,<IP_ADDRESS>,android-1e2400e714d055e9,30A8DBAF97D8,,1477509350,0,,,,0x6468637063642D352E352E36,dhcpcd-5.5.6,,,0x0109696E7465726E616C32,0
24,10/24/17,18:22:14,Database Cleanup Begin,,,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,Windows-Phone,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,Windows-Phone,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,VM-STUD01.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,VM-ZAM01.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,0
30,10/24/17,18:22:14,DNS Update Request,<IP_ADDRESS>,DESKTOP-39HQR3F.acskola.cz,,,0,6,,,,,,,,,0
25,10/24/17,18:22:14,0 leases expired and 0 leases deleted,,,,,0,6,,,,,,,,,0
25,10/24/17,18:22:14,0 leases expired and 0 leases deleted,,,,,0,6,,,,,,,,,0

Thanks mate.
I count 19 fields now when it was at 10 previously. And they seem reorganized :(
I based my extractor based on this doc: https://technet.microsoft.com/en-us/library/dd183591(v=ws.10).aspx?f=255&MSPPError=-2147217396
So I'll have to find the relevant documentation to update the extractor with the new DHCP format... any help is welcomed

I thank you mate, for your interest :).
here i added the fields name from DHCP log.
Microsoft DHCP Service Activity Log
Event ID  Meaning
00	The log was started.
01	The log was stopped.
02	The log was temporarily paused due to low disk space.
10	A new IP address was leased to a client.
11	A lease was renewed by a client.
12	A lease was released by a client.
13	An IP address was found to be in use on the network.
14	A lease request could not be satisfied because the scope's address pool was exhausted.
15	A lease was denied.
16	A lease was deleted.
17	A lease was expired and DNS records for an expired leases have not been deleted.
18	A lease was expired and DNS records were deleted.
20	A BOOTP address was leased to a client.
21	A dynamic BOOTP address was leased to a client.
22	A BOOTP request could not be satisfied because the scope's address pool for BOOTP was exhausted.
23	A BOOTP IP address was deleted after checking to see it was not in use.
24	IP address cleanup operation has began.
25	IP address cleanup statistics.
30	DNS update request to the named DNS server.
31	DNS update failed.
32	DNS update successful.
33	Packet dropped due to NAP policy.
34	DNS update request failed.as the DNS update request queue limit exceeded.
35	DNS update request failed.
36	Packet dropped because the server is in failover standby role or the hash of the client ID does not match.
50+	Codes above 50 are used for Rogue Server Detection information.
QResult: 0: NoQuarantine, 1:Quarantine, 2:Drop Packet, 3:Probation,6:No Quarantine Information ProbationTime:Year-Month-Day Hour:Minute:Second:MilliSecond.
ID,Date,Time,Description,IP Address,Host Name,MAC Address,User Name, TransactionID, QResult,Probationtime, CorrelationID,Dhcid,VendorClass(Hex),VendorClass(ASCII),UserClass(Hex),UserClass(ASCII),RelayAgentInformation,DnsRegError.

We'll be migrating to 2016 fairly soon so I'll end up with the same problem eventually. Might as well deal with it now and if it can help someone while at it, why not :)
I can't provide an ETA but it'll get to it, promised!

that would be perfect if you update the extractor with the new DHCP format. i need this to look for user by MAC address.
thank you mate :)

now i have found it this https://gist.github.com/Eagle6705/3d91b2270bf60b7cff12. i was wondering if i delete IIS lines and create gelf input in graylog it could be work.

As mentioned in the comments:  "Tested on Server 2008, Server 2008 R2"
So no, you'll likely end up with the same result as with my content pack.

it is true, my bad.
all I have done now is that I am split message by "," in graylog. :)

Yes, that should be fairly simple to achieve, even using the CSV parser :)
Problem is assigning a meaning to all those fields

yes, but i need only IP, MAC address, timestamp and there is the issue, because it set up the timestamp to the delivery time and with +1 second poll interval.
do you know how to fix it please? I think it should be on the nxlog side, because i cant rewrite timestamp in graylog. :(
thank you !

Hi,
Sorry if I write something wrong I am new in Graylog, I've try this plugin and it seems ok for me with Windows Server 2016, but it seems that Dashboard is not working. Can you help me with some instructions?

I imported content pack and I am receiving logs

Dashboard is empty

Thank you

Mmm looks like your messages aren't get parsed at all...

If they should look like this:

they are there.
I think that problem is with gl2_source_input? Is it possible to edit this value on imported content pack or I should re-import content pack and dashboard?

The content pack is in JSON format so you should be able to easily modify it


Fix sink-record to get topic name

Motivation
Right now, many times Sink requires topic-name to compute partitioned key if the record doesn't have any key setup. But right now, record.getTopicName() always returns empty as method is not implemented into SinkRecord.
Modifications
implement SinkRecord::getTopicName

rerun java8 tests


Check variable significance

Project Robyn
Describe issue
Is it necessary to first run a Time Series regression to check whether the variables you want to include are having a significant effect on the dependent variable? For example first check if the Time Series of Temperature is significant before including it as a variable in Robyn.
I am wondering if someone has a thought about this or already implemented a regression model to check for significance?
I like hearing from you.
Thank you in advance!

please reopen if necessary.


Remove jeep reverse toggle

To test:

Animations while driving in a jeep.

About some more personal feedback, I'm not too sold on the removal of the rear drive feature, I guess part of that may be due to nostalgia. But I would like to share my point of view.
I understand that the new ability to reverse will be useful to control the Jeep with more freedom and helps you get out when the jeep gets stuck, more quickly and naturally.
However it restricts vision. This new way of moving backwards leaves the camera too close so it's hard to see when the Jeep is approaching an edge, or unable to show where obstacles or enemies may be while driving backwards).
The traditional method of turning the controls, may be more clunky, but it provided a way to drive in such direction with a better camera an a good range of animations to sort that movement. Rear drive was not the most important feature of the Jeep since a gameplay perspective, but I think is one of those unique features that support the veicle identity and differenciate it from the others.
So would suggest, don't replace one with another, but make both available.

Perhaps the new backwards by DOWN key could persist (although I recommend move the camera a bit further so it were possible to see the full jeep on the screen).
And the old method, could be moved to the ROLL key, what could make more sense than the original WALK key.

I think the new method will welcome and mostly used, but if someone still want to use the classic method (for personal taste, nostalgy or necesity in an specific scenario, it could keep using it). Besides, all animations and states are already implemented, so I think would be a shame they get blocked and unused by code.
At least, that's how I see it.


Add pythonpath "." before loading modules

Tracking issue
Why are the changes needed?
Sometimes User defined Dockfile is missing "ENV PYTHONPATH /root", it will cause execute errors.
What changes were proposed in this pull request?
This change will add pythonpath "." before  loading modules
How was this patch tested?
build a docker image with Dockfile missing "ENV PYTHONPATH /root"
Setup process
Screenshots

Check all the applicable boxes

[x] I updated the documentation accordingly.
[x] All new and existing tests passed.
[x] All commits are signed-off.

Related PRs

Docs link

@pingsutw I has removed the pythonpath in Dockerfile.dev.
And It works.

@wild-endeavor I think append "." into sys.path is not going to affect python module  loading. Just to make sure that user defined Dockerfile works without pythonpath=/root

Could we add dest_dir to PYTHONPATH as well?

Yea, that would give a better experience. In that case, I would inject it into PYTHONPATH in Popen:
https://github.com/flyteorg/flytekit/blob/4309c5ea92014f67204d245108e536ed2b82d83e/flytekit/bin/entrypoint.py#L545
I'll say do it in another PR, so we can discuss dest_dir independent of this PR.

I'll say do it in another PR, so we can discuss dest_dir independent of this PR.

I think i will create another PR to discuss dest_dir.
Some of the test errors are due to this error.

yeah let's discuss dest dir in another pr... if we ever move to a different model, like one where another worker maybe written in a different language is first in charge of downloading and extracting the package, it may be weird for that worker to be setting the pythonpath.

@thomasjpfan @wild-endeavor @pingsutw
I think we can add dest_dir instead of adding "."  because if we don't give --destination-dir=/xxx when we executes, it uses the default value "." .

I think we can add dest_dir instead of adding "." because if we don't give --destination-dir=/xxx when we executes, it uses the default value "." .

I think it's possible to have dest_dir be different from the working directory.
For example, one can set dest_dir=/home/flytekit/my_dest and working directory is /root. There could be python modules in working_dir=/root baked into the image and then fast registration copies files to dest_dir=/home/flytekit/my_dest. In this case, adding /home/flytekit/my_dest to sys.path is not enough. Either the image or flytekit needs to still needs to add /root into the PYTHONPATH.

Got it. Thank you. i will create another PR to discuss dest_dir


splitAt does not behave like documented

The documentation of splitAt<T>(index : Int, list : [T]) -> ([T], [T]) in ArrayExt.swift gives this example:
splitAt(4, [1,2,3])     == ([1,2,3],[]) which does not work with the current implementation. This PR tries to fix that.

Nice catch!
I've left some style notes in comments.

Should be all fixed now.

Indeed it is.  Thank you kindly for the pull request!
:boat:


Initialize and UI implementation for Wrap/Unwrap functionalities on Bridge

Summary of changes
Provide a detailed description of proposed changes.

Initialize and UI implementation for Wrap/Unwrap functionalities on Bridge

Proposed area of change
Put an x in the boxes that apply.

[x] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[ ] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #1427

Screen Recording
If possible provide a screen recording of proposed change.
https://github.com/webb-tools/webb-dapp/assets/69841784/f7e5f3f4-d6a7-4f3e-b4eb-28c576f52aa7

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Why does it say "--webbWETH" but when you look at the balance it says infinity? Seems like we're not handling edge cases.

Why does it say "--webbWETH" but when you look at the balance it says infinity? Seems like we're not handling edge cases. If there's no balance it should say "0.00" or "0" in both places.

I just updated the code, you can check the new updated video up there


Macro for reloading drawing sheets

Hi, I would like to request a macro that can reload the drawing sheet

is this maybe a good idea to add to the replace drawing sheet macro as a sub? SolidWorks did not reload the sheet format for me when I changed it.


You can test this PR using the following package version. 11.0.999-cibuild0029431-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

DataType is not needed there, and I don't think DataGrid needs an ItemsRepeater dependency.
I need to check why it fails.

@hez2010 CI is green on master and I can build both DataGrid and ControlCatalog projects there with no build errors.
Can you share more information?

DataType is not needed there, and I don't think DataGrid needs an ItemsRepeater dependency. I need to check why it fails.

You can try dotnet publish to reproduce the build failure.

@hez2010 no luck, I am able to build control catalog with dotnet publish and even AOT without any changes on master

I am closing this PR for now. @hez2010 please let me know if you find any more details.


Improve Changelog when we create a release

We have some code in the release.yml workflow to create an automated release with changelog.  However, the changelog is currently empty, and it would be nice to have it include the list of changes.

Perhaps I should name the tag something other than the branches name...

Or swap to a different action for this task, there are tons of "changelog" actions out there.  Another one might do a better job using our branch-based release flow.

Just tested it on my own repo, I think I was right about just switching the tag from release. I had the same problem then changed it to latest and this is the ensuing result (after the original commit after the switch which released from the last head to the new latest tag so this one is from previous latest to current)

This wasn't enough: https://github.com/DevelopingSpace/starchart/releases/tag/latest
Re-opening.

Seems to have problems in the following method, I'm unsure why still.
https://github.dev/marvinpinto/actions/blob/9ecff5f61cbff8b2d81c3a064cb805931e730fdd/packages/automatic-releases/src/main.ts#L179

I don't think we should spend too much time on this Action, since there are lots of others we could use.  If this won't do what we want, let's use another.

If we use the following https://github.com/ardalanamini/auto-changelog we can create changelogs that look like the following:

but the problem would be that the commit messages require a prefix or else they will not show up.
To be categorized under the changelog they would need to be one of the types defined by us in the .yml file here:
commit-types: |
            breaking: Breaking Changes
            feat: New Features
            fix: Bug Fixes
            revert: Reverts
            perf: Performance Improvements
            refactor: Refactors
            deps: Dependencies
            docs: Documentation Changes
            style: Code Style Changes
            build: Build System
            ci: Continuous Integration
            test: Tests
            chore: Chores
            other: Other Changes

Special commit messages won't work with student developers, in my experience.


Electron menubar not loading file in Production - but working properly in development

Expected Behaviour
The app is supposed to open with the route specified, as it works for other windows with the same configuration.
Actual Behaviour
But the app opens with a white color window, without any route or page showing.
Specifications

Menubar version:  "menubar": "^9.3.0",
Platform: macOS
Electron version: 22.0.2

Other information
Hi guys, I am creating an electron application and I am using the menubar package for electron to create a menubar app.
This is the code I am using to load the menubar route, but once I package the app and open it, the window is not showing the route, it is showing a white screen.
I used the same configuration for creating other windows, and it works properly, only for the menubar it does not properly.
If anyone can let me know what is the error and how I could fix it, it would be really helpful.

  const devServerURL = createURLRoute("http://localhost:3000", "menubar");

  const fileRoute = createFileRoute(
    path.join(__dirname, "../", "index.html"),
    "menubar"
  );

  menubarWindow = menubar({
    index: !app.isPackaged ? devServerURL : fileRoute,
    preloadWindow: true,
    browserWindow: {
      width: 550,
      height: 675,
      show: true,
      resizable: false,
      icon: path.join(__dirname, "Minder-Logo.png"),
      webPreferences: {
        nodeIntegration: false, // Default is false
        contextIsolation: true, // Default is true
        preload: path.join(__dirname, "preload.js"), // Use of preload.jsc
        backgroundThrottling: false,
        devTools: true,
        // preload: path.join(__dirname, 'preload-alt.js')  // Use of preload-alt.js
      },
    },
  });

Hi,
I got the same problem when setting nodeIntegration to false and contextIsolation to true.
Do you find a solution about this ?
Thanks

Have you found a solution to the problem you also encountered

I don't know what your createFileRoute function does but you need to make sure that the file loaded in the index prop is prefixed with file://

indeed, it needs to start with file://, this is how it worked for me:
const mb = menubar({
  index:
    is.dev && process
      ? process.env['ELECTRON_RENDERER_URL']
      : join('file://', __dirname, '../renderer/index.html'),
  preloadWindow: true,
  browserWindow: {
    width: APP_WIDTH,
    height: APP_HEIGHT,
    webPreferences: {
      preload: join(__dirname, '../preload/index.js'),
      sandbox: false,
      nodeIntegration: true
    }
  }
})


test(queryClient): Adjust performance threshholds

The pipeline was found to sometimes exceed the threshold of 500ms. Since I expect the the required time to increase linearly, increase the row count by *1.5 and quadruple the time-threshhold.
Local Tests on my Machine

(red is prior to #6918, yellow is after, all times in ms)

Unfortunately, I'm a bit unsure whether the best approach would be to increase or decrease the threshold & query count here.
Increasing will potentially slow down the pipeline whereas decreasing might lead to false negatives. 500ms felt like a good compromise but apparently wasn't lenient enough. (https://github.com/TanStack/query/pull/6918#issuecomment-1952314381)
@milhamm do you have opinions on this?

how about a test that tests if execution doesn't grow exponentially instead ?

how about a test that tests if execution doesn't grow exponentially instead ?

I fear we would run into similar flakiness, because the cpu-load might not be constant over the course of the test (eg see the variance in my local tests). Also, the tests would become more complex since it would need some kind of regression algorithm.
I would suggest the following plan for now:

Try these new defaults and see if the problem occurs again
If it does, enable retries in vitest like so:test(
    'should set 15k data in less than 2000ms',
    () => {
         // ...
    }
    { retry: 3 },
 )

3. If that still fails, I'll consider an alternative testing approach.

we have retries in vitest enabled

I've created a PR to remove these tests: #6946


Deploy crypto contract

Addresses: https://github.com/onflow/fcl-dev-wallet/issues/136

Crypto contract is required for harness so deploy it
Add an npm command to deploy with npm run dev
Remove duplicate readme

Getting stale cc @justinbarry


Linking doesn't inherit members added after the fact

There are two issues I'd like to discuss here.
The first is that, it seems if you link a subgroup to a supergroup, all the players of the supergroup are added to the subgroup. That's fine, but then (I haven't tested this heavily), once you add a new player to the supergroup, they're not automatically added to the subgroup. Logically, I think a subgroup should either always inherit members or the supergroup (which probably makes the most sense) or never inherit members and you have to add them individually.
Finally, before I was banned, we tried to create a defensive subgroup to protect all the towns in Aleph. Our reason for doing this is that we thought all members of every supergroup would be automatically added to the subgroup. So if the group ACD is a subgroup of Claystone, Heartland, Meridian, and Boundary, we'd have one auto updated giant group where we can all communicate in case of emergencies. It seems that each subgroup can only have 1 supergroup assigned to it, which personally I don't see any reason to restrict it to 1.

Linking shouldn't move any members between groups. I'll doublecheck the code.
Merging removes all members from the source group into the destination group; but linking is not supposed to share that behavior iirc.

Well it doesn't move the members, it just adds them. Initially after linking, both groups have all the members.
Really what I'm getting at is with the way linking works, its hard to imagine a proper use case. In the Meridian group, all members have permission to break blocks, so we have a subgroup called MeridianFactories or something well members no longer have that permission. This way, since they share the same members, we can restrict that permission from the factory group and they're unable to join and break our factories. However even this use case doesn't seem to be properly supported since members aren't automatically added to all subgroups after being initially linked.

@TrentWDB
Hm, a single subgroup sharing many supergroups would be extremely handy, and thinking about it, my suggestion https://github.com/Civcraft/NameLayer/issues/187, kinda requires that functionality, at least for the cooperation use case. I'm not sure how that slipped my mind, considering how long I spent writing the post. That said, multiple supergroups per subgroup would also kinda be sketchy without my suggestion too, otherwise the only thing you could really use it for without worry would be groupchat.
@ProgrammerDan
I think he means that the inherited members of subgroups don't seem to be updating when a player is added/removed from a supergroup. I noticed this too very early on in civtemp but forgot about it. It might update on server restart, but I'm not sure, as the first thing I do after a restart is re-link groups, which is esentially manually updating them.

@TrentWDB been thinking a lot on this. I will probably be shortly gutting sub/super group and largely recoding, we'll see. I may include some form of this functionality as I do that.
My thinking has mostly centered on Permission Flow, and I have two main ideas right now.
First, permission flows from subgroup up to supergroup. E.g. When a group is linked as a subgroup, the members of the supergroup gain new privileges in the subgroup, but the members of the subgroup gain no privileges. In this model, a group can be supergroup to many groups but subgroup to only one.
Thoughts:

Add a new special "permission class" called SubGroup. This defines the maximum permission that flows up to the supergroup. Default would be approximately equivalent to MOD but without permission to invite other players. In this way the subgroup still controls how much a member of the supergroup can actually do.
When resolving the permission of a particular person, the following is done:

Get set of permissions for this person if they are a member of the group. If they are, use those permissions (no traversal).
If not a member, get the set of permissions of that group's SUBGROUP type. This is the "highest" permissible activity set for any members of any supergroup. Now traverse the supergroup chain; at each step in the chain compute the intersection of the current highest permissible activity set and the SUBGROUP activity set.
Also keep and maintain a merge-set of permissions based on individuals' level within the supergroup. E.g. if MEMBER in supergroup 1, but OWNER in highest-supergroup, use the merge of permissions from both levels.
So we do two things: We find the broadest possible set of permissions for the user across the parent-chain (flowing up) and the smallest possible set of permissions for any user across the parent-chain (w/ limit).
Then, we compute the intersection of these two sets. Why do this last? Early intersection would remove later-permissions; so we must compute the limiting set (SUBGROUP perms) and the maximized set (member's own perms) to find the "largest" permission set for the user
Practical example: User isn't a member of subgroup. On supergroup they are MEMBER, but on that's supergroup's supergroup, they are MOD. The "maximized" set is CHEST, DOOR, BLOCK, INVITE. However, the first subgroup has SUBGROUP's permission set as DOOR, INVITE; the first supergroup has SUBGROUP permission of DOOR. Consequently, the "limiting set" is DOOR, so the only permission that the user has is DOOR, even though they are MOD on a supergroup "two levels above".

The second approach doesn't use a SUBGROUP special permission type. Instead, we just use a more traditional "overrides" approach. When checking a user's permission, we use their highest permission set along the Supergroup chain. This is what I think is currently in place, but it's not doing it correctly if I'm understanding folks. Here's how it should work:

Starting at group-of-interaction and for a particular player, find the maximized set of permissions for the player. At each group along the supergroup chain, find the merge of that user's "lower" permissions and the current group's permissions for that user, until the final supergroup is found.
Practical example: User isn't a member of subgroup, but is a member of a middle supergroup and topmost supergroup (supergroup that has no supergroup). Initial permission is empty set {}. First supergroup we find they are MODS, and for that group permissions are "DOOR, CHEST, INVITE, BLOCK". However, in the second supergroup they are also MODS but have permissions "DOOR, BLOCK, INVITE, STATS". We merge the two, meaning the user's final permissions are "DOOR, CHEST, INVITE, BLOCK, STATS" on the group-of-interaction.

If nothing else simply having consistent behavior would solve most problems. What do you think of these approaches? The first is a significant complexity bump for players to conceptualize, although it's not that complex from an ACL perspective.
Concerning multiple supergroups, I can tell you right now I don't intent to implement that. N:M group relationships just get messy within the concept of ACL. By restricting to 1:N, we simply need to better define how ACL filters down to subgroups; once that definition is firm and enforced through out the plugin we'll all be happier.

Wow this is a lot to respond to. This type of permission flow is always pretty complicated and hard to visualize.

First, permission flows from subgroup up to supergroup. E.g. When a group is linked as a subgroup, the members of the supergroup gain new privileges in the subgroup, but the members of the subgroup gain no privileges. In this model, a group can be supergroup to many groups but subgroup to only one.

This is how I was originally thinking it should work. It makes the most sense to me that all members of a supergroup have all permissions of a subgroup. However, given this structure, I think it might be more helpful if a supergroup inherits all permissions of a subgroup without automatically adding all supergroup members as members of the subgroup like it currently does (or should be doing). I think this is what you were suggesting anyways. This way, you could manually add people to a subgroup and they can sort of be half members of your city since their permissions will be restricted to that of the subgroup. Also this way you can divide your town into squads or something where each squad can chat with each other privately. These are both use cases that I've been wanting.
I feel like the current method where their rank is inherited from the supergroup is logical. Basically, if they're an admin in the supergroup, they should most likely have admin abilities in the subgroup. If they're an admin in the subgroup... then they have admin abilities in the subgroup. Maybe this is what you were saying as well, I'm not completely sure.
After re-reading your post I think your method makes the most sense, especially if you want to allow multiple a subgroup to have multiple supergroups. And nothing is preventing you from adding admins as admins to the subgroup as well.
In response to your second post, this was something that I was going to bring up. I would like both of these options... somehow... It's probably too much to ask since it's not actually necessary, but one use we have for subgroups as it stands is our factories are subgrouped to a factory specific group where our members can only use them, not break them. Maybe you can have an inclusive subgroup and exclusive subgroup type thing.... No idea.
Anyways, I think if you have to choose one, your first idea is the best and most logical. Let me know if you have any questions about my responses, I kept dozing in and out of writing this.

Linking was reworked to no longer inherit members, so closing this


Suggest what more can be added in pomodoro spiritual break

https://akshitagupta15june.github.io/Moksh/reward.html

We can add the pause feature with start and reset feature

I am interested to work on this issue under Quine. So please assign me this issue to me.

Hi, I am a GSSOC'23 contributor we can add a ticking clock sound, or other soothing/ calming sounds to the background. The user would have an option to mute the sound if they want to. I would like to work on this task if you like the idea.


Ways MLang could be improved drastically

First up, I want to say that there is an error when I run index.js.
/home/runner/mlang/index.js:44
                console.log(result[0] == null ? "\u001b[1;35mundefined\u001b[0m" : (result[0]?.type == "STRING" ? `\u001b[92m"${result[0].value}"\u001b[0m` : (result[0]?.type == "BOOLEAN" ? "\u001b[1;91m" + result[0]?.value + "\u001b[0m" : result[0].value)));
                                                                                ^

SyntaxError: Unexpected token '.'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at Module._compile (internal/modules/cjs/loader.js:963:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
    at internal/main/run_main_module.js:17:47

Second, there are so many ways that MLang could be improved.
First up, the REPL is very inefficient. Instead of using the official REPL module, it uses Readline. If you want an example on how to use the REPL module, take a look at index.js in my Elephant REPL.
Second, it does not use an efficient way of looking through the arguments, but I have already talked about it here in my pull request.

The only reason you get that error is because you're using an old version of Node. This was intended for use in Node 16, not below (most places like replit use node 12). The ?. syntax isn't supported in older versions of Node.
About the "REPL" module: It doesn't seem official and I see no use. Besides, that file is only a placeholder and will be changed in the future. I have bigger priorities for this language then a repl.

The REPL module is official. You can find its API documentation at https://nodejs.org/api/repl.html

I'm not worried about the REPL. At this point in time, I could hardly have a care in the world for the REPL. I'm focused on the actual language. If you want to contribute, go ahead, but I couldn't be bothered about a REPL.


Add Grpc function to get root bounding cube

Fixes part of #86

@wally-the-cartographer merge

Merge requested by authorized user SirVer. Merge queue now has a length of 1.


Error Message at start up

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
2023-03-15.txt
Loading from autostart results in this error message, asking to report it to github
✔ Expected Behavior
no error!
❌ Actual Behavior
error message,
Other Software
No response

/dup #19871


conn_mgr misc improvements

Miscellaneous improvements to conn_mgr based on feedback over the last few months

If you are tagged for review and a bit confused, it's because this PR renames a header file and a couple KConfig arguments that get used in a decent number of samples. You can focus on just the last commit if that's the case.

The last commit changes a function name and a filename name, seems like this needs to go through the stable API change process (assuming it is part of networking, which is stable)?

The last commit changes a function name and a filename name, seems like this needs to go through the stable API change process (assuming it is part of networking, which is stable)?

Connection manager is still experimental: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/conn_mgr/Kconfig#L5

The last commit changes a function name and a filename name, seems like this needs to go through the stable API change process (assuming it is part of networking, which is stable)?

Connection manager is still experimental: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/net/conn_mgr/Kconfig#L5

Can this be added to the API table? https://docs.zephyrproject.org/latest/develop/api/overview.html

@rlubos I can't use :ref: at the moment because conn_mgr still doesn't have docs, so I am just using an inline literal for now instead


Provide a command to link executables

Brought here by your talk on PyConTW. I am taking your suggestions and found this great project.
Can you add a new command link(add perhaps unlink, too) to link the executables under PythonUp/versions/x.x/bin to PythonUp/bin?

There is a command use that you can use like this
pythonup use 3.6
# Links things in versions/3.6/bin to bin

pythonup use 3.6 3.5
# Links things in versions/3.6/bin and version/3.5/bin to bin
# The former takes precedence if there are name conflicts.

pythonup use --reset
# Removes things in bin

Oh, great-------- 原始邮件 --------主题：Re: [uranusjr/pythonup-macos] Provide a command to link executables (#1)发件人：Tzu-ping Chung 收件人：uranusjr/pythonup-macos 抄送：Frost Ming ,Author There is a command use that you can use like this
pythonup use 3.6
Links things in versions/3.6/bin to bin
pythonup use 3.6 3.5
Links things in versions/3.6/bin and version/3.5/bin to bin
The former takes precedence if there are name conflicts.
pythonup use --reset
Removes things in bin
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/uranusjr/pythonup-macos/issues/1#issuecomment-399732483","url":"https://github.com/uranusjr/pythonup-macos/issues/1#issuecomment-399732483","name":"View Issue"},"description":"View this Issue on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/uranusjr/pythonup-macos","title":"uranusjr/pythonup-macos","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/uranusjr/pythonup-macos"}},"updates":{"snippets":[{"icon":"PERSON","message":"@uranusjr in #1: There is a command use that you can use like this\r\n\r\nbash\r\npythonup use 3.6\r\n# Links things in versions/3.6/bin to bin\r\n\r\npythonup use 3.6 3.5\r\n# Links things in versions/3.6/bin and version/3.5/bin to bin\r\n# The former takes precedence if there are name conflicts.\r\n\r\npythonup use --reset\r\n# Removes things in bin\r\n"}],"action":{"name":"View Issue","url":"https://github.com/uranusjr/pythonup-macos/issues/1#issuecomment-399732483"}}}
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [uranusjr/pythonup-macos] Provide a command to link executables (#1)",
"sections": [
{
"text": "",
"activityTitle": "Tzu-ping Chung",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@uranusjr",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "uranusjr/pythonup-macos",\n"issueId": 1,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueClose",\n"repositoryFullName": "uranusjr/pythonup-macos",\n"issueId": 1\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/uranusjr/pythonup-macos/issues/1#issuecomment-399732483"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 349438727\n}"
}
],
"themeColor": "26292E"
}

Oh, greate


Add wxPython

Adding a language

[x] The code displays "Hello World"
[x] I have updated the readme to include the new language
[x] I have incremented the language count in the readme
[x] I have no association with the language

Link to programming language: https://wxpython.org

Does this print "Hello World" or make a window called "Hello World"?