graphql and onchain storage support

Fixes #1724 .

Changes proposed in this PR:

Adding GraphQL integration
Adding OnChain data integration

Current state:

To do:

[x] merge IPFS / Arw integration
[x] change textarea component
-- [x] install codemirror
-- [x] integrate codemirror into component
-- [x] style codemirror for dark/light + small fixes
-- [x] change layout & validation component
-- [ ] find a way to validate graphql query without schema
-- [x] create headers component
-- [x] integrate headers on graphql?
-- [x] integrate headers on url?
-- [x] uncomment headers to provider calls? #1554
[x] connect both storages to provider
[x] finish validation
[x] add to publish form
[x] add to edit form
[x] fix preview edit form
[x] review tooltip text
[x] fix tab selection when step changes
[x] cleanup
[x] Testing

[x] unit tests
[x] test publish / edit / view
[x] test download (free and fixed) / consume / C2D

Testing User Flow:
Publish (Free and Fixed price):

publish assets with GraphQL queries (test more than 1) without headers
publish assets with GraphQL query (test more than 1) with headers (you can use: https://dgraph.io)
publish assets with SM (smart contract) abi (test more than 1, with different response type)

Asset Details

all assets should be correctly detailed (check specially the file informacion)

Edit Asset:

edit storage type of some of your assets. If you published with GraphQL, than change it any of the other and make sure it's all working as expected

Consume

consume and download all assets

Compute

compute what published previously

@alexcos20 I'm getting this error when testing 'smartcontract':

Not sure why is malformed, according to the example here should work.

hi @bogdanfazakas , can you check please how we're handling the address,  headers, query and abi in asset details in edit for example? When we're in preview and in the asset details page, we're not getting values back to be able to show the correct information. I don't see anything related to this in ocean.js, but can you please take a look?

I'd suggest that this should be merged first into the release-candidate-09-01-2023 branch

Conflicts need to be resolved

This PR also solves: https://github.com/oceanprotocol/market/issues/1844

Hi @alexcos20 do we have an example of a graphql query with headers on our infrastructure?

We need to show an error message when the input fails the validation so the user knows how to fix it. For example, the input below should show a message saying "Invalid json input"

I think there should also be some info somewhere explaining to the user that they only need to input part of the ABI, otherwise it looks like they need to paste the whole ABI

@EnzoVezzaro can you share an example smart contract that you're using to test this? I tried the one that you sent me but it's not working:

@EnzoVezzaro can you share an example smart contract that you're using to test this? I tried the one that you sent me but it's not working:

Mmmm ... 🤔 not sure why you're having this. I'm not having this issue with the example smart contract.

are you on the correct chain? you need to be on mumbai for this to work.

are you on the correct chain? you need to be on mumbai for this to work.

Yeah, actually I was on the wrong network. Smart contract publishing and consume seems to work well, I think we just need to improve the error messages as I said above

are you on the correct chain? you need to be on mumbai for this to work.

Yeah, actually I was on the wrong network. Smart contract publishing and consume seems to work well, I think we just need to improve the error messages as I said above

great 👍🏽 thanks for reviewing
we're only missing the headers for now, I'm debugging it. But I'm not having too much luck on what's wrong with the asset.

@EnzoVezzaro we can use this to have queries with headers as well:
https://pipedream.com/auth/signup?r=/workflows. Through this, you can create a dummy endpoint and print or do whatever you want with the headers. But just to make sure everything is working as expected.

@EnzoVezzaro we can use this to have queries with headers as well: https://pipedream.com/auth/signup?r=/workflows. Through this, you can create a dummy endpoint and print or do whatever you want with the headers. But just to make sure everything is working as expected.

Thanks Ana 👍🏽 great suggestion

You can't change the method right? from get to post

You can't change the method right? from get to post

Yeah, It's not included. I might have forgotten 🙁 but from the typing I can see it's only available on url.

I'll add it 👍🏽

i'll check for graphql, it's probably as you said, just for url and i think we need just get and post , don't think the other ones make any sense

All seems to be working, just add the http method for url and it's good

ready for review @jamiehewitt15 @bogdanfazakas

ready for review @jamiehewitt15 @bogdanfazakas

i ve retested this, the only problem i found seems to be with the arweave assets, was able to publish and edit files with this storage type but the consume is not working. The provider calls are for file info and intialize are failing, and the provider errors are the following:
`

2023-01-25 14:51:33
2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] DEBUG Validating :{'type': 'arweave', 'userdata': None}

2023-01-25 14:51:33
2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.file_types.file_types_factory[14] DEBUG validate_dict passed on {'type': 'arweave', 'userdata': None}

2023-01-25 14:51:33
2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] ERROR malformed service files, missing transactionId.

`

, so not sure if we set something wrong or the problem is not from the market side.

I tested this using the transactions from the realse candidate file, here is an eg of an asset: did:op:2ada75c05cdb1791bc489bbabeebb9a07d4d59f8f30c3bb15f70c838fb5d7a76

ready for review @jamiehewitt15 @bogdanfazakas

i ve retested this, the only problem i found seems to be with the arweave assets, was able to publish and edit files with this storage type but the consume is not working. The provider calls are for file info and intialize are failing, and the provider errors are the following:
2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] DEBUG Validating :{'type': 'arweave', 'userdata': None} 2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.file_types.file_types_factory[14] DEBUG validate_dict passed on {'type': 'arweave', 'userdata': None} 2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] ERROR malformed service files, missing transactionId
so not sure if we set something wrong or the problem is not from the market side.
I tested this using the transactions from the realse candidate file, here is an eg of an asset: did:op:2ada75c05cdb1791bc489bbabeebb9a07d4d59f8f30c3bb15f70c838fb5d7a76

@bogdanfazakas Thank you for reviewing the other Storage Types, can you retry now?

ready for review @jamiehewitt15 @bogdanfazakas

i ve retested this, the only problem i found seems to be with the arweave assets, was able to publish and edit files with this storage type but the consume is not working. The provider calls are for file info and intialize are failing, and the provider errors are the following:
2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] DEBUG Validating :{'type': 'arweave', 'userdata': None} 2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.file_types.file_types_factory[14] DEBUG validate_dict passed on {'type': 'arweave', 'userdata': None} 2023-01-25 14:51:33	2023-01-25 12:51:33 provider-mumbai-58bc97c64b-fng6d ocean_provider.routes.consume[14] ERROR malformed service files, missing transactionId
so not sure if we set something wrong or the problem is not from the market side.
I tested this using the transactions from the realse candidate file, here is an eg of an asset: did:op:2ada75c05cdb1791bc489bbabeebb9a07d4d59f8f30c3bb15f70c838fb5d7a76

@bogdanfazakas Thank you for reviewing the other Storage Types, can you retry now?

Retested everything, the consume works well now for all storage types supported, the only issue encountered was that adding an algorithm to the publisher trusted algorithm, with on chain storage type is not working, other types seem to work well. Not sure if market issue or a validation is failing on a backend service, will update this comment if i discover anything new.

Not sure if market issue or a validation is failing on a backend service, will update this comment if i discover anything new.

Mmmm... 🤔 can you share the link to test it too? maybe I can quickly find the issue 👍🏽

Not sure if market issue or a validation is failing on a backend service, will update this comment if i discover anything new.

Mmmm... 🤔 can you share the link to test it too? maybe I can quickly find the issue 👍🏽

After more debugging it looks to be a checksum difference / error, i was not able to find logs for that when editing compute settings, but i've allowed all algo's on a dataset and tried to initialize for a compute job and that failed with the checksum error.
Not sure if we should keep this issue blocked just for that or handle it in a sepparate issues, we need to compare the cheksum sent by market with the one computed on the  backend side, since it works for all other storage types

Not sure if we should keep this issue blocked just for that or handle it in a separate issues, we need to compare the checksum sent by market with the one computed on the backend side, since it works for all other storage types

@bogdanfazakas I think we should ship this, it's been hanging for a while now. Do we need anything else on the market for this issue (https://github.com/oceanprotocol/provider/issues/595)?

Not sure if we should keep this issue blocked just for that or handle it in a separate issues, we need to compare the checksum sent by market with the one computed on the backend side, since it works for all other storage types

@bogdanfazakas I think we should ship this, it's been hanging for a while now. Do we need anything else on the market for this issue (oceanprotocol/provider#595)?

yep also discussed this with the team today and decided to approve and merge this, and fix the provider issue independently.

I have a small bit of feedback on the visual side - why is the the font styling different for the placeholder text on the example query? It would be better if it has the same styling as all of our other placeholder text

I have a small bit of feedback on the visual side - why is the the font styling different for the placeholder text on the example query? It would be better if it has the same styling as all of our other placeholder text

I'm not sure since this is a field that is replicating a small code editor. It has the correct font for a code editor.


EEL behind a reverse proxy

Hello,
I got eel working on my local network and I would like to acces my app from the internet behind my NGINX reverse proxy. I have this error :

Traceback (most recent call last):
File "C:\Program Files\Python38\lib\site-packages\bottle.py", line 865, in handle
return route.call(**args)
File "C:\Program Files\Python38\lib\site-packages\bottle.py", line 1745, in wrapper
rv = callback(*a, **ka)
File "C:\Program Files\Python38\lib\site-packages\bottle_websocket\plugin.py", line 5, in wrapper
callback(request.environ.get('wsgi.websocket'), *args, **kwargs)
File "C:\Program Files\Python38\lib\site-packages\eel_init.py", line 225, in _websocket
msg = ws.receive()
AttributeError: 'NoneType' object has no attribute 'receive'

