Why are we forced to use clang?

Hi,
I'm trying to use this great project to run flutter application based on my buildroot toolchain. In this line:
https://github.com/sony/flutter-elinux/blob/b09a90eee643859ce4e676839227edd9fd3feba8/lib/elinux_build_target.dart#L376
you force the compiler to be clang. I could not get things working from clang if I want to use the linker built from buildroot, getting errors saying crtbegin.o and -lgcc not found.
However, from my testing, gcc will also build a runnable application. I changed this line to use aarch64-buildroot-linux-gnu-gcc (and g++), it worked, and the build result is runnable on target. So, I wonder if there's special reason to force the compiler to be clang, and if not, could we add the compiler name as a command line argument to let the user decide which toolchain to use?
Please share your thoughts, thank you very much!

getting errors saying crtbegin.o and -lgcc not found.

Did you use target-compiler-triple option?

. So, I wonder if there's special reason to force the compiler to be clang, and if not,

Because Flutter basically support only clang/llvm. Some source code will cause build errors with gcc.

could we add the compiler name as a command line argument to let the user decide which toolchain to use?

Technically possible.

getting errors saying crtbegin.o and -lgcc not found.

Did you use target-compiler-triple option? Also did you install clang/llvm?

Yes, the build command is as follows:
~/flutter-elinux/bin/flutter-elinux build elinux --target-arch=arm64 \
                                                   --target-compiler-triple=aarch64-buildroot-linux-gnu \
                                                   --target-sysroot=/home/dev/buildroot/output/staging --target-backend-type=wayland --release \
                                              --system-include-directories=/home/dev/buildroot/output/staging/usr/include

The fail log:
💪 Building with sound null safety 💪

Building an eLinux application with wayland backend in release mode for arm64 target...            ⣷
   541ms
Failed to cmake:
-- The CXX compiler identification is Clang 11.0.1
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: /usr/bin/clang++
-- Check for working CXX compiler: /usr/bin/clang++ - broken
-- Configuring incomplete, errors occurred!
See also "/home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeOutput.log".
See also "/home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeError.log".

CMake Error at /usr/share/cmake-3.18/Modules/CMakeTestCXXCompiler.cmake:59 (message):
  The C++ compiler

    "/usr/bin/clang++"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/gmake cmTC_ad829/fast && /usr/bin/gmake  -f CMakeFiles/cmTC_ad829.dir/build.make CMakeFiles/cmTC_ad829.dir/build
    gmake[1]: Entering directory '/home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeTmp'
    Building CXX object CMakeFiles/cmTC_ad829.dir/testCXXCompiler.cxx.o
    /usr/bin/clang++ --target=aarch64-buildroot-linux-gnu    -o CMakeFiles/cmTC_ad829.dir/testCXXCompiler.cxx.o -c
    /home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeTmp/testCXXCompiler.cxx
    Linking CXX executable cmTC_ad829
    /usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_ad829.dir/link.txt --verbose=1
    /usr/bin/clang++ --target=aarch64-buildroot-linux-gnu --sysroot=/home/dev/buildroot/output/staging CMakeFiles/cmTC_ad829.dir/testCXXCompiler.cxx.o -o cmTC_ad829
    /home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld: cannot find crtbegin.o: No such file or directory
    /home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld: cannot find -lgcc
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    gmake[1]: *** [CMakeFiles/cmTC_ad829.dir/build.make:106: cmTC_ad829] Error 1
    gmake[1]: Leaving directory '/home/dev/gui/build/elinux/arm64/release/CMakeFiles/CMakeTmp'
    gmake: *** [Makefile:140: cmTC_ad829/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:18 (project)

It uses the host clang, with the buildroot-toolchain gcc as linker. The linker is not specified with corrent option by clang.  search starts here:
#include <...> search starts here:
 /usr/lib/llvm-11/lib/clang/11.0.1/include
 /home/dev/buildroot/output/staging/usr/include
End of search list.
 "/home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld" --sysroot=/home/dev/buildroot/output/staging -EL --hash-style=both --build-id --eh-frame-hdr -m aarch64linux -dynamic-linker /lib/ld-linux-aarch64.so.1 -o a.out /home/dev/buildroot/output/staging/lib/../lib64/crt1.o /home/dev/buildroot/output/staging/lib/../lib64/crti.o crtbegin.o -L/home/dev/buildroot/output/staging/lib/../lib64 -L/home/dev/buildroot/output/staging/usr/lib/../lib64 -L/home/dev/buildroot/output/staging/lib -L/home/dev/buildroot/output/staging/usr/lib /tmp/1-05db34.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed crtend.o /home/dev/buildroot/output/staging/lib/../lib64/crtn.o
/home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld: cannot find crtbegin.o: No such file or directory
/home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld: cannot find -lgcc
/home/dev/buildroot/output/host/bin/aarch64-buildroot-linux-gnu-ld: cannot find -lgcc
clang: error: linker command failed with exit code 1 (use -v to see invocation)

This is basically because the directory where crtbegin.o and libgcc.a resides (/home/dev/buildroot/output/host/lib/gcc/aarch64-buildroot-linux-gnu/10.4.0) is not passed to the linker as an option. The root cause is that clang is not trying to find gcc installation of my cross compile toolchain. If I try to run clang alone without cross compiling, it will try to find gcc installation and get the correct linker option passed (note the Selected GCC installation message):
clang 1.c -v
Debian clang version 11.0.1-2
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/10
Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/10
Selected GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/10
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Candidate multilib: x32;@mx32
Selected multilib: .;@m64
 "/usr/lib/llvm-11/bin/clang" -cc1 -triple x86_64-pc-linux-gnu -emit-obj -mrelax-all -disable-free -disable-llvm-verifier -discard-value-names -main-file-name 1.c -mrelocation-model static -mframe-pointer=all -fmath-errno -fno-rounding-math -mconstructor-aliases -munwind-tables -target-cpu x86-64 -fno-split-dwarf-inlining -debugger-tuning=gdb -v -resource-dir /usr/lib/llvm-11/lib/clang/11.0.1 -internal-isystem /usr/local/include -internal-isystem /usr/lib/llvm-11/lib/clang/11.0.1/include -internal-externc-isystem /usr/include/x86_64-linux-gnu -internal-externc-isystem /include -internal-externc-isystem /usr/include -fdebug-compilation-dir /home/dev/smartcross_gui -ferror-limit 19 -fgnuc-version=4.2.1 -fcolor-diagnostics -faddrsig -o /tmp/1-63b050.o -x c 1.c
clang -cc1 version 11.0.1 based upon LLVM 11.0.1 default target x86_64-pc-linux-gnu
ignoring nonexistent directory "/include"
#include "..." search starts here:
#include <...> search starts here:
 /usr/local/include
 /usr/lib/llvm-11/lib/clang/11.0.1/include
 /usr/include/x86_64-linux-gnu
 /usr/include
End of search list.
 "/usr/bin/ld" --hash-style=both --build-id --eh-frame-hdr -m elf_x86_64 -dynamic-linker /lib64/ld-linux-x86-64.so.2 -o a.out /usr/bin/../lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu/crt1.o /usr/bin/../lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu/crti.o /usr/bin/../lib/gcc/x86_64-linux-gnu/10/crtbegin.o -L/usr/bin/../lib/gcc/x86_64-linux-gnu/10 -L/usr/bin/../lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu -L/lib/x86_64-linux-gnu -L/lib/../lib64 -L/usr/lib/x86_64-linux-gnu -L/usr/bin/../lib/gcc/x86_64-linux-gnu/10/../../.. -L/usr/lib/llvm-11/bin/../lib -L/lib -L/usr/lib /tmp/1-63b050.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/bin/../lib/gcc/x86_64-linux-gnu/10/crtend.o /usr/bin/../lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu/crtn.o

The current solution to me, as mentioned above, is to replace clang with my toolchain gcc.

. So, I wonder if there's special reason to force the compiler to be clang, and if not,

Because Flutter basically support only clang/llvm. Some source code will cause build errors with gcc.

OK, I understand. But at least gcc is working fine to me.

could we add the compiler name as a command line argument to let the user decide which toolchain to use?

Technically possible.

Update: after applying https://github.com/sony/flutter-elinux/pull/105 , and add the following line to CMakeLists.txt:
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} --gcc-toolchain=/home/dev/buildroot/output/host")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} --gcc-toolchain=/home/dev/buildroot/output/host")

The clang compiler now works.


Add support for reverse and ram payer in Get Table

EOS 1.5.0 made a number of changes to Get Table: https://github.com/EOSIO/eos/releases/tag/v1.5.0
We need to add support for reverse search and showing ram payer.

add this fix: https://github.com/GetScatter/eos-sharp/commit/e5b09f4cbdd566fccdc12f86a7758c28857c41cf
Give it a try


Release v2.0.0

Coverage remained the same at 87.143% when pulling 4daa73b0786168b0ae4f8250a5658e51893ea3c1 on jshmrtn:release/v2.0.0 into 2466a739f475cf9df4e7204727211cbb0481ba32 on c-rack:master.


Support PaySafeCard

Via https://gratipay.freshdesk.com/helpdesk/tickets/3837 (Facebook):

Can you please support PaySafeCard?

This was mentioned previously in https://github.com/gratipay/gratipay.com/issues/126.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Re-ticketed to https://github.com/gratipay/inside.gratipay.com/issues/974 for further discussion and planning. After deciding on concrete implementation plan and dev work starts, this will be reopened for tracking purpose.


Can't set folder size

In Anywhere Widgets for iPad 1.1, the folder size was 4x5.
Is there any way you could add the functionality to change folder size to PadGrid? It's a little ridiculous that on my iPhone, I can use a 6x7 folder (using Folded) but on my larger iPad, I have to settle for 4x4...

I take that Folded isn't compatible with iPad?

@PoomSmart At least not with my iPad setup, and I can't figure out why. It's not officially compatible with iOS/iPadOS 14, or being updated, so it's kind of a hit-or-miss thing, I guess.
Anyways, I would really appreciate it if any option to increase folder size were added, to this tweak or a separate tweak.


Adding bullets and periods

closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/3012

@kaarins Can you please check and merge? Thanks

@tiburd can you help me getting this merged?

Confirmed markdown is correct. Merging this PR.


setInitialRotation doesn't work until you zoom/unzoom image

Hi,
setInitialRotation doesn't work until you zoom/unzoom image.
(on Samsung Galaxy S6 running Android 6.0.1)
Really nice library, congrats !

It is possible that the issue is with exif data.
When you explicitly set the initial rotation it is NOT in addition to the exit data but overrides it.
Example:
If the image has exif data telling it to be rotated by 90 degrees then the library will read it and rotate it for you.
If you set initial rotation to be 90 degrees then it will also rotate it by 90 degrees resulting in the same image rotation so you may think that rotation didn't work at all.
In that case you have to read the exit data and add the required rotation to it.
see BitmapUtils.rotateBitmapByExif method on how to read exif data.
Let me know if this was the issue.

Hi, this is a very belated comment, but the issue seems to be similar to what I see, with:
CropImage.activity(mCurrentPhotoURI)
                                .setInitialRotation(mCropRotation)
                                .setInitialCropWindowRectangle(mRotatedRect)
                                .start(this);

The rotation is properly set and accounts for EXIF, but visually, it doesn't show until I drag the crop rectangle somewhere such that the image view pans or zooms. Then the image view suddenly refreshes to the expected rotation.
It only happens when initial rotation is used in combination with a setInitialCropWindowRectangle() -- with just a setInitialRotation().start(), it works fine as expected.
If you read this, please don't feel obligated to do anything--I just wanted to document the behavior. Thank you and congratulations on such a lightweight, comprehensive, and fun cropping library!


Widgets API

This is the beginning of the high level widgets API, there are a number of things in this code that need to be cleaned up - but I think this is what we're looking for from an infrastructure standpoint.
This breaks the existing legend/slider widgets until we solidify this and rewrite those to conform.
Overview

Every widget is created as a top level element within the UI Layer, which used to be impeded by the renderer

Widgets have basic positioning functionality, sticky or not, absolute positioned on a map according to viewport or GCS

Widgets can tell whether they are in the viewport or not

Widgets can be nested within each other

Example Usage
var widget = uiLayer.createWidget('dom', {
    sticky: true,
    positionType: 'gcs',
    position: {
        x: -73.7572,
        y: 42.6525
    }
});

widget.$el.attr('id', 'sticks-to-albany-div');

Todo

Canvas/el/$el is pretty confusing
@jbeezley - How do we feel about access to a jQuery object? In terms of convention, the UI layer canvas returns the DOM element so I suppose we should stick to that?

svgWidget needs to inherit from domWidget

svgWidget addChild
This needs to be implemented so that adding a child to an svgWidget correctly handles the grouping of SVG elements.

Passing arg to widget init
I think I'm doing unnecessary variable passing - still need to look into it.

Tests, documentation, updated examples
I think this will warrant a good amount of documentation and testing - waiting for the design decisions to settle before starting them.

This is looking great!
I would prefer to pass raw DOM elements through the public API.  I might consider removing the jquery dependency at some point.

@jbeezley The slider widget mostly works. It doesn't appear at first because I create svg elements with no height/width specified - is there a reason for it being the whole viewport by default?
This code will generate a slider that works with the exception of parts disappearing on mouseover, still looking into that.
  var slider = uiLayer.createWidget('slider');

  $(slider.canvas()).attr({
    height: 200,
    width: 100
  });

Left to do:

[ ] Fix slider widgets mouseover events
[ ] Make legend widget use the new widget API
[ ] Add proper nesting (with svg groups) to svg widget
[ ] Fix tests, add documentation

The slider widget mostly works. It doesn't appear at first because I create svg elements with no height/width specified - is there a reason for it being the whole viewport by default?

No, I don't think so.  You can probably get the actual size of the widget using a getBBox call and use that to set the svg size.

0e653ff definitely broke a few things and will have to be fixed, but I want to make sure I'm not going down a wrong path by moving the functionality from the d3Renderer to the svgWidget.
Aside from refactoring and ensuring tests pass, finding bugs, etc - the main problem left is handling the nesting of SVG widgets.
Nesting in the general sense of:
<svg id="foo">
    <svg id="bar"></svg>
</svg>

Already works by specifying a parent widget, since svgWidget inherits from domWidget. So what's left is re-introducing the logic which allows the nesting of widgets as svg group elements.
@jbeezley Ping me if anything sounds far gone.

It looks like you can actually nest svg elements.  I don't really have a problem with doing that instead of making it a group element if it is easier.  The most likely use case I can imagine is nesting multiple svg widgets inside a dom widget anyway.

I think this is ready for review -
The only "breaking" change this may have to applications is that the UI layer is now positioned relatively.
Testing
There are some basic jasmine tests setup and passing, they could certainly be more exhaustive. @jbeezley Do you think there is any point at all to making browser-based (non-jasmine) tests since we're at a pseudo-shift in the way we test?
Documentation
I think the code I've added has pretty good documentation (note places you find it lacking) - but do we want an additional example, or something on readthedocs?

I still need to go over the code, but it looks pretty good.  It would be nice to have an example of some sort.  Also, could you go over the class overview on rtd and update it for the new API?

@jbeezley @danlamanna if its not urgent, would it be okay if I look at the code as well before we merge this one. I am hoping to get it done by tomorrow.

@danlamanna I made one small change to this branch that will make the example look better when it is merged with master.

Other than my comment above, I'm happy this this.  Very nice!  I'll let @aashish24 have his say before final approval though.

@aashish24 Nope not urgent - let me know what you think.
I've caught a few small bugs myself that I still need to work on.

LGTM :+1: good stuff, only had some minor suggestions some of them are beyond the scope of this branch.

Nice image you got @jbeezley!.. very cool.

@jbeezley Take a look at 2e578db - getting it to "stick" to the top right throughout resizes is a bit cumbersome.
Since it's easier to just do uiLayerWidth - widgetWidth - margin do you think we should allow position to be some sort of function that takes the uiLayer node as an argument?

There may be a word definition problem here.  Sticky in the layer API means that it sticks to the map not to the screen.  What should happen in this case is that the legend will stay in place without having to reposition on pan.
There is a complication in that for core, the sticky parameter goes on the layer, but here it is on the widget.  What happens when the layer is sticky, but he widget is not?  Does it stay fixed relative to the layer or the map?

There is a complication in that for core, the sticky parameter goes on the layer, but here it is on the widget. What happens when the layer is sticky, but he widget is not? Does it stay fixed relative to the layer or the map?

I think sticky here should refer to the fact that it is stick w.r.t layer or not. So if you move the layer does the widget moves with it or not. What you think?

Is there a reason widgets would need a stickiness property independently of
the layers?  Why not just rely on the application writer to put widgets
into sticky/non-sticky layers as needed?  It seems to really complicate the
semantics of stickiness otherwise.
On Mon, Aug 17, 2015 at 1:33 PM Aashish Chaudhary<EMAIL_ADDRESS>wrote:

There is a complication in that for core, the sticky parameter goes on the
layer, but here it is on the widget. What happens when the layer is sticky,
but he widget is not? Does it stay fixed relative to the layer or the map?
I think sticky here should refer to the fact that it is stick w.r.t layer
or not. So if you move the layer does the widget moves with it or not. What
you think?
—
Reply to this email directly or view it on GitHub
https://github.com/OpenGeoscience/geojs/pull/425#issuecomment-131899182.

Is there a reason widgets would need a stickiness property independently of
the layers?  Why not just rely on the application writer to put widgets
into sticky/non-sticky layers as needed?  It seems to really complicate the
semantics of stickiness otherwise.

I could go either way. I think having a stick property for the widget would be nice but not required. It does complicate things. Cost of creating a dom layer shouldn't be that high so a more simple approach would be fine I guess. I don't think we will be creating thousands of widgets. It will be in the range of 1-20 for most applications.

Is there a reason widgets would need a stickiness property independently of the layers? Why not just rely on the application writer to put widgets into sticky/non-sticky layers as needed? It seems to really complicate the semantics of stickiness otherwise.

The initial idea was that since layers are somewhat heavy, being able to have only one UI layer with any amount of widgets, positioned however they each want to be positioned, was more desirable.

I could go either way as well.  I can see enabling a layer that is fixed to the screen, but a widget that moves with the map.  The other way around is more difficult because you need to invert the transform of the parent, which will invariably lead to errors due to numerical precision at some point.  So the widget would wobble back and forth +/- 1 pixel (or more maybe).
For the moment, I'm in favor of doing what is the least amount of work as long as it doesn't limit us too severely at an API level.

For the moment, I'm in favor of doing what is the least amount of work as long as it doesn't limit us too severely at an API level.

+1
I would suggest that here sticky should use layer as the reference point like I suggested earlier if that's not the case already.

@danlamanna looks like one of the style tests is failing (sliderWidget). Can you fix that? thanks

thanks! @jbeezley  can we merge this now? I am ok with removing stick from widgets.

LGTM (for reals this time).


Adjust path in security.yaml for 1.10 to match behat tests

Q
A

Branch?
1.10+

Bug fix?
yes

New feature?
no

BC breaks?
yes (people expecting the API to be at /new-api in their old test will have their tests fail)

Deprecations?
no

Related tickets
fixes #310

License
MIT

New pull request incoming that fixes more issues


Ability to compare ElementHandle instances

My use case is for scraping sites with infinite scrolling, I want to do the following:

create an array of elements matching the selector
scroll to the bottom of the page (making it load new elements)
wait for new elements matching the selector
iterate through the new elements

However, it's quite difficult to determine what's a new element as there doesn't seem to be a way to determine if an ElementHandle is pointing to the same element as another instance of ElementHandle.
I hope this makes sense and fits in with your vision of what you want puppeteer to become!

(I think I've already answered this on our Slack channel)
You can compare two element handles with === inside the page:
const e1 = await page.$('.foo');
const e2 = await page.$('.bar');
const equal = await page.evaluate((e1, e2) => e1 === e2, e1, e2);

Hope this helps!


test code notation with break

Just idea.
Now we need to set breakpoints with break commands on tests like:
    def program
      <<~RUBY
       1| def foo a
       2|   b = a + 1  # break
       3| end
       4| x = 1        # break
       5| x = 2
       6| x = 3
      RUBY
    end

    def test_foobar
      debug_code program do
        type 'break 2'
        type 'break 4'

On this code, line 2 and 4 is breakpont, and using 2 break commands. On the other hands, if we use IDE we can set breakpoints on a code (with editor's support). So we can write breakpoints on the code.
IDEA1: Use comment on the line.
      <<~RUBY
       1| def foo a
       2|   b = a + 1  # break
       3| end
       4| x = 1        # break
       5| x = 2
       6| x = 3
      RUBY

and issues two break commands by the test framework.
IDEA2: Check on the left of line number:
      <<~RUBY
       1| def foo a
      *2|   b = a + 1
       3| end
      *4| x = 1
       5| x = 2
       6| x = 3
      RUBY

and issues two break commands by the test framework.

Of course we can write debugger method, but it can be different from normal break points so it should be different.

We can enhance this idea for normal Ruby programming by parsing the source code (with IDEA 1), but it's too much and almost case debugger method is enough. The difference between break command and debugger method is we can control (add/delete) breakpoints added by break commands (we can not delete deubgger method on a running process).

I like the IDEA 1 better but only for debugger tests. I think putting breakpoint comment in the code will cause more trouble than convenience. For example: users may need to exit other people's breakpoint comments before entering the one they set.


Implement icon picker

With changes from 851ca9d an icon picker is needed in order to be able to change the icon. This will show up after clicking the "Pick..." button.

Added in https://github.com/egartley/game-tracker/commit/3524d67d0482b9982b2c8efd54df83b3f7b6e39c


[AspireDeployment] Update JSON RPC AspireService and EnvironmentService APIs to include AppHost project path.

Update the EnvironmentService and AspireService JSON RPC APIs to include the AppHost project path. This will enable scenarios where more than one AppHost project exists.
These are the APIs requiring the update:
IAspireService
ValueTask<AspireHost> GetAspireHostAsync(Session session, string? hostProjectPath, string aspireEnvironment, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

ValueTask RenameAspireHostAsync(Session session, string? hostProjectPath, string newPath, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

IEnvironmentService
ValueTask<bool> CreateEnvironmentAsync(Session session, string? hostProjectPath, Environment environment, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

ValueTask<Environment> OpenEnvironmentAsync(Session session, string? hostProjectPath, string name, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

ValueTask<Environment> LoadEnvironmentAsync(Session session, string? hostProjectPath, string name, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

ValueTask<Environment> RefreshEnvironmentAsync(Session session, string? hostProjectPath, string name, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

ValueTask<Environment> DeployAsync(Session session, string? hostProjectPath, string environmentName, IObserver<ProgressMessage> outputObserver, CancellationToken cancellationToken);

Please refer to https://devdiv.visualstudio.com/DevDiv/_git/WebTools/pullrequest/538405 for the actual code changes in VS IDE.

Multiple app host projects in a solution is not supported and is out-of-scope for Aspire 8.0/GA
cc @bradygaster @ellismg @abpiskunov

Duplicate of #3563


Document balance warnings

Modify this link to include the branch name, and possibly the page this PR modifies:
https://osc.github.io/ood-documentation-test/develop/
Add your description here
Document balance warnings and OOD_DASHBOARD_HELP_CUSTOM_URL. This is part of 1.7 docs.

lgtm


Broken URL: Link to Molecule docs outputs a 404

In the latest version of the book on page 350, the link to Molecule docs is broken.

I fixed this in the latest version of the book, which should be up on LeanPub soon (Amazon and iBooks to follow later). Thanks for reporting! There are a few links to Ansible Lint and Molecule that were broken.


On Linux, launching the game with Steam but with Steam Input disabled causes the game to instantly crash

IDE: 2<IP_ADDRESS>
Input version 6.1.5 (also tried 6.2.1 beta)
I'm experiencing a bug where launching the game with Steamworks enabled but Steam Input disabled causes the program to instantly close. I can't even get a single frame, so I'm unsure of how to debug it. I can't even playtest because of a bug that renders the Steamworks APIs unreachable, and are therefore not loaded. With Steam disabled, the game works. Launching the game with Steam Input enabled also works.
Update: I've tried setting INPUT_ALLOW_STEAMWORKS to false, and that still didn't fix it. Also, this bug happens on the Steam Deck too.

Please test a blank project with the Steamworks extension to confirm the issue is with Input and not the Steamworks extension

The project originally didn't use Input -- it used the native Game Maker commands, and it worked. Attached is a Linux
build of my game created before Input support was implemented: onu181223bII_l.zip

if you'd like me to create a new project with the requirements you specified, let me know

Yes, please

You've probably already seen in the Discord, but the problem is in fact Input related. Here's the new project I used:
Blank.zip
If there's any way I can troubleshoot this, I'll let you know.

Closed - may not be input related after all, need to do some more troubleshooting

It appears that this IS in fact Input related. The error logs show that an erroneous variable, _default_xbox_type, is not defined. In fact, this variable is only used in this one instance. See the images below.

Regression from ee08eba148a0ddf5b6d59f2882b7aafe6af7b9a7
Fixed by 5024d71475528075690f4d7497a1d9d093432e50


Shortcut Guide not appearing in v0.25

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)
Hold-pressing Win does not show Shortcut Guide.
I think it used to work in v0.23.4
✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

@stdedos what app is in focus?  Task manager or like Remote Desktop?
what language is your computer in?  wondering why only one part of that UI is localized.

@stdedos what app is in focus? Task manager or like Remote Desktop?

No idea, and I cannot re-trigger it. ... Although, I did with with ProcessHacker (which itself is in admin mode), and PowerToys are not (maybe add a "gray notice of non-admin mode" if that's a limitation?)

what language is your computer in? wondering why only one part of that UI is localized.

Greek. No part of the PT is localized (and I don't care / maybe forced the language to English).

What happened now is this:

I was playing with a feature, and it was working okay
...
I updated ProcessHacker (done it twice because weird reasons, always in admin mode)
I brought it on the foreground (was on tray)
I played with it, dismissed it with Esc
And I thought "well, let's try ShortcutGuide!

Expected:
ShortcutGuide appears
Actual:

Is PT being run as admin?  If process hacker was dismissed, it still could technically be in focus.

Yes, I think this must be it.
Releasing a holded WinKey gives Start Menu (Open Shell-Menu to be precise), on which then holding WinKey is responding.

Yes, I think this must be it.
Releasing a holded WinKey gives Start Menu (Open Shell-Menu to be precise), on which then holding WinKey is responding.

@stdedos
can we close the issue?

@stdedos
can we close the issue?

Thank you for asking (although I guess it is redundant 😝).
Well, I think it is triaged pretty nicely; so yes.

Thank you for asking (although I guess it is redundant 😝).
Well, I think it is triaged pretty nicely; so yes.

You might be interested on some knowledge given in the PH ticket I have opened: https://github.com/processhacker/processhacker/issues/770

You might be interested on some knowledge given in the PH ticket I have opened: https://github.com/processhacker/processhacker/issues/770

@enricogior
I believe its an issue with PowerToys incorrectly using GetAncestor for UWP based on the code highlighted here?
https://github.com/processhacker/processhacker/issues/770#issuecomment-755125173
The method we use with GetPropW was based on window enumeration in twinapi.dll from the OS:
GetProp(WindowHandle, L"Windows.ImmersiveShell.IdentifyAsMainCoreWindow")
This property is either NULL or 0x1 when the window is the actual focused UWP window. PowerToys is using the class name "Windows.UI.Core.CoreWindow" when there are multiple windows with this class. You have to use the above property to distinguish between the two (just like twinapi.dll does).

@enricogior
I believe its an issue with PowerToys incorrectly using GetAncestor for UWP based on the code highlighted here?
https://github.com/processhacker/processhacker/issues/770#issuecomment-755125173
The method we use with GetPropW was based on window enumeration in twinapi.dll from the OS:
GetProp(WindowHandle, L"Windows.ImmersiveShell.IdentifyAsMainCoreWindow")
This property is either NULL or 0x1 when the window is the actual focused UWP window. PowerToys is using the class name "Windows.UI.Core.CoreWindow" when there are multiple windows with this class. You have to use the above property to distinguish between the two (just like twinapi.dll does).

Hi @dmex
I haven't found any documentation regarding Windows.ImmersiveShell.IdentifyAsMainCoreWindow

Hi @dmex
I haven't found any documentation regarding Windows.ImmersiveShell.IdentifyAsMainCoreWindow

@enricogior
It's not documented but without it you can't locate UWP windows/processes correctly. The other requirement is the PowerToys executable manifests are required to include the disableWindowFiltering attribute and disable immersive window filtering so you can actually enumerate immersive windows from desktop applications.
The disableWindowFiltering attribute is documented here: https://docs.microsoft.com/en-us/windows/win32/sbscs/application-manifests
You can see the IdentifyAsMainCoreWindow property on the main UWP window and it's been there since 2012 - you only need to check for it's existence and not the value:

@enricogior
It's not documented but without it you can't locate UWP windows/processes correctly. The other requirement is the PowerToys executable manifests are required to include the disableWindowFiltering attribute and disable immersive window filtering so you can actually enumerate immersive windows from desktop applications.
The disableWindowFiltering attribute is documented here: https://docs.microsoft.com/en-us/windows/win32/sbscs/application-manifests
You can see the IdentifyAsMainCoreWindow property on the main UWP window and it's been there since 2012 - you only need to check for it's existence and not the value:

@dmex

It's not documented but without it you can't locate UWP windows/processes correctly

Do you have an example/repro steps that shows the problem?

@dmex

It's not documented but without it you can't locate UWP windows/processes correctly

Do you have an example/repro steps that shows the problem?

@enricogior
An example would probably be how some UWP windows with the class Windows.UI.Core.CoreWindow can become foreground/focused but won't contain the property. In this case you keep searching until locating IdentifyAsMainCoreWindow:

@enricogior
An example would probably be how some UWP windows with the class Windows.UI.Core.CoreWindow can become foreground/focused but won't contain the property. In this case you keep searching until locating IdentifyAsMainCoreWindow:

@dmex

An example would probably be how some UWP windows with the class Windows.UI.Core.CoreWindow can become foreground/focused but won't contain the property.

We need an actual real case, since we are not going to make any change to the code unless it can be tested against a reproducible problem.

@dmex

An example would probably be how some UWP windows with the class Windows.UI.Core.CoreWindow can become foreground/focused but won't contain the property.

We need an actual real case, since we are not going to make any change to the code unless it can be tested against a reproducible problem.

@enrico

We need an actual real case

@stdedos reported the "real case" here:
https://github.com/microsoft/PowerToys/issues/7814#issue-735550432
and here:
https://github.com/microsoft/PowerToys/issues/7814#issuecomment-727179919
Just to recap; The issue was closed blaming\referring the issue to my project. I reviewed the issue but wasn't able to reproduce by testing GetForegroundWindow: https://github.com/processhacker/processhacker/issues/770
We tested GetAncestor in the past when coding our handling\support for UWP windows with Process Hacker (11 years ago) and learned from that experience GetAncestor doesn't work properly and after seeing the PowerToys source I know its very likely responsible for incorrect foreground application focus problem reported by @stdedos.
I mean you won't find GetAncestor anywhere in our sourcecode and yet we correctly determine the foreground window/application when using the Find Window functionality and/or when using the WindowTitle\WindowStatus processes columns: https://github.com/processhacker/processhacker/search?q=GetAncestor

we are not going to make any change to the code unless it can be tested against a reproducible problem

That's reasonable I guess.... At least when you do get a repro case you'll know how to fix it 😛

@enrico

We need an actual real case

@stdedos reported the "real case" here:
https://github.com/microsoft/PowerToys/issues/7814#issue-735550432
and here:
https://github.com/microsoft/PowerToys/issues/7814#issuecomment-727179919
Just to recap; The issue was closed blaming\referring the issue to my project. I reviewed the issue but wasn't able to reproduce by testing GetForegroundWindow: https://github.com/processhacker/processhacker/issues/770
We tested GetAncestor in the past when coding our handling\support for UWP windows with Process Hacker (11 years ago) and learned from that experience GetAncestor doesn't work properly and after seeing the PowerToys source I know its very likely responsible for incorrect foreground application focus problem reported by @stdedos.
I mean you won't find GetAncestor anywhere in our sourcecode and yet we correctly determine the foreground window/application when using the Find Window functionality and/or when using the WindowTitle\WindowStatus processes columns: https://github.com/processhacker/processhacker/search?q=GetAncestor

we are not going to make any change to the code unless it can be tested against a reproducible problem

That's reasonable I guess.... At least when you do get a repro case you'll know how to fix it 😛

@dmex

The issue was closed blaming\referring the issue to my project

No, we didn't blame your project, we pointed out that if an app is running elevated, PowerToys also needs to run elevated so it can process the Win key state and open the Shortcut Guide.
If Shortcut Guide fails to retrieve the window that is currently holding the focus, nothing really bad happens, it simply doesn't show any active window.
I appreciate your feedback, but as I said we need to find a case where the focused window is causing the problem.

@dmex

The issue was closed blaming\referring the issue to my project

No, we didn't blame your project, we pointed out that if an app is running elevated, PowerToys also needs to run elevated so it can process the Win key state and open the Shortcut Guide.
If Shortcut Guide fails to retrieve the window that is currently holding the focus, nothing really bad happens, it simply doesn't show any active window.
I appreciate your feedback, but as I said we need to find a case where the focused window is causing the problem.

we didn't blame your project

The main feature of Git and Github is blaming? 😕
https://www.atlassian.com/git/tutorials/inspecting-a-repository/git-blame
https://git-scm.com/docs/git-blame

we pointed out that if an app is running elevated, PowerToys also needs to run elevated so it can process the Win key state and open the Shortcut Guide.

Lets blame these changes:
https://github.com/microsoft/PowerToys/blame/212ea2de30d69dbbfebf86c9d3604008d2272346/src/modules/shortcut_guide/shortcut_guide.cpp#L246-L262
It's a really cool feature for reviewing what, how, why and when code was added while helping with context and history when dealing with issues or bug reports. You should be using it more often if you're not already?

PowerToys also needs to run elevated

The blame shows PowerToys is using global hotkeys and lowlevel keyboard hooks which were primarily designed for accessibility applications. PowerToys should have added uiAccess="true" to the executable manifest so the binaries are automatically elevated and able to receive hotkeys and keyboard hooks for elevated applications.
There are basically three requirements for global hotkeys and keyboard hooks:

The binary is digitally signed.
The binary is installed into Program Files.
uiAccess="true"

If the executable contains true for uiaccess in the manifest then it'll be silently and automatically elevated to administrator without prompting the user or requiring the user manually elevate... You will need to add this otherwise that code ever won't work properly because PowerToys would have to be manually elevated every single time.

we need to find a case where the focused window is causing the problem.

It's basically a race condition between GetForegroundWindow and GetAncestor... The values of both functions are going to change almost immediately so I doubt you'll ever get a reliable reproduction case. I've mentioned the fixes already so eventually someone can resolve the issues.

we didn't blame your project

The main feature of Git and Github is blaming? 😕
https://www.atlassian.com/git/tutorials/inspecting-a-repository/git-blame
https://git-scm.com/docs/git-blame

we pointed out that if an app is running elevated, PowerToys also needs to run elevated so it can process the Win key state and open the Shortcut Guide.

Lets blame these changes:
https://github.com/microsoft/PowerToys/blame/212ea2de30d69dbbfebf86c9d3604008d2272346/src/modules/shortcut_guide/shortcut_guide.cpp#L246-L262
It's a really cool feature for reviewing what, how, why and when code was added while helping with context and history when dealing with issues or bug reports. You should be using it more often if you're not already?

PowerToys also needs to run elevated

The blame shows PowerToys is using global hotkeys and lowlevel keyboard hooks which were primarily designed for accessibility applications. PowerToys should have added uiAccess="true" to the executable manifest so the binaries are automatically elevated and able to receive hotkeys and keyboard hooks for elevated applications.
There are basically three requirements for global hotkeys and keyboard hooks:

The binary is digitally signed.
The binary is installed into Program Files.
uiAccess="true"

If the executable contains true for uiaccess in the manifest then it'll be silently and automatically elevated to administrator without prompting the user or requiring the user manually elevate... You will need to add this otherwise that code ever won't work properly because PowerToys would have to be manually elevated every single time.

we need to find a case where the focused window is causing the problem.

It's basically a race condition between GetForegroundWindow and GetAncestor... The values of both functions are going to change almost immediately so I doubt you'll ever get a reliable reproduction case. I've mentioned the fixes already so eventually someone can resolve the issues.

@dmex
we don't want to do automatic elevation.
There are users that don't want to run it elevated and we respect that.

@dmex
we don't want to do automatic elevation.
There are users that don't want to run it elevated and we respect that.


Add "tags" support

We need to introduce a new column of type json into notes table, which will store all the tags for a note.
We should also support filter by tags eventually.

Up to take this one.  Assign when available.

[ ] Will add tags as csv to sqlite db and write a helper to encode and decode.

[ ] Will add a text field with autocomplete to enter tags in the diary page.

Hey @sameer1612 , can you post the detailed approach. If we store it as csv column, will we able to search it via tags?

Another appraoch would be create a new table with following columns: note_id and tag_name. This way we would be able to fetch all the tags which can be used for autocompletion, we can also search via tags this way


MuseV failing with comfyui now

I cannot get MuseV to work on my ComfyUI
Windows 11 OS
Python 3.11
Cuda 1.12
Folder downloaded to comfyui\models\diffusers\TMElyralab\MuseV
Poetry installed
Requirements.txt installed
Help?!?
(Thanks)

Check your log, it's possible to miss ffmpeg/diffusers/xformers


Handle gracefully invalid base64 text

Codecov Report
Patch coverage: 81.81% and project coverage change: -0.46 :warning:

Comparison is base (b89b3e0) 92.35% compared to head (3525246) 91.90%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   92.35%   91.90%   -0.46%
==========================================
  Files           5        5
  Lines         314      321       +7
==========================================
+ Hits          290      295       +5
- Misses         24       26       +2

Impacted Files
Coverage Δ

src/fast_scram_parse_rules.erl
96.62% <81.81%> (-2.16%)
:arrow_down:

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


Rename LiteralConvertible conformances to ExpressibleByLiteral

In accordance with SE-0115 and apple/swift#3474.

LGTM, feel free to merge when ready.

@swift-ci please test and merge.

Test failure is irrelevant.


Create FAQ Page in wiki

This is a sloppy issue I've began to put together but I will be adding to this as we figure out what should go into the FAQ page. All the suggestions below are from GeniusVentures.
Current questions to address:

`When converted to symbol the rectangles are hidden, but they are probably still exported and used to figure out that all the text, images, etc are in that rectangle, but that rectangle is not in the the bounds of symbol, so it doesn't use column layout...

This shold be documented in in a FAQ`

flutter note, on the kerning feature I just added, they add an extra spacing after the last character.  Kind of sucks, but we could hack around it by changing the width or just put that in our FAQ also.

one more for the FAQ, with kerning on Flutter adds kerning after the last character.  It's supposed to be only between characters, so if your rect is exactly the font rectangle with kerning it will wrap the last letter sometimes.

@siliconivan I am a beginner. Please assign this issue to me.


dev-guide: Fix minor typos

Fix minor typos in the development guide.
Fixes #53
Signed-off-by: Nitesh Konkar<EMAIL_ADDRESS>

@klynnrif - heads up - a small one for you!

Thanks @nitkon!
lgtm.


[Question] [core-http/search-sdk] How can I specify max sockets or provide my own HTTP agent to an Azure SearchClient?

Is there a way to pass in our own http.Agent or to specify maxSockets when creating an Azure SearchClient? I'm aiming to allow the use of both node-fetch and Azure Search SDK in an Azure Function to make requests to <mysearch>.search.windows.net, but need to ensure I don't run into port exhaustion (max 128 connections per host/port combination).
I started this question here and @xirzec kindly offered some tips, but I haven't managed to piece it all together yet. The suggestion there was:

I think you could do this by creating a custom PipelinePolicy that sets the agent property on each PipelineRequest made by the Search SDK when performing an operation. To have the SDK use your custom policy, you can pass it via the additionalPolicies client option.

I haven't seen where I can pass the additionalPolicies option for the search client.
The constructor for SearchClient allows passing SearchClientOptions (which is a small extension of PipelineOptions). Of these options, I take advantage of keepAliveOptions, but I haven't figured out how to provide a customized HttpClient. I can construct an instance of NodeFetchHttpClient, but it doesn't appear configurable. Any hints would be appreciated.
const client = new SearchClient<MyEntity>(endpoint, index, new AzureKeyCredential(apiKey), {
  keepAliveOptions: { enable: true },
  httpClient: /* is there a way to configure an HttpClient? */
});

In case it's useful, I drafted a PR of what adding max sockets support would look like: https://github.com/Azure/azure-sdk-for-js/pull/25523 .

Oh I see the issue here, the problem is that the GA version of @azure/search-documents still depends on @azure/core-http, a soon to be deprecated package.
@dgetu is there a plan to release a new beta or GA of search anytime soon? It looks like the last beta pre-dates the most recent GA and I'm not sure what will be lost if we suggest @mdmower switch over to it?
@mdmower if you wanted to at least test out the new pipeline functionality you could try pulling down the latest dev tag of the package to get what is in main and see the policy pipeline options I mentioned.

@xirzec The most recent beta release is at feature parity with GA. The more recent 11.3.1 GA release was a hotfix for a bug that isn't present in the beta.
@mdmower I would give the latest beta a try, 11.3.0-beta.8. So long as you avoid using preview features, you shouldn't have too much trouble moving back to GA as soon as we're able to migrate away from @azure/core-http.

Thanks @dgetu. I took a peek at the beta and the capabilities of additionalPolicies in SearchClient construction. From what I can see, it's possible to affect the request and response with a PipelinePolicy, but not to tinker with the https.Agent (Node.js). I'd have to submit a PR. Is that right? For example, the disableKeepAlivePolicy isn't sufficient as a standalone policy; explicit handling had to be added to getOrCreateAgent() to support the feature. I just want to make sure I'm not missing anything before I work on a PR.

@mdmower You can modify in-flight PipelineRequests using a PipelinePolicy. For example,
`ts
import { PipelinePolicy } from "@azure/core-rest-pipeline";
import { SearchClient } from "@azure/search-documents";
import https from "https";
const agentPolicy: PipelinePolicy = {
name: "agentPolicy",
async sendRequest(request, next) {
request.agent = new https.Agent({
/**/
});
return next(request);
},
};
const client = new SearchClient("endpoint", "indexName", "credential" as any, {
additionalPolicies: [{ policy: agentPolicy, position: "perCall" }],
});
`
Let me know if there's anything I can clear up for you.

Aha! Thank you for the example @dgetu! I see where it's utilized here. It's working as expected with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I look forward to the GA release. Again, thanks so much for your help @xirzec and @dgetu.

Aha! Thank you for the example @dgetu! I see where it's utilized here. It's working as expected with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I look forward to the GA release. Again, thanks so much for your help @xirzec and @dgetu.

As of 11.3.2, you should be able to use the same technique with the stable version of @azure/search-documents.


Flush data from Vuex only when needed instead of once per second

I created this issue to track a suggestion from our performance notes.

Neil has added a web worker to change the way the counts and schema resource types are emitted from the sockets (and thus hitting the store). We should look at extending that to other types

Should be addressed based on @Sean-McQ's web-worker work in #5997 rancher/dashboard#5997

Closed as duplicate.


Governance translation

All functionality implemented.
Missing test for methods with array input waiting for a solution in ScoreClient.
Missing tests for balance setup, to be decided if we want to keep them
Using temporary interfaces, to be changes to real once as soon as they are merged to main
Introduce vote action verification.
Currently also pushed local version of javee-unittest since a change is needed to test vote action verification

https://github.com/balancednetwork/balanced-contracts/blob/bb59d33079c619638382c510f18dea6f8b0f4c47/core_contracts/governance/governance.py#L87-L97
set_zero_hour_dev method is missing in java contract

https://github.com/balancednetwork/balanced-java-contracts/blob/78327028d10c2ca35bd2a65864b6eee3a66deab9/core-contracts/Governance/src/main/java/network/balanced/score/core/governance/GovernanceImpl.java#L199-L227
The actions parameter must be optional with default value "[]".
https://github.com/balancednetwork/balanced-contracts/blob/bb59d33079c619638382c510f18dea6f8b0f4c47/core_contracts/governance/governance.py#L222-L265


BZ 1174894 Process instance list sorting ends with an exception

I find this issue reason is SimpleTable set default datastorename use caption.
So i re-set   datastorename after listGrid.addColumns(columnMetas);
Code in here
https://github.com/uberfire/uberfire-extensions/blob/master/uberfire-widgets/uberfire-widgets-commons/src/main/java/org/uberfire/ext/widgets/common/client/tables/SimpleTable.java#L380
This is workaround way, best way should be remove default datastorename.
here:https://github.com/uberfire/uberfire-extensions/pull/8
But, i do not know is it reasonable .
Please check it.

already merged in master


@charset declarations causing problems in webkit browsers

Minify version: 2.1.4

What steps will reproduce the problem?
1. place combined css call in header

Expected output: that it doesn't break the design across all browsers

Actual output:  The design breaks in chrome and safari (webkit browsers)

The CSS is completely valid

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Jul 2010 at 8:35

I can also confirm that removing the @charset fixed it on mine too.. Now to
find why combined  & minified javascript doesn't work!

Original comment by<EMAIL_ADDRESS>on 9 Sep 2010 at 11:00

I had solved the same problem just in this moment. The problem was that one
stylesheet contained @media and @charset tags.

Original comment by<EMAIL_ADDRESS>on 3 Aug 2010 at 7:14

I can't help without knowing what CSS gets mangled.

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 1:41

Changed state: NeedInfo

Original comment by<EMAIL_ADDRESS>on 19 Sep 2010 at 4:48

Changed title: @charset declarations causing problems in webkit browsers

r423 strips all @charsets in Minify_CSS

Original comment by<EMAIL_ADDRESS>on 30 Sep 2010 at 4:35

Changed state: Fixed
Added labels: Release-2.1.3

Minify shouldn't be in the business of managing charsets, hence I'm inclined to
have it strip all @charset declarations and assume the user is competent enough
to manage getting all files in UTF-8 or another encoding of their choice. If
not UTF-8, they can then change the charset header Minify sends on all requests:

$min_serveOptions['contentTypeCharset'] = 'myEncoding';

Original comment by<EMAIL_ADDRESS>on 9 Sep 2010 at 3:52


Alternative locations for configuration file

We are using serve as our local dev server for three.js. After upgrading to the latest version, I've realized we need cleanUrls set to false so our documentation still works with serve. The problem is now we don't want to pollute the root directory of the repo with even more files (see https://github.com/mrdoob/three.js/pull/14210). Is there a different way to pass in the mentioned configuration?

+1, I use serve on a number of projects and cleanUrls=true seems like a very unexpected default setting to me. Adding another config file to the root of my projects would be nice to avoid. Some non-mutually exclusive alternatives:

Default to false as before
Allow configuration through a serve property in package.json
Allow setting the config file through a CLI parameter, like serve --config scripts/serve.json so developers can keep the top-level project clean

I would be happy to accept a PR for --config <path>... 😊 👍

Hi everyone, here's my naive attempt in implementing the --config flag.

The current implementation just accepts a raw String without any form of checks in place.

Running serve -c script/myServeConfig.json would work
Should we be enforcing a rule that serve's configuration can only be contained in an external file named serve.json? (other than now.json or package.json)

Should I be throwing an error if a file at <CUSTOM-PATH>/serve.json does not exists?

In this case mentioned above, currently serve will ignore it & attempt to parse other possible places containing the configurations in the following order: serve.json --> now.json --> package.json.

Would love to hear feedback from you guys 😅


Ensure we wait for the promise to succeed.

Motivation:
If we don't wait for the promise to succeed it may not be succeded
before we tear the loop down. This causes precondition failures.
Modifications:
We now always wait for the promise to succeed.
Result:
My tests will run.
Resolves #199

@Lukasa oh my... my fault. Merge once green


Release the GVL

It'd be useful to release the GVL while hashing to allow parallelism when using threaded applications.
It takes a bit of refactoring to get it done in Ruby though.
What do you think? Is it worthwhile doing?

@thedrow sorry but what's GVL? Or you mean GIL?

In Ruby the GIL is called the Global VM Lock.

@thedrow oh, I see. If it's something you need in your app and it won't break anything for existing users I think it's worthwhile doing.

I'll close it for now. @thedrow feel free to ping me if you need anything!


Update nuspec information per guidance

@heaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Build vswhere 1.0.68+g06d613db1c completed (commit https://github.com/Microsoft/vswhere/commit/06d613db1c by @heaths)


Actions.refresh renderRightButton has no effect

Version
Tell us which versions you are using:

react-native-router-flux v4.0.1 (v3 is not supported)
react-native v0.56.0

Expected behaviour
The navbar refreshes with the new right button component
Actual behaviour
No effect
Steps to reproduce
For non-obvious bugs, please fork this component, modify Example project to reproduce your issue and include link here.
  static onEnter = () => {
    const { wrappedInstance } = Actions.refs.profile

    // this works
    Actions.refresh({
      rightTitle: "Logout",
      onRight: wrappedInstance.onLogout
    })

   // this has no effect
   Actions.refresh({
      renderRightButton: () => <MoreButton onPress={() => alert('hi')} />
   })
  }

Here is how I worked around it
        <Scene
          back
          key={routes.PROFILE}
          component={ProfileScene}
          renderRightButton={
            <MoreButton
              onPress={() => Actions.refs.profile.wrappedInstance.onMorePress()}
            />
          }
          title="Profile"
        />

I made an example code to show you what i mean go here: https://github.com/pierol/rnrf-example it's an EXPO example try to open it in a ios simulator. Make login inside the app with these credentials: pierol and pw: giulia09. You have to swtch the version of react-native-router-flux from 4.0.0.beta.27 and 4.0.1 to see the two different behaviours. There is a console log in the DayList componenetWillMount method this component is part of the first Scene. After the Login is successfully i will make an Actions.choorHour() call to return to the first Scene, the one with the Daylist component. Only using 4.0.0.beta.27 will reload the component DayList when i return to the main scene from the Login form.

Thank you for the update about the workaround. This seems to be sort of a duplicate of #3188. I'll keep it marked as bug and work on both in the same PR.

@jamesholcomb It looks like a hack and probably just not supported by React Navigation 2.x (its SET_PARAMS action). Instead you should define one renderRightButton that changes its content depending from navigation state. Actions.refresh should be used just to update navigational state (i.e. pass values not functions)

Actions.refresh should be used just to update navigational state (i.e. pass values not functions)

I am attempting to upgrade a 3.X project to 4.0.  Actions.refresh is used throughout the app to wire up navbar buttons to their scene handlers based on data/conditions that are determined within the scene.
The example does not seem to demonstrate how to go about this and the migration guide does not mention it.
Can you elaborate or point to an example?

For react-navigation version 3.X use following example
class Companies extends Component {

    componentDidMount(){
        this.props.navigation.setParams({
          right: () => (
              <MaterialIcons
                name={'mail'}
                size={48}
                color={'black'} />
          )
        })
     }

    render(){
      ...
    }
}

For react-navigation version 4.X use following example
class Companies extends Component {

    componentDidMount(){
        this.props.navigation.setParams({
          headerRight: () => (
              <MaterialIcons
                name={'mail'}
                size={48}
                color={'black'} />
          )
        })
     }

    render(){
      ...
    }
}


Add optional containerProps, mainProps and drawerProps props

This would be helpful to allow a great amount of customization. Would also help a lot in making the components accessible (the user can decide which component to have which accessibility prop etc).

@rt2zz Hi, would you be able to look into this PR? This is a non-breaking change.

@mmazzarolo Would you be able to look at this?

@SuhairZain did you ping me by mistake? 🤔

@mmazzarolo My bad. Didn't mean to tag you. Sorry. 😄


Fixing TabSet.js error when expected elements are not available

Updating TabSet.js to open tabs correctly on page load when a hash is added to URL within the CMS.
This was done to fix a JS error found when clicking the "Manage roles" link inside the "Roles" tab when viewing a Group within the CMS. The error was:
Uncaught TypeError: Cannot read property 'indexOf' of undefined
It looks like the openTabFromURL method was looping over a JS selector that has been updated or is only used in some cases so when it tries to find the href property it can't (and hence calling indexOf on this.href produces the above error).
The suggested fix changes the following line as the ajaxComplete event is not being fired so needed to use another appropriate event instead:
$(window).one('ajaxComplete', function () { ... });

@timkung Did you test this in IE8? The ajax bits were in there to ensure a tab remains active after saving (which replaces the entire HTML including tabs). SecurityAdmin and ModelAdmin shouldn't be your only test cases there, since the "top right" tabs are handled differently to the tabs below (part of the <form>. Can you test with a Page view that has multiple tabs?

@chillu Have just tested this in IE8 and it seems to work as expected. And have also just tested a Page that has more than one tab, e.g. a "Root.Test" tab where #Root_Test appended to the URL opens the "Test" tab on page load.

Yep, confirmed this works locally as well (by installing the https://github.com/silverstripe/silverstripe-frameworktest/ module which has some preconfigured tabs for Page)


Laravel 5 compatibility

I updated the base classes to be compatible with Laravel 5.
However, the Guard dependency injection in __construct (see for example Middleware\Authenticate) doesn't work yet.
For that, the Serviceprovider needs to register this singleton correctly:
$this->app->singleton('auth.driver', function($app)
{
    return $app['auth']->driver();
});

Hi Ramon how you resolved problem of "BindingResolutionException in Container.php line 785: Target [Illuminate\Contracts\Auth\UserProvider] is not instantiable." It appears in the constructor of the Guard class (first parameter) [ __construct(UserProvider $provider...)]?

@ldynia check my post here
https://laracasts.com/discuss/channels/general-discussion/multiple-authentication-in-laravel-5-ldap-and-default-auth?page=1#reply-44260
you can use the Auth Facase instead of the Guard injection, that works for now. I haven't had time to look at it in depth yet.

@sboo I have promble installing your fork. Could you provide me the line of code for composer installation ? I tried this but indicates that repository doesn't exist. Thanks :)
	"repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/sboo/multiauth"
        }
    ],
	"require": {
		"sboo/multiauth" : "master",

The package is still called ""ollieread/multiauth".
Add this instead:
"ollieread/multiauth" : "dev-master"
I would advise you not to use this in a production right now, or only with caution. I will work on it shorty and fix a few more things

Thanks and it sounds awesome. I will watch it then :)

To use my fork for now, add the following:
"repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/sboo/multiauth"
        }
    ],
    "require": {
        "ollieread/multiauth" : "dev-master"
   }

This is compatible with L5

Thanks! Finally I can install it :) I will play with it, and I will update you If I will find something suspicious. Cheers!

Does this actually provide all of the same functionality that the 4.1 uses and that the default 5 Auth does?

There is no functional difference with your version at all, I just made it 5.0 compatible. Even 5.1, except that the default auth traits can't be used.

Did you create replacement traits?

Will do.  I haven't had time to create the traits yet, I was away until just a few days ago. Will try and do it this week.


TypeError: POST when trying to download videos as registered user of Lynda.com

[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://www.lynda.com/Tkinter-tutorials/Entering-displaying-multiple-lines-Text-widget/163607/184111-4.html?autoplay=true', '-v', '-u', 'PRIVATE', '-p', 'PRIVATE']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.06.25
[debug] Python version 3.4.3 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[lynda] Logging in as<EMAIL_ADDRESS>Traceback (most recent call last):
File "C:\Python34\lib\runpy.py", line 170, in run_module_as_main
"main", mod_spec)
File "C:\Python34\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "D:\Desktop\youtube-dl.exe_main.py", line 9, in
File "C:\Python34\lib\site-packages\youtube_dl_init.py", line 406, in main_real_main(argv)
File "C:\Python34\lib\site-packages\youtube_dl_init_.py", line 396, in _real_main
retcode = ydl.download(all_urls)
File "C:\Python34\lib\site-packages\youtube_dl\YoutubeDL.py", line 1501, in download
url, force_generic_extractor=self.params.get('force_generic_extractor', False))
File "C:\Python34\lib\site-packages\youtube_dl\YoutubeDL.py", line 654, in extract_info
ie_result = ie.extract(url)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 272, in extract
self.initialize()
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 266, in initialize
self._real_initialize()
File "C:\Python34\lib\site-packages\youtube_dl\extractor\lynda.py", line 25, in _real_initialize
self._login()
File "C:\Python34\lib\site-packages\youtube_dl\extractor\lynda.py", line 41, in _login
request, None, 'Logging in as %s' % username)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 424, in _download_webpage
res = self._download_webpage_handle(url_or_request, video_id, note, errnote, fatal, encoding=encoding)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 331, in _download_webpage_handle
urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 312, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "C:\Python34\lib\site-packages\youtube_dl\YoutubeDL.py", line 1729, in urlopen
return self._opener.open(req, timeout=self.socket_timeout)
File "C:\Python34\lib\urllib\request.py", line 461, in open
req = meth(req)
File "C:\Python34\lib\urllib\request.py", line 1112, in do_request
raise TypeError(msg)
TypeError: POST data should be bytes or an iterable of bytes. It cannot be of type str.

I managed to solve the problem by using the youtube-dl GUI at youtube-dlg. I'm closing the issue.


rac-5400 - Failed at case SELPollerAlertTests

Added logic skip test of AMQP server not active

@johren @jimturnquist @BrentHiggins

BUILD RackHD #482 : FAILURE


Fix key fuzzer for more coverage

Summary

Fix the logic of KeysFuzzer to get more coverage.
Release Note

Documentation

@AdamKorcz

replaced by #626


Specify capacity at construction of Vector

If efficiency is one of the main points of this extension, and the main point of the Vector class is for a value’s index to be a direct mapping to its index in the buffer, wouldn't having to increase the capacity at some point after construction be a potentially expensive operation? I guess if you immediately call allocate() after constructing it, there is nothing that needs to be copied. But still, why waste the initial allocation operation if it just has to be done over again, potentially even from scratch if there is not enough memory next to the initial allocation? Construction time seems like the most useful time to allow the programmer to specify the capacity and allocate a custom amount.
It could be a second parameter passed into the constructor. A PHP programmer could explicitly pass $values as null if they want, or pass something in there, and then a second parameter would be $capacity.
This would be particularly helpful when you already know how large the capacity will end up being, when you don't actually need it to grow or know that part of the code well enough to know it won't grow. It is also possible you know exactly how large it needs to be initially to store all the values you are about to put in it, and it is not super likely to grow after that. In that case, you may want to allocate it to that size, or something a little larger to give it a little room to grow.

Alternatively, maybe I should just use SplFixedArray for that case? I was sold on the idea that the Spl classes are not well designed and kind of pointless, but maybe the SplFixedArray is the exception?

I've been drafting an Allocation type that is immutable and typed, to simulate a fixed array. I'll expedite that a bit, I have some time now.

Awesome. I still think it would be helpful for the Vector constructor to allow the programmer to specify a starting capacity. There are also situations where you don't know the end capacity, but you know it is likely to be much higher than 8. I guess I just feel if it is possible to set the capacity in a custom way, this is an incomplete feature without it also being a constructor option.
I'm curious what you mean about the Allocation being being "typed". At work we rely heavily on Psalm as a static analysis tool for the types in our code, and as a way to add a much more sophisticated type system to PHP (at static analysis time anyway). I was thinking of contribution to the polyfill project for this extension by adding Psalm typing to the docblocks. Basically it would give the programmer the option to declare in code comments what types a particular data structure object will contain, and then Psalm can verify those types were honored/used correctly. This would go a long way to prevent people that rely on Psalm from having to either wrap these classes or create a Psalm plugin to get better typing.

That is an excellent idea, I will work on that if you don't beat me to it. By typed I was thinking you pass a type enum in the constructor that defaults to any/mixed, but can be any of the internal types. If the type is primitive we don't need to allocate a 16 byte reference-counted container for every value

I see, only allocating the memory needed for a particular type sounds great.

It turns out what I said about adding types to the polyfill may be unnecessary. I'm not sure what I'm doing differently, but Psalm is now allowing me to add types to Vector objects I create, and enforcing those types. For example if Psalm were to analyze the code, it would report an issue when trying to push something that is not an int:
/**
 * @var Vector<int>
 */
$vec = new Vector();
$vec->push('hello');

I'm not sure why or now this works, but it does..

I'm going to close this for now. With allocations incoming I'm considering completely removing allocate/capacity functionality - doesn't seem worth the added complexity at the interface level. Something to consider for v2.


Docs/diagram

Added activity diagram into README.md to briefly explain the function
Removed build-wheels-ad-hoc.yml (function has been replaced by build-wheels-defined.yml)

Docs (RADME.md) preview

Thank you @dobairoland for suggestions and language improvements. I have edited these and also removed .png file.
README.md preview

Thank you for the corrections!


fix(hook): avoid failure on task retry in before handler

avoid following error when a tast is retried in before handler :
An error was thrown in your plugins file while executing the handler for the before:spec event.

The error we received was:

Error: Unexpected state in beforeSpecHandler: before-spec (this might be a bug, please report at https://github.com/badeball/cypress-cucumber-preprocessor)
...

You need to provide a reproducible example that illustrates the behaviour you’re trying to fix. And nothing gets merged without a corresponding change to the tests.


[Snyk] Security upgrade jquery from 2.2.4 to 3.5.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

701/1000  Why? Mature exploit, Has a fix available, CVSS 6.3
Cross-site Scripting (XSS) SNYK-JS-JQUERY-565129
Yes
Mature

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

Already fixed here https://github.com/Expensify/k2-extension/pull/143.


Upload directly to S3 the exported data from Firebase

Time Estimate:
2 hours

Time to complete:
I felt that I  had a good grasp on how to implement this update, so it only took a couple of hours with testing. Since I had some free cycles, I did some research into how S3 works on the backend and shared some resources with other students in the class. I also had time to focus on code review, which is pretty involved at this point.
Outcome/Acceptance Criteria:
The output of the script is written directly to a dedicated S3 bucket, rather than saved locally. See documentation section for details.
Documentation:
Update to write directly to S3 from floop data collection script. I added some functionality so that the input steps can be bypassed by passing parameters to the script directly from the terminal like so:
python3 floop_data_collection.py <path to your floop firebase json credential file> <aws_access_key_id> <aws_secret_access_key> which will enable automation of this script on a remote server.
For example, the path to my floop firebase json credential file is: /Users/daleberg/Downloads/floop-firebase-development-firebase-adminsdk-g8bpy-8c72987529.json
You can confirm that the file was uploaded by checking the floop-dataset bucket, and looking for the file floopData<$time_and_date_uploaded>.json
Pull Request
This ticket was completed by the work submitted in this PR: https://github.com/North-Seattle-College/ad440-winter2022-tuesday-repo/pull/61


Documentation improvements

This pull request performs the following improvements:

Remove dead code.
Improve slightly some parts of the code that were forgotten in the previous pull requests.
Add as much documentation as possible.

Most of the pull request consists in documentation improvements.

I can take a look

The only commit that adds more than just comments and docstrings is 168842a.


Automatically ignore anyone with +B (bot)

The +B flag is used to designate a user as a bot.  Bots should be automatically ignored to prevent loops.

Sorry, I meant to add this to lita-irc...


Improve error message logging when error response contain json

We currently treat all http error messages as strings when storing them. Several of the responses are actually json. This results in strings with escaped json. This doesn't have the best readability.
I suggest that we check the response for json before passing on the response. If it's json, we pass a long the object instead. (Might want to append i.e. the status code before passing it along.)
Instead of this:
{
  "id": "wpm78z4e45",
  "title": "TEST-owner-anders-dev-export",
  "start": "2023-03-08T15:29:59.092614+01:00",
  "end": "2023-03-08T15:30:01.080508+01:00",
  "lastError": "Received http sink error (401): {\"message\":\"code=400, message=missing or malformed jwt\"}\n"
}

Do something like this:
{
  "id": "wpm78z4e45",
  "title": "TEST-owner-anders-dev-export",
  "start": "2023-03-08T15:29:59.092614+01:00",
  "end": "2023-03-08T15:30:01.080508+01:00",
  "lastError": {
    "body": {
      "message": "code=400, message=missing or malformed jwt"
    },
    "title": "Received http sink error (401)",
    "statusCode": 401
  }
}

If the error doesn't contain json, it should use the old behavior.

I guess an option would be to do this on the client side. I'm not sure what we prefer.


Missing files

What happened to xs-security.json? Do we still need it?

Seems like the download & installation instructions are outdated. Pls update them according to the new code. Thanks


Updates to accommodate FaceID and cancellable locks

This pull request encompasses two issues.

With the release of iOS 11.0 Apple released FaceID. Fortunately all of the calls are identical to using TouchId so there weren't any changes needed there. However, the current property names and text on the view controller do not reflect that. They were changed to a more generic "Biometric Authentication" to better reflect functionality. A potential improvement for this is that as of iOS 11.0 the device can differentiate between which functionality the device supports and we could change the text of the button on the controller to reflect this.

Currently the only cancellable lock type is .remove which is done via the declaration of the type. This really doesn't make any sense. We should allow users to decide which locks are and are not cancellable. I have added a property to the PasscodeLockConfigurationType class which the view controller will key off of to show/hide the cancel button. Personally I think this makes more sense as the cancellation seems more like a configuration thing rather than a lock type thing.

@mjmarathon nice work man, I will try to review it locally soon. I don't see any changes to the strings file and also the image for TouchID (fingerprint) in the UI remains the same, do you think it will be a good idea to change those too? Can we somehow check if the supported method is TouchID or FaceID?

@ziogaschr In the order of things you asked.

Looking at the strings file I can see that we're using "PasscodeLockTouchIDReason" as a default when we're presenting the TouchID prompt. FaceID will use this in the same way so no change is needed other than to make the naming scheme more accurate.
I am not seeing which images you're referring to, if you mean on the prompt for face/touchid itself those are handled by the OS. It that's not what you're talking about lemme know where these images are referenced and i'll take a look.
FaceID is only supported by iOS 11 and up. With that OS release we can indeed check which method is supported. The way apple handled this code makes it seem like a device is either going to have face OR touch and not both.


Events Tab on Team Pages

Ability to view all events that a team has been to since starting to compete at FIRST

Added as of 3afdbfd50e860e1491fb7ec3b5315707d54a9cf9.


fix: window nuxt build by using cross-spawn

node spawnSync have issues on windows and building fails with it with ENOENT error

Codecov Report

Merging #3833 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #3833   +/-   ##
=======================================
  Coverage   97.84%   97.84%
=======================================
  Files          18       18
  Lines        1162     1162
  Branches      317      317
=======================================
  Hits         1137     1137
  Misses         24       24
  Partials        1        1

Continue to review full report at Codecov.

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


A black bar appears after clicking on a contact in the F&F list.

On clicking on the contact of F&F list, a black color strip appears in the right and left side.

Verified this issue on dev app v1.7.9(315)


Please compile for darwin arm64

yea, I know it's really annoying.
Mac m1 and all that, and perhaps I could build it myself (have not really read)
it was a tough cookie to get terraform to work and providers had to be built manually for darwin_arm64
but there we go, new architecture and all that.
Would it be possible for you to release an arm64 darwin build? and perhaps even an arm64 Linux/windows build, cos why not eh?

NVM.
Homebrew already installed an arm64 version for me, aren't them guys nice? :)

Thanks for the heads-up @ameeno! Going to close this issue as it's a duplicate of https://github.com/infracost/infracost/issues/428, now that Go 1.16 is released I've added it to the Todo list. Homebrew folks are indeed very nice!


feat(deps): update deps matching '@opentelemetry/*'

Description
This PR updates all OpenTelemetry dependencies from the core repo to the latest versions.
Partially created using workflow from #2536 (it failed on PR creation, so it's me not @opentelemetrybot opening the PR 😞 )

Note to self: I guess this summary of updates from the failed workflow (https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/12164311207/job/33926373050) is relevant:
Summary of changes (possible commit message):
--
chore(deps): update deps matching "@opentelemetry/*"

    0.55.0 -> 0.56.0 @opentelemetry/api-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-fetch (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-grpc (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-http (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-xml-http-request (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/otlp-transformer (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-node (range-bump)
    1.27.0 -> 1.28.0 @opentelemetry/semantic-conventions
    1.28.0 -> 1.29.0 @opentelemetry/context-async-hooks
    1.28.0 -> 1.29.0 @opentelemetry/context-zone-peer-dep
    1.28.0 -> 1.29.0 @opentelemetry/core
    1.28.0 -> 1.29.0 @opentelemetry/exporter-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/propagator-b3
    1.28.0 -> 1.29.0 @opentelemetry/propagator-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/resources
    1.28.0 -> 1.29.0 @opentelemetry/sdk-metrics
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-base
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-node
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-web
--

I also opened #2583 to fix the workflow so that we can actually use it all the way to PR creation next time :)

Hmm, TAV for tedious on Node.js v22 failed - maybe flaky? I'll re-run to see if it passes on the second try.

all green now :)

I just realized that I cannot rebase this because I opened it from @opentelemetrybot's branch - so I'm running the workflow again.


Port to Python 3.

Nice pull request, however, this doesn't keep backwards compatibility, so it should be a different package and / or fork ): (Also, this appears to be unmaintained so it's probably not worth spending time writing pull requests)


Use JSON5 instead of CJSON to parse .babelrc.

Fixes #362

That would have also been fixed by #330 ;)

Yeah. I just wanted to get only this.


Can't generate binary. Suggest changing linker script or ELF to merge sections.

Some cryptic message from the linker, keeps me flabbergasted for now.  Seen this before ?
Feels like ESP8266 all over again ,running out of memory ?
......
CXX workspace/esp-ebos/build/main/Config.o
AR workspace/esp-ebos/build/main/libmain.a
LD workspace/esp-ebos/build/esp-ebos.elf
esptool.py v2.1
A fatal error occurred: Segment loaded at 0x401389f4 lands in same 64KB flash mapping as segment loaded at 0x401389d4. Can't generate binary. Suggest changing linker script or ELF to merge sections.
/home/lieven/esp/esp-idf/components/esptool_py/Makefile.projbuild:51: recipe for target '/home/lieven/workspace/esp-ebos/build/esp-ebos.bin' failed
make: *** [/home/lieven/workspace/esp-ebos/build/esp-ebos.bin] Error 2

Ok found a trail : https://esp32.com/viewtopic.php?f=13&t=435 , old esp8266 attributes lingering in the code
I had something like =
 #define ICACHE_RAM_ATTR  __attribute__((section(".iram.text"))) ICACHE_RAM_ATTR void dwt_isr(void) // Assume interrupt can supply context


Simplify Code

Currently, Fuzzy-Finder uses THREE different libraries to make filtering. One of them use Native Bindings, which are not context-aware.
So, we either must decide on using a single one, or migrate to Zadeh (the move I actually want to work later).
I'm drafting this PR so this change does not get lost somewhere :)

@mauricioszabo closed this 22 minutes ago

Why?

Because Atom will never merge this PR - I actually opened in the wrong repo, it was meant to be opened here: https://github.com/pulsar-edit/fuzzy-finder/pull/1

Because Atom will never merge this PR - I actually opened in the wrong repo, it was meant to be opened here: pulsar-edit#1

Thank you for answer, completely understandable.


passing all kwargs in neid's read_data()

Sorry, I should have caught this earlier. Also might want to fix the "specific" typo in the about section

Eric,
Right now I have them pointed at a local fork so it isn't critical yet
-Christian
On Thu, Apr 13, 2023, 16:43 Eric Ford @.***> wrote:

Merged #59 https://github.com/RvSpectML/EchelleInstruments.jl/pull/59
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/RvSpectML/EchelleInstruments.jl/pull/59#event-9001048087,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXW3ZG32YNEMJ2LG5EMKMLXBBQPPANCNFSM6AAAAAAW5R3GRM
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


Search is not trimming

Hey!
Let's say one of the columns has the words Free Ads.
If I search for fre, it will filter the row(s)
if I search for ad, it will filter the row(s)
However, if the user types a space after or before the above searches, it won't bring any results.
The source code seems to have some searchText.trim() in it, but it does not seem to help in those cases.
How can I trim what the user is typing?
Thanks in advance,
Sergio

Hi Sergio,
Do you have fixed this error?

Hi Sergio,
Do you have fixed this error?


Build apple_framework for device does not generate arm modules

I am a noob as far as ios as well as using bazel for building ios targets. I am trying to build an apple_framework target for ios.  I am using this demo repo https://github.com/vincephamgrab/demo-mixedobjcswift  (borrowed from vince's example for mixedobjcswift).
When I build it as follows
bazel build //src:MixedObjcSwift  --ios_multi_cpus=armv7,arm64,x86_64

I still only see the x86_64.swiftmodule  and do not see anything for arm.
Any help/suggestions is appreciated.
Thanks

You probably also need to pass --apple_platform_type=ios


Add update option for embeddings

I read on the readme file that if the changes happen to the database then new embeddings are required. However, the user should manually go and delete the old embeddings. Therefore I thought that adding an update paramter to the load_embeddings function can make the process more automatic.

The tests failed due to the sentence-transformers version conflicting with the huggingface-hub version (which is probably the latest since it isn't specified in the requirements file). The solution is to either upgrade the sentence-transformers package or downgrade the huggingface-hub package. I chose the first option and modified the requirements file. The tests run OK in my local machine.


VRF assignment

I wanted to ask how netbox-sync assigns VRF to ip address. We are encountering some problem and in settings we only found mentions about VRF in the comments for cluster_site_relation setting.
Can you shine some light please?

Hi @rizlas:
The VRF is "graded" from the prefix which matches the IP (search for possible_ip_vrf): https://github.com/bb-Ricardo/netbox-sync/blob/main/module/sources/common/source_base.py

Hi,
thank you for pointing me out. I'll have a look. Currently the assigned vrf seems to be wrong but "I need to go deeper" 😄
I'll continue here and not via email.

I think that the problem is here.
https://github.com/bb-Ricardo/netbox-sync/blob/main/module/sources/common/source_base.py#L198
Currently we are using the concept of stretched vlans, so the prefix has no site set cause the prefix is in more than one site.
Stretched vlans are implemented using vlan groups that enforce uniqueness (https://docs.netbox.dev/en/stable/models/ipam/vlangroup/).
Dunno how to proceed here 😕

Hi,
i think that the entire function return_longest_matching_prefix_for_ip needs a rework and I'll try to propose an alternative in the next days. Currently I've this questions in my mind:

Why not using native library ip_addresses to calculate an IP prefix
Why tenant is not considered
Why looking for the biggest/longest prefix

Hi @rizlas,
finally got around to add a warning regarding possible wrongly assigned VRF to an IP address.
can you test it out and see if it shows the conflicts?
Thank you

Traceback (most recent call last):
  File "/app/netbox-sync/./netbox-sync.py", line 146, in <module>
    main()
  File "/app/netbox-sync/./netbox-sync.py", line 111, in main
    source.apply()
  File "/app/netbox-sync/module/sources/vmware/connection.py", line 393, in apply
    view_details.get("view_handler")(obj)
  File "/app/netbox-sync/module/sources/vmware/connection.py", line 2383, in add_virtual_machine
    self.add_device_vm_to_inventory(NBVM, object_data=vm_data, vnic_data=nic_data,
  File "/app/netbox-sync/module/sources/vmware/connection.py", line 1121, in add_device_vm_to_inventory
    nic_object, ip_address_objects = self.add_update_interface(nic_object_dict.get(int_name), device_vm_object,
  File "/app/netbox-sync/module/sources/common/source_base.py", line 417, in add_update_interface
    log.warning(f"Possibly wrongly assigned VRF for {current_ip_nic.get_display_name()} IP "
AttributeError: 'NoneType' object has no attribute 'get_display_name'

You need another check for current_ip_nic

oh, damn it. sorry. I just pushed another commit which should fixes this.

Yeah working now. Thank you

Awesome. Can we close the issue or is still something missing?

Yes we can close it, thank you


How to return current node title

I want to return the current node title in the same way <stream-root-title> returns the title of the current tiddler, how can I do it?

@kebifurai inside a node, <> points to the node and not the story tiddler.
You can also access the immediate parent via {{!!parent}} and the storyTiddler (root node) via <>

sorry, I didn't use the code block in the question.

@kebifurai apologies, I've corrected the above comment.
Inside a node, <<currentTiddler>> will give you the node title.

Thank you for the fast reply!

Sorry to reopen but how do I recall the title of a node outside that node?
<<currentTiddler>> returns the title of the note where my macro is defined not the actual node, but <<stream-root-title>> works just fine, what am I missing...

Sorry to reopen but how do I recall the title of a node outside that node?

I don't understand what this means or what you are attempting to do.

<<currentTiddler>> returns the title of the note where my macro is defined not the actual node, but <<stream-root-title>> works just fine, what am I missing...

This seems related to your code and how it is being used within Streams, most likely you are re-assigning currentTiddler to a different value in your macro. I cannot offer further assistance without seeing your code.

I'm using Stream in combination with another plugin TiddlyRemember
TiddlyRemember also uses an ID field generated from the current time, the same way Stream does.
TiddlyRemember has the following macro:
$:/plugins/sobjornstad/TiddlyRemember/toolbar/remembercz

\define getMacro()
<<remembercz "$(clozeid)$"
	"Cloze">>
\end

<$set name=clozeid
      value=<<now "[UTC]YYYY0MM0DD0hh0mm0ss0XXX">> >
	<$action-sendmessage
		$message="tm-edit-text-operation"
		$param="replace-selection"
		text=<<getMacro>>
	/>
</$set>

I want to change the clozeid value to the already generated id of the node I'm editing.
I tried to use<<currentTiddler>> instead of <<now "[UTC]YYYY0MM0DD0hh0mm0ss0XXX">> but when I recall this toolbar with a shortcut, it returns $:/plugins/sobjornstad/TiddlyRemember/toolbar/remembercz instead of the node title.

This isn't an issue related to Streams, but rather to TiddlyRemember.
Trying to make TiddlyRemember use the currentTiddler variable in  a regular tiddler (not a Streams node) causes the same issue. Try using the targetTiddler variable set by the code that setups the edit toolbar.
If that doesn't help, I suggest posting on the google group and asking for assistance.


UI becomes unresponsive after suspending computer for a while

Description
After leaving the Deephaven UI open while the computer is suspended for a while, UI is unresponsive and server logs are flooded with No AuthenticationRequestHandler registered for type Bearer.
Steps to reproduce

Open the Web UI in the browser
Suspend/Sleep your computer for ~15 minutes
Wake up computer and look at Web UI

Expected results
3. Web UI should be working, or should have a popup asking to re-authenticate
Actual results
3. Web UI is not working, and there are errors in the logs. If left suspended long enough, it will be completely unresponsive. You may need to wait a few minutes before the "Authentication failed" message appears, prompting you to refresh (assuming the UI hasn't locked up completely).
Additional details and attachments
In the UI logs, there are a bunch of errors from the HeapUsage component:
[HeapUsage] Unable to get heap usage Error: Authentication details invalid
    at Object.onEnd (dh-internal.js:1:724077)
    at dh-internal.js:1:55831
    at dh-internal.js:1:42819
    at Array.forEach (<anonymous>)
    at e.rawOnEnd (dh-internal.js:1:42781)
    at e.onTransportEnd (dh-internal.js:1:42488)
    at Object.$onMessage (dh-core.js:20455:35)
    at MultiplexedWebsocketTransport$3methodref$onMessage$Type.handleEvent_2 [as handleEvent]

Right now this component just polls, we should probably be smarter about how we queue up the requests instead of just using setInterval, so we don't accidentally queue up a bunch of events; also should handle the error better.
On the server there are a bunch of logs repeated:
2023-08-04T14:40:51.529Z | qtp1871312485-127    |  INFO | i.d.s.s.SessionService    | No AuthenticationRequestHandler registered for type Bearer
2023-08-04T14:41:01.529Z | qtp1871312485-127    |  INFO | i.d.s.s.SessionService    | No AuthenticationRequestHandler registered for type Bearer
2023-08-04T14:41:07.072Z | qtp1871312485-127    |  INFO | i.d.s.s.SessionService    | No AuthenticationRequestHandler registered for type Bearer
2023-08-04T14:41:07.090Z | qtp1871312485-113    |  INFO | i.d.s.s.SessionService    | No AuthenticationRequestHandler registered for type Bearer

Not sure why that is getting triggered repeatedly, but takes a while for the EVENT_RECONNECT_AUTH_FAILED to fire.
Versions
Engine Version: 0.26.1
Web UI Version: 0.44.1
Java Version: 17.0.7
Barrage Version: 0.6.0

Reported in Community Slack originally: https://deephavencommunity.slack.com/archives/C036HT73T43/p1691154016732149


Remove Markdown dependency

the list of macro seems a bit arbitrary for users:
https://github.com/JuliaPluto/ExpressionExplorer.jl/blob/5766ad187fc2d812d21e0fdac4604eebacf2a77c/src/explore.jl#L1034-L1040
Maybe provide a way for configurations to provide with defaults:
ExpressionExplore.can_macroexpand(::AbstractExpressionExplorerConfiguration, e::Union{Expr,Symbol}) = false
ExpressionExplore.macroexpand_module(::AbstractExpressionExplorerConfiguration) = MacroExpandInHere

Maybe users (including Pluto) should first macroexpand what they want, and then call ExpressionExplorer on the result? Then EE never needs to expand, and we don't need the AbstractExpressionExplorerConfiguration anymore?


Dashboard: Define dashboard configuration / customization api

https://quip-amazon.com/k2nqA34fGnb2/Dashboard-v2-Register-Widget-Schema

https://quip-amazon.com/Bc1cAjvksc8S/Dashboard-Customization


Homepage: Change categories titlebox from round corners to pointed corners

Reason (Why?)
When looking at the four suggestions, the user should focus on the images or their text. Unfortunately, the black corners distract the user.
Solution (What?)
The white rectangles should not contain round corners, but pointed corners.
Not doing
/
Relation to other Issues
/
Acceptance criteria
The white rectangles contain pointed corners.

Done in https://github.com/hochschule-darmstadt/einander-helfen/pull/74


HGNC Genes

Is there a way to filter genes returned to only return ENSG's of HGNC genes in Ensembl? Maybe this should be added as a function.

@ahmdeen Do you know of a good Python library for mapping between Hugo gene names and Ensembl IDs?


Cherry pick over 9577d5b66150491770775c3e8f4389a2cdadc8e7

Cherry pick over https://github.com/jboss-fuse/camel-quarkus/commit/9577d5b66150491770775c3e8f4389a2cdadc8e7 from 2.2.0 product branch to 2.2.1 product branch, use log4j 2.17.0

@ppalaga Does this look okay?