Conformance: update objects in MustApplyObjectsWithCleanup

What type of PR is this?

What this PR does / why we need it:
MustApplyObjectsWithCleanup updates the objects if they are already available available
Which issue(s) this PR fixes:

Fixes #2230
Does this PR introduce a user-facing change?:

Bug Fixes : Updates the resource objects instead of keeping them same if the resource already exists

/ok-to-test

/test pull-gateway-api-verify

/cc

@LiorLieberman  any further suggestions on the updated on code ?

Thanks @sadath-12 , apart from two small comments, looks good to me. Did you test it locally ? I can help with explaining how if you need.

Thank you @LiorLieberman , Yes I tried creating a secret in the cluster with same resource specifications but with different data and passed it to this function (which creates the secret with another data) and that seems to give me the modified data content on the secret

Can you test that the conformance tests are passing?
Using the makefile I sent you for example in envoy-gw. (
https://github.com/envoyproxy/gateway/blob/554176b8cbbc5ffa627dc7109e1b3d2b59474e99/tools/make/kube.mk#L95
)
Remove the  “delete-cluster” from the end of the line, it will spin up a
kind cluster with meta lb and will leave it there.
Then you can run locally from the gateway-api repo folder (that contains
your changes).
one with cleanup  go test -v  ./conformance --gateway-class=envoy-gateway --cleanup-base-resources=true
and one without  go test -v  ./conformance --gateway-class=envoy-gateway --cleanup-base-resources=false
w
On Wed, 13 Sept 2023 at 07:40, syedsadath-17 @.***>
wrote:

Thanks @sadath-12 https://github.com/sadath-12 , apart from two small
comments, looks good to me. Did you test it locally ? I can help with
explaining how if you need.
Thank you @LiorLieberman https://github.com/LiorLieberman , Yes I tried
creating a secret in the cluster with same resource specifications but with
different data and passed it to this function (which creates the secret
with another data) and that seems to give me the modified data content on
the secret
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/gateway-api/pull/2380#issuecomment-1717035233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLANQQQ6NY3QNITNBP6ALTX2FIN7ANCNFSM6AAAAAA4OGL3MY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Looks like this PR has gone stale so I'm going to close it out for now. Feel free to reopen if you find time to work on it again.


Is there a way to encrypt the generated default.isar

Is there a way to encrypt default.isar? For example, Hive has an Encrypted box method that encrypts the box, and the key needs to be passed when opening the box, does Isar have a schema for this?

This is not possible in the v3, but it will be supported in v4, using the SQLite backend.


otherwise not working after match block returns false

When I use match block and otherwise, I want to skip some addresses in match so they would be caught by otherwise block. But it doesn't go to otherwise block.
RubyDNS.run_server(listen: INTERFACES, asynchronous: false) do
  upstream = RubyDNS::Resolver.new([[:udp, "<IP_ADDRESS>", 53], [:tcp, "<IP_ADDRESS>", 53]])

  match(/^([\d\.]+)\.in-addr\.arpa$/, IN::PTR) do |transaction, match_data|

    domain = nil # just for test

    if domain
      transaction.respond!(Name.create(domain))
    else
      # Pass the request to the otherwise handler
      # !!! this doesn't work
      false
    end
  end

  otherwise do |transaction|
    transaction.passthrough!(upstream)
  end
end

When I return false from match block - it doesn't go to otherwise block.
What am I doing wrong?

https://github.com/ioquatix/rubydns/blob/8b8d43e426f10b2486a61e2d7f63221f9709409d/lib/rubydns/rule_based_server.rb#L158-L161
Please see the documentation here. Thanks :)


Add proxy for ansible for local development

Signed-off-by: Kevin Cormier<EMAIL_ADDRESS>

/cc @chenz4027

Fix included in https://github.com/stolostron/console/pull/1078


Bluetooth: tester: Change RPA timeout

This changes RPA timeout so that the address is changed every 60
seconds. This change is needed to speed up privacy related test cases.
Currently, the new RPA was initiated on request that was a bit hackish
and could be done via BT_RPA_TIMEOUT Kconfig option.
Signed-off-by: Mariusz Skamra<EMAIL_ADDRESS>

Please rebase this one @MariuszSkamra

Manual backport is included in #18122


Needs updating for X

It doesn't work any more since the domain is now X.com.

I’m sorry, but as I don’t use Twitter anymore, I’m not going to update the extension for X.


homework-1

KDU homework-1

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
54

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Handle exception thrown PlatformEventHandler deconstructor

Catch exception when accessing Application::Current in WindowsRTPlatformEventHandler deconstructor like in the constructor as both have the same risk of throwing the exception.
Manually tested that it fixes the issue where the exception was thrown on the first deconstruction of a PlatformEventHandler on Windows.

@j388923r - Would you like to merge the changes?

@lalitb Yes, I don't think I have write access to merge, so if you know who could do it for me, that would be great.


importing withdrawn records

I think we should decide how to represent withdrawn records and import the withdrawn records where we can parse the log string for the new record (usually "withdrawn in favor of...").

I agree that the withdrawn resources should be imported. Why not simply use:
[] adm:status bdr:StatusWithdrawn

which can be filtered on as in the legacy system?

yes, we already have the properties so it should just be a matter of parsing the log strings

@xristy what's the correct property to indicate that an entity has been withdrawn as being a duplicate of another? there's adm:replaceWithIndividual, but it seems to be about individuals in vocabularies... ?

We can add adm:replaceWith: rdf:Resource ==> rdf:Resource which should allow to replace :Person by a :Person or a :Topic or an :Event or whatever is needed. adm:replaceWithIndividual can be made a subproperty.

sounds good!


[Presentation API] Adds a receiver page that accepts connections & shows fruits

@beaufortfrancois

@zhaobin2016

Okay to merge @beaufortfrancois ?

PTAL @beaufortfrancois

LGTM with nits

@beaufortfrancois Can this be merged now?  I don't have write access. Thx

Just merged! It is now live at https://googlechrome.github.io/samples/presentation-api/

I've added this sample to https://www.chromestatus.com/feature/5414209298890752 as well

And removed trailing whitespaces at https://github.com/GoogleChrome/samples/pull/516


links to examples and notebooks broken in the landing page

This was due to a deletion of "develop" branch. All links point to "master" now, and this one ain't going anywhere!


Added Linked_List_Quick_Sort in cpp

Assign it to me

There is no assigning.  Whoever wants to can submit a pull request and it will be reviewed.


feat: first version of hooks library

Description

first version of hooks library

Related issue(s)
See also https://github.com/asyncapi/generator/issues/318

@fmvilas @Tenischev I pushed a change to have createAsyncapiFile supporting parameters as mentioned during review and also made nicer docks. Have a look please
https://github.com/asyncapi/generator-hooks/pull/1/commits/26c8b857f6c462f9ee28a5d7f518825970c36339


Hardenings

Codecov Report

Merging #215 (39bfdfe) into main (812bd98) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##              main      #215   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         1    -1
  Lines           93        30   -63
  Branches         4        16   +12
=========================================
- Hits            93        30   -63

Flag
Coverage Δ

linux-clang-14
53.57% <ø> (-46.43%)
:arrow_down:

linux-gcc-12
100.00% <ø> (ø)

macos-clang
?

windows-cl
?

windows-clang
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes


How can I troubleshoot PrometheusRule CRDs?

What did you do?
How does one verify that PrometheusRule objects are being picked up and utilized by the prometheus operator correctly?  I've added 63 rule objects to kubernetes:
% k get prometheusrules.monitoring.coreos.com
NAME                                                              AGE
alertmanager                                                      1m
chefs                                                             59s
commit-workers                                                    58s
dashboard-gitlab-com-status                                       57s
deadman-switch-alerts                                             57s
dev-gitlab-orgs                                                   56s
error-rates                                                       55s
<snip>

Looking at the logs of the operator, we see this line item every time a file is applied:
level=info ts=2019-07-06T14:21:29.889515026Z caller=operator.go:1050 component=prometheusoperator msg="sync prometheus" key=monitoring/gitlab-monitoring-promethe-prometheus

