feat(#228): Simplify XML Comparision in 'disabled' Integration Test"

In this PR I simplified XML comparision in 'disabled' integration test.
Closes: #228.
History:

feat(#228): make 'disabled' integration test pass
chore: remove the puzzle for 228 issue

PR-Codex overview
This PR updates XML file comparison logic in verify.groovy. It now compares XML files directly without ignoring spaces.
Detailed summary

Removed temporary space removal logic for XML file comparison.
Changed comparison logic to directly compare XML files.
Updated assertions to compare XML files directly.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 6min)


Edges in graph

https://github.com/schillij95/ThaumatoAnakalyptor/blob/613a8b1fb680509bbad51a57ad0ce63db65a56d3/ThaumatoAnakalyptor/instances_to_graph.py#L649
Given a patch p in a block A and an overlapping subvolume B, are we adding only in the graph the edge between p and the patch q in B for which the certainty is maximal? Isn't this too restrictive, especially if the estimate of the weight is not accurate?

Good point. It might be better to take the top n or so!
How would the ideal graph given our instances look like?
Some ideas:

Same Block edges
Other Block edges
Good certainty estimation function: What properties? Use distance from umbilicus? use 2.5D surface alignment? ...
Discard unlikely edges/nodes to make the problem smaller. (scalability)
...

Top-n seems a good idea!
For the weight function we can use:

Sinkhorn divergence (approx of Wasserstein distance), this gives a similarity in shape
Cosine similarity between eigenvectors of PCA, this gives a proximity in orientation
Relative distance from umbelicus
... Others?

We need a weight function that imposes the partial ordering we want and takes into account the highest number possible of "edge-cases"


Update NPM version.

@IvanAxilis Are you able to push the new version to npm?

I have opened PR for it; https://github.com/axilis/react-native-responsive-layout/pull/45
Will release as soon as it is merged :)

Closing this issue as v1.2.0 was released to npm.


pypi.io only showing the sdist  / pypi.python.org only showing the wheel

We just made a release of ipywidgets. Both sdist and bdist_wheel.
Oddly, the source tarball only appears on pypi.io and the wheel only on pypi.python.org
https://pypi.org/project/ipywidgets/6.0.0.rc1/#files
https://pypi.python.org/pypi/ipywidgets/6.0.0rc1
cc @jasongrout

We attempted an rc2  release and are observing the same behavior.

Closing. this is an issue with rc1 vs .rc1. Sorry for the noise.


maxHeap, minHeap and Heapsort

🚀 Feature
Implementing maxHeap, minHeap and heapSort in C language.
Have you read the Contributing Guidelines on Pull Requests?
yes
Motivation
I would like to work on this under GSSoC '21
Pitch
I have found heap implementation in C++ and python but not in C language. Also, I would like to write the code for heapsort in C. Please assign this to me.
in referrence to: https://github.com/TesseractCoding/NeoAlgo/issues/2541#issuecomment-797266194

You can assign the Issue for yourself by using the '/assign' command. Make sure The algorithm/ data structure you are trying to add doesn't already exist. read the Doc before you proceed.

@taruni-always I assigned you, Happy Coding!😊

@ankitaggarwal23 Thank you for assigning it tome. On it now!

Please add the level labels for this.

@taruni-always Don't worry labels will be added to all the issues and merged PR's by this weekend. Our team is working on it.

@ankitaggarwal23 Should I create a seperate directory for heaps or should I add the code i C/ds folder?

@yes you have to add your code in C/ds section and also same thing you have to update in the readme as well.


PS5

I thought PS5 is dead, it's strange to see new products introducing their programmatic support using it.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1c1e2969-8162-7cad-99b7-803ba089081d
Version Independent ID: cca0134b-5685-ae09-f9f3-275426e6e3b8
Content: What is Azure AD Connect v2.0?
Content Source: articles/active-directory/hybrid/whatis-azure-ad-connect-v2.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

Hi @gogbg are you talking about PlayStation or some other technology?
We will now proceed to close this thread since the feedback for the document is unclear.
If needed, please create a new issue detailing your feedback for this document, or ask a question in Microsoft Q&A. https://docs.microsoft.com/answers/topics/azure-active-directory.html?filter=unanswered

Of course that I'm not talking about PlayStation, I'm talking about PowerShell


Bugs/152

Add buttonProps prop to allow passing additional props to the button component.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add close() to type definition

close function as described in the documentation here > https://docs.surrealdb.com/docs/integration/sdks/nodejs#close  not added to type definition

this doesn't do anything to the code. the actual implementation should be done in rust


Google Chrome

Hi,
I'm trying to use your Tabs app but it seems I can only do any customization using Edge and not in Google Chrome.
Any ideas?
Thanks

Bug fix with version 1.0.4 (https://github.com/OlivierCC/spfx-40-fantastics/releases/tag/1.0.4). Thanks for this bug report.


No pretict masks output

Hi,i am trying to predict imgs with using predict_folder.py
It doesn't look wrong but after command,there is still nothing in the folder {predicted_masks}
Could you help me solve it?
Any comments are greatly appreciated
Thank you

I solved it,wrong environment.
Thank you

Glad you solved it.


Movement handler stuck when clicking certain key combinations

What happened?
If you do the reproduction steps, you get stuck with dragging mode even when you do not hold LEFT_CLICK.
https://github.com/CesiumGS/cesium/assets/27568710/9809b620-4960-4b9b-b8ba-47f8bd3b28cd
Reproduction steps

Press and hold LEFT_CLICK Down
Press and hold ALT Down
LEFT_CLICK Up
ALT Up

Sandcastle example
No response
Environment
Browser: Google Chrome Version 123.0.6312.59 (Official Build) (64-bit)
CesiumJS Version: 1.115
Operating System: Windows 10

Thanks for the report @Dphalos, I think this is an oversight in ScreenSpaceCameraController.

Is this a result of our intentional choice to separate mouse events based on what modifiers are held down? So the event at step 3 in the reproduction steps is "Left click up + alt" instead of just "Left click up"?


Horizontal Page Links broken after Spicetify v2.35.

Environment

Spotify Version: <IP_ADDRESS>2.g26c92729
Spicetify Version: v2.35.1
Platform: Win 11 (64bit)
Runtime: Chrome

Describe the bug
The "Horizontal Page Links" option is broken. It only shows the first two horizontal and the rest of them vertical after Spicetify v2.35 release.
Screenshots
Ignore the different symbol of the second custom app. I just only had an old screenshot with other customapps installed.

after v2.35

before v2.35

will be resolved once beta branch is merged into main


Adding missing junit tests for packages persondir, tenants, portlet/r…

…egistry, uportal-content

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included
[x] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

/cc for review @cbeach47


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1177 ms

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


Failed when I try to add a proto file with import syntax.

Protoman-0.3.1.dmg

Failed when I try to add the following protos.
account/api/account_test.proto -> Protobuf cannot be read
syntax = "proto3";

import "account/data/account.proto";

message AccountList {
    repeated account.data.Account accounts = 1;
}

account/data/account.proto -> Added successfully.
syntax = "proto3";

package account.data;

import "common/valid.proto";

message Account {
    int64 account_id = 1;
    string first_name = 2 [(valid.pattern) = "[a-zA-Z]{5,20}"];
    string last_name = 3 [(valid.pattern) = "[a-zA-Z]{5,20}"];
    string email = 4 [(valid.email) = true];
    string phone = 5 [(valid.pattern) = "[a-zA-Z]{5,20}"];
}

Thanks for the issue! I'll investigate and let you know. It seems like an issue with what root directory we use to parse the protobuf files.

@spluxx If i want buid a dmg package from source code, what do I need to do ?

Run npm run dist:mac :)

Run npm run dist:mac :) Did the patch work for you?

Thanks, the patch works for me. But I found another problem, as follow:

I have imported all proto files into Protoman, and try to send a message, but it fails.
no such type: .account.rest.api.CreateAccountRequest

As you can see, .account.rest.api.CreateAccountRequest has been imported and chosen as the request message.

@DoneSpeak would it be possible for you to try the fix on the v0.3.3 branch and see if there are additional problems? Thank you for the patience :)

No problem. I will take more try today. It's my honor to help you to improve Protoman. I holp it will work well.

@spluxx Maybe I can try to debug by myself. Could you please tell me what I have to learn for this project besides Electron and TypeScript

><EMAIL_ADDRESS>start /Users/yangguanrong/Codes/Github/Protoman
> electron dist/main.js

MAIN PROCESS STARTED
Initializing events
data directory: /Users/yangguanrong/Library/Application Support/Electron/data
Made sure datadir exists
Starting cleanup process
Finished initializing.
Loading most recent state...
Done loading most recent state...
Saving...
Saving...
Saving...
Making request with nonce: 4464740
Saving...

I don't know what's that means.

index out of range: 128 + 10 > 128

index out of range: 128 + 10 > 128

This means that the given buffer had 10 more bytes than what's expected from the schema.
Perhaps the data that you're sending back from the server doesn't match what you selected on "Expected Message" tab?
The only way I was able to replicate that error was by setting the wrong expected message.

You are right. I tried again and it worked. Maybe you can add a panel to show the raw response which would be very helpful to let the user speculate the error reason. Of course, it would be better if the message shows the probable cause.

