Player not running on Mobile data

<com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView
android:id="@+id/youtube_view"
android:layout_width="match_parent"
android:layout_height="@dimen/size_200dp"
android:background="@android:color/black"
app:measureBasedOnAspectRatio="true"
app:useDefaultControls="false"
app:videoScale="center"
/>
Player working fine on wifi but not working on mobile data.
Error: Please click here to watch this video on youtube

I am facing the same issue. The player is throwing this error "VIDEO_NOT_PLAYABLE_IN_EMBEDDED_PLAYER". when on mobile data but is working on Wifi.

This is most likely not an issue with the library but with YouTube's embedded web player.

I'm getting this error too VIDEO_NOT_PLAYABLE_IN_EMBEDDED_PLAYER, but it doesn't seem related to mobile data or at least doesn't always happen.

This may be a duplicate of #202.

@PierfrancescoSoffritti You are right the problem is not with the library but with Youtube iFrame API. But, this is not related to #202
and in my case, it is explicitly and consistently happening on mobile data, as the same video is streaming fine on Broadband.


Handle whitespace in entry values in the AAPT badging parser

Fixes https://github.com/flutter/flutter/issues/16200

Hey we are getting the same error pointed out in 16200. However going by the feed, the solution is not clear for the same. Can anyone help?


store: convention to remove active sessions for some identifier

There are multiple ways to go about this, the easiest would be to enforce a some key, like $userId, and if it exists in the session we store it in a new column. Koa-session still determines the session key, but the user is able to remove all sessions for a user with a simple delete query.

Project specific, depends on what they sotre in the session data.


Release Approach Link is dead

Bug Report
Describe the Bug
Clicking at release approach at https://github.com/eclipse-edc/Connector/blob/main/README.md#releases returns a non-existing page.
Expected Behavior
That hyperlink should point to one of the following:

https://github.com/eclipse-edc/docs/tree/main/developer/decision-records/2023-01-26-release-process
https://github.com/eclipse-edc/docs/blob/main/developer/decision-records/2023-05-17-release-process/README.md
https://github.com/eclipse-edc/docs/blob/main/developer/decision-records/2022-11-28-release-management/README.md

On a second thought, none of them are guiding enough. I would actually expect a document, ideally not dated, that shows the latest state of the release approach.
Observed Behavior
Directing to an 404 page
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://github.com/eclipse-edc/Connector/blob/main/README.md
Scroll to Releases
Click on release approach
See error

Context Information
I do not have knowledge about how the development of EDC is happening, I am purely a user.
Possible Implementation
Listed in Observed Behavior

the release document was moved here: https://github.com/eclipse-edc/docs/blob/main/developer/releases.md
could you please provide a PR to update the link?

@ndr-brt I have opened #3608 Thanks for pointing me to the correct location as well.


hooks do not work for tv channel buttons when tv is off

I want to remap the "Sling" button on my magic remote to open YouTube Ad Free. When the TV is on, this works fine. But the tv channel buttons can wake the tv and when that happens the hooks are ignored.
For when the TV is truly powered off (Quick Start+ is off or power is lost or TV crashes) then that probably isn't possible to hook into it fast enough to redirect. But when Quick Start+ is on (the default behavior), the TV doesn't truly power off, and I theorize that the hooks might be able to work in this state.

Sorry for the late response, but no, this still isn't possible, as when Quick Start+ is on the tv basically suspends everything running when turned off.