However, I can't find a rule files in the prometheus Pod, and going to the prometheus /rules endpoint confirms that my rules are not being applied to the running Pod.
The running prometheus instance only has one location specified to look for rules files:
/etc/prometheus $ head config_out/prometheus.env.yaml
<snip>
rule_files:
- /etc/prometheus/rules/prometheus-gitlab-monitoring-promethe-prometheus-rulefiles-0/*.yaml
<snip>

I see the rules that get shipped with the operator helm chart here but not any of my custom rules:
/etc/prometheus/rules/prometheus-gitlab-monitoring-promethe-prometheus-rulefiles-0 $ ls
monitoring-gitlab-monitoring-promethe-alertmanager.rules.yaml                    monitoring-gitlab-monitoring-promethe-kubernetes-resources.yaml
<snip>

Environment

Prometheus Operator version:
v0.31.1

Kubernetes version information:
v1.13.6-gke.13

Kubernetes cluster kind:
GKE

Manifests:

Example rule:
---
apiVersion: monitoring.coreos.com/v1
kind: PrometheusRule
metadata:
  name: alertmanager
spec:
  groups:
  - name: alertmanager.rules
    rules:
    - alert: AlertmanagerNotificationsFailing
      expr: rate(alertmanager_notifications_failed_total[1m]) > 0
      for: 1m
      labels:
        pager: pagerduty
        service: alertmanager
        severity: s1
      annotations:
        title: Alertmanager is failing sending notications
        runbook: troubleshooting/alertmanager-notification-failures.md
        description: Alertmanager is seeing errors for integration {{$labels.integration}}

Anything else we need to know?:
Prometheus Operator installed via stable/prometheus-operator helm chart version 5.14.1.  The custom rule files were manually created.  All rules were correctly entered into Kubernetes without error.  Only the rules that ship with the helm chart are being inserted into prometheus.
I'm unable to determine a few things:

How long does it take the operator to recognize and insert a new rule into prometheus, from kubectl apply -f custom-rule-file.yaml to the rules endpoint of prometheus telling me it's processing my rule?
Are there other logs I should be looking at to determine where my rules may be?  I suspect maybe there's a validation/format problem with my rules, but I can't find this type of detail.
Does the namespace for which rules live matter?  Currently they all reside in the same namespace as the operator.

You probably are not specidying the Selectors currectly- Check the labels of the default rules and apply them to your file. Also the selector rules can be recearched on the Prometheus object and Alertmanager object

That is precisely what it was.  I had no idea there was a selector in play for this.  Thank you for the pointer.
The documentation for the stable helm chart is incorrect as there are values set for the selector.  To answer my own questions above:

How long does it take the operator to recognize and insert a new rule into prometheus

Roughly a minute

Does the namespace for which rules live matter?

Yes there's an option for this, if using the stable/prometheus-operator helm chart, ruleNamespaceSelector, this defaults to all namespaces.


Selecting a row should add a CSS class to the row

Selecting a row should add a CSS class to the row. The class added should be something like "select-row". The select-row should use a different default color other than odd-row and even-row.

I got this done through some workaround code like this:
handleSelectRow(row, rowIndex) {
    if (this.selectedRow) {
      const prevSelector = `table.${this.props.className} tbody tr:nth-child(${this.selectedRow})`;
      const prevNode = document.querySelector(prevSelector);
      prevNode.className = this.selectedRow % 2 ? 'even-row' : 'odd-row';
    }
    const selector = `table.${this.props.className} tbody tr:nth-child(${rowIndex + 1})`;
    const node = document.querySelector(selector);
    node.className = 'select-row';
    this.selectedRow = rowIndex + 1;
    this.props.onSelectRow(row);
  }

Preferably, odd-row, even-row, select-row classes should be part of reactabular render.

Check the onRow part here. I do a check like row.selected && 'selected-row' there. The class related logic lives outside.
What you suggested can work, but it's likely neater to go through onRow.

Thanks for sharing the select api. Having the classnames in the onRow does not work for me. Whenever the state changes and there is a re-render, the onRow is not called. I think onRow is called only for new rows or rows with new keys. Because the onRow is NOT called, the selected row does not get the class.

The underlying logic expects the selection state lives at row level. If it's separate, the rendering logic won't be able to detect the change by default. That could explain it.
I could make it possible to override the current shouldComponentUpdate. There's a flag to disable it, but maybe there's a better way.

The select api works fine. The code should not use rowIndex but use the row.id as follows:
handleRowEvent(row, { rowIndex }) {
    return {
      className: row.selected && 'select-row',
      onClick: () => this.handleSelectRow(row)
    };
  }

  handleSelectRow(row) {
    this.setState({ selectedRowId: row.id });
    this.props.onSelectRow(row);
  }


28911 - change Diagnostic.code to 'string | number'

Fixes #28911

@ajaff ok, will add changes to DiagnosticWithLinePosition, DiagnosticRelatedInformation.

The unfortunate thing here is that we're going to cause so many bugs where people create a Set<number | string>  and end up using .has on the wrong type.

Hey @amcasey, can VS handle non-numeric diagnostic codes?

@DanielRosenwasser, I believe so, but I'll confirm.

@DanielRosenwasser I misspoke - Roslyn can accept strings but TS deserializes them as int so old versions of VS will break (probably by dropping these diagnostics).

Sounds like this API would also need to support a new option on configuration to say that the client can support string error codes. Even if we decided to add this functionality, we would need to keep numeric error codes around f o r e v e r.
As terrible as it sounds, it might be better to return a new property on Diagnostic objects.

@weswigham

@DanielRosenwasser Your suggestion doesnt sound terrible. Infact i like having optional friendlyName property sounds good. That way even we could add the names to error codes (instead of numbers) and editors can decide to use them over number if present to displayed. We should keep errorCode required so that editors not handling the additional optional property can show something that user can search about.

Friendly human readable short names are good. Although question, if we go there: Can multiple messages share a short name (so, eg, we can group all the excess property variant messages under one "name")? I don't really see why not.

I think this PR has to be closed.

@DanielRosenwasser just to clarify on https://github.com/Microsoft/TypeScript/pull/28959#issuecomment-447091812  I feel like a configuration option isn't necessary if we simply add a new optional friendlyName field - errors that support friendly names would always populate the friendlyName and old editors would just ignore it.


React step 5

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@baraaabdelsalam Almost there 😎! Could you please specify the url to take a look at your deployment? The url https://carbon-tutorial-baraa-abdelsalam.mybluemix.net/#/ gives me 404. In addition I noticed strange white space around the main content that should not be there.


Cloning Device or IP with a tag applied results in a Server Error

Environment

Ubuntu 20.04
Python version: 3.8.5
Nautobot version: 1.0.2

Steps to Reproduce

Install NautoBot 1.0.2 per the directions on https://nautobot.readthedocs.io
Create a Site, Manufacturer, Device Role, Tag, and then a Device with all applied
Click clone on the created device and receive a Server Error
Remove the tag from the device (so there are no tags at all) click Clone and it works
Issue also occurs with IPs

Expected Behavior
Clone button works without error allowing the device to be cloned

Observed Behavior
Server Error like the following - the invalid UUID is not always 'a', it changes alphabet or number
Server Error
There was a problem with your request. Please contact an administrator.
The complete exception is provided below:
<class 'django.core.exceptions.ValidationError'>
['“a” is not a valid UUID.']
Python version: 3.8.5
Nautobot version: 1.0.2
If further assistance is required, please post to the #nautobot channel on NetworkToCode's Slack.

Thanks for the report! This may be related to #499.


Mismatch between values in CLI man page and documentation

In the section "Retrieve or create necessary IIS/Kudu files" it specifies that you can use Typescript or Javascript as arguments to the --lang flag:

Retrieve or create necessary IIS/Kudu files

For C# bots
cmd

az bot prepare-deploy --lang Csharp --code-dir "." --proj-file-path "MyBot.csproj"

You must provide the path to the .csproj file relative to --code-dir. This can be performed via the
--proj-file-path argument. The command would resolve --code-dir and --proj-file-path to >"./MyBot.csproj"

For JavaScript bots
cmd

az bot prepare-deploy --code-dir "." --lang Javascript

This command will fetch a web.config which is needed for Node.js apps to work with IIS on Azure App >Services. Make sure web.config is saved to the root of your bot.

For TypeScript bots
cmd

az bot prepare-deploy --code-dir "." --lang Typescript

This command works similarly to JavaScript above, but for a Typescript bot.

This is not correct, as the help page for the  --lang flag states:
[joseph@epiphytelcl ~]$ az bot prepare-deploy --help
Command     az bot prepare-deploy : Add scripts and configuration files to your local source code directory     to be able to publish back using az webapp deployment source`
Arguments     --lang [Required] : The language or runtime of the bot.  Allowed values: Csharp, Node.
Running the command as specified in the documentation results in the following behavior:
[joseph@epiphytelcl ~]$ az bot prepare-deploy --code-dir "." --lang Typescript az bot prepare-deploy: 'Typescript' is not a valid value for '--lang'. See 'az bot prepare-deploy --help'. [joseph@epiphytelcl ~]$ az bot prepare-deploy --code-dir "." --lang Javascript az bot prepare-deploy: 'Javascript' is not a valid value for '--lang'. See 'az bot prepare-deploy --help'
Proposed Solution:
Please update the documentation to reflect the values of the CLI man page, or update the code to reflect the documentation
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 096d934c-2856-e868-efeb-6622e2a5eafb
Version Independent ID: 8bf74926-6a35-5753-0a1a-6663ba5965b2
Content: Deploy your bot - Bot Service
Content Source: articles/bot-builder-deploy-az-cli.md
Service: bot-service
Sub-service: abs
GitHub Login: @ivorb
Microsoft Alias: v-ivorb

@os2tux - what is the version of az cli you are using?

Iqbal:
I’m using v2.0.63 of the Azure CLI. Please let me know if you need anything else from me regarding this.
Thanks!
PS /home/joseph> az --version
azure-cli                         2.0.63 *
Respectfully,
Joseph Shelby
Sr. DevOps Engineer, EPS/Platform Architecture
zoom: https://asurion.zoom.us/my/joseph.shelby | slack: @jcshelby
voice:<PHONE_NUMBER> | mobile:<PHONE_NUMBER>
mail: Asurion I 648 Grassmere Park, Nashville, TN 37211
www.asurion.comhttp://www.asurion.com/ I Twitterhttps://twitter.com/Asurion I Facebookhttps://www.facebook.com/Asurion
From: Kamran Iqbal<EMAIL_ADDRESS>Sent: Monday, May 06, 2019 19:52
To: MicrosoftDocs/bot-docs<EMAIL_ADDRESS>Cc: Shelby, Joseph<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/bot-docs] Mismatch between values in CLI man page and documentation (#1118)
@os2tuxhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fos2tux&data=02|01|joseph.shelby%40asurion.com|c3e7fa114a7e40957ef508d6d286393e|64e5ad32cb0444df8896bed5d7792429|0|0|636927871261957292&sdata=UOEkw%2Ffqq66CpvYc8j6X1qJFeZfu57Cx5CEUQvEKqpE%3D&reserved=0 - what is the version of az cli you are using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fbot-docs%2Fissues%2F1118%23issuecomment-489864013&data=02|01|joseph.shelby%40asurion.com|c3e7fa114a7e40957ef508d6d286393e|64e5ad32cb0444df8896bed5d7792429|0|0|636927871261967304&sdata=D43LhQMNm8bMcaaUQricHFon2gNiQFrDLuMUA0dLvfw%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEOXHIGIMGUAIHEXPLDOSCTPUDHDHANCNFSM4HLB6ITA&data=02|01|joseph.shelby%40asurion.com|c3e7fa114a7e40957ef508d6d286393e|64e5ad32cb0444df8896bed5d7792429|0|0|636927871261967304&sdata=m1X%2F0r3SuH5ilQEH%2BeelqqtAVi7U1eqgJeZKsj%2BFfhc%3D&reserved=0.

This message (including any attachments) contains confidential and/or privileged information. It is intended for a specific individual and purpose and is protected by law. If you are not the intended recipient, please notify the sender immediately and delete this message. Any disclosure, copying, or distribution of this message, or the taking of any action based on it, is strictly prohibited.

@os2tux You should be using 2.0.64 version of azure cli to use TyepScript and JavaScripts with prepare-deploy command.

@os2tux For updates of the man page (which I agree should list all available languages, and give more information than you were receiving). Would you log an issue with the az cli? Feel free to also mention the issues you have with versioning numbers etc.
Once the CLI info matches what we have, it should be much clearer for users. Thanks for the feedback!


CI proposal: switch to renovate

At the moment this repo uses dependabot for semi-automated dependency bumps for both Go and JS dependencies. For JS dependencies this is not working anymore, because dependabot only bumps package versions in the respective package.json file but not in the pnpm lock file. Since we're installing dependencies in CI with frozen lockfile (i.e. pnpm lockfiles can't be updated in CI, only the current state of the lock file is allowed) the JS depency bumps by dependabot will always fail with an out of date lock file error. One example for this is https://github.com/owncloud/ocis/pull/4990 where I needed to update the pnpm lock file myself to get green CI (well, if it weren't for flaky tests).
I'd like to propose to switch to renovate entirely, i.e. for both Go and JS dependencies. The web repo already uses renovate (and our sysadmin team for a long time anyway). As an alternative we could run dependabot for Go and renovate for JS dependencies in parallel, but I'd like to avoid that complexity. So the first question is if we can switch to renovate entirely.
cc @micbar @wkloucek @C0rby

@micbar what's your opinion?

I would like to switch, i am just asking if the security requirements are covered by renovate.
@C0rby Any concerns?

Sorry, I was busy last week but will look into this today.

We can switch to renovate but we should make sure that vulnerability alerts are enabled and properly configured: https://docs.renovatebot.com/configuration-options/#vulnerabilityalerts

@micbar still relevant or should we close here?

Still relevant.


Astro 2

Astro 2 update

Reminder: Check for issue mentioned in https://github.com/smartcontractkit/documentation/pull/1206#pullrequestreview-1351728077

Reminder: Update data feeds addresses pages to mdx here. https://github.com/smartcontractkit/documentation/pull/1206#discussion_r1144237439

Needed to upgrade typescript version to resolve this error: https://github.com/withastro/astro/issues/6616

https://github.com/withastro/astro/issues/4590 suggests that we could specify preact as the jsxImportSource in our tsconfig.


[chore] Update angular defaults to support hmr

I think this needs to be done manually in the project.json of the client-side generated. One time deal though.


Swamp like biomes from Biomes You'll Go seem te be overwritten by biome makeover

When used in Combination with Biomes You'll Go, it seems Biome Makeover replaces the swampy biomes from BYG with it's own swamp biome (at least the vibrant and cold swampland biomes, cypress swamp did generate properly). Is this intended?

It's supposed to generate it's features in any modded Biome marked as a "swamp".
It shouldn't replace any modded exclusive features, but it'll still replace vanilla-style swamp trees.
Feel free to post some screenshots if you think something is wrong, otherwise I'll check it out sometime in the future.

It's supposed to generate it's features in any modded Biome marked as a "swamp".
It shouldn't replace any modded exclusive features, but it'll still replace vanilla-style swamp trees.
Feel free to post some screenshots if you think something is wrong, otherwise I'll check it out sometime in the future.

Oh my bad, I thought it was weird that there were only biome makeover trees in the biomes, but the biomes I mentioned (vibrant swamplands and cold swamplands) normally only have vanilla-style trees, so it makes sense.

Oh my bad, I thought it was weird that there were only biome makeover trees in the biomes, but the biomes I mentioned (vibrant swamplands and cold swamplands) normally only have vanilla-style trees, so it makes sense.


Team Section

Description
Created Team section.
Relevant GitHub Issues that will be fixed are: #141
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update
[ ] Chore (anything else)

How Has This Been Tested?

[x] Responsiveness on various devices
Desktop Browser

[ ] Apple Safari
[ ] Brave
[x] Google Chrome
[ ] Microsoft Edge
[ ]Mozilla FireFox

Mobile Browser

[x] Google Chrome
[ ] Mozilla FireFox
[ ] Microsoft Edge
[ ] Apple Safari

Other Tests:

Checklist

[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

PR Closure

Use Squash And Merge to Close Pull Request
Delete any branch after merging

Note: there are added Footer files, just ignore them for the purposes of this PR. It does not break any code or cause any problems, just a basic setup that made it through on the push to remote branch by accident.


numeric list functions

Per #25, implement the Sprig functions for Bartholomew. Rather than file an issue for each function, I figured I'd break these down into small batches. (Note: I am not entirely sure all of the functions can be done with the same syntax. I am no pro in writing the Handlebars functions.)

[ ] until COUNT generate a list of integers from 0 to COUNT.
[ ] until_step COUNT STEP generate a list of integers from 0 to COUNT incrementing by STEP
[ ] seq described here OPTIONAL... not sure we need it. Or maybe we need this and not until...

Closed with #133


⚠ TorrentLeech has degraded performance

In c863278, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 2806 ms

Resolved: TorrentLeech performance has improved in 05dbc2b after 6 minutes.


Support for ranch 1.2.1

Hi,
I am using lager (tag 3.2.1), with the version of ranch (1.2.1 ) I noticed that error_lager_lager_h crashes with a case clause of the form:
error_logger_lager_h exited with reason
{'EXIT',{{case_clause,[Ref,{ack,<0.3770.0>,{error,normal}}]}, -- Ranch CRASH EVENT (2 params)
[{error_logger_lager_h,log_event,2[{file,"/home/ubuntu/transecur/_build/default/lib/lager/src/error_logger_lager_h.erl"},{line,184}]}

Thanks for the report.


[BUG] Publish target fails when a package with the same name already exists

See pipeline execution log

This seems to happen when pushing from both ubuntu and mac-os agent (https://github.com/candoumbe/MiscUtilities/runs/4180566536?check_suite_focus=true).
The pipeline on the ubuntu agent completes successfully and packages are published to Nuget and Github with no error.
The pipeline on mac-os always fails on publish step even though the --skip-duplicate switch is set.

Dropped macOs pipeline https://github.com/candoumbe/MiscUtilities/commit/969d7c65be42294b73ac71a706a01a3410d14ce9


Better tooltips, with headings

Styling not included

@tehandyb and @YeahButHats can you guys pull in what is absolutely necessary from this? I am thinking it is just the mobile compat fixes.


MudDataGrid loses ability to scroll when Sticky is enabled along with Virtualize

Bug type
Component
Component name
MudDataGrid
What happened?
Hello,
I have sort of odd issue.
When I try to use StickyLeft="true" or StickyRight="true" to any of the PropertyColumn tags along side of attribute Virtualize="true" and ColumnResizeMode="ResizeMode.Container" attributes in MudDataGrid tag, the data grid losses ability to scroll. It is possible to view the issue in Reproduction link
Expected behavior
I expect to scroll over x and y axis when sticky attribute is enabled.
Reproduction link
https://try.mudblazor.com/snippet/GumHlkHkxiLSlkiU
Reproduction steps

Add MudDataGrid component
Add attribute Virtualize, Height, FixedHeader, 'ColumnResizeMode'
Add inside of MudDataGrid component add Columns tag
Add inside of Columns tag few PropertyColumn tags and one of them add attribute StickyLeft or StickyRight
Now you can only navigate via x axis but not via y axis

Relevant log output
No response
Version (bug)
6.11.0
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

If you look at the sticky column header when you scroll the x axis, it disappears behind the other column headers too.

Also with disabled virtualization I can't scroll vertically. I think sticky columns and fix height of the grid are the problem...


Use setuptools-scm to get migration from Git

Refs #242

Successful PR Checklist:

[ ] Tests

[ ] (not applicable?)

[ ] Documentation

[ ] (not applicable?)

[ ] Had a good time contributing?
[ ] (Maintainers: add PR labels)

@mgu If I understand it correctly, there may be a link between pep517-enabled (pyproject.toml) projects and pip install --editable.
This issue is closed so I'm not sure what the problem is.

After a long and fruitful discussion, we're adding python-build to the mix.

FWIW it looks great! :smile:

Do not forget about the PR labels.

Like my FedEx driver says, move fast & break things
(if things go wrong, we'll iterate)

So long that you're not an aircraft pilot… :-)


Reuse Route when Route errors with Insufficient_Fee

I propose a feature request where we try the same route and hence the same channel pair if our rebalance attempt fails because of Insuffienct Fees.
We already do this for when the probing fails, so it would be good to add it when our first rebalance fails.

hmm I guess when we want to take account for a feeupdate we could expand also to the other failures like:
time_lock_delta, min_htlc_msat  max_htlc_msat

This PR only considers changes in the cltv_delta and fee_update case.


Move away from git to pip for nbconvert

Bug: T308975

Tested against k8s 1.21. Wait for https://github.com/toolforge/paws/pull/148 and T308172 to resolve before merging.


The challenge is missing the original code in the editor (see) sample test 3 & 4

Challenge Return Early Pattern for Functions has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
// Setup
function abTest(a, b) {
  // Only change code below this line

  // Only change code above this line

  return Math.round(Math.pow(Math.sqrt(a) + Math.sqrt(b), 2));
}

// Change values below to test your code
abTest(2,2);

I don't understand what you're saying the issue is.
Are you saying that when you loaded the page there was no code?  If so, where did you get the code above?

@SaintPeter He is saying that tests 3 & 4 don't pass.
@tommygebru That's normal - you have to write code to pass the challenge :D

@AkiraLaine is correct. Thanks and happy coding!

sorry guys I just mean to say If i remember correctly the editor or the
question already had this
return "undefined";
and part of solving the entire challenge was to remove the quotations from
undefined
return undefined;
but i see that, was not the case, it has changed..... is it a bug or an
update to the challenge ?
On Sat, Jan 23, 2016 at 12:54 PM, Logan Tegman<EMAIL_ADDRESS>wrote:

@AkiraLaine https://github.com/AkiraLaine is correct. Thanks and happy
coding!
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6392#issuecomment-174221699
.

It's very likely that you clicked on the link to someone's incorrect code while you were in help chat which then saved itself to your local storage.
There have been no changes to the code of this challenge as far as I can tell,
https://github.com/FreeCodeCamp/FreeCodeCamp/blame/f813d33ac5c71f8dec8b4f945ba0e4ff8a99778d/seed/challenges/01-front-end-development-certification/basic-javascript.json#L3019-L3033

thanks! i understand now +1 Brian
On Sat, Jan 23, 2016 at 2:10 PM, Brian<EMAIL_ADDRESS>wrote:

It's very likely that you clicked on the link to someone's incorrect code
while you were in help chat which then saved itself to your local storage.
There have been no changes to the code of this challenge as far as I can
tell,
https://github.com/FreeCodeCamp/FreeCodeCamp/blame/f813d33ac5c71f8dec8b4f945ba0e4ff8a99778d/seed/challenges/01-front-end-development-certification/basic-javascript.json#L3019-L3033
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6392#issuecomment-174226167
.


powershell instructions in README don't include deploying build

OK so I've ran Invoke-OpenConsoleBuild and the build is done:
Build succeeded.

"C:\Users\mikem\Code\terminal\OpenConsole.sln" (default target) (1) ->
"C:\Users\mikem\Code\terminal\src\host\ft_uia\Host.Tests.UIA.csproj.metaproj" (default target) (34) ->
"C:\Users\mikem\Code\terminal\src\host\ft_uia\Host.Tests.UIA.csproj" (default target) (37) ->
(ResolveAssemblyReferences target) ->
  C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\MSBuild\15.0\bin\Microsoft.Common.CurrentV
ersion.targets(2110,5): warning MSB3245: Could not resolve this reference. Could not locate the assembly "M
icrosoft.VisualStudio.QualityTools.UnitTestFramework". Check to make sure the assembly exists on disk. If t
his reference is required by your code, you may get compilation errors. [C:\Users\mikem\Code\terminal\src\h
ost\ft_uia\Host.Tests.UIA.csproj]

    1 Warning(s)
    0 Error(s)

Time Elapsed 00:02:02.23

What do I run? I'm not familiar with .net, or VS solutions and the README doesn't say.

Note the README does say

make sure you're building & deploying CascadiaPackage in Visual Studio

But I (and many others) have no idea how to do this (or why, since we already just built the app in powershell per the instructions).
It seems like there's a command missing in the powershell instructions, which tells us how to either a) install or b) run the Terminal.

Right click on CascadiaPackage from Solution explorer and click Deploy. Then you will find Windows Terminal(Dev Build) in your Start Recently Added apps

@nntantidakis Thanks. But that seems to do an unnecessary second build - we've already followed the powershell instructions which built the app, per the README. Is there a way I can install the app from powershell?

There's a bit more discussion on the topic in #855 - let's keep the discussion to that thread.


BUG: Navbar is not proper in Contact Page

Describe the bug
In Contact page Navbar's height is very big and not align with login and logo of website.
Expected behavior
It should be properly align with other icons in Navbar.
Screenshots (Mandatory)
This is problem :

Are you contributing under any open-source program ?
Please Assign this task me Under GSSOC'24

Please Assign this to me under GSSOC'24.

@zalabhavy this issue is already present in the repo & already assigned to another contributor. Closing this issue.


Concurrency issue leading to corrupted uploads

I know the title is bit vague so let me explain.
During my tests I wanted to simulate big file uploads with relatively small files so on the front end I set:
simultaneousUploads: 3 (which is I believe default setting)
`chunkSize: 2048 (which is 1024*1024 by default)
This gave me kind of simulation of multiple chunk uploads. Using the library on the back end you have to follow four steps:

validateChunk()
saveChunk()
validateFile()
save()

If you have multiple concurrent uploads this may lead to corrupted files. Lets take a case of last two chunks being uploaded where x axis is time:
chunk1:  |---validateChunk---||---saveChunk---||---validateFile---||---save---|
chunk2:               |---validateChunk---||---saveChunk---||---validateFile---|

when validateFile is called on chunk1 the saveChunk for chunk2 already begun so chunk1::validateFile will return true and proceed to save file with chunk2 not fully saved.
I saw this error in my logs already few times. The above example will also lead to double save call. And in my case it leads to duplicate key error in the database.
To fix the problem library would have to implement locking not only in save but in validateFile methods . Its not enough to call file_exists in  validateFile. We have to know the chunk is fully uploaded.

I tried simultaneousUploads: 1, but it just stops there and doesn't upload the rest of the chunks.

I have this issue as well: simultaneous pdf uploads appear to work but the files arrive corrupted, sometimes it works and sometimes doesn't. I am using Ruby on Rails as my server.

Could you share your ruby on rails file handling code? Does it do locking?

I has a similar issue with some uploads not arriving, but reported as uploaded. I was getting errors like Unable to clear session lock record... and Failed to read session data: memcached... thrown at the same time.
Setting simultaneousUploads: from 3 to 1 solved the issue.


Add device path node types to uefi-raw

These nodes are very similar to the nodes generated in the uefi crate. The main difference is in DST nodes: instead of creating a trailing DST slice, a zero-length array is created instead. This ensures that pointers are thin, and matches the API style used elsewhere in uefi-raw.
Closes https://github.com/rust-osdev/uefi-rs/issues/1441
Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
[ ] Update the changelog (if necessary)

They are already optional in that they won't prevent the PR from being merged. I think I'd rather they didn't mark the PR red, though I'm not yet sure how to do that.

@kukrimate FYI I'm hoping this covers the same stuff as https://github.com/rust-osdev/uefi-rs/pull/1435 -- if you have time, please let me know if this works for your use case, or if there's anything missing.


[WIP] Schema Object and Array types

[WIP]
Adds support for Object and Array types at Schemas

Just remove [WIP] when it's ready to be reviewed @padiazg, so far I see no potential collision. 🚀


_SinkTransformerStreamSubscription is not a subtype of type 'StreamSubscription

Upgraded my Dart VM version from 1.16.1 to 1.17.1 running on OSX 10.11.5
I get following exception when using package connection_pool version 0.1.0+2 from pub.dartlang.org.
Unhandled exception:
ConnectionPoolException: Failed to open connection. Cause: type '_SinkTransformerStreamSubscription<List<int>, MongoReplyMessage>' is not a subtype of type 'StreamSubscription<List<int>>' of 'value' where
  _SinkTransformerStreamSubscription is from dart:async
  MongoReplyMessage is from package:mongo_dart/mongo_dart.dart
  List is from dart:core
  int is from dart:core
  MongoReplyMessage is from package:mongo_dart/mongo_dart.dart
  StreamSubscription is from dart:async
  List is from dart:core
  int is from dart:core

#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:938)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:41)
#2      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
#3      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:149)

I don't get any exception, when I downgrade my VM to 1.16.1.
Has something been changed in dart:async package for the new version of VM.

I recently improved the generic typing of the dart:async library.
I believe that the error message is correct: the _SinkTransformerStreamSubscription<List<int>, MongoReplyMessage> is a StreamSubscription<MongoReplyMessage> and not StreamSubscription<List<int>>.
This seems to be a bug in the mongo package.
I have created a pull request to update the type: https://github.com/mongo-dart/mongo_dart/pull/99
(Tbh I didn't test it. Maybe you could give it a try, and report to the maintainers of the package, if that fixes your issue).

Florian, thank you very much for PR.
I've just published v0.2.8 with that fix aplied.


NuGet Package Not Current

The classes I generated today using the NuGet package does not seem to create the ClientBase class. Is there some reason it's so out of date compared to the code here?
And on that note, since the NuGet package is out of date, how can I use this code directly? Simply building it and adding the DLL doesn't seem to do it, which isn't too suprising since I'm not sure how it's kicking off the auto-generated code on a build (honestly, that's a bit of magic to me!)

I just checked, the NuGet Package was last published on 1/15/2014. That's pretty out of date! There seem to have been a lot of commits in the last year!

Hi @Pharylon I haven't quite got around publishing a new version on Nuget, but will soon. Was busy dealing with safely merging all the other pull requests lately whenever I got a moment. I shall make time to push a more up to date copy to nuget soon, sorry for the inconvenience!
I am also considering a major refactor to the core design. This will solve the "magic" code generation and give you more control. Will keep you posted.


GitHub API open fails when behind proxy

Not sure what's causing this, but behind a pretty standard corporate web proxy that otherwise works properly (and worked with older versions of homebrew), we're now seeing:
Error: 407 "Proxy Authentication Required"
Please report this bug:
    https://git.io/brew-troubleshooting

Full traceback indicates the issue is in this function: https://github.com/Homebrew/brew/blob/master/Library/Homebrew/utils.rb#L599-L623
Setting the HOMEBREW_NO_GITHUB_API environment variable seems to be a temporary workaround.

@mikemcquaid - I think the issue may be in https://github.com/Homebrew/brew/commit/6135da800e4f93ded3a5a331d0efa691f4822afb but I'm a little lost in trying to bisect to the error.  Let me know if we can provide any help in testing or trying to provide a fix.
The way our proxy works, we set a username and password in the proxy url.  I'm wondering if the code here https://github.com/Homebrew/brew/blob/6135da800e4f93ded3a5a331d0efa691f4822afb/Library/Homebrew/cmd/gist-logs.rb#L122-L133 is somehow overriding our proxy environment variables?  We haven't set up GitHub credentials locally, so I'm not sure we should even be trying to login with them and instead should be falling back to unauthenticated requests.

The way our proxy works, we set a username and password in the proxy url.

It sounds like this may be interfering. Homebrew's proxy support is not great and I'm sorry that this behaviour (which makes things better for non-proxy users) has made things worse for you. It's pretty much impossible for me to create a fix without your proxy to test. Could you try and open a pull request? This document should help and I'm happy to walk you through anything else.
Thanks!

Thanks Mike, I'm pretty sure this is going to be biting a few people so I'll take a look at it this week if I get some time.

Okay, two ideas.

Don't try to login if there are no GitHub credentials stored on the system, instead just fall back to non-authenticated connection.  This seems the easiest to me and to be the most correct solution.

Check for the presence of the HTTPS_PROXY environment variable. If it's been set and is of the form prot://user:password@URL, then don't try to use Ruby basic authentication, since I believe this interferes with the basic authentication to the proxy.

I think [2] might work but that [1] is better for everybody involved.  Let me know if you have any ideas on this, otherwise I'll a little bit deeper into [1].

@ahmadia I'd be surprised if 1) isn't already the case. 2) sounds like a reasonable compromise, though.

+1 as i just tried to upgrade to use brew 0.9.9 and ran into a couple issues because im behind a proxy server.  First, I kept getting this error:
  Error: Non-HTTP proxy URI<EMAIL_ADDRESS>
Then found a workaround by setting https_proxy=$http_proxy which leads me to the Error: 407 "Proxy Authentication Required  I reverted back to the homebrew-legacy github project, which is working fine

@britztopher Could you file a separate bug report? The homebrew-legacy github project will not receive any more formula updates.

@mikemcquaid  This bug is not for legacy homebrew


When installing the update to Brackets, your extension fails

Your Brackets extension forces the user to go through a bunch of steps that require editing a text file, uploading to a server, copying the URL and pasting it into your extension JUST TO USE BRACKETS. If I don't want to do that for whatever reason, you make brackets lock up and there is nothing I can do to continue. I am going to have to dig into the settings in another text editor since you bricked this one to figure out where your extension is loaded to disable it.
Since I am reviewing Brackets, your extension and its brick behavior is going to give brackets (and your extension) a terrible review. I can't even use the software. So frustrating.
Please revise this extension with a non-modal announcement that encourages NOT REQUIRES the user to configure your extension. At least you could allow a CANCEL or SKIP button.
Thanks.

Thanks for your support, I hadn't noticed it, you are absolutely right, in the next update I will solve that and more details.


Add banner component and banner to home page (Closes #18)

Tested on desktop and mobile. This looks great @dboydgit!

@all-contributors please add @dboydgit  for design, code, review and ideas
Per spec at:
https://allcontributors.org/docs/en/emoji-key


How to use the free click to get the X Y coordinate point？

How to use the free click to get the X Y coordinate point？Sometime  like tileview.OnClickListener?

This has been asked and answered - search issues please


The app could disallow Emoji’s in brand names

Problem

From a contributor: The app could disallow Emoji’s in brand names @Pierre (teolemon)
we could put a strong red warning when using emojis in any text field (to let users know "we know")
I would prefer something slightly pedagogical (and non blocking for the very rare product with an actual emoji), since it allows to show malicious users that we 👀 see them.
I was also thinking we could have that in:
product_name
labels
ingredients
…

I would prefer something slightly pedagogical

100%. Ideally, we should not allow the submission on the first attempt, but still allow the edit in case the emoji use is intended (though, there is no situation that I can think of where an emoji would legitimately be needed).
A message linking to a page that opens in a web browser discussing why data quality is important to the goal of the project may help lower the amount of malicious edits as discussed on slack.


GitHub Workflows security hardening

This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

Hello, thank you for the PR. As you may have noticed this repo is configured to always require manual approval before any workflow runs, but I'd be happy to strengthen this further, especially as this could help improve the security of forks.
The workflows require the ability to create tags, releases and release artefacts, which I think means we'll also require the addition of contents: write at the job level.

Permissions are not as granular as they could be. contents: write covers all the cases you mentioned.

But currently I set contents to read. Do I understand correctly that
        env:
          prebuild_upload: ${{ secrets.GITHUB_TOKEN }}
        run: npx prebuild --runtime napi --target 7

creates GitHub release and needs the write permission?

Yes, the prebuild step creates a release (if it doesn't already exist) with the same name as the tag and adds artefacts to that release.

Coverage remained the same at 100.0% when pulling 8d8bc2b60637157d4b4eef951d113558030c0181 on sashashura:patch-1 into 70e6bb016271ae943d1f7695de8cef7559ffbb1a on lovell:main.

Just for the reference these are current permissions it runs with:

Thank you!


constructors return (null) without parent param

var text = new sketch.Text({
  text: 'my text',
  alignment: sketch.Text.Alignment.center,
})

returns (null)
var text = new sketch.Text({
  parent: page,
  text: 'my text',
  alignment: sketch.Text.Alignment.center,
})

returns the layer and inserts it into a page.
var text = new sketch.Text({
  text: 'my text',
  alignment: sketch.Text.Alignment.center,
})
page.layers = [text];

works fine, so the layer is clearly being constructed OK behind-the-scenes.
I think the (null) printing is the misleading bit, as the top example is the constructor example from the API and so I thought it was broken when it was printing (null)
tested happening on Text, Group and Shape, so I assume all Layer ones

yes I think it's because of the way log works. If you log text.toJSON() does it work?
I've reworked how the logging works in Sketch 50, with first class support console.log so it should disappear naturally.
(logging a Page also prints (null))

yeah toJSON() works - the object is valid behind-the-scenes. Yep I see console.log working OK in Sketch 50 - will close this, hopefully doesn't trip anyone up in the interim :)


feat: Only initialize the audio manager when necessary

A slight improvement to only use the audio manager (beep sound) when necessary.
Also, when the camera is not used (paused state or disposed), this sound manager is disposed
Will fix #2334

Codecov Report

Merging #2335 (803f795) into develop (2ea0da3) will decrease coverage by 1.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2335      +/-   ##
==========================================
- Coverage     8.86%   7.51%   -1.35%
==========================================
  Files          161     190      +29
  Lines         6623    9383    +2760
==========================================
+ Hits           587     705     +118
- Misses        6036    8678    +2642

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
8.73% <0.00%> (-23.57%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
60.00% <0.00%> (-22.98%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
15.11% <0.00%> (-19.10%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
24.65% <0.00%> (-6.78%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

packages/smooth_app/lib/main.dart
14.91% <0.00%> (-2.99%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

...smooth_app/lib/pages/product/new_product_page.dart
0.00% <0.00%> (-0.88%)
:arrow_down:

... and 183 more

Continue to review full report at Codecov.

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


TotalVariation cannot be used with SIRF objects

The TotalVariation requires a geometry for the gradient to be defined
https://github.com/TomographicImaging/CIL/blob/e12c21bebbc364a581277c19d06fd15e9a9ae2ed/Wrappers/Python/cil/optimisation/functions/TotalVariation.py#L264
As the geometry isn't defined in SIRF objects this method fails.

In addition, in SIRF there are cases where the 2D phantoms where in fact are 3D arrays where the first coordinate is 1, e.g., (1, 128,128) for the thorax data. This will cause problems when GradientOperator is called. An alternative is to create a BlockOperator with FiniteDifferenceOperators that compute the fdiff in the second and third directions.

Closed by #1028, #991, #974


Support writes in the SQL adapter

The trickiness here is in supporting all of the various special edgeqlisms that affect writes:

Rewrites
Triggers
Access policies? (We'd probably want that to work on reads also -- which is doable)
Cross-table exclusive constraints
Built-in foreign keys
required multi pointers

My theory for how to implement this, which is not fully formed, is to turn the DML operations in the SQL queries into not-DML CTEs, and then basically use the edgeql compiler to compile edgeql DML that is rigged up to take those CTEs as input. (Using a generalization of external rvars, probably).
That should then let all our hooks operate.
We'd also probably need to compile in assertions that link uuids actually map to a real object.
There are still a bunch of questions, especially around link tables, but I think this should be workable.

One thing we need to figure out an answer for is how to specify global variables to the SQL protocol. The simplest approach would be to expose it as a config value where you specify JSON encoded globals, but we might want something smarter.

My strategy is to first implement INSERT and then DELETE and UPDATE with reusing code blocks / functions from the INSERT implementation.
Todo for INSERT:

[ ] multiple values (properly set for iterator),
[ ] links (INSERT "Movie" (director_id) VALUES ("61e477c9-5ef7-419f-804c-86f5f1a2d78e"::uuid)),
[ ] make sure that triggers run at the end of the top-level-query, not at the end of current DML query,
[ ] DML in CTEs (push generated CTEs one level up so they are always top-level, and reject DML in non-top-level CTEs)
[ ] reuse alias generator between compiler invocations to prevent name conflicts,
[ ] RETURNING values,

Modified rows in query result:
When INSERT is the top-level statement in the query, Postgres response will contain a CommandComplete (B) with tag INSERT oid rows, where oid is always 0 and rows is number of inserted rows.
Because we compile INSERT into SELECT with many CTEs, INSERT is not top-level statement in the query, which means that the postgres response will not contain the appropriate tag.
Also, if the INSERT did not specify RETURNING columns, there should be no result. Again, because we compile to SELECT with CTEs, that will produce a result, with no columns.
To solve this, we will modify the server IO code that proxies compiled SQL requests to Postgres:

compiler will detects presence of a top-level INSERT,
when there is INSERT, server-io will count the number of rows being returned (even if there is 0 columns)
it will then construct appropriate CommandComplete tag with the number of rows in result, which should always match the number of rows inserted,
if the insert did not have RETURNING clause, it will also discard the result relation.

More on modified rows:
When user does not specify RETURNING clause and modifies many many rows, current implementation will return many-many empty rows to the edgedb server process only for them to be counted and discarded.
A better way would be to inject a COUNT(*) into the top-level SELECT when RETURNING is not specified, and then catching the resulting row and passing that data into CommandComplete message.
This means that we have two ways of providing the modified row count, two implementations that need maintenance, but very low impact on performance.

TODO Delete:

[x] DELETE FROM object table,
[ ] DELETE FROM multi-link table,
[ ] DELETE FROM single-link with properties table,
[ ] DELETE FROM multi-property table,
[x] WHERE, USING
[x] RETURNING & CommandComplete tag
[ ] ~WHERE CURRENT OF~
[ ] WITH ... DELETE FROM

TODO:

figure out what to do about INSERT/DELETE/UPDATE ONLY


Dynamic Ipv6 fw rules handling for networks

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ x ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[ x ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Issue #4923 implements the dynamic ipv6 host type alias, which is a /128 address inside a network with a variable prefix.
This is in itself quite useful, but often it is necessary to not only use single hosts in a rule, but whole prefixes, which in this case must also be dynamic.
Now since we already have the whole mechanism of tracking interfaces for the dynamic ipv6 host, I imagine it would not be that hard to do the same for the whole network of said interface or a sub prefix of variable length thereof.
If pf supported source and destination interfaces in the same firewall rule, it would not be necessary to have such a thing since it would be possible to just filter on interfaces to achieve this, but the way it is, it is very difficult to allow a single interface (possibly a vlan) with a variable delegated prefix to reach just the internet and nothing else.

What I just noticed, is this for the whole interface network already possible via the alias type "Internal (automatic)" and then for example "__lan_network" ?

With that I'm going to provisionally close this and am going to test it first.


Array field in inline admin: losing data on first save

What version of Unfold are you using?
For example: 0.40.0
What version of Django are you using?
For example: 5.1.2
What browser are you using?
For example: Chrome
Did you check changelog/commit history, if the bug is not already fixed?
For example: Yes
Did you search other issues, if the bug is not already fixed?
For example: Yes
Did you check the documentation?
For example: Yes
Are you able to replicate the bug in the demo site?
Didn't find an example.
Describe your issue
Code:
class Child:
    array_field: list[float] = ArrayField(models.FloatField(), null=True, blank=True)

class ChildInlineAdmin():
    model = Child

@admin.register(Parent)
class ParentAdmin():
    inlines = [ChildInlineAdmin]

Steps to reproduce:

I add a new child to the parent
I add a new array_field item to the child
I save the parent

Issue: only the first array_field item is saved, all others are lost.
I managed to fix the problem, by adding the next snippet to inline.js updateElementIndex().
            if (el.tagName === 'TEMPLATE') {
                const fragment = $(el.content);
                fragment.find("*").each(function() {
                    updateElementIndex(this, prefix, ndx);
                });
            }

But I needed to copy the whole file to my app. I wanted to create a PR to unfold (or probably django), but honestly I don't know what is the best solution here.

Thanks for report. Fixed in #836

thank you @lukasvinclav


CMake/CI fixes

Remove boost dependency when OPENVDB_USE_DELAYED_LOADING=OFF
Better support for building with CMAKE_FIND_PACKAGE_PREFER_CONFIG=ON
CI fixes

@danrbailey hopefully the last changes needed for VDB11's CMake/CI. Running the weekly CI on my fork: https://github.com/Idclip/openvdb/actions/runs/6632302974


Local Conda env found, but not activating properly.

A project-local conda environment will be seen by VS Code, but it will fail to activate automatically. I would guess this is because it looks at the end of the path for the environment name and always activates by name, rather than by path. However, for environments created with the -p pathname flag, this will fail (as you can't even set environments name for them with -n).
Environment data

VS Code version: 1.30.1
Extension version (available under the Extensions sidebar): 2018.12.1
OS and version: Windows 10 V 1803
Python version: 3.7.0, conda 4.5.12
Type of virtual environment used: conda

Steps to reproduce:

Create empty directory (mine was D:/TST) for project and add a random .py file (for ext to load), open in VS Code.
Run conda create -p ./localenv python=3.7 in that directory.
Select this environment as the Python interpreter.
Make sure "python.terminal.activateEnvironment": true (or unset) in settings.
Open new terminal (EDIT: as in interactive terminal, e.g. a REPL; for me, this was the cmd, but running code in the REPL has the same effect: it fails to activate the env, and thus runs in the default environment).

Expected behaviour
Automatic environment activation by path, rather than name:
D:\TST>activate ./localenv

(D:\TST\localenv) D:\TST>

OR
D:\TST>activate "D:\TST\localenv"

(D:\TST\localenv) D:\TST>

Actual behaviour
When opening a terminal:
D:\TST>activate localenv
Could not find conda environment: localenv
You can list all discoverable environments with `conda info --envs`.
D:\TST>

I am seeing the same issue with local conda environments created with prefix (e.g. -p ./env). Discovery works but then activation fails every time I open a new terminal (cmd in my case).

The same problem.

I was pointed here from my stack-overflow question, so I assume this is a similar problem: https://stackoverflow.com/questions/54922399/activate-conda-environment-stopped-working-in-vscode

I have the same problem

Changing to enhancement, as this is by design, currently we only support envs with a name see here https://code.visualstudio.com/docs/python/environments#_conda-environments

Changing to enhancement, as this is by design, currently we only support envs with a name see here https://code.visualstudio.com/docs/python/environments#_conda-environments

Sorry for the confusion, but I do have an environment name in the problem I experience. So are they not related?

conda create -n myenv python=3.7
// myenv klasörünü oluşturur.
activate myenv
// envirementi aktif eder
conda install tensorflow

I have the same problem

I think I have found the solution eventually in some other thread here. However, don't really know which one, so I can't give a proper credit to the person who came up with this. However, putting:
    "terminal.integrated.env.osx": {
        "PATH": ""
    }

into settings.json file did the trick, until the bug is fixed.

@lpravda Tried on Windows. This has a lot of side-effects (can't reach any system exec files, for example), unless I'm missing something obvious.

@lpravda Tried on Windows. This has a lot of side-effects (can't reach any system exec files, for example), unless I'm missing something obvious.

I have a mac, so I assume that terminal.integrated.env.osx option works on mac only. There are other options terminal.integrated.env.linux and terminal.integrated.env.windows for different OS. But dont have a windows, so cant really test it.

I obviously used the Windows version. 😛
The output I get for setting the PATH to null, when starting a terminal through the normal Conda, is this:
D:\.......>D:/Conda/Scripts/activate
'chcp' is not recognized as an internal or external command,
operable program or batch file.

(base) D:\.........>conda activate myenv
'chcp' is not recognized as an internal or external command,
operable program or batch file.

(myenv) D:\..............>

This is with a named env, not a local env (which is what this issue was for). I haven't tested whether it works with a local env, but the negative effects of having a broken PATH would be definitely worse than having it work. Can you point to the thread where you got this?
Anyways, this is marked as an enhancement now, so hopefully there's a less-hackish way to do it in the future. For now I've just made a global environment per (group of) project(s).

I confirm this issue occurs on Windows 10 in VS Code version 1.36.1
conda env list  shows all my Conda environments, but
conda activate gym (or whatever env_name) does absolutely nothing.  The virtual environment does not activate in the VS Code terminal.
Now when I run the exact same command conda activate gym from a command prompt, it successfully activates the environment with no problem.  So I'd posit that there's strong evidence that the issue is related to a VS Code bug.

I have a similar problem.
To create a local venv in the project directory, let's call it gym for the sake of the example, I use the command:
conda create --prefix ./gym

Problem is that vscode tries to activate it with activate gym instead of activate ./gym yielding an error of not found.
Running conda env list returns something like this:
base                 <base env path>
                     <project_path>/gym

The venv created from path doesn't have a name, so activating without the ./ prefix yields an error.

I have this issue too. Can't use various features when I have conda environments created with -p. In my case it's necessary to use prefixes in conda environments due to integration with our compute cluster. Looking forward to seeing a solution.

I found a hacky workaround on linux, if you created environments with -p. First find where your activate script is (I'm gonna assume that it's under ~/miniconda3/bin/). Then do
cd ~/miniconda3/bin/
mkdir vscode && cd vscode
cp ../activate .
sed -i 's/conda activate "$@"/[[ -d "$@" ]] \&\& conda activate "$@\/" || conda activate "$@"/g' activate

Finally add "python.condaPath": "~/miniconda3/bin/vscode/activate" to your settings.json. You can do this in your workspace or on the user level.
Not sure how to do this on Windows, but the idea is to check if the argument to conda activate is a directory, and if it is, append the separator so that conda treats it as a path rather than a name of an environment.
You might run into troubles if there is an environment with the same name as the path you specified to conda create --prefix.

Proposed Fix:
Need to update parseCondaEnvironmentNames, if we don't find a name we should leave it empty.
src\client\interpreter\locators\services\condaHelper.ts
Anywhere we use envInfo.name should be updated to use path if the name is empty string:
src\client\common\terminal\environmentActivationProviders\condaActivationProvider.ts see getActivationCommandsForInterpreter
Ensure that the path has a trailing path separator.

@karthiknadig
conda run can be used with path names as well. Validated.

✅ Validated using 2019.12.48501-dev


Any chance for support for 3rd party cookies

Would be handy for embedded iframes in HA.
At least not working for me for google calendar.

How well the webpage renders is entirely up to which version of WebView (the version of Chrome used for Android applications) is supported on your device and operating system.  I am not blocking any 3rd party cookies, but this may be a security measure of the WebView sandbox.

Yes, if I go to the google calendar in WallPanel and login, it keeps the login.
But, if I embed a google calendar in an iFrame in home assistant, it does not access cookie. This works on Fully Kiosk, and Chrome on same device. Same behaviour on devices with Android 5, 5.1, and 8.
Not sure if there is anything I can do?

Well then let’s investigate how we manage 3rd party cookies in Android WebView.   I will let you know what I discover.

Any news on this @thanksmister ?
I think I have the same problem, but I am not sure.
I have my dakboard put in an iframe. On my pc it works fine, except for a gap between header and first card in lovelace that I cannot figure out how to remove:
Picture
It also works fine in my Lenovo Android tablets own browser, on my iPhone's browser etc.
But on wallpanel, it doesn't seem to load properly and gives me this:
Picture
For reference, I get the same issue when trying to run it on the Home Assistant app on iOS.

This sounds like a CSS/HTML rendering problem.  The version of ChromeOS used internally by the Android application is not the same as the browser on the device, so the it won't support all web pages in the same manner.     Some compatibility issue with a recent addition to your HASS dashboard?


changing accent color only works in  Reactour Sample Project but not in imported App

I downloaded the Widget from Mendix Marketplace within Mendix 10.11.0 Studio Pro. Then I imported the ReactTour Sample module from the sample project into my App. I kept all default values from the DS_Helper Microflow and only changed the accentColor attribute to a different color. Although the Color changes, it is from yellow to blue but I inserted a completly  different color #7DDA58 (green)
Are there any styling scss Files that need to be imported to the App aswell?
Or is this a bug ?

I probably solved the Issue when I configured all access rules to the entity and added a change activity in a separate nanoflow.
It is now working as intended


Leaflet 0.7

branch leaflet 0.7 version with link to leaflet 1.0 version

Thank you for the additional PR!
If I understand correctly, I should:

Create a new leaflet-0.7 branch.
Have you change this PR to point into ghybs:leaflet-0.7 branch.
Merge the PR?

If I do the command lines and push it directly into the new leaflet-0.7 branch, will it consider this PR as having been merged?
Sorry I am not very comfortable yet with GitHub…

yes

Ok I just created leaflet-0.7 branch.

Replaced by PR #7 so that it points into leaflet-0.7 branch.


Suggestion: Refactor extract to type alias/typedef

TypeScript Version:  2.9.0-dev.201xxxxx

Search Terms:  Refactor, extract type, typedef
In a TypeScript file Extract to type alias:
var x: { a: number, b: string } = { .. };

would generate:
type newType = { a: number, b: string }
var x: newType = { .. };

In a JavaScript file Extract to typedef:
/** @type {import("./c2").mytype} */
var x;