Nice! For the improved error messages and adding all protobuf files in folder-level feature, I'll create a separate issue for them and tag you on it :) Thank you for the contribution!

Oh and @DoneSpeak if you want to try tackling one of those issues let me know :)

You are welcome. Yes, of course. I would love to do it. And I will spend some time to learn  react + typescript + electron.
By the way, could you please tell me how to package it for windows and ubuntu system?


Use --follow-tags flag for git push

docs: https://git-scm.com/docs/git-push#Documentation/git-push.txt---follow-tags
The --follow-tags flags will make this one command instead of two, but it will also prevent pushing all of your local tags, and instead only pushes the ones that are missing from the remote but are pointing at commit-ish that are reachable from the refs being pushed. In practice this means that only the tag just created by npm version will be pushed ☺

Thanks!


Upgrade RN image-manipulator package to allow Android GIF avatars

Details

Enables uploading of GIF avatars on Android.
This change compliments the merged PR on the react-native-image-manipulator fork.
Fixed Issues

$ https://github.com/Expensify/App/issues/14751
PROPOSAL: https://github.com/Expensify/App/issues/14751#issuecomment-1420266690
Tests

Add an animated GIF image to an Android simulator or device
Log in to the app, and navigate to the profile screen
Press on the current avatar to change it, and press "Upload photo"
Press "Choose from gallery" and select the GIF image on your device
Proceed to crop, rotate, and zoom the avatar as desired
Press "Save" to finish uploading the avatar.
Observe that the avatar was uploaded successfully
Observe that the saved avatar is a static image that only contains of the first frame of the animation

[X] Verify that no errors appear in the JS console

Offline tests

To test functionality offline, disable the network on your device and proceed with the steps above.  After uploading the GIF your avatar should appear slightly transparent to indicate that it was not fully uploaded.  Change network status to online and confirm that the avatar was uploaded.
QA Steps

Add an animated GIF image to an Android simulator or device
Log in to the staging environment, and navigate to the profile screen
Press on the current avatar to change it, and press "Upload photo"
Press "Choose from gallery" and select the GIF image on your device
Proceed to crop, rotate, and zoom the avatar as desired
Press "Save" to finish uploading the avatar
Observe that the avatar was uploaded successfully
Observe that the saved avatar is a static image that only contains of the first frame of the animation

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Screencast from 2023-02-13 07:59:51 PM.webm

Mobile Web - Chrome
rnim-android-chrome2.webm

Mobile Web - Safari
ios-safari-rnim-test2.webm

Desktop
desktop-rnim-test.webm

iOS
ios-rnim-test2.webm

Android
Before change (uploading GIF generates error)
Screencast from 2023-02-13 04:28:30 AM.webm
After change (uploading GIF successfully)
Screencast from 2023-02-13 04:34:10 AM.webm
Demonstrating rotate, zoom, and crop in one action using GIF
Screencast from 2023-02-13 08:42:18 PM.webm

@Beamanator @mananjadhav One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@mananjadhav PR ready for review.

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/3069065/219549505-2ff2fbe9-221e-4cd6-835a-5fcf918ab5ef.mov

Mobile Web - Chrome
https://user-images.githubusercontent.com/3069065/219550694-2dbaf093-0d8e-49ea-a8b1-7667397add71.mp4

Mobile Web - Safari
https://user-images.githubusercontent.com/3069065/219549525-7022cb55-c446-428f-b3d3-15f288fbab1f.mov

Desktop
https://user-images.githubusercontent.com/3069065/219549572-30e617a8-ce0d-4ff5-b149-f8f01a713213.mov

iOS
https://user-images.githubusercontent.com/3069065/219549585-ff79a4f1-5e9f-4438-b6e3-1ec2ce6509c2.mov

Android
https://user-images.githubusercontent.com/3069065/219550703-34305748-9a4c-4b12-a85f-32c19887bfe1.mov

Thanks for the PR @redstar504. This works well. @Beamanator All yours.

Oops yep i'll get to this very soon :D

🚀 Deployed to staging by https://github.com/Beamanator in version: 1.2.75-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/melvin-bot[bot] in version: 1.2.75-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Change to use instance-identity/document to fetch region directly

Issue #, if available:
#466
We are using ZONE to infer the REGION which is not the best way. There're edge cases like #466 describes that this method can not cover. The most elegant way is to read REGION directly from ec2 metadata service.
https://github.com/awslabs/amazon-eks-ami/blob/master/files/bootstrap.sh#L198-L199
http://<IP_ADDRESS>/latest/dynamic/instance-identity/document provides the information and here's an output example
{
  "accountId" : "xxxx",
  "architecture" : "x86_64",
  "availabilityZone" : "us-west-2d",
  "billingProducts" : null,
  "devpayProductCodes" : null,
  "marketplaceProductCodes" : null,
  "imageId" : "ami-0c13bb9cbfd007e56",
  "instanceId" : "i-040xxxxx5b",
  "instanceType" : "m5.large",
  "kernelId" : null,
  "pendingTime" : "2020-04-02T01:03:19Z",
  "privateIp" : "192.168.x.169",
  "ramdiskId" : null,
  "region" : "us-west-2",
  "version" : "2017-09-30"
}

Description of changes:
Change to use instance-identity/document to fetch region directly
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mogren address the comment. Thanks for the suggested scripts. :D


Usage

Just wondering how to use stc, at its current state.
Do I need to consume it as a library or run it through a test shim?
I cloned the repository, updated the dependencies through yarn and compiled the project with rustc build.
Alas, the resulting stc binary only contains two subcommands (help and iterate) and simply providing a TypeScript source file simply returns a Found argument '<pathToTSFile>' which wasn't expected, or isn't valid in this context

It's under active development

@markusbkk depends what you mean by 'use'. It's trivial to implement a new command (e.g. stc check <file-path>). You will need to update https://github.com/dudykr/stc/blob/main/src/main.rs and https://github.com/dudykr/stc/blob/main/src/check.rs. But as @kdy1 says, it's under active development so there is currently no way to use it like you would tsc.

@markusbkk depends what you mean by 'use'. It's trivial to implement a new command (e.g. stc check <file-path>). You will need to update https://github.com/dudykr/stc/blob/main/src/main.rs and https://github.com/dudykr/stc/blob/main/src/check.rs. But as @kdy1 says, it's under active development so there is currently no way to use it like you would tsc.

Understood
I wasn't aware that it was still very much pre-alpha


The chart "hydrogen production for transport" should not respond to adding power to gas for industry

Right now, adding power to gas for industry influences the "Hydrogen production for transport" chart.
The p2g_in_hydrogen_production gquery seems to include all P2G conversion, not just P2G for transport.
I think we should either change the queries to just include P2G for transport, or change the slide (text, chart, etc), to P2G general.
This image shows a scenario with no P2G for transport, but with some P2G for industry.

Including @ChaelKruip and @jorisberkhout

I think we should either change the queries to just include P2G for transport,

Sounds like the way to go!

I just found out that all queries for this chart are actually more general then transportation. Perhaps we should think a bit more about what we actually want to show.

Reconsidering I think that this chart originates from the time that in thet ETM hydrogen was just used for transport.
I think there are two things "wrong" with the queries in combination with the current chart title

The chart name is "Hydrogen production for transport", but the queries consider all hydrogen production
The chart name is "Hydrogen "production for transport", but the P2G in hydrogen production.gql extracts export. So it doesn't show production, but production minus export. Currently excess production results in large
P2G is not part or the queries

2 in combination with 3 results in negative Power-to-gas in the chart when a scenario has excess P2G for industry
My proposal:

Change the chart title to "Hydrogen production"
Remove the subtraction of hydrogen export from P2G in hydrogen production.gql
This way the chart would correspond better to for example the chart showing electricity production in which production is shown and export and curtailment is not subtracted for example from wind.
Add P2G for industry to P2G for hydrogen production.gql

, so change the query to
query = DIVIDE((V(energy_flexibility_p2g_electricity,output_of_hydrogen) + V(industry_flexibility_p2g_electricity,output_of_hydrogen)),BILLIONS).

