[Feature Request] Implement Stylesheet

Is your feature request related to a problem? Please describe.
In OpenXml's Stylesheet class, things like CellFormat are stored. We need these to correctly set a cell we want to save as pure text as it otherwise leads to problems like in #19.
Describe the solution you'd like
Implement styles and map them from the Stylesheet object.

This might become critical in the future. Swate excel logic was formerly dependant on the numberFormat. A parameter which is currently not modeled via FsSpreadsheet.
For now i will set this issue to P1 and discussion. So its easier to keep track of!

https://github.com/nfdi4plants/ARCtrl/issues/81 <-

Important: Due to @Freymaurer 's mentioned urgency, we best aim for just implementing NumberFormat (aka CellFormat in MS Excel) since it is critically needed in ARCtrl.


[UTOPIA - 1472] - (TESTING) Transfer old tests to use Playwright.

🎯 Summary

UTOPIA-1472

Created tests/ folder as the root for all end-to-end tests.
Created modules/ subfolder under tests/ for reusable test modules for common components and actions.
Created piaFill/ subfolder under modules/ for testing the 'piaFill' component with different roles.
Created basicPiaFill.ts for basic 'piaFill' component tests, applicable for all roles.
Created cpoPiaFill.ts for 'piaFill' tests specific to the 'CPO' role.
Created drafterPiaFill.ts for 'piaFill' tests specific to the 'Drafter' role.
Created mpoPiaFill.ts for 'piaFill' tests specific to the 'MPO' role.
Created index.ts under modules/ to bundle and export modules.
Created utils/ subfolder under tests/ for utility functions and helpers supporting tests.
Created auth/ subfolder under utils/ for authentication helpers for different user roles.
Created cpoAuth.ts for login functions for users with the 'CPO' role.
Created drafterAuth.ts for login functions for users with the 'Drafter' role.
Created mpoAuth.ts for login functions for users with the 'MPO' role.
Created env/ subfolder under utils/ for environment variable related files.
Created env.ts for retrieving usernames and passwords from environment variables.
Created uuid/ subfolder under utils/ for functions related to UUID operations within tests.
Created uuid.ts for generating UUIDs for test data.
Created uuidSearch.ts for performing search operations using UUIDs.
Created index.ts under utils/ to export utility functions.
Created authorized.spec.ts for tests ensuring authorized users access appropriate features.
Created autosave.spec.ts for tests verifying autosave functionality across the application.
Created comments.spec.ts for tests of the commenting feature within the app.
Created filter.spec.ts for tests validating filtering logic and UI components.
Created fullPia.spec.ts for a complete test suite for end-to-end 'pia' process validation.
Created landingPage.spec.ts for tests on the functionality and elements of the landing page.
Created sort.spec.ts for tests on sorting features in the PIA table in the app.
Created search.spec.ts for tests on search functionality, including edge cases and search accuracy.
Created tooltip.spec.ts for tests on tooltip integration and functionality within the app.
Created unauthorized.spec.ts for tests ensuring unauthorized users cannot access restricted areas.
Created a README.md file for the project.

🔰 Checklist

[x] I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.

I have performed a self-review of my code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation where required.
I have tested my changes to the best of my ability.
I have consulted with the team if introducing a new dependency.
My changes generate no new warnings.

Tests run great! Just need to update the branch now


