test: mark failing dask tests, add dask to constructor

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[x] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #674
Closes #

Checklist

[x] Code follows style guide (ruff)
[ ] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.

I didn't get how to fix this part yet


!): Fix for tensorflow-gpu == 1.12.0 case.

Or hint reqirement: tensorflow-gpu==1.13.0

Thanks for the help!


Incorporate auto-TLS for domains associated with each app

https://doc.traefik.io/traefik/https/acme/

Need to require an email address from the user for certificatesResolvers in the traefik static config

Essential to use sudo chmod 600 acme.json after transferring the file to the ec2 server. Previously, was getting the error {"level":"error","msg":"The ACME resolver \"myresolver\" is skipped from the resolvers list because: unable to get ACME account: permissions 644 for /assets/letsencrypt/acme.json are too open, please use 600","time":"2022-03-26T19:25:31Z"}

Need to make sure assets folder containers letsencrypt/acme.json
The hosted domain needs to be pointing at the proper IP address prior to starting up traefik or deploying the new app, otherwise the tlschallenge will fail, and we need to run docker service update traefik_traefik for the change to register.

dynamically rename the datasource to the SECURED version of the user's prometheus hostname:
datasources:
  # <string, required> name of the datasource. Required
- name: Prometheus
  # <string, required> datasource type. Required
  type: prometheus
  # <string, required> access mode. direct or proxy. Required
  access: direct
  # <int> org id. will default to orgId 1 if not specified
  orgId: 1
  # <string> url
  url: https://prometheus-2.michaelfatigati.com

For testing, use a staging environment, but remove this line for production:
certificatesResolvers:
  myresolver:
    acme:
      email<EMAIL_ADDRESS>      storage: /assets/letsencrypt/acme.json
      tlsChallenge: {}
      caServer: "https://acme-staging-v02.api.letsencrypt.org/directory"


[v3.4] Remove babel-jest since it automatically loaded by Jest

From https://github.com/babel/babel-jest:

The contents of this repository were moved into the Jest repository. The source code can be found in packages/babel-jest. babel-jest is now automatically loaded by Jest and fully integrated.

Just checked - all tests are passed without babel-jest explicitly specified in package.json. However, babel-jest is present in node_modules folder (jest-cli#jest-runtime depends on it)

@nktssh Thanks for the heads up! If you have tested thoroughly and found no issues, would you open a PR?
Either way, I've made a note to keep this issue in mind for the next dependency update.
Thanks for contributing 🎉

@gihrig with pleasure!

Awesome, thanks for digging in!


no libmxnet.so file after building

Hi,
I am on OS X 10.10.5. I was able to build and install mxnet (both python and R)with USE_CUDA = 0 successfully. However, when I try to build and install mxnet with USE_CUDA = 1 (GPU gtx 960), I got the following error:
RuntimeError: Cannot find the files.
List of candidates:
/Users/Jun/mxnet/python/mxnet/libmxnet.so

I could not find libmxnet.so in mxnet/python/mxnet/.
Here are some details of compiling:
in config.mk
ADD_LDFLAGS = -I/usr/local/opt/openblas/lib -I/Developer/NVIDIA/CUDA-7.5/lib
ADD_CFLAGS = -I/usr/local/opt/openblas/include -I/Developer/NVIDIA/CUDA-7.5/include
USE_CUDA_PATH = /Developer/NVIDIA/CUDA-7.5

in terminal:
export PATH=/Developer/NVIDIA/CUDA-7.5/bin:$PATH
export DYLD_LIBRARY_PATH=/Developer/NVIDIA/CUDA-7.5/lib:$DYLD_LIBRARY_PATH

also used
make clean_all

after above modifications, I was able to compile using gpu with no error, but warnings like: warning:
‘dmlc::__make_ParserFactoryReg_uint32_t_csv__’ defined but not used [-Wunused-variable]
   static EntryType & __make_ ## EntryTypeName ## _ ## Name ## __ =      \

Then I run python setup.py install, I got:
RuntimeError: Cannot find the files.
List of candidates:
/Users/Jun/mxnet/python/mxnet/libmxnet.so

Any idea?
I am sorry for the long post, but I searched web, and tried https://github.com/dmlc/mxnet/issues/1174 with no luck. Thank you very much!

Looks like your build failed.

I will check again. Thanks for the suggestion.

I was unable to build in OS X with gnu, but could for cpu. I did not set:
ADD_LDFLAGS = -I/usr/local/opt/openblas/lib -I/Developer/NVIDIA/CUDA-7.5/lib
ADD_CFLAGS = -I/usr/local/opt/openblas/include -I/Developer/NVIDIA/CUDA-7.5/include

but did set:
USE_CUDA = 1
USE_CUDA_PATH=/usr/local/cuda

After that I received a warning about /usr/local/cuda/lib64 missing. This directory does not exist in the OS X installation of the cuda toolkit. For a workaround I created a soft link /usr/local/cuda/lib64 -> /usr/local/cuda/lib and then was able to build with gpu enabled.

@mndrake Thanks for your info.
@piiswrong I build it again, and saw this error (sorry I missed it at the first place):
nvcc fatal   : GNU C/C++ compiler is no longer supported as a host compiler on Mac OS X.

This problem might be related: https://devtalk.nvidia.com/default/topic/879431/cuda-programming-and-performance/nvcc-amp-clang-7-no-typo-here-/ although I can successfully build cuda samples.
I am downgrading Xcode now to check the solutions and will update this thread. Thank you all.

Was able to build mxnet after downgrading Xcode to 6.2. However the same issue https://github.com/dmlc/mxnet/issues/570 appears....I will update if I found a fix.

Sorry for another post, I am quite new with deep learning on gpu.
So I was able to train using gpu in OS X terminal:
python train_mnist.py --gpus 0
OpenCV is unavailable.
2016-01-11 19:46:51,725 Node[0] start with arguments Namespace(batch_size=128, data_dir='mnist/', gpus='0', kv_store='local', load_epoch=None, lr=0.1, lr_factor=1, lr_factor_epoch=1, model_prefix=None, network='mlp', num_epochs=10, num_examples=60000)
[19:46:54] src/io/iter_mnist.cc:91: MNISTIter: load 60000 images, shuffle=1, shape=(128,784)
[19:46:54] src/io/iter_mnist.cc:91: MNISTIter: load 10000 images, shuffle=1, shape=(128,784)
2016-01-11 19:46:54,750 Node[0] Start training with [gpu(0)]
2016-01-11 19:47:03,866 Node[0] Epoch[0] Batch [50]	Speed: 141320.50 samples/sec	Train-accuracy=0.700469
2016-01-11 19:47:03,910 Node[0] Epoch[0] Batch [100]	Speed: 145289.52 samples/sec	Train-accuracy=0.796094
2016-01-11 19:47:03,964 Node[0] Epoch[0] Batch [150]	Speed: 120573.08 samples/sec	Train-accuracy=0.837656
2016-01-11 19:47:04,003 Node[0] Epoch[0] Batch [200]	Speed: 164006.17 samples/sec	Train-accuracy=0.863047
2016-01-11 19:47:04,060 Node[0] Epoch[0] Batch [250]	Speed: 111228.93 samples/sec	Train-accuracy=0.878719
2016-01-11 19:47:04,100 Node[0] Epoch[0] Batch [300]	Speed: 163265.02 samples/sec	Train-accuracy=0.889948
2016-01-11 19:47:04,146 Node[0] Epoch[0] Batch [350]	Speed: 139103.03 samples/sec	Train-accuracy=0.899196
2016-01-11 19:47:04,189 Node[0] Epoch[0] Batch [400]	Speed: 148079.44 samples/sec	Train-accuracy=0.905430
2016-01-11 19:47:04,238 Node[0] Epoch[0] Batch [450]	Speed: 132316.34 samples/sec	Train-accuracy=0.910295
2016-01-11 19:47:04,255 Node[0] Epoch[0] Train-accuracy=0.912059

However, still got the same error in python IDE. I am using anaconda and spyder. Any idea?

Maybe there are two python versions installed.

@sxjscience I also suspect it has something to do with python IDE, but can confirst only one installed on my machine, and one python version as well.

Hi,I am on OS X 10.10.4.I installed mxnet with cpu only(have no gpu),however met the same error with you.Did you meet the same error with USE_CUDA = 0 ? I did not install cuda.

I have the same problem:
ImportError                               Traceback (most recent call last)
 in ()
----> 1 import mxnet as mx
2
3 def to4d(img):
4     return img.reshape(img.shape[0], 1, 28, 28).astype(np.float32)/255
5
/home/d/mxnet/python/mxnet/init.py in ()
5
6 from .context import Context, current_context, cpu, gpu
----> 7 from .base import MXNetError
8 from . import base
9 from . import ndarray
/home/d/mxnet/python/mxnet/base.py in ()
8 import atexit
9 import numpy as np
---> 10 from . import libinfo
11
12 all = ['MXNetError']
ImportError: cannot import name libinfo

I got a similar problem:
>>> import mxnet
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/lustre/home/eefxz/mxnet/python/mxnet/__init__.py", line 7, in <module>
    from .base import MXNetError
  File "/lustre/home/eefxz/mxnet/python/mxnet/base.py", line 12, in <module>
    from . import libinfo
ImportError: cannot import name libinfo

When I run make -j2 && cd python && python setup.py install, there is no error information raised by the machine.
Other information:
mxnet (has been updated to the latest)
cuda/8.0
cudnn/5.1
CentOS/7.3.1611
anaconda/2 (python 2.7.12)
mkl/11.3

Any ideas ?


add option to allow standard users on W7 Home Premium to be logged into remotely

With rdpwrap it's possible to make remote connections to a Windows 7 Home Premium Edition, even though RDP is normally not available on such systems.
Unfortunately, only admin users are permitted to receive remote connections. Even with a retrofitted gpedit.msc and making the needed security adjustments, to also let standard users accept remote connections, it did not work.
It would be great if rdpwrap made this configurable in the program. I have no idea how to implement this and have a strong feeling this is totally out of scope for the project, but atleat I wanted to suggest it.

See #107.


README in yeoman generator samples is misleading

The README.md generated by yeoman needs to address proper use of CLI tooling, prereqs, etc.
The documentation is scattered and only covers a happy path in .NET. There is no explicit deployment documentation that addresses how to deploy a JavaScript or TypeScript bot. Developers need a single document that covers end-to-end deployment in each language. This deployment document should also address the architecture of the deployed solution. As of now the msbot cli tool spins up several resources with little mention of what they do/how they are connected. Developers (especially those new to Azure) will have a hard time justifying the value of these resources without understanding how they work together. To make matters worse, scattered documentation (e.g. README in yeoman generator) is misleading.
Additional context
Add any other context about the problem here.
[bug]

@Kaiqb are you planning a "How to deploy a JavaScript bot to Azure" topic?


Ivp with and without adjoint method.

Hi, I want to know if torchode always doing integration with adjoint method?
How about integration without adjoint method, so that no gradient is recorded?
I did not find information about it in the document. I wonder what's the best practice if without "AutoDiffAdjoint".
Or should I give a drift function where "with torch.no_grad()" used?

Hey, just wrap the solve to in no_grad, i.e.
with torch.no_grad():
    sol = solver.solve(...)


Simplify client/server code

The current manual serialisation is too complex. This needs to be refactored to use Thrift or Protobuf kind of approach with more automated checks for packet types, etc. Instead of handlers a controller pattern should be implemented.

This is linked to #49

The number of packets has been limited and new protocol.c has been implemented


test: add a size limit test

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Test coverage

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #4272 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #4272   +/-   ##
=======================================
  Coverage   92.28%   92.28%
=======================================
  Files          52       52
  Lines        1712     1712
  Branches      447      447
=======================================
  Hits         1580     1580
  Misses        126      126
  Partials        6        6

Continue to review full report at Codecov.

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

I was thinking about such test, although wanted something that will show difference between new size vs old size :)
What the advantage loading page and getting js instead of getting sizes of *.js inside .nuxt/dist