I read the bottlepy faq (https://bottlepy.org/docs/0.12/faq.html) and seen this fonction that could possibly resolve my issue :

def fix_environ_middleware(app):
def fixed_app(environ, start_response):
environ['wsgi.url_scheme'] = 'https'
environ['HTTP_X_FORWARDED_HOST'] = 'example.com'
return app(environ, start_response)
return https_app
app = bottle.default_app()
app.wsgi = fix_environ_middleware(app.wsgi)

I don't know how to use it, I heard about a WSGI server that I have to use to do what I want but I don't know if it's possible to do that in docker.
If someone can help me with that...
Regards,
Thomas

Hi @TheToto318, did you manage to solve this?

this is kind of old but, I got the same issue running behind a docker container solve it by running with host='<IP_ADDRESS>' as mentioned here https://bottlepy.org/docs/dev/deployment.html


Fix caffemodel_to_t7.lua

Thanks for providing these scripts! Just a couple small fixes: lua comments, and variable references. Had to clone the network outputs or the tests weren't passing.

This pull request solves the issue of:
ubuntu@ip-Address:~/test_models$ th caffemodel_to_t7.lua
/home/ubuntu/torch/install/bin/luajit: caffemodel_to_t7.lua:8: unfinished string near '"'
stack traceback:
        [C]: in function 'dofile'
        ...untu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
        [C]: at 0x00405d50


Reuse the idGetter when doc repeats

@marcovzla, this is on top of (in addition to) the LazyIdGetter.  If the same doc comes up again, it would be nice to not look it up twice.  There is a little overhead and it might not happen very often, but this is how it might be done.  One goal was not to clutter up that loop in the ExtractorEngine and that seems to have worked.

It seems like the MostRecentlyUsed class relies on the fact that the docs are traversed in order, right?
Maybe we could add a comment explaining that? it took me a while to figure it out

Yes.  I was probably programming in a vacuum.

I added some documentation.  If anyone can think of a better name, please do.  Instead of getOrNew it could be previousOrNew, for example.
It often works better to merge in PR order.  Fewer merge conflicts seem to arise that way.


automating setup of rosa hcp prerequisites

@xinredhat There's one small comment that needs your attention, and I suppose the version now needs to be bumped to 4.14.

@xinredhat There's one small comment that needs your attention, and I suppose the version now needs to be bumped to 4.14.

@Roming22 For the setup of rosa hcp prerequisites, it doesn't care about HCP version. We just need to specify the HCP version while creating HCP cluster.


Handling shadows

Any thoughts on how the shadow can be handled?
I was trying to use the codes, when I did a comparsion.. The figma scrrenshot came with shadow and codes have the shadow outside.. so overall the figma image was slightly smaller than the actual code..

I used a container outside for Figma and code as a wrapper and control the positioning of the component both in figma and code to be same to do exact comparisons.

Glad ya got it workin! Sometimes it's not perfect and you need to do some adjustments like that because the way figma does things isn't also the same


[Swift 4.0] Argument labels '(extendingOrTruncating:)' do not match any available overloads

Version
origin/swift-develop at f2e5b35
Environment
Xcode 9 Beta 5
Running on macOS Sierra 10.12.5 (16F73)
Description
Experimenting with Xcode 9 beta 5, while bootstrapping with Carthage this library failed to build. I cloned the source and tried building manually and it also fails. I see several errors like this:
Argument labels '(extendingOrTruncating:)' do not match any available overloads
Removing the extendingOrTruncating label does allow the build to run but I do not know if these are required for the library to work correctly.
Happy to submit a PR if just removing the label is the correct fix.
Steps To Reproduce

Install Xcode 9 beta 5 (I recommend using xcversion via gem install xcversion).
Clone source
Checkout origin/swift-develop
Build in Xcode

Expected Result
Build to pass
Actual Result
Build failing with error Argument labels '(extendingOrTruncating:)' do not match any available overloads.

The method init(extendingOrTruncating:) has been renamed init(truncatingIfNeeded:) as per proposal SE-104.

Resolved by the merge of #235.


panic: interface conversion: interface {} is bool, not string

What happened?
When passing strings into a gcp:pubsub:Subscription labels, Pulumi yaml panics and crashes to stack trace. Please see the example for much more detail.
Example
# new
name: activtrak-identity-pubsub
runtime: yaml
config:
  vanta_label_contains_user_data:
    type: string
  vanta_label_non_prod:
    type: string

resources:
  identity-change-detection-topic:
    type: gcp:pubsub:Topic
    properties:
      name: identity-change-detection
      project: ${project}
    options:
      protect: true

  identity-change-detection-sub:
    type: gcp:pubsub:Subscription
    properties:
      labels:
        vanta-contains-user-data: ${vanta_label_contains_user_data}
        vanta-description: shared-data-across-services
        vanta-non-prod: ${vanta_label_non_prod}
        vanta-owner: matthew_finlayson
      ackDeadlineSeconds: 10
      expirationPolicy:
        ttl: ""
      messageRetentionDuration: 86400s
      name: identity-change-detection-sub
      project: ${project}
      topic: projects/${project}/topics/identity-change-detection
    options:
      protect: true
      dependsOn:
        - ${identity-change-detection-topic}

  user-topic:
    type: gcp:pubsub:Topic
    properties:
      name: user
      project: ${project}
    options:
      protect: true

  identity-user-creation-sub:
    type: gcp:pubsub:Subscription
    properties:
      labels:
        # vanta-contains-user-data: ${vanta_label_contains_user_data}
        vanta-contains-user-data: "false"
        vanta-description: shared-data-across-services
        # vanta-non-prod: ${vanta_label_non_prod}
        vanta-non-prod: "true"
        vanta-owner: matthew_finlayson
      ackDeadlineSeconds: 10
      expirationPolicy:
        ttl: ""
      messageRetentionDuration: 86400s
      name: identity-user-creation-sub
      project: ${project}
      topic: projects/${project}/topics/user
    options:
      protect: true
      dependsOn:
        - ${user-topic}

config:
  gcp:project: myProject
  activtrak-identity-pubsub:project: myProject
  activtrak-identity-pubsub:vanta_label_contains_user_data: "false"
  activtrak-identity-pubsub:vanta_label_non_prod: "true"

At one point, I was able to get it to work and I got the following output in my plan when I was still trying playing with it.
~ gcp:pubsub/subscription:Subscription: (update) 🔒
    [id=projects/myprojectid/subscriptions/identity-user-creation-sub]
    [urn=urn:pulumi:dev::activtrak-identity-pubsub::gcp:pubsub/subscription:Subscription::identity-user-creation-sub]
  ~ labels: {
      ~ vanta-contains-user-data: "0" => "false"
      ~ vanta-non-prod          : "1" => "true"
    }
~ gcp:pubsub/subscription:Subscription: (update) 🔒
    [id=projects/myprojectid/subscriptions/identity-change-detection-sub]
    [urn=urn:pulumi:dev::activtrak-identity-pubsub::gcp:pubsub/subscription:Subscription::identity-change-detection-sub]
  ~ labels: {
      ~ vanta-contains-user-data: "true" => false
      ~ vanta-non-prod          : "false" => true
    }

What's odd to me is that they are the same resource type, but one is 0 and 1 and "true" and "false", but the other is "true" and "false" and true and false. It's almost as if they are different resource types. They are being passed the same inputs, but they behave differently and store them in the state differently, too.
Unfortunately, I am unsure how I even got it to create because I can't get it to do that again. So I can't be certain what state it was in when I saw this. I'm still playing with it. But this should be a provider bug, no? Should I keep playing with it or submit an issue at this point? What do you tell a customer if it's a bug stopping them?
Here is the stacktrace:
Diagnostics:
  pulumi:pulumi:Stack (activtrak-identity-pubsub-dev):
    panic: interface conversion: interface {} is bool, not string
    goroutine 44 [running]:
    github.com/hashicorp/terraform-provider-google-beta/google-beta/tpgresource.SetLabelsDiff({0x140071766c0?, 0x140071765d0?}, 0x14007144720?, {0x1098f3ee0?, 0x14006cc2a80})
        /home/runner/work/pulumi-gcp/pulumi-gcp/upstream/google-beta/tpgresource/labels.go:84 +0x678
    github.com/hashicorp/terraform-provider-google-beta/google-beta/services/pubsub.ResourcePubsubSubscription.All.func1({0x10997ee68, 0x10d7095e0}, 0x10810228f?, {0x1098f3ee0, 0x14006cc2a80})
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-plugin-sdk/v2@v2.0.0-20230912190043-e6d96b3b8f7e/helper/customdiff/compose.go:53 +0x98
    github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.schemaMap.Diff(0x140011dcf60, {0x10997ee68, 0x10d7095e0}, 0x140071244e0, 0x14006ff2f30, 0x140008e0ec0, {0x1098f3ee0, 0x14006cc2a80}, 0x0)
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-plugin-sdk/v2@v2.0.0-20230912190043-e6d96b3b8f7e/helper/schema/schema.go:698 +0x3b8
    github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).SimpleDiff(0x0?, {0x10997ee68?, 0x10d7095e0?}, 0x140071244e0, 0x140071241a0?, {0x1098f3ee0?, 0x14006cc2a80?})
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-plugin-sdk/v2@v2.0.0-20230912190043-e6d96b3b8f7e/helper/schema/resource.go:967 +0x9c
    github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfshim/sdk-v2.v2Provider.simpleDiff({0x14000d0d560?, {0x1400143ce70?, 0x14006611c70?, 0x1a?}}, 0x0?, 0x8000106?, 0x0?, 0xffffffffffffffff?, {{{0x10997f568?, 0x1400660c9a0?}}, ...}, ...)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v3@v3.66.0/pkg/tfshim/sdk-v2/provider_diff.go:101 +0x808
    github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfshim/sdk-v2.v2Provider.Diff({0x14000d0d560?, {0x0?, 0x0?, 0x0?}}, {0x108147c83, 0x1a}, {0x10997f3a8?, 0x1400421a720}, {0x10994eb20, 0x14006ff2f30})
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v3@v3.66.0/pkg/tfshim/sdk-v2/provider_diff.go:63 +0x1fc
    github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfbridge.(*Provider).Diff(0x1400074b340, {0x10997ed88?, 0x14006ff26f0?}, 0x14006fcd280)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v3@v3.66.0/pkg/tfbridge/provider.go:672 +0x4a4
    github.com/pulumi/pulumi-terraform-bridge/x/muxer.(*muxer).Diff.func1({0x109997a48?, 0x1400074b340?})
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.7-0.20230801203955-5d215c892096/muxer.go:338 +0x3c
    github.com/pulumi/pulumi-terraform-bridge/x/muxer.resourceMethod[...](0x140012d1130?, 0x1400651b778, 0x1400651b758?)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.7-0.20230801203955-5d215c892096/muxer.go:303 +0xc4
    github.com/pulumi/pulumi-terraform-bridge/x/muxer.(*muxer).Diff(0x1400651b7b8?, {0x10997ed88?, 0x14006ff26f0?}, 0x1090087c0?)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.7-0.20230801203955-5d215c892096/muxer.go:337 +0x5c
    github.com/pulumi/pulumi/sdk/v3/proto/go._ResourceProvider_Diff_Handler.func1({0x10997ed88, 0x14006ff26f0}, {0x1096f0e20?, 0x14006fcd280})
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v3@v3.94.2/proto/go/provider_grpc.pb.go:575 +0x74
    github.com/grpc-ecosystem/grpc-opentracing/go/otgrpc.OpenTracingServerInterceptor.func1({0x10997ed88, 0x14006ff2270}, {0x1096f0e20, 0x14006fcd280}, 0x14006fded80, 0x1400421a528)
        /home/runner/go/pkg/mod/github.com/grpc-ecosystem/grpc-opentracing@v0.0.0-20180507213350-8e809c8a8645/go/otgrpc/server.go:57 +0x2e8
    github.com/pulumi/pulumi/sdk/v3/proto/go._ResourceProvider_Diff_Handler({0x1098a7980?, 0x140012d1130}, {0x10997ed88, 0x14006ff2270}, 0x14006fd3dc0, 0x14000ada4c0)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v3@v3.94.2/proto/go/provider_grpc.pb.go:577 +0x12c
    google.golang.org/grpc.(*Server).processUnaryRPC(0x140004dc960, {0x109992480, 0x140059104e0}, 0x14006fecea0, 0x1400640bb60, 0x10d6b64c8, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.58.3/server.go:1374 +0xb44
    google.golang.org/grpc.(*Server).handleStream(0x140004dc960, {0x109992480, 0x140059104e0}, 0x14006fecea0, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.58.3/server.go:1751 +0x7c4
    google.golang.org/grpc.(*Server).serveStreams.func1.1()
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.58.3/server.go:986 +0xb4
    created by google.golang.org/grpc.(*Server).serveStreams.func1 in goroutine 72
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.58.3/server.go:997 +0x160

  gcp:pubsub:Subscription (identity-change-detection-sub):
    error: error reading from server: EOF

Output of pulumi about
CLI
Version      3.99.0
Go Version   go1.21.5
Go Compiler  gc
Plugins
NAME  VERSION
yaml  unknown
Host
OS       darwin
Version  14.2.1
Arch     arm64
This project is written in yaml
Current Stack: GeoffMillerAZ/activtrak-identity-pubsub/dev
TYPE                                  URN
pulumi:pulumi:Stack                   urn:pulumi:dev::activtrak-identity-pubsub::pulumi:pulumi:Stack::activtrak-identity-pubsub-dev
pulumi:providers:gcp                  urn:pulumi:dev::activtrak-identity-pubsub::pulumi:providers:gcp::default
gcp:pubsub/topic:Topic                urn:pulumi:dev::activtrak-identity-pubsub::gcp:pubsub/topic:Topic::user-topic
gcp:pubsub/topic:Topic                urn:pulumi:dev::activtrak-identity-pubsub::gcp:pubsub/topic:Topic::identity-change-detection-topic
gcp:pubsub/subscription:Subscription  urn:pulumi:dev::activtrak-identity-pubsub::gcp:pubsub/subscription:Subscription::identity-change-detection-sub
Found no pending operations associated with dev
Backend
Name           pulumi.com
URL            https://app.pulumi.com/GeoffMillerAZ
User           GeoffMillerAZ
Organizations  GeoffMillerAZ
Token type     personal
No dependencies found
Pulumi locates its logs in /var/folders/39/pjbcs1t54djdg68ptpxw5q880000gn/T/ by default
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

cc @pulumi/providers this is probably a validation issue in gcp/tfbridge, but likely also a yaml bug here around config/property types.

That looks like you've hit https://github.com/pulumi/pulumi-gcp/issues/1377
Did you upgrade your gcp provider version to get past the issue?

Curious also why pulumi about does not report the gcp version?

Curious also why pulumi about does not report the gcp version?

Because pulumi about derives this information from pulumi SDKs specified (requirements.txt, package.json, etc.). YAML doesn't have this.

Logged request here: https://github.com/pulumi/pulumi-yaml/issues/543 thank you!

I updated in the following manner:
pulumi plugin install resource gcp
[resource plugin gcp-7.5.0] installing
Downloading plugin: 67.24 MiB

And it is now working for my test case. Thanks.


Send custom HTTP headers when publishing

Hello,
When publishing to a private repository, it's necessary for us to send custom HTTP headers when doing this. These headers contain, for example, authentication tokens.
I know that Maven uses a ~/.m2/settings.xml file for this where custom httpHeaders can be configured per server/repository. I haven't found an analogous functionality in SBT to do this aside from basic authentication.
Is it possible to do this at the moment with SBT? If not, please consider this a feature request. :+1:

I don't think it's possible currently.
For potential contributors, see code that was changed in https://github.com/sbt/librarymanagement/pull/246 for the relevant code.

@mvdstam Did you find a workaround for this ?
SBT developers, please consider this feature 😃

@gael-ft No, unfortunately I did not find a workaround for this particular issue. 😞
I haven’t found the time either to dive into this myself, so putting this on the agenda for the SBT team would be great.

Ok thank you for your answer @mvdstam, I'll have to go through mvn deploy:deploy-file... 😢

Workaround for this:
If you set up a custom URLHandlerDispatcher, sbt will respect that starting with 1.0.4 (sbt/librarymanagement#183), unless you're trying to use HTTP or HTTPS. To force these two protocols to go over a custom URLHandler, you'd need an instance whose setDownloader methods don't do anything, and set up your custom URLHandler on instantiation:
onLoad in Global := (onLoad in Global).value andThen { state =>
  import org.apache.ivy.util.url._

  val urlHandlerDispatcher = new URLHandlerDispatcher {
    super.setDownloader("http", CustomURLHandler)
    super.setDownloader("https", CustomURLHandler)

    override def setDownloader(protocol: String, downloader: URLHandler): Unit = {}
  }

  URLHandlerRegistry.setDefault(urlHandlerDispatcher)

  state
}

Then, you just implement a CustomURLHandler that extends GigahorseUrlHandler and overrides upload:
object CustomURLHandler extends GigahorseUrlHandler {

  private lazy val okHttpClient: OkHttpClient = {
    GigahorseUrlHandler.http
      .underlying[OkHttpClient]
      .newBuilder()
      .authenticator(new sbt.internal.librarymanagement.JavaNetAuthenticator)
      .followRedirects(true)
      .followSslRedirects(true)
      .build
  }

  private val EmptyBuffer: Array[Byte] = new Array[Byte](0)

  override def upload(source: File, dest0: URL, l: CopyProgressListener): Unit = {

    if (("http" != dest0.getProtocol) && ("https" != dest0.getProtocol)) {
      throw new UnsupportedOperationException("URL repository only support HTTP PUT at the moment")
    }

    IvyAuthenticator.install()
    ErrorMessageAuthenticator.install()

    val dest = normalizeToURL(dest0)

    val body = RequestBody.create(MediaType.parse("application/octet-stream"), source)

    val request = new Request.Builder()
          .url(dest)
          .addHeader("Custom-Header", "custom-header-value")
          .put(body)
          .build()

    if (l != null) {
      l.start(new CopyProgressEvent())
    }
    val response = okHttpClient.newCall(request).execute()
    try {
      if (l != null) {
        l.end(new CopyProgressEvent(EmptyBuffer, source.length()))
      }
      validatePutStatusCode(dest, response.code(), response.message())
    } finally {
      response.close()
    }
  }
}

@cemelo Thanks so much, your code works like a charm! :+1:

I'm going to close this if https://github.com/sbt/sbt/issues/4382#issuecomment-469734888 works.

Hi @mvdstam and @cemelo , could you briefly comment where / how you add the CustomURLHandler exactly, I have the same issue and would like to get it to work, yet my sbt skills are novice. Thank you!

@alterationx10 @cemelo I'm using this same approach to add headers to the download request of artifacts. I'm overriding GigahorseUrlHandler's download but is not working, any idea?

@alterationx10 Can you publish your plugin on Maven Central? Publishing it locally to use is a bit cumbersome on CI/CD. :)

@kevmo314 I could try and get something set up in the next few days, if it'd be helpful! Ironically, I haven't published it's artifact to it's own GitLab repo 🤣

@alterationx10 Yeah that would be helpful, thanks!

@kevmo314 Better late than never? In the process of getting my account set up to publish to Maven Central. In the meantime, I've got it published to GitLab: https://gitlab.com/k8ty/gitlab-sbt/-/packages/294966
Please note the group change + some name refactoring (and therefore some command changes). Not wanting to use this thread as a news letter 🤣 so I'll make a note there in the projects README once it's published with the MC info

Awesome! Definitely better late than never, this is an easy switch flip for us and saves us one step in the manual process :)

@alterationx10 Your first plugin rocks!
I am new to gitlab; and want to offer some changes.
Not sure how to branch and/or MergeRequest.
Anyway we can discuss the changes I needed?
slugd_patch.txt

@ramabu Sure! It looks like you just want to add in some general options to pass in header - seems sensible to add in! I notice you did add it to the old repository that I hadn't cleaned up much - but there weren't a lot of changes, so should be easy to add in.
I seem to get a lot more interaction of GitHub, so I've moved the repository over here. Feel free to make a PR or an issue there if you're more comfortable with GitHub:  https://github.com/k8ty-app/sbt-publish

Thanks @alterationx10!
One other thing I noticed is - Your plugin allows csrf-token-based upload.
What about "GET", used to resolve dependencies? This too may not succeed without a token

also late to this but for dependency resolution as well as publish  this should be resolved for sbt v1.3.10 using @alterationx10 plugin + this below that got added to sbt-coursier which is the default for dependency resolution since v1.3
https://github.com/coursier/sbt-coursier/pull/220
resolvers +=
  "gitlab" at "https://gitlab.com/api/v4/projects/1234567/packages/maven"

csrConfiguration ~=
(_.withAuthenticationByRepositoryId(Vector(("gitlab",Authentication("","",false,None,Some(Seq(("Private-Token","xxxx"))))))))
updateClassifiers / csrConfiguration ~= (_.withAuthenticationByRepositoryId(Vector(("gitlab",Authentication("","",false,None,Some(Seq(("Private-Token","xxxx"))))))))

If you are on an older version and need both publishing and dependency resolution:
https://github.com/gilcloud/sbt-gitlab I'll try and publish this properly as well if you want to test just add following to project/plugins.sbt
lazy val root = (project in file(".")).dependsOn(gitlabPlugin) lazy val gitlabPlugin = RootProject(uri("git://github.com/gilcloud/sbt-gitlab"))

Hi Richard
No older version.
So - let me see if I understood you correctly.
For Token-based auth for resolve/download

no need for a plugin. It is built into coursier, which is the default
download for sbt1.3.10+ (we are with 1.3.13)

For Token-based auth for publish/upload:

@Alterationx10's plugin does it.
It, or a modified version of it, will probably be built into future sbt
Till then - we ca use our modified version in a private repo

So far so good?
If so - then we need the following changes in the upload plugin (see
attached patch):

Instead of [env-var+special publish verb] per token-type (Job,Private)

use a couple of env vars - token-type + token value

Use a keys argument for gitlab host (as well as project id)
(The request/response printlns can be ignored, though were quite
useful debugging this)

I'll recreate the patch for the newer
https://github.com/k8ty-app/sbt-publish
How soon do you think we can find these in master of
https://github.com/k8ty-app/sbt-publish?
Thanks,
rama
On Wed, Jul 22, 2020 at 11:10 PM Richard Gilmore<EMAIL_ADDRESS>wrote:

also late to this but for dependency resolution as well as publish this
should be resolved for sbt v1.3.10 using @alterationx10
https://github.com/alterationx10 plugin + this below that got added to
sbt-coursier which is the default for dependency resolution since v1.3
coursier/sbt-coursier#220
https://github.com/coursier/sbt-coursier/pull/220
resolvers +=
"gitlab" at "https://gitlab.com/api/v4/projects/1234567/packages/maven"
csrConfiguration ~=
(.withAuthenticationByRepositoryId(Vector(("gitlab",Authentication("","",false,None,Some(Seq(("Private-Token","xxxx"))))))))
updateClassifiers / csrConfiguration ~= (.withAuthenticationByRepositoryId(Vector(("gitlab",Authentication("","",false,None,Some(Seq(("Private-Token","xxxx"))))))))
If you are on an older version and need both publishing and dependency
resolution:
https://github.com/gilcloud/sbt-gitlab I'll try and publish this properly
as well if you want to test just add following to project/plugins.sbt
lazy val root = (project in file(".")).dependsOn(gitlabPlugin) lazy val gitlabPlugin = RootProject(uri("git://github.com/gilcloud/sbt-gitlab"))
—
You are receiving this because you were mentioned.
  Aborting"))

override lazy val headerName: String = sys.env.getOrElse(s"GL_TOKEN_TYP", throw new RuntimeException("GL_TOKEN_TYP not set!  Aborting"))
}

