DID Exchange - Issue with implicit invitations

I'm experimenting with the DID Exchange protocol in order to create a connection based on a public DIDs implicit invitation.
A public DID allows to retrieve the necessary keys and service information from a given VDR, i.e. all information that is needed to pack and send the respective request message.
ACA-Py currently provides the following APIs:

create-request allows to create a connection request and returns it, but there seems to be no exposed functionality to actually send the request.
I tried a simple hack adding
 # Send request
        responder = self._session.inject(BaseResponder, required=False)
        if responder:
            await responder.send(request, connection_id=conn_rec.connection_id)

for testing, but fetch_connection_targets fails because there's no invitation attached to the connection.
In addition, I don't understand the use case for receive-request, since I'd say requests should use the DIDcomm channel. Everything else should use the Out of Band API.

Ok, what does work is using /out-of-band/receive-invitation with a payload like this:
{
  "@id": "3fa85f64-5717-4562-b3fc-2c963f66afa6",
  "handshake_protocols": [
    "did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/didexchange/1.0"
  ],
  "services": [
    "did:web:478eb39f6464.ngrok.io"
  ]
}

So this works similar to the connections API.

@domwoe while that works, sounds like the create-request approach should work.  Do you agree?  Or are we good with what you found did work?
I'm kind of with your initial comment -- that if you know a public DID with a DIDComm service block, you should be able to use it without having to use the OOB protocol.  However, not sure what complexities that might create within ACA-Py.

@domwoe Will this work for your use case?
https://github.com/hyperledger/aries-cloudagent-python/pull/1174


iOS 11 Support

Hello.
Has anyone tried LGAlertView on iOS 11 Beta 3? Alert buttons appear deformed in my project. May be someone has investigated height issues already?

Hi crazyjooe,
This worked for me.
index e6f1210..24aae57 100644
--- a/LGAlertView/LGAlertView.m
+++ b/LGAlertView/LGAlertView.m
@@ -2882,6 +2882,7 @@ LGAlertViewType;
                 [self.tableView registerClass:[LGAlertViewCell class] forCellReuseIdentifier:@"cell"];
                 self.tableView.frame = CGRectMake(0.0, 0.0, width, CGFLOAT_MAX);
                 [self.tableView reloadData];
+                [self.tableView layoutIfNeeded];

                 if (!offsetY) {
                     offsetY = -self.innerMarginHeight;

++ @yamoridon's patch worked for me (thanks, btw :)

#37

That works to me!

thanks

@yamoridon thanks

@yamoridon patch work for me too thanks

Hi all,
You are welcome!

Merged.


method meta still uses "modern?"

Description
While removing the "modern?" method I was surprised to see that I was still getting deprecation messages in the logs while running a test suite even after it was replaced with the new recommended method. The issue seems to be that the "meta" method that the browser.to_s uses, still uses the deprecated "modern?" method.
How to reproduce
browser = Browser.new('Some User Agent' , accept_language: 'en-us')
browser.to_s
What do you expect
No deprecation message.
What happened instead
Deprecation message shows:
Browser::Base#modern? is now deprecated.
Check https://github.com/fnando/browser/issues/435 for details.
Software:

Rails version: 2.6.2
Browser gem version: 3.0.3, master

Fixed by v4.0.0. ;)


NDC-SDG Linkages backend changes

adds statuses, climate_responses and types_of_information to meta
merges goal number + goal title into "sdg", target number + target title into "sdg_target
removes document_type and language from response
reorders columns in csv

The new filters (status, climate_response, and types_of_information) in NDC-SDG Linkages are not working in the backend. Do they need to be added or there is already something there but not documented?
Try: http://localhost:3000/data-explorer/ndc-sdg-linkages?ndc-sdg-linkages-status=Present&page=1

@Bluesmile82 Please check if there's anything else missing in this PR

Thanks for the changes! It works fine


Reduce binary size #35

via -ldflags="-s"
Signed-off-by: Arun Koshy<EMAIL_ADDRESS>

@susmithp let's run some extended tests on this to see if there is any side-effect or surprise

Tested, it's working as expected.


⬇ feat: Assistant File Downloads

Summary
Implements file downloading for files generated by Assistants
Closes #2030
Other Changes

Handles assistant annotations better
Fixed an edge case where assistant messages would not sync correctly when requests were cancelled due to messages not being sorted correctly
Added file logging in areas still using console logs

Change Type

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

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.
[x] New documents have been locally validated with mkdocs

Nice! 🚀 🙏


Non-GO sites do not need GO-specific code

Non-GO sites do not need GO-specific code. For example, Planteome does not need to /ever/ contact QuickGO for most things. There should simply be a condition to skip the very GO-specific stuff.

Tagging on @elserj to monitor for progress.

Adding on to this as a further example of "de-GOing" the code, the list of species used for the base_statistics page is currently hard-coded in https://github.com/geneontology/amigo/blob/master/scripts/amigo-create-base-stats-cache.js#L62.  Would be nice if this could be configured in a yaml file instead.


Incompatibility with auto-detect-indentation

When I activate both this plugin and auto-detect-indentation I get a strange issue about closing brackets: "E133 for closing bracket which is missing indentation".
This happen at every closing bracket with indentation. So a code like the folliwing will report the issue:
a = {
    'fff': 5
}

If I change the indentation as the following the error disappear (but obviously the indentation is wrong):
a = {
    'fff': 5
    }

...not sure if the issue is here on in the other plugin.

This package is just forwarding on the messages from flake8, you would need to file an issue on that if you think this is something that should be fixed there.
That being said pycodestyle (which flake8 integrates) claims that in their default configuration E133 is ignored. Because of the way flake8 integrates it, this default ignore is... ignored. I would recommend just adding E133 to your list of Ignored Error Codes in this package.

Thanks for details.
The fact is that if I manually call pyflakes on the file I have the right output (the same pyflakes configured to be used by the linter).
But I still think there's (again) something wrong with my environment (I'm the same guy of issue #140 and I never fixed that one either 😄)


Distance Formula added

Added the actual formula and a brief explanation on how to use it.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for contributing to freeCodeCamp @absentMindedDeveloper!


can not build with android platform

Hi，dear Ryan A. Pavlik ：
I have developed my openxr runtime on android platform. I want to run a CTS test, But I encounted a compile problem  which can not find "src/version.gradle" file, when I compiled the source code on android platform with android studio.
How can I build the source code on android platform with android studio? or how can I get the file version.gradle?
I am looking forward to your reply ,thanks!

Hi,rpavlik:
I am looking forward to your reply ，thank you very much！

Hi，dear Ryan A. Pavlik ： I have developed my openxr runtime on android platform. I want to run a CTS test, But I encounted a compile problem which can not find "src/version.gradle" file, when I compiled the source code on android platform with android studio. How can I build the source code on android platform with android studio? or how can I get the file version.gradle? I am looking forward to your reply ,thanks!

I remember OpenXR-SDK-Source(https://github.com/KhronosGroup/OpenXR-SDK-Source/blob/master/src/version.gradle) has a version.gradle, maybe you can copy it and try it.

This should be buildable in the upcoming OpenXR release. There are a few more improvements for the CTS I'm waiting for WG approval to merge.


Update vendors

Hopefully this time is the good one :joy_cat:
Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

:+1:

Hmm it seems to bump too many indirect dependencies without reason.
github.com/asaskevich/govalidator v0.0.0-20200108200545-475eaeb16496 // indirect
...
github.com/go-openapi/analysis v0.19.7 // indirect
github.com/go-openapi/errors v0.19.3 // indirect
github.com/go-openapi/runtime v0.19.11 // indirect
github.com/go-openapi/spec v0.19.6 // indirect
...
github.com/go-openapi/swag v0.19.7 // indirect
github.com/go-openapi/validate v0.19.6 // indirect
# and many more

What should I do ? remove sed -i /indirect/d from go.mod ?

Yes, this + go mod tidy should work.

@simonpasquier done

:+1:


Add tutorial file to CubitPy

Add a tutorial that aims to clarify and demonstrate the functionality of CubitPy.
This tutorial is based on the Cubit tutorial provided within Cubit 15.2

@BishrMaradni I added some comments / explanations / functionality. I suggest you have a final look and then we can merge this (I also included a small change in the package configuration). I did not have time to add testing for the tutorial, but that is something we can consider in the future.
Thanks for the effort @BishrMaradni

@BishrMaradni you can merge if you think this is ready.


MVP to v2 data migration pipeline

AS AN Architect
I WANT to have MVP to v2 data migration pipeline
SO THAT implementing v2 could be done in parallel with MVP without affecting each other
Acceptance Criteria

[ ] MVP uses its own database and is not affected by v2 db
[ ] v2 uses its own database and is not affected by MVP db
[ ] Data is migrated from MVP db tables to appropriate v2 ones
[ ] Data is not migrated from v2 db tables to any MVP ones
[ ] No db schema changes are made to either MVP or v2 due to the nature of this issue
[ ] Data migration from MVP to v2 can be initiated by a separate data migration script directlly and/or CronManager

Not needed. Clients don't want to have historical chats


updated defaults to current stable

Updated defaults to current stable upstream

Thank you for contributing to openstack/tripleo-image-elements!
openstack/tripleo-image-elements uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Expose status code as part of the error object

Thanks so much for your excellent work. I was wondering if there was a way to pass the statusCode from the http response as part of the returned error object. Likely an additional property added here:
https://github.com/jsforce/jsforce/blob/master/lib/http-api.js#L251
Is it possible to access this currently?
Thanks!

https://github.com/jsforce/jsforce/issues/490#issuecomment-2127711394


Reuse index.json in fullIndex.json

Currently, we have to duplicate code between index.json and fullIndex.json . Instead. there may be a way to extend siblings of a schema to eliminate duplication and drift caused from maintaining redundant copies.
fullIndex.json is more than ornamental, because the API which needs the auto-generate YAML spec requires the additional components available in fullIndex.json.

Can do something like https://stackoverflow.com/questions/52566472/how-to-extend-a-schema-in-json-schema however, will need 3 schema:

BaseIndex ( minimum fields without additionalProperty or required restrictions).
AtlasEditorIndex (inherits base using allOf, but adds required and additionalProperty and whatever else necessary)
FullSearchIndex (inherits base using allOf, adds name, collectionName and database fields alongside its own required and additionalProperty settings.)

We solved this problem, but it required adding the following bloat: https://github.com/manoj-pillay-10gen/simple-search-index-validator/blob/main/schema/fullIndex.json#L12-L17
Note that this is not necessary if using JSON Schema draft 2019-09 onwards. We could replace additionalProperties:false  with unEvaluatedProperties:false in order to permit subschema elements but not any unknown elements. Unfortunately monaco-editor which uses https://github.com/microsoft/vscode-json-languageservice is yet to support drafts 2019-09 or 2020-12 . See https://github.com/microsoft/vscode/issues/98724 . Consequently, we will have to deal with the bloat to permit use of elements in the subschema while avoiding any other field definitions.


Clipboard History

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Expected Behavior:
Button and shortcut in powertoys to display list of past clipboard items.
Scenario:
When you accidentally copy over something you needed or what you see past copied text.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

@ITSRI-AG
isn't this already covered by Win + C/V?

Looks like it is, thanks.


New package: HiteVision.HitePai6 version 6.2.312

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 549.)