Add CSV export of datasets [#158718021]

Not sure who should review, @mklewandowski who requested the feature or @dstrawberrygirl who currently has a PR in with this same file so should be familiar with it.

@mklewandowski I modified the CSV output to include a unit column for each measurement.  To get the units I had to update rhizo-server to return them (https://github.com/rhizolab/rhizo-server/pull/4).  This code falls back to using n/a in the units column value if rhizo-server doesn't return the units value.
The new format looks like this:
date,timestamp,humidity,humidity units,temperature,temperature units
"8/17/2018, 9:04:42 AM",1534511082,42.4,percent,,degrees C
"8/17/2018, 9:04:43 AM",1534511083,42.3,percent,25.4,degrees C
"8/17/2018, 9:04:44 AM",1534511084,42.3,percent,25.4,degrees C


Fix Build Error - Too Many Arguments

Fix the "too many arguments in the call to cs.NetworkOffering.NewCreateNetworkOfferingParams" error by ensuring that the function call at line 65 in cloudstack/resource_cloudstack_network_offering.go includes only the required four-string arguments.
Error is:
cloudstack/resource_cloudstack_network_offering.go:65:104: too many arguments in call to cs.NetworkOffering.NewCreateNetworkOfferingParams
	have (string, string, string, []string, string)
	want (string, string, string, string)
make: *** [build] Error 2

Thanks for the PR @FarnazBGH I still see the error from the CI:
make: *** [GNUmakefile:32: fmtcheck] Error 1
gofmt needs running on the following files:
./cloudstack/data_source_cloudstack_instance_test.go
You can use the command: `make fmt` to reformat code.
Error: Process completed with exit code 2.


Filter by own orchestrator

Just match any field with the orch address

Added a post request which accepts a orch addr in the body. Still needs testing to see if the cache works as expected though


burrow process crashed without any infomation

I use the Dockerfile to build burrow image, the process crashed after sometimes, and I'm not able to start it again because of the burrow.pid is not deleted in "main.go, defer removePidFile".
It seems that "defer" is not guaranteed to be executed 100%,  any idea about this issue?
1471231004131214854 [Critical] Cannot write PID file: open /var/tmp/burrow/burrow.pid: file exists
shawguo@shawguo-intg-server:/var/lib/docker$ docker start -ai a02fab4f91e1
1471231017000196692 [Critical] Cannot write PID file: open /var/tmp/burrow/burrow.pid: file exists

The root cause is burrow process crashed without any information.  I use the main branch(0815).
I'm trying to catch any useful information but I'm not quit familiar with golang, any suggestion is appreciated. I can reproduce it and I want to print more information about crash stack.

@toddpalino

I'm having similar issue, it would be useful to have possibility to enable debug mode.

I know that I have encountered this issue when I force stop Burrow. I am making the assumption that Burrow inadvertently shut down; which would cause Burrow (like when I force stop it) to not gracefully shutdown and clean up any remnants of data that it depends on.
My suggestion is to: Look at the burrow. config file you used to start the process. In the [general] section you will see a "logdir" and a "pidfile". I would just delete that file, and you should be able to start burrow again.
As far as debugging burrow, you will find some really helpful files in the log directory that should be able to give you some additional output.

This might be a combination of #57 and #152.  By default the app intercepts all stdout/stderr logs and sends them to /var/tmp/burrow/burrow.out.  If you're starting with the compose config included then the logs should be in $(pwd)/tmp/ but if you're running it by hand then they're going to the bit bucket.  I've opened #152 to change container logging to default to stdout/stderr.

Resolved by #152


RHBPMS-4526: Test case to reproduce deployment failure of kjar with r…

…esource type suffix in the artifactId

+1

Replaced and fixed by https://github.com/droolsjbpm/drools/pull/1031


[Do not merge] Test issue #20921 via md smoketest

Do not merge!
This PR tests the issues found in #20921 in the markdown smoketest page. Mostly for curiosity, but some of the topics also display other, more innocent formatting errors (codeblock indentation woes) that might cause issues.

/hold

I'm going to close this for now.


dotnet rule updates

adding new dotnet rules and updating existing rules with dotnet APIs.

ping @mike-hunhoff for comments above by moritz


Cannot retrieve SAS token

I'm trying to construct a SAS URI to use in a BlobContainerClient instance:
            var storages = _mediaKindClient.StorageAccounts.List();
            var credentials = _mediaKindClient.StorageAccounts.ListCredentials(
                (Guid)storages.First().Metadata.Id
            );
            var credential = _mediaKindClient.StorageAccounts.GetCredential(
                (Guid)storages.First().Metadata.Id,
                (Guid)credentials.First().Metadata.Id
            );

            string storageAccountUrl = $"https://{_storageAccountName}.blob.core.windows.net";
            string containerName = data.DestinationContainer;
            string sasToken = credential.Spec.AzureCredential.SasToken;

            string sasUriString = $"{storageAccountUrl}/{containerName}/{sasToken}";
            BlobContainerClient destinationBlobContainer = new BlobContainerClient(sasUri);

In this case, the SasToken is null and the blob container lacks the proper auth, but I do see a value in the SasTokenSanitized (with the redacted portion). Why is SasToken NULL but SasTokenSanitized not?
What is the pattern with this MK.IO library to use to achieve the goal here of having a SAS URI constructed?
I also noticed that the library has no way to retrieve the MK.IO Storage Account name and URL which would also be super useful in constructing such URIs.

Hello,
The .GetCredential() method will only return a sanitized SAS token. This is not an issue with the SDK. This is a by design feature of the MK.IO REST API : https://docs.mk.io/reference/get_apiaccountscustomer_idsubscriptionssubscription_idstoragestorage_idcredentialstoken_id
SasToken is present (as NULL) because this property is used by storage credential create method.
You can retrieve the MK.IO storage account name using
storages.First().Spec.Name
and storage Url using
storages.First().Spec.AzureStorageConfiguration.Url
I hope it helps.


HA MQTT integration unstable?

v0.5.0b7 is installed, and I have tested the Home Assistant MQTT integration.
Does not seem to be very useful at the moment. Just a few telegrams after a reboot.
I have tested running tcpdump on the HA/Mosquitto Pi. Nothing coming from Kegmon.
Apart from that: Many thanks for yet another useful project you have made!
Cheers
Lars

I've not tested that part that much yet.
It will only push data when the level or pour has been detected. So I might need to trigger this at more events.
How would you expect it to work ? What data would you want to see in HA ?

Ah, I suspected that. I must have wiggled one of the kegs, because I did get name, abv, ibu and ebc from that one time.
It would be nice if a telegram i sent maybe once an hour to confirm the unit is still alive - in addition to when a pour i detected.
For me, name, abv & volume is enough. "Nice to have" are: glasses left and temp

Having data sent on a regular interval is a good idea. Temp should have a more frequent push to track that value, like 5 or 10 min. Glasses left is there as an attribute so you can get that data.

I've done some changes to the HA integration and added so it will push data every 5 minutes when enabled. You can get the 0.5b9 build from the dev branch.

I regard this as closed, please reopen if the issue persists.


implement indexing logs solution for the failures

as a user I'd like to have possibility to search by exception via all logs DB

Duplicate for #962


Use != 0 instead of == 1 for check of masked value

Recently [0] we added a check on a value masked against SEQUENCE_LOCKTIME_DISABLE_FLAG, this check uses x == 1 but x can never be zero since SEQUENCE_LOCKTIME_DISABLE_FLAG is equal to 0x80000000.
Looking at the original commit it looks like we are trying to check for a value higher than the flag value, so we can mask then use x != 0 to see if the flag bit is enabled.
Found by clippy:
error: incompatible bit mask: _ &<PHONE_NUMBER> can never be equal to 1
CC: @darosior, have I understood this correctly man?
[0] commit: commit a3dd9fe840c8cdae84e9d586cd9a9d6c195f0092

Oh, right :facepalm:. Good catch. ACK b747e518f6061ee1281c0031154d5f82c3243b42.
-------- Original Message --------
On May 2, 2022, 5:38 AM, Tobin C. Harding wrote:

Recently [0] we added a check on a value masked against SEQUENCE_LOCKTIME_DISABLE_FLAG, this check uses x == 1 but x can never be zero since SEQUENCE_LOCKTIME_DISABLE_FLAG is equal to 0x80000000.
Looking at the original commit it looks like we are trying to check for a value higher than the flag value, so we can mask then use x != 0 to see if the flag bit is enabled.
Found by clippy:
error: incompatible bit mask: _ &<PHONE_NUMBER> can never be equal to 1
[0] commit: commit a3dd9fe840c8cdae84e9d586cd9a9d6c195f0092
CC: @.***(https://github.com/darosior), have I understood this correctly man?

You can view, comment on, or merge this pull request online at:
https://github.com/rust-bitcoin/rust-miniscript/pull/380
Commit Summary

b747e51 Use != 0 instead of == 1 for check of masked value

File Changes
(2 files)

M src/miniscript/types/extra_props.rs (4)
M src/miniscript/types/mod.rs (8)

Patch Links:

https://github.com/rust-bitcoin/rust-miniscript/pull/380.patch
https://github.com/rust-bitcoin/rust-miniscript/pull/380.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@apoelstra, does this make for trying out clippy here? I think maybe we can find some middle ground with clippy.
We should not enforce it on every CI run. But we can schedule CI runs every week Sunday night running the clippy CI on the latest tip? This way

New contributors won't be annoyed by CI everytime.
Every Monday after we see the red cross on latest master tip, we can resolve and fix the issues.

We can debate the schedule period if there is interest.

I am attempting to clear all the clippy warnings here and everywhere in our stack, I'm doing this in an attempt to get clippy running on CI for ever PR - I see no reason not to, is there some reason I'm missing? From my understanding the problems were all related to Rust 1.29, that's why I've waited til now to do this work.

From my understanding the problems were all related to Rust 1.29, that's why I've waited until now to do this work.

FTR, I am in favor of it. But several contributors don't always like the suggestions Clippy presents. It is yet another thing to check while submitting the pull request. And I am sure that we will end up with lots of cases where the last commit is just "Fix up Clippy lints".
In addition to every commit compiling tests, contributors need to run Clippy on every commit which people can forget and would get annoyed whenever the CI yells at low priority things are false positives. Then we need to figure out how to disable some lints that are not annoying etc.
As I said, I am in favor, I am just stating the reasons why people might oppose it.

oh I forgot about how annoying it is going back over patch sets to fix clippy stuff. Thanks for reminding me, we definitely should find a solution that does not lead to that.

Note one solution to this is to add a commit hook to git which fails to commit unless clippy passes. According to this SO you can commit a repo to rust-miniscript like: .githooks and get people to opt in to opt into the hooks with:
git config --local core.hooksPath .githooks/

Sweeeet, thanks man.


new post: scaling inlets PRO tunnels with Helm

Signed-off-by: Johan Siebens<EMAIL_ADDRESS>

LGTM, thank you for the updates


Expandable Attribute label is broken in custom editor window

When displaying an editor in a custom window by using Editor.CreateEditor from a class using the Expandable attribute the label is displayed twice as seen in this image:

public class LevelData : ScriptableObject
{
    [Expandable]
    public LevelSettings Settings;
    ...
}

var editor = Editor.CreateEditor(this.level);
editor.OnInspectorGUI();

i got the same problem ,If you don't have to display the variable name ,you can use
[Expandable, Label(" ")]


UnhandledPromiseRejectionWarning: can not create app folder and start installation

Description
The command create-react-native-app app-name cannot create app folder and start installation
Observed Behavior
What actually happened when you performed the above actions?
$ create-react-native-app AwesomeProject
(node:1490) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Error: spawn ENOTDIR
(node:1490) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

$ cd AwesomeProject
cd: no such file or directory: AwesomeProject

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts: empty
npm ls react-native: empty
npm ls expo: empty
node -v: v8.9.1
npm -v: 6.4.1
yarn --version: 1.7.0
watchman version: command not found: watchman

Also specify:

Operating system:  macOs 10.13.4
Phone/emulator/simulator & version: -

Check the readme file,looks like create-react-native-app is replaced with new expo-cli. Just figured it.

@yolossn Thanks. But readme also says

The create-react-native-app command is provided for backwards compatibility.
And also getting started section in facebook RN docs also have old way of creating RN app

Yes, backward compatibility must be insured ! As you said, even the React Native documentation is not up to date.
A new developer on React Native types 2 lines of command and is already blocked, I understand why many beginners are giving up.
We create a React JS app with:
create-react-app myapp
and React Native app with:
expo init myapp
So logical ¯_(ツ)_/¯

Same issue here. It took me several days to figure out what the issues were. Make sure you're using create-react-native-app @ v2. Then follow the steps I've outlined in this issue to fix all of the other bugs/issues that were introduced when they merged create-react-native-app with Expo a few days ago.
https://github.com/react-community/create-react-native-app/issues/777

@caseychoiniere Thanks, I updated my CRNA CLI to v2 and everything is working.


Yearly rural employee numbers for GOPB

It's been a year since we added this. Have we or do we still do this @jacobdadams?

This is a recurring task that started back in 2019. I've been in discussion with DHRM about the need for it. Once we know whether they need us to keep doing it or we can just nuke it, I'll either update this task or close it out.


Readme improvement

:tophat: What is the goal?
Improve documentation.
:memo: How is it being implemented?
Readme uses as example the deprecated method "Dexter.withActivity",
it should use "Dexter.withContext" instead.

Thanks!


FYI: --matlabmcr option is apparently not available in neurodocker anymore

@DavidjWhite33 FYI
Building containers with matlab MCR does not work because neurodocker does not receive anymore the --matlabmcr option.
It is not listed in the help of neurodocker:
https://www.repronim.org/neurodocker/user_guide/cli.html
And is indeed not in the github code of neurodocker, as far as I can see:
https://github.com/ReproNim/neurodocker
It did exist in the documentation of older versions:
https://hub.docker.com/r/repronim/neurodocker/
https://hub.docker.com/r/kaczmarj/neurodocker
I'm not sure why the change, but I opened an issue in neurodocker:
https://github.com/ReproNim/neurodocker/issues/421
Let's see what they say.
@stebo85 @aswinnarayanan Is our neurodocker fork always is in sync with neurodocker master, or do you sync it once in a while? What would be the better option?

They seem to install the matlab MCR manually in their SPM template: https://github.com/ReproNim/neurodocker/blob/master/neurodocker/templates/spm12.yaml
We manually sync the fork when needed.

Yes. I found the same re spm12. Wonder why they made the change?
Is there any way to trigger a test rebuild (that doesn't actually update
the packages repository, but only checks the exit codes) of all containers
every time we sync neurodocker? This will ensure that the syncing does not
break anything.
Or is it too expensive? what are the limits on our CI resources?
On Wed, Sep 22, 2021 at 8:45 PM Steffen Bollmann @.***>
wrote:

They seem to install the matlab MCR manually in their SPM template:
https://github.com/ReproNim/neurodocker/blob/master/neurodocker/templates/spm12.yaml
We manually sync the fork when needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NeuroDesk/neurocontainers/issues/57#issuecomment-924812717,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7FUI2VTNOAGSXBNSJI2OTUDGXU7ANCNFSM5EQ2ZRRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

it's possible, you could add this check in the gitlab actions run. There is currently no limit on CI.

Ok. I'll have to learn how to do that.
I'll add it to my to do list.
--
Oren Civier
Neuroimaging Informatics Fellow
Swinburne Neuroimaging | National Imaging Facility
https://anif.org.au/team/oren-civier/
https://www.swinburne.edu.au/neuroimaging
Swinburne University of Technology
Hawthorn Campus, Advanced Technologies Centre, Room 914
+61 3 9214 4628 | +61 4 3101 6603
@.*** (Email/Teams video call)
https://twitter.com/orencivier
On Wed, 22 Sep 2021, 21:41 Steffen Bollmann, @.***>
wrote:

it's possible, you could add this check in the gitlab actions run. There
is currently no limit on CI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NeuroDesk/neurocontainers/issues/57#issuecomment-924849290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7FUI3222U6SA4DWC5Z65TUDG6GXANCNFSM5EQ2ZRRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This has been added back to neurodocker: https://github.com/ReproNim/neurodocker/pull/422

Hello @stebo85
@DavidjWhite33 , @TomEmotion and I are working on getting back a proper Neurodesk version like the one you used when building CONN. We tested it, and the version that they want to merge (it actually hasn't been merged yet) does have Matlab MCR, but apparently, is missing other required utils.
You built CONN last on 22/3/2021:
https://github.com/NeuroDesk/neurocontainers/pkgs/container/caid%2Fconn_20b
This is the last commit of Neurocontainers on 22/3/2021:
https://github.com/NeuroDesk/neurocontainers/tree/6215a0fcf540a6f69dbb9f4afa55228d92c77170
The main_setup.sh file on that commit was:
https://github.com/NeuroDesk/neurocontainers/blob/6215a0fcf540a6f69dbb9f4afa55228d92c77170/recipes/main_setup.sh
When I go to our fork on Neurodocker
https://github.com/NeuroDesk/neurodocker
... and press commits, I see the history of commits in the ReproNim/neurodocker mixed with commits in NeuroDesk/neurodocker
https://github.com/NeuroDesk/neurodocker/commits/master
Is there any way to investigate the history of the fork, in order to find out what was the state of the fork (to what commit ID of Neurodocker/master it was synchronised), when you built CONN?
Also -- do you ever do any  commits directly to NeuroDesk/neurodocker? Are they getting deleted when you sync with ReproNim/neurodocker? Is there any trace of them?
Thanks again for your assistance. As you see, we still learn all the fine details of Github.
Oren

@stebo85 Notice that I edited the previous comment!

@civier and @aswinnarayanan - I had to reset the fork in neurodesk to the upstream one because we were not able to merge anymore. I kept a backup here: https://github.com/stebo85/neurodocker-old-fork - Oren, if you use the old fork conn should build again.

Thanks Steffen. Dave and I found your backup. Did you put in a pull request to repronim for the changes you made to get CONN working and it either wasn't merged or was subsequently deleted? In any case, Dave will grab the changes you made in there (mainly the relevant yaml files) and use them to get the MCR working again in Neurodocker. Once we have that working we'll submit a pull request to get it merged with the master branch.

@civier. Yup I was just referring to the neurocontainers version

Yes, I had submitted multiple pull-requests to the neurodocker-upstream project, but they didn't get merged. Then the forks got out of sync and I had to reset it.

Yes, exactly - that is what I was suggesting. In that way, for example, we would have a branch that includes the changes you made to get CONN built.

Thanks @stebo85 - from the looks of the merged changes in the Repronim repo, our only required changes will be to include more recent runtime versions (so we can disregard the matlabmcr.yaml I've added to our fork, apart from grabbing the links to more recent versions).

Yes, that looks good :)
I had to move your changes to a separate branch, otherwise we cannot sync with upstream. Do not commit changes to the master branch - always keep it in dedicated branches, otherwise we can not send pull requests or update from upstream :)
your branch is here now: https://github.com/NeuroDesk/neurodocker/tree/david-white-add-matlab-mcr

closing as MCR is added back in and we submitted the pull request


fix haproxy::globals::sort_options_alphabetic not being respected

Summary
I believe commit e0835f86149a4708ae493fb5bf387b61562503f3 and its siblings to be in error.
Additional Context
When specifying "haproxy::globals::sort_options_alphabetic: false" it will be overwritten/ignored by
Boolean                                 $sort_options_alphabetic  = true,

with
$_sort_options_alphabetic = pick($sort_options_alphabetic, $haproxy::globals::sort_options_alphabetic)

pick() according to documentation:
"Returns: Any the first value in a list of values that is not undefined or an empty string.
If sort_options_alphabetic isn't specifically passed into a listen resource we want to global value to be applied.
Current version of haproxy ignores the global and shuffled all my entries into alphabetical order.
Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified. (For example puppet apply)

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


New site

Codecov Report

Merging #80 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #80   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          32     32
  Lines         252    252
=====================================
  Hits          252    252

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 abf217d...05ed010. Read the comment docs.


Wrong result on Dropout layer

import torchvision.models as models
from torchsummary import summary

mobilenet = models.mobilenet_v2()
summary(mobilenet, (3, 224, 224))

On running this code, I got an error in the output shape of the Dropout layer. Here are the last few lines from the output :

InvertedResidual-153            [-1, 320, 7, 7]               0
Conv2d-154           [-1, 1280, 7, 7]         409,600
BatchNorm2d-155           [-1, 1280, 7, 7]           2,560
ReLU6-156           [-1, 1280, 7, 7]               0
Dropout-157                 [-1, 1280]               0
Linear-158                 [-1, 1000]       1,281,000
===============================================================
Total params: 3,504,872
Trainable params: 3,504,872
Non-trainable params: 0

The result for the Dropout layer is wrong because Dropout does not change the shape of the input as per the PyTorch docs.

This is not an error from torchsummary, rather a lack of information on the functional API of torch from torchsummary.
You pointed out that Dropout does not change the shape of the input, which is correct. But you assume that the input to the  Dropout layer is the output of  ReLU6-156. Always check the ̀forward` implementation of the model ;)
In this case, there is an adaptive average pooling and a .reshape(x.shape[0], -1) over here:
https://github.com/pytorch/vision/blob/master/torchvision/models/mobilenet.py#L155
There are not picked up by the hooking mechanism, hence your output!


Investigate .nvmrc as a way to enforce specific Node version in SPA projects

We currently specify the minimum version of Node in our README, but it would be nice if we could enforce it in the actual project.  The .nvmrc config file might allow us to do that.
https://github.com/creationix/nvm#nvmrc

Internally we use nvm as it allows for rapid testing in other version of node as well as better handling of global permissions.  If we do use the .nvmrc file, we should also document the requirement of nvm.
Perhaps as an alternative we could use https://docs.npmjs.com/files/package.json#engines

Closing now that all three referenced above have been approved and merged.


[Issue] v0.4.3 regression handling DOCKER_CONFIG environment variable

The DOCKER_CONFIG environment variable is commonly used to override the path to the directory that contain the config.json file. One important note is that it's the location of the containing directory, not the JSON file itself. For example:
export DOCKER_CONFIG=/my/custom/.docker

Implies the config file is at /my/custom/.docker/config.json.
Prior to v0.4.3, this worked perfectly. regctl command used to pick up this environment variable as expected.
As of release v.0.4.3, this no longer worked. A regctl image copy ... command will produce this error:
time="..." level=warning msg="Failed to load docker creds" err="read /my/custom/.docker: is a directory"

It seems that the changes introduced in https://github.com/regclient/regclient/pull/234 incorrectly treated DOCKER_CONFIG as a file path instead of a directory path.

Woops, thanks for catching that! Let me know if you still see any issues with the latest commit.


runtime: "SIGBUS: bus error code=0x1" on unchecked type assertion with generics

Go version
from go1.18 to gotip(go1.23-ccbc725f)
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/meroje/Library/Caches/go-build'
GOENV='/Users/meroje/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/meroje/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/meroje/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/Cellar/go/1.22.0/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/Cellar/go/1.22.0/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.22.0'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/meroje/projects/memcrash/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/fq/_z724rr10kb802r6v4m4jnjw0000gq/T/go-build1130271614=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
This came up during an internal code review on a convoluted dynamodb unmarshaler implementation, the crash could be striped down to this:
https://go.dev/play/p/buw65E2QFzU
package main

import "fmt"

type I interface {
	x()
}

func check[T bool]() {
	var value bool
	if v, ok := any(value).(I); ok {
		_ = v.(T)
		panic("unreachable")
	}
	_ = any(value).(I)
}

func main() {
	defer func() {
		err := recover()
		fmt.Printf("recovered: %v\n", err)
	}()
	check()
}

The crash only occurs if the v.(T) is present, even though the condition evaluates to false (shown by not reaching the unreachable panic). Checking any of the 2 assertions also prevents the crash.

We expected to recover from a common type assertion panic, as is the case without the unreachable if branch.

While step-debugging, I saw the segmentation fault occurring at this line (which can be seen in above stack trace)
https://github.com/golang/go/blob/f29208030ab80769ce61dedb5a419821abf92113/src/runtime/iface.go#L240

The value of the indirect store register x2 is a readonly symbol:
->  0x1000093d8 <+824>: str    xzr, [x2, #0x18]

x2 = 0x00000001000d48e8  x`go:itab.bool,main.I

nm: 1000d48e8 R go:itab.bool,main.I

The runtime is trying to (re-)initialize an itab that lives in read-only memory. It can only happen when initializing an itab that represents the "type does not implement interface" condition, which in turn can only happen in certain generics scenarios.
We're reinitializing the itab just for the side-effect of finding a missing method name to report. That seems unnecessary.
Simpler repro:
package main

type I interface {
	foo()
}

type B bool

//go:noinline
func f[T any]() {
	var x I
	if _, ok := x.(T); ok {
	}
}

func isI(x any) {
	_ = x.(I)
}

func main() {
	f[B]() // add <B,I> itab to binary
	isI(B(false))
}

In particular, generics is used just to load the offending itab into read-only memory. Once it is there, no generics is needed to trigger the bug.


Container cant start

container failed to start when deploying to cloud run:
ERROR: (gcloud.beta.run.deploy) Revision 'hashicorp-vault-00004-2fb' is not ready and cannot serve traffic. The user-provided container failed to start and listen on the port defined provided by the PORT=8200 environment variable within the allocated timeout. This can happen when the container port is misconfigured or if the timeout is too short. The health check timeout can be extended. Logs for this revision might contain more information.
here is my cloud service run:
… Service hashicorp-vault in region europe-west3
URL:     https://hashicorp-vault-XXXXXXXXX.europe-west3.run.app
Ingress: all
Traffic:
100% (currently -) LATEST (currently hashicorp-vault-00002-5zh)
Revision hashicorp-vault-00005-8vn
Container vault-server-1
Image:           europe-west3-docker.pkg.dev/XXXX-XXXXX/vault-docker-repository/vault-server:12421d80466a34d35f3be868b4587417fd369762
Port:            8200
Memory:          512Mi
CPU:             1000m
Secrets:
/etc/vault     vault-server-config:latest
Startup Probe:
TCP every 240s
Port:          8200
Initial delay: 180s
Timeout:       240s
Failure threshold: 5
Type:          Custom
Service account<EMAIL_ADDRESS>Concurrency:       20
Min Instances:     1
Max Instances:     1
Timeout:           300s
Execution Environment: Second Generation

Yes, there's an issue with the schema not recognizing the env parameter, which is the way I had been passing the 3 environment variables (near the bottom in the deploy step).  As a result when the container starts, the required settings are not being passed to Vault's config file and hence it's failing to start.  What you can do for now is add:
  - '--set-env-vars'
  - 'GOOGLE_PROJECT=${PROJECT_ID},GOOGLE_STORAGE_BUCKET=${_GCS_BUCKET_NAME},VAULT_GCPCKMS_SEAL_KEY_RING=${_KMS_KEY_RING}'

to pass the env vars instead.  I was using the env method because it looked cleaner (but it doesn't seem to work anymore, but docs don't mention that in the Cloud Build schema
I'll be making this change when I release some updates most likely this weekend, but you can make the change to the cloudbuild.yaml file yourself and recommit to rerun your build.

I created a bug report for this

after adding
  - '--set-env-vars'
  - 'GOOGLE_PROJECT=${PROJECT_ID},GOOGLE_STORAGE_BUCKET=${_GCS_BUCKET_NAME},VAULT_GCPCKMS_SEAL_KEY_RING=${_KMS_KEY_RING}'

it worked like a charm

v0.10.0 should address the issue


Remove duplicate error/warning messages

Issue description / feature objectives
Some errors trigger redundant messages, e.g.:
trestle.core.base_model:327 ERROR: Failed to parse OSCAL object: OSCAL version: 1.0.1 is not supported, use 1.0.0 instead.
trestle.core.commands.validate:52 ERROR: Error while validating contents of a trestle model: Error parsing file C:\tmp\test\profiles\fedramp\profile.json OSCAL version: 1.0.1 is not supported, use 1.0.0 instead.

This is due to an inner exception being caught and outputting a message, then later an outer exception is caught and this is repeated:
inner:
        except Exception as e:
            logger.error(f'Failed to parse OSCAL object: {e}')
            raise err.TrestleError(f'Error parsing file {path} {str(e)}')

outer:
        except TrestleError as e:
            logger.debug(traceback.format_exc())
            logger.error(f'Error while validating contents of a trestle model: {e}')
            return CmdReturnCodes.COMMAND_ERROR.value

Caveats / Assumptions
The redundant info is distracting
Completion Criteria
Decide how to handle this situation consistently
Also - should this be a warning?  It is due to bad data from the user and not a problem in trestle code.

I believe the proper way to handle it would be

To switch all the inner logger error messages to the debug level and print only the outer error.
As for warnings, I believe some of them do not halt the execution, thus printing them is fine. However, if there are some "warnings" that do in fact end the execution and raise an exception, they should be treated as errors and moved to the debug level as well.


Rebase on synthetichealth/synthea

Summary
Rebasing our fork with the most recent version of synthetichealth/synthea
New Behavior
Bug fixes and new behavior introduced by synthetichealth since our last rebase
Code Changes
simple rebase
Testing Guidance

Run the unit tests using ./gradlew build check test. This took about 13 minutes on my machine and produced 103 warnings, which was equivalent with synthetichealth/synthea:master.
Download the .sh file posted in Slack into the same directory as your synthea repo
Navigate to that repo and run chmod +x synthea-test.sh
Then run ./synthea-test.sh
Ensure no errors occur while the file runs. You should see build successful after each command runs

Codecov Report

Merging #37 (cfd2d37) into master (0133f46) will decrease coverage by 16.91%.
The diff coverage is 53.84%.

@@              Coverage Diff              @@
##             master      #37       +/-   ##
=============================================
- Coverage     82.33%   65.41%   -16.92%
+ Complexity     2859     2530      -329
=============================================
  Files           107      109        +2
  Lines         17078    17311      +233
  Branches       2436     2468       +32
=============================================
- Hits          14061    11324     -2737
- Misses         2270     5329     +3059
+ Partials        747      658       -89

Impacted Files
Coverage Δ

.../main/java/org/mitre/synthea/engine/Generator.java
76.24% <0.00%> (-1.48%)
:arrow_down:

src/main/java/org/mitre/synthea/engine/Module.java
78.57% <0.00%> (-2.49%)
:arrow_down:

...c/main/java/org/mitre/synthea/export/FhirStu3.java
78.57% <ø> (-4.23%)
:arrow_down:

...va/org/mitre/synthea/helpers/ValueSetResolver.java
0.00% <0.00%> (ø)

...org/mitre/synthea/world/concepts/HealthRecord.java
86.92% <0.00%> (-6.88%)
:arrow_down:

.../org/mitre/synthea/world/concepts/Terminology.java
0.00% <0.00%> (ø)

src/main/java/org/mitre/synthea/engine/Logic.java
83.54% <80.00%> (+0.53%)
:arrow_up:

src/main/java/org/mitre/synthea/export/FhirR4.java
83.12% <83.20%> (-1.47%)
:arrow_down:

src/main/java/org/mitre/synthea/engine/State.java
84.99% <86.66%> (-2.84%)
:arrow_down:

...n/java/org/mitre/synthea/export/CPCDSExporter.java
0.00% <0.00%> (-92.99%)
:arrow_down:

... and 27 more

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 0133f46...cfd2d37. Read the comment docs.


Data writer calls close() mistakenly

In the DataWriter class, we mistakenly close the luceneDataWriter during each insertTuple(). As a consequence, when inserting a large number of tuples, it takes a lot of time. The same mistake also exists in deleteTuple().

Performance test results:
https://docs.google.com/spreadsheets/d/1vFlBGLD0gOuhkEG-7-da7o59DEiiewmp09FgZdiVnRw/edit#gid=0

For a dataset of 100,000 Medical records, the fix reduced time of building a standard analyzer index from 12941.507 seconds to 33.35 seconds. The trigram index time was reduced from 14660.86 seconds to 118.63 seconds.

Fixed in https://github.com/TextDB/textdb/pull/328.


Make command registry push with cython-annotate instead of extension

HI, thanks for the contribution. But can you explain why is this change needed?
Also - this doesn't seem to work, the keybinding still searches for the old names.

It makes the prompt in visual studio code more descriptive; but perhaps I
missed a second reference.
I found it that it was extension because the plugin actually doesn't seem
to work under Windows
Sent from a high-spec phone. Please excuse any imaginative autocorrect
errors.
On Sun, 22 Dec 2019, 09:04 Itai In<EMAIL_ADDRESS>wrote:

HI, thanks for the contribution. But can you explain why is this change
needed?
Also - this doesn't seem to work, the keybinding still searches for the
old names.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/itaiin/vscode-cython-annotate/pull/2?email_source=notifications&email_token=AD5ZGULBLRRAXSJLH5QW4CDQZ4US7A5CNFSM4J5RSJJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPLQAQ#issuecomment-568244226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5ZGUMHXOYAP2RVWYNIT4LQZ4US7ANCNFSM4J5RSJJQ
.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
Yannlecun, @yl-jiang, @ViniciusBorgesdeAraujo, @sxmawl, @pjox, @lanma, @lxuechen, @VanekPetr, @prajwal89, @ZiluWa, @underwoodnoble, @lambda7xx, @MachineLovesLearning, @0ihsan, @Jeditrix, @kmatsoukas, @MichaelFullStackk, @rmateusc, @namtranase, @jdjin6000


Automate UML image generation

#177 added UML diagrams to the CONTRIBUTING.md file. At the moment these are manually generated by
pyreverse -o png causalpy --output-directory img

It would be great to get this done automatically. I believe it might be hard to do it in the makefile and get that run remotely, so it may have to happen locally as part of the pre-commit checks.

Update: when #201 is merged, this will be done by make uml

Update: when #201 is merged, this will be done by make uml

#201 was already merged.


check correctness of test cases

configuration for packages: https://github.com/octolab/Cleaner/blob/master/tests/fixtures/packages.yml
test cases: https://github.com/octolab/Cleaner/tree/master/tests/fixtures/testcases/matcher/weight
example:
project configuration https://github.com/octolab/Cleaner/blob/master/tests/fixtures/testcases/matcher/weight/mixed/setup.yml#L5-L11
categories for removing https://github.com/octolab/Cleaner/blob/master/tests/fixtures/testcases/matcher/weight/mixed/expected.yml#L1-L5

@mlocati could you help with the issue?

@mlocati could you help with the issue?

Sorry but I don't understand what would you like me to do... Should I manually check if the expected.yml files are good for their corresponding setup.yml files?

That all test cases have correct behavior. package.yml (input) - setup.yml (rules) - expected.yml (output)

Yes, everything is fine. Good work!

ok, thanks


Arrays inside arrays

Hello.
I was trying to document my ruby rails webserver that returns following JSON
{
  "lists": [
    [
      {
        "name": "test",
        "urls": [ ]
      }
    ]
  ]
}

I am using swagger-blocks and I could not create such a structure that has two or more arrays inside each other.
I tried several ways. Here is something I have now, but obviously it did not work.
class MyClass < ActiveRecord::Base
  include Swagger::Blocks

  swagger_schema :MyOutput do
    property :lists, type: :array do
      items type: :array do
        items type: :object do
          property :name, type: :string, example: "test"
          property :urls, type: :array do
            items type: :string
          end
        end
      end
    end
  end

  # ...
end

Am I doing something wrong or is this not possible?

Swagger 2.0 appears to support nesting items within items for 2D arrays.  Plans to match this functionality?

Hey @lorainekv, would you be interested in taking on the implementation of this and sending a PR? It should be relatively straightforward, I think all one needs to do is: add an items method to the ItemsNode here (the method will be almost exactly like this), as well as adding a test for it.

Sure thing, thanks for the helpful response! Here's a PR
PS thank you for this gem :)

Fixed by #99 and released in v2.0.1 of the gem. Thanks!


Nginx - Host breakout does not work

ERROR:
This chart's data failed to load
[object Object]
is anybody having a similar issue ?

Any help would be appreciated.


Missing libhyprutils.so.1

The new version 0.3.0 no longer provides libhyprutils.so.1, which is required by Hyprland. I believe this is the reason why everyone, including myself, is now encountering the error: Hyprland: error while loading shared libraries: libhyprutils.so.1: cannot open shared object file: No such file or directory.

recompile hyprland then?

Bro, I saw your previous comment on another forum. Recompiling Hyprland doesn't work. Also, from what I can see, the hyprutils package should provide the file, but it looks like that file is not included in the AUR.

Packaging is not something I am tasked with :)
I can assure you compilation does work unless you really have a broken system. Check out hyprland-meta-git in the aur


VFS-257 - FTP: Add support for MDTM to get more accurate last modified times

Adding support for MDTM command when obtaining a FtpFileObject timestamp.
The patch attached on the original JIRA was quite old, so I started fresh.
Contains two test cases:

Server supporting MDTM command
Server not supporting MDTM command

bump?

Thanks for your time reviewing @garydgregory. Latest changes pushed

The Java 11 build failed: https://travis-ci.org/github/apache/commons-vfs/jobs/695737649
You might want to rebase on master to make sure you have the latest, it should not matter but you never know.

error details from travis-ci (java 11):
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.4 sec <<< FAILURE! - in org.apache.commons.vfs2.provider.sftp.test.SftpProviderClosedExecChannelTestCase
junit.framework.TestSuite@58f437b0(org.apache.commons.vfs2.provider.sftp.test.SftpProviderClosedExecChannelTestCase$1)  Time elapsed: 0.4 sec  <<< ERROR!
org.apache.commons.vfs2.FileSystemException: Could not determine the type of file "sftp://testtest@localhost:35683/read-tests".
	at org.apache.commons.vfs2.provider.AbstractFileObject.getType(AbstractFileObject.java:1389)
	at org.apache.commons.vfs2.provider.AbstractFileObject.exists(AbstractFileObject.java:967)
	at org.apache.commons.vfs2.test.AbstractTestSuite.setUp(AbstractTestSuite.java:168)
	at org.apache.commons.vfs2.provider.sftp.test.AbstractSftpProviderTestCase$SftpProviderTestSuite.setUp(AbstractSftpProviderTestCase.java:265)
	at org.apache.commons.vfs2.test.AbstractTestSuite.lambda$run$0(AbstractTestSuite.java:129)
	at junit.framework.TestResult.runProtected(TestResult.java:142)
	at org.apache.commons.vfs2.test.AbstractTestSuite.run(AbstractTestSuite.java:134)
	at org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:90)
	at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:367)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:274)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:238)
	at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:161)
	at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:290)
	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:242)
	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:121)