@aldarund That'd be achievable with the current test as well, we'd only need to log the results. (Though they'd be hidden on CI anyway, so maybe an extra CI task would be good here).
Taking the .js files from the folder will result in roughly the same numbers (169.9 kB for the folder). But as they don't include the "initial page load"/the rendered HTML I decided to do it that way so we can catch potential size increases in base Nuxt components.

@manniL yes, if we have current size results we obv can compare them :) I saw it in react github
https://github.com/facebook/react/pull/14119
But it dont work, their prev size always zero :)

@aldarund Ah, a bot! Nice idea
We could write our own for the scenario (which could be configurable/adaptable)  🤔


Check Quarkus provided fabric8 version

Given that everything has been merged upstream we should be able to perform this last remaining check:
https://github.com/quarkiverse/quarkus-operator-sdk/blob/c2c17dd01b562984f6bfb5eead70d8f576374a73/core/deployment/src/main/java/io/quarkiverse/operatorsdk/deployment/OperatorSDKProcessor.java#L159-L160

This won't be available for consumption before 2.17, I think, though…

possible, opened the issue to avoid forgetting about 🙂


Improve error logs in IDP REST API flows.

Is your suggestion related to an experience? Please describe.
In the wso2carbon.log there are messages with String templated values.
Ex: Note%s below sample
[2020-09-29 11:29:05,144] [4991fae6-525f-40ee-bc3f-c8e275b58fd9] ERROR {org.wso2.carbon.identity.api.server.idp.v1.core.ServerIdpManagementService} - errorCode: IDP-65034 | message: Server encountered an error while updating the identity provider outbound connector for identifier %s. org.wso2.carbon.idp.mgt.IdentityProviderManagementException: Error occurred while updating Identity Provider information  for tenant 1

Describe the improvement
Ideally, the template should be replaced with the corresponding value.
Additional context
In this line errorEnum.getDescription() is sent as it is without being formatted.
But for the API response, the message is formatted.

Fixed in https://github.com/wso2/identity-api-server/pull/244


Package needs a new name

Turns out there is https://cran.r-project.org/web/packages/qra/index.html
How about qrensemble?

Should probably do this today to avoid issues at the workshop tomorrow to avoid people installing (or having installed) the CRAN package.

agree - do we want to try and have any name linkage between this and stackr?

I was wondering about this, also in the light of the terminology I used in https://nfidd.github.io/nfidd/sessions/forecast-ensembles.html  (e.g. stackr could become mixensemble) - one way or the other this would be good to match.
But there's also a larger conversation to be had about whether this should be just one package (which we currently promise in the README) where using a mixture or quantile average is something the use chooses from a single entry point.


Unable to push to repository

Hi, I've upgraded plugin to v.1.0.7 and know I'm unable to push to repository.
git push -u origin master returns this error:
FATAL: W any zabbix-service-dashboard redmine_a_tatarnikov_1389767240_850295 DENIED by fallthru
(or you mis-spelled the reponame)
fatal: The remote end hung up unexpectedly

I've added this key to my redmine user, and redmine user is a project administrator.

I've solved this by deleting all keys for my user.
After recreating keys the problem has gone.


Filesystem Session

Is there any example of doing a filesystem Session?

Support of file based sessions is not bundled. You can implement a custom
backend for that.
On Sun, Nov 7, 2021, 00:23 Landon Patterson @.***>
wrote:

Is there any example of doing a filesystem Session?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alex-oleshkevich/starsessions/issues/6, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE3HSFDU4HVYKEJ3BT3TZ3UKWTEZANCNFSM5HQASHPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Additional documentation folders & readmes

Plus Font Awesome 5 config info and files.

This PR is great!

Thanks! I'm glad you like.

Oh, I didn't make any updates to the main readme... Maybe you could just add a link so people know the documentation is available?

I already did that :)
On Sun, 29 May 2022 at 10:05, Heather Floyd @.***>
wrote:

Oh, I didn't make any updates to the main readme... Maybe you could just
add a link so people know the documentation is available?
—
Reply to this email directly, view it on GitHub
https://github.com/skartknet/Iconic/pull/14#issuecomment-1140346457, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM3ODT7JTYBBRYNGQD2E6LVMKYGFANCNFSM5XG2C2CA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Oh, looks great!


[2.11][2.13] Optimized the code for faster processing. Updated the readme. Corrected spell errors in UML

I approve


Fix some basic lint issues in vitess/py.

@michael-berlin Lint-compliance changes, from dumbunny/vitess branch lint0 to youtube/vitess master.

@michael-berlin I don't understand "This branch has conflicts that must be resolved". What are these conflicts?

It means your change has conflict with current master branch. Do a "git rebase master" to resolve conflict locally. It is pretty much like you and someone else touches same file and the other guy has merged change into master first, so you have to accommodate the change in your branch...

I haven't figured it out. I might abandon this and make another branch and pull request.

Noo, don't do that :-)
I recommend that you merge the master into your feature branch at this point. Therefore, do as follows:
# If you haven't committed all your pending changes yet, stash them away.
# We do this because you must not have any pending changes before you switch branches.
# (Stash is an extra place in Git where you can temporarily stash away changes.  Check git status. It will tell you.
git status
# If there are conflicts, you can edit the files manually to resolve the conflicts.
# In order to tell Git that you have resolved the conflicts, follow its instructions in "git status".
# It probably asks you to "git add" the changed files and then "git commit" them. This way Git knows that you resolved the conflicts.
# Alternatively, you can run git mergetool to resolve the conflicts. I've configured my Git setup to use p4merge. That brings up a super nice UI where it's very easy to fix merge conflicts.
git mergetool

# At this point, you have merged the latest changes from master into your feature branch.
# What's left is to get your pending changes back from your stash:
git stash pop
# Here conflicts may occur again. If that's the case, you have to resolve them manually e.g. with "git mergetool" or by editing the files manually.

Let me know how that goes. You don't need to use the stash when you commit all pending changes before switching branches.
Regarding rebase vs. merge: This site seems to have good explanations for it. I recommend to read it: https://www.atlassian.com/git/tutorials/merging-vs-rebasing/conceptual-overview
For your workflow, I would say the following:

if your pull request hasn't received any comments yet, rebase your feature branch and force push new changes. In that case, previous commits get overridden in the pull request.
If your pull request has already received comments, I recommend to merge the master branch instead. This way, comments on existing commits won't be hidden. Also, the reviewer better sees which  commits they already reviewed and which are new.

$ git remote -v
origin	git@github.com:youtube/vitess.git (fetch)
origin	git@github.com:dumbunny/vitess.git (push)
upstream	git@github.com:youtube/vitess.git (fetch)
upstream	git@github.com:youtube/vitess.git (push)
Notice that I changed my origin to fetch from youtube/vitess.git and push to dumbunny/vitess.git. This seems to have unstuck me, but it is different from @michael-berlin's setup.

@dumbunny Yes, please change your remote setup such that origin fetch points to your "dumbunny" fork again.
You were both right that the changes for master were not pulled from "upstream", but from "origin".
Jeff's instructions can be simplified by letting master track the official repo and not the fork:
$ git branch master --set-upstream-to upstream/master
You can see the correct tracking in branch -vv
$ git branch -vv | grep master
* master                                  a1a9b57 [upstream/master] Merge pull request #1028 from michael-berlin/netutil_fixes

From now on, you can always omit "upstream" when pulling the master:
$ git checkout master
$ git pull upstream