Manual Validation ended with:

Hi @SpecterShell,
The installation isn't unattended. Is there an installer switch to have the package install silently?
(Automated response - build 550.)

@wingetbot waivers Add Validation-Domain


Issue: Could not resolve hostname h01-118: nodename nor servname provided, or not known",

Issue
When we execute the following commands
export k8s_version=118
export VM_NAME=h01-${k8s_version}
export PASSWORD_STORE_DIR=~/.password-store-snowdrop

ansible-playbook hetzner/ansible/hetzner-delete-server.yml -e vm_name=${VM_NAME} -e hetzner_context_name=snowdrop
ansible-playbook ansible/playbook/passstore_controller_inventory_remove.yml -e vm_name=${VM_NAME}  -e pass_provider=hetzner
ansible-playbook ansible/playbook/passstore_controller_inventory.yml -e vm_name=${VM_NAME} -e pass_provider=hetzner -e k8s_type=masters -e k8s_version=${k8s_version} -e operation=create
ansible-playbook hetzner/ansible/hetzner-create-server.yml -e vm_name=${VM_NAME} -e salt_text=$(gpg --gen-random --armor 1 20) -e hetzner_context_name=snowdrop -e pass_provider=hetzner -e k8s_type=masters -e k8s_version=${k8s_version}
ansible-playbook ansible/playbook/sec_host.yml -e vm_name=${VM_NAME} -e provider=hetzner

under k8s-infra project, then ansible reports the following error during the execution of the role sec_host
TASK [Gathering Facts] *****************************************************************************************************************************************************************************************************************
task path: /Users/cmoullia/code/snowdrop/infra-jobs-productization/k8s-infra/ansible/playbook/sec_host.yml:2
Friday 19 February 2021  09:15:42 +0100 (0:00:00.140)       0:00:00.140 *******
<h01-118> ESTABLISH SSH CONNECTION FOR USER: snowdrop
<h01-118> SSH: EXEC ssh -C -o ControlMaster=auto -o ControlPersist=60s -o Port=22 -o 'IdentityFile="/Users/cmoullia/.ssh/id_rsa_snowdrop_hetzner_h01-118"' -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o 'User="snowdrop"' -o ConnectTimeout=10 -o ControlPath=/Users/cmoullia/.ansible/cp/0ab23a8fdc h01-118 '/bin/sh -c '"'"'echo ~snowdrop && sleep 0'"'"''
<h01-118> (255, b'', b'ssh: Could not resolve hostname h01-118: nodename nor servname provided, or not known\r\n')
fatal: [h01-118]: UNREACHABLE! => {
    "changed": false,
    "msg": "Failed to connect to the host via ssh: ssh: Could not resolve hostname h01-118: nodename nor servname provided, or not known",
    "unreachable": true
}

Is the problem related to another issue which is that the property ansible_ssh_host does not exist ?
 pass-team hetzner/h01-118
hetzner/h01-118
├── ansible_ssh_port
├── groups
│   ├── k8s_118
│   └── masters
├── id_rsa
├── id_rsa.pub
├── os_password
├── os_user
└── ssh_port

The ansible_ssh_host is not created during the execution of this playbook :
ansible-playbook ansible/playbook/passstore_controller_inventory.yml -e vm_name=${VM_NAME} -e pass_provider=hetzner -e k8s_type=masters -e k8s_version=${k8s_version} -e operation=create -vvv

See result generated: https://gist.githubusercontent.com/cmoulliard/096729ecc24aa393d8b6dc2194c7ab51/raw/d58baee5946a0796957b335eb8da04fa3826d12b/gistfile1.txt
The ip_address of the vm is well created during the creation of the VM BUT it is not at all stored within the passwordstore db
See: https://github.com/snowdrop/k8s-infra/blob/d6ea90e2fbb2e57c94b09c9c4307ad895032a2f2/ansible/roles/hetzner/create_server/tasks/main.yml#L58

I'll take a look on why the passwordstore is not storing the ip address correctly.

Another problem is that such a variable os_user is not at all declared/define: https://github.com/snowdrop/k8s-infra/blob/366b00974f530d5662abc7d7f40cc52daa2b25f4/ansible/roles/hetzner/create_server/templates/user-data.j2#L23

I have fixed this on my branch.


quote the version of Go

I recommend you to alway quote the version of Go.
Even if it seems to be working now, there may be problems in the future.
According to Go's release cycle, Go 1.20 will be released in February next year.
The configuration file of GitHub Actions is YAML, and YAML parser treats 1.20 as a number.
Many parsers do not distinguish between the numbers 1.20 and 1.2.
It confuses GitHub Actions.
If you write "1.20", the parsers treat it as a string.
You will not have this problem.

@shogo82148
Thank you for a lot of PR!!

The configuration file of GitHub Actions is YAML, and YAML parser treats 1.20 as a number.
Many parsers do not distinguish between the numbers 1.20 and 1.2.

It was a very easy-to-understand comment, so I will be careful from now on.
I haven't considered the internal implementation of the parser until now.
Your comment sounds as if you've experienced the same bug.

It is a very common mistake in YAML...
Please be careful.
https://ruby-jp.slack.com/archives/CLVTA9DV1/p1641342863083800

I couldn't find the source, but I've seen some people make the same mistake with Perl 5.20.


The icon and title is not changing dynamically according to rendered page

I can create one hook which will render on each page to change the title
The icon of page will change to app icon present in img folder named logo.png

already done


As a developer I want to update the database to contain relevant information for being able to select popular and trending challenges.

[x] Create new column in challenge table.
[x] Update/create new function to update new column every time challenge is run.
[x] Come up with way to track trending challenges

[ ] Do we need an aggregate table of instances over last x days? Discuss with Logan.

This is completed by #153.


[ADD] RST versions of the current legal documents

Add:

Bylaws
ECLA
ICLA

Warning:
Links on https://odoo-community.org/page/cla must be updated to point to the pdf at the root directory.

:+1:

:+1:
Maybe a short note should be added in the header of the documents to remind that a vote is needed to amend them, not just a PR.

@clonedagain it's done in the README.rst:
https://github.com/OCA/odoo-community.org/pull/20/files#diff-88b99bb28683bd5b7e3a204826ead112R11

Obviously I have not tested this, but seems a great initiative, thank you @max3903 :+1:

:+1:


Cache the factorized matrix

I have found some situations where caching the inverse of the covariance matrix leads to unstable variance predictions. We should cache the factorized matrix instead, which makes the variance predictions more stable. I will fix on master and write a unit test that reproduces the result.

Fixed in PR #62, closing.


text.ttl - refining some time-spans

1. date-Elements with only @notBefore
These are being ignored right now, if I see this correctly. There is a time-span as an object in the F30_Publication_Event, but no statement for the timespan itself. The same with the time-spans linked to F28_Expression_Creation.
In these cases, please create a time-span as usual, just only use the cidoc:P82a_begin_of_the_begin. Don't use the cidoc:P82b_end_of_the_end.
Also add the line
cidoc:P2_has_type <https://sk.acdh.oeaw.ac.at/types/date/approx> ;
2. date-Elements with only @notAfter
Like above, just vice versa (use only cidoc:P82b_end_of_the_end).
3. Date Type "approx"
I thought we already had that, but maybe I was wrong; anyway: If a date-element has @type="approx", please add this line to the time-span (like above):
cidoc:P2_has_type <https://sk.acdh.oeaw.ac.at/types/date/approx> ;

I applied all three changes to the timespan elements.

date type approx goes with both the "onle-notBefore" and the "only-notAfter" cases.

yes, it should already work like this:
        <xsl:if test="tei:date[@type='approx']">
          <xsl:text>  cidoc:P2_has_type <https://sk.acdh.oeaw.ac.at/types/date/approx></xsl:text>
        </xsl:if>

@BergListe To clarify: Not just the @type="approx", but also the other two cases mentioned here need this type.

oh, I overlooked this line in your initial statement

Also add the line cidoc:P2_has_type https://sk.acdh.oeaw.ac.at/types/date/approx ;

@BergListe Some new bug? Now those with just notBefore have no time-span any more.

Do you have an example bibl?

https://sk.acdh.oeaw.ac.at/DWbibl02259/publication/time-span

Done

How should the priority look like?

tei:date/@key
tei:title[@level='m']/@key
tei:citedRange/@xml_id

date-key, else title m key, else citedRange wholeText (else bibl ID)


Bug: Information Missing from 3 Ghost Types

Information for three of the ghosts is currently missing. Info missing for: Deogen, Moroi & Thaye. Tried toggling ghost info off and on, clearing cookies, and using a different computer, so it's not my system doing something funky. Wasn't like this a few days ago when last using it.

Looks like the HTML for those ghosts' descriptions was removed entirely.

@zuffdaddy I don't know how this open source stuff works, but I can work on making additions if it's no problem.

@RySkream I read in the readme that anyone can work on this, so I made a fork and have started gathering info for the 3 ghosts that have missing information.


Add ccf-what extension

Description
Loop up the rankings of China Computer Federation (CCF) recommended publications in Raycast.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Researchers can look up a paper's publication in the China Computer Federation (CCF) recommended list, where they highlight some high-quality publications.
A similar extension would be show-rank in Chrome.

Hi there, I have new updates regarding your comments and new features:

Accepted your advice: Search in DBLP, Toggle Details, and built-in filtering
Flexible data source URL and update policy: set in the preference
Frequency sorted
Localization: set in the preference
Thanks in advance for your work and any feedback! 🔥

