wiremock return wrong content-length in response for 200 status code

We are using wiremock-jre8-standalone 2.21.0. We start wiremock as a standalone server.
We have a mapping file like below.
{   "request": {     "method": "POST",     "url": "/api/work/123/start"   },   "response": {     "status": 200   } }
When the client sends a request to this mapping, it gets the below response.
Matched-Stub-Id →1868d9cc-3201-411d-aa8f-e31711123551 Vary →Accept-Encoding, User-Agent Content-Encoding →gzip Content-Length →20 Server →Jetty(9.4.z-SNAPSHOT)
The Content-Length = 20 is obvious wrong since this response has no body. It causes the jersey log in the client code throws exception since the log code tries to read more bytes from the response stream but there are no more content.
In this case, 204 should be more appropriate status code since it means no content. If we switch to 204 code, it will return with out Content-Length and client code will work fine. We made this change to fix the issue. But this still looks like a bug with 200 status code if there is no body.

Thanks for reporting. Seems to be specific to gzipped responses that would normally be chunk encoded.
You can work around the problem by upgrading to WireMock 2.22.0 and disabling chunked encoding in the options.

Thanks. Actually we didn't use gzip compression in our client code. The response I posted is when I test this in POSTMAN which uses gzip by default. I checked my client log. The transfer-Encoding is Chunked.

One more thing I want to mention is that this issue only occurs on some platform. I run the same client and wiremock server on windows 10 without any issue. But it fails on macos and linux container. So it may be related to *nix os family only.

OK, so in some cases you've seen this happen without a gzip response being sent?

yes, in all the real cases we we don't use gzip compression. The only time using gzip compression is when I tested the mapping directly using POSTMAN tool today. Below is the log when I run our bdd test in a windows machine. I can't get the response log from the failed deployment since it throws exception when try to log.
36 > POST http://localhost:9091/api/work/123/start 36 > Accept: application/json 36 > Content-Type: application/json {} 36 < 200 36 < Matched-Stub-Id: 7c981af3-757a-43f3-8ce8-cfc2b9a8dfa5 36 < Server: Jetty(9.4.z-SNAPSHOT) 36 < Transfer-Encoding: chunked 36 < Vary: Accept-Encoding, User-Agent

I'm going to close this because, as far as I can tell, this is working OK in 2.23.2-jre8.
If you find you're still having this problem with the latest version, please post the info required to replicate (or the full test source ideally) and I'll take another look.


refactor: transition to flamework

While certainly stupidly and utterly massive commit, I've managed to transition to Flamework will keeping nearly all features.
There are still a few things I want to finish up, but it's about on par with what it was previously.

Before merging, I’d like to change a few more things.

[ ] Add a GUI for the user balance
[ ] Add a GUI for the ore count
[ ] Introduce ore count limitations
[ ] Refactor item placement controllers to make it easier to understand
[ ] Convert remotes to take in arrays for bulk processing

Since ‘@rbxts/netbuilder’ doesn’t want to serialize things inside of arrays/serialize recursively, I’m going to just roll my own simple networking library to add support for serialization.

Y'know what, I'm practically using this as a new main branch. I might as well merge it.


Fix typo in contributing

Summary
오타 수정
리뷰 참고 사항
없음
Glossary
없음

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

기여해 주셔서 감사합니다 👯


use C struct for accessing Windows APIs

Using a pointer of Go struct as an argument of Windows API causes panic. To fix this, C struct should be used.
This pull request provides a helper method to retrieve a value with C struct of Windows API, and changes to use it, not to use methods with Go struct.
ref. https://github.com/lxn/win/issues/12

Great!

:+1:


[FEAT]: Add git to alpine image

Clear and concise description of the problem
Hi,
I am trying to execute chromatic with turbosnap in alpine image but I can't because it wouldn't have git, so if would be great if this image can has git
Suggested solution
No response
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

@RodrigoTomeES Is using a multi-stage build not enough? Can you push and share a minimum reproducible repository for me, so I can understand more of your requirements?
Example:
FROM imbios/bun-node:latest-hydrogen-alpine
# build stage here

FROM alpine/git:user
# runtime stage here

@ImBIOS is for a bitbcuket pipeline so I can't not build an image for the pipeline I can only use it, right now I am using the debian image because it has git. My code is something like this:
pipelines:
  branches:
    master:
      - step:
        name: deploy chromatic
        image: imbios/bun-node:latest-18-debian
        script:
          - bun install
          - bunx chromatic

@RodrigoTomeES Here's the good news:

https://github.com/ImBIOS/bun-node/actions/runs/8307579135

@ImBIOS thanks you!!

@ImBIOS I already using in our pipelines ❤  and all works perfect


MXNET-1302 Exclude commons-codec and commons-io from assembled JAR

Consumers of MXNet are exposed to the commons-codec and commons-io library versions MXNet uses internally. This change excludes those JARs from the JARs created with make scalapkg.
Description
This PR removes commons-codec and commons-io from the assembled JAR and adds both packages as dependencies in deploy.xml
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] commons-io and commons-codec remove from assembled JAR
[x] commons-io and commons-codec added as dependencies to deploy.xml

Comments
My Maven knowledge is sparse so I may have missed something.
For testing:

The JAR no longer contains org.apache.commons classes
make scalapkg && make scalaunittests && make scalaintegrationtests passed on my Ubuntu 18.04 machine (JDK 8).

Originally reported in #13929

@gordon1992 could you resolve branch conflicts?
@frankfliu for review

@mxnet-label-bot update [Maven, Scala, pr-awaiting-response]

I shall

resolve the merge conflict
remove the unnecessary commons dependencies in submodules
and update this pull request.

Thank you for looking at this

@mxnet-label-bot remove pr-awaiting-response

@mxnet-label-bot update [Maven, pr-awaiting-review]

@mxnet-label-bot update [Scala, Maven, pr-awaiting-response]

I see the mxnet-validation/website has failed due to
./core/src/main/scala/org/apache/mxnet/Serializer.scala:24: error: object commons is not a member of package org.apache

import org.apache.commons.codec.binary.Base64

I'll fix that before placing pull request back to pr-awaiting-review

@mxnet-label-bot update [Maven, Scala, pr-awaiting-review]

Please take a look at the website crash, there seemed to be some problem with it

@mxnet-label-bot update [Maven, Scala, pr-awaiting-review]

Please take a look at the website crash, there seemed to be some problem with it

Fixed. All other CR comments addressed.
Thanks for your time.

Thanks for the approvals.
I note the failing of windows-gpu, looks like it was aborted. I'm not sure how to retry this check so I have a green tick which I assume is a requirement prior to merging.  Any advice welcomed.
Thanks,
Gordon

@gordon1992 The path forward is to pull --rebase with master since we decided to disable the Windows test for now

Ah yes, I see the commit disabling the Windows test. My fork is now up to date with that change.
Thanks,
Gordon

@mxnet-label-bot update [Maven, Scala, pr-awaiting-merge)


Add files via upload

.

@selimyalcin I am sorry, the domain ad.just.edu.jo is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Correcting Information on Game Modes

I had incorrectly assumed that the set of valid Atari game modes is of the form {0, ..., n}. I was corrected here: https://github.com/Farama-Foundation/gym-docs/pull/39#issuecomment-1030656646 . Thus, I am raising this pull request to correct the mistake
I implemented a script to automatically generate a table of valid game modes, valid difficulties and default game modes (default difficulty seems to be 0 everywhere). The resulting table is gigantic, so we might not want to include it directly in index.md, unless there is some way to collapse it in the HTML page. Nevertheless, I have added it for now for reference. What's your opinion on where this information should go?
I still need to go through the individual environment pages and update the information there.

I had incorrectly assumed that the set of valid Atari game modes is of the form {0, ..., n}. I was corrected here: #39 (comment) . Thus, I am raising this pull request to correct the mistake
I implemented a script to automatically generate a table of valid game modes, valid difficulties and default game modes (default difficulty seems to be 0 everywhere). The resulting table is gigantic, so we might not want to include it directly in index.md, unless there is some way to collapse it in the HTML page. Nevertheless, I have added it for now for reference. What's your opinion on where this information should go?
I still need to go through the individual environment pages and update the information there.

The index page is fine.


Suggestion: option to skip creating release instead of writing "Maintenance release" when all commits are excluded

There should be a flag to tell it not to create the release when there are no commits to display.
It will be useful if releases will only be created when certain types of commits are made.

Sounds good! PR welcome


In which gstreamer version can I find the bundle-policy property of Webrtcbin?

In the latest webrtc-sendrecv.c, I see the bundle-policy being added to the pipeline. In which version of gstreamer I could get this support? I have tried till 1.14.4 but it doesn't look like it has been added. When I tried building from master branch I get errors in gst-uninstalled. Any suggestions to get the bundle policy patch applied?

@aswathisivan ,
Bundle works on master.

Please use the GStreamer mailing list or IRC for support questions.
Bundle is in GStreamer >1.16


CLI: Add checksum calculation to avoid unecessary processing when generating code

Append checksum on the generated source code based on the input schema file + generator options (namespace + client name). This checksum is initial read on subsequent generation. If the input graphql schema and generator options hasn't changed, then the process can be aborted since the source code is up to date.
Purpose
I think the ZeroQL is a good framework to use as typed client for our backend tests. We output the schema file from our backend and then our xunit test project compiles the ZeroQL client as part of pre build step. Since ZeroQL.CLI tool is now baked into our build pipeline, we want optimise the performance. The best approach is to skip source generation when the generated code is up to date with the existing schema.
Benchmarks

Method
Mean
Error
StdDev

GenerateWithoutChecksumOptimization
7,640.75 us
151.703 us
141.903 us

GenerateWithChecksumOptimization
83.76 us
1.624 us
1.356 us

CLI
Added new option "Force" to skip checksum check and output generate source code (if this is somehow needed)

Overall nice addition! Thanks!

I have updated the CLITests that checks both the outputted message but also checks that the write timestamp is also correct on the physical source code file. I also decided to add the ZeroQL.CLI version number into the checksum since a new version might produce different source code than earlier versions


[Feature Request] Inspect a project's details without editing or starting.

Be able to see the path, summary, commands, and more without starting or editing the project

Created the Info option.
smp info <project_title>
Additionally, project details are displayed when a project is chosen to edit.


@conda-forge-admin, please add user @ocefpaf

Additional comment:
@conda-forge-admin, please add user @ocefpaf

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @ocefpaf in conda-forge/gmprocess-feedstock#75.


