Unknown MIME type! Can you tell me how to solve this issue?

Carrierwave::Base64::Base64StringIO::ArgumentError (Unknown MIME type: data:image/jpg):
I know you said using MIME::Types#add. But how? Thanks
I am using Rails 5.1 and was trying to upload a image using curl.
(echo -n '{"data":{"cover": "data:image/jpg;base64,';
base64 app/images/cover.jpg; echo '"}}') |
curl -X PATCH -H "Content-Type: application/json" -d @-
http://localhost:3000/api/books/45

white list defined in uploader
def extension_whitelist
%w(jpg jpeg png)
end

@BranLiang an easy fix would be to use data:image/jpeg mime type (with e)
For adding a mime type, you need to write an initializer

@lebedev-yury thanks, that works :dancing_men:

You're welcome!


[The D Bug Tracker] std.traits.isAggregateType chokes on modules

jl reported this on 2020-04-23T19:42:48Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=20764
CC List

moonlightsentinel

Description
It seems that you can throw just about anything at isAggregateType, but not a module:

module test;
import std.traits;
pragma(msg, isAggregateType!test);

t.d(3): Error: template instance `isAggregateType!(test)` does not match template declaration `isAggregateType(T)`
t.d(3):        while evaluating `pragma(msg, isAggregateType!(test))`

Suggested fix: change the definition to:

enum isAggregateType(T...) =
  is(T[0] == struct) || is(T[0] == union) || is(T[0] == class)
  || is(T[0] == interface);

moonlightsentinel commented on 2020-04-25T23:10:46Z
A module is not a type (strictly speaking). Might be less problematic if this trait was named isAggregate


Improve Filter docs on apply/verify + correct simple example + add a practical complex example

Description
The filter section in the documentation is lacking in describing what value is for newcomers who want to apply a custom filter. It's easy for newcomers to assume that value is the string value of the query param of the filter, when it is actually the array of string params by default (if not provided a verify callable).
The example also confuses this distinction too, by comparing the value to the symbol :public - where value would actually be an array of ['public'], since there is no verify callable to modify value.
Changes Proposed:

Add clarifying note describing what value is to educate newcomers.
Simplify the example by providing the query param in comments and explicitly using the apply callable in the default case with a value array without a verify callable.
Add a more complex example that combines both verify and apply callables to show how to use them in actual practice.

@lgebhardt 👍  moved the note up

@kevinzwh Thanks!


Suggestion to improve documentation

Thanks for this great project. I have a few suggestions to improve the documentation to avoid installation errors:

in the config sample of the readme there is a comma missing at "        RefreshInterval: "5000" //milliseconds"
in the config sample of the readme I believe the variable should be called"        RefreshIntervalLAN: "5000" //milliseconds" and not "RefreshInterval"
in the readme there should be a step that after running git clone the npm install command be run from the MMM-Shelly-PM  folder to install "request" dependency that doesn't come with the Magic Mirror app anymore

Thank you. Sorry for the late reply, I will look into this.

Added all your suggestions, thank you for your message (and again sorry for the late reply)


setup local oracle pool services

setup oracle for RNG

Will there be any changes needed when oracle v2 is live?


udp: remove sendto/recvfrom

We only use these APIs in tests and it's better to
unify around sendmsg/recvmsg with appropriate
wrappers.
Risk Level: Low
Testing: Existing tests
Docs Changes: N/A
Release Notes: N/A

@danzh2010 @goaway PTAL should be a quick one.

@danzh2010 I'm going to merge and I will put the nits into my next PR.


[Enhancement] Add docs for policy exceptions support in the CLI

Description
Add docs for policy exceptions support in the CLI.
https://github.com/kyverno/kyverno/pull/9525
Slack discussion
No response

/assign


iOS 13 UISceneDelegate Support

Any interest in adding UISceneDelegate support? It seems that when you adopt the UISceneDelegate, some app lifecycle methods don't get called, instead the corresponding SceneDelegate methods are called. Example,  sceneDidBecomeActive instead of applicationDidBecomeActive.

Thanks for bringing that up! I'd definitely be interested to support UISceneDelegate. I'll have a look at it, as soon as I find the time. If you find a solution before that feel free to open a pull request.


Add DenseClus Implementation notebook for jumpstart

Summary
Add DenseClusImplentation notebook for jumpstart, it includes:

Kmeans implementation on numerical features
umap embeddings + hdbscan on numerical and categorical features separately
DenseClus implementation on all features together using 5 umap combine methods
Analysis on the results
Conclusion

Resolved most of the comments. Added descriptive stats for analysis on clusters formed and observations.

@momonga-ml ready for your review


Unable to be used with terraform workspaces

Describe the Bug
Hi,
not sure if it's a bug or a not implemented feature but it looks like the module is working properly together with terraform workspaces.
It seems the issue is that when a workspace is switched this module is going to try again to create the s3 bucket and dynamodb. but this two resources are already existing so it will fail.
Using the workspace in the bucket and dynamo table name with case issues with the backend.tf because this will alter the whole time.
Using enabled = false will cause the first terraform workspace which has created the s3 bucket to destroy it again. Also sounds not good :D
Expected Behavior
The module is checking if the expected bucket is already and then skip the creation.
Maybe something like the enabled flag but just skip_creation_if_resources_exists or so on.
Steps to Reproduce
Steps to reproduce the behavior:

Create two workspaces
Apply the first workspace and see s3 bucket and dynamo table created
Switch to second workspace
Apply second workspace and see errors while s3 bucket and dynamo table creation

Additional Context
Maybe its also not possible and for multi workspace usage we need to create a separate terraform project (workspace) which is taking care of the resource but then an example would be nice.
Thanks

Instead of setting enabled to false, since you're working with workspaces you can have the default workspace house the state for the backend, and after bootstrap work simply with other workspaces. Then, set enabled as follows:
module "terraform_state_backend" {
  enabled = terraform.workspace == "default"
  source  = "cloudposse/tfstate-backend/aws"
  version = "0.38.1"
  # ...
}

This way, the module will set the count argument of every resource block it uses to 0, turning them into no-ops. Additionally, if you want to avoid provisioning infrastructure of your own save for the one supporting the S3+DynamoDB backend, you can separate the rest of your configuration into a module and use count:
module "rest_of_my_config" {
  # Avoid creating anything within if we're in the default workspace.
  count = terraform.workspace == "default" ? 0 : 1
  # ...
}

The module creates an S3 backend to be used by Terraform. Usually this is done once per company/organization, so I do not understand why you are trying to create one per workspace.
Whether or not you are using workspaces, it is your responsibility to ensure you are not creating duplicate resources by using different inputs for different instantiations of this module. This module provides several inputs you can use to vary the name of created resources:

namespace
tenant
environment
stage
name
attributes

Closing this out as it's a misunderstanding by the module consumer on how to use this module. @eloo if you're still struggling with how to use this module correctly following the above comments, please feel free to ask questions here, ping me, and I can try to help you understand the correct usage in your scenario.


Request : add LOW_PRIORITY statement

Hello,
It would be perfect if you could add this kind of statement to manage the priority of ours insert/update queries.
An idea : $db->insert('exemple', $insert, 'low');
Thanks
Regards,
Vincent

You should explain more about the priority of statement.

Admitting that your server is dealing with a high traffic, like 500 requests per second.
The internal mechanism of MySQL is to lock tables (MyISAM) or raws (InnoDB) when an insert/update request is fire, so no SELECT can be fired meanwhile.
On heavy loaded tables, that could make thousands of requests in a waiting state (and sometimes can cause MySQL crashes).
To avoid this fact, MySQL allow you to specify a priority on your request, then MySQL will choose the "best" (or "less worst") moment to fire your insert/update.
(if you add this statement, here is another usefull one : SQL_NO_CACHE)


Fixes Issues #2065 #2066 #2067: Added Other Information, Storage Inst…

…ructions and Recycling details to the product details.
Description
Added Other information, storage instructions and recycling instructions to the product activity and addProduct activity.
Related issues and discussion
#fixes #2065 #2066 #2067
Screen-shots

LGTM. I'm merging. Thanks @prashantkh19 👍


How to use this html object? any Idea

How can we show the same html page in JS

Hi @vinodchauhan7 with the output, you should be able to render the html document as you normally would. If you're looking to render it on the client via Javascript, you can replace the current document with the new HTML document e.g
document.replaceChild(docGeneratedWithMhtml2html.documentElement, document.documentElement);

You could also potentially render it in an iframe


sitetree batch actions: select all/none

It would be tree-mendous if the sitetree batch actions included "select all" and "select none" options. Right now I'm looking at a site with a few hundred pages that need to be individually ticked... also the "select all" feature should honour any filtering. Sorry about the pun.

Please keep improvements like this to uservoice.silverstripe.com :) "select all" is particularly difficult since we lazy load the tree - so "select all" might need to execute dozens of XHR calls to actually get a full representation of items on the client. For this reason, I don't see it being implemented anytime soon. In general, I think those batch operations are better suited for a GridField list view (where they're also missing at the moment). Closing


Not able to push api

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v14.17.6
Amplify CLI Version
6.1.1
What operating system are you using?
mac
Amplify Categories

api
Amplify Commands
push, update
Describe the bug
i am trying to push or update api but it's not working.

Expected behavior
i should be able to push the api
Reproduction steps
i am not sure
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Hi @akash-gupt,
There might be a JSON config file that's in a bad state - have you edited any JSON files since changing the API? In the amplify directory

Hey @akash-gupt :wave: just wanted to follow up to @marcvberg 's message and see if you were able to identify a JSON file that might have been manually edited?

Yes i found it by making some changes on amplify cli.
On Tue, 2 Nov 2021 at 12:13 AM, josef @.***> wrote:

Hey @akash-gupt https://github.com/akash-gupt 👋 just wanted to follow
up to @marcvberg https://github.com/marcvberg 's message and see if you
were able to identify a JSON file that might have been manually edited?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/8473#issuecomment-956491694,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVCNGB5N7TMTL3NPWA7YCTUJ3NUXANCNFSM5GGAEEYA
.

Thanks for clarifying @akash-gupt! Were you able to successfully push your API after identifying the affected JSON file?