Assigning myself as well and adding deploy milestone. I am going to work on related issues (quintel/etmodel#2397, quintel/etmodel#2354, quintel/etmodel#2272 and quintel/etmodel#2257) tomorrow morning and will look into this one as well.

Moved to October deploy.

We decided to changed the graph such that hydrogen production for transport is shown, rather than all hydrogen production (see: https://github.com/quintel/etsource/pull/1399). The rationale behind this is that the chart is shown on the H2O for transport slide, so the chart now (only) shows how this H2O for transport is produced (p2g, methane reforming, methane reforming ccs, local electrolyses). Ive added a demand bar to show when p2g production exceeds hydrogen for transport demand. I will create a similar graph for hydrogen production for the fertiliser industry.
As @ChaelKruip noted, the queries are more general than transportation (hydrogen production happens in the energy sector) but so far the hydrogen produced there is only used for transport (or export). If this were to change in the future, the queries of this chart have to be updated.


Having to restart exe when feeds.json is changed is annoying

Would be better to monitor the file and re-read it automagically upon changes.

I think fsnotify is overkill, simply looking at changes in the file modtimes and/or size will be good enough.

Is it that bad tho? This goes away when config editing is put in the UI. Hmmm.

I don't think this is necessary now there is editing of feeds in the UI.


File database relative path connection string issue

From @andrew-vandenbrink on February 19, 2016 11:2
Not sure if this can be solved but an issue anyway
This is the configuration

When developing with local machine its expect the databases to reside in the same level of source code

When you deploy it, it expect the databases to be resided at

where it "ideally" it should expect the databases to reside at approot/src/{project_name}/
Of course I know there is workaround to put additional production appsettings.json and use different path.
Copied from original issue: aspnet/Identity#751

@Eilon any insights on this feedback and what the "ideal" location for a database file would be at development time and on deployment?
FWIW, our SQLite provider simply uses AppContext.BaseDirectory (or equivalent) to resolve file name locations in the connection string.

From @blowdart on March 2, 2016 22:5
This really isn't an identity issue, it's more a configuration one. @Eilon where should this bug end up?

From @Eilon on February 27, 2016 18:59
I have no strong opinion on this, as long as it's not in wwwroot :smile:

From @MaximRouiller on February 29, 2016 13:47
App_Data? :stuck_out_tongue_winking_eye:

It a question/suggestion about what should the base path be at development and deployment so I don't think @blowdart means "configuration" in the aspnet/Configuration sense.
Maybe aspnet/Hosting?

I'm not sure this belongs anywhere. It's about guidance, there's no code change here

@davidfowl Not true, if the code use this
IApplicationEnvironment app;
app.ApplicationBasePath;

It will ensure the root path of application will be at

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Projecting 3D labels from point cloud to 3D labels in range image

Hi,
I have tried the 3D box projection from point cloud labels to camera images and it works.
But, I wish to do object detection using range images and hence want to generate 3D labels in the range image, using the labels from the point cloud.
Is this possible?
In other words, can I take a box label in the point cloud and get the same object in the corresponding in the range image and then draw a bounding box around it?
Please explain with a short code if possible.
Best Regards
Sambit

Hi,
See my response to issue #2 which you opened earlier. I don’t have the time to write and test code for you however if you take inspiration from the links that I have included, it shouldn’t be too difficult to do what you want.


Ensure  nodes are not removed after syntax highlighting

After upgrading to rouge, I noticed the syntax highlighting in octodown has
disappeared. It seems that the pre tags were being removed, which are
neccesary for correct highlighting with rouge. This commit fixes this by
replacing the inner_html of the preblock with the code highlighted by rouge.

Looks great! Thanks.


Is there any way to parse dynamic unknown JSON keys in the same order like JSON provided?

Your JSON dictionary:
{
        "roomType": {
                "SSS": {
                       "name": "Deluxe King"
                },
                "ABC": {
                       "name": "Executive King"
                },
                "SXW": {
                       "name": "Theme Suit"
                }
        }
}

Your model:
class RoomType: Mappable {
    var data: [String: Room]?

    required init?(map: Map) {

    }

    // Mappable
    func mapping(map: Map) {
        data   <- map["roomType"]
    }
}

class Room: Mappable {
    var name: String?

    required init?(map: Map) {

    }

    // Mappable
    func mapping(map: Map) {
        name   <- map["name"]
    }
}

What you did:
if let result = Mapper<RoomType>().map(JSONObject: json) {
       if let data = result.data {
              for (key, value) in data {
                     print(key)
              }
       }

}

What you expected:
What I expect is, I want to get the order exactly like json string
"SSS", "ABC", "SXW"

What you got:
"SXW", "ABC", "SSS"

What I get is dynamic order due to dictionary container. So, is there any way to parse unknown dynamic key with the server JSON response provided order? Like Android Map?

Since you are using data as dictionary, have you tried data.sorted.. method?


Show suggestion pane again after hitting enter

type
make selection with keys
--> suggestion pane hides
type again for new search
==> no suggestion pane!

Fixed and released in 1.4.0.

Does anyone is still seeing this error?
After make the selection with keys and press enter, the selection pane never comes back.


why do i get different numbers of values using page queries and api queries?

i have a metric named 'test'  with  a  scrape_interval : 2s .  and  i get differrent numbers of values when using page queries and api queries .
like this "test[30s]" , i get only 3 values when using page queries but get 15 values when using api queries .
Why do I get different numbers of values using page queries and api queries?

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Use new methods that cut out the student_school_year_model ...

... when computing school year absences/tardies/discipline for the school overview page and student profile page
Plus a bunch of Sublime text whitespace autofixes.

Issues

Implements methods from #931
Works toward #611

These new methods yield the exact same distribution of discipline incidents, absences, and tardies in prod as the old methods did. 👍


vagrant provision setup error

hi, I used the directive in the readme to launch the playbook with vagrant, but I encounted an error:
==> default: Running provisioner: ansible...
ERROR! Specified --limit does not match any hosts
Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

I already renamed the Vagrantfile and the ansible_hosts file, also uncommented the line in imgur-display-setup.yml to vagrant user. all with the instructions in the readme. The vm was initialized and up but provision step throwed the error.  I don't know where is wrong, could you help me ?

I solved the problem, adding this line in the Vagrantfile:
config.vm.provision :ansible do |ansible|
    ansible.playbook = "imgur-display-setup.yml"
    ansible.inventory_path = "ansible_hosts"
    ansible.limit = "all"  ### <<-- adding this line
 end

@lhrkkk fix worked for me. Thx

Thanks for the fix!

Awesome

thank you! 🙏


Add markers for the moments jit code is added into the a process

This will be useful for comparing when functions compiled with particular JIT tiers become available.
The implementation would detect jitdump style .so's and include a marker at the time the mmap happens along with the single symbol that they contain.

It looks like mmap2 events don't necessarily come with timestamps so this might be a little more painful than it needs to be.


Padding on columns

Hi Guys ,
I need some assistance please , i needed 2 columns but then need to insert a space between them . padding would be ideal i think and according to you documentation  your columns don't support padding.
Please can you assist me or point me in the right direction.

Hi @lrae24, this is not "natively" possible but we look forward to adding support for gutters. However, there are a few workarounds, such as adding an empty column between the columns:
<mjml>
  <mj-body>
    <mj-container>
      <!-- First solution -->
      <mj-section>
        <mj-column background-color="red" width="49%">

          <mj-image width="100" src="/assets/img/logo-small.png"></mj-image>

          <mj-divider border-color="#F45E43"></mj-divider>

          <mj-text font-size="20px" color="#F45E43" font-family="helvetica">Hello World</mj-text>
        </mj-column>
        <mj-column background-color="#FFFFFF" width="2%">
        </mj-column>
        <mj-column background-color="blue" width="49%">

          <mj-image width="100" src="/assets/img/logo-small.png"></mj-image>

          <mj-divider border-color="#F45E43"></mj-divider>

          <mj-text font-size="20px" color="#F45E43" font-family="helvetica">Hello World</mj-text>

        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

It should look ok in all clients: https://litmus.com/checklist/emails/public/ba3d879
PS: prefer the v3.2.2 for this, there's an issue in the beta version of v3.3.0 when using such a layout, it will be fixed next week.
I'm closing as it's a duplicate of #160.

Thanks but just one more question i have tried this  now and currently it works fin on desktop and on mobile the columns no fall underneath each other  but they exclude the spacing added that you see on desktop mail clients so is there any solution that caters for both scenarios  , as the column solution you gave is perfect for desktop  but the spacing falls away or is non existent on mobile??

You can probably add a mj-spacerin it


Research if csi-driver-iscsi works on Windows.

The effort here is to understand csi driver capabliities against Windows worker nodes.
I am capturing the details here for now, later it can be converted to sub tasks/issues:
https://hackmd.io/oYCS4DviSY2_TTMLft7YPA

Instead of https://github.com/cloudbase/wnbd ?

Instead of https://github.com/cloudbase/wnbd ?

The experiment here aims an alternate way for mapping /mounting RBD devices in Windows compared to winbd. This will be purely based on CSI protocol spec.

Instead of https://github.com/cloudbase/wnbd ?

NBD exporting and consuming is indeed an alternative. Ideally there should be a csi-driver that supports it on both Linux and Windows. Because csi-driver-iscsi exists, and Ceph can export RBD images over iSCSI already, this might be a relatively simple approach.

As far as my research goes the solution here would be :

Run Windows Nodes and make it part of the kube cluster
Run csi-proxy inside the Windows worker node to facilitate the  CSI actions on the windows node
Then deploy csi-driver-iscsi as the CSI driver. That should help us to get the ISCSI mounts working, rest would be the experiment on exporting RBD volumes as iscsi shares and making it accessible in the nodes

Listing limitations is definitely good!
Please make sure to add links to the documentation and other resources where you found the information in your HackMD notes.

Listing limitations is definitely good!

:+1:

Please make sure to add links to the documentation and other resources where you found the information in your HackMD notes.

Sure @nixpanic , updating the details over there :+1:

https://kubernetes.io/docs/setup/production-environment/windows/intro-windows-in-kubernetes/
This is good to mention. The table "Support matrix"  seems to come from this doc.

As far as my research goes the solution here would be :
1. Run Windows Nodes and make it part of the kube cluster

2. Run `csi-proxy` inside the Windows worker node to facilitate the  CSI actions on the windows node

3. Then deploy `csi-driver-iscsi` as the CSI driver. That should help us to get the ISCSI mounts working, rest would be the experiment on exporting RBD volumes as iscsi shares and making it accessible in the nodes

It seems that, even though csi-proxy surface iscsi interface, csi-driver-iscsi is lack support for its windows version. If thats done, ideally we should be good to get this combo working.

Research Doc : https://hackmd.io/oYCS4DviSY2_TTMLft7YPA
From the restrictions listed above, it looks like we can not do "block device mapping" to windows containers. What that means to us ? it looks like we can not expose PVC with block mode to windows containers . May be most of the users are fine with filesystem mode, but listing it out the limitation.

As far as my research goes the solution here would be :
1. Run Windows Nodes and make it part of the kube cluster

2. Run `csi-proxy` inside the Windows worker node to facilitate the  CSI actions on the windows node

3. Then deploy `csi-driver-iscsi` as the CSI driver. That should help us to get the ISCSI mounts working, rest would be the experiment on exporting RBD volumes as iscsi shares and making it accessible in the nodes

It seems that, even though csi-proxy surface iscsi interface, csi-driver-iscsi lack support for its windows version. If thats done, ideally we should be good to get this combo working.

Research Doc : https://hackmd.io/oYCS4DviSY2_TTMLft7YPA
From the restrictions listed above, it looks like we can not do "block device mapping" to windows containers. What that means to us ? it looks like we can not expose PVC with block mode to windows containers . May be most of the users are fine with filesystem mode, but listing it out the limitation.

[Update]
Managed to get the windows server running cluster
Also with good amount of hacks, I could get the csi-proxy windows version..
Just to mention, the subjected/related  projects looks at early stage, so good amount of hacks required to get the experiment going.. :)

............

So to sum up: it won't work just yet, and in order to make it work, we need to teach csi-driver-iscsi to delegate to csi-proxy on windows nodes. This is the main task, and then we need to eliminate the rough edges.

It seems that, even though csi-proxy surface iscsi interface, csi-driver-iscsi lack support for its windows version. If thats done, ideally we should be good to get this combo working.

I think that was the short version of what I wrote above, but had to read the code to understand what you were saying. wink

Exactly @obnoxxx . indeed this need good amount of work! and thats the current state. I updated the hackmd and also discussed in detail  in CSI call.
To elaborate some more on this point:  The current iscsi csi driver indeed lack the windows support. When we say it lack the support,  first of all it has to run inside the windows node, and to support iscsi server operations , the csi iscsi driver has to be implement the csi proxy APIs which I have listed in the issue referenced in earlier comment. Just to mention, it has to implement APIs like below
  rpc AddTargetPortal(AddTargetPortalRequest)
      returns (AddTargetPortalResponse) {}

  // DiscoverTargetPortal initiates discovery on an iSCSI target network address
  // and returns discovered IQNs.
  rpc DiscoverTargetPortal(DiscoverTargetPortalRequest)
      returns (DiscoverTargetPortalResponse) {}
...
  // ConnectTarget connects to an iSCSI Target
  rpc ConnectTarget(ConnectTargetRequest) returns (ConnectTargetResponse) {}

  // DisconnectTarget disconnects from an iSCSI Target
  rpc DisconnectTarget(DisconnectTargetRequest)
      returns (DisconnectTargetResponse) {}

Making/teaching iscsi csi driver to run and support windows version need effort and support from community. We have to see how this work out against release plans..etc. The linux part also has to be scrutinized with more testing..etc. So there is good amount of work left to get this going.

Research Doc : https://hackmd.io/oYCS4DviSY2_TTMLft7YPA
From the restrictions listed above, it looks like we can not do "block device mapping" to windows containers. What that means to us ? it looks like we can not expose PVC with block mode to windows containers . May be most of the users are fine with filesystem mode, but listing it out the limitation.

Thats my understanding too.

[Update]
Managed to get the windows server running cluster
Also with good amount of hacks, I could get the csi-proxy windows version..

As @nixpanic mentioned, would be great to document these hacks instead of mentioning that hacks were needed. smile

The experiments were carried out in a loaned cluster  with very short lived life ( a few hours left before it auto decommissioned) and was facing issues here and there to get it going , also it was first attempt on windows, so hiccups here and there. will try out once again and document it correctly, not sure all those were required though.

Just to mention, the subjected/related projects looks at early stage, so good amount of hacks required to get the experiment going.. :)

