Can't update ruby-saml under older rubies

We have an existing app running with an older version of ruby and ruby-saml, integrated with omniauth-saml and devise.  We're in the middle of the long incremental process to modernize it, and we are currently trying to update ruby-saml to address the recent critical vulnerability, but we're having a problem.  No matter what I do, bundler refuses to install ruby-saml 1.17.0 under ruby 2.3.
If I try updating the version specified in my Gemfile and then running bundle update ruby-saml, I get the following:
% bundle update ruby-saml
Fetching gem metadata from https://rubygems.org/...........
Fetching gem metadata from https://rubygems.org/.
Resolving dependencies...
Bundler could not find compatible versions for gem "nokogiri":
  In Gemfile:
    nokogiri (= 1.10.10)

    ruby-saml (= 1.17.0) was resolved to 1.17.0, which depends on
      nokogiri (>= 1.13.10)

If I manually update Gemfile.lock to try to bypass Bundler's dependecy resolution, I get this error instead:
% bundle install
[...snip...]
Using omniauth 1.3.2
Fetching rexml 3.2.5
Installing rexml 3.2.5
Fetching ruby-saml 1.17.0
Downloading ruby-saml-1.17.0 revealed dependencies not in the API or the
lockfile (nokogiri (>= 1.13.10)).
Either installing with `--full-index` or running `bundle update ruby-saml`
should fix the problem.

Notably, these errors occur outside of our app as well.  Under ruby 2.3, rubygems 3.3.26, and bundler 2.3.26, the following minimal Gemfile fails with an error roughly equivalent to the first one above:
# frozen_string_literal: true

source 'https://rubygems.org'

ruby '2.3.8'

gem 'ruby-saml', '1.17.0'

...and instead I get something more like the second error above if I add this minimal lockfile:
GEM
  remote: https://rubygems.org/
  specs:
    mini_portile2 (2.4.0)
    nokogiri (1.10.10)
      mini_portile2 (~> 2.4.0)
    ruby-saml (1.17.0)
      nokogiri (>= 1.10.10, < 1.11.0)

PLATFORMS
  x86_64-linux

DEPENDENCIES
  ruby-saml (= 1.17.0)

RUBY VERSION
   ruby 2.3.8p459

BUNDLED WITH
   2.3.26

I tried similar experiments under ruby 2.5, but I got similar results.
In researching this issue, I did find #588, but the blog entry referenced in that issue doesn't provide any advice beyond "try updating the lockfile by hand", which as I mentioned has not helped.
Looking at the gemfile, the errors I quoted above suggest that bundler is hitting the else clause at https://github.com/SAML-Toolkits/ruby-saml/blob/v1.17.0/ruby-saml.gemspec#L57C1-L60C6 when trying to build runtime dependencies based on RUBY_VERSION.  I don't know enough about the inner workings of bundler to guess at why it's doing that though...
EDIT: minor clarification

Interesting twist: I tried switching to a :git source to see if that would be sufficient to cause Bundler to take a different path for dependency resolution, and...it worked.  Here's a minimal Gemfile that installs successfully under ruby 2.3.8, rubygems <IP_ADDRESS> (included with ruby 2.3.8), and bundler 1.17.3:
# frozen_string_literal: true

source 'https://rubygems.org'

ruby '2.3.8'

gem 'ruby-saml', :git => 'https://github.com/SAML-Toolkits/ruby-saml.git',
                 :tag => 'v1.17.0'

I was able to use the same trick in my application's Gemfile.  At the very least, this confirms that my manually updated Gemfile.lock was syntactically correct.  It doesn't help me understand the errors I reported earlier, but at least I can move our application forward...

@ilikepi  I also delivered 1.12.3 in case to cover those very old projects using ruby-saml.
Glad you were able to solve your use case by using code from Github rather than from Rubygems.

Thanks, I did see that 1.12.3 was available, but I wanted to be current if possible in our application.
I see that there are CI runs for which 1.17.0 passed under old ruby versions, but I don't really know anything about how that works to be able to figure out how the CI environment differs from my own local environment.


Rename force_parallelism -> verify_parallelism

The documentation still mentions the pragma as force_parallelism. It needs to be updated.
Mentioned here: https://duckdb.org/docs/sql/pragmas

Fixed by #350


[DOC] Fix broken link and update shell script location in HowTo.md

What changes were proposed in this pull request?
Fix broken link and update shell script location in HowTo.md
How was this patch tested?
Not applicable.

@zhztheplayer Can you also help review it? I just update the doc broken link and update shell location.

Thank you for the fix!


location save

Hello, how can i save screen record video in another place, or in cloud like firebase

Hi,
when you stop the video you get the URI (path) of the saved file, this is usually a path to the local sandbox of the app.
From there you can use other libraries to save it to the photo library, firebase, etc or use filesystem API to copy the file to another location...

sorry but i can't get the video from local sandbox of the app. a path like "/storage/emulated/0/android/data/" and i can't save it to the photo library or another one. can you give me a example to save video to photo library. i code android app with react native

tks bro


groups: s3 image upload buttons

I think i caught all the places where you're supposed to paste an image url with these, let me know if i'm missing any. (notebook cover image is covered by #1142)

@jamesacklin all the talk stuff works for me – are you making sure to run npm run chat and not npm run dev?


Script does not delete "down" hosts

Hi!
Through this point:
The script will iterate through each network to find and delete any hosts registered in NetBox that did not respond to scan, and have the tag defined in the configuration file.
i can't understand, how script understands that host is down and script should delete this host.
I tried some scenarios, but nothing works.

[GENERAL]
tag       = test
unknown   = unknown host
log       = .
nmap_args = -p 22,53,80,443

[NETBOX]
address = https://netbox-test
token = fadbdf2179d438f086
tls_verify = False

[TACACS]
user     =
password =
command  = show run | inc hostname
regex    = hostname ([A-Z|a-z|0-9|\-|_]+)
regroup  = 1

[SCAN]
networks = <IP_ADDRESS>/24

yop. problem is this commit. https://github.com/lopes/netbox-scanner/commit/0b03893204a2d9487e3789cb157a19f09ba2707c

Yeah, it was pointed out in #6 . It'll be fixed on the next version, see #18 , thanks.


A better representation of ISO HTML

Today's style of ISO HTML isn't very pretty or useable:
https://www.iso.org/sites/directives/2017/consolidated/index.xhtml#footnote-003-backlink
Maybe there should be an alternative style, with these features:

TOC in sidebar, highlighting current section
relevant footnote entry appears dynamically when footnote citation in view
anchors named according to the section name

This has now been done.


Overflow file copy for string PK copy node

Implement direct copy of the overflow file from string column chunk to hash index builder

This is closed for now as this is stale now and we plan to rework hash index completely later. Changes here will be considered in later PR.


Clarify content and audience in self service permissions page

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
The audience on the self service permissions page is unclear as it covers both team and enterprise permissions. self service usually means team plans and it's not clear if the content is relevant to team or enterprise users.
we need a clearer definition of the audience:
self service = teams plan OR
self service = cloud user
What part(s) of the page would you like to see updated?
https://docs.getdbt.com/docs/collaborate/manage-access/self-service-permissions#owner-and-member-groups-in-dbt-cloud-enterprise
Additional information
No response

this has come up again here and here

has been made clear now, closing


Automated Package Upgrade

:tada: This PR is included in version 2.2.165 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


character editor

Describe your bug here.
so when i go to use ghost animation the one that is all the way down when i try to ghost it the game just crashes 😢 😢 😢  and im using the newest version idk why
Command Prompt/Terminal logs (if existing)
😢

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no i didnt code anything

show the logs

i think this only happens when you add new animations
so the solution is to add the new animations, save the character, and then re-enter the character editor and resume


[top] Re-organize reset domains

The current reset domain organization, specifically the lc and sys resets, cause a lot of cross domain communication that becomes difficult to manage when ndm_reset is requested.
This leads to potentially a lot of instability in the debug module usage and would become problematic on silicon.
This proposal outlines a different way to handle the reset domains such that it has significantly fewer cross domain signals.  This will then improve the usability of stability of the rv_dm domain.

@matutem
@sriyerg
@cdgori
@alphan
@dmcardle
@a-will

@tjaychen This looks like the issue with the resets I had with prim_pulse_sync which I fixed with new cell I added, ast_pulse_sync.

It's very similar but it isn't exactly the same issue. This issue here specifically relates to ndm triggered resets. I think the one in your situation was a bit different.
We are still addressing this from an instantiator level, just that this one is a complete re-organization of the ndm case.
We actually investigated the path of placing the cross reset directly in the module, but actually identified a lot of drawbacks. So likely in the future we are going to add assertions to force the instantiator of the modules to deal with it.

To elaborate a bit more. What we noticed with modules like these is that even if have the resets affect each other, the upstream / downstream logic is still broken a lot of times.  Ie, even if both sides of the prim_pulse_sync reset, it doesn't really help since the upstream that initiated the request effectively "lost" its request.  So we don't want the cross reset to give the illusion of solving the problem.  So instead we will likely opt for a version that asserts a failure if this situation occurs, and force the instantiator to deal with it.
The second point you made is valid.  The pulse modules do not back-pressure and will break when a new pulse shows up.  I believe we have also added an assertion for this behavior.   I guess I'm not completely clear on why you feel an assert is not sufficient. It is true in synthesized code there will be no ASSERT, but isn't that why we do simulations and have equivalency checks? Or are you concerned about simulations not having enough coverage? Because by this logic none of our assertions are valuable, since they would all disappear after synthesis.

@tjaychen I understand your analysis for the two issues.
In the AST case, I back-pressured the source pulse to solve the second issue while maintaining the ASSERT  in place.
As for the resets, loosing a pulse was less significant the breaking the function of the cell.
Most likely, the reset asserted will take care for re-issueing the lost pulse, but it will be a good idea to issue a warning/add ASSERT to identify this condition.

#15602 has been merged, so this issue can be closed.


Fix typo in CONTRIBUTING.md

Fix a typo.
/sig multicluster

I think this is superseded by #103, but it's small enough and I dislike typos enough that I think I'll merge it regardless.


documentation request: how to make custom components

In the concert demo code you introduce a custom component :
function bounce() {
	let bouncing = false;
	let timer = 0;
	return {
		id: "bounce",
		require: [ "scale" ],
		update() {
			if (bouncing) {
				timer += dt() * 20;
				const w = Math.sin(timer) * 0.1;
				if (w < 0) {
					bouncing = false;
					timer = 0;
				} else {
					this.scale = vec2(1 + w);
				}
			}
		},
		bounce() {
			bouncing = true;
		},
	};
}

Which appears to have a kind of API with things like require, and update() and add()
This is also in other demo examples like the drag example .
Creating these custom off the cuff components seems to be a core feature of Kaboom, yet the way to make them and how they work is not documented anywhere.
Can you please  walk through all the options for creating a custom component?
what is the minimum object to return?
what does update() do ?
what does add() do in a custom component?
what other lifecycle methods are there? destroy?
I'm sure these could be "guessed" at, but this seems like a core functionality that's not documented.

These are documented here and will be in a demo here, and will be surfaced properly on the website next week!

Awesome! I'm glad I'm building a simple game where I'm coming across the need for all of this. The more I use Kaboom the more I like it.

This is so funny! I just stumbled across the need for this, and you already have it all spelled out plus you referenced the demos I found the components in ... guess I will go over the source more thoroughly before filing bugs !

No prob! I know the doc is still a bit rough these days, but it'll get better every week.

There's also a tutorial in this demo


Fix category dropdown of mentor application to match with endpoint update

Purpose

The purpose of this PR is to fix #63
Goals

The category dropdown of mentor application needs to be updated to comply with the backend endpoint changes introduced with this pull request.
Approach

Update the code to use category.uuid instead of category.id.
Screenshots

Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Related PRs

PR #83 on scholarx/backend
Test environment

Learning

Also the documentation needs to be update to have,
{
    "categories": [
        {
            "category": "Data Science",
            "id": "95ba00c0-8173-43c4-9573-6b481b4085e2"
        },
        {
            "category": "Random Category",
            "id": "fe7ca629-4f5e-440a-86cb-d54f8b98ec5f"
        },
        {
            "category": "Computer Science",
            "id": "fef68adb-e710-4d9e-8772-dc4905885088"
        }
    ],
    "message": "All Categories found"
}


GEOMESA-1976 Upgrade to Arrow 0.6

Using .setSafe in mutators instead of reAlloc (which doesn't work)
Fixes offset alignment in arrow output files

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

also needs arrow to be published to maven central


Set Up Frontend Base

Set Up Frontend using React + Tailwind

You should remove node_modules from your commit and add it to the gitignore else it might inflate the repo


Account numbers unique id, Sending emails when loan-applied, approved, paid, closed & celery task for updating Savings Account interest amount adding daily basis

@ashwin31 @shirishagaddi

[x] Account numbers unique id
[x] Sending emails when loan

applied
approved
paid
closed

[x] Celery task for updating Savings Account interest amount adding daily basis

Coverage decreased (-0.5%) to 88.671% when pulling a6e81a4d7db437acaabd165b15b8a04f90eee76b on vidyasagar-r:user_move_branch into 69f8887aba43e90315c3d4dd28c830e5c2596dd2 on MicroPyramid:master.


Correction on Azure client security

Correcting the typo

@ChackDan : Thanks for your contribution to the Azure documentation! The author, @ChackDan, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


background tasks, MySQL gone away

When Combine background tasks attempts to run spark jobs in the background, there is occassionally the problem of:
"django.db.utils.OperationalError: (2006, 'MySQL server has gone away')\n"], 'ename': 'OperationalError'}

