xml to json And json to xml

i want parse a xml file , change a value, then save...there will happen a problem..
For example, file's content is  :
 .....
<dict>
    <key>bundle-identifier</key>
    <string>xxx.xxx.xxxxxxx</string>
    <key>bundle-version</key>
    <string>1.1.1</string>
    <key>kind</key>
    <string>software</string>
    <key>title</key>
    <string>xxxx</string>
</dict>
......

code :
parseString(content, function (err, result) {
            console.log(JSON.stringify(result));
            var builder = new xml2js.Builder();
            var xmlResult = builder.buildObject(result);
            console.log(xmlResult);
        });

the xmlResult's content:
.....
<dict>
          <key>bundle-identifier</key>
          <key>bundle-version</key>
          <key>kind</key>
          <key>title</key>
          <string>xxx.xxx.xx</string>
          <string>1.1.1</string>
          <string>software</string>
          <string>xxxxxxx</string>
</dict>
.....

why ? and i how to do ? thanks ...

thats apperently how it works you can provide some options for the parser to preserve order like so:
parser = new xml2js.Parser({ explicitChildren: true, preserveChildrenOrder: true, charsAsChildren: true });
but unfortunately the builder fails to put them together correctly like at all, and also the json output is a salad of different formats :(  +1 for fixing this issue

If you are parsing Property List files (plists) maybe the plist module is a better fit.

@maa105 how to reconstruction   json -> xml
after used new xml2js.Parser({ explicitChildren: true, preserveChildrenOrder: true, charsAsChildren: true });

nothing on this? i mean it has been 4 years and the issue is still there, cannot take an xml input, preserve its node order, get a damn json, and put the xml back together

Nothing on this I'm afraid. I mean, it has been 4 years and still nobody pays me to work on xml2js. My current schedule does not leave me enough time to do a proper rework of the code to parse it in a way where it is properly roundtripable.


csat_apex missing a comma

Bug report

Basic Information

Mission version:
0.96.4
Map used:
Tanoa
Mods used:
ace,CBA_A3, Jsrs soundmod, advanced towing
Dedicated or local game:
Source of the mission file (URL):

https://github.com/KillahPotatoes/KP-Liberation/tree/master/Missionframework
Individual Things
Did you edit anything within the mission files?
yes
If yes, what did you edit exactly?
changed blufor preset to csat_apex by changing the number in the kp_liberation_confing.sqf
Short Issue Description
the game is completely broken, like explained in the faq when there is a missing comma
,due to a missing comma in line 115 of csat_apex.sqf between the vehicle name and the supply requirement
Expected Behaviour

see above
Steps to Reproduce the Issue

change preset to csat_apex
Screenshots, Serverlog or any other helpful Information

Deprecated/Duplicate
https://github.com/KillahPotatoes/KP-Liberation/commit/18ee1e7a2155b8e3cb16c88a95c379e9031ae6b7


Possibility to send some analytics about the use of our project

Maybe thanks to Google Analytics ?

Any others ideas?


Failed openssl 1.4.1 installation on AIX7.2 since XCOFF32 object files are not allowed in 64-bit mode

I am trying to install R packages (R base is 3.6.1) on AIX7.2 server (64 bit) with no internet access.  The openssl 1.4.1 package is failed with the following error message.  Can you please help me to resolve the issue?
R CMD INSTALL -l /software/R/site-library openssl_1.4.1.tar.gz
ld: 0711-738 ERROR: Input file /usr/lib/libssl.so:
XCOFF32 object files are not allowed in 64-bit mode.
collect2: error: ld returned 8 exit status
make: 1254-004 The error code from the last command is 1.
openssl.log

Hi @bergen288,
we distribute OpenSSL only as source code and not in compiled form. Also, 1.4.1 is not one of our version numbers. You might be better off to ask your Manufacturer or Distributor this question.
P.S: A quick google search for

openssl XCOFF32 object files are not allowed in 64-bit mode

brought me to this IBM support page, maybe it helps answering your question:

IBM support - 64-bit applications fail to link against libssl.a or libcrypto.a

Thanks for your hints.  The openssl package has been installed successfully after those 2 obsolete "so" file were renamed.


Bug fix: Page is not being passed to queryPage from Cards.search

paging test was comparing the last card to the first card and giving a false positive
Let me know if you want me to make any changes to this.

Thank you!


ROE-2393 Remove unused sign out page param

JIRA link
https://companieshouse.atlassian.net/browse/ROE-2393
Change description

This was confusing the implementation required for the Redis cache removal. Neither seem to not be used, so deleting.

Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

it looks like url is used in the includes/sign-out-user-banner.html and that is included by layout.html

But the 'url' that was defined here (in the GET method of sign.out.controller.ts) and being passed into a template was going into the sign-out.html page and even though that template does include the layout and banner, in the case of the sign-out page the sign-out banner is not shown.


Pdftops error on MacBook Pro M1

I am trying to install export_fig on my MacBook Pro (Apple M1 Pro) without success. I have used export_fig for many years and really rely on it, so I hope this issue can be solved.
I receive the 'Pdftops error' when I try to save a figure as *.eps, and based on the suggestions, I have tried to follow the details in Issue #137. /usr/local/lib/libfreetype.6.dylib did not exist for me so instead I used the following.
sudo scp /opt/homebrew/Cellar/freetype/2.12.0/lib/libfreetyp
e.6.dylib /Applications/MATLAB_R2021b.app/bin/maci64/libfreetype.6.dylib

When I then try to open Matlab, I get the following error:
Caught "std::exception" Exception message is:
Error loading /Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/mwhg_builtinimpl.dylib. dlopen(/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/mwhg_builtinimpl.dylib, 0x000A): Library not loaded: @rpath/libfreetype.6.dylib
  Referenced from: /Applications/MATLAB_R2021b.app/bin/maci64/libpodofo.0.9.6.dylib
  Reason: tried: '/Applications/MATLAB_R2021b.app/bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/../../sys/os/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/../../sys/os/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/./libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../sys/os/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/./libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../sys/os/maci64/libfreetype.6.dylib' (no such file), '$ORIGIN:/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/Contents/MacOS/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/Contents/MacOS/./libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../standalone/bin/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../sys/os/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/usr/local/lib/libfreetype.6.dylib' (no such file), '/usr/lib/libfreetype.6.dylib' (no such file): No such file or directory: No such file or directory
Error occurred during background graphics initialization: Error loading /Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/mwhg_builtinimpl.dylib.
dlopen(/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/mwhg_builtinimpl.dylib, 0x000A): Library not loaded: @rpath/libfreetype.6.dylib
  Referenced from: /Applications/MATLAB_R2021b.app/bin/maci64/libpodofo.0.9.6.dylib
  Reason: tried: '/Applications/MATLAB_R2021b.app/bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')),
  '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')),
  '/Applications/MATLAB_R2021b.app/bin/maci64/../../sys/os/maci64/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/libfreetype.6.dylib' (mach-o
  file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible
  architecture (have 'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/../../sys/os/maci64/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/./libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have
  'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../sys/os/maci64/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/./libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have
  'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/bin/maci64/builtins/matlab_graphics_hg/../../../../sys/os/maci64/libfreetype.6.dylib' (no such file),
  '$ORIGIN:/libfreetype.6.dylib' (no such file), '/Applications/MATLAB_R2021b.app/bin/maci64/./libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have
  'arm64', need 'x86_64')), '/Applications/MATLAB_R2021b.app/Contents/MacOS/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/Contents/MacOS/./libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../standalone/bin/maci64/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../sys/os/maci64/libfreetype.6.dylib' (no such file),
  '/Applications/MATLAB_R2021b.app/Contents/MacOS/../../bin/maci64/libfreetype.6.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')),
  '/usr/local/lib/libfreetype.6.dylib' (no such file), '/usr/lib/libfreetype.6.dylib' (no such file): No such file or directory: No such file or directory

Have I done something wrong or is this related to the M1 architecture?

As far as I can tell, this is due to the fact that you are using an Intel-based Mac but the FreeType lib you're trying to use is based on the Arm64 architecture, which is incompatible. Try to reinstall pdftops from the Mac64 download link here: https://www.xpdfreader.com/download.html (or just the FreeType lib from the link at the bottom of that page).


Unable to run on Wayland with NixOS!

Bevy version
0.11
[Optional] Relevant system information
If you cannot get Bevy to build or run on your machine, please include:

cargo 1.70.0
NixOS + Hyprland

You should also consider testing the examples of our upstream dependencies to help isolate any setup-specific issue:

wgpu for rendering problems
winit for input and window management
gilrs for gamepad inputs

What you did
I tried running Bevy on Wayland on NixOS, but it crashes!
thread 'main' panicked at 'Failed to initialize any backend! Wayland status: "backend disabled" X11 status: LibraryOpenError(OpenError { kind: Library, detail: "opening library failed (libXcursor.so.1: cannot open shared object file: No such file or directory); opening library failed (libXcursor.so: cannot open shared object file: No such file or directory)" })', /home/jackson/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.28.6/src/platform_impl/linux/mod.rs:757:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

It works on every other distro, and I did make sure to install all the needed dependencies!
Additional information
I have a shell.nix file in my project that contains this:
{ pkgs ? import <nixpkgs> {} }:

pkgs.mkShell {
  buildInputs = [
    pkgs.alsaLib
    pkgs.udev
    pkgs.pkgconfig
    pkgs.rustc
    pkgs.cargo
    pkgs.xorg.libX11
    pkgs.xorg.libXcursor
  ];
}