override def projectSettings: Seq[Def.Setting[_]] = Seq(
 publishMavenStyle := true,

publishTo := Some("GitLab" at s"https://gitlab.com/api/v4/projects/${gitlabProjectId.value}/packages/maven"),
configureCIDispatcher := {

 val log =  streams.value.log

 log.info(s"Updating urlHandlerDispatcher to use GitlabCIURLHandler")

 val urlHandlerDispatcher = new URLHandlerDispatcher {

   super.setDownloader("http", GitlabCIURLHandler)

   super.setDownloader("https", GitlabCIURLHandler)

   override def setDownloader(protocol: String, downloader: URLHandler): Unit = {}

 }

 URLHandlerRegistry.setDefault(urlHandlerDispatcher)

 log.info(s"Publishing to ${publishTo.value}")

},

publishTo := Some("GitLab" at s"https://${gitlabRepo.value}/api/v4/projects/${gitlabProjectId.value}/packages/maven"),
configureTokenDispatcher := {
val log =  streams.value.log
@@ -55,12 +34,6 @@ object GitlabPlugin extends AutoPlugin with GitlabKeys {
publish
).value,

gitlabCIPublish := Def.sequential(

 configureCIDispatcher,

 publish

).value

)

}
diff --git a/src/main/scala/com/sludg/sbt/gitlab/GitlabURLHandler.scala b/src/main/scala/com/sludg/sbt/gitlab/GitlabURLHandler.scala
index 3f2e120..338df96 100644
--- a/src/main/scala/com/sludg/sbt/gitlab/GitlabURLHandler.scala
+++ b/src/main/scala/com/sludg/sbt/gitlab/GitlabURLHandler.scala
@@ -49,11 +49,14 @@ abstract class GitlabURLHandler extends GigahorseUrlHandler(GitlabURLHandlerHelp
if (l != null) {
l.start(new CopyProgressEvent())
}

println(s"request=$request")
println(s"headers=${request.headers}")
val response = GitlabURLHandlerHelper.http.newCall(request).execute()
try {
if (l != null) {
l.end(new CopyProgressEvent(EmptyBuffer, source.length()))
}

 println(s"response=$response");
 validatePutStatusCode(dest, response.code(), response.message())

} finally {
response.close()

@ramabu have a try with this version published last night, currently pending addition to sbt pluglin community releases so needs extra resolver, I've added a couple of things for your use case a new setting gitlabDomain which defaults to gitlab.com, but can be overridden to what ever suits for private.
it supports credentials file for local dev, and will now auto build a credentials in ci pipeline, with publishTo also optionaly being auto generated from the $gitlabProjectId and gitlabDomain pull request above open if you want to review to check it meet self hosted requirements
resolvers += Resolver.bintrayIvyRepo("gilcloud", "sbt-plugins")
addSbtPlugin("com.gilcloud" % "sbt-gitlab" % "0.0.3")

I didn't use coursier (yet)
Other than that -

Added the plugin in the correct version (project/plugin.sbt)
Enabled the plugin (build.sbt)
defined the plugin's settings:

GitlabPlugin.autoImport.gitlabProjectId :=  Option(1234)
GitlabPlugin.autoImport.gitlabDomain :=  "my.gitlab.net"
GitlabPlugin.autoImport.gitlabCredentials := Option(GitlabCredentials(s"$glTokenType", s"$glTokenValue"))

Worked like charm!

Excellent glad to hear.
FYI on credentials front it should support adding your credentials like this for local development:
~/.sbt/.credentials using the standard mechanisms
realm=gitlab
host=""
user=Private-Token
password=<api-token>

and in pipeline it will auto build credentials from the pipeline environment, so you can keep api keys well away from source control.

Hi @gilandose
Thanks for all the explanations.
However- I don't follow exactly how I should use coursier.
You say it is built into sbt 1.3.10 as default for dependency resolution (we use 1.3.13).
Does this mean it is no longer needs to be added and enabled as a plugin?
If not - then can you refer me to a working example (or past one here)?
I tried simply using the lines above, but they result in ambiguity as to what "Authentication" class is to be used, and/or the overloaded apply method of either.

I just receive the error was refused by the server, the url is corret, I'm using my private token. what can be wrong?

@robkler
First check sbt version >= 1.3.10
Next, verify the groupid
Also, on the same client machine I'd try to curl get from the repo, to shorten the cycle making sure I have the correct token, group etc.

I have added custom header support to sbt-aether-deploy now, so that can also be used with this.

I plublished, but most often fails and return the error: was refused by the server. One time I try to publish 2 times and got the error

@gilandose It appears that your plugin is currently failing if multiple sbt commands are chaitn together.
ie:
sbt clean test publish -> Results in an Authorization Error
while
sbt clean publish -> Works as expected
I haven't had time to dive into your code to try and debug it. Also it appears as though your master branch is not current with what is deployed (master is still at 0.1.0)

I’m glad that @gilandose’s plugin is working for people. Nevertheless, I ask that all discussion of the plugin move to https://github.com/gilcloud/sbt-gitlab. Any issues with the plugin should be reported there so they can be tracked and fixed.

Hi @gilandose
Regarding coursier fetch of dependencies - The authentication settings as they are don't apply to sub-projects.
The error distinctly indicates dependencies-resolve issue in the subproject. The root project depends on the same library, same version, and gets it.
[error] (subP / ssExtractDependencies) sbt.librarymanagement.ResolveException: Error downloading com.mine:common-deps:0.0.12-SNAPSHOT
[error]   Not found
[error]   Not found
[error]   not found: /Users/ramabu/.ivy2/local/com.mine/common-deps/0.0.12-SNAPSHOT/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/com/mine/common-deps/0.0.12-SNAPSHOT/common-deps-0.0.12-SNAPSHOT.pom
[error]   not found: https://my.gitlab.net/api/v4/groups/4321/-/packages/maven/com/mine/common-deps/0.0.12-SNAPSHOT/common-0deps.0.12-SNAPSHOT.pom
I tried putting in "in ThisBuild", no luck
I also tried modify the csrConfiguration of the subproject - no luck either
I am probably doing something wrong
`scala
resolvers in ThisBuild += s"gitlab at $glGrpRegistry" // 'in ThisBuild' mandatory
val auth:Authentication = Authentication(Seq(("Private-Token","some-value")))
csrConfiguration  ~= (.addRepositoryAuthentication("gitlab",auth)) // 'in ThisBuild' makes no difference
updateClassifiers / csrConfiguration ~= (.addRepositoryAuthentication("gitlab",auth))
subP / csrConfiguration ~= (_.addRepositoryAuthentication("gitlab",auth)) // makes no difference either
`

open an issue here, and we'll resolve issues against the project https://github.com/gilcloud/sbt-gitlab/issues

I don't see an option to create an Issue, nor to request access.
But - Is this supposed to work? Was this tested?

@robkler
First check sbt version >= 1.3.10
Next, verify the groupid
Also, on the same client machine I'd try to curl get from the repo, to shorten the cycle making sure I have the correct token, group etc.

Sorry @ramabu, but I noticed that Groups are only available for Gilab Premium versio. What if I do not want to specify it? Does the plugin still work. For now I have the same error as @robkler:

I just receive the error was refused by the server, the url is corret,

But Im using a Deploy-Token instead, because I need to access an organization Package Repository. Do you have any suggestion about why I get REFUSED by the server? Thanks

@neboduus Late to the party, but I think the problem lies with GitLab and not sbt in your case: https://gitlab.com/gitlab-org/gitlab/-/issues/273178 My current workaround is having a separate user account with access only to the project in question and using a Private-Token.

How was this resolved?  I see that someone wrote a plugin, but seems that this should be a feature.  What company allows a personal user/password to authenticate over tokens anymore?

That's just the groupid of the project i am publishing to.
On Thu, Feb 10, 2022, 19:41 Todor Kolev @.***> wrote:

@ramabu https://github.com/ramabu how did you get the glGroup?
—
Reply to this email directly, view it on GitHub
https://github.com/sbt/sbt/issues/4382#issuecomment-1035220532, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMY2JCIMWYOR4H2BEAGX7LU2P2DVANCNFSM4FXIKN6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


for S3 contrib package move to boto3

Updates the s3 contrib package to boto3 - pulled from Spotify's current version of the library - luigi/luigi/contrib/s3.py at master · spotify/luigi

Thank you Alan! will do


dataset.lastModifiedDate is not updated

Describe the bug
When looking at the code and the data, it seems that  lastModifiedDate in the dataset is not updated. I would expect this to be the last time a file was added/removed from the data.

If the metadata for a dataset is changed, we have to update the lastModifiedDate

I created a PR. I just modified the date when a file is added or removed. There are other methods like addFileInfoToZip etc. Do we want to modify the date there as well?

Add the lastModifiedDate to the UI

Adding lastModified date to UI


Documentaion: v0.13.3 displays old version (v0.12) with no context information

The latest version of the documentation seems to be an older version of the docs and has no reference to accessing the router via the context property (unless the way the router works has changed back to the pre-Context way).

current docs - http://rackt.github.io/react-router
previous docs (what works) - https://github.com/rackt/react-router/blob/v0.13.2/docs/api/RouterContext.md

Yes. This documentation inconsistency combined with the deprecating-then-non-deprecating of the mixins is making this library pretty confusing to work with right now.

@kiddhustle thank you for bringing this to our attention. we'll definitely update the docs. this.context.router is no longer part of our user-facing API. it's still public API, but people shouldn't have to know about context in order to use the router.
@davidtheclark apologies for the confusion. yes, things are a bit messy right now. we're working hard on a new release that should clear up a lot of confusion very soon! see #1158

updating now

but the current docs are correct, in v0.13.3 you don't access context directly anymore

Oh OK now that's strange I recently upgraded from v0.12 to v0.13.3 and started seeing errors related to the context object. That's when I decided to switch to using the context directly.After that the errors went away.
I'll have to try reverting the code back to when I get time to investigate the problem again.
Thanks

What are the errors?

https://github.com/rackt/react-router/blob/master/UPGRADE_GUIDE.md#0132---0133

How should we continue if we wish to use ES6 classes?

OK so I've just reverted the mixin calls in my project to follow the latest documentation and everything seems to work OK now.
I think maybe some of the dependencies weren't all up to date when I last tried so that could have caused the errors I was seeing.
Sorry for the false alarm!


chore: improve cypress happo screenshots (part 5)

FX-2850
Description
Cypress tests with happo screenshots are being improved. I separated commits to ease reviewing process
In this PR, I focused on 8 components:

PromptModal (improved with names)
Rating (cleaned up and improved with names)
RatingThumbs (spec file deleted because of redundant tests with storybook)
RichText (spec file deleted because of redundant tests with storybook)
RichTextEditor (cleaned up and improved with names)
Section (spec file deleted because of redundant tests with storybook)
Select (cleaned up and improved with names)
ShowMore (cleaned up and improved with names)

How to test

happo review and code review would be enough

Screenshots
happo reports are our screenshots
Development checks

Add changeset according to guidelines (if needed)
Read CONTRIBUTING.md and Component API principles
Annotate all props in component with documentation
Create examples for component
[x] Ensure that deployed demo has expected results and good examples
Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
Covered with tests

Breaking change

codemod is created and showcased in the changeset
test alpha package of Picasso in StaffPortal

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

We should cover the loading Select component only from Cypress, for both native and non-native selects.

Double check RichTextEditor invalid state visual tests.

seems, this one is missed in Storybook screenshots: https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Rating\.Thumbs\ when\ using\ different\ values\ renders\ the\ correct\ thumb\ as\ active\ if\ any\ \[value\3A\ true\]-default

This one is also interesting: https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Section\ when\ collapsible\ renders\ initially\ collapsed-default
I couldn't find out the same screenshot in Storybook screenshots

several missed screenshots for Select: loading state, render options and with custom background
https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Section\ when\ collapsible\ renders\ initially\ collapsed-default
https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Select\ sets\ background\ correctly\ to\ various\ select\ states

This one is also interesting: https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Section\ when\ collapsible\ renders\ initially\ collapsed-default I couldn't find out the same screenshot in Storybook screenshots

here, it is already covered. https://happo.io/a/675/p/1189/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=added#Section-Collapsible

several missed screenshots for Select: loading state, render options and with custom background
https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Section\ when\ collapsible\ renders\ initially\ collapsed-default
https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Select\ sets\ background\ correctly\ to\ various\ select\ states

Select examples are now renamed so you can find them under Added section here: https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=added

seems, this one is missed in Storybook screenshots: https://happo.io/a/675/p/848/compare/5119b77c986be7e694814c68e13d6da80f5443e7/d040b87afdc5b569943a88f8c342f74f529762a3?t=deleted#Rating\.Thumbs\ when\ using\ different\ values\ renders\ the\ correct\ thumb\ as\ active\ if\ any\ \[value\3A\ true\]-default

I have added back the spec file for RatingThumbs to cover all values now, good catch 👍


Support osx and tvos

The podfile only allows for ios installations, but there doesn't seem to be any platform specific code. Can we open it up to osx and tvos?

In the "ravero" fork: ravero/SwiftTryCatch#1.

Hi kengruven. Cool fix, mind doing a pr so this repo also gets those fixes?  Thanks!


Confusing CLI help for mikro-orm migration:create

Describe the bug
There is a line in the CLI help for mkro-orm migration:create which states:
--disable-fk-checks  Do not skip foreign key checks                  [boolean]

, but the name of the flag combined with the description is confusing. --disable-fk-checks sounds like it will skip FK checks, but the description says the opposite.
Expected behavior
Should one of these (probably the description) be changed?
Versions

Dependency
Version

node
12.14.1

typescript
3.7.5

mikro-orm
3.1.0

your-driver
postgresql

This looks like the flag is not even taken into account. Only the ORM's configuration disableForeignKeys is used. I guess we could just remove the flag, it makes sense only when running the migration.


[StyleFix][Update] Documentation

Changes:

Removed duplicate documentation in lib.rs
Updated readme file with latest improvements and changes

@sholderbach - Let me know if you see something that i can improve in the readme file

Looks great to me! Let me know if you are ready to merge (or if @jntrnr wants to chime in)

@sholderbach - Yeah for me it's okay to merge, maybe some comment could be more explicit but it's fine even like this.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/59712

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/60855

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/62771

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/63406


Update Tag.swift

Is there a reason the required init can't take both contents: String? and children: [Tag]?
So typography tags can be more readable?
For example:

Some text before a styled text and more after

Could be written...
H4("Some text before a") {
Span("styled text")
Text("and more after")
}

Yep, it's an intended behavior, I don't want to mix those init methods, but you can use:
H1 {
    Text("foo")
    Span("bar")
    Text("baz")
}


feat: SysStats.Version

Include version number in SysStatsResponse.
Useful when monitoring from grpc api.

Can you let me know what is the intented usage of this data? There maybe better place to put this version data.


[BUG] longhorn-driver-deployer pod not starting

[BUG] longhorn-driver-deployer pod not starting
I have a multi node cluster with 3 nodes, if i select the storage replicas to 2 or more i see the longhorn-driver-deployer is not starting and longhorn installation getting failed. ( Using the same nodes if i do a cluster installation with single replica, everything works fine)
engine-image-ei-4dbdb778-b9krp              1/1     Running            0             17m
engine-image-ei-4dbdb778-wzpgx              1/1     Running            0             17m
instance-manager-e-639affb0                 1/1     Running            0             17m
instance-manager-e-a294542d                 1/1     Running            0             17m
instance-manager-r-013cd8d7                 1/1     Running            0             16m
instance-manager-r-4dada94e                 1/1     Running            0             16m
longhorn-driver-deployer-57c578d85b-j5r9p   0/1     Init:0/1           0             17m
longhorn-manager-9ng7l                      1/1     Running            0             17m
longhorn-manager-vqzqx                      1/1     Running            1 (17m ago)   17m
longhorn-ui-687875bbf-54rjc                 0/1     CrashLoopBackOff   8 (62s ago)   17m
I mainly see 2 errors in the pod logs,
** in one of longhorn-manager pods:**
time="2023-09-19T06:23:41Z" level=error msg="error getting fully qualified domain name for longhorn services: lookup longhorn-backend on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:55457-><IP_ADDRESS>:53: i/o timeout"
in one of instance-manager pods:
tgtd: iser_ib_init(3431) Failed to initialize RDMA; load kernel modules?
tgtd: work_timer_start(146) use timer_fd based scheduler
tgtd: bs_init(387) use signalfd notification
time="2023-09-19T06:23:01Z" level=info msg="Storing process logs at path: /var/log/instances"
[longhorn-instance-manager] time="2023-09-19T06:23:01Z" level=info msg="Instance Manager listening to <IP_ADDRESS>:8500"
[longhorn-instance-manager] time="2023-09-19T06:23:01Z" level=debug msg="started new process manager update watch"
kube-dns and flennal pods are running fine without any errors.
To Reproduce
Create a multi node cluster, install the new version of longhorn
Expected behavior
install the longhorn successfully

Support bundle for troubleshooting
unable to take the support bundle as ui service is down,
Environment

Longhorn version: 1.2.4
Installation method: Helm
Kubernetes distro: kubeadm, Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.6"

Number of management node in the cluster: 1
Number of worker node in the cluster: 2

Node config

OS type and version: Ubuntu 22.04
Kernel version: 5.15.0-83-generic
CPU per node: 26 cores
Memory per node: 64gb
Disk type(e.g. SSD/NVMe/HDD): NVMe
Network bandwidth between the nodes: same DC

Underlying Infrastructure: Baremetal
Number of Longhorn volumes in the cluster: Nothing yet created
Impacted Longhorn resources:

Volume names:

Additional context

i see the below one of the longhorn node is not ready.. is there a way to check what is wrong in the node  ?
eic@devcpu05:~$
eic@devcpu05:~$ kubectl  -n longhorn-system  get nodes.longhorn.io
NAME       READY   ALLOWSCHEDULING   SCHEDULABLE   AGE
devgpu01   False   true              True          5h37m
devgpu02   True    true              True          5h37m
eic@devcpu05:~$


JavaDoc do not link to other modules

Example: The CdiContainerManager implements the ContainerManager interface that is part of another Dolphin Platform module but the JavaDoc do not provide a link.
See https://canoo.github.io/dolphin-platform/javadoc/server-jee/com/canoo/dolphin/server/javaee/CdiContainerManager.html

merged


Could this be helpful? - CMS Hints

To let non-technical people edit a site, something like http://prose.io is helpful.  Is there any interest in the following type of enhancements?
Content Regions
Static site builders could allow named content regions to be marked as editable (maybe with special comments?). So you could have a region called "headline". Then external tools could be used by prose for simple CMS functionality.  For example a user could choose to only edit defined content areas.
Verbs
A verb could be defined in a config file that would let a user do something like "Create new job posting". The verb is essentially a named rule that specifies what file(s) to create and where to put them.
Thoughts?

Is there any interest in the following type of enhancements?

@WhitneyLand They both sound interesting.  :)
What are you proposing specifically?
Phillip.

Static site builders could allow named content regions to be marked as editable (maybe with special comments?). So you could have a region called "headline". Then external tools could be used by prose for simple CMS functionality. For example a user could choose to only edit defined content areas.

You can see an example of this in action if you try out Cloudcannon's Jekyll beta.

You can see an example of this in action if you try out Cloudcannon's Jekyll beta.

Both of the features -- editable elements and how they've polished collections -- are mighty impressive.
So, just for the sake of clarity: What are you proposing / wanting feedback on?

Sorry for delay I've been flying too much.  If you are willing to leave the issue open a bit longer I can respond tomorrow when I arrive at a real Internet connection.  Thx.

On Aug 5, 2015, at 1:46 PM, Phillip Smith<EMAIL_ADDRESS>wrote:
You can see an example of this in action if you try out Cloudcannon's Jekyll beta.
Both of the features -- editable elements and how they've polished collections -- are mighty impressive.
So, just for the sake of clarity: What are you proposing / wanting feedback on?
—
Reply to this email directly or view it on GitHub.

All good. Take your time. :)