Thats the summary so far !

[Status update and some findings after going through various github issues/code/PR..etc on this topic]  iow, where we are at:

Good progress on having first release of csi-iscsi-driver which include the Linux part of Node Controller.
Along with other PRs and work, the release is getting close:

Have to address the comments along with few other things https://github.com/kubernetes-csi/csi-driver-iscsi/pull/45#issuecomment-848625777
NAME                       READY     STATUS    RESTARTS   AGE
pod/csi-iscsi-node-7gwkp   3/3       Running   0          15m
pod/csi-iscsi-node-8dvcj   3/3       Running   0          15m
pod/csi-iscsi-node-b26rh   3/3       Running   0          15m
NAME              TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)                        AGE
service/kubelet   ClusterIP   None         <none>        10250/TCP,10255/TCP,4194/TCP   3h43m
NAME                            DESIRED   CURRENT   READY     UP-TO-DATE   AVAILABLE   NODE SELECTOR            AGE
daemonset.apps/csi-iscsi-node   3         3         3         3            3           kubernetes.io/os=linux   15m
[hchiramm@localhost csi-driver-iscsi]$

[hchiramm@localhost csi-driver-iscsi]$ kubectl get csidriver
NAME               ATTACHREQUIRED   PODINFOONMOUNT   STORAGECAPACITY   TOKENREQUESTS   REQUIRESREPUBLISH   MODES        AGE
iscsi.csi.k8s.io   false            false            false             <unset>         false               Persistent   16m
[hchiramm@localhost csi-driver-iscsi]$

[hchiramm@localhost csi-driver-iscsi]$ kubectl get pv,pvc
NAME                              CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS    CLAIM                                                 STORAGECLASS   REASON    AGE
persistentvolume/pv-name   1Gi        RWO            Delete           Bound     test/iscsi-pvc                            3m22s

NAME                              STATUS    VOLUME           CAPACITY   ACCESS MODES   STORAGECLASS   AGE
persistentvolumeclaim/iscsi-pvc   Bound     pv-name   1Gi        RWO                           4s
[hchiramm@localhost csi-driver-iscsi]$

configuring WMCO :  The deployment has to be done using the Red Hat Shipped version of WMCO by following the process mentioned at https://github.com/openshift/windows-machine-config-operator/blob/master/docs/HACKING.md#build  . Pretty much got it working but the WMCO pod was getting into crashloopback due to a missing thing in the network configuration in my OCP cluster running on AWS with 4.8.0 build.

  Last State:    Terminated
      Reason:      Error
      Message:     system:serviceaccount:openshift-marketplace:wmco\" cannot get resource \"networks\" in API group \"config.openshift.io\" at the cluster scope","errorVerbose":"networks.config.openshift.io \"cluster\" is forbidden: User \"system:serviceaccount:openshift-marketplace:wmco\" cannot get resource \"networks\" in API group \"config.openshift.io\" at the cluster scope\nerror getting cluster network object\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.getNetworkType\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:250\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.networkConfigurationFactory\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:169\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.NewConfig\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:90\nmain.main\n\t/build/windows-machine-config-operator/main.go:89\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1371\nerror getting cluster network type\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.networkConfigurationFactory\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:171\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.NewConfig\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:90\nmain.main\n\t/build/windows-machine-config-operator/main.go:89\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1371\nerror getting cluster network\ngithub.com/openshift/windows-machine-config-operator/pkg/cluster.NewConfig\n\t/build/windows-machine-config-operator/pkg/cluster/config.go:92\nmain.main\n\t/build/windows-machine-config-operator/main.go:89\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1371","stacktrace":"main.main\n\t/build/windows-machine-config-operator/main.go:91\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225"}

      Exit Code:    1
      Started:      Wed, 26 May 2021 23:32:22 +0530
      Finished:     Wed, 26 May 2021 23:32:22 +0530
    Ready:          False
    Restart Count:  9
    Requests:
      cpu:        10m
      memory:     50Mi
    Liveness:     exec [grpc_health_probe -addr=:50051] delay=10s timeout=1s period=10s #success=1 #failure=3
    Readiness:    exec [grpc_health_probe -addr=:50051] delay=5s timeout=5s period=10s #success=1 #failure=3
    Environment:  <none>
    Mounts:
..
Above is with custom images or newly built images:
  Normal   Pulled          24m                    kubelet, ip-10-0-135-111.ec2.internal  Successfully pulled image "quay.io/humble/wmco:3.0.0" in 401.153338ms
  Normal   Pulled          24m                    kubelet, ip-10-0-135-111.ec2.internal  Successfully pulled image "quay.io/humble/wmco:3.0.0" in 436.096547ms
  Normal   Pulled          23m                    kubelet, ip-10-0-135-111.ec2.internal  Successfully pulled image "quay.io/humble/wmco:3.0.0" in 420.653162ms

