New nullability annotations on custom type adapters break nullable types

In 0.4.2 - nullable fields backed by custom types will break because the CustomTypeAdapter API only allows for returning nonnull values. in loosely checked Java code this flies, but in kotlin it becomes either a compile error or runtime error (if it is null and you used !!).
Will link a repro branch tomorrow but putting up for visibility now. I think either the return nullability should be relaxed or a nullsafe wrapper should be introduced and used in the event of nullable types. The former is probably more flexible, as the only way the latter works is if the nullsafe adapter try/catches the delegate adapter

@digitalbuddha  @hzsweers I'm sorry, a little confused, what the issue? For nullable fields backed by custom types if it's value null we don't call registered CustomTypeAdapter but set to null right away.

Here's a repro branch: https://github.com/hzsweers/CatchUp/tree/z/apollo042
The generated code issue is in GithubSearchQuery$AsRepository$Mapper#map, this line in particular
final HttpUrl homepageUrl = reader.readCustomType((ResponseField.CustomTypeField) $responseFields[4]);

This doesn't handle the nullability of the homepageUrl field, and delegates directly to the HttpUrlAdapter implementation, which must nullcheck now in order to compile
class HttpUrlApolloAdapter : CustomTypeAdapter<HttpUrl> {
  override fun decode(s: String) = HttpUrl.parse(s)!!

  override fun encode(o: HttpUrl) = o.toString()
}

This results in the following trace at runtime during deserialization if that field is null
10-21 04:00:46.340 2603-2603/io.sweers.catchup.debug E/ServiceController$loadD: HttpException
                                                                                com.apollographql.apollo.exception.ApolloParseException: Failed to parse http response
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor.parse(ApolloParseInterceptor.java:101)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor.access$100(ApolloParseInterceptor.java:28)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor$1.onResponse(ApolloParseInterceptor.java:53)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloServerInterceptor$1$1.onResponse(ApolloServerInterceptor.java:93)
                                                                                at com.google.firebase.perf.network.zzg.onResponse(Unknown Source:16)
                                                                                at okhttp3.RealCall$AsyncCall.execute(RealCall.java:153)
                                                                                at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
                                                                                at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
                                                                                at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
                                                                                at java.lang.Thread.run(Thread.java:764)
                                                                                Caused by: kotlin.KotlinNullPointerException
                                                                                at io.sweers.catchup.data.HttpUrlApolloAdapter.decode(HttpUrlApolloAdapter.kt:26)
                                                                                at io.sweers.catchup.data.HttpUrlApolloAdapter.decode(HttpUrlApolloAdapter.kt:25)
                                                                                at com.apollographql.apollo.internal.response.RealResponseReader.readCustomType(RealResponseReader.java:189)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$AsRepository$Mapper.map(GitHubSearchQuery.java:920)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Node$Mapper$1.read(GitHubSearchQuery.java:667)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Node$Mapper$1.read(GitHubSearchQuery.java:664)
                                                                                at com.apollographql.apollo.internal.response.RealResponseReader.readConditional(RealResponseReader.java:215)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Node$Mapper.map(GitHubSearchQuery.java:664)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Search$Mapper$2$1.read(GitHubSearchQuery.java:461)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Search$Mapper$2$1.read(GitHubSearchQuery.java:458)
                                                                                at com.apollographql.apollo.internal.response.RealResponseReader$ListItemReader.readObject(RealResponseReader.java:306)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Search$Mapper$2.read(GitHubSearchQuery.java:458)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Search$Mapper$2.read(GitHubSearchQuery.java:455)
                                                                                at com.apollographql.apollo.internal.response.RealResponseReader.readList(RealResponseReader.java:163)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Search$Mapper.map(GitHubSearchQuery.java:455)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Data$Mapper$1.read(GitHubSearchQuery.java:314)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Data$Mapper$1.read(GitHubSearchQuery.java:311)
                                                                                at com.apollographql.apollo.internal.response.RealResponseReader.readObject(RealResponseReader.java:136)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Data$Mapper.map(GitHubSearchQuery.java:311)
                                                                                at io.sweers.catchup.service.github.GitHubSearchQuery$Data$Mapper.map(GitHubSearchQuery.java:306)
                                                                                at com.apollographql.apollo.internal.response.OperationResponseParser$1.read(OperationResponseParser.java:56)
                                                                                at com.apollographql.apollo.internal.json.ResponseJsonStreamReader.nextObject(ResponseJsonStreamReader.java:86)
                                                                                at com.apollographql.apollo.internal.response.OperationResponseParser.parse(OperationResponseParser.java:51)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor.parse(ApolloParseInterceptor.java:87)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor.access$100(ApolloParseInterceptor.java:28)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloParseInterceptor$1.onResponse(ApolloParseInterceptor.java:53)
                                                                                at com.apollographql.apollo.internal.interceptor.ApolloServerInterceptor$1$1.onResponse(ApolloServerInterceptor.java:93)
                                                                                at com.google.firebase.perf.network.zzg.onResponse(Unknown Source:16)
                                                                                at okhttp3.RealCall$AsyncCall.execute(RealCall.java:153)
                                                                                at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
                                                                                at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
                                                                                at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
                                                                                at java.lang.Thread.run(Thread.java:764)

That's weird because if you take a look at https://github.com/apollographql/apollo-android/blob/master/apollo-runtime/src/main/java/com/apollographql/apollo/internal/response/RealResponseReader.java#L183
We handle the null value

Agh wait, are you returning null as the result of fun decode(s: String) in your adapter?

No, the point is I can't because the annotations don't allow it :)

Hm I see, so yeah if we pass to custom adapter a raw value (string) that is not null then we expect adapter to return decoded non null value back .
I guess in your case the problem is HttpUrl.parse  returns null, I wish OkHttp3 provided smth similar to Uri.EMPTY instead of null.
Doesn't it looks strange backend returns some non-null value in the response but custom type adapter converts it to null?
What we can do is to remove NonNull annotation from adapter interface to allow return null values and validate nullability for non-null fields on our end.
@Nullable
public static HttpUrl parse(String url)
Returns a new HttpUrl representing url if it is a well-formed HTTP or HTTPS URL, or null if it isn't.

@swankjesse any chance you've thought about changes to how the http parse api works? We can work around it but I wanted to reach out to see if you had pending changes to be more kotlin friendly.

@hzsweers  took this offline with Jesse this was the tldr It's not a null/non-null null problem, it's a valid/invalid URL problem, And you can decide what to do with invalid URLs however you like Crashing, returning a default value, returning null
The issue isn't that we don't handle null correctly, the issue is that you are not validating whether something is an actual URL prior to sending it to the HttpURL.parse. I don't believe we need to make any changes on our end as this is an implementation detail of your adapter and what it delegates to

@hzsweers @digitalbuddha so should we close this issue? Or we need to do some changes on our end?

I like your suggestion
What we can do is to remove NonNull annotation from adapter interface to allow return null values and validate nullability for non-null fields on our end.
I vote for that if not much effort

Yep I agree with the plan above :)


Prometheus TSDB compactions failed

What did you do?
I have alert on increase(prometheus_tsdb_compactions_failed_total[1m]) > 0 (with 1.3333 value)
In prometheus logs:
prometheus[3247006]: ts=2022-11-16T02:48:26.172Z caller=db.go:908 level=error component=tsdb msg="compaction failed" err="compact head: persist head block: populate block: chunk iter: populateWithDelChunkSeriesIterator: unexpected empty chunk found while rewriting chunk"

I assume this can be bug because of comment:
// Empty chunk, this should not happen, as we assume full
// deletions being filtered before this iterator.
p.err = errors.New("populateWithDelChunkSeriesIterator: unexpected empty chunk found while rewriting chunk")

from https://github.com/prometheus/prometheus/blob/fa6e05903fd3ce52e374a6e1bf4eb98c9f1f45a7/tsdb/querier.go#L711
May be it is related - before it is started I deleted some old data after configure relabeling with
curl -X POST -g 'http://localhost:9090/api/v1/admin/tsdb/delete_series?match[]={instance="some.server:1234"}'

What did you expect to see?
no errors
What did you see instead? Under which circumstances?
errors as described
System information
Debian 11 with 5.10.0-10-amd64 #1 SMP Debian 5.10.84-1 (2021-12-08) x86_64 GNU/Linux
Prometheus version
prometheus, version 2.40.1 (branch: HEAD, revision: 965274259f993801d63f553639d248588287120b)
  build user:       root@d020f7085e1e
  build date:       20221109-14:30:50
  go version:       go1.19.3
  platform:         linux/amd64

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
prometheus[3247006]: ts=2022-11-16T02:48:26.172Z caller=db.go:908 level=error component=tsdb msg="compaction failed" err="compact head: persist head block: populate block: chunk iter: populateWithDelChunkSeriesIterator: unexpected empty chunk found while rewriting chunk"

cc @codesome

I have the same error!
System information
Centos 7 Linux 3.10.0-1160.66.1.el7.x86_64 #1 SMP Wed May 18 16:02:34 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Prometheus version
Version	2.40.1
Revision	965274259f993801d63f553639d248588287120b
Branch	HEAD
BuildUser	root@d020f7085e1e
BuildDate	20221109-14:30:50
GoVersion	go1.19.3

