feat(query): Add many types

Codecov Report
Patch and project coverage have no change.

Comparison is base (607fa6a) 92.14% compared to head (1916dac) 92.14%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #742   +/-   ##
=======================================
  Coverage   92.14%   92.14%
=======================================
  Files         199      199
  Lines       40790    40790
=======================================
  Hits        37584    37584
  Misses       3206     3206

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Incorrect depth sorting for materials using Depth Prepass and materials using blend modes like Add. (Also affects Godot 4)

Godot version:
3.3.2.stable
OS/device including version:
Both Rendering backends
OS: Linux Mint 20.1 x86_64
GPU: NVIDIA GeForce GTX 1060 6GB - Driver version: 460.80
Issue description:
https://user-images.githubusercontent.com/781121/121025186-33f29800-c7a5-11eb-9b49-4f4638db550a.mp4
https://user-images.githubusercontent.com/781121/121025398-69978100-c7a5-11eb-95bd-0119a6260588.mp4
I'm working on a scene with trees, for the trees I need to use a material that has "depth_draw_alpha_prepass" enabled, to have the transparent leaves texture cast shadows.
I also use materials for particle systems and effects with the "Add" blend mode enabled, as shown in the videos.
Godot for some reason does not sort these correctly when the tree leaves material has "depth_draw_alpha_prepass"  enabled.
As soon as I turn that off, the sorting behaves like it should again, but then my leaves do not cast shadows anymore, and don't render correctly anymore either.
Is this working as intended? Am I expecting too much of the engine here?
(I ask this because I know depth sorting with transparent materials is hard to get right on most engines), or is there actually something going wrong here?
Thanks in advance.
Steps to reproduce:

Open the MRP.
Move the camera in the editor so that the tree displays behind the sphere.
Move the camera backwards away from the sphere to reproduce.

Minimal reproduction project:
mrp.zip

Can you test this in a more recent release? There were some fixes to the depth prepass and transparency sorting that came in 3.3.
For reference, current stable is 3.3.2 https://godotengine.org/download
Also, is this GLES2 or GLES3?

@clayjohn Apologies, I typo'd when writing down the Godot version number, I meant to type 3.3.2 when I created this issue.
This is in GLES3.

Do you have the same issue when switching to GLES2?

@clayjohn Yes, here's a video of the MRP in GLES2.
https://user-images.githubusercontent.com/781121/121059817-31526b80-c7c2-11eb-8686-74ad56556730.mp4

@clayjohn
I tested this again in v3.4.beta.custom_build d3f500cf3, and it looks like this is fixed now.
Likely thanks to this PR: #50721.
Thanks again!

Fixed by #50721.

I'm guessing developers are probably already aware of this, but just in case, this issue is still present in the latest Godot 4 beta. (Beta5 as of writing)
Video: https://streamable.com/ni92le


Service Catalog manager constantly makes requests to etcd if ServiceBinding has ReferencesNonexistentServiceClass status

Bug Report
What happened:
Service Catalog manager constantly makes PUT request to etcd if Service Binding has reference to non existing Service Class.
It causes among others OOMKilled status on etcd pods when etcd is run with default snapshot count configuration which is 100000.
What you expected to happen:
Manager should stop sending requests after some time with the same message or sending them with increasing time intervals.
How to reproduce it (as minimally and precisely as possible):

Add custom namespace
Add custom Service Broker
Create Service Instance
Create Service Binding
Remove namespace

Similar situation occurs when SerciveBroker is unregistered without remove ServiceBinding or ServiceInstance before.
Removing is called asynchronously and is happens often that ServiceBroker is removed before ServiceBinding and ServiceInstance.
Result
After delete namespace, ClusterServiceBroker and ClusterServiceClass are removed, ServiceInstance and ServiceBinding
stay on cluster without ServiceClass parent:
  NAME                             CLASS                          PLAN      STATUS
  orders-48ab05bf-critical-score   ServiceClass/orders-48ab05bf   default   DeprovisionBlockedByExistingCredentials

  NAME                   SERVICE-INSTANCE                 SECRET-NAME            STATUS
  kind-newton            orders-48ab05bf-critical-score   kind-newton            ReferencesNonexistentServiceClass

When we try watch etcd key connected with ServiceBinding we got:
PUT
/registry/servicecatalog.k8s.io/servicebindings/production/kind-newton
{"kind":"ServiceBinding","apiVersion":"servicecatalog.k8s.io/v1beta1","metadata":{"name":"kind-newton","namespace":"production","uid":"74df4c91-f923-11e8-a2ff-0242ac110013","generation":2,"creationTimestamp":"2018-12-06T06:52:11Z","deletionTimestamp":"2018-12-06T08:27:38Z","deletionGracePeriodSeconds":0,"finalizers":["kubernetes-incubator/service-catalog"]},"spec":{"instanceRef":{"name":"orders-48ab05bf-critical-score"},"parameters":{},"secretName":"kind-newton","externalID":"74df4bb0-f923-11e8-a2ff-0242ac110013","userInfo":{"username":"minikube","uid":"","groups":["system:masters","system:authenticated"]}},"status":{"conditions":[{"type":"Ready","status":"False","lastTransitionTime":"2018-12-06T06:52:33Z","reason":"ReferencesNonexistentServiceClass","message":"The binding references a ServiceClass that does not exist. References a non-existent ServiceClass \"48ab05bf-9aa4-4cb7-8999-0d3587265ac3\" - {ServiceClassExternalName:\"orders-48ab05bf\"}"}],"asyncOpInProgress":false,"currentOperation":"Unbind","reconciledGeneration":1,"operationStartTime":"2018-12-06T06:52:33Z","externalProperties":{"userInfo":{"username":"system:serviceaccount:kyma-system:core-ui-api","uid":"","groups":["system:serviceaccounts","system:serviceaccounts:kyma-system","system:authenticated"]}},"orphanMitigationInProgress":false,"unbindStatus":"Required"}}

PUT
/registry/servicecatalog.k8s.io/servicebindings/production/kind-newton
{"kind":"ServiceBinding","apiVersion":"servicecatalog.k8s.io/v1beta1","metadata":{"name":"kind-newton","namespace":"production","uid":"74df4c91-f923-11e8-a2ff-0242ac110013","generation":2,"creationTimestamp":"2018-12-06T06:52:11Z","deletionTimestamp":"2018-12-06T08:27:48Z","deletionGracePeriodSeconds":0,"finalizers":["kubernetes-incubator/service-catalog"]},"spec":{"instanceRef":{"name":"orders-48ab05bf-critical-score"},"parameters":{},"secretName":"kind-newton","externalID":"74df4bb0-f923-11e8-a2ff-0242ac110013","userInfo":{"username":"minikube","uid":"","groups":["system:masters","system:authenticated"]}},"status":{"conditions":[{"type":"Ready","status":"False","lastTransitionTime":"2018-12-06T06:52:33Z","reason":"ReferencesNonexistentServiceClass","message":"The binding references a ServiceClass that does not exist. References a non-existent ServiceClass \"48ab05bf-9aa4-4cb7-8999-0d3587265ac3\" - {ServiceClassExternalName:\"orders-48ab05bf\"}"}],"asyncOpInProgress":false,"currentOperation":"Unbind","reconciledGeneration":1,"operationStartTime":"2018-12-06T06:52:33Z","externalProperties":{"userInfo":{"username":"system:serviceaccount:kyma-system:core-ui-api","uid":"","groups":["system:serviceaccounts","system:serviceaccounts:kyma-system","system:authenticated"]}},"orphanMitigationInProgress":false,"unbindStatus":"Required"}}

...

The only change that appeared on is deletionTimestamp value. Request is send every 10 second.
Environment:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.0", GitCommit:"0ed33881dc4355495f623c6f22e7dd0b7632b7c0", GitTreeState:"clean", BuildDate:"2018-09-28T15:18:13Z", GoVersion:"go1.11", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:44:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

service-catalog version: v0.1.38
Cloud provider or hardware configuration: minikube version: v0.28.2
Do you have api aggregation enabled?

Do you see the configmap in kube-system?
yes
Does it have all the necessary fields?

kubectl get cm -n kube-system extension-apiserver-authentication -o yaml and look for requestheader-XXX fields
yes it does

Install tools:

Did you use helm? What were the helm arguments? Did you --set any extra values?
yes

Are you trying to use ALPHA features? Did you enable them?
no

I tested this scenario on SC ver 0.1.41. I was able to remove the namespace, all the service classes were automatically removed as well. Looks like its fixed.


Added a file named reverse_string.cpp

thanks, bro for contributing pls star this repo and follow me ur pr is been merged


New availableModes for google home thermostat

