Terrain geometry seem shows unsightly crack in terrain.

In the screen-shot below, notice the seem in the terrain.  It appears that LOD tiles for the terrain are never loading as I zoom in.  This can be readily found/reproduced by using my web-app indicated in the screen-shot's URl line.

@spencerparkin What terrain provider are you using?
This might be fixed by this PR: #7061

Here are my image providers...
var terrain_image_provider = new Cesium.ArcGisMapServerImageryProvider({url:'//services.arcgisonline.com/ArcGIS/rest/services/World_Imagery/MapServer'});
var topo_image_provider = new Cesium.ArcGisMapServerImageryProvider({url:'//services.arcgisonline.com/ArcGIS/rest/services/World_Topo_Map/MapServer'});
I think you're right.  I just need to wait until that PR is merged into master.  My app is currently deployed with a built copy of Cesium.  I'm going to change that once a new release of it is available that also has my recently merged PR in it.  I just need to hang tight.  Thanks for looking into this.

@spencerparkin I was wondering what terrain provider you were using, not imagery provider.  Are you using terrainProvider: Cesium.createWorldTerrain() to add Cesium World Terrain?

Oh!

No problem!  I wasn't able to reproduce the terrain cracking on my computer, but I'm pretty sure what you're seeing will be fixed by #7061
I'm going to close this issue for now, but feel free to re-open it if you upgrade Cesium after #7061 is merged and you're still seeing problems.  Thanks!


[BUG] Space station dimensions seem to break after upgrading E2L server

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry

AdvancedRocketry-1.12.2-1.6.0-190-universal.jar

Have you verified this is an issue in the latest unstable build

[x] N/A

I did try to reproduce this behavior by manually updating our E2L modpack to AR build 193 and experienced what appeared to be the same issue (which has also been present on older versions), but I'm not sure if manual, piecemeal upgrade scenarios are valid.

Version of LibVulpes

LibVulpes-1.12.2-0.4.2-69-universal.jar

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] N/A

We're running the Enigmatica 2 Light modpack on our server - unfortunately I don't really have a good way to test AR in isolation in this server/client scenario.

Crash report or log or visualVM (if applicable)