This issue causes our WALs logs to keep growing (> 200G) and when we restart the server, the replay WALs cause OOM, and the system kills the Prometheus process.  Finally, to recover the system, I removed all WALs logs, and then the error is gone. Now I keep watching the status.

Looks like this should have happened pre 2.40.x as well (the error existed before). Testing out what changed in between. We should not be erroring out here ideally.

Good to know that this existed before as well, and we don't need to error here. Earlier it didn't give a problem because of a bug, which 2.40.x fixed.
See https://github.com/prometheus/prometheus/blob/release-2.39/tsdb/querier.go#L694
It uses errors.Wrap on a nil error, which resulted in a nil error at the end. Tt actually had to create a new error, which 2.40.x fixes. Will cut a 2.40.3 with the fix.

This was fixed by https://github.com/prometheus/prometheus/pull/11623


Add capacity hint when deep copying maps

cc @mlsteele

do i need to bump a version number or something?

yes, and you need to copmile

Ok, does that look ok?

should we do the same for slices?

yes, that append is suboptimal...

Ok, still looks good?

will this do the right thing for empty or null slices?

there's a nil check already, make with 0 should be good.
https://play.golang.org/p/hs9WvqV3cDT


tls: Account for a change in BoringSSL verify depth semantics

Commit Message:
Prior to OpenSSL 1.1.0, the certificate depth limit in OpenSSL omitted the leaf but included the trust anchor. That is, if your chain was Leaf, Intermediate, Root, any depth limit of 2 or more allowed the certificate.
OpenSSL 1.1.0 included d9b8b89bec4480de3a10bdaf9425db371c19145b, which was described as a cleanup change to X509_verify_cert. However, this change the semantics of the depth limit to omit both the leaf and trust anchor. So the example above was accepted also at depth limit 1. This is also why common.proto had a comment about different semantics between the libraries.
BoringSSL originally forked a little before 1.0.2, so it had the older OpenSSL behavior. Now that the new behavior has been in OpenSSL upstream for a while, BoringSSL plans to match the new behavior in https://boringssl-review.googlesource.com/c/boringssl/+/64707/
This change makes Envoy compatible with BoringSSLs before and after that change. When BORINGSSL_API_VERSION is new enough, we adjust the value before passing it in, to preserve the original semantics. I'm assuming here that Envoy would prefer to maintain its existing semantics, rather than change the test expectation. I've also removed the comment about backend-specific behavior difference. Supposing Envoy prefers to maintain existing semantics, any OpenSSL port of Envoy should similarly adjust the value on OpenSSL 1.1.0 and up.
Along the way, fix an overflow. maxVerifyDepth is a uint32_t, but the OpenSSL API takes an int. When we exceed INT_MAX, saturate the cast.
Additional Description:
Risk Level: Low (PR is intended to prevent a behavior change)
Testing: Ran default_validator_test on old and new BoringSSL
Docs Changes: Clarified docs for max_verify_depth
Release Notes: N/A
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Can you please merge head again?

Done.

/retest


twister: tests: Unexpected files left after Twister tests

Describe the bug
Unexpected files left in the source tree after Twister's  test execution (./scripts/tests/twister and ./scripts/tests/twister_bllackbox)
To Reproduce
Steps to reproduce the behavior:
$ pytest $ZEPHYR_BASE/scripts/tests/twister
$ git status

$ pytest $ZEPHYR_BASE/scripts/tests/twister_blackbox
$ git status

Expected behavior
no extra files left in the source tree after the test execution
Impact
confusion, working commit possible poisoning with unexpected files.
Logs and console output
$pytest $ZEPHYR_BASE/scripts/tests/twister
...
========== 537 passed, 273 warnings in 30.36s =
$ git status
On branch main
Your branch is up to date with 'upstream/main'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        mock_platform/
        scripts/tests/twister/test_data/demo_board_2/

$pytest $ZEPHYR_BASE/scripts/tests/twister_blackbox
==== 22 passed, 4 warnings in 186.53s (0:03:06) ==
$ git status
On branch main
Your branch is up to date with 'upstream/main'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        OUT_DIR/
        TEST_DIR/

Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.16.4
main branch 8c83185f6e3d5b41e

@LukaszMrugala, one more observation in regard of scripts/tests/twister_blackbox execution:
it creates quite a few (62) ./twister-out.* directories which also remains, what's the value to always keep these ?

@LukaszMrugala, one more observation in regard of scripts/tests/twister_blackbox execution: it creates quite a few (62) ./twister-out.* directories which also remains, what's the value to always keep these ?

That's true. twister-out folders were deemed as acceptable to leave, as they hold the exact results of every test. Their removal by the user is also easy (rm -rf ./twister-out*), so I deemed it safe to leave.
On the other hand, as the number of twister-outs rises, their usefulness wanes. Twister only allows for 100 such folders to exist, so in a short order they won't hold the detailed results anymore.
I will look at some clearing fixture.

just consider someone who prepares PR already having few own twister-out directories, so it becomes a problem to decide what to delete manually.

I will look at some clearing fixture.

it might be a pytest parameter to keep logs (with default - don't keep)


rev helm to latest v1.5 release

LGTM

incomplete

When upgrading the version of helm for 1.5, K8S_HELM_VERSION_1_5 should also be updated.

fair, I'll issue a new PR.


Fix bug when calling tests with --ansi and --no-ansi parameters

Fix for https://github.com/laravel/framework/issues/45340

Can you test this against PHPUnit and Pest please?

I can't run tests in the project itself due to errors:

I can't install:

I can't start tests:


.so libs are reported missing on the CNS disk mounted at /opt

This may be a GCP or the OS issue. Out of the blue, a job fails with the message logged from slurm.pl-produced batch script, like
     38 + nnet3-chain-train --use-gpu=yes [...snip]
     39 nnet3-chain-train: error while loading shared libraries: libkaldi-nnet3.so: cannot open shared object file: No such file or directory
     40 + ret=127

The files are definitely there. To investigate, we need to catch a possible kernel message. And since the nodes are transient, we do not get to see them.

[x] Implement a quick workaround by configuring Slurm to restart jobs with exit code 127. This is a bit risky, as a genuine code 127 will catch the job in an unbounded restart loop. (P0)
[x] This could be remedied in the batch itself, since a requeued job has SLURM_RESTART_COUNT integer set in its environment. A sensible hardcoded value like 4 or 5 will do. (P0)
[ ] Implement log forwarding from compute nodes (P2)

/cc @jtrmal

eb01e06  addresses the P1 part (first 2 checkboxes).


Federico: Watch activate-iolo.online

Federico requests the watch of the watch_keyword activate-iolo\.online. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

activate-iolo\.online has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Mac address not found enter manually Router Ip not found enter manually

New updated Debian On Windows 10
Everything installed and up to date
Sudo python3 kickthemout.py
Default Gateway MAC Address could not be obtained. Please enter MAC manually.
Enter your gateway's MAC Address (MM:MM:MM:SS:SS:SS): ScanniNg your^Canning your neTwork, hang on... --
Please help

Hi there. Currently, KickThemOut is not compatible with the Windows OS. Thanks!

please I need an answer so0n!!!

Hi there. Please provide us with the information asked for in the issue template so that we have sufficient information to help you out. Thanks.

Closing due to inactivity. Feel free to re-open this issue if you are still experiencing a problem. Thanks.


Hive 1160 css

Description
JIRA ticket
HIVE-1160
Summary
This PR adds new branding to login and consent forms.

I tried updating facet to
body {
min-height: 100%;

and it doesn't work on chrome https://stackoverflow.com/questions/8468066/child-inside-parent-with-min-height-100-not-inheriting-height
looks like we have to go with setting both to height: 100%


Possible error in Get started docs

I couldn't compile my project with the basic steps in the documentation.
I had to add this in my settings.gradle since the link command was not doing it:
include ':lottie-react-native'
project(':lottie-react-native').projectDir = new File(rootProject.projectDir, '../node_modules/lottie-react-native/lib/android')

and this in my build.gradle:
  //compile 'com.airbnb.android:lottie:2.2.5'
    compile project(':lottie-react-native')

Notice that the first commented line is the one in the docs that to me was not working. Second one worked.
Just leaving it here in case there is something I am missing.
Thanks

@sfratini Thanks. I will update airbnb.io/lottie to reflect this.


update readme

The readme file is outdated and needs to be updated to show the new class name and how to use the encrypt methods for AES

worked on this, not sure if we should leave this open. unless we want to create a robust readme

I will on the readMe Issue


Fix typo in Readme

Btw. Nice way to wrap service objects alike as model methods :)

Ah nice, thank you for the PR and the kind words 😊


Add pause and resume to EmitterController

This would support low bandwidth situations when emitting events should be paused.

Related discussion on iOS issue: https://github.com/snowplow/snowplow-objc-tracker/issues/672


Long strings exceed maximum line length

The output of this file contains lines that are longer than the maximum line length of 80.  In the output file, the lines in question are 40 and 48.
As a sidenote: The way these particular long strings are displayed in said file seems more pleasing to my eye than letting them start right after description:. I can see that this is a matter of taste though.

Thanks for filing an issue. Here's a reduced test case so it's easier to talk about :)
    ({
      totalCount: {
        type: GraphQLInt,
        resolve: conn => conn.totalCount,
        description:
`A count of the total number of objects in this connection, ignoring pagination.
This allows a client to fetch the first five objects by passing "5" as the
argument to "first", then fetch the total count so it could display "5 of 83",
for example.`
      }
    });