would generate:
/** @typedef {import("./c2").mytype} newType */

/**@type {myType} */
var x;

gif courtesy of @DanielRosenwasser

it's very helpful to me

@Kingwl is that something you would be interested to implement as well? we would accept a PR for this one.

sure，but it going to be late，my deadline is coming😂

something need to consider:

option parameter?
rest parameter?
update all CallExpression ?

For optional parameters I would strip off the |undefined from the type e.g. function f(a? :number | undefined) =>  type newType = number ; function f(a?: newType)
initializers need to be maintained, e.g. function f(a : number | string = 0) should be type newType = number | string; function f(a: newType = 0)
rest parameters are fine, since you are extracting their type
do not think you need to update call expressions..

I think you are confusing this feature with extract to named arguments. e.g. function f(a: number, b:string) => function f({a, b}: {a: number, b:string}). this one is tracked by https://github.com/Microsoft/TypeScript/issues/23552.

Yes, I confused the two feature (:sad) , this one looks like this is a relatively simple operation

dose not matter what name you pick really. it has to be unique. The new name will be the rename location for the refactoring, and thus the user will get to update it immateriality after the refactor is applied.

I suppose so.. any type node really should be extractable..

@Kingwl we were just wishing we had this 😉

I'll on it😂

The 3.4 release notes mention this issue for providing feedback about this feature were they to expand it to create a type for the new parameter object.
I feel like I nice convention would to be to tap into the name of the function and its class if the function is a method to arrive at the following convention: FunctionNameOptions OR ClassNameMethodNameOptions.
Example:
Before refactor:
class Foo {
    bar(a: string, b: number) {
      console.log('bar');
    }
}

