Documentation: Documentation does not include example of downloading document to data folder

I set data folder with env

I added the document

Document shows up in list

The data folder is absolutely empty

The tutorial or example MUST include example of downloading or fully syncing the data in a document.
Should data be downloaded automatically? Should the data folders be empty?
What command do you run to actually download the document file contents or sync them?

I set data folder with env

can you please describe which commands you exactly ran?

We added a section to clarify documents storage. Iroh handles its own storage. You can import and export data out of it. This is described in https://iroh.computer/docs/layers/documents#storage and in https://iroh.computer/docs/api#document-commands with (doc export and doc import). For blobs there are similar commands (blob export and blob add)


Add hyper HTTP/2 framing support.

@vjn Do you have a production use for this feature ?

@gabis-precog there is no existing impl supporting http/2. Thus, lets close this issue for now and reopen once we see the need


one question for transductive SSL

In this paper, are the training set and the test set put together to construct a graph?

In our paper, we do not include the test set in the graph. We include the training set containing labeled and unlabeled examples. We use a transductive approach (label propagation) to train an inductive classifier, which generalizes to unseen test images.


Option to pre-load a string to query

Hello,
Thank you for this tool, which makes nucleo easy to use.
I want to migrate from skim to nucleo and I'm missing an option.
In skim I can specify a query that pre-feeds the search with a string.
https://docs.rs/skim/0.10.4/skim/prelude/struct.SkimOptionsBuilder.html#method.query
Is it possible to do the same with nucleo and nucleo-picker?
Thanks