This is very similar to problems the front-end GUI had with MySQL, perhaps the fix will be similar as well.

Addressed by maxing out MySQL's wait_timeout.  Can revisit if necessary.


Fix a bug in IE11

Fix a bug that occurs in IE11 when switching from active to inactive state.

Fixes #31


Supporting Power BI Embedded Playground features in Portal Embedded Reports

Is there any short term plans to encorporate the features that can be used for custom embedding of reports within Portals from the Embedded Playground?
The ability to embed a Power BI report in the wave 1 release is excellent, but for my clients I need the ability to amend the settings of the embedded report - such as hiding the Pages, Filter Page, background and Edit features etc.
Is this possible already, as there is litte guidance as to how these could be incorporated without being able to hand-craft web templates - just being able to drop in code snippets would really make this beneficial to 'super citizen developers' such as myself and my clients

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

ID: 15411392-e791-a6f2-990e-e46f07c2c7cd
Version Independent ID: 929a4c75-03be-5125-369d-9a36aae92d09
Content: Set up Power BI integration with your portal - Power Apps
Content Source: powerapps-docs/maker/portals/admin/set-up-power-bi-integration.md
Service: powerapps
GitHub Login: @neerajnandwana-msft
Microsoft Alias: nenandw

Thank you for reaching out. Please submit an idea in the Power Platform Community - Ideas forum for a feature ask; and the idea can be voted for this request.


Websocket error 400 again

@howanghk
Hi all, since yesterday im getting the error 400 again...
I updated the ewe link max plugin, but still the same, this is the error:
[2019-12-24 13:15:03] [eWeLink] WebSocket messge received: {"error":400,"reason":"Bad Request","apikey":"fc37d794-8eff-420f-b91f-55ddef613649","sequence":"1577164503449","actionName":"userOnline"}

Duplicate of #67


Glyph icons don't work in the customized version.

When using a customized version of Bootstrap 3.0 the Glyph icons don't work. The font file sizes differ from the "normal" version.

glyphicon-icon not working on codeigniter framework. Normaly work fine but when i convert into codeigniter framework its glyphicon not work.other things work fine.

I can confirm that - not working on Codeigniter framework.

Not working with DataTables download builder either. I'm trying a concatenated CDN link and the glyphicons font files 404 no matter how I try to build the download.


Update documentation for axmol build commands when targetting the iOS simulator

Describe your changes
Update Dev Setup info to show correct commands for generating iOS simulator projects
Issue ticket number and link
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[ ] I have performed a self-review of my code.
Optional:

[ ] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

@halx99 Just to verify, does the x64 simulator build also require the -sdk simulator? For instance: axmol build -p ios -a x64 -sdk simulator -c

@halx99 Just to verify, does the x64 simulator build also require the -sdk simulator? For instance: axmol build -p ios -a x64 -sdk simulator -c

x64 ios will be auto traits as simulator


passing objects to params command

It's possible to define a param as an object, for example:
ingress:
  publicHosts:
  - domain: foo.example.com
    paths:
    - path: /foo
      service: foo.ns.svc
      port: 80
    - path: /bar
      service: bar.ns.svc
      port: 90
  - domain: bar.example.com
    paths:
    - path: /foo
      service: foo.ns.svc
      port: 80
    - path: /bar
      service: bar.ns.svc
      port: 90

Is it possible to override these params using the -p option?

Unfortunately, the path implementation to address fields doesn't support arrays right now. You would like to type -p ingress.publicHosts[0].paths[2].port 8080 but can't.
Good catch, added as feature request!


Support multiple Terraform versions

Different projects might require different versions of Terraform
Use case at Wise as shared by Pavel

We use tfenv to manage various terraform versions when working locally,  Which means we define .terraform-version files in our projects to define the various versions.  Would be very nice if it could be supported.

I would like to work on this issue, please assign it to me .
Thank you.


feat: functions and literals in text format

Adds support for functions
Adds support for most literal types (including Decimal)
Quality improvements

Adds a new matcher that allows only symbols of a particular type to be matched
Added PROTO_LOCATION macro to make specifying locations more convenient.
Upgraded the converter tool to accept globs (only useful for testing as shells already expand globs).
Many error messages are now returned to the ParseResult instead of being left in text for a human to detect.

The two major review items not addressed by the updates are the changes to the field references (implemented in a PR that's waiting for #33, #34, and #35 to merge first) and the format of Decimal (waiting for at least #33 but given that it's going to depend on a breaking change to substrait should be addressed separately after this repo has been updated to the latest substrait version).
So this should be ready for rereview.

LGTM


Mossy Cobble Vertical Slabs + Biome Moss mod rendering transparent

Vertical Mossy Cobble Slabs are rendering the stone invisible with moss visible on top. I think this is due to the Biome Moss mod, as it adds a layer of moss with a color map on top.

Please test this again and see if it's still an issue.


(re)Starting K8s cluster with lab charts resets Jupyter Token

Experimenting with the new approach to starting Jupyter in the development version of the Egeria-lab charts I found that each time the k8s cluster restarts, a new token is generated (and would thus have to be conveyed to users). This may be inconvenient for demo and non-developer usage. Suggest further investigation and discussion on Jupyter Server authentication approaches?

The Jupyter token is generated at startup if it does not exist.
When the password is generated Jupyter adds it into a file in the ~/.jupyter directory called jupyter_notebook_config.py
This directory is not persisted in the current deployment, so the next time Jupyter starts the same thing happens.
Possible fixes include

Making this directory (any others?) persistent, just as we do for ~/work (which holds the notebooks)
Allowing the password to be explicitly set (defaults to existing behaviour if not)

Ideally both probably make sense.
There are also other authentication related settings, and general jupyer settings, so possibly an option to specify other overriding values could be useful, though more complex as they'd need to be merged in.

One additional thought to consider is that Jupyter allows a user defined password to be created using the token - perhaps we should do that to make it easier for users to login?

Yes - that's what I'm suggesting in the previous entry (second bullet) :-)
Allowing the installer to specify a password is fine. I would not create a default password - so having an auto-generated token and persisting should be default.


R6 counter json list to big to manage

Issue
Vertical Growth Issue with R6 counters (1500+ lines of code)
Reason
With the popularity of R6 Siege, it seems that this game will have many seasons and new characters added. If all R6 character's nodes are in 1 file, it will be harder to manage and track which characters connect.
Solution
I would like to propose a rework on how the nodes are created/managed, making future and current characters easier to work with.
By refactoring how the counters are managed/written, it will be easier to add/update/remove nodes

I like the idea behind this. Something that is difficult about working with this file is that it is hard to follow which IDs match to which operator.
Do you have any thoughts on how to improve this file? Describing relationships using operator names and breaking down the file would be a massive improvement I think.

Just the top of my head, I would start with how layout of files/folders.
First level folders would hold be labeled with seasons (s1, s2,etc) , then second layer would have folders named after the teams the game characters belong to (eg Spetznaz, GIGN, etc), then the third layer would be .js files that will be named after an individual character (eg glaz.js, rook.js).

I created a quick, non working repo to show off my thoughts: https://github.com/Mr-Martinez/r6-counters-refactored .
Basically just refactoring json code to its own single file/operator.
Then a module that imports each individual .js file/object to create one big json object that the html file can use to iterate its graph.
We can essentially do the same for images as well.
Basically each file will be responsible for handling data for a single operator.
A global enum for making sure that you don't have to memorize node numbers, opting to use names instead. Easier to see what's going on inside files

@marcopixel sent me this library that they've been working on: https://github.com/marcopixel/r6operators that contains all the operator icons + operator bio data. Would it be worth using this library in some way to retrieve the bio data for each operator (e.g. attacker/defender, real name etc.) and then in r6-operator-counters add the counter data?
I have a question about the method "getHardCounter();", would this return who that operator hard counters, who that operator is hard countered by, or both?

