dockerTools.pullImage: use imageDigest

Docker provides the "image digest" which is an immutable image
identifier (unlike image name and tag).
It now has to be specified to pull an image.
Note Docker supports URI such as imageName:imageTag@imageDigest however
this is not yet supported by Skopeo:
"Docker references with both a tag and digest are currently not supported"
Motivation for this change
This improve reproducibility of docker pullling operations since it now use a immutable Docker image identifier.
Things done
nix-build ./ -A dockerTools.examples.nix && docker load -i result

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] 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"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This looks fantastic, thanks!

Big fan of digests, so +1 from me. Can you also add a commit to remove the pull.sh script that got orphaned in #29302?

+1 also cc @offlinehacker and @matejc

@aneeshusa oups... commit pushed. Thanks!

Thanks @nlweo! I just noticed that pullImage is also described in the docs (docs/functions.xml) - can you update it to mention that the digest is now required? Might also be a good idea to update the release notes with the same info.

@aneeshusa done.

Nice, looks good, thanks :)

Since dockerTools.pull implementation with Skopeo has been reverted from master, this PR is deprecated.
I have to rework on all of these PRs before resubmitting them. So,for now, I close this PR.


Update the macosx image to the latest to try to fix compile times

Codecov Report

Merging #519 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #519      +/-   ##
==========================================
+ Coverage   84.68%   84.74%   +0.05%
==========================================
  Files          41       41
  Lines        6268     6291      +23
==========================================
+ Hits         5308     5331      +23
  Misses        960      960


chore: update stripe

I'm not sure why https://github.com/freeCodeCamp/freeCodeCamp/pull/46571 failed in the way it did, but this should fix it.
edit: I suspect life will be easier if we group these, so let's try that.

Yes.


Failed Signup yields wireframe page without styling

An issue for this may already exist, but could not find it on a cursory search.
If a field is missing or is not entered correctly when the Sign Up button is pressed for a new user account and validation fails, a wireframe page is returned without any of the proper site theming.

@mjstealey I believe this has been fixed.

resolved


Make port 0 mean any free port

Currently, when using port 0, gRPC will use a free port as expected. but, the system does not know what port this is and thus PID lookups fail.
We need to either extract the port from gRPC, or find a free port if port 0 is used.

The actual port being used should be set here:
https://github.com/AsynkronIT/protoactor-dotnet/blob/master/src/Proto.Remote/RemotingSystem.cs#L19

Hi. I've had a look at Chat example and it seems that "port 0" solution doesn't work as expected (I'm working on dev branch), i.e. whenever I specify a non-zero port in Client code I succeed in receiving (broacasted) server responses, but of course I can only connect a single client, while if I do a "Remote.Start("<IP_ADDRESS>",0) client never gets any response (despite its request are correctly received by server).
Any suggestion? Thanks.

@maiorfi sounds like something we should look into. Can you open a separate issue with a repro?


BE changes for Add Extension (#858)

Title of ticket:
Description
BE changes for Add Extension
DOD

[x] migration script(create and insert) for Extension tables (4 tables)
[ ] GET API for extension reasons
[ ] Service to save data to FOIRequestExtesion table
[ ] Service to save data to FOIRequestDocumentMapping table
[ ] POST API for Add extension

GET API for Extension Reasons

http://localhost:15000/api/foiflow/extensionreasons
Sample Value:

[{"isactive": true, "extensionreasonid": 1, "reason": "Public Body - Consultation", "extensiontype": "Public Body"},
{"isactive": true, "extensionreasonid": 2, "reason": "Public Body - Further Detail from Applicant Required",
"extensiontype": "Public Body"}, {"isactive": true, "extensionreasonid": 3, "reason": "Public Body - Large Volume and/or
Volume of Search", "extensiontype": "Public Body"}, {"isactive": true, "extensionreasonid": 4, "reason": "Public Body -
Large Volume and/or Volume of Search and Consultation", "extensiontype": "Public Body"}, {"isactive": true,
"extensionreasonid": 5, "reason": "OIPC - Applicant Consent", "extensiontype": "OIPC"}, {"isactive": true,
"extensionreasonid": 6, "reason": "OIPC - Consultation", "extensiontype": "OIPC"}, {"isactive": true,
"extensionreasonid": 7, "reason": "OIPC - Further Detail from Applicant Required", "extensiontype": "OIPC"},
{"isactive": true, "extensionreasonid": 8, "reason": "OIPC - Large Volume and/or Volume of Search", "extensiontype":
"OIPC"}, {"isactive": true, "extensionreasonid": 9, "reason": "OIPC - Large Volume and/or Volume of Search and
Consultation", "extensiontype": "OIPC"}, {"isactive": true, "extensionreasonid": 10, "reason": "OIPC - Fair and
Reasonable", "extensiontype": "OIPC"}]

Sample POST request
POST http://localhost:15000/api/foiextension/ministryrequest/1
body:
{
"extensionreasonid": 6,
"extendedduedays": 45,
"extendedduedate": "2022-04-15"
}
response:
{
"id": 1,
"message": "Extension created",
"status": true
}


Get-DbaAuditSpecification should be renamed

Both "ServerAuditspecifications" and "DatabaseAuditSpecifications" exist,
so either the function name should clarify which are included,
or both types should be returned.
The latter option only if the object types match, and I assume they don't.

this is a server one, ill rename


Fixes csv issue

This fix is related to issue #32 where if the csv output is specified, Run
gets blocked on the stopCh

@rakyll can you review and merge this? Would be a great to get it running

Hi, I just ran into the same issue. It will be nice if the issue is solved in the master branch.

Fixed via 2969c6a, please open an issue again if you experience something odd.


SetCertAccess script doesn't work in every case

Depending on the type of certificate, the code that tries to get the private key might fail.  Here's the modification we made
$keyName=$cert.PrivateKey.CspKeyContainerInfo.UniqueKeyContainerName
$keyPath = "C:\ProgramData\Microsoft\Crypto\RSA\MachineKeys"
if ($keyName -eq $null){
$privateKey = [System.Security.Cryptography.X509Certificates.RSACertificateExtensions]::GetRSAPrivateKey($cert)
$keyName = $privateKey.Key.UniqueName
$keyPath = "C:\ProgramData\Microsoft\Crypto\Keys"
}

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

ID: 3ac102a5-e9ad-0046-f217-446f1ad263c5
Version Independent ID: a47523e2-0c30-dee4-0037-5f58836fa37f
Content: Add an HTTPS endpoint using Kestrel - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: atsenthi

Thanks for the issue, we will take a look.

Hi @njallain,
We have modified the code in the document in the PR - https://github.com/MicrosoftDocs/azure-docs-pr/pull/199522
#please-close this issue


[FLINK-13938][yarn] Enable configure shared libraries on YARN

What is the purpose of the change
Enable configure shared libraries on YARN. See also https://issues.apache.org/jira/browse/FLINK-13938
Brief change log

Add command-line option -ysl --yarnsharedLibs and config option yarn.shared-libraries
respect options

Verifying this change
I am looking for adding a test case but it seems hard without a real HDFS env. Maybe e2e tests required. But one can still verify it manually.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes, enable configure shared libs on YARN)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs)

cc @wangyang0918

cc @yanghua @MalcolmSanders

I'd like to add a unit test but fail to have an idea how to start a mini HDFS. Let me think of how to introduce it within YarnTestBase...

I've manually verify the changes locally.
My rough thought about testing is using a test job running with YarnTestBase and verify the resource, maybe we have to verify no files uploaded by Flink also.

closed for now in favor of another issue.
After discussion with @wangyang0918 I notice FLINK-13938 is not what proposed here.

sorry for the late follow up @TisonKun . I come to understand that this PR is for something different from FLINK-13938. did we create a new JIRA ticket? I think the approach this PR implements is highly valuable especially for a production cluster setup :-) --Rong


Add support of "open_database" and "html_injection" tests

:tada: This issue has been resolved in version 4.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


have you planned to support generic type?

assume array named 'arr' that has three object [a, b, c]
a's type is A
b's type is B
c's type is C
I want to make function find which need generic type argument
like this
function findWithGeneric(value: any) {
....
}
I wonder if typia.is support generic type like this:
function findWidthGeneric(value: any) {
return typia.is(value)
}
const result = arr.find((value) => find(value) )
console.log(result) // 'a'

No plan. Use functional parameter instead.


fix(tests): fixed tests

Intent
Fix unit test failure.
Implementation

Fixed cbd command.
Set run tests sequentially rather than in parallel.

Checks

[x] Code is formatted correctly (npm run lint:fix).
[x] Any new functionality has been unit tested.
[x] All unit tests are passing (npm test).

[ ] All CI checks are green.
[ ] JSDoc comments have been added or updated.

:tada: This PR is included in version 1.19.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add MITx Online Open edX data to the Production Data Lake

In order to finish the "course settings" report (https://github.com/mitodl/hq/issues/244), we need to include course settings from the MITx Online production Open edX instance.
@pdpinch needs this report to finalize course settings for the launch of "paid" courses in MITx Online on Oct. 7th.

[ ] Add MITx Online Open edX data to the production data lake.
[ ] Provide "reasonably stable" ecosystem in support of the work leading to the Oct. 7th launch.

On hold.


expert view "stop" buttons are in the wrong color

^ wrong
make it like in the figma:

Reviewed and closed.


X is a required field

I've been tearing my hair out, I just cannot see what I'm doing wrong.
You can download the CSV I am importing here: http://csvlint.io/validation/54f84a2f6373760234060000
Seems to import fine but when I look at the log...
No new entries are created at all.
Why is this happening?

%$@£(%$£()@U*%$B 38 530%$£@%
I hadn't created the associated entry.
Need some more feedback if an associated entry is not found.

This is how Craft returns the error, there's not much I can do.

Ok no problem. Thanks so much for this plugin bob, it's an absolute lifesaver.


Security level negotiation between call credentials and channel

This PR adds security level attribute to grpc_call_credentials and grpc_auth_context. The security level associated with a call credential dictates the minimum security level a channel should be associated with in order to transfer call credentials. If the channel's security level is insufficient, call credentials will not get communicated.

Each call credential (except for fake) is associated with GRPC_PRIVACY_AND_INTEGRITY security level by default.
A security level mutator is also provisioned to allow modification of default security level, which is needed internally (i.e., allowing a plugin call credential to be transferred over integrity-only protected channels).
Local TCP channel is associated with GRPC_PRIVACY_AND_INTEGRITY security level in order to unbreak internal use cases if exist, and I added a TODO to lower its security level to  GRPC_SECURITY_NONE.

I will prepare for a patch CL (for internal channel and call creds) before this PR gets merged.

Please hold on reviewing this PR. I will make additional changes based on our offline discussion.

Let me know when this is ready for review again.

The PR is ready for review now. @jiangtaoli2016 @markdroth

Friendly ping.

Upon merge, please don't forget to clean up the internal definition of grpc_security_level and tsi_security_level.

@jiangtaoli2016 Thanks jiangtao for a detailed review. Yes, I will prepare for a patch CL before merging this PR. Thanks for reminding.

Friendly ping @markdroth

Thanks @markdroth for the review. All comments make sense to me.
I will provide a ctor that takes in security level as a parameter instead of providing a setter API. Regarding that ctor, I do not think we need to expose it in CallCredentials defined in credentials_impl.h since for all call credential implementations except for plugin credentials, their security level can just be hard-coded. For plugin credentials, we will need to have a new ctor since it will be used by both internal and C# call credential implementations that require different security levels, and I will prepare a gRFC for it. For C-core implementation of call credential (i.e., grpc_call_credentials), I will also provide a new ctor for it and add it into the gRFC.

@markdroth I am wondering what will be the next step here. Do you want to continue the review process after gRFC https://github.com/grpc/proposal/pull/167 gets accepted?

Friendly ping.

@lidizheng I made the following C-core changes in this PR that may affect gRPC pyhton: 1) adding a new enum grpc_security_level to grpc_security_constants.h and 2) updating the API - grpc_metadata_credentials_create_from_plugin to take a new parameter - security level. Could you please review the files related to gRPC python and see if they make sense?

Thanks @lidizheng for a quick review!

Friendly ping @markdroth

@markdroth Thanks for pointing them out. They are both fixed.

Known issues: https://github.com/grpc/grpc/issues/18840, https://github.com/grpc/grpc/issues/16135

F.Y.I. The PR is updated to resolve the conflict with upstream changes - https://github.com/grpc/grpc/pull/21423

https://github.com/grpc/grpc/issues/14128, https://github.com/grpc/grpc/issues/19627, https://github.com/grpc/grpc/issues/21525

Thanks all for the review! Merging now.


fix: cursor blink lags when pointer down at text range

close #xxx

Pull Request Checklist

[ ] Related tickets or issues have been linked in the PR description (or missing issue).
[ ] Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
[ ] Unit tests have been added for the changes (if applicable).
[ ] Breaking changes have been documented (or no breaking changes introduced in this PR).


Use model_selection instead of deprecated cross_validation

Presently, when training a model, we see the following deprecation message.
$ python training/datatrain.py
/Users/demo/.virtualenvs/gabbar/lib/python2.7/site-packages/sklearn/cross_validation.py:44: DeprecationWarning: This module was deprecated in version 0.18 in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
  "This module will be removed in 0.20.", DeprecationWarning)
