Target runtime win-x64 publish failed

Describe the bug
Everything went well until I attempted to publish the app using win-x64. The publish failed without any error message to identify the issue.
To Reproduce

Clone the project from GitHub.
Right-click on the Wpf.Ui.Gallery project and select "Publish."
Choose the "ClickOnce" option for the target.
Keep the publish location as default.
Set the install location to a valid domain under "From a website."
Keep the settings as default.
Keep the "Sign manifests" as default
Keep all configurations as default, except for the "Target runtime," which should be set to "win-x64."
Click "Finish" to close the profile creation window.
Click "Publish."
Wait for the publish failure to appear.

Expected behavior
The publishing process completed successfully with the 'Target runtime' set to win-x64.
Screenshots

OS version
Windows 10 Pro, OS Build: 19045.4291
.NET version
Target OS: Window
Target OS version: net8.0-windows10.0.22621.0
Supported OS version: 10.0.18362.0
Target Framework: net8.0
WPF-UI NuGet version
Github source code cloned at: 2/29/2024
Additional context
No response

For those encountering the same issue, adding $(InterceptorsPreviewNamespaces);Microsoft.Extensions.Configuration.Binder.SourceGeneration fixed it.


Repeated Overlap Notifications

The application notifies about a possible overlap with a Covid-19 patient.
After I acknowledge that I wasn't in the specified location, the same overlap notification keeps popping again from time to time.
I would expect it to remember my acknowledgement and not pop this notification anymore.

Thanks for your comment, This issue resolved in version 1.1.2, currently available in apple store, will be available in google play soon

Thanks


app: FPS drop with avatar in view

FPS are suddenly dropping due to this avatar in view. As avaer and vis mentioned this is due to avatar having too many materials(50).
https://user-images.githubusercontent.com/35926530/150647998-6d661ad0-572c-4e6d-98bd-3fa915cedfae.mp4

not relevant for miku but we should start thinking abt optimising our avatars and environment assets wherever possible

polycount
textures
texture size


CheckBox should provide a mode to control rendering of nulls as indeterminate vs. unchecked

We have a client app that does not always return values for controls that should be treated as 'false'. Would be great to not have to specify which fields need to be converted. If our checkbox could be told to treat nulls and undefined as false, that would eliminate the need to maintain this kind of fiddly code in the client app.

My immediate reaction to this ticket was that the checkbox was acting as a bit of a canary, indicating that the underlying app code really should/could coerce nulls to false if a T/F model was really what was required.
@lbwexler made the good point, however, that we want to be careful with such routines that would modify data on load to client / population into a control, when otherwise the user might not have interacted with or otherwise meant to change the data value.
And I do agree that the indeterminate [-] checkbox is plain weird and not likely to be understood by many users.
So in short - I support this change, and think it should be the default behavior of our checkbox to render null as unchecked.


Day 1: works like a charm, Day 2: Timeouts - Salesforce Penalities?

I discovered your awesome lib while looking for a way to parse a 1 MB Metadata API WSDL in Apex. I made it work and it was lightning fast. But today I just get timeouts in my scratch org.
HttpRequest request = new HttpRequest();
request.setEndpoint(URL.getOrgDomainUrl().toExternalForm() + '/services/wsdl/metadata');
request.setMethod('GET');
request.setHeader('Cookie','sid=' + UserInfo.getSessionId());
request.setHeader('Content-Type', 'application/xml');

Blob wsdl = new Http().send(request).getBodyAsBlob();
XPath schema = new XPath( wsdl.toString() );

System.debug( SCHEMA.find('/definitions/types/xsd:schema/xsd:complexType').size() );

Also asked a question on StackExchange https://salesforce.stackexchange.com/q/353924/256

Was related to my org always doing FINEST logging.

Ah, thanks for the update! I was wracking my brain trying to think of how this could be happening & how to troubleshoot it.


[backport] Add imports of initializers and math, required in "Define your own function" examples

Backport of #6179

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit e640e43fa6a502b8ef070560beaa6dee0a727fe9, target branch v5) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

The Jenkins failure is unrelated to the PR.


Unwanted namespacing warning/ or error

I originally posted this issue in grpc-go and then here - they told me to post it here instead:
What version of gRPC are you using?
1.59.0
What version of protoc are you using?
libprotoc 25.1
What version of Go are you using (go version)?
1.21.4
What operating system (Linux, Windows, …) and version?
Linux 6.6.4-arch1-1
What did you do?
I have multiple protos defined in a git repo and the generated code is in this repo as well and I use it to import the functionality into the server and clients.
├── events_service
│   ├── admin.proto
│   ├── golang
│   │   ├── admin
│   │   │   ├── admin_grpc.pb.go
│   │   │   └── admin.pb.go
│   │   ├── generate.sh
│   │   ├── go.mod
│   │   ├── go.sum
│   │   ├── shared
│   │   │   └── shared.pb.go
│   │   └── user
│   │       ├── user_grpc.pb.go
│   │       └── user.pb.go
│   ├── shared.proto
│   └── user.proto
├── mailbox_service
│   ├── admin.proto
│   ├── golang
│   │   ├── admin
│   │   │   ├── admin_grpc.pb.go
│   │   │   └── admin.pb.go
│   │   ├── generate.sh
│   │   ├── go.mod
│   │   └── user
│   │       ├── user_grpc.pb.go
│   │       └── user.pb.go
│   └── user.proto
├── member_service
│   ├── admin.proto
│   ├── golang
│   │   ├── admin
│   │   │   ├── admin_grpc.pb.go
│   │   │   └── admin.pb.go
│   │   ├── generate.sh
│   │   ├── go.mod
│   │   ├── registration
│   │   │   ├── registration_grpc.pb.go
│   │   │   └── registration.pb.go
│   │   ├── shared
│   │   │   └── shared.pb.go
│   │   └── statistics
│   │       ├── statistics_grpc.pb.go
│   │       └── statistics.pb.go
│   ├── registration.proto
│   ├── shared.proto
│   └── statistics.proto
└── README.md

events service
mailbox service
member service

They are not related to eachother.
Inside some of them have a protofile called shared (its only used inside the domain and not shared with the others.)
When I have a client application that use all of the protos I get a panic, but I managed to reduce it to a warning after reading the FAQ in the error message.
I have even namespaced the package name in the proto - for instance:
syntax = "proto3";

package eventsservice.shared;

option go_package = "github.com/dk-slack/protos/events_service/golang/shared";

What did you expect to see?
Nothing, no warning and no errors.
What did you see instead?
After running this:
❯ GOLANG_PROTOBUF_REGISTRATION_CONFLICT=warn go run ./cmd

WARNING: proto: file "shared.proto" is already registered
	previously from: "github.com/dk-slack/protos/events_service/golang/shared"
	currently from:  "github.com/dk-slack/protos/member_service/golang/shared"
See https://protobuf.dev/reference/go/faq#namespace-conflict

WARNING: proto: file "admin.proto" is already registered
	previously from: "github.com/dk-slack/protos/events_service/golang/admin"
	currently from:  "github.com/dk-slack/protos/member_service/golang/admin"
See https://protobuf.dev/reference/go/faq#namespace-conflict

I've successfully updated all package names for both the Golang files and the general package names to ensure uniqueness. Despite these changes, I've noticed that warnings persist, specifically related to the names of the proto files.
I find this situation a bit puzzling. The reason is that, despite the presence of warnings, the files have been successfully compiled, and the resulting code is fully functional. The warnings appear to be more like artifacts—remnants from an earlier stage of development—rather than genuine indications of a current problem or issue.

Closing as a duplicate of https://github.com/golang/protobuf/issues/1122
This warning is not an artifact, and still has relevance. The warning even directs you to the relevant information: https://protobuf.dev/reference/go/faq/#namespace-conflict
While your code appears to be fully functional this is likely because you’ve avoided using any of the code that breaks. There is still a subtle conflict of assumptions that could result in misbehavior in the future. Note from the original thread:

The protoregistry package permits looking up a .proto source file by name. If there are two files with same name and path, it has no way to pick between the them.


send email through contact form using firebase realtime database [GSSOC'23]

Description 📝
Fixes/Closes: #287

store user data from contact form

Changes 💬

remove formsubmit.co
added firebase realtime database
changes in Landingcontact.tsx file

Share the video!

🎉 Thank you for your valuable contribution @shivaniyuvrajyadav


column slicing of csc_matrix is two times slower than that of csr_matrix.

Official document says the csc_matrix performs column slicing efficiently. However, in my experiment, column slicing of csc_matrix is two times slower than that of csr_matrix. Why does it happen? Thank you.
Reproducing code example:

This is my experiment result.

And, this is the code.
import scipy.sparse as sp

a = sp.random(100, 10000)

a_csr = a.tocsr(copy=True)
a_csc = a.tocsc(copy=True)

print(type(a_csr), type(a_csc))

%timeit -n 10000 a_csc[:, 10]
%timeit -n 10000 a_csr[:, 10]

Scipy/Numpy/Python version information:
 1.1.0 1.15.3 sys.version_info(major=3, minor=6, micro=6, releaselevel='final', serial=0)

This is a quirk of our current code for handling CSC slicing, which essentially does: mat.T[slices].T
This allows us to share code with the CSR implementation, but does come with the cost of two transpose operations.
I have a PR that should fix this to some degree at gh-7827.

Thank you for your answer and PR. As your PR seems to be merged, I close this issue.
Thank you so much.

With current master, the results are indeed more in line with our expectations:
In [1]: import scipy

In [2]: scipy.__version__
Out[2]: '1.3.0.dev0+47976a8'

In [3]: import scipy.sparse as sp

In [4]: a = sp.random(100, 10000)

In [5]: a_csr = a.tocsr(copy=True)

In [6]: a_csc = a.tocsc(copy=True)

In [7]: %timeit a_csr[:, 10]
64.9 µs ± 1.82 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

In [8]: %timeit a_csc[:, 10]
57.2 µs ± 2.54 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)


Allow context-specific resource limits to be provided