With the instructions in my last comment, you can omit "upstream" from a "git pull" while in "master".
With the instructions below, we also achieve that a "git push" no longer requires to specify a remote target.
$ git config remote.pushDefault origin
$ git config push.default current

The first command ensures that "git push" will always use the "origin" remote when nothing else is specified. This is great for two reasons:

it's a safe guard against accidental "master" pushes. If you would accidentally push the "master" branch now, it would push to "origin" now and not to "upstream" anymore. (And that despite the fact that we just changed "master" to track "upstream".)
When you push a feature branch now, it will automatically set up the tracking.

When you need to make changes, you would just run git push again.
For completeness: The second command git config push.default current is necessary to change Git's behavior how it associates local to remote branches. "current" means that the local and remote branch name will be the same. We have to set this explicitly because the default mode "simple" is more advanced and does not work with the "pushDefault" setting from the first command.

Merging.

(all edits addressed).


Implement Developer Settings/Switch to Test Env in BC Wallet Regression Tests

As of build 575 the app pushed to Sauce Labs is set to the Prod environment. The settings/Developer mode is used to switch to Test. Until this is implemented all Credential issuance and Proof test will fail in the nightly regression cycle.

Should this be assigned to a developer?

Nope. I have to update the tests to always switch to TEST env for every test in the Automated regression suite.


Compile API Documentation

As a team member, I need to compile documentation on the APIs involved in the receipt scanning feature for the high-level system overview. This documentation will provide a comprehensive understanding of the external interfaces and integrations.
Acceptance Criteria

Identify all APIs integrated into the receipt scanning feature.
Document the purpose and functionalities of each API.
Specify the communication protocols and data formats used by the APIs.
Ensure the documentation is organized and easily accessible.
Include any authentication mechanisms or security considerations associated with the APIs.

A draft fot this documentation is in https://github.com/tiki/apps-receipt-capacitor/issues/441#issuecomment-1867703337


Epic: implement SCRAM for e2e auth

We want to avoid storing any kinds of passwords in the control plane or proxy. We could use secrets storage like Vault or network auth like Kerberos. But not having passwords in a first place is a better option.
Current thinking:

Usual login is done via SCRAM, where proxy performs MITM. Same user/password is used to establish user<->proxy and proxy<->postgres connection.
Web console generates special user account with random password which is stored in the browser cookie. Cookie is encrypted so user can't see it and console will see that password only during the request.
We setup two differently configured proxy services for link-based auth and password-based one (instead of looking on @zenith suffix)
Postgres management operations are done through zenith_ctl http api (and that is fixed set of operations, arbitrary query is not allowed)
Onboarding login/creation is done by "trust" to the proxy hostname. Here login process itself involves github oauth, which should be good enough. However, this login method should be optional and we should promote it usage only for dev clusters. Optionally we can pass github oauth token to the proxy and made postgres validate it. That way we will protect ourselves from connecting to a wrong cluster.

Sub-tasks:

[x] neondatabase/cloud#373
[x] #918
[x] neondatabase/cloud#532
[x] neondatabase/cloud#478
[x] neondatabase/cloud#1182
[ ]  neondatabase/cloud#1314

#431
could be related, but is unconfirmed

reopened to track the task for deployment of this to prod


Return RDS Instance Endpoint field in the ConnectionSecret without port

Description of your changes

Fixes #778: Previously the ConnectionSecret endpoint field contained the hostname including the port.
This is not expected behavior and makes the ConnectionSecret unusable by downstream consumers of the Secret, like
https://github.com/crossplane-contrib/provider-sql, which expect the endpoint and port to be in separate fields.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
TBD

 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.

@l0wl3vel, if you are considering taking this PR forward, could you please fill out the section on how you tested it and have it ready?

@turkenf The fields in the commit looks correct, I had checked them via simple bash script:
DB_NAME=test-rds-postgresql
diff -u <(kubectl get instances.rds.aws.upbound.io $DB_NAME -o jsonpath='{.status.atProvider.endpoint}'; echo) <(kubectl get instances.rds.aws.upbound.io $DB_NAME -o jsonpath='{.status.atProvider.address}'; echo)

--- /dev/fd/11	2023-10-04 16:50:09
+++ /dev/fd/12	2023-10-04 16:50:09
@@ -1 +1 @@
-test-rds-postgresql.xxx.eu-west-1.rds.amazonaws.com:5432
+test-rds-postgresql.xxx.eu-west-1.rds.amazonaws.com

original error message:
 Warning  CannotObserveExternalResource  2m52s (x21 over 58m)  managed/database.postgresql.sql.crossplane.io  cannot select database: dial tcp: lookup test-rds-postgresql.xxx.eu-west-1.rds.amazonaws.com:5432:5432: no such host

P.S. I have the same issue, so please approve this PR as soon as possible

The current state of the endpoint definition seems to be consistent with the underlying provider documentation, and changing this will cause a breaking change. It seems more logical to fix this on the provider-sql side.
I am closing PR because it does not make sense and there has been no action for a long time.

@turkenf Unfortunately, provider-sql seems to have little drive for changes.

@turkenf It looks like the SQL provider maintainers are of opposite address https://github.com/crossplane-contrib/provider-sql/pull/154#issuecomment-2506020178

FYI https://github.com/crossplane-contrib/provider-upjet-aws/issues/1578


Move telemetry down in README

I've found it very annoying that the very top of the cloud-nuke README file is a note about telemetry. Being up front about this is important, but it shouldn't appear before the README even introduces what the tool is or does!
This PR moves the telemetry note below that, but still above install, so it should still get high visibility.

Description
Fixes #000.

TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[ ] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[ ] Include release notes. If this PR is backward incompatible, include a migration guide.
[ ] Attention Grunts - if this PR adds support for a new resource, ensure the nuke_sandbox and nuke_phxdevops jobs in .circleci/config.yml have been updated with appropriate exclusions (either directly in the job or via the .circleci/nuke_config.yml file) to prevent nuking IAM roles, groups, resources, etc that are important for the test accounts.

Release Notes (draft)

Added / Removed / Updated [X].
Migration Guide

Thanks for the review! Merging now.


SAKIII-4629 Allow embedding of collections in pages

Jira: https://jira.sakaiproject.org/browse/SAKIII-4629
Previous: https://github.com/sakaiproject/3akai-ux/pull/1555

Squashed and merged


changed base template in email.html

The email page appears without styling. To fix this i changed the base template from {% extends "account/base.html" %} to {% extends "base.html" %}.
account/base.html was probably deleted in previous commits and email.html was not updated.

Thank you!


Dkeras on a local machine (Intel® Xeon(R) Gold 6128 CPU @ 3.40GHz × 12 ) takes for ever

I have a model based on keras and it takes about 25 seconds for inference. I wanted to speed this up and implemented dkreas but I didnt get the performance improvment that I was expecting. Instead it takes for ever. Do you know why this happen or have I done anything wrong?
from dkeras import dKeras
import ray
ray.init()
model = dKeras(my_model, init_ray=False, wait_for_workers=True, n_workers=None)
segmented = model.predict(data)
model.close()

n_workers=None will cause an error as you don't initialize any workers, try n_workers=10.

I have tried by setting n_workers to 4, 10 and None but it didn't help. Instead I get the following warning which I think it wont be a problem.
(pid=128326) Using TensorFlow backend.
(pid=128326) /home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:523: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
(pid=128326)   _np_qint8 = np.dtype([("qint8", np.int8, 1)])
(pid=128326) /home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:524: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
(pid=128326)   _np_quint8 = np.dtype([("quint8", np.uint8, 1)])
(pid=128326) /home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:525: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
(pid=128326)   _np_qint16 = np.dtype([("qint16", np.int16, 1)])
(pid=128326) /home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
(pid=128326)   _np_quint16 = np.dtype([("quint16", np.uint16, 1)])
(pid=128326) /home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
(pid=128326)   _np_qint32 = np.dtype([("qint32", np.int32, 1)])

Those warning shouldn't be a problem. As a quick test, what happens when you run a keras.applications ResNet50?

I get the following error when I run keras.applications ResNet50
TypeError: predict() got an unexpected keyword argument 'callbacks'
2020-03-11 10:10:24,965	ERROR worker.py:998 -- Possible unhandled error from worker: ray::dkeras.workers.worker.worker_task() (pid=22884, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 452, in ray._raylet.execute_task
  File "/home/bisrat/anaconda3/envs/cpu_fast/lib/python3.6/site-packages/dkeras/workers/worker.py", line 109, in worker_task
    use_multiprocessing=use_multiprocessing)
TypeError: predict() got an unexpected keyword argument 'callbacks'


Reuse changelog action

Acceptance criteria

[ ] Put a workflow in each repo that calls the changelog action

per #1057

Minor update to changelog action to append rc0 to prereleases. Corresponding update to release workflow to not append the rc0
Changelog action should increment the rc number if we're on a release branch


Tables

Users
Products
Wishlists

Further tasks

Need to see which fields we need for each table.
Which field should be the Primary key
These are basic tables, what tables do we need more of in the future? (Reviews.. etc)

You don't need a table for Users or Products. The Customer service and the Products Service will take care of these. You only need to create a field for the user id and a field for the product id. That is all you need for get any information you may need from those other services as needed.
You will need a two tables however, one for Wishlists and one for Items that are in the wishlist. That should be the focus of your service.


Remove EmptyApp tests as its covered eslewhere

Fix Issue # .
Changes
(Please provide a brief description of the changes here.)
Checklist

[ ] I ran Unit Tests locally.
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #
[ ] Changes in public surface reviewed

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:

FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Notes for reviewers:

We support comment build triggers