training samples: 12364
[testing] good samples: 5299
[testing] problematic samples: 671
precision = 0.915625
recall = 0.442348
f1_score = 0.596514

This deprecation message is being logged in osmcha-django where it is currently being used which is making the logs larger and cluttered to look at. We might need a fix for this sooner.

cc: @rodowi

which is making the logs larger and cluttered to look at

🤔  osmcha shouldn't be running datatrain.py

🤔 osmcha shouldn't be running datatrain.py

Yeah. I assumed that the logs I saw was ^, but in-fact when I looked at this again today, it was different. The following is the deprecation message that we are getting.
DeprecationWarning: Passing 1d arrays as data is deprecated in 0.17 and will raise ValueError in 0.19. Reshape your data either using X.reshape(-1, 1) if your data has a single feature or X.reshape(1, -1) if it contains a single sample.
gabbar prediction: -1.0

Ex:
>>> x = np.asarray(range(3))
>>> x
array([0, 1, 2])
>>> x.reshape(-1, 1)
array([[0],
       [1],
       [2]])

I will work on a fix soon.

Not seeing this presently. Will revisit again later if required.


Elevator pitch

Brainstorming. Thinking about putting this in the README.

Can be gradually replaced with production code
yes
n/a

Running locally requires
node
probably a lot more than that

See code changes
when you hit save
after compile + restart / deploy

Change behavior without losing server-side state
yes (hot reload)
wait, what?

Play with the state in a REPL
yes
not likely

Maintainable by front end devs
yes
maybe?

Reproducing weird stuff that happened in prod
easy
hard / impossible

API response time
negligible
varies

Maintaining test accounts is
optional
required

Integrates with UI tests (Jest, Playwright, etc.)
[coming soon]
no

Seed with test data / scenarios
[coming soon]
much slower if possible at all

Optimized for
developers
end users

Developer experience
😁
😣

1 a change big enough to call out in release notes

Added the table to the README


Add include for std::bind.

Hi, this patch adds in  because the GUI code makes use of std::bind.
That's all.

Not sure if it is worth to do one-off fixes like this. How would reviewers check that the patch is both necessary and complete?

Not sure if it is worth to do one-off fixes like this. How would reviewers check that the patch is both necessary and complete?

As opposed to what? It's a bug and should be fixed...

As opposed to what? It's a bug and should be fixed...

There are also missing <cmath> includes all over the place. What I am saying is that we need an automated tool like icwyu to do this in a reproducible and automated manner. Otherwise this will be a waste of our precious reviewers' time.

There are also missing <cmath> includes all over the place. What I am saying is that we need an automated tool like icwyu to do this in a reproducible and automated manner. Otherwise this will be a waste of our precious reviewers' time.

