Overreliance on e_os.h

e_os.h is private and quite "heavy-weight" header. It's meant to be used in "exceptional" situations to resolve real system dependencies. And it being private arguably disqualifies it for use in apps/*. In sense that it's more than appropriate if apps* set an example of adhering to exclusively public headers. In other words, an overhaul is due...

I agree.

Especially, I agree that it shouldn't be used in apps/*

I also find it questionable that we send the same C flags to the build of apps/* as for the library.

apps looks like a bit of a nightmare in this regard.  Lots broke when I tried removing e_os.h from apps.h.

Might it make sense to split the functionality in e_os.h into separate include files (in an include/<something> directory) and include them piecemeal?  Where something is neither internal nor openssl.  There is already a define that can be used pre-inclusion of e_os.h to indicate networking is required.  This would be the first natural split.
Splitting would provide a better indication as to what is being asked for and reduce the unnecessary weight.  Many files only want one thing from e_os.h.  e.g. many just wanted OSSL_NELEM and quite a few only require strcasecmp for Windows.

I also find it questionable that we send the same C flags to the build of apps/* as for the library.

I agree.

Yes, I think splitting is a good idea.  openssl/port or something.
If we're going to say apps should not include internals, that's fine.  But we've never really enforced that before.

I don't think that it's a good idea to move the e_os.h stuff to a public header (which is what @richsalz seems to suggest).  Some of it is really application stuff, so could be moved to apps/apps.h, while other parts could be moved to become internal headers to be used by the stuff in crypto/ and in ssl/.
Also, thinking back to earlier team discussions, we might want to consider an internal headers directory for architecture dependent stuff...  include/arch, maybe?

The apps run everywhere that openssl builds.  If the apps cannot use the openssl headers, etc., that we have created so that our code looks and feels portable, then large parts of some internal files will have to be replicated into the apps directory.  I think that is moving backwards.  We never said apps is the model for how to build any portable openssl application.  We have only said it's the best example for how to use the OpenSSL API's.  The fact that it uses internal header files does not change that assertion.

If we're going to say apps should not include internals, that's fine. But we've never really enforced that before.

But we haven't claimed that it has to be that way either. I mean there are things the way they are, but it's not necessarily the way they should or ought to be. If given a choice or opportunity to do something about it, which one would one choose? Make things worse or better?

parts of some internal files will have to be replicated into the apps directory.

Trouble is that it might only look that way. I mean same kind of things might be appropriate in crypto/* and apps/* contexts, but for completely different reasons. It's easiest to explain with an example. On Windows we have a number of defines that add underscore to some POSIX calls, e.g. stat is re-defined as _stat. In crypto/* context it is done in order to ensure that object binaries can be used in either context, shared or static. it is and should be up to application to choose how to link object files. While in apps/* context on the other hand, it's about shutting up compiler warnings, nothing else. So should header that redefines stat as _stat be shared? I'd argue no...

I'm not opposed to making apps be more like an external program.  It's really only worth doing if someone with good windows experience and knowledge wants to take it in.  Yes, looking at you... :)

apps looks like a bit of a nightmare in this regard.  Lots broke when I tried removing e_os.h from apps.h.

Moving this to post-1.1.1 milestone.

See also https://github.com/openssl/openssl/pull/5438 discussion.

Marking as inactive, to be closed at the end of 3.4 dev barring further input


Fix bug with cancelling taking notes, investigate saving attachment links

Report from @snoopyuri

when he tries to paste in a Google Doc link in Notes, it doesn’t take.  He replicated it for me….the second you click back in the notes box or type of note, the link disappears and turns back to the Example link.

In looking at this the first thing that I noticed is that the cancel button also doesn't work.  At first look, https://github.com/studentinsights/studentinsights/blob/master/app/assets/javascripts/student_profile/NotesDetails.js#L77 seems like it's buggy but not sure yet the implications of changing or fixing.

stale


Missing Metrics endpoints

Checks

[X] I've already read https://github.com/actions-runner-controller/actions-runner-controller/blob/master/TROUBLESHOOTING.md and I'm sure my issue is not covered in the troubleshooting guide.

Controller Version
actions-runner-controller-0.21.0
Helm Chart Version
actions-runner-controller-0.21.0       	App Version 0.26.0
CertManager Version
v1.8.2
Deployment Method
Helm
cert-manager installation
Yes Cert-manager is installed within the EKS-Cluster
Checks

[X] This isn't a question or user support case (For Q&A and community support, go to Discussions. It might also be a good idea to contract with any of contributors and maintainers if your business is so critical and therefore you need priority support
[X] I've read releasenotes before submitting this issue and I'm sure it's not due to any recently-introduced backward-incompatible changes
[X] My actions-runner-controller version (v0.x.y) does support the feature
[X] I've already upgraded ARC (including the CRDs, see charts/actions-runner-controller/docs/UPGRADING.md for details) to the latest and it didn't fix the issue

Resource Definitions
## RunnerDeployment

Name:         github-runner-deployment
Namespace:    github-actions-runners-prod01
Labels:       app.kubernetes.io/component=RunnerDeployment
              app.kubernetes.io/instance=github-actions-runner-deployment
              app.kubernetes.io/managed-by=Helm
              app.kubernetes.io/part-of=github-actions-runners
              app.kubernetes.io/version=0.1.0
              helm.sh/chart=github-actions-runners-0.1.0
Annotations:  meta.helm.sh/release-name: github-actions-runner-deployment
              meta.helm.sh/release-namespace: github-actions-runners-prod01
API Version:  actions.summerwind.dev/v1alpha1
Kind:         RunnerDeployment
Metadata:
  Creation Timestamp:  2022-09-28T10:41:22Z
  Generation:          3
  Managed Fields:
    API Version:  actions.summerwind.dev/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:spec:
        f:replicas:
      f:status:
        .:
        f:availableReplicas:
        f:desiredReplicas:
        f:readyReplicas:
        f:replicas:
        f:updatedReplicas:
    Manager:      manager
    Operation:    Update
    Time:         2022-09-28T10:41:53Z
    API Version:  actions.summerwind.dev/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:meta.helm.sh/release-name:
          f:meta.helm.sh/release-namespace:
        f:labels:
          .:
          f:app.kubernetes.io/component:
          f:app.kubernetes.io/instance:
          f:app.kubernetes.io/managed-by:
          f:app.kubernetes.io/part-of:
          f:app.kubernetes.io/version:
          f:helm.sh/chart:
      f:spec:
        .:
        f:template:
          .:
          f:spec:
            .:
            f:image:
            f:imagePullPolicy:
            f:imagePullSecrets:
            f:organization:
    Manager:         terraform-provider-helm_v2.4.1_x5
    Operation:       Update
    Time:            2022-09-28T10:50:33Z
  Resource Version:  49721181
  UID:               b6c34592-eebf-44ee-a032-065e16b40155
Spec:
  Effective Time:  <nil>
  Replicas:        2
  Selector:        <nil>
  Template:
    Metadata:
    Spec:
      Dockerd Container Resources:
      Image:              snowplow/github-actions-runner:0.1.68
      Image Pull Policy:  IfNotPresent
      Image Pull Secrets:
        Name:        snowplow-github-runner-deployment-dockerhub
      Organization:  snowplow-devops
      Resources:
Status:
  Available Replicas:  2
  Desired Replicas:    2
  Ready Replicas:      2
  Replicas:            2
  Updated Replicas:    2
Events:
  Type    Reason                   Age    From                         Message
  ----    ------                   ----   ----                         -------
  Normal  RunnerReplicaSetDeleted  7m29s  runnerdeployment-controller  Deleted runnerreplicaset 'github-runner-deployment-pv8xr'

## HorizontalRunnerAutoscaler

Name:         github-actions-runner-deployment-horizontalrunnerautoscaler
Namespace:    github-actions-runners-prod01
Labels:       app.kubernetes.io/component=HorizontalRunnerAutoscaler
              app.kubernetes.io/instance=github-actions-runner-deployment
              app.kubernetes.io/managed-by=Helm
              app.kubernetes.io/part-of=github-actions-runners
              app.kubernetes.io/version=0.1.0
              helm.sh/chart=github-actions-runners-0.1.0
Annotations:  meta.helm.sh/release-name: github-actions-runner-deployment
              meta.helm.sh/release-namespace: github-actions-runners-prod01
API Version:  actions.summerwind.dev/v1alpha1
Kind:         HorizontalRunnerAutoscaler
Metadata:
  Creation Timestamp:  2022-09-28T10:41:22Z
  Generation:          1
  Managed Fields:
    API Version:  actions.summerwind.dev/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:meta.helm.sh/release-name:
          f:meta.helm.sh/release-namespace:
        f:labels:
          .:
          f:app.kubernetes.io/component:
          f:app.kubernetes.io/instance:
          f:app.kubernetes.io/managed-by:
          f:app.kubernetes.io/part-of:
          f:app.kubernetes.io/version:
          f:helm.sh/chart:
      f:spec:
        .:
        f:maxReplicas:
        f:minReplicas:
        f:scaleDownDelaySecondsAfterScaleOut:
        f:scaleTargetRef:
          .:
          f:name:
        f:scaleUpTriggers:
    Manager:      terraform-provider-helm_v2.4.1_x5
    Operation:    Update
    Time:         2022-09-28T10:41:22Z
    API Version:  actions.summerwind.dev/v1alpha1
    Fields Type:  FieldsV1
    fieldsV1:
      f:status:
        .:
        f:desiredReplicas:
        f:lastSuccessfulScaleOutTime:
    Manager:         manager
    Operation:       Update
    Time:            2022-09-28T10:41:53Z
  Resource Version:  49718237
  UID:               3da2356d-bf77-4dcb-8d33-be98505358b9
Spec:
  Max Replicas:                              10
  Min Replicas:                              2
  Scale Down Delay Seconds After Scale Out:  300
  Scale Target Ref:
    Name:  github-runner-deployment
  Scale Up Triggers:
    Duration:  10m
    Github Event:
      Workflow Job:
Status:
  Desired Replicas:                2
  Last Successful Scale Out Time:  2022-09-28T10:41:53Z
Events:                            <none>

To Reproduce
1. Deploy the Actions-Runner-Controller / Runners using Github App Auth within a Github ORG
2. Scrape metrics with Prometheus
3. Check Metrics

Tried the following
- Hard Delete and reinstall CRDS
- Manual Replace of CRDS
- Destroying and relaunching the complete deployment
- Updating Runner to the latest image

Initially Discussed new metrics here :
https://github.com/actions-runner-controller/actions-runner-controller/discussions/1713

Release Notes :
https://github.com/actions-runner-controller/actions-runner-controller/blob/master/docs/releasenotes/0.26.md#enhancement--more-autoscaling-related-metrics

Describe the bug
Chart has been upgraded but new metrics endpoints are not available.
Describe the expected behavior
Expected to have the additional new metrics within Prometheus that are part of the new chart release
Controller Logs
https://gist.github.com/gthomson31/5b60c49470b5235de78a0a8d3a945841

Runner Pod Logs
https://gist.github.com/gthomson31/5b60c49470b5235de78a0a8d3a945841

Additional Context

@gthomson31 Hey! The new metrics are recorded and emitted only when you use pull-based autoscalers like PercentageRunnresrBusy and TotalNumberOfQueuedAndInProgressWorkflowRuns, but webhook-based autoscaler.
More metrics for webhased-autoscalers sare being added via https://github.com/actions-runner-controller/actions-runner-controller/pull/1814.

Thanks @mumoshu  for the quick response
Will await the future release for building out metrics dashboard 👍🏻


FIX: mypy issues with usermanager extension

fixing mypy errors for usermanager and remove it from pyproject exclude list.
renamed models of usermanager:
Wallets -> Wallet
Users -> User

Codecov Report

Merging #1223 (c90f787) into main (c5fdd35) will decrease coverage by 0.12%.
The diff coverage is 72.97%.

:exclamation: Current head c90f787 differs from pull request most recent head 29869cf. Consider uploading reports for the commit 29869cf to get more accurate results

@@            Coverage Diff             @@
##             main    #1223      +/-   ##
==========================================
- Coverage   49.87%   49.75%   -0.13%
==========================================
  Files         255      255
  Lines       14721    14725       +4
==========================================
- Hits         7342     7326      -16
- Misses       7379     7399      +20

Impacted Files
Coverage Δ

lnbits/extensions/usermanager/crud.py
31.25% <61.53%> (ø)

lnbits/extensions/usermanager/views_api.py
47.61% <75.00%> (-3.23%)
:arrow_down:

lnbits/extensions/usermanager/models.py
96.66% <100.00%> (ø)

lnbits/extensions/usermanager/views.py
88.88% <100.00%> (ø)

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add back description for the Hack language

Feels weird to not have a description between Hack and Getting Started. I think we should add back a meaningful one or two-line description.

Closed by @AndrewDiMola (me)


Teams: Change the deadline for one team is not working

Describe the bug
We extended the deadline to one team in a text exercise. However, the student could not see the button to start the exercise.
To Reproduce

Create a text exercise for teams
In the participation tab, extend the individual deadline to one team
As a team member, verify that you cannot see any button to start the exercise or open the text editor (see the screenshot)

Expected behavior
The student member of the team who was extended the deadline should be able to do the exercise.
Screenshots

Which version of Artemis are you seeing the problem on?
7.0.5
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
No response

This seems to be a problem for individual excersies too. It only happens if the student/team has not participated before the due date but tries to start the exercise after, then the individual deadline is changed.


Add Event / Hook for Plugin Update

This is a feature request and / or enhancement proposal:
I would like to have the possibility for plugins to register to an onPluginsUpdate Event which would allow to perform housekeeping tasks upon plugins update, such as data format migration - see this discussion .
btw., Wordpress (which I used before switch to Grav) has such a hook - I find this very useful and would like to have this also in Grav.

You might consider doing a search before creating a new issue.... Feature: onPluginInstalled and onPluginUpdated events
You should probably close this one.

well, I did a search some time before and found nothing then...
closing this now.


Unresolved/Resolved comment filter does not filter

Even if I disable both, I still see all the comments available :(

Which filter do you mean? Maybe I've forgotten some parts of the extension :)

This button, under the comments tab

Ah yeah that's not something I added... Think that's native to VSCode and a pretty new feature (at least it wasn't there when I built the extension). Will try to add support for it though.

Yeah it was indeed something new, but luckily pretty easy to integrate since the extension already supported it. Should be live in the next version (1.2.42)


Update labels of metric descriptors if they change

This PR includes few minor changes that fixes issues in the prometheus-to-sd component:

Component is not trying to update metric descriptor if this metric was not whitelisted.
Updates metric descriptor if new label was added.
Doesn't send broken metrics to the stackdriver (for example when name is longer than 100 characters or number of labels bigger than 10)
If sending of batch of timeserieses to the Stackdriver fails, correct number of sent timeserieses is logged.

This change is

Closing this PR, because component is moved to different repo. Corresponding PR in new repo:
https://github.com/GoogleCloudPlatform/k8s-stackdriver/pull/5


button shows the first season everytime

When you click on the button, the first season is always displayed, even if you are currently watching another season.
If you then click on the correct season, the current episode will be displayed expanded as it should be

After running a few tests, it seems like the value is being checked correctly, but either not set or dropped somewhere. The first season gets the checkmark, too. I would think if it weren't displaying the value correctly, they would all get marked.


Pricing confusion?

How can the prices for storing data in Azure Data Lake Storage Gen 2 be 50% of the Block Blobs storage prices, if they actually use the same underlying storage mechanism?
Azure Data Lake Gen 2 (Hot): $0.0092 per GB
Block Blob (Hot): $0.0184 per GB

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

ID: b975977b-abaa-8429-c902-30fd470c43ac
Version Independent ID: 289cf357-a6d6-7c00-d282-fe93f66a5c4e
Content: Azure Data Lake Storage Gen2 Preview Introduction
Content Source: articles/storage/data-lake-storage/introduction.md
Service: storage
GitHub Login: @jamesbak
Microsoft Alias: jamesbak

@olavt We are investigating the question for you.

@olavt asa

Hi @olavt - these prices are preview prices, which are always 50% of GA prices. The pricing page will be updated when ADLS Gen2 goes GA.
Cheers,
James

Then I think the page should clearly state that these prices are preview prices.

This issue is outdated as ADLS Gen2 is GA. Requesting that it be closed. Thanks.
#please-close


Option for preflight checks to hide steps

Possible approach for hiding steps based on preflight checks. I'm looking for some sanity checks before I work on the tests.

@davisagli I like it generally. Just some small legibility comments.


Bump version of toolchain and update dependencies

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

@kfichter is this PR still relevant? We should update it & ship it if so

@kfichter is this PR still relevant? We should update it & ship it if so

I believe so, let's merge it

I believe so, let's merge it

We deleted the ovm-toolchain


Create some automated load/performance tests for the backend

I think a good choice for this is the locust framework . Originally started hacking something together with multiprocessing but this is better IMO.


luci-app-clamav: added clamav application

This application was moved from the packages repository to luci.
Signed-off-by: Dan Luedtke<EMAIL_ADDRESS>This  PR depends on: https://github.com/openwrt/packages/pull/4013

Something strange with the commits. First two add the same files?
Please squash these into one commit.

Done. What a strange commit, that's what you get from working with too many terminals open.


Logo use

Hi!
Optuna coder here. Would like to use the Keras logo on our Optuna site for users who are looking for code to use Optuna with Keras, like we do with the Tensorflow and PyTorch logos.
Ok to use the Keras logo?
Thanks,

@Crissman
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


RetroMCP-Java-CLI.jar stopped working

Today I tried to recompile my mod and I noticed that RetroMCP-Java-CLI.jar just stopped working.
It halts after displaying RetroMCP v1.0.1.
$ java -jar RetroMCP-Java-CLI.jar recompile client
Changing language to to en_US
Changing language to to en_US
RetroMCP v1.0.1

My java version: 1.8.0_412 Zulu <IP_ADDRESS>-CA-linux64
UPD: I think this is something with your website. Because I can't open it through "mcphackers.org" or "mcphackers.github.io".

Looks to be an issue on your end. It is working fine on my side, not to mention the fact that the website is hosted through GitHub Pages so the website shouldn't be going down unless GitHub Pages goes down (which is unlikely).
I've tried with the latest commit #[d079ca1](https://github.com/MCPHackers/RetroMCP-Java/commit/d079ca1ef34a21640959e20bf2061125974a9225 and running with and without the recompile client command is working fine.

I tried to open website with VPN, but still nothing.
My github pages work correctly.
RetroMCP-Java-CLI.jar still halts after RetroMCP v1.0.1 message.
I'll try other router.

Yeah, issue with my new router.
I'll try something but is there a way to disable update checks or something that .jar is trying to do?

Since it looks like you built this from source, you can just comment out the checks. Otherwise, there isn't a built in method to do it.
Something to consider for the future, it seems.

Ok, I'll disable it in my fork.

Now website and .jar works. I don't know what with my router caused this issue, but now it's completely solved.


Generate pretty errors for validation errors

Instead of
     Blumquist::Errors::ValidationError:
       [["The property '#/name' of type integer did not match the following type: string in schema 658659b8-41dc-5211-
9cb4-0912a8ab1cc4"],{"$schema":"http://json-schema.org/draft-04/schema#","definitions":{"address":{"type":"object","pr
operties":{"street_address":{"type":"string"},"city":{"type":"string"},"state":{"type":["string","null"]}},"required":
["street_address","city","state"]},"sibling":{"type":"object","properties":{"age_difference":{"type":"number"},"name":
{"type":"string"}},"required":["age_difference"]},"ancestor":{"type":"object","properties":{"family_name":{"type":"str
ing"}},"required":["family_name"]}},"type":"object","properties":{"name":{"type":"string","maxLength":16},"phone_numbe
rs":{"type":"array","items":[{"type":"object","properties":{"prefix":{"type":"number"},"extension":{"type":"number"}}}
]},"parents_address":{"$ref":"#/definitions/address"},"current_address":{"oneOf":[{"type":"null"},{"$ref":"#/definitio
ns/address"},{"type":"object","properties":{"planet":{"type":"string"}},"required":["planet"]}]},"relatives":{"type":"
array","items":{"oneOf":[{"$ref":"#/definitions/sibling"},{"$ref":"#/definitions/ancestor"}]}},"old_addresses":{"type"
:"array","items":[{"$ref":"#/definitions/address"}]}}},{"name":1,"phone_numbers":[{"prefix":555,"extension":1234}],"cu
rrent_address":{"street_address":"Chausseestr. 111","city":"Berlin","state":"Berlin"},"old_addresses":[{"street_addres
s":"Friedrichstr. 58","city":"Berlin","state":"Berlin"},{"street_address":"Mehringdamm 33","city":"Berlin","state":"Be
rlin"},{"street_address":"Bluecherstr. 22","city":"Berlin","state":"Berlin"}]}]

You see:
Blumquist::Errors::ValidationError:
  [
    [
      "The property '#/name' of type integer did not match the following type: string in schema 658659b8-41dc-5211-9cb4-0912a8ab1cc4"
    ],
    {
      "name": 1,
      "phone_numbers": [
        {
          "prefix": 555,
          "extension": 1234
        }
      ],
      "current_address": {
        "street_address": "Chausseestr. 111",
        "city": "Berlin",
        "state": "Berlin"
      },
      "old_addresses": [
        {
          "street_address": "Friedrichstr. 58",
          "city": "Berlin",
          "state": "Berlin"
        },
        {
          "street_address": "Mehringdamm 33",
          "city": "Berlin",
          "state": "Berlin"
        },
        {
          "street_address": "Bluecherstr. 22",
          "city": "Berlin",
          "state": "Berlin"
        }
      ]
    },
    {
      "$schema": "http://json-schema.org/draft-04/schema#",
      "definitions": {
        "address": {
          "type": "object",
          "properties": {
            "street_address": {
              "type": "string"
            },
            "city": {
              "type": "string"
            },
            "state": {
              "type": [
                "string",
                "null"
              ]
            }
          },
          "required": [
            "street_address",
            "city",
            "state"
          ]
        },
        "sibling": {
          "type": "object",
          "properties": {
            "age_difference": {
              "type": "number"
            },
            "name": {
              "type": "string"
            }
          },
          "required": [
            "age_difference"
          ]
        },
        "ancestor": {
          "type": "object",
          "properties": {
            "family_name": {
              "type": "string"
            }
          },
          "required": [
            "family_name"
          ]
        }
      },
      "type": "object",
      "properties": {
        "name": {
          "type": "string",
          "maxLength": 16
        },
        "phone_numbers": {
          "type": "array",
          "items": [
            {
              "type": "object",
              "properties": {
                "prefix": {
                  "type": "number"
                },
                "extension": {
                  "type": "number"
                }
              }
            }
          ]
        },
        "parents_address": {
          "$ref": "#/definitions/address"
        },
        "current_address": {
          "oneOf": [
            {
              "type": "null"
            },
            {
              "$ref": "#/definitions/address"
            },
            {
              "type": "object",
              "properties": {
                "planet": {
                  "type": "string"
                }
              },
              "required": [
                "planet"
              ]
            }
          ]
        },
        "relatives": {
          "type": "array",
          "items": {
            "oneOf": [
              {
                "$ref": "#/definitions/sibling"
              },
              {
                "$ref": "#/definitions/ancestor"
              }
            ]
          }
        },
        "old_addresses": {
          "type": "array",
          "items": [
            {
              "$ref": "#/definitions/address"
            }
          ]
        }
      }
    }
  ]

It is quite long but it's at least readable. Do you think we could remove the schema from the exception? The schema should be available for the user somewhere, but then you can't find the error by just looking at the exception.

Yeah, out with the schema!

Coverage remained the same at 88.108% when pulling 89a9571ee28caa055ee6f9531a1f5883b545603b on prettier-errors into fb3d6501b3e63e2e6322735017577ca25c0dbfda on master.

@jayniz I had to add this because when there were errors in arrays you'd get:
         "The property '#/segments/0' of type object matched more than one of the required schemas. The schema
 specific errors were:\n\n- oneOf #0:\n    - The property '#/segments/0' did not contain a required property o
f 'title'\n- oneOf #3:\n    - The property '#/foo/0' did not contain a required property of 'other prop'\n- The property
 '#/foo/1' did not contain a required property of 'title'\n- oneOf #3:\n    - The property '#/foo/1'
did not contain a required property of 'other prop'\n
...

As the \n are inside a string, those get unscaped when transformed to json.
To avoid that, I transformed the errors from strings to arrays. It looks a bit better:
           [
             "The property '#/foo/0' of type object matched more than one of the required schemas. The sc
hema specific errors were:",
             "",
             "- oneOf #0:",
             "    - The property '#/foo/0' did not contain a required property of 'title'",
             "- oneOf #3:",
             "    - The property '#/foo/0' did not contain a required property of 'other prop'",

At some point I think we could dump all the json errors part.


RIM and SIM are not on the 8080

From a comment "Small thing i noticed, 0x20 RIM and 0x30 SIM were introduced in the 8085 according to http://www.nj7p.org/Manuals/PDFs/Intel/9800301D.pdf

I didn't implement these in the emulator or disassembler.  I removed them from the reference page.


Resolve pytest warnings (maint-1.8)

Two files (tests/test_doctests.py and tests/test_hash.py) are refactored from older testing paradigms into pytests to resolve a few warnings from tests that returned non-None objects.
=============================== warnings summary ===============================
tests/test_doctests.py::test_suite
  $PREFIX/lib/pypy3.8/site-packages/_pytest/python.py:201: PytestReturnNotNoneWarning: Expected None, but tests/test_doctests.py::test_suite returned <unittest.suite.TestSuite tests=[<doctest._DocTestSuite tests=[$SRC_DIR/tests/test_binascii_hex.txt]>]>, which will be an error in a future version of pytest.  Did you mean to use `assert` instead of `return`?
    f"Expected None, but {pyfuncitem.nodeid} returned {result!r}, which will be an error in a "

tests/test_hash.py::test_point
  $PREFIX/lib/pypy3.8/site-packages/_pytest/python.py:201: PytestReturnNotNoneWarning: Expected None, but tests/test_hash.py::test_point returned True, which will be an error in a future version of pytest.  Did you mean to use `assert` instead of `return`?
    f"Expected None, but {pyfuncitem.nodeid} returned {result!r}, which will be an error in a "

tests/test_hash.py::test_multipoint
  $PREFIX/lib/pypy3.8/site-packages/_pytest/python.py:201: PytestReturnNotNoneWarning: Expected None, but tests/test_hash.py::test_multipoint returned True, which will be an error in a future version of pytest.  Did you mean to use `assert` instead of `return`?
    f"Expected None, but {pyfuncitem.nodeid} returned {result!r}, which will be an error in a "

tests/test_hash.py::test_polygon
  $PREFIX/lib/pypy3.8/site-packages/_pytest/python.py:201: PytestReturnNotNoneWarning: Expected None, but tests/test_hash.py::test_polygon returned True, which will be an error in a future version of pytest.  Did you mean to use `assert` instead of `return`?
    f"Expected None, but {pyfuncitem.nodeid} returned {result!r}, which will be an error in a "

tests/test_hash.py::test_collection
  $PREFIX/lib/pypy3.8/site-packages/_pytest/python.py:201: PytestReturnNotNoneWarning: Expected None, but tests/test_hash.py::test_collection returned True, which will be an error in a future version of pytest.  Did you mean to use `assert` instead of `return`?
    f"Expected None, but {pyfuncitem.nodeid} returned {result!r}, which will be an error in a "

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.84%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3019

Relevant Lines:
3517

💛  - Coveralls

Thanks!


Long error message pushes down the next line

It only happens with huge error messages like in vtsls lsp.
https://github.com/user-attachments/assets/4eb272d5-1d74-484b-a273-3f0805fb4d8b

This error also happens too me, but only if the the number of lines in the file is less than the terminal height. For example, if there are 10 lines in a typescript file, the inline error preview pushes the next line to the bottom. If I added code in this file such that it has, let us say, 100 lines, then the same inline error does not push the line to the bottom

See https://github.com/rachartier/tiny-inline-diagnostic.nvim/issues/53#issuecomment-2435255822
I do not know what the ideal solution looks like yet, but I will take a look.

Hi again,
It should be fixed now !

yes, but error signs are still shown in the gutter along the height of an error message

Do you use a plugin with your status column ?

Do you use a plugin with your status column ?

yes, folke/snacks.nvim

I will open an issue on snack.nvim. I'll come back to you when I have more info

Seems fixed now @nowaylifer: https://github.com/folke/snacks.nvim/issues/198
Thanks Folke!

Good job, thanks!


Demo site: images/assets broken

Just started playing with this lovely theme - noticed the live demo is partially broken, e.g. images do not load and give a 404, e.g.
https://cboettig.github.io/hugo-now-ui/assets/img/landing.jpg  or https://cboettig.github.io/hugo-now-ui/assets/img/avatar.jpg
Would look even better and more convincing if you fixed this... ;)

Thanks, fixed now, sorry for the incredibly slow turn-around.

Some/all(?) of the JS/CSS assets are broken currently. The theme looks beautiful on the screenshot, thogh!


now can query urlpath from transform catalogs

urlpath is not an arg for the transform catalogs, but users want to be able to see what they are for the target source. Now the transform is followed through to the target with cat.follow_target() which also sets the urlpath attribute in the target source from the orig source. This is run in mc.setup() and mc.find_availability so that it is always updated. mc.select_date_range() runs cat.update_urlpath to bring the newly discovered urlpath list to the target and makes it discoverable like in the other cases too with source.urlpath.
Pull Request Reminders

[ ] Make sure the notebooks in the docs directory all run.
[ ] Add tests for the new functionality.
[x] Add a bullet to docs/whats_new.rst describing your new work. If not already present, add a new section at the top of the document stating "[expected new version number] (unreleased)", for example: "v0.7.3 (unreleased)"

Example code (this will be in docs soon):
import model_catalogs as mc
import pandas as pd

main_cat = mc.setup()

main_cat['CBOFS']['nowcast'].urlpath  # shows 2 default files

yesterday = pd.Timestamp.today() - pd.Timedelta("1 day")
today = pd.Timestamp.today()
source = mc.select_date_range(main_cat['CBOFS'], timing='nowcast', start_date=yesterday, end_date=today)
source.urlpath  # shows files for the date range selection

@AmyMacFadyen here are the changes we discussed by email. I'm happy with how they turned out!

Codecov Report

Merging #23 (cdd0e3c) into main (4671c16) will decrease coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   72.46%   72.25%   -0.21%
==========================================
  Files           9        9
  Lines         730      739       +9
==========================================
+ Hits          529      534       +5
- Misses        201      205       +4

Flag
Coverage Δ

unittests
72.25% <100.00%> (-0.21%)
:arrow_down:

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

Impacted Files
Coverage Δ

model_catalogs/model_catalogs.py
79.00% <100.00%> (ø)

model_catalogs/process.py
86.25% <100.00%> (+0.33%)
:arrow_up:

model_catalogs/tests/test_catalogs.py
87.28% <0.00%> (-2.55%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I'll cover the other check boxes in a subsequent PR.

@lukecampbell See the code snippet above for how your fetch scripts can be simplified now that I've exposed the urlpath. docs updated and tests added in https://github.com/NOAA-ORR-ERD/model_catalogs/pull/24.


Form fields not marked as touched when form is updated from storage

This causes validation errors to be hidden until invalid fields are focused again. Maybe there is a workaround for this.

Solved in commit c5d54bbcfdb4d0fbb5f98ba6dcc77b9e7e531f75](https://github.com/JDAV-Bayern/fahrtkostenabrechnung/commit/c5d54bbcfdb4d0fbb5f98ba6dcc77b9e7e531f75)


httpx version fix

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 88.798%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
831

Relevant Lines:
906

💛  - Coveralls


⚠ Bofrost.de has degraded performance

In b58d597, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1431 ms

Resolved: Bofrost.de performance has improved in b20d078 after 34 minutes.


Frontend JavaScript is untested

Investigate both unit-test (jest-based) and end-to-end (selenium?) support before the frontend grows much bigger.

From experience with IAR, IMO there's not much value in javascript unit testing. However, I think there's definite value in some kind of end-to-end, multi-browser testing

Moved to #144


Error in the notebooks/Overview.ipynb notebook

Hi,
I got the following error in cell number 3 while exploring the Overview.ipynb notebook in google colab. I used the link  provided in the main README file to open it in colab.
# You can access various attributes of the datasets before downloading them
squad_dataset = list_datasets()[datasets.index('squad')]

pprint(squad_dataset.__dict__)  # It's a simple python dataclass

Error message
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-5-8dc805c4949c> in <module>()
      2 squad_dataset = list_datasets()[datasets.index('squad')]
      3
 ----> 4 pprint(squad_dataset.__dict__)  # It's a simple python dataclass

AttributeError: 'str' object has no attribute '__dict__'

The object squad_dataset is a str not a dataclass .

Do this:
squad_dataset = list_datasets(with_details=True)[datasets.index('squad')]
pprint(squad_dataset.__dict__)  # It's a simple python dataclass

Thanks! This worked. I have created a PR to fix this in the notebook.


blender: fix on darwin

Motivation for this change
Enable blender on darwin
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

@GrahamcOfBorg build openimageio
@GrahamcOfBorg build blender


OpenCC Python 1.1.1 is broken on Ubuntu

The 1.1.0 version works fine, but 1.1.1 version is broken:
docker run -it python:3.8-buster /bin/bash
root@f3e42ddba779:/#  cat /proc/version
Linux version 4.15.0-88-generic (buildd@lgw01-amd64-036) (gcc version 7.4.0 (Ubuntu 7.4.0-1ubuntu1~18.04.1)) #88-Ubuntu SMP Tue Feb 11 20:11:34 UTC 2020

root@f3e42ddba779:/# pip install opencc
...
Successfully installed opencc-1.1.1

root@f3e42ddba779:/# python
Python 3.8.3 (default, May 16 2020, 07:08:28)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import opencc
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/site-packages/opencc/__init__.py", line 44, in <module>
    libopencc = CDLL(_libopenccfile, use_errno=True)
  File "/usr/local/lib/python3.8/ctypes/__init__.py", line 373, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /usr/local/lib/python3.8/site-packages/opencc/clib/lib/libopencc.so.1.1)

same issue found on centos7
[root@sxcmvn~]# python3.6 a.py
Traceback (most recent call last):
File "a.py", line 1, in
import opencc
File "/usr/local/lib64/python3.6/site-packages/opencc/init.py", line 44, in
libopencc = CDLL(_libopenccfile, use_errno=True)
File "/usr/lib64/python3.6/ctypes/init.py", line 343, in init
self._handle = _dlopen(self._name, mode)
OSError: /lib64/libm.so.6: version `GLIBC_2.29' not found (required by /usr/local/lib64/python3.6/site-packages/opencc/clib/lib/libopencc.so.1.1)

@mingruimingrui Can you help solving this problem?

I realized this problem as well.
For the time being, I'd recommend downgrading back to version 1.1.0, which I've tested on most minimal distributions to work.
The alternative is to download this repository and build from source with the setup.py script. (But this would require cmake and a c++ compiler)
In the meanwhile, I'll rebuild the Linux files.
In the long term, it might better to build and upload to PyPI directly with CI/CD so that build environments would be less of an issue.

Thank you for your fix.
1.1.0 now works on centos7
but 1.1.1 not

It's done, pip install OpenCC should now download a patched version of the library for Linux users.
I'm sorry for the problems this caused for everyone.

@nqzhang Could you clear your pip cache and try again?

yes. it works now
(opencc) [root@sxcmvn~]# pip install OpenCC
Collecting OpenCC
Downloading OpenCC-1.1.1.post1-py2.py3-none-manylinux1_x86_64.whl (1.3 MB)
|████████████████████████████████| 1.3 MB 5.9 MB/s
Installing collected packages: OpenCC
Successfully installed OpenCC-1.1.1.post1
(opencc) [root@sxcmvn~]# python a.py
(opencc) [root@sxcmvn ~]#

Thanks! It works for me too.


Add flask section to config file

This allow us unify configuration for all web-services,
and give more flexibility

Coverage increased (+0.3%) to 84.364% when pulling 2b0c69b7c664593cf51c704d314c47348defe8c1 on ccp into 0f16dc14b482201142231152e4a54e72959811dd on master.

Coverage increased (+0.6%) to 84.588% when pulling 5a91aab014936c7e121cda4a595300584b8a05cc on ccp into 0f16dc14b482201142231152e4a54e72959811dd on master.

LGTM, please squash your commits into one and rebase it.


Rework totals

Rework the totals. Make sure the totals also get applied to the prices.

Fixed with previous commit

Fixed with previous commit


property name posits: is underlined and clickable

Anywhere I place the characters posits: VSCode will make it a link.

I couldn't find a mention that posits: would behave in this fashion.

@tommulkins does this happen if you disable all installed extensions. You can start code from a terminal with --disable-extensions

@dbaeumer It must be an extension causing that somewhere.  Thanks!  Now off to chase down which one 🤔


Icon styling

The icons for minimising and refreshing look a bit off on my laptop.

Worth adding user-select: none; here as well

Considering changing from font awesome to svg. This should solve the above issue as I think this is due to hidpi screens. Also means don't have to import the entire font awesome stylesheet


fix: Updated metal-go client for sub-commands Virtual Network

Overriding test results cause it's due to being out of m3.smalls in Dallas.


Make generative art!!! 🌈

Ways you can contribute by making generative art:

Make a parametric art generator. Drawingbots is a good resource. Specifically the tool section has a bunch of examples of parametric line art generators that produce plottable patterns.
Make some parametric art. Make some generative art which could be drawn by a plotter. For some examples of this check out  some these blogs:

https://revdancatt.com/
https://penplotterartwork.com/
https://greweb.me/
https://targz.fr/
https://larswander.com/
https://www.v3ga.net/
https://bylr.info/
http://andymakes.com/#Plotter
https://www.eyesofpanda.com/
https://mattdesl.svbtle.com/pen-plotter-1
https://muffinman.io/art/

Host your code somewhere where it's publicly accessible, like GitHub, and post what you make in the #ship and #development-of-stuff-and-things channels on the Hack Club Slack!

This would be really cool to work on! Let me see if I can spend some time on an editor haha


[FLINK-35944] Add CompiledPlan annotations to BatchExecUnion

What is the purpose of the change

Adds Compiled Plan annotations to BatchExecUnion and BatchExecHashAggregate (which is needed for the UnionTestPrograms).
Tests the new annotations with the existing TestPrograms.

Verifying this change
This change adds a BatchRestoreTest to cover the new annotations and show that the batch compiled plan can be restored and executed correctly.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

@flinkbot run azure

@flinkbot run azure


remove references to exports/ folder

Codecov Report

Merging #14 (e3905d7) into main (d4544f2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          190       190
  Lines         2027      2027
  Branches       225       225
=========================================
  Hits          2027      2027

Continue to review full report at Codecov.

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


(rbenv's fault?) Can't run xcode-version: "xcode-install:23:in `load': cannot load such file -- "

I know I've hit this before, and I don't think it's xcode-version's fault, but… anyone got a fix for this?
% ○ xcode-install --help                                                                                2015-12-0 [09:41:30]
/Users/cbowns/.rbenv/versions/2.2.1/bin/xcode-install:23:in `load': cannot load such file -- /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/xcode-install (LoadError)
	from /Users/cbowns/.rbenv/versions/2.2.1/bin/xcode-install:23:in `<main>'

% ○ ls -l /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/*                                                                 2015-12-0 [09:42:04]
-rwxr-xr-x  1 cbowns  staff  290 Dec  9 09:37 /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/xcversion*
-rwxr-xr-x  1 cbowns  staff  290 Dec  9 09:37 /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/🎉*

I'm not quite sure what's wrong here:
% ○ gem list xcode-install                                                                                2015-12-0 [09:43:21]

*** LOCAL GEMS ***

xcode-install (1.2.2, 1.2.1, 1.1.0, 1.0.1)

cbowns at Atlas in ~ ☆
% ○ gem contents xcode-install                                                                                2015-12-0 [09:43:23]
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/Gemfile
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/LICENSE.txt
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/README.md
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/Rakefile
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/xcversion
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/🎉
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/cleanup.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/cli.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/command.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/install.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/installed.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/list.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/select.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/selected.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/simulators.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/uninstall.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/update.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/lib/xcode/install/version.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/cli_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/curl_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150414.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150427.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150508.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150601.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150608.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150624.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/devcenter/xcode-20150909.html
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/hdiutil.plist
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/xcode.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/xcode_63.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/fixtures/yolo.json
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/install_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/installed_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/json_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/list_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/prerelease_spec.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/spec/spec_helper.rb
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/xcode-install.gemspec

The rbenv side looks right here, but the core mismatch of:
"Cannot load file: /Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/xcode-install"
and the paths of:
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/xcversion*
/Users/cbowns/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.2.2/bin/🎉*

doesn't seem like an rbenv thing.

There's no xcode-install binary anymore, it was deprecated in 1.0 and removed in 1.2. Not sure about the rbenv error, does it simply assume if there any binaries, there has to be one named like the gem?

The error is to be expected -- just invoke xcversion

(It is an rbenv thing -- it maintains binstubs that in turn load the RubyGems binstubs.)

@segiddins but it comes up with binstubs for ones that don't even exist in the actual gem? That seems really broken.

Tbh, rubygems should probably load the last version that has that binstub?

ah, I see now that older versions are installed which did have it — anyway, not an issue with this project :)

Ah, interesting, the command was renamed. Cool, thanks.

I'm pulling together a RubyGems PR to fix this :)

:clap: merci!


Add [Serializable]

[Serializable]
public class RootDialog : IDialog<object>
Exception thrown: 'System.Runtime.Serialization.SerializationException' in mscorlib.dll

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

ID: 5668ed00-c579-8520-c30f-6008ff4bf9d1
Version Independent ID: a1d437ab-faa9-fbfa-d380-d1e4b36fdd8e
Content: Create a bot with the Bot Builder SDK for .NET - Bot Service
Content Source: articles/dotnet/bot-builder-dotnet-quickstart.md
Product: bot-framework
GitHub Login: @RobStand
Microsoft Alias: kamrani

@jeraewyr - you can report this issue on Stack Overflow for our support team to look at.
https://stackoverflow.com/questions/tagged/botframework


(373) Add tier and date options to CRU dashboard

This adds some new search options to the CRU dashboard, allowing filter by Tier and Start / End date
Screenshot

Yeah, exactly. Date pickers are more suited to filtering types of interaction too


[Do NOT merge] Reset latency metrics before starting scalability tests

This is a follow-up on #10216 - should be merged only after that PR is merged.
cc @piosz

To be clear - only the second commit is actually added in this PR.

Apart from one comment LGTM

LGTM

LGTM
This is just a test change, so doesn't require additional reviews.


Feature/importing options

Needs to be merge after #81

I'm going to add  new options for importing the skeleton as simple or complex


Update API docs and add embedded player examples

Update the readme
Move API docs out of the readme to the docs directory
Add embedded player examples for MSE, WebRTC, and HLS

readme preview:
https://github.com/deepch/RTSPtoWeb/blob/258dc6cd775a95d3cdc30a71924fef789db14588/README.md
api docs preview:
https://github.com/deepch/RTSPtoWeb/blob/258dc6cd775a95d3cdc30a71924fef789db14588/docs/api.md

thanks, great job!


[Task]: Define models for various domain objects

In summary, the change comprises adding JsonView annotations on domain/model fields and controller methods and arguments.
View applied on controller methods uses the view for serializing the response.
Similarly, usage on controller method arguments specifies which field to deserialize, which is helpful in case we don't want the client to send some fields for security reasons.
This change replaces the previous use of JsonIgnore for restricting fields in API response and allows for more flexibility. This change will enable us to have fine-grain control over fields serialized for different contexts, such as API response, export, and import.
We have defined the following views for now.
View.Internal - View used to serialize for internal use. It inherits from Views.Public and Views.Export, so it will also include those fields.
Views.Public - View used for API request/response.
Views.Export - View used for making field exportable, like in the case of Git sync and file export.
Also, we can define more views by adding an interface inside the Views class.
A small tutorial on JsonView - Jackson JSON Views | Baeldung

PR: https://github.com/appsmithorg/appsmith/pull/19963

@sidhantgoel Lets close this?

CE PR https://github.com/appsmithorg/appsmith/pull/20086
EE PR https://github.com/appsmithorg/appsmith-ee/pull/1011


Desktop Optimization

The document does not provide official guidance on use of the Virtual Desktop Optimization Tool. https://github.com/The-Virtual-Desktop-Team/Virtual-Desktop-Optimization-Tool

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

ID: cdad1b23-8c75-cd08-41cc-6313fc7016bd
Version Independent ID: f7979a0f-a1be-100f-33b5-01627c45f783
Content: Prepare and customize a master VHD image - Azure
Content Source: articles/virtual-desktop/set-up-customize-master-image.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@Snozzberries
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@Heidilohr
Can you please check and add your comments on this doc update request as applicable.


Adding whitelist changes whitespace detection behavior

Environment:
Python / shell input

Tesseract Version:
Tesseract Open Source OCR Engine v4.1.1-rc2-21-gf4ef with Leptonica
Tesseract Open Source OCR Engine v5.0.0-alpha-635-g90405 with Leptonica

Commit Number:
N/A

Platform:
Linux 4.15.0-88-generic

Current Behavior:
Using test image attached as example and running command:
tesseract -l eng test-image.png test-out
gets output with correct spaces like the below attached:
However, when you add a character whitelist like so:
tesseract -l eng -c tessedit_char_whitelist=ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz test-image.png test-out-2
Tesseract no longer recognizes the whitespace (spaces between words):
Confirmed issue with above versions of tesseract, as well as the stock eng.traineddata and best eng.traineddata files.  However, if you use the combined eng.traineddata and add --oem 0 to the command, whitespace detection works as expected even with a whitelist, but with much worse ocr.  I originally encountered this while using python bindings to access tesseract, and know it also occurs if you request HOCR output instead of plain text.
Expected Behavior:
Whitespace detection should not change when adding character whitelist
Suggested Fix:
(I tried to look through the codebase but had no luck finding a better reference than the following)
Since it works with --oem 0, I suspect there is a change to how blobs are classified as whitespace vs. character in --oem 1, and somehow the whitelist interacts with that decision tree.  Changing that tree so that the tessedit_char_whitelist flag does not affect the tree should correct the issue

test-out-5_00.txt
test-out-5_00-2.txt

Have you tried putting your whitelist in quotes and including a space in it?

I thought I did, but when I double checked it correctly detected spaces.  Looks like this can be closed.  Thanks for the quick response

For anyone else running into this issue, it seems to be a behavior change between 4.0.0 and 4.1.1, which can be triggered by upgrading from buster debian to bullseye and apt install tesseract-ocr. In the changelog for 4.1.0, it lists "Implemented support for whitelist/blacklist in LSTM engine.", so it seems that previously the --tessedit-char-white was a no-op and only including whitespace if requested is correct behavior.


Add support for sharing pid namespace

Upstream k8s merged support for this recently.
We need to:

Share the pid namespace for the containers.
Provide a simple init container to use as pod infra container.

@mrunalp Did you complete this?

Not yet. I'll take this up next.


Support returning IMAP delimiter.

This PR exposes the IMAP delimiter character, which is important for many use cases.

Need tests

Merged to develop for release with 2.7.0.


fix(schema): reorganize statement

Currecntly gemini has wierd wa to attaching values to the statement, it is done in a form of function, which makes no sense.
Let's remove function and use raw values stored in Stmt struct

Please, add more details to the commit messages describing the benefits of this change.

Done


Spotify stock color scheme

Could you please add a color scheme like in stock Spotify?
For example, like this:
[Spotify]
text=D4D4D4
subtext=b3b3b3
main=121212
main-transition=121212
sidebar=000000
player=121212
card=232323
button=1DB954
buttonActive=1ED760
buttonDisabled=535353
notification=4687D6
notificationError=E22134
shadow=000000
selected-row=DEDEDE
tabActive=333333
playbackBar=DEDEDE
misc=535353
heart=1DB954
progress-fg=1DB954
progress-bg=636363
play-button=1DB954
play-button-active=1DB954
radio-btn-active=1DB954
pagelink-active=333333

Or your own version.

can do, can't promise how soon I'm in the middle of rewriting the settings menu so maybe half a week to a week?

OK thanks!

[Spotify]
main-transition=121212
heart=1DB954
progress-fg=1DB954
progress-bg=636363
play-button=1DB954
play-button-active=1DB954
radio-btn-active=1DB954
pagelink-active=333333

probably going to be like this, fill out our custom ones, let spotify handle their own.
Not going to make it the default color scheme but it will be near the top once i get time.

sorry im dumb that wont work if you enable it in the settings menu it'll only work locally or in marketplace, feel free to use it like that in the meantime though

it'll only work locally or in marketplace, feel free to use it like that in the meantime though

Thanks a lot, it's great!


Docker on Windows - Bind storage to host folder

Describe the bug
Binding /var/lib/ocis to any host folder results in missing files.
Files lacking any extension are translated to folders in Windows filesystem.
Steps to reproduce
Steps to reproduce the behavior:

Running Docker on Windows through WSL2.

docker-compose

version: "3.7"
services:
  ocis:
    container_name: owncloud
    image: owncloud/ocis:latest
    restart: unless-stopped
    ports:
      - 9200:9200
    entrypoint:
      - /bin/sh
    command: ["-c", "ocis init || true; ocis server"]
    environment:
      OCIS_URL: "https://domain.com"
      PROXY_TLS: "false"
      OCIS_INSECURE: "false"
      OCIS_LOG_LEVEL: error
    volumes:
      - ./config:/etc/ocis
      - ./data:/var/lib/ocis

Expected behavior
Running without issues even with /var/lib/ocis mounted inside Windows filesystem.
Actual behavior
Binding /var/lib/ocis to any host folder leads ocis into a loop after login.
The same docker compose without mounting /var/lib/ocis to the host filesystem, enables ocis to run without issues.

Hello @micbar
Thank you for replying back so soon.
Been using docker desktop on windows doing bind mount to save databases on SSD and data files in HDD pools for dozen of other containers without issues.
./config:/etc/ocis bind and works just fine.
For /var/lib/ocis the issue comes down to some files without file extension under metadata folder (at least that is where I found the first culprit). There may be more.
It is either a limitation of docker, or a bad translation from file system (linux) to file system (windows). When a file does not have an extension, it creates an empty folder on the host instead of the file.
The sole issue with docker volume is having them saved along side the VM (running WSL2), and not elsewhere that I would perfer (different drive).
Adding a file extension would solve it. Something remotely doable?

@cascaabreu Are you sure that windows bind mounts support extended file attributes? I doubt that, Docker for mac is also not able to do so.
To clarify why this is so important: oCIS doesn't need a database for file metadata and access control. Metadata and grants are stored inside the xattrs of a file. If a filesystem doesn't support xattrs, ocis will never work with the default file backend.
AFAICT Windows NTFS does not support POSIX like extended attributes.
So IMO this cannot work on windows Desktops.
In the future we will write a Driver for SMB filesystems (available on Linux and on Windows Server). SMB supports xattrs and ACLs.

For /var/lib/ocis the issue comes down to some files without file extension under metadata folder (at least that is where I found the first culprit). There may be more.

Yes. That is intended. The ocis DecomposedFS deconstructs the file tree and uses "nodes" and "symlinks" to do efficient lookups.

@micbar thank you for explaning, I'm thrilled that you are planning an alternative driver!


add .editorconfig file

I only hand-picked the important source files for now, this can obviously be expanded upon in the future but should make it easier for future contributors as well a Microsoft since hopefully incoming Pull Requests will use proper formatting afterwards.

What about end_of_line? If it's crlf (IDK), then insert_final_newline = true probably doesn't make sense.

I see it as Best Practice(TM)

Honestly, if we didn't have to sync changes back into the Windows repository I'd say we should go fully unix-line-endings mad and use autocrlf to make up for it. I do, however, approve insert_final_newline regardless. It's just good sense.

@miniksa done!

/azp run


interface disappearing

Hi,
We're running version 1.6.1 on Amazon Linux 2, instance provisioned with static private IPs through cloudformation. Sometimes it got to a state where any of the interface got in a bad state. and ip addr would display the interface but without an assigned ip address. Restarting network would bring it back. I have instances running 1.5.3 and I'm not seeing this issue. I'm not sure if this is already reported. is the tool causing this issue? it feels like IMDSv2 is failing because of token. How is that provided to the tool?
Each interface config file has this entry.
BOOTPROTO=dhcp
from /var/log/messages
May 25 09:41:56 test-server dhclient[2953]: DHCPREQUEST on eth1 to xxx.xxx.xxx.xxx port 67 (xid=0xb7ed6ba) May 25 09:41:56 test-server dhclient[2953]: DHCPACK from xxx.xx.xx.x (xid=0xb7ed6ba) May 25 09:41:56 test-server ec2net: [get_meta] Querying IMDS for meta-data/network/interfaces/macs/xx:xx:xx:xx:xx:xx/local-ipv4s May 25 09:41:56 test-server ec2net: [get_meta] Getting token for IMDSv2. May 25 09:42:26 test-server ec2net: [get_meta] Failed to get IMDSv2 metadata token after 60 attempts... Aborting May 25 09:42:26 test-server ec2net: [rewrite_rules] Could not get IPv4 addresses for eth1 from IMDS. Aborting May 25 09:42:26 test-server ec2net: [get_meta] Querying IMDS for meta-data/network/interfaces/macs/xx:xx:xx:xx:xx:xx/local-ipv4s May 25 09:42:26 test-server ec2net: [get_meta] Getting token for IMDSv2. May 25 09:42:26 test-server dhclient[2953]: bound to xxx.xx.xx.xxx -- renewal in 1588 seconds. May 25 09:42:57 test-server ec2net: [get_meta] Failed to get IMDSv2 metadata token after 60 attempts... Aborting May 25 09:42:57 test-server ec2net: [rewrite_aliases] Failed to get secondary IPs from IMDS. Aborting

I'm not changing anything IMDS. At least not that I'm aware of.

Yeah, I've confirmed that the instance will still renew DHCP leases and maintain its network configuration even if IMDS goes away, which is expected as it's fairly common to disable IMDS access after initial instance setup.

Obviously, if the issue affects eth0 this curl will fail.

So you've seen this behavior on eth0 as well as on secondary interfaces? That's a relevant detail because eth0 is handled slightly differently.
Can you share the full output of ip addr, ip rule, and ip route show table all from an instance that is experiencing this issue?
The logs you originally provided seem to indicate that the DHCP lease is successfully renewed, and nothing indicates that anything is removing routes or address configuration, so we'll need to collect some more data.
You can enable tracing on the dhclient-script by creating the file /etc/dhcp/dhclient-enter-hooks.d/01_debug.sh with content exec 2>> /tmp/dhclient-debug.log ; set -x and setting the execute bit on it (chmod +x /etc/dhcp/dhclient-enter-hooks.d/01_debug.sh). If you can install that and leave it in place on a working instance until a failure occurs, then send me the output of journalctl -t ec2net -t dhclient and the content of /tmp/dhclient-debug.log, that may provide some clues.

@nmeyerhans I have an open case with AWS support. can I upload the logs there? Due to security concerns I can't upload them on here.

@.*** I have an open case with AWS support. can I upload the logs
there? Due to security concerns I can't upload them on here.

I don't have access to AWS support cases.  You can email the logs to me.
I am nmeyerha at amazon.com

This was fixed in a recent update to the dhcp packages on Amazon Linux 2:
[ec2-user@ip-10-0-0-237 ~]$ rpm -q --changelog dhclient | head -n 2
* Wed Aug 03 2022 Dirk Harms-Merbitz<EMAIL_ADDRESS>4.2.5-79.amzn2.1.1
- Backport fix for negative timestep. RedHat RHBA-2020:1087


Potential fix to stoch (maybe)

Hi ,
So i used .corr on pandas_ta stoch with the stoch in trading view. I might be doing it wrong but the correlation for slowk/%k is 0.599302 and slowd/%d is 0.605692 . So i made my own code by referring to the stoch page provided by pandas_ta (https://www.tradingview.com/support/solutions/43000502332-stochastic-stoch/) and it seems i think a potential fix is to change the slow_k in rolling to fast_k
lowest_low   =  low.rolling(slow_k).min()
highest_high = high.rolling(slow_k).max()

Here is my code and i got both 1 correlation for k and d.
data = pd.read_csv("Kc.csv")
high = ta.verify_series(data['high'])
low = ta.verify_series(data['low'])
close = ta.verify_series(data['close'])
lowest_low = low.rolling(14).min()
highest_high = high.rolling(14).max()

nom = (close - lowest_low)
dem = highest_high - lowest_low
results = 100*(nom / dem)
slow_k = ta.sma(results , length = 3).rename("slow_k")
slow_d = ta.sma(slow_k , length = 3).rename("slow_d")
df = pd.concat([slow_k , slow_d] , axis=1)
data.ta.stoch(append = True , fast_k = 14 , slow_k = 3 , slow_d = 3)
dropped = ['time' , 'close' ,'open', 'high', 'low', 'EMA #1' , 'EMA #2' , 'EMA #3', 'EMA #4' , "Upper","Basis","Lower"]
data.drop(columns=dropped , inplace=True)
df_cc = pd.concat([data,df] , axis=1)
print(df_cc.corr())

and here is the result comparing tradingview,pandas_ta and my code.

One thing to note is that, theirs a small difference for the last row of data between my slowk and slowd with tradingsview %k and %d. However, all the other rows are the same.
Column Names according to the dataframe
%K - %D - STOCHFk - STOCHFd - STOCHk - STOCHd - slow_k - slow_d

So as you can see , the last data has a slight difference but if im not mistaken , the others are exactly the same
Thanks

Awesome @SoftDevDanial!
Just needed fresh eyes to solve it. Thanks! If you know where in the code to fix, please fork, edit and make a pull request and I'll review it on the development branch.
Thanks,
KJ


SOC: Incident response policy

Adds an incident response plan policy

Broken link in the target branch at the time the test ran - since fixed via #874

Have pushed an update that I think addresses the feedback. Also squashed some other non-ascii whitespace that had crept in.


⚠ Bofrost.es has degraded performance

In 2becf34, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Bofrost.es performance has improved in ec4658c.


Catch with_items loop task without use of {{ item }}

Issue Type

Feature request

Ansible and Ansible Lint details
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides
ansible-lint 3.4.12

ansible installation method: pip
ansible-lint installation method: pip

Desired Behaviour
Sometimes, it can be a bit of a headscratcher when Ansible reports a directory as not existing when the directory in question has been included in a with_items loop task to create a bunch of directories.  It's only after a fair amount of searching that one notices what should be obvious, that the {{ item }} variable has never actually been referenced in the task.
I'm sure this has happened to others at one point, and I think it would be a good help to have this caught before it causes anguish.

Someone would need to come up with a suitable mechanism for catching this (including loop vars that aren't called item)

Closing as it seems obsolete but if you think the bug is real, just make PR with an xfail marked test that reproduces it.


Element of result error - ios

Hi, i installed the version 3.0.0-beta.4.
I am trying to build the project in ios but it failed with a message:
SwiftMobileScannerPlugin.swift:41:43: Generic parameter 'ElementOfResult' could not be inferred
I was trying flutter clean and pub get and yet same result.
I need this version specific so I can handle the onError page.
There is a fix for this?

Updating OSX and Xcode to the latest version resolved this issue for me.

I am just throwing an educated guess that SwiftMobileScannerPlugin.swift file row 41 let barcodesMap = barcodes!.compactMap { barcode in is missing element for compactMap, that causes this error

Same issue for me! But I cant update my Mac and XCode its old but 13 version... Totally dont know what to do...

All i can say is that you should use the latest version of xcode for this project.


JWT addon

JWT token authentication for PR.
Users need to add this to .travis.yml.
addons:
  jwt:
     secure: <SAUCE_ACCESS_KEY=123456789 encrypted>

The JWT addon will then create a time-limited JWT token and export it to the SAUCE_ACCESS_KEY environment variable.
This is not specific to sauce labs several services may be configured, for instance:
addons:
  jwt:
     saucelabs:
        secure: <SAUCE_ACCESS_KEY=123456789 encrypted>
     moonwalk:
        secure: <MOONWALK_SECRET=123456789 encrypted>

In this case 2 tokens will be generated, one in the SAUCE_ACCESS_KEY env variable, the other in MOONWALK_SECRET.

Could you add documentation on how to use it, and security implications thereof, in the form of a PR to https://github.com/travis-ci/docs-travis-ci-com?

This has been deployed to staging. Please go to https://staging.travis-ci.org/profile, sync data with GitHub and enable a repository to test this.
The staging environment might be refreshed with a different code without prior notice, so if you don't see Initializing JWT in the build log when you have the addon set, do let us know, so that we can get the correct code out.

Thank you @BanzaiMan for moving this along. The JWT addon is not expect to work without the changes submitted as part of the PR in https://github.com/travis-ci/travis-core/pull/418. - I now realize we should have included that in the initial description. Is it possible to get the travis-core changes deployed to staging as well?

@sourishkrout The PR to travis-core has been deployed to staging. Could you try it? (Make sure that you use no critically sensitive data.)

Unfortunately it doesn't look like it worked.
Added jwt addon config to my .travis.yml (https://github.com/sourishkrout/osbdemo/blob/master/.travis.yml) according to this:
Repository slug: |travis-ci/travis-core| sourishkrout/osbdemo
Reading from stdin, press Ctrl+D when done
SAUCE_BLA=12345678
Please add the following to your .travis.yml file:

  secure: "eDQEcNFA0nQzTUj1HmnedM8+vWtaxLehmEgDzf/EEI/Bk6tYcnJMqpA0VMklUDsdmkpxpMRgspdrv1+T4LdYFGMJUOJ+4A7DjkEiTvpKammhgRRYja3VjSpBjx8svxvxwLOHY6l81xRglsdjyWrCx87gUNBr5jFZ5Pd0SJM3LqI="

Which didn't produce the JWT ENV var I expected (SAUCE_BLA=<jwt token>):
> export | grep SAUCE
export SAUCE_USERNAME='sourishkrout'
export npm_lifecycle_script='export | grep SAUCE'
export npm_package_scripts_test='export | grep SAUCE'

@BanzaiMan I wonder how we can debug this. I'm curious to find out if the decrypted JWT addon config from travis-core made it into travis-build at all?
Travis CI Job: https://staging.travis-ci.org/sourishkrout/osbdemo/builds/400203

Hi, @sourishkrout and @sebv,
We are itching to test this further. Could you advise us when we can coordinate our efforts to do so? You can respond here, or email<EMAIL_ADDRESS>Thank you!

@BanzaiMan also rebased this one.

btw  this yml format is correct.

Hi @BanzaiMan, we are really interested in getting this project completed, Any chance you can reach out to me<EMAIL_ADDRESS>so I can work to arrange a live session where we can get everyone to work together to finish this? Thanks!

Sorry for the delay. I'll look at this shortly!

By the way, here's the build from the rebased try on Travis: https://travis-ci.org/travis-ci/travis-build/builds/63575373, which passes. (Note that I made slight changes to the spec syntax, to silence the warnings about old "should" syntax.)
I apologize for not understanding the mechanism behind this encryption scheme enough to offer a better approach. How can we make these specs platform-independent? I see that a good portion of the encrypted data do match up in the beginning. Is that good enough for specs?

@BanzaiMan could you put some debug here: https://github.com/saucelabs/travis-build/blob/jwt/lib/travis/build/addons/jwt.rb#L25 and run the test for mac and linux, so we can see the difference before encoding.
We never ran the test on mac, not sure how to set those up.

I didn't to the digging to figure out why but the iat (issued at) is different for aboves spec run on Mac which will result in different payloads and signed hashes:
{
  "slug": "travis-ci/travis-ci",
  "pull-request": "",
  "iat": 18000
}

vs
{
  "slug": "travis-ci/travis-ci",
  "pull-request": "",
  "iat": 0
}

Looks like timezone issue, should probably initialize with GMT here https://github.com/saucelabs/travis-build/blob/jwt/spec/build/addons/jwt_spec.rb#L9 .

@BanzaiMan could you also please make sure you fully understand and grasp how JWT works and how it can be used with Sauce Labs.

Confirmed that the specs are time (zone) dependent.

I fixed the specs with https://github.com/travis-ci/travis-build/commit/e1e54d0

This (along with https://github.com/travis-ci/travis-core/pull/418) has been deployed to staging. My limited test seems reasonable (works as far as I can tell), and I would like to have a larger set of users test it.
To use this in staging, follow the following steps:

Enable a repository at https://staging.travis-ci.org/profile (You can build against only one of Travis environment, so if you are currently using Travis CI production, i.e., https://travis-ci.org, you need to switch back to it when you are done with the experiment.)

Encrypt the variable against the staging API endpoint:
travis encrypt -r REPO/OWNER -e https://api-staging.travis-ci.org SUPER_SECRET_STUFF

Create a PR against it. When you try dumping SUPER_SECRET_STUFF, you should see JWT-encoded string instead of the plain text value (https://staging.travis-ci.org/BanzaiMan/travis_staging_test/builds/430639#L101-L102):
eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzbHVnIjoiQmFuemFpTWFuL3RyYXZpc19zdGFnaW5nX3Rlc3QiLCJwdWxsLXJlcXVlc3QiOjksImlhdCI6MTQzMjgzNzA4OH0.7GBV_50n04rTwBHzxTpBMFBqWAkojdlsW5yMTnrzQ_Y

The staging environment may be used for other tests, so if you suspect it's not working, please check with me if the correct code is deployed. Thank you!

@sebv @sourishkrout @samccone @santiycr
Could you describe how the user is supposed to decode the payload? We will need documentation on usage when we deploy this to production!
Thank you.

@BanzaiMan I think @sourishkrout already has an example here: https://github.com/sourishkrout/osbdemo , just need to be configured with staging.
The user doesn't need to decode, just use the variable SAUCE_ACCESS_KEY (or else)  in the test script.
The missing doc is probably what thirdparty need to do to validate the key. (This has already been implemented on Sauce side.)

@sebv Ah, OK. I guess I misunderstood the use case.
Yes, if @sourishkrout could reconfigure his demo to use staging, and ensure it works as expected, that would be great.
Please write up a HOWTO on this addon. Thanks.

I'm in the process of re-configuring the sourishkrout/osbdemo to test JWT on staging. I'll report my findings in a bit.

I am in the same boat as @sebv with the sauce key.  Also it looks like Orgs do not work on the staging server, so it is hard for me to test the todoMVC example currently.

@samccone you can probably just fork the repo to test it against staging, merge back the changes later.

I've set up another repo (https://staging.travis-ci.org/sourishkrout/grunt-saucelabs) and hooked it up to the Travis' staging. Good news is that the JWT addon generates valid tokens and passes them into the build job ENV as expected. It's being rejected (HTTP 401) on our REST API now which I will debug and hopefully fix. I'll keep you guys posted.
Great work @BanzaiMan + @sebv. That's a great milestone! I think we're really close.
POST /rest/v1/osbdemo/js-tests HTTP/1.0
X_FORWARDED_PROTO: https
X_FORWARDED_HOST: sebastian.dev.saucelabs.net
X_FORWARDED_FOR: <IP_ADDRESS>
Authorization: Basic
<JWT TOKEN stripped>
Host: resto_clones
Connection: close
accept: application/json
content-type: application/json
content-length: 281

{"platforms":[["Windows 7","chrome",""]],"url":"http://<IP_ADDRESS>:9999/custom/succeeds.html","framework":"custom","build":"430668","public":"team","tags":["v8.6.1","grunt-saucelabs"],"name":"saucelabs-custom:tunnel-test","video-upload-on-pass":false,"tunnel-identifier":"202172686"}HTTP/1.0 401 Unauthorized

Date: Fri, 29 May 2015 23:04:48 GMT
Content-Type: application/json
WWW-Authenticate: Basic realm="Sauce"
Server: monocle/0.35

{"error": "Not authorized"}

@sourishkrout @sebv @samccone How is the document coming along?

Once @sourishkrout has finished his testing, sauce ecoteam need to plan a writedown (cc @jlipps).

@sebv OK. As I indicated earlier, staging env may not have the code deployed at any given time, since we are testing a few things. If you don't see it working there, do get in touch.

@BanzaiMan the JWT addon works as expected. However, I've added a exp property to the token. That'll make sure the tokens will expire eventually and, even if leaked, can't cause harm indefinitely.
I'm still running into an issue with Sauce's implementation on our side. We will have to make additional changes to Sauce Connect to make the end to end integration work. I'm currently working on that.
The JWT addon and the remaining work on Sauce Connect aren't mutually exclusive so I suggest we get the JWT addon merged asap.

@BanzaiMan I see you've merged the code. Does this mean it'll be available in production soon?

@sourishkrout I did? If I had done so, it would have been an accident. I am waiting for the documentation before merging it all in. :-D

@BanzaiMan I think the doc text is ready, we need to move it somewhere, could you give us indication.
See: https://github.com/sebv/jwt-doc/issues/4

We're eagerly awaiting this change to use Travis+Sauce securely for Closure. Is this still moving along? Any idea when it will land?

Hi @BanzaiMan. I represnet the Product Team here at Sauce Labs and we've just released the version of Sauce Connect that provides a full support for JWT. At this point this pull request is the only thing that is left before the process can be considered complete. Please let us know if there is anything else we need to for you to merge it. The documentation can be found here https://github.com/sebv/jwt-doc.

Any updates here? Waiting eagerly for secure SauceLabs pull requests.

hey @joshk just wanted to bump this up. What do we need to do to "synergize" this over the "goal line"? 💼

@BanzaiMan I've rebased both patches so they should now work against master again
Also updated the old tests to the new rspec format
Let me know if there's anything else.

I'll be testing this shortly.

https://staging.travis-ci.org/travis-ci/travis_staging_test/builds/507067#L148 shows the JWT token thus generated and seen in a PR build.

@BanzaiMan should all be addressed now

With the suggested changes (https://github.com/travis-ci/travis-build/commit/e46ff5a6316baef36ea43c081e53d42c9cf18b92), it would look like this: https://staging.travis-ci.org/travis-ci/travis_staging_test/builds/507077#L138

Awesome
I figured out how to reproduce the testcase, and moved things slightly, but that should be everything.
I'll do another pass over of the docs tomorrow

I guess https://github.com/travis-ci/travis-core/pull/418 needs reviewing too

@halkeye travis-core is getting deprecated, so that PR is replaced by other logic. I'll look at the PR and update it for the new logic.

Heh. That's what happens when a pr is over a year old :D
That's great. Let me know if you need anything else.
On Jul 28, 2016 7:22 AM, "Hiro Asari"<EMAIL_ADDRESS>wrote:

@halkeye https://github.com/halkeye travis-core is getting deprecated,
so that PR is replaced by other logic. I'll look at the PR and update it
for the new logic.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/travis-ci/travis-build/pull/356#issuecomment-235909331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGuBzwK9M7PvMnkstT1XOf3jvpW2ojrks5qaLsTgaJpZM4DHeYE
.

For reference, https://github.com/travis-ci/travis-scheduler/pull/29 is the PR.
I ran a test here: https://staging.travis-ci.org/travis-ci/travis_staging_test/builds/507127
It ran the JWT addon: https://github.com/travis-ci/travis_staging_test/blob/ae7a834f9e7b7122f9bdd31df1fe024cc9305a2c/.travis.yml#L5-L7
which encrypted the value: SAUCE_ACCESS_KEY=foobar.
As you can see on https://staging.travis-ci.org/travis-ci/travis_staging_test/builds/507127#L135, the token is:
eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJUcmF2aXMgQ0ksIEdtYkgiLCJzbHVnIjoidHJhdmlzLWNpL3RyYXZpc19zdGFnaW5nX3Rlc3QiLCJwdWxsLXJlcXVlc3QiOiIiLCJleHAiOjE0Njk3MjY1MjQsImlhdCI6MTQ2OTcyMTEyNH0.FDO_9iHOX5a1WcWUul-hshKYeJ2H7TIQFPbuA2mrcnk

Using https://jwt.io/, with the secret foobar, we see that the signature is correct.

I see that travis-ci/travis-scheduler#29 needs more work, to always let jwt decrypt. (Which was a part of the old travis-core PR.) I'll work on this.

Or, maybe no further work is necessary. In this PR build, you can see that the PR receives a valid JWT.

Yea, if foobar was your original secured token, that looks good to me.

All right, then. I'll merge this (and the scheduler PR) and deploy to production shortly. I recommend using test data and credentials while we work out the rough edges.
Thanks for the efforts and patience, everyone!

Once we have good results in production, we'll merge the docs PR and announce it.

@BanzaiMan how will we know when it's in production to start testing?

It's there now.

@BanzaiMan I just put in a support request, but could the abuse detected/gatekeeper stuff have any relation to this PR?
https://travis-ci.org/saucelabs-sample-test-frameworks/Java-TestNG-Selenium/requests

@halkeye That should be lifted now.

@BanzaiMan Okay, looking at this test case.
it looks like the jwt stuff is running after install.
Should the function be changed to before_before_install (is that a thing)?
https://travis-ci.org/saucelabs-sample-test-frameworks/Java-TestNG-Selenium/jobs/148427092

Yes, you can use before_before_install if it is the appropriate place to run the code.

@halkeye Could you open a new issue in https://github.com/travis-ci/travis-ci to track further improvements (except code changes, of course)?

I updated this branch, but I can totally create a new issue if you want

@BanzaiMan new PR or new issue?

should I create a new PR for the change?

It wasn't picking up the new code and retesting, so just opened a new PR


fix: graphql peerDependency

Using valid syntax for multiple peer versions.

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

npm package (@next-major dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add simple smoke tests for identity API

In order to be comfortable regularly deploying the identity APIs we want to be sure that a deployment has been successful. This change adds the framework for (and an initial example of) simple smoke tests to run against deployed services to ensure they are operating as expected.
When run this test will make an authenticated request against the /users/me endpoint and check that a 200 OK is returned.
This change uses a test client application set up in Auth0 to allow the Resource Owner Password Flow.
Remaining to do:

[ ] Provision auth0 client app in terraform
[ ] Hook into buildkite deployment pipeline
[ ] Put test user credentials in secrets manager

For https://github.com/wellcomecollection/identity/issues/133


聊天崩溃

运行以下命令，出现崩溃，该怎么解决？
python3 main.py chat
Doragd > 你在 干嘛

崩溃信息：
Traceback (most recent call last):
  File "main.py", line 38, in <module>
    fire.Fire()
  File "/home/chenjm/.local/lib/python3.8/site-packages/fire/core.py", line 141, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/home/chenjm/.local/lib/python3.8/site-packages/fire/core.py", line 475, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/home/chenjm/.local/lib/python3.8/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "main.py", line 28, in chat
    output_words = train_eval.output_answer(input_sentence, searcher, sos, eos, unknown, opt, word2ix, ix2word)
  File "/home/chenjm/chat-ai/Chinese-Chatbot-PyTorch-Implementation/train_eval.py", line 291, in output_answer
    tokens = generate(input_seq, searcher, sos, eos, opt)
  File "/home/chenjm/chat-ai/Chinese-Chatbot-PyTorch-Implementation/train_eval.py", line 202, in generate
    tokens, scores = searcher(sos, eos, input_batch, input_lengths, opt.max_generate_length, opt.device)
  File "/home/chenjm/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/chenjm/chat-ai/Chinese-Chatbot-PyTorch-Implementation/utils/greedysearch.py", line 17, in forward
    encoder_outputs, encoder_hidden = self.encoder(input_seq, input_length)
  File "/home/chenjm/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/chenjm/chat-ai/Chinese-Chatbot-PyTorch-Implementation/model.py", line 51, in forward
    packed = torch.nn.utils.rnn.pack_padded_sequence(embedded, input_lengths)
  File "/home/chenjm/.local/lib/python3.8/site-packages/torch/nn/utils/rnn.py", line 262, in pack_padded_sequence
    _VF._pack_padded_sequence(input, lengths, batch_first)
RuntimeError: 'lengths' argument should be a 1D CPU int64 tensor, but got 1D cuda:0 Long tensor

去这个文件夹下面找到/home/chenjm/.local/lib/python3.8/site-packages/torch/nn/utils/rnn.py文件，把_VF._pack_padded_sequence(input, lengths, batch_first)改成_VF._pack_padded_sequence(input, lengths.cpu(), batch_first)

要么换torch版本，要么把lengths改成lengths.cpu()

要么换torch版本，要么把lengths改成lengths.cpu()

torch为最新版本，lengths.to(opt.device)改了之后会触发另一个runtime error
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument index in method wrapper_CUDA__index_select)

do you have any solutions to solve the problem?

do you have any solutions to solve the problem?

model.py
Line 51             packed = torch.nn.utils.rnn.pack_padded_sequence(embedded, input_lengths)
just   edited      packed = torch.nn.utils.rnn.pack_padded_sequence(embedded, input_lengths.cpu())

do you have any solutions to solve the problem?

model.py Line 51  packed = torch.nn.utils.rnn.pack_padded_sequence(embedded, input_lengths) just edited  packed = torch.nn.utils.rnn.pack_padded_sequence(embedded, input_lengths.cpu())

Woo, haven't check it but thank you!

我运行main.py之后没有进入对话模式


Survey Back button

Dangerous back button, takes you completely out of the survey / site.
QA test

What does this mean?

Uncertain what this means. Closing.


Show doc for type on the field

Snippet:
/**
 * This is very useful interface
 */