Hello,
I am trying to build an app for controlling a thermostat using google home. The problem that I have is that this thermostat has some custom functions (healing/cooling) and custom modes ( automatic, manual, protection, boost), and I will need some custom availableModes in order to achieve this.
Here you can find my sync response in order to see better how I intend you map this custom modes:
{
  "requestId": "ff36a3cc-ec34-11e6-b1a0-64510650abcf",
  "payload": {
    "agentUserId": "1836.15267389",
    "devices": [{
      "id": "123",
      "type": "action.devices.types.THERMOSTAT",
      "traits": [
        "action.devices.traits.TemperatureSetting","action.devices.traits.Modes"
      ],
      "name": {
        "defaultNames": ["Honeywell Thermostat T-1000"],
        "name": "Homer Simpson Thermostat",
        "nicknames": ["living room thermostat"]
      },
      "willReportState": false,
      "attributes": {
        "availableThermostatModes": "off,heat,cool,on",
        "thermostatTemperatureUnit": "F",
	"availableModes": [{
          "name": "function",
          "name_values": [{
            "name_synonym": ["function"]
            "lang": "en"
          },{
            "name_synonym": ["fonction"]
            "lang": "fr"
          },{
            "name_synonym": ["funzione"]
            "lang": "it"
          }],
          "settings": [{
            "setting_name": "heating",
            "setting_values": [{
              "setting_synonym": ["heating","heat"]
              "lang": "en"
            },{
              "setting_synonym": ["chauffage","chaleur"]
              "lang": "fr"
            },{
              "setting_synonym": ["riscaldamento","calore"]
              "lang": "it"
            }]
          }, {
            "setting_name": "cooling",
            "setting_values": [{
              "setting_synonym": ["cooling", "cool"],
              "lang": "en"
            },{
              "setting_synonym": ["refroidissement", "rafraîchir"],
              "lang": "fr"
            },{
              "setting_synonym": ["raffreddamento", "freddo"],
              "lang": "it"
            }]
          }],
          "ordered": true
        },
	{
          "name": "mode",
          "name_values": [{
            "name_synonym": ["mode"]
            "lang": "en"
          },{
            "name_synonym": ["modalità"]
            "lang": "it"
          },{
            "name_synonym": ["mode"]
            "lang": "fr"
          }],
          "settings": [{
            "setting_name": "automatic",
            "setting_values": [{
              "setting_synonym": ["automatic"]
              "lang": "en"
            },{
              "setting_synonym": ["automatico"]
              "lang": "it"
            },{
              "setting_synonym": ["automatique"]
              "lang": "fr"
            }]
          }, {
            "setting_name": "manual",
            "setting_values": [{
              "setting_synonym": ["manual"]
              "lang": "en"
            },{
              "setting_synonym": ["manuale"]
              "lang": "it"
            },{
              "setting_synonym": ["manual"]
              "lang": "fr"
            }]
          },
	 {
            "setting_name": "protection",
            "setting_values": [{
              "setting_synonym": ["protection"]
              "lang": "en"
            },{
              "setting_synonym": ["protezione"]
              "lang": "it"
            },{
              "setting_synonym": ["protection"]
              "lang": "fr"
            }]
          },
	{
            "setting_name": "boost",
            "setting_values": [{
              "setting_synonym": ["boost"]
              "lang": "en"
            },{
              "setting_synonym": ["impulso"]
              "lang": "it"
            },{
              "setting_synonym": ["renforcer"]
              "lang": "fr"
            }]
          }],
          "ordered": true
        }]

      },
      "deviceInfo": {
        "manufacturer": "hrxhgfst",
        "model": "t-1000",
        "hwVersion": "3.2",
        "swVersion": "11.4"
      },
      "customData": {
        "fooValue": 74,
        "barValue": true,
        "bazValue": "lambtwirl"
      }
    }]
  }
}

Let me know you can help me to achieve this.
Thank you

To hijack this, are Custom modes going to be part of the Smart Home actions any time soon? I understood it was being internally tested a couple of months back?
I notice the documentation on the Mode trait uses the word arbitrary, but having to get you to hard code them isn't very arbitrary :)

Definitely the goal is to have custom modes so that developers can have a lot of flexibility. It's not ready yet.

Someone, like myself, would need to manually add each new mode or toggle to an internal list.

Coming back to this feature request:
The function/heating/cooling already exist as part of the TemperatureSetting trait.
Mode names which exist:

mode

Mode settings which exist:

Auto (automatic is a synonym) ("auto" in FR, "automatico" in IT)
manual (EN) manuale (IT)
boost (EN, FR, and IT, but only as "boost")

Mode settings to add

Auto (FR)
manual (FR)
protection (EN, FR, IT)
boost (FR, IT)

Boost won't be an availableThermostatMode, it exists as a generic mode.

Hello Nick, I've added this but it fails at sync.
Also is there a way to set a duration for this boost? (30/60/90 minutes) ?
Thank you

There's no parameter for time.

@Fleker Hi.
I am trying to add modes for Oven but it is not working. What's wrong with it.
For example I want to ask asistant: set my oven to bake
availableModes: [{
  name: 'cook',
  name_values: [{
    name_synonym: ['cook', 'cook mode'],
    lang: 'en'
  }],
  settings: [{
    setting_name: 'bake',
    setting_values: [{
      setting_synonym: ['bake', 'bread baking'],
      lang: 'en'
    }]
  }, {
    setting_name: 'pizza',
    setting_values: [{
      setting_synonym: ['pizza'],
      lang: 'en'
    }]
  }],
  ordered: false
}]

@kirill3333 can you make this a separate issue. This is unrelated to the original issue.


feature: helm chart separation and crd inclusion

Fixes #
Description

Split charts between karpenter-core and karpenter
Include CRDs as a separate chart

How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing in favor of https://github.com/aws/karpenter/pull/3260, The Chart separation piece will come later


Restructure subscription tier types

[x] Remove Pro Enum
[x] Rename Hobby Enum to Individual
[ ] Use color schema and icon of Pro for Individual
[x] Migrate existing Pro tiers to Individal tier type

Context: https://discord.com/channels/1078611507115470849/1192087765031067719

🥳

🚀


Please add ARMv9.2, SVE2, Dot Product and Neon support

It's sad that it's not possible to compile the newest version on Android and Linux devices.

I think that changes made since the opening of this issue have resolved any problems viri has with compiling for ARM, at least in terms of performance.


Feature: Move auth to middleware in things service

Is your feature request related to a problem? Please describe.
Related to issue https://github.com/absmach/magistrala/issues/2028
Describe the feature you are requesting, as well as the possible use case(s) for it.
Move existing auth implementation to middleware for the things service
Indicate the importance of this feature to you.
Must-have
Anything else?
No response

It was resolved by #2424.


"Cluster undefined does not exist" when deploying pre 1.7.x prisma.yml with unknown cluster

Bug Report
Current behavior
Running prisma deploy with a prisma.yml in pre 1.7.x format that references a cluster that is not in ~/.prisma/config.yml returns "Cluster undefined does not exist" error message.

Expected behavior?

create backup file with old values

then, here's my suggestion:

prompt for old cluster endpoint
prompt to set environment variable
migrate prisma.yml to new values
include link to upgrade guide

This does not happen anymore.


Add detail for required field error

Description
I use go-env module in our project, It's was cool.
But I found ErrMissingRequiredValue does't show important detail about required field that miss.
Then I try to add what's required field that missing in error message 😊

Prefer if you just refactor to:
type ErrMissingRequiredValue struct {
    Value string
}

@hinshun I'm not sure if that's right.

Thanks @PePoDev !


Move Code from README to a getting_started.ipynb

The code examples currently present in the README file can be moved to a Jupyter Notebook. This will provide users with an interactive environment to explore the toolkit and improve the clarity and usability of the documentation.

This already exists right? It's good to have the code examples in the readme for clarity I think


Update setup.py

They told something to make a pull request, so I did instead. I'd like my GitHub sponsor thanks.

Yes I plan to add ColossialAI as someone suggested on the LAION Server

i'm so confused

Try again


Recommend addition of ssl cert copy feature

Recommend creating a root folder for ssl certs then at the top of the tower-setup.yml copy the certs into the roles/nginx/files, roles/postgres/files, and roles/ca_cert/files directories. In this example, the folder is sslcerts, which is added to the root directory.
This can be consolidated most likely using a loop, but:
- name: Create directories for various ssl certs
    file:
      path: "{{item}}"
      state: directory
    with_items:
      - "{{ playbook_dir }}/{{ tower_installer_current }}/roles/ca_cert/files"
      - "{{ playbook_dir }}/{{ tower_installer_current }}/roles/nginx/files"
      - "{{ playbook_dir }}/{{ tower_installer_current }}/roles/postgres/files"

  - name: Copy CA certs from root directory
    copy:
      src: "{{ item }}"
      dest: "{{ playbook_dir }}/{{ tower_installer_current }}/roles/ca_cert/files/"
      owner: "root"
      mode: 0600
    with_fileglob:
      - "sslcerts/CA/*"

  - name: Copy postgresql certs from root directory
    copy:
      src: "{{ item }}"
      dest: "{{ playbook_dir }}/{{ tower_installer_current }}/roles/postgres/files/"
      owner: "root"
      mode: 0600
    with_fileglob:
      - "sslcerts/postgres/*"

  - name: Copy nginx certs from root directory
    copy:
      src: "{{ item }}"
      dest: "{{ playbook_dir }}/{{ tower_installer_current }}/roles/nginx/files/"
      owner: "root"
      mode: 0600
    with_fileglob:
      - "sslcerts/nginx/*"

Another option is to change the .ini inventory files to yaml so that perhaps jinja templating could be used.

Closing for lack of interest.


Generate fails on 5.2 with unsupported type

Hi Prisma Team! The following command just crashed.
Command
generate
Versions

Name
Version

Platform
darwin-arm64

Node
v18.17.1

Prisma CLI
5.2.0

Engine
2804dc98259d2ea960602aca6b8e7fdc03c1758f

