API: Add support for RBAC Auth.

This uses Kubernetes
TokenReview
and
SubjectAccessReview
APIs to authenticate and authorize incoming user tokens using built-in
cluster RBAC.
While this change adds and tests the support, it does not enable it since
additional changes are needed to watcher to support the new behavior.
User documentation will be added when this behavior is added.
Additional changes:

Moves pkg/watcher/internal/test to pkg/internal/test in order to make
this visible to API integration tests (necessary since gRPC metadata
only works if we are using a real gRPC client).
Fixes issue with Delete methods erroring when returning on nil Empty
responses (caught because of the new gRPC client based tests).
Creates new Service Accounts and Roles for the API Server and Watcher
(previously we were reusing the Pipeline Controller SA).

Partially addresses #58

/hold
Service account separate turned out to be a little more involved - split out in https://github.com/tektoncd/results/pull/61
(this is still ready for review for the auth impl though)

/hold
Service account separate turned out to be a little more involved - split out in https://github.com/tektoncd/results/pull/61
(this is still ready for review for the auth impl though)

/hold cancel

/hold cancel

/lgtm

/lgtm


Sending commands based on only the object name is confusing and error-prone

Its more of a design descicion in the Nextion firmware than the lib here, but still ...
It took me quite some time to realize that all command regarding components on te nextion are send using the component ObjName as the identifying element, while th object name can easily appear on multiple pages. This means that I have to check and control on the arduino which components I can reach and which I don;t.
It would be a major improvement if identifiation of componts would be by PageId and ObjId .. (i would think they are made for this).
It does mean that the GUI editor for Nextion should keep its hands off of the PageIds and ObjIds of exisiting component and not start renumbering them while addng or moving components around

#7
I know this is a duplicate of this closed issue  .. but still no solution


phpsdk_deps update  exception on Windows x64 (empty user agent)

TLDR: phpsdk_deps --update ...  step throws exceptions on Win 8.1 x64
I use Windows 8.1 pro x64. Installed Visual-Studio-Community-2017 (v15.7.3) and Git (2.17) from git-scm.com. I also get the same symptom on Windows 7.1 pro. Both are physical machines.
PHP is not installed. I want to build PHP-7.2.6 because I want to compile a non-core extension (build DLL) for php-7.2.6.
I have a problem with the instructions shown in the 'Basic usage example' section of the readme, specifically with the step:
"phpsdk_deps --update --branch master, use phpsdk_deps --update --branch X.Y for a non master branch"
That phpsdk_deps step fails with this message:
$ phpsdk_deps --update --branch master

 Fatal error: Uncaught SDK\Exception: Failed to fetch supported branches in C:\php-sdk\lib\php\libsdk\SDK\Config.php:154
 Stack trace:
 #0 C:\php-sdk\lib\php\libsdk\SDK\Config.php(164): SDK\Config::getKnownBranches()
 #1 C:\php-sdk\bin\phpsdk_deps.php(105): SDK\Config::setCurrentBranchName('master')
 #2 {main}
   thrown in C:\php-sdk\lib\php\libsdk\SDK\Config.php on line 154

Before I got the above problem, I ran these commands, successfully:
git clone https://github.com/Microsoft/php-sdk-binary-tools.git c:\php-sdk
cd C:\php-sdk
git checkout php-sdk-2.0.12
phpsdk-vc15-x64.bat
phpsdk_buildtree phpmaster
git clone -b PHP-7.2.6 https://github.com/php/php-src.git
In Config.php line 130 $tmp = $fetcher->getByUri(self::$depsBaseUri . "/series/");
is returning in $tmp a message complaining about empty user agent. An earlier issue mentioned this. I'm using what I think is the latest sdk.  phpsdk_version reports 2.0.12

Thanks for the report. 2.0.12 is not the latest, it's 2.1.5 at this time. This issue was fixed back in winter, AFAIR.
Thank.

FileOps.php, line 133
curl_setopt($ch, CURLOPT_USERAGENT,'Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:<IP_ADDRESS>) Gecko/20080311 Firefox/<IP_ADDRESS>');
works for me.


add missing german translations. all of them. (#4080)

Co-authored-by: Katja Süss<EMAIL_ADDRESS>

@erral there was no removals on this one. So all good.


HPC Save Writes Multiple Checkpoints

🐛 Bug
Currently the hpc_save function (https://github.com/PyTorchLightning/pytorch-lightning/blob/6e8721e7ae881cc54ec1f6580d85eb95507861e5/pytorch_lightning/trainer/connectors/checkpoint_connector.py#L201) doesn't respect the behavior of save_last meaning that only one checkpoint should be written. This means that every time the job is preempted (on slurm) it writes another checkpoint which caused me to run out of disk space recently.
To Reproduce
This can't exactly be reproduced using the requested BoringModel method, it requires a cluster (I know for sure slurm will repro this), set a short timeout, and run. Each time the limit is reached there will be a new checkpoint written. If this should be controlled separately from the existing save_last flag, then another flag should be introduced. This should be an easy fix and I'd be happy to PR it if the owners are agreeable to the solution.
Expected behavior
Only one checkpoint is written.
Environment

CUDA:
- GPU:
- available:         False
- version:           10.2
Packages:
- numpy:             1.20.1
- pyTorch_debug:     False
- pyTorch_version:   1.7.1
- pytorch-lightning: 1.2.0
- tqdm:              4.57.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.8.1
- version:           #1 SMP Thu Jan 21 16:15:07 EST 2021

Hey @Queuecumber,
We are in the process of setting up a SLURM cluster to help with Slurm related bugs.
Any chance you would contribute a fix for this issue in the meanwhile ?
Best,
T.C

It seems to me like all this logic should be integrated into ModelCheckpoint, so when training is over and on_train_end is called ModelCheckpoint is the one who does all this.
Like, why do we hpc_save manually here? https://github.com/PyTorchLightning/pytorch-lightning/blob/6e8721e7ae881cc54ec1f6580d85eb95507861e5/pytorch_lightning/trainer/connectors/slurm_connector.py#L35

LMK what you guys decide the best approach is and I can make the PR, are you sure that the model gets checkpointed using on_train_end if the job is preempted?

We have this code to handle a KeyboardInterrupt and call on_train_end to save the state before shutting down
https://github.com/PyTorchLightning/pytorch-lightning/blob/925f082572500a8c3b97e1e8c9d614f6de73b232/pytorch_lightning/trainer/trainer.py#L631-L641
on_train_end calls:
https://github.com/PyTorchLightning/pytorch-lightning/blob/925f082572500a8c3b97e1e8c9d614f6de73b232/pytorch_lightning/trainer/training_loop.py#L132
and that checks ModelCheckpoint:
https://github.com/PyTorchLightning/pytorch-lightning/blob/925f082572500a8c3b97e1e8c9d614f6de73b232/pytorch_lightning/trainer/training_loop.py#L151-L162
Ideally, the SLURMConnector would somehow register handlers for SLURM signals to the loop so the shutting down logic is the same.
@awaelchli, you had a PR to refactor signal handling. Was it to do something like this? What happened to it?

We could just have SLURMConnector call train_loop.on_train_end() for a simple solution

Did you guys have any further thoughts on how to handle this? There's a conference deadline next week after than I plan to start on PRs for the two issues I filed

Resolving this will be a bit complex, additionally we need to first setup SLURM testing so we can make sure everything is OK. I don't expect you to work on this for this reason.
We could just loop through the existing callbacks setup and see if any has save_last=True but this would break some people's workflow as they expect this SLURM checkpoint to always be created.
I'm re-labeling this as a feature as it is working as expected given the current design.

What about something to disable the checkpointing and requeuing? I actually want that anyway because I'm using a slurm wrapper that takes care of all that already. Also I do have access to several slurm clusters I can test this on so I do have the resources to develop this as long as we can agree on the solution you guys want.

If you want to disable it quick-n-dirty you could either monkey-patch the SlurmConnector.register_slurm_signal_handlers or resetting the signal hooks
https://github.com/PyTorchLightning/pytorch-lightning/blob/6e8721e7ae881cc54ec1f6580d85eb95507861e5/pytorch_lightning/trainer/connectors/slurm_connector.py#L26-L27

That works temporarily but I was proposing a PR for a supported way of disabling it

Resolving this will be a bit complex, additionally we need to first setup SLURM testing so we can make sure everything is OK. I don't expect you to work on this for this reason.
We could just loop through the existing callbacks and see if any has save_last=True but this would break some people's workflow as they expect this SLURM checkpoint to always be created.
I'm re-labeling this as a feature as it is working as expected given the current design.

I agree this is pretty complicated, there are some complex interactions between the CheckpointConnector and the ModelCheckpoint callback which would need to be worked out, it probably makes sense to keep this logic in one place instead.

Not stale


specto cannot update, including URL resolver, kodi 17 beta 5

specto cannot update, including URL resolver, kodi 17 beta 5
any suggestions?

This is Kodi bug, i think.
On  https://kodi.tv/kodi-v17-krypton-beta-5/
[..]
Known problems
• Upgrading the skin you are currently using fails on Windows.
• Upgrading any add-on currently fails on Windows.
• Possible video freezes on Raspberry Pi and Android.

I am using mac if that makes any difference. Other add-ons are updating just fine.

I can confirm that all other add-ons update fine on Kodi 17 Beta 5 on Windows, Mac, and Android except the following 2 add-ons that both fail trying to update online or manually. I also get server timeouts now trying to import my Trakt TV Watchlist and no new episodes of the previously added shows in my library have appeared.
Mrknow URLResolver 2<IP_ADDRESS>
Specto-...Fork 2<IP_ADDRESS>


Running Locally with --rpccookiefile

Hey just trying to get this running locally and want to ask is --rpcuser --rpcpassword a requirement? All my setups specifiy --rpccookiefile
I could change for testing but just wanted to ask about this before I change any settings on systemd / electrs etc.
I see from the help there is no option
      --datadir <DIR>              Bitcoin data directory path, saved
      --rpcconnect <IP>            Bitcoin RPC ip, default: localhost, saved
      --rpcport <PORT>             Bitcoin RPC port, default: 8332, saved
      --rpcuser <USERNAME>         Bitcoin RPC username, saved
      --rpcpassword <PASSWORD>     Bitcoin RPC password, saved
      --delay <SECONDS>            Delay between runs, default: 0, saved
      --dry-run <BOOL>             Start a dry run, default: false, not saved [possible values: true, false]
      --record-ram-usage <BOOL>    Record ram usage, default: false, not saved [possible values: true, false]
      --recompute-computed <BOOL>  Recompute all computed datasets, default: false, not saved [possible values: true, false]
      --first-defragment <BOOL>    Start the program by defragmenting all databases to reduce their footprint, default: false, not saved [possible values: true, false]
  -h, --help                       Print help
  -V, --version

Might this be implimented in future?

Hi !
Just fixed it for you 👍, you can specify it now but it will auto find it if the path is --datadir/.cookie

Thanks, that is now working. At the moment, the parser is doing it's thing.
If I have any questions, is it best to open an issue or is there any place for discussion?
Appreciate you doing that. I'm going to close this.


Add .NET Core Sample

It would be much helpful and useful if you add .NET Core sample within this documentation as it is mainstream technology nowadays for enterprise application development

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 61d73b50-b26b-3e9e-e8c2-37669424793c
Version Independent ID: ebde85d2-1f0a-afa8-0597-d084c2b64870
Content: Deploy to a Linux VM - Azure Pipelines
Content Source: docs/pipelines/ecosystems/deploy-linux-vm.md
Product: devops
Technology: devops-cicd-ecosystems
GitHub Login: @N-Usha
Microsoft Alias: ushan

Thanks! I've added this in a different space.

@RiadKatby To follow-up, I'd recommend following the Windows VM quickstart and then following the same steps but selecting Windows for the virtual machine resource.


Specify Error Prone version.

Instead of wildcarding.
Sorry about that. I did it in LeakCanary, too.

We need to resolve the new failures as well with this PR

yeah, sorry, will do in a new commit/patch. one sec.


Document example files

The example files for launching experiment in the examples should be documented fully.

@scottsievert: Scott, how is this going? Can we close this issue?

I believe it's done but it probably needs review. A sample is in the examples directory (every example file has been commented).
https://github.com/nextml/NEXT/blob/dev/examples/cartoon_cardinal/experiment_cardinal.py

Looks good.


Mypy plugin

Change Summary
Add a mypy plugin with a number of features; see the added docs for more details.
Other notable improvements for pydantic developers:

The mypy tests have been reorganized to be easier to work on

Eventually, it will probably make sense to better organize individual tests, but
this seemed like the most practical solution for now

Thanks to the reorganization, the mypy tests now run after all others (I think this is preferable)
The mypy tests and codebase check now run much faster on repeated execution because they no longer invalidate each other's caches

The tests now make use of separate cache directories by plugin settings, all contained as subfolders of the previously used .mypy_cache folder.
Because they are subfolders of .mypy_cache, the same make clean works to clean up all of the cache directories

Related issue number
Closes #460
Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] change description file added to changes/,
see changes/README.md for details
on the format

In principle this looks great, but as you say let's defer it until 1.1 - it shouldn't cause any backwards incompatibility issues I assume?

You can always just turn the plugin off (by not incluiding it in mypy.ini/setup.cfg), so it is strictly opt-in. So I don't think there are backwards compatibility issues.

For what it's worth, if anyone wants this in the interim, you can just copy/paste the contents of the pydantic/mypy.py file into your own project and point mypy at it, and it should just work. (Doesn't require any changes to pydantic.)
It would be nice to be able to distribute it with the library though.