Yes, I did.
On Tue, Nov 2, 2021 at 9:05 PM josef @.***> wrote:

Thanks for clarifying @akash-gupt https://github.com/akash-gupt! Were
you able to successfully push your API after identifying the affected JSON
file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/8473#issuecomment-957838586,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVCNGBG56ML6UD6BCL6NFLUKAHMVANCNFSM5GGAEEYA
.

Awesome! Glad to hear you're back up and running 🙂
Closing issue

Thank you.
On Wed, Nov 3, 2021 at 1:21 AM josef @.***> wrote:

Closed #8473 https://github.com/aws-amplify/amplify-cli/issues/8473.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/8473#event-5557924648,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVCNGA2MH7S5RFYRCKE4Z3UKBFN3ANCNFSM5GGAEEYA
.


Stacked Methods Do Not Pass with removeJSComments

Challenge: https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/functional-programming/use-the-filter-method-to-extract-data-from-an-array
Forum post: https://forum.freecodecamp.org/t/bug-found-in-this-challenge/458226
Is some part of the code processing chain causing this challenge to fail with the following code?
This fails
var filteredList = watchList.map(movie => ({
   title: movie["Title"],
   rating: movie["imdbRating"]
}))
.filter(r => {
  return parseFloat(r.rating) >= 8.0;
})

This passes:
var filteredList = watchList.map(movie => ({title: movie["Title"],rating: movie["imdbRating"]})).filter(r => {
  return parseFloat(r.rating) >= 8.0;
})

The match for .filter is failing. But it doesn't seem to me like it should be the regex that is the issue assert(code.match(/\.filter/g)); so maybe we inadvertently broke something, somehow?

@ojeytonwilliams @nhcarrigan Did we look at other libraries?
I tested a tiny bit using strip-comments and it seemed to work. I only tested the challenge from this issue (without the PR) and the other challenge linked to in the other issue and then I just ran the test:curriculum script. Not really sure how to test it.

@lasjorg  Nope, I don't think so. That library looks promising though.
For testing purposes we could simply write unit tests that include strings that we know the old, Babel, approach modified and see if the new library handles them correctly.
Would you be interested in creating a PR?

@ojeytonwilliams I tested the lib against the unit tests we have (npm test). It is preserving the indentation before/after the comment which the tests do not expect (fails removeJSComments tests 2 and 4). Preserving the indentation and whitespace does seem more correct even if it leaves the code with some odd formatting. It literally just removes the comments and doesn't touch any whitespace (as far as I can tell).
Input:
function nonMutatingPush(original, newItem) {
  /* This is a
    multi-line comment
    that should be removed. */
  return original.push(newItem);
}

Output:
function nonMutatingPush(original, newItem) {
    return original.push(newItem);
}

The 4 space indentation is the 2 in front of the comment block and the 2 before the return statement.
The library comes with unit tests which I would expect covers a lot of cases.

I will create PR so we can better test it.

Looks like another one: https://forum.freecodecamp.org/t/command-chaining-bug-in-functional-programming-section-possible-incorrect-solution/458829
The first solution on the hints page to Use the reduce Method to Analyze Data is failing.

Closing this, tracking further in  #42101

Reopening as related to a different fix.

For those wanting to help with this, the quick fix is to update the regex tests in these lessons to allow for white space between the . (period) and method name.
Note: Every lesson must be updated, before this issue can be closed

An issue with the help wanted or first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum

@ShaunSHamilton This should get fixed with https://github.com/freeCodeCamp/freeCodeCamp/pull/42254 which is a comment removal implementation using our own fork of the strip-comments lib.

@ShaunSHamilton This should get fixed with #42254 which is a comment removal implementation using our own fork of the strip-comments lib.

can this still be closed then?

@Sembauke Thank you, for keeping up with this. I was a bit out of the loop with what was happening between this issue and the Safari issue. I will close this, once tested on .dev

I have tested on staging, and the original issue appears to have been resolved. Thank you, all involved 🎉


wherekeys() expects both partitionKey and rowKey to be not null

Current implementation basically emulates the GetEntity functionality. It would be highly beneficial to allow whereKeys to be passed with a partitionKey and a null rowKey, so that queries can be further optimized to return all records in a partition.

Hey @nikhilhasija  -- Wouldn't it be best if you just used the following example to return all records given a specific partition key?
https://www.windowsazure.com/en-us/develop/nodejs/how-to-guides/table-services/#query-set-entities

We can close this issue -- if @nikhilhasija has another use case that the above example doesn't solve, we can re-open.

+1 on that @andrerod -- Hint hint to the community to land this feature + tests ;)

Thanks @andrerod I support it :)
I think most of the time, the way I am designing my interfaces, PartitionKey is extracted from the data and often there is no where clause (although a where clause simply filters it down further). When I looked at the code, it seemed that making a call using only whereKeys may be more efficient that building a where clause (few ticks but in node i try to avoid, any compute necessary).
Also there is a matter of convention causing confusion.. In node.js variable convention the partition key is referred to as 'partitionKey' while in Azure tables it is 'PartitionKey'... moving the query to a where clause took me a while to decipher as why my data was not showing up as I was using 'partitionKey eq ?' erroneously...

Glad you guys aprove :-) If no one else takes this, expect some updates after the holidays.


Hyderabad is mapped as state in Death_and_Recovered Data

Line 159 in Death_and_Recovered Sheet in Spreadsheet.

Thanks a lot for reporting. Fixed it.


Overflow in IN endpoint handler when using DMA

The following line does not handle the case when Progress is less than the max packet size.  This causes Progress to wraparound and causes enumeration to fail when DMA is enabled (since the device descriptor is less than the packet size).
https://github.com/IntergatedCircuits/USBDevice4Cube/blob/5871c435dee1511c799e577a79bbaa1010f794e5/XPD_USB/xpd_usb_otg.c#L1114
I changed it to the following, which fixed the problem.
if (pxEP->Transfer.Progress > pxEP->MaxPacketSize)
    pxEP->Transfer.Progress -= pxEP->MaxPacketSize;
else
    pxEP->Transfer.Progress = 0;

Good catch! I reviewed the DMA transfer handling a bit, and found that the OUT direction is also completely flawed. I pushed an update that hopefully resolves these issues, looking forward to your feedback on it.


DRAFT TC-1668 Correct the issue of incorrect timestamp sorting

Backport of https://github.com/trustification/trustification/pull/1747/files
This is just a draft PR to trigger the Conflux pipelines, generate an image container for downstream and test the solution provided in the original PR. This PR should not be merged, this is just for testing purposes.

Closing in favor of https://github.com/trustification/trustification/pull/1752


When the date cell is set to required, the data picker shouldn't show the clear button

Private Zenhub Video

@carinanfonseca It seems the video is private , we are getting File Permission Denied.
or can you confirm, that we have to disable this clear button for date picker , if the date cell is set to required.

@appsmithorg/query-js-pod @carinanfonseca can you please add a reveiw for this PR

hey @Nikhil-Nandagopal @carinanfonseca , I would like to work on this issue if no one's working, could you please assign it to me!


ci(BLE, workflow): Add RSSI readings and other info in PER test

Added RSSI reading in PER test and more info in the PER result plot.

Need to add us a reviewers. Accidentally added us as assignees. Also, it says 0 files changed?

Already merged before.


[1.12.2] Game Crash on World Load