Once this is corrected, I believe our Windows Machine configuration should be fine.
[Other findings]

It has confirmed that, exporting block device to the windows container is not possible at this moment due to an issue in OCI runtime mount specs.  So indeed its a limitation for windows containers.

csi-proxy layer faciilates the operation on behalf of windows csi containers to avoid any vulnerabilty or security which can arise if the drivers directly started to talk to the windows server.

[Quick update]
I could get the windows working finally!

[hchiramm@localhost windows-machine-config-operator]$ oc get pods
NAME                                                              READY   STATUS    RESTARTS   AGE
windows-machine-config-operator-6b4656684-d74fj                   1/1     Running   0          2m12s
windows-machine-config-operator-registry-server-5b5b864596xq7zg   1/1     Running   0          2m34s
[hchiramm@localhost windows-machine-config-operator]$

[Status update]
The iscsi experiments are going on heavily to get iscsi csi project in a better shape. Many hiccups due to the fact that, it was never tested in real kube cluster on both codewise or setup wise. Solving one by one and making progress. Will continue experimenting on this as this node part is very important in the workflow of this effort.

Finally able to get the mount part working on ISCSI side:
I0603 07:11:36.907283       7 mount_linux.go:405] Attempting to determine if disk "/dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1" is formatted using blkid with args: ([-p -s TYPE -s PTTYPE -o export /dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1])
I0603 07:11:36.925643       7 mount_linux.go:408] Output: "DEVNAME=/dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1\nTYPE=ext4\n", err: <nil>
I0603 07:11:36.925721       7 mount_linux.go:298] Checking for issues with fsck on disk: /dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1
I0603 07:11:37.028261       7 mount_linux.go:394] Attempting to mount disk /dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1 in ext4 format at /var/lib/kubelet/pods/e64ff6a0-fc0a-4f32-a947-04cd873974ce/volumes/kubernetes.io~csi/static-pv-name/mount
I0603 07:11:37.028292       7 mount_linux.go:146] Mounting cmd (mount) with arguments (-t ext4 -o rw,defaults /dev/disk/by-path/ip-<IP_ADDRESS>:3260-iscsi-iqn.2015-06.com.example.test:target1-lun-1 /var/lib/kubelet/pods/e64ff6a0-fc0a-4f32-a947-04cd873974ce/volumes/kubernetes.io~csi/static-pv-name/mount)
I0603 07:11:37.086370       7 utils.go:53] GRPC response: {}
I0603 07:11:41.058886       7 utils.go:47] GRPC call: /csi.v1.Identity/Probe
I0603 07:11:41.058968       7 utils.go:48] GRPC request: {}
I0603 07:11:41.059147       7 utils.go:53] GRPC response: {}
[root@dhcp53-171 csi-driver-iscsi]# kubectl get pods |grep task
task-pv-pod                                                       1/1     Running             0          39s
[root@dhcp53-171 csi-driver-iscsi]#

Unfortunately the unmountis failing which I find as the final blocker for the iscsi csi driver to be in a consumable state, refer https://github.com/kubernetes-csi/csi-driver-iscsi/pull/45#issuecomment-853807843.  Debugging it and the RCA is known at this time though.

Unfortunately the unmountis failing which I find as the final blocker for the iscsi csi driver to be in a consumable state, refer kubernetes-csi/csi-driver-iscsi#45 (comment). Debugging it and the RCA is known at this time though.

Further experiments were carried on this and various issues are noticed in the driver due to broken library which iscsi driver currently consumes for its operations. Most of the  issues are fixed and few are still left to fix. There is a  refactoring happening on the library and after that, it has to be retested.
In short, this is getting good pace https://github.com/kubernetes-csi/csi-lib-iscsi/pull/29#issuecomment-918826158 and tagetted for GA at 1.23 timeframe.

[Update]
Finally after many discussions and experiments, https://github.com/kubernetes-csi/csi-lib-iscsi/pull/29 is merged.  this should give us decent stable platform to continue on the iscsi driver. Will refresh and update the progress here :+1:

One great update here : Finally kubernetes-csi ISCSI driver get its first release https://github.com/kubernetes-csi/csi-driver-iscsi/releases/tag/v0.1.0 :tada:  .. one more step ahead on this effort !

There going to be another iscsi release with 1.24 kube with more fixes..etc. With that in place, we will be able to continue on this effort, meanwhile I am removing this from the 3.6 tracker.

v1.0.0 of iscsi csi-driver is yet to be available https://github.com/kubernetes-csi/csi-driver-iscsi/ . keeping this issue tracker for 3.8

@humblec any plan to work on it? if not feel free to unassing/close it.


Fix typo

Thanks for bringing this to my attention. Typo fixes are lovely to have. In future, make sure you point at develop though, main is reserved for releases... even though I haven't followed the rules myself. Trying to maintain good standards across all my work in future and make sure other people will as well. I will get this merged in for the next release.


How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

What's the type of your software?
zip
Please provide the following information
Download Page Url: https://www.x-ways.net/winhex/
Install Options: The application can either be used as a portable application (winhex.exe) or can be installed (setup.exe).
Publisher: X-Ways Software Technology AG
Publisher Page: https://www.x-ways.net
Package Page: https://www.x-ways.net/winhex/
Package Name: WinHex
Description: Universal hexadecimal editor, particularly helpful in the realm of computer forensics, data recovery, low-level data processing, and IT security.
Package Version: 20.9
Installer URL: https://www.x-ways.net/winhex.zip
License: Proprietary
License Page: https://www.x-ways.net/winhex/comparison.html
Copyright: X-Ways Software Technology AG

Oh no,
CLI in my machine strike again

@AsciiWolf - Sorry, we cannot achive this. Neither winhex.exe nor setup.exe are supported by winget.exe. 😞

Both wingetcreate.exe and YamlCreate.ps1 are unable to inspect setup.exe inside the ZIP archive.
winhex.exe packaged in winhex.zip is a "loose executable".

This might be a blocking issue and should be closed as "not planned", or there will be someone else successfully identified the installer technology of setup.exe.

This sounds like portable nested inside zip.

The setup.exe binary should be an installation program:

The supplied setup.exe program only copies the shipped files to the destination folder (plus all .whs files it finds), sets the desired language (English, German, French, Spanish, Italian, or Portuguese), and creates a program shortcut in the start menu. All other settings are initialized by winhex.exe/xwforensics.exe itself.


Can't specify features for individual crates in a virtual workspace

When you have a workspace containing multiple crates in subdirectories and those crates have their own feature flags, it's not possible to specify which feature flags to use when checking/navigating that project.
Setting rust-analyzer.cargo.features doesn't work because cargo errors with "error: --features is not allowed in the root of a virtual workspace", same as if I ran cargo check --features foo from the workspace root.
This becomes particularly evident if you are continuously recompiling and running tests in a background terminal (e.g. cd sub-crate && cargo watch -x "check --features foo"). Rust Analyzer ends up compiling the crate/workspace without the feature and cargo watch compiles with it. That means you'll constantly be rebuilding the entire crate... Which can be quite expensive when the feature flag enables several transitive dependencies (e.g. tokio and a HTTP server).

If you are running cargo watch anyway, you should set rust-analyzer.checkOnSave.enable: false. Cargo watch should still provide you with inline error markers, via the problem matcher.

Cargo watch should still provide you with inline error markers, via the problem matcher

Does the problem matcher need cargo watch to be running inside a VS Code terminal? I run cargo watch completely separately from VS Code, and when I disabled checkOnSave and restarted VS Code it didn't show me any inline error messages.

You need to run it as a vscode task, the following definition should work
    {
      "label": "Watch",
      "group": "build",
      "type": "shell",
      "command": "cargo watch --package rust-analyzer",
      "problemMatcher": "$rustc-watch",
      "isBackground": true
    },

This is a Cargo limitation that will be addressed by https://github.com/rust-lang/rfcs/pull/2957.

I belive this was fixed on the cargo side with the introduction of resolver=2


OpenAPI type specification for workflow fields

Coverage increased (+3.8%) to 47.421% when pulling 5e6315a195fccf31bf340d46b56ae9a86de7334f on maxhq:develop-maxhq into e6d78a09ef96853dca9f38930dbafcf655b96635 on openxpki:develop.


brightness 0 is not really 0

What's the best way to turn off all leds? send #000?

The problem is when we detect brightness 0 or even color all-zero, we just turn off power. This doesn't work when you have your own power. Need to fix.

The problem is when we detect brightness 0 or even color all-zero, we just turn off power. This doesn't work when you have your own power. Need to fix.

as a workaround, you can setall #010101 with brightness 1; it should clear

as a workaround, you can setall #010101 with brightness 1; it should clear

Is it bad to power off / on these little lights a lot?

Is it bad to power off / on these little lights a lot?

I don't think it's a problem; the LEDs are powered on/off at 400Hz, and I doubt the IC cares.

I don't think it's a problem; the LEDs are powered on/off at 400Hz, and I doubt the IC cares.

Set 010101 color would destroy the color pattern on the strip... will wait for fix.

Set 010101 color would destroy the color pattern on the strip... will wait for fix.

still an issue?

still an issue, on todo list but low

this is fixed in the leddisplay, ledstrip still has a problem


Outputting a GLB file