After some painful edge case exploration and mypy debugging, I've gotten the test coverage of the plugin itself up to 100% (modulo two minor no-covers related to import edge cases, where I've just copied the logic from the official dataclasses plugin).
Still needs:

[ ] docs
[ ] support for type checking the construct method

I think it might make sense to (eventually) support other config settings and/or finer-grained strictness controls, but I'm more inclined to just release this and see what features people actually request.

@samuelcolvin I think this is basically done! I added docs and updated the description at the top with some more detail.
The docs cover basically everything the plugin adds.
The tests are a little haphazard organized, basically just dumped into the big plugin-success.py and plugin-fail.py files; we can fix that eventually but I personally think this is good enough for a first release. I'm happy to put more effort into cleaning it up though if you want that to happen before a merge.
Special note: these changes should make running the mypy tests and mypy check faster on subsequent runs due to better caching (even though there are now more tests).

Also, @koxudaxi if you have a chance to review I think you might find it interesting and/or have some inputs.

I'm away from my computer but will look on Tuesday if not before.
Are we caching the mypy cache directory (sorry can't see on my phone)?

@samuelcolvin Sounds good, no rush.
I’m not sure what you mean by “are we caching the mypy cache directory” — could you clarify?
I put the test caches inside the default one (used during checks). The structure of the caching directories is such that this doesn’t cause any conflicts.

Sorry, I mean there's a .mypy_cache or simpler directory. Have we added that to the list of cached directories in Travis?

Yeah, the directory is .mypy_cache. I didn’t know you could cache directories in Travis. Actually I don’t know much about Travis in general. I’m not at my computer now either but can try to figure it out when I can if you’d like.

Well, might make things faster. Don't worry if it's a pain.

@samuelcolvin I think I've incorporated all your feedback on the non-docs files, or at least responded above where I had concerns.
I renamed the plugin config settings so they could all be False by default (but basically have the same meaning).
I'm going to wait until we reach consensus on the non-docs stuff before updating the docs.

It looks like this is what django-stubs uses.
Here's a blog article discussing: https://sobolevn.me/2019/08/testing-mypy-types
It looks like a massive improvement over what is used in this PR, but 1) I'm not sure how fast it would run, and 2) I'm not sure how maintained it is.

Looking good.
Apart from the docs and a few pretty trivial potential fixes, I think this is more or less ready.

I tried using dataclasses but it caused the cythonized builds to fail due to the __init__ method not getting created properly.
Maybe there's a workaround, but since it's not really giving much benefit here I figured hand-writing the init function was fine.

Also, I believe I have now addressed all your concerns with regard to the docs.

this is awesome.
Thank you very much!


Can i use Klaviyo.INSTANCE.setPushToken(token) in another process

Checklist

[X] I have read the contributing guidelines
[ ] I have determined whether this bug is also reproducible in a vanilla Android project
[ ] If possible, I've reproduced the issue using the master branch or latest release of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or pull request.

Description
My FirebaseMessagingService is in another porocess,an Klaviyo ini in main process.
how can i use Klaviyo.INSTANCE.setPushToken(token) in FirebaseMessagingService onNewToken

Expected behavior
always

Actual behavior
always

Steps to reproduce
always

The Klaviyo Android SDK version information
3.0.0
Device Information
No response
Android Studio Version
No response
Android API Level
No response

Thanks for filing this! While it's definitely recommended that you use the application to encapsulate both your Klaviyo instance and Firebase service, there are some workarounds for messaging between the two. I'd recommend looking into using shared preferences to hold the firebase token, or creating a broadcast between the services that handles the token setting. You definitely don't want to set the token in the separate process, it would be better to set some shared store with the value and have your main process read that. If you do try and set from a different process and get some strange behavior, please feel free to share and I can look into whats going on further with a dummy application or logs.


⚠ ClamAV has degraded performance

In 76a01d5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in 882a75d.


nf_conntrack_max: permission denied after upgrade to Rancher 2.5.9

Rancher Server Setup

Rancher version: 2.5.9
Installation option (Docker install/Helm Chart): Docker

Describe the bug

Some days ago, I've upgraded from Rancher 2.5.8 to Rancher 2.5.9. Rancher 2.5.8 worked for months without any issue and also 2.5.9 started without having any issue. But today, I wanted to access my cluster and now, Rancher is down. When I start Rancher, the logs says:
...
I0723 07:37:57.557877      25 leaderelection.go:243] attempting to acquire leader lease  kube-system/cloud-controller-manager...
E0723 07:37:57.608854      25 leaderelection.go:325] error retrieving resource lock kube-system/cloud-controller-manager: endpoints "cloud-controller-manager" is forbidden: User "cloud-controller-manager" cannot get resource "endpoints" in API group "" in the namespace "kube-system"
I0723 07:37:57.615584      25 cache.go:39] Caches are synced for APIServiceRegistrationController controller
I0723 07:37:57.624867      25 cache.go:39] Caches are synced for autoregister controller
E0723 07:37:57.625805      25 controller.go:156] Unable to remove old endpoints from kubernetes service: no master IPs were listed in storage, refusing to erase all endpoints for the kubernetes service
I0723 07:37:57.712453      25 cache.go:39] Caches are synced for AvailableConditionController controller
I0723 07:37:57.714791      25 shared_informer.go:247] Caches are synced for cluster_authentication_trust_controller
I0723 07:37:57.743705      25 shared_informer.go:247] Caches are synced for crd-autoregister
I0723 07:37:57.762793      25 trace.go:205] Trace[632932033]: "GuaranteedUpdate etcd3" type:*core.Node (23-Jul-2021 07:37:56.995) (total time: 767ms):
Trace[632932033]: ---"Transaction prepared" 733ms (07:37:00.729)
Trace[632932033]: [767.613912ms] [767.613912ms] END
I0723 07:37:57.763741      25 trace.go:205] Trace[351955239]: "Patch" url:/api/v1/nodes/local-node/status,user-agent:k3s/v1.19.8+k3s1 (linux/amd64) kubernetes/95fc76b,client:<IP_ADDRESS> (23-Jul-2021 07:37:56.994) (total time: 768ms):
Trace[351955239]: ---"Object stored in database" 762ms (07:37:00.763)
Trace[351955239]: [768.82707ms] [768.82707ms] END
I0723 07:37:57.780705      25 flannel.go:78] Wrote subnet file to /run/flannel/subnet.env
I0723 07:37:57.780823      25 flannel.go:82] Running backend.
I0723 07:37:57.780901      25 vxlan_network.go:60] watching for new subnet leases
I0723 07:37:57.782909      25 iptables.go:145] Some iptables rules are missing; deleting and recreating rules
I0723 07:37:57.783009      25 iptables.go:167] Deleting iptables rule: -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/16 -j RETURN
I0723 07:37:57.784680      25 iptables.go:167] Deleting iptables rule: -s <IP_ADDRESS>/16 ! -d <IP_ADDRESS>/4 -j MASQUERADE --random-fully
I0723 07:37:57.785648      25 iptables.go:167] Deleting iptables rule: ! -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/24 -j RETURN
I0723 07:37:57.786567      25 iptables.go:167] Deleting iptables rule: ! -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/16 -j MASQUERADE --random-fully
I0723 07:37:57.788534      25 iptables.go:155] Adding iptables rule: -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/16 -j RETURN
I0723 07:37:57.792812      25 iptables.go:155] Adding iptables rule: -s <IP_ADDRESS>/16 ! -d <IP_ADDRESS>/4 -j MASQUERADE --random-fully
I0723 07:37:57.806556      25 iptables.go:155] Adding iptables rule: ! -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/24 -j RETURN
I0723 07:37:57.807100      25 iptables.go:145] Some iptables rules are missing; deleting and recreating rules
I0723 07:37:57.807374      25 iptables.go:167] Deleting iptables rule: -s <IP_ADDRESS>/16 -j ACCEPT
I0723 07:37:57.817401      25 iptables.go:167] Deleting iptables rule: -d <IP_ADDRESS>/16 -j ACCEPT
I0723 07:37:57.818741      25 iptables.go:155] Adding iptables rule: -s <IP_ADDRESS>/16 -j ACCEPT
I0723 07:37:57.820612      25 iptables.go:155] Adding iptables rule: -d <IP_ADDRESS>/16 -j ACCEPT
I0723 07:37:57.822954      25 iptables.go:155] Adding iptables rule: ! -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/16 -j MASQUERADE --random-fully
I0723 07:37:58.516012      25 controller.go:132] OpenAPI AggregationController: action for item : Nothing (removed from the queue).
I0723 07:37:58.516145      25 controller.go:132] OpenAPI AggregationController: action for item k8s_internal_local_delegation_chain_0000000000: Nothing (removed from the queue).
I0723 07:37:58.542033      25 storage_scheduling.go:143] all system priority classes are created successfully or already exist.
I0723 07:37:59.129987      25 node.go:136] Successfully retrieved node IP: <IP_ADDRESS>
I0723 07:37:59.130174      25 server_others.go:143] kube-proxy node IP is an IPv4 address (<IP_ADDRESS>), assume IPv4 operation
I0723 07:37:59.132593      25 server_others.go:186] Using iptables Proxier.
I0723 07:37:59.133930      25 server.go:650] Version: v1.19.8+k3s1
I0723 07:37:59.135211      25 conntrack.go:103] Set sysctl 'net/netfilter/nf_conntrack_max' to 131072
F0723 07:37:59.135290      25 server.go:495] open /proc/sys/net/netfilter/nf_conntrack_max: permission denied
2021/07/23 07:37:59 [FATAL] k3s exited with: exit status 1