@marcopixel, this is what I had in mind indeed!

Would it be worth it using this library...

Yes! Although it really depends... Will @marcopixel update the file with every season of the year and/or allow anyone to help with the project? (I'd like to help out when I'm able to)
Reading https://github.com/marcopixel/r6operators , the heart of the code is 'operators.json' file, and nothing there looks like they rely on each other. Meaning it would be easy for anyone to help out on the project as well if there are times where @marcopixel is not able.

I have a question about the method "getHardCounter();", would this return who that operator hard counters, who that operator is hard countered by, or both?

In this case the operators are returning who they are hard counter by.
Although a change in naming convention is needed avoid confusion.
Any ideas?
So far I'm thinking...
"get" vs"retrieve" vs other synonyms.
"HardCounteredby()" vs "HardCounteredFrom()"
Although, with each file having the responsibility of taking care of only one operator, I can as well add code that shows who the operator hard counter as well if you want to add that as well.
I'm coming from a typescript background, but in my pseudo-code I'm trying to return back one big Class Object with functions that can handle all sorts of goodies for a single operator. In this cases the three options you can pick on the website,
I wanted feedback on this style in addition to the folder structure. Is there a benefit of 3 layers deep, or two layers deep when using gulp? I have to read the Gulp API documentation properly, but to my understanding gulp can grab data from a single file, then add it to a completely new file essentially combining all data? So in my mind I'm thinking gulp can combine outputs of all files in lets say the GIGN folder making a GIGN json file.
I basically want to make it easier for future use, lets say someone wants to make a filter list for organizations operators belong to.

I'm intending to support it as long i've got interrest into the game and i'm also open to add contributors to the repo :)

@Mr-Martinez Potentially GetCountersTo() and GetCountersFrom()? Unsure. Maybe these methods could take in an array of enums of what levels of counter to return?
Folder structure wise, would it make sense to divide operators up by Year then Season? As an operator may come out in Year 5 that is apart of an organisation already previously used. I think as long as we get the main benefit of the data being split up. Even splitting up each operator into their own file before combining would be of large benefit.

Oh that's nice!
I'll get on it now.

That's also how i've intended it on r6operators when i'll decide to include additional assets like front shots & gadgets. One main folder called operators and subfolders for each operator, each of them including the JSON metadata and the assets.
Seems like the most reasonable & clean approach for this problem in my opinion.


Incomplete change of session ID after Tomcat failover (with jvmRoute + securityConstraint/Valve)

From<EMAIL_ADDRESS>on March 23, 2011 14:56:36
Observed for the TC 7 branch, but likely not restricted to that.
Directly after a Tomcat failover the first request R1B on the new node B triggers changing the session ID (e.g. in cookie) but still operates on the old session. After that request there are two sessions, one with the old ID, one with the new one (see the session list in the Tomcat manager webapp).
The next request R2B - still send to the failover Tomcat node B - now uses the "new" session, which has the same attributes as the old one, except that the values are the ones from the time the id was changed, i.e. when R1B started. Any changes done by R1B are lost.
Correct handling of ID changes can be studied e.g. in org.apache.catalina.ha.session.JvmRouteBinderValve. In method SessionTrackerValve.changeRequestedSessionId() in addition to request.changeSessionId( newSessionId ) there should be called something like:
Session catalinaSession = request.getSessionInternal(false);
catalinaSession.setId(newSessionId);
At least this fixes the dublicate sessions after request R1B and the missing attribute updates.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=92

From<EMAIL_ADDRESS>on March 23, 2011 07:00:18
Simple "session.jsp" JSP to reproduce
<% String attributeName = "COUNTER"; %>
RSID:   <%= request.getRequestedSessionId() %>
Valid:  <%= request.isRequestedSessionIdValid() %>
SID:    <%= request.getSession().getId() %>
Primary:<%= request.getAttribute("isPrimary") %>
Value before increment: <%= session.getAttribute(attributeName) %>
<%Object o = session.getAttribute(attributeName);
if (o==null) {
session.setAttribute(attributeName,1);
} else {
String s=o.toString();
session.setAttribute(attributeName,Integer.parseInt(s)+1);
}
%>
Value after increment: <%= session.getAttribute(attributeName) %>
Reuse Session

From martin.grotzke on March 23, 2011 13:14:38
Tomcat failover is handled in MemcachedBackupSessionManager.changeSessionIdOnTomcatFailover and handleSessionTakeOver, this also sets the newSessionId on the session.
I can imagine that it's directly related to  issue 93 , as it does not check if the session perhaps has already been loaded and removes it in this case.
Summary: Incomplete change of session ID after Tomcat failover (with jvmRoute set)
Status: Accepted
Owner: martin.grotzke

From<EMAIL_ADDRESS>on March 24, 2011 02:00:31
Hmmm, the TC cluster valve uses session.setId(), your manager uses session.setIdInternal. The former removes the session from the manager, then changes the id and finally adds it again to the manager. At the end it calls tellNew(). You internal method setIdInternal() only sets the id.
I removed my changes to the SessionTrackerValve and replaced session.setIdInternal() in MemcachedBackupSessionManager.handleSessionTakeOver(). by sesion.setId(). Surprise: it does not work there. Using the testing JSP after failover I still get the same behaviour.
Note that when using the above JSP the first request on the failover node gets for request.getRequestedSessionId() the new id and for request.getSession().getId() the old id. Both sessions are still in the manager and the attribute update actually happen to the old session.
Regards,
Rainer

From martin.grotzke on March 24, 2011 07:01:07
That setId is not used is intentional: as manager.remove(Session) would remove the session also from memcached if it's not handled otherwise.
What should be done is to replace session.setIdInternal by session.setIdForRelocate, which does all the "magic". This might already do the trick.

From martin.grotzke on March 24, 2011 07:01:43
Btw, if you want to provide patches you can also use github + fork + pull-requests.

From<EMAIL_ADDRESS>on March 24, 2011 08:06:19
Unfortunately replacing session.setIdInternal( newSessionId ) by         session.setIdForRelocate( newSessionId ) inside MemcachedBackupSessionManager.handleSessionTakeOver() doesn't work for me. Still the same behaviour. Can you reproduce it?
Concerning git: yeah, just trying to get used to it. Once patches get bigger than just a line or two ...
Regards,
Rainer

From martin.grotzke on March 24, 2011 11:50:05
Yes, I can reproduce it when I add a security-constraint to my web.xml (which adds the additional valve) and set a jvmRoute on the engine.
There's a quick fix possible (in changeSessionIdOnTomcatFailover before loading from memcached we must check the local sessions map, in handleSessionTakeOver there must be added s.th. like sessions.remove( origSessionId )). However this would break HttpSessionActivationListeners as they'd already got bound in initial findSession, therefore the findSession and changeSessionIdOnTomcatFailover (including Set-Cookie) must be brought together.
I hope I find time to fix this the next days.
Summary: Incomplete change of session ID after Tomcat failover (with jvmRoute + securityConstraint/Valve)

From martin.grotzke on March 25, 2011 16:23:41
Fixed in master and tomcat7 branch.
Status: Fixed
Labels: Milestone-1.4.1

From<EMAIL_ADDRESS>on March 31, 2011 09:04:13
Fix validated on my TC 7 installation.
Thanks!


[Pluggin: google_maps_flutter] How to set maps language

Hi @fauzipadlaw
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself, if you disagree please write in the comments and I will reopen it
Thank you


Css not running

Hi!
I've some problems with my css, i've the same than your demo but i've this on my app:

If you've a idea for fix this?
Thank's!

Not sure what could be causing this. Could you provide a codepen demo?

Closing this since there has been no reply.


laravel 5.1 issue

I have read about https://github.com/nWidart/laravel-modules/issues/316
and I've tried to bring it down to 0.1 but
composer update result:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for nwidart/laravel-modules ^0.1 -> satisfiable by nwidart/laravel-modules[0.1].
    - nwidart/laravel-modules 0.1 requires laravel/framework 5.2.* -> satisfiable by laravel/framework[5.2.41, 5.2.x-dev, v5.2.0, v5.2.0-beta1, v5.2.1, v5.2.10, v5.2.11, v5.2.12, v5.2.13, v5.2.14, v5.2.15, v5.2.16, v5.2.17, v5.2.18, v5.2.19, v5.2.2, v5.2.20, v5.2.21, v5.2.22, v5.2.23, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.29, v5.2.3, v5.2.30, v5.2.31, v5.2.32, v5.2.33, v5.2.34, v5.2.35, v5.2.36, v5.2.37, v5.2.38, v5.2.39, v5.2.4, v5.2.40, v5.2.42, v5.2.43, v5.2.44, v5.2.45, v5.2.5, v5.2.6, v5.2.7, v5.2.8, v5.2.9] but these conflict with your requirements or minimum-stability.

Hi,
That probably means 5.1 isn't supported.

thx for fast clarification sir.. :+1:

Indeed you can use pingpong-labs/modules on laravel 5.1. the pingpong version doesn't a lot of optimizations I made here, but it'll work.

I'll close this as there hasn't been a reply in a while, feel free to comment if needed.


[Q] Merge two VBR files with related PCRs

Hi @lelegard,
I'm trying to merge two files which are related by PCR. The main file (AV) isn't CBR so I'm adding stuffing with --add-input-stuffing and then I'm using merge with --no-pcr-restamp  to merge the second file.
Merge is inserting the packets as soon as there are NULL pkts available without regarding to the stream PCR. This means that if I add-input-stuffing in the rate 1/1 the merged files gets merged into the beginning to the output file but if I reduce that value to 1/60 there's a more even distribution along the file but the streams are not in sync.
I was trying to avoid using regulate (by PCR).... since I'm dealing with files.
What is the recommended strategy to merge to VBR files which streams are related by PCR?
Thank you

Just sharing my current findings:

merge is for live, mux is for files (what I'm using to test)
tsstuff adds NULL pkts to target file
interleave option of file input exists!

Using the above concepts I ended with two different non-working results:

Interleaving the files to mux using file input option --interleave. Used tsstuff to increase bitrate of all input files to the same value.
For a 10 minute file this results in a playable file but I'm guessing that for a longer file the PTSs of the inserted stream will diverge from PCRs.

Zoom from start if the resulting file:

Zoom at 10 minutes:

Muxing the second file in AV stream with mux plugin: Only AV Stream had stuffing added with tsstuff. The secondary file had PAT, PMT and NULL removed to avoid mux conflicts. Bitrate of the secondary file was calculated with tsanalyse and used as mux --bitrate option.

After a while PTSs start diverging from PCRs:

Conclusions:
File interleave might no be very useful in this scenario because it doesn't look into contents only to N-packets chunks.
Mux could be a viable solution if insertion was delayed/accelerated until the right PCR appears in the main stream. Right now when a packet is read (even if it's a NULL pkt) it's inserted in the next available NULL pkt of the main stream (if it min_pts, max_pts, inter_time are not specified): https://github.com/tsduck/tsduck/blob/v3.28-2551/src/tsplugins/tsplugin_mux.cpp#L377.
Maybe mux could support a concept like: mux a stream from another file (read PCRs and insert them aligned into current packet flow).
Just my 2 cents.

Hi @lyncher0 ,
Here my experience with the muxing functionality of the TSDuck:

Don't try to mux VBR streams. It's a pain! In fact, a lot of professional devices and tools overcome this. They convert previously any VBR streams to CBR and then they do the mux.
I'm muxing without troubles using files. The trick is to convert all input streams to CBR with idential bitrate, then "mux" them with interleaving, and finally reduce the output stream to the target bitrate. The result then is nearly perfect after you reprocess the PCR values.

I hope that in the near future @lelegard will want to implement this technique for realtime processing.
Regards.


Introduce relations & injectables (0.13)

A beta branch was published, if you are interested in testing
npm i nekdis@beta


Allow CSV export to go ahead when there are data issues with the members API

Changes
Because of data inconsistencies between GIAS and the Parliamentary Members API,
we have had issues getting the CSV export to work properly when a school's
parliamentary constituency is unclear (e.g. "Middlesbrough" is not a single
constituency, but GIAS insists some schools are in "Middlesbrough")
To get round this, if we query the Members API for a constituency but get
multiple results back, still allow the CSV to be downloaded but with empty
values for the MP details. The CSV requester will then have to fill in the
blanks themselves.
We also send a warning to Sentry to let us know this has happened. This way we
can track which constituencies are returning multiple results.
The solution for this is a little complex in the CSV exporter, due to the way
the Members details objects are set up, but this should capture everything.
Checklist

[ ] Attach this pull request to the appropriate card in Trello.
[x] Update the CHANGELOG.md if needed.

 When the members API returns multiple constienies in the search we should make it into a nil


Add Slider component

@0ortmann - I'm pulling this back to a draft to see if I can rewrite the raw js function to Reason to keep things clean.


add ITPro Release Blog

Signed-off-by: Bill Mulligan<EMAIL_ADDRESS>

Checks failing, @xmulligan

@liquidat the checks are still failing, but the deploy preview works now. I guess it maybe doesn't run the tests again? https://isovalent.slack.com/archives/C027PHUB0S0/p1658818899412429


Update use-dkim-to-validate-outbound-email.md

Update DKIM config to account for possible missing selector.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@MSFTTracyP - PR has been copy edited and is ready for final review.
Thank you.
cc: @kenwith


[WSL2] running multiple processes at once results in extreme slowdown of some (filesystem?) operations and htop reporting impossible CPU usage

Windows Build Number
Microsoft Windows [Version 10.0.19041.928]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version 4.19.128-microsoft-standard (oe-user@oe-host) (gcc version 8.2.0 (GCC)) #1 SMP Tue Jun 23 12:58:10 UTC 2020
Distro Version
Ubuntu 20.04
Other Software
GNU bash, version 5.0.17(1)-release (x86_64-pc-linux-gnu)
htop 2.2.0 - (C) 2004-2019 Hisham Muhammad
git version 2.25.1
Repro Steps

Create an executable file spinp.sh with the following contents

#!/usr/bin/env bash

cat > spin.sh <<'EOF'
#!/usr/bin/env bash
times=`seq 1 10000000`
function large(){
    for i in ${times}; do
        echo -n "stuff"
    done
}
for i in 1; do
    large >/dev/null
done
EOF

chmod +x spin.sh

J="$1"
PIDS=()

function finish {
    for i in "${PIDS[@]}"; do
        kill -9 "$i" || :
    done
}

trap finish SIGINT SIGTERM SIGKILL EXIT

for i in $(seq 1 $J); do
    ./spin.sh &
    PIDS+=("$!")
done

echo "${PIDS[@]}"

for i in "${PIDS[@]}"; do
    wait "$i"
done

Open a couple of WSL2 terminal tabs.  Start htop in one of them.  Run while true; do time for i in $(seq 1 100); do touch foo; done; sleep 1; done in another tab.  In another run ./spinp.sh 4 replacing 4 with however many physical cores you have.
Inspect htop and the tab running while true; do time for i in $(seq 1 100); do touch foo; done; sleep 1; done

Expected Behavior
Since I have 16 GB of RAM (running 10th Generation Intel® Core™ i7-10510U Processor (1.80 GHz, up to 4.90 GHz with Turbo Boost, 4 Cores, 8 Threads, 8 MB Cache); currently seems to be running at 2.30 GHz), and each of these processes allocates only about 1GB and is a serial program, and I'm only running half as many processes as I have hyperthreads, I expect to be able to continue interacting with WSL2, to be able to do other operations such as file editing and git diff and run other processes.
Actual Behavior
However, after a couple of seconds, I see individual bash ./spin.sh processes taking 800% CPU in htop, the time ... touch foo ... slows down by about 10x--100x, and interaction is generally laggy.

Qualitatively it feels like anything that touches the filesystem slows down way way more than things that don't (for example, scrolling up and down in git diff can be responsive, but when I type q, it takes a long time before the display goes away, and even longer before the terminal prompt is displayed again), but killing processes, for example, and interacting with htop also seems slow.
For reference, spinp.sh is not running, the touch foo loop gives timing numbers such as
real    0m0.082s
user    0m0.062s
sys     0m0.030s

or
real    0m0.068s
user    0m0.053s
sys     0m0.024s

Even with just two threads running (./spinp.sh 2), the timing slows down to
real    0m1.385s
user    0m0.091s
sys     0m1.309s

or
real    0m2.833s
user    0m0.096s
sys     0m2.753s

and with ./spinp.sh 8 I get numbers like
real    0m7.320s
user    0m0.103s
sys     0m7.120s

and
real    0m10.180s
user    0m3.071s
sys     0m5.332s

Here's a gif showing the behavior on my machine:

Diagnostic Logs
No response

I have tried it but after few seconds spinp.sh is exiting.
souradeep@DESKTOP-GQJDF6F:~$ ./spinp.sh 4
72 73 74 75
./spinp.sh: line 23: kill: (72) - No such process
./spinp.sh: line 23: kill: (73) - No such process
./spinp.sh: line 23: kill: (74) - No such process
./spinp.sh: line 23: kill: (75) - No such process

Hmmm.  I have noticed that sometimes WSL2 ends up being very laggy even when I'm not doing much, and then quitting and restarting it fixes the issue (temporarily).  I'll have to see whether or not killing the VM and restarting it also temporarily fixes this issue.
I can give more involved instructions about how it ends up showing up in my actual workflow.
Another possibility: if you start more processes than you have hyper threads, does this change the behavior?
(I'll also note that this issue is painful enough that it's making me reconsider whether or not WSL2 is an adequate Linux box for my needs, or if I need to go actually install Linux.)

Try a .wslconfig with memory=8G, which simulates 20175 behavior. Or maybe even 6G.
Looks like:

You hit 99% (Windows) memory consumed in your Task Manager screengrab at one point.

You hit 99% (Windows) memory consumed in your Task Manager screengrab at one point.

So is the issue here that if Windows has less memory free than WSL thinks and WSL consumes all of it, then the CPU usage shown by ps/htop starts going wild?  (This still seems like a bug, though perhaps a less serious one.)

The htop output is linux behavior.

The htop output is linux behavior.

It's not.  The sum of the percentages is more than 3x the number of threads (it's not possible to be utilizing 2770% of a single CPU when I only have 8 virtual cores).  Moreover, these are single-threaded processes which should not be distributed across multiple CPUs.  Furthermore, this behavior is apparently only triggered when the parent OS runs out of RAM?  This doesn't seem like typical htop/linux behavior to me.


handle wasm_opt_main return value

Right now it's ignored but it needs to be returned from rust main.

done


Upgrade to latest dependencies and fix compile issues (#3148)

Upgrade to latest dependencies

Fix compile issue

Update patch

Update dependencies again

Update deps to get latest reconciler-test updates

Fix DLS e2e tests

OCF Webhook is merging this PR


Drag and drop to open files when no file is open

Describe Your Suggestion
Currently drag and drop is only reserved for adding files to a VPK, but it would be nice to open VPKs via drag and drop if one isn't open.

Perhaps it could ask the user what they'd like to do in this scenario? There's a valid reason why someone might want to add a pack file to another, for example adding a BSP to a VPK


Is it not for Windows ?

Is this not available for Windows User currently ?
I would love to explore this

You could probably just use wsl to get lolcat, I've done it before.

No need for WSL. Just install Ruby from https://rubyinstaller.org and run gem install lolcat.

Can we compile this to work on any windows machine

Ruby is an interpreted language, so you'll always need the Ruby interpretor to run lolcat. However, you can use a bundler like ocra to make a standalone self-extracting executable for lolcat.

No need for WSL. Just install Ruby from https://rubyinstaller.org and run gem install lolcat.

Its better on Linux and macos because thats what its made for- theres a reason why it doesn't have a windows install section in the readme


Global is Not defined Angular 7

After adding in the fix #9  compiling in angular 7 is fine but then I run into this error. Please test if this is compatible with Angular 7
js2excel.min.js:1 Uncaught ReferenceError: global is not defined
at js2excel.min.js:1
at push../node_modules/js2excel/dist/js2excel.min.js.Zd (js2excel.min.js:1)
at Object../node_modules/js2excel/dist/js2excel.min.js (js2excel.min.js:1)
at webpack_require (bootstrap:78)
at Module../src/app/download/download.component.ts (main.js:264)
at webpack_require (bootstrap:78)
at Module../src/app/app-routing.module.ts (main.js:74)
at webpack_require (bootstrap:78)
at Module../src/app/app.module.ts (app.component.ts:12)
at webpack_require (bootstrap:78)

Also running into this, unfortunately :(

Hello @emilionavarro @Bmastaz  add this to your polyfill.ts file in your angular project

(window as any).global = window;


release: release-notes: Release notes should not list closed stale issues as "addressed"

Spotted this completely by chance: the "Addressed Issues" list of issues in the release notes includes all closed issue in the release time frame, no matter why they were closed. This includes issues that have been closed as being stale, and hence not acted upon in any way. These issues may or may not have been "invalid"/"not a bug", but since no human got involved in making this decision, I don't think they should be considered "Addressed".
From Release notes 3.3, see e.g. https://github.com/zephyrproject-rtos/zephyr/issues/51873 that got as one of the many issues listed as addressed
Relevant section of list_issues.py
https://github.com/zephyrproject-rtos/zephyr/blob/7039f8ce9c2787f37f5a97c3aca1e1541a0d7d07/scripts/release/list_issues.py#L108

To fix this, two things need to be done:

Make the "stale bot" close issues as "not planned" instead of "completed."
Make list_issues.py filter by reason:completed.

Make the "stale bot" close issues as "not planned" instead of "completed."

That was (unintentionally) done by https://github.com/zephyrproject-rtos/zephyr/commit/f03190a2e1ce6f442f0b956de51229851846e3c8, because actions/stale@v8 closes stale issues as not_planned.
So only the list_issues.py needs to be updated.

Make the "stale bot" close issues as "not planned" instead of "completed."

That was (unintentionally) done by https://github.com/zephyrproject-rtos/zephyr/commit/f03190a2e1ce6f442f0b956de51229851846e3c8, because actions/stale@v8 closes stale issues as not_planned.

Neat :)

So only the list_issues.py needs to be updated.

Yes and no: the status of issues that where improperly marked as completed pre https://github.com/zephyrproject-rtos/zephyr/commit/f03190a2e1ce6f442f0b956de51229851846e3c8 would still need to be fixed (or worst case scenario treated as a special case in list_issues.py)


Set CurrentPointerTarget properly when updating PointerData

Overview
This change addresses the bug described in #3216. The method used for updating PointerData with canvas raycast information was missing a line to assign the current pointer target. I fixed this with a slight refactor that should make errors like this harder to make by reducing duplicate code and properties on the object.
Changes

Fixes: #3216

Always forget about those shorthand properties, good idea

Haha it's much cleaner. Looking forward to Unity 2019 with the c# 7 features


Add default export in type definitions to support ES6 import syntax.

By changing "export = ExpressValidator" to "export default ExpressValidator", the module can be imported using ES6 syntax:
Instead of forcing users to use import expressValidator = require('express-validator');, they can now use import expressValidator from 'express-validator';.
I ran into this issue initially when trying to move to all ES6 syntax. It also led me to https://github.com/DefinitelyTyped/DefinitelyTyped/issues/10742 which seems to suggest that ES6 syntax isn't supported.

Coverage remained the same at 96.498% when pulling fa59eac477d3cc5dc9a7dbd4541a2457b7780c4d on btraut:master into 8e8f635ffdb32862fd8e076a4103c9fef29e651b on ctavan:master.

Coverage remained the same at 96.498% when pulling fa59eac477d3cc5dc9a7dbd4541a2457b7780c4d on btraut:master into 8e8f635ffdb32862fd8e076a4103c9fef29e651b on ctavan:master.

@IOAyman, thoughts?

It doesn't fix it for me.

I think this PR is now superseded since #352 landed.


VSTS vnext build issue if the project name contains []

The build will be failed if the project name contains [] (Visual studio build task). Unable to process command '##vso[task.logdetail id=2ba6c5c9-62aa-40e1-8233-1ec87e7b3e04;parentid=7a11fcfc-16f0-4f49-b8e5-25f368be30a9;name=ClassLibrary2 [test].sln;type=Build;starttime=2016-11-30T08:01:14.8159738Z;state=InProgress;]' successfully. Please reference documentaion (http://go.microsoft.com/fwlink/?LinkId=817296)

turning off record timeline details may workaround the issue

@ericsciple is correct.   Other obvious workaround is to not put brackets in your sln name.  low pri fix.

Hi.
I have the same issue, I turned off Record Project Details flag, but the build failed.
2017-07-14T14:52:29.4485972Z ##[error]Unable to process command '##vso[task.logdetail id=b0253836-ade4-409c-835b-806479abafca;parentid=;name=Pactual.MarketData.Portal[4] - Presentation\Controller\MainHostController.cs;type=Build;starttime=2017-07-14T14:52:29.4485972Z;state=InProgress;]' successfully. Please reference documentation (http://go.microsoft.com/fwlink/?LinkId=817296)
2017-07-14T14:52:29.4485972Z ##[error]Type is required for this new timeline record.
2017-07-14T14:52:29.5423507Z ##[error]Unable to process command '##vso[task.logdetail id=b0253836-ade4-409c-835b-806479abafca;parentid=;type=Build;result=Failed;finishtime=2017-07-14T14:52:29.4485972Z;progress=100;state=Completed;parentid=;name=;]' successfully. Please reference documentation (http://go.microsoft.com/fwlink/?LinkId=817296)
2017-07-14T14:52:29.5423507Z ##[error]Name is required for this new timeline record.


Torchscript access from Libtorch.

❓ Questions & Help
I saw recently that pytorch geometric now supports torchscript (at least a bit)! :tada:
This is exciting to me since this was the main reason why I hadn't used it about a year ago.
My custom graph neural net code is very slow right now and I'm hoping to use pytorch geometric to make a better implementation.
I'm currently able to use torchscript to export my old model for importing in libtorch from C++, but I'm wondering how this could work with things like pytorch geometric Data objects...Although I saw elsewhere that pytorch geometric is "python only", does this mean that torchscripted pytorch geometric models can't be used from libtorch in C++? What if I committed to have the jitted models only take in standard libtorch types?
Thanks,
Jack

Currently, Data is not supported by TorchScript and we wait for better custom class TorchScript support to be able to convert it. However, besides data handling, you can now convert your PyG models to TorchScript (and therefore import it in libtorch). However, instead of passing the data method to the model's forward call, you need to pass in the data attributes explicitly, e.g.:
def forward(self, edge_index: Tensor, x: Tensor):
     ...

import torch
import torch.nn as nn
import torch.nn.functional as F
import torch_geometric.nn as pyg_nn
class GNN(nn.Module):
def init(self, input_dim, hidden_dim, output_dim, train=True):
super(GNN, self).init()
self.convs = nn.ModuleList()
self.convs.append(self.build_conv_model(input_dim, hidden_dim))
self.lns = nn.ModuleList()
self.lns.append(nn.LayerNorm(hidden_dim))
self.lns.append(nn.LayerNorm(hidden_dim))
    for l in range(2):
        self.convs.append(self.build_conv_model(hidden_dim, hidden_dim))

    #post message-passing
    self.post_mp = nn.Sequential(nn.Linear(hidden_dim, hidden_dim), nn.Dropout(0.25), nn.Linear(hidden_dim, output_dim))
    self.dropout = 0.25
    self.num_layers = 3
    self.train = train

def build_conv_model(self, input_dim, hidden_dim):
    return pyg_nn.GINConv(nn.Sequential(nn.Linear(input_dim,hidden_dim), nn.ReLU(), nn.Linear(hidden_dim, hidden_dim)))

def forward(self, x, edge_index, batch):

    x = self.convs[0](x, edge_index)
    emb = x
    x = F.relu(x)
    x = F.dropout(x, p=self.dropout, training=self.train)
    x = self.lns[0](x)

    x = self.convs[1](x, edge_index)
    emb = x
    x = F.relu(x)
    x = F.dropout(x, p=self.dropout, training=self.train)
    x = self.lns[1](x)

    x = self.convs[2](x, edge_index)
    emb = x
    x = F.relu(x)
    x = F.dropout(x, p=self.dropout, training=self.train)

    x = pyg_nn.global_mean_pool(x, batch)
    higher_emb = x
    x = self.post_mp(x)

    return F.log_softmax(x, dim=1)

def loss(self, pred, label):
    return F.nll_loss(pred, label)

model -> trained model
model = GNN(num_node_features, hidden_dim, num_classes, train=False)
model.eval = True
with torch.jit.optimized_execution(True):
traced_script_module = torch.jit.trace(model, (x, edge_index, batch))
I am getting the below error.Will appreciate any inputs here.
TracerWarning: Output nr 1. of the traced function does not match the corresponding output of the Python function. Detailed error:
With rtol=1e-05 and atol=1e-05, found 2 element(s) (out of 2) whose difference(s) exceeded the margin of error (including
0 nan comparisons). The greatest difference was 0.5916104316711426 (-6.0773491859436035 vs. -5.485738754272461), which occurred at index (0, 0).
_module_class,

Looks to be a problem with dropout in conjunction with jit.trace. In particular, you are using PyTorch eval mode wrong. You need to remove the self.train attribute from the model, and pass training=self.training to the dropout layers. Then, you need to set your model into eval mode via model.eval().


Cannot download. stuck at Building COurse Data"

Hello I am trying to downoad a course from udemy.
This get stuck at "building Course Data".
Please advise.

SAME

Can you please elaborate a little more?
Which version are you using?
Which course? Do you have a log? What settings are you using?

Also experiencing the same, using the latest version.

Any update on the fix?

This is a really hard problem to reproduce.
Does it happen with all courses or just some?

Any update on the fix?

investigating

Uncaught (in promise) TypeError: Cannot read property 'status' of undefined
at app.js:615
(anonymous) @ app.js:615
Promise.catch (async)
getLecture @ app.js:614
(anonymous) @ app.js:620
each @ D:\Github\udemy-downloader-gui\node_modules\jquery\dist\jquery.js:385
(anonymous) @ app.js:402
Promise.then (async)
downloadButtonClick @ app.js:370
(anonymous) @ app.js:330
dispatch @ D:\Github\udemy-downloader-gui\node_modules\jquery\dist\jquery.js:5494
elemData.handle @ D:\Github\udemy-downloader-gui\node_modules\jquery\dist\jquery.js:5298
Don't know if this well help or not? Also it doesn't see any of the videos I'm trying to download as DRM protected and It can see all of them

What the course?

Same here. Latest version, latest macOS! So far unable to download any of the courses in my library.

Guys, just to make it clear, the inability to download by DRM Protection is not an error, in these cases the videos will not be downloaded at all.
As for the problem with loading data getting stuck, I need some course names to help with the investigation, this is a difficult problem to reproduce, most of the time I just need to restart the tool, testing with courses where the problem occurs frequently may I hope I find the cause.

I encounter a similar issue when tried to download this course https://www.udemy.com/course/developer-to-architect/learn/lecture/24350002#overview.
Does this tool depend on the type of Udemy account (personal plan, business account, etc.)?

failing to download the following :
https://www.udemy.com/course/100-days-of-code/
Cannot download ..  This get stuck at "building Course Data".

I encounter a similar issue when tried to download this course https://www.udemy.com/course/developer-to-architect/learn/lecture/24350002#overview. Does this tool depend on the type of Udemy account (personal plan, business account, etc.)?

yes, you need an active account and be enrolled in the desired course

The videos in this course cannot be downloaded due to DRM protection, but I had no problems starting the download.
I will continue looking for the cause of the personal problem.

Among others I tried with this free one.
https://www.udemy.com/share/101to23@RYa_lRaYafAPG9Y1KxR19wFy0xl_uESQWWDTDcgeDf0ssPH56-v4CAlflVGK3ZuWOQ==/
Stuck in building. Hope that helps.

I have the same problem. It stays in the "Building data" state and never downloads the course, I want to download courses from a Business account. I am using version 1.12.4

I have the same error, and not one course out of 70 was not downloaded. After I changed the language and pressed the save button in the settings, restarted, everything started downloading. I returned the old language, and everything is downloaded, so I think it's the save settings button.

I have the same error, and not one course out of 70 was not downloaded. After I changed the language and pressed the save button in the settings, restarted, everything started downloading. I returned the old language, and everything is downloaded, so I think it's the save settings button.

It sounds right, dude! Following your advice I clicked on the settings tab, saved them (without making any change, in my case), back to the courses list and…it worked well!
Congrats!

check new version
https://github.com/heliomarpm/udemy-downloader-gui/releases/tag/v1.12.5


SASS Error since update to 2.0.0-rc.2 and app-script 0.0.45

Short description of the problem:
SASS Error since update to 2.0.0-rc.2 and app-script 0.0.45
What behavior are you expecting?
To see my App
Steps to reproduce:

ionic serve --lab

Which Ionic Version?
2-RC2
App Script Version?
0.0.45
Error output
Sass Error
File to import not found or unreadable: ionic.globals Parent style sheet: /Users/Carl/CraftingCoach/src/theme/variables.scss

src/theme/variables.scss
// http://ionicframework.com/docs/v2/theming/
@import "ionic.globals";

Ionic Framework: 2.0.0-rc.2
Ionic Native: 2.2.3
Ionic App Scripts: 0.0.45
Angular Core: 2.1.1
Angular Compiler CLI: 2.1.1
Node: 6.7.0
OS Platform: macOS Sierra
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.98 Safari/537.36

Run ionic info from terminal/cmd prompt: (paste output below)

This issue was moved to driftyco/ionic-app-scripts#422


fix timestamp

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#17185

@samdark @GHopperMSK
@filemtime($dstFile)

I don't think that's good. It's better to get an error than to guess what goes wrong

@s1lver yes but it's in the same way that other timestamps added in the same class.

really,but don't keep doing that

Won't do in Yii 3.


Allow users other than root to use helm plugins

Hi,
as discussed in this issue  https://github.com/roboll/helmfile/issues/1134, I propose to allow all users to go through root directory to be able to use helm plugins in container running as non root users.

Hi @mumoshu, what do you think about that?

@aperigault Hey! Thanks. Honestly speaking I don't understand for what usecase this can be helpful.
How did you test this? What was your example usecase and the exact docker-run command you used to verify this?

Or maybe you tried to complement https://github.com/roboll/helmfile/issues/1134#issuecomment-605624954 and tried to fix https://github.com/roboll/helmfile/issues/1134#issuecomment-610836923?

Hi!
My use case is to run this container as non root in kubernetes due to PSP limitations and use plugins installed in /root folder.
Yes I want to fix https://github.com/roboll/helmfile/issues/1134#issuecomment-610836923

Hi @mumoshu,  do you think this is a good workaround ?


Cant boot

Hi I have the same Laptop like you, but when i boot from the USB it is stuck like this and cant find the installer ...
https://preview.redd.it/wdf9gxk23zj81.jpg?width=991&format=pjpg&auto=webp&s=2aae8068717e90d6b58a36c672ea9f2a43b733e2

Laptop boot press alt to enter the boot selection screen because I hid it #3


Deployment commit for prod-beta

Merged stage-stable branch to prod-beta.
Use latest commit to update namespace ref in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch.

Codecov Report
Attention: Patch coverage is 23.07692% with 10 lines in your changes are missing coverage. Please review.

Please upload report for BASE (prod-beta@4333557). Learn more about missing BASE report.

Files
Patch %
Lines

src/routes/components/charts/common/chartDatum.ts
0.00%
10 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             prod-beta     #332   +/-   ##
============================================
  Coverage             ?   71.45%
============================================
  Files                ?       81
  Lines                ?     1058
  Branches             ?      240
============================================
  Hits                 ?      756
  Misses               ?      290
  Partials             ?       12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Real-Time Sched Search deprecation notice

Purpose of this pull request
Update, per Jay S., we need to go with the original version. I've deleted deprecration-edit.md.

This pull request is adds a temporary Real-Time Sched Search deprecation notice. NOTE: In this initial PR, you'll see two versions of the doc. Only one will be published.
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
https://sumologic.atlassian.net/browse/DOCS-102

@kimsauce, the two files look good, but they are almost identical. Why isn't there just one article for the deprecation?

See note in my description :D


[gitops-wg] docs(README.md): 1st & 3rd Wednesdays at 19:00 GMT

https://github.com/open-gitops/project/discussions/33

Please review, approve, merge:

https://github.com/cncf/tag-app-delivery/pull/166
https://github.com/gitops-working-group/gitops-working-group/pull/181
https://github.com/open-gitops/website/pull/46

Thank you!

Thank you @lloydchang - resolved merge conflicts 👍


Upgrade to reakit 1.x?

Is there a plan (i.e. document or discussion someplace) in place to upgrade to reakit 1.x?
I did a bit of searching in the issues, but didn't see anything. I realize things are still a bit volatile, as they just released a beta version.
Since they pulled styled-components, (https://reakit.io/docs/styling/), and are using an experimental system feature, this seems like a significant change.
Here is the migration guide.

Hey!
There sure is a plan to migrate to Reakit v1. I just wanted the initial release of the beta to sit for a short period before I start writing a migration document/path.
I’ll be planning to start this process in the next couple of weeks.
I’ll first post an issue which will list out the things that need to be done to make this happen & we would love for anyone to have input!

I'll be glad to help on this. It's completely possible to keep fannypack API using Reakit v1. Even though many things should be re-created, others should be easier (like DialogModal). Here's an example of how flexible the v1 API is: https://reakit.io/docs/tooltip/#abstracting
I don't recommend relying on reakit-system-* packages for now since they're experimental and I don't expect them to be stable so soon. On the other hand, despite being in beta, the reakit package is pretty stable right now. It's unlikely that it'll have drastic changes until the final release.
That said, I think it's really valuable for the community that fannypack exists. Thank you. ❤

Hey @diegohaz,
Yeah of course, I think a few components definitely will need to be re-written. I'd love to strip out some of Fannypack's logic/components for some of Reakit's new logic/components (such as Menu, VisuallyHidden, etc), however, some new functionalities & components will need to be implemented (such as the Overlay, Sidebar, Field, typography components, etc).
Early next week, I'll create an issue of a detailed migration path to Reakit v1 which I'd love to get some feedback on. Hopefully we can easily incrementally migrate to v1! I think we can achieve a Reakit v0 to v1 migration without changing Fannypack's API or breaking anything (hopefully).


UPDATE_SYNC_ERRORS being called after I start typing

So something weird is going on in my form.  When I focus into an input to start typing, it stops after the first letter and I have to focus into the input again with my mouse to continue typing.  I looked at the actions and when I type in the first letter, it dispatched action UPDATE_SYNC_ERRORS.  Could this be the reason why this issue is occurring?  If so, what can I do to fix this?

Are you creating a component inside the render function like this?
<Field
  name='test'
  component={() =>
}
/>

If so, React is unable to detect that it's the same component because it's actually a different instance. You need to declare your components outside of render.

@clayne11 I was creating the component inside the render.  When I took it out, it worked!  Thank you!!
But now I've run into another problem.  The form I'm using is repeated based on the number of travelers.  So there are 2 instances of the same form.  When I fill in the value of the first form, it also fills it in the for the second form - so there's some sort of two-way binding going on.  Do you know what is causing this?

The whole point of redux-form is that it stores all of the values of the form inside the redux store. There is a single source of truth which is the store. You can only have one instance of each form.
You can either:

Create a new form name, such as traveler-1, traveler-2 dynamically
Use a FieldArray and have each instance of the form be a member of the array.

Depending on your use case one of those should work.

Ok I see.  I'm having a little trouble understanding how FieldArray works in my case.  The examples I see are mapping through a property called 'fields'.  I need mine to map through the number of travelers, which in this case is 2.  Traveler's is an array of traveler objects which is on props, so it's accessed via this.props.travelers.  This is what my render looks like:
class TravelerForm extends React.Component {
  render () {
    return (

        <FieldArray name='travelers' component={renderTravelers} props={{travelers: this.props.travelers}} />

    )
  }
}

And this is what renderTravelers looks like:
const renderTravelers = props => (

  {props.travelers.map((traveler, index) =>
    <Row key={index}>
      <Field name={`first_name${index}`} component={renderField} placeholder='First Name' type='text' />
      <Field name={`last_name${index}`} component={renderField} placeholder='Last Name' type='text' />
      <Field name={`gender${index}`} component={renderSelect} />
      <Field name={`date_of_birth${index}`} component={renderField} placeholder='Date of birth' type='date' />
    </Row>
  )}

)

Is this the right way to do it?  I'm not mapping though props.fields at all and to be honest, don't really know how to use it in my case.  The examples of FieldArray I'm seeing are with a form that has an "Add another field" button, which I don't have. Something tells me that I should be mapping through props.fields instead of props.travelers but how can I tell props.fields that I want it to map the number of travelers on this trip?  I hope my question isn't too confusing.

Hi @nyjy85, try defining initialValues.travelers.
This way props.fields will be bound to it (as long the FieldArray name prop is the same), and you can just follow the documented example for FieldArray.


Support MySQL / MariaDB Sockets

Hello,
most databases I have to work with have no password for root user.
They use socket auth instead.
 mysql -S /var/run/mysqld/mysqld.sock

  -S, --socket=name   The socket file to use for connection.

Can that be supported?
Cheers!

Hi,
Yes, that can be eventually supported. I have to address a couple of issues,  first.
Nice suggestion!
Thanks


The time which the glInvalidateFrameBuffer function spend seems to be wrong.

Description
When i profile on SamSung S20, i found that glInvalidateFrameBuffer function seems to use too much time.

when i use snapdragon profiler, it seems the glInvalidateFrameBuffer function is very fast.

glInvalidateFrameBuffer is in the glClear

So, is the renderdoc get the wrong time or the snapdragon profiler get wrong time?

It seems renderdoc fill the RT with string "invalidated", is this process use too much time?
Steps to reproduce
I create a simple project to reproduce the problem, it's a unity project:
testshadow.zip
If you can't test it on android, you can test it on windows, set project api to opengles, you can test the function time use renderdoc and nsight, you will get similar result.
this is the compiled exe for windows:
Output.zip
Environment

RenderDoc version: 1.1
Operating System: win7
Graphics API: opengles
Phone: SamSung S20

RenderDoc is not a profiler, so you should not rely on the timings provided as they may be affected by other factors. In this case it's most likely caused by the discard pattern, but in general this may be caused by driver issues, hardware variances, internal renderdoc overheads or more.


[OUT-3518] Integrating CI

Changes

Setup Travis CI for this repo
Introduced a travis.yml file to configure the build settings
Introduced a travis.sh file to setup and control the build steps
Fixed some dodgy dependencies causing gradle to fail builds

This involved excluding support library from the firebase dependency

Removed GCM from sample app

The first Travis build actually caught this syntax error introduced after removing GCM from the SDK previously

Removed an unused test directory from the SDK module

Notes
There was some code in the sdk/build.gradle that I have commented out rather than deleting. This is a remnant of the original build process that was causing builds to fail. I'm keeping it around until we have a new process in place.
Reviewers
@zendesk/adventure-android @zendesk/two-brothers-android @zendesk/atom-android
Risks

Low
Affected areas:
This should make PR safer by ensuring tests are run before changes can be merged but there's the chance of fragile tests preventing merges or travis misconfiguration rearing it's head at a later stage.

To QA

Expected Result:
We should see Travis builds triggering in this PR and on any PR into master once this branch gets merged.

I understand there might be some overlap between the two tasks, but it might be handy when looking back in future to see them as 2 separate PRs, especially if you're going to squash/rebase.

Yeah it makes sense to split it up. I'll have to merge PR 2 first because PR 1 will trigger the CI builds and fail because of the messed up dependencies. Cheers

Closing this in favour of the separated PRs


Implement start_function section

Decode start_function section and run upon instantiateModule()

Hi @commonlisp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@dotnet-bot test this please

Hi @commonlisp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Hi @commonlisp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please

@dotnet-bot test this please

@MikeHolman Any additional feedback on this one? Thanks!

lgtm


Add hostname to metadata

"hostname" parameter should be present in metadata
go-metadata should be updated as well

Tested with latest build from master:
Able to get hostname from rancher-metadata using http://rancher-metadata/latest/self/host/hostname


FSCalendar for MacOS

Hi,
This is a general open question ... is there an equivalent of FSCalendar for the Mac? I want to utilize something like this inside my Mac app, but not sure if FSCalendar can be adapted for it, or if there is an equivalent control for AppKit. I've been looking but I haven't found one yet.
Thanks.

@zulfishah Did you find any alternative until FSCalendar supports macOS?

I didn't find any alternative. I'm was going to look into if the NSDatePicker with graphical mode can be made to work like FSCalendar, but I wasn't sure if it's possible to customize the size and look of it. I haven't had a chance to dig deeper into that.

It's worth pointing out that FSCalendar works pretty well with Catalyst. There is an issue with selecting dates that I haven't been able to figure out yet, but it does work.

Hi,
I did find a useful alternative, called NMDatePicker. It's a little basic compared to FSCalendar, and it doesn't allow for multiple date selection out of the box, but it was easy to add it on.


Implement Alien Animals Basic

Description
This feature seeks to implement basic alien animals which will include visuals, movement, behaviour patterns and interaction with player/environment. The animals can help progress towards the win condition, they may assist the player by providing a means of resource generation. Animals are initially wild and can be tamed/domesticated using their favourite plants.
SubFeature/component (optional)

Visuals and animations
Movement
Behaviour patterns (etc., skittish, neutral). Tamed vs untamed.

Examples (optional)

Alien equivalents of cows, chickens and dogs. Details may change later.

Dependencies

Can be coded without plants but will be required for future functionality.

Milestones
List of sub-features you intend to have done and in Master by the end of the sprint.

[x] Develop characters for animals including description, visuals and behaviour (Aug. 16)
[x] Create visual assets, standing animation, moving animation (calm, frightened, interested), death animation (Aug. 23)
[x] Integrate animals into the game which includes entity representation, basic movement (Aug. 26)
[ ] Implement behaviours such as tamed, frightened etc., (Aug. 28)

Completion Deadline: Aug. 30
Documentation

Main description of feature
User documentation
Animals

Members

@27abdullah
@v1dyut
@pchsa
@UniversalTze
@Bradbowi

Despite most features being implemented, due to dependencies on other teams, the tameable functionality will be finalised in Sprint 2.


handle RST packets under the status of tcp established

when tcp connection was established, if one RST packet
received from peer next, the current connection should
be free.

I noticed the same issue and send a bug fix before I noticed this patch. I will remove my patch as it came later.

@jukkar : With all due respect to @Likailiu's work, I would prefer your patch (also @tbursztyka, @lpereira, @andyross 's) to a patch of someone who just starts with the Zephyr IP stack (this includes me). While investigating few last issues, I already got an impression that contributors oftentimes literally patching the IP stack on spot, instead of trying to find a better underlying solution to a problem. My "draft" and "WIP" patches definitely show that tendency, and my real hope is that by posting them, I would prompt the core maintainers to look into a proper fix instead.
Given that we're single weeks away from the 1.8 freeze, it really amounts to whether 1.8 will have "more working" or "more broken" TCP support, so @jukkar, I would encourage you to continue with your patch instead, and I hope @Likailiu would agree.
Thanks!

OK, I agree with pfalcon and will  remove this patch.@jukkar @pfalcon @simple2017


Move from hexdump-nodejs to hexy

Move dependency from hexdump-nodejs to hexy and plug in some configurability.
Following workspace settings are now available:
    //show offset on top:
    "hexdump.showOffset": true | false (default true)
    //show address on the left:
    "hexdump.showAddress": true | false (default true)
    //ascii annotation at end of line:
    "hexdump.showAscii" : true | false (default true)
    //bytes per line:
    "hexdump.width": N (default 16)
    //hex digits format:
    "hexdump.caps": "lower" | "upper" (default "upper")

If this PR makes into master I have following ideas/proposals:
0. Change settings on the fly

Variable number of nibbles - 2 or 4, had to drop support from this PR dues to some issues
Multiselect - highlight ascii when hex is selected and vice versa
Improve inspector - currently two selected bytes are not always correctly annotated

Good call, hexy is way more flexible than hexdump-nodejs. Thanks for the PR!
Don't hesitate to add your proposal as Issues, it will be easier to track.


Merge greenkeeper

Closes #126
Closes #132
Closes #133
Closes #134
Closes #135
Closes #136
Closes #137
Closes #138
Closes #139
Closes #140
Closes #141
Closes #142
Closes #143
Closes #144
Closes #145
Closes #146
Closes #147
Closes #148
Closes #149
Closes #150
Closes #151
Closes #152
Closes #153
Closes #154
Closes #155
Closes #156
Closes #158
Closes #160

@koistya @frenzzy Please :-)

Coverage remained the same at 100.0% when pulling 67027b0e8ea94e067448d42ac16d0f0bd52ad5e5 on langpavel:merge-greenkeeper into 0038b35c704113f4927fc9d578161e45b9666584 on kriasoft:master.


rustup

Adjusts for https://github.com/rust-lang/rust/pull/96657. For now we use the fallback path in libstd.

@bors r+

:pushpin: Commit 5a6c4a60fedf354f10309047455420edb1dc9968 has been approved by RalfJung

:hourglass: Testing commit 5a6c4a60fedf354f10309047455420edb1dc9968 with merge f791e2ee50395f01f2b9cea0418c12ce413208ff...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing f791e2ee50395f01f2b9cea0418c12ce413208ff to master...


Generated infrastructure model

Hello! 👋
DESCRIPTION
This pull request adds a UML-deployment diagram to your documentation.
The diagram has been automatically generated and includes multiple levels, making it easy to edit, extend, or update. You can view and customize the diagram here.
Additionally, if you find this useful, you might consider using the Infragenie GitHub Action to keep the diagram in sync with your repository.
Feedback
I apologize for any inconvenience this unsolicited PR may cause. I'm currently developing Infragenie (infragenie.eu) as part of my master's thesis, and it is in the early preview stage. Your feedback would be incredibly valuable to me.
Whether you decide to accept or reject this pull request, I kindly invite you to participate in a short, 5-minute survey to help improve the application. Your insights will guide future features and enhancements.
Please consider answering our survey here.
Thank you for your time and understanding.
Best regards

It might be a good idea to ignore the tests directory as I think that's adding a lot of noise to the diagram.

Hi @mkst ,
Thank you for your time and feedback. I completely agree with your suggestions, and we will take them into account for the next release. In the meantime, I have updated the diagram for you to take a better look.
If you could also take a moment to answer the brief survey related to the automatic generation of architecture documentation, it would be incredibly helpful for improving the tool for community use.
Thank you so much for your support!
Best regards 🙂


cli: add new "--limit-" options

To give the user more control over the bst process, add three new options for
runing setrlimit before exec.
--limit-core: RLIMIT_CORE
--limit-nofile: RLIMIT_NOFILE
--limit-nproc: RLIMIT_NPROC

Each option requires an argument which is parsed as a colon-separated tuple of
rlim_max, rlim_cur values.  rlim_cur is optional, and defaults to rlim_max when
not provided.
--limit-nofile=100 => setrlimit(RLIMIT_NOFILE, (100, 100))
--limit-nofile=100:50 => setrlimit(RLIMIT_NOFILE, (100, 50))

Looks good, with a few comments. There are two things I'd also like you to add:

Could you add an entry in bst.1.scd documenting the new --limit- flags?
Will do.

I'd like the rlimit implementation to be complete -- let's add support for all RLIMIT_* values.
Prefer to do that in a follow up of some kind.


Add s390x support

This PR adds support to build on s390x architecture.

We are changing the image based on BCI, so this PR needs to be rebased later.
Could you check if Nginx package is able to support from BCI builtin repos? actively released?

nginx is available for SLES 15.3: https://scc.suse.com/packages/21182999
and I can confirm it it can be installed on the bci-base:15.3 image with the default repositories.

nginx is available for SLES 15.3: https://scc.suse.com/packages/21182999
and I can confirm it it can be installed on the bci-base:15.3 image with the default repositories.

Is it actively released? What's the current version?

nginx version in SLES 15.3 is 1.19.8-3.6.1 at the moment and it's in the SLE-Module-Server-Applications15-SP3-Updates
so it looks like it's actively maintained.

nginx version in SLES 15.3 is 1.19.8-3.6.1 at the moment and it's in the SLE-Module-Server-Applications15-SP3-Updates so it looks like it's actively maintained.

@smallteeths will you have any concerns if we use this version instead of the current one 1.21.x?

nginx version in SLES 15.3 is 1.19.8-3.6.1 at the moment and it's in the SLE-Module-Server-Applications15-SP3-Updates so it looks like it's actively maintained.

@smallteeths will you have any concerns if we use this version instead of the current one 1.21.x?

longhorn-ui starts a web service with nginx, using the most basic configuration of nginx.As long as the current version of nginx has no CVE security holes, we can change the nginx version.


Supermium does not play some videos

I've noticed that Supermium v115 refuses to play some videos that are played by Chrome v109 (last official release before dropping Win7 support) just fine.
For example, open this page: https://tekeye.uk/html/html5-video-test-page
You'll probably notice that "HTML5 MPEG Video Test" does not work.
It looks like some Chromium v115 built-in codecs are missing or they malfunction on Win7.

Yes, I did not add support for H.264/H.265 codecs at first. Indeed this is yet another part of Chromium building that isn't well documented.
I will try building with:
proprietary_codecs=true
ffmpeg_branding="Chrome"

next time. This should bring it up to Google Chrome's multimedia support level unless the process has changed recently.

was H.264/H.265 already added to 115 bulid of this SW?

Not yet. In testing builds I added H.264 successfully but H.265 does not seem to be possible with any of the build config flags I've tested, so that might be restricted to "Google Chrome"-branded builds.
I haven't been able to release them yet due to some issues that persist in these builds (which are not related to media playback).

OK, great news anyway!
any chance to have a possibility to test at least H.264 build?
thanks in advance


chore: Drop conversion webhooks

Fixes #N/A
Description
Dropping conversion webhook
How was this change tested?
Manually tested
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 78.863%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.4%

Covered Lines:
5727

Relevant Lines:
7262

💛  - Coveralls

We still need to drop the changes to kubelet config https://github.com/aws/karpenter-provider-aws/blob/8071346bfe6064d83af8a80023bbcefdbe1196e9/pkg/utils/utils.go#L99

Hello,
Do you have any ETA for this to be released for v0.37.x?
@engedaam @jigisha620

This will not be released for v0.37.x. This will be included for v1.1.0


[LSP] Failed to handle notification initialize

I'm working on integrating vim-ale directly with omnisharp-roslyn's LSP server, as omnisharp-vim does not use the LSP as far as I can tell. This is so I can get auto-completion through vim-ale and LSP and not through omnicomplete itself.
In doing so, I've been getting errors back from omnisharp's LSP server. More specifically:
{"jsonrpc":"2.0","method":"window/logMessage","params":{"type":1,"message":"Failed to handle notification initialize - System.NullReferenceException: Object reference not set to an instance of an object\n  at OmniSharp.Extensions.LanguageServer.Server.LanguageServer.OmniSharp.Extensions.Embedded.MediatR.IRequestHandler<OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams,OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeResult>.Handle (OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams request, System.Threading.CancellationToken token) [0x0022d] in <c33e73e38fa14bf293bda2f62d3759c4>:0 \n  at OmniSharp.Extensions.LanguageServer.Server.Pipelines.ResolveCommandPipeline`2[TRequest,TResponse].Handle (TRequest request, System.Threading.CancellationToken cancellationToken, OmniSharp.Extensions.Embedded.MediatR.RequestHandlerDelegate`1[TResponse] next) [0x000e1] in <c33e73e38fa14bf293bda2f62d3759c4>:0 \n  at OmniSharp.Extensions.LanguageServer.Server.LspRequestRouter.RouteRequest (OmniSharp.Extensions.JsonRpc.IHandlerDescriptor descriptor, OmniSharp.Extensions.JsonRpc.Server.Request request) [0x002e2] in <c33e73e38fa14bf293bda2f62d3759c4>:0 "}}Content-Length: 129

{"jsonrpc":"2.0","method":"window/logMessage","params":{"type":3,"message":"Finished: Routing Request (1) initialize in 1000ms"}}Content-Length: 1158

{"jsonrpc":"2.0","id":"1","error":{"code":-32602,"message":"Internal Error - System.NullReferenceException: Object reference not set to an instance of an object\n  at OmniSharp.Extensions.LanguageServer.Server.LanguageServer.OmniSharp.Extensions.Embedded.MediatR.IRequestHandler<OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams,OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeResult>.Handle (OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams request, System.Threading.CancellationToken token) [0x0022d] in <c33e73e38fa14bf293bda2f62d3759c4>:0 \n  at OmniSharp.Extensions.LanguageServer.Server.Pipelines.ResolveCommandPipeline`2[TRequest,TResponse].Handle (TRequest request, System.Threading.CancellationToken cancellationToken, OmniSharp.Extensions.Embedded.MediatR.RequestHandlerDelegate`1[TResponse] next) [0x000e1] in <c33e73e38fa14bf293bda2f62d3759c4>:0 \n  at OmniSharp.Extensions.LanguageServer.Server.LspRequestRouter.RouteRequest (OmniSharp.Extensions.JsonRpc.IHandlerDescriptor descriptor, OmniSharp.Extensions.JsonRpc.Server.Request request) [0x002e2] in <c33e73e38fa14bf293bda2f62d3759c4>:0 "}}

In other words, this is the exception that omnisharp-roslyn is giving me during initialization:
System.NullReferenceException: Object reference not set to an instance of an object
  at OmniSharp.Extensions.LanguageServer.Server.LanguageServer.OmniSharp.Extensions.Embedded.MediatR.IRequestHandler<OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams,OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeResult>.Handle (OmniSharp.Extensions.LanguageServer.Protocol.Models.InitializeParams request, System.Threading.CancellationToken token) [0x0022d] in <c33e73e38fa14bf293bda2f62d3759c4>:0
  at OmniSharp.Extensions.LanguageServer.Server.Pipelines.ResolveCommandPipeline`2[TRequest,TResponse].Handle (TRequest request, System.Threading.CancellationToken cancellationToken, OmniSharp.Extensions.Embedded.MediatR.RequestHandlerDelegate`1[TResponse] next) [0x000e1] in <c33e73e38fa14bf293bda2f62d3759c4>:0
  at OmniSharp.Extensions.LanguageServer.Server.LspRequestRouter.RouteRequest (OmniSharp.Extensions.JsonRpc.IHandlerDescriptor descriptor, OmniSharp.Extensions.JsonRpc.Server.Request request) [0x002e2] in <c33e73e38fa14bf293bda2f62d3759c4>:0

For reference, this seems to be the command that vim-ale is sending to initialize the LSP:
{"method": "initialize", "jsonrpc": "2.0", "id": 1, "params": {"initializationOptions": {}, "rootUri": "file:///Users/luiz/projects/mgsb", "capabilities": {}, "rootPath": "/Users/luiz/projects/mgsb", "processId": 28524}}

Any idea what's up?

@luizribeiro I think this is most likely because you don't send any client capabilities in your initialization request. Try to add at least support for text synchronization in your client implementation.

Thanks @mholo65, you are right! Looks like vim-ale doesn't send any capabilities because of a TODO that was left behind:
https://github.com/w0rp/ale/blob/3b7c86e401eb488acd8713f7c9ee81344095080d/autoload/ale/lsp/message.vim#L32-L40
Aren't all these fields optional on the LSP spec?
LSP should announce the capabilities it supports, but this is the first server I encounter that doesn't work with vim-ale because of this.
Either way, sending this as the capabilities made me get past the initialize message:
{
   "workspace":{},
   "textDocument":{
      "completion":{
         "completionItem":{
            "snippetSupport":false
         }
      }
   }
}

When I tried sending the text synchronization, I ran into #1191.

Some capabilities are now sent by ALE, but you should also allow clients to connect to the server without sending any capabilities. All of the keys are optional.

@w0rp yes, that’s a bug in the LSP implementation we are using. We are currently seeing other issues regarding initialization (and capabilities) and will put this on the list of things to do. //cc @david-driscoll

Tracking fix here https://github.com/OmniSharp/csharp-language-server-protocol/issues/141

This should be fixed via https://github.com/OmniSharp/csharp-language-server-protocol/pull/147

Just got hit by this using omnisharp-mono.tar.gz from v1.34.0. I guess the fix hasn't been incorporated here yet.
I worked around it by adding "workspace": {} to my request. Thank you @luizribeiro for sharing your code.

I'm pretty sure we can close this. I've been using omnisharp-roslyn successfully with vim-ale.
Thanks for all your work on this!


Add a Custom tab to Dashboard for Project Stats widget

I think we can rename and repurpose the FVT tab to a Custom tab, and make it a space where we can create a new Project Stats widget and drop it into that view help us track project stats.

A Custom tab view is one that people can 'personalize' as their selection of widgets in their view is remembered (via browser cookies).  Reset button gets them back to the set of default widgets in a view.
Custom creates a location for potentially other widgets to be written and dropped in, focussed on things that may be helpful for daily tasks and tracking.

FYI @andrew-m-leonard


Test fixes

Fix tests breakages related to https://github.com/apache/incubator-openwhisk/commit/aeabc3515eae4da0db523a78acadf984e5a9ba01 and https://github.com/apache/incubator-openwhisk/commit/e88257ea72dd4341726f50af72e8924fc1ddb966.

Thanks @rabbah.


explore real-time multi-thread recover_key capability

Change Description
Related to #6149, this allows recover_key to be handled for real-time transactions in other threads, freeing up the main thread for over 60% more TPS.


Why the result of using multiple evaluation is 0

my settings

Are you running the execution inside the docker image we provide for MultiPL-E? as it requires extra dependencies


Browser-Specific should be based on "two or more others"

The metrics doc defines browser-specific as:

How many APIs does this engine ship that appear in fewer than 2 other engines for at least a year?

README.md, confluence.html, and the metric calculator (lib/confluence/browser_specific.es6.js) document and implement this as:

How many APIs does this engine ship that appear in no other engines for at least a year?

This is important to fix especially because the common WebKit lineage of Safari and Chrome deflate their metric.

Talking more with @foolip, I think maybe "browser-specific" is fine as it is - it's what we want for our use case of talking to chromium teams about APIs they've shipped that failed to get traction elsewhere.  So we should fix this bug just by updating the confluence metrics doc to reflect the implementation.
But for the "eliminate legacy WebKit APIs" use case, perhaps we'd eventually like another "Fragmented APIs" list that's between "vendor-specific" and "failure-to-ship" - exists in only 2 browsers for some time.

I'd be interested to see the "fragmented" graphs. If we thrown in an "exists everywhere" metric we'll have all permutations of existence, 1,2, 3 and 4. I think I'd be interested in seeing all of those without a grace period, to see if the graphs are totally smooth and non-actionable, or if we see some larger jumps.