On Aug 5, 2015, at 2:01 PM, Lee Whitney<EMAIL_ADDRESS>wrote:
Sorry for delay I've been flying too much. If you are willing to leave the issue open a bit longer I can respond tomorrow when I arrive at a real Internet connection. Thx.

On Aug 5, 2015, at 1:46 PM, Phillip Smith<EMAIL_ADDRESS>wrote:
You can see an example of this in action if you try out Cloudcannon's Jekyll beta.
Both of the features -- editable elements and how they've polished collections -- are mighty impressive.
So, just for the sake of clarity: What are you proposing / wanting feedback on?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub.

Thought I was inventing something interesting :).  However it looks like this is exactly what cloudcannon is already doing.  However...
It could be great for prose to support some the same standards for editable sections.  For example, why not use the same CSS class to designate what is editable?
The advantage to piggybacking would be it first, it starts to create a defacto standard for static sites, and secondly prose is still a free, open, alternative.

It could be great for prose to support some the same standards for editable sections. For example, why not use the same CSS class to designate what is editable?

That's an interest idea, indeed.
@WhitneyLand Would you mind researching what selectors are used elsewhere for the sake of documenting what an enhancement like this might involve?

Glad to do so.
Thanks for your response -

On Aug 12, 2015, at 5:48 PM, Phillip Smith<EMAIL_ADDRESS>wrote:
It could be great for prose to support some the same standards for editable sections. For example, why not use the same CSS class to designate what is editable?
That's an interest idea, indeed.
@WhitneyLand Would you mind researching what selectors are used elsewhere for the sake of documenting what an enhancement like this might involve?
—
Reply to this email directly or view it on GitHub.