prisma                  : 5.2.0
@prisma/client          : 5.2.0
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine 2804dc98259d2ea960602aca6b8e7fdc03c1758f (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 2804dc98259d2ea960602aca6b8e7fdc03c1758f (at node_modules/@prisma/engines/schema-engine-darwin-arm64)
Schema Wasm             : @prisma/prisma-schema-wasm 5.2.0-25.2804dc98259d2ea960602aca6b8e7fdc03c1758f
Default Engines Hash    : 2804dc98259d2ea960602aca6b8e7fdc03c1758f
Studio                  : 0.494.0

Error
RustPanic: RuntimeError: panicked at 'internal error: entered unreachable code: No unsupported field should reach that path', query-engine/schema/src/build/input_types/mod.rs:26:40

Description
Trying to do a version update from 4.14 to 5.2, no errors in 4.14 but seeing this crash on update. Postgres version is <IP_ADDRESS>, hosting with Supabase, cloud provider AWS. We are using the type "Unsupported("geometry(Point, 4326)")" to support postGIS, and I saw there is some history of unsupported types (particularly when used as indices) causing trouble in previous versions.
https://github.com/prisma/prisma/issues/18517
https://github.com/prisma/prisma-engines/pull/3960
I have confirmed that if I temporarily remove the unsupported fields, the generate command succeeds (but not if I just remove the indices). We are using 'unsupported' as follows:
coords              Unsupported("geometry(Point, 4326)")?

@@index([coords], map: "location_coords", type: Gist)

I couldn't reproduce with this schema snippet, but I could reproduce it with the schema from the other issue that also triggers the same error: https://github.com/prisma/prisma/issues/20986 (it also uses an unsupported type in an index).
I can also confirm that it's a regression in 4.16.0.

After diving into the code I'm not sure this is exactly the same issue as https://github.com/prisma/prisma/issues/20986 anymore as this issue mentions @@index and not @@unique.
@HillaryManning is there any chance you could isolate a minimal reproduction so we could take a look?

@aqrln I haven't had a chance yet but I will get you a minimal reproduction! Thanks for looking into this.

@HillaryManning if it will save time, you can also test if Prisma 5.3.0 fixes the issue for you. If it does, then we can close it and the reproduction will not be required (though still much appreciated so we can add a test if necessary), and if not, then we'll definitely need a reproduction to investigate.

@aqrln apologies for the delay:  I confirmed this morning that I am able to generate successfully with version 5.4.1.  Minimal reproducible with  5.2.0  involves a unique constrain on unsupported AND another value (tried string and float types, both failed). If I remove either field from the unique constraint, it generates successfully.
model Location {
  id        String   @id @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  desc String?
  coords      Unsupported("geometry(Point, 4326)")

  @@unique([coords, desc])
  @@map("locations")
}

@HillaryManning thank you for checking this and for following up! Then it is indeed the same issue that was fixed by https://github.com/prisma/prisma-engines/pull/4219. I'll go ahead and close this issue as fixed.


u'/' must not be unicode - both because os.sep is a byte string

and not a unicode string and it causes UnicodeErrors with code
using pyfilsystem for transparent access to arbitrary filesystems
using the same code as
www.xml-director.info.

Do you have a traceback for the unicode errors? Which platform / Python version?
The fix might not be so simple, os.sep is unicode on Python3.

I am getting this error with OSFS on Mac with Python 2.7.X.
The same code (including tests) is passing on Linux and Mac with DAVFS.
The goal of my project is support all common FS types (WebDAV, LocalFS, FTPSH, SFTPSH)
Traceback (most recent call last):
File "/Users/ajung/.buildout/eggs/unittest2-0.5.1-py2.7.egg/unittest2/case.py", line 333, in run
self.setUp()
File "/Users/ajung/sandboxes/xmldirector.plonecore/xmldirector/plonecore/tests/test_connector.py", line 43, in setUp
with handle.open(PREFIX + u'/üöä/üöä.xml', 'wb') as fp:
File "/Users/ajung/.buildout/eggs/fs-0.5.2-py2.7.egg/fs/iotools.py", line 126, in wrapper
**kwargs)
File "/Users/ajung/sandboxes/xmldirector.plonecore/xmldirector/plonecore/davfs.py", line 94, in open
self.check_lock(path, op='open{}'.format(mode[0]))
File "/Users/ajung/sandboxes/xmldirector.plonecore/xmldirector/plonecore/davfs.py", line 75, in _check_lock
log_info = lm.get_lock(path)
File "/Users/ajung/sandboxes/xmldirector.plonecore/xmldirector/plonecore/locking.py", line 88, in get_lock
with handle.open(lock_filename, 'rb', lock_check=False) as fp:
File "/Users/ajung/.buildout/eggs/fs-0.5.2-py2.7.egg/fs/iotools.py", line 126, in wrapper
**kwargs)
File "/Users/ajung/sandboxes/xmldirector.plonecore/xmldirector/plonecore/davfs.py", line 95, in open
return super(BaseWrapper, self).open(path, mode, **kwargs)
File "/Users/ajung/.buildout/eggs/fs-0.5.2-py2.7.egg/fs/errors.py", line 257, in wrapper
return func(self,*args,**kwds)
File "/Users/ajung/.buildout/eggs/fs-0.5.2-py2.7.egg/fs/osfs/init.py", line 221, in open
sys_path = self.getsyspath(path)
File "/Users/ajung/.buildout/eggs/fs-0.5.2-py2.7.egg/fs/osfs/init.py", line 158, in getsyspath
path = relpath(normpath(path)).replace(u"/", os.sep)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 45: ordinal not in range(128)

I don't think it will fix it, but can you try against 0.5.3?
Can I just check you are calling open with a unicode string? Paths within a FS object should all be unicode.

fs 0.5.3 has the same problem. And yes, the related unittest creates a filename as unicode string
with handle.open(PREFIX + u'/üöä/üöä.xml', 'wb') as fp:
....

I can't reproduce that here. On OSX, Python2.7. Any chance you could give me a short script that shows the problem?
btw, test_connector.py has a strange coding declaration, first line. Not sure if that's related.

I can't reproduce this issue. Let me know if you can reproduce it with a simple code snippet.


[FEATURE] Install deps from pyproject/setup.cfg into active environment using conda/mamba

Is your feature request related to a problem? Please describe.
Very often when developing (setuptools-based) packages, one sets up their local env with pip install -e .[dev] or something like that.  If, however, you wanted to use conda-forge to grab everything, the primary "solution" seems to be to maintain a duplicate environment.yaml file.  That bugs me :)
Describe the solution you'd like
I'd like to be able to do something like grayskull install -e .[dev], and have it parse my install_requires, along with any requested extras, and use mamba to install them.  Since grayskull already has all the logic to convert from pip deps to conda reps, it seems sort of in the wheelhouse here?
Additional context
I hacked together a quick-and-dirty proof of principle here: https://gist.github.com/tlambert03/cdc63d68f4fd1bce20f8f3e769217759
using  extract_requirements, solve_list_pkg_name, and  ensure_pep440_in_req_list from this package.
Is this a feature you'd consider supporting?  If so, I'd be happy to put it together in a PR, and take your advice on cleaning it up.  If you think that sort of functionality is outside of the scope here, I understand that too.
thanks for your work!

on further thought... I do think that the full installation procedure is probably out of scope here (though curious to hear your thoughts).  What would probably be useful is a slightly more public way to take a list of PyPI requirements and return a list of conda-ready requirements:
def condafy_requirements(requires: List[str], ...) -> List[str]:
    """Where `requires` is a list of pip requirement specifiers"""

we can create an environment.yaml based on the run requirements or conda-lock file, but that is really out of scope of grayskull

fair enough! thanks for your time :)

(side note, proof of principle package pamba here, using grayskull: https://github.com/tlambert03/pamba)


Readme improvements

Readme improvements with typo fixes and some additional details such as local component registration.

Thanks!


Masked images do not render on Android

I'm using the react-native-figma-squircle package alongside this package, for whatever reason the following code renders nothing of the underlying image at all on Android, however it works perfectly fine on iOS:
type Props = { uri: string; size: number; style?: ViewStyle };

function PerkLogo({ uri, size, style }: Props) {
  const dimensionsStyle = { width: size, height: size };

  return (
    <View style={[styles.imageContainer, dimensionsStyle, style]}>
      <MaskedView
        // style={[mainStyles.overflowHidden]}
        maskElement={
          <SquircleView
            style={StyleSheet.absoluteFill}
            squircleParams={{
              cornerSmoothing: 1,
              fillColor: colors.white,
              cornerRadius: 30,
            }}
          />
        }>
        <Image source={{ uri: CDN(uri) }} style={[styles.image, dimensionsStyle]} />
      </MaskedView>
    </View>
  );
}

const styles = StyleSheet.create({
  imageContainer: {
    // backgroundColor: '#fff',
    borderRadius: defaultBorderRadius,
    overflow: 'hidden',
  },
  image: {
    ...mainStyles.imageBorderRadius,
    resizeMode: 'contain',
  },
});

react-native: 0.63.3
react-native-masked-view: 0.2.5
react-native-figma-squircle: 0.1.2
I've wasted hours on this between forcing a re-render using a key prop on MaskedView to changing all the styles of every element in this code. Every once in a while randomly the image shows up when I navigate away from the screen using this component. Otherwise it seems to be completely broken on Android.
Any suggestions? Is this a known bug on Android with image masking?

I have that same issue. Debugged that a bit and seems that onDescendantInvalidated is not being triggered when path in SquircleView is rendered after onLayout. So nothing is invalidating the mask.

@split Good insight! Have you found a solution of some sort? I'm guessing onDescendantInvalidated is in native code?

try with this: https://github.com/Only-IceSoul/react-native-drawableview

I'm having the same issue, how can this be fixed?

Any update on this ?

thanks @lyubo-and
it looks like its working for me!

The issue is the view is not re-rendering on either:

The content loading/changing
The mask loading/changing
Both of the above

Below is very sub-optimal workaround, but it works.
Assuming 1) is the issue I set a 1pt top-margin on LoadEnd to force a re-layout/render
    <MaskedView
      maskElement={
        <Image
          source={require('../assets/images/hex-mask.png')}
          style={{
            width: '100%',
            height: '100%',
          }}
        />
      }
      style={{
        marginTop: loaded ? 1 : 0,
      }}>
      <Image
        onLoadEnd={() => {
          setLoaded(true);
        }}
        source={{
          uri: '...someImageUrl',
        }}
        style={{
          width: '100%',
          height: '100%',
        }}
      />
    </MaskedView>


Global constant string.

Global constant strings cannot be generated in LLVM IR.
Foo :: "Hello world!";

Test case:
#load "std/core.bl"

Foo :: "Hello world\n";

main :: fn () s32 {
  print(Foo);
  ptr := Foo.ptr; // LLVM error here
  return 0;
};

abort [ir.c:593]: invalid constant type

Caused by missing generation of constant pointer in ir.c.


Feature Request: Add text-underline utility class

Hello there,
An utility class for text-underline would be awesome!

Has PR https://github.com/twbs/bootstrap/pull/23881

Closing per #23881.


Wikisearch pipeline error