Caused by: org.apache.commons.vfs2.FileSystemException: Could not connect to SFTP server at "sftp://testtest@localhost:35683/".
	at org.apache.commons.vfs2.provider.sftp.SftpFileSystem.getChannel(SftpFileSystem.java:164)
	at org.apache.commons.vfs2.provider.sftp.SftpFileObject.statSelf(SftpFileObject.java:106)
	at org.apache.commons.vfs2.provider.sftp.SftpFileObject.doGetType(SftpFileObject.java:76)
	at org.apache.commons.vfs2.provider.AbstractFileObject.getType(AbstractFileObject.java:1383)
	... 14 more
Caused by: com.jcraft.jsch.JSchException: java.net.SocketException: Broken pipe (Write failed)
	at com.jcraft.jsch.Channel.connect(Channel.java:159)
	at org.apache.commons.vfs2.provider.sftp.SftpFileSystem.getChannel(SftpFileSystem.java:138)
	... 17 more
Caused by: java.net.SocketException: Broken pipe (Write failed)
	at java.base/java.net.SocketOutputStream.socketWrite0(Native Method)
	at java.base/java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:110)
	at java.base/java.net.SocketOutputStream.write(SocketOutputStream.java:150)
	at com.jcraft.jsch.IO.put(IO.java:60)
	at com.jcraft.jsch.Session._write(Session.java:1379)
	at com.jcraft.jsch.Session.write(Session.java:1372)
	at com.jcraft.jsch.Channel.sendChannelOpen(Channel.java:732)
	at com.jcraft.jsch.Channel.connect(Channel.java:151)
	... 18 more