Code wrapping for real (try #2)

Another approach.  Updated index file so you can see some examples.

are <line>...</line> elements really a thing though? I can't find them in any standard? I like the solution and the semantics of <line> I just can find anything to reference that's suggests its a real html tag.

Live example: http://markbucciarelli.com/2016-06-22_parsing_text_with_erlang_pattern_matching_and_guards.html
Renders fine on the following browsers:
OSX, Chrome Version 51.0.2704.103 (64-bit)
OSX, Firefox 47.0
Windows 8 Mobile (IE 11)
Android Chrome <IP_ADDRESS>.arm

@mbucc thanks for providing the live example. Looks great! My only gripe is that the image does not appear to be retina. Would you mind adding retina support?

@mbucc thanks for the effort, works pretty good on mid-sized screens and resolutions, but sadly the issue is still present on smaller widths.
To see what I mean, open up the live example in Firefox, then use Firefox's Responsive Design Mode and try out various resolutions, such as 320x480.

@aminb When I open in 320x480, the only issue I see is that long strings without whitespace don't wrap.
For example, the parse_time_part in fourth link in the attached screen shot is actually parse_time_parts(strip_tokens(S,.
But I don't see what you can do about that with CSS, that is more about how the browser decides to wrap.

@adamschwartz  I don't think I can increase the pixels, as the pixel height of each arrow must match the pixels of the code line-height.  (This is the main fragility of this approach).
But I can try to add more colors than just a single gray, which will make the arrow look smoother.
Would that solve your objection?

@mbucc what if you adjust the background-size property?

What can I do to finish this?  Here are the only two issues I know of:

reduction in code font not applied consistently
address @adamschwartz retina gripe
Are there others, or is this not heading in the right direction ...

@mbucc I just committed a version which addresses the issues I had. It uses a  ::before pseudo-element so background-repeat can be used and uses an SVG image for non-pixelated scaling.

@adamschwartz  Looks great, arrows are cleaner and perfectly centered vertically.
One thing I miss is the full width code.  With a pre tag like this:
<pre class="code with-wrapping fullwidth">

I don't get full-width code block unless I include this css rule:
pre.code.fullwidth { width: 90%; }

Can you put that back in?

That’s a little out of scope of this issue (by its title). Please create a new issue and I’d be happy to address it there. However, I’ll likely want to defer to @daveliepmann on how best to handle that. Thanks!

That arrow is really nice, Adam. Unfortunately I find line wrapping not served well by a consistent indent. Line wrapping well is hard! The proposed solution here demonstrates how complex the topic is: some elements are indivisible (e.g. single quoted HTML attributes), there are logical places to split lines (e.g. the end of an HTML tag) that need to be detected, and often the degree of indentation should match the first line (for instance, in HTML or lisps). The web folks I've showed this to in person agree.
I think the relevant principle here is that given the choice, I'd rather have 'simple imperfect solution' than 'imperfect solution that introduces complexity'. That's why Tufte CSS doesn't provide general-case classes for tables. It's why Tufte CSS, as I see it, should avoid most problems of displaying code! Tufte CSS covers the most basic case with the principle of least surprise: you format your code the way you want it; we will mark it as code with a font, and provide a home for it. People who want more functionality can reach for tools like embedded GitHub snippets, which provide syntax highlighting, line numbering, and access to the raw text.
One way I do think this should be fixed is with smaller text within code snippets, so that line wrapping happens less frequently.

Agreed!

@daveliepmann Thanks for your considered opinion!
I will maintain a fork with this patch, as to my eyes is it more correct than either (1) wrapped lines with no visual indicator or (2) hidden content that requires a scroll right.  (And my blog contains a lot of code.)
This is a typesetting challenge that currently has no good solution!

@mbucc Best of luck. Communicating code is indeed one of the hard things.

For reference to future readers, Google Code Prettify is another possibility alongside embedded Gists.

Done: https://github.com/mbucc/tufte-css_with-codewrap.  Thanks @adamschwartz!


feat: Add types

Why do we duplicate the catalogue model? Does seem a good idea.

As we can not use catalogue model in this repo all dependant code was moved.


[bgfx] New ports: bgfx, bx, bimg

bgfx and family of libraries.

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

Closing PR due to inactivity, If you have time to come back to this then please just reopen it.  Thanks!

@vii1 any plans to continue work on this port?

@Outurnate whilst this isn't related to those issues, this project could deem useful: https://github.com/JoshuaBrookover/bgfx.cmake

@handsomematt currently what I'm using to build it locally

May I ask why this PR was closed? I did test it right now and it works just fine. I'll be happy to provide another PR based on those, just with repo hashes updated to reflect the latest status.
Let me know if interested and I'll be more than happy to credit @vii1 as well as add bgfx to the ports.

@julianxhokaxhiu I would be interested in using this as well so a PR would be appreciated from me at least.

Is the port dead ?

I would love to see bgfx available with vcpkg.


Pond service: provide GET route which shows the status of the tanks

Pond service should provide all Redis data related to tanks, when queried at GET /tanks.
Temp & pH are fine to start with, though we may want to expose some of the other "trivia" data, later.
Crib notes on data structures
Return value example
[
     {
           "tank": { "id": 1, "name": "The Mothership" },
           "temp": { "f": 82.18, "c": 27.88 },
           "ph": { "val": 8.11, "mv": 500.15 }
     },
     {
           "tank": { "id": 2, "name": "The Pond" },
           "temp": { "f": 83.1, "c": 28.39 },
           "ph": { "val": 7.98, "mv": 488.33 }
     }
]

Rust struct example

/// A struct to hold some data from the HTTP request
/// for temp/ph info.
#[derive(Debug, Serialize, Deserialize)]
pub struct TankStatus {
   pub tank: Tank,
   pub temp: Temp,
   pub ph: Ph,
}

#[derive(Debug, Serialize, Deserialize)]
pub struct Tank {
   pub id: i32,
   pub name: String,
}

#[derive(Debug, Serialize, Deserialize)]
pub struct Temp {
   pub f: f32,
   pub c: f32,
}

#[derive(Debug, Serialize, Deserialize)]
pub struct Ph {
   pub val: f32,
   pub mv: f32,
}

Implemented in #49

curl -k -H "Authorization: Bearer $FIREBASE_JWT" https://$FIREBASE_HOST/tanks | python -m json.tool

[
    {
        "id": 1,
        "name": "The Mothership",
        "ph": 7.72,
        "ph_mv": 461.05,
        "ph_update_count": 2454127,
        "ph_update_time":<PHONE_NUMBER>,
        "temp_c": 27.56,
        "temp_f": 81.61,
        "temp_update_count": 2535781,
        "temp_update_time":<PHONE_NUMBER>
    },
    {
        "id": 2,
        "name": "The Pond",
        "temp_c": 24.62,
        "temp_f": 76.32,
        "temp_update_count": 191511,
        "temp_update_time":<PHONE_NUMBER>
    }
]


[branch-2.0][opt](jdbc catalog) close when jdbcClient is not empty

Proposed changes
Issue Number: close #xxx
pick #32747
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Patch tls test

@objectiser @jpkrohling Please review

@kevinearls Can you create the PR to revert this change - and the subject clearly prefixed with "[REQUIRED 1.15.0]" (or something like that).

@objectiser  See https://github.com/jaegertracing/jaeger-operator/pull/732


Create aggregate function

I'm trying to create an aggregate function to concat arrays even if they're empty, as described here
Running this directly on Postgres works
CREATE AGGREGATE array_accum (anyarray)
(
    sfunc = array_cat,
    stype = anyarray,
    initcond = '{}'
);

But I don't know how to (or if it's even possible) do that with alembic_utils.
I tried adding this to my functions.py file:
ARRAY_ACCUM_DEFINITION = """
(
    sfunc = array_cat,
    stype = anyarray,
    initcond = '{}'
);
"""

array_distinct = PGFunction(
    schema='public',
    signature='array_accum(anyarray)',
    definition=ARRAY_ACCUM_DEFINITION
)

But if fails because the SQL code outputted tries to create a normal function, instead of an aggregate one, and I couldn't find any references on how to create aggregate functions
[SQL: CREATE FUNCTION "public"."array_accum"(anyarray) (
    sfunc = array_cat,
    stype = anyarray,
    initcond = '{}'
)]
(Background on this error at: http://sqlalche.me/e/13/f405)

So what would be the right way to do this?
Thanks in advance, and thanks for making this library, is great to have things much cleaner

Aggregate functions aren't currently supported
The create statements are pretty similar, which makes it look like it'd be a small change to add, but aggregate functions' properties are stored in a different view, pg_aggregate vs pg_function, and the properties are significantly different.
Due to those differences, it needs its own class to correctly detect when a replace operation is needed.
I don't currently have a plan to implement them but would consider a PR that doesn't reduce coverage
Since that functions definition isn't likely to change much you could create it using raw sql in the migration. Since it won't be registered with the pg_function view, alembic_utils wont attempt to drop it in subsequent revisions that use --autogenerate.
Alternatively, you could avoid needing a custom aggregate by unpacking the arrays and using array_agg
select
	array_agg(unpacked.elem)
from
	(
		VALUES
		('{5}'::int[]),
		(null),
		('{1,2,3}'),
		('{}')
	) abc(arr),
	-- Unnest the arrays
	lateral (
		select * from unnest(abc.arr)
	) unpacked(elem)

Hope that helps
Leaving this open to track changes wrt aggregate functions in the future

Thanks for the quick reply! I think I'll go with creating it directly on the DB.
That's the 1st aggregate function I ever created, so I'm missing too much of the big picture to be able to create a PR for this.


1inch origin /swap API param

As requested by 1inch, the origin query param should be passed to the /swap endpoint.
API docs

A bit more context: https://cowservices.slack.com/archives/C0371SB243E/p1718722940784089


about the data order of roi_pooling (cpu version)

I have read the code in "lib/layer_utils/roi_pooling/src/roi_pooling.c" which is the cpu version of roi_pooling. I found that the data order follow the NHWC format when indexing bottom data, but pytorch follow the NCHW format. it is my mistake or something else?

I actually never tried cpu version. It should be trivial to test. I'm busy recently; I would appreciate if you can test it.

you can check the lines 22~27
// data height
int data_height = THFloatTensor_size(features, 1); // it could be THFloatTensor_size(features, 2);
// data width
int data_width = THFloatTensor_size(features, 2); // it could be THFloatTensor_size(features, 3);
// Number of channels
int num_channels = THFloatTensor_size(features, 3); // it could be THFloatTensor_size(features, 1);
and line 87
const int index = (h * data_width + w) * num_channels + c; // it could be (c*data_height+h)*data_width + w
thanks for your great job

I actually didin't write these part. It's ported form other repository,

it is my mistake. The order of feature have been changed in  line 23 of "lib/layer_utils/roi_pooling/roi_pooling.py" :
_features = features.permute(0, 2, 3, 1)


Not found in the Gallery

"Sage Intaact" is no longer found in the Gallery

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

ID: 2809f192-e944-902e-db51-149939878611
Version Independent ID: 2d788b57-9c1a-3124-690f-4469bc552cee
Content: Tutorial: Azure Active Directory integration with Sage Intacct
Content Source: articles/active-directory/saas-apps/intacct-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@kphtx Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@chetansriv We need to fix the app name in the gallery and also fix the left navigation in the docs.

Issue got resolved as suggested above.
#please-close

Issue got resolved as suggested above.
#please-close


Make env files export compatible

In #28 I forgot to push a commit that makes env files export compatible :disappointed:.
So this PR adds support for env files like this:
MY_ENV=value

and this as well:
export MY_ENV=value

Thank you!


Added point functionality to Basic Calculator

Fixes Issue🛠

Closes #634
Description👨💻
Added additional functionalities in the basic calculator for the point "."
Type of change📄

[ ]  Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
Tested on various test cases manually.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Sir kindly label this PR with GSSOC.

It is working in the UI only, pls do the same when user type point from the keyboard @shivangi-singhjha

It is working in the UI only, pls do the same when user type point from the keyboard @shivangi-singhjha

Sir I have updated it. Kindly look into it.
Please label it under gssoc

Good, Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳 @shivangi-singhjha

Good, Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳🎉  @shivangi-singhjha

Good, Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳🎉 @shivangi-singhjha

Thank You! So much sir🙏


Add some functionalities

I added some functions for my startup blog, consider including them to lagom. Possibly mathjax, and email icon should be optional for general template, i added them as fixed feature.

Cool! I'm interesting in adding the email link to the main theme. Could you create a new PR with just those changes (or update this one)? I'm not planning to add MathJax or Disqus to the theme at this time; I think those are more specialized.

I'm not sure if it is possible to create pull request for one specific commit. Maybe you can fetch and then checkout only the social.html file?

@swanson can you re-consider your position on disqus? I think it's would be a really neat feature to have.

@clearnote01 This project is a theme -- it covers the styling and structure of a Jekyll site. When you use it, you are forking it into your own repository, so you can add whatever extra functionality you'd like.
I don't think that the majority of people will want to include Disqus comments out of the box, but those that do want it can add it to their own blogs/sites with modifying the template.


segfault due to lax Flow analysis and a bug in RangeForeachLowering

try to compile the following
enum e = func();
int func() {
  foreach(x;2..2) {
    return 1;
  }
}

Long story short, the problem is that the function doesn't terminate properly.
The flow analyzer actually detect function termination, but doesn't make it an error as assert(0); is not implemented at this stage.


Verify Serializer Should Honour Blacklist

With this patch, the TokenVerifySerializer checks the blacklist if the app is installed.
See Issue #239.

Closes #231
Hopefully that links with the issue so that the issue automatically closes.

Hey, I add your suggestions, thanks for them.
Now I have a problem I can't seem to fix. The test checks that the blacklisted token does not verify, fails. This is due to the RefreshToken constructor throwing a InvalidToken exception. I can't figure out how a freshly created Token for a valid user could not properly verify. Any hints?

Quick question: would it be better if we had Travis? Would probably make reviewing PRs and seeing where tests failed easier since it's embedded into the GH site.

Ok, now all tests pass.
An attacker could still infer that a token is an access token, but only if it is blacklisted, so I see no problem.

I think some form of CI would be great. Github has Actions, witch are pretty simple to set up.

Github has Actions, witch are pretty simple to set up. I see you use CircleCI.

Yea, not a fan of CircleCI; that was here before I started using GitHub. I like GitHub actions for CI/CD when it comes to testing certain things and publishing/deployment (e.g. AWS services testing without Moto and AWS deployment). But Travis CI has a nicer interface, especially for open source projects because the builds are individualized via the TOXENV. Additionally, testing the configuration of .travis.yml is a little more smooth and visibly more appealing.
Some great things about GitHub actions are like if a certain path has a file change, then execute an action, whereas Travis doesn't support that.

An attacker could still infer that a token is an access token, but only if it is blacklisted, so I see no problem.

Perhaps with certain configurations. Not too big of a problem for most Django websites since an attacker is not going to know some random website.

Hi guys
When is this going to be merged?

@auvipy do you mind reviewing this as well? I was waiting for David for a second opinion, but obviously he's absent now.


Fix dequeue race condition

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6f21dd1). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master     #10   +/-   ##
========================================
  Coverage          ?   97.5%
========================================
  Files             ?       1
  Lines             ?      40
  Branches          ?      12
========================================
  Hits              ?      39
  Misses            ?       1
  Partials          ?       0

Impacted Files
Coverage Δ

src/SerializingLink.ts
97.5% <100%> (ø)

Continue to review full report at Codecov.

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


Error on align types

Hello,
I'm facing a weird bug when I try to align different links. It seems that for some reason including numbers in the type name causes some issues.
Example:
The following doesn't work properly
d1 = [
 {'source': 'start', 'target': 'E02000314_1', 'value': 32.0, 'type': 't41037'},
 {'source': 'E02000314_1', 'target': 'E09000025_2', 'value': 32.0, 'type': 't41037'},
 {'source': 'E09000025_2', 'target': 'end', 'value': 32.0, 'type': 't41037'},

 {'source': 'start', 'target': 'E02000313_1', 'value': 15.0, 'type': 't39174'},
 {'source': 'E02000313_1', 'target': 'E09000025_2', 'value': 15.0, 'type': 't39174'},
 {'source': 'E09000025_2', 'target': 'E02000321_3', 'value': 15.0, 'type': 't39174'},
 {'source': 'E02000321_3', 'target': 'end', 'value': 15.0, 'type': 't39174'}]

sankey(links=d1, align_link_types=True)

while this does:
d2 = [
 {'source': 'start', 'target': 'E02000314_1', 'value': 32.0, 'type': 'Α'},
 {'source': 'E02000314_1', 'target': 'E09000025_2', 'value': 32.0, 'type': 'Α'},
 {'source': 'E09000025_2', 'target': 'end', 'value': 32.0, 'type': 'Α'},

 {'source': 'start', 'target': 'E02000313_1', 'value': 15.0, 'type': 'Β'},
 {'source': 'E02000313_1', 'target': 'E09000025_2', 'value': 15.0, 'type': 'Β'},
 {'source': 'E09000025_2', 'target': 'E02000321_3', 'value': 15.0, 'type': 'Β'},
 {'source': 'E02000321_3', 'target': 'end', 'value': 15.0, 'type': 'Β'}]

sankey(links=d2, align_link_types=True)

I'm happy to try and find a solution but I'd need some initial guidance as to where I should look for. I have a feeling this is an issue of js rather than python right?
Cheers

Hi @harisbal, thanks for the issue and sorry for the slow reply!
Yes, you're right this is probably in JS rather than Python -- specifically probably in the d3-sankey-diagram package, which ipysankeywidget wraps around.
What happens is that the align_link_types option sets the "ports" of the links -- link to code.
So I would probably start by testing directly in d3-sankey-diagram whether you see the same issue, using a list of links with sourcePort and targetPort set like this:
d1 = [
 {'source': 'start', 'target': 'E02000314_1', 'value': 32.0, 'sourcePort': 't41037', 'targetPort': 't41037'},
...
]

Hope that makes sense. If you're still up for trying to find a solution, please do have a go and ask if anything's not clear! You might find the example here useful for setting up a test with d3-sankey-diagram.

Update: I think it's not the numbers in the type name that are the issue, it's just the sort order of the type names.

t39174 < t41037 but
B > A

I'm not sure at the moment how this is causing a problem.

Hi @ricklupton
Can you please help me understand why dummyNodes are used? I have a feeling that for some reason a not required dummyNode is created in this example.
When I disable this part (by always returning an empty dummyNodes) the links are properly aligned.

Moreover, how could I test my changes to js in the ipysankeywidget. Modifying js files in js/lib does not seem to have any effect.
Apologies for all these questions..

This version does the trick although it is quite intrusive and it probably breaks other examples.
In an nutshell I sort incoming and outgoing edges inside the prepareNodePorts function (based on the port's name) and disable the orderEdges function.
Here you can find the example I'm working on


chore: prepare for release 3.8.2

Summary

Prepare for release 3.8.2

Pull Request Test Coverage Report for Build 1695

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.377%

Totals

Change from base Build 1694:
0.0%

Covered Lines:
4480

Relevant Lines:
4957

💛  - Coveralls


Feature: Turn System

Description
The turn system will be responsible for switching between the player and the Kaiju, keeping track of the current turn and the amount of turns per wave.

The game designer will be able to set the amount of turns per wave
The game designer will be able to set the positions of the Kaiju per turn

Functional Requirements

[ ] It should be able to switch the game state between the player and the Kaiju
[ ] It should be able to keep track of the current turn
[ ] It should be able to set the positions of the Kaiju per turn
[ ] It should be able to set the amount of turns per wave
[ ] It should be able to fire a UnityEvent when the turn is over/switched
[ ] It should be able to fire a UnityEvent when a new turn is started

Acceptance Criteria

[ ] There should be a turn system prefab that can be placed in the scene
[ ] It should use ScriptableObject to define and save the turns

Dependencies

#177
#183

Definition of Done

[ ] Follows the naming conventions
[ ] Follows the coding conventions
[ ] Meets the acceptance criteria
[ ] Pull request has been opened

Please add an option to fire a UnityEvent when the last turn is over


New Home Page 🎉

Tracking issue: https://github.com/opencollective/opencollective/issues/1123
This is a big PR that includes a new home page, as well as redesigned Header and Footer components. See the tracking issue for screenshots.

Merging as failed cypress tests pass locally. Need to work on making those less flakey.


How to set up for Caddyfile?

I am trying to set this up to autoformat my Caddyfile, but when I save I get the error "WARNING: The file has been changed since reading it!!!"
This is my config
require("conform").setup({
  formatters_by_ft = {
  caddyfile = { "caddyfile" },
 },
  formatters = {
      caddyfile = {
        command = "caddy",
        args = { "fmt", "--overwrite" },
        stdin = true,
        cwd = require("conform.util").root_file({ "Caddyfile" }),
        require_cwd = true,
      }
  }
})

fixed with changing
args = { "fmt", "--overwrite" }, which does print to stdout
to
args = { "fmt"}, prints to std out

The - argument can be used to tell caddy to read from stdin and avoid depending on cwd:
{
  command = "caddy",
  args = { "fmt", "-" },
  stdin = true,
}


Some of the recipes do not work until you unlock the recipe normally - Fabric 1.20.1

Some of the recipes such as the bowl and chest are unlocked and available in the recipe book when you have the required ingredients but selecting them does nothing. Having the instant craft enabled also does not craft them, you are only able to start crafting them from the recipe book once you have already crafted them manually.
Tested on both 1.8 and 1.8.2.

When you select them does it show a ghost recipe? Or does it just do nothing?
IIRC I didn't implement that code as I figured Minecraft would unlock those recipes when you gained the ingredients. Therefor I just setup a ghost recipe system.

If you don't have the ingredients in your inventory it will show a ghost recipe but if you have the necessary ingredients it will do nothing when you select it.

Yeah experiencing this

I've made a branch containing a WIP fix for this. It hasn't been tested much and doesn't currently support batch crafting.
It adds an option to the new recipes settings tab that allows you to enable forcePlaceRecipe where the client will manually place the selected recipe into the crafting grid.
I might work on this some more later.

@TauCubed can you make a PR for that branch just so we can track this

Done. #86

fix coming in 1.10.0 full release


Two simple adjunctions between setoids, categories and groupoids.

Following @JacquesCarette's suggestion I tried to mechanize a simple (or so I thought) adjunction. I failed to prove that the relevant pair of functors actually forms an adjunction (they probably don't), but ended up mechanizing some other stuff, which is in this PR now.
I don't think the PR should be merged as is, though.
My code probably doesn't follow the library conventions (I'm not sure what exactly these are) and I left quite a few comments in the code with questions around library design and such that I think should be removed first. My intention is to do that and update the PR if someone can guide me and help answer my questions.
Here is a summary of the questions/comments I have:

I'm not sure what should go into Categories.Functor.Instance and what should go into Categories.Functor.Construction.  For example, where should Categories.Functor.Instance.Core go?
I find it a bit confusing that properties and structures are sometimes named interchangeably. For example the Groupoid record is not really a groupoid, it's a predicate on categories that says that the category in question is a groupoid. In the standard library, there are usually two distinct records for structures and properties, e.g. Setoid and IsEquivalence or Poset and IsPartialOrder.  Maybe this convention could be used here too? E.g. Groupoid and IsGroupoid?
Should the Isos category defined in Categories.Morphism.IsoEquiv be renamed to Core? Generally, I wonder whether there is some official "reference" for naming things in the library. nLab?
I reimplemented some existing functionality/proofs because I did not realize that it was already included in some other part of the library. In general, I removed my duplicates when I noticed the original, but I might have missed some. I at least one case, I decided to keep a reimplementation that is equivalent but different (equality of isomorphisms).

Thanks for all the feedback @JacquesCarette!
I'll continue splitting this into smaller PRs and will address your comments there. Then I'll rebase this PR and see if there's anything left to cleanup (hopefully not).

@JacquesCarette, I think I addressed all your concerns about the remaining changes in this PR, so it should be ready for merging now.


[Download Bundle Error]

OS: darwin - x64
Error: Error
Message: ENOENT: no such file or directory, open '/Users/sorinvstanila/.vscode-insiders/extensions/tabnine.tabnine-vscode-3.2.10/binaries/3.3.96/x86_64-apple-darwin/TabNine.zip'
Stack: Error: ENOENT: no such file or directory, open '/Users/sorinvstanila/.vscode-insiders/extensions/tabnine.tabnine-vscode-3.2.10/binaries/3.3.96/x86_64-apple-darwin/TabNine.zip'

We've released a fix.
Duplicate of #383


Bounty for warm clothes does not work properly

Description

Most of the winter coats are counted as 2 warm clothes, so with only 4 winter coats the bounty is marked as completed.
Reproduction

skip bounties until the warm clothes bounty appears.
spawn a steel crate.
print and attach the bounty to the crate.
spawn 4 winter coats.
check the bounty status with a appraisal tool.
bounty marked as completed.
money glitch real.

Screenshots

https://github.com/space-wizards/space-station-14/assets/72604018/2b0bf785-3077-4511-ae15-80ab088fcaec
Additional context

Some winter coats are not affected.
The bounty works as intended for winter boots.

A lot of winter clothing also does not count as it for some reason, scarfs for example...

its counting the hoods as clothing
also i imagine it would count firesuits hardsuits etc as winter clothing, should just be changed to a tag all sensible items have


can't install packages in flatpak

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I am unable to get packages to install on the flatpak version of Atom
Steps to Reproduce

OS is Fedora 32 Silverblue
Install Atom - flatpak version from flatub
Install a package in Atom - settings/+install
I have tried installing atom-material-ui for example

Expected behavior:
The package should show under "Installed Packages" in settings/packages
Actual behavior:
no installation
Reproduces how often:
100%
Versions
Atom flatpak
Atom    : 1.48.0
Electron: 5.0.13
Chrome  : 73.0.3683.121
Node    : 12.0.0
OS is Fedora 32 Silverblue
Additional Information

Thanks for reaching out!
We've determined that you are currently using an unofficial build or distribution of Atom. Often these customized versions of Atom are modified versions of the Stable branch of Atom with mismatched versions of built-in components. These updated components are taken from the Beta channel or master branch and then injected into the Stable version and a new Atom package is generated. Because of the way Atom is constructed, using these mismatched components can cause mysterious and hard-to-diagnose problems. You can find out more about why we chose to not support unofficial distributions here.
You can find instructions for installing an official version of Atom in the Flight Manual. If you are still seeing this problem on an official build please file a new issue, thanks!


fix: correct conda package name

What was the problem/requirement? (What/Why)
Blender cloud submitter was defaulting with a conda package name that is inconsistent with the github and pypi package name.
What was the solution? (How)
Rename the package variables from "blender-openjd" to "deadline-cloud-for-blender".
What is the impact of this change?
Changes the default value.  Can be changed via deadline Conda Queue Environment, or manually in the submitter dialog.
When overriding python wheels, it should exclude the correct package.
How was this change tested?
In Blender, Render->Submit to AWS Deadline Cloud, Conda Packages line editor shows the package name.
Was this change documented?
Not applicable.
Is this a breaking change?
No

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hey jflatt-gia, thanks for creating a pull request!
Since the conda package in the deadline-cloud channel is called openjd-blender this would break the default workflow in service-managed fleets since deadline-cloud-for-blender does not exist. Can you explain a little more about your use-case for this change? The override adaptors should also still work.
Is it specifically just the inconsistent naming between repo/package (deadline-cloud-for-blender) vs. adaptor application (blender-openjd) being used?

Hey jflatt-gia, thanks for creating a pull request!
Since the conda package in the deadline-cloud channel is called openjd-blender this would break the default workflow in service-managed fleets since deadline-cloud-for-blender does not exist. Can you explain a little more about your use-case for this change? The override adaptors should also still work with the previous naming.
Is it specifically just the inconsistent naming between repo/package (deadline-cloud-for-blender) vs. adaptor application (blender-openjd) being used?

Okay, that makes sense, I was never able to get into the deadline-cloud channel.  Instead, I created a custom conda channel using the deadline-cloud-for-blender package name.
There's another issue where my values are being overwritten with these defaults every time.  If I can have it respect my Conda queue environment values, it would eliminate this issue.  It currently works great for the CondaChannels variable, but the CondaPackages variable gets replaced in shared_job_settings_tab.py using these default values.


Resolver file to use

Hello quick question, the resolver file got deleted out of the repo but when running resolve it needs the resolver file as a reference. is it still viable to use the previous one that existed?

Yes, it's usable (see https://github.com/opensanctions/opensanctions/issues/246), but if you continue to make your own judgements on it, it'll be out of sync with opensanctions.org.


CTRL + Backspace to remove previous word.

In word editors (word, libreoffice, google docs) CTRL + Backpace deletes the last word you typed.
Why?
This is a very useful feature when writing plaintext or even code. Right now when I mistype a word I have to mash the delete key many times to delete itl; with this feature it would remove the hassle.
Thanks in advance!
(btw i couldnt find a questions and answers section so I made an issue)
Commit hash: cfcb2e45
OS: Peppermint OS 10 (Based on Ubuntu 18.04)
Terminal: Sakura
edit: changed Delete to Backspace

This is to do with how different terminals encode Ctrl-Delete to the programs running in them. You can use the raw command in micro, then press Ctrl-Delete and see what input micro receives from the terminal, and then bind it manually in ~/.config/micro/bindings.json. That's what I've done too, since it's very useful.

In my case just "CtrlDelete": "DeleteWordRight" worked.

This is to do with how different terminals encode Ctrl-Delete to the programs running in them. You can use the raw command in micro, then press Ctrl-Delete and see what input micro receives from the terminal, and then bind it manually in ~/.config/micro/bindings.json. That's what I've done too, since it's very useful.

Hi, thanks so much for your quick response. If you look at the issue you'll see I've updated it. I meant to write CTRL + Backspace, not Delete.
I did try you solution and it did not work for me, here is the output of the raw command
Just Backspace:
EventKey: Backspace2: "\u007f"
CTRL + Backspace:
EventKey: Backspace: "\b"
I appreciate your help!

You can try
{
  "Backspace2": "DeleteWordLeft",
  "Backspace": "Backspace"
}

or "\u007f" and "\b" respectively. Some combination ought to work, although I recall some issue with binding the Backspace key...

So I tried that and nothing. Also, when I'm running micro it gives me the error:
"Backspace2 is not a bindable event"
Which is weird considering thats what it told me in the raw thing.
Would it be possible to fully remap the keys? or would that ruin everyting?

Usually terminals do not send a different event for backspace and control backspace (unfortunately). You can use alt-backspace, which is already bound to this by default.

Since your terminal appears to differentiate between Backspace and Backspace2, it might be possible to support this with a modification to micro's binding parser. In the code I treat Backspace and Backspace2 as equivalents because in my testing it seemed like terminals only ever used one or the other.

For the future record -- i had both ctrl-backspace and ctrl-delete working in both kitty and konsole

Usually terminals do not send a different event for backspace and control backspace (unfortunately). You can use alt-backspace, which is already bound to this by default.

This is actually a great solution, thanks so much!

For the future record -- i had both ctrl-backspace and ctrl-delete working in both kitty and konsole

I just installed konsole and tried it as well, works fine. Seems like its a terminal issue.
Thanks for your help!

Closing Remarks
While using the Sakura terminal and changing the binding to the above mentioned ones, it did not work.
When using micro on the konsole terminal (and possibly other terminals) with the same modified bindings it works flawlessly.
Semi-Solution
If your terminal is affected by this problem, one simple make-shift solution is to use Alt-Backspace
This is natively built into micro and nothing needs to be altered.
Thanks to @MasFlam and @zyedidia for their help!

Since your terminal appears to differentiate between Backspace and Backspace2, it might be possible to support this with a modification to micro's binding parser. In the code I treat Backspace and Backspace2 as equivalents because in my testing it seemed like terminals only ever used one or the other.

My terminal behaves like that and I'd be interested in this.
The backspace key gives Backspace2, and Ctrl-Backspace gives Backspace.

I just took a cursory glance at the code for this and it turns out this is already supported (somewhat confusingly though). Basically, you should use OldBackspace in your bindings file for Ctrl-Backspace (which is what the raw terminal pane lists as Backspace).

Awesome! I just tested and it work nicely.
This might be worth mentioning in the keybinds help page.
For info, I am using Windows Terminal (the new one not the old conhost), and I recall seeing the same behaviour back when I was using PuTTY.

this works for me
{
    "CtrlDelete": "DeleteWordRight",
    "OldBackspace": "DeleteWordLeft"
}

thanks

I just took a cursory glance at the code for this and it turns out this is already supported (somewhat confusingly though). Basically, you should use OldBackspace in your bindings file for Ctrl-Backspace (which is what the raw terminal pane lists as Backspace).

Great solution! Gnome Terminal also behaves that way, and will work with OldBackspace

I had a similar problem but removing a whole previous word didn't work only in micro opened from VSCode terminal.
I added to .micro/bindings.json field:
"Ctrl-W": "DeleteWordLeft"

And after it started to work not only the ctrl-w shortcut but also alt-backspace!
FYI:
› micro -version
Version: 2.0.11
Commit hash: 225927b9
Compiled on August 01, 2022

this works for me
{
    "CtrlDelete": "DeleteWordRight",
    "OldBackspace": "DeleteWordLeft"
}

thanks

Maybe this should be included in docs somehow to spare others the time for search. I think lots of people are coming to Micro for these "standard" key combinations.

I just took a cursory glance at the code for this and it turns out this is already supported (somewhat confusingly though). Basically, you should use OldBackspace in your bindings file for Ctrl-Backspace (which is what the raw terminal pane lists as Backspace).

As a thank you and in hopes that this is searchable if anyone else encounters it: this works perfectly on Black Box Terminal to reverse Backspace and Ctrl-Backspace bindings.


epub3 with ".." in the toc item url crashes on windows

try to open the following epub in launcher-windows
https://code.google.com/p/epub-samples/downloads/detail?name=page-blanche-20130322.epub&can=2&q=
it causes crash. The issue is in readium-sdk/ePub3/utilities/path_help.cpp in CleanupPath() function.
Here is a patch which fixes the issue. Please review and submit the change.
==== readium-sdk/ePub3/utilities/path_help.cpp
35c35
< 	for (auto pos = begin; pos < end; ++pos)
---
> 	for (auto pos = begin; pos < end;)
52c52
< 			components.erase(parent, dotDot);
---
> 			pos = components.erase(parent, ++dotDot);
54a55,56
> 		else
> 			++pos;

Let's revive this issue + patch proposal.
(1) First of all, the fix is in https://github.com/readium/readium-sdk/blob/develop/ePub3/utilities/path_help.cpp#L27
string CleanupPath(const string& path)
{
	static REGEX_NS::regex _PathSplitter("/");
	std::vector<string> components = path.split(_PathSplitter);

	auto begin = components.begin();
	auto end = components.end();
	for (auto pos = begin; pos < end; ++pos)
	{
		if (*pos == ".." && pos != begin)
		{
			decltype(pos) dotDot, parent;

			dotDot = pos--;
			if (pos == begin)
			{
				parent = begin;
				pos = dotDot;
			}
			else
			{
				parent = pos--;
			}

			components.erase(parent, dotDot);
			end = components.end();
		}
	}

	std::ostringstream ss;
	for (auto& str : components)
	{
		ss << str;
		ss << '/';
	}

	string result = ss.str();
	if (path[path.size() - 1] != '/')
		result.erase(result.size() - 1);

	return result;
}

(2) Secondly, the CleanupPath() function is used only by NavigationPoint::AbsolutePath() in (as far as I can tell):
https://github.com/readium/readium-sdk/blob/develop/ePub3/ePub/nav_point.cpp#L87
(3) Thirdly, is the proposed fix really specific to Windows? This seems like a generic path processing bug that should trigger on any platform, given the input EPUB3 "page blanche"
(4) Fourthly, I am concerned that multiple "path normalization" -related functions are inconsistently used, for example in PackageBase::ManifestItemAtRelativePath() at https://github.com/readium/readium-sdk/blob/develop/ePub3/ePub/package.cpp#L173 notably the usage of url_util::DecodeURLEscapeSequences(), or in the SMIL / Media Overlays path resolver (getReferencedManifestItem()) at https://github.com/readium/readium-sdk/blob/develop/ePub3/ePub/media-overlays_smil_model.cpp#L653 which performs its own canonical-ization logic! (e.g. removal of "parent_folder/.." path sequences)
(5) Fifthly, the download link for the EPUB "page blanche" sample is now: https://drive.google.com/uc?export=download&id=0B9g8D2Y-6aPLMmpCYktzbkJlQ1E (source: https://github.com/IDPF/epub3-samples/tree/master/30/page-blanche )
Note that this is also a ReadiumJS sample ebook: https://github.com/readium/readium-js-viewer/tree/develop/epub_content/page-blanche , see live demo https://readium.firebaseapp.com/?epub=epub_content%2Fpage-blanche&


Fix MiddleWare.md typo

grok isn't a word :)

It is: https://en.wikipedia.org/wiki/Grok

@timdorr  That's cute and all but maybe we should use words that people actually understand instead of le cute easter eggs in technical docs?

It's not an easter egg. It's a word: https://www.merriam-webster.com/dictionary/grok https://en.oxforddictionaries.com/definition/grok

It’s a cutesy developer word that requires someone who learned normal English to look up. It has no place in a technical document.
Jackson "Jake" Haenchen
<EMAIL_ADDRESS>832-423-3342

From: Tim Dorr<EMAIL_ADDRESS>Sent: Tuesday, May 22, 2018 2:07:12 PM
To: reduxjs/redux
Cc: Jackson Haenchen; Author
Subject: Re: [reduxjs/redux] Fix MiddleWare.md typo (#3004)
It's not an easter egg. It's a word: https://www.merriam-webster.com/dictionary/grok https://en.oxforddictionaries.com/definition/grok
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/reduxjs/redux/pull/3004#issuecomment-391105933, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHaVfx5X7St5E0SgEvXoA97L9XyURkqJks5t1GHggaJpZM4UJIKc.


[FLINK-22142][ci] Don't log Kafka connector output to the AZP console

This commit removes the console logging of the Kafka connector to the AZP console.

Thanks for the review @rmetzger. Merging this PR now.


Custom card

Where can I get the custom card for lovelace as is not installed with the integration.
resources:

url: /local/hass-xiaomi-mi-flora-and-flower-care/hass-xiaomi-mi-flora-and-flower-care.js
type: js

The Integration works well.

https://github.com/r-renato/hass-xiaomi-mi-flora-and-flower-care#manual-installation
or via HACS

Dear,
I used HACS but the *.js for the lovelace card did not install. I'm just following the instructions hoe to use the custom card in lovelace:
type: custom:xiaomi-mi-flora-and-flower-care-card name: "Zamioculcas Zamiifolia" entity: plant.plant_zamioculcas_zamiifolia

Aah yes i have the same Issue on my installation. The expected folder is nowhere.
But since I'm using the the Mi Flora card, I didn't noticed it.

I installed with HACS but did neither get the .js or the custom_components folder. Folder is fixed by manual installation, but how can I fix the .js?

Hi, I have update the custom component and fixed this.

Hello, after installing and configuring, only such a panel appears to me:
What could be wrong ???

Hi,
this can be a configuration problem.
Check if your configuration is ok or put it here if you want help.

HI grittisemper :)
I checked the configuration and it's probably ok.
The data displayed from the sensor works on the built-in plug-in while there is nothing through huahuacacao.
The attached file describes the configuration.
I have no idea where the problem is ...
huahuacaocao.txt

hi,
I think should be "draceana arborea" the right plant_id, not "dracena".

hi :)
I removed the flower from the mobile application, I configured the sensor from the beginning. Now the flower is "dracaena fragrans". I have corrected entries in the hassio configuration and now something is wrong, but the description has already appeared, but there is no information from the sensor. What could be the reason?