While working on my personal 1.12 modpack I found a rather annoying crash. While the game starts up & loads just fine, as soon as you load a world, the game crashes. Sometimes it's instant, sometimes you can actually roam around the world for a while, but either way: The game crashes very shortly after a world loads. At first I thought this was a Simply Jetpacks 2 issue, since I was seeing error messages in the log regarding it. However, after taking a closer look at it I noticed that the main crash pointed to Thaumcraft. I've no clue whether this is just an incompatibility with another mod or not, but either way I figured it was best to bring up here. If you wish to see the log in more detail, it can be found here: [https://paste.ee/p/pcbeV]

I can confirm this! Exactly the same stacktrace.

mine is on a server version 1.12.2 the crash stops as soon as i remove Thaumcraft-1.12.2-6.1.BETA15.jar
crash-2018-06-02_14.02.38-server.txt
debug.log

Hi I am getting the same crash error cannot upload log atm but i will tomorrow since its on my server computer.
But the crash the people above have posted is same as what I am getting.
But anyways I made a  new dimension using the Simple Dimensions since my main worlds uses different Terrain generator and it seems to not generate the Thaumcraft Magical Forest biome. So i used the Simple Dimensions mod to make a BiomesOPlenty world where i knew the Thaumcraft biomes would generate. It was all working fine until i came across the Magical Forest biome and as soon at the chunk is fully loaded the server crashes and gives the same error as the above people.
Seem to be with the EntityPech which is causing it.


Fix #38 use expand()

Fix #38

@Shougo expand() would result to empty if wildignore contains *.exe for Windows, I think it's better to expand(xxx, 1) to ignore wildignore ?

Please test https://github.com/roxma/nvim-yarp/pull/40.

tested, thanks


No need to call datetime.replace for utc_now

utc_now already has the timezone set to UTC, so calling replace(tzinfo=timezone.utc) is redundant

thanks so much.


react-native run-ios  BUILD FAILED !!! hint not Octicons.ttf file but has Octicons.ttf

** BUILD FAILED **

The following build commands failed:

	CpResource /Users/angel/RNcomponent/node_modules/react-native-vector-icons/Fonts/Octicons.ttf build/Build/Products/Debug-iphonesimulator/Jzcapp.app/Octicons.ttf
(1 failure)

➜  react-native-vector-icons git:(master) ✗ cd Fonts
➜  Fonts git:(master) ✗ ls
Entypo.ttf                 MaterialCommunityIcons.ttf
EvilIcons.ttf              MaterialIcons.ttf
FontAwesome.ttf            Octicons.ttf
Foundation.ttf             SimpleLineIcons.ttf
Ionicons.ttf               Zocial.ttf

Ran into the same issue, any updates on this?

Resolved the issue with a hack on Xcode, on Build Phases > Copy Link Binaries there were duplicate font files, and I removed the ones not from the "react-native-vector-icons" folder and it compiled afterwards. I'm sure it's going to break when we upgrade, and hopefully with that upgrade this goes away.

Not sure but maybe you did both manual and automatic integration?

Closing for lack of replies

This type of error started with 'CpResource' occurs when you are trying to load assets from a directory and that asset file is not present in that directory.

Hi, how did you fix this?

@angelporo @oblador

@Kailash23 yes, but how can the font be missing from the node_modules folder after an npm i / yarn install?

@Kailash23 yes, but how can the font be missing from the node_modules folder after an npm i / yarn install?


[Table] headerStyle and bodyStyle not working

Table: headerStyle and bodyStyle not working
Steps to reproduce
I'm using SimpleGrid and it uses vertical-align: baseline across the griding system. This means that the inherited vertical align in table, specifically on the text is off.

Removing the vertical align from simplegrid.css:

Tried to fix using headerStyle and bodyStyle in table, and setting it to be vertical-align to middle did not work. The current fix is to apply the inline style at all TableHeaderColumn and TableRowColumn individually.
Obviously I don't want to change SimpleGrid. I'd like to be able to simply enforce below using headerStyle and bodyStyle in table.
export const styles = {
  table: {
    verticalAlign: 'middle',
  },
};

Thanks in advance.
Tat
Versions

Material-UI<EMAIL_ADDRESS>React<EMAIL_ADDRESS>Browser: Chrome on Ubuntu

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


[cdc] Adjust-CdcJsonDeserializationSchema

Purpose
The serialization entry of cdc kafka_table/database_sync is CdcJsonDeserializationSchema, but CdcJsonDeserializationSchema#deserialize returns CdcSourceRecord(JsonNode). When users extend other format, there will be limitations. Most data serialization is based on byte[], not JsonNode.
For example:
Users need to use other serialization protocols to serialize data based on byte[], such as binlog of other databases.

Tests

API and Format

Documentation

closed, recommended to use flink-cdc Paimon Pipeline Connector first


Gateway locality not observed with multi-cluster / multi-network mesh

I'm trying to setup a mesh with three clusters on two networks. The simpler two-cluster/two-network and two-cluster/same-network mostly work. When combining the two configurations into a single mesh I've observed that the gateways are sometimes routing to other gateways when local endpoints are available.
For example, Cluster A's ingress gateway routes to Cluster B's ingress even though they're on the same network. Or Cluster's ingress is routed to Cluster C on a different network even though Cluster A/B have local endpoints ready.
Setup:
Cluster A and B are on vpc0. Cluster C is on vpc1. The control plane is replicated in each cluster. The meshnetwork are almost identical in each cluster (see below) with one exception - the local kube-apiserver is always named Kubernetes instead of cluster's unique ID. I've also plumbed through the network name to the ingress and sidecar proxies.
      # ClusterA's version
      meshNetworks:
        networks:
          vpc0:
            endpoints:
            - fromRegistry: Kubernetes
            - fromRegistry: <clusterB ID>
            gateways:
            - address: <ClusterA ingress IP>
              port: 443
            - address: <ClusterB ingress IP>
              port: 443
          vpc1:
            endpoints:
            - fromRegistry: <clusterC ID>
            gateways:
            - address: <ClusterC ingress IP>
              port: 443

Omitting the cluster A/B's gateway entries in cluster A/B's meshnetwork also didn't work. Pilot stopped pushing endpoints (via eds) from the vpc0 network to proxies. Inbound traffic falls over to the passthrough envoy cluster which fails when mTLS is enabled.

The mesh network gateway has a locality field. The comment says this is used when an explicit IP is provided for the gateway. As far as I can tell this doesn't seem to be used by Pilot.

I do see a warning in the ingress-gateway logs about an invalid cluster load assignment in the single-network / two-cluster case. Not sure if that's related but the missing load balancing weight looked suspicious.
[Envoy (Epoch 0)] [2019-11-07 05:12:20.049][18][warning][config] [external/envoy/source/common/config/grpc_mux_subscription_impl.cc:82] gRPC config for type.googleapis.com/envoy.api.v2.ClusterLoadAssignment rejected: Proto constraint validation failed (ClusterLoadAssignme
ntValidationError.Endpoints[i]: ["embedded message failed validation"] | caused by LocalityLbEndpointsValidationError.LbEndpoints[i]: ["embedded message failed validation"] | caused by LbEndpointValidationError.LoadBalancingWeight: ["value must be greater than or equal to
 " '\x01']): cluster_name: "outbound_.15030_._.istio-ingressgateway.istio-system.svc.cluster.local"

I have the very same problem including identical messages in the ingress-gateway. Did you solve it?


Replace Promise with PromiseLike in resolver types

Is your feature request related to a problem? Please describe.
When using non-native Promise library like Bluebird, the return type of resolvers will often be PromiseLike, but it doesn't adapt with generated resolver-types because the generated code is using native Promise type.
https://github.com/dotansimha/graphql-code-generator/blob/8e4d5826df7c01fec3fa26d200d171881ee3a04a/packages/plugins/typescript/resolvers/src/index.ts#L138
sample code:
const resolvers: Resolvers = {
  Query: {
    name: () => { // string field
      return Promise.resolve('1') as PromiseLike<string>
    }
  }
}

error message:
(...omitted)
    Type 'PromiseLike<string>' is not assignable to type 'ResolverTypeWrapper<string> | Promise<ResolverTypeWrapper<string>>'.
      Type 'PromiseLike<string>' is missing the following properties from type 'Promise<ResolverTypeWrapper<string>>': catch, finally, [Symbol.toStringTag]

Describe the solution you'd like
Can Promise be just replaced with PromiseLike?
:memo: https://github.com/graphql/graphql-js/blob/0fef3c49907b63b1ea5a4ff1da7011775f465fc2/src/execution/execute.ts#L505-L506
Describe alternatives you've considered
We can customize it by using customResolverFn.
So this feature request is not mandatory, but I feel it's better if I don't have to customize and manage it in our code.
Additional context

Hi @qsona ! This is a great idea. What do you think about creating a PR for this? :)

Yes, I'd like to try this :)

I realized it's necessary to clarify if graphql-js expects PromiseLike value as resolver's return value. I'm waiting and seeing the discussion on graphql-js.
https://github.com/graphql/graphql-js/pull/3243#pullrequestreview-739333027

Hi, I'm sorry for not updating this long time. My idea was simply replacing Promise with PromiseLike (firstly in graphql-js firstly, then this repository) but it seems it needs some works to be accepted in graphql-js. Actually I'm not so enthusiastic for this issue (because casting PromiseLike to Promise is easy by using async-await) so please feel free to close this issue and PR.


Epic: Testing

Epic
It is important to have a majority of our code tested for any scenario it'll receive, so that we don't get any unexpected bugs.
We want our frontend and backend code fully tested - unit and integration. Also, we need to add code coverage so that we know how much of our code is tested.
User Stories

[x] story #31 (Integration testing set up)
[x] story #54 (Frontend tests)
[x] story #41 (add code coverage)

Closing this since it is done.


Fix NAT Pool Migration

Fixes a number of issues related to NAT Pool migration which were not surfaced in the light initial testing of this feature. Primary changes are:

Skipping the migration in NAT Rules which appear to have been created to support a NAT Pool (where the names match the pattern .\d{1,3} and where the frontend port is in the NAT Pool range.
Additional logic added to associate the IP configurations in the VMSS network profile with the migrated NAT pool and perform a subsequent update of the VMSS and VMSS instances to ensure the change is rolled out
Reordering the steps in the migration scenarios to ensure that the LB NAT Rules are not modified after the NAT Pool is migrated (doing so causes a conflict)

This PR fixes issue #6


Added postfix to the classic mint

Thank you for your contribution to the KodaDot NFT gallery,
we really appreciate your contribution!
PR type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Before submitting this PR, please make sure:

[x] Your code builds clean without any erros or warnigns
[x] You've posted screenshot of demonstrated change in this PR
[x] Merged recent default branch, main and you have no conflicts
[x] Didn't break any original functionality

added advanced to classic mint?


Deriving tags by folder names

It is convenient to automatically create tags for roots and favorite folders.
And also label all resources from such folder with its tag.
Example:
Pictures
\---Cats
     |---a.png
     |---b.png
     |---c.png
\---Dogs
     |---x.jpg
     |---y.jpg
     |---z.jpg

Here we want to automatically label a.png,b.png,c.png with cats and x.jpg,y.jpg,z.jpg with dogs.
It's also would be nice to allow user to rename automatically created tags, if she prefers dog over dogs.
This feature is useful when a user has resources belonging to one category in separate roots.
Should be possible to turn on/off by folder.

Similar to #195


TS debug adapter broken on master.

Using git bisect i found it was this commit that broke TS debug adapter:
5aaca27b5f5e058334963f8b4716e1be287a68f9 is the first bad commit
commit 5aaca27b5f5e058334963f8b4716e1be287a68f9
Date:   Thu May 7 20:31:59 2020 -0600

    Logging cleanup. (#11606)

    For #9837

To repro, use build from master opt out of ptvsdwheels37 - experiment and debug a script.

We will create a branch off of master. We will be reverting the logging change in master. Remove the debugAdapter in the branch. do other necessary fix and logging work in that branch. and finally merge it back to master.


Update ThaiNER 1.4 to ThaiNER 1.5

What does this changes
Update ThaiNER 1.4 to ThaiNER 1.5
GitHub: https://github.com/wannaphong/thai-ner/tree/master/model/1.5
File model: https://github.com/wannaphong/thai-ner/releases/download/1.5/thai-ner-1-5-newmm-lst20.crfsuite
Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Passed code styles and structures
[x] Passed code linting checks and unit test

Thai NER
v1.4
Model Details

Developer: Wannaphong Phatthiyaphaibun
Model date: 2021-1-16
Model version: 1.5
Used in PyThaiNLP version: 2.3 +
Filename: ~/pythainlp-data/thai-ner-1-5-newmm-lst20.crfsuite
CRF Model
License: CC0
GitHub for Thai NER 1.5 (Data and train notebook): thai-ner-1-5-newmm-lst20.ipynb https://github.com/wannaphong/thai-ner/tree/master/model/1.5

Intended Use

Named-Entity Tagging for Thai.
Not suitable for other language or non-news domain.

Factors

Based on known problems with thai natural Language processing.

Metrics

Evaluation metrics include precision, recall and f1-score.

Training Data
ThaiNER 1.5 Corpus Train set (5089 sent)
Evaluation Data
ThaiNER 1.5 Corpus Test set (1274 sent)
Quantitative Analyses
                precision    recall  f1-score   support

        B-DATE       0.93      0.82      0.87       350
        I-DATE       0.95      0.94      0.95       665
         B-LAW       0.85      0.54      0.66        87
         I-LAW       0.85      0.64      0.73       253
         B-LEN       1.00      0.75      0.86        12
         I-LEN       1.00      0.69      0.82        26
    B-LOCATION       0.81      0.70      0.75       620
    I-LOCATION       0.74      0.72      0.73       533
       B-MONEY       1.00      0.91      0.95       131
       I-MONEY       0.99      0.95      0.97       321
B-ORGANIZATION       0.92      0.70      0.80      1334
I-ORGANIZATION       0.80      0.73      0.76      1198
     B-PERCENT       0.94      0.88      0.91        17
     I-PERCENT       0.91      0.95      0.93        22
      B-PERSON       0.96      0.78      0.86       607
      I-PERSON       0.94      0.88      0.91      2181
       B-PHONE       1.00      0.50      0.67         2
       I-PHONE       1.00      1.00      1.00         8
        B-TIME       0.93      0.66      0.77        87
        I-TIME       0.97      0.77      0.86       158
         B-URL       0.91      0.83      0.87        12
         I-URL       0.93      0.96      0.94        94

     micro avg       0.89      0.79      0.84      8718
     macro avg       0.92      0.79      0.84      8718
  weighted avg       0.90      0.79      0.84      8718
   samples avg       0.16      0.16      0.16      8718

Ethical Considerations
no ideas
Caveats and Recommendations

Thai text only

Hello @wannaphong! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tests/test_tag.py:

Line 293:9: E265 block comment should start with '# '


Verify bot has access to user repo on project creation

Fixes #881.

[x] I have run rspec and corrected all errors
[x] I have run rubocop and corrected all errors
[x] I have tested this change locally and tried to launch the server as well as access a project, and with that at least one build
[x] If there is an existing issue, make sure to add Fixes ... as part of the PR body to reference the issue you're solving

Changes

Split responsibilities of ConfigurationRepositoryService into new CollaboratorService with a method to validate if the bot user is a collaborator on a given project (bot_user_collaborator_on_project?)

TODO

[x] Fix broken tests for the ConfigurationRepositoryServiceSpec
[x] Add tests for the CollaboratorService
[x] Do the verification on project creation

@nakhbari, How were you envisioning validation for this? Were you envisioning a popup in the frontend notifying the user that the bot_user does not have access to the repository they're trying to create a project for?
Should I try and add the bot_user to the repository if they can't access it via first add_bot_user_as_collaborator!(repo_shortform:), and then notify the user if that was unsuccessful in the frontend?

I think there are two ways to do this.

we can show an intersection of repos that the user has access to and the bot has access to
we can show all the repos the user has access to and error out if the bot doesn't
I think showing an error message on the repo dropdown or a little red banner might be the way to go.

As for inviting the bot user to the repo, i'm wondering if we'll need an extra scope on the API token to do this.
Also, we just thought of moving to OAuth instead of API tokens. So having the bot_user_collaborator_on_project? method might make more sense in the GitHubService, which will eventually be stateless. Since we're removing the provisioning profiles in favor of passing along the GitHub OAuth token every time.


Upgrade terraform-provider-linode to v2.2.0

This PR incorporates the PF based resources into the provider.
This PR was generated via $ upgrade-provider pulumi/pulumi-linode --upstream-provider-name=terraform-provider-linode --pulumi-infer-version=true.

Upgrading terraform-provider-linode from 2.0.0  to 2.2.0.
Fixes #274
Fixes #270
Fixes #267
Upgrading pulumi-terraform-bridge from v3.44.3 to v3.47.2.

@aq17 Did you match our breaking changes with upstream? I couldn't so if you did that would be great.

@iwahbe the missing resource/ function databaseMongodb looks correct, but it appears stackScript still contains the attribute userDefinedFields https://github.com/linode/terraform-provider-linode/blob/dev/linode/stackscript/framework_resource_schema.go#L122
Looks like a PR was just merged to resolve some issues with the migration, so will hold off on merging this and wait for the next upstream version release
cc @t0yv0 on ops

Ack, thank you.

@aq17 looking at this again, it seems that https://github.com/linode/terraform-provider-linode/releases/tag/v2.2.0 already includes the referenced fix for upstream 227 issue. Can you clarify what upstream issue we need to wait on or what is the blocker here?
Looking closely at missing userDefinedFields this may be an issue with our tfgen recognizing list-of-object attributes, let me have a closer look. Perhaps if we can fix that we can go forward.

Yeah I think it's a blocker, originating in a likely bug in pulumi-terraform-bridge; will dig in shortly.

jq '.resources["linode:index/stackScript:StackScript"].inputProperties|keys' provider/cmd/pulumi-resource-linode/schema.json                                                  ~/code/pulumi-linode
[
  "description",
  "images",
  "isPublic",
  "label",
  "revNote",
  "script"
]
t0yv0@Antons-MacBook-Pro> jq '.resources["linode:index/stackScript:StackScript"].properties|keys' provider/cmd/pulumi-resource-linode/schema.json                                                       ~/code/pulumi-linode
[
  "created",
  "deploymentsActive",
  "deploymentsTotal",
  "description",
  "images",
  "isPublic",
  "label",
  "revNote",
  "script",
  "updated",
  "userDefinedFields",
  "userGravatarId",
  "username"
]

So on second look it looks like it's working as expected.
Upstream field is marked Computed. This means it's always set by the provider and ignores user input. Bridge maps that to an output property but not an input property.
		"user_defined_fields": schema.ListAttribute{
			Description: "This is a list of fields defined with a special syntax inside this " +
				"StackScript that allow for supplying customized parameters during deployment.",
			Computed:    true,
			ElementType: udfObjectType,
		},

Here's the change: https://github.com/linode/terraform-provider-linode/commit/d47b869cfc34121e9de2b6f5e354fbfa7106ecbd#diff-5af75137b7aae5257e45b292214d3df63f67baa252fcb900de8e663727a515f4L77
It used to be Computed+Optional upstream meaning that it used to be overridable from the user program. It is no longer. I think this explains the breaking change.
Are we good to go here @aq17 or any other concerns?

https://github.com/linode/terraform-provider-linode/pull/804 MongoDB removal

Here's the change: linode/terraform-provider-linode@d47b869#diff-5af75137b7aae5257e45b292214d3df63f67baa252fcb900de8e663727a515f4L77
It used to be Computed+Optional upstream meaning that it used to be overridable from the user program. It is no longer. I think this explains the breaking change.
Are we good to go here @aq17 or any other concerns?

@t0yv0 that makes sense, think it's good 👍🏼


LeaderLayer anchor must come before FollowerLayer in paint order, but the reverse was true.pen

When I used TextField widget
the following error occurred
════════ Exception caught by scheduler library ═════════════════════════════════
The following assertion was thrown during a scheduler callback:
LeaderLayer anchor must come before FollowerLayer in paint order, but the reverse was true.
'package:flutter/src/rendering/layer.dart':
package:flutter/…/rendering/layer.dart:1
Failed assertion: line 2502 pos 7: 'leader._lastOffset != null'

Please use v1.0.0 which I have fixed the bug :)

I already using it.

I see. Then that is bug of flutter 2.10 stable, which will be solved in 2.13

see https://github.com/fzyzcjy/flutter_portal/issues/56#issuecomment-1070959360 . Though that post is about how I fix bug of this repo, that post also tells you why this error message is a bug of flutter

Thanks

I got something similar when I tried to compose portal-target > portal > portal-target or portal-target > portal-target.
════════ Exception caught by scheduler library ═════════════════════════════════
LeaderLayer anchor must come before FollowerLayer in paint order, but the reverse was true.
'package:flutter_portal/src/enhanced_composited_transform/flutter_src/rendering_layer.dart':
package:flutter_portal/…/flutter_src/rendering_layer.dart:1
Failed assertion: line 383 pos 7: '_debugCheckLeaderBeforeFollower(forwardLayers, inverseLayers)'

With some experimentation, I found a workaround for nesting level of two, not sure about higher nesting.
Use portal-target inside portal-target without any portal between them in the hierarchy and use below anchor on inner portal-target:
anchor: const Aligned(
    follower: Alignment.center,
    target: Alignment.center,
    alignToPortal: AxisFlag(x: true, y: true),
    portal: Alignment.center,
)

I got something similar when I tried to compose portal-target > portal > portal-target. portal-target inside portal-target worked but inner one was in background instead of foreground.

I have had this error also when having this nest, but I have fixed it. Could you please give a reproducible sample, and ensure it fails in 1.0.0?

I got something similar when I tried to compose portal-target > portal > portal-target. portal-target inside portal-target worked but inner one was in background instead of foreground.

I have had this error also when having this nest, but I have fixed it. Could you please give a reproducible sample, and ensure it fails in 1.0.0?

I tried to change flutter channel to beta but it's still the same problem..

Interesting. Could you please reproducible sample, possibly by giving a github repo

please check this file https://github.com/ElsayedDev/errors/blob/main/portal_text_field_bug.dart

I see. could you please make it shorter to ease debugging

I got something similar when I tried to compose portal-target > portal > portal-target. portal-target inside portal-target worked but inner one was in background instead of foreground.

I have had this error also when having this nest, but I have fixed it. Could you please give a reproducible sample, and ensure it fails in 1.0.0?

Oh, it could be me then, I was trying to do some weird stuff. I will try to untangle my codebase and reproduce the error in a new app.

Take your time.

@ElsayedDev is your problem fixed? if so how? I'm facing the same issue right now myself

Here's the minimal reproducible example.
when user taps on the TextField the the assertion fails,
the only solution  I was able to find was to comment out the assertion in the flutter sdk source code
import 'package:flutter/material.dart';
import 'package:flutter_portal/flutter_portal.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const Portal(
      child: MaterialApp(
        title: 'Flutter portal layers bug',
        home: MyHomePage(title: 'Flutter Demo Home Page'),
      ),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  bool _showPortal = false;

  void _togglePortal() {
    setState(() => _showPortal = !_showPortal);
  }

  ElevatedButton _buildPortalChild() {
    return ElevatedButton(
      onPressed: _togglePortal,
      child: const Text('Toggle portal'),
    );
  }

  Widget _buildPortalFollower() {
    return SizedBox(
      width: 400,
      height: 300,
      child: Card(
        child: Center(
          child: TextFormField(
            decoration: const InputDecoration(
              label: Text('Text form field'),
              border: OutlineInputBorder(),
            ),
          ),
        ),
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: PortalTarget(
          visible: _showPortal,
          anchor: const Aligned(
            follower: Alignment.topCenter,
            target: Alignment.bottomCenter,
          ),
          portalFollower: _buildPortalFollower(),
          child: _buildPortalChild(),
        ),
      ),
    );
  }
}

environment:
sdk: ">=2.17.0 <3.0.0"
flutter version: 3.0.0
and
flutter_portal: ^1.1.0

Good reproduction! So, to make reproducible sample smaller, does _showPortal matter here? Or can we remove that button and _showPortal logic?

Once it gets so simple, we can use things like https://docs.flutter.dev/testing/code-debugging debugDumpLayerTree etc to see what is happening (with a very small tree - clean and easy-to-debug result)

To fix it or make a workaround, needs to find the cause of bug, which needs a good reproducible sample :)

Good reproduction! So, to make reproducible sample smaller, does _showPortal matter here? Or can we remove that button and _showPortal logic?
Similar applies to TextFormField.decoration - can we remove it?
And TextFormField, can we use TextField. etc
And remove the Card

Nope these do not matter it was for the easier understanding of the behaviour from the user perspective

To fix it or make a workaround, needs to find the cause of bug, which needs a good reproducible sample :)

