Multiplot: add mouse interactivity

Roadmap

[x] Extend Multiplot to support rending onto an existing surface (#4628)
[x] Add Multiplot to IPlotControl (#4628)
[x] Remove Interaction property and related code (#4630, 889b4acfdc3c6f6ee1fa16da110a77effb21e438)
[x] Remove Plot.PlotControl property (#4631)
[ ] Update all UserInputProcessor and Interactivity code to act at the Multiplot level
[ ] Add advanced Plot layout features to Multiplot
[ ] Add multi-plot alignment features to Multiplot
[ ] Showcase new features in the demo application
[ ] Update website documentation

Goal
It should be easier to create plots like this which have multiple stacked Y axes and a shared X axis.

Option 1: ~Refactor the Layout System~

Current Layout System
The Render() method calls Plot.LayoutManager.LayoutEngine.GetLayout() which returns a Layout which has Size (thickness) and Offset (distance from the data area) for each
Users can create their own layout engine that implements ILayoutEngine and replace the default one to customize layouts for complex multi-axis plots
However, since Layout currently only holds size and offset for each panel (not an actual pixel rectangle), full customization cannot be achieved. The present system was designed assuming all layouts are aligned with a single rectangular data area, but for users wishing to create stacked plots (e.g., financial charts that display price on an upper axis and RSI on a lower axis, #4483) this is not the case and the existing layout system cannot lay-out these types of plots
Proposed Solution
Refactor the Layout object and remove these properties:
public readonly Dictionary<IPanel, float> PanelOffsets { get; }
public readonly Dictionary<IPanel, float> PanelSizes { get; }

and replace them with something like:
public readonly Dictionary<IPanel, PixelRect> PanelRectangles { get; }

Potential Challenges

Mouse Interaction Issues: Click-dragging vertically should probably be disabled, otherwise it could cause some very weird behavior... or we have to figure out which axis was clicked and only drag that axis. This is doable, but will be a lot of work (refactoring the mouse hit detection and dragging system) so it should be challenged in a different issue/PR

Multiple Data Area Issues: Supporting stacked axis layouts like this may pose new serious issues because fundamentally this means plots have more than one "DataArea", so clipping may be a serious issue. Similarly, the "frame" around plots won't be drawn by default since frame lines around standard plots are drawn by each of the 4 standard axes.

Option 2: Improve interactive support for Multiplot
Extend the Multiplot which was recently added but does not (yet?) support mouse interactivity. https://scottplot.net/cookbook/5.0/MultiplotRecipes/

The implementation strategy I'm learning toward is as follows:

Extend the existing MultiPlot functionality by making it mouse interactive
Add a MultiPlot property to user controls

The default state is that the MultiPlot contains a single Plot named Plot for backward compatibility
Users can formsPlot1.MultiPlot.AddSubplot() to add additional panels

The entire user input processor must be refactored to support MultiPlot instead of Plot (a very large amount of work)

There's a lot to do to improve mouse interaction support, but this wasn't too bad to get off the ground...
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();

        // the Plot is actually the single Plot inside a Multiplot
        formsPlot1.Plot.Add.Signal(Generate.Sin());

        // subplots can be created and added to the plot
        var plot2 = formsPlot1.Multiplot.AddPlot();
        plot2.Add.Signal(Generate.Cos());
    }
}

This is going really well. This feature may be achieved in a few days instead of a few weeks. As of #4632, a single control may now be used to display multiple interactive plots. This should allow for easier implementation of plots with shared axes and shared layouts because we no longer have to sync settings across multiple plot controls.
// work with the plot like normal
formsPlot1.Plot.Add.Signal(Generate.Sin());

// add a sub-plot and interact with it
var plot2 = formsPlot1.Multiplot.AddPlot();
plot2.Add.Signal(Generate.Cos());

Hi @quantfreedom and @VladislavPustovarov, I want to draw your attention to recent merges which have added multiplot functionality to the main branch of this repository. It's not pushed to NuGet yet, but it is now developed enough that you may experiment with it and let me know if you recommend any API changes.
Specifically, check out the custom multi-plot demo recently added to the WinForms demo application:
https://github.com/ScottPlot/ScottPlot/blob/16728cb48659e67e67d09aa967cf9beba6edf04b/src/ScottPlot5/ScottPlot5 Demos/ScottPlot5 WinForms Demo/Demos/MultiplotCollapsed.cs#L12-L75

@swharden looks great ... is there any way in this demo that you could put the time statistics grid at the bottom and on the y-axis for the time statistics it wouldn't be numbers it would be labels
Is that something that we should be able to do or know how to do because I think at the core of us needing the separation is that time statistics grid
But if that's something that we should be able to do then I'll be happy to figure that out

Hi @quantfreedom,
To achieve an effect similar to the one in your screenshot, setup a Multiplot with two Plot objects we'll call plotPrice and plotGrid. To set Y axis labels for plotGrid you can use the strategy here described on https://scottplot.net/cookbook/5.0/CustomizingTicks/SetTicks/ and it should get you were you want to go! 🚀
There may be some nuance around using right ticks instead of the standard ones... I'll explore that and ensure multiplot properly supports plots using nonstandard axes.

There may be some nuance around using right ticks instead of the standard ones...

Wasn't hard, but I'll update the demo to showcase this style

This feature is amazing, I will try to create the same, but vertically as Overchart does

Based on your suggestion I added a demo application showcasing multiplot controls with resizable subplots:

https://github.com/ScottPlot/ScottPlot/blob/e650e7b5ff89c1ba547191866980c6be72f01016/src/ScottPlot5/ScottPlot5 Demos/ScottPlot5 WinForms Demo/Demos/MultiplotCollapsed.cs#L1-L221


Setting recurse_submodules = true for git_files picker breaks ff

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10.0 (release)
Operating system/version
Ubuntu 20.04
Describe the bug
I use the following configuration in .config/nvim/lua/plugins/telescope.lua so that the git_files picker also looks for files in submodules.
return {
  {
    "nvim-telescope/telescope.nvim",

    opts = {
      pickers = {
        find_files = { follow = true },
        git_files = { show_untracked = false, recurse_submodules = true },
      },
    },
  },
}

However, that breaks the <leader>ff picker when in a git repository, because it gets this error message from git ls-files:
[telescope.builtin.git_files]: Git does not support both --others and --recurse-submodules

This seems to indicate that the LazyVim picker that automatically selects find_files or git_files when in a git repository is somehow passing both options, which is surprising because the show_untracked option for telescope git_files is supposed to be false by default (show_untracked will map to use --others on the git command line).
As you can see above, I also tried to explicitly set the show_untracked option to false, but that does not fix the issue. Using :Telescope find_files works perfectly, and so does :Telescope git_files, so it seems to be coming from the automatic picker of LazyVim.
Steps To Reproduce

Add the telescope.lua plugin configuation file as above
Run vim in a git repository
Attempt to use <leader>ff

Expected Behavior
It should work just as :Telescope find_files.
Repro
No response

LazyVim.pick automatically sets show_untracked to true if you don't pass explicit option via the keymap. So, either make a keymap for LazyVim.pick("auto", { show_untracked = false }) or use Telescope git_files if you don't want that behavior.

I see. Wouldn't it make more sense for LazyVim.pick to follow the options configured by the user for git_files though?

Better wait for maintainer's input regarding that, since I'm just a simple user.

Makes sense. I feel the situation is quite inconsistent right now, because LazyVim.pick will honor the user setting the recurse_submodules option to true, but won't do so for the show_untracked option.

Can you try the following
-- ~/.config/nvim/lua/plugins/core.lua
return {
  {
    "dpetka2001/LazyVim",
    branch = "fix/telescope-show_untracked",
  },
}

Then open Lazy UI, press C to check for updates and U to apply updates. Then restart Neovim and see if you observe the same behavior. If it is to your liking I can submit a PR.

I just tried this and it did not fix the issue somehow? Something weird is going on because I added this to my telescope.lua configuration, and it also did not help:
keys = {
  { "<leader>ff", LazyVim.pick("auto", { show_untracked = false }), desc = "Find Files (Root Dir)" },
  { "<leader>fF", LazyVim.pick("auto", { root = false, show_untracked = false }), desc = "Find Files (cwd)" },
},

Strange for me it didn't show any error with this change, whereas with your initial telescope config it showed the error that you mentioned. Do you get the same error or something different?

Yeah something is off here. I get the exact same error. I have been double-checking to make sure I didn't do anything dumb but everything looks right. Updating with the fix plugin in core.lua worked for sure, I got a bunch of notifications because of file changes, and yet I have the exact same behavior as before. I am also curious to know if overriding the keymaps like I showed in my last message works for you? The reason this doesn't work might be the same reason why pulling in your plugin does not.

Ah, I believe I understand what is happening. I had removed show_untracked = false from my telescope configuration because it shouldn't technically be required since Telescope's default is false. So, I do not know LUA well, or its definition of truthiness and comparisons, but I assume that a nil value doesn't compare to equal with false, so opts.show_untracked ~= false would end up evaluating to true. I wonder if that is why the LazyVim picker also sets show_untracked to true; that would be the case if the opts dict is actually coming from the telescope configuration, but I am just widly guessing here, I need to check the code.

Following up on the above, trying a LUA expression evaluator online, I see that nil ~= false indeed evaluates to true. I am assuming that opts.show_untracked would be nil if the option is not set.

Does it work correctly if you put back show_untracked = false in your Telescope spec?

Yes it does. I just don't think that should be needed.

It should because the default behavior of LazyVim is to set it to true. And in my fix I only check if the user has explicitly set in his Telescope spec show_untracked = false, otherwise if he hasn't I leave the default LazyVim value untouched.

Yeah, that does makes sens if you are trying to preserve the current behavior of LazyVim. I am simply coming at this from another angle though, and saying that LazyVim should honor the Telescope defaults and the user overrides. I guess we will have to see what the maintainer thinks about it.

In other words, I personally feel that the most sensible fix would be to simply remove this line:
      opts.show_untracked = opts.show_untracked ~= false

But then again, I do not know the motivation for this behavior in the first place.

Yeah it's not my place to make decisions. I just tried to work around the problem for setting the value in the user configuration and LazyVim to respect that value and at the same time also respect the default LazyVim value if the user hasn't defined anything in his personal configuration.

Yeah, and thank you for that! By the way, did you get around to trying the keymaps override like I posted earlier? I am confused as to why this isn't working.

and saying that LazyVim should honor the Telescope defaults and the user overrides

Most Neovim distros are quite biased depending on each maintainer's experience and personal preferences. LazyVim already has a lot of default options changed regarding other plugins as well. So, in my personal opinion, I don't agree with it having to respect the plugins' default options. It only has to respect the user's personal configuration settings and overrides. But that's just me and I don't expect everyone to agree with that take.
Regarding the keymaps, no I didn't try that. It's getting really late here, so I'm just gonna call it a night for today. Hopefully, the maintainer will shed more light to clear any questions you might have regarding LazyVim defaults.

Leader ff automatically uses git files or find files. It works as it should. Closing.

@mux I tried the following configuration with default LazyVim configuration and it seems to work (I don't get any error when I press <leader>ff)
  {
    "nvim-telescope/telescope.nvim",
    opts = {
      pickers = {
        find_files = { follow = true },
        git_files = { show_untracked = false, recurse_submodules = true },
      },
    },
    keys = {
      { "<leader>ff", LazyVim.pick("auto", { show_untracked = false }) },
    },
  },

Although I believe that setting show_untracked = false in telescope.pickers.git_files is kind of redundant since you already define it in the keymap itself.

@folke Should I withdraw the PR I made, since the keymap works correctly? The PR is just for taking into account the option set in Telescope's spec in user configuration without the user having to define a different keymap or change the default ones. I don't know what the correct approach should be, should a user prefer such a behavior with show_untracked = false, recurse_submodules = true in Telescope spec.

@folke Thank you for getting back to us. I understand that <leader>ff is supposed to automtically use git_files or find_files depending on the circumstances and that is just fine. The problem I am describing here is that a perfectly legitimate git_files configuration by the user (setting recurse_submodules to true) leads to <leader>ff not working anymore. This leads to two questions:

Why would LazyVim honor the recurse_submodules setting and yet ignore show_untracked, forcing it to true and making  <leader>ff impossible to use unless by remapping it completely? This is arguably a bug and the patch from @dpetka2001 fixes this.
Why is LazyVim forcing show_untracked to true in the first place when telescope defaults to false? I personally feel that it would make more sense to use the same defaults as telescope, although that is clearly debatable.

Even if LazyVim wants to continue forcing show_untracked to true for some reason, it should probably avoid getting in a situation where the picker just doesn't work anymore, by not setting show_untracked to true when recurse_submodules is.
@dpetka2001 You are correct - the keymap setting actually fixes <leader>ff, what I was observing is that selecting Find File in the menu after starting nvim without parameters shows the same problem and error message. I have no idea why would Find File in the menu would behave differently from <leader>ff but that is apparently the case.

@mux Yes on the dashboard menu it wouldn't work. That's because when we define a keymap we only pass the options we specify to that keymap alone. All other keymaps would behave with LazyVim defaults and that's what you observe I believe. If my PR gets accepted, I believe the user won't have to change all possible keymaps. He will just have to define in the Telescope spec show_untracked = false to override the default LazyVim behavior and that's all.
Hopefully the maintainer will accept my PR or maybe he will come up with an even better solution than mine, since my knowledge is quite limited if I may say so.


Feature Request: Popup on unsupported/buggy OS detected

@glouel wanted to know yours/community thoughts on adding a pop-up or some kind of notification for OS versions that are either not supported or partially supported?
This could be during installation or when they launch the options? I think this would cut down on new issues being created, especially when someone doesn't read the readme or issues list and makes a new post? just a suggestion. verbiage could point them to the standalone app to configure?

Pretty good idea especially with the new AerialConfig.app I introduced in 1.4.5 beta2 !
I can't do it at installation (I can't do anything at installation !) but I can at first launch indeed ! Thanks for the idea !

@dVIAN13 I thought I could simply pop a dialog box but those don't really exist on macos, we are already the equivalent of a dialog box (a modal sheet). So right now, maybe adding an extra tab to the panel on those older versions of macOS with a checkbox "I understand" and until you checked that, when you open preferences it will show that pane ?
Maybe it's a bit forceful but I was very reluctant to do the AerialConfig.app thing because I knew most people won't read instructions correctly anyway (we had 2 "done" button issues in 2 days after all !)

@glouel thanks! I see the issues come in too and yeah, I completely agree.. I like the checkbox on a tab.. are you able to dynamically add a tab based on OS version? I don't think the tab with agreeable terms should be the default for users on a supported version.

@dVIAN13 yep that's the goal, only show to people on < mojave, and only until they click the checkbox (I won't show it on upgrade, etc). I think I'll add a direct link to the config app download in the Advanced tab too.
I'll remove the tab for mojave and once they click. Here's what I got so far :

(obviously it won't show on mojave but that's where I made the mockup ;)) Maybe you or @SeanMSmith has opinion about the text ?

@glouel I like the text, but I would defer to @SeanMSmith to validate it. :) or if someone else wants to offer suggestions.
Again, great work, and SUPER fast turn around. We all appreciate the continued work to improve this project.