Hi, probably the specify huahuacaocao sensor name is not correct. Try to check it.


Cookies not being sent to webview for iOS

Bug description:

Web view should be rendering a web page that has a cookie authentication. Instead it is rendering sign in page. This is happening on iOS after I upgraded to react native 0.73+. It was working properly before the upgrade. Android is working fine.
I have both thirdPartyCookiesEnabled={true} and sharedCookiesEnabled={true} enabled. Also in handleNavigationStateChange I reset the cookies for iOS:
async function resetCookies() {
    const cookies = await CookieManager.getAll(Platform.OS === 'ios');
    Object.keys(cookies).forEach(key => {
      const cookie = cookies[key];
      if (cookie.domain === domain) {
        CookieManager.clearByName(url, cookie.name, Platform.OS === 'ios');
      }
    });
  }

I also tried to sync the cookies manually with this in the onLoad and the onNavigationStateChange of <WebView />, but it still didn't work.
  const synchronizeCookiesiOS = async () => {
    if (Platform.OS === 'ios') {
      try {
        const allCookies = await CookieManager.getAll(true);
        for (const cookieKey of Object.keys(allCookies)) {
          const cookie = allCookies[cookieKey];
          if (cookie) {
            await CookieManager.set('https://' + cookie.domain, cookie);
          }
        }
      } catch (error) {
        console.error('Error synchronizing cookies:', error);
      }
    }
  };