interface MyInterface {
  /**
   * This is very useful property
   */
  prop: string
}

type Obj = {
  my: MyInterface
}

const obj: Obj = {
  my: {

  }
}

I can see docs for prop

But property my on Obj has no docs. As a suggestion as property's type has docs for it maybe it is worth to show this doc if there are no docs for the property itself?

The comments are for the interface, and not the obj. i do not think you want to see the documentation for Array interface, on every property that happens to be an array of something. if you want the documentation to be on my, add a comment on my.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


docs: remove dual-homed region tags

Removes unnecessary region tags now that new ones have proliferated in docs. Added wrapped region tags to align with other same-language Firestore samples.

Region tag violations are resolved in (submitted) cl/376017137, which hasn't quite percolated apparently.

Could I please get someone to refresh the snippet bot? I am expecting these violations to resolve.

Not sure why the units would have failed, but I need the updated status reports so I can see what to fix (if problems are still present).

Could someone please rerun or otherwise expedite the tests?

@Strykrol I added kokoro:force-run and pushed the approve button for Github workflow.

Two things:

I cannot merge this myself, someone else will need to please :)
I'm wrapping up my contract @ Google, but can continue updating this PR if needed

Closing in favor of https://github.com/googleapis/java-firestore/pull/1122


feat(logs): add alloy + loki