And, since every commit of this pr excluding the last one have no such error, I agree with gary this time, you might merge what in master and test it again(or several times), to see if this bug still occurs, or it occurs evertime, or it occurs sometimes.
The other commits fails on travis-ci only for openjdk-ea thing. ea is (or was) not compatible with one of a plugin we use (in commons-parent, so in nearly all commons project).  I put openjdk-ea in this repo into allow-failure at Feb 26, but this pr is when Feb 14, so...
(Btw, thanks for gary & other cool guys the new version of that plugin has released, and which is compatible with openjdk-ea.)
Anyway, I understand the frustration you felt when you see the red cross on travis-ci, and somethimes it just error/fail something with you do nothing wrong. Maybe a network error(in travis-ci, and makes it cannot download some dependencies), maybe a plugin not compatible, maybe another guy made a test that runs perfect in most times but will throw error when you are unlucky (this is real...especially for some memory-limit/gc related tests), and really, maybe it is not your fault for the tests fail.
But it is still quite helpful, so we should always care about the output result, and try to find out why it failed. or at least, find out whether the failures be caused by this pr.
Thanks.

@garydgregory @XenoAmess odd random test failure... I rebased from upstream master and looks like the test is passing now...

Implemented differently in git master.


URI scheme case

URI schemes are not case sensitive.

