SSL Certificate validation errors on CentOS 6.

Everything was working great on my Mac until I tried to deploy to Linux:
irb(main):006:0> nb = NationBuilder::Client.new slug, token; nil
=> nil
irb(main):007:0> nb.call :people, :count
OpenSSL::SSL::SSLError: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed
	from /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/ssl_socket.rb:46:in `connect'
	from /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/ssl_socket.rb:46:in `ssl_connect'
	from /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/ssl_socket.rb:24:in `create_socket'
	from /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/session.rb:746:in `block in connect'
	from /home/campaign/.rubies/2.3.1/lib/ruby/2.3.0/timeout.rb:91:in `block in timeout'

I suspect the problem is actually from HTTPClient:
$ ruby -r httpclient -e "puts  HTTPClient.new.get_content('https://ertw.nationbuilder.com')"
/home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/ssl_socket.rb:46:in `connect': SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (OpenSSL::SSL::SSLError)
	from /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/ssl_socket.rb:46:in `ssl_connect'

My CA bundle is up to date, but it seems like HTTPClient doesn't use the system CAs. I can overwrite it:
 cp /etc/pki/tls/cert.pem /home/campaign/.gem/ruby/2.3.1/gems/httpclient-<IP_ADDRESS>/lib/httpclient/cacert.pem

And then it works:
irb(main):005:0> nb.call :people, :count
=> {"people_count"=>65, "status_code"=>200}

But it would be nice not to have to do that. Do you know of any workarounds?
By coincidence (and really, it is!) #31 would let us access the httpclient object and manually configure the SSL browser path:
client.ssl_config.add_trust_ca "/etc/pki/tls/cert.pem"

This works now that I'm using the code from #31:
def nb
    http_client = HTTPClient.new
    http_client.ssl_config.add_trust_ca CA_BUNDLE_LOCATION if CA_BUNDLE_LOCATION
    NationBuilder::Client.new(nation_builder_slug, nation_builder_auth_token, retries: 8, http_client: http_client)
  end


App: make HTML preview work on opensea

https://opensea.io/assets/0xabefbc9fd2f806065b4f3c237d4b59d9a97bcac7/307
I don't know how this is implemented yet; probably animation_url or similar.

Some reverse engineering --
Looking at the tokenMetadataURI https://etherscan.io/address/0xabefbc9fd2f806065b4f3c237d4b59d9a97bcac7#readContract
points to: https://cloudflare-ipfs.com/ipfs/QmS23rGm5yLW9c4gAWKtn9na4n3k2YWjT2LzDKiUWdmrZs
{"description":":musical_note::sun_with_face::musical_note: internet pirate radio.","mimeType":"text/html","name":"zora.fm v1","version":"zora-20210101"}
and the tokenURI points to: https://cloudflare-ipfs.com/ipfs/QmUTMQLgvE2o2ZooR1gGsYjznE4rsYTKRLSwDo5wwHhjBW
So it looks like... they use tokenMetadataURI for the metadata to show on OpenSea, and the tokenURI points to the html file that is iframed on OpenSea

Perfect, thanks for the investigation.
Does this mean we would need to be on IPFS?

There isn't any documentation on tokenMetadataURI -- searched the OpenSea Discord, twitter, docs, google
I don't think IPFS is what matters, it's that:

tokenMetadataURI has the metadata JSON
tokenURI points to the html file that will be iframed

in this contact (https://etherscan.io/address/0xabefbc9fd2f806065b4f3c237d4b59d9a97bcac7#writeContract) they have both a updateTokenMetadataURI and updateTokenURI -- I can deploy the same contract and see if it is reproducable
a few weeks ago Dan from OS:

It's just a hardcoded iframe for now.  But I have a feeling that the community cares deeply about this and will call for it as a feature.

Initially, we'll probably just dynamically add an iframe.  Eventually, we might end up supporting scripts as a metadata type

https://twitter.com/dan_OpenSea/status/1352763071660494852

Related: https://github.com/webaverse/app/issues/889

https://twitter.com/pxlqart/status/1382679722056953862


Why AmfLongString bodySize is same as AmfString?

https://github.com/pedroSG94/RootEncoder/blob/8f034fce93b5a176448ca0a6c2e83f44d1ec9701/rtmp/src/main/java/com/pedro/rtmp/amf/v0/AmfLongString.kt#L31
The length of long string should be + 4 ?
Let me know if i wrong, thank you.

Yes, you are right.
Fixed here:
https://github.com/pedroSG94/RootEncoder/commit/2cdd1c1948b34dec4088cc091d70667629094f28

Thanks


Set up code guidelines for the project

User story

As a technical team
I want to set up all code guidelines
So that we have a clean codebase

Acceptance criteria

code guidelines for kotlin are set up and communicated to the team

Currently discussing this on MS-Teams.


Add Bill of Materials

This commit adds a BOM with the dependencies that this project should
manage. To make sure that the BOM is valid, the project has been
restructured to use the BOM internally.
As a result, the root POM is only used to invoke the reactor and does
not need to be published anymore. A new parent, inheriting from the bom
provides the required build configuration and external dependency
management.

Great question. They won't and we use the flatten-maven-plugin anyway so all the unnecessary stuff is reduced. If you build that branch, here is what the published bom looks like:
<?xml version="1.0" encoding="UTF-8"?>
<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.wavefront</groupId>
  <artifactId>wavefront-spring-boot-bom</artifactId>
  <version>2.0.0-RC1-SNAPSHOT</version>
  <packaging>pom</packaging>
  <name>Wavefront Spring Boot Bill of Materials</name>
  <description>Wavefront by VMware integration for Spring Boot</description>
  <url>https://www.wavefront.com/wavefront-spring-boot-bom</url>
  <organization>
    <name>VMware, Inc.</name>
    <url>https://www.vmware.com</url>
  </organization>
  <licenses>
    <license>
      <name>Apache License, Version 2.0</name>
      <url>https://www.apache.org/licenses/LICENSE-2.0</url>
    </license>
  </licenses>
  <developers>
    <developer>
      <name>Clement Pang</name>
<EMAIL_ADDRESS>      <organization>Wavefront</organization>
      <organizationUrl>https://www.wavefront.com</organizationUrl>
    </developer>
  </developers>
  <scm>
    <connection>scm:git:git@github.com:wavefrontHQ/wavefront-spring-boot.git/wavefront-spring-boot-bom</connection>
    <developerConnection>scm:git:git@github.com:wavefrontHQ/wavefront-spring-boot.git/wavefront-spring-boot-bom</developerConnection>
    <url>git@github.com:wavefrontHQ/wavefront-spring-boot.git/wavefront-spring-boot-bom</url>
  </scm>
  <dependencyManagement>
    <dependencies>
      <dependency>
        <groupId>com.wavefront</groupId>
        <artifactId>wavefront-spring-boot</artifactId>
        <version>${project.version}</version>
      </dependency>
      <dependency>
        <groupId>com.wavefront</groupId>
        <artifactId>wavefront-spring-boot-starter</artifactId>
        <version>${project.version}</version>
      </dependency>
      <dependency>
        <groupId>com.wavefront</groupId>
        <artifactId>wavefront-opentracing-sdk-java</artifactId>
        <version>1.16</version>
      </dependency>
      <dependency>
        <groupId>com.wavefront</groupId>
        <artifactId>wavefront-sdk-java</artifactId>
        <version>2.3</version>
      </dependency>
    </dependencies>
  </dependencyManagement>
</project>

LGTM then


Unit-details-editor Componet Review

Ontrack Component review
Team Member Name
Xin Huang | 218161625
Component Name
UnitDetailsEditor Component
Files in this Component

unit-details-editor.coffee
unit-details-editor.tpl.html

Component purpose
The unit-details-editor component is responsible for editing the basic details of a unit. This includes attributes such as the unit name, code, start and end dates, associated teaching periods, and Docker images for automated task assessment. The component provides an interface for instructors to configure these details conveniently.
Screenshots:
Original (PDF):
Component Functions
The unit-details-editor component allows users to:

Update unit details through a form interface.
Select teaching periods and task definitions dynamically.
Manage unit start and end dates using date pickers.
Fetch and display Docker images available for automated task assessments.
This is a functional component with moderate interactivity but no subcomponents.

Interaction with other components

Interaction with newUnitService:
Output: saveUnitEvent event, triggered when the unit is saved.

Interaction with newTeachingPeriodService:
Output: teachingPeriodSelected event, triggered when the user selects a teaching period.

Interaction with TaskSubmission:
Output: dockerImages data, providing a list of available Docker images.

Interaction with DoubtfireConstants:
Output: Configuration constants IsOverseerEnabled and ExternalName, used for controlling functionality and displaying the external name.

Migration Steps:
I will be following
doubtfire-web's Migration Guide

Analyze Existing Component:

Review the current implementation in AngularJS.
Identify key functionalities and dependencies.

Setup Angular Environment:

Ensure the Angular project is properly set up.
Install necessary dependencies (e.g., Angular Material).

Create Angular Component:

Generate a new Angular component.
Transfer the HTML template, styles, and logic from AngularJS to Angular.

Integrate Services:

Migrate and integrate the newUnitService,newTeachingPeriodService,TaskSubmission,DoubtfireConstantsinto the new Angular component.

Downgrade Component:

Use UpgradeModule to downgrade the Angular component for use in the AngularJS app.

Testing:

Test the new component to ensure it replicates the behavior of the old one.
Fix any issues that arise during testing.

Component Review Checklist

[ ] Ability to collect details from the user

The modal allows users to enter and select grading details.

[ ] Succeeds when data is valid

The modal successfully closes and returns valid data.

[ ] Handles errors

The component gracefully handles errors such as invalid ratings or missing grades.

[ ] Created unit is shown on success

The updated task details are visible in the UI after successful grading.

Fix the before image link it is currently a dead link. The plan looks extensive and well thought out apart from that


Fix concierge not found error!

Details
Fixed Issues

$ https://github.com/Expensify/App/issues/26314
PROPOSAL: https://github.com/Expensify/App/issues/26314#issuecomment-1699819617
Tests

Visit ND and access concierge chat while logged out using deeplink like staging.new.expensify.com/concierge
You will be redirected to the login screen.Enter an email and sign in.
You should be redirected to concierge chat after sign in.

Note: Perform the above steps on an existing as well as a new account.

[ ] Verify that no errors appear in the JS console

Offline tests

Same as above but I do not think we can test this in offline mode since we need to sign in using an OTP.
QA Steps
Same as above

[ ] 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[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 code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[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] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Existing Account
https://github.com/Expensify/App/assets/36226639/fbe866e5-3cb9-412c-808b-5828e2c36fd5
New Account
https://github.com/Expensify/App/assets/36226639/4a24fac1-0a41-4437-889d-ac52d99940f2

Mobile Web - Chrome
https://github.com/Expensify/App/assets/36226639/1e405baa-05c5-4053-a93c-32eb4e159036

Mobile Web - Safari
https://github.com/Expensify/App/assets/36226639/f4ca7659-b445-49a8-b36a-b9e511339b7e

Desktop
https://github.com/Expensify/App/assets/36226639/b11c789c-83b5-47d3-b7ff-f71a9adfdd22

iOS
https://github.com/Expensify/App/assets/36226639/8cb07323-7270-4731-81ea-4d0108016d86

Android
My android build is crashing and taking forever so I am unable to test on it but should be fine as it is working fine on other native apps.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/deetergp in version: 1.3.66-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.66-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Test build bot

@mpsuzuki I uploaded a test PR from my fork and it seems that the build system gets triggered. I am not sure why this doesn't work for your PRs. Is your repo private maybe? You may create new branches here on the adobe repo for now until we figured out why it is not working for you.


TUI residual artefacts on the last page when listing workspaces

Describe the bug
TUI residual artefacts on the last page bottom when listing workspaces.
To Reproduce
Steps to reproduce the behavior:

run any listing command with TUI like daytona logs
go to the last page (have less workspaces then it can be listed on full screen)
observe phantom ids without name of workspace
observe how they have ids from previous page

Screenshots
Second page:

Those ids are from last two workspaces on the first page.
Desktop (please complete the following information):

OS: [Macos]
Daytona Version: [e.g. v0.47]

This Issue has been automatically marked as "stale" because it has not had recent activity (for 14 days). It will be closed if no further activity occurs. Thanks for the feedback.


Add Zasco's channel packages to dependency selector

I like this. What about adding a checkbox or something that would toggle whether or not to include the extra channel(s) ... a bit worried about accidentally selecting a dependency from the wrong channel.

Could do that. However, I ensure there is no duplication between my channel and the default one.
This solution is just in the meantime I add the feature to manually add a channel as it is currently hard coded. It was for my personnal use, but I wanted to share it with the community too...

I like this. What about adding a checkbox or something that would toggle whether or not to include the extra channel(s) ... a bit worried about accidentally selecting a dependency from the wrong channel.

Could do that. However, I ensure there is no duplication between my channel and the default one. This solution is just in the meantime I add the feature to manually add a channel as it is currently hard coded. It was for my personnal use, but I wanted to share it with the community too...

Gotcha. I'm fine doing whatever you think is best. If you're going to use it a lot for personal reasons I don't see why we couldn't do it ... one of the advantages you get as an early adopter lol :)


Address #17 by filtering Macro and Label images from SVS files

This should fix #17 by filtering out macro & label images from SVS slides.  Currently looking for publicly available test data to validate this change.

@mdushkoff instead of filtering out the macro and label, what do you think of separating them into different image sets? Since the code currently groups images by aspect ratio and maintains them as separate image sets - which then get flagged as tiled or not. I'm assuming the issue you're facing is an image where the aspect ratio of the macro or label matches the regular tiled image, causing it to accidentally be grouped with the pyramid.
It might be desirable to keep track of the macro or label image for rendering elsewhere.
@pearcetm what do you think?

Strangely, the scenario that prompted this was actually a case where the macros and labels had completely different aspect ratios from the real tile images and were still getting rendered over the real slide data.  It could definitely be useful to keep the macro and label images for rendering elsewhere, but they should definitely be excluded from the pyramid.  If they were kept what would be a good way to represent them to the user, @abiswas97 ?

Strangely, the scenario that prompted this was actually a case where the macros and labels had completely different aspect ratios from the real tile images and were still getting rendered over the real slide data.

That's strange. When I use the demo page to open an SVS file, it opens the images in sequence mode in OpenSeadragon. The image pyramid works correctly as the first image:

If you use the button to navigate to the next pages, you'll see the macro and then the label:

Is the demo page giving you the same results with your SVS files as your local build?

With this branch, the demo page displays the slides correctly and only the slides, no macro/label.

Upon further inspection @abiswas97 was correct about the aspect ratio being relatively close in that they fall within the current tolerance, however they're at the very far end of this.  I've modified the existing filter to include information about the image description.  With this change, the label/macro images are preserved as separate sets as mentioned above, but never included in the pyramid.

Upon further inspection @abiswas97 was correct about the aspect ratio being relatively close in that they fall within the current tolerance,

I figured this must be true. With the wide variety of TIFF files out there, I tried to detect which pages belong in a pyramid automatically, but obviously this approach can fail in edge cases like the one you found.
I'm definitely open to checking for special cases we know about in advance (like SVS files). Let's just make it as robust as possible while we're making the change.

I've included your suggestions above to the latest.
Speaking of robustness, do you believe that there should be additional checks based on the file type itself or should they all be handled in the mapping function regardless?  I'm not sure how likely or if ever it would be the case that a non-SVS file implements a label/macro that is expected to be part of the tileset.

@mdushkoff My setup is similar to the demo page, where I use OSD's sequence mode to allow the user to view supplementary images if present.
I think having a check beforehand might help, since having file type specific context can be useful for the entire class. Maybe an ImageReader class that can have its own pre/post processing step when determining tilesets?
As for how likely other file types are to have a macro, I've noticed that most tiff based format (svs, tiff, ome.tiff, qptiff etc) do include one

@abiswas97 In those tiff based formats, do you know if it is expected that the macro is a separate image set to the pyramid, or is that an SVS-specific case to keep it separate?

@abiswas97 In those tiff based formats, do you know if it is expected that the macro is a separate image set to the pyramid, or is that an SVS-specific case to keep it separate?

Although I'm not away of the other TIFF formats that include macro images, I suspect that they, like SVS, would not expect the macro to be part of the main image pyramid.

This is looking good to me, @mdushkoff.
@abiswas97 did you have any further thoughts, or should I go ahead and merge this as-is?

Checking back in on this @abiswas97 and @mdushkoff. Do you think this is ready to merge?

I'd say so, unless there's any other concerns.

@pearcetm @mdushkoff just getting to this - sorry for being late to this discussion.
In images where I've noticed macro images, they've been separate images independent of the main image pyramid. So far I've seen this in tiff, ome.tiff, qptif, and svs (mostly svs). I'll see if I can find an open source example to attach here.
Given how freeform TIFF is, maybe some won't adhere to this standard? But maybe we can take it into account if we get pointed at an example from any future issue?
I think this looks good to merge. My only nit would be some clarity what s is, but this should be a very helpful change!

Thanks for this fix! Merging now.


Publish to PyPi

Actually publish the FASM library onto PyPi.

This looks fine, but we'll see what happens when it actually pushes to PyPi.


Missing library files in Java-11

Panel Version: 1.4.2
Daemon Version: 1.4.4
Service: minecraft
Docker Image: quay.io/pterodactyl/core:java-11
Modified: no
Errors that you are experiencing:
The library libXrender.so.1 seems to be missing from the default installation of pterodactyl. This messes with some mods since the server will crash without it.
[20:49:19] [main/FATAL] [minecraft/Main]: Failed to start the minecraft server
net.minecraftforge.fml.LoadingFailedException: Loading errors encountered: [
        Brazier (brazier) encountered an error during the load_registries event phase
§7java.lang.UnsatisfiedLinkError: /usr/local/openjdk-11/lib/libawt_xawt.so: libXrender.so.1: cannot open shared object file: No such file or directory
]
        at net.minecraftforge.fml.ModLoader.waitForTransition(ModLoader.java:263) ~[forge:?] {re:classloading}
        at net.minecraftforge.fml.ModLoader.dispatchAndHandleError(ModLoader.java:230) ~[forge:?] {re:classloading}
        at net.minecraftforge.fml.ModLoader.gatherAndInitializeMods(ModLoader.java:196) ~[forge:?] {re:classloading}
        at net.minecraftforge.fml.server.ServerModLoader.load(ServerModLoader.java:44) ~[forge:?] {re:classloading}
        at net.minecraft.server.Main.main(Main.java:95) ~[?:?] {re:classloading}
        at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?] {}
        at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:?] {}
        at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?] {}
        at java.lang.reflect.Method.invoke(Method.java:566) ~[?:?] {}
        at net.minecraftforge.fml.loading.FMLServerLaunchProvider.lambda$launchService$0(FMLServerLaunchProvider.java:51) ~[server.jar:36.1] {}
        at cpw.mods.modlauncher.LaunchServiceHandlerDecorator.launch(LaunchServiceHandlerDecorator.java:37) [modlauncher-8.0.9.jar:?] {}
        at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:54) [modlauncher-8.0.9.jar:?] {}
        at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:72) [modlauncher-8.0.9.jar:?] {}
        at cpw.mods.modlauncher.Launcher.run(Launcher.java:82) [modlauncher-8.0.9.jar:?] {re:classloading}
        at cpw.mods.modlauncher.Launcher.main(Launcher.java:66) [modlauncher-8.0.9.jar:?] {re:classloading}
        at net.minecraftforge.server.ServerMain$Runner.runLauncher(ServerMain.java:63) [server.jar:?] {}
        at net.minecraftforge.server.ServerMain$Runner.access$100(ServerMain.java:60) [server.jar:?] {}
        at net.minecraftforge.server.ServerMain.main(ServerMain.java:57) [server.jar:?] {}

Why would a headless server even need that library?
Have you tried starting it with nogui?

Why would a headless server even need that library?
Have you tried starting it with nogui?

I tried starting it with nogui but it didn't change the result.

The issue seems to have resolved itself after restarting the whole machine. Not sure how that fixed it but it worked. Sorry for wasting any time.


Lint on build + serve

Currently ng lint can only be run as a stand-alone command. This limits its usefulness as developers have to remember to manually run it before committing or install a linting package for their editor. Lint rules are most effective when violating them results in build failure. This makes failure obvious to developers regardless of their editor setup or workflow. While it is currently possible to accomplish this by chaining ng lint and ng build in npm scripts, this solution is not compatible with watcher tasks such as ng serve.
To fix this I recommend adding a flag to the ng serve and ng build commands that automatically runs ng lint before each build and causes a build failure on any linter rule violation.

At least for me, the lint command is fired in my CI process. It's not possible to merge a branch whose CI is failing. Apart from that, I really don't want the lint to slow down the already very slow build or serve commands. (lint takes forever on large projects)
Summed up, I don't only find this suggestion unnecessary, but also very disturbing. At least if this would be the default behaviour.

We actually have tslint running on ng test cycles at the moment.
I'm not too sure that I'd want linting running on all rebuilds though... it'd slow them down somewhat, which I think is suboptimal but maybe not everyone agrees on that. I'll label it as a discussion.

I agree it shouldn’t be on by default. My suggestion is only that an option be added to enable linting on auto-rebuid.

Hello there.
Is there in the roadmap (or currently) a way to configure our own commands as the npm one's for the ng-cli?

@fbobbio please open a new issue instead of asking questions on unrelated issues.

I totally agree with @diminutivesloop, an option for ng serve would be the best of both worlds.
I would be happy to jump into this if you think its a useful feature @filipesilva.

Just to also add to this, I've seen some other Angular 2 / Webpack boilerplates have the linting results be shown in the browser console, that is extremely handy too (along with any compilation errors).

I'd also be very interested in this - though for now I'll see if I can mimic what @christiandreher is doing.

Alright, I see a fair amount of interest in it so I'll tag it for work instead of just discussion. It's not a high prio though.

Totally agree! 👍

I'm running beta.24 of the cli and it seems to me, that ng test is currently NOT running tslint automatically (only if I pass --lint), whereas an earlier comment stated that it would do. Is this correct or am I missing sth.?
If I am correct and it is currently disabled, is there a way to enable it per default?

I'm running beta.24 of the cli and it seems to me, that ng test is currently NOT running tslint automatically (only if I pass --lint), whereas an earlier comment stated that it would do. Is this correct or am I missing sth.?
If I am correct and it is currently disabled, is there a way to enable it per default?

Hmm I also noticed that running ng test --lint doesn't give same lint errors as npm run lint - is there a reason for this?

I noticed that in #4114 , the --lint option was removed from ng test.
Would the team still consider a PR for this issue,  #2551 , that would add a --lint option to ng build/serve ? @filipesilva

@tarlepp I guess that is https://github.com/angular/angular-cli/issues/4114 the reason

Heya, I have bad news. We don't intend to have it on build/serve for the same reasons as described in https://github.com/angular/angular-cli/issues/4114.

What about incremental linting?
I built this within a starter.  Then you update that map only for changed files and output all errors.
It's blazing fast.

@filipesilva I understand why you would remove the --lint flag in the context of unit tests, it's a trap. But in the case of a serve there are obviously people that would find this feature useful (including my self)
Beside the fact people simply want this, the reasons listed in #4114 are not as relevant to the serve. I think having the option there and disabling it by default would be the best of both worlds.

I would really appreciate having this built into ng serve so that my teammates see this when running.  Am just exploring this cli and I think this might be a deal breaker for me.

Either NG Serve integration or a way of running ng lint with a watch would be useful to my workflow. I work with pretty strict expanded lint rulesets and while WebStorm does give you context highlighting on TSLint violations when you are inside of a file with violations, it does not give immediate feedback for a solution-level analysis unless you explicitly run TSLint. This means that you may be running development via ng serve and not realize that there are linting errors and must then remember to manually lint your code, which slows down the process. Sure, the CI process catches the error, but you hate to see folks breaking the build when tooling like ng serve could tell them in advance that they have linting issues, or a separate watching process could be set up for ng lint, which seems like it might be a better solution and one that could be easier to implement..

@IntegerMan if your concern is having linting errors in CI, you could give husky a try.

Hey, thanks @svi3c I'd not heard of that. That just might meet my needs. I'd rather get that feedback from a tool in real-time via serve or a watching lint, but this would meet that concern. I'll try it out over the weekend.


feat: Move shuffle block decompression and decoding to native code and add LZ4 & Snappy support

Which issue does this PR close?

Part of https://github.com/apache/datafusion-comet/issues/1123
Closes https://github.com/apache/datafusion-comet/issues/1178
Rationale for this change

We currently perform encoding + compression in native code and decoding + decompression in JVM code. There are some downsides to this approach:

We need compatible and efficient JVM and Rust compression libraries (this seems challenging for LZ4)
We need compatible and efficient JVM and Rust encoding libraries (we use Arrow IPC currently)
We cannot have unit tests for roundtrip encoding and compression; only integration tests (results in slow dev cycles)
Makes it difficult to experiment with different encoding and compression libraries and techniques
We are missing out on performance, potentially

What changes are included in this PR?

Call native code for decompression + decoding
Add metrics for decode + decompress
Add support for LZ4
Implement unit tests for round-trip

ZSTD

LZ4

Microbenchmarks
shuffle_writer/shuffle_writer: encode (no compression))
                        time:   [10.044 µs 10.371 µs 10.698 µs]
shuffle_writer/shuffle_writer: encode and compress (lz4)
                        time:   [132.60 µs 133.01 µs 133.51 µs]
shuffle_writer/shuffle_writer: encode and compress (zstd level 1)
                        time:   [217.81 µs 218.01 µs 218.25 µs]

TPC-H

How are these changes tested?

Existing tests

Codecov Report
Attention: Patch coverage is 69.86301% with 22 lines in your changes missing coverage. Please review.

Project coverage is 34.67%. Comparing base (ea6d205) to head (62e4aa6).

Files with missing lines
Patch %
Lines

...sql/comet/shuffle/NativeBatchDecoderIterator.scala
65.51%
11 Missing and 9 partials :warning:

...e/spark/sql/comet/CometBroadcastExchangeExec.scala
71.42%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #1192       +/-   ##
=============================================
- Coverage     55.28%   34.67%   -20.61%
- Complexity      868      952       +84
=============================================
  Files           112      115        +3
  Lines         10969    43576    +32607
  Branches       2116     9528     +7412
=============================================
+ Hits           6064    15110     +9046
- Misses         3826    25502    +21676
- Partials       1079     2964     +1885

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

The status is that this is working, but I am running into some executor OOMs when trying to run complete benchmarks. I will pick this up again after the holidays.

@Dandandan you may be interested in the benchmark results

Added Snappy.
shuffle_writer/shuffle_writer: encode and compress (snap)
                        time:   [87.556 µs 88.328 µs 89.092 µs]

Hmm, the difference between ZSTD and LZ4 looks not significant? Does the benchmark of TPC-H shown in the description include native shuffle block decompression and decoding? Do we have the number of difference between current Comet (ZSTD) and this patch (ZSTD + native decompression and decoding)?

Hmm, the difference between ZSTD and LZ4 looks not significant? Does the benchmark of TPC-H shown in the description include native shuffle block decompression and decoding? Do we have the number of difference between current Comet (ZSTD) and this patch (ZSTD + native decompression and decoding)?

Sure, here are fresh benchmarks

Branch
Compression Codec
TPC-H Time

0.4.0
ZSTD
364

main
ZSTD
364

native-decode
ZSTD
367

native-decode
Snappy  343

native-decode
LZ4
340

I had hoped for a bigger win, but 340 compared to 364 is still a 7% improvement overall.

I enabled the unsafe version of lz4_flex, and the time reduced from 340s to 336s.


Key Messages for the product

Create Key Messages for GC Forms. Enable people to communicate about the product in a consistent way.
Success Criteria:

[x] Draft created in collab with Outreach
[x] Reviewed by Lesley
[ ] Reviewed by Stevie-Ray
[ ] Approved by Leanne and Chelsea
[ ] Translated

Yes! Believe it's a duplicate
*Jose Jimenez *[him/il]
Canadian Digital Service (CDS)/Service numérique canadien (SNC)
Treasury Board of Canada Secretariat/Secrétariat du Conseil du Trésor du
Canada
digital.canada.ca / numerique.canada.ca
613-293-8364
On Mon, Aug 21, 2023 at 9:00 PM Abi-Nada @.***> wrote:

Hey @jose-cds https://github.com/jose-cds ! Can I go ahead and close
out this ticket?
—
Reply to this email directly, view it on GitHub
https://github.com/cds-snc/platform-forms-client/issues/2554#issuecomment-1687257931,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGCLMG6GDHDQR5YV3KFJBLXWQADLANCNFSM6AAAAAA3TB6BWE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Key Messages are ready for PM review! @srtalbot
https://docs.google.com/document/d/1gDN6G80oS3IXWvC01y1Ulnr0nrNneKvcb1BU_ERdCR0/edit

To review to align with language on Marketing page.


example for custom resolve functions

would love to see some better documentation on using a custom resolve function. the docs have a pretty good primer, but an example might help the less intuitive among us. (read: “me”).

ps: loving the built in resolutions– quick and easy to set up, and they've covered most use-cases I've come across till now!

An example could be the following:
Say you wanted a resolver that would camelcase all matching files.

I'm a assuming the following directory structure in your source:

--- src
 |- main.js
 |- some-other-file.js
 `--- includes
    |- one-include.js
    `- another-include.js

main.js could contain the following code:
const includes = require('./includes/*.js', { mode: 'hash', resolve: function my_camelcase_resolver(base, files, config) {
    function camelize(str) { // camelize taken from http://stackoverflow.com/a/2970667
      return str.replace(/(?:^\w|[A-Z]|\b\w)/g, function(letter, index) {
        return index == 0 ? letter.toLowerCase() : letter.toUpperCase();
      }).replace(/\s+/g, '');
    }
    return Object.keys(files).reduce(function(resolved_files, file) {
        resolved_files[camelize(file)] = files[file];
        return resolved_files;
    }, {});
} });

Writing this down, I realized this is maybe not the most user-friendly way to do this, especially if you want to use the same resolver multiple times. Also, since it's been a long time since I've worked on this plugin, the example might not be functional. I suggest you try it out and let me know how it works out. I have several ideas of how this use-case might be improved upon.

Woah, would not have gotten that! I ended up getting pretty damn good results from the built-in resolvers... Still, great to see an example of how this works. Anyways, appreciate the response; a simplified/streamlined interface for implementing custom resolvers. More than happy to help if/where I can.


CI: add a timeout on docker build

buildjet internet was extremely slow in this run (not sure why, was fine on rerun), and caused this setup to run for 30+ minutes until I manually stopped it. 15 minutes should be plenty to rebuild even the full image.
https://github.com/commaai/openpilot/actions/runs/7117904537/job/19379679547?pr=30618

timeout-minutes isn't supported by github composite actions :/

seems like buildjet internet might just be very slow today, had same issue on the aarch build: https://github.com/commaai/openpilot/actions/runs/7118335440/job/19381050609


Failed to install with CocoaPods

Console says:
[!] A dependency with an external source may not specify version requirements (Yoga).
A part of Podfile:
  pod 'RSocket', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/RSocket.podspec?token=ADr9NE_I05Vu8g7oq_g6g_9FLx784NFmks5bJ5LvwA%3D%3D'
  pod 'DoubleConversion', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/DoubleConversion.podspec?token=ADr9NOxtIEmr5ODP9PWq6-sht-Ye6UYGks5bJ5MjwA%3D%3D'
  pod 'glog', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/glog.podspec?token=ADr9NBHbrlbkFR3DQTPzj0CnZdria4jvks5bJ5M3wA%3D%3D'
  pod 'Folly', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/Folly.podspec?token=ADr9NNTjwJ8xqLFwc3Qz3xB3GsCk-Esmks5bJ5NGwA%3D%3D'
  pod 'PeerTalk', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/PeerTalk.podspec?token=ADr9NB8frQTrUWytsMXtdv_P8km7jV_Mks5bJ5NbwA%3D%3D'
  pod 'Yoga','~> 1.8.1', :modular_headers => true
  pod 'Sonar', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/xplat/Sonar/Sonar.podspec?token=ADr9NFO7byH9uAuhGAIEYuoJeBNyBxf6ks5bJ5N8wA%3D%3D'
  pod 'SonarKit', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'SonarKit/SonarKitLayoutComponentKitSupport', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'SonarKit/SKIOSNetworkPlugin', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'ComponentKit', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/ComponentKit.podspec?token=ADr9NNV9gqkpFTUKaHpCiYOZIG3Ev-Hyks5bJ5O-wA%3D%3D'

  post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            config.build_settings['SWIFT_VERSION'] = '4.0'
        end
     end
  end

I've solved that issue removing the version on the Yoga dependency.
But after that I get
[!] Failed to load 'SonarKit' podspec:
[!] Invalid `SonarKit.podspec` file: undefined method `static_framework=' for #<Pod::Specification name="SonarKit">
Did you mean?  weak_framework=.