We allow limit_overrides_context to be passed in to jail_code.
If settings.CODE_JAIL['limit_overrides'][limit_overrides_context] exists, then prefer those values to the ones in settings.CODE_JAIL['limits'].
In the edx-platform PR, we pass in the course run id is as the limit_overrides_context.
Per @nedbat 's comment, the PROXY setting cannot be overridden. An error will be logged and the override will be ignored.
(also, this removes outdated AUTHORS file, as we've done with most other repos)
Links

Customer request for root cause (codejail times out sometimes): https://openedx.atlassian.net/browse/CR-2519
Customer request for a workaround for a specific course run: https://openedx.atlassian.net/browse/CR-2862
TNL ticket for workaround: https://openedx.atlassian.net/browse/TNL-7649
edx-platform change (requires this): https://github.com/edx/edx-platform/pull/25483
Add limit overrides for specific course run: https://github.com/edx/edx-internal/pull/3562

:+1: to the test mixin.


Question about using react-router and Flux together

https://github.com/rackt/react-router/blame/master/docs/guides/flux.md#L148
As you said above - that's not how you use them together. What is your way?

I think this is only meant to be related to the last approach with putting Handler into payload.
Other approaches are more “ok”.

we'll have the flux.md guide filled out eventually


Dash index

Based on the question that's come up from time to time of, "where is this thing", I've added an index to dash. and moved the toggles and pa11y dashes into a single app.

I'll remove the other apps once this is up and running.

@jamesgorrie When I can't get to https://dash.wellcomecollection.org/pa11y/index.html any more.
When I try my structured data browser extension kicks in and I get:

Is that because of this work?

@pollecuttn my bad, I removed the old pa11y dash before instating the new one.
It'll be back shortly.


Simple image dialog should remember file name when switching folders

I often write the file name and then realize I'm in the wrong folder and switching clears what I wrote.

It's very hard to tell if the last path segment that a user types here is intended to be the filename or a folder. We would need a multistep input to handle this properly.

@alexr00 assuming it's a file if it has an extension seems like a safe approach?

I can try that. It won't work in every case since not all files have extension, but it might be an improvement.

Thanks, should cover the only cases I would hit the problem


Allow Outputs of Other Sheets/Comands

I would like to better document my inventory, not all boto3 client actions return details of my resources. I would like it to be possible to use outputs from other spreadsheets to generate spreadsheets with more details of AWS resources.
Possible config.yaml
---
Sheets: &sheets
  - name: EKSClusterNames
    service: eks
    function: list_clusters

  - name: EKSClusterDetails
    service: eks
    function: describe_cluster
    output_parameter: EKSClusterNames # loop the client action based in EKS Cluster Name list

inventories:
  - name: aws-test
    aws:
      profile: aws-test
      region:
        - us-east-1
        - sa-east-1
    excel:
      transpose: false
    sheets: *sheets

Hi @evairmarinho , thank you for opening this issue. I'm not sure if I followed your suggestion, could you please give me an example?

Closing this issue for now, please re-open if more details can be provided.


Support more expensive samplers/optmizers

Enable returning results once the minimum cumtime + waiting time for sampling exceeds cumtimes.
e.g. (sss is sampling time for a worker, fff is function evaluation time, and www is waiting time of sampling for the other workers)

Worker 1: ssssfff
Worker 2: wwwwssssff
Worker 3: wwwwwwwwssssfff

Assume that the example shows the simulated runtime.
In this example, Worker 3 should already have two observations when the sampling starts.
However, our implementation does not provide these two observations because we only check when each function evaluation finishes and do not check when SAMPLING starts.
For example, our wrapper does not recognize the fact that Worker 1 and Worker 2 would have returned their results before Worker 3 starts to sample.
Hence, if a sampler or an optimizer is very expensive, our simulation may not give correct behavior. (By design, batch optimization also has not been supported yet.)
In this PR, we would like to address this issue.
There are several things to address:

[ ] Detect min_cumtime + sampling_time somehow
[ ] Release results once their cumtime becomes smaller than min_cumtime + sampling_time (Although we can do it in other ways for non-initialization phases, the initialization phase definitely requires this implementation)
[ ] Do the same for the ask-and-tell version

Codecov Report

Merging #6 (ea64fcb) into main (16500a6) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   99.67%   99.67%
=======================================
  Files          14       14
  Lines         923      923
  Branches      120      120
=======================================
  Hits          920      920
  Partials        3        3


Assembly loader tests

Make the required changes to the FunctionAssemblyLoader to enable better tests and create unit tests.

This addresses test gaps discussed in #214


[Dubbo-6189] 优化部分泛化调用 #6189

https://github.com/apache/dubbo/issues/6189

Codecov Report

Merging #6191 into master will decrease coverage by 0.12%.
The diff coverage is 53.84%.

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

@sqtds hi, thanks for your contribution.
Please resolve confilcts with the latest master branch.

close for long time no response
Please feel free to reopen if you have any question


Use std::iterator_traits to extract iterator_category

In third-party STL implementations, array_t::iterator might be a pointer (e.g., vector in EASTL) rather than a class, in which case directly using array_t::iterator::iterator_category is invalid. This commit fixes it with std::iterator_traits, which handles pointers correctly.

Coverage remained the same at 100.0% when pulling 3972633c00cc34ff4cc6df6063682fb6e01571b3 on Mike-Leo-Smith:patch-1 into f6acdbec2c99670cf932d83cd797f08662e0b564 on nlohmann:develop.

Please run make amalgamate to also add the changes to the single-header version, single_include/json.hpp.

Thanks for the instructions. Should be good now.

Thanks!


Remove 'LCache' from class names in the LCache namespace.

Rename namespace from LCache\LCache to LCache.
Fixes #8.

Coverage remained the same at 100.0% when pulling 048db768c636c7ff79b0063a3c6f0629152438cc on rename-classes into a1b861696f28a544e814c7bebef1157f86073a5e on master.


Runtime Environment Informations

That first Note "When running in Office on Windows, your add-in uses the Internet Explorer 11 browser." means, that as soon as you have specified it, an Add-In will use IE11 regardless of the Office version documented in Browsers used by Office Add-ins, right?
So in my case IE11 is used instead of Microsoft Edge with original WebView (EdgeHTML).
How does that look in the future with the chromium based Edge or is this Information already outdated?
Any additional informations regarding this topic would be highly appreciated.
Some design decisions depend on the embedded browser controls.
Thank you in advance.

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

ID: 4a8cf67b-951d-7bae-5584-565ae9ec45c1
Version Independent ID: b7b60c57-bcb0-679e-5163-97b62fced5d3
Content: Runtimes in the manifest file - Office Add-ins
Content Source: docs/reference/manifest/runtimes.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for reporting this @heebinho.
@Rick-Kirkham, can you please take a look at the Runtimes manifest article? That note does seem out of date.

@heebinho  You are correct. The note is out-of-date. A Shared Runtime will now use Edge (Chromium) on the same Windows and Office version combinations that would support Edge (Chromium) in a task pane. I will get this article updated.


Cannot embed Tweets

Plugin works great for Facebook, Instagram, YouTube, and Vimeo, but I cannot get Tweets to embed. When I add the url, the tweet appears in the editor, but when I save my changes, the Twitter iframe html has an empty head and body. Am I missing something here, or is anyone else having the same issue?

Could you give us a jsfiddle so we can reproduce what you described? Thanks

I can include a jsfiddle of the end result: https://jsfiddle.net/gjaoLjes/
I am using this plugin for my Laravel CMS application. The Tweet appears when entered into the form, but when I save to the database, this is what happens.

Maybe you need to manually call twttr.widgets.load() to re-enable the tweet.

Yes, I know about this problem. That's why the recommended approach is to use oembedProxy and let an oEmbed provider handle embed codes.

I had the same problem, I solved with this script. It must be added on the editing page and on the rendering page to work. It basically get the tweet ID, remove the old one and add a new one at the same place :
              $.getScript('//platform.twitter.com/widgets.js', function(){
	    	var k = 0;
	    	var tweet = document.getElementById('twitter-widget-'+k);
	    	var tweetTest = tweet ? true : false;
	    	var tweetParent, tweetID;
	    	while(tweetTest) {
	    		tweetParent = tweet.parentNode;
	    		tweetID = tweet.dataset.tweetId;
	    		$(tweet).remove();
		        twttr.widgets.createTweet(
				  tweetID,
				  tweetParent
			);
			k++;
			tweet = document.getElementById('twitter-widget-'+k);
	    		tweetTest = tweet ? true : false;
		    }
	    });

I know this is not really clean code (mixing jQuery & javascript) but it works for me.
Note that when you add an embed tweet it also adds a script tag that I remove server-side directly.

I also had this problem:
  $("iframe[id^='twitter-widget-']").each(function() {
      twttr.widgets.createTweet($(this).data('tweet-id'), this.parentNode);
      $(this).remove();
  });

Most likely you will need to clean the parent folder from the other stuff. At least from <script async="" src="//platform.twitter.com/widgets.js" charset="utf-8"></script>
It's a good idea to wrap the original embed code with div and use
  $(this).parent().empty();

Fixed in #277


The 'Review changes' dropdown does not work anymore

Chromium Version 71.0.3578.98 (Offizieller Build) Built on Ubuntu , running on Ubuntu 18.04 (64-Bit)
Firefox 65.0 (64-Bit)
After disabling the plugin everything works normal.

Duplicate of #1805.


json-schema middleware should disallow extra properties

Right now if extra properties are appended we allow them; we probably want a strict schema. This probably means we need to enable the filter property:
var filter = validator.filter({
  required: true,
  type: 'object',
  properties: {
    hello: {type: 'string', required: true}
  },
  additionalProperties: false
})

var doc = {hello: 'world', notInSchema: true}
console.log(filter(doc)) // {hello: 'world'}

lol, no actually if the additionalProperties flag is set it'll be all strict and throw - not an issue heyyyyy


[sugession] New build with DSOAL support

dxvk+DSOAL+OpenAL
repo:
https://github.com/kcat/dsoal
builds:
https://ci.appveyor.com/project/ChrisRobinson/dsoal
https://github.com/kcat/dsoal

Does this do anything other than enable 3D sound? It seems pretty easy to set up, I don't think it needs to be included with VanillaFixes (I only included DXVK because it improves performance).

Added in v1.3.


Update unit test mocks to handle the new query_id field

Problem
We added query_id as a new field on the AdapterResponse event. This needs to be a string to pass validation.
Solution
Mock the sfqid method to return a string literal instead of the mocked object it gets via MagicMock.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

Kick the CI


FreeBSD Support

I am running FreeBSD 12.1. Is there any support to run this application?

Hi @joaquinalcerro - unfortunately we haven't tested on FreeBSD yet - we have a list of supported systems / requirements here: https://onivim.github.io/docs/getting-started/installation#system-requirements

Tinkering with this.

I got Ocaml 4.12 compiled from source and fresh opam, and built esy using opam. The FreeBSD ports have older 4.05 that wouldn't do that.
To build esy, I had to replace cudf dependency version 0.9+dune with just 0.9. I was unable to figure out where to get the 0.9+dune
However, esy fails on building esy-ocaml - the scripts esy-configure and esy-build hardcode bash location and use make instead of gmail, and I was not able to work around this
I tried to patch esy-ocaml and use GH reference to patched version in package.json, but this resulted in esySolveCudfCommand error, which I'm not sure how to fix.

Generally it seems like esy support for FreeBSD is very lacking, so that seems to be the first thing to fix.


Disable non-material zoom buttons and enable double tap to zoom

I removed the zoom in/out buttons, which are not material design (also requested in #214 No. 6), and enabled the wide viewport to allow double tapping to zoom. This also coincidentally fixes #231, and might also fix #67.

I've tried something similar a couple of times and I faced some side effects that made me revert the changes. I'll give your code a try and get back to you.
Thanks!

Markdown files are less readable after applying this patch
Before

After

We might be able to use this only in code files.

I've fixed it so WideViewPort is only used with non-markdown sources.

I've done quite a bit of testing and I'm quite happy with the result.
Thanks!


Update aws-core, http-client-spi, ... to 2.17.269

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.104 to 2.17.269.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #267.


Get the current pull up or pull down state for a pin

The gpioSetPullUpDown function can be used to set or clear resistor pull up or down on a gpio. Is there a function which can be used to get the current pull up or pull down state for a pin?

There is no way of reading the settings of the GPIO internal pulls.
They are set at power-on (GPIO 0-9 pull-up, the rest pull-down) but there is no way of knowing what software may affect the state before your program runs.  You will have to explicitly set the state.
The pull-ups/downs are quite weak, about 50k, so if you need a fixed state use an external resistor of an appropriate value.

I should have read the BCM2835 ARM Peripherals datasheet  before opening the issue :) All the information is there:

Note that it is not possible to read back the current Pull-up/down settings and so it is the
users’ responsibility to ‘remember’ which pull-up/downs are active. The reason for this is
that GPIO pull-ups are maintained even in power-down mode when the core is off, when
all register contents is lost.

Thanks.


Localization issue in holobarrier projector

Description

Reproduction

Screenshots

Additional context

change to limited-charges-
(even though it doesnt use it...)


🐞 Bug: The search feature should be a little more flexible and user-friendly.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The main search option in CircuitVerse is very strict. Making it a little bit flexible might help users find things easily.
Examples:
If we try to search user "chandrachur", our input should be exactly "chandrachur" and seaching "chandrachu" won't give any results.

Similar is the case for projects also.
Searching "big project" gives many results, but "big projec" gives no results.
Steps To Reproduce

Go to https://circuitverse.org/
Search chandrachur and chandrachu and see the results.
Try with some other projects or users also

Expected Behavior
This behaviour should be probably little more flexible. maybe we should implement a better searching algo to get better results.
Screenshots

Device Information [optional]
- OS: MacOS
- Browser:
- version:

Additional context
N/A
Are you working on this issue?
Yes

Hey @tanmoysrt I would like to work on this issue.
Also, I would like to know about your suggestions to improve the search feature.

@Malavi1 yes got it, thanks. But seems like tinkercad also has very strict matching restrictions. We should probably improve that also. @vedant-jain03 WDYT?


Update the UI

The current UI is hard to parse and does not scale with more and more platforms being added.

The UI has gotten an overhaul, while not perfect, it is a lot beter.


Remove Ian Campbell email address from system

resolution_fixed   |    by<EMAIL_ADDRESS>
We need to remove Ian Campbell's email address from the Accessdev system.
What steps are needed for this ?
And update wiki page with this info
https://accessdev.nci.org.au/trac/wiki/Admin Guides/New Admin
Robin

Issue migrated from trac:147 at 2024-01-31 18:08:20 +1100

<EMAIL_ADDRESS>commented

This depends on what's using the email. He was removed from admin lists in changeset:0f0395a07aba245b8e6e72ea6699e85428bd5182/puppet, however some things (like trac tickets he's commented on) will mail his address directly.