/AzurePipelines run will queue all builds
/AzurePipelines run <pipeline-name> will queue a specific build

Need to remove it from the Everything.sln as well


docs: remove non rendered emoji

Description
This doesn't render in catalog so removing it:
Catalog

GH:

Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This PR is included in version 7.0.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix horizontal scroll

the .w-screen class on the header produces a tiny bit of extra width space that makes the page scroll-able horizontally.
platform: Firefox 124 on Linux

@sudanchapagain does this still occur for you?

Yes. I assumed it was because of my always show scrollbar setting but i created a new profile and opened the site. Same issue.
this is on: Firefox developer version 129.0b3 Windows.
I just opened it on chrome, same issue.
It's the issue with .w-screen on the parent div of header and the anchor tag for skip to main content. Does this not happen in your device? It should no?

Found the solution, add max-width: 100% on the div.
https://stackoverflow.com/questions/23367345/100vw-causing-horizontal-overflow-but-only-if-more-than-one


Update static analyse rules

Codecov Report

Merging #9 (384d5f5) into 1.x (b7179e0) will decrease coverage by 2.44%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
see 3 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


formatting on save

when set onSave is true, i save some file, the file formatted, but file status is "change" i need to save again,Then Formatting Again And Again....

Thanks for taking the time to open this issue.
Unfortunately there is no fix for this as of now. Microsoft is working on a preSave event for VSCode, which would prevent this issue from cropping up. It is currently in the backlog of the September milestone.
For now, you can work around this issue by creating a custom command that triggers this extension's fix functionality.
In order to avoid future confusion, I will add this to the extension's docs and readme.

You can add Format Code to Right click menu like  Native Command Ctrl+F

That's a great suggestion. Do you happen to know how the when condition should be setup to allow file extensions other than php?
I'm trying something like this:
"contributes": {
    "commands": [
      {
        "command": "phpformatter.fix",
        "title": "Format PHP",
        "icon": {
          "light": "./images/fix-icon-light.svg",
          "dark": "./images/fix-icon-dark.svg"
        }
      }
    ],
    "menus": {
      "editor/context": [
        {
          "when": "resourceLangId == php || vscode.workspace.getConfiguration('phpformatter').get('additionalExtensions', []).indexOf(resourceLangId) > -1",
          "command": "phpformatter.fix"
        }
      ]
    }
}

htm* or  tpl  ?

