"Configure Default Build Task" doesn't work with Cake

Normally, you can configure a default build task so that pressing ctrl+shift+b will immediately run the task. You can set this up by selecting from the command palette "Tasks: Configure Default Build Task". It will then give you a list of tasks it understands for you to pick from. Select "Cake: Default". This adds an entry to your tasks.json file. That entry is:
{
	"type": "cake",
	"script": "Default",
	"group": {
		"kind": "build",
		"isDefault": true
	}
}

However, the console immediately shows the error:
Error: there is no registered task type 'cake'. Did you miss to install an extension that provides a corresponding task provider?

Now, pressing ctrl+shift+b opens a dropdown to select which task to run. I would expect it to run the configured task immediately without any prompting.  This is the behavior I get if I set up some other kind of tasks as my default build task.

@WalkerCodeRanger can you help me understand what you are doing here?  Where are you finding the option to select Cake: Default?  I am not seeing that option.

Sorry, that was confusing. I have a task named "Default" in my cake build file. Just pick any cake task that you want to run as the default build task.

I can not reproduce that problem. I'm guessing  @gep13 fixed it in https://github.com/cake-build/cake-vscode/commit/5cd4777f30d09b8f21815b0a42f2351db5365968.
@WalkerCodeRanger Do you still have the problem?

I'm no longer using Cake so I can't say if the issue still exists or not.

@gep13 I'd vote to close this issue. I think the problem was fixed (see above) and I can no longer reproduce it.


#312 Update travis to use xcode 9.1

Closes #312
Checklist

[x] I checked the Contributing Guidelines before creating this request.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

fails due to [!] SwifterSwift did not pass validation, due to 29 warnings (but you can use --allow-warnings to ignore them).
should i wait for #305 to be merged?

Hey @kaphacius, please wait for #305 and #309 to be merged,
Thank you for taking care of this :)

@kaphacius Please update from master to merge this PR :)

looks like the new image has a different tv os simulator name, have to change that as well

Thanks @kaphacius!


Portals - Remove node naming restrictions + further work

Portals - Remove node naming restrictions
The use of special prefixes is only actually required during the import phase - the first conversion of rooms, roomgroups, and portals from Spatials and MeshInstances (based on the workflow of importing from blender).
Once converted to the native Godot nodes there is no longer a need for the naming requirements.
This PR removes the requirements except for the import. Manual portal linking after the initial conversion is now done exclusively using the linked_room nodepath property of the Portal.
Portals - change import naming convention to postfix
In response to user demand, the naming convention for importing levels from blender etc is changed from prefixes Room_ and Portal_ to postfixes -room, -roomgroup, -portal.
Notes

This is a popular request - there is no need to have the strict naming requirements for the rooms and portals once the initial import has taken place. Or indeed if a user is not using the import from blender workflow, and is creating the nodes manually in the editor.
There is a small possibility of regressions, which is why I left changing this to after the first beta. I've tried to test this with as many scenarios as possible.
This is well worth getting working as it should make things simpler for users, less things to get wrong / error messages during conversion.
Naming conventions for import from Blender etc are changed to match Godot 2, and follow the -col type convention.

Thanks!


Unhandled Rejection (Error): GraphQL error: String cannot represent value:

Hi,
I hope this is the right repo for this, as it is linked here from https://pypi.org/project/django-graphql-jwt-reload/
Using:
django-graphql-jwt-reload 1.3.6
and the mutation:
mutation TokenAuth($username: String!, $password: String!) {   tokenAuth(username: $username, password: $password) {     token,     payload,   } }
I get the error
Unhandled Rejection (Error): GraphQL error: String cannot represent value: b'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VybmFtZSI6InRyYWluZXIiLCJleHAiOjE2MTcxOTA0ODZ9.owSp5Bn-T8Hhh1vI6wr4DTdWLm42-4ZKLy3V30rIp9w'
when I remove token from the query it seems to work, however then I do not have the token.
Any ideas on how to fix this?
thanks

Ok, looks like the issue comes from unstable versions required by graphene-django-extras: 0.5.1 . reverted back and am now removing this dependency. hopefully this will solve my issue

I'm still facing this bug. Any solutions?
These are the dependencies:
aniso8601==7.0.0
asgiref==3.2.3
astroid==2.3.3
autopep8==1.5
Django==2.2.10
django-cors-headers==3.2.1
django-environ==0.4.5
django-filter==2.4.0
django-graphql-jwt==0.3.2
django-safedelete==0.5.4
djangorestframework==3.11.0
djangorestframework-jwt==1.11.0
graphene==3.0b7
graphene-django==3.0.0b7
graphql-core==3.1.4
graphql-relay==3.1.0
gunicorn==20.0.4
isort==4.3.21
jsonfield==2.0.2
lazy-object-proxy==1.4.3
mccabe==0.6.1
Pillow==8.1.0
promise==2.3
psycopg2-binary==2.8.6
pycodestyle==2.5.0
PyJWT==1.7.1
pylint==2.4.4
pytz==2019.3
Rx==1.6.1
singledispatch==<IP_ADDRESS>
six==1.14.0
sqlparse==0.3.0
text-unidecode==1.3
typed-ast==1.4.1
whitenoise==5.2.0
wrapt==1.11.2

I recreated a python3.7 venv, reinstalled all my packages, and it fixed itself.
I'd suggest you have a broken python3 environment somehow, mine was very messy after an upgrade from 3.6 to 3.7


bump hcsshim version to v0.8.17

What this PR does / why we need it:
This PR bumps the hcsshim version to v0.8.17
Additional context
Full diff<EMAIL_ADDRESS>Link to the new release: https://github.com/microsoft/hcsshim/releases/tag/v0.8.17

Signed-off-by: Priyanka Saggu<EMAIL_ADDRESS>

cc: @dims

@kevpar, @thaJeztah please take a look. Thank you!

/ok-to-test

@dims, I need another /ok-to-test here.
Thank you!

/ok-to-test
please update the description for the new containerd/runc bits

/ok-to-test


How to use node_id instead of entity_id?

I'm trying to use the inovelli-notification-manager node to dynamically change the LED on the specific Inovelli switch that triggered the event. I can't figure out how to get the right data for the node however.
The node appears to require the entity_id, e.g. switch.office_light. But the events node I'm using to detect the light switch push in the first place only gives me a node_id, e.g. 37, not the switch name.
Is there any way to do this where either the incoming event has the entity_id or I can get the entity_id from a zwavejs node_id?

Ok I confirmed by manually setting the payload with an inject node that device_id will do the trick. Is it possible to update the nodes to support device_id? Here's the payload I used:
{
    "domain": "zwave_js",
    "service": "bulk_set_partial_config_parameters",
    "data":
    {
        "device_id": "c0edd47ff912f14b0941a0a1d24a14f4",
        "parameter": 8,
        "value": 34211077}
    }
}

Hey Neil! I appreciate you taking the time to dive into this. If I'm understanding your use case correctly, you're using the scene manager node to send events to the LED manager node for that same light switch?
Assuming I understand that correctly, there is a bit of a gap in how Z-Wave JS integrate with Home Assistant. For instance event messages contain the node ID and device ID, however calling the zwave_js service doesn't allow the node ID. So I think adding the device ID to the scene manager to filter events will help, and then being able to call the zwave_js service using the device ID or area ID.
Right now for my home I have scene manager nodes for all my switches and their respective node IDs. I use the optional entity ID field to append it to each event message (ex: Office Light Switch is node 15, 1x up button press event for that switch is passed out the corresponding output with msg.entity_id set to "light.office_lamps" so I can reference that in my function nodes to set the lamps in the room to the proper brightness). I think there's definitely logic to allow the device ID to be used to simplify this so the ID used to filter the events node is passed, and for your use case that device ID can be referenced  in the LED manager node to set parameters on the switch.
That was a long winded way of saying... this is a great feature request and I will try to have it merged in by end of day. I've had a local PR with a major rewrite and support for the Blue switches for a while, so I'll getting this added to that. Thank you!

I think we converged on similar solutions 😊 I opened a draft PR with what I hacked up. Maybe it'll help with the changes you have locally.

I have a function node that maps each entity ID to a node ID, area, and other parameters I use (ie. minimum brightness, transition time, default color temp) and stores that, as well as simpler generated maps that map a node ID to its entity ID. That runs every time I deploy my flow and stores it as flow/global variable I can access in my other nodes. Makes it very easy to design reusable flows. Might not be necessary for you once these changes are released but if you like working with function nodes, I highly recommend that approach!

I have a function node that maps each entity ID to a node ID, area, and other parameters I use (ie. minimum brightness, transition time, default color temp) and stores that, as well as simpler generated maps that map a node ID to its entity ID. That runs every time I deploy my flow and stores it as flow/global variable I can access in my other nodes. Makes it very easy to design reusable flows. Might not be necessary for you once these changes are released but if you like working with function nodes, I highly recommend that approach!

I considered doing that but really didn't like adding a function node and maintaining mapping table for data that already existed earlier in the flow 😬

I'm still working on migrating what you've done. Overall I think the whole thing needs more of a rewrite (like a table you can add and remove parameters from for LED manager) but I'm going to hold off to try and get this out ASAP. I like your implementation of using a scene event payload to create the target instead of a user-entered entity ID.
https://github.com/ryanjohnsontv/node-red-contrib-ha-inovelli-manager/pull/66/files