<EMAIL_ADDRESS>changed status from new to closed

<EMAIL_ADDRESS>changed resolution from `` to fixed


[翻译] Zero-Trust Security with Service Mesh

地址：https://thenewstack.io/zero-trust-security-with-service-mesh/

take

已发布，关闭issue，见https://cloudnative.to/blog/zero-trust-cybersecurity/


PSG-1034: add sdk version header

Overview of Changes in this PR

add sdk version headers to all requests
set version to short git commit in deploy workflow

The version set in the deploy workflow should be included in the release archive.
How to Test
./scripts/set-version

to see how the version changes.

Won't work as go get will clone the repo at the given version, instead of using the release archive.


Add adaptiveStreamSetting for pausing video when tab in background

Adds an additional setting to adaptive stream for pausing video when the browser tab is in the background/hidden.
Defaults to true.
adaptiveStream: { pauseVideoInBackground: true}

instead of a boolean we could also let this setting have three different options like mobile | always | never in order to differentiate between desktop and mobile usage.


Add different types of volumes to the glossary

Related issues (optional)
Fixes #2732

@mstanleyjones something went wrong with that PR, created a new one

@mstanleyjones, I reworded it and changed the order based on what I understood. PTAL.

You need to remove the unrelated merge commit which is adding commits from another of your PRs here.

Fixed it.


Added traditional Chinese language.

Traditional Chinese was added, I think it would be better to separate Simplified Chinese and Traditional Chinese for Chinese users in different regions.
Not really sure about what does the "Standard" means in the all.js, so I just change them to different kind of Chinese, hope this helps ;)

Sorry man, I'll update the repository. I haven't been receiving the notifications from Github :(

Don't worry at all :)


Don't Merge! PR real time test for source branch full build

OPS Build status updates of commit 23751f5:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

Line 208: [Error] Error happened to execute the task _dependentPackages/CommonPlugins/tools/JoinTOC.ps1
Line 386: [Error] Error happened in build.entrypoint.ps1. Please build your target branch first and then reopen this pull request to do another round of verification. If the issue still happens, open a ticket in http://SiteHelp and include the following error message for our team to troubleshoot: Cannot process argument transformation on parameter 'exitCode'. Cannot convert the "System.Object[]" value of type "System.Object[]" to type "System.Int32".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Issue while building a arm package/binary

I cloned the golang-builder repo and tried building the arm version - but it fails
docker run --rm -ti -v $(pwd):/app quay.io/prometheus/golang-builder:arm     -i "github.com/prometheus/prometheus"     -p "linux/arm64"
# linux-arm64
# Building with: CC='aarch64-linux-gnu-gcc' CXX='aarch64-linux-gnu-g++'
make[1]: Entering directory '/app/1.22/base'
>> building quay.io/prometheus/golang-builder:1.22.6-base
/bin/sh: 1: docker: not found
make[1]: *** [../Makefile.COMMON:26: build] Error 127
make[1]: Leaving directory '/app/1.22/base'
make: *** [Makefile:22: build] Error 2

Can you please assist

For questions/help/support please use our community channels. There are more people available to potentially respond to your request and the whole community can benefit from the answers provided.


Bug: on: { '*': ... } typing has broken after upgrade to v4.32

Description
My mailbox project has broken after upgrading XState from v4.30 to v4.32
Expected result
Works as before
Actual result
(property) StateNodeConfig<Context, any, EmptyAction<"mailbox/ACTOR_IDLE"> | PayloadAction<"mailbox/ACTOR_REPLY", { message: EventObject; }> | PayloadAction<...> | PayloadAction<...> | PayloadAction<...> | { ...; }, BaseActionObject>.on?: TransitionsConfig<context.Context, EmptyAction<"mailbox/ACTOR_IDLE"> | PayloadAction<"mailbox/ACTOR_REPLY", {
    message: EventObject;
}> | PayloadAction<...> | PayloadAction<...> | PayloadAction<...> | {
    ...;
}> | undefined
The mapping of event types to their potential transition(s).

Type '{ "mailbox/ACTOR_IDLE": { actions: any[]; target: string; }; "mailbox/ACTOR_REPLY": { actions: any[]; }; }' is not assignable to type 'TransitionsConfig<Context, EmptyAction<"mailbox/ACTOR_IDLE"> | PayloadAction<"mailbox/ACTOR_REPLY", { message: EventObject; }> | PayloadAction<...> | PayloadAction<...> | PayloadAction<...> | { ...; }> | undefined'.
  Type '{ "mailbox/ACTOR_IDLE": { actions: any[]; target: string; }; "mailbox/ACTOR_REPLY": { actions: any[]; }; }' is not assignable to type 'undefined'.ts(2322)
types.d.ts(339, 5): The expected type comes from property 'on' which is declared here on type 'StateNodeConfig<Context, any, EmptyAction<"mailbox/ACTOR_IDLE"> | PayloadAction<"mailbox/ACTOR_REPLY", { message: EventObject; }> | PayloadAction<...> | PayloadAction<...> | PayloadAction<...> | { ...; }, BaseActionObject>'

Reproduction
https://github.com/huan/mailbox/blob/9a1a72ee189c0c465d34363d5891b250f020fb90/src/wrap.ts#L110-L113
Additional context
The reproduction link is the code that I'm testing today.
The repo will be able to reproduce the typing problem by running:
npm install
npm run lint

cc. @Andarist
Possibly related to https://github.com/statelyai/xstate/pull/3252

Additional information:
This does not work
    on: {
      '*': {
        actions: context.queue.newMessage(actorMachine.id)(normalizedOptions.capacity) as any,
      },
    },

This works
    on: {
      '*': {
        actions: context.queue.newMessage(actorMachine.id)(normalizedOptions.capacity),
      } as any,
    },

So it's wired that I can not understand why "xxx  is not assignable to type 'undefined'."

After the above-linked commit, I can confirm that:

the behavior changed between v4.31 to v4.32
with the v4.31, I can use the Workaround 1 above to make it works; but it does not work anymore with v4.32

This is a backlash from https://github.com/statelyai/xstate/pull/3228/files . So far I've diagnosed this to be related to using generic TEvent["type"] here. I've created a minimal repro demonstrating the problem here.
The interesting part is that even with the old version of thus type the assignability problem doesn't go away. It goes away in the more complex situation that involved createMachine call though. This leads me to believe that perhaps this was never OK - but TS just allowed it accidentally/silently in this more complex scenario. I will take another look at this in the upcoming week.

I've created a TS issue about this ( https://github.com/microsoft/TypeScript/issues/49063 ) and even prepared a potential fix for the issue in the compiler ( https://github.com/microsoft/TypeScript/pull/49064 ). Let's see what happens.

This is an issue in TypeScript itself. Unfortunately, it's not really actionable on our side 😢 I think it's only reasonable to close it here and continue tracking what happens with this in TypeScript


Add a command line flag to specify a different user configuration file to use

Personally, I would find it very useful if I could specify a path in command line for a user configuration file to be used instead of the default one.
I feel that this is the common behavior for command line utilities that use configuration files.

Could you elaborate on the use cases? The way configuration files work is that you can override the default config through local files, is there anything you would like to do that can't be achieved through this mechanism?

The main idea is that I want to have a "default" clangd configuration, which will be used every time I start a new project or open files I don't have specific clangd configuration for. Then, for the projects I do have specific clangd configuration for, I personally never want them to be based on that default configuration.
It's true that I could try to override all my default configurations in each per-project configuration, but there are two issues with that:

It could easily become a headache maintaining this. If I have a lot of projects each with their own configuration, every time I modify the default configuration one I would have to modify all the per-project configurations to override that.
Perhaps I'm wrong, but I think that for some fields it's not really possible to cancel them by overriding them? For instance, let's say I have a compilation flag I want to add for every file in the default configuration, but not in per-project configurations. If in each of the per-project configurations I add a Remove: for that, won't it also remove it from instances where that flag is specified in compile_commands.json?

This all makes sense, but I am not sure if being able to set a custom location through a command line flag helps with this. Usually command line arguments used to spawn clangd are specified in a per-editor manner, which means you'll need to keep editing that argument everytime you want to switch configs. (vscode might have some per-workspace option for this but most of the vim plugins doesn't).
What about putting projects with "default" configuration under a different root and putting your clangd config there?
e.g. you can have a layout like:
| default-projects
- | .clangd
- | project1/
- | project2/
| other-projects
- | .clangd
- | project1/