You Can  add  a  item like Format As  PHP when  The Extension may has Php code,Or Just Use  in php  file, another file  neednt  Format  such as  in  html or  template :  `

Yes, that would indeed be unnecessary. However, what I meant is that it would be even less necessary to have the context menu item at all, it the current file is not a PHP file.

Last Night,I  use PHP Function token_get_all to  Compile template file  ( Copy from  Laravel ) ,I guess  Maybe Can  use something like this?

@Dickurt Since version 1.6 VS Code supports the onWillSaveTextDocument-event. Also there is a new editor.formatOnSave-setting that will invoke registered formatters on save, see #5

Resolved with conversion to TypeScript in latest milestone.


spfx 1.13 and PropertyFieldListPicker context

Category

[ ] Enhancement
[x] Bug
[ ] Question

Version
"@pnp/spfx-property-controls": "^3.6.0
spfx 1.13.1
Expected / Desired Behavior / Question
when using latest spfx with latest spfx property controls, it should be possible to pass the webpartContext property.
Observed Behavior
when using latest spfx with latest spfx property controls, following error appears, when I want to use the PropertyFieldListPicker:
Type 'WebPartContext' is missing the following properties from type 'BaseComponentContext': _isServedFromLocalhost, isServedFromLocalhost
Do you have the same behaviour?
Thanks....

Yeah we've noticed the same thing. Does 3.6 require a newer version of spfx? Should this be detailed in the docs?

Was discussed at #434.


Add sample for pubsub_update_topic_schema

For adding a Node sample updating a topic schema.
ref: 266566558

Closed in favour of https://github.com/googleapis/nodejs-pubsub/issues/1741


Add additional indel realigner arguments

Currently, the transform command from the CLI only uses the default indel realigner arguments.

Closed by #427.


Python ret

I think reticulate will be needed other places too and its not too big of a package so I already added it to the main docker image and you can pull it and use it if you need (that way you don't have to start doing dockerfile management on this PR just for this one little thing.


pythonPackages.apio: init at 0.4.0

Motivation for this change
Add new Python library.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This seems to be stalled and should probably be closed. @FRidh

The expression looks good so it could go in after a rebase, but yes, it needs a bit more pushing from the PR author.

Nothing happening so closing.


Is sudoku valid

🚀 Feature
Is sudoku valid is a problem that to be solved using hashing in c++
Have you read the Contributing Guidelines on Pull Requests?
yes
Motivation
Competitive coding
Pitch
C++ can be used to solve this problem using hashing.

/assign
please assign me this issue

Why it's not assigned to me, I have created the issue as a gssoc 21 participant.

@Debashish-hub I assigned you.

I am a participant of GSSOC'21
I would like to contribute in this

@einhawking Someone is already working in this issue. If you want to choose another language and open your issue.


Angular 14

First of all thanks for that little package.
Is it possible to upgrade your package to support angular v14?
At this time i get an error if i want to upgrade my projects dependencies:
Could not resolve dependency: npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@ultimate/ngx-fullscreen npm ERR<EMAIL_ADDRESS>from the root project
The package works fine with v14 but the peer dependency errors are a bit annoying.

I created a pull request https://github.com/ultimatecourses/ngx-fullscreen/pull/4


Tammy would like the "important" output variables to be included in the generated terraform template for use by other terraform layers

Output variables in the template get printed out for the user when the apply finishes. This would be a good way to document what was provisioned by the automation.
Iascable should generate an output.tf file in the terraform template that includes the "important" outputs

Changed the title back because Tammy would like to print a listing of the resources that were provisioned when terraform automation has completed is actually a different requirement


[youtube] <urlopen error [SSL: SSLV3_ALERT_HANDSHAKE_FAILURE] sslv3 alert handshake failure (_ssl.c:598)>

ERROR: unable to download video data: urlopen error [SSL: SSLV3_ALERT_HANDSHAKE_FAILURE] sslv3 alert handshake failure (_ssl.c:598)

Hello, i have this SSLV3_ALERT_HANDSHAKE_FAILURE error when running the Windows executable.  Is it compiled with an old version of python or something?  How can I get it to download thumbnails (soundcloud in my case)?

@gak1234 I guess you have encountered the same problem as #5366. Yes soundcloud thumbnails is available only with Python 2.7.9 and 3.x, while the provided youtube-dl.exe is bundled with Python 2.7.8. You can try to download the whole source zip and run youtube-dl with:
python.exe youtube-dl/__main__.py

Next time please post your problem in a separate issue. It's likely that the problem is quite different for different sites. Also, remember to include the full verbose log

@MikeCol Could you open a new issue and paste the full verbose log? It helps us on tracking problems.

@yan12125 Sorry, will open a separate issue next time.  I was able to work around the soundcloud thumbnail SSLv3 issue by installing Python 3.4.x and running:
pip install --upgrade youtube_dl
this command created \Scripts\youtube-dl.exe (only 90kb ?) with a python icon.  That EXE has no problem with SSLv3 thumbnail soundcloud URLs!

I had the above problem too on Linux Mint.  I installed through apt-get, but thanks to @gak1234's suggestion, I removed the apt-get version and used pip3 install --upgrade youtube_dl and things are working well again.  It might be worth investigating Ubuntu/ Linux mint package version Inst youtube-dl (2014.02.17-1 Ubuntu:14.04/trusty [all]) to see if somethings broken.


Add pypy 5.0.0

http://morepypy.blogspot.com/2016/03/pypy-50-released.html :-)

v20160310 coming to homebrew in homebrew/homebrew#50001


Scanning

Coverage increased (+0.19%) to 94.72% when pulling 8a736723987a4240f5b584c58257a81504026eb9 on isocolsky:scanning into c42b4a46fb809d9dd4241a58e46cf5902679166b on bitpay:master.

Well done!


remove-get-static-props: always use arrow function

 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.

Current dependencies on/for this PR:

main

PR #474   👈

This stack of pull requests is managed by Graphite.


Code metrics fail if a project has classes inheriting from Cimbalino.Toolkit.Behaviors.Behavior

Error:
Message: An error occurred while calculating code metrics for target file 'xxx' in project xxx. The following error was encountered while reading module 'Cimbalino.Toolkit': Assembly reference cannot be resolved: Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35.

Tested on both WPA81 and Win81. Works fine if you build from source. Seems to happen only when using the nuget package.

So from what I can see here, you run the code-metrics in Visual Studio and it gave you this error message?
Microsoft.Xaml.Interactivity assembly is now added through the XAML Behaviors Extension SDK, it might be related to that, but I'll run a couple of tests to confirm!

I'm closing this as I haven't been able to reproduce the issue while using v2.0.0, but please fell free to re-open if you do!


ANGLE gives hard-to-understand error about control paths

When applied to this shader:
https://github.com/mc-imperial/shader-compiler-bugs/blob/master/angle/front-end/control-paths/shader.frag
ANGLE in Google Chrome Version 57.0.2987.110 (64-bit) gives the following error:
"
Failed to link shader program.
C:\fakepath(33,29-64): error X3507: '_map': Not all control paths return a value
Warning: D3D shader compilation failed with default flags. (ps_5_0)
Retrying with skip validation
C:\fakepath(33,29-64): error X3507: '_webgl_3543ec1d70156653': Not all control p
aths return a value
Warning: D3D shader compilation failed with skip validation flags. (ps_5_0)
Retrying with skip optimization
C:\fakepath(33,29-64): error X3507: '_webgl_3543ec1d70156653': Not all control p
aths return a value
Warning: D3D shader compilation failed with skip optimization flags. (ps_5_0)
Failed to create D3D shaders.
"
Simplifying the shader further makes the error disappear.

Reported to ANGLE here:
https://bugs.chromium.org/p/angleproject/issues/detail?id=1946


Advanced Terraform checking

Running TFLint is nice and all, but we can do more:

check proper formatting
let Terraform validate the config
try to init the config
finally also run TFLint

And it's producing a nice job summary.
Examples:

https://github.com/wiebeck/tflint-renovate-1/actions/runs/5066461533
https://github.com/wiebeck/tflint-renovate-2/actions/runs/5066618331

@HardTy Shall we do anything about the linter issues?


chore: add plumbing to allow customer to enable reporting OT metrics

This change prepares for a future change that will actually consume the flag to enable/disable reporting of OpenTelemetry metrics.

No, as yet the flag is not used. Next checkin factor out the metrics reporting logic and will begin using it.


test: repair the cross_module_derivative_attr test

Convert to dynamic libraries, fix the run lines, remove the workaround
for the TBD validation.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @marcrasi @dan-zheng @ematejska

@swift-ci please smoke test


Change stanzas in English to match Arabic

The English translation has additional blank lines, creating stanzas where they don't exist in Arabic. These blank lines can simply be removed.

Pretty sure these are all fixed


Fix utils::string::ssplit() to handle delimiter in the end of string

This closes #1743.
Also, just for the record: the action-tnf-base64.json test includes both encoding and decoding tests. Decoding part was modified in this PR to match valid string without leading \n. Encoding part still works due to the fact that regression_test.cc adds extra \n to every constructed body:
https://github.com/SpiderLabs/ModSecurity/blob/v3/master/test/regression/regression_test.cc#L51
I suppose changing this behavior will require adjustments in other tests as well, and it is not directly related to this PR and linked issue, so I'd leave it as is for now.

Merged! Thanks.


Could not find method nexusPublishing() for arguments on project ':library' of type org.gradle.api.Project.

Hello
I am getting this error ... how can I fix it? 🙏🏼
Caused by: org.gradle.internal.metaobject.AbstractDynamicObject$CustomMessageMissingMethodException: Could not find method nexusPublishing() for arguments [publish_mavencentral_b59r6yy6puwpa4f9u3qxqhqlx$_run_closure5@4a07f25b] on project ':library' of type org.gradle.api.Project.

Having your configuration (or at least its relevant fragments) would make it easier to spot the answer :-)

could it be that it was missing this? apply plugin: 'io.github.gradle-nexus.publish-plugin'

Definitely that could be one of the reasons.

it worked after adding it 😅  ... would be nice to add that to the readme, as it isn't mentioned there

It is: https://github.com/gradle-nexus/publish-plugin#applying-the-plugin
Applying the plugin always occurs and then you can publish to Maven Central (Sonatype Nexus) or to any other Nexus instances (the next two paragraphs).

I think it's because I'm using an "old format" like this:
apply plugin: 'io.github.gradle-nexus.publish-plugin'
buildscript {
  repositories {
    classpath "io.github.gradle-nexus:publish-plugin:1.0.0"
  }
}

Yes, in that case you need to apply the plugin manually.
I wonder, if there is a corner case in the "new" plugin applying scheme in Gradle that you keep using the old one?

I just don't understand it and struggle while trying to migrate, plus I haven't found the need to do so, so 🤷🏼♂😬😅

also... would you happen to know why this happens in a GitHub Action?
Task :closeAndReleaseStagingRepository SKIPPED
You can check the action code here, and the gradle configuration here.
Thanks

A blind shot - you have a SNAPSHOT version in that build?

No, :closeAndReleaseStagingRepository is SKIPPED if you don't have any staging repositories configured, but you seem to have one. Could you call closeAndReleaseSonatypeStagingRepository instead?

also, when trying to run closeAndReleaseSonatypeStagingRepository, this happens:
Execution failed for task ':closeSonatypeStagingRepository'.
> No staging repository with name sonatype created

Not sure if this helps, but this is what I see on sonatype site

oh, and I'm not sure what you mean with SNAPSHOT version 😬🤦🏼♂

That confirms that initializeSonatypeStagingRepository task wasn't used (the logs say the same). Could you call publishToSonatype closeAndReleaseStagingRepository --info instead?

oh, and I'm not sure what you mean with SNAPSHOT version grimacing🤦🏼♂

With version 0.8.0-SNAPSHOT staging repositories are created nor closed.

oh, I'm using v 1.0.0 of this plugin

oh, I'm using v 1.0.0 of this plugin

The version of your project :-). You can call ./gradlew version before publishing to get it.

I am getting this:
* What went wrong:
Task 'publishToSonatype' is ambiguous in root project 'Frames'. Candidates are: 'publishAllPublicationsToSonatypeRepository', 'publishReleasePublicationToSonatypeRepository'.

Both are good, but the second is exactly for the publication that you configured.

This is the output running with --info
> Task :library:publishReleasePublicationToSonatypeRepository
Task :library:publishReleasePublicationToSonatypeRepository in app Starting
Caching disabled for task ':library:publishReleasePublicationToSonatypeRepository' because:
  Build cache is disabled
Task ':library:publishReleasePublicationToSonatypeRepository' is not up-to-date because:
  Task has not declared any outputs despite executing actions.
Publishing to repository 'sonatype' (https://s01.oss.sonatype.org/service/local/staging/deploy/maven2/)
Uploading Frames-337.aar to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337.aar
Uploading Frames-337.pom to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337.pom
Uploading Frames-337-sources.jar.asc to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337-sources.jar.asc
Uploading Frames-337.aar.asc to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337.aar.asc
Uploading Frames-337-sources.jar to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337-sources.jar
Uploading Frames-337.pom.asc to /service/local/staging/deploy/maven2/dev/jahir/Frames/337/Frames-337.pom.asc
Resource missing. [HTTP GET: https://s01.oss.sonatype.org/service/local/staging/deploy/maven2/dev/jahir/Frames/maven-metadata.xml]
Uploading maven-metadata.xml to /service/local/staging/deploy/maven2/dev/jahir/Frames/maven-metadata.xml
Task :library:publishReleasePublicationToSonatypeRepository in app Finished
:library:publishReleasePublicationToSonatypeRepository (Thread[Daemon worker,5,main]) completed. Took 7.654 secs.
:closeAndReleaseStagingRepository (Thread[Daemon worker,5,main]) started.

> Task :closeAndReleaseStagingRepository SKIPPED
Task :closeAndReleaseStagingRepository in app Starting
Skipping task ':closeAndReleaseStagingRepository' as task onlyIf is false.
Task :closeAndReleaseStagingRepository in app Finished
:closeAndReleaseStagingRepository (Thread[Daemon worker,5,main]) completed. Took 0.0 secs.

This is the output running with --info

Can you remove that extra repository configuration section?
https://github.com/jahirfiquitiva/Frames/blob/master/scripts/publish-mavencentral.gradle#L98

removing that caused this to happen:
> Configure project :library
Evaluating project ':library' using build file '/Users/jahir/dev/jahir/android/frames/library/build.gradle'.
Creating configuration androidTestUtil
Warning: The 'kotlin-android-extensions' Gradle plugin is deprecated. Please use this migration guide (https://goo.gle/kotlin-android-extensions-deprecation) to start working with View Binding (https://developer.android.com/topic/libraries/view-binding) and the 'kotlin-parcelize' plugin.
WARNING: API 'variant.getJavaCompile()' is obsolete and has been replaced with 'variant.getJavaCompileProvider()'.
It will be removed in version 5.0 of the Android Gradle plugin.
For more information, see https://d.android.com/r/tools/task-configuration-avoidance.
To determine what is calling variant.getJavaCompile(), use -Pandroid.debug.obsoleteApi=true on the command line to display more information.
Configuring Java sources compilation with source level 1.8 and target level 1.8.
Configuring Java sources compilation with source level 1.8 and target level 1.8.
All projects evaluated.
Analytics other plugin to proto: Unknown plugin type io.github.gradlenexus.publishplugin.NexusPublishPlugin expected enum IO_GITHUB_GRADLENEXUS_PUBLISHPLUGIN_NEXUSPUBLISHPLUGIN
Analytics other plugin to proto: Unknown plugin type org.jetbrains.kotlin.gradle.internal.AndroidSubplugin expected enum ORG_JETBRAINS_KOTLIN_GRADLE_INTERNAL_ANDROIDSUBPLUGIN
Analytics other plugin to proto: Unknown plugin type org.jetbrains.kotlin.gradle.internal.AndroidSubplugin expected enum ORG_JETBRAINS_KOTLIN_GRADLE_INTERNAL_ANDROIDSUBPLUGIN
Analytics other plugin to proto: Unknown plugin type org.gradle.api.publish.plugins.PublishingPlugin expected enum ORG_GRADLE_API_PUBLISH_PLUGINS_PUBLISHINGPLUGIN

FAILURE: Build failed with an exception.

* What went wrong:
Task 'publishReleasePublicationToSonatypeRepository' not found in root project 'Frames'.

This suggests that GNPP (this plugin) doesn't correctly detect publishing configuration in an Android project. As a result, it doesn't enhance the publishing task to use the explicitly created staging repository and in general - doesn't work.
I don't have an Android SDK installed locally and I would prefer not to setup it. However, if you created/generated (or stripped down the current one) a totally minimal Android project which does nothing, but builds fine and publishes artifacts to Nexus (as your does), I/we could try to take a look at it to solve the problem with GNPP <-> Android project interaction. Otherwise, it could be hard to do anything about that :-(.
It would be best to publish it to a separate GitHub repo with set up GH Actions (to be able to play with it there) and after (hopefully) successful triage and fix, it could be eventually also converted to an automatic regression e2e tests (as we currently have for single and multi-project regular Java builds) to prevent that kind of problems also in the future. Does it sound good for you? :-).

oh, that looks like quite some work to be done ... I will create that project in the next days and let you know... thanks for the help

Maybe removing (most of) the source code from your current project + some dummy GH action which builds the project (and where it is possible to add later on secrets and try to push it to Nexus) will be easier that configuring it from scratch with a new generated project? Just choose the line of least resistance.

As the first several comments are not much related to the conclusion, I created #75 to cover better support for releasing Android projects. Closing this one to avoid confusion.

[[Solved] AbstractDynamicObject$CustomMessageMissingMethodException error in android
]([Solved] AbstractDynamicObject$CustomMessageMissingMethodException error in android
)


coq_8_13: 8.13.0 → 8.13.1

Motivation for this change
Bugfix release: https://coq.github.io/doc/V8.13.1/refman/changes.html#changes-in-8-13-1
cc maintainers @roconnor @thoughtpolice @Zimmi48
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114089 at e5ca0971 run on aarch64-linux 1

1 package built:

coq_8_13

1 suggestion:

warning: unclear-gpl
lgpl21 is a deprecated license, check if project uses lgpl21Plus or lgpl21Only and change meta.license accordingly.
Near pkgs/applications/science/logic/coq/default.nix:180:5:
    |
180 |     license = licenses.lgpl21;
    |     ^


Handler is being set to the Runtime

This just appears to an oversight, but the handler is being set to the runtime value. This is evident locally, but after looking at the README you see it in the example output too.
https://github.com/sparkleformation/sfn-lambda#example
...
"Handler": "python2.7",
...

It also appears that the Runtime is not being set as well which is required.

Fixed via #2. Thanks!


[PluggableDevice] New device type support in python front-end

Enhance python front-end valid device type check , making dynamically registered device type be a valid device type.

@penpornk @allenlavoie This PR is to enhance python front-end valid device type check for those dynamically registered device types, please help to review. Thanks very much.


Enhance TeamCity reporter

Enhance the TeamCity reporter:

Add build status messages
Better test failure detection

Sweet! Thanks so much for the patch :beers:


Connectivity callback prints exception trace in background thread

gRPC Python 1.19.0 (most likely in 1.20.0 too) on Linux (not platform specific)
We create a gRPC channel and immediately call subscribe on it passing a connectivity callback. At some point, we call close on the channel without having called unsubscribe beforehand. We see the following exception:
Traceback (most recent call last):
  File "threading.py", line 801, in __bootstrap_inner
    self.run()
  File "threading.py", line 754, in run
    self.__target(*self.__args, **self.__kwargs)
  File "../com_github_grpc_grpc/src/python/grpcio/grpcio-cpython-27/grpc/_channel.py", line 915, in _poll_connectivity
    time.time() + 0.2)
  File "external/com_github_grpc_grpc/src/python/grpcio/grpc/_cython/_cygrpc/channel.pyx.pxi", line 487, in cygrpc.Channel.watch_connectivity_state
  File "external/com_github_grpc_grpc/src/python/grpcio/grpc/_cython/_cygrpc/channel.pyx.pxi", line 349, in cygrpc._watch_connectivity_state
  File "external/com_github_grpc_grpc/src/python/grpcio/grpc/_cython/_cygrpc/channel.pyx.pxi", line 357, in cygrpc._watch_connectivity_state
ValueError: Cannot invoke RPC: Channel closed!```