This touches more non GUI files, thus should perhaps have been opened at the main repository (but let's keep it here for now).
utACK 2a39ccf1334ef3c48c6f9969a0fc916b9e10aae1

This touches more non GUI files, thus should perhaps have been opened at the main repository (but let's keep it here for now).
utACK 2a39ccf1334ef3c48c6f9969a0fc916b9e10aae1


Add Metamask connect support to DApp

Completed w https://github.com/hashgraph/hedera-smart-contracts/pull/272


np.diagonal fails

The following code fails
import autograd.numpy as np
from autograd import grad

def f(x):
    d = np.diagonal(x)
    s = np.sum(d)
    return s

df = grad(f)
x = np.array([[1, 2], [3, 4]], dtype=float)
print(df(x))

The traceback is
Traceback (most recent call last):
  File "test_diagonal.py", line 13, in <module>
    print(df(x))
  File "C:\Python36\lib\site-packages\autograd\wrap_util.py", line 20, in nary_f
    return unary_operator(unary_f, x, *nary_op_args, **nary_op_kwargs)
  File "C:\Python36\lib\site-packages\autograd\differential_operators.py", line 28, in grad
    return vjp(vspace(ans).ones())
  File "C:\Python36\lib\site-packages\autograd\core.py", line 14, in vjp
    def vjp(g): return backward_pass(g, end_node)
  File "C:\Python36\lib\site-packages\autograd\core.py", line 21, in backward_pass
    ingrads = node.vjp(outgrad[0])
  File "C:\Python36\lib\site-packages\autograd\core.py", line 61, in <lambda>
    return lambda g: (vjp(g),)
  File "C:\Python36\lib\site-packages\autograd\numpy\numpy_vjps.py", line 533, in <lambda>
    lambda g: anp.make_diagonal(g, offset, axis1, axis2))
  File "C:\Python36\lib\site-packages\autograd\tracer.py", line 48, in f_wrapped
    return f_raw(*args, **kwargs)
  File "C:\Python36\lib\site-packages\autograd\numpy\numpy_wrapper.py", line 126, in make_diagonal
    raise NotImplementedError("Currently make_diagonal only supports offset=0, axis1=-1, axis2=-2")
NotImplementedError: Currently make_diagonal only supports offset=0, axis1=-1, axis2=-2

However, the arguments of make_diagonal are offset=0, axis1=0, axis2=1, and they are identical to offset=0, axis1=-1, axis2=-2 for a 2D array. If I comment the raise lines in numpy_wrapper.py, the code works as intended.

How about using np.diag instead of np.diagonal in your function f?  They both do the same thing there, and it looks like autograd can differentiate np.diag just fine.
import autograd.numpy as np
from autograd import grad

def f(x):
    d = np.diag(x)
    s = np.sum(d)
    return s

df = grad(f)
x = np.array([[1, 2], [3, 4]], dtype=float)
print(df(x))

[[ 1.  0.]
 [ 0.  1.]]


For Motoko and other language interop, not sure if I'll be able to use a procedural macro

It might be better to simply build up the Rust or Motoko code manually, even with just string manipulations. It shouldn't be all that complicated, and then that generated code could...well, wait...I guess we'll have to see. Because we could simply export the Rust binary that includes the macro expansion, and then that could be imported into Motoko. But, I think the biggest problem is allowing the user to add custom types and resolvers of their own, in the language of their choice. This will require dynamic addition of types and resolvers, and the cross-language barrier might be complex to implement as well

The procedural macro is working great, and interop with other languages for now can just be done with cross-canister calls


Black Screen doesn't do anything. Can't find ptyw.js

"Uncaught Error: Cannot find module 'ptyw.js'", source: module.js (338)

Bug achieved by going through the installation steps:
git clone https://github.com/black-screen/black-screen
cd black-screen
npm install
npm run package
ls
npm start

are you using node v4 ?

@mzgnr black-screen should work with node v4. The problem is that the node module 'ptyw.js' wasn't installed/compiled correctly.  System Darwin 14.5.0
gyp ERR! command "/usr/local/bin/node" "/Users/hdavidzhu/.node/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/hdavidzhu/black-screen/node_modules/ptyw.js
gyp ERR! node -v v4.0.0
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/Users/hdavidzhu/.node/bin/npm" "install" "ptyw.js"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the ptyw.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls ptyw.js
npm ERR! There is likely additional logging output above.

@hdavidzhu check if you have install gyp it may be conflicting with node-gyp for me apt-get remove gyp worked

@vikas17a I'm in OSX. Would this be how you do it? It's not working either.
➜  ~  brew remove gyp
Error: No such keg: /usr/local/Cellar/gyp
➜  ~  brew remove node-gyp
Error: No such keg: /usr/local/Cellar/node-gyp
➜  ~

This one should help:
rm -r ~/.node-gyp
npm install

@shockone that worked! Thanks.


Unraid docker

Hey,
I'm using unraid, and I would just like to be aple to pull the docker, with everything within that docker to make this work.

Hi @Flexz9 - The latest revision has full (and now only) docker support - please refer to the README.md file for further details.


[SPARK-35050][DOCS][MESOS] Document deprecation of Apache Mesos in 3.2.0

What changes were proposed in this pull request?
Deprecate Apache Mesos support for Spark 3.2.0 by adding documentation to this effect.
Why are the changes needed?
Apache Mesos is ceasing development (https://lists.apache.org/thread.html/rab2a820507f7c846e54a847398ab20f47698ec5bce0c8e182bfe51ba%40<dev.mesos.apache.org>) ; at some point we'll want to drop support, so, deprecate it now.
This doesn't mean it'll go away in 3.3.0.
Does this PR introduce any user-facing change?
No, docs only.
How was this patch tested?
N/A

Test build #137292 has started for PR 32150 at commit 11abe90.

Test build #137292 has finished for PR 32150 at commit 11abe90.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41872/

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

Merged to master.


Move towards JSON API standard for responses

Start moving towards the JSON API standard for responses of the API.
In particular, the actual data is now returned in a data field.
We are not fully adopting the JSON API standard here just yet, since the responses of the aiida-core REST API are not fully compatible.
aiida-core REST API example:
{
  "data": {
    "nodes  ": [
      {
        "ctime": "Sun, 21 Jul 2019 11:45:52 GMT",
        "full_type": "data.dict.Dict.|",
        "id": 102618,
        "label": "",
        "mtime": "Sun, 21 Jul 2019 11:45:52 GMT",
        "node_type": "data.dict.Dict.",
        "process_type": null,
        "user_id": 4,
        "uuid": "a43596fe-3d95-4d9b-b34a-acabc21d7a1e"
      }
    ]
  },
}

How a JSON API conforming response would look like
{
  "data": {
    "nodes  ": [
      {
        "type": "node",
         "id": 102618,
          "attributes": {
            "ctime": "Sun, 21 Jul 2019 11:45:52 GMT",
            "full_type": "data.dict.Dict.|",
            "label": "",
            "mtime": "Sun, 21 Jul 2019 11:45:52 GMT",
            "node_type": "data.dict.Dict.",
            "process_type": null,
            "user_id": 4,
            "uuid": "a43596fe-3d95-4d9b-b34a-acabc21d7a1e"
         }
      }
    ]
  },
}

@chrisjsewell I've explicitly excluded the vendored json_api.py from the mypy checks, yet the pre-commit still complains about it.
Can you please tell me how to fix this?

Codecov Report

Merging #18 (f65ab59) into master (9abe7f7) will increase coverage by 0.86%.
The diff coverage is 89.39%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   88.88%   89.75%   +0.86%
==========================================
  Files           6        9       +3
  Lines         153      205      +52
==========================================
+ Hits          136      184      +48
- Misses         17       21       +4

Flag
Coverage Δ

pytests
89.75% <89.39%> (+0.86%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aiida_restapi/models/json_api.py
75.00% <75.00%> (ø)

aiida_restapi/routers/users.py
96.55% <94.73%> (+9.59%)
:arrow_up:

aiida_restapi/models/__init__.py
100.00% <100.00%> (ø)

aiida_restapi/models/entities.py
93.02% <100.00%> (ø)

aiida_restapi/models/responses.py
100.00% <100.00%> (ø)

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 9abe7f7...f65ab59. Read the comment docs.

We'll also need to decide whether to make the response fully JSON API compliant - would be straightforward but would definitely mean having to edit all response processing of existing clients (since the data moves from data to data -> attributes

@chrisjsewell I've explicitly excluded the vendored json_api.py from the mypy checks, yet the pre-commit still complains about it.

So I imagine you want something like this in the mypy.ini:
[mypy-aiida_restapi.json_api]
check_untyped_defs = False

would be straightforward but would definitely mean having to edit all response processing of existing clients (since the data moves from data to data -> attributes).
Perhaps we start without it for the time being...

Well I guess if we are going to make breaking changes, lets not be shy about it lol. I think it is better if possible to have one major change that clients can deal with in one go, rather than multiple stages of changes


Add predicateHints and jsonPredicateHints parameters to load_as_pandas methods

Hi everyone,
I'm currently working on a code snippet to demonstrate the basics of Delta Sharing.
While working on this, I came across the following implementations:

https://github.com/delta-io/delta-sharing/blob/7f0126073db3bf26c29648687c996db20e1b64b5/python/delta_sharing/__init__.py#L23
https://github.com/delta-io/delta-sharing/blob/7f0126073db3bf26c29648687c996db20e1b64b5/python/delta_sharing/delta_sharing.py#L54
https://github.com/delta-io/delta-sharing/blob/7f0126073db3bf26c29648687c996db20e1b64b5/python/delta_sharing/rest_client.py#L271

After reviewing these implementations, it seems that the Python client library does not consider the JSON predicates public.
I think this feature should be public though, is there any reason why this feature is not treated as public?
Thanks.

I’ve already implemented this feature on my local repository so if it is a reasonable feature request, I can make a PR.
As for ‘load_as_spark’, I thought it would be better to treat it as another feature, so I did not mention it, but I can handle it as well.
Thanks.


If no document, omit "documents" key from DeviceResponse map

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

Ping. Also please squash all your commits into one when possible.

As discussed, Akash will open a new PR which includes the fixes for PR #590, #585, #580, #577, #510.


Enable CPAOT build of S.P.Corelib; improve field access

Based on JanK's suggestion I have temporarily commented out the
CoreLib-specific optimization in getCallInfo; re-enabling this
optimization would require either replicating CoreCLR method table
construction or not yet existing support for persisting method
table layout info in R2R PE files. I have dared swap the "default"
vs. "conditional" case, thus slightly diverging from the CoreCLR
original, for the sole purpose of easier readability.
To reduce the remaining codegen differences between CPAOT and
Crossgen I have ported an extra bit of code from ZapInfo::getFieldInfo
regarding the different field access methods. Without this change,
we fail at runtime in various assemblies where CPAOT calculated
a different field layout than Crossgen. With the change, we still
fail in a couple of places due to
https://github.com/dotnet/corert/issues/6665
To further fix S.P.Corelib build, I also had to add a small hack
to remove the "ni" part of the filename extension, otherwise the
CPAOT-built S.P.Corelib gets tracked by ILC under the simple name
"System.Private.CoreLib.ni" and fails to match the hard-coded BCL name.
Thanks
Tomas

@dotnet-bot please test corert-ci


Ftp Client from Ethernet connections

Hi
good work for wifi client,
but can you help me for set library for use it for ethernet connection to ftp server?
Thanks
Umberto

https://playground.arduino.cc/Code/FTP/

https://playground.arduino.cc/Code/FTP/

Thanks. It use also with esp32 board?
Is compatible with lan8720?
Very thanks
Umberto

Thanks. It use also with esp32 board?
Is compatible with lan8720?
Very thanks
Umberto


Im a noob how do i make this run?

I want to use this so my bots from agarpowerups will have face book mass. I put my access token in agario-client.js and put my facebook cookie info in auth_token.js When i go on to agar.io and run my bots it dosnt work. Any help??

same

Read all this page https://github.com/pulviscriptor/agario-client
There is all info you need.

How do I check the console to see if there's errors

@BLUclanTDS
There would be error message and script will stop running.

What scrip where's the error message??

@BLUclanTDS
This project made for developers. I assume you are not one of them and you do now know JavaScript language.
If you want to learn JavaScript you can google some basics for beginners and look if you like it and learn it.

My friend made a modification for his agar.io bots. He told me how to do it and set it up but since he is studying for finals he hasn't been able to tell me how to run it or see if there's a error.

Do I just download the developer thing you have in the read me

Do I just download the developer thing

Its little more complicated than this.
Lets try this: https://www.codecademy.com/learn/javascript
Go there and look if you like it. See if you like JavaScript language and programming at all.


Unblock main

After https://github.com/thanos-io/thanos/pull/1906 we need to create:

https://quay.io/thanos/thanos-linux-amd64
https://quay.io/thanos/thanos-linux-arm64
https://hub.docker.com/r/thanosio/thanos-linux-amd64
https://hub.docker.com/r/thanosio/thanos-linux-arm64

Following the same naming as https://quay.io/repository/prometheus/prometheus-linux-amd64, https://quay.io/repository/prometheus/prometheus-linux-arm64, and so on.
Need to perform the following actions to avoid these problems in the future:

Add thanosbot user as the owner of the thanosio organization on Dockerhub because otherwise it is impossible to create a new repository there;
Fix quay.io credentials in password storage. Google has noticed a "strange pattern" when I tried to log into quay.io with Google SSO and asked me to enter a number sent via SMS. Impossible to work around this by myself.

Assigning to @bwplotka because due to 2FA I cannot perform these actions :'/
Example failing run: https://app.circleci.com/pipelines/github/thanos-io/thanos/8364/workflows/f709bc0b-713e-4de6-80be-813a1a474124/jobs/17614

Thanks for clear steps. Doing this now.

Repos created
Thanosbot has admin access (not sure if this is a good pattern) to both quay and dockerhub.

Retried, let's see

Let's see now https://app.circleci.com/pipelines/github/thanos-io/thanos/8372/workflows/5948f554-82f0-4dd9-b0a2-a1ffac1972be

I think it almost works. I think latest is pushed instead of main-sha that's why the issue: https://app.circleci.com/pipelines/github/thanos-io/thanos/8372/workflows/5948f554-82f0-4dd9-b0a2-a1ffac1972be

Fixed in https://github.com/thanos-io/thanos/commit/7793ab637780826c88741a4783f0b3ae7dffe6a4 :tada: Images available now:
https://hub.docker.com/layers/thanosio/thanos/main-2021-12-14-7793ab63/images/sha256-9dd0cee98ac7dc4f122e75d4fade9ba2452e8a9a2c0b8cb9a160e55ce1bae0f1?context=explore
https://quay.io/repository/thanos/thanos/manifest/sha256:ba0b9192a0db3693afcb220c4c1fc8578a1c25df20dc41697e57b3329fe1d7f8


[Work in Progress] Bill pages

Add bill overview and detail pages

Closes #16
Closes #17
Closes #18


Customer would like 2.3.0  available in Confluent Hub for download

Customer would like 2.3.0  available in Confluent Hub for download.
https://www.confluent.io/hub/newrelic/newrelic-kafka-connector/
Description
Customer cannot select version 2.3.0 from Confluent Hub
Steps to Reproduce
Try to download previous version from https://www.confluent.io/hub/newrelic/newrelic-kafka-connector/
Expected Behaviour
Can version 2.3.0 be a version available for download.

https://github.com/newrelic/kafka-connect-newrelic/blob/db2eb1dda605d9c5f0b117adea7943f2252ef9b1/newrelic-kafka-connector/src/main/java/com/newrelic/telemetry/TelemetrySinkTask.java#L58

Closing, since 2.3.3 is the current version on Confluent Hub now.


[Messages Grid Widget] Turns LCD on unnecessarily

@rigrig
Affected hardware version
Bangle 2
Your firmware version
2v17.39
The bug
I though the LCD of my watch lighted up to often and tracked it down to 'Messages Grid Widget'. It makes sense for the screen to light up when a message is automatically displayed (which it does), but not when only updating the widget (which it also does), is my preference. This seems to mainly happen if there are unread/not deleted messages on the watch so the widget is showing.
These are some of the situations where the LCD will turn on but I think it shouldn't:

The widget is showing on the watch, I press the power button on my phone, the phones lock screen shows and the watch's LCD is activated.
The widget is or isn't showing (makes no difference), auto-open is turned off in settings, an e-mail arrives on the phone and is forwarded to the watch which prompts the LCD turning on.
The widget is showing, auto-open music is NOT turned on in settings, the music app Spotify is active and the track ends moving automatically to the next one, the LCD of the watch turns on. (I belive this happens with other media apps which also displays a notification)
The widget is showing, auto-open music is NOT turned on in settings, the music app Spotify is active and I toggle play/pause (either on the phone or via the Spotify desktop client), the LCD of the watch turns on. (I know this happens with other media apps which also displays a notification)

I did some testing of 'Messages Widget' and this doesn't seem to happen with it.
My phone runs Android 13. The Bangle is connected to a version of Bangle.js Gadgetbridge in this PR with CompanionDevice Pairing.
Installed apps
notify (0.12)
health (0.22)
clock_info (0.02)
widminbat (0.02)
podadrem (0.07)
spotrem (0.07)
messages (0.58)
android (0.23)
lightswitch (0.06)
widanclk (0.02)
backswipe (0.02)
torch (0.11)
calculator (0.07)
agenda (0.13)
smpltmr (0.07)
kbmulti (0.05)
widram (0.03)
widbthide (0.01)
messageicons (0.05)
recorder (0.22)
widalarm (0.01)
boot (0.57)
setting (0.59)
runplus (0.18)
sched (0.22)
dtlaunch (0.23)
agpsdata (0.06)
messagesmusic (0.05)
alarm (0.41)
quicklaunch (0.14)
hrmmar (0.02)
taglaunch (0.03)
swscroll (0.03)
messagegui (0.65)
longpressbuzz (0.01)
clkinfostopw (0.02)
sleeplog (0.14)
bwclklite (0.33)
bwclk (0.32)
widmsggrid (0.04)

It seems it's the       Bangle.setLCDPower(1); // turns screen on on line 73 in the widjet.js code. Maybe some conditions could be checked to decide if the LCD should be activated or not.

Thanks for looking into this!
There's really no need for the widget to turn on the screen, so I made a PR to remove that line.


⚠ Bofrost.fr has degraded performance

In ced166e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2210 ms

Resolved: Bofrost.fr performance has improved in dfd7e91 after 8 minutes.


Android TextField obscureText  = true

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Write two TextFields;
The second obscureText is set to true;
First click on the default (false), and then click again on the TextField whose obscureText is true;
Unable to obtain focus when obscureText is true.

Expected results
Focus cannot be obtained when obscureText is true.
Actual results
When obscureText is true, focus cannot be obtained and two clicks are required.
Code sample
Code sample
class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  TextEditingController controller1 = TextEditingController();
  TextEditingController controller2 = TextEditingController();

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          backgroundColor: Theme.of(context).colorScheme.inversePrimary,
          title: const Text('Test'),
        ),
        body: Container(
          padding: const EdgeInsets.symmetric(horizontal: 15),
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: <Widget>[
              TextField(controller: controller1),
              TextField(controller: controller2, obscureText: true),
            ],
          ),
        ),
      );
}

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output