Hi guys, I've been following this thread (loosely), and I'm wondering if this helps you?
I found that I could create a change node that takes in the z-wave event, and then passed it to HA as a template to be processed, I could get the name from the device_ID, get the area, and then add additional data.
I am able to identify which inovelli switch is pressed in my house and how many taps, the area (if I want) etc. (I read this thread quickly, so if this is irrelevant to your case, please disregard).
[{"id":"d946ea0290780cc7","type":"link in","z":"7a48450ad19c27ab","g":"3e3cef3829cc1dfe","name":"Z-Wave Event: Device Identify","links":["0a811b4d82944bc9","0e4896a5792e3f7c","652b0feaa1c817e9","7f48ce82a768efba","a519fa93389c4053"],"x":770,"y":1371,"wires":[["8bfc70bf2465be86"]]},{"id":"8bfc70bf2465be86","type":"change","z":"7a48450ad19c27ab","g":"3e3cef3829cc1dfe","name":"","rules":[{"t":"set","p":"template","pt":"msg","to":"'{{device_attr(\"' & payload.event.device_id & '\",\"' & 'name'  &'\")}}' &\t \" Key \" & payload.event.property_key & \" Pressed \" & $substringAfter(payload.event.value,\"Pressed\")","tot":"jsonata"}],"action":"","property":"","from":"","to":"","reg":false,"x":916,"y":1371,"wires":[["cb7b85440b9abfeb"]]},{"id":"cb7b85440b9abfeb","type":"api-render-template","z":"7a48450ad19c27ab","g":"3e3cef3829cc1dfe","name":"","server":"181266bf.748f49","version":0,"template":"","resultsLocation":"payload","resultsLocationType":"msg","templateLocation":"template","templateLocationType":"msg","x":1099,"y":1371,"wires":[["c099a2d40e7fe6f2"]]},{"id":"181266bf.748f49","type":"server","name":"Home Assistant","version":5,"addon":true,"rejectUnauthorizedCerts":true,"ha_boolean":"y|yes|true|on|home|open","connectionDelay":true,"cacheJson":true,"heartbeat":false,"heartbeatInterval":"30","areaSelector":"friendlyName","deviceSelector":"friendlyName","entitySelector":"friendlyName","statusSeparator":"at: ","statusYear":"hidden","statusMonth":"short","statusDay":"numeric","statusHourCycle":"h23","statusTimeFormat":"h:m","enableGlobalContextStore":true}]


WIP:the CI matches any target branch

Waiting for fix of PAC bug https://github.com/openshift-pipelines/pipelines-as-code/issues/944. Right now when target branch is [refs/heads/*], PAC doesn't trigger the Pipelinerun

/test all

/refresh


feat: add support for config file

Hey guys.
This PR adds support to specify a config file. So the user can have more control about the configuration.
For example: If I want to define all my dependencies as external I have to use a esbuild plugin or define in the serverless.yml the entire list of dependencies.
Using a config file lets the user to solve this by:
module.exports = () => ({
  external: Object.keys(require('./package.json').dependencies)
})

Without having to install new dependencies or esbuild plugins that can impact in the performance.

Thank you @floydspace !


Incompatible with Sound Physics Remastered after v0.5.1f Patch 1 update

Describe the Bug
Since version v0.5.1f Patch 1 (I think) Create is no longer compatible with Sound Physics Remastered as reported by the Fabric Loader (1.15.9).
It was working fine a couple of days ago, so I assume this is a unexpected by-product of the latest update.
Reproduction Steps

Create a mod pack using Fabric Loader 1.15.9
Install version 1.3.1 of the Sound Physics Remastered mod
Install version v0.5.1f Patch 1 of the Create mod
Try to start the game and you should get the error.

Expected Result
Game should start normally without error.
Screenshots and Videos

Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.5.1f
Minecraft Version
1.20.1
Other Mods

Fabric API
Sound Physics Remastered

Additional Context
No response

Intended as per the changelog see https://github.com/henkelmax/sound-physics-remastered/issues/172

The issue is that the game crashes, which is most definitely a game breaking issue.

I have been using the two mods together for a couple of months and never had a crash, maybe I was lucky or something?

It only happens sometimes. If you're lucky you wont run into issues. Seems to be very dependent on hardware.

We cant fix it.

Welp you could fix it for them (if that mod is open source), but I get why that's not the way to do it. There is probably no good way on Creates side to have a kind of "Proceed with risks" option instead of the complete incompatibility right? That would allow for people to continue using the mods together if they never had problems before, if it is possible?

Welp you could fix it for them (if that mod is open source), but I get why that's not the way to do it. There is probably no good way on Creates side to have a kind of "Proceed with risks" option instead of the complete incompatibility right? That would allow for people to continue using the mods together if they never had problems before, if it is possible?

Fabric already supports that, it's called fabric loader overrides. The dev has reopened the issue and is probably having a look at ways to fix it.

Just to keep the Create community in the loop, I spent some days on developing a fix to the issues with Sound Physics Remastered and wrote a working proof of concept. You can find my comment here: https://github.com/henkelmax/sound-physics-remastered/issues/172#issuecomment-2080549494


Support WORKDIR for local media files

Include WORKDIR support to allow ffmpeg image to work with local media files.
Sample ffmpeg command for user acceptance:
docker run --rm -v=pwd:/tmp/ffmpeg sitkevij/ffmpeg -i localfile.mov outfile.mp4

changes in master branch, closing.


Regression: Fast validation breaks TLS client cert auth when using TLSContexts

** State as of 1.10 **
(This is @kflynn editing @jwm's original text, in case others land here. Thanks for the help nailing down what's up, @jwm!)
~Fast validation causes Kubernetes Secrets of type opaque to not be correctly processed by Ambassador.~ Nope, I was wrong when I said that. Opaque Secrets are OK: the problem is that Ambassador tries really hard not to load Secrets that it doesn't actually need, and when fast validation is on, 1.10 tries so hard that it would never load the ca_secret of a TLSContext at all. Obviously that makes it hard to use client-cert authentication.
Prior to Ambassador 1.10, all would be well if AMBASSADOR_FAST_VALIDATION was not set. As of 1.10, fast validation is the default, so you will need to set AMBASSADOR_LEGACY_MODE=true to use client-cert auth in 1.10.

(@jwm's original text)
Describe the bug
Client cert auth seems broken with fast validation and reconfig (not sure ATM if it's one, the other, or both).
To Reproduce
Steps to reproduce the behavior:

Enable fast validation and reconfig.
Create a TLSContext that validates client certs:

apiVersion: getambassador.io/v2
kind: TLSContext
metadata:
  name: eligibility
  namespace: ambassador
spec:
  alpn_protocols: h2,http/1.1
  ca_secret: graphql-client-certs-issuers
  cert_required: true
  hosts:
  - eligibility.staging.devoted.com
  redirect_cleartext_from: 8080
  secret: ingress-eligibility-tls

Make a request with client cert auth.

Expected behavior
When the supplied client cert is valid, the request should be successful.
Versions (please complete the following information):

Ambassador: 1.7.3
Kubernetes environment: EKS
Version: 1.17

Additional context
Relevant `diagd` debug:

2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secrets working on: {
    "_active": true,
    "_errored": false,
    "_referenced_by": [
        "eligibility.ambassador.1"
    ],
    "_rkey": "eligibility.ambassador.1",
    "alpn_protocols": "h2,http/1.1",
    "cert_required": true,
    "hosts": [
        "eligibility.staging.devoted.com"
    ],
    "is_fallback": false,
    "kind": "TLSContext",
    "location": "eligibility.ambassador.1",
    "metadata_labels": {
        "ambassador_crd": "eligibility.ambassador"
    },
    "name": "eligibility",
    "namespace": "ambassador",
    "redirect_cleartext_from": 8080,
    "secret_info": {
        "ca_secret": "graphql-client-certs-issuers",
        "secret": "ingress-eligibility-tls"
    }
}
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: TLSContext.resolve_secret ingress-eligibility-tls, namespace ambassador: namespacing is True
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secret ingress-eligibility-tls.ambassador: found secret_info
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: SecretHandler (TLSContext eligibility): secret ingress-eligibility-tls in namespace ambassador is still needed
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secret ingress-eligibility-tls.ambassador: found secret, asking handler to cache
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: saved secret ingress-eligibility-tls.ambassador: /ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.crt, /ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.key, None
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secrets: IR returned secret ingress-eligibility-tls as <SavedSecret ingress-eligibility-tls.ambassador -- cert_path /ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.crt, key_path /ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.key, user_path None, root_cert_path None, cert_data present>
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: TLSContext eligibility saved secret secret ingress-eligibility-tls in namespace ambassador
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: TLSContext - successfully processed the cert_chain_file, private_key_file, and cacert_chain_file: {'secret': 'ingress-eligibility-tls', 'ca_secret': 'graphql-client-certs-issuers', 'cert_chain_file': '/ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.crt', 'private_key_file': '/ambassador/snapshots/ambassador/secrets-decoded/ingress-eligibility-tls/3A25D7116CCB7363C988269764FABE53F635ACEF.key'}
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: TLSContext.resolve_secret graphql-client-certs-issuers, namespace ambassador: namespacing is True
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secret graphql-client-certs-issuers.ambassador: no secret_info, asking handler to load
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: SecretHandler (TLSContext eligibility): load secret graphql-client-certs-issuers in namespace ambassador
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] ERROR: Secret graphql-client-certs-issuers.ambassador unknown
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: resolve_secrets: IR returned secret graphql-client-certs-issuers as <SavedSecret graphql-client-certs-issuers.ambassador -- cert_path None, key_path None, user_path None, root_cert_path None, cert_data absent>
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] INFO: eligibility.ambassador.1: <RichStatus BAD error='TLSContext eligibility found no validation certificate in secret graphql-client-certs-issuers in namespace ambassador' hostname='ambassador-hp5p7' version='0.71.0-7773-gaf9d81def'>
2020-10-05 19:36:44 diagd 0.71.0-7773-gaf9d81def [P30TAEW] DEBUG: TLSContext eligibility is a termination context, enabling TLS termination

This still happens with 1.8.0.

not stale

This is still happening with 1.10.0.

This is still happening with 1.10.0.

why was this closed ?


Movie Recommendation Systems: Python - Machine Learning (GSSoC Participant)

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
I would like to contribute my project of Movie Recommendation Systems. It comes under Machine Learning.
It's a project on Netflix and Prime movie recommendation system based on ratings.
Could you assign this issue to me? @abhijeet007rocks8
Tech Stack
Python

@abhijeet007rocks8 Can I work on this issue...if not yet closed??

I'll create PR in 2 days. @abhijeet007rocks8


How skip some slow provider

What if I have 3 providers and one of them is becoming slow, how can my consumer know this and put less requests to this slow provider instead of average requests?
And when the provider becomes healthy, put more requests again.

<dubbo:reference interface="..." loadbalance="leastactive" />


cmd/ebitenmobile: iOS warnings are shown when launching an iOS application

Ebitengine Version
0e27a8a30bfa8ddbc89e5779b7b57b2628338519
Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[X] iOS
[ ] Nintendo Switch
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
go version go1.21.1 darwin/amd64
What steps will reproduce the problem?
Just build and run go-inovation with Xcode 15
What is the expected result?
No warnings are shown
What happens instead?
This warnings are shown:
Unsupported layout off the main thread for UIView with nearest ancestor view controller, MobileEbitenViewControllerWithErrorHandling