And some minor icon issues.


Fixing FAQ Areas

Corrected the area of "Find all Histories and purge" to remove the duplicated area on the GTN FAQ
Corrected the area of "How can I  upload data using EBI-SRA?" since it was not showing up on both sections (Galaxy and GTN) of the GTN FAQ. (or this can go in the Galaxy FAQs under the data upload section)

Thanks @Melkeb!

Corrected the area of "How can I  upload data using EBI-SRA?" since it was not showing up on both sections (Galaxy and GTN) of the GTN FAQ. (or this can go in the Galaxy FAQs under the data upload section)

I think that it fits best in the Galaxy FAQs under Data Upload. It seems to me that the introduction section of the GTN FAQs is more generic about loading data.

Okay :) will move it to the Galaxy faqs and change the area.


[Component] Button

As a product developer, I want to use buttons in my product that follow the guidelines set by the CA Style Guide.
AC:
Schema for component considerations
Things needed for a "button" component:

Typography

currently looks like Open Sans
system font back ups

Color
Spacing
Icons
Animation
States
Size / Shape
Variants (kinds)
A11y
Development Docs (props, code, examples)
Design Docs (usage, philosophy, patterns)

Considerations:
We should probably think about what an MVP of this component is.


Fix structured array IO

Fixes #249 and fixes theislab/scanpy#832 by saying we'll always read unicode strings in as variable length. This still needs an editing pass, and another look over in the morning, but I'd like to get feedback as soon as possible. @flying-sheep, thoughts?

I think we should either allow any length of strings or warn the user if their strings are too long.

Totally agree with that being the eventual goal. This is easy to do by just keeping all values as variable length strings. Until those are supported in structured arrays by zarr, I'm thinking we just keep the old behavior of using an arbitrary fixed length that works for most gene names (since that's the only time we use structured arrays). Previously this was fixed to 50 bytes for zarr, but 200 for hdf5 (based on the pbmc68k dataset).

This definitely shouldn't be squashed. The granularity of the commits will be useful in the future. While writing this I wished I still had the commit history from previous implementation so I could see what things I had tried before. This is especially the case for working with poorly documented interfaces like structured dtypes.
I could potentially reorder the commits, and squash some formatting ones, but since I typically navigate git history through blames I'm not sure how useful this would be.

Of course blame should be maximally useful, so there’s two options:

Squash everything (low effort, low granularity)
Manually rebase: Every commit done to fix tests or formatting or bugs you introduced in earlier commits of the same PR need to be squashed. (high effort, perfect granularity)

Keeping “Formatting” and “whoops” commits are detrimental to squash being useful.

@flying-sheep, I've opened a new issue about managing git history so we can split that discussion off. For this PR, I'll follow the strategy I propose in my top comment. Once this is ready to merge, I'll remove formatting commits, and consider squashing other ones. We can figure out a different strategy, but I don't think it's worth holding up this PR for.

Back on topic of this PR, is there code that should change?

No, looks good to go!

@flying-sheep, I've removed the formatting commits. I think I want to test this more thoroughly, but I'll do that in a separate PR. Ready to merge.

Great, thank you! Now let's get this released!


More incorrect triple-backtick code block formatting

Corrects a couple more triple-backtick code-block formatting errors

I am working on fixing the linter errors in this PR.

/backport release/1.2 release/1.0


Serialization of nested classes produces faulty JSON