Hey,
I want to check your issue. But i am new to open source.
can u tell me the sdk path after clonning the repository?

Hi @mz-myfc
Your expected result is reversed, please update it.
I checked this issue on my device Realme 6, Android 11 and saw the obscureText TextField still can get focus with only one tap/click. What is your device (OS, model)? Could you provide a recorded video for this?

嘿， 我想检查一下你的问题。但我是开源新手。 你能告诉我克隆存储库后的sdk路径吗？

嘿， 我想检查一下你的问题。但我是开源新手。 你能告诉我克隆存储库后面的sdk路径吗？

Thanks for the reply. I have tried other phones without this problem. It is a problem with my device.

Hi @mz-myfc Your expected result is reversed, please update it.
I checked this issue on my device Realme 6, Android 11 and saw the obscureText TextField still can get focus with only one tap/click. What is your device (OS, model)? Could you provide a recorded video for this?

Thanks for the reply. I have tried other phones without this problem. It is a problem with my device.


Add Changelog To F-Droid

When updates are pushed to F-Droid, it doesn't show the changelogs. Suggesting adding the changelogs to F-Droid.

Changelogs for F-Droid now live under fastlane/metadata/android/en-US/changelogs/.


Geth suddenly unable to connect to peers and can't sync

Checked my Geth container logs today and found that it's stuck on a block and can't connect to peers. Any suggestions as to what might be wrong and how to fix it?
Here's the log output: https://gist.github.com/ardevd/6c2906db9606e51bba6ec38a10eae790
Any help would be much appreciated.

That's not an eth-docker issue. Please try ethstaker Discord or Geth discord.


master clue jorral strange device

I had just finished a Falo step and got the jorral step.
As I was standing at Falo and hitting the strange device, it gave me 2 possible locations, macgruber woods and seer's village.
Neither was correct, it was north of sincliar mansion.
Plugin couldn't figure it out while I was standing next to Falo.

This is unrelated to the Clue Step Tracker plugin, but instead would be an issue with Clue Scroll, a plugin native to Runelite. You should report this issue here:
https://github.com/runelite/runelite/issues


Pmap error

Fixes #1991.

This is definitely an improvement, thanks! I would go even farther and plumb in some more debugging information to make this more actionable, e.g. which argument index is missing a leading axis? I'm happy to submit this as-is and iterate as well, lemme know what you wanna do.

! I would go even farther and plumb in some more debugging information to make this more actionable, e.g. which argument index is missing a leading axis?

That's a good idea, though I'm not sure the cleanest way of doing that. Ideally, it would point at the exact pmap line that failed, but I'm not sure how.
Something like:
jax.pmap(f)(jnp.ones((8, 100)), 10)
                                ^
ValueError: Argument #2 is missing a leading axis

?

I think you need to plumb the argument indices through into shard_args here: https://github.com/google/jax/blob/master/jax/interpreters/pxla.py#L456 (i.e. map over enumerate(avals) instead of avals, and then plumb the index through to shard_args and the underlying handlers). That should let you specify the argument index in the error message, and the stack trace will contain the failing pmap line, which is pretty close!

I believe the underlying issue is fixed (see the linked issue.)


add ./ to tests-env.sh

make check was broken on machines that don't have ./ in $PATH. This commit fixes that.

@blippy Thank you for all the great work. When adding any scripts or files to be executed by the autotools build system like tests-ens.sh, please make sure to include ./ so that they run on machines without ./ in PATH env variable.

Good tip. Thanks for pointing that out. I'll try to remember.


macos10.15.4 ntfstool2.2.0 bootcamp无限挂载、死机

macos10.15.4系统、ntfstool2.2.0版本、bootcamp无限挂载、死机。一旦启动这个软件，bootcamp就自动挂载，然后马上被卸载，再重新挂载，无限循环，导致电脑卡死。麻烦查一下原因，谢谢。

The reason is: due to the rapid shutdown of windows10, cache caching, forced loading will damage the data, so linux directly denied access.
The solutions:Re-enter windows and shut down normally.
However, this problem occurs again every time Windows quickly shuts down-restart into Windows, and turn off the Quick Start function in Control Panel-Power Options
As of NTFSTool2.2.0, there is no function to repair the cache forcibly mounting, you can try the above methods to solve it.
We will add the function of automatic detection and repair in subsequent versions.

The latest version V2.3.0 has been updated, and you can experience whether the problem is solved.If it is still not resolved, please contact the official technical service.

macos10.15.6，ntfstool2.3.2， windows系统盘 自动挂载，然后马上被卸载，再重新挂载，循环好多次才停止。如何禁止自动挂在某个盘？

The new version 3.5+ already supports Apple M1/M2/M3 and other chips.
Sorry for taking so long to update, because before that there was no good way to solve the compatibility problem of the new version of the system.
Fortunately, we have now been authorized by Apple to support the new version of the system kernel, so that we can fully support all macos systems through kernel drivers. Please experience it again, and if you have any questions, please update and communicate.


app.component is referencing templateUrl in wrong directory?

The app.component.ts is located in the folder: /src/app/components. It is pointing to the templateUrl in templateUrl: './app/components/app.component.html'
It seems that the templateUrl should be ./app.component.htm
Am I missing something?

The server provides files from both src and node_modules (i.e. two root directories), which means that ./app/components/app.component.html is absolute URL. We do this because there's no universal way to get the current module location (i.e. something like module.id in cjs).

@mgechev
Then how do you explain this error:
angular2.js:23877 EXCEPTION: Failed to load http://localhost:5555/app/components/app/components/app.component.html
(note app/components is there twice)
This is what I get when I start the app using ngp start