Seems like the connectivity tag is seen on the completion queue, but the channel is closed when gRPC Python layer sees that tag, and thinks the user has called a method in an invalid closed state, as opposed to the underlying system.  The Channel.close() should make it so that callbacks should not be called once close() is received and no exception is thrown either.
cc @lidizheng @gnossen

Please note that the stack trace above is printed from a background thread and no exception is thrown in the main thread.
I investigated this a bit further trying to minimize the test case and I am less confident about the circumstances that leads to this are as I described. There's another possibility I am worried about, which is something is calling subscribe  in the background. However, the following is my understanding reading gRPC source code for _subscribe: it will not raise an exception even if channel is closed (which may or may not be the intended behavior), but it might cause an exception trace to be printed in the background thread. I suspect we either want an exception to be raised in the caller thread immediately and not start polling connectivity, or make it a no-op (no exception) but do not start watching either, and in either case we should not see that exception logged to console.

This issue https://github.com/grpc/grpc/issues/16290 may have some relevant info.

I made a minimal test case that consistently reproduces the problem:
from __future__ import print_function
import logging

import grpc
import time

import helloworld_pb2
import helloworld_pb2_grpc

def watch(state):
    print(state)

def run():
    # NOTE(gRPC Python Team): .close() is possible on a channel and should be
    # used in circumstances in which the with statement does not fit the needs
    # of the code.
    with grpc.insecure_channel('localhost:50051') as channel:
        channel.subscribe(watch, try_to_connect=True)
        stub = helloworld_pb2_grpc.GreeterStub(channel)
        response = stub.SayHello(helloworld_pb2.HelloRequest(name='you'))
        print("Greeter client received: " + response.message)
        #channel.unsubscribe(watch)  # works fine when uncommented

if __name__ == '__main__':
    logging.basicConfig()
    run()
    time.sleep(10)

Reopening as a tracking issue for adding a regression test.

@lidizheng and I just had a conversation about this general class of problem. When I pull in a library and see that it logs errors (without crashing) under normal operation, it leaves a bad taste in my mouth and I'm likely to have a poor opinion of it. Exceptions on threads besides the main thread are just one example of causes of a such a situation.
Our test suite should verify this behavior for all use cases currently under test and the effort to add a new  test case should be minimally. Ideally, we would be able to just check in @mehrdada 's repro case above and call it a day. Perhaps we should write a new test runner that performs this check and add a new kokoro job for it.


dbus: 1.12.8 -> 1.12.10

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/dbus/versions.
meta.description for dbus is: '"Simple interprocess messaging system"'.

Checks done (click to expand)

built on NixOS
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-daemon passed the binary check.
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-monitor passed the binary check.
Warning: no invocation of /nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-send had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-test-tool had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-update-activation-environment had a zero exit code or showed the expected version
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-cleanup-sockets passed the binary check.
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-run-session passed the binary check.
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-uuidgen passed the binary check.
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-launch passed the binary check.
6 of 9 passed binary check by having a zero exit code.
0 of 9 passed binary check by having the new version present in output.
found 1.12.10 with grep in /nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10
directory tree listing: https://gist.github.com/209eee495f578b2f8d0ed9772f4720ae
du listing: https://gist.github.com/10946d6fdff9619fe1d92992cb9bf8da

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 66ad6f81287a290380c23b4d8167f60f6361d400 && nix-shell --pure -I nixpkgs=. -p dbus

Success on x86_64-darwin (full log)
Attempted: dbus
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/iamn85z2x7ng6n5lxm8k9rsiy7qj8nj6-dbus-1.12.10-dev/lib
patching script interpreter paths in /nix/store/iamn85z2x7ng6n5lxm8k9rsiy7qj8nj6-dbus-1.12.10-dev
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/ajgj91srysadn268zc7rprrcdv78q547-dbus-1.12.10-lib/lib
patching script interpreter paths in /nix/store/ajgj91srysadn268zc7rprrcdv78q547-dbus-1.12.10-lib
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/kwxlw6fqw4k8msnnzfn9sagp65ylzj2g-dbus-1.12.10-doc
Moving /nix/store/sa6in8im3wd82r9923008s22j0bg762p-dbus-1.12.10/bin/dbus-launch to /nix/store/ajgj91srysadn268zc7rprrcdv78q547-dbus-1.12.10-lib/bin/dbus-launch
rmdir: failed to remove '/nix/store/sa6in8im3wd82r9923008s22j0bg762p-dbus-1.12.10/bin': Directory not empty
/nix/store/sa6in8im3wd82r9923008s22j0bg762p-dbus-1.12.10

Success on x86_64-linux (full log)
Attempted: dbus
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/xh9sm8nqy07ajlc7va8rs9h9j0h5dsi1-dbus-1.12.10-lib/lib
patching script interpreter paths in /nix/store/xh9sm8nqy07ajlc7va8rs9h9j0h5dsi1-dbus-1.12.10-lib
checking for references to /build in /nix/store/xh9sm8nqy07ajlc7va8rs9h9j0h5dsi1-dbus-1.12.10-lib...
shrinking RPATHs of ELF executables and libraries in /nix/store/58dis8kla61569ybgglyyffqi615wk00-dbus-1.12.10-doc
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/58dis8kla61569ybgglyyffqi615wk00-dbus-1.12.10-doc
checking for references to /build in /nix/store/58dis8kla61569ybgglyyffqi615wk00-dbus-1.12.10-doc...
Moving /nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin/dbus-launch to /nix/store/xh9sm8nqy07ajlc7va8rs9h9j0h5dsi1-dbus-1.12.10-lib/bin/dbus-launch
rmdir: failed to remove '/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10/bin': Directory not empty
/nix/store/s892qdqhk2hsm4gn52iby0by9hpm6xlq-dbus-1.12.10

