[BUG] Crashes when pressing the arrow keys

General Information
Version: main
Installation Method: local build
Operating System: macOS
Describe the bug
Program crashes with an stack overflow error when pressing the arrow keys inside of a view.
To Reproduce
Steps to reproduce the behavior:

Build wiki-tui locally
Remove any keybindings configuration in the config file
Start wiki-tui
Search for anything
Press the down key inside of the search preview view
wiki-tui crashes

Expected behavior
Program works fine and does not crash when pressing keys
Additional context
This bug was found in another issue (#39)
Checklist

[x] checked other issues for the same bug
[x] read CONTRIBUTING.md

@ThomasFrans I've uploaded a patch onto the development branch (45-bug-crashes-when-pressing-the-arrow-keys). This fixes the issue for me. Could you please check if that also fixed the issue for you?

Yes, fixed it.

Nice, I'll put up a PR and merge it

It would be nice tho if the vim keybindings worked by default, alongside the arrow keys, because now they don't work anymore. Maybe they can be added to the config by default, and the user can then change them if they want. Or add them directly in the code, if hjkl isn't overridden in the config.


String option eats a value without its flag after repeatable values

It looks like a bug to me, but it is highly plausible that I misunderstand repeatable values or something. I would appreciate if you could explain the logic behind the following parsing result.
Code:
void main(int argc, char* argv[]) {
    std::vector<int> vec;
    std::string str = "hungry";
    auto cli = (
      clipp::option("-v", "--vec") & clipp::integers("label-v", vec),
      clipp::option("-s", "--str") & clipp::value("label-s", str)
    );
    auto parsed = clipp::parse(argc, argv, cli);
    if (!parsed) std::cout << "ERROR!\n";
    clipp::debug::print(std::cout, parsed);
    std::cout << "vec : [";
    for (const auto i: vec) std::cout << i << ",";
    std::cout << "]\n";
    std::cout << "str : " << str << "\n";
}

Product:
% ./a.out -v 1 2 hello
#1 -v -> -v
#2 1 -> label-v
#3 2 -> label-v         [repeat 1]
#4 hello -> label-s
vec : [1,2,]
str : hello

I expect "ERROR!" is printed in this case.
Thanks

You are right, that looks like a bug. I will look into it.

Fixed in the latest release.


Update some devDependencies and to Node 6

npm test passes after these changes.

I think browersify needs to be updated, too

Done (and tests look like they're passing now).

:+1:

👍


DiagTrack is no longer used starting with RS5 or newer

🚨 Looks like you arrived from the internal review site
🚨 Do not enter Microsoft confidential information here
[Enter feedback here]
DiagTrack is no longer used by the MDE sensor starting with Windows RS5 or newer, as per the Product Engineering Team. Sense hosts its own mechanism to send CyberData and no longer relies on DiagTrack, starting with RS5.

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

ID: 76a978b2-de1e-d59d-016d-023cb2ff4851
Version Independent ID: d2891133-a976-edbf-2aad-d6971182dbbf
Content: Fix unhealthy sensors in Microsoft Defender for Endpoint
Content Source: microsoft-365/security/defender-endpoint/fix-unhealthy-sensors.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks


Needed node APIs for thread management

Starting down the path of implementing thread creation and management. Looks like we need some additional APIs:

[ ] Join existing thread (will the existing addThread just work for this?)
[ ] List all threads I participate in
[ ] Thread metadata (members, invite link, etc)
[ ] Leave thread

Thread metadata could just come back as part of the list all threads API

list threads will need:

thread id, name, public key, num peers

notes from related call:

somehow include usernames in thread metadata
how to include the "active" users in a thread?
how to invite user to a thread that's not yet in the system?


New array as model not working in the playground

Type relation PR seems to have broken arrays in the playground
Playground Link

nvm was an issue with web caching, works fine


Daemons as services

Žinoma makes it possible to define services and use them as dependencies of other resources. At the moment, a service is specified as a script to run. That script will be killed as Žinoma exits. In watch mode, Žinoma restarts services by killing and starting them.
Žinoma could also accept services accessible as daemons.
These services can usually be operated by a set of commands (start, stop, and sometimes restart). For instance:
targets:
  mongodb:
    service:
      start: sudo systemctl start mongod
      stop: sudo systemctl stop mongod
      restart: sudo systemctl restart mongod

See also https://github.com/fbecart/zinoma/issues/48


fix: 4606 - translucent top status bar on on-boarding page 1

What

In order to have a translucent top bar, the trick is to start with a colored Container AND THEN use a SafeArea.
This way, the system will use the Container color with its own shade for the top status bar, and the body of the page will still use a SafeArea.
@bishalbera Very strange that on your android you already had a translucent top bar. I had not. The solution is in the end not what I assumed, but it mimics what is coded in other similar pages.

Screenshot

Fixes bug(s)

Fixes: #4606

Impacted file

reinvention_page.dart: put the color (container) and THEN the SafeArea

Codecov Report

Merging #4615 (bbcd820) into develop (c6d8aec) will not change coverage.
Report is 3 commits behind head on develop.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #4615   +/-   ##
========================================
  Coverage    10.14%   10.14%
========================================
  Files          299      299
  Lines        15686    15686
========================================
  Hits          1591     1591
  Misses       14095    14095

Files Changed
Coverage Δ

...oth_app/lib/pages/onboarding/reinvention_page.dart
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

ya that was strange.
Thank you, @monsieurtanuki, for sharing this  valuable tip. and I believe that using SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle(statusBarColor: Colors.transparent)); in the main function before calling runApp could be another helpful approach. In any case, I want to express my gratitude once more. My journey into open source development has begun, and I'm already gaining valuable insights and knowledge.

@bishalbera Enjoy the journey!

Thank you @M123-dev for your review!


Need to automate the search and release of helm binaries, say once per week!

yo this is a test issue.

made a mistake - was a test only!


reworked AudioBuffer

Hi,
i tested RPlay on the Raspberry pi and noticed that the Audiobuffer was a common source for failure.
The problem was related to how the Audiobuffer calculated the buffered elements to restart getNextFrame() after an underrun. Also the performance was not very good because of some inefficient locks.
I also entangled the Source code a bit and converted the AudioData to a "right" class instead of some proved C construct.
It now performs much better on the raspberry pi. :)

Hi,
Thanks for improving RPlay!
I developed it long time ago and, indeed, my code isn't really efficient.
However, there seems to be a problem with your code.
I get this error message:
"...
Waiting
Exception in thread "Thread-18" java.lang.NullPointerException
at AudioBuffer.calculateActualBufferSize(AudioBuffer.java:178)
at AudioBuffer.putPacketInBuffer(AudioBuffer.java:135)
at AudioServer.packetReceived(AudioServer.java:120)
at UDPListener.run(UDPListener.java:35)"
It looks like "audioBuffer[i]" returns null.
I'll try to look into it tomorrow ;-)

i have made some other changes...maybe i forgot to copy something


where is the dummy key and password?

It mentions that "Hackathon Starter 2.0 Update: I have included dummy keys and passwords for all API examples to get you up and running even faster."
where is the dummy keys and passwords?
I suppose I can use them during login?

Hey @3rcok
They are there: https://github.com/sahat/hackathon-starter/blob/master/.env.example


Airtable Submission Form

Create custom Airtable form that will allow Documenters to claim an open public meeting sent via Zapier/the aggregator. Creation of the form will allow CB staff to begin testing Documenters management and deployment using the Airtable aggregator.

gaaah my bad for not getting to this, I'll try and take a whack this week.

hey @jim i just wanted to quickly leave my note on formatting here before I forget. Basically, the form seems to be formatting scraped meetings differently from meetings that are manually added to the same airtable table.


[TASK] Mixer API

To make the API more configurable and more stable, I propose the following struct:
struct MixerR1CSProver<E: PairingEngine, H: FieldHasher<E::Fr>, const HEIGHT: usize> {
    engine: PhantomData<E>,
    hasher: PhantomData<H>,
}

Which will be defined as:
type MixerR1CSProver_Bn254_Poseidon_30 = MixerR1CSProver<Bn254, Poseidon<Bn254Fr>, 30>;