Expected behavior:

Web view should render account page, not sign in page
Environment:

OS: Android, iOS
OS version: Android 34+, iOS 14+
react-native version: 0.73.6
react-native-webview version: 13.8.1

One think that could be causing this is that you are not setting the webkit boolean to true here:
await CookieManager.set('https://' + cookie.domain, cookie);
In resetCookies function you are reading those cookies from WebKit WKHTTPCookieStore storage. Also you have to turn sharedCookiesEnabled for ios off, when using WebKit

Thanks @karel-suchomel-ed! That fixed it for the initial Webview load, but when I leave the webview and then return to it, the cookies are not being synced and I am back to the sign in screen.
Any ideas on why re-entering the Webview, the cookies don't work?
<WebView
          style={styles.webview}
          source={{ uri: url }}
          javaScriptEnabled
          domStorageEnabled
          startInLoadingState={false}
          injectedJavaScript={script}
          onLoad={synchronizeCookiesiOS}
          onMessage={() => {
            // nothing
          }}
          onNavigationStateChange={handleNavigationStateChange}
          thirdPartyCookiesEnabled={true}
        />

I think that for your use case you could use something similar that I used for sharing sessions between RN and WebView.
In our app we had a PHPSESSID cookie encoded in a JWT token and this worked for me (not 1:1 code):
const Example = () => {
  ...
  const [isReady, setIsReady] = useState(false)

  useFocusEffect(
    useCallback(() => {
      const getCookieHeaderString = async () => {
        const token = getUserAccessToken()
        if (token) {
          const cookie = getCookieFromToken(token)

          if (Platform.OS === 'android') {
            await CookieManager.clearAll(useWebKit)
          }

          await CookieManager.set(
            url,
            {
              name: 'PHPSESSID',
              value: cookie.sub,
              httpOnly: true,
              secure: true
            },
            useWebKit
          )
        }
        setIsReady(true)
      }

      if (customer) {
        getCookieHeaderString()
      }

      return () => {
        setIsReady(false)
      }
    }, [customer])
  )

  if (isReady) {
    return null
  }

  return (
    <Webview
      ...
    />
  )
}

