Don't remove unserializable attributes from active sessions (with default/java serialization)

From<EMAIL_ADDRESS>on March 23, 2011 15:14:42
Observed for the TC 7 branch, but likely not restricted to that.
The JavaSerializationTranscoder removes any attributes that do not implement Serializable from the active session.
Of course it can not serialize and push attribiutes to memcached if they are not serializable. But in addition now it smashes the consistency of the local session by removing those attributes.
One can imagine a situation, where the webapp developpers only want to keep certain attributes from the session in memcached and take their own responsibility for recreating the remaining data after failover (e.g. getting them from some database or other backend).
My current patch looks like this:
--- a/core/src/main/java/de/javakaffee/web/msm/JavaSerializationTranscoder.java
+++ b/core/src/main/java/de/javakaffee/web/msm/JavaSerializationTranscoder.java
@@ -126,11 +126,11 @@ public class JavaSerializationTranscoder implements SessionAttributesTranscoder
final Object value = attributes.get( keys[i] );
if ( value == null ) {
continue;

       } else if ( ( value instanceof Serializable ) && ( !session.exclude( keys[i] ) ) ) {

       } else if ( session.exclude( keys[i] ) ) {

           session.removeAttributeInternal( keys[i], true );

       } else if ( ( value instanceof Serializable ) ) {
           saveNames.add( keys[i] );
           saveValues.add( value );

       } else {

           session.removeAttributeInternal( keys[i], true );
       }
   }

I'm not sure whether the "exclude()" attributes should get removed, but I didn't want to change the behaviour for those.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=94

From martin.grotzke on March 23, 2011 13:52:28
The current implementation was taken from StandardSession.writeObject.
Your patch actually removes an attribute if it matches session.exclude( keys[i] ), which in StandardSession is returning true for an attribute "javax.security.auth.subject" - this would drop auth information from the session AFAICS.
If you don't want to remove unserializable attributes from the session, I'd suggest this (more or less how DeltaSession.writeObject is implemented):
--- a/core/src/main/java/de/javakaffee/web/msm/JavaSerializationTranscoder.java
+++ b/core/src/main/java/de/javakaffee/web/msm/JavaSerializationTranscoder.java
@@ -126,11 +126,11 @@ public class JavaSerializationTranscoder implements SessionAttributesTranscoder
final Object value = attributes.get( keys[i] );

       if ( value == null ) {

       if ( value == null || session.exclude( keys[i] ) ) {
           continue;

       } else if ( ( value instanceof Serializable ) && ( !session.exclude( keys[i] ) ) ) {

       } else if ( value instanceof Serializable ) {
           saveNames.add( keys[i] );
           saveValues.add( value );
       } else {

           session.removeAttributeInternal( keys[i], true );

           if ( LOG.isDebugEnabled() ) {

               LOG.debug( "Ignoring attribute '" + keys[i] + "' as it does not implement Serializable" );

           }
       }
   }

The only potential problem I see with this is that it might break servlet compliance, as 7.7.2 "Distributed Environments" says:
"The distributed servlet container must throw an IllegalArgumentException for objects where the container cannot support the mechanism necessary for migration of the session storing them."
(which would require a different patch for throwing an IllegalArgumentException)
However, 7.7.2 states also that "The container may choose to support storage of other designated objects in the
HttpSession, such as references to Enterprise JavaBeans components and transactions."
which is a conflict with the previous statement - from my understatement.
AFAICS tomcat follows the last quotation as it does not throw an IllegalArgumentException for non serializable session attributes, so it should be ok ;-)
Status: Accepted
Owner: martin.grotzke
Labels: -Type-Defect Type-Enhancement

From<EMAIL_ADDRESS>on March 24, 2011 01:18:31
I fully agree with your patch. I kept removing the exclude attributes because that was what your original code did. I think the patch you proposed is fine.
Concerning servlet spec: unfortunately it doesn't help a lot when thinking about scaling and replication. More precisely: I'd say when adding a replication mechanism you want as little negative influence as possible as long as you are not failing over (sticky case) and a good way of finding out, whether fail over will actually work.
So logging replication problems is helpful, breaking the local app not. Since on a production system the problems will happen often - if they happen at all - e.g. because of several session attributes not being serializable, those kind of messages should have a low log level or a special logger in order to be able to silence them without removing real (unexpected) error logging.
Thanks for caring!
Rainer

From martin.grotzke on March 24, 2011 10:49:35
Changed in master and tomcat7 branch.
Summary: Don't remove unserializable attributes from active sessions (with default/java serialization)
Status: Fixed
Labels: Milestone-1.4.1


[BUG] TrafficManagerMonitorConfig.Path Property not allowing "/{0}/health/check"

Library name and version
Azure.ResourceManager.TrafficManager
Describe the bug
When I try to send the Path property = "/{0}/health/check", is not allowing to do the creation of a new TrafficManagerProfile
Expected behavior
Accept the designated path and create the new TrafficManager.
Actual behavior
I received:
ErrorCode: `BadRequest`

Content:
{"error":{"code":"BadRequest","message":"The supplied monitoring path \/{0}\/health\/check is not well-formed."}}

Reproduction Steps
Code:
TrafficManagerProfileData trafficManagerProfileData = new TrafficManagerProfileData
            {
                Name = "profileName",
                Location = new AzureLocation("location"),
                TrafficRoutingMethod = new TrafficRoutingMethod("routingMethod"),
                DnsConfig = new TrafficManagerDnsConfig { RelativeName = "profileName" },
                MonitorConfig = new TrafficManagerMonitorConfig
                {
                    Port = 80,
                    Protocol = TrafficManagerMonitorProtocol.Http,
                    Path = "/{0}/health/check",
                }
            };

            await profileCollection.CreateOrUpdateAsync(
                WaitUntil.Completed,
                "profileName",
                trafficManagerProfileData,
                cancellationToken);

Environment
No response

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

Hi @abiisnn Thank you for using Azure SDK for .NET.
I tried to use same path on the portal, but get the same error. Looks like the service doesn't support such types of path.

Hey @abiisnn , thank you for reaching out to us about the Azure SDK for .NET. Unfortunately, this issue is related to the service side and not the SDK side, so we are not able to help you with it. Therefore, we are closing this issue for now. If you need more assistance, please open an Azure support request. We apologize for any inconvenience this may cause you.
We value your feedback and we want to make sure that your problem is solved. If you think that we have misunderstood your issue or closed it incorrectly, please feel free to comment on this thread and reopen the issue. We will be happy to assist you further. Thank you for your understanding and cooperation.


Add subutaitray lastest

Does not install, fails with:
Error: pkg source file not found: 'subutai-tray.pkg'
Many attempts at workarounds tried, both documented and from greping through /usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask/Casks
version and sha256 are fixed to prevent redownloading.
The project is on github but they are not using it for releases/distribution:
https://github.com/subutai-io
More info at end of [PR]

have followed:
https://github.com/caskroom/homebrew-cask/blob/master/doc/cask_language_reference/stanzas/pkg.md
and
https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/source_is_not_there_fix.md
and
https://github.com/caskroom/homebrew-cask/pull/35896/commits/17cb30246ad7381535ea54b9f350258d978d2975
Tried to document where each attempt at solving problem came from

See gist:
https://gist.github.com/BenjaminHCCarr/923c7d57a2a961c8a79f2d7fa1a80698

After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[X] Named the cask according to the [token reference].
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no [open pull requests] for the same cask.
[X] Checked the cask was not already refused in [closed issues].
[X] Checked the cask is submitted to [the correct repo].

Subutai is a new distributed "personal" cloud, where resources can be "donated": https://www.subutai.io/
It is endorsed by "Maddog" which is how I learned about it: http://www.linux-magazine.com/Online/Blogs/Paw-Prints-Writings-of-the-maddog/Subutai-Peer-to-peer-private-secure-stable-cloud-software-that-gives-you-control

I tried looking for help on gitter for a couple days, and for a brief period on IRC freenode, but hopefully someone can figure this out so I can PR this and the daemon:
https://github.com/BenjaminHCCarr/homebrew-cask/blob/master/Casks/subutai-p2p.rb

Should also note that this:
tyr:~/Library/Caches/Homebrew/Cask benc$ open subutai-tray--6.0.1.pkg
Successfully launches a working installer.

LGTM!
# This is a hack to force the file extension.
preflight do
    system_command '/bin/mv', args: ['--', staged_path.join('get'), staged_path.join('subutai-tray.pkg')]
end

Was NOT a hack I was going to come up with! You can see all the attempts though!


Bumped webdeployment-common-v2 to remove lodash vulnerability in Tasks/AzureSpringCloudV0

Task name:
Description: Bumped webdeployment-common-v2 to remove lodash vulnerability that sits in the archiver dependency
Documentation changes required: N
Added unit tests: N
Attached related issue: Y
https://github.com/advisories/GHSA-jf85-cpcp-j695
Checklist:

[X] Task version was bumped - please check instruction how to do it
[X] Checked that applied changes work as expected - smoke tested in the local pipeline

Will be merged together with this task https://github.com/microsoft/azure-pipelines-tasks/pull/16310


e3-core: init at 22.3.1

Description of changes
Adds the e3-core python library. This has some general utility, though it's primarily used by AdaCore for build automation.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Thanks so much - I believe I've addressed all of that, do tell me if there's anything else!

I believe that's that; thanks again!

Result of nixpkgs-review pr 267758 run on x86_64-linux 1

4 packages built:

python310Packages.e3-core
python310Packages.e3-core.dist
python311Packages.e3-core
python311Packages.e3-core.dist

Waiting for orfborg


Large video buffering

Help.
When working with video, video freezes every +- 300 frames, followed by buffering.
Is there any way to fix it?
https://youtu.be/LrYkPyqjNqQ - example

Hi @outforz
Thanks for the question.
 We were looking for ways to fix it, but no decisions was applied yet.


Please help to solve this error

"Assets\GeometryGraph\Runtime\Graph\Node\Curve\RecalculateCurveDirectionsNode.cs(20,33): error CS0115: 'RecalculateCurveDirectionsNode.OnConnectionRemoved(Connection, RuntimePort)': no suitable method found to override
"