With the name convention: [prover name]_[ec]_[hasher]_[tree height]
With interface:
trait MixerProver<E: PairingEngine, H: FieldHasher<E::Fr>, const HEIGHT: usize> {
    // For creating leaves where we supply the secret and the nullifier, for generating new values, pass None
    pub fn create_leaf_with_privates<R: RngCore + CryptoRng>(
        curve: Curve,
        secret: Option<Vec<u8>>,
        nullifier: Option<Vec<u8>>
        rng: &mut R,
    ) -> Result<Leaf, Error>;
    // For making proofs
    pub fn create_proof<R: RngCore + CryptoRng>(
        curve: Curve,
        secret: Vec<u8>,
        nullifier: Vec<u8>,
        leaves: Vec<Vec<u8>>,
        index: u64,
        recipient: Vec<u8>,
        relayer: Vec<u8>,
        fee: u128,
        refund: u128,
        pk: Vec<u8>,
        rng: &mut R,
    ) -> Result<MixerProof, Error>;
}

NOTE: This interface can be implemented for both R1CS and PLONK systems

[ ] Update R1CS circuit API
[ ] Update PLONK circuit API

Will leave this open, since we need to do the same for PLONK


Option to retry state transition on gate opening?

I want to propose a feature for state transitions described in the title.
Basically, we have a transition with both an event and a closed gate. I want there to be a toggle that makes the transition happen when a gate opens even after the event has been sent. That is without sending the event again, only sending it once before the gate opens.
I think the most optimal way would be is for a gate to somehow hook to relevant expression properties -> re-evaluate and send an internal transition event if gate is satisfied.
Feel free to let me know if there's an easy way around with current tools (I tried transition delay but it only evaluates the gate at the start, not after delay) and if my proposed feature has any practical value

Yeah I was mainly asking because I had experience in programming where "gates" meant slightly different mechanism, and I was wondering if it would make sense to have that behavior here.
I ended up changing Lookout(Atomic) state into a compound chain of states (getting_up→rotate_to_outpost→go_to_outpost→lookout→rotate_to_rest_place→go_to_rest_place) and it didn't make sense to hardcode animation length values (since most animations were procedural), so I just used the gate I mentioned.
It's just that I didn't want to add these intermediate states, because it's not obvious if they should be inside Lookout or Resting.

Hmm okay, I think i have an idea now how this could work. Basically we have two parts in a transition:

the guard - this determines if the transition will be taken at all.
the delay - this determines when the transition actually will be executed after it has been taken

Now the delay currently can only be a timer, but if there was an option to use an expression as a delay, this could nicely solve the issue without introducing a lot of edge cases (except for the migration headache as this would be a breaking change... ).  So this would look like:

And in code:
# called when the unit should travel somewhere
func travel_to(target):
     _state_chart.set_expression_property("target", target)
     _state_chart.send_event("travel")

func on_orienting_physics_processing(delta):
     # code here to orient the unit
     ...
     if [unit is oriented]:
          _state_chart.send_event("oriented")

func on_moving_physics_processing(delta):
     # code here to move the unit where it needs to be
     ...
     if  [unit has arrived at target]:
         _state_chart.send_event("arrived")

What do you think?

Yeah, I think this makes much more sense than my initial approach.
I think when I started I just looked at the platformer example where animation states and logic states are parallel/independent of each other and assumed that was the only right way to do it. Where as in my example animations do in fact affect the logic.

With the new automatic transitions, you can actually do this without any custom delays. If you write a bit of code like this:
@onready var _animation_player:AnimationPlayer = %AnimationPlayer
@onready var _state_chart:StateChart = %StateChart

func _ready():
    animation_player.animation_changed.connect(_on_animation_changed)

func _on_animation_changed(_old_animation:StringName, new_animation:StringName):
   # set the name of the currently running animation as expression property.
   _state_chart.set_expression_property("animation", new_animation)

And then you can use automatic transitions from Arriving at home , Arriving at lookout and Preparing for travel  based on the current animation.


Strange Window Behavoir C#

Godot version
v4.0.stable.mono.official [92bee43ad]
System information
Linux Opensuse Leap
Issue description
So I finally figured out all the things I need to interact with the editor, and have finally started to create a plugin, however I can't get a window to show properly.
I created a window entirely in code.  I then created an object that inherits from Popup menu and was going to use it to to launch the window.  However upon launch the window content is entirely black.
I also had a test item in the tool menu so I tried launching it from there, and it worked fine.  To help clarify this is what I mean:

When I click the Test item it works, when I click the sub menu item Generate Script it appears entirely black.
In the next image the window launched from Test is on the left, and the window from Generate Script is on the right.

Not sure if this is a bug or I am just missing something.
Steps to reproduce
Create a tool sub menu item and launch a window from it.
Minimal reproduction project
Sharpener.zip

Please create a minimal reproduction project in GDScript instead of C# for two reasons:

Non-Mono builds of Godot are easier to set up and have less moving parts.
This rules out C# being the cause of the bug.

Had some time to look further into things and attached is a simple project that demonstrates things with almost no code.
The project consists of three scenes with root node types of MenuBar, PopupMenu, and Window.
The Popup menu scene has window scene attached, the menu bar scene has the popup scene attached to it.
The only 'script' is attached to the popup, and the entire code follows:
public partial class Pmenu : PopupMenu
{
	private Window testDialog;

	// Called when the node enters the scene tree for the first time.
	public override void _Ready()
	{
		testDialog = (Window) GetNode("TestDialog");
		IdPressed += OnPressed;
		testDialog.CloseRequested += OnWindowCloseRequest;
	}

	private void OnWindowCloseRequest()
	{
		testDialog.Hide();
	}

	private void OnPressed(long id)
	{
		testDialog.Show();
	}

	// Called every frame. 'delta' is the elapsed time since the previous frame.
	public override void _Process(double delta)
	{
	}
}

As can be seen it is bare bones, just two signals attached and handled.
So when I run this with project settings  defaults I get this:

So I must admit at this point I was wondering if there was a problem with my original code, because the window appears fine, only it's embedded so it it ignores the initial position property, so easy enough to fix go into advanced settings and uncheck embed windows.  And you then get this:

You'll find the project from the screenshots below.
plugintest.zip

Found an even easier way to duplicate, and it's definitely related to visibility of the menu when embed sub windows is set to false.
Attached is a project consisting of two scenes.  No code or scripts.  If I toggle visibility of the popup in the Godot editor, the window content will vary with it.  For example with pop up visible:

If I then set the popup to be hidden then when the scene is run it looks like this:

popupscene.zip


LocalSettings throws an UnauthorizedAccessException after AppInstance.Restart

Describe the bug
LocalSettings throws an UnauthorizedAccessException after AppInstance.Restart.
How can I fix this?
Expected: should be able to access LocalSettings after a restart.
Steps to reproduce the bug
Using Windows 10 and Windows App SDK, create a WinUI 3 desktop application using the template.
Change MainWindow.xaml to contain this:
<StackPanel Orientation="Vertical" HorizontalAlignment="Center" VerticalAlignment="Center">
    <TextBlock x:Name="TimeTextBlock">(unkown)</TextBlock>
    <Button x:Name="SetButton" Click="Set_Click">Set time</Button>
    <Button x:Name="GetButton" Click="Get_Click">Get time</Button>
    <Button x:Name="RestartButton" Click="Restart_Click">Restart</Button>
    <TextBlock x:Name="StatusTextBlock">Startup</TextBlock>
</StackPanel>

Change MainWindow.xaml.cs to contain this:
    public MainWindow()
    {
        this.InitializeComponent();
    }

    private Windows.Storage.ApplicationDataContainer LocalSettings => this._localSettings ??= Windows.Storage.ApplicationData.Current.LocalSettings;
    private Windows.Storage.ApplicationDataContainer? _localSettings;

    private void Set_Click(object sender, RoutedEventArgs args)
    {
        try {
            DateTimeOffset now = DateTimeOffset.Now;
            this.LocalSettings.Values["Time"] = now;
            this.StatusTextBlock.Text = "Set value " + now.ToString();
        } catch (Exception e) {
            this.StatusTextBlock.Text = $"Exception when setting value, type '{e.GetType().Name}' message '{e.Message}'.";
        }
    }

    private void Get_Click(object sender, RoutedEventArgs args)
    {
        try {
            DateTimeOffset? time;
            if (this.LocalSettings.Values.TryGetValue("Time", out object? value)) {
                time = (DateTimeOffset)value;
            } else {
                time = null;
            }
            this.TimeTextBlock.Text = time?.ToString() ?? "(null)";
            this.StatusTextBlock.Text = "Got value shown above from LocalSettings";
        } catch (Exception e) {
            this.StatusTextBlock.Text = $"Exception when getting value, type '{e.GetType().Name}' message '{e.Message}'.";
        }
    }

    private void Restart_Click(object sender, RoutedEventArgs e)
    {
        AppRestartFailureReason failure = Microsoft.Windows.AppLifecycle.AppInstance.Restart("");
        this.StatusTextBlock.Text = $"Restart failed, '{failure.ToString()}'";
    }