The key was to set the cookie before every webview mount. I didn't experience any false positives this way, even tho the documentation warns about this. If you need to persist browser history between WebView mounts, you could write your own logic and handle the URL with state. I was using this hook for managing the history stacks:
export default function useBrowserHistory(homepage: string) {
  const [backStack, setBackStack] = useState<string[]>([homepage])
  const [forwardStack, setForwardStack] = useState<string[]>([])

  const visit = useCallback(
    (url: string) => {
      if (!backStack.includes(url)) {
        setForwardStack([])
        setBackStack((prev) => [...prev, url])
      }
    },
    [backStack]
  )

  const back = useCallback(
    (steps: number = 0) => {
      let i = steps
      const tempBackStack = [...backStack]
      const tempForwardStack = [...forwardStack]
      while (tempBackStack.length > 1 && i-- > 0) {
        tempForwardStack.push(tempBackStack[tempBackStack.length - 1])
        tempBackStack.pop()
      }
      setForwardStack(tempForwardStack)
      setBackStack(tempBackStack)
      return tempBackStack[tempBackStack.length - 1]
    },
    [backStack, forwardStack]
  )

  const forward = useCallback(
    (steps: number = 0) => {
      let i = steps
      const tempBackStack = [...backStack]
      const tempForwardStack = [...forwardStack]
      while (tempForwardStack.length > 0 && i-- > 0) {
        tempBackStack.push(tempForwardStack[tempForwardStack.length - 1])
        tempForwardStack.pop()
      }
      setForwardStack(tempForwardStack)
      setBackStack(tempBackStack)
      return tempBackStack[tempBackStack.length - 1]
    },
    [backStack, forwardStack]
  )

  const resetHistory = useCallback(() => {
    setBackStack([])
    setForwardStack([])
  }, [])

  return {
    visit,
    back,
    forward,
    backStack,
    forwardStack,
    resetHistory
  }
}

I only reset the cookies for Android, because I had some issues overwriting other cookies there.
You should also store that session cookie in secure storage (MMKV for example), so you can set it before WebView load, because session cookies are wiped. I took inspiration from this: https://stackoverflow.com/questions/62057393/how-to-keep-last-web-session-active-in-react-native-webview but instead of passing the cookie through cookie header in source I set it via the CookieManager.
I would not reset and sync cookie in the onNavigationStateChange function as that is unnecessary.


Style odd Rows through options

Hey,
Is it possible to change the background of a Odd row through the config options ? Looking at the docs I'm thinking I would need to use either trStyleCallback: row => {} or trClassCallback: row => null.  Is this correct? If so how would that look exactly ?
I could just override the css rules but looking to make sure there isn't an option available for this already. Thanks.

Its all in the CSS for now. And probably much easier to work with than the
row callback
On Wed, Jan 18, 2017 at 11:12 AM Hartley Jean-Aimee <
<EMAIL_ADDRESS>wrote:

Hey,
Is it possible to change the background of a Odd row through the config
options ? Looking at the docs I'm thinking I would need to use either trStyleCallback:
row => {} or trClassCallback: row => null. Is this correct? If so how
would that look exactly ?
I could just override the css rules but looking to make sure there isn't
an option available for this already. Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/31, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFUmCe5-6Z8C367BXZ7LK2p66F5PTb12ks5rTlYfgaJpZM4LnNtc
.


[FLINK-19945][Connectors / FileSystem]Support sink parallelism config…

…uration to FileSystem connector
What is the purpose of the change
give user access to configuring the sink parallelism of FileSystem.
Brief change log

add SINK_PARALLELISM into optionalOptions of FileSystemTableFactory
configure parallelism of writing file transformation both in streaming and batch in FileSystemTableSink
update docs, introducing sink.parallelism

Verifying this change
This change added tests and can be verified as follows:

FileSystemTableSinkTest ensure the configured parallelism is applied

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? ( docs)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@wuchong @JingsongLi  Would u mind taking a review?

@wuchong @JingsongLi  Would u mind taking a review?

Hi @shouweikun , I have went through the pull request. However, supporting sink parallelism for Hive and Filesytem is not just changing parallelism of the writer DataStream. We should first support ParallelismProvider for DataStreamSinkProvider first. Because if the sink parallelism is different than the upstream operator, we should implicitly add a keyby shuffle if there is changelog in the stream, otherwise the changelog will be out of order. See https://github.com/apache/flink/blob/95257a255f0da0a95b31647c6d057914d5748871/flink-table/flink-table-planner-blink/src/main/java/org/apache/flink/table/planner/plan/nodes/exec/common/CommonExecSink.java#L116

Hi @shouweikun , I have went through the pull request. However, supporting sink parallelism for Hive and Filesytem is not just changing parallelism of the writer DataStream. We should first support ParallelismProvider for DataStreamSinkProvider first. Because if the sink parallelism is different than the upstream operator, we should implicitly add a keyby shuffle if there is changelog in the stream, otherwise the chvangelog will be out of order. See
https://github.com/apache/flink/blob/95257a255f0da0a95b31647c6d057914d5748871/flink-table/flink-table-planner-blink/src/main/java/org/apache/flink/table/planner/plan/nodes/exec/common/CommonExecSink.java#L116