This way other-projects/project1 won't make use of settings in default-projects/.clangd. You won't need to update your command line arguments everytime you switch projects.

@kadircet I am actually using neovim (with its built-in LSP), and it's really not that hard to change clangd command line arguments per project (I actually have to do it anyway, as I have a project which has its compile_commands.json in a different directory). There are even plugins (e.g. https://github.com/windwp/nvim-projectconfig) that allow me to do that without needing to add any file to the project directory itself.
While your solution could work, there is actually another issue I have which I forgot to mention before. Some of the projects I work on are on a read-only filesystem, and are mounted in such way that all the directories in the path leading to them are also read-only. I can't really add a .clangd anywhere in their path, but if I could actually rely on the default clangd configuration for them (or perhaps specify a different configuration file in the command line) that would work.

I've also added a use-case to #1336 (https://github.com/clangd/clangd/issues/1336#issuecomment-1305200154) which basically is that we'd like to check-in a default set of settings but allow the individual contributors to override (parts of) the config.

Another use-case: I have crossplatform setup of Neovim with clangd via lsp. I don't want to create config.yaml file in home directory on every computer where I work. I want to store config.yaml inside editor's config files.

Just in case we are interested in moving forward with this feature, a potential implementation is at https://github.com/llvm/llvm-project/compare/main...daiyousei-qz:llvm-project:add-workspace-config-option

Is there any progress for this issue? Having a command line parameter to specify the configuration file location would be a nice feature.

I think there can be both use cases.
We'd like to have a default configuration which is always used unless there is a local configuration in the project (I think this is the same use case mentioned in https://github.com/clangd/clangd/issues/845#issuecomment-904166063).
Perhaps this would better be called --default-config.

I tested this and what works is that the specified config file is used. However, I'd expect that when there is a .clangd file in the project, it should have the higher priority, which doesn't seem the case in the implementation.

@ddscharfe IIRC, some clangd person mentioned somewhere that this is expected behavior. They expect .clangd in the project folder to be bundled with the actual project and user .clangd is used to customize with your settings.

Considering the popularity of this, @sam-mccall do you think an option to override the user config file location an acceptable addition? That is, a "--user-config " to specify a clangd.yaml file. Different from my last proposal at #1336, this doesn't introduce any new concept.
For me, the use case is that I have one source folder shared by multiple projects who need different customization.

I tested this and what works is that the specified config file is used. However, I'd expect that when there is a .clangd file in the project, it should have the higher priority, which doesn't seem the case in the implementation.

@ddscharfe IIRC, some clangd person mentioned somewhere that this is expected behavior. They expect .clangd in the project folder to be bundled with the actual project and user .clangd is used to customize with your settings.

(That may have been me in this comment, though note that I was just speculating about the reason for the existing behaviour, not giving any sort of authoritative explanation.)

I created a PR for the behavior @ddscharfe mentioned, maybe anyone can review it https://reviews.llvm.org/D151315


soc: nxp: imxrt: fix PDRV field setting for drive strength

In the IOMUXC controller, the PDRV field uses 0b0 to set the pin drive to high, and 0b1 to set the pin to normal drive. Fix the pinctrl_soc.h definitions for the iMXRT11xx parts to use the correct setting for this register, based on the documentation for the pin control binding
Note that for PDRV type pins, this commit effectively switches their drive strength setting.

Relevant section from the iMXRT1170 reference manual:

I'm conflicted on whether to identify and change all PDRV type pins within NXP boards to use the opposite driver strength. If we do this, it will reduce the odds this PR breaks any samples (which will only be caught at runtime testing). However, we typically reuse pin settings from MCUX SDK when defining board pin control, so in all likelihood these pin settings are currently correct, and have been working by luck.

@danieldegrasse , how was this discovered. Was there a test that was failing?

I was trying to debug the SD test failures in #78077, and noticed that the pins were not being configured how I expected. However, that was not the root cause of the failure in #78077, so I am not aware of any tests were that were failing explicitly because of this issue.


Update README.md

Removed old and redundant documentation and added a reference to our documentation platform

Thanks @roald-teunissen


use /bin/bash over /bin/sh if it exists

/bin/bash returns more consistent kill code than /bin/sh, but /bin/bash it isn't guaranteed to exist on all systems.

-1 to using sync method here. maybe just make the string an option that can be passed in, that way the user can choose their tradeoffs?

Totally agree with @maxogden
child_process.exec supports a shell option:
I node core it is implemented like this:
  if (options && options.shell) {
    file = options.shell;
  }

I think execspawn should support that option too.

Sounds good
On Monday, January 4, 2016, Andreas Madsen<EMAIL_ADDRESS>wrote:

Totally agree with @maxogden https://github.com/maxogden
child_process.exec supports a shell option
https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback
:
I node core
https://github.com/nodejs/node/blob/master/lib/child_process.js#L96L97
it is implemented like this:
if (options && options.shell) {
file = options.shell;
}
I think execspawn should support that option too.
—
Reply to this email directly or view it on GitHub
https://github.com/AndreasMadsen/execspawn/pull/3#issuecomment-168776261
.

--
Karissa McKelvey
http://karissa.github.io/

what would execspawn give differently than child_process.spawn then?

Well the input API is child_process.exec and the output API is child_process.spawn so I don't see a problem there.

ok was just curious. going to close this and hopefully the kill bug will be fixed deeper in core


Uncaught Error: ENOENT: no such file or directory, open '/Users/ben/filename'

Open the advanced-new-file dialog (via keyboard shortcut or command palette).
Type in a file name and confirm
The new file is opened and the error is shown in the red dialogue box. Everything is working fine. The only (visible) issue is the error reporting tool.

Atom Version: 0.196.0
System: Mac OS X 10.10.3
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, open '/Users/ben/filename'
At events.js:141

Error: ENOENT: no such file or directory, open '/Users/ben/filename'
  at Error (native)