I might have to downgrade to flutter 2.10.x which did not had this problem, ah but now I'm using the new super param fields of the constructor, it's going to be a lot of work

do you want be to simplify the code?

Yes. Simplest code possible makes it easy to debug and fix.

Good reproduction! So, to make reproducible sample smaller, does _showPortal matter here? Or can we remove that button and _showPortal logic?
Similar applies to TextFormField.decoration - can we remove it?
And TextFormField, can we use TextField. etc
And remove the Card

Here's you go.
import 'package:flutter/material.dart';
import 'package:flutter_portal/flutter_portal.dart';

void main() => runApp(
      Portal(
        child: MaterialApp(
          home: Material(
            child: Center(
              child: PortalTarget(
                anchor: const Aligned(
                  follower: Alignment.topCenter,
                  target: Alignment.bottomCenter,
                ),
                portalFollower: SizedBox(
                  width: 400,
                  height: 300,
                  child: TextFormField(),
                ),
                child: const Text('Portal child'),
              ),
            ),
          ),
        ),
      ),
    );

Great! The next step is to add a reproducible sample in https://github.com/fzyzcjy/flutter_portal/blob/master/test/widget_test.dart. Maybe you need pumpWidget and tester.tap

I mean I do not see errors when using lib in my app

hmmm but we know the error only happens when we placed a TextField widget inside the portalFollower, so this is an issue and your widget tree configuration is different and might remain different forever...