Is the build not signed ? I was pretty sure I did ?

when I tried to open app, it had the right-click and open

I can confirm redownloading the app. Weird thing is, it shows as signed ! I'll look into it tommorrow, must be something I missed regarding signing, maybe needs an entitlement or something, bah !

no worries! thanks.

Is there no way to make it possible to type into textfields inside a screensaver preference pane on HS? If a particular feature only works on Mojave I'd suggest purely disabling the feature (with a notice) instead of having the user download a separate configuration app. Especially since it's only affecting the custom coordinates feature (which is relatively niche) and not core functionality.

Closing this as I have another workaround coming.


Assessment text is displaying ' instead of an apostrophe

Describe the bug
The text for assessments (tasks.text) is displaying ' instead of an apostrophe (single quote).
To Reproduce
Steps to reproduce the behavior:

Go to an effort / assessment
Select the Notes icon at the top right of the page. Result: assessments will be displayed
See error in the text of the assessment

Expected behavior
An apostrophe (single quote) should be displayed instead of &#39
Screenshots
None
Environment (please complete the following information):
Firefox 78.2.0esr
Additional context
N/A

Duplicate of NCI-Agency/anet#3065


1.13.1 support

Hey!
1.13.1 is #RestrictedToVanilla
1.13.1 hasn't thousand changes also I hope 1.13.1 support comes out before 1.14 ;)

wait for mcp1.13.1

How can I check on the status for MCP? The only thing I've found is the Old MCPatcher for beta Minecraft that was basically just Optifine.

1.13.1 is not restricted to Vanilla and will be arriving within the next few weeks. I'm locking this as there is no need for talk on this.