Run the app. Click on the Get and Set buttons, observe it works.
Then click on Restart. After the application restart, click on Get or Set and receive an UnauthorizedAccessException.
Expected behavior
Should be able to access LocalSettings after a restart.
Screenshots
No response
NuGet package version
None
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 22H2 (19045, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

Hi @DarranRowe I am using C# and the packaged desktop (WinUI 3) app template.
Not sure where the partial trust setting is but if you tell me I will take a look.

It is very unlikely that you have it set then. The default isn't partial trust, and it is a setting that you have to go out of your way to enable.
On the off chance that you enabled it though. If you have a separate packaging project, then you have to select the project name in solution exporer and then select it in properties.

If you are using the single project layout, then it is even less likely that you could stumble onto this. You would have to manually edit the Package.appxmanifest file and type out either windows.partialTrustApplication and/or appContainer.

Yes it’s the single project layout and I did not type the phrase you mention.


Bugfix for duplicating vis when building state from URL hash

Please confirm if this fixes the problem -- at least in my tests it is working as expected.

Explanation
While waiting for some results from await the columns have been changed and thus used a wrong number of columns when splitting the multiforms. I tried to avoid using the class variable this.filterHiearchy. Instead I sliced a copy so that I can ensure the elements (https://github.com/Caleydo/mothertable/compare/thinkh/bugfix-loading-from-hash?expand=1#diff-5af33055eeb4b6024eada05612cd2bceR158) and passed these columns directly from function to function.

No duplicates. But the scaling is messed up:

Why is that not happening in the screencast?

@mstreit I don't know. Could you please give a URL or steps to reproduce your state?

It happens for both URLs listed in #209

Ok, I'll check those URLs

It's not deterministic. Sometimes it works.

This is a scaling and update issue, which is probably unrelated to the URL restoring.

Yes, true. Anyway there is a bug that matrices do not appear and barplots have negative values.

I will see why matrices are not restored correctly. The other errors are a different PR.

Seems working for me . Thanks @thinkh 👍


(Just a question)How to do something similar to numpy.where in Zarr?

Hi, all!
How can achieve something like:
selection =  np.where(dateSeries == today)
where dateSeries is a Zarr array. Or how can I dump a zarr array to NumPy, is there any underlying attribute that holds the values or maybe an iterator, as data maybe chunked? I think it's easy to store NumPy values into Zarr, but the recovery process into NumP seems to me a little bit obscure ...
Kind regards, and thank you for your great software!
Manuel.

Hi @manolinux,
For computing over Zarr arrays, I use Dask. E.g.:
import dask.array as da
xz = ...  # obtain a zarr array somehow
xd = da.from_array(xz, chunks=xz.chunks)
selection = da.where(xd == today).compute()

If you ever want to manually load data from a Zarr array into a Numpy array, use slicing, e.g.:
x = xz[:]  # load entire array into numpy array
x = xz[:10]  # load first 10 elements into numpy array

Hth.

Thanks a lot, @alimanfoo !
Yes, i was thinking of something like that, also needed performing some statistics over the arrays, so Dask offers that power ...

You can also use zarr with xarray, which is convenient for indexing. (The time based lookup you mentioned above is very easy with xarray.)
http://xarray.pydata.org/en/stable/io.html
Sent from my iPhone

On Jun 27, 2019, at 8:04 AM, manolinux<EMAIL_ADDRESS>wrote:
Closed #446.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Brian/fix timelock

Pull Request Test Coverage Report for Build 7618

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 7615:
0.0%

Covered Lines:
1710

Relevant Lines:
1710

💛  - Coveralls


Dialog.Close used in documentation. Should be Dialog.Cancel

Documentation
Relevant Radix Component(s)
Dialog
The anatomy: https://www.radix-ui.com/docs/primitives/components/dialog#anatomy
import * as Dialog from '@radix-ui/react-dialog';

export default () => (
  <Dialog.Root>
    <Dialog.Trigger />
    <Dialog.Portal>
      <Dialog.Overlay />
      <Dialog.Content>
        <Dialog.Title />
        <Dialog.Description />
        <Dialog.Close />
      </Dialog.Content>
    </Dialog.Portal>
  </Dialog.Root>
);

Dialog.Close does not exist in 0.1.5 instead it should be Dialog.Cancel
The code sandbox also appears to say this. Is this a feature from an older version or something?

I think you are confusing Dialog and AlertDialog.
Dialog has Dialog.Close, AlertDialog has AlertDialog.Cancel and AlertDialog.Action.

@benoitgrelard Ahhh, you made me realize that vscode auto imported the wrong dialog haha. Thanks


Add Plutip+E2E to CI

Closes #1213  .
Pre-review checklist

[ ] All code has been formatted using our config (make format)
[ ] Any new API features or modification of existing behavior is covered as defined in our test plan
[ ] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

Great!
We need to:

create a new test/e2e-ci.env with only plutip tests
do not include any wallet setup there
start webpack-dev-server in background (like in make run-dev), get it's PID, store in a variable
run the tests
kill the server by PID

Continued in https://github.com/Plutonomicon/cardano-transaction-lib/pull/1253


Clarification regarding pytest and lint with flake8

I am doing  documentation part for the HW4. Yesterday, When i create pull request, there was pytest and  lint with flake error for documentation. Can i have more clarification on this?

Its currently testing and linting every file in the repository so its ok that yours fails because some of the files already merged do not pass.

Ok. Thankyou so much for clarification.


fix: add admin menu

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3781
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

btw, do we still need AdminMenu? because the menu was removed in this PR https://github.com/kodadot/nft-gallery/pull/3817

btw, do we still need AdminMenu? because the menu was removed in this PR https://github.com/kodadot/nft-gallery/pull/3817

Ah, I thought we wanted to remove this because it was not functioning properly (in need of readjusting query etc), but maybe I'm mistaken @vikiival can you enlighten us? Is admin panel gone in general?
I remember talking about it in relation to managing empty collections on BSX for example.

enlighten us? Is admin panel gone in general?

Admin panel heavily relies on SubQuery + It wasn't perfectly done.
So next steps:

Close this
Close #3781 as not planned
Let's continue on #2347


Move files in cpp-linear-search-2 folder to cpp-linear-search

Fixes #29

Please consider adding your profile to the Contributors section.

Please consider adding your profile to the Contributors section.

Done!

Merged. :)


Add Grover operator to the circuit library

Summary
Added the GroverOperator to the circuit library to construct the Grover operator used in Grover's search algorithm and Quantum Amplitude Amplification/Estimation.
Details and comments
Provided with an oracle in form of a circuit, GroverOperator creates the textbook Grover operator. To generalize this for amplitude amplification and use a generic operator instead of Hadamard gates as state preparation, the state_in argument can be used.
This is the fundamental building block in streamlining and refactoring the amplitude estimation algorithms and Grover's algorithm.

It's specific in the sense that you need to provide a circuit for "mark" (and, optionally, "diffuse") yes but otherwise it should cover the general cases for Grover search and amplitude amplification & estimation I think.
I like the support of QI objects, I'll add it 👍  An efficient way to synthesize the statevector you supplied for marking the good states would be to either use #4758 or the oracle compiler, but I'll just use Diagonal for now 🙂

Wouldn't you prefer an oracle that works from "a specification" rather than a circuit? Your implementation is probably more efficient than diagonal synthesis, but I'm worried it ties the implementation to the specification too much. This is what we tried to separate with the oracle compiler, the MCX, etc.

Agreed, I think it would be better to have it on a higher level of of abstraction. My reasoning is: if you do the synthesis of the circuit for the oracle yourself, then you are just one small step away from implementing this operator (to the point that I think this won't actually be necessary).

Agreed, I think it would be better to have it on a higher level of of abstraction. My reasoning is: if you do the synthesis of the circuit for the oracle yourself, then you are just one small step away from implementing this operator (to the point that I think this won't actually be necessary).

I agree, it would be nice if we could allow this level of abstraction 👍  We can always add support for higher level inputs.
But we should keep support for circuits I think.


README documentation cleanup/updates

Updates to the README documentation for CONFIG object, pages, groups, and tiles.

Thanks so much doing it! This will help newcomers a lot.


fixldap set passwd need forward to master (#36436)

pick: https://github.com/apache/doris/pull/36436

run buildall

run external


Quest v0.2 / G6PD: test results questionnaire

[PLACEHOLDER]
This issue covers all the steps necessary to allow a user to submit a G6PD test result questionnaire:

[ ] Establish method of tagging the specific questionnaire so that the end user can open it when they hit the "record test result" button.
[ ] Establish how to integrate images into the questionnaire (for training step) - will this be all one questionnaire?
[ ] Confirm questionnaire logic

Mocks TBC

There are two extensions to change text style into a questionnaire but none is working

"http://hl7.org/fhir/StructureDefinition/rendering-style",
"http://hl7.org/fhir/StructureDefinition/rendering-xhtml",

Which means that we can not make the questionnaire dynamic and independent of G6PD flow or do all into one questionnaire unless we have these functional in SDK.

Update
This is WIP as of Oct 12th 2021


Fixes #134 and #28

Using keyRef.updateChildren instead of keyRef.setValue also fixes a huge amount of unnecessary 'onDataExited(DataSnapshot dataSnapshot)' events, which has not been reported yet. keyRef.updateChildren understands if the data needs to be inserted or updated. No need to use setValue here. Since priorities are not relevant anymore (https://stackoverflow.com/questions/31577915/what-does-priority-mean-in-firebase) i decided to not consider them anymore.

I signed it!

lets try again :D

..

@hkj would love your input here.
It seems like in the past, geofire was relying on the coincidence of the Android and Java RTDB SDKs having the same API surface.  Then we use provided dependencies to provide the right code at runtime.
The recent Java changes break this (which is fine) but I wonder how we should proceed here.  Should we lock geofire into the 5.x version of the Admin SDK forever?


OCPBUGS-22628: Bump otelhttp to >= v0.44.0

/retest

/label acknowledge-critical-fixes-only

This bug also updates opentelemetry-go-contrib to v0.46 which resolves https://issues.redhat.com/browse/OCPBUGS-23880

/jira refresh


initial implementation of replies for commands

Idea:

Commands that want a reply need to add a Reply<T> field annotating the type of reply they want to provide and than can be used to construct the reply.
new EntityRef::ask will help if a reply is needed
What to do about commands that do not require a reply? I wouldn't want to force them to add a Reply<()> as well, but that means that we will have to support to different ways of dealing with a command: with reply and without reply. This adds a small potential for runtime errors i.e. a handler could forget to use the reply.
On the other hand, we could enforce the reply style for all commands which could force you to specify a full type like Result to both deal with user replies and errors.

Nice!
To keep the API surface minimal and (if I understand correctly) to allow per command errors, I would like to enforce the "reply style" (if no reply is needed, that would be a Result<(), SomeCmdError> and only have a single method handle_cmd (no more ask).
We could rename everything from reply to result, if possible, thereby potentially creating aliases or proper types (like CmdResult).

What to do about commands that do not require a reply? I wouldn't want to force them to add a Reply<()> as well, but that means that we will have to support to different ways of dealing with a command: with reply and without reply. This adds a small potential for runtime errors i.e. a handler could forget to use the reply.

On the other hand, we could enforce the reply style for all commands which could force you to specify a full type like Result to both deal with user replies and errors.

Following up on that, in #185, you can either have fire-and-forget style commands that cannot return any kind of reply (including errors) or you will have to add a Reply field to the command even if just to supply an error.

To keep the API surface minimal and (if I understand correctly) to allow per command errors, I would like to enforce the "reply style" (if no reply is needed, that would be a Result<(), SomeCmdError> and only have a single method handle_cmd (no more ask).

This is basically what is done in #185 (we can remove the fire-and-forget method).

Incidentally, the Reply<T> type now has no real functional behavior any more but is just a marker type that tries to give some type-safety. It probably works well in most situations, especially if we remove Clone and Copy and if we decide to do away with the fire-and-forget method, so that you are forced to have a Reply in the command to be even able to create a result in the command handle.
So, in theory, we could do away with Reply and fail at runtime when types do not match up between command handler and what the code running the command is requesting but I think it's worth it to keep it, even if it seems a bit alien to declare the return type as a field of the cmd value.

Closing this one in favor of #185


Update README.md

Import completeness

Guys we need to merge this sort of stuff.  It's from Feb 14!

The same change is needed on line 187.

@tomchentw please delegate access :)

Dup of merged #438.

Released v6.0.2


UI: Ability to quickly filter by clicking on different metadata like version or user

Improve filtering experience by allowing you to quickly apply filters by clicking on a version, user, or simple extended data.

https://github.com/exceptionless/Exceptionless/issues/1271


add support for role icons

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
this PR adds support for role icons. see https://github.com/discord/discord-api-docs/pull/3847

Looks pretty good to me. @MinnDevelopment?

The docs PR is only a datamine PR for an experiment, so I would be against merging it at this time. See https://github.com/discord/discord-api-docs/pull/3815

that's one of the reasons this PR is a draft. I haven't tested it

Okay, going to mark as freezer until the feature is no longer experimental.
Additionally, this PR will need to be redirected at v5 as we're closing down feature-additions on v4

seems like the PR has been merged. I'll still wait until the feature is fully released to test it out properly

it'd be great if someone with access to role icons (owning a server with boost tier 2+) could test this PR because I don't have one. (I think the PR should be fine though)

When will this be available? I'm waiting for it


Heap use-after-free in mrb_ary_ref

The following input to mirb demonstrates a crash: 215447.txt
(Note that this input must be supplied to mirb, not mruby.)

This issue was reported by https://hackerone.com/ston3

@matz This issue needs to be re-opened since the change was reverted. The crash is still present in the latest master.


Fix typo for serviceAccountName in RBAC example

serviceAccountName was transposed onto the non-RBAC example

signed CLA

👍


to_hash returns empty hash.  Allows libraries like RABL to gracefully handle nulls

After I recently upgraded the Twitter library, I kept getting "TypeError - Twitter::NullObject.to_hash() did not return a Hash." in our app. I tracked the bug down to our RABL views calling the to_hash() on almost every property returned by Twitter::Tweet. The change I made allows RABL to gracefully handle null objects.

@avdi Should this be covered by define_implicit_conversions?

I didn't think of that.  It would be much cleaner to update the define_implicit_conversions method in naught.

I’ve opened https://github.com/avdi/naught/pull/68. Let’s see what @avdi has to say about that.

Just to reiterate my response over there: very yes!

This has been fixed in Naught.


Support predefined configurations/configurations

https://github.com/microsoft/vscode-cmake-tools/issues/1003#issuecomment-574020010
Instead of just letting the user to select: build variant, kit, etc. the extension can have named configurations, including predefined configurations like in VS2019. These can be extended to support configurations defined in a CMakeSettings.json file like in VS2019.

I'd really like this, because I need configurations that bundle variants and kits together for cross-compilation.
e.g.
Build:

STM32F205 Release µC always with arm-none-eabi toolchain
but build:
Unit Tests Debug always with clang on windows.

This feels like a duplicate of #529.  Correct me if I'm wrong, but I think that support for CMakeSettings.json would address this.

Thinking about this more, I believe there might be more to it than that.  Though I'm unsure of what exactly you would like that is different from the default variants.  Based on the comment from the linked thread it appears you would like to have different default buildDirectorys based on the Kit/generator.

Adding this feature to vscode would bring more operability to c++ projects using microsoft tools.
The use case that should be common is to provide a crossplatform enviroment. The current limitation is that there is no common aproach to support developers in diferent OS (aka. there is no visual studio for linux -> so a person in the development team must use microsoft windows + Visual Studio).
So in short words, the way to develop with cmake and Visual Studio, is to EDIT in Visual Studio and launch a remote build.
So what will bring this feature is to edit in a crossplatform code editor as vscode and launch local builds using the CmakeSettings.json.

Adding this feature to vscode would bring more operability to c++ projects using microsoft tools.
The use case that should be common is to provide a crossplatform enviroment. The current limitation is that there is no common aproach to support developers in diferent OS (aka. there is no visual studio for linux -> so a person in the development team must use microsoft windows + Visual Studio).
So in short words, the way to develop with cmake and Visual Studio, is to EDIT in Visual Studio and launch a remote build.
So what will bring this feature is to edit in a crossplatform code editor as vscode and launch local builds using the CmakeSettings.json.

@orion160 Just as side note AFAIK both Visual Studio and I guess this extension will be moving to support CMake presets and CMakeSettings.json will be probably deprecated.
I really wish tighter integration between Visual Studio and Visual Studio Code/CMake tools to support cross-platform development seamlessly. There are number of areas they defer like the VSC extension doesn't support install and debug yet as well passing command line arguments via launch.vs.json incompatible or at least it used to be.

@orion160 Just as side note AFAIK both Visual Studio and I guess this extension will be moving to support CMake presets and CMakeSettings.json will be probably deprecated.
I really wish tighter integration between Visual Studio and Visual Studio Code/CMake tools to support cross-platform development seamlessly. There are number of areas they defer like the VSC extension doesn't support install and debug yet as well passing command line arguments via launch.vs.json incompatible or at least it used to be.

This seems to be a duplicate of #529

This seems to be a duplicate of #529

Actually not quite. I am not sure how relevant is this stuff currently.

Actually not quite. I am not sure how relevant is this stuff currently.


minor tweaks to store histogram tool

Problem
histogram tool works great. A few tweaks from using it.
Summary of Changes
tweak it. Mainly add % size
Fixes #

lgtm.
Need to fix the clippy warning.

error: empty string literal in `eprintln!`
--
  | --> accounts-db/store-histogram/src/main.rs:127:5
  | \|
  | 127 \|     eprintln!("");
  | \|     ^^^^^^^^^^--^
  | \|               \|
  | \|               help: remove the empty string
  | \|
  | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#println_empty_string
  | = note: `-D clippy::println-empty-string` implied by `-D warnings`
  | = help: to override `-D warnings` add `#[allow(clippy::println_empty_string)]`


Fix signing_secret examples

@jimsynz You aren't. I used Application.fetch_env!/2 and had to return the tuple manually, but fetch_env/2 already does that of course.. Shouldn't have worked until 3am :)


Change sound device Mobile to Computer

[x] I have checked that a similar feature request does not already exist.

I'm always frustrated when I want to play sounds. when i put on my headphone which is connected to computer. then open scrcpy the sounds coming from phone. This is the minor feature we can make

this feature doesn't exist

https://github.com/Genymobile/scrcpy#audio-forwarding


SSLify or run builder services on localhost

The builder services either need to use the SSL certs we provide them or we can just start running things on localhost and stop caring.
If we do want to use SSL everywhere:

we need to modify the api-proxy nginx template to use them
modify the builder-api source to allow us to configure ssl
rebuild memcached to support

Sprint Planning Notes:

most of the services inside of automate use SSL to communicate with each other
this is the standard we've set so far

TODO: add notes on why this is the standard and whether it should be

right now the depot services listen unauthenticated on all interfaces

the goal of this work is for that to not be the case

option 1: all services use mTLS for authentication between services

we had the vision that different services would be running on different machines and we'd need to be able to secure those connections

that hasn't necessarily played out

we have played with using mTLS for authz

not clear how useful this has been

something else

option 2: all services only listen for connections on localhost
bldr services

automate-bldr-api
automate-bldr-proxy
automate-bldr-memcached

list of things needed

[ ] talk to other services via mTLS
[ ] bldr api doesn't support listening on mTLS
[ ] can build memcached w SSL support

not clear on mTLS implementations past that

because every service still needs to talk to the backends via mTLS we still need to do the work of setting up the certs and the CA for them
just listening on localhost is not as small of a task as we think it is

other things are calling back on the IP address

accidentally closed

If we need Automate Builder to support FIPS we are going to need to patch memcache and/or not use TLS on it.


9 various new icons

Description
Added 9 various icons

I used the guides from contributions and they fit perfectly for me. Do let me know how they aren't fitting so i can fix them

I tend to group all my paths and make either the width, height or both (depending which is bigger) 148px if stroke is 12px or 152px if its 8px. Some other factors can make it have to be 150px width/height sometimes. Maybe its an issue from my end. Wanna do that and I'll check if it still appears wrong for me
The sizes were not terribly wrong. Usually like 1 to 4 pixels off.

The Asphalt icon is a fill icon which means me resizing will make the stroke thicker. Not sure if it was meant to use fill or stroke.

Alright so I merged acode.svg made by @thismoon. Both look good
@thismoon and @Grabstertv, wanna decide which one would be more suitable? Maybe a Discord poll can be made or something?

I tend to group all my paths and make either the width, height or both (depending which is bigger) 148px if stroke is 12px or 152px if its 8px. Some other factors can make it have to be 150px width/height sometimes. Maybe its an issue from my end. Wanna do that and I'll check if it still appears wrong for me
The sizes were not terribly wrong. Usually like 1 to 4 pixels off.
The Asphalt icon is a fill icon which means me resizing will make the stroke thicker. Not sure if it was meant to use fill or stroke.

fixed scaling. Take a look and let me know if they look better now. As for the Asphalt 9 icon, i had to make it an outline from stroke to add some inner rounded corners.

Alright so I merged acode.svg made by @thismoon. Both look good
@thismoon and @Grabstertv, wanna decide which one would be more suitable? Maybe a Discord poll can be made or something?

I like their's tbh but lets have a community vote. Now we wait.

I like their's tbh but lets have a community vote. Now we wait.

I think yours is simpler therefore more fitting in lawnicons, and probably look better in small size (mine is detailed and look a bit weird)
by the way I wasn't aware of this pr, so it is a coincidence
also you won

by the way I wasn't aware of this pr, so it is a coincidence

Yeah thats what I thought so i just decided that making a poll will probably be best since its community based

B is the community choice


Refactor config file & custom rewrites

Been looking at more of the source code to try and understand what's going on before I start implementing the enhancements mentioned earlier.
I had missed the fact that we now have support of custom user rewrites. The code seemed very hard to maintain so I went ahead and simplified the parsing of config files (now depending on yaml and aeson, the latter already being a dependency of Agda).
I changed the flag --rewrite-rules to --config as it appears the file already contains more than rewrite rules (e.g: prelude import options). And it's likely we will use the config for more.
The config isn't parsed using unsafeIO anymore when parsing CLI options, but rather fetched safely using the preCompile hook of the backend.
Still need to update the documentation.

Done with this first refactor. Feedback welcome. We don't have tests for config files yet but I won't make it part of this PR.


Frontend: Add the best pictures from past editions in a carousel on welcome page?

something like this?: https://daisyui.com/components/carousel/#full-bleed-carousel
but with our cards:

I like it Picasso


Go layer - cannot run main in the way VSCode does

I'm doing a simple exercise in Go, which is comprised of 2 files:

A main.go file that runs the program
A deck.go file that holds a custom type deck, and a function called print() to be called from main.

So far, nothing unseen in other programming languages.
main.go:
func main(){
    variable := deck{"element1", "element2"} // if correctly imported, `deck` is a mnemonic type alias for `[]string`

    cards.print()
}

deck.go:
import "fmt"

type deck []string // now 'deck' is a type alias for `[]string`
    func (x deck) string {
        for i, card := range d { // for index and element ’card’ in all indexes in slice ‘cards’, print index and element
	fmt.Println(i, card)
     }

|
However, when hitting M-return x x to invoke go-run-main, I get an error message:
# command line arguments
./main.go:8:11: undefined: deck

This may happen because go-run-main is running go run main.go, without additional files.
Thing is, for a correct output, what should be run is go run main.go deck.go, as apparently VSCode does, since the latter shows the expected output: a printout of elements via the mentioned print() function above.
My question is: how can the go-run-main be customized, so that additional files can be added (or detected) as parameters?

go-run-main does not seem to be a function defined in go-mode.

Thanks for the prompt reply, I'll ask the Spacemacs maintainers, where the behavior was identified.
Cheers!


Use conditional tables instead of elseif

https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L16
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L30
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L74
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L89
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L104
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L119
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L130
https://github.com/a327ex/SNKRX/blob/master/enemies.lua#L141

you store the data in a table where the key is forcer/swarmer etc and it runs the function from this table

same for
https://github.com/a327ex/SNKRX/blob/master/player.lua#L17


detect confusable nicks during database import

If we import records for, e.g., bob and b0b, the lexicographically first entry (in this case b0b) will be unusable because of a skeleton conflict.
We should detect this and warn about it at import time. We can suggest that the operator configure casemapping: ascii to prevent the conflict.

I actually tested this and for authenticated users, force-nick-equals-account (or always-on) bypasses the code path that checks nick reservations (sensibly so). So being lexicographically first is irrelevant. The first user to connect will get through, and the second won't.


fix: improve the contrast of the autocomplete

Changes proposed
Proposal of issue #1

Additional context

Thanks for your contribution, it really helps in reading the auto-complete .


Add GitHub workflow to check spelling

It wasn't so bad as I expected :D

It wasn't so bad as I expected :D

Indeed the spelling so bad.
I've found LanguageTool which can do some basic proof-reading. There is a github action that I tried to integrate, but it didn't work correctly, so I've removed it. :/ Though running LanguageTool locally works fine for me locally, so I could write a simple shell script that runs it over the repo.


Laravel 8.72.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.72.0 branch.
Before merging, you need to:

Checkout the shift-ci-v8.72.0 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:warning: The Shift build servers run PHP 8. If you are still running PHP 7.4 or lower you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 7 environment.


Toggling views/panels has inconsistent focus behaviour

While investigating https://github.com/microsoft/vscode/issues/198293 I noticed that our action to toggle visibility of primary or secondary side bar or panel has inconsistent behaviour when it comes to passing focus to the view that becomes visible or not.
Notice when the explorer is active how focus remains in the editor:

And now with the SCM view:

First of all, these actions seem to call into methods to toggle visibility of the container via layout service which eventually calls into openPaneComposite, e.g. for the primary sidebar:
https://github.com/microsoft/vscode/blob/4520d915c98954dc96dd0bc00b8bb68181cbf2b6/src/vs/workbench/browser/layout.ts#L1750
The last parameter is a true to indicate that focus should move to that pane.
However, at the point where we want to focus the pane, it is not yet visible because the grid only updates a few lines below:
https://github.com/microsoft/vscode/blob/4520d915c98954dc96dd0bc00b8bb68181cbf2b6/src/vs/workbench/browser/layout.ts#L1756
Any view that implements focus by e.g. focussing the list will be a no-op because the DOM nodes are not yet visible. The SCM view is probably a bit async and that is why it works by chance.
I am actually not sure how to address this: people might have gotten used to the fact that these commands typically preserve focus for most of our views. I still think that the implementation is currently buggy around focussing the pane when the container becomes visible, so maybe a fix would need to be:

handle focus properly after the container in the grid is visible
make focus more explicit so that the toggling actions can explicitly pass in focus: false to preserve todays behaviour

//cc @sbatten

I started to investigate this last week. The the SCM view has a custom hack to make this work. But for all other views focus is not set correctly because the dom elements are not visible at the time of focus being set. Moving the this.workbenchGrid.setViewVisible(this.sideBarPartView, !hidden);  statement up before the open composite call fixes this. It's an easy fix for both side bars, however, there seems to be a bit more going on in the panel which is why I have not pushed a fix yet
https://github.com/microsoft/vscode/blob/0334df2899efa05a65a86f79e3f76588e17c667f/src/vs/workbench/browser/layout.ts#L1899-L1905

Yeah but I am also not certain what the right fix is here actually: should toggling focus or not. So far, most views are not focused and changing that might break muscle memory 🤔

Yeah I'm also not to sure. In most cases we do move focus. For example when you use the keybinding to show a view (Ctrl+Shift+E) so it might make sense to be consistent and always move focus.
@joaomoreno what do you think? You reported this behaviour as being a bug microsoft/vscode-copilot#9799

So far, most views are not focused and changing that might break muscle memory 🤔

I feel the same way: let's take the path of least breakage when trying to be consistent. I.e. let's not focus views when side bars come and go, unless it's an explicit call like workbench.scm.focus.

I agree fully for when these actions are triggered via keybinding. I wonder about other locations though, such as the layout controls where you already loose focus when you click it. In that case I think I would find it more natural to actually move focus into the respective view.

The fact that toggling panel visibility now doesn't focus terminal breaks a lot of muscle memory for me. I bet many users will feel this, especially because cmd+j is a lot easier to use than ctrl+`.

Reverted the suggested change as there seems to be problems with views such as the terminal and users muscle memory being broken.

(I don't want to push my use case too hard, but I would imagine I'm not the only person who'd suffer from this)

From an accessibility standpoint, I worry about breaking muscle memory for keyboard only or screen reader users, so thanks for reverting this.


chore: release main

:robot: I have created a release beep boop
finalbiome: 0.2.1
0.2.1 (2022-09-13)
Bug Fixes

deps: update rust crate clap to 3.2.21 (85685a1)

finalbiome-node: 0.1.1
0.1.1 (2022-09-13)
Bug Fixes

deps: update rust crate clap to 3.2.21 (85685a1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/finalbiome/finalbiome-node/releases/tag/finalbiome-node-v0.1.1 :sunflower:

:robot: Release is at https://github.com/finalbiome/finalbiome-node/releases/tag/finalbiome-v0.2.1 :sunflower:


Sc-8383: fix missing addresses on legal review page

Scope of changes
Sc-8383: fix missing addresses on legal review page
Type of change

[x] bug fix
[ ] new feature
[ ] documentation
[x] other (refactor)

Acceptance criteria

Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #735 (68c871a) into main (df6a38c) will not change coverage.
The diff coverage is 50.00%.

:exclamation: Current head 68c871a differs from pull request most recent head b3e3610. Consider uploading reports for the commit b3e3610 to get more accurate results

@@           Coverage Diff           @@
##             main     #735   +/-   ##
=======================================
  Coverage   43.42%   43.42%
=======================================
  Files         583      583
  Lines       17505    17505
  Branches     1317     1317
=======================================
  Hits         7601     7601
  Misses       8716     8716
  Partials     1188     1188

Impacted Files
Coverage Δ

...ui/src/components/TrixoQuestionnaireForm/index.tsx
2.27% <0.00%> (ø)

web/gds-user-ui/src/utils/address-utils.tsx
73.33% <ø> (ø)

...ponents/CertificateReview/TrixoReviewDataTable.tsx
60.60% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix: add dummy comments to both packages to trigger release

As per the title 🤞

:tada: This PR is included in version graphql-gene-v1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug] MediaElement crash when trying to play on iOS ( removing status observer error )

Description
MediaElement crashes when trying to play video
Steps to Reproduce

Create a page with a MediaElement tag

<xct:MediaElement x:Name="VideoPlayer" Source="{Binding Url}" ShowsPlaybackControls="True" IsLooping="False" MediaEnded="VideoPlayer_MediaEnded" AutoPlay="True" />

(In my case I have a PopupPage from Rg.Plugins.Popup, but I don't think that is the issue)

Open the page

await PopupNavigation.Instance.PushAsync(page);

Expected Behavior
MediaElement opens and plays normally
Actual Behavior
App crashes with an exception when video is about to play.
Exception thrown and app crashes:
Objective-C exception thrown.  Name: NSRangeException Reason: Cannot remove an observer <__XamarinObjectObserver 0x282bd7a00> for the key path "status" from <AVPlayerItem 0x282d4ae30> because it is not registered as an observer.
Basic Information

Version with issue: 1.0.0-preview1022

Platform Target Frameworks:

iOS:  12.4.9

Nuget Packages: Rg.Plugins.Popup

Workaround
I am not aware of any workaround.
I just upgraded from using Xamarin.Forms preview MediaElement to CommunityToolkit,
while XF one had issues, I had my own MediaElementRenderer which worked,
but since I upgraded to CommunityToolkit, my own renderer can't be used because some types are now internal
I can't register on SeekRequested / StateRequested, etc, and I cannot use my own renderer,
Reproduction Link
I may create a blank project to reproduce this issue later if the information above is not sufficient.

1.0.0-preview1022 is this correct? Because I see we're up to 1099 now, so would you mind upgrading and see if that fixes it, because I'm pretty sure we fixed this bug :)

my own renderer can't be used because some types are now internal
I can't register on SeekRequested / StateRequested, etc, and I cannot use my own renderer,

So, this is a different issue I guess. Why did you need that renderer? We'd love to take away the need for any renderers

Hi, this version mentioned is the latest available on the nightly nuget sources
https://pkgs.dev.azure.com/xamarin/public/_packaging/XamarinCommunityToolkitNightly/nuget/v3/index.json
should I download sources from github e try linking it manually?
I just thought the nightly would be updated already,
I will try downloading sources and adding them to my project anyway

I've got the exact same error:
Objective-C exception thrown.  Name: NSRangeException Reason: Cannot remove an observer <__XamarinObjectObserver 0x281d92140> for the key path "status" from <AVPlayerItem 0x281b02440> because it is not registered as an observer.
when using 1.0.0-preview1100
I will try to create a sample project so it can be reproduced

Bug reproduction project
MediaElementObserverCrash.zip

Yeah I'll have to clean up the nightly feed. Sorry about that :)
Could I ask you one more favor? Try the pre5 release on the official nuget from yesterday? Shouldn't be different from that preview1100 that was mentioned, but just to be sure. Thanks!

I tried it
xamarin.forms.<IP_ADDRESS>8-pre3
xamarin.communitytoolkit.1.0.0-pre5
An exception error occurs when executing the bind.
Objective-C exception thrown.  Name: NSRangeException Reason:
Cannot remove an observer <__XamarinObjectObserver 0x281951060>
for the key path "status" from <AVPlayerItem 0x281c19390>
because it is not registered as an observer.


Always have notification "initializing service"

Riot Version: 0.8.25 (G-b167)
Android version 9
From last update I always see notification from RIOT "initializing service". I can't remove this notification, only with killing RIOT, and restarting it, but after some time the message is visible again.

Same here, any workaround?

You can kill the app to make the notification to disappear.
Also the issue is fixed and will be released soon

@bmarty There was an update pushed to GPlay on the 25th of March, and this issue wasn't a part of that update.
How come?

I'm working on it :)

Removed in Riot 0.8.27


controller.loadIPAMDriver: Unwrap error type returned by PluginGetter

splitting this of https://github.com/docker/libnetwork/pull/2118, but a similar change was already merged in https://github.com/docker/libnetwork/pull/2183, which had a nice description so I blatantly copied that 😂
moby/moby commit https://github.com/moby/moby/commit/b27f70d45 (https://github.com/moby/moby/pull/36119) wraps the ErrNotFound error returned when
a plugin cannot be found, to include a backtrace.   This changes the type of the error, so contoller.loadIPAMDriver no longer converts it to a
ibnetwork plugin.NotFoundError.
This is a similar patch as was merged in aacec8e1adb71934c7fa56404c024eb74ad7fdc0 (https://github.com/docker/libnetwork/pull/2183)

ping @euanh @selansen @mavenugo PTAL

ping @arkodg @joeabbey ptal


feat(stream): support non-append-only  process time temporal join

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Resolve https://github.com/risingwavelabs/risingwave/issues/15218
Support non-append-only process time temporal join which will introduce a state called  memo_table to the temporal join.
memo_table's pk is join_key + left_pk + right_pk. And its columns are almost the same as the right table.
Write pattern:
for each left input row (with insert op), construct the memo table pk and insert the row.
Read pattern:
for each left input row (with delete op), construct pk prefix (join_key + left_pk) to fetch rows.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

This looks good.
Correct me if I'm wrong: we face the exactly same issue with indeterministic UDFs. Is it possible to make the memoization(materialization) into a separate operator? So that we can reuse it for these similar cases.

we face the exactly same issue with indeterministic UDFs. Is it possible to make the memoization(materialization) into a separate operator

There should be an abstraction for the logic, but IMO it shouldn’t be a separate operator.

This looks good.
Correct me if I'm wrong: we face the exactly same issue with indeterministic UDFs. Is it possible to make the memoization(materialization) into a separate operator? So that we can reuse it for these similar cases.

Even though deterministic UDFs also need a memo table, I don't think we can use a shared operator to handle both cases. Because the memo table access pattern and the way how to reconstruct rows from the memo table are totally different.

Because the memo table access pattern and the way how to reconstruct rows from the memo table are totally different.

You're right. I was solely focused on using temporal join for point-lookup. 😄


Build issue with Xcode 10.2.1

Showing Recent Messages
Build system information
error: /Users/iraniya/Downloads/SwiftyGiphy-master/Pods/Target Support Files/Pods-SwiftyGiphySample/Pods-SwiftyGiphySample.debug.xcconfig: unable to open file (in target "SwiftyGiphySample" in project "SwiftyGiphy") (in target 'SwiftyGiphySample')
I have tried pod install pod update but stuck at
"Analyzing dependencies
[!] The version of CocoaPods used to generate the lockfile (1.6.1) is higher than the version of the current executable (1.5.3). Incompatibility issues may arise.
Fetching podspec for SwiftyGiphy from ../SwiftyGiphy
[!] No podspec found for SwiftyGiphy in ../SwiftyGiphy"
Thanks in advance :-)

Hi! Did anybody solve this issue? Thanks!

Just save this in Podfile:
Uncomment the next line to define a global platform for your project
platform :ios, '9.0'
target 'SwiftyGiphySample' do
Comment the next line if you're not using Swift and don't want to use dynamic frameworks
use_frameworks!
Pods for SwiftyGiphySample
pod 'SwiftyGiphy'
end


Input and textarea with autosize feature

There's a couple of different approaches.
The first one is to position the input/textarea absolutely inside of a .container, and place an invisible statically positioned .ghost alongside it, and make the .ghost content be synced with the input/textarea.
Another is to attach the .ghost element to the body, and use it to recalculate the proper size in px for the input/textarea whenever it's value changes. Multiple instances can share the same .ghost.
References:

Creating an Auto-Growing Text Input by Brian Chu
react-input-autosize
react-autosize-textarea
autosize-input
autosize

There is vue-autosize-textarea by @egoist and vue-autosize by @mage3k, both for the textarea.

vue-comps/vue-textarea

With the autosize npm package, implementing this feature is actually really trivial. See here for an example.

I would love Auto resize text area in Vue!
I've been looking at these links you guys gave, but none of them had clear instructions on how to install them... I'm still a newbie. >.>

@mesqueeb Here's a simple example in action.

I put this together for text inputs (not textareas) today - https://github.com/syropian/vue-input-autosize

@chrisvfritz doesn't work in my browser (Chrome 62.0.3202.94)

@warent That's because the example was written for Vue 1, but pinned to the latest version of Vue. I just updated the link and it should work now. 🙂

Since handling this use case without a dedicated Vue wrapper is so simple that Egoist even deprecated his package, I'm going to close this.

For future reference to anyone else reading, I ended up using mage3k/vue-autosize and it works nicely.


[WIP] Sync ui metrics

Address #1278

Brainstorm:

even though details and canvas metrics are are requested at different times, the samples' dates should be the same, there is potential to sync them
maybe remove metrics feeder HOC, to get the sync right between details and canvas, then re-add feeder
central metrics object for that stores metrics by node id, can be queried with a timestamp, feeder HOC feeds that timestamp
global metrics timestamp clock, dynamically adjusted based on what sample ranges are available, keeps all metrics in sync

Mmm, I like the idea of global metrics timestamp clock!

LGTM


Error using selected REPL back-end

Python 3.7 (64 bit)
When using the Python interactive window in Visual Studio 2019, I always get the following error:
Error using selected REPL back-end:
IPython mode requires IPython 0.11 or later: No module named 'IPython'
Using standard backend instead

The main docs assume you have minconda install...
[(https://docs.microsoft.com/en-us/visualstudio/python/installing-python-support-in-visual-studio?view=vs-2019)]
If you want to manually install the needed packages for the REPL take a look at the Packages (PyPI) tab in the Python Environments window.
[https://docs.microsoft.com/en-us/visualstudio/python/python-environments-window-tab-reference?view=vs-2019]
Search for the follwing packages:
ipython
ipykernel
and then click "Run command: pip install XXX" for each
Note: there is a bug with latest version of pyzmq (18.0.1) that ipython includes, so you will need to downgrade it by:

Enter pyzmq in the search box in Packages(PyPI) tab
click the "x" to uninstall it
type "pyzmq==17.1.3"
Click "Run command: pip install pyzmq==17.1.3

Please reopen if it the work around doesn't address the issue


use presence of window for SSR injection warnings

in idiomatic react-testing-library tests, it shouldn't be necessary to inject reducers/middleware manually

fixed in 0.12.1


Unavailable

Plugin disappeared from sidebar, when removed it is no longer available to be installed from plugin hub.

Experiencing the same on macOS.

The plugin is currently disabled by the RuneLite team because some users were experiencing an error that made RuneLite unusable. We have a general idea of the cause, but do not have a fix ready or timeline given the holidays.

PR ready needs testing #88


Allow Companion to switch tally lights

Hi,
We use a ptz-camera as main camera. Companion is used to use presets on this camera. One of these presets is a long distance shot of the organ and choir. Is it possible to set a specific tally for a preset so that the choir can see when the camera is aimed at them?

Yes. In Companion, you can add an action upon preset recall that uses the Generic OSC module to send OSC to Tally Arbiter for this. Create a new device for each preset, and assign it a device source address for your OSC source in Tally Arbiter.
In TA, create a new OSC source type.
Then create two Devices, Organ, and Choir. For Organ, assign a Device Source of organ to your OSC source. Similar with choir.
Then, for your Organ preset recall in Companion, in addition to having the PTZ recall action, you would have the following:

Generic OSC to TA: /tally/program_on with a string argument of organ
Generic OSC to TA: /tally/program_off with a string argument of choir

Let me know how this goes.

Yes. In Companion, you can add an action upon preset recall that uses the Generic OSC module to send OSC to Tally Arbiter for this. Create a new device for each preset, and assign it a device source address for your OSC source in Tally Arbiter.
In TA, create a new OSC source type.
Then create two Devices, Organ, and Choir. For Organ, assign a Device Source of organ to your OSC source. Similar with choir.
Then, for your Organ preset recall in Companion, in addition to having the PTZ recall action, you would have the following:

Generic OSC to TA: /tally/program_on with a string argument of organ
Generic OSC to TA: /tally/program_off with a string argument of choir

Let me know how this goes.

Hi Joseph,
This works great! Thank you!

Op 11 jan. 2021, om 17:33 heeft Joseph Adams<EMAIL_ADDRESS>het volgende geschreven:
Yes. In Companion, you can add an action upon preset recall that uses the Generic OSC module to send OSC to Tally Arbiter for this. Create a new device for each preset, and assign it a device source address for your OSC source in Tally Arbiter.
In TA, create a new OSC source type.
Then create two Devices, Organ, and Choir. For Organ, assign a Device Source of organ to your OSC source. Similar with choir.
Then, for your Organ preset recall in Companion, in addition to having the PTZ recall action, you would have the following:
Generic OSC to TA: /tally/program_on with a string argument of organ
Generic OSC to TA: /tally/program_off with a string argument of choir
Let me know how this goes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/josephdadams/TallyArbiter/issues/43#issuecomment-758071066, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFBN5SMTQGBS4L5EAIKBANDSZMR67ANCNFSM4V5ZCWJA.

Hi Joseph,
This works great! Thank you!

Op 11 jan. 2021, om 17:33 heeft Joseph Adams<EMAIL_ADDRESS>het volgende geschreven:
Yes. In Companion, you can add an action upon preset recall that uses the Generic OSC module to send OSC to Tally Arbiter for this. Create a new device for each preset, and assign it a device source address for your OSC source in Tally Arbiter.
In TA, create a new OSC source type.
Then create two Devices, Organ, and Choir. For Organ, assign a Device Source of organ to your OSC source. Similar with choir.
Then, for your Organ preset recall in Companion, in addition to having the PTZ recall action, you would have the following:
Generic OSC to TA: /tally/program_on with a string argument of organ
Generic OSC to TA: /tally/program_off with a string argument of choir
Let me know how this goes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/josephdadams/TallyArbiter/issues/43#issuecomment-758071066, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFBN5SMTQGBS4L5EAIKBANDSZMR67ANCNFSM4V5ZCWJA.

This works great! Thank you!

This works great! Thank you!


Unc0ver reboots phone at step 2 during exploiting kernel.

My phone is the iPhone XR, I have the latest version of Unc0ver which I installed using altstore. My current version for my phone is 13.4 beta. It loads normally until step 2. It reaches exploiting kernel and then reboots the phone. I have tried for about 26 times and still havent got past exploting kernel

13.4 beta is not supported.  How many times are you going to ask this?  The answer is not likely to change any time soon as beta IOS are not usually priority.


Projection coordinate lookup

 Koordinaten in der Projektion erst checken, dann umprojezieren.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 17 (76.47%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 84.142%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

operators/src/adapters/raster_subquery_adapter.rs
13
17
76.47%

Files with Coverage Reduction
New Missed Lines
%

datatypes/src/error.rs
1
0%

datatypes/src/operations/reproject.rs
4
85.51%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
23887

Relevant Lines:
28389

💛  - Coveralls

redesigned the bounds check and generate valid bounds only once


[Suggestion] Modify colours for block header in dark and monochrome theme

Hi,
Still loving your theme, I've been using poet for a few weeks and just switched to monochrome.
I would like to suggest some colour tweaks to make it even better :

In Monochrome (and dark poet) #BEGIN_SRC like headers are not as much distinctive as they are in poet.

Could it be possible to make theme a little darker ? I think the global readability will be enhanced with such tweaks.
Thanks

That makes sense: the monochrome themes are completely automatically generated at the moment by desaturating and inverting the manually chosen colors in poet. I'll see if I can tweak that slightly to increase the generated value.
I'm hoping to release a manually created dark brown version of poet at some point as well.

That's great news :)
Will hang around to make sure I test that brown one.
Le 11 octobre 2018 20:56:05 GMT+02:00, Kunal Bhalla<EMAIL_ADDRESS>a écrit :

That makes sense: the monochrome themes are completely automatically
generated at the moment by desaturating and inverting the manually
chosen colors in poet. I'll see if I can tweak that slightly to
increase the generated value.
I'm hoping to release a manually created dark brown version of poet at
some point as well.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/kunalb/poet/issues/6#issuecomment-429078207

--
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.


nixos/bind: harden systemd service

Add systemd sandboxing, partially taken from https://salsa.debian.org/dns-team/bind9/-/merge_requests/1
✗ SystemCallFilter=~@resources                                System call deny list defined for service, and @resources is not included (e.g. ioprio_set is allowed)      0.2
✗ SystemCallFilter=~@privileged                               System call deny list defined for service, and @privileged is not included (e.g. chown is allowed)          0.2
✗ SystemCallFilter=~@obsolete                                 System call deny list defined for service, and @obsolete is not included (e.g. _sysctl is allowed)          0.1
✗ RootDirectory=/RootImage=                                   Service runs within the host's root directory                                                               0.1
✗ AmbientCapabilities=                                        Service process receives ambient capabilities                                                               0.1
✗ RestrictAddressFamilies=~AF_NETLINK                         Service may allocate netlink sockets                                                                        0.1
✗ RestrictAddressFamilies=~AF_UNIX                            Service may allocate local sockets                                                                          0.1
✗ RestrictAddressFamilies=~AF_(INET|INET6)                    Service may allocate Internet sockets                                                                       0.3
✗ ProtectSystem=                                              Service has very limited write access to the OS file hierarchy                                              0.1
✗ CapabilityBoundingSet=~CAP_NET_(BIND_SERVICE|BROADCAST|RAW) Service has elevated networking privileges                                                                  0.1
✗ PrivateNetwork=                                             Service has access to the host's network                                                                    0.5
✗ PrivateUsers=                                               Service has access to other users                                                                           0.2
✗ DeviceAllow=                                                Service has a device ACL with some special devices: char-rtc:r                                              0.1
✗ IPAddressDeny=                                              Service does not define an IP address allow list                                                            0.2
✗ UMask=                                                      Files created by service are world-readable by default                                                      0.1

→ Overall exposure level for bind.service: 1.8 OK 🙂

I'm not sure if bind writes to files next to the zone files, if it does ReadWritePaths needs the dir of the zone file rather than the file itself.
Things done

[x] Tested on my server (running as an recursive and secondary authoritative) by overwriting the module in config

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[ ] Tested basic functionality of all binary files (usually in ./result/bin/)

24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I remembered correctly that bind wants to write files next to zone files (for example the journal), in which case ReadWritePaths needs the folder. I am not sure what is the best way to do that in the nix language.


Replace dl-elements by simple level-3-headings

There was no way to place a small comment, so making this pull request for a small change.
The sections about different Windows versions are created using dl, dt and dd HTML tags. Not only does that seem semantically incorrect (because this is not a dictionary with descriptions / definitions), but more importantly: they currently aren't formatted as a header on the microsoft docs site (see https://docs.microsoft.com/nl-nl/windows/win32/sysinfo/acquiring-high-resolution-time-stamps). This makes the structure less obvious and makes reading a bit harder than necessary. This change fixes that by using simple headers.
Thanks!

@ElmarJ : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Enable nexus deploy for incubator-kie-optaplanner

Nexus snapshot deploy is currently disabled for incubator-kie-optaplanner. Now that nightly build is fixed we can enable the nexus deploy in pipeline configuration.

Finished by @cimbalek


Use ring-buffer in StreamReader ChunkedVideoBuffer

Currently, all the Buffer implementation uses std::deque<torch::Tensor> for storing decoded frames.
In case of video stream, the decoded frames are converted to tensors newly allocated each time, then
the resulting tensors are stored in the std::dequeu. They are concatenated at the time they are popped.
This commit replaces the buffer implementation for ChunkedVideoBuffer with ring buffer implementation.
In the new implementation the buffer is pre-allocated. Each time a decoded frame is given, it is written to
the buffer. No memory allocation happens. At the time of popping, the slice of the buffer is created and
then cloned. Memory allocations still happen at the clone time, but this reduces the number of memory copies.
As a result, the wall time performance is 30~40% improved.

Is there any way to document some baseline numbers here? I assume you have them in another notebook or something.

Is there any way to document some baseline numbers here? I assume you have them in another notebook or something.

Will do once the CL is finalized.

Closing due to conflict