I agree. But an open source library maintainer has no time to fix each and every bug as he/she has full-time jobs to do.

I also get the same issue when putting a text field inside a portal. I'm on the latest flutter stable version 3.0.1.

I get the issue on macOS and web. Not sure if you have tried to reproduce it there.

Just some added details to this issue. In web release mode it works fine. In debug though it throws an error.

@caleb654 Came across the same issue with TextField inside the portal. But the EditableText widget works fine.

If you cut and paste the entire Modal example into a new project and swap out the Hello World Text with a TextField, it throws this error.

I get this issue with a slider inside a portal entry as well.

I believe it has something to do with the default Portal placement trying to render a TextField or any other element that needs to be a child of MaterialApp or a Scaffold. For example moving Portal from above to under MaterialApp seems to solve the problem. Or like @ abhimanyupandian said, using EditableText which probably doesn't depend on as much also works.

@fzyzcjy on that note, can you show an example of properly using multiple portals? Seems to work inconsistently for me. My goal is to use a portal above the router that persists on navigation and another below that will transition with the page. Is that possible?

can you show an example of properly using multiple portals?

I personally just put Portal and PortalTarget whenever I want it and has no problem...
I do have cases, when Portal 1 is above MaterialApp, and Portal 2 is under Scaffold.

Btw feel free to debug into the issue! Probably just create a reproducible sample, and use https://docs.flutter.dev/testing/code-debugging#layer-tree etc to dump all trees to see what is happening

Any update on this?

Feel free to reproduce and make a PR! I currently do not have much bandwidth on this since working on https://github.com/fzyzcjy/flutter_smooth

My rule of thumb with open source packages I maintain is that if I have to suggest people contribute more than twice, it's time for me to schedule some time to fix it myself.

@lukepighetti I have just spent a full month (without holidays or rests) working on https://github.com/fzyzcjy/flutter_smooth, which "achieves ~60 FPS, no matter how heavy the tree is to build/layout", and have submitted (https://cjycode.com/flutter_smooth/insight/status) >30 PRs to Flutter framework and engine, and that one is quite hard because nobody (even googlers) thought that approach could work (but indeed it works after a month).
So please give me a bit time to rest (and finish remaining work of smooth) :)

@lukepighetti I have just spent a full month (without holidays or rests) working on https://github.com/fzyzcjy/flutter_smooth, which "achieves ~60 FPS, no matter how heavy the tree is to build/layout", and have submitted (https://cjycode.com/flutter_smooth/insight/status) >30 PRs to Flutter framework and engine, and that one is quite hard because everybody (even googlers) thought that approach would not work before (but indeed it works after a month). So please give me a bit time to rest (and finish remaining work of smooth) :) I do all open-source libraries completely for free.

As for the direction to solve this issue, I guess dumping the whole layer tree (and render object tree etc) on the minimal reproducible sample as well as some logging may help.

I'm having this issue with two PortalTargets (nested) when the visibility on the descendent changes from true to false
flutter: LeaderLayer anchor must come before FollowerLayer in paint order, but the reverse was true.
flutter: 'package:flutter_portal/src/enhanced_composited_transform/flutter_src/rendering_layer.dart':
flutter: Failed assertion: line 384 pos 7: '_debugCheckLeaderBeforeFollower(forwardLayers, inverseLayers)'

We  are getting exactly the same issue with nested PortalTarget. We are trying to show/hide portal based tooltip inside portal based popup window. We use MouseRegion.onEnter / MouseRegion.onEnter to show/hide tooltip.
I just found a workaround which solves the issue in our project.
Instead of doing
PortalTarget(
  isVisible: <visibility>,
  child: <child>,
);

I am doing
PortalTarget(
  child: Visibility(
    isVisible: <visibility>,
    child: <child>,
 ),
);

So this works for me for now, hope this helps somebody as a temporary workaround.


Example code suddenly stopped working

Library version: all
Repro steps, stacktrace, screenshots:
Code was working for a solid year unchanged, just like the example:
config = new SessionConfiguration.Builder()
                .setClientId(getString(R.string.uberClientId)) //This is necessary
                .setRedirectUri(getString(R.string.uberRedirectURI)) //This is necessary if you'll be using implicit grant
                .setServerToken(getString(R.string.uberServerToken))
                .setClientSecret(((MyApplication)getApplication()).getUberSecret())
                .setEnvironment(SessionConfiguration.Environment.PRODUCTION) //Useful for testing your app in the sandbox environment
                .setScopes(Arrays.asList(Scope.PROFILE)) //Your scopes for authentication here
                .build();
loginManager = new LoginManager(accessTokenStorage, new LoginCallback() {
                    @Override
                    public void onLoginCancel() {

                    }

                    @Override
                    public void onLoginError(@NonNull AuthenticationError error) {
                        System.out.println(error.toStandardString());
                    }

                    @Override
                    public void onLoginSuccess(@NonNull AccessToken accessToken) {
                        ((MyApplication)getApplication()).setUberAccessToken(accessToken.getToken(), accessToken.getRefreshToken());
                    }

                    @Override
                    public void onAuthorizationCodeReceived(@NonNull String authorizationCode) {

                  }
                }, config, UBER_RESPONSE_CODE);
                loginManager.login(context);

            }
        });

Now any and all scopes, including general scopes setup in the developers center throw an INVALID_SCOPE. I suspect something changed in the uber app that is causing an incompatibility with the SDK. Nothing at all has changed on my side.
Expected Behavior: User Authentication

Have you ever found a solution to this problem?


GH-1900 Server responds with SHACL validation report if validation failed

GitHub issue resolved: #1900
Briefly describe the changes proposed in this PR:

Use the current exception handling to extract the validation report and output it as NQUADS in the response
Extend the error handling in the client to pick up this error and create an appropriate exception (preferably a ShaclSailValidationException, but that needs more work)
Detect this exception in the Workbench and print a nice turtle report to the user

This is a fairly hacky solution that gives us a fix that we can remain compatible with. By idea is that in the future we can use either a header or the http status line to inform the client that this should be treated as a ShaclValidationException. At the moment the data is just available as the message.

This is how it looks in work bench at the moment.

Would be nice to do something about the exceptions being thrown though, so people don't rely on how the exception looks today....since that's something we would like to change in the future.

This the output in the Workbench that I'm proposing now.

Got my issues numbers crossed, correct one is #1900

Will need to look into renaming the branch and the commits

Also: could you rename your branch please? Thanks.

Content-type approach looks good! Fine with that for a first time around.
As for the exception class and where it should go: I just realized that you can't put it in rdf4j-util and have it extend RDF4JException, because that is in rdf4j-model. Which is stupid, it shouldn't be there, it should be in util (which contains the common package, which is supposed to contain all this kind of generally reusable stuff). But, you know, legacy.  Stick it in rdf4j-model for now and we'll clean up the exception hierarchy and which module contains what as a separate issue.