turns into
({
  totalCount: {
    type: GraphQLInt,
    resolve: conn => conn.totalCount,
    description: (
      `A count of the total number of objects in this connection, ignoring pagination.
This allows a client to fetch the first five objects by passing "5" as the
argument to "first", then fetch the total count so it could display "5 of 83",
for example.`
    )
  }
});

https://jlongster.github.io/prettier/#{"content"%3A"    ({\n      totalCount%3A {\n        type%3A GraphQLInt%2C\n        resolve%3A conn %3D> conn.totalCount%2C\n        description%3A\n`A count of the total number of objects in this connection%2C ignoring pagination.\nThis allows a client to fetch the first five objects by passing \"5\" as the\nargument to \"first\"%2C then fetch the total count so it could display \"5 of 83\"%2C\nfor example.`\n      }\n    })%3B\n"%2C"options"%3A{"printWidth"%3A80%2C"tabWidth"%3A2%2C"singleQuote"%3Afalse%2C"trailingComma"%3Afalse%2C"bracketSpacing"%3Atrue%2C"doc"%3Afalse}}

What is the expected output here? :)

We are unlikely going to try to try and accommodate long strings. I'm closing this issue in order to keep the issue list actionable.


Post network verifier run failure at the end of investigation

What?
Fixes a small bug where the note would get posted twice. Changed to follow the idiom of adding everything to the note and posting once done with all checks.

/retest

/lgtm


Retail Channel not Commerce Channel

The operating unit type is still labelled in D365 as a 'Retail Channel'. Please update (either docs or in the application!) to ensure consistency.

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

ID: 0bc0b18d-10eb-b4f6-5b38-3668e737c726
Version Independent ID: 767da595-baa2-6216-85e0-ed44267117a8
Content: Organizations and organizational hierarchies overview - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/fin-ops/organization-administration/organizations-organizational-hierarchies.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @sericks007
Microsoft Alias: sericks

Also, 'Online Marketplace' no longer exists in Commerce as a type of retail channel. It should really mention call center as the other channel type.

@andyNewnham, thanks for the feedback. @sericks007, please review.

Thanks for noticing this!  I've updated the content, which will be published later today.


[Bug]: Tesseract not installed

Describe the bug
Running tests fails on a machine that does not already have tesseract installed.
To Reproduce
https://github.com/MLDSAI/OpenAdapt/issues/295#issuecomment-1602616550

the blocker for Windows is that there is on 1 line installation for Windows.
We have to download the binary file and then manual add the dir. to PATH variable.

I am encountering this strange error for Windows installation

@0dm I have currently implemented installing homebrew in the install script and tesseract

@abrichr The Powershell script is ready to be merged


Editing custom layout with zone touching right/bottom edges, can't resize smaller

•	Trying to make a custom layout, I seem to be able to resize from the right and bottom edges of the custom zone box easily, but the top and left edges won’t resize sometimes.  They show me the cursor that has the right kind of resize but when I click and drag they don’t resize.
[jbo] I haven’t seen this behavior. But based on your description, I’m thinking that my clamping logic may be flawed. So I moved a window to the bottom right corner of the screen and then tried to resize the top left corner. Sure enough, I can resize it bigger, but not smaller. This is my bug. Good find. I’ll add this to my todo list

This was fixed in a previous release.


Update TOC.yml

Minor update to TOC

Docs Build status updates of commit 3d9a8bd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Adding forgit to Utils

Hi,
I am the maintainer of forgit for fish.
Check it out here, and see if you think it's worthy of being "awesome"!
https://github.com/wfxr/forgit
Thanks :)

Definitely worthy of being awesome! But this is not the kind of projects that I like to collect here. I think forgit should be in awesome shell or awesome command line, if it's not already there.
Thanks!

Sorry for the late response. forgit is awesome, but I only want Fish projects here.


Contact Section

I will add contact section at the bottom of the webpage.  So that users can contact admin if they have any issues.
kindly assign me this issue #gssoc24

I can resolve this issue. Please assign me this issue.

I would like to work on this issue by creating a contact us option where i plan on adding links to the social platforms.
Please assign me this issue.

I would like to resolve this issue by adding all the social media pages/links and also add an form where use can write his message and this will received to admins email.
Please assign it to me if you think my solution is beneficial to this problem.
Also you can suggest me what I can more add to this.
Thanks

Hello I am Gargi from IIT Kharagpur! I work with html,css,js,react.js and so on and am eager to learn more. I'd appreciate being assigned this task for GSSoC 24.

assign me this issue #gssoc24

please assign me under gssoc24


AdapativeChoices + CustomChoices + AutoChoices.DISABLED + Enum column -> wrong mix!

(Original issue 16 created by None on 2011-05-26T13:13:50+00:00)
What steps will reproduce the problem?
1. Define the filter header as autoadaptative
2. Set the autoChoices to DISABLED
3. Set the column type to a given enumeration
4. Define specific CustomChoices

What is the expected output? What do you see instead?

The normal output, if all enumeration values are available in the column, would be
to have all those values PLUS the defined custom choices on the editor's combobox

However, only the custom choices, and some of the enumeration values are present (one
enumeration value missing for each given custom choice)

What version of the product are you using? On what operating system?
4.1.4, XP

Please provide any additional information below.
The problem seems to be on line 629 in AdaptiveChoicesHandler, where it is checked
if (maxChoices > 0), instead of checking is autoOptions are disabled.

Original issue reported on code.google.com by luismpena on 2011-05-26 13:13:50

(Original issue 16 last updated on 2015-04-26T00:23:51.723397+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2011-06-07T15:03:11+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-06-07 15:03:11

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2011-10-13T10:08:04+00:00)
Indeed, the same problem happens as well if AutoOptions has value Enum and a column
does not contain an enumeration type.

Original issue reported on code.google.com by coderazzi on 2011-10-13 10:08:04

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2011-11-10T16:19:51+00:00)
Fixed on 4.2.0

Original issue reported on code.google.com by coderazzi on 2011-11-10 16:19:51

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2011-11-14T23:13:13+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-11-14 23:13:13

(Original comment by {u'display_name': u'coderazzi coderazzi', u'account_id': u'557058:790eb198-2ae4-49b4-b920-8d654aa86b3f'} on 2015-04-26T00:23:51.731191+00:00)
Already resolved (previously, before migrating to BitBucket)


Pushing policies to android results in Invalid DeviceIdentifiers found

Description:
I believe this is happening because the IoT server is trying to push the policies to removed phones but I am not 100% sure. Sometimes the policies still get pushed but I've seen cases with application white lists enabled where nothing happened but that might be another issue all together.
Attached logs:
Identifier error.txt
Suggested Labels:
Policy, Android, Error
Affected Product Version:
IoT 3.1.0
OS, DB, other environment details and versions:
Linux OS & Postgresql database
Steps to reproduce:
In policy management change a policy
Apply changes to devices
Check logs for errors

Started happening in IoT3.1.0 update 4 again now too.. Policies are not functioning anymore after a while.


gatsby-plugin-manifest issue

gatsby-plugin-manifest is throwing a Plugin Error

This issue was fixed in one of the PRs. Thanks for your contributions @subodhk01 :)


(maint) bump travis/appveyor lth version to 0.7

CLA signed by all contributors.

@MikaelSmith I've had no luck trying to add the export helper macro to other targets to get this running on Windows. Adding it to just the libfactersrc target resulted in the same error as before, and it doesn't work to add it to libfacter_test because it only works on library targets. Do you have any thoughts?

👍


Orange Pi should also append company number.

As per current functionality we have hard coded company numbers, so the TTGO TCall auto delete these messages after forwarding them to user but we don't need to always update that row by updating code of TTGO TCall so we need to update these numbers using orange pi, what I planned is we should append company messages numbers like we are doing with authentic mobile numbers.

@mhamidjamil I don't think it's a good approach, although this issue must be resolved as soon as possible as you said you accidentally subscribed other message package, so we need to ignore these company messages but OTP like messages must forwarded, but the approach should be updated, I think we should not just update at the bootup time we should do these changing in the SPIFFS so TTGO can be independent of orange pi in this scenario too. So I'm also creating task in TTGO which will now deal with the strings too after that task you should just write down company messages in a txt file orange pi just read these numbers as TTGO to update its SPIFFS and after confirming this update orange pi update its txt file, hope you under stand, feel free to ping me if you have any query regarding this case.


Refactor trace_boundaries and fix documentation errors

Codecov Report

Merging #237 into master will increase coverage by 14.36%.

@@             Coverage Diff             @@
##           master     #237       +/-   ##
===========================================
+ Coverage   46.42%   60.79%   +14.36%
===========================================
  Files           2        8        +6
  Lines          56      352      +296
===========================================
+ Hits           26      214      +188
- Misses         30      138      +108

Impacted Files
Coverage Δ

web_client/js/app.js
3.44% <ø> (ø)

web_client/js/views/annotationSelectorWidget.js
71.42% <ø> (ø)

web_client/js/views/visualization.js
80.48% <ø> (ø)

web_client/js/views/body.js
77.27% <ø> (ø)

web_client/js/views/header.js
4.54% <ø> (ø)