The readme suggests that it is possible to write a .glb file instead of a .gltf, .bin and folder
--out arg            The output path to the .gltf or .glb file.
                     Defaults to "<working-directory>/<FBX-filename-basename>.gltf"

However, as far as I can tell there is no option to do so. When I give an output path with a .glb extension the utility still outputs a .gltf file, just with a different extension.
Is it possible to export .glb files with this program?

Resolved in #74  .
Released as https://github.com/cocos/FBX-glTF-conv/releases/tag/release-v1.0.0-alpha.39 .
Feel free to reopen this issue if this release doesn't work.


docs: fix duplicate-text link

Hi!
The "Noise Protocol Framework" link included the text twice - once in the highlighted color, and again in the default color.
CLA signing: https://github.com/build-trust/ockam-contributors/pull/271

@Celeo Thank you for accepting our CLA. And thank you for your contribution! I am adding the HACKTOBERFEST-ACCEPTED label on this PR as well.


[Meta] list for metadata

Replace hashtable (glib) to list, setting/getting metadata of edge handle.
Signed-off-by: Jaeyun<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #45. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

Anyway, please hide internal data structure from other .C files. The internal data structure of a handle should not be exposed to its users.


validation dataset cropsize

Thank you very much for your efforts. I have observed that while running train_MambaSCD.py, the dataset size during the validation stage was not cropped. I would like to inquire about the data processing procedures employed during the validation phase.

Hi,
Thank you so much for your question. During the training phase, the image is randomly cropped to 256x256 (of course you can crop it ahead) as well as some other data augmentation methods are applied. During the testing phase, the network infer the results directly on the images with original size without cropping.
Best,


[#3264] Update logic to handle file browser's navigation state

This PR fixes the logic of navigation buttons in the resource landing page file browser. The error was caused by an outdated value predating work done recently which changed how we construct the file paths in the front end.

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[x] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a resource.
Add various folders to the resource (including nested ones).
Navigate around in the file browser and confirm that navigation works as expected.

@sblack-usu This PR solves an outstanding issue. The fix is quite small and could be quickly tested and review it. Could we push to include this in this release?

@Maurier - Is this due to changes with data/contents path requirements in the file browser?

@sblack-usu - Yes. This bug was introduced as product of that work.

So this isn't a regression in 1.21 and exists in previous releases?

This bug was introduced recently, but it is not a regression of this release. However, it was discussed in the coordination call today that we might want to include this in the release since it fixes an outstanding issue.

@Maurier did you mean for this to go to the RC or to develop?

After the discussion I created another PR with this changes targeting the RC: https://github.com/hydroshare/hydroshare/pull/3278
We can remove this one and proceed with that one.

This will be included in RC instead. Closing and moving conversation to https://github.com/hydroshare/hydroshare/pull/3278


Task list: Project 5 - Classification and Clustering

Clustering:

[ ] Plot script
[ ] Model script
[ ] Test
[ ] Documentation

Classification:

[ ] Plot script
[ ] Model script
[ ] Test
[ ] Documentation

Feel free to add anything. @pttzty @changsiyao @ye-zhi @lizhua @lichenzhi

Sure, I will take plot script and a part of the model script in clustering.

Maybe wrap something into functions, like when fitting model.

@MengfeiJiang @pttzty So we are not using the 2-hr bin features for clustering, is that right?
@ye-zhi @lizhua @lichenzhi New data loading function for 2-hr bin features is merged, please continue classification using this new data loading function. Classification performs better using the new data structure.

@changsiyao Mengfei and I are not doing 2-hour bin features because a large number of features will make the high-dimension curse problem in clustering. since we are using the euclidian distance

I'm still a little confused what we should output in our classification models. My current idea is to at least output our model, parameter and CV performance measures. But should we also output the predictions we make? If so, our functions need to take in train_x, train_y and test_x set separately. (Currently, we are randomly splitting data into training and test inside our function, which doesn't make much sense to output test predictions.) @ye-zhi @lichenzhi @lizhua @MengfeiJiang @pttzty


rfc: Propose Pallas Addresses

Summary
A Rust library encapsulating logic and data structures to deal with Cardano addresss of any type.
Motivation
Addresses in Cardano are complex. There're multiple formats and combination of options. At the same time, usage of these primitives is ubiquotus in almost every tool / project that requires to interact with the Cardano blockchain. Having a small but comprehensive Rust library to deal with Addresses can allow downstream tools (Oura, Scrolls, etc) to avoid re-inventing the wheel.

Resolving this PR without discussion from community since it's for an already implemented proposal.


Fix typo in JDBC docs

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/460e1d1b680152059bad676c0040d871a8f027ba/
Edited pages:

v20.2/build-a-java-app-with-cockroachdb.md
v21.1/build-a-java-app-with-cockroachdb.md


fix gather/scatter type verification

Computes and Fixes are allways doubles. This is just a small fixup - I think In the future we should use

LAMMPS_INT
LAMMPS_INT_2D
LAMMPS_DOUBLE
LAMMPS_DOUBLE_2D

for the type parameter instead of Int32 and Float64. This makes the API more consistent and we're able to return Vectors instead of 1xN Matrices while preserving type stability.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.91%. Comparing base (913e7c3) to head (180f973).
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #57       +/-   ##
===========================================
+ Coverage   54.56%   67.91%   +13.34%
===========================================
  Files           2        2
  Lines         449      427       -22
===========================================
+ Hits          245      290       +45
+ Misses        204      137       -67

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


ftl init should create hermit at the root of the repo (or look up and update hermit if it exists)

Currenly ftl init will create a new bin folder in the folder it was issued.

Should we just get rid of this altogether perhaps?


Speed up the virtualization detection

Virtualbox and Hyper-v detection takes roughly 4 seconds each on my machine. If we plan to speed up java detection, it might be necessary to improve this as well

For hyper-v we could simply check if the services are running instead of querying windows features (takes ~200 ms instead of ~3-4 seconds).
Plus direct wmi queries from cmd seem to perform a lot faster than powershell calls.

Done in #1262


InspectorShowIf/InspectorHideIf attributes for multiple properties

It would be great if it would be possible to use multiple proprties to be able to show/hide other properties if every single one of the properties is true (so this would be a logical AND). Or even go further and allow an OR with something like that:
public bool Cond1;
public bool Cond2;

[InspectorShowIf(LogicalOperators.OR, "Cond1", "Cond2")]
public string ShowIfAnyConditionTrue;

[InspectorShowIf(LogicalOperators.AND, "Cond1", "Cond2")]
public string ShowIfEveryConditionTrue;

Done; this is available in the latest build from access. LogicalOperators is renamed to fiLogicalOperators.


FeedRange - Scaling out support

With the new Change Feed pull model approaching and the use of FeedRange, one pending item is support for scaling out running iterators.
Currently, the flow is:

User gets FeedRanges
User starts 1 iterator per FeedRange on each machine/instance
Some time later, some of the machines are running hot on CPU because possibly the ranges they are handling are hotter than the other machines (or those ranges had splits)

How do we allow users to scale out if they want to?

Open to discussion

One idea would be to have a method on the FeedRange itself, possibly List<FeedRange> FeedRange.Scale(int? ranges) that will attempt to split the range into the required # of ranges. If the parameter is not passed, we can either split using physical partition affinity or other limits.
Another idea would be to use it on the Continuation tokens. Since this is an scenario for when an iterator has been already running, we have a Continuation token. We could then take them as input and return new continuations after the split: List<string> FeedRange.Scale(string currentContinuationToken, int? expectedRanges).
Limits
Scale out operations can always face a limit. When EPK filtering is in place in the backend, a FeedRange cannot be split further than a single Partition Key value. Without EPK filtering in place, the limit is on the physical partition.
In any case, the API can always return a negative or "cannot scale out further" semantics.

I believe we talked about this and voted to not go for a Scale method.
Instead you can have:
bool FeedRangeFactory.TrySplit(FeedRange source, out (FeedRange target1 FeedRange target2))
bool FeedRangeFactory.TryMerge((FeedRange source1, FeedRange source2), out FeedRange target)

The Try Pattern will naturally imply to the user that the operation may fail. You can add monadic overloads to return a reason for the failure:
TryCatch<(FeedRange, FeedRange)> FeedRangeFactory.Monadic.Split(FeedRange source)
TryCatch<FeedRange> FeedRangeFactory.Monadic.Merge((FeedRange source1, FeedRange source2))

I like the pattern Brandon proposed of just exposing Split/Merge - it provides the necessary building blocks for customers to achieve different scenarios. Any abstraction - like the Scale method - would just result in more confusion when it isn't possible - and the higher the level of abstraction the harder to explain the reasons why you can scale up to 8 but not 12 now - and a couple of hours later it works etc. (because more physical partitions exist) - Adding higher level abstractions might be a next step when Split/Merge/EPK support etc. have settled some more

Is there any scenario where user actually need to scale beyond a single range for a physical partition? I can't think of scenario where the user would want to scale to a range to the point of a single partition key value. The other question is can the service even handle that many connections or will that hit some restriction that will cause more CRIs.
My only suggestion with Brandon's proposal would be to make it a method rather than a static factory. It's easier to discover and unit test.
bool myCurrentFeedRange.TrySplit(out FeedRange target1, out FeedRange target2)
bool myCurrentFeedRange.TryMerge(FeedRange source1, FeedRange source2, out FeedRange target)