Here's the relevant snippet from my client log trimmed to a size pastebin allows: https://pastebin.com/cFTfz7nt
Here's my whole client log file:
latest.log
(I think errors would keep piling up if I didn't cancel my connection attempt)

Description of the problem

Space station dimensions appear to break when upgrading mods on the server.
For example:

player logs off while in space station
server shuts down
mods are updated (including dumping old config/mods/schematics/scripts folders, with manual preservation of any config changes)
server is restarted
player tries to join server
In this case that player is stuck at the 'logging in' step as errors continue to pile up in the client log (attached).

The relevant item here appears to be:

java.util.concurrent.ExecutionException: java.lang.IllegalArgumentException: Could not get provider type for dimension -2, does not exist
But this dimension does indeed exist, and was working fine prior to updating the server mods.

I wouldn't be surprised if this may be related to something else in the E2L pack, and if some manual preservation/repair of dimension provider data is needed to work around this issue I'm happy to do that, but I'm not sure where to look for that sort of thing...

I don't see any indication dim-2 is being registered, can you send both client and server AR configs?

Sure thing, here they are (renamed to indicate source and switched to .txt so github doesn't complain):
advancedRocketry.cfg_client.txt
advancedRocketry.cfg_server.txt

I noticed that spaceStationId differs between the two, before build 192, that would probably be an issue.  Config sync was revamped in 192.  What happens if you change spaceStationId from -2 to 99 on the client?
Now that you mention it, I think the space dimension is usually registered before the config is synced, so the space station id may sync, but this problem could still occur.

Looks like if I set the client config to spaceStationId=-2 (to match the server) I'm able to connect successfully!
(In other news, I left the game trying to join the server while I ate dinner and came back to a >700mb log file :p)
Sounds like the workaround for my current situation is to move our server's space station dimension to 99 to match the client defaults - if that's as easy as renaming the dimension folder and updating the config that's no problem.
I also ran the following test:

Update AR version on server from 190 to 193 (just replace the .jar in the mods folder)
Update the AR version on the client from 190 to 193 (same as above)
With server and client config spaceStationId values set to -2, client was able to connect successfully
After changing client spaceStationId back to 99 (different than the server) and relaunching the client, got stuck on the 'Logging in' step again

Just an anecdote but this seems like the issue is also present in 193. If there's any additional testing I can do just let me know!


Options to install newer Tomcat and Java

https://linuxacademy.com/community/posts/show/topic/3197-apache-tomcat-7-vagrant-provision-script

This helped:
https://linoxide.com/linux-how-to/install-tomcat-8-centos-7/
Might be nice to just install tomcat manually for all tomcat versions; the installer would do latest, and use var to pick older version.
Note that when starting up tomcat, this proved to be useful (based on instructions in above article):
$ sudo -u tomcat ./startup.sh


API error in Disney+ Hotstar addon

Add-on name: Disney+ Hotstar
Add-on version: BA v1.1.1
Kodi version: Kodi 19.3
Operating system version/name: Android TV v.10
Using from country:India

Getting API error. Raise issue if you are continuously facing this error . Invalid Handler
To Reproduce
Steps to reproduce the behavior:

Go to any program to play and it will show error on top right side.
See an error

I can't even log into it, there must be some kind of bug in it.

Facing the same issue.

yes. same issue facing ..

same here

https://paste.kodi.tv/irisaboqiv
this is the logfile for the error

I faced the same issue and to solve it , i thought let's logout and login again but now I am unable to login. When I try to login the code is not generating it keeps showing error .....what would be the solution for the problem

Only if the admin can put some light on it, solve the bug, and provide the updated version. Or does anyone have any other repository Hotstar add-on...

I am also facing the same issue. Tried logging out. But for logging in, error is displayed. Tried updating widevine but not succesful.

Same issue, I tried logging out but now I can't log in, the code doesn't generate.

Same issue from last 2day please update the hotstar 1.1.1 version please fix the problem soon sir

same issue. cannot loging. cannot play videos

Facing same issue. Not working in any Android or Windows device - API error.

Facing same issue unable to login

Facing sane issue
I request the admin to resolve the issue ASAP, its been 4 days the app isnt working. Please update or remove the bug for your us.

It seems admin forgot it's repo..

++

Log-in window can show only half second then gone.

Getting same issue facing, login issue , nonetype ,
Could you please check the issue.
Add-on name: Disney+ Hotstar
Add-on version: BA v1.1.1
Kodi version: Kodi 19.4
OS version: Android TV v.10
Using from country:India

Getting same issue facing, login issue
Raspberry PI 4
Add-on name: Disney+ Hotstar
Add-on version: BA v1.1.1
Kodi version: Kodi 19.4
OS version: LibreELEC V ersion 10.0.2 RPi4.arm
Using from country: India
The Log-in window can show only half a second then gone.
(https://user-images.githubusercontent.com/67773396/187884762-14e26023-32c2-4431-bf7b-f549f136503f.jpg)

any new version??????


Changing render resolution mid game

Hello, I might be missing something but is there any way to change the resolution in game?
By this I mean when you set the project settings in godot for the display resolution, this changes the resolution the 3d game is rendered at.
I would like to implement a method to change resolution while in game, such as how games like overwatch offer the resolution option as overall quality control. (As shown here https://i.imgur.com/0DEq988.jpg)
So far everything I have done seems to just change the size of the window while not affecting render resolution.
Thanks for the help.

Viewport undersampling isn't built into the engine yet, but I was told that you could implement it yourself using a custom viewport.

It seems that viewport.set_size(Vector2(w,h)) does what I need. But you need to adjust it in game after the viewport is loaded. Closing the issue.


Hyundai: Add Car Port for 2022 Kia Niro EV

Checklist

[x] added to RELEASES, CARS
[x] test route added to test_routes.py
[x] route with openpilot: c011e480f2430f67|2021-10-28--11-09-58

Thanks to community Niro EV owner Kolibabas.

Not sure why it would fail on Route Not Found. I have the route preserved and set to public.
c011e480f2430f67|2021-10-28--11-09-58
https://useradmin.comma.ai/?onebox=c011e480f2430f67|2021-10-28--11-09-58

It's unclear that this needs a new car port. The existing Niro EVs never fingerprint correctly with the FW we have, so I suspect they're having the same issue as this 2022 Niro (i.e. the esp fw isn't from a Niro).

Closing this PR as a car port is not required for the 2022 Kia Niro EV. The issue is caused by an incorrect firmware esp from a different car that was added to KIA_NIRO_EV during a previous firmware dump. New PR can be found here: #23253.


Add MS Windows (MinGW) support

Added singlecall SetConsoleMode initialization code to the VirtualTermintal constructor. Tested on MinGW 8.1 compiler and Windows 10 native console (cmd.exe)

Thanks for the PR. Why not just call eg. system(" ") to enable console mode at the beginning of the app?

just for completeness, this picture summarizes it best:

Thanks for your contribution!


Brazilian Portuguese translation of Responsive Design with Bootstrap

[ ] Use Responsive Design with Bootstrap Fluid Containers Complete
[ ]  Make Images Mobile Responsive Complete
[ ]  Center Text with Bootstrap Complete
[ ]  Create a Bootstrap Button Complete
[ ]  Create a Block Element Bootstrap Button Complete
[ ]  Taste the Bootstrap Button Color Rainbow Complete
[ ]  Call out Optional Actions with Button Info Complete
[ ]  Warn your Users of a Dangerous Action Complete
[ ]  Use the Bootstrap Grid to Put Elements Side By Side Complete
[ ]  Ditch Custom CSS for Bootstrap Complete
[ ]  Use Spans for Inline Elements Complete
[ ]  Create a Custom Heading Complete
[ ]  Add Font Awesome Icons to our Buttons Complete
[ ]  Add Font Awesome Icons to all of our Buttons Complete
[ ]  Responsively Style Radio Buttons Complete
[ ]  Responsively Style Checkboxes Complete
[ ]  Style Text Inputs as Form Controls Complete
[ ]  Line up Form Elements Responsively with Bootstrap
[ ]  Create a Bootstrap Headline Complete
[ ]  House our page within a Bootstrap Container Fluid Div Complete
[ ]  Create a Bootstrap Row Complete
[ ]  Split your Bootstrap Row Complete
[ ]  Create Bootstrap Wells Complete
[ ]  Add Elements within your Bootstrap Wells Complete
[ ]  Apply the Default Bootstrap Button Style Complete
[ ]  Create a Class to Target with jQuery Selectors Complete
[ ]  Add ID Attributes to Bootstrap Elements Complete
[ ]  Label Bootstrap Wells Complete
[ ]  Give Each Element a Unique ID Complete
[ ]  Label Bootstrap Buttons Complete
[ ]  Use Comments to Clarify Code

Closed in favor of https://github.com/FreeCodeCamp/TranslationExpansion/issues/42


Consolidate AccountsDb constructors

Problem
Currently, the implementation for creating a new AccountsDb is spread across AccountsDb::new_with_config() and AccountsDb::default_with_accounts_index(). The implementation being fragmented duplicates some work/variable setting, and is a little hard to follow.
Summary of Changes
Roll the logic for AccountsDb::default_with_accounts_index() into AccountsDb::new_with_config()

Are there any changes to the actual values used to create new instances of AccountsDb? I would imagine 'no', but want to double check.

No, the first commit was:

Remove ..Self::default_with_accounts_index(...) from new_with_config()
Attempt to compile and see error that I'm missing ~30 fields; compiler lists out the missing fields
Copy/paste those fields over from default_with_accounts_index()
Remove default_with_accounts_index()

The subsequent changes were the less mechanical stuff

Github doesn't make the diff particularly easy though...

Yeah, I see that. Not sure what I could have done differently; maybe break the move of fields into different commits, 5 fields per commit or something so they're more manageable ?

Did another manual review of the first commit (shifting of fields) and I believe those line up 1-to-1, so between that / your two reviews / green CI, I think we're good


Infinite recursion error

I would love to use this template.
When NIX_PATH=nixpkgs=http://nixos.org/channels/nixpkgs-unstable/nixexprs.tar.xz nix-shell I'm getting

error: infinite recursion encountered, at /nix/store/2fry3p9f13z4icn9a973fwaj6n4hgnz1-source/contrib/flake.nix:98:18

This is my neovim overlay:
# file ~/.config/nixpkgs/overlays/neovim.nix
(import (builtins.fetchTarball {
      url = https://github.com/nix-community/neovim-nightly-overlay/archive/master.tar.gz;
    }))

Hmm it's not the template, but neovim-nightly-overlay which is causing the issue. Others have reported this too, but no one has filed an issue yet. nix shell should work on nixUnstable not sure why nix-shell on 2.3 is causing issues.

I can't really reproduce this, for reference, you can try the following nixos-shell (available in nixpkgs), and it works:
{ pkgs, ... }: {
  boot.kernelPackages = pkgs.linuxPackages_latest;
  nixos-shell.mounts = {
    mountHome = false;
    mountNixProfile = false;
    cache = "none"; # default is "loose"
  };
  nixpkgs.overlays = [
    (import (builtins.fetchTarball {
      url = https://github.com/nix-community/neovim-nightly-overlay/archive/master.tar.gz;
    }))
  ];
  environment.systemPackages = with pkgs; [
    neovim-nightly #neovim also works here
  ];
  # programs.neovim.enable = true; # this also works for me
}

Can you send me a minimal nixos-shell (by modifying the above) that reproduces the problem? You can copy the nix-shell into this environment and try it.

I think my nix-store had a problem, optimized it and now this works !
Thanks !

Have to reopen.

Still the offending command from OP runs into the error described there.
Will try the nixos-shell approach.

With this nix expression

I can't really reproduce this, for reference, you can try the following nixos-shell (available in nixpkgs), and it works:
{ pkgs, ... }: {
  boot.kernelPackages = pkgs.linuxPackages_latest;
  nixos-shell.mounts = {
    mountHome = false;
    mountNixProfile = false;
    cache = "none"; # default is "loose"
  };
  nixpkgs.overlays = [
    (import (builtins.fetchTarball {
      url = https://github.com/nix-community/neovim-nightly-overlay/archive/master.tar.gz;
    }))
  ];
  environment.systemPackages = with pkgs; [
    neovim-nightly #neovim also works here
  ];
  # programs.neovim.enable = true; # this also works for me
}

Can you send me a minimal nixos-shell (by modifying the above) that reproduces the problem? You can copy the nix-shell into this environment and try it.

I get

[nixos-shell-2]$ NIX_PATH=nixpkgs=http://nixos.org/channels/nixpkgs-unstable/nixexprs.tar.xz make reproduce-issue
these derivations will be built:
/nix/store/xicv7gn13gnzaxp7k67akw0flyrc4inf-nixos-shell.drv
building '/nix/store/xicv7gn13gnzaxp7k67akw0flyrc4inf-nixos-shell.drv'...
unpacking sources
unpacking source archive /nix/store/ggm7fwh7m0zp4swhi4vzkvb1g108klnq-nixos-shell-2
source root is nixos-shell-2
patching sources
configuring
no configure script, doing nothing
building
build flags: SHELL=/nix/store/68c5b0h2w2ab57sbas1b4m073ik8hlm5-bash-4.4-p23/bin/bash
make: Nothing to be done for 'all'.
installing
install flags: SHELL=/nix/store/68c5b0h2w2ab57sbas1b4m073ik8hlm5-bash-4.4-p23/bin/bash install
install -D bin/nixos-shell /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell/bin/nixos-shell
install -D share/nixos-shell/nixos-shell.nix /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell/share/nixos-shell/nixos-shell.nix
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell
strip is /nix/store/5xyjd2qiily84lcv2w2grmwsb8r1hqpr-binutils-2.35.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell/bin
patching script interpreter paths in /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell
/nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell/bin/nixos-shell: interpreter directive changed from "#!/usr/bin/env bash" to "/nix/store/68c5b0h2w2ab57sbas1b4m073ik8hlm5-bash-4.4-p23/bin/bash"
checking for references to /tmp/nix-build-nixos-shell.drv-0/ in /nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell...
/nix/store/hq0lchn93mk9ycly46gasgx49hz7ygmb-nixos-shell/bin/nixos-shell examples/defaults-vim.nix
error: infinite recursion encountered, at /nix/store/qkx75z9mbnpq8rm88mk56kgavamk3ar5-source/contrib/flake.nix:98:18
(use '--show-trace' to show detailed location information)
make: *** [Makefile:36: reproduce-issue] Error 1

Still not able to reproduce:

nixos-shell with the following vm.nix

{ pkgs, ... }: {
  boot.kernelPackages = pkgs.linuxPackages_latest;
  nixos-shell.mounts = {
    mountHome = false;
    mountNixProfile = false;
    cache = "none"; # default is "loose"
  };
  virtualisation.diskSize = 5000;
  virtualisation.writableStoreUseTmpfs = false;
  environment.systemPackages = with pkgs; [
    git
  ];
}

git clone https://github.com/mjlbach/defaults.nvim.git && cd defaults.nvim
3.
NIX_PATH=nixpkgs=http://nixos.org/channels/nixpkgs-unstable/nixexprs.tar.xz nix-shell

Doing it like this works as well for me:

Still not able to reproduce:
1. `nixos-shell` with the following `vm.nix`

{ pkgs, ... }: {
  boot.kernelPackages = pkgs.linuxPackages_latest;
  nixos-shell.mounts = {
    mountHome = false;
    mountNixProfile = false;
    cache = "none"; # default is "loose"
  };
  virtualisation.diskSize = 5000;
  virtualisation.writableStoreUseTmpfs = false;
  environment.systemPackages = with pkgs; [
    git
  ];
}

1. `git clone https://github.com/mjlbach/defaults.nvim.git && cd defaults.nvim`

2. `NIX_PATH=nixpkgs=http://nixos.org/channels/nixpkgs-unstable/nixexprs.tar.xz nix-shell`

Still have to figure out how to use w/o nixos-shell.

Seems like it's something strange with your system configuration then, I would check your overlays/channels/etc.

You were right there was one offending overlay definition in my ~/.config/nixpkgs/overlays folder. Completely removed it temporarily and now all works.


Possible "direct binding" between ViewModel object and View?

I'm referring to the ViewState MVVM architecture from now.
I'm wondering how to use direct binding between ViewModel and View.
For example NavigationLink is accepting isActive parameter of type Binding<Bool>. I want to control this value through ViewModel instead of View's @State property. That is because only after ViewModel has finished with Input Trigger action, this binding value of ViewModel will be set to true what will cause NavigationLink to proceed.
Another example: how to move @State  private var newMessage = "" from the View to ChatDetailState/ChatDetailViewModel? It will ensure that ViewModel will always have latest message value and could react accordingly. As a benefit, .addMessage trigger will not handle message value since it's already in ViewModel.

Hey @Devepre !
I was also wondering about this problem. Since in ViewState MVVM there should only be a read-only state available to the view and all state changes should be initiated by an input being triggered. Therefore, I would suggest the following extension to create bindings. To make this API even easier to use, you can create extensions using keypaths as well, just as an example.
extension AnyViewModel {
    func bind<Value>(_ value: @escaping (State) -> Value,
                     to input: @escaping (Value) -> Input?) -> Binding<Value> {

        Binding(get: { value(self.state) },
                set: { input($0).map { self.trigger($0) } })
    }
}

This would allow you to create bindings like the following for a text property of your state and an input for that property:
viewModel.bind({ $0.text }, to: { .text($0) })

In your viewModel, you can then change the state depending on that input - that may be a bit of an overhead, but it would ensure that the viewModel state is read-only. Another option would be to make the state read-write and then ensuring that you are only writing to the state, whenever it would seriously impact code overhead.

Hi @pauljohanneskraft ,
Sounds reasonable, thanks a lot!
I've tried both approaches a little bit in practice. For now, second approach with read-write state and KeyPaths "everywhere" is more enjoyable to me, because of cleaner API and less boilerplate code. It will look like this code snippet:
viewModel.bind(on: \.text)
Also it's critical to have read-only state, that's why I've used private(set) var state but func forceUpdate(_ newState: State) as ViewModel protocol requirement while AnyViewModel's implementation will assert failure.

@Devepre
Could you be so kind to share how you realized this?
viewModel.bind(on: \.text)

@savage7 it was tricky task, since I've moved to another approach a long time ago and didn't have commits pushed to the server. Anyway I've managed to find my old implementation. If it makes sense, I can make pull request to the original repo, anyway you can take a look implementation in forked version here https://github.com/Devepre/SwiftUI-Architectures/tree/develop commit c69e644e6dfac461e172adb1804723b50c4eba01


About the data preprocessing?

Dear authors, I am interested in this wonderful work, and I'd like to train and test the code.
However, I meet some troubles in the data processing, could you share more detail about the training and testing about this?

the data processing about the data path list in the code, the "scannet_bbox_dataset_folder_path, scannet_object_dataset_folder_path, scan2cad_object_model_map_dataset_folder_path, shapenet_udf_dataset_folder_path" is how to be processed and means what?

The testing weight could be shared?
Thank you very much.

Hello, I apologize for the delay. The data initialization script has always been included in the code repository, but it might not be easily noticeable. I have now reorganized the script and provided some explanations in the README. Feel free to give it a try!
And the testing weight will also be released soon~ Thank you for your attention!


RockyLinux 8/9 support

Added support for RockyLinux 8/9
I have successfully created images using commands:
magnum-cluster-api-image-builder \
  --operating-system rockylinux-8 \
  --version v1.27.11 \
  --image-builder-version v0.1.25

magnum-cluster-api-image-builder \
  --operating-system rockylinux-9 \
  --version v1.27.11 \
  --image-builder-version v0.1.25

And I have successfully booted Kubernetes clusters using these images
/build-new-images

Thank you for your contribution!  This is great, seems like the failure is unrelated.  I am working on a few improvements in #327 so we can actually do a few things:

Test this code to build images
Run RockyLinux built images through CI
Test bumping the image-builder version globally so it works out of the box

I think we're probably good to land this, since you made a PR from your own fork, Jenkins won't take this into place.  I'll merge and see how it goes.


Add worst case travel to 10 nearest providers

From conversation with @viviandien:

For each point A, find distance to nearest point B
Find the maximum of these distances
For the resulting point A, find the 10 closest point Bs

@ericboucher This requires returning not just the closest provider id, but the 10 closest provider ids for a given rp id. What do you think - new API? Reuse an API? Punt on this for now?
cc @viviandien


GitHub Repository Setup

Repository and branch structure created. A github action was added to commits on dev to build and run tests.


The context view shows as blank if the string on the address bar got changed for one Azure AD attached regular folder

Storage Explorer Version: 1.30.0-dev
Build Number: 20230519.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.3.1 (Apple M1 Pro)
Architecture: x64/x64/arm64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container and assigned a RBAC role 'Storage Blob Data Owner' to it.
Go to the blob container -> Create a folder -> Upload one blob to the folder.
Right click the folder -> Click 'Copy URL'.
Open connect dialog -> Click 'Blob container or directory -> Sign in using Azure Active Directory (Azure AD)'.
Attach the folder via Azure AD.
Go to the attached folder -> Right click the blob -> Click 'Manage History -> Manage Versions'.
Observe whether the context view shows well.

Expected Experience
The context view shows well.

Actual Experience
The context view shows as blank.

Addition Context

This issue reproduces with RBAC role 'Storage Blob Data Contributor'.
This issue reproduces when clicking 'Manage History -> Manage Snapshots'/creating a new folder/running filter by prefix/tag query.

Update: This issue also reproduces for a Public connect regular folder.

This should have been fixed in main.

Verified this issue on the build main 20230524.4. Fixed. So, we are going to close.


Add .gitignore

I see the point of this merge requets. However the .idea folder is added on purpose. I use some of Intellij IDEA's feature to buid the release package and manage the test cases.

I see the point of this merge requets. However the .idea folder is added on purpose. I use some of Intellij IDEA's feature to buid the release package and manage the test cases. It's an important part of the project.

ok, normally these jobs(test, release) could be done by maven plugins. i will try it.

I see the point of this merge requets. However the .idea folder is added on purpose. I use some of Intellij IDEA's feature to buid the release package and manage the test cases. It's an important part of the project.

ok, normally these jobs(test, release) could be done by maven plugins. i will try it.

Maven cannot meet my demands so I choose Intellij IDEA embedded build options.

Closing for now.


remove json camelCasing with .net core 3.0

Returning camelCasing JSON is now the default in net core 3.0. However, as shown in the sample app, the formatter is modified to return pascal casing as the casing is changed on the client side.
The thing is that everything still work fine if I remove that code
.AddNewtonsoftJson(opt => {
      // Set Breeze defaults for entity serialization
      var ss = JsonSerializationFns.UpdateWithDefaults(opt.SerializerSettings);
      if (ss.ContractResolver is DefaultContractResolver resolver)
      {
          resolver.NamingStrategy = null;  // remove json camelCasing; names are converted on the client.
      }
      ss.Formatting = Newtonsoft.Json.Formatting.Indented; // format JSON for debugging
  });

So is it needed ?
I'd like to return camelCase Json because I'm mixing up breeze and non-breeze controllers.

That should be fine, as long as your client uses the same NamingConvention.


Ctrl-C on signal

I'd like to run a command until a signal is fired:
- sh: perf record -a
  - await: FABAN_RAMP_DOWN
  - ctrlC

An alternative could be:
- sh: {
  command: perf record -a
  background: true
  save-pid: PERF_PID
}
- waitFor: FABAN_RAMP_DOWN
- sh: kill ${{PERF_PID}}

We can run
- sh: perf record -a > /tmp/out.log 2 > /tmp/err.log & export PID="$!"

This will put PID into the environment for the current script. If we need to share the value with another script
- sh: perf record -a > /tmp/out.log 2 > /tmp/err.log & export PID="$!"
- sh: echo ${PID}
  - set-state: RUN.SHARED_PID

I like the idea of waiting for a signal while a command is running in the foreground because that is what we normally do with multiple terminals. Now we have on-signal
- sh: perf record -a
  on-signal:
    FABAN_RAMP_DOWN:
    - ctrlC

We can also use read-signal which will only invoke its children commands if the signal is already reached
- sh: perf record -a
  on-signal:
    FABAN_RAMP_DOWN:
    - read-signal: SOMETHING_BAD
      then:
      - ctrlC

That last example uses the new yaml (actual yaml) syntax based on an overloaded MapConstructor

on-signal is in qDup although not often used. I will close the issue, open it again if there i something I missed from the discussion


added UUID option for element_id arg in element constructor

Addresses #1071
I followed how NoID() is used as a value for element_id by analogy.

Thanks @ChrisPappalardo ! Could you add a changelog entry? Otherwise looks great


Add Works About tab and list to events (from 1303)

Problem Description:  Periods (events) are not displaying lists of Related Works: https://lux-front-dev.collections.yale.edu/view/activity/6297d4d4-ea61-45b2-8c47-9a0e6298fdcc
Expected Behavior: Front end should render the list Works Related on Event page.  Middle tier will provide a HAL link which searches for works about this event.
Needed:

[x] Implementation plan
[x] Create HAL Link ticket - https://git.yale.edu/lux-its/lux-backend/issues/107 - Peter

Dependency:

[x] Create WorksAboutEvent HalLink (from 107) #9
Old ticket - 107
~https://git.yale.edu/lux-its/marklogic/issues/899~

UAT Review:

Shows Works About: https://lux-front-tst.collections.yale.edu/view/activity/6297d4d4-ea61-45b2-8c47-9a0e6298fdcc
Does not show Works About: https://lux.collections.yale.edu/view/activity/6297d4d4-ea61-45b2-8c47-9a0e6298fdcc

Links

Work with concept: https://lux-front-dev.collections.yale.edu/view/text/b73b66ae-a434-4214-adae-ba33427ae713
https://www.bugherd.com/projects/284041/tasks/1084
https://www.bugherd.com/projects/284041/tasks/1982

Approved per UAT

deployed


Do not fill the RequestBody description with the first parameter of a…

Fixes the issues #3038 and #2062.
Better to document each member of a FromForm independently and do it only on the properties.
The original commit that deleted the possibility of documenting a parameter of a FromForm request was this one:https://github.com/domaindrivendev/Swashbuckle.AspNetCore/commit/c1fb18c2c2f864285a7b811a147d8a9ed8c17993.
As you see in the SchemaFilter it deleted the posibility of documenting a Parameter of a FromForm request. My first try was just to look for it again, but it generated duplicated nodes description. For a parameter it ended up documenting the parameter itself and the Schema.. So I opted to just document the Parameter in the RequestBodyFilter (For C# parameters, because the properties were already set)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 84.90566% with 8 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@8b0e896). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

...gerGen/XmlComments/XmlCommentsRequestBodyFilter.cs
84.90%
8 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3076   +/-   ##
=========================================
  Coverage          ?   90.45%
=========================================
  Files             ?       76
  Lines             ?     3123
  Branches          ?      516
=========================================
  Hits              ?     2825
  Misses            ?      298
  Partials          ?        0

Flag
Coverage Δ

macOS
90.45% <84.90%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Another possibility for fixing this issue is to pass the Binding info and apply in the SchemaFilter the change only when the object is a Parameter and also when is FromForm

@jgarciadelanoceda In the issue descriptions can you use the syntax fixes #N so that the auto-close works please? fixes the issue #n doesn't work 😃


Docker Swarm events support

At the moment, Swarm API does not support events retrieval using the parameters since and until.
Track the following issue for updates:

https://github.com/docker/swarm/issues/1203

Closed in favor of #920


DescribeAllParametersInCamelCase

Hi!
Need a little help how to configure Swagger to do DescribeAllParametersInCamelCase
Best Regards
Benny

swagger param names are camelcase by default.

above is what i see for the following code:
var bld = WebApplication.CreateBuilder(args);
bld.Services
   .AddFastEndpoints()
   .SwaggerDocument();

var app = bld.Build();
app.UseFastEndpoints()
   .UseSwaggerGen();
app.Run();

sealed class MyRequest
{
    public string FirstName { get; set; }
    public string LastName { get; set; }
}

sealed class MyEndpoint : Endpoint<MyRequest>
{
    public override void Configure()
    {
        Get("test");
        AllowAnonymous();
    }

    public override async Task HandleAsync(MyRequest r, CancellationToken c) { }
}

if that doesn't help, let me know how to replicate the issue you're having.


Seeing threading errors since last pull

Seeing these errors since the fixes yesterday.
Exception in thread Thread-159:
Traceback (most recent call last):
File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
self.run()
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 98, in run
[cuda:0] >>> received hypernetwork query response: SD indicates 0 hypernetworks available for use...
self.callback(response)
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 515, in sampler_response
[cuda:0] >>> received model query response: SD indicates 0 models available for use...
samplers.append(i['name'])
TypeError: string indices must be integers
[cuda:0] >>> received LoRA query response: SD indicates 0 LoRAs available for use...
Exception in thread Thread-163:
Traceback (most recent call last):
File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
self.run()
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 158, in run
self.callback(response)
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 678, in script_response
Exception in thread Thread-164:
Traceback (most recent call last):
File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
for i in r['txt2img']:
KeyError: 'txt2img'
self.run()
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 170, in run
self.callback(response)
File "/mnt/data/AI/dream-factory/scripts/sdi.py", line 696, in upscaler_response
upscalers.append(i['name'])
TypeError: string indices must be integers

I should also note that the errors are intermittent

It looks like something happened with your Auto1111 instance where the initial API calls to fetch models, LoRAs, etc all failed. I haven't seen that happen on my end ever so I'm not sure what might cause it. I'm assuming if you restart everything it should work properly.

It seems to persist through restarts, I can try to reinstall Auto1111, what gets me is that its intermittent. Loading Auto1111 itself seems to work just fine. I will reinstall and let you know.

Pulling the latest dream factory and reloading auto1111 from the newly supported commit appears to have resolved this.

reloaded auto1111 and pulled the new version of dream factory, but the issue persists. I will continue to try various fixes and report back if i am able to resolve the issue

Doing a little digging on this... updated some code
`    # handle server sampler response
def sampler_response(self, response):
try:
r = response.json()
except ValueError as e:
self.log(f'Error decoding JSON: {e}')
return
    samplers = []
    sampler_str = ''
    if isinstance(r, list):  # Add this check
        for i in r:
            try:  # Add a try/except block to handle dictionaries without 'name'
                samplers.append(i['name'])
                sampler_str += '   - ' + i['name'] + '\n'
            except TypeError:
                self.log(f'Error: expected a dict, got {i}')
            except KeyError:
                self.log(f'Error: dict does not have "name" key: {i}')
    else:
        self.log(f'Error: expected a list, got {r}')

    self.busy = False`

and now seeing this error in the startup logging, [cuda:0] >>> Error: expected a list, got {'detail': 'Not Found'}

Narrowing this down, it looks like sometimes (often) it takes longer than what is expected for the /sdapi/v1/samplers API and I assume others as well, to be ready. I can curl the endpoint as dream factory is starting and get the detail: not found response but a few minutes later the endpoint will begin to return correctly. I am going to guess this is due to my models being on a NAS and having to load before the API begins to respond.

That makes sense. If you want a short-term fix, you can open scripts/sdi.py and add this right before line 276:
time.sleep(5)

That will make DF wait 5 seconds before proceeding after the Auto1111 API first becomes available. Increase the time if 5 secs isn't enough. Leave this issue open and I'll get a config.txt option for a startup delay added as soon as I can.

I think i rigged up a wait and retry for the server initialization. I will make a pull request for you to review

Looks like other APIs may finish getting ready after the samplers API does, but the code fix looks to have reduced the error rate significantly. The other api that caused it to fail was sd-models.


[python/ci] Provisional/evaluation Linux + arm64 builds

Context
As discussed in Slack. Non-urgent but worth some experimentation.
At present we have Linux x86, MacOS x86, MacOS arm.
Use cases:

AWS Graviton
Anyone doing docker-build/docker-run of Linux images on MacOS M-series systems

Many thanks to @ihnorton and @jdblischak for support on making these evals happen.
Evaluation results: Docker
To build: please place the following in an otherwise empty directory as Dockerfile.eval:
FROM python:3.11

SHELL ["/bin/bash", "-c"]

RUN apt-get update && \
    apt-get install -y build-essential zip unzip tar ninja-build && \
    apt-get install -y libhdf5-dev && \
    apt-get clean && rm -rf /var/lib/apt/lists/*

WORKDIR /app

# required for libtiledb source build

ENV VCPKG_FORCE_SYSTEM_BINARIES=1
ENV VENV="/opt/env"
ENV PATH="$VENV/bin:$PATH"

RUN python3 -m venv $VENV

RUN pip3 install --upgrade pip wheel

RUN pip3 install cellxgene-census

Then:
docker build -f Dockerfile -t linux-aarch64-eval .

To run:
alias dreb='docker run --rm -it --entrypoint=/bin/bash'
dreb linux-aarch64-eval
export MAMBA_NO_BANNER=1
mamba install -c jdblischak/label/experimental -c conda-forge -c tiledb tiledbsoma-py
python

import tiledbsoma
tiledbsoma.show_package_versions()
# ...

Evaluation results: Conda
Artifacts: https://anaconda.org/jdblischak/tiledbsoma-py/files?channel=experimental
Usage:
alias dreb='docker run --rm -it --entrypoint=/bin/bash'
dreb condaforge/miniforge3
export MAMBA_NO_BANNER=1
mamba install -c jdblischak/label/experimental -c conda-forge -c tiledb tiledbsoma-py

Full support
Tracked in #2482 and [sc-46090]

[sc-44849]

Completed provisional test. Follow-up on #2482.


Simplifies loop on MySQL dependency linking

fyi @hfgbarrigas

rationale for this is I'm trying to squeeze out the internal DependencyLinkSpan type (for #1499), and this simplifies the code that depends on it.


Nested MudMenu doesn't work in MudDialog

Bug type
Component
Component name
MudMenu or MudDialog
What happened?
Add a nested MudMenu to a MudDialog and the second level is not displayed.
Expected behavior
The second level of the nested muddialog should be displayed
Reproduction link
https://try.mudblazor.com/snippet/GkGouGwLmICTOhpG
Reproduction steps

Add a muddialog
Place a nested mudmenu in the muddialog  DialogContent
.

Relevant log output
No response
Version (bug)
6.16.0
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Technically it's displayed, it's just behind the dialog, so I'm guessing this is related https://github.com/MudBlazor/MudBlazor/issues/7634 which will lead us to this problem https://github.com/MudBlazor/MudBlazor/issues/6909
Have you tried to increase the mudpopover zindex?
https://github.com/MudBlazor/MudBlazor/issues/6909#issuecomment-1926794841
but you probably need to do in against dialog not the appbar.

Technically it's displayed, it's just behind the dialog, so I'm guessing this is related #7634 which will lead us to this problem #6909 Have you tried to increase the mudpopover zindex? #6909 (comment) but you probably need to do in against dialog not the appbar.

I did notice that it's behind, but didn't want to "guess" a solution since u guys r the experts in this domain. :) I did try setting z-index: 9999 with no result. i can try<PHONE_NUMBER> (highest zindex value) if it works, until the issue you references is resolved.
As always, thanks for the reply - i'll keep an eye open for the fix.


Investigate and create XCall standards for Balanced

While starting to implement the xTokenStandard spend time on defining how we want to work with XCall.
PR should contain:
How do we write our xCall methods?
How do we unit test our xCall methods?

WIP in https://github.com/AntonAndell/XCall-lib
TODO in balanced repo, create structure to create mocked xCalls on integration tests


[mbta] Location-based stop picker

@betterengineering Thanks! Are there any other steps needed to get the app into the apps directory?


UIButton.setBackgroundColor(_:for:)

🚀 Is your feature request related to a problem? Please describe.
We already have UIButton.setTitleColor(_:for:), but sometimes we need to set background color for different states.
Describe the solution you'd like
Took it from https://stackoverflow.com/a/30604658/2229783
extension UIButton {
    func setBackgroundColor(color: UIColor, forState: UIControl.State) {
        self.clipsToBounds = true  // add this to maintain corner radius
        UIGraphicsBeginImageContext(CGSize(width: 1, height: 1))
        if let context = UIGraphicsGetCurrentContext() {
            context.setFillColor(color.cgColor)
            context.fill(CGRect(x: 0, y: 0, width: 1, height: 1))
            let colorImage = UIGraphicsGetImageFromCurrentImageContext()
            UIGraphicsEndImageContext()
            self.setBackgroundImage(colorImage, for: forState)
        }
    }
}

[x] I've read, understood, and done my best to follow the Contributing guidelines before opening this issue.

Hey @RomanPodymov. I would really love to contribute to this. We have to add the extension method in the UIButtonExtensions.swift.
    /// SwifterSwift: Set background color for specified state.
    /// - Parameters:
    ///   - color: UIColor.
    ///   - forState: set the UIControl.State for the desired color.
    func setBackgroundColor(color: UIColor, forState: UIControl.State) {
        self.clipsToBounds = true  // add this to maintain corner radius
        UIGraphicsBeginImageContext(CGSize(width: 1, height: 1))
        if let context = UIGraphicsGetCurrentContext() {
            context.setFillColor(color.cgColor)
            context.fill(CGRect(x: 0, y: 0, width: 1, height: 1))
            let colorImage = UIGraphicsGetImageFromCurrentImageContext()
            UIGraphicsEndImageContext()
            self.setBackgroundImage(colorImage, for: forState)
        }
    }

I also added a test for the method in UIButtonExtensionsTests.swift. It was passed without errors.
    func testSetBackgroundColorForState() {
        let button = UIButton()
        let color = UIColor.green

        button.setBackgroundColor(color: color, forState: .highlighted)

        let backgroundImage = button.backgroundImage(for: .highlighted)

        let averageColor = backgroundImage!.averageColor()!

        XCTAssertEqual(averageColor, color, accuracy: 1.0)
    }

If you wish to accept these changes, I will gladly create a PR for this. Thanks.

Hello @TwizzyIndy
Yes, you can open a pull request with that feature.


Add a Gitter chat badge to README.md

aurelia/validation now has a Chat Room on Gitter
@janvanderhaegen has just created a chat room. You can visit it here: https://gitter.im/aurelia/validation.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Actually...I'd really prefer to keep all official aurelia talk in the same gitter room if possible :) Can we redirect to thee aurelia/discuss room? Also, I guess we should add these badges to the other repos too.

Updated :)


"corrupt deflate stream" when inferring schema from CSV file during CREATE EXTERNAL TABLE

Describe the bug
Inferring the schema does not work with a compressed file, in some cases.
To Reproduce
Create a file like this one, and gzip it:
❯ cat file1.csv
Region,Units
United States,10
Canada,20
United States,10
Canada,20
United States,10
Canada,20
Canada,20
❯ gzip file1.csv

I couldn't duplicate this error with a gzipped file of exact repeating rows. It requires some variation in the rows (or there's something else that triggers it).
Then attempt to make a table from it:
❯ create external table file_csv stored as csv with header row compression type gzip location './file1.csv.gz';
ArrowError(CsvError("corrupt deflate stream"))

Expected behavior
It shouldn't throw an error and should be able to create the external table.
Additional context
It works fine with the same file if you specify the table schema:
❯ create external table file_csv(a string,b int) stored as csv with header row compression type gzip location './file1.c
sv.gz';
0 rows in set. Query took 0.000 seconds.
❯ select * from file_csv;
+---------------+----+
| a             | b  |
+---------------+----+
| United States | 10 |
| Canada        | 20 |
| United States | 10 |
| Canada        | 20 |
| United States | 10 |
| Canada        | 20 |
| Canada        | 20 |
+---------------+----+
7 rows in set. Query took 0.003 seconds.

It looks like the schema inference logic is applying the newline delimiting chunking before applying the decompression, it should be a case of reversing the order of these two. In particular using FileCompressionType::convert_stream instead of FileCompressionType::convert_read


Invalid syntax

from word2word import Word2word
Traceback (most recent call last):
File "", line 1, in
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 664, in _load_unlocked
File "", line 634, in load_backward_compatible
File "C:\Users\Admin\AppData\Local\Programs\Python\Python35\lib\site-packages\word2word-1.0.0-py3.5.egg\word2word_init.py", line 6, in
File "", line 969, in _find_and_load
File "", line 954, in _find_and_load_unlocked
File "", line 896, in _find_spec
File "", line 1136, in find_spec
File "", line 1112, in _get_spec
File "", line 1093, in _legacy_get_spec
File "", line 444, in spec_from_loader
File "", line 530, in spec_from_file_location
File "C:\Users\Admin\AppData\Local\Programs\Python\Python35\lib\site-packages\word2word-1.0.0-py3.5.egg\word2word\word2word.py", line 44
f"custom bilingual lexicon is only partially provided. "
^
SyntaxError: invalid syntax

Hi @prasanna532,
It looks like you're using word2word with Python 3.5, but the package requires Python 3.6 or higher.
Please try installing it with Python 3.6+ and see if the problem still exists.
Thanks for trying out our package!


highlight name clicked on

When clicking on a variable (or other), the name gets bold and change size thus causing the rest of the line in the document to move. Visually, this makes it really hard to follow what is going on. Any way to make it go bold without moving the rest of the line ?

This should be avoided if you use a monospace font

Apparemtnly it's a known problem of Fira. Any way to change the behavior in Juci++ ?


Resolve a couple of bugs in webhook creation

Coverage: 87.526%. Remained the same when pulling 824b09bbff2a53fca5e86447813eb638a6b816db on bug/webhook-creation into 57f89a0589efa1aa4f6138cfa64caa7e72f6fc1c on main.


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2023-01-30T18:12:26+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
423ff53135ac5d2ad90a0ce4b6a39509316d374c
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests-arm64

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test lint

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test lint

integ-ipv6_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test integ-ipv6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test lint

integ-ipv6_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test integ-ipv6

integ-security-multicluster_istio
475580003a48a9f26a350dfc3014a6a26899375d
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Models for Stream Layer added

Models were generated using Swagger from OLP Stream API v2 and adapted
to be consistent with existing models for other layers.
All Doxygen documentation corresponds to OLP Stream API v2 descriptions.
Original OLP Stream API v2 models CommitOffsetsRequest and
SeekOffsetsRequest are represented in the code by single OffsetsRequest
model to reduce code duplication because they are completely the same.
Relates-To: OLPEDGE-1196
Signed-off-by: Mykola Malik<EMAIL_ADDRESS>

Would it make sense to keep with the widely-accepted convention of using imperative in commit messages, e.g. "Fix bug" and not "Fixed bug"? This convention matches up with commit messages
generated by commands like git merge and git revert.

Closed due to splitting PR into couple smaller ones - one model per one PR.


Set ExpansionFactor to 0.06 when it's equals to 0 in the private dict of CFF fonts

That's a fascinating find, how did you figure it out?

I compared the original font I extracted from the pdf and the one we generate.
I ran tx on both fonts and noticed some differences in the charsets, so I manually fix the charset to have exactly the same as in the original font but nothing.
I ran fontlint on the generated font and noticed few errors about BlueValues, I fixed them but nothing. I was pretty sure that something was wrong in the top dict.
Finally I had the wonderful idea to just remove the private dict and magically it worked, then I just tried to remove the properties one by one and finally the last one (as usual) was the culprit.

/botio test

/botio test

@Snuffleupagus instead of removing the property I set it to 0.06 to follow Jonathan's advice: https://github.com/mozilla/pdf.js/issues/15289#issuecomment-1374484221
Are you ok with this change ?

Are you ok with this change ?

Yes, since I do agree that it sounds like an even safer approach here.
Given the amount of duplicates, we might want to uplift this to previous Firefox versions?

As a follow-up to this PR, to prevent future bugs, I do wonder if we want to add similar code when parsing "regular" Type1 fonts as well or if should we perhaps wait until an actual bug is reported?
https://github.com/mozilla/pdf.js/blob/d6984f93954a8ac9e92cc9f156f7e909d3e332de/src/core/type1_parser.js#L657

/botio makeref

/botio-windows makeref


Convert shader compilation and occlusion buffer to use parallel for_range()

Simplifies threading compared to direct usage of WorkerThreadPool, since there is no longer a need for intermediate functions or structs.
Some instances of WorkerThreadPool have been omitted from this PR to avoid conflicting with #72687
RaycastOcclusionCull::RaycastHZBuffer::update_camera_rays() may experience performance regressions until #72716 is merged.
This PR passes the regression test here: https://github.com/godotengine/godot-benchmarks/issues/34
This PR has not been tested on https://github.com/Calinou/godot-rendering-tests

I'm only a bit concerned with the capture-by-value ([=]) thing. In this case it's fine, but there's a risk of over-capturing as the code evolves.

I think I initially went with [=] where possible for safety reasons, but for consistency sake perhaps it's better to just standardize on [&](const ...) {} so that it can be a drop-in replacement for any for loops that are embarrassingly parallel. Over-capturing shouldn't be a problem here, since parallel for loop bodies should generally be kept as small as possible anyway to make the multithreading easier to reason about.
Of course, other lambda use cases might need different rules.

We can just let this go and try to come up with rules (like capturing this explicitly and using interim locally-scoped structs with all the values needed inside the lamba when the list would be long) for this and revise all usages at some point.

If for_range() becomes more widely used across the Godot codebase, it might be a good idea to make specialized macros that hide the lambda entirely.
The current for_range() signature is a bit ridiculous anyway since I was prioritizing feature parity with WorkerThreadPool::add_*_group_task() and for(), so replacing it at some point is basically mandatory.

Well, in #79490 you are using [&]. I'd suggest this one being consistent with it indeed.


Include prebuilt macOS binary or build instructions

I'm getting the following error message when following installation instructions on macOS 10.11.6:

I then checked the contents of UE4OSCPlugin folder and I can see that there indeed are no macOS binaries inside:
./Binaries
./Binaries/Win64
./Binaries/Win64/UE4Editor-UE4OSCPlugin.dll
./Binaries/Win64/UE4Editor-UE4OSCPlugin.pdb
./Binaries/Win64/UE4OSC.dll
./Resources
./Resources/Icon128.png
./Source
./Source/UE4OSCPlugin
./Source/UE4OSCPlugin/Classes
./Source/UE4OSCPlugin/Classes/UE4OSCPluginOSCInterface.h
./Source/UE4OSCPlugin/Private
./Source/UE4OSCPlugin/Private/UE4OSCPlugin.cpp
./Source/UE4OSCPlugin/Private/UE4OSCPluginOSCInterface.cpp
./Source/UE4OSCPlugin/Private/UE4OSCPluginPrivatePCH.h
./Source/UE4OSCPlugin/UE4OSCPlugin.Build.cs
./UE4OSCPlugin.uplugin

Hi.
I do not provide prebuilt package for any platform.
Did you read the README.md? "Integration" section?

You do actually provide a Windows binary in the UE4OSCPlugin folder (and I am certain about that because I'm on a Mac that for sure has no ability produce Windows DLLs), which the integration instructions suggest I should copy in the newly created Plugins folder ("Copy UE4OSCPlugin folder to that directory").

DUH! Looks like my issue is simpler than that… There's a plugin 'UE4-OSC' and one called 'UE4OSC' and I mixed up the two when googling to find the project GitHub page… I was attempting to install your plugin with instructions for https://github.com/zachrkn/UE4OSC :D

:)
It felt weird indeed.


Does this action also auto-creates tags, while protecting from malicious tag override?

I'm planning on using this action to lower the risk associated to GitHub Actions we are using in my company.
Basically, referencing an external action in our actions, such as: uses<EMAIL_ADDRESS>might result in fatal failure if the repository owner (rlespinasse in this case) would delete their repository from GitHub.
Also, referencing tags (e.g: v3.4) isn't actually safe, as a repository could be hacked, or an error could be made, and a git tag could be changed to point to another commit containing malicious code. Therefore, forking a repository is the safest way of protecting our own work, by not relying on something that might break at any time, due to unforeseen and uncontrollable circumstances.
That's why I'm thinking to manually fork each action and use those forks, instead of using the real action. But then, keeping those forks up to date becomes a real pain.
That's where I'm thinking of using your work tgymnich/fork-sync, to keep those forks up-to-date.
And I need to auto-update the fork based on the source's changes (which is supported), but I'd also need to auto-create the new tags, but not allow editing of existing tags (e.g v4.3.1), unless they're allowed to be updated (e.g: v4.x, v3).
Is that something you might consider adding? I'm interested to know a bit better what's your personal use-case for using this action.

I am using this action just to periodically merge changes from a forked repository into my fork. Creating tags would still fit into the scope of this action. The verification logic not so much, because it is quite a special use case.

I am using this action just to periodically merge changes from a forked repository into my fork. Creating tags would still fit into the scope of this action. The verification logic not so much, because it is quite a special use case.


Add CI support

There is no automated CI for HDFS-on-K8s. Let's add one.

We learned travis CI now supports minikube. (blog) This could be useful for our CI as contributors can easily run integration tests using a minikube.

I played with travis + minkube. And success! The following is from inside a travis build (link].
$ kubectl cluster-info
Kubernetes master is running at https://<IP_ADDRESS>:8443
To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.

Done. Your build exited with 0.

CC @cvpatel @foxish @erikerlandson

That is awesome. Thanks for testing that. We should probably use that
moving forward for all single node testing. cc/ @dimberman
On Wed, Mar 28, 2018, 2:31 PM Kimoon Kim<EMAIL_ADDRESS>wrote:

I played with travis + minkube. And success! The following is from inside
a travis build (link
https://travis-ci.org/kimoonkim/kubernetes-HDFS/builds/359574323].
$ kubectl cluster-info
Kubernetes master is running at https://<IP_ADDRESS>:8443
To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.

Done. Your build exited with 0.
CC @cvpatel https://github.com/cvpatel @foxish
https://github.com/foxish @erikerlandson
https://github.com/erikerlandson
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache-spark-on-k8s/kubernetes-HDFS/issues/25#issuecomment-377044118,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3U54c-wlWrDnUJGZrDhjz-VEwqa3Koks5tjAFKgaJpZM4Pjb_i
.

Yes, airflow and other k8s projects can benefit from this.
@foxish I want to hook up travis in this repo apache-spark-on-k8s/kubernetes-HDFS. Currently I only see the spark repo under https://travis-ci.org/apache-spark-on-k8s/. Maybe you can enable travis in the settings page of the kubernetes-HDFS repo?
FYI, my fork settings/installations page, https://github.com/kimoonkim/kubernetes-HDFS/settings/installations, has the following:
Installed GitHub Apps
GitHub Apps augment and extend your workflows on GitHub with commercial, open source, and homegrown tools.

Services
Services are pre-built integrations that perform certain actions when events occur on GitHub.

Travis CI

Or, if I can be made admin of this repo, I can do it myself.

Made you an org owner.

Thanks. I see the settings button now.

Seems to work. #34 triggered a build.


hide remove button until for now

 BUG: Delete user breaks - Hide Link for now

I think just delete the line entirely rather than leave commented out code in there. Can always restore from version control.

I think just delete the line entirely rather than leave commented out code in there. Can always restore from version control.

Done


feat(custom components page)

[ ] Master component only be edited on the custom components page

[ ] Make ui for expose as

[ ] No interaction for the children of custom components in main page.

[ ] Enable styling like padding margin etc. for the created custom components in the main page.

[ ] Enable expose as for the components in custom components page.

[ ] Get the PR ready for review


C++11 alignas instead of our existing hack

Summary:
Now we malloc and check for alignment. C++11 provides an alignas mechanism where we can guarantee alignment on double.
Current Version:
v2.15.0

I think alignas with malloc will not work. But once we move to C++17 we can use std::aligned_alloc: https://en.cppreference.com/w/cpp/memory/c/aligned_alloc
We will be able to clean the stack_alloc.hpp file then.

Leaving this as we should do what @rok-cesnovar is saying if we ever bump to 17


Either misaligned overlay or linked brushing failure when datasource is shared but all axes are not

I am overlaying a Scatter object onto a BoxWhisker object using kdim='cyl' and vdim='mpg' and coloring the scatter points and box using dim('cyl').str(). I would now like to add another Scatter object to the layout using the same datasource and color based on cyl, but use different variables for the x and y axes.
The code below behaves as desired (linked brushing, different x and y axes between Scatter objects), with the noted exception that the second Scatter object is not colored by cyl.
import holoviews as hv
from holoviews import dim,opts
from holoviews.plotting.links import DataLink

from bokeh.models import HoverTool
from bokeh.sampledata.autompg import autompg as df

hv.extension('bokeh')

df = df.sort_values(by=['cyl'])

tooltips = [
    ('cyl', '@cyl'),
    ('mpg', '@mpg'),
    ('accel', '@accel'),
    ('displ', '@displ')
]

hover = HoverTool(tooltips=tooltips)

boxwhisker = hv.BoxWhisker(df, 'cyl', 'mpg').opts(show_legend=False,
                                                  box_fill_color=dim('cyl').str(),
                                                  cmap='Set2',
                                                  outlier_alpha=0.0,
                                                  box_fill_alpha=0.1)

scatter1 = hv.Scatter(df, 'cyl', ['mpg','displ','accel','origin'])
scatter2 = hv.Scatter(df, 'displ', ['accel'])

overlay = scatter1.opts(jitter=0.6) * boxwhisker

dlink = DataLink(overlay, scatter2)

((overlay + scatter2).opts(
    opts.Scatter(tools=['box_select', 'lasso_select','tap',hover],
                 color=dim('cyl').str(),
                 cmap='Set2',
                 alpha=0.5,
                 selection_fill_alpha=1.0,
                 nonselection_fill_alpha=0.1,
                 size=5,
                 line_color='k',
                 show_legend=False),
    opts.Layout(shared_datasource=True)))

However, if I change scatter2 to include cyl in its vdims, i.e. scatter2 = hv.Scatter(df, 'displ', ['accel','cyl']), the colors match between Scatter objects, but the scatter1 points become misaligned with the boxplots. If I try sorting the Scatter objects, e.g. scatter1.sort(['cyl']), the colors and alignment are as desired, but linked brushing is then broken.
Thanks for your consideration and you contributions on this beautiful library.

Additionally, I notice that "live" brushing works for Lasso Select but not Box Select. Is there a setting or configuration that I'm missing?

This issue is the same as #3935, I realize this one came earlier but that one has a more precise title.

Additionally, I notice that "live" brushing works for Lasso Select but not Box Select. Is there a setting or configuration that I'm missing?

That's odd, can't reproduce this.


Trouble after eject —at Sign in Sign out

So this project works as expect until ejecting. Android still works from command line, but cannot npm run IOS from command line, because of this error:

Manfreds-MacBook-Pro:mvp manfredneustifter$ npm run ios

<EMAIL_ADDRESS>ios /Users/manfredneustifter/mvp
react-native run-ios

Scanning folders for symlinks in /Users/manfredneustifter/mvp/node_modules (34ms)
Found Xcode project mvp.xcodeproj
xcodebuild: error: SDK "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk" cannot be located.
xcrun: error: unable to find utility "instruments", not a developer tool or in PATH
Command failed: xcrun instruments -s
xcodebuild: error: SDK "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk" cannot be located.
xcrun: error: unable to find utility "instruments", not a developer tool or in PATH
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>ios: react-native run-ios
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>ios script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/manfredneustifter/.npm/_logs/2017-11-04T04_42_49_406Z-debug.log
Manfreds-MacBook-Pro:mvp manfredneustifter$

--> But if I open the project in Xcode, it does build successfully, and can run on emulator in Xcode.
However, after the next steps: Sign in Sign out, neither android nor ios work.
https://github.com/awslabs/aws-mobile-react-native-starter#sign-up-and-sign-in-
On android, the build seems to work, but when opening the app in the android emulator I get this:

and on IOS this:

App.js
`import React from 'react';
import { StyleSheet, Text, View, Button} from 'react-native';
import { WithAuth } from './lib/Categories/Auth/Components';
import { SignIn, SignUp } from './lib/Categories/Auth/Components/Examples';
export default WithAuth(class App extends React.Component {
render() {
const { session } = this.props;
return (
  session ?
    (<View style={styles.container}>
      <Button title="Sign Out" onPress={() => this.props.doSignOut()} />
    </View>)
    :
    (<View style={styles.container}>
      <SignIn {...this.props} />
      <SignUp {...this.props} />
    </View>)
);

}
});
const styles = StyleSheet.create({
container: {
flex: 1,
backgroundColor: '#fff',
alignItems: 'center',
justifyContent: 'center',
},
});`

and, is there any way to make this project work with Expo? I dont want to eject

You can resolve PropType Error by implementing react-codemod.
You should this process.

$ yarn global add jscodeshift
In your app directory..
$ git clone https://github.com/reactjs/react-codemod.git
In ../your-app-directory/react-codemod
$ yarn install
In your app directory..
Here,  is your app directory root.
$ jscodeshift -t react-codemod/transforms/React-PropTypes-to-prop-types.js


Question about new network validation on Android

On V2EX I have found a discussion for Android 7.1.1, where I found this version reformed network validation mechanism for WiFi connection, does DNSCrypt affect it?
See: https://www.v2ex.com/t/325630

Unfortunately, my Mandarin Chinese is not so hot. Can you summarize what the concern is about?
Captive portals either hijack HTTP queries, or DNS queries, or both. This makes them fundamentally incompatible with systems such as DNSCrypt that are designed to prevent hijacked responses from reaching your applications.

OK I have a chance now, so let me translate.
As their discussion, this is used to judge whether you have to use additional validation on web browser when you connected a WiFi and give you a notice. But considering several WiFi hotspots may use a LAN connecting for this validation, I concerned DNSCrypt would making this validating failed.

Android 7.1.1 的网络验证改了

Android 7.1.1’s network validating have been changed.

多了个 /generate_204
看来需要在 http204.sinaapp.com 作说明了，这改动还是挺大的

Added /generate_204. It seems to me there is necessary to notice on http204.sinaapp.com, this is such a big change.

请教一下，一直很好奇这个 204 服务是干嘛的？ iPhone 用户没接触过，在 V2 上看到了好多次
iOS 上也有, 比如你到了麦丹劳连上了麦当劳的 wifi, iOS 能判断出这个 WIFI 需不需要再经过一次 web 验证, 其实跟这个一个原理. 在 Android 上, 这个地址本来是连到 Google 服务器上的, 因为墙的原因自然就连不上了, 所以得替换成国内的地址.

I’ve a question to consult you, (I) still curioused what does the 204 service do? iPhone user have not used, but seen it many times on V2 (V2EX).
iOS has it too. For example, in you go to McDonald you connect their WiFi, iOS can judge whether this WiFi need additional web validating, in fact there is the same mechanism on Android, this site in intended to connecting Google’s server, but unvailable because of GFW, so you have to connect domestic site.

哦哦，是不是连接一个公共 wifi 之后，系统会访问这个地址来检验是否需要 web 验证？如果成功返回 204 则表示不需要？
这是一个功能， 另外一个基本功能就是判断你现在链接的 wifi 到底有没有连上互联网，没的话就会有个小图标

Oooh, would the system access this site to check whether this need web validating after connecting a public WiFi? If successed then give back 204 to indicate unecessary?
This is just a function, another basic fonction is to judge whether the Wifi you connected is connecting the Internet, if not then give you an icon.

Captive portals often require disabling DNSCrypt, but you can reenable it after the portal is passed.


Move gallery user-visible strings (e.g. in exceptions) to resources

See #2816

/cc @joyhui @xavierdecoster @joelverhagen

:shipit:

:shipit:


How to handle "Access Denied" Window popup using WinAppDriver ?

Hi,
I am not able to click on the Window popup using winappdriver. I already tried keypress event but no use.

Hi @rajeeshmenoth,
Thanks for reaching out. This is known, and is actually a security feature - posing a security risk is one of the reasons why WinAppDriver does not support this.

Hi,
Any proper solution for this ?

There might be a registry key you can set to automatically launch the application in Administrator mode. We've had mixed results with that at work with a different application (it works for a bit, then it doesn't.)
To get around this, I turned off UAC entirely on our test agents. This is super not recommended as it's a main security feature for Windows and requires admin privileges to do this. But I can confirm it works as a last resort.

Hi @rajeeshmenoth,
This is by design, WinAppDriver is not able to interact with elevated system prompts.


simverb3500 link broken

Description
http://ci.mxnet.io/blue/organizations/jenkins/GluonNLP-py3-master-gpu-doc/detail/PR-783/3/pipeline#step-64-log-1246
What have you tried to solve it?
We will skip its test for now which needs to be added back.

is this fixed or skipped?

skipped. the functionality is broken.


Added a new resource

Hey, I have added a reference link for Event Loop in JS. This reference will make the content more valuable. Hope this small contribution will help other learners.

Hey, I have added a reference link for Event Loop in JS. This reference will make the content more valuable. Hope this small contribution will help other learners.

Thanks for your contribution.


Rename the client / transport interface

IStatsDUdpClient renamed to IStatsDTransport
Since we expect to have a TCP one

50% of the way there on the timings...


Booleans are handled as strings

Hi,
I like this library, and I switched to it from another Arduino library a few days ago mainly because the error handling is way better.
However, this library seems to save booleans as strings in Firebase. Is there any way to avoid or change that?
The problem is that other applications (such as Android SDK, which I use) use real booleans, and this obviously is incompatible (unless handled specially).
I am no Firebase expert, but I think it would be wise to use the method provided by Firebase: https://firebase.google.com/docs/firestore/manage-data/data-types
Meanwhile, I will make a work-around somehow, but as far as I can tell in your library, it's not straight forward to modify your central function FirebaseESP8266::sendRequest() since it already in the parameters takes a string as input.
Anyway - I already modified your getBool to make it possible to read boolean values. Not the most elegant and bullit-proof way (but it does read "real" booleans off Firebase):
bool FirebaseESP8266::getBool(FirebaseData &dataObj, const String &path)
{
    dataObj.queryFilter.clear();
    bool flag = sendRequest(dataObj, path.c_str(), FirebaseMethod::GET, FirebaseDataType::BOOLEAN, "");
	if (flag && dataObj._dataType == FirebaseDataType::BOOLEAN) {
		flag = true;
	} else if (flag && dataObj._dataType == FirebaseDataType::INTEGER) {
		flag = true;
	}
	else {
		flag = false;
	}
	return flag;
}

Sorry for this issue due to my misunderstanding at the first time I implement it.
Now the boolean issue was solved in version 1.0.5 which you can update.

Seems to work in 1.0.5 - thanks for quick reply!!!


[BUG] Traceback (most recent call last)

Example file
comment_hashtags

Describe the bug
i type "pip install -U instabot"
after that i type "python comment_hashtags.py comments.txt ceramah videoislam fotoislam"
this problem arose
Log
C:\Users\nizar\examples\comment>python comment_hashtags.py comments.txt ceramah videoislam fotoislam
2020-02-23 05:58:15,093 - INFO - Instabot version: 0.110.0 Started
Which account do you want to use? (Type number)
1: ferdikun9
0: add another account.
-1: delete all accounts.
1
2020-02-23 05:58:17,151 - INFO - Not yet logged in starting: PRE-LOGIN FLOW!
Traceback (most recent call last):
  File "comment_hashtags.py", line 30, in <module>
    bot.login()
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\bot\bot.py", line 443, in login
    if self.api.login(**args) is False:
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api.py", line 266, in login
    self.pre_login_flow()
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api.py", line 182, in pre_login_flow
    return pre_login_flow(self)
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api_login.py", line 107, in pre_login_flow
    self.sync_device_features()
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api.py", line 155, in sync_device_features
    return sync_device_features(self, login)
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api_login.py", line 23, in sync_device_features
    data["_uid"] = self.user_id
  File "C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\api\api.py", line 616, in user_id
    return self.cookie_dict["ds_user_id"]
KeyError: 'ds_user_id'

To Reproduce
(EXAMPLE:) Steps to reproduce the behavior:
cd examples\comment
python comment_hashtags.py comments.txt ceramah videoislam fotoislam
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.
Version
Python version (Python 3.8.1):
OS: Win 10

Please post the log file that is saved in the config/log/ folder.

2020-02-23 05:58:15,093 - instabot version: 0.110.0 (bot) - INFO - Instabot version: 0.110.0 Started
2020-02-23 05:58:15,093 - instabot version: 0.110.0 (bot) - DEBUG - Bot imported from C:\Users\nizar\AppData\Local\Programs\Python\Python38-32\lib\site-packages\instabot\bot\bot.py
2020-02-23 05:58:17,151 - instabot version: 0.110.0 (api_login) - INFO - Not yet logged in starting: PRE-LOGIN FLOW!
2020-02-23 05:58:19,559 - instabot version: 0.110.0 (api) - DEBUG - POST to endpoint: accounts/contact_point_prefill/ returned response: <Response [200]>

Merging with this issue: https://github.com/instagrambot/instabot/issues/1317


Problem importing paypal-checkout xcomponent_delegate_paypal_checkout_login.

When I import paypal-checkout show me the follow error in console:

And my project dies.
I'm trying to implement a ButtonComponent in react.

Thank you for your help!!!

Hmm. Is it possible the paypal script is being pulled in multiple times? Do you have a script tag which loads paypalobjects.com/api/checkout.js?

No, that place is the only place when im import it, that is weird.

I went to the checkout.lib. js file to verify the error, and printed the existingListener variable, I don't know why the listener is duplicated but is there any way to remove the duplicate?

Im using:
"react": "15.3.2",
"react-dom": "15.3.2",
"webpack": "1.12.11"

Can you add a console.trace there so you can see why it's being registered the first time? I think we're gonna have to figure that out to get around this, as it could cause more problems even if we work around this specific instance.
if (name === 'xcomponent_delegate_paypal_checkout_login') {
    console.warn('Registering xcomponent_delegate_paypal_checkout_login on', window.location.href);
    console.trace();
}

Hello again, @bluepnume Here is the result. What do you think? thanks.

Can you do the trace when (name === 'xcomponent_delegate_paypal_checkout_login')? In your screenshot you're only getting a trace for the call which we know about. I'm interested to know why this listener was registered the first time.

Ok, look @bluepnume, I appreciate your help..

OK so -- looks like it's running through the exact same points in your own bundle.js and PaypalButton.js multiple times. Is it possible PaypalButton.js is being loaded multiple times in the page?
If you have an example of this up somewhere I'll be happy to help debug.

Ohh, it's my mistake @bluepnume!!, I hadn't noticed that I had used the script tag before and between the tests I added it again and this last one was the one that I removed and which I was sure that only that put, but well I eliminated the missing script tag and solved the problem, thank you very much for your time and support. ;), Thank you very much!!.


[Improvement][Seatunnel-web] Support H2 database in integration tests

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs

integration test execution result using default h2 database

integration test execution result using mysql data by passing -DdbType=mysql

Thanks @Hisoka-X  for reviewing and merging this PR.

With the removal of the dependency on an external database, integration tests can now run using an in-memory H2 database. However, the Seatunnel distribution is still required.
@Hisoka-X, could you provide guidance on how to run the integration tests as part of the PR CI process?

The test in CI in fact is trigger mvn script. Please refer https://github.com/apache/seatunnel/blob/dev/.github/workflows/backend.yml#L645
We already have some config in https://github.com/apache/seatunnel-web/blob/main/.github/workflows/backend.yml#L161
Just need add new script into backend.yml.

However, the Seatunnel distribution is still required.

We can execute mvn install before test with seatunnel-web.

Checkout seatunnel dev branch, https://github.com/apache/seatunnel/blob/dev/.github/workflows/backend.yml#L86
execute mvn install and other script which execute in local before. (Like set SEATUNNEL_HOME)
Run mvn test by mvn script.
all these step can be done with github action run https://github.com/apache/seatunnel/blob/dev/.github/workflows/backend.yml#L315


Crash when fetching data

It is unable to fetch details for more repos and sometimes same repo also if the date range chosen is more than 7 days. Getting the below when All repos flag is selected

It is used as extension in Azure DevOps


Remove check to ensure .cache/json folder exists

Thanks for suggesting a new feature!
Please fill in the sections below.
This is not an enhancement, more of a code cleanup.
#124 ensured the .cache/json folder existed, but this appears to no longer be necessary based on the fix in https://github.com/gatsbyjs/gatsby/pull/33431 (merged in October 2021) that resolved https://github.com/gatsbyjs/gatsby/issues/33262.
Describe the solution you'd like
Revert #124 as this check is no longer necessary.
Describe alternatives you've considered
N/A
Can you submit a pull request?
Yes.
Pull requests are welcome! If you would like to help us add this feature, please
check our contributions guidelines.

Thanks, @nickytonline! We'll get this done in the next release


Falcon bootstrap

Have a repo and a built-in tool or something to pull it down easily to bootstrap new projects.

This can be a separate project under falconry.

Is there still interest in doing something like this (yeoman/generator style or some sort of skeleton hello world code)? If so, there would have to be a falcon-based script/bin to handle this execution, and it would likely be either installed separately, or brought in under setup.py.

This is the old issue. New work is being tracked in #1051.


[Service Bus] Bug with partioned and non-session entities in receiveAndDeleteMode

The first two (commented out) tests in "Receive Deferred messages in ReceiveAndDelete mode" suite of tests are failing.
Steps to reproduce:

Add in tests and run them, they fail and end up with having two messages in the queue/subscription.

Expected behavior:

The tests should pass just like other tests (including counter part tests using session enabled entities)

Looks like the deferred messages aren't being deleted in the Partitioned Queue/Subscription when receiving in ReceiveAndDelete mode.
This could be a Service side issue as we don't do anything different for partitioned vs unpartitioned entities.
Please check the .Net SDK and see if the same operations are successful there.

Findings is that - sample replicating the receiveDeferredMessage() usecase in receiveAndDeleteMode works as expected against partitioned entities.
let sequenceNumber: any;

async function main(): Promise<void> {
  await sendMessages();
  await receiveMessages();
  await receiveDeferredMessages();
}

async function sendMessages(): Promise<void> {
  const ns = ServiceBusClient.createFromConnectionString(connectionString);
  const client = ns.createQueueClient(queueName);
  try {
    const sender = client.createSender();

    const totalNumOfMessages = 1;
    let num = 1;
    while (num <= totalNumOfMessages) {
      const message = {
        messageId: num,
        body: "test",
        label: `${num}`,
        partitionKey: "dummy" // Ensures all messages go to same parition to make peek work reliably
      };
      num++;
      await sender.send(message);
      console.log(`Sent message ${message.messageId}`);
    }
  } finally {
    await client.close();
    await ns.close();
  }
}

async function receiveMessages(): Promise<void> {
  const ns = ServiceBusClient.createFromConnectionString(connectionString);
  const client = ns.createQueueClient(queueName);

  try {
    const receiver = client.createReceiver(ReceiveMode.peekLock);

    const onErrorHandler: OnError = (err: Error) => {
      throw err;
    };

    const receivedMsgs: ServiceBusMessage[] = [];

    const onMessageHandler = async (brokeredMessage: ServiceBusMessage) => {
      receivedMsgs.push(brokeredMessage);
      console.log(`Received message ${brokeredMessage.messageId}`);
      console.log(`Sequence num ${brokeredMessage.sequenceNumber}`);
      sequenceNumber = brokeredMessage.sequenceNumber;
      await brokeredMessage.defer();
      await delay(5000);
    };

    receiver.registerMessageHandler(onMessageHandler, onErrorHandler, {
      autoComplete: false
    });

    await delay(15000);
    await receiver.close();
  } finally {
    await client.close();
    await ns.close();
  }
}

async function receiveDeferredMessages(): Promise<void> {
  const ns = ServiceBusClient.createFromConnectionString(connectionString);
  const client = ns.createQueueClient(queueName);

  try {
    const receiver = client.createReceiver(ReceiveMode.receiveAndDelete);

    const msg = await receiver.receiveDeferredMessage(sequenceNumber as any);
    console.log(msg);
    await receiver.close();
  } finally {
    await client.close();
    await ns.close();
  }
}

It's unclear why the tests are failing. Troubleshooting some more, seems like it might have to do with handling of sequence number / long datatype.
@ramya-rao-a @AlexGhiondea

A key difference between the tests and the above sample is that a whole new ServiceBusClient and QueueClient are being used for receiving deferred messages. Please try the sample by re-using the queue client to be closer to what the test does.

A key difference between the tests and the above sample is that a whole new ServiceBusClient and QueueClient are being used for receiving deferred messages. Please try the sample by having a single ServiceBusClient and QueueClient instance to be closer to what the test does

Yes, able to consistently replicate issue on updating sample to:
let sequenceNumber: any;

let receiverClient: QueueClient;

async function main(): Promise<void> {
  const ns = ServiceBusClient.createFromConnectionString(connectionString);
  try {
    await sendMessages();

    receiverClient = ns.createQueueClient(queueName);

    await receiveMessages();
    await receiveDeferredMessages();
  } finally {
    await receiverClient.close();
    await ns.close();
  }
}

async function sendMessages(): Promise<void> {
  const ns = ServiceBusClient.createFromConnectionString(connectionString);
  const client = ns.createQueueClient(queueName);
  try {
    const sender = client.createSender();

    const totalNumOfMessages = 1;
    let num = 1;
    while (num <= totalNumOfMessages) {
      const message = {
        messageId: num,
        body: "test",
        label: `${num}`,
        partitionKey: "dummy" // Ensures all messages go to same parition to make peek work reliably
      };
      num++;
      await sender.send(message);
      console.log(`Sent message ${message.messageId}`);
    }
  } finally {
    await client.close();
    await ns.close();
  }
}

async function receiveMessages(): Promise<void> {
  const receiver = receiverClient.createReceiver(ReceiveMode.peekLock);

  const onErrorHandler: OnError = (err: Error) => {
    throw err;
  };

  const receivedMsgs: ServiceBusMessage[] = [];

  const onMessageHandler = async (brokeredMessage: ServiceBusMessage) => {
    receivedMsgs.push(brokeredMessage);
    console.log(`Received message ${brokeredMessage.messageId}`);
    console.log(`Sequence num ${brokeredMessage.sequenceNumber}`);
    sequenceNumber = brokeredMessage.sequenceNumber;
    await brokeredMessage.defer();
    await delay(5000);
  };

  receiver.registerMessageHandler(onMessageHandler, onErrorHandler, {
    autoComplete: false
  });

  await delay(15000);
  await receiver.close();
}

async function receiveDeferredMessages(): Promise<void> {
  const receiver = receiverClient.createReceiver(ReceiveMode.receiveAndDelete);

  const msg = await receiver.receiveDeferredMessage(sequenceNumber as any);
  console.log(msg);
  await receiver.close();
}

Looking into logs and code.

Last update on this is that Python and Go SDKs have tested this feature and results are similar indicating service side issue potentially. Service Bus team is taking a look.


Feature: Burn equivalent to torch.retain_grad

Feature description
I'm writing some tests to check gradients again a reference implementation. This works great for leaf nodes, but I can't atm seem to get gradients of intermediate nodes. PyTorch solves this with my_tensor.retain_grad(), which instructs the autodiff engine to keep the gradients during the backward pass. An equivalent in Burn could help with this.
Feature motivation
Testing of gradient activations.
Suggest a Solution
An exact equivalent a la my_tensor.retain_grad(), or, alternatively, make my_tensor.require_grad() valid on non-leaf nodes (currently panics). The semantics of retain/require are sligthly different, but, the use-cases for retained-but-only-if-calculated gradients don't seem that massive to me... not sure!

Any updates with these features? Maybe it's available now in burn, so then how's it possible to use it? I'm working on Gan, and it would be nice if I could use this feature, like in pytorch. Because currently I have to run the generator twice in one iteration to generate fake data in one iteration, in other cases, when I clone fake data, the generator loss grows into NaN after a couple iterations.

Any updates with these features? Maybe it's available now in burn, so then how's it possible to use it? I'm working on Gan, and it would be nice if I could use this feature, like in pytorch. Because currently I have to run the generator twice in one iteration to generate fake data in one iteration, in other cases, when I clone fake data, the generator loss grows into NaN after a couple iterations.

I am implementing my own backward propagation, and I have found a similar approach to Torch's tensor.retain_grad(). However, it is a bit complicated.
First, I can implement Backward trait for my own operator, where I need to register my derived gradients:
use burn::backend::autodiff::{
    checkpoint::{base::Checkpointer, strategy::NoCheckpointing},
    grads::Gradients,
    ops::{Backward, Ops, OpsKind},
    NodeID,
};

#[derive(Debug)]
struct MY_BACKWARD;

// 2-D Current node, 1 Parent node
impl<B: Backend> Backward<B, 2, 1> for MY_BACKWARD
{
    type State = (NodeID, NodeID);

    fn backward(
        self,
        ops: Ops<Self::State, 1>,
        grads: &mut Gradients,
        _checkpointer: &mut Checkpointer,
    ) {
        // TODO: Computing gradients of parents

        // Add entry (ops.state.YOUR_NODE_ID, YOUR_TENSOR_GRAD) to grads' internal container
        grads.register::<B, 2>(ops.state.YOUR_NODE_ID, YOUR_TENSOR_GRAD);
    }
}

Second, remove out the gradient tensors after calling Tensor::backward:
// Deriving the output and get a non-leaf tensor
let (output, YOUR_TENSOR) = model.forward(YOUR_PARAM);

// Computing gradients
let loss = YOUR_LOSS_FUNCTION(output, reference);
let mut grads = loss.backward();

// Extract the gradient of tensor
let YOUR_TENSOR_GRAD = YOUR_TENSOR.grad_remove(&mut grads);

To implement model.forward, you can check the burn-autodiff source code: https://github.com/tracel-ai/burn/blob/756931211c9e1343fe21363fa0febce45869ffb9/crates/burn-autodiff/src/ops/module.rs#L286
Also, it is recommended to see the chapter 11.1.1 of the book here: https://burn.dev/burn-book/advanced/backend-extension/custom-wgpu-kernel.html

@AsherJingkongChen Wow, very good implementation. Maybe when I'll more time I investigate it. thanks


Use geometric algebra?

This looks incredible, but I am not mathy enough to see if and how this can be useful in this project. I think one requisite should be that it does not make things (much) harder than linalg.
Questions:

What would the advantages be exactly?
Can we wrap this stuff up in an API that's actually easy to use?
What are the implications in terms of maintainability (e.g. dependence on someone who understands this stuff)?
Do we only need the PGA3D thing?

Also relevant: https://github.com/pygae/clifford
I don't think this will help making our library easy to adopt though :) I'd rather not spend time on it for now

I don't think this will help making our library easy to adopt though :)

That depends on the second question above :)  If this can be made easy to work with, it'd be a very powerful tool.

I think that's a big if. :)

I made a start: https://github.com/pygfx/pga3
I think this could work, but there are a few open questions:

So far I have not yet seen scaling transforms. Maybe because they cannot be described? If not, is that a bad thing, or could we implement a pre_scale property to scale each world-object (but not its children)?
If we don't allow scaling in our transforms, then we could also represent transforms as a translation plus a quaternion, or using a dual quaternion. Though with a minor optimization, a PGA3 transform would have the exact same performance as a dual quaternion.
I'm pretty sure its possible (because it's done here), but I do not know yet how to create a 4x4 matrix from a pga3 transform.

On scaling: https://discourse.bivector.net/t/scaling-with-pga/43/3 -> no scaling, but this restriction may be a good thing?

On the 4x4 matrix: https://discourse.bivector.net/t/generating-a-4x4-transformation-matrix-from-pga3d/110/6

In pygfx we have the following uses for a linalg system:

Describe points, lines, planes, etc. and do operations on these to e.g. find intersections etc.
Do these for a batch of objects at the same time in one call (vectorized with numpy for performance).
Describe the scale/rotation/translation of world-objects.
Compose the total transform of a world-object with that of its parents (in the scene graph).

For (1) geometric algebra would be awesome. I suspect we could make (2) work as well. For (3) we have a bit of a problem, because geometric algebra cannot describe non-uniform scaling (see link in previous post). This may be overcome by doing a pre-scaling. However, this means that we cannot describe a transform that is the composition of two world-object transforms.
This means that we either:

Accept that we don't have non-uniform scaling - one can use the geometry for this.
Use a pre-scaling that only applies to the object and not it's children.
Accept that we can't do (4), and chain the transforms of the scene graph in the shader.

We can close this in favor of #40


After 1.0a9, datasette-edit-schema to use alter-table permission

See:

https://github.com/simonw/datasette-edit-schema/issues/56

This is done.


outage due to python 3.9 upgrade

At an unspecified date, Colab with upgrade to python 3.9, see https://github.com/googlecolab/colabtools/issues/3450
This will surely cause all packages to break. Recompiling them will solve the issue, but we need to sort https://github.com/fem-on-colab/fem-on-colab/issues/19 out first.

python 3.9 upgrade was committed to https://github.com/googlecolab/backend-info/commit/c004c9961d1462fe3e84dbc6ae6848020051505a, but is not live yet on runtime instances.

python 3.9 upgrade is live, see https://medium.com/google-colab/colab-updated-to-python-3-9-2593f8b1eb79
You may now expect all current packages to be broken. Rebuild is in progress.

All packages have been rebuilt, except for ngsolve (which is blocked by https://github.com/ngsxfem/ngsxfem/issues/19). I'll keep this issue open to collect further feedback, if any, from users, until also ngsolve package gets fixed.

ngsolve is now fixed too.


Airelle-hrsk blocks "Amazon.com"

This file is marked to have no notable false-positives, but blocks Amazon.com (line 15165). This can obviously be whitelisted, but should at least be marked otherwise.
Edit: Sorry, this is in the wrong repo.

Hey @IAreKyleW00t, and apologies for the time it took to reply to this. You're right in that my other repo would be the best one to submit issues on, but none the less, this site does seem to be a false positive.
I've sent an email off to the list maintainer, asking for their input. I'll happily update this ticket once I get a reply.
Thanks for the report! :smile:

Airelle's just informed me that the entry should be removed in the next update.

That's great! Thank you for taking care of that. In the future, should I report false-positive domains to your other repo or try to contact the list maintainer directly? I didn't see any contact info for the lists.

As I'm just forwarding a message to the respective list maintainer, it'd be better to contact them directly. If they don't follow up in a timely manner (or shoot down your requests for valid whitelist removals), then opening a ticket on my other repo (or contacting me via other methods) means I can at least not recommend them as a list if I don't already have them marked with a cross.
You should be able to find a list maintainer email by toggling the source, heading to their website and then snoop around for their contact details (or email the admin listed on the whois lookup).

I completely missed the "Toggle list sources" button - my bad! Thanks for the help and info. :)