there are 44 same errors in other node.cs files

Hey there @jane00!
The reason for these errors is that GeometryGraph relies heavily on generated code which is not included in the repository.
You can solve these by compiling the UnityGeometryGraph-CodeGen  project into a DLL, and then copying it and its dependencies to the Unity project and setting it up according to this Unity documentation.
Or alternatively, extract this archive into Assets/Plugins/UnityGeometryGraph-CodeGen and (hopefully) it will work:
UnityGeometryGraph-CodeGen.zip.
Note that after extracting the archive you might need to edit some code in GeometryGraph for the code generator to trigger.
Thanks for your interest in the project and I hope this helps solves your issues!

Thanks very much for your reply.
Unfortunately, it doesn't work.
I am looking foward to get a working version some day. Thanks.
In fact, I just need a simple function of graph workflow for instancing some prefeb and caculating their posiiton/scale/rotation visually just like in Blender.


Replace mountpoint with auth request returned value

This PR is supporting to modify mountpoint field by the AUTH response.

Please add test cases and docs.


Upgrade Nuget references and add new Web API specific enrichers and

I got a bit carried away, and this PR contains a bit more than initially planned ... I can try and split it in several PRs if you think it's better ...
This PR contains :

upgrade of the whole project to later versions of SerilogWeb.Classic (v3.0.20) ... which therefore upgrades the Serilog reference to v2.5.
After this upgrade, generating the nuget package produces something like this :

This means that the Major version number of this package should be incremented too (to v2.0)

several enrichers in the namespace SerilogWeb.Classic.WebApi.Enrichers built specifically to capture some Web API specific information :

WebApiActionNameEnricher
WebApiControllerNameEnricher
WebApiRouteDataEnricher
WebApiRouteTemplateEnricher (this is actually the one I was trying to do as for datalust/seq-releases#622 , but I thought I might as well do the other ones too ...)

A configuration like this :
Log.Logger = new LoggerConfiguration()
    .Enrich.With<WebApiRouteTemplateEnricher>()
    .Enrich.With<WebApiControllerNameEnricher>()
    .Enrich.With<WebApiRouteDataEnricher>()
    .Enrich.With<WebApiActionNameEnricher>()
    .WriteTo.Trace(new JsonFormatter())
    .CreateLogger();

in a Web API app with a controller like this one :
public class ValuesController : ApiController
{
    // GET api/values
    public IEnumerable<string> Get()
    {
        return new string[] { "value1", "value2" };
    }

    [Route("api/values/shouldthrowexception")]
    [HttpGet]
    public string ShouldThrowException()
    {
        throw new SyntheticException(nameof(ShouldThrowException));
    }

    // GET api/values/5
    public string Get(int id)
    {
        return "value";
    }
}

generates events like those in the Trace output when accessing /api/values, api/values/6 or /api/values/shouldthrowexception:
{
	"Timestamp": "2017-10-26T21:58:52.6501081+02:00",
	"Level": "Information",
	"MessageTemplate": "HTTP {Method} {RawUrl} responded {StatusCode} in {ElapsedMilliseconds}ms",
	"Properties": {
		"Method": "GET",
		"RawUrl": "/api/values/",
		"StatusCode": 200,
		"ElapsedMilliseconds": 6,
		"SourceContext": "SerilogWeb.Classic.ApplicationLifecycleModule",
		"WebApiRouteTemplate": "api/{controller}/{id}",
		"WebApiController": "Values",
		"WebApiRouteData": {
			"controller": "values"
		},
		"WebApiAction": "Get"
	}
}

{
	"Timestamp": "2017-10-26T21:59:21.3880735+02:00",
	"Level": "Information",
	"MessageTemplate": "HTTP {Method} {RawUrl} responded {StatusCode} in {ElapsedMilliseconds}ms",
	"Properties": {
		"Method": "GET",
		"RawUrl": "/api/values/6",
		"StatusCode": 200,
		"ElapsedMilliseconds": 2,
		"SourceContext": "SerilogWeb.Classic.ApplicationLifecycleModule",
		"WebApiRouteTemplate": "api/{controller}/{id}",
		"WebApiController": "Values",
		"WebApiRouteData": {
			"controller": "values",
			"id": "6"
		},
		"WebApiAction": "Get"
	}
}

{
	"Timestamp": "2017-10-26T21:59:52.8379675+02:00",
	"Level": "Error",
	"MessageTemplate": "HTTP {Method} {RawUrl} responded {StatusCode} in {ElapsedMilliseconds}ms",
	"Exception": "SerilogWeb.Classic.WebApi.Test.SyntheticException: ShouldThrowException\r\n... snip ...",
	"Properties": {
		"Method": "GET",
		"RawUrl": "/api/values/shouldthrowexception",
		"StatusCode": 500,
		"ElapsedMilliseconds": 2512,
		"SourceContext": "SerilogWeb.Classic.ApplicationLifecycleModule",
		"WebApiRouteTemplate": "api/values/shouldthrowexception",
		"WebApiController": "Values",
		"WebApiRouteData": {},
		"WebApiAction": "ShouldThrowException"
	}
}

In order to do that, the library automatically injects an ActionFilter in the Web API pipeline : StoreWebApiInfoInHttpContextActionFilter. That action filter stores some of the ActionContext in HttpContext.Current to make them available to the enrichers.
Obviously, because it relies on HttpContext, this cannot work in self-hosted scenarios, but I believe this was already expected.
I believe those would be some rather useful enricher (at least for me :p ) , and I would love some feedback, mostly I have doubts about :

is it ok to add stuff to HttpContext.Current ? is there a cleaner way that would be more Web APish , but still allow the enriched data to be captures by the SerilogWeb.Classic module ?
are the names and namespaces good enough ?
are the default names under which those properties are stored intuitive enough ?
are there some edge cases I might have missed ?

Thanks in advance !

Very nice! Will sort out the major version and push this through.

@nblumhardt Any chance you can publish a new version of this one (with an increased major version number) ? I know the SerilogWeb projects have a bit of a lower priority, so if I can help, I'd be happy to :)
Also I'd like to do something similar for ASP.NET MVC (not Core) .... would it be possible to create a new repo in this organization. The repo's name could be something like classic-mvc and the correspounding Nuget package could be SerilogWeb.Classic.Mvc ? and I could take it from there to prepare a first version :)
Thanks in advance 🙇

Looks like the CI webhooks aren't firing - I probably need to re-authorize GitHub. I'll hit the manual build button and see what happens :-)

The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Unable to find 'C:\projects\classic-webapi\src\SerilogWeb.Classic.WebApi\bin\Release\SerilogWeb.Classic.WebApi.dll'. Make sure the project has been built.

Hmm....

https://github.com/serilog-web/classic-mvc created, you should have admin access there, let me know if you need a hand.
I've added it to AppVeyor, build status badge is:
[![Build status](https://ci.appveyor.com/api/projects/status/3yidmjq6pm0f2pii?svg=true)](https://ci.appveyor.com/project/serilog-web/classic-mvc)

It'll need an appveyor.yml file to do anything interesting :-)
I'm not sure what the story with the build in this repository is - please feel free to jump in, and let me know if the CI problem still needs some action from me 👍
Cheers!

Great, thanks a lot ! I'll start to get it started soon 👍


[test_subprocess]:  test_assert_nested fails to run correctly

The end-to-end test test_assert_nested fails to run run correctly on a PVC GPU.

This work item is blocked. The IGC compiler has a problem described in https://github.com/intel/intel-xpu-backend-for-triton/issues/256#issuecomment-1907085632 which needs to be resolved in order to unblock this work item.

Verified can be fixed by https://github.com/intel/intel-xpu-backend-for-triton/issues/358.


Added contributing guidelines about WikiToLearn

@tapasweni-pathak @nikhita Have a look, and let me know if this is fine.

Just one more thing - can you add a note about what languages/technologies one should know before contributing? :)

After @nikhita's suggested changes, perfect for me too. :)

@Srijancse no hurries, but a friendly ping :)

Oh, also noticed this has conflicts with README.md. You probably need to do a rebase too.

bump ^^


Update stylelint dependency

v7 is out
There may be other changes necessary here. Not too sure tbqh. Just figured I'd open the PR to get things started.

Merged and published as v3.0.0 on npm. Thanks for your contribution!

Thanks! ✌


Fix some typos about error

Modify it's to its.
The information of error should be begin with low-case letter and keep the same style in this file.

Signed-off-by: yuexiao-wang<EMAIL_ADDRESS>

@aboch @sanimej

LGTM


[DRAFT] :construction_worker: Docker caching

I will be actively testing a bunch of things here as a sandbox

I've refactored the docker workflow to use the podman engine instead and added some cache mounts. Note that before swapping to podman, the cache mounts caused a hang in arm64 for whatever reason.
I've gotten a 44s total build time for a consecutive build, each layer hit the cache, but that is very likely an overly eager and unrealistic scenario. The next test will be ensuring the compiled deps stay cached when actual code changes

I've gotten successful builds multiple runs now, so I think this PR should be good to merge


Add: 2 ropsten signers

Adds addresses of 2 validators to ropsten config for integration testing on testnet.

It was already implemented


Release 0.14.2

Major changes:

llama-core crate

New feature: #196 by @suryyyansh
Bugfix: #204 by @L-jasmine

chat-prompts crate

Add BreezeInstructPrompt to support second-state/Breeze-7B-Instruct-v1_0-GGUF model

Thanks for the contributions from @suryyyansh and @L-jasmine.

Hello, I am a code review agent on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Add ppc64le and arm64 support

Signed-off-by: Lukas Kral<EMAIL_ADDRESS>This PR adds support for ppc64leand arm64 to our images.

@scholzj sure thing :)


Support multiple windows

In Zed, we are initializing a graphics context for every window. However, if there is more than one window, we are wasting time re-initializing the connection to the graphics driver, recompiling all the pipelines, etc. If we support multiple windows natively, we could save that time.

