SoupMessageHeaders

Hi,
I'm trying to integrate an oauth sign-in page in my gtk3-sharp mono app, which requires an integrated web browser from which I can read the HTTP response headers.
I tried using webkit2-sharp, which seems to work fine - except that the methods/properties for reading the headers are missing.
I noticed these warnings (among many others) when building:
WebKit2.URIRequest.GetHttpHeaders - Unknown return type: SoupMessageHeaders*
WebKit2.URIResponse.HttpHeaders - property has unknown type 'SoupMessageHeaders*'
WebKit2.URIResponse.GetHttpHeaders - Unknown return type: SoupMessageHeaders*
Do you have any ideas about how I might be able to go about solving this?
Thanks,
Dave

It means that these APIs could not be bound automatically. I've adapted a couple of APIs for my use cases, but not the whole set. You can add methods and their adjustments in https://github.com/hbons/webkit2-sharp/blob/master/sources/webkit2-sharp.metadata but I'm not really sure how this works anymore...

btw, this project was automatically generated by Bindinator, which itself uses Gobject Introspection to generate the bindings. I can't quickly find the manual for the metadata scheme, but that would be a good place to start. I'd accept your PRs if you can make it work. :)

Found that gtk-sharp has lots of examples: https://github.com/mono/gtk-sharp/blob/fa0897dc2cd15f76ca09ad08de9fbf9efa411f8d/gtk/Gtk.metadata

Thanks for the help. I'll fiddle around with it and see if I can get it to work


chore: add nodejs minimum version requirement

optional chaining is supported after nodejs 14.

Please resolve the conflicts before merging.

Please resolve the conflicts before merging.

Resolved.


LPFG-717: adding audience create/get methods to course controller

Adding audience create/get methods to course controller

@elxsy re removing hashcode impl - couldn't see the benefit added in current setting and there was no equals implementation, so it seemed to be done halfway anyway

That's fine, just wanted to double check. Update and merge when needed please.


Create camel JMS example

Something that shows some interaction with the swarm messaging fraction through camel-jms and maybe some transactional behaviour that makes use of the swarm transaction fraction.

https://issues.jboss.org/browse/SWARM-429


filetype(): Lstat failed

Please refer to the following discussion.
https://fuelphp.com/forums/discussion/comment/11037
The issue is five years old, but it has not been resolved yet.

How can it be reproduced? I never got a response on what caused it, and none of our applications, and none of our clients applications have it.

p.s. you've created this issue in the wrong repo, please re-create it in https://github.com/fuel/core/issues.


Very similar project

Hey @dburles,
I just saw this repo and wondered if you know about my project at https://github.com/mizzao/meteor-user-status. Your rewrite, as of right now, looks really similar to that code, including what happens in the main publish function. Would it make sense to potentially collaborate and maintain the same project rather than two projects that do pretty much the same thing?

I was looking for a native, self-hosted, Meteor analytics solution, and have found nothing. The closes would be these two packages and @andrewreedy's visit-tracker. Thoughts?


Add link to bioportal (meddracode); tidy up layout and export values

This PR updates the Pharmacovigilance (adverse events) table in the drugs page as per issue 1035

link adverse event to Bioportal with meddracode
fix width of columns for tidy layout
fix export value fields in downloaded file

Note: before merging please for @d0choa or @andrewhercules to confirm link is correct.

The links look something like this for me:
https://deploy-preview-275--platform-app.netlify.app/drug/https://identifiers.org/meddra:10037400
They will be perfect if you remove the first part and they look like this:
https://identifiers.org/meddra:10037400

The links look something like this for me:
https://deploy-preview-275--platform-app.netlify.app/drug/https://identifiers.org/meddra:10037400
They will be perfect if you remove the first part and they look like this:
https://identifiers.org/meddra:10037400

Thank you @d0choa - the link is now fixed. PR ready for review.

Thank you @d0choa - the link is now fixed. PR ready for review.

PR is now ready for review.

PR is now ready for review.


Extract image direction data from Exif

The Exif data also stores the direction of the image.
For simplification, the commit ignores the difference between true and magnetic north.

Codecov Report

Merging #340 into master will decrease coverage by <.01%.
The diff coverage is 57.14%.


No such file or directory

I have just built from source on Ubuntu 22.04 x86_64, and I get many warnings during build, which seem to be present in the CI job.
Worse, it seems to not function correctly, reporting an error when running the binary:
$ ./lua-language-server
Error: ExternalError(Os { code: 2, kind: NotFound, message: "No such file or directory" })

Because it must run publish script, the executable file and resources need to be packaged together through the publish script. I'm not sure if there is a mature solution in Rust, so I chose to write a shell script to handle this:
./publish/UnixPublish.sh

The complete package structure looks like this.

I see, I thought the build step embedded the resources in the binary. After moving dist/ over to replace the server/bin folder in my VS Code client, it started working for VS Code! I had to also copy over my meta definitions from vscode-client/server/meta to vscode-client/server/bin/resources/meta in order to get the definitions for everything, but it seems to work just fine!
This makes me wonder, how hard would it be to add HTTPS support to this Rust runtime? It could allow the addon manager to be built into the language server in Lua. Perhaps it is best as an optionally installed tool, though.
I also wonder if some of the hardest jobs like the AST parsing could be done in Rust to increase performance.

Referencing my project, I have completed the tool for managing packages through GitHub. If you only want to use just one project, these two projects can be merge easily.

As for performance, if we really want to fully improve its performance, I think switching to LuaJIT as the runtime might be an easier choice. It might be difficult for the previous implementation based on bee.lua, but not too hard for this project. It just requires modifying a few features that use Lua 5.4. However, if we do this, we won't be able to easily migrate to LuaLS 4.0. As for considering a complete implementation in Rust, I would prefer a complete rewrite of the language server.

I have completed the initial migration to LuaJIT. You can try switching to the LuaJIT branch to experience it.


Bug when dealing with Double numbers from RDBs

The condition used within this function to remove trailing decimal points .0 is affecting proper double numbers. For example, 19.045 becomes 1945.
The solution is simply checking if the number actually ends with .0 to perform the transformation:
if (DOUBLE.equals(dataType) && data.endsWith(".0")) {
   // remove trailing decimal points (Quirk from MySQL, see issue 203)
   return data.replace(".0", "");
}

Fixed on development branch

Fixed in 950a77cb8f2a9254cf387cf3b6bbdca778fb49fb


ci: add 6GiB for ci runs

Summary:
This PR adds memory while running the validator on ci.
Expected behavior:
No OOM when executing the validator on Flanders' gtfs dataset (https://openmobilitydata.org/p/vlaamse-vervoersmaatschappij-de-lijn/530)

Please make sure these boxes are checked before submitting your pull request - thanks!

~[ ] Run the unit tests with gradle test to make sure you didn't break anything~
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
~[ ] Linked all relevant issues~
[x] Include screenshot(s) showing how this pull request works and fixes the issue(s)

Thanks, LGTM!
Just curious: would 4 or 5 GB be enough?

it would have probably be enough, I set it to 6GiB to be consistent with the end_to_end_big workflow :)


prompt-toolkit should auto-color and auto-complete color names as user types

auto-complete is working, auto-color seems more difficult; need to use syntax formatting


Roaming between multiple APs

Thanks for this integration, it looks very interesting.
How does it behave with clients roaming between multiple APs on the same network?
Have you considered integrating it with DAWN? This would allow users to roam around the house from AP to AP without false disconnect notifications.

Thanks for the detailed explanation. This sounds very useful.
I'll give it a spin once your OpenWrt Packages PR is merged.


feat(deps): update icr.io/ext/sysdig/agent docker tag to v13

This PR contains the following updates:

Package
Update
Change

icr.io/ext/sysdig/agent
major
12.20.0 -> 13.0.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

BREAKING CHANGE: dependency major version update

/run pipeline

/run pipeline

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


Marker are http 404

I just add simple marker like:
		markers: { 			carMarker: { 				lat: 56.99611, 				lng: 40.97858, 			} 		},
but not see it in browser, in console 404 error:
http://domain.com/css/images/data:image/png;base64,iVBO......==

+1

Please, can you provide us more details, an example in JSFiddle would be great.
Thanks.