If we serialize nested classes (depth of 2 or more), it produces faulty JSON code (a closing '}' is missing)
The code below shows the bug. It produces the following JSON:
{"A":{"a":[{"B":{"b":[]}]}}
instead of:
{"A":{"a":[{"B":{"b":[]}}]}}
The '}' closing the B object is missing.
Code sample producing the error:
class C{
public:
	string c;

	void serialize(JSON::Adapter& adapter){
		JSON::Class root(adapter, "C");
		JSON_T(adapter, c);
	}
};

class B{
public:
	vector<C> b;

	void serialize(JSON::Adapter& adapter){
		JSON::Class root(adapter, "B");
		JSON_T(adapter, b);
	}
};

class A{
public:
	vector<B> a;

	void serialize(JSON::Adapter& adapter){
		JSON::Class root(adapter, "A");
		JSON_T(adapter, a);
	}
};

int main(int argc, char** argv) {

	A a;
	a.a.push_back(B());
	cout << JSON::producer<A>::convert(a) << endl;

	return 0;
}

Yes, thanks for the report. The generation for nesting does need to be a little smarter I think.

I created a pull request with a fix. Basically, the "(de)serializing vector" state was overwritten when (de)serializing nested vectors.

Fixed, Many thanks.


Custom XERC20 Multisend

Background
Write and deploy a custom Multisend contract that combines Lockbox deposit and xcall together. This enables single txn bridging of xERC20s that have a lockbox on the source chain but would be specific to Connext.
Tasks

[ ] Write XERC20Multisend.sol
[ ] Write unit tests
[ ] Deploy to all chains using existing hardhat deploy utils setup

Wait until we get feedback from users to do this.

This will be tackled in the bridge adapter.


Wrapper Function Tensorflow Error Handling

I want to quit a program when it fails to upload checkpoint file.
However, Tensolayer catches an exception (e.g. tf.errors.NotFoundErrors) inside a function and it only prints an error message.
How could I directly handle an error from tensorflow if a wrapper function already catches it?
If this problem is just easily handled by python programming, I'll close this issue.
Besides, thank you a lot for awesome tensorflow library, I really enjoy it!
def load_ckpt(sess=None, mode_name='model.ckpt', save_dir='checkpoint', var_list=[], is_latest=True,
printable=False):

try:
    saver = tf.train.Saver(var_list)
    saver.restore(sess, ckpt_file)
except Exception as e:
    print(e)
    print("[*] load ckpt fail ...")

@chaos5958 , to make it easy, I suggest you to copy the code inside this API. Could I know what is the error about?

@zsdonghao, Actually, this is not error, but suggestion for improving Tensorlayer.
I found Tensorlayer is extremely useful for people who want to use parts of tensorflow semantics and control tensorflow semantics in detail.
However, above example might be limitation of Tensorlayer, since we can't control errors occurred from Tensorflow if we don't modify APIs itself as you suggested.
Then, let's close this issue and, I'll send a pull request if I can design better code.
Thank you for answering me,
Hyunho


Display class info in search panel even if class isn't offered in the current quarter

Currently, if you search for a class in a certain quarter, you get this error page:

Even if the class isn't offered in the selected quarter, it would still be useful to see the class info, reviews, zotistics, and especially past enrollment, since past enrollment gives you an idea of when the course will be offered again. I feel like the error page should look more like this, so you still have some information available:

Or even just replace the error image with "No matching classes found this quarter. See Past Enrollment to see when the class was last offered."

@ap0nia @srukelman What's the idea on implementing/not implementing this?
I think it's a fine idea, but it could be a little difficult because you might have to change the API.

Thanks for the suggestion. We'll go with #1024, since this is more PeterPortal's realm of functionality. I suggest you use it for these tasks (e.g., 4-year planning and class history).


Added x and xx example, fixes #287. Also fixes tutorial.

This PR adds an x and xx example (removing the hello world example). It also updates the tutorial's lists of builtin functions and methods (though it does not include regex or type methods). (The reason I updated the tutorial was that I didn't know about the concat or push list methods and so initially wrote a big mess. I don't want anyone else making the same mistake.)
Potential issues:

I implemented xx to flatten list input. So 1 xx 3 is [1, 1, 1] and [1, 2] xx 3 is [1, 2, 1, 2, 1, 2]. Is this the desired behavior?
I don't know how to run all the tests. All I did is create a new test file and run that, while deleting the hello world test file. Are there any other style tests or such I should run?

This was meant to be "fixes #278", not "fixes #287", of course.

I just discovered this PR. Good work! I will comment on lines of code individually, but what I see already looks good enough that I'd like to merge ASAP.
Re your two questions:

I hesitated in many different directions with respect to flattening. Even had to go back to see how the old, removed infix:<xx> worked. (It flattens.) For now we'll let the new one flatten too.
The proper incantation for running the test suite is prove -e'perl6 -Ilib' -r t/. (I'll consider putting it somewhere in the vicinity of the README file.) I have an alias set up locally which runs the prove command after first running an empty program in bin/007 (because this compiles all the module files first before running the test suite, which avoids getting those errors through the test harness).

All I made was minor, non-merge-blocking comments, so I'm going to go ahead and merge this now. Again, great work, especially for a first contribution!


Error importing on Python 3.5

While checking fix for https://github.com/TileDB-Inc/TileDB-Py/issues/241, I saw the following:
ihnorton@DESKTOP-2S2KI39:~$ python3
Python 3.5.2 (default, Oct  8 2019, 13:06:37)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tiledb.cloud
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ihnorton/.local/lib/python3.5/site-packages/tiledb/cloud/__init__.py", line 3, in <module>
    from .client import (
  File "/home/ihnorton/.local/lib/python3.5/site-packages/tiledb/cloud/client.py", line 5, in <module>
    from . import config
  File "/home/ihnorton/.local/lib/python3.5/site-packages/tiledb/cloud/config.py", line 64, in <module>
    logged_in = load_configuration(default_config_file)
  File "/home/ihnorton/.local/lib/python3.5/site-packages/tiledb/cloud/config.py", line 32, in load_configuration
    if not os.path.isfile(config_path):
  File "/usr/lib/python3.5/genericpath.py", line 30, in isfile
    st = os.stat(path)
TypeError: argument should be string, bytes or integer, not PosixPath
>>>

Not sure what version of Python we want to support as a minimum, but noting that the default in Ubuntu 16.04 is Python 3.5.

We have CI setup for 3.5, 3.6 and 3.7, so we do want to support 3.5.

Ok. Noting that this was on WSL1 under Windows 10 with Ubuntu 16.04.

Issue is that os wasn't made fully compatible with pathlib until 3.6, e.g. see the Changed in version 3.6 comment for os.path.isfile.  For compatibility with 3.5 you'll need to explicitly coerce pathlib.Path objects to string.
Failing example and workaround for 3.5:
In [1]: import os; import pathlib; import sys

In [2]: sys.version_info
Out[2]: sys.version_info(major=3, minor=5, micro=6, releaselevel='final', serial=0)

In [3]: foo_path = pathlib.Path('foo.txt')

In [4]: os.path.isfile(foo_path)
---------------------------------------------------------------------------
TypeError: argument should be string, bytes or integer, not PosixPath

In [5]: os.path.isfile(str(foo_path))
Out[5]: False

Both the failing example and workaround are fine in 3.6+:
In [1]: import os; import pathlib; import sys

In [2]: sys.version_info
Out[2]: sys.version_info(major=3, minor=6, micro=9, releaselevel='final', serial=0)

In [3]: foo_path = pathlib.Path('foo.txt')

In [4]: os.path.isfile(foo_path)
Out[4]: False

In [5]: os.path.isfile(str(foo_path))
Out[5]: False

Thanks for the pointer, @jschendel!


Dev Home crashes after clicking on Settings>Account

Dev Home version
0.801.344.0
Windows build number
10.0.22631.2715
Other software
OS Build Version: 10.0.22631.2715.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.25
Steps to reproduce the bug
N/B: I wasn't signed(Logged) in.
Go to Settings>Account
Expected result
To load the menu
Actual result
Once I clicked on "account", the "account" option became grayed out, and after that, the Dev home app crashed.
Included System Information
CPU: Intel(R) Core(TM) i3-1005G1 CPU @ 1.20GHz
Physical Memory: 11.79GB (6.7GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.801.344.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.800.344.0_x64__8wekyb3d8bbwe

/logs

Duplicate of #1639, #1634.

@skanda890 Was there a solution? I started following the other issues but couldn't find one. If not, here are the details from my system which has the same error.
Dev Home Version
0.801.344.0
Windows build number
10.0.22631.0
Included Extensions Information

Name
Id
Version
Source

Dev Home (Preview)
Microsoft.DevHome
0.801.344.0
winget

Dev Home GitHub Extension (Preview)
Microsoft.Windows.DevHomeGitHubExtension_8wekyb3d8bbwe
0.800.344.0

Dev Home Azure Extension (Preview)
Microsoft.Windows.DevHomeAzureExtension_8wekyb3d8bbwe
0.300.344.0

Additional Details
Same experience as reported above. The crash does not happen if you disable the github extension (but then of course you cannot log in!). Same error with the Azure extension.
DevHome_TempState.zip
DevHomeAzureExtension_TempState.zip
DevHomeGitHubExtension_TempState.zip

@simonwoodhead, I actually don't know. It just got fixed after the next build.

Probably check #1634

Clean install of Windows 11 23H2 (22631.2861), just installed DevHome to try it out (from the Store). I only installed the GitHub extension, as I don't use Azure. When I go Settings -> Accounts, as the OP reported, the account option greys out, the app freezes and then crashes. The same happens if I go Machine configuration -> Clone repositories  and then select "Account".
DevHomeTempState.zip
DevHomeGitHubExtensionTempState.zip

Yeah, something is up.

Yeah, something is up.

@simonwoodhead, I later realised that

@simonwoodhead, I later realised that

I'm having the same issue. It crashes whenever I try to do anything related to GitHub at all for me, which almost renders it useless for me. My pc has a 4th gen i3, 8gb ddr3 and runs a modified version of windows 11 called AtlasOS 11, though I don't think that's the issue as I've never had problems with that kind of thing using Atlas before. (I also posted this on the duplicate thread)


Lecture 4 Slide 30 Term (Word) Correction

I believe it should be "training set" instead of calling the data "train set".

Yep. Thanks for submitting. 1 point


Issue number  #16  fix

Allows user to tab between fields to enter data.

@papergirl thanks for the PR. Will take a look and let yku know.

no worries, happy to make any changes per request, relatively new to using gtk so theres probably better ways of doing it.

At the moment, using Xamarin Studio 6,3 and Addin Maker 1.3.2 the solution (before you PR) doesn't event compile.
@papergirl what version of Xamarin Studio and Addin Maker do you have installed?

I'm using  monodevelop 5.10 looking into it this appears to be because they no longer update the package on apt, and have instead moved to using flatpak which I was previously (naively) unaware of, I will correct this.
Looks like from those errors list we need a different task, to make the plugin compatable with 6.*. However looking at Xamarin's site and release notes too attempt identify the breaking changes I also noticed that Xaminin Studio it's self, is no longer going to be developed for mac/windows and they're recommending using the visual studio extension. I don't know where this leaves MonoDevelop (I assume, it will be still developed), but  if you guys use ios or windows primarily, then this plugin won't be very useful to you in the future ;/

@papergirl yes, the Visual Studio for Mac will probably replace Xamarin Studio. So it would be nice to make this addin work on visual Visual Studio for Mac.

You'd need to confirm the functionality doesn't exist already in other versions of VS on windows it does, (...barring my ideas of editing multiple langs at once...)  and if didn't it  would require a rewrite of all of this barring the resx/file handling stuff. :/

@papergirl  ATM the VS for mac don't support event resx file editing, so would be nice to port the atual library to run on VS for mac.

OK I've done the work to get it too compile and run under the latest version however I can't do another PR separate from this one :S, Think I should've used different branches for each piece of work :/ I'm relatively new to git .

@papergirlYou used the same branch to fix that issue? I will try to fix that later. Thank you.

I've tested it and everything seems to be ok.

@papergirl I've tested it and everything seems to be ok.


Get png chart problem with lines and options

when i click download from the menu, the image comes wrong, is there any way i can get that image?
i used:
var dataURL = chart.dataURI().then((uri) => {
		 $('#img').html('<img src="'+uri.imgURI+'" />');
    console.log(uri);
	});

but the lines and options do not appear in the image, am i doing something wrong?
CODEPEN: https://codepen.io/andre-gasparin/pen/RwKwRyv
Thanks

I found the problem, it is generating an image before completing the effect, fixed it by removing the effect.
(a settimeout can also be used).
the solution is in the same pen as the question:
CODEPEN: https://codepen.io/andre-gasparin/pen/RwKwRyv


route service is not enabled successfully

The following commit removed the router.route_services_secret property from the gorouter job and added the same property under the cloud_controller_ng job.
https://github.com/genesis-community/cf-genesis-kit/commit/b88090a29a0bd5df7f673390ecc874a427392662
In this case, after binding route service, when we tried to use the servive, a 502 error is returned.
In our test, we found out that the router.route_services_secret needs to be configured for both the gorouter and cloud_controller_ng job to make route service work. '
The spec for both jobs  are as follows
https://github.com/cloudfoundry/routing-release/blob/develop/jobs/gorouter/spec#L160-L162
https://github.com/cloudfoundry/capi-release/blob/develop/jobs/cloud_controller_ng/spec#L953-L955

closed by #126


chore: remove duplicate words

remove duplicate words to enhance readability.

@bors r+

:pushpin: Commit 87084c176bd8103f90366f9fe8fa7b61a966a8c5 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 87084c176bd8103f90366f9fe8fa7b61a966a8c5 with merge 7c8a52444f0451db815e9543a8631e69980e88d8...


Add --disable-binaries

mmdblookup now depends on pthread.h which can be disabled on some
toolchains so add an option to be able to compile libmaxminddb without
this binary
Signed-off-by: Fabrice Fontaine<EMAIL_ADDRESS>

Thank you!


gdm install hangs

Running it my machine just hangs while cloning repos.  You should add a -v | --verbose flag to show std_out/std_err

:gem:

I think you might need to do this: http://stackoverflow.com/a/11428767/429533
Try running with the -vv option.


Mouse click doesn't select the item, Enter Key does!

As the title says, i see that happening in demo too.

I have 0.2.2 and the mouse click is not selecting.. what am I missing?
ping @oferh


Feature request: automatic splits similar to tiling window managers

Description of the new feature/enhancement
Creating new splits could be made easier by providing a keybinding that automatically splits the currently active pane based on the current split type, similar to tiling window managers such as bspwm.
When activating the keybinding (e.g. Alt+Shift+Enter) following would happen:
If no split exists yet, the terminal would be split vertically. If a split exists and the currently active pane is part of a vertical split, it would be split horizontally and vice versa.
This is also visualized in the documentation of bspwm
Additionally, other schemes such as a spiral scheme could also be implemented

Proposed technical implementation details (optional)
The implementation could in theory be very simple: Just query the split type of the active pane
(vertical/horizontal) and create a new split of the other type when pressing the keybinding.
I would try to implement this feature myself but unfortunatley do not have the time at the moment.

Okay so this sounds like two different requests here

Use the current profile for creating a new split. This is already tracked by #1756, so let's direct that discussion to that thread.
"automatic" or "spiral" split mode - I don't hate this. And with horizontal and vertical already being parameters for splitPane, we could totally extend that to split in automatic ways like this.

You're right, I didn't think about different profiles because I currently only use one.
Simply alternating the split types would propably be way easier to implement than
something more complex like a spiral (sometimes also called fibonacci) mode (which personally
I also find not very useful anyway, I just wanted to give another example).
If this would be implemented there could also be a split ratio parameter (also found in bspwm)
which decides if a new split should split the current pane in the middle (ratio 50%) or at another point.


md025 updating pages with business attributes information

Updating campaign and Bulk API pages with business attributes info.

Docs Build status updates of commit b07eee0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/customer-management-service/advertiseraccount.md
:bulb:Suggestion
View (=bingads-13)
Details

Advertising/bingads-13/bulk-service/account.md
:white_check_mark:Succeeded
View (=bingads-13)

Advertising/bingads-13/campaign-management-service/accountproperty.md
:white_check_mark:Succeeded
View (=bingads-13)

Advertising/bingads-13/campaign-management-service/accountpropertyname.md
:white_check_mark:Succeeded
View (=bingads-13)

Advertising/bingads-13/customer-management-service/advertiseraccount.md

Line 102, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Training code for efficient x3d

Hi,
I want to train "Accelerator model" (efficient x3d) with my own dataset (only 4 classes).
Can I use the training code from the tutorial ?
Is it just changing the model from resnet to efficient_x3d, or there is something that I miss?
(pytorchvideo.models.resnet.create_resnet() to pytorchvideo.models.accelerator.mobile_cpu.efficient_x3d.create_x3d(num_classes=4))
Thanks a lot!

Can you share your solution please?

how to you can do it...? T.T


[Bug] [Workflow Definition] [UI] Non-existent resource front ends cannot be culled

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
After a resource is deleted or its name is changed, the resource referenced by the current task node cannot be displayed or deleted. As a result, a message indicating that the resource does not exist is displayed during task running
What you expected to happen
It should be possible to remove non-existent resource options
How to reproduce
Select the resource, save the workflow, and go to the resource center to change the reference resource name
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The interface return exist value, because the resource name has been changed, resulting in no content in the selection box; And resources cannot be selected by selecting overlays

The interface return existed value, because the resource name has been changed, resulting in no content in the selection box; And resources cannot be selected by selecting overlays

This is caused the resource metadata doesn't store in database, we can only parse the resource file from task param and display the error message.

But a resource method that can remove incorrect references should be provided


Elements in Accordion Content loose focus

Describe the bug
When the initial focus is not already in the accordion content, clicking on a child element of accordion content (e.g. an input) will focus it, but then instantly will focus the accordion trigger instead.
To Reproduce
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/github-324i4c-kfz11q?file=src%2FApp.tsx
Open one of the accordion items
Click on the page body
Click on one of the input elements (focus is instantly lost)
Click again on that input (now it gets proper focus)

Expected behavior
When focusing elements in the accordion content, the focus should not be moved to the accordion trigger.
Screenshots
https://github.com/kobaltedev/kobalte/assets/4012401/8f0cf4aa-205f-4639-9925-d336ab859eef
Desktop (please complete the following information):

OS: Linux Masterrace
Browser: Cutting Edge Firefox and Chrome
Kobalte Version: v0.9.8

Minor update on this. I figured out the lines which result in the accordion trigger being focused instead of the proper accordion content element:

https://github.com/kobaltedev/kobalte/blob/5dd86f7f557fcb9f00b7b004aa0686ee61769330/packages/core/src/selection/create-selectable-item.ts#L208
https://github.com/kobaltedev/kobalte/blob/5dd86f7f557fcb9f00b7b004aa0686ee61769330/packages/core/src/selection/create-selectable-collection.ts#L381
https://github.com/kobaltedev/kobalte/blob/5dd86f7f557fcb9f00b7b004aa0686ee61769330/packages/core/src/selection/create-selectable-collection.ts#L382

If all of these lines are commented out, then the bug is fixed ':). Obviously thats not the way we can ultimately solve this issue...

I took another look at this and think I understand it better now. The source of this problem is, that the accordion is based on the concept, that the triggers (button) are the managed selection items, instead of the accordion items themselves. Afaik the selection manager seems to know nothing about the accordion items. It only knows about the triggers.
createSelectableItem in accordion trigger:
https://github.com/kobaltedev/kobalte/blob/2f5802af077f0bb66d507ee6f38163d33a1b4f84/packages/core/src/accordion/accordion-trigger.tsx#L63
no createSelectableItem in accordion item:
https://github.com/kobaltedev/kobalte/blob/2f5802af077f0bb66d507ee6f38163d33a1b4f84/packages/core/src/accordion/accordion-item.tsx
And because of this, the selection manager has neither a way to know that the accordion item is already focused, nor a method to focus it. Instead it will try to find the first selectable key, which only can be a trigger:
https://github.com/kobaltedev/kobalte/blob/2f5802af077f0bb66d507ee6f38163d33a1b4f84/packages/core/src/selection/create-selectable-collection.ts#L366
Summary: I could not find a way to fix this bug, without changing the fundamental concept of the accordion 🙈.

Try this workaround...
function stopPropagation(e) {
  e.stopPropagation();
};

<Accordion.Content
  onKeyDown={stopPropagation}
  onClick={stopPropagation}
  onPointerDown={stopPropagation}
  onMouseDown={stopPropagation}
  onFocusIn={stopPropagation}>

fixed by https://github.com/kobaltedev/kobalte/pull/483


Merging Federation API with node API into single repo

Given the majority of the code in federation API is often derived and copied over from the node API, we want to discuss whether this service can exist as a module in the node API to facilitate a more efficient development process.
TODO:

[x] Make a list of logic that is shared in both API repos and put it in a shared document
[ ] explain how we can build two separate docker images, one per app, from the same repo
[ ] How do releases work if both tools are in the same repo
[ ] If both apps share code and we factor that out, how do we handle it during the docker image build process (i.e. is the common part a dependency or just included in both builds directly)
[ ] What tools exist to make handling monorepos easier, and what do they do for us

Tasks

[x] Create docker-compose that builds and then deploys both apps as they would run in production
[x] Document how to setup dev environment for each app and run tests
[x] Document how to run integration tests (that don't mock either app)
[ ] Make a decision on releasing the tools using their current versions, on releasing them independently, or with a shared version

some overlap: a good deal of overlapping logic coupled with app-specific logic so it's possible to merge but requires refactoring
little overlap:  the files could be merged but will run into some conflicts (same function names with similar functionality but different logic)
no overlap: the files could be merged with almost no conflict
a lot of overlap: essentially very similar code and logic, could be merged with a small refactor
routers:

assessment.py: some overlap
attributes.py: only in n-api
diagnoses.py: some overlap
pipelines.py: some overlap
query.py: a lot of overlap
node.py: only in f-api
route-factory: some overlap

other:

crud.py: little overlap
models.py: some overlap
security.py: a lot of overlap
utility.py: no overlap
main.py: a lot of overlap

Some initial thoughts after our chat today:

Having two separate apps in the same repo, but each potentially needing to be released independently of the other, seems it could be challenging to maintain (since we'd essentially be violating the expectation of what a repo is). My sense is that what we'd reduce for ourselves in terms of cross-repo changes, we would gain in devops work needed to make and maintain separate, automatic releases of two apps from a single repo.

If the goal is to still release each API separately, I think keeping them two repos with common logic factored out would be an option that improves dev experience without adding a lot of overhead for managing essentially two distinct codebases in one place

If we want to prioritize having the two apps in one repo, I think at minimum we should release them both together (i.e., whenever one API needs to be released, both are)

This gives us a bit more flexibility if we decide to internally refactor the logic of the APIs, and also means we can still take advantage of GH's release system since we're staying closer to the expectation of what a repo is (e.g., v0.5.0 could be now just associated w/ two compatible Docker images, each also being v0.5.0)

Some questions:

From a user's perspective, assuming we do proceed with merging the API repos, how would they report bugs for just one API? Maybe we use separate issue templates for the f-API and n-API?

I agree with Alyssa's points. We have talked a good bit about this now, and decided to not do the API repo merge. The main reasons for not doing the merge are

an n-API and f-API that talk to each other are most often parts of different deployments (e.g. one sits at site A and talks to the other at site B). And so we don't help ourselves by just ensuring that the most recent version of n-API and f-API can talk to each other, because across the network they are not upgraded at the same time (again, because part of different deployments)
merging the repos makes it more involved to version n-API and f-API separately - which we need to continue doing for above reason
there are other ways to achieve the same DX improvement of ready-to-go integration tests, development environments, and ability to stand up the other needed services during testing - but that's a separate conversation

In the process of discussing all this, we also made it a bit clearer that unlike n-API and f-API, most of our other tools actually can be considered part of the same deployments. i.e. CLI, graph data schema, graph, and n-API (arguably also annotation tool, and data dictionary schema) together all make up a "node bunndle" (name tbd), and f-API and query tool together make up a "federation bundle". Most places deploy both both "bundles" at the same time, or even as part of the same stack. But it is reasonable to ask users to upgrade everything in a "node bundle" together, at least for now.
So we'll close this issue, retire the merged repo, and discuss our versioning and DX improvements in other issues.

Closing, and continuing the conversation in https://github.com/neurobagel/planning/issues/190


Scp 4572 Fixed a balancing bug in Marlowe transaction creation

@bwbush This nix-code-formatting test failure, is it indicating that I need to updateMaterialized? I did add new files to this repo.


Reference known issues and solutions in README

👍  Good idea, I would also little bit expand README - minimum PHP Version would be nice to have there.


Image galleries on page bundles do not work

First of all, congratulations for your great work and for the quality of this theme.
I am writing because as I am reading from release notes of v.3 content should be placed in page bundles. If I am not wrong, from what I've read at hugo's documentation page bundles are aimed to place all the content related to a certain content together. This includes the content itself (index.md), featured image in academic theme (featured.png/jpg) but also any asset of files or images and translated content. Sounds quite a sensible approach.
Unfortunately, image galleries don't seem to work with files located in page bundles, as only images located in static/img or external urls seem to be accepted (see documentation).
As current implementation is not very consistent (some files need to be placed in the page bundle whereas others should be placed elsewhere), would it be possible to fix that or remove the need to arrange content in page bundles?

See #708 - adding support for gallery images in page bundles is already on the roadmap. Happy to review any contributions for this.

Thank you!


Setting up primary and secondary logins - Cannot login with the email address

Description:
Document: Maintaining Logins and Passwords
Section: Setting up primary and secondary logins
<LoginConfig>
    <UserIdLogin primary="true">
         <ClaimUri></ClaimUri>
    </UserIdLogin>
    <EmailLogin primary="false">
         <ClaimUri>http://wso2.org/claims/emailaddress</ClaimUri>
    </EmailLogin>
 </LoginConfig>

As per the doc, when the above configuration is enabled in api-manager.xml file, user should be able to login with the email address as well as the username.  But once user tries to login, it is not allowed and getting the below error message.
ERROR - APIStoreHostObject Invalid tenant domain.
Suggested Labels:
Type/Bug
Severity/Critical
Priority/Highest
Suggested Assignees:
N/A
Affected Product Version:
apim-2.1.0 update 6
OS, DB, other environment details and versions:
OS: macOS High Sierra
Browser: Google Chrome - Version 63.0.3239.132 (Official Build) (64-bit)
apim-2.1.0 update 6
am2xx docs
java version "1.8.0_144"
Config changes in APIM pack except above: /repository/conf/tomcat/catalina-server.xml, compression property is set to 'off' due to the issue of [1](mail thread)
[1] - [APIM]Cannot login to APIM Management console with admin credentials
Steps to reproduce:

Enable login config property in <APIM_HOME>/repository/conf/api-manager.xml file as above.
Start the APIM server and navigate to API Store.
Self-signup with following details.

username: Ann123
password: Test123
First name: Ann
Lastname: Ann Jay
email<EMAIL_ADDRESS>

Try to login with email and password.

Expected: user should be able to login.
Actual: An error message pops up.
ERROR - APIStoreHostObject Invalid tenant domain.
Related Issues:
Mail thread: Setting up primary and secondary logins

This feature is not supported in APIM. Docs have been updated to leave out 'Setting up primary and secondary logins' part. Anyway the error message you saw is because APIM identified gmail.com part in email as a tenant domain.


Descendants on multiple members

Unable to build the DESCENDANTS mdx expression against a set of multiple members.
hierarchy_set = MdxHierarchySet.descendants( MdxHierarchySet.members([Member.of('Dimension', 'Hierarchy', 'Member1'), Member.of('Dimension', 'Hierarchy', 'Member2')]).to_mdx()

getting error:
AttributeError: 'ElementsHierarchySet' object has no attribute 'unique_name'
mdxpy\mdx.py:869: AttributeError

Hello, I tried to solve the issue.
This is what I did:
Modify the DescendantsHierarchySet class to handle multiple members and update the MdxHierarchySet.descendants method to support sets of members.
You can review changes in this commit: https://github.com/jjyutm4532/cubewise-code-mdxpy-51/commit/ccd54563bd8b73d1d9153054e597573f091747d2.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


🔮 Stable Release

Speckle Blender is currently in Alpha - let's get it to stable release!

[x] #56
[x] #60
[ ] maybe #3 ? this is prob a big task tho so it may need to wait if we're keen to push this to stable asap

all done and building now!
closed by #60 and #78


Command R and R+ support

Hello! first off thanks for the hard work!
is there plans to add Command R and R+ support?
I am attempting too but im a little out of my depth.
Thanks again!

Hi @Steel-skull, thanks for asking - we're looking into MoEs, watch this space!


Incorrect representation of permissions

The PermissionsChecker returns error if a path has more permissions than required.
Fx. storage/logs/ has 0777 as the permission, which means read+write+exec for all.
But in config/requirements.php it is configured for permission 0775, which is what is required to remove the error.
It would be more correct if it would only return error if an item in the permissions array had less permissions that required. Fx. 0644 where 0775 is required.
Can be reproduced on any linux based webserver that is not opened as the current user (prebuilt LAMP stack). Tested with Arch/Manjaro Linux and Debian 7.6

Thanks for talking time to write this issue.
I am not quite i understand what you meant, correct me if i am wrong but this :
if(!($this->getPermission($folder) == $permission))

Should be :
if(!($this->getPermission($folder) >= $permission))

I am waiting for your feedback so i can update the class.
Thank you.

I can confirm that the change is working in both test systems.

Fixed.
Thank you.


now-cli not uploading tsconfig.json

Hi! I have a pretty basic typescript project https://github.com/XLNT/at-p which uses micro and now.
It was previously not build my typescript (using the npm deployment type) because when it ran npm run build the tsc command just output its help information (which is what happens if there's no tsconfig.json file in the current folder.
This was fixed in this commit: https://github.com/XLNT/at-p/commit/89a676b933db1506405a2ba47257e07624915ec3#diff-96c813af76efae3c872399365c060e54 by adding the files key in now.json to directly whitelist src and tsconfig.json
It now builds correctly. My tsconfig.json is not included in gitignore, so https://github.com/zeit/now-cli/issues/1129 isn't the same issue.
I'm hoping that I can remove the files key in now.json in the future, since I wouldn't expect it to be necessary.

I have the same problem.

Thank you so much for the effort you have put into this issue! 😊
However, since we completely re-worked the entire codebase of Now CLI and this issue does not apply to the new code anymore, I'm closing it now.
You can read more about our biggest release yet here: https://zeit.co/blog/now-2


Add specs for rb_str_times

rb_str_times is defined in https://github.com/ruby/ruby/blob/b6639a84095b2d5a829d2a277716d9e7c64db81b/string.c#L1830-L1885.

It would be interesting to share some logic with String#* since it's the same thing, just the C version of it.
This could be achieved with a shared spec under shared/string, as for instance https://github.com/ruby/spec/blob/master/shared/time/strftime_for_date.rb.
Then the 2 usages would do something like:
it_behaves_like :string_times, :*, lambda { |str, times| str * times }
it_behaves_like :string_times, :rb_str_times, lambda { |str, times| @s.rb_str_times(str, times) }

and then @object in the shared spec refers to that passed lambda.
Do you think you could do that?
It would improve both specs.

@eregon Thank you for your suggestions. Sounds nice. I updated the patch to share examples between two tests. Could you please take another look?

This look great, thank you for your work!


Links to changelog are dead

It looks like this was removed in 95b8f01bc9495964cadc5126917ddc54fd716d09
According to readme you are supposed to look at the changelog to figure out what version of the lambda layer to use. I'm assuming this should be updated to look at GitHub releases but want to double check

Thanks for catching this, the link to the removed CHANGELOG has been updated to github releases


[Bombastic] Sign SBOM on publish

Bombastic needs an identity which it can use to sign an SBOM when published. This would also provide an audit log.

Is there a SPIFFE/SPIRE task hiding behind this one?

Should there be some kind of "workflow" attached to that? Like: developers can upload SBOMs, but "someone-special" needs to approve the signing process?

I think bombastic the 'infrastructure' is the signer, so it's not done on behalf of some external/special entity. I think of it primarily as a way to say that 'bombastic accepted and stored this artifact', nothing more.

Basically I'm thinking of this in the same way crates.io, maven central or any repository would sign the package index.


Adding the Budapest event

"This year, registering your event at coderetreat.org is easier than it has ever been!" >> In what universe is this process easy? It couldn't be more complicated... The UX guy in me just died of a heart attack...

Hi @devill, happy to hear your suggestions on how we can make it easier next year ☺👌. So far, the transparency and semi-automation with github and Travis proved themselves to be a sustainable way of processing the over hundred events we registered this year.
In any case,  Your pr looks good, welcome to #gdcr17!

I'm happy to give advice :D I would create a simple HTML form that sends a POST request to a server, validates it, and saves the data into a database :) To make editing existing events easy, let's use GitHub as a login provider. Google maps can be integrated with the form so that when the user types the address lat/long is automatically suggested.
I'm happy to write that code for next years event. In fact, I just have to extract it from the Lean Poker app, since I already wrote that code :)

Actually here is the event registration for Lean Poker: http://live.leanpoker.org/events/create

By the way, sorry if I came off as a bit arrogant. I very rarely do that, and usually realize just a few minutes later that I did it again. It's just that I was a bit surprised by this approach.
The HR person organizing the event had no clue how to do this, and she had to ask me. (I will be facilitating, but the organization is done by the HR person at the sponsor's end.) It might look like a funny solution to use GitHub for this process, but you have to keep in mind that many events are not organized by the developer facilitating the event. Registering the event should be accessible for non-technical people, and pull requests are not common knowledge outside our bubble.


[loc][Query][DevDiv] Please clarify "running"

Hi,w e have a query from our translator and I was hoping you could help. Thanks!
source:
running
File Name:
src\vs\workbench\parts\debug\browser\debugViewer.nls.resx
Resource ID:
".resx", 0; "running"
Query:
Does it indicate progress or state?	Cristina Carrasco	6/9/2016 9:16:18 AM

'running' indicates state in this case - I have added a comment to the nls.call to make this clearer in the future


Create a convert command

It should be able to point to an ambient module, like in DefinitelyTyped, and pull it into an external module definition. Or specify as ambient output and pull into modules.
typings convert tape.d.ts --name tape
typings convert node.d.ts --ambient

What is the second command supposed to do?
typings convert node.d.ts --ambient

node.d.ts is already ambient, right?  I think I'm misunderstanding something simple.

@cspotcode Don't worry about it, it is pretty unclear. Currently, if you're using DefinitelyTyped, it's one giant declaration. However, it could be 11 files where every module inside the declaration gets split into their own files and the one ambient imports and declares them all. This is useful, for example, when someone writes a module that replicates some of the existing node typings (e.g. graceful-fs or node browser polyfills for Browserify) without having to redeclare the entire fs module interface.


Implement MuSig and test against Taproot channels

t would be nice to support Taproot channels in time for lnd 0.17 release.

Duplicate of https://github.com/NYDIG-OSS/lndsigner/issues/3


APL documentation links

Regarding APL study session 4, this adds APL documentation links to nbprocess.
I'm not sure exactly if this is the best way to do it, for example how to disable it a specific project.
It does work for local tests, I couldn't test the actions and automatic deployment to conda and pypi
Cheers, Johannes

Oh good idea @jaidmin ! Hey does mk_index.py need to be updated too? Currently I think it might raise an error due to nothing being in the dict? I assume you created the index by hand? If so -- that's fine, but do we need something in mk_index.py that checks for missing dict entries and skips them?

Hi @jph00 I did create the index with python. It's simply a requests.get and then parse both tables.
Yes indeed, mk_index.py would error now, as there is no APL entry.
We can either skip missing entries, or add another generator that parses the APL site. I don't think it's really necessary because the symbols are probably not going to change, unlike the documentation for the other libraries.
What do you prefer?

Let's have the index creation included here, because actually it turns out that they do make changes on many minor version upgrades

Actually I've decided to merge this - thanks for the mk_index fix. Perhaps you could do a PR which just includes your .py script which we can manually run whenever we need to re-build the APL index?


#1 - header has gap at right side

it is fixed now,
margin: auto -10px; was changed it margin: auto 0px; for header in App.css


GISAID ingest keeps running out of memory

These workflows:

https://github.com/nextstrain/ncov-ingest/blob/master/.github/workflows/fetch-and-ingest-gisaid-master.yml
https://github.com/nextstrain/ncov-ingest/blob/master/.github/workflows/ingest-gisaid-master.yml
and any that run https://github.com/nextstrain/ncov-ingest/blob/master/bin/ingest-gisaid

keep running out of memory on AWS and being killed, e.g. https://github.com/nextstrain/ncov-ingest/runs/3764464129?check_suite_focus=true.
This likely happens during the run of https://github.com/nextstrain/ncov-ingest/blob/master/bin/transform-gisaid since it takes gisaid.ndjson (the raw GISAID full dataset, which is over 100GB) as input, and performs a bunch of operations on it.
To avoid continually increasing the resources we ask for on the batch job, here are some ideas:
@tsibley said:

We should also understand why the memory needs increased even though the core of the ETL is streaming, and maybe also consider running these on m5 instance family instead of c5.
(which could be as small a change as adding m5 instances to the job queue used.)

@rneher said:

there are some lengthy compression steps that could happen in parallel to the rest of the pipeline.
the gzip compression is about 1h. Changing the compression of the ndjson to xz -2 already saved a lot.

Last increase in memory we ask for was earlier this week: https://github.com/nextstrain/ncov-ingest/commit/5db5d2574210d60b4ae6434248af64dd3187a781. Maybe we should raise it again for now while we implement a more scalable solution?


Special target option value to math the default behavior

Addresses issue #2224

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


selectedItem not being updated

Hi,
I added a watcher to watch when the carousel slide changes.
$scope.$watch('selectedItem', function(newVal,oldVal){
alert(newVal + ',' + oldVal);
});
However, when I slide the carousel, this function never kicks off and selectedItem remains unchanged.
This function only kicks off once initially when I load the webpage (when my js code initially set
selectedItem = items[2]; )
Is there something that I'm doing wrong? I'm trying to find codes in morphCarousel.js that sets selectedItem when the carousel is slided, but I'm not sure where....
Any help is deeply appreciated!

Yea, you are not working in the right way with the scope of your controller.
You should do it like this:
$scope.data = {
        selectedItem: $scope.items[2]
};

$scope.$watch('data.selectedItem', function(newValue){
        console.log(newValue);
});

<morph-carousel data-items="items" on-selected="data.selectedItem" show-value="number"></morph-carousel>


Add Ukrainian translation

Description
As a native Ukrainian & Russian speaker, I want to add translation in Ukrainian here (and also, spellcheck the Russian version)
Context
I think, it's very cool for anyone, when you can find some doc / guide in your native language
Possible Solution
If it's suitable, I can create a PR with these changes, as my Hacktoberfest contribution to OpenEBS

This sounds good. Please go ahead.
Need to find someone to get this reviewed :) cc @muratkars @kmova

Thank you @daniloff200
You can find my first attempt to translate the README.MD to Russian here which is out-dated and needs update as well as all other MD files.
You can place them under openebs/translations/ directory following the FILENAME.ru.md format.


Changing Permissions of Workers at Nonprofit

Organization is White Council, I am Gandalf Grey, who is an Admin. I have a worker, Peregrin Took, and I try to set permissions to Admin as well so that he can see our clients' documents. See that "Save Changes" feature is coming soon. OK, but this also means that right now nonprofit organizations beta testing cannot change their workers' viewing permissions and neither their own viewing permissions (I can also not change my own permission to Worker).

Related - when I click View Profile under the Client, it redirects me 404 to the home page logged out. This makes sense, because I don't have the permission to see Gollum's profile. But I'm an Admin, right? So I should?


[BUG] Summons Not Working: "RangeError: Maximum call stack size exceeded"

Describe the bug
No summoning spell or ability works. The developer's tools console log shows the following error:
summons-actor.mjs:214 Uncaught (in promise) RangeError: Maximum call stack size exceeded
    at SummonsActor._cleanRollData (summons-actor.mjs:214:23)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
(...skipping a huge amount of lines...)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at SummonsActor._cleanRollData (summons-actor.mjs:221:6)
    at Actor5e.getChanges (summons-actor.mjs:114:35)
    at summon (summon.mjs:53:1)

To Reproduce
Steps to reproduce the behavior:

Open the browser's developer's tool to see the console output
Use any spell/ability following the instructions
See error in the console

Expected behavior
The summon to be placed in the canvas
Screenshots

Environment:

Foundry Core Version: 0.12.325
Module Version: 1.1.6
Host: self hosted in Oracle Cloud (Linux)
OS: Tested on MacOS, Windows and Fedora
Browser: Tested with the latest versions of Google Chrome, Microsoft Edge and Mozilla Firefox

Warpgate doesn't support V12 and has ceased development, plus all of the module features have been integrated into the DnD 5e system, so there will be no V12-compatible version of the module.


Pause function

Description
Function to catch Ctrl-C and ask user if they want to pause execution. Seems to work fine. Could use some eyes.
Type of change

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

Test Configuration:

Device Model or Emulator: ONEPLUS A5010
Android Verison: 9 API 28
Instagram Version: <IP_ADDRESS>.119

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 tested my code in every way I can think of to prove my fix is effective or that my feature works
[X] Any dependent changes have been merged and published in downstream modules

Shit, I obviously haven't slept enough to be doing this. My apologies.

Cool idea!
Pressing Ctrl-C now effectively pauses the script. The choice should obviously be "Continue or Quit [C/Q]: ". It makes no sense having to press P for Pause, when the script is already paused, and having to press P and THEN [RETURN] to have the script continue instead of quitting.

It’s fine. I was gonna change that anyway.🙂

Great! Don't drink and PR, man.


Remove legacy unknwon/com package

Follows:

#19284

Changes:

The CopyDir is only used inside test code
Rewrite ToSnakeCase with more test cases
The RedisCacher was not well designed, it only put strings into cache, here we use toStr function to replace the legacy ToStr
The UniqueQueue can use string as ID directly, no need to call ToStr

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@58c9d12). Click here to learn what that means.
The diff coverage is 51.08%.

@@           Coverage Diff           @@
##             main   #19298   +/-   ##
=======================================
  Coverage        ?   47.42%
=======================================
  Files           ?      942
  Lines           ?   131356
  Branches        ?        0
=======================================
  Hits            ?    62294
  Misses          ?    61560
  Partials        ?     7502

Impacted Files
Coverage Δ

cmd/manager_logging.go
0.00% <ø> (ø)

modules/cache/cache_redis.go
2.15% <0.00%> (ø)

modules/nosql/manager.go
46.42% <ø> (ø)

modules/util/legacy.go
40.00% <28.57%> (ø)

services/webhook/webhook.go
48.81% <33.33%> (ø)

models/unittest/fscopy.go
37.70% <37.70%> (ø)

models/unittest/testdb.go
15.62% <50.00%> (ø)

modules/sync/unique_queue.go
35.84% <54.54%> (ø)

modules/context/csrf.go
64.82% <55.55%> (ø)

modules/util/string.go
89.65% <89.65%> (ø)

Continue to review full report at Codecov.

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


assistants API file upload

Added a new cli command process-repo-files which clones the repo, switches to the right commit hash, and iterates [sequentially for now] over all the files, uploading them to assistants-api. This gets us embeddings and RAG for free:
python -m swe_bench_util process-repo-files

I clone to /tmp for now, not sure if that's what we want.
In the end it dumps a list of file_ids that you can give your assistant for searching.
If this is interesting I'm happy to help optimize how assistants chunks and embeds for our purposes.

Thanks @phact! Looks like a minor rebase needed. At first glance I think this is pretty good start, just except a couple notes:

It looks like this would clone a repo once per SHA, since we're going to loop through them that will take time and space. We only need one clone per repo and we can checkout (or do worktrees if we want to get fancy later) the different SHAs from that clone
I would prefer to checkout into a gitignored subdir of the repo like "checkouts" instead of tmp.

I'm getting this for most files, maybe will need to do something to make OpenAI understand these are text files
Dockerfile: Error code: 501 - {'message': 'Unsupported file type'}

Merging, going to structure a bit and make my notes a new issue


Added description of the "CIA Triad"

Added the text: "The CIA triad is a model used in Information Security to build sound policy. The acronym "CIA" stands for Confidentiality, Integrity, and Availability. These three items are the most important components of security."

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


'Add organization' as admin doesn't notify user with successful message

Steps

Visit https://staging.apinf.io
Login as admin
Click on 'Add Organization'
Fill the form with all the required details
Click on 'Create Organization' button

Expected Result

Notify user with successful message

Actual Result

User not notified with successful message

@bajiat It's really simple to fix. Can I take it on work?

@marla-singer Yes, sure, go ahead.

@bajiat , verified this resolved issue and it is not repeoducible.


Logging out of WSL after failed SSH results in error

Environment
Windows build number: 10.0.19041.0
Windows Terminal version (if applicable): 1.0.1811.0
WSL v2 running Ubuntu
Unmodified OpenSSH client running in WSL 2

Steps to reproduce

Open a WSL tab in the Windows Terminal.
Try to connect to any hostname via ssh that cannot be resolved, example:

$ ssh foobar
$ ssh: Could not resolve hostname foobar: Name or service not known

Exit the session via logout or ctrl+d to receive the following error message:

[process exited with code 255]

Expected behavior
The terminal window/tab/pane closes.
Actual behavior
The terminal window/tab/pane stays open displaying the aforementioned error message. In case of panes, this is problematic since they cannot be closed any more without closing the entire window.
I realize that this might be a bug in WSL but cmd and powershell don't show an error when exiting from WSL in this case.

AFAIK that's the default tab exit behavior: if the underlying process (last executed command) finishes with an error, then the tab shows this message with the error code. This can be changed with the "closeOnExit" option (e.g. "closeOnExit": "always").

Excellent, I was unaware of this option. I will close this issue then.


changed @zone_ns from string to array to enable multiple NS records for ...

...the domain.
I commented out the "validate_re($zone_ns..." in line 97 of zone.pp because I had no good idea how to solve this.
additionally fixed SOA record to print name of the primary NS (first value in array; see also http://www.ripe.net/ripe/docs/ripe-203).
So "bind::zone" should be
bind::zone {'test.tld':
  zone_contact => 'contact.test.tld',
  zone_ns      => [ 'ns0.test.tld', 'ns1.test.tld', ],
  zone_serial  => '2012112901',
  zone_ttl     => '604800',
  zone_origin  => 'test.tld',
}

when using this patch - which leads to this zonefile:
; File managed by puppet
$TTL 604800
@ IN SOA ns0.test.tld. contact.test.tld. (
<PHONE_NUMBER>  ; serial
      3h ; refresh
      1h   ; retry
      1w; expiracy
      604800 )   ; TTL
      IN NS ns0.test.tld.
      IN NS ns1.test.tld.

$ORIGIN test.tld.

Hello!
Thank you for the pull request. We had to correct our unit tests as they broke lately, it would be marvelous if you could rebase your PR and resubmit it.
Regarding the line 97, it's a bit tricky. I guess this test would need to be done in the zone template instead, for each ns record.
Also, you might want to update the unit tests related to the zone definition — else, we'll be unable to merge your modifications.
Thank you!
Cheers,
C.


Scrobbler doesn't grab the Spotify album info

When I'm scrobbling on Spotify, I'd expect the extension to grab the album metadata of the song I'm playing. However, it only grabs the song and artist, while the album field is left blank. I'd have to fill it with each song, but that's too much work for someone who listens to more than 300 songs a day while multitasking. Is it possible to resolve this?

I second that. Been having the same problem for a while now and can't really figure out wether it
if it comes from the last.fm side or the web-scrobbler (I use the firefox version btw)
Just now I have tried playing the same song both on web open.spotify and on the desktop (windows) app. As you can see here the first one did not scrobble album, but the second did.

However this is not always the case, as this album I played yesterday from the Web app was scrobbled correctly even though the web-scrobbler left the album field blank...
https://www.last.fm/user/nico_sasshi/library/music/Cloud+Control/Zone?date_preset=LAST_7_DAYS

On the other hand, all LCD Soundsystem new record scrobblings I've been doing from open.spotify are missing the album, so maybe it has just something to do with new releases that are not yet present in last.fm database, but then again Spotify desktop app (or playing tagged mp3 files from Winamp ) don't seem to be affected by the problem.
That said, album stats on last.fm are definitely skewed by this and I a fix would be appreciated.

There's no simple way to get album info from Spotify Web Player. This issue is partially resolved in c7b5bc84105bd894785f5f44b492678d3bb89d19; the extension will get album info using Last.fm API.

Closed as a duplicate of #1320.


Include command info and index in completion listener Promise value

Return command information and index in addition to the exit code in the command close stream and the completion listener Promise value. This way we can associate the exit codes with the commands when using concurrently programmatically.
Resolves #181.
I also tried if I could use another stream (something like closeWithCommandInfo) for just the additional information, so that I wouldn't need to change so many places, but I had some confusing errors in the binary unit tests, and in the end, having a single close stream seems more reasonable.

Coverage remained the same at 100.0% when pulling e904449a1373f161e7bb25b7bb4fa9cb5ce14333 on peruukki:include-command-info-and-index-in-completion-listener-promise into 54b6456eb5054ec32de9d81717b143d2ca8f8f6d on kimmobrunfeldt:master.

Coverage remained the same at 100.0% when pulling e904449a1373f161e7bb25b7bb4fa9cb5ce14333 on peruukki:include-command-info-and-index-in-completion-listener-promise into 54b6456eb5054ec32de9d81717b143d2ca8f8f6d on kimmobrunfeldt:master.

Release as v5.3.0!


PATMICK's png, won't load in

Describe your bug here.
when i try to load in PATMICK's json file, his png doesn't show but his json and xml are fine and his png is their too. is there any way this can be fixed?
Command Prompt/Terminal logs (if existing)
crash message when loaded: Null object reference

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
not really, i only added PATMICK's file stuff in

you probably didn't put in the right png name in the character json

hmmm

wait nvm I can look why it's crashing

it doesnt for me

send me the right and fixed one
pls

alr

mediafile

i guess

what about the stupid's week issue too

idk, might be an issue with your pc

ok, thanks for letting me know

you're welcome


KEP-1669: update docs for ProxyTerminatingEndpoints

Signed-off-by: Andrew Sy Kim<EMAIL_ADDRESS>Update feature gate references for ProxyTerminatingEndpoints.
https://github.com/kubernetes/enhancements/issues/1669

Hi @andrewsykim! This PR needs a doc review by Mon Nov 28th to get this into the release. Please reach out to required SIGs to get their review. Thank you!

@andrewsykim  please can you review the suggesting changes?
This PR needs a doc review by Mon Nov 28th to get this into the release. Please reach out to required SIGs to get their review. Thank you!

@andrewsykim please can you review the suggesting changes?
This PR needs a doc review by Mon Nov 28th to get this into the release. Please reach out to required SIGs to get their review. Thank you!

done, thanks


Generate default JSON object from JSON schema

Is it possible to generate a JSON object with default values from a JSON schema?
eg:
use schema:
{
    "title": "master data",
    "type": "object",
    "properties": {

        "Base": {
            "title": "base data",
            "type": "object",
            "options": {
                "collapsed": true
            },
            "properties": {
                "Value1": { "type": "number", "default": 1.1, "minimum": 0, "maximum": 1000, "title": "Value 1" },
                "Value2": { "type": "number", "default": 2.2, "minimum": 0, "maximum": 1000, "title": "Value 2" },
                "Value3": { "type": "number", "default": 3.3, "minimum": 0, "maximum": 1000, "title": "Value 3" }
			}
		}
	}
}

to produce valid JSON with default values:
{
	"Base": {
		"Value1": 1.1,
		"Value2": 2.2,
		"Value3": 3.3
	}
}

The actual schemas are more complex - also with array definitions and default array values - used for https://github.com/json-editor/json-editor
thx

I looked at doing this (and generating a schema from a type) back when draft 6 was the latest.  It gets really complex pretty quickly.  It's the *Of and not keywords mostly that make things difficult.
I'll add this as a feature request, though, and have a look at it again.

This also gave me an idea: #280.

Looks like this is a duplicate of #12.


Update the library for nestjs v8.0.0

The library is not compatible with the latest nestjs version. It is showing dependency conflict issues

please !!!!


Fix MayUseToken vulnerability in AccountUpdates

Description:
This PR addresses a security vulnerability in the MayUseToken field of AccountUpdate. The issue could potentially lead to miscalculation of total balance changes due to improper handling of token inheritance.
Key changes:
Changes

Added a check() method to ensure that parentsOwnToken and inheritFromParent flags are not both set to true simultaneously.

These changes prevent potential misuse of the API where a parent AccountUpdate might incorrectly treat a child as using the parent's own derived token when it's actually inheriting the token ID or using the default token ID.
Repo updates:
bindings: https://github.com/o1-labs/o1js-bindings/pull/279

Closed for https://github.com/o1-labs/o1js/pull/1741


Fix argument to work properly

Description
複数コマンド渡すときはこっちのほうがよさそうだった・・・
Fixes # (issue)
Type of change

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

How Has This Been Tested?

[ ] I have checked backward/forward compatibility that may cause regarding this change.

これでもまだ環境変数周りで問題が出てそう
kubectl logs -n monitoring mysql-o11y-27390897-pqrvs
Enter password: mysqldump: Got error: 1045: Access denied for user<EMAIL_ADDRESS>(using password: NO) when trying to connect

パスワードとかに特殊文字使っていると途中で区切られてしまったりするので、"$(MYSQL_PASSWORD)" でなおるかも？


How to update font size of an empty line?

Hello Tomas,
This relates to #148. I am trying to change the font size of an area's text and line number, so that they are the same.
I'm using the style-remapping approach (#142) to update the text's style. I learned that, after remapping, the area only updates the text and line numbers if the mapper returns a new Style object. I can return a new Style object for every line that has content or text, but I can't do that if it's an empty line.
For example, the sixth line below (a blank line) does not update when I use this approach.

Sorry for late response on this. Style information is only associated with text (that is, characters). No text means no way of specifying style. This will change when paragraph-level styles are supported (#6).
To change the default font size of the whole text area, setting the -fx-font-size property on the area should work. See the FontSizeSwitcher demo.
Binding line number font size to the value from your drop-down menu should work, independently of any style remapping. Do you mind sharing the code of your line number factory?

No worries! The lateness was an exception, not the norm. Thanks for the response.
I was using an InlineStyleTextArea that was initialized with a style object that could restyle the font size of the text. There was a second object that styled any text a user typed into the area. That was why I was using the style remapping approach.
However, if the style object's toCss() method specifies a font size, the area.setStyle("-fx-font-size:" + size) approach won't work. Once I removed the font size option from my style object's ````toCss()``` method, the above approach worked fine. Fortunately, since both of my style objects would style the text's font size to the same value anyway, this part of my style object is easily fixed.

The LineNumberFactory (LNF) below  was a quick test solution. The code below shows where I made minor modifications to yours and isn't the full class.
No doubt the code could be better optimized. If I use the styles remapping approach above, it only updates the label's font size in the LNF of the lines that haven't been rendered yet. Only when a user scrolls, so that the labels that were styled to the previous font size are hidden, and then scrolls again, so that these labels are shown again (and thus re-rendered), will it properly display the labels as the updated font size.
// DEFAULT_FONT was changed from a Font into a Closure (which is basically a method)
private static final Closure<Font> DEFAULT_FONT = { double size ->
        return Font.font("monospace", FontPosture.ITALIC, size)
    }

// the property used to determine the font size is held within a class of all static variables.
// when calling its apply method, I reference that property like so:
public Node apply(int idx) {
        Label lineNo = new Label()

        // 'UserSettings.fontSize.get()' was abbreviated to 'UserSettings.fontSize()'
        lineNo.setFont(DEFAULT_FONT(UserSettings.fontSize()))

        lineNo.setBackground(DEFAULT_BACKGROUND)
        lineNo.setTextFill(DEFAULT_TEXT_FILL)
        lineNo.setPadding(DEFAULT_INSETS)

To create a Line Number Factory that also updates according to a font size change, could a BiFunction<Integer, DoubleProperty, Node> be used instead?

However, if the style object's toCss() method specifies a font size, the area.setStyle("-fx-font-size:" + size) approach won't work.

Yes, the CSS returned from toCss() will override the CSS on the area, because the CSS from toCss() will be set on the specific Text node; standard CSS precedence rules apply, namely, CSS set on a child node takes precedence over CSS inherited from a parent.
If your style object has the capacity to override font size, but in a particular case does not (e.g. has null or empty Optional in place of font size), then the toCss() function does not need to return "-fx-font-size" in the result at all, letting the font size set on the area take effect.

include a Property that is used instead for determining the Default Font's font size, and thus

Yes, conceptually this:
lineNo.fontProperty().bind(UserSettings.fontSizeProperty);

but in order to make sure that the line numbers do not keep being updated (and thus be prevented from garbage collection) after they disappear from the scene, use this trick where the binding to fontSizeProperty depends on the label being part of the scene:
lineNo.fontProperty().bind(Val.flatMap(
        lineNo.sceneProperty(),
        scene -> scene != null
                ? UserSettings.fontSizeProperty
                : Val.constant(null)));

Same trick is already used in the original LineNumberFactory with observing area's total number of lines.

(Sorry for the late reply. I've been sick the past few days.)
The above did the trick. Thank you again!

No need to be sorry. Get well soon!


update and tweak

I have used more community plugins I think would go well with Ano

Addressed in:

Release v1.2.0.