I'm starting the container using the following command:

docker run --restart=unless-stopped -p 80:80 -p 443:443 -v /root/rancher:/var/lib/rancher --privileged rancher/rancher:2.5.9 --acme-domain something-domain.com

To Reproduce
On my side, I can restart the Rancher docker container and the error appears everytime. But I'm not sure if it's a problem of Rancher 2.5.9 or how I can solve this issue.
Result
Expected Result
Rancher running as expected :-)

Merging into https://github.com/rancher/rancher/issues/33360


Path to dnscrypt-resolvers.csv changes when the version number increments.

The last dnscrypt upgrade through homebrew changed the path to the dnscrypt-resolvers.csv file from /usr/local/Cellar/dnscrypt-proxy/1.6.0/share/dnscrypt-proxy/dnscrypt-resolvers.csv to /usr/local/Cellar/dnscrypt-proxy/1.6.0_1/share/dnscrypt-proxy/dnscrypt-resolvers.csv
Is there any way to automatically correct those changes, so it doesn't break the homebrew.mxcl.dnscrypt-proxy.plist on each upgrade?

I experienced the same issue, and fixed it by changing the --resolver-name line in /Library/LaunchDaemons/homebrew.mxcl.dnscrypt-proxy.plist to <string>--resolvers-list=/usr/local/share/dnscrypt-proxy/dnscrypt-resolvers.csv</string>.
/usr/local/share/dnscrypt should be symlinked to /usr/local/Cellar/dnscrypt-proxy/1.6.0_3/share/dnscrypt-proxy, or whatever the current version is.


GPII-3860: scavenged fixes from #402

lgtm

I see that the code applies the changes to your dev cluster, is this a WIP PR?

This is ready for review again, btw.
Deploy plan will be in ticket (in progress).

LGTM!

LGTM


chore: Update to Next14

Updates this repo to use Nextjs 14
Removes workaround for prisma as it is fixed now - https://github.com/prisma/prisma/issues/16117
Adds workaround for Image component in mdx files for next after 13.5 as suggested in https://github.com/hashicorp/next-mdx-remote/issues/405
Bumps all dependancies except a few content layer /mdx ones
Replaces deprecated domains in next.config.js and with remote domains
Moves themeColour in viewport rather than metadata ( change in next14)

Closes: https://github.com/shadcn-ui/taxonomy/issues/251 https://github.com/shadcn-ui/taxonomy/issues/192

Thank you


Fix helm test for prosody connection test

According to https://prosody.im/doc/setting_up_bosh, the BOSH insecure endpoint should be something like http://example.com:5280/http-bind.

Hello @hhewei!
Sorry for late reply. Thank you for your contribution, I'll merge this PR right now.


reduce bounding Box memory footprint

this PR is a proposal for reducing the memory size of a BoundingBox by removing the two Array of 8 Vector3 from the members vectors and vectorsWorld.
For backward compatibility, those are still available as getter that return a new array at each call. Those vectors can also be computed on an already existing array.
As solidParticlSystem relied on them, the code has been updated accordingly, and the API of BoundingSphere and BoundingeBow slightly updated so that an optional worldMatric can be passed to the reconstruc method, which is the one that must be called in case of any change of a local value (center, radius, center, ...). If my understanding is correct, _update should be used only if the world matrix changed.
Note that this PR might be a bit risky, but I think it is really worth it. BoundingBox memory footprint would  drop from 3 array + 27 Vector3 to 1 array + 11 Vector3

@deltakosh I'll close it for now and keep it for me as it would require true performance figures to figure out whether there is a real perf hit or not

no problemo, c'est normal entre anciens montrougiens ;)

The Monrouge Mafia!!
Pour la blague j'habite maintenant pres d'une ville nommee Redmond :)


Exercise ICS20 timeouts end-to-end against a Cosmos-SDK chain.

Is your feature request related to a problem? Please describe.
In advance of getting automatic testing as part of ongoing InterchainTest integration, we should manually exercise the end-to-end flow of timeouts against a Cosmos-SDK chain.
This will require a counterparty chain with ICS23 non-inclusion proof support (#3045) and using our Hermes fork with the config option added in https://github.com/penumbra-zone/hermes/pull/3  to create a client with the correct proof spec.
Currently, this is blocked by:

[x] #3082

To test this, we can make a packet with near-immediate timeouts. This test should be manual, so we can observe behavior and document results.

Achieved! @avahowell managed to get a local testing setup together to validate the behavior. She's going to write up those testing steps in this ticket, then close out.

I was able to successfully perform a timeout of a packet from a local osmosis testnet (built using the osmosis make localnet-* tooling) to a penumbra testnet.
The steps were as follows,

perform an ibc handshake using hermes
perform a transfer using rly, with a timeout height of +5 blocks
wait 5 blocks, then run hermes start
observe that hermes constructs a timeout, using the nonexistence proof from the penumbra chain, commits it to the osmosis chain,
try another transfer and verify that it still succeeds after timeout

A small change to our query api was required. When that's merged, I'll close this issue
https://github.com/penumbra-zone/penumbra/pull/3163

Closing as completed.

Let's try this again, but this time using public testnets, not local nets. This is recently made possible by the dep changes in Osmosis testnet.

This has been demonstrated, per @avahowell


A user should confirm that they want to cancel or abort the creation process and lose their selections

Is your feature request related to a problem? Please describe.
The creation experience in Composer should notify the user that if they abort or cancel the creation modal that their selections will be lost.
Describe the solution you'd like
On the template selection screen there should be a link to GitHub that properly tags a request from a user and allow them to be able to request a template that they would like to have added by Microsoft.
User has entered data...

User clicks on the X or Cancel

Additional context
Creation Functional Spec 1.17

Determined that was not needed.  Not enough information to be concerned about loosing this content.  Agreed and decided by team.


Search view should be closed on back key

Open the demo app
Tap on the search view to open it
Tap back - keyboard is closed
Tap back again - the application is closed

Expected behaviour:
tapping back when the search view is opened should close the search view, not the application

You can close it by listening for onBackPressed() events.
The following code checks if the search bar is focused and closes it if necessary.
if (mSearchFSV.isSearchBarFocused()) {
    mSearchFSV.clearSearchFocus();
    return;
}


[Enhancement]: restructure Bootstrap and use unity_rs fully

Describe the new feature or enhancement
Currently, Bootstrap is a fair bit messy, and goes deep into unity_rs for grabbing mono/il2cpp methods
Restructure Bootstrap to be more readable, and finish the abstracted implementation of unity_rs

implemented in alpha-development


fix(people/search) : 'or email' placeholder removed #2753

fix for #2753

@JoseChavez98, looks good. Would you mind globalizing it, though? E.g. using t(:search_by_name) instead and adding the search_by_name key to en.yml? If that's too much, no worries, we can just merge this in and fix it later.

yes, I can do that.
I'm working on it RN

Thanks, @JoseChavez98!


KHP order finder halts with "Invalid object identifier"

I executed examples/circuit_simulation.py on my laptop (WSL: Ubuntu 18.04) and the workstation machine (Ubuntu 20.04).  The code worked on my laptop, but the program prints Invalid Object Identifier and deadlocks.
This error seems to be related to multiprocessing, and the only place I could find this message was line 65 of https://github.com/kahypar/kahypar/blob/a05d75b9fd37880ab2853fad8e7ff54acd265247/kahypar/meta/abstract_factory.h .
The program worked as wished after I changed the order_finder_name in khp_params.json to default.  Besides the contraction costs were lower and the results had higher precision compared to the answers I got from my laptop.  I wonder what makes the difference.

Hello ChienKaiMa,
This indeed seems like a problem within KaHyPar. One possible problem I could think of is that the KaHyPar versions installed on different Ubuntu releases could be different. As I have not personally encountered this problem I am not sure what to make of it. Could you please provide instructions on how this error could be reproduced on a Ubuntu 20.04 machine? If it could be identified as a problem within KaHyPar, I would suggest you to directly contact KaHyPar.
From my understanding (please correct me if I am wrong), you are comparing the contraction on your laptop with a KaHyPar order finder, and another one on your workstation with a default order finder. The contraction cost and contraction precision depend on multiple factors. Generally speaking, the lower the contraction cost, the fewer number of floating point operations will be issued, resulting in a lower truncation error. Hypergraph-decomposition-based contraction order finding highly depends on hyper parameters one chooses in KaHyPar, and does not always guarantee a superior performance compared to alternative approaches. It could be the case that  the default order finding algorithm performed better in this specific case.
Hope that this answers your problem.
Cupjin

I didn't intend to compare two different order finders on two different machines, but that just happened.  You're right: the Kahypar version on two machines are actually different!
On my laptop:
Name: kahypar
Version: 1.1.6
Summary: Python Inferface for the Karlsruhe Hypergraph Partitioning Framework (KaHyPar)
Home-page: https://www.kahypar.org
Author: Sebastian Schlag
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /home/koova18/miniconda3/lib/python3.8/site-packages
Requires:
Required-by: acqdp

On the workstation:
Name: kahypar
Version: 1.1.7
Summary: Python Inferface for the Karlsruhe Hypergraph Partitioning Framework (KaHyPar)
Home-page: https://www.kahypar.org
Author: Sebastian Schlag
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /home/b07901108/quantum-benchmarks/bin/conda/lib/python3.9/site-packages
Requires:
Required-by: acqdp

I also tested on the workstation with Python 3.8 instead of 3.9, and the issue persisted.
Name: kahypar
Version: 1.1.7
Summary: Python Inferface for the Karlsruhe Hypergraph Partitioning Framework (KaHyPar)
Home-page: https://www.kahypar.org
Author: Sebastian Schlag
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /home/b07901108/.local/lib/python3.8/site-packages
Requires:
Required-by: acqdp

Maybe it would work if I revert back to 1.1.6, and I will reply to this issue if there are more insights.  Thank you for the detailed explanations!

Hello @ChienKaiMa ,
I have made slight change in the version dependency, forcing ACQDP to use kahypar==1.1.2 . It seems that the newest version 1.1.7 does not support the configurations previously supported.
Please let me know if this solves your problem.
Cupjin


Problem generating the mongoDB template

I'm encountering an error while attempting to render the MongoDB template, and I suspect the issue might be stemming from the cli.js file. Would you mind taking a look at it to see if anything stands out?

It works fine at my end. I have also tried in an online sandbox and it worked fine.

Oh okay, then I will try it again , thanks.


ERROR in ./node_modules/ng2-smart-table/components/cell/cell.module.js Module not found: Error: Can't resolve 'ng2-completer'

Not working in Angular 6

npm i rxjs-compat --save

Is possible you require install manually the dependency:
npm i rxjs-compat --save

No luck !! any plan to upgrade to 6?

npm i rxjs-compat --save works for me. Thanks.

On<EMAIL_ADDRESS>helped me

yarn add --dev ng2-completer
yarn add rxjs-compat

Still broken for me with rxjs-compat @ 6.3.3 and ng2-smart-table 1.3.5 on Angular 6.1.9.

I have on the package.json file this configuration for Angular 6.1.6:

...
"ng2-completer": "^2.0.8",
"ng2-smart-table": "^1.3.5",
"rxjs": "^6.2.2",
"rxjs-compat": "^6.2.2",
...

Works correctly. (make sure install n2-completer, is required by ng2-smart-table)

npm i ng2-completer --save
works for me. Thanks.

i am getting the same issue in Angular 7 ... the main point is i implemented the above and still its not solved :disappointed: ... and also i am getting all the data in the table from backend but this console will show this error as given below ...

Solved
npm i rxjs-compat --save
npm i ng2-completer --save

@almgwary ... it didn't work for me :( ... i don't know why this is happening, but from my 8 tables, only 3 tables have this error and that also on first column only, other tables don't show this type of error and most importantly every table works good and have no other issue at all ! .... so what should i do now ?

@almgwary ... it didn't work for me :( ... i don't know why this is happening, but from my 8 tables, only 3 tables have this error and that also on first column only, other tables don't show this type of error and most importantly every table works good and have no other issue at all ! .... so what should i do now ?

it works for me, because it just need these modules to start.

Try this 3 steps
1.npm install --save ng2-smart-table
2.npm i ng2-completer --save
3.npm i rxjs-compat --save
Hope it may works.

Solved this by including ng2-completer as a dev dependency
npm install ng2-completer --save-dev
reference: https://github.com/akveo/ng2-smart-table/issues/845#issuecomment-408239379

Can anyone  help me to create a search box I which I need to add my own notes

@nnixaa
I think these two modules should be imported by the library itself then the issue should be closed

Solved
npm i rxjs-compat --save
npm i ng2-completer --save

Yes. Please fix this issue. It comes up again and again.

What worked for me :
npm i @akveo/ng2-completer

npm i rxjs-compat --save
npm i ng2-completer --save
it working success :)