I have the same problem and I using ui-leaflet 1.0.3 and leaflet 1.0.0.
I found the issue '#89 (Get ready for Leaflet 1.0)' to make ui-leaflet compatible with leaflet 1.0.x, but it was opened on Nov 2 2015 and it's not closed yet.
If you are using ui-leaflet 1.0.3, can you try to use leaflet 0.7.x? (that's the version that appers in bower.json)

Please use leaflet-1.X branch.

I made a fork of the repository to have a pre-release version of that branch available via bower.
Thank you.

@elesdoar
It does not help.

@morfair if you are using Leaflet 1.x is required to be used leaflet-1.X branch, it's supporting Leaflet 1.x.
Please try to be nice next time. This work is done with love, but we lack hands to it be perfect, so we do what we can.
Sometimes we also need examples, not just messages that something is not working.
Have a great day.


Need a list of file extensions the will not be delivered statically

In the default implementation a default (and configurable) list of extensions delivery is not allowed. This will be used to hide code of jsp, php ... if the processor is not present or disabled.

Done by the getResourceRenderer() method now and deployment in private content sections.


Issues installing on Ubuntu 22.04

Fresh Crowdsec install on bare metal Ubuntu 22.04.
$ sudo apt install -y crowdsec-firewall-bouncer-iptables
0 upgraded, 3 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/3,093 kB of archives.
After this operation, 9,225 kB of additional disk space will be used.
Selecting previously unselected package libipset13:amd64.
(Reading database ... 193173 files and directories currently installed.)
Preparing to unpack .../libipset13_7.15-1build1_amd64.deb ...
Unpacking libipset13:amd64 (7.15-1build1) ...
Selecting previously unselected package ipset.
Preparing to unpack .../ipset_7.15-1build1_amd64.deb ...
Unpacking ipset (7.15-1build1) ...
Selecting previously unselected package crowdsec-firewall-bouncer-iptables.
Preparing to unpack .../crowdsec-firewall-bouncer-iptables_0.0.24_amd64.deb ...
Unpacking crowdsec-firewall-bouncer-iptables (0.0.24) ...
Setting up libipset13:amd64 (7.15-1build1) ...
Setting up ipset (7.15-1build1) ...
Setting up crowdsec-firewall-bouncer-iptables (0.0.24) ...
cscli/crowdsec is present, generating API key
API Key : 1b17dbc05bffaa6cad79d3025b8ff91a
Created symlink /etc/systemd/system/multi-user.target.wants/crowdsec-firewall-bouncer.service → /etc/systemd/system/crowdsec-firewall-bouncer.service.
Job for crowdsec-firewall-bouncer.service failed because the control process exited with error code.
See "systemctl status crowdsec-firewall-bouncer.service" and "journalctl -xeu crowdsec-firewall-bouncer.service" for details.
dpkg: error processing package crowdsec-firewall-bouncer-iptables (--configure):
 installed crowdsec-firewall-bouncer-iptables package post-installation script subprocess returned error exit status 1
Processing triggers for man-db (2.10.2-1) ...
Processing triggers for libc-bin (2.35-0ubuntu3.1) ...
Errors were encountered while processing:
 crowdsec-firewall-bouncer-iptables
needrestart is being skipped since dpkg has failed
E: Sub-process /usr/bin/dpkg returned an error code (1)

$systemctl status crowdsec-firewall-bouncer.service
× crowdsec-firewall-bouncer.service - The firewall bouncer for CrowdSec
     Loaded: loaded (/etc/systemd/system/crowdsec-firewall-bouncer.service; enabled; vendor preset: enabled)
     Active: failed (Result: exit-code) since Tue 2022-12-13 14:19:37 GMT; 1min 16s ago
    Process: 3831897 ExecStartPre=/usr/sbin/crowdsec-firewall-bouncer -c /etc/crowdsec/bouncers/crowdsec-firewall-bouncer.yaml -t (code=exited, status=1/FAILURE)
        CPU: 10ms

Dec 13 14:19:37 hda systemd[1]: Starting The firewall bouncer for CrowdSec...
Dec 13 14:19:37 hda crowdsec-firewall-bouncer[3831897]: time="2022-12-13T14:19:37Z" level=info msg="crowdsec-firewall-bouncer v0.0.24-debian-pragmatic-8e00af2c9e83af22deab8c0c49a4ad9b8fc57a3f"
Dec 13 14:19:37 hda crowdsec-firewall-bouncer[3831897]: time="2022-12-13T14:19:37Z" level=fatal msg="unable to load configuration: config does not contain mode and log mode"
Dec 13 14:19:37 hda crowdsec-firewall-bouncer[3831897]: time="2022-12-13T14:19:37Z" level=fatal msg="unable to load configuration: config does not contain mode and log mode"
Dec 13 14:19:37 hda systemd[1]: crowdsec-firewall-bouncer.service: Control process exited, code=exited, status=1/FAILURE
Dec 13 14:19:37 hda systemd[1]: crowdsec-firewall-bouncer.service: Failed with result 'exit-code'.
Dec 13 14:19:37 hda systemd[1]: Failed to start The firewall bouncer for CrowdSec.

Resolved.
sudo apt purge -y crowdsec && sudo apt purge -y crowdsec-firewall-bouncer-iptables
sudo apt install -y crowdsec crowdsec-firewall-bouncer-iptables


Display of Attributed Variables

Some queries return "attributed variables" from SWI-Prolog, which when they pass through MQI get converted into an additional variable called "Attributes". I need to look into understanding when this variable can be ignored, and when it is useful, and what to do with its contents.

Note that this is affecting the display of answers in questions that place a variable where we are expecting a date or duration value to be returned, forcing the user to use a calculator block to be able to see the variable in the query response.

This has been added to the current work, because we need to deal properly with constraints on abducible queries in order to be able to use those constraints to determine relevant inputs for BlawxBot. Might as well figure out how to display them in the reasoner at the same time.

It seems like the Attributes variable contains a raw representation in Prolog of the applied attributes. If the name is generated by s(CASP), that name is included as put_attr(A,scasp_variable,name(A)). That is probably OK to ignore entirely, as it shows up as the name of the value in the bindings. So we probably need to distinguish between attributes placed on variables in the bindings, and attributes that are not, and then find out which attributes to ignore (such as names on bound variables), and then figure out how to display the remainder based on their Prolog representation, and whether they are bound or not.

The changes in #245 make it possible to display numerical constraints, non-membership constraints, and ignores names. If there are other attributes that become relevant later, additional attribute types will throw console warnings.


[AMBARI-25282] : Prevent Auth error for hbck

What changes were proposed in this pull request?
hbase hbck is not working correctly and causing Authentication error from Zookeeper. Introduced new property HBASE_SERVER_JAAS_OPTS to deal with the error.
How was this patch tested?
The patch was tested locally. hbase hbck is working fine for secure cluster and showing the region consistencies correctly.
This is a backport PR to branch-2.6 from here

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5232/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5233/
Test FAILed.
Test FAILured.

retest this please

retest this please

error is not relevant

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5234/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5236/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5235/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5237/
Test PASSed.


Add transport for FreeBSD jails

FreeBSD has support for jails since FreeBSD 4.0 (released in 2000).
Add a new 'jail' transport to Bolt to allow management of jails on FreeBSD.
Also add integration tests for jail transport.  This require a jail named bolt where python3.9 is installed.
While here, use /bin/sh when only a basic shell is required and avoid depending on bash or "bashisms" (code that is not portable and only works in bash) for the code used for FreeBSD jails.

Now that the CI is fixed, I rebased on top of the main branch and added a commit to fix the relevant CI failures (you can't run acceptance tests on a non-FreeBSD node). This is now ready for review, thanks!

Nice! I added a section to the doc, this allowed me to spot that I should remove service-url from the config :wink:
Because I had to amend my old commit, I rebased on top of master and squashed the fixes to their respective commit.

Thanks, trying to fix up that modules test in https://github.com/puppetlabs/bolt/pull/3194 . it is unrelated to your PR.

:100:  I am keeping on this PR and will rebase on top of it when it is ready!  Thanks!

Looks good!  For the sake of safety, I switched the PR to draft because I did some changes that should not change anything, but I have no access to my FreeBSD work station ATM to be 100% sure I did not messed something while doing it.

I just checked that the inventory sample using a structure I am not used to and I copied from another example is working as expected (it is :wink:)!  Ready for final reviews!

Thanks! We are having some infra issues currently that is preventing me from getting this merged and released. We are working through that and I will merge this ASAP.


use the "saturated" amount as the reimbursed amount

closes #281
use the amount derived from transfer_funds_saturated() as the amount to reimburse.
General overview of the changes:

Create a storage map called CancelledRedeemAmount which will be the amount to reimburse
An item is inserted to CancelledRedeemAmount when calling cancel_redeem().
An item is removed in CancelledRedeemAmount when calling mint_tokens_for_reimbursed_redeem().

Things left to do:

[x] fix test case

Now @b-yap removed the line from the case the vault is liquidated.
What happens now if the vault is liquidated, below a secure threshold when the redeem is cancelled and the user wants to reimburse? Is this a case that can happen?
I unfortunately need more time to understand these cases.

@b-yap @TorstenStueber how should we continue with this?

I will add my comment once done with the fee model. Stay tuned, almost there.

@TorstenStueber this is the PR that we talked about in today's sync meeting, blocking Spacewalk's public launch. We still need to find some action here.

Ah understood, I give this now highest priority.


Fix/docs

Add DOI, fix typos, update documentation, make writing smoother.

Also update pre-commit library versions.


Massive performance degradation in JSON parsing from dart 1.9.3 to 1.10

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
We have a polymer  dart chrome app that stores data in indexeddb tables. Users can dump/restore the tables to a file which contains one json object per line. This file may have several thousand entries.
Under 1.9.3, reading the file, and parsing each json object takes a few seconds.
Under 1.10, watching log messages, we are seeing that parsing times have apparently drastically increased, seemingly taking ~100-250 ms per entry at least.
Downgrading to 1.93 fixes the issue.
This issue is seen in the Chrome Javascript build after dart2js has run. Will provide more details on if this issue is seen with dartium.
What is the expected output? What do you see instead?
Json parsing is reasonably fast.
What version of the product are you using?
1.10, works fine on 1.93
On what operating system?
MacOS
What browser (if applicable)?
Chrome
Please provide any additional information below.

We still haven't reproduced the slowdown.


feat: Change navigate to UI

Description
This pull request changes the navigate to global function's UI to solve the error where a user uses the "#" placeholder with a page name or URL giving an error. We also wanted to solve the case where "#" suggests a number to be entered.
We have introduced a tab-based design now that separates the Page name and URL fields. Additionally, the page name field now has a dropdown of page names for the user to select.
Designs - https://www.figma.com/file/TVXPmmFkgnIccJIFMR0MH1/Actions-Selector-Improvements?node-id=1%3A2

Fixes #9199 #1033 #1004
Type of change

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

How Has This Been Tested?

Manually

https://user-images.githubusercontent.com/10229595/177514103-a1ab2d8d-531b-4df9-bef4-f0fa709a1afd.mov

Test Plan
[ ] [[Test Case]: https://github.com/appsmithorg/TestSmith/issues/2053
[ ] [Test Case]: verify the self-referencing functionality with “#” placeholder under Page Name or URL in “Navigate to” option
[ ] [Test Case]: verify the self-referencing functionality with Page Name or URL along with placeholder “#” in “Navigate to” option
[ ] [Test Case]: verify that list of available pages appears as dropdown under “Navigate to” Option in trigger field

Checklist:

[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
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=d2213a8

/ok-to-test sha=90e4897

[ ] Page navigation does not happen when page name is changed and the dropdown still contains the old page name. So selecting old page name breaks flow
[ ] UI gets messed up when a large URL path is pasted on the field

/ok-to-test sha=0fa81cf

/ok-to-test sha=345282e

/ok-to-test sha=932ff73

/ok-to-test sha=932ff73

/ok-to-test sha=0ab1d60

/ok-to-test sha=c5ab649

/ok-to-test sha=bcdd34f

/ok-to-test sha=0dd5927

Tested design and feature changes on DP. Looks gtg from manual QA
CC: @rimildeyjsr

/ok-to-test sha=6851d38

/ok-to-test sha=0be56ad

/ok-to-test sha=e6eef72

/ok-to-test sha=8efdb50

/ok-to-test sha=e0b871f

/ok-to-test sha=5c62d3a

/ok-to-test sha=e6b4592

/ok-to-test sha=1ce91ed

/ok-to-test sha=ec10503

/ok-to-test sha=68d74ae


Integrated Avro decoding

Right now the responsibility of handling avro schemas and decoding is left to the handler. It would be nice to have an option to pass an already parsed message to the handler while performing all the mundane decoding tasks inside Phobos.

Phobos doesn't decode JSON messages, leaving that to the handlers. I think it's reasonable to leave Avro decoding up to the handler as well. There seems to be an opportunity for a library for encoding/decoding Avro with a similar focus on convenience as Phobos has for producing/consuming Kafka messages, but the two concerns are mostly orthogonal.
wdyt?

Exactly -^

We could have something similar to https://github.com/klarna/phobos_db_checkpoint
A drop in replacement for Phobos::Handler with Avro capabilities.

@tulios - What would it do that's only relevant in a Phobos context? I'm all for making it easier to produce and consume Avro messages. But let's imagine encoding/decoding Avro was as easy as JSON payload, is there still some rough edge in the interaction between Avro and Kafka that would make sense to smooth out in Phobos?

When using Avro you have to download the schemas to be able to decode the messages and upload your own schemas to the registry before you produce messages. I can see those tasks been automated. Also, when decoding the messages you have to provide the Schema to the decoder and this could also be handled in a nicer way. But like you said it's not much but there is room for improvements.

With a plugin, it will be much easier to start new projects since you wouldn't have to setup anything special for Avro, just a gem install phobos_avro and (now I'm guessing) phobos_avro setup.

Yes, but making Avro more ergonomic in Ruby can be done in a library with no dependency on Phobos. It's useful outside of a Kafka context. A library for Avro with some conventions and helpful setup tasks would be excellent, I imagine. (At least coming from the exposure to the current Avro gems I've had this week.)
We'll try to keep this in mind in our current work with Avro, and we can discuss more when we have a more concrete proposal for a library API.

What are our conclusions on this subject?

None in regards to a better Avro library. But I think this issue should be closed.


Output encoding

Ox can use document encoding: <?xml version="1.0" encoding="Windows-1251" ?>.
I want convert this encoding like this string&.encode('UTF-8', invalid: :replace, undef: :replace, replace: '?')
But my parsed item is Saxerator::Builder::HashElement
I can't do this way item.transform_values! { |value| value&.encode('UTF-8', invalid: :replace, undef: :replace, replace: '?') }
because item can contains elements of types:  Saxerator::Builder::ArrayElement and Saxerator::Builder::HashElement recursively.
Moreover I must convert encoding for attributes.
So there are 2 ways to resolve this problem.

Worst way: Add method deep_encode like https://apidock.com/rails/Hash/deep_merge
Best way: Add output_encoding to saxerator configuration and convert on parsing

P.S. And there is another question. Why we are using this types? Can we simplify like this: https://github.com/savonrb/gyoku

The primary reason for HashElement et al. was to capture name and attributes. When this was written it was useful to be able to treat values as though they were language primitives (Array, Hash, etc) most of the time and so I attempted to lightly decorate those core classes. If I were to write it again I think I would avoid the confusion/ambiguity of Hash-like and Array-like classes and instead rely on more traditional Node classes that could be converted via to_a and to_h to "pure" standard classes. I'm less-inclined to pass around primitives these days in comparison to writing my own classes in the way that I write code.
Gyoku's hash syntax is interesting for representing attributes, I could see that working. It would be a significant change to the generated structure, and a breaking change. This library is not very actively developed at the moment, if that's something you're interested in carrying forward then a fork would be the way to go.


Jumping Line UX improvement

I find it terrible trend in LLM models to generate stream of text that consistently keeps on moving during generation.
While this allows me to catch the first line quicker, eventually text moves too fast, and working with Copilot actively for a month now, I can feel how badly it hurts my eyes, I basically have to look somewhere else to avoid seeing these jumping lines.
Nevertheless, I appreciate having early output.
What would be desired:

Clear the page
Move cursor to the top left corner of the outputs widget
Disable auto-scrolling of copilot output widget
Start generating text top to bottom
If text overflows the visible screen it's okey: no-one reeds that fast

I agree, would like to improve this: https://github.com/microsoft/vscode-copilot-release/issues/138


Another interesting alternative la/matrix crate

dfdx now supports compile time and runtime axes:
https://coreylowman.github.io/2022/12/13/shapes.html
Possibly, householder could be build on top of dfdx? Just throwing this idea out....

Related to #20


black fails with hanging line comments

Howdy! Sorry you're having trouble. To expedite your experience,
provide some basics for me:
Operating system: macos
Python version: Python 3.6.4
Black version: 18.4a0
Does also happen on master: yes
This file fails:
def main():
    if True:
        showconfig()
        sys.exit(42)  # a comment
                      # a hanging comment

This is the error:
error: cannot format /Users/chrisros/tmp/failure.py: Cannot parse: 5:0:                       # another

Thanks for your report! Hm, I fixed a problem like this already. Those unaligned comments come back to haunt me! Will fix for the next release.

Well, I cannot repro with this file on master. It formats fine to:
def main():
    if True:
        showconfig()
        sys.exit(42)  # a comment
        # a hanging comment

Can you test with 18.5b0 instead of 18.4a0 which is super old?

I've confirmed what I said on Twitter; it turns out that the repro-with-head instructions need an extra step: rehash the shell PATH. I was still running the system black even though I thought I was using -dev.


Owl

Description
Create a new class to get ontology directly from github owl file and plug it with the rest of the code base
Motivation
Not to depend on bioportal, ontology can be downloaded and parsed directly from bioportal. To do that, it is enough to set the class of interest (e.g. GitHubOwlTechnique) in the config file, under the class field
Changes:
Following the commits should help review the changes.

technique-getter.js is now more generic. Each concrete class inherits from a base class which defines common methods
technique-getter.js got rid of some unused logic
technique-filter-builder.js can now instantiate different types of ontology parsers based on a config file
jsdom has been added as a dependency
utils.js defines a removeExtraSpaces method which removes extra spaces from string

Tests included/Docs Updated?

[x] Included for each change/fix?
[x] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?
[x] New packages used/requires npm install?
[x] Toggle added for new features?

We can merge this PR, as long as we include some documentation soon.

I will close this, as the whole logic has been moved to a new micro service


CI: Refactor GHA workflow files into corresponding subdirectories

About
Improve coherency between example snippets and corresponding validation jobs.

It looks like GitHub still does not support subfolders within the .github/workflows directory. Please upvote.

https://github.com/orgs/community/discussions/18055


Copy .dist files automatically on fresh install

The documentation explains that .dist files need to be copied in order to get Sulu working. This should be automated in a post-install/update hook instead. Users who install Sulu for the first time shouldn't need to care about this.

Made a PR https://github.com/sulu-io/sulu/issues/2123

Fixed with the introduction of sulu-minimal.


ci: fix failing Test (CGroup V1)

It appears that now the default remote images is https://images.lxd.canonical.com.
I don't know why, but we can't remove images.
The following command does nothing, so this step fails.
lxc remote remove images

Test on my sample repository.
https://github.com/daipom/test-lxd-on-github-actions/blob/main/.github/workflows/test.yml
https://github.com/daipom/test-lxd-on-github-actions/actions/runs/10484877405/job/29039978764#step:3:9

CI has passed.

I'm not sure but even though target LXD version (we are using) seems different, recently similar changes was introduced, IMHO.
https://discourse.ubuntu.com/t/lxd-4-0-10-lts-has-been-released/46687
https://discourse.ubuntu.com/t/new-lxd-image-server-available-images-lxd-canonical-com/43824

Thanks! I see!


creating captain's log menu

As a user,
I would like to see a summary of my searches
So that I can revisit them
Acceptance Criteria

Website must have a visible "Captain's Log" Menu
clicking ont he captain's log, presents the user with past "missions"
clicking on a mission displays the mission images

NOTE: "mission" is defined as previous search and the resulting images form that search

done


[Bug report]: Disabled dropdown can still be opened

Requirements before reporting

[X] No duplicated issue reported.
[X] I have checked the latest version if the bug exist there. See all available packages at npmJS.com
[X] I have followed the installation guide.

Package versions
<EMAIL_ADDRESS>Browser
Chrome
Framework
Plain HTML
Version
No response
Reproduction steps

Go to storybook
Go to Dropdown
Disable it using the controls
Click the arrow icon
The dropdown still opens.

Code example

Screenshots
https://github.com/scania-digital-design-system/tegel/assets/62651103/519c09c8-8298-497b-a5ab-e9ce96e0536b
Expected behaviour
The dropdown should never open when it is disabled.
Console errors

Contact information
No response

Hey @mJarsater! Tried to recreate this one both locally and https://tds-storybook.tegel.scania.com/?path=/docs/components-dropdown--default, but could not? Emelie failed to recreate it also.

Sorry, missed to highlight that this is for the filter version:
https://tds-storybook.tegel.scania.com/?path=/story/components-dropdown--default&args=filter:true;disabled:true&globals=backgrounds.value:!hex(F9FAFB)
https://github.com/scania-digital-design-system/tegel/assets/62651103/8bfd75a1-5892-40a2-b67a-51c633178152

@timrombergjakobsson
Sorry, missed to highlight that this is for the filter version: https://tds-storybook.tegel.scania.com/?path=/story/components-dropdown--default&args=filter:true;disabled:true&globals=backgrounds.value:!hex(F9FAFB)
Screen.Recording.2024-01-18.at.12.33.02.mov

Ah ok! Nice find man!


⚠ Bofrost.de has degraded performance

In d52cabc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1061 ms

Resolved: Bofrost.de performance has improved in 7356558 after 9 minutes.


(18.1 dev 5) Domain name cannot contain a slash (MBErrorDomain/205)

EDIT : Nevermind, I think that's because apple patched it. You can't enable it anymore on iOS 18.1 Beta 5
Step 2:
Patching your iPhones Eligibility.plist file! Please plug it in now to continue and hit Trust this computer when prompted
Press any key to continue...
iOS 18.1 (22B5054e)

An error occurred!
Traceback (most recent call last):
  File "/Users/joffrey/Nextcloud/Developer/iphone-mirroring-eu-activate/eligibility.py", line 122, in main
    cli(standalone_mode=False)
  File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/cli/cli_common.py", line 150, in wrap_callback_calling
    callback(service_provider=service_provider, **kwargs)
  File "/opt/homebrew/lib/python3.12/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/joffrey/Nextcloud/Developer/iphone-mirroring-eu-activate/eligibility.py", line 110, in cli
    raise e
  File "/Users/joffrey/Nextcloud/Developer/iphone-mirroring-eu-activate/eligibility.py", line 103, in cli
    perform_restore(back, reboot=False)
  File "/Users/joffrey/Nextcloud/Developer/iphone-mirroring-eu-activate/sparserestore/__init__.py", line 17, in perform_restore
    mb.restore(backup_dir, system=True, reboot=False, copy=False, source=".")
  File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/services/mobilebackup2.py", line 140, in restore
    dl.dl_loop(progress_callback)
  File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/services/device_link.py", line 61, in dl_loop
    raise PyMobileDevice3Exception(f'Device link error: {message[1]}')
pymobiledevice3.exceptions.PyMobileDevice3Exception: Device link error: {'ErrorCode': 205, 'ErrorDescription': 'Domain name cannot contain a slash (MBErrorDomain/205)', 'MessageName': 'Response'}

The same...
Ios 18.1 beta 5
Press any key to continue...iOS 18.1 (22B5054e) An error occurred! Traceback (most recent call last):   File "/Users/gidro/iphone-mirroring-eu-activate/eligibility.py", line 122, in main     cli(standalone_mode=False)   File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1157, in __call__     return self.main(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1078, in main     rv = self.invoke(ctx)          ^^^^^^^^^^^^^^^^   File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 1434, in invoke     return ctx.invoke(self.callback, **ctx.params)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/opt/homebrew/lib/python3.12/site-packages/click/core.py", line 783, in invoke     return __callback(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/cli/cli_common.py", line 150, in wrap_callback_calling     callback(service_provider=service_provider, **kwargs)   File "/opt/homebrew/lib/python3.12/site-packages/click/decorators.py", line 33, in new_func     return f(get_current_context(), *args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Users/gidro/iphone-mirroring-eu-activate/eligibility.py", line 110, in cli     raise e   File "/Users/gidro/iphone-mirroring-eu-activate/eligibility.py", line 103, in cli     perform_restore(back, reboot=False)   File "/Users/gidro/iphone-mirroring-eu-activate/sparserestore/__init__.py", line 17, in perform_restore     mb.restore(backup_dir, system=True, reboot=False, copy=False, source=".")   File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/services/mobilebackup2.py", line 140, in restore     dl.dl_loop(progress_callback)   File "/opt/homebrew/lib/python3.12/site-packages/pymobiledevice3/services/device_link.py", line 61, in dl_loop     raise PyMobileDevice3Exception(f'Device link error: {message[1]}') pymobiledevice3.exceptions.PyMobileDevice3Exception: Device link error: {'ErrorCode': 205, 'ErrorDescription': 'Domain name cannot contain a slash (MBErrorDomain/205)', 'MessageName': 'Response'}

Apple patched the exploit in iOS 18 Beta 5.

You can always restore to beta 4 https://ipswbeta.dev/ios/18.x

do you know it works at official version 18.0?

I mean, the README specifically states it requires 18.1, so I don't think so

I'll wait until the weekend for an update to this script, which might solve the problem. If not, I'll be rolling back the iPhone.

well there's no patch coming in soon it's patched by Apple

Wel maybe there's hope : https://x.com/misakaSoft/status/1838638290246996264

Not good news for now : https://x.com/Little_34306/status/1839078296165953567
:/

very bad :(

After rolling back to beta 4, I can tell you it's not worth it. The permission reset itself every hour, and it'll tell you that it's not available in your region, and you'll have to connect by cable your phone again and run the command.

So shitty...


docs: add Apple Pay Setup guides

Description
Adds guides to help users setup their Merchant's Apple Pay

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


HeadDrops pvp/team kill flags

Moved to initial release, the flags are easily bypassable so I'm not sure if they're worth implementing


Add measurements of Lick indices

It will be nice to get measurements of lick indices/absorption line equivalent widths, etc from the spectra.

@biprateep how high priority is it for fastspecfit to measure Lick indices? There are a number of subtleties associated with these measurements (e.g., convolving the data to a specific instrumental resolution to bring the measurements onto the Lick system) that I'm not excited about doing in terms of the computational cost. I'm also not sure how many spectra will have enough signal-to-noise to yield a useful, e.g., H-delta_A measurement.
Could these measurements be done perhaps as an 'after-burner' on dedicated subsets of spectra / galaxies that satisfy certain criteria, including targeting criteria?
Thoughts?

@moustakas In short: No it is not a very high priority, you can skip it for the next release.
I am measuring lick indices as an afterburner using an external library. I added this feature request so that all the measurements I was using come from a consistent processing pipeline. One thing we can do is to provide a set of functions that people can use if they want to get Lick indices for their specific work.
I did plan to work on it, currently I am on a break from working on K+A galaxies as I am working on editing and finishing up a paper, will get back to it in a month or so.

Thanks, that sounds good. Is this the library you're using?
https://mfouesneau.github.io/docs/pyphot/licks.html
It looks awesome!

Yup, this is the one I am currently using. I have seen plenty of others use it and its pretty good!
There's this very recent library (PyLick) dedicated to Lick indices I saw on arxiv a few weeks back. Will check this one out sometime. https://gitlab.com/mmoresco/pylick/


Support transparency

Great plugin (we've used it at Keybase!), but it looks like it doesn't support swatches with transparency.

That's awesome you guys are finding it useful!
I have another issue open to support rgba colors. I still need to decide on a new file format to implement that. But, it is planned.

Was very happy to have found this plugin but without alpha channel it’s a lot of work to re-apply transparency onto colors.

@riddle this is in fact planned. But I have it covered in another issue: https://github.com/andrewfiorillo/sketch-palettes/issues/5

@cecileboucheron Sketch Palettes now supports transparent colors.


[Minor] Add Events banner for upcoming Linkedin Live event

Change Logs
Site changes
Impact
Site change to add event banner
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Tested locally!


[FIX] User Suggestions & Bug Fixes

19b7131 Fix KeyCloak API URL

#564

33a2693 Fix guest has config access

#590

e2b9c15 Fix collapsible content in multi-page support

#626

24e487c Fix layout and item size buttons

#629

9eda048Refactor make request in RSS widget

#632

2fe0110 Fix material-design-icons header in schema

#640

4287092 - Add option to hide seconds in clock widget

#644

9b33a6e Fix pageInfo not being read in router

#645

9b33a6e Fix startingView not honored

#646

7b39bde - Fix Status Check default

#651

8c43442 Add option to hide image in SportsScores Widget

#654

3417b46 Add Adventure-basic theme

#655
-Write docs for sub-items
#657

c04e80b Add Font-Awesome displaying as square to troubleshooting guide

#659

7e547e7 - Show expand / collapse in context menu

#660

064c644 - Only deploy new release when relevant files have changed

Code Quality Checklist (Please complete)

[ ] All changes are backwards compatible
[ ] All lint checks and tests are passing
[ ] There are no (new) build warnings or errors
[ ] (If a new config option is added) Attribute is outlined in the schema and documented
[ ] (If a new dependency is added) Package is essential, and has been checked out for security or performance
[ ] Bumps version, if new feature added

Remaining tasks:

[ ] Add docs, and update schema for sub-items

#657

[ ] Bump to 2.0.9 and update changelog


Missing Nested folder from explorer view.

Type: Bug
Having a folder with this structure:
ParentFolder
-->  NestedFolder
When opening a folder which contains a nested folder called "CVS" then the CVS folder is missing from the explorer window.
It only happends when the NestedFolder is called CVS Upper case and nothing more.

VS Code version: Code 1.76.2 (ee2b180d582a7f601fa6ecfdad8d9fd269ab1884, 2023-03-14T17:55:54.936Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz (8 x 2995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.71GB (3.04GB free)

Process Argv
. --crash-reporter-id 19c929b2-ea4f-4568-930b-f05f480f72b3

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

eml
lei
0.4.0

azure-pipelines
ms-
1.208.0

vscode-docker
ms-
1.24.0

csharp
ms-
1.25.4

vscode-kubernetes-tools
ms-
1.3.11

isort
ms-
2022.8.0

python
ms-
2023.4.1

vscode-pylance
ms-
2023.3.20

jupyter
ms-
2023.2.1200692131

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.282.0

remote-wsl
ms-
0.76.1

azure-account
ms-
0.11.3

powershell
ms-
2023.2.1

java
red
1.16.0

vscode-yaml
red
1.12.2

intellicode-api-usage-examples
Vis
0.2.7

vscodeintellicode
Vis
1.2.30

vscode-java-debug
vsc
0.49.0

vscode-java-dependency
vsc
0.21.1

vscode-java-pack
vsc
0.25.9

vscode-java-test
vsc
0.38.1

vscode-maven
vsc
0.40.4

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewvextcfv2:30681851
azure-dev_surveyone:30548225
vscccc:30610679
pyindex848:30662994
nodejswelcome1:30587005
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
vscrp:30673768
pythonsymbol12:30671437
vsctsb:30677850
azdwalk:30687957
pythonms35:30686772

Yes, I also tried to create the UPPERCASE letter folder and it disappears but it exists in the file explorer.
I also try to refresh the folder tree by clicking the refresh button.
I also try to restart my vs code but it didn't show the UPPERCASE Folder.
My Operating System is Linux Mint
Using The Latest Version Of VS Code

Open the Settings Editor from Command Palette or via the gearwheel menu. Search for files.exclude and remove the default entry **/CVS

@gjsjohnmurray That did the trick, thank you.
I was wondering why **/CVS is a default value on that list. I am familiar with the other default values but no for CVS.


Setting up Mail

I want to suggest a better info (wiki) on setting up mail to work with ticketit, the default laravel doc referred to isn't detailed enough and that seems confusing. I still haven't figured it out -_- its returning
Trying to get property of non-object (View:
/var/www/laravel/ticket/vendor/kordy/ticketit/src/Views/emails/assigned.blade.php)

Which email provider and method do you use?

I have the same issue. I use smtp. Such error occurs when I try to add new ticket.

<EMAIL_ADDRESS>

@melihovgv Try this tutorial

I have the same issue. I can send email from my one of my routes.

And it arrives

But when I want to add new ticket it returns next

check that you have values for each status priority category and someone is assigned to that category

I add status, priorities, categories, agents and it works now. I think we may close issue. Thanks @andylord56.


feat: add product behavior api validation

https://github.com/raystack/proton/pull/338
fix missing Trait in Invite Mail Body

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5987

Relevant Lines:
14734

💛  - Coveralls


[IDE] Inspect all equivalent type variables to replace a type variable by an archetype

Sometimes the type variable itself doesn't have have an originator that can be replaced by an archetype but one of its equivalent type variable does. Search thorough all equivalent type variables, looking for one that can be replaced by a generic parameter.

@swift-ci Please smoke test

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test macOS


SwiftLint was integrated - Arrangements were made

@microsoft I've completed the SwifLint integration. I added the rules that I thought were necessary. I figured out the warnings.

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

@mustafagunes really appreciated your PR. I'm sorry that we can't merge this as we will soon merge an Outlook Mobile lint style, but anyway, great thanks!


Out of spec typo on clientDataJson -> clientDataJSON

Specs on https://www.w3.org/TR/webauthn/
use the clientDataJSON not the clientDataJson
See change in file.

This is in the demo usernameless.register.js

Thanks, Fixed in ee52d89a57b6d96


Gradle upgrade superset including Jenkins work and more!

I've built this primarily on #3806 with great thanks to @DarkWeird !
It finishes some of the more quirky parts of the overall logistics, does other minor stuff, enhances support for testing things outside our "live" environment in Artifactory, Jenkins, and on GitHub, updates to use Jenkins pipelines properly and the Jenkins Templating Engine (for modules). Also pulled in a couple other PRs and tested other stuff yet.
I believe this is merge-ready now and have new build jobs in the CBC Jenkins queued up to then start building everything fresh and neat in the new Jenkins, including a working Omega zip pulling in modules from new GitHub org mega-jobs automatically created by scanning for repos on GitHub - no more manually jobs in Jenkins and PRs will get tested for modules as well as the engine!
There are still improvements to make before we can use the new system for releases, primarily the approach remains hard coded to use the engine develop branch and snapshot versions for modules (much like before). Some tweaking needs to happen like consider develop a producer of snapshots and master a producer of released artifacts.. Not hard, just more work to do.
Hard to directly test some of this since Jenkins, but see #3806 for some of the central details and try to find any sort of Gradle task that appears broken. It is quite possible there could still be edge cases related to more rarely used stuff or rare combos involving user environments.

Merges #3806 (which has links to more associated PRs)
Merges #3833 (tested with https://github.com/Terasology/GooeyDefence/pull/36)
Merges #3814 (which in turn includes https://github.com/MovingBlocks/TeraConfig/pull/8)
Involved updates to https://github.com/MovingBlocks/ModuleJteConfig (module mega-jobs)
Involves https://github.com/MovingBlocks/FacadeServer/pull/30 although that may need more testing
Bumps version for engine and its embedded modules to 3.0.0 - this will require a bunch of module version bumps that'll be easiest to do in a big refactor done while merging this in a mega-workspace.
Maintains the bundled version data used in the main menu and as part of the launcher (I think?) but can't yet test the launcher with new URLs as the CBC Jenkins still isn't exposed publicly (and has more complicated URLs in any case)
Opens up properly attaching module build steps for maintaining the module browsing site (hi @majordwarf!) since those details can now be added to the JTE template Jenkinsfile.

Hooray Jenkins reported success with all tests good!


Fail to paste ADLS Gen2 snapshot to another ADLS Gen2 blob container

Storage Explorer Version: 1.14.1
Build: 20200710.5
Branch: hotfix/1.14.1
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ macOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Go to 'Settings' -> 'Services' -> 'Storage Accounts' -> Enable the setting 'Enable ADLS Gen2 snapshots (preview)' -> Restart Storage Explorer.
Expand one ADLS Gen2 storage account -> Blob Containers.
Create one blob container -> Upload one blob to it.
Right click the blob -> Click 'Manage Snapshots' -> Copy the blob under snapshots view.
Paste it to another ADLS Gen2 blob container -> Check the result.

Expect Experience:
Succeed to paste.
Actual Experience:
Fail to paste and show the below failed activity log.

More Info:

This issue also reproduces when pasting the blob to the same ADLS Gen2 blob container.
When trying to paste the blob to regular blob containers. The behavior as below:
(1). Show a successful activity log.
(2). The pasted blob doesn't display on the destination blob container's editor.

This issue is fixed on master build 20200722.6. So we close it.


added “ios.json” to the available formats

I have added "ios.json" to the list of available formats

That's looking great, thanks a lot for your contribution!


Create Readme.md

Current Readme is the default init file. Add details about site.

This should align with the About/Help page


add create and read tests for tpr-backed storage

Follows up on #699
Partially overlaps #714-- whichever is merged second will require a rebase.
Cliff notes:

Adds more create tests and new get/list tests to the tpr-backed storage implementation
To make these pass, the tpr-backed storage implementation was amended to check for the existence / non-existence of a resource before creating or retrieving... and handle appropriately. (i.e. handle consistently with the etcd3-backed storage implementation)
There has been a major revelation that the inputs and outputs of the storage layer should be internal / unversioned resources. There's a lot of refactoring of the tests included herein (all in commit 8368ad4f2fcd198f2f522cda62233aa6712fbda7) to account for this.
Update and delete tests are deliberately omitted from this PR because:

This PR is already big enough
I want this additional test coverage sooner, rather than later
Update and delete have known issues. Correcting those issues is probably enough work that I'd like to tackle them separately from the rest of this.

Addresses #700, but does not complete it


Add @douph1 to the installer team

@douph1 has been particularly useful helping us setup the MSI installers. Would be great to get him invited to the org so he can contribute further.
Can I get an approval please @AdoptOpenJDK/getopenjdk

Approved

invited and onboarded


Error: Can't open file: /data/local/tmp/detox/Application.apk while running in github actions

What happened?
While running the release variant of the detox android configuration, the tests pass successfully. However, when running in CI (github actions in this case), tests fail at the detox.init() stage.
What was the expected behaviour?
Tests would execute and pass in CI the same as they would locally.
Was it tested on latest Detox?

[X] I have tested this issue on the latest Detox release and it still reproduces.

Did your test throw out a timeout?

[X] I have followed the instructions under Identifying which synchronization mechanism causes us to wait too much.

Help us reproduce this issue!
No response
In what environment did this happen?
Detox version: 19.6.2
React Native version: 0.67.2
Node version: v14.17.0
Device model: NEXUS_5X_API_30
Android version: system-images;android-30;google_apis;x86_64
Test-runner (select one): jest-circus
Java: OpenJDK Runtime Environment Temurin-<IP_ADDRESS>+1 (build <IP_ADDRESS>+1)
Detox logs
Detox logs
trace level logging shows the hang that occurs at the detox.init() step
detox[4636] DEBUG: [DEVICE_LOOKUP] Found a matching & free device emulator-5554
detox[4636] DEBUG: [ALLOCATE_DEVICE] Settled on emulator-5554
detox[4636] DEBUG: [EXEC_CMD, #3] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "getprop dev.bootcomplete"
detox[4636] DEBUG: [EXEC_CMD, #4] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "getprop ro.build.version.sdk"
detox[4636] DEBUG: [EXEC_CMD, #5] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "settings put global animator_duration_scale 0"
detox[4636] DEBUG: [EXEC_CMD, #6] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "settings put global window_animation_scale 0"
detox[4636] DEBUG: [EXEC_CMD, #7] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "settings put global transition_animation_scale 0"
detox[4636] DEBUG: [EXEC_CMD, #8] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "dumpsys power | grep \"^[ ]*m[UW].*=\""
detox[4636] DEBUG: [EXEC_CMD, #9] "/Users/runner/Library/Android/sdk/build-tools/32.0.0/aapt" dump badging "/Users/runner/work/testproject/testproject/TestNative/android/app/build/outputs/apk/testapp/release/app-testapp-release.apk"
detox[4636] DEBUG: [EXEC_CMD, #10] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "am force-stop com.testproject.test"
detox[4636] DEBUG: [EXEC_CMD, #11] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm list packages com.testproject.test"
detox[4636] DEBUG: [EXEC_CMD, #12] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm list packages com.testproject.test"
detox[4636] DEBUG: [EXEC_CMD, #13] "/Users/runner/Library/Android/sdk/build-tools/32.0.0/aapt" dump xmlstrings "/Users/runner/work/testproject/testproject/TestNative/android/app/build/outputs/apk/testapp/release/app-testapp-release.apk" AndroidManifest.xml
detox[4636] DEBUG: [EXEC_CMD, #14] "/Users/runner/Library/Android/sdk/build-tools/32.0.0/aapt" dump xmlstrings "/Users/runner/work/testproject/testproject/TestNative/android/app/build/outputs/apk/androidTest/testapp/release/app-testapp-release-androidTest.apk" AndroidManifest.xml
detox[4636] DEBUG: [EXEC_CMD, #15] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "rm -fr /data/local/tmp/detox"
detox[4636] DEBUG: [EXEC_CMD, #16] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "mkdir -p /data/local/tmp/detox"
detox[4636] DEBUG: [EXEC_CMD, #17] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 push "/Users/runner/work/testproject/testproject/TestNative/android/app/build/outputs/apk/testapp/release/app-testapp-release.apk" "/data/local/tmp/detox/Application.apk"
detox[4636] DEBUG: [EXEC_CMD, #18] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm install -r -g -t /data/local/tmp/detox/Application.apk"
detox[4636] ERROR: Exceeded timeout of 300000ms while handling jest-circus "setup" event

Further inspection of the logs artifacts produced by detox running in CI reveal the failure occurs at the step of pm install of the Application.apk from the tmp directory
detox[4636] DEBUG: [EXEC_CMD, #41] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "rm -fr /data/local/tmp/detox"
detox[4636] TRACE: [EXEC_SUCCESS, #41]
detox[4636] DEBUG: [EXEC_CMD, #42] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "mkdir -p /data/local/tmp/detox"
detox[4636] TRACE: [EXEC_SUCCESS, #42]
detox[4636] DEBUG: [EXEC_CMD, #43] "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 push "/Users/runner/work/testproject/testproject/TestNative/android/app/build/outputs/apk/testapp/release/app-testapp-release.apk" "/data/local/tmp/detox/Application.apk"
detox[4636] ERROR: [EXEC_FAIL, #18] ""/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm install -r -g -t /data/local/tmp/detox/Application.apk"" failed with error = ChildProcessError: Command failed: "/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm install -r -g -t /data/local/tmp/detox/Application.apk"
Error: Unable to open file: /data/local/tmp/detox/Application.apk
Error: Can't open file: /data/local/tmp/detox/Application.apk

Exception occurred while executing 'install':
java.lang.IllegalArgumentException: Error: Can't open file: /data/local/tmp/detox/Application.apk
	at com.android.server.pm.PackageManagerShellCommand.setParamsSize(PackageManagerShellCommand.java:519)
	at com.android.server.pm.PackageManagerShellCommand.doRunInstall(PackageManagerShellCommand.java:1282)
	at com.android.server.pm.PackageManagerShellCommand.runInstall(PackageManagerShellCommand.java:1248)
	at com.android.server.pm.PackageManagerShellCommand.onCommand(PackageManagerShellCommand.java:184)
	at android.os.BasicShellCommandHandler.exec(BasicShellCommandHandler.java:98)
	at android.os.ShellCommand.exec(ShellCommand.java:44)
	at com.android.server.pm.PackageManagerService.onShellCommand(PackageManagerService.java:21646)
	at android.os.Binder.shellCommand(Binder.java:929)
	at android.os.Binder.onTransact(Binder.java:813)
	at android.content.pm.IPackageManager$Stub.onTransact(IPackageManager.java:4603)
	at com.android.server.pm.PackageManagerService.onTransact(PackageManagerService.java:4305)
	at android.os.Binder.execTransactInternal(Binder.java:1159)
	at android.os.Binder.execTransact(Binder.java:1123)
 `"/Users/runner/Library/Android/sdk/platform-tools/adb" -s emulator-5554 shell "pm install -r -g -t /data/local/tmp/detox/Application.apk"` (exited with error code 255) (code=255), stdout and stderr:

Device logs
Device logs
I have the device logs but they do not show any detox-related logging. I could product them if necessary (they are quite long) but having combed through them, I don't see anything like the `java.lang.IllegalArgumentException` pasted above, which leads me to believe the error throws in the java running adb on the host machine and not within the emulator printing out to logcat

More data, please!
Build step succeeds without issue. Emulator is started with command:
          echo "Starting emulator"
          nohup $ANDROID_HOME/emulator/emulator -avd NEXUS_5X_API_30 -no-boot-anim -camera-back none -camera-front none -memory 2048 -timezone 'US/Arizona' &
          $ANDROID_HOME/platform-tools/adb wait-for-device shell 'while [[ -z $(getprop sys.boot_completed | tr -d '\r') ]]; do sleep 1; done'

and the output of the step does indicate it was successful in starting.
The possible explanations for the error that I can think of are:

There is a problem with root permissions for the runner in github actions, but I haven't been able to find another issue in the Detox repo similar to this.
The previous step to push the release apk to /data/local/tmp/detox/Application.apk failed even though it says it succeeded
The Application.apk is actually corrupt and the build step is succeeding despite this corruption

I think I can rule out permissions as a possibility
After following the steps here:
https://stackoverflow.com/a/36266874/16318310
I added a step to the github action workflow and the returning permissions looked good:
/Users/runner/Library/Android/sdk/platform-tools/adb shell ls -ld /data/ /data/local/ /data/local/tmp/
drwxrwx--x 47 system system 409[6](https://github.com/synapsestudios/tivity-commercial-product/runs/5889729687?check_suite_focus=true#step:18:6) 2022-04-0[8](https://github.com/synapsestudios/tivity-commercial-product/runs/5889729687?check_suite_focus=true#step:18:8) 11:38 /data/
drwxr-x--x  4 root   root   4096 2022-04-08 11:19 /data/local/
drwxrwx--x  2 shell  shell  4096 2022-04-08 11:19 /data/local/tmp/

@d4vidi interesting perspective. Looks like a workaround with adb push and consequent installation via shell has its own glitches sometimes.

@zacharyweidenbach I'm having the same issue as you, but on Microsoft App Center.
Would be able to share the Github Step you added please?

@paulclarkin
My steps are nearly identical to the react-native-firebase steps, with the exception that I was building the release version of the app for my tests instead of prefetching the bundled JS and running the debug build.
https://github.com/invertase/react-native-firebase/blob/main/.github/workflows/tests_e2e_android.yml


GraphQL Transformer not providing $condition input for update mutation

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
I've seen in many examples (e.g. https://github.com/dantasfiles/ExploreAmplifyAPI) where $condition input (in the form of Model{name}ConditionInput) is generated onto the create, update, and delete mutations (presumably via codegen). However, I'm not seeing this in my implementation:
To Reproduce
schema.graphql:
type Order @model
  @key (name: "byQueue", fields: ["queue"], queryField: "ordersByQueue")
  @auth (rules: [
      { allow: private, provider: iam, operations: [create, read, update, delete] },
      { allow: groups, groups: ["admin", "support"], operations: [create, read, update, delete] },
    ]
  ) {
   id: ID!
   restaurantID: ID!
   order: AWSJSON
   executionArn: String
   queue: String
   queStatus: String
   dispOps: AWSJSON
   abortDispatch: Boolean
   isSimDispatch: Boolean
   restaurant: Restaurant @connection(fields: ["restaurantID"])
   meta: AWSJSON
}

Grahpql transformer output (mutation.js):
export const updateOrder = /* GraphQL */
  mutation UpdateOrder($input: UpdateOrderInput!) {
    updateOrder(input: $input) {
      id
      restaurantID
     ...
   }
 }

Expected mutation:
export const updateOrder = /* GraphQL */ `
  mutation UpdateOrder($input: UpdateOrderInput!, $condition: ModelOrderConditionInput) {
    updateOrder(input: $input, condition: $condition) {
      id
      restaurantID
     ...
   }
 }

NOTE: I get around this by manually adding $condition: ModelOrderFilterInput to the schema in AWS console after every amplify push that updates the API and works but obviously not an ideal solution...
Additional environment details

OS version:
macOS Mojave 10.14.6
Output of amplify --version:
5.0.0

Did this work in previous versions?:
No
Additional Context:
Initially, my .graphqlconfig.yml was:
projects:
  wl1:
    schemaPath: amplify/backend/api/wl1/build/schema.graphql
    includes:
      - src/graphql/**/*.js
    excludes:
      - ./amplify/**
    extensions:
      amplify:
        codeGenTarget: javascript
        generatedFileName: ''
        docsFilePath: src/graphql
        maxDepth: 6
extensions:
  amplify:
    version: 3

Subsequently, I manually updated the yml to amplify: version: 5  but still get the same results.

Hi, any update or suggestions on this? I see that 'doc-gen' label was add to this issue -- does that mean it's simply a documentation issue and not a bug? How can I get codegen to generate the $condition input on mutations? Any reply would be appreciated - thanks!

Hi @djwood I cannot seem to reproduce this issue. Are you using plain JS or any JS framework like angular, react etc.?

Hi Srikar,
I'm using React.
Just to clarify, are you saying $condition input should be automatically
generated by the transformer? Note that none of the generated mutations
in the schema has the condition input exposed. It also seems from this doc
https://docs.amplify.aws/cli/graphql-transformer/model/#generates that
conditional inputs are not generated for any of the mutations -- only
filter inputs for list* queries.
Appreciate your help,
Dan
On Mon, Sep 20, 2021 at 1:23 PM Phani Srikar Edupuganti <
@.***> wrote:

Hi @djwood https://github.com/djwood I cannot seem to reproduce this
issue. Are you using plain JS or any JS framework like angular, react etc.?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-codegen/issues/230#issuecomment-923268146,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGVNXBGE3DH3O5VKHBQQMLUC6J25ANCNFSM5CPGY2MA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@djwood Thanks for reporting the issue. I cannot reproduce the issue(my version is v6.3.0) but do notice there is a difference for the documentation. Just ping CLI team on this to check if this is a feature added recently.
Meanwhile, could you install the latest CLI to check if your problem still exists? Since your reported version is 5.0.0, the issue is probably due to the outdated version.


TypeScript compilation no longer works after upgrade to TypeScript 2.0.3

Hi guys,
I have recently upgraded to TypeScript 2.0.3 to get past a problem with the firebase nativescript plugin.
Everything is compiling and building but my javascript files are no longer generated.
Maybe it is due to my lack of knowledge about the matter but I have not been able to find any help through google or anywhere else.
Following is my package.json dependecies:
{
  .
  .
  .
  "dependencies": {
    "@angular/common": "2.0.0-rc.5",
    "@angular/compiler": "2.0.0-rc.5",
    "@angular/core": "2.0.0-rc.5",
    "@angular/http": "2.0.0-rc.5",
    "@angular/platform-browser": "2.0.0-rc.5",
    "@angular/platform-browser-dynamic": "2.0.0-rc.5",
    "@angular/platform-server": "2.0.0-rc.5",
    "@angular/router": "3.0.0-rc.1",
    "email-validator": "1.0.4",
    "moment": "2.13.0",
    "nativescript-angular": "0.4.0",
    "nativescript-iqkeyboardmanager": "1.0.1",
    "nativescript-loading-indicator": "^2.1.0",
    "nativescript-plugin-firebase": "^3.7.1",
    "nativescript-timedatepicker": "^0.2.2",
    "nativescript-unit-test-runner": "0.3.3",
    "nativescript-web-image-cache": "^2.0.1",
    "rxjs": "5.0.0-beta.6",
    "tns-core-modules": "^2.3.0",
    "zone.js": "0.6.12"
  },
  "devDependencies": {
    "babel-traverse": "6.8.0",
    "babel-types": "6.8.1",
    "babylon": "6.8.0",
    "filewalker": "0.1.2",
    "jasmine-core": "^2.4.1",
    "karma": "^0.13.22",
    "karma-jasmine": "^1.0.2",
    "karma-nativescript-launcher": "^0.4.0",
    "lazy": "1.0.11",
    "nativescript-dev-typescript": "^0.3.2",
    "nativescript-dev-webpack": "0.0.20",
    "typescript": "^2.0.0"
  }
}

And this is my tsconfig.json:
{
    "compilerOptions": {
        "outDir": "app",
        "rootDir": "app",
        "module": "commonjs",
        "target": "es5",
        "inlineSourceMap": true,
        "experimentalDecorators": true,
        "removeComments": false,
        "emitDecoratorMetadata": true,
        "noEmitOnError": true
    },
    "exclude": [
        "node_modules",
        "platforms"
    ]
}

Also, I have made sure that I have run npm install -g typescript and tsc --version gives me the correct version.
Again, everything compiles:
Executing before-prepare hook from C:\Users\Brian\Desktop\Development\student-application\hooks\before-prepare\nativescript-dev-typescript.js
Found peer TypeScript 2.0.3
Project successfully prepared (android)

Any help?

I believe I can help. I had the same issue. I was able to fix it by updating my tsconfig.json to be the following:
{
    "compilerOptions": {
        "target": "es5",
        "module": "commonjs",
        "declaration": false,
        "removeComments": true,
        "noLib": false,
        "emitDecoratorMetadata": true,
        "experimentalDecorators": true,
        "lib": [
            "dom"
        ],
        "inlineSourceMap": true,
        "pretty": true,
        "allowUnreachableCode": false,
        "allowUnusedLabels": false,
        "noImplicitAny": false,
        "noImplicitReturns": true,
        "noImplicitUseStrict": false,
        "noFallthroughCasesInSwitch": true,
        "typeRoots": [
            "node_modules/@types",
            "node_modules"
        ],
        "types": [
        ]
    },
    "exclude": [
        "node_modules",
        "platforms"
    ],
    "compileOnSave": false
}

And I was pointed in the right direction by looking at these:

https://github.com/NathanWalker/angular-seed-advanced/blob/master/nativescript/tsconfig.json
https://github.com/NathanWalker/angular-seed-advanced/blob/master/nativescript/package.json

HI @mcordes,
Thanks a lot for your answer.
It indeed build generates .js files with your tsconfig file. After switching my application simply crashes on start up without any error messages except for the "has stopped working" on android.
Also, I am getting the following errors on build (but it still builds successfully):
Found peer TypeScript 2.0.3
error TS2318: Cannot find global type 'Boolean'.

error TS2318: Cannot find global type 'Function'.

error TS2318: Cannot find global type 'IArguments'.

error TS2318: Cannot find global type 'Number'.

error TS2318: Cannot find global type 'Object'.

error TS2318: Cannot find global type 'RegExp'.

error TS2318: Cannot find global type 'String'.

Does that tell you anything?

Hey @blaur
I see you are using quite old version of nativescript-angular and @angular.
Have you tried updating you project to the current official versions:

1.1.1 for nativescript-angular
2.1.1 for @angular packages
3.1.1 for @angular/router

Hi @vakrilov
I have not tried because it introdues a lot of refactoring in bascially everythng but it seems like I have no choice.
One wonders me is that the build is still successful.

I'm and currently using version 2.3.0 of the NativeScript CLI (a.k.a. tns) and the the run/livesync commands fail whenever there are compile errors - indeed this is the expected behavior.

I am not really sure how that helps me.
I am getting the above errors but the run commands do not fail.

It disappeared after upgrading to newest version 2.5 of nativescript and 2.1.6 of TypeScript.


Missing info regarding how ValueName field can be used

An important piece of information that should be explicitly documented is what format the markup should take if you are specifying a default key value (i.e. a value that in .reg format would be named '@'). I ASSUME that I should just leave the ValueName attribute off and provide a Value, but I wonder if instead I should use ValueName="@" or ValueName="" (I assume not the latter because the specification says it must begin and end with a non-whitespace character, and this does neither). Anyway, it's ambiguous.

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

ID: fb781ee5-9428-073e-2227-179cd2384503
Version Independent ID: 865d9dca-6230-5b65-bc71-abe01c8c84e8
Content: rescap4:ClassicAppCompatKey - UWP app developer
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-rescap4-ClassicAppCompatKey.md
Product: windows
GitHub Login: @laurenhughes
Microsoft Alias: lahugh

Hi - Thanks for taking the time to let us know about this. Due to the sheer volume of feedback in our repo, we're currently prioritizing new issues. I'm going to close this but if it's still impacting you or others, feel free to leave a comment and we can keep it open and dig in. Alternatively, you can submit a new issue or pull request with the latest details. Thanks so much and apologies for the delayed communication here.


Plot of dN/dS along the genome

Interesting to look at easy signals of selection along the genome?

It would be nice to be able to pick an internal sample node and get a genome-wide dN/dS profile for itself and its descendants. This would allow us to compare dN/dS profiles between clades easily (or between a clade and the rest of the samples in an ARG).


Add Kubernetes native authentication flow

This PR is still WIP

This change has now been tested in dev and is now ready for review


Reflected XSS in terminal

Hey,
I found that this website uses a terminal module that allows users to echo text in the terminal. Unfortunately this terminal does not sanitize user input.
If a user enters the following command: echo <svg><svg onload=alert(0)>, a Javascript alert pops up on the screen as shown below.

This is obviously a non-priority issue because it is not "stored". But a good idea would be to sanitize all user input.
Hope this helps!

Oh, i didn't think it through,
I'll convert that soon!


Reload when clicking a link of current url

It gets ignores for now, but we have to perform the equivalent of location.reload()

getInitialProps
trigger router events
possibly unmounting / remounting

Is this the exact same URL?
Then the routes should be idempotent.
There will be definitely an other way to do what we are doing.

The idea is to emulate the browser behavior by @rauchg . I think it makes sense since you might want to recall getInitialProps and reset state even if it's a same route.

Okay. Then it's about removing these lines.
@nkzawa That's it right?

@arunoda I think so.

@nkzawa let me work on it.


Figure out how to express Lineage non-emptiness constraint

This problem has existed since the beginning with Thema, but working on #82 has made it especially clear: it is important to constrain that valid lineages must contain at least one schema. However, using a simple constraint like this:
import "list"

#Lineage: {
	// ...

	// schemas is the ordered list of all schemas in the lineage. Each element is a
	// #SchemaDef.
	schemas: [...#SchemaDef] & list.MinItems(1)

	// ...
}

Makes referencing #Lineage to specify the expected type of an argument to a pseudofunc, like this:
#LatestVersion: {
	lin: #Lineage
	out: lin.schemas[len(lin.schemas)-1].version
}

Result in those pseudofuncs be in a constant state of invalidity:
#LatestVersion.lin.schemas: invalid value [] (does not satisfy list.MinItems(1)): len(list) < MinItems(1) (0 < 1):
    lineage.cue:67:7
    lineage.cue:63:18
    lineage.cue:67:21

This feels very much like an "i'm holding CUE wrong" problem. Should i not even be trying to specify arg types? Is there a way of separating the constraint check so that these pseudofuncs retain the correctness bound, but it no longer bubbles up in the same way?

One pattern you might consider is schemas: [#SchemaDef, ...#SchemaDef] which has the effect of requiring at least one element

yeah, that's the workaround we currently rely on. But that hits on the issue i frequently have with CUE - unification actually makes it hard to require a user to provide certain input.
One path i haven't explored, however, is having a default value for schemas that satisfies the MinItems(1) check, but is internally invalid in some way that effectively allows deferring these errors. That seems like the plausible path, here.


Simulator/UI: Print appropriate covariate labels in the regression output table

Problem
Covariate Labels are not available at the output to be printed into a table
Tasks
Need to modify coinstac_drne_fsl and coinstac_ssr_fsl repos

Done!!! Corrected coinstac_drne_fsl at https://github.com/MRN-Code/coinstac_drne_fsl/commit/93c165317cdf6d168e92e13e89cde8e3543ce1c5 and coinstac_ssr_fsl at https://github.com/MRN-Code/coinstac_ssr_fsl/commit/3e9d00022689ee0fb2ba04b0165b74ea0a3bca52


Prepare for upgrading of @build_bazel_rules_nodejs

What is the goal of this PR?
As migration to latest @build_bazel_rules_nodejs is planned, we need to adapt assemble_npm/deploy_npm to newest rules [while also preserving compatibility with currently-used version].
What are the changes implemented in this PR?

Use @nodejs//:npm (compatible with both old and new rules-nodejs)
Append external/nodejs_<platform>_amd64/bin/ to PATH when invoking npm. Adding /bin/ and /usr/bin is also required to have bash/dirname available (needed by npm script)

LGTM


Pocketmine

Support pocketmine??

Only as a proxy apart from the server altogether only nukkit

Plz in https://github.com/Trollhunters501/Barrel-CREA-Edition/discussions

theres a bug where when you leave the server you gain back the hearts you lost on top of the default hearts

If the next update would be fixed


add up to date nuke versions to cmake module

Description
Adds additional Nuke versions that the FindNuke cmake module can look for.
Checklist:

[x] I have read the contribution guidelines.
[ ] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[ ] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

LGTM, thanks, merging.


Allow group argument in "ifHasRole" signature

The "alanning:roles" package allows to check permission by group since version 1.1.0.
It would be great to be able to use the "userIsInRole(user, roles, group)" signature from meteor-security.
e.g:
Posts.permit('remove').ifHasRole('admin', 'company1.com').apply();

or even in array syntax:
Posts.permit('remove').ifHasRole(['admin', 'posts-admin'], 'company1.com').apply();

Implemented with object arg since we only support a single arg. See updated docs: https://github.com/ongoworks/meteor-security#built-in-rule-chain-methods


Ignore baseUrl with absolute URIs

When using a configured instance of http-client with a baseUrl, using the shortcut methods (e.g. get(url)) won't work with absolute URIs, where in many cases one should deal with absolute URIs from the same API (e.g. HTTP Link header fields).
The PR adds this support ignoring the baseUrl with absolute URIs, using a RegExp to have the match case-insensitive.

This sounds pretty reasonable. @bryanrsmith Please review when available. Thanks guys!

Thanks, @heruan! Great suggestion. Couple things...

It'd be great to use the regexp here: https://github.com/aurelia/history-browser/blob/master/src/index.js#L217, which accepts all valid schemes as well as scheme-relative URLs.
Would you mind updating your commit message to follow our contributing guidelines? Something like feat(http-client): ignore baseUrl when using absolute URLs

Done! Thank you @EisenbergEffect and @bryanrsmith :+1:


Nice to have: add configmap for dynamodb schema and periodic table configuration

Right now I need to copy carefully this bit (with my values):
- -s3.url=s3://abc:123@s3.default.svc.cluster.local:4569
- -dynamodb.url=dynamodb://user:pass@dynamodb.default.svc.cluster.local:8000/cortex
- -dynamodb.periodic-table.prefix=cortex_weekly_
- -dynamodb.periodic-table.start=2017-01-06
- -dynamodb.daily-buckets-from=2017-01-10
- -dynamodb.base64-buckets-from=2017-01-17
- -dynamodb.v4-schema-from=2017-02-05

between querier, ruler, table-manager and ingester. Would be nice to state that somewhere, in single place.

Yeah, we would support reading this from a file (which you could put in a config map).  At Weaveworks, we version all our YAMLs, and are working on a method for sharing config between these YAMLs.


update docker images usage for Falco 0.40

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind event

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area community

What this PR does / why we need it:
See https://github.com/falcosecurity/falco/issues/3165 and https://github.com/falcosecurity/falco/pull/3273
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/hold until Falco 0.40 is out

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ekoops: changing LGTM is restricted to collaborators

In response to this:

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.


Wasteful communications between instances?

Let me preface I have not started using cote yet - still in research phase before execution.
I opened this after looking into internals and I believe this is a source of @claustres's issue brought up in slack.
In a full mesh network of 100 instances I expect 200 messages per heartbeat interval (100 incoming, 100 outgoing)
Based on this:
https://github.com/dashersw/cote/blob/6b6d037f1cbad78417fb98bec4427d480d9dc2f9/src/components/component.js#L22
https://github.com/dashersw/cote/blob/955b85394ab246f3aced81f22f93ba07dece27d5/src/components/requester.js#L19
https://github.com/dashersw/cote/blob/c517ae2a74dfc56b2e0cb8b324362b4f2b8c9c08/src/components/responder.js#L14
https://github.com/dashersw/cote/blob/6b6d037f1cbad78417fb98bec4427d480d9dc2f9/src/components/subscriber.js#L13
https://github.com/dashersw/cote/blob/ac31387382e98a7bd22e37207a22fed08ba70496/src/components/publisher.js#L11
Each instance starts their own heartbeat process, so factor in say 10 requesters and 10 responders you now go to 20k messages per interval per process (2 million for the entire network per interval)
This is really inefficient.
Ideally, there should only be a single Discovery instance managed by the cote instance, and each requester/responder is tracked by the cote manager and the messages relayed accordingly.
This appears it would be an API breaking change if you consider .discovery public API (which seems like it might not be considered public API based on https://github.com/dashersw/cote/issues/99), but I believe this is critical to solving wasteful message processing.

I think there are some design choices here. It seems that cote components are not lightweight elements regarding discovery so that it's better to only have some per process. It appears that using namespaces/keys does not help as it enters the game after discovery. As a consequence, when having a finer granularity it's seems to be a good idea to map a set of finer elements to a single requester/responder or publisher/subscriber in cote. Segmentation has thus to be managed after routing. Of course this also depends on the use case.
Multicast might help to address the network overhead but the problem is that it does not work in most cloud environments and you have to rely on Redis in this case. Using multiple redis instances will probably help as well but at the cost of a more complex system configuration.

In the meantime, I have a pending PR to node-discover (upstream) that will reduce the cpu cost pain of all the additional packets:
https://github.com/wankdanker/node-discover/pull/40/files

In the meantime, I have a pending PR to node-discover (upstream) that will reduce the cpu cost pain of all the additional packets:
wankdanker/node-discover#39

Sorry to come in late to the discussion. @claustres you have summed it up quite well — that's the current scenario. I believe centralizing discovery per node as @aikar suggested earlier might be a viable approach to reduce network chatter at the cost of increasing hello package sizes, and the PR submitted above, if we have actual numbers on performance gains, might help with CPU processing.
Also in the meantime I recommend checking out https://github.com/dashersw/kotelett, which, admittedly, lacks documentation and is a heavy WIP, but it removes the need for a lot of this fuss —
intelligent routing based on message types removes the need for keys and segregation and multiple responders / requesters, as they are created dynamically for you. This also results in at most two discovery mechanisms per process because there's exactly one requester and one responder in each process (this 2 discovery instances could further be improved based on a similar approach above).
The code also becomes much more clear. there's only const kote = require('kotelett') and kote.send and kote.on. No instances, no requesters, or anything. Message configuration and routing is dynamic, and it still works for load balancing as well.
Obviously, it was an experiment, so it currently doesn't have pub/sub, but that's also very easy to implement.

FYI this line and maybe more will be an issue with my cache pr: https://github.com/dashersw/cote/blob/master/src/components/responder.js#L27
as it mutates the object directly instead of using the .advertisement() method.


Replace @Helper sections with local methods where possible

Is this related to #76 ?

Yes, whoops. These are duplicates.


steam fails to start on unstable

Issue description
$ steam --version
cp: reguläre Datei '/home/ronny/.local/share/Steam/bootstrap.tar.xz' kann nicht angelegt werden: Permission denied
/home/ronny/.local/share/Steam/steam.sh: Zeile 102: Keine Entsprechung: **:

Steps to reproduce
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Nix version: (run nix-env --version)
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

Could you delete /home/ronny/.local/share/Steam and then try again if you don't have anything important in that directory. Otherwise you might try fixing the owner of the file mentioned and the parent directories.

I just tested the one from the unstable channel and seems to work fine for me. Notice that the 'permission denied' error you're seeing is 'normal', it shouldn't prevent steam from running.
Was this a clean install? Did you try what @globin suggested?

also you can delete ~/.steam, dunno if that helps. i deleted both directories the few times steam broke.

For what it's worth, I'm experiencing the exact same thing as @RonnyPfannschmidt.

@clhodapp, @RonnyPfannschmidt can you give us more details? is this nixos? are you subscribed to steam beta? I suspect my steam.sh is not the same. Can you paste the offending line?

$ steam
cp: reguläre Datei '/home/ronny/.local/share/Steam/bootstrap.tar.xz' kann nicht angelegt werden: Permission denied
/home/ronny/.local/share/Steam/steam.sh: Zeile 102: Keine Entsprechung: **:

$ ls /home/ronny/.local/share/Steam -la
insgesamt 2656
drwx------ 1 ronny users     164 24. Mär 15:59 .
drwxr-xr-x 1 ronny users     860 24. Mär 15:59 ..
-r--r--r-- 1 ronny users 2676956 24. Mär 15:59 bootstrap.tar.xz
drwx------ 1 ronny users      36 17. Nov 2015  linux32
-rw------- 1 ronny users     826 26. Nov 2013  steamdeps.txt
-rw------- 1 ronny users    8822 26. Nov 2013  steam_install_agreement.txt
-rwx------ 1 ronny users   22532 17. Nov 2015  steam.sh
drwx------ 1 ronny users      66 17. Nov 2015  ubuntu12_32

line 102 is in
function show_license_agreement()
{
	LICENSE="$STEAMROOT/steam_install_agreement.txt"
	if [ ! -f "$STEAMCONFIG/steam_install_agreement.txt" ]; then
		if [ ! -f "$LICENSE" ]; then
			show_message --error $"Couldn't find Steam install license agreement, aborting!"
			exit 1
		fi

		# See if they have been grandfathered in through the beta
		SSAVersion=$(find -L "$STEAMCONFIG/`detect_package`" -name sharedconfig.vdf -exec fgrep SSAVersion {} \;)

		if [ "$SSAVersion" != "" ]; then
			answer=accepted
		else
			answer=declined
			output=$(zenity --width 650 --height 500 --text-info --title=$"Steam Install Agreement" --filename="$LICENSE" --checkbox=$"I have read and accept these terms." 2>&1)
			STATUS=$?
			if echo $output | grep "status 1:" >/dev/null; then  # this is line 102
				# Zenity couldn't launch a window
				STATUS=-1
			fi
			case $STATUS in
			0)	# The agreement was accepted
				answer=accepted
				;;
			1)	# The agreement was declined
				;;
			*)	# zenity wasn't available, try a fallback
				tmpfile="$(mktemp || echo "/tmp/steam_message.txt")"
				command="more \"$LICENSE\" || cat \"$LICENSE\"; echo -n $'Do you accept the terms of this agreement? [y/N]: '; read input; if [ x\$input = xy -o x\$input = xY ]; then echo accepted >\"$tmpfile\"; fi"
				xterm -bg "#383635" -fg "#d1cfcd" -T $"Steam Install Agreement" -e "$command" || \
					/bin/bash -c "$command"
				if [ -f "$tmpfile" ]; then
					read answer <"$tmpfile"
					rm "$tmpfile"
				fi
				;;
			esac
			if [ "$answer" != "accepted" ]; then
				exit 0
			fi
		fi

		cp "$LICENSE" "$STEAMCONFIG"/
	fi
}```

Can you try this patch: https://github.com/abbradar/nixpkgs/commit/c33fbe00312252c790475d1e47befeb1a8d02eff?

i just tested on master and in plasma5 (gnome3 is still broken according to @globin )
it starts up just fine

will try on 17.03 next

Sadly you won't be able to test now: Steam needs to show Zenity only once to ask about license or something like that and them marks somewhere that it was done.

i#ll just delete all its folders

works on the release-17.03 branch as well

Pushed to master; thank you very much for testing!

@abbradar i missed a grave mistake of mine, i did not actually apply your patch

:D Well, you can just try to run Zenity. If it works for you so will Steam.

zenity already runs for me apparently without that patch (currently running a release-17.03 from about 30 minutes back)

Hm, so you can't reproduce that bug now at all?

at least not on plasma5 (currently i cant run gnome3)

Well, Zenity needed that patch anyway. Let's close this now then; please reopen if you encounter that again! (and backup .steam, .local/share/steam and others in case you do).

btw, one striking change is, that steam no longer tries to generate a steam.desktop file


Handle Unstructured Points Case when points are unused

Description
Resolves #18716

I added in some logic to handle the unstructured points case where the topo does not use all of the points.

Type of change

[x] Bug fix~~
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?

built and ran on rztopaz. The old tests and my new test pass.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[x] I have commented my code where applicable.~~
[x] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[x] I have added tests that prove my fix is effective or that my feature works.~~
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[x] I have added new baselines for any new tests to the repo.~~
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

@JustinPrivitera  did we get this change on develop as well?

Nope. I'm on it after lunch.

Awesome, thanks!

Merge w/ develop in #19282