SWITCHYARD-2694 Tough uninstallation of atom and rss quickstarts

ok to test

ok to test

Can one of the admins verify this patch?

ok to test

ok to test

ok to test

ok to test

ok to test

ok to test

ok to test

ok to test

ok to test


Add support for different cryptographic backends

The idea is to provide a common interface to do elliptic curve cryptography, and choose the "backend" at runtime based on how ecpy was installed.
This will allow us to centralise all cryptographic code in Python, used internally at Ledger, instead of having to maintain custom wrappers based on the use-case. Some examples are given below:

ledgerblue => uses libsecp256k1 if present, else falls back to ecpy.
ledgerctl => uses python-ecdsa for elliptic curve cryptography.
some internal Ledger package => uses openssl for elliptic curve cryptography.

I think this also adds value to the open source community, by providing a generic wrapper for all elliptic curve operations.
Here's how the installation would look like:
Using pure Python primitives
pip install ecpy

Using libsecp256k1 for the said curve, and pure Python for rest
pip install ecpy[libsecp256k1]

Using libsodium
pip install ecpy[nacl]

Using openssl
pip install ecpy[cryptography]

Let me know if this is outside the scope of ECPy, and not something that we want to do.

The initial goal of ECPy is to provide pure python ECC.
Thus, adding various native dependencies is not really in the right way.
Also, note that ECPy is not a ledger dedicated library  but a more general purpose lib.
Nonetheless we can discuss the  needs of Ledger and why they needs libsodium/openssl.... and why dont use pure python everywhere.
I invite you to PM me in order to discuss

I agree with you. I realised that ecpy shows up as the first google search result for “pure python ecc library”, and therefore the original purpose of the package should not change. Additionally, ecpy supports more curves than most alternatives out there, so maybe the focus could be on readability and curve support.
Here are the curves supported by competing libraries:

starkbank/ecdsa-python: https://github.com/warner/python-ecdsa/blob/333ee3feb1dfc6797db7a83d221e5a3a9fafdc3f/src/ecdsa/curves.py#L13-L27
warner/python-ecdsa: https://github.com/warner/python-ecdsa/blob/333ee3feb1dfc6797db7a83d221e5a3a9fafdc3f/src/ecdsa/curves.py#L13-L27
justdvnsh/elliptic-py: https://github.com/justdvnsh/elliptic-py/blob/1740cb72f1a43d0aac67c5b41ddf1aeef40ead1d/src/ellipticpy/curves/curves.py#L47
AntonKueltz/fastecdsa: https://github.com/AntonKueltz/fastecdsa/blob/master/fastecdsa/curve.py

Closing this issue.


(bug) capture setting not respected

Describe the bug
When running
  it.only('looks right', () => {
    cy.url().should(($url) => {
      expect(new URL($url).toString()).to.equal(new URL(config.dashboardUrl).toString());
    });
    cy.document().toMatchImageSnapshot({ capture: 'runner' });
  });

the option I provide to capture does nothing.  It will always use the fullScreen setting, no matter what I do.
To Reproduce
Steps to reproduce the behavior:

Write a simple toMatchImageSnapshot test
Try changing the capture setting
notice nothing changes

Expected behavior
It should respect the capture setting
Desktop (please complete the following information):

OS: macOS
Cypress Version: 6.5.0

Additional context
Another way to show this is to do something like
  it.only('looks right', () => {
    cy.url().should(($url) => {
      expect(new URL($url).toString()).to.equal(new URL(config.dashboardUrl).toString());
    });
    cy.screenshot({ capture: 'runner' });
    cy.document().toMatchImageSnapshot({ capture: 'runner' });
  });

As you start changing the capture settings to both screenshot and toMatchImageSnapshot, you'll see that the screenshot will update the image to look very different each time, but toMatchImageSnapshot looks identical

Looking into this more, it gets even weirder.  It looks like this bug only happens the FIRST time you open cypress...sometimes.
If I have a line like
    cy.document().toMatchImageSnapshot({ capture: 'runner' });

Then about 50% of the time it will use fullScreen, and the other 50% it will use viewport.  I still cannot get runner to work.

I'm not totally sure, but I think I figured it out (mostly).
Looks like
    cy.document().toMatchImageSnapshot({ screenshotConfig: { capture: 'viewport' } });

works.
Related to #152 and #153

Related to those, please update the types because


Remove version: 7S2P.Effie version 3.11.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://download.effie.pro/effie/effieapp_setup_3.11.3.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 896)


Glob Imports missing new files

It seems that when doing a global folder import, new files added to that folder tend to be missed during compilation. Removing tmp/cache/assets and restarting the server helps, however we also experienced this when deploying to heroku.
For now, the way we solve it is to make any change to a file containing global import (adding extra space), so it forces file to be recompiled. Any better way to do this?

Seems to be a duplicate of https://github.com/rails/sass-rails/issues/189


Add option to create test cases if not present

Test cases should not be created by default. Test case creation should be performed based on the existence of a runtime option.

Fixed in #5
Set force_case reporter option to false to disable this since it is true by default.


Performance metrics

99% of time of the witness generation is spent in RPC calls. At least when the node is remote. Hopefully when the node is local it is very fast, but still to be tested.

I feel like we might need to refine it one more tim with something like https://crates.io/crates/tracing cuz now how we time things are very elementary and creates lots of extra code.

I'm not sure how that would save much code. There will be places where we want to start measuring stuff so we need some code there, and then some code at the place we want to stop measuring. Which the current code does quite minimally. So don't really see how you can significantly reduce the amount of code except some very small helper functions to combine stop + print time or something like that.


CPU Arch: added helper script for simple string replacements

@ko-be @bossjones @adobejmong

@tnaroska have a look in its pseudo final form now :)

thanks for all your comments! for the latest commit...Travis ARM version is simply failing to spin up or connect to git. I'll keep retrying the job until it works...https://app.travis-ci.com/github/behance/docker-base/jobs/551065422

finally, passing tests! merging.

Great! What do we need to do to get these as multiarch images deployed to dockerhub?
Still seems intel only:
https://hub.docker.com/r/behance/docker-base/tags

@tnaroska working on that separately. if you are interested in swapping travis for github actions to do a publish?


[tool_crash] ProcessException: No such file or directory Command: /Applications/Google Chrome.app/Contents/MacOS/Google Chrome, OS error code: 2

Command
flutter run

Steps to Reproduce

...
...
...

Logs
ProcessException: No such file or directory Command: /Applications/Google Chrome.app/Contents/MacOS/Google Chrome, OS error code: 2
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:401:33)
#1      Process.start (dart:io-patch/process_patch.dart:38:20)
#2      Chrome._startProcess (package:browser_launcher/src/chrome.dart:145:26)
#3      Chrome.start (package:browser_launcher/src/chrome.dart:137:18)
#4      FlutterResidentDevtoolsHandler._launchDevToolsForDevices (package:flutter_tools/src/resident_devtools_handler.dart:124:24)
#5      FlutterResidentDevtoolsHandler.launchDevToolsInBrowser (package:flutter_tools/src/resident_devtools_handler.dart:112:7)
#6      TerminalHandler._commonTerminalInputHandler (package:flutter_tools/src/resident_runner.dart:1662:55)
#7      TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1680:13)
#8      _rootRunUnary (dart:async/zone.dart:1434:47)
#9      _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#10     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#11     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#12     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#13     _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:378:25)
#14     _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:244:5)
#15     _AsBroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:475:11)
#16     _rootRunUnary (dart:async/zone.dart:1434:47)
#17     _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#18     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#19     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#20     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#21     _SinkTransformerStreamSubscription._add (dart:async/stream_transformers.dart:63:11)
#22     _EventSinkWrapper.add (dart:async/stream_transformers.dart:13:11)
#23     _StringAdapterSink.add (dart:convert/string_conversion.dart:238:11)
#24     _StringAdapterSink.addSlice (dart:convert/string_conversion.dart:243:7)

[✓] Flutter (Channel master, 2.6.0-12.0.pre.806, on macOS 11.6.1 20G224 darwin-x64, locale en)
    • Flutter version 2.6.0-12.0.pre.806 at /Users/vungo/tools/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fa100d23d3 (7 weeks ago), 2021-11-23 05:08:07 -0500
    • Engine revision 2cc81657bc
    • Dart version 2.16.0 (build 2.16.0-36.0.dev)
    • DevTools version 2.8.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/vungo/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.62.1)
    • VS Code at /Users/vungo/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (2 available)
    • iPhone 12 Pro Max (mobile) • D6D5D367-E6E2-4906-915D-BF1DECB11074 • ios        • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                • darwin-x64 • macOS 11.6.1 20G224 darwin-x64

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 4cc385b4b84ac2f816d939a49ea1f328c4e0b48e
Plugins
amplify_analytics_pinpoint-0.2.10
amplify_auth_cognito-0.2.10
amplify_core-0.2.10
amplify_flutter-0.2.10
camera-0.9.4+1
camera_web-0.2.1+1
device_info-2.0.2
flutter_plugin_android_lifecycle-2.0.3
flutter_qr_reader-1.0.5
image_picker-0.8.4+2
image_picker_for_web-2.1.3
local_auth-1.1.8
OneSignal-Flutter-SDK-952e78c1c6f7ace65082914f4e90f6667b6b382c
path_provider-2.0.5
path_provider_linux-2.1.0
path_provider_macos-2.0.2
path_provider_windows-2.0.3
qr_code_scanner-0.5.2
shared_preferences-2.0.8
shared_preferences_linux-2.0.2
shared_preferences_macos-2.0.2
shared_preferences_web-2.0.2
shared_preferences_windows-2.0.2

Hi @TuanVu1905, can you install Google Chrome to see whether or not you still experience this issue?


I think this cheat has been detected as my and one of friends account were VAC banned.

Osiris is not detected, more likely your injector is.
Inject Osiris using MemJect and use VAC-Bypass for extra precaution so you'll be fine.

Ok, thank you so much.

Osiris is detected!


Tracking: Introduce per-module error type for frontend

As mentioned in #1367.
Tracking:

[x] #3247
[x] https://github.com/singularity-data/risingwave/issues/2035
[ ] Introduce HandleError Type
[ ] Introduce PlanError Type
[x] Introduce ScheduleError Type
[x] #3082
[x] #3089

#2035 can be tracked in the issue

How about OptimizerError?

How about OptimizeError?

Maybe an enum of PlanError? It seems the optimization rules return Option instead of Result, similar to Calcite returning a Null in the event of a failure of rule application.
But we will see whether it is necessary later and we can add it then.

We do need it to check whether apply can be unnested or not.
https://github.com/singularity-data/risingwave/pull/2187#discussion_r861475292

The reason our current optimizer returns Option is for writting code easily........

After #14953, I believe there's a stronger motivation for addressing this issue.


Support custom Doctrine Type for IDs in Nested Set

Attempt to resolve #1978.

Duh...  didn't run the tests.  BRB


Using JFXDialog without Flow API

I only need to display an error message or an information message. Including Flow Api in my project just for this would be overkill. Is there any alternate way to do it? Like manually passing instances of stackpane and region?

Please note that the Flow API is used in JFoenix Demo, and all of JFoenix controls are not dependent on any 3rd party api/jar.
you can use JFXDialog directly in your code using:
dialog = new JFXDialog(dialogContainer, content, transitionType)


Fixes + documentation to run Coconet code with Python 3

These changes were required for me to run the sample_bazel.sh and train_bazel.sh scripts in the Coconet project, using Python 3, and I tried to improve the README to make it clearer to someone new to the project.
I haven't contributed to this project before, but I thought it would be helpful to submit a PR! Happy to incorporate any comments/feedback.
Testing notes:

Follow the usage instructions in the README in this PR.
If you have a pip-installed version of magenta, set your $PYTHONPATH to the location of the magenta directory from this branch, so that the local version of magenta (containing these changes) takes precedence when magenta is imported.
The pre-trained model downloaded from here requires a fix in order to work. The download contains a checkpoint file named checkpoint, which needs to point to the checkpoint binaries in that directory. Change the checkpoint file to contain only the line model_checkpoint_path: "best_model.ckpt".

To Do:

[ ] This doesn't block this PR because it only uses the .npz file, but the pickled files in the repo containing the full JSB chorale datasets, can't be unpickled using Python 3 without some trickery, because apparently Python 3 doesn't like to unpickle files that Python 2 pickled. Would be helpful to submit a separate PR containing a Python 3-compatible version of those files.

I signed it!

Thanks for the work so far! This has been a problem for us too currently.
Is there anything left to do or could this be pulled now with the requested change?


Markdown updates for 3.0.4

We need to handle the changes to anchor links and headings (see #3548  for context). We also have #3596 to add tooling, which I will work on after we've done this initial cleanup.
The commits tell their own story, but this is all very repeatable and hopefully transferrable between branches. Shout out to @handrews who got me started with the anchor/link rewriting script.
The process goes like this:

Run prettier for formatting

prettier --write --single-quote 3.0.4.md

Run markdownlint to fix whatever it can

markdownlint --fix 3.0.4.md
Configuration file .markdownlint.yaml:      MD007:
        indent: 2

      MD012: false # allow blank lines

      MD013:
        line_length: 800
        tables: false

      MD024: false # duplicate headings
      MD033: false # inline HTML

Manually fix additional markdownlint problems

heading levels aren't continuous MD001
code fences need a language MD040
table has the wrong number of cells MD056

Take out the table of contents and its comments, replace with a single

Run a magical one-off script to update/fix/rewrite/remove all our anchors and internal links. Basic idea:

make all our anchor links kebab-case
update all other internal document links
remove the ones that are in headings
add a lookup/override to fix things that either were inflected
differently or we were using an anchor link that didn't match the
title
(script is in comment)
use markdownlint again to check that this all worked because it can check internal links

Put the TOC in with all the new structure and links using markdown-toc

markdown-toc --maxdepth 4 -i 3.0.4.md

It's 850 lines of change, I don't know how we're going to review it, but take a look!

Super special python script:
from sys import argv
from pathlib import Path
import re

# this script tries to inflect the old links, some are just missing and we need to use the title's version instead
updates = {}
updates["revision-history"] = "appendix-a-revision-history"
updates["data-type-conversion"] = "appendix-b-data-type-conversion"
updates["using-r-f-c6570-implementations"] = "appendix-c-using-rfc6570-implementations"
updates["serializing-headers-and-cookies"] = "appendix-d-serializing-headers-and-cookies"
updates["percent-encoding-and-form-media-types"] = "appendix-e-percent-encoding-and-form-media-types"
updates["document-structure"] = "openapi-description-structure"
updates["oas-object"] = "openapi-object"
updates["components-security-schemes"] = "security-scheme-object"
updates["schema-composition"] = "composition-and-inheritance-polymorphism"
updates["http-codes"] = "http-status-codes"
updates["oas-document"] = "openapi-description"
updates["rich-text"] = "rich-text-formatting"
updates["relative-references"] = "relative-references-in-urls"
updates["runtime-expression"] = "runtime-expressions"
updates["runtime-expression-examples"] = "examples"

def kebab_it(c):
    if c.lower() != c:
        return f'-{c.lower()}'
    return c

if __name__ == '__main__':
    text = Path(argv[1]).read_text()

    names = {}
    removals = {}
    for match in re.finditer(r'\n(.*)<a name="([^"]*)"', text):
        name = match.group(2)
        names[name] = ''.join([kebab_it(c) for c in name])

        # was it a heading? file it for removal
        if len(match.group(1)) and match.group(1)[0] == "#":
            removals[name] = True

    for current, replacement in names.items():
        if replacement in updates:
            replacement = updates[replacement]
        text = text.replace(f'(#{current})', f'(#{replacement})')

        # only remove if removal is indicated, otherwise update
        if current in removals:
            text = text.replace(f'', '')
        else:
            text = text.replace(f'', f'')

    print(text)

Run it like: python kebab_it.py 3.0.4.md > temp.md and then if temp.md looks good, copy it back over 3.0.4.md

Do we need to keep the TOC? GitHub creates one for us ...

About the TOC, if I understood the thread correctly, respec builds a TOC as well, so we do not need one in our source at all

I'm not sure respec does the table of contents in a way we can use, so I've regenerated it in this update. Thanks @ralfhandl for getting into the respec details, I hadn't got there yet!
Changed items:

rebased to pick up newest additions to the dev branch
switched required bullet point format to * and make the toc command use this format as well
reapplied all other changes

It did occur to me that we could keep the  notation in the source and just generate that bit when we're building the HTML version. It's the last commit on the branch here anyway, so super easy to remove.

I'm not sure respec does the table of contents in a way we can use, so I've regenerated it in this update.

Our HTML build script removes the table of contents from the Markdown, the ToC we see for example in https://spec.openapis.org/oas/latest.html is generated by ReSpec from the section headlines.
Here's the relevant part of our build script:
https://github.com/OAI/OpenAPI-Specification/blob/6bd37a356740b8128bfcca02b2d12d745def5135/scripts/md2html/md2html.js#L176-L178
Every line between lines starting with ## Table of Contents and  everything except the first 18 lines would be removed unless we adjust the build script to the new ToC tool.
A better way forward would be to completely remove the "Table of Contents" subsection because

we don't need it for producing the published HTML
we don't need it for viewing the raw *.md files in GitHub because the GitHub Markdown viewer now has an Outline pane on the right with an auto-generated table of contents.

Should also tackle #1720

My notes on the process that @lornajane put in the PR description:
get checkout v3.0.4-dev
git checkout -b fix-markdown
cd versions

Run prettier for formatting

npx prettier --write --single-quote 3.0.4.md

Run markdownlint to fix whatever it can

Create markdown-lint.yaml with contents given, then
npx markdownlint-cli --config markdown-lint.yaml --fix 3.0.4.md

Take out the table of contents and its comments, replace with a single

awk '/## Table of Contents/{f=1} //{f=0; print ""; next} {if (f==0) {print}} ' 3.0.4.md > temp.md; mv temp.md 3.0.4.md

Run a magical one-off script to update/fix/rewrite/remove all our anchors and internal links.

python kebab_it.py 3.0.4.md > temp.md; mv temp.md 3.0.4.md
npx markdownlint-cli --fix 3.0.4.md

use markdownlint again to check that this all worked because it can check internal links

npx markdownlint-cli --config markdown-lint.yaml --fix 3.0.4.md

Can we use

npx prettier --write --single-quote --prose-wrap preserve 3.0.4.md

to make diffing with the previous spec versions easier?

Updated to apply the changes to the newest 3.0.4 version, and marked as ready to review as I don't think we have any more changes in flight for this branch.


Covid Result Metadata Table Improvements

Now that the covid_result_metadata table is live in production, there are a few improvements we can make:

Change patient age to be a numeric type
Add field for resident_congregate_care
Add field for upcoming site_of_care field. h/t @nickscialli-usds
Saving all of the report information exceeds our timeout. It would be better to return an Action ID, and then kick off a process that does not hold up the web response, but expose an API end point that lets us get information about an action: api/action/{action id}with the results. The workflow would then look like asking ReportStream to refresh the covid_result_metadata table with all information. The workflow creates an Action of a new type called metadata. The background process would then run through the reports, refresh them, and update the record in the DB table action with the results. The original invoker could periodically poll the new endpoint which would eventually show that the action is completed and the data is refreshed.
Create a simple API endpoint that pulls the COVID Result Metadata for a report ID and returns it as a JSON object. This doesn't need to be anything special or fancy. Invoke GET on api/report/{report id}/results which would return all the rows for that report ID
Build some tests:

A test should run that submits a file to Docker and then invokes the above API for the report ID and verifies the data matches
A test should be run that reprocesses the same file again, and verifies that the old data was deleted and then successfully reinserted. There's two parts to this test: save the data from the original insert so you have the created_dt and the primary key. Then reprocess the same file, and pulls the data for the file. The data should exist, the row count should be the same, but the primary keys and the created dates should be updated. This should also use the above API endpoint
Once the above issue with submitting a "refresh ALL" request is done, a test should be created to get the Action ID and poll the action API until it lists as completed, and then verify that the results have been written by invoking the API for each report ID and checking the results.

fysa: @jlusds

Will we work on this in sprint 22?

Hey team! Please add your planning poker estimate with ZenHub @MauriceReeves-usds @Adrian-Brewster

Research required to see if this is still valid or if we want to address other necessary improvements.

@MauriceReeves-usds is this still needed?

Closing. Mo will recreate with different requirements.


Eth1 connection uncatched exception

Describe the bug

(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 699)
(node:1) UnhandledPromiseRejectionWarning: Error: missing response (requestBody="{\"method\":\"eth_blockNumber\",\"params\":[],\"id\":887,\"jsonrpc\":\"2.0\"}", requestMethod="POST", serverError={"code":"ECONNRESET","path":null,"host":"goerli.prylabs.net","port":443}, url="https://goerli.prylabs.net", code=SERVER_ERROR, version=web/5.0.1)
    at Logger.makeError (/usr/app/node_modules/@ethersproject/logger/lib/index.js:179:21)
    at Logger.throwError (/usr/app/node_modules/@ethersproject/logger/lib/index.js:188:20)
    at /usr/app/node_modules/@ethersproject/web/lib/index.js:142:36
    at step (/usr/app/node_modules/@ethersproject/web/lib/index.js:33:23)
    at Object.throw (/usr/app/node_modules/@ethersproject/web/lib/index.js:14:53)
    at rejected (/usr/app/node_modules/@ethersproject/web/lib/index.js:6:65)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 700)

Getting this error on Medalla
Expected behavior

Stable eth1 connection, or handled errors
Steps to Reproduce

Connect with a loaded eth1 node, like prysmatic eth1 endpoint, and sometimes get an error. Lodestar can't handle it.
Screenshots

Desktop (please complete the following information):

OS: ubuntu, running docker image
Version: sha256:6680b25b46c5f8f1c951cb567242f50cf53055fda58ffcf98dbbae1874b969de docker image (master branch ~10 hours ago)
Branch: master
Commit hash: Not sure which commit exactly, see docker image hash

Might be worth including the solution for this issue into #1179


Power Calculator

Before jumping into Pr first comment for assign.
Problem statement : Create a function that takes voltage and current and returns the calculated power.
Probelm Link : https://edabit.com/challenge/a2D3cva6PKrHyqdXC
Choose one Cateorgy - Very Easy
question category : Math

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

Can I work on this?

[✓]  Task assigned
@jayanthj737

Added circuitPower #64 @aayushi-droid


Redundancies removed from delete-comment logic

I took out the write/read/write aspect of my comment deletion logic and am now using KeyFactory to delete individual comments. It's much simpler and faster now!

I took out the write/read/write aspect of my comment deletion logic and am now using KeyFactory to delete individual comments. It's much simpler and faster now!

Quick question: did this change your program's speed to the point that you noticed it at runtime? Or was it largely theoretical?

Quick question: did this change your program's speed to the point that you noticed it at runtime? Or was it largely theoretical?

Oh, sorry, that's totally theoretical! It seems to run at about the same speed but I imagine if there were lots of comments in the database this would have some impact on the runtime


Bug unexpected extrapolation resampling

close #241

Hi @Alek050
Thanks for noticing. You're right, the doc says it doesn't extrapolate, this is what I wanted to achieve, but apparently I forget to program it well 😃
I think you solved it. However, solving this bug created a new bug in cycles.time_normalize() where some corner data is now replaced by nans. I need to figure it all because I wouldn't want to introduce this new bug. I'll add the fix to this pull request before merging.
Thanks again for contributing to ktk, it's much appreciated and it makes a greater toolkit at the end.

Hi @Alek050
Yeah, I'm still waiting that the next version of ezc3d is out to push your work on rotation matrices. I may ping @pariterre for this.
Since other code needed an extrapolation feature, I figured that it's an interesting feature and therefore I added an extrapolate option to TimeSeries.resample. It's False by default so that the code now behaves following the original spec.
Thanks again!

Sorry @felixchenier
I did not notice I did not released the last version, doing that now!


Error with CSV and Changing units to Metric

Hi,
Thanks so much for building this code. I have purchased the e-Paper HAT and installed the code, but I am getting two issues:

Traceback (most recent call last): File "weater.py", Line 145, in  with open('records.csv', 'a', newline='') as csv_file: TypeError: 'newline' is an invalid keyword argument for this function and;
If I change line 80 to 'Metric' it still shows Fahrenheit. Do I need to change this to Metric, then go through and change all references to 'F' to 'C'?

Thank you!

It looks like this may be your issue

https://stackoverflow.com/questions/51832593/typeerror-newline-is-an-invalid-keyword-argument-for-this-function/51833105
https://stackoverflow.com/questions/32811992/python3-csv-writer-failing-exiting-on-error-typeerror-newline-is-an-invalid

I've create a PR for Metric conversion:

https://github.com/AbnormalDistributions/e_paper_weather_display/pull/27

Here is the changed file, feel free to test it, but be sure to backup the one you are using locally first. https://github.com/AbnormalDistributions/e_paper_weather_display/blob/fc880ab2fc9e2536288cb97e51989c3299c2b324/weather.py


Preconnect link with crossOrigin

What version of Remix are you using?
1.7.5
Steps to Reproduce
// <link rel="preconnect" href="https://YOUR_APP_ID-dsn.algolia.net" crossorigin />

export const links: LinksFunction = () => [
  { rel: 'preconnect', href: `https://YOUR_APP_ID-dsn.algolia.net`, crossOrigin: "true" }
]

Expected Behavior
Expecting typescript to pass
Actual Behavior
npx tsc
app/routes/docs.tsx:25:5 - error TS2322: Type '"preconnect"' is not assignable to type '"preload"'.

25   { rel: 'preconnect', href: `https://YOUR_APP_ID-dsn.algolia.net`, crossOrigin: "true" }
       ~~~

app/routes/docs.tsx:25:67 - error TS2322: Type '"true"' is not assignable to type '"anonymous" | "use-credentials" | undefined'.

25   { rel: 'preconnect', href: `https://YOUR_APP_ID-dsn.algolia.net`, crossOrigin: "true" }
                                                                     ~~~~~~~~~~~

  node_modules/@remix-run/server-runtime/dist/links.d.ts:11:5
    11     crossOrigin?: "anonymous" | "use-credentials";
           ~~~~~~~~~~~
    The expected type comes from property 'crossOrigin' which is declared here on type 'LinkDescriptor'

I just hit this issue embedding a Google font (which has a link with rel, href, and crossOrigin.)
Going by the MDN page, it looks like the three valid values for crossOrigin are:

anonymous
use-credentials
"" (which is the same as anonymous)

So crossOrigin: true doesn't seem to be valid.
For me, I was able to fix the issue doing something like this:
export const links: LinksFunction = () => [
  {
    rel: 'preconnect',
    href: 'https://fonts.googleapis.com',
  },
  {
    rel: 'preconnect',
    href: 'https://fonts.gstatic.com',
    crossOrigin: 'anonymous',
  },
  {
    href: 'https://fonts.googleapis.com/css2?family=MyFontName&display=swap',
    rel: 'stylesheet',
  },
];


Builder: allow user to generate seq_mem_d1

As far as I can tell, the builder library does not yet expose a way for the user to create seq_mems. This is a quick PR to allow that.

LGTM! I think in the long term we can probably define a seq_mem method that just takes an array for the dimensions, checks that we pass the right number of indices, and instances the right dimensional version.

Indeed; excellent!


undo change style.css

hi
i change style.css file in scss folder but when start project from cmd command my changes clear from file

Do not have a solution to solve this problem

@hananf1994
You can modify scss files instead


Automatic Relationship Names

One issue that we have been having is that relationship names are defaulted and can clash when two developers both create new relationships in their own branches, then merge into the develop branch. (We are using the folder method of serialization)
In order to solve this could we have TabularEditor automatically create a new name on creation / edit.
VS just creates a GUID, which may be the best option. However perhaps it would be better to name it in a descriptive way. E.g. Table1 - Table2 (Table1Field - Table2Field). This would always create a unique name even with multiple relationships between the same tables.
This descriptive method would then work the same way as all the other measures/columns in the model so I don't think this could cause any issues?

This is a good point. The TOM API used by Tabular Editor does not uniquely name a newly created relationship like VS does, and the Tabular Editor UI does not allow you to set the name of a relationship explicitly. The name shown in Tabular Editor, is not the actual relationship name, but just a value that shows which columns are involved and how they are related.
To make it consistent with VS, I will make sure that a new GUID is assigned as the internal relationship name, without changing anything in how the object is displayed in Tabular Editor.
As a temporary workaround, I noticed that if you cut (ctrl-x) a Relationship from the Tabular Editor tree, and then paste it back in (ctrl-v), the TOM API will actually assign a new GUID as the name of the re-inserted relationship. Not sure if this is a bug or a feature :-)


TOMPD 63A Breaker I,A and P Unreachable after upgrade

Device: tompd 63A breaker
Issue:
V,A and P Unreachable after upgrade up to 2023.4.1
Device diagnostics:
tuya_local-8efbcb4bdf021dcf1f7326cf746b0f12-Breaker-1845b3de8bbbae32a83639bcd937b26f.json.txt
Some errors from HA logs:
Logger: custom_components.tuya_local.device
Source: custom_components/tuya_local/helpers/device_config.py:387
Integration: Tuya Local ([documentation](https://github.com/make-all/tuya-local), [issues](https://github.com/make-all/tuya-local/issues))
First occurred: 08:50:15 (3 occurrences)
Last logged: 09:06:15

Breaker receive loop terminated by exception argument should be a bytes-like object or ASCII string, not 'NoneType'
Traceback (most recent call last):
  File "/config/custom_components/tuya_local/device.py", line 200, in receive_loop
    entity.async_schedule_update_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 721, in async_schedule_update_ha_state
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 568, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 632, in _async_write_ha_state
    state = self._stringify_state(available)
  File "/usr/src/home
[tuya_local-8efbcb4bdf021dcf1f7326cf746b0f12-Breaker-1845b3de8bbbae32a83639bcd937b26f.json.txt](https://github.com/make-all/tuya-local/files/11292832/tuya_local-8efbcb4bdf021dcf1f7326cf746b0f12-Breaker-1845b3de8bbbae32a83639bcd937b26f.json.txt)
assistant/homeassistant/helpers/entity.py", line 574, in _stringify_state
    if (state := self.state) is None:
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 470, in state
    value = self.native_value
  File "/config/custom_components/tuya_local/sensor.py", line 72, in native_value
    return self._sensor_dps.get_value(self._device)
  File "/config/custom_components/tuya_local/helpers/device_config.py", line 363, in get_value
    bytevalue = self.decoded_value(device)
  File "/config/custom_components/tuya_local/helpers/device_config.py", line 387, in decoded_value
    return b64decode(v)
  File "/usr/local/lib/python3.10/base64.py", line 80, in b64decode
    s = _bytes_from_decode_data(s)
  File "/usr/local/lib/python3.10/base64.py", line 45, in _bytes_from_decode_data
    raise TypeError("argument should be a bytes-like object or ASCII "
TypeError: argument should be a bytes-like object or ASCII string, not 'NoneType'

Screenshots:

Updated form git - all looks good


Color of connection line from Interfaces.Support

Reported by tobolar on 20 Nov 2013 14:14 UTC
The color of a connection line is gray when connecting from a support connector of an element (e.g. Modelica.Mechanics.Translational.Components.SupportFriction). According to the other translational connectors (e.g. Flange_a) the color should be green {0,127,0}.
Recomentation: Correct the color definition in Modelica.Mechanics.Translational.Interfaces.Support

Migrated-From: https://trac.modelica.org/Modelica/ticket/1353

Comment by beutlich on 16 May 2014 15:02 UTC
Resolved by f711abbc22c2afd6c03ba504451b0ff65905c731 and 7b94924429facca9757574e799e414b3c07ea291.

Comment by beutlich on 16 May 2014 19:56 UTC
I reverted f711abbc22c2afd6c03ba504451b0ff65905c731 and 7b94924429facca9757574e799e414b3c07ea291 and introduced a new fix in 9adf64cbd70fc7b0a36ccbc46a3bd12f8349afb3 and r7560 that applies the same hack as in Modelica.Mechanics.Rotational.Interfaces.Support to draw a background rectangle with a defined lineColor and that does not have pattern=LinePattern.None


icX_diagnostic treats "<=" and ">=" as errors

Describe the bug
icX_diagnostic treats "<=" and ">=" as errors:

This is likely caused by the recent change in icX.diagnostics.ts:
https://github.com/Traineratwot/vscode-stationeers-ic10/blob/faf7dfceec4cc1d92c260c81e3f758976e642985/core/icX.diagnostics.ts#L283-L287
To Reproduce
Steps to reproduce the behavior:
var a = 1
if a == 0 # linter happy

end
if a <= 1 # linter unhappy

end

Expected behavior
No error message.
Additional context
Add any other context about the problem here.

Year. it`s my mistake :). Thank you for taking your time to figure out my code.
I have updated the extension

if you are interested, we can do localization for China together

Thanks for the quick fix :) And I have to say this extension (and icX) is awesome!
As for translation, I'm playing the English version of the game as well, so might not be a very good person for localization. Though, I found Stationeers/rocketstation_Data/StreamingAssets/Language/<english|russian|chinese|etc.>.xml may be good reference for tooltip translation, there are translations for all logic variable descriptions - some regex based replacing could help :)


Update Legal agreement text on registration and sign-in screen

Description:
LEARNER-7291
Screen shots:

New
Old

iOS screen shots:

Link to the ticket is wrong in the description.


./gradlew setup exception

$ ./gradlew setup

> Task :compileJava FAILED
/path/to/RetroMCP-Java/src/main/java/org/mcphackers/mcp/tasks/TaskDecompile.java:80: error: reference to newFileSystem is ambiguous
                                        MappingUtil.modifyClasses(mappingTree, FileSystems.newFileSystem(originalJar, null).getPath("/"), className -> {
                                                                                ^
  both method newFileSystem(Path,ClassLoader) in FileSystems and method newFileSystem(Path,Map<String,?>) in FileSystems match
/path/to/RetroMCP-Java/src/main/java/org/mcphackers/mcp/tools/FileUtil.java:40: error: reference to newFileSystem is ambiguous
                try(FileSystem fs = FileSystems.newFileSystem(sourceZip, null)) {
                                               ^
  both method newFileSystem(Path,ClassLoader) in FileSystems and method newFileSystem(Path,Map<String,?>) in FileSystems match
/path/to/RetroMCP-Java/src/main/java/org/mcphackers/mcp/tools/FileUtil.java:52: error: reference to newFileSystem is ambiguous
                try(FileSystem fs = FileSystems.newFileSystem(sourceZip, null)) {
                                               ^
  both method newFileSystem(Path,ClassLoader) in FileSystems and method newFileSystem(Path,Map<String,?>) in FileSystems match
/path/to/RetroMCP-Java/src/main/java/org/mcphackers/mcp/tools/FileUtil.java:59: error: reference to newFileSystem is ambiguous
                try(FileSystem fs = FileSystems.newFileSystem(sourceZip, null)) {
                                               ^
  both method newFileSystem(Path,ClassLoader) in FileSystems and method newFileSystem(Path,Map<String,?>) in FileSystems match
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
4 errors

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileJava'.
> Compilation failed; see the compiler error output for details.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 962ms
1 actionable task: 1 executed

For Java 8+, you'll need to hardcode a cast.

what do i cast it to?

ClassLoader IIRC. Don't use Java 17 for compiling with Gradle. You can use it to run for debugging, but the Gradle JVM should be Java 8.

how do i change it? i have a jvm8 in /opt/java8/bin/java

Best way it to use your IDE settings, or alternatively, manually set the PATH and JAVA_HOME variables as necessary.
For Intellij, click on the Gradle vertical taskbar on the right, select the cog/settings icon at the top of the pane that opens, and set the Java version to a different version.
As for Eclipse, I am not too sure of the steps involved for that, but I do know it is possible.

You'll need to set JAVA_HOME and the PATH variable. The JAVA_HOME path should be above bin, lib, and the other folders. The PATH environment variable requires the bin path. For example, if I had Java at C:/Java, I would do
set JAVA_HOME=C:/Java
set PATH=C:/Java/bin;%PATH%

As for Linux, it would be something like this:
export JAVA_HOME=/usr/java
export PATH=/usr/java/bin:$PATH

Fixed in #6df5355bd472693b4fbfb54f51efb6478f6b8d4d. Closing issue, thanks for reporting!


Improve Path to Leadership Portion

Signed-off-by: Matt Germonprez<EMAIL_ADDRESS>I've made a few changes to help focus the various paths

@jaskiratsingh2000,
Can you please make these changes in the GitBook version of the handbook?


[api-docs] document removal of message gateway limitation

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/5f4decb7c488064d469188bc631e23fef7adf6ae

ncn


Error Thrown When Opening And Trying To Rip Any URL

Ripme version: 1.7.59
Java version: openjdk version "10.0.1" 2018-04-17
Operating system: Linux Mint 19.0

Exact URL you were trying to rip when the problem occurred: All URLs
Please include any additional information about how to reproduce the problem:

Expected Behavior
Rip Images
Actual Behavior
Output of error log here: https://pastebin.com/WY6uz9Z6
When opening the program. Also when trying to rip any url, it fails as if it cannot find the url.

Is Mint 19 based on Ubuntu 18.04? If so follow these instructions to fix it

@MrRossDoughty
Did you get it working?

Hey thanks for the advice, I got it working last night. More of a java thing than a issue with the ripper eh?

More of a java thing than a issue with the ripper eh?

Yup


Tests: Upgrade dependencies & remove explicit ATS version.

For changes in the chart, chart templates and container images, I executed the following tests, using the hello-world app, to verify them in live environments:

[ ] Upgrade from previous version

[ ] AWS
[ ] Azure

[ ] Existing Ingress resources are reconciled

[ ] AWS
[ ] Azure

[ ] Fresh install

[ ] AWS
[ ] Azure

[ ] Fresh Ingress resources are reconciled

[ ] AWS
[ ] Azure

You'll need to upgrade architect-orb to the latest version to get ats 0.5.0 otherwise it'll use 0.2.x

Where can I do so?

Nevermind, got it.


Header row style does not extend to detail panel column

Describe the bug
When the detailPanel prop is provided to add an accordion to each row, a caret is added to the left side of each row. The header in the space above that does not get the same styling as other column headers. Refer to screenshot below.
To Reproduce
        <MaterialTable
          ...
          detailPanel={renderDetailPanel}
        />

Expected behavior
The (grey) background color of the header should extend to all across to the left edge of the header row.
Screenshots

Each column in the columns prop defines its own headerStyle that is used to set its header's style independently of the others. That's a powerful feature, but it means that there's no fallback header style for columns that aren't explicitly defined, which in this case is the detail panel caret column. Here are some paths forward:

Add a faux column to the columns list with a field dedicated to the details panel column. While this would be convenient for the table header component, it would add complexity and confusion to the other components.
Add a style option specifically for the detail column header.
Add a global header style that applies to all header cells.

The global header style would be the most useful and least intrusive. It covers most use cases that set the same background color along the entire header. For cases that want to style specific column headers, the headerStyle field in each column definition overrides the global header style. The detail column's header could still be styled separately by providing a global header style, plus a different headerStyle in each column.

Each column in the columns prop defines its own headerStyle that is used to set its header's style independently of the others. That's a powerful feature, but it means that there's no fallback header style for columns that aren't explicitly defined, which in this case is the detail panel caret column. Here are some paths forward:

Add a faux column to the columns list with a field dedicated to the details panel column. While this would be convenient for the table header component, it would add complexity and confusion to the other components.
Add a style option specifically for the detail column header.
Add a global header style that applies to all header cells.

The global header style would be the most useful and least intrusive. It covers most use cases that set the same background color along the entire header. For cases that want to style specific column headers, the headerStyle field in each column definition overrides the global header style. The detail column's header could still be styled separately by providing a global header style, plus a different headerStyle in each column.

There is a global headerStyle prop under options. Thankfully no changes are needed.
This underscores the need for high quality prop documentation.

There is a global headerStyle prop under options. Thankfully no changes are needed.
This underscores the need for high quality prop documentation.


Projects API says it supports sort & filter but doesn't

Remove the sortLinks and filters from the response, collectionFilters from the schema.

I would prefer if sorting/filtering worked for this resource.

Tested with build from master .
/v1/projects  still exposes "Filter" and "Sort By" actions but we have no options to choose from for "Filter"/"Sort By".


ASAN error of bpf2c_fuzzer test.

Happened on CI/CD Release Validation run: https://github.com/microsoft/ebpf-for-windows/actions/runs/7147631976/job/19467797260
Test unit written to Artifacts\crash-3688af1375d9360872b65d0e67f31e5d9aa8166b:
Artifacts-bpf2c_fuzzer-x64-FuzzerDebug.zip

As discussed, we should add the artifact to the bpf2c repo, so to augment its test model, eventually producing more consistent failures.

This is being fixed as https://github.com/serge1/ELFIO/pull/132


Fix newlines for new release cycle report

Our automation reports new unknown release cycles noticed for any tracked products: https://github.com/endoflife-date/endoflife.date/pull/1867#issuecomment-1318549168.
It does this via This GitHub Workflow, which leaves a new comment when the dependabot PR is filed, and the latest automation runs.
The code that generates the list of product warnings is here.
It currently generates everything in a single line, due to GitHub limitations, and ease of coding. GitHub now supports a delimiter for newlines, so this needs to be used: https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#multiline-strings
Updating the latest.py script should be good enough.

Just an idea (you may already had) but it could be great if the bot could open an issue for each product.
The issue could be title [product name]  x.y and the body :
A new version has been detected, here is the snippet :
-   releaseCycle: "x.y"
    releaseDate: yyy-mm-dd
    eol: false
    latest: "x.y.z"
    latestReleaseDate: yyy-mm-dd

The YAML would not be complete of course but that would be a good start for anyone willing to contribute to endoflife.date.

Yeah, but then it increases burden on the maintainers, since only they can commit to those local branches. Contribution workflows that fix this local branch are not very nice either, and it might get too confusion.
I was considering keeping an always pinned issue that is updated each time it runs, that is more contributor friendly - look here for new releases we need to add.
Still open to trying, but also keep in mind the additional complexity this will bring (keeping a lot of state to remember which PRs are filed etc). We also have a lot of false positives, and it's hard to guess the next release cycle.

I agree with you on this point : in my mind only an issue would be created, not a PR. And the code snippet would be written in the issue description.
And I also agree with you on the single pinned issue. Always updating the issue description (or adding a new comment to the issue) would be simpler than creating and maintaining an individual issue for each product.


error when running demos

when I run the python demo using the command python demo.py --model human-pose-estimation-3d.pth --video 0
I got the following error. I am running it within Anaconda on my MacOS 10.13. Running with the OpenVino option gave me similar error.
I was able to access my camera by using a little test script. So there should be no problem there. Any idea how to solve it? Thanks!
2020-04-21 16:21:48.662 python[693:7647] +[AVCaptureDevice authorizationStatusForMediaType:]: unrecognized selector sent to class 0x7fffb06446a0
[ERROR:0] global /localdisk/jenkins/workspace/OpenCV/OpenVINO/2020.2/build/osx/opencv/modules/videoio/src/cap.cpp (265) open VIDEOIO(AVFOUNDATION): raised unknown C++ exception!
Traceback (most recent call last):
File "demo.py", line 87, in
for frame in frame_provider:
File "/Users/yongtang/Documents/DL/CV/pose/Pose3D/modules/input_reader.py", line 36, in iter
raise IOError('Video {} cannot be opened'.format(self.file_name))
OSError: Video 0 cannot be opened

Hi, do you use OpenCV to check if camera works? I suggest to try other values instead of 0, e.g. 1, or 2, or 3...

Thank you for your quick response! Yes, I did verify that the camera works using OpenCV. I think the problem lies somewhere else but not figured out yet.

That is strange, because demo also uses OpenCV, so behavior should be similar.

Hope, it works.


Update Early Access Info

Updated early access version and removed old release notes link.

//cc @vivmishra


[TASK] Sync the order of public inputs for R1CS and PLONK

For Mixer:

nullifier_hash
root
arbitrary data

For Anchor:

chain_id
nullifier_hash
arbitrary_data
roots[]

For VAnchor:

public_amount
public_chain_id
arbitrary_data
in_nullifier_hashes[]
in_root_set[]
out_commitments[]

We would have to sacrifice some of the performance to achieve this in PLONK. For example:
Set gadgets expects a set to be Vec<F> which are public inputs (https://github.com/webb-tools/arkworks-gadgets/blob/master/arkworks-plonk-circuits/src/set_membership/mod.rs#L15). They are inserted into the circuit at the specific time to avoid additional gates. We would have make a standard for PLONK to give us control where the public inputs are defined, which will make it easier for make PLONK and R1CS interoperable.

Should be equal to solidity side


updated js: steps for card API & animation to movie API

Top of the js section now outlines how card API and movie API connect. Further down is a bunch of stuff related to the movie API. That can all be commented out to test card API call functionality.

great!


Future of rstfmt on PyPI

Considering that the last commit to dzhu/rstfmt was over a year ago, with the program in an unfinished state and pull requests languishing, I think the upstream repository is dead (unless @dzhu begs to differ).
With that in mind, what are your plans for this fork?
I've been working on rstfmt too. I've discovered a number of bugs, which I have now fixed, on my local fork. I'm not entirely sure which fork to contribute towards, though.

Hello! Indeed, I got rstfmt to do what I needed it to do a while back and I haven't had much energy to spare for it since then. I'd be open to the possibility of bringing on someone else as a co-maintainer. However, it happens that I am just about to go on a trip where I may have limited Internet access for the next few weeks. I can come back to the topic in September.

Hi @EnronEvolved, @dzhu,
Sorry for the late reply, and thank you both for the context and feedback.
My initial goal with this fork was to ensure it's usable with pre-commit hooks, that's why I only tagged, but never put it on PyPI.
Regarding the PyPI package, I think we have two possible solutions:

Add maintainers to the original repo: we don't have to co-maintain the PyPI package separately. The existing access token for PyPI could be integrated into a GitHub Action for streamlined releases.
Maintain it in this fork: If the upstream repository remains inactive, I’d be happy to maintain it here and co-maintain the PyPI package as needed.

Both options work for me, and I’m open to suggestions. Looking forward to hearing your feedback!


--storepass won't re-prompt for Password if used again

Sometimes we change our PennKey passwords. Right? ;)
I use --storepass in various places to safely store it and reduce password fatigue. After changing my password, instead of having to figure out how to use OS-specific ways to delete my stored PennPass, I'd prefer that simply running aws-federated-auth --storepass will replace the current pass with the new one. OR there's a --removepass option. Or both.
I believe that the first option (re-prompt and store) could be as simple as:
$ git diff
diff --git a/shib/cli.py b/shib/cli.py
index 3c6106f..59d0ea6 100644
--- a/shib/cli.py
+++ b/shib/cli.py
@@ -259,7 +259,7 @@ def main():
     password = None
     password_stored = False

-    if keyring is not None:
+    if keyring is not None and not args.storepass:
         try:
             password = keyring.get_password("aws-federated-auth", "password")
         except Exception:

Happy to push that to a feature for review, if others think it's the right way to go.

Created pull #6


switch to ml_configuration in agent response as backend changed

Description
The backend ml-commons changed its configuration field from configuration to ml_configuration and lead to error.
GET /_plugins/_ml/config/os_chat

{
  "config_type": "os_chat_root_agent",
  "ml_configuration": {
    "agent_id": "******************"
  }
}

Related change in ml-commons:
https://github.com/opensearch-project/ml-commons/pull/2710
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test.

[ ] New functionality has user manual doc added.
[ ] Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is this change target for 2.16?

Is this change target for 2.16?

The ml-commons related change https://github.com/opensearch-project/dashboards-assistant/pull/239#issuecomment-2247335910 has backported to 2.16, so I think this change should target for 2.16

It would be better to add your change in https://github.com/opensearch-project/dashboards-assistant/blob/main/release-notes/dashboards-assistant.release-notes-<IP_ADDRESS>.md as this should be released in 2.16.

It would be better to add your change in https://github.com/opensearch-project/dashboards-assistant/blob/main/release-notes/dashboards-assistant.release-notes-<IP_ADDRESS>.md as this should be released in 2.16.

@raintygao  But it cannot be called a feature nor bug fix. Maybe Maintenance ?

It would be better to add your change in https://github.com/opensearch-project/dashboards-assistant/blob/main/release-notes/dashboards-assistant.release-notes-<IP_ADDRESS>.md as this should be released in 2.16.

@raintygao But it cannot be called a feature nor bug fix. Maybe Maintenance ?

Im okay to this.


Add ExternalStorageDirectoryPath

I have an error after this commit
.../node_modules/react-native-fs/android/src/main/java/com/rnfs/RNFSManager.java:407: error: cannot find symbol
    File externalStorageDirectory = this.getReactApplicationContext().getExternalStorageDirectory(null);
                                                                     ^
  symbol:   method getExternalStorageDirectory(<null>)
  location: class ReactApplicationContext
Note: .../node_modules/react-native-fs/android/src/main/java/com/rnfs/RNFSManager.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error
:react-native-fs:compileReleaseJavaWithJavac FAILED

Any idea ?


Using the "next field" button on iOS skips the date picker

Using the "next field" button on iOS skips the date picker. See attached GIF for a demo.
This seems to be related to having tabindex="-1" and a custom on-focus listener on the <input> element. Not sure how this should behave though, as entering any date values with keyboard is not yet supported.
Originally this issue was reported by @vaadin-marcus as part of the https://github.com/vaadin/expense-manager-demo application.

The input should be focusable, but read-only. Once focused, it should open with the keyboard arrow keys. On iOS (touch devices), I don’t think we should auto-open the popup on focus, so the only way to open it is by tapping the field or calendar icon.

Update: This is kinda fixed with https://github.com/vaadin/vaadin-date-picker/pull/247.
But in the vaadin-date-picker-behavior.html _noInput property is computed as true in iOS. Meaning that input can be focused (won't skip it), but pressing any of the keyboard keys won't fill or open the overlay. If _noInput would be false then overlay would be open at same as the keyboard.

@samiheikki nice catch 👍🏻
We should allow keyboard input on iOS as well, but we should not open the overlay in that case (at least with the current fullscreen implementation, which uses position: fixed; which fails on iOS when an input is focused).
When we fix the overlay positioning and stacking context issues (move it under the <body>), we should re-evaluate the fullscreen mode design at the same time.

This is now reported at vaadin/vaadin-text-field#479, so closing this in favor of that issue.
The fix mentioned above was targeting Polymer 1 and is therefore no longer relevant.


Revert broken code introduced in 1.10.0

Here I revert only the breaking changes in 1.10.0.

Hi Bas - I couldn't sleep knowing the mess I played a part in here. I was not completely following all of the refactors and other changes to the code during the review process. I still don't quite understand the _AsyncSingleShotTimer so I'll be more honest when I don't understand something moving forward.
I looked over the code in 1.10.0, everything matches the working branch solely except the transition timer being reset.
If you're going to go through with this PR here, it's worth mentioning:

Everything in #510 needs to be reverted, not just transition timers. This includes things like significant_change. Everything in that PR will only work if a transition is finished.

It may be easier to fix the timer rather than revert all the broken changes.
All the best,

Benjamin

Since @th3w1zard1 properly fixed this in #544, we can close this now :)


Implement SSL Listeners

Lets you enable and use SSL listeners, with supplied key/cert files.
ssl.PROTOCOL_SSLv23 just selects the highest available SSL/TLS protocol that both the client and server support, from SSLv2 to TLSv1.2. I haven't done much with SSL/TLS, but that should be what we want?
(table of the various protocols supported by Python's SSL library just under this heading)

It may also be worth looking over this blog post, see if we wanna take any of that into our SSL implementation.

Is the TLS implementation in-process or out-of-process?  We want to keep it out of process for performance reasons, like charybdis ssld.

Regarding SSLv23: Don't. Simple TLS or TLSv1 will negotiate higher, but SSLv2 and SSLv3 are both broken irreparably at this point. As for the rest: see what @kaniini said.

@Aerdan we discovered that some clients require an SSLv2 ServerHello in charybdis.

@Aerdan These lines disable SSLv2 and v3 in Python's SSLv23
context.options |= ssl.OP_NO_SSLv2
context.options |= ssl.OP_NO_SSLv3

SSLv23 is also the only protocol in Python's SSL library that will negotiate multiple protocols, whereas the explicit TLSv1, TLSv1.1, and TLSv1.2 protocols will not negotiate higher because Python's SSL is dumb (eg: TLSv1.1 will not negotiate TLSv1.2, etc). Might not be able to do that though, considering @kaniini's note up there.
@kaniini this one's in-process, so basically we'd just run an external ssld basically equivalent to what charybdis does, instead?

@DanielOaks yes, i was thinking of like multiprocessing or something, we could merge this for now just for having something, but ultimately moving it out of process is something we should do so that it can run parallel with the actual ircd processing

@kaniini Fair enough, that makes sense. Could have it pop up a message saying mammon: note: SSL support is not optimized and may cause slowdowns in your server or something that it bugs us to fix it?

yeah, sounds good


File extension throws parsing error

I am receiving an error message when I try to parse a pdf.
However, my file has the correct extension.
Parsing files: 100%|██████████| 94/94 [00:00<00:00, 2550.72it/s]
Error while parsing the file 'Doc ID: 6b7d681b-846d-4b7b-91cd-78d69ba56c8f
Text: Core – Assistance with Social, Economic and Community
Participation NDIS Pricing Arrangements and Price Limits 202 3-24
Version 1.3 (published 7/02/2024 ) Page 59 of 94 Participation in
Community, Social and Civic Activities – High Intensity Supports
These support items enable a participant who requires High Intensity
Supports , to engage...': Currently, only the following file types are supported: ['.pdf', '.doc', '.docx', '.docm', '.dot', '.dotx', '.dotm', '.rtf', '.wps', '.wpd', '.sxw', '.stw', '.sxg', '.pages', '.mw', '.mcw', '.uot', '.uof', '.uos', '.uop', '.ppt', '.pptx', '.pot', '.pptm', '.potx', '.potm', '.key', '.odp', '.odg', '.otp', '.fopd', '.sxi', '.sti', '.epub']
Current file type: .
Error while parsing the file 'Doc ID: 4ced617d-fe8f-469e-b164-b491de36b970
Text: General Claiming Rules NDIS Pricing Arrangements and Price
Limits 202 3-24 Version 1.3 (published 7/02/2024 ) Page 21 of 94
10_799_0133_5_3 Provider travel - non-labour costs Each $1.00
$1.00 $1.00 11_799_0110_7_3 Provider travel - non-labour costs
Each $1.00 $1.00 $1.00 11_799_0117_7_3 Provider travel - non-
labour costs Each...': Currently, only the following file types are supported: ['.pdf', '.doc', '.docx', '.docm', '.dot', '.dotx', '.dotm', '.rtf', '.wps', '.wpd', '.sxw', '.stw', '.sxg', '.pages', '.mw', '.mcw', '.uot', '.uof', '.uos', '.uop', '.ppt', '.pptx', '.pot', '.pptm', '.potx', '.potm', '.key', '.odp', '.odg', '.otp', '.fopd', '.sxi', '.sti', '.epub']
Current file type: .
Error while parsing the file 'Doc ID: 64876982-3824-4046-bbd3-dae1c6aa5e02
Text: Core – Assistance with Social, Economic and Community
Participation NDIS Pricing Arrangements and Price Limits 202 3-24
Version 1.3 (published 7/02/2024 ) Page 60 of 94 This support item
is not subject to price limits. It should only be used to recover the
costs of the participant’s attendance at the community social and
participation activ...': Currently, only the following file types are supported: ['.pdf', '.doc', '.docx', '.docm', '.dot', '.dotx', '.dotm', '.rtf', '.wps', '.wpd', '.sxw', '.stw', '.sxg', '.pages', '.mw', '.mcw', '.uot', '.uof', '.uos', '.uop', '.ppt', '.pptx', '.pot', '.pptm', '.potx', '.potm', '.key', '.odp', '.odg', '.otp', '.fopd', '.sxi', '.sti', '.epub']
Current file type: ."

docs = SimpleDirectoryReader(input_files=['newsletter.pdf']).load_data()
try:
    docs_parsed = LlamaParse(result_type='markdown', parsing_instruction="This is a pricing guide for a company. ANY .au or seeming extensions are just emails or websites and should be ignored. Please parse the document and return the parsed document in markdown format.").load_data(docs)
    print(docs_parsed)
except Exception as e:
    print("Error:", e)

Using 0.4.1

I realized I wasn't using the correct input format


Introduce class with test rpm

Introduce class in the test scope with test rpms, class have to:

hide paths to test rpms
be able to add these test rpms into supplied storage (one by one)
add invalid rpm into storage (see RpmTest.doesntBrakeMetadataWhenInvalidPackageSent)

Then use this class in all tests that use test rpms.
Also, it would be useful to add one any rpm into test resources.

@g4s8/z please, pay attention to this issue

@0crat assign me

@0crat assign me (here)

@olenagerasimova The job #235 assigned to @olenagerasimova/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@olenagerasimova the puzzle #248 is still not solved.

@olenagerasimova/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@rultor release, tag=0.9

@rultor release, tag=0.9

@g4s8 OK, I will release it now. Please check the progress here

@rultor release, tag=0.9

@g4s8 Done! FYI, the full log is here (took me 9min)

@olenagerasimova the puzzle #265 is still not solved; solved: #248.

@olenagerasimova all 2 puzzles are solved here: #248, #265.


Vite-Rollup with Sveltekit without ESbuild : can only import JS file in a PNPM workspace, not TS files

Describe the bug
In this monorepo, within Vite + Sveltekit projects :

✅ TS files from the same project work
✅ JS file from the PNPM workspace work
✅ On a regular Sveltekit install, TS files on the PNPM workspace work
BUT When I import a TS file on the PNPM workspace work I get this :

Parse failure: Unexpected token (3:26)
At file<EMAIL_ADDRESS>Contents of line 3: export const messageLoaded: string =

If I remove TS syntax from the TS file it works again. It seems the file is imported as JS
The file that loads other files is TurboSvelteKitTypia\apps\SvelteKitTS\src\routes+page.svelte
Tried :

putting the rollup-plugin-typescript2 plugin before the sveltekit plugin in the vite.config.ts, it didn't work
importing the file with .ts at the end of the import, no effect
importing the file from a relative path .ts at the end of the import, no effect (https://github.com/vitejs/vite/issues/5370#issuecomment-1339022262)
setting sourceMap: false, in the rollup-plugin-typescript2 plugin config, no effect (in tsconfig.json it does suppress a warning)
setting allowImportingTsExtensions: true in the rollup-plugin-typescript2 plugin config & tsconfig, no effect (https://github.com/samchon/typia/issues/812#issuecomment-1738303267)

Tried solutions proposed in :

https://github.com/ezolenko/rollup-plugin-typescript2/issues/465
https://github.com/vitejs/vite/issues/5370#issuecomment-1063613175
https://github.com/samchon/typia/issues/812

Reproduction
https://github.com/MarArMar/TurboSvelteKitTypia
Steps to reproduce
Setup
pnpm add turbo --global
pnpm install

Working
Vite & Sveltekit with Js : working test
Dev
npx turbo run dev --filter=SvelteKitJS

Vite & Sveltekit with Ts : not working example
Dev
npx turbo run dev --filter=SvelteKitTS

System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
    Memory: 2.75 GB / 15.74 GB
  Binaries:
    Node: 18.17.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 9.6.7 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.8.0 - ~\AppData\Local\pnpm\pnpm.EXE
  Browsers:
    Edge: Spartan (44.19041.1266.0)
    Internet Explorer: 11.0.19041.1566

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Passing
include: ["../../**/*.ts+(|x)"],

to rollup-plugin-typescript2 worked.
Closing as it's not a bug in Vite.

Thank you sooooooo much @sapphi-red, it worked  !!!!


Add support for FUSE3 on Linux

This pr adds support for FUSE3 on Linux and and makes a slight modification to the FileSystemBase interface to allow building for both FUSE2 and FUSE3. FUSE2 support is maintained and the default build option. To build with FUSE3 support you must use the fuse3 build tag which will then link to fuse3.
Additionally, some of the command line options changed in FUSE3 so direct_io and use_ino can no longer be set as command line options when starting a mount. These must be set in fuse_config so there are some additional functions introduced to set these options.
This also updates the CI/CD to test for FUSE2 and FUSE3 on Linux. It also only checsk the ability to build on MacOS since the old image is no longer available as a GitHub action and requires a restart to test the filesystem in GitHub actions.
This resolves #50.

Very nice. Rclone users have been requesting this 😄
I agree with Bill this shouldn't change the interfaces and break backwards compatibility though.
Can we make the old interfaces work and do something sensible with fuse3. I guess it might require multiple calls to emulate something where a fh is passed in.
Another alternative would be to only provide the new interface when the fuse3 tag is supplied. This would complicate the implementation though.
Another alternative would be to declare a v2 for cgofuse. That would be the least extra code and would give a chance to clean up the Ex interfaces if we wanted.
I thought the whole v2 thing was poorly thought through when I first read about it, but having been through it with some popular packages (eg ncw/swift ) it isn't too scary and works well. We would make a v1 branch which could receive minimal or no support (no support was what I chose) and then master becomes v2.

I agree that that it would be better to not change the interface. Based on that feedback, I have added Fuse3 equivalent calls for Chmod, Chown, Utimens, Rename, and Readdir as this seemed to be the easiest to implement. Most filesystems on Fuse3 will not need to implement these methods as I doubt many will use the additional arguments so this means that we can still support Fuse2 and Fuse3 on the original interface by not passing the additional arguments from Fuse3.

@billziss-gh Just a gentle ping on if the changes I to keep FileSystemInterface backwards compatible are a good solution.

@jfantinhardesty sorry for that. I have been very busy with real-life events this last couple of weeks. I may be able to find some time to review over the next few days during the holidays.


#1167 Cannot use custom id as TreePathSource

solved by other commit

Hello, and thank you for your contribution.
In the interest of spring cleaning and package modernization, I'm going through and closing old issues and pull requests that have had no recent activity, do not have associated tests, or have failing tests.
If you are still experiencing this issue or wish to contribute an up-to-date pull request, please create a new issue.
Thank you!


sov-modules: #[init] methods should be called only once.

The init method should be callable only once.
Question: should we provide a default implementation or should we force developers to always initialize a module?

fixed #53


RAM optimizations

Discussed here:
https://forum.littlevgl.com/viewtopic.php?f=15&t=87&p=658#p658
So far implemented:

2 and 4 bitfield optimizations

Reduced dot_tmp to 12 bytes as an initial optimization. Next step is to dynamically allocate it ondemand.

5 changed these fields to uint16_t. Added an additional bit in the bitfield to check if selection is enabled/disabled. Also added setters/getters to translate these back and forth to a int so that not much other code had to be changed. Also added an option to lv_conf.h to disable these fields completely.

Added another commit that dynamically allocates tmp_dots in lv_label as necessary according to the following rules:

If the dots replaces 4 or less bytes (typical case is 4 bytes, 3 dots plus the null pointer), the characters are stored directly in ext. The stored string is not NULL-terminated
If the dots replaces more than 4 bytes, instead of storing them directly in ext, a pointer to dynamically allocated memory is stored in ext.  The stored string here is NULL-terminated, this is so that it can be properly copied when duplicating an object.

For a list of 25 elements without dots, this saved me about 200 bytes of RAM.

If the dots replaces 4 or less bytes (typical case is 4 bytes, 3 dots plus the null pointer), the characters are stored directly in ext. The stored string is not NULL-terminated

Awesome idea!

Or to indicate "no selection" we can use define LV_TA_SELECTION_OFF 0xFFFFanduint16_treturn values. This way we don't needint32_tonly for -1`.

I fixed most of the small changes (documentation, data types, etc).
For the selection enable/disable macro, I always find "negative" macros confusing, but I can make the change if you think it's best.
If we are comfortable with the current changes, we can make more invasive changes such as removing lv_ta_get_selection and getting rid of the -1 sentinel value.  I just wanted to break as few things as possible in one go :).

@kisvegabor Could we ever have a case where someone had more than 64K of characters in a text area? I admit that I didn't consider that when I implemented this a few weeks back. In that case the integer overflow could cause strange behavior.

@BrianPugh  What about my suggestion in this post?
@embeddedt I've checked a random short story see how long 65k character is. This story has 52k characters: https://www.theatlantic.com/magazine/archive/2001/05/the-hunters-wife/302198/
It thinks it's not a real-life scenario to store it in one label. With such a long text, even the coordinates could overflow. (now -32k..32k).

@kisvegabor Okay, no problem then.

Is there a way to test ta selections in the pc simulator? I can't get it working, even on just dev-6.0. Need to do that to test more modifications.

@BrianPugh You have to enable it with lv_ta_set_sel_mode(ta, true);

I test the selection with the demo application.
I added #define LV_LABEL_TEXT_SEL_OFF 0xFFFF to lv_label.h and modified the functions accordingly.
There were some bugs with selection but I fixed them. Now it seems working.
I renamed the selection to text_sel to be more descriptive. selection was too general IMO.

These changes sound good to me; i won't be able to test the code for a day or two, but i trust that if it works in the demo application, it works for me.

Okay, I merge it then.
You can open an issue of you find any problems with it.


Use IEnumerable for list parameters

Fixes https://github.com/microsoft/typespec/issues/4786
Also makes the serialization ctors use IEnumerable which is different from Autorest.

API change check
API changes are not detected in this pull request.

do we have something to configure so that we could change this back in azure libraries?

do we have something to configure so that we could change this back in azure libraries? and would this lead to an extra ToList?

Yeah this leads to an extra ToList so I am thinking we need to keep the IList for the serialization ctor.


Annotate the return type of getFromDomain

With this change we can use an interface to annotate what is returned from getFromDomain so that we can use type hints:
@Injectable({ domain: 'A' })
class Car implements Drivable {
  drive() {
    console.log('Driving');
  }
}

const injector = new Injector();
injector.addProviders({ token: 'Drivable', useClass: Car });

const vehicles = injector.getFromDomain<Drivable>('A');
vehicles.forEach(i => i.drive());

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

Nice 👍, Thanks for taking the time to contribute to the code!


docs: Mark host-policies as tech preview

The Cilium v1.8 blog describes this feature as tech preview, make sure
the docs are consistent with this.

I was wondering about that myself, we should come up with criteria and define what these mean :-)

Should we also apply this change to doc that refer to Hubble Relay? It's also "tech preview". I think the doc currently mentions it's "beta" but this generic warning seems better.

Coverage increased (+0.009%) to 37.177% when pulling c5c4a3c68a3bb4acf13699ca6adeba915991525b on joestringer:submit/host-policies-tech-preview-docs into d4b7936557c96f8792bb668b79aaccce6f4af29d on cilium:master.

@Rolinh filed https://github.com/cilium/cilium/issues/12280 to follow up on this. I think it'd be good to have a discussion on what we mean when we use these terms and how we decide to apply them.


Add verbosity and quiet options to the utility

As a user, I would lite to have the possibility to run the utility in quiet and/or verbose modes

Added -q/--quiet option that allows to not show INFO messages


config: enforce stricter validation rules and tune default config

Enforce stricter validation rules and tune default configuration in misc/snapshotter/config.toml

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (f89d069) 37.13% compared to head (c0ead75) 37.11%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #464      +/-   ##
==========================================
- Coverage   37.13%   37.11%   -0.03%
==========================================
  Files          60       60
  Lines        6912     6924      +12
==========================================
+ Hits         2567     2570       +3
- Misses       4048     4052       +4
- Partials      297      302       +5

Impacted Files
Coverage Δ

config/config.go
33.33% <0.00%> (-0.91%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added support for MONTH_DAY_NANO interval type

Based on

[ ] https://github.com/apache/arrow/pull/10177

which contains the new schema.fbs, that this PR is based on.
The changes are rather simple since the physical type can be represented as months_days_ns(i32, i32, i64), which fulfills all the conditions of the trait NativeType.
Backward incompatible changes

IntervalUnit has a new variant, IntervalUnit::MonthDayNano.

Thanks!  Curious if you have any thoughts on Rust CI integration failures I'm seeing on the PR

Integration tests are passing against apache/master with the patch that unskips most of the cases including generate_month_day_nano_interval_case  🎉


nullcheck added, related to rm 20729

I want to change toFixed so it returns empty string if given bad data.

This will return a empty string if a zero is used.

yikes, yeah would not be good. Added strict number 0 check.

Maybe im missing something, but can't we just check for typeOf !== 'number'?

Yeah that would work for null/undefined.


Can not mock useGooglePay with jest

Describe the bug
When attempting to mock useGooglePay with jest I am getting the following error
TypeError: Cannot read properties of undefined (reading 'mockImplementation')

To Reproduce
Attempt to mock useGooglePay with the following
import { useGooglePay } from '@stripe/stripe-react-native';
...
  const useGooglePayMocked = jest
    .mocked(useGooglePay, true)
    .mockImplementation(() => ({
      isGooglePaySupported: jest.fn(async () => true),
      loading: false,
      initGooglePay: jest.fn(),
      presentGooglePay: jest.fn(),
      createGooglePayPaymentMethod: jest.fn(),
    }));

Expected behavior
Properly be able to mock useGooglePay hook with jest.
Screenshots
N/A
Additional context
I am currently using "@stripe/stripe-react-native": "0.17.0",

@charliecruzan-stripe thanks for the quick response. I realized this was a misconfiguration with jest and nothing to do with the strip lib!
Closing the issue.

@JperF are you able to share what the configuration fix was for this?? (If you recall). Experiencing the same issue  - Thanks!

UPDATE: We were also missing a config.  From the readme mentioned above - adding these two lines to our jestSetupFile.js fixed all issues.
`import mock from<EMAIL_ADDRESS>jest.mock('@stripe/stripe-react-native', () => mock);`


Fixing autodetected version for Debian Jessie, which should use postgresql 9.4

BTW, this would solve MODULES-1603

Are there plans for a new release? Would be great to have 9.4 support in the module available.

@mfn: you could propose a release yourself, like so: https://github.com/puppetlabs/puppetlabs-postgresql/pull/478

Wouldn't pg_hba_rule.php also need an entry for 9.4?

@mfn looking at this code, it should work with the default, aaand, we should fix it so it works better.
There's way too much repetition, and, i have no idea what, exactly, that default is supposed to be.


Update macos.md

macOS 12.5.1 was released on August 17, 2022.

Added macOS 12.5.1 (released August 17, 2022) as the latest version of macOS Monterey.

This was gonna get automatically updated in a few hours: https://github.com/endoflife-date/release-data/blob/main/releases/macos.json#L39


Fix robots metadata #42

I'd rather see this property removed altogether, refer my comment on the initial issue.


Option to upload a quiz through frontend

The admin could add questions and the user can give the quiz and analyze one's performance.
I would like to contribute to it under GSSOC'21. Please assign me the same.

Can you please elaborate what exactly are you planning to do? @somya51p

@diyajaiswal11, Like if mentors are signing up then they would have the option to upload new quizzes and review the results of various students to keep track of their students. Hope I am able to explain to you my idea!


failure to set DF bit is okay, some platforms do not support it even though their SDK might

With 10.16 being released, macOS SDK now has the definition of IP_DONTFRAG, but it is impossible to set that socket option when running 10.15.
Hence demoting the error to a warning.
See also: https://github.com/h2o/h2o/pull/2489.

Travis is having some hiccups, I'm going to merge this anyways.


set read or write type when select

I want to know the read or write type from QueryExecuted event when i dont set ::read or ::write after connection name,because not all model use R/W Splitting

Seems this is breaking tests.

I only change the property of readWriteType in Connection,adding one line.
I dont know where breaking the tests.
I use pull request for the first time,Thanks for pointing it out for me