Incorrectly highlight as an error expanded function (...) in loops

Prerequisites

[X] Ensure you have latest version of plugin installed
[X] Search for possible issue duplicates

Installation details

[X] IDE version (PyCharm->About PyCharm->Copy to Clipboard)

PyCharm 2020.1.1 (Professional Edition)
Build #PY-201.7223.92, built on April 30, 2020
Licensed to Anton Stamenov
Subscription is active until November 27, 2020
Runtime version: 11-internal+102 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.15.4
GC: ParNew, ConcurrentMarkSweep
Memory: 2014M
Cores: 4
Non-Bundled Plugins: zielu.gittoolbox, IdeaVIM, Key Promoter X, YAML/Ansible support,
 anton_dev_ua.PipeTableFormatter, com.jetbrains.ChooseRuntime,
 com.jetbrains.jbre.bintray.downloader, org.intellij.plugins.hcl, com.paperetto.dash,
 com.vexus2.intellij.marked, mobi.hsz.idea.gitignore, org.asciidoctor.intellij.asciidoc,
 org.jetbrains.plugins.go-template, com.intellij.plugin.adernov.powershell, AWSCloudFormation,
 aws.toolkit, com.jetbrains.intellij.datalore, ru.meanmail.plugin.requirements,
 com.intellij.kubernetes, com.atlassian.bitbucket.references, org.toml.lang

[X] intellij-hcl plugin version (Settings->Plugins)

0.7.8

[X] Terraform version (terraform -v)

Terraform v0.12.24

Terraform Configuration Files
locals {
  input = {
    "parent/child1" = [
       {k1: "v1"},
       {k2: "v2"}
    ]
    "parent/child2" = [
       {t1: "tv1"},
       {t2: "tv2"}
    ]
  }

  setting = {
    for k, v in local.input:
    dirname(k) => {
      basename(k) = merge(v...)
    }...
  }
}

Exception
It is a valid terraform syntax works as expected, but it the second expand function is marked as error with this error message: "An expanded function argument must be immediately followed by closing parentheses."

Expected Behavior
Not highlighted as an error
Actual Behavior
Highlighted as an error.
Steps to Reproduce
Copy the example in new tf file and open it with PyCharm

Fixed, will be available in 0.7.9


Can multiple tcp sockets listening on the same endpoint of one interface?

In linux, the listen syscall can accept a backlog parameter, so it can accept multiple connection request meantime.
Since the listen method in smoltcp does not has a backlog paramter now, I wonder if I can construct multiple socket  listening on the same address to mimic the backlog behavior.
I have tried it but the second connection will reports error that ACK cannot be accepted. So is it impossible by design? or is it only my programming mistake?

There's no equivalent of linux "backlog", each socket can accept only 1 connection. If you want to accept multiple, create N sockets, call listen() on them on the same port -- it's fine to have N listening sockets on the same port, incoming connections go to a random one.

Thanks a lot. The problem is solved.


Added missing aria labels to pagination component

Description
Added missing aria labels to the pagination component
Impacted Areas in Application
Pagination component, Accessibility
Additional Notes
They were described in the documentation as having a value https://material-table.com/#/docs/features/localization but their actual value was null

Thank you @vmihalachi !!

Thank you @vmihalachi !!

Thanks @oze4, I appreciate what you guys are doing!

Thanks @oze4, I appreciate what you guys are doing!

@vmihalachi just saw this, and thank you very much! It feels good to hear that.
Butttt

you guys

I wish I could take credit, but it's what we (including you, per this commit) are doing together that makes this awesome! :smile:
Cheers.

@vmihalachi just saw this, and thank you very much! It feels good to hear that.
Butttt

you guys

I wish I could take credit, but it's what we (including you, per this commit) are doing together that makes this awesome! :smile:
Cheers.


Consider adding TryParse() methods to every AsX method in FieldValue and DocumentField

Feedback from our customers where when trying our AsX where X is a type methods in FieldValue and DocumentField we throw exception when the Type doesn't match.
It will be more convenient to have a TryParse method that the user can call that will return a bool value.
See https://docs.microsoft.com/en-us/dotnet/standard/design-guidelines/exceptions-and-performance#try-parse-pattern

We need to understand which scenarios are causing the throwing and if the if ( xx is type chu) is enough or not

Moving to backlog while working on the design. According to when it is ready and approved will add to an active milestone

After discussing this matter with architects, we won't implement AsX methods as it could just make the class more confusing to customers.
We decided to add an DocumentFieldType.Unknown enum value that can be used in these scenarios. More information here: https://github.com/Azure/azure-sdk-for-net/pull/30909

