Update Essence Pouch Tracker to v1.5.9 & Update repo URL

(Hopefully) Fixed a few issues since releasing Essence Pouch Tracker initially onto the plugin hub (https://github.com/Infinitay/essence-pouch-tracker/tree/bbc4c83fd2dee9935dd6d460185d345ebe686277 v1.4.0) as well as add a few new features
Fixes

Fixed the plugin being broken/uninitialized when using using it for the first time when the player is already logged in with essence already in their inventory

Refactored resetting config and plugin start up/initialization

Only manipulate pouches that are currently within the player's inventory and not every pouch regardless if it's in the inventory or not
Mitigate wrong pouch states by changing the behavior to not update the pouch's respective states if a property (stored/decay count) is unknown
Fix game stutters that would be caused when withdrawing large quantities of stackable items

Also optimized EssencePouches type retrieval

Fixed some cases in which pouches weren't being reflected as repaired due to missing dialog options

Fixed not being able to repair colossal pouches, and added some of Cordelia's wrong/missing repair options

Features

Add support for specialized equipment preventing pouch decay such as Runecrafting/Max capes and the Abyssal Lantern
Add a chat message when a user has a pouch with any unknown state in their inventory on a new login session (similar to Daily Tasks Plugin IIRC)
Add support for pouch state resetting when a user upgrades their pouch from a regular pouch to a colossal pouch or downgrades from a colossal pouch to a regular pouch

Other Changes

Renamed the plugin (Plugin Description) from Essence Pouch Tracking to Essence Pouch Tracker
Renamed the GH repo and updated the URL to reflect the changes: https://github.com/Infinitay/essence-pouch-tracker.git
Removed left-over debug statements that occurred on every MenuOptionClicked

[x] nit: line 19 of readme, remove [ before the "Install the plugin" text

[x] nit: line 29 of readme: update repo URL

[x] nit: EssencePouch.java line 49: the debug message will post the current current/new stored essence as the "previously" because you log it after you've set this.storedEssence

[x] nit: EssencePouch.java line 61: same as above

[x] code style: EssencePouchTrackingOverlay.java line 34: The if-statement is needlessly complicated. If I understand it correctly, it's if (a == null || !(b || c)) while it could be if (a == null || !b || !c)

[ ] code style: For the RepairDialog.java messages, you could use Text.removeTags to clean up the tags, that way your list of dialog options would look a bit cleaner
---
I'm not using Text#removeTags to keep it simple and avoid doing more calls with text formatting when a simple comparison would work. Furthermore, I tested it out and noticed that it would have made the code or rather the text look worse (imo). For example, Fine. A simple transfiguration spell should resolve things<br>for you. => Fine. A simple transfiguration spell should resolve thingsfor you.

[x] nit: EssencePouchTrackingPlugin.java line 181: a == 1 ? true : false is equivalent to a == 1

[x] nit: EssencePouchTrackingPlugin.java line 218: same as above


push_next being safe to call is unsound

This code triggers UB using only safe code:
use ash::vk;

let create_info = vk::DeviceCreateInfo::default();
let mut features11 = vk::PhysicalDeviceVulkan11Features {
    p_next: 1 as _,
    ..vk::PhysicalDeviceVulkan11Features::default()
};
let create_info = create_info.push_next(&mut features11);

Personally I would prefer having a function that doesn't dereference raw pointers at all, because at least we I don't think need this functionality and it would alleviate having the unsafe block when calling push_next.

Yeah that is definitely a problem. Any solution involves a breaking change, and I'm not sure about yanking or releasing a breaking followup soon because of this. It's been there for quite some time.
For solving this, I see two options: mark push_next() as unsafe, and add a push_next_one() version that is safe, but asserts that p_next equals null() to not risk breaking an existing chain.

We should mark push_next as unsafe. Perhaps we can justify doing so in a patch release:

it's fixing a serious bug
it's rare to be building Vulkan structs outside of an unsafe block anyway, so cases of downstream breakage should be few.

Opened a draft at https://github.com/ash-rs/ash/pull/909 following my suggestion from https://github.com/ash-rs/ash/issues/905#issuecomment-2052062228, while keeping implementation discussions here.
I've also considered not marking push_next() as unsafe, but stripping this chain-walk pointer cast out of it (and obviously inheriting the assert!(next.p_next.is_none()) to ensure the caller is not loosing any information). I'm not entirely certain if such behavioural changes fall behind "semver compatible" (it surely is walking the edge) because the signature would no longer be changing and allow us to push this as a patch release.
After all I've never actually seen anyone building up pointer chains like this, as @Rua brought up in #906 we don't even provide push_next() functions on non-root structs making it even less likely (though we can probably come up with one or two examples where this is a possibility).

I concur that the use case for pushing a whole chain at once is unclear, and a simpler method is appealing. I'm not even sure we should it's worth preserving the existing logic under another name. Still, I'm wary because (semver break or not) introducing a dynamic assert is much easier to miss than a change that actually breaks the build.

Still, I'm wary because (semver break or not) introducing a dynamic assert is much easier to miss than a change that actually breaks the build.

Exactly, this is my only reason to hold back such a "sneaky" UB ~fix~ workaround.

Just to chime in my opinion as a user of this lib: I've always assumed that .push_next() will only push one struct to the linked list, and will assert NULL or even silently overwrite anything in the pNext chain. Didn't even cross my mind that it would walk the linked list (because that's obviously unsafe).
For most use cases the compiler ought to be smart enough to strip away the assert!(pNext == null), so I don't see much value in adding a .push_next_unchecked.
Having unsafe push_next_many() is not a common pattern either, but can be added in case someone finds it useful.
tl;dr: IMO just make push_next() assert pNext == null.


WebSocketApi CORS error

I'm getting a CORS error when I try to use the WebSocketApi with my frontend running on the my local host. On an ApiGatewayApi, I can pass in cors: true to enable CORS, but I can't find anything similar for websockets. Am I missing something?

Hey @tysoncadenhead, sorry for the late reply. Please correct me if I'm wrong, I think CORS doesn't apply to WebSocket.

 Sorry I should have closed this out. The error was from me trying to use socket.io with API Gateway and they apparently don't play nice. I switched to the native WebSocket API and the CORS error went away.


ERROR:   _invertColors = MediaQuery.of(context, nullOk: true)?.invertColors ??

Runing flutter doctor

Error:

Before this error happen[Timeline]:
On trying to run my app.

The same here.

use dev version for now . #258
i will support null-safety and non-null-safety later

Confirmed:
  extended_image:
    dependency: "direct overridden"
    description:
      name: extended_image
      url: "https://pub.dartlang.org"
    source: hosted
    version: "1.5.0-dev"

dependency_overrides:
  extended_image: ^1.5.0-dev

Allows to start app with Flutter: 2.0.0
Another error occured (probably not related to extended_image)
Expected a value of type 'String', but got one of type 'bool'

pls use 1.5.0-dev


BlazonHelper . Blazons in bigger than 50px seem to show unproper

little movement in the overlays

is less adaption of Issue #100 & done


DragAdornerTemplate gets cut off with negative margins

What steps will reproduce this issue?
Clone and run https://github.com/brather1ng/GongDragAdornerIssue. Drag the brown rectangle in the middle.
The rectangle in the DragAdornerTemplate is cut off instead of being moved by the negative left margin.
It was hard to reproduce this:

Without MahApps.Metro, it works fine.
With small changes (not enough to allow me to do what I want to) it also works. E.g. replace the Width="{Binding Image.Width}" with Width="94" and it works.

So it seems to be some kind of really weird interaction between MahApps.Metro, Gong DragDrop and specific types of properties.
Expected outcome
The rectangle should be offset to the left and not cut off.
Repo
https://github.com/brather1ng/GongDragAdornerIssue
Environment

MahApps.Metro v1.4.3
GongSolutions.WPF.DragDrop v1.1.0
Windows OS 10
Visual Studio 2015
.NET Framework 4.5.2

@brather1ng Thx for the sample! You can move the adorner by setting the dd:DragDrop.DragMouseAnchorPoint e.g.
<UserControl x:Class="GongDropAdornerIssue.View"
             xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:dd="urn:gong-wpf-dragdrop"
             xmlns:local="clr-namespace:GongDropAdornerIssue"
             dd:DragDrop.IsDragSource="True"
             dd:DragDrop.DragMouseAnchorPoint=".3 1"
             dd:DragDrop.DragHandler="{Binding}">
    <UserControl.DataContext>
        <local:ViewModel />
    </UserControl.DataContext>
    <dd:DragDrop.DragAdornerTemplate>
        <DataTemplate DataType="local:ViewModel">
            <Rectangle Fill="Aqua"
                       Width="{Binding Image.Width}"
                       Height="94"
                       Margin="0 0 0 0" />
        </DataTemplate>
    </dd:DragDrop.DragAdornerTemplate>
    <Rectangle Fill="Brown"
               Width="94"
               Height="94" />
</UserControl>

this means 0.3 x the adorner width to the left of the mouse point and 1 x height to the top.

Thanks! Missed that property, it's a much better solution than the margin stuff I tried to do.


Difficulty finding custom Type in TypeMapDatabase in 0.8.23

Brief Description
When generating from a typemap, using the new Type signatures of 0.8.23, I have difficulty finding custom types in the type database.
Working Code from 0.8.22
if (TypeMapDatabase.FindTypeMap("CustomType", out TypeMap map))

Correctly finds CustomType and outputs map.
Attempted conversion to 0.8.23
if (TypeMapDatabase.FindTypeMap(new CustomType("CustomType"), out TypeMap map))

Fails to find the CustomType, despite a key existing in the database with the string "CustomType".
Workaround for 0.8.23
var customTypeDoesNotWork = new Field("CustomType", new QualifiedType(), AccessSpecifier.Public);
if (TypeMapDatabase.FindTypeMap(customTypeDoesNotWork, out TypeMap map))

This successfully finds the type in the database.
OS: Windows (0.8.23)
Used settings
Target: MSVC

Hello @xsacha . The overload with a string is still there but it's private. You could call it through reflection or make it public and build C++# from source code.
However, type maps are not meant too be manually obtained by users. After all, the whole point of C++# is automation. So could you please let us know why you need to find it yourself? There might be a totally unrelated bug of ours which made you do this by hand, or there might be an undocumented feature which would do what you need.

For me, this doesn't explain why a CustomType differs to a Field. I'd expect them both to work.
This is in a method for a callback type, in CSharpMarshalToNative, to check if the type is in the map. It uses a string to represent a callback type with multiple parameters.

You can directly use TypeMapDatabase.TypeMaps.TryGetValue instead.

I've also restored that overload to be public again.

Thanks.
.TypeMaps wasn't available for me.


feat(api): Ideas/examples for a possible optional JsonType API

Optional JsonProperties for the Configurations of the Telestion Verticles
Problem
As soon as verticles get complexer, optional JsonProperties for the defining Configuration are helpful to reduce the work by the configurating party - the end user - who expectedly has little to no experience at all with the Telestion Ecosystem. Therefore, these should be deemed a highly important role, making the project a lot more appealing to annoyed software developers, being woken up at 9 am (very early for a developer) to fix the missing parts of the configurations... 😆
Current state
At the moment, there is no special support for this feature apart from the native support out of the box from the Jackson Library or additional libraries. Unfortunately, the Jackson API does not easily support default parameters for inferring other than for documenation reasons as the Java Type System and its stubborness makes a good design difficult (or even impossible) and no possible solution. To be fair, there is one possibility to add those to the existing system. One has to add an additional constructor without one of the fields, which does not only stack up with more and more record parameters (problem 1), but also imposes problems with parameters of the same type in one class (problem 2). An example showcasing both of those problems is seen below. We want to give Default parameters to every parameter:
public record Foo(String param1, int param2, String param3) {
    // No parameters
    public Foo() {
        this("param1Foo", 42, "anotherFoo");
    }
    // Only param 1
    public Foo(String param1) {
        this(param1, 43, "anotherFoo");
    }
    // Only param 3
    public Foo(String param3) {  // <----- actually does not work as this constructor already exists for param 1 -> problem 2
        this("param1Foo", 42, param3);
    }

   // ... -> we need 4 contructors more to cover every edge case (one for param 2 alone and then 3 more for the different pairs)
}

Note that with this solution and the massive amounts of equal implementations also small mistakes as shown in the constructor for String param1 lead to fast bugs in the development of more complex verticles. Here, obviously the 43 should be a 42. This obviously can also be targetted by constant static variables, but imho this is still a bad style of implementation if you need an extra mechanic to fix a problem that shouldn't even exist in the first place.
Implementation Suggestions
In all cases, the manual overhead of adding default information to each of the Record values should be as low as possible. Therefore, an annotation-based approach is suggested which complements the existing @JsonProperty from Jackson. As seen in the examples, the Annotations are only added additionally to the fields. In a future release, it could also be thought of combining those additional Annotations with the JsonProperty into one Property, reducing the overhead even further. In the background a manual ObjectMapper (already provided by the Jackson API) could be used, which could work hand in hand with the proposed new Launcher API by @fussel178 - the registering would take place in this new Launcher API at the start of the Application. Currently, up to this point, the registering alternatively would take place in the Telestion class in the application package.
Details about the implementation
These 4 suggestions show possible implementations for the support of OptionalJsonProperty without relying on a big number of manual constructors as suggested by the creators of Jackson. Consequently, none of those implementations are working at this point and should only be seen as examples of how an API could look like, especially because the rest of the API would be hidden, and only the annotation are used by the users of the Telestion System.
Meta information about this PR
Related links
As far as I know, no current Issue is targetting this problem.
CLA

[x] I have signed the individual contributor's license agreement and sent it to the board of the WüSpace e. V. organization.

I would love some feedback and an open discussion about this problem! 👀

https://docs.telestion.wuespace.de/application/tutorials/adding-configuration-options/#step-4-adding-default-configuration-parameters

I believe that the cleanest, most readable, and most flexible approach to go about this is to override the getter methods in the records. Then, it looks something like this:
public record Room(@JsonProperty(defaultValue = "10") int size) implements JsonRecord {
    public Room {
        if (size() < 0) {
            throw new IllegalArgumentException("Size must not be negative");
        }
    }

    public int size() {
        return Objects.requireNonNullElse(size, 10);
    }
}

By "only" having the one constructor (and no need for a default constructor anymore), we can have additional validation in this constructor (as you can see here) which we can also use to determine if a message on the event bus is a valid object of that type (in this case, { size: 20 } would be a valid Room, but { size: -20 } not) in terms of the type parsing, and we can even use more elaborate defaults (e.g., the default port could be different depending on the protocol selected in another parameter).
This also doesn't require developers to learn additional concepts (such as the handling of default constructors or new annotations), can handle complex objects, etc.
The only downside I see with this, which compared to the other methods is in my opinion far less severe in terms of issues, is that you have to be careful to call the actual getter function (instead of the parameter itself) inside the record's functions (e.g., in the validation size() < 0).


Add support for overriding _call in GDScript

Godot version:
Godot 3.2+
Issue description:
Is there a reason we can't implement a _call method in GDScript the same way we can override _get, _set, and _get_property_list? I tried doing it earlier today and suddenly realized I couldn't. I was wanting to say, "if attempting to call method X, instead call X on my child node", and it wasn't a property, so I couldn't just use a setget to handle the override.
Just wondering if we could add that as a possibility or if there is some certain reason why it isn't supported.

@aaronfranke I mean, if the current master branch is already in feature freeze, then obviously anything I propose as a future feature would apply to some future version of the engine, starting with 3.2 at the earliest. Is it preferred that I just omit that portion of the Issue text entirely? Cause that's pretty much the available option.

@willnationsdev it's simpler if you state the current version of Godot you're using, since it's where you're missing the feature.
That said, I don't think it's a bad idea. I think we can do it in an efficient way.

This should probably be bumped to the 4.0 milestone since it's a non-vital feature.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


[3.x] Improve the clarity of Viewport's documentation

Salvage of #73023. The author does not have a public email and they deleted their repo so I can't properly credit them as co-author.
This change is only relevant for 3.x, because the 4.x Viewport/SubViewport documentation is completely different. It could be backported to 3.5 I guess, it's pretty minor either way.

OppositeNor<EMAIL_ADDRESS>should've worked?

@YuriSizov How did you obtain that?

I usually check the preferred email by checking the autogenerated .patch from the PR: https://github.com/godotengine/godot/pull/73023.patch

Cherry-picked for 3.5.2.


SAK-29814 get site id only from site collection id

The old logic of getting siteId via while loop might cause endless loop for those ids with format of '/usr/' or '/my', which are valid entity request format.

All looks good!


Repi

small fix to pion resampler

Hi @brownd1978,
You have proposed changes to files in these packages:

Scripts
JobConfig

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for 7371e2bf6c9b3a659a543ff229b8b677e14ad9c0: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at 7371e2bf6c9b3a659a543ff229b8b677e14ad9c0.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged 7371e2bf6c9b3a659a543ff229b8b677e14ad9c0 at 6a45aaf5b6740fd827764b4bceaf99cc3b3cf796

build (prof)
:white_check_mark:
Log file. Build time: 13 min 24 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 7371e2bf6c9b3a659a543ff229b8b677e14ad9c0 after being merged into the base branch at 6a45aaf5b6740fd827764b4bceaf99cc3b3cf796.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


JS Doc Content Block

Description
Adds JS Doc Parameter block to gatsby-theme-parliament
The JSDocParmeters component loops through a list of children provided to it and creates an Accordion populated with AccordionItems.
The MDXFilter component has been modified to add a new filter method called jsDocFilter which runs before the regular filterChildren method to shrink the number of children passed into filterChildren.
The above is expecting JS Docs to be converted to markdown using jsdocs2md. This gives the markdown it's reliable order of items.

anchor
header
any number of additional items until:

another anchor
a header, whose level is larger than the JSDocsParameters block
the end of the file

Folks who want this behaviour will need to annotate their markdown with <JsDocParameters/> tags.
Related Issue
n/a
Motivation and Context
Satisfies the JS Doc Parameters block component
How Has This Been Tested?
The Accordion and AccordionItems components come from parliament-ui-components where they have their own stories.
The JsDocParameters code has been testing in both gatsby dev and gatsby build to ensure it works in both development and server side rendered production mode.
An example JS Docs markdown files is part of this PR.
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Awesome thanks !

Can you please check the above JS error ? To reproduce you can try following:

go to the JS doc page,
go to storage folder entry by clicking on the "On This Page" link
scroll to the top

@icaraps whoa, that is a weird bug. I'll fix it.

@icaraps it was a one character fix so I just pushed the commit to main.


Access previous value of a FutureProvider

Is your feature request related to a problem? Please describe.
I have a search page that refreshes whenever the input text field is updated. I currently do this by having a searchResultProvider (FutureProvider) that watches searchTextProvider (StateProvider<String>) and makes a search request. Then the search result UI watches searchResultProvider and updates itself. The problem is, whenever a key is typed the Future changes back to loading, so the interface repeatedly flickers between results and a loading page.
Describe the solution you'd like
I would like a way of accessing the previous value so I can just assign searchResults.when(loading: (previous) => ... use previous value ...). Or if you could tell me the recommended solution to this problem, I would appreciate it.
Describe alternatives you've considered
My previous code used vanilla Flutter with a FutureBuilder. This code never had any flickering problem, as the snapshot always 'had data'.
I considered using a Stream or wrapping the provider with another one that remembers the previous value, but my Flutter/Riverpod skill is too low to understand how to do this.

A FutureProvider returns nothing more than an AsyncValue. When the future, that gets provided, changes (e.g. when your watched searchTextProvider changes), the state of the AsyncValue gets set to AsyncValue.loading by default until the future completes with value or error. This results in always showing your loading page whenever the future of the FutureProvider gets updated.
Accessing the previous state in the loading track of an AsyncValue was actually part of flutter_riverpod before, but got removed with version 1.0.0. You can read more about this here: https://github.com/rrousselGit/river_pod/discussions/905
An easy fix would be to make a ConsumerStatefulWidget where you store the valuein theConsumerState, whenever the FutureProvideremits a new value and executes the data track, but without callingsetState`. And in the loading part of the provider, you could return the result page, if the value stored in the widget is not null. Otherwise, you show the loading page. I would consider this only a workaround until a better solution is available. Like so:
class TestWidget extends ConsumerStatefulWidget {
  @override
  _TestWidgetState createState() => _TestWidgetState();
}

class _TestWidgetState extends ConsumerState<TestWidget> {
  int? previousValue;

  @override
  Widget build(BuildContext context) {
    return ref.watch(testProvider).when(
          data: (value) {
            previousValue = value;
            return ResultPage(data: value);
          },
          error: (_, __) => const Text('Error ocurred!'),
          loading: () {
            if (previousValue != null) {
              return ResultPage(data: previousValue);
            }
            return LoadingPage();
          },
        );
  }
}

Another solution would be to create a StateNotifierProvider that provides an AsyncValue<SearchResult>. So you can handle the loading state of the AsyncValue on your own in the corresponding StateNotifier/StateController.

Thanks, I tried your approach and it works great!

Do note that I plan on streamlining this sort of thing. Although I'm still in the design phase of this API

Closing since #1036 was implemented


Adjust version.json

Codecov Report

Merging #44 (2191d3e) into master (2988e02) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #44   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            2         2
=========================================
  Hits             2         2

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 2988e02...2191d3e. Read the comment docs.


dot/GraphViz diagrams are cut off

I recently upgraded to v1.6.0 (thanks for the project BTW).  I use dot toe generate collaboration diagrams which are then placed in the docs.  This is how they look.  The first diagram is cut off to the right.

Hm, can you provide me some information on how I can reproduce that? Is the documentation publicly available so I can have a look at it? 🤔

Unfortunately this is a private project.  I do have a public one up and I was able to reproduce this when making the window's width smaller.  The project is here:  https://github.com/define-private-public/PSRayTracing
With my doxygen file here: https://pastebin.com/5v26jcJJ  place that in the root of the repo, as well as doxygen-awesome.css.  Then run Doxygen with dot/Graphviz installed.  If you then open up the source and navigate to cxxopts::values Namespace reference page there should be some diagrams.  Then when making the window less wide, it should start to cut off the graphs.
What's interesting is that there are already some larger graphs, But doxygen add a sort of "pan-zoom" tool for them.

What's interesting is that there are already some larger graphs, But doxygen add a sort of "pan-zoom" tool for them.

This is because your Doxyfile has
# If DOT_IMAGE_FORMAT is set to svg, then this option can be set to YES to
# enable generation of interactive SVG images that allow zooming and panning.
#
# Note that this requires a modern browser other than Internet Explorer. Tested
# and working are Firefox, Chrome, Safari, and Opera.
# Note: For IE 9+ you need to set HTML_FILE_EXTENSION to xhtml in order to make
# the SVG files visible. Older versions of IE do not have SVG support.
# The default value is: NO.
# This tag requires that the tag HAVE_DOT is set to YES.

INTERACTIVE_SVG        = YES

What's interesting is that there are already some larger graphs, But doxygen add a sort of "pan-zoom" tool for them.

This is because your Doxyfile has
# If DOT_IMAGE_FORMAT is set to svg, then this option can be set to YES to
# enable generation of interactive SVG images that allow zooming and panning.
#
# Note that this requires a modern browser other than Internet Explorer. Tested
# and working are Firefox, Chrome, Safari, and Opera.
# Note: For IE 9+ you need to set HTML_FILE_EXTENSION to xhtml in order to make
# the SVG files visible. Older versions of IE do not have SVG support.
# The default value is: NO.
# This tag requires that the tag HAVE_DOT is set to YES.

INTERACTIVE_SVG        = YES


Update prepare-for-directory-synchronization.md

Minor typo change AD SD to AD DS at end of paragraph 1.

@JoeDavies-MSFT - Can you please assist to check, approve and merge the content of this PR?
Thank you.


⚠ Giovanni Gardin has degraded performance

In e61db59, Giovanni Gardin (https://www.giovannigardin.com) experienced degraded performance:

HTTP code: 500
Response time: 8637 ms

Resolved: Giovanni Gardin performance has improved in 42a35ef.


[BUG]:  CI stucks on network related tests on windows

Description of the bug
olink tests are sometimes stuck.
Topic may be investigated. How to proceed with that? for now timeout was added for CI step, but it will fail job for anyway
Steps To Reproduce
trigger CI to build  - sometimes it fails
Additional Information
No response

Check how often it fails, ideally comment here so that we get a better understanding of the frequency.


Implement a persistent node name

As we've seen in issue #5130, and demonstrated experimentally on a test node, a node in distributed mode can return nonode@nohost for the node() call. Our code, especially in upgrade_epochs/1, expects that node() will always return the configured distributed node name when in distributed mode.
This commit implements the node_name/0 function that works like node() but will always return the distributed node name from when the system first starts up. The name value is saved in a persistent term and then it can accessed throughout the life of the node, even during the shutdown phase, when the dist controller may have exited already.
Persistent term usage should avoid this being a bottleneck. It turns out it's even a bit faster than calling the native node() function:
> timer:tc(fun() -> [config:node_name() || _ <- lists:seq(1, 1000000)], ok end).
{628895,ok}

% dummy tc call to check for overhead
> timer:tc(fun() -> [ok || _ <- lists:seq(1, 1000000)], ok end).
{325742,ok}

% overhead over a dummy tc call
><PHONE_NUMBER>42.
303153

% 0.3 microseconds
> 303153/1000000.
0.303153

% compare to `node()`
> timer:tc(fun() -> [node() || _ <- lists:seq(1, 1000000)], ok end).
{480578,ok}

In addition to a persistent node name, add a few a assertions in the earliest startup code in config:init/1 to check that the configured node name and the result of node() stop the VM if user specified a name but we still ended up with a standalone mode anyway.

This was folded into https://github.com/apache/couchdb/pull/5132, closing


Auto format code with black via pre-commit

This will auto-format all python code consistently with black via pre-commit on commit.

Added black lint to github CI action (check if any file would be changed)
Added pre-commit with config to format with black, remove trailing whitespace and always end files with newline
Document how to install the pre-commit hook for development (pre-commit install)
Manually trigger the hook for all files to do the initial formatting (pre-commit run -a)

Using black improves collaboration by erasing any inconsistencies in code formatting, letting reviews etc be purely focused around code and functionality.
The git-hook must be installed manually by contributors via pre-commit install before the hook is actually run on commit. But the lint task in CI should stop PRs from merging if code is not aligned with black formatting.
Something to consider if/when merging this PR is any current work-in-progress PRs would probably face conflicts and need to run pre-commit on their branch.

Hm a few knobs like line-length can be set with black but it's mostly all or nothing.
Before I tried black I was also skeptical, but after using it I think it's such a relief not having to think about formatting at all.
Everyone has different styles and preferences so a formatter will never fully align, but the style is decent and consistent which I think is worth more.
Here's a nice talk
that mentions the built in formatter of golang and the reasoning behind it.

Yeah I don't like how it formats, but I really like that it formats
Gofmt is no one's favorite, yet gofmt is everyones favorite

I'm closing this PR but can reopen it if you'd reconsider.

Thanks for the talk. There are some really good points! I will think about it when I have some time on my hands.


Not getting any response from search

Hi there,
Me again, apologies if this is something obvious.
// Search with a term query
termQuery := elastic.NewTermQuery("name", "test")

searchResult, err := search.client.Search().
	Index("artist").      // search in index
	Query(termQuery).  // specify the query
	Sort("name", true). // sort by type field, ascending
	From(0).Size(100). // take documents 0-9
	Pretty(true).      // pretty print request and response JSON
	Do()

fmt.Println(searchResult)

fmt.Println prints &{66  0xc8201d2480 map[] map[] false <nil>}. But when I run http://localhost:9200/artist/_search?q=name:test it returns an entry.
Thanks in advance!

Hi!
It really depends on the mapping of the artist index. AFAIK if you run http://localhost:9200/artist/_search?q=name:test, you're running a QueryStringQuery against the index, not a TermQuery. The former is a full text query which also finds documents that look like a match while the latter looks for exact matches. Remember: ES is not a database, and not all queries are the same.
Can you please post the output of curl 'localhost:9200/artist/_mapping?pretty'?

Ahaa, I think I know what you mean.
The output is:
{
  "artist" : {
    "mappings" : {
      "name" : {
        "properties" : {
          "avatar" : {
            "type" : "string"
          },
          "bg_img" : {
            "type" : "string"
          },
          "followers" : {
            "type" : "long"
          },
          "genre" : {
            "type" : "string"
          },
          "id" : {
            "type" : "string"
          },
          "location" : {
            "type" : "string"
          },
          "name" : {
            "type" : "string"
          }
        }
      }
    }
  }
}

Maybe reading this in The Definite Guide will help you understand a bit more about all of this. The book is really helpful, highly recommended.

This section of the mapping e.g. illustrates that your index has a field called name of type string (see string type). All the other properties of the field have default values. This means that the field is analyzed, and uses the standard analyzer by default. The default analyzer e.g. splits on whitespace etc. If you want exact matches, you have to disable analysis on the field.
Run this to see what the default analyzer on the "name" field is doing with input: curl -XGET 'localhost:9200/artist/_analyze?pretty' -d '{"name" : "test"}'. Read more about testing analyzers here.

This section of TDG exactly describes your problem.

I'm getting...
{
  "error" : {
    "root_cause" : [ {
      "type" : "illegal_argument_exception",
      "reason" : "Unknown parameter [name] in request body or parameter is of the wrong type[VALUE_STRING] "
    } ],
    "type" : "illegal_argument_exception",
    "reason" : "Unknown parameter [name] in request body or parameter is of the wrong type[VALUE_STRING] "
  },
  "status" : 400
}

when I run curl -XGET 'localhost:9200/artist/_analyze?pretty' -d '{"name" : "test"}'

Syntax was wrong. Read it up here, but I think the documentation is a bit misleading here:
# Create the artist index
$ curl -XPOST 'localhost:9200/artist'
# Test analyzer
$ curl -XGET 'localhost:9200/artist/_analyze?pretty&analyzer=standard' -d 'this is a test'
{
  "tokens" : [ {
    "token" : "this",
    "start_offset" : 0,
    "end_offset" : 4,
    "type" : "<ALPHANUM>",
    "position" : 1
  }, {
    "token" : "is",
    "start_offset" : 5,
    "end_offset" : 7,
    "type" : "<ALPHANUM>",
    "position" : 2
  }, {
    "token" : "a",
    "start_offset" : 8,
    "end_offset" : 9,
    "type" : "<ALPHANUM>",
    "position" : 3
  }, {
    "token" : "test",
    "start_offset" : 10,
    "end_offset" : 14,
    "type" : "<ALPHANUM>",
    "position" : 4
  } ]
}

Not sure about the syntax. Please try to figure out with ES documentation, sorry.

Ah. I was using it against 1.7.5. Documentation for 2.2 is good.

I switched my query type to NewCommonTermsQuery(field, term) and it worked perfectly!
Thanks for all your help/advice :)


Add orders list to the customers details view

Feature request
Description
Just as we did with the address list, we should also add a list with all orders related to the customer.

Waiting on #3834 before we can fix this issue.


Added DropCollection unit test.

Here's a trivial DropCollection unit test, if you want it!

I will made some tests now.


APEXMALHAR-2078: Thread issue due to data structures at thread level.

@DT-Priyanka : I think this change is not the right fix. What happens when Operator is checkpointing while scanner is updating inputDirTolastModifiedTimes. Will this not cause issues?

@DT-Priyanka please look at the flow with referenceTimes. The state that is checkpointed by the operator thread is updated by the operator thread.
Also why do we need another map?  why can't the entries of inputDirTolastModifiedTimes be added to the same data-structure referenceTimes

Let me check if I can merge two data structures (referenceTimes and inputDirTolastModifiedTimes) by changing data structure of referenceTimes.

Can you please explain what inputDirTolastModifiedTimes is for ?

The reason behind introducing this data structure is,
When we ask filesplitter to scan multiple directories. We do maintain time stamps of seen files using their relative path. Now provided you have asked to scan dir1 and dir1/dir2 having following structure,
dir1/myDir
dir1/dir2/myDir
Now the /myDir, though lie in different source directories will be considered as duplicates by operator and one of the entries will be skipped.

I guess I can move the logic to detect these duplicates from scan time to the process time, so that I can reuse referenceTimes. Will update on this soon.

Can the problem be solved by the key in referenceTimes be absolute path for all the files?

Ya I am actually trying to evaluate that, will get back to you.

we can't do it with absolute path because, say in above example dir to scan was dir1 and dir2 in output you will expect something like,
Output

dir1/myDir
dir1/dir2/myDir
dir2/myDir

If we use absolute path output won't have dir2/myDir after copying "dir1"

Here is one proposed solution,

We do keep the "inputDirTolastModifiedTimes" in operator thread.
We pass a copy of inputDirTolastModifiedTimes to scan thread and mark scan thread copy transient.
We do the filtering twice, during scanning and then processing. So if operator is running from long time scanner filtering should take care of avoiding duplicates. If operator is redeployed the scan thread won't remove duplicates (as it didn't save inputDirTolastModifiedTimes) and this time the operator thread will take care of filtering.
This sounds like some processing overhead, but that's not much. I will still think further to optimize it. Meantime if you, @chandnisingh , have any suggestion let me know.

If you use the abs path
will it not be as -

/user/priyanka/dir1/myDir
/user/priyanka/dir1/dir2/myDir

Since /user/priyank/dir1/dir2/myDir will already be there, it will be ignored?

@DT-Priyanka I may have misread the problem.
Is the problem that if the user provides 2 directories to scan but they are duplicates then we do want to have duplicated output as well?

@DT-Priyanka @chandnisingh this is marked for 3.4.0, can it be wrapped up or moved out of the release?

@chandnisingh  updated the code to address threading issue, please review and merge.

@tweise  have updated code, hopefully it will make it in current release.

Addressed review comments, also added a test to check if empty input directory gets copied.


[BUG]: drizzle-kit fails to generate a PRIMARY key even though the id column has primaryKey()

What version of drizzle-orm are you using?
0.28.6
What version of drizzle-kit are you using?
0.9.13
Describe the Bug
Have the following two files.
// db/auth-user.ts
import { auditAwarePartial } from './shared/audit-aware';

export const authUser = pgTable(
    'auth_user',
    {
        id: varchar('id', {
            length: 15 // change this when using custom user ids
        }).primaryKey(),
        // other user attributes
        email: varchar('email', {
            length: 255
        }).notNull(),
        roles: varchar('roles', { length: 50, enum: ['ADMIN', 'USER'] })
            .array()
            .notNull(),
        emailVerified: boolean('email_verified').notNull().default(false),
        ...auditAwarePartial
    },
    (table) => ({
        emailIdx: uniqueIndex('email_idx').on(table.email)
    })
);

// ./shared/audit-aware.ts

export const auditAwarePartial = {
    ...createdAtPartial,
    updatedAt: date('updated_at', { mode: 'date' }).notNull(),
    updatedBy: varchar('id', {
        length: 15 // change this when using custom user ids
    }).notNull()
} as const;

When you run drizzle-kit generate:pg (or w/e db) -> you will see that the definition for the table auth_user is missing PRIMARY for the id column. Alternatively if you remove the spread of auditAwarePartial (or simply move it to the top of the table) -> then the PRIMARY keyword will be present in the definition in the id column.
Expected behavior
PRIMARY keyword should always be present.
Environment & setup
Nothing special.

@callmeberzerker how did you resolved this issue
i am unable to set primaryKey on the varchar type

@imopbuilder my issue was that I was overriding the field definition with that ... auditAwarePartial spread unintentionally.


Provide better defaults when user creates package.json file from Add New Item JSON template

If the user names their json file ```package.json````, we should automatically provide better defaults based on the folder name it is being added to (we can probably use the same wizard we use to fill in the package.json file for "New Project")
  <WizardExtension>
    <Assembly>Microsoft.NodejsTools.ProjectWizard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a</Assembly>
    <FullClassName>Microsoft.NodejsTools.ProjectWizard.NodejsPackageParametersExtension</FullClassName>
  </WizardExtension>

Actually, nvm. That wouldn't work because we'd need to replace the content altogether in this case.


Pin Werkzeug < 1.0 for 0.12

Pins Werkzeug to a compatible version.

@ThiefMaster thought about it but i would prefer not to. 3.x seems to work with it and that way people can update without setuptools failing.


MSID: 100491 Version: 1 DOI: 10.1101/2<IP_ADDRESS>0070v

MSID: 100491
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0070v
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=100491
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0070v

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/100491v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0070v
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/100491/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

queried author about a reference


Call for papers translation is broken

Problem and impact
This PR: https://github.com/fossasia/eventyay-talk/pull/33 changed CfP to Call for Papers which breaks all translations, since they are translating CfP. @Aryan20 please fix those translations in al .po files.

I also see some other appearances of CfP, can we change them all so there's consistency through out the entire app?
@Aryan20 I can't assign to you for some reason, maybe @mariobehling can but please start working on this.

I did change it but was told to roll back those changes from the PR.
I can use "manage.py makemessages --all" if required and create PR.

@Aryan20 Please feel free to make this PR here. I was previously under the impression you created new translations. Sorry for my confusion mistake. Would be great to get the missing translations here now.

@mariobehling Sure, no issues. Will do as soon as possible.

@mariobehling @marcoag Please check the PR. Apologies for the delay, got caught up in University work.

Is their any way to discuss about project?

Is their any way to discuss about project?

https://app.gitter.im/#/room/#fossasia_open-event-frontend:gitter.im


Bqiao/twitter api change

TweetDetail seems to have been replaced by TweetResultByRestId

400 bad request

well, I tested this api "https://api.twitter.com/graphql/ncDeACNGIApPMaqGVuF_rw/TweetResultByRestId" and it worked.


Workflow throughput falls when submitting ~50 of them

Very general issue. Since the Jan 26th switch firmware update, we have not managed to crash the switch. However, when submitting ~50 workflows at a time, there is an enormous drop in performance. Most of the workflows are stuck in DriverWait, sometimes for ten minutes at a time.  They seem to progress in fits and starts.

I looked into this and as far as the Setup phase is concerned, there are a few optimizations when can make. One is a trivial change and improves the setup performance slightly (~10%), the other is a bit more complex but would improve setup performance greatly. I'll work with Matt on the necessary changes in the coming days.

Setup timings for a series of 50 workflows which were scheduled to 2 computes as they became available.  Each job specified a single xfs filesystem, #DW jobdw capacity=1TiB type=xfs name=test1.  It took about 24 minutes to work through all of the workflows and the setup times ranges from as low as 9 seconds to as high as146 seconds.

When does flux progress a workflow to setup, immediately after proposal, or after the computes for that workflow become available, or some other timing?
After post_run, does flux attempt to drive the workflow through data_out and teardown as quickly as possible, such that workflow n is going through the teardown while workflow n+1 is progressing to setup?

Are both computes assigned to each worklow such that the workflows serialize on the computes at pre_run?

For the results above each workflow requests a single compute.

When does flux progress a workflow to setup, immediately after proposal, or after the computes for that workflow become available, or some other timing?

The workflow is progressed to setup when the compute is scheduled.

After post_run, does flux attempt to drive the workflow through data_out and teardown as quickly as possible, such that workflow n is going through the teardown while workflow n+1 is progressing to setup?

Currently Flux will progress the workflow through post_run and teardown as quickly as possible.  However, once the compute has been released a new workflow can progress through setup while the previous workflow is still progressing to teardown.

These were the only allocations on the system for data in the https://github.com/NearNodeFlash/NearNodeFlash.github.io/issues/29#issuecomment-1435419842 above.

 That latest fix seems to show an improvement. There are more setup and teardown times in the single digit seconds. I checked the logs from the system. It looks like the major outliers (~60 seconds+) are caused by a timeout error from the switch. We're able to see the same thing internally, and we're investigating.


'predict_by_points' function

Environment Information

samgeo version:
Python version:
Operating System:

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
What I Did
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

Hi!
Thanks for your nice work and continued updates! I am facing the following error:
AttributeError: 'SamGeo2' object has no attribute 'predict_by_points'
I also tried to re-install SAM geo to see if the updates appear or not. However, I faced the installation problem below:
**Using cached sam2-0.4.0.tar.gz (75 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Using cached sam2-0.3.0.tar.gz (75 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
ERROR: Cannot install segment-geospatial because these package versions have conflicting dependencies.
The conflict is caused by:
sam2 0.4.1 depends on torchvision>=0.18.1
sam2 0.4.0 depends on torchvision>=0.18.1
sam2 0.3.0 depends on torchvision>=0.18.1
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip to attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts**
Thanks in advance for your help!

This is your own conda env problem. Try creating a fresh conda env to install samgeo or use Google Colab.
https://samgeo.gishub.org/examples/sam2_point_prompts/

This is your own conda env problem. Try creating a fresh conda env to install samgeo or use Google Colab. https://samgeo.gishub.org/examples/sam2_point_prompts/

Many thanks for your prompt reply. It's working nicely in Google Colab!


XSS attack

First of, thanks for this library. I'm going nuts about the lack of table functionality in many of the free wysiwyg editors. This editor has good support for tables 🥇
I have however found an XSS vulnerability in this editor. It appears the editor does filter out script tags, unfortunately that is not sufficient for all XSS attacks.
Reproduction

Go to http://suneditor.com/sample/index.html
Switch to code view
Insert <img src="#" onerror="alert('hi')"/>
Switch to normal view

@Darthmineboy Thank you for your reporting.
Will be fixed in the next version.
Thank you.

The 2.34.2 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


[Other issue]: No license in this repo

Before proceeding, is there an existing issue or discussion for this?

[X] I have done a search for similar issues and discussions.

Description
Hey there, team. It doesn't look like there's a license in this repo yet. I'm happy to send over a PR adding the Apache-2.0 (like the rest of the OSRF ecosystem), if you'd like?

Ah there's one inside the rmf_site_editor. That was probably the original license file which was accidentally moved into a subdirectory during a refactor.
I'd suggest a PR that simony

Ah there's one inside the rmf_site_editor. That was probably the original license file which was accidentally moved into a subdirectory during a refactor.
I'd suggest a PR that simply does a git mv from the subdirectory into the root.

Ah, good catch. Thanks for finding that. I've sent over a PR moving it back to root.


Switching from a sword to a different item and trying to use that item will not register

What is the output link of /viaversion dump?
https://gist.github.com/171998bbe3f09596b6eb16cccbc92aa1
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
ProtocolLib and AAC on the latest version of Spigot for 1.8
How does this error happen? login? Using an item?
When quickly switching from a sword to another item then trying to use it, the item wont be used and you'll instead get a shield and block.
Is there an error in the console? Use pastebin.com. Is there a kick message?
No

I think I made an issue about this in the past, sorry I didn't provide much information that time.

As an example say I am holding a sword then the client sends a CPacketHeldItemChange to a health potion, then it sends a CPacketPlayerTryUseItem. It won't register unless you wait roughly 200ms after it sends the new slot.

(I also exempted myself from AAC and tried this again and it still occurred.)

Here's a video https://www.youtube.com/watch?v=x0YcGKEN7JM

Does this require AAC?
I can't replicate this otherwise. (Normal spigot)

No, when I get home today I'll provide specific versions of Spigot and ViaVersion as well as update everything.

Is there any update on this?

See above, otherwise will follow up when we can replicate.


Move NR Vega-Lite specs from data repo to site repo

And save as JavaScript files, so we can use data_branch in TopoJSON paths

Closing and consolidating here.


 All transformations?
pom-tuner is a library potentially used by many Maven plugins or other tools. Should they all ignore the given element, regardless of their intents and purposes?

@ppalaga sorry, all these issues were created hastily by me. You are right, this is a "low level" lib. I will pull something a top of it in toolbox.


Add shader source editor

Adds a shader source editor to babylon.

You can edit both the fragment and vertex shaders, and it updates the scene in real time. The strip in the middle is the history bar which lets you roll over snapshots of your edits to compare or revert to different versions.
Not ready to merge yet, there's still some issues with the way things are packaged.

@deltakosh I do know Monaco. The problem is, Monaco doesn't support GLSL and it's a huge multi-MB dependency.
I want something lightweight because the shader editor has to be dynamically loaded when the user clicks the "Edit Source" button. If I used Monaco then the user would suffer increased load times without actually deriving any benefit from it. CodeMirror does everything I need it to, it's 10x smaller, loads almost instantly and supports GLSL out of the box.

Looks better to me but I ll leave it open for @deltakosh to decide on the amount of code being added to the inspector. I am quite pleased it does not impact the core lib though :-)
@deltakosh is off for a couple days but should be able to look into it on Tuesday.

Yeah I've still got a few more things on my todo list but I've cleaned it up a lot.

You should also credit yourself in the what's new file in the dist/preview release folder (please mark it as major feature)

@atg, I am looking into the rebuild as I am not sure we could use it for shaderMaterial. using RebuildProgram is meant to change output code not shader source. So it won t handle webgl1 to webgl2 or defines and so on.
Here the functionnality is identical to spector and I am not sure it is intented. It should allow to modify the sources as they are so we can save back the original shader not the output after processing ???

@sebavan Yes that's the intention. I'm adding a rebuild method to Effect that does the preprocessing and then calls _rebuildProgram.

In this case, maybe the easiest and less intrusive would be to add a update method to the shaderMaterial itself to recreate the effect ?
The effect is shared with all the framework and keeping it as thin as possible is always good if possible.

+1. This must be on a shaderMaterial.updateShadeCoder() function

This should be possible without modifying effect at all ? I would really prefer this.

Agree with @sebavan: Effect should remain unchanged
On a side note, I will add API on the ShaderMaterial to serialize it to files and to the snippet server so we will be able to load materials the same way we load NME shaders
https://github.com/BabylonJS/Babylon.js/issues/8267

To prepare it please add similar code: https://github.com/BabylonJS/Babylon.js/blob/master/inspector/src/components/actionTabs/tabs/propertyGrids/sprites/spriteManagerPropertyGridComponent.tsx#L71
Only difference will be that to save to the snippet Server, the payload will be on .shaderMaterial instead of .spriteManager: https://github.com/BabylonJS/Babylon.js/blob/master/inspector/src/components/actionTabs/tabs/propertyGrids/sprites/spriteManagerPropertyGridComponent.tsx#L148

I added these API for you in this PR: https://github.com/BabylonJS/Babylon.js/commit/f9a5f6371a2a68f7748d1a8be06a191a4c823d95

Up?

up?

@atg any update on this one ?

Last call before I close it

Closing it for being idle for too long. Please feel free to reopen when you can


Most recent NPM package missing the dist directory

Hi. Sorry to bring bad news, but the current npm package missing the dist directory.

I published v1.1.1


parameters in PCA-ICA analysis

Dear Biafra,
we are using Ciatah to analyse our Ca imaging data, but we are struggling finding good parameter values for the PCA-ICA cell extraction.
Can you tell me what the mu and term_tol paramters are? or point me towards any documentation or literature where they are explained?
Thank you!
Natalia

Hi Natalia,
Some explanations below. I'll add some of this information to the docs (https://bahanonu.github.io/ciatah/pipeline_detailed_signal_extraction/).
As a quick question: what type of data are you analyzing and do you have a snippet of a representative movie you are trying to analyze or can share a frame? That'll also help determine whether anything can be done on the pre-processing end to improve PCA-ICA runs or if the data will work well with PCA-ICA.
Mukamel, 2009 (mu)
The original Mukamel, 2009 (https://doi.org/10.1016/j.neuron.2009.08.009) paper describing PCA-ICA gives an explanation of mu:

If you look at Fig. S3, it also provides some information on the effects that varying mu 0 to 1 have on cell extraction quality (we've often found lower values, e.g. 0.1, work well):

Ahanonu, 2022 (mu)
We also describe it in our recent calcium imaging experiments and analysis book chapter, see section 3.15 Extraction of Neuron Shapes, Locations, and Activity Traces: https://link.springer.com/protocol/10.1007/978-1-0716-2039-7_13#Sec19.

term_tol
The term_tol parameter is the termination tolerance, e.g. when min difference between ICA iterations is below this value, the algorithm will exit (if it hasn't already reached max_iter).
cheers!
-Biafra

Hi Biafra,
thank you very much for all the info, it is very helpful.
We are analysing 1p data of interneurons (obtained with GCaMP6f), so the signal-to-noise ratio is not super good.
Any recommendations will be much appreciated.
thanks!!
Natalia

Glad to hear!
Would you be able to provide a link (e.g. Dropbox/Box) to one of the movies? I can then take a look and get back with pre-processing/cell extraction suggestions.

Hi Biafra,
here is a link to one of the movies[movie_exmaple] (https://drive.google.com/file/d/1ROwykUCIMlNQtzyCrNj_g2ja279q2AVI/view?usp=sharing)
Our videos are already spatially filtered and spatially downsample 4x (so the movies are 270 x 270).
Thank you very much for the help!
Natalia

Thanks, will take a look. Could you send a movie without spatial filtering? Will make it easier to test pre-processing on my end.

Hi Biafra, unfortunately we don't have the original inscopix videos :/
Thank you so much for your help!


CAS support and other useful things

Do you plan add support for CAS and ability to specify prev values for keys/ttl and other stuff?

All the etcd api including CAS  is supported

How can I set key with new value providing prev id?
20 нояб. 2015 г. 8:41 пользователь "Shafreeck Sea"<EMAIL_ADDRESS>написал:

All the etcd api including CAS is supported
—
Reply to this email directly or view it on GitHub
https://github.com/shafreeck/cetcd/issues/19#issuecomment-158284532.

@vtolstov Check about the master branch. There is an api called cetcd_cmp_and_swap_by_index .  May be this is what you want ?

Thanks ! That what i need. May be write comments to code for autogenerating docs?

@vtolstov  Good suggestion ！ Thanks, I will do it.


Follow HTTP redirects when importing code profiles.

As a user I need vscode to follow HTTP redirects when importing code profiles.
I am a content creator (mostly YouTube). I often create resources for my viewers and when I do I create short (usually) subdomains on my website to redirect to the real content.
I created a starter code-profile here https://code-profile.devinsideyou.com/ which redirects to https://raw.githubusercontent.com/DevInsideYou/code-profile/main/DevInsideYou.code-profile.
However a profile import only succeeds with the longer direct url.
It's probably just a limitation as opposed to a bug and as such I created this issue as a feature request.
XHR failed is all I see in the logs:
2023-02-18 11:56:16.444 [error] RequestService#request (browser) - error https://code-profile.devinsideyou.com/ {}
2023-02-18 11:56:16.447 [error] XHR failed: Error: XHR failed
    at v.onerror (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:72:1428)

@agilesteel I am seeing CORS error. It seems the server https://code-profile.devinsideyou.com/ is not accepting CORS requests. You would need to enable CORS on the server.


postbuild.sh creates invalid docker-in-docker.sh

Looking at this code, I don't know what it is doing because there isn't a docker.sock in the container.
postBuild..sh directly cats code into the container to talk to /var/host-run/docker.sock.  The container doesn't actually have  that socket in it.  This code doesn't create that
Environment
Host: Linux
Version: current version as of 2024/08/04
OS: Ubuntu
Actual socket: /var/run/docker.sock
Code
postBuild.sh
cat <<EOM \| sudo tee /etc/profile.d/docker-in-docker.sh > /dev/null
--
 if ! groups workbench \| grep docker > /dev/null; then
  docker_gid=\$(stat -c %g /var/host-run/docker.sock)
  sudo groupadd -g \$docker_gid docker
  sudo usermod -aG docker workbench
 fi

EOM

logs
stat: cannot statx '/var/host-run/docker.sock': No such file or directory
groupadd: invalid group ID 'docker'
usermod: group 'docker' does not exist
Traceback (most recent call last):

Note that nim anywhere appears to do something different but they do a hostmount for /var/host-run and /var/run


Put flake8-logging next to the other flake8 plugins in registry

Summary
This is just a nitpicky improvement, but I thought it'd be a good opportunity to look at the ruff source.

The rules list in the documentation is generated using the registry order. Currently, flake8-logging is separated from the rest of the flake8 plugins. This patch puts it next to them.

https://docs.astral.sh/ruff/rules/
If it makes sense, we could alternatively just sort the linters in https://github.com/astral-sh/ruff/blob/main/crates/ruff_dev/src/generate_rules_table.rs.
Test Plan

I think the sorting there is intentional (Flake8 stuff, followed by the more specific linters like the numpy rules, followed by the Ruff set), so this fix seems right.


ghidra failed to disassembly and decompile

Describe the bug
I just analyzed a elf file, which will be attached with ghidra, but ghidra could not disassembly properly and made it hard to read. Even external funtion calls are processed as local functions with improper name. I'm a ctf player in pwn direction, so the program DOES have bugs. The screenshot of BinaryNinja showed the proper decompiling result.
To Reproduce
Steps to reproduce the behavior:
Just auto analyze the file.
Expected behavior
Like the screenshot of BinaryNinja.
Screenshots

Attachments
If applicable, please attach any files that caused problems or log files generated by the software.
format_level0.zip
Environment (please complete the following information):

OS: Arch Linux 6.5.2
Java Version: 17.0.8 Graal VM
Ghidra Version: 10.3.3
Ghidra Origin: official GitHub distro

Additional context
Add any other context about the problem here.
The binary belongs to a ctf match organization which is holding a match till 14th Oct. Don't spread it.

It appears we do not currently handle the PLT thunking scheme employed by this binary.  This will take some effort on our part to add support for.  I will submit as an internal ticket to address although I can't comit to a timeframe.  Thank you for reporting this and providing the sample.

You can try the following steps to get the thunks to resolve in the .plt.got and .plt.sec sections which involves setting the EBX value equal to the address of .got section over the sections.  For the sample provided this is 0x13fac.
1) Select the entire section (e.g., `.plt.got`)
2) Invoke the Set Register Values... popup menu action and assign EBX value as 0x13fac
3) With selection still in place, invoke Analysis->Auto Analyze action (thunks should resolve)
4) Repeat for other EBX based PLT thunks (e.g., .plt.sec)

I had also altered the init function signature which may have been improperly set.  I assumed this since it did not match the signature which had been produced by DWARF and appears to have been applied from one of the datatype archives.

I've checked your solution and it works perfectly. But what confused me is that why other binaries are not effected, whose .plt.got and .plt.sec sections are not assigned a ebx value, either.
BTW, could you inform me of what is the font id of the font when I open set pre comments dialog in theme?

In most cases we rely on a constant propogation analyzer to compute the destination address which in this case is indirect and requires a valud for EBX.  There are numerous GOT/PLT mechanisms for each processor and most do not utilize a global register value such as this.  The ultimate fix for this case when handled by the Ghidra ELF Loader will likely require setting the EBX value as I have outlined.  The rest is handled by analysis.
Regarding the UI font for the Comment Dialog, it appears that there is currently a hard-coded Monospaced font forced for JTextArea.  Thanks for pointing out this concern.  A fix for this is in the works.  Since this dialog does not use any custom IDs, it relies on the default for laf.font.JTextArea.font within the theme editor.  At present, the hard-coded font setting (not reflected by the theme editor) is overriding the Theme setting.

One more suggestion: please enlarge the icons, if you have time. Recently, more and more developers use hi-res screens, old icons are a bit difficult to click on since they are too small; if just relying on the system to enlarge icons, they looks vague. So providing a icon pack and adding a button to load it in Theme Config dialog seems to be a good choice.
Anyway, thank you, @ghidra1 , for solving my problem. The improvement in suggestion is optional, whether implementing it or not depends on you (or your team).

You can try the following steps to get the thunks to resolve in the .plt.got and .plt.sec sections which involves setting the EBX value equal to the address of .got section over the sections. For the sample provided this is 0x13fac.
1) Select the entire section (e.g., `.plt.got`)
2) Invoke the Set Register Values... popup menu action and assign EBX value as 0x13fac
3) With selection still in place, invoke Analysis->Auto Analyze action (thunks should resolve)
4) Repeat for other EBX based PLT thunks (e.g., .plt.sec)