Hi @wuchong, thanks for ur review and comment. As far as I’m concerned,  it shall be under well discussed whether DataStreamSinkProvider inherits ParallelismProvider or not. Referring to FLIP-146, only SinkFunctionProvider and OutputFormatProvider  inherit ParallelismProvider.  Well, correct me if I'm wrong, both DatastreamScanProvider and DataStreamSinkProvider are designed for advanced user, which means that  once user choose DataStreamProvider, the more freedom the user has and the more responsibility the user take. What 's more, DataStreamSinkProvider inheriting ParallelismProvider  may still guarantee the parallelism, cuz user can do anything in DataStreamSinkProvider#getRuntimeProvider.

Hi @shouweikun , I have went through the pull request. However, supporting sink parallelism for Hive and Filesytem is not just changing parallelism of the writer DataStream. We should first support ParallelismProvider for DataStreamSinkProvider first. Because if the sink parallelism is different than the upstream operator, we should implicitly add a keyby shuffle if there is changelog in the stream, otherwise the changelog will be out of order. See
https://github.com/apache/flink/blob/95257a255f0da0a95b31647c6d057914d5748871/flink-table/flink-table-planner-blink/src/main/java/org/apache/flink/table/planner/plan/nodes/exec/common/CommonExecSink.java#L116

Coincidently, the code quoted above was written by me (LOL). Actually, the aim of  this exception is trying to preventing user from making DataStreamSinkProvider inherit ParallelismProvider.

@flinkbot run azure

@wuchong  Thanks for ur review!  I'll take care of the comments later.

@wuchong conflict fixed, CI passed, comments addressed, would u mind taking a review？

@wuchong Thank u so much for the code improvement. Learnt a lot from that. While the build seems failed, I‘ll try to figure it out~


Publishing to the topic accosiated with the different type

minimum_jerk_ros publishes to the topics [/translate/_action/feedback, /rotate/_action/feedback] messages of type [minimum_jerk_msgs/action/Translate_FeedbackMessage, minimum_jerk_msgs/action/Rotate_FeedbackMessage], but at the same time recoveries_server also publishes to that topic names but with the different type.
As minimal_jerk was created after the recoveries server I consider it a minimal_jerk bug :)

@arekin19 @cmrfulvius


Twig "debug" functionality broken

OctoberCMS Build: 2.2.33
DebugBar Plugin Version: 3.3.3
PHP Version: 7.x irrelevant

Description:
You've disabled twig functions in 279609efa38e2e62f22f75cca20d24191c1cabfb, which causes any pages using the twig "debug" function to fail with "Unknown "debug" function. Did you mean "date", "dump", "random", "include", "redirect", "placeholder"?"
A fix is to edit composer.json to fix the plugin version to 3.2:
"rainlab/debugbar-plugin": "3.2"

and then running
composer update
php artisan october:migrate

However this is hardly obvious and took me 2 hours to figure out. Other devs can run into the same. Please consider adding a note to explain this to README.md to save other devs the hassle until whatever the issue is that lead you to disable twig functionality is resolved?
Steps To Reproduce:
Add the current version of the plugin to a project:
composer require rainlab/debugbar-plugin

or upgrade an existing project:
composer update

Hey @lieszkol
Apologies for the frustration here. We anticipated this to be fixed rather quickly, instead, it has taken a bit longer than expected.
We will keep the Twig support around for v2 (43679d96a702ed52082015424009d4788db005d4), hopefully, it helps.

Thanks daftspunk!!! And thank you for the plugin.


Dead link in Readme.md

Getting a 404 for the link to LLVM-AMDGPU-Assembler-Extra in Readme.md

This is fixed.


Improve Mango test suite performance

Overview
877c886d added delays into the Mango test suites to improve stability. However, the failures were never reproduced locally and significantly increase the time that it takes to run the tests. This is a PR to experiment with removing those delays to see whether we can get the test suite stable in Travis without them.
Testing recommendations
Run the Mango test suite.
Related Issues or Pull Requests
#923
Checklist

[ ] Code is written and works correctly;
[ ] Changes are covered by tests;
[ ] Documentation reflects the changes;

This seems pretty solid now so I think is ready for review. I've run the latest commit through Travis >10 times without a failure. 697345f needs to be removed before merging to master.

also worth adding that this brings the time to run the Mango tests down from ~15 minutes to 20 seconds on my local machine.

+1, smart use of list_indexes to see if the index is available to use and to make sure our index is deleted. let's give this a try


mopidy-iris: 3.32.5 -> 3.33.0

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/issues/56940
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @Tomahna @worldofpeace

Other than that this should be fine

Unless you actually tried running it, I'd like to wait for feedback by a user.

Other than that this should be fine

Unless you actually tried running it, I'd like to wait for feedback by a user.

Note an explicit approval wouldn't have been in the format of a comment.

As far as I can tell it works fine. I'm not a user of lastFM though.


No support for Raw Bytes from scan

Some barcode implementations include a FNC1 character, specifically CODE_128's. These are necessary to detect in the event that a barcode contains two variable length values encoded to denote the end of one section, and the beginning of the next identifier.
I attempted to get the bytes from the rawData from a barcode, but the FNC1 character byte is missing. I've confirmed that the barcode does indeed contain one with zxing.

I've confirmed that our Code128 implementation doesn't handle FNC characters at all. We'll look into returning control codes in the rawValue for future releases.

Awesome. Thanks for giving me an update. We use them often in our mobile solution, and I'd love to read it in quickly in-app instead of outsourcing the functionality to zxing using Barcode Scanner.

The latest version of our Barcode API (part of Google Play Services 10.0) now returns FNC1 characters in Code128 results.


Nested Route

I was testing the router package to do a simple SPA with Link. It's seem that only standard redirect is supported. Is Nested Routes will be available ?
Or maybe I'm not using it the right way !
Something like that
InfernoDOM.render(
  (
    <Router history={browserHistory} >
      <Route path="/" component={ App } >
        <Route path="/todos" component={ Todos } />
        <Route path="/welcome" component={ Welcome } />
      </Route>
    </Router>
  ), document.getElementById("app")
)

// App with children
export default class App extends Component {
  render() {
    return (

        { this.props.children }

 )
  }
}

Thanks

@vinej I'll work to add this functionality ASAP :)

I think that it's working but not the way React do it,
I followed the example under the source code.
-> the Router must have the component set to the 'App' and Routes will be automatically nested under the App
<Router history={browserHistory} component= { App } hashbang="true"  >
      <Route path="/todos" component={ todos } />
      <Route path="/welcome" component={ Welcome } />
</Router>

for now, I don't need an extra layer of Nested Routes
You can close it
Thank

At the same time, I think that the Link component do not passes all props to the 'a' tag. I tried to set a className to the Link and it's not working
Thanks

@vinej Okay, that is clearly a bug. I'll look at that too :)

@vinej this should be fixed in 0.7.14. Can you confirm please?

This code below is working as expected with 0.7.14, but it was also working with 0.7.13. Here the master component { App } is defined into the Router tag
<Router history={browserHistory} component={ App } hashbang="true">
     <Route path="/welcome" component={ Welcome } />
     <Route path="/todos" component={ RequireAuth(todos) } />
     <Route path="/signin" component={ SignIn } />
     <Route path="/signup" component={ SignUp } />
     <Route path="/signout" component={ SignOut } />
 </Router>

The code below is working with React but not with Inferno. for this reason I use the method above
In the React version, the master component is defined into the Route tag and under you can used nested routes. With version 0.7.14 I don't have an error, but it's not working as React, because the nested components are not render as children of the App component.
<Router history={browserHistory} >
    <Route path="/" component={ App } >
        <Route path="/welcome" component={ Welcome } />
        <Route path="/todos" component={ RequireAuth(todos) } />
        <Route path="/signin" component={ SignIn } />
        <Route path="/signup" component={ SignUp } />
        <Route path="/signout" component={ SignOut } />
    </Route>
</Router>

export default class App extends Component {
  render() {
    return (

        { this.props.children }

 )
  }
}

render(
	<Router url={ '/foo/bar' } history={ browserHistory } component={ App }>
		<Route path={ '/foo' } component={ () =>
Bad Component
 }>
			<Route path={ '/bar' } component={ () =>
Good Component
 } />
		</Route>
	</Router>,
	container
);

In the above example, the nested route's component gets passed to the route's component. Is this not the correct behaviour?

@vinej is this not working as you expect it to still?

I switched to the  package for the routing.
I don't have a user case for you question, then I don't know if this is the correct behaviour.
Thanks

Is this being worked on? I need this to work:
InfernoDOM.render((
	<Provider store={ store }>
	    <Router history={ browserHistory }>
	        <Route path="/" component={ Layout }>
	        	<Route path="home" component={Home} />
	        </Route>
	        <Route path='*' component={NotFound}/>
	    </Router>
    </Provider>
), document.getElementById("app"));

export default class Layout extends Component {
    render() {
        return

        	{this.props.children}

;
    }
}

@deamme it's on the backlog for Inferno 1.0 :)


[BUG] When using post requests, you can add parameters after the url to crack signature authentication

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When using post requests, you can add parameters after the url to crack signature authentication
Expected Behavior
No response
Steps To Reproduce
When I catch a correct request on the network

I try to tamper with the parameter content, the program will report a signature error

However, when I concatenate the previous parameters after the url, I can crack the parameter signature

Environment
ShenYu version(s):2.5.0

Debug logs
No response
Anything else?
The problem here is that the parameter after the URL overrides the parameter with the same name in the request body

should put the request header and url parameters on one map and the request body on the other

There is no way to prevent the user from entering the same parameter name in parameter and body.

can you perfect him.

OK,I trry


feature/ldap_grouper_delta -> develop

See iss #16

Mistake on the base branch.  It should have been develop.


Meta: Make better release notes

Is your feature request related to a problem? Please describe.
I'm always frustrated when I want to know which new features are included in a new version of openslides like here:
https://github.com/OpenSlides/OpenSlides/releases/tag/4.0.0
Describe the solution you'd like
Make notes in new release.
Describe alternatives you've considered
Make readable changlog somewhere else and link to it in the release notes.
Additional context
Some good examples:
https://github.com/mastodon/mastodon/releases/tag/v4.0.0
https://github.com/Zverik/every_door/releases/tag/v4.0
https://github.com/alephdata/aleph/releases/tag/4.0.0