Refactored:
interface FooBarOptions {
    a: string;
    b: number;
}

class Foo {
    bar(o: FooBarOptions) {
        console.log('bar');
    }
}

Instead of Options, the suffix could also be things like Params, Args, etc. Whatever the community thinks makes the most sense.

Thanks @kingwl!


Is Sane Stack dead?

There doesn't appear to be any recent progress.

there are a few of us that intend on picking things back up here but so far we've all been swamped with other things for the past few months so things have been at a standstill here for a while. if your interesting in helping with the project any help is appreciated. 😊

I've been looking around trying to take in all the different frameworks trying to choose something for ember to pair up with and this sounds like it should be a great two-in-one but it's also concerning choosing something that has 1-2 years as the last touched timestamp, I'm also now realizing this post is over a year old (I thought it was this year's post for a while)
I'm assuming that I can continue to move forward and still use the latest and greatest with ember and sails while using this.

I'm still using this framework as well I love how it is setup also with Docker and having a nice stack to quickly prototype. Though I wouldn't run this in production

@g5codyswartz Yeah it is unfortunately not being actively developed but if you want to move things forward here I did give you dev access.

Ye that would be awesome, at least to get the build passing. Also want to introduce this to my work since we are scaffolding a lot this framework is perfect.

I'm thinking about putting some work into updating this. Anyone interested?