Cleanup

various chore/cleanup actions

:books: Documentation preview :books:: https://mrmoe--138.org.readthedocs.build/en/138/

Codecov Report

Merging #138 (33f2630) into master (6e1cec1) will decrease coverage by 0.58%.
The diff coverage is 96.85%.

@@            Coverage Diff             @@
##           master     #138      +/-   ##
==========================================
- Coverage   98.66%   98.07%   -0.59%
==========================================
  Files          60       66       +6
  Lines        3814     3898      +84
==========================================
+ Hits         3763     3823      +60
- Misses         51       75      +24

Flag
Coverage Δ

network
38.78% <31.82%> (+1.70%)
:arrow_up:

unit
97.17% <96.85%> (-0.57%)
:arrow_down:

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


Open all in new tabs not working

When I right click on a folder and press "open all in new tabs" nothing happens same with "new window" options.

What is your browser/version and operating system?
Does it happen for all folders or only a specific one?
Will you please take a screenshot of any errors that appear in the browser console at the time you "open all in new tabs"?
I'm not able to reproduce this on any of my computers, and it's never been reported as an issue before, so I think it may be a specific bookmark that is causing this to happen for you.

Thank you for the additional information. I just updated Firefox and can now reproduce the issue as well. I'm going to do some research to determine what changed with the latest update so I can fix it.