It builds with the following set up
target 'Sample' do

  pod 'RSocket', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/RSocket.podspec'
  pod 'DoubleConversion', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/DoubleConversion.podspec'
  pod 'glog', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/glog.podspec'
  # pod 'CocoaLibEvent', '~> 1.0'
  #:podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/glog.podspec?token=ADr9NImskgi8eZGAADCQH2_PXtgB7-b-ks5bIDrowA%3D%3D'
  pod 'Folly', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/Folly.podspec'
  pod 'PeerTalk', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/PeerTalk.podspec'
  pod 'Yoga','~>1.8.1', :modular_headers => true
  pod 'Sonar', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/xplat/Sonar/Sonar.podspec'
  pod 'SonarKit', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'SonarKit/SonarKitLayoutComponentKitSupport', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'SonarKit/SKIOSNetworkPlugin', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/SonarKit.podspec'
  pod 'ComponentKit', :podspec => 'https://raw.githubusercontent.com/facebook/Sonar/master/iOS/third-party-podspecs/ComponentKit.podspec'
  post_install do |installer|
        installer.pods_project.targets.each do |target|
            if ['YogaKit'].include? target.name
                target.build_configurations.each do |config|
                    config.build_settings['SWIFT_VERSION'] = swift_version
                end
            end
        end
    end
end

Let me know if it works

I had same error
I used CocoaPods(1.2.0)
> Console says:
> [!] A dependency with an external source may not specify version requirements (Yoga).

I tried CocoaPods (1.5.2) and it worked!

Awesome!, I will close the issue then.


SSH to AWS EC2 with Key File

I was connecting to a virtual server on namecheap via SSH as follows
gateway = Net::SSH::Gateway.new(host, username,
  password: "xxxxxxxx",
  port: "21098",
  )

we have now migrated to AWS EC2 and connect to the instance using a key file *.pem. How do I use this gem with the key file to create an SSH tunnel?
Yours assistance is greatly appreciated. Many thanks

Digging around online I found that you can specify "keys" as part of the options after host, username.
Net::SSH::Gateway.new(remote_host, remote_user, keys: ['/path/to/key.pem'])


Add creation of bazelrc for running unnested in prow

What this PR does / why we need it:
bootstrap image from test-infra creates the bazelrc for us which we copy at the moment into ci.bazelrc to have it available for the build. When using kubevirt/builder with KUBEVIRT_RUN_UNNESTED=true we don't have it any more.
Therefore we borrow parts of create_bazel_cache_rcs.sh from test-infra/bootstrap, which now creates ci.bazelrc so that we have a bazel config when running in prow. Also we add KUBEVIRT_CREATE_BAZELRCS env var to selectively enable it on prow builds when using KUBEVIRT_RUN_UNNESTED.
Fix some shellcheck errors BTW.
First time usage is here: https://github.com/kubevirt/project-infra/pull/630
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

make generate fails due to swagger problem, need to wait until fix is in

Rebase done, we should be good now.

/retest

/lgtm

/lgtm

/approve

/override pull-kubevirt-e2e-k8s-1.18
This is a test unrelated to the change.


[Fixes #254] Fix some chinese translation and date display format

fix for issue #254
fix: the date following localized date description
fix: some translation in Chinese Simplified

Hi @wywwwwei sorry I missed the GitHub notification so sorry for late reply,
Thank you for your contribution, it's mean a lot!


Segmentation fault (core dumped)

When I run main.py ，I get this error: Segmentation fault (core dumped)

me too. I got the same error at the begging.
I'm using server gpu(no GUI possible). so I set gui=False in the command line. but still I got the error eventhough I installed all requirements successfully.
umber of points at initialisation :  5000
[INFO] loading zero123...
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please run
python -m bitsandbytes
and submit this information together with your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
bin /usr/local/lib/python3.10/dist-packages/bitsandbytes/libbitsandbytes_cuda118_nocublaslt.so
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:145: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/usr/local/nvidia/lib64'), PosixPath('/usr/local/nvidia/lib'), PosixPath('/home/jovyan/.local/lib/python3.10/site-packages/cv2/../../lib64')}
warn(msg)
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so.11.0
CUDA SETUP: Highest compute capability among GPUs detected: 7.0
CUDA SETUP: Detected CUDA version 118
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:145: UserWarning: WARNING: Compute capability < 7.5 detected! Only slow 8-bit matmul is supported for your GPU!
warn(msg)
CUDA SETUP: Loading binary /usr/local/lib/python3.10/dist-packages/bitsandbytes/libbitsandbytes_cuda118_nocublaslt.so...
Loading pipeline components...: 100%|████████████████████████████████████████████████████████████████| 6/6 [00:00<00:00,  7.65it/s]
[INFO] loaded zero123!
100%|████████████████████████████████████████████████████████████████████████████████████████████| 500/500 [00:43<00:00, 11.58it/s]
[INFO] save model to logs/name_model.ply.
/usr/local/lib/python3.10/dist-packages/torch/functional.py:504: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at ../aten/src/ATen/native/TensorShape.cpp:3483.)
return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
Tensor occ torch.Size([128, 128, 128]) torch.float32 cuda:0 ∈ [0.0, 81.30426788330078] μ = 0.06861298531293869 σ =
1.0725795030593872
[INFO] mesh cleaning: (9162, 3) --> (5310, 3), (18332, 3) --> (10628, 3)
[INFO] marching cubes result: torch.Size([5310, 3]) (-0.6954547166824341-0.7155003547668457), torch.Size([10628, 3])
[INFO] unwrap uv...
libEGL warning: DRI3: Screen seems not DRI3 capable
libEGL warning: DRI2: failed to authenticate
Segmentation fault (core dumped)

I meet this problem too, have you fixed it?

Could you try appending force_cuda_rast=True and run again?


Bro Doublehooked it

LL

no?

no ?

where is the BAT file

Y'all are saying no. Look at the 2nd folder in the main folder there's his web hook and his advertising stuff

Imagine getting scammed while trying to scam, noobs


CNDB-11508: Enable our test config explicitly for JDK22 - WIP branch, NO COMMIT, NO REVIEW NEEDED YET

What is the issue
We cannot run tests on JDK22 with the current config
What does this PR fix and why was it fixed
Port CASSANDRA-18467, CASSANDRA-18179, CASSANDRA-18258 and add additional configuration for JDK22
Porting those tickets and adding updates based on testing we already did in a POC branch will enable us to run tests on JDK22 from the main branch.
Removes the use of CASSANDRA_USE_JDK11 (we should verify that will not affect CI systems and other infra scripts in any way) and introduces the build.xml properties java.default and java.supported.
ant generate-idea-files now support JDK 8, JDK 11, and JDK 22.
To add support for another JDK the java-jvmargs property must be set for the JDK in question (see how it's done in build.xml for Java 11 and 22)
Other minor, but notable changes are:

test jvmargs are now added to idea run configurations
.idea dir and project iml file are first removed and then recreated during ant generate-idea-files
Adds some additional add-opens and add-exports needed and upgrades mockito for testing.

Checklist before you submit for review

[ ] Make sure there is a PR in the CNDB project updating the Converged Cassandra version
[ ] Use NoSpamLogger for log lines that may appear frequently in the logs
[ ] Verify test results on Butler
[ ] Test coverage for new/modified code is > 80%
[ ] Proper code formatting
[ ] Proper title for each commit staring with the project-issue number, like CNDB-1234
[ ] Each commit has a meaningful description
[ ] Each commit is not very long and contains related changes
[ ] Renames, moves and reformatting are in distinct commits

The PR is not finished, but I wanted to do preliminary JDK11 testing to see whether I broke anything so far

I think this looks good for a full review.  When you put it up for full review, please include links to 11/17/22 based CI runs.

Cleaned JDK1.8 from allowed JDKs and fixed a bug in the tooling config.
CI links and additional business-related questions will be posted on the ticket for discussion in a bit.
Moving out of draft to trigger the JDK11 CI run.

There was some issue with CI.
I rebased the branch and force-pushed it to trigger it again, hoping that it was some environmental temporary problem.
The old build was also in a weird state in Jenkins—it was shown as failed, but some jobs were still marked in progress. Hopefully, the new trigger clears the old run and triggers a fresh new one without problems.

Thank you for the reviews.
@jacek-lewandowski , I pushed a few small commits to address your feedback and I hope I responded to all your questions in comments. Please let me know.
Side note: The jdk22 server.options files mostly mimic what we had for JDK17 with additions for JDK22. Wondering what we want to leave as jvm options (other than add-exports/add-opens which are clear), probably just mimic what we would use in server.options for now? Same as we did in Apache?

CI is in a bad shape today going up and down and I am not being able to really check test results.
I am pushing some review feedback updates to docs and I will move the PRs to draft so they do not trigger useless CI triggers. I will move to another issue I am working on and check back later.

Feedback addressed. @jacek-lewandowski , I believe the main questions are:

can we remove dead code, leftovers (port CASSANDRA-18255)in a follow up ticket so we do not delay further this one:
https://github.com/datastax/cassandra/pull/1398#discussion_r1850891892
let me know what you want me to do - https://github.com/datastax/cassandra/pull/1398#discussion_r1850776852
Option 1: remove it from both main and main-5.0 and diverge from Apache (I am against that);
Option 2: remove the deletions in main and open follow up ticket to improve that target for all branches? (I prefer this one)

The PR failed with no space left - https://jenkins-stargazer.aws.dsinternal.org/job/ds-cassandra-pr-gate/job/PR-1398/6/cloudbees-pipeline-explorer/. I will follow up with the db team as there was some ongoing work on these matters yesterday.

Pushed empty commit for new CI run

The failing tests marked as regressions are not related, and we've seen before - timeouts, some leakage we see a lot these days in CI. They did not also fail locally for me.

The two failures marked as new after rebase are totally unrelated to what we do here. I will open tickets for them.


YAML 1.1 Pair, Key is y, get parsed to true (bool)

Describe the bug
When parsing a Unity YAML file, one object is defined with key-value pairs, including one with a y in it (like in X,Y coordinates). When parsed, the key is transformed to a boolean true.
To Reproduce
Using the Playground, parse the following YAML
%YAML 1.1
%TAG !u! tag:unity3d.com,2011:
--- !u!213 &21300000
Sprite:
  m_Rect:
    x: 0
    y: 128
    width: 128
    height: 128

gives the following result:
[
  Pair {
    key: Scalar { value: 'x', range: [Array], source: 'x', type: 'PLAIN' },
    value: Scalar { value: 0, range: [Array], source: '0', type: 'PLAIN' }
  },
  Pair {
    key: Scalar { value: true, range: [Array], source: 'y', type: 'PLAIN' },
    value: Scalar { value: 128, range: [Array], source: '128', type: 'PLAIN' }
  },
  Pair {
    key: Scalar { value: 'width', range: [Array], source: 'width', type: 'PLAIN' },
    value: Scalar { value: 128, range: [Array], source: '128', type: 'PLAIN' }
  },
  Pair {
    key: Scalar { value: 'height', range: [Array], source: 'height', type: 'PLAIN' },
    value: Scalar { value: 128, range: [Array], source: '128', type: 'PLAIN' }
  }
]

Expected behaviour
The key of a key-value pair, should not be transformed according to the boolean rule.
Versions (please complete the following information):

Environment: Node 22.8.0
yaml: 2.5.1

Additional context
I found the schema used, but I have no clues where it get used when parsing the key.
https://github.com/eemeli/yaml/blob/5adbb605b64094b57bc95cbc24587e6f36b3f9a8/src/schema/yaml-1.1/bool.ts

TIL, this is by design for YAML 1.1 that the keys are parsed like anything else.  It might make sense to add an option like stringKeys to enforce their parsing as such.


Validation: Auth during server-side Blazor static prerendering

Previously it crashed if you had <CascadingAuthenticationState> and were doing static prerendering.
https://github.com/aspnet/AspNetCore/pull/12318

Done.


Process would not work and more than likely should be removed.

Not only does the template not exist for this process in Power Automate, but you cannot create a composite model with a Real Time streaming dataset which means you cannot generate duration or meaningful data for the most part with the streaming dataset option. Also the screenshot for Power Automate isn't correct as you'd need to generate a payload in JSON format.

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

ID: bec98c05-d02a-8bf4-52e9-3f3783accfc0
Version Independent ID: 8247ddbe-2928-e10a-8bb5-8f42ed73016b
Content: Load Data into a Power BI streaming dataset and build a Dataflows Monitoring Report with Power BI
Content Source: powerquery-docs/dataflows/load-dataflow-metadata-into-power-bi-dataset.md
Service: powerquery
GitHub Login: @miquelladeboer
Microsoft Alias: mideboer

Please ignore (I'm using bot automation to add labels). #please-open

Hi @cvillegashak
Thanks for your feedback! The article in question has been updated several times since you provided this issue. Is the issue still there in the article? Please let me know so I can update the article.

Hi @cvillegashak
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Sequence send not found

My first test of two-chainz was successful, but having issues after retrying due to sequence send not found. I've closed and re-opened the channel, re-started both chains locally, verified balances, and initiated lite clients. I'm not sure what else I am missing. The following error occurs with both chains trying to transfer to each other.
stoner@stonerzone:~/iqlusion/relayer$ rly tx transfer ibc0 ibc1 10000n0token false $(rly keys show ibc1 testkey) [2020-04-14|12:20:10.690] ✘ [ibc0]@{56323} - msg(0:transfer) err(channels: sequence send not found) Error: failed to send first transaction

@ronaldstoner thanks for the report. If you are having issues please include full reproduction steps and run the commands with -d. This will include full output from the command including raw transaction data.

Same issue here:
rly tx transfer cosmoshub irishub 1000uatom true cosmos1zzcuqga83przn9lkfxzza6p5arftrhcmxwystc -d
I[2020-04-17|08:41:52.661] - [cosmoshub] -> sending transaction:
[{"type":"ibc/transfer/MsgTransfer","value":{"source_port":"transfer","source_channel":"cosmoshubchan","dest_height":"90159","amount":[{"denom":"transfer/irishubchan/uatom","amount":"1000"}],"sender":"cosmos12pnvxv2ee5ruvgykz2md46rhd5tkk29eqsvkn0","receiver":"cosmos1zzcuqga83przn9lkfxzza6p5arftrhcmxwystc"}}]
I[2020-04-17|08:41:52.662] ✘ [cosmoshub]@{89928} - msg(0:transfer) err(channels: sequence send not found)
{"height":"89928","txhash":"5C05F8B253BC6681107B48CCECA1AC1808EB032AF494D855DBD6A69B3AB02515","codespace":"channels","code":8,"raw_log":"failed to execute message; message index: 0: module does not own channel capability: channel capability not found","gas_wanted":"200000","gas_used":"46655"}
Error: failed to send first transaction

Resolving this as I have not been able to reproduce.


Bug not generating entries for all headers in a file

In my document, here: Link, doctoc stops generating entries in the table of contents at 'concrete types' heading, while there are still headings after.

That is very odd.
Could you please condense this bug into a simple test so we can repro this consistently and then fix?


Texture

Hi, using
import pywavefront
from pywavefront import visualization
obj = pywavefront.Wavefront('me.obj')
visualization.draw(obj)
I get AttributeError: 'ImageData' object has no attribute 'texture'.
What am I doing wrong?

error line is
image = pyglet.image.load(name).texture

Do you have a full stack trace?

What version of pywavefront and pyglet? It might be related to the recent pyglet 1.4.x release?

@Fivetuple I think this issue is still relevant as the error message given to the user is very poor. Can you share what triggered this error? Did you resolve the issue considering it was closed?

Hi,
pyglet is 1.4.1
pywavefront is 1.0.5
Sorry, I closed it accidentally.  I'll post the full trace tomorrow - not at work now.  The texture is held in a separate isomap.png file - how do I tell pywavefront about this file?

Textures are loaded from the same directory as the obj file. The texture referenced in the material can of course also use a relative path. This is pretty standard.

pywavefront 1.1.0 was released and should solve this issue. Pyglet 1.3 and 1.4 should work.

@Fivetuple All well on your side? Just making sure 😄

Yes, fixed!  Thanks for checking.


UXUI-239 new lozenge component

What does this PR do? (please provide any background)
Adds a new lozenge component to the UI-toolkit
What tests does this PR have?
I've added new tests for the component in the src/components/lozenge/__tests__ directory
run npm test
How can be this tested?
run grunt docs and have a play
Screenshots / Screencast

What gif best describes how you feel about this work?

Developer Definition of Done/Quality Checklist (for PR author to complete BEFORE code review):

[x] I have checked the contributing document and I'm happy for this to be reviewed.

Software Engineer or Developer review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve run all the tests locally and they pass.

Software Engineer or project guru review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve run all the tests locally and they pass.

Taking Dev

:+1: Once comments have been addressed

Looks good :+1: will need size prop adding at a later date.


metrics filter not working for EMR

I get following warning
(py36) bash-3.2$ custodian run -c unused-emr.yml --verbose -s out
2018-08-03 10:28:21,406: custodian.commands:DEBUG Loaded file unused-emr.yml. Contains 1 policies
2018-08-03 10:28:21,418: custodian.aws:DEBUG using default region:eu-west-1 from boto
2018-08-03 10:28:21,874: custodian.output:DEBUG Storing output with <DirectoryOutput to dir:out/unused-emr-clusters>
2018-08-03 10:28:21,875: custodian.policy:DEBUG Running policy unused-emr-clusters resource: emr region:eu-west-1 c7n:<IP_ADDRESS>
2018-08-03 10:28:23,365: custodian.filters:DEBUG Querying metrics for 1
2018-08-03 10:28:23,420: custodian.filters:WARNING CW Retrieval error: 'ClusterId'
2018-08-03 10:28:23,420: custodian.resources.emrcluster:DEBUG Filtered from 1 to 0 emrcluster
2018-08-03 10:28:23,420: custodian.policy:INFO policy: unused-emr-clusters resource:emr region:eu-west-1 count:0 time:1.54
2018-08-03 10:28:23,421: custodian.output:DEBUG metric:ResourceCount Count:0 policy:unused-emr-clusters restype:emr scope:policy
when attempting to run
  - name: unused-emr-clusters
    description: |
      Retrieve all unused emr clusters
    resource: emr
    filters:
      - type: metrics
        name: IsIdle
        value: 1
        op: equal

Note that i do have a cluster thats currently idle.
Is there any other way we can query for unused EMR clusters?
Note that filtering for metrics is working just fine for other resources, EMR is the only one we've had issues with.

Tested the patch in #2704 - it passes validation, however the feature is not working.
Still returning 0 resources (not that i did test this in a context is IsIdle is set to true
2018-08-07 00:09:27,221: custodian.commands:DEBUG Loaded file /custodian/checks/unused-emr.yml. Contains 1 policies 2018-08-07 00:09:27,758: custodian.output:DEBUG Storing output with <DirectoryOutput to dir:/custodian/out/unused-emr-clusters> 2018-08-07 00:09:27,759: custodian.policy:DEBUG Running policy unused-emr-clusters resource: emr region:eu-west-1 c7n:<IP_ADDRESS> 2018-08-07 00:09:29,316: custodian.filters:DEBUG Querying metrics for 1 2018-08-07 00:09:30,091: custodian.resources.emrcluster:DEBUG Filtered from 1 to 0 emrcluster 2018-08-07 00:09:30,092: custodian.policy:INFO policy: unused-emr-clusters resource:emr region:eu-west-1 count:0 time:2.33 2018-08-07 00:09:30,093: custodian.output:DEBUG metric:ResourceCount Count:0 policy:unused-emr-clusters restype:emr scope:policy
Also tried toggling the value in the filter to 0 - yields same results.


Swift 4.1 support (minor change)

@danieldahan Please consider this, my projects ain't building anymore without me editing the .pbxproj first each time I update carthage dependencies.

Have you tried the latest release of Motion? I built with the latest Xcode using it. Let me know and I will be happy to make the change, as I am making some releases today. Thank you!


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/buildbot-worker-feedstock#10.


Awesome Electron

Hi,
Please add Trilogy to "Awesome Electron" (15,200 stars) to increase visibility (users).
Thank you.
https://github.com/sindresorhus/awesome-electron/issues/548

I'll add it but now's not a great time just because the TS rewrite isn't out yet and if trilogy's going to be on that list I'd like to note that it's a well-typed TS library. Let's wait until v2 drops.


Official dockerfile for the course

Hi!
Is there a need for an official dockerfile for the course? I've already created one on the forum, but maybe it's a good idea to have it in the repo?
Obviously, I'm willing to help with that.

I've created a separate repo for now: https://github.com/Liberus/fastai_part1_docker

Thanks a lot for the offer! We'd rather not offer an official dockerfile, but thanks a lot for the offer, and for making yours available on the forum. Feel free to add a link to it on one of the wiki topics there too.


Uncaught error: OK [200]

Hello,
I am now receiving this error online when I used advanced search, Uncaught error: OK [200].
See error yourself at: http://<IP_ADDRESS>/index.php?action=searchplayer&server=0&player=%25

Hey @DMTGCory
It seems you messed with some of the action pages, because if you take a look in your developer console, you can see that the XHR request (which should actually only return JSON), returns HTML markup. And that breaks the JSON, thus the search is invalid and cant get parsed properly:

(http://<IP_ADDRESS>/index.php?action=searchplayer&player=%&server=0&excluderecords=undefined&ajax=true&size=10&page=0&sortby=column[0]=0&filter=filter)
So, just remove the HTML parts and it should work again :)


WebGL

Hi! First of all: awesome work! Thank you very much to have this available for all of us.
Wanted to ask if this works on WebGL. I made the WebGL build but I am only getting a white texture.
Much apreciated!

Barracuda on WebGL only supports CPU inference while this sample uses GPU inference.
You may be able to change it to use CPU inference, although I have never tried it.
Please check the Barracuda documentation for details.


Fix wrong format specifiers in e_aptypes_is_null_str_nr

Whoops! But: I actually wanted the %d as the first specifier in the format string (it's easier to see it, because the string might be very long), but the arguments in the siemg() should have been the other way around.
Thanks for spotting this!


Documentation updates for return types

Description of change
I have added return types docs for several services. This is in relation to pull request #891.

Hey @AliSajid ! How's the work going, do you need help with anything?

Hey @AliSajid ! How's the work going, do you need help with anything?

Hi @iulianbarbu . I have warp and poise left to fix. Otherwise the tests pass and the format check also passes. I think these should be good to go.


drive qr can't find drive context

Inside one of my mounted drives, running drive list and drive pull works, but drive qr gives me:
no drive context found; run `drive init` or go into one of the directories (sub directories) that you performed `drive init`

Hello again @indragiek, as discussed offline, you were passing in a path relative to root '/' as in '/home's parent :) so this was making drive run relative to that path in which it wasn't mounted.

:+1:


RmlUI doesn't renderer text in Web after window resize

Textures generated by RmlUI are permanently lost on context loss.
RmlRenderer has to store both Image and Texture in RmlRenderer::GenerateTexture

Fixed by you 👍🏻

Fixed by you 👍🏻


I think a typo exists at Custom configuration section's example code

about 'EFConfigurationProvider/EFConfigurationExtensions.cs:'
title is 'EFConfigurationProvider/EFConfigurationExtensions.cs:'
but code class name is 'public static class EntityFrameworkExtensions'
thanks

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

ID: 0eb4d762-3910-2c57-8741-3306c15537b6
Version Independent ID: ba2254a0-ea8e-9b28-e94b-edd0fc58c043
Content: Configuration in ASP.NET Core
Content Source: aspnetcore/fundamentals/configuration/index.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @devtedlee ... I'll take a look as soon as I get back to the office.

Ok ... took a look. Yes, I think let's make the file name match. This class should hold any EF extensions ... not just this one. I'll take care of it this afternoon. Thanks for letting us know.


Set Log Level > All is not changing my output log level

Testing #164434
When i update the log level selecting the 'All' option, the log level output is not propagating to others.

But the logs are still showing the logs related to the past value.

/duplicate
https://github.com/microsoft/vscode/issues/164606

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Consider moving away from promStatsdExporterContainer

Description:

Describe the issue.

https://github.com/envoyproxy/gateway/pull/2729 added support for enabling scraping prometheus metrics using a sidecar container based off prom/statsd-exporter:v0.18.0 .
Pulling this image is tricky for air gapped clusters and adds another configuration point for syncing an extra image into a private registry
Can this be avoided if ratelimit introduced a prom endpoint natively, tracked with https://github.com/envoyproxy/ratelimit/issues/285
[optional Relevant Links:]

Any extra documentation required to understand the issue.

hey @shawnh2 interested in investigating the feasibility of this one ? There's a PR linked to the above issue, unsure what's missing to get it through the finish line
cc @zirain

I cannot recall all the details last time I tried(it's long ago), most likely because https://github.com/lyft/gostats/pull/119 didn't merge to master.

assign myself now, I will take a deeper look.

assign this to me

xref: https://github.com/envoyproxy/ratelimit/pull/681

looks like the ratelimit PR got merged !


reduce cpp

plan: move to http-types >= 0.11, wai to >= 3.2.2
@RyanGlScott what's your opinion on dropping GHC 7.6.3 and 7.8? because of prelude-compat it's not hard to support those. really the only annoyance is when i screw up and CI fails.

Dropping support for 7.6 and 7.8 sounds fine to me.

that's my preference too.
i'm working up toward one big majour release with a lot of improvements, so breaking changes make sense to do now.

@chessai we've now restricted CI to GHC >= 8.10 , what should we do with this ticket?

Big step forward in #333 . Closing this!


begin drafting Bound migration guide

This PR is a start of the migration guide to Bound<T>. I think there's still a lot more to be added here, but I wanted to push this anyway so that we can have a central place to start collecting migration notes as well as agree on the way we want to communicate this to users.

Superseded by #3707


Installing node-linq throws error

I am unable to install node-linq used in our source code. It was installing properly until now. Because of this error we are not able to deploy code to production as installations on production are failing. Error is due to coffee-script which node-linq is using internally. I have attached the screenshot of error. Please Help.

Facing the same issue.

The versions of coffee-script are pinned but apparently coffee-script force published this crap over existing versions? I'll look into it right now.

Downstream coffee-script issue - you can edit your package-lock to change the coffee-script version to the one in that issue to fix it temporarily. Sorry for the inconvenience, but not really much I can do - they published a broken package as a patch release so everyone is facing it atm.

I have created a pull request to specific version of coffee-script.


Setup code coverage

Move project directories to root dir and generate code coverage report with coverlet when Travis runs the tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bae642c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #9   +/-   ##
=========================================
  Coverage          ?   84.69%
=========================================
  Files             ?       40
  Lines             ?      882
  Branches          ?        0
=========================================
  Hits              ?      747
  Misses            ?      135
  Partials          ?        0

Impacted Files
Coverage Δ

ArkEcosystem.Crypto/Enums/TransactionTypes.cs
100% <ø> (ø)

...to/Transactions/Serializers/DelegateResignation.cs
0% <ø> (ø)

...ransactions/Builder/SecondSignatureRegistration.cs
100% <ø> (ø)

...sactions/Serializers/MultiSignatureRegistration.cs
100% <ø> (ø)

...actions/Serializers/SecondSignatureRegistration.cs
100% <ø> (ø)

...rypto/Transactions/Builder/DelegateRegistration.cs
100% <ø> (ø)

ArkEcosystem.Crypto/Utils/Message.cs
100% <ø> (ø)

ArkEcosystem.Crypto/Identities/PublicKey.cs
100% <ø> (ø)

...cosystem.Crypto/Transactions/Deserializers/Vote.cs
100% <ø> (ø)

...rkEcosystem.Crypto/Transactions/Builder/Builder.cs
100% <ø> (ø)

... and 30 more

Continue to review full report at Codecov.

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


Bagel Update 13.9

changelog
-removed mass scanner i forgor about
-added jetpacks to eva
-added a singular jetpack to engi for them to fight over
-added cargo shuttle stuff
-removed centcom headset (its useful now sorry drippers)
-added armory high sec door
-added flaps over the cargo blast doors (i will add flaps over cargo shuttle blast doors)
-skub

if anyone knows why yaml linter is failing something to do with nugets please let me know how to fix very confus


standard_zip_dir() tries to zip its own output directory

Thanks for this awesome tool!
I ran into an issue today trying to create a function; standard_zip_dir() creates its output zip file before building the list of files to zip. This appears (for me at least) to cause it to recurse until the zip file size limit is hit.
I propose excluding dest_zip when zipping:

Happy to submit a PR if that's helpful.

Hi, sorry for the delay - later in the day we will file a PR solving this, along with some refactors

recreate jira


codecleanup produces java compile error

this code is optimized to something that cannot be compiled
 String all  = "12345678901234567"
 // ...
 String other = all.substring(12, 13) + all.substring(14, 15);

will be "optimized" to
 other = all.charAt(12) + all.charAt(14);

which cannot be comiled: Wrong expression type String = char + char
This would work:
other = "" + all.charAt(12) + all.charAt(14);
String = String + char + char
I am using IntelliJ-ultimate 2020.2 with SaveActions 2.2.0 and jdk11

Hello @k3b, thank you for your bug report, but I'm unsure this is a Save Actions problem. AFAIK there's no quick fix implemented in Save Actions that will optimize from String other = all.substring(12, 13) + all.substring(14, 15); to other = all.charAt(12) + all.charAt(14);. Can you disable Save Actions and see if the problem persists? If it doesn't, reopen this issue.


[BT] Voting power display not correct (TBC: text to shown- related to showing Voting power component)

Context & versions
staging.sanchonet
Steps to reproduce
See loom video https://www.loom.com/share/e5b37153717c4c25aec7f069935fead5?sid=08859958-8fae-4420-8adb-698bee4352fb

Seems like the designs are appropriate to the screen you provided @pmbinapps is anything else wrong with that issue?

On current implementation:
Missing Show transaction button as the transactions are not working properly currently

DB-sync is updated but the transaction still fails due to the wrong drep registration status.
@jankun4 needs to correct the data fetching from db-sync

Not blocked anymore

TODO: To refine all the voting-power related issues

Voting power is only for any role that is registered as a voter: Dreps and Direct Voters. It's been agreed that VP is calculated at epoch boundaries.
The voting power 'chip' does not show if a user is not in a role that can vote, it displays and says 0 if a user can vote, but has no power in their 'account' because an epoch has not elapsed yet.

So that, it seems like the voting power chip issue has been fixed.
@pmbinapps could you please reverify the issue?

OK - Voting power chip on shown any more for Retired Direct Voter.

Closing this as it now works


Ansible Tower ServiceNow Integration Failed

ISSUE TYPE
Error dynamic inventory parsing.
COMPONENT NAME
API
SUMMARY
I'm basically following the steps mentioned in ansible servicenow integration official repo to integrate servicenow and get hosts from CMDB. I have all the dependencies installed ( python-configparser) and my ansible version is 2.7.0 and my tower version is 3.3.0 . I have the now.ini file in my /etc/ansible directory and also my now.py script. I have pasted the contents of now.py in my custom script contents in tower. Created an inventory and used the custom script in sources. I have all the credentials set up and also enabled .ini, script plugins in ansible.cfg file . When i'm trying to sync the inventory I'm facing the below error. Could anyone please help me out with resolving this problem?
Please find the error below
[WARNING]: * Failed to parse /tmp/awx_69_rRSdgQ/tmpZkM0LD with script plugin:
Inventory script (/tmp/awx_69_rRSdgQ/tmpZkM0LD) had an execution error:
Traceback (most recent call last): File "/tmp/awx_69_rRSdgQ/tmpZkM0LD", line
321, in main(sys.argv) File "/tmp/awx_69_rRSdgQ/tmpZkM0LD", line
316, in main inventory.generate() File "/tmp/awx_69_rRSdgQ/tmpZkM0LD",
line 205, in generate content = self._invoke('GET', path, None) File
"/tmp/awx_69_rRSdgQ/tmpZkM0LD", line 155, in invoke 'http': self.proxy,
'https': self.proxy}) File "/var/lib/awx/venv/ansible/lib/python2.7/site-
packages/requests/sessions.py", line 535, in get return self.request('GET',
url, **kwargs) File "/var/lib/awx/venv/ansible/lib/python2.7/site-
packages/requests/sessions.py", line 522, in request resp = self.send(prep,
**send_kwargs) File "/var/lib/awx/venv/ansible/lib/python2.7/site-
packages/requests/sessions.py", line 642, in send r = adapter.send(request,
**kwargs) File "/var/lib/awx/venv/ansible/lib/python2.7/site-
packages/requests/adapters.py", line 503, in send raise ConnectionError(e,
request=request) requests.exceptions.ConnectionError:
HTTPSConnectionPool(host='none', port=443): Max retries exceeded with url: //ap
i/now/table/cmdb_ci_server?sysparm_exclude_reference_link=true&sysparm_display
value=true&sysparm_fields=sys_class_name,fqdn,ip_address,name,host_name (Caused
by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConne
ction object at 0x7f9fc37e23d0>: Failed to establish a new connection: [Errno
-2] Name or service not known',))
[WARNING]: Unable to parse /tmp/awx_69_rRSdgQ/tmpZkM0LD as an inventory source
ERROR! No inventory was parsed, please check your configuration and options.
ADDITIONAL INFORMATION
Link to the now.py file.
https://github.com/ServiceNowITOM/ansible-sn-inventory/blob/master/now.py

While this may be an old post, it is presently not marked as resolved.
In reviewing the output, the following immediately "jumps out" in the STDERR (below).
In short, it appears you have a network block, and thus are unable to "reach" your SN instance.
If you have not figured it out already, check for connectivity to your "SN_INSTANCE_URL" on 443:TCP (from your TOWER node(s)).
Also, check if you have SN_PROXY set as an ENV variable within the TOWER UI.
File "/var/lib/awx/venv/ansible/lib/python2.7/site-packages/requests/adapters.py", line 503, in send raise ConnectionError(e,request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='none', port=443):
Max retries exceeded with url: //api/now/table/cmdb_ci_server?sysparm_exclude_reference_link=true&sysparm_displayvalue=true&sysparm_fields=sys_class_name,fqdn,ip_address,name,host_name
(Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f9fc37e23d0>: Failed to establish a new connection: [Errno-2] Name or service not known',))


Port ADT macros to scalameta/paradise

AFAIK the ADT macros do not depend on any compiler internals.
The Mirror use is also minor, there no def macros.
This seems like the prime candidate for really testing out the new scalameta macro annotations.
Theres 2 reasons:

Intellij support when maintaining meta is shocking, due to the use of the old macros.

It's basically just a memory hogging text editor.

If all macros can be ported to meta, meta support for 2.10 and SJS/SN should be easier.

Would require:

scalameta/common dependency on scalameta
scalameta/common dependency on paradise 3.0
possible dependency on scalameta/contrib

We may want to wait for a more stable semantic api and/or def macros. But I think incrementally moving to meta macros is better then doing it all at once

Some examples... Basically all of this prevents any symbol lookup, or refactoring helpers from working.

I had a quick go at this but I don't see an easy way to solve the cyclic dependency between common and scalameta.
scalameta depends on common, so of course we can't add a dependsOn(scalameta) to the common project.
I wondered if we could naively add a dependency on "org.scalameta" %% "scalameta" % "1.6.0" and everything would just work. But unsurprisingly that was also a non-starter. It means we have old and new versions of duplicate scala.meta classes on the classpath, so we get compile errors like this:
[error] /Users/chris/code/scalameta/scalameta/common/src/main/scala/scala/meta/internal/ast/Liftables.scala:37: value q is not a member of StringContext
[error]       q"""
[error]       ^
[error] /Users/chris/code/scalameta/scalameta/common/src/main/scala/scala/meta/internal/ast/Liftables.scala:56: type mismatch;
[error]  found   : StringContext
[error]  required: ?{def q: ?}
[error] Note that implicit conversions are not applicable because they are ambiguous:
[error]  both method XtensionQuasiquoteTerm in trait Api of type (ctx: StringContext)Api.this.XtensionQuasiquoteTerm
[error]  and method Quasiquote in trait Quasiquotes of type (ctx: StringContext)LiftableMacros.this.c.universe.Quasiquote
[error]  are possible conversion functions from StringContext to ?{def q: ?}
[error]         val init = q"""c.universe.Ident(c.universe.TermName("_root_"))""": Tree

Any ideas on how to work around this bootstrapping problem? If we can solve this, it looks like rewriting the macros should be reasonably simple. As far as I could see they are entirely syntactic.

What happens if you make the meta dependency compile time only? Does that actually work?

No, I tried making it provided scope but it didn't help

Do you have a branch at all? I would like to experiment.

There's not much to it, but here it is https://github.com/cb372/scalameta/tree/port-adt-macro-annotations

@cb372 I believe the problem is the provided exports the dependency as a runtime dependency. What we really want is a compile time only dependency, which SBT does not support out of the box (As it mimics maven).
Also, scalameta/paradise can expand old macros aswell as new ones, so instead of the enable macros configuration we will need to have a custom one. We will not be able to have scalamacros/paradise and scalameta/paradise as plugins at the same time.
See this SO post: http://stackoverflow.com/questions/21515325/add-a-compile-time-only-dependency-in-sbt
Are you still willing to approach this?

Just a thought, maybe we could use the coursier shading plugin to bootstrap https://github.com/coursier/coursier/releases/tag/v1.0.0-RC1

We're on the verge of releasing 2.0, and afterwards we plan to start being really conservative about breaking changes. The matter discussed in the ticket can probably be improved, but there's not enough interest to justify potential breakages.


Parjanya/pancake sort

Pancake Sort Added
Checklist

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

Thanks!
@all-contributors please add @parjanyaacoder for code.


AttributeError: module 'torch.fft' has no attribute 'rfft2'

x2 = torch.fft.rfft2(x2, dim=(2, 3), norm='ortho')
AttributeError: module 'torch.fft' has no attribute 'rfft2'

Hi @1648027181, thanks for your interest in our work. To use torch.fft.rfft2, you need to upgrade PyTorch to 1.8 or a later version. Besides, it is necessary to import both torch and torch.fft like https://github.com/raoyongming/HorNet/blob/d3ae1bbca34da6e1c2ff98db3076f1eacd502117/hornet.py#L2-L7.


D435i Not connecting to RPI 4

Hi there,
I am currently working with a D435i camera on an RPI4. I am running the newest version of Raspberry Pi OS on the Pi and have successfully installed the SDK. I hooked the camera up to my 3.0 USB port, however when I run the python-tutorial-1-depth.py test I get a "No device connected" error. If I run lsusb command, the device is found and when I plug the camera in and run a dmesg command, it gives the following output:

*Note the code compiles, but it does not connect. Also, I have used this camera and USB cord to connect to a Linux VM and know that it is possible to connect. Am I missing something in how the RPI accesses the pipeline?

Hi @HFeustel   If you are using a Linux VM with librealsense then you may encounter problems due to difficulty in interfacing with USB 3.   If you do need to install librealsense on a VM, Intel's documentation suggests using VMWare Workstation Player.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation.md#linux-ubuntu-installation

Hi @HFeustel   If you are using a Linux VM with librealsense then you may encounter problems due to difficulty in interfacing with USB 3.   If you do need to install librealsense on a VM, Intel's documentation suggests using VMWare Workstation Player.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation.md#linux-ubuntu-installation

Hi @MartyG-RealSense thanks for the reply!
I have already done that and I am fine when connecting to the camera. For more context, I am using the RealSense camera for a robot to do image detection. For testing, I used my Windows machine with a linux VM in VMWare and developed my code and everything worked. Now, I am going to copy my code to an RPI because that is the computer on the robot. However, when I compiled the realsense library and ran a test file, it is saying I am not connected to a device even though the camera is plugged in. The image I put in the original post was the response to the dmesg command on the RPI to see if the machine was connecting to camera (which it is), but my code is saying that it isn't.

Hi @MartyG-RealSense thanks for the reply!
I have already done that and I am fine when connecting to the camera. For more context, I am using the RealSense camera for a robot to do image detection. For testing, I used my Windows machine with a linux VM in VMWare and developed my code and everything worked. Now, I am going to copy my code to an RPI because that is the computer on the robot. However, when I compiled the realsense library and ran a test file, it is saying I am not connected to a device even though the camera is plugged in. The image I put in the original post was the response to the dmesg command on the RPI to see if the machine was connecting to camera (which it is), but my code is saying that it isn't.

@MartyG-RealSense,
I deleted the library and recompiled it on the RPI and it works now. The only tweak I made was that when I copied the udev rules, I made sure the camera was unplugged this time. Last time, I set the udev rules with the camera plugged in.

@MartyG-RealSense,
I deleted the library and recompiled it on the RPI and it works now. The only tweak I made was that when I copied the udev rules, I made sure the camera was unplugged this time. Last time, I set the udev rules with the camera plugged in.

Thanks very much for the update @HFeustel  - great news that you succeeded!  Thanks for sharing the solution that you used.

Thanks very much for the update @HFeustel  - great news that you succeeded!  Thanks for sharing the solution that you used.


Cloning the repository and using make fails

Doing a fresh clone of the repository fails with:
 make
cargo build
    Updating crates.io index
   Compiling containerd-shim-wasm v0.1.0 (/home/jstur/projects/runwasi/crates/containerd-shim-wasm)
   Compiling lock_api v0.4.9
   Compiling parking_lot_core v0.9.6
   Compiling wasmedge-sys v0.12.2
   Compiling wasmedge-types v0.3.1
   Compiling oci-tar-builder v0.1.0 (/home/jstur/projects/runwasi/crates/oci-tar-builder)
The following warnings were emitted during compilation:

warning: [wasmedge-sys] Failed to locate lib_dir, include_dir, or header.

error: failed to run custom build command for `wasmedge-sys v0.12.2`

Caused by:
  process didn't exit successfully: `/home/jstur/projects/runwasi/target/debug/build/wasmedge-sys-380b7131222322c7/build-script-build` (exit status: 101)
  --- stdout
  cargo:warning=[wasmedge-sys] Failed to locate lib_dir, include_dir, or header.

  --- stderr
  thread 'main' panicked at '[wasmedge-sys] Failed to locate the required header and/or library file. Please reference the link: https://wasmedge.org/book/en/embed/rust.html', /home/jstur/.cargo/registry/src/github.com-1ecc6299db9ec823/wasmedge-sys-0.12.2/build.rs:30:25
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
make: *** [Makefile:19: build] Error 101

Should wasmedge be behind a feature flag in the crate or should we add something to the makefile to install the correct dependencies?

Another option would be a build container or use cross-rs.

After installing wasmedge with curl -sSf https://raw.githubusercontent.com/WasmEdge/WasmEdge/master/utils/install.sh | bash
I am getting:
 make
cargo build
   Compiling wasmedge-sys v0.12.2
The following warnings were emitted during compilation:

warning: [wasmedge-sys] libwasmedge found in /home/jstur/.wasmedge/lib

error: failed to run custom build command for `wasmedge-sys v0.12.2`

Caused by:
  process didn't exit successfully: `/home/jstur/projects/runwasi/target/debug/build/wasmedge-sys-380b7131222322c7/build-script-build` (exit status: 101)
  --- stdout
  cargo:warning=[wasmedge-sys] libwasmedge found in /home/jstur/.wasmedge/lib

  --- stderr
  thread 'main' panicked at 'Unable to find libclang: "couldn't find any valid shared libraries matching: ['libclang.so', 'libclang-*.so', 'libclang.so.*', 'libclang-*.so.*'], set the `LIBCLANG_PATH` environment variable to a path where one of these files can be found (invalid: [])"', /home/jstur/.cargo/registry/src/github.com-1ecc6299db9ec823/bindgen-0.61.0/./lib.rs:2276:31
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
make: *** [Makefile:19: build] Error 101

was able to compile after sudo apt-get -y install libclang-15-dev

found docs on the readme:
$ sudo -E sh -c 'echo "$HOME/.wasmedge/lib" > /etc/ld.so.conf.d/libwasmedge.conf'
$ sudo ldconfig

and tests pass now

Ok, so there is a make target that helps with install and setup: https://github.com/containerd/runwasi/blob/a2f86e43e8df4efece9ec4b3f9460421f450c8fc/Makefile#L63-L65

Ok, so there is a make target that helps with install and setup:
https://github.com/containerd/runwasi/blob/a2f86e43e8df4efece9ec4b3f9460421f450c8fc/Makefile#L63-L65

You may also need to set enviroment varibles after make bin/wamsedge:
export WASMEDGE_INCLUDE_DIR=$(PWD)/bin/wasmedge/include && \
export WASMEDGE_LIB_DIR=$(PWD)/bin/wasmedge/lib

It will probably be fixed by #103 if you followed the README file instructions to install WasmEdge.

@jsturtevant I think we can close this one since a lot has changed.


Cleanup - Consistent code style

As per the contributing guidelines, we should adhere to blue-style.
However, some parts of the code seems to not following the guideline, e.g.
https://github.com/alan-turing-institute/MLJModels.jl/blob/885bcc0a33a86fe2ee46340f20a9d99c351b1251/src/builtins/Transformers.jl#L252-L253
Which according to bluestyle, this should be
function MLJBase.inverse_transform(
    transformer::UnivariateDiscretizer, result , k::Integer
)

If the contributors are expected to follow the guidelines, the code will be inconsistent. For that reason, I think we might need to do clean up at some stage to align with the style.
Maybe the easiest is to run JuliaFormatter? With this, the future contributors can easily run the formatter as well. Although, I think at some parts the formatter still doesn't satisfy BlueStyle? (See this issue)

Ha I didn't know of JuliaFormatter, it could be a good idea, do you have experience with it?
Also generally in terms of formatting, I think it would be great to generally have a consistent style but I think Anthony would agree with me that we also don't want to be super strict with it as long as the code is readable.
But otherwise I'm for it, I'll be happy to review a PR, possibly I would suggest to do  this in chunks starting small to begin with to see if there's anything we really don't like and adjusting accordingly 😄
Thanks!

@tlienart I just tried it last week so haven't had a lot of experience with it. However, so far this seems promising given this is width-sensitive formatter and inspired by other formatters, such as black (python), which I used a lot in the past.
For sure, I can submit a PR starting with formatting 1 file and we can look how's the result. :smile:


scale out the table on sysbench for intertable parallelism using SF

removed sysbench. added scale out on YCSB.


headerStyle option is not being applied to spacing cell when grouping

Describe the bug
Styles provided with the headerStyles option are not being applied to the spacing cell when using grouping.
Demo
Demo
To Reproduce
Steps to reproduce the behavior:

Make a table with that uses the headerStyle option.
Make sure the table is using grouping.
Inspect the element to see if styles are being applied to the left-most cell in the header.

Expected behavior
Styles provided with the headerStyle option should be applied to the cell.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version:

Additional Notes
I have found the issue and have fixed it in a fork. I'll submit a PR for it.

#284 Thank you for your contribution!