What is required for this to be implemented and what would the implementation look like?

We'd need to introduce Window as a new type of things. It would be created from the Context.
Overall, I haven't thought through the details of how this will work. I typically start prototyping it to see the issues that would arise and then design as I go.


File datas.csv does not exist: 'datas.csv'!

where is the problem?

Duplicated https://github.com/bukosabino/ta/issues/161


Summer 2021 not 2020

The page says "Viva Connections for mobile will become available in Summer 2020". Presumably this should be "Summer 2021"...

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

ID: 7210783d-e96e-9d14-9120-553f731d20ed
Version Independent ID: 84854221-f26b-4a19-b95e-0d5d7c640a46
Content: Add Viva Connections for Microsoft Teams desktop - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/viva-connections.md
Service: sharepoint-online
GitHub Login: @Holland-ODSP
Microsoft Alias: hokavian

Duplicate issue


Xiwi External Storage

name: precise
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/precise...
crouton: version 1-20160330180425~master:43e2fca0
release: trusty
architecture: amd64
xmethod: xiwi
targets: xiwi,keyboard,xfce
host: version 7834.66.0 (Official Build) stable-channel lumpy
kernel: Linux localhost 3.8.11 #1 SMP Tue Mar 29 03:53:16 PDT 2016 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/precise...

I can't seem to see external storage drives (USB, SD) within a Xiwi app (full linux seems to work okay). I'm using VLC in particular; in xfce it shows all the drives when trying to open a file, but in a windowed xiwi app version it only shows "Computer" (/) and my home directory (/home/...).

The files should be bind mounted properly in /media.


Prototype test: Greenhouse Effect 1.1.0-rc.3

Prototype Test
Mentions: @arouinfar, @kathy-phet, @katiewoe
Simulation links

github repository for issues
phet top level directory
sim: all_phet.html

Test Matricies

Tester = NS , Platform = mac + safari , Time = .75
Tester = KW, Platform = Win 11 Chrome, Time = .5

Features included

[x] Alternative Input
[x] UI Sound
[x] Sonification

Focus and Special Instructions
Three issues were addressed as a result of the previous round of testing, and two were pretty minor, so this round of
testing can be relatively light.  Verifying the issues should be the focus, and there should be additional verification
of the energy balance indicator and the flux meter, since there were some non-trivial changes made to them to improve
their performance.

Issues to Verify

[x] https://github.com/phetsims/greenhouse-effect/issues/246
[x] https://github.com/phetsims/greenhouse-effect/issues/260
[x] https://github.com/phetsims/greenhouse-effect/issues/265

For QA...

General features

What to Test

Play with the simulation normally.
Make sure you can't lose anything.
Complete the test matrix.
Try to include version numbers for browsers
If there is a console available, check for errors and include them in the Problem Description.
Focus on pedagogy issues and bugs that break the sim.
If you find bugs beyond pedagogy/breaking, still document these bugs as github issues.
If a11y aspects are present, test them, but with an eye for "normal use".

Accessibility features

What to Test
If a11y aspects are present, test them, but with an eye for "normal use".

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

If this sim is not in this list or up to date there, make an
issue in website to ask if PhET research page links need updating. Please
assign to @terracoda and @emily-phet.

Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

Windows 10 + Latest Chrome + Latest JAWS
Windows 10 + Latest Firefox + Latest NVDA
macOS + Safari + VoiceOver
iOS + Safari + VoiceOver (only if specified in testing issue)

Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a
screen reader bug, please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel
controls. Please test magnfication and make sure it is working as intended and well with the use cases of the
simulation. Due to the way screen readers handle user input, magnification is NOT expected to work while using a screen
reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

In rc.3, on the Photons screen the text at the bottom of the concentration panel is cut off slightly on the right side with stingTest=long.

QA is done

Thanks QA team!  I'll check over the issues and take the next steps.  Closing.


Adding and removing dropdown menu items programmatically

In order to add or remove dropdown menu items using javascript, I have to add a
 child to the
 DOM, then re-initialize the dropdown module.  If the dropdown initialization is complex, this makes for messy and seemingly unnecessary code.
Any plans for this?  I could take a stab at it, but would appreciate a recommended approach.

You can do $(".menu").dropdown("refresh") everytime you add or remove an item.

Great, I'll try this.  Perhaps it should be documented?

In 2.0 dropdown automatically watches for DOM mutations not just in <select> but also menu


[BUG]: Updating core components solution import error

Describe the bug
I am attempting to update the CenterofExcellenceCoreComponents solution with a from version 1.45 to 1.85
Import is unsuccessful due to Attribute admin_rpamodifiedon is a String, but a DateTime type was specified.
I am unable to delete this column in the RPA table.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hello, you will need to upgrade to the interim solution (1.62) shown in #515
And then try again to upgrade to 1.85.
Thanks for using CoE!
Duplicate of #515

Hi Jennifer Thanks.  I was able to import the version 1.60.5 from the #515 successfully.  However, when I attempt to import 1.85 I am getting a missing dependencies error:
[{"SolutionValidationResultType":"Error","Message":"The following solution cannot be imported: CenterofExcellenceCoreComponents. Some dependencies are missing. The missing dependencies are : <Required key="16" type="1" schemaName="admin_archiveapproval" displayName="Archive Approval " solution="Active" /><Dependent key="17" type="1" schemaName="admin_app" displayName="PowerApps App" /><Required key="16" type="1" schemaName="admin_archiveapproval" displayName="Archive Approval " solution="Active" /><Dependent key="18" type="1" schemaName="admin_flow" displayName="Flow" /> , ProductUpdatesOnly : False","ErrorCode":-2147188707,"AdditionalInfo":null}]

Hello, yes we shipped with an accidental dependency. The new version of core that is out will work for you.

Hello, yes we shipped with an accidental dependency. The new version of core that is out will work for you.