Improve product_variant filters performance (#8334)

I want to merge this change because it port changes from 3.0 to 3.1
https://github.com/saleor/saleor/pull/8334
* Improve product_variant filters performance.

Drop unused gin index

Co-authored-by: Tomasz Szymanski<EMAIL_ADDRESS>
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3312 Refactor products filtering(except ProductFilter).


NI timetable data research

Holding higher level ticket for NI timetable work - likely to be split into individual tickets.

Source and check NI timetable data - create plan for extracting and processing.

The Translink Group comprises three main operating companies Ulsterbus, Citybus (trading as Metro and Glider) and Northern Ireland Railways. As far as I can tell data for these three companies should cover all public transport routes in NI.
Citybus (Metro and Glider)
Ulsterbus
NI railways
Datasets come in ATCO.CIF format (so hopefully we can use some of the code for GB train timetable extraction) and look to be updated fairly regularly. URLs change, so we cant scrap this page for updates but will have to store a hard copy of a certain download like we do for GB.


Instant.io/rtcConfig

Instant.io, catlobby and cors pages are 404 so can we replace them? Max can you please help us and make the app work again ?

Issues#67


UX: Convert admin leaderboard forms to FormKit

Also use new AdminPageSubheader component on the
index page for the title and buttons.

nitpick and only UI related but this back buttons looks a little bit odd to me

this back buttons looks a little bit odd to me

@jjaffeux this same one is used in a few places now, it's the BackButton component part of the new UI guidelines. Feel free to bring it up in /t/129128 internally


Use dedicated commit ID database instead of Git tags

Motivation:
GitRepository currently uses a tag to maintain the mappings between
revisions and commitIds. It has the following problems:

When using the default refstorage implementation (refdir), it creates
a symbolic link for each revision, which is slow and which creates too
many files in the long term.
When using the 'reftree' refstorage implementation, the disk usage of
the repository increases way too much (more than 2x). Also, it becomes
impossible to browse the repository using vanilla git command
because reftree is a jGit-only extension.

Modifications:

Replace Git tags with CommitIdDatabase which is better suited for
storing and accessing Revision-to-ObjectId mappings
Implement automated migration

Result:

Fixes #100

Codecov Report

Merging #104 into master will increase coverage by 0.28%.
The diff coverage is 72.93%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
+ Coverage   57.06%   57.35%   +0.28%
==========================================
  Files         228      229       +1
  Lines        7828     7937     +109
  Branches      908      927      +19
==========================================
+ Hits         4467     4552      +85
- Misses       2794     2812      +18
- Partials      567      573       +6

Impacted Files
Coverage Δ

...internal/storage/repository/git/GitRepository.java
79.36% <69.23%> (-0.09%)
:arrow_down:

...ernal/storage/repository/git/CommitIdDatabase.java
73.83% <73.83%> (ø)

...internal/replication/ZooKeeperCommandExecutor.java
67.22% <0%> (+0.33%)
:arrow_up:

...erver/internal/mirror/DefaultMirroringService.java
63.29% <0%> (+1.26%)
:arrow_up:

...ava/com/linecorp/centraldogma/common/Revision.java
58.82% <0%> (+4.41%)
:arrow_up:

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 f573548...f77d285. Read the comment docs.

Thanks for reviewing!


Custom event handler are missing from $attrs

Version
3.2.23
Reproduction link
jsfiddle.net/ronzwj6g/
Steps to reproduce

Create a custom component
Apply a listener for a custom event (e.g. @custom="..." and @other="..." as per the example)
Observe how these are listed in $attrs of the component as $attrs.onCustom, $attrs.onOther etc.
Now implement the custom event by adding emits: ['custom'] to the custom component
Observe how $attrs.onCustom is now undefined
As a result, there appears to be no way to detect if a custom event listener is present on a component or not

What is expected?
For custom event listeners to show up in $attrs even if the component has them in its emits array.
What is actually happening?
The listener is undefined

Refer to https://stackoverflow.com/questions/46706737/check-if-a-component-has-an-event-listener-attached-to-it for more details on the use case etc.

they are missing just like declared props are missing.
We have an open issue about this already where we noted that we might consider a way to access cust event listeners like you can access props. Can't find it right now though.
For now, you can declare the listener as a prop with on* prefix (which is how listeners are passed under the hood anyway).

Alright thanks, if you can find the open issue so that I (or anyone finding this in the future) can track the progress for it that would be much appreciated.

Duplicated of https://github.com/vuejs/vue-next/issues/4736#issuecomment-934212424


How to run baby llama?

Hi. Thanks for this port.
I was trying to inference the babyllama, but seems that this port doesn't support it anymore?
Like the original llama2.c compatible stories*.bin model.
How can I do this ?

Currently it doesn't support that model, I'll add a way to run it.


Convert UTF8 String error

When read UTF8 string from mysql json column, result string converting with error:
Original string
[
	{
		"key" : "новый"
	}
]

Bytes
5b0a097b0a0909226b657922203a2022d0bdd0bed0b2d18bd0b9220a097d0a5d

Result string after hibernate request
{"key":"??????????"}
Bytes
7B226B6579223A223F3F3F3F3F3F3F3F3F3F227D

If read varchar column no error occure.

Send a Pull Request which proves the use case with a test case.

Closed as no test case was provided.


[WIP] IP/dht automate

Generated using https://github.com/input-output-hk/pos-prototype-deployment/commit/39848368865a33817d31edab754a80c76eb62edf
This will probably break the installer due to too long line, let's wait on results.

Yeah, there's an issue with too long desktop shortcut. Will push a fix.

Note that now node can't connect anymore to any of the peers, probably a bug in the backend. Pinged @neongreen


OpenFPGA flow parsing LUT INIT params

Is your feature request related to a problem? Please describe.
During adder implementation, if yosys has to synthesise an adder as a hard-macro and then in OpenFPGA flow, this is implemented as LUT+carry_follower where LUT can be a combination of LUT4 with LUT2s or LUT3 with LUT2s. As yosys is implementing it as a hard-macro and it comes as a subckt in openfpga with INIT params that is an alternate to truth table that generally present in blif when luts are represented as .names.
Describe the solution you'd like
OpenFPGA should enable parsing parameters that has information similar to what a truth table provides to .names instances in blif
Describe alternatives you've considered
Alternate solution considered is to implement adder module in yosys in terms of LUT3+LUT2s+carry_follower. However, there is no guarantee in vpr that packers packs them all together so it may not provide a robust solution.
Additional context
Add any other context or screenshots about the feature request here.

Just checked VPR's blif parser. It seems that .init is not supported yet.
Then, it means that we may have to consider a different file format to take the .init parameters.

Just checked VPR's blif parser. It seems that .init is not supported yet.
Then, it means that we may have to consider a different file format to take the .init parameters.

I think we can take either of these two approaches:

Refer to https://docs.verilogtorouting.org/en/latest/vpr/file_formats/#vpr-eblif-cname: .eblif (Extended BLIF) format supports both .param and .attr on subckt. This can enable us to specify the parameters.
Refer to section 6 of doc: https://docs.verilogtorouting.org/en/latest/_downloads/a79b6634b582f56c053f2abad923112a/blif.pdf: Need to write model-reference in the generated blif file. Currently, blif just add subcircuit instantiations in blif but no definition of it. For example, generated blif has these lines:

.model pipelined_8bit_adder
.inputs clk ren wen raddr_0 raddr_1_ raddr_2_ raddr_3_ raddr_4_ raddr_5_ waddr_0_ waddr_1_ waddr_2_ waddr_3_ waddr_4_ waddr_5_ a_0_ a_1_ a_2_ a_3_ a_4_ a_5_ a_6_ b_0_ b_1_ b_2_ b_3_ b_4_ b_5_ b_6_
.outputs q_0_ q_1_ q_2_ q_3_ q_4_ q_5_ q_6_ q_7_
.names $false
.names $true
1
.names $undef
.subckt soft_adder A=a_st1[6] B=b_st1[6] CI=AplusB_soft_adder_O_CO[5] CO=AplusB_soft_adder_O_CO[6] I3=$false O=AplusB[6]
.subckt soft_adder A=a_st1[5] B=b_st1[5] CI=AplusB_soft_adder_O_CO[4] CO=AplusB_soft_adder_O_CO[5] I3=$false O=AplusB[5]_
However, there is no definition of subckt - soft_adder, if we can dump even soft_adder definition in terms of .names and another subckt for carry in the same blif, then openfpga can get the truth table of the luts used in soft_adder.

I think we can take either of these two approaches:

Refer to https://docs.verilogtorouting.org/en/latest/vpr/file_formats/#vpr-eblif-cname: .eblif (Extended BLIF) format supports both .param and .attr on subckt. This can enable us to specify the parameters.
Refer to section 6 of doc: https://docs.verilogtorouting.org/en/latest/_downloads/a79b6634b582f56c053f2abad923112a/blif.pdf: Need to write model-reference in the generated blif file. Currently, blif just add subcircuit instantiations in blif but no definition of it. For example, generated blif has these lines:

.model pipelined_8bit_adder
.inputs clk ren wen raddr_0 raddr_1_ raddr_2_ raddr_3_ raddr_4_ raddr_5_ waddr_0_ waddr_1_ waddr_2_ waddr_3_ waddr_4_ waddr_5_ a_0_ a_1_ a_2_ a_3_ a_4_ a_5_ a_6_ b_0_ b_1_ b_2_ b_3_ b_4_ b_5_ b_6_
.outputs q_0_ q_1_ q_2_ q_3_ q_4_ q_5_ q_6_ q_7_
.names $false
.names $true
1
.names $undef
.subckt soft_adder A=a_st1[6] B=b_st1[6] CI=AplusB_soft_adder_O_CO[5] CO=AplusB_soft_adder_O_CO[6] I3=$false O=AplusB[6]
.subckt soft_adder A=a_st1[5] B=b_st1[5] CI=AplusB_soft_adder_O_CO[4] CO=AplusB_soft_adder_O_CO[5] I3=$false O=AplusB[5]_
However, there is no definition of subckt - soft_adder, if we can dump even soft_adder definition in terms of .names and another subckt for carry in the same blif, then openfpga can get the truth table of the luts used in soft_adder.


[tesseract] Build error

When i want to compile tesseract with vcpkg i get this error:
Copy issue body from C:/Users/Desktop/vcpkg-master/installed/vcpkg/issue_body.md
Computing installation plan...
The following packages will be built and installed:
tesseract:x86-windows -> 5.3.1#1
Detecting compiler hash for triplet x86-windows...
Restored 0 package(s) from C:\Users\AppData\Local\vcpkg\archives in 129 us. Use --debug to see more details.
Installing 1/1 tesseract:x86-windows...
Building tesseract:x86-windows...
-- Using cached tesseract-ocr-tesseract-5.3.1.tar.gz.
-- Cleaning sources at C:/Users/Desktop/vcpkg-master/buildtrees/tesseract/src/5.3.1-b7e70178d8.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Users/Desktop/vcpkg-master/downloads/tesseract-ocr-tesseract-5.3.1.tar.gz
-- Applying patch fix_static_link_icu.patch
-- Applying patch fix-aarch64-mfpu-not-available.patch
-- Using source at C:/Users/Desktop/vcpkg-master/buildtrees/tesseract/src/5.3.1-b7e70178d8.clean
-- Configuring x86-windows
-- Building x86-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: "C:/Program Files/CMake/bin/cmake.exe" --build . --config Debug --target install -- -v -j13
Working Directory: C:/Users/Desktop/vcpkg-master/buildtrees/tesseract/x86-windows-dbg
See logs for more information:
C:\Users\Desktop\vcpkg-master\buildtrees\tesseract\install-x86-windows-dbg-out.log
Call Stack (most recent call first):
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
ports/tesseract/portfile.cmake:36 (vcpkg_cmake_install)
scripts/ports.cmake:147 (include)
error: building tesseract:x86-windows failed with: BUILD_FAILED
Elapsed time to handle tesseract:x86-windows: 12 s
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+tesseract
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[tesseract]+Build+error&body=Copy+issue+body+from+C%3A%2FUsers%2F%2FDesktop%2Fvcpkg-master%2Finstalled%2Fvcpkg%2Fissue_body.md

@emxcode, Thanks for posting this issue, please provide error log:
C:\Users\Desktop\vcpkg-master\buildtrees\tesseract\install-x86-windows-dbg-out.log

I could install this port successfully with the latest vcpkg 2023-08-09-9990a4c9026811a312cb2af78bf77f3d9d288416, could you please run following commands in your vcpkg folder:
1. git pull
2. .\bootstrap-vcpkg.bat
3. .\vcpkg install tesseract:x86-windows --binarysource=clear

Ping @emxcode  for response.

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!

Hello dear friend I'm sorry for late answer; i move my project to Linux and i tried to compile my code on different OS. i did not have problem with compiling tesseract in Ubuntu but tesseract does not execute my command correctly with Persian language if you have solution for me please help me.
i get unknown character as output when i use images.....

@emxcode, Could you send me your error log? Or you can take a screenshot, thanks


Small tweaks to prepare for more ingestion

improve the filtering of directory entries so that it applies to validation, and preprocessing
strip IDs during ingest to ensure the preprocess phase works correctly
add initial handling of withdrawn

ignore new reports that are already withdrawn
fail when encountering a withdrawn report

This is related to #61


fix #229 allow date.created to get creation date

@Guts
please check

Hi @copdips, thanks for the PR!
Can you add a test case related to this change please?

@Guts
Yes, test has been added, please approve the workflows.

Can you also add a fixture file (mkdocs conf and markdwon page with your use cas) which are used as a kind of integration test?
And they are also useful to five examples and remember every use cases.

Can you also add a fixture file (mkdocs conf and markdwon page with your use cas) which are used as a kind of integration test?
And they are also useful to five examples and remember every use cases.

an example has been given in the doc now.


add new recipe to export to deephub format

[ch 73795]
only add object detection recipe for now, but we might add one also for image classif in the future.
This recipe was first implemented as part of plugin https://github.com/dataiku/dss-plugin-image-annotations-to-dataset but is finally moved here to regroup everything linked to labeling plugin.

[sc-73795]

Works well. Only a few comments.
Can we remove all "comments" (//) from recipe.json? The json format does not support comments.

removed all the description comments. I'd like to keep the todo comment if that's ok with you? it doesn't seem to impact the execution


contrib/python/test/test_tunnel: Fix -nNT -> -nNTq

Catching the tests up with 60427ab3 (#986) to avoid non-fatal smoketest failures like:
  ======================================================================
  FAIL: test_tunnel (test.test_tunnel.TestTunnel)
  ----------------------------------------------------------------------
  Traceback (most recent call last):
    File "/usr/lib64/python3.6/unittest/mock.py", line 1179, in patched
      return func(*args, **keywargs)
    File "/go/src/github.com/projectatomic/libpod/contrib/python/test/test_tunnel.py", line 79, in test_tunnel
      mock_Popen.assert_called_once_with(cmd, close_fds=True)
    File "/usr/lib64/python3.6/unittest/mock.py", line 825, in assert_called_once_with
      return self.assert_called_with(*args, **kwargs)
    File "/usr/lib64/python3.6/unittest/mock.py", line 814, in assert_called_with
      raise AssertionError(_error_message()) from cause
  AssertionError: Expected call: Popen(['ssh', '-nNT', '-L', '/tmp/user/socket:/run/podman/socket', '-i', '~/.ssh/id_rsa', 'ssh://user@hostname'], close_fds=True)
  Actual call: Popen(['ssh', '-nNTq', '-L', '/tmp/user/socket:/run/podman/socket', '-i', '~/.ssh/id_rsa', 'ssh://user@hostname'], close_fds=True)

Maybe these Python tests should be fatal to protect against this sort of thing in the future?  As it stands there are a number of Python failures (including this one) which make it hard to find the fatal errors when the smoketest CI finds one.

Possibly related comment from #986:

-q should only be added if not PODMAN_DEBUG

@jwhonce, I don't have a preference one way or the other, as long as the tests pass ;).

@baude @jwhonce PTAL

Rebased onto master with 926f54ee -> a490fc5e to pick up #1044.

LGTM
@jwhonce @baude PTAL

LGTM and this one seems to have gotten happy green test buttons before the tests systems went south.

Long as Homu doesn't need to actually test it. With Alpine inaccessible, all our tests are failing right now, no matter where they're run

@rh-atomic-bot r+


HDDS-9943. TokenRenewer should close OzoneClient after use

What changes were proposed in this pull request?
Ozone's TokenRenewer implementations do not close OzoneClient after use.
Both O3FS and OFS define their own TokenRenewer implementation.
Hadoop finds implementations via ServiceLoader, and uses the first one that handles the kind of token being used:
https://github.com/apache/hadoop/blob/7db9895000860605a66dd6403005b0c61a6ed744/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/Token.java#L454-L470
The two implementations are the same (not FileSystem-specific), kind is the same for both, so we can remove the duplicate one.
https://issues.apache.org/jira/browse/HDDS-9943
How was this patch tested?
Ran ozonesecure-mr/test.sh, checked OM audit log for delegation token operations:
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=GET_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862, renewer=rm} | ret=SUCCESS |
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=RENEW_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862, renewer=rm} | ret=SUCCESS |
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=GET_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862, renewer=rm} | ret=SUCCESS |
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=RENEW_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862, renewer=rm} | ret=SUCCESS |
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=CANCEL_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862} | ret=SUCCESS |
...  | OMAudit |<EMAIL_ADDRESS>| ip=<IP_ADDRESS> | op=CANCEL_DELEGATION_TOKEN {kind=OzoneToken, service=<IP_ADDRESS>:9862} | ret=SUCCESS |