Shuffling Parities( Codechef September Challenge 2021 Division 2 (Unrated))

Enter your question -
Chef is given an array A consisting of N positive integers. Chef shuffles the array A and creates a new array B of length N, where Bi=(Ai+i)mod2, for each i(1≤i≤N).
Find the maximum possible sum of integers of the array B, if Chef shuffles the array A optimally.
Enter link to the question(if question belongs to any online platform) -
https://www.codechef.com/SEPT21B/problems/SHUFFLIN
Tags for the question(eg - Array, Basic, Stack, etc.) -
Basic, NumberTheory, Java

I would like to work on this issue??

@manvikri22,
Kindly add your solution to "codechef" folder.
Deadline - 03/10/2021

@manvikri22,
Kindly add your solution to "codechef" folder.
Deadline - 03/10/2021
Gentle advice for futher Issues -
A bit more tough questions would be appreciated for Hacktoberfest tag.


[SCHEMA] The MetricExporter schema is unsafe

Currently:

there is a unique schema for MetricExporter, which can represent otlp, console, prometheus, or an extension point.
a PeriodicMetricReader uses a MetricExporter
a PullMetricReader uses a MetricExporter

This is unsafe, because push and pull exporters are mixed in a common schema, which makes it possible to represent:
meter_provider:
  readers:
    - periodic:
        exporter:
          prometheus:
    - pull:
        exporter:
          console:

This is invalid:

a periodic exporter is not going to use a prometheus scraper,
a pull exporter is now going to use the console to scrape.

Please define instead two separate schemas:

PushMetricExporter, which can represent otlp, console, or an extension point,
PullMetricExporter, which can represent prometheus, or an extension point,

This also imply that a extension points for PushMetricExporter and PullMetricExporter are different, because they have a different type, which is good.
The component registry will need to expose both types.

Good call. Would you be willing to open a PR to fix?


Fix error when no secret file provided for a complex field.

Closes https://github.com/pydantic/pydantic-settings/issues/113

please review


feature request: remove unused prefixes

Option to remove prefixes (yes, I'm aware I could lose my saved games) by game or all unused.

I agree, thanks for the suggestion! I already thought in that direction a bit and I'm planning different compatdata commands like clean-up, backup, and so on.

Implemented in v1.6.1:
https://github.com/nning/protonutils/releases/tag/v1.6.1


Add instructions on how to install VSCode plugin

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind documentation
[skip ci]
What does does this PR do / why we need it:

Adds instructions on how to install the VSCode Plugin
Remove references to old version of odo

Which issue(s) this PR fixes:
Closes https://github.com/openshift/odo/issues/3970
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[X] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Signed-off-by: Charlie Drage<EMAIL_ADDRESS>

documentation
/override ci/prow/v4.5-integration-e2e
/lgtm
/approve


Attempt to fix the release issue

Binaries are failing to run when I try to run them on my Macbook. Trying to fix it in this PR

Fixed it and pushed directly to the main branch


[Package Issue]: Remove All Beta/Eap Version from JetBrains.Kotlin.Compiler

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Remove all unstable version from JetBrains.Kotlin.Compiler
Steps to reproduce
N/A
Actual behavior
N/A
Expected behavior
N/A
Environment
N/A

Screenshots and Logs
No response

@Trenly @mdanish-kh @stephengillie close with reason: resolved in mentioned PRs

K.


Add benchmark suite

(not blocking for 1.0)
It would be nice to have a benchmark suite so that it is easy to measure versions of serde and our ser/de implementations.

Currently the built-in benchmarking in Rust is nightly only. Is the plan to use that and only run benchmarks on nightly or is there another tool we can use.

I have a benchmark here that tests a bincode-like format against serde_json on some common benchmark datasets from the C/C++ world.
https://github.com/serde-rs/json-benchmark/blob/master/src/binary.rs

@dgriffen I was just going to go for nightly-only benchmarks, but if there's another benchmark runner, I'd be happy using that .
@dtolnay Great!  Looks like a perfect starting point!

bincode-bench was written a while ago


clientconfig generates json files at toplevel

When I built a gapic client for logging/v2 API, I've noticed that the generated place for clientconfig is different from the code location. client config json files are always generated at the toplevel directory, while py files are in google/logging/v2 (rb files are lib/google/logging/v2).
But our generated code expects the JSON files in the directory where the code files reside.
This means client-config target directory has to be injected by some language-specific parameters -- I don't have concrete ideas to achieve this.

@geigerj @garrettjonesgoogle

The toolkit currently drops the client config at the top level without trying to read in any configuration (see here).
Where do the JSON client config-using languages expect the file to be located? If it can be derived from the package name, that would be an easy fix. Otherwise, let's add a field to LanguageSettingsProto in config.proto to specify the client config path.

It looks like PHP expects the config to be located under a directory called "resources". Therefore, let's make the location optionally configurable in the GAPIC yaml. The code generator call fall back to using the package name if the JSON location is unspecified.

Actually, I'm now realizing that there is language-specific logic in converting a package name to a directory structure, so it's not possible to fall back to package name in a language-independent way by default.
At the same time, specifying the full path to the client config in the GAPIC yaml would have a lot of redundancy with the package name.
@jmuk @garrettjonesgoogle thoughts?

We could treat this the same way we treat the paths for the api wrapper classes - each language might or might not base it on the package name, and has a way of translating the package name to the path that it wants. So, maybe each language needs a context function to provide the right path.

To clarify, are you suggesting that each language would use a different provider for running the JSON client config? Right now, there's only a single, language-independent context for the client config.

I was kind of suggesting that, but that is a mistake. We just need this particular value parameterized per language somehow, but using code that transforms the existing package config value we have already into the right path for the client config for the language.

OK, that seems possible. We can

Remove getFilename() from the JSON config snippets interface
Make ClientConfigGapicProvider abstract, with output() depending on the abstract method String getFilename(String packageName). Each language can have a provider that extends ClientConfigGapicProvider and implements this method.

I'd prefer to use the Strategy pattern over the Template Method pattern, it makes the code a lot more modular.


Laravel 9.14.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v9.14.0 branch.
Before merging, you need to:

Checkout the shift-ci-v9.14.0 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:warning: You are running version 5.2.0 of spatie/laravel-translatable. The latest version is 6.0.0. You should review this dependency to see if you may upgrade to the latest version.


Use webpack dev server in all existing examples

As a dev, I want to be able to easily tinker with the code in these examples. Manually triggering a build every time I poke at an example is tiresome.
AC

[x] update all examples to have a command 'dev-server'? that runs webpack-dev-server
[x] do not remove the current functionality of 'start'
[x] add explanation of 'start' vs 'dev-server' on main README file.

@roboza there's no PR associated with this story. Moving back to in-progress.


Create Tables that related Bands with Artist and Album, side Bands

Please provide a description to this issue!

I suggest change the column name "id" to "bandID" .
CREATE TABLE bandDB (
    id text primary key,
    name text not null,
    genre text not null
);

should be
CREATE TABLE bandDB (
    bandID text primary key,
    name text not null,
    genre text not null
);

It is to follow the convention used  by Nazareno.

Hi @lbuttignol , thanks !! Ready the changes !


Issue #116: Add observation of angle between instrument and target

Description
Closes #116
Now it is possible to add the angle between the sensing instrument and the target in the observations using the target_angle option.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How should this pull request be reviewed?

[X] By commit
[ ] All changes at once

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Passes Tests

[X] Unit tests (General Environment only) pytest --cov bsk_rl/envs/general_satellite_tasking --cov-report term-missing tests/unittest
[X] Integrated tests (General Environment only) pytest --cov bsk_rl/envs/general_satellite_tasking --cov-report term-missing tests/integration
[X] Examples (General Environment only) pytest tests/examples

Test Configuration

Python: 3.11.6
Basilisk: 2.2.2
Platform: MacOS Sonoma 14.2.1

Checklist:

[X] My code follows the style guidelines of this project (passes Black, ruff, and isort)
[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] Commit messages are atomic, are in the form Issue #XXX: Message and have a useful message
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works

Could you add a unit test or two for this functionality? Otherwise looks good


Update Nearby.js

Prevent an Identify Task from executing and opening an info window when a point is dropped on the map.

Brian, please review and merge if appropriate.
Thank you!


Fix swagger-schema-official version

Fix swagger-schema-official directory name in order to match prerelease version

Travis job throws this weird error for nodejs 8...
ERROR: Error finding latest commit hash for definitions/npm/underscore_v1.x.x/flow_v0.38.x-/underscore_v1.x.x.js: undefined

This change is causing a failure for me when I run flow-typed install
[peterconerly@sandbox]$ flow-typed install
• Found<EMAIL_ADDRESS>installed. Installing libdefs compatible with this version of Flow...
• Found 117 dependencies in package.json. Searching for libdefs...
• flow-typed cache not found, fetching from GitHub...done.
UNCAUGHT ERROR: Error: swagger-schema-official_v2.x.x-: 'swagger-schema-official_v2.x.x-' is a malformed definitions/npm/ directory name! Expected the name to be formatted as <PKGNAME>_v<MAJOR>.<MINOR>.<PATCH>
    at validationError (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/dist/lib/libDefs.js:804:11)
    at parseRepoDirItem (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/dist/lib/libDefs.js:731:5)
    at addLibDefs$ (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/dist/lib/libDefs.js:304:27)
    at tryCatch (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:64:40)
    at Generator.invoke [as _invoke] (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:355:22)
    at Generator.prototype.(anonymous function) [as next] (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:116:21)
    at tryCatch (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:64:40)
    at invoke (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:154:20)
    at /Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:201:11
    at callInvokeWithMethodAndArg (/Users/peterconerly/.nvm/versions/node/v7.0.0/lib/node_modules/flow-typed/node_modules/regenerator-runtime/runtime.js:200:16)

👆 +1

@mat1h I was able to get a temporary fix by:
cd ~/.flow-typed/repo
git rm -r definitions/npm/swagger-schema-official_v2.x.x-/
git commit -m "Remove swagger"

Then you won't have swagger, but flow-typed will still be able to rebase on master.  (You won't be able to rebase if you are uncommitted changes.)

