[Extensibility Request] Page 5076 "Interaction Log Entries" - Protected variable

Hello, we need the following currently global variable defined as protected for our page extension. Can you please check that? Thank you!
protected var
    ShowEnable: Boolean;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fixed a few typos in the French translation

I've fixed a few typos and added true (curly) apostrophes instead of straight single quotes for a better typography.
I have a doubt about translating "you" as "tu" (instead of "vous"). "Tu" can be too informal in French in some contexts. I would love to hear other opinions on this point. Thanks!

@thibaultmalfoy thanks for the fixes! 💯 Ok to merge @daniellockyer
Regarding "tu" vs "vous", that's a great point. Also have a doubt. I think "vous" is more common and more professional, whereas "tu" brings more proximity with the reader. I checked out a few websites in the publishing space – most of them use the "vous" form, I could only find one using the "tu" form (Snowball). So, tend to agree with using the "vous" form instead. @thibaultmalfoy would you like to give it a go in a separate PR and then we can compare the two versions?

Thanks both! Happy to receive further PRs 🙂


Default peer service name to blank as it is required in Thrift Endpoint

Fixes #45

PS verified this fixes it!

LGTM pending the null conversation.

OK I figured it out. It was really a bug: reusing getPeer method as both lazy initializer and getter. I split them into plain getter and getOrMakePeer, the latter is only used by the methods that actually set the values on the peer, so the defaulting makes a lot more sense.

released as 0.8.1

thanks
On Sat, Sep 10, 2016 at 4:29 AM, Yuri Shkuro<EMAIL_ADDRESS>wrote:

released as 0.8.1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/uber/jaeger-client-java/pull/47#issuecomment-246030681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61ytUxesvrWAeQS9o8tDqn6kRWdB4ks5qocG1gaJpZM4J4nqH
.


Create ARM Templates for Azure Components

Jira Description
Template should:

Link Azure Analysis Services with the Production ODS database
Include an Azure Function to refresh AAS data from the underlying ODS
Include Azure Automation to schedule on/off time for AAS, to save deployment costs (nice to have)

Jira Metadata
Created: 2017-03-15T14:08:42.000-0500
Reporter: Dean Goodman

Jira Comment
2017-03-23T13:03:56.543-0500
Jira Metadata
Created: A requirement of CODS-206 is to integrate the Analytics installer as an option during the CODS installation.  There are some credentials and information that are captured during the CODS installation that can (and should) be shared with the Analytics installer:

SQL Server Connection String - This needs to get injected into the .xmla as the data source prior to deploying the data model to the AAS server.
Resource Group Name
Resource Group Location

Commenter: Dan Malagari

Jira Comment
2017-03-27T09:48:54.207-0500
Jira Metadata
Created: [~Malagari] As discussed with [~goodmanmd] in the last status meeting, we do not want to integrate the analytics installer as an option in the Cloud ODS deployment - so this ticket (and related ones) should move to the Analytics project. Is is semantics at some point, but we want to keep the Cloud ODS offering distinct from a variety of analytics offerings
cc [~Eric.Jansson], [~cyjones], [~goodmanmd]
Commenter: Chris Moffatt

Jira Comment
2017-03-30T13:49:19.977-0500
Jira Metadata
Created: Moved this issue from the CODS project to BIA project.
Commenter: Dan Malagari


Implement Dynamic Provider Selection for Telephony Services in Twilio and Vonage Integrations

Implement Dynamic Provider Selection for Telephony Services in Twilio and Vonage Integrations

Summary

The objective is to implement dynamic provider selection for telephony services in Twilio and Vonage integrations. This will allow the system to switch between different telephony services based on the requirements, enhancing flexibility and efficiency.

Blockers

[x]  None

Outcome

Improve the flexibility and adaptability of the Vocode open source library by implementing dynamic provider selection for telephony services, thereby enhancing developer engagement and the overall quality of the product.

Technical Details

The task involves modifying several methods in the vocode/streaming/models/synthesizer.py, vocode/streaming/models/transcriber.py, vocode/streaming/telephony/constants.py, vocode/streaming/telephony/client/vonage_client.py, and vocode/streaming/telephony/conversation/vonage_call.py files. The goal is to switch from static to dynamic telephony service providers (Twilio and Vonage).

Subtasks