@dioptre I don't have a lot of extra time these days but am interested in keeping things alive here too.  I have to admit I haven't used sails in a while though (i've switched my api's to lux), do you know the relative health of sails?  Otherwise, I'd advocate swapping sails out for lux, which being a JSON:api fluent framework it's been a better fit over sails, for me anyways.

Sails is alive and well, they've done great work in the 1.0 release (which https://github.com/mdconaway/sails-ember-rest has maintained compatibility with) and I'm fairly happy with where things are headed. I'm interested in Sane stack - would be interested in helping out as well.
Hadn't seen Lux, thanks for the tip - I'll check it out.


Book Management System

This is a book management system using C++,
@bhumikatewary kindly assign me this under Hacktoberfest 2022.

@bhumikatewary kindly review my PR #272


Added the ability to get closest airport within a set radius

Added InMemorySource - this was declared in SaunaApi but really belongs here. Used at the moment to store airports found in scenario files.
Added GetClosestAirportWithinRadius(GeoPoint position, double radiusM) to DataSource.
N.B. the behaviour of this is slightly different for DFDSource for performance reasons - rather than searching in a circle, we search in a square. That way we can offload the filtering to SQL.
Some attempts have been made to make this work near the poles, but currently if the search square spans a pole we get some strange behaviour. Needs fixing by checking the distance to the pole first.
Also added some unit tests using the NUnit framework.