@pconerly @mat1h take a look here -> #905


Integrate new CompilationKind values for social laws

added compilationKind values for social laws integration

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (f9eba25) 85.05% compared to head (683feb6) 85.05%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #447   +/-   ##
=======================================
  Coverage   85.05%   85.05%
=======================================
  Files         198      198
  Lines       25722    25727    +5
=======================================
+ Hits        21878    21883    +5
  Misses       3844     3844

Impacted Files
Coverage Δ

unified_planning/engines/mixins/compiler.py
78.68% <100.00%> (+1.90%)
:arrow_up:

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

@karpase Can you extend the documentation explaining the meaning of the new compilation kinds here?

Done, thanks @alvalentini


Cannot install rbac-manager using Helm and RBACDefinition with Terraform

Is your feature request related to a problem? Please describe.
Do to Terraform not handling CRDs and CRs very well yet:

https://github.com/hashicorp/terraform-provider-kubernetes/issues/1583
https://github.com/hashicorp/terraform-provider-kubernetes/issues/1367

when you try to install the helm release with Terraform for rbac-manager and also apply a CR RBACDefinition you get:
│ Error: Failed to determine GroupVersionResource for manifest
│
│   with module.addons.module.rbac_manager.module.helm_chart.kubernetes_manifest.this,
│   on addons/rbac_manager/helm_chart/main.tf line 15, in resource "kubernetes_manifest" "this":
│   15: resource "kubernetes_manifest" "this" {
│
│ no matches for kind "RBACDefinition" in group "rbacmanager.reactiveops.io"

using this Terraform code:
resource "kubernetes_namespace" "this" {
  metadata {
    name = local.namespace
  }
}

resource "helm_release" "this" {
  name       = local.name
  repository = local.chart
  chart      = local.name
  version    = local.chart_version
  namespace  = kubernetes_namespace.this.metadata.0.name
}

resource "kubernetes_manifest" "this" {
  manifest = yamldecode(templatefile("${path.module}/rbac.template.yaml", {
    NAMESPACE = kubernetes_namespace.this.metadata.0.name
  }))

  depends_on = [
    helm_release.this
  ]
}

Describe the solution you'd like
It would be great if we could define a full RBACDefinition in the values.yaml of the helm chart for rbac-manager.
Describe alternatives you've considered
An alternative is to deploy my own helm chart with the RBACDefinition and install that using Terraform, however it is quit a bit of overhead and not super clean IMO.
Additional context
Add any other context or screenshots about the feature request here.

Ah yeah the kubectl provider seems to work perfectly. I was using the official k8s one with the kubernetes_manifest resource and it was failing but the kubectl provider seems 👌
Thanks!

Awesome! Thanks for updating!


Week three step five tutorial

Finish up to step five of the week three tutorial.

Fix please take a look

Fix and commit changes


Create games with ÖbEngine

Creating games with ÖbEngine also helps a lot because it can shows that there's missing features or bugs that I did not see.
You can create a Pull Request of this repository to add a game creating with ÖbEngine : https://github.com/Sygmei/ObEngineExamples
Bonus if you made a tutorial on how you made the game !

@PierrickLP Added Sokoban on https://github.com/Sygmei/ObEngineExamples/tree/master/Sokoban

@Sygmei Added Pong on https://github.com/Sygmei/ObEngineExamples/tree/master/Pong


Permissions

For some reason, the players on my server who do not have operator status cannot craft the wands in the crafting table, they have the craft and use perms and it shows they can craft the item in the crafting table, but when the materials are put into the crafting table, it has no output. They have the pack and it shows all the items, and they have the perms to craft but still cant.

Have you changed or set permissions? (Players have all the perms they need by default)
What permission manager are you using?

I am currently using LuckPerms and PowerRanks

Players couldnt even see the recipes without the magic.craft.* permission

Why are you using two different permission plugins?
PowerRanks has a history of not working with Magic, for reasons out of my control. If you don't need it, I suggest you remove it and stick with LuckPerms.
If you do need it.. then um I'm not sure you can fix this sorry :(

If you are able to just use LuckPerms you will probably want to reset any Magic permissions you have set. A permission plugin that actually uses and respects default permissions (e.g. not PowerRanks) should work fine out of the box.

Ohhhh, that makes alot of sense, will be removing it. Thank you so much!

Removing the PowerRanks plugin currently, will test the perms immediately afterwards

Oh, it works now! Thank you so much :D


Sidecars

Description
Changes proposed in this pull request:

Bump version of sidecars from v1alpha3 to v1beta1

Related issue(s)

/retest


Test new notebook renderer API

Refs: https://github.com/microsoft/vscode/issues/123540

[ ] anyOS @aeschli
[ ] anyOS @andreamah

Complexity: 4
Authors: @connor4312, @mjbvz
Create Issue

The issue provides a good description of the problems and features. You can test this with an existing extension, like github-issues-notebooks, or create your own renderer via npm install -g generator-code && yo code --insiders
Verify that the API and described typings work as expected. Note that messaging is separate (https://github.com/microsoft/vscode/issues/123601)

I believe that Joh created a test for this: #124995
Closing as a duplicate

Ah the bot snuck in while I was  leaving the comment. Instead of running assignments again, please be available to help out on other test items


Add built-in Material UI support

Being one of the most popular React component libraries, I think this would be a welcome addition to this plugin.

PR Welcome!


[CORE-559] add a http handler for oidc health check

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


Stuck at Resolvers.get

From @eernstg on November 3, 2015 14:57
In some cases the get method on an instance of Resolvers from 'resolvers.dart' gets stuck, in the sense that the next statement after await never gets executed:
  Resolvers resolvers = ..;
  Resolver resolver = await resolvers.get(myTransform);
  // not reached

The problem is easy to reproduce:
  git clone 'https://github.com/dart-lang/reflectable'
  cd reflectable/tool
  git co 64be223a843f31b799b1170de4fe786e0468c32a
  ./bottest

This will end in a state where the pub build --mode=debug test process is stuck. The actual location where it is stuck is at line 2741 in 'lib/src/transformer_implementation.dart'. Note that the stuck state may hereafter be reproduced much faster by running pub build --mode=debug test directly in the 'test_reflectable' subdirectory (cd ../test_reflectable in the above setting).
We have noticed that the problem disappears when we use a dependency_override such that the version of the analyzer is 0.26.1+14 rather than 0.26.2. Because of this fact, we have reported the issue here as well as on the 'code_transformers' repo: https://github.com/dart-lang/code_transformers/issues/29.
Update: Reflectable has a new commit doing the dependency override on the analyzer that prevents the above problem, so the git clone above must fetch commit 64be223a843f31b799b1170de4fe786e0468c32a rather than the most recent one. I edited the text above to reflect this fact.
Copied from original issue: dart-lang/analyzer_cli#73

Please reopen this: Reflectable cannot allow newer versions of the analyzer than 0.26.1+14 before this gets fixed.

Please reopen this: Reflectable cannot allow newer versions of the analyzer than 0.26.1+14 before this gets fixed.

It's still open!  Just moved repos.

I created this as an 'analyzer_cli'  issue (Nov 3rd), but later I discovered that it should have been categorized as 'sdk' with 'area-analyzer', and I submitted a very similar issue there (Nov 24th) at https://github.com/dart-lang/sdk/issues/25032. Now this issue has moved from analyzer_cli to sdk (cf. 'Copied from..' above). I think one of them should be closed, marking it as a duplicate of the other.

Thanks!  Closing this one as a duplicated by #25032 (where it really belongs anyway).
Cheers!


Git initialization error

There is a strange issue during initialization of git repo. The only output in the command line is:
✔ Done!

Error: Exited with status 128

I've investigated a bit, and it appears that this line of code is cause of the issue.
await spawn('git', ['commit', '--author', gitUser, '-m', 'initial commit from Preact CLI'], { cwd });

It looks like the issue is caused by missed email in the global config of git.


error in nushell 0.97.2

Hey, I just updated nushell to 0.97.2, but when doing plugin add ~/.cargo/bin/nu_plugin_port_scan i get this error:
Error: nu::shell::plugin_failed_to_load

  × Plugin failed to load: Plugin `units` is compiled for nushell version 0.94.0, which is not compatible with version 0.97.2

Thanks for updating me, it's been a bit hard to keep up with all the breaking changes that occur - hoping one day, things stabilize on their end. I'll fix this this weekend, but for now, you might want to just deregister the plugin so you can use others, if its blocking you.

I've published a new version of nu_plugin_units that should run with that latest version of nushell. Let me know if you have any issues!


no terminal

Issue Type: Feature Request
i can not find easy way to find terminal in this new release
VS Code version: Code 1.35.0 (553cfb2c2205db5f15f3ee8395bbd5cf066d357d, 2019-06-04T01:17:12.481Z)
OS version: Windows_NT x64 10.0.17763

/needsMoreInfo


Acs-engine latest version 0.19.0 fails with kubenet plugin default install

https://github.com/digeler/scripts/blob/master/acs-engineissue
full error log is here

Looking
/assign

Can you try:

Switch to Kubernetes v1.10.5
Switch to availabilitysets from VMSS

And see if either of those resolve it for you?

Thanks Patrick
Looks like issue is fixed with cni 1.0.7
Ill give it a try.

Can you close issue if it's working for you?
I also added internal tests with kubenet and they're passing. You can use that as an option if you prefer that instead of azure-cni


Update TRANSLATORS

I dont know if it's right, if it is not, please let me know.
I've got email from joe2010xtmf because I contribute Indonesian language for gogs, and I can open a pull request to file https://github.com/gogits/gogs/blob/master/conf/locale/TRANSLATORS.
Should I pull request this to develop branch? (because I see "master" from the blob url file.

Thank you!


Filter conditions need to be change according to string in downloadable products grid for order id column at customer end

Bug report
Title
Filter conditions need to be change according to string in downloadable products grid for order id column at customer end
Issue Description
as of now numeric conditions are given
Preconditions

framework Version.v0.1.8
Steps to reproduce
step1: go to downloadable products grid at customer end and check filter conditions for order id column
Expected result
string conditions should be there

Actual result
numeric condition available , hence filter is not working properly for order id column.

framework Version.v0.1.8
issue fixed


Unable to set EXTRA_DATA params to BARCODE plugin

Hi Darryn,
i'm trying to set/update EXTRA_DATA params like decoder_code39_convert_to_code32 without success.
This is my configuration that I send to 'com.symbol.datawedge.api.SET_CONFIG':
const profileConfig = {           PROFILE_NAME: AppProfileName,           PROFILE_ENABLED: 'true',           CONFIG_MODE: 'UPDATE',           PLUGIN_CONFIG: {             PLUGIN_NAME: 'BARCODE',             RESET_CONFIG: 'true',             PARAM_LIST: {               EXTRA_DATA: {                 decoder_code39: 'true',                 decoder_code39_convert_to_code32: 'true',               },             },           },           APP_LIST: [             {               PACKAGE_NAME: AppBuildName,               ACTIVITY_LIST: ['*'],             },           ],         }
DataWedge version 7.0.4
Where am I wrong?

Looks like you have some invalid JSON, please take a look at https://github.com/darryncampbell/DataWedgeCordova/blob/master/www/js/index.js#L68 which is a known working example and build from there

Sorry for the code formatting. The configuration updated is:

The datawedge profile is correctly created as before with decoder_code39 option set to true but without the option decoder_code39_convert_to_code32
I'm following this guide: https://techdocs.zebra.com/datawedge/7-6/guide/api/setconfig/
How can I pass it the EXTRA_DATA decoder Parameters?

That's still not valid JSON though is it?  I mean you have trailing commas at the end of your lists.  Also, things like PROFILE_NAME should be strings.

That's my typescript object that will convert into JSON when sent to datawedge API just like the example you given me: https://github.com/darryncampbell/DataWedgeCordova/blob/master/www/js/index.js#L68.
The trailing commas are added automatically from the Prettierjs/TSlint configuration, in no way affects the generation of the JSON.
AppProfileName and AppBuildName are variables that contain the name and the buildName of my application.

:/ I don't know... the next step would be to test with native code to see if it is a bug in DataWedge or something else.  I will try to take a look if I get a chance over the next few days - the plan is to just modify https://github.com/ZebraDevs/samples-datawedge/blob/master/DataWedge-API-Exerciser/app/src/main/java/com/zebra/datawedgeexerciser/MainActivity.java#L324 to add the convert_to_code32 attribute and see if that works.

You are passing the correct strings at the correct place, I tested with:
barcodeProps.putString("decoder_code39", "true");
barcodeProps.putString("decoder_code39_convert_to_code32", "true");

in native code as I said in my previous comment and that works.
You might want to try specifying
scanner_selection: 'auto'

To ensure you are applying the configuration to the correct scanner

Hi, I'm having trouble with something similar, and I was hoping you could help out. I'm trying to set extra parameters on my barcode scanner profile (ie aim_type, same_barcode_timeout, decode_haptic_feedback, etc), and nothing I'm doing is changing the settings on my profile. I'm confused by this line in the docs:

BARCODE – takes a value from the Scanner Input Parameters table below; specify decoder and other input settings as EXTRA_DATA in the PARAM_LIST nested bundle.

What does it mean EXTRA_DATA? Is that just supposed to be another JSON object? Here is what I have:
const profileConfig: DatawedgeProfileConfig = {
      PROFILE_NAME: 'InventoryManagement',
      PROFILE_ENABLED: 'true',
      CONFIG_MODE: 'OVERWRITE',
      PLUGIN_CONFIG: {
        PLUGIN_NAME: 'BARCODE',
        RESET_CONFIG: 'true',
        PARAM_LIST: {
          configure_all_scanners: 'true',
          scanner_input_enabled: 'true',
          beam_timer: '60000',
          same_barcode_timeout: '5000',
          different_barcode_timeout: '0',
          aim_type: '5',
          upc_enable_marginless_decode: 'true',
          '1d_marginless_decode_effort_level': '3',
          poor_quality_bcdecode_effort_level: '3',
          decode_haptic_feedback: 'true',
          decoding_led_feedback: 'true',
        },
      },
      APP_LIST: [
        {
          PACKAGE_NAME: ApplicationConstants.APP_STRING,
          ACTIVITY_LIST: ['*'],
        },
      ],
    };

I've also tried changing PARAM_LIST to:
PARAM_LIST: {
          configure_all_scanners: 'true',
          scanner_input_enabled: 'true',
          EXTRA_DATA: {
              beam_timer: '60000',
              same_barcode_timeout: '5000',
              different_barcode_timeout: '0',
              aim_type: '5',
              upc_enable_marginless_decode: 'true',
              '1d_marginless_decode_effort_level': '3',
              poor_quality_bcdecode_effort_level: '3',
              decode_haptic_feedback: 'true',
              decoding_led_feedback: 'true',
          }
},

but still no luck. Can anyone provide some insight here? Thank you.

Hi @kylebrennan , sorry I just saw your comment - are you still facing issues?  There is a demo app that calls the DataWedge API at https://github.com/darryncampbell/DataWedgeCordova.

For the original issue, I have asked the docs team to update the docs & examples to clarify that the scanner must be specified as part of the call to SET_CONFIG: https://techdocs.zebra.com/datawedge/8-0/guide/api/setconfig/ (we had another, very similar issue raised along these same lines)

Techdocs has now been updated.


Publish npm package for v2.18.1

Please publish a NPM package for latest version for mejs plugin.

In fact, none of the v2.18.x releases are currently on NPM. Could you please at least publish the latest one (2.18.2 I think)? Particularly because of #1542.

@pluma As I am using 'mediaelement-and-player.js' instead of 'mediaelement.js' as my initial entry into mediaelement package, I am specifying path to target JS file for plugin when retrieving package.

Sorry about that. It's published now.
On Sun, Oct 11, 2015 at 10:13 AM, Yatin Ranadive<EMAIL_ADDRESS>wrote:

@pluma https://github.com/pluma As I am using
'mediaelement-and-player.js' instead of 'mediaelement.js' as my initial
entry into mediaelement package, I am specifying path to target JS file for
plugin when retrieving package.
—
Reply to this email directly or view it on GitHub
https://github.com/johndyer/mediaelement/issues/1601#issuecomment-147208663
.

--
John Dyer - http://j.hn/

@yatinrandive-lynda can you close this issue, please?


Add speciesProfile information

Do we know the following information for the fishes in this dataset?

isMarine
isFreshwater
isTerrestrial: yes, false for all.

If so, I would add habitat information to the source file, include it in the description extension and populate the speciesProfile.

All species are freshwater species (double-checked with Hugo). I will integrate this in the speciesProfile information.


Microsoft.AspNetCore.Authentication.OpenIdConnect 1.1.0 does not redirect to identity provider

Functional impact
Cannot log in (blank page displayed).
Minimal repro steps

Get repro project - https://github.com/qvsmith/IdentityServer4.Samples
Open /Quickstarts/3_ImplicitFlowAuthentication solution.
Start QuickstartIdentityServer and MvcClient project.
On MvcClient project's home page click 'Secure' link.

Expected result
User should be redirected to login page.
Actual result
Blank page is displayed. Redirect not occurring.
Further technical details
The response status code is 401. If I add custom middleware (just before oidc mw) that changes the response status code to 302 it resolves the problem. Perhaps has something to do with me using netcoreapp1.0 ipo 1.1...

See https://github.com/aspnet/Security/issues/1062

This is a known issue, which we are fixing in the upcoming 1.1.1 patch. You can refer to https://github.com/aspnet/Security/issues/1062 for more details, plus workarounds.

@Eilon Tx! I tried some of those workarounds mentioned in #1062 but, I couldn't get them to work. The workaround with custom middleware that I mentioned above at least does the trick... Fortunately this is not critical for me at this stage... just wanted to log it :)

@qvsmith got it, thanks for the info!


ubuntu 16.04 x64 --Temporary failure resolving archive.ubuntu.com container  i386/ubuntu

Output of docker version:
Client:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Tue Apr 26 23:43:49 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Tue Apr 26 23:43:49 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 2
 Running: 0
 Paused: 0
 Stopped: 2
Images: 4
Server Version: 1.11.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 9
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.4.0-22-generic
Operating System: Ubuntu 16.04 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.503 GiB
Name: lpt
ID: DPPO:663B:P43K:Q27E:EHMV:3VUO:5CKT:6SFB:SJTO:LMHI:F6XP:C4KE
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support

Additional environment details (AWS, VirtualBox, physical, etc.):

Ubuntu 16.04
Linux lpt 4.4.0-22-generic #40-Ubuntu SMP Thu May 12 22:03:46 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Steps to reproduce the issue:
1.docker run -it --rm i386/ubuntu apt-get update
2. does not work
1.docker run --privileged -it --rm i386/ubuntu apt-get update
2.does work
Describe the results you received:
docker run -it --rm i386/ubuntu apt-get update
Err:1 http://archive.ubuntu.com/ubuntu xenial InRelease
  Temporary failure resolving 'archive.ubuntu.com'
Err:2 http://archive.ubuntu.com/ubuntu xenial-updates InRelease
  Temporary failure resolving 'archive.ubuntu.com'
Err:3 http://archive.ubuntu.com/ubuntu xenial-security InRelease
  Temporary failure resolving 'archive.ubuntu.com'
Reading package lists...  They have been ignored, or old ones used instead.

Describe the results you expected:
apt-get should update the container without privileged flag.
Not able to build a container from a Dockerfile because privileged flag not available on docker build.
Additional information you deem important (e.g. issue happens only occasionally):
docker run -it --rm ubuntu apt-get update works
docker run -it --rm i386/ubuntu apt-get updatedoes not work

Can you try using --security-opt seccomp=unconfined instead of --privileged? This may be resolved on master

docker run --security-opt seccomp=unconfined -it --rm i386/ubuntu apt-get update works also instead of --privileged but how to use it in a Docker file with Docker build -t .

@chrisber did that work for you?

yes docker version 1.11.0 worked. thanks


[Spanner] Slow (over 1000ms) connection on App Engine SE PHP 7.2 even using caches

Hi!
I've tried to use Spanner from App Engine Standard Environment PHP 7.2 recently released.
And it's average response is over 1000ms.
Any idea to improvements?
code
<?php

require __DIR__ . '/vendor/autoload.php';

use Google\Cloud\Spanner\SpannerClient;
use Google\Cloud\Spanner\Session\CacheSessionPool;
use Symfony\Component\Cache\Adapter\FilesystemAdapter;

$cache = new FilesystemAdapter(); # GAE doesn't support semaphore
$sessionPool = new CacheSessionPool($cache);
$projectId = getenv('GOOGLE_CLOUD_PROJECT');
$spanner = new SpannerClient([
    'projectId' => $projectId,
    'authCache' => $cache,
]);
$database = $spanner->connect(
    'XXXXX',
    'XXXXX',
    [
        'sessionPool' => $sessionPool
    ]
);

$results = $database->execute('SELECT "Hello Spanner World" as test');
foreach ($results as $row) {
    print($row['test'] . PHP_EOL);
}

$database->close();

$ ab -n 10 -c 1 https://XXXX.appspot.com/spanner.php
...
Document Path:          /spanner.php
Document Length:        20 bytes

Concurrency Level:      1
Time taken for tests:   12.349 seconds
Complete requests:      10
Failed requests:        0
Total transferred:      2698 bytes
HTML transferred:       200 bytes
Requests per second:    0.81 [#/sec] (mean)
Time per request:       1234.942 [ms] (mean)
Time per request:       1234.942 [ms] (mean, across all concurrent requests)
Transfer rate:          0.21 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:       55   71  18.8     68     110
Processing:   764 1163 290.1   1255    1572
Waiting:      764 1163 290.1   1255    1572
Total:        819 1235 288.8   1331    1628

Percentage of the requests served within a certain time (ms)
  50%   1331
  66%   1385
  75%   1497
  80%   1558
  90%   1628
  95%   1628
  98%   1628
  99%   1628
 100%   1628 (longest request)

# For comparison:  empty page
$ ab -n 10 -c 1 https://XXXX.appspot.com/empty.php
...
Requests per second:    14.50 [#/sec] (mean)
Time per request:       68.959 [ms] (mean)

Spanner / GAE / clients are placed on asia-northeast-1.

Hi @yayugu,
I'm investigating this issue and should have an update for you tomorrow.

Hi @yayugu,
I'm seeing similar response times to those you're reporting. I've raised the issue with some people who have more knowledge than I do, and I'll be sure to keep you posted on their response.

@yayugu
Thanks for your report!
One of our engineer wants to know the details of your project, etc. Have you opened a support case, if so can you tell me the support case number?

Hi @tmatsuo
We have not issued a ticket.
I found a problem while considering using GAE. We will actively cooperate if you need more information.

@yayugu
Can you send me the following info to<EMAIL_ADDRESS>?

project id
app engine service and version
spanner instance id and database id

Thanks!

@tmatsuo
I have mailed from yuya.yaguchi gree.net

@yayugu Thanks!

cc @JustinBeckwith
@tmatsuo - is there an internal bug that I could follow along ?

@snehashah16 Yes. We're working on it.

@tmatsuo @JustinBeckwith
is there any update?

@yayugu Sorry for the delay, and unfortunately we're still working on it. Let me put the internal issue #: 110329035

@yayugu based on internal issue #110329035 this should be resolved now. I'm going to close this out, but if the issue persists please feel free to re-open and we can look deeper in to this. Thanks!

@dwsupplee Thanks!
I confirmed this issue has been fixed:
$ ab -n 10 -c 1 https://xxxx.appspot.com/spanner.php
...
Percentage of the requests served within a certain time (ms)
  50%     90
  66%     91
  75%     93
  80%     95
  90%    261
  95%    261
  98%    261
  99%    261
 100%    261 (longest request)


Update README.rst to point to marketsquare

update docs to point to marketsquare

oh, there are probably a bunch more files that need to get this same update.


I2C: Add clock stretching support.

This change adds the ability of handling i2c clock stretching. Clock stretching could happen on certain devices, such as BNO055 and PN532. When it happens, we could not sending the I2C read/writes blindly, but have to wait until the SCL coming back to high before sending the next bits. Otherwise, the SDA signals will be ignored by the device and ends up with incorrect communication.
However, clock stretching can cause I2C bus hang indefinitely by faulty devices, so many devices doesn't do clock stretching at all. So due to the rareness of clock stretching, this feature is turned off by default. And we need to explicitly enable it with the following command when running in i2c mode:
clock-stretch <clock-tick-count>

If we specify 0, it will disable the feature, otherwise it means how many full clock ticks we wait. This also means the final time that we waits depends on the frequency that we use. If the frequency is 100Khz, it means every clock tick will be 10us. If we specify clock-stretch 100, we will be waiting at maximum 1ms. But if the frequency is 1Mhz, we will be waiting 100us.
We choose the ticks instead of time in the setting, because it match more on the actual behavior and not every time interval can be mapped to a set of complete full clock ticks.

Thanks for your contribution

woot! thank you!


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it! and we'll verify it.
On Tue, Jul 14, 2020 at 7:47 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it! and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgoogle%2Fit-cert-automation-practice%2Fpull%2F4758
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/it-cert-automation-practice/pull/4758#issuecomment-658206353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOOM76E7FFTUAI2S7OFYSULR3RSGFANCNFSM4OZSABAQ
.


Write quick tutorial on how to run the local dev version of nuclio on Minikube

works on MacOS, needs to be tested for Linux


Question [File Locating]

In clockwork base files, they have it where if you put a rank in someones name (RCT, DvL, EpU, SeC, CmD) and I was wonder where I can find this, so I can change the models it changes to automatically, I specifically need the MPF model it sets to for like 05.

Bump?

The file in question is sv_hooks, but don't modify it unless you know explicitly what you're doing. Modifying a core schema file can lead to problems and possibly break your framework. I recommend creating a plugin for it. You can find out how to do that here and here.

As an addition to the comment above, it's highly recommended for any server owner to learn even some basics of Lua to the level where they don't make any syntax mistakes (e.g. forgot a bracket, forgot to close 'if' statement or a function block, used an uninitialized variable etc) and know how to use gmod wiki (wiki.garrysmod.com).
Any Garry's Mod gamemode requires this knowledge to perform any customization, so learning Lua will be a worthwhile investiment of time in case you're not only planning to create one server. It can also serve as a first step in learning programming in general (if you're interested in that kind of stuff).
It may take quite a while to get yourself comfortable with Lua's syntax and functions, but once you do - any customization that you could possibly think of would be a piece of cake. I'd recommend looking at existing code and trying to modify it to do something else or to add functionality. Sandbox gamemode is a great way to start as it's very lightweight and you have a lot of space to experiment.
tl;dr: Learn Lua.
(sorry for a wall of somewhat unrelated text, I just felt it belonged here)

The ranks are actually in the HL2RP files, not the Clockwork files. Specifically lines 28-43 in sv_hooks.lua. Modifying the schema directly is not recommended nor endorsed. Here's an easy way to do it using a plugin:

Start creating a plugin using this website (it's a web app made by myself): https://clockworkconstructors.com/construct-plugin.php

Check the /sh_plugin.lua and /sv_hooks.lua checkbox.

Use this code in the sv_hooks.lua file that was just created in your plugin:

`-- Called when a player's default model is needed.
function PLUGIN:GetPlayerDefaultModel(player)
local faction = player:GetFaction();
if (Schema:IsCombineFaction(faction)) then
	if (Schema:IsPlayerCombineRank(player, "ENTER RANK HERE")) then
		return "ENTER MODEL PATH AND MODEL NAME HERE";
	-- Uncomment these lines to add multiple ranks/models:
	--[[elseif (Schema:IsPlayerCombineRank(player, "ENTER RANK HERE")) then
		return "ENTER MODEL PATH AND MODEL NAME HERE";
	elseif (Schema:IsPlayerCombineRank(player, "ENTER RANK HERE")) then
		return "ENTER MODEL PATH AND MODEL NAME HERE";
	elseif (Schema:IsPlayerCombineRank(player, "ENTER RANK HERE")) then
		return "ENTER MODEL PATH AND MODEL NAME HERE";--]]
	end;
end;

end;`


Update getting-started.md

Change from
peer chaincode install -p chaincodev/chaincode/contract-tutorial -n mycc -v 0

to
peer chaincode install -p chaincode/contract-tutorial -n mycc -v 0

Hi @sonicskye, thanks for the PR. Unfortunately the builds were broken for a while and the tutorials are now out of date due to changes in fabric-samples so I'm going to close this PR. If you submit another, please include a DCO sign-off so that it can be merged.
Thanks again, James


[Backport] [BugFix] Hide public repos owned by private orgs (#9609)

Backport #9609

For 1.10.3 ...

ready for merge :rocket:


Add map_variables parameter to read_tmy3 function

[x] Closes #1517
[ ] I am familiar with the contributing guidelines
[x] Tests added
[ ] Updates entries in docs/sphinx/source/reference for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

Hi @ooprathamm
Thank you very much for tackling this issue 😀
There's a couple of formatting issues (stickler). If you can solve those, then I'll go ahead and give a detailed review. If you have any issues feel free to post here and we'll get it sorted!

Hi @ooprathamm
Thank you very much for tackling this issue 😀
There's a couple of formatting issues (stickler). If you can solve those, then I'll go ahead and give a detailed review. If you have any issues feel free to post here and we'll get it sorted!
Also, if you update the description with the issue number (closes #xxxx), then GitHub will automatically link the two.

@ooprathamm Mapping of variables using the _recolumn function is not in line with standard pvlib names. Thus I suggest you create a dictionary mapping the TMY3 names to standard pvlib names. An incomplete list of standard names can be found here. You can check out some of the other iotools to get an idea of how this is done, e.g., PSM3, PVGIS, and CAMS (SODAPRO).

@ooprathamm are you still working on this issue? Again, don't hesitate to reach out for help if you are stuck.

yes, i got occupied by exams ,will soon reach back
On Mon, Jan 16, 2023 at 8:34 PM Adam R. Jensen @.***>
wrote:

@ooprathamm https://github.com/ooprathamm are you still working on this
issue? Again, don't hesitate to reach out for help if you are stuck.
—
Reply to this email directly, view it on GitHub
https://github.com/pvlib/pvlib-python/pull/1623#issuecomment-1384187321,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVMUIALVCMHQYBIEOHTDU5LWSVPOTANCNFSM6AAAAAATF2HWNA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think the current test failures are not related to this PR and can be ignored. @pvlib/pvlib-core it would be great if anyone wants to give a review.

What is going to happen to recolumn? It seems awkward to me to have both. Should it be an error if both are True?

I'm in strong favor of eradicating recolumn, i.e., removing it completely in 0.11.0. Standardization is the way forward!

While we're at it, could you (@ooprathamm) add the following DOIs to the two references? The DOI of the first reference is https://doi.org/10.2172/928611 and the second reference is https://doi.org/10.2172/901864.

I'm in strong favor of eradicating recolumn, i.e., removing it completely in 0.11.0. Standardization is the way forward!

I wonder if it is worth soliciting input from users on this point. I checked the first few pages of these search results and did not find a single case of anyone specifying a value for recolumn.  But if we are going to break peoples' code with map_variables, we might as well break recolumn at the same time.

I wonder if it is worth soliciting input from users on this point. I checked the first few pages of these search results and did not find a single case of anyone specifying a value for recolumn. But if we are going to break peoples' code with map_variables, we might as well break recolumn at the same time.

Agree, hence why I suggest that when we do the breaking change (v0.11) we implement the following changes map_variables=True and remove the recolumn parameter. I suppose we could add this information to the user warning.

@ooprathamm Nice work so far. Still a few things that need to be resolved, including adding map_variables=False to existing tests, updating the variables and style list, and adding a whatsnew entry here.

@AdamRJensen made the changes :)

I'm still wondering what should happen when read_tmy3(..., map_variables=True, recolumn=True). If recolumn is to have no effect when map_variables=True, maybe that should be documented?

I agree on it to be documented, and also we should consider this:

if map_variables is not True:
it also solves the failing tests.

I am wondering if we have gone about this PR a bit backwards. Instead of having recolumn=True, map_variables=None in the function signature, would it be better to use recolumn=None, map_variables=True? The way it is now, we need two separate deprecation messages. I wonder if we can simplify things by thinking through the various combinations of recolumn and map_variables with an enumerated truth table or even a Karnaugh map.

@kandersolar Could you elaborate a bit more on this?

Sorry for the opaque comment :)  I'm not sure what I was thinking regarding the function signature.  For the second half, I meant that there are a lot of combinations of (recolumn, map_variables) pairings, and it's not clear to me that the current code is handling all of those combinations appropriately.  I thought it would be helpful to make a table like this for all the possibilities that forces us to consider all cases:

Make a table like this for the renaming behavior, another one for deprecation messages for map_variables becoming the default, and possibly a third if we also want to warn about recolumn being deprecated?  Then you can check whether the code is doing the right thing in each case.

@kandersolar I think the table should look like this:
I see that you had "recolumn" for map_variables=False and recolumn=None. However, I think that when a user sets map_variables=False and does not pass a value for recolumn (which now defaults to None) then the new behavior should be in place - i.e., there should not be any changes to the column names.

map_variables

True
False
None

True
ValueError
ValueError
recolumn - warning

recolumn
False
ValueError
ValueError
nothing - warning

None
map_variables
Nothing
recolumn - warning

Thanks @ooprathamm and @AdamRJensen!


close session in exception case

Before: When the worker quit with "exception" or "not started" it kept the session alive until queuelimit

Could you also wrap self.api.get_map_objects with try..except that will close session on an exception?


change order by to leverage existing index

After quick manual testing on the staging API, I realized two more bock query are not leveraging the index, this is now fixed. get_highest_block and get_last_blocks end point would take more than 10 seconds to return the value because it was doing a sequential scan of the db. Now the response time should be lowered in the ms range

Pull Request Test Coverage Report for Build 655

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 57.939%

Files with Coverage Reduction
New Missed Lines
%

apps/neoscan_node/lib/neoscan_node/notifications/notifications.ex
2
82.61%

Totals

Change from base Build 653:
0.0%

Covered Lines:
1029

Relevant Lines:
1776

💛  - Coveralls


Team2 Contributions

Description
Team 2's cumulative proposed changes
Type of Change

[ ] Feature
[ ] Bug fix
[x] Documentation

Contributors

@jnormile
@AidanNeeson
@ningerson2002
@gormley01
@brendengray

Reminder
All GitHub Actions checks must be in a passing state in this branch or fork before this pull request can be merged.

Everything looks good to me! Nice job, these are good points for the code of conduct.


Configure placeholder

Can we configure the place holder text by row? Maybe too rigid?

Ability to configure placeholder labels in Texture in general would be an interesting feature.

OK. This is related to linking semantically link terms with controlled vocabularies. It is quite core to the structured data-figure package idea so let's discuss these things together


error  'App' is not defined  no-undef

Hi,
I get the following error:
var App = React.createClass({
  render: function() {
    return (

My Component Example

        <MyComponent />

    );
  }
});

React.render(<App />, document.getElementById('app'));    // error  'App' is not defined  no-undef

Here is my .eslintrc file:
{
  "env": {
    "node": true,
    "browser": true
  },
  "parser": "babel-eslint",
  "rules": {
    "space-after-keywords": "always",
    "quotes": "single"
  }
}

I use babel-loader to run eslint with WebPack. Here is the relevant bit from webpack.config.js:
module: {
  loaders: [
    { test: /\.js$/, loaders: ["react-hot", "babel-loader", "eslint-loader"], exclude: /node_modules/ }
  ]
}

What am I doing wrong?

What am I doing wrong?

Nothing, it's a bug.

Seems to be a dupe of #8.

Nope. Actually it's a dupe of https://github.com/babel/babel-eslint/issues/15.

Probably need to check this with eslint 0.17 changes for jsx and react.

I am not having the issue anymore with eslint 0.17 and eslint-plugin-react 1.5.0

Can you please verify that this issue still exists on 3.0.1?

Is this still an issue after using eslint-plugin-react?

Not at all. You can close this. Eslint doesn't handle this anymore & the plugin does this like a charm.

Ok great!

From eslint docs we can find these rules,

Disallow Undeclared Variables (no-undef)
Any reference to an undeclared variable causes a warning, unless the variable is explicitly mentioned in a /global .../ comment.

/*global someFunction b:true*/
/*eslint no-undef: "error"*/

var a = someFunction();
b = 10;


Introduce the build time transformation infrastructure

We should introduce a build time transformation infrastructure to spring-graalvm-native that will materialize for the user as Maven and Gradle plugins (without the need for a dependency on external projects like spring-init).
This infra will be used for a wide range of build time optimizations, for example::

Automatically set optimization flags #225
Remove the need for class resources #341
Turn the GraalVM feature into a pure build time transformation generating JSON configs #344
Generate a programmatic version of spring.factories at build time #345

A starting point with the Maven plugin has been created by @dsyer here with some insights shared by @aclement about how to improve it:

I thought we’d be closer to those if the subclasses you generated implemented BeanFactoryAware (and so had a setBeanFactory method) and then called that from the overridden @ Bean methods to check if a bean already existed - that’s what those CGLIB proxies do.

The target should probably be at some point inclusion in Spring Framework and seamless usage by Spring Boot Maven and Gradle plugins.

@christophstrobl Do you want to create a dedicated issue to incubate Spring Data build time transformations you are experimenting on, like I did for the issues mentioned in this issue? The idea would be to plug them intro Spring Data with substitutions for now.

@sdeleuze Not right away.
For the time being tend to stick to the SpringDataComponentProcessor and continue our investigation in a dedicated project under Spring Experimental. Generated/provided entity metadata is a use case, not limited to GraalVM native images, that might turn out to be beneficial in everyday projects.

@christophstrobl OK, feel free to integrate this experimental Spring Data project here with a dedicated issue/PR when you feel it is ready.

The first version of the build time infrastructure was introduced in 5f94fd4674. We're refining this arrangement with other issues.

The first version of the build time infrastructure was introduced in 5f94fd4674. We're refining this arrangement with other issues.