Thanks for the suggestion! This isn't currently possible but should be easy to implement; I've been planning to set up more general configuration options for the Picker in any case (e.g. to also fix #2) and I'll implement this at the same time. It will be a breaking change but should be in v0.5.0.


Add streaming notes to update example file

PR summary

Fixes: <! -- link to issue -->
Note: An existing issue is required before opening a PR.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the
Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Add stream handling examples in note comments to update document example.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@vmatyusGitHub can we update the byte stream comments in the other SDKs?

Updated also other language examples and unified the variable names to documentAsByteStream:
node:

https://github.com/IBM/cloudant-node-sdk/pull/267/commits/b81b399dcd419e3f9c3a18b7153a936a9caca92c
https://github.com/IBM/cloudant-node-sdk/pull/267/commits/4e3978298dd0c92ec4db77d09a611e54b56786c6

go:

https://github.com/IBM/cloudant-go-sdk/pull/140/commits/23a277d9c76f962fa59d1111bdb5be4abfb5bacf

java:

https://github.com/IBM/cloudant-java-sdk/pull/117/commits/3fff1fe1f9aa89e902cbd6d71c8051d805030c12

Thank you for reviewing! 🙂


Feature/di friendly server

Sync with master.
Updating to governator 1.10.4

NetflixOSS » eureka » eureka-pull-requests #473 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » eureka » eureka-pull-requests #474 SUCCESS
This pull request looks good


Storybook preview hooks can only be called inside decorators and story functions.

I am getting the following error when using this plugin.
Storybook preview hooks can only be called inside decorators and story functions.

I am using . . .
{
    "@storybook/react": "^6.3.12",
    "@react-theming/storybook-addon": "^1.1.3"
}

I am also having this error. I am using...
{
    "@storybook/react": "^6.4.9",
    "@react-theming/storybook-addon": "^1.1.3"
}

I am using it with MUIv5 and I am defining my preview.js as follows
// preview.js
import { theme as OverallTheme } from '../src/theme';

import { createTheme } from '@mui/material/styles';
import { ThemeProvider } from '@mui/material';
import { withThemes } from '@react-theming/storybook-addon';

const providerFn = ({ theme, children }) => {
  const muTheme = createTheme(theme);
  return <ThemeProvider theme={muTheme}>{children}</ThemeProvider>;
};

export const decorators = [withThemes(null, [OverallTheme], { providerFn })];

Another one facing the same issue, any progress on this?
@austinmccalley and @clayrisser did you find a solution?

No solution yet ☹

The plugin looks so nice but I cannot use it because of this issue. Any workaround that would help?

@usulpro we could very much use your assistance

@totszwai This addon is being maintained. Not sure TS definition is related to this issue, but anyway PRs with fixes are always welcomed

Same problem here. From a preliminary look it seems that context is not being passed in like it is in other decorators. Anything that relies on storybook context, like useArgs(), triggers the bug.

could be related to this issue——https://github.com/storybookjs/storybook/issues/12006
basically the decorator renders it like this:
<>{providerFn({ theme, children })}</>

making that internal react JSX the “root” of the story as far as Storybook is concerned.
Storybook doesn’t allow the use of any of its cool toys less than one level deep into a component tree. so this is what breaks it.


added initial architecture diagrams to docs

#5

Not sure if already discussed with @Cottand but is there any benefit in having SSH access to a bastion on a non-standard SSH port?

Working towards: https://github.com/shiftcollective-org/Shift-Collective-Projects/issues/5

Not sure if already discussed with @Cottand but is there any benefit in having SSH access to a bastion on a non-standard SSH port?

Generally I don't do that, because with sshguard + disabling root login + disabling password authentication, the risk is essentially null.
That said, since here we are going to anyway automate the process (so you don't even need to change ~/.ssh/config, I am not against it. We just need to pick another port (2222?)

for ingress I would argue it is easier to use an ingress revers proxy like traefik from the start, because it allows us to not pin workloads to specific nodes with open ports.

@Cottand Once in k8s, this is not a problem anymore (it's one of the downsides of Docker, in fact!). Ingress (controller) will need to be bound to specific nodes (if we use the host-port strategy), but general workload will expose its ports only within the cluster itself (through services ).
Traefik can be used as ingress-controller though (https://doc.traefik.io/traefik/providers/kubernetes-ingress/), but every controller is basically identical (nginx-ingress-controller is the standard). Whatever controller we use, it will sit within the cluster which means we can scrape metrics with Prometheus. The only advantage of traefik (I think) is that it has built-in Let's encrypt integration for Certificates renewal, but we probably will need cert-manager anyway, which can be consumed by nginx-controller too.
I am not against using traefik, we might as well do it (I use it for some of my setup, in fact), but there is functionally no difference depending on which controller we use.

Sounds good, I am not that opinionated for traefik, I just read the diagram as 'at first we won't use a controller' which stood out to me

Probably I should change the diagram, I looked at it again and it does sound ambiguous. What I meant there is that to run the ingress pods themselves (i.e., traefik/nginx/etc. pods) we can use host ports (and not a cloud Loadbalancer).  Basically I am referring to this option (the doc is for nginx, but the problem is identical for whatever controller we pick).

Just a nit and unrelated to the content itself, but maybe instead of "architecture files" you can name the folder just "architecture", or "files/architecture" or similar? Having a url path with a space in it just looks weird to me 🤔

Good point, I usually stay away from spaces. I'll update and push in a few

Probably I should change the diagram, I looked at it again and it does sound ambiguous. What I meant there is that to run the ingress pods themselves (i.e., traefik/nginx/etc. pods) we can use host ports (and not a cloud Loadbalancer). Basically I am referring to this option (the doc is for nginx, but the problem is identical for whatever controller we pick).

Nice workaround to deploy the ingress controller. As the documentation says, we can use a specific node pool to deploy nginx as a daemonset, and use ClusterFirstWithHostNet dnsPolicy to be able to solve internal service names.
Awesome diagrams! 👍


Chaos 5381 review

CHAOS-5381 docs review

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: SmritiSatya:x: ispeakc0deYou have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66424519ddc18b0f7f753fd8--harness-developer.netlify.app


Disabling ios 'contacts permissions prompt' while running mobile-spec

Work still in progress.
related to: https://github.com/apache/cordova-plugin-contacts/pull/108
The associated binary file (TCC.db) contains records of apps that have permissions to access 'contacts'.
Here's a dump of its contents:
omefire-Mac:TCC omefire$ sqlite3 TCC.db "select * from access"
kTCCServiceUbiquity|com.apple.CloudDocsDaemon|0|1|1||
kTCCServiceUbiquity|com.apple.Accounts|0|1|1||
kTCCServiceUbiquity|com.apple.PassKit|0|1|1||
kTCCServiceAddressBook|org.apache.mobilespec|0|1|1||

ToDOs:

Hook this code up so that it gets executed as one of cordova-medic's setup steps
Grab the mobile-spec app name at runtime, instead of setting it in stone

This PR seeks to disable the prompt that requests permissions to access the contacts when running mobile-spec on the ios platform, for the cordova-plugin-contacts plugin.

This approach turns out not to work either, just as the first approach: https://github.com/apache/cordova-plugin-contacts/pull/108


Use ConfigureHostConfiguration to initialize GenericHost.

GenericHost initialization steps are as follows.

Build HostConfiguration
Create HostingEnvironment
Build AppConfiguration

ref: https://github.com/aspnet/Extensions/blob/release/2.2/src/Hosting/Hosting/src/HostBuilder.cs#L110-L114
Currently, the default builder for BatchHost uses AppConfiguration only. However, I think it should use HostConfiguration to initialize HostingEnvironment according to the standard way of GenericHost.

you're right, thanks!


could not invoke CustomCropManager.crop

on cropping image, shows error.

@Michaelvilleneuve any idea?, I have linked manually, here is the code.
import com.facebook.react.ReactApplication;
import com.horcrux.svg.SvgPackage;
//import org.opencv.RNCustomCropPackage;
import fr.michaelvilleneuve.customcrop.RNCustomCropPackage;
import com.facebook.react.ReactNativeHost;
import com.facebook.react.ReactPackage;
import com.facebook.react.shell.MainReactPackage;
import com.facebook.soloader.SoLoader;

Have you properly installed OpenCV as suggested in the Readme ? This step needs to be done manually

Thanks for the quick response, followed the same steps in read me, the cropper shows on clicking on the crop text in example, it shows this error.

same probem here, any sugestion?

@Michaelvilleneuve any suggestions?

Ok, obviously there seem to be a documentation problem, I am not sure which step I missed in the doc but I'll try to list the changes made to our project :
Here is our settings.gradle :
include ':react-native-perspective-image-cropper'
project(':react-native-perspective-image-cropper').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-perspective-image-cropper/android')
include ':openCVLibrary310'
project(':openCVLibrary310').projectDir = new File(rootProject.projectDir,'../node_modules/react-native-document-scanner/android/openCVLibrary310')

Since we are using OpenCV with react-native-document-scanner the openCV step is done once but you should be able to have the following :
package.json
"react-native-perspective-image-cropper": "^0.4.3",
 "react-native-svg": "^9.4.0",

settings.gradle :
include ':react-native-perspective-image-cropper'
project(':react-native-perspective-image-cropper').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-perspective-image-cropper/android')
include ':openCVLibrary310'
project(':openCVLibrary310').projectDir = new File(rootProject.projectDir,'../node_modules/react-native-perspective-image-cropper/android/openCVLibrary310')

notice the path change
build.gradle
implementation project(':react-native-perspective-image-cropper')
implementation project(':react-native-svg')

MainAplication.java
import fr.michaelvilleneuve.customcrop.RNCustomCropPackage;
import com.horcrux.svg.SvgPackage;

and then ...

new RNCustomCropPackage(),
new SvgPackage(),

Thing to note, though I seriously doubt this has anything to do with this error :
We changed our manifest to make it look like this :
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
   // The below line was added
    xmlns:tools="http://schemas.android.com/tools">

    <application
   // The below line was added
      tools:replace="android:allowBackup"
      android:name=".MainApplication"
      android:label="@string/app_name"
      android:icon="@mipmap/ic_launcher"

If you have followed exactly this, it may have something to do with the fact that we use openCV from my other lib react-native-document-scanner. In that case, you can try to install the android branch of react-native-document-scanner:
"react-native-document-scanner": "Michaelvilleneuve/react-native-document-scanner#android",

and use this OpenCV in settings.gradle
include ':openCVLibrary310'
project(':openCVLibrary310').projectDir = new File(rootProject.projectDir,'../node_modules/react-native-document-scanner/android/openCVLibrary310')

@Michaelvilleneuve same issue exits,i have followed the steps once again, let us find the exact problem. I have my source in this Dropbox link. Please check the source, it seems everyone has this issue, let's solve this issue.

Have you tried using react-native-document-scanner as source for open CV lib ?
I’ll take a look at your source code tomorrow

No, let me check that. Thanks

i think we should pass uri using react-native-camera package ,
cause our uri should look like this .
-> file:///data/data/com.lastscan/cache/Camera/a72a0abe-0dec-4bf1-9d54-30a5321f7e53.jpg
causes imgcodes.imread(imageuri.replace("file//,'') ,that type of function written in the bridge module ,
my assumptions -_-

Have you tried using react-native-document-scanner as source for open CV lib ?
I’ll take a look at your source code tomorrow
Tried react-native-document-scanner as source for open CV lib, same issue @Michaelvilleneuve

@zahidahmed024, may i know, how you have linked opencv manually, which you have mentioned in this link

actually i used https://www.npmjs.com/package/react-native-opencv3 this package
and  used index.js as component

,and paste the crop method and imports in RNOpencv3Module.java file

check my code
https://drive.google.com/open?id=1sR21BRm_X1Z_vid8GSLJ2F_zMa0bV79l
in updateImage method you will get cropped base64 image then use it where you want to .
i dont know weather it is right way or  wrong -_-

@zahidahmed024, after cropping image , crop() function can't return any values, it's return error.

any idea about this?

capture image from camera , and check in console.log that u are getting uri like
-> file:///data/data/com.lastscan/cache/Camera/a72a0abe-0dec-4bf1-9d54-30a5321f7e53.jpg .
i had that issue
see the  crop method. Imgcodecs.imread() taking uri that start with file:///
 fucntion https://github.com/Michaelvilleneuve/react-native-perspective-image-cropper/blob/master/android/src/main/java/fr/michaelvilleneuve/customcrop/RNCustomCropModule.java

@zahidahmed024 my log url is
content://com.re.provider/root/storage/emulated/0/Pictures/image-62b82ef6-8d2f-4822-943e-13c9dbbfc9c1.jpg
tried with replace "content" to "file" manually, throws same error, any suggestions

Have you tried using react-native-document-scanner as source for open CV lib ?
I’ll take a look at your source code tomorrow

@Michaelvilleneuve any suggestions regarding this issue.

@Sride hey i solved that issue  by providing image path ,, imgcodecs.imread() function can read original path . see the changes ,

@Sride actually this library is beautifully written by @Michaelvilleneuve  . but installing opencv in native android or react native is always a pain and image picker uri problem  -_-

Yes @zahidahmed024 this is unique library, thanks for giving such a library @Michaelvilleneuve, really opencv integration is a pain.

@zahidahmed024 I tried with your proposed solution, but it does not work :(

@DavitVosk
I found the solution.
In your MainApplication.java,
import org.opencv.android.OpenCVLoader;
public void onCreate() {
    super.onCreate();
    OpenCVLoader.initDebug();
   ...
}

Okay. After struggle fof 2days,here is how i solve the issue completely.

Install opencv310 manually and import the modules
Copy the lib file to jnilibs folder inside android main.
3.install react native opencv3
Edit the build gradle to implement your imported opencv module.

I'm getting issue when scanning is done the area in which scanning is done but the image is croppping somewhere else.

The easiest way to integrate openCV was to add  implementation 'com.quickbirdstudios:opencv:3.4.1' in my project build.gradle file and then export RNCustomCropPackage as native-modules.

@SL-A-SH  and @jackyleehong : Could you please explain more about the way you setup your project? Also if you have a repo, that would be helpful. I am struggling to set this up

Hi, please send your project. or uninstall
react-native-perspective-image-cropper, install "react-native-opencv3":
"^1.0.4",after that create, Custom crop component separately . then pass
the arguments through props to Custom crop component (where you
want 😊 👍).if any clarification please let me know.
On Tue, Nov 12, 2019 at 2:58 AM srinivas valekar<EMAIL_ADDRESS>wrote:

@SL-A-SH https://github.com/SL-A-SH and @jackyleehong
https://github.com/jackyleehong : Could you please explain more about
the way you setup your project? Also if you have a repo, that would be
helpful. I am struggling to set this up
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Michaelvilleneuve/react-native-perspective-image-cropper/issues/26?email_source=notifications&email_token=AIJMT7J4G4SZZVM43DVLWT3QTHFARA5CNFSM4H2RF6PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYGBPA#issuecomment-552624316,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJMT7OXPOBIAWWJYJFWEUTQTHFARANCNFSM4H2RF6PA
.

--
Thanks and Regards,
Elango.K

@ElangoPrince : Thanks for the info! This react-native-opencv3 looks like really an amazing project. I would let you know once I update my code-base. :)

@zahidahmed024 : i cant able to use library's function in ios. Please help me
i cant use .crop method in ios how can i add .crop method manually?

having the same issue and tried everything...
this lib just doesn't work :(

imgcodecs.imread()

In which file I can find the method imgcodecs.imread() ?

actually i used https://www.npmjs.com/package/react-native-opencv3 this package
and used index.js as component

,and paste the crop method and imports in RNOpencv3Module.java file

check my code
https://drive.google.com/open?id=1sR21BRm_X1Z_vid8GSLJ2F_zMa0bV79l
take photo using camera
in updateImage method you will get cropped base64 image then use it where you want to .
i dont know weather it is right way or wrong -- opencv is a pain -- .

I've not been able to

@Sride hey i solved that issue by providing image path ,, imgcodecs.imread() function can read original path . see the changes ,

in update function u will get your cropped image in base64 .
*sometimes imagepicker package gives uri like file:/// sometimes like content:// ,also uri changes when u put your phone on usb , i donno if u faced this situation too -_-

I still haven't been able to success. Could you please tell me if you had previously installed opencv in Android Studio? Do you have a working example I can use? Thanks in advance.

@Sride hey i solved that issue by providing image path ,, imgcodecs.imread() function can read original path . see the changes ,

in update function u will get your cropped image in base64 .
*sometimes imagepicker package gives uri like file:/// sometimes like content:// ,also uri changes when u put your phone on usb , i donno if u faced this situation too -_-

I still haven't been able to success. Could you please tell me if you had previously installed opencv in Android Studio? Do you have a working example I can use? Thanks in advance.

@DavitVosk
I found the solution.
in the CustomCropModule.java
public RNCustomCropModule(ReactApplicationContext reactContext) {
  super(reactContext);
 OpenCVLoader.initDebug();
  this.reactContext = reactContext;
}

you also have to install opencv using this script
wget https://sourceforge.net/projects/opencvlibrary/files/opencv-android/3.4.1/opencv-3.4.1-android-sdk.zip
unzip opencv-3.4.1-android-sdk.zip
cd android/app/src/main
mkdir jniLibs
cp -r ./../../../../OpenCV-android-sdk/sdk/native/libs/ ./jniLibs
cd ../../../../
rm -rf opencv-3.4.1-android-sdk.zip
rm -rf OpenCV-android-sdk/

@ethienneroy this increases size of my final apk by 10MB 😧 (for every build of the separate builds)

For me, this happens only on android.
When I take photo using RN Camera, it returns "could not invoke CustomCropManager.crop".
But when I take photo with @woonivers/react-native-document-scanner it works fine.
Image path returned from RN Camera is:
"file:///data/user/0/com.my_app/cache/Camera/351b8cab-eb13-4851-918a-0e7218c7278a.jpg"
Image pathe returned from scanner is: "file:///data/user/0/com.my_app/cache/documents/351b8cab-eb13-4851-918a-0e7218c7278a.jpg"
The only difference is in these paths, everything else is the same.

import  org.opencv.android.OpenCVLoader;

  public RNCustomCropModule(ReactApplicationContext reactContext) {
    super(reactContext);
    OpenCVLoader.initDebug(); // --> add this line
    this.reactContext = reactContext;
  }

Adding this "RNCustomCropModule.java" worked for me.

Hi guys,
I have forked this lib and fixed this issue. Also added some improvements.
https://github.com/Poojan31/react-native-perspective-image-cropper

Hi guys,
I have forked this lib and fixed this issue. Also added some improvements.
https://github.com/Poojan31/react-native-perspective-image-cropper

I'm getting this in Anddoi-
cv::Exception: /Volumes/Linux/builds/maser_pack-android/opencv/modules/imgproc/src/color.cpp:7943: error:(-251) scn ==3 || scn == 4 in function void cv::cvtColor(cv::InputArray, cv::outputArray, int, int)
I am using your forked library. Can you help with this?

which is way lighter..

Can you pls share location where you add this. I'm using perspective-cropper pkg

In IOS After Capturing image cropping is not working properly. Anyone has managed to solve this issue? Thanks in advance..

Hi guys, I have forked this lib and fixed this issue. Also added some improvements. https://github.com/Poojan31/react-native-perspective-image-cropper

In IOS After Capturing image cropping is not working properly. Anyone has managed to solve this issue? Thanks in advance..

For anyone else running into this annoying error, it seems that the documentation is just wrong. The initialImage prop on the CustomCrop component expects a URI (not the base64 image as the docs suggest)  while the updateImage prop actually receives a b64 string with the cropped image... We were having issues with this component on both Android and iOS, and changed the initialImage prop to be a file URI, and (at least the crop function) is now working as expected.


devnum mismatch UBoot vs. Kernel

We are using meta-tegra to build a custom image for our jetson-tx2 and stumbled upon a problem when booting the rootfs from the SD card:
UBoot considers the external SD card to be device number 1 (interal eMMC is 0), but once the Kernel boots the SD card is mapped to /dev/mmcblk2
Since extlinux.conf uses the {devnum} placeholders for the root= image, the mounting fails and we have to modify the extlinux.conf. Is there a chance to have the Kernel address the external SD card as mmcblk1, so the numbering matches UBoots numbering?

Boot logging looks like this:
[    4.624690] mmc0: SDHCI controller on 3460000.sdhci [3460000.sdhci] using ADMA 64-bit with 64 bit addr
[    4.632716] mmc1: SDHCI controller on 3440000.sdhci [3440000.sdhci] using ADMA 64-bit with 64 bit addr
[    4.644879] mmc2: SDHCI controller on 3400000.sdhci [3400000.sdhci] using ADMA 64-bit with 64 bit addr

Looks like NVIDIA started enabling mmc1 for the M.2 slot in recent L4T releases, which is what's causing the numbering to be out of sync with U-Boot.  I've patched U-Boot to match the kernel's numbering and added those changes to master.

You can also work around this by setting KERNEL_ROOTSPEC in your local.conf to replace the devnum U-Boot variable reference with a hard-coded 2 if you know you'll be booting the image you're building off the SDcard.

The Jetson-TX1 dev kit also suffered from sdhci device numbering compatibility issues, so I have also updated U-Boot for that platform.


ITMS-90809:

ITMS-90809: Deprecated API Usage - App updates that use UIWebView will no longer be accepted as of December 2020. Instead, use WKWebView for improved security and reliability. Learn more (https://developer.apple.com/documentation/uikit/uiwebview).

This project does does not use UIWebView. Please make sure you are using the latest version.

It has been updated to the latest version，thanks a lot.


visualvm: 2.1.2 -> 2.1.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/visualvm/visualvm.src/releases.
meta.description for visualvm is: "A visual interface for viewing information about Java applications"
meta.homepage for visualvm is: "https://visualvm.github.io"
meta.changelog for visualvm is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.1.4 with grep in /nix/store/z36cr4wp6lxwlxcr324lgcishl0i2w17-visualvm-2.1.4
found 2.1.4 in filename of file in /nix/store/z36cr4wp6lxwlxcr324lgcishl0i2w17-visualvm-2.1.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
visualvm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/z36cr4wp6lxwlxcr324lgcishl0i2w17-visualvm-2.1.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A visualvm https://github.com/r-ryantm/nixpkgs/archive/a98fc8a64c6a254260d16fdd230d5ff92bb748ed.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z36cr4wp6lxwlxcr324lgcishl0i2w17-visualvm-2.1.4
ls -la /nix/store/z36cr4wp6lxwlxcr324lgcishl0i2w17-visualvm-2.1.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

visualvm

Maintainer pings
cc @michalrus @moaxcp for testing.

Briefly tested on x86_64-linux.


Hosting should take in hosting file name as an argument

This would allow us to do the following.
    "web": "Microsoft.AspNet.Hosting --config hosting.ini",

The code that needs to be changed is https://github.com/aspnet/Hosting/blob/5b851c49a337e49001ed2351253ab0f52b696040/src/Microsoft.AspNet.Hosting/Program.cs

@muratg @Eilon This is for Beta5 from the templates review

https://github.com/aspnet/Hosting/issues/277

@lodejard

@rustd Template change for the web command should be unblocked now.


Fix gosec issue: Use of unsafe calls (go pointers)

Review the usage of unsafe pointers in code. Unsafe pointers can introduce security risks as they expose C-like pointers.
In some cases (e.g., direct syscalls and ioctl ) it is required to use unsafe pointers but the usage need to be carefully reviewed.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

"NONE"

/hold

/unhold

/lgtm

/retest

/retest

/approve


Made changes to migrate aws_subnet resource from v1 to v2

Made changes to migrate aws_subnet resource from AWS SDK v1 to v2 and fixed the specs for the same.
Signed-off-by: NAshwini<EMAIL_ADDRESS>

I have made the changes as per review comments
Thank you @NimishaS , @Vasu1105 and @harikesh-kolekar for your valuable input.


Provide way to link job results with Investigation

My understanding of performing analytics within Atlascope is that this is primarily done with the use of jobs, which will eventually need to be spawned by users from the front end.
This places these jobs within the context of an investigation, and therefore I believe there should be a way to obtain Jobs and job output datasets for a single investigation.
Right now we have GET /jobs. From this endpoint, I don't see a clear connection to job results or the investigation from which that job was launched.
We also have GET /datasets. Some datasets in the populate script are created by jobs, and this is reflected in the dataset's metadata. However it doesn't include the ID of the job, or the investigation from which this job was spawned.
I think these connections are all useful. Connecting a Job to its output (via new tables or more specific metadata) provides additional information about why those Datasets exist. Connecting the Job to the Investigation makes it easier to determine what the system should present to a user when looking at a specific Investigation.
@annehaley @waxlamp Interested in hearing your thoughts on this. Or if a solution already exists that I can't seem to find.

I think you're right, and there are definitely places I overlooked the need to specify the context Investigation on other models. I tried again at a class diagram (shown below). I want to be sure we cover all relationship paths before implementation. Are there any relationships we need to access (remember each of these arrows represents a forward relationship, but the API can expose reverse relationships)?

This looks great! Connecting everything back to an Investigation is going to be really helpful moving forward.
Some relationships I know will be handy for jobs are:

All Jobs for a given Investigation
All input/output Datasets for a given Job

Those two as relationships would allow the client to display only relevant jobs to a user looking at an investigation, and display the results of those jobs based on the resultant datasets.
Endpoints that expose those relationships would be enough close this issue.


Add applicationinsights/data-plane/readme.typescript.md

Autogenerated

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Waiting on @daschult's review before I take further action.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go


MathUtils infinite loop

There is an infinite loop here caused by a misnamed variable:
https://github.com/newhoryzon/farmers-delight-fabric/blob/master/src/main/java/com/nhoryzon/mc/farmersdelight/util/MathUtils.java#L21

Deployed in 1.3.4 for MC1.19.X and in 1.2.2 for MC1.18.2 !


support for id, and name attributes on types

example:
openapi: "3.0.0"
components:
  schemas:
    products.inventory:
      type: object
      properties:
        id:
          type: string
        name:
          type: string
      x-seal-actions:
      - allow
      - deny
      x-seal-verbs:
      - inspect
      - use
      - manage
      x-seal-default-action: deny

@rdv821

tags will be done in https://github.com/infobloxopen/seal/issues/39


multi: restructure packages.

This ultimately fixes a design issue that causes a cyclic import error with database upgrades.

testing on dcrpool mainnet instance, looks good so far

Needs rebase

looking good


[CircularProgress] Port component

[ ] Component
[ ] Tests (at least unit tests)
[ ] Docs
[ ] Demo
[ ] Fix #4747

There are a few open CircularProgress PRs, but this stands out as worth trying to tackle when migrating: https://github.com/callemall/material-ui/issues/707

@mbrookes thanks, 100%!!!

I've managed to reproduce this component with keyframes, maintaining all flexibility except thickness. FFS.
I have an idea for thickness -- but it would involve creating multiple sets of keyframes (for each thickness). Any opinions?

I guess it will depend on how cumbersome it becomes... thickness isn't in a released version yet, so worst case if we had to drop it, it wouldn't be completely the end of the world.

Would it be possible to add a "loaded" prop to the progress components such that when loaded is false the progress component shows and loaded is true the progress component hides?

@turnerniles What the advantage of having this logic in the component instead of in userland? What you described seems pretty common to every component. Wouldn't it be better to use composition. Does it involve a transition?

@oliviertassinari I think it is specifically applicable to the linear and circular progress components since anyone who is using a progress component will most likely want to hide or remove the component once a load is complete.
Yes it is easily done in composition; however it would be convenient to have the logic in the component.
In the angular material progress component there is the attribute ng-disabled which hides the component when true:
https://material.angularjs.org/latest/api/directive/mdProgressCircular
What do you think?

@turnerniles Sounds like what to propose is to allow the following pattern:
<CircularProgress opened={opened} />

Instead of the following one
opened & &  <CircularProgress />

I can't see any advantage here. Actually, you probably want some more logic. E.g. Displaying the loader after X ms for at least Y ms. That logic would be better off in another module, as has nothing to do with the rendered component.
Any light?
Another topic now, I think that we should remove the thickness property from the master branch for consistancy with the next one.

Yeah, I don't think that having an opened/loading state is a good idea, it is unnecessary given the simplicity of implementing it in userland.

@oliviertassinari, @nathanmarks ok, makes sense

Regarding the thickness prop that's currently just a const.
It should be possible to achieve that using jss function-values. However i didnt find a way to access that update method when using jss-theme-reactor, or the ability to use dynamic-rules.

@slavab89 I think that we have to upgrade jss first.

That could be useful in a few places we've had to revert to inline styles for dynamic behaviour.

CircularProcess is in the docs page of current alpha docs https://material-ui-1dab0.firebaseapp.com/component-demos/progress
Probably "Component", "Docs", "Demo" can be checked

I'm closing, let's fix #5524 aside


Kodi 18 RC5.2 Android Netflix plugin gives a python error

I'm submitting a ...

[ ] support request

[x] Are you running the latest version?
Yes

Installation

[x] I´m using the Netflix Repo

Operating System

[x] Android

Additional informatin on the environment
Xiaomi MiBox S
Debug log
https://paste.kodi.tv/rigekodimu.kodi

Same issue here, also on a Mi Box S, with both "Connection refused" errors and "TypeError: b2a_base64() argument 1 must be string or buffer, not None" error. Seems like there are several related issues around here, probably something has changed in Netflix and needs to be addressed in order for this plugin to work. Is the plugin working currently for anybody?

The plugin is working for me, on Zidoo H6 Pro, which is Android 7.1. It does not work on Xiaomi MiBox S and Mecool KM9, which are Android TV 8.1, giving the same widevine errors, so i guess it has something to do with the android version 8.1 or Android TV....

In Xiaomi Mi Box 4K it's the same.


Proposal: Knative E2E Test Framework

Please see presentation: https://docs.google.com/presentation/d/1x-OJAQqAW7l5GDCO1WHWOIP7OMdbGXAYDN0uFtMGRlA/edit#slide=id.p

/assign

What do you think of renaming:
Global Environment -> Environment
Environment -> Rig
Rig as in test rig... I still like that concept that the rig is the thing you are strapping your device under test into to perform the test...

@n3wscott nah, GlobalEnvironment and Environment are far more immediate IMO

@n3wscott I would also lean the GlobalEnvironment and Environment way. It's more clear what they represent, and having Global in the name does bring about extra caution when writing tests, i.e., it's well understood you should never be tampering with global state.

Looks pretty good. I put some minor comments.

thanks for the review @tayarani !! I will fix this up and maybe we can land this first version.

/lgtm
/hold
since others have comments, remove if you're fine...

/assign @lionelvillard

/unhold


httputil/header: unescaping fix

It wasn't failing tests because i == 1 in each test case.

Seems like a good fix to me.


Multi-byte characters in URLs are now supported.

added support for multi-byte characters in locale url segments

Not sure why tests are breaking. They are passing on my local.

@robbieaverill Can you please merge this into 4.1 as well?

@mfendeksilverstripe done =)

@robbieaverill Nice, thanks :)


Strip net 4.5.2. Upgrade libraries to latest.

This should also address the issue https://github.com/AsynkronIT/protoactor-dotnet/issues/460

@rogeralsing Should I go ahead and merge it? 4.5.2 will not be supported after the merge = )


Visualize Hydroelastic and Point Contacts (continued from #25)

This brings @gbalke's PR #25 up to date with both the latest changes in Drake and Drake-ROS.
Quite a bit has changed, so I changed the demo to get contact information from a multibody plant instead of the scene graph. I added the ability to visualize point contacts (with inspiration from gazebo it uses a ball for the contact point and a line for the contact normal). I mimicked ContactResultsToLcmSystem and ConnectContactResultsToDrakeVisualizer() as ContactMarkersSystem and ConnectContactResultsToRviz().
 The lower ball is very soft and compliant and the upper ball is rigid. The slope is also rigid. Hydroelastic collisions are present in two places: between the compliant and rigid balls, and between the compliant ball and the slope. A point contact is present between the rigid ball and the slope.

ros2 run drake_ros_examples collisions

rviz2 -d src/drake-ros/drake_ros_examples/examples/collisions/hcollisions.rviz

Issues

The texture on the hydroelastic collision mesh is only visible in one direction. On the other side it's all black. I don't know why.

This change is

Which OS / ROS distro should I use for this? Focal+Galactic, Jammy+Humble, or Jammy+Rolling?

Noted the following on 90a770b (slightly older rev)

The rate of update between sphere and contact patch is different - why?
On restart, the spheres don't move?
Ignore comment about only centroids moving, didn't observe that correctly :facepalm:

https://user-images.githubusercontent.com/26719449/190832508-ca9a9ab9-1572-4f09-a0e8-ce232c7ebd79.mp4

I think we could use the drake find results, configure a file, and then have that put Drake on LD_LIBRARY_PATH for CMake env?

We should be able to make an ament hook for this.

Sorry :(.  We haven't finished vendoring VTK yet in Drake (https://github.com/RobotLocomotion/drake/issues/16502).  Hopefully it'll land within a month or two.

@sloretz Is this something we could land soon? (tomorrow, if possible?)

Thanks!


Update footnote 4 in tour/roadmap page to link to actual merged PR

The original PR linked was closed in favor of a later one, which is of course reachable from it. However, it's nicer for the link to just bring readers directly to the correct PR right away.

appreciate the catch @davidreghay ! Would you mind signing your commit? we have it turned on an org-wide thing, sorry for the hassle for a one-liner

Yeah, no problem @alixander I'll find some time for that tomorrow. Happy to help!

Ok @alixander I've overwritten the commit with a signed version 👍

thanks @davidreghay !


fix: Fixed format of env file

Changed format so it works with docker compose v2

Changed format so it works with docker compose v2

You skipped export KRATOS_FEATURE_FLAG="true" and export CUSTOM_MODULES="default"

Isn’t this redundant? We are using .envrc https://github.com/GaloyMoney/web-wallet/blob/main/.envrc.
Perhaps you just need to run ‘direnv allow’

Isn’t this redundant? We are using .envrc https://github.com/GaloyMoney/web-wallet/blob/main/.envrc.
Perhaps you just need to run ‘direnv allow’

I was just talking to @Extheoisah about this.  We both have a local .env file despite it not existing on main.  I must have created it manually months ago before docker compose supported .envrc.  Going to close this PR and switch to use .envrc.


add ability to disable/remove modules

example:
{
	"disable": [
		"aspect-ratios",
		"tables"
	]
}

This should already work, from the readme:

You can also omit the partials you don't need with the key skipModules, for example if you don't want normalize.css in the bundle you can do:

{
  "skipModules": ["normalize"]
}

Yup, it works as the example


What does it replace exactly ?

https://github.com/awslabs/amazon-eks-ami/blob/eb0239fb52f524d8c1da2bb91d4c9df1190e1e29/files/bootstrap.sh#L98

That command looks like it replaces the string CLUSTER_NAME with the ENV var $CLUSTER_NAME at https://github.com/awslabs/amazon-eks-ami/blob/eb0239fb52f524d8c1da2bb91d4c9df1190e1e29/files/kubelet-kubeconfig#L23

This inserts the cluster name into the kubeconfig so authenticator knows which cluster to authenticate to


fix variable declaration for fs in host builtins check

Description
This pull request addresses an issue with the variable declaration of fs in the host builtins availability check.
Changes Made

Updated the declaration of the fs variable from let fs inside the try block to fs = to ensure that the variable is properly scoped and accessible outside the try-catch block.

Thanks


Upgrade Google Cloud libraries-bom to 25.4.0

Companion camel-karaf PR here: https://github.com/apache/camel-karaf/pull/169.


Feature: Add ability to generate bindle versions with prerelease info

Add a --pre bool flag to spin bindle prepare|push, wherein, if supplied, spin adds prerelease info for a resulting name/version-<prerelease data> bindle ID. The end result must be a unique, sortable, semver-compliant bindle ID.

One common source for prerelease data that lends itself to ordering is a timestamp.  The Hippo CLI uses this approach.

I believe this is accomplished with the generation of build metadata.  Please reopen if it needs further action!


Some kernels and tests were not ROCm enabled.

tensorflow/core/kernels/constant_op_test.cc
tensorflow/core/kernels/cwise_op_bessel.cc
tensorflow/core/kernels/cwise_op_bessel.cu.cc
tensorflow/core/kernels/cwise_op_zeta.cc

@whchung do you know if cwise_op_zeta had been previously disabled on purpose?  The ROCm build is failing with nans for zeta.  Test is cwise_ops_test.py line 712.
@sunway513 you had asked for a separate PR for enabling these ops but it looks like zeta is failing at the moment.  Need to investigate further.

@jeffdaily i'm not aware of the history of why cwise_op_zeta was disabled. If you check its history you can see GPU support was added very recently.


Print styles prevent SVGs from printing well

By default you just get solid black SVGs, regardless of print settings. In my apps where I want to print SVGs, I've had to override the foundation-print-styles() mixin changing the CSS universal selector here…
https://github.com/zurb/foundation-sites/blob/ff60ab247e9e3995d8a004c1aaae126ce7bb9b46/scss/typography/_print.scss#L20-L21
to this…
 @media print {
   *:not(svg) {

Hi @andycochran,
It seems you have deleted our issue template.
Please fill it out and provide all details.

PS: Why not use fill instead of currentColor?

I would say it is not a bug (black is better for printing).

Closing as this might be the wished behavior by default and it can be overwritten in projects.

black is better for printing

🤔
When I said "you just get solid black SVGs" I didn't mean e.g. that a red icon prints black. I meant that Foundation's print styles cause the entire SVG view box to be black (not just what's drawn in it). It's not that it's the wrong color. You cannot see the SVG's content at all; it's just a solid black box.

It probably depends on the svg. We could still see the icon shapes but with black fills in projects afaik. color should only affect shapes which have currentColor in an attribute.

If this is not the case please provide a reproducible testcase.


Non-dev dependency in ferry_generator package

I have the following in my generated serializers.gql.dart:
import 'package:gql_code_builder/src/serializers/operation_serializer.dart'
    show OperationSerializer;

This ends up imported into non-dev code that is used when making the actual GraphQL calls, but gql_code_builder is a dependency of ferry_generator which is a dev-dependency of the end-code according to the docs: https://ferrygraphql.com/docs/setup
This is not a problem when ferry_generator is directly included into pubspec.yaml of an app because dev dependencies are still pulled there.
However, when using ferry_generator as a dev-dependency in a package, it's not pulled into an end-app because dev-dependencies are not pulled recursively.
This breaks the build for me with this error despite all pub get run successfully:
Error: Couldn't resolve the package 'gql_code_builder' in 'package:gql_code_builder/src/serializers/default_scalar_serializer.dart'.
Error: Couldn't resolve the package 'gql_code_builder' in 'package:gql_code_builder/src/serializers/operation_serializer.dart'.
../dart_gh/lib/src/graphql/__generated__/schema.docs.2024-05-01.schema.gql.dart:9:8: Error: Not found: 'package:gql_code_builder/src/serializers/default_scalar_serializer.dart'
import 'package:gql_code_builder/src/serializers/default_scalar_serializer.dart'
       ^
../dart_gh/lib/src/graphql/__generated__/serializers.gql.dart:620:8: Error: Not found: 'package:gql_code_builder/src/serializers/operation_serializer.dart'
import 'package:gql_code_builder/src/serializers/operation_serializer.dart'
       ^

The workaround is simple: ferry_generator should be included as a non-dev dependency of the package. However, this brings redundant dependencies and breaks a lot of the original ideas (like the idea to separate ferry and ferry_generator in the first place).

Thanks!
You're right, this should be fixed, probably the serializers should move to another package.
For now, I suggest adding gql_code_builder as a direct dependency instead of the entirety of ferry_generator.

please check out ferry_generator 0.12.0-dev.1
note that you need to add gql_code_builder_serializers, which a package that just contains the common serializers that the code generation assumes exist


B<1> is parsed as markup in a verbatim paragraph

In a verbatim paragraph (i.e. one starting with one or more spaces) I would expect that markup is ignored - but it isn't.  The screenshot shows the resulting syntax highlighting and parse result in Emacs.

latest commit on #3 improves this; but I think it's still not quite correct.
Behavior as implemented won't end the verbatim-ness until there's a double newline. Is that correct to spec? I think it's easier to ask than look b/c I'm not actually so fluent in POD

With that patch applied it works as I'd expect it.  I don't even observe a need for a double newline if the next line is a valid POD command.  Per spec, an empty line is needed to separate paragraphs, and other parsers like pod2html will not end verbatim if the next line has a POD command without leading spaces.
Nitpick: Your parser doesn't require a space between =head1 and NAME


SMS > Send an SMS

See Gist:
https://gist.github.com/adambutler/fdfeba152303388adc3fbc7c99272123

https://gist.github.com/RabebOthmani/e103287be2f0a11b115cd22e9ba0e223

👎 Please see specification.

@adambutler I updated the GIst, could you please check your happy with it now? Thanks

Had a quick look and looks good. I'll implement on Monday and report back.


Is there a way to have partial slides on either side of the active?

I can get a partial slide on the right if I use a fraction for visible - but is there a way to have the slides on both sides of the active slide(s) be partially visible?

Any special reason for closing this issue? I´m curious..


Screen Blackout

Microsoft PowerToys version
idk
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
just use your shitty computer and it happens....3rd time I think
✔ Expected Behavior
the computer I bought for $2,000 in 2021 to fuction
❌ Actual Behavior
Screen randomly blacks out, sometimes resets, this time turned on after a minute
C:\Users\Atkin\AppData\Local\Microsoft\PowerToys\PowerToys Run\Logs\<IP_ADDRESS>\2024-12-04.txt
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19045.0
IntPtr Length: 8
x64: True
Date: 12/4/2024 2:25:31 AM
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
Other Software
No response

Thanks! Tracking on another issue.
/dup #30507


json serialization compatible problem

For example, in java interactions, fields of type String are serialized directly to sds instead of /"sds/", and using serde results in an error when serialization fails
expected value at line 1 column 1
Hopefully this is compatible

Can not reproduce this on my end. Using the latest version of serde and serde_json and this simple test code:
use serde::{Deserialize, Serialize};
#[derive(Serialize, Deserialize)]
struct Test {
    sds: String
}
fn main() {
    let test = Test { sds: "sds".to_string() };
    let res = serde_json::to_string(&test).unwrap();
    println!("{}", res);
}

yields:
{"sds":"sds"}

as expected, please provide more information and perhaps an simple example illustrating your issue.

like
    let str = "sds";
    let str_json = serde_json::to_string(str).unwrap();
    println!("{:?}", str_json); //   "\"sds\""
    let res: Result<String, serde_json::Error> = serde_json::from_str(str);
    println!("{:?}", res); //  Err(Error("expected value", line: 1, column: 1))

serde_json::from_str()  deserialize a structure from a json string. I don't think
sds

is a valid json string.

Maybe you have confused the variable str with str_json?

Oh, I guess you have confused "the string type in json (as well as in javascript)" with "a string that is in the json format", am I right?

I have a project that communicates with Java projects, and in some Java json parsing libraries, the String type is treated directly as
sds
Instead of
\"sds\"
so
let res: Result<String, serde_json::Error> = serde_json::from_str("sds");
println!("{:?}", res); //  Err(Error("expected value", line: 1, column: 1))

let res: Result<String, serde_json::Error> = serde_json::from_str("\"sds\"");
println!("{:?}", res); //  Ok("sds")

Now I receive a Java response that I must process before I can use serde_json
like
pub fn json_field_compatible(ty: &str, mut field: String) -> String {
    if ty == "String" && !field.starts_with("\"") {
        field.insert(0, '\"');
        field.insert(field.len(), '\"');
    }
    field
}

... the String type treated directly as sds instead of "sds"

Such implementation would be incompatible with the JSON format specification, as both  RFC 8259 and ECMA-404 define string as:
'"' characters '"'

Here is a simple JSON format verifier if you don't want to look into the specs (remember to uncheck the Fix JSON option as this will enable auto-correction).

i know  , so i hope have like a "Fix JSON" Feature,

In that case, you probably should change your title to "Feature request: apply simple auto-corrections when deserialize", though I don't think that would be accepted.

Deserializing things other than JSON is out of scope for this library.


Show proper error messages during login.

Steps to reproduce

Go on to your Sync Endpoint server https://[Your_hosted_url]:40000
Create a user with Default group other than 'site admins' . For info see this
Try to login into the app with the aforementioned new user.
App will show this message "Invalid username/password, please try again."

Expected Behaviour
App should show proper error message with possible solution.

@opendatakit-bot claim

@opendatakit-bot claim --force


[Feature Request] Allow randomAmount option to be randomized

Hello there,
I really like this plugin and its ability to add a certain amount of random trades per config to a wandering trader. To enhance this feature, I would like to see an optional minimum and maximum value for the randomAmount option for each trade config. This would make the plugin more customization, and also introduce a chance for a config not to be picked even if it is enabled by setting the minimum randomAmount to 0.
Thank you for your time.

Added in b2b22b2
I also added this feature for the amount of player head trades.
You can download the snapshot build here


reflect: TypeOf(unsafe.Pointer(nil)).PkgPath() returns "", not "unsafe"

The documentation for reflect.Type.PkgPath says:
    // PkgPath returns a defined type's package path, that is, the import path
    // that uniquely identifies the package, such as "encoding/base64".
    // If the type was predeclared (string, error) or not defined (*T, struct{},
    // []int, or A where A is an alias for a non-defined type), the package path
    // will be the empty string.

unsafe.Pointer is not listed as a predeclared type: https://golang.org/ref/spec#Predeclared_identifiers (Though go/types lists it as one: https://golang.org/pkg/go/types/#BasicKind)
So it seems to reason that PkgPath should return "unsafe", but it instead returns "": https://play.golang.org/p/ZnUOgHjwz58
/cc @griesemer @ianlancetaylor

@mdempsky, thank you for catching this discrepancy, nice work! If I may ask, perhaps should we repurpose this as a proposal to include unsafe.Pointer into builtin but also to update the spec? I think this was just an oversight from commit https://github.com/golang/go/commit/20850fc014c27a9df4f4201844c0136725202800 almost 12 years ago in which we perhaps forgot to update the spec. @robpike @rsc @griesemer @ianlancetaylor is my assumption correct? Thank you.

unsafe.Pointer is mentioned in the spec. It's also declared in package unsafe, not the universe, so it's properly not listed as a predeclared type/identifier. I don't see any reason to change the spec here.

At first glance this seems like a compiler bug to me.  It's probably treating unsafe.Pointer as a predeclared type rather than as a type defined in the "unsafe" package, so it's probably generating an uncommonType struct with a zero pkgPath field.  Or something like that.

I can believe it's actually a compiler bug.
FWIW, it looks like with gccgo, both PkgPath and also Name return "".

For gccgo it's a libgo bug, because libgo holds the unique type descriptor for unsafe.Pointer (https://go.googlesource.com/gofrontend/+/refs/heads/master/libgo/runtime/go-unsafe-pointer.c).  But as far as I know the gc compiler doesn't work that way.

Sure. I'm just pointing out a similar issue also affects gccgo.


⚠ wakapi has degraded performance

In 46d5fd2, wakapi (https://waka.mcha.cloud/api/health) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: wakapi performance has improved in 6c623e9 after 3 minutes.


Bounce back if drag in quad crop mode

I choose to use quad crop and after taking a picture I want to select a quad crop of the picture, when I drag the picture it will bounce back, I think it's a little bit not user friendly. Is this a bug or I use it at a wrong way?
Appreciate your great work!

The imageView will bounce back to the last position where its rect contains the rect of the cropRect.

Not perfect solution.
https://github.com/oney/DBCamera/commit/ddf44f469d5e86597e5fd04b2c47ff9a0b342774


Make it possible to use async version of length_function

So due to the fact that the issue with the Elasticsearch support has been hanging since 2022, I've been trying to implement async ElasticSearch integration myself using async_paginator.paginate function.
It's kind of hacky, but can be done by creating a wrapper function:
new_search = Document.search()

async def _pagination_transformer(sequence: AsyncSearch):
    return list(await sequence.execute())

return await paginate(
    sequence=new_search,
    transformer=_pagination_transformer
)

The thing is that in the current implementation the length_function is sync, and there is no way to make a call of a new_search.count() coroutine inside it, because it's currently in the running loop. So the only solution I came up with is calling count() before paginate, and passing the result in a lambda function, but it's too much hacky, not to mention that it will be called even if raw_params.include_total = False. So having an opportunity to use async length_function would be very nice.

@mtsarev06 Makes sense to me, I will take a look

Hi @mtsarev06
New version 0.12.26 has been released. This issue should be fixed now.


add on update for my sql

Hi thanks for the awesome library and your work
Closes https://github.com/sqlparser-rs/sqlparser-rs/issues/504

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 89.813%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
8737

Relevant Lines:
9728

💛  - Coveralls


Append instruction to Getting started

In https://github.com/genesys-neu/ni-rf-data-recording-api/blob/main/docs/Getting_Started_Guide_of_NI_RF_Data_Recording_API.pdf Page 4, I suggest add below content.

I have tested, guide along with the Getting_Started_Guide_of_NI_RF_Data_Recording_API.pdf(In a brand new Ubuntu20.04). Error prompted:

Actually, this is because the python file in /usr/bin not linked to Python3.9.
So it needs a change(apped instruction to fix that link) for a better guide to users.

Hi, thanks for your comment. @Abdo-Gaber any chance we can include this information in the Getting Started Guide pdf?


sast-coverity-check: added stats for Coverity scans

Solves: https://issues.redhat.com/browse/OSH-769
Adding the stats to Coverity scans in the result's SARIF file of successful scans.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

This MR was discussed here: https://github.com/jperezdealgaba/build-definitions/pull/4
and we were waiting for #1411 to be merged to create this MR

@tnevrlka Would you mind enabling this pipeline please?

/ok-to-test

@dirgim Would you mind revieweing this? It is very related with: https://github.com/konflux-ci/build-definitions/pull/1633

/ok-to-test

@kasemAlem @dirgim Could you please check what happened that e2e-tests failed like this?
365 Skipped
--- FAIL: TestE2E (5476.37s)
FAIL

I do not have access to the corresponding logs myself.

@kasemAlem @dirgim Could you please check what happened that e2e-tests failed like this?
365 Skipped
--- FAIL: TestE2E (5476.37s)
FAIL

I do not have access to the corresponding logs myself.

Seems like the PLR logs are removed , can I rebase and rerun then we follow result ?

@kasemAlem Yes, please.  Note that I rebased this pull request 4 hours ago.  It did not help.

LGTM

/ok-to-test


Avoid --no-sandbox with a non-root user.

The Official Puppeteer Docker guidance includes various steps to avoid operating as a root user, which allows sidestepping the sandbox issues. I configured something based on the official docs, with a bit of flare for this project, by adding the following layers to my Dockerfile:
# Add user so we don't need --no-sandbox.
RUN groupadd -r pptruser && useradd -r -g pptruser -G audio,video pptruser \
    && mkdir -p /home/pptruser/Downloads \
    && chown -R pptruser:pptruser /home/pptruser \
    && chown -R pptruser:pptruser /screenshots \
    && chown -R pptruser:pptruser /usr/local/share/.config/yarn/global/node_modules

USER pptruser

We are using Jenkins so the build nodes are new every time and the latest image will be pulled from dockerhub.

@grayside would you please mind sharing your Docker file?
I have been all over the internet to figure out the correct way to run my node project which uses Puppeteer in Docker. No matter what I have tried I end up with,

Failed to move to new namespace: PID namespaces supported, Network namespace supported, but failed: errno = Operation not permitted

I copied in the exact Docker file from here and I still get the same errors.
Any help or examples would be greatly appreciated.
The other thing I tried was running as root instead of switching to  pptruser. When I do that it throws an error about needing to run with

--no-sandbox

I have not been able to figure out how to pass --no-sandbox to Docker when I run to get past that error. If you know how to overcome either one of these problems it would be helpful to us.

Here is the project: https://github.com/phase2/docker-pa11y
The current master branch is likely broken, this PR needs a bit more testing but better reflects what you would mimic: https://github.com/phase2/docker-pa11y/pull/8


Use Infura API v3

This PR updates the project to use Infura's v3 API.

I've created #33 to note that rate limits aren't supported yet

I've created #34 to note that currently requests cannot be cancelled.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220422155057574 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220422155057574 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 42.196%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
3
77.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
27752

Relevant Lines:
61529

💛  - Coveralls


Are there any newer / more active projects, or is this unmantained just because it's complete?

This is exactly what I needed for serving a few small HTML files with some ruby in them. But it seems that the repo has not been touched in 3 years, so I was wondering if there are newer ways to do this (which I couldn't find) or basically this is it and it has not been touched because it's feature complete? :) Thank you.

I had the same question and wanted to try and take this project over, contribute/fix. Didn't get a response from @migrs, maybe he's reading this? I want to start with #5.

I made pull requests and tried to contact @migrs by various ways. For now didn't get a reply, but I'll keep trying before forking. In the meantime I'll collect all the fixes in https://github.com/dblock/rack-server-pages/tree/next. Feel free to give it a try.

@jjgh @dblock I'm sorry I have left this project for a long time. I just merged @dblock 's PR(#5) and I will release new version asap.

@migrs If I can help with the maintenance I would be more than happy to! Feel free to add me to the repo/gem owners - I can make a release too.

@migrs and I have connected and the project is now maintained. I've added support for Rack 3.0 in #16 and will make a release soon.


Database architectures documentation

Your documentation about the different database architectures is wrong, I invite you to visit: https://learn.microsoft.com/en-us/azure/azure-sql/database/saas-tenancy-app-design- patterns?view=azuresql
Regards

Actually the pictures of my documentation are adapted from this link, and I already quote it as reference in https://github.com/go-saas/saas/blob/main/README.md#references
The shared means Catalog from the original one
The shared database may store more information then the catalog from the host side.
Would you please point out which part of the documentation is wrong or misunderstanding?


Work through "from 0 to proving with K"

The video documents the process of starting from an empty file and working up towards doing small proofs. The exercises come from here: https://github.com/ehildenb/understanding-k-framework
Complete the exercises, looking at the solutions as little as possible. Make sure you understand the point of each exercise (in terms of what it teaches you about K).
Make sure you can run the proofs at the end with the Haskell backend, and can step through their execution with the debugger.
Raoul

[x] Video 1: https://www.youtube.com/watch?v=QHsERS1PyXk
[x] Video 2: https://www.youtube.com/watch?v=aRvYFAcKdD4
[x] Video 3: https://www.youtube.com/watch?v=p7M8gnLmWh0
[x] Video 4: https://www.youtube.com/watch?v=3OiEOUr-9XM
[x] Video 5: https://www.youtube.com/watch?v=P9PSqRCv4p0
[x] Video 6: https://www.youtube.com/watch?v=JGUXrNyfU_A
[x] Can build all the specifications there.
[x] Can run all the tests there (including the proofs).
[x] Can run the kore-repl on a proof and step through the execution.
[x] Discussion about the semantics and the proofs.

Hao

[x] Video 1: https://www.youtube.com/watch?v=QHsERS1PyXk
[x] Video 2: https://www.youtube.com/watch?v=aRvYFAcKdD4
[x] Video 3: https://www.youtube.com/watch?v=p7M8gnLmWh0
[x] Video 4: https://www.youtube.com/watch?v=3OiEOUr-9XM
[x] Video 5: https://www.youtube.com/watch?v=P9PSqRCv4p0
[x] Video 6: https://www.youtube.com/watch?v=JGUXrNyfU_A
[x] Can build all the specifications there.
[x] Can run all the tests there (including the proofs).
[x] Can run the kore-repl on a proof and step through the execution.
[x] Discussion about the semantics and the proofs.

Calin

[ ] Video 1: https://www.youtube.com/watch?v=QHsERS1PyXk
[ ] Video 2: https://www.youtube.com/watch?v=aRvYFAcKdD4
[ ] Video 3: https://www.youtube.com/watch?v=p7M8gnLmWh0
[ ] Video 4: https://www.youtube.com/watch?v=3OiEOUr-9XM
[ ] Video 5: https://www.youtube.com/watch?v=P9PSqRCv4p0
[ ] Video 6: https://www.youtube.com/watch?v=JGUXrNyfU_A
[ ] Can build all the specifications there.
[ ] Can run all the tests there (including the proofs).
[ ] Can run the kore-repl on a proof and step through the execution.
[ ] Discussion about the semantics and the proofs.

Mihai

[ ] Video 1: https://www.youtube.com/watch?v=QHsERS1PyXk
[ ] Video 2: https://www.youtube.com/watch?v=aRvYFAcKdD4
[ ] Video 3: https://www.youtube.com/watch?v=p7M8gnLmWh0
[ ] Video 4: https://www.youtube.com/watch?v=3OiEOUr-9XM
[ ] Video 5: https://www.youtube.com/watch?v=P9PSqRCv4p0
[ ] Video 6: https://www.youtube.com/watch?v=JGUXrNyfU_A
[ ] Can build all the specifications there.
[ ] Can run all the tests there (including the proofs).
[ ] Can run the kore-repl on a proof and step through the execution.
[ ] Discussion about the semantics and the proofs.

You can likely start this tutorial after the first 13 lessons of the K tutorial: https://github.com/kframework/evm-semantics/issues/1075
It might be helpful to complete these videos after lesson 13 because the following lessons are more advanced.


Behavior of 'delete-window' on REPL windows is not as expected

When trying to delete a window with "C-x 0" on a REPL popup, all other windows, except for the actual popup window, get deleted.
To reproduce:

Open a lisp file (e.g. init.el)
Create a second window (C-x 3)
Open the REPL for the mode (M-x +eval/open-repl)
In the REPL window, try to delete the window with "C-x 0"

I've identified the origin of the problem. This happens only if "evil" is not activated.
You've probably figured out by now that I don't use evil :D.
Anyhow, the problematic function is located in core/autoload/popups.el, in the function in line 234. The function reads as follows:
(defun doom/popup-close-maybe ()
  "Close the current popup *if* its window doesn't have a noesc parameter."
  (interactive)
  (if (doom-popup-property :noesc)
      (call-interactively
       (if (featurep 'evil)
           #'evil-force-normal-state
         #'keyboard-escape-quit))
    (delete-window)))

When pressing C-x 0 this function is called. Because the REPL window has the property :noesc, the function keyboard-espace-quit will be called. I don't think this is the function we want to call. As stated by the documentation, this function " ... [goes] back to just one window (by deleting all but the selected window)". In theory, we would like to call delete-window, isn't it?

Good catch, this is definitely the problem, but for a popup with a :noesc property, doom/popup-close-maybe should simply no-op. There are two problems here:

It may not be appropriate that I [remapped delete-window in core-popups.el](https://github.com/hlissner/doom-emacs/blob/develop/core/core-popups.el#L140), as :noesc should only inhibit ESC, so I'll remove this.

You are correct, keyboard-escape-quit isn't the correct no-op command to use. I'm thinking I may replace it with keyboard-quit instead. Are you aware of any side effects to that command?

keyboard-quit or simply calling nothing seems to be the appropriate choice.
I'm not aware of any side-effects of just calling keyboard-quit in this context.
All looks good.


[1.x] Allow to handle any RuntimeExceptions occurring when delivering an InboxMessage

This changeset addresses #1472 and brings other improvements into the codebase.
Failure of signal reception and DeliveryMonitor to respond
InboxMessages which are processed in scope of each Delivery run, are eventually unpacked, and the respective signals are dispatched to their targets. In case a signal target is an Entity, it is possible that the signal reception does not go well for various reasons. In particular, an aggregate may have "broken" @Apply-er, making it impossible to load the aggregate state. Or, some other unexpected error may occur upon loading or storing an entity.
Previously, it was impossible to trace such failures of signal reception. And the InboxMessages causing the errors were left in their inboxes in TO_DELIVER status forever, being delivered over and over again — most likely with the same reception failure repeating.
Now, the Delivery API allows to subscribe for any failures which occur during the reception of each signal. Additionally, end-users may now choose the way to handle the reception failures in terms of action in respect to the InboxMessage of interest.
Out-of-the-box, there are two actions provided:

mark the InboxMessage as delivered — so that it does not block further delivery of messages;
repeat the dispatching of the InboxMessage. Please note, this is a synchronous and immediate action in terms of a delivery run.

Alternatively, end-users may implement their own way of handling the reception failure.
The corresponding functionality is provided via the API of DeliveryMonitor.
Here is a code sample:

public final class MarkFailureDelivered extends DeliveryMonitor {

    /**
     * In case the reception of the {@code InboxMessage} failed,
     * mark it as {@code DELIVERED} anyway.
     */
    @Override
    public FailedReception.Action onReceptionFailure(FailedReception reception) {

    	//// Error details are available as well:
        // InboxMessage msg = reception.message();
        // Error error = reception.error();
        // notifyOf(msg, error);

        return reception.markDelivered();
    }
}

// ...

// Plugging the monitor into the Delivery:

DeliveryMonitor monitor = new MarkFailureDelivered();

Delivery delivery = Delivery
        .newBuilder()
        .setMonitor(monitor)
        // ...
        .build();

ServerEnvironment
        .when(MyEnvironment.class)
        .use(delivery);

By default, InboxMessages are marked as DELIVERED in case of failure of their reception. This is so, because otherwise the issue described in #1472 arises.
Internally, such a functionality became possible by making MessageDispatcher.dispatch(envelope) to return DispatchOutcome. Previously, it was void, which was not suitable for handling any of the issues synchronously — as it was required for InboxMessage to allow users choose the way to handle the reception failures.
Other changes
It was reported, that at run-time the routing of events by their interfaces may lead to ConcurrentModificationException.
This PR addresses the issue for v1 by making the Map of signal routes concurrency-friendly. See io.spine.server.route.MessageRouting for more details.
Even more minor changes are made to fix typos and remove the unused code.
The library version is set to 1.9.0-SNAPSHOT.6.

@alexander-yevsyukov PTAL again.


enhance app import/ref listing

Enhance the way imports and refs are determined from the flogo.json and update the list command to support list filtering.  Also added the ability to list orphaned references.

@debovema If you get a chance, can you take a look as well.  Thanks

@fcastill This is just an update that this PR will slightly modify theflogo list command.  By default flogo list will return all contribs that in the flogo.json.  You can specify an optional -f/--filter where you can filter the list by "used" contribs that are used in the file, and "unused" which means that it is in the imports section, but not referenced anywhere.   There is no longer a need for a --all since that is the default behavior.
Example json entry from list:
[
  {
    "name": "flow",
    "type": "flogo:action",
    "description": "Simple Flow Action",
    "homepage": "https://github.com/project-flogo/flow",
    "ref": "github.com/project-flogo/flow",
    "path"<EMAIL_ADDRESS>    "descriptor": "descriptor.json"
  }
]


fix(renovate): *_test.go regex manager

should fix https://github.com/kairos-io/community-bundles/pull/60 https://github.com/kairos-io/community-bundles/pull/38 after rebase

review and merge would be nice @mauromorales @Itxaka @jimmykarily


PCMR bits swapped in documentation

Looking at the RTL code, the function of Global Enable and Saturation is swapped in the documentation. Saturation is really controlled by bit 1 and enable by bit 0.
Please indicate which one is correct. RTL or doc?
https://github.com/pulp-platform/zero-riscy/blob/c6eeb34f57ff1cec354a74e025b889c26138146b/doc/performance_counters.rst#L16-L26
https://github.com/pulp-platform/zero-riscy/blob/b1b89f55fb1dbbf6f35ac5f92c5c1d12b2f41d45/zeroriscy_cs_registers.sv#L401-L406

Hi @brabect1 ,
thanks for creating this issue. You are right. Bit 0 is the global enable, and Bit 1 is the saturation control bit. I changed the documentation. The fix will be upstream in my next PR.
Best regards,
Pirmin


Self Contained Build Folder

Is your feature request related to a problem? Please describe.
Currently when building in server mode a .nuxt folder is created with 99% of needed artifcated for your website. However we are still required to deploy the entire project which includes the .nuxt folder, node_modules and all the source code.
This is not ideal for a variety of reasons:

Lots of file duplication
Lots of node modules that will not be used after build
The above 2 points mean that when storing artifacts for every build, the artifacts are huge, and becomes a problem when you are dealing with storage quotas

Describe the solution you'd like
I believe the best solution is to copy whatever remaining dependencies that exist outside the .nuxt folder to be copied into the .nuxt folder so that everything is self contained and run in the .nuxt folder. I will also only need to store the .nuxt folder artifacts.
Perhaps a bespoke package.json for the .nuxt folder can be generated to ensure all dependencies of dependcies make their way in too. This might also be useful for people who want to further reduce their artifacts size and wont need to store node_modules

This is now done in Nuxt 3 - the .output folder is self-contained and has everything neeeded to deploy your site, including 'tree-shaken' node_modules based on usage.


Drop "equivalent to" in noexcept specifications

"The exception specification is equivalent to"
drop "equivalent to"; the exception specification is a boolean
expression, and what follows is the boolean expression you want.
No reason for "equivalent to".

see PR #373


Mils 4 PR

In this milestone, I implement two JavaScript modules: calculate.js and operate.js which are responsible for all calculations. Finally, I include those modules in the App component so we can hook them up to other components in the next milestone.

STATUS: APPROVED 🥳 👯
Hello team @vichuge 👋🏾 👋🏾 congratulations on making it this far in this milestone 🥇
Highlights

You have met the specifications needed for this milestone ✔
Your Linters are not forming any errors ✔

With that, I approve of this project 😄
Regards
__Peris Ndanu __
GitHub | LinkedIn | Twitter


fix: wrong escape with link, image url when converting

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has a description of the breaking change

Description

related issue

#1964
#2099

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Hi. Why do you need to escape &?

@ibedarev
It's mistake. It will be fixed.

Why not using encodeURI for url encode


Fix --bdist-all handling multiple console_scripts

--bdist-all is broken and only creates the first entry in console_scripts.
This commit fixes this issue and adds a couple of regression tests.

regression was introduced in https://github.com/pantsbuild/pex/pull/1358

Thank you @jsirois for the extra fast review.


Line feed from CSV file field

Good day,
dont know if it's the right place to ask this, but, i'm trying to have line feed from a specific field when merging my CSV file with Scribus.
I've read the doc and try many test, but could not get a good result.
As an exemple, if my CSV has 2 fields like this
name, colors
Bob, Green\nRed
Paul, Green\nRed\nBlue
How can the colors field be listed inside my textbox, like a bullet list for exemple.
By the way, this script is awesome.
Many Thanks.

Hello.
Glad you like the script.
Linebreaks and tabulations in your CSV data are currently replaced by the scribus equivalent (newlines and tabulations)... Or you can decide to remove them and turning them to simple spaces set the setting KEEP_TAB_LINEBREAK to 0.
So, it won't show as a bullet list yet, but maybe we can find a way to improve the script in this direction. I'd rather not simply replace all linefeeds in CSV by bullet lists... as that may not suit everyone, but rather (in the idea of Markdown) a combination of "\n- ".
If you feel like forking the code and trying to improve the code in this direction I'd gladly help/review your contribution. It happens around https://github.com/berteh/ScribusGenerator/blob/master/ScribusGeneratorBackend.py#L459
Leet me know, and have a nice day,
Berteh.

This could also take the direction of integrating (partial) markdown import, if you have such an interest.
Or you can just use any UTF character that would display as a nice bullet directly in your CSV. This character needs to be supported by the font in use. Some examples:
Paul," • Green\n  • Red\n  • Blue" ,
Bob," ● Green\n  ↝ Red\n  ∘ Blue\n, 😀Green\n 🚙 Red\n ✔Blue" ,
Just copy-past the utf-8 symbol from any characters table on the internet, like https://unicode-table.com/en/#emoticons... and make sure you save your CSV as UTF-8.
Or you could another script after the merge, such as bullets, or bullets and numbered lists. Never tried them but they seem to do the job.

I'm also having a problem getting the linefeed to work correctly. I've tried a couple different tests, but in each one it's printing the \n. The \n is not replaced with the linefeed. Using the ScribusGenerator-python3 version. Thoughts?

Hi Rob.
Are you maybe writing "\n" in your CSV? As in a backslash character and a 'n' character?
If that's the case then sorry for not making the documentation more explicit: your should simply use a real line feed character ('enter' or 'return'). The CSV should look like the examples/business-card.csv file, have a look eg at the line below, where the data with a newline must be quoted:
Rachel Banks,"Expert

Assistant",

This should get properly replaced by ScribusGenerator into a linefeed Tag. If not please kindly tell me your OS and Scribus version. I'll look into it further.
Hope it helps.
Berteh.

PS: just ran the "business cards" example in scribus <IP_ADDRESS> (with python3 script version), and it seems to work fine on linux:

... so kindly send me your CSV as attachment if you can confirm the problem.

I was writing the "\n" right in the spreadsheet and exporting that to the
csv. I thought the script replaced that string with an actual linefeed.
Adding it to the cell using Alt-Enter in Google Sheets works perfectly. The
exported csv look exactly like your example. I'll continue doing that.
Thank you so much for your help. This script is great!
On Fri, Mar 11, 2022, 8:37 PM Berteh @.***> wrote:

Hi Rob.
Are you maybe writing "\n" in your CSV? As in a backslash character and a
'n' character?
If that's the case then sorry for not making the documentation more
explicit: your should simply use a real line feed character ('enter' or
'return'). The CSV should look like the examples/business-card.csv file,
have a look eg at the line below, where the data with a newline must be
quoted:
Rachel Banks,"Expert
Assistant",
This should get properly replaced by ScribusGenerator into a linefeed Tag.
If not please kindly tell me your OS and Scribus version. I'll look into it
further.
Hope it helps.
Berteh.
—
Reply to this email directly, view it on GitHub
https://github.com/berteh/ScribusGenerator/issues/114#issuecomment-1065782174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPJXVBSEHLMMBHWAUETAWLU7PYOXANCNFSM4F75JIQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

You're welcome.
Feedback's always welcome, and showcasing what you did with it as well, directly in the wiki: https://github.com/berteh/ScribusGenerator/wiki


[WIP] Windows support

@sschuberth heyho can pls test this build?
gradle build install shadow
Using the Cli with --project and --config
You can check the gradle-plugin on detekt itself, the versions should be ok (as gradle install overrides 1.0.0.M12.1)

For me already gradle build fails with:
Failures (1):
  JUnit Jupiter:ConditionalPathVisitorTest:test()
    MethodSource [className = 'io.gitlab.arturbosch.detekt.formatting.visitors.ConditionalPathVisitorTest', methodName = 'test', methodParameterTypes = '']
    => java.lang.AssertionError:
Expecting:
 <0>
to be equal to:
 <5>
but was not.

Test run finished after 1728 ms
[        21 containers found      ]
[         0 containers skipped    ]
[        21 containers started    ]
[         0 containers aborted    ]
[        21 containers successful ]
[         0 containers failed     ]
[        55 tests found           ]
[         0 tests skipped         ]
[        55 tests started         ]
[         0 tests aborted         ]
[        54 tests successful      ]
[         1 tests failed          ]

:detekt-formatting:junitPlatformTest FAILED

I deleted my previous comment as I was testing the wrong branch, master instead of windows. However, the windows branch yields the same errors. I'll try to look into it.

Ok, so with #136 and #137 merged in addition to this PR gradle build install shadow works fine for me on Windows.

Awesome will check the PR's out in one or two hours , big thx !!

Did you also checked out gradle detektCheck on detekt itself?

I managed to gradle build install shadow detektCheck detektFormat on my virtual Windows!

Great, so looks like we're good to merge! :-)

oh man ... xD

The history in the windows branch is now pretty much messed up, there are multiple times the same commit message of mine, but the changes are not mine, but at least it's merged now 😉

Yeah ... sorry for that, normally there is a git merge --rebase automatically, but because I pulled too many changes from master and did a git rebase --incremental to delete/rewrite my \n mistake, I pretty much destroyed the history xD. Rebase again on master had too many conflicts, so yeah I just merged it xD

I hope I did not steal your

did a git rebase --incremental

I guess you mean --interactive? BTW, note that you usually also want to pass --preserve-merges if you rebase a branch that has merge commits.

I hope I did not steal your commits ...

No worries, it's more the opposite: At least one of your commits now looks as if it was done by me 😉


Debug Adapter: all debug bar buttons dimmed after breakpoint hit

Issue Type: Bug
In my custom debug adapter, I find that the Debug bar buttons and Debug menu options for continue/step/etc are all dimmed even though the thread is marked as "PAUSED ON BREAKPOINT".  Is that combination of states even possible?  The debugger adapter is still connected, and I can explore the stack frames and variables just fine.
It seems like the UI state is out of sync - is there some mitigation where I can restore that state?

VS Code version: Code 1.32.3 (a3db5be9b5c6ba46bb7555ec5d60178ecc2eaae4, 2019-03-14T23:43:35.476Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3667U CPU @ 2.00GHz (4 x 2494)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.70GB (0.74GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (15)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.8.2

dialog-debugger
Mic
0.0.1

tslint
eg2
1.0.43

vscode-npm-script
eg2
0.3.5

Ionide-FAKE
Ion
1.2.3

Ionide-fsharp
Ion
3.34.0

Ionide-Paket
Ion
1.12.0

json-to-ts
Mar
1.5.5

azurecli
ms-
0.4.4

csharp
ms-
1.18.0

powershell
ms-
1.12.0

vscode-typescript-tslint-plugin
ms-
1.0.0

team
ms-
1.149.2

azurerm-vscode-tools
msa
0.5.0

debugger-for-chrome
msj
4.11.3

Can you reproduce this with vscode insiders https://code.visualstudio.com/insiders/
We have changed how these actions are contributed and this should be fixed with latest

I just installed VS Code Insider and there are different problems:

the pause button lights up, but the only thread is already paused at a breakpoint
the casing of the file system drive path now seems to be different (possibly a different issue - c: is cased differently)

Here is a log of the messages sent back and forth with output events removed (I rarely see configuration done, by the way):
READ: {"command":"initialize","arguments":{"clientID":"vscode","clientName":"Visual Studio Code - Insiders","adapterID":"dialog","pathFormat":"path","linesStartAt1":true,"columnsStartAt1":true,"supportsVariableType":true,"supportsVariablePaging":true,"supportsRunInTerminalRequest":true,"locale":"en-us"},"type":"request","seq":1}
SEND: {"body":{"supportsConfigurationDoneRequest":true},"request_seq":1,"success":true,"command":"initialize","message":null,"seq":1,"type":"response"}
SEND: {"body":{},"event":"initialized","seq":2,"type":"event"}
READ: {"command":"setBreakpoints","arguments":{"source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"lines":[1],"breakpoints":[{"line":1}],"sourceModified":false},"type":"request","seq":2}
SEND: {"body":{"breakpoints":[{"verified":false,"message":null,"source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":null,"endLine":null,"endColumn":null,"id":1}]},"request_seq":2,"success":true,"command":"setBreakpoints","message":null,"seq":4,"type":"response"}
READ: {"command":"setExceptionBreakpoints","arguments":{"filters":[]},"type":"request","seq":3}
SEND: {"body":{},"request_seq":3,"success":true,"command":"setExceptionBreakpoints","message":null,"seq":5,"type":"response"}
READ: {"command":"threads","type":"request","seq":4}
SEND: {"body":{"threads":[]},"request_seq":4,"success":true,"command":"threads","message":null,"seq":6,"type":"response"}
SEND: {"body":{"threadId":1,"reason":"started"},"event":"thread","seq":8,"type":"event"}
SEND: {"body":{"reason":"changed","breakpoint":{"verified":true,"message":null,"source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":null,"endLine":null,"endColumn":null,"id":1}},"event":"breakpoint","seq":11,"type":"event"}
READ: {"command":"threads","type":"request","seq":5}
SEND: {"body":{"threads":[{"id":1,"name":"threadOne"}]},"request_seq":5,"success":true,"command":"threads","message":null,"seq":14,"type":"response"}
SEND: {"body":{"reason":"breakpoint","description":"'threadOne' is Breakpoint","threadId":1,"text":"'threadOne' is Breakpoint","preserveFocusHint":false,"allThreadsStopped":true},"event":"stopped","seq":13,"type":"event"}
READ: {"command":"threads","type":"request","seq":6}
SEND: {"body":{"threads":[{"id":1,"name":"threadOne"}]},"request_seq":6,"success":true,"command":"threads","message":null,"seq":15,"type":"response"}
READ: {"command":"stackTrace","arguments":{"threadId":1,"startFrame":0,"levels":20},"type":"request","seq":7}
SEND: {"body":{"stackFrames":[{"name":"FRAMENAMEREMOVED","source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":1,"endLine":83,"endColumn":1,"id":1},{"name":"planning[]","source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":1,"endLine":83,"endColumn":1,"id":2}]},"request_seq":7,"success":true,"command":"stackTrace","message":null,"seq":16,"type":"response"}
READ: {"command":"stackTrace","arguments":{"threadId":1,"startFrame":0,"levels":20},"type":"request","seq":8}
SEND: {"body":{"stackFrames":[{"name":"FRAMENAMEREMOVED","source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":1,"endLine":83,"endColumn":1,"id":3},{"name":"planning[]","source":{"name":"FILENAMEREMOVED","path":"FULLPATHREMOVED"},"line":1,"column":1,"endLine":83,"endColumn":1,"id":4}]},"request_seq":8,"success":true,"command":"stackTrace","message":null,"seq":17,"type":"response"}
READ: {"command":"scopes","arguments":{"frameId":1},"type":"request","seq":9}
SEND: {"body":{"scopes":[{"expensive":false,"name":"FRAMENAMEREMOVED","variablesReference":1}]},"request_seq":9,"success":true,"command":"scopes","message":null,"seq":18,"type":"response"}
READ: {"command":"variables","arguments":{"variablesReference":1},"type":"request","seq":10}
SEND: {"body":{"variables":[{"name":"user","value":"Count = 0","variablesReference":2},{"name":"conversation","value":"Count = 0","variablesReference":3},{"name":"dialog","value":"Count = 0","variablesReference":4},{"name":"turn","value":"Count = 0","variablesReference":5},{"name":"entities","value":"Count = 0","variablesReference":6},{"name":"tags","value":"Count = 0","variablesReference":7}]},"request_seq":10,"success":true,"command":"variables","message":null,"seq":19,"type":"response"}

@isidorn do you need more information than the above comment?

Please change the mock debug adapter and create a repo on github with reproducable steps https://github.com/Microsoft/vscode-mock-debug
Otherwise I will unforunetly not be able to help since I can not reproduce this.

@isidorn do you know why in the above trace configurationDone is never sent after my response to setExceptionBreakpoints?  It might help me narrow down the difference between the mock debug adapter and my custom adapter.

@willportnoy you have to "opt into" the "configurationDone" mechanism by returning a value of true for the supportsConfigurationDoneRequest capability.

@weinand I believe supportsConfigurationDoneRequest is already (hopefully correctly?) included in the response body for initialize in the trace above.

VSCode > F1 > Developer Tools > sources > put a breakpoint here and investigate why the sendConfigurationDone is not called.
Also make sure you send out an initialized event from your debug adapter. Put a breakpoint here on the vscode side to verify all is good.

@isidorn @weinand thank you for the help - it turned out that I had some errors encoding the protocol json into the TCP stream, causing some response messages to be apparently skipped.


✨ [Feature] Route name

Before feedback (在反馈之前)

[X] There are no features that I want to submit in Issues (当前 Issues 中没有我将要提交的新功能)

Describe Feature (描述功能)
Support name with routes
Ref: https://laravel.com/docs/10.x/routing#named-routes
https://docs.gofiber.io/api/app#name

Hey guys ,
did anyone find a way to do this ?
If not I would try to dive deep in the framework and try to implement it .

Sure, please, I appreciate that. 👍

Gin doesn't support named routes :slightly_smiling_face: .

We can implement it in Framework.