Commands
 18x -0:36.3.0 editor:move-to-end-of-word (atom-text-editor.editor.is-focused)
  2x -0:34.4.0 core:move-left (atom-text-editor.editor.is-focused)
     -0:32.1.0 core:save (atom-text-editor.editor.is-focused)
     -0:10.9.0 core:close (atom-text-editor.editor.is-focused)
     -0:09.3.0 tree-view:reveal-active-file (atom-text-editor.editor)
     -0:08.2.0 tree-view:move (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -0:07.6.0 core:copy (atom-text-editor.editor.mini)
     -0:06.7.0 core:select-all (atom-text-editor.editor.mini)
     -0:06.5.0 core:copy (atom-text-editor.editor.mini)
     -0:06.2.0 core:cancel (atom-text-editor.editor.mini)
     -0:05.5.0 application:new-file (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -0:04.5.0 core:close (atom-text-editor.editor.is-focused)
     -0:03.8.0 advanced-new-file:toggle (atom-text-editor.editor)
     -0:03.5.0 core:paste (atom-text-editor.editor.mini)
  4x -0:02.9.0 core:move-left (atom-text-editor.editor.mini)
     -0:01.4.0 core:confirm (atom-text-editor.editor.mini)

Config
{
  "core": {
    "ignoredNames": [
      ".git",
      ".svn",
      ".DS_Store"
    ],
    "disabledPackages": [
      "linter-jshint",
      "atom-handlebars",
      "autosave"
    ],
    "themes": [
      "atom-dark-ui",
      "twilight-syntax-spacegray-ui"
    ],
    "projectHome": "/Users/ben/projects",
    "audioBeep": false
  },
  "editor": {
    "fontFamily": "Inconsolata",
    "showIndentGuide": true,
    "autoIndentOnPaste": false,
    "invisibles": {},
    "fontSize": 26
  }
}

Installed Packages
# User
advanced-new-file, v0.4.1
editorconfig, v1.0.0
emmet, v2.3.8
file-icons, v1.5.4
filetype-color, v0.1.4
git-tab-status, v1.7.0
highlight-line, v0.10.2
javascript-snippets, v1.0.0
jsformat, v0.8.0
language-clojure, v0.14.0
language-docker, v1.1.3
language-groovy, v0.4.0
language-jade, v0.4.0
language-scala, v1.1.0
language-typescript, v0.15.0
linter, v0.12.1
linter-csslint, v0.0.11
linter-eslint, v0.5.3
project-manager, v1.15.5
script, v2.19.0
twilight-syntax-spacegray-ui, v0.4.0

# Dev
No dev packages

Do you have still this problem? I am not able to reproduce it.

The bug no longer occurs. I did not install any new plugins though. Only updates...


Local inference reports cycle, suspiciously

Compiler version
3.4.1
Minimized code

class Test {
  def local: Unit = {
    val timeout = 5.seconds
    implicit val askTimeout = Timeout(timeout)
  }
}

with supporting code via Akka
import java.util.concurrent.TimeUnit

import java.time.{Duration => JDuration}

import scala.concurrent.duration._

import language.implicitConversions

case class Timeout(duration: FiniteDuration) {

  /**
   * Construct a Timeout from the given time unit and factor.
   */
  def this(length: Long, unit: TimeUnit) = this(Duration(length, unit))
}

/**
 * A Timeout is a wrapper on top of Duration to be more specific about what the duration means.
 */
object Timeout {

  /**
   * A timeout with zero duration, will cause most requests to always timeout.
   */
  val zero: Timeout = new Timeout(Duration.Zero)

  /**
   * Construct a Timeout from the given time unit and factor.
   */
  def apply(length: Long, unit: TimeUnit): Timeout = new Timeout(length, unit)

  /**
   * Create a Timeout from java.time.Duration.
   */
  def create(duration: java.time.Duration): Timeout = {
    import JavaDurationConverters._
    new Timeout(duration.asScala)
  }

  implicit def durationToTimeout(duration: FiniteDuration): Timeout = new Timeout(duration)
}

object JavaDurationConverters {
  def asFiniteDuration(duration: JDuration): FiniteDuration = duration.asScala

  final implicit class JavaDurationOps(val self: JDuration) extends AnyVal {
    def asScala: FiniteDuration = Duration.fromNanos(self.toNanos)
  }

  final implicit class ScalaDurationOps(val self: Duration) extends AnyVal {
    def asJava: JDuration = JDuration.ofNanos(self.toNanos)
  }
}

Edit: the minimization is trivial
class C {
  import C.*
  def local = {
    val x = 42.z
    implicit val y = f(x)
  }
  def f(s: String): String = s
}
object C {
  implicit class Z(val v: Int) {
    def z: String = "z"
  }
}

Output
-- [E045] Cyclic Error: timeout.scala:60:38 ----------------------------------------------------------------------------
60 |    implicit val askTimeout = Timeout(timeout)
   |                                      ^
   |                                      Recursive value timeout needs type
   |
   |                                      The error occurred while trying to compute the signature of value timeout
   |                                        which required to compute the signature of value askTimeout
   |                                        which required to compute the signature of value timeout

Expectation
Although there are many implicits in the environment, local inference should just work, especially since the implicit value is defined after the first "site of inference".
Observed at https://github.com/akka/alpakka-kafka/pull/1727
Maybe it's just complaining that the implicit is in local scope while inferring the first definition. Edit: see minimal example.

Since we will remove old style implicits I don't think we should spend cycles tweaking them.

Agreed.
Seems like we will not be working on this, so it can be closed.

Thanks again for the explanation. I'm sure it's tiresome to field settled issues. For Scala 2, Seth added "questions are not bugs" to the template to make everyone think twice before posting.


Create hardcoded temp file inspection

Implements #54

Codecov Report

Merging #60 into master will decrease coverage by 0.12%.
The diff coverage is 56.25%.

@@             Coverage Diff              @@
##             master      #60      +/-   ##
============================================
- Coverage     61.91%   61.78%   -0.13%
- Complexity      191      195       +4
============================================
  Files            38       39       +1
  Lines           722      738      +16
  Branches        315      324       +9
============================================
+ Hits            447      456       +9
  Misses           67       67
- Partials        208      215       +7

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/security/Checks.kt
100% <100%> (ø)
27 <1> (+1)
:arrow_up:

...security/validators/HardcodedTempFileInspection.kt
53.33% <53.33%> (ø)
3 <3> (?)

Continue to review full report at Codecov.

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


context.adapter.getUserToken is not a function

In the following sample https://github.com/OfficeDev/Microsoft-Teams-Samples/blob/main/samples/app-sso/nodejs/server/bots/botActivityHandler.js it seems that the context.adapter should be of type BotFrameworkAdapter, however it appears to be of type CloudAdapter, which means getUserToken and getSignInLink in the function handleTeamsMessagingExtensionFetchTask throw an exception. Is there a version of these functions that works with CloudAdapter or a different way to authenticate within a message extension?

@smokranova - Thanks for reporting your issue.
We will check this at our end and will get back to you.

@smokranova - Please have look into this sample-https://github.com/OfficeDev/Microsoft-Teams-Samples/blob/main/samples/bot-conversation-sso-quickstart/js/index.js#L34
const adapter = new CloudAdapter(botFrameworkAuthentication);
server.post('/api/messages', async (req, res) => {
    // Route received a request to adapter for processing
    await adapter.process(req, res, (context) => bot.run(context));
});

Also refer the document for the Cloud Adapter class-https://learn.microsoft.com/en-us/dotnet/api/microsoft.bot.builder.integration.aspnet.core.cloudadapter?view=botbuilder-dotnet-stable

Hi, thanks for your answer. I have seen both of the things you linked while looking for an answer, however neither quite helps me with how to get the token or sign in link

@smokranova -Please have look into this sample-
https://github.com/OfficeDev/Microsoft-Teams-Samples/blob/f7779e42022176c3ae807e4e23acf0d0479c6c39/samples/app-complete-auth/csharp/AppCompleteAuth/Bots/ActivityBot.cs
Hope it is helpful.

 protected override async Task<MessagingExtensionActionResponse> OnTeamsMessagingExtensionFetchTaskAsync(ITurnContext<IInvokeActivity> turnContext, MessagingExtensionAction action, CancellationToken cancellationToken)
        {
            if ((!string.IsNullOrEmpty(action.State) && action.CommandId.ToLower() == "sso") || action.CommandId.ToLower() == "sso")
            {
                var state = action.State; // Check the state value
                var tokenResponse = await GetTokenResponse(turnContext, _botConnectionName, state, cancellationToken);

                if (tokenResponse == null || string.IsNullOrEmpty(tokenResponse.Token))
                {
                    // There is no token, so the user has not signed in yet.

                    // Retrieve the OAuth Sign in Link to use in the MessagingExtensionResult Suggested Actions
                    var signInLink = await GetSignInLinkAsync(turnContext, _botConnectionName, cancellationToken).ConfigureAwait(false);

                    return new MessagingExtensionActionResponse
                    {
                        ComposeExtension = new MessagingExtensionResult
                        {
                            Type = "auth",
                            SuggestedActions = new MessagingExtensionSuggestedAction
                            {
                                Actions = new List<CardAction>
                                {
                                    new CardAction
                                    {
                                        Type = ActionTypes.OpenUrl,
                                        Value = signInLink,
                                        Title = "Bot Service OAuth",
                                    },
                                },
                            },
                        },
                    };
                }

I looked at what you sent and tried it (in nodejs no csharp, but same bot). I still get stuck at the signinlink. I see the SignIn button that is part of the Bot Service OAuth card, however after clicking it I get "Something went wrong, please try again". The sign in url is in the form of "https://europe.token.botframework.com/api/oauth/signin?signin=" and when i open it in a browser it forwards to a page where i can copy some code and it says "Please enter this validation code into the chat window to complete the sign-in:". This does not seem like the desired behaviour of the login flow at all

@smokranova -Here is node sample-https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/f7779e42022176c3ae807e4e23acf0d0479c6c39/samples/app-complete-auth/nodejs
Also please have look into this document to add the authentication to teams bot-https://learn.microsoft.com/en-us/microsoftteams/platform/bots/how-to/authentication/add-authentication?tabs=dotnet%2Cdotnet-sample#prerequisites


Create a command to generate the Swagger.Json

Hello,
I was wondering if is possible to create a feature to generate and make available the json with swagger specs to be read by the swagger-ui.
I am user of the older swagger-node-express mw and I was using the function below to create my docs.
swagger.configureSwaggerPaths("", "/api-docs", "");
I did not find how to achieve it with these news changes, so if it is not possible, I will let this feature suggestion.
ps.: I don't know if this place is the correct repo or I should move this suggestion to "swagger-express-mw" repo.
Thanks,
Danilo

Once you have started project using "swagger project start", did you try /swagger?

I was just going to create a similar issue and ask what's the purpose of docEndpoints in the config as it doesn't seem to be working:
swagger:
  mapErrorsToJson: true
  docEndpoints:
    raw: /swagger

When I start up the app with debugging turned on, it prints out these log statements in the console:
$ DEBUG=swagger swagger project start
  swagger project.api: {"swaggerFile":"/Users/behrangsa/Projects/foo/api/swagger/swagger.yaml","name":"foo","main":"app.js","host":"localhost:10010","basePath":"/","localUrl":"http://localhost:10010/","port":"10010"} +0ms
Starting: /Users/behrangsa/Projects/foo/app.js...
  project started here: http://localhost:10010/
  project will restart on changes.
  to restart at any time, enter `rs`
  swagger read config file: /Users/behrangsa/Projects/foo/config/default.yaml +0ms
  swagger config from file: {"swagger":{"mapErrorsToJson":true,"docEndpoints":{"raw":"/swagger"}}} +1ms
  swagger loaded env vars: {"corsOptions":{}} +1ms
  swagger resolved config: {"swagger":{"corsOptions":{},"appRoot":"/Users/behrangsa/Projects/foo","mapErrorsToJson":true,"docEndpoints":{"raw":"/swagger"},"validateResponse":true,"controllersDirs":["/Users/behrangsa/Projects/foo/api/controllers"],"mockControllersDirs":["/Users/behrangsa/Projects/foo/api/mocks"],"configDir":"/Users/behrangsa/Projects/foo/config"}} +0ms
  swagger swagger security config: undefined +146ms
  swagger validator config: {"validateResponse":true} +0ms
  swagger swagger.config will be attached to requests +0ms
  swagger express compatibility middleware configured +1ms
  swagger swaggerTools router config: {"useStubs":false,"controllers":["/Users/behrangsa/Projects/foo/api/controllers"]} +0ms
  swagger cors config: {} +1ms
  swagger swagger raw doc path: /swagger +0ms
  swagger swagger doc filter: /^(?!x-.*)/ +0ms

So it reads the config, but going to either of /swagger or /swagger.json it prints out the following error message in the response:
Error: Route defined in Swagger specification (undefined) but there is no defined get operation.
   at send405 (/Users/behrangsa/Projects/foo/node_modules/swagger-express-mw/node_modules/swagger-node-runner/node_modules/swagger-tools/middleware/swagger-router.js:289:13)
   at swaggerRouter (/Users/behrangsa/Projects/foo/node_modules/swagger-express-mw/node_modules/swagger-node-runner/node_modules/swagger-tools/middleware/swagger-router.js:398:16)
   at Layer.handle [as handle_request] (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/layer.js:82:5)
   at trim_prefix (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/index.js:302:13)
   at /Users/behrangsa/Projects/foo/node_modules/express/lib/router/index.js:270:7
   at Function.proto.process_params (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/index.js:321:12)
   at next (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/index.js:261:10)
   at expressCompatibility (/Users/behrangsa/Projects/foo/node_modules/swagger-express-mw/node_modules/swagger-node-runner/lib/connect_middleware.js:247:3)
   at Layer.handle [as handle_request] (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/layer.js:82:5)
   at trim_prefix (/Users/behrangsa/Projects/foo/node_modules/express/lib/router/index.js:302:13)

There is a bug in the interaction between swagger-

Sorry about that. This should now be fixed. Please update or reinstall the deps in your project and try again. Let me know. Thanks!

@theganyo  Thanks. It is working now!

Fantastic!


CentOS 7.7: Xvnc session working OK, but Xorg session does not...

Hello,
Trying to get Xorg session working on my system, but having some difficulties...appreciate your help.
System: CentOS 7.7
XRDP: xorgxrdp-0.2.13-1.el7.x86_64, xrdp-0.9.13-2.el7.x86_64
xrdp.ini.txt
sesman.ini.txt
Issue:
Once authenticated on Xorg session, session starts but immediately closes without showing any content. Tested with various RDP clients (Windows, Mac). Xvnc session is working without any issues.
Logs:
xrdp.log.txt
xrdp-sesman.log.txt
xorgxrdp.15.log.txt

The GLX extension is not loading, and the GNOME desktop requires this. From your xorgxrdp log file:-
[142193.430] (EE) Failed to initialize GLX extension (Compatible NVIDIA X driver not found)

We can confirm the effects of this by using this procedure to see what the desktop is logging:-

Make a note of the current time (hh:mm:ss)
Try to create a session
After the session has exited, use sudo journalctl -S <hh:mm:ss> to dump out everything logged by the session.

You might see messages like the following :-
gnome-session-binary[]: WARNING: software acceleration check failed: Child process exited with code 1
gnome-session[]: gnome-session-binary[]: CRITICAL: We failed, but the fail whale is dead. Sorry....

These would indicate that GLX is inoperative.
It looks to me like you've got some bits and pieces from an old NVidia installation knocking round on your machine in /usr/lib64/xorg/modules/extensions. If that's the case you might also have a file in /etc/ld.so.conf.d/ which needs removing. I don't have any NVidia hardware myself to play with here, but hopefully that's helpful to you.

Many thanks for you reply... Attaching log as requested:
xrdp.log
Found the following lines which may be related to what you said:
Jun 29 16:40:07 pukeko gnome-session[23811]: gnome-session-binary[23811]: WARNING: software acceleration check failed: Child process exited
Jun 29 16:40:07 pukeko gnome-session-binary[23811]: WARNING: software acceleration check failed: Child process exited with code 1
Jun 29 16:40:08 pukeko gnome-session-f[24482]: Cannot open display:

Also ...
root@pukeko[04:51 pm]:/ ls -l /usr/lib64/xorg/modules/extensions/
total 502
-rwxr-xr-x 1 root root 297960 Jan 29  2019 libglx.so
-rwxr-xr-x 1 root root 583832 Aug  9  2019 libvnc.so
drwxr-xr-x 2 root root      4 Feb 18 12:05 nvidia
root@pukeko[04:51 pm]:/ ls -l /usr/lib64/xorg/modules/extensions/nvidia/
total 3713
lrwxrwxrwx 1 root root       17 Feb 18 12:05 libglx.so -> libglx.so.390.116
-rwxr-xr-x 1 root root 14013096 Feb 15 11:53 libglx.so.390.116

I'm using a NVidia Quadro K4200 graphics card with drivers installed directly from NVidia site (v390.116) (not nuveau).
The NVidia graphics card works fine when logged on the interactive session (with GNOME). By removing you mean removing the driver completely from the system ? Unfortunately that is not possible, as I need the drivers installed for 3rd party app to work (3D hardware acceleration).
Thanks

The NVidia drivers always work well with the main console as that's why they sell the cards. Sadly the NVidia drivers have a history of breaking other bits of the system as you'll find by perusing this forum and others.
I can offer you three ways forward which might work

Try installing the NVidia drivers from ELREPO. These are packaged for RHEL/CentOS and can work better than the generic NVIdia run script.
See if the Xvnc back end works. It may or may not. See for example here
Use a desktop for remote sessions which doesn't need 3D acceleration.

As an aside, I don't believe that xorgxrdp is yet able to make use of NVidia hardware for accelerating XRDP sessions.

Thanks for your suggestions. I installed drivers from ELREPO but that did not make any difference.
I ended up using your last suggestion, a Desktop which does not require 3D acceleration (MATE).


stream: Deduplicate template code for stream privacy types.

Follow-up of #9243.
Manual testing: GIF

Merged, thanks @shubhamdhama!  @eeshangarg FYI :)


Delay page rendering until page resources are fully loaded

how can I achieve "Delay page rendering until page resources are fully loaded",
I am new to php,
now i can export pdf and img successfully, you do a fantastic job.
but some dynamic chart like echarts or fushioncharts can't export completely
since the page is not delayed, any hints or help? Thank you

I found the answer from the website,
$delay = 5; // 5 seconds
$request = $client->getMessageFactory()->createCaptureRequest('http://jonnyw.me');
**$request->setDelay($delay);**


Provide CLI option for adding a file-level comment on the output file

🚀 Feature Proposal
I would love to have a code comment at the top of the generated TypeScript file to make clear to other developers that the file is generated, so that they don't make edits that get wiped away by the next time the file is generated.
Example
For example:
npx i18next-resources-for-ts toc /
  -i ./locales/en /
  -o ./i18n-resources.ts /
  --comment "This file is generated by i18next-resources-for-ts"

Output:
/**
 * This file is generated by i18next-resources-for-ts
 */
import common from './locales/en/common.json';

const resources = {
  common
} as const;

export default resources;

try v1.3.0 with:
npx i18next-resources-for-ts toc /
  -i ./locales/en /
  -o ./i18n-resources.ts /
  -c "This file is generated by i18next-resources-for-ts"

Perfect, thank you!


Slow page routing during development.

Dear Maintainers,
I recently began using this repo for a project of mine.
I tested the production builds at first.
The pages were 👌 blazingly fast with the Wordpress setup.
So I began to work on development mode and that's where I encountered very slow routing of around 4-6 seconds in average and even in incognito.

Filtered Network Tab showing time taken to get the json file for the page from wordpress.

Network Tab showing all the requests.
Here's what I did.

Installed Wordpress using Local App from WpEngine and setup a test site Foobar.
Installed WpGraphQL plugin.
Installed WP Lorem Ipsum plugin which helped me to generate dummy pages(10) and dummy blog posts(10).
I then forked this starter repo and then cloned it using git clone
Install packages using yarn
Set the graphql endpoint in env.local as WORDPRESS_GRAPHQL_ENDPOINT="http://foobar.local/graphql"
I then tested the production build first using yarn build && yarn start which worked like a charm.
Then I thought of playing around in the development. I ran yarn dev and that's where I found that all the pages load very slowly. They are even slower than the actual Wordpress site as well so I thought there must be something going on.

Here's my console logs.

& Here's my repo. https://github.com/Keshavdulal/next-wordpress-starter
From my guess/research so far it's something to do with Fast Refresh and HMR maybe but I am not sure so I thought of reaching you guys here. I am fairly new to WordPress & NextJS but seasoned in React.
Let me know If I am missing something during setup. Any insights are welcomed. 🙌

Additional Information.
I also tried the "minmal version of this repo".
It performed quite better with an average of around 1.3 seconds for the same Wordpress setup.
Here are some snapshots.

Filtered Network Tab showing time taken to get the json file for the page from wordpress.

Console logs for the development build

Hey - thanks for reaching out!
In production, a lot of the logic work that happens in getStaticProps is handled at build time. During development, this is refreshed during each Fast Refresh - so all of those lovely Successfully created messages are from quite a lot of heavy lifting when it comes to each refresh. Alongside that, the getInitialProps in _app.js is doing a lot of work.
I wonder if there is a "rapid development" option or similar that could minimise these features if you weren't working directly with them.

@doingandlearning Thanks for clarifying that bit regarding getStaticProps and getInitialProps.
I am digging into them right now.
And even I wonder if there's a way to minimize such delays during development as that might be a deal breaker to use the starter for long term projects.
Hoping to hear from @colbyfayock as well :)

yeah @doingandlearning @GuilleAngulo and I were chatting about it a bit in my discord https://spacejelly.dev/discord
so a little context, we're currently using getInitialProps in a custom _app.js to grab globally available data as there really isn't any type of global data layer mechanism to do that right now, beyond adding those requests on each page in something like getStaticProps, and i guess setting up a new context or pass those as props each time, which feels like the wrong way to handle it
part of the downside is this is typically discouraged, as it automatically disables some optimizations. the reason we're thinking it's okay here is because the app ends up being statically compiled anyways, so this is at no cost to the user
now i haven't run any tests to confirm this but i believe that to be the reason why it's slow page to page, because that ends up making all of those requests every time a new page loads, and there are a few in there to get all the relevant WP data
i think a big thing that currently isn't working properly (or might just not be set up) is a caching mechanism for those requests. each time the apollo client is being used, it seems to be creating a new instance, which im guessing means it's also not doing any sort of caching

that wasn't the intent but it didnt seem to work as expected
@GuilleAngulo said he's going to poke at it a bit and see if he can figure anything out. im not a graphql expert but trying to also do a little digging around to see if i can find anything that make ssense as to how to properly set up apollo to take advantage of caching
this article for instances sets up the client a little differently, could help: https://www.apollographql.com/blog/getting-started-with-apollo-client-in-next-js/
More info from their docs: https://www.apollographql.com/docs/react/caching/cache-configuration/
if we can't seem to figure something out i can reach out to people over there and see if they have any recs

hey @Keshavdulal just merged in some caching changes. would love to get your thoughts as to how you think that improves things. seems to be much snappier for me. i think the file generation improvements will help as well once they get in

Great! Thanks @colbyfayock.I will check it out and let you know.

Closing this, but feel free to reopen if it remains an issue.

Pardon me coming back at this after so long.
Here's an update incase anyone comes across this.
This is an issue with the nextjs framework itself.
Here's an ongoing discussion on Github Issues
This slow routing issue has nothing much to do with this or any starter for that matter. I tried making one from scratch but faced the same issue.
My Use Case/Context
I was making some network requests (for global data such as header/footer navigation items) in a getInitialProps method at _app.js so that those data can be reused across all the pages.
But not so fast.
According to Nextjs Docs as of writing this says: (May be this will change in future)
- Adding a custom getInitialProps in your App will disable Automatic Static Optimization in pages without Static Generation.
- App currently does not support Next.js Data Fetching methods like getStaticProps or getServerSideProps.

So, How I solved it?

I used file-based local data caching mechanism to speed up the slow-routing.
Reference Solution Article by Flavio Copes

There are other solutions discussed in the github discussion too but the above one worked in my case.
So hope this helps.


Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Problem Description
问题：当服务在服务器部署的时候（如ECS），存在多个服务使用同一个agent的情况，无法获得服务名称，导致无法判断服务的覆盖率。
需求：agent获得当前插桩的jar包名称（或者war包）
Proposed Solution
在agent端获得当前的服务名称
Alternatives Considered
No response
Additional Information
No response

Thanks for the suggestion, we will discuss it, and then solve it

After discussion, we believe that this feature is necessary. This feature will be developed in subsequent iterations.


iOS crashes if index in navigationState is updated too quickly

Current behavior
On iOS (Android works fine) there is a hard crash (no logs) that happens if index in navigationState is updated too quickly.
This happens when a user might press a button to switch to the next tab  too quickly (multiple times in a row):
Example code to reproduce (roughly):

import * as React from 'react';
import { View, useWindowDimensions, Button } from 'react-native';
import { TabView, SceneMap } from 'react-native-tab-view';

const FirstRoute = () => (
  <View style={{ flex: 1, backgroundColor: '#ff4081' }} />
);

const SecondRoute = () => (
  <View style={{ flex: 1, backgroundColor: '#673ab7' }} />
);

const renderScene = SceneMap({
  0: FirstRoute,
  1: SecondRoute,

  2: FirstRoute,
  3: SecondRoute,
  /* ... a few more entries */
});

export default function TabViewExample() {
  const layout = useWindowDimensions();

  const [index, setIndex] = React.useState(0);
  const [routes] = React.useState([
    { key: 0, title: 'First' },
    { key: 1, title: 'Second' },
    { key: 3, title: 'Third' },
    { key: 4, title: 'Fourth' },
      /* ... a few more entries */

  ]);
  const routesCount = routes?.length;
  return (<View>

    <Button
        onPress={() => setIndex(Math.max(index - 1, 0))}
        disabled={index === 0}
      >
        Backwards
      </Button>
      <Button
        onPress={() => setIndex(Math.min(index + 1, routesCount - 1)}
        disabled={index === routesCount - 1}
      >
          Forward
      </Button>

        <TabView
          navigationState={{ index, routes }}
          renderScene={renderScene}
          onIndexChange={setIndex}
          initialLayout={{ width: layout.width }}
        />
    </View>
  );
}

The problem seems to be in a library react-native-pager-view used inside PagerViewAdapter. This component calls:
   pagerRef.current?.setPage(index);
Which if switched to:
    pagerRef.current?.setPageWithoutAnimation(index);
With this change, it seems to work  fine but lacks the animation.
Note: There is no crash if navigating through the tabs via quickly swiping instead of  pressing the buttons to navigate through the tabs.
Expected behavior
For it to not crash.
Reproduction
Example Provided above
Platform

[ ] Android
[X] iOS
[ ] Web
[ ] Windows
[ ] MacOS

Environment

package
version

react-native-tab-view
^3.1.1

react-native-pager-view          5.4.9

react-native                   0.64.3

expo                                ~44.0.0

node

npm or yarn

getting same issue in ios for version 3.1.1


Weird arrow symbol on the left margin of iTerm2

My setup
iTerm2 Build 3.4.19
$ uname -a
Darwin as-macbook.local 22.5.0 Darwin Kernel Version 22.5.0: Mon Apr 24 20:53:44 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T8103 arm64
$ tmux -V
tmux 3.3a
$ cd ~/.tmux && git --no-pager log -n 1 --pretty=oneline
fd1bbb56148101f4b286ddafd98f2ac2dcd69cd8 (HEAD -> master, origin/master, origin/HEAD) fixed TMUX_PROGRAM introspection on FreeBSD, fixes #639

The problem
Please focus on the left margin of the screencast below. You should have observed some weird arrow symbols (>) appearing there, and they don't even disappear when creating new windows. What confuses me is that those weird arrow symbols only appear on the left margin of iTerm2, tmux should not be able to draw in that area (if I understand correctly). And most importantly, this problem only occurs when using gpakosz/.tmux.
https://github.com/gpakosz/.tmux/assets/5037285/deda13bd-9f7c-4ea7-9f13-59690f2f902f

Hello @aofei 👋

And most importantly, this problem only occurs when using gpakosz/.tmux.

I'm pretty sure you can reproduce this keeping the return key pressed in a new iTerm tab / window, without tmux.
It may be related to Zsh or Oh My Zsh.

I'm pretty sure you can reproduce this by keeping the return key pressed in a new iTerm tab / window, without tmux.

Yeah, I can reproduce that way. 🙈

It may be related to Zsh or Oh My Zsh.

Well, that explains why I rolled back my gpakosz/.tmux version and it still didn't fix the problem.
Sorry for opening this issue, I'll just close it. 🫡

But if you happen to know what's going on I'm interested!

Hi @gpakosz, I found something.

The blue arrow indicates a mark that you can navigate to with cmd-shift-up/down. You can hide them by turning off Prefs>Profiles>Terminal>Show mark indicators.
-- https://gitlab.com/gnachman/iterm2/-/issues/4744

Apparently, I never had Shell Integration for iTerm2 installed, so I didn't see these "Mark Indicators" before.

Likely the same for me.
I also found https://github.com/ohmyzsh/ohmyzsh/issues/5088.


example: Fatal: Failed precondition: must SETUP before PLAY

root@39e12bac9e7a:/home/dev/external/retina# cargo run --example client mp4 --url rtsp://<IP_ADDRESS>:10554/udp/av0_0 --username admin --password 123456 out.mp4
    Finished dev [unoptimized + debuginfo] target(s) in 0.27s
     Running `target/debug/examples/client mp4 --url 'rtsp://<IP_ADDRESS>:10554/udp/av0_0' --username admin --password 123456 out.mp4`
E20211128 20:55:52.260 main client] Fatal: Failed precondition: must SETUP before PLAY

The only reason I see that this would happen is if the example doesn't set up any video or audio streams:

It won't set up video if you pass --no-video or if the server doesn't offer any H.264 video streams (I haven't added H.265 support yet).
It won't set up audio if you pass --no-audio or if the server doesn't offer any AAC audio streams. (Only AAC can be placed into a .mp4 file, and this simple example doesn't do any transcoding.)

I just pushed a commit which will better describe what streams it's setting up or ignoring and will produce a more helpful error message when it doesn't set up any streams. I marked it as fixing this issue, but please let me know if it doesn't explain what's happening and I'll reopen.


update prod settings image

What are you changing in this pull request and why?
We've changed the UI for checking production in an env. This is because we're very soon releasing a 'staging' environment as well.

Could be better, but we're releasing the new staging type soon, so this will be fine temporarily imo:

Checklist

[x] Review the Content style guide so my content adheres to these guidelines.
[x] For docs versioning, review how to version a whole page and version a block of content.
[x] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Thank you @MitchellBarker !

@matthewshaver It's good as of the release today!

To be clear, I can't merge this, so @matthewshaver if you could 🙏


Unused co-located fragment definition(s) false-negatives?

I get a ton of "Unused co-located fragment definition(s)" warnings but I don't understand why...
For instance, I get this warning:

Even though I do use the fragment in this very same file as you can see further in the same file code:

Why is that?

Are you exporting that fragment? Basically if you follow the principles of fragment co-location an exported fragment imposes a data requirement on the importer so it gives the expectation that the importer uses this fragment in its query.
https://gql-tada.0no.co/guides/fragment-colocation
You can turn this setting off if you are not following these principles

Well, we use Remix (React Router), so this is a route component with a loader function which sends the GQL queries with imported fragments from child components and then renders each of those child component with the resulting data as props. Each child then reads its fragment from the resulting data (fragment masking), etc.
Something like:
// components/ProgrammeHeader.tsx
export const ProgrammeHeaderFragment = graphql(
  `
    fragment ProgrammeHeaderFragment on Programme {
      id
      name
    }
  `,
);

export type ProgrammeHeaderFragmentType = FragmentOf<
  typeof ProgrammeHeaderFragment
>;

export default function ProgrammeHeader({ data }: {  data: ProgrammeHeaderFragmentType; }) {
  const header = readFragment(ProgrammeHeaderFragment, data);

  return (

Whatever {header.name}

  );
}

import ProgrammeHeader, {
  ProgrammeHeaderFragment,
  ProgrammeHeaderSkeleton,
} from "./components/ProgrammeHeader"; // <= Warning: Unused co-located fragment definition(s) "ProgrammeHeaderFragment" in "./components/ProgrammeHeader"

// route.tsx
export async function loader({ request, context, params }: LoaderFunctionArgs) {
  invariant(params.slug, "Slug not defined");
  const slug = params.slug;

  const programmeHeaderPromise = context.apollo
    .query({
      query: graphql(
        `
          query GetProgramme($slug: String!) {
            programme(slug: $slug) {
              ...ProgrammeHeaderFragment
            }
          }
        `,
        [ProgrammeHeaderFragment],
      ),
      variables: {
        slug,
      },
    })
    .then(({ programme }) => {
      if (!programme) {
        throw notFound("Programme not found", request);
      }

      return programme;
    });

    return defer({
      programmeHeaderPromise,
    });
}

export default function ProgrammePage() {
  const { programmeHeaderPromise } = useLoaderData<typeof loader>();

  return (
    <>
      <Suspense fallback={<ProgrammeHeaderSkeleton />}>
          <Await resolve={programmeHeaderPromise}>
            {(data) => (
              <ProgrammeHeader data={data} />
            )}
          </Await>
        </Suspense>
      {/*...*/}
    </>
  );
}

I mean the previous code is a very common use case.
I agree that the "fragment" itself is not "really used" directly in the route component, but it is in the GraphQL query and I think the developer shouldn't be warned about it being unused here. Perfectly fine usage of fragments colocation I think.
Do you mean we should read the fragment before passing it to the child component and not in the child itself? I won't go into details but that creates other edge cases.
I don't want to disable the feature entirely because we still want to warn the developer when his fragment is really not used.

Oh if it's warning you for that second example then we probably have a missing case in our search because we should find
      query: graphql(
        `
          query GetProgramme($slug: String!) {
            programme(slug: $slug) {
              ...ProgrammeHeaderFragment
            }
          }
        `,
        [ProgrammeHeaderFragment],
      ),

Which clearly uses this fragment, the logic that needs debugging can be found here

I did a reproduction here https://stackblitz.com/edit/remix-run-remix-fh1a6h?file=app%2Froutes%2Fprogramme%2Froute.tsx
I don't know why Stackblitz doesn't show the warning in the editor directly, but if you run npx gql.tada check you can see the warning.

Basically, it starts working when you extract the graphql() call to the global scope, this is because we don't traverse nested call expressions at this point in time. We could make this work by adding it here however then we'd also have to find a way to avoid traversing deeply into other files/... which is a performance concern we could address by checking whether the callExpression contains nested callExpressions for instance.

@JoviDeCroock I just updated to the last version and it fixed a lot of warnings!
But it seems that there are still some issues when only "some" of the fragments of an import are used.
For instance, the following component exports two different fragments. Each of them can be imported and used from different files:

Only one them (fragment) is imported/used in this file:

The other fragment is imported and used in another file:

You can see the issue in the warnings messages.


Flows with same name in different namespaces cause configcheck error

Describe the bug:
When creating flows with the same name (in my case logs-to-loki) in different namespaces, the configcheck will fail
Expected behaviour:
As flows are namespaced resources, it should be allowed to use the same name in different namespaces
Steps to reproduce the bug:

Create a clusteroutput loki
Create two flows with the same name (logs-to-loki in this example) in two separate namespaces
Check configcheck logs

Additional context:
Output from configcheck for me:
~ » kl -n monitoring logs -f default-logging-fluentd-configcheck-11af17c7
fluentd -c /fluentd/etc/generated.conf --dry-run
2020-01-29 11:36:50 +0000 [info]: parsing config file is succeeded path="/fluentd/etc/generated.conf"
2020-01-29 11:36:50 +0000 [info]: starting fluentd-1.7.4 as dry run mode ruby="2.5.7"
2020-01-29 11:36:50 +0000 [error]: config error file="/fluentd/etc/generated.conf" error_class=Fluent::ConfigError error="Duplicated plugin id `_logs-to-loki_loki_loki`. Check whole configuration and fix it."

Environment details:

logging-operator version (e.g. 2.7.0):
Install method (e.g. helm or static manifests): static manifests

/kind bug

Brilliant, thanks so much!

Any workaround ideas while waiting for the release?

You could try and go with the master image tag as it's built directly from the master branch. The only caveat is that some deprecated fields have been removed, so I would recommend checking if you are running on k8s 1.14 or lower because as of k8s 1.15 validation should catch these type of issues automatically.

We can also create a 2.7.x branch and backport the fix there and tag a patch version of the image if that would help you better.

Thinking it through wasn't too bad to namespace the objects temporarily. Thanks for the quick response!


Please update readme

Update information more about tests and functionalities of your repo.
What are the important components?

Are you talking about how to use it in another program?

I am talking about , suppose someone wants to know about how this module works? and he is a complete beginner how would he understand this repository.
Just describe which file does what and how the file is built.

Okay, I added some info on how to use the module.

Actually, I'm having some merge issues with Git. It's here, though.

And now they're fixed. @GeekyShiva

Looks great now


BUG: Text doubling when using toolbar!

Write some code.
Place the cursor inside it.
Use the Toolbar to convert it to a code block
Behold: The code is converted, but the original plain text is doubled beneath it!
(Markdown shortcut does not do this — i.e., triple backtick plus language name plus space: ```js ...

This is a regression!
I haven't seen this behavior in awhile. Self: Track it down and fix it. May also be causing follow-on effects...

All set.
The real problem was that I hadn't properly updated the Toolbar component in the sandbox. It was using the old wrapNodes function. It should instead be doing something like this:
      editor.update(() => {
        const selection = $getSelection();
        const codeNode = $createLinedCodeNode();

        if ($isRangeSelection(selection)) {
          codeNode.insertInto(selection);
        }
      });

I ended up gutting the Toolbar component in CodeSandbox while trying to get my bearings. You'll now see a single button to convert editor text to the LinedCodeNode. There's no button to convert it back, or controls to use the test editor generally.
For purposes of demonstration, this is probably OK.
I'm closing this issue now with a personal reminder to return to the lingering markdown issue, which I'd forgotten.


VR-133 Make panel positioning work with HeadScale

What happened?

Found a little bug, I'm gonna reopen it once I fix that.

Ah, makes sense.
-----Original Message-----
From: "Kyle Emmerich"<EMAIL_ADDRESS>Sent: 5/4/2016 7:33 PM
To: "ROBLOX/Core-Scripts"<EMAIL_ADDRESS>Cc: "BinaryResolved"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [ROBLOX/Core-Scripts] VR-133 Make panel positioning work withHeadScale (#393)
Found a little bug, I'm gonna reopen it once I fix that.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

lol we all make mistakes. I would be worried if you didn't.


Feature/sentry integration

First step forward Sentry integration. Mostly backend stuff. Code cleanup will arrive as soon as possible.

Just got @EvilCodingMonkey's approval on Slack.


URL in description and README broken

The URL in the repository description and README are pointing to abl-dev-workshop's copy of the game instead of yours.
Please fix both, so they point to your copy of the game at https://abl-dev-workshop.github.io/github-games-cody-carlson-learning.

hey thanks for the report we are working on it!

issue resolved


extract css from SylesSheet to separate css file

Hello,
is there some way how to extract css from SylesSheet to separate css file?
When I have server side rendering, it looks weird when the component is missing styles for a while.

import { StyleSheet } from "react-primitives"

StyleSheet.renderToString() // return styles

Comes from react-native-web.

I tried using renderToString and doesn't exists that method, instead I used StyleSheet.getStyleSheets who returns and array of object with an ID and textContent (the styles)

The StyleSheet.getStyleSheets doesn't work when I tried it

@sergiodxa Thanks! StyleSheet.getStyleSheets works:
const html = renderToString(
        <StaticRouter location={location} context={{}}>
            <Root/>
        </StaticRouter>
    );
    const css = StyleSheet
        .getStyleSheets()
        .map(({id, textContent}) => `<style id=${id}>${textContent}</style>`)
        .join('');
    res.status(200).send(
        HTML
            .replace(/<!-- SSR:CSS -->/, css)
            .replace(/<!-- SSR:HTML -->/, html)


Add TLS verify option to destination

Let verify be on, off, or a path to a file. Essentially pass its value to Requests.

I did. 56488bdbcb2f73ec4a85196cbb1a6d6e86dff35e, except we don't allow for a path to a file. See the warning message that I added in the source on that commit.


Manifest version 2 is deprecated. Is there a new version?

Hi, Thank you for your work.
But Manifest version 2 is deprecated, and support will be removed in 2024. See https://developer.chrome.com/docs/extensions/develop/migrate/mv2-deprecation-timeline for details.
 "manifest_version": 2,

Google seems to be deprecating manifest_version = 2. Do you have a new version update?

I easily achieved screen sharing using getDisplayMedia(). Sorry for not reading your Readme. There are no problems now.


Labels are not displayed

I'm not sure if this project is abandoned, but I wanted to use my Twatch again this week, formatted it and added new views using the Twach SK Designer 1.2.
After clicking on "Download DynamicViews" and watiting for the restart, the views are there, but I can only see a the button and the switch.

However, the 2 labels doesn't appear on the watch. The space they should take is there though, but it's completely empty,

Could you share your json file? Labels are working, maybe there is some issue with settings of those labels :-) I see you're C# guy, am I correct?

Yup, I work mainly with C#, don't know how you knew that :O
Here's the json for this test screen:
{   "views": [     {       "name": "New view",       "layout": "column_mid",       "components": [         {           "text": "TWS",           "font": "montserrat14",           "type": "label",           "size": [             100,             40           ]         },         {           "text": "--",           "font": "montserrat28",           "binding": {             "path": "environment.wind.speedOverGround",             "multiply": 1.944,             "offset": 0.0,             "decimals": 0,             "period": 1000           },           "type": "label",           "size": [             100,             40           ]         },         {           "text": "test",           "font": "montserrat14",           "put": {             "path": "test.number",             "value": 1           },           "type": "button",           "size": [             100,             40           ]         },         {           "type": "switch",           "size": [             60,             40           ]         }       ],       "type": "normal",       "background": "white",       "size": [         0,         0       ]     }   ] }
I must add than when I first tested this (at that time there were only labels, no gauge, button or switch), it used to work perfectly fine.
Oh, and also I can see data is being received on the console:
(141462) WS: WEBSOCKET_EVENT_DATA
(141462) WS: Received opcode=1
(141462) WS: Got SK value update environment.wind.speedOverGround
(141472) WS: Got message Delta from websocket with len=207
(141472) WS: Total payload length=207, data_len=207, current payload offset=0

Yup, I work mainly with C#, don't know how you knew that :O

Yeah, because you have Xamarin Forms fork you are mobile developer! Non C# guy would fork Xamarin Forms!
Also checkout my new - hot project https://github.com/JohnySeven/SKipper it's KIP replacement I'm working on done in C# and Avalonia UI - runs everywhere!

Yeah, because you have Xamarin Forms fork you are mobile developer! Non C# guy wouldn't fork Xamarin Forms!
Also checkout my new - hot project https://github.com/JohnySeven/SKipper it's KIP replacement I'm working on done in C# and Avalonia UI - runs everywhere!

haha yes, that was from a veeery old project :P
I made a quick look to SKipper and liking it so far. I put it a star so I follow it's evolution.

I made a quick look to SKipper and liking it so far. I put it a star so I follow it's evolution.
Thanks!


Follow / Unfollow feature

Tasks need to completed

[ ] Create Follower entity
[ ] Create toggleFollow mutation.
[ ] Create getFollowSuggestions query.
[ ] Display follow suggestions in frontend
[ ] Profile page should include Follow/Unfollow options

backend files are not committed to this branch


Exit with non-zero status code on failure

Closes #228
This is how it works after changes:
Dec 24 13:49:11 vinco systemd[1]: Starting Electrum Personal Server...
Dec 24 13:49:11 vinco systemd[1]: Started Electrum Personal Server.
Dec 24 13:49:11 vinco eps[19385]: INFO:2020-12-24 13:49:11,344: Starting Electrum Personal Server <IP_ADDRESS>
Dec 24 13:49:11 vinco eps[19385]: INFO:2020-12-24 13:49:11,344: Logging to /home/eps/log.txt
Dec 24 13:49:11 vinco eps[19385]: WARNING:2020-12-24 13:49:11,344: Unable to find .cookie file, try setting `datadir` config
Dec 24 13:49:11 vinco systemd[1]: eps.service: Main process exited, code=exited, status=1/FAILURE
Dec 24 13:49:11 vinco systemd[1]: eps.service: Failed with result 'exit-code'.
Dec 24 13:49:41 vinco systemd[1]: eps.service: Scheduled restart job, restart counter is at 1.
Dec 24 13:49:41 vinco systemd[1]: Stopped Electrum Personal Server.
Dec 24 13:49:41 vinco systemd[1]: Starting Electrum Personal Server...
Dec 24 13:49:41 vinco systemd[1]: Started Electrum Personal Server.
Dec 24 13:49:41 vinco eps[19632]: INFO:2020-12-24 13:49:41,520: Starting Electrum Personal Server <IP_ADDRESS>
Dec 24 13:49:41 vinco eps[19632]: INFO:2020-12-24 13:49:41,521: Logging to /home/eps/log.txt
Dec 24 13:49:43 vinco eps[19632]: INFO:2020-12-24 13:49:43,733: Displaying first 3 addresses of each master public key:
Dec 24 13:49:43 vinco eps[19632]: INFO:2020-12-24 13:49:43,737:

systemd now restarts eps because of Restart=on-failure and RestartSec=30 and the fact that eps now returns non-zero status code on failure while bitcoind is starting up.

Thanks for the PR
Does this work as you'd expect when installing and running with pip?
I ask because of this https://github.com/chris-belcher/electrum-personal-server/blob/master/setup.py#L15 the entry point in those cases is the function main() rather than the if __name__ == "__main__" branch.

Does this work as you'd expect when installing and running with pip?

Hmm, I just applied these modifications manually in the already installed /opt/python3_venv_eps/lib/python3.9/site-packages/electrumpersonalserver/server/common.py file.
I need see where that main() is called...

$ cat /opt/python3_venv_eps/bin/electrum-personal-server
#!/opt/python3_venv_eps/bin/python
# EASY-INSTALL-ENTRY-SCRIPT: 'electrum-personal-server==0.2.0.dev0','console_scripts','electrum-personal-server'
__requires__ = 'electrum-personal-server==0.2.0.dev0'
import re
import sys
from pkg_resources import load_entry_point

if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw?|\.exe)?$', '', sys.argv[0])
    sys.exit(
        load_entry_point('electrum-personal-server==0.2.0.dev0', 'console_scripts', 'electrum-personal-server')()
    )

Looks like bin/electrum-personal-server uses sys.exit() already, so I guess that's why it works? I just accidentally made it work also for the pyinstaller?

If it already worked before, then why didn't your systemd setup from issue #228 work?

If it already worked before

It was not returning non-zero status code before? main() retruned nothing, I guess it's None in pythonb, and sys.exit() interpreted as 0?

What I wanted to say is by fiddling with __main__-stuff, I also made it to return status code in pyinstaller use case. I didn't knew that this code was only for pyinstaller.

Ah OK got it. Merging