#451  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Question: Can we make PS1 async?

It might be a stretch but things would be way smoother.

What are you thinking? Like use an alternate prompt until powerline-rs is done?

I don't know but, it seems that if we could make PS1 somehow be loaded asynchronously the responsiveness of the shell could be improved. It is not an issue with this project though. FYI, I recently stopped showing git info on my powerline because it was not responsive, I wanted it to be faster so now I am using gitmux to show the info on my tmux window. Since it is not part of PS1, my terminal does not hang while powerline is building my PS1.
Thanks for the great work of putting this together!

I've also found the git integration slow with powerline-rs. I've tried updating the git2 dependency to to the latest 0.13.6 but haven't noticed any appreciable difference - https://github.com/jD91mZM2/powerline-rs/blob/master/Cargo.toml#L29. The biggest slowdown is on large complex repos like https://github.com/envoyproxy/envoy

Yes, git2 seems to be incredibly slow at large repos, not sure why :(

Looks like the underlying libgit2 is the issue - https://github.com/libgit2/libgit2/issues?q=is%3Aissue+is%3Aopen+slow

Yes, precisely why I've not been able to fix it

Seems like changes are on their way! https://github.com/libgit2/libgit2/issues/4230 :)

Unfortunately that issue has been open for years but little forward motion.  I've been trying to work out how to pull in the fork that has better performance.
https://crates.io/crates/git2 pulls in https://crates.io/crates/libgit2-sys as a dependency. From that crates repo https://github.com/rust-lang/git2-rs#version-of-libgit2

The source for libgit2 is included in the libgit2-sys crate so there's no need to pre-install the libgit2 library, the libgit2-sys crate will figure that and/or build that for you.

It looks like https://github.com/rust-lang/git2-rs/tree/master/libgit2-sys then pulls in the libgit2 library pinned to https://github.com/libgit2/libgit2/tree/86b85492b1bc8eeb9ff1af25b92877969f17833a
So maybe I should attempt to fork libgit2-sys to point to the higher performance fork at https://github.com/romkatv/libgit2 somehow?

It looks like https://github.com/rust-lang/git2-rs/tree/master/libgit2-sys then pulls in the libgit2 library pinned to https://github.com/libgit2/libgit2/tree/86b85492b1bc8eeb9ff1af25b92877969f17833a
So maybe I should attempt to fork libgit2-sys to point to the higher performance fork at https://github.com/romkatv/libgit2 somehow?

Good work digging through! Let me help you with that last bit: Yes, you could update the submodule for sure. Using [patch] sections in Cargo.toml, you can recursively override libraries of your choice from any binary. But it seems like you can also just install your libgit2 fork on your system, and git2-rs will recognize that it doesn't need to compile the code itself :) source

As a workaround, in zsh I use powerline-rs without git status on PROMPT, and the git status in RPROMPT asynchronously with the following snippet in ~/.zshrc :
# Remove the trailing space in RPROMPT. Used for compat
# with older terminals that would span a newline.
ZLE_RPROMPT_INDENT=0

# Async solution found at https://unix.stackexchange.com/a/645949 :
async_rprompt_callback() {
    RPROMPT="$(<&$1)"
    zle reset-prompt
    zle -F $1
}

async_rprompt() {
    # At first tell the current permissions in rprompt.
    RPROMPT="$(powerline-rs --rtl --shell zsh --modules perms)"

    # Then asynchronously check the git details.
    exec {FD}< <(
        powerline-rs --rtl --shell zsh --modules perms,git,gitstage
    )

    zle -F $FD async_rprompt_callback
}

powerline_prompt() {
    PROMPT="$(powerline-rs --shell zsh --cwd-max-depth 2 --modules ssh,host,cwd,root $?)"
    async_rprompt
    # Let terminals display working dir :
    echo -en "\e]2;${PWD/$HOME/~}\a"
}

if $(which powerline-rs); then
    precmd_functions+=(powerline_prompt)
else
    # call your alternative prompt here
fi

It's a bit verbose but it works like a charm - I was able to use it when compiling the linux kernel over and over (huge git repo).

Is... this going to get an update?

Is... this going to get an update?

jD91mZM2 has passed away https://www.redox-os.org/news/open-source-mental-health/

Oh, wait, I knew about this; I hadn't realized it was the same guy - that was a little while ago...
Ugh, I feel bad :man_facepalming:

I am very sorry to hear about this.
At the same time it's probably the first time I receive news of a human
whose code I contributed to. I read jackpot51's post with great
interest. If they can read me and hoping it helps, in my life this has
been a step forward humanizing my relation to code ... and to you guys.
BTW powerline has continuously been in service on my machine ; I have
no need for any enhancement to it now, but will try to help if need
arises.
Le vendredi 14 octobre 2022 à 05:55 -0700, Artturi a écrit :

Is... this going to get an update?
jD91mZM2 has passed away
https://www.redox-os.org/news/open-source-mental-health/
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID:
@.***>