web_client/js/0init.js
57.89% <ø> (ø)

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 6413085...ba865de. Read the comment docs.


fix: Filter extracted merchant name before display

Description

This fixes an issue where it sets the extracted vendor to vendor field even when its not present in the company's vendor list
Added a method for the fix which loops through the vendor list and checks if the extracted merchant is present

Clickup
https://app.clickup.com/t/86cwdt5t1
Code Coverage
Please add code coverage here
UI Preview
Please add screenshots for UI changes

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Introduced enhanced vendor filtering functionality when adding or editing expenses, improving the accuracy of vendor assignment.
Automatically updates the vendor information based on available transaction data, ensuring users have the correct vendor displayed.

@suyashpatil78 @arjunaj5 have extracted the vendor options into separate array which eliminated the method to rely and return on observables


Documenting how to get Bash completion in Krew mode

Did not seem to be considered in #170.

@superbrothers
This PR looks similar to #193.
Could you please tell me how to merge it? :bow:

@rkmathi @superbrothers #241 probably will fix the problem that we cannot merge PRs that only change README.md.

@tksm Thank you!! 😄


The route's origin receives the request even if filter says not to

Describe the bug
Spring Cloud Gateway 2.2.2 (also tried 2.1.5).
I was trying to implement a custom filter that will prevent hitting the origin if condition is failing. It is very similar to RequestSizeGatewayFilterFactory from SCG.
Apparently, even if I do return exchange.getResponse().setComplete(), the origin still receives the request (and, what's more important - the body).
Sample

Runnable (in SCG's test scope) sample that reproduces the issue:
@RunWith(SpringRunner.class)
@SpringBootTest(webEnvironment = RANDOM_PORT)
@DirtiesContext
public class GatewayFilterTest extends BaseWebClientTests {

	static AtomicLong requests = new AtomicLong(0);

	private static HttpServer httpServer;

	@BeforeClass
	public static void beforeClass() throws Exception {
		httpServer = HttpServer.create(new InetSocketAddress("localhost", 0), 0);
		httpServer.createContext("/", exchange -> {
			requests.incrementAndGet();
			exchange.sendResponseHeaders(400, 0);
			exchange.close();
		});
		httpServer.start();
	}

	@After
	public void tearDown() {
		httpServer.stop(0);
	}

	@Test
	public void shouldNotCallOrigin() {
		assertThat(requests).hasValue(0);
		testClient.post().uri("/post")
				.exchange()
				.expectStatus().isEqualTo(HttpStatus.PAYLOAD_TOO_LARGE);
		assertThat(requests).hasValue(0);
	}

	@EnableAutoConfiguration
	@SpringBootConfiguration
	@Import(DefaultTestConfig.class)
	public static class TestConfig {

		@Bean
		public RouteLocator testRouteLocator(RouteLocatorBuilder builder) {
			return builder.routes().route("skip_origin", r -> r
					.path("/post")
					.uri(String.format(
							"http://%s:%s/post",
							httpServer.getAddress().getHostString(),
							httpServer.getAddress().getPort()
					))
					.filter((exchange, chain) -> {
						ServerHttpResponse response = exchange.getResponse();
						response.setStatusCode(HttpStatus.PAYLOAD_TOO_LARGE);
						return response.setComplete();
					})
			).build();
		}
	}
}

Workaround:
Downgrade to 2.1.5 and use the following:
r.method(HttpMethod.PUT)
.filters(f -> f.filter((exchange, chain) -> {
    var response = exchange.getResponse();
    response.setStatusCode(HttpStatus.PAYLOAD_TOO_LARGE);
    return response.setComplete();
}))
.uri("https://bsideup.requestcatcher.com");

Okay, it was an issue on my end.
In both 2.1 and 2.2, one should use filters before uri to intercept the call. Need to get some sleep 😅


Browse All bugs from bugbash

"Runs" column: total number of runs is filled up to 4 digits with zeros.

Some test output in the list of objects when hovering over "X more objects" in the "Annotated Objects" column

Not sure if bug: Clicking on the blue "+X more" next to the author list opens the Browse Dataset page. I would have expected to expand the list of authors instead.
https://github.com/chanzuckerberg/cryoet-data-portal/assets/6641113/c1682963-5538-4609-a31d-feba719c147d

This isn't working -- it needs to compare the range to the tiltseries.tilt_range field

The EMPIAR ID filter seems not to work on certain entries that it should display, and causes other run to be displayed.
https://github.com/chanzuckerberg/cryoet-data-portal/assets/6641113/a15985c4-1596-480e-bd74-8ca2f0103888

Dataset IDs can be searched with glob patterns, but EMPIAR IDs cannot.
https://github.com/chanzuckerberg/cryoet-data-portal/assets/6641113/9debf11d-626b-4e35-a850-24d7a4e69077

EMDB ID-Filter does not work at all. Neither with purely numeric input, nor with EMD- prepended, nor with just EMD.
https://github.com/chanzuckerberg/cryoet-data-portal/assets/6641113/855cbe6f-281d-4086-adc3-a16793cd1c68

Reconstruction method filters have one option in all-caps and one in mixed capitalization. This field has controlled vocabulary according to the schema: "Weighted backprojection" "SART" "SIRT"

Object name Overflow tooltip seems to have placeholder object names

I think there are missing spaces in the object shape type filter list:

Segmentation Mask
Oriented Point

Tilt Range filter "Apply" button should be disabled until user adds/makes a change to either value

There are 2 Gatan options in camera manufacturer filter, one title case and one all caps

May help clarify how search works to change default text to "Search Dataset Name"?

Runs tooltip doesnt align with run column if browser window is narrower. Would expect it to change alignment if it needs more space.

"Runs" column: total number of runs is filled up to 4 digits with zeros.

created ticket #265 to capture this

The far right column is pushed off the edge of the screen (probably because the description of the datset is so long)

I wasn't able to recreate this but will keep this in mind in case it shows up again. will hold off on creating a new bug for now

Dataset table does not self-adjust to screen size and wiggles around.
Screen.Recording.2023-12-06.at.1.52.02.PM.mov

we weren't able to get around to auto resizing for this phase so instead of creating a bug, i'll just log this as something to consider for H1 work

 The "no results found" message looks like it needs more spacing between it and the horizontal line above it

created #267 to capture this bug

Some test output ("A really long name ...") in the list of objects when hovering over "X more objects" in the "Annotated Objects" column

this has been logged as an existing bug

The number of displayed objects is larger than 12, contrary to the label.

created #268 to reflect this bug

Not sure if bug: Clicking on the blue "+X more" next to the author list opens the Browse Dataset page. I would have expected to expand the list of authors instead.
Screen.Recording.2023-12-06.at.2.09.57.PM.mov

** this is expected behavior so that we can keep each row a consistent height without having to account for a really long list of authors

The tilt range filter isn't working -- it needs to compare the range to the tiltseries.tilt_range field

Created #269 to reflect this bug

The EMPIAR ID filter seems not to work on certain entries that it should display, and causes other run to be displayed.
Screen.Recording.2023-12-06.at.2.16.23.PM.mov

Created #270 to reflect this bug

"Organism Name" search box behaves strangely when scrolling on page.
Screen.Recording.2023-12-06.at.2.31.57.PM.mov

I'm not certain this is a bug but will tag @Janeece to confirm

"Organism Name" search box behaves strangely when scrolling on page.
Screen.Recording.2023-12-06.at.2.31.57.PM.mov

I'm not certain this is a bug but will tag @Janeece to confirm

@junxini  I don't think this is a bug, this menu should show to the side where it has space to display (as shown).
There might be some changes we'd want to consider making to how it displays/behaves while scrolling (ex: should it flip or stay put after opening?), but I would say thats a potential design refinement.

Filtering "Yes" yields two datasets (that don't have fiducials) and filtering "No" yields 4 datasets. 4+2 != 10 (total number of datasets).

Created ticket #290 to capture this

Reconstruction method filters have one option in all-caps and one in mixed capitalization. This field has controlled vocabulary according to the schema: "Weighted backprojection" "SART" "SIRT"

Created ticket #304 to capture this

Tilt Range filter "Apply" button should be disabled until user adds/makes a change to either value

covered in ticket #208

There are 2 Gatan options in camera manufacturer filter, one title case and one all caps

this was fixed

Runs tooltip doesnt align with run column if browser window is narrower. Would expect it to change alignment if it needs more space.

@Janeece should this be part of the responsive behavior work instead of logging it as a one off bug? that way this behavior can be accounted for across all tooltips

All bugs listed here have been accounted for, either as new tickets or cross-referenced to existing bugs created by Laura. See inline responses above for more details

Runs tooltip doesnt align with run column if browser window is narrower. Would expect it to change alignment if it needs more space.

@Janeece should this be part of the responsive behavior work instead of logging it as a one off bug? that way this behavior can be accounted for across all tooltips

@junxini we could be group this refinement into the responsive behavior scope of work


Suggestion for README.md

Add periods to make the use of symbols more uniform.
“real time” and "server to client" missing a hyphen, so need to add some hyphens.
"Application" seems missing a determiner before it.

ok


Exception blocking Java functions from starting in 1.7.1

This exception is preventing Java functions from starting in Greengrass 1.7.1 today:
Exception in thread "main" java.lang.NoClassDefFoundError: com/amazonaws/services/lambda/runtime/LambdaLogger

Fixed in #50


Unexpected behaviour when using replace and moving cursor

Hello! I'm very new to react and family, so please bear with me if this is a simple mistake on my part.
I'm trying to implement simple Markdown formatting with the Controlled component and I'm having issues with the cursor ending up in an unexpected place when performing a replace + setCursor operation.
I have an example application below which has a "bold" button that can be pressed. The expectation is that when pressed it will enter "****" and put the cursor in the middle of them. When the cursor is at the end of a line, the cursor ends up at the end of the line after insertion. If the cursor is in the middle of a line, it ends up 2 characters over from the end of the insertion. If I change my offset to -2, it moves backwards from the start position.
This exact approach seems to work fine with react-codemirror.
App.js
import './App.css';

import React, { Component } from 'react';
import 'codemirror/lib/codemirror.css';
import { Controlled as CodeMirror } from 'react-codemirror2';

class App extends Component {
  constructor(props) {
    super(props);
    this.cm = React.createRef();
    this.state = { value: "starting\n" }
  }
  bold = () => {
    if (!this.cm.current) return;
    const cm = this.cm.current.editor;
    cm.focus();
    const doc = cm.getDoc();
    const c = doc.getCursor();
    doc.replaceRange('****', { line: c.line, ch: c.ch });
    doc.setCursor({ line: c.line, ch: c.ch + 2 });
  }
  render() {
    return (
      <div className="App">
        <CodeMirror
          value={this.state.value}
          onBeforeChange={(editor, data, value, next) => {
            this.setState({value});
          }}
          onChange={(editor, data, value) => {
          }}
          ref={this.cm}
        />
        <button onClick={this.bold}>
          bold
        </button>
      </div>
    );
  }
}

export default App;

Thanks

@mjessome hm, seems as if you're wrestling the instance a bit with the calls to doc.replaceRange and doc.setCursor. Why not manage range and cursor through the props?. This would be the "react way" to do this.
Perhaps modifying the value, finding the cursor mark, then setting all three cursor, selection and value within the callback of your bold button click? I'd be happy to help you further but I suspect that to be a good starting point. By the way I may end up closing this out as it seems to be a troubleshooting issue and not a defect with the lib itself. In which case asking a well-composed question on stackoverflow.com will likely yield fast and quality results


feat: Update Terraform minimum supported version to v0.13.1

Description

Update Terraform minimum supported version to v0.13.1
Fix cross-account TGW sharing via RAM (see https://github.com/terraform-aws-modules/terraform-aws-transit-gateway/commit/120533a498639d79b32586b25edce6151694b9d9)
Re-order tag values to allow resource specific tags to override Name tag used (this change should be a no-op for users)
Formatting and code organization updates to align with current standards
Replace use of lookup() with try()
Fix examples to use VPC and subnets created by the examples rather than creating the VPCs but relying on the users account default VPC
Replace deprecated aws_subnet_ids data source in examples with aws_subnets
Update provider required version schema to use current format
Remove example data sources and use module outputs directly

Motivation and Context

To align with current standards
Closes #67
Relates to #46
Relates to #47
Relates to #56
Relates to #61

Breaking Changes

No

How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

Validated before/after of examples/complete
Unable to validate examples/multi-account due to issue stated in #67 (exists prior to this change so unable to apply)

This PR is included in version 2.6.0 :tada:

Good to see that you have some time for this module. I made it a long time ago and used it just on one project. Thanks!


Freezing in v1.7.1

First, thank you so very much for AsciidocFX. I am writing a book for Manning Publications, and when I asked what .adoc editor I should use, my editor told me that she uses AsciidocFX. It's a fabulous solution.
I am using this version of Windows:

And I get freezes when AsciidocFX is "idle" for a short period of time. If I'm continuously editing, it's no problem, but if I go do something and come back (I haven't timed it but it feels like less than 5 minutes), I cannot type or open any documents from the side pane. I may be able to scroll, but after selecting a paragraph where I'd write, it does not accept my keyboard input.
I have to close AsciidocFX, which works with no resistance. Then I reopen and have no problems until I'm idle again.
I reverted to 1.7.0 and will see if that helps.

Can confirm 1.7.0 still worked after 20 minutes of being idle 👍

New release is available https://github.com/asciidocfx/AsciidocFX/releases/tag/v1.7.4
Please check if that fix your issue.

I’m closing the issue, let me know if I can help further

Thank you!


Add borderCapStyle and borderJoinStyle options to BoxAnnotation

Add borderCapStyle and borderJoinStyle options to BoxAnnotation
Move setBorderStyle function to the helpers, from line annotation, in order to be used by both BoxAnnotation and LineAnnotation label

Maybe we should add exports to package json, to be explicit these are internal.

At the moment, I'd suggest to leave it as is. I think this is part of the discussion  #480.
Furthermore, I'd like to add another discussion soon, based on #431, where we could implement custom annotation types. In that case, that method (and all helpers methods) should be exported as public methods.
Make sense?


[!] Unable to find a specification for BSStackView when installing pod

Update pod repo by running this command in terminal.
pod repo update
It solves the problem.


feat: Detect interpreter in shutdown state on thread spawn

This detects if the interpreter is already in shutdown state and no longer spawns a background thread.
Fixes #2299

@mitsuhiko I will take over and also add checks for the other places we start threads. Thanks for the contribution!

@sl0thentr0py I will merge this, because it fixes the one place where the problem arises that is mentioned in https://github.com/getsentry/sentry-python/issues/2299
I will create another PR for the other places we start threads.

Oh, just saw Ivana already fixed this in our python312 branch.
Ok wurscht, I will make it work.


(feat) add support for attestations stored as image manifest

Description of the PR
Currently, guac only works with certain OCI images. Build tools store attestations in a variety of ways as of now and it might take them a while to adopt to using referrers. As of now, Docker stores the attestation in an image manifest and it would be wonderful for GUAC to support that as docker is a popular tool for building containers.  I'm not entirely sure if this is the right way to implement this,  I'd appreciate any feedback to make this change a reality.
PR Checklist

[x] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

Hi @RealHarshThakur thanks for the PR! This is actually something I created an issue on #1370 so it's great to see somebody picking up this work. There is a bit of overlap here with #1449 which I believe will also cover collection of these types of attestations. But, this change also will move away from regclient to google/go-containerregistry as the library to access OCI packages. The community is discussing this change in #1449. Don't take it as a reason to stop further work on this - just wanted to bring you up to speed on this topic :)

My bad 😓 I had it on my todo list before the issue was created and just found this weekend to hack on it.
From skimming the other PR, I think one way to move forward would be for me to refactor this PR to focus on processor and parser. Essentially, all changes to introduce DocumentITE6SPDX.

@RealHarshThakur marking this as not stale as it is an important topic to keep working on.

sorry for delay, I'll try to make some time this month

Adding comment to ensure this does not close.


Bug: Caching response throws AttributeError on GET: Can't pickle local object 'DependencyCleanupGroup.wrap_asgi..wrapped'

Describe the bug
When caching the response of a get request for a Template, I get the following exception which throws a server 500 error:
AttributeError on GET:  Can't pickle local object 'DependencyCleanupGroup.wrap_asgi.<locals>.wrapped'
To Reproduce
Here is a reproducible example:
import uvicorn

from starlite import Starlite, get, Request, State, Template, Controller, TemplateConfig
from starlite.template.jinja import JinjaTemplateEngine

class FrontEnd(Controller):
    @get("/", cache=60)
    async def serveHomepage(self, request: Request, state: State) -> Template:
        return Template(
            name="homepage.html.j2",
            context={"request": request},
        )

    @get("/hello-world", cache=60)
    async def hello_world(self) -> dict[str, str]:
        """Handler function that returns a greeting dictionary."""
        return {"hello": "world"}

app = Starlite(
    route_handlers=[FrontEnd],
    template_config=TemplateConfig(
        directory="src/templates", engine=JinjaTemplateEngine
    ),
    debug=True,
)

if __name__ == "__main__":
    uvicorn.run(app, host="<IP_ADDRESS>", port=8000)

You can use this for the Jinja template:

Homepage Header

Additional context
If you remove the cache value from the Template route (or set it to zero), the get requests are served as expected.
In the example code, I also included a route for a non-template request, /hello-world, which does not throw an error.  So the problem seems to lie with Template responses, or some subset of responses.

Attaching traceback:

Nope, I’ve been on 3.11 since the day it came out

@Goldziher I think it has nothing to do with the Python version. The offending code is the fact that now response is wrapped into DependencyGroup to be able to do cleanup - here
Obviously this can not be cached and before saving response to cache we need to unwrap it first.

@jtraub You're right. DependencyCleaupGroup essentially creates an ASGI middleware here.
The reasoning so we don't have to pass the cleanup group all the way up to where the response is returned, which we need to we can consistently call the cleanup immediately before the response is returned.  I'm thinking of either making a transparent wrapper so it can be unwrapped if needed, or forego the wrapping and simply accept that we have to pass it around a bit. WDYT?

We can do both I think. The wrapper can be made into an ASGI callable that can be pickled.

After thinking about this, the proper way to do this seems to be calling the cleanup before the response is cached. This would also mean we can omit passing the cleanup group around.

@ThinksFast I created a fix for this in #902. Can you confirm this works for you?

Confirmed, this fixed my issue, thanks!


Creating a DACI or RACI for onboarding processes

Problem statement
In order to see improvements to our onboarding process, we need to know what roles team members are playing in consistently implementing processes and guidelines.
What you need to know

Onboarding Journey Improvement Synthesis

Acceptance criteria

[ ] Relevant stakeholders gathered and aligned on roles for maintaining and implementing onboarding processes

Blocked for this sprint based on some contract lead level DACI things to sort out first

@MikaelahD13 Contract leads need to finalize some things we thought were ready, and so we won't have the full meeting until next sprint. So I had moved it to blocked, but moved it briefly out because Chris pointed out that we can still have a brief prep discussion with our team async or in a 16th minute. But it will probably go back to blocked until the contract leads can decide what they need to.

We have added our ideas of potential responsibilities.
Blocked until base-level DACI for RS is finalized

Meeting scheduled for tomorrow

@MikaelahD13 @chris-kuryak I realized this AC was actually met last sprint. I think we weren't sure if it needed a follow up meeting at the time, but LaFreda is handling next steps mostly async


Illegal instruction when trying to perform an operation on a mp4 file

I'm getting an error when trying to perform autosub on a mp4 file:
python3 main.py --model "/media/segundohd/app_repo/autosub_deepspeech_mozilla/AutoSub/autosub/deepspeech-0.8.2-models.pbmm" --scorer "/media/segundohd/app_repo/autosub_deepspeech_mozilla/AutoSub/autosub/deepspeech-0.8.2-models.scorer" --file "test.mp4"
The error message:
Illegal instruction (core dumped)
I think my computer is too old. I have an Intel integrated graphics card. Maybe that could be the reason? I'm buying a new video card next month and I'll try to use it again if that's the problem.
If it works, I'll consider making a kivy GUI for it.

Hey
DeepSpeech works fine without a graphics card. One of these two approaches should solve your issue:

What's your processor and RAM? I came across this issue. If the processor is too old, it’s very likely that it doesn’t support AVX/AVX2. Refer this, and this.
Do you have Sox and FFmpeg installed? Sox shouldn't be necessary but try after installing it. For Ubuntu, you can do:
sudo apt-get install -y sox

Just as a sanity check, double-check your argument paths, please.

Yep, unfortunately my processor is too old. I'm buying a new one from China. It'll takes something. I'll have to postpone using your application but I intend to make a Kivy GUI out of this. Thanks a lot.


When CategoryManyToManyField is used in model, makemigrations wont create M2M table

When CategoryManyToManyField() is used, no M2M table is created when manage.py makemigrations is issued.
Versions used:
aldryn-categories==0.1.1
Django==1.7.4
Will provide more specific info, when needed.

Hi @Edke can you install this build and see if it fixes things for you?
pip install https://github.com/aldryn/aldryn-categories/archive/mkoistinen/fix_17_migrations.zip


External Links in InDesign don't work

A document created in InDesign has external links.  They "work" in iBooks (though the target is 404) but not in the Chrome Extension.  The user posted the file for me (1.45 GB!) which I downloaded.  Loaded it into ID and removed all but 2 pages.  Still demonstrated the problem (works in iBooks, not in RCE nor in SDKLauncher-OSX).
Here's the HTML:
Taking the EPUB apart a bit I see that here is the calling HTML

And here is the JS that handles the MouseUp, finds the data-release actions and invokes the browser to go eval the link:
function onMouseUp(element, event) {
var is_touch_device = 'ontouchstart' in document.documentElement;
if (is_touch_device) return;
removeClass(element, '_idGenStateClick');
var actions = element.getAttribute("data-releaseactions");
if(actions) {
eval(actions);
}

And here is the console output:

So it may be that we are doing the right thing?  However, this is export from ID, so either way this is a problem.

The document is highly proprietary so contact me if you want a copy to test.

Can you find and paste the contents of the goToDestination function? This is what is trying to open the link.

Ah, thought I had. Here it is:
function goToDestination(ref) {
window.location.href = ref;
}

Further comment from the user:
"What I found to work was:
Originally I created url link's as buttons in Indesign's Interactive Panel - Buttons and forms.  I created each as a button and assigned it an action:  “Go to URL”.  And that worked before update. The way I fixed it was I went to the linked item on page…. right clicked it and selected “Interactive" and selected “convert to object” causing it to no longer be a button or anything. Then I select my item again and went to “interactive panel” And instead of button choose hyperlink and placed my url there and I have had no problems since."

Unfortunately, I don't see an easy way to fix this. We would have to inject a script into every single book on the possibility it might be authored by ID.
So the script is basically saying "open this external url inside the epub viewer." Seems like bad practice to me.

What would such a script do?  Note that this approach works in iBooks.  Any speculation on how they manage it?

iBooks is a native application that can make its own rules. We're restricted by the Chrome app security model. That security model prevents us from opening external content inside an iframe.
There is a special tag for embedding web content in apps but we lose the ability to talk directly to the iframe which is required by readium.
We could inject a script that overrides goToDestination to call window.open instead. This would open the url in a separate tab or window instead of trying to navigate the epub viewer to an external url.

Ah, OK. Makes sense.  I think that is way over the top, especially as the user found a workaround.  It is also another item to be documented...

I have been using this HTML snippet to test various flavours of hyperlinks:

http://readium.org

same link with @target=_BLANK

<button onclick="Javascript:window.location.href='http://readium.org';">location.href=xxx</button>

<button onclick="Javascript:window.open('http://readium.org');">window.open(xxx)</button>

As Ryan said, window.open() works fine.

Note that this is solved in Readium2 Electron-TypeScript navigator thanks to low-level handling of webview navigation events via the Electron API. Other R2 platforms (e.g. Android-Kotlin, iOS-Swift) probably have to handle this differently.


Spyder 3 hangs on Windows 10 when working in IPython console

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Spyder starts normally. After 2-3 directory navigation commands in the IPython console, it freezes. "Not Responding" appears in the title bar (as well as beside one of the Python processes in Task Manager).
What steps reproduce the problem?

Start Spyder any way.
Enter 2-3 directory navigation commands in the console, e.g.
cd
cd Desktop
ls -> freeze!
(entering math commands does not cause any problem, e.g. 1+2, ...).
OR
Execute a small script calling a function to print using run button, and then try to write into console.
OR
presumably many more particular combinations of working directly in the console.

What is the expected output? What do you see instead?
The console doesn't respond, then the entire Spyder window is shaded white and "Not Responding" appears in the title bar (as well as beside one of the Python processes in Task Manager).
Paste Traceback/Error Below (if applicable)
Nothing appears in the internal console, nothing appears in the Anaconda Prompt when Spyder is started there. Using flag SPYDER_DEBUG=3, nothing unusual appears, the output ends at "Initialized Rope".
 We haven't found a fix for that yet, although things could be better in our next release (4.0.2).


sink(ticdc): add total write bytes counter to sink

What problem does this PR solve?

Issue Number: close #10114
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
storage sink

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/cc grovecai

/retest

/retest


Dead link in README.md

The link indicated in the Configuration section of README.md seems dead, even if copied and pasted:
https://zotplus.github.io/better-bibtex/chrome://zotero/content/preferences/preferences.xul#better-bibtex

OK, the double header is gone, but from what link did you get there?

The link I get is '
http://zotplus.github.io/zotero-better-bibtex/chrome://zotero/content/preferences/preferences.xul#better-bibtex
'
I'm using firefox 42.0 for Ubuntu canonical, with 14.04.
On 19 January 2016 at 16:06, retorquere<EMAIL_ADDRESS>wrote:

OK, the double header is gone, but from what link did you get there?
—
Reply to this email directly or view it on GitHub
https://github.com/ZotPlus/zotero-better-bibtex/issues/436#issuecomment-172987089
.

But where is this link?

You mean the link under the "configuration" header in the README? That link is not clickable.

You can get also there by pasting this link in Firefox. Clicking the link won't work, sorry.

Yes, I'm copying this link with a right click (copy link location)) and
pasting it. Should the user do something else?
On 19 January 2016 at 16:21, retorquere<EMAIL_ADDRESS>wrote:

You mean the link under the "configuration" header in the README? That
link is not clickable.
You can get also there by pasting this link in Firefox. Clicking the link
won't work, sorry.
—
Reply to this email directly or view it on GitHub
https://github.com/ZotPlus/zotero-better-bibtex/issues/436#issuecomment-172990533
.

Ah, I see the problem now! Hmm, Jekyll doesn't seem to want to render these links anymore. I've replaced that description.

Ok, thanks @retorquere for the quick response.
On 19 January 2016 at 16:32, retorquere<EMAIL_ADDRESS>wrote:

Ah, I see the problem now! Hmm, Jekyll doesn't seem to want to render
these links anymore. I've replaced that description.
—
Reply to this email directly or view it on GitHub
https://github.com/ZotPlus/zotero-better-bibtex/issues/436#issuecomment-172993930
.


Delete infobody maxWidth

https://github.com/kadirahq/react-storybook-addon-info/pull/31#issuecomment-229544024
Delete maxWidth as suggested by @mnmtanish.

Published v3.0.9


Color callbacks cause errors when using the theme() function in CSS

I was so darn happy that you had added a way to add custom property color support with color callback functions, but after reconfiguring all my values, I now get errors whenever the color values are used with the theme() function:
.no-touch input + div:hover:before {
  border-color: theme('colors.gray.400');
}

In my case, I will just replace these theme values with my custom properties directly, but just wanted to let you know about this side effect.

It turns out that it also causes build error. I'll have temporary disable these color callbacks for now :(


Pinning a tab bug

Version: 1.85.0
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows x64 10.0.22621
( I am using a Python interpter )
Steps to Reproduce:

Have at least 1 tab/file open
Pin the tab ( at the top )
Issue: The run button disappears

It's a very very minor bug and simply you just open another file to have it reappear.

Thanks for reporting this! It lead to fixing this and other bugs we had :)

Thanks for reporting this! It lead to fixing this and other bugs we had :)

I am glad to hear!


Miscellaneous fixes

RamaprtEngine.java - Allow RSA15 (which the latest update of wss4j had now defaulted to not supported)
TestCBHandler.java - USERNAME_TOKEN_UNKNOWN no longer available so USERNAME_TOKEN was used, should have been WSPasswordCallback.UNKNOWN
AsymmetricBindingBuilder.java - Removed unnecessary key addition (in recently added code - not present in earlier releases) that was causing issues

Thanks a lot for debugging some of the guess work I had when upgrading these dependencies - good job!


Remove Stackdriver exporter

The Stackdriver exporter has been moved to a different repo:
https://github.com/census-ecosystem/opencensus-go-exporter-stackdriver
New import path: contrib.go.opencensus.io/exporter/stackdriver
Fixes: #789

Could we gradually sunset this change say in a couple of weeks? I ask because OpenCensus has gotten a bunch of attention lately, and this change is going to break a bunch of apps that I and others have freshly pushed out to encourage people to check it out, but also as quick starts. A gradual sunset gives me and others time to go update our source code, say in 2 or 3 weeks? I am currently so busy to go update all my apps instantly.

ok, I will leave the PR open and merge in two weeks.

Friendly ping - what's the timeline for this?

Can we remove this now? goimports automatically picks up this package and confuses the user.

Please go ahead @ramonza, sorry and thanks for the patient wait everyone!

Please revert  "Remove Stackdriver exporter" as it breaks google.golang.org/api/transport/http and thus a slew of cloud.google.com packages.

Thanks!


bundles: addition of cleancssurl filter

Signed-off-by: Harris Tzovanakis<EMAIL_ADDRESS>

Closed since we're moving away from webassets


Proposal: Throwable responses

Would you be interested in a feature like https://github.com/laravel/framework/pull/13473?
Currently, throwing a response requires the following. As more, SlimException isn't really appropriate.
$response = $response->withRedirect($url);
throw new \Slim\Exception\SlimException($request, $response);

Whereas a few additions would allow to do this:
$response->withRedirect($url)->throwResponse();

Note the request isn't used actually.

Exceptions as control flow are generally considered an anti pattern, there is some discussion about it on stackoverflow.

Sure thing, but there are exceptions to this. In this situation, exceptions play really nice.

Otherwise you have to propagate returns all the way up, which sometimes isn't even doable.
These exceptions allows fine-grained and centralized handling (error handling, logging, and much more).
Features like Laravel's terminable middlewares free you from the sequential flow constraint.

As a result, you have a really great application flow that wouldn't be possible without these exceptions.

And exceptions are not not designed for this kind of control flow.
In a website that has thousands of hits an hour, using Exceptions as Control Flow is terrible performance wise, and doesn't scale. At all.
Why encourage bad design?

I can't get behind this idea at all.
Throwable response will not work in Slim.
The idea being that you can immediately terminate from the app strikes me as weird as it's you need to go back through your middleware to complete any actions, throwing response breaks that.

Populus locutus est, closing as there is a clear consensus, no need to pursue.


ADDED - New field to add github link on AddNewPoject Page

Description
Now users can add a GitHub link for a particular project while submitting a new project idea.
Fixes #222
Type of Change:

New Feature

Checklist:

[x] My PR follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

resolve the conflicts @naveen8801

resolve the conflicts @naveen8801

done @kanak22  😄


Mm44851 get involved

Summary
Get involved flow.
If a viewer user wants to get involved in a run, can click the get involved button and:

A: if the channel is accessible will join it directly
B: a message will be sent to the channel otherwise

Copies are not final.
Demo B
https://user-images.githubusercontent.com/4096774/177756194-1e407760-1286-48c9-9d91-a59fe78aebd6.mp4
Demo A
https://user-images.githubusercontent.com/4096774/177756213-9fe18668-b2f1-452d-8b19-a6d6cb36dd47.mp4
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-44851
Checklist

[ ] Telemetry updated
[X] Gated by experimental feature flag
[x] Unit tests updated

cc @justinegeffen , here we have the efforts for the get involved feature. Feel free to suggest awesome copies right here.

This looks great! I left a bunch of comments, but mostly nits and typos. I have a general UX comment, though: Should we also show a confirmation modal when the user will directly join the channel? Something like "Do you want to be added to this run's channel, ~channelname?" People may not be aware that "Get involved" means joining a channel.

I've mixed feelings about that. Modals are educative but also stop the flow. Since the user could leave the channel I would start like this unless we hear about tons of accidental channel join. Additionally, the modal to confirm the request also has a motivation to avoid flooding the channel (also could be done through a debounce tho).

Also, future improvements for v2: Wen someone requests to get involved to a private channel and a message is sent to the channel, we may render a custom post with an Invite button that automatically invites the member to the channel.

Couldn't agree more. Nice topic to revisit after checking usage metrics.

@agarciamontoro I fixed the flow problem that @abhijit-singh noticed in https://github.com/mattermost/mattermost-plugin-playbooks/pull/1295/commits/c796033f890c8d9f9fb259314e58b5ab2d8cb8f1 . However, please take a look because it seems a bit hacky to me.
details
the useChannel hook is built with useThing hook, which tries to get data from redux before doing a new fetch. If the data lacked some fields, we don't ask again for the full data.
The run we sent from server to webapp after Run creation was not the same as the Run we extracted from DB.
My change: I forced the participantIds in run-creation to respect all Role-checks in the webapp side.
I'm wondering if it's not better to force a complete run retrieval from DB, instead of hacking tons of things.

Thanks, @trilopin! The flows look great! I just found 1 functional issue:

When I start a run, and then navigate to the run detail page, I still see the Get involved button. This should not be the case because I'm already a member and owner of that run.

    [

        ![2022-07-12 12 10 36](https://user-images.githubusercontent.com/14369607/178426375-bd9137a7-3b4f-45bc-a68c-5ce79dafc068.gif)

    ](https://user-images.githubusercontent.com/14369607/178426375-bd9137a7-3b4f-45bc-a68c-5ce79dafc068.gif)

      [

      ](https://user-images.githubusercontent.com/14369607/178426375-bd9137a7-3b4f-45bc-a68c-5ce79dafc068.gif)

wow, very good catch

Also a couple of minor visual issues. These are non-blocking and can also be fixed with other smaller UX fixes...

Add 8px space between the get involved button and the info icon.
For the get involved button, font-size should be 12 px.

Thanks a ton @abhijit-singh, I pushed some changes that should have fixed the style and flow issues.


Remove CFHandles from engine_traits

What problem does this PR solve?
Part of https://github.com/tikv/tikv/issues/6402
Problem Summary:
While column families are mostly used by name, there are a few places where engine_traits exposes CFHandles. This patch removes them.
What is changed and how it works?
Related changes
Check List
Release note

No release note

/merge


which ghc version to use?

Which version of ghc is supposed to be used when building kmonad?


[API Bug] Tag Image Display Issue

Raycast version: 1.26.1
Description
When selecting a tag that as for image source a large external image – the tag don't handle properly the size of the image.
Here's the tag before selecting:

And once selected:

Steps To Reproduce

Create a TagPicker with as for icon {source: SOME_LARGE_IMAGE_URL}
Open the form and select the tag

The current behavior

The expected behavior
The image should stay small as it is in the list.

Hey @HenriChabrand! Thanks for the report. Could you please share an URL with this large image, please? Cannot reproduce the bug.

omg, the longest open issue, but I finally had a chance to catch this bug and resolve the issue. @HenriChabrand It's finally will be gone on the next release 😉


[QUERY] How to access the search results from calling SearchClient.SearchAsync

Library name and version
Azure.Search.Documents (11.4)
Query/Question
Original Code (v10.):
var client = _serviceClient.Indexes.GetClient(indexName);
Task<DocumentSearchResult> results = await client.Documents.SearchAsync(azureQuery.SearchValueParam, searchParams);
New Replacement Code (v11):
var client = _searchIndexClient.GetSearchClient(indexName);
Response<SearchResults<SearchResult>> results = await client.SearchAsync<SearchResult>(azureQuery.SearchValueParam, (SearchOptions)searchParams);
I am trying to determine how to get the "SearchResult<Document>" from results. Is there a way to do this? Which object inside Response class and also inside the SearchResults class do I need to reference to get down to this information?
I am asking here because of all of the name changes in the API.
Environment
.Net 6.0
Visual Studio 2022

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hello @MerjTek, you can access your documents using results.Value.GetResultsAsync():
Response<SearchResults<MyDocument>> results = await client.SearchAsync<MyDocument>("*");
await foreach (SearchResult<MyDocument> result in results.Value.GetResultsAsync())
{
   ...
}

Alternatively, you can directly access your documents by using the SearchResults<T> type from the search call:
SearchResults<MyDocument> results = await client.SearchAsync<MyDocument>("*");
await foreach (SearchResult<MyDocument> result in results.GetResultsAsync())
{
    ...
}

For more examples and details, please refer to the official documentation.

The Search method returns a Response<SearchResults<T>>, so you should assign the results to Response<SearchResults<T>>, not Response<SearchResults<SearchResult<SearchDocument>>>.
Here's an example of how to access SearchResult property values:
Response<SearchResults<SearchDocument>> response = await client.SearchAsync<SearchDocument>("*");

// Accessing SearchResults property values
SearchResults<SearchDocument> searchResults = response.Value;

// Accessing each SearchResult property values
await foreach (SearchResult<SearchDocument> result in searchResults.GetResultsAsync())
{
    // Accessing your Documents
    SearchDocument doc = result.Document;
    string id = (string)doc["Id"];
    Console.WriteLine($"{id}");
}

For more details please refer this sample.

The Search method returns a Response<SearchResults<T>>, so you should assign the results to Response<SearchResults<T>>, not Response<SearchResults<SearchResult<SearchDocument>>>.
Here's an example of how to access SearchResult<T>  from the response:
Response<SearchResults<SearchDocument>> response = await client.SearchAsync<SearchDocument>("*");

// Here's how you can access SearchResults<T> from the response
SearchResults<SearchDocument> searchResults = response.Value;

// Here's how you can access each SearchResult<T> from the response
await foreach (SearchResult<SearchDocument> searchResult in response.Value.GetResultsAsync())
{
    // Here's how you can access each document
    SearchDocument doc = searchResult.Document;
    string id = (string)doc["Id"];
    Console.WriteLine($"{id}");
}

For more details, please refer to this sample.


⚠ API - Lens has degraded performance

In 9b130af, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 581 ms

Resolved: API - Lens performance has improved in 37ca84e after 7 minutes.


Rework Logging in General Troubleshooting section

Summary
Added some detail to the logging part of the General Troubleshooting section.
Ticket Link
There is no ticket for this.

Thanks @faase! Added a suggestion on the sub-section heading, otherwise couple of minor notes. Great iteration


Add SuppressMissingDependenciesErrors option support

Works for both project and package references

@ErikEJ @jeffrosenberg Can either one of you have a look at this as well?

which issue does this resolve?

Read the docs now, see the purpose


can not fire !!!!

Hiya! So you think it's broken or missing a feature eh?
Welllllll, it just might be.  Do me a favor and check the following.

[ ] Issue does not already exist
Identify exactly what aspect your issue is about

[x] ignite base application
[ ] ignite the cli
[ ] ignite generated code

Also, for the quickest assistance please run ignite doctor and paste in
everything (including the lines with 3 backticks).
  // ...

   create test/fastlane/Fastfile
   create test/fastlane/Matchfile
⠋ Copying Ignite goodies
➟ Package manager:	[ ] npm  |  [✔] yarn
𝗫 Installing Ignite dependencies (~ 1 minute-ish)
𝗫 Linking external libs
⠋ Linking external libsfs.js:557
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^

Error: ENOENT: no such file or directory, open 'test/android/app/src/main/AndroidManifest.xml'
    at Object.fs.openSync (fs.js:557:18)
    at Object.fs.readFileSync (fs.js:467:33)
    at Object.insertInFile (/usr/local/lib/node_modules/generator-react-native-ignite/utilities.js:68:27)
    at performInserts (/usr/local/lib/node_modules/generator-react-native-ignite/app/index.js:124:13)
    at AppGenerator._updateAndroid (/usr/local/lib/node_modules/generator-react-native-ignite/app/index.js:446:7)
    at ChildProcess.<anonymous> (/usr/local/lib/node_modules/generator-react-native-ignite/app/index.js:540:18)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:885:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
yoos-Mac-mini:workspace yoohoo$

and the doctor message:
Computer
  Platform: darwin

Ignite
  Version: 1.12.0
  Path: /usr/local/bin/ignite

Node
  Version: v7.3.0
  Path: /usr/local/bin/node

NPM
  Version: 3.10.10
  Path: /usr/local/bin/npm

Yeoman
  Version: 1.8.5

React Native CLI
  Version: 2.0.1

App
  React Native Version: ¯\_(ツ)_/¯

Looks to me like android did not get generated.  So I'm guessing you can't do react-native init at all is that correct?
What error do you get with react-native init test

I can pass react-native-cli , both iOS and Android is normal. Only one thing just I still don't know how to use genymotion it will tell me can't patch bundle .
Now I reinstall and then post more details
Sent from my Samsung Galaxy smartphone.
-------- Original message --------
From: Gant Laborde<EMAIL_ADDRESS>Date: 12/25/16 04:06 (GMT+08:00)
To: infinitered/ignite<EMAIL_ADDRESS>Cc: Aooren<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [infinitered/ignite] can not fire !!!! (#562)
Looks to me like android did not get generated. So I'm guessing you can't do react-native init at all is that correct?
What error do you get with react-native init test

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/infinitered/ignite/issues/562#issuecomment-269098403, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHNwuNDItgFnTAXM0nKm7TWn1wq0ebjvks5rLXsigaJpZM4LVQ-q.

First new I will meet this problem..
Igniting app

✔ Finding react-native
✔ Finding android
✔ Finding git
⠋ Finding gitshell.js: internal error
Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/generator-react-native-ignite/app/templates'
    at Object.fs.mkdirSync (fs.js:854:18)
    at /usr/local/lib/node_modules/generator-react-native-ignite/node_modules/shelljs/src/mkdir.js:65:10
    at Array.forEach (native)
    at Object._mkdir (/usr/local/lib/node_modules/generator-react-native-ignite/node_modules/shelljs/src/mkdir.js:48:8)
    at Object.mkdir (/usr/local/lib/node_modules/generator-react-native-ignite/node_modules/shelljs/src/common.js:240:23)
    at emptyFolder (/usr/local/lib/node_modules/generator-react-native-ignite/app/index.js:62:21)
    at AppGenerator.cleanBeforeRunning (/usr/local/lib/node_modules/generator-react-native-ignite/app/index.js:287:7)
    at Object.<anonymous> (/usr/local/lib/node_modules/generator-react-native-ignite/node_modules/yeoman-generator/lib/base.js:431:23)
    at /usr/local/lib/node_modules/generator-react-native-ignite/node_modules/run-async/index.js:25:25
    at /usr/local/lib/node_modules/generator-react-native-ignite/node_modules/run-async/index.js:24:19

and then I chmod -R 777 to avoid this problem
then meet the issue problem I just posted . T. T sad

@GantMan

I have the same problem

@yoohan-dex and @gaboelnuevo - check out this: https://github.com/nodeschool/discussions/issues/305 and see if following suggestions in that thread help to solve the problem.

The issue was about permitions on create template folder. I created it and add permition manually and it works.

Had the same issue, i had react cli and failed install of ignite using sudo install.
Worked after i removed files that my user did not have access to rm -r /usr/lib/node_modules/

I have been stuck around this for a while and what worked for me was:
sudo rm -r -f /usr/lib/node_modules/generator-react-native-ignite/app/templates
sudo chmod 777 /usr/lib/node_modules/generator-react-native-ignite/app
ignite new myAppName

Depending on how node.js was setup on your machine, things can get hairy.
In Ignite 1 we write temporary files under the global node_modules path.  Combine this with a less-than-ideal node.js setup, and stuff like this happens.
In Ignite 2 the only file writing we do is to the directory your project is going.
Better days ahead!


Multilingual version?

Hi @hteumeuleu — everything is in the issue title 😉
How can one help?

Someone recently shared their version translated in german: https://hocusfocus.marcus-herrmann.com/
Considering Jekyll is not really fitted for multi-languages site, perhaps this is the best approach? (Do your own fork and self-host it?) Or should I host it under the same repo? (in a /fr/, /de/, … folder for each language) I'm open to ideas.

I like the idea of having multiple languages under the same umbrella domain.
If you go with this option let me know, I'd gladly contribute to the French version. 🙂

Let's do this! If you can do this, I'd suggest copying all the content within the pages folder and duplicating it to a /pages/fr/ subfolder. Then you can translate everything in it. (Make sure to also change any link starting with / to /fr/.) Let me know if you need anything.


Improve the community resources page

Extend the community resources page with what each communication method is better for. Add links to the Ironic and cluster API communities.

/assign @SafeEHA @hroyrh

/triage accepted

Assign this to me

/assign @SafeEHA

this is completed


BREAKING: Format changelog using Prettier

MetaMask/auto-changelog#155 adds a method to format changelogs using Prettier. I've enabled this option here.
This is a breaking change, since it will change the default formatting of the changelog in new release PRs. Packages updating to the next version of this package should simultaneously enable the --prettier flag on auto-changelog.

Could we add a BREAKING changelog entry in the PR description (and eventual commit message) detailing the required build script changes, and any other considerations to introducing the updates made in this PR?
Otherwise looks good!


Prune old versions from TUF repo

Currently we just add new Flynn versions to the TUF repo and never remove them. We should only keep a fixed number of recent releases (5?).

Alternatively we could implement delegations and split each version into a separate targets file.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.