Updated the way DocumentField is handled. When the field value is not returned by the service (which happens when they can't parse the extracted value into the expected type), we set DocumentField.FieldType to DocumentFieldType.Unknown in the SDK. Customers could then use the DocumentField.ExpectedFieldType in addition to the raw string DocumentField.Content to manually parse the field value in this case. This change permits handling this scenario without try/catch blocks (current approach).


try h2spec 2.6.0

relates to #262
the h2spec v2 GitHub release files are now .tar.gz -- except the Windows one which is .zip
the h2spec v2 GitHub release artifacts have no directories embedded in them
the h2spec command no longer supports the -s option so that causes some test failures

needs investigation

Fixed the test problems in #266.
The problem was, that with the latest version of h2spec, it seems selecting a group executed all examples even in subgroups while the previous version seemed to either select fewer examples or new examples were added in the meantime.


'ODIN' object has no attribute 'visualize_pred_on_scannet'

"visualize_pred_on_scannet" has no Implementation but be called
here

i'll add a fix

Is there any update or fix?

Fixed


Cle/4824 fhir safe

This PR includes the work to process an incoming HL7 message, turn it into FHIR, store the FHIR, and put a message in the translation queue for further action.
Test Steps:

manual send an HL7 post as sender ignore.ignore-full-elr, check the database and azure queues to verify reports, task, action, lineage, storage, and queue messages.
auto run unit tests

Changes

update the full ELR Report constructor to accept lineages
un-deprecate the 'TRANSLATE' event
refactor, update various methods for better data passing and testability
add FHIREngine class, to contain logical methods for handling FHIR interaction
update Message.kt for better serialization/deserialization handling
add/update tests
Add BaseEngine to abstract shared functionality of WorkflowEngine and FHIREngine
Update all references to BaseEngine companion object

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Time to review
Total comments

jeremy-page
27
36m
3

carlosfelix2
13
14h 16m
45

kevinhaube
11
1h 21m
8

clediggins-usds
9
1h 41m
11

MauriceReeves-usds
8
7h 40m
0

TomNUSDS
6
44m
19

bgantick
5
1h 22m
2

jorg3lopez
5
2h 30m
0

brick-green
5
42m
0

oslynn
4
16h 17m
2

sethdarragile6
4
1h 47m
3

mreifman
3
32m
1

ahay-agile6
3
3m
1

JosiahSiegel
1
3m
0

RickHawesUSDS
1
6m
2

jimduff-usds
1
12h 26m
0

doug-s-nava
1
3h 17m
9

You have a failure on your tests:
Test test processHl7 happy path() FAILED (10.8s)
  java.lang.Exception: Error loading schema catalog: ./metadata/schemas
      at gov.cdc.prime.router.fhirengine.engine.FhirEngineTests.test processHl7 happy path(FhirEngineTests.kt:124)
  Caused by: java.lang.IllegalStateException: Schema Error: 'LIVD-SARS-CoV-2' is missing in element '{Element(name=device_id, type=TABLE, valueSet=null, valueSetRef=null, altValues=null, table=LIVD-SARS-CoV-2, tableRef=null, tableColumn=null, cardinality=null, pii=null, phi=null, maxLength=null, default=null, defaultOverridesValue=null, mapper=obx17(), mapperOverridesValue=null, mapperRef=null, mapperArgs=null, reference=Follows guidance for OBX-17 as defined in the HL7 Confluence page, referenceUrl=https://confluence.hl7.org/display/OO/Proposed+HHS+ELR+Submission+Guidance+using+HL7+v2+Messages#ProposedHHSELRSubmissionGuidanceusingHL7v2Messages-DeviceIdentification, hhsGuidanceField=Device ID, natFlatFileField=null, hl7Field=null, hl7OutputFields=[OBX-17-1, OBX-17-9], hl7AOEQuestion=null, csvFields=null, fhirField=null, documentation=Device_id is a generated value for the OBX-17 field. It is based on the device model and the LIVD table., delimiter=null, nullifyValue=false).name}'
      at gov.cdc.prime.router.fhirengine.engine.FhirEngineTests.test processHl7 happy path(FhirEngineTests.kt:124)


FIX README badges

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

I signed it!

Sorry, I don't see a signature.

Hey @patoroco - we can't find your signature in either of our CLA response tables. Can you try and resubmit?


Client Allocation Request for: WillTechMusing

Client Allocation Questions
Core Information

Name: WillTechMusing
Website / Social Media: https://ctzero.com/
Region: North America
DataCap Requested: 5TiB
Addresses to be Notarized: f17du2ulfhtpwgs4rwd4ngnrblueatjv4c5n57umy
Notary Requested: andrewxhill

duplicate


[v0.2] upgrade intel-tensorflow to 2.10.0

works well with v0.1 and v0.2 workloads

@zigzagcai, let's hold this PR until PDT replied, thanks

This PR fixed issue


clarification on service account permissio

in the section "Choosing MIM for use with SharePoint Server", step 3:  "Run as a service"  is a policy or any other kind of permission?

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

ID: 4efa9cc0-f34e-5987-5e4e-ed7740af689a
Version Independent ID: 61dfe25a-aa67-7d00-edab-3b2647f53bab
Content: Install Microsoft Identity Manager for User Profiles in SharePoint Servers 2016 and 2019
Content Source: SharePoint/SharePointServer/administration/install-microsoft-identity-manager-for-user-profiles-in-sharepoint-server-2016.md
Product: sharepoint-server-itpro
GitHub Login: @Techwriter40
Microsoft Alias: kirks

@AndreaBarr assign to me, please.
@adalfa Yes, this is a Local Security Policy (secpol.msc) under User Rights Assignments. I will add some verbiage to point people towards that tool.

@tseward thanks for the response, but while "log on as service" is a well know settings, "run as service" is for what I know absent in "User Right Assignment" or i am missing something.
thanks

@adalfa Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close


⚠ Bofrost.lu has degraded performance

In 2532afa, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 6242 ms

Resolved: Bofrost.lu performance has improved in facdfd8 after 19 minutes.


Not able to import directives from common packgage '@angular/common';

Not able to import directives from common packgage
import {FORM_DIRECTIVES} from '@angular/common';
we are getting exception while importing 'PAGINATION_DIRECTIVES', 'CORE_DIRECTIVES' the common module
error TS2305: Module '"@angular/common"' has no exported member 'FORM_DIRECTIVES'

Yes, FORM_DIRECTIVES is not in @angular/common.
Not sure if you're using the seed or that's just an error you get in your custom build/custom app?

i am using the angular seed structure. And module  @angular/common also available in node_modules folder. But while importing directives, i am getting the above exception.


acs-aem-commons-ui.content-4.7.0.zip stuck in distribution queue on AEM cloud service

Required Information

AEM as a Cloud Service 2020.6.3731.20200615T120157Z
ACS Commons 4.7.0
ACS Commons included in an AEM project generated by AEM project archetype 23 based on these instructions: https://adobe-consulting-services.github.io/acs-aem-commons/pages/maven.html
sample project to reproduce the problem on this branch: https://github.com/stefanseifert/aem-cloud-service-deployment-issues/tree/feature/archetype-23-acscommonsacs

Expected Behavior

ACS commons 4.7.0 should deploy without problems on author and publish instances

Actual Behavior

Author deployment via cloud manager works fine, but installing the mutable content part of ACS commons (acs-aem-commons-ui.content-4.7.0.zip) on publish via replication often (not always) fails
this also happens when the package acs-aem-commons-ui.content-4.7.0.zip is replicated manually on author via the package manager
in the distribution queue it is visible that the package acs-aem-commons-ui.content-4.7.0.zip cannot be deployed to publish - even after hundres of attempts. unfortunately there is not trace about the reason to be found in the logs downloadable via the cloud manager UI (author/publish error logs).

screenshots:

Steps to Reproduce
try to deploy the sample project https://github.com/stefanseifert/aem-cloud-service-deployment-issues/tree/feature/archetype-23-acscommonsacs to an AEM cloud DEV instance, and check the distribution queues afterwards.

the source of the problem may be #2341 (have not validated yet)

Thanks @stefanseifert - I do believe the /var nodes are the cause of this.
https://docs.adobe.com/content/help/en/experience-manager-learn/cloud-service/debugging/debugging-aem-as-a-cloud-service/build-and-deployment.html#including-var-in-content-package


Client StreamPeerTCP::get_status() returns STATUS_CONNECTED even after server close

Operating system or device:
Win10 x64
Godot commit 15d1fca branch master
Issue description:
I'm checking if a client StreamPeerTCP connection is still alive in _process by checking its status but it seems that it doesn't know if server is shutdown because after I close the server, it still returns STATUS_CONNECTED. The server is netcat (Tried Hercules too) but the problem is from Godot and StreamPeerTCP::get_status()
Link to minimal example project:
Client code

in theory you have to check reading or writing for a conection to be alive. I know there are a few workarounds you can do to check if something disconnected, but haven't implemented them.
If anyone wants to add this be welcome

maybe some function to check this would work? or should check be done during  get_status()

Not critical for the upcoming 2.1, so moving to the next milestone.

What's the status in the current master branch?

https://github.com/godotengine/godot/issues/15393


[BEAM-370] Remove more .named() methods

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[*] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[*] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[*(] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[*] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @dhalperi or @davorbonaci

-R: davor / dan
R: @kennknowles


Certain Path Styles break "Type Generation"

Description
I've noticed that certain paths formatted with underscores seem to break type generation (but no errors are actually given). There ends up being a mismatch in the "options" generated that doesn't match the search path
The generated types.gen.ts from my provided config below yields the following
export type GetProgramSearchData = {
  path: {
    searchText: string;
  };
};

export type GetProgramSearchResponse = unknown;

export type GetProgramSearchError = unknown;

export type $OpenApiTs = {
  '/programs/search/{search_text}': {
    get: {
      req: GetProgramSearchData;
      res: {
        /**
         * Success
         */
        200: unknown;
      };
    };
  };
};

I've noticed that the path ends up being /programs/search/{search_text}, but in the "path options", we get
path.searchText (not path.search_text)
I then see the fetch reaching out to /programs/search/%7Bsearch_text%7D
search_text is being hard coded (well url encoded) instead of passing in the actual variable. I'm able to fix this by manually patching '/programs/search/{search_text} to '/programs/search/{searchText}
OpenAPI specification (optional)
{
  "swagger": "2.0",
  "basePath": "/api/v1.0",
  "paths": {
    "/programs/search/{search_text}": {
      "parameters": [
        {
          "name": "search_text",
          "in": "path",
          "required": true,
          "type": "string"
        }
      ],
      "get": {
        "responses": {
          "200": {
            "description": "Success"
          }
        },
        "operationId": "get_program_search",
        "tags": [
          "programs"
        ]
      }
    }
  },
  "info": {
    "title": "API",
    "version": "1.0"
  },
  "produces": [
    "application/json"
  ],
  "consumes": [
    "application/json"
  ]
}

Configuration
import { defineConfig } from '@hey-api/openapi-ts';

export default defineConfig({
  client: '@hey-api/client-fetch',
  input: './sample_api.json',
  output: {
    format: 'prettier',
    lint: 'eslint',
    path: './src/client',
  },
  types: {
    enums: 'javascript',
  },
});

System information (optional)
No response

Should have said that I am using<EMAIL_ADDRESS>to generate the client and services and "<EMAIL_ADDRESS>

Hey @adawalli, the addendum is the key part. Pretty sure this is fixed in the latest version of both. Please upgrade and let me know


Cannot download win32-x64-47_binding.node

I am running the following npm install:
$ npm install node-sass
I get the following error logged in the console:
Cannot download "https://github.com/sass/node-sass/releases/download/v3.4.1/win32-x64-47_binding.node"
It seem that win32-x64-47_binding.node is missing from the list here:
https://github.com/sass/node-sass/releases

Node 5 only just came out. The system we use to build windows binaries
hasn't been updated yet.
On 30 Oct 2015 10:41 pm, "heikkilamarko"<EMAIL_ADDRESS>wrote:

I am running the following npm install:
$ npm install node-sass
I get the following error logged in the console:
Cannot download "
https://github.com/sass/node-sass/releases/download/v3.4.1/win32-x64-47_binding.node
"
It seem that win32-x64-47_binding.node is missing from the list here:
https://github.com/sass/node-sass/releases
—
Reply to this email directly or view it on GitHub
https://github.com/sass/node-sass/issues/1230.

I have the same issues. What is the other option for this? like how to install previous version instead of win32-x64-47_binding.node

The solution is not using node 5.
On 31 Oct 2015 1:23 am, "Russell Pabon"<EMAIL_ADDRESS>wrote:

I have the same issues. What is the other option for this? like how to
install previous version instead of win32-x64-47_binding.node
—
Reply to this email directly or view it on GitHub
https://github.com/sass/node-sass/issues/1230#issuecomment-152539538.

i ran into the same issue, and yes, node-sass works just fine under v4.2.1.

I have installed Visual Studio Community 2015 and node-sass works perfectly with node 5. I don't know if this will help.

I already have VS2015 and  no help.

Windows Node 5 binary is now available for v3.4.1 only. Just npm install.


build/do.rq: build binary for tasks list if needed

build is not build(fresh), and will only rebuild the ./regal binary if passed true. If passed false, it will not rebuild the binary if some binary is present in the given location.
build(false) is thus to be used when any regal version will do, like for printing tasks.

I’m not sure rq is even intended to work on Windows

It's not? That's surprising. Many of our users surely use windows for their stuff. Contributing to Regal should also be open to everyone.
🤔 I'll see if I can't do this better with rq.tree instead.

It's not? That's surprising.

Oh, rq most certainly works, but I meant running as a script like this. I haven't used Windows in ages, but outside of emulation, do even things like:
#!/usr/bin/env rq

work? Or then you'd just call the binary explicitly I suppose.


(#1123) Fix file sealing on tmpfs

unlink dynamic configuration file before creating a new one

Closes #1123

Reproduction steps:
htop
sudo ldscope -a `pidof htop`
sudo ldscope -d `pidof htop`
sudo ldscope -a `pidof htop`
sudo ldscope -a `pidof htop`

See that there is no message on console like mentioned in https://github.com/criblio/appscope/pull/1110#issuecomment-1265808778


project mangement

Need to create a branching model

Approves/Reviewers

Need to create a test coverage
Need to update documentation

added pre-commit feature


Slice view not updated when focused and first image is loaded

Reproduce:

Open Glance
Switch to 2D view
Open an image file

Expected: Slice UI updates
Actual: slice UI doesn't update

First step is to allow disabling of automatic proxy focus on-creation in vtkjs: https://github.com/Kitware/vtk-js/pull/592

Second step is to activate source proxies after setting proxy input dataset.


With Ink installed, evaluation in REPL should only show inline results

When Ink is installed and I evaluate a selection in the REPL, I get an inline result, but I also see non-inline output as in => <result>.

This seems like the same issue as #58. I'm going to close this one to track this as a single issue.


chore: update develop & deploy & settings & modules ui to match new spec

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#54
#53
#52
#51  👈
#50
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


nixos/tinc: remove restartTriggers

Motivation for this change
restartTriggers pointed to the constant files in /nix/store/ and had to effect.

@fpletz Can you please revert it? I did not understand that it is here for restarting the daemon when its peer is removed.

or better
restartTriggers = let
    drvlist = [ config.environment.etc."tinc/${network}/tinc.conf".source ]
                ++ mapAttrsToList (host: _: config.environment.etc."tinc/${network}/hosts/${host}".source) data.hosts;
  in # drvlist might be too long
    [ (builtins.hashString "sha256" (concatMapStrings (d: d.outPath) drvlist)) ];


filters: optimize cel expression context with constant-time lookups

Description
This PR optimizes the CEL expression context implementation by replacing if-else chains with hash maps for O(1) lookups and improving memory management. This would enhance the performance for header and filter state access patterns while maintaining the same functionality.
This change introduces constant-time lookups using absl::flat_hash_map and improves the overall design through static initialization of lookup maps for better performance.
 It also improves memory management and provides better encapsulation of implementation details.
Risk Level: Low

No functional changes
Only performance optimizations
Existing test coverage maintained

Testing:

All existing tests pass
No behavioral changes

Docs Changes: N/A - internal implementation change only
Release Notes: N/A

Thanks. This is great. Any performance improvements are welcome if it does't break the security and our style.
But could you also add a benchmark test to prove it works as our expection? Thanks.
/wait

Thanks. This is great. Any performance improvements are welcome if it does't break the security and our style.
But could you also add a benchmark test to prove it works as our expection? Thanks.
/wait

@wbpcode Thanks for taking a look. I can certainly add a benchmarking test. This would be my first time. If possible, could you please share some pointers on how to write such a test?

Thanks. This is great. Any performance improvements are welcome if it does't break the security and our style.
But could you also add a benchmark test to prove it works as our expection? Thanks.
/wait

@wbpcode Thanks for taking a look. I can certainly add a benchmarking test. This would be my first time. If possible, could you please share some pointers on how to write such a test?

Every test is different. But you can find lots test files that end with benchmark or contain speed. There are lots of test cases. Besically, you only need to define a static function to run the code you want to test. And then the benchmark macro could help you to resolve other things.

/assign @wbpcode

Seems the performance improvement is trival. But I also agree the new look up table pattern is better. So, will take a look tomorrow.

cc @tyxia @kyessenov (extension owners)

LGTM apart from a change in filter_state lookup logic. I agree, this is better, and would allow us to implement Keys() quickly in a followup, even though the performance benefit is not significant.

@agrawroh What's up with filter state benchmark:
bmFilterState/10000000            3806409 us      3806192 us            1

Is it a benchmark problem or some implementation difference?

@agrawroh What's up with filter state benchmark:
bmFilterState/10000000            3806409 us      3806192 us            1

Is it a benchmark problem or some implementation difference?

I think it's just a benchmark issue. I will take a look and see if I can improve it


Repository should be renamed to reflect launch naming

Our working title aws-cloudformation-rpdk should be updated to aws-cloudformation-cli. GitHub will handle the repository rename cleanly so this should be minimally impactful.

If the organization is already named "aws-cloudformation", can we just name it "cloudformation-cli" to match the name in PyPI?

Yes that works for me. 👌🏼

sounds good, we'll do this this week. i think you're right that it'll be unproblematic, but do need to check the CI will handle this equally gracefully.

https://help.github.com/en/github/administering-a-repository/renaming-a-repository

Looks like it'll handle redirects so shouldn't be too much of a change. Might have to update readmes is the biggest thing. The CI should work but will be good to follow up and update it once the name change is done.

https://github.com/aws-cloudformation/cloudformation-cli/blame/master/README.rst#L96-L98 need to update this

keeping open for cleanup


invalid testnet endpoint https://btc-testnet.horizontalsystems.xyz/api

getting error when testing in testnet (https://btc-testnet.horizontalsystems.xyz/api )
error: Error Domain=NSURLErrorDomain Code=-1003 "A server with the specified hostname could not be found."

horizontal systems closed all the test node, this kit will try the the other nodes automatically.
"testnet-seed.bitcoin.petertodd.org", // Peter Todd
"testnet-seed.bitcoin.jonasschnelli.ch", // Jonas Schnelli
"testnet-seed.bluematt.me", // Matt Corallo
"testnet-seed.bitcoin.schildbach.de", // Andreas Schildbach
"bitcoin-testnet.bloqseeds.net", // Bloq

Thanks for the reply!
The kit do not try other nodes automatically for me though...
Anyway I tried the 5 host you listed
"testnet-seed.bitcoin.petertodd.org", // Peter Todd. -> no valid endpoint (404)
"testnet-seed.bitcoin.jonasschnelli.ch", // Jonas Schnelli -> Time out
"testnet-seed.bluematt.me", // Matt Corallo. -> Time out
"testnet-seed.bitcoin.schildbach.de", // Andreas Schildbach -> not reachable ( ping )
"bitcoin-testnet.bloqseeds.net", // Bloq -> not reachable -> ping

I am wondering if the kit support other api like mempool.space
https://mempool.space/testnet/docs/api/rest#get-address-transactions

@ieow just try the following command, the first two nodes worked well, you can check whether your local network can reached them well.
and mempoll is just an explorer, I'm not sure whether it provides a full node.
telnet testnet-seed.bitcoin.jonasschnelli.ch 18333
Trying <IP_ADDRESS>... Connected to seed.tbtc.petertodd.net. Escape character is '^]'. ^] telnet> quit Connection closed.
telnet testnet-seed.bitcoin.jonasschnelli.ch 18333 Trying <IP_ADDRESS>... Connected to testnet-seed.bitcoin.jonasschnelli.ch. Escape character is '^]'.

Sorry, I under provide the informations.
The issue I have is with the apiTransactionProvider which use
https://btc-testnet.horizontalsystems.xyz/api during testnet
        let apiTransactionProvider: IApiTransactionProvider?
        switch networkType {
        case .mainNet:
            let hsBlockHashFetcher = HsBlockHashFetcher(hsUrl: "https://api.blocksdecoded.com/v1/blockchains/bitcoin", logger: logger)

            if case let .blockchair(key) = syncMode {
                let blockchairApi = BlockchairApi(secretKey: key, chainId: network.blockchairChainId, logger: logger)
                let blockchairBlockHashFetcher = BlockchairBlockHashFetcher(blockchairApi: blockchairApi)
                let blockHashFetcher = BlockHashFetcher(hsFetcher: hsBlockHashFetcher, blockchairFetcher: blockchairBlockHashFetcher, checkpointHeight: checkpoint.block.height)

                apiTransactionProvider = BlockchairTransactionProvider(blockchairApi: blockchairApi, blockHashFetcher: blockHashFetcher)
//                    let blockchainComProvider = BlockchainComApi(url: "https://blockchain.info", blockHashFetcher: blockHashFetcher, logger: logger)
//                    apiTransactionProvider = BiApiBlockProvider(restoreProvider: blockchainComProvider, syncProvider: blockchairProvider, apiSyncStateManager: apiSyncStateManager)
            } else {
                apiTransactionProvider = BlockchainComApi(url: "https://blockchain.info", blockHashFetcher: hsBlockHashFetcher, logger: logger)
            }
        case .testNet:
            apiTransactionProvider = BCoinApi(url: "", logger: logger)
        case .regTest:
            apiTransactionProvider = nil
        }

https://github.com/horizontalsystems/BitcoinKit.Swift/blob/683a8cae5ca7f9a24c548d21e0cf7cf46b4f1359/Sources/BitcoinKit/Classes/Core/Kit.swift#L60
I think the full node do not provide the api needed for the apiTransactionProvider...
may be I could assign the apiTransactionProvider as nil?

Yeah, you're correct. the full node does not provide the http or https api, we should use the explorer api here.
You can't assign the txProvider as nil. If you'd like to use the mempool as your explorer, you should implement the logic by yourself.
I just implementted the regtest logic with an open source explorer days before.

created a pr here
https://github.com/horizontalsystems/BitcoinCore.Swift/pull/33


[Monitor]Add new command:az monitor action-group test-notifications create

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Monitor


[Request] notepads

Package Request
Criteria
For a package to be acceptable in the Extras bucket, it should be:

[x] a well-known and widely used tool (e.g. if it's a GitHub project, it should have at least 100 stars and 50 forks)
[x] the latest stable version of the program

Information
Name: notepads
Description: A modern, lightweight text editor with a minimalist design.
Homepage: www.notepadsapp.com
Download link(s): https://github.com/0x7c13/Notepads/releases
Some indication of popularity/repute: 6.7k GitHub star

The Non-portable bucket already has this app here, so I am going to close this, but feel free to reopen if I am misinterpreting your request.


Fix a trivial mistake of README.jp.md

Thanks @53ningen! My Japanese is not good enough to verify this, but I'll take your word for it =)


ToValue should support map types

Right now, ToValue treats objects of type t map[string]interface{} as objectGoReflect but should probably treat as standard objectGoMapSimple.

There is an unfortunate ambiguity here that prevents 1-to-1 mapping between Go and ECMAScript. Let's assume you have your type and then you define a method:
func (t t) Method() bool {
    return true
}

Then you can set a map value using 'Method' as the key:
var t t
t["Method"] = 42

Then you convert t into a Value and access it in ECMAScript:
t.Method

Question: what should that evaluate to, number or function?
Currently because t is treated as a reflect value it will evaluate to a function. The reason being if you created a custom map type it's probably because you want to define methods on it and most likely you will be using these methods rather than directly accessing the map. If you want access to the map as well you can just write a wrapper.
Hope this helps.

@dop251 yeah, it makes sense and I figured as much. I still think that the default behavior should include both and you should first use reflection to see if there is a method and if not then just fallback to the map behavior.

Actually good idea, otherwise you end up with an empty object... I'll implement it shortly

👍


the polyline could be automatically stretched

Draw a polyline with three points.
The coordinates of the starting point are the same as the coordinates of the ending point.
The polyline could be automatically stretched

Sandcastle example: https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/#c=rVVtb9o6FP4rFl+gKtdAy8LLaHVb6NZqKanWrOhqmSo3OYBVx+Y6DpRN/PfZSSAvVLr7cPlQ8OPnnJ7Xx2si0ZrCBiS6QBw2aAwRjUP8lGCNup8cx4IrQjnI+slHj6d8rEBKDT5IsaYByIvM0pdAFMyEZIGbMhq/PI6yj4R/Y4jUTFPkPYle0RApGUPziPEEUsHbVMiQsChjpaRdEoOOOyCKPIpY+lCOfRxHSoSTw22jHm4Ntxh8bhthEgSN/GxIHm+1ViKiigqOiE5iiwzkLgH5QsiAch1/hMQcKQ1FikhF+QKtBOVK8yGFSQiIRMnvd6yABwcbnCa0/5eRzmefi3atfxF+judSJwULCRBdmZAa3z3+V6fTx71B2zrvd8/65k/zvI+t/mDQs3p9a9Dtd63mgWad9zTQ6wy6vW7C67XPrO7A+tA+61nd5p+5+5EWyNQjkGSTJb2hgVoiRl6AeXwuZMPkQ3Ue7Y/6a5SnhhnwhVpq9PT0BP1CeePzFmDgSrOz1hSmZ+9leHD3nf5oFu91LENUsEhQ+gbskf6EIeq0m+U7XzAhh4dqmxP+58a2nVmFuAS6WKqvMAcJ3IeDyW0Zx2P76v7h2XWeP391vk0nuZNdwWFSp6M49bzr4L3a1MFe7ZRWApgLk5q+7vRXb+gW2BoU9YlXq/IoY+M/zUrEium9fo9/bV+Nv7xPn5lmD9FZ5TZSW5YXxjY5PhoIf7qz7eer6eTZ+ebad9Obqt1SbK6J/7qQIubBkSAk1ReS/jQixBxJF5Tn5a9cYPvmk1sxXmsp0ZWqmj6VYHztuK5z//90veIloBF5YTCBlVq6WtsmVEtG4i5XtWRECgK3K60Y25q6e/y/doQbzRmi+t6ivo9kD6Dy1OWKk68UKkbvMxKuXPE5ac2xWGd7P0TdIhgadaeEabwgyw9ZCE46RPcZSb8fK92JbaOyDelqospUzm7v3Or8lMYSHc9lacrfdVjg706yg2nGLtW67M2Ys60rSi9FrVkbJVN/mRr9TcOVkArFkjUwbikIV8xofusl9l9BYT+KjMdRa280Cuga0eDCq1UeWq9mSh9F+mYes0S9vNrlqKX5JTMmiHlGHD3jjGwNZdm5tFMQYzxq6eOxlRKCvRBZ8Pgb
Browser:Google Chrome  71.0.3578.98
Operating System:Windows 10 Home edition

Hi @venci01, thanks for the clear example!
I wasn't able to reproduce this with https://github.com/AnalyticalGraphicsInc/cesium/pull/7885, but could you confirm?
Here's a link: http://cesium-dev.s3-website-us-east-1.amazonaws.com/cesium/fixGroundPolylinesCoplanar/Apps/Sandcastle/index.html#c=rVVtb9o6FP4rFl+gKtdAy8LLaHVb6NZqKanWrOhqmSo3OYBVx+Y6DpRN/PfZSSAvVLr7cPlQ8OPnnJ7Xx2si0ZrCBiS6QBw2aAwRjUP8lGCNup8cx4IrQjnI+slHj6d8rEBKDT5IsaYByIvM0pdAFMyEZIGbMhq/PI6yj4R/Y4jUTFPkPYle0RApGUPziPEEUsHbVMiQsChjpaRdEoOOOyCKPIpY+lCOfRxHSoSTw22jHm4Ntxh8bhthEgSN/GxIHm+1ViKiigqOiE5iiwzkLgH5QsiAch1/hMQcKQ1FikhF+QKtBOVK8yGFSQiIRMnvd6yABwcbnCa0/5eRzmefi3atfxF+judSJwULCRBdmZAa3z3+V6fTx71B2zrvd8/65k/zvI+t/mDQs3p9a9Dtd63mgWad9zTQ6wy6vW7C67XPrO7A+tA+61nd5p+5+5EWyNQjkGSTJb2hgVoiRl6AeXwuZMPkQ3Ue7Y/6a5SnhhnwhVpq9PT0BP1CeePzFmDgSrOz1hSmZ+9leHD3nf5oFu91LENUsEhQ+gbskf6EIeq0m+U7XzAhh4dqmxP+58a2nVmFuAS6WKqvMAcJ3IeDyW0Zx2P76v7h2XWeP391vk0nuZNdwWFSp6M49bzr4L3a1MFe7ZRWApgLk5q+7vRXb+gW2BoU9YlXq/IoY+M/zUrEium9fo9/bV+Nv7xPn5lmD9FZ5TZSW5YXxjY5PhoIf7qz7eer6eTZ+ebad9Obqt1SbK6J/7qQIubBkSAk1ReS/jQixBxJF5Tn5a9cYPvmk1sxXmsp0ZWqmj6VYHztuK5z//90veIloBF5YTCBlVq6WtsmVEtG4i5XtWRECgK3K60Y25q6e/y/doQbzRmi+t6ivo9kD6Dy1OWKk68UKkbvMxKuXPE5ac2xWGd7P0TdIhgadaeEabwgyw9ZCE46RPcZSb8fK92JbaOyDelqospUzm7v3Or8lMYSHc9lacrfdVjg706yg2nGLtW67M2Ys60rSi9FrVkbJVN/mRr9TcOVkArFkjUwbikIV8xofusl9l9BYT+KjMdRa280Cuga0eDCq1UeWq9mSh9F+mYes0S9vNrlqKX5JTMmiHlGHD3jjGwNZdm5tFMQYzxq6eOxlRKCvRBZ8Pgb

I don't think #7902 is caused by the curvature of the earth.
Coincidence points on the polyline cause this problem.
You can see the pictures below.
ven.png
GIF2.gif

Hi @venci01, #7885 also includes some behavior changes for co-incident points. I'm no longer able to reproduce the problem using the same Sandcastle code but in the deployed branch, would you mind confirming?

@likangning93 哥们你会汉语吧，我英文不好，交流起来太费劲，不介意得话加qq吧（215782823）

Hi @venci01, I'm no longer able to reproduce this in cesium master, so I'm going to go ahead and close this.


Bug report: Commit 20c02ff broke something concerning building example /label_image for Raspberry Pi

This is the diff in question:
https://github.com/tensorflow/tensorflow/commit/20c02ffe0e6b8af4902487f852e15daa16caa523#diff-48887acc98c15e53f942842ecd65190fR318
I managed to successfully build this example by changing code. Since I'm not really good with C++ I'm not sure if my solution is the best. It would be great if someone more experienced could take a look at it.
Here is the issue where I explain how I fixed it: https://github.com/tensorflow/tensorflow/issues/5200

See #5200, PR coming.


German localization misses Extensions viewlet title

The Extensions viewlet title should be Erweiterungen and it seems that it has changed to Extensions. Not sure if this was a purposeful decision from the localization team.
Commented out the smoke test assertion for now: https://github.com/microsoft/vscode/commit/709a07d51919d3266ca71699c6ddfb2d3547c0e1

I think it is more likely that code got shuffled around and this needs a new LP from the translation team which should come with when we release 1.37. Let's see if this fixes it.

Actually, there is no a mix of Extensions and Erweiterung in the UI even embedded in German sentences. So it doesn't look like a LP issue.

@danyeh can you please make sure this gets consistent, either using Erweiterungen or Extensions. But the current form is unhappy.

Strings localized correctly in a MLCP and json file.
https://github.com/madcoder2k17/visualstudio-code/blob/master/i18n/deu/src/vs/workbench/parts/extensions/browser/extensionsActions.i18n.json
"showOutdatedExtensions": "Veraltete Erweiterungen anzeigen",
"showPopularExtensions": "Beliebte Erweiterungen anzeigen",
"showRecommendedExtensions": "Empfohlene Erweiterungen anzeigen",
"showWorkspaceRecommendedExtensions": "Für den Arbeitsbereich empfohlene Erweiterungen anzeigen",
"showRecommendedKeymapExtensions": "Empfohlene Tastenzuordnungen anzeigen",

The strings are fixed in the freshly published language pack (1.38.1):

Please close this issue if you are satisfied with the fix.


Game crash on load screen

https://paste.atlauncher.com/view/7842c12d-942f-4f9a-b590-b72eb932a2fd

Thank you for the report!
It looks like you are reporting that the game Minecraft crashed for you. Please try the following:

Update your graphics drivers, see this Minecraft Help Center article, section "Update your Video Card Drivers";
often Minecraft crashes are caused by outdated graphics drivers
In case you are using mods, try disabling them one by one until you find the mod causing the crash, then report the crash to the mod author
If you are using a third-party launcher, try the official launcher instead

If none of the above helped, and you are still experiencing the crash with the official launcher and without any mods installed, please submit this issue over at the Mojang bug tracker. Please search for existing reports first; in case you do not find any, create a new report and let us know about the issue number here (e.g. MC-123456).
The Mojang team will take the first look. If an OpenJDK bug is identified, the Mojang team will contact the Microsoft Build of OpenJDK team to address the issue.


one ip address works, one times out

When I start the phonegap server it appears to alternate between two IP addresses: <IP_ADDRESS>:3000 and <IP_ADDRESS>:3000.  When I enter these into the phonegap developer app, the first one works successfully.  With the second one, the app tries to open but returns to the start screen and when I submit the second time it returns error - time out.
What this means in practice is that I have to restart the server until I get the working IP address.  Note that I'm trying both addresses on the same network, and both my pc and device (iOS) were not disconnected from the network after I started the server.

Hi @jegarne,
This is an interesting issue that I've never experienced. It looks as though your network subnet mask is configured differently than most networks that I've used.
The next time that this happens, can you enter the server address into your device's browser? If the browser connects to the server, you'll see your app but deviceready will not fire. I'm simply curious whether a default browser can reach the server - if it can't, we may see a meaningful error message.

Closing since we haven't seen very much activity on this issue. Feel free to reopen.


Deploys are failing due to docker pulls

Deploys to preview started failing yesterday. The error message is:
Pulling binfmt Docker image
  /usr/bin/docker pull tonistiigi/binfmt:latest
  Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
Error: The process '/usr/bin/docker' failed with exit code 1

Let's fix that.

Moving the auth steps earlier in the container build workflows resolved the pull problem, but now we've got a genuine build error that's clearly new. Continuing investigation in #2218.


[Bug] Non interactive charts on mobile

Version
5.1.2
Link to Minimal Reproduction
No response
Steps to Reproduce
Select options;
{
      animation: false,
      title: {
        text: total,
        textStyle: {
          fontSize: 28,
        },
        subtextStyle: {
          fontSize: 18
        },
        textAlign: 'center',
        subtext: 'Total',
        left: '24%',
        top: '36%'
      },
      tooltip: {
        trigger: 'item'
      },
      legend: {
        orient: 'vertical',
        left: '50%',
        top: 'middle',
        icon: 'circle',
        selectedMode: false,
        align: 'left',
        formatter: name => {
          const value = this._data.filter(row => row.name === name)[0].value;
          if (value > 100) {
            return `${value}\n${name}`;
          }
          return `${value} ${name}`;
        },
        textStyle: {
          fontSize: 18,
          // lineHeight: 24
        },
        itemGap: 20
      },
      color: ['#8AAD70', '#D7A675', '#A38874'],
      series: [
        {
          type: 'pie',
          center: ['25%', '50%'],
          radius: ['60%', '85%'],
          avoidLabelOverlap: true,
          data: this._data,
          emphasis: {
            itemStyle: {
              shadowBlur: 10,
              shadowOffsetX: 0,
              shadowColor: 'rgba(0, 0, 0, 0.5)'
            }
          },
          itemStyle: {
            normal: {
              label: {
                show: false,
              },
            },
          }
        }
      ]
    };

Current Behavior
I have a pie chart which is selectable on iOS which makes it impossible to scroll if the user clicks on the chart. E.g. having a finger on the screen trying to scroll the page down is snatched up by echarts.
Expected Behavior
I would like to make the chart not interactable - just a static image, no clicking into it so a user can scroll down normally. I cannot find the option to make this happen in the API.
Environment
- OS: iOS15.3.1
- Browser: safari
- Framework: angular

Any additional comments?
No response

This was an issue with the interaction with HammerJS + Echarts. I've fixed it from Hammer side (tap event).


No options to define nullability of fields

Currently there is no way to set the nullability of a field and when calling the get_structtype function  to get the schema the nullability is hardcoded as true.
https://github.com/kaiko-ai/typedspark/blob/a0c03e8cce8cac630d02516d6347209666f335c8/typedspark/_schema/structfield.py#L42

Hi @ram-seek !
We've previously played around with making nullable a parameter in the schemas, but it turned out that in pyspark nullabillity is pretty dynamic. For example, consider the following example, where column b switches from nullable to non-nullable and back again.

Given how counter-intuitive this works, it really didn't add much to have nullability as an option in the schema. I think nullability is mostly used under the hood by Spark for optimizations.
Does this answer your question? Or do you have a different scenario in mind in which you'd like to set the nullability?

@nanne-aben  thanks for that, what I was trying to do was use typedspark for schema definitions and also for function signatures.
Schemas I work with are complex and highly nested and using typedspark to define the schema makes it more manageable. I could also reuse some of the classes in other nested structures like so.

another place I was looking to use typed spark is to define functions and validate the types of the argument as well the return types.

Yup, that's exactly what typedspark is for!
Although for the last code block, you should use concat() rather than addition. In Spark, you can't add StringType columns.
from pyspark.sql.functions import concat, lit

df.withColumn(
    Address.street.str,
    concat(
        Address.street,
        lit(" "),
        Address.number,
    ),
)

You could also consider using transform_to_schema() (doc):
from typedspark import transform_to_schema

transform_to_schema(
    df,
    Address,
    {
        Address.street: concat(
            Address.street,
            lit(" "),
            Address.number,
        )
    },
)

Lemme know if you have any other questions!

sorry If I was not being clear, the sample code was just to illustrate the use of datasets in the function signatures and the implementation of the function. The point I was trying to make was , for schema definitions and function signatures it is essential to have the nullability of the column defined in the schema.

Do you mean that you want to be able to define which columns are nullable, so that you can be certain street and number are not null and hence you're not concatenating something with a null column?
If so, I see your point, but that's not really how Spark works. Suppose we do define nullability in the schema somehow:
class Address(Schema):
    street: Annotated[Column[StringType], ColumnMeta(nullable=False)]
    number: Annotated[Column[LongType], ColumnMeta(nullable=False)]

df = create_partially_filled_dataset(spark, Address, {Address.street: ["high street"], Address.number: [1]})

We might think of four ways to enforce that.
First option: make sure that df.filter(Address.street.isNull()).count() == 0 (and idem dito for Address.number). It's effective. But it will trigger a compute of the df (remember that Spark is evaluated lazily), which I don't want to do for every cast.
Second option: make sure that df.schema[0].nullable == False (and idem dito for df.schema[1]). However, this nullable attribute can be True even when there are no Null values at all. This happens a lot, for example when you load a hive table (see the example in my first post in this issue).
Third option: overwrite the value such that df.schema[0].nullable = False. This doesn't enforce anything though, you can set it to False on a column full of Null values.

Fourth option: create a new df after changing the schema:
df.schema[1].nullable = False
new_df = spark.createDataFrame(df.rdd, df.schema)

Unfortunately, it is super costly to do this.
Hence, none of the options are great. In my opinion, typedspark is not the best way to enforce nullability, because nullability is something you should check all the way at the end, when you're computing your entire df, not at every intermediate step.
I know DBT offers exactly this: a way to test for nulls in your data after you've computed the entire table. DBT also works well together with typedspark: DBT for the pipeline, typedspark for development in your IDE. We have some code that automatically transforms schemas between DBT and typedspark.
If DBT is too heavy of a solution, then the best would be to add some assertions that check df.filter(Address.street.isNull()).count() == 0.
Lemme know if that answers your question!


Domain and server management

Issue description
Heyo, first off, awesome stuff you making here :)! second my issue is that i've been trying to get this up and running on a server, vultr ubuntu 16 server, now i can only go into the streama page by going "localhost" or the ip "xxx.xxx.xxx.:8080" but as i got a domain for it, is it poseble to add that domain for it? i've tried to first just go to the ip, without the port, but didnt work out, i've been checking around in some files and tried to edit the url in application file etc, didnt find anything. im kinda new to specielly groovy, and never even heard of it, im sorry if im asking a dumb or just out right wrong question here, but thanks in advance, and sorry for my grammar as well!
-benno1214
Steps to Reproduce

go to localhost/ip of server and then use port 8080, that works fine.
go to only ip/the domain that is set to the ip of the server, doesnt work at all, nothing.
tried to edit applications and edit the url set of the application also tried to change ports and look into other files what i could find, nothing.

Expected Behaviour
get the appliction working on the server ip/domain and not only localhost
Actual Behaviour
only works on localhost:8080
or the server ip (server localhost) :8080
Environment Information

Ubuntu 16.04 x64: TODO
1.5.2.war: TODO
Custom streama build, describe customizations or provide link to fork (If Applicable): TODO
Container Version (If Applicable): TODO

Example Application / access to your live version
If avalible

Please refer to this doc:
https://github.com/dularion/streama/wiki/FAQs#8-how-do-i-change-the-port


Logout Users automatically, when token expired ...

fastapi token is 1440 minutes (1 day) valid
cookie.set is set to 7 days


Warning about missing package manager variable

When I startup a project that has vite-serve included I get the following warning each time:
The \"VITE_JS_PACKAGE_MGR\" variable is not set. Defaulting to a blank string.
If I add the variable with an empty string .env the message disappears.
Is this intended?
vite-serve's auto dectection works very fine for me so I don't understand why I should still add a variable with no content?

This is a known issue, and relates to how auto-detection works and interacts with the YAML parser that DDEV uses. You can definitely ignore this.
I'm not sure what happens if you specify an empty string, but if it does not cause trouble for you and makes the error disappear, that's fine.

Yep, it doesn't cause any trouble to specify the variable as empty string.
Thanks for the information, I'll close this issue.


Fixed p2p section

I went through, fixed up a bit of the grammar/wording of a few things.
Updated the styling so that code will line wrap, that way all code can be seen before copying it.
For now, I have only updated MAINNET, TODO: TESTNET.

Morning!  Okay, I have updated what was changed on my end (IE: the rpc server) in the instructions for state sync.  Commit 0326b00 will just contain the edits for all currently viewable documentation.  It just contains the initial validator setup instructions with state sync.
bd7aba3, this part will contain all of the maintenance additions that I have added.

State Sync for current validators
Pruning for current validators
Running Sentry's

I have a live feed of this documentation that you can read here:  http://erialosofastora.asuscomm.com:8080/docs/getting-started/

I just saw the comment above to change to 2VDL.  Let me fix that, Sorry about that.

4892f35
I felt it best to add the warning regarding unsafe-reset-all when it comes to how the priv_state.json file holds the validators last signed block info.


about remote mode

The program will start a TUN network card for communication
Do you have an idea of a mode that can directly establish a p2p TUN network card through udp (fake tcp)
without forwarding just direct communication?

Similar to https://github.com/dndx/phantun/issues/60?]
Short: it is possible, but it is not a priority for me ATM.


Athena is expecting an outputLocation ?

I have been receiving this error since yesterday:
validation error detected: Value null at 'outputLocation'
This is my connection string:
   conn = connect(
        s3_staging_dir='s3://aws-athena-query-results-{}-us-east-1/'.format(env.get('AWS_ACCOUNT_ID')),
        region_name='us-east-1',
        access_key=kms.decrypt(access_key_encrypted),
        secret_key=kms.decrypt(secret_key_encrypted),
    )

You may find the cause by setting the Trace log output as follows.
conn = connect(s3_staging_dir='s3://YOUR_S3_BUCKET/path/to/',
               region_name='us-east-1',
               LogLevel='6',
               LogPath='./')


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 976 ms

Resolved: Bofrost.de performance has improved in 128ccab.


Merge jazzy ➡  ros2

➡  Forward port
Port jazzy  ➡  ros2
Branch comparison: https://github.com/gazebosim/ros_gz/compare/ros2...jazzy
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

CI issues should hopefully be fixed after https://github.com/ros2/ros2/pull/1574


:tada: init

Description
Llama Index should have an out-of-the-box callback that allows the user to stream all internal events through an async iterator.
The current streaming misses all the internal events like tool usage and agent actions.
This async iterator should also stream token events as well.
Fixes # (issue)
Type of Change
Please delete options that are not relevant.

[ X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
This is still in draft and needs input and fixes.

@jordanparker6 are you still picking away at this? If not one of us can probably figure this out sometime in the coming week. It feels like a useful feature :)

Hey,
Sorry, I have add to move to high priority tickets. Was going to get back to it in time but happy for you guys to take it on.
Jordan

Going to close this one out for now, its quite outdated at this point, especially given instrumentation has been introduced to replace the callbacks system
If its still an issue, please raise an updated PR
Sorry for letting this one sit for so long!


Suggest UTF-8 character encoding for all text files (e.g., text/html)

Rationale
In addition to small text/plain documents, Tika detects some small text/html documents as IBM500 character encoding (see https://issues.apache.org/jira/browse/TIKA-2771). The WikiLongTest WIKI_PAGE7 Markdown Test document showed this problem.
Related Pull Requests

https://github.com/LabKey/platform/pull/2186

Changes

Suggest UTF-8 character encoding for all text files

WikiLongTest passes with this change: https://teamcity.labkey.org/viewLog.html?buildId=1387817&buildTypeId=bt20&tab=testsInfo&branch_LabkeyTrunk=improve_encoding_detection

SMWorkflowSearchTest is failing... sometimes. But @labkey-danield confirmed that it passed once on this FB and that the tests has failed in a similar manner on develop.


Test - Resend Invitation

Test Description:
Create a test to check if the resend invitation functionality is working as expected

Implemented tests successfully - Pull Request #30


Aspire deployment - An unhandled error has occurred. Reload

After using Azure Developer CLI to deploy to Azure, when I view the Aspire Dashboard, I got this error
There was an unhandled exception on the current circuit, so this circuit will be terminated. For more details turn on detailed exceptions by setting 'DetailedErrors: true' in 'appSettings.Development.json' or set 'CircuitOptions.DetailedErrors'.

Initializing an app to run on Azure (azd init)
? How do you want to initialize your app? Use code in the current directory
(✓) Done: Scanning app code in current directory
Detected services:
.NET (Aspire)
Detected in: C:\dotnet\sample\eShop\src\eShop.AppHost\eShop.AppHost.csproj
azd will generate the files necessary to host your app on Azure using Azure Container Apps.
? Select an option Confirm and continue initializing my app
? Enter a new environment name: [? for help] eshopdev
? Enter a new environment name: eshopdev
Generating files to run your app on Azure:
(✓) Done: Generating ./azure.yaml
(✓) Done: Generating ./next-steps.md
SUCCESS: Your app is ready for the cloud!
You can provision and deploy your app to Azure by running the azd up command in this directory. For more information on configuring your app, see ./next-steps.md
PS C:\dotnet\sample\eShop\src\eShop.AppHost> azd up
? Select an Azure Subscription to use:  1. Visual Studio Professional Subscription
? Select an Azure location to use: 16. (Asia Pacific) Southeast Asia (southeastasia)
Packaging services (azd package)
Provisioning Azure resources (azd provision)
Provisioning Azure resources can take some time.
Subscription: Visual Studio Professional Subscription
Location: Southeast Asia
You can view detailed progress in the Azure Portal:
https://portal.azure.com/#view/HubsExtension/DeploymentDetailsBlade/~/overview/id
(✓) Done: Resource group: rg-eshopdev
(✓) Done: Log Analytics workspace: law-mn6lmqyu7hkig
(✓) Done: Container Registry: acrmn6lmqyu7hkig
(✓) Done: Container Apps Environment: cae-mn6lmqyu7hkig
Deploying services (azd deploy)
(✓) Done: Deploying service basket-api

Endpoint: https://basket-api.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service catalog-api

Endpoint: https://catalog-api.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service eventbus

Endpoint: https://eventbus.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service identity-api

Endpoint: https://identity-api.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service mobile-bff

Endpoint: https://mobile-bff.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service order-processor

Endpoint: https://order-processor.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service ordering-api

Endpoint: https://ordering-api.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service payment-processor

Endpoint: https://payment-processor.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service postgres

Endpoint: https://postgres.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service redis

Endpoint: https://redis.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service webapp

Endpoint: https://webapp.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service webhooks-api

Endpoint: https://webhooks-api.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

(✓) Done: Deploying service webhooksclient

Endpoint: https://webhooksclient.internal.proudcliff-xxx.southeastasia.azurecontainerapps.io/

Aspire Dashboard: https://aspire-dashboard.ext.proudcliff-xxx.southeastasia.azurecontainerapps.io
SUCCESS: Your up workflow to provision and deploy to Azure completed in 11 minutes 1 second.
PS C:\dotnet\sample\eShop\src\eShop.AppHost>

Fixes are this are rolling out to various regions.


HostnameMatch should probably default to Domain instead of Exact

@jpeach do you happen to remember if we had a reason for the Exact default here? I can't remember why we did this. More context here: https://github.com/kubernetes-sigs/service-apis/pull/336#discussion_r492272218.

@jpeach do you happen to remember if we had a reason for the Exact default here? I can't remember why we did this. More context here: #336 (comment).

I think the defaults are correct as they are. If you omit the Listener.Hostname field, the default is "Any", which matches any requested hostname. If you specify this field, then the default for the struct is Exact so you only have so specify the exact hostname to match (I expect this to be the most common usage).
I don't think that we should default HostnameMatch.Match to "Any" because it's not the dominant use case, and we should optimize the YAML grammar so that the most common cases can be the most succinct.

I agree that Any doesn't make sense as a default here, but I can see an argument for Domain as a default. Do you think there's a compelling reason to choose Exact matching as the default instead of the more lenient Domain match?

I agree that Any doesn't make sense as a default here when a hostname is specified, but I can see an argument for Domain as a default. Do you think there's a compelling reason to choose Exact matching as the default instead of the more lenient Domain match?

Domain creates ambiguity with certificates (bad for operators) and encourages wildcards (bad for the ecosystem). I also expect that domain matching is also a less common need than exposing a specific hostname.

👍 That seems reasonable enough as long as we make it clear for users what our defaults are. I'll close this for now, anyone can feel free to reopen to discuss further.

Copying Daneyon and Harry before I resolve related feedback on API cleanup PR.
/cc @danehans @hbagdi


Settings button not showing up

When you first install the extension it shows you where to find the settings button, but it's not there.
Also I don't seem to be able to save settings either, which is a problem.

Which browser do you use and its version? It would also be useful for me if you could tell me if you're using Tampermonkey/Greasemonkey to install YouTube Center or the native extension.

Not OP, but I'm having the same problem with Firefox 36.0 and the native FF add-on.

I am using Chrome, the most recent released build.
I just installed the extension by turning developer mode on and dragging the extension into Chrome.

I'm pretty sure that doesn't work. If you want to just drag the extension in like that you need to be using the developer version of Chrome, not just turn on developer mode in the normal one. To get the extension to run in normal Chrome you need to do something more fiddly like use the Group Policy Editor or load an unpacked extension through developer mode.
Personally, I'd suggest just using Tampermonkey (unless you happen to be using a Pro version of Windows and have the Group Policy Editor). The only real downside is that it's going to lag when there's a lot of embedded videos on a page, but it's the easiest install method currently and doesn't need any fiddling around.

Are you using 2.1.5 or a dev build of YTC? A while ago youtube rearranged its UI, removing the cog button next to the Upload button. This meant that to access the configurations menu, one had to uninstall and reinstall the extension. The dev version adds a new button to open up the menu, but the instructions that appear when you first install YTC still show an outdated screenshot:

I'm using 2.1.7 and I see the icon in neither of those places :/

I didn't realise there was a new version (I thought 2.1.5 was the newest stable build). After seeing your comment I downloaded 2.1.7 and it still wasn't fixed. I was talking about the unstable developer version, v489, which you can get here: https://github.com/YePpHa/YouTubeCenter/wiki/Developer-Version


Clean-up ANTHROPIC_TOOL_USE_META_PROMPT

Description
Context: introduced here: https://github.com/dust-tt/dust/pull/8637 and then made obsolete here: https://github.com/dust-tt/dust/pull/8651 without proper clean-up.
Risk
N/A
Deploy Plan

deploy front

PMRR


Timed out trying to read data from the socket stream!

I saw this posted on here a couple of times but it doesn't seem to apply to what I'm doing.
I've set up credentials on a CentOS 6 box for SFTP and can use putty's SFTP client to connect a-ok.
I then use the following code:
               if (szUserName != string.Empty && szPassword != string.Empty)                     oCreds = new NetworkCredential(szUserName, szPassword);                   FtpClient client = new FtpClient(szAddress, nPort, oCreds);                   client.Connect(); // <--- this is where I get the message / exception
I've verified that the Server address, username, and password are all what I used in the SFTP client.
I'm using VS2017, FluentFTP 18.0.2, Windows 10 (client), CentOS 6 (server)
Any ideas why I may be seeing this?

Can you try with the latest build : https://www.nuget.org/packages/FluentFTP/19.0.0

I just changed the code to this:
`                FtpClient client = new FtpClient(szAddress);
client.Credentials = new NetworkCredential(szUserName, szPassword);
            client.Connect();`

And now get a different error:  No connection could be made because the target machine actively refused it <IP_ADDRESS>:21
I can use putty sftp to make a connection just fine...
I went back into the original code and changed the port to 21 and the same error as above showed.  I do believe that SFTP should be port 22?

It just occurred to me...  I don't see any reference to downloading the SSH certificate from the server...  This could be the issue?

It looks like you've changed the description on the library to ftps from sftp (which are very different services).  It works well as ftps.  You can close this as I think it may have been a typo in the description.

Description? Typo? AFAIK I have always mentioned FTPS, never SFTP, as I am well aware that they are completely different protocols. Where is the typo?

It did say SFTP and it still comes up in NUGET when you search for SFTP…
<EMAIL_ADDRESS>From: Robin Rodricks<EMAIL_ADDRESS>Sent: Thursday, October 5, 2017 12:22 AM
To: robinrodricks/FluentFTP
Cc: Scott Johnson; Author
Subject: Re: [robinrodricks/FluentFTP] Timed out trying to read data from the socket stream! (#208)
Description? Typo? AFAIK I have always mentioned FTPS, never SFTP, as I am well aware that they are completely different protocols. Where is the typo?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/robinrodricks/FluentFTP/issues/208#issuecomment-334362386, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APR11W7Grixok0Vq-dE29_qtfuweqwflks5spGdpgaJpZM4PlHXc.


ek.data.from_source, No longer supports the CDS since the update to the split infrastructure CDS.

Is your feature request related to a problem? Please describe.
Since the update to the CDS, the CDS is no longer available as a source in the from_source function.
Additionally the EWDS, ECDS and the ADS will need to be supported.
The key has changed format for the User Key for the CDS User so this needs to be taken into account.
Describe the solution you'd like
add EWDS and other CDS Infrastructures to the possible list of from_source.
Describe alternatives you've considered
Using the cdsapi
Additional context
No response
Organisation
ECMWF

@enyfeo, please can you me an example that shows how you want to access the CDS? What earthkit-data and cdsapi version are you using?
Not sure about the comment related to changed format for the User Key. earthkit-data normally does not have anything to do with it since the credentials are loaded by cdsapi that performs the retrieval under the hood.

Not sure about the comment related to changed format for the User Key. earthkit-data normally does not have anything to do with it since the credentials are loaded by cdsapi that performs the retrieval under the hood.

I think they are referring to scenario where the user has neither the environment variables set nor a credentials file. In this case, earth kit provides a prompt to provide the URL and the key. However, the regex fails as the key format is now different. This is the related announcement.
See below error:

An API key is needed to access this dataset. Please visit
https://cds.climate.copernicus.eu/ to register or sign-in
then visit https://cds.climate.copernicus.eu/api-how-to to retrieve you API key.
You can store the credentials in ~/.cdsapirc or in the file
pointed by the CDSAPI_RC environment variable. If you follow
the instructions below ~/.cdsapirc will be automatically created
for you.
Alternatively, you can use the CDSAPI_URL and CDSAPI_KEY environment variables to specify the credentials.
API url: https://cds.climate.copernicus.eu/api
API key:
Traceback (most recent call last):
File "./ccri/era5.py", line 10, in
ds = earthkit.data.from_source("cds",
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/init.py", line 150, in from_source
src = get_source(name, *args, **kwargs)
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/init.py", line 131, in call
source = klass(*args, **kwargs)
File ".venv/lib/python3.10/site-packages/earthkit/data/core/init.py", line 22, in call
obj.init(*args, **kwargs)
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/cds.py", line 121, in init
self.client()  # Trigger password prompt before threading
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/cds.py", line 173, in client
return client(self.prompt)
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/cds.py", line 80, in client
prompt.check()
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/prompt.py", line 174, in check
self.ask_user_and_save()
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/prompt.py", line 191, in ask_user_and_save
input = self.ask_user()
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/prompt.py", line 188, in ask_user
return self.validate(prompt.ask_user())
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/prompt.py", line 108, in ask_user
value = validate(value)
File ".venv/lib/python3.10/site-packages/earthkit/data/sources/prompt.py", line 66, in call
assert re.fullmatch(self.pattern, value), (self.pattern, value)
AssertionError: ('\d+:[\-0-9a-f]+', 'my-redacted-api-key')

@spmvoss, thank you for the explanation. This will be fixed. In the meantime you can still turn off the prompt for the cds source by specifying  the prompt=False kwarg  in from_source()

See #504
The  EWDS and ECDS sources will be added to the next major release 0.11.0.

Hi Chris,
Good to hear it works now. Nevertheless, there were issues, which are now fixed in #504. It was released as0.10.9.


Search Editor peek support for notebook search

Open the search editor and search for something in an open notebook
Double-click on the name of the file in the result.
Doesn't show preview :bug:

I actually see now that open notebooks aren't even showing in the search editor. Will investigate.

https://github.com/microsoft/vscode/issues/200172


Fix Image -> Texture2D inspector auto-conversion

Regression from #71683
The PR replaced the base type with a list of all sub-types, so the condition was never true.

Does this fix the crash when creating an Image resource straight from the inspector?

idk, I didn't encountered such crash.
But I did run into #77752, so maybe it's somehow related.

Thanks!


DirectXTK Sprites Init Crash

Hello guys! I'm desperated to fix this error. My renderer is DX11 and I've already included all headers and the DirectXTK.lib so the problem is when my program reachs this code in runtime 6a5cf6e1439f7facb5d8b192510daad9.png
https://gyazo.com/6a5cf6e1439f7facb5d8b192510daad9
where pSpriteBatch is defined like this:
std::unique_ptr<SpriteBatch> pSpriteBatch;
This is the DXInitializer:
void DX11Renderer::InitDevice()
{
DXGI_SWAP_CHAIN_DESC scd;

ZeroMemory(&scd, sizeof(DXGI_SWAP_CHAIN_DESC));
scd.BufferCount = 1;
scd.BufferDesc.Format = DXGI_FORMAT_R8G8B8A8_UNORM;
scd.BufferDesc.Width = ScreenX;
scd.BufferDesc.Height = ScreenY;
scd.BufferUsage = DXGI_USAGE_RENDER_TARGET_OUTPUT;
scd.OutputWindow = DXWindow;
scd.SampleDesc.Count = 4;
scd.Windowed = TRUE;
scd.Flags = DXGI_SWAP_CHAIN_FLAG_ALLOW_MODE_SWITCH;

///DEVICE INIT
D3D11CreateDeviceAndSwapChain(NULL, D3D_DRIVER_TYPE_HARDWARE, NULL, NULL, NULL, NULL, D3D11_SDK_VERSION, &scd, &DX11SwapChain, &DX11Device, NULL, &DX11DeviceContext);

DX11SwapChain->GetBuffer(0, __uuidof(ID3D11Texture2D), (LPVOID*)&BackBuffer);

DX11Device->CreateRenderTargetView(BackBuffer, NULL, &DX11RenderTargetView);
BackBuffer->Release();

DX11DeviceContext->OMSetRenderTargets(1, &DX11RenderTargetView, NULL);

///VIEWPORT INIT
D3D11_VIEWPORT viewport;
ZeroMemory(&viewport, sizeof(D3D11_VIEWPORT));

viewport.TopLeftX = 0;
viewport.TopLeftY = 0;
viewport.Width = ScreenX;
viewport.Height = ScreenY;

DX11DeviceContext->RSSetViewports(1, &viewport);

///FONTS INIT
pSpriteBatch.reset(new SpriteBatch(DX11DeviceContext)); // <<-- crash
pPurista12.reset(new SpriteFont(DX11Device, L"font.spritefont"));

///BUFFER INIT
D3D11_BUFFER_DESC bd;
ZeroMemory(&bd, sizeof(bd));

bd.Usage = D3D11_USAGE_DYNAMIC;
bd.ByteWidth = sizeof(VERTEX) * 30;
bd.BindFlags = D3D11_BIND_VERTEX_BUFFER;
bd.CPUAccessFlags = D3D11_CPU_ACCESS_WRITE;

DX11Device->CreateBuffer(&bd, NULL, &VertexBuffer);

///BLEND INIT

///SHADER INIT
InitRenderShaders();
}

When it reachs the line it shows me this exception 02589a97c1d19a0e5a02aaae9103ecd1.png
https://gyazo.com/02589a97c1d19a0e5a02aaae9103ecd1
and this is the breakpoint details 5aca89f56827d44d5a938b412aef40f9.png
https://gyazo.com/5aca89f56827d44d5a938b412aef40f9
To be specific, the code will crash when it reachs this code from the file 'mutex.c':
5a320b954b101b87e97fa7623913ad1a.png
https://gyazo.com/5a320b954b101b87e97fa7623913ad1a
Does anybody any idea why this happens? I'm building it as Release in x64 bits.
Thanks in advance.

So there are a lot of places where your code could have gone wrong, but your code is not checking for failed HRESULTs. For example D3D11CreateDeviceAndSwapChain could have failed immediately and you have no indication of that until you crashed later. As such, I suspect your problem is indeed that your DX11DeviceContext is invalid at the time you try to set up your SpriteBatch.
You should take a little time to work through the tutorials and read this topic in particular.


Wrote startupandshutdown test file and modified NewTestController to include a value for OncePerServerBlock

Thanks for taking the time to figure this out Patel! :+1:


Handle Outputs in derived Stacks

I want to be able to write a base class for a stack, which will be reused for a few projects which have separate, but similar infrastructures. Currently, the Output properties defined on the base class are not being registered in the derived stacks because of the BindingFlags.DeclaredOnly flag used in the Stack.RegisterPropertyOutputs method.
In this PR I remove this flag. Instead, I explicitly cut off properties from ancestors of the Stack class (currently the Resource class has Urn output property and I believe this was the reason to add the DeclaredOnly flag in the first place).
Check the test I added to see exactly what I wanted to achieve.

@Frassle thanks for the approve. It looks like some checks failed in the Merge Queue (same that passed in the PR, doesn't look correlated with my changes). Any idea what happened and how can we merge this PR?

Odd, we'll try another merge if that fails I'll dig into what's up. The PR test passed so it probably isn't due to this change.

Yeh that test is failing on main now, something must have changed in the CLI. Raised https://github.com/pulumi/pulumi-dotnet/issues/252 to fix it.

Thanks, can't wait to use it 👍


Reloading host files on Ubuntu

Hi
Reloading on ubuntu is outdated. Should be
sudo service network-manager restart

Thanks for the project!

HI @SUlbrichA and welcome here!
What is your version of Ubuntu ? Why are you talking about service when much Ubuntu are now under systemd/systemctl (since 2014) ?
Cheers,

Hi
the command on the readme did not work under 18.04 so I googled for
refresh/flush DNS and that was on the first hit
Plus, it worked :)
If there is a better alternative I'd be happy to know it (have not tried to
change startup scripts in a looong while)
Cheers
Stef
On Fri, Aug 10, 2018 at 6:02 PM Nissar Chababy<EMAIL_ADDRESS>wrote:

HI @SUlbrichA https://github.com/SUlbrichA and welcome here!
What is your version of Ubuntu ? Why are you talking about service when
much Ubuntu are now under systemd/systemctl (since 2014) ?
Cheers,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StevenBlack/hosts/issues/750#issuecomment-412128239,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ackq4q8PcYVIph2u83_s8X_ySaQjpC23ks5uPa6igaJpZM4V352c
.

@SUlbrichA Normally sudo systemctl restart network.service should work under Ubuntu as it use systemd :thinking:

Thanks everybody.  When I regenerate the next release, it will say
sudo service network-manager restart

... for Ubuntu.  This is straightforward and easier to remember, I feel.
Closing.  Thanks Stefan @SUlbrichA!


Empty generated enum file

I have an api-docs.json which looks like this. I have stripped down some unrelated stuff.
{
  "openapi": "3.0.1",
  "components": {
    "schemas": {
      "TestlogResponse": {
        "type": "object",
        "properties": {
          "id": { "type": "integer", "format": "int64" },
          "testType": {
            "type": "string",
            "enum": ["WEB", "MOBILE_ANDROID", "MOBILE_IOS"]
          },
          "testResult": {
            "type": "string",
            "enum": ["SUCCESSFUL", "ABORTED", "FAILED", "IN_PROGRESS"]
          }
        }
      }
    }
  }
}

Command line I used to generate the enums: genum-openapi docs/api/api-docs.json -o src/lib/api/enums.ts
But the enum.ts is an empty file.

Hi. Are you able to pinpoint the version of CLI where the conversion worked properly for your file?

No, this was my first time with this package.

I probably know what the issue is.
There's notihing wrong with your file. When comparing to the shape of  json within the test folder:
{
  "openapi": "3.0.3",
  "info": {
    "title": "Example of the API document",
    "version": "0.0.0",
    "description": "Description of the API document"
  },
  "components": {
    "schemas": {
      "AgencyStatus": {
        "enum": ["ACTIVE", "ARCHIVED"],
        "type": "string"
      }
    }
  }
}

I see that the enums are deeper than on the level one (in your case in the properties object). In the source code of the CLI, there's function which is looking for enum key (and some other conditional statements) only on the first level of object, right after TestLogResponse key in your case.
Well, given that the object with properties key is still a valid OpenAPI document, the CLI will need an improvement in this matter to properly handle enums placed deeper.
I'm thinking about the name of the enum i.e. for provided example by you.
export enum TestlogResponse__testType {
  WEB = "WEB",
  MOBILE_ANDROID = "MOBILE_ANDROID",
  MOBILE_IOS = "MOBILE_IOS",
}

export enum TestlogResponse__testResult {
  SUCCESSFUL = "SUCCESSFUL",
  ABORTED = "ABORTED",
  FAILED = "FAILED",
  IN_PROGRESS = "IN_PROGRESS"
}

If this output look good to you i will work on the improvement of the CLI in the coming days.

I have the same enum keys in different object on the same level. This is for a request:
...
"TestlogRequest": {
  "type": "object",
  "properties": {
    "date": { "type": "string", "format": "date" },
    "testType": {
      "type": "string",
      "enum": ["WEB", "MOBILE_ANDROID", "MOBILE_IOS"]
    },
    "testResult": {
      "type": "string",
      "enum": ["SUCCESSFUL", "ABORTED", "FAILED", "IN_PROGRESS"]
    }
  }
},
...

Well, I'am fine with both of the enum names you provided. Maybe pick a default one, and add a CLI option for the other one?

Yeah, I'm considering adding an additional CLI option. By default, generated enums will follow the format of those in the second example. However, with the --with-parent option, the generated enums will adopt the format of the first example to prevent name collisions in such cases.
I'll let you know once the feature is implemented and the new package is released on npm.

Hi!
I've just released 0.8.0-alpha.0 version of the package with the improved function for collecting deep nested enums along with the --with-parent option. Try it and let me know if the generated file is not empty and enums are collected properly :wink:

Looking good:
enums.ts default
export enum testType {
  WEB = "WEB",
  MOBILE_ANDROID = "MOBILE_ANDROID",
  MOBILE_IOS = "MOBILE_IOS",
}

export enum testResult {
  SUCCESSFUL = "SUCCESSFUL",
  ABORTED = "ABORTED",
  FAILED = "FAILED",
  IN_PROGRESS = "IN_PROGRESS",
}

enums.ts with --with-parent
export enum TestlogResponse__testType {
  WEB = "WEB",
  MOBILE_ANDROID = "MOBILE_ANDROID",
  MOBILE_IOS = "MOBILE_IOS",
}

export enum TestlogResponse__testResult {
  SUCCESSFUL = "SUCCESSFUL",
  ABORTED = "ABORTED",
  FAILED = "FAILED",
  IN_PROGRESS = "IN_PROGRESS",
}

Probably I will use the --with-parent option, because I just saw an enum with a type name in my schema, which is not so self explanatory :) I got it from the backend.
Thank you for your work!

Awesome :tada: I'm glad it works.
In the meantime I'll fix issues rised by linter and typescript and release it as a proper v0.8.0. Then close this issue.
But for you nothing should change :wink:

I might have find another example where the enum is not generated:
---
openapi: 3.0.0
components:
  schemas
    PaymentMethod:
      required:
        - type
      description: "A Payment Method is a pure-virtual super-class that defines a specific method of payment such as Direct Debit, Cash, Digital Wallet,Tokenized Card, Bank Account Transfer, Bank Account Debit, Bank Card, Voucher, Check, Bucket PaymentMethod, Account PaymentMethod, and Loyalty PaymentMethod with all details associated. Use the @type attribute to specify the concrete type in the API calls."
      allOf:
        - $ref: '#/components/schemas/BaseObject'
        - type: object
          properties:
            type:
              type: string
              description: "Type of payment method (e.g.: bank card, cash, voucher, etc)."
              enum:
                - none
                - cash
                - online
                - bank-transfer
                - token-online
            period:
              $ref: '#/components/schemas/Quantity'

The problem might be similar to the previous one. I'll look on it and let you know.

Yep. Almost there, should be ready for tommorow :smile: Just doing some small improvements and typechecking fixes.

Hey @borzaka
As promised, I've released new version to the npm registry 0.8.0-alpha.1. Let me know if it resolves the issue with collecting enums from allOf properties. (allOf, oneOf, anyOf, and not should be supported :smiley: )

Thank you!
More enums got generated as before, so it's definitely better! :)
I still found some enums which are missing from the generated file. Can you tell me that they should be included, or not?
This:
components:
  schemas:
    PatchDocument:
      required:
        - op
        - path
      properties:
        op:
          type: string
          description: Művelet kódja
          enum:
            - add
            - remove
            - replace
            - move
            - copy
            - test
        path:
          type: string
          description: "[JSON pointer](https://tools.ietf.org/html/rfc6901) arra az elemre, amire a művelet vonatkozik \\n"
        value:
          type: object
          description: A művelet adat része
        from:
          type: string
          description: "[JSON pointer](https://tools.ietf.org/html/rfc6901) arra a forrás elemre, amire a művelet vonatkozik  \\n"

And this:
paths:
  /shopping-carts/{cartId}/lovs:
    get:
      tags:
        - ShoppingCart
      summary: Az adott kosáron érvényes LOV-ok lekérdezése
      description: Az adott kosáron érvényes LOV-ok lekérdezése
      operationId: listShoppingCartLov
      parameters:
        - name: lovType
          in: query
          description: LOV típus kódja
          required: false
          style: form
          explode: true
          schema:
            type: string
            enum:
              - PrimaryPersonIdCardTypeList
              - SecondaryPersonIdCardTypeList
              - MNPTimeWindowList
              - MNPOperatorList
              - SIMTypeList
              - TelcoBillingModeList
              - DeliveryModeList
              - DeliveryTimeWindowList
              - PickupPointList
              - ShopList
              - OrderPaymentModeList
              - BillingEntityList
              - MSISDNAssignmentModeList
              - DeliveryCarrierList
          example: PrimaryPersonIdCardTypeList

I've just released 0.8.0-alpha.2 to the npm. I was excessively checking if type === "object" property exists to fetch enums from properties. Now it should collect PatchDocument__op correctly.
As for the listShoppingCartLov the CLI is currently collecting enums only from the components object, not from the paths. I'll look into types definition and rethink if the CLI should also collect all enums from the paths object :thinking:


max characters

'The right side of the error text is hidden in report. Making difficult to understand what the problem is'
https://bsi-technicalservices.atlassian.net/browse/VAL-203
The issue appeared only when the browser was not maximized in the screen. If it was (or in fullscreen), a scroll bar appeared below.
A solution is to allow only for 75 characters in a row of a message and, if exceeded, display the rest of the text in a new row.
Now, the result does not overflow.

Did you experiment with overflow-wrap: break-word; in CSS. Manually breaking based on actual characters in JS has some downsides: not every character has same width and changes selection behaviour.

overflow-wrap: break-word;
Didn't work unfortunately. I've tried a couple of things, in the end what worked is modifying the whitespace and wordBreak variable.
          ".pre": {
            whiteSpace: 'pre-wrap',
            wordBreak: 'break-word',
          }

Looks good, but have a look at how it looks with syntax errors as well please.


bugs about bounds.

The width and height of FocusBorderView doesn't fit the ItemView well if the root layout of ItemView has margins. Even of 0 margins, the bounds don't act well either.

call 'setSelectPadding(int left, int top, int right, int bottom)' function to controll the width and height of FocusBorderView


hue passthrough requires link button press on reboot

Hello,
I love this project--extremely useful!
One problem I cannot seem to solve is that rebooting / restarting ha-bridge requires that we re-press the link button on our hue bridge in order to enable passthrough control of the hue lights. Upon pressing the link button, all is well again, but this is not optimal if the power ever goes out or just for general reboots.
Here are the two errors that popup on the server:

Popup seen on the website states: "Request Error, Pleae look in your habridge log: error {"type":101,"address":"","description":"link button not pressed"}"
Logs record "registerWithHue needs link button pressed on HUE bridge: living room primary".

Any idea how to get a more "permanent" registration saved in the config file or db so that the link button doesn't need to be pressed every time the software restarts?
Thanks and keep up the good work!
Nick

I should put a note out there, if you do a save in the bridge control tab after you have all controllers attached to the ha-bridge, it will save all the usernames it has accumulated. The press the button issue wont occur again for the ha-bridge.

I still can't seem to get it to stick. Here is the procedure I followed:

Start with a fresh startup (i.e., no data folder)
In the web user interface add IP address and name ("myhue") in the "Bridge Control" form.
Click "Save". Bridge reinitializes and "Hue Devices" tab appears.
Press the physical link button on my Hue bridge.
Click on the "Hue Devices" tab--all Hue lights appear in the list. As I understand from the hue api, this means the handshake between ha-bridge and the hue bridge worked, and ha-bridge should now have a username.
Click on the "Bridge Control" tab and click "Save".

When re-entering the "Hue Devices" tab, the lights no longer appear in the list and the "Logs" show the error "registerWithHue needs link button pressed on HUE bridge: myhue". Adding a light (i.e., add bridge device) before performing the save does not help.
Am I performing the steps in the correct order? Are you sure the hue bridge-provided username is being set and reused from the database?
Thanks! Nick

Ahh, did not realize you were using the passthrough. That's interesting as the real bridge should be OK as i use the same ID to it Everytime as the user it needs. Will take a look.

I have same issue, I have to press button on hue bridge everytime I need to reboot my raspberry pi.

Just dropped in to say same here. Similar setup as OP. On reboot I have to press relink on the hue.

And the same here after restart. :)

Well, on restart there is no saved userid for that Hue. I will look into a way to save that for each hue. It will still have to be once.

Looks like there is a way and will take a one more step after you get the bridge linked to the hue hub, you will have to go to the bridge control tab ant hit save when it is done.


memory leak

code below

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>d3-timer memory leak</title>

  <script src="https://cdn.jsdelivr.net/npm/d3-timer@3"></script>
  <script>
    const t = d3.timer((elapsed) => {
      // run your code
    }, 150);
  </script>

<body>
</body>
</html>

start in the browser

after 5 min

@hustcc  I'm starting to use this timer in some of my projects and saw this issue.
I'm using the test script you provided and the memory footprint goes back down after about a minute or so, goes back up, then back down.


Set preceding/succeeding pathType to range scope

Moved this PR from https://github.com/provectus/kafka-ui/pull/3933

[x] Breaking change? (if so, please describe the impact and migration path for existing application instances)
Yes, but I'm not sure if anyone was using this correctly to begin with. See explanation below.

What changes did you make? (Give an overview)
#3259 introduced the ability to set the prefixType, however the preceedingPaths and succeedingPaths reference a variable that is out of scope of the range.
          {{- range .Values.ingress.succeedingPaths }}
          - path: {{ .path }}
            pathType: {{ .Values.ingress.pathType }}
            backend:
              service:
                name: {{ .serviceName }}
                port:
                  number: {{ .servicePort }}
          {{- end }}

In the range section, .Values.ingress.pathType refers to .Values.ingress.succeedingPaths[n].Values.ingress.pathType which I don't think was the intention.
This change switches it to .pathType, which refers to .Values.ingress.succeedingPaths[n].pathType, similar to path, serviceName, and servicePort variables.
Before this change in order to get the pathType set you'd need to configure the preceding/succeeding path to:
    succeedingPaths:
      - path: "/kafkaui/"
        Values:
          ingress:
            pathType: Prefix
        serviceName: kafkaui-service
        servicePort: 80

After this change it would reference be set by:
    succeedingPaths:
      - path: "/kafkaui/"
        pathType: Prefix
        serviceName: kafkaui-service
        servicePort: 80

Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Set new schema value and verified helm deployment worked.
Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)
Meet my dog Lupin!

@skooNI hey, sorry for the delay, figured out that I haven't been subscribed to this particular repo so far. We'll take a look

@skooNI Hello
Seems your changes can break ingress if only .Values.ingress.pathType value is set in values.yaml.

@skooNI thanks for your contribution!


FR: Version controlled Configmaps

Describe your feature request (FR):
As a user, I would like my configmaps to be version controlled in Git. This helps keep a track of all the changes over a period of time. It also helps in mitigating scenarios where someone with Admin access to an app, accidentally deletes its configmap.
Possible Solution:
While saving my configmap, it should also be committed to my app's config repository owned by Devtron app.

Configmap, Secrets and deployment template versioning is handled on the UI now although it is not committed to the gitops repo. Hence, closing this as the usecase is addressed by the current implementation.


Upgrade from version 5 to 6 migration

In version 5 we used to do something like these:
return SentryEvent(
        release: (await PackageInfo.fromPlatform()).version,
        environment: 'production',
        extra: extraParams,
        exception: exception,
        stackTrace: stackTrace,
      );

exception and stackTrace are not available anymore. Is there a migration guide?

The migration guide is here.
SentryEvent now supports a list of exceptions and the stacktraces is now a property of SentryThread.
See https://pub.dev/documentation/sentry/latest/sentry_io/SentryEvent/SentryEvent.html constructor.
I hope that helps.
May I ask why you create the event yourself?

you can still call:
var event = SentryEvent(
  throwable: throwable,
);

and call Sentry.captureEvent(captureEvent, stackTrace: stackTrace)

@ueman Sorry, I hadn't seen that before. I looked at this, but we don't have stackTrace property anymore.

May I ask why you create the event yourself?

It wasn't me, I'm refactoring this. Isn't this the expected way?
I see in docs this way Sentry.captureException(exception, stackTrace: stackTrace) but I guess in our way we pass more infos.

@joaovirgilicw take a look at the official docs, release and environment can be passed only once during SDK init btw (SDK detects a few things automatically already eg. release).
extra can be set by the Sentry.configureScope API (see docs)
let us know if that helps then we can close the issue, thanks

@marandaneto release, environment and  extra aren't the problem. My problem is that exception and stackTrace aren't available anymore.
But the exception I can pass via exceptions new property as exceptions: [exception], right?

@joaovirgilicw as stated before, stackTrace is a method param from the capture methods eg captureException(event, stackTrace: stackTrace)
if your exception is of the type SentryException, then yes exceptions: [exception], otherwise, see my comment above https://github.com/getsentry/sentry-dart/issues/601#issuecomment-927797526
if you want to capture the exception that comes from the catch blocks, you can use the throwable field from the SentryEvent ctor.

@joaovirgilicw did it work for you? let us know, thanks

@marandaneto I didn't have time to test it, sorry. I will do it asap

Closing the issue as a part of large repository cleanup, due to it being inactive and/or outdated. There's a large chance that this issue has been solved since.
Please do not hesitate to ping me if it is still relevant, and I will happily reopen and work on it.
Cheers!


Added filter state outputs and did some minor refactoring of the C++ device examples

@microstrain-sam I merged the other two PRs and merged the updated develop branch with those into this one. Let me know if you need anything else from me for this!