After doing this I achieved the following result for main method decompilation:

@ghidra1 I recently encountered this issue in Ghidra 11.0.3 - is there a planned fix?

You can try the following steps to get the thunks to resolve in the .plt.got and .plt.sec sections which involves setting the EBX value equal to the address of .got section over the sections. For the sample provided this is 0x13fac.
1) Select the entire section (e.g., `.plt.got`)
2) Invoke the Set Register Values... popup menu action and assign EBX value as 0x13fac
3) With selection still in place, invoke Analysis->Auto Analyze action (thunks should resolve)
4) Repeat for other EBX based PLT thunks (e.g., .plt.sec)

After doing this I achieved the following result for main method decompilation:

@ghidra1 I recently encountered this issue in Ghidra 11.0.3 - is there a planned fix? The manual steps that you outlined worked but these are not great for automation.

I have a fix in for Ghidra 11.1.1 patch release

Fixed by b1a67cce480e2223b0d6e2472f82d6c8a9bb6e48


Add delegate_collection_authority

In our Node.js SDK, we have the delegateCollectionAuthority method, which delegates a collection authority to a new address. This is handy when used with our Mint API. We should add this functionality to the Rust SDK

Can you add assign this to me?

@davidkathoh go for it!


How do you setup for multiple back-end APIs?