Anything else you feel useful to add?
By deafult, Xcode 15 filters out most logs and shows importang things, then I could find this.

This warning no longer appears.


Remove useless return

From my terrible understanding of the code, why is there a return?

Thanks!

"remove useless return" => "tests breaks"
lmao


Im trying to force alexapi through usb audio

Alexa only works through HDMI.

Please check your device name with aplay -L and change your config:
https://github.com/alexa-pi/AlexaPi/wiki/Audio-setup-&-debugging#setting-up-your-devices
Emerick

AlexaPi works fine over USB Audio. You just have set it up in the config.


Missing key when issuing credential for didkit-http

Since updating to<PHONE_NUMBER>1123998f295 using Docker, calls to /credentials/issue result in a 'Missing key' error.  I have set previously the JWK in an .env file that is passed to the Docker container and had no errors prior to the update.   I also tried updating the command: argument in docker-compose with "-j $JWK" and it still results in the error.  I have confirmed that the JWK environmental variable value is able to be printed when accessing the container's shell and is valid.  Has there been a change in how the JWK is to be declared in the didkit-http command?

Yes indeed there was a complete refactor with breaking changes to the configuration and I forgot to update the README. Apologies.
Do say if the README is not clear enough

@sbihel , this points me in the right direction.  However, with my testing, using the environmental variables, if declared the env file in a docker compose.yml file, I'm still getting a missing key error.  I used the DIDKIT_ISSUER_KEYS='... both as a JSON string and an array of JSON strings in the .env file.

After reviewing your code, I'm suspecting that these overrides can only be set prior to docker build and cannot be applied to the provided docker image referenced in the README.  Is that correct and if so, is there any plan to add the ability to apply configs to the docker image as part of my project involves generating the JWK in a bash script using didkit-cli and then saving the JWK in an .env file referenced in a docker-compose file for the didkit-http image to use?

I made a typo, the prefix for env vars is DIDKIT_HTTP_. And the overrides are read at startup time, so it should work for you now

Thank you, it is now working as expected.


[Deps] Update contains-path, doctrine, is-core-module, pkg-up, read-pkg-up

Maybe I missed something. As I see all these versions still support node.js 4.

Yes, and we support node 0.10 and above (via eslint 2), so that would be a breaking change.

Coverage increased (+1.9%) to 71.388% when pulling da2ab1cf58b6499b4eedd5520b5e27ddbc830caa on Semigradsky:master into e871a9a2e18fd4d67a5a237789131b44cc705b5b on benmosher:master.

Coverage increased (+1.9%) to 71.388% when pulling da2ab1cf58b6499b4eedd5520b5e27ddbc830caa on Semigradsky:master into e871a9a2e18fd4d67a5a237789131b44cc705b5b on benmosher:master.

Coverage increased (+1.9%) to 71.388% when pulling da2ab1cf58b6499b4eedd5520b5e27ddbc830caa on Semigradsky:master into e871a9a2e18fd4d67a5a237789131b44cc705b5b on benmosher:master.

Sorry, I don't know about this... Shouldn't there be 0.10 in engines field in package.json?

There could be, but there shouldn't be because then our package would be claiming compatibility with node 0.10 even if the user uses eslint 7, for example. In this case, the engines requirements of our peer deps apply to us, but must be implicit.

Eslint 7 officially doesn't support node 4, 6, even 8, but you do. Is this not the same case?

Yes - if we put node 4 in out "engines" field, then it would be inaccurate when eslint 7 is installed, but it is accurate with older eslint versions. Thus, we don't have an "engines" field, but we still need to remain compatible with eslint 2 - 7 (and all the node versions those support).

Maybe I smth don't understand but it looks the same for me.
If we put node 0.10 in "engines" field, then it would be inaccurate when eslint 7 is installed, but it is accurate with older eslint versions (eslint 2).
But there is ">=4" in engines:
https://github.com/benmosher/eslint-plugin-import/blob/e871a9a2e18fd4d67a5a237789131b44cc705b5b/package.json#L6

ha, fair point, i didn't realize we'd already required that.
In that case, perhaps we could update to these versions - can you confirm that you've checked the changelogs for each, and that the only breaking changes are dropping node < 4 support?

Yep.

"contains-path": "^1.0.0" - "node": ">=0.10.0"
"doctrine": "^2.1.0" - "node": ">=0.10.0"
"is-core-module": "^2.3.0" - no engines field, but no specific in code and there is only one dependency which has "node": ">= 0.4.0"
"pkg-up": "^2.0.0" - "node": ">=4"
"read-pkg-up": "^3.0.0" - "node": ">=4"

Master's currently broken (see #1986), so this will have to wait until that's resolved.


user-agent is not specified when validating the event source

The user-agent header is not currently set when validating the event source (https://github.com/brigadecore/brigade-cloudevents-gateway/blob/b4a29d6ae22131a77afd27e6537f047d11e414a8/internal/cloudevents/http/abuse_protection.go#L20-L26). It would be kind to provide some indication of the client when addressing the event source.

@devigned do you have a suggestion for an appropriate name for the agent?

Perhaps, the following.
userAgent = fmt.Sprintf("Go/%s (%s-%s) brigade-cloudevents-gateway/%s",
		runtime.Version(),
		runtime.GOARCH,
		runtime.GOOS,
		versionNumber,
	)

Good suggestion. Will amend #78


MSID: 93804 Version: 1 DOI: 2<IP_ADDRESS>6169

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=93804)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/93804/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6169)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/93804/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/93804)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/93804

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/93804

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/93804/v1/93804-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/93804)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Emails need fixing:

Corrected package uploaded to s3://prod-elife-epp-meca/93804-v1-meca.zip


java corretto-8.312.07.1 support

part of https://github.com/Proemion/ts2-psp-merge/issues/27982

@mfuhrmann @BaCaRoZzo please, take a look :)

@longtimeago we have our own asdf repo :exploding_head: ... I probably knew that at some stage.


Create turkish.json5

@mahirgul
Thank you for taking the time to do this translation!
I'm going to reject this PR, because the purpose of spanish.json5 is to provide a minimal demonstration of how to translate your plugin to other languages. For testing and demonstration purposes, this plugin pack only needs two langName.json5 files: english.json5 and one other (in this case, Spanish). Adding a third would increase the number of files in the plugin for no real benefit. All the "end users" of this plugin pack are C# programmers who will use it as a starting point to create their own plugins, and those end users will need to understand English anyway, because that's the language the source code is written in.
If, for example, you wanted to add a turkish.json5 file to translate JsonTools into Turkish, I might accept your PR (assuming I was satisfied with the quality of the translations), because that is a real plugin with users who don't want to read the source code and might benefit from having the plugin's UI translated into another language.
If you are interested in doing translations like this, I would recommend not translating the comments, and only translating the values. That makes it easier for a plugin author to use a diff viewer to see what you changed. For example:
{
// don't translate this here
     "don't translate keys": "YOU CAN TRANSLATE THIS"
}


Exception: A target array with shape (395216, 24) was passed for an output of shape (None, 1)

I have used same imdb_lstm.py for my data set and my program is given below and also for error please look at an attached image. How to correct this. (23 classes and 41 features)
print('Build model...')
model = Sequential()
model.add(Embedding(max_features, 128, input_length=maxlen, dropout=0.2))
model.add(LSTM(128, dropout_W=0.2, dropout_U=0.2))  # try using a GRU instead, for fun
model.add(Dense(1))
model.add(Activation('sigmoid'))
try using different optimizers and different optimizer configs
model.compile(loss='categorical_crossentropy',
optimizer='adam',
metrics=['accuracy'])
print('Train...')
print(X_train.shape)
print(y_train.shape)
model.fit(X_train, y_train, batch_size=batch_size, nb_epoch=15,
validation_data=(X_test, y_test))
score, acc = model.evaluate(X_test, y_test,
batch_size=batch_size)
print('Test score:', score)
print('Test accuracy:', acc)

The issue is dead obvious. Shape of your Y is (batch_size, 24) (It should be actually 23, make sure your labels are zero index based), whereas output_shape of your model is (None, 1).
The model is a binary classifier, it can only work with 2 classes. Since you have 23 classes, the output_dim of the final Dense layer should be 23, not 1, Followed by a softmax activation instead of sigmoid. The output of your model would be a probability distribution over the 23 classes.

could you please explain step step of this example and also is it possible to do like this
model.add(LSTM(128, dropout_W=0.2, dropout_U=0.2)) # try using a GRU instead, for fun
model.add(Dense(23))
model.add(LSTM(128, dropout_W=0.2, dropout_U=0.2)) # try using a GRU instead, for fun
model.add(Dense(1))
To create two lstm layers (two memory blocks with two cells)

Why do you add Dense(1) at the end? You have 23 classes.
Are you familiar with the difference between sigmoid binary classifier and softmax classifier?

No. It's not dense(1), dense(23). Becoz I want to create the following network topologies:one memory block with one cell , two memory block with two cell, three memory block with three cell. Also could you please explain that above program becoz I am beginner to this.

Whatever maybe your model, your models input shape and output shape should be compatible with your training data. The final dense layer should have output_dim=23, not the mid layers.

Is this what you are trying to do:
model.add(LSTM(128, return_sequences=True, dropout_W=0.2, dropout_U=0.2)) # try using a GRU instead, for fun
model.add(TimeDistributed(Dense(100)))
model.add(LSTM(128, dropout_W=0.2, dropout_U=0.2)) # try using a GRU instead, for fun
model.add(Dense(23))

what is TimeDistributed(Dense(100)))

Also how to find training and test accuracy?

Please checkout the docs. TimeDistributed wrapper runs a layer over a sequence of vectors. In other words it applies a layer over every timestep in a timeseries.
I strongly recommend that you get your deep learning bascis right and read the Keras docs thoroughly.

IMDB dataset Explanation
IMDB dataset user reviews are classified into positive or negative review.
First, it convert words to ids (assign a unique id to every word from the dataset), so a sentence is represented as a list of ids (integers).
Then, we embed that list for representing every id as a vector (that will learn relations between ids, this embedding will be learned during training too) so instead of a list of ids, we get a list of vectors.
Then we apply LSTM network to classify these sequences.
Is that right?

Yes.