CI:
https://github.com/adoroszlai/ozone/actions/runs/7237484751

Thanks @jojochuang for the review.


Fix flaky e2e error.

  Expected success, but got an error:
      <*errors.errorString | 0xc000c8c110>:
      assert managed cluster autoimport-test-vj4vh6 import secret applied failed
      {
          s: "assert managed cluster autoimport-test-vj4vh6 import secret applied failed",
      }
  In [It] at: /home/runner/work/managedcluster-import-controller/managedcluster-import-controller/test/e2e/e2e_suite_test.go:557 @ 08/06/24 15:25:34.43

The root cause is that: in the auto-import-controller, we are using 2 managedcluster informer:
The first informer used to trigger Reconcile is from main.go:
https://github.com/stolostron/managedcluster-import-controller/blob/716acc3a8e6b0ee2f585e10ef0ae47c8bfd165f7/cmd/manager/main.go#L212
https://github.com/stolostron/managedcluster-import-controller/blob/716acc3a8e6b0ee2f585e10ef0ae47c8bfd165f7/pkg/controller/autoimport/manager.go#L99-L100
The second informer which also provide the ability of r.client(the mgr.GetClient()) is because anther controller Watches the managedcluster resource:
https://github.com/stolostron/managedcluster-import-controller/blob/716acc3a8e6b0ee2f585e10ef0ae47c8bfd165f7/pkg/controller/selfmanagedcluster/manager.go#L64-L65
When first informer trigger reconcile, it doesn't generatee the second informer's cache get updated.
That where the inconsistent happens.

/assign @zhujian7

@qiujian16  The latest logger removed Warn funcs, I raised log level of all Info added in this time to 5.

/approve

/lgtm


Fix development instructions in README.md

Without WP_ENV defined, we get white screen on php errors.
This constant is used in config/app.php.
fixes #2656

I don’t think this needs to be in the Sage README since it is more so a concern of the WordPress install (see Bedrock or a regular WordPress install).


error "... referenced here", I don't know what to do

Describe your problem here.
When I compile my mod, I get that error and I don't know how to fix it.
I have all the libraries I need updated, I have everything I requested, I even have the Psych engine and the original FNF, the only thing I didn't do because I don't know if it had to be done, is if I had to put the Psych engine folder in the FNF folder or the other way around, please help me )=
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

screenshot pls

Your screenshot is not working


[Outlook] (event-based activation) Add note about loopback exemption

At present, users running this type of add-in on Win32 through yo office may need to include a loopback exemption.

Docs Build status updates of commit 32d9403:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

[Warning-ServiceAccountEnabledButInsufficientPermissionWarning] Your repository https://github.com/OfficeDev/office-js-docs-pr has enabled Docs Build service account. It seems that at least one of the account, opbld38, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master) and then retry the build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit d1357ae:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 2badd22:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/custom-functions-troubleshooting.md
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@alison-mk Can you review the changes to the custom functions page? Thanks.


Issue #94: Fix constraint violation in anchor generation

This is a fix for issue #94
It seems this is a problem primarily if one has categorical variables AND either they are part of the constraint(s) formulae or you only have constraints on other variables but some categorical variables are entered into the space list before them.
I have just edited the constraints check to use the zipped inputs so that the correct constraint check is done.
This fix was previously shared in the thread for the issue - SheffieldML#94.
Folarin

Codecov Report

Merging #271 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
+ Coverage   77.45%   77.46%   +<.01%
==========================================
  Files          97       97
  Lines        4631     4633       +2
==========================================
+ Hits         3587     3589       +2
  Misses       1044     1044

Impacted Files
Coverage Δ

GPyOpt/methods/bayesian_optimization.py
88.88% <ø> (ø)
:arrow_up:

GPyOpt/acquisitions/base.py
91.17% <100%> (+0.55%)
:arrow_up:

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 249b8ff...9e20ff2. Read the comment docs.

Hi @apaleyes,
Will these tests suffice?


raft: unembed etcd/raft.Node

Such embedding hurts readability and also exposes many public methods
from underlying type.

Current coverage is 54.83% (diff: 72.72%)

Merging #1603 into master will decrease coverage by 0.02%

@@             master      #1603   diff @@
==========================================
  Files            84         84
  Lines         14174      14176     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           7776       7774     -2
  Misses         5381       5381
- Partials       1017       1021     +4

Powered by Codecov. Last update c7ade46...18d5d6a

@aaronlehmann Updated

CI is failing

@aaronlehmann fixed!

LGTM


"gzip: stdin: not in gzip format"

In step 5 of Usage:

Download repl.deploy to the root of your repl -- Open the shell, and run
curl -sL https://repl-deploy.vercel.app/ -o repl.deploy.tar.gz
tar xf repl.deploy.tar.gz
rm repl.deploy.tar.gz

tar xf repl.deploy.tar.gz returns:
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
I feel like this is more of a repl.it problem, but I'll still open and issue here just in case...

Hey, those instructions are outdated, you should be doing
curl -sL https://repl-deploy.vercel.app/ -o repl.deploy
chmod +x ./repl.deploy

as newer releases (since the rust rewrite) are no longer gzipped.
I've updated the README, thanks for letting me know!


CORS support?

Wondering if there is built-in cors support

The sensible tack onto this will be #51


add KasmVNC module

[ ] Test
[ ] Add screenshot

Closes #1

This only works on Debian based workspaces for now.


Add tests for ambiguous constructor selection to TestSpecification