Success on aarch64-linux (full log)
Attempted: dbus
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/sd6ga62cf4fn73cbfrwg4pwhzqqgr18r-dbus-1.12.10-lib/lib
patching script interpreter paths in /nix/store/sd6ga62cf4fn73cbfrwg4pwhzqqgr18r-dbus-1.12.10-lib
checking for references to /build in /nix/store/sd6ga62cf4fn73cbfrwg4pwhzqqgr18r-dbus-1.12.10-lib...
shrinking RPATHs of ELF executables and libraries in /nix/store/spi4jryj9nqdzm7dkgzaph8rngw02c8y-dbus-1.12.10-doc
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/spi4jryj9nqdzm7dkgzaph8rngw02c8y-dbus-1.12.10-doc
checking for references to /build in /nix/store/spi4jryj9nqdzm7dkgzaph8rngw02c8y-dbus-1.12.10-doc...
Moving /nix/store/4r9s0pal2z8gsz2i2ayq0sscpq0ym68h-dbus-1.12.10/bin/dbus-launch to /nix/store/sd6ga62cf4fn73cbfrwg4pwhzqqgr18r-dbus-1.12.10-lib/bin/dbus-launch
rmdir: failed to remove '/nix/store/4r9s0pal2z8gsz2i2ayq0sscpq0ym68h-dbus-1.12.10/bin': Directory not empty
/nix/store/4r9s0pal2z8gsz2i2ayq0sscpq0ym68h-dbus-1.12.10

Timed out, unknown build status on x86_64-linux (full log)
Attempted: dbus
Partial log (click to expand)
cannot build derivation '/nix/store/6vbawqkmmrggsim79fc2jm670lp216mb-python3.6-lxml-4.2.3.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/m2kzmq3bjn9b0rg2y87lvgji6a05fl3b-gnome-doc-utils-0.20.10.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/ddvzmvkrn915xrrx4kb85j0f69196z34-python3-3.6.6-env.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/0jf29nsx90xb0b4mh2aqglz6k69jpz5r-shadow-4.6.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/2mg8pawksn2gg0wgwn33y2f01jm7hi24-util-linux-2.32.1.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/1g6595jjv8c8lr53snphvqjyxnz5cni2-glib-2.56.0.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/9qmk58ss88gvlmagbx6iwcs08xhm0701-libSM-1.2.2.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/nx1ll3102fcdhp7jiq5kc1xxb0ys2pr7-systemd-239.drv': 42 dependencies couldn't be built
cannot build derivation '/nix/store/nqijpq0nbcxi0mqj50cj4smln7cpqdhl-dbus-1.12.10.drv': 9 dependencies couldn't be built
error: build of '/nix/store/nqijpq0nbcxi0mqj50cj4smln7cpqdhl-dbus-1.12.10.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: dbus
Partial log (click to expand)
cannot build derivation '/nix/store/6vbawqkmmrggsim79fc2jm670lp216mb-python3.6-lxml-4.2.3.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/m2kzmq3bjn9b0rg2y87lvgji6a05fl3b-gnome-doc-utils-0.20.10.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/ddvzmvkrn915xrrx4kb85j0f69196z34-python3-3.6.6-env.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/0jf29nsx90xb0b4mh2aqglz6k69jpz5r-shadow-4.6.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/2mg8pawksn2gg0wgwn33y2f01jm7hi24-util-linux-2.32.1.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/1g6595jjv8c8lr53snphvqjyxnz5cni2-glib-2.56.0.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/9qmk58ss88gvlmagbx6iwcs08xhm0701-libSM-1.2.2.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/nx1ll3102fcdhp7jiq5kc1xxb0ys2pr7-systemd-239.drv': 42 dependencies couldn't be built
cannot build derivation '/nix/store/nqijpq0nbcxi0mqj50cj4smln7cpqdhl-dbus-1.12.10.drv': 8 dependencies couldn't be built
error: build of '/nix/store/nqijpq0nbcxi0mqj50cj4smln7cpqdhl-dbus-1.12.10.drv' failed

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: dbus
Partial log (click to expand)
cannot build derivation '/nix/store/39kv0y7r9w2d5qgy339fvbxqdnmrvy87-IOKit-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/r553g5gxs33zqcrgrb9q47vkpzwpb48a-configd-osx-10.8.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/1cqfilyx6v39csqab43sp4pwg1fd3c1p-python-2.7.15.drv': 15 dependencies couldn't be built
cannot build derivation '/nix/store/lsqagxvmawdd8kvga4langyr52bn16ym-libxml2-2.9.8.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/p2ir0217l79f36v5yszzpan5zx8z58v2-xcb-proto-1.12.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/4dl0yv9s0v9ncln5jw065w914fi4cybb-libxslt-1.1.32.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/y2z7wbpq47ls9kgb0lsjrqrkd9bpi8p5-libxcb-1.12.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/hai5v7m0fz07gpq85c4nmnql65dbb617-libX11-1.6.5.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/4w232lb399mxa6imphpbx19kndib31kp-dbus-1.12.10.drv': 7 dependencies couldn't be built
error: build of '/nix/store/4w232lb399mxa6imphpbx19kndib31kp-dbus-1.12.10.drv' failed

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: dbus
Partial log (click to expand)
cannot build derivation '/nix/store/39kv0y7r9w2d5qgy339fvbxqdnmrvy87-IOKit-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/r553g5gxs33zqcrgrb9q47vkpzwpb48a-configd-osx-10.8.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/1cqfilyx6v39csqab43sp4pwg1fd3c1p-python-2.7.15.drv': 15 dependencies couldn't be built
cannot build derivation '/nix/store/lsqagxvmawdd8kvga4langyr52bn16ym-libxml2-2.9.8.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/p2ir0217l79f36v5yszzpan5zx8z58v2-xcb-proto-1.12.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/4dl0yv9s0v9ncln5jw065w914fi4cybb-libxslt-1.1.32.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/y2z7wbpq47ls9kgb0lsjrqrkd9bpi8p5-libxcb-1.12.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/hai5v7m0fz07gpq85c4nmnql65dbb617-libX11-1.6.5.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/4w232lb399mxa6imphpbx19kndib31kp-dbus-1.12.10.drv': 7 dependencies couldn't be built
error: build of '/nix/store/4w232lb399mxa6imphpbx19kndib31kp-dbus-1.12.10.drv' failed

Changelog LGTM :).


Remove 1.22 Prow Jobs

Description of changes:
Updates dependency on eks-anywhere-prow-jobs to remove 1.22 prow jobs.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold

/approve


Research patching VMProtect and Themida packers

https://github.com/secretsquirrel/the-backdoor-factory#the-backdoor-factory-is-no-longer-maintained-or-supported


move stores into sveltik context

The stores were shared globally which means only one form could work on the page at one time.
Moving the stores into the context allows you to have multiple sveltik contexts i.e. forms
on the page at the same time.
A side effect of this is that Field only works when inside a Sveltik context, hence the changes
to the tests.
The tests were using spread props on the field variable, which contains the handleBlur and handleInput function.
These functions are rendered as strings in the html.
This was previously not showing up in the tests for some unknown reason, but with this change it does show up, so I changed some spread props from {...field} to {...{ name: field.name, value: field.value }}.
Maybe the handleBlur and handleInput functions should be passes seperately, not part of the field variable.
See https://svelte.dev/repl/56c033cf42934e0da9983202866864e5?version=3.24.0 (inspect element the input field)
<input type="text" name="color" handleblur="function handleBlur({ target: { name } }) {
    		touched.update(_t => ({ ..._t, [name]: true }));

    		if (validateOnBlur) {
    			handleValidate();
    		}
    	}" handleinput="function handleInput({ target }) {
    		const { name, type, checked, value } = target;
    		let nextValue = value;

    		if (type === "range" || type === "number") {
    			nextValue = nextValue === "" ? undefined : +nextValue;
    		} else if (type === "select-multiple") {
    			nextValue = [].map.call(target.querySelectorAll(":checked"), option => option.value);
    		} else if (type === "checkbox") {
    			nextValue = checked;
    		}

    		values$1.update(_v => ({ ..._v, [name]: nextValue }));

    		if (validateOnChange) {
    			handleValidate();
    		}
    	}">

Thank you for this

This is released in 0.1.14.


Added the rest of the Uri/UriBuilder properties

Uri/UriBuilder wrappers only exposed Uri.GetScheme and UriBuilder.GetQuery/SetQuery.
This PR makes sure that both of them have access to all of the properties that make up a URI, that is;
Scheme, Host, Port, Path, Query and Fragment.
For reference:
[scheme]://[host]:[port][path[?query][#fragment]
[http]://[www.contoso.com]:[80][/catalog/shownew.htm][?date=today][#search]

Sounds reasonable :-) We'll take a look! Thanks for submitting!