LGTM!


Delete deprecated neurom.features.register_neurite_feature

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.4.0@78ea518). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             2.4.0      #940   +/-   ##
=========================================
  Coverage         ?   100.00%
=========================================
  Files            ?        37
  Lines            ?      2119
  Branches         ?         0
=========================================
  Hits             ?      2119
  Misses           ?         0
  Partials         ?         0

Thank you!


Implementation of identicon TKBA

Changes in detail:
This PR is an implementation to Identicon TKBA as a new type that we intend to support.
What problem does this change solve?
Currently, we have a limited number of available tacit knowledge (Formosa and Fractal). Individuals are different in how they perceive different types of tacit knowledge, so we need to provide an acceptable number of tacit knowledge types to suit different perceptions among individuals.
Related issues:
This PR will address the issue #91.
Screen record of before and after the changes:
In progress...
PR Checklist:

[ ] All new features have been tested
[ ] All new features have been documented
[x] I have read the CONTRIBUTING.md file
[x] I will abide by the code of conduct

Close this PR as we will not implement this feature for now.


"404 Not Found" after redirect back to signin-oidc in deploy environment

I am publishing the official sample of "IdentityServer4.Samples\Quickstarts\5_HybridFlowAuthenticationWithApiAccess" to cloud server (windows 2017 + iis) but get a "404 Not Found" when redirect back to the MvcClient's \Home\Secure after successful login and authentication from the IdentityServer.
There was pretty little change to the official sample code except some configuration of authority and redirectback urls:
identityserver.Config.cs:
                // OpenID Connect hybrid flow client (MVC)
                new Client
                {
                    ClientId = "mvc",
                    ClientName = "MVC Client",
// these code works well:
                    AllowedGrantTypes = GrantTypes.Implicit,
// these code result to the signin-oidc 404 error:
                    AllowedGrantTypes = GrantTypes.Hybrid,

                    ClientSecrets = { new Secret("secret".Sha256()) },

                    RedirectUris           = { "http://www.someweb.cn/signin-oidc" },
                    PostLogoutRedirectUris = { "http://www.someweb.cn/signout-callback-oidc" },

                    AllowedScopes =
                    {
                        IdentityServerConstants.StandardScopes.OpenId,
                        IdentityServerConstants.StandardScopes.Profile,
                        "api1"
                    },
                    AllowOfflineAccess = true
                }