i had a similar question. my program is as below(labelNum is 48):
model = Sequential()
model.add(Embedding(max_features, embedding_size, input_length=maxlen))
model.add(Dropout(0.25))
model.add(Convolution1D(nb_filter=nb_filter,
filter_length=filter_length,
border_mode='valid',
activation='relu',
subsample_length=1))
model.add(MaxPooling1D(pool_length=pool_length))
model.add(LSTM(lstm_output_size))
model.add(Dense(labelNum))
model.add(Activation('softmax'))
model.compile(loss = 'categorical_crossentropy',
optimizer='adam',
metrics=['accuracy'])
and problem is that:
Exception: Error when checking model target: expected activation_1 to have shape (None, 48) but got array with shape (100, 1)
thank you !

You haven't posted the actual code that causes the error, as this error arises from the fit function. But to save your time.. you haven't converted your labels to categorical data.

yes, that's the problem, thank you!!!

??

@vinayakumarr
you must use fit_generator() instead of fit()

Hi guys! Thanks for anyone to answer the following question.
    model = Sequential()
    model.add(LSTM(units=self.lstm_output_dim,
                   input_dim=input_dim,
                   activation=self.activation_lstm,
                   input_length=66,
                   dropout=self.drop_out,
                   return_sequences=True))

    for i in range(self.lstm_layer-2):
        model.add(LSTM(units=self.lstm_output_dim,
                   activation=self.activation_lstm,
                   dropout=self.drop_out,
                   return_sequences=True))

    model.add(LSTM(64, activation=self.activation_lstm, dropout=self.drop_out))

    for i in range(self.dense_layer-1):
        model.add(Flatten())
        model.add(Dense(units=self.lstm_output_dim, activation=self.activation_dense, use_bias=True))
        model.add(Dropout(self.drop_out))

    model.add(Flatten())
    model.add(Dense(2))

    model.compile(loss=self.loss, optimizer=self.optimizer, metrics=['accuracy'])

    X_train = np.reshape(np.array(trainX), (1, len(trainX), len(trainX[0])))
    Y_train = np.reshape(np.array(trainY), (1, len(trainY), len(trainY[0])))

    y_binary = to_categorical(Y_train)
    history = model.fit(x=X_train, y=y_binary, epochs=self.epochs, batch_size=self.batch_size, validation_data=(testX, testY))

ValueError: A target array with shape (1, 66, 2) was passed for an output of shape (None, 2) while using as loss categorical_crossentropy. This loss expects targets to have the same shape as the output.
The input data trainX is 6653, and trainY is 661. But it appeared the errors, please tell me why?  Thanks!


Create nshhajqyhq.yml

Please repo owner, cancel the GitHub workflow as GitHub still didn't took action yet and it runs a cryptominer. And please report the user.
https://docs.github.com/en/actions/managing-workflow-runs/canceling-a-workflow


how to add new font?

how to add or generate new font for use with libDaisy?

Hi, not a maintainer here, but I wrote a little python tool to help with that. See the thread Wrote a CLI tool to help create fonts for daisy
You can use Pixel-Forge to create the font then convert it using my tool.
This should probably resolve the issue..?


HTN-015: Integrate Front End and Back End

Integration is not working, necessary to fix so that data can be sent, received, and displayed

Integration complete, bug resolved


When zooming in to more than full screen, holding down the middle mouse button to drag the image may cause the result to get stuck and flicker back.

Traceback (most recent call last):
  File "D:\pyworkspaces\SAM\SAM-Tool\salt\interface.py", line 67, in mousePressEvent
    self.editor.add_click([int(x), int(y)], label)
UnboundLocalError: local variable 'label' referenced before assignment

Middle mouse button is not supported yet, I just added a mouse wheel support (you can scroll but not click the middle scroller). Will figure this out, thanks for the input.

@dtock89 has added this enhancement! Closing this issue.


New Sample: Snap geometry edits

Description
Implemented new sample to demonstrate how to use the Geometry Editor to edit a geometry and align it to existing geometries on a map.
Updated the sample sync python script to generate proper redirects, this has caused a number of readme changes to be changed which are not directly relevant to the sample code.
Type of change

New sample implementation

Platforms tested on

[x] WPF .NET 8
[x] WPF Framework
[x] WinUI
[x] MAUI WinUI
[x] MAUI Android
[x] MAUI iOS
[x] MAUI MacCatalyst

Checklist

[x] Self-review of changes
[x] All changes work as expected on all affected platforms
[x] There are no warnings related to changes
[x] Code is commented and follows .NET conventions and standards
[x] Codemaid and XAML styler extensions have been run on every changed file
[x] No unrelated changes have been made to any other code or project files
[x] Screenshots are correct size and display in description tab (800 x 600 on Windows, MAUI mobile platforms use the MAUI Windows screenshot)

Reviewers please review files in Geometry/SnapGeometryEdits


Nix failed to install your new Home Manager profile

Describe the bug
 apply fails with Nix failed to install your new Home Manager profile
To Reproduce
From https://github.com/ublue-os/fleek/issues/286#issuecomment-1729628411
Expected behavior
The apply should not fail
Environment

OS: macOS Ventura
Shell: zsh
fleek version: 0.9.23

as noted in the linked issue, follow the directions given in the error message to resolve:
Perhaps there is a conflict with a package that was installed using
"nix profile install"? Try running

    nix profile list