Fair enough, but using stripos() is more effective than strtolower() and strpos().

@narfbg Agreed.


in windows 10 node.js tools doesnt update

on windows 10 node.js tools update in visual studio 2017.  after update
i still see the node.js tools 1.3 installed on 3-12-2016 in extensions and update,  and no update available. on my windos 7 machine node.js tools <IP_ADDRESS> is present updated 12-05-2017

Microsoft Visual Studio Community 2017
Version 15.2 (26430.6) Release
VisualStudio.15.Release/15.2.0+26430.6
Microsoft .NET Framework
Version 4.7.02500
Installed Version: Community
JavaScript Language Service   2.0
JavaScript Language Service
JavaScript Project System   2.0
JavaScript Project System
Node.js Tools   1.3.41102.00
Adds support for developing and debugging Node.js apps in Visual Studio
NuGet Package Manager   4.2.0
TypeScript   <IP_ADDRESS>
TypeScript tools for Visual Studio

Is one of the VS 2017 releases targeting the preview releases?
Can you post the top 3 lines:
Microsoft Visual Studio Community 2017 Version 15.2 (26430.6) Release VisualStudio.15.Release/15.2.0+26430.6
for both installs?

on windows 7
Microsoft Visual Studio Community 2017
Version 15.2 (26430.6) Release
VisualStudio.15.Release/15.2.0+26430.6
Microsoft .NET Framework
Version 4.6.01055

on windows 10 insider preview
Microsoft Visual Studio Community 2017
Version 15.2 (26430.6) Release
VisualStudio.15.Release/15.2.0+26430.6
Microsoft .NET Framework
Version 4.7.02505

Curious, I assume in both cases the node workload was installed and neither show an update available in the Visual Studio Installer.

indeed

Can you try remove the node workload from the older install, and re-installing? And reboot the machine in between...

how do i remove the node workload or what is the node workload ?

Now I'm curious how you installed VS...  I was talking about this option in the VS installer.

i reinstalled  vs completely , it only shows visal studio rc now on the taskbar but node is up to date.

actually figured this out... should be fixed in VS 2017 update 3


Rename addFIPSsModeSecret to addFIPSModeSecret

Here you go @mheon
Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

LGTM. Thanks for fixing that up, I can't type sometimes :p

LGTM
@rh-atomic-bot r+


Updating page: Analyse

Updating the content of the page analyse to add ERIM relevant information

@nsunami, I have now also added the ERIM info to the analyse page, added a bunch of link mostly


Overlay app not registering clicks in 3.0.2

Environment

OS: Windows 10 Pro
Scrcpy version: 3.0, 3.0.2
Installation method: Windows release
Device model: Pixel 8
Android version: 15

Describe the bug
I use an app called Poke Genie that creates an overlay over Pokémon Go and uses the screen recording API to calculate information about Pokémon not normally shown. When scanning, this app should update every time you click behind it, so that the overlay constantly updates with new details. In scrcpy 3, it doesn't update unless I interact with the overlay directly. It seems like Poke Genie isn't aware of the clicks for some reason. Tapping the phone's display directly while scrcpy 3 is open results in expected behavior.
The first video shows expected behavior recorded in scrcpy 2.7. The overlay updates as I move through Pokémon.
https://github.com/user-attachments/assets/5bee8ac3-9ec0-4f42-9bcc-ac81284bfce6
This video is recorded in scrcpy 3.0.2. The overlay doesn't update unless I interact with it directly and dismiss it.
https://github.com/user-attachments/assets/1c649fba-2fc8-497c-896d-25397286155f

Please test #5614.

Solved issue. Thanks!

Thank you for the test :+1:
It will be merged before the next release.


ENH: update package version to 2.3.2

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #24.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #24

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ Use importlib-metadata instead of importlib_metadata
ℹ No valid build backend found for Python recipe for package metomi-rose using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ No valid build backend found for Python recipe for package metomi-rose-base using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ No valid build backend found for Python recipe for package metomi-rose using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

No dep changes this release:
$ git diff 2.2.0 2.3.2 setup.*


Add Automation Test suite

The entire tool needs an automated test suite, ideally running on every push / PR via github actions

Pending addition of tests to the github actions

Closing as github actions are introduced for every test pull requests now


Major update to the repo to support train-valid-test data split

This major update supports a train-valid-test split of the data per Issue #4. It includes:

A training script that uses train and valid data, and logs running statistics of model performance on valid.
A test script that uses a downloaded model to score the test data.
A new notebook that uses the best hyperparameters to train a model using a large number of estimators.

Abandoned because the wrong PR


[Automation] Generate Fluent Lite from edgezones#package-2024-04-01-preview

[Automation] Generate Fluent Lite from edgezones#package-2024-04-01-preview

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-edgezones

/azp run java - edgezones - ci

/azp run java - edgezones - ci

/azp run java - edgezones - ci