This question was asked and closed with no resolution once before:
https://github.com/MicrosoftDocs/azure-docs/issues/16325
Is there an update for this? How do you go about configuring an app service to access more than 1 back-end API? It's one thing to just add more permissions to the app registration's permissions, but the token that's stored in the X-MS-TOKEN-AAD-ACCESS-TOKEN and available via the /.auth/me endpoint has the audience field automatically set to only the 1st back-end API configured in the app registration and is unusable for accessing other APIs. Are we supposed to roll our own auth and manually reobtain a new token for every other resource?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8ff46628-b5dc-8b02-02b6-8f510d8059b0
Version Independent ID: 24d09b70-d144-d152-ac5a-c49219a5d8e8
Content: Authenticate and authorize users end-to-end - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-auth-aad.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@ththiem, Thanks for the question! Apologies for any inconvenience! We are taking a look into this and will get back to you soon.

@ththiem ,
You will get a token specifically for each resource/back-end. So, we would have to obtain a new token silently.
We won't be asked for authentication. We just need to obtain a new token silently.

@ththiem , Just wanted to understand if you need further assistance on this issue.

@ththiem , I will proceed with the closure of the issue. Please feel free to reopen if you still need assistance.

What happened to this? I have just followed the tutorial assuming it would support multiple backends but now find it will only do one at a time.
Is this mentioned in the Tutorial?
i.e METHOD WILL ONLY SUPPORT A SINGLE BACKEND APP SERVICE.
Is there a useful link for how to configure?
Cheers