[ ]  Modify the [from_telephone_output_device](file:///home/arpagon/Workspace/ArtisanLabs/vocode-python/vocode/streaming/models/synthesizer.py#60%2C9-60%2C9) and [from_telephone_input_device](file:///home/arpagon/Workspace/ArtisanLabs/vocode-python/vocode/streaming/models/transcriber.py#82%2C9-82%2C9) methods in synthesizer.py and transcriber.py respectively to support both Twilio and Vonage.
[ ]  Update the constants in constants.py to support both Twilio and Vonage.
[ ]  Implement the [validate_outbound_call](file:///home/arpagon/Workspace/ArtisanLabs/vocode-python/vocode/streaming/telephony/client/vonage_client.py#127%2C9-127%2C9) method in vonage_client.py to validate outbound calls.
[ ]  Make the [send_dtmf](file:///home/arpagon/Workspace/ArtisanLabs/vocode-python/vocode/streaming/telephony/conversation/vonage_call.py#92%2C9-92%2C9) function in vonage_call.py asynchronous and use aiohttp with the Vonage client.

Related to #459


Deactivate caching in fulltest pipeline

This PR removes the last remaining caching steps from our pipeline due to the mysterious interference with serialization. Can be reactivated once the root cause is identified.

closign due to #298


The role of the wh_ratio_clip  of delta2bbox？

same as title

It limits the regression range of w and h to prevents too large or small predictions.


Call for code reviewers!

As of right now, we've got about 8 open PRs. Let's those reviewed and approved+merged or closed! If you're one of those contributors, thank you for your patience and for your work.
I'm not an active user of this chart, myself, but if you're reading this, then you probably are. I need more folks like you to help review PRs. There's changesets more ambitious than I'm personally comfortable with reviewing on my own, but if they're to benefit the community, they need :eyes:.
If you're interested, please go right ahead and review some PRs!
There aren't currently any formal contribution guidelines [yet] for this repo, so I'll just outline the one I think is of utmost importance:
Whether you're a PR author or a reviewer, show respect and gratitude for one another. Move forward in good faith. All opinions and voices are welcome here. That said, please be willing to give and receive criticism from time to time, and do so in a kind, respectful, and gracious manner.

@canterberry I'd be happy to become a reviewer

I'ved added you both to the project with Maintainer permissions. Thanks for volunteering! You're welcome to contribute in any way you're comfortable, as little or as much as you'd like.
I'll keep an eye open to see how things are going and if there's anything you need from me.

@vyas-n @canterberry What does the release process look like?
From what I can tell we just need to add a chart tar to the gh-pages branch, but it looks like the tar is signed with a pgp key?
https://github.com/twuni/docker-registry.helm/commit/ee6524465fa9c6891fc64e4916aed8b74da49817
I'm not familiar with this signing process, if we need a special private pgp key, etc?

Looks like the CI process is currently failing:
I don't have access to the Circle CI Org, but I'll create a PR to move the logic from Circle CI over to GitHub Actions.

@vyas-n @canterberry What does the release process look like?
From what I can tell we just need to add a chart tar to the gh-pages branch, but it looks like the tar is signed with a pgp key?
ee65244
I'm not familiar with this signing process, if we need a special private pgp key, etc?

Yes, the gh-pages branch is probably the best spot for releases. I still own and operate the repo at https://helm.twun.io/ , but with GitHub Pages, I'm personally removed as a dependency/bottleneck and the release process can be more easily delegated and collectively governed.
Here's the issue/comment where the gh-pages branch release strategy started: https://github.com/twuni/docker-registry.helm/issues/22#issuecomment-1004519481
Here's how release signing works: https://helm.sh/docs/topics/provenance/
I recommend generating your own designated release signing key for this repo and publishing that public key here somewhere, so folks know who the key belongs to and that the signature belongs to an authorized releaser.
:memo: I'll defer to y'all on whether to sign releases at all -- I'm not sure how important provenance is for most people. I get the sense that it's a less commonly used feature.
This looks like it could be a decent starting point for a CI-automated release process: https://helm.sh/docs/howto/chart_releaser_action/

@canterberry @vyas-n  I was thinking about using GitHub pages for the releases, but I'm worried that if we change the registry url, we'll lose a lot of community traction.
For example, Linode has this chart as their recommended way to install a docker registry: https://www.linode.com/docs/guides/how-to-setup-a-private-docker-registry-with-lke-and-object-storage/#:~:text=deploy your docker registry using the configurations

helm repo add twuni https://helm.twun.io

I was wondering if there was any good solutions to keep using helm.twuni.io, but still delegate releases?

@joshsizer

I was wondering if there was any good solutions to keep using helm.twun.io, but still delegate releases?

Two options come to mind:

Set up a permanent redirect from https://helm.twun.io to https://twuni.github.io/docker-registry.helm using HTTP 301 response status with an appropriate Location header. If Helm's tooling follows redirects, this should work just fine.

Set https://helm.twun.io as a passthrough proxy for https://twuni.github.io/docker-registry.helm. This change would be completely transparent to Helm tooling, but I would need to enforce an exception for paths with the prefix /.well-known/ in order to guard against certificate hijacking and other related attacks involving HTTP-based domain validation.

I'm inclined toward the second option. Either way, the result would be to manage releases via GitHub Pages and to distribute releases via both GitHub Pages and helm.twun.io.
I'll get started on this now and report back when it's ready.

Quick update:
I ran a pilot of the second option above on a separate, temporary domain, with success. I'm working on the next step now, applying that to https://helm.twun.io. In the meantime, the repo will be temporarily unavailable. ETA on completing this work is another 15-20 minutes.

@joshsizer This work is now complete.
https://helm.twun.io is now a passthrough proxy for https://twuni.github.io/docker-registry.helm and can be updated via the gh-pages branch.


gmail3 should parse Hibben's new hire emails

THIS IS VITAL TO MY WORKFLOW WHY HAVEN'T YOU IMPLEMENTED IT YET

I'm not particularly interested in that feature, but I'll take a pull request :)

:+1:


Button layout should be clearer & consistent

[ ] Buttons should be consistent
[ ] Buttons should be close to each other


Polymer: add possibility to define group as Object ({name:name, pull: true,put: false})

Allow to define group property as an Object declaratively (Polymer).
 <sortable-js group='{"name":"foo","pull": false, "put": true}'   handle=".sortable-drop-handle">
          <template id="templateRepeat" is="dom-repeat" sort="sortByIndex" observe="value.index" items="{{items}}">

: :

[[item.value.title]]

[[item.key]]

[[item.value.description]]

          </template>
        </sortable-js>

o.k. didn't see that there is a separate repo for polymer (SortableJS/polymer-sortablejs).
Will commit PR on this one
C.

@christophe-g I can't seem to get this to work, what this PR accepted?

A similar PR was merged under https://github.com/SortableJS/polymer-sortablejs. Use this element instead.
C.

@christophe-g Thanks, that's the one I'm using but the snippet at the top you posted doesn't seem to be working for me group='{"name":"foo","pull": false, "put": true}'
Although if I define the group name for both <sortable-js> Polymer components, I'm able to drag between them. Soon as I add any put or pull options, they can't drag between them.

And if you try like the snippet here:
https://github.com/SortableJS/polymer-sortablejs/pull/14


Refactor tasks individual methods into single class task.Tasks

I'd suggest turning this all into a class that can be instantiated with a db session, library, libraryId, etc. This makes it a bit easier to interact with since I believe that it is only called from a single file. It simplifies the calls then because for each method here the only parameter that would be passed in would be the array of resources. This should make maintenance easier in the future as well, otherwise this potentially exposes a bunch of methods needing to be changed individually if something basic changes above.
Originally posted by @mwbenowitz in https://github.com/BookOps-CAT/NightShift/pull/14#discussion_r799848745

fixed in f93446579ac3d6a5910cbae351552aac63a41012

submitted in PR #19


Fix orientation of splats

At present, the noise texture visible on splats does not look good.  It turns as motion blur rotates them and stretches them, and is currently oriented the same on splats right next to one another making the repetition obvious.

I believe my attempt to improve this has introduced a slight skewing when moving at speed.


How can I ban a user as a relay operator?

Didn't find any web UI or similar to achieve this.
Thanks.

https://github.com/scsibug/nostr-rs-relay/issues/65

All events can now be passed to an external gRPC server.  That will allow decisions such as blocking all events by pubkey, IP, etc. to be made.  So, you can implement any policy you desire, and web UIs could be built on top of that as well.
I imagine there are a wide variety of things people want to do with respect to policy and UI.  In order to keep the main codebase small and manageable, I'm encouraging people to build on via these gRPC extensions.


Exceptions captured in threads not associating with correct transaction

How do you use Sentry?
Sentry Python SDK
Version
2.19.0 and 1.43.0 (tried with both versions)
Steps to Reproduce
I am having an issue with correctly tracking a transaction in a multithreaded context.  We use a threadpool executor to execute some functions on a series of tasks. On each task, we use the executor to perform one function synchronously (wait for the result) and one function asynchronously (do not wait for the result). The asynchronous function will finish the transaction when it completes, meanwhile the main thread moves on to the next task. As a result, there will be conditions where the main thread has started a new transaction while the previous one is still completing in the previous task's async_work function.
On the sentry performance tab, I see correct traces. The child transactions are correctly rendered. My issue is for manually captured exceptions, sentry_sdk.capture_exception. These issues maintain a link to the correct trace, but for some reason are rendered with the wrong transaction name. Appreciate all insight here!
Below is a minimal example of the circumstance:
import time
from concurrent.futures import ThreadPoolExecutor

import sentry_sdk
import sentry_sdk.scope
from structlog import get_logger

from src.utils.sentry import init_sentry

logger = get_logger(__name__)
TASKS = [{"taskId": i} for i in range(5)]

def sync_work(transaction):
    with transaction.start_child(op="task", name="sync"):
        logger.info("Running sync work")

def async_work(transaction):
    with transaction.start_child(op="task", name="async"):
        logger.info("Running async work")
        time.sleep(2)
        logger.info("Finished async work")
        sentry_sdk.capture_exception(Exception("Async work failed"))
    transaction.finish()

def main():
    init_sentry()
    with ThreadPoolExecutor(max_workers=4) as executor:
        for task in TASKS:
            try:
                transaction = sentry_sdk.start_transaction(name="test-task", op="task")
                sync_result = executor.submit(sync_work, transaction)
                sync_result.result()
                executor.submit(async_work, transaction)
            except Exception as err:
                logger.error(err)

if __name__ == "__main__":
    main()

Expected Result
The exception should show the right transaction name
Actual Result

Thanks @jvntf for raising this -- we will take a look


Fix review excerpt (#18502)

Backport #18502
Currently the "File Changed" tab of a PR is somehow broken. This is also true for the current release 1.16.0.
When you are on the "File Changed" tab, and want to look at code excerpt before or after the code changes, the layout breaks. You can test this on try.gitea.io here: https://try.gitea.io/testnotexisting/magic_enum/pulls/2/files
The problem occurs for the unified view and for the split view.
Kind of the same problem was there for commenting a line of code, this was fixed in #18321 and #18403.
For consistency, I changed the solution of #18321, I removed the colspan and instead added a <td>. The goal was to have code similarly with the split view.
Also the separator line in the split view was in the wrong column, this was fixed too.* more consistent unified review comment
Fix #18516
Co-authored-by: Andrew Thornton<EMAIL_ADDRESS>Co-authored-by: confusedsushi<EMAIL_ADDRESS>

make lgtm work


Refactor component READMEs & macros

We need to refactor each of the component READMEs and macros to make sure they are consistent. There is a new README template to use across all components.

Make sure each of the component READMEs are consistent with each other.

Each component has a GitHub issue and is linked to from the README.

Make sure each of the parameters are consistent with each other.

Make sure each of the macros can pass classes  and attributes.

Create a table of macro arguments for each component

Decide on a policy on our advice to using macros due to the security risk https://mozilla.github.io/nunjucks/api.html#user-defined-templates-warning

[x] Action link

[x] Breadcrumb

[ ] Primary Care Card

[ ] Emergency Care Card

[ ] Urgent Care Card

[ ] Contents list

[ ] Details

[ ] Do and Don't list

[ ] Emergency alert

[ ] Expander

[ ] Expander group

[ ] Feedback banner

[ ] Footer

[ ] Header

[ ] Header transactional

[ ] Header with navigation

[ ] Header with search

[ ] Hero

[ ] Hero with image

[ ] Hero with image with content

[ ] Images

[ ] Inset text

[ ] List panel

[ ] Pagination

[ ] Panel

[ ] Panel with grey background

[ ] Panel with label

[ ] Promo

[ ] Promo with no description

[ ] Promo small

[ ] Promo with image

[ ] Review date

[ ] Skip link

[ ] Tables

[ ] Warning callout

READMEs are actually being built into the github-pages docs, but they're not linked to yet.
https://nhsuk.github.io/nhsuk-frontend/components/care-card/README.html


Dasher elections

Goes through dasher for all CRUD functions for elections.

Would it be easier to make a separate dev-only view to hold the buttons? However, I think it's fine to just delete them. I built them when I hadn't yet integrated the new routes into the app (and wasn't sure when that would happen).


Tree Recombination

What's the new feature?
Sub task of #71

1-Tree (a.k.a. LittererLyons) recombination. Refs: Litterer & Lyons 2012 and Tchernychova 2015 Chapter 3.2 Algorithm 5
M-Tree (a.k.a TchernychovaLyons) recombination Refs:  Tchernychova 2015 Chapter 3.3 Algorithm 7

The SVD based Caratheodory reduction algorithm is shared across variations and can be implemented independent of the tree-based data structure. Feasibility and utility of the Tree-based data structures is to be determined.

See here for a tree-free implementation in numpy/python.
See here for a tree implementation in rust.

TODO:

[ ] Add unit tests
[ ] Add performance tests
[ ] Add coresubset integration

What value does this add?
Provides additional algorithms for generating coresets (reduced measures)
Is there an alternative you've considered?
No response
Additional context
No response

Awaiting #581.

Reassigning to @db091756 for tracking purposes and as likely reviewer. @tc85324 to continue as code author.


Documentation: version only by major version

We currently offer separate documentation for each minor release of the tool (3.4, 3.5, …).
This has a few downsides:

when making improvements to the docs, we often need to edit multiple copies of the docs, which is cumbersome
not much is changing between each minor version anyway

At the same time, we do not have a good mechanism to publish the documentation of the 4.0 branch.
If we create a 4.0 version in Docusaurus, it will be the one displayed by default to the users (because it is the latest one), but since we have not published a stable release in that branch, this is undesirable (people should see 3.x first).
Proposed solution
Switch to only one documentation per major version instead. This means that at the moment, we would have version for OpenRefine 3, and the development docs would be for OpenRefine 4.
Alternatives considered
I am not sure how else we could publish docs for 4.0 with the current system.
Additional context
Also, that solution would mean that we would need to edit the docs for OpenRefine 4 on the master branch, which might be a bit confusing. If we also want to migrate our openrefine.org website to Docusaurus, perhaps this is a sign we should be using a different repository after all?

I want work on it

Thanks! This one might not be such a great first issue though - but I will certainly not try to hold you back!


Openstack hypervisor manages nodes before they are created

When enabling root on nodes, the subloop attempts to manage all nodes before they are created

Sounds like we need a loop here to check on node status before attempting further action.

Follow up here:
https://tickets.puppetlabs.com/browse/BKR-25


[COOK-4499] Updated README to correctly refer to the attribute as openld...

...ap[:cafile] instead of openldap[:cacert].

Merged as an obvious fix per the CLA policy


docs: create and deploy docs website

Closes #3.
Documentation

Create docs website using Docusaurus
Brand Docusaurus theme
Add logo and favicon

CI

Deploy docs website to GitHub Pages

Build

Update Node.js to version 16.16.0 to support Docusaurus
Add the @nx-plus/docusaurus Nx plugin

To do
Before approval

[ ] Make all builds complete successfully
[ ] Configure Yarn workspace to isolate Docusaurus dependencies in the docs-app project
[ ] Create Project overview docs page
[ ] Create Installation docs page
[ ] Create Contributing docs page
[ ] Create Sponsoring/Maintainers docs page
[ ] Remove generated Intro docs page
[ ] Add docs links to header
[ ] Add docs links to footer

Before merge

[ ] Filter branches to main for push events in the CI workflow
[ ] Enable condition for Upload docs website step in the checks job in the CI workflow
[ ] Enable condition for deploy-docs job in the CI workflow
[ ] Configure Docusaurus for upstream organization and repository (analogjs/analog)
[ ] Configure Docusaurus for custom domain (https://analogjs.org)
[ ] Configure GitHub Pages for custom domain

@brandonroberts
I'm seeing build errors for the analog-app project in this branch:
failed to load config from /home/runner/work/analogjs-analog/analogjs-analog/apps/analog-app/vite.config.ts
error during build:
Error: Cannot find module '@ngtools/webpack/src/ivy/transformation'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /home/runner/work/analogjs-analog/analogjs-analog/apps/analog-app/vite.config.ts
- /home/runner/work/analogjs-analog/analogjs-analog/node_modules/vite/dist/node/chunks/dep-71eb12cb.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at angular (/home/runner/work/analogjs-analog/analogjs-analog/node_modules/@analogjs/vite-plugin-angular/src/lib/angular-vite-plugin.js:22:54)
    at /home/runner/work/analogjs-analog/analogjs-analog/apps/analog-app/vite.config.ts:50:54
    at loadConfigFromFile (file:///home/runner/work/analogjs-analog/analogjs-analog/node_modules/vite/dist/node/chunks/dep-71eb12cb.js:62783:15)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async resolveConfig (file:///home/runner/work/analogjs-analog/analogjs-analog/node_modules/vite/dist/node/chunks/dep-71eb12cb.js:62[39](https://github.com/LayZeeDK/analogjs-analog/runs/7786750995?check_suite_focus=true#step:5:40)8:28)
    at async doBuild (file:///home/runner/work/analogjs-analog/analogjs-analog/node_modules/vite/dist/node/chunks/dep-71eb12cb.js:[43](https://github.com/LayZeeDK/analogjs-analog/runs/7786750995?check_suite_focus=true#step:5:44)3[58](https://github.com/LayZeeDK/analogjs-analog/runs/7786750995?check_suite_focus=true#step:5:59):20)

 >  NX   ERROR: Something went wrong in run-commands - Command failed: vite build

   Pass --verbose to see the stacktrace.

I have tried various combinations of regenerating the lockfile without any lock.

Explicitly adding @ngtools/webpack as a dev dependency seems to do the trick 💡

Ready for review, @brandonroberts.

Awesome work Lars, this is looking great! I'm wondering if the planned features that aren't available yet should be removed from the homepage, such as the file-based routing and SSR/SSG. Or denoted as upcoming.

All right, @brandonroberts. I marked the upcoming features on the home page. You won't be able to see it in the preview environment but here's a screenshot. Other than that, I rebased on top of the latest commit in the main branch of this repository.

Feel free to re-approve and merge after taking care of the remaining to-dos which I leave up to you 🙂

Thanks @LayZeeDK!


refactor(tabs): remove duplicate styling

Removes some styling for the .mat-tab-label being repeated across multiple files.
Removes some, what appears to be, left over debugging CSS from the tabs demo.

For reference:

@josephperrott or @andrewseguin, can you take a look at this one?


dotnet new webapi ignoring -au flag for template generation

From @WingedDante on May 31, 2018 19:55
Steps to reproduce
Ensure you're on 2.1.300
Run dotnet new webapi -n="example"
Examine Startup.cs in the example project folder.
As far as I'm aware and can find in the docs, and looking through the template code, the UseHsts and UseHttpsRedirection are only supposed to be output if the -au(--auth) flag is set, and it's supposed to default to none if one isn't supplied.  Unless this functionality has changed.
Expected  behavior
Creates a new webapi project from template with no SSL created in Startup.cs.
Actual behavior
Creates a new webapi project from template with SSL created in Startup.cs, ignoring the default 'None' value for the auth flag.
These lines are added
else
            {
                app.UseHsts();
            }

            app.UseHttpsRedirection();

I compared with the older sdk using a global json setting it to 2.0.3, and the webapi project was created correctly.
Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300
 Commit:    adab45bf0c

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.15063
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.300\

Host (useful for support):
  Version: 2.1.0
  Commit:  caa7b7e2ba

.NET Core SDKs installed:
  2.0.3 [C:\Program Files\dotnet\sdk]
  2.1.4 [C:\Program Files\dotnet\sdk]
  2.1.102 [C:\Program Files\dotnet\sdk]
  2.1.103 [C:\Program Files\dotnet\sdk]
  2.1.300 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

Copied from original issue: dotnet/cli#9387

@mkArtakMSFT @ryanbrandenburg PTAL

@javiercn, can you please look into this?

I do believe that this is controlled simply by the --no-https option and that Auth doesn't play a role in it at the templates level.
I do believe VS has some logic around auth and HTTPS, but that's it.
I don't think it matters that much either as we changed the redirection middleware after preview2 to NOOP if it can't figure out an HTTPS port to redirect to.
I would just close this as by design. See https://github.com/aspnet/templating/commit/cb7942abe41eebbbe574c515b171e6bb9f48fbdd for the original change

Hi @WingedDante.
Can you please share the link of the docs you've read about the auth configuration using --au option?
This is more of a docs issue, which would be great to fix.

Sure, the documentation I was looking at for the dotnet new command was here.  I had just assumed it was from the -au flag because there wasn't a --no-https listed as either one of the base dotnet new options, nor one of the template specific ones, and I saw parts of the changes @javiercn mentioned.  I did try it with the --no-https option and did get the expected output.  Adding in something about the --no-https option would definitely be helpful.  Thanks everyone.

@javiercn, can you please take care of getting the docs updated to mention about no-https option. Thanks!

@Rick-Anderson Can you take care of this?

Closing this in favor of the referenced docs bug: https://github.com/aspnet/Docs/issues/6923


Remove seeds

Do not generate the seeds files on install. That's no longer necessary because default layout, components, and home page is generated on the site boot, and guides will be available to teach how to create more resources using the live admin web interface.

Looks like this was already taken care of in a previous PR


[protobuf] Build error: "ninja: error: mkdir"....

Host Environment

OS: Windows 10 Pro
Compiler: Visual Studio 2019

Steps to reproduce the behavior:
Running in VCPkg Manifest mode, to install Protobuf and GRPC. Logs report the exact command that produced the error is this. I pasted this to the (Powershell)command line and got the same Ninja error.
vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j45
Failure logs
-- Using cached protocolbuffers-protobuf-v3.18.0.tar.gz.
-- Cleaning sources at C:/source/project-main/src/FooModule/ExternalDependencies/vcpkg/buildtrees/protobuf/src/v3.18.0-44e21376af.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/source/project-main/src/FooModule/ExternalDependencies/vcpkg/downloads/protocolbuffers-protobuf-v3.18.0.tar.gz
-- Applying patch fix-static-build.patch
-- Applying patch fix-default-proto-file-path.patch
-- Applying patch fix-uwp-build.patch
-- Using source at C:/source/project-main/src/FooModule/ExternalDependencies/vcpkg/buildtrees/protobuf/src/v3.18.0-44e21376af.clean
-- Configuring x64-windows-static
-- Building x64-windows-static-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:158 (message):
    Command failed: C:/source/project-main/src/FooModule/ExternalDependencies/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j45
    Working Directory: C:/source/project-main/src/FooModule/ExternalDependencies/vcpkg/buildtrees/protobuf/x64-windows-static-dbg
    See logs for more information:
      C:\source\project-main\src\FooModule\ExternalDependencies\vcpkg\buildtrees\protobuf\install-x64-windows-static-dbg-out.log
      C:\source\project-main\src\FooModule\ExternalDependencies\vcpkg\buildtrees\protobuf\install-x64-windows-static-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_cmake.cmake:106 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_cmake.cmake:44 (vcpkg_build_cmake)
  buildtrees/versioning_/versions/protobuf/19c2bf45c235e6126161bae36aa7ff7e1fcda479/portfile.cmake:47 (vcpkg_install_cmake)
  scripts/ports.cmake:145 (include)

Error: Building package protobuf:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+protobuf
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[protobuf]+Build+error
including:
  package: protobuf[core,zlib]:x64-windows-static -> 3.18.0
    vcpkg-tool version: 2022-03-08-85dc3d9d4e1e7b643f811eac0b9d884419fead79
    vcpkg-scripts version: 857bccdad 2022-03-08 (16 hours ago)

Additionally, attach any relevant sections from the log files above.

ninja: error: mkdir(CMakeFiles/libprotobuf-lite.dir/C_/source/project-main/src/FooModule/ExternalDependencies): No such file or directory

Additional context
The overall project is CMake driven, and is using VCPkg in manifest mode (there is a vcpkg.json, alongside the top-level CMakeLists.txt) to pull in GRPC and Protobuf automatically.
{
    "name": "ourproject",
    "version":  "0.2",
    "dependencies": [
        {
            "name": "protobuf",
            "default-features": true,
            "features": [ "zlib" ]
        },
        {
            "name": "grpc",
            "default-features": true,
            "features": [ "codegen" ]
        }
    ],
    "builtin-baseline": "4ffe8b020f04c3acbd5883566672c19a022c4394",
    "overrides": [
        {
            "name": "protobuf",
            "version": "3.18.0"
        },
        {
            "name": "grpc",
            "version-string": "1.41.0"
        }
    ]
}

To my knowledge this was working last week, possibly earlier this week; then today I did a fresh checkout from our version-control, kicked off CMake, and after VCPkg installed itself(*), it failed during the Protobuf build/install.
(*)That is to say our setup.bat that I actually run does a lot, including checking for .\ExternalDependencies\vcpkg, and if needed running git clone  clone https://github.com/Microsoft/vcpkg.git, followed by the vcpkg\bootstrap-vcpkg.bat, and ultimately kicking off CMake with the appropriate arguments.

Thanks for posting this issue. Maybe PR https://github.com/microsoft/vcpkg/pull/23410 and https://github.com/microsoft/vcpkg/pull/23411 can fix this issue. There two PRs fix find moudle on CMake. I test this locally, and I can installed protobuf and grpc succeed. Could you update VCPKG and try again?

Ping @Sand2Silicon for response.

I do not believe #23410 is involved as we hit the corresponding bug #23402 and then saw it resolved before encountering this issue.
I just re-ran this and it it succeeded with warning. Not sure if the warnings are related to the issue I was reporting. Total time was 15min 53 seconds for 18 packages, based on the manifest above requesting GRPC and Protobuf; much longer than usual, so I think everything in the /appdata/ cache got updated. The specific warnings in the logs are:
Starting package 8/18: openssl:x64-windows-static
Building package openssl[core]:x64-windows-static...
-- Installing port from location: C:\source\project-main\ExternalDependencies\vcpkg\buildtrees\versioning_\versions\openssl\d25384246619019a1e44f018546cdfcaf1800174
-- Downloading https://www.openssl.org/source/openssl-1.1.1l.tar.gz;https://www.openssl.org/source/old/1.1.1/openssl-1.1.1l.tar.gz -> openssl-1.1.1l.tar.gz...
-- Downloading https://strawberryperl.com/download/<IP_ADDRESS>/strawberry-perl-<IP_ADDRESS>-32bit.zip -> strawberry-perl-<IP_ADDRESS>-32bit.zip...
CMake Warning at scripts/cmake/vcpkg_fail_port_install.cmake:45 (message):
  vcpkg_fail_port_install has been removed and all values should be moved by
  adding `supports` field to manifest file or directly adding `openssl:=fail`
  to _scripts/ci.baseline.txt_.

  Please remove `vcpkg_fail_port_install(...)`.

Call Stack (most recent call first):
  buildtrees/versioning_/versions/openssl/d25384246619019a1e44f018546cdfcaf1800174/windows/portfile.cmake:1 (vcpkg_fail_port_install)
  buildtrees/versioning_/versions/openssl/d25384246619019a1e44f018546cdfcaf1800174/portfile.cmake:20 (include)
  scripts/ports.cmake:145 (include)

----------------------
  Starting package 12/18: openssl:x64-windows
Building package openssl[core]:x64-windows...
-- Installing port from location: C:\source\project-main\ExternalDependencies\vcpkg\buildtrees\versioning_\versions\openssl\d25384246619019a1e44f018546cdfcaf1800174
-- Using cached openssl-1.1.1l.tar.gz.
CMake Warning at scripts/cmake/vcpkg_fail_port_install.cmake:45 (message):
  vcpkg_fail_port_install has been removed and all values should be moved by
  adding `supports` field to manifest file or directly adding `openssl:=fail`
  to _scripts/ci.baseline.txt_.

  Please remove `vcpkg_fail_port_install(...)`.

Call Stack (most recent call first):
  buildtrees/versioning_/versions/openssl/d25384246619019a1e44f018546cdfcaf1800174/windows/portfile.cmake:1 (vcpkg_fail_port_install)
  buildtrees/versioning_/versions/openssl/d25384246619019a1e44f018546cdfcaf1800174/portfile.cmake:20 (include)
  scripts/ports.cmake:145 (include)

----------------------
Starting package 13/18: c-ares:x64-windows
Building package c-ares[core]:x64-windows...
-- Installing port from location: C:\source\project-main\ExternalDependencies\vcpkg\buildtrees\versioning_\versions\c-ares\d434c999ea00b87bfbee3d79ddc6fc1b6f12d7fe
CMake Warning at scripts/cmake/vcpkg_fail_port_install.cmake:45 (message):
  vcpkg_fail_port_install has been removed and all values should be moved by
  adding `supports` field to manifest file or directly adding `c-ares:=fail`
  to _scripts/ci.baseline.txt_.

  Please remove `vcpkg_fail_port_install(...)`.

Call Stack (most recent call first):
  buildtrees/versioning_/versions/c-ares/d434c999ea00b87bfbee3d79ddc6fc1b6f12d7fe/portfile.cmake:1 (vcpkg_fail_port_install)
  scripts/ports.cmake:145 (include)

----------------------
Starting package 16/18: c-ares:x64-windows-static
Building package c-ares[core]:x64-windows-static...
-- Installing port from location: C:\source\project-main\ExternalDependencies\vcpkg\buildtrees\versioning_\versions\c-ares\d434c999ea00b87bfbee3d79ddc6fc1b6f12d7fe
CMake Warning at scripts/cmake/vcpkg_fail_port_install.cmake:45 (message):
  vcpkg_fail_port_install has been removed and all values should be moved by
  adding `supports` field to manifest file or directly adding `c-ares:=fail`
  to _scripts/ci.baseline.txt_.

  Please remove `vcpkg_fail_port_install(...)`.

Call Stack (most recent call first):
  buildtrees/versioning_/versions/c-ares/d434c999ea00b87bfbee3d79ddc6fc1b6f12d7fe/portfile.cmake:1 (vcpkg_fail_port_install)
  scripts/ports.cmake:145 (include)

This warning is not related with your project, it just remind the deprecated function vcpkg_fail_port_install. And I think manifest doesn't support use ./vcpkg list, this command only can check the ports which installed in vcpkg directory, but manifest mode installed port to vcpkg_installed directory. And when you build your project, the dependency ports in vcpkg.json will install again exery time.

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Implement sharing of bones for skinning

The API now allows to create a standalone BufferObject containing
Bones for skinning. This BufferObject can even be very large and
contains bone data for several renderable.
A new API on RenderableManager allows to set the BufferObject to use.
This is still WIP:

currently not possible to set the binding offset
currently not possible to go back to the old API after using the new one

are there any samples for using SkinningBuffer?😊


Doc fixes

Why are these changes needed?
Some typos and minor editorial suggestions. A few additions to avoid errors I encountered.
Related issue number (if any).

@deanwampler these are nice changes, but my recent push (at least) seems to have caused some conflicts

Okay. Fixed...


Apply #527 changes to Japanese docs

Summary
This pull request applies the changes by #527 to the corresponding document in Japanese language.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

@MarkHash @naotozz Can you review the change?

@MarkHash Thanks for taking the time to review this!
Indeed, "自前で" may not be necessary but I think it's okay to have it in the sentence.

I would use "格納" instead of "代入"

I don't think your idea is more natural. "格納" is not commonly used for translating "assign to a variable". We may use "格納" for storing files/data, though.
The PR seems to be already merged and I don't see any critical errors with it. So, I won't apply any further changes this time.


bug: js_library with 'deps' no longer pulls deps into sandbox output tree

After updating to the latest release of rules_js, I hit an issue where js_library targets no longer pull the files from their deps into the sandbox's bin directory. I bisected the problem to #252, which modified the DefaultInfo for js_library.
I'm not sure if I'm not following the correct usage pattern for js_library, but also noticed that there are no cases of js_library with deps on a :node_modules/* target within the examples/ or e2e/ directories.
This PR reproduces what I'm seeing by building on top of the existing webpack_cli example. You can see the build failure with bazel build //examples/webpack_cli:bundle
Should js_library work in the way showcased here? If not, what is the recommended way to express a library target with srcs and a dependency on an npm library that gets consumed by a js_run_binary target?

fyi @gregmagolan

I think your usage looks right and we should probably revert that change and merge this.

@mrmeku I re-read your description on #252 and I guess we'll be back to command-line spam and crashing BEP stream when we revert. But I don't think runfiles is sufficient to propagate 3p dependencies to an action like webpack, and so far @gregmagolan has resisted adding any additional providers that would let us propagate these outside of DefaultInfo.

https://github.com/aspect-build/bazel-examples/pull/40 - we also broke our Angular example with that change, so I'm reverting it now, sorry @mrmeku


Password prompt

Prompt user for a password if it is not yet defined, and write to localrc. e.g.:
################################################################################
ENTER A PASSWORD TO USE FOR MYSQL.
################################################################################
This value will be written to your localrc file so you don't have to enter it again.
It is probably best to avoid spaces and weird characters.
If you leave this blank, a random default value will be used.
Enter a password now:

if you're attempting to read a password from a bash sourcable file, the sane thing to do is use bash to do so, rather than grep.
Ie:
bash -c '. localrc && echo $MYSQL_PASSWORD'
However, you should not even have to do that, as you're calling this code after you've sourced stackrc, and stackrc sources localrc. so the values will already be set in the current shell.
[ -n "$MYSQL_PASSWORD" ] || you-need-to-set-or-prompt
MYSQL_PASSWORD
Also, we should really set a variable like 'TOP_DIR' indicating a full path to the devstack directory, rather than assuming that it is '.'.  Ie, something like:
TOP_DIR=$(cd $(dirname "$0") && pwd)
then, localrc would always be referenced as "$TOP_DIR/localrc"

smoser - the grep was a paranoid check to make sure that I wasn't adding multiple definitions for the same variable in localrc, which I added when I was first getting started.  Line 161 does this check now, so I removed the grep statement.
I also added a TOP_DIR var.  Thanks for the feedback!

lgtm and verified with UEC-natty and:
#!/bin/sh
apt-get update
apt-get install git -y
git clone https://github.com/cloudbuilders/devstack.git
git checkout password_prompt
cd devstack
echo ADMIN_PASSWORD=changeme >> localrc
echo MYSQL_PASSWORD=changeme >> localrc
echo RABBIT_PASSWORD=changeme >> localrc
echo SERVICE_TOKEN=123124123124 >> localrc
echo FLAT_INTERFACE=br100 >> localrc
./stack.sh


[Dev docs] Add thumbnail provider

Currently having trouble to install and test this locally on my machine.
The instructions are quite focused on preview handlers, whereas this is just a thumbnail provider (for now, at least)

Originally posted by @pedrolamas in https://github.com/microsoft/PowerToys/pull/14827#issuecomment-986092691
As in the above comment mentioned the implementation of thumbnail providers should be added/improved in our dev docs.

honestly, i think doing heavy docs here isn't great use, i'd rather self-document source code as that is where someone would look and have a list of touch points


Document API

API needs to be finalized first, then documented. Project is still very experimental at this point.

API has been documented/designed, but not implemented yet


Update CONTRIBUTORS.md

Added my name to Contributors. First time contributing to a project so just seeing what the process entails. Thank you.

Congratulations on your first PR @StephenCahillane


erc721 transfer test

@seabornlee is this still draft?

Yes, encountered some issue when adding custom token.


Propagate informations from inbound messages up to Client

Now MessageContext is not replaced anymore (like done in #634) but it can be used together with a context decorator function.
Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>

cc @n3wscott

cc @n3wscott

For this to be useful towards #658, we'd also need a way to add custom decorators to ceClient.inboundContextDecorators.
Perhaps a new option?
// it may also be useful to type the decorator signature to make it easier to read
type ContextDecorator func(context.Context) context.Context

func WithInboundContextDecorators(decorators ...ContextDecorator) Option {
  return func(i interface{}) error {
    if c, ok := i.(*ceClient); ok {
      c.inboundContextDecorators = append(c.inboundContextDecorators, decorators...)
    }
    return nil
  }
}

For this to be useful towards #658, we'd also need a way to add custom decorators to ceClient.inboundContextDecorators.
Perhaps a new option?
// it may also be useful to type the decorator signature to make it easier to read
type ContextDecorator func(context.Context) context.Context

func WithInboundContextDecorators(decorators ...ContextDecorator) Option {
  return func(i interface{}) error {
    if c, ok := i.(*ceClient); ok {
      c.inboundContextDecorators = append(c.inboundContextDecorators, decorators...)
    }
    return nil
  }
}

@dan-j Added a new option

@dan-j Added a new option

LGTM

LGTM

LGTM

LGTM


Wsi image index?

Hi,
What is the purpose of the libisyntax_get_wsi_image_index-method and the wsi_image_index-parameter to the libisyntax_get_image-method?

The purpose is to allow access to the wsi_image pointer which allows access to all the interesting things like level resolutions etc.
The reason we have that indirection instead of e.g. libisyntax_get_wsi_image(..) is mostly historical - there was libisyntax_get_macro/label_image_index(..), but we opted to have more specific methods to access the macro/label images - libisyntax_read_macro/label_image(..). This is because they have different specifics - no levels, and no need for fancy access patterns - they are usually just read in full.
It probably would be best to replace libisyntax_get_image(isyntax, index) with libisyntax_get_wsi_image(isyntax) and get rid of the .._index method.
Hope that helps.

It probably would be best to replace libisyntax_get_image(isyntax, index) with libisyntax_get_wsi_image(isyntax) and get rid of the .._index method.

Agreed!


Syntax error on new install

I'm doing a new Laravel install, PHP v 7.2 on a shared hosting server. After running composer install, it fails with the following error:
Generating optimized autoload files
> Illuminate\Foundation\ComposerScripts::postAutoloadDump

Parse error: syntax error, unexpected '=' in /home/brianbe2/photos.ohheybrian.com/vendor/laravel/framework/src/Illuminate/Support/Arr.php on line 380

This line is throwing the error:
380         [$value, $key] = static::explodePluckParameters($value, $key);

https://laracasts.com/discuss/channels/laravel/unexpected-illuminatesupportarrphp-on-line-388

My phpinfo shows Apache is using 7.2.17.

The question is which php is running on CLI as composer is using the CLI.

Ok, after much digging and testing, I was able to get it running. On shared hosting with cPanel, there is a MultiPHP Manager tool which allows the user to set the PHP version per domain. Setting to 7.2 runs in the CLI but for some reason, composer doesn't see the updated version.
Running composer install --no-dev --ignore-platform-reqs installs all dependencies successfully and the application runs as expected. I may add a line to the wiki.

yay :)


updated the example

Updated createSession to createEmailSession as it was a bit confusing because createSession isn't defined.

@faizanpasha265, thanks for creating this PR! 🙏 This repo is actually read-only as the code is auto generated from other sources. In the case of this EXAMPLES.md file, the source is in the appwrite repo.

No worries ✌> @faizanpasha265, thanks for creating this PR! 🙏 This repo is actually read-only as the code is auto generated from other sources. In the case of this EXAMPLES.md file, the source is in the appwrite repo.

@stnguyen90
No worries ✌


cannot run tests in release mode (miniscript overflow)

Executing tests in release mode I receive the following error:
error[E0275]: overflow evaluating the requirement `F: FnMut<(&Pk,)>`
  |
  = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`miniscript`)
  = note: required for `&mut F` to implement `FnMut<(&Pk,)>`
  = note: 128 redundant requirements hidden
  = note: required for `&mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut &mut ...` to implement `FnMut<(&Pk,)>`
  = note: the full type name has been written to '/mnt/dmc/zoe/work/bitfinex/rgb-lib-kotlin/rgb-lib/target/release/deps/miniscript-f3a95c2934347d02.long-type-692420154592931688.txt'

For more information about this error, try `rustc --explain E0275`.
error: could not compile `miniscript` (lib) due to previous error

I believe we need to wait https://github.com/rust-lang/rust/issues/110475 to be resolved

Related: https://github.com/rust-bitcoin/rust-miniscript/issues/549

I thought I had fixed this here: https://github.com/rust-lang/rust/issues/110475#issuecomment-1664451519 by patching bdk to miniscript 10, but apparently -- that only fixed it on my local mac build --release (using latest nightly.) It still fails on building in linux in github CI. Here is a run showing all the rust versions for nightly:
https://github.com/redgold-io/redgold/actions/runs/5755791489/job/15603953718
And corresponding cargo file for sub-module: https://github.com/redgold-io/redgold/blob/5ce484a7d881e4ca99bfdb34b4ff088250d340eb/schema/Cargo.toml#L36
The crazy part is this has been happening randomly coming back and forth depending on which dependencies I add to my Cargo.build -- this issue started happening again when I removed a dependency (libp2p in this case,) which is wild to me because how could removing a dependency break it? There's another dependency (extism) which if I add it back makes a working build fail with this miniscript problem. I am not certain I applied the cargo patch correctly to upgrade to 10, or if something else in my project is causing it to fail? I can't reproduce this issue in a small library either. If I pull that module I referenced out into its own project, it builds with --release no problem which is the strangest part. At this point I'm thinking of just forking and increasing the recursion limit and publishing a separate artifact to get around this. Again not sure if I'm doing something wrong but other people seem to have this issue too.


Next.js webpackJsonp_N_E

Good morning,
Decompiling it does not work for a site made in Next.JS.
The structure is as follows:
(window.webpackJsonp_N_E = window.webpackJsonp_N_E || []).push([[4], {     "wd/R": function(e, t, n) {         (function(e) {             e.exports = function() {                 "use strict";                 // module code here             }()         }         ).call(this, n("YuTi")(e))     } }]);

Will work on this after the CNY. It looks like the support coverage of webpackJsonp_N_E is not good enough.


remove not consensus data

It is necessary to remove conflicts that have arisen after the merge of changes with checking for valid characters in the permlink.

#485


[Cthulhu Awakens] Fix: Tweak the compendium drop handler after new rake

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Small fix: Tweak the compendium drop handler after new rake (which renamed one of the categories)

Character Sheet Info Roll20 Internal Use only.


Unable to get account config data.

me too

the same

same for me any fix

yes same please fix [Master HamsterKombat Bot][INFO] [Account 1] Account ID: 795302543, Account detected as bot: False
[Master HamsterKombat Bot][INFO] [Account 1] Getting account config data...
[Master HamsterKombat Bot][ERROR] [Account 1] Unable to get account config data.

Just submitted a PR: https://github.com/masterking32/MasterHamsterKombatBot/pull/115
comment line 1068 in main.py with a # at the beginning of the line if you want to do this yourself:
#            or "clickerConfig" not in AccountConfigData

Just submitted a PR: #115 comment line 1068 in main.py with a # at the beginning of the line if you want to do this yourself: # or "clickerConfig" not in AccountConfigData

Thanks

I have the same issue :/

Just submitted a PR: #115 comment line 1068 in main.py with a # at the beginning of the line if you want to do this yourself: # or "clickerConfig" not in AccountConfigData

thanks working


Autoupdate missing after changing motion line length config

Reload required in motion detail view after changing the motion line length config.

@emanuelschuetze This works for me in the motion detail view as well as in the projector.

Already fixed in current master.


Travis CI: Add a parallel test run on xcode10.2

Thank you for this contribution.
If I understand correctly, it will build iOS_system on two different versions of Xcode, 9.2 and 10.2?

That is correct.


[BUG] PDF does not align to the center

Describe the bug
PDF content doesn't align to the centre.
How to Reproduce
Steps to reproduce the behaviour:

Load the PDF using:

<PDFViewer document={{
            url: 'https://arxiv.org/pdf/quant-ph/0410100.pdf',
}} />

The PDF loads with left alignment

Expected behaviour
The PDF should align with the centre by default. If not there should be an option to align the pdf content.
Screenshots

PDF File
https://arxiv.org/pdf/quant-ph/0410100.pdf
Details (please complete the following information):

Device: [Desktop]
OS: [Windows 10]
Browser [safari, chrome]
Version [2.2.3]

It does...

Use css & canvasCss props to customize...


fix oss ak expire handler bug

fix oss ak expire handler bug

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[3.2.1] struct error in shader

Godot version:
3.2.1
OS/device including version:
Win 10
Issue description:
the struct key word doesn't seem to exist as per the godot documentation
(doc)
when typed in to the shader editor in the global scope it returns the error [ Expected constant, function, uniform or varying]
Steps to reproduce:

open attached project
open errorous shader.shader
note error
Minimal reproduction project:
errorous project.zip

https://docs.godotengine.org/en/latest/tutorials/shading/shading_reference/shading_language.html#structs is the documentation for the latest branch, i.e. 4.0.
Godot 3.2 doesn't have structs: https://docs.godotengine.org/en/3.2/tutorials/shading/shading_reference/shading_language.html

Ah, my bad! thanks


Can't update on v1.13.x

I can't update to version 2.6.5 on nodebb v1.13.1

In log i can read this.
Command failed: npm install<EMAIL_ADDRESS>--save npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself. npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself. npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself. npm ERR! code EEXIST npm ERR! path /home/user_radforum/radforum/nodebb/node_modules/.bin/qrcode npm ERR! Refusing to delete /home/user_radforum/radforum/nodebb/node_modules/.bin/qrcode: is outside /home/user_radforum/radforum/nodebb/node_modules/qrcode and not a link npm ERR! File exists: /home/user_radforum/radforum/nodebb/node_modules/.bin/qrcode npm ERR! Remove the existing file and try again, or run npm npm ERR! with --force to overwrite files recklessly. npm ERR! A complete log of this run can be found in: npm ERR! /home/user_radforum/.npm/_logs/2019-12-29T14_40_20_938Z-debug.log
So i delete this file -> ..Path_to_nodebb../nodebb/node_modules/.bin/qrcode
After this, i can update to latest version.

Problem solved.


TypeError: urlopen() got multiple values for keyword argument 'body' ON patch_namespaced_deployement

With the call "patch_namespaced_deployment('service', 'default', 'hello')" on AppsV1beta1Api()
I get the below error. Am I doing something wrong here?

Traceback (most recent call last):
File "deploy.py", line 26, in
v1.patch_namespaced_deployment(d[3], 'default', 'hello')
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/apis/apps_v1beta1_api.py", line 1550, in patch_namespaced_deployment
(data) = self.patch_namespaced_deployment_with_http_info(name, namespace, body, **kwargs)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/apis/apps_v1beta1_api.py", line 1646, in patch_namespaced_deployment_with_http_info
collection_formats=collection_formats)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/api_client.py", line 329, in call_api
_return_http_data_only, collection_formats, _preload_content, _request_timeout)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/api_client.py", line 153, in __call_api
_request_timeout=_request_timeout)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/api_client.py", line 399, in request
body=body)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/rest.py", line 295, in PATCH
body=body)
File "/usr/local/lib/python2.7/site-packages/kubernetes/client/rest.py", line 166, in request
headers=headers)
File "/usr/lib/python2.7/dist-packages/urllib3/request.py", line 78, in request
**urlopen_kw)
File "/usr/lib/python2.7/dist-packages/urllib3/request.py", line 141, in request_encode_body
**urlopen_kw)
TypeError: urlopen() got multiple values for keyword argument 'body'

I get the same error when specifying the path op:
 v1.patch_namespaced_deployment(d[3], "default", {"op": "replace", "path":"/spec/template/spec/containers/0/image", "value":Url})

what is your kubernetes and urllib3 version?

urllib3 (1.8.2)
kubernetes (2.0.0-snapshot)

your urllib3 is old. see here: https://github.com/kubernetes-incubator/client-python/blob/master/requirements.txt

@mbohlool You state that @arrkaye 's urllib3 version is old. He states the has a version of 1.8.2 and you point him to a link requiring 1.19.1. Would you please clear that? I have just upgraded mine and got:
`Package python-urllib3:
p   1.9.1-3                                       oldstable                 500
i   1.16-1+deb8u1                                 oldstable                 500
Package python-urllib3-whl:
p   1.9.1-3                                       oldstable                 500
Package python3-urllib3:
i A 1.9.1-3                                       oldstable                 500 `
but still can't resolve this error.


Forms : Refactor ComboBoxFieldState into CodedValueFieldState

Summary of changes

Renamed ComboBoxFieldState to CodedValueFieldState.
CodedValueFieldState class is now open.
Renamed ComboBoxFieldProperties to CodedValueFieldProperties.
CodedValueFieldProperties is now open.
Renamed combo package to codedvalue.
Moved placeholder calculation from CodedValueFieldState and into the ComboBoxField. This removes the Context dependency for the state object and a possible bug when the isRequired flow changes.
Changed List<CodedValue> to a List<String> in CodedValueFieldProperties, since this mapping happens at the picker level anyway.
Updated the ComboBoxDialog to use a Map<Any?,String> which represents <CodedValue.code, CodedValue.name> and to display the name instead of the code. ComboBoxDialog.onValueChange is called with the code.
Added compose preview for a ComboBox.

Thanks @sorenoid, will make a separate PR for that. Merging this.


[CRITICAL] [VIP Travel] Handle tripName updates to display Trip Title and dates in newDot

We need to display a Trip Title on newDot, but we weren't getting it with just PNR updates (which we decided to get here).
We asked Spotnana to send us both the trip_v3 webhooks and PNR webhooks. We'll be receiving the PNR data with the trip_v3 webhooks, which we should ignore. We'll take the trip data and always update the trip title:

...from the data returned via the trip_v3 web hook:
{
"tripId": "6926658168",
"tripName": "JFK SFO Trip",
"tripDescription": "JFK SFO Business Trip",
"applicationId": "97ab27fa-30e2-43e3-92a3-160e80f4c0d5",
"startDate": {
"iso8601": "2017-07-21"
},
"endDate": {
"iso8601": "2017-07-21"
}
}

will deal with this during the weekend probably

This should be ready to review

should go with the next deploy


Fix Pull Request Template

Describe the bug
I tried renaming the file but still could not get the pull request template to work as it does in the other repos. This is the pull_request_template.md
To Reproduce
Steps to reproduce the behavior:

Go to 'Pull Requests'
Click on 'New Pull Request'
Select base and head or fork
See error 'Pull request template isn't loaded in the PR box'

Expected behavior
Pull request template is pre-loaded when contributor clicks new pull request

This might be useful

@bartekpacia would you like to work on this?


Integrating with OSS-Fuzz

Greetings SPIRV-Cross developers and contributors,
We’re reaching out because your project is an important part of the open source ecosystem, and we’d like to invite you to integrate with our fuzzing service, OSS-Fuzz. OSS-Fuzz is a free fuzzing infrastructure you can use to identify security vulnerabilities and stability bugs in your project. OSS-Fuzz will:

Continuously run at scale all the fuzzers you write.
Alert you when it finds issues.
Automatically close issues after they’ve been fixed by a commit.

Many widely used open source projects like OpenSSL, FFmpeg, LibreOffice, and ImageMagick are fuzzing via OSS-Fuzz, which helps them find and remediate critical issues.
Even though typical integrations can be done in < 100 LoC, we have a reward program in place which aims to recognize folks who are not just contributing to open source, but are also working hard to make it more secure.
We want to stress that anyone who meets the eligibility criteria and integrates a project with OSS-Fuzz is eligible for a reward.
If you're not interested in integrating with OSS-Fuzz, it would be helpful for us to understand why—lack of interest, lack of time, or something else—so we can better support projects like yours in the future.
If we’ve missed your question in our FAQ, feel free to reply or reach out to us at<EMAIL_ADDRESS>Thanks!
Tommy
OSS-Fuzz Team

Dawn currently has fuzzers for some of the spirv-cross API that are run via Chromium & Clusterfuzz. These fuzzers are limited to how Dawn uses spirv-cross, and will go away once Dawn is no longer directly accessing spirv-cross directly, but instead using spvc in shaderc.
As an aside fuzzing, spirv-cross in general is difficult since it depends on using exceptions/asserts for error reporting in the parsing code. In general it assumes well formatted, trusted input to make progress. I had to use a signal trap to filter the assertions during fuzzing to allow the fuzzers to make progress. You can see https://dawn.googlesource.com/dawn/+/refs/heads/master/src/fuzzers/DawnSPIRVCrossFuzzer.cpp for how I did this. The comments in this code are a touch stale, since it was decided to not modify spirv-cross's behaviour, but instead put a wrapper around it for integrating with Dawn.

Ok, thank you.


[MLIR] Adds a runtime switch to trigger MLIR

Currently, in order to use MLIR, we need to rebuild the MIGraphX with a CMake flag.
This commit changes this have an switched polarity in the cmake flag -- i.e. MIGRAPHX_DISABLE_MLIR_BUILD. Then adds ENV var MIGRAPHX_MLIR to enable MLIR offloads in the fuse_mlir pass.
closes : https://github.com/ROCmSoftwarePlatform/llvm-project-private/issues/810

This will also break packages being built since it wont build MLIR. MLIR needs to be added to requirements.txt(and removed from the dockerfile). However, we need to make sure we dont build MLIR on GH actions.

This will also break packages being built since it wont build MLIR. MLIR needs to be added to requirements.txt(and removed from the dockerfile). However, we need to make sure we dont build MLIR on GH actions. It would be better to just download prebuilt packages from repo.radeon.com.

I see. I think our packaging process is a bit WIP.
How about I re-instate the explicit enabled cmake flag and also keep the runtime switch ? (so that we can do measurements w/o needing to rebuild).
cc: @sjw36

How about I re-instate the explicit enabled cmake flag and also keep the runtime switch ?

Sounds good to me.

@pfultz2 , I ve made the changes now. PTAL when you have some time.

Im trying to get this PR past CI...
https://github.com/ROCmSoftwarePlatform/AMDMIGraphX/actions/runs/4373827517
Does anyone know what the following mean:
release / MIGraphX Performance
Input required and not supplied: token

OK, I think I dont know/understand how the CI works here. So the comments kind of mean there is an CI (other than GH Actions) that uses the jenkinsfile and runs the all tests 'MLIR Debug'. It would be great to know where to look for this CI.

ok got it -- somehow, I didn't see them being run in the PR. I can see the jenkins runs now.


App model doesn't appear in Django admin in production

Danny,
First of all, thank you for this. It exponentially speeded up my starting a new project. Everything was going along swimmingly until I moved to production. I put this up as a question on StackOverflow that is totally empty.
The short version is this: The model for my app works fine in my local runserver. When I deploy to Heroku, everything works perfectly except my app doesn't show up in the Django admin.
Here's the long version:
I've been building a Django app and testing locally for some time. On my machine, when I visit the admin page, my app appears. When I deploy to Heroku, it no longer does. The odd thing is that when setting user permissions in the admin, I can access the tables for my app in the postgres db, so it doesn't appear to be a db issue, just my custom admin is not being found or loaded.
My directory listing is roughly this
overall
    - myproject
        manage.py
        urls.py
        wsgi.py
        -templates
        -contrib
        -users
        -myapp
            admin.py
            forms.py
            models.py
            fixtures

My urls.py file
# -*- coding: utf-8 -*-
from __future__ import unicode_literals

# Uncomment the next two lines to enable the admin:
from django.contrib import admin
admin.autodiscover()

from django.conf import settings
from django.conf.urls import patterns, include, url
from django.conf.urls.static import static
from django.views.generic import TemplateView

urlpatterns = patterns('',
    url(r'^$',  # noqa
        TemplateView.as_view(template_name='pages/home.html'),
        name="home"),
    url(r'^about/$',
        TemplateView.as_view(template_name='pages/about.html'),
        name="about"),

# Uncomment the next line to enable the admin:
    url(r'^admin/', include(admin.site.urls), name="admin"),

# User management
    url(r'^users/', include("users.urls", namespace="users")),
    url(r'^accounts/', include('allauth.urls')),

# Uncomment the next line to enable avatars
    url(r'^avatar/', include('avatar.urls')),

# Your stuff: custom urls go here

) + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)

In myapp/admin.py, I have this:
from django.contrib import admin
from myapp.models import State, Facility, Report
from forms import ReportForm

@admin.register(State)
class StateAdmin(admin.ModelAdmin):
    list_display=('state_name','state_abbrev')
    ordering=('state_name',)

@admin.register(Faculty)
class FacultyAdmin(admin.ModelAdmin):
    pass

In my django settings on heroku, i have this:
INSTALLED_APPS = ('django.contrib.auth', 'django.contrib.contenttypes', 'django.contrib.sessions',
'django.contrib.sites', 'django.contrib.messages', 'django.contrib.staticfiles', 'suit',
'django.contrib.admin', 'crispy_forms', 'allauth', 'allauth.account', 'allauth.socialaccount',
'users', 'myapp', 'djangosecure', 'gunicorn', 'storages', 'collectfast')

But my admin only contains users, sites, accounts in it, and not myapp, like it does on my local testing environment.
(In a separate issue, when I try to use heroku's foreman to test the app locally, it can't seem to find the environment variables to run gunicorn, so that's tripping up my debugging too. )
Please, please help me figure this out. It's driving me crazy. Any advice would be greatly appreciated.
Thanks!

closing this as we don't have any conclusive answer for it. Thanks @tommeagher for reporting this.
Anyone facing the same issue, please re-open it and we would be happy to help.

i have also the problem, i dont know how resolve this pls any help me

Hey @dkpapo, if you can show some code then I'm happy to help.
PS: If you are able to isolate your problem and make it reproducible, that going to be super helpful.
cheers,

thanx theskumar i  resolved the problem was use admin.autodiscover() in my urls
from django.conf.urls import patterns, include, url
from django.contrib import admin
admin.autodiscover()
urlpatterns = patterns('',
# Examples:
# url(r'^$', 'mysite.views.home', name='home'),
# url(r'^blog/', include('blog.urls')),
url(r'^admin/', include(admin.site.urls)),
.........

)

:+1:

try to restart uwsgi. it works for me.

Hey @dkpapo, if you can show some code then I'm happy to help.
PS: If you are able to isolate your problem and make it reproducible, that going to be super helpful.
cheers,

from django.contrib import admin
Register your models here.
from .models import Contact
admin.site.register(Contact)

In production make sure you make the debug mode in settings.py to FALSE
and ALLOWED_HOST =["*"]
First write this command:
sudo systemctl reload nginx.service
if this doesn' work then write this
sudo reboot
after this run
systemctl status nginx.service
this shoudl do the job


Upgrade to Polymer 3.

Fixes #73
Mostly automated migration by the Polymer modularizer tool:
https://polymer-library.polymer-project.org/3.0/docs/upgrade
Additional manual steps taken:

migrate intex.html and iron-component-page
modularizer dropped html templates, had to restore them
removed bower
migrated tests to node imports; removed instantation tests which don't seem
to translate well to polymer 3; added test command to README
bumped package version to 3.0.0
update travis-ci config to run polymer test
auto-format everything using webmat

Lovely, thank you @jheidel!


Fix bug with Bootbox in templates

There was a bug with the implementation of the base and inherited templates that prevented Bootbox dialogs from being displayed, as the JavaScript wasn't loaded in the right sequence. This commit fixes it.

There was another bug with the message display when updates were submitted. The second commit fixes that bug.


Created stdlib folder - no functionality change.

This PR creates a stdlib folder for storing distributions.jl and RandomMeasures.jl.

Ready to merge from my side.


Fixed python3 compatibility of the rouge module

The rouge.rouge module cannot be properly run in python3 due to the use of python2's itertools.izip_longest function. In python3 this function has been renamed to zip_longest. This PR uses the six.moves.zip_longest function to fix the compatibility with python3.

I signed it!


Tag commits with dev (and other) npm release versions

Tagging all commits with the Npm release version this commit was released under will help us to better understand which Npm package version includes which code.
Until now we only tag release versions of prisma on the commits in this repository (Released as latest on Npm). In the past it was not viable to also tag the dev and patch-dev or integration releases, as all tags were listed on the Releases page of the repository - far to noisy. This is not the case any more <3

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Changes for visual studio 2017

This change is for visual studio 2017

Can one of the admins verify this patch?

Oh, I just found it is duplicated with #8008

@snnn shall we close it as duplicate?

Let's close this as the other PR is almost ready to merge.


[Bug Report] table in the Chrome version of the 31.0.1650.63 appears to be not aligned layout situation

Element UI version
1.2.9
OS/Browsers version
windows 10 / Chrome 31.0.1650.63
Vue version
2.3.0
Reproduction Link
https://jsfiddle.net/pj71jkyw/148/
Steps to reproduce
打开链接
What is Expected?
表格head的线与body的线不对齐
What is actually happening?
表格head的线与body的线对齐

Chrome 31 is too old, so we do not guarantee compatibility.


Snapcast: Add configuration options

We have a new contributor, @whc2001 thank you for your help 🎉🎉
Some configuration options and some related details are fixed.
Fixes the problem with tts cutting off in the middle of the time, but there is still a problem with very short tts, like “Hi” where it just stops the music that is playing and comes back with the music after a few seconds and you never hear the chime or the “Hi”.

@whc2001 are you using pre commit ?

@whc2001 are you using pre commit ?

Ah sorry I think it's not properly configured on the Windows machine I used to push

Ah sorry I think it's not properly configured on the Windows machine I used to push

Windows is not recommended for python development, especially not for MA development.
Best is to use MacOS or Linux. Dev container could potentially work but has caveats.

Want to merge this now or do you want to adjust those category labels first ?

Want to merge this now or do you want to adjust those category labels first ?

The categories are already set but we are going to do the localization here, otherwise everything will remain without description

@whc2001 I am already editing the frontend en.json

@whc2001 I am already editing the frontend en.json

https://github.com/music-assistant/frontend/pull/673

@whc2001 I am already editing the frontend en.json

music-assistant/frontend#673

Just got there, I'll check

Already got my latest change in https://github.com/music-assistant/frontend/pull/674 and https://github.com/music-assistant/server/pull/1695


Update transparent-data-encryption-byok-sql-managed-instance-powershe…

…ll.md

@CarlRabeler : Thanks for your contribution! The author, @MladjoA, has been notified to review your proposed change.


Fixed final value of linear decay lr

I found that the current linear decay scheduler is incorrect. When self.num_iters reached self.end_iter, the learning rate lr is not 0

Thanks for catching this! This PR also made me realize the exponential lr decay needs the same fix. I'll add that PR as well shortly.


Initial support for revoking user certs

@jtimberman appreciate a review if possible please.

Hey @flaccid, I don't work on cookbooks regularly and don't have cycles for code review. If you don't have cycles either, perhaps this cookbook should be transferred to sous-chefs (formerly chef-brigade)?
http://sous-chefs.org/

We'd love to get this one into sous chefs. It's an important cookbook for the community and it would help get additional eyes on the code.

@jtimberman time is not in issue, was just after a good pair of different eyes and you're in this org still. @tas50 happy to transfer it to sous-chefs if I can be added to the org; failing that sous-chefs are welcome to just help right here :)

I would prefer if we could use a custom-resource for users rather than using/adding a data_bag.


Performance enhancements to collecting labeled metrics

Replaced the use of ConcurrentDictionary<T, K>.GetOrAdd in Collector.GetOrAddLabelled as the function was allocating a closure
Made LabelValues a struct to avoid heap allocations
LabelValues precomputes it's hash code now- as this never changes, it doesn't make sense to re-compute it. Saves CPU when repeatedly recording metrics with the same labels.
LabelValues.LabelValues is only initialised once per unique combination of label values, this saves CPU + memory when repeatedly recording metrics with the same labels.

Changes allow recording metrics with labels to be ~3x faster and allocate ~12x less memory

Method
Mean
Error
StdDev
Gen 0
Allocated

RecordLabelsNew
321.5 ns
9.348 ns
13.99 ns
0.0176
56 B

RecordLabelsOld
1.169 us
0.0204 us
0.0306 us
0.1888
600 B

@djluck Can you attach benchmark code? I think, I can use it in my fork: https://github.com/phnx47/Prometheus.Client

@phnx47 sure, it is:
void Main()
{
	var config = DefaultConfig.Instance
		.With(new Job("X64-Clr-RyuJit", RunMode.Medium, EnvMode.Clr)
		{
			Accuracy = { RemoveOutliers = false },
			Env = { Runtime = Runtime.Clr, Platform = Platform.X64, Jit = Jit.RyuJit },

		})
		.With(MemoryDiagnoser.Default);

	BenchmarkRunner
		.Run<PrometheusBenchmark>(config);
}

public class PrometheusBenchmark
{
	private Counter nolabels, labels;
	public PrometheusBenchmark()
	{
		nolabels = Prometheus.Metrics.CreateCounter("my_counter", "help");
		labels = Prometheus.Metrics.CreateCounter("my_counter_labels", "help", "label_one", "label_two", "label_three", "label_four");
	}

//	[Benchmark]
//	public void RecordNoLabels()
//	{
//		nolabels.Inc();
//	}

	[Benchmark]
	public void RecordLabels()
	{
		labels.Labels("label_one_val", "label_two_val", "label_three_val", "label_four_val").Inc();
	}
}

@phnx47 out of interest, why did you fork this repo?

prometheus-net is slowly developing, I can't waiting. I fixed many bug, supported netstandard and tested it. I'm working on several extensions for my projects and publish on GitHub

Replaced the use of ConcurrentDictionary<T, K>.GetOrAdd in Collector.GetOrAddLabelled as the function was allocating a closure

It isn't correct. Tests no working, when you combination with Label and without Label https://github.com/andrasm/prometheus-net/blob/77428872f1ae94f81f72badf67eaa58654bce6a9/prometheus-net.tests/MetricsTests.cs#L48

@phnx47 not sure if you saw but I pushed a commit to fix the breaking tests.

@djluck I saw. Thanks. But I have some problems in my fork. I'll come back to this a later


sourcesContent added by angular serve in *.map files, cause vscode Go To Implementations not work

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
When I ng build or ng serve, the anguar libraries' *.map files will be modified, a sourcesContent property will be added, which cause vscode Go To Implementations not work.
PS: Those libraries are written by me, and I shipped with both typescript source code and source map. After uninstall then reinstall my libraries, Go To Implementations will work again. I think the modification made by angular caused this. Is there a way to disable this behavior?
🌍 Your Environment

Angular CLI: 11.2.18
Node: 16.13.1
OS: win32 x64

Angular: 11.2.14
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
... service-worker
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1102.18
@angular-devkit/build-angular   0.1102.18
@angular-devkit/core            11.2.18
@angular-devkit/schematics      11.2.18
@angular/cli                    11.2.18
@schematics/angular             11.2.18
@schematics/update              0.1102.18
rxjs                            6.6.7
typescript                      4.0.8

The is caused by NGCC which is needed for Angular libraries that have been compiled with older versions of Angular. NGCC is no longer needed when compiled with Angular 13+.
That said, in general it is not expected that TypeScript sources are included in node_modules See: https://github.com/angular/angular-cli/issues/22701
Closing as NGCC doesn't support such a library setup.


Accept strings for file fields

Data URLs would be useful to allow - like images created with Canvas.

This is easily done in userland though... fetch(objectUrl).then(res => res.blob()).


modified merge variables in Tuolumne template

Modifications in files other than TuolumneLetterLoad.vue are due to Prettier

Merged per @manishapriya94 who's in a bit of an access pinch 👍


Fix code typo in model.save section

Noticed this inconsistency in the model.save section. I'm guessing this code example was slightly copied from the original Backbone docs, see: http://devdocs.io/backbone/index#Model-save

Travis build failed, but I'm fairly certain it's not my fault 😅

That's embarrassing! Thanks for catching that


create database seed

I can do this for u and u do blade and mock plsssss

user seed 1 manager 1 founder
@omhsin007


[Feature Request:] Overriding the stdenv not yet possible

Describe the bug
Currently it is not easily possible to override the stdenv, i. e. to compile to certain targets in rust / cargo.
To reproduce
See sscce for this issue.
Version

devenv: 0.6.2

We're getting closer to fixing this, I'm not sure if overriding makes sense for the whole devenv but rather than parts of it?

For Rust see #900
Otherwise I'd suggest we add an option to devenv.yaml:
stdenv: pkgs.swift.stdenv

Would that work for your use case?

#900 looks like what is missing, the devenv.yaml option could be feasible from flakes as well then further up the road, so yeah, this would be a nice addition.

my use case would be stdenv = pkgs.stdenvAdapters.useMoldLinker pkgs.clangStdenv;

Can you try https://github.com/cachix/devenv/pull/1092 and tell me if that works for you as expected?

Tried and worked. Thanks all for the work !


Other functions while streaming

Hi,
I'm working with the Arduino IDE example on a TTGO T camera.
Once the streaming starts is it possible to run other functions on the device?
I want to use the PIR sensor on this board to wake the device and allow video to be viewed for 5 minutes and then the device to sleep until the next input from the PIR sensor.

I worked out a way using core 0 for my other functions.


Option to drop the UUID from filenames

I have a use case where I need the filename in S3 to match the original. As far as I know, this is currently not possible with the library as the signing endpoint prepends the filename with UUID. Is there currently a way to bypass this? If not, would it be ok to implement an option to do so?

Pull request #112 resolves this.

Pushed v4.0.2 with this change.  Thanks

Thank you!


Detect Intel vs ARM

Hi,
thank you for this amazing tool. I'm trying to create a script around it to be able to have some kind of reinstall station that would detect if a macbook is connected and then install the latest ipsw.
I have it kinda worked out but I want to be able to detect an Intel mac. Do you think it could be possible to implement?
Thanks!

It should at least be possible to pick up on differences between Intel and ARM machines based on vendor command responses, but I think you might be able to get a more accurate answer via some product ID thing. I'd suggest looking at the TI programming manual for the TPS6598x and seeing if there is any command or register to get the VID/PID of the remote device.

Thank you I will try to look at it. In the meantime I found a workaround by looking if it behaves like an M1 mac and disregarding everything else.

@marcan Can I use this tool to boot the Inter device to DFU mode?
I have been using it to solve the problem of M1 brushing, and I hope Intel machines can support it as soon as possible.


[Bug] Ingress Metadata cannot obtain the 'field. title. io/publicEndpoints' annotation

Search before asking

[X] I had searched in the issues and found no similar issues.

Java Version
No response
Scala Version
2.12.x
StreamPark Version
2.0.0
Flink Version
1.16.1
deploy mode
kubernetes-application
What happened
After I configured Ingress, I found that I could not access the web UI.. After investigation, it is found that in the IngressController.ingressUrlAddress() method, the ingress address is obtained by obtaining the field.cattle.io/publicEndpoints annotation in the Ingress Metadata. When I debugged at the breakpoint, I found that the annotation did not exist in the Metadata of the ingress I deployed, which caused the failure to obtain the ingress address.
This is related to the ingress controller type, not all ingress controller types support generating this annotation. Therefore, it may be necessary to change the way to obtain the Ingress address.
Error Exception
No response
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!(您是否要贡献这个PR?)

Code of Conduct

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

This is certainly an issue that needs to be addressed. The original code fetches the ingress address information from field.cattle.io/publicEndpoints, but this is specific to the Rancher ingress provider and is unlikely to work with other providers.
Can you provide more detailed suggestions for fixing this? I'm eager to hear your ideas.😃

@Al-assad  As you said, after reviewing some information, I found that field.cattle.io/publicEndpoints annotation is related to the 'Rancher address provider'.But I don't have the Rancher platform for testing, so I'm not sure if it's really what I think :sweat_smile: .
I suggest modifying it as follows:
try {
    val client = new DefaultKubernetesClient
    val ingress = client.network.ingress.inNamespace(nameSpace).withName(clusterId).get
    val rules = ingress.getSpec.getRules
    val hostname = rules.get(0).getHost
    val path = rules.get(0).getHttp.getPaths.get(0).getPath
    logger.info(s"Retrieve flink cluster $clusterId successfully, JobManager Web Interface: https://$hostname$path")
    s"https://$hostname$path"
} catch {
    case e: Exception => throw new RuntimeException("[StreamPark] get ingressUrlAddress error.", e)
}

Use the above method instead of the old way of getting information from annotations, which will be more general. What do you think of this plan ?

Good Job! I noticed something interesting when reading the Streampark code - it obtains Ingress information through Rancher's specific metadata. However, the default fabric8 kubeclient uses the networking.k8s.io/v1beta1 API, which unfortunately has been officially deprecated since Kubernetes 1.22.
For changes to the Ingress API, please refer to: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#ingress-v122
To ensure a more complete modification approach, both v1 and v1beta versions of the Ingress API should be retained, with v1 taking priority.
I've written a quick implementation that I hope will be helpful to you. Looking forward to seeing your PR soon :)
// for kubernetes 1.22+
lazy val fromV1 = Option(client.network.v1.ingresses.inNamespace("infra").withName("minio-test").get)
  .map(ingress => ingress.getSpec.getRules.get(0))
  .map(rule => rule.getHost -> rule.getHttp.getPaths.get(0).getPath)

// for kubernetes 1.22-
lazy val fromV1beta1 = Option(client.network.v1beta1.ingresses.inNamespace("infra").withName("minio-test").get)
  .map(ingress => ingress.getSpec.getRules.get(0))
  .map(rule => rule.getHost -> rule.getHttp.getPaths.get(0).getPath)

Try {
  fromV1.orElse(fromV1beta1)
    .map((host, path) => s"https://$host/$path")
    .map(println)
}.getOrElse(throw new RuntimeException("[StreamPark] get ingressUrlAddress error."))

@Al-assad thanks for your advice :smile:


I want to know how to keep the original mark after reading the image

I want to know how to keep the original mark after reading the image. There is no color mark when the image is reconstructed in 3D :(

by the way, I'm using vtk.vtkNIFTIImageReader() to read the nifty image and want to reconstruct it in 3D

Thanks  for reaching out, to ensure your question reach the appropriate audience, could you post it on the VTK forum. See https://discourse.vtk.org/
In the meantime, I am closing this issue.


Add App Insights dashboard to Terraform

To assist with this issue - They have a query etc already to go, we just need to translate it into IaC and then show the team how to add more graphs moving forward.
Penny Lischer is the primary contact for this on the dev side.

Attached is the query definitions for the dashboard.
queries.json.txt

Latest query definitions for the dashboard.
queries.json.txt

resolved with PR #https://github.com/CDCgov/prime-reportstream/pull/8438 & # https://github.com/CDCgov/prime-reportstream/pull/8150


Manual approval required for workflow run<PHONE_NUMBER>: Aprove of release on Prod environment

Please approve or deny the deployment of the relaese version to the Prod environment.
Workflow is pending manual review.
URL: https://github.com/Water-XP-Scrumban-Fall/BigProject/actions/runs/6302818450
Required approvers: [DrShadowPLay]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

yes

All approvers have approved, continuing workflow and closing this issue.


Specify minSdkVersion in the README

The minSdkVersion for the library is 17. A lot of developers are still supporting lower versions, i.e they can't use this library.
Perhaps the minSdkVersion should be specified in the README?

Thanks! Good call. The minSdkVersion is noted on the README now.
I've brought the minSdkVersion down to 16, and did some stuff to workaround lack of Canvas.clipPath support on SDK < 18.


React native style  ".native.ts" will create type overlaps

Noticed this in SnackUI today. The use of the same exports in a .native.ts file will cause overlaps in the bundled output.
This one seems a bit simpler than sourcemaps, as I think we could have simple rules to decide which to select.
Not sure which is better, allowing config based on filename, or more programmatic (filterTypes: (fileName, types) => ) maybe).

Sorry, did not understand this. What kind of overlaps?
The tool adds nothing to the generated .d.ts file. Just renames modules and removes some declaration.
Any abnormally means incorrect original .d.ts file.
Can you provide a simple repro?

I think it has to do with the name collision issue, if you have:
src/Component.ts with:
export type uid = number

and also have src/Component.native.ts with the same code, you'll get an error. I noticed that TypeScript seems to just choose one of them for you, so it's not a big deal for me, though it'd be nice to be able to choose.

Just marking any unwanted symbol as with @internal jsdoc comment would exclude it from exports.
There is also an internal option. Setting it to something like **/*.native would exclude every .native.ts file from export. Would it help?


add test for #46

Hey @tanner0101, you just merged a pull request, have a coin!
You now have 662 coins.


Update client "ready" call example

Update client "ready" call example to indicate not to discard it. With the example as is, a client would not receive a "ready" reply from the Interactive service.

That's intended. Calls that don't return a result and don't need to be awaited upon, such as the ready call, are generally permitted to be 'fire and forget'.

Agreed, however, the documentation as written can lead a developer to believe that they should expect a reply (as an example successful reply is shown) whereas the example request would not have a reply returned. I wonder if it could be adjusted slightly to make this more clear?

Reviving this since it has been stuck in my "review requested" forever.
They can expect a reply from the method if they ask for one. But, in this and a couple other instances, the reply isn't meaningful, so I'd rather not take up resources on the client-side keeping track of the calls. In the protocol document thus far I have marked the examples for methods where replies aren't meaningful with discard.

Sounds fair to me!


Small typo

There was a small typo in the LP page.
Please use this template to ensure easier processing of your pull request
Category

[x] Content fix
[ ] New article

Fixed a small typo in the Learning pathways page.

Thanks!


Mihir0106 recursive binary search

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added the code for Recursive Binary search using C#
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#6393

@ankitaggarwal23 Please review this PR.


Add support for AWS SecretsManager as BK token provider

Intent
To allow retrieving the BK API token from AWS Secrets Manager.
Problem
The current approach is not flexible enough if we want to keep adding different methods to retrieve the BK API token.
Solution
To represent the behaviour of retrieving a BK API token using an interface and provide particular implementations to all the sources we currently support plus a new one for AWS SecretsManager.

Thanks for this @alloveras! This looks great. Looks like the token package would make for a great candidate for extraction into a library, but let's get this in place for now and we can consider next steps.

Thank you @lox . I absolutely agree about extracting it into a library but I can definitely do that in a separate PR if you guys find it useful 😄

This should be backwards compatible, right?

This should be backwards compatible, right?

I am pretty sure it is given that all the new code is behind new environment variables so they are opt-in features.
The only existing code that I slightly changed is the ssm.go file which I re-arranged and refactored a bit. However, its behaviour has remained unchanged if I am not mistaken.
What do you think ?

I reckon let's move the README.md from the token dir into a section of the main README so folks know how to use it, then let's merge this.

Sure, will do that ASAP and submit a new revision :D

Thanks for the contribution @alloveras!


Thanks for writing the plugin

Is that for the latest ironSource SDK or old one?

It's not ready but its will use the latest one.

Great!
Let me know if you need any help or have question about IronSource SDK.

Thank you.
I think i will ask you, when I can't go forward.