[next][android] fix gap

          ![Screenshot_1693505469](https://github.com/callstack/react-native-pager-view/assets/12766071/2f8e894b-ffb6-4680-89ba-906e263fca46)

Spacing issue
Originally posted by @troZee in https://github.com/callstack/react-native-pager-view/issues/763#issuecomment-1701540408

Closing as we are moving away from next experimental implementation.


折线图 splitArea 设置 grid 背景色位置偏差

Version
4.2.1
Reproduction link
https://gallery.echartsjs.com/editor.html?c=xLpHI9LzcD&v=1
Steps to reproduce
不需要执行任何操作
What is expected?
期望第7列有背景色
What is actually happening?
第6列有背景色

In my computer, the grid background color is in the seven columns, see the below picture:

so, I don't understand what's your problem is? Please provide more detailed message.

Try resize the window, pull it width enough and refresh the page.

@chezhe, I have reproduced this problem, thank you for your feedback.

@chezhe, I have reproduced this problem, thank you for your feedback.

I met this problem on different mobile devices too, especial on some Android.

Hi, I think it has been fixed in the PR https://github.com/apache/incubator-echarts/pull/11186 . It will be included in the version 4.5.0


map not shown, when DOCTYPE tag for HTML5 is included

map is not displayed, if I include <!DOCTYPE html> tag for HTML5 document.
workaround I found is adding:  before the DOCTYPE declaration. this of course renders my pages not valid.

I ran into this issue as well, it only arises when height is set to a percentage and the DOCTYPE is set.
Here are some examples:

DOCTYPE included, height set in pixels: http://jsbin.com/sotahurilu/2/
DOCTYPE included, height set in percentage: http://jsbin.com/sotahurilu/3/
DOCTYPE removed, height set in percentage: http://jsbin.com/sotahurilu/4/

Width seems to work without any issues. I believe this to be the case in both the latest version of Chrome and Firefox. I haven't tested any other browsers. Let me know if you need any additional information.
Thanks

@mourner So possible after all, huh? Maybe it would be nice to be less condescending and more helpful next time.
@mattlucas121 Thank you so much for identifying the issue I haven't been able to find out what's causing it! Now I can just go and set the height to 100% some other way (calculated via JS I Guess).

This is not a Leaflet bug. http://stackoverflow.com/a/1966377/52207 explains what's happening.

@jfirebaugh Thanks, that works.

html, body { height: 100% } doesn't work for me. Tried width:100% and both. Still no good. Tried in FIrefix and Chrome.

Hi @simon1tan,
Please note that this issue tracker is used for bugs. For support or questions please use Stack Overflow or gis.stackexchange.com.
If you are really sure this is a bug in Leaflet, please open a new issue and follow the issue template, in particular providing steps to reproduce and if possible providing a live reproduction case (like a JSBin, JSFiddle, Plunker, etc.)


C3 no internet connection after boot with 3rd party SIM card

Describe the bug
Every time I start the C3, I have no internet (bars). The APN is set ("net"). I use a 3rd party SIM card (Orange Romania).
It was working fine with another SIM card with another carrier (also from Romania). It happened roughly after I changed SIM cards.
The only way I can make it work is going to APN and saving it again, without editing it. I need do this every time I plug my C3.
Provide a route where the issue occurs
b387d7bf0f78af1b|2023-08-31--15-36-35
openpilot version
0.9.4
Additional info
Started happening on sunnypilot (twilsonco fork) and switched to official release3 for this bug report.
I preserved 3 consecutive routes:
b387d7bf0f78af1b|2023-08-31--15-36-35
b387d7bf0f78af1b|2023-08-31--15-41-19
b387d7bf0f78af1b|2023-08-31--15-55-49

Duplicate of #29205

Just to clarify for when #29205 is getting fixed. In my case the APN doesn’t dissapear from the menu. When I go to APN, the APN is already there.
Pressing Save in the APN menu, without editing it, restores the internet connectivity.

@adeebshihadeh how hard you think #29205 is to fix? do you think it could be a “good first issue”? I could try to fix it, if yes. thanks!

Yes it’s likely an easy fix.

https://github.com/commaai/openpilot/pull/30517


Netplan: Add guidance on setting up how-to environment

Context
Three of the how-to guides for Netplan feature very similar scenarios, and all of them start with the user having to configure their environment in a specific way. These are the how-tos:

How to configure a VM host with a single network interface
How to configure a VM host with a single network interface and three VLANs
How to configure a VM host with bonded network interfaces and three VLANs

In the beginning of each how-to, the user is asked to set up their networking, but specific instructions on how to perform such set up are missing.
Task
Either develop the content (instructions) on how to set up the environment at the beginning of each how-to and include it in the how-tos, or find suitable instructions in other docs and link to them (such as in existing Netplan how-tos).
Given there are a number of things that need to be set up, it's perfectly fine to provide instructions just for one or more of those -- you don't need to worry about not describing every single detail of the setup. Any improvement is worth it.
Note: It is rather probable that the same instructions on environment setup will apply to all three of the how-tos (or just with minor adjustments).
Reference

Diataxis: Tutorial
ReadTheDocs: Netplan documentation
GitHub: netplan

Hello @rkratky,
I'm interested in working on this task. If it's still available, could you please assign it to me?
Thank you!

Sure thing! Thank you for picking this up, @renukashiwakoti. Let me know if you need anything.


Fix broken link in Shopify transform package README

Fixes a broken link in the Shopify transform package README, as noted in Slack. The rest of the README's URLs are functional.

Backend change was needed in order for the integration tests to pass. Once that was updated I was able to see success. LGTM!

Thank you!


partial functions

We have decided we want to be able to support relational semantics for partial functions in cpmpy, this is however a long term goal.
For now the use of partial functions will be restricted to it's safe domain, to ensure backwards compatibility.
You can read more about the different semantics for partial functions here: “The Proper Treatment of Undefinedness in Constraint Languages”   https://www.cs.york.ac.uk/aig/constraints/AutoModel/undefinedness.pdf

For those having trouble with their index being out of bounds: the general solution is to create a new index variable, with the correct bounds, and adding a constraint that equals the new variable to you old index.
It is of course better to just reduce the bounds of the index you are using, if possible, since this will avoid the need for extra variables and constraints.


Reentrancy fix (TOB-OUSD-10)

💀

Done 👍
Made the public versions nonReentrant and switched all contract calls to the internal functions.


The costs of storage decrease when the losses increase

When the heat losses of storage are increased less heat is sold. The costs of the heat storage is based on euro/MWh of sold heat. As a result the costs decrease when the loss increases. When making a scenario the cost input should take into account the losses, but for a user this is not so obvious.

Fair point, I'll add a note to the slider description to remind the user to change storage costs as well when changing losses.
Ideally, this should happen automatically in the future. E.g. we could specify investment + O&M costs of storage rather than a fixed cost per MWh. However, it is currently not supported to calculate a number of units for the storage node, so we cannot specify investment costs per unit.

I think this is a valuable point still.


build: restrucutre github workflows

Changed to Terraform version 1.8 and trigger e2e tests using comments
Description
All the tests should now run using terraform 1.8
The end to end tests now takes about 20 minutes. As we dont need to do this for every new commit to the PR I propose that we can trigger this using a comment when needed. After the new changes it should now only run if you write the comment /start_e2e in the pull request or when creating a new release.
Fixes # (issue number)
Type of Change

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

Testing
See the testing docs for more information about how to test your code

[ ] Performed a local terraform plan, apply, and destroy to validate changes.
[ ] All tests are passing.
[ ] Added, updated, or removed tests when appropriate

Checklist
Before submitting this PR, please ensure the following:

[ ] My commits follow the Conventional Commits style guide
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings. I have ran terraform validate in the root module and all sub modules where I have made changes

Additional Information
Provide any additional information or context about the pull request here.

Thank you for contributing to Station!

/start_e2e

/start_e2e

/start_e2e

this is a test

/deploy


Vis and save embeddings

Fixes #86
Saves latent space

Training/val/test legend is interactive together with dots in scatter plot. I tried to make both training and classes legends interactive but reached some unexpected known behaviours that I thought it was not worth spending too much time on.
Example here:

Default view

Click on the scatter plot, it will select the class and data type given where you click

3 Use the interactive legend to change between data types for that class (ignore the sound symbol :-) ) .

Ok, that's fine then, thanks for trying :)


tour: mixing terminology "rune" and "character"

Context: https://tour.golang.org/basics/11
on tutorial page 11, it mentions "rune" (uint32), but on page 15 it is called "character" with no indication that it means the same thing