Hi , pls I need your help with the same message, maybe is happening again, I just trying to import the CenterofExcellenceCoreComponents_4.16_managed.zip but i got the same error message, Is something wrong with the package again? (pls feel free to mailing the response<EMAIL_ADDRESS>


fixing the download task example

Fixing the download task example to use the artifact keyword

Fixed few fit and finish changes to downloadBuild macro and build resources.


[help] how to summary with attach files ,use --files ,i did but it refused

➜  ~ cat file1.txt
History
Originally published by Continuum,[3] the series was founded by editor David Barker in 2003.[1] At the time, Continuum published a series of short books on literature called Continuum Contemporaries. One-time series editor Ally-Jane Grossan mentioned that Barker was "an obsessive music fan who thought, 'This is a really cool idea, why don't we apply this to albums'.[3] PopMatters wrote that the range consists of "obscure classics to more usual suspects by the Beach Boys, the Beatles, and the Rolling Stones".[1]

In 2010, Continuum was bought out by Bloomsbury Publishing, which continues to publish the series.[3] Following a leave, Barker was replaced by Grossan in January 2013.[2] Leah Babb-Rosenfeld has been the editor of the series since 2016.[4]

Several independent books have been spun off of the series. The first, Carl Wilson's 2007 entry on Celine Dion's Let's Talk About Love, was expanded for a 2014 Bloomsbury reissue with material not specifically pertaining to the Dion album and retitled Let's Talk About Love: Why Other People Have Such Bad Taste. Joe Bonomo, at the invitation of Barker, expanded his 33+1⁄3 proposal on Jerry Lee Lewis's Live at the Star Club, Hamburg album into a full-length book about Lewis, the album, and his career titled Jerry Lee Lewis: Lost and Found, published by Continuum in 2009. A rejected proposal from writer Brett Milano for an entry on Game Theory's 1987 album Lolita Nation was instead expanded by Milano into a biography on the band's leader Scott Miller; that project, titled Don't All Thank Me at Once: The Lost Genius of Scott Miller, was released by 125 Books in 2015.[5]

In August 2017, Bloomsbury announced the launch of 33+1⁄3 Global,[6] an extension of the 33+1⁄3 series to popular music from around the world. The first two sub-series launched were 33+1⁄3 Brazil, edited by Jason Stanyek, and 33+1⁄3 Japan, edited by Noriko Manabe.[7] The first book for 33+1⁄3 Brazil was Caetano Veloso's A Foreign Sound by Barbara Browning.[8] The first books for 33+1⁄3 Japan were Supercell ft. Hatsune Miku by Keisuke Yamada and Yoko Kanno's Cowboy Bebop Soundtrack by Rose Bridges.[9]
➜  ~ claude --files file1.txt
Welcome to the Claude CLI!
? Select a conversation: Start new conversation
? (claude-2) > summary into english from the file file1.txt
Asking claude
11:43:28 - (claude-2)

Unfortunately I do not have access to read any files. However, I can summarize information if you provide it to me directly.

"It seems Claude is unable to read the contents of the file."

Same problem here

I figured how to summarize the attached file. You need to use square brackets to wrap the file name as in #10. Also make sure your node is the latest version that supports File object

Node.js v20.5.0. is ok, worked.
but Node.js v18.17.0 , not worked,
i have fixed it ! thank you

v20.5.0
find some in [file1.txt]


Hotfix BlockCache accumulating writes when it shouldn't

This is a near-term fix for https://github.com/hyperledger/burrow/issues/713


startup crash with the new 1.15 version

crash-2020-11-05_09.31.15-client.txt
latest.log
debug.log
there's not much more info i can give, as far as i know, but ask if you think otherwise

Only major thing I see is this:
java.lang.NullPointerException: Registry Object not present: productivebees:sugar_cane_nest
Talk to the productive bees devs about this. I can’t point out anything wrong from the perspective of Dungeons Gear.

Do you happen to know if you are playing with any mods that give you extended attack reach in any way, besides Dungeons Gear?

well, there's astral sorcery, that comes to mind immediately, and one or two other might also do it (boutiful baubles, artifacts)
and i know that productive bees isn't the culprit, as even when removing it, the crash happens

Astral Sorcery may be the culprit then. They are using Javascript coremods to modify attack reach, which I am also doing but with mixins instead. Since they change it before I do, my mixin fails and then the entire modloading state fails. I can’t really say as to whether or not this incompatibility could be fixed on our 1.15 versions.

i see...
in that case, imma go bug hellfire about it, but wouldn't it be possible to revert that spacific feature to be like in the previous version, which caused no problems (as a config, maybe)

I don't think it can be configurable, I would likely have to remove the mixin entirely and rewrite it using the old way. This whole situation is just more proof that Forge needs a standardized attack reach attribute, as Hellfire and I seem to be doing the exact same thing in a conflicting manner.

I will be releasing an updated version for 1.15 that no longer uses mixin for attack reach and should be working completely separately to what Hellfire and other modders try to do with attack reach.

nice ^^
once again, thank you, and sorry for the bother
feel free to message me if ya need any help, i owe ya one, and will do my best

It should be up now. Thanks for bringing it up.


HttpClientExtensions doesn't handle null IFormFile

When using the HttpClientExtensions for integration testing and posting to an endpoint which takes in form data, the static method ToForm<TRequest>(this TRequest req) is called to transform the request into MultipartFormDataContent.
I have an endpoint with a request DTO that looks something like
public record Request
{
    [HideFromDocs]
    [FromClaim(ClaimTypes.Role, isRequired: false)]
    public string? OptionalRoleClaim { get; init; }

    public int RouteParam { get; init; }

    public string? Name { get; init; }

    public IFormFile? File { get; init; }

    public SomeObject? AdditionalInfo { get; set; }
}

ToForm contains a check that looks like if (p.PropertyType == Types.IFormFile) AddFileToForm((IFormFile)p.GetValue(req)!, p); which asserts that my file is not null, but it's allowed to be null in my request.

thanks for reporting.
fix is available in v<IP_ADDRESS>-beta


reaction doesn't trigger a re-render for array

First of all, thanks for this great "framework" :+1: , just been aware of it yesterday and already spent the next 20hours on it (it's really fun to play with :))
Anyway, I have an app-db like this
(def app-db
  (->> (range 2015 2023)
       (map #(vector % (int-array 10 0)))
       (into {})
       (assoc other-data :finance)))

Somehow the subscription/reaction doesn't work for array (not reactive enough, the dispatch fn changed the app-db, yet doesn't trigger re-render components that subscribed to it). It is however changed once I navigate to other page and then go back to that page (an act which mutate the value of app-db). When I changed it to a vector then it works as expected.
I haven't look into the source code yet, but my guess is that the mutation inside array-cells won't be counted as an object change since the reference to the object remain unchanged and thus do not trigger re-rendering.
If it's intended to be that way so be it (it enforces value-based state anyway, instead of a reference-based). But just in case it wasn't, then a heads-up would probably be nice (I'm a rookie in this front-end world, so probably it's just me that is unaware of this behaviour).
Thanks

Hi @squest, I'm not totally clear on what you're trying to do here, and I didn't realise you could create an int-array in ClojureScript. However your most general point is true: Re-Frame (and much of the Clojure ecosystem) leverages value's, not references for state tracking. So you're right that any mutations to a JS array will not be picked up by Reagent's change tracking.

Thanks for the reply @danielcompton , I am trying to build a numbers/excel-like matrix-cells web app for financial simulation which I think best modelled using array-like data structure (for easy in-place mutability). However, now I prefer using value-based structure like map & vector instead (just discovered that Clojure has this update-in fn to easily manipulate such structure).
Relevant to this issue I'm raising (maybe it's also the issue for ratom), probably it's better to inform the users that the re-rendering won't be triggered for reference-based type (although it's quite obvious for me now, it's still a gotcha).
So should we close this issue then?

@squest  I'm going to close this without changes being made to docs. While it managed to trip you up, I'm not sure its a common enough problem to warrant specific warnings (not sure where I'd put them in any case).


[WFLY-11847] Upgrading ASM to 7.1

https://issues.jboss.org/browse/WFLY-11847

The windows failure is unrelated.

@scottmarlow Please approve - I made a quick comment on the Jira

List of current dependencies on asm.asm is:
org/jboss/as/webservices/server/integration/main/module.xml
org/apache/cxf/main/module.xml
org/apache/cxf/impl/main/module.xml
org/apache/cxf/services-sts/main/module.xml
org/apache/cxf/ws-security/main/module.xml
org/hibernate/main/module.xml
org/hibernate/bytecodetransformer/main/module.xml
I don't think org/hibernate/main/module.xml actually needs asm.asm module dependency anymore, as Hibernate is using Javassist + ByteBuddy, ByteBuddy includes its own copy of ASM 7.1 classes.  We could either remove ASM from org/hibernate/main/module.xml for this pr or do that as a separate change.
I didn't see anything of concern when Hibernate upgraded to ASM 7.1.  @ropalka where there any ASM commits that you are concerned about?  I assume not since you didn't mention any, but just checking.
@asoldano @jimma can jboss/as/webservices/server/integration/* + cxf work with ASM 7.1?

@scottmarlow Thank you for that analysis. @ropalka (and @ everyone really) please do that kind of thing when proposing an upgrade for a component you don't 'own'. It's what identifies who needs to be asked for approval.
Scott, please file a JIRA and do a separate change re dropping unneeded dependencies.

Not concerned @scottmarlow about particular commit but demanding commit:
https://gitlab.ow2.org/asm/asm/commit/fc2fa0bedc9cd43680b758602149f433fc4c7948
so we can run WildFly on top of JDK13 without ASM choking on "Unsupported class file major version "
I will do this kind of analysis ( @scottmarlow did ) next time @bstansberry .

Created [https://issues.jboss.org/browse/WFLY-11874] for reviewing Hibernate ORM dependencies like ASM and remove unneeded ASM dependency.

@ropalka @scottmarlow @bstansberry   ASM 7.1 works with CXF 3.2.7 and JBossWS.


refactor(protocol): Variable Declaration Updates

Changed const declarations to let for objects that are reassigned, ensuring better compliance with JavaScript best practices and improving the flexibility of our code.

According to JavaScript best practice, for Objects, I think we should const at those places.


revert product-os/flowzone/pull/953

installing Python dependencies on runner breaks other dependencies (e.g. AWS-CLI)
run steps on Linux/amd64 until https://github.com/actions/setup-python/issues/705 is resolved

change-type: patch

I prefer this approach rather than hardcoding runner tags.

I prefer this approach rather than hardcoding runner tags.

How is this approach fix the underlying condition that the setup-python action will only run on X64/Linux currently?

I prefer this approach rather than hardcoding runner tags.

How is this approach fixing the underlying condition that the setup-python action will only currently succeed on X64/Linux? I must be missing something, perhaps you can add you commit directly to this branch?
We have seven environment repositories that are affected by this. We could technically change them all to only run on X64/Linux, but I think a one line change here is better than having to change x7, especially given that it will be temporary.

HOW temporary? Do we know if/when this will be fixed upstream?
I see the benefit of fixing it in Flowzone as it's less work, but then we have to make assumptions about what the calling repository has access to (self-hosted vs GitHub hosted) AND we need to remember and come back later to remove all the places in Flowzone where we are ignoring the runs_on input to provide a hardcoded one.
Would this be a suitable compromise?

Would this be a suitable compromise?

Ugh..

Would this be a suitable compromise?

Ugh..

We'll have to do it upstream and revert when they get their sh*t together.


feat: add gci and gofumpt

Run make format to sort imports deterministically and apply an enhanced gofmt code simplifier.

This change is

Codecov Report

Merging #2691 (3f3356e) into master (f50cc34) will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2691      +/-   ##
==========================================
+ Coverage   63.53%   63.59%   +0.06%
==========================================
  Files         236      236
  Lines       26606    26606
==========================================
+ Hits        16905    16921      +16
+ Misses       8182     8167      -15
+ Partials     1519     1518       -1

Impacted Files
Coverage Δ

pkg/shed/schema.go
75.58% <0.00%> (ø)

pkg/retrieval/retrieval.go
62.34% <0.00%> (+0.30%)
:arrow_up:

pkg/localstore/mode_set.go
55.22% <0.00%> (+1.11%)
:arrow_up:

pkg/localstore/mode_put.go
60.05% <0.00%> (+1.14%)
:arrow_up:

pkg/localstore/subscription_pull.go
86.98% <0.00%> (+2.73%)
:arrow_up:

pkg/localstore/subscription_push.go
77.19% <0.00%> (+3.50%)
:arrow_up:

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 f50cc34...3f3356e. Read the comment docs.


HOW to get the inline styles where the cursor is pointed to

How are you able to know the inline styles that a content/text have if it is focused or where the cursor is positioned?

Like the one shown in the image where the cursor is positioned it is bold.

Already found it using getSelectionInlineStyle of draft js utils. Thanks!


cannot resolve ENS reg with the new ens contract

Try
uni.bloomfilter.eth
or
bloomfilter.eth
They work fine in iOS and metamask, but not in Andriod

fixed in the pre-released build


ValueError: The number of weights does not match the population

How should this problem be solved, the environment configuration is the same as yours

Can you provide the detailed error information?

Have you solved this problem?I meet the same question now!

Can you provide the detailed error information?

i still cant fix it

Can you provide the detailed error information?

Can you provide the detailed error information?

Have you solved this problem?I meet the same question now!

no, have you fix it?

@JAYCHOU2020
It means your number of datasets and your number of probabilities of your datasets are not same.

@JAYCHOU2020 It means your number of datasets and your number of probabilities of your datasets are not same.

yes I'm only training one class "dog" of the lasot dataset, so where can I change the number?

See yaml file you used under experiment folder

yes I'm only training one class "dog" of the lasot dataset, so where can I change the nu

See yaml file you used under experiment folder

it works!thanks

yes I'm only training one class "dog" of the lasot dataset, so where can I change the nu

See yaml file you used under experiment folder

it works!thanks

我在训练的时候一直卡在这个位置，也不报错，请问是什么原因啊？

See yaml file you used under experiment folder

我在训练的时候一直卡在这个位置，也不报错，请问是什么原因啊？


Add FAQ.md to provide sample estimation guidelines

The new FAQ.md file includes detailed explanations and examples on how to estimate the number of synthetic samples produced at various stages of the SDG training process. This addition aims to enhance user understanding of the sample generation methodology.
I believe it's an MVP to resolving: https://github.com/instructlab/sdg/issues/307

These are a good initial set of FAQs that we have seen pop up with what I feel (based on my studies of the codebase) are the appropriate answers. More than happy to adjust anything that is inaccurate though based on expert opinion!


vpu computations location

I'm trying to figure out which files contain the actual code, which performs computation of operators (layers) on VPU.
For example, for CLDNNPlugin the code I'm refering to is located in .cl files.
For VPU, as far as I see, the serialised graph is being enqued to the device. But I can't locate the source of the actual operators.
Could someone please provide a reference if the source is available? If it is in mvcmd file being downloaded is it possible to open it in human-readable form?

Found the answer.


server: provide server health status when stats disabled

Description: This fixes a bug where the server health check returns an incorrect response when server stats are disabled i.e. due to relying on the server.live gauge.
Risk Level: Low
Testing: Added a new test
Docs Changes: N/A
Release Notes: N/A
Fixes #8473

@envoyproxy/senior-maintainers

@venilnoronha please do not force push, thank you!

/azp run

@mattklein123 I had to force push coz I rebased this branch with master. Will merge next time on.


Specifying Geometry in Moltemplate

Dear Andrew,
I'm wondering if there's an easy way to fill up the specified simulation box with particles.
I tried many ways of manipulating this matrix but I either get arrangments that are too small or outside the simulation box.
Thank you for your help.
Best regards,
Yunes Salman

I'm wondering if there's an easy way to fill up the specified simulation box with particles.

Hello Yunes
Please forgive me. This has been a particularly bad month, and I've been forced to ignore moltemplate questions for a couple weeks. (Unfortunately moltemplate is not relevant to my current job. So I reply moltemplate questions intermittently when I have time.)
Firstly, did you obtain the coordinates of your system from a PDB file or an XYZ file?
Or did you manually place the molecules where you want them to go using the "new Mol ...move()" syntax.  (As an example of the later, see the "system.lt" file from the water example.  This example is also discussed at the beginning of the moltemplate manual here.
I assume you have read chapter 4 of the moltemplate manual?  It's only a few pages long.  You can skip the first 3 chapters.)
In the later case, if you are creating a 3-dimensional array of molecules, it's usually possible to guess the size of the simulation box you need by multiplying the number of molecules by the spacing between molecules in the X,Y,Z directions.  In the example above, I created a 10x10x10 array of water molecules, each of which is placed 3.10342 Angstroms in the X,Y,Z.  So the resulting simulation box in that example had a width of 31.0342 Angstroms.  Keep in mind that by default the first molecule is placed at the origin (0,0,0), and the remaining molecules are shifted relative to that molecule.  (This doesn't have to be the case.  In this example, you can shift the entire block of water molecules after you create them using this command "wat[][][*].move(0,0,-15.5171).)
Here are some other suggestions which can help.
(See the second suggestion at the end of this message.)
1) Use the "-vmd" argument when you run moltemplate.sh
You can display the system in 3-D every time you run moltemplate, to visually inspect the system right after you created it by using the "-vmd" argument.
I suggest that you install VMD and run moltemplate.sh using the "-vmd" argument.  For example:
moltemplate -atomstyle full -vmd system.lt

This will automatically start the VMD molecule visualizer and display the system on your screen.  Using the "-vmd" argument makes this easier.  Once you have done that, you can display the periodic boundary conditions and customize the appearance of the atoms by following the instructions [here] (https://github.com/jewettaij/moltemplate/blob/c76cb98a817c1c7fc5b0f8b926b72fa032709dcd/examples/all_atom/force_field_explicit_parameters/waterSPCE%2BNa%2BCl/README_visualize.txt#L52), if you haven't already.  (The default visualization style is quite ugly, but you can change it by going to the "Graphics"->"Representations" menu option.  Note that when reading this README file, you can can skip to section 3, because moltemplate.sh will take care of the first step for you.  This README file also explains how to display the periodic images (copies) of your system to make sure atoms are not overlapping with their periodic copies.)
I realize you are probably already using VMD, but perhaps you don't know about the "-vmd" argument.  It makes starting VMD easier.
2) Use VMD to measure the distance between atoms.
Once your molecule is displayed in VMD, you can use the "Mouse"->"Label"->"Bonds" menu option and click on any pair of atoms.  (They don't have to be bonded together.  They could be on opposite sides of the simulation box, for example).  The distance between the atoms will be printed to the screen.  You can use this distance to estimate how large you need to make the simulation box.

Dear Andrew,
Thank you so much for this detailed response. I appreciate it.
I will try your suggestions.
Best regards,
Yunes Salman


Typo

There was an extra "b" at the start of the Guide section

lol WHAT? thank you for catching :)


Idle MemLeak fixes for Reflection and Injectable beacon

Applies the fixes @bblenard submitted in PR #34 to the reflection and injectable beacons.

Seems pretty solid, nice work

Seems pretty solid, nice work


[DCJ-627][risk=low] Remove deprecated Dataset needsApproval and active state attributes from consent service and database

Addresses
Addresses the consent and database portion of removing deprecated Dataset properties
Summary
Removes properties and related getters/setters from Dataset class, DTOs
Removes columns from SQL queries
Adds liquidbase script to drop the columns from the dataset table
See Also:
https://github.com/DataBiosphere/duos-ui/pull/2663

Thanks!


Bullet chart tooltip styling

Describe the bug
The bullet chart tooltips could look more like the PF4 tooltips.
To Reproduce
Steps to reproduce the behavior:

Go to 'Overview'
Click on 'AWS'
Hover over any chart with multiple lines of data
See error

Expected behavior
Style the bullet chart tooltips to use white text with a black background. Will not be implementing animation here.
Screenshots

Fixed by https://github.com/project-koku/koku-ui/pull/747


Android 4.4 KitKat translucent system bar issues

If you use the translucent status bar on Android 4.4 KitKat with the android:fitsSystemWindows theme attribute, AppMsges are shown above the action bar in the status bar area and you can't see much of them.

how do that @matthew-reilly

appMsg.setLayoutParams(getParams());

 /**
     * The new theme requires some layout tweaking for croutons
     */
    public FrameLayout.LayoutParams getParams() {
        FrameLayout.LayoutParams layoutParams = new FrameLayout.LayoutParams(
                LinearLayout.LayoutParams.MATCH_PARENT, LinearLayout.LayoutParams.WRAP_CONTENT);
        layoutParams.setMargins(0, getSupportActionBar().getHeight(), 0, 0);
        return layoutParams;
    }

Works for top. Not works for the bottom of the screen (transparent navbar).
setMargins(0, 0, 0, MARGIN_FOR_BOTTOM);


Fix missing handling of raw cells

I added additional code to change the cell type to or from "raw". Keyboard shortcut for changing to raw is "r".

Thanks @heseber!


Use helper to lookup fingerprinted files

Using a helper allows us to throw an error if the file is not found in the fingerprint array, making it harder to break the build without realising.

You can preview this change here:
Built with commit 30b669a2aa791eb1110d9715e4cad8cb7c04a246
https://deploy-preview-503--govuk-design-system-preview.netlify.com


Guardian UI fix

🗃 Github Issue Or Explanation for this PR. (What is it supposed to do and Why is needed)
No issue
✅ Checklist

[x] Github issue details are up to date for people to QA.
[x] I have tested all my changes.

🕵♂ Notes for Code Reviewer
UI fix for guardian flow.
🙈 Screenshots
For all UI changes

👯♀ Paired with
"nobody"

@gguij004  this is ready to merge


Fixed example for Basic filters for lists

In the example the code is:
const linksBeforeChristmas = await prisma
  .links({
    where: {
      createdAt_lt: "2017-12-24"
    }
  })

But the description says: Fetch links created before December 24, 2018:

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

thanks!


Add CI problem matcher for Sphinx.

Let's see if we can have CI show doc errors in the PR files view, which would be useful to diagnose/fix docs mistakes.

The following issues present themselves:
Checking sphinx-doctest for errors picks them up, but since we are in the working directory /docs, the files are not resolved to the correct location, so the errors are annotated at the correct location, and are not displayed in the PR's "Files" section
Sphinx-doctest error messages consist of one line for the path, and then several lines for the error message, followed by a line o ****. Github actions problem matchers cannot deal with this correctly currently. See also https://github.com/actions/toolkit/issues/193 https://github.com/actions/toolkit/issues/319
Interestingly, the VS code problem matchers this seems to originate from supports a  fileLocation field where you can indicate how the path is supposed to be read.

Reported a problem resolving the relative paths out of make doctest here: https://github.com/actions/runner/issues/765.

I have squashed & pushed this directly to master, but wanted to avoid force-pushing here so that the logs presented in https://github.com/actions/runner/issues/765 don't disappear. Closing as everything relevant is in master now.


fontFamily 'Roboto_medium' is not a system font and has not been loaded through Exponent.Font.loadAsync

Loading the app in the Expo client in Android triggers red screen with error

fontFamily 'Roboto_medium' is not a system font and has not been loaded through Exponent.Font.loadAsync.

If's simple test app, just displaying an Native-base card.  Works fine on iOS.  Even in Android, I can see it display the NativeBase card briefly before it red screens.   I am loading the font as per the instructions in the readme:
import { Font } from "expo";

class NHCardShowcase extends Component {
  async componentDidMount() {
    await Font.loadAsync({
      Roboto: require("native-base/Fonts/Roboto.ttf"),
      Roboto_medium: require("native-base/Fonts/Roboto_medium.ttf")
    });
  }

Update
After posting this, I noticed that the readme uses componentWillMount rather than componentDidMount.  When I changed my code accordingly though, I get the error

You started loading 'Roboto_medium', but used it before it finished loading

Googling for that "used it before it finished loading" error came with this SO post, which gave me the workaround:
https://stackoverflow.com/questions/47308557/loading-font-native-base-error
Basically, I had to delay rendering of my top-level component until the font has been loaded, like so:
class HomeNav extends Component {
  constructor(props) {
    super(props);
    this.state = { loading: true };
  }

  async componentWillMount() {
    await Font.loadAsync({
      Roboto: require("native-base/Fonts/Roboto.ttf"),
      Roboto_medium: require("native-base/Fonts/Roboto_medium.ttf")
    });
    this.setState({ loading: false });
  }

  render() {
    if (this.state.loading) {
      return (
        <Root>
          <AppLoading />
        </Root>
      );
    }
    return (
      <Root>
        <AppNavigator />
      </Root>
    );
  }
}

I'm not happy that you have to put in Expo-level hacks like this, but it is, at least, only on one component.

For anyone still having issues, the following worked for me:
export default class App extends React.Component {
  state = {
    fontLoaded: false,
  }
  async componentWillMount() {
    await Expo.Font.loadAsync({
      'pixel-font': require('./assets/fonts/PixelFJVerdana12pt.ttf'),
    });
    this.setState({ fontLoaded: true });
  }
  render() {
    return (
      <View style={styles.container}>
        {
          this.state.fontLoaded ? (
            <Text style={{ fontFamily: 'pixel-font', fontSize: 56 }}>
              Hello, world!
            </Text>
          ) : null
        }
      </View>
    )
  }
}

Hmm I'm having this issue, not finding a solution yet :(

@arrygoo can you try brownieboy's solution

For me it turned out to be a completely unrelated issue. In fact, a lot of time when my expo app throws an error, it shows the error above. Once I fix the issue, the error about exponent loading the font goes away too.
I wonder if this means that the error is always there but masked by something else? Or if expo is throwing this error incorrectly when it's supposed to show the actual place of error/crash.

After cobbling together bits and pieces on loading the icons for react-native-elements I finally figured this out. Check out my blog for a write-up.
https://javascriptrambling.blogspot.com/2018/03/expo-icon-fonts-with-react-native-and.html


Add configuration options to local transfer service

Part of #7592   Add configuration options to local transfer service.
Signed-off-by: Tony Fang<EMAIL_ADDRESS>

@dmcgowan  I addressed your comments, fixed CI failures, and rebased main.
Can you take another look?

@dmcgowan  @fuweid
Thanks for your reviews!
Followup added in #8165


Add a configurable repeatable break button

Please add a button where I can just click give me a Xmin break, rather than having to type it every time. I use the custom break times function in conjunction with website blocking, having an option to just select give me a 20min break, instead of having to type it every time would be very convenient.

Hi @fassina2 ,If the extension remembers the last Xmin you typed, will it fit your workflow?

That'd be wonderful @ofekmiz ;)


Openssl 1.0.2 has no compatibility with FIPS object module v2.0.

I tested FIPS object module v2.0 and Openssl 1.0.1 on my source code.
Openssl 1.0.1 was EOL at End of 2016 then I shifted to Openssl 1.0.2j.
However I got compile error on same source code.
"error: unknown type name 'ASN1_OBJECT'"
I found typedef ASN1_OBJECT was missing since OpenSSL 1.0.2.
/usr/local/ssl/fips-2.0/include/openssl/evp.h
-> /usr/local/ssl/include/openssl/objects.h
-> /usr/local/ssl/include/openssl/asn1.h
OpenSSL 1.0.1:
 * These are used internally in the ASN1_OBJECT to keep track of whether the
 * names and data need to be free()ed
 */
 # define ASN1_OBJECT_FLAG_DYNAMIC         0x01/* internal use */
 # define ASN1_OBJECT_FLAG_CRITICAL        0x02/* critical x509v3 object id */
 # define ASN1_OBJECT_FLAG_DYNAMIC_STRINGS 0x04/* internal use */
 # define ASN1_OBJECT_FLAG_DYNAMIC_DATA    0x08/* internal use */
typedef struct asn1_object_st {
    const char *sn, *ln;
    int nid;
    int length;
    const unsigned char *data;  /* data remains const after init */
    int flags;                  /* Should we free this one */
} ASN1_OBJECT;

OpenSSL 1.0.2 and OpenSSL 1.1.0:
 * These are used internally in the ASN1_OBJECT to keep track of whether the
 * names and data need to be free()ed
 */
 # define ASN1_OBJECT_FLAG_DYNAMIC         0x01/* internal use */
 # define ASN1_OBJECT_FLAG_CRITICAL        0x02/* critical x509v3 object id */
 # define ASN1_OBJECT_FLAG_DYNAMIC_STRINGS 0x04/* internal use */
 # define ASN1_OBJECT_FLAG_DYNAMIC_DATA    0x08/* internal use */
struct asn1_object_st {
    const char *sn, *ln;*
    int nid;
    int length;
    const unsigned char *data;  /* data remains const after init */
    int flags;                  /* Should we free this one */
};

Could you check this issue to use FIPS object module v2.0 and Openssl 1.0.2?

The typedef is in ossl_typ.h in 1.0.2.

Dear snhenson,
Thank you for your support.
I will check it.

Facing the following error when trying FIPS 2.0 with SSL 1.0.2n:
[('SSL routines', 'ssl3_read_bytes', 'sslv3 alert bad record mac')]
Could this be because of any compatibility issues with the two?


Terra-toolkit Nightwatch Testing Uplift

Summary
Terra-toolkit has be moved it its own repository. Those changes allow cleanup the package level in regards to nightwatch testing. The changes in this PR include:

Removes terra-toolkit from the repository
Removes terra-toolkit as a dev-dependency at the package level
Updates nightwatch test scripts to use the root level node_modules/terra-toolkit
Removes the nightwatch.conf.js file from each package

Addresses issue https://github.com/cerner/terra-core/issues/430 and https://github.com/cerner/terra-clinical/issues/82

Probably should update the readme.md too. to remove terra toolkit from the list of packages

See build errors along the following:
ERROR in ./packages/terra-i18n/lib/translationLoaders.js
Module not found: Error: Can't resolve 'en.js' in '/home/travis/build/cerner/terra-core/packages/terra-i18n/lib'

Gonna restart the build

@emilyrohrbough<EMAIL_ADDRESS>is released which should help with translation path issues we're seeing on travisCI


Can't find the laptop in the same local network

I have had 4 devices test.
1 PC, 1 Thinkpad laptop, 2 Pixel phones
The PC and 2 phones can find each others but the PC and phones can't find the Thinkpad.
However the Thinkpad is able to find the PC and the phones. Test and files can be sent to the PC.
remark:

all are in the same local network/WiFi;
all app with server started;
all with default port

Thank you for your nice work.

Thanks. You are right.
it's done now.

Thanks. changing the firewall rules on parrot os to allow in-requests for port 53317 fixed it


Installation with Helm fails behind proxy

Installation with Helm fails behind proxy because "values don't meet the specifications of the schema".
How to reproduce the bug

Kubernetes cluster behind proxy
Install Superset with local (pulled) Helm chart without any additional values.
Installation fails

Expected results
Superset is successfully installed in the cluster.
Actual results
Installation fails.
Screenshots
This is the full error text:

Error: INSTALLATION FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
superset:
Get "https://raw.githubusercontent.com/yannh/kubernetes-json-schema/master/v1.23.0/_definitions.json": dial tcp: lookup raw.githubusercontent.com: getaddrinfow: This is usually a temporary error during hostname resolution and means that the local server did not receive a response from an authoritative server.
Proxy works correctly:

Environment

Kubernetes: 1.23.4
Helm: 3.7.2
Superset: 0.6.4

Checklist
Make sure to follow these steps be
fore submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
To test if the dependencies (Redis and Postgres) are somehow responsible for the issue, I have tried the same with an external installation of both. The same error occurs. I couldn't yet try installation in a cluster without a proxy, because I don't have easy access to one.

Hey, do you still have this error ? Because I think it was a temporary error threw by your local DNS Resolver while resolving raw.githubusercontent.com

Hey, sadly, the error persists. In the second image ("Proxy works correctly:"), I use wget to call and get the same address as in the error message and it works. So my assumption was, that there must be some other error. Or, conversely, that "helm install" or some other aspect of the installation doesn't respect the proxy settings. Other charts install just fine, so I am somewhat confused why this one doesn't.

Hey, I looked it up and these URLs that's Helm is failing to call are legit (it's a schema verification process on the Chart itself). So it's Helm that does not use the Proxy to fetch those files and I found this (a bit sketchy issue) that claim that a simple export HTTP_PROXY=some-ip:some-port made Helm pass through the desired proxy.
Let me know if this works.

Thank you, this solved the issue.
Since I am on Windows, I employed this solution:[https://github.com/helm/helm/issues/9576#issuecomment-817896885] (Github Helm Issue) to enforce the proxy.
Now, why all other Charts have worked so far without this additional step, and why Helm doesn't respect the general proxy settings on Windows (which is a problem with Helm, not Superset), are still open questions. But the installation was now successful.


Default Region in collect.py code

Note the Network Visualization functionality (command prepare) is no longer maintained.
Code being called - collect.py
Working out of a virtualenv environment and Docker
I have a few questions regarding the default region in the collect.py code.

What does the following code snippet do ?

default_region = os.environ.get("AWS_REGION", 'us-east-1')
if 'gov-' in default_region:
    default_region = 'us-gov-west-1'
elif 'cn-'  in default_region:
    default_region = 'cn-north-1'
else:
    default_region = 'us-east-1'

Does the code in the snippet set the default region to us-east-1 if the other conditions are not met ?
What if I have a default region set to us-west-1 in my aws config ? Will us-west-1 be used instead of the above ?

Can anyone respond to my queries?

Seems like it gets the environment variable 'AWS_REGION' and if its not set it makes the default region be 'us-east-1', but I am guessing.


Add zenn

General:

[ ] The pull request title is descriptive. (ex. Added Lunar Network, not Updated metadata.json)
[ ] The pull request does not contain any unrelated commits. (ex. commits from previous pull requests)

Mapping Additions or Updates:

[ ] Folder with appropriate name has been created / updated (must not include any non-alpha characters or whitespace).
[ ] No changes were made to the file's formatting (4 spaces per tab).
[ ] There are no syntax errors.
[ ] There is no pre-existing mapping matching my id (check if there is an existing folder).
[ ] My field values match your requirements.
You can view our patterns here: metadata.schema.json, or take a look below and complete the field checklist:

[ ] id: a lowercase string, which should match the folder name (ex. myserver)
[ ] name: a string (ex. MyServerPvP)
[ ] description: hook between 16 and 80 characters (ex. Home of Competitive Minecraft PvP)
[ ] addresses: an array with lowercase strings (ex. ["my.server", "your-server.com"])

You do not need to specify subdomains, Lunar Client services automatically detect them.

[ ] primaryAddress: the primary address that people connect to with (please include the subdomain if required) (ex. mc.my.server)
[ ] minecraftVersions: an array with Minecraft versions as strings _(ex. ["1.18._", "1.19.2"] - Must be versions or subversions supported by Lunar Client)*
[ ] primaryMinecraftVersion: a Minecraft version as a string (ex. 1.19.2 - Must be a subversion supported by Lunar Client)
[ ] primaryColor: a hexademical color code that primarily distinguishes the server (ex. #00FFFF)
[ ] secondaryColor: a hexademical color code that accompanies the primaryColor of the server (ex. #FF0000)
[ ] primaryRegion: the primary region where your server operates in (ex. NA)
[ ] regions: a list of regions where you have servers located that service your players (ex. ["NA", "EU", "AS"])
[ ] gameTypes: a list of games that describe the content on your server, must be a max of 3 listed (ez. ["PVP", "UHC", "HCF"])
[ ] crossplay: whether the server has support for Bedrock Edition players (through proxies such as GeyserMC)
[ ] (optional) presentationVideo: YouTube video ID (in slug) to server trailer / introduction (ex. 7EV4cPuJvXE)
[ ] (optional) website: url of server website, must include URL schema (http:// or https://) (ex. https://www.your-server.com)
[ ] (optional) store: url of server store, must include URL schema (http:// or https://) (ex. https://store.your-server.com)
[ ] (optional) wiki: url of server wiki, must include URL schema (http:// or https://) (ex. https://wiki.your-server.com)
[ ] (optional) merch: url of server merchanise site (if seperate from store), must include URL schema (http:// or https://) (ex. https://merch.your-server.com)

Socials

[ ] (optional) twitter: username of twitter account without the @ (ex. MyServer)
[ ] (optional) discord: invite link to discord (ex. https://discord.com/invite/4ceEJuH)
[ ] (optional) youtube: slug or username of youtube channel (ex. MyServer)
[ ] (optional) instagram: username of instagram account (ex. MyServer)
[ ] (optional) twitch: username of twitch account (ex. MyServer)
[ ] (optional) telegram: slug of telegram group (ex. MyServer)
[ ] (optional) reddit: slug of subdreddit wtih 'r/' (ex. MyServer)
[ ] (optional) tiktok: username of tiktok account (ex. MyServer)
[ ] (optional) facebook: slug of facebook page (ex. MyServer)

Compliance

[ ] (optional) privacyPolicy: url to your Privacy Policy (ex. https://www.lunar.gg/privacy)
[ ] (optional) termsOfService: url to your Terms of Service (ex. https://www.lunar.gg/terms)
[ ] (optional) rules: url to your Rules (ex. https://www.lunar.gg/rules)
[ ] (optional) support: url to your Support Site (ex. https://www.lunar.gg/support)

Media:
Logo

[ ] My logo is a png file.
[ ] I have uploaded my logo to my server folder (ex. lunarnetwork) and named it logo.png.
[ ] My logo has a transparent background and is square (1:1 aspect ratio).
[ ] My logo is 512 pixels in width and height.
[ ] My logo is my own property and complies with relevant copyright/privacy laws.

Background

[ ] My background is a png file.
[ ] I have uploaded my background to my server folder (ex. lunarnetwork) and named it background.png.
[ ] My background has an aspect ratio of 16:9.
[ ] My background is atleast 1920 pixels in width and 1080 pixels in height (can be bigger, but must remain the same aspect ratio).
[ ] My background contains relevant content pertaining to the server, is my own property, and complies with relevant copyright/privacy laws.
[ ] My background doesn't contain any markings, text, or logos (unless part of a build).

You need an actual logo

That is our actual logo, made by ourselves and used for everything. Not our website yet (Needs to be uptaded)

Just need to put an actual logo (Not a single poorly drawn letter).

I don't get what you don't get. This is our actual logo. And yes it is a single letter. I have seen many other servers with a single letter in the servers folder.

Yes, some servers have a single letter as their logo. However, there is an effort with them such as a sort of design on the letter or background they also don't have an obvious uneven border around the letter.

Yes, some servers have a single letter as their logo. However, there is an effort with them such as a sort of design on the letter or background they also don't have an obvious uneven border around the letter.

Ok, i guess i will have to redesign our logo. But i an not home so please keep this request open. I will come home on friday.

Reopen when it's done


[Feature]: App extension development - Ability to customize which files are pushed

What area(s) will this request affect?
Extension
What type of change do you want to see?
Substantial change to existing feature
Overview
Currently when developing an app extension the CLI attempts to push all subfolders into the app. However, during development we may have other folders in that directory we want to ignore (node_modules, dist, babel.rc, webpack, etc.).
There is no way (that I know of) to ignore or customize what gets parsed and sent up.
We could use an included .gitignore file to specify what folders and files to ignore, or even use the .theme-check.yml file to determine extra customization on parsing.
Motivation
I was attempting to build a simple app extension, and the inclusion of my node_modules folder seemed to absolutely destroy my CPU. It seems that ruby was doing some parsing on this folder.
Also, I'm trying to do some weird stuff with metafields inside my {% schema %} and it is not letting me push my *.liquid files because of a parsing error. I'd like to just be able to ignore the parser and force the changes up no matter what.

Hi there, just double checking, are you trying to create a theme app extension?


Add GPT-2 with flash attention

What does this PR do?

This change adds FlashGPT2ForCausalLM and wires it up. The model itself is pretty straightforward, the main difference from other models is that it uses trained position embeddings and that all weight matrices are transposed compared to other models (due to the use of Conv1D in the upstream model).

Fixes # (issue)
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[x] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[x] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@Narsil

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


fix fix wait publish txn too long when not meet quorum

Suppose there are n commit txn but not visible,  and alive replica num is not meet quorum.
Then  the first txn will wait time t,  then publish succ.
Then  the second txn will wait  another time t,  (total time 2t),  then publish succ.
...
So the n-th txn will wait total time n * t, then publish succ.
If comming more txns, the num of waiting txns will keep piling up.
So we modify the wait time, if time from its first publish time, the txn will publish succ event if not meet quorum.
Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall


How to use

localhost:3000 got response 404

Hi!
Sorry for the late reply (didn't have notifications turned on).
This project is not complete at the moment, so unless you know what to do, it's not recommended to be used yet.
I'll be updating the documentation over time, so please check this repo in a couple months when it is more ready.


Appsec action: deployment.yml securityContext settings

Container 'find-moj-data' of Deployment 'find-moj-data-deployment' should set securityContext.readOnlyRootFilesystem to true
(An immutable root file system prevents applications from writing to their local disk. This can limit intrusions, as attackers will not be able to tamper with the file system or write foreign executables to disk)
See https://avd.aquasec.com/misconfig/ksv014

Container 'find-moj-data' of Deployment 'find-moj-data-deployment' should set securityContext.allowPrivilegeEscalation to false
(A program inside the container can elevate its own privileges and run as root, which might give the program control over the container and node)
See https://avd.aquasec.com/misconfig/ksv001

Container 'find-moj-data' of Deployment 'find-moj-data-deployment' should set securityContext.runAsNonRoot to true
(Force the running image to run as a non-root user to ensure least privileges)
See https://avd.aquasec.com/misconfig/ksv012

Container "find-moj-data" of deployment "find-moj-data-deployment" in "default" namespace should specify a seccomp profile
(A program inside the container can bypass Seccomp protection policies)
See https://avd.aquasec.com/misconfig/ksv104

[ ] set 'securityContext.readOnlyRootFilesystem' to true
[ ] set 'securityContext.allowPrivilegeEscalation' to false
[ ] set 'securityContext.runAsNonRoot' to true
[ ] specify a seccomp profile
[ ] Regression testing - does readOnlyRootFilesystem or seccompProfile RuntimeDefault break anything? (privilege escalation and running as root shouldn't be required 😬 )

Examples
spec:
  [...]
  template:
    [...]
    spec:
      [...]
      containers:
        - name: find-moj-data
          [...]
            seccompProfile:
              type: RuntimeDefault

https://github.com/ministryofjustice/find-moj-data/pull/767


BFT chain test: update

Type of change

Test update

Description
This PR includes rebasing and updates to the BFT chain unit tests

Just in case I was not clear, we are not going to merge this anytime soon. Let's not waste time on rebasing it.

That is correct. We do not plan to merge this one as is, but rather use it as a "blue print" for writing a new chain_test.go that does unit testing for the chain.  The plan is to create chain_test.go in steps where we control every step via a PR, rather then trying to merge this big PR. Rebasing it is basically done, it was not too difficult.

completed in #4704


[FLINK-30845] HttpArtifactFetcher: Fix filename

What is the purpose of the change
The HttpArtifactFetcher now only keeps the actual file name without the URI params as the file name. Previously the params were part of the resulting file name. This lead to problems submitting job files as Flink checks that the name of job files submitted end with ".jar".
Brief change log

Filename determined from Job URIs do not contain URI-params anymore

Verifying this change
This change is already covered by existing tests, such as ArtifactManagerTest.
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 changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? no

Sorry I accidentally linked the wrong issue in the title. The git-bot does not seem to pick up the change thou...


docs: update config files

Description
Updated config files make clear that the firebase name is relative to the project ID and added missing env variables
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

@rhahao
CROWDIN_PROJECT_NAME used in src/v2/utils/announcement-utils.js. Should we keep it?
CROWDIN_PROJECT_ID not used in the app?
should we remove CROWDIN setup from the local setup guide?
Thanks!

Hi @ild0tt0re, we no longer need the Crowdin setup so these can be removed. Thanks.

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


[Text Analytics] Add samples for dynamic classification

Resolves: #31882

API change check
API changes are not detected in this pull request.


OpenSslEngine may use networkBIO after calling shutdown

Motivation:
To ensure that all bytes queued in OpenSSL/tcnative internal buffers we invoke SSL_shutdown again to stimulate OpenSSL to write any pending bytes. If this call fails we may call SSL_free and the associated shutdown method to free resources. At this time we may attempt to use the networkBIO which has already been freed and get a NPE.
Modifications:

Don't call bioLengthByteBuffer(networkBIO) if we have called shutdown() in ReferenceCountedOpenSslEngine

Result:
Fixes https://github.com/netty/netty/issues/6466

@rkapsi - can you verify this fixes the issue?
I will try to come up with a unit test tomorrow.

@Scottmitch just merge if a unit test is not feasible

4.1 (70bcb40) 4.0 (417eea8)

Creating a unit test for this requires more investigation. Reducing the BIO size gets us into this spot but I'm not sure how to make SSL_shutdown return an error.

@Scottmitch I'll try to verify before EOD. Trying to squeeze multiple things through our release pipeline.


test(0): shallow clone

This PR is going to test that the backport-action can run in a shallowly cloned repo (checkout action with depth=1).

This test was successful 🎉


Added Muhammad Usman to contributors list

Hi, I am a senior Android and web developer with over 6 years of experience in the development domain. I have developed various mobile and web solutions and am strongly quality-oriented, attentive to detail, and a tech enthusiast. I am willing to work with the revivalists movement. Thanks!

welcome @msusman1


YouTube ERROR: content too short when used with "-f best"

Please follow the guide below
Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.06.25. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.06.25

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

C:\>youtube-dl.py "https://www.youtube.com/watch?v=QLvJ5R0tv9Y" -v -f best
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=QLvJ5R0tv9Y', '-v', '-f', 'best']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2017.06.25
[debug] Python version 3.5.1 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-71727-g46778ab, ffprobe 3.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] QLvJ5R0tv9Y: Downloading webpage
[youtube] QLvJ5R0tv9Y: Downloading video info webpage
[youtube] QLvJ5R0tv9Y: Extracting video information
[youtube] QLvJ5R0tv9Y: Downloading MPD manifest
[debug] Invoking downloader on 'https://r8---sn-cxaaj5o5q5-tt1k.googlevideo.com/videoplayback?source=youtube&mm=31&mn=sn-cxaaj5o5q5-tt1k&ip=<IP_ADDRESS>&ratebypass=yes&mt=1498767836&mv=m&pl=24&id=o-AE
KgjQqusKB-C-24G543Cq1T71KVNcgeHjFYWvzkLntI&ei=RGJVWbDzCpnkDsnDhxg&ms=au&key=yt6&initcwndbps=13148750&lmt=1469779567592727&requiressl=yes&sparams=dur%2Cei%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Clmt%
2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpcm2cms%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cexpire&expire=1498789540&ipbits=0&signature=B67A6182AF7437EDFC129E015C685B1BB8289327.46F0A1031FCA407A18831D7FA63661187754
458C&pcm2cms=yes&dur=43.305&mime=video%2Fmp4&itag=22'
[download] Destination: Shapovalov Wows Crowd With Double Hot Shot Toronto 2016-QLvJ5R0tv9Y.mp4
[download]   0.0% of 8.46MiB at 17.01KiB/s ETA 08:29         ERROR: content too short (expected 8875291 bytes and served 1087)
Traceback (most recent call last):
  File "C:\TRANSMOGRIFIER\youtube-dl.py\youtube_dl\YoutubeDL.py", line 1834, in process_info
    success = dl(filename, info_dict)
  File "C:\TRANSMOGRIFIER\youtube-dl.py\youtube_dl\YoutubeDL.py", line 1776, in dl
    return fd.download(name, info)
  File "C:\TRANSMOGRIFIER\youtube-dl.py\youtube_dl\downloader\common.py", line 361, in download
    return self.real_download(filename, info_dict)
  File "C:\TRANSMOGRIFIER\youtube-dl.py\youtube_dl\downloader\http.py", line 238, in real_download
    raise ContentTooShortError(byte_counter, int(data_len))
youtube_dl.utils.ContentTooShortError: Downloaded 1087 bytes, expected 8875291 bytes

<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=QLvJ5R0tv9Y

Description of your issue, suggested solution and other information
The exact same YouTube URL downloads correctly without the "-f best" option.
Thanks as always...
Ringo

Not available at Youtube side.


[BACKLOG-32395] Update 'JSON Input' step to use enhanced VFS browser. …

…Update to File or Folder dialog.

:white_check_mark: Build finished in 2m 41s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/json/core

:ok_hand: All tests passed!
Tests run: 80, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Roll engine 3f27c5724c1d..bf91bdb8b223 (1 commits)

https://github.com/flutter/engine/compare/3f27c5724c1d...bf91bdb8b223
git log 3f27c5724c1d92c14f8353419dfc6cb40a063aa2..bf91bdb8b223bf39572db9318318aebe88477d72 --no-merges --oneline
bf91bdb8b Roll src/third_party/skia 563d044c0e76..9d844d8ce5a2 (10 commits) (flutter/engine#9831)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (garyq@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/4810354767429632

Commit queue failed; closing this roll.


Metadata

Is there any way to access the metadata.account_id parameter in the callback controller?

Can you give me an example of what you're looking for?  I searched the Plaid docs for the metadata.account_id field and wasn't clear what you're after.  I'm sure there is a way to access the data you're looking for in that controller.
Thanks!

It seems that to actually get a bank account token back you need to pass
public key and account_id.
As far as I know, the only way to get that is through the callback from the
plaid js, it is passed as metadata.account_id.
Thanks for any help!
James
On May 10, 2016 8:23 PM, "Jason Barry"<EMAIL_ADDRESS>wrote:

Can you give me an example of what you're looking for? I searched the
Plaid docs for the metadata.account_id field and wasn't clear what you're
after. I'm sure there is a way to access the data you're looking for in
that controller.
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/JCBarry/argyle/issues/4#issuecomment-218352113

To be more clear, I'm wondering if you have successfully received a stripe_bank_account_token from Plaid? If so, can you perhaps share an example of the code used to do so?

Hey! Alright, you're using Stripe -- I haven't used Argyle for this purpose -- but I can't see why it wouldn't work.  You can try to take a look at the params in the controller.  I suspect you should see the account ID in addition to the public token.
I'll take a look at this as well.

Okay! That makes sense. I ended up working around Argyle, replacing the plaid_link with js/html from Plaid's docs, this allows me to customize the onSuccess function and send the account_id as :account_id.
The challenge I'm facing now, is that when I use the client Argyle passes to exchange the token, with that access_id plaid just passed (required to get a bank account token), Plaid says 'account not found'.
I have one idea -- perhaps creating the 'user/account' on Plaid first is necessary, before exchanging the token.

Doing a quick test, I was able to see that "account_id" will be available your callback controller as params[:account_id].
Here is a look at the raw request params to the callback using Argyle:
{
"utf8"=>"✓",
"authenticity_token"=>"Si5StRJt234VEDnxihQhcGuKpqOJ/QFYM2342ufOyk=",
"account_id"=>"",
"institution"=>{"name"=>"Chase", "type"=>"chase"},
"public_token"=>"test,chase,connected",
"controller"=>"user/settings/plaid",
"action"=>"callback"
}

I believe it is empty because I don't have Stripe enabled on our account.
In order to use this, we should update the gem and change the plaid_exchange_token method to optionally send the account_id.  In the meantime, you could override the method in your controller to make the request like this:
def plaid_exchange_token
  @plaid_access_token_response ||= Argyle.plaid_client.exchange_token(plaid_public_token, account_id: params[:account_id])
end

Thanks so much for the help. That is precisely what I did. The problem is that, when I try to use Argyle's plaid_client to do the exchange, I get the error:
[2016-05-11T13:06:58.301870 #17658]  INFO -- : Completed 500 Internal Server Error in 494ms (ActiveRecord: 10.0ms)
F, [2016-05-11T13:06:58.304443 #17658] FATAL -- :
Plaid::NotFound (account not found):
app/controllers/plaid_controller.rb:51:in `plaid_exchange_token'
Whereas when I simply use a Faraday request to exchange the token myself, it actually works, passing exactly the same data.
Any ideas?

When I last published the gem, plaid-ruby was stable at 1.7.1.  It looks like the account_id options were added in the 2.0.0.beta version of the gem.
It also looks like they made massive changes to the gem -- so this will need some testing anyway.

Thanks so much for this!
Just to be clear, when you checked earlier for the account_id being passed along, was that with the alpha version of the gem? Because I was pretty sure I tried using params[:account_id] and it wasn't accessible, using 1.7.x.
James

It does look like there are a lot of breaking changes in 2.x, including parameter changes for 3+ functions.

Yeah lots of breaking changes in the 2.0 release of the gem -- I'll start working through those.  It's still in alpha but it looks like they are prepping for a stable release shortly.
The account_id being passed along isn't contingent on the gem version -- when the user finishes going through the Plaid flow modal, it will post to the form.  The JS loaded directly from Plaid takes care of this and I would assume it added this field to the form when they made the Stripe announcement.

Check out the 2.0 branch in your project and see if it solves some issues for you -- tests are passing and following the plaid-ruby upgrade document I think I covered everything.
If you wouldn't mind giving it a shot that would be helpful.

I released a new version of the gem 1.0.0.beta that includes the recently released 2.0 version of the official plaid ruby gem which has support for the account_id user exchange.
Unfortunately, I don't have access to Stripe ACH with Plaid so I can't fully test that flow.