Our most-up-to-date release notes are here (engl./ger.):
https://support.openslides.com/help/en-gb/45-uber-openslides/80-versionshistorie
You can find this link in the legal notice from every openslides instance (e.g. https://demo.os4.openslides.com/info)
Additionally, you can find the faq if you want more in depth explanations of some features
Our github official realease information can be found here:
https://github.com/OpenSlides/OpenSlides/blob/main/CHANGELOG.md


firewall: tcp flags not applied if protocol is any

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md

[x] I have searched the existing issues and I'm convinced that mine is new.

Describe the bug
The firewall offers the option to introduce tcp flags within the advanced settings. This options are shown even if TCP isn't selected as the current protocol. I presumed that it would automatically create one rule for udp without the flags and one with tcp and it's selected flags. But this isn't the case if any protocol is selected the rule gets applied without any tcpflags.
To Reproduce
Steps to reproduce the behavior:

Go to 'Firewall'
Click on 'Rules'
Add new rule
Select protocol 'any'
Select some TCP Flags
Apply rule
Observe pfctl -sr

Expected behavior
I presumed that it would automatically create one rule for udp without the flags and one with tcp and it's selected flags. I don't know enough about PF if it's ignoring the protocol if tcp flags are set. I found that the default rule has S/SA activated and is applied to UDP and TCP.
Additional context
I could imagine three possible solutions to this problem.

State the observed behavior within the documentation or webinterface. Like for e.g.

Please notice that TCPFlags are only applied if the protocol tcp is selected.

Hide the TCPFlags within advanced menu if protcols UDP or any are selected.
Introduce two rules one for UDP without any flags and every other setting matching and one for tcp with flags set.
Introduce one rule which has the flags applied and protocol any. This depends on PF if it's allowed to create such rules.

The easiest solution would be the first one. But I would tend towards the second or fourth option as this makes it more convenient to use. Although I came up with the third option I would not go down this route as it's non verbose to create two rules out of one (I don't know why I presumed that this would be the implemented way).
Edit: I totally forgot to add that I would like to work on this if one solution is selected. Good opportunity to get involved.

my suggestion would be to keep it simple, how about https://github.com/opnsense/core/commit/04c1decd190bdefcab9d0f2a79acd3c641b05db5 ?
In which case it matches the parsing logic found here : https://github.com/opnsense/core/blob/eeae08415038e80285c100c5e4c425830adc40b3/src/opnsense/mvc/app/library/OPNsense/Firewall/FilterRule.php#L176-L191

Your implementation looks good to me, thanks for the fast resolving.

@tobiaskohlbau ok, thanks for confirming.


Provide documentation on which generated files / directories to commit to version control

