fatal error: 'boost/preprocessor/control/expr_iif.hpp' file not found

Description
Followed instructions for using the Expo SDK with TV apps
Throws:
> Task :expo-modules-core:buildCMakeDebug[arm64-v8a]

... long stack trace ...

C/C++: /Users/justinkek/.gradle/caches/transforms-3/03658dcb6075d6cbc41f64dd0325babd/transformed/jetified-react-android-0.73.1-2-debug/prefab/modules/folly_runtime/include/folly/functional/Invoke.h:22:10: fatal error: 'boost/preprocessor/control/expr_iif.hpp' file not found
C/C++: #include <boost/preprocessor/control/expr_iif.hpp>
C/C++:          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
C/C++: 1 error generated.

> Task :expo-modules-core:buildCMakeDebug[arm64-v8a] FAILED

React Native Version
0.73.1-2
Output of npx react-native info
System:
OS: macOS 14.1.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 35.32 MB / 16.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 18.19.0
path: ~/.nvm/versions/node/v18.19.0/bin/node
Yarn:
version: 1.22.18
path: /usr/local/bin/yarn
npm:
version: 10.2.3
path: ~/.nvm/versions/node/v18.19.0/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /usr/local/bin/watchman
Managers:
CocoaPods:
version: 1.12.1
path: /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 23.2
- iOS 17.2
- macOS 14.2
- tvOS 17.2
- watchOS 10.2
Android SDK:
API Levels:
- "28"
- "29"
- "30"
- "31"
- "32"
- "33"
- "34"
Build Tools:
- 28.0.3
- 29.0.2
- 30.0.2
- 30.0.3
- 31.0.0
- 32.0.0
- 33.0.0
- 33.0.1
- 34.0.0
System Images:
- android-30 | Google TV Intel x86 Atom
- android-30 | Google APIs Intel x86 Atom
- android-30 | Google APIs Intel x86 Atom_64
- android-30 | Google Play Intel x86 Atom
- android-31 | Google APIs Intel x86 Atom_64
- android-31 | Google Play Intel x86 Atom_64
- android-33 | Google APIs Intel x86_64 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9514443
Xcode:
version: 15.1/15C65
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.9
path: /Users/justinkek/.sdkman/candidates/java/current/bin/javac
Ruby:
version: 2.7.4
path: /Users/justinkek/.rbenv/shims/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native: Not Found
react-native-macos: Not Found
react-native-tvos:
installed: 0.73.1-2
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: false
Steps to reproduce

npx create-expo-app -e with-tv
cd into the projectexport EXPO_TV=1
npx expo prebuild
yarn android

Snack, screenshot, or link to a repository
https://github.com/expo/examples/tree/master/with-tv

Confirming the same.  Is expo with tv support not yet stable enough for general use?

The same but with metro

Downgrading to 0.73.1-1 worked for me

Just discovered that setting:
newArchEnabled=true
when using:
"react-native"<EMAIL_ADDRESS>is also triggering the above error...
3 errors generated.
ninja: build stopped: subcommand failed.

C++ build system [build] failed while executing:
/Users/joshlongbrake/Library/Android/sdk/cmake/3.22.1/bin/ninja
-C
/Users/joshlongbrake/Projects/FrontRow/frontrow-creator-xptv/android/app/.cxx/Debug/554b1l4y/arm64-v8a
appmodules
react_codegen_ReactSlider
react_codegen_rnasyncstorage
react_codegen_rnscreens
react_codegen_rnsvg
react_codegen_safeareacontext

This might be an issue with specific packages and them not playing well with the newer architecture?

Confirmed, this issue is reproducible with 0.73.1-2, and not 0.73.1-1. Investigating. I suspect there was some problem with the build of the Android Maven artifacts. The problem does not occur when running yarn ios (Apple TV) instead of yarn android.
@joshuaben per the README in this repo, Expo SDK support is still considered experimental. Note that SDK 50 itself is not generally released yet anyway. Once the SDK 50 release happens, I'll do additional testing before promising production support 😄

Absolutely!  I appreciate you being ahead of the curve.  Happy to help test when we get there.

Confirmed, this was a build issue, likely to do with a problem with the boost artifact downloaded from JFrog during the build (there was an outage). Bumping to 0.73.1-3 today and rebuilding the Maven artifacts, I do not reproduce this issue locally. Publishing shortly. @juskek @joshuaben

Fixed in 0.73.1-3.


[DRAFT] MGDCTRS-2034: Remove dbapi.ConnectorDeployment.Annotations

=== DRAFT ===
Depends on a fix to gorm. See https://github.com/go-gorm/gorm/pull/6058
Depends on https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1637/ to upgrade to latest gorm libraries.
Description
See https://issues.redhat.com/browse/MGDCTRS-2034
dbapi.ConnectorDeployment.Annotations is not needed. See JIRA for more information.
Checklist (Definition of Done)

[x] All acceptance criteria specified in JIRA have been completed
[x] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
~[ ] Documentation added for the feature~
[x] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
~[ ] Required metrics/dashboards/alerts have been added (or PR created).~
~[ ] Required Standard Operating Procedure (SOP) is added.~
~[ ] JIRA has been created for changes required on the client side~