Hi,
I was trying to get txtchat up and running (thanks for the great application!) and I ran into an issue with the wikitalk example. The persona I am using is wikitalk.yml available on HF hub: https://huggingface.co/NeuML/txtchat-personas/blob/main/wikitalk.yml.
The issue arises when trying to initialize the agent. I get the following traceback:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtchat/agent/__main__.py", line 21, in <module>
    agent = AgentFactory.create(sys.argv[1])
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtchat/agent/factory.py", line 34, in create
    return RocketChat(config)
           ^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtchat/agent/rocketchat.py", line 30, in __init__
    super().__init__(config)
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtchat/agent/base.py", line 32, in __init__
    self.application = Application(config)
                       ^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtai/app/base.py", line 72, in __init__
    self.pipes()
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtai/app/base.py", line 129, in pipes
    self.pipelines[pipeline] = PipelineFactory.create(config, pipeline)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtai/pipeline/factory.py", line 55, in create
    return pipeline if isinstance(pipeline, types.FunctionType) else pipeline(**config)
                                                                     ^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/anaconda3/envs/txtchat/lib/python3.11/site-packages/txtchat/pipeline/wikisearch.py", line 32, in __init__
    self.workflow = Workflow([Question(action=application.pipelines["extractor"]), WikiAnswer()])
                                              ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^
KeyError: 'extractor'

To my untrained eye, the issue appears to be coming from the Wikisearch pipeline constructor here:
https://github.com/neuml/txtchat/blob/master/src/python/txtchat/pipeline/wikisearch.py#L32
Specifically, the action argument: action=application.pipelines["extractor"]),
When txtai.application.Application is creating pipelines, it attempts uses PipelineFactory to create the Wikisearch pipeline here: https://github.com/neuml/txtai/blob/f9229bc27c5160402ffb8caee3ab24620c8e602a/src/python/txtai/app/base.py#L129
self.pipelines[pipeline] = PipelineFactory.create(config, pipeline)
config contains the application key which is just a reference to Application. Wikisearch tries to access application.pipelines["extractor"] but that's not defined yet since it is in the process of being defined on the line where the error is thrown.
Perhaps the Wikisearch pipeline code is not longer in sync with the latest txtai API?
Any help is appreciated.
Thanks!
Hythem

Hello, thank you for the detailed write-up.
You are correct, there is an issue with how the Wikisearch pipeline is being created. I just checked in a change that fixes the issue. If you install from source, you'll be able to use txtchat as you intend.
pip install git+https://github.com/neuml/txtchat


SL90 and Infinity lights BLE Command Structure

Hi @keefo - not an issue, more of a curiosity - someone asked about the SL90 light on the Issue tracker of my NeewerLite-Python program, so I got one and ran a Bluetooth packet logging session on it.  I was surprised to see the same basic command the app sends is the same as the base vanilla Neewer light command, but with a lot of extra information prepended to it.  I also analyzed the Swift file for this project and saw a description of this new command under getSceneCommand(), but this (at the moment) only seems to work with the Scene tab (I tried NeewerLite with the SL90 and it worked, but only on the Scene tab).  Do you have any information on how this new command set is laid out other than the note listed under getSceneCommand()?  I understand the setSCESubTag and setSCEDataTag values, and the MAC address, but in the other (RGB/CCT) modes, those values differ from the 0x8B and 0x91 values that are programmed into NeewerLite.

This is my research note on new command
https://github.com/keefo/NeewerLite/blob/main/Docs/research.md#cb60-rgb-command-reverse-engineering
The newer model of their light seems follow this pattern.

Ah, excellent - thank you!  I did run the packet analyzer, but didn't check into the APK yet... will definitely do that when I get a chance to.  I also noticed when analyzing the stream coming from the light, there tend to be around a half-dozen calls to and from the new series of lights when they're first pairing to the iOS Neewer app - probably querying what the current mode/HSL/RGB settings are, which is another upgrade from the old scheme they used!


Incorporate specific volume on data dictionaries of lower-dimensional subdomains

Typically, this is done through the specific volume, which can be obtained from the typical length of the subdomain (i.e., the aperture). Ask Ivar for help on this one.

This now has been solved.


Update to match quirks of unified diff format

Based on the description here:
https://www.artima.com/weblogs/viewpost.jsp?thread=164293
which matches the behavior of the diff/patch tools.
For all of the tests created or modified, behavior was verified
to match that of diff/patch.
Fixes #296

Let me know if there are any other changes or tests needed.  Thanks!

I'll take this if you drop the package.json changes.

@kpdecker done!

Thank you, and far faster response than I have here :)


bug(modal): if backdrop === 'static' outside click should not close modal

Provided I'm doing this correctly, I've set the modal's config to:
<div bimodal #myModal="bs-modal" class="modal fade" [config]="{ keyboard: false, backdrop: false }" aria-labelledby="myLargeModalLabel" aria-hidden="true"> ... </div>
However this does not prevent the modal from closing when clicking outside of the modal.

oh, thanks! my bad, I forgot to add check here:
https://github.com/valor-software/ng2-bootstrap/blob/development/components/modal/modal.component.ts#L79
it should be
@HostListener('click', ['$event'])
  protected onClick(event:any):void {
    if (this.config.backdrop === 'static' && event.target === this.element.nativeElement) {
      this.hide(event);
    }
  }

and options values should be added to docs (from original bs docs)
backdrop	boolean or the string 'static'	true	Includes a modal-backdrop element. Alternatively, specify static for a backdrop which doesn't close the modal on click.
Nice reason to make PR :)

Thanks!
Sent from my iPhone

On Jun 2, 2016, at 6:29 PM, Dmitriy Shekhovtsov<EMAIL_ADDRESS>wrote:
oh, thanks! my bad, I forgot to add check here:
https://github.com/valor-software/ng2-bootstrap/blob/development/components/modal/modal.component.ts#L79
it should be
@HostListener('click', ['$event'])
protected onClick(event:any):void {
if (this.config.backdrop === 'static' && event.target === this.element.nativeElement) {
this.hide(event);
}
}
and options values should be added to docs (from original bs docs)
backdrop boolean or the string 'static' true Includes a modal-backdrop element. Alternatively, specify static for a backdrop which doesn't close the modal on click.
Nice reason to make PR :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Since this is an easy fix, based on the effort tag, is it possible to expedite it?

Also, the code is supposed to be:
this.config.backdrop !== 'static'
If it is static, it shouldn't hide the modal when clicked.

Hi,
What's the solution for this bug ?

@calvinKG - Bug has been fixed, update you version to get the fix.

hi,
ran into the exact same problem, trying to download the recent version but unsuccesfully.
when i used 'npm install ng2-bootstrap --save' the ng2-bootstarp version in the package.json remained 1.0.17.
when i downloaded ng2-bootstarp-develpoment.zip and replaced the files manually (either those of the modal libarary or the entire folder) i got complation errors.
in both cases the result remained the same - the model was closed when clicking on the background even when i set 'data-backdrop="static"'
can you please help me with that
cheers,
eRez

@eRez-ebs - So I am also using 1.0.17.  So just passing data-backdrop won't work since that only works for the bootstrap version.  To get it to work with this library you have to pass it via the config input like so:
[config]="{'backdrop':'static', 'keyboard': false}"