Hi and thanks for the feedback!
I tried out the scheme property for UriBuilder and it appears the documentation is a bit misleading, since a user can e.g. clear the scheme by providing an empty-string and might even provide something like:
"invalidscheme://microsoft.com", by doing uriBuilder.Scheme = "invalidscheme"; which throws no error.
What appears to be invalid are schemes with characters like space in them, like "invalid scheme", so I at least added a test case for that and also ports over and under the valid number range.
I also added the Enum, but I don't know of how much value it'll be. There already was a test in UriTest.Codeunit.al since Scheme getter was already supported for that wrapper, would you want me to modify that one to use the enum too?
Btw. before I submitted the PR initially AL Extension for vscode got frozen so I had to disable it (along with the intellisense, which made the missing quotations slip by, sorry about that. I've addressed these too. :)

I also added the Enum, but I don't know of how much value it'll be. There already was a test in UriTest.Codeunit.al since Scheme getter was already supported for that wrapper, would you want me to modify that one to use the enum too?

I see. My idea was mainly to try limit the possible values of the URI scheme used from the BC application. For instance, it is super fishy if ftp scheme is used.

Hey @bjarkihall
I ran the tests and there are some failures. Can you take a look, please.
Failure - Codeunit Uri Builder Test -  Duration 0.123 seconds
   Success - Test method: InitInvalidURITest - Duration 0.007 seconds)
   Failure - Test method: SetAndGetSchemeTest - Duration 0.027 seconds
      Error:
         Assert.AreEqual failed. Expected:<https://microsoft.com> (Text). Actual:<https://microsoft.com:80/> (Text). The scheme does not match.
      Call Stack:
         Library Assert(CodeUnit 130002).AreEqual - Library Assert by Microsoft
         Uri Builder Test(CodeUnit 135071).SetAndGetSchemeTest - System Application Test by Microsoft
         Test Runner - Mgt(CodeUnit 130454).RunTests - Test Runner by Microsoft
         Test Runner - Isol. Codeunit(CodeUnit 130450).OnRun - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunSelectedTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunNextTest - Test Runner by Microsoft
         Command Line Test Tool(Page 130455).RunNextTest - OnAction - Test Runner by Microsoft
   Failure - Test method: SetIncorrectPortTest - Duration 0.01 seconds
      Error:
         Assert.ExpectedError failed. Expected: A call to System.UriBuilder.Port failed with this message: Specified argument was out of the range of valid values.\Parameter name: value. Actual: A call to System.UriBuilder.Port failed with this message: Specified argument was out of the range of valid values.\Parameter name: value.
      Call Stack:
         Library Assert(CodeUnit 130002).ExpectedError - Library Assert by Microsoft
         Uri Builder Test(CodeUnit 135071).SetIncorrectPortTest - System Application Test by Microsoft
         Test Runner - Mgt(CodeUnit 130454).RunTests - Test Runner by Microsoft
         Test Runner - Isol. Codeunit(CodeUnit 130450).OnRun - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunSelectedTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunNextTest - Test Runner by Microsoft
         Command Line Test Tool(Page 130455).RunNextTest - OnAction - Test Runner by Microsoft

   Failure - Test method: SetAndGetPathTest - Duration 0.007 seconds
      Error:
         Assert.AreEqual failed. Expected:</segment2/segment3> (Text). Actual:</segment2/segment3/> (Text). GetPath does not work as expected.
      Call Stack:
         Library Assert(CodeUnit 130002).AreEqual - Library Assert by Microsoft
         Uri Builder Test(CodeUnit 135071).SetAndGetPathTest - System Application Test by Microsoft
         Test Runner - Mgt(CodeUnit 130454).RunTests - Test Runner by Microsoft
         Test Runner - Isol. Codeunit(CodeUnit 130450).OnRun - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunSelectedTests - Test Runner by Microsoft
         Test Suite Mgt.(CodeUnit 130456).RunNextTest - Test Runner by Microsoft
         Command Line Test Tool(Page 130455).RunNextTest - OnAction - Test Runner by Microsoft

These were a bit weird, since I used the output of GetLastErrorText and the -newline character didn't seem to work in strict string comparisons but \n\r actually said they were equal. I also didn't think the default port would be part of AbsoluteUri (according to the docs) but I guess UriBuilder adds it?
Oh well, at least they're all green now:

Availability update: We will publish a fix for this issue in the next update for release 17.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


processModelMaterialsCommon fix

The processModelMaterialsCommon for generating glTF shaders/materials/techniques based on the KH_model_materials_common extension used to produce slightly out-of-spec glTF techniques.
I only caught this while trying to run a model with the extension through the pipeline twice, which caused a bizarre crash in addDefaults.

This may be the cause of https://github.com/AnalyticalGraphicsInc/gltf-pipeline/issues/239

Yes it is.. thanks for finding this.

Fixes #239


[2008-10-14] Support pluralization in i18n _t()

created by: @chillu (ischommer)
assigned to: @chillu (ischommer)
created at: 2008-10-14
original ticket: http://open.silverstripe.org/ticket/2949

Add new _t_plural() method to i18n which can choose between a singular and plural translation based on a passed numeric value. This is preferred to pluralization by inflection, as it tends not not work so well (or at all) for languages other than english.
The method naming is up for discussion.
sprintf(_t_plural('MyModule.MYSHEEPCOUNT','My %s sheep','My % sheeps', $count), $count);

Make sure that textcollector identifies two different strings here. An easy way to keep the existing lang-table and translate.ss.com infrastructure would be to split it up into _SINGULAR and _PLURAL - so the example would be MyModule.MYSHEEPCOUNT_SINGULAR and MyModule.MYSHEEPCOUNT_PLURAL

this same issue was pointed out to me when translating a module... there is no real way to handle plural as of now? Especially with complex plural rules...
Though Zend seems to have the methods to handle plural correctly, but I guess we will run into issues on how to store the stings?
Since this is quite a complex issue, this wont be fixed any time soon, but might be good to revive it?

@colymba Happy to review a pull request to _t() against master which introduces pluralisation :) Rough steps:

Document the pluralisation format for strings - the Symfony one looks OK
Create a new i18n::transChoice($entity, $count, $string, $context, $injection)
Parse out the pluralisation formats in i18n::_t()
Create a new template parser rule in SSTemplateParser.php.inc for <% tChoice %>, see Translate__*() methods
Hook up new rules to i18nTextCollector

Done with https://github.com/silverstripe/silverstripe-framework/pull/6558


Auto-convert Spanner tests

Changed SpannerIO tests so that they actually run and test something meaningful, as well as altering expected data so it would be the same going in and coming out of Spanner.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python 3.7 PostCommit

Thanks, @BjornPrime .


not able to open link in browser

not able to open search results in the browser. I have added -kb-accept-entry but nothing happens I am able to copy the URL by pressing control+y.

if you did not set ROFI_SEARCH_CMD env variable as described in readme, it will call xdg-open <url> with selected url.
So investigate if xdg-open is setup to open urls on your system.

also if you press enther or other key you configured as -kb-accept-entry and nothing happens, that means its configured incorrectly..


Don't consider tool down on pointerenter with button pressed (fix #218)

This is intended to fix https://github.com/edemaine/cocreate/issues/218, an issue when using stylus on Chromium on Linux/X11.
The issue is caused by a combination of Chromium's behavior on this platform, and Cocreate treating a pointerenter event with a button pressed as marking the current tool as "down" even in the absence of any pointerdown event.
This commit disables that.
As a (side-)effect, if this commit is applied, one can no longer start to draw by starting dragging the mouse (or finger or stylus) on a Cocreate toolbar and then dragging in into the drawing board. I wouldn't consider this a disimprovement. (Note that if the dragging starts outside the browser window, or even on the browser's toolbars, it wouldn't start drawing even before this commit.) But if you want I can limit the change by platform using the user agent string.

Thanks for the PR! My one hesitation here is the following existing behavior:

User starts drawing with pen tool, but accidentally goes slightly outside the window (or iframe, in the case of Comingle).
User returns to the window with the button still down, and expects the drawing to still continue (though it obviously didn't draw for the outside duration).

This currently works and I believe your PR would disable it.  That said, the current behavior in the above scenario is pretty much only good with the pen tool; with the rectangle tool, the current behavior starts a new rectangle when returning to the window, instead of editing the old one.
Perhaps a better behavior would be to detect that we're pointentering with buttons down and those buttons match a pointerleave event. Then we'd like to continue the previous operation, though this might need to be tool-specific instead of the generic code you're touching here.
What do you think?

I added a setPointerCapture in the pointerdown event handler. This way, the tool simply continues if the pointer leaves the window, and also if it returns.
This way the pen tool continues drawing even when the pointer is outside the board. Other whiteboard/drawing programs I tried work like this too. It can be considered good (the line is not interrupted if the cursor briefly moves outside the window) or bad (it might draw over stuff). If you prefer it to not work like this, I can try to make the pen tool work in a special way, approx. like you suggested.
I removed the code that marked the tool as "up" on pointerleave, because

it's not needed: With pointer capture, we get a pointerup even if the pointer is released outside the window.
For some reason, when the pointer leaves the browser window (not the board itself) Firefox sends a pointerleave immediately followed by a pointerenter (https://bugzilla.mozilla.org/show_bug.cgi?id=1666851), which would stop the tool.

Oh wow, the setPointerCapture behavior is exactly what I wanted. What a nice improvement! (and code simplification) I've used setPointerCapture in other projects, but surprised I never added it to Cocreate. Thanks for this nice improvement in behavior, in addition to the bug fixes for X11.


build-sample missing from Gulp tasks?

The readme was fine until trying the gulp build-sample command.   Any suggestions on how to build and run the sample?

@jdanyow Ping.

a few days ago I added a message to the readme identifying which parts were related to the old version of this library.  sorry for the confusion.


readme: workaround for intellij bsp bug

re https://youtrack.jetbrains.com/issue/SCL-19517
copy of https://github.com/sbt/sbt-protobuf/pull/91

Thanks for the contribution to the docs. I have missed the notification for this PR, sorry it took so long!


Jobs typo ("bobs") in cache manager

Sorry, I'm not at my computer or I'd link to this properly, but it looks like the jobs API call may have a problem working as there's a typo in the cache manger. In line 56 it references "bobs" not "jobs"

Fixed. 0.3.7 released on Pypi.
Thanks! :+1:


DragDrop on DataGrid Crashes application with error "A Drag Operation is already in progress"

What steps will reproduce this issue?

Download current source for Showcase.WPF.DragDrop.Debug.
Build
Run
Select DataGrid
Attempt to drag a row

Expected outcome
Row Drags
Repo
This is in the vanilla sample application
Environment

GongSolutions.WPF.DragDrop v1.1.0 (also trunk)
Windows OS 10
Visual Studio 2017
.NET Framework 4.6.2

@iamwyza I can't reproduce this with the current source.

It seems the issue happens when another app is in the middle of a drop. In my case the other app was asking for approval before continuing with the drop. I fixed the issue by canceling the drop if this exception occurs. You may want to implement this fix in your repository.