To rule out the obvious: did you enable the wayland feature of Bevy?
(And disable the x11 feature? I don't think this is necessary but just in case)

To rule out the obvious: did you enable the wayland feature of Bevy?
(And disable the x11 feature? I don't think this is necessary but just in case)

I tried running it on X11. Same issue.

Did you try running your code on another Wayland compositor (e.g., Sway) to see if the problem is specific to Hyprland?

Hello, could you try this shell.nix
{ pkgs ? import <nixpkgs> { } }:

with pkgs;

mkShell rec {
  nativeBuildInputs = [
    pkg-config
    cargo
    rustc
    rustfmt
  ];
  buildInputs = [
    udev alsa-lib vulkan-loader
    xorg.libX11 xorg.libXcursor xorg.libXi xorg.libXrandr # To use the x11 feature
    libxkbcommon wayland # To use the wayland feature
  ];
  LD_LIBRARY_PATH = lib.makeLibraryPath buildInputs;
  RUST_SRC_PATH = "${pkgs.rust.packages.stable.rustPlatform.rustLibSrc}";
}

I was able to run bloom 2D from the examples with cargo run --example bloom_2d

For the sake of cleaning up issues, @Oglo12 I assume based on discussion in discord that you got it working? If not I can give you my shell.nix because I have the same setup.


Fix documentation in JSDoc comments

The PR fixes some @code/@see tags in JSDoc that point to wrong methods.
It also fixes usage of HTML in JSDocs. In the changed files, the html tags should be wrapped in . Otherwise, it is treated as HTML formatting in comments.

looks like there are conflict, do you want to update this?

Yes, I'll resolve it shortly.


Improve SharedProcessPool tests performance

Description

Some minor improvements to make SharedProcessPool related tests to run faster. Those are mostly combining multiple tests into one unit test function to reduce the number of times that the process pool being initialized and reset.
The total execution time of test_shared_process_pool.py and test_multi_processing_stage.py should be improved from ~60s to ~30s (when running locally).
Closes #1951
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/merge


Create Check_prime.sol

Description
Please include a summary of the change and which issue is fixed.
#115
Type of change

[x] Add a new project

Domain of Contribution

[x] Project using only Solidity

Checklist ✓

[x] l have performed a self-review of my own code or work.

[x] My changes generate no new warnings

why u r gernating pr before issue..This is not good practice . please raise an issue for this first


Delete deprecated configuration properties for aad and aadb2c.

Delete deprecated configuration properties for aad and aadb2c.
Update java doc for configuration properties for aad and aadb2c.

Hi, @saragluna , @moarychan , please help to review this PR when you have time.


AttributeError: module 'jsonschema._utils' has no attribute 'load_schema'

jsonschema 4.18.0a1 breaks compatibility, I can't use pytest on my project.
See also https://github.com/python-openapi/openapi-schema-validator/issues/70

Hi @nmoreaud
thanks for the report. We will need to lock deps to supporting version (<4.18).


[New App Request] Microsoft VC++ redistributables

As Microsoft have not until recently published the latest version number on the VC++ release url (https://learn.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist?view=msvc-170), it was previously necessary to download the version available and then check the version of the downloaded file.
This appears to be no longer necessary as they are finally publishing the release version on the page,
This can be found directly below the heading on this link:
https://learn.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist?view=msvc-170#latest-microsoft-visual-c-redistributable-version

The latest version is 14.40.33810.0

So it is now possible (with the Nevergreen script below) to return the release number and resolve the urls for x86, x64 and ARM64 version downloads.
N.B. Evergreen currently serves a very good 3rd Party alternative product "VisualCppRedistAIO" from https://github.com/abbodi1406/vcredist for VC++ redistributables, but as this software is not directly from the source vendor (Microsoft), some users / orgs may prefer not to use this repackaged version.
The script:
# Get-MicrosoftVCPPRedist.ps1

# Define AppName
$AppName = "Microsoft Visual C++ Redistributable"

$ReleaseUrl = "https://learn.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist?view=msvc-170"

Write-Verbose "Obtaining $($AppName) Release Versions from $($ReleaseUrl)...`n"

# Main script to fetch and process links
$AppVersions = @(
    @{AppName = $($AppName); Architecture = 'x64'; Type = 'exe'; Pattern = 'x64\.exe'; VersionPattern = 'The latest version is (\d+\.\d+\.\d+\.\d+)'}
    @{AppName = $($AppName); Architecture = 'x86'; Type = 'exe'; Pattern = 'x86\.exe'; VersionPattern = 'The latest version is (\d+\.\d+\.\d+\.\d+)'}
    @{AppName = $($AppName); Architecture = 'ARM64'; Type = 'exe'; Pattern = 'arm64\.exe'; VersionPattern = 'The latest version is (\d+\.\d+\.\d+\.\d+)'}
)

foreach ($AppVersion in $AppVersions) {
    $SearchCount = 1 # This may need increasing as future versions are released

    #Build each link with File Type specific versions
    $URL = (Get-Link -Uri $ReleaseUrl -MatchProperty href -Pattern $AppVersion.Pattern)
    $URL = (Resolve-Uri -Uri (Get-Link -Uri $ReleaseUrl -MatchProperty href -Pattern $AppVersion.Pattern)).Uri
    $Version = Get-Version -Uri $ReleaseURL -Pattern $AppVersion.VersionPattern

    do {
        if ($URL) {
            New-NevergreenApp -Name $($AppVersion.AppName) -Architecture $($AppVersion.Architecture) -Version $($Version) -Uri $($URL) -Type $($AppVersion.Type)
            break
        }

        $SearchCount--
    } until ($SearchCount -eq 0)

    if ($SearchCount -eq 0) {
        Write-Warning "Could not find release for $($AppName) $($AppVersion.Type)"
    }
}

$ReleaseNotesUrl = "https://learn.microsoft.com/en-us/cpp/windows/latest-supported-vc-redist?view=msvc-170"
Write-Verbose "$($AppName) (Version: $($Version)) release notes are available here: $($ReleaseNotesUrl)"

There is already a separate module VcRedist that caters for these (by the same author as Evergreen, Aaron Parker). I don't want to support any apps in Nevergreen that are already in Evergreen or VcRedist, unless there's a good reason.

Thanks @DanGough
I'd overlooked that module - Now added to my PowerShell module arsenal ;-)

Not required as VcRedist PowerShell Module solves this issue


Locate Now button added in chat activity

#152 Locate now button added in chat activity.

@maniyaom can you share the screenshots?


Support SRTP 2.1.0 and OpenSSL 1.1.0 (Build linker error JRTPLIB 3.1.1)

Hello.
I need help. I try to build JRTPLIB 3.1.1 with SRTP 2.1.0 and OpenSSL 1.1.0  but got linker error while try to make JRTPLIB,
Dependencies:
Debian 9  (also tried on Ubuntu 16.04 LTS)
JRTPLIB v.3_1_11(from git+changes)
JTHREAD v.1_3_3 (from git);
LIBSRTP  v.2_1_0 (by tag) (https://github.com/cisco/libsrtp#installing-and-building-libsrtp)
LIBSSL v.1_1_0 Stable (by branch) (https://github.com/openssl/openssl/tree/OpenSSL_1_1_0-stable)
Source changes:
JRTPLIB v.3_1_11 (from git): changed srtp to srtp2; link to lib source archive: https://drive.google.com/drive/folders/0B_Q0HXvHrCzabVpsSUp0TXNka1U?usp=sharing
Cmake settings (image cmake_JRTPLIB.JPG): https://drive.google.com/drive/folders/0B_Q0HXvHrCzabVpsSUp0TXNka1U?usp=sharing
Build:

OpenSSL:
build from source and also tried on a Virtual Machine to install libssl1.1 and libssl-dev by sudo apt-get install libssl1.1 libssl-dev)
2)libSRTP:
$./configure --enable-debug-logging --enable-log-stdout --enable-openssl

3)JTHREAD:
build with cmake
4)JRTPLIB:
Cmake config - see image cmake_JRTPLIB.JPG
$make
Error log:
Scanning dependencies of target jrtplib-static
[  1%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpapppacket.cpp.o
[  2%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpbyepacket.cpp.o
[  3%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpcompoundpacket.cpp.o
[  4%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpcompoundpacketbuilder.cpp.o
[  4%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcppacket.cpp.o
[  5%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcppacketbuilder.cpp.o
[  6%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcprrpacket.cpp.o
[  7%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpscheduler.cpp.o
[  8%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpsdesinfo.cpp.o
[  8%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpsdespacket.cpp.o
[  9%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtcpsrpacket.cpp.o
[ 10%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpcollisionlist.cpp.o
[ 11%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpdebug.cpp.o
[ 12%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtperrors.cpp.o
[ 12%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpinternalsourcedata.cpp.o
[ 13%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpipv4address.cpp.o
[ 14%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpipv6address.cpp.o
[ 15%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpipv4destination.cpp.o
[ 16%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpipv6destination.cpp.o
[ 16%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtplibraryversion.cpp.o
[ 17%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtppacket.cpp.o
[ 18%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtppacketbuilder.cpp.o
[ 19%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtppollthread.cpp.o
[ 20%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtprandom.cpp.o
[ 20%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtprandomrand48.cpp.o
[ 21%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtprandomrands.cpp.o
[ 22%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtprandomurandom.cpp.o
[ 23%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsession.cpp.o
[ 24%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsessionparams.cpp.o
[ 25%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsessionsources.cpp.o
[ 25%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsourcedata.cpp.o
[ 26%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsources.cpp.o
[ 27%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtptimeutilities.cpp.o
[ 28%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpudpv4transmitter.cpp.o
[ 29%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpudpv6transmitter.cpp.o
[ 29%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpbyteaddress.cpp.o
[ 30%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpexternaltransmitter.cpp.o
[ 31%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpsecuresession.cpp.o
[ 32%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtpabortdescriptors.cpp.o
[ 33%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtptcpaddress.cpp.o
[ 33%] Building CXX object src/CMakeFiles/jrtplib-static.dir/rtptcptransmitter.cpp.o
[ 34%] Building CXX object src/CMakeFiles/jrtplib-static.dir/extratransmitters/rtpfaketransmitter.cpp.o
[ 35%] Linking CXX static library libjrtp.a
[ 35%] Built target jrtplib-static
Scanning dependencies of target jrtplib-shared
[ 35%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpapppacket.cpp.o
[ 36%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpbyepacket.cpp.o
[ 37%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpcompoundpacket.cpp.o
[ 38%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpcompoundpacketbuilder.cpp.o
[ 39%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcppacket.cpp.o
[ 39%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcppacketbuilder.cpp.o
[ 40%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcprrpacket.cpp.o
[ 41%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpscheduler.cpp.o
[ 42%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpsdesinfo.cpp.o
[ 43%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpsdespacket.cpp.o
[ 44%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtcpsrpacket.cpp.o
[ 44%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpcollisionlist.cpp.o
[ 45%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpdebug.cpp.o
[ 46%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtperrors.cpp.o
[ 47%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpinternalsourcedata.cpp.o
[ 48%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpipv4address.cpp.o
[ 48%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpipv6address.cpp.o
[ 49%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpipv4destination.cpp.o
[ 50%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpipv6destination.cpp.o
[ 51%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtplibraryversion.cpp.o
[ 52%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtppacket.cpp.o
[ 52%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtppacketbuilder.cpp.o
[ 53%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtppollthread.cpp.o
[ 54%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtprandom.cpp.o
[ 55%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtprandomrand48.cpp.o
[ 56%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtprandomrands.cpp.o
[ 56%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtprandomurandom.cpp.o
[ 57%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsession.cpp.o
[ 58%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsessionparams.cpp.o
[ 59%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsessionsources.cpp.o
[ 60%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsourcedata.cpp.o
[ 60%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsources.cpp.o
[ 61%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtptimeutilities.cpp.o
[ 62%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpudpv4transmitter.cpp.o
[ 63%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpudpv6transmitter.cpp.o
[ 64%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpbyteaddress.cpp.o
[ 64%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpexternaltransmitter.cpp.o
[ 65%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpsecuresession.cpp.o
[ 66%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtpabortdescriptors.cpp.o
[ 67%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtptcpaddress.cpp.o
[ 68%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/rtptcptransmitter.cpp.o
[ 69%] Building CXX object src/CMakeFiles/jrtplib-shared.dir/extratransmitters/rtpfaketransmitter.cpp.o
[ 69%] Linking CXX shared library libjrtp.so
[ 69%] Built target jrtplib-shared
Scanning dependencies of target example4
[ 69%] Building CXX object examples/CMakeFiles/example4.dir/example4.cpp.o
[ 70%] Linking CXX executable example4
[ 70%] Built target example4
Scanning dependencies of target example2
[ 71%] Building CXX object examples/CMakeFiles/example2.dir/example2.cpp.o
[ 72%] Linking CXX executable example2
[ 72%] Built target example2
Scanning dependencies of target example5
[ 73%] Building CXX object examples/CMakeFiles/example5.dir/example5.cpp.o
[ 74%] Linking CXX executable example5
[ 74%] Built target example5
Scanning dependencies of target example8
[ 75%] Building CXX object examples/CMakeFiles/example8.dir/example8.cpp.o
[ 76%] Linking CXX executable example8
[ 76%] Built target example8
Scanning dependencies of target example1
[ 77%] Building CXX object examples/CMakeFiles/example1.dir/example1.cpp.o
[ 77%] Linking CXX executable example1
[ 77%] Built target example1
Scanning dependencies of target example6
[ 78%] Building CXX object examples/CMakeFiles/example6.dir/example6.cpp.o
[ 78%] Linking CXX executable example6
[ 78%] Built target example6
Scanning dependencies of target example7
[ 79%] Building CXX object examples/CMakeFiles/example7.dir/example7.cpp.o
[ 80%] Linking CXX executable example7
/usr/local/lib/libsrtp2.a(datatypes.o): In function octet_string_set_to_zero': datatypes.c:(.text+0x1d91): undefined reference to OPENSSL_cleanse'
/usr/local/lib/libsrtp2.a(aes_icm_ossl.o): In function srtp_aes_icm_openssl_set_iv': aes_icm_ossl.c:(.text+0x77): undefined reference to EVP_EncryptInit_ex'
/usr/local/lib/libsrtp2.a(aes_icm_ossl.o): In function srtp_aes_icm_openssl_encrypt': aes_icm_ossl.c:(.text+0xe8): undefined reference to EVP_EncryptUpdate'
aes_icm_ossl.c:(.text+0x113): undefined reference to EVP_EncryptFinal_ex' /usr/local/lib/libsrtp2.a(aes_icm_ossl.o): In function srtp_aes_icm_openssl_context_init':
aes_icm_ossl.c:(.text+0x1f9): undefined reference to EVP_aes_128_ctr' aes_icm_ossl.c:(.text+0x20d): undefined reference to EVP_EncryptInit_ex'
aes_icm_ossl.c:(.text+0x231): undefined reference to EVP_aes_256_ctr' aes_icm_ossl.c:(.text+0x241): undefined reference to EVP_aes_192_ctr'
/usr/local/lib/libsrtp2.a(aes_icm_ossl.o): In function srtp_aes_icm_openssl_dealloc': aes_icm_ossl.c:(.text+0x26c): undefined reference to EVP_CIPHER_CTX_free'
/usr/local/lib/libsrtp2.a(aes_icm_ossl.o): In function srtp_aes_icm_openssl_alloc': aes_icm_ossl.c:(.text+0x36a): undefined reference to EVP_CIPHER_CTX_new'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_get_tag': aes_gcm_ossl.c:(.text+0x18): undefined reference to EVP_Cipher'
aes_gcm_ossl.c:(.text+0x2c): undefined reference to EVP_CIPHER_CTX_ctrl' /usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_set_aad':
aes_gcm_ossl.c:(.text+0x71): undefined reference to EVP_CIPHER_CTX_ctrl' aes_gcm_ossl.c:(.text+0x81): undefined reference to EVP_Cipher'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_context_init': aes_gcm_ossl.c:(.text+0xeb): undefined reference to EVP_aes_256_gcm'
aes_gcm_ossl.c:(.text+0x102): undefined reference to EVP_CipherInit_ex' aes_gcm_ossl.c:(.text+0x129): undefined reference to EVP_aes_128_gcm'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_dealloc': aes_gcm_ossl.c:(.text+0x147): undefined reference to EVP_CIPHER_CTX_free'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_encrypt': aes_gcm_ossl.c:(.text+0x19e): undefined reference to EVP_Cipher'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_decrypt': aes_gcm_ossl.c:(.text+0x1e3): undefined reference to EVP_CIPHER_CTX_ctrl'
aes_gcm_ossl.c:(.text+0x1f9): undefined reference to EVP_Cipher' aes_gcm_ossl.c:(.text+0x208): undefined reference to EVP_Cipher'
/usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_set_iv': aes_gcm_ossl.c:(.text+0x293): undefined reference to EVP_CipherInit_ex'
aes_gcm_ossl.c:(.text+0x2c1): undefined reference to EVP_CIPHER_CTX_ctrl' aes_gcm_ossl.c:(.text+0x2db): undefined reference to EVP_CIPHER_CTX_ctrl'
aes_gcm_ossl.c:(.text+0x2f2): undefined reference to EVP_CIPHER_CTX_ctrl' /usr/local/lib/libsrtp2.a(aes_gcm_ossl.o): In function srtp_aes_gcm_openssl_alloc':
aes_gcm_ossl.c:(.text+0x3cb): undefined reference to EVP_CIPHER_CTX_new' /usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_start':
hmac_ossl.c:(.text+0xe): undefined reference to HMAC_Init_ex' /usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_update':
hmac_ossl.c:(.text+0x61): undefined reference to HMAC_Update' /usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_init':
hmac_ossl.c:(.text+0x8e): undefined reference to EVP_sha1' hmac_ossl.c:(.text+0xa2): undefined reference to HMAC_Init_ex'
/usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_dealloc': hmac_ossl.c:(.text+0xc9): undefined reference to HMAC_CTX_free'
/usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_compute': hmac_ossl.c:(.text+0x129): undefined reference to HMAC_Update'
hmac_ossl.c:(.text+0x142): undefined reference to HMAC_Final' /usr/local/lib/libsrtp2.a(hmac_ossl.o): In function srtp_hmac_alloc':
hmac_ossl.c:(.text+0x21c): undefined reference to `HMAC_CTX_new'
collect2: error: ld returned 1 exit status
examples/CMakeFiles/example7.dir/build.make:96: ошибка выполнения рецепта для цели «examples/example7»
make[2]: *** [examples/example7] Ошибка 1
CMakeFiles/Makefile2:415: ошибка выполнения рецепта для цели «examples/CMakeFiles/example7.dir/all»
make[1]: *** [examples/CMakeFiles/example7.dir/all] Ошибка 2
Makefile:127: ошибка выполнения рецепта для цели «all»
make: *** [all] Ошибка 2

Can you try specifying -lcrypto in the ADDITIONAL_LIBRARIES entry of the cmake configuration?


WEB_CONCURRENCY environment variable missing

In baf08b2, WEB_CONCURRENCY was set to 4 in env.example so that users didn't accidentally deploy gunicorn with only 1 worker process.  In splitting up environment variables in 3f8aa26, we seem to have lost WEB_CONCURRENCY.  I'm assuming it should go in the new .envs/.production/.django file.

@reggieriser true! My bad. Fixing this now.

@reggieriser by the way, there's a safety net for that, in the Heroku deployment guide:
heroku config:set WEB_CONCURRENCY=4

Thanks @browniebroke :)


feat(prettier): wrap return statements with parentheses

Current dependencies on/for this PR:

main

PR #1583   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 29, 6:31 AM: @Boshen merged this pull request with Graphite.


Return rbd image ID from Stat function

currently, it returns below info when Stat function is called
https://github.com/ceph/go-ceph/blob/217e502afa108dd892a78fcdda3190aa7d8caa2c/rbd/rbd.go#L49-L57
it's missing the rbd image ID which is part of rbd info CLI command.
ID is required  for few operation ex: adding a task to remove an image from the trash
cc @nixpanic

I had a look at this, and the ID is not part of the librbd.rbd_image_info_t struct, so we will not include it by default in the rbd.ImageInfo type either. There are a few struct that contain the ID of the image, but these are all returned when listing many images (not efficient).
However, the rbd.Image type has a method GetId() that returns (string, error). So, you can efficiently get the ID of the image with id, err := myImage.GetId().


Document reason for choosing long as numeric tag value type.

See https://github.com/census-instrumentation/opencensus-java/pull/276#discussion_r123872681.

Obsoleted by #710.


Storing FDW's CPCV2 code in HarvestStat

I’m considering storing FDW’s cpcv2 code in the HarvestStat procedure. It was previously okay to ignore, but now that we have over 90 crop types, it's becoming difficult to differentiate them by names alone, both within FDW and between FDW/FAO-STAT. Additionally, there are numerous ambiguous cases, as you know.

Copying Weston's comments

We may not want to include the cpcv2 code just yet. Doing so will likely create headaches because right now we occasionally harmonize time series by renaming crops or combine crops after renaming. If there is a unique cpcv2 code that will need to be changed in the same way in all places. I’d suggest we hold off on including the cpcv2 code until after we get a working version of the database out to co-authors

There’s also reason (occasionally) to aggregate crop types before we harmonize the data. In the case where a country changes from reporting just “millet” to different subtypes of millet, for example, we’ll want to aggregate back into a composite class before harmonizing the time series. So for now let’s hold off on the aggregation of subcrops script as well, prioritizing instead getting the current version out.


docs: Fix track function definition in helpers/html_helper.rst

Description
Fix track function definition in helpers/html_helper.rst
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[ ] Conforms to style guide

@obozdag Thank you!


type_function()

That would be nice, no?
Similar to: https://ggplot2.tidyverse.org/reference/geom_function.html

We had this as a desideratum in https://github.com/grantmcdermott/tinyplot/issues/97 (at the bottom of the OP). I marked as completed with your implementation of custom types support... but in retrospect those are different things.
Agree that this would be nice to support. Good call out.

https://github.com/grantmcdermott/tinyplot/pull/250

Should "function" also be added to the types in sanitize_type?

Probably. Don't know if that would create problems, and might not have time to check for a while 😞

Do you have any particular aspect that you think might be affected by this outside of sanitize_type?
I just quickly added "function" to the types and corresponding switch() in https://github.com/grantmcdermott/tinyplot/pull/267. It worked as intended in the examples I tried and the checks still worked ok as before. Should I simply push it into that PR?

Yeah, I think it should fine. Just overlooked it.

OK, added it in https://github.com/grantmcdermott/tinyplot/pull/267 now.


charlock_holmes 0.7.7

This bumps charlock_holmes to a minimum of 0.7.7. This version has some improvements to the binary build, but no actual code changes.
/cc @febuiles

@mistydemeo thank you 🙇
@lildude: To provide a bit more context here: The older charlock_holmes version sometimes fails to build on macOS 10.14.6 with the command-line tools ~11.3.1. This changeset addresses the problem.


Trying to install Kubernetes client-go package fails

Hi, and thanks for the awesome project!
I'm trying to install the Kubernetes client-go so I can use it in a notebook. This is what I'm trying to do in the Dockerfile:
RUN go get k8s.io/client-go/...

RUN go get github.com/tools/godep

RUN cd /go/src/k8s.io/client-go && godep restore
RUN lgo installpkg k8s.io/client-go/...

While most of the packages seem to be installed correctly, some of them fail in the following way:
 /tmp/go-build734525261/libk8s.io-client-go-tools-portforward.so
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: missing section for relocation target k8s.io/apimachinery/pkg/util/sets.String.PopAny
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: reloc 8 to non-elf symbol k8s.io/apimachinery/pkg/util/sets.String.PopAny (outer=k8s.io/apimachinery/pkg/util/sets.String.PopAny) 0
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: undefined: "k8s.io/apimachinery/pkg/util/sets.String.PopAny"
(336/348) failed to install "k8s.io/client-go/tools/portforward": exit status 2
# /tmp/go-build519294519/libk8s.io-apimachinery-pkg-util-remotecommand.so
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: missing section for relocation target k8s.io/apimachinery/pkg/util/sets.String.PopAny
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: reloc 8 to non-elf symbol k8s.io/apimachinery/pkg/util/sets.String.PopAny (outer=k8s.io/apimachinery/pkg/util/sets.String.PopAny) 0
k8s.io/apimachinery/pkg/apis/meta/v1.ParseToLabelSelector: undefined: "k8s.io/apimachinery/pkg/util/sets.String.PopAny"

Although it seems to be an issue with the package itself, I want to make sure that:
 reloc 8 to non-elf symbol
 missing section for relocation target

are not issues with the compatibility of this project.
Thanks!

And I just found your issue https://github.com/golang/go/issues/22998

And I just found your issue golang/go#22998

Yes, that is the root cause of this. lgo relies on -buildmode=shared internally. There is a bug in go that buildmode=shared fails if the target packaeg (A) has an indirect dependency to another package (B) and the shared library of B (e.g. libB.so) is already installed.
Supprisingly, packages with indirect dependencies are rare, but it still sometimes happens.
In this case, I think the easiest workaround is just adding k8s.io/apimachinery/pkg/util/sets to the blacklist lgo maintains because errors are caused by the indirect dependencies to k8s.io/apimachinery/pkg/util/sets by PopAny.

I blacklisted k8s.io/apimachinery/pkg/util/sets and updated the docker image (yunabe/lgo:latest).
Can you docker pull and try it again?

I successfully built the image and tried to run a basic example - this is the output:
# /tmp/go-build478673581/libgithub.com-yunabe-lgo-sess7b2274696d65223a313532343234363035373238363437363030307d-exec4.so
github.com/yunabe/lgo/sess7b2274696d65223a313532343234363035373238363437363030307d/exec4.LgoExport_main: missing section for relocation target k8s.io/client-go/kubernetes/typed/core/v1.(*CoreV1Client).Pods
github.com/yunabe/lgo/sess7b2274696d65223a313532343234363035373238363437363030307d/exec4.LgoExport_main: reloc 8 to non-elf symbol k8s.io/client-go/kubernetes/typed/core/v1.(*CoreV1Client).Pods (outer=k8s.io/client-go/kubernetes/typed/core/v1.(*CoreV1Client).Pods) 0
github.com/yunabe/lgo/sess7b2274696d65223a313532343234363035373238363437363030307d/exec4.LgoExport_main: undefined: "k8s.io/client-go/kubernetes/typed/core/v1.(*CoreV1Client).Pods"
Failed to build a shared library of github.com/yunabe/lgo/sess7b2274696d65223a313532343234363035373238363437363030307d/exec4: exit status 2

Interesting. Can you share the code you executed in your basic example?

I tried to get started with the example in the client-go repo

Thanks. It seems like the workaround I implemented to bypasss https://github.com/golang/go/issues/22998 to fix https://github.com/yunabe/lgo/issues/11 is not working in this case.
Maybe because Pods is an interface method or Pods is defined via embedding.

I'm investigating the condition to reproduce this issue. For the meanwhile, you can bypass the issue by adding import  _ "k8s.io/client-go/kubernetes/typed/core/v1" to your code.

Now it works, thanks a lot!


post_live_metrics: Add _post_in_chunks.

Post metrics and parameters separately
Try to post as many plots as possible and stop before reaching 30MB.
Discard plots higher than 10MB.

Optimistically? expect metrics and parameters to not exceed 30MB.

Any concerns about this spamming studio too much? Thinking of https://iterativeai.slack.com/archives/C01116VLT7D/p1692452930044229.

Any concerns about this spamming studio too much? Thinking of https://iterativeai.slack.com/archives/C01116VLT7D/p1692452930044229.

As I understood it, the problem in the link was not about too many requests from the client to the server.
The problem was (still is) in the backend querying the database in a very inefficient way, so a single request in a project with a lot of steps per linear plot caused / can trigger this (https://github.com/iterative/studio/issues/7256).
Anyhow, the change here mainly affects images, which are handled differently from linear plots (the source of the linked problem).
The main difference is when there the size of the images exceeds the server limit per single request:
Without this P.R, we just fail to send data on each step and try to send it in the next one. Failing forever and the request keeps getting bigger.
With this P.R:, we are able to send the data at each step bypassing the server limit.

I think it doesn't have to wait. The only thing that comes to my mind to check (besides the basic review, @amritghimire can you do that please?)- align the limits with Studio big_files check. See here for example - https://github.com/iterative/studio/pull/7299/files#diff-a67d283c73e8021405df0b1de3fb8dcf570563c21c70de5324cc2d8b661e561fR36
We skip 10Mb files on Studio (so they will disappear as we then parse an experiment).
Also, I now put a limit of 200 files per directory (not ideal, see the discussion and chime in the PR)- but as a safe guard. Here (or on the Studio side? or both?) we probably also want to limit the number of chunks, or number of files, etc.

I think it doesn't have to wait. The only thing that comes to my mind to check (besides the basic review, @amritghimire can you do that please?)- align the limits with Studio big_files check. See here for example - https://github.com/iterative/studio/pull/7299/files#diff-a67d283c73e8021405df0b1de3fb8dcf570563c21c70de5324cc2d8b661e561fR36
We skip 10Mb files on Studio (so they will disappear as we then parse an experiment).

Also, I now put a limit of 200 files per directory (not ideal, see the discussion and chime in the PR)- but as a safe guard. Here (or on the Studio side? or both?) we probably also want to limit the number of chunks, or number of files, etc.

Added both checks


Use ava tap output for generator tests

This is kind of hilarious, actually. The default ava reporter was filling up
the lerna run input buffer by erasing and re-drawing its spinner while it
waited for the test to complete.
This patch just switches to the tap reporter, which doesn't do that.

I had to patch lerna to track this down.

👍  cool stuff.  Thanks @gigabo


Any way to make Fresh Moves compatible with Customizable Player Models?

I am using the Customizable Player Models mod, and I was wondering if there was a way to make it and Fresh Moves compatible, specifically the movement aspect. In theory, the custom model I am using still has the same parts as a vanilla model: head, body, right and left arms, right and left legs; the only thing it doesn't have is a secondary layer, like the second head layer, second body layer, etc. I don't know to what extent that matters though (I'm using the eyeless version of the resource pack), so I simply ask: could I somehow be able to make Fresh Moves and CPM compatible, even if it involves editing the files of the resource pack? Simply curious.

I've been screwing around with this to figure out what's wrong with it.
From what I've found the animations only work with vanilla parts.
Anything under else isn't effected.

But there is always the option to make your own animations.

Do wish that the black eyes thing wasn't a problem with CPM.

But I understand why it can't be fixed.

CPM and FM store data in the same place on the skin.

well thing is if you make the canvas bigger (I use 128x128) then it works. If there's a way to make this support bigger canvas sizes then that would be great!


Use iss from token only for payload verification.

CrateDB must not fire any requests against token's iss as it can be some malicious endpoint.
Instead, we should use it only for the payload verification (pre signature check)
Not a bug because token claims verification fails without this change (we have a failure before we fire a request) but can be an issue with caching in place as we may cache by wrong(malicious) endpoint.
Relates to https://github.com/crate/crate/pull/16937#discussion_r1836245300

I will update an assertion in another test - it used to pass because malformed URL was in the token. Will move it to the user properties.

LR failure
retest thi please

LR failure
retest this please


installation is too distro specific and needs sudo

I tried to install helm-secrets and not sure if this is helm-secrets issue or helm issue... I am running Arch Linux and helm 2.8.0.
First, I had to set up --version because of this.
then I didn't have lsb-release:
$ helm plugin install https://github.com/futuresimple/helm-secrets --version v1.2.8 --debug
[debug] updating https://github.com/futuresimple/helm-secrets
[debug] setting version to "v1.2.8"
[debug] symlinking /home/dan/.helm/cache/plugins/https-github.com-futuresimple-helm-secrets to /home/dan/.helm/plugins/helm-secrets
[debug] loading plugin from /home/dan/.helm/plugins/helm-secrets
[debug] running install hook: &{/usr/bin/sh [sh -c $HELM_PLUGIN_DIR/install-binary.sh] []  <nil> <nil> <nil> [] %!s(*syscall.SysProcAttr=<nil>) %!s(*os.Process=<nil>) <nil> <nil> <nil> %!s(bool=false) [] [] [] [] %!s(chan error=<nil>) %!s(chan struct {}=<nil>)}
/home/dan/.helm/plugins/helm-secrets/install-binary.sh: line 38: lsb_release: command not found
Error: plugin install hook for "secrets" exited with error

installing lsb-release helped. It should be clearly definied in README that it is a dependency.
Then, I ended up with this issue:
$ helm plugin install https://github.com/futuresimple/helm-secrets --version v1.2.8 --debug
[debug] updating https://github.com/futuresimple/helm-secrets
[debug] setting version to "v1.2.8"
[debug] symlinking /home/dan/.helm/cache/plugins/https-github.com-futuresimple-helm-secrets to /home/dan/.helm/plugins/helm-secrets
[debug] loading plugin from /home/dan/.helm/plugins/helm-secrets
[debug] running install hook: &{/usr/bin/sh [sh -c $HELM_PLUGIN_DIR/install-binary.sh] []  <nil> <nil> <nil> [] %!s(*syscall.SysProcAttr=<nil>) %!s(*os.Process=<nil>) <nil> <nil> <nil> %!s(bool=false) [] [] [] [] %!s(chan error=<nil>) %!s(chan struct {}=<nil>)}
which: no dpkg in (/home/dan/.local/opt/miniconda/bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl)
Sorry only installation via dpkg (aka Debian distros) is currently supported

* Helm-secrets wrapper for helm binary: ln: failed to create symbolic link '/usr/bin/helm-wrapper': Permission denied
Error: plugin install hook for "secrets" exited with error

Furthermore, it shows that it got installed:
NAME   	VERSION	DESCRIPTION
secrets	1.2.8  	This plugin provides secrets values encryption for Helm charts secure storing

but I doubt it will be working, since sops isn't installed. Shouldn't this be rather pre-install hook? And then permission error is at least because of this, which usually resolves to /usr/bin/, which is terrible to do for custom scripts.
Why not just let everyone to install sops using his package manager?

I am still encountering this error on Manjaro:

I'm also on Manjaro and this worked for me:

install sops from aur
install plugin

Yeah, I also still had the same issue. Installing sops before did fix it.


Misc: fix test filename for disallowSpaceBeforeObjectValues

Was doing another issue and found this? disallow-space-before-object-values

Coverage remained the same at 99.02% when pulling 7d5a6fb7d6a0976475d0f8adde0c7f31e994fc70 on hzoo:rename into 527771d4bb5f10783e199a84c3beebeba7ad01d0 on jscs-dev:master.

Odd. Lgtm

I guess it doesn't affect anything but yeah.


platformio: 3.5.3 -> 3.6.1

Motivation for this change
I'll only get the chance to try this out properly on the weekend so not tested yet. It builds and runs but nothing more than pio --version and pio update has been attempted.
Cc: @makefu @mogorman
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Unexpected error: command failed with exit code 1 on aarch64-linux (full log)
Attempted: platformio
Partial log (click to expand)
applying patch /nix/store/s8vy0rphkkg0qxrhh02las9w60s6fg6l-fix-searchpath.patch
patching file platformio/util.py
Hunk #1 succeeded at 431 with fuzz 1 (offset 36 lines).
updateAutotoolsGnuConfigScriptsPhase
configuring
building
copying path '/nix/store/am52jmd5gpnc0ykn7q1ywwrrpcwhi1v6-binutils-wrapper-2.30' from 'https://cache.nixos.org'...
copying path '/nix/store/3f7g71flhmd0jlqbcjjmbn5nlp98v65w-gcc-wrapper-7.3.0' from 'https://cache.nixos.org'...
copying path '/nix/store/dlx700d5hssg7x1a1zjhisr92km7pq1r-stdenv-linux' from 'https://cache.nixos.org'...
error: a 'i686-linux' is required to build '/nix/store/qri1rklf4i7a9iylqn3gzwafwpjhj1a9-python2.7-platformio-3.6.1.drv', but I am a 'aarch64-linux'

Success on x86_64-linux (full log)
Attempted: platformio
Partial log (click to expand)
----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
building '/nix/store/g3jfccdipf29nds1pxczqc2wrlfyvlbq-platformio-usr-multi.drv'...
created 936 symlinks in user environment
building '/nix/store/503pgg9lqxzs6g86iycq2cvwkfvv6s8w-platformio-fhs.drv'...
building '/nix/store/8r6kqgddj974h3mbjxa09hzqxmw9n7dw-platformio-init.drv'...
building '/nix/store/xyj28r7rpgl5l3dqnnsvi8jwmpgvfmgs-platformio.drv'...
/nix/store/7vvsy7445kh76r1g375qqq0s0qb8h2k5-platformio

The aarch64 error looks weird, and is a regression.

for better tracking we could also start the test environment as described in https://github.com/platformio/platformio-core/blob/develop/tox.ini#L62-L64 , or we explicitly disable the tests.

@xeji, are you sure this ever worked on aarch64?

for better tracking we could also start the test environment

@makefu, many of the tests require an internet connection. I have changed the PR to disable the tests that cannot pass in our sandbox.

Oh, and the pypi package doesn't contain the tests so we need to fetch from GH instead.

Unexpected error: command failed with exit code 1 on aarch64-linux (full log)
Attempted: platformio
Partial log (click to expand)
applying patch /nix/store/s8vy0rphkkg0qxrhh02las9w60s6fg6l-fix-searchpath.patch
patching file platformio/util.py
Hunk #1 succeeded at 431 with fuzz 1 (offset 36 lines).
updateAutotoolsGnuConfigScriptsPhase
configuring
building
copying path '/nix/store/am52jmd5gpnc0ykn7q1ywwrrpcwhi1v6-binutils-wrapper-2.30' from 'https://cache.nixos.org'...
copying path '/nix/store/3f7g71flhmd0jlqbcjjmbn5nlp98v65w-gcc-wrapper-7.3.0' from 'https://cache.nixos.org'...
copying path '/nix/store/dlx700d5hssg7x1a1zjhisr92km7pq1r-stdenv-linux' from 'https://cache.nixos.org'...
error: a 'i686-linux' is required to build '/nix/store/mwcajwiw41hh2d7sbrs232zcmna4is4w-python2.7-tox-3.2.1.drv', but I am a 'aarch64-linux'

Success on x86_64-linux (full log)
Attempted: platformio
Partial log (click to expand)
tests/commands/test_settings.py::test_settings_check PASSED              [100%]

============= 10 passed, 1 skipped, 27 deselected in 0.87 seconds ==============
pytestcachePhase
building '/nix/store/m1ppcrcm7iciih03ar8ymha36xqjrzqw-platformio-usr-multi.drv'...
created 936 symlinks in user environment
building '/nix/store/rd9yx6g26qyzp6b6m8h9cizs8fzdmgss-platformio-fhs.drv'...
building '/nix/store/pbq1rxns6jmjsm14f8kjx1j5mvybf3z6-platformio-init.drv'...
building '/nix/store/m90y2z0861rr6i9h0sngn1fdbxi61hjj-platformio.drv'...
/nix/store/9npan03nlab77nvbxjrr2qk2nmrxp0vc-platformio

@peterhoeg wow, fantastic. Thanks for the effort! I think we should just disable aarch64 builds.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: platformio
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: platformio
Partial log (click to expand)
collision between `/nix/store/3v5r7fkrbkw2qajadvjbf6p6qriz9p1i-gcc-7.3.0-lib/lib/libgcc_s.so.1' and `/nix/store/7sdj151dlg1khkkqrycvs5jfp9i59ydp-glibc-multi-2.27/lib/libgcc_s.so.1'
collision between `/nix/store/vslcivmlk95wbib95zpizycg712rcfxl-shadow-4.6/etc/login.defs' and `/nix/store/3cbbzmfqlgry4q6pb0hbahkqhv1jc12r-platformio-chrootenv-etc/etc/login.defs'
collision between `/nix/store/vslcivmlk95wbib95zpizycg712rcfxl-shadow-4.6/etc/pam.d' and `/nix/store/3cbbzmfqlgry4q6pb0hbahkqhv1jc12r-platformio-chrootenv-etc/etc/pam.d'
created 1763 symlinks in user environment
building '/nix/store/m1ppcrcm7iciih03ar8ymha36xqjrzqw-platformio-usr-multi.drv'...
created 936 symlinks in user environment
building '/nix/store/rd9yx6g26qyzp6b6m8h9cizs8fzdmgss-platformio-fhs.drv'...
building '/nix/store/pbq1rxns6jmjsm14f8kjx1j5mvybf3z6-platformio-init.drv'...
building '/nix/store/m90y2z0861rr6i9h0sngn1fdbxi61hjj-platformio.drv'...
/nix/store/9npan03nlab77nvbxjrr2qk2nmrxp0vc-platformio

xeji, are you sure this ever worked on aarch64?

Not sure it  works, but the current version builds on aarch64.  I think disabling aarch64 is fine for now.


chore: set package.json indent size to 2

NPM enforces its own indent rules and when package.json is updated by NPM during various processes, will always revert the indent size to 2. Adding an override to our standard indent size for package.json makes our IDE lives easier. See https://github.com/npm/npm/pull/3062 for context.

@onebytegone any thoughts on this?

Sorry, I didn't read the entire long-ranted thread on the NPM issue, but I thought this wasn't necessary? NPM seems to keep whatever indentation you have in the file.  I don't know why I didn't think to verify it myself 🤦
This being the case @onebytegone - I guess we should change any package.json files that don't adhere to our preferred indentation?

@pbredenberg Yes, let's switch our package.json files over to 3 space indentation. Thanks!


HelloServiceImplBase : file does not exists in gRPC article

Irrespective of which method you use for code generation, following key files will be generated:

HelloRequest.java – contains the HelloRequest type definition
HelloResponse.java – this contains the HelleResponse type definition
HelloServiceImplBase.java – this contains the abstract class HelloServiceImplBase which provides an implementation of all the operations we defined in the service interface

The article mentions this but HelloServiceImplBase is not generated. It is also not present in the project.
link: https://www.baeldung.com/grpc-introduction

Hi @swayamraina,
How do you try to generate the classes - with Protocol Buffer Compiler or using the Maven plugin?

@kwoyke I am using the protobuf compiler
The article mentions,

protoc -I=$SRC_DIR --java_out=$DST_DIR $SRC_DIR/HelloService.proto

While in this SO post I found out we also need to add --plugin=protoc-gen-grpc-java=path/to/protoc-gen-grpc-java

Yes @swayamraina, if you use protobuf compiler you need to use protoc-gen-grpc-java plugin as well. The maven configuration works fine though.
Anyway, soon we'll update the article!

Adding the below snippet to under build also works.
Make sure you have the correct path mentioned.
<protoSourceRoot> 						${basedir}/src/main/resources/proto/ </protoSourceRoot>


Feature: make SEO Friendly

Create a nice SEO will be awesome to make easier the users find the website!

Resolved in #7


thermostatMode issue

i try to using ac_unit with trait like this
app.onSync((body) => {
return {
requestId: body.requestId,
payload: {
agentUserId: USER_ID,
devices: [{
id: '123',
type: 'action.devices.types.AC_UNIT',
traits: [
'action.devices.traits.FanSpeed',
'action.devices.traits.OnOff',
'action.devices.traits.TemperatureSetting',
],
name: {
defaultNames: ['Air Condition'],
name: 'Air Condition',
nicknames: ['Air Condition'],
},
deviceInfo: {
manufacturer: 'smart-home-inc',
model: 'hs1234',
hwVersion: '3.2',
swVersion: '11.4',
},
willReportState: true,
attributes: {
availableThermostatModes: [
'off',
'heat',
'on',
],
thermostatTemperatureRange: {
minThresholdCelsius: 15,
maxThresholdCelsius: 30,
},
pausable: true,
availableFanSpeeds: {
speeds: [{
speed_name: 'speed_low',
speed_values: [{
speed_synonym: ['low'],
lang: 'en',
}],
},
{
speed_name: 'speed_medium',
speed_values: [
{
speed_synonym: ['medium'],
lang: 'en',
},
],
},
{
speed_name: 'speed_high',
speed_values: [
{
speed_synonym: ['high'],
lang: 'en',
},
],
},
],
ordered: true,
},
commandOnlyFanSpeed: true,
supportsFanSpeedPercent: true,
queryOnlyTemperatureSetting: true,
thermostatTemperatureUnit: 'C',
},
}],
},
};
});
const queryFirebase = async (deviceId) => {
const snapshot = await firebaseRef.child(deviceId).once('value');
const snapshotVal = snapshot.val();
return {
on: snapshotVal.OnOff.on,
currentFanSpeedSetting: snapshotVal.FanSpeed.currentFanSpeedSetting,
currentFanSpeedPercent: snapshotVal.FanSpeed.currentFanSpeedPercent,
thermostatMode: snapshotVal.TemperatureSetting.thermostatMode,
};
};
when i running it, its return this error message at firebase
TypeError: Cannot read property 'thermostatMode' of undefined
please help me

The error message "Cannot read property 'thermostatMode' of undefined" indicates that the thermostatMode property is being accessed on an undefined object. This suggests that snapshotVal.TemperatureSetting is undefined.
To resolve this , you should first ensure that the TemperatureSetting property exists within your Firebase data structure for the specified deviceId. If it does not exist, you need to handle this case gracefully in your code.
You can modify the queryFirebase function to check if snapshotVal.TemperatureSetting exists before accessing its properties.


[CELEBORN-1814][CIP-14] Add transportMessage to cppClient

What changes were proposed in this pull request?
Add transportMessage to cppClient.
Why are the changes needed?
TransportMessage is the building block of controlMessages.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Compilation and UTs.

@FMX @RexXiong Could you kindly help review this PR? Thanks a lot.
Any suggestion is welcome.


WIP: Support special elements

https://github.com/kitsuyui/sxd_html/issues/13

Codecov Report
Base: 34.91% // Head: 34.91% // No change to project coverage :thumbsup:

Coverage data is based on head (bcbcde3) compared to base (ef6b010).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   34.91%   34.91%
=======================================
  Files           4        4
  Lines         295      295
=======================================
  Hits          103      103
  Misses        192      192

Flag
Coverage Δ

unittests
34.91% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Remove local caches of data

There are quite a few places where large local caches of data are held, and updated with changes from the server and the local system. This is very dangerous as it is very easy for the client to get out of sync with the server, and I see no logic in the code to resolve data conflicts.
The UI needs to fetch data from the server when the user completes actions and to treat it's local cache as disposable.

Local cache should be handled by the browser in 95% of the cases.

Aslak@ml:
Raw data cache should primarily happen per URL on HTTP level via Cache-Control Headers sent from API. This is standard functionality that the HTTP Client Lib (Browser) provide. UI's main concern should be to follow the links and react to them based on what the server is sending. Caching and refresh of data is a concern between the Browser and Backend.
UI currently doesn't refresh the data from WorkItemList to WorkItemDetails, but the conflict is handled in the backend. The version field stored in DB has to match the PATCH request version field else it's a VersionConflict error. UI generally ignores most errors.
The way it currently works should be replaced by ETag/If-Match but that's not pushed yet. There is also a PR to add Last-Modified and If-Modified-Since support for refreshing data. Both of those options are just a more appropriate protocol level support for what Version already does.
All Core objects store created_at, updated_at but not all REST API exposing them atm. Iæve created this issue for tracking the timestamps; https://github.com/almighty/almighty-core/issues/939
[..]
What should happen even in the current version, unless UI does some auto version update increment is that Max should fail to update 123. If you see Max's change in the UI and there is no complaint(also check Console log for API error), also check if the changes Max did is still there after a refresh. If it's not, then UI is updating the cache and displaying it to the user regardless of it managing to store it to the backend or not.


Develop/ram console option known addr buf

This PR added an option for RAM console driver to locate the console buffer to a predefined memory region, so that the RAM console can be used easily, and also added an example on imx8mmevk.

This update fixed some check failures.

Updated to fix LONG_LINE check warning.

This will conflict with the HWMv2 so we will block this till the collab branch merges

Okay, will rebase then.

Needs converting to hwmv2

Yes, will rebase after the platform hwmv2 migration PR merged.

@Zhiqiang-Hou HWMv2 has merged into main. Please rebase and adjust your PR.

@Zhiqiang-Hou HWMv2 has merged into main. Please rebase and adjust your PR.

Thanks for the reminding! Will rebase later.

rebased to hwmv2 and added imx8mpevk, imx8mnevk and imx93evk A-core RAM console variant.
based on https://github.com/zephyrproject-rtos/zephyr/pull/67886

It seems some commits in another PR https://github.com/zephyrproject-rtos/zephyr/pull/67886 is pushed to this PR.

updates:

Changed to add a ram-console Snippet, so that other platforms/variants can easily enable this feature.
Added a Config entry to control if the RAM console buffer uses a dedicated memory-region/section.
Moved the RAM console buffer memory mapping to ram-console driver.
Changed the CONFIG_RAM_CONSOLE_BUFFER_SIZE as the total buffer size and leaved one byte internal to ensure the NULL-termination.

@nordicjm can you revisit this PR

@nordicjm can you revisit this PR

Sure and thanks for your review and approval!

@kartben can you review the changes to your change request
@carlocaione you had a previous approval but that was dropped to address a different request.

@carlocaione could you please refresh your +1? thanks!


"Parse error on line 1: Expecting 'STRING'..." - How do I find out what extension is causing this issue?

Issue Type: Bug
I have a very reliable issue when it comes to opening new, untitled files that start out as plaintext (a very common usage pattern for me). I start typing, and everything is fine, but then when I delete that line of text, I get the following error which does not go away until I reload the window:
Parse error on line 1:

^
Expecting 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '[', got '1'

Here's a gif demonstrating the issue.
My question is not how do I fix this, because I've already searched other issues that have been opened here (e.g. #67046, #56193) and see that it's probably an extension and I have to go hunt it down. The problem is that I have a lot of (probably too many) extensions, and I can imagine that this will get tedious trying to hunt it down.
VSCode doesn't really provide anything useful about what extension generated the error that I'm seeing. When I right click and copy the message, I'm given this information, which I don't find to be particularly helpful:
{
	"resource": "Untitled-1",
	"owner": "languageErrors31",
	"severity": 8,
	"message": "Parse error on line 1:\n\n^\nExpecting 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '[', got '1'",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 1
}

Is owner supposed to have meaningful significance to help me debug this? I've searched around and this doesn't really give me much to go by.
Can I get some pointers on how to tackle this properly so that I don't waste an hour trying to figure it out (teach a man to fish and all that)? If the only solution really is "disable extensions one by one", can we use this issue as a catalyst to make this better/easier to debug? I couldn't find any issues that are trying to tackle this problem specifically, but please let me know if this is a duplicate and I'll go chime in there.
 I do not think we have extension information available in our diagnostics model and I see this is useful in the case.
@jrieken Is it possible to add extension id as source if missing?

@jrieken Is it possible to add extension id as source if missing?

Sure but I don't think that makes sense because we show them quite prominent in the UX and extension ids and often also names aren't fit for that.

Ok. How about having extension info (id) available in the marker so that we can provide this info in the copy action?

That. Or how about trace logging what extension is adding what marker to what files?

Either works for me.

Thanks, all! Whatever makes these kinds of issues easier to debug and makes me less likely to open a new issue is great in my book 🙂

@sandy081 I'll push a change for some trace logging

There is no (very) verbose logging in the 'Extension Host Log' whenever an extension updates diagnostics

to verify:

enable verbose logging
check the extension host log for message starting with [DiagnosticCollection]

@jrieken I can't figure out how to enable verbose logging and can't find it in the documentation, do you have any pointers?

Ah okay I figured out, set it to "trace" and I see the following output:
[2020-05-29 13:08:28.260] [exthost] [warning] DiagnosticCollection with name 'languageErrors' does already exist.
[2020-05-29 13:08:28.267] [exthost] [trace] [DiagnosticCollection] change many (extension, owner, uris) Togusa09.tmlanguage languageErrors68 [[{"$mid":1,"fsPath":"extHostLog","external":"output:extHostLog","path":"extHostLog","scheme":"output"},[]]]

Guess I gotta go bring this up in the TextMate Languages extension, since disabling that fixes the issue. Thanks again!


Remove publisher from details page

I created an "accessible" <Menu /> component (which can be re-used around the app).
On the publisher page it will represent a hamburger button than can open a menu to "edit" and "delete" the publisher.

this is super cool


Added failed scenario statistics to progress statistics printer

When using the --progress option with behat there was a situation where an issue failed in an @AfterStep context. The information printed to the output was not sufficient to determine in which feature file or scenario the issue occured. With the simple addition proposed a listing of the failed scenarios will be printed at the end of the run, thus greatly simplifying the task of finding the offending scenario, especially in a large suite such as ours with hundreds of feature files.
Example:
`
[2022-06-20T14:18:31.605Z] ...................................................................... 5180
[2022-06-20T14:18:36.955Z] ...................................................................... 5250
[2022-06-20T14:18:40.835Z] ..................................................----
[2022-06-20T14:18:40.835Z]
[2022-06-20T14:18:40.835Z] --- Failed hooks:
[2022-06-20T14:18:40.835Z]
[2022-06-20T14:18:40.835Z]     AfterStep # ****Context::verifyHtml()
[2022-06-20T14:18:40.835Z]       Behat\Mink\Exception\ExpectationException: HTML errors were found in page: http://web:8080/************ Errors:
[2022-06-20T14:18:40.835Z]       Line: 183 Error: Element “div” not allowed as child of element “a” in this context.
(Suppressing further errors from this subtree.)
[2022-06-20T14:18:40.835Z]       Line: 183 Error: Heading cannot be a child of another heading.
[2022-06-20T14:18:40.835Z]       Line: 183 Error: Stray end tag “h2”. in sites/default/modules/custom/eyp/eyp.behat.inc:900
[2022-06-20T14:18:40.835Z]       Stack trace:
[2022-06-20T14:18:40.835Z]
[2022-06-20T14:18:40.835Z] 689 scenarios (688 passed, 1 failed)
[2022-06-20T14:18:40.835Z] 5304 steps (5300 passed, 4 skipped)
[2022-06-20T14:18:40.835Z] 10m33.02s (195.65Mb)
script returned exit code 1
`

Is it OK to modify an existing test or do I need to create a new one?
Kind regards,
Anthony
On Thu, Jun 30, 2022 at 4:18 PM Ciaran McNulty @.***>
wrote:

Can you add a test for this please?
—
Reply to this email directly, view it on GitHub
https://github.com/Behat/Behat/pull/1392#issuecomment-1171279677, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRDVFK3Z4YTI5GDIJT55DVRWUEBANCNFSM5ZWZJ4BA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Anthony J. Patrinos (MSc, PhD)
Mechanical Engineer & Informatics Consultant

If there's one you can modify that is clear, that's fine.
I'm working on fixing the test suite so you may need to rebase

Ok, thank you.
Anthony
On Thu, Jun 30, 2022 at 4:57 PM Ciaran McNulty @.***>
wrote:

If there's one you can modify that is clear, that's fine.
I'm working on fixing the test suite so you may need to rebase
—
Reply to this email directly, view it on GitHub
https://github.com/Behat/Behat/pull/1392#issuecomment-1171326454, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRDVFFAD2YGAVMI44GLF3VRWYUXANCNFSM5ZWZJ4BA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Anthony J. Patrinos (MSc, PhD)
Mechanical Engineer & Informatics Consultant

Hello Ciaran,
I'm having trouble setting up a local system to work on the test. If my
proposed change is deemed worthy, maybe  someone else can take over the
testing part. Otherwise, I will regretfully have to withdraw my pull
request.
Kind regards,
Anthony
On Thu, Jun 30, 2022 at 4:58 PM Anthony Patrinos @.***>
wrote:

Ok, thank you.
Anthony
On Thu, Jun 30, 2022 at 4:57 PM Ciaran McNulty @.***>
wrote:

If there's one you can modify that is clear, that's fine.
I'm working on fixing the test suite so you may need to rebase
—
Reply to this email directly, view it on GitHub
https://github.com/Behat/Behat/pull/1392#issuecomment-1171326454, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRDVFFAD2YGAVMI44GLF3VRWYUXANCNFSM5ZWZJ4BA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Anthony J. Patrinos (MSc, PhD)
Mechanical Engineer & Informatics Consultant

--
Anthony J. Patrinos (MSc, PhD)
Mechanical Engineer & Informatics Consultant

@patrinos can you check the feature I've added captures what you intended please?

Hm, when it's a failing step the extra information seems redundant
Maybe a better solution would be to make sure the scenario is included in the failing step output?

Personally I wouldn't mind a couple of extra lines of output when already thousands have been avoided. I'm not sure how easy it would be to implement a more granular approach. Until this is done, I think this a good enough addition that guides the developer to exactly what needs to be fixed.

@patrinos Thanks for your proposal and sorry for the long wait for an update. I have tested your proposed solution and it does not seem to work as intended. If for example you have this feature file:
Feature:
  Scenario: Failing scenario
    Given a scenario that fails in an after scenario hook

And this context file:
<?php

use Behat\Behat\Context\Context;
use Behat\Behat\Hook\Scope\AfterScenarioScope;

class FeatureContext implements Context
{
    /**
     * @Given a scenario that fails in an after scenario hook
     */
    public function failsInAfterScenario() {
        return;
    }

    /**
     * @AfterScenario
     */
    public function failingAfterScenario(AfterScenarioScope $scope)
    {
        throw new \Exception('oops');
    }
}

Then the output still does not include the failing scenario. This is the output:
.
--- Failed hooks:
    AfterScenario # FeatureContext::failingAfterScenario()
      oops (Exception)

1 scenario (1 passed)
1 step (1 passed)

You can see that the problem is that a failure in the after scenario hook does not make the scenario fail. The scenario actually passed successfully and it just the hook that failed. So your solution does not solve the problem. I think that the fix would be to update the code that prints the hook results so that it references the step, scenario or feature for which the hook was called but this would be more complex to implement.
So:
a) Can you confirm if my interpretation is correct? I am not sure if your results differ
b) Do you want to work on this alternate solution?
c) If not, can you create an issue that captures the problem so that someone may try to implement a solution?
Thanks!!

@carlos-granados I haven't tested this but AFAIK you're right - it was actually discussed  with someone who posted in the long-closed #842 this week. I think you're right that the solution would be to print the related scenario/step information when printing the failed hook.

Closing in favor of https://github.com/Behat/Behat/issues/1495


Feat/graceful handle unknown variables and other errors during parsing

@k0pernicus this is a take to solve #205
It's a bit hacky since liquid has no great way to figure out what's the exact problem or to ignore undefined variables. But eventually this should do the trick. If you see more test cases that we should cover feel free to point them out.

coming back to this PR after a while I would say that the safest way to handle unknown placeholders is to keep them as they are. Presenting a warning message, but then keeping them as they appeared would be the less harmful way.
So I will change the replacement of unknown placeholders the way that they will be replaces by itself as they are, since that is the only way to convince liquid to not break.


Add support for positional arguments to lncli

Currently, lncli (cmd/lncli) only support fully specifying each argument for each command. This parsing should be extended to also allow positional arguments, and also combo of positional+keyword arguments.

I would take this one.
I also have some ideas for bash completion for pubkey or channel ids.
Whats your take on that?

@tompre excellent!
In order to do bash completion for the pubkey's channelID's, you'll need to before to a call to describegraph in order to obtain the available list of pubkey's/chanID's. If we had a sort of "console" mode, you could put both sets in their own trie then use that for auto-completion.

I wanted to use the bash completion functionality provided by urfave/cli. link
Basic completion can be enabled by a single flag (app.EnableBashCompletion = true) and completion candidates are generated in go using the BashComplete flag in the command definition.
Unfortunately @aantonop did not see this before writing pull request #109 and I did not see his pull request before it was merged.

Closed by #120.


change: default BT beacon now OmnectWifiConfig instead of DmWifiConfig

Additionally, the used name can now be set via environment variable SCAN_SERVICE_BEACON.

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


[Bug] Weird behavior if I put multiple characters in trigger of selection insert

Watch the video:
Screencast from 2022-08-17 06:36:03 PM.webm
Notes:

This behavior occurs to every character I enter.
When I go back into plugin settings, the incorrect rule is replaced, but I still get errors and am unable to type.
Issue is solved when I Reload App Without Saving

so weird, Obsidian v0.15.9? I'm not able to reproduce it on MacOS

When I go back into plugin settings, incorrect rule is replaced, but I still get errors and am unable to type.

'replaced' means the rule editor is empty?
'errors' means rule editor errors is still displayed in the settings tab?
'unable to type' can you delete or click other buttons and get repsonse?

Yes, obsidian 0.15.9

Yes, or it is reverted back to the older version.
Error means error in console, the one that is shown in the video
Clarification: Any select insert becomes weird, but I can still type.

Screencast from 2022-08-18 08:23:34 AM.webm

it looks like my video recording is broken :grimacing:
I'm really sorry! :sweat:
Let me record another one with the actual errors.

This time I have truly captured the error.
I'm really sorry for the confusion😬
I hope this can help

👏 Yes, this error is what I have seen. pr #48 already fixed it, I merge it now and you can try again


Networking issues with TP5 on Azure

The following issues did not occur with my TP5 on-premise installation, but only occur on Azure.
1. Can only reach http listener inside Docker container on port 8082 when adding firewall rule to the host
I start a http listener inside the docker container on port 8082. I can reach it via "wget http://<containerHostname>:8082" as well as "wget http://<containerIP>:8082" from inside the container. But the same does not work from the container host:
I can only reach "wget http://<containerIP>:8082" when I add a firewall inbound rule for port 8082 to the host, which in my opinion does not make much sense, since we are talking about the communication between host->container, not the outside communication public->host. It's not clear to me why firewall rules applied to the host have an effect to the container (=the container is re-using the firewall rules of the host).
Sorry, I'm not a networking expert, maybe there is a technical explanation for that - why it makes sense.
2. Host cannot resolve container hostname (WIN-DE6U4068NAF)
Even with the firewall rule, I cannot reach the http listener of the container via hostname: "wget http://<containerHostname>:8082". The host cannot resolve the hostname of the container properly.
(I get the container hostname by typing hostname inside the container - it's: WIN-DE6U4068NAF. The container hostname differs from the hostname that I get in the docker inspect response.)
docker inspect output: http://pastebin.com/LYg4KvBE
Pinging works fine on-premise, it does not work on Azure:
On-Premise works fine:
PS C:\Windows\system32> ping WIN-DE6U4068NAF

Ping wird ausgeführt für WIN-DE6U4068NAF [fe80::8c5:22c6:b1d9:a3d%3] mit 32 Bytes Daten:
Antwort von fe80::8c5:22c6:b1d9:a3d%3: Zeit<1ms
Antwort von fe80::8c5:22c6:b1d9:a3d%3: Zeit<1ms
Antwort von fe80::8c5:22c6:b1d9:a3d%3: Zeit<1ms
Antwort von fe80::8c5:22c6:b1d9:a3d%3: Zeit<1ms

Ping-Statistik für fe80::8c5:22c6:b1d9:a3d%3:
    Pakete: Gesendet = 4, Empfangen = 4, Verloren = 0
    (0% Verlust),
Ca. Zeitangaben in Millisek.:
    Minimum = 0ms, Maximum = 0ms, Mittelwert = 0ms

On Azure, it cannot resolve:
PS C:\Windows\system32> ping WIN-DE6U4068NAF
Ping request could not find host WIN-DE6U4068NAF. Please check the name and try again.

Note that both points mentioned above work fine with my on-premise installation, just not on Azure.

@mathiasconradt Just to clarify on point 4., you have inbound rule on Network Security Group you have (?) an Inbound Security Rule to "allow tcp/80" to the host. (Example)

@pmsousa Ahem, sorry, I totally forgot about setting the rule on Azure! I just just setting it on the TP5 host machine. Now actuall everything from outside works fine for me. I can both reach any httpd server that I run on the host (i.e. Apache httpd), or I can run a http server inside the container and I can reach it from outside through the host's hostname or public IP (http://xxxx.westeurope.cloudapp.azure.com:80). This makes two of my previous reported issues obsolete.

@pmsousa I think I should delete this ticket again, because it's a different topic that your initial problem (which actually works on my side now after I added the Azure Firewall rule that I forgot.)
Maybe better to create a new separate ticket for your problem. If you want me to check anything on my machine where everything is working - for you to compare, let me know.
The remaining above two issues are actually no show stoppers and maybe even intended to work like that, not sure.

Go ahead and close this one. I'll open a ticker has soon I can get my subscription access again and do further testing. This afternoon my spending limit for this subscription was reached, due to several VMs being "mysteriously" changed from Stopped (Deallocated) to only Stopped during the weekend and burning ~350€!!! Still have to figure it out with Support... :(

Closing this ticket (see comment above)

Closing this ticket (see comment above)


feat(go/adbc/driver/snowflake): add support for a client config file

Adds a new option to set a client config file to set https://github.com/snowflakedb/gosnowflake/blob/a26ac8a1b9a0dda854ac5db9c2c145f79d5ac4c0/doc.go#L130 as a driver option.

@lidavidm - anything more needed on this?


Falcon7b prefill MLP perf optimizations

Part of: #8349
Goal is to optimise MLP mms (FF1 & FF2) on 128, 1024, 2048 sequence lengths.
The max that we can hit is calculated using this formula, and is the same for FF1 & FF2: (M * N * K / (64)) * 16
M, N, K - are matmul dimensions (M is seq_length / 32, N is 144, and M is 576) , 64 is the number of cores available on WH, and 16 is a little debatable number representing num_cycles to do a single computation (using LoFi). I'm aware of this number going between 16 and 22 depending on various factors.
So, anyway, using 16 cycles, these are the numbers we'll use as our max:
128 sequence length      - 83k ns
1024 sequence length    - 663k ns
2048 sequence length  - 1.327 mil ns
Current state present on main
Here is the perf for 3 important sequence lengths:
128  - FF1=518k ns, FF2=448k ns
1024 - FF1=2.64mil ns, FF2=2.38 mil ns
2048 - FF1=5.28 mil ns, FF2= 4.76mil ns
These all matmuls have subblocks (1, 1) (due to di/dt avoidance), use HiFi2, use bfp8 weights, and are 2D block sharded matmuls.
When running Falcon prefill model on those sequence lengths, we get output, k and v cache pcc of 0.99 in all variants (single chip, and multi chip)
I've run some experiments in order to try to speed up these matmuls, and here's what I've found (note - 128 seq_len is using a bad variant of matmul, and none of the experiments helped it, it needs fixing asap, 2048k is always x2 of 1024, so when talking about perf I'll focus on 1K only)
Experiments
I've ran experiments with upping up subblocks (on N150 machine where we don't see di/dt), reducing math fidelity, and using bfp4 weights
Using proper subblocks
When we set the proper subblocks, we immediately see perf gains on matmuls.
Here is the current state on main if we set proper subblocks on 1k seq_length:
FF1 = 1.62 mil ns, FF2 = 1.50 mil ns (compared to 2.64 and 2.38 with (1, 1) subblocks.
One important thing here: While running various experiments, I've noticed that none of the experiments (fidelities, bfp4 weights), produced any perf gain, as long as the subblocks are (1,1). This is I think because in that case, matmuls are trisc bound, and none of the settings impact the perf of the matmul, so the next experiments that I mention will assume that subblocks are properly set.
Usage of LoFi, instead of HiFi2
If we use Lofi, the PCC of the prefill stays the same on all sequence lengths, and on single and multi chip variants. It is 0.99 in all cases.
The perf gets immediately improved:
FF1 = 1.33 mil ns, and FF2 = 1.20 mil ns (down from 1.62, 1.50 when using HiFi2)
Usage of BFP4 weights:
When we use BFP4 weights on FF1 and FF2 there is a pcc drop. Output, and v_cache pcc can drop to 0.97, while the the K cache remains 0.99. If we use BFP4 on only one of the mlp matmuls (FF1 or FF2, does not matter which one), the pcc also drops, but to 0.98 in this case.
Using bfp4 instead of bfp8 (while HiFi2 was on), did not gain much perf benefits:
FF1 = 1.56 mil ns, FF2 = 1.47 mil ns (down from 1.62, and 1.50 in case of bfp8).
This is because when HiFi2 is on, MM is compute bound, so reducing the amount of data doesn't seem to help.
BFP4 + Lofi
If we now use BFP4 + Lofi, interestingly, the PCC is the same as if we were using BFP4 only (output, and v_cache can drop to 0.97 or 0.98 depending of if we set it on both matmuls or just one).
Now, the MMS are no longer compute bound, and usage of BFP4 immediately helps:
FF1=1.02 mil ns, FF2 = 0.820 mil ns (down from 1.33 and 1.20 of Bfp8 +  HiFi2 case) (@davorchap any ideas how to squeeze this more ^^?)
IF we compare these numbers to the theoretical max (663k ns for both MMs), we see that they are pretty close.
Now I was interested to see if that perf is compute or dram/noc bound, so I've gone over kernels and disabled those, and here's what I've found:
Disabling DRAM reads
Activations and outputs are shraded on both MMS, so I just needed to disable weight dram reads and use dummy data for that.
If that is done, perf is exactly the same as with reading weights from dram, so it isn't dram bound :)
Disabling mcasts in addition of DRAM reads
There's two multicasts going on in these matmuls (shard mcast between neighbouring cores, and weight mcast down the column)
When those two are disabled, and dummy data is used instead, the perf does get slightly better:
FF1=944k ns, FF2=792k ns (compared to 1.02 mil, and 820k)
FF1 & FF2 difference in perf
Theoretical value for both of these is the same, but in practice, their perf differs due to the different inner dim and N of the two matmuls).
FF1 is (in tiles) [seq_len // 32, 144] * [144, 576]
FF2 is (in tiles) [seq_len // 32, 576] * [576, 144]
For FF1, this leads to per_core_M of 4, and per_core_N of 72, which doesn't look balanced, and we can chose max in_block_w of 3 before running out of memory on this one.
For FF2, per_core_M is 4 and per_core_N is 18, and we can chose 8 as in0_block_w.
In addition to that, FF1 is also running a fused gelu. If we remove that, in the best scenario, FF1 drops from 944k to 870k ns, bringing it closer to 790k of FF1)
I remember that on Buda we had option to do GELU on packer thread on WH_B0 instead of on math thread, so we might get rid of gelu difference, don't think that's enabled in metal atm, @ttmtrajkovic @rtawfik01 did that provide any gain, and is it supported on BH as well?

Ff1 should be a bit slower due to gelu in case it's compute bound?

Ff1 should be a bit slower due to gelu in case it's compute bound?

Yes, its mentioned above

Well it's seems close to 80% of theoretical number which is best we have seen so far?

Here's the perplexity and top1/top5 measures of bfp8 weights vs bfp4 weights. (For decode and prefill)
Top1/top5 seem pretty unchanged, while the perplexity can increase a bit.

@pavlepopovic Thanks for running the experiments. I think the slight pcc/perplexity drops are acceptable for decode given the strong device perf improvements for ff1/ff2 which are the main bottlenecks. If the output tokens also look good then it should be fine.

fyi @uaydonat


remove: pathRewrite limitation of 1 rewrite

Only 1 rewrite is allowed (#52 )
Remove this limitation.

won't fix


Need 2x2 thrusters

Should provide more than 4x the thrust, like 16x to encourage its use on bigger ships like atlas.

Even though we need stuff to use up power, having thrusters use piped gas would be a neat option, or burn plasma.
I have a doc somewhere. Something something make the inside face atmos blocking so you can access it from inside the ship, and also leads to the ship being spaced if destroyed.

willing 2 code just need sprites

willing 2 code just need sprites

/tg/ has one though you may need to slap on the burn effect to it.


RemoveUnusedimports removes used imports

I ran this recipe on a few private repositories, and I had some compilation failures because of imports that where removed, but were in fact used. I fail to fin a pattern to explain the error, and thus create a unit test: sometimes the class was used in javadoc (@throws or @link), sometimes in annotations, ...
So I played a bit with Moderne to reproduce on some public repositories.
Here's one: https://github.com/code4craft/webmagic/pull/1037/files
[INFO] --- maven-compiler-plugin:3.8.1:testCompile (default-testCompile) @ webmagic-extension ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 24 source files to /tmp/webmagic/webmagic-extension/target/test-classes
[INFO] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/MockPageModelPipeline.java: Some input files use or override a deprecated API.
[INFO] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/MockPageModelPipeline.java: Recompile with -Xlint:deprecation for details.
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[932,26] cannot find symbol
  symbol:   class Request
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[932,43] cannot find symbol
  symbol:   class Task
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[932,12] cannot find symbol
  symbol:   class Page
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[933,9] cannot find symbol
  symbol:   class Page
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[933,25] cannot find symbol
  symbol:   class Page
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[936,29] cannot find symbol
  symbol:   class Request
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[ERROR] /tmp/webmagic/webmagic-extension/src/test/java/us/codecraft/webmagic/downloader/MockGithubDownloader.java:[937,25] cannot find symbol
  symbol:   class PlainText
  location: class us.codecraft.webmagic.downloader.MockGithubDownloader
[INFO] 7 errors

@murdos thanks for reporting the issue!
I looked into the cause and the removed used imports are typically missing type attribution.
In the linked example, the classes belong to different source sets but have the same FQN.
Ex:
us.codecraft.webmagic.SpiderTest is in:
"webmagic/webmagic-core/src/test/java/us/codecraft/webmagic/SpiderTest.java"
and
"webmagic/webmagic-samples/src/test/java/us/codecraft/webmagic/SpiderTest.java"
The source files are collected in the parser and processed at the same time. So, the compiler throws a duplicate class exception.
Unassignning myself, since I think this one will require Jon or Tyler.

FYI I still reproduce this kind of issue (probably with a different origin) while applying the recipe to https://github.com/murdos/OpenCyclos/blob/master/src/main/java/nl/strohalm/cyclos/controls/accounts/cards/SearchCardsAsBrokerAction.java using last code from main branches of rewrite and rewrite-maven-plugin.

Hi
I have the same problem with enums
For example in this enum: RemoveUnusedImports remove SerializedName
import com.google.gson.annotations.SerializedName;

public enum PKIState {
    @SerializedName("active") ACTIVE,
    @SerializedName("dismissed") DISMISSED
}

all the best
matteo baccan

@matteobaccan I have moved your problem to a separate issue #1251

@murdos Finally got some time to investigate your last issue:
Having a @link immediately after an @See is illegal according to the JavaDoc specification. OpenRewrite relies on the Javadoc parsing that is native to the platform and @see {@link ActionPolicyRepository} to view the required permissions is parsed as an ErroneousTree and therefore the link is never visited (nor is the type information in the link collected).
The solution to this specific issue is to either use @See ActionPolicyRepository or {@link ActionPolicyRepository} (without the @See).
A couple of references that describe this specific behavior:
https://docs.oracle.com/javase/7/docs/technotes/tools/windows/javadoc.html#see
https://stackoverflow.com/a/65511981

@tkvangorder: I don't think this issue should be closed, the original issue is still investigated, according to last comment of @traceyyoshima : https://github.com/openrewrite/rewrite/issues/1066#issuecomment-952123316

@murdos the new type cache accounts for the same FQN in multiple packages, but we're still working on a release.