One other possibility is to have a call back method when a split\merge happens. That way the user doesn't have to constantly check if one or the other operation is possible.
myCurrentFeedRange.ScalingCallback(Action <FeedRange, ScaleType> scalingCallback);

I don't think a callback would address the most commons scenarios. Those - IMO - are when Compute is also dynamically scaling - for example in the Spark connector the number of workers you have in Spark could change - or when you realize that the change feed processing for a certain feedRange lacks behind others you might want to have more workers for that feedRange -s o just Split that particular FeedRange. Callback would only be triggered when the number of physcial partition changes - but the main job often is do establish the right balance between compute resources available and number of FeedRanges available in the backend etc.

For Merge() api, is the expectation that customers will change it on all possible combinations to find out which ranges can be merged?  Excluding the intra-partition feed range case, how can customer deterministically split/merge feed ranges to match physical layouts

@sivethe depending on the implementation we can allow for two ranges that are not adjacent to be merged together. We would just need to add a FeedRange implementation that can compose other FeedRanges. Early iterations won't support this, but that is the nice part of the Try based API .. as we get more advanced in the scale out story our users should see more of their try split/merge calls succeed.

Any news here? I'm interested in a simple scenario, where we want to end with one FeedIterator per physical partition. Currently we're missing some functionality to split the FeedIterator in case of partition split. Also some notification that the split happened would be nice, but not necessary.
Is there currently any workaround for splitting it?

This has been solved with ChangeFeedCrossFeedRangeState:
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos/src/ChangeFeed/ChangeFeedCrossFeedRangeState.cs
And the PR has documentation (refer to "Feed Range Scale Out")
https://github.com/Azure/azure-cosmos-dotnet-v3/pull/1978

How do you use the ChangeFeedCrossFeedRangeState.TrySplit, when it's all internal?
Am I missing something?

The code uses the following macro pattern:
#if INTERNAL
#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
#pragma warning disable SA1600 // Elements should be documented
#pragma warning disable SA1601 // Partial elements should be documented
    public
#else
    internal
#endif

You can build your project with the following:
<DefineConstants>$(DefineConstants);INTERNAL</DefineConstants>

And you should access to all these types. Note that the type isn't actually GA'd, so you may encounter build breaks in the future, but you should be able to use it as a decent stop gap solution.

This needs to be part of the public API before it can be resolved. Asking user to do a custom build to access an internal API is not a reasonable ask. Also internal APIs are can have breaking changes in the future making it not a sustainable path for customers.

just checking if there is anything new here. Is there really no plan to support scale-out scenarios for long running change feed queries?
This was possible to do in v2 but it's not possible in v3, because partition splits are hidden inside FeedIterator. Which is nice in most cases, but not so good for long running change feed queries. Because with data growth, that ends up with few FeedIterators having quite many physical partitions inside and no way to split that.

@zbynek001 as a workaround is there any reason you can't stop all the changefeed iterators. Then get a list of the new FeedRanges, then from the list create new changefeed iterators using the start from date?

@j82w short answer: no, that's not really possible.
The application is running distributed across several servers, each one is processing a set of partitions, and each partition is processed independently to some extend.
The result is, the processing state of each partition is at different position in the changefeed. So we'd have to wait for everything to be at the same position. And even if we could do that, how would we translate this to some datetime to start from?
In v2, we were able to detect the partition splits via the exception with special StatusCode/SubStatusCode, then enumerate the PartitionKeyRanges, from it detect what are the new partition it was split into, and start new changefeed processing.
In v3, we cannot detect the split, and even if we could, there is currently no way to e.g. split the FeedRange/FeedIterator

We will have a update next week with a plan to support this scenario.

@zbynek001 @j82w Should we open a separate thread to discuss this particular scenario? This is mostly a Design Issue to track Design iterations.
@zbynek001 When you create the new Issue, could you explain the nature of your workload and how scaling out to more machines would benefit it? Is the workload CPU bound and hence it benefits from adding more and more machines? Or is it I/O bound? Would the Change Feed Processor help?

@ealsur @j82w Here is the new ticket: #2759


Add testing for ColoredFormatter

Checks ANSI escape codes for correct coloring based on the DEFAULT_COLORS variable using the LogCapture class from testfixtures and the ColoredFormatter subclass.  100% coverage on log.py.
#44
#48

I ran the file through a PEP8 checker, found a few other things. Should tox be catching these?

We don't run any PEP8 checks, only flake8, but we could easily add PEP8 checks if we want to.

@leportella just opened a PR to add flake8 and isort on tests.


Mark reports "no matches found"

I have a repo where I'm trying to push markdown docs to Confluence. Right now, for test purposes, I'm doing this manually, however the hope is to be able to automate this, so that it's automatic when we merge changes to the main branch.
The problem I'm facing is that when I try running Mark against one of the directories in the repo, I'm met with no matches found error message. The files are where they're supposed to be, and I'm specifying the correct path to the files. I double-checked the config, and that all looks good too.
Any ideas what might be going wrong? I'm running Mark 9.1.4 on a M1 Mac, which is running macOS Ventura. Mark has been installed using Brew.

How does the command you're running look like?

I have since got it working. However, it appears that Mark is not picking up/using the options in the config. I have my username and token defined in the config file (at ~/.config/mark) along with the options for title-from-h1 and h1-drop. But, if I do not specify all of those options on the command line, I get an error and Mark does not run.
Also, even with --title-from-h1 and --drop-h1 set, it's titling the Confluence page correctly, however it isn't dropping the H1, so the page appears to have 2 titles.
Any ideas on how I can get this working better? Anything special I should add to the path so that Mark can find and use the config file? Any specific environment settings?
I also noticed another issue, but I'll create a separate issue for that, to avoid confusions.


Husky hooks bugged out when attempting a commit

Had to delete all hooks for the commit to succeed.
The commit in question:
https://github.com/wix/detox/pull/604/commits/6508b9867c0697b3c4d21688d8f583cbb1ce8c17
I see husky was added in Generation to run lint before committing. I am not sure whether my problem is due to husky or our own precommit hook, but either way, I am not sure this is the right place to run lint.
Regarding husky, since git hooks are considered personal and installed by developer, I do not like type of "devious" injection of code—I felt "violated" 🙂. We can run the lint on CI.

It is possible to have a script that installs some hooks that can then, by developer's choice, install these hooks. Similar to how bootstrap.sh works, maybe something for the hooks. But if the hooks can cause issues, I think they shouldn't run by default.

Thanks

Ping
@DanielMSchmidt What will happen if I remove it from package.json inside generation?

You will gain the possibility to check in unformatted code I guess

OK, thanks


Versions of actions referenced are not available in my CodeCatalyst workflow environment

Describe the bug
As part of our projen-pipelines implementation I am making use of the Workflows SDK
When using

I am getting the following error message when executing the generated workflow:

I think the error is in here and I dont really know if thats intended or not:
https://github.com/aws/codecatalyst-blueprints/blob/6e4e870fc9b6c818158d447a2e0a6552c940eff6/packages/components/workflows/src/actions/action.ts#L25

Tried to workaround it by setting this.context.environmentId = 'prod';
But thats not possible as its readonly 👎

A potential workaround for me could be to set process.env.CONTEXT_ENVIRONMENTID=prod but that doesnt feel to be correct?

A potential workaround for me could be to set process.env.CONTEXT_ENVIRONMENTID=prod but that doesnt feel to be correct?

That worked as a workaround...

process.env.CONTEXT_ENVIRONMENTID=prod

Is currently the way blueprints deal with this problem. But yes, I agree this doesnt seem quite right. I'll take it back to the team and give it a review

Maybe it might make sense to switch it - the "default" is the "prod" version and if you want to use the other possible type you need to explicitely set it?


Nftlist component with mock json

https://linear.app/pillarproject/issue/PRO-2128/etherspot-ui-nft-list
Description

Create NFT list component
How Has This Been Tested?

tested on local

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

@kio @lalitkumarrock As per discussion over standup call, I am creating list using mockdata of nftlist. Once project key issue will solve I will uncomment function  and remove mock logic.


I would like to add my membership request

I would like to add a membership request. I have written an element which displays a messagebox with the possibility to choose like a dialog box. The element is called MessageBox.

Invitation sent. Now you need to transfer your repo to one of the core debs of kivy and it will be transferred to garden.


fixes for grid and min rate

min rate was not restricted - went crazy with delay of 0.
grid wasn't properly clipped for clip:1

issues I reported are exactly the same

there is this error when adjusting dimensions and the plot does not adjust to the dimensions anymore and get clipped

the rotation seems better now, rotatex rotatres horizontally, but counter clock wise if look from above and that doesn't seem right at all to me, cause I am dragging to the right I want to see it rotate in the same direction. maybe y is reversed too...
rotate message still has x/y reversed...
but I also get "scope3d~: no method for 'float'" when adjusting it
still seems CPU heavy as GUI freezes

just fixed 'rotate'

can you push the fix?
i'm trying to find out where that error message comes from. when i test dim here with new scope, it's all good. totally weird ...

my fix fucked with rotatex, so I don't know what is going on and reverted back, you fix this :)