While working on this I've created a separate module to see what I actually need. I've created an interface for the actual exception, and then implemented two classes for either of remote or local shacl exception.
I'm thinking of removing the model interface.
I've also changed up how the java based report looks. To avoid dependencies. Might also turn that into interfaces instead.
This is a sort of breaking change. It's fairly small though. Users will have to import a different package, but can use the same class name for handling the shacl exception. Those relying on the java based report will have to make more changes.
What are your thoughts @jeenbroekstra ?

I’m wondering if this might be ready now. I’ve squashed some commits. Still need to rename the branch before I merge it.

Still need to rename the branch before I merge it.

If it's easier, you can also just edit the commit message when you click merge:

Just change 'eclipse/issues/GH-1502_' to 'eclipse/GH-1502-' and then confirm.
The main reason I'm obsessed with the branch name is that I'm trying to make sure this commit message for the merge commit is easy to read and (in the github UI) actually makes the issue number a clickable link. To do that, it needs to be followed by a dash rather than an underscore.

@hmottestad  as a heads up: after thinking about my nagging you about squashing, and branch names, and all of that, I've reopened the discussion around merge strategies (see #2011 ) with a compromise that I hope makes both our lives easier. The TL;DR is to go back to preferring the use of  "Squash and merge" for all feature PRs, and optionally allow rebase in cases where you explicitly want preserve multiple commits.
This PR is probably a good example of where using 'rebase and merge' instead of 'squash and merge' makes sense: there's a limited number of commits, they're all meaningfully described, and because one of them is for a different issue than the main issue being solved, it makes sense to keep them separate.


Generate fstar on CI

Here is a starting point (note that it sill fails as per this failed job):
  build-fstar:
    runs-on: ubuntu-latest
    permissions:
      contents: write
    steps:
      - name: Checkout code
        uses: actions/checkout@v3
        with:
          ACCESS_TOKEN: ${{ secrets.ACCESS_TOKEN }}

      - name: Checkout hacspec-v2
        uses: actions/checkout@v3
        with:
          repository: 'hacspec/hacspec-v2'
          path: hacspec-v2-repo

      - uses: ocaml/setup-ocaml@v2
        with:
          ocaml-compiler: 4.14.x

      - uses<EMAIL_ADDRESS># TODO: change this to `stable` when we update nightly

      - run: cargo install --path cli/driver && cargo install --path cli/subcommands

      - run: opam install . --deps-only
        working-directory: hacspec-v2-repo/engine

      - run: opam exec -- dune build
        working-directory: hacspec-v2-repo/engine

      - run: |
          sed -i -E "s/(edhoc-crypto.*), default-features = false(.*)/\1\2/" Cargo.toml
          cargo circus fstar
        working-directory: hacspec

This is ongoing on #59.
Can build hacspec-v2 docker image, however the call to cargo-circus still fails. See here.

Solved in #81


Fix securityContext settings in job template

Intends needed to be fixed and containerSecurityContext value settings are used
What this PR does / why we need it:
When I want to use the options:
securityContext:
  runAsGroup: 65534
  runAsNonRoot: true
  runAsUser: 65534
  seccompProfile:
    type: RuntimeDefault
containerSecurityContext:
  allowPrivilegeEscalation: false
  capabilities:
    drop:
    - ALL
    - CAP_NET_RAW

the entries will not be set correctly in the resulting manifest
**Which issue this PR fixes:
fixes: #1087
Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Hi @andreasgeisslerdt , it will be nice if the conflicts with the base branch can be resolved.


Redaction App - Unable to Specify Pages in 'Save As'

Describe the bug in current situation
In the redaction app / doc reviewer, the OOTB 'Save As' functionality is not allowing users to specify pages to include in the save file. See images below.
This may have been introduced when we implemented the stitching. The 'Save As' option will be valuable to have working come go-live, as it was intended to be used to produce packages for consultations (as we will not have this functionality built yet). To produce consultation pages, users will need the ability to specify what pages to include (i.e., those flagged with the 'Consult' page flag and consultee).
Currently, the 'Specify Pages' in the 'Print' menu option works correctly (in that you can select specific pages from across the package). This is how the 'Specify Pages' under the 'Save As' menu option should work.
Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Medium
Chance of Occurring (high/medium/low/very low)
High
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Test-Marshal
See EDU-2023-889944
Steps to Reproduce
Steps to reproduce the behavior:

Select 'Save As' on a request in the redaction app
Select 'Specify Pages' and enter several page numbers or page ranges
Click off the open text field
See error

Actual/ observed behaviour/ results
Unable to use 'Specify Pages' to select a range of pages to save. This may also only be looking at the pages in the first file in the package.
Expected behaviour
A clear and concise description of what you expected to happen. Use the gherking language.
GIVEN a user has selected 'Save As'
AND selected 'Specify Pages'
WHEN the user enters multiple pages or page ranges from across various files in the package
THEN the pages will be accepted
AND when the file is saved, only the specified pages will be included
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.
Private Zenhub Image
Private Zenhub Image
This is working correctly in the 'Print' menu option, where I can specify pages throughout the package to include.

bumping estimate to 5, as the fix for this involved updating pdftron to latest version and testing + fixing bugs related to the update

working good , so closing it


SignalR Core on .NET Core 2.2 not connecting on Azure but works locally

Describe the bug
I've been using SignalR Core for quite some time and it have been working nicely. When upgrading versions it stopped working on Azure but it still works locally. I upgraded these packages:

The javascript client from 1.1.0-preview2-35157 to 1.1.0 (@aspnet/signalr)
The server client from 1.1.0-preview2-35157 to 1.1.0 (Microsoft.AspNetCore.SignalR)
The server from .NET Core 2.1.4 to 2.2

I did change the CORS settings since "AllowCredentials" is no longer allowed in combination with "AllowAnyOrigin". The CORS settings are only used for localhost development, in Azure environment everything runs on the same origin so no CORS is necessary.
Everything works perfectly in localhost but in Azure (I host the solution as an app service) the client cannot connect to the server. No error, no log, only "pending" connection, hubConnection.start() does not finish.
I've tried to apply CORS even for Azure but it does not help.
It is worth noting that the Azure environment uses HTTPS but localhost does not.
Expected behavior
That the connection establishes correctly.
Screenshots
Additional context
dotnet --info:
.NET Core SDK (reflecting any global.json):
 Version:   2.2.200
 Commit:    27f814f6ba

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

Host (useful for support):
  Version: 2.2.2
  Commit:  a4fd7b2c84

.NET Core SDKs installed:
  2.1.600 [C:\Program Files\dotnet\sdk]
  2.2.200 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

It is somewhat similar to https://github.com/aspnet/AspNetCore/issues/4483 (but CORS does not help). It is also somewehat similar to https://github.com/aspnet/AspNetCore/issues/4389 (but I do not use MessagePack).
Some relevant code snippets:
Startup.cs :: ConfigureServices:
services.AddSignalR();
Startup.cs :: Configure:
app.UseSignalR(routes =>{ routes.MapHub("/ws"); });
websockets.ts:
try {
  let options = {
    logMessageContent: true,
    transport: HttpTransportType.WebSockets
  };
  this.hubConnection = new HubConnectionBuilder()
    .configureLogging(LogLevel.Error)
    .withUrl(this.uri, options)
    .build();

  this.hubConnection.on('MessageFromServer', (data: any) => {
    this.handleMessage(data);
  });

  this.hubConnection.onclose = e => {
    this.log('connection closed');
  };

  this.hubConnection.start()
    .then(() => {
      this.log('Hub connection started');
      this.connectionEstablishedEvent();
      this.checkConnection();
    })
    .catch(err => {
      this.log('Error while establishing connection');
      this.connectionFailedEvent();
      setTimeout(() => { this.connect(); }, this.RECONNECT_DELAY);
    });
} catch (exc) {
  this.log('websockets - exception:', exc);
}

Any ideas?

Ok, after spending a couple of days I finally solved it (just after posting the question of course). The problem was hidden in the exception in the catch clause. I use typescript and webpack to bundle the solution. Previous versions of @aspnet/signalr had some kind of babel issue so I had to use a babel-loader to transpile the js code. This seems to have changed between 1.1.0-preview2-35157 and 1.1.0. By removing the babel-loader from node_modules/signalr the code compiles correctly.
When using the babel-loader I got an exception in the signalr source: RangeError: Maximum call stack size exceeded.


Make applications.TeamMember extend users.User plus chunk changes

Summary

request presences when chunking on guild create if possible
Make applications.TeamMember extend users.User

Checklist

[x] I have run nox and all the pipelines have passed.
[x] I have made unittests according to the code I have added/modified/deleted.

Related issues

Could you please add integration tests for this too? I believe they will fail as you need to implement eq (check guilds.Member for more info 👍)

The equality override and integration tests is already there, this pr just adds in delegation.

Oh, didnt see that 😅


Disclaimer to be updated since it's mainnet soon?

Rationale
On the settings screen, there's a legal part with a disclaimer link leading to this info: https://testnet.spacemesh.io/#/disclaimer
Very soon we will have the Mainnet, so it should be removed or the link should be updated to some other (not-testnet) info.
Prerequisites

User story

[ ] I open smapp
[ ] I go to the settings screen
[ ] I can see the "Legal" section
[ ] I click the link and get a disclaimer information

Acceptance criteria

[ ] The basic scenario works as described
[ ] the content of the resource linked to this sections relevant to the situation

Out of scope
How to implement
How to test

#1668 is an updated one


Extract data used on knooppuntnet.nl

Love the site, and I don't really have an issue with the data, but I'd like to extract the data used on knooppuntnet.nl (so the routes and the knooppunten). I'm usually extracting data from Overpass Turbo. But the data I extract from there appears to be a bit messy. For example, I'm having some issue to locate the 'knooppunt numbers' from OSM. I'm using this query:
[out:json][timeout:600];
(
relation"network:type"="node_network";
);
out meta;

;
out skel qt;
--
Maybe the clean data used on knooppuntnet.nl can be extracted from somewhere? It would be very helpful for me :)

Regards,
Niek

The knooppuntnet logic retrieves all data from its own Overpass Api database.  After analysis, the information is written to a mongodb database.  I have made extracts from the mongodb nodes and the routes collections for you to download:

https://knooppuntnet.nl/public/nodes.json.gz (13Mb)
https://knooppuntnet.nl/public/routes.json.gz (207Mb)

Does this help?

Wow! What a nice early Christmas gift, this saves me so much time! Thank you very much Marc. Regards, Niek


Rename variables in code to match document names

Rename all of the variables in all of the code files to match the names of those variables in the document to improve readability and debugging later on.

The bulk of this is finished.  The bequest and income variables need to be changed in SS and TPI.  Also, the code should be run to make sure that everything runs (deleting all pickles first to make sure everything is fresh).

The bequest and income variables have been changed.  Code has been tested, it runs.  Everything now corresponds to the document.


[doc] fix some links

Update oss docker image links to new Elastic registry
Fix some other links

backported to 6.8, 7.8and 7.x branches


Updated index.md for Spanish language

The index.md file has been translated directly from the original file in English into Spanish.

I'll trust your Spanish and just approve/merge.
Thanks for the pull-request, @MoisesMN !


dockerize for demo code

Run code in a docker container with all dependencies for demo purpose

I will do it.. Do assign this to me..


Overuse of power when inactive

Issue Description:
What happens:
Any Machine over uses RF/I power when inactive
What you expected to happen:
Machine to stop drawing power when inactive
Steps to reproduce:

Place any machine
Connect any creative power to a cell/bank
cell/bank to any machine...
can reproduce with industrial machine

Affected Versions (Do not use "latest"):

EnderIO: 1.12.2 - 5.0.26
EnderCore: .0.5.31
Minecraft: 1.7.10
Forge: <IP_ADDRESS>05
FTB Revelations

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

That not any machine, that's a simple alloy smelter. It has power loss when inactive. Upgrade to a normal alloy smelter to stop this.

But (going by the screenshot), 3.2 MRF in 5s?

Simple machines drain 1 RF/t. Add the machine buffer size to initial powercell drain. And test more thoroughly next time, you have 3 machines connecting and god knows what the third is and how it affects the tests.

Creative energy from draconic. I do believe powerloss is configurable


Client Allocation Request for: Pinata

Client Allocation Questions
Core Information

Name: Pinata
Website / Social Media: https://pinata.cloud
DataCap Requested: 5TiB
Addresses to be Notarized: f17kamajrdzjcjzj6b3y2ovc64kc7qxvkbx2dzvza
Notary Requested: philippbanhardt

Hey @obo20 - thanks for you request. I know we know each other well and Pinata is a highly respected collaborator in the Filecoin ecosystem, however for sake of process and transparency: can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks

sure thing! I wasn't how much information you were looking for as part of the application process.
Myself: CTO of Pinata ( an IPFS pinning service ).
Some notable social media links of ours:

Twitter: https://twitter.com/IPFSPinata
Blog: https://medium.com/pinata
Email<EMAIL_ADDRESS>News mention: https://twitter.com/Filecoin/status/1319372971685838849?s=20

History: We've been working with IPFS for about 3 years now and have been running Pinata for most of that time. Pinata offers our users an easy, user friendly way to store data on decentralized storage networks. Our biggest contribution is most likely our blog series that I linked above. We've placed a lot of importance around education and want to continue educating the community by sharing all the things we've learned while building on IPFS / Filecoin. Right now our services are purely IPFS based, but we've been working on a Filecoin integration for a few months now.
Type and source of the data: The data we're looking to store on filecoin will be snapshots of our users' IPFS pinsets. Essentially we'll give users the ability to take a snapshot of their pinset at any point in time and back it up to filecoin for 6 months.
How we intend to store the data on Filecoin: Right now we're planning on just storing to one miner (until we get that perfected and then will move to multiple miners). We have a preference for miners in NA / EU.

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzaceb2m3npowgmz42s5lakwmdjh5lwc5ignbrgzqpcs5owsv23evdvv6

Address

f17kamajrdzjcjzj6b3y2ovc64kc7qxvkbx2dzvza

Datacap Allocated

5 TiB


1530 qids for replacing damaged bar codes

Motivation and Context

In case theres some damaged barcodes on peoples responses during the Census, the business needs to have some extra qids they can use to link to the correct case.
What has changed

Added two qid batch files containing for England & wales and Northern Ireland.

How to test?

Should be able to generate qids based on the files

Links

Trello

 Create Run Book for QIDs for replacing damaged bar codes - not for print (5)


Type hints and enhancements

Hello, I've added some small improvements as explained below:

Added appropriate type hints that provide clear documentation for the methods' expected inputs and outputs
Instead of using strpos, we use str_starts_with, which is more concise
Added support for Less and Stylus and used in_array for the $entryIsStyle

Edit: For some reason I get 'The system cannot find the path specified.' when trying to run tests on local

Thanks for the PR! Not sure why the test are failing locally (I'm using ubuntu via WSL2) and can run them.

If you wan't to appear in the release message you can use conventional commits in your PRs (I edited this release message manually to include you). Thanks again for the contribution :)


Authenticated rate limit

Authenticated basic rate limit (after authentication) for all endpoints exposed by the underlying APIs. All endpoints share the same bucket, so if rate limit exceeded for one endpoint, it will also exceed for the remaining endpoints of the same APIProduct.
apiVersion: networking.kuadrant.io/v1beta1
kind: APIProduct
metadata:
  name: animaltoys
spec:
  information:
    description: My super nice API Product.
    owner<EMAIL_ADDRESS>  routing:
    hosts:
      - api.animaltoys.<IP_ADDRESS>.nip.io
    expose: true
  APIs:
    - name: dogs
      namespace: default
      tag: production
      mapping:
        prefix: /dogs
    - name: cats
      namespace: default
      tag: production
      mapping:
        prefix: /cats
  securityScheme:
    - name: MyAPIKey
      apiKeyAuth:
        location: authorization_header
        name: APIKEY
        credential_source:
          labelSelectors:
            secret.kuadrant.io/managed-by: authorino
            api: animaltoys
  rateLimit:
    authenticated:
      max_value: 5
      period_in_seconds: 30
    preAuthenticated:
      max_value: 10
      period_in_seconds: 60

Implementations details:

It reconcilliates changes in the rateLimit.authenticated object. Removing the rateLimit.authenticated section, the controller will disable auth rate limit.
It is specifically designed to be applied after auth http filter.
Rate limit is based on user_id
Integration with APIKey and OIDC authN
Global rate limit
Limitador as the gRPC rate limiting service
Upgraded Authorino to v0.4.0

Depends on #46

Closing in favor of #48


Overriding Expired items deletion operation in SQL server cache table - ExpiredItemsDeletionInterval

Hi Team,
we are using SQL Server for distributed cache implementation using (Microsoft.Extensions.Caching.Distributed) "IDistributedCache" , the property "ExpiredItemsDeletionInterval" which trigger operation to delete all expired cache records in cache table creating lot of performance and deadlocks in DB server.
We are looking for the solution to disable this operation and have a simple DB job to delete cache records overnight everyday instead of deleting from application/API.
Issues with ExpiredItemsDeletionInterval property is set with default value to 30 min and no way to set it to null.
Implementation details:
We have developed a Microservice which is hosted in Openshift container platform with Pod configuration min 24 to Max 32 as this API is used by all other microservices for caching.
As ExpiredItemsDeletionInterval is not able to disable, each pod will trigger command to DB to delete all  expired records and resulting in deadlock on cache table and DB will be killed hence due this all microservices will be impacted.
Is there a better way to override this ExpiredItemsDeletionInterval property so that deletion can be stopped  ??
Thank you.
Hiremath

ExpiredItemsDeletionInterval =  TimeSpan.FromDays(2);

@Rick-Anderson : I don't want to set any time to delete expired cache records, looking for the solution to disable this operation, we have a separate DB Job to delete these expired records.
As i mentioned in problem statement, if I set ExpiredItemsDeletionInterval = sometime; then each and every Pod in Openshift container Platform trigger delete operation and ending db deadlock.

TimeSpan.FromDays(365 * 3);

Issues with ExpiredItemsDeletionInterval property is set with default value to 30 min and no way to set it to null.

https://stackoverflow.com/a/62142926/502537


Concourse Readiness/Liveness Probe fails

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Version of Helm and Kubernetes:
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.7", GitCommit:"0c38c362511b20a098d7cd855f1314dad92c2780", GitTreeState:"clean", BuildDate:"2018-08-20T10:09:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"9+", GitVersion:"v1.9.7-gke.5", GitCommit:"9b635efce81582e1da13b35a7aa539c0ccb32987", GitTreeState:"clean", BuildDate:"2018-08-02T23:42:40Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}
Which chart:
Concourse
What happened:
Readiness/Liveness Probes failed.
Liveness:   http-get http://:atc/ delay=120s timeout=5s period=10s #success=1 #failure=3
Readiness:  http-get http://:atc/ delay=5s timeout=1s period=10s #success=1 #failure=3
What you expected to happen:
Probes to work
How to reproduce it (as minimally and precisely as possible):
helm install --name concourse stable/concourse -f values.yaml
Anything else we need to know:
Events:
Type     Reason                 Age                 From                                                  Message

Normal   Scheduled              43m                 default-scheduler                                     Successfully assigned concourse-web-6bdcc4cf45-wg76s to gke-cluster-default-pool-bec82955-0rtc
Normal   SuccessfulMountVolume  43m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  MountVolume.SetUp succeeded for volume "concourse-keys"
Normal   SuccessfulMountVolume  43m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  MountVolume.SetUp succeeded for volume "concourse-web-token-lbgzh"
Normal   Pulling                43m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  pulling image "concourse/concourse:4.2.1"
Normal   Pulled                 42m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  Successfully pulled image "concourse/concourse:4.2.1"
Normal   Created                42m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  Created container
Normal   Started                42m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  Started container
Warning  Unhealthy              40m (x3 over 40m)   kubelet, gke-cluster-default-pool-bec82955-0rtc  Liveness probe failed: Get http://<IP_ADDRESS>:8080/: dial tcp <IP_ADDRESS>:8080: getsockopt: connection refused
Normal   Killing                40m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  Killing container with id docker://concourse-web:Container failed liveness probe.. Container will be killed and recreated.
Normal   Pulled                 40m                 kubelet, gke-cluster-default-pool-bec82955-0rtc  Container image "concourse/concourse:4.2.1" already present on machine
Warning  BackOff                8m (x45 over 26m)   kubelet, gke-cluster-default-pool-bec82955-0rtc  Back-off restarting failed container
Warning  Unhealthy              3m (x150 over 42m)  kubelet, gke-cluster-default-pool-bec82955-0rtc  Readiness probe failed: Get http://<IP_ADDRESS>:8080/: dial tcp <IP_ADDRESS>:8080: getsockopt: connection refused

I have the same issue, do you have any update ?

Same issue here with rabbitmq-ha :
Warning  Unhealthy         28s (x18 over 1m)  kubelet, worker1  Readiness probe failed: Get http://<IP_ADDRESS>:15672/api/healthchecks/node: dial tcp <IP_ADDRESS>:15672: connect: connection refused

Kubernetes version : v1.11.3

In fact my mistake came from a dns problem : Kubedns was not able to connect to the api server.
To fix it, I add this rules in my iptables (was missing) :
iptables -P FORWARD ACCEPT

I installed concourse in the sandbox k8s cluster helm install --name concourse stable/concourse.


[Performance] General performance investigation for opening app from customer profile trace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

What performance issue do we need to solve?
We had a customer meeting and they had really bad performance. We recorded multiple profile traces with them.
This issue is about the profile trace for app open.
The collection of issues:

https://github.com/Expensify/App/issues/46590
[Partially backend issue] https://github.com/Expensify/App/issues/46591
https://github.com/Expensify/App/issues/46593
https://github.com/Expensify/App/issues/46594
https://expensify.slack.com/archives/C049HHMV9SM/p1722434442673559
https://expensify.slack.com/archives/C049HHMV9SM/p1722438425183629

What is the impact of this on end-users?
Slow or completely hanging/blocked app when starting.
List any benchmarks that show the severity of the issue
https://share.firefox.dev/4fgLVcm
Proposed solution (if any)
None yet, I will go through the profile and see what can be optimised, what exactly caused those lags.
List any benchmarks after implementing the changes to show impacts of the proposed solution (if any)
not available yet
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Version Number: v9.0.11-5
Reproducible in staging?: not tested
Reproducible in production?: yes
Email or phone of affected tester (no customers): customer
Logs: See performance file
Notes/Photos/Videos: See attached video
Expensify/Expensify Issue URL: n/a
Issue reported by: @hannojg
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1721919928992729
View all open jobs on Upwork

cc @sakluger (feel free to assign me as I (or someone from my team) will work on this ticket!)

Hey @hannojg, any updates on this collection of issues?

Working through the issues one by one, currently working on this one:

https://github.com/Expensify/App/issues/46591

I will also find someone from my team to help with these performance issues!

Looks like we are adding some timing performance logging: https://github.com/Expensify/App/pull/46807

I'm going to change this to a Weekly since it's a tracking issue related to four specific performance investigations.

One important thing we just found out.
So someone from our team created in old dot ~2000 reports. When I opened new dot OpenApp took in total ~3 min to complete (and also returned over 20mb of data).
During that time the whole app seemed block, presumably due to the sequential queue waiting for OpenApp to finish.
I feel like this is something we maybe want to improve on the backend?
During that time I was seeing a constant skeleton loader (I actually thought the app would be broken).
cc @chrispader

Hanno distributed the issues above to a few other people so we can work on them in parallel. Otherwise, no major updates. I'll be OOO for the next week so I will check back then.

How are these coming along @hannojg ?

Hey @hannojg, do you have any updates on any of the performance issues we're tracking here?

We are working on a new PR here which improves the search speed by 2456x (344ms to 0.14ms):

https://github.com/Expensify/App/pull/48652

I am OOO from 09.09-22.09, please assign @kirillzyusko meanwhile as he will take care of finishing the PR while I am away!

Hey @kirillzyusko could you please leave a comment here so I can assign you to the issue? Thanks!

@sakluger yes, sure, feel free to assign me on this ticket 👋

No updates. @kirillzyusko have you been working on that WIP PR, or are we waiting for Hanno to return?

@sakluger yeah, I've been working. I cleaned up the implementation, tested more carefully and discovered, that a new implementation produces a different search output (i. e. sometimes it doesn't find users). Currently investigating that and trying to fix!