I think the page /15 should be modified to "Unicode code point" to be more specific, although the terms are used as synonyms from what I saw ( https://blog.golang.org/strings ).

The language spec (https://go.dev/ref/spec#Constants) says:

There are boolean_constants, rune_constants, integer_constants,
floating-point_constants, complex_constants, and string_constants.
Rune, integer, floating-point, and complex constants are collectively
called numeric_constants.

Rune constants are included in numeric constants, so neither is need here.


[release-4.16] deps: bump deployer dep

consume u/s fixes

/hold
for @Tal-or to review

@Tal-or lanes are failing with the exact error this PR should solve. But this IIRC should be caused by the very old bug/misconfiguration which causes the lane run with images built against EXISTING codebase (not the patched codebase changed by the PR under test), right?


feat: add params to plural translation

Add support for params in plural translations :
{
"user_apples": "{username} has no apple|{username} has one apple|{username} has {count} apples",
}

$tc('user_apples', {count: 10, username: 'Alice'})

<i18n-t
  keypath="user_apples"
  :plural="appleCount"
  :params="{ username: 'Alice' }"
/>

thx


No way to know if the data was served from cache.

Currently there's no property in the response object which tells me if the data was served from cache or if it had to execute the queryFn to return the response.
IIRC, previous version had an isCached flag which provided this information.
I tried to use isFetched flag. But it's set to true in both cases.
Is there something in the response object that can be used to determine if the data was served from cache?

I don’t think so. What do you need it for?


Converting a RLMResults to [RLMObject] most efficiently

As of now I'm using:
extension RLMResults {
  func toArray () -> [RLMObject] {
    var array = [RLMObject]()
    for result in self {
      array.append(result)
    }
    return array
  }
}

Is there a faster way available to get this conversion? I use it with generic global functions to implicitly cast the RLMResults to the appropriate RLMObject subclass.
public func RLMQuery <T:RLMObject> (_ predicate: String = "TRUEPREDICATE") -> [T] {
  return T.objectsWithPredicate(NSPredicate(format: predicate)).toArray() as [T]
}

class Photo {
  init (disk: RLMPhoto) {}
}

// first photo in the default realm
let photo = Photo(RLMQuery().first!)

// all photos by the user
let photos: [RLMPhoto] = RLMQuery("creator == \(user.id)")

Thanks! :grin:

I tried to replicate the solution that @aleclarson showed with generics.
func toArray<T>(ofType: T.Type) -> [T] {
    var array = [T]()
    for result in self {
        if let result = result as? T {
            array.append(result)
        }
    }
    return array
}

This allows me to do:
 let tracks = Track.allObjects().toArray(Track.self) // tracks is of type [Track]

I met the same problem using Swift and found a better way of doing that, without any enumeration at all.
Of course, it can be improved a lot more! It's currently a kind of wrapper for RLMResults, but sure can be used as a pure Swift dynamic collection instead.
extension RLMRealm {
	func allObjects<T: RLMObject>(fromClass cls: T.Type) -> RLMSwiftyResults<T> {
		return RLMSwiftyResults<T>(results: cls.allObjectsInRealm(self))
   	}
}

class RLMSwiftyResults<T: RLMObject> {
   	let results: RLMResults
    	init(results: RLMResults) {
        	self.results = results
    	}

    	func objectAtIndex(i: UInt) -> T {
        	return self.results.objectAtIndex(i) as! T
    	}

    	// And all other methods from RLMResults…
}

let mediaItems = RLMRealm.defaultRealm().allObjects(fromClass: MediaItem.self) // mediaItems is now a RLMSwiftyResults<MediaItem>
let firstMediaItem = mediaItem.objectAtIndex(0) // firstMediaItem is now a MediaItem

I also ran into this problem today
Since swift 2.0, we can do this (based on @duemunk's solution):
extension Results {

    func toArray() -> [Results.Generator.Element] {
        var array = [T]()
        for item in self { array.append(item as Results.Generator.Element) }

        return array
    }
}

You could just use map:
extension Results {
    func toArray() -> [Results.Generator.Element] {
        return map { $0 }
    }
}

For those with still working with RLMObject in Swift (objc & swift projects), you can put the implementation @duemunk used into one line:
extension RLMResults {
    func toArray<T>(ofType: T.Type) -> [T] {
        return (0..<self.count).flatMap { self.objectAtIndex($0) as? T }
    }
}

Thanks @duemunk!

@chrism21
extension RLMResults {
    func toArray<T>(ofType: T.Type) -> [T] {
        return flatMap { $0 as? T }
    }
}

Even nicer, thanks! @jpsim

Is there a way to do this on JS?

@cmajordev sorry I never saw your last message. RLMResults conforms to SequenceType if you're using RLMSupport.swift as we recommend in our installation instructions: https://realm.io/docs/objc/latest/#installation
flatMap is defined on SequenceType.
@DHidee66 I don't know, this is the cocoa repo. You can file issues towards the realm-js repo for questions like that: https://github.com/realm/realm-js/issues/new

@jpsim just wanted to say thanks for the sweet toArray extension!

@jpsim I guess your nice extension would work with Swift 2.x only?
I get compilation error:

Extension of generic Objective-C class cannot access the class's generic parameters in runtime

I guess this is a limitation.

@tomekc just use the Array() initializer as I mentioned in https://github.com/realm/realm-cocoa/issues/1046#issuecomment-143804257
let results: Results<...> // Results
let array = Array(results)

I did this and it seemed pretty neat.
let results = Results<MyObjectModel>
var dataSource = [MyObjectModel]()
dataSource.append(contentsOf: results)

Again:
let results: Results<MyObjectModel>
let dataSource = Array(results)

But if you're finding yourself copying data out of Realm Collections, you're losing many of their advantages, such as their live updating, lazy-loading, notifications & the ability to pass them across threads via ThreadSafeReference to name a few.

Hi guys I have an RLMObject
      @interface Job : RLMObject <NSCoding, NSCopying>
@property (nonatomic, strong) Contact  *contact; // this is another RLMObject
@property (nonatomic, strong) Location  *location; // this is another RLMObject
Its saving fine and once I try to do it also giving excellent result ,
RLMResults<Job *> *jobListArray = [Job allobjects];
but issue is once I get job object from this jobListArray its returning me Contact and Location Object Nil.If any one can help would be appreciated


Make the equivalence reasoner work with polymorphic relations

Consider the following example of a polymorphic equivalence relation R and a function f compatible with R:
axiomatization
  R :: "'a ⇒ 'a ⇒ bool"
and
  f :: "'a ⇒ 'a"
where
  equivp_R[equivalence]: "equivp R"
and
  f_R_compat[compatibility]: "⋀x. R (f x) (f (canonical R x))"

Also consider the following simple example lemmas:
lemma "R a b ⟹ R b a"
  oops

lemma "R a b ⟹ R (f a) (f b)"
  oops

The above-mentioned lemmas should be automatically proved by the equivalence reasoner. However, this is not the case:
lemma "R a b ⟹ R b a"
  by try_equivalence (*
    Failed to finish proof⌂:
    goal (1 subgoal):
     1. [b]⇘R⇙ = [a]⇘R⇙
  *)

lemma "R a b ⟹ R (f a) (f b)"
  by try_equivalence (*
    Failed to finish proof⌂:
    goal (1 subgoal):
     1. [f (canonical R a)]⇘R⇙ = [f (canonical R b)]⇘R⇙
  *)

Interestingly, the lemmas in question are successfully proved by the equivalence reasoner if we get rid of polymorphism (that is, if we replace 'a with something concrete like nat). Also, the problem disappears if we use a locale instead of an axiomatization.
Please note that the premise in both examples gets lost on the way, which seems to indicate that the problem may lie in the relax method: The relax method seems to lose the premise. We guess for it the R in the premise is different from the R in the conclusion because of some typing issue, which then results in the premise being dropped as illegal.
We shall fix the equivalence reasoner so that it works with polymorphic relations.

I must admit that I hadn’t polymorphism in mind when developing the next-generation equivalence reasoner, which is the root cause of this issue. In principle, it shouldn’t be difficult to understand why the current implementation shows the behavior you observed, as this most likely has to do with matching in the presence of type variables, which is quite well documented in the Eisbach User Manual. Changing the code such that it shows reasonable behavior is a different matter, though.
I’d argue that the equivalence reasoner should not just work in situations like yours, where equivalence relations, inclusions, and compatible functions are all fully polymorphic, but also in situations where partly polymorphic types like 'a list are involved and where type variables need to be specialized, like when an equivalence relation has arguments of type 'a list, but the conclusion uses arguments of type nat list. We should take care to test the solution with which I hope to come up with several examples involving such more complicated settings.
Before tackling this issue, I’d like to resolve #20, as this will considerably change the way matching is used, so that the resulting version of the reasoner will most likely need a different approach to handling polymorphism than the current version.

I’ve investigated this problem in quite some detail. Here are my findings:
Polymorphic rewrite rules use schematic type variables (things like ?'a) in their types. If these schematic type variables would always be handled in what I consider the most intuitive way, the equivalence reasoner probably would be able to sensibly cope with polymorphic rewrite rules. However, it seems that Isabelle, especially Eisbach, doesn’t always handle schematic type variables as one would expect. I found the following issues:

During the generation of relax inclusions, when searching for extensions of an inclusion S ≤ R via matching, schematic type variables in the type of S are apparently not instantiated, but just turned into non-schematic type variables, which seem to be fixed, causing the matching to fail. Apparently, only type variables introduced in for fixes of match patterns can be instantiated. https://github.com/input-output-hk/equivalence-reasoner/blob/f48fbae19717debac7c48ad97c1aff799088c4d5/src/Equivalence_Reasoner.thy#L210-L213
During relaxation, when checking whether the current rewrite rule uses the relation S from an inclusion S ≤ R, the same problem as described above occurs, with the same consequence. https://github.com/input-output-hk/equivalence-reasoner/blob/f48fbae19717debac7c48ad97c1aff799088c4d5/src/Equivalence_Reasoner.thy#L242-L245
If the check mentioned in the previous item is removed, the construction of the relaxed rewrite rule fails, even if the inclusion fits the rewrite rule. Apparently, also the THEN attribute, used in the construction of the relaxed rule, turns schematic type variables into fixed non-schematic ones. https://github.com/input-output-hk/equivalence-reasoner/blob/f48fbae19717debac7c48ad97c1aff799088c4d5/src/Equivalence_Reasoner.thy#L250
Strangely, the search for a base inclusion S ≤ R during the generation of relax inclusions apparently works even if the type of R contains schematic type variables, although matching is also used here. https://github.com/input-output-hk/equivalence-reasoner/blob/f48fbae19717debac7c48ad97c1aff799088c4d5/src/Equivalence_Reasoner.thy#L206-L217

The iprover-based approach has turned out to be unfeasible, and consequently we have decided to stick to the next-generation equivalence reasoner. As a result, the problem described here is relevant again, and so I’m re-opening this issue.


[SPARK-17617][SQL] Remainder(%) expression.eval returns incorrect result on double value

What changes were proposed in this pull request?
Remainder(%) expression's eval() returns incorrect result when the dividend is a big double. The reason is that Remainder converts the double dividend to decimal to do "%", and that lose precision.
This bug only affects the eval() that is used by constant folding, the codegen path is not impacted.
Before change
scala> -5083676433652386516D % 10
res2: Double = -6.0

scala> spark.sql("select -5083676433652386516D % 10 as a").show
+---+
|  a|
+---+
|0.0|
+---+

After change
scala> spark.sql("select -5083676433652386516D % 10 as a").show
+----+
|   a|
+----+
|-6.0|
+----+

How was this patch tested?
Unit test.

@cloud-fan

Test build #65693 has started for PR 15171 at commit 955f49a.

LGTM

Test build #65693 has finished for PR 15171 at commit 955f49a.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65693/
Test FAILed.

retest this please

Test build #65701 has started for PR 15171 at commit 955f49a.

Test build #65701 has finished for PR 15171 at commit 955f49a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65701/
Test PASSed.

Merged build finished. Test PASSed.

thanks, merging to master/2.0!

also 1.6


Element#fill should not clear the element first when element is a select

Clearing a select element produces an error with Chrome: invalid element state: invalid element state
I'm not sure if this should be completely removed but it breaks in this situation.

woops, this was not an issue with this library


Refactor HTMLRewriter Settings to make HTMLRewriter::new infallible

Add AsciiCompatibleEncoding wrapper type
Provide ways to convert between Ascii... and Encoding
Make new infallible
Fix tests, examples, and benches

This turned out very well - you only have to worry about fallible
conversions if you use an encoding other than the default, which is
fairly rare.
This is a breaking change.
Closes https://github.com/cloudflare/lol-html/issues/46.

This is currently broken because unwrap_or_ret_err (unstandably) requires a reasonable error to store in LAST_ERROR and right now I'm only giving it (). This is fixable by adding a new error type but I'm probably not going to get around to it any time soon.

This is fixable by adding a new error type

Done, I realized I could reuse the existing EncodingError.


remove broadcast com.tonyodev.fetch2.action.QUEUE_BACKOFF_RESET

Can the broadcast be changed to an observer in another way? The system application prohibits sending broadcasts.
Sending non-protected broadcast com.tonyodev.fetch2.action.QUEUE_BACKOFF_RESET from system

Issues with the broadcast has been fixed in the latest version

Oh , no fixed .
The system application can only send protected broadcasts.
Fetch still sendBroadcast in https://github.com/tonyofrancis/Fetch/blob/860e26d75c72b9c07f43371909e8962e5e95103a/fetch2/src/main/java/com/tonyodev/fetch2/downloader/DownloadManagerImpl.kt#L114

@Reginer
Released a new version that removes the broadcast and uses a callback instead. Please try version 3.4.1 and let me know if that fixes your issue. Thanks.

Elegant and silky smooth, wonderful beyond words.  Fixed


[Bug]: FAQ dark-mode heading

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Title should be black or of darker color for better visibility
Expected behavior
Title should be black or of darker color for better visibility
Add ScreenShots/Videos

On which device are you experiencing this bug?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a  IEEE-IGDTUW Open Source Week 2024 contributor
[X] I have starred the repository

@saumyayadav25 i have assigned you . u can start working on it & submit a new pR


Matrix url parameters break route matching

I'm currently configuring our existing single-spa app to use layout-engine.
<single-spa-router>
  <application name="navigation"></application>

  <route path="some-mfe">
    <application name="some-mfe"></application>
  </route>
</single-spa-router>

When navigating to mydomain.com/some-mfe I see the app being bootstrapped and briefly appearing. However as soon as any matrix parameters are added to the url (ex mydomain.com/some-mfe;pageSize=50;pageNumber=8) the app is unmounted by single-spa. Matrix parameters are commonly used in Angular as routing parameters.
It seems that the route matching algorithm/regex doesn't like the semicolon ?

correct, because matrix url parameters aren't an accepted web standard regardless of Angular's usage so support hasn't been added. however, i would think it would be relatively easy to support them in matchRoute.js which has an accompanying suite of tests. would the criteria be that those URL params are ignored during matching, just like search params are?


Integration renaming to PEP8 compatible

Renaming from ha-google-home to PEP8 compatible google-home.
Closes #69

Also, @DurgNomis-drol do you mind if we make the lovelace cards section a bit more minimalistic like so:


Stacked cards with and without generative art tag

https://kodadot.xyz/ahp/u/1KxzJHoRJHi8EgWVhrGDueeyzg8PyihAvSL3NP6QMcxEve6
What's the difference between left and right

👋

The right ones does not have metadata
This has been fixed in https://assethub-polkadot.subscan.io/extrinsic/0x8fe404b3e1073d39e2bb2daa72aed318b72915ee8a13b12e9dfbe83e52b55104
The left one are the legit generated same nfts by you - that should not happen - we can do ranges of sn based on the amount that you have minted

e.g

const STEP = 64
const entropyRange = [STEP * minted, STEP * (minted + 1)] // so you have 64 options per each minted NFT

Therefore you have 230 possible ranges with no overlap

👋


Change gRPC template logging level to info in development

Addresses #9408

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=161479

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2300
https://github.com/aspnet/AspNetCore-Internal/issues/2267

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=161548

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2332

This hasn’t passed yet!?

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=161600

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2332

@JamesNK just re-run the thing...


nvim-lsp: add noPackage option for servers

By default, for each enabled plugins.lsp.servers, nixvim installs the corresponding language server thanks to nixpkgs.
For some reason, a user might want to disable this behavior and limit the role of nixvim to wrap the nvim-lspconfig options without interfering more.
Hence, this PR adds a noPackage option (defaulting to false) that, if enabled sets the cmd option default value to null and ignores the package option value.
Please note that the package option does not accept null anymore.

Maybe an option wrapPackages set to true by default ? Negative booleans are best avoided I find


NotFoundError: Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node

Error with a library built using tsup-preset-solid
When using the library error occurs
If copy the library source code into project the error goes away
The error only occurs if something that is stateful is on top
Here's a reproducer
https://github.com/Qawaz/solid-start-issue
Library is built using tsup-preset-solid, You can consider that library only contains one file baseStyled which I've already included in the project If you make a package of that file the error occurs
I don't know where to report this, If this is not an issue with tsup-preset-solid and is an issue with something this preset relies on (esbuild-plugin-solid), I could report it further !

esbuild-plugin-solid relies on babel-preset-solid I am going to report this to babel-preset-solid


Choice of Integrators

Hi,
I'm a researcher and I'm trying to use MuJoCo for RL-related tasks.
I'm looking for some suggestions on the choice of integrators.
My question is for grasping-like contact rich scenarios, which integrator is the most appropriate, among available choices Euler, RK4, implicit etc, in terms of anti-slipping and less penetrations?
I searched over existing issues and found some relevant ones : https://github.com/google-deepmind/mujoco/issues/548#issuecomment-1293417831
but are not fully aligned with my targeted scenario.
Thanks for your help!

Integrators won't usually make a big difference for contacts. implicitfast is usually your best choice generally, RK4 and implicit are good in specific situations related to energy and momentum conservation.
We should update the relevant section about tuning contacts. (And choosing integrators, I suppose)
Ping us here if it doesn't happen by the end of the month.

Thanks for the reply and the general guidance. Looking forward to the updated section.

See end of this section (in Latest docs branch)
https://mujoco.readthedocs.io/en/latest/computation/index.html#numerical-integration

See end of this section (in Latest docs branch) https://mujoco.readthedocs.io/en/latest/computation/index.html#numerical-integration

Thanks for the followup on this issue. Great update on the doc. Thanks!


[Doc] Modules Update

This pull request has automatically created by PinpointAutoBot
Updated Item : [mariadb-java-client] has been updated
*Updated item list may differ if this PR is updated*

Codecov Report

Merging #6232 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6232      +/-   ##
==========================================
+ Coverage   39.52%   39.54%   +0.01%
==========================================
  Files        3051     3051
  Lines       91609    91609
  Branches    11932    11932
==========================================
+ Hits        36205    36223      +18
+ Misses      52396    52385      -11
+ Partials     3008     3001       -7

Impacted Files
Coverage Δ

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70% <0%> (-3.34%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.98% <0%> (+0.7%)
:arrow_up:

...point/rpc/client/DefaultPinpointClientHandler.java
73.05% <0%> (+0.97%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0%> (+1.42%)
:arrow_up:

...cluster/zookeeper/ZookeeperClusterDataManager.java
54.83% <0%> (+1.93%)
:arrow_up:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
88.74% <0%> (+4.63%)
:arrow_up:

...m/navercorp/pinpoint/rpc/client/ConnectFuture.java
90% <0%> (+5%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
68% <0%> (+16%)
:arrow_up:

Continue to review full report at Codecov.

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


HOI-A/HOI-W annotation json files

Hi Yueliao,
I'm running experiments on HOI-A/HOI-W dataset. In the src dataset/hoia.py there are two input json files: train_hoia.json, and test_hoia.json. I can't find them in the current repo.
From the PIC website in the BaiduDrive, is the 'trainval.json' the file for 'train_hoia.json' in hoia.py?
And, where is test_hoia.json? Need it to evaluate the test results. I also checked your PIC-HOIW github, but still cannot find it.
Thank you for your help!

All the above are responded through E-mail.

Hello, I'm also looking to experiment with the HOI-A dataset, how did you resolve this issue / can you make the files publicly available?

Hello, I'm also looking to experiment with the HOI-A dataset, how did you resolve this issue / can you make the files publicly available?

Hi, you can take a look at issue first.

Ah must have missed that. Thanks!


feat: include surrounding whitespace

Thanks for the useful plugin, being able to use the highlighted context as a textobject is really convenient.
I would like to request a feature/option to include surrounding whitespaces, like the builtin ap textobject.
In short, including the empty lines below the original selection target as a part of the textobject.
When the target is at the end of the buffer, empty lines above the target is included instead.
Example
foo
foo

bar <- cursor
bar

baz
baz

dap
foo
foo

baz <- cursor
baz

dap
foo
foo

Rationale
This is mostly intended for the require("treesitter_indent_object.textobj").select_indent_outer(true, "V").
Invoking a linewise indent-outer textobject means that you want to operate on the whole thing that has caused the indentation.
And it is very common to insert empty lines between functions definitions and control flow statements to improve readability.
After the function/control flow itself is gone, the separator empty line is no longer needed.
def foo(arg: str):
    if arg == "칼퇴근":  # <- cursor on this line
        return

    extra_work()
    imagine("happiness")

def imagine(arg: str):
    raise KeyError(arg)

For example, if I were to remove the guard clause with daI, I no longer need the empty line below it.
Similarly, daI on def imagine() should remove the empty line above it.
This is also useful for reordering functions, as you don't need to re-insert the extra empty line after pasting.
Existing Implementations
nvim-treesitter-textobjects has include_surrounding_whitespace option in its module config.
Related utility functions can be found at lua/nvim-treesitter/textobjects/select.lua

Thanks for the great suggestion!
I remember the implementation in nvim-treesitter-textobject is quite messy because it had to deal with many different vim settings which are not even resolved yet (https://github.com/nvim-treesitter/nvim-treesitter-textobjects/issues/488, https://github.com/nvim-treesitter/nvim-treesitter-textobjects/issues/575)
I can't guarantee when I can actually support this, but in the meantime you can do the following:
      {
        "aI",
        function()
          require("treesitter_indent_object.textobj").select_indent_outer(true)
          local prev_cursor = vim.api.nvim_win_get_cursor(0)
          vim.cmd [[normal! w]]
          local cursor = vim.api.nvim_win_get_cursor(0)
          vim.print(cursor)

          -- loop until line is not empty, or cursor didn't move
          local line = vim.fn.getline(cursor[1])
          while line:match "^%s*$" and cursor[1] ~= prev_cursor[1] do
            print(line)
            vim.cmd [[normal! )]]
            prev_cursor = { cursor[1], cursor[2] }
            cursor = vim.api.nvim_win_get_cursor(0)
            line = vim.fn.getline(cursor[1])
          end

          if cursor[1] ~= prev_cursor[1] then
            -- line is not empty. Adjust cursor position (col-1 or row-1)
            if cursor[2] == 0 then
              vim.cmd [[normal! k$]]
            else
              vim.cmd [[normal! h]]
            end
          end
        end,
        mode = { "x", "o" },
        desc = "Select context-aware indent (outer, line-wise, include trailing whitespaces)",
      },

Simply you can move around the cursor as you want after calling the function and it will be your new textobject.
Note it still has one problem where, when the indent level changes after the selection like below
    if True:
        pass  # cursor around here

if __name__ == "__main__":
    main()

After deleting with daI it will break the indentation
    if __name__ == "__main__":
    main()

I already have a better idea. Let's just use line-wise selection because it's an indent textobject anyway.
      {
        "aI",
        function()
          require("treesitter_indent_object.textobj").select_indent_outer(true, "V")
          local prev_cursor = vim.api.nvim_win_get_cursor(0)
          vim.cmd [[normal! w]]
          local cursor = vim.api.nvim_win_get_cursor(0)
          vim.print(cursor)

          -- loop until line is not empty, or cursor didn't move
          local line = vim.fn.getline(cursor[1])
          while line:match "^%s*$" and cursor[1] ~= prev_cursor[1] do
            print(line)
            vim.cmd [[normal! )]]
            prev_cursor = { cursor[1], cursor[2] }
            cursor = vim.api.nvim_win_get_cursor(0)
            line = vim.fn.getline(cursor[1])
          end

          if cursor[1] ~= prev_cursor[1] then
            -- line is not empty. Adjust cursor position (row-1)
            vim.cmd [[normal! k$]]
          end
        end,
        mode = { "x", "o" },
        desc = "Select context-aware indent (outer, line-wise)",
      },

We don't need to worry about the following indent mismatching issue.


Fixed shellcheck.net errors and switched to true temporary directory.…

Fixes done:

Fix all shellcheck.net errors
Create temporary directory with mktemp -d
Set executable file permisions

Kind regards,
gehaxelt

Hi @gehaxelt!,
Thank you very much for the improvements!, I have updated the project to v0.2 based on your feedback.
Thanks!
Mazin