FFUUUUUCCCKKK ... i found the problem and it's quite funny: scope currently defaults to a receiver named empty and obviously, your knobs are sending to empty by default. i fixed it for scope3d~ now and there's no receiver set at all. but i think, you should also fix your knobs to not send any messages by default. PR incoming ...

ok, about the rotation: which axis should i invert now? :)

i'm trying to find out where that error message comes from. when i test dim here with new scope, it's all good. totally weird ...

really weird you can't reproduce this... since you included 'flames' I guess, this started happening
performance is not doable here with 50ms refresh rate, still bad, it got worse with clipping I guess
It seems things were really ok and better before flames and clip :)

ok, about the rotation: which axis should i invert now? :)

rotate gets y and x, should be x and y

It seems things were really ok and better before flames and clip :)

yeah ... it included many changes. but flames shouldn't really be the problem since the performance is dependant on perform and paint routines. i changed quite some stuff there. but with the last optimizations, it's faster for me. which patch are you testing with? the help itself?

can we make an issue for the open scope3d~ things? or use the other one? so that we don't discuss in the PRs?

sure, let's go back to the issue, or even reboot it


Supergroups error

When the bot is added to the supergroups it crashes, The error is: largestInt out of range.

Done, commit: 1916b3707e
Thanks


Dead code？

arch/x86/crc_folding.c 451
CRC_SAVE(s) is dead code??

This might be intentional due to need for closing the do block started in CRC_LOAD(). There is no macro to discard the result and close the do block.


[Feature Request] Multiple HTTP Listeners

Feature Request
Opening a feature request kicks off a discussion.
Requests may be closed if we're not actively planning to work on them.
Proposal: [Description of the feature]
add multiple  HTTP listeners in the same way we can configure multiple graphite/udp/collectd listeners.
Desired behavior: [What you would like to happen]
I need add more than one HTTP input with different HTTP configuration.
Use case: [Why is this important (helps with prioritizing requests)]
For security reasons we need add a port with HTTPS  for peripherals devices on a already working in production server configured with HTTP for already configured agents on internal devices.
In this scenario with current behaviour I will need to update  to use HTTPS all internal devices ( that is not really needed) previously  to deploy agents to the peripheral devices.
Thank you very much.

We talked about doing this in the past, but we decided that InfluxDB wasn't supposed to be an HTTP server and we didn't feel adding multiple HTTP listeners was useful. Can you install Nginx and set that up as a proxy? Nginx is designed to do this already and it's much better to use it than for us to add a setting for another HTTP listener.

is this maybe a better feature add for telegraf ? Per the training i just attending telegraf design is to offload all "translation and additive features" from influxdb. If the intent is to remove the http server from influxdb itself, this seems like the logical next place.

I don't think this is the correct place for telegraf, but @sparrc may be able to comment better than me on that. I think Nginx is the correct place to go if you need multiple HTTP listeners for dealing with things like SSL. Nginx will be much better at being an HTTP server than we will ever be anyway.
If there's a situation where Nginx or Apache doesn't work, please comment. I'm going to close this for now since modifying the configuration file to allow multiple HTTP listeners would also introduce complications in the configuration file.


CLI: studio should not hang and return a non-successful exit code (1) if an error occurs

Bug description
From https://github.com/prisma/prisma/issues/12349
Studio, when an error occurs, does not exit.
How to reproduce
# Create an empty `schema.prisma`
❯ npx prisma studio
Environment variables loaded from prisma/.env
Prisma schema loaded from prisma/schema.prisma
Prisma Studio is up on http://localhost:5555
Unable to get DMMF from Prisma Client:  Error:
You don't have any datasource defined in your schema.prisma.
You can define a datasource like this:

datasource db {
  provider = "postgresql"
  url      = env("DB_URL")
}

More information in our documentation:
https://pris.ly/d/prisma-schema

    at getGenerators (/Users/j42/Repros/exit-codes/node_modules/prisma/build/index.js:90249:11)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async $fc77884679bbcc28$export$569332143708bf93 (/Users/j42/Repros/exit-codes/node_modules/prisma/build/index.js:96078:28)
    at async $527b23cbb13e8bc6$var$PCW.getDMMF (/Users/j42/Repros/exit-codes/node_modules/prisma/build/index.js:96138:39)
    at async $b22dca7664055e05$export$cf1ce5b2c3d7dc8c.respond (/Users/j42/Repros/exit-codes/node_modules/prisma/build/index.js:96343:37)
    at async /Users/j42/Repros/exit-codes/node_modules/prisma/build/index.js:96510:26

And here it hangs forever, until the process is killed.
Every reload of http://localhost:5555 prints the same error over and over.
Expected behavior
The CLI should exit with an exit code 1 for these unrecoverable errors. (reloading the page will not fix it, the schema would need to be fixed and the command restarted)
Prisma information

Environment & setup

OS: macOS

Prisma Version
4.3.0-dev.27

For this error Unable to get DMMF from Prisma Client: it seems the origin is here:

it logs the error
and throws
https://github.com/prisma/studio-code/blob/v0.472.0/packages/pcw/src/pcw.ts#L243-L247

this.pcw.getDMMF(); called in a try catch here, which seems like it only returns the error to the frontend
Maybe we could process.exit(1) here in the case of an unrecoverable error (= getDmmf error)
https://github.com/prisma/studio-code/blob/v0.472.0/packages/server/src/channels/prisma.ts#L52-L92
On the CLI side, await studio.start() is called and there is no error handling logic that could be done.
https://github.com/prisma/prisma/blob/4.2.1/packages/cli/src/Studio.ts


A Error about DLL encounterd importing python package (in tmux environment)

I encounter a problem about DLL(libcudart.so.11.0). When I run 'ipython'/'python' then import 'inplace_abn', it can import package successfully.

$ ipython
Python 3.7.0 (default, Jun 28 2018, 13:15:42)
Type 'copyright', 'credits' or 'license' for more information
IPython 6.5.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import inplace_abn

In [2]:

But when I do the same operation in a tmux environment. It fails... I also try this in a new tmux environment. it fails too.
`python
Python 3.7.0 (default, Jun 28 2018, 13:15:42)
[GCC 7.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.

import inplace_abn
Traceback (most recent call last):
File "", line 1, in
File "/home1/pbliu/anaconda3/lib/python3.7/site-packages/inplace_abn/init.py", line 1, in
from .abn import ABN, InPlaceABN, InPlaceABNSync
File "/home1/pbliu/anaconda3/lib/python3.7/site-packages/inplace_abn/abn.py", line 8, in
from .functions import inplace_abn, inplace_abn_sync
File "/home1/pbliu/anaconda3/lib/python3.7/site-packages/inplace_abn/functions.py", line 8, in
from . import _backend
ImportError: libcudart.so.10.0: cannot open shared object file: No such file or directory
`

What is the principle of this phenomenon...? I am very confused. Hope someone can help me.
Thanks~

I find some components in this package  is compiled by cuda. I add the path of cuda in ~/.bashrc explicitly then source it can solve this problem( But I did not understand what changes in Tmux environment, why explicit path is needed).
export PATH=/usr/local/cuda/bin:$PATH export LD_LIBRARY_PATH=/usr/local/cuda/lib64:$LD_LIBRARY_PATH export LIBRARY_PATH=/usr/local/cuda/lib64:$LIBRARY_PATH export CUDA_HOME=/usr/local/cuda:$CUDA_HOME


make TargetPixelFile.meta read-only

avoid potential inconsistency if underlying hdu header is modified.

See the discussion in:
https://github.com/KeplerGO/lightkurve/commit/b8f0f7993a28b6b46b28cc96973e2bc111ed6e3a

Sorry I haven't found the time to review this PR yet.  Thank you and stay tuned!

superceded by PR #864


[ConnectionFactory] Return future instead of taking callback

Makes error handling easier.  Fits with the result of the planned API changes

Will get back to this after I finish other API changes.


Cite this repo on the Apache docs page, please!

I finally found this page (thanks to a reader) after asking for help at Let’s Encrypt.
Great module!

On Mon, Jun 13, 2022 at 02:49 Stefan Eissing @.***>
wrote:

Thank you! Glad you enjoy it.

I remember when its status was first announced, and I was estatic when it
was first released and put it to use at once. And I haven't looked back.
Starting was fairly easy, but recently I ran into trouble when I started
moving vhosts to different physical servers--that's when the tutorial would
have really been helpful.

Where would you expect to find the information on the apache httpd site
and what would it look like?

Hi, Stefan, thanks for replying so quickly. I would like to see it first on
the top of the docs section, say in the How To / Tutorials. Also a link at
the top of the mod_md page in the introduction would be great.
Blessings,
-Tom

Thank you, Stefan!


Create a Logo for AstroChart

We don't have a logo to use for AstroChart, so we need to create one.
It will be used in documentations, favicon, and elsewhere we need :)
If you want to contribute, please send your suggestion here.

Hey @WitchieXCabot , this is a very nice image! But I don't see it being used as a logo, I was thinking in something more simpler. the image that you make would be really good to use in places that can use large images, but in places like favicon it would not fit. Makes sense? (don't know if I was able to express myself good)

How you like this (the idea in general, not as final logo)?