I can see why this is failing.. and it highlights why ConnectorAnnotations also lives on ConnectorDeployment.
When a Connector is deleted the ConnectorDeployment is left dangling. Retrieval of the soft-deleted Connector.Annotations is not that easy with gorm (https://github.com/go-gorm/gorm/issues/6054) and use of native SQL has other issues (https://github.com/go-gorm/gorm/issues/6052).
I am investigating a bit more.. but the simple solution is to keep the hack and close this PR.

@manstis just reading the error message without any more sophisticated investigation, I would have assumed that gorm still try to load the connectorDeployment -> Annotation relation somehow?

@manstis just reading the error message without any more sophisticated investigation, I would have assumed that gorm still try to load the ConnectorDeployment -> Annotation relation somehow?

Yes, keeping ConnectorDeployment.Annotations works around a bug in gorm but this PR tries to remove ConnectorDeployment.Annotations as it really lives on ConnectorDeployment.Connector.Annotations at the database level. i.e. there is no foreign key between connector_deployments->connector_annotations only between connector_deployments->connectors->connector_annotations. If a FK is added it causes other problems (I encountered when adding support for Smart Event Processors).
When gorm runs this for the "dangling" ConnectorDeployment's test it fails to return Connector instances as they have been soft-deleted. I tried adding Unscoped() to the query locally but it is not honoured by the Preload("Annotations") and hence zero Annotations are returned making the test still fail. I have reported it and provide a fix: https://github.com/go-gorm/gorm/pull/6058.
I'll keep this PR pending.. as when gorm is fixed we may be able to upgrade and complete this PR.

RHOC has been cancelled.


Proposal: add context.SetHTTPCookie

Hi,
I'm trying to add 'Set-Cookie' header in Response but need some value computed inside RPC.
Setting context value with a middleware doesn't seem to work since twirp doesn't allow rpc to modify context value (correct me if I'm wrong but only request hooks and middlewares can do that).
Wondering if anyone had a similar requirement and what would be the recommended solution.
Thank you!

I need to set multiple Set-Cookie headers. Having a twirp.AddHTTPResponseHeader which invokes http.Header.Add would be convenient.  If this sounds good I can help prepare a patch.
Right now I'm bypassing this by injecting ResponseWriter (again) in request's context since twirp's one is internal. Additionally I'm wondering if the idea of exposing ResponseWriter directly had been brought up before.
Thanks! Looking forward to the discussion.

Hi,
After some thought I believe adding AddHTTPResponseHeader alongside with SetHTTPResponseHeader feels too generic. Sent a PR #177 to add SetHTTPCookie to directly  address the need since Set-Cookie is the only common use case I can think of.
Let me know what you think. Thanks!

This seems unusual to me. I'm used to Twirp communicating through the request payload, rather than through headers. Can you say more about the use case of this?

This seems unusual to me. I'm used to Twirp communicating through the request payload, rather than through headers. Can you say more about the use case of this?

@spenczar sure!
The use cases for Set-Cookie headers, specifically for CloudFront signed cookies which requires 3 Set-Cookie headers. Currently twirp's SetHTTPResponseHeader API will overwrite if provides the header with the same key.

Thanks. What sort of RPC are you doing where Twirp is the right definition language, but you care about CloudFront's signed cookies?
My gut is that this is something that's hard to represent via Twirp, but maybe that difficulty is correct. Twirp is best when you control both client and server, and can fully describe the request/response payloads with protobuf. I'm a little concerned that cookies are a way to pass state that's pretty distant from the protobuf definition of the RPC. Are you able to say more about the details here - why a Twirp RPC is the best way to set a CloudFront signed cookie?
The risk here is that when we add more ways to pass state, it's easier for engineers to make mistakes and choose the wrong tool for the job.

@spenczar yeah I realized more details are helpful here.
The use case is around video playback. I control both client and server. The rpc is to fetch playback information given an asset. Response looks like
{
  "manifest_url": "http://cloud-front.com/foo/bar.m3u8"
}

An m3u8 is a manifest file contains relative links to actual media fragments which also resides on CDN.
To protect access to the objects on cdn, server generates a signature token can be either carried as query string  in the URL, or set as cookie. It would be a non-issue for the manifest itself but to use query string for all the fragments described in manifest, we'll have to edit it on the fly. I'm hoping to utilize set-cookie so that manifest and fragments can share the same token and all clients can share the same CDN object.
To your concern on choose the wrong tool for the job, I understand. This is basically why I was initially proposing having a AddHTTPResponseHeader along side with SetHTTPResponseHeader to make multiple headers with same key possible. Having a SetHTTPCookie API does sound a bit out of the place.

@Bankq Could the server's response type be this?
{
  "manifest_url": "http://cloud-front.com/foo/bar.m3u8",
  "signature_token": "xxxxxxxxxxx"
}

Then the client decides to set a Cookie header or use a query string when it requests the M3U8 or the fragments. It has the right data to do that. It also makes more sense than using Set-Cookie; unless your server is under the cloud-front.com domain, it's not permitted to Set-Cookie for cloud-front.com - cross-domain cookie setting would be a big security hole.
If you separate things in this way (where the client has to pull out the signature token and decide how to use it when requesting from cloudfront), the Twirp communication between client and server is pure Twirp; it doesn't have a dependency on how CloudFront implements auth.

@spenczar
Yes they are on the same domain. I used cloud-front.com as an example.
Puting token in response is what I meant "manifest will be a non-issue". The difficult part is re-using the token for all the underlying fragments that are linked in the manifest file. Client feeds the manifest url to the player which operates as a black box. I can perhaps intercept all the outgoing network calls and append the token there but as you can see cookies is perfect for this use case (which is also why cloudfront supports it I believe).
Is there a particular reason Twrip doesn't support response headers with same key?

Client feeds the manifest url to the player which operates as a black box.

Ok, now I've got it! In my mind, this means you don't really control the client, since it sounds like it's operating within a browser or something with a media engine.

can perhaps intercept all the outgoing network calls and append the token there but as you can see cookies is perfect for this use case (which is also why cloudfront supports it I believe).

This seems terrible, yes, but it also doesn't seem necessary. Why can't the client logic be to set its own cookies before passing the manifest into the player?
get {"manifest_url": "...", "signature_token": "..."} response from server
set cookies (eg via javascript's 'setCookie')
pass manifest to player

Is there a particular reason Twrip doesn't support response headers with same key?

Nothing technical. I just like to ask a lot of questions to really understand before adding to Twirp's API area. It has worked well for us in the past. The small size of Twirp's API is a big benefit.

@spenczar thanks!

Client feeds the manifest url to the player which operates as a black box.

Ok, now I've got it! In my mind, this means you don't really control the client, since it sounds like it's operating within a browser or something with a media engine.

Hah one could say that yes!

can perhaps intercept all the outgoing network calls and append the token there but as you can see cookies is perfect for this use case (which is also why cloudfront supports it I believe).

This seems terrible, yes, but it also doesn't seem necessary. Why can't the client logic be to set its own cookies before passing the manifest into the player?

Yeah that's one of the options the team and I discussed about. We are trying to simplify the client logic. If using Set-Cookie header, client works out of the box.

get {"manifest_url": "...", "signature_token": "..."} response from server
set cookies (eg via javascript's 'setCookie')
pass manifest to player

Is there a particular reason Twrip doesn't support response headers with same key?

Nothing technical. I just like to ask a lot of questions to really understand before adding to Twirp's API area. It has worked well for us in the past. The small size of Twirp's API is a big benefit.

I understand. Simplicity is the big reason we adopt twirp at the beginning.
AddHTTPResponseHeader  feels more "twirp-y" but definitely looks verbose side by side by SetHTTPResponseHeader. SetHTTPCookie addresses the most common use case of same key headers but feels not "twirp-y".
Right now I'm injecting ResponseWriter using a middleware to bypass this. Wondering if supporting same key response header is something Twirp plans to support.

I think that, all things considered, this isn't something we should add to Twirp, at least not yet. Your workaround through middleware is appropriate; it's not a common-enough problem to justify inclusion in Twirp's API.
If more people encounter this, please - chime in on this issue, and we can revisit it.
For posterity, here's how I picture the middleware working. It's not very onerous:
package twirpcookies

import (
	"context"
	"errors"
	"net/http"
)

// WithCookies configures an http.Handler (like any Twirp server) to enable
// setting cookies with the SetCookie function.
func WithCookies(h http.Handler) http.Handler {
	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		ctx := r.Context()
		ctx = context.WithValue(ctx, rwkey, w)
		r = r.WithContext(ctx)
		h.ServeHTTP(w, r)
	})
}

// SetCookie sets an http Cookie for a response being handled in the given
// context. It returns an error if and only if the context has not been
// configured through the WithCookies function.
func SetCookie(ctx context.Context, cookie *http.Cookie) error {
	w, ok := ctx.Value(rwkey).(http.ResponseWriter)
	if !ok {
		return errors.New("unable to get ResponseWriter from context, middleware might not be set up correctly")
	}
	http.SetCookie(w, cookie)
	return nil
}

type ctxkey string

const rwkey ctxkey = "responsewriter"

@spenczar This actually looks scarily similar to my code :)
Thanks for the discussion!


[ADD!]: Need to create About us Page

I shall see this

Okay, Change the status to Progress in the Project section located at right bar once you start working.

Sure!

It seems that I do not have access to modify changes in the Project section, @gaurovgiri.

Got it. You can start working, I will fix it later


Feature/refactor

Se hizo un refactor general de todo el modelo.

Codecov Report

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

@@            Coverage Diff             @@
##             develop      #10   +/-   ##
==========================================
  Coverage           ?   73.07%
  Complexity         ?       50
==========================================
  Files              ?       16
  Lines              ?      130
  Branches           ?        3
==========================================
  Hits               ?       95
  Misses             ?       35
  Partials           ?        0

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 c1062dc...d12021e. Read the comment docs.


fatal error: concurrent map iteration and map write

Describe the bug
The error reported by my saweedfs2.67 cluster is as follows.
The reason for the error is:  concurrent map iteration and map write

System Setup

OS version
centos 7
output of weed version
seaweedfs version 2.67
if using filer, show the content of filer.toml

How should I solve this problem? Thank you!

@liangyihan You should upgrade, because in the new version this part has been greatly optimized


added method to re-sync custom drop-downs if native element changes

I'm not sure if this would work for everyone, but I added a refreshSelect() method to jquery.customforms.js that gets attached to the change event of the native select to re-populate and re-flow the custom select in the event that the native hidden form element changes.
This is useful to me because I use Ajax methods to update the contents of selects in my forms (like the typical "change country, update states/provinces" scenario) and I need the custom drop-down to re-render if the contents of the native drop-down changes.
Note that you must actually call $el.trigger("change") in your code that re-populates the select because changing a select in code doesn't trigger the change event automatically (boooo!)

This actually needs more work, I found a few bugs in my testing.  I'll be back!


How to install?

Is there any other way to install this other than using Arduino ?

No. It's Arduino language, so it's not possible.


ScopeDoctorSetup report as a check failing initially, but fix is successful

With config:
apiVersion: scope.github.com/v1alpha
kind: ScopeDoctorSetup
metadata:
  name: bundle-update
spec:
  # order: 100 # default value
  cache:
    paths:
      - Gemfile*
      - .ruby-version
  setup:
    exec:
      - bash -c "bundle check >/dev/null || bundle install"
  description: Check for updates to bundled dependencies

I get:
 INFO Check initially failed, fix was successful, group: "bundle-update", name: "1"

This feels a little weird, because it's not a check, and it's saying it failed (what failed?), and that a fix (what fix?) was successful. Possibly related to https://github.com/ethankhall/scope/issues/60
Or maybe worth considering for https://github.com/ethankhall/scope/issues/49

Can you switch to https://ethankhall.github.io/scope/docs/models/ScopeDoctorGroup ? I've deprecated Setup and Check and unified it under Group, which makes the verbiage more consistent.

Confirmed switching to the group makes it more consistent 👍🏻


[ci]  Support Test Alias

Description
This PR address support for test alias, so that future tests that support multiple run modes may be correctly reported.

Merging this PR because it concerns the regression system.


not able to generate access token

I am not able to generate an access token please help in SASas online
I tried your config code but it is not able to generate access token

Please always share some snapshot or output when you open issues. I was working on this last week and it was working fine.
The server is generally down at night, I am assuming that you tried it after 1 am and before 5 am late night.
Check if issue is reproducible due market time and close this issue if it is not reproducible


Don't crash when code is not a string.

React caused the crash as code was an object and it got all confused about what to do.
Extract the code block from the object and print as string, as it would if code were a string.

@HookyQR Thanks for the fix 🎉  I fixed the eslint errors and merged it in #18, so should be fixed in the latest version of the plugin :blush:


Hover text not rendering

Describe the bug
What went wrong? What did you expect to happen?
Apparently since at least yesterday and perhaps Friday: If a remark in a template is text and not a URL the data editor form no longer displays hover text. The graphic of the "i" in a circle is rendering fine, differently from the graphic for an external link, only no further information is available. This is while using Chrome/Windows.

What is the impact on your work? (can't work / work stoppage; workaround exists; minor issue) Moderate importance: most important to have this text in place as useful background information when we train staff to input metadata
What environment are you in? (stage or production) Stage
What OS and browser are you using? (Mac/Windows; Safari, Firefox, etc.) Chrome/Windows
If you're cataloging, what Resource Template were you using (name or ID) UCSD:RT:BF2:RightsModuleTest:Work:CcLicenseModel

Please include a screenshot
(Mac: shift-command-4; Windows: ; then drag the image file into the issue)

another example can be found in DEV, resource template ID Sinopia:RT:Demo:SampleResourceFuller, fields called Example 20 and Example 22

Dupe of #1520.

Thanks for reporting @jsoenyun. Fix forthcoming.


Choose two branches to see what’s changed

Pull Request Test Coverage Report for Build 658294718

36 of 36 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.602%

Totals

Change from base Build 645909992:
0.2%

Covered Lines:
2050

Relevant Lines:
2117

💛  - Coveralls


Update dev branch due to master branch hot-fix of onnx example error

ready for merge after the github actions checks are done


feat(payments): PAYPAL-1713 braintree ach form

What?
Add form for braintree ach
Why?
Form implementation for braintree ACH strategy

Depends on
https://github.com/bigcommerce/checkout-sdk-js/pull/1891
https://github.com/bigcommerce/bigpay-client-js/pull/175
Testing / Proof
...
@bigcommerce/checkout

Is there a reason why we didn't create a package for the new payment method?
A new payment method should ideally follow the monorepo architecture now.

Is there a reason why we didn't create a package for the new payment method? A new payment method should ideally follow the monorepo architecture now.

@bc-peng I'm currently moving the new method to a separate package. Thanks!


Fix#37

Maybe just return XML without using parse_template+php_execute+cache.
Sitemap not high loaded for caching and information must be fresh as possible.

[ ] Fix blog rss

Not valid batflat feed

Fix #37
Ready to merge.


Update README.md

added new site
Description

Added a new site under When you get stuck

Checklist

[x] My change follows the Contributing Guidelines
[x] I have added only one new link to the list.
[ ] I have sorted the link alphabetically under the related section.

added a new sitelink


GUI apps does not respect windows display zoom setting

Environment
Windows build number: Microsoft Windows [Version 10.0.22000.100]
Your Distribution version: Ubuntu 20.04
Your WSLg version: 1.0.24

Steps to reproduce
Set a screen to 150% zoom in Windows display settings.
Expected behavior
The GUI apps should respect the zoom settings on each host display when moving across them and zoom in/out accordingly.
Actual behavior
The GUI apps stay at 100% zoom regardless of the Windows display settings.

@rabbit-aaron, thanks for reporting the issue. Currently WSLg doesn't enable fractional scaling, like 150%, this is reported at https://github.com/microsoft/wslg/issues/23. thanks!


Adds escaping for underscores in paths and param names

This might be a little naive.  I can implement with something a little more robust than regex if need be but I'm thinking this might be good enough for these particular fields.

Looks good!
Thank you, sir!

New version is available here.
P.S. added you as an author, if you don't mind ;)

Thanks!  I'll be sure to respect the workflow outlined in the Readme.


Out of memory when checking format

It seems that the Fyne theme directory may be a bit large but I don't have the error when I run locally...
An error occurred while running this test (AddError: could not parse "_repos/src/fyne.io/fyne/theme/fatal error: runtime: out of memory:1::warning: file is not gofmted with -s (gofmt)" - strconv.Atoi: parsing " runtime": invalid syntax)
https://goreportcard.com/report/fyne.io/fyne
Is there anything I can do to help resolve this? It took our score from A down to C! :(

@andydotxyz yeah, looks like our server is reaching its capacity :) I re-ran the report and it seems to be back to normal.

Thanks :)

It happened again :( https://goreportcard.com/report/fyne.io/fyne and https://goreportcard.com/report/fyne.io/desktop

This has happened again.
Is there something that I can do to help this stop happening?

@andydotxyz There's nothing you can do on the repo itself as far as I know. It's basically because we run this app on a single small instance and we've had to push it to its limit.
A couple of weeks ago I added an in-memory cache for the badge URL lookups. But that cache only gets bigger unless we restart the app (which I just did, and then re-ran your report which now seems fixed). I might try adding timeouts to the cache entries and have a poller running in the background which purges entries > a day old or something like that.
I was recently accepted into the GitHub Sponsors program, so I might possibly be able to take donations through that on behalf of Go Report Card and upgrade the server with the money. In the meantime we do have a Patreon as well: https://www.patreon.com/goreportcard
With more servers we'd probably split it out into 2 "worker" nodes and then a memcache/redis/etc. server for the caching. It would take some time and effort to re-architect everything. Unfortunately it hasn't been feasible since we don't make enough money at the moment.

It's happened again :-/

I'm getting the same error when I include large (>50MB) files (which I have because I haven't found a better way to include data files). Could you just check the file size first and ignore anything too big?


Consider using mime magic to detect file type, instead of only excluding by file extension

SSIA

So we actually only look at file extensions to exclude certain files. It's up to each backend to decide how it wants to scan things. At the moment we exclude certain files like .bmp. So if someone hides GPL source code in such a file, we will miss it. I think that's a fair trade off for now, but maybe an improvement would be to just skip certain binary files in general. (Ones that we don't decompress for example).
This is a good issue to track the discussion and feature, thanks!

Let's explore our options. I would like to understand how the mechanism works.

FYI: This issue will depend on https://github.com/awslabs/yesiscan/issues/46 getting fixed first.

long run for MVP, moving to backlog


Figure out why field descriptions aren't rendering

fields like serviceName have a description, but the output doesn't show them
ex. http://zipkin.io/zipkin-api/#/paths/%252Ftraces doesn't render the description for serviceName

updated to latest sway and got a warning.. maybe the schema is different than what we are using
$ npm test
><EMAIL_ADDRESS>test /Users/acole/oss/zipkin-api
> node validate.js

Warnings:
[{"code":"EXTRA_REFERENCE_PROPERTIES","message":"Extra JSON Reference properties will be ignored: description","path":["paths","/trace/{traceId}","get","responses","200","schema"]}]
Validation passed

That means a $ref has a sibling somewhere. All of $ref siblings will be overwritten with the result of the pointer by resolver.

Thanks for the tip. maybe an upgrade fixes the field description thing.. we'll see https://github.com/openzipkin/zipkin-api/pull/12

Fixed by #12


duplicate file

What is the problem you are having with rclone?
I have two file with the same file path on google drive, so must to rename one of them to sync to local.
I know there is a command 'rclone dedupe', but what I need is renaming to the format I need, and decide which file should be renamed. In this case, I try to rename using 'moveto', but command 'moveto' is path base, it doesn't know which duplicate file should be renamed.
What is your rclone version (output from rclone version)
1.45
Which OS you are using and how many bits (eg Windows 7, 64 bit)
linux 64 bit
Which cloud storage system are you using? (eg Google Drive)
Google Drive

you could use --backup-dir to have it move the file being replaced elsewhere but not quite sure if this will fit your use case.

The solution to this is to use rclone moveto to rename one of the files.  You can look then to see which one got the new name, and then use rclone moveto again to rename them to the correct names.
So
rclone moveto drive:dir/duplicated-file drive:dir/temp-file-name
rclone lsl --max-depth 1 drive:dir
# work out which file is which
rclone moveto drive:dir/temp-file-name drive:dir/new-name-1
rclone moveto drive:dir/duplicated-file drive:dir/new-name-2
```

Thanks for your reply.
It solve my problem.
I have another question about account, if the account has been revoked, why the exit code is 1 not 7?

I have another question about account, if the account has been revoked, why the exit code is 1 not 7?

What was the error message?  Can you post it please?  And what happened to your account?

I revoke the google drive account and try to do any operation. I think it will return exit code 7 but it return 1.
ERROR : : error listing: couldn't list directory: googleapi: Error 401: Invalid Credentials, authError
Failed to lsl: couldn't list directory: googleapi: Error 401: Invalid Credentials, authError


Restore Halogen recipes

Fixes #203
Restoring some broken Halogen recipes. The DriverWebSocketsHalogenHooks is adapted from the work in this PR https://github.com/purescript-halogen/purescript-halogen/pull/804, consolidated to a single component to be TryPureScript-friendly, and using the -hooks version of the component from the original recipe. Aside from that, the other recipes were fixed by just following the guidance of the changelog https://github.com/purescript-halogen/purescript-halogen/blob/master/docs/changelog/v6.md.
Also made an unrelated change to fix the naming of a function in one of the DogImages recipes (randomCatGif → randomDogImage) 42d3448

Thanks!


add note about placement of usegoogleauth

Fixes #2345

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/security/authentication/social/google-logins.md
:white_check_mark:Succeeded

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/security/authentication/social/google-logins.md
:white_check_mark:Succeeded

aspnetcore/security/authentication/social/index.md
:white_check_mark:Succeeded

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.


Refacoring: Improve XcresultPathType enum and clean the code

Messages

:book:
Executed 16 tests, with 0 failures (0 unexpected) in 0.443 (0.496) seconds

:book:
Project coverage: 58.87%

DangerSwiftCoverage.framework: Coverage: 74.65

File
Coverage

DangerSwiftCoverage.swift
68.42%
✅

  Generated by :no_entry_sign: Danger Swift against cbf2e14c29ec325ebda5bf8acf2665ef7d5d32d6

Messages

:book:
Project coverage: 58.87%

DangerSwiftCoverage.framework: Coverage: 74.65

File
Coverage

DangerSwiftCoverage.swift
68.42%
✅

  Generated by :no_entry_sign: Danger Swift against cbf2e14c29ec325ebda5bf8acf2665ef7d5d32d6

File
Coverage

Sources/DangerSwiftCoverage/DangerSwiftCoverage.swift
68.0%
✅

Tests/DangerSwiftCoverageTests/CoverageTests.swift
98.0%
✅

  Generated by :no_entry_sign: Danger Swift against cbf2e14c29ec325ebda5bf8acf2665ef7d5d32d6


BUG: typeof(item) has no type

I try to write unwrap method but get error
  def unwrap(array)
    result = [] of typeof(array) | typeof(array.first)

    array.each do |item|
      if item.is_a?(Array)
        result = ([] of typeof(item) | typeof(item.first)) + result
        result.concat(item)
      else
        result << item
      end
    end

    result
  end

BUG: typeof(item) has no type

https://play.crystal-lang.org/#/r/3p6i

Simplified version:
foo = 42
if foo.is_a?(String)
  [] of typeof(foo) | Int32
end

I think this might be the same problem as #5717.

This, #7441, and #7461 all seem to be hitting the same piece of code that raises that exception HERE.  So all the issues seem to be related in that something is causing @type and @freeze_type to not exist.  However, I don't know enough to take a guess as to why :/

This and #7441 are similar and I already have a fix for them. #7461 is different, I need to reduce it more (that means it should be able to be reproducible if you pass --prelude=empty). I will try to do it later if I have time.

Actually this and #7441 are also different :-P


Make SFM_Module::deleteUser functional

The current implementation of  SFM_Module::deleteUser() is not working at all.
According to the SFM-V1.7 datasheet, when deleting a user, P1 should be the 8 higher bits of the UID, P2 should be the 8 lower bits:

The current code doesn't do that. It only bit shifts, but that's not the same as retrieving the higher/lower byte.
This PR fixes the deleteUser() implementation to calculate the higher and lower byte properly. Of course, I tested it, and deleteUser() works flawlessly with this change.
I also added a newline at the end of the file, as that's both required by C/C++ standards and recommended by Git.

Whoops, I wanted to make a pull request to the main repo...


Unhandled exception in script

Error occurred while launch
Traceback (most recent call last):
  File "main.py", line 42, in <module>
  File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
  File "buzz\gui.py", line 14, in <module>
ImportError: DLL load failed while importing QtGui: The specified procedure could not be found.

Error GUI

OS info

Error occurred while launch
Traceback (most recent call last):
  File "main.py", line 42, in <module>
  File "PyInstaller\loader\pyimod02_importers.py", line 499, in exec_module
  File "buzz\gui.py", line 14, in <module>
ImportError: DLL load failed while importing QtGui: The specified procedure could not be found.

Error GUI

OS info

Sorry fella...I think this is why it didn't work for ya.


upgrade commands of zou:cli.py because of the upgrade of click

Click dependancy in Zou have been upgraded, it implies a change with underscores, they are replaced by dashs

Great thank you!


Include source-profile in shell & removed util-path from hyperconverged & feature

Signed-off-by: Shashank Ranjan<EMAIL_ADDRESS>What this PR does / why we need it:

Adds source in kubectl command.
Removes utils-path from vars file as it is present in global vars.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

There are many issues with my PR, so i am going to close it.


Update eslint-config dependencies

Update eslint-config dependencies.

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

I missed this one when I worked on https://github.com/open-wc/open-wc/pull/710. I tried updating to eslint v6, but that was giving issues for me. Does it work for you?

Updated! 🚀

Of course the day after I try it, air bnb base comes out with eslint v6 support 👍

@LarsDenBakker I'm not sure why the tests fail after this PR.

I think you regenerated the lockfile, which pulled in the latest versions of all dependencies. This changes the integration test snapshots, because they contain file hashes. It's a bit cumbersome, but it's a great way to ensure stability of the tools.
I've merged your changes with https://github.com/open-wc/open-wc/pull/710 and regenerated the snapshots in https://github.com/open-wc/open-wc/pull/733 so I'll close this one.
Thanks a lot of the help, and sorry for the bumpy ride :)

Thanks @LarsDenBakker! 🙂


HTML5 instead of !

hello, i have an issue with starting my html file, i write my normal title ______.html and the icon shows well, i think i might have done somwthing, becuase when i type ! + tab at the start of a project, it does not work, instead it only works with html5+tab and it is really uncomfortable, and it does not include meta tags

/gifPlease

/duplicate of https://github.com/microsoft/vscode/issues/154375

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


tx upgrade chain does not respect the format of other tx commands

Summary
Every tx command use dst-chain and src-chain. It should also be the case of tx upgrade-chain:
Instead of :
tx upgrade-chain [OPTIONS] --reference-chain <REFERENCE_CHAIN_ID> --host-chain <HOST_CHAIN_ID> --host-client <HOST_CLIENT_ID> --amount <AMOUNT> --height-offset <HEIGHT_OFFSET>
The command should be :
tx upgrade-chain [OPTIONS] --dst-chain <DST_CHAIN_ID> --src-chain <SRC_CHAIN_ID> --src-client <SRC_CLIENT_ID> --amount <AMOUNT> --height-offset <HEIGHT_OFFSET>

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

We chose these names on purpose because the notion of source and destination does not really apply here. Rather we use host and reference to distinguish between the client on the chain being upgraded and the one it points too.


Retrain a model with Detectron2 using a medium-sized PIP dataset

Validation predictions are visualized here https://owncloud.ut.ee/owncloud/index.php/s/WotTJwNzoebFQyA
Training was fast, ~2 min. The result is good given that the validation dataset is actually not very big. So, it makes sense now to expand both the training and testing datasets.


Separate requiredPermissions For Servers And DMs

Describe your proposal
I would like to see if it is possible to make an klasa in built method of having separate requiredPermissions for each command. Currently, setting a requiredPermission can and will break that command on all DM just because you required a perm like manage messages.
As an example, I added a cool upgrade enhancement to the help command on my bot. However, this broke the entire help command on DM.
It would be really cool to have separate permission settings for DMs because just enhancing a bot's command should not disable it.
Essentially, make requiredPermission be unobtrusive to DMs similar to unobtrusive javascript coding on web dev. If someone(server) has the permission they have a better experience, else (dm) they use it without the improved version.
Usecases for your proposal
Setting any command to require a perm will break it in DM, depending on the perm you require. A command should not be forcibly prohibited by default.
Expected and actual behaviour
Even if the bot does not have Manage Messages in a DM the command should still work.
Actual Behavior:

Further details
To be honest, this is a simple workaround on the missingBotInhibitor but I still think it could be a nice enhancement to klasa to have these separate by default.
<3

If we don't inhibit the command, discord api will reject your request. You can't have Manage Messages in DMs under any circumstance.


mount USB read-only

Read-write mount of USB raises problems.  This is especially bad for FAT. I guess that the problem's masked by background checks of Windows during mount. But that doesn't fully eliminate it.
Can I suggest using RO as much as possible. Game start is a good time to switch RW->RO AFAIK but you know better, of course.

I suppose (s)he is talking about IGR (at least the reset-function), where the USB-Stuff is not de-initialized (before reset), even tho' it does not create/cause problems (most of the time)!
It MIGHT cause problems, how more often someone IGR's out of a game however!
It would be 'neat' to 'unmount' the stuff before... However, that would need additional code running in-game and obviously that is not preferred/preferable...

Yes, but there's no unmounting process because we don't do writing. There's no write cache to flush either (we don't implement one, even for VMC).
Hence I think this issue shouldn't be considered an issue and can be closed...

I suppose you are correct for USB, where it should not cause any issues...
What about SMB? Does it 'close' the handles on IGR (even if that is off topic here)?

No, which is why one cannot use the same VMC image in consecutive boots, as the file was previously locked. This also happens, even if you do not use IGR.
It just so happens that IGR also has no code for letting the device drivers close open files. Or if it does, it doesn't work...

Well, that's definitely an 'issue'... :-/ (from a user-perspective and convenience-'point of view')

The original post is not a concern since we don't do cached writes for USB devices, but the VMC & disc image files will be closed for SMB once this pull request is merged: https://github.com/ifcaro/Open-PS2-Loader/pull/144
As for the HDD unit, further transfers will be blocked, so that the hardware will not be left in an indeterminate state.

@ElPatas1
This appears to be solved, since #144 has been merged thanks to @sp193
Are we able to close this?

Closed.
Best regards.


Lazy loading error with Ionic 3

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x ] 3.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[x ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I added the lazy loading using the guide but I keep on getting :

Runtime Error
Cannot read property 'links' of null

Uncaught TypeError: Cannot read property 'links' of null
at setupPreloadingImplementation (VM3387 main.js:56270)
at VM3387 main.js:56302
at t.invoke (VM3370 polyfills.js:3)
at n.run (VM3370 polyfills.js:3)
at NgZone.runOutsideAngular (VM3387 main.js:4315)
at VM3387 main.js:56301
setupPreloadingImplementation @ VM3387 main.js:56270
(anonymous) @ VM3387 main.js:56302
t.invoke @ VM3370 polyfills.js:3
n.run @ VM3370 polyfills.js:3
NgZone.runOutsideAngular @ VM3387 main.js:4315
(anonymous) @ VM3387 main.js:56301

ERROR Error: Uncaught (in promise): invalid link: HomePage
at g (VM3370 polyfills.js:3)
at l (VM3370 polyfills.js:3)
at VM3370 polyfills.js:3
at t.invokeTask (VM3370 polyfills.js:3)
at Object.onInvokeTask (VM3387 main.js:4415)
at t.invokeTask (VM3370 polyfills.js:3)
at n.runTask (VM3370 polyfills.js:3)
at a (VM3370 polyfills.js:3)
defaultErrorLogger @ VM3387 main.js:1364
ErrorHandler.handleError @ VM3387 main.js:1424
IonicErrorHandler.handleError @ VM3387 main.js:112467
next @ VM3387 main.js:5053
schedulerFn @ VM3387 main.js:4127
SafeSubscriber.__tryOrUnsub @ VM3387 main.js:39215
SafeSubscriber.next @ VM3387 main.js:39164
Subscriber._next @ VM3387 main.js:39106
Subscriber.next @ VM3387 main.js:39070
Subject.next @ VM3387 main.js:56368
EventEmitter.emit @ VM3387 main.js:4113
NgZone.triggerError @ VM3387 main.js:4484
onHandleError @ VM3387 main.js:4445
t.handleError @ VM3370 polyfills.js:3
n.runGuarded @ VM3370 polyfills.js:3
r @ VM3370 polyfills.js:3
a @ VM3370 polyfills.js:3
VM3387 main.js:45009 Native: tried calling StatusBar.styleDefault, but Cordova is not available. Make sure to include cordova.js or run in a device/simulator
cordovaWarn @ VM3387 main.js:45009
checkAvailability @ VM3387 main.js:56592
callCordovaPlugin @ VM3387 main.js:56680
(anonymous) @ VM3387 main.js:56799
value @ VM3387 main.js:104231
(anonymous) @ VM3387 main.js:104398
t.invoke @ VM3370 polyfills.js:3
onInvoke @ VM3387 main.js:4424
t.invoke @ VM3370 polyfills.js:3
n.run @ VM3370 polyfills.js:3
(anonymous) @ VM3370 polyfills.js:3
t.invokeTask @ VM3370 polyfills.js:3
onInvokeTask @ VM3387 main.js:4415
t.invokeTask @ VM3370 polyfills.js:3
n.runTask @ VM3370 polyfills.js:3
a @ VM3370 polyfills.js:3
invoke @ VM3370 polyfills.js:3

Expected behavior:
The application should open and load the main page.
Steps to reproduce:
https://docs.google.com/document/d/1vGokwMXPQItZmTHZQbTO4qwj_SQymFhRS_nJmiH0K3w/edit#
Related code:
https://docs.google.com/document/d/1vGokwMXPQItZmTHZQbTO4qwj_SQymFhRS_nJmiH0K3w/edit#
Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:
Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 3.0.0-beta.5
ios-deploy version: 1.8.5
ios-sim version: 5.0.4
OS: macOS Sierra
Node Version: v7.4.0
Xcode version: Xcode 7.2.1 Build version 7C1002

Which version of @ionic/app-script do you use?

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!

First of all you should open thr bug not here but in the app-script repro.
For second on version 1.3.2 the bug should be resolved. Try to update to latest 1.3.3 and aee this issue
https://github.com/driftyco/ionic-app-scripts/issues/892

Or probabbly @jgw96 are right. You are not loading thr right page

i keep getting this error in ionic 3.6.0. Am lazy loading pages as described above. When i stop server and restart, error disappears. After a while, it comes back. Is it an ionic bug that you guys keep brushing aside? Its really frustrating and time wasting to keep restarting the ionic server.


Make emoji selector darker for improved consistency

I could not figure out how to make it perfect (#39404D), but it's better than the default.

Thanks for notifying me about this issue, but I think there is a better way to do this.
The emoji button has a hovered class you can use instead of :not([style*="scale(1.1"]) which is quite messy. Also, you can use sepia and hue rotate to get a more accurate color.

Ok fixed in https://github.com/refact0r/midnight-discord/commit/6b68d4603eb19813cd2bb15f39da5a0fe0c1b323

Yeah, that's a way better fix, thank you. I tried messing around with filters, but for some reason I couldn't get the right result, but I never thought about using sepia.


I can't see any text

I've tried using the default font and I tried using GlobalFonts with a custom font and it doesn't show any text.
Any ideas?

Same here

Could you please provide minimal reproducible code snippets so I can debug it?

@Brooooooklyn I'm having the same issue
The code runs okay on my Mac but in a docker container the text is empty
I've verified that fonts load in the docker container by running console.dir(GlobalFonts.families);
Here's a code snippet
GlobalFonts.registerFromPath('../fonts/helvetica.ttf', 'helvetica');
GlobalFonts.registerFromPath('../fonts/helvetica-bold.ttf', 'helvetica-bold');

context.font = 'bold 15px/1 helvetica-bold';
context.fillText(user.steamId, -575, 619);

@gegtor could you also upload the helvetica-bold.ttf ?

@Brooooooklyn Yes here is it
helvetica-bold.ttf.zip

I found a fix
the font must be installed in docker (alpine)
in Dockerfile:
COPY ./fonts/helvetica-bold.ttf ./ RUN mkdir -p /usr/share/fonts/truetype/ RUN install -m644 helvetica-bold.ttf /usr/share/fonts/truetype/ RUN rm ./helvetica-bold.ttf

I found a fix the font must be installed in docker (alpine) in Dockerfile:
COPY ./fonts/helvetica-bold.ttf ./ RUN mkdir -p /usr/share/fonts/truetype/ RUN install -m644 helvetica-bold.ttf /usr/share/fonts/truetype/ RUN rm ./helvetica-bold.ttf

It works! Thank you! 🥰
@Brooooooklyn Could you put it in the docs or possibly make a workaround and fix it?

Could you try 0.1.38 without the workaround?

@Brooooooklyn I have tested writing text on my image it seems to work great.
Here is the full code:
import fs from "node:fs";
import { createCanvas, loadImage, GlobalFonts } from "@napi-rs/canvas";

// Load your custom font
GlobalFonts.registerFromPath("./ReenieBeanie-Regular.ttf", "myfont");

// Define the image and text properties
const inputImage = "./welcome-template.jpg";
const outputImage = "output-image.jpg";

const text = `Shivam,
I am so happy
to have you!`;

const fontSize = 90;
const xCoordinate = 715;
const yCoordinate = 510;
const lineHeight = fontSize * 0.9;

// Load the input image using Canvas
loadImage(inputImage).then((image) => {
  const canvas = createCanvas(image.width, image.height);
  const ctx = canvas.getContext("2d");

  // Draw the image onto the canvas
  ctx.drawImage(image, 0, 0, canvas.width, canvas.height);

  // Set the text properties
  ctx.font = `normal ${fontSize}px "myfont"`;
  ctx.fillStyle = "black";

  // Draw the text on the image with new lines
  const lines = text.split("\n");
  lines.forEach((line, index) => {
    ctx.fillText(line, xCoordinate, yCoordinate + index * lineHeight);
  });

  // Save the canvas as an image
  const buffer = canvas.toBuffer("image/jpeg", 50);
  fs.writeFileSync(outputImage, buffer);
});

It does not work for me. No text visible in canvas!
Since I am not sure if there is something specific about my Google Cloud Run node:18-alpine installation, I opened a new ticked for GCR usage here....
Please help. I am pretty desperate about this issue!

@Brooooooklyn  We are facing the same issue. The text is shown on local but no on Lambda. Could you please help?

The problem is fixed, but it's very strange.
When I register the file from a Base64 string, it only has a 5% chance of working on Lambda.
GlobalFonts.register(Buffer.from(CALIBRI_FONT_BASE64, "base64"), "Calibri");
However, when I register the font from a file, it works fine on Lambda.
I guess there is a bug in the GlobalFonts.register method, where it is not waiting for the loading process to complete.
@Brooooooklyn


support for visionOS

Would be great to have support for visionOS (Apple Vision Pro). Should be very similar to iOS and hopefully compiling isn't too hard.

Some discussion here: https://github.com/microsoft/onnxruntime/discussions/19313


Having a jumps in heading levels ruins the indentation

For example. I like to embed the TOC and some dataview metadata stuff at the top of my notes in a lvl4 heading so that I can fold it out of the way. But if I do that all the indentation above the lvl 4 get removed.

Can you share with me an example + a screenshot if at all possible.

Can you share with me an example + a screenshot if at all possible.

Look how lvl1 heading shown at lvl2 becauae Toc on top of it starts at lvl2

Short Answer
You cannot have a level 2 heading first in the document and then have a level 1 heading.
Detailed Answer
I don't think this can be fixed. The first heading in a document should be the lowest (level 1). It's quite difficult to explain so hopefully images will help.
In this example, I moved toc to level 1, this fixed the indentation.

The logic that is in play is:

Find all headings in a document
Get take the first heading
Get the level of that heading, this sets the basis of the indentation (we need indentation to render markdown lists correctly)
For each heading we subtract the first headings level found in step 3 this is because we cannot have a list that has a first item that is indented more than the subsequent items, it produces invalid markdown.

So in your example:

toc = level 2

2 - 2 = 0 (no indentation)

lvl1 = level 1

1 - 2 = -1 (negative indentation isn't a think in markdown so it's 0)

Removing the subtract logic produces invalid markdown which won't render

Recommendation
You are using headings a little unconventionally. Headings aren't for style, they provide lexical meaning to a document.
If your documents were structured like this, by having a top level 1 heading, that will fix your issue.
With the logic that we're forced to apply by subtracting the first headings level in a document by a headings level:

page title - level 1

1 - 1 = 0 (no indentation)

toc - level 2

2 - 1 = 1 (one level if indentation)

lvl1 - level 2

2 - 1 = 1

# Page title
## toc
## lvl1
### lvl2

That will fix your issue. Alternatively, make toc a level 1 header

Short Answer
You cannot have a level 2 heading first in the document and then have a level 1 heading.
Detailed Answer
I don't think this can be fixed. The first heading in a document should be the lowest (level 1). It's quite difficult to explain so hopefully images will help.
In this example, I moved toc to level 1, this fixed the indentation.
The logic that is in play is:

Find all headings in a document
Get take the first heading
Get the level of that heading, this sets the basis of the indentation (we need indentation to render markdown lists correctly)
For each heading we subtract the first headings level found in step 3 this is because we cannot have a list that has a first item that is indented more than the subsequent items, it produces invalid markdown.

So in your example:

toc = level 2

2 - 2 = 0 (no indentation)

lvl1 = level 1

1 - 2 = -1 (negative indentation isn't a think in markdown so it's 0)

Removing the subtract logic produces invalid markdown which won't render

Recommendation
You are using headings a little unconventionally. Headings aren't for style, they provide lexical meaning to a document.
If your documents were structured like this, by having a top level 1 heading, that will fix your issue. With the logic that we're forced to apply by subtracting the first headings level in a document by a headings level:

page title - level 1

1 - 1 = 0 (no indentation)

toc - level 2

2 - 1 = 1 (one level if indentation)

lvl1 - level 2

2 - 1 = 1

# Page title
## toc
## lvl1
### lvl2

That will fix your issue. Alternatively, make toc a level 1 header

Thanks for your detailed explanation, I agree that my use of heading here is a bit unconventional but the thing is that I like folding and unfolding and separating document in manageable and referenceable chunks a lot and it is the only heading that I can do it in markdown. I will use a first level heading at the top for putting front matter out of the way after this. it is going to be a bit distracting and ugly but would do the job.


compiling test version...

i am curious if everything that is needed to compile a running version of this code is available and up on this repository?
the last time i looked, i (and my java friends) had some trouble.  i even tried de-compiling the 5.0 release to see if the missing class files were there.
i would like to do some testing and am curious about creating new jar files.
appreciate any input.

Hi, as noted in the readme, all the code is available in the repository, expect one file:

I tried cloning the repo but the build fails on my machine
Yes, this is by design. There is a particular file missing from this repo and unfortunately I cannot add it here.

Like you said, you can perhaps obtain that file/value from some other location.


After the xdata header, comes the epilog scope info. Fix confusion with "exception information", which comes last.

…th "exception information", which comes last.

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

@corob-msft - Could you review the proposed changes? Thank you.


Check value passed to set_header_value_type

The details of the change are given in the commit message.
I'm requesting this change because a user of Faye is having trouble on JRuby 9.2. Faye depends on em-http-request, which uses http_parser.rb. On this line, it calls into set_header_value_type in the Java implementation:
https://github.com/igrigorik/em-http-request/blob/v1.1.5/lib/em-http/http_connection.rb#L115
This worked on JRuby 1.7 but not on 9.2, so this method always fails to accept the given argument, even if it should be valid, and this means we cannot make HTTP requests.
Here's the relevant Faye issue: https://github.com/faye/faye/issues/502

I'd also note that I cannot get the gem to compile; running rake gives me this error:
ext/ruby_http_parser/org/ruby_http_parser/RubyHttpParser.java:113: error: cannot find symbol
    this.settings.on_status = new HTTPDataCallback() {
                 ^
  symbol:   variable on_status
  location: variable settings of type ParserSettings
Note: ext/ruby_http_parser/org/ruby_http_parser/RubyHttpParser.java uses or overrides a deprecated API.

It looks like other builds on Travis have the same problem. If there's a way we can compile this gem ourselves, we can work around the bug affecting em-http-request until an official release of this gem arrives.

@headius forgive my Java ignorance but is there a way of creating a static final List from an array literal? Like, can I say private static final List<String> = ...? Or do I need to create the array from a literal and then create the List in another statement? I'm currently doing the following to statically declare a List:
    private static final String[] accepted = {"mixed", "arrays", "strings"};
    private static final List<String> valid = Arrays.asList(accepted);

I've ended up doing this -- if this is the idiomatic way to do it, I'll update my PR:
  private static final List<String> VALUE_TYPES = new ArrayList<String>(
    Arrays.asList("mixed", "arrays", "strings")
  );

I confirmed that it fixes broken tests like this:
    Fix broken tests like the following:

      1) HTTP::Parser should allow us to set the header value type
         Failure/Error: @parser.header_value_type = type

         ArgumentError:
           Invalid header value type
         # org/ruby_http_parser/RubyHttpParser.java:478:in `header_value_type='
         # ./spec/parser_spec.rb:36:in `block in <main>'
         # ./spec/parser_spec.rb:35:in `block in <main>'

Thanks for you contribution!

BTW original code should use String.equals() instead of !=, different String instance never matched by == on Java.


Add custom header X-Clerk-SDK in every request for clerk-sdk-node and vercel-edge packages

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/backend-core
[x] @clerk/clerk-sdk-node
[x] @clerk/edge
[ ] tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

Add a custom header X-Clerk-SDK in every request which contains the current SDK version

For clerk-sdk-node with the format node/<version>
For vercel-edge with the format vercel-edge/<version>

@chanioxaris I believe we should change this as we include the packageRepo attribute on Base and RestClient.

@igneel64 We don't want to modify the User-Agent for the moment, as it's out of scope of this change. We can do a cleanup in a following PR


ContextMenu.close() does not work also contextmenu.element.childElementCount never becomes zero

I have modified the fiddle created by you and created a new one here jsfiddle.
below is the code with the change, I have marked the lines which I have changed.
`contextmenu.on('open', function (evt) {
var feature =	map.forEachFeatureAtPixel(evt.pixel, ft => ft);
if (feature && feature.get('type') === 'removable') {
contextmenu.clear();
removeMarkerItem.data = { marker: feature };
contextmenu.push(removeMarkerItem);
} else {
contextmenu.clear();
contextmenu.extend(contextmenu_items);
contextmenu.extend(contextmenu.getDefaultItems());
}
contextmenu.clear(); //added this line
contextmenu.close(); ////added this line
});`
for the second issue. after calling contextmenu.clear() still contextmenu.element.childElementCount is showing as 1.
ol3-contextmenu - v2.4.1

I don't understand the logic here.

The two lines which i have inserted is just an over simplified version of
our code. My aim by putting those two lines together is to show that even
though the context menu is cleared and then when we call close on context
menu it is still displayed. And even after we call clear on context menu
still the childElementCount is equal to 1. Let me know if this still sounds
confusing
On 30-May-2017 7:13 pm, "Jonatas Walker"<EMAIL_ADDRESS>wrote:

I don't understand the logic here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jonataswalker/ol3-contextmenu/issues/119#issuecomment-304882095,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOziV5hlMd5BbJAdUWf76KQnGGrJv6gfks5r_BzkgaJpZM4NqLkM
.


New logic app designer is not saving a workflow when a change is made to a workflow in the old designer.

Describe the Bug with repro steps
Open a logic app that was created in the old designer and try to make a change in this case I have a recurrence trigger.  When I click on delete and click ok, then click on save.  nothing happens, but the trigger does not get deleted.  This happens for multiple different actions or triggers created in the old designer.
The new designer does not allow editing.

What type of Logic App Is this happening in?
Standard (Portal)
Are you using Preview Designer or GA Designer
Generally Available
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
above the questions asks if I am using the preview designer, I am using the newly GA designer.
AB#24299508

@Imelda-irby - Can you check what error you are seeing in the console? Does this happen with any action deletion only after you try to delete the trigger? Is this only happening for deletion or even editing parameter values without trying the deletion first.
If you are totally unable to edit your LA then please file a case with CSS so you can share network and portal logs.

Hello, @Priti @.***>,
I am from CSS, and this request is for a customer of mine.  I was not sure if we just open all the requests for the new designer issues in Github.
What I am sharing here is my repro, what do I need from the customer?
The customer is totally unable to edit for multiple actions.  Either edit or delete action that were created on the old designer.
From: preetriti1 @.>
Sent: Friday, May 26, 2023 3:47 PM
To: Azure/LogicAppsUX @.>
Cc: Imelda Irby @.>; Mention @.>
Subject: Re: [Azure/LogicAppsUX] New logic app designer is not saving a workflow when a change is made to a workflow in the old designer. (Issue #2582)
@Imelda-irbyhttps://github.com/Imelda-irby - Can you check what error you are seeing in the console? Does this happen with any action deletion only after you try to delete the trigger? Is this only happening for deletion or even editing parameter values without trying the deletion first.
If you are totally unable to edit your LA then please file a case with CSS so you can share network and portal logs.

Reply to this email directly, view it on GitHubhttps://github.com/Azure/LogicAppsUX/issues/2582#issuecomment-1565040869, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVY6MB6V2VL57R5N5TCFOYLXIEXGNANCNFSM6AAAAAAYQXNUBU.
You are receiving this because you were mentioned.Message ID: @.***>

Please share workflow.json, connection strings, network logs and portal console logs if you have the repro. We will need that for investigation because its something specific you guys are doing because we have been able to edit it.
Github issues are for quick turnaround time and to make it easier for customers. But they can't share the logs there so we ask then to open incident if it is customer or scenario specific.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Imelda Irby @.>
Sent: Friday, May 26, 2023 3:56:37 PM
To: Azure/LogicAppsUX @.>; Azure/LogicAppsUX @.>; Priti Sambandam @.>
Cc: Mention @.***>
Subject: RE: [Azure/LogicAppsUX] New logic app designer is not saving a workflow when a change is made to a workflow in the old designer. (Issue #2582)
Hello, @Priti @.***>,
I am from CSS, and this request is for a customer of mine.  I was not sure if we just open all the requests for the new designer issues in Github.
What I am sharing here is my repro, what do I need from the customer?
The customer is totally unable to edit for multiple actions.  Either edit or delete action that were created on the old designer.
From: preetriti1 @.>
Sent: Friday, May 26, 2023 3:47 PM
To: Azure/LogicAppsUX @.>
Cc: Imelda Irby @.>; Mention @.>
Subject: Re: [Azure/LogicAppsUX] New logic app designer is not saving a workflow when a change is made to a workflow in the old designer. (Issue #2582)
@Imelda-irbyhttps://github.com/Imelda-irby - Can you check what error you are seeing in the console? Does this happen with any action deletion only after you try to delete the trigger? Is this only happening for deletion or even editing parameter values without trying the deletion first.
If you are totally unable to edit your LA then please file a case with CSS so you can share network and portal logs.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/LogicAppsUX/issues/2582#issuecomment-1565040869, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVY6MB6V2VL57R5N5TCFOYLXIEXGNANCNFSM6AAAAAAYQXNUBU.
You are receiving this because you were mentioned.Message ID: @.***>

I am getting the same issue - when you have an old flow and try the new designer.
Making a change and trying to save the change does not take, then trying to exit it prompts you about leaving before saving.
In mine the following steps:
1 - open old workflow
2 - try new preview
3 - make changes - delete an action and add an action - in this case a Start a VM action is being removed and another added.

4 - click save and prompt says all changes saved
Saving:

5 - try to navigate away, prompt asks to save before navigating away.
Navigate back:

When you refer to "2 - try new preview", it makes me think that you still have the option to go back and forth between preview and old designer; can we check and see if this is in consumption or standard?

Hi Eric,
The issue I reported is in Standard Logic app.
From: Eric Wu @.>
Sent: Tuesday, May 30, 2023 5:17 AM
To: Azure/LogicAppsUX @.>
Cc: Imelda Irby @.>; Mention @.>
Subject: Re: [Azure/LogicAppsUX] New logic app designer is not saving a workflow when a change is made to a workflow in the old designer. (Issue #2582)
When you refer to "2 - try new preview", it makes me think that you still have the option to go back and forth between preview and old designer; can we check and see if this is in consumption or standard?

Reply to this email directly, view it on GitHubhttps://github.com/Azure/LogicAppsUX/issues/2582#issuecomment-1568332516, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVY6MB2THTGSKYFHV26TN6DXIXQLJANCNFSM6AAAAAAYQXNUBU.
You are receiving this because you were mentioned.Message ID: @.***>

When you refer to "2 - try new preview", it makes me think that you still have the option to go back and forth between preview and old designer; can we check and see if this is in consumption or standard?

Hi @Eric-B-Wu - yes - consumption plan -

I can go back and forth:

This issue has been resolved for my customer.
Thank you,
From: github-actions[bot] @.>
Sent: Friday, June 16, 2023 3:49 PM
To: Azure/LogicAppsUX @.>
Cc: Imelda Irby @.>; Mention @.>
Subject: Re: [Azure/LogicAppsUX] New logic app designer is not saving a workflow when a change is made to a workflow in the old designer. (Issue #2582)
We requested more information to triage this issue. If the issue goes more than 7 days with no more information it'll be automatically closed.

Reply to this email directly, view it on GitHubhttps://github.com/Azure/LogicAppsUX/issues/2582#issuecomment-1595413531, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVY6MB6S4II4LPPJQXTUSYDXLTPERANCNFSM6AAAAAAYQXNUBU.
You are receiving this because you were mentioned.Message ID: @.***>


feat: :sparkles: added ability to output exitcode in Terraform Plan task

This feature allows the exitcode of Terraform Plan to be consumed within the pipeline after a run

Hi @JaydenMaalouf , for now you need to add the same changes to task.loc.json as you did for task.json.

Hi @JaydenMaalouf , for now you need to add the same changes to task.loc.json as you did for task.json.

Thanks, this has been updated!

Hi, finally got the extension installed in our dev env.
Unfortunately the suggested changes does not work.  Azure Pipelines picks up on terraform exit code and fails.

same for linux and windows agents.

Hi, finally got the extension installed in our dev env. Unfortunately the suggested changes does not work. Azure Pipelines picks up on terraform exit code and fails.

same for linux and windows agents.

Could we potentially change the exec to ignore the result?
Something like this:
let result = terraformTool.exec(<IExecOptions> {
    cwd: planCommand.workingDirectory,
    ignoreReturnCode: true
});

if (result === 1) {
    throw new Error('Terraform failed with exit code: ' + result);
}

return result;

Yeah that looks good!  was looking through the azure-pipelines-task-lib and found the same. update the code and I will run another round of tests in our dev environment.

Yeah that looks good! was looking through the azure-pipelines-task-lib and found the same. Please update the code and I will run another round of tests in our dev environment.

Awesome, thanks for that!
I've updated the code - decided to add the error message in the loc files for consistency sake too
Hope I haven't missed anything haha

Hi, I am curious if you see the section 'This branch is out-of-date with the base branch' in checks? if you do please click 'update branch' to integrate the latest changes from main before I merge.  If you can't see it, I can do it but since it is your fork it might be better if you do it.

Hi, I am curious if you see the section 'This branch is out-of-date with the base branch' in checks? if you do please click 'update branch' to integrate the latest changes from main before I merge.  If you can't see it, I can do it but since it is your fork it might be better if you do it.

Yep!
Merged in main into my branch.
Appreciate your time!

this feature is now released to public (0.1.14)

this feature is now released to public (0.1.14)

Thanks!
I can see the variable is now being set appropriately.

How do I reference this in another job? As it seems isOutput=false, which I think is required to use it as a dependency
The best I've got so far is to run a script with the following
echo "##vso[task.setvariable variable=outputChangesPresent;isOutput=true]$(echo $(changesPresent))```
To reference the variable outside of the scope of the current job.

this feature is now released to public (0.1.14)

Thanks! I can see the variable is now being set appropriately.
How do I reference this in another job? As it seems isOutput=false, which I think is required to use it as a dependency The best I've got so far is to run a script with the following
echo "##vso[task.setvariable variable=outputChangesPresent;isOutput=true]$(echo $(terraformPlan.changesPresent))

I had to add a name to the task to reference it
- task: TerraformTaskV2@2
  name: terraformPlan
  displayName: "Terraform: Plan"

This allows me to reference the variable in another job - but should this not be possible with the changesPresent variable directly off the task?

@mericstam I dig some digging and we can do this directly in the task.setVariable
PR has been raised here: https://github.com/microsoft/azure-pipelines-terraform/pull/41

I am not sure it this is really the same as https://github.com/microsoft/azure-pipelines-extensions/issues/962
there the idea, to me, was to allow to gracefully skip apply task in case plan didn't detected any changes.
Using exit codes has this disadvantage that the pipeline is not "green" and for example Bitbucket can see it as failed.
Any thoughts? Should I open another issue for it?

Hi @vanmash, detailed-exitcode exits almost as before but adds the exit code 2 which indicates changes present.  In the terraformtask we ignore the returncodes see: line 133  and then trap the exit-codes and if it is 0 or 2 we don't cast an error.   see: line136   then  on line 217 we set an outputvariable named changesPresent which you can use in an condition on the apply step.  In my tests we got the build green.  Did you encounter any issues? If so please provide some more information so we can fault trace

Hi @mericstam , thank you. I am like 50% there :)
I added name: to the plan task. And condition: to the apply stage.
stages:
  - stage: terraform_plan
    jobs:
      - job: plan
        ...
        steps:
        ...
        - task: TerraformTaskV2@2
          name: 'plan'
          inputs:
            provider: 'azurerm'
            command: 'plan'
        ...

  - stage: terraform_apply
    dependsOn: [terraform_plan]
    condition: eq(dependencies.terraform_plan.outputs['terraform_plan.plan.changesPresent'], 'true')

The apply stage is skipped when there are no changes. Good.
But it's also skipped for runs with changes. Not so good.
Do you see any obvious mistakes?


[FIX] Threading stack overflow size

Fix threading stack overflow size from python-pillow/Pillow#1935

Awk fixed


How to use fingers to move the GPUImageUIElement?

I am using GPUImage to make a camera app, and I add a GPUImageUIElement to the filter, but I think the GPUImageUIElement was only rendered to the view, it wasn't become the subview of the view. I want to use fingers to move the element, but the userinteraction can't get the element, how can I move the element?

My recommendation would be to apply a transform / scale operation on it before passing it into your blend, and have the transformation track finger position.


In ContainerServiceManager, are some operations missing on KubernetesClusters?

Query/Question
I'm wondering if I use correctly the KubernetesClusters resource.
var clusters = ContainerServiceManager
				.authenticate(clustersClientConfig.credential(), clustersClientConfig.azureProfile())
				.kubernetesClusters();

1st question
When defining a cluster (interface KubernetesCluster and class KubernetesClustersImpl)
var cluster = kubernetesClusters.define(name)
				.withRegion(region)
				.withExistingResourceGroup(resourceGroupName)
				.withVersion(version)
				.withSystemAssignedManagedServiceIdentity()
				.withDnsPrefix(name.toLowerCase())
				.withTags(tags);

I don't see any withSku. Is there a way to define the SKU like in ManagedClusterInner line 96?
2nd question:
When updating an existing cluster, there is no way to upgrade the kubernetes version, in:
kubernetesCluster.update()
                                .withoutAgentPool(agentPool)
				.applyAsync();

We should have a withNewKubernetesVersion, am I wrong?
Why is this not a Bug or a feature Request?
I don't know for now, maybe it's a feature request, or a better documentation.
Setup (please complete the following information if applicable):

OS: Linux Ubuntu
IDE: Eclipse
Library/Libraries: com.azure.resourcemanager:azure-resourcemanager-containerservice:2.35.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@weidongxu-microsoft could you follow up with @agateaux

@agateaux Thanks for the report. This part of the code is handwritten. Hence, sometimes it lags behind what backend offers.
We will add the way to set SKU, as well as e.g. withKubernetesVersion to set/update version.

Thanks.
For the creation, the definition stage already supports the 'withVersion', to specify the Kubernetes version.
That's just for the update, the update stage doesn't offer this feature.

PR merged: https://github.com/Azure/azure-sdk-for-java/pull/39110
2.37.0 will be around mid March.

Thank you

@agateaux 2.37.0 released. Let us know if it works for you.


Add support for mixed content in Side-nav

This task is for adding support for different types of content in Side-nav as seen in:

Latest sketch file: https://ibm.box.com/s/4e4zrz6mlo0jpr8lw4vimwlxn4vc59cy

Is this likely to be updated in this sprint, or would we be better putting into Backlog in preparation for next sprint planning on Monday?

@shinytoyrobots was planning on tackling it today but I can defer till next week as well

Do we consider this to be necessary within the Q3 milestone?

I think this could be considered a part of the core functionality of this component, but it can definitely be deferred.

Going to go ahead and move it back to the backlog for next sprint 👍


merging non-canonical environments fails

There are cases where environments are specified at the transform/windowing level that are not included in the top-level components' environments.
This is particularly present when running sql-backed provider on multiple inputs (i.e. when joining 2 pcollections) in Beam YAML which creates 2 default environments, but only declares one in the proto.components.environments section.
Perhaps the real issue lies in how Beam YAML providers create transforms, but this is a nice safeguard regardless.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @robertwb

That would definitely be an erroneously constructed graph. Can you give a concrete example of how this is crated?

@robertwb After further investigation, it appears this occurs anytime that 2 subsequent ExternalTransforms are declared that use the same gradle target, and the first is given multiple inputs (PCollectionTuple). I haven't been able to repro unless the input to the first ExternalTransform is a dict of PCollections.
Python example:
import apache_beam as beam
from apache_beam.transforms import external
import logging

from apache_beam.utils import subprocess_server

logging.getLogger().setLevel(logging.INFO)

with beam.Pipeline('DirectRunner') as p:
  i1 = p | "i1" >> beam.Create([beam.Row(name='john', id=1)])
  i2 = p | "i2" >> beam.Create([beam.Row(name='jane', id=1)])
  result = {'i1': i1, 'i2': i2} | 'Sql1' >> external.ExternalTransform(
        'beam:external:java:sql:v1',
        external.ImplicitSchemaPayloadBuilder(
          {'query': 'SELECT * FROM i1 INNER JOIN i2 ON i1.id = i2.id'}
        ).payload(),
    external.JavaJarExpansionService(
      subprocess_server.JavaJarServer.path_to_beam_jar(
        gradle_target='sdks:java:extensions:sql:expansion-service:shadowJar',
        artifact_id=None
      )
    )) | 'LogForTesting' >> external.SchemaAwareExternalTransform(
        'beam:schematransform:org.apache.beam:yaml:log_for_testing:v1',
    external.JavaJarExpansionService(
      subprocess_server.JavaJarServer.path_to_beam_jar(
        gradle_target='sdks:java:extensions:sql:expansion-service:shadowJar',
        artifact_id=None
      )
    ), rearrange_based_on_discovery=True)

YAML example:
pipeline:
  transforms:
    - type: Create
      name: table1
      config:
        elements:
          - name: "john"
            id: 1
    - type: Create
      name: table2
      config:
        elements:
          - name: "jane"
            id: 1
    - type: Sql
      name: Join
      input:
        i1: table1
        i2: table2
      config:
        query: "SELECT * FROM i1 INNER JOIN i2 ON i1.id = i2.id"
    - type: LogForTestingJava
      input: Sql

# Force same gradle target variant of LogForTesting
providers:
  - type: 'beamJar'
    config:
      gradle_target: 'sdks:java:extensions:sql:expansion-service:shadowJar'
    transforms:
      LogForTestingJava: 'beam:schematransform:org.apache.beam:yaml:log_for_testing:v1'

@robertwb After further investigation, it appears this occurs anytime that 2 subsequent ExternalTransforms are declared that use the same gradle target, and the first is given multiple inputs (PCollectionTuple). I haven't been able to repro unless the input to the first ExternalTransform is a dict of PCollections.

Note: changing the gradle target for LogForTesting on either example I posted will result in a successful run.
The transforms also do not have to be connected to each other:
  service = external.JavaJarExpansionService(
        subprocess_server.JavaJarServer.path_to_beam_jar(
          gradle_target='sdks:java:extensions:sql:expansion-service:shadowJar',
          artifact_id=None
        )
      )

  result = {'i1': i1, 'i2': i2} | 'Sql1' >> external.ExternalTransform(
          'beam:external:java:sql:v1',
          external.ImplicitSchemaPayloadBuilder(
            {'query': 'SELECT * FROM i1 INNER JOIN i2 ON i1.id = i2.id'}
          ).payload(),
      service)

  result2 = {'i1': i1, 'i2': i2} | 'Flatten' >> external.SchemaAwareExternalTransform(
    'beam:schematransform:org.apache.beam:yaml:flatten:v1',
    service, rearrange_based_on_discovery=True)

and it only fails if one transform is ExternalTransform and one is SchemaAwareExternalTransform. i.e. the following does not fail:
service = external.JavaJarExpansionService(
        subprocess_server.JavaJarServer.path_to_beam_jar(
          gradle_target='sdks:java:extensions:sql:expansion-service:shadowJar',
          artifact_id=None
        )
      )
result2 = {'i1': i1, 'i2': i2} | 'Flatten' >> external.SchemaAwareExternalTransform(
    'beam:schematransform:org.apache.beam:yaml:flatten:v1',
    service, rearrange_based_on_discovery=True
  ) | 'LogForTesting' >> external.SchemaAwareExternalTransform(
    'beam:schematransform:org.apache.beam:yaml:log_for_testing:v1',
    service, rearrange_based_on_discovery=True)


Fix non-existent domains (typo)

I noticed that 2 non-existent domains were included previously.

@Gallaecio ahh good catch! yep, it's those domain suffixes that were missing.


Feature Request: MacOS: Overlay to be carried across all full screen window and other desktops

When moving across full screen windows and multiple desktops, the overlay only stays on the first desktop. Realise this is likely a limitation to macOS, but if it can be sticky across every application that would be great
Tested on MacOS Sequoia 15.2
Many thanks

I think I did a similar thing with my beacon project so maybe we can implement that here.

https://github.com/josephdadams/beacon


compress/gzip: panic in Writer.Close()

What version of Go are you using (go version)?
go1.7.5 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/will/src/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.7.5/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.7.5/libexec/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/bm/zycksyvx1h39404g4v7wk42c0000gn/T/go-build363334644=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

But I was cross compiling with CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -installsuffix cgo.
And ran it on
cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.1 LTS"

On that machine, it is running in docker container (docker version: Docker version 1.13.0, build 49bf474) using the base image alpine:3.3.
What did you do?
We wrap an http.ResponseWriter in gzip.Writer. When the CloseNotifer of the http.ResponseWriter fires, we call Writer.Close().
The source code of this file is here.
I am aware this might not be the correct thing to do since closing the gzip.Writer will attempt to flush to a closed connection, but I wouldn't have expected such a low-level panic as the symptom.
What did you expect to see?
Either it to have no effect, or return an error.
What did you see instead?
This code will have been executed tens of thousands of times, and this panic only occurred a single time.

I have also observed a panic that is even more problematic: it can occur from a call to Writer.Flush(). I am posting it as a comment here because it seems related.
Maybe it is caused by attempting to flush to a closed connection? The reason I'm saying it's more problematic is because we do not know at the point at calling Flush whether the underlying Writer has been closed.
Here's is the panic:
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: panic: runtime error: index out of range
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: goroutine 23215 [running]:
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: panic(0x8196e0, 0xc420016060)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /usr/local/Cellar/go/1.7.5/libexec/src/runtime/panic.go:500 +0x1a1
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: compress/flate.(*huffmanBitWriter).flush(0xc422294b40)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /usr/local/Cellar/go/1.7.5/libexec/src/compress/flate/huffman_bit_writer.go:126 +0xec
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: compress/flate.(*compressor).syncFlush(0xc420f08000, 0x7, 0x40)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /usr/local/Cellar/go/1.7.5/libexec/src/compress/flate/deflate.go:563 +0xb4
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: compress/flate.(*Writer).Flush(0xc420f08000, 0x7, 0xc4216e9600)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /usr/local/Cellar/go/1.7.5/libexec/src/compress/flate/deflate.go:715 +0x2d
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: compress/gzip.(*Writer).Flush(0xc422542c60, 0x0, 0x100)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /usr/local/Cellar/go/1.7.5/libexec/src/compress/gzip/gzip.go:220 +0x61
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: github.com/pusher/feeds/vendor/github.com/pusher/httputil2.(*gzipResponseWriter).Flush(0xc4202f5f90)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/httputil2/gzip_middleware.go:106 +0x2e
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: github.com/pusher/feeds/vendor/github.com/pusher/httputil2.(*recoverResponseWriter).Flush(0xc42022cfc0)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/httputil2/recovery_middleware.go:70 +0x73
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go.sendMessage(0x7f9205599928, 0xc42022cfc0, 0xad5a20, 0xaf82b0, 0x0, 0x0)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go/event_writer.go:337 +0xcf
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go.(*EventWriter).openState(0xc42028a770)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go/event_writer.go:269 +0x661
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go.(*EventWriter).initState(0xc42028a770)
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go/event_writer.go:193 +0xaa8
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]: created by github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go.newEventWriter
Feb 01 10:14:25 ip-10-10-24-233 docker[1525]:         /Users/will/src/go/src/github.com/pusher/feeds/vendor/github.com/pusher/elements-server-go/event_writer.go:138 +0x305

Flush() is called for every write to a response body, so again, this code will have been executed tens of thousands of times, and I have only observed this panic a single time.

Can you run your code with the race detector enabled, the failures that you getting in compress/flate are unlikely to happen.

Your logic looks racy. self.gz.Close is called in a separate goroutine and can potentially run at exactly the same time as any of the other gzip operations.

I wasn't clear whether the library was threadsafe or not. I will wrap the Close in a mutex.
I will also leave it running with the race detector on. It could be a while before I have any results.
I'm not sure the "racy" behaviour (when closing) explains the second panic (when flushing) however. This should only be executed in the same goroutine. Again, hopefully the race detected will provide a definitive answer on this.
I will keep you posted.

Ok, I managed to catch a data race with the race checker enabled. It does look like it was caused by the call to Close flushing data concurrently with data being written to the underlying Writer. I will close this issue. Thanks for your help.


Improve security of the kubelet-summary-exporter

Use non-root user in the container image
Upgrade dependencies to fix security issues found using Snyk

Close the PR so the SLA-bot can see the signature


Realm: Deferred allocation breaks under a tight memory budget

reproducer.rg
import "regent"
import "bishop"

mapper
task#f region#r {
  create: demand;
}
end

task f(r : region(ispace(int1d), int)) where reads writes(r) do end

task main()
  var size = 16777216
  var r = region(ispace(int1d, size), int) -- 64MB
  var s = region(ispace(int1d, size), int) -- 64MB
  for i = 0, 10 do
    f(r)
    f(s)
  end
end

regentlib.start(main, bishoplib.make_entry())

Command line
regent.py reproducer.rg -lg:window 10 -ll:csize 256 -level 5

Symptom
$ regent.py reproducer.rg -lg:window 10 -ll:csize 256 -level 5
terra: /gpfs/fs1/wonchanl/legate.core/legion/runtime/realm/mem_impl.cc:787: void Realm::MemoryImpl::deferred_destruction_triggered(Realm::RegionInstanceImpl*, bool, Realm::TimeLimit): Assertion `future_allocator.lookup(it2->inst->me, f_first, f_size) && (f_first == offset) && (f_size == it2->bytes)' failed.

I believe this is fixed with commit 41aeaa3.  @magnatelee can you poke at it some more and see if you agree?

I threw several different flags to the test and I wasn't able to make it break, so I'd say the issue is fixed.


Grav Memcached call connect / addServer

Grav appears to be making an improper call that ( at least in my environment ) prevents it from working.
Standard zip install using 1.1.3 zip file results in the following when attempting to configure memcached:
Whoops \ Exception \ ErrorException (E_ERROR)
Call to undefined method Memcached::connect()

0 Whoops\Exception\ErrorException
…/system/src/Grav/Common/Cache.php186

            case 'memcached':
                $memcached = new \Memcached();
                $memcached->connect($this->config->get('system.cache.memcached.server', 'localhost'),
                    $this->config->get('system.cache.memcached.port', 11211));
                $driver = new DoctrineCache\MemcachedCache();
                $driver->setMemcached($memcached);
                break;

Per this discussion at StackOverflow I make the following change to line 186:
$memcached->connect($this->config->get('system.cache.memcached.server', 'localhost'),

becomes
$memcached->addServer($this->config->get('system.cache.memcached.server', 'localhost'),

System functionality is immediately restored.

Will fix. Thanks!


Update OpenNURBS to v7.0

Update OpenNURBS v6.15 to v7.0
Reference: https://github.com/mcneel/opennurbs/releases/tag/v7.0.19309.17460

It looks like OpenNURBS v7.0 still work-in-progress and it throws some linking errors. Let's wait for a while till those issues are fixed.

OpenNURBS submodule is updated to the reference https://github.com/mcneel/opennurbs/commit/a59f3b7565355d80fdddb95b80a52c740806e248


:book: Create section navigation that works across any level of nesting

Resolves #3218
The fix_url function that we are using doesn't work at a nested level. Given how we are using this partial, using a "current level" value (i.e. ./workspace-types) vs a "go up levels" (i.e. ../concepts/workspace/workspace-types) will be portable across any level of nesting.

/ok-to-test

/lgtm
/approve


Allow iterator in perceptron tagger training

This pull request resolves issue #1486.
Goal: To allow the training sentences for the PerceptronTagger class to be a generator/iterator of sentences as well (in addition to a list).
Because self._make_tagdict(sentences) runs at the beginning of the train method and iterates through the sentences, we first initialize an empty list of self._sentences  and make self._make_tagdict populate self._sentences with all the sentences. This saves the overheard of just iterating through sentences to get the list by sentences = list(sentences). Now random.shuffle can happily take self._sentences.
With this update/fix, the following doesn't throw an error anymore:
>>> from nltk.tag import PerceptronTagger
>>> from nltk.corpus import alpino as alp
>>> training_corpus = alp.tagged_sents()
>>> tagger = PerceptronTagger(load=False)
>>> tagger.train(training_corpus)

thanks @jacksonllee – looks good


Move files between PC and device

possibility to move files from the mobile phone to PC and From PC to mobile phone

When enabling MTP, scrcpy disconnects and you have to relaunch it. Either fixing that or allowing drag n drop for files would make this tool way more useful.

from PC to device

how would you select the target destination?
there is already a drag&drop feature to install APK: what if you want to copy an apk to the device?

from device to PC

how would you select the file at all?
how would you select the target destination?

I do not know how easy it is. I would simply like to be able to move files bi-directionally

Transferring file from PC to device is implemented by https://github.com/Genymobile/scrcpy/pull/226.
I see no way to do the reverse (how to select the file on the device?), so I think I can close this issue.

Transferring file from PC to device is implemented by #226.
I see no way to do the reverse (how to select the file on the device?), so I think I can close this issue.

to select the file on the device simply you could use an external application to select the files from it and transfer them to computer.
transferring files from device to host through wireless would be much easier than some other applications which need so many handshakes like Samsung flow or Airdroid (which is I don't trust them at all) so it's completely a cool feature.

Hi, I really like scrcpy. I find several topics to request file transfer feature from device to PC, and I also like this idea.
How about operate like this:

user press a hot key
popup a android file explorer, which looks like windows file explorer, and shows folders and files from device
drag & drop files from android file explorer to PC
the selected files are pulled from device to PC

Thank you for your great job on scrcpy!

Hi, I really like scrcpy. I find several topics to request file transfer feature from device to PC, and I also like this idea.
How about operate like this:

user press a hot key
popup a android file explorer, which looks like windows file explorer, and shows folders and files from device
drag & drop files from android file explorer to PC
the selected files are pulled from device to PC

Thank you for your great job on scrcpy!

yes

From Mobile To PC
To transfer files from mobile to pc you can use  android studio.
-> in the right conrner click tap devices and files explorer in this tab you will find all your device files system

Simple GUI file manager will be fine like in Myphoneexplorer or like in https://www.tightvnc.com/download.php

or just a simple set of CLI commands because they alone could get a lot of things done


workflows, remove deprecated functions

Description
start of https://github.com/dapr/python-sdk/issues/738
adds comments indicating the workflow functions that are deprecated
updates the existing example app to use the recommended functions
pending updating the contents of daprdocs
pending support for the purge API to finish this work
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

ping

ping


Revert "Bump netty.version from 4.1.82.Final to 4.1.84.Final"

This reverts commit fd3955ec39fc2897a3fd671557d50ea2ac12208b.
Changes in latest version of netty (https://github.com/netty/netty/pull/12760) cause existing unit tests to fail. Reverting until the correct logic is being tested in littleProxy.

@pietrygamat Thanks. My bad, I didn't checked that the build broken. :(
Now we need to investigate why Netty update breaks the build.