@jgenchik  can you show your app.component.ts code ? or at least the templateUrl Key

app.component.ts
import {Component} from 'angular2/core';
import {ROUTER_DIRECTIVES, RouteConfig} from 'angular2/router';
import {NavbarComponent} from './navbar.component';
import {ToolbarComponent} from './toolbar.component';
import {AddNameComponent} from '../../add-name/components/add-name.component';
import {AboutComponent} from '../../about/components/about.component';
import {CampaignsListComponent} from '../../campaigns/list/campaigns-list.component';
import {MyPledgedCampaignsListComponent} from '../../campaigns/list/my-pledged-campaigns-list.component';
import {CampaignComponent} from '../../campaigns/campaign/campaign.component';
import {NameListService} from '../../shared/services/name-list.service';

@Component({
  selector: 'sd-app',
  viewProviders: [NameListService],
  moduleId: module.id,
//   templateUrl: './app.component.html',
  templateUrl: './app/components/app.component.html',
  directives: [ROUTER_DIRECTIVES, NavbarComponent, ToolbarComponent, CampaignComponent]
})
@RouteConfig([
  { path: '/',              name: 'CampaignsList',  component: CampaignsListComponent},
  { path: '/my-pledged-campaigns-list',         name: 'MyPledgedCampaignsList',  component: MyPledgedCampaignsListComponent},
  { path: '/campaign/:id',  name: 'Campaign',       component: CampaignComponent},
  { path: '/add-name',      name: 'AddName',        component: AddNameComponent},
  { path: '/about',         name: 'About',          component: AboutComponent}
])
export class AppComponent {}

app.component.http
<div class="container-fluid">
    <sd-navbar></sd-navbar>
    <router-outlet></router-outlet>
</div>

project.config.ts
import {join} from 'path';
import {SeedConfig} from './seed.config';
import {InjectableDependency} from './seed.config.interfaces';

export class ProjectConfig extends SeedConfig {
  PROJECT_TASKS_DIR = join(process.cwd(), this.TOOLS_DIR, 'tasks', 'project');

  constructor() {
    super();
    this.APP_TITLE = 'madelab.co';
    let additional_deps: InjectableDependency[] = [
        {src: 'chart.js/Chart.min.js', inject: 'libs'}
      // {src: 'jquery/dist/jquery.min.js', inject: 'libs'},
      // {src: 'lodash/lodash.min.js', inject: 'libs'},
    ];

    const seedDependencies = this.NPM_DEPENDENCIES;

    this.NPM_DEPENDENCIES = seedDependencies.concat(additional_deps);

    this.APP_ASSETS = [
      // {src: `${this.ASSETS_SRC}/css/toastr.min.css`, inject: true},
      // {src: `${this.APP_DEST}/assets/scss/global.css`, inject: true},
      { src: `${this.ASSETS_SRC}/main.css`, inject: true },
    ];

    (<any>this.SYSTEM_CONFIG_DEV.paths)['angular2-jwt'] =
        `${this.APP_BASE}node_modules/angular2-jwt/angular2-jwt`;

    // (<any>this.SYSTEM_CONFIG_DEV.paths)['ng2-charts'] =
    //     `${this.APP_BASE}node_modules/ng2-charts`;
    // (<any>this.SYSTEM_CONFIG_DEV.packages)['ng2-charts'] = { 'defaultExtension': false };
  }
}

Thank you for your help

@jgenchik  try to remove themoduleId: module.id,

@gvsdan That did it. Thank you very much!


Fixed a bug in the execute xml jinja template

Overview
Small issue that was causing the rendering of a result XML to crash when using a bbox input.
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[X] I'd like to contribute this bugfix to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[ ] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage decreased (-0.8%) to 73.256% when pulling f7786b7c1e308a83c080addc813f8a847a97f035 on tommygod3:master into 1f6aa6155289aabfbf714f8d43df00f8cc7cf544 on geopython:master.

This is partial fix for #405. It occurred when using lineage=true and the inputs were rendered in a execute response document.

@tommygod3 thanks :)


use monito client cannot automatically update the configuration

Version：

SpringBoot 2.0.2.RELEASE
SpringCloud Finchley.RELEASE

Description：
https://stackoverflow.com/questions/51256955/springcloud-bus-does-not-refresh-using-webhook-client

Demo Project：
https://github.com/Licoy/springcloud-learning

No need to cross post. Closing in favor of stack overflow.


ATLAS and CEM Results

When CEM produces results it should generate a results schema in CEM.RESULTS.  Right now CEM is writing  to writing to WEBAPI database.

Fixed as of: https://github.com/OHDSI/WebAPI/commit/ed524d7e800b9a08918771882f477e9c93cc3fa0


[SPARK-12740] [SPARK-13932] support grouping()/grouping_id() in having/order clause

What changes were proposed in this pull request?
This PR brings the support of using grouping()/grouping_id() in HAVING/ORDER BY clause.
The resolved grouping()/grouping_id() will be replaced by unresolved "spark_gropuing_id" virtual attribute, then resolved by ResolveMissingAttribute.
This PR also fix the HAVING clause that access a grouping column that is not presented in SELECT clause, for example:
select count(1) from (select 1 as a) t group by a having a > 0

How was this patch tested?
Add new tests.

cc @cloud-fan

Test build #55201 has started for PR 12235 at commit 2412671.

overall LGTM

Test build #55201 has finished for PR 12235 at commit 2412671.

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

Merged build finished. Test PASSed.

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

Test build #55229 has started for PR 12235 at commit 8c4bd6c.

@marmbrus Could you also take a quick look on this one?

Test build #55229 has finished for PR 12235 at commit 8c4bd6c.

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

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

Merged build finished. Test PASSed.

LGTM

Merged into master, thanks!


sort by order before validation

It has been observed that while running mongration on multiple dbs for multiple migration scripts , the order in which the records get saved in migration collection is not saved as per their given script sequence. Therefore while validation step , comparison for checksum goes wrong and hence throws an error. This PR is to include a safety measure, where migration db records will always be fetched as per the order sequence so that no comparison goes wrong.

@Harry-027, @sakulstra fixed this for us in #43. I merged that now.
Need to align with @eberhara so we can publish a new version with that.
I'm closing your PR in favor of #43.


Bug/required multiple select

Closes the following:
https://github.com/angular-ui/ui-select/issues/850
and
https://github.com/angular-ui/ui-select/issues/914
The fix adds a custom validator to multiple selects that enables the required tag.  The problem was that required checks the ngModel's viewModel, which when empty on a ui-select-multiple is an empty array...which is technically not empty.  Now it checks properly without interfering with the standard required validator.
Also fixed a test bug where $item.name wasn't being used in the multiple example.

I can remove the Git ignore on the /dist folder if need be.

I have found this problem: When I also have the ng-change attribute pointing to a function and when I remove the last element from the multiple selection then the model is set to undefined.

I believe #1492 fixes this and has been merged, so closing. If you disagree please let us know.


Recipe for building a toolbar app

Description (required)
Build upon https://github.com/withastro/docs/pull/7821, adding a recipe showing how to build a toolbar app. This is intended as the "Non-Reference Guide to Making A Toolbar App:tm:" as the reference can sometimes be a bit too technical when you're getting started.
I tried my best to toe the line regarding "How to build a package" vs "How to build an app". In a fair amount of cases, apps are inside packages, but explaining how to build a package is kinda not the responsability of this page, so I instead choose to show moreso the use case of creating an app for your own website in the code examples, while giving more general advice in the text. I don't know if I succeeded, but I tried, which is what matters.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note
Locale

src/content/docs/en/recipes/making-toolbar-apps.mdx
Source added, will be tracked.
en

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@Princesseuh I added a bit more polish, linked to/from other pages for discoverability etc.
The only comment I'm not sure about is Matthew's -- I tried both append and appendChild and they seemed to work the same for me in this situation. Can you make the call which one we should use here?
Then, I think this is just ready for your review! The failing test won't happen once the new dev toolbar API page is also available. I'll start pointing both of these PRs to a new 4.7.0 branch and then that link should be fine.

Merging into 4.7.0 branch! Will perform final reviews/checks/polish in there if needed!


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: S-Anoob sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Zoom requires 'host' to start meeting?