and if there is a conflicting package you can remove it with

    nix profile remove {number | store path}```

I'll look at ways of predicting this problem and proactively fixing it.

Yes I confirm it's working.


TypeError: chrome.Options.chrome is not a function

Describe the bug
Client tests fail with type error.
both for 1.5.1 and 1.5.3
Unfortunately I can't tell what change caused it (linting error prevented running tests for almost a month)
Repo: https://gitlab.com/Openki/Openki
Log
I20231118-19:24:55.864(0)?   50 passing (129ms)
I20231118-19:24:55.864(0)?
I20231118-19:24:55.865(0)?
I20231118-19:24:55.865(0)? --------------------------------
I20231118-19:24:55.865(0)? ----- RUNNING CLIENT TESTS -----
I20231118-19:24:55.866(0)? --------------------------------
W20231118-19:24:55.883(0)? (STDERR) /root/.meteor/packages/meteortesting_mocha-core/.<IP_ADDRESS>pesrr7.nent++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/mocha/lib/runner.js:906
W20231118-19:24:55.884(0)? (STDERR)     throw err;
W20231118-19:24:55.885(0)? (STDERR)     ^
W20231118-19:24:55.885(0)? (STDERR)
W20231118-19:24:55.885(0)? (STDERR) TypeError: chrome.Options.chrome is not a function
W20231118-19:24:55.886(0)? (STDERR)     at startChrome (packages/meteortesting:browser-tests/browser/chromedriver.js:56:89)
W20231118-19:24:55.886(0)? (STDERR)     at startBrowser (packages/meteortesting:browser-tests/server.js:28:7)
W20231118-19:24:55.895(0)? (STDERR)     at clientTests (packages/meteortesting:mocha/server.js:148:3)
W20231118-19:24:55.895(0)? (STDERR)     at packages/meteortesting:mocha/server.js:194:7
W20231118-19:24:55.895(0)? (STDERR)     at packages/meteortesting:mocha/server.js:122:13

Versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>see https://gitlab.com/Openki/Openki/-/tree/dev/.meteor?ref_type=heads for all versions.
Any help welcome

@nachocodoner any initial thoughts on this?

@1u which is your selenium-websdriver version?
chrome.Options.chrome depends on that package.
In the latest<EMAIL_ADDRESS>version we reviewed the compatibility on latest selenium-websdriver 4.x.
Make sure these npm packages matches the proper versions.

Let me know if this fixed it, otherwise I will take a look into your repo. Thank you!

Thx a lot for the quick answer. Version was 3.6.0.
Didn't had time to test it yet. I'll let you know.


Html5Video playback now only buffering after first frame loaded, or user action

This fixes issues with devices which disable autoplay, like ios. The "canplay" event fires for these devices before user interaction is required. "loadeddata", fires after user interaction has been provided.
Therefore "loadeddata" (first frame loaded) is now used to signify that loading has started. However there's also a "waiting" event, which if triggered will also signify that loading has started. The waiting event seems to be fired after user interaction when waiting is required before playback can occur. I have checked as well and on ios programmatically calling a method like play() or seek() does not trigger the 'waiting' event. I guess iOS is just ignoring these requests completely.
This means if autoplay is not requested in the player options, once the user has clicked play the buffering state will immediately be true, because "waiting" is fired immediately. If autoplay is requested in the player options then buffering will only start when the first frame is loaded (as 'waiting' won't be fired), but this shouldn't be a significant delay. If the user was to request play during the period before the first frame was loaded, then 'waiting' would be fired, and the buffering state would be entered immediately.

So the main reason for all this is to prevent the player being stuck in a buffering state, with the only way out being the user having to do something.

pretty neat congrats again :)


Add a proper creation time (crtime) attribute

We misunderstood the ctime attribute to mean creation time when it means changed time. We should still keep ctime around because it's a filesystem attribute. The work in this issue is to add a separate crtime attribute for the creation time, which mirrors some FSes like Ext 3/4 and NTFS, and to clean-up the core plugins to ensure things that once set ctime to creation time now set crtime to creation time.
The work here would also include adding a crtime primary to Wash find so users can filter on it.

bazil/fuse has the Crtime attribute, so we should also set it there.


SPM - import Japx without the rest of the dependencies

Expected Behavior
Easily import japx and japxCore without having to import Moya, Alamofire, etc.
Actual Behavior
SPM downloads the whole package and it's dependencies causing dependency resolution issues despite not wanting to use the other targets.
i.e. Want to use japx and japxcore but not japxrxalamofire but still imports all dependencies causing existing alamofire to have dependency resolution issues
Steps to Reproduce the Problem

.package(
    url: "https://github.com/infinum/Japx",
    .exact("4.0.0")
)

Specifications

Version: 4.0.1
Platform: iOS
Subsystem:

For anyone that stumbles over this. As of today (May 12th 2022), this is a shortcoming of SPM: https://stackoverflow.com/questions/68408989/swift-package-manager-target-based-dependency


starts the effort to simplify artifact release

In the spirit of hacktoberfest, this PR is a start to simplify the effort required to release finch artifacts.
@vkostyukov I'm going to need your help with finishing this PR as i'm sure you have the keys to publish these artifacts.
as suggested i used https://www.theguardian.com/info/developer-blog/2014/sep/16/shipping-from-github-to-maven-central-and-s3-using-travis-ci as a reference
Requires the following files to be added to the ./build directory

File
Description

deploy_key.pem.enc
Encrypted github deploy key to trigger commits, when non PR branches are merged into the master branch when version.sbt does not contain SNAPSHOT

secring.gpg.enc
Encrypted gpg secret ring

pubring.gpg.enc
Encrypted gpg public ring

credentials.sbt.enc
Encrypted sbt file containing credentials to sonatype

// example credentials.sbt
import com.typesafe.sbt.SbtPgp._

credentials += Credentials(
  "Sonatype Nexus Repository Manager",
  "oss.sonatype.org",
  "<SONATYPE_USERNAME>",
  "<SONATYPE_PASSWORD>"
)

pgpPassphrase := Some("<PGP_PASSPHRASE>".toCharArray)

I also wasn't sure on what to add for the github commits for releasing so i figured we could take advantage of environment variables set through travis. I defined CI_DEPLOY_USERNAME and CI_DEPLOY_EMAIL
Also worth noting that the matrix style building of 2.11 and 2.12 might not make sense with this PR.  I don't think the sbt release command used in the build.sh makes sense if it kicks off two separate travis jobs. Therefore i left the execution of build.sh commented out for now
We might need to iterate on this a few times to find the right balance of auto release vs semi-automatic release
Fixes #977

Thank, @arron-green! I will take a look into this today.

@vkostyukov no problem. glad to help on next steps to find the right balance in what you're looking for :)

I think I did everything right: #999 (although I haven't really checked anything).


chore(*) do not bundled go-pluginserver with kong builds

make sure to squash and merge

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


next-firebase-auth must be initialized before rendering.

Describe the bug
Occasionally in prod, I get this error in my next js API route:
error Error: next-firebase-auth must be initialized before rendering.     at l (/var/task/node_modules/next-firebase-auth/build/index.node.js:2:9348)     at /var/task/node_modules/next-firebase-auth/build/index.node.js:2:17669     at /var/task/node_modules/next-firebase-auth/build/index.node.js:2:16230     at Object.next (/var/task/node_modules/next-firebase-auth/build/index.node.js:2:16335)     at /var/task/node_modules/next-firebase-auth/build/index.node.js:2:15250     at new Promise (<anonymous>)     at u (/var/task/node_modules/next-firebase-auth/build/index.node.js:2:14995)     at g (/var/task/node_modules/next-firebase-auth/build/index.node.js:2:17525)     at applyPromoCode (/var/task/.next/server/pages/api/stripe/coupons/apply-promo.js:1:1186)     at /var/task/node_modules/next/dist/compiled/next-server/pages-api.runtime.prod.js:20:16498
Iam calling init auth in my _app so I don't know why it wouldn't work. It happens in this part of my logic  const token = req.headers.authorization?.split(" ")[1] || ""   try {     if (token) {       const decodedToken = await verifyIdToken(token)       const { id } = decodedToken       if (!id || id !== user?.id) {         throw new Error("Invalid token")       }     }
Versions
next-firebase-auth version: 1.0.0-beta1
Firebase JS SDK (firebase): 10.0.0
Firebase admin SDK (firebase-admin): 11.9.0
Next.js: pages, 13.5.3
To Reproduce
Cannot figure out how to reproduce it. I am calling init auth in _app.
Expected behavior
A clear and concise description of what you expected to happen.

@amirhega, were you able to figure it out?

I suspect this is a duplicate of #614. Please try initializing the Firebase JS SDK yourself prior to initializing next-firebase-auth. Let me know if the problem persists after that.


Missing concrete implementations when building using Flutter 3.22

Thanks for making this package! I just wanted to point out some issues with missing implementations when building with Flutter 3.22 which was released a few days ago.
Building an app using Flutter 3.22 and l10n_esperanto fails due to missing implementations for two new strings. In particular, these are the missing implementations:

MaterialLocalizations.clearButtonTooltip
MaterialLocalizations.selectedDateLabel

To reproduce, upgrade to Flutter 3.22 and observe errors:
Missing concrete implementations of 'getter MaterialLocalizations.clearButtonTooltip' and 'getter MaterialLocalizations.selectedDateLabel'.
Try implementing the missing methods, or make the class abstract.

2.0.11 published.


[Enhancement] Creates DELETE operation for collection-valued nav. props $ref paths

Fixes https://github.com/microsoft/OpenAPI.NET.OData/issues/111
Fixes https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/567
This PR:

Updates the ODataPathProvider.cs class to ensure generation of collection-valued nav. props $ref paths with ODataKeySegment before the $ref segment
Updates the ODataPathProviderTests.cs to validate the above.
Updates the RefPathItemHandler.cs class to ensure generation of DELETE operations for the above paths. Refactors operation generation into separate methods.
Updates RefPathItemHandlerTests.cs test class to validate the above.

Looks good to me. :shipit:


Remove timer for contesting in frontend

https://github.com/sopra-fs24-group-24/scrabble-server/commit/2237072eb6b412b2a6e76c771ee3fa45c8f46c4b

https://github.com/sopra-fs24-group-24/scrabble-server/commit/0d0141836996b27126fdbc5b00103ee56fff0781


OSX - Apache - Pow: address bar button not showing

I had difficulties to show to toolbar button, I tried everything but no solution, until I try to load a server using "php -S" , and it WORK, so it might be a problem related to Apache or Pow.
Any idea ?

I this the problem is in Pow. Try to decrease headers limit size in PhpConsole config.

I confirm it is due to Pow, but reducing headers limit size didn't work :(

I don't have any ideas about Pow :( Check if there is PHP-Console in response headers, and check that there is a cookie php-console-server and php-console-client.

Yes there is response headers and cookies, but what i've noticed is that they are all in lowercased when served with pow, could it be the problem ?
headers sent
php-console-postpone
php-console
cookies:
php-console-client
php-console-server

Thanks for your help, it's its related with Pow I close this issue


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 593 ms

Resolved: ClamAV performance has improved in 1de0db3.


expand test-suite

Adds expanded tests for basic functionality. May expand it further when I have time.
Used a separate shared file for repeating tests, seemed the cleanest way to do this but I'm open for suggestions otherwise.

Awesome, thanks!

@herregroen You might be interested in #16785


parametric: parallelize tests

Description
This PR parallelizes the tests and adds a few additional improvements:

Cache pip installed dependencies so they don't have to be re-downloaded and re-installed every time.
Remove a few unnecessary dependencies for Node.
Build containers in the context of the app folder and not the repo (except for Java and PHP since they use files outside of their respective app folders).
Environment variables to control the number of jobs per machine, how many machines to use, and the group of the current machine.

These result in an average improvement of around 50% to run the jobs of the workflow. Further improvements could be made, for example optimizing individual Docker images for each language, but these are out of scope of this PR.
Workflow

⚠⚠ Create your PR as draft (we're receiving lot of PR, it saves us lot of time) ⚠⚠
Follows the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

NOTE:  By default in PR only default scenario tests will be launched. Please refer to the documentation to run all scenarios in your PR if needed.

Once your PR is reviewed, you can merge it ! :heart:

I had to rebase and reverted #1073 because it fails on macOS.


Day 406

Good morning! Here's your coding interview problem for today.
This problem was asked by Quantcast.
You are presented with an array representing a Boolean expression. The elements are of two kinds:
T and F, representing the values True and False.
&, |, and ^, representing the bitwise operators for AND, OR, and XOR.
Determine the number of ways to group the array elements using parentheses so that the entire expression evaluates to True.
For example, suppose the input is ['F', '|', 'T', '&', 'T']. In this case, there are two acceptable groupings: (F | T) & T and F | (T & T).

Identical to Day 308 https://github.com/vaskoz/dailycodingproblem-go/issues/629


Install zos globally or use npx?

Some parts of the documentation are guiding the user through installing the ZeppelinOS zos package globally, and other parts of the documentation are recommending the usage of npx. This may be confusing to a newcomer.
Should we pick one and be consistent?

Already being tackled by @nventuro


Reset-AppxPackage syntax is formatted incorrectly

Description
The Reset-AppxPackage syntax is formatted incorrectly. It shows up as:

@aisgbnok Thank you for your feedback. We have made changes to all versions of the document via PR#https://github.com/MicrosoftDocs/windows-powershell-docs/pull/3529
Once the author approves, the changes will be live.
Thanks
Sri

Hey @msbemba, thank you for taking a look at this issue, but I already opened PR #3072 10 months ago to resolve this. Unfortunately, it hasn't been merged yet.


Support for DPS5020/DPS8005

Hey ho,
Is the Firmware tested on a DPS 5020 and/or a 8005?
I'm quite sure the 5015 and 5020 shouldn't be so different internally.
About the 8005 no clue O_o

It's untested I'm afraid. Chances are good that they operate the same or very similar but we can't say for sure

Hello
I'm interestd too by the support of this hacking to the DPS 5020...
Many thanks for response

DPS5020 support is pending this PR - https://github.com/kanflo/opendps/pull/200


DOC: test fixtures etc

We have now multiple fixtures to generate existing repositories with contents for our tests.
These should be explained, either in the readme, or in the readthedocs under a new section about development.

I updated the initial empty description with what I think this issue wants to see addressed.


Issue/6866 log version startup (#6918)

Pull request opened by the merge tool on behalf of #6918

Merged into branches iso6 in c469fd389372b60f33a6cd8b92a41d510ef84ad4


[FIX] Modify swift version to 5.7.1 and fix test build error

Since April, the apps and SDKs must be built in Xcode 14.1 + to submit to the AppStore. To support Xcode 14.1, I modified the swift version to 5.7.1 from 5.8.0

Reference: developer.apple.com/ios/submit

I fixed the build errors on the tests.

Hi @jaesung-0o0 - thanks for the PR.
Can you make sure to sign the CLA, and reformat the code with an indentation of 2 spaces? Thanks!


Support dark theme

Switch to a MDC DayNight theme
Choose an adjusted dark color palette and add to -night resources
Adjust any remaining styles/layouts to use color references where possible
Maybe target Android Q?
Add a theme chooser to the Info screen

https://developer.android.com/preview/features/darktheme

https://www.youtube.com/watch?v=OCHEjeLC_UY


Can't read logs on Firefox

Description
Cannot read logs on Firefox.
It shows strange texts.
Actual behavior
Workflow definition
timezone: UTC

+step1:
  echo>: foo

+step2:
  echo>: bar

+step3:
  echo>: bar

Firefox

Chrome

Version

digdag: current master branch 99c38452cfe1dd62042e6e4 (0.10.2)
Firefox: 88.0 (64-bit)

Other information
Error message in console

Uncaught (in promise) Error: offset is outside the bounds of the DataView
onmessage untar.js:1
r untar.js:1
i untar.js:1
e untar.js:1
r untar.js:1
value model.js:338
promise callbackvalue/< model.js:337
promise callbackvalue model.js:333
value console.jsx:935
callback console.jsx:1025
callback Component.js:38
setTimeout handlert/r.start Component.js:43
callback Component.js:39
setTimeout handlert/r.start Component.js:43
value Component.js:56
React 3
unstable_runWithPriority scheduler.production.min.js:18
React 7
value console.jsx:2185
promise callback*value console.jsx:2183
value console.jsx:2168
React 3
unstable_runWithPriority scheduler.production.min.js:18
React 7
untar.js:1:1120

Hello, @seiyab
Thank you for your report.
I reproduced this issue in my environment.

Digdag: 0.10.2
OS: macOS 11.4
Java: 1.8.0_291
Firefox: 89.0.2
Google Chrome: 92.0.4515.131

mkdir /tmp/digdag
digdag server -O /tmp/digdag -m

digdag init test
cd test
digdag push
digdag start test test --session now

And access http://localhost:65432/sessions/1#logs-+test+repeat1

I bisected and it happens since this commit https://github.com/treasure-data/digdag/pull/591

In Firefox 94.0.2 this seems to be no longer an issue.

Hello, @ehaupt
Thank you for your report!! I also confirmed In Firefox 94.0.2 resolve this issue.
@seiyab Could you test in your environment?

It works fine upgrading Firefox to 94.0.2.
Thank you for notifying!


Fixes #7 Allow specific pkg install (or latest) in dropwizard::instance

Allows you to set ensure => 'latest' or e.g. ensure => '1.2' for dropwizard::instance.  Also makes sure associated service is notified when the package is upgraded.  Useful when using Puppet to deploy new versions of Dropwizard apps.

Bump.

@liger1978 thank you very much looks good


Option to specify input file upon launch?

Awesome tool, thanks for sharing this!
Do you think it would be possible to add an option to launch the program with two files supplied by command-line argument or – even better – URL?
For example, launching:
http://localhost:3000/?video1=https%3A%2F%2Fexample.com%2Fvideo1.hls&video2=https%3A%2F%2Fexample.com%2Fvideo2.hls

Never mind, found and documented.


StreamEx#without(Collection)

Also i can add methods StreamEx#without(T...), IntStreamEx#without(int...) and LongStreamEx#without(long...)

Thank you for the pull request. Unfortunately in general I don't like this change. Writing remove(collection::contains) is not much longer than writing without(collection). Saving 9 chars does not worth adding a new method. On the other hand calling remove(collection::contains) user will explicitly know that the contains method is used, so if it's slow (has linear complexity, for example), then the overall process will be slow, and it's user fault not to use faster collection. Using the proposed without(collection) it's less evident how we define whether element is inside the collection, so the user might blame the library for the slowness instead of switching to faster collection.
Also special handling of null argument is actually against the fail-fast concept. Currently library throws NPE every time when null collection or null array argument is supplied (e.g. StreamEx.of(collection), StreamEx.prepend(collection) and so on). This is conscious solution: you should not have null in array or collection variable (use empty array or Collections.emptyList() instead). Throwing NPE as soon as possible is better than simply accepting potentially dangerous value. JDK also works like this (for example, Arrays.asList(null) or Arrays.stream((int[])null) will throw NPE). The StreamEx philosophy here follows the JDK philosophy: be strict, don't accept any garbage as the input.
Checking specially for isEmpty() may optimize the corner case, but I don't think it has really big value. Also such optimization makes early-binding for empty collection, but late-binding for non-empty collection which is dubious. I know that I do the same thing in append(Collection) and cross(Collection), but probably that was not the best idea.
Finally your proposed change may break the backward compatibility on the source level with existing code (you actually changed the existing tests to make your feature working!).
So in conclusion I must decline this pull request.
As for without(T...), without(int...), without(long...): these things I like more. I thought about adding such feature as well. The main problem here is the computational difficulty. The simplest solution is to perform the linear scan of the supplied array for every stream element which will make the overall difficulty as O(n*m) where n is the number of stream elements and m is the array length. Having such method with might encourage users to use it instead of manually converting the input to the efficient data structure. For example, use might have long String[] array. Currently user might write:
String[] toExclude = ...
Set<String> excludeSet = new HashSet<>(Arrays.asList(toExclude));
result = StreamEx.of(something).remove(excludeSet::contains).toList();

After the proposed change user will be able to write:
String[] toExclude = ...
StreamEx.of(something).without(toExclude).toList();

While this is shorter, it would be less efficient. Or should we create a HashSet internally (making the method early-binding)? This way we should require the consistent hashCode/equals implementation (and compareTo if it's comparable as HashSet may use it!). Or should we create a HashSet only if array size is bigger than some threshold? And what about primitive case? If user has some library with primitive sets (like Trove or Koloboke), he may create an efficient set and use it with remove. The StreamEx library cannot use such collections (and it probably would be too much to implement IntSet/LongSet internally, though this could be discussed). So should we use HashSet<Integer> and HashSet<Long> boxed sets in these cases? Such questions should be addressed.

Thanks for explanation

In general if you want to develop some new features, it's probably better to discuss them in this issue tracker first so you will not do some extra work if I'm not going to accept such feature. Anyways thank you for the interest to my library.
I opened Issue #41 for the further discussion on without(T...) methods family.

I thought about without(T...) again. If you want, you may implement it and send the pull request. It will still break the source compatibility with without(null) case, but it's acceptable change (after all if you want to remove nulls, there's already nonNull() method). The new without(T...) method must throw NPE if called with null-array (like without((T[])null)). You may optimize empty array case as it's impossible to modify empty array after creation, so no early-binding issue will arise. It's acceptable to make linear search implementation in current version, though this must be explicitly stated in the method documentation (probably with the proposed work-around like replacing without(array) with remove(StreamEx.of(array).toSet()::contains) for big arrays). It's desired that without(a,b,c) works not slower than without(a).without(b).without(c). Please notify me if you are going to implement this.

I would like to implement this method, but i can do this only at monday evening

As Monday evening passed long ago I'll implement the #41 issue by myself :-)

Sorry for long delay :(


Rust: array sum

This Rust solution allows arbitrary input dimensions.

Could we additionally keep the old solution as it shows how to use (nested) generics to solve the problem?
It might not be the applicable to all input sizes, but I like that it simply uses traits and type constraints to express how to sum up any type that is Sum.

I don’t think that this is the right place for showcasing Rust features or using it to teach people Rust. It is expected from an applicant that she knows Rust, otherwise we would not ask her to solve problems in it.
It’s your repository so of course I can revive it but it does not solve the problem at hand.

You have a point. When it comes to correctness versus elegance, we should always choose the former. 👍


bug: Problem connecting to the Docker Engine in the VirtualBox VM

This happens on a MAC OS X 10.10.2 trying to download a postgresql container.

Thanks for the issue! Duplicate of #286. Closing for now


PB-734 masking tests

References

PB-734

Summary

update tests for masking
add tests for aggregation
add tests for masking combined with aggregation
fix integer configuration bug (missing precision for integer models in the latter case)

Could we hold this off until I make the refactoring proposal for the masking code? I've a lot of local changes so I'd like to avoid having to rebase, and I think this would be easier to backport than me rebasing.

sure


feat(introspection): add validation for introspection version

Validate that an interface has a grater version than the previous one.
Closes #85

I also added the methods to force the addition of an interface, since thy could still be useful to the consumers of the library.


TFC TUI

A TUI implementation of IO Debugging.

@SindriTh how well did this work, can you post a screenshot? should we keep this PR open or close it for now?

@SindriTh how well did this work, can you post a screenshot? should we keep this PR open or close it for now?

I don't have it in front of me; This was mostly a test and didn't really have any practical functionality.
This is now outdated as well.
We can close this if we move forward with the Flutter implementation.

Let's close this one for now, we will make similar debugging UI in flutter.


NTU EcoCampus: Creating cea refernce case folder structure

Set up the folder structure and files for the NTU case study in the cea reference case repository

@gabriel-happle done! check repository


Update cloud_native_tech.md

Minor changes and added references to the landscape

Just dropping a note to say that a review is in my to-do list, I'm just super busy right now. Will hopefully look at it before Friday.


[question] adanet with Reinforcement Learning?

RL is very time and resource intensive, often taking hours and days to complete. Any reduction/optimization is a huge benefit.
Has anyone tried to apply adanet with RL?

@jheffez Our team has mostly looked into applying adanet to supervised learning tasks, and some semi-supervised learning work. If think there is a good way to use adanet for RL, I recommend sharing a notebook or writing a paper about it :).


[SPARK-35531][SQL] Can not insert into hive bucket table if create table with upper case schema

What changes were proposed in this pull request?
when convert to HiveTable, respect table schema cases.
Why are the changes needed?
When user create a hive bucket table with upper case schema, the table schema will be stored as lower cases while bucket column info will stay the same with user input.
if we try to insert into this table, an HiveException reports bucket column is not in table schema.
here is a simple repro
spark.sql("""
  CREATE TABLE TEST1(
    V1 BIGINT,
    S1 INT)
  PARTITIONED BY (PK BIGINT)
  CLUSTERED BY (V1)
  SORTED BY (S1)
  INTO 200 BUCKETS
  STORED AS PARQUET """).show

spark.sql("INSERT INTO TEST1 SELECT * FROM VALUES(1,1,1)").show

Error message:
scala> spark.sql("INSERT INTO TEST1 SELECT * FROM VALUES(1,1,1)").show
org.apache.spark.sql.AnalysisException: org.apache.hadoop.hive.ql.metadata.HiveException: Bucket columns V1 is not part of the table columns ([FieldSchema(name:v1, type:bigint, comment:null), FieldSchema(name:s1, type:int, comment:null)]
  at org.apache.spark.sql.hive.HiveExternalCatalog.withClient(HiveExternalCatalog.scala:112)
  at org.apache.spark.sql.hive.HiveExternalCatalog.listPartitions(HiveExternalCatalog.scala:1242)
  at org.apache.spark.sql.catalyst.catalog.ExternalCatalogWithListener.listPartitions(ExternalCatalogWithListener.scala:254)
  at org.apache.spark.sql.catalyst.catalog.SessionCatalog.listPartitions(SessionCatalog.scala:1166)
  at org.apache.spark.sql.execution.datasources.InsertIntoHadoopFsRelationCommand.run(InsertIntoHadoopFsRelationCommand.scala:103)
  at org.apache.spark.sql.execution.command.DataWritingCommandExec.sideEffectResult$lzycompute(commands.scala:108)
  at org.apache.spark.sql.execution.command.DataWritingCommandExec.sideEffectResult(commands.scala:106)
  at org.apache.spark.sql.execution.command.DataWritingCommandExec.executeCollect(commands.scala:120)
  at org.apache.spark.sql.Dataset.$anonfun$logicalPlan$1(Dataset.scala:228)
  at org.apache.spark.sql.Dataset.$anonfun$withAction$1(Dataset.scala:3687)
  at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$5(SQLExecution.scala:103)
  at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:163)
  at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$1(SQLExecution.scala:90)
  at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:772)
  at org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:64)
  at org.apache.spark.sql.Dataset.withAction(Dataset.scala:3685)
  at org.apache.spark.sql.Dataset.<init>(Dataset.scala:228)
  at org.apache.spark.sql.Dataset$.$anonfun$ofRows$2(Dataset.scala:99)
  at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:772)
  at org.apache.spark.sql.Dataset$.ofRows(Dataset.scala:96)
  at org.apache.spark.sql.SparkSession.$anonfun$sql$1(SparkSession.scala:615)
  at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:772)
  at org.apache.spark.sql.SparkSession.sql(SparkSession.scala:610)
  ... 47 elided
Caused by: org.apache.hadoop.hive.ql.metadata.HiveException: Bucket columns V1 is not part of the table columns ([FieldSchema(name:v1, type:bigint, comment:null), FieldSchema(name:s1, type:int, comment:null)]
  at org.apache.hadoop.hive.ql.metadata.Table.setBucketCols(Table.java:552)
  at org.apache.spark.sql.hive.client.HiveClientImpl$.toHiveTable(HiveClientImpl.scala:1082)
  at org.apache.spark.sql.hive.client.HiveClientImpl.$anonfun$getPartitions$1(HiveClientImpl.scala:732)
  at org.apache.spark.sql.hive.client.HiveClientImpl.$anonfun$withHiveState$1(HiveClientImpl.scala:291)
  at org.apache.spark.sql.hive.client.HiveClientImpl.liftedTree1$1(HiveClientImpl.scala:224)
  at org.apache.spark.sql.hive.client.HiveClientImpl.retryLocked(HiveClientImpl.scala:223)
  at org.apache.spark.sql.hive.client.HiveClientImpl.withHiveState(HiveClientImpl.scala:273)
  at org.apache.spark.sql.hive.client.HiveClientImpl.getPartitions(HiveClientImpl.scala:731)
  at org.apache.spark.sql.hive.client.HiveClient.getPartitions(HiveClient.scala:222)
  at org.apache.spark.sql.hive.client.HiveClient.getPartitions$(HiveClient.scala:218)
  at org.apache.spark.sql.hive.client.HiveClientImpl.getPartitions(HiveClientImpl.scala:91)
  at org.apache.spark.sql.hive.HiveExternalCatalog.$anonfun$listPartitions$1(HiveExternalCatalog.scala:1245)
  at org.apache.spark.sql.hive.HiveExternalCatalog.withClient(HiveExternalCatalog.scala:102)
  ... 69 more

Does this PR introduce any user-facing change?
No
How was this patch tested?
UT

cc @cloud-fan @yaooqinn @AngersZhuuuu

Can you post the full stacktrace? I'm a bit curious about how/where the error happens.

@cloud-fan I hive add stacktrace to PR description.

the table schema will be stored as lower cases while bucket column info will stay the same with user input.

I'm not sure this is true. Table schema and bucketed columns are both stored in the Spark-specific table properties which are case-preserving.

the table schema will be stored as lower cases while bucket column info will stay the same with user input.

I'm not sure this is true. Table schema and bucketed columns are both stored in the Spark-specific table properties which are case-preserving.

the schema here comes from metastore when we call methods like 'getRawTableOption' and the Spark schema info in properties don't overwrite the schema field in those cases.


Add flash & recaptcha partial to each component folders

Hello,
I have a small feature request about the recaptcha and flash partials. Actually, they are in a '/partials' folder because they are shared between all components. Sounds logical.
However, the fact to not be present in the component folder prevent to call them using this syntax in your theme.
{% partial<EMAIL_ADDRESS>%}
When called from the theme, it seems that the @syntax only look in the used component folder, so, by doing that, it throw an error saying the recaptcha partial cannot be found.
I don't have solution to that, I don't think we can browse between component folders using the @syntax.
The solution I suggest is to create a recaptcha.htm and flash.htm in each component folders, that directly reference the partials that are in /partial folder (to continue to share code).
What do you think about it ? I think it could be really useful because we rarely need to customize the recaptcha partial.

@skydiver any update about this suggestion ?

Hello @PubliAlex
Do you wanna submit a pull request for this?

No problem, if the idea is validated, I can work for a PR.


To be able to select "Volume up/down" as rebound shortcut

Summary of the new feature/enhancement

Proposed technical implementation details (optional)

what is a rebound shortcut? please add more details

what is a rebound shortcut? please add more details

Re-binding: replacing some functionality on keyboard

hi @suraj-grewel, you should be able to do this in the latest release: 0.20. Please let me know if this is still an issue

Can confirm that.
Nice improvement.


.count metric naming convention only applies to UpDownCounters

Fixes https://github.com/open-telemetry/opentelemetry-specification/issues/3457
(port of https://github.com/open-telemetry/opentelemetry-specification/pull/3493)

pinging @reyang @jack-berg @jsuereth @jmacd @MrAlias @gouthamve who approved this same PR before the repo split
(and @joaopgrassi who commented on it)

@trask would you resolve the merge conflicts? I think it's time to get this merged.


Am 180 mfa with totp and email

:id:
Fixes AM-180
:pencil2:
Added MFA TOTP using a "virtual' authenticator to generate codes.
:memo: Test scenarios
:computer: Add screenshots for UI
:books: Any other comments that will help with documentation
:man: :woman:@mentions of the person or team responsible for reviewing proposed changes
This is a nice example: https://github.com/gravitee-io/gravitee-access-management/pull/1822

IMHO, some test cases are missing (agree with the fact that the issue description is light)

We currently test the enrollment step but what about a MFA challenge on a second connection (with factor already enrolled)
we do not test the case where multiple factor are available (can be provisionned either by selfaccount api or by using the "Enrollment factor policy"), we should create a user with multiple factor and try to switch the method

also we should include the recovery code
Maybe a new issue need to be created to cover the cases with recovery code and alternative factor selection, but the authentication test with a factor already enrolled should be done in this one IMHO.

I agree but that wasn't the goal of this particular piece of work. It was merely to add an OTP MFA test. There simply isn't time to add those things to the MFA tests and it's out of scope for this work also.


Update user story and fix file name

What type of PR is this?
Add one of the following kinds:

correction
documentation

What this PR does / why we need it:
Update User Story and fix file name.
Related to Pr #125 and Issue #122
Closing previous PR as I couldn't include new commits.

@gregory1g in this PR are included the changes suggested to @jgarciahospital

lgfm

Hi @AxelNennker please take a look so we can merge this one. Thanks!
(@hdamker let me ping you just in case we have to merge this) 👀

Why is numberverification changing from Customer:Foobar to ASP:Fobar?
There does not seem to be any guideline in Camara and user stories are somewhat free-form, but shouldn't we stick to Customer like the user stories of other subprojects e.g. https://github.com/camaraproject/SimSwap/blob/main/documentation/API_documentation/SIM-Swap_check_User_Story.md
Why mention "channel partner"?

Why is numberverification changing from Customer:Foobar to ASP:Fobar?
There does not seem to be any guideline in Camara and user stories are somewhat free-form, but shouldn't we stick to Customer like the user stories of other subprojects e.g. https://github.com/camaraproject/SimSwap/blob/main/documentation/API_documentation/SIM-Swap_check_User_Story.md
Why mention "channel partner"?

Both User Stories are now aligned (SIM Swap and Number Verification), let's see if for further versions we have a CAMARA strict aligned format.
Changes applied are ok for me.

In the summary we use the term "phone", should we change to "device", can't find the term "phone" in the commonalities glossary.

In "Pre-Conditions" number 4, should we give a "hint" on the mechs used (e.g. standard "OIDC Authorization Code Flow")?

In the summary we use the term "phone", should we change to "device", can't find the term "phone" in the commonalities glossary.

mm I don't know since we also don't manage the concept device in this API


FeatureData[Sequence] does not appear to be a semantic type

@ebolyen tells me that this behaviour is not ok:
In [1]: from qiime.core.type.semantic import is_semantic_type

In [2]: from q2_types.feature_data import FeatureData, Sequence

In [3]: is_semantic_type(FeatureData[Sequence])
Out[3]: False

I found it when I got this exception (first part of exception omitted):
  File "/Users/ben/Projects/q2-feature-classifier-bk/q2_feature_classifier/_classifier.py", line 100, in <module>
    description='Train a scikit-learn classifier to classify reads.'
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/plugin/plugin.py", line 211, in register_function
    outputs, self._package)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/callable.py", line 363, in from_function
    function, inputs, parameters, outputs)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/type/signature.py", line 169, in from_function
    cls, function, inputs, parameters, outputs)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/type/signature.py", line 320, in function_to_signature
    return cls(input_types, param_types, defaults, output_types)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/type/signature.py", line 227, in __init__
    super().__init__(inputs, parameters, defaults, outputs)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/type/signature.py", line 41, in __init__
    self._assert_valid_inputs(inputs)
  File "/Users/ben/miniconda2/envs/qiime2/lib/python3.5/site-packages/qiime-2.0.6-py3.5.egg/qiime/core/type/signature.py", line 60, in _assert_valid_inputs
    (input_name, semantic_type))
TypeError: Input 'reference_reads' must be a semantic QIIME type, not FeatureData[Sequence]

Thanks!

@BenKaehler just did a fresh install and could not replicate this, however I think I know what happened.
@jairideout and @thermokarst just recently finished renaming the python package from qiime to qiime2. It looks like your environment still has qiime as a functional package, however q2_types did not change package names and relies on qiime2, so when it performs its registration it receives some base-classes (from qiime2) that are constructively identical to what is in qiime, but nominally unique. So when qiime's code is invoked, it doesn't recognize the base-class because isinstance operates on memory address, not class definition (using qiime2.core.type.semantic:is_semantic_type should have worked just fine).
A completely fresh environment should fix this. (Note, I still have a small chunk of qiime.core.type hanging around in a fresh conda environment for some reason, need to investigate that further)

Ah, the leftover qiime package is an implicit namespace package (thanks Python 3) leftover because I didn't delete qiime/ after pulling from master.

Thanks @ebolyen, I have checked with a fresh install and yes, it is not a problem.

Thanks @ebolyen and @BenKaehler for tracking this down!
I've run into similar issues in the past with lingering .pyc/__pycache__. Here's an alias I use all the time to delete these files. Just run kpyc from the root of your repository or package, and it'll recursively crawl and delete those files. It's tested on OS X, and @ebolyen ported it to Ubuntu (ask him if you need that alias, not sure what type of machine you're using).
alias kpyc='find . \( -type d -name __pycache__ -or -type f -name *.pyc \) -exec rm -rf {} +'


Fixed #43

as mentioned in #43, the addon has problems with vscode. This comes down to a bug, where we do not check, if we are already inside a valid summary tag.
Note that this fix is very straight forward and could be improved by e.g. using the regex method as done in other methodes in the analyzer as well.

Coverage increased (+0.5%) to 41.463% when pulling 7aa3b8f7046d435d4d23ad6433c79af9e9029815 on xlaech:FixProbWithVSCodeVim#43 into a35ed46b49ad91808d0ed28b91f10e233dd11691 on k--kato:master.

Thanks @xlaech
I reviewed your PR. That's really nice. I hope to work as before. Please see the below images.

Vim
#46
Gif

Off
Off

On
Off

On
On

While i might certainly be able to fix the problem with param, the current approach will never work with .
This means we need to come up with a complete new idea on how to distinguish between an vim insert and a user insertion. As far as i know this is not possible. I will discuss this in the vim repository.


SQLiteReporter: escape symbols properly.

Results can be found at '/Users/stanislaw/Projects/LLVM/BuildXcode/Debug/bin/1479765000.sqlite'
Cannot execute 'INSERT INTO execution_result VALUES ('0','0','','LLVM ERROR: Program used external function '__ZTIN4llvm20SectionMemoryManagerE' which could not be resolved!
');'

How can I reproduce this? SQLiteReporter test seems normal.

@Quolyk can be that it has gone. At least I have completely lost original context. Will close :)

@Quolyk @stanislaw it's not gone.
There are two places where we insert execution_result into a database: around lines 79 and 155.
We do it properly only In the second case.

@Quolyk are you willing to make a contribution? We are happy to help if you are :)
You could extend https://github.com/mull-project/mull/blob/master/unittests/SQLiteReporterTest.cpp with a test that would write the failing line from:
Cannot execute 'INSERT INTO execution_result VALUES ('0','0','','LLVM ERROR: Program used external function '__ZTIN4llvm20SectionMemoryManagerE' which could not be resolved!

P.S. This is how I would do it :)

@stanislaw I want to make a contribution :) Thanks for the help.

How can I create such test?
I suppose I should make ExecutionResult that produces
'LLVM ERROR: Program used external function '__ZTIN4llvm20SectionMemoryManagerE' which could not be resolved!?
Btw, there are no MutationResults in TestResult

You can try to construct an execution result with stderrOutput equal to 'LLVM ERROR: Program used external function '__ZTIN4llvm20SectionMemoryManagerE' which could not be resolved! and attempt to save it.

Thanks @Quolyk !


[WIP] platform to group hosts with default connection variables

Proposal: platforms
Author: Your Name <@yourGHid> IRC: handle (if different)
Date: 2017/xx/xx

Status: New
Proposal type: core
Targeted Release:
Estimated time to implement: X days

Motivation
Allow quicker setup to users by defining 'prexisting' platforms/groups that will configure a set of connection variables 'known to work' in conjunction.
Solution proposal
shell/become/prefered module extension
ansible_platform: posix  # this would be 'deafult'
    # implies the following
    ansible_transport: smart
    ansible_shell: sh
    ansible_become_method: sudo

ansible_platfrom: windows
   # implies
    ansible_transport: winrm
    ansible_shell: powershell
    ansible_become_method: runas
    ansible_remote_tmp: C:\temp

ansible_platfrom = SunOS ...AIX … HP-UX
....

ansible_platform=ios
    ansible_persistent=True
...

Anything else?
This can work well with a persistent fact cache and the contstructed inventory plugin and/or some 'pre detection' to classify the hosts to make new setups/adding newly provisioned hosts in a very quick way to inventory and start working with Ansible

use case?
Are there existing issues that would be resolved by this?

'ansible_platform' would be a fact?
Would there be a 'platform' keyword for plays/tasks ?
Are the platform definitions built from config? Could playbooks or roles define them? Could playbooks or roles extend or override them?
How would someone share a platform definition?
Can the platform definitions include variables and templating?
Whats the var precedence?

What all can a platform definition set? Only connection variables?

use case?
Are there existing issues that would be resolved by this?

No open issues, as 'motivation' states this just makes it easier for users to get started with Ansible

'ansible_platform' would be a fact?

No, we might need to change the name to disambiguate ...but it can be used with facts/inventory data to autoassign

Would there be a 'platform' keyword for plays/tasks ?

No,  it fits more with inventory than with play objects.

Are the platform definitions built from config? Could playbooks or roles define them? Could playbooks or roles extend or override them?

yes ... possibly, not sure why they would need to expand them, they are mostly intended for quick setup to be able to have hosts targeted with the correct connection/plugins/etc

How would someone share a platform definition?

depends on implementation .. but currently we could already do this by defining them as a group with associated variables.

Can the platform definitions include variables and templating?

I think the above already answers that

Whats the var precedence?

depends on implementation, but mostly they should be pre/post inventory vars

What all can a platform definition set? Only connection variables?

depends on implementation, it is geared mostly to connection variables ... not sure what else would make sense under these, but should be easy enough to expand.

I have some concerns with implementing this simply as built-in groups with no other changes:

It's difficult to reason about the state of connection vars introduced by other group memberships.

Could be mitigated by making platform groups the lowest in precedence, so that any "real" group the host is a member of can have normal group precedence behavior WRT any connection vars they might set.

Host->group membership feels "wonkier" than just setting a hostvar like ansible_platform=windows, and also allows a host to "belong" to multiple platforms (which is almost certainly not correct), rather than being a "property" of the host like a var would be.

No reason we can't actually do the var assignment as a group under the covers, but maybe by generating the group name and controlling assignment later in the inventory construction. But at that point, given some of the other things above, would it just make sense to have a mapping of "final platform var value just sets all the other var values for that platform"?
The bogus multi-membership thing could be mitigated by enforcing that a host is always a member of exactly one platform group- either the default or an explicit/group assignment. If a host ends up in more than one platform group, it's a failure.

Exposing "well-known" group names for platform potentially calcifies this implementation (eg, people will have to reference the groups in their inventories, and could potentially even use them as play targets)

Not sure if this is really an issue, but if we decide we need more than "just a group" down the line, this could be problematic.

Plugins can't rely on group membership to do conditional behavior based on platform (eg, Powershell shell plugin on Linux must behave differently than Powershell on Windows, though largely the same)

Could be mitigated by adding a "platform" fact (could be problematic for non-Windows though- how granular do you get?), or by subclassing the Powershell shell plugin for Linux/Windows and doing the shared stuff on the base, and having the default windows platform select the windows+powershell subclass.

I think controlling all this stuff via vars makes more sense than using a simple group membership- the number of exceptions we need to make to current group behavior for robustness makes it feel like maybe groups aren't exactly the right thing for this.

It could probably be argued that platforms should be composable like normal groups (eg, platform: windows + platform: server_2008r2), but I think we're getting into a major new precedence headache there.

For precedence issues we can set ansbile_group_priority on all to make sure they are the lowest, or even make them 'top' groups like 'all' itself and outside normal heir, if someone has a reason to override these vars at a group level they should be able to.
As for being 'targetable', yes, this was a feature in my view.
composing the platform groups .. I had not thought of this, could be a problem and restriction seems clunky solution.
as for your example, I would have made server_2008r2 a 'child' of the windows one so it inherits vars but can override what it needs.
not sure what the powershell issue is but seems like a connection var (which shell modules soon will support) would solve that issue.

Yeah, sounds like there are decent workarounds to everything except the "can be a member of multiple platforms" issue. I'm not sure that issue is worth going a "harder" route or not. I haven't floorplanned a solution for "platform-as-a-var" yet, but I suspect it will be significantly "gnarlier" than this. If we can agree on the "one platform group per host" restriction, I think I'd be willing to call it "good enough". I also think this would work well enough for what networking needs, but I'd want to kick that around with @Qalthos and maybe @privateip a bit before making that call.

@bcoca I understand this is a readmap item for ansible 2.7 so I hope this comment is still timely.
This may be scope creep but I just noticed yesterday that the default module for ad-hoc ansible commands does not appear to be sensitive to the connection in use, meaning you can't use command lines like
ansible windows_host -a 'dir'
(Just to be clear, running
ansible windows_host -m raw -a 'dir'
does what you would expect though).
So would it possible as part of this to specify default module - or would this be a separate change to make default module a property of the connection plugin?

@jhawkesworth any of the general configuration items is available, so it does seem to make a lot of sense to change to win_command when platform == windows
cc @nitzmahone


Popover add 'trigger'('click' | 'hover') prop  and 'animation'

🚀 Feature Proposal
Popover add 'trigger'('click' | 'hover') prop and 'animation'
Motivation
trigger open by clicking or hovering
add animation effect to popup

Popover is gone in v11, please migrate to Reakit Popover.


support nibabel 5.0.0+

Summary

related to #758, after merging #757. we do not support nibabel 5.0.0 because of test failures.
Additional details

supporting nibabel 5.0.0
Next steps

identify places where we pass an int array into a nifti object

Passing an int64 array to Nifti1Image without a header or dtype argument will raise a ValueError

Probably good to use float32s instead of floats as well, I think.