@mpetkov thanks for the quick reply.
your solution resolved the ESC issue, which no longer closes the moda, but sadly not the background - clicking on it still closes the modal :-(

@mpetkov am also exprinceing the same problem.
"ng2-bootstrap": "^1.0.17", thats the version that am using.

i found out why it didn't work for me.
i've looked into 'node_modules\ng2-bootstrap\components\modal' folder localy, where there's a 'modal.component.d.ts' file and 'modal.component.js' file.
the d.ts file has only declerations in it, but indside the js file i found this:
/** Host element manipulations */
// @HostBinding(`class.${ClassName.IN}`) private _addClassIn:boolean;
ModalDirective.prototype.onClick = function (event) {
        if (event.target === this.element.nativeElement) {
            this.hide(event);
        }
    };
    // todo: consider preventing default and stopping propagation
    ModalDirective.prototype.onEsc = function () {
        if (this.config.keyboard) {
            this.hide();
        }
    };

doesn't look much like the file here - https://github.com/valor-software/ng2-bootstrap/blob/development/components/modal/modal.component.ts#L79
anyhow, it explains why the onClick fails but the onEsc works.
i simply added to the condition of the onClick this.config.backdrop !== 'static' and it worked (meaning the modal wasn't closed when the background was clicked).
i just can't figure out why after i update bootstrap library i still get an oudated version.
eRez

How are we going to implement this updates ?

[config]="{backdrop: 'static'} works for me in 1.1.5


Error on Blogger import

/usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importers/blogger.rb:258:in `post_data_from_in_entry_elem_info': Original URL is missing (RuntimeError)
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importers/blogger.rb:190:in `tag_end'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/rexml-3.2.6/lib/rexml/parsers/streamparser.rb:36:in `parse'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importers/blogger.rb:47:in `block in process'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importers/blogger.rb:45:in `open'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importers/blogger.rb:45:in `process'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import/importer.rb:25:in `run'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-import-0.22.0/lib/jekyll-import.rb:30:in `block (3 levels) in add_importer_commands'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `block in execute'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `each'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `execute'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/mercenary-0.4.0/lib/mercenary/program.rb:44:in `go'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/mercenary-0.4.0/lib/mercenary.rb:21:in `program'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/gems/jekyll-4.3.2/exe/jekyll:15:in `<top (required)>'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/bin/jekyll:25:in `load'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/bin/jekyll:25:in `<main>'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/bin/ruby_executable_hooks:22:in `eval'
        from /usr/share/rvm/gems/ruby-3.2.2@mnyon.com/bin/ruby_executable_hooks:22:in `<main>'

Needed to add jekyll-import


Ruby kumite is broken: uninitialized constant Test

Well, I didn't really do much, just trying to run Ruby with the default kumite template:
# TODO: Replace examples and use TDD development by writing your own tests
# These are some of the methods available:
#   Test.expect(boolean, [optional] message)
#   Test.assert_equals(actual, expected, [optional] message)
#   Test.assert_not_equals(actual, expected, [optional] message)

describe "Solution" do
  it "should test for something" do
    Test.assert_equals("actual", "expected", "This is just an example of how you can write your own TDD tests")
  end
end

and it gives uninitialized constant Test. Is it something related to the new runner?

Do you have Codewars selected for test framework? Having RSpec will throw that error.

Okay, so I managed to figure out what that happened: I select shell before selected Ruby, and so the framework choice goes like:
Codewars (JS) -> Ruby/Rspec (shell) -> RSpec 3 (Ruby)
But of course, if you don't choose shell in the process it goes like
Codewars (JS) -> Codewars (Ruby)
I guess this is some kind of issue?

I guess. Consistent default will be better. But I'm going to close this one because the original issue is invalid. Feel free to open a new one for that.


GitHub Actions: Transitioning from Node 16 to Node 20

https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/

I consider this change like a breaking change and so this will be in the v2 and the v1 will be set as deprecated.
This change will enable us to use more recent semantic-release versions which have been impossible since the node16 wasn't enough anymore. Since this action act as a wrapper for some use cases around the awesome semantic-release-action. We will wait the support of node20.

https://github.com/cycjimmy/semantic-release-action/releases/tag/v4.0.0 support Node 20

I change my mind about this issue's breaking change part. I will make the move to node 20 in the v1 branch.


JWT and Auth0 authentication failure

So, I've followed the last part of this tutorial right to finest details and implemented it accordingly, and when I create a post request signed with JWT that was made out of {   "alg": "HS256",   "typ": "JWT" }
{   "iss": "https://{mycompany'sname}.auth0.com/authorize" //don't worry I did insert my actual company name }
and of course signed it with the secret key that I've been given and performing this operation mentioned in the tutorial curl -i -X POST http://localhost:8001/consumers/{consumer}/jwt \     -F "algorithm=RS256" \     -F<EMAIL_ADDRESS>\     -F "key=https://{COMPAYNAME}.auth0.com/" # the iss field
I get an error saying No credentials found for given \'iss\' I'm trying to authorize a backend service to talk to another backend service

@Harry97 you may want to try asking Gitter or the mailing list to see if anyone in the community has any feedback here, as GitHub issues should be reserved for actual bugs with Kong. See https://github.com/Mashape/kong/blob/master/CONTRIBUTING.md#where-to-seek-for-help. Thanks!

I apologize, this was a problem with my configuration of Auth0 and has nothing to do with Kong and this should be kept for bugs reporting only, sorry for the inconvenience.


[Enhancement] Add tests

Summary
Some of this code can be really easily tested, such as the value converters. We should add some testing to also include in our build/release pipelines.
Intended Use Case
To improve code quality and help avoid changes breaking other things they shouldn't be breaking.

Adding blocker label to this. I would like to see as much tests as humanly possible for all the things we have merged now before we go live with this.


Register on code.dlang.org

@MartinNowak do you want to register drepl on dub, s.t. ideally it's just:
dub run drepl

I guess currently the biggest problem is to bootstrap / install the C library? Is there a nice way to handle that? (see also https://github.com/dlang/dub/issues/852)

At best replacing it with a D library, at worst shipping object files with drepl.

It works w/ a system wide installation of liblinenoise available on most distributions.
So we just need to document that and maybe detect missing libs in the dub file (or can dub already do that)?

https://code.dlang.org/packages/drepl


HDFS-15927. Catch polymorphic type by reference

Need to catch polymorphic exception types by
reference in order to realize the polymorphic
usage. Otherwise, the functionality of the
caught object is restricted to only that of
the base class.

This PR fixes the following warnings reported in https://github.com/apache/hadoop/pull/2792 in CI run - https://github.com/apache/hadoop/pull/2792#issuecomment-808709573
[WARNING] /home/jenkins/jenkins-agent/workspace/hadoop-multibranch_PR-2792/src/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/lib/common/hdfs_configuration.cc:138:22: warning: catching polymorphic type 'const class hdfs::uri_parse_error' by value [-Wcatch-value=]
[WARNING] /home/jenkins/jenkins-agent/workspace/hadoop-multibranch_PR-2792/src/hadoop-hdfs-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/lib/common/hdfs_configuration.cc:151:27: warning: catching polymorphic type 'struct hdfs::ha_parse_error' by value [-Wcatch-value=]

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 22s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m 41s

trunk passed

+1 :green_heart:
compile
2m 54s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
2m 51s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 27s

trunk passed

+1 :green_heart:
shadedclient
57m 46s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 15s

the patch passed

+1 :green_heart:
compile
2m 42s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

-1 :x:
cc
2m 42s
/results-compile-cc-hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 generated 3 new + 55 unchanged - 5 fixed = 58 total (was 60)

+1 :green_heart:
golang
2m 42s

the patch passed

+1 :green_heart:
javac
2m 42s

the patch passed

+1 :green_heart:
compile
2m 48s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

-1 :x:
cc
2m 48s
/results-compile-cc-hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08.txt
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 generated 8 new + 50 unchanged - 10 fixed = 58 total (was 60)

+1 :green_heart:
golang
2m 48s

the patch passed

+1 :green_heart:
javac
2m 48s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 15s

the patch passed

+1 :green_heart:
shadedclient
15m 52s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
92m 45s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

176m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2824

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux 6188b69dedf5 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 447b75b83e4353e56ffd55489dd08816aaa01399

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/1/testReport/

Max. process+thread count
595 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 38s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 31s

trunk passed

+1 :green_heart:
compile
2m 40s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
2m 43s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 29s

trunk passed

+1 :green_heart:
shadedclient
51m 40s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 18s

the patch passed

+1 :green_heart:
compile
2m 31s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

-1 :x:
cc
2m 31s
/results-compile-cc-hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 generated 2 new + 48 unchanged - 4 fixed = 50 total (was 52)

+1 :green_heart:
golang
2m 31s

the patch passed

+1 :green_heart:
javac
2m 31s

the patch passed

+1 :green_heart:
compile
2m 31s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

-1 :x:
cc
2m 31s
/results-compile-cc-hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08.txt
hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 generated 6 new + 44 unchanged - 8 fixed = 50 total (was 52)

+1 :green_heart:
golang
2m 31s

the patch passed

+1 :green_heart:
javac
2m 31s

the patch passed

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 18s

the patch passed

+1 :green_heart:
shadedclient
13m 19s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
35m 11s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 36s

The patch does not generate ASF License warnings.

109m 25s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2824

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux ccde94a4c725 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0879d0b79dfe918d3076c8c485279d9c79009bc1

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/2/testReport/

Max. process+thread count
611 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 36s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 34s

trunk passed

+1 :green_heart:
compile
2m 39s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
2m 38s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
shadedclient
51m 38s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 16s

the patch passed

+1 :green_heart:
compile
2m 29s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
cc
2m 29s

hadoop-hdfs-project_hadoop-hdfs-native-client-jdkUbuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 generated 0 new + 44 unchanged - 2 fixed = 44 total (was 46)

+1 :green_heart:
golang
2m 29s

the patch passed

+1 :green_heart:
javac
2m 29s

the patch passed

+1 :green_heart:
compile
2m 30s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
cc
2m 30s

hadoop-hdfs-project_hadoop-hdfs-native-client-jdkPrivateBuild-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 generated 0 new + 44 unchanged - 2 fixed = 44 total (was 46)

+1 :green_heart:
golang
2m 30s

the patch passed

+1 :green_heart:
javac
2m 30s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 18s

the patch passed

+1 :green_heart:
shadedclient
13m  8s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
35m  1s

hadoop-hdfs-native-client in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

108m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2824

Optional Tests
dupname asflicense compile cc mvnsite javac unit codespell golang

uname
Linux 012cc96289c9 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0b9284d1c3b1e85a54d403493dcd933b222beb8f

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/3/testReport/

Max. process+thread count
541 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-native-client U: hadoop-hdfs-project/hadoop-hdfs-native-client

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2824/3/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


How to make goldmine_snippet_count100k_topk1_temp2.jsonl?

Hello,
I want to run python datasets/conala/dataset.py --pretrain path/to/conala-mined.jsonl --topk 100000 --include_api apidocs/processed/distsmpl/snippet_15k/goldmine_snippet_count100k_topk1_temp2.jsonl
However, goldmine_snippet_count100k_topk1_temp2.jsonl is missing (but there is some txt files in snippet_15k directory).
How to make this jsonl file?
I think, retrieve.py with --method dist makes the jsonl file, but get_distribution() does not make jsonl files.
Thank you.

Sorry, it was a mistake.

Sorry, it was a mistake.


Commands/features missing from documentation

The whois command is missing from the commands documentation.
https://github.com/SurPathHub/Dayong/blob/3abcf95101f61c2ce7a819bd26024729ed17e5ad/dayong/components/privilege_component.py#L45

Also, please document event listeners/handlers. They should be in a table, in a separate file called events.md, inside the docs directory. The table should have a name and a description column. In the name column is the event listener's function name, and in the description column, the first paragraph of the function docstring.
An event listener has this decorator:
https://github.com/SurPathHub/Dayong/blob/3abcf95101f61c2ce7a819bd26024729ed17e5ad/dayong/components/event_component.py#L17

Hey,
I can document the command and listeners, can you assign this issue to me?

@lokerhp go ahead!

Can you also assign me this issue?

@aryabagde I'm sorry, but this issue really isn't big enough to warrant multiple assignees —it wouldn't make sense. We'll be posting more issues later, which you can look forward to, or maybe look for one yourself. Cheers!


Desktop(Windows, macOS, Linux) Support

Are there any plans to support the Windows version?

hi @friebetill , we currently don't have a plan to support Desktop apps unless there are more requests from the community. Thanks for bringing it up.

hi, we need windows support

Just one more vote for Desktop support... I've got an app that is targeting iOS, Android and desktop, but unless I've got an common analytics provider that works for all of them, then it likely means choosing another provider.
In the meantime, do you have a recommended workaround? Anyone know if https://github.com/koa-health/mixpanel_analytics works and is reliable?

We need Windows and Mac support for Mixpanel Flutter.

Windows support would be really good, I gave up using analytics for Firebase because it doesn't have Windows support. I wish I didn't have to do the same with Mixpanel. 🥺

Just one more vote for Desktop support... I've got an app that is targeting iOS, Android and desktop, but unless I've got an common analytics provider that works for all of them, then it likely means choosing another provider.
In the meantime, do you have a recommended workaround? Anyone know if https://github.com/koa-health/mixpanel_analytics works and is reliable?

@edwardaux have you tried  https://github.com/koa-health/mixpanel_analytics ?  would you recommend using it?

@marconemagnus Yep, I ended up using that package. My analytics requirements are pretty modest at the moment, but it has been great so far.

+1 for Desktop (Windos & MacOs) Support

Thanks for all the feedback. We will consider adding Desktop Support soon and will give you the update in terms of the timeliine.

Hi @zihejia , any chance you can give us a timeline on this?
I'm using a third party lib that works like a wrapper around Mixpanel's API, but it doesn't give me the default capabilities that your SDK does. It does work as a temporary workaround though.
thank you I appreciate you attention.

Hi @zihejia ,
I would like to know if there is any update on this? Many of us want to use Mixpanel on desktop apps

Hi @zihejia,
Any update on the desktop support timeline?

We will consider adding Desktop Support soon and will give you the update in terms of the timeliine.

Hey @zihejia, any updates on the timeline? I think the community would very much appreciate this feature.

I'm also interested in desktop support, especially as Flutter was made for supporting all of the major application paradigms, mobile, web and desktop, so this would be useful.

macOS support for Flutter is really needed. I'm about to release a mac app and am now not sure what I'll do re: analytics going forward.

Windows support desired here. The 3rd party wrapper is fine but doesn't cover automatic events. Having to roll the current automatic events into manually tracked events would be a bunch of work and would remove the usefulness of the flutter mobile sdk for organizations forced to do it. Maybe a the desktop projects could be created and stubbed out to show some progress towards desktop support in flutter?

Yes, please add desktop support. This would give you a strong competitive advantage

Any update for desktop support ?

Support for desktop would be much appreciated. Any updates on timeline?

+1

Vote +1 for Windows support. Appreciated!

There is very minimal work required to get a MacOS version out. I'd say that is the bare minimum when you already have an iOS app.

I gave up, no longer using Mixpanel because of this.

Same here, you should check out https://www.june.so/
There is a flutter package for that: https://pub.dev/packages/june_flutter

Was switching to MixPanel assuming this was taken care of. It really is a requirement for an analytics tool to support all the platforms.

Would love to see support for Mac and windows

Me too Me too

+1 for macOS support

still there's no macOS support? at least you guys can give us a workaround if we want to support it.
I really need macOS support!

+1 for macOS support

+1 for macOS and Windows support

Any updates on this? @friebetill

Any updates on this? @zihejia

+1 for Windows support


How can we control which testNG test class we want to run?

We are using AWS device farms using appium. The biggest problem we are facing is we are not able to pick which test class we want to run. It basically runs all the test classes. Is there anyway we can control that?

Hi, you can control the test to be executed by creating a testng.xml file(The same xml file that you run your testng locally), and put the testng.xml file to src/test/resources/testng.xml  Device Farm will detect this testng.xml file and run tests that are specified in this file.

Here is a pull request which explain how to do that
https://github.com/aws-samples/aws-device-farm-appium-tests-for-sample-app/pull/14


Security improvements

Tasks

[x] https://github.com/EasyDesk/easydesk-clean-architecture/issues/64
[x] Setting Allowed-Hosts
[x] Static code analysis (ex. CodeQL, Sonarcloud)

One problem would be supported language versions. A solution could be running during periods of versions alignment between compiler and analisys tool
Another problem: security mode vs. security+quality mode

References

https://learn.microsoft.com/en-us/aspnet/core/security/?view=aspnetcore-7.0
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/best-practices?source=recommendations&view=aspnetcore-7.0
https://learn.microsoft.com/en-us/aspnet/core/mvc/models/file-uploads?view=aspnetcore-7.0

Done with 0ffa0aff50d0f2572272feea8ac481ea57dfa998


Azure KeyVault secretstore BulkGet also retrieve deleted secrets

Expected Behavior
Azure KeyVault secretstore BulkGet does not list deleted secrets.
Actual Behavior
Deleted secrets are not excluded.
Steps to Reproduce the Problem
Use Azure KeyVault secret store BulkSet
Release Note

RELEASE NOTE: FIX Exclude delete secrets in AzureKeyVault secret store BulkGet.

cc @cmendible @yaron2

Hmmm that's weird.
Since there's no deleted attribute in the response<EMAIL_ADDRESS>we can check for secretResp.Attributes.Enabled == true, but not sure it will work as a fix.
Also cause there is a GetDeletedSecrets (https://docs.microsoft.com/en-us/rest/api/keyvault/getdeletedsecrets) in the API I would expect GetSecrets (https://docs.microsoft.com/en-us/rest/api/keyvault/getsecrets) to return only available secrets.

I just tested the GetSecrets API with a key vault containing an enabled and a deleted secret and the API did not return the deleted secrets.
But I did find that if at least one secret is disabled  then the Bulk Secret API fails.
I'll submit a PR to fix this behavior.


Implicit typing inside a foreach statement

I noticed something in the sample code at https://docs.microsoft.com/en-us/dotnet/csharp/tutorials/intro-to-csharp/list-collection?tutorial-step=1
The example is using the var keyword inside the foreach statement. But the C# coding conventions here claim that implicit typing shouldn't be used for foreach loops.
This means that the following code is wrong
foreach (var name in names)
{
  Console.WriteLine($"Hello {name.ToUpper()}!");
}

and should be replaced with
foreach (string name in names)
{
  Console.WriteLine($"Hello {name.ToUpper()}!");
}

I'm not sure if this is considered an error since it's about coding style, but I thought that perhaps it will be worth pointing out.

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

ID: 67ff1559-a7c6-9c97-e543-298fbf63f57f
Version Independent ID: ec7a656b-a94e-7d88-5162-32e931f2e094
Content: Data collections - Introduction to C# interactive tutorial
Content Source: docs/csharp/tutorials/intro-to-csharp/list-collection.yml
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for writing this @nathaniades
I've added it to our backlog for the next time we update this article.


Cannot run the mod on 1.8.9

Whenever I try and launch minecraft 1.8.9 with the mod in my mods folder, I get the error in the image. However I have downloaded the latest version available for 1.8.9

update your SkyblockAddon to 1.6.0

But where can I do this as if I download the latest version on curseforge for 1.8.9, I get that error


Sync changes from current version WebSkype

Added ability to retrieve history msg greater than 2 month ago.

Looks good, thanks!


'The name WebApplication does not exist in the current context' when swapping to ASP.Net 5 RC2

When you make an ASP.Net 5 application in Visual Studio you get this line at the end of your Startup.cs file

// Entry point for the application.
public static void Main(string[] args) => WebApplication.Run<Startup>(args);

(The class is Microsoft.AspNet.Hosting.WebApplication.)
If you replace rc1-final with * in the project.json file and thus pick up the latest RC2 the project no-longer builds, giving the error "The name WebApplication does not exist in the current context".
Looking at the 1.0.0.RC2 breaking changes there is one titled Hosting renaming WebApplication to WebHost. I cannot see how to take account of this change in the boiler-plate code. What line should replace the line of code above when swapping from ASP.Net 5 RC1 to RC2?
(I've posted this to Stack Overflow too.)

See the updated templates: https://github.com/aspnet/Templates/blob/dev/src/BaseTemplates/EmptyWeb/Program.cs

@JunTaoLuo can you make sure the announcements have the appropriate updates?

@dumbledad In addition to the announcement you mentioned, there is also https://github.com/aspnet/Announcements/issues/157 and https://github.com/aspnet/Announcements/issues/163.
@davidfowl Each announcement illustrate the usage before and after the change. It might be difficult to keep track of the changes required for upgrading an older sample. The best way to do that is probably to look at the templates @Tratcher linked above.


Packaging request: owl-compositor

Project description
Owl is a portable Wayland compositor written in Objective-C, using Cocoa as its backend. Owl makes it possible to run Wayland clients inside OS X's native Quartz graphics environment.
Metadata

homepage URL: https://github.com/owl-compositor/owl
source URL: https://github.com/owl-compositor/owl
license: gpl3Plus
platforms: unix

An initial version is available via the following command:
nix build github:wegank/nur-packages#owl-compositor

To achieve this, I packaged epoll-shim, then ported Wayland to macOS via a handmade patch.
An interesting idea would be to revive #122475, which is reverted by #123686.


Help Wanted

If you like using this project and want to help add features and fix bugs, please feel free to claim an issue and provide a pull request.
I will review and merge valid pull requests and cut releases as needed... I'm just short on development time right now.

This project is going into maintenance mode.


Remove XML re-parsing from Player Abnormalities

XML for Player Abnormalities is parsed only once at the beginning to make it faster and simpler.
I also cleaned up attributes and made some of them optional:

IsDebuff defaults to False
IsGear defaults to False
IsInfinite  defaults to False
HasConditions defaults to False
ConditionOffset defaults to 0
Stack defaults to 0
IconName defaults to ICON_MISSING

As discussed, the style is changed.
We'll probably do something similar with monster data.
I haven't changed memory reading logic, but it can be made faster by reading whole block of player data at once and using it later, to reduce the number of ReadProcessMemory calls and allocations.

Thanks!


[otbn,crypto] Create .bss section for OTBN and use for RSA

This PR allows OTBN programs to specify data in a .bss section instead of a .data section in order to indicate that the data does not have to be initialized; only the size and symbol locations will be included in the resulting binary, not the code in between.
This PR also uses the new .bss section on the RSA-3072 code used by mask ROM sigverify (mask ROM is sensitive to code and those buffers are a lot of space to spend on zeroes; that was the original motivation for this change!) The size of the .elf file for sigverify's OTBN application (run_rsa_verify_3072_rr_modexp) went from 11036 to 6488 bytes as a result of this change. That's honestly a bit of a surprisingly large reduction to me (I'd expect it to save ~1.5k bytes, not 4.5k), but the tests are passing. If anyone could educate me on that one, I'd be curious to know! :slightly_smiling_face:
I also checked locally that the OTBN standalone tests still work; there's no issue with .weak symbols in .bss being overridden by symbols in .data that do need to be initialized.

Happy to see that this works! This is much cleaner than the allocs in #9786.

The size of the .elf file for sigverify's OTBN application (run_rsa_verify_3072_rr_modexp) went from 11036 to 6488 bytes as a result of this change. That's honestly a bit of a surprisingly large reduction to me (I'd expect it to save ~1.5k bytes, not 4.5k), but the tests are passing. If anyone could educate me on that one, I'd be curious to know! 🙂

I took a quick look and run_rsa_verify_3072_rr_modexp is built using run_rsa_verify_3072_rr_modexp.s, rsa_verify_3072_rr.s, and rsa_verify_3072.s and all of these files define buffers in the .data section. I didn't add up all the bytes but this may help explain the difference in size that you are seeing.

I took a quick look and run_rsa_verify_3072_rr_modexp is built using run_rsa_verify_3072_rr_modexp.s, rsa_verify_3072_rr.s, and rsa_verify_3072.s and all of these files define buffers in the .data section. I didn't add up all the bytes but this may help explain the difference in size that you are seeing.

They define buffers, but all except the ones in run_rsa_verify_3072_rr_modexp.s are declared .weak, so I'd expect them to be overridden by the ones in that file and never defined -- if this didn't happen, I'd expect to get an error because it would exceed available non-scratchpad DMEM space!

I'm not very familiar with otbn build process but it's not immediately clear to me why .bss does not end up in the final binary since it is assigned to the same memory region as .data (>dmem AT>dmem_load). otbn_build.py explicitly removes .scratchpad but I couldn't find any references to .bss. Also, should this section be NOLOAD?

My understanding here is that the .bss section does end up in the binary, in a way, but it's correctly marked as the BSS section of the ELF file and only the size of it is stored, not the actual data. I think it works without NOLOAD here becasue .bss is considered special and handled automatically; if I change .bss to .foo everywhere, it appears to get dropped from the binary entirely (presumably I'd have to do some further adjustments).
Here are some printouts from riscv32-unknown-elf-size -- they show exactly what I'd expect, 1600 bytes of data moving from the "data" to the "bss" segment of the ELF file. (Note that the "data" and "bss" in this printout are not derived from the definitions in otbn.ld, they're just the kinds of segments all ELFs have; if I make the .data section NOLOAD, it actually ends up under bss here.) Here's the printout from master:
jadep@jadep-l:~/opentitan$ /tools/riscv/bin/riscv32-unknown-elf-size -d ~/opentitan/build-out/sw/otbn/run_rsa_verify_3072_rr_modexp.elf
   text	   data	    bss	    dec	    hex	filename
   1084	   1600	      0	   2684	    a7c	/home/jadep/opentitan/build-out/sw/otbn/run_rsa_verify_3072_rr_modexp.elf

...and here's this PR:
jadep@jadep-l:~/opentitan$ /tools/riscv/bin/riscv32-unknown-elf-size -d ~/opentitan/build-out/sw/otbn/run_rsa_verify_3072_rr_modexp.elf
   text	   data	    bss	    dec	    hex	filename
   1084	      0	   1600	   2684	    a7c	/home/jadep/opentitan/build-out/sw/otbn/run_rsa_verify_3072_rr_modexp.elf

I realized upon closer inspection that I wasn't actually making the .rv32embed.o file smaller, because one of the arguments to riscv32-unknown-elf-objcopy in otbn-build.py sets all sections to load (and removes scratchpad, so it doesn't get overridden there). I've updated the PR so that these arguments now set only the .data section to load, to avoid overriding the default noload for BSS.

To get a handle on what's going on, try linking the Ibex binary with -Map, which should show which input section is ending up where.

Thanks for the tip to look at the map.  It looks like the build automatically produces some map files, such as build-out/sw/device/silicon_creator/mask_rom/mask_rom_fpga_cw310.map, so I was able to inspect those for the final mask ROM binary as well. Unfortunately, this PR reduces the size of the .rv32embed.o and .elf files produced from the OTBN code, but currently does not reduce the size of the final mask ROM binary.
The reason is that sections called .bss* are handled specially by riscv-unknown-elf-ld, which knows to put these in the BSS section of output ELF files. The util/otbn_build.py script prefixes the .bss section so it becomes .rodata.otbn.bss, which doesn't get recognized as a BSS section later in the mask ROM linker script (which matches everything with .rodata* and puts it in a loaded .rodata section):
https://github.com/lowRISC/opentitan/blob/a510f8e1f1ae7efd0f6924d376cea58295f2ce3b/sw/device/silicon_creator/mask_rom/mask_rom.ld#L108-L119
To solve this, we'll need to either change the prefix or change the rules in the mask ROM linker script. My suggestion is to change the renaming in otbn_build.py so that it continues to rename .data. and .text to .rodata.otbn.data and .rodata.otbn.text, but renames .bss to .bss.otbn.bss (since it's not really "read-only data" anway). This will match the existing .bss* rule in the mask ROM linker script and place it in the BSS section. WDYT?

A couple observations after looking at this some more:

The loader at sw/device/lib/runtime/otbn.c isn't involved for mask ROM; it uses separate code at sw/device/silicon_creator/lib/drivers/otbn.c, which has a otbn_zero_dmem function; looks like the current code calls it when an app is initialized, then copies over the app's DMEM contents based on the _dmem_end symbol. So to avoid copying zeroes, we could change the _dmem_end symbol to _data_end or something; however, this is at runtime, so I'm not sure it affects binary size.
I was looking at the size of the mask ROM elf file, mask_rom_fpga_cw310.elf, when I said the current version doesn't change the size; however, this code does reduce the size of mask_rom_fpga_cw310.bin by exactly the expected 1600 bytes. @alphan , which do we care about? (Not certain why the .elf size isn't changing; riscv32-unknown-elf-size shows 1600 bytes moving from text to bss, as expected. But if we care about the .bin, it may not be worth debugging.)

One local change I've made doesn't reduce the size of the mask ROM elf file, mask_rom_fpga_cw310.elf; however, this code does reduce the size of mask_rom_fpga_cw310.bin by exactly the expected 1600 bytes. @alphan , which do we care about? (Not certain why the .elf size isn't changing; riscv32-unknown-elf-size shows 1600 bytes moving from text to bss, as expected. But if we care about the .bin, it may not be worth debugging.)

Ultimately we care about the bin file, so it's good that its size decreases. You mentioned that 1600 bytes move from text to bss in the elf file. How does the file size and mem size of these change? @mundaym can you help with @jadephilipoom's question?

I've worked on this most of today and I think I'm on the path to a working solution, although I haven't finished it yet. Quick summary: Just renaming the sections isn't enough; the OTBN drivers (for both mask ROM and runtime) try to copy over all the data between the _dmem_start and _dmem_end symbols. Because the data and bss and sections will no longer be continuous when the mask rom linker script is applied, these drivers need to be updated to copy only between the start/end of the loaded data section, and also to allow DMEM writes to both the data and bss sections.

Should be working now, I've finished fixing up the loaders. stat tells me that mask_rom_fpga_cw310.bin is 1568 bytes smaller as a result of this change, which I'm guessing is the 1600 bytes that moves to BSS plus some overhead for having extra sections and symbols.

Ok, I think I've understood what's going on and my "can't you just do..." comments ain't gonna work :-) So I think we should go with this approach for now: I'll have a last pass through the PR and then approve.
BUT: It's still a bit silly. In particular, we're allocating a big slab of zeros in the Ibex bss section that we'll never actually use. That doesn't cost us anything in the binary, but it does cost data memory on the host CPU. I've done a bit of thinking, and I think I've got a plan for how to do this more nicely. Basically, we run objcopy twice (with some judicious renamings). The first run generates something like what we've got already, but drops both .bss and .scratchpad. The second run uses the --extract-symbol argument to generate an ELF with just symbols, giving OTBN-side addresses. Once we link everything together, we can copy the data and text segments across as before. But we drop the otbn_ptr_to_dmem_addr function entirely, doing DMEM address translation statically by just using the two symbols. I'll have a play locally and see if I can get everything working: I think it should be a bit simpler than Philipp's (admittedly clever!) approach that we've been using up to now.

Right, I've put my master plan into action! See #10105.


Fix trace_context

related to https://github.com/membraneframework/membrane_rtc_engine/pull/76

Solved in https://github.com/membraneframework/membrane_rtc_engine/pull/76


[Bugs] Can't pass any test, upf error, illigal option: f

I was following the passes described in the official site (https://www.free5gc.org/installations/stage-3-free5gc-install/) but I can't pass any test.
When I run any test, for example TestRegistration, I get the following outputs:
./bin/free5gc-upfd: invalid option -- 'f'
NAME:
upf - 5G User Plane Function (UPF)
USAGE:
main [global options] command [command options] [arguments...]
COMMANDS:
help, h  Shows a list of commands or help for one command
GLOBAL OPTIONS:
-c FILE  Load configuration from FILE
-l FILE  Output NF log to FILE
-g FILE  Output free5gc log to FILE
-h       show help
2022-02-18T18:13:06Z [ERRO][UPF][Util] Illigal option: f
2022-02-18T18:13:06Z [ERRO][UPF][Util] Error parsing args
...
And then the test stay stuck on Remove Request Transaction [1]
Others error I can retrieve, before the Remove Request Transaction [1] are:
2022-02-18T18:13:58Z [ERRO][SMF][CTX] In ActivateUpLinkTunnel UPF IP:  <IP_ADDRESS>
2022-02-18T18:13:58Z [ERRO][SMF][CTX] Allocate PDR Error:  this upf do not associate with smf
To Reproduce it, just follow the steps as described on official site (https://www.free5gc.org/installations/stage-3-free5gc-install/)
I know Stage 3 is not the actual, but the steps seems to be coherent with documentation here on github.
Environment (please complete the following information):

free5GC Version: v3.0.6
OS: Ubuntu 20.0.04
Kernel version: 5.4.0-100-generic
go version: go version go1.15.8 linux/amd64
gtp:
gtp5g                 118784  0
udp_tunnel         16384   1 gtp5g

Hello，I meet this problem yesterday，and now I have solved it through vim test.sh and modify it code cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -f to cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -c. It's modify its param f to c. Then you can see PASS.

Hello. Sorry, maybe I misunderstood. I enter command "NFs/upf/build && ${EXEC_UPFNS}" in the "free5gc" folder. But I can't understand how modify from "./bin/free5gc-upfd -f" -> "./bin/free5gc-upfd -c". Could you tell me which commands to use and where, please?

Hello，I meet this problem yesterday，and now I have solved it through vim test.sh and modify it code cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -f to cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -c. It's modify its param f to c. Then you can see PASS.

Hello，I meet this problem yesterday，and now I have solved it through vim test.sh and modify it code cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -f to cd NFs/upf/build && ${EXEC_UPFNS} ./bin/free5gc-upfd -c. It's modify its param f to c. Then you can see PASS.

I modified tesh.sh file through nano, is it ok? Because after modification I didn't receive PASS and still receiving FAIL, but now for another purpose(two lines) - "
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
"

Hi all,
Please get the latest release v3.0.7 and try again:
git clone --recursive -b v3.0.7 -j `nproc` https://github.com/free5gc/free5gc.git

Hello. Yeah, it fixed the problem with ./test.sh TestRegistration and all other tests also PASS. Thank you so much:)
But I have FAIL only for this last test - ./test_ulcl.sh -om 3 TestRegistration. Is this test important to PASS? Could you tell me what this test -  ./test_ulcl.sh -om 3 TestRegistration?
Output of the - ./test_ulcl.sh -om 3 TestRegistration
FAIL    test    602.088s
free5gc-upfd: no process found
ls: cannot access '*sslkey.log': No such file or directory
87 packets captured
87 packets received by filter
0 packets dropped by kernel
71 packets captured
71 packets received by filter
0 packets dropped by kernel
64 packets captured
64 packets received by filter
0 packets dropped by kernel

@TulebaevTemirlan
It is fixed on v3.1.0.
Please get v3.1.0 and try again:
Usage: ./test_ulcl.sh [ TestRequestTwoPDUSessions ]


workspace folder error

When adding a workspace folder, and that folder contains files which are only readable by an extension.  How can I turn off the cannot resolve workspace folder error ?

What does "only readable by an extension" mean? Is this a file permission thing?


UX improvement of project parameter administration

Closes #30
For now it's only on the "environment project". Once we agree on the technic, I will extend it to the "page of the project".

@phacks I noticed that in the data row, data width sum to 100% more 5% for the delete button. In e responsive point of view, I think it's better to fix button width and let input grow as they want.

Also, would it possible to align the Close icon in both row types?

Thanks!

Looks neat, thanks!
It’s not directly related to your PR so we may treat it in a separate issue, but I’ve noticed that all the available configurations are in a selected (or hover?) state even though I’m not interacting with them:

Do you know where can it come from?

On pages I corrected a bug
When you start writing the URL, after typing the first letter, the focus move back to name input.
To catch modification of the state isAddin, I used the solution given in the documentation
https://reactjs.org/docs/hooks-faq.html#how-to-get-the-previous-props-or-state
The bug exists also in "project environement" but it's not a problem since there is only one text input

@phacks I resolved the selector problem in PR #48


gh-97 Fix Docs aren't deployed to artifactory

Update Maven Plugin Versions

Resolves spring-cloud/spring-cloud-dataflow-server-kubernetes#97

I don't think we have this problem with spring-cloud-dataflow-server-kubernetes as I can see the docs [here] (http://repo.spring.io/libs-snapshot-local/org/springframework/cloud/spring-cloud-dataflow-server-kubernetes-docs/1.0.0.BUILD-SNAPSHOT/)


docs: Update preview for getting started guide for MNG and Fargate

Fixes #N/A
Description

The preview docs should reference the main branch as the is the branch where permission and API changes are pushed
These changes impact the MNG and Fargate steps

How was this change tested?

N/A

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 79.086%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
91.6%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5933

Relevant Lines:
7502

💛  - Coveralls


Fix detection of changed template unit specializations in switch-to-configuration.pl

Description of changes
Currently, template unit specializations were ignored when determining whether a unit has changed.
This means that any instanciated services of template units (e.g<EMAIL_ADDRESS>from base<EMAIL_ADDRESS>would not restart on switch, if the change was to a specialization of a specific instance instead of a change to the base service:
{
  systemd.services."microvm@a-specific-instance" = {
    overrideStrategy = "asDropin";
    # Changes in here were not detected.
  };
}

This fixes adds unit specialization support to the unit parsing logic.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Could you also add respective test cases to switchTest to prevent future regressions?

I've added new test cases which I believe should be correct, but they currently fail because there seem to be some new .slice units that get generated and restarted as-well. I'm not familiar enough with the details of systemd to know whether that is to be expected (and should be added to the assertions) or whether it isn't.
Ignoring the slice units, there might be an (unrelated ?) bug hiding in switch-to-configuration.pl that also causes the new tests to fail. I named all unit instances in the test like<EMAIL_ADDRESS>but the script seems to be hallucinating unit instances named<EMAIL_ADDRESS>and tries to start them. The affected units are exactly those instances that have wantedBy = ["multi-user.target"]. Since those only show up in the list of restarted units as observed by get_active_units(), I believe this is unrelated to my change. Unfortunately this is too deep into the details of systemd for me to be able to figure this out without guidance, so any pointers are highly appreciated.

Failed test output
(finished: must succeed: /nix/store/641lmcaylhy3qydm8682gx0rxg6c44lh-nixos-system-machine-23.05pre-git/specialisation/restart-and-reload-by-activation-script/bin/switch-to-configuration test 2>&1, in 16.32 seconds)
The haystack that will cause the following exception is:
---
stopping the following units: test.service
activating the configuration...
setting up /etc...
setting up tmpfiles
starting the following units: no-restart-service.service, reload-triggers-and-restart-by-as.service, simple-reload-service.service, simple-restart-service.service, simple-service.service<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>the following new units were started: no-restart-service.service, reload-triggers-and-restart-by-as.service, reload-triggers-and-restart.service, reload-triggers.service, simple-reload-service.service, simple-restart-service.service, simple-service.service, system-templated\x2dno\x2drestart\x2dservice.slice, system-templated\x2dreload\x2dtriggers.slice, system-templated\x2dreload\x2dtriggers\x2dand\x2drestart.slice, system-templated\x2dreload\x2dtriggers\x2dand\x2drestart\x2dby\x2das.slice, system-templated\x2dsimple\x2dreload\x2dservice.slice, system-templated\x2dsimple\x2drestart\x2dservice.slice, system-templated\x2dsimple\x2dservice.slice<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
---
Test "restart and reload by activation script" failed with error: "Expected string 'the following new units were started: no-restart-service.service, reload-triggers-and-restart-by-as.service, reload-triggers-and-restart.service, reload-triggers.service, simple-reload-service.service, simple-restart-service.service, simple-service.service<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>' was not found"
cleanup

Formatted version with line breaks and my annotations
# This list is what I would expect. All templated-* units are new and were restarted correctly.
starting the following units:
no-restart-service.service
reload-triggers-and-restart-by-as.service
simple-reload-service.service
simple-restart-service.service
simple-service.service
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
the following new units were started:
no-restart-service.service
reload-triggers-and-restart-by-as.service
reload-triggers-and-restart.service
reload-triggers.service
simple-reload-service.service
simple-restart-service.service
simple-service.service
# Are these .slice files fine? And if so, do we want to assert they are there or are they just "systemd implementation details" that we don't care about?
system-templated\x2dno\x2drestart\x2dservice.slice
system-templated\x2dreload\x2dtriggers.slice
system-templated\x2dreload\x2dtriggers\x2dand\x2drestart.slice
system-templated\x2dreload\x2dtriggers\x2dand\x2drestart\x2dby\x2das.slice
system-templated\x2dsimple\x2dreload\x2dservice.slice
system-templated\x2dsimple\x2drestart\x2dservice.slice
system-templated\x2dsimple\x2dservice.slice
<EMAIL_ADDRESS><EMAIL_ADDRESS># #### HERE ####
# What happened here??
# Although the correct units are in the restart list above, the instance name seems to have been altered (multi-user instead of instance). The corresponding services have `wantedBy=multi-user.service`.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Sorry for the delay, I kind of forgot that this PR was still open. I've now finished the necessary test cases to make sure that this is working as expected.
(The @multi-user issue came from an unfinished test case that was setting wantedBy on the base unit instead of the instance specialization)

@ofborg test switchTest


B-17661 Add MTO Dates for TOO/TIO

Agility Ticket Number E-04636
Summary
As a TOO and TIO I need to see the Scheduled and Actual Pickup Dates and the Scheduled and Actual Delivery
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

How to test

Access the officelocal
Login as a TIO or TOO
Check on move that has a MTO (not PPM).
Verify dates or emdash display in the table.

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 77e14a0019d2ec24b72c945fe8dd8d23a87c07fa