I followed the docs here: https://zulip.readthedocs.io/en/stable/production/video-calls.html
I added an app on Zoom and the I can create a Zoom link in Zulip chats.
However, when the Zoom is started it is always displays "Please wait for the host to start the meeting".
As I understand it, Zoom is supposed to use the first person who clicks the link, not the Zoom app creator.
So currently it means that no one can use it except the 'host' who created the Zoom app :(
Update: in case it's relevant, the Zulip server is v4.4

@ebecaf thanks for the report. Everyone who joins the meeting should be a host. I think the most likely explanation is you have enabled a Zoom organization-level setting that controls this.
I'm not sure how that detail of Zoom's integrations system works; it's reasonably likely that there's some toggle you need to do in the Zoom App's configuration, but I'm not sure which. You might try asking Zoom support what aspect of their API controls that -- I don't remember seeing anything that would control that.


i cant it libery in eclipce

hi dear,
I want libery android-shape-imageview import into eclipse
but i cant.i want just  diamond shape image view
Is it possible for me to put diamond class ????

@iman100 I am sorry I didnt understand what you mean.


Loader: Fix OOM in loaderGetDeviceRegistryFiles()

loaderGetDeviceRegistryEntry() out of memory result was not handled properly.

 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.

This should be marked with bug label


Added new game

PR Description 📜
Added a new game which is a simple and fun game built using HTML, CSS, and JavaScript. Your objective is to whack the balls that appear on the screen and earn points.
Fixes #3015

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

Thank you @Code1705 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @Code1705 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Spec improvement: Enable data to reside in files while still describing relationships via foreign keys for discoverability

Schematize relationships of files
Summary of the issue
More and more data is not tabular: images, videos, html, etc… But it does have relationships.
The current datapackage.org spec allows to define relationships between table fields, but it does not allow it on pure file resources.
A workaround is to have a special table resource that lists the files and leverages the foreignKeys spec, but it mixes data and schema concepts. Indeed, one column of the table resource is a path to another resource.
Why is this a problem?
Well, the great benefit of using datapackage.org is to enable discoverability of data and a programmatic one even. So that schemas are defined and a whole database schema can be generated without reading the content of each resource. If the relationship is defined inside the data (resource content), then discoverability cannot happen because a script cannot discover these relationships by purely following the datapackage.org standard.
What could be done?
We will use the following use case
./people-data/
  |__ people.csv # list of people
  |__ traits.csv # several traits per person, one row = one trait
  |__ people_files/
    |__ person1/
      |__ profile.jpg
      |__ randomName.png
    |__ person2/
      |__ profile.jpg
      |__ randomName.png

Option 1: Special fileTable table resource that describes data located in files and its references to other resources via foreign keys
We extend the table type for a fileTable type which enforces the first field to be file_path

file_path must be a a valid path to a file in the datapackage

Q: Do we need to declare each file resource when they are described in a fileTable?
resources:
  - name: people
    type: table
    path: ./people.csv
    scheme: file
    format: csv
    mediatype: text/csv
    schema:
      fields:
        - name: id
          type: string
        - name: name
          type: string
  - name: traits
    type: table
    path: ./traits.csv
    scheme: file
    format: csv
    mediatype: text/csv
    schema:
      fields:
        - name: person_id
          type: string
        - name: eye_color
          type: string
    foreignKeys:
      - fields: ["person_id"]
        reference:
          resource: people
          fields: ["id"]
  - name: files
    type: fileTable # special resource type
    schema:
      fields:
        - name: file_path # first field has to be filepath
          type: string
        - name: person_id
          type: string
        - name: mediatype
          type: string
    foreignKeys:
      - fields: ["person_id"]
        reference:
          resource: people
          fields: ["id"]
    data:
      - file_path: ./people_files/person1/profile.jpg
        person_id: person1
        mediatype: image/jpeg
      - file_path: ./people_files/person1/randomName.png
        person_id: person1
        mediatype: image/png
      # etc ...

Option 2: File resources can reference a virtualTable parent
The idea here is that the schema can still be defined in a table like resource but the data is in files that have to reference it AND have the same fields as properties
This allows for data frames to be in separate JSON files for examples and leverages the existing metadata definition for files like format, mediatype etc…
resources:
  - name: people
    type: table
    path: ./people.csv
    scheme: file
    format: csv
    mediatype: text/csv
    schema:
      fields:
        - name: id
          type: string
        - name: name
          type: string
  - name: traits
    type: table
    path: ./traits.csv
    scheme: file
    format: csv
    mediatype: text/csv
    schema:
      fields:
        - name: person_id
          type: string
        - name: eye_color
          type: string
    foreignKeys:
      - fields: ["person_id"]
        reference:
          resource: people
          fields: ["id"]
  - name: files
    type: virtualTable # special resource type, virtual, only has schema
    schema:
      fields: # some fields have to be defined in each frame
        - name: person_id
          type: string
    foreignKeys:
      - fields: ["person_id"]
        reference:
          resource: people
          fields: ["id"]
  - name: people_files_person1_profile_jpg
    type: virtualTableFrame
    path: ./people_files/person1/profile.jpg
    mediatype: image/jpeg
    format: jpg
    data:
      person_id: person1
    # path: path/to/json/file.json {"person_id": "person1"}

Hi, why not use option 1, but as a regular tabular data resource?
resources:
  - name: people
  - name: traits
  - name: files
    type: table # Regular tabular resource
    schema:
      fields:
        - name: person_id
          type: string
        - name: file_path # Does not need to the first field
          type: string
          format: default # We could benefit here from an additional format "path", cf. "uri" for URLs
        - name: mediatype
          type: string
    foreignKeys:
      - fields: ["person_id"]
        reference:
          resource: people
          fields: ["id"]
    data:
      - person_id: person1
        file_path: ./people_files/person1/profile.jpg
        mediatype: image/jpeg
      - person_id: person1
        file_path: ./people_files/person1/randomName.png
        mediatype: image/png
      # etc ...

We use this approach to describe media files in a camera trap study (including foreign keys):

Table schema: https://github.com/tdwg/camtrap-dp/blob/main/media-table-schema.json
Human readable: https://camtrap-dp.tdwg.org/data/#media

Hi @peterdesmet 👋  Thank you very much for reviewing this 🙏

Why not a regular table resource?

Ive described the limitation of this approach in the description

Why is this a problem?
Well, the great benefit of using datapackage.org is to enable discoverability of data and a programmatic one even. So that schemas are defined and a whole database schema can be generated without reading the content of each resource. If the relationship is defined inside the data (resource content), then discoverability cannot happen because a script cannot discover these relationships by purely following the datapackage.org standard.

in more practical terms: would a system be able to auto-discover the data is also contained in files just thanks to the datapackage specs?
answer is no atm: one has to know there is a file_path field that relates to the current datapackage:

what if there are several files?
what if the field is called image_file_path, img_path, etc...

basically data in files is not first class citizen the same way table resources are and it limits automatic discoverability


[Explore] Adding Adhoc Filters

Here I'm following a similar design pattern as Adhoc Metrics (https://github.com/apache/incubator-superset/pull/4663 and https://github.com/apache/incubator-superset/pull/4736). Adhoc Filters introduces a new control that unifies the 4 previous filter controls (free text where, free text having, structured where filter, structured having filter). It allows you to select a column or any saved metric / adhoc metric currently being used in the metric field to filter by:

and then once a metric or column is selected lets you switch between writing a SQL filter or simple filter for that metric/column:

(for druid datsources, metric and column filters can only be edited in the simple view. for table datsources, column filters can be edited in either but metrics filters can only be edited in the sql view)
This PR also removes the legacy filter controls. When loading a chart that has legacy filters already attached to it, they will be cast to adhoc filters:

Here are some more demos of the feature in action,
creating and editing filters:

transitioning between simple tab and sql tab:

selecting adhoc metrics to filter on:

notes to reviewers:
The meat of the frontend lies in the three AdhocFilterEditPopover* files and the AdhocFilterControl file. The meat of the backend lies in viz.py where I break adhoc filters out into the four base types of filters for processing.
reviewers:
@michellethomas @williaster @graceguo-supercat @john-bodley @mistercrunch

Codecov Report

Merging #4909 into master will increase coverage by 0.06%.
The diff coverage is 92.85%.

@@            Coverage Diff            @@
##           master   #4909      +/-   ##
=========================================
+ Coverage   77.14%   77.2%   +0.06%
=========================================
  Files          44      44
  Lines        8539    8578      +39
=========================================
+ Hits         6587    6623      +36
- Misses       1952    1955       +3

Impacted Files
Coverage Δ

superset/connectors/druid/models.py
81.11% <100%> (-0.06%)
:arrow_down:

superset/utils.py
88.12% <100%> (+0.05%)
:arrow_up:

superset/viz.py
80.22% <91.66%> (+0.32%)
:arrow_up:

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 e1d2150...c85c7f1. Read the comment docs.

Made a couple of aesthetic changes based on feedback from the team
Pills now appear darker and more clickable:

the text area is also wrapped in a thin border (and by wraps overflow text to new lines rather than hiding it):

I also introduced some ways to give feedback that simple filters cannot filter on metrics in table datasources:

@williaster @michellethomas @john-bodley comments addressed, PTAL a when you have the chance!

🔥 🎆 🍾 🎊

This is causing a bug with filters not being saved when visualizations are changed. specifically when switching to Table View: https://github.com/apache/incubator-superset/issues/5172


 We can set workdir, which tells us where to run the action. But it'd be nice if there was a way to ignore a list of directories.

Yes, you can pass arguments through detect_secrets_flags.
detect-secrets can ignore different cases, for example, you can ignore directories using regex:
    - uses<EMAIL_ADDRESS>      with:
        github_token: ${{ secrets.github_token }}
        reporter: ${{ github.event_name == 'pull_request' && 'github-pr-review' || 'github-check' }}
        workdir: ${{ matrix.project }}
        fail_on_error: 'true'
        detect_secrets_flags: --all-files --force-use-all-plugins --exclude-files (.git|.github/actions)/

Thank you! This solution does seem to work.


Impossible to add time_precision in SeriesHelper / commit

Hello,
Could you add a new feature to SeriesHelper ?
I can't add time_precision whereas write_points function afford it.
write_points(points, time_precision=None, database=None, retention_policy=None, tags=None, batch_size=None, protocol=u'json', consistency=None)
whereas commit() countains: rtn = client.write_points(cls.json_body()) (line 146)
I thank you for this new feature.
Kind regards

**InfluxDB version: 1.7.9
**InfluxDB-python version: 5.2.3
**Python version: 3.7.3
**Operating system version: Debian9

@syndrael thanks for opening this. If you're up for it, we'd love to review your PR to fix this issue. Otherwise, we will consider it for a future release.

Fixed in #502 and #719


add import as keyword

Some tinkering!

I prefer as, it's clearer than in, although you could keep in and not add a token.
I like the syntax of import "io" for File as F, Process as P it reads nicely
It's optional, so import "io" for File as F, Process is valid

Thoughts welcome!

I love tinkering!
I did something similar with "as" way back when I was working on some other language. I agree it's clearer than in. I did always feel weird that it reads in the opposite direction as assignment. Assignment is dest = source and as is source as dest.
My general tendency towards minimalism makes me hesitate to pull this in, but I'm curious to see what others think. It does look nice to my eye, the implementation is simple and clean (always a good sign), and other languages have similar functionality.

Yea that makes sense. To add more reasoning, I have definitely found as projects get larger that the overlap of variable names can become a frustrating problem, not just because of the encouragement to decorate variable names unnecessarily (Button vs UIButton), but specifically because it creates an impossible situation that is dumped on the users lap, with no tools to tackle it.
I find for example, not being able to name a class anything that matches the name of an import, the only option is forcing my code into less intuitive naming patterns.
Another example is two unrelated modules, maybe there's a Button in moduleA and a Button in moduleB - both from third parties. My code cannot use both in the same place, it is impossible. The user is forced to rearrange their code, edit the third party module code, or just not use them. None of that sits well with me.
While I highly appreciate the minimalism, the intent is to offer a tool to mitigate impossible or frustrating situations that the user didn't create, but the language allows :)

I would love having at least the import option there - being able to "rename" a variable during imports may come in really handy! Also, I agree that in and as read totally the opposite way... Well, in the for loop, it does make sense for in to be used.
for(line in text) ...

Whereas this would feel odd again:
for(line as text) ...

So I totally support @underscorediscovery 's 2nd idea, but am unsure about the first. Though, third is absolutely important.


Open Source Maintenance

all-contributors please add @HamzaMateen for code

@all-contributors please add @HamzaMateen  for code

I can't thank you enough!!
I am so much delighted to see this.. Thank you so much.
One request tho!
Will I have to accept the pull request?
I dont know much of git and GitHub yet!
On Sun, Oct 25, 2020, 10:27 AM allcontributors[bot] <
<EMAIL_ADDRESS>wrote:

@AhmedRaja1 https://github.com/AhmedRaja1
I've put up a pull request
https://github.com/AhmedRaja1/Hacktoberfest/pull/183 to add @HamzaMateen
https://github.com/HamzaMateen! 🎉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AhmedRaja1/Hacktoberfest/pull/161#issuecomment-716097056,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKKXMVZOLV7XG4DV26MF6BTSMOZMJANCNFSM4S2HUYXQ
.


Reload spread if Webview is terminated

Fix for https://github.com/readium/swift-toolkit/issues/309

Hi @triin-ko, did you find an issue with this solution?

@mickael-menu yes, I looked over it with a clear head this morning and now I created a new PR for the same issue: https://github.com/readium/swift-toolkit/pull/311


Restarting game freezes entire system

Tested versions
This is for 4.3.beta1.
System information
Godot v4.3.beta1 - EndeavourOS #1 SMP PREEMPT_DYNAMIC Fri, 31 May 2024 15:14:45 +0000 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2080 SUPER (nvidia; 550.78) - Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz (6 Threads)
Issue description
When I click the restart game button while the game is running, it completely freezes my entire system until the game restarts. It also shuts down recording with software like OBS. I tried to make a video, but the last thing recorded is right when the system freezes.
Also of note is that I upgraded my game from 4.2.2 to 4.3.beta1.
https://github.com/godotengine/godot/assets/3057229/41d66bb9-5b57-4c0d-82ab-520dbbeff68f
Steps to reproduce
Start game, then restart it.
Minimal reproduction project (MRP)
I don't have one as of yet.

I can't reproduce the issue here with the projects i have so probably is something specific in your game, without a mrp will be hard do something

Indeed please upload an MRP:

A small Godot project which reproduces the issue, with no unnecessary files included. Be sure to not include the .godot folder in the archive (but keep project.godot).
Having an MRP is very important for contributors to be able to reproduce the bug in the same way that you are experiencing it. When testing a potential fix for the issue, contributors will use the MRP to validate that the fix is working as intended.
Drag and drop a ZIP archive to upload it (max 10 MB). Do not select another field until the project is done uploading.
Note for C# users: If your issue is not C#-specific, please upload a minimal reproduction project written in GDScript. This will make it easier for contributors to reproduce the issue locally as not everyone has a .NET setup available.

Also your video is broken for at least me

Closing due to lack of response, if you provide the requested information this can be reopened


Removing, umount and detaching loop device doesn't clear the btrfs.list

Scenario
1-  Do client.btrfs.device_remove
2- umount and detach the loop device
Actual Result
3- List btrfs filesystems using client.btrfs.list(), the btrfs filesystem is still there
Expected result
The btrfs filesystem should be removed

@john-kheir according to u, this is no longer an issue since it's just cached and need few seconds to clear. Can u close this issue now?


Create an SAP Fiori App Using SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
In step 3.6 new parameter "Minimum SAPUI5 version" has been added to the template. So the screenshot needs to be updated and also the new parameter has to be added in the table.
Expected:

Actual:

Scenario Label: SCPTest-fbizapp
Sprint Label: SCPTest-2203A
Landscape Label: trail3, trail2
Priority: Medium

Fixed and PR created.


Adds ability to cancel jobs associated to a commit.

Towards https://github.com/ocurrent/ocurrent/issues/286
See also https://github.com/ocurrent/ocaml-ci/pull/452

For reference the Cancel button should appear on the commit page for a PR
eg https://opam.ci.ocaml.org/github/ocaml/opam-repository/commit/1250827a23788c3530de1e072eb297fa067dcdec


remove the closeCallback from the session

This is what I mentioned in https://github.com/lucas-clemente/quic-go/issues/564#issuecomment-296434663.
The closeCallback was run when a session was closed, i.e. after the run loop of the session stopped. Instead of explicitely calling this callback from the session, the caller of session.run() can just execute the code after session.run() returns.

Codecov Report

Merging #572 into master will decrease coverage by 0.03%.
The diff coverage is 91.3%.

@@            Coverage Diff             @@
##           master     #572      +/-   ##
==========================================
- Coverage   90.79%   90.76%   -0.03%
==========================================
  Files          79       79
  Lines        5407     5413       +6
==========================================
+ Hits         4909     4913       +4
- Misses        318      319       +1
- Partials      180      181       +1

Impacted Files
Coverage Δ

session.go
86.54% <100%> (-0.06%)
:arrow_down:

server.go
90% <100%> (+0.14%)
:arrow_up:

client.go
83.56% <84.62%> (-0.72%)
:arrow_down:

Continue to review full report at Codecov.

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

@lucas-clemente: I changed the error handling in the session. Please have another look.


Implement SRTCP Decryption

Extend the SRTP package to also support SRTCP, this only implement
the decryption side
Relates to #117

Pull Request Test Coverage Report for Build 842

68 of 90 (75.56%)  changed or added relevant lines in 5 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.09%) to 48.465%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/network/port-send.go
0
1
0.0%

