[Palworld] XGP-Save-Importer?

Hello! Love this tool, and with Palworld being on both XGP and Steam I was looking into ways of popping a save back in forth to play with xbox and steam friends. Also want my steam progress on XGP as I didn't realize it was released there. I see that there was a previous issue where another user created a tool to do this for Starfield.
You mentioned at the end of that thread you're considering adding this script to your tool. Since I have no knowledge of the structure of the Palworld save file, was wondering if you think this is possible, and if you had done any work to add this to your script.
I'd be happy to help, but figured asking before doing work that may already be in progress.

Would love this as well. There are already save editors and other tools being made which can be used once the saves are extracted but no way to put them back in.

import.txt
So basically i tried to write python code to import it but somehow its not working at all and i dont know why. Would really appreciate any clues what to do.

Would love this as well. There are already save editors and other tools being made which can be used once the saves are extracted but no way to put them back in.

Oh and if ya want just to edit your gamepass save through steam editor just go to User/AppData/Local/Packages/(something that starts with PocketPair)/SystemAppData/wgs and choose last edited folder. Then change the file with long string of letters and numbers (copy that name before) to Level.sav and you can edit it with your program. Afterwards change it back to coppied name and all should work.
P.S. Remember toalways backup your save files before attemting to edit them

Would love this as well. There are already save editors and other tools being made which can be used once the saves are extracted but no way to put them back in.

Oh and if ya want just to edit your gamepass save through steam editor just go to User/AppData/Local/Packages/(something that starts with PocketPair)/SystemAppData/wgs and choose last edited folder. Then change the file with long string of letters and numbers (copy that name before) to Level.sav and you can edit it with your program.  This works perfectly, I can finally transfer my save from Steam to Xbox X again

Importing saves into Xbox is always a finicky process as the cloud save container binary format is not documented (I believe even in the NDA-gated Xbox developer documentation), and not all fields are reverse engineered. My importer should be considered a hack because of that. It might break the container.
For save editing for this game, you can just directly edit the files in the containers as they have the same format (unlike Starfield where Bethesda decided to store individual chunks in Xbox version), but you will need to play with the container format if you want to add new things to the container.
I haven't tried but probably you can create a new game in Xbox version, then replace the save files with the steam save (file names are difficult without tools, probably comparing the file size would work?)
I'm porting the Starfield importer right now.

Thanks @HarukaMa.
There's some open documentation from Microsoft regarding the XGameSave API, for example this and this, but it doesn't help much when taking a look at the "raw" files. There's also some tooling (like xgamesaveutil), but I've had no success using those with existing production containers.
As this tool currently stands, I'd like to keep it one-way until we have a deeper understanding of the data format and the requirements, in which case this tool could support easy two-way transfers. It might be that we can't create valid containers without some magic Microsoft code.

Yeah, those documents only described how to use it, but not the internal file structure. I believe the container is called Connected Storage, and there is another tool called gamesaveutil for that (docs here), but the XML file referred there is behind the NDA. However, as this tool is a part of Windows SDK instead of GDK, the symbols are available from Microsoft symbol server. I didn't try to look into it more though, as I wanted a clean room approach for my tool.

@HarukaMa I see that you have created the Palworld and Starfield importers, can you maybe create one for Forza Horizon 5 as well?


"Select All" hotkey not implemented

When using the search field the cmd+a "Select All" hotkey doesn't work. I never realized how much I use this to clear the field before searching again.

I'm here looking for the new beta, to fix the new issues but this small tick has bothered me before. Just commenting to confirm select all would be great

Works like a charm!  Many thanks!


Fix #440 by listening to event target on mobile

I believe this addresses #440 and #9.
The underlying issue with dragging on mobile seems to be related to this issue. This pull request implements this proposed solution, by simply listening to event.target if the event is a mobile event (ie event.touches).
This works in my tests, but obviously open to discussion.

Hey @davidchappy, thanks for submitting this PR. I'm going to close it and re-open as a separate PR as there are too many unrelated changes in this PR.


"This version of Mac-Morpho is contained in a single file for training and another for testing..." where could I find?

Hi!
read in the docs that:
 "This version of Mac-Morpho is contained in a single file for training and another for testing, unlike the original distribution. Many errors were corrected and many sentences were discarded, providing a more reliable resource. Also unlike the original, each line contains a sentence, and POS tags are appended to tokens after a "_".

So, I see those files are no longer in the repo, where can I find them?
Thanks and great job in nlpnet!
PS: http://www.nilc.icmc.usp.br/macmorpho/ is offline

Thank you!
The corpus should be available in the URL you mentioned. There seems to be some problem with the server, though... I'll see if I can get it back online tomorrow.

Great! Waiting for it. Thanks again!

The site is up now.

Probably not the best place to ask this, but is there a big reason for the annotations to not use the peen treebank tags ?

Penn Treebank tags were specifically designed for English; it wouldn't make sense to use them in another language. Mac-morpho uses a tagset tailored for Portuguese.


Managed Identity - Azure Event Hub

I tried to connect using Managed Identity, but that does not work.
MID has Azure Event Hubs Data Receiver assignment on the resource.
Is this documented anywhere? Is this supported?
SAS key and Entra Auth works like a charm.

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

ID: 6606f6c7-6243-13d6-9fde-15f387f0e331
Version Independent ID: 875d1e7a-abbc-6cee-e7af-0f9786769917
Content: Connect to Azure Event Hubs - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-azure-event-hubs.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @BakkerJan,
Did you follow the necessary steps in the following article for using a managed identity with the Azure Event Hubs connector?
Authenticate access and connections to Azure resources with managed identities in Azure Logic Apps

@ecfan Yes, but I'm not sure of it's supported. I'm using a consumption based Logic App.

@bakkerjan: Should be supported then, based on the article and the UX. Let's see what the moderator has to say.

@ecfan I have added all the required Event Hub permissions to the system managed identity. I even tried owner at one point, but the triggers keeps failing with error 500.

@BakkerJan This should be supported as previously suggested. Since this is technical issue and requires investigation on backend, I suggest opening a ticket with azure support team or post your question with full error message, and other details in Microsoft Q&A. The experts can assist you further with the issue.
This repo is mainly for doc-related feedback and questions. Let us know if you have any other questions.

For anyone reading this, having the same "issue" :
use the sb:// format, not the https:// (rookie mistake)


Upgrade gitlab-ci-multi-runner to v1.7.0

The gitlab multi runner 1.7.0 just released.
I tested the 1.7.0-rc1 and 2 since 2 weeks with your image, in my company.
Nothing to report so far, the multiple runners are really busy and never showed any sign of weakness :)

@sameersbn hi :) could you consider reviewing this PR please ?

Would be great to have this merged


initializer is inaccessible due to 'private' protection level error when not all struct properties are private

Description
Consider the following code:
struct Struct {
  var x: Int

  private var y: Int = 0
}
do {
  let _ = Struct(x: 0)
}

Despite x not being private, the "inaccessible initializer" error is emitted.
One fix is to change y to a constant so that it doesn’t contribute to the generated initializer:
struct Struct {
  var x: Int

  private let y: Int = 0
}

However, this is not always a desired behavior.
Solution
Generate an additional, internal memberwise initializer for x, or more fine control over which properties contribute to the generated init.

You might want to check out the discussion and references here:
https://forums.swift.org/t/pitch-synthesize-init-of-structs/63688

Using this also works:
private(set) var y: Int = 0


gcc-sh-elf-newlib build fails for GCC 10.1