We have a bunch of tests around constructor resolution in ServiceTest. If the intent is to rely on the resolution behavior in code (https://github.com/aspnet/HttpAbstractions/issues/475), we need to add it as part of the test specification.

Out of curiosity, what's our feedback mechanism for these specs? Say we add these tests and some DI systems fail. How do we find out and what do we about it?

They file a bug here :smile:

Oh, I'd make sure you guys hear it :wink:

@pranavkm , see? :smile:


Is an appellation unique to its bearer?

There is two way of understanding appellations of actors:

On the one hand, we could see the name of someone as unique to that person
On the other hand, this name can be shared between homonyms, and is therefore not unique.

To exemplify this, we could have two homonyms, Person A, called John Doe and born in Montreal and Person B, also called John Doe and born in Ottawa. In the first conceptualization, the appellation of person A is unique to him, and therefore the URI of each appellation John Doe is unique (mic.ca/uri/appellation/1234 and mic.ca/uri/appellation/6573). In the second conceptualization, because person A and B are being called with the same first and last name, they both share the same appellation that has just one URI (mic.ca/uri/appellation/john_doe).

What are the benefits of linking people by their name? I'm afraid that it would create meaningless links?
What are the benefits of not linking people by their name?

I think from a social standpoint we do share names. For example, even though parents individually name their children, first names have trends and are linked to socio-historical elements such as celebrities or rulers naming their own children, beloved characters in works of fiction being used as inspiration, etc. As such, humans are assigning names en masse in a way and this act is closely tied to their place in society.
Cognitively speaking, we use names uniquely I believe. We might know many Johns, but if we do not specify which one we are talking about what comes to mind is an aggregation of all the Johns we know, not a class of Johns with the single characteristic of sharing a name.
So generally speaking both conceptualizations are accurate with the individualized URI being closer to cognitive reality and the shared one closer to social reality. I am not sure what would be the implications of each pattern from an efficiency point of view, however?
All this is to give the background to my view of this: if linking people by their name is mostly a social evaluation of trends and patterns, it seems more statistical in nature than semantic. Moreover, the semantics are more closely resembling cognition than social patterns I believe (from a functional standpoint). As such, and I have very limited knowledge on this, it would seem to me that using individualized URIs would be closer to how we function in everyday life. In other words, the absence of a link between two Johns illustrates how they do not put forth an "intentional sharing" of their name but an incidental one.

By discussing with Ludovic, we realized that the E41 Appellation of an Actor receives the specific E55 Type of being preferred or non-preferred.
Therefor, this E41 Appellation has to be specific to one actor, as the same name could be preferred for Person A and non-preferred for Person B. If I'm not mistaken, that means that we have to go with option 1.

interesting discussion. The correct pattern is 2. There would be no point to generate an appellation node if you did not say that one name was one name. The point of an appellation node is to identify the name in itself independent of the entity it names. If you land on a name appellation node/record then you want to know the things that it names. You would lose that functionality by making my 'George Bruseker' different than that of the other 'George Bruseker'. It is true that you put a preferred identifier type on a name sometimes. Since the data will be loaded in a named graph you can see whose preference via the named graph. To be even more specific you should even create an attribution node so that you can say who says it is preferred. If you are not interested in the name itself then you can just put an rdfs label on the actor/object itself and save yourself trouble (though I wouldn't recommend it).

@Habennin indeed you are quite right, having a separate appellation for each actor is anti-lod logic.
I think both solutions : the named graph and the E13 Attribute assignment should be tested, to see which one is easier to implement and which one gives the best answers to requests.
We would still need to be careful, as some names could have different parts. In the case of @KarineLeonardBrouillet "Léonard Brouillet" is her last name, but another Karine Léonard Brouillet could have "Léonard" as a middle name, which means it would be two separate appellations.

I think you'll find that in a lot of data about names and identifiers you have specific documentation of who said it within your source data. In that case it is a case of 'quoted speech', so you would most easily use E13

From the discussion during the meeting of the 20th of December, we've decided to chose the proposition 1 for the TM 2.0, where the appellation of an actor is unique to it bearer, as it seems easier to manage at first.
But we will still leave this discussion open and think more about this issue for the next version of the TM.

I read the thread once again more carefully and I would like to clarify something. For the moment, we keep option 1 because it was easier to implement according to our delay. That said, I highly recommend to go with option 2 for the next version. Some reasons:

We want to describe the appellation. At least, we need to state the preference and the type (e.g. "preferred" and "last name"). So the rdfs:label is clearly not enough here.

In my opinion, the social reality vs cognitive reality question raised by @KarineLeonardBrouillet is really interesting and I would probably argue that social reality "includes" in a way the cognitive reality. Even if someone refers to a name to designate a unique person and he is not aware of all the social construct behind it; the social reality is still valid. So your parents can name you "John" just because they like this name but there is still a social construct behind it and I think it's mandatory to keep this visible in our model.

For the appellation type that can be confusing, like "Léonard", I think our model is able to handle the different options. Also, we could reconcile those names with other controlled vocabularies. For instance, Wikidata offers two different URIs for "Léonard": male given name and family name.

I also agree with @Habennin, we should see the graph as the "validity context" of the embedded data. So the "preferred" type is in the context of the graph (the institution) and not the whole universe. I think we should also use E13_Attribute Assignment since it doesn't seem so easy to retrieve data at the graph level (see #38)

In my experience appellations are not shared. Museums don't provide them shared, they might have per-use characteristics (preferred/non-preferred, official/nickname, or why not even male/female?)...
the First/Last name breakdown/distinction is dependent on Western culture. Arabic, Japanese, American Indian/ First Nations, even Hungarian have very different rules, and so I'm not sure that distinction is very useful or universally applicable
As @ilip wrote, Wikidata has Project Names that creates items for first/last names, and links cross-culture name forms like John-Ivan... but I don't think that's part of CIC's scope

So I think option 1 is better for CIC

Thanks @VladimirAlexiev,
For the moment, our model allows to breakdown an appellation in different parts. However, the part type is managed using a vocabulary. Thank you for the advice about first/last name, we will take care of recommending a proper set of terms.
For the moment we will go with option 1. I don't think we are going against the scope note by doing so. However, since @Habennin has mentioned that we should go with option 2, this is something we will explore in the next version.

I also think option 1 doesn't make E41_Appellation useless and its use pointless. You often need to record title details (eg official vs nickname, first vs last name, preferred vs non-preferred name) and you need a node for that.
I believe E13 and Named graphs are a distraction in this case because often preferred/non-preferred is "universal knowledge" and you don't know or need to record who expressed that preference. And just because a museum recorded it, doesn't necessarily mean they prefer it.
I believe this is unlike AAT label props contributor, contributorPreferred, contributorNonPreferred etc because those are used for concepts having many terms, and often the preference of one or another term is a matter of historic tradition or "personal preference". I believe it's not so about person names.
To record more details about an Appellation, you can use frbroo's Name Use class.

By reading this issue again prior to the Semantic Committee meeting, it seems that the shared appellation is the most semantic. But two issues seems of importance:
What to do with the Preference Type?
As stated above, some appellations are preferred to some actors, and some are non preferred. The preference of an appellation is directly linked to the appellation in the TM 2.0.
If the appellations are shared by actors, that mean that some appellations would have multiple preference types. How to differenciate to which actor the preference type is linked to?
Let's take an example to illustrate that. Actor number 64862 is called John Doe. This appellation is therefore the preferred appellation for that person. Actor number 11603 is called John William Doe, but can some time be refered to as John Doe. Therefore, the appellation John Doe is a non-preferred one for that other person.
In this example, the appellation John Doe will have to E55 Type linked to it, one is preferred, the other one non-preferred.
The question is: How to specify that the Preferred Type is for the Person 64862 and the Non-Preferred Type is for Person 11603?
I see two solutions for that:

By using Named Graphs, the preferred type would only be present in the graph containing the appropriate Person. Is this solution viable? I guess this solution would impact the SPARQL requests, as it will be mendatory to add the Named Graph associated to the Preference Type. Moreover, this solution would mean that the Named Graph would not be at the institution level, but at the record level (see issue #45), solution I personally don't like.

Change the E55 Type location from the Appellation to another instance. The only option would be the F52 Name Use Activity. Would that make sens?

Another solution, of course, would be to NOT share the appellations between actors.
Link the Name Use Activity to the Actor
In the TM 2.0, the F52 Name Use Activity is linked to the appellation with the property R64 used name, but there is no link between the F52 Name Use Activity and the E39 Actor, as you can see here:

If we use shared appellation between actors, there is nothing that links the E39 Actor to its F52 Name Use Activity (which is not shared). It seems that the property R63 named between the the two entities is necessary. The pattern would therefore be the following:

Is this addition useful or necessary? I think it is, where or not the appelaltion is shared.

After discussing this with members of CHIN's semantic committee, we have come to the conclusion that there is no immediate and foreseeable use to the use of a "shared appellation" approach to the application of the appellation. That said, it might be relevant in the future for certain disciplines such as genealogy. For the moment, CHIN will continue implementing the "individual appellation" approach, although this position might be reassessed if relevant use cases establish the need to do so.

@KarineLeonardBrouillet glad you took that decision!


Make sure uvs for the texture array-based fragment shader are not interpolated

Currently, we rely on casting uv.z as i32. Does this have any weird edge cases? Better thing would be to add in a flag on the uv that specifies it should not be interpolated. See here for an example of how to do this: https://github.com/bzm3r/doug_renderers/blob/428a400d85ba989e5f78d5d7b4eb9686334bc12c/src/shaders/vpull.wgsl#L43

Turns out this is indeed an issue. It happens when zooming the projection matrix in and out sometimes a tile uv.z can end up being 0.9999 because of interpolation and since we cast to i32 it drops the decimals leaving us with zero instead of one.


[T3A4][T13-A1] Aloisius Stephen

Add Exception to warn user if address.txt file is deleted

@aloisiusStephen
Some comments added. Please ack & close the PR after reading comments.

Thanks for the comments!
i tried carrying out the check in save() method however i was faced with issues on the catching of Deletion Exceptions in the functions which calls save(), didn't have enough time to identify and fix the issue.


fix(videos): videos not working on iOS 13 and throwing a fatal crash when selected

Motivation (required)
Fixes #1742 where the app crashes on iOS 13 when picking a video.
Did some additional investigation on this issue to the cause and additionally into why it caused a fatal crash and found two things.

On iOS 13.5 a NSError is set to the error variable moveItemAtURL while on iOS 14+, the error is set to Nil. The current check on the error if (error) matches incorrectly and ends up returning nil from the function when it should instead proceed. The original PR for this issue corrected this conditional error check.

The part of this issue that resulted in a fatal crash to the app is because the self.callback method requires an "errorMessage", which in the particular path resulted in the error of error.localizedFailureReason being nil. The React Native error handler threw errors attempt to insert nil object from objects[1] and then `Terminating app due to uncaught exception NSInvalidArgumentException'.

The Fix
The fix in this PR adds the additional error conditional check for nil around copying video. For the fatal crash, the fix also  includes error message handling to prevent a hard crash on the error callback when localizedFailureReason in not available.
Test Plan (required)

To test the error handling around the fatal crash, remove the && *error conditional check in the PR, but keep the error handling message fix to allow the error callback to be called. On iOS 13, you should see an error being returned in the javascript output.

To test the full fix on iOS 13, upload a video and see the expected results in the example app output.

To make sure iOS 14+ still works, run the app on iOS 14 or 15 and verify uploading a video results in the expected output in the example app.

:tada: This PR is included in version 4.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Thoughts on Dependent Rules vs a Must() extension method

Hi Jeremy,
Below, I'm trying to solve the same problem two different ways, what are your thoughts?
internal class CreditCardVerification : AbstractValidator<CreditCardVerification>
{
    internal CreditCardVerification()
    {
        RuleFor(x => x.RiskData)
            .Must(NotBeAHighRiskCard)
            .WithMessage("Sorry, but we currently cannot accept your credit card. Please try a different credit card.")
            .DependentRules(() =>
            {
                RuleFor(x => x.CreditCard.CardType)
                    .Must(BeAcceptedCreditCard)
                    .WithMessage(x => "Sorry, but we currently do not accept your card.")
                    .DependentRules(() =>
                    {
                        RuleFor(x => x.CreditCard.CountryOfIssuance)
                            .Must(BeIssuedInValidCountry).WithMessage(x => "Sorry, but we currently do not accept payments from credit cards issued in this country.")
                            .DependentRules(() =>
                            {
                                RuleFor(x => x.CreditCard.Prepaid)
                                    .Must(NotBePrepaid).WithMessage("Prepaid cards are not accepted.");
                            });
                    });
            });
    }
}

OR

internal class CreditCardVerification : AbstractValidator<CreditCardVerification>
{
    internal CreditCardVerification()
    {
        // Defining a rule for Id, but I really don't care about id
        RuleFor(o => o.Id)
            .Cascade(CascadeMode.StopOnFirstFailure)
            .Must(NotBeAHighRiskCard).WithMessage("Sorry, but we currently cannot accept your credit card. Please try a different credit card.")
            .Must(BeAcceptedCreditCard).WithMessage(x => "Sorry, but we currently do not accept your card.")
            .Must(BeIssuedInValidCountry).WithMessage(x => "Sorry, but we currently do not accept payments from credit cards issued in this country.")
            .Must(NotBePrepaid).WithMessage("Prepaid cards are not accepted.")
    }
}

Second approach is definitely cleaner (I’m not a fan of DependentRules), but leaves the validation results associated with the wrong property. (You could also use RuleFor(x => x) as an alternative to associating with the ID)

Thank you Jeremy. Just though I'd get your input. I also like the second approach better, a lot more readable. Thanks again!


Set up project Pull request

In this part of the project I have setup webpack, jest and linters.

Hi @HERMON-1995 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Linters properly setup and passing :heavy_check_mark:
Good use of GitHub flow :heavy_check_mark:

Cheers and Happy coding!👏👏👏

Thank you so much for your time and consideration in reviewing my work. 😊


Fixed build and all tests pass in Travis with Rust nightly-1.10

https://travis-ci.org/yacoder/brainfuck_macros/builds/131853974

Thanks for the pull request, and welcome! You should hear from @huonw (or someone else) soon.

Hey @huonw, would you like to accept the fix? :)


Different waveform on mobile and desktop

I have integrated wavesurfer in my project, but it renders different wave design on desktop and mobile
see attachment

desktop and mobile

Why not mention what platforms and versions?

Maybe try changing pixelRatio?

Why not mention what platforms and versions?

window 10, Chrome 85, wavesurfer.js 4.1.1

And your wavesurfer config? sigh, have you read the ticket instructions?

var waveOptions = {
container: '#waveform',
barWidth: 1,
hideScrollbar: true,
height: 90,
cursorWidth: 0,
minPxPerSec: 2,
responsive: true,
waveColor: '#717171',
barGap: 2,
normalize: true
}

what mobile platform/versions etc?

I can reproduce this with barWidth: 1. Not a platform problem I think. The size of the canvas seems to affect it.
Maybe related?
https://stackoverflow.com/questions/8696631/canvas-drawings-like-lines-are-blurry


fix failed test

Issue

close #776

Overview (Required)

In the test for reading JSON files, the test was failing because the file could not be found when the target was iOS. According to this slide, since the working directory differs by platform, hardcoding the path can prevent resource retrieval on one of the platforms. Therefore, we implemented a utility function in core:testing for reading files specific to each platform and abstracted the file reading method on the consumer side.

Acceptance conditions

Passing ProfileCardJsonTest with iOS

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before
After

Movie (Optional)

Before
After

I don't think we have a CI set up for running this test, which means it could break at any time. Could you add it? I think running just one task is enough.


All Creators should boot into the file system first

All Creators should boot into the file system first, even if a game is loaded. If a game, exists, boot into the save tool.

This has been fixed.


Show subprojects in search results

Currently, file/section/etc searches show results from a queried project, but it would also be helpful to display results from subprojects. This could easily be performed by appending the subproject slugs/ids to the search queries that are searching for a specific project. Display would be altered comparing the project slug, and if different, noting "from project X" in a fashion similar to the way domain objects are displayed separately in project search.
Routing
The index routing will need to be changed here as well, as it is currently per-project, which will limit project responses. Projects can be routed to their root parent projects, to keep sub/super-projects all on the same shard. However, this creates some difficulty querying from inside a subproject:

Which route do we use when a project has multiple parent projects and a query is requested directly for this project?
Do we update the indexes for each route when a project has multiple parent projects?

Indexing should be updated to respect many to many relationships between projects, and multiple indexes will be updated when updating search indexes.

[x] Update search queries to include subprojects, if any
[x] Also include superprojects, if any?
[x] Update routing on indexing, route to parent or root parent project
[x] Display results in search output with project designation
[x] Correct links to subproject pages in search results

This relies on #1808 and rtfd/readthedocs-sphinx-ext#16
In action:

LGTM :+1:

I addressed a few issues and rebased this. #1808 and this are unblocked now.

Can someone please confirm that it only work with sphinx and not with mkdocs?

@pgiraud This is a pretty old issue, and your comment is likely to be lost, I think. I would encourage opening a new one and linking here. Thanks.


Fix oc-mirror unit tests

Since oc-mirror is not installed by default, we need to assert
the command args without the command itself

/retest

/retest

/lgtm

/approve

/approve


Add support for configuring authenticated origin pulls

high level schema:
// Global AOP
resource "cloudflare_authenticated_origin_pulls" "foobar" {
  zone_id = ""
  enabled = true
}

resource "cloudflare_authenticated_origin_pulls_certificate" "foo" {
  certificate = ""
  private_key = ""
  type = ""
}

// Per Zone AOP
resource "cloudflare_authenticated_origin_pulls" "foobar" {
  zone_id = ""
  authenticated_origin_pulls_certificate = authenticated_origin_pulls_certificate.foo.id
  enabled = true
}

// Per Hostname AOP
resource "cloudflare_authenticated_origin_pulls" "foobar" {
  zone_id = ""
  hostname = "foobar.com"
  authenticated_origin_pulls_certificate = authenticated_origin_pulls_certificate.foo.id
  enabled = true
}

Through these two resources, and combinations of their attributes we can allow for all AOP endpoints to be hit based on the data provided.
Closes #736

Tests will fail until https://github.com/cloudflare/cloudflare-go/pull/511 is merged

Oh and good catch about docs I completely forgot they were a thing -- plus this resource is pretty confusing without some supporting docs about how to use it!

I will also look into importing as there is a use case where one has already configured all of the different types of AOP / has uploaded certificates that they'd like to import into terraform.

@jacobbednarz Ready for review again! Though I'm not sure what's going on with the website build 🤔

Though I'm not sure what's going on with the website build

If you pull in the latest master, the website-build steps should be fixed again with a temporary workaround while the new documentation stuff is sorted out for the registry.

Hope you don't mind @dhaynespls but I pushed up a couple of commits that addressed some issues in the integration suite where parallelisation of the certificate creation tests collided as there is a one per zone restriction on it. With the fixes, the integration suite for these changes is ✅

@patryk @jacobbednarz this MR appears to have removed the cloudflare_custom_hostname resource.

Good catch thank you @cdloh. I went ahead and merged as it's just a single line change in provider.go.


[Feature]: Multiple Repository

If user can add additional repository in sub folder to be synced separately and the settings of each repo can be separated from each others, it would be great. For example, currently I have 5 vaults to write note about different stuff, but I want them to use the same .obsidian setting. My temporary way to handle is to create a copy of this extension (named obsidian-git-copy) locally.

My temporary way to handle is to create a copy of this extension (named obsidian-git-copy) locally.

Is there a step by step guide for this? I copied the plugin folder from obsidian-git to obsidian-git-personal, but that doesn't seem enough because reloading plugins doesn't show me a second plugin.
I then changed the name in obsidian-git-personal/manifest.json from "Git" to "Git (Personal)" but then after reloading the plugins on Obsidian, the original plugin was renamed instead of showing a second plugin. I guessed it might be getting deduplicated by id and changed the id in obsidian-git-personal/manifest.json from obsidian-git to obsidian-git-personal, then the UI starts showing both plugins but when I try to enable the copy, it fails with failed to load obsidian-git-personal.

@villasv these two extensions seem to need to use different git instance. In my usage, the original extension use a node version of git, and the copied extension use a git.exe in my windows system. And you also need to change name of some variables in the .js files correspondingly. See if this solve your question. Or maybe I will write a guide for this in the weekend.
However, after seeing #773 , I realized maybe git submodule is appropriate in my situation. I just forget to try it at that time.

I check my local settings and realized I can not remember the process very clearly. However, there are 4 main files about this extension in total, in which data.json, manifest.json, style.css seem to be not important. So I put my modified my files here. Hope this is helpful. I change many "git" in the file to "git-copy" and some extra thing which I dont recall well.
obsidian-git-copy.zip


Enable crossbuilds with Promu

What's this PR actually attempting to do?
Also I've just committed a different upstream fix for the Makefile - thanks Docker for breaking that :)

it enables the application to build for different platforms like linux-amd64, linux-i386, windows via the prometheus builder utility called promu.

Closing this in favor of the final fate of #79 (which will probably be broken up into a few other changes).


Test RID_Owner thread safety

This is very "drafty" at the moment. Just seeing if I can come up with some reliable test and if my hypothesis that something else is needed for true safety is needed or not.

Closing to make a new one with the takeaways and a clean timeline.


Metrics integration with datadog

Adds many APM trace metrics and error alerting via datadog integration.

Resolves #611

Codecov Report

Merging #688 (b1c57ab) into main (7969317) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #688   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           68        70    +2
  Lines         2925      3018   +93
  Branches       310       310
=========================================
+ Hits          2925      3018   +93

Impacted Files
Coverage Δ

src/spellbot/cli.py
100.00% <100.00%> (ø)

src/spellbot/client.py
100.00% <100.00%> (ø)

src/spellbot/cogs/about_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/admin_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/ban_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/block_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/config_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/events_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/leave_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/lfg_cog.py
100.00% <100.00%> (ø)

... and 13 more

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 0dc4360...b1c57ab. Read the comment docs.


Divide-and-conquer strategy for intersections of unions

With this PR we implement a "divide and conquer" strategy for handling large intersections of union types. Specifically, when creating an intersection with four or more constituents, some of which are unions, we divide the constituents into two groups such that A & B & C & D is processed as (A & B) & (C & D). Since intersections of unions often produce far smaller unions of intersections than the full cartesian product (due to some intersections becoming never), this can dramatically reduce the overall work. For example, the slightly reduced repro from #57863 that barely passes before this PR takes 0.28s to type check. That drops to 0.08s with this PR.
Fixes #57863.

@typescript-bot test top200
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this faster

@typescript-bot run dt
@typescript-bot perf test this faster

Tests and performance are unaffected. This is ready for a review.

@typescript-bot cherry-pick this to release-5.4

This is huge, thank you so much! I've been working around this limitation in my library for almost a year now. These changes allow me to do so much more now! 🎉❤

@ahejlsberg I would like to see a patch release on top of 5.4 with these changes. Is there anything I can do to help get this out there?

This is backported (see above), and will be in the next patch.


DM-9790: Items missing from the pybind11 wrapping guide

Thanks for updating that. I guess @ktlim should probably sign off on the style guide changes, to make it official. But I'm happy with them.


Populate the documentation

Clean doc about classification, with clean doctrines
Talk about the playground in the index and README

Fixed by #53


ControlPath uses feature not present in CentOS7's version of OpenSSH

Issue Type

Bug report

Molecule and Ansible details
ansible --version
ansible <IP_ADDRESS>

molecule --version
molecule, version <IP_ADDRESS>rc6

Molecule installation method: pip
Ansible installation method:pip

Desired Behaviour
Under CentOS7, when running Molecule test with the vagrant driver, Molecule will generate ansible_ssh_common_args in molecule/default/.molecule/ansible_inventory.yml. Molecule should generate usable arguments based on the hosts OpenSSH version
Actual Behaviour (Bug report only)
Molecule instead is hardcoded to use the following ssh arguments:
molecule/lib/python2.7/site-packages/molecule/driver/base.py
154     def _get_ssh_connection_options(self):
155         return [
156             '-o UserKnownHostsFile=/dev/null',
157             '-o ControlMaster=auto',
158             '-o ControlPersist=60s',
159             '-o IdentitiesOnly=yes',
160             '-o StrictHostKeyChecking=no',
161             '-o ControlPath=~/.ansible/cp/%C',

ControlPath=~/.ansible/cp/%C is not valid in openssh-6.6. Instead we have to use 'ControlPath=~/.ansible/cp/%r@%h-%p'
It would be awesome if we could have molecule detect the host OS version / OpenSSH version and use the correct arguments for that version. I could try to hack something up if you want.

Would it be better to replace %C with %r@%h:%p? After taking another look at the code it might be a bit tricky to use os detection

@ragingpastry implemented in 31d68aeb5008528ec493dec783b726aead894862.
Update your molecule.yml with the ssh config options you desire.  Molecule does not merge lists, so you need to provide everything when overriding ssh_connection_options.
driver:
  name: ec2
  ssh_connection_options:
    - -o UserKnownHostsFile=/dev/null',
    - -o ControlMaster=auto',
    - -o ControlPersist=60s',
    - -o IdentitiesOnly=yes',
    - -o StrictHostKeyChecking=no',
    - -o ControlPath=~/.ansible/cp/%r@%h-%p


Center content?

Is it possible to center the items horizontally?  Everything seems to be pushed over to the left
Thanks!

Hey @beamercola, thanks for posting your question here.
Could you give us an example? A Codesandbox maybe?
I’m afraid I didn’t get your issue because the grid doesn’t have any direction actually. Once the width of the columns is the same, it is evenly distributed.
I don’t if it helps you, but what about to center the container?
.grid {
   margin-left: auto;
   margin-right: auto;
}


BSHTMLLoader not working for urls

System Info
BSHTMLLoader not working for urls
from langchain.document_loaders import BSHTMLLoader

url = "https://www.google.com"

loader = BSHTMLLoader({"url": url})
doc = loader.load()

I tried this one also not working
from langchain.document_loaders import BSHTMLLoader

url = "https://www.google.com"

loader = BSHTMLLoader({"url": url})
doc = loader.load()

Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[X] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.document_loaders import BSHTMLLoader

url = "https://www.example.com"

loader = BSHTMLLoader(url)
doc = loader.load()

Expected behavior
It has respond html data of url. I believe BSHTMLLoader won't work with url and only work files(.html files).

I am currently developing a custom load function as an alternative to BSHTMLLoader for handling URLs within my local codebase


Incorrect redirect from argoproj/argo to argoproj/argocon21

Summary
It looks like https://github.com/argoproj/argo now redirects to https://github.com/argoproj/argocon21 (forked from https://github.com/todaywasawesome/argo).
This would break all the old issue/PR links that uses "argoproj/argo" and make it hard to find past discussions.
cc @todaywasawesome who owns the original fork https://github.com/todaywasawesome/argo and might have a better idea on the best way to fix it.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I don't think this is fixable.

All fixed!

Thank you!


But please use some good quality images, and find images that are not having a watermark.
And do add your app in Directory: BulbOnOff/USERNAME
Thank You! 😄


Non-fatal errors are grouping up in dashboard

Non-fatal JS-errors are all grouped together when using logException meaning that its not possible to actually see all the different logs that has been done.
Everything is grouped under:
Non-fatal Exception: java.lang.RuntimeException
Is it possible to go around this somehow so the errors are grouped correctly?

I'm seeing this as well. Only the latest non-fatal error is visible in Fabric

I'm getting the same issue, it's very frustrating. It does look like you can see each individual non-fatal report with the View all sessions button (see screenshot). It's a halfway-decent workaround but still not very good.

yeah its kinda getting frustrating. Does anyone have an idea how to solve this?

Apparently Fabric really isn't designed to handle JS non-fatals (at least not currently)... I've switched to BugSnag which explicitly supports React Native. So far I like it a lot better, even on just the free tier.

this seems to be fixed? can anyone confirm?

@proProbe yes, i checked that this morning here

I don't think this is fixed. If you use Crashlytics.logException it is always a RuntimeException and so grouped together.
    public void logException(String value) {
            Crashlytics.logException(new RuntimeException(value));
        }

Source: https://github.com/corymsmith/react-native-fabric/blob/master/android/src/main/java/com/smixx/fabric/SMXCrashlytics.java#L31

As @Gumija pointed out, using plain Crashlytics.logException still groups all errors on the interface under the same Java file and line - i.e., "SMXCrashlytics.java:33".
To work around that I'm using Crashlytics.recordCustomExceptionName directly. I followed the example of react-native-fabric-crashlytics and used stacktrace-js's StackTrace.fromError to prepare the stack and keep column numbers around:
const stackTrace = await StackTrace.fromError(error, { offline: true });
const stackTraceWithFilename = stackTrace.map(trace => (
  {
    fileName: trace.source,
    functionName: trace.functionName,
  }
));
Crashlytics.recordCustomExceptionName(`${isFatal ? 'fatal': 'non-fatal'}:`, message, stackTraceWithFilename);

If column numbers are not needed, just delivering the result of fromError should be enough.
This is still not ideal, as we can't disable the line break on the exception name and must make an adaptation to keep the column numbers around, but it's an improvement on the default handling.

@moret I've seen the docs refer to recordCustomExceptionName as a rather expensive method call ("This call can be expensive and should only be used shortly before process termination.").  Have you seen any performance issues after calling this method?

@mdstaff no, but I actually only call it on terminal exceptions as the docs indicate, so I probably wouldn't notice if it did.


Problem

Domain
https://wordcounter.icu
Link
https://wordcounter.icu/erHk4
Version
Versión 0.2279-Mv3
What browsers are you seeing the problem on?
Microsoft Edge
What OS are you seeing the problem on?
Windows
Anything else?

There is no error. You need to manually bypass this website, so others can bypass it too (as read on the pop-up). This happens because the website uses crowdbypass.
Also is this really FastForward?

you seem to be using something called Streamus ByPasser. this is not made by us, but is based off Fastforwards code. we would be more than happy to help you if you (installed the official version of Fastforward)[https://github.com/FastForwardTeam/FastForward/blob/main/INSTALLING.md]


Remove version package from API types.

It's an internal type that only extends string.
Signed-off-by: David Calavera<EMAIL_ADDRESS>

LGTM

LGTM


When scrolling vim, text is not displayed properly.

This bug-tracker is monitored by Windows Console development team and other technical types.  We like detail!
If you have a feature request, please post to the UserVoice.

Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues. Instead, send dumps/traces to<EMAIL_ADDRESS>referencing this GitHub issue.

Please use this form and describe your issue, concisely but precisely, with as much detail as possible

Your Windows build number:  (Type ver at a Windows Command Prompt)
Windows Home Insider Preview build 17704.rs_prerelease.180623-1611 (fast ring)

What you're doing and what's happening: (Copy & paste specific commands and their output, or include screen shots)
When using vim - both on local or SSH remotes and on both Ubuntu 16.04 as well as 18.04 - when scrolling only the last few lines are shown, while the others aren't rendered. When activating mouse support using :set mouse=a and scrolling down a file, only the last 3 lines are rendered. When using the arrow keys to do so, only the last 1 line is shown. Due to this happening in all the distributions that I've tested so far as well as remotes running different ones, I believe that this is an issue in conhost. Also this doesn't happen in my laptop which runs Arch Mate and on which I'm using Terminology, further supporting that the issue may lie in conhost. Lastly I've tested this issue in SUSE Leap 42 on WSL where the issue is occurring as well.
One thing that I found noteworthy is that this issue doesn't seem to happen in vi while in SSH sessions to other hosts - on WSL Ubuntu it seems to be linked to vim without any straightforward option to install vi. The aforementioned SSH remotes are running Arch Linux with their default vimrc. Another peculiarity is that Ctrl-L (which redraws the shell) makes the rendering happen correctly. Needless to say, having to do that every time when scrolling is a PITA.

What's wrong / what should be happening instead:
The text should be rendered properly and be kept like that during scrolling.

This is #210. The fix is on the way out to Insiders.

Thanks for the quick follow-up, much appreciated. My apologies for not thoroughly checking the existing issues for this.


fix(#249): regarding issue introduced in IOS 14 UISlider needs to forc…

Closes #249
Summary:
Fixes issue mentioned in this article and reported in #249
https://developer.apple.com/forums/thread/658526
Test Plan:

Use reproducible code from #249
Tap multiple times on disable button

https://user-images.githubusercontent.com/16048381/198872127-0a211628-7bf6-4736-8e3b-a4d54d54f224.mov

@MateWW I allowed myself to apply the changes to push the topic forward.
Thanks again for this 👍

Sorry for late replay. I wanted to handle that during the weekend


fix: Public header uses private thing

Since the value used in the public header is arbitrary and the constant
is not necessary, it's just changing the code to use the value directly.
Signed-off-by: Flavio Ceolin<EMAIL_ADDRESS>

problem spotted in https://github.com/solettaproject/soletta/issues/1234

Fine by me

merged, thanks

I wonder why these symbols are defined in sol-platform.h at all, and worse: why are them missing "SOL_" prefix?


Fix SEI example notebook

Fix the failing notebook, as identified in https://github.com/pybamm-team/PyBaMM/pull/3110#discussion_r1267382517. I also wrote some extra details about how to fix the moving boundary, which were part of the non-dimensionalisation and thus were lost with the shift to dimensional models.

@arjxn-py this should fix the problems with #3110, so once we merge this into develop you can merge develop back into your branch.

@arjxn-py this should fix the problems with #3110, so once we merge this into develop you can merge develop back into your branch.

Thanks so much @brosaplanella 😄


naoqi_libqi: 3.0.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository naoqi_libqi to 3.0.0-1:

upstream repository: https://github.com/ros-naoqi/libqi.git
release repository: https://github.com/ros-naoqi/libqi-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.2
previous version for package: null

naoqi_libqi
* Add iron workflow for CI
* Restore GitHub workflows
* Contributors: Victor Paléologue
* Update README
* Avoid boost bind global placeholders
  Finally, at last.
* Use .value() when getting a service
* Fix usage of boost::asio for 1.74
* Use explicit placeholders
* Port to ROS 2 restarted from version qi-framework-v1.8.7

Also, same as here: https://github.com/ros/rosdistro/pull/38898#pullrequestreview-1720846474


Open modal is shifting body content to the left

Possibly the worst reported issue so far. Since this is so poorly formatted, I'm going to close this until you add all required info and reopen.
Read this before posting issues
https://github.com/angular-ui/bootstrap/blob/master/CONTRIBUTING.md

Sorry.
http://plnkr.co/edit/ynWJUbnryVkSHdEjFRMw?p=preview


Linkcheck Report

DON'T CLOSE THIS ISSUE!
It will be auto-updated by a workflow to list broken links in the website.


Vickel: Watch toom.de

Vickel requests the watch of the watch_keyword toom\.de. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

toom\.de has been seen in 1 true positive, 1 false positive, and 0 NAAs.

Approved by tripleee in Charcoal HQ