internal/srtp/srtcp.go
16
20
80.0%

internal/srtp/context.go
20
24
83.33%

internal/network/port-receive.go
0
13
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/dtls/dtls.go
1
18.25%

internal/srtp/context.go
1
84.62%

Totals

Change from base Build 831:
0.09%

Covered Lines:
2700

Relevant Lines:
5571

💛  - Coveralls


sslHostnameValidationEnabled=false is actually enabling hostname validation

Hello Team
When investigating another issue I raised : https://github.com/spring-cloud/spring-cloud-netflix/issues/2498
I found that the zuul.sslHostnameValidationEnabled option is doing the opposite of what it should do:
sslHostnameValidationEnabled=true will disable ssh validation
sslHostnameValidationEnabled=false will enable ssh validation
https://github.com/spring-cloud/spring-cloud-netflix/blob/4e8d58869e800ab5491635e046b240e3eb235b36/spring-cloud-netflix-zuul/src/main/java/org/springframework/cloud/netflix/zuul/filters/route/SimpleHostRoutingFilter.java#L145
and the prototype is:
	@Override
	public HttpClientConnectionManager newConnectionManager(boolean disableSslValidation,
			int maxTotalConnections, int maxConnectionsPerRoute, long timeToLive,
			TimeUnit timeUnit, RegistryBuilder registryBuilder) {

I will try to contribute this trivial fix

I fixed it in my environment by overriding DefaultApacheHttpClientConnectionManagerFactory and fixing the way ssl is enabling hostname validation.
https://github.com/qcastel/forgerockmtlswithzuul/blob/dfa0f833622bff41ebbc9429104c985a2f719c9c/forgerock-zuul/src/main/java/com/forgerock/example/mtls/zuul/ssl/ZuulApacheHttpClientConnectionManagerFactory.java#L90


Make the GUI interfaces selection actually honored by ntpd

Without the "interface ignore wildcard" directive, the service still listens on wildcard.

Funky... thanks!

Well yes, this annoying behaviour has been a source of complaints probably since forever - e.g. Ubuntu bug from 2011. Don't know why upstream insists on this. 🤦 🤷♂
There's some write-up and discussion about this at their support website
Thanks.


Dependency definition issues

I have observed that throughout the Simphony project dependencies are defined implicitly. The use of >= sign instead of == and not putting the full output of pip freeze in requirements.txt file would be problematic in future.
I suggest to define direct and indirect dependencies in requirements.txt explicitly. This will let pip to install the exact versions that we are sure the application works fine with them. Moreover, in case of reusable components we can use a dependency range inside setup.py files. This will allow the component to support minor dependency upgrades while major updates would need exclusive attention of developers and an update in our side.
Since this project is like a toplevel project, I register this issue here.

I have observed that throughout the Simphony project dependencies are defined implicitly. The use of >= sign instead of == and not putting the full output of pip freeze in requirements.txt file would be problematic in future.

Using == might be a safe solution (and it is usually the case for application development) but is not always the best for libraries. It does not make sense for example to set  a version for flake8  or mock. In general it is in the developers discrepancy to decide when it makes sense to use version selection logic. For example, I think that having a specific version for simphony-common is a good choice but for mayavi the api does not change that much. I personally prefer to have >= to make it easier to test with new versions.
pip freeze is in general has more that the actual (first level) dependencies (including my personal developing tools on windows) so I do not advise people to use pip freeze for more than a starting point.
Please also note that requirement is not the official way to install python packages. The -r option is a quick a nice way to setup an environment for a specific use (e.g documentaiton build, testing, developments on windows, python 2.6 testing). In reality if we want to be right one needs a separate 'requirements.txt' for different platforms (especially if one uses freeze to create the requirements).
Having a properly defined setup.py and an updated README documentation, is the right way to go and all first level dependencies should be mentioned there, optional dependencies can be also included (see setup.py for simphony-common). Again there my preference is to use >= unless I know a case where a new version of does not work.
These are my rule of thumb. But in general, the version selection is a big subject and without specific items on the discussion one cannot make inform choices.

Since this project is like a toplevel project, I register this issue here.

I think that for version issues is better to point to specific problems on the related package(s).

Finally as along as we use continuous integration in a sensible way (see example https://github.com/enthought/traits-enaml/blob/master/.travis.yml) we can quickly get a sneak peak at possible future problems and act accordingly

Using >= sign makes it easier for developers to install and work with packages specially during the development phase, but later when we move the releases to PyPI (I assume we are going to do this) then having explicit versions will assure that a user will get the correct dependencies and pip package manager would be able to download and install them correctly.
Packages such as flake or similar ones are required for development but not for end users so it is not important here. For other symphony and other third party package that we rely on their API, it is much better to define the exact version. In this way we can make sure that in future, when there could be multiple newer releases for the dependencies either simphoy or third party, the package manager would be able install a working combination of dependencies.
There are some rare occasions that it is important to define second level dependencies as well which I don’t find it necessary for now.
The last but not least, this project like many others will not be forever in hot development phase to have dependency problems fixed when they happen. Software is subject to change, therefore I think it is much safer to define explicit dependencies at least for major releases.
Von: Ioannis Tziakos<EMAIL_ADDRESS>Gesendet: Freitag, 15. Mai 2015 17:29
An: simphony/simphony-framework
Cc: Sadeghi, Mehdi
Betreff: Re: [simphony-framework] Dependency definition issues (#15)
I have observed that throughout the Simphony project dependencies are defined implicitly. The use of >= sign instead of == and not putting the full output of pip freeze in requirements.txt file would be problematic in future.
Using == might be a safe solution (and it is usually the case for application development) but is not always the best for libraries. It does not make sense for example to set a version for flake8 or mock. In general it is in the developers discrepancy to decide when it makes sense to use version selection logic. For example, I think that having a specific version for simphony-common is a good choice but for mayavi the api does not change that much. I personally prefer to have >= to make it easier to test with new versions.
pip freeze is in general has more that the actual (first level) dependencies (including my personal developing tools on windows) so I do not advise people to use pip freeze for more than a starting point.
Please also note that requirement is not the official way to install python packages. The -r option is a quick a nice way to setup an environment for a specific use (e.g documentaiton build, testing, developments on windows, python 2.6 testing). In reality if we want to be right one needs a separate 'requirements.txt' for different platforms (especially if one uses freeze to create the requirements).
Having a properly defined setup.py and an updated README documentation, is the right way to go and all first level dependencies should be mentioned there, optional dependencies can be also included (see setup.py for simphony-common). Again there my preference is to use >= unless I know a case where a new version of does not work.
These are my rule of thumb. But in general, the version selection is a big subject and without specific items on the discussion one cannot make inform choices.
Since this project is like a toplevel project, I register this issue here.
I think that for version issues is better to point to specific problems on the related package(s).
—
Reply to this email directly or view it on GitHubhttps://github.com/simphony/simphony-framework/issues/15#issuecomment-102432748.

Using >= sign makes it easier for developers to install and work with packages specially during the development phase, but later when we move the releases to PyPI (I assume we are going to do this) then having explicit versions will assure that a user will get the correct dependencies and pip package manager would be able to download and install them correctly.

This is not exactly true. As I said == is advised for applications deployed on separate environment. However on a library environment it does not help much.  For example pandas is very common scientific package, both simphony and pandas depend on numpy, but pandas tends to use the latest numpy for features and speed, If we set numpy==1.9.1 for simphony-common and release it no PyPi it will be likely that after a few months people will not be able to properly install simphony-common in a virtual environment that uses the latest pandas which will require numpy>=1.9.2.
We agree however that we should alway use an exact version for simphony-commons in simphony-frameworks (see https://github.com/simphony/simphony-framework/blob/master/simphony_packages.txt).

Packages such as flake or similar ones are required for development but not for end users so it is not important here.

We agree on that.

There are some rare occasions that it is important to define second level dependencies as well which I don’t find it necessary for now.

we also agree on that (that is why pip --freeze is not advised).

The last but not least, this project like many others will not be forever in hot development phase to have dependency problems fixed when they happen.

Having == for all depedencies, in a lot of cases it is actually hindering use of the library (see examples above). One needs to practice common sense and make the best of testing tools. If one needs to be in the conservative side on  you could use package>=1.4.4,<2 because one might expect that the api will not be backwards compatible when major version changes.

Software is subject to change, therefore I think it is much safer to define explicit dependencies at least for major releases.

This is problematice it is confusing sense for a package at version 1.2.3.dev23 to require requirement-0.8>=0 and the release version 1.2.3 to require requirement-0.8==0

Today I came across a good read on dependency definition problem in python projects. In that article, Kenneth Reitz, suggests a simple workflow, which is having two different requirements files for each project, one with explicit version numbers, and the other one without them.
As a result, a requirements-to-freeze.txt will contain only package names, without version information and requirements.txt will contain the output of pip freeze, after a fresh install in a virtual environment of course.

Final decision on this topic?


"shodowPassword VirtualSession failed" when visit "/admin/config"

Gitea version (or commit ref): 1.10.0 / e3875ac
Git version: 2.17.1
Operating system: Ubuntu 18.04
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

...uters/admin/admin.go:224:shadownURL() [E] shodowPassword VirtualSession failed: parse {"Provider":"file","ProviderConfig":"/path/data/sessions","CookieName":"i_like_gitea","CookiePath":"","Gclifetime":86400,"Maxlifetime":86400,"Secure":false,"CookieLifeTime":0,"Domain":"","IDLength":0,"Section":"","IgnoreReleaseForWebSocket":false}: first path segment in URL cannot contain colon

Description
N/A
Screenshots
N/A

same issue with
Database: PostgreSQL
Gitea version: 1.11.0+dev-164-g8d8aecd3c built with GNU Make 4.2.1, go1.13.4 : bindata, sqlite, sqlite_unlock_notify
Git version: 2.22.0


This library is unusable

https://github.com/ThePhD/sol2/blob/develop/sol/load_result.hpp#L83
https://github.com/ThePhD/sol2/blob/develop/sol/load_result.hpp#L98
seriously

Sob. You found my weakness.

See this is why you run a grammar pass on comments during CI builds.


Recent Activity returning league does not exist for private football league

Sport
Football
Summary
Trying to access the trade activity for my league. Have included the s2 and swid cookies when defining the league and am able to access league data (team records, player performance, etc.). However, when I try to request recent activity, I get: ESPNInvalidLeague: League ##### does not exist. I have tried this across multiple leagues with the same result. Using latest release version (0.36.0).
ps. Thank you for all the hard work on this. It's incredibly useful!
Logs/Data
No response

Can confirm that this is happening for me as well on 0.36.0

I'm sadly not sure the historical data recent_activities() would pull from even exists anymore. If you go to your league homepage and select the drop down to look at another year's data, "recent activities" is no longer an option (or maybe never was, but I feel like this used to work).
So it seems there has either been a breaking change with recent activities, or it will only return the current seasons data. Again, I don't really remember how this worked in the past.
I'd still love to be able to analyze transactions of past years, and the data exists in each player card (go back to whatever season, click a player, look at their transaction history), so maybe the data can still be scraped yet.

Yeah it does seem like they have removed the ability to see previous years recent activity.
I could definitely look at adding transaction history to the player data. I think it would have to be accessed through the player_info call but looks like they still include that data.


cantata: 2.5.0 -> 3.2.0

Description of changes
Original developer archived the repo a few years ago: https://github.com/CDrummond/cantata
Switched to an updated Qt6 fork: https://github.com/nullobsi/cantata
Also enabled libvlc for playback over http.

Built on platform(s)

[x] x86_64-linux

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

we also need taglib2 for the new version of cantata

@peterhoeg good call. are taglib1 apis implemented in taglib2 (backwards compatible) or should I create a new taglib2 package?

Added taglib2. It's just a slight modification taglib's default.nix only changing the version number & hash and adding utf8cpp buildInput:
$ diff -u pkgs/development/libraries/taglib/default.nix pkgs/development/libraries/taglib2/default.nix
--- pkgs/development/libraries/taglib/default.nix       2024-05-08 02:28:50.857004855 +0300
+++ pkgs/development/libraries/taglib2/default.nix      2024-06-07 12:22:03.017977473 +0300
@@ -4,22 +4,23 @@
 , cmake
 , zlib
 , testers
+, utf8cpp
 }:

 stdenv.mkDerivation (finalAttrs: {
   pname = "taglib";
-  version = "1.13.1";
+  version = "2.0.1";

   src = fetchFromGitHub {
     owner = "taglib";
     repo = "taglib";
     rev = "v${finalAttrs.version}";
-    hash = "sha256-QX0EpHGT36UsgIfRf5iALnwxe0jjLpZvCTbk8vSMFF4=";
+    sha256 = "sha256-zoZir1TtYtEEZBudlGRLYPFIFJl5oad4+l64lytJ+bc=";
   };

   nativeBuildInputs = [ cmake ];

-  buildInputs = [ zlib ];
+  buildInputs = [ zlib utf8cpp ];

   cmakeFlags = [
     "-DBUILD_SHARED_LIBS=ON"

The original taglib wasn't replaced since some APIs were deprecated with the new major version: https://github.com/taglib/taglib/blob/master/CHANGELOG.md

The package naming sanitizer is complaining about me adding pkgs/development/libraries/taglib/2.x.nix to pkgs/top-level/all-packages.nix due to the new pkgs/by-name hierarchy despite the migration guideline saying to "Please hold off migrating your maintained packages to this directory"...
Feedback?


Add --device

Codecov Report

Merging #426 into master will increase coverage by <.01%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##             master     #426      +/-   ##
============================================
+ Coverage     79.18%   79.18%   +<.01%
- Complexity      605      613       +8
============================================
  Files            69       69
  Lines          1763     1773      +10
  Branches        267      269       +2
============================================
+ Hits           1396     1404       +8
  Misses          196      196
- Partials        171      173       +2


can you give me a demo for binance trading example can Take Profit Stop Loss?

can you give me a demo for binance trading example can Take Profit Stop Loss?  thanks in advance.

In exc, we prefer to do it manually, therefore we haven't add support to the stop-loss/take-profit APIs.
To setup a stop-loss / take-profit condition manually, the following approach may be helpful:

Firstly, you need to determine your current position status. You can do this in a per-order way with CheckOrderService.
Secondly, you should subscribe to the TickerStreams of the instruments of your positions.   Every time a new Ticker arrives, you can calculate the unrealized profit and loss of each orders to see if you need to perform a stop-loss / take-profit.

We have also provided utils for these kinds of calculations:

For managing the positions status, see https://docs.rs/positions/latest/positions/instrument/struct.Instrument.html#method.position .
And for calculating the unrealized profit and loss, see https://docs.rs/positions/latest/positions/position/struct.Position.html#method.closed.

I think I have answered your question, so I am going to close this issue. Feel free to reopen if you need more explanation.


[armadillo] update to 12.6.7

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

This isn't the latest version, but it seemed reasonable to fill in the 12.6.* version support before adding support for newer versions. I plan to add 12.8.4 after this is submitted.

Thanks for the update!