CI in #31 revealed, that gcc-newlib-sh-linux step is failing (run#3651807217).
There seems to a problem with duplicate _errno symbols (see line 65617 of logs).
Here it is cleaned up a little bit:
$PREFIX/sh-elf/bin/ld:
  $PREFIX/sh-elf/lib/libc.a(lib_a-syscalls.o):(.bss+0x0): multiple definition of `_errno';
  $PREFIX/sh-elf/lib/libc.a(lib_a-reent.o):(.bss+0x0): first defined here

Below is a comparison of *_errno symbols in newlib libc compiled with GCC 9.2 and GCC 10.1:
$ # GCC 9.2
$ sh-elf-nm -g sh-elf/lib/libc.a | grep _errno | sort | uniq -c
     20          U ___errno
     17          U _errno
      1 00000000 T ___errno
      2 00000004 C _errno

$ # GCC 10.1
$ sh-elf-nm -g sh-elf/lib/libc.a | grep _errno | sort | uniq -c
     20          U ___errno
     17          U _errno
      2 00000000 B _errno
      1 00000000 T ___errno

Looks like _errnos are in BSS and not in COMMON section.
I'm pretty confident, that this is a reason: https://wiki.gentoo.org/wiki/Project:Toolchain/Gcc_10_porting_notes/fno_common
Although neither adding CFLAGS=-fcommon in make call or before configure seems to help.
For this reason we skip the gcc-sh-elf-newlib-linux for now, but it needs to be repaired.

@cr1901 -- Any ideas?

@mithro No, I've never seen this error before, and I would expect adding -fcommon to CFLAGS would fix it. This might require a dive into the autotools machinery to figure out why -fcommon isn't being honored.
Did you do a make distclean before adding the new flags?


Remove white top-border from header when maximizing Hyper

when maximizing the window on Windows 10, there is a one-pixel white border around the header. so I add the .hyper_main in line 135.

Thank you a lot for contributing : )

I like this theme so much, It's my pleasure to make it better.


Shorting with a price limit results in unexpected resulting short quantity

Currently, the quantity field is inaccurate when creating a short order with a price limit if that short order is executed at the price limit rather than at the feed.
This may be a UI issue with the quantity/collateral calculations, or it may require deeper changes.  Since the underlying function only takes the collateral amount and the price limit and not the desired amount of MIA to create, does the actual position opened depend on price of execution or on feed at time of order creation?

duplicate: #1532


Stack routing failures

expected:
when you start a stack with a public route like "http://go.mycounter.localhost.tv/" it routes to your stack.
actual:
the first time you start a stack things work.  However, subsequent stack starts frequently result in routing misses eg "no server found for stack=mycounter service=go" and previously correctly routing stacks sometime lose routing as well.

This issue can be replicated with most of the stack examples.

I'm on it


feat: highlight top langs

 Let me know if the color and styling looks good

i feel the colors are not consistent row by row? probably because some even rows have a basic gray background.

O yeah you are right, I am seeing it now. Lemme see what I can do

just to check: did you do pre-commit

pre-commit run --all-files


fix coordinate incorrectly displaying x: then z::

[Fixed coordinate bug]

Description 🗒
The coordinates on the map were incorrectly showing x: and z: changed it to x: and y:

Examples 📸

Related Issues 🔗

 Expand for Help

Have questions about the review or deployment process? View our contributing docs
Need additional help and want to chat in real time? Join our community Discord

By submitting this pull request, you agree to our code of conduct

This actually isn't a bug. The y-axis is used for height in the in-game coordinates and we display coordinates using their in-game axis labels.

That's my mistake, sorry.

No worries; it's definitely not what one would expect from a top-down map view.


fix tests colliding names

There are two tests with the same name (test_unicode_set_deserialize), so one of them does not run.

Coverage increased (+0.07%) to 97.836% when pulling 499d5acad576e29536934855c0d4802299e29863 on yedpodtrzitko:colliding_names into ce741ab2129bde469942c002246e9240b4b38920 on jlafon:devel.


Conditional Access?

I do not see any Conditional Access policies created by the enrollment of Autopatch. Is this not expected behaviour? If so why do I not see them? The requirement checks did not fail

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

ID: 0b585379-5f7d-61e7-cd7c-964ca6d9df72
Version Independent ID: 0b585379-5f7d-61e7-cd7c-964ca6d9df72
Content: Maintain the Windows Autopatch environment - Windows Deployment
Content Source: windows/deployment/windows-autopatch/operate/windows-autopatch-maintain-environment.md
Product: w11
Technology: windows
GitHub Login: @tiaraquan
Microsoft Alias: tiaraquan

@wUEZRs
Thank you for your feedback. Conditional access policies are no longer created in the Autopatch service as of August 15, 2022. We have updated our documentation to reflect this change.


Look into bugs with new server smoke tests

Seems related to https://github.com/denali-js/denali/commit/ad7d25ac613a96ebc33a207205a623806386e8db I'm seeing broken tests hitting on my end.
  1) server command "before all" hook:
     Error: ENOENT: no such file or directory, symlink '/<path>/open-source/denali-js/denali' -> '<path>/open-source/denali-js/denali/dist/test/fixtures/cli/basic-app/node_modules/denali'
      at Error (native)
      at Object.fs.symlinkSync (fs.js:1054:18)
      at Context.<anonymous> (test/acceptance/cli/server-test.js:23:18)

  2) server command "after all" hook:
     Error: ENOENT: no such file or directory, unlink '/<path>/open-source/denali-js/denali/dist/test/fixtures/cli/basic-app/node_modules/denali'
      at Error (native)
      at Object.fs.unlinkSync (fs.js:1089:18)
      at Context.<anonymous> (test/acceptance/cli/server-test.js:27:18)

I also ran into this #97 after I did the rebase. You should check against #172

Should be fixed now - and snazzier!


Implement remaining Cyber Observables

In #4 we completed captured most of the CybOX 2.x types that are used in the AIS profile.  However, there are several additional CybOX 2.x types that are convertible to STIX 2.0 that we haven't covered:  e.g., various file extension, artifact, AS, X509, etc.

sigh - closing #69 as duplicate

Current state of CybOX objects in Elevator 1.0.0

STIX 1.x Type
in Elevator
STIX 2.0 (if different)

Account

User Account

Win User

Address
X

- IPv4
X

- IPv6
X

- MAC
X

- Email
X

Artifact

AS

Domain Name
X

DNS Query
not in 2.0

Email Message
X

File
X

- Archive File

- Image File

- PDF File

- Win Executable File

HTTP Session
not in 2.0

Link
?

Mutex
X

Network Connection
X

- Socket Address

- Hostname
?

Process

- Win Process

- Win Service

Product

Software

Port
n/a
Integer

URI						X

Windows Registry Key
X

X509 Certificate

Here is the way I would prioritize adding the remaining needed CybOX objects
1:  Account (et. al), Artifact, Process (et. al), Win Executable File
2:  AS, Archive File, Image File
3: PDF File, Product, X509 Cert

Closing, as there are separate issues for important objects.  Also the table here is out of date - see readthedocs for a more up-to-date list.


[Feature] UX Study follow up 2

[ ] Rename property  sentimentScores to confidenceScores
[ ] For LinkedEntity, rename Id to DataSourceEntityId and getDataSourceLinkId()
[ ] Rename the class SentimentScorePerLabel to SentimentConfidenceScorePerLabel to reflect swagger naming.
[ ] API Tool feedbacks: https://apiview.dev/Assemblies/Review/942b652edaf44b7189f41270cf07e54f
[ ] Delete all intermediate WithResponse suffixed and Update APIs
[ ] Unroll defaultCountryHint and defaultLanguage to builder API and remove TextAnalyticsClientOptions.
[ ] Remove InnerError
[ ] Change all input List to Iterable
[ ] Change all collection APIs to return PagedFlux for Async and PagedIterable for Sync APIs.
[ ] Delete DocumentResultCollection class and create TextAnalyticsPagedResponse class to include modelVersion and TextAnalyticsBatchStatistics.

Consider also adding here: https://github.com/Azure/azure-sdk-for-net/issues/9985

This issue addressed in #8391 and #8301


What exactly does this app do?

Sorry for posting this here, I just really don’t understand what the purpose of the app is, can you sync music with other devices or something?

If you run a beets web server you can sync the music from your desktop computer to your android.
I'll change this to a task to describe this in the README.md

Ok got it, thanks for that maxammann!


Fixing targetproxy deletion to not return an error when it does not exist

Ref https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/issues/155
Also added tests to ensure that we do not regress.
The tests wont fail however, until we update the ingress-gce fakes to return NotFound as expected (details in issue)
cc @G-Harmon @csbell

/LGTM, assuming tests pass.
Feel free to merge when test comes back green.


Idea

I would love to see noclip accuracy get implemented into GDH
i think i would be very cool :)

Added in 3.0


Urgent Change: Add Quick Guide for Admin user on Monthly Report when using it. Make it Display all the time. No pop-ups

Make font color - grey

FInished


Azure Image builder fails while performing CIS Hardening script with cleanup failing Error uploading file to $env:TEMP\winrmcp-xxxxx

Environment: Azure Image builder
Packer Version: [INFO] Packer version: 1.8.1 [go1.17.8 linux amd64]
while executing the CIS hardening fails with below error.
[9def6e3b-8c07-49b5-95f9-6dd79e4e7b68] PACKER ERR 2022/12/22 12:19:45 packer-provisioner-powershell plugin: failed to upload the remote cleanup script: "clean up script "c:/Windows/Temp/packer-cleanup-63a44abb-d76b-86d9-71a9-7005cab80e63.ps1" failed to upload: Error uploading file to $env:TEMP\winrmcp-cbccd5ef-5991-4bf6-7c70-4d4761743931.tmp: Couldn't create shell: http response error: 401 - invalid content type"
[9def6e3b-8c07-49b5-95f9-6dd79e4e7b68] PACKER OUT     azure-arm: PSParentPath  : Microsoft.PowerShell.Core\Registry::HKEY_USERS.DEFAULT\Control Panel\International\User Profile
[9def6e3b-8c07-49b5-95f9-6dd79e4e7b68] PACKER ERR 2022/12/22 12:19:45 packer-provisioner-powershell plugin: Retryable error: clean up script "c:/Windows/Temp/packer-cleanup-63a44abb-d76b-86d9-71a9-7005cab80e63.ps1" failed to upload: Error uploading file to $env:TEMP\winrmcp-cbccd5ef-5991-4bf6-7c70-4d4761743931.tmp: Couldn't create shell: http response error: 401 - invalid content type
====
Image Template reference:
Customize section:
        {
            "inline": [
                "Expand-Archive -Path c:\\scripts\\Hardeningfolder.zip -DestinationPath c:\\scripts"
            ],
            "name": "cisHardening script running-1",
            "runAsSystem": false,
            "runElevated": true,
            "type": "PowerShell"
        },
        {
            "inline": [
                "C:\\scripts\\Run-HardeningSteps.ps1 -config cis1-2019 -rootPath $env:SystemRoot -userDrive C -caption Companyauthorised"
            ],
            "name": "cisHardening script running-2",
            "runAsSystem": false,
            "runElevated": true,
            "type": "PowerShell"
        },

=====
Please suggest if we have to add any more switch or parameter to resolve winrm communication.

1 @p9139010 did you find a solution to your problem?

Yes @samaea. There were 3 things on hardening has to be corrected.
1.            Registry key SOFTWARE\Microsoft\Windows\CurrentVersion\Policies\System,LocalAccountTokenFilterPolicy  from 0 to 1 2.            Registry Key - SOFTWARE\Policies\Microsoft\Windows\WinRM\Service,DisableRunAs from 1 to 0 3.            Powershell customization should run as elevated to True 4.            But it shouldn't be run as system - since packer user account isn't SYSTEM. Run as System to False 5.            Check the system locale to be native as much as possible. we had to change System Locale from EN-GB to EN-US


Elements with onPress capture swipe events

I am currently using a FlatList and am having issues with touch events. I saw the ticket here and have been testing myself over the last few hours. The issue seems to arise from setting the sensitivity value to high. Just wanted to post this in case anyone else has issues in the future.

Bump. This is an issue. Swipeout will swipe when TouchableWithoutFeedback is commented out but will not swipe if it is not commented out
 <Swipeout
        // autoClose={true}
        sensitivity={40}
        style={{ backgroundColor: "white" }}
        right={swipeoutBtns}
      >
        <TouchableWithoutFeedback onPress={() => this.selectResult(index)}>
        <View>
          <Results result={item} />
        </View>
        </TouchableWithoutFeedback>
      </Swipeout>

@mikeKane Have you tried removing the sensitivity property?

Yes. Is it working for you now?
<Swipeout
        autoClose={true}
        onOpen={(sectionID, rowID) => {
          this.setState({
            sectionID,
            rowID
          });
          close = true;
        }}
        onClose={() => console.log("===close")}
        scroll={event => (close = false)}
        style={{ backgroundColor: "white" }}
        right={swipeoutBtns}
      >
        <TouchableWithoutFeedback onPress={() => this.selectResult(index)}>
          <View>
            <Results result={item} />
          </View>
        </TouchableWithoutFeedback>
      </Swipeout>

That is my latest code.

Yeah removing sensitivity fixed my issue.

This issue wasn't solved for me by removing sensitivity. Are we able to reopen this?


No translation for the second-line of forecast

Fore some reason the second-line forecast is not translating.
My request: http://wttr.in/Кельн?lang=ru
Response:

My request: http://wttr.in/Москва?lang=ru
Response:

My request: http://wttr.in/Калининград?lang=ru
Response:

My request: http://wttr.in/Магадан?lang=ru
Response:

My request: http://wttr.in/Краснодар?lang=ru
Response:

My request: http://wttr.in/Магадан?lang=ru
Response:

My request: http://wttr.in/Ставрополь?lang=ru
Response:

My request: http://wttr.in/Дербент?lang=ru
Response:

My request: http://wttr.in/Атырау?lang=ru
Response:

All of these strings have a common problem: they are not present in the ru.txt translation file. Most of these do have partial translations in the file, but since these are multiple-weather strings (separated by a comma), they are not translated.
Can you provide translations for those strings? Or alternatively, is the google translate translation sufficient?
If you do provide translations, the preferred format is this:
: [translated string] : [Original Text exactly] :

The main problem here (at least what I saw from the posted screenshots) was
that the comma-separated terms were not translated, and some terms were missing indeed.
I've translated them into German and Russian, and some of them into French, but I wan not sure about French for things like  Low drifting snow (posted them to #324).
@keinen87 Nurbek, thank you for the bug report; I hope it is fixed now; if you see it again, please let us know.

The main problem here (at least what I saw from the posted screenshots) was
that the comma-separated terms were not translated, and some terms were missing indeed.
I've translated them into German and Russian, and some of them into French, but I wan not sure about French for things like  Low drifting snow (posted them to #324).
@keinen87 Nurbek, thank you for the bug report; I hope it is fixed now; if you see it again, please let us know.

@keinen87 I've added the translations. Thank you for reporting!

Closing this in favor of #535


FND-341 - Complete the situational lattice for the 'long way around' the pattern

Signed-off-by: Elisa Kendall<EMAIL_ADDRESS>Description
Augmented the Parties ontology with two additional property chains that complete the 'long way around' the lattice for use in AML and other use cases
Fixes: #1479 / FND-341
Checklist:

[x] I'm familiar with the FIBO developer quide. My contribution meets all the requirements described there.
[x] My contribution follows the principles of best practices for FIBO.
[x] My changes have been reconciled with latest master and no merge conflicts remain.
[x] This PR is related to exactly one issue. The issue is referenced by using a GitHub keyword such as "fixes", "closes", or "resolves".
[x] Hygiene tests have been applied by a PR with "(WIP)" in title.
[x] The issue has been tested locally using a reasoner (for ontology changes).

@dallemang You're exactly right - that's what these are for, and this is the start of a series of revisions to better integration our properties, eliminate a few that are unnecessary, etc.  The ownership example will be done in the next round of updates (attempting to do this one pattern/example at a time).  Thanks!

There has always been some contention around whether (for example) owns/isOwnedBy links a relative thing to another relative thing or an independent party to another independent party.  This can be formally modeled in OWL, so we have to be clear about which one it is.   This should be a worked example in the next issue.

Can it not be both, or either ?

On Apr 14, 2021, at 11:48 AM, Dean Allemang @.***> wrote:
There has always been some contention around whether (for example) owns/isOwnedBy links a relative thing to another relative thing or an independent party to another independent party. This can be formally modeled in OWL, so we have to be clear about which one it is. This should be a worked example in the next issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/edmcouncil/fibo/pull/1480#issuecomment-819751538, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB72TFR57VPIZ2EAYZ56AMLTIXPRTANCNFSM4252O5CA.

That's where the contention comes in - logically, it can indeed be either (both doesn't make a lot of sense).   So we need a policy; e.g., we fill in one of each (and then we have to name all of them - ugh!), or we always fill in the relative/relative (respectively independent/independent); or we do either one, and document it (formally) in the model (if that's the case, we still have to do both in the template, which is what Elisa is working on now).   I don't have a strong opinion on which of these policies we want to recommend, but I do want us to be clear about whatever policy we choose.

@dallemang @tahoeblue When I update the ownership pattern, I'll also update the examples to make it clear.  Right now, it is definitely not clear as Dean says above.  I have to draw a bunch of pictures every time I add a new entity and want to include their ownership details to remind myself of how it works, and then re-run the reasoner to make sure it all worked properly as well as run a handful of queries.  And, if it takes me forever to figure it out, the likelihood of someone else being able to do so is quite low.  The pattern needs to be clean and the examples will reflect that once I've fixed it.

I am no doubt exhibiting my ignorance of specific semantic technology definitional details, but I’m not sure why both independent things and relative things can not each have “ownership" relationships.  I am quite open to be schooled on the topic, however ! :)

On Apr 14, 2021, at 1:08 PM, Dean Allemang @.***> wrote:
That's where the contention comes in - logically, it can indeed be either (both doesn't make a lot of sense). So we need a policy; e.g., we fill in one of each (and then we have to name all of them - ugh!), or we always fill in the relative/relative (respectively independent/independent); or we do either one, and document it (formally) in the model (if that's the case, we still have to do both in the template, which is what Elisa is working on now). I don't have a strong opinion on which of these policies we want to recommend, but I do want us to be clear about whatever policy we choose.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/edmcouncil/fibo/pull/1480#issuecomment-819798960, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB72TFVCTNCMZCLKRH4TPLLTIXY3JANCNFSM4252O5CA.

Your proposal (use the same property for relations between independent parties as between relative ones) is another possibility, and there is no technical barrier to it.  Elisa could indeed use the same property in both contexts.   The downside of this approach is that for someone who is already a bit confused about the difference between an independent and relative thing (which is everyone when they first come to FIBO; it's a subtle distinction), this invites confusion.
The point of my original comment was that whatever policy we choose, we need to document it clearly and use it consistently.  This PR is the place to make this point, since this activity is where we are designing the template that these things will be mapped to.

But @dallemang we removed classes IndependentThing and RelativeThing from FIBO Foundations since they were too academic and irrelevant; not to mention unhelpful and often leading to inconsistencies (since whether something is relative not infrequently depends on context and time frame).

IndependentParty is still there, and the pattern is still in use.     @ElisaKendall  can correct me if I am wrong, but my understanding of the "situational lattice" is that this is a specific pattern where we are using these things in a particular way.  my original comment was a request to document this clearly and unambiguously, so that we know how this pattern relates to specific instances of it  (like owns and controls) so that discussions like this aren't just abstract; we know how they look with relatiosnhips we  (and other users) are familiar with.

@dallemang That's correct, independent party is still in use, and 'ThingInRole' is essentially 'relative thing', and contract and situation are both 'mediating thing's.  Situation addresses the need @rivettp mentions with respect to context and time, which 'mediating thing' did not really do.


ticker+queue: run go mod tidy

Apparently my IDE runs go mod tidy on all subfolders that have a go.mod in them.
This is quite annoying since it always changes my git working tree to dirty.
So the easiest fix would be to commit the changes, which I hereby suggest.
I know, I should probably use a "real" IDE that doesn't do stuff like this but I really like IntelliJ...

The file queue/go.sum is not removed, just emptied.
I tried importing github.com/lightningnetwork/lnd/queue in an example project and it works as expected. It still pulls in the correct version of ticker as a transitive dependency.


shoal-server exits if download not successful

shoal was down, from the logfile it seems that a download of a new database failed.
shoal-server should continue running, print a warning, and possibly notify the admin, possibly per email to heprc.

Updated to show a warning message in log file that "logger.warning('GeoLiteCity database needs manually updating.')" instead of auto downloading it

fixed in latest version, closing


Materialbox removal of ancestor overflow-x/y

When running the materialbox code it makes all ancestors overflow:visible. It does not handle overflow-x or overflow-y cases, which becomes a problem.
https://github.com/Dogfalo/materialize/blob/b03030ed5c5ea2018424e2b6a73084f65e8ec63d/js/materialbox.js#L65

How would you want it to handle those cases?

If it would check the ancestors for
overflow: hidden;
overflow-y: hidden;
overflow-x: hidden;

This would eliminate most problems with ancestors having any kind of overflow defined.

So there are cases where you would want the materialbox to be clipped in either the x or the y direction? The reason this code was added was so that materialbox could "break out" of any overflow type container.

Sorry if I wasn't describing it properly. Materialbox IS clipping if you have overflow-x or y on a parent, which is the problem.


Why the method don't cast the ToString() automatically?

This issue has been moved from a ticket on Developer Community.

Why the method don't cast the ToString() automatically?
            fcSqlConnectionStringBuilder = new SqlConnectionStringBuilder();
            fcSqlConnectionStringBuilder.DataSource = host;
            fcSqlConnectionStringBuilder.UserID = "sa";
            fcSqlConnectionStringBuilder.Password = "[Redacted]";
            fcSqlConnectionStringBuilder.InitialCatalog = FcDb.FcDefaultDbName;
            fcSqlConnection = new SqlConnection(fcSqlConnectionStringBuilder); <-- Error, need write . ToString()

Original Comments
Feedback Bot on 9/22/2021, 00:02 AM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

As @roji mentioned, this is by design.


Decide upon and implement scivision installation mechanism for pre-trained model packages

For example, there may be repo packages that are set up with:

conda
python poetry

And there may be packages on repositories other than pypi like:

conda-forge

@quantumjot any thoughts? conda and mamba are quite famous package managers within the Earth and Environmental Science community. poetry is an emerging one which seems to be famous among computer scientists.

and perhaps the second part of this will be to add to the model scivision.yml config something that specifies the installer for scivision to use:
name: test
url: https://github.com/edwardchalstrey1/scivision-test-plugin.git
import: scivision_test_plugin
model: ImageNetModel
args:
    model_name: resnet18
prediction_fn:
    call: predict
    args:
        X: image
    kwargs: None
installer: pip

Or alternatively, don't have that installer flag, but just automatically recognise from the repo somehow which installer to use e.g. look for requirements.txt for pip, environment.yml for conda and pyproject.toml for poetry

This is issue is related to a new model repo for tree crown detection which uses a facebook library, detectron2, for semantic segmentation and object detection. The library depends on pytorch which seems to be a really conflicting one to install via pip, at least for linux OS.

Currently in installer.py we have the following code:
def _install(package):
    """Install a package using pip."""
    subprocess.check_call([sys.executable, "-m", "pip", "install", package])

So it should be a case of just adding extra install functions with the commands for conda or poetry

Another thing to consider here is that the current setup only installs from pip. It would also be useful to have scivision attempt to install a plugin package from the github repo directly, if it can't find it on pypi

We could add option to load model from binary or json

We can also add extras_require to scivision setup.py to install user-defined deep learning libraries.
e.g. pip install scivision[torch]


x/build/cmd/coordinator: ssh gomote proxy should support Windows buildlets

As of https://golang.org/cl/50750 the build coordinator has an ssh server to proxy to buildlets. (That was bug #19956)
We have Unix (ssh->ssh) and Plan 9 (ssh->drawterm) support, but no Windows story.
How do we get a "shell" (cmd.exe session) on a remote Windows machine?
https://technet.microsoft.com/en-us/library/cc732046(v=ws.10).aspx suggests there's a Windows Telnet Server.
Telnet would be perfect. We're already running in an isolated network, so internal security isn't important. The ssh server will still be secure to the public.
@johnsonj, without modifying all our Windows images, is there a programmatic way for the buildlet binaries to enable the Telnet server at runtime?

The feature needs to be enabled and service needs to be set to start (or disabled and manually started)
Install-WindowsFeature -Name 'Telnet-Server'
Set-Service -Name 'TlntSvr' -StartupType 'Automatic'

It may be tricky to run this powershell from buildlet.exe. I can add this to sysprep1.ps1 and build a new version of the windows images easily. Only one builder type isn't build off of these images today (windows-386-gce).

Excellent, thanks.
I wouldn't worry about windows-386-gce. That builder needs to die once we get the 2008, 2012, and 2016 ones booting to buildlet reliably.
Rebuilding the images works.
Feel free to do that whenever you'd like, if there's no way to run the powershell from buildlet.exe. We can't just run net start TlntSvr ?

Guess not:
$ gomote run -system user-bradfitz-windows-amd64-2008-0 net start TlntSvr
The service name is invalid.

More help is available by typing NET HELPMSG 2185.

Error running run: exit status 2

And MSDN docs say:
$ gomote run -system user-bradfitz-windows-amd64-2008-0 net start telnet
The service name is invalid.

More help is available by typing NET HELPMSG 2185.

Error running run: exit status 2

So I guess the code is there but not enabled to be enabled?

Oh, there's also the sc command to interact with services. No luck so far there.

Notes on disabling NTLM auth later:
https://www.petri.com/enable_telnet_on_windows_2003


default_site_template: adding the opportunity to generally choose a different site template

Hi,
I'd like to have the opportunity to put a different site template in for all sites. This would be the change to allow overriding this.
My main interests are not to cruelly reuse the rules folder and the append options and to have different confs included for the https and http vhost. I'd also like to include different confs for different sites, some of them are also provided by packages.
Perhaps the resulting template will be a separate Pull Request later, if I'll be successful in keeping the compatibility to your native template. But at first I'd like to simply override it for all my sites. ;-)
Kind regards,
Lars

hi & thanks for the PR.
The change is fine with me, I just changed the name to apache2_sites_template.


Use regular expression instead of String#endsWith() check

Alternative to #4826.

I loke this one better than #4826. Please merge once the checks are green.


Update to better test solution

This test solution follows the course guide but corrects the codes to be able to run.
The difference: hasDescendant.

@schordas Please kindly include the new changes in the codelab instructions. The instructions in the codelab do not tally with this final code solution and create errors. Thank you.


Run schema tests in CI

Fixes #2679

@alexcrichton mind putting out 0.2.78 after this lands?

Oh nice, thanks for checking for this!

Sure yeah I can do the bump later tomorrow


[Release 0.49] Switch to semantic DeepEquals()

This is a backport of https://github.com/kubevirt/kubevirt/pull/7135 which fixes some wonkyness we've experienced with our controllers recently.
Switch from reflects.DeepEquals to equality.Semantic.DeepEquals() across the entire project

Seems like a make run is needed:

oops, thanks. Fixed

/lgtm
/approve

/retest


[macOS] Add toolcache to the macOS 14

Description
Add toolcache to the macOS 14
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run macos12


Fix Pipelines

Codecov Report

Merging #472 (ad007c6) into main (4cca069) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ad007c6 differs from pull request most recent head bfdbf35. Consider uploading reports for the commit bfdbf35 to get more accurate results

@@           Coverage Diff           @@
##             main     #472   +/-   ##
=======================================
  Coverage   36.97%   36.97%
=======================================
  Files          73       73
  Lines        2718     2718
  Branches      512      512
=======================================
  Hits         1005     1005
  Misses       1711     1711
  Partials        2        2


Request new module: aro

Related user story
Provide the issue number for the user story related to this request
Type of module
terraform
If requesting a terraform module, what is the target cloud platform?
azure
What is the purpose of the module?
Provision an ARO cluster in an existing VNet subnet
What is the category of the module?
cluster

/group openshift


Cluster security config doesn't work properly with multiple clusters having same name

This is reported by @satishd while testing secure HDFS with secured HDP & HDF clusters having same name.
In point of user's view, users completely don't know about cluster id. They only know about cluster name, so they end up input cluster name.
In point of backend's view, cluster name is not unique (in other words, not distinguishable), as we can import multiple Ambari clusters having same name without rearranging cluster's name.
So we should make cluster name unique (user should input cluster name while importing then), or UI should show cluster selection which also shows Ambari URL, and change cluster security config to use cluster id instead of cluster name.

Please note that even if we show cluster name as dropdown and let users select one, only showing cluster name is not distinguishable for users too, hence we need to show another information of cluster. Maybe easier way is letting users put cluster name and not allowing duplicated cluster name.

@shahsank3t
We would be better to take same approach as source / sink hint.

change cluster name input field to drop-down
values of cluster name input field will be filled with available clusters in topology environment
cluster name input field should show additional information like Ambari URL
when user selects one and configure them, UI should pass the field with cluster ID rather than cluster name

Please let me know whether UI can handle above thing or not. If you confirm UI can handle that, I'll start doing the backend work.
Other thing to fix is, we need to put storm principal as default value instead of nimbus principal. I'll see it can be handled only with backend side.
Thanks in advance!

@HeartSaVioR
Yes, we can handle those points on the UI. Actually, 1 & 2 is already handled, we need to make changes for 3 & 4 points. You can start on the backend changes required for the same.
Regarding the storm principal to be a default, we actually let the user pick any principal/keytabs in the UI from the available list or create one themselves.

@shahsank3t
Unfortunately, backend side seems not be implemented properly...
Backend provides cluster lists (and their keytab/principal pairs) which has Storm as service, but it shouldn't have such limitation. Some users set their cluster without Storm, even though they're using Ambari.
An environment (namspace) should associate only one Storm service, so we can get them from that service and provide them for all cluster lists.
(Yes that's redundant but make no change for UI side.)
I'll try to address it from only backend side, but will mention you if UI change is needed.

@HeartSaVioR
Alright. Let me know if you need any changes. Assigning this issue to you and removing UI label.

@shahsank3t
3 and 4 are unrelated to the backend issue and should be addressed from both backend and UI side.
So if you want to label UI issue please re-add label.

@HeartSaVioR
Have made the UI changes for 3 (commit id: https://github.com/hortonworks/streamline/commit/62bd7c50d25cf85a3a706b3a0061d0a8809f4485)
Let me know when you are done with backend changes to accept clusterId instead of clusterName and I'll make the appropriate changes in UI.

PR for backend is ready: #704


Change the default color of days in the planning

I'm not able to change the default color of days in the planning. I'm setting the color to blue (#0000FF) in Line 6 for example and the color stays white. NB : it works for weekends, holidays...
timeFrames: {
                'day': {
                    start: moment('00:00', 'HH:mm'),
                    end: moment('23:59', 'HH:mm'),
                    working: true,
                    color: '#0000FF',
                    default: true
                },
                'noon': {
                    start: moment('12:00', 'HH:mm'),
                    end: moment('13:30', 'HH:mm'),
                    working: true,
                    default: true
                },
                'weekend': {
                    working: false,
                 color: '#CCCCCC'
                },
                'holiday': {
                    working: false,
                    color: '#F2D8D8'
                }
            },

I can reproduce the issue. It seems there's something wrong when trying to set a whole day timeFrame.

It works well when setting working: false property. I think there's something that doesn't work when a working timeFrame is larger than a non-working one.

You have to set time-frames-working-mode to visible.
There's a bug in timeFrame computations that gives an invalid color to some timeFrames.

it currently works if you set time-frames-working-mode to visible, and time-frames-non-working-mode to hidden.


build: allow building with dispatch and Foundation from the toolchain

This allows building without dispatch and Foundation build roots, which
allows for a multi-phase build.  This is intended to support the Windows
staged build where dispatch and Foundation are part of the SDK and
SourceKit-LSP is part of the devtools.

@swift-ci please test

@swift-ci please test


ng-admin debug mode

ng-admin currently compiled and minified with all possible dependencies. I think having some sort of debug mode, when sources of ng-admin are readable, could be very helpful.
@goliney is it possible to build ng-admin without minification or attach source maps somehow?

It's shipped with source-maps.
In order to be able to use source maps you have to enable them first:

Good, then we all set here.


GCNSVD predict()

As GCNSVD model will process the input adjacency matrix using truncatedSVD(), we cannot directly use the predict() function inherited from GCN.
This is my personal implementation,
 def predict(self, features=None, adj=None):
        self.eval()
        if features is None and adj is None:
            return self.forward(self.features, self.adj_norm)
        else:
            modified_adj = self.truncatedSVD(adj, k=self.k)
            features, modified_adj, labels = utils.to_tensor(features, modified_adj, self.labels, device=self.device)
            if utils.is_sparse_tensor(modified_adj):
                modified_adj = utils.normalize_adj_tensor(modified_adj, sparse=True)
            else:
                modified_adj = utils.normalize_adj_tensor(modified_adj)

            return self.forward(features, modified_adj)

@ChandlerBang I see your point, but I would still vote to include an overridden version of the predict function for GCNSVD, just to keep the consistency of interfaces among different models. If kept unchanged, users using the predict() function for GCNSVD may think they are getting predictions on the GCNSVD, but in fact they are getting results from GCN.

@jiong-zhu Yeah, that makes sense. I've updated the code as you suggested to avoid confusion. See details in commit https://github.com/DSE-MSU/DeepRobust/commit/000f86d124f75ce3b21e47588a79ffe24fb94bf7. Thank you for the advice, Jiong and Mark!

Thank you!


netstandard/neo4j v4/compatibility with Neo4jClient

Migrate to new version of neo4j syntax
Use same contractresolver than neo4jClient

I know that I did not take the time to do a nice PR, and it should have been separated PR.
As I'm rushing to launch our product, this was not my priority (and I certainly should have waited to do that PR).
For the CamelCase things :
In Neo4jClient, the default JsonContractResolver is not CamelCase (as your can see here )
In Neo4JClient.Extensions, by default the resolver was CamelCase, I switched it to using the DefaultJsonContractResolver of the GraphClient instead of an hard coded one (See this change )
If you don't do this, Neo4jClient and Neo4jClient does not use (by default) the same casing.

Yes, but in CypherExtensionContext I don't have access to neo4j current
GraphClient. Maybe I need to change the signature of the constructor of
this class to take the current GraphClient ?
I will try to look at it when my product is released.
Thank you
On Thu, Mar 18, 2021 at 11:00 PM Simon @.***> wrote:

Appreciated @kYann https://github.com/kYann - ok then instead of using
GraphClient.DefaultJsonContractResolver you should use the graphClient
instance JsonContractResolver property, otherwise if the graph client in
use has been set toCamelCasePropertyNamesContractResolver then you'll be
out of sync.
https://github.com/simonpinn/Neo4jClient.Extension/pull/30/files#diff-009515381c89ea310a3f146aae7e24aca34aff0e7aa8e5c3328ad9c2d8ff991eR23
Will need updating
tldr - If the user has set graphClient.JsonContractResolver = new
CamelCasePropertyNamesContractResolver(); then
GraphClient.DefaultJsonContractResolver will still be
Neo4jContractResolver since
https://github.com/DotNet4Neo4j/Neo4jClient/blob/master/Neo4jClient/GraphClient.cs#L42
is readonly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/simonpinn/Neo4jClient.Extension/pull/30#issuecomment-802333560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAY2WFWIQA254HXDAQTBIDTEJZXPANCNFSM4ZAOG7ZA
.

--
Yann


General: Replacement of the old translation feature

Checklist
General

[X] I chose a title conforming to the naming conventions for pull requests.

Client

[X] I strictly followed the client coding and design guidelines.

Motivation and Context

As discussed in this issue (#8561) and this PR (#8132), all the strings with format
<...>{{ 'artemisApp.xxx...' | artemisTranslate }}</...>
should be changed to
<... jhiTranslate="artemisApp.xxx..."></...>
Description

The strings with format <...>{{ 'artemisApp.xxx...' | artemisTranslate }}</...> have been changed to <... jhiTranslate="artemisApp.xxx..."></...>
It was important to only replace cases where there is ONLY {{ 'artemisApp.xxx...' | artemisTranslate }} between the tags since jhiTranslate removes everything between the tags.
Here are the regex expressions that I used for replacing:

For searching: (<(\w+)([^>]*)>)\s*\{\{ 'artemisApp\.([\w\d.-]*)' \| artemisTranslate \}\}\s*(<\/\2>)
For replacing: <$2$3 jhiTranslate="artemisApp.$4">$5

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Can we also replace the occurrences with translation parameters? Will this be done in a follow up?

What do we mean by "translation parameters"? Could you provide a code example please?

Good that someone clears up this mess for once :D. I checked it out locally and as far as I tested it (which is difficult as this PR affects basically the application in every place) all text I encountered was displayed correctly. Though, there seem to be some files that still make use of the artemisTranslate pipe. Like exam-status.component.html e.g. Probably the regexs didn't catch those as they have another pattern like {{"text" | translate }} {{ ... }} ...

I think that case should not be updated with jhiTranslate because there are at least two items in a tag in that case, and when we use jhiTranslate it removes everything between the tags. (as discussed here: https://github.com/ls1intum/Artemis/issues/8561#issuecomment-2107015124)
To check that case I used
{{ 'artemisApp.assessmentDashboard.pageHeader' | artemisTranslate }} {{ course?.title }}
 in assessment-dashboard.component.html. I converted it to
 {{ course?.title }}
, and I could not see the course title.


Add error handling for (*bytes.Buffer).ReadFrom and Pass new context to cleanup process

Also, add test cases for coverage

Codecov Report

Merging #13 (594c5af) into master (38efbca) will increase coverage by 2.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   72.37%   74.51%   +2.14%
==========================================
  Files           1        1
  Lines         409      412       +3
==========================================
+ Hits          296      307      +11
+ Misses         78       66      -12
- Partials       35       39       +4


Add support for uri linking per feature and scenario

Currently, dynamic tests don't backlink to a location of scenario or feature.

Here I can't locate a file where this feature is defined. It makes the whole process frustrating. It locates me to the main test file that defines those tests:

@dmi3coder is it something that can be handled by the extension? If so a PR would be welcome.


Add default repr for Model class

Add default __repr__ for Model to reduce boilerplate
Fixes #528

I'd like to add unit-tests but perhaps there is an issue?
#529

Codecov Report

Merging #530 into 2.2-maintenance will decrease coverage by 0.15%.
The diff coverage is 75%.

@@                 Coverage Diff                 @@
##           2.2-maintenance     #530      +/-   ##
===================================================
- Coverage            88.18%   88.02%   -0.16%
===================================================
  Files                   16       16
  Lines                 1007     1019      +12
  Branches                93       93
===================================================
+ Hits                   888      897       +9
- Misses                  83       86       +3
  Partials                36       36

Impacted Files
Coverage Δ

tests/test_model_class.py
100% <100%> (ø)
:arrow_up:

flask_sqlalchemy/__init__.py
75.82% <25%> (-0.43%)
:arrow_down:

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 80a7cce...3e2c899. Read the comment docs.

what's going on here? :)
should i finish this PR, or someone of you has taken it?

I've rebased against next release branch and am doing some further fixes. The test wasn't testing anything before, and we need to handle Unicode keys. I'll finish this when I get back from lunch.

+1 for unicode, this str(..) was a bit bold
Could you explain why test was broken?

assert instance.__repr__, "<SomeModel({})>".format(instance.pk) is really assert instance.__repr__, which is always true.

oh crap :) i should have put
assert instance.__repr__ == "<SomeModel({})>".format(instance.pk)

nope, repr(instance) - there's no reason to call __repr__ directly (and to call it you'd have needed () there anyway ;p)

yup, right __repr__ it's method but in this way.. it at least have failed :)


Correct PropertiesPolicyType formatting [10117]

Fixes failing nightly build, caused by an incorrect setting of the PropertiesPolicyQoS in the XMLTester.xml example code.
Signed-off-by: Ignacio Montesino<EMAIL_ADDRESS>

Build status:


Anagram Problem added under Python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
This is Anagram Problem pull request. This problem is used in my competitive coding exams and interviews. I am a GSSOC'21 participant and it would be really helpful if this pull request is accepted. I have written the problem statement for this and every necessary detail required for it.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#fixes #3711

I have added all the necessary changes. @HarshCasper, please review it.


intelli sense union type with spread

Search Terms

spread
union

Suggestion

intelli sense union type with spread
Use Cases

union type with spread is so great
Examples

import React, { SFC } from 'react'
import ReactDOM from 'react-dom'

const handleRef = (ref: React.ReactInstance) => {
  (ReactDOM.findDOMNode(ref) as HTMLElement)!.style.cursor = 'pointer'
}

export const Clickable: SFC<
  | {
      tag?: string
      onClick?: React.MouseEventHandler<Element>
    }
  | {
      tag: SFC | React.ComponentClass
    }
> = ({ children, tag, ...props }) => {
  Object.assign(props, {
    ref: handleRef,
  })

  // there will be error about `props` not incompatible
  return React.createElement(tag || 'span', props, children)
}

props type is
typeof props === { onClick?: React.MouseEventHandler<Element> } | {}

But actually when tag is string, it should be { onClick?: React.MouseEventHandler<Element> }, otherwise it should be {}, what means both of them should be acceptable.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

Could you try to write an example that doesn't use react? Hard to tell what part of this problem is the type system vs react types.

The error is on React.createElement which is defined in DefinitelyTyped's react.d.ts. You'd want to submit an issue/PR over at https://github.com/DefinitelyTyped/DefinitelyTyped/ - AFAIK the issue is simply that the react types have no overload which can handle a union of an SFC and a tag name string (perhaps reasonably so).

Here is an example without React, you can check whether is it working as intended.
interface Attributes {
  key?: string
}

interface HTMLAttributes extends Attributes {
  onClick?: (e: Event) => void
}

function createElement(tag: string, props: HTMLAttributes): string
function createElement(tag: Element, props: Attributes): string
function createElement(test: Element | string, props: Attributes): string {
  return test.toString() + props.toString()
}

interface Stateless<P> {
  (props: P): void
}

const content: Stateless<
  | {
      tag?: string
      onClick?: (e: Event) => void
    }
  | {
      tag: Element
    }
> = ({ tag, ...props }) => {
  createElement(tag, props) // emit error about type incompatible
}

However there will be no error as following:
const content1: Stateless<{
  tag: string
  onClick?: (e: Event) => void
}> = ({ tag, ...props }) => {
  createElement(tag, props)
}

const content2: Stateless<{
  tag: Element
}> = ({ tag, ...props }) => {
  createElement(tag, props)
}


feat(*): Improve Web Socket notifications and some log formats

This patch includes:

feat(api-server): Add config options whether to enable WebSocket server
feat(agent): Improve console log and WebSocket message format

@shimos Thank you for your review.
I fixed the PR based on your comments. Also, I added linting target to Makefile.
Please confirm the updated PR.


Parser lost zero point

<svg viewBox="0 0 2000 2000">
<path d="M0,0 H124.659 V71.565 H0 Z"/>
</svg>

Parser lost second horizontal point "H0" in the result array.

I just noticed this too. It may make sense for h and v with value of 0, but not H and V.
I'm working on a fix and will submit a PR soon

PR made: #2


ISPN-16645 Fix RemoteCacheMetricBinderTest flaky test

https://issues.redhat.com/browse/ISPN-16645

The test didn't fail on either CI.


nonce 0 got modified to next nonce....

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Do POST on '...'

Provide json you are submitting to the service (if it applies)

Then GET on '....'
Links to issues in other repos (if possible)

Expected behavior
A clear and concise description of what you expected to happen.
Environment (please complete the following information):

Staging or production?
Which chain?
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

this happened on our safe and multiple invalid transactions got created

  const staticNonce = nonce || MULTI_SIG_NONCE;

  const safeTransactionData: SafeTransactionDataPartial = {
    to: target,
    value: "0",
    nonce: staticNonce
      ? staticNonce + idx
      : await safeService.getNextNonce(MULTI_SIG),
    operation,
    data,
  };

I specified MULTI_SIG_NONCE to be 0 but safe didn't use it


⚠ Bofrost.be has degraded performance

In e593cb1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1347 ms

Resolved: Bofrost.be performance has improved in d36c381.


Merge features?

In May, I wrote my first PostCSS plugin, short. It shorthanded a bunch of properties, including position. Today I revisited short and started breaking it (which was really a plugin pack) into smaller packages like https://github.com/jonathantneal/postcss-position.
Then I saw we’ve got practically the same thing going on! So, I’d love to team up, merge the asterisk feature into your plugin, and then use this plugin in the next short plugin pack.
Before I make a pull though, does this interest you at all? I can keep them separate (and add a short prefix to mine) if you really think we’re doing different enough things.

I'd like to keep this plugin as close to spec as possible (ie: same logic as margin/padding shorthands only), if only because this is a part of Rucksack and we're trying to keep feature bloat to an absolute minimum. Especially since you can achieve the same thing as the asterisk with the proper property (auto). Sorry. But feel free to fork this and merge it as a new repo :-)


closes #256 - update review and challenge links

Currently, the curriculum page is redirecting to outdated links:

'Review Submissions':
teacher route is deprecated -> review route
'View Challenges':
student route is deprecated -> curriculum route

The correction on the curriculum page now uses:

the review route for reviewing submissions
the curriculum route for viewing challenges

This other PR covers this issue and then more, so merging that PR should address this one: https://github.com/garageScript/c0d3-app/pull/259/files


Teleportation to distant coordinates

This issue occurs when only Valkyrien Skies and addons are installed and no other mods

[X] I have tested this issue and it occurs when no other mods are installed

Minecraft Version
1.19
Mod Loader
Forge
Issue description
when you are on a created boat and break all its blocks without leaving it, the game teleports you to distant coordinates
Issue reproduction

create a boat
sit on it
brake all blocks without leaving
done

Logs

Having the same issue.

Same issue, happened on a multiplayer server by disassembling an airship comprised of only a steering wheel while diagonal and upon trying to reconnect, the game hangs on "Loading terrain..." while things like music and sound keep going.


Ignore anchors when using "Issue title contains page URL" mode5

I use autogenerated anchors for every heading on my site for a table of contents. When clicking such an anchor link the URL changes to example.com#anchor-title. When I now refresh the page (i.e hit F5), the comment section stays empty because utterances looks for an issue title with the correct #anchor-title. If I now post a comment a new issue is created for the same page.
To fix this, I would propose to just ignore any anchors when doing URL matching, or at least providing an option to do so. What do you think?

I agree, I think this is a bug. We should ignore both querystring/search and hash/anchor by default. Also need to normalize trailing backslash.
For example, this url in the address bar:
https://os.phil-opp.com/minimal-rust-kernel/?foo=bar&b#anchor

Should result in a search for issues whose title contains:
https://os.phil-opp.com/minimal-rust-kernel

Thoughts?

Sounds good to me! However, I think there are some sites that use the query string for determining which article to show (e.g. ?article_id=1234).
Maybe we can use the rel=canonical link when present and default to ignoring query string/anchor else?

I know that I'm a bit late, but thanks for fixing this so quickly! It works without problems now.


No such file or directory: 'save_model/fashion_iq/val_resnet_v2_50_ml/query_images.npy'

I am running bash scripts/run_fashion_iq.sh and falling to this error.
Traceback (most recent call last):
  File "test_val.py", line 220, in <module>
    main()
  File "test_val.py", line 48, in main
    query_images = np.load(filename)
  File "/home/huynhtruc0309/anaconda3/envs/truchlp/lib/python3.7/site-packages/numpy/lib/npyio.py", line 416, in load
    fid = stack.enter_context(open(os_fspath(file), "rb"))
FileNotFoundError: [Errno 2] No such file or directory: 'save_model/fashion_iq/val_resnet_v2_50_ml/query_images.npy'

Please tell me where to get this file.

And can you provide your checkpoint in the paper

Hi the missing file can be generated after you train the model.
You would have to run the model to get the checkpoint.


Slow replace with clipboard unnamedplus

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.operators
Description
When vim.opt.clipboard=unnamedplus, replace takes about 1 second to execute. I know this must be some problem with my system clipboard, but I cannot figure out what it is.
substitute.nvim does not lag at all.
Neovim version
0.10
Steps to reproduce
nvim -nu minimal.lua
local path_package = '/tmp/nvim/'
local mini_path = path_package .. 'pack/deps/start/mini.nvim'
if not vim.loop.fs_stat(mini_path) then
  local clone_cmd = {
    'git', 'clone', '--filter=blob:none',
    'https://github.com/echasnovski/mini.nvim', mini_path
  }
  vim.fn.system(clone_cmd)
end
vim.opt.rtp:prepend(mini_path)

require('mini.deps').setup({ path = { package = path_package } })

vim.opt.clipboard='unnamedplus'
require('mini.operators').setup()

Thanks for the issue!
I can not reproduce. Would you mind listing exact steps that makes you reproduce the issue on your end?

When vim.opt.clipboard=unnamedplus, replace takes about 1 second to execute.

With this information (specifically 1 second), my guess is that there is some conflicting mapping starting with gr. This makes Neovim wait after gr a 'timeoutlen' milliseconds (1000 by default, i.e. 1 second) until executing the currently typed keys.
What does :map gr show? It should show only mappings from 'mini.operators' (and possibly 'mini.clue' if you use it with g as trigger). Note that it is not :nmap gr, but :map gr, which will show available mappings in several modes.

I'm using the minimal.lua provided, so no additional mappings should be present.
Anyway:
x  gr          * <Cmd>lua MiniOperators.replace('visual')<CR>
                 Replace selection
n  grr           gr_
                 Replace line
n  gr          * v:lua.MiniOperators.replace()
                 Replace operator

I've tested that changing vim.opt.clipboard=unnamedplus to vim.opt.clipboard='' fixes the delay. So it must be some problem with my system clipboard, but I cannot figure it out...

I've profiled the mini.operators code and most of the time is spent in
https://github.com/echasnovski/mini.nvim/blob/ebc8cb0c1f73ee6c53e0560957a37a2307a70f98/lua/mini/operators.lua#L1027-1031

I still can not reproduce this, but it indeed looks like an issue with caching and restoring " register (which maybe automatically touches system clipboard if 'clipboard' is not empty).
Would you mind confirming a couple for things for me, please?

Does explicitly yanking and replacing from register produce the delay? So steps are now "ayy and "agrr.
Does the following patch fix the issue?diff --git a/lua/mini/operators.lua b/lua/mini/operators.lua
index cddea21..535b557 100644
--- a/lua/mini/operators.lua
+++ b/lua/mini/operators.lua
@@ -1027,6 +1027,8 @@ H.replace_do = function(data)
   H.do_between_marks('d', delete_data)

   -- Paste register (ensuring same submode type as region)
+  local cache_clipboard = vim.o.clipboard
+  vim.o.clipboard = ''
   H.with_temp_context({ registers = { register } }, function()
     H.set_reg_type(register, submode)

@@ -1036,6 +1038,7 @@ H.replace_do = function(data)
     local paste_keys = string.format('%d"%s%s', data.count or 1, register, (is_edge and 'p' or 'P'))
     H.cmd_normal(paste_keys)
   end)
+  vim.o.clipboard = cache_clipboard

   -- Adjust cursor to be at start mark
   vim.api.nvim_win_set_cursor(0, { from_line, from_col })

Does explicitly yanking and replacing from register produce the delay? So steps are now "ayy and "agrr.
It does not produce any delay.

If using system clipboard with "+grr, there is a delay.
(that delay does not exists with simple paste with "+p

Does the following patch fix the issue?
Nop, same delay

If using system clipboard with "+grr, there is a delay.

And I presume there is no delay if using regular register (like a)?
Some other questions that might help:

What clipboard/OS you're using here?
Does delay happen on other text manipulation operators? Like exchange (gxx) and multiple (gmm)?

Unfortunately, that diff was the best attempt at fixing this. I'd need more profiling information. My suggestion would be to create a _G.log variable and iteratively add table.insert(_G.log, { place = '<descriptive place name>', time = vim.loop.hrtime() }) before and after suspected lines that might cause delay. Identifying the exact line (which is not a call to a function from this module) would massively help. Because there are a lot going on inside those 4 lines.

If using system clipboard with "+grr, there is a delay.

And I presume there is no delay if using regular register (like a)?

That is correct. My previous answer was misquoted.

What clipboard/OS you're using here?

I'm on Debian Gnome, using wayland (nvim is using wl-copy and wl-paste)

Does delay happen on other text manipulation operators? Like exchange (gxx) and multiple (gmm)?

gmm and gxx work fine, with no visible delay

Unfortunately, that diff was the best attempt at fixing this. I'd need more profiling information.

I've managed to pinpoint the issue.
vim.fn.getreginfo
It takes ~0.5ms with regular registers
and ~150ms with system registers.
I think it gets called more than once..
vim.fn.getregtype has the same problem.
I think this is related to
https://github.com/neovim/neovim/issues/12622 and https://github.com/neovim/neovim/issues/23748
This coment https://github.com/neovim/neovim/issues/23748#issuecomment-1980626397 explains that wl-copy has some hacks with undesirable effects.

The thing is that substitute.nvim has no noticeable delay. I think the reason is that vim.fn.getregtype and vim.fn.getreginfo are being called only once, and mini.operators calls then multiple times.

@Susensio, would you mind applying the following patch and see if the delay persists?
diff --git a/lua/mini/operators.lua b/lua/mini/operators.lua
index cddea215..249fd5d4 100644
--- a/lua/mini/operators.lua
+++ b/lua/mini/operators.lua
@@ -997,8 +997,8 @@ H.replace_do = function(data)
   local mark_from, mark_to = data.mark_from, data.mark_to

   -- Do nothing with empty/unknown register
-  local register_type = H.get_reg_type(register)
-  if register_type == '' then H.error('Register ' .. vim.inspect(register) .. ' is empty or unknown.') end
+  local reg_info = vim.fn.getreginfo(register)
+  if reg_info.regtype == nil then H.error('Register ' .. vim.inspect(register) .. ' is empty or unknown.') end

   -- Determine if region is at edge which is needed for the correct paste key
   local from_line, from_col = unpack(H.get_mark(mark_from))
@@ -1026,16 +1026,18 @@ H.replace_do = function(data)
     { mark_from = mark_from, mark_to = mark_to, submode = forced_motion, mode = data.mode, register = '_' }
   H.do_between_marks('d', delete_data)

-  -- Paste register (ensuring same submode type as region)
-  H.with_temp_context({ registers = { register } }, function()
-    H.set_reg_type(register, submode)
+  -- Modify register data to have proper submode and indent
+  local new_reg_info = vim.deepcopy(reg_info)
+  if new_reg_info.regtype:sub(1, 1) ~= submode then new_reg_info.regtype = submode end
+  if should_reindent then new_reg_info.regcontents = H.update_indent(new_reg_info.regcontents, init_indent) end
+  vim.fn.setreg(register, new_reg_info)

-    -- Possibly reindent
-    if should_reindent then H.set_reg_indent(register, init_indent) end
+  -- Paste
+  local paste_keys = string.format('%d"%s%s', data.count or 1, register, (is_edge and 'p' or 'P'))
+  H.cmd_normal(paste_keys)

-    local paste_keys = string.format('%d"%s%s', data.count or 1, register, (is_edge and 'p' or 'P'))
-    H.cmd_normal(paste_keys)
-  end)
+  -- Restore register data
+  vim.fn.setreg(register, reg_info)

   -- Adjust cursor to be at start mark
   vim.api.nvim_win_set_cursor(0, { from_line, from_col })
@@ -1152,26 +1154,6 @@ end

 H.is_content = function(x) return type(x) == 'table' and H.islist(x.lines) and type(x.submode) == 'string' end

--- Registers ------------------------------------------------------------------
-H.get_reg_type = function(regname) return vim.fn.getregtype(regname):sub(1, 1) end
-
-H.set_reg_type = function(regname, new_regtype)
-  local reg_info = vim.fn.getreginfo(regname)
-  local cur_regtype, n_lines = reg_info.regtype:sub(1, 1), #reg_info.regcontents
-
-  -- Do nothing if already the same type
-  if cur_regtype == new_regtype then return end
-
-  reg_info.regtype = new_regtype
-  vim.fn.setreg(regname, reg_info)
-end
-
-H.set_reg_indent = function(regname, new_indent)
-  local reg_info = vim.fn.getreginfo(regname)
-  reg_info.regcontents = H.update_indent(reg_info.regcontents, new_indent)
-  vim.fn.setreg(regname, reg_info)
-end
-
 -- Marks ----------------------------------------------------------------------
 H.get_region_data = function(mode)
   local submode = H.get_submode(mode)

As far as I can tell, now there should be a single call to either getregtype() and getreginfo() when doing replace.

If anything, this indeed sounds like an issue with the clipboard provider. I don't think it is a good idea to optimize the code specifically under the assumption that getreginfo() and getregtype() may take a long time.

I was thinking the same and I totally agree with you. A hacky workaround in wl-clipboard should not make mini.nvim code messier.

@Susensio, would you mind applying the following patch and see if the delay persists?
As far as I can tell, now there should be a single call to either getregtype() and getreginfo() when doing replace.

It works! The delay is barely noticeable! Thank you so much for your time on this.
I don't know if this affects much more people. The lack of bug reports on this topic is strange. From what I've been reading, this hack depends on the wayland protocol implemented by each compositor. Mutter (gnome wayland compositor) forces wl-clipboard to use that focus-stealing hack, and thus the delay. Maybe the intersection of people using wayland, gnome and mini.operators is small.
Topics I found about this, for cross reference:
https://github.com/bugaevc/wl-clipboard/issues/12#issuecomment-451977175
https://github.com/bugaevc/wl-clipboard/issues/163
https://gitlab.gnome.org/GNOME/mutter/-/issues/524
https://github.com/bugaevc/wl-clipboard/issues/227
It may be a niche problem but, from what I've read, it is not going to be fixed upstream anytime soon...
If you think this should not be part of mini.operators, I would keep this as a local branch in my fork. Anyhow, this is super good enough for me, thanks!

Thanks for the follow up!
That diff should now be part of the main branch. It reduces relatively big piece of code while making the rest not that bigger. So it's a win-win kind of situation :)


[Install] Don't set :no_install unecessarily

@homu r+

:pushpin: Commit 43b4291 has been approved by indirect

:zap: Test exempted - status


Add linalg detensorizing behind a flag.

Progress on https://github.com/google/iree/issues/1159
With this pass enabled, some programs do not yet compile fully through IREE's full mlir-to-vm pipeline. I think it would still be useful to wire up this way so others can more easily experiment.

Looks ok, apart from the fact that adding yet another flag makes me sad (my own doing at this point since its on me to clean this up).
Why do we need the flag though. What doesnt work if we leave it on by default?

Why do we need the flag though. What doesnt work if we leave it on by default?

See https://github.com/google/iree/issues/1159#issuecomment-891245839 (I'm working through these)


chore: Ignore cache dir

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
136

Relevant Lines:
138

💛  - Coveralls


Update boot versions

Update boot versions in gradle standalone samples and in spring cloud contract samples

Done via https://github.com/spring-cloud-samples/spring-cloud-contract-samples/commit/3271d9ee149f0895e4bbaeae1276e7a6ee362625 and 42a2d1dfb


Flutter Web: Embind reports leak of Typeface with CanvasKit

Steps to reproduce
Turn on -sSAFE_HEAP and -sASSERTIONS on the Canvaskit wasm build and do something that loads a font and renders text (I used google fonts package) with the wasm build selected as the local engine.
Expected results
Typeface doesn't leak
Actual results
Typeface leaks:

canvaskit.js:129 Embind found a leaked C++ instance Typeface <0x00227930>.
We'll free it automatically in this case, but this functionality is not reliable across various environments.
Make sure to invoke .delete() manually once you're done with the instance instead.
Originally allocated
    at Function._MakeTypefaceFromData (http://localhost:10803/canvaskit/chromium/canvaskit.js:141:227)
    at a.Typeface.MakeTypefaceFromData (http://localhost:10803/canvaskit/chromium/canvaskit.js:82:299)
    at g.TypefaceFontProvider.registerFont (http://localhost:10803/canvaskit/chromium/canvaskit.js:71:455)
    at Object.TypefaceFontProviderExtension$124registerFont [as TypefaceFontProviderExtension|registerFont] (http://localhost:10803/dart_sdk.js:202827:18)
    at [_registerWithFontProvider] (http://localhost:10803/dart_sdk.js:151213:64)
    at _engine.SkiaFontCollection.new.registerDownloadedFonts (http://localhost:10803/dart_sdk.js:151328:38)
    at _engine.SkiaFontCollection.new.loadAssetFonts (http://localhost:10803/dart_sdk.js:151304:14)
    at loadAssetFonts.next (<anonymous>)
    at http://localhost:10803/dart_sdk.js:48832:33
    at _RootZone.runUnary (http://localhost:10803/dart_sdk.js:48689:59)
    at _FutureListener.thenAwait.handleValue (http://localhost:10803/dart_sdk.js:44085:29)
    at handleValueCallback (http://localhost:10803/dart_sdk.js:44689:49)
    at _Future._propagateToListeners (http://localhost:10803/dart_sdk.js:44727:17)
    at [_completeWithValue] (http://localhost:10803/dart_sdk.js:44568:23)
    at async._AsyncCallbackEntry.new.callback (http://localhost:10803/dart_sdk.js:44602:35)
    at Object._microtaskLoop (http://localhost:10803/dart_sdk.js:49145:13)
    at _startMicrotaskLoop (http://localhost:10803/dart_sdk.js:49151:13)
    at http://localhost:10803/dart_sdk.js:44951:9

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

-sSAFE_HEAP and -sASSERTIONS on the Canvaskit wasm build

Could you please elaborate on this?
I assume that this error can be reproduced with a simple text using the google fonts package as shown below. If not, please share a minimal sample code. Thanks!
import 'package:flutter/material.dart';
import 'package:google_fonts/google_fonts.dart';

void main() {
  runApp(
    MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Center(
          child: Text(
            'This is Google Fonts',
            style: GoogleFonts.lato(),
          ),
        ),
      ),
    ),
  );
}

@huycozy this is a compiler option that can be used when compiling the engine which prints out when memory leaks happen. It can be turned on here in a custom canvaskit build:
https://github.com/flutter/engine/blob/main/skia/modules/canvaskit/BUILD.gn#L162
I enabled this setting to help track down some memory leaks and bugs we were seeing in our apps, and it is reporting a memory leak is happening. I believe that code sample should be sufficient, but to get the debug build you need to turn on those options in the .gn file and then use felt to rebuild the engine and run your sample against the local engine.

Thanks for the update. Unfortunately, I have my issue with Python 12 when building local engine (it was working before), so I can't replicate this right now, but I will give it a try later.
In the meantime, I will label the issue for other's thoughts. Could you please also share the engine hash you are building at, along with the output of flutter doctor -v that may help others looking at this issue can have more information to investigate further?

@jezell hey, I just wanted to provide some more context as why we can't enable or exceptions and why trying to recover from a malloc failure is a bad idea:
*) Every professional C++ project I have worked on has explicitly disabled C++ exceptions. The Google C++ style guide has some context as to why we've made this choice
*) Recovering from a malloc failure is not feasible in a large software project. Imagine if every place we called malloc() we had some code that handles the NULL case and tries to recover. This code wouldn't be well tested because we can't easily make specific malloc() calls fail, so we would end up with a really complicated code base with error handling code that never actually runs -- and thus is guaranteed to contain bugs.
The Flutter project will never enable C++ exceptions and we simply cannot litter our codebase with attempts to recover from memory allocation failures.
My 2c is that the WASM runtime is bugged if a call to abort() doesn't terminate the program.
I strongly recommend that you do not compile the engine with C++ exceptions enabled or other changes discussed in this bug because we simply can't support those builds.


feat: Add snippet for storing RRI-claim in dyn memory

Here's the pattern I would go with. I think it can be generalized somewhat to handle more claims as well, but I'm not sure that's actually a good idea. Let me know what you think.

Two more claim-generations added to this PR.
There are two benefits of this approach:

The tasm code for populating memory with the claims is tested in isolation. So if this breaks, you don't have to go through the entire SingleProof program to find the problem. The tests comparing a BFieldCodec encoded claim stored to memory is compared with the memory after the program execution on Triton VM.
Having a seperate snippet for each claim is a more flexible approach if and when the transaction sub-claim architecture changes.


Website [Expressive theme sunset] Update Designing page

We'll need to review the content on the Designing page to capture the latest information about the kits and type styles etc.
Note from Jan: I've tightened up this page to create a narrative flow for a designer coming in and getting set up the first time, or coming back to find new information.
There is another ticket detailing future changes to the Designing page, possibly breaking the content into separate pages but this would be better placed after we have some new steps to take for kits and new resources.
The PR for this ticket is here and ready to review.

@shixiedesign, @oliviaflory This page is ready for review! 👓


Use ParamSpec builders as soon as they are released

See e.g https://gtk-rs.org/gtk-rs-core/git/docs/glib/struct.ParamSpecInt64Builder.html

Once they are released, the properties macro would be as well.

In that case that issue can be closed


B, A, X, and Y buttons not binded to operate outside the crossbar

Stuck on Map Buttons screen. I'm using an Xbox One Controller.
Holding the right trigger and pressing 'B' or 'A' doesn't work as intended. I have installed AutoHotkey and set up the pad config as instructed.

Had to run ffxi_xinput.ahk as admin to capture input.
I'm still unable to open the in-game menu and confirm/cancel outside of the crossbar

I tried remapping and it also didn't work

Fixed by reloading the script.


home-manager: 2021-03-21 -> 2021-09-13

Motivation for this change
This package is over 6 months out of date. I've updated it to its release-21.05 branch.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

I don't think home-manager's release-21.05 is meant to be used with nixpkgs unstable: https://github.com/nix-community/home-manager/blob/ddeeb031fdf9b82e7a964bbe8a2df84c4546c2a1/modules/home-environment.nix#L440.

Is this package used anywhere? When you add HM it installs this script itself from its own repo if i recall correctly?
(I use flakes and HM is set up to switch when I do nixos-rebuild)

you don't want to rebuild all your system each time you want to modify a color in your config file.
this is a little comparison between each one:

nixos-rebuild switch (without home-manager): 6s
home-manager switch: 3s

you triple the time require to change config. this is very annoying to iterate rapidly over a config file.

This is fixed by #152243.


supabase support

it would be really good to see supabase support as a backend, since it supports authentication, API, and storage. One more added benefit is it would help transition to local first app https://localfirstweb.dev/ using tools like tinybase

Thanks @Ashraf-Ali-aa for your suggestion, but unfortunately we can't include all solutions out there. Our goal is to provide a solid foundation for React Native projects, and it's up to people to add what they want.


podman commands documentation on --detach-keys wrong

[//]: kind bug
Description
The documentation on detach-keys legal values does not match the source code.
Included in the documentation is:

CLI argument descriptions in go code
markdown documents
man pages
API.md

Steps to reproduce the issue:

Review containers/libpod/vendor/github.com/docker/docker/pkg/term/ascii.go vs. documentation listed above.

Describe the results you received:
Describe the results you expected:
Code and documentation should align regarding legal values.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
from source, master branch

Output of podman info:
from source, master branch

Additional environment details (AWS, VirtualBox, physical, etc.):

@jwhonce Can you open a PR to fix the documentation. I tried ctr-p, ctr-q to detach from a container and it does not seem to work.

@mheon is this still an issue?

@jwhonce What documentation, where? I don't know of any documentation beyond the Manpages about legal values


Default value for parameters with a class type hint can only be NULL

it's happen when uploaded the project to the server and it's work localhost

The problem appears in this code
protected static function toHijri(Carbon $date, int $adjustment = 0)
{
$adjusted = (new Carbon($date))->addDays($adjustment);
    $jd = Converter::gregorianToJulian($adjusted->year, $adjusted->month, $adjusted->day);
    $hijri = Converter::julianToHijri($jd);

    return new Date($hijri->day, $hijri->month, $hijri->year, $jd, clone $date, $adjustment);
}

Sorry the issue is not clear

have a look here
http://ahmedyaghi.com/basic/

Still not clear the link open a website

I just updated the PHP version to 7.1 and it's work :+1:


"Debug Failure. False expression." after each change with ts-loader and webpack-dev-server

Bug Report
I have a setup with webpack, ts-loader and typescript where after a single change typescript errors out and I have to restart WDS.
ERROR in ./src/index.tsx
Module build failed (from ../node_modules/ts-loader/index.js):
Error: Debug Failure. False expression.
    at updateRedirectsMap (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:41751:22)
    at Object.update (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:41844:13)
    at Object.createProgram (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\ts-loader\dist\servicesHost.js:419:91)
    at createProgram (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:121308:28)
    at executeSteps (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:121469:25)
    at Object.done (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:121274:13)
    at buildNextInvalidatedProject (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\typescript\lib\typescript.js:122055:32)
    at Object.buildReferences (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\ts-loader\dist\servicesHost.js:520:13)
    at buildSolutionReferences (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\ts-loader\dist\instances.js:324:38)
    at Object.loader (C:\temp\repro_for_ts_loader_issue\TestApps\avicadYC\node_modules\ts-loader\dist\index.js:21:45)

I'm reporting this here as the callstack points to typescript, though the root cause may as well be either in webpack or ts-loader.
🔎 Search Terms
Debug Failure. False expression.
updateRedirectsMap

🕗 Version & Regression Information
This was tested with 4.5.0-dev.20211006
⏯ Sample Repo
I'm currently in the process of reducing my project down to a manageable repro
👀 Details
This is with a typescript project reference, and the crash only occurs when editing a file in the linked project, not when updating a file in the main project.

I’m pretty sure this is due to a misuse of the module resolution cache within ts-loader.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4153

Questions ?
Please refer to the Backport tool documentation


[BUG] DistutilsMetaFinder breaks distutils submodule imports when old setuptools is higher on path

setuptools version
setuptools====60.7.0
Python version
Python 3.7
OS
macOS
Additional environment information
Similar to #2957, this occurs when an old setuptools is on path but a newer setuptools (with a distutils shimming .pth file) is installed.
Description
This is a followup to #2957 where the .pth shim broke distutils imports when an older setuptools is higher on path.  The fix in #2962 allows distutils to be imported as well as submodules, but those submodules are not usable.
I'm running into this issue when running a PEX which contains an older version of setuptools but the Python interpreter has a newer version of setuptools.  In the bootstrap phase pex removes the new setuptools from sys.path and adds the older setuptools.  DistutilsMetaFinder is not removed from sys.meta_path during this phase and causes distutils submodules to fail.
Expected behavior
The presence of a new-but-not-loadable setuptools on sys.path doesn't break distutils submodules.
How to Reproduce
As in #2957 a reproducer is:
python3 -m venv /tmp/bang && source /tmp/bang/bin/activate  # give us a clean venv to work in
python -m pip install 'setuptools>60'  # install new setuptools to venv site-packages
python -m pip install 'setuptools<45' -t .  # install old setuptools to cwd- higher than site-packages on path for most systems
python -c "import distutils.util; print(distutils.util)"

Output
Traceback (most recent call last):
  File "<string>", line 1, in <module>
AttributeError: module 'distutils' has no attribute 'util'

Thanks for your patience. I'm getting ramped up on this issue today and I'd like to come up with a robust fix.
Thanks for your investigation and linking to prior issues. I'll continue to explore this issue here.

I put together this patch:
setuptools bugfix/3073-old-setuptools $ git diff main
diff --git a/_distutils_hack/__init__.py b/_distutils_hack/__init__.py
index 1f8daf498..14d9bd996 100644
--- a/_distutils_hack/__init__.py
+++ b/_distutils_hack/__init__.py
@@ -96,6 +96,13 @@ class DistutilsMetaFinder:
         import importlib.abc
         import importlib.util

+        spec = importlib.util.find_spec('setuptools')
+        setuptools_site = os.path.dirname(os.path.dirname(spec.origin))
+        hook_site = os.path.dirname(os.path.dirname(__file__))
+        if setuptools_site != hook_site:
+            print(setuptools_site, hook_site)
+            return
+
         try:
             mod = importlib.import_module('setuptools._distutils')
         except Exception:

It does have the desired effect of working around the issue, but it has the undesirable effect that it breaks pip upgrades of setuptools from source (because the directory with the .pth file is different from the source directory, so the distutils hack doesn't load, and setuptools assertion to ensure_local_distutils fails).
In other words, some workflows are actually depending on the behavior that distutils is loaded from a setuptools where the hack is not.
I considered perhaps making this exception only when imported from .pth but not when installed as part of ensure_local_distutils, but I'm not confident that can be made to work.
Other options I'm considering:

allow alternate copies of setuptools to supply distutils as long as they actually have it (only bypass after detecting very old setuptools).
not supporting mixed environments with older setuptools.

I don't have any good solutions yet. The problem landscape is already very complicated (also by things like pip exclusions), so I'm not confident that the problem space is well understood or even has a viable solution.

(original reporter of this issue here): Yeah, we're also still hitting different permutations of this with current setuptools where the "wrong distutils" assert is firing in mixed situations where it shouldn't (usually when a system-site-packages venv with ancient setuptools is involved)- we're doing the envvar hack disable to limp along, but we really need to figure out a broader solution before this gets bigger (and it likely will).
Still willing to pitch in more on that if needed...