Thank you for making me aware of this bug. The issue has been fixed in release 2.6.1.
Fixed by e8ce8aa.


Whats the entry point?

Its very difficult in comparison to the other dom libraries to see where the entry point is. There is no example where you load HTML from a string or initialize a Document object. Can you provide an example?

If you have some experience with JavaScript, see Synonyms - Go and JavaScript.
Otherwise you can see Frontend Programming in Go.

I add this in README. If this is not what you want, you can re-open this issue and give more details.


⚠ mstdn.jp has degraded performance

In 4f18f41, mstdn.jp (https://mstdn.jp/health) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: mstdn.jp performance has improved in 1741fa8 after 3 hours, 16 minutes.


Fix NDArray delay allocation inconsistency

Here the TBlob is used before allocation. This bug happens if one ndarray is binded to executor before allocation.

@tqchen

LGTM,


Challenge 1

We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This PR is now closed! 🎉


Issue with recognizing load panel 2D elements

Hello team!
I've been using the GsaGH plugin with great pleasure. However, I ran into an issue when using the 2D element load panels.
When I create the panels and load them, it does not recognize the load panels correctly.
It creates them perfectly fine, but it seems to not automatically detect the load panel edges.
When I open the model in GSA, and go to the 2D property wizard, and simply click "Ok" through it, it does fix the issue.
Is this an issue on my end, or could it be that the GsaGH plugin does not correctly/fully initialize the load panels?
As the panels are not initialized correctly, I'm not able to directly access the results in GH which is a shame (also making optimization not possible sadly).

Hi @Marzalon,
Thanks for the feedback.
We are aware of this limitation and have already started working on a fix;
The upcoming version 10.1.65 of GSA will include the ability to set support type of load panels through the API and we will soon after the release make that available in the Grasshopper plugin as well.

Thanks for the information!
Then I'll be awaiting the upcoming updates.


Remove Etrigan and add a new web-ui

[x] remove etrigan

[x]  remove etrigan gui -> pykms_GuiBase
[x]     remove requirements from requirements.txt
[x]     remove config options for etrigan
[x]     update docs

[x] remove sqliteweb support

[x]     from docker
[x]     port 8080? reuse and / or remove from docs

[x] change db path to point to the db instead of the dir

[x]     backward compatible - if to dir, use it (warning!), if to file, use it
[x]     update docs

[x] if have db arg

[x]     create table if not exist

[x] remove dummy client from docker image
[x] add flask

[x]     bulma css?
[x]     do not start webui if db arg is missing (error)
[x]     add/remove requirements from requirements.txt
[x]     update docs
[x]         port 8080?
[x]         add screenshots! -> readme?

[x] docker

[x]     new tag? or replace the old?
[x]     change logging to also log to stdout by default

[x] update install instructions

[x]     use requirements.txt from full docker

[x] changelog! new tag and release?
[x] extend web interface

[x]     embed version info (commit time/hash)
[x]     show known products -> allow click to see them all (with keys!)

This is a chonker, will review ASAP.

@Matthew-Beckett Off - that's a good question... How do we handle betas? Shall we introduce a new tag and branch for that first?

Let's go beta!


How to distinguish the occlusion relationship between objects in the result ？

I have completed the training and got the KINS dataset training results as in the article, but found that the instance masks in the result are unordered, I can't distinguish the occlusion relationship between objects ,like the the order of red, green and blue cars in the image. Faithfully ask you how to distinguish in the code.

Hi @JaggerYang-code, Can I ask about the score of AP_det and AP_seg that you have reproduced on KINS?

I used a small dataset with only a portion of the KINS data, and I only identified one class, so unfortunately our results cannot be compared. Here are my results:
[02/08 10:29:27] d2.evaluation.coco_evaluation INFO: Evaluation results for bbox:

AP
AP50
AP75
APs
APm
APl

75.080
91.016
88.814
78.812
nan
nan

[02/08 10:29:27] d2.evaluation.coco_evaluation INFO: Evaluation results for segm:

AP
AP50
AP75
APs
APm
APl

:------:
:------:
:------:
:------:
:-----:
:-----:

74.571
91.207
85.328
77.822
nan
nan

I see, thanks for your reply @JaggerYang-code!


Investigate how to reduce the time necessary to run a full CI build

Describe the issue
Investigate how to reduce the time necessary to run a full CI build
Some options:

Can we run individual Cypress tests in parallel?
Can we reduce some of the Cypress waits or change those to timeouts?
Pipeline tests are taking 28mins, break those into multiple ones? Run in parallel?

Did some investigation and if we rewrite out GitHub Actions to call our cypress tests directly instead of using the makefile then we can use the following doc to parallelize the tests https://docs.cypress.io/guides/continuous-integration/github-actions#Parallelization we would most likely also want to split the pipeline test up as well to further speed things up

Temporarily putting this in the 3.8 milestone to discuss when planning for next sprint


Docker support

I found this image on docker hub for luigi https://hub.docker.com/r/spotify/luigi/tags
The version over there is 2.7.2 and the latest version of this repo is 2.8.11 .
So do you have any plans for supporting the image or is it already dead?

It would also be nice to see the Dockerfile.

For anyone else who comes across this hunting for a Dockerfile: it looks like it's just based off of the python image on Alpine. Something like this:
FROM python:alpine
RUN /bin/sh -c pip install luigi
EXPOSE 8082
ENTRYPOINT ["luigid"]

Of course, you'll have to make sure that the Python tag you're using is one supported by Luigi, so probably python:3.9-alpine right now.


New Cura 4.4 slicer plugin

I've made a few tweaks to a slicer plugin that now adds support for Cura 4.4 with Cura 4.4 printer profiles.
Plugin is here.
I've been testing it and it slices files exactly as Cura 4.4 will, with all the options still available for changes (see the example printer profile folder in the git repository).
Since this is a completely up to date slicer now able to slice via octoprint, would you consider maintaining this plugin again?

I've just made a fork that allows the new slicer to work.
The patches do make the UI for other slicers a little wonky but other than the appearance, it works really well.
The patches also allow for dynamic slicer variables as well.

I was hoping I'll be more free this week but only getting busier. Sorry folks! maybe toward the end of February

Hi!
Any updates on this? Sounds promising 👍

The issue I've found is with the function/formula based values.
I have an idea for solving this issue since its causes silent failures in the gcode output and affects ALL versions of curaengine, since curaengine doesn't do any of the value resolutions for formulas causing invalid slicing settings. (note; it also affects the docker container version as well).
The only work around is to resolve every single slicing option and hand each one to curaengine in the -s option (iirc).

So we need our own "hierarchical value resolver" that mimics cura behavior on top of curaprofile parser itself, and then all absolute values to curaengine... Why doesn't it do that by itself? 😆
I haven't digg into this too much, but there may be another way to solve this:
Write a Cura plugin (for the desktop version) that exports full printing settings in a format we can easily parse on our side (octoprint). I know it's not ideal (support curaprofile directly would be easier for end users), but it can be a solution, and it seems fairly easier to develop.
For reference, this repo: 5axes/CuraSettingsWriter, which is in active development, seems to do just that.
I discovered it reading this thread; https://community.ultimaker.com/topic/21945-output-print-settings-to-text-file/page/2/

This isn't hard to do, I just need to write some regexes to do a few things to substitute variables out with values and then run a python exec/eval over them. I just haven't gotten around to it and I'd want to do it properly.
I wouldn't want to do a settings export because then formula values would remain static forever and wouldn't update properly when you change base values (eg, nozzle size).

Of course built in profile support is always better (I didn't know about the complexity of this).
Then I guess your fork (https://github.com/Red-M/OctoPrint-Slicer) it's usable with basic profiles, isn't it?
I'll give it a try.

Sorry guys for not being active in this project any more. I have been super busy with https://github.com/TheSpaghettiDetective/TheSpaghettiDetective.
But if you guys can come up with a PR that is tested, I'll be happy to merge.

I may not get around to this but if I do, I'll be sure to offer my patches upstream :)

@Red-M Why didn't you just expand on the CuraLegacy Plugin?  Was it too difficult to do?

@Red-M Why didn't you just expand on the CuraLegacy Plugin?  Was it too difficult to do?

Cura legacy is so old that it doesn't fit into Cura's new versioning system.
Yes it runs but there is so many improvements and new features in Cura 4, that really, Cura should be offering a usable CLI because even the docker container version isn't properly filling values due to the way that Cura now doesn't inherit values or have any defaults, EVERY value that the UI has must be to be provided to Cura's CLI when you ask for Cura's CLI to slice a file.
Profiles are not transferable from the CLI to the GUI (or visa versa) due to the way that the GUI does all the config work to then provide the CLI with every value in the GUI.

Cura legacy is so old that it doesn't fit into Cura's new versioning system.
Yes it runs but there is so many improvements and new features in Cura 4, that really, Cura should be offering a usable CLI because even the docker container version isn't properly filling values due to the way that Cura now doesn't inherit values or have any defaults, EVERY value that the UI has must be to be provided to Cura's CLI when you ask for Cura's CLI to slice a file.
Profiles are not transferable from the CLI to the GUI (or visa versa) due to the way that the GUI does all the config work to then provide the CLI with every value in the GUI.

I heard that the new profile for Cuts are so different from how Cute used to work that the Cuts Legacy plugin is unusable.
So how did you get around this problem?

I heard that the new profile for Cuts are so different from how Cute used to work that the Cuts Legacy plugin is unusable.
So how did you get around this problem?

I built a system using the profile metadata for Cura to support changing values and to present them to the user in this octoprint plugin.
That all kinda stopped when I realised that the GUI does all the expressions and math plus defaults for Cura profiles.

I built a system using the profile metadata for Cura to support changing values and to present them to the user in this octoprint plugin.
That all kinda stopped when I realised that the GUI does all the expressions and math plus defaults for Cura profiles.

Do you know jautocalc ? I'm using this on my blog to automagically compute formula without bothering much about writing a javascript function per formula.
If I understand the difficulty correctly here, it might be able to solve the variable replacement and evaluation for you directly, without requiring complex regex work and writing a calculator here.

I'm closing this, as much as I'd love to have something on device for slicing anywhere and with any end user device, this isn't feasible with my other works.
The main issue is that there is no dependency resolution in how to "resolve"/render values to something you could pass to the Cura CLI.
There is no easy way to do this and in someway ultimaker is making sure that nothing else can use the Cura CLI with another GUI or interface even though its used internally in every platform they make releases for Cura to (its done as an exec to the CLI binary with all the parsed and rendered values provided).
As much as I hate to say it, I'd frankly suggest changing your workflow and saving a copy of the Cura profile onto your octoprint instance to make sure you're using the right profile to slice your models with.
I think something should happen in this space to make Cura 5 available as a plugin but there needs to be serious work put in to do that currently (unless Cura change how to provide profiles and values to their CLI)


Update npm package

Latest commit on github is working but npm package strapi-provider-upload-firebase is still outdated. Can u plz update that too

pls update npm package


Enhancement: Support for setCustomValidity()

I currently use setCustomValidity against a textarea where I key in the string containing a JSON object.
eg
{ "prop": "value" }
Changing the textarea to a code-input gives J/S errors indicating the function is missing!
Is it something you would consider implementing?
Would it also be easy enough to take attributes from the code-input and apply them to the interior textarea like form, required, maxlength, minlength, cols, rows, wrap, etc.

Thank you for this suggestion. Seamless swapping of textareas for code-input elements is one of the main aims of the project, so I will make it a priority to add this.
However, I am afraid that I will be very busy until the end of June, so would appreciate it if you could add this via a pull request (you can override setCustomValidity in the codeInput.CodeInput class and follow the existing name attribute setup for the HTML attributes). Please let me know if you can't and I will work on it when I have time.

To clarify this, these methods are being called on a normal code-input element:
<code-input lang="JSON" id="input-json"    name="json"  placeholder="JSON File Contents"  required></code-input>

sorry if I didn't explain correctly...
I was thinking just to take the attributes minlength, maxlength, required and copying them from the code-input (or the original textarea when you get the seamless swap done) to the textarea, it would respond.
Also if I used codeinput.setCustomValidity('message'), it would push the message onto the textarea.

I have added this to the branch mentioned above. I will carry out some final testing then merge and release it when I have time.

Thank you again for this suggestion; I have released it in v1.5.0. You can see a demo of how to use it here.


Deleting a heading leaves document structure invalid

E.g. after hitting backspace on this selection:

no typing possible afterwards

Fixed using a guard.
https://github.com/substance/ui/commit/7a6cb53b1d6a1e55fa022c2b69a1b8f938f5ae4f


problem with villages

There are some problems with villages:

Villages spawn without any villagers or chests
"VillageDistance" in WorldConfig doesn't work properly. If set to 64, there are still several villages spawning right next to each other
Villages are spawning way too high up in the air, sometimes 5-10 blocks (even in flat areas)

I'm using TC build #162 on Spigot 1.10.2 version 9797151-301db84

I second that. I have tons of villages and 25% of them are way up there in no man's land.

I would rather have them completely disabled than having these monstrosities as villages

It seems like the villages up in the air is not TC's doing.
This happens without TC. It must be Multiverse. It doesn't happen in single player

Good, lets go bash them too then

Create another ticket if this occurs with only TC and latest builds.


Add egs/mandarin_voice demo

This is a mandarin speech synthesis demo using only 250 wav from thchs30 dataset (A11 speaker)

Synthetic speech samples: Listen to https://jackiexiao.github.io/MTTS/
A simple mandarin frontend is in https://github.com/jackiexiao/mtts

Some change (in src/frontend/label_normalisation.py )

add question = question.replace('\\?', '.') to support Question format like QS "R-iong"  {*+iong?=*}
frame_number = int(end_time/50000) - int(start_time / 50000) because my label's time stamp is exactly divisible by 50000 (different from like egs/slt_arctic‘s label), this modification reduces error

@ronanki Sorry about the mistake, I commit egs/mandarin_voice/s1/thchs30_250_demo.tar.gz which is not supposed to add here, could you delete it ?


Add validations for commitment, receipt, and ack

closes #415

Thank you for reviewing!
I'll add stricter checking for modules' keys, and another type for sending a packet at other PRs.


fill-pdf fs.createReadStream

Is there a way to pass in a readStream or something from an API instead of a local path? thanks!

Hey, I'm not doing much with this module anymore. But, will happily except PRs if you want to extend functionality.


Fix PWA Support

Since I started this project, I've had problems getting the Progressive Web Apps (PWA) stuff that is built into create-react-app to work. I've just been remarking out that code. But I should dig into the problem and see if there isn't a way to get it working properly in create-react-wptheme since the popularity of PWAs is on the rise.

Facebook put in a fix for Firefox in the serviceWorker.js file. I'm not sure if this was the only problem here... but I do use Firefox. So it might be...

@devloco I'm this bad-precaching-response: bad-precaching-response  500 when i register a serviceWorker. Any fix for this one?


Read/Write session is not available - V4 Java cosmos SDK

Java V4 SDK intermittently sees the below exception.
Read/Write session is not available

.Net has a fix for it.
Related PRs : https://msdata.visualstudio.com/_git/CosmosDB/pullrequest/383598?_a=overview
https://msdata.visualstudio.com/CosmosDB/_git/CosmosDB/pullrequest/401811

This is done.


Tags - Group By

Description
Here a simple example:
I have assigned a tag to my favorite files. I want them to be listed above the unlisted file.
For this I have done the following:

Created the tag
Assigned them to the desired items
Sort by: Alphabetically
Group by: Tags
Resulting in this, which is what I am aiming for

Problem is that if I leave the "Marvel Heroes" folder and then reopen it, it looks like this:

It still "applies" my sort by and group by preferences (as you can tell, there is still a group called "Untagged") but my tagged items are now also in it. The group with my favorite heroes which was above the Untagged group is now gone.
Steps To Reproduce

Create a tag (Favorites for example)
Assign the tags to some items (folders, files, videos, ...)
Set the Group By to be "Group By: Tags"
Leave the folder in which you applied these tags and group by preferences
Reopen the folder. The items still have tags but are under the group "Untagged" instead of the tag group they have assigned.

Requirements
I expect the group by to still work properly after leaving the folder. It does still apply it on some level (Untagged group is still present)
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4169
User ID
files-community
Log File
debug.log

Thanks for the feedback, We're tracking this in #15530. You can check there or click subscribe in the right sidebar for updates.


NPM version check for tip

Implemented a version check of npm to give a soft tip during the install procedure

@gaearon any movement on this?

Sorry for very long review. Requested a chance, once that is fixed it should be good review and include.

Tagging for next release so we don't lose it.

@mobinni Would you like to address my comment above?

@gaearon Oh sorry, I didn't seem to receive the e-mail update. Will fix!

Do you mind also resolving the conflict?

sure!

I think this should be good. Going to land now.

This is now out! Thanks for contributing.


Created sequence diagram for undo and Logic class diagram. Minor tweaks to code to match diagram

Thanks for working on the user guide


Notifications don't work on Android

I've got everything working on iOS but I struggled to get it working on Android. I tried everything and in  the end I decided to test your sample application. Notifications are received (validated with breakpoint) but notifications are never shown. Exactly the issue I had with my own app. Every time I receive a notification I find these lines in the log:

[Notification] Use of stream types is deprecated for operations other than volume control
[Notification] See the documentation of setSound() for what to use instead with android.media.AudioAttributes to qualify your playback use case

Did I forget something? Or is it related to Android Oreo?

@jantanis : Did you test on Android 6?
I have got it working on iOS, on android 6 but on android 8.1.0 I see the breakpoint in the line
             CrossAzurePushNotification.Current.OnNotificationReceived += (s, p) =>
but I don't see any notification.
I added the lines for Oreo as descibed in the documentation:
      //Set the default notification channel for your app when running Android Oreo
        if (Build.VERSION.SdkInt >= Android.OS.BuildVersionCodes.O)
        {
            //Change for your default notification channel id here
            AzurePushNotificationManager.DefaultNotificationChannelId = "DefaultChannel";

            //Change for your default notification channel name here
            AzurePushNotificationManager.DefaultNotificationChannelName = "General";
        }

So I am wondering if you have it working on Android 6 like I do.

I’ve implemented notifications with AppCenter Push now so can’t test it anymore.
From: Michel Moorlag<EMAIL_ADDRESS>Reply-To: CrossGeeks/AzurePushNotificationPlugin<EMAIL_ADDRESS>Date: Wednesday, 27 June 2018 at 16:32
To: CrossGeeks/AzurePushNotificationPlugin<EMAIL_ADDRESS>Cc: Jan Tanis<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [CrossGeeks/AzurePushNotificationPlugin] Notifications don't work on Android (#7)
@jantanishttps://github.com/jantanis : Did you test on Android 6?
I have got it working on iOS, on android 6 but on android 8.1.0 I see the breakpoint in the line
CrossAzurePushNotification.Current.OnNotificationReceived += (s, p) =>
but I don't see any notification.
I added the lines for Oreo as descibed in the documentation:
  //Set the default notification channel for your app when running Android Oreo

    if (Build.VERSION.SdkInt >= Android.OS.BuildVersionCodes.O)

    {

        //Change for your default notification channel id here

        AzurePushNotificationManager.DefaultNotificationChannelId = "DefaultChannel";

        //Change for your default notification channel name here

        AzurePushNotificationManager.DefaultNotificationChannelName = "General";

    }

So I am wondering if you have it working on Android 6 like I do.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/CrossGeeks/AzurePushNotificationPlugin/issues/7#issuecomment-400694395, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKfv_eJU6XyOuowXe3GQbrfTZntOmJWwks5uA5eXgaJpZM4TIFQq.

@jantanis  I am thinking doing the same. Is it easier to implement then this plugin?
I love this plugin but not being able to send notification to the latest Android versions makes it unusable for my project.
On a personal note, are you Jan Tanis from Holland that implemented the prognose module from R&R?

@michelmoorlag , @jantanis It is working for me on Anddroid 8.0.0.
You just need to remove:
'//Set the default notification channel for your app when running Android Oreo
if (Build.VERSION.SdkInt >= Android.OS.BuildVersionCodes.O)
{
//Change for your default notification channel id here
AzurePushNotificationManager.DefaultNotificationChannelId = "DefaultChannel";
        //Change for your default notification channel name here
        AzurePushNotificationManager.DefaultNotificationChannelName = "General";
    }'

Seems like, it is not needed to specify channelId and Name


Not working with typescript

Hi could you help me, How can I declare the types for react-code-input

This issue is still here, does that mean the package does not support Typescript at the moment?

@thakuranujsingh can i close this issue?

@40818419 yes you cab close it, thanks 👍

Hi, is the issue(package does not support Typescript) actual yet?

Hi, is the issue(package does not support Typescript) actual yet?


Similar(ish) project to be aware of?

https://github.com/mxcl/Workbench/
The one thing I really appreciate about Workbench is that it bypasses the privacy problem. It is sad that it's not cross-platform or cross-mac but it is good.

It's a cool project, and I like the GUI a lot. I think that's the one core feature that shallow-backup is missing.
We could maybe fix the privacy problem by encrypting when pushing to github.


[WIP] Client auth

Most server-side authorization schemes (implemented by the auth plugins) will require matching client side code that can populate the HTTP headers of requests to the Intake server with some information.  This PR implements a minimal plugin API that queries a client auth plugin for extra HTTP headers prior to each call.
One subtle issue is that to enable serialization of data sources to remote workers, we have to freeze these headers when the RemoteDataSource is created, as we can't rely on the client auth plugin to work properly on the workers.  (There are likely use cases where this will be insufficient, but to resolve it we first need to decide on a reliable way to run Dask tasks back on the client, even when the client is not running a worker process in the Dask cluster.)
To do:

[ ] Integration test demonstrating that secret auth / client auth plugins cooperate properly
[ ] Client plugin discovery and configuration
[ ] Documentation

Is the intent to specify an auth class in configuration (like you can do for the server), or always to pass a configured instance to the call to Catalog? Can we do both?

Yes, the idea is to allow both setting the auth kwarg on Catalog and also having a default auth set in a configuration file.

Although I'm not sure if having a global config file is the right usage model, so maybe just explicit auth for now...

OK, I've put in some placeholder documentation.  I think I want to add more documentation as part of a larger docs overhaul once we finish the other things in flight.

OK, going to merge this PR and start another one with some follow up fixes


Port buffer file improvements from Serilog.Sinks.Seq

Currently, this sink is using RollingFile as buffer, and it has some serious limitations, that whether the buffer file grows without limits, or the file is size-limited and the sink stops working after that. This is making the durable mode unusable with this sink.
I'm currently using the sink without durable mode, but this is a problem as some events may go missing for various reasons. The Seq sink had the same issues, and they were fixed on version 4.0. It would be great if these changes, that is https://github.com/serilog/serilog-sinks-seq/pull/92 and https://github.com/serilog/serilog-sinks-seq/pull/99 would be ported to this sink.

Is it possible to pay to get this implemented? Is anyone interested in resolving this?

Would love to have a better/improved version of the buffer, unfortunately too little time to implement. IMHO it would also be nice if this was a more generic Serilog buffer which would pick up events when the configured sink was unable to work.

I currently have this sink and the seq sink logging the same events. For some reason, I have lots of missing events in elastic search that are not logged nor reported in the failure sink/self log. I enabled the selflog, and eventually I get a few failures, but that is probably 1% of what is missing on elastic search.
I'm betting the lack of durable mode is the problem, but that's just a guess. I'll try to debug this at some point.

Old issue, cleaning up. There have been some improvement. Still facing this issue, feel free to reopen.


Unknown command 'inspect'

I'm not sure, what I'm doing wrong, but I can't get schema from introspection API, because of that error:

error: unknown command 'inspect'

Context:
I have package elm-gql added in devDependencies, installed using yarn, but I'm not able tu run it directly from CLI, so I running it using npx elm-gql inspect ....

Hah, looks like readme is not updated for current version. The initialization part is completely missing.

👍 The warning at the top of the README is meant to flag curious folks to beware.

Ha, thanks!  Yeah, the commands changed but I didn't update the docs.  Now they should be updated though!


Armor Trims Backport

I have released Armor Trims Backport 1.2 for 1.19.2, which includes the ability to add custom trimmable items (with associated textures) using the API (which will require an actual mod for due to its nature).
Since you opened Issue #4 on my Bug Tracker asking for the feature, I feel it fitting to leave one on yours stating that it has been implemented.
Due to the recent issue on CurseForge (that you have probably heard about), the mod will probably take up to a day to be added. However, the mod CAN already be found on Modrinth (here).
If you need my help with creating the integrated mod, you can ask me (the API might not always be clear; it's my first time doing something like it after all).

I should probably tell you that I modified the API slightly because I noticed a bug that requires some corrections in the API.
If you want the mod to look as good as possible you might want to install the new version of 1.2.

It’s been a while and I haven’t heard from you. Are you still working on this and need help? If you need help, I now have a Discord server where you can ask.

Haven't gotten around to the backport just yet, will work on it soon

Hello, I was wondering if you were either busy or had forgotten, but I want to know if you plan to/are working on this. This is just a follow up and there is no rush.


JSON/YAML compatibility issue in results

Describe the bug
NaNs in results.yaml files appear as ".nan", which causes JSON Decode errors when submitting them over the REST API.
To Reproduce
Steps to reproduce the behavior:
include a NaN in your results dictionary prior to writing the results .yaml file.
Expected behavior
Results parse correctly and can be sent to REST API
Temporary fix would be to have the medperf client include a workaround. Catch the JSONDecodeError, replace '.nan' with 'nan' and resubmit.

This appears to be more an issue of how python encodes/decodes JSON. Inside the json documentation they specify the following:

If allow_nan is true (the default), then NaN, Infinity, and -Infinity will be encoded as such. This behavior is not JSON specification compliant, but is consistent with most JavaScript based encoders and decoders. Otherwise, it will be a ValueError to encode such floats.

So we have to either:

throw an error if one of the NaN, Infinity and -Infinity show up in any of the objects we send to the server. Not User-friendly
Do our own encoding to JSON-Compliant equivalents. This encoding needs to be decoded by the other party that wants to process the data, and there might be situations in which the encoding collides with the actual results (Say we enconde NaN as a "nan" string, but in the future an actual result legitimately outputs "nan" as a string).

I've been doing some work on the second option. I'm currently changing the NaNs that json generates into Nones. This assuming that they are interchangeable. I don't know how to encode the Infinity options though. Some people suggest storing it as a huge value that will most probably get converted to infinity by any recipient of the data (e.g. 1e99999). Any other ideas?


✨ Databases/messages fixes

#337  👈
main

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

Join @RTLS and the rest of your teammates on  Graphite


Translate getting started guide to French

Create the file guides/getting_started_fr.textile and translate to French.
https://github.com/arquillian/arquillian.github.com/blob/develop/guides/getting_started.textile
For hints about using Textile, see the RedCloth reference guide: http://redcloth.org/textile/

Done by @antoinesd, Alexis Hassler and Badr ELHOUARI.


Server-Side Streaming

Currently, RipeSeed uses a simple JSON-based REST architecture, which is not suitable for this type of application. I believe we should switch to SSE (Server-Sent Events), as it would significantly improve the application's performance. SSE allows the server to push real-time updates to the client via HTTP, making it ideal for scenarios where continuous or intermittent updates are needed.
The appropriate Content-Type for SSE is: text/event-stream; charset=utf-8.

I have a query; please guide me. Where it is required to add Server-Sent Events?

@ghulamhussainjoyo on this /api/chat/ask-ripeseed

thank you very much for immediate response