After following the tutorial and executing the init and build commands, you end up with a directory with a lot of files and directories:
.
|-- .gradle
|-- android.keystore
|-- app
|-- app-release-signed.apk
|-- app-release-unsigned-aligned.apk
|-- assetlinks.json
|-- build
|-- build.gradle
|-- gradle
|-- gradle.properties
|-- gradlew
|-- gradlew.bat
|-- settings.gradle
|-- store_icon.png
`-- twa-manifest.json

As a web developer who knows precious little about Android development, I'm not sure which of those files should go into version control and which are generated during the build step. I guess you don't commit the generated apk files? What about the app folder and all the different gradle files / directories? Finally, do I commit the keystore or put it securely in a key vault (not sure if that file is encrypted ...)?
I get that it's only tangentially related to this library, but I think it would be great for the documentation to have a couple of pointer for people unfamiliar with Android development:

Which files should I commit to version control?
On a new system, how do I reproduce the build from version control?
How to handle the keystore file safely?

A related topic: I'm having trouble understanding all the different app signing methods and keystore formats (app signing key vs. upload key). A simple 'follow those steps to get the generated app in the play store' guide would be great, though I understand that's a lot of work. The current documentation just points to the app signing guide in the Android developer docs, but those prominently recommend using app signing by Google, and I have no idea how that relates to the signed app release generated by bubblewrap.
Since bubblewrap appears to be targeted towards non Android developers, it would be great to get some guidance for those topic in the documentation. Thanks!

Thanks for the feedback.

Which files should I commit to version control?

You'd want to add the /build folder, .apk files, .aab files, the .gradle folder to .gitignore. You probably want to ignore android.keystore too, but you want to save that to a secure place. Would it be helpful if Bubblewrap generated .gitignore along with the other files?
It's also interesting to note that, if you backup twa-manifest.json and android.keystore you will be able to regenerate the project by running bubblewrap update in the folder where they are. You can also do manual changes to twa-manifest.json and apply them to the generated project with bubblewrap update.

On a new system, how do I reproduce the build from version control?

Install bubblewrap on the new system and configure the JDK and Android SDK
Run bubblewrap build on the folder of the checkout.

How to handle the keystore file safely?

I wouldn't add android.keystore to the version control, specially if that's something like GitHub.
There's strict access control where only people who need to sign the APK for production and upload to the store have access.

I'd also opt-in into App Signing so Play can manage the key for you - when opting into Play Signing, the key generated by Bubblewrap becomes the "upload key" and the "signing key" is managed by Play. Otherwise, the key generated by Bubblewrap is the "signing key". With App Signing, you can change the "upload key" if you lose it or if it leaks. Without App Signing, it isn't possible to update it and losing means you can't update your app on the store anymore.

Regarding understanding all the options, have you seen this document: https://developers.google.com/web/android/trusted-web-activity/quick-start. Let me know if it helps and what's missing, and we'll be happy to update with more information.

@andreban Thank you very much for the reply!

Would it be helpful if Bubblewrap generated .gitignore along with the other files?

I'd say that would be a useful addition. Though from my perspective it would be enough to add a section to the documentation which explains the generated directory strutcture / files a bit for users who don't regularly work with Android Studio / Android apps. Like a simple list of the generates files/directories which tells me what the file is and if one should commit or otherwise save it (like the keystore).

when opting into Play Signing, the key generated by Bubblewrap becomes the "upload key" and the "signing key" is managed by Play.

Thanks, that's the piece I was missing! Maybe that can be mentioned in the Quick Start guide as well? It's probably obvious to most people, but wasn't for me ^^

Would it be helpful if Bubblewrap generated .gitignore along with the other files?

Oh yes, definitely. This would help us immensely.


Thoughts on Authorization Server Obtains End-User Consent/Authorization

From https://openid.net/specs/openid-connect-core-1_0.html#Consent

<IP_ADDRESS> Authorization Server Obtains End-User Consent/Authorization
Once the End-User is authenticated, the Authorization Server MUST obtain an authorization decision before releasing information to the Relying Party. When permitted by the request parameters used, this MAY be done through an interactive dialogue with the End-User that makes it clear what is being consented to or by establishing consent via conditions for processing the request or other means (for example, via previous administrative consent). Sections 2 and 5.3 describe information release mechanisms.

Since this is "MUST", I believe it would be great if boruta includes the consent phase as configuration option.
boruta-server has it as a part of boruta_identity (link) - btw.

The server is designed to keep user data out of the authorization domain brought by OAuth 2.0 / OIDC. I think the consent is to be implemented in the application layer along with the user business logic.
Does it make sense?

I agree that adding the whole consent feature to boruta core doesn't make sense - but we may add no-op callback or generate code for this.

How would you see such an update? As it is implemented in boruta-server, there is no interaction with the core domain.


Consider supporting json style field names

given a proto message
message SomeRequest{
    string some_field_name = 1;
}

the following would be an acceptable json payload when calling grpcannon
'{"someFieldName":"value"}'

currently, the above json would appear to return Unknown field name

Hello, thanks for reporting the issue. Can you provide more details on your usage please? We just pass the JSON data to grpc, so this error would be coming from the library itself (or possibly the server). Are you using a Node.js server? My understanding it automatically does the camelcase conversion, and should "just work". I;ll investigate further but some more details and context would be helpful.

Sorry, I kept starting to clarify and getting sidetracked.
I don't think this is an important feature, just mostly convenience to put the syntax closer to grpcurl.  I absolutely love how similar the 2 tools are, but since they are so similar when things I naturally expect to work don't, it can be surprising. This is one of those, but it's nit picky so take all this with a grain of salt.
Using this message as the example
message SomeRequest{
    string some_field_name = 1;
}

The field some_field_name is the literal string required by grpcannon inside thejson body of the request.  e.g. the request must be:
-d '{"some_field_name":"value"}'

Compared with grpcurl, where you could put either the cannonical json field name, someFieldName or the actual proto field some_field_name, e.g.,
-d '{"someFieldName":"value"}'

Doing the same in grpcannon results in Unknown field name
For posterity, here is what the grpcurl "describe" on the object would look like.
{
  "name": "SomeRequest",
  "field": [
    {
      "name": "some_field_name",
      "number": 1,
      "label": "LABEL_OPTIONAL",
      "type": "TYPE_STRING",
      "options": {

      },
      "jsonName": "someFieldName"
    }
  ],
  "options": {

  }
}

Hello, this was implemented in #11 and released in 0.3.0. Hope this works for your needs. If any issues please feel free to reopen.


Revert "Do ModelView initialization actions before others (#11512)"

Fixes #11666
Fixes #11665
This reverts commit 9bdd5f2899b36a44bca4add1f05164cf1489929b.
I'll be coming back to this to try and dig into what exactly is going on here because this is going to keep causing us problems - but this broke a lot of core scenarios so rolling back for now.

@bergeron I'll be looking again to make sure we're not regressing the dashboard behavior here - and if we are try to find some other solution while I work on a longer term fix.

Coverage decreased (-0.007%) to 40.402% when pulling 5174888f4f03d08bcb59222eb85150696a79af35 on chgagnon/undoSerializeModelActions into 9c6783288063a826578ec30d50eee975fa286d28 on main.


Fix: rotateAuthAccount failed when called twice

When giveOwnership and addOwnedAccount are executed in the same transaction
rotateAuthAccount will be called twice.
However, during the second call, since authAcctPath is the same, it will cause acct.linkAccount to return nil as a result.
Personal opinion, it would be better to change HybridCustody overall to CapCons coding style.

Codecov Report

Merging #154 (0f8e7fe) into main (dd77f1b) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #154   +/-   ##
=======================================
  Coverage   82.50%   82.50%
=======================================
  Files           4        4
  Lines         343      343
=======================================
  Hits          283      283
  Misses         60       60

Files Changed
Coverage Δ

contracts/HybridCustody.cdc
81.41% <100.00%> (ø)

Updated with unsafeRandom on new flow-cli v1.4.4
@sisyphusSmiling @austinkline

Testnet and Mainnet contracts have been updated.
Testnet updating transaction- c3c31132c1217fb5e993281fa4618e0bd2684a409e1f8ef9ca4d119bddef678d
Mainnet updating transaction - 7a757e20c55149567715fb7135ef521e02e4345ed19fd4685e2b4b2a4b9f9030


Move wind and graph examples to showcases

Change List

Move wind example from website/ to showcases
Move graph example from website/ to showcases
Lint

@ibgreen Shall I move them now? No other part of this repo is depending on these.

Absolutely, if you agree with the idea, that would be great!


WebApp:unable to upload new django project to web app

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'NoneType' object has no attribute 'reserved'
Traceback (most recent call last):
Local\Temp\pip-unpacked-wheel-lhx4371t\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 718, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 709, in _run_job
azure\cli\command_modules\appservice\commands.py, ln 55, in _polish_bad_errors
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
pip-unpacked-wheel-c8n8j8wo\azure\cli\core\__init__.py, ln 784, in default_command_handler
azure\cli\command_modules\appservice\custom.py, ln 3587, in webapp_up
AttributeError: 'NoneType' object has no attribute 'reserved'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up --name {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.13.0

Additional Context

@dvdtho is this Django app a Github repo app you can point us to also. Can you  check defaults subscription & local config set on this & if they look correct?
az account show -> using the right subscription
check no local config set from the folder you are running the app from look for .azure folder & see if you have any default values set for  app service plan or resource group. Thanks!

webapp

https://github.com/Azure/azure-cli/issues/14287

please upgrade to the latest cli version and let us know if there is still an issue

closing due to no response, please reopen if still an issue


Rationalize IndexedDB indexes

Today IndexedDB backend for Orbit creates four indexes but only one is used (recordIdentity). Can we rationalize that? @dgeb do you have any future plans for other indexes?
objectStore.createIndex('type', 'type', { unique: false });
objectStore.createIndex('recordIdentity', 'recordIdentity', { unique: false });
objectStore.createIndex('relatedType', 'relatedType', { unique: false });
objectStore.createIndex('relatedIdentity', 'relatedIdentity', { unique: false });

Thanks for catching this. Looks like the other indices were added prematurely, so it's probably best to remove them for now.

Closed via #623


perf: Improved performance and reduced memory consumption

Hello. I profiled my application to eliminate micro lags in a large table. There are 3 fields in it that are Multiselect with a large number of options, and 100 rows per page. The table is reactive and updated via WebSocket, but each update was accompanied by micro lags, especially noticeable on mobile devices.
After reviewing the code, I identified several opportunities for improvement, here are some of them:

Avoid unnecessary array allocations, especially for functions like isNullish() and aria-* getters.
Eliminate prop checks after toRefs(), they are specified in SFC, so they will never be undefined. The same goes for resolveDeps(), a hook can't be undefined.
Quickly exit the arraysEqual() function if the array sizes do not match. In the current implementation, it sorts the array first and then checks.
Replace ref() with shallowRef() for binding HTML. This is a common use case for shallowRef(); it will only track the setting of the value, not every HTML property.
Replace most computed() with toRef(). Instead of having a cache for simple getters, it's cheaper to directly call them. Currently, a polyfill is used to implement toRef(). Since the change toRef() appeared only in Vue 3.3+, this polyfill is slightly slower than the standard implementation, but it's the same one used in VueUse.

After these changes and subsequent profiling, I conclude that the micro lags have disappeared, the average memory consumption per page has decreased, the page opening time has become approximately twice as fast on mobile devices, and the INP metric has also improved.

Impressive - thanks. These definitely improves the lib's performance.


drawstart does not seem to fire with AGS layers

nodes = new gvector.AGS({
                url: "http://example.com/ArcGIS/rest/services/TOML/MLTS_Test/MapServer/0",
                fields: "*",
                scaleRange: [13, 20],
                showAll: true,
                symbology: {
                    type: "single",
                    vectorOptions: {
                        strokeWeight: 4,
                        strokeOpacity: 0.8,
                        strokeColor: "#004a00"
                    }
                },
                infoWindowTemplate: '

TOML

  Consensus name:	{Consensus_Name}

'
            });

the appropriate setmap doesn't seem to fire:
google.maps.event.addListener(nodes, 'drawstart', function(){
                document.getElementById('drawingNODES').style.visibility = 'visible';
            });

BTW, this library rocks!

oh my bad, that event doesn't even exist! sorry!

I don't see where you're calling setMap anywhere though. Or am I missing
something?

ok, so you're all set?

I was calling setMap with a button (as per your examples). the data shows up like a dream, but i was thinking about building a spinney wheel affair to ease the users nerves while waiting.

I'm good, thanks for your code!

Awesome, glad to help.


[TextField] draws narrow drop down list when using SelectProps with start/endAdornment Icons

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
MUI TextField draws narrow drop down list when using Select menu with startAdornment or endAdornment Icons.
Codesandbox link example is below:
https://codesandbox.io/s/musing-johnson-lyqvs?from-embed=&file=/src/index.tsx

Expected behavior 🤔
The box of drop down list has the same length as the TextFiled length and have an opportunity to Customize that drop down list box.
Steps to reproduce 🕹
Steps:
1.
2.
3.
4.
Context 🔦
No response
Your environment 🌎
System:
OS: Linux 4.15 Ubuntu 18.04.3 LTS (Bionic Beaver)
Binaries:
Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
Yarn: Not Found
npm: 8.2.0 - ~/change_account_client/node_modules/.bin/npm
Browsers:
Chrome: 97.0.4692.71
Firefox: 96.0.1
npmPackages:
@emotion/react: ^11.5.0 => 11.7.0
@emotion/styled: ^11.3.0 => 11.6.0
@mui/base:  5.0.0-alpha.59
@mui/icons-material: ^5.0.5 => 5.2.1
@mui/material: ^5.0.6 => 5.2.3
@mui/private-theming:  5.2.3
@mui/styled-engine:  5.2.0
@mui/styles: ^5.0.2 => 5.2.3
@mui/system:  5.2.3
@mui/types:  7.1.0
@mui/utils:  5.2.3
@types/react: ^16.14.2 => 16.14.21
react: ^17.0.2 => 17.0.2
react-dom: ^17.0.2 => 17.0.2
styled-components: ^5.2.3 => 5.3.3
typescript: ^4.1.3 => 4.5.2``

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

Please find CodeSandbox link below:
https://codesandbox.io/s/musing-johnson-lyqvs?from-embed=&file=/src/index.tsx
How we can get the box of drop down has the same length as the [TextFiled] length and have an opportunity to Customize styles of drop down box.


Configuration model binder

The ability to map a strongly typed model from a configuration.

The existing Configuration class can be used to support this functionality by adding a new extension method.  I would fork and upload this code, but I'll wait until the kproj >> xproj change is supported by VS so that I can use the IDE (I know, lazy me).
Anyway, here is a simple version of the code that will map (recursively) a config key (and subkeys) to a POCO object:
'''C#
/// Map configuration values to object
/// Base key to use for mapping
/// Object filled with values from configuration settings
public static T Map(this IConfiguration configuration, string key)
{
return (T)Map(configuration, key, typeof(T));
}
    /// <summary>Map configuration values to object</summary>
    /// <param name="key">Base key to use for mapping</param>
    /// <param name="type">Type for activation of object</param>
    /// <returns>Object filled with values from configuration settings</returns>
    public static object Map(this IConfiguration configuration, string key, Type type)
    {
        if (type == null)
        {
            throw new ArgumentNullException(nameof(type));
        }
        if (string.IsNullOrWhiteSpace(key))
        {
            throw new ArgumentNullException(nameof(key));
        }

        object obj = Activator.CreateInstance(type);

        foreach (PropertyInfo property in type.GetProperties())
        {
            string propertyPath = key + ":" + property.Name;
            object value = (property.PropertyType.IsPrimitive || property.PropertyType == typeof(string)) ? configuration[propertyPath] : Map(configuration, propertyPath, property.PropertyType);
            property.SetValue(obj, value);
        }

        return obj;
    }

'''
I'm sure it needs to be updated to be production ready, but you get the idea and it works for my purposes.  Obviously this doesn't need to be an extension method in the future.

This got moved to the aspnet/Options repository


[jni] Bump minSDK and publish 0.9.2

Closes #1119.

coverage: 90.957%. remained the same
when pulling fcffffc9d9e299bb83e511de23dfe71ad7ce3549 on jni-0.9.2
into e8b7d6c715f0abbb192ee681f045bc7af6d103f4 on main.


added stats feature

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Wells -

The design is really cool. The hard work behind the CSS is really commendable. 👏🏼
The clever use of new JS utilities like Promise.all(), for-of construct, etc. is good.

Improves -

Code Splitting - It's important to separate the code so that the next person finds it easy to read.
Example, the contributorsList can be taken out as a function and then can be used inside the Promise.
ref - https://github.com/MLH-Fellowship/prep-portfolio-23.MAR.PREP.2/pull/16/commits/c6658c6ef3174536554e3b2df762fc62fcd1c944#diff-61def0c226ce2f96e27720bf43b7a239822732fa59ca003f3d30bd8f5a060128R26-R43

cc: @SandyCreek


Feat/shortcuts and UI improvements

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Exporting RTF now exports no entries

The thing is i have loaded a save file and then export because you just updated and i wanted to see how it is working.
So for me this happens right now is this what needs to happen ?
My settings are Prerender : ALL and Events : ALL

I only hope if i play another new game and 1 year goes by, when i export everything, I hope will not get blank days just like in the screen shot

Unluckily, I can't do anything for the above reason except importing the current memory interactions (maybe I'll give it a try) and waiting for Tynan to implement a higher limit for interactions in vanilla Rimworld.
I'll take another look to Rimworld code to check if I'm wrong and all interactions are stored somewhere, but I'm almost sure what I've read is good.

Something like auto-export each day can be done.
I misread this message.
If you start and do your game with the diary mod (and auto-write enable), there will be no problem for the export.
Everything written in the diary is stored differently than social interactions and events. I never remove anything from the diary, so it should not be any blank in the export.
Just remember to keep write filter to "All", "Events" or "Chats", because I currently can't bring back these entries if you decide to enable the mod after these things occured.