MvcClient.startup.cs:
                {
                    options.DefaultScheme = "Cookies";
                    options.DefaultChallengeScheme = "oidc";
                })
                .AddCookie("Cookies")
                .AddOpenIdConnect("oidc", options =>
                {
                    options.SignInScheme = "Cookies";
                    options.Authority = "https://identity.someweb.cn";
                    options.RequireHttpsMetadata = false;
                    options.ClientId = "mvc";
                    options.ClientSecret = "secret";
                    options.SaveTokens = true;
                    options.GetClaimsFromUserInfoEndpoint = true;
                    options.Scope.Add("offline_access");
                    options.ClaimActions.MapJsonKey("website", "website");
// these code works well:
                    options.ResponseType = "id_token";
// these code result to the 404 error
                    options.Scope.Add("api1");
                    options.ResponseType = "code id_token";
// these code result to "invalid_scope" error:
                    options.Scope.Add("api1");
                    options.ResponseType = "id_token";
                });

It seems to be an issue of using HybridFlow mode in publishing environment only. It works well when debugging with Visual Studio and local address like http://localhost:5000. (I tried adding AddCors to the cloud server but it didn't make a difference.
Some logs of the cloud server (MvcClient)
2019-03-23 11:36:07.588 +08:00 [INF] User profile not available. Using 'HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\ASP.NET\4.0.30319.0\AutoGenKeys\S-1-5-82-1592072215-1740167268-757123690-3585059337-856257778\DataProtection' as key repository and Windows DPAPI to encrypt keys at rest.
2019-03-23 11:36:08.070 +08:00 [INF] Request starting HTTP/1.0 GET http://www.yingyu88.cn/Home/Secure
2019-03-23 11:36:08.187 +08:00 [INF] Route matched with {action = "Secure", controller = "Home"}. Executing action MvcClient.Controllers.HomeController.Secure (Yingyu88web)
2019-03-23 11:36:08.197 +08:00 [INF] Authorization failed.
2019-03-23 11:36:08.202 +08:00 [INF] Authorization failed for the request at filter 'Microsoft.AspNetCore.Mvc.Authorization.AuthorizeFilter'.
2019-03-23 11:36:08.209 +08:00 [INF] Executing ChallengeResult with authentication schemes ([]).
2019-03-23 11:36:08.713 +08:00 [INF] AuthenticationScheme: oidc was challenged.
2019-03-23 11:36:08.722 +08:00 [INF] Executed action MvcClient.Controllers.HomeController.Secure (Yingyu88web) in<PHONE_NUMBER>0000008ms
2019-03-23 11:36:08.742 +08:00 [INF] Request finished in 673.8004ms 302
2019-03-23 11:36:15.048 +08:00 [INF] Request starting HTTP/1.0 POST http://www.yingyu88.cn/signin-oidc application/x-www-form-urlencoded 1532
2019-03-23 11:36:15.418 +08:00 [INF] AuthenticationScheme: Cookies signed in.
2019-03-23 11:36:15.419 +08:00 [INF] Request finished in 370.649ms 302

You may find that the client stopped at "AuthenticationScheme: Cookies signed in." and didn't go with further redirection to "Home\Secure".
I wonder anybody knows what has resulted to this issue and how to solve it. Thanks.

The problem was turned out to be an issue caused by the http deployment of my MvcClient. This is not an issue of IdentityServer, but a compulsory restrict by IdentityModel that limits https deployment only in production environment.
I will close this issue.

@silentrock How did you determine that was the issue, and what did you do to resolve it? I'm struggling with the same issue right now.

@jimford180 you can just simply deploy all the sites with https and that will solve the problem.


BeforeRouteEnter hook executes mounted hook

Version
3.0.2
Reproduction link
https://codesandbox.io/s/yqj0olnkk9
Steps to reproduce
In the sandbox go to /about manually.
What is expected?
The mounted hook should not be executed.
What is actually happening?
If you open the console you can see that mounted hook gets executed.

I don't know if this is made by design, but since beforeRouteEnter runs before the component gets mount, the mounted hook should not be executed if from beforeRouteEnter hook you redirect to another path. It seems like a bug to me.

The callback passed to next gets executed after mounted. This is by design.
BTW, you can probably extract the code to a method and call the method in the next callback :)
Also, don't pass artificial params that have not been declared as props. This has never been supported and we do not plan on supporting local state on navigations


Issue / RFE for the DBB migration tool

Info about migrating datasets to git: https://www.ibm.com/support/knowledgecenter/en/SS6T76_1.0.7/migration.html
On that page it says:
'After the local Git repository has been created on zFS, you can start using the the migration tool to copy source files from data set libraries to the local repository.
In addition to copying the source from PDS to zFS, the migration tool also creates and updates the .gitattributes file. The .gitattributes file is required by Rocket's Git client to perform automatic codepage conversion between the Git server and the local Git repository during the migration.'
The migration tool might work ok when you are using the codepage IBM-1047 but when you are using something else it doesn't work so well.
What I would like is that the step where the DBB migration tool creates/updates the .gitattributes file is a choice (new parameter) for the user, If I want it created/updated or not.
What happens today?
We are using codepage IBM-278 (Swedish). My mappings file is correct. My .gitattributes file is correct.
After migration the code is correct, but the .gitattributes file is corrupt. Migration tool has added rows to the .gitattributes file with the standard codepage of IBM-1047.
Filing an issue/RFE on request by D. Behm
Thanks!
/N

What version of the DBB toolkit are you using?  This issue was fixed in DBB v1.0.5 released in June 2019.

OK, that's weird. We are using dbb 1.0.6 (will upgrade to dbb 1.0.7 asap).
According to D. Behm same thing happens when using german codepage while running dbb 1.0.7.


Add more precomputation rules for apiserver latencies.

They should be rather cheap, so I don't think we need to be cautious about enabling them.
ref https://github.com/kubernetes/perf-tests/issues/498
/assign @mm4tt

/lgtm
/approve


Cannot find module error

Hello,
I recently upgraded to NS 2.2 from 2.0 and now I am having trouble with loading the plugin in Android.  Everything works fine in iOS, but in android I am getting the following error:
Error:  Cannot find module "nativescript-push-notifications"
The error is in everlive.all.min.js line 339.  I am using Everilve SDK version 1.8.1
Here is what my package.json look like:
{
"nativescript": {
"id": "com.macfarlanegp.awlmobilenative"
},
"dependencies": {
"moment": "2.14.1",
"moment-timezone": "0.5.5",
"nativescript-background-http": "2.4.0",
"nativescript-email": "1.3.3",
"nativescript-intl": "0.0.4",
"nativescript-iqkeyboardmanager": "1.0.0",
"nativescript-loading-indicator": "2.0.1",
"nativescript-maskedinput": "0.0.3",
"nativescript-pdf-view": "1.2.0",
"nativescript-phone": "1.2.3",
"nativescript-social-share": "1.3.1",
"nativescript-telerik-ui": "1.5.0",
"nativescript-touchid": "2.1.0",
"tns-core-modules": "2.4.0",
"nativescript-push-notifications": "0.0.15"
},
"devDependencies": {}
}
Any help is greatly appreciated.
Thank you.

@dpdragnev Sorry for the late reply. This seems like a discrepancy in some of the runtime and dependency versions.
May I suggest that you ensure upgrading to:

the latest 2.3 version of NativeScript
the latest 2.3 core NativeScript modules
the plugin to 0.0.19

Thanks Anton,
Unfortunately, this did not help.  I am still getting the same error.  Do you have any other suggestions?

@dpdragnev
I managed to reproduce this issue in the NativeScript developer app (especially after syncing some changes).
Please ensure that you are running and testing the application package deployed on a device and not in the developer app. This should solve the problem you reported.
Tested on:
Android 6.0
NS run-time 2.3.0
NS modules 2.3.0 and 2.4.0 ( I can see you have in the package.json a reference to 2.4.0)
JS SDK 1.8.1

@dpdragnev It appears that the issue is reproducible for some cases. Please consider downgrading the JS SDK to 1.7.2 or 1.7.3, for example following the instructions from here or as provided in your private support case.
I am closing this case, however, because it is not related to the plugin, it concerns the way the JS SDK utilizes the plugin.


AdditionalProperties has AdditionalPropertiesAllowed=false

When JsonDictionaryHandler sets the OpenApiSchema.AdditionalProperties, that OpenApiSchema will have  AdditionalPropertiesAllowed == false. This in turns will add "additionalProperties": false to every AdditionalProperties-entry in the generated swagger.
Probably not a big issue, but I think it is unnecessary.

This happens to me as well.

More context: https://github.com/Azure/autorest/issues/3497

@richardscholten73 your workaround didn't work for me unfortunately. I still get those properties. I don't get how it's different from your setup somehow. Could there be something else involved?

@richardscholten73 your workaround didn't work for me unfortunately. I still get those properties. I don't get how it's different from your setup somehow. Could there be something else involved?
Not that i know of. It's been awhile since i worked on that project. Maybe the workaround does not work anymore on newer versions? My project is on 5.5.1

@ffMathy the workaround works for me in latest Swashbuckle version 6.1.4. Make sure you register AdditionalParametersDocumentFilter after all other filters, so it can update all schemas properly 😉

Worked like a charm, thank you! As the additionalProperties are set to true by default (at least in Azure API Management), the generated Swagger document just omits the whole property using your filter.


Orchid 14 support

It is planned to implement a package for Orchid 14.x
Thank you.

Right now its not planned, due to lack of resources.
Not sure about how many breaking changes was introduced with 14, but this package haven't followed the breaking changes from 13 either.


List[Parent | Child] can't be assigned to List[Parent]

Environment data

Language Server version: 2022.3.1 (pyright bd47ac0b)
OS and version: WSL(idk if 1 or 2) - Ubuntu 20.04.1 LTS
Python version (& distribution if applicable, e.g. Anaconda): 3.10

Expected behaviour
Variables of type List[ChildClass | ParentClass]  should behave exactly like List[ParentClass], as ParentClass and ChildClass | ParentClass are identical (there is no object which is in one of the types but not the other).
In particular, passing a List[ChildClass | ParentClass] to a function which expects a List[ParentClass] should work.
I did a sanity check in TypeScript, and it works like that there.
I really hope this is not justme misunderstanding how invariance works, but I genuinely think that invariance shouldn't affect me in this case.
Actual behaviour
I get the following error on the last line:
Argument of type "list[Action | LearnAction]" cannot be assigned to parameter "actions" of type "List[Action]" in function "do_stuff_with_actions"
  TypeVar "_T@list" is invariant
    Type "Action | LearnAction" cannot be assigned to type "Action"
      "LearnAction" is incompatible with "Action"Pylance[reportGeneralTypeIssues](https://github.com/microsoft/pylance-release/blob/main/DIAGNOSTIC_SEVERITY_RULES.md#diagnostic-severity-rules)

Logs
https://pastebin.com/uN3y1dk1
Code Snippet / Additional information

from typing import List
from typing_extensions import reveal_type

class Action:
    pass

class LearnAction(Action):
    pass

def do_stuff_with_actions(actions: List[Action]):
    pass

my_actions: List[Action | LearnAction] = []
reveal_type(my_actions) # Type of "my_actions" is "list[Action | LearnAction]"Pylance
do_stuff_with_actions(my_actions) # Error here

Thanks for the bug report. This will be fixed in the next release.

Thanks for the fast response!
It came up in the following code:
from typing import List
from typing_extensions import reveal_type

class Action:
    pass

class LearnAction(Action):
    pass

def get_action_object():
    action_type = input()
    if action_type == "LEARN":
        return LearnAction()
    return Action()

reveal_type(get_action_object()) # Type of "get_action_object()" is "LearnAction | Action"

This is pretty bad code (at least in the context of my project), but it's just a temporary state while I'm refactoring it.
I believe that in the end, I will have to use Sequence instead of list in most parts, even after this bug is fixed, which seems fine to me.


remove bootstrap classes and use mui components instead

What's wrong

no bootstrap should be used in the application, we want to only have one styling library in the tool

What done looks like

no bootstrap should be used in the application

Where the changes need to be made

[x] About
[x] App.js
[x] index.js
[x] CropDetailsModal
[x] WeatherConditions
[x] WeatherFrostDates
[x] WeatherFrostFreeDays
[x] WeatherPrecipitaion
[x] CoverCropExplorer  - in progress
[ ] CropCalendarView
[ ] CropTableListItem
[x] CropSidebar - in progress
[x] GoalSelector
[x] Help
[x] InformationBar
[x] DictionaryContent
[x] InformationSheetDictionary
[x] InformationSheetConnent
[x] CoverCropInformation
[ ] InformationSheetReferences
[x] Location - in progress
[x] LocationConfirmation - in progress(Ameya)
[x] SoilCondition
[x] SoilComposition
[x] SoilDrainage
[x] SoilFloodingFrequency
[x] MyCoverCropList
[x] MyCoverCropComparison
[x] MyCoverCropListWrapper

there might be a few other ones. this is all of the files using bootstrap for grid work

InformationsheetReferences.js
CropTableCard.js
constants.js
RenderTableItems.js

and then npm uninstall bootstrap


Fix the Chat-bot button size

In the Reference of #103
Modify the size to Chat-bot button

Please do not change size for PC.

Please do not change size for PC.

No, I don't change anything for pc


[Bug] createIndex switches to "Ready" state before it is actually is "Ready" for upserting

Is this a new bug?

[X] I believe this is a new bug
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I have a function that programmatically creates a pinecone index. It should be resolved only once the index is "ready" for upserting.
async createIndex(owner: string, repo: string) {
    const name = this.getIndexName(owner, repo)

    await this.client.createIndex({
        createRequest: {
            name,
            dimension: 1536,
        },
    })

    const checkStatus = async () => {
        const { status } = await this.client.describeIndex({
            indexName: name,
        })
        if (status?.state === IndexMetaStatusStateEnum.Ready) {
            return status
        } else {
            return new Promise((resolve) => {
                setTimeout(() => resolve(checkStatus()), 5000)
            })
        }
    }

    const status = await checkStatus()
    return this.getIndex(owner, repo)
}

However, I experience an issue with this approach: even though the status returns me state: "Ready", the upserting command fails. If I create an index manually, the upserting works as intended. Hence, I think the issue is inside the logic that switches state to Ready way too early.
Expected Behavior
index's state switches to "Ready" only when index can be used for operations like upsert.
Steps To Reproduce

Create a sample node application
Install pinecone
Create an instance of the pinecone using new PineconeClient()
Call .createIndex with arbitrary parameters. For example,

const index = await client.createIndex({
    createRequest: {
        name,
        dimension: 1536,
    },
})

Implement a similar function to the one in the example to await for index status to switch to Ready
Try to upsert some data. For example, you can write a function similar to this:

async store(docs: Document[], pineconeIndex: VectorOperationsApi) {
    for (const chunk of this.splitDocuments(docs)) {
        await PineconeStore.fromDocuments(
            chunk,
            new OpenAIEmbeddings({
                verbose: true,
            }),
            {
                pineconeIndex,
            }
        )
    }
}

You will have an error without a stack trace. However, if you pre-create an index and remove index creation part from the code, everything works fine.

Relevant log output
No response
Environment
- **OS**:
- **Language version**:
- **Pinecone client version**:

Additional Context
No response

Hello! Are you still experiencing this issue? From everything we've seen on our end, the status of an index should only return Ready when it is indeed ready for upserting.
Please let us know!


Plugin tries to transform unsupported file extensions

Current regular expression test succeeds on unsupported file extensions. Particullarly .json.
transform(code, id) {
    if (/\.[j|t][sx|s]|vue$/.test(id)) {
        // ...

Regular expression should be something like
/\.(?:[jt]sx?|vue)$/

Fixed, thx


Not sure if this is an issue or not.

New to all of this. I have three accessories id like to add to my avatar but they only have meshes and prefabs. in the Heirarchy they do not have armatures. Attempting to apply them will hide them like normal but it does not generate parameters or controls in the selected expression menu. Not sure if there is something I need to do or they are simply incompatible.

The author doesn't seem to be replying so I'll try to help instead.
I'm guessing by the name of this accessory that it isn't supposed to be dynamic and can simply be parented to your avatar's neck bone.
If it is supposed to have an armature, you need to select the asset in the project tab and set it's rig type to Generic.

You will need to add those directly under the armature. Aka your necklace should be placed under you neck bone, your aviators under your head bone, etc.


Remove supervisord.conf volume from Dockerfile

Not needed any more.

We have to wait until the changes of new multiinstance backend is merged und in production. Otherwise current instances does not work with updated Dockerfile.

See #3804