Hey @hannojg and @kirillzyusko, any updates on the WIP PR?

Provided an update here:

https://github.com/Expensify/App/issues/46591#issuecomment-2374350034

PR should become ready today!!

Looks like steady progress is being made in the PR review.

Looks like all reviews are complete on https://github.com/Expensify/App/pull/48652, but it's failing one test. Hopefully this will be live very soon!

The PR was merged but seems to have caused a deploy blocker: https://github.com/Expensify/App/issues/51123

This went to prod!

Hm, did it? I think the PR was reverted

I am currently working on re-applying + fixing issues

No updates, we're still working on getting this fixed.

How is this going @hannojg?

New PR will be ready in a moment:

https://github.com/Expensify/App/pull/51954

Hey @hannojg, what's the latest? Looks like there is a bit more cleanup needed before we can merge the PR you linked above?

Yes exactly, would be awesome if we could work through that list here:
https://github.com/Expensify/App/pull/51954#issuecomment-2464282215
After those are merged I can work on the next set of cleanup PRs. The final goal is to cleanup the existing code so that i can reuse a stable ordering logic for the new performance PR :)

Latest update here: https://github.com/Expensify/App/pull/51954#issuecomment-2485511954

There is one cleanup PR left, then a few additional changes needed to fix an ordering bug.

The final cleanup PR has been created (more info: https://github.com/Expensify/App/pull/51954#issuecomment-2511170844)

As of yesterday, cleanups are complete. Now Hanno is working on fixing the Search suffix tree implementation PR. Latest update here: https://github.com/Expensify/App/pull/51954#issuecomment-2539512480

Still testing things in the PR.


feat: make MsgUnpacker more lenient when parsing number types, issue #81

Addresses issue #81

Didn't run the tests first, will resubmit with a working more concise solution


Add twoslash support

PR Checklist

[x] Addresses an existing issue: closes #43
[x] That issue was marked as status: accepting prs

Overview
Adds support for the following syntax:
const square = (x: number) => x * x;
const four = square(2);
//    ^? const four: number

Multiline types look like:
const vector = {
  x: 3,
  y: 4,
};
vector
// ^? const vector: {
//        x: number;
//        y: number;
//    }

A comment line is a continuation of the expected type if it starts with the exact same sequence of spaces and // as the ^? line (with ^? replaced with more spaces).
This also includes a fixer. I've only enabled it for twoslash-style assertions, but enabling it for $ExpectType would be trivial. This might really improve the dev experience for DefinitelyTyped contributors!
Here are some GIFs of it in action.
Single line fix:

More complex fix that involves multiple files / a multiline type:

This should be good to go! I imagine @JoshuaKGoldberg is busy putting the final touches on his book, so no rush on reviewing. But I would love to start using this in my project: https://github.com/danvk/crudely-typed/blob/main/test/type/select.typetest.ts

Haha you're right, I am a bit swamped this week (sorry!). But I hope to review as soon as I can!

We're scheduled to go over this on Twitch tomorrow 😎 https://www.twitch.tv/joshuakgoldberg/schedule?seriesID=c53bbfdd-92bd-487b-bb14-1ffc5965c191


Creating a "stats page" on http.

start of a flask to show some stats on http.

:+1:

consider using a simpler httpd and static pages instead?
maybe use stagit? http://git.2f30.org/stagit/log.html

I avoided httpd because i wouldn't want to deal with having to kill my processes if/when the bot dies/restarts. In my opinion this approach is easier because the sever is killed when the script stops. The server in the server file uses httpd and solved it with the application name. maybe that is a better approach.

Agreed, using external tools would lead to an unmaintanable project.
I guess it leaves us with to choices:
Don't use any external tools or use something like ansible to manage everything.

I think it makes more sense to do one http server for everything and since the other server is already up and running i'm retracting this PR.


Change error detection logic

Report agent unreported only when there are no updates for 2+ report intervals.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


WindowBuilder UI Support

Should download the version (source code perhaps) and build it, have it ship with the engine

This already works with the downloaded one, there is no benefit to supporting this for the engine.


Problem php artisan tracker:tables

When I tried to run
php artisan tracker:tables
Am getting
[ReflectionException]                                                              Method PragmaRX\Tracker\Vendor\Laravel\Artisan\Tables::handle() does not exist
Running in Laravel 5.5
PHP 7.1.10
Please let me know how to fix this.

Same Issue
I Think The Package Not Compatible With Laravel 5.5.20

I ran these commands:
php artisan vendor:publish --provider="PragmaRX\Tracker\Vendor\Laravel\ServiceProvider" --tag="config"
php artisan vendor:publish --provider="PragmaRX\Tracker\Vendor\Laravel\ServiceProvider" --tag="migrations"
php artisan migrate


Great theme

Thank you for this excellent theme. I'm delighted with it. Do you have plans to create a version of this theme for Visual Studio Code as well? I will be using both editors since there are features that Zed does not currently support. Having a consistent theme on both sides would be ideal.

I don't have any immediate plans for a VS Code theme, because I just don't use it often enough anymore to justify the time and effort that it would require.
I did find a couple of themes on the Visual Studio Marketplace that seem to be based on the same color palette. I spot-checked a handful of common colors in their source repositories, but I haven't tried either of these myself, so I can't make any guarantees about how complete they are or how consistent the experience would be with this theme in Zed. Might be worth a try, though.

JetBrains New UI Dark Theme
JetBrains New Dark


JS is disabled alert

Pre-Submission Checklist

[X] Your pull request targets the staging branch of freeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[X] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[X] Tested changes locally.
[X] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16362
Description
Changed "JS is disabled" message from a span to a react-bootstrap Alert component in order to use the info styling.

@joshuaswift You have conflicts on this PR. Please resolve them.
Also, ensure npm run test completes without failing

I've now resolved the conflicts and ran tests again which all passed.


Set text format to markdown for Pypi

Trying to fix with https://github.com/mjmlab/barseq/pull/6. Online merge conflict resolving tool is not working at all. Merge conflict is also not showing up on github desktop, or terminal/