Is the appropriate link?
https://docs.microsoft.com/en-us/archive/blogs/mihansen/azure-active-directory-authentication-easy-auth-with-multiple-backend-apis


Wont install

Grav v1.1.17 - Admin v1.2.14
The system says to report that there is an issue.
I cant give more info than this

Thanks! I will take a look into it.


Feature request: Client phone number / custom fields

I believe it would be very useful if it was possible to include the client's phone number in the invoices, a long with ID, address etc.

Maybe this could be implemented if custom fields were a feature. So you could also add other relevant details.

Yeah, i agree, custom fields would be awesome!
But probably something that will take a while to implement


Add support for deploy percentage

Following discussion in https://github.com/fregante/chrome-webstore-upload/pull/91#issuecomment-2154102315 here is an update of the CLI

3.3.0 is out 🙌


vscode opens a random file and wont let me close it

VSCode Version: 1.41.1
OS Version: macOS 10.14

Steps to Reproduce:

Open up the Interactive Playground.
For me atleast, a file named "vs_code_editor_walkthrough.md" will open. WHenever you close it, it will re-open itself, and it will not allow you to focus on a different file.

Does this issue occur when all extensions are disabled?: No

Unfortunately I can't reproduce this. Can you provide any other details? What settings do you have set?

Does this issue occur when all extensions are disabled?: No

@GingkathFox it looks like the problem is caused by one of your extensions. I suggest you bisect (disable 50% of your extensions, re-test, repeat) until you find which one, then report it to the extension developer.

Alrighty, will do.

Thanks.


Network request failed

this is my info.plist,but i still get Network request failed,
react-native:0.42.0
ios simulator
who can help me? thankyou so much

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information, or the list of reproduction steps is not complete. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


How Can I check network error when mutation failed or something errors when mutation failed

How Can I check network error when mutation failed or something errors when mutation failed

Try this
final QueryResult result = await GraphQLProvider.of(context).value.mutate(MutationOptions());
if(result.hasException){
    print(result.exceptions);
}

you can also use  try {} catch(e){} with previous code to be sure

https://github.com/zino-app/graphql-flutter/tree/beta/packages/graphql#exceptions


Bootstrap4 Templates

Bootstrap4 is now up to version 4.3 and is stable. As Bootstrap3 templates are included, perhaps providing Bootstrap4 ones could also be done?

Hi @PaulBrownMagic. Sure, I'd be happy to have a set of templates for bootstrap 4. If you have experience with it, I'd gratefully accept a pull request?

Sure, I'd be happy to, I think I can drag the code from an existing project. Give me a week or so to fit it into my schedule.

Thanks @PaulBrownMagic, that'd be great. Please take a look at our [contributing guidelines]( https://wagtailmenus.readthedocs.io/en/stable/contributing/index.html] There are some good tips in there for developing locally, which could save you a bit of pain in getting set up :)

Hey @PaulBrownMagic, did you manage to get anywhere with this? If not i'll pick this up as I'm keen to use them too :).

Sure, I've pulled my templates out of a project and pushed them to my fork: https://github.com/PaulBrownMagic/wagtailmenus
If you'd double check them for me I'll create the pull request?

Your fork works quite good for me.
Only in main_menu_dropdown.html I needed to change
<a href="{{ item.href }}" class="nav-item{% if item.has_children_in_menu %} dropdown-toggle" ...
to
<a href="{{ item.href }}" class="nav-link{% if item.has_children_in_menu %} dropdown-toggle" ...

Fixed it.


chore: Elevate the gopsutil module to v3

Upgrade the gopsutil module to facilitate the correct compilation of the goofys binary on macOS M1.

Would love to see this getting merged. Probably closes #798

I agree with @martincpt, for anyone else that stumbles here, I was able to get it running on M1 Mac 12.7.5 after checking out this branch and running go build. The brew install failed as well as installing v0.24.0.


Relative time timezone fix

Changing attime.py's treatment of relative times with specified timezones to have consistent behaviour with absolute times/restore behaviour from 0.9.12, as discussed in #1151

@MFAnderson Do you think you'd be up to porting a similar fix to 0.9.x?

Yeah, absolutely.

@obfuscurity #1157 is the pull request for the fix ported to 0.9.x

What is the state of this?

@MFAnderson Sorry for the slow response on this, would you mind rebasing this one?

@obfuscurity No worries. Rebase done.

Awesome, thanks.


Workaround for "no bin target named serde_derive_tests_no_std"

Fixes #988.

AppVeyor is borked.
http://help.appveyor.com/discussions/problems/7115-curl-cant-download-files-through-ssl


win-2 is in Quarantined state.

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-2 is in Quarantined state.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Replaced the phone with another phone due to battery of the phone is bad.


Add release remediation steps

Add the ability to ensure we can create:

homebrew-bump
cli docs build
all docker images build

Closed via a7fd1e12585da24f313048ab8d66d0ba15496216


bug(ci): test failed occasionally due to log size is significantly large

Describe the bug
https://buildkite.com/risingwavelabs/pull-request/builds/10321#0183e4d2-d59d-4428-beeb-32255fe523da
Encountered another log size is significantly large (2.2M) failure due to timeout when waiting for version update.
2022-10-17T07:32:51.724620Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.724661Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.724701Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.724743Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.724791Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.724832Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>375488 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.725111Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>750912 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.
2022-10-17T07:32:51.725559Z  WARN risingwave_storage::hummock::local_version::local_version_manager: wait_epoch<PHONE_NUMBER>750912 timeout when waiting for version update. pinned_version_id 372, pinned_version_epoch<PHONE_NUMBER>246336.

log path:  src/storage/src/hummock/local_version/local_version_manager.rs +246
 match tokio::time::timeout(Duration::from_secs(10), receiver.changed()).await {
                Err(err) => {
                    // The reason that we need to retry here is batch scan in chain/rearrange_chain
                    // is waiting for an uncommitted epoch carried by the CreateMV barrier, which
                    // can take unbounded time to become committed and propagate
                    // to the CN. We should consider removing the retry as well as wait_epoch for
                    // chain/rearrange_chain if we enforce chain/rearrange_chain to be
                    // scheduled on the same CN with the same distribution as
                    // the upstream MV. See #3845 for more details.
                    tracing::warn!(
                        "wait_epoch {:?} timeout when waiting for version update. pinned_version_id {}, pinned_version_epoch {} err {:?} .",
                        wait_epoch, pinned_version_id, pinned_version_epoch, err
                    );
                    continue;
                }
                Ok(Err(_)) => {
                    return Err(HummockError::wait_epoch("tx dropped"));
                }
                Ok(Ok(_)) => {}
            }

when the log occur , it means that we meet some unexpect stuck about version updating.
To Reproduce
No response
Expected behavior
No response
Additional context
No response

Quick fix: adjusted the timeout from 10s to 30s


ADDED : Strapi Fundamentals Training Course - Community Edition

@ishandeveloper Hi, please help with the failed greeting check?


[bug] super constructors don't work with Custom Elements v1

Buble compiles super() calls to something like ParentClass.call(this).
This causes any and all Custom Elements (v1) to not work because they require the use of new (ES2015 classes), with an error like the following in Chrome:
Uncaught TypeError: Failed to construct 'HTMLElement': Please use the 'new' operator, this DOM object constructor cannot be called as a function

For example, see this output.
Try to run the output code in your browser, and you'll get the error.

Unfortunately, the only solution that would sort of work is to use Refelct.construct, but that is only supported in Edge 12+.

Updated OP link to example: https://buble.surge.sh/#class FooBar extends HTMLElement {
  constructor() {
    super()
    console.log('foo-bar')
  }
}

customElements.define('foo-bar'%2C FooBar)

new FooBar

Hmm, I don't think this is a bug. It's just not possible to compile anything for Custom Elements v1 to ES5 it seems. :(

@justinfagnani made this incredible hack which makes transpiled code work in older browsers! https://github.com/trusktr/custom-elements-1/blob/3984e69c85376d36d83b8a602cc5202c1edd9eed/src/native-shim.js
To be specific, that one makes transpiled code work in IE11. For IE10, let, const, and Map needs to be replaced by ES5 equivalents then it will work.

Sorry for the red flag here!


c2w matrices cannot have 0 translation vector?

Describe the bug
One of my cameras is located at the origin: [tx, ty, tz] = [0,0,0], hence the last column of the matrix in my frame['transform_matrix'] in transforms.json is the column vector [0,0,0,1]. This causes an 'nan' in the poses matrix from the scale factor becoming infinity here.
This might be a bug in the way the scale factor is being computed, since the scale factor should be the value of the [:,3,3]th elements of the c2w matrices, for homogeneous coordinates, but the current implementation drops the last row.

It seems like the pose will only go to nan if all of the cameras a located at the origin (which is not supported by NeRF). I just tested by setting one position to [0,0,0] and the code seemed to run fine. Can you post your transforms.json file?

Closing as this appear stale, feel free to reopen if it is still and issue.


Better multi-cursor support (similar to VSCodeVim) to allow per-cursor motions

It'd be wonderful is this extension could treat each cursor that is generated as its own "context", and allow normal-mode to work with multiple cursors. I have no idea how complicated this would be to execute given the way that it is implemented, but let me explain.
Say I want to use VSCode's built-in functionality to add a cursor below the current line while in normal mode. This feels natural to me, and I use things like cmd+d to add cursors to the next occurrence of a given word all the time (for example). When I have added all the cursors I want, my first thought is "okay time to do a motion", so I execute something like ciw. In VSCodeVim, this behaves as I expect, but in vscode-neovim, it does not.
Here are some gifs that explain things visually a lot better than I can via text. Here's what happens in neovim:

And here's what happens in VSCodeVim:

In order to get to what I think would be "good multi-cursor support", I think that the following would probably need to happen:

Figure out a way to make it so that if you have multiple cursors, the first instance of ESC is interpreted as "go into normal mode".
When in normal mode + multiple cursors, treat each cursor independently, and send motions/commands to those cursors specifically. If I hit ciw (like I did in the gifs about) each cursor should interpret its "context" to execute the motion on the text it has

Lastly, I'd just like to note that this issue is pretty much the only thing keeping me on VSCodeVim at this point. Aside from that, I really like this extension's implementation, and it has a ton of upsides in comparison to VSCodeVim's "emulation-focused" approach (which requires re-inventing the wheel, which feels like an uphill battle). If this extension got first-class multi-cursor support, there'd be nothing holding me back from singing this extension's praises and using it every single day.
Please let me know if you have any questions about the above details!

Looking for similar things.
Found this https://medium.com/@schtoeffel/you-don-t-need-more-than-one-cursor-in-vim-2c44117d51db
Might be useful for you

A lot of things with multiply cursors just can be done faster. For vim / neovim there is awesome vim-visual-multi.

I use vim-visual-multi, and while it (surprisingly) works when used with this extension, I don't think that it's that great when used in a VSCode context when VSCode already has such great multi-cursor support.
As far as arguments about whether or not one needs more than one cursor in vim, I get that some folks have been able to change their workflow to accommodate not having multiple cursors, I'm unfortunately not one of them - I love having multiple cursors, so I'd like to just have a way to make it work the best that it can with VSCode + vscode-neovim 🙂

ooh cool, I haven't use vim-visual-multi yet.
is there any example where vim visual multi is way to user compared to traditional approach?
also curious why vim-visual-multi not working great with VSCode, what's missing?

@muhajirdev vim-visual-multi is great, don't get me wrong - it just feels like it's not ad straightforward/integrated as using VSCodeVim's multi-cursor support (which uses VSCode's native multi-cursor support to get things done). The experience of using it feels like there were compromises made to add it, rather than something that feels native to the tool. It also has come odd issues where it will stop updating text in all of the places where there's a cursor until you hit ESC, which I mean is "fine", but it makes me feel like maybe I messed something up until I've actually "committed" my changes.
In general, just wish this extension would integrate with VSCode's cursors similar to how VSCodeVim does 🙂

I just tried vim-visual-multi. For some reason it doesn't work on my VSCode but work on terminal

vim-visual-multi works for me in vscode, just not well.

Dup of #97

Dup of #97

For anyone who wants to give vim-visual-multi in VSCode a try, make sure to add this to your keybindings.json.
{
        "command": "vscode-neovim.send",
        "key": "ctrl+n",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<c-n>"
}

However, I agree vim-visual-multi does not work optimal in VSCode and feels like a hack. I can see adding true multi-cursor support is a challenge and lets hope @asvetliakov finds the time to give it a try.

For anyone who wants to give vim-visual-multi in VSCode a try, make sure to add this to your keybindings.json.
{
        "command": "vscode-neovim.send",
        "key": "ctrl+n",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<c-n>"
}

However, I agree vim-visual-multi does not work optimal in VSCode and feels like a hack. I can see adding true multi-cursor support is a challenge and lets hope @asvetliakov finds the time to give it a try.

Thanks trkoch, below is to replicate the functionality fully from 'vim-visual-multi' - in case anyone wants to copy and paste
'''
{
"command": "vscode-neovim.send",
"key": "ctrl+n",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
{
"command": "vscode-neovim.send",
"key": "ctrl+up",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
{
"command": "vscode-neovim.send",
"key": "shift+up",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
{
"command": "vscode-neovim.send",
"key": "shift+down",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
{
"command": "vscode-neovim.send",
"key": "shift+left",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
{
"command": "vscode-neovim.send",
"key": "shift+right",
"when": "editorTextFocus && neovim.mode != insert",
"args": ""
},
'''

somehow ctrl+n doesn't work for me and it just moves the cursor to next line.
btw it works fine when I tested it out in nvim itself.

Same...

Thanks for the snippet @echaya, sadly there is no realtime feedback on all cursors like vim-multiple-cursors, any idea why?

Hi, I found that when I press I key to start insert via vim-visual-multi, however the cursor go to start of line, not start of selection which I am expected.

Hi, I found that when I press I key to start insert via vim-visual-multi, however the cursor go to start of line, not start of selection which I am expected.

Same, but for me the multi-cursor works by pressing ⌘ + SHIFT + L instead of ⌘ + SHIFT + I

Thanks trkoch, below is to replicate the functionality fully from vim-visual-multi in case anyone wants to copy and paste
    {
        "command": "vscode-neovim.send",
        "key": "ctrl+n",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<c-n>"
    },
    {
        "command": "vscode-neovim.send",
        "key": "ctrl+up",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<c-up>"
    },
    {
        "command": "vscode-neovim.send",
        "key": "shift+up",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<S-up>"
    },
    {
        "command": "vscode-neovim.send",
        "key": "shift+down",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<S-down>"
    },
    {
        "command": "vscode-neovim.send",
        "key": "shift+left",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<S-left>"
    },
    {
        "command": "vscode-neovim.send",
        "key": "shift+right",
        "when": "editorTextFocus && neovim.mode != insert",
        "args": "<S-right>"
    },

For anyone come here for this solution. Unfortunately, it is broken after version 0.90.0


Zero-Copy IPC between Native Windows Applications and Linux Containers on WSL

Hi,
Is there any support for zero-copy between native Windows applications and Linux containers running on Windows Subsystem for Linux (WSL)?
Use Case: We are using Media Foundation to encode video streams from a camera on a native Windows application. The raw frames are also needed by an AI component running in a Linux container on WSL2. Efficient zero-copy IPC would greatly enhance performance in this scenario.
If there is support, could you provide guidance on how to achieve this? If not, are there any alternative approaches you recommend?
Thanks!

Thank you for your response.
Regarding the need for this feature, we can work around the limitation at this time. For example, instead of using Media Foundation on Windows, we can move that component to WSL 2, forward the camera USB port, and use ffmpeg, or write a proxy on WSL 2 that publishes the data to iceoryx2.
However, our use case is quite complex and this feature would be very helpful.
Our product is an ADAS/AD/CDC system for cars. It would be incredibly useful to have zero-copy pub/sub from different OSs using hypervisor primitives. For now, we can proceed with a single OS, but in the future, it would be more flexible if we could relocate ADAS/AD components to different OSs.
Thanks again for your support!

Our product is an ADAS/AD/CDC system for cars. It would be incredibly useful to have zero-copy pub/sub from different OSs using hypervisor primitives. For now, we can proceed with a single OS, but in the future, it would be more flexible if we could relocate ADAS/AD components to different OSs.

On our roadmap is also exactly this use case. The thing is that the Windows <-> WSL2 communication is realized completely different than a hypervisor communication between for instance QNX <-> Linux, the typical automotive use case. But from an iceoryx2 user perspective you shouldn't care about those details and iceoryx2 will in the future take care of this completely.
The iceoryx2 hypervisor support should be available beginning of next year but it will be a commercial extension to iceoryx2 and not part of the open-source project.


External CSS file in header and footer

I am trying to use en external CSS file in my header and my footer, but it doesn't seems to works. Even bootstrap style that works in the body doesn't work in footer and header. However, the inline CSS works, but I would prefer to have cleaner code and to split the CSS from the HTML.
Thanks.

it doesn't appear to be supported at the moment. What I did, was to use cheerio to grab all the link element, resolve them, and wrap the resolved style inside a <style></style> tag. Be also aware that for now the only style that is grabbed for the header and footer is the first one declared. I am going to open a PR to solve that

You can use this PR to work around the issue :)

This should work now with the newest version

doesn't work for me with the newest version, im referencing bootstrap remotely in the head with a link element.
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.5/css/bootstrap.min.css">


Create "Media Sync" command

Having to move the mouse over to the file selector to trigger media sync is not always desireable. I would like to be able to just do "command + p" to select the command with keyboard and not have my hands leave the keyboard.

@pkej
Thank you!
Fixed below PR.
https://github.com/fnya/media-sync/pull/9


fix #3337

coverage: 92.829% (-0.01%) from 92.843%
when pulling a24b4e7a0a909accf8d253f5576b67a86443943d on antgonza:fix-3337
into 6c77fe42e64fc4e3cc5deb7d5e5552be958d71e9 on qiita-spots:dev.

Tested in qiita-rc and this is how it looks: