Multipe Charts Per Column

We often create a chart per variable
e.g. A Batch can be of multiple Wafers and Wafers can be of different Technologies
I would create a chart Per Technology so that the data on each chart is comparable
Can this be done today with Brunel ?

Yes, but the caveat is that you would need to explicitly set the axes for each chart so they have the same range, as otherwise Brunel will auto fit them.
Here is a base example of one chart per variable:
data('sample:US States.csv') x(region) y(latitude) | x(region) y(longitude)
When you try this, you see the y axes are not comparable, so to make this work, you'd need to do this
data('sample:US States.csv') x(region) y(latitude:[-160,70]) | x(region) y(longitude:[-160,70])

Close but I think it was my explanation was not clear ...
What if the data is vertical and a column LABEL has the Variable name in it - 'longitude' and 'latitude' - and I want a chart for each - I don;t want to know what all those labels are - we tend to work with a lot of vertical data.
data('sample:US States.csv') x(region) y(y) chartby(LABEL)
and it would spit out two charts one for longitude and the other for latitude

Ok, got it. This is a feature we have wanted for a while. We term it 'faceting' and other teams can call it "small multiples" or sometimes "trellis plots"
A lot of the details for it are quite tricky -- specifying the styles and layouts for the "inner charts" as well as axes and whether or not to have the scales consistent across charts or varying per chart.
Our approach is going to be to define an outer element -- the main chart -- that sets up the spaces into which the little charts would be drawn. Something like: data('sample:US States.csv') bar x(LABEL) and that would allow you to specify axes, horizontal or vertical layouts, use two fields -- lots of options with the standard syntax.
THEN (and this is the fun part) we would add a second chart to be inserted into each of the elements of the main chart (the inner one), filtering its data to match the data in that space. So something like this: data('sample:US States.csv') bar x(LABEL)  > x(region) y(y)
Again, you would then get complete control over the display in this inner level.
So the bad news is ... not ready yet. The good news is; we are working on it. This following does something close to what you want:
data('sample:US States.csv') x(region) style(".element{opacity:0.1} line,path {visibility:hidden}") axes(x:'') split(#count) > data('sample:US States.csv') x(latitude) y(longitude) axes(x,y)
note all the extra annoying fiddly code we'll make optional later. But it basically works. The two main major bits of work we need to fix are: (1) also allowing ">>" to work which will say to make the axes for each inner chart independent of each other, not fixed at the same value, and (2) better layout and fixing some issues with the scales.
So, this new feature is on the way, and you're request has bumped it up the priority table!


parameters and variable have no purpose?

I'm not sure whether this would be considered an issue in itself, but perhaps is the cause of other issues?
It seems that, in Private Function json_BufferToString (below), the parameter ByVal json_BufferLength As Long is required but not referenced anywhere within the function.  The function is called several times elsewhere in the module, with the variable supplied.  If it doesn't have a "purpose", why is in even listed?
Also, in Private Function json_StringIsLargeNumber, declared variable json_Index As Long is unused.
Is this unnecessary overhead?
Private Function json_BufferToString(ByRef json_buffer As String, ByVal json_BufferPosition As Long) As String
#If Mac Then
	json_BufferToString = json_buffer
#Else
	If json_BufferPosition > 0 Then
		json_BufferToString = VBA.Left$(json_buffer, json_BufferPosition \ 2)
	End If
#End If
End Function

Thanks for the code review @ashleedawg, I'll get those cleaned up.


Listening "enter" press when picking a color

Yo!
I love your plug-in—it excluded the need of opening Photoshop for picking colors, thank you very much for it!
By the bye, the matter here is slightly serious: every time I select a color then press enter, it closes the popup without pasting the hex/rgba/etc of the selected color.
Is this a bug? A non-thought feature?
Thanks.

Thanks! I recently rewrote the entire color-picker source. Replacing or pasting current color on enter used to be a feature, but I seem to have forgotten to include it. Expect a patch in a few minutes!

Thanks again, @chiefGui, this is now included in v2.0.5.

Thank you very much, man! :+1:


EarClippingTriangulator produces incorrect results with simple concave polygon

Issue details
Using the test case below, it appears that the Ear Clipping triangulator is producing incorrect results. It only appears when I have a concave shape like below. If I drag point 3 to be convex, the triangulation is correct. When I drag point 3 to be concave like in the image below, I (sometimes) get the incorrect triangulation. Specifically the [0, 3, 2] triangle in incorrect.
If I scale the points by 10x, I get the same results so I don't believe that it is a precision error. If I translate the first point to the origin (0,0), it appears I get the correct results.
Reproduction steps/code

float[] f = new float[] {47668.66f, 61427.285f, 47662.305f, 61433.39f, 47667.86f, 61441.543f, 47666.0f, 61434.414f};
mEarClippingTriangulator.computeTriangles(f);
// Produces result: [0, 3, 2, 2, 1, 0]
// Example expected result: [0, 1, 2, 1, 2, 3]

Version of LibGDX and/or relevant dependencies
1.9.3 running on Android

Please provide an SSCCE and I'll reopen. Eg: http://n4te.com/x/235-7ujK.txt

Hopefully this visualizes this in a clear way. The original polygon is outlined in green. The triangles it produces are filled in red with 50% alpha. You can see below that it produces two triangles that overlap each other and also fall outside the polygon.
public class EarClip extends ApplicationAdapter
{
    private ShapeRenderer renderer;
    EarClippingTriangulator triangulator = new EarClippingTriangulator();

    public void create()
    {
        renderer = new ShapeRenderer();
    }

    public void render()
    {
        float[] points = new float[] {
                47668.66f, 61427.285f, //
                47662.305f, 61433.39f, //
                47667.86f, 61441.543f, //
                47666.0f, 61434.414f //
        };

        ShortArray triangles = triangulator.computeTriangles(points);

        Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);
        Gdx.gl.glEnable(GL20.GL_BLEND);
        Gdx.gl.glBlendFunc(GL20.GL_SRC_ALPHA, GL20.GL_ONE_MINUS_SRC_ALPHA);

        renderer.begin(ShapeType.Filled);
        renderer.setColor(new Color(1,0,0,0.5f));
        for (int i = 0; i < triangles.size; i += 3) {
            int p1 = triangles.get(i) * 2;
            int p2 = triangles.get(i + 1) * 2;
            int p3 = triangles.get(i + 2) * 2;
            renderer.triangle( //
                    points[p1], points[p1 + 1], //
                    points[p2], points[p2 + 1], //
                    points[p3], points[p3 + 1] //
            );
        }
        renderer.end();

        renderer.begin(ShapeType.Line);
        renderer.setColor(new Color(0,1,0,0.5f));
        int last = points.length - 2;
        for (int i = 0; i < points.length; i += 2) {
            renderer.line(points[i], points[i + 1], points[last], points[last + 1]);
            last = i;
        }
        renderer.end();
    }

    public void resize(int width, int height)
    {
        renderer.getProjectionMatrix().setToOrtho2D(0, 0, width, height);
        renderer.getTransformMatrix().idt();
        // Scale it up a bit
        renderer.getTransformMatrix().scl(10f, 10f, 1);
        // Center it in the camera
        renderer.getTransformMatrix().translate(-47665.533f + (width / 20f), -61433.867f + (height / 20f), 0);
    }

    public static void main(String[] args) throws Exception
    {
        new LwjglApplication(new EarClip());
    }
}

Produces output:

Thanks. Working at such large scales isn't great. Try this:
	private ShapeRenderer renderer;
	EarClippingTriangulator triangulator = new EarClippingTriangulator();

	public void create () {
		renderer = new ShapeRenderer();
		renderer.getTransformMatrix().translate(100, 100, 0);
	}

	static public void ensureCCW (float[] polygon, int offset, int count) {
		int lastX = offset + count - 2;
		for (int i = offset, n = offset + count / 2; i < n; i += 2) {
			int other = lastX - i;
			float x = polygon[i];
			float y = polygon[i + 1];
			polygon[i] = polygon[other];
			polygon[i + 1] = polygon[other + 1];
			polygon[other] = x;
			polygon[other + 1] = y;
		}
	}

	public void render () {
		float[] points = new float[] { //
			47668.66f, 61427.285f, //
			47662.305f, 61433.39f, //
			47667.86f, 61441.543f, //
			47666.0f, 61434.414f //
		};
		for (int i = 0, n = points.length; i < n; i += 2) {
			points[i] -= 47600;
			points[i] *= 10;
			points[i] -= 500;
			points[i + 1] -= 61400;
			points[i + 1] *= 10;
			points[i + 1] -= 100;
		}
		ShortArray triangles = triangulator.computeTriangles(points);

		Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);
		renderer.begin(ShapeType.Line);

		renderer.setColor(Color.RED);
		for (int i = 0; i < triangles.size; i += 3) {
			int p1 = triangles.get(i) * 2;
			int p2 = triangles.get(i + 1) * 2;
			int p3 = triangles.get(i + 2) * 2;
			renderer.triangle( //
				points[p1], points[p1 + 1], //
				points[p2], points[p2 + 1], //
				points[p3], points[p3 + 1] //
			);
		}

		renderer.setColor(Color.GREEN);
		renderer.polygon(points);
		renderer.end();
	}

	public void resize (int width, int height) {
		renderer.getProjectionMatrix().setToOrtho2D(0, 0, width, height);
	}

	public static void main (String[] args) throws Exception {
		new LwjglApplication(new EarClip());
	}
}

That shows the magnitude of your vertices is the problem. Specifically, areVerticesClockwise sums the area of triangles which doesn't fit in a float.

As for a solution, I'm not sure we should use doubles in areVerticesClockwise (which really should be using GeometryUtils isClockwise). It would fix your problem but may negatively affect others who don't need the doubles. You could adjust your vertices, eg find the lowest X and Y values and subtract that from all vertices, or you could fork the ear clipper for your project.

I see - so p1x*p2y is losing precision and therefore the CCW check is failing. That makes sense.

If I translate the first point to the origin (0,0), it appears I get the correct results.

So that makes sense why doing this works. I will probably employ this as a workaround to the points in my code before calling the triangulation. We could consider doing it automatically in the areVerticesClockwise method.
Perhaps this is also the issue with #5314 although those points seem like they would be within tolerance.


Loggers cannot be reused

Loggers are disposed after calling Invoke-CommandWithLogging. Calling the command multiple times with the same loggers does not cause errors but will only log the first execution to the configured loggers since they are disposed after first use.
The console logger is the only one this working after being disposed.

It is important that loggers are disposed so rather than not disposing them, using the Invoke-CommandWithLogging command now fails if any loggers are disposed.


NullPointerException in the method addCraftingInformationToTooltip

Describe the bug
Bug Report Summary for Developers:
The crash occurs while rendering the Ammo Press GUI in Modern Warfare Cubed. Specifically, it fails when trying to display crafting information tooltips for ammo press recipes. The issue is caused by a NullPointerException in the method addCraftingInformationToTooltip of GUIContainerAmmoPress.
Details:

Error location:

com.paneedah.weaponlib.crafting.ammopress.GUIContainerAmmoPress.addCraftingInformationToTooltip(GUIContainerAmmoPress.java:168)
The error occurs when crafting tooltips are added to the GUI, causing a rendering failure.

Stacktrace Highlights:

Error in: GUIContainerAmmoPress.addCraftingInformationToTooltip()
Method called during: drawTooltips() in GUIContainerStation
Resulting in: NullPointerException while rendering the ammo press screen.

Affected Mods/Components:

Modern Warfare Cubed (mwc: 0.1.8): The crash happens in its ammo crafting GUI logic.
OptiFine: It's active, which might alter rendering behavior but isn't necessarily the root cause.
CoreMods: Several mods affecting rendering and GUI handling are present, but weaponlib from Modern Warfare Cubed is the direct source of the error.

Possible Causes:

Null Data in Tooltips: The crash seems related to the crafting tooltip data being null or improperly initialized in the addCraftingInformationToTooltip method.
Mod Conflicts: Potential conflicts with other mods (e.g., MalisisCore, Techguns, or OptiFine) might be affecting rendering or GUI handling.
OptiFine Interference: While not directly mentioned in the error, OptiFine may interfere with GUI rendering and should be tested with and without it enabled.

Steps to Reproduce:

Open the Ammo Press GUI in Modern Warfare Cubed.
Attempt to view or interact with crafting tooltips for any ammo press recipe.
The crash occurs when the tooltip should render.

Reproducibility

[X] I reproduced this issue with as few other mods as possible installed.
[x] I am unable to reproduce this issue consistently.

Screenshots and/or videos
No response
Logs
https://mclo.gs/4kz7GFl
Debug Logs
No response
Crash Reports
No response
MWC Version
0.1.8
Forge Version
1.12.2
Red Core Version
0.5.1
MixinBooter Version
9.4
Java Version
1.8
Java Virtual Machine
Hotspot
Operating System
Windows 10
GPU Vendor
NVIDIA
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Did you ask ChatGPT to write this report for you? 🤔 This doesn't sound very human.
Regardless, I'm sure @Desoroxxx will look into this.

To some extent, yes, I used the chat gpt to correct my grammatical errors as well as Eto all the normal form. I wrote it all myself and I figured it out. I am a man from Ukraine and do not really know English.

And thank you so much for your response

Fixed in 0.1.9


Check incomplete foreign-keys

See http://stackoverflow.com/questions/7947085/are-incomplete-key-references-in-propel-useful
When foreign-key points to a table with composite primary key but we have only defined one reference in our foreign key then we should throw a error.

It may be useful to update the documentation somewhere near this link: http://propelorm.org/documentation/04-relationships.html#more-than-two-foreign-keys


Can`t see other Players

What version of ViaVersion are you using? Type /ver viaversion
0.5.8 (It also isn`t working in newer versions!)
What version of Spigot are you using? Type /ver
http://prntscr.com/advsil
What plugins are you using? Type /plugins
http://prntscr.com/advsu7
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
Yes, im using the newest version of BungeeCord
How does this error happen? login? Using an item?
Its completely random. Sometime everyone can see everyone, and sometimes nobody saw nobody.
Is there an error in the console? Use pastebin.com. Is there a kick message?
There are no errors :c

Have you tried without AAC?

Yes, it is also not working.

Same here, some Players are invisible. Don't know if it is a 1.9 Bug or a ViaVersion Bug.

@XNoki200010 I think its a ViaVersion problem, because i also cant see other players in the 1.8.9, but only on ViaVersion servers

@XNoki200010 could you list your spigot version with your plugins and bukkit version with your plugins as well?

@HugoDaBosss if you mean me, its all in the Screenshots up here.

@MultiDJRoni Are players sometimes invisible if they walk/fly in some places? But become visible if they fly/walk back?

@Matsv They are completely random invisible. I also cant see them in the TAB until the Server restarts.

@MultiDJRoni Does it also happen without TablistPrefix?

Im not using Tablist Prefix on my BedWars servers

But you said it also happend to 1.8.9 users?

Yes, it happens to everyone. 1.9 and 1.8.x

@MultiDJRoni Does it also happen without viaversion?

Nope

Going to close this issue, you may say the bug does not occur without ViaVersion but sadly ViaVersion doesn't touch any 1.8 clients. I've seen this in vanilla bukkit and it's a teleport packet order issue.


Update climate_HassClimateGetTemperature.yaml

@LaStrada please coordinate with @OliverSkivdal to see if this PR is still needed

This PR is very outdated, and there has been major changes since this was opened.


Registration Issue

For porton mail imaganed to download it but the biggest problem or issue is registration ,ihave tired all ways to register but it failed
Additional context
Add any other context or screenshots about the feature request here.

Manged

Okay thaxs good evening
On Sun, Mar 21, 2021, 8:25 PM Bart Butler @.***> wrote:

Closed #237 https://github.com/ProtonMail/WebClient/issues/237.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ProtonMail/WebClient/issues/237#event-4486985179, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATKTOFWN3JAH2JEDGNB2HF3TEYT2JANCNFSM4ZRTQRJA
.


Proper way to avoid kebab-case translation?

Hi,
I see that d3Render changes camel-case in JavaScript to kebab-case.  I am wondering how I should specify properties like markerHeight that must appear as a camel-case attribute in the SVG?
Ref: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/markerHeight
Thanks!

Hi emrul,
Right now, all attributes are converted from camel case to kebab case.
I totally forgot about real camel case attributes! Probably the easiest thing is to have a list to check before camel->kebab conversion.
https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute has a list I could work off.
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes doesn't seem to have any camel case attributes (makes my job easier)
Thanks for letting me know.

Hi emrul,
Should be fixed in v0.2.5
https://github.com/unkleho/d3-render/blob/v0.2.5/src/render.ts#L325


Running into errors trying to train

Hi, I am trying to train a playable the AI to learn ZvT, but am encountering some errors.

I made a trainingreplays folder and put some replays from Patch 4.9. Command I ran was python -m distar.bin.sl_train --data ./trainingreplays:

Start decoding replay with player 1, path: ./trainingreplays\reynorvssoul1.SC2Replay
12380 [ERROR] parse replay error SC2APIProtocol.ResponseReplayInfo.Error.InvalidReplayPath: ''
  File "C:\Users\Steve Lu\Downloads\di-star\di-star\distar\agent\default\replay_decoder.py", line 380, in run
    self._replay_info = self._parse_replay_info()
  File "C:\Users\Steve Lu\Downloads\di-star\di-star\distar\agent\default\replay_decoder.py", line 343, in _parse_replay_info
    replay_info = self._controller.replay_info(replay_path=self._replay_path)
  File "C:\Users\Steve Lu\Downloads\di-star\di-star\distar\pysc2\lib\remote_controller.py", line 69, in _check_error
    return check_error(func(*args, **kwargs), error_enum)
  File "C:\Users\Steve Lu\Downloads\di-star\di-star\distar\pysc2\lib\remote_controller.py", line 60, in check_error
    raise RequestError("%s.%s: '%s'" % (enum_name, error_name, details))

I tried the reinforcement learning selfplay command python -m disatr.bin.rl_train --task selfplay. Am I understanding correctly that selfplay is when it trains by playing against me?

train !!!!!!!!!!!!!!!!!!!
Traceback (most recent call last):
  File "C:\Users\Steve Lu\Downloads\miniconda\lib\multiprocessing\process.py", line 315, in _bootstrap
    self.run()
  File "C:\Users\Steve Lu\Downloads\miniconda\lib\multiprocessing\process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
TypeError: learner_run() takes 2 positional arguments but 5 were given
{'checkpoint_paths': ['C:\\Users\\Steve Lu\\Downloads\\di-star\\di-star\\experiments\\test\\league_models\\MP0H1_MP0_ckpt.pth.tar', 'C:\\Users\\Steve Lu\\Downloads\\di-star\\di-star\\experiments\\test\\league_models\\MP0_ckpt.pth.tar'], 'env_info': {'map_name': 'random', 'player_ids': ['MP0H1', 'MP0'], 'side_id': [0, 1]}, 'frac_ids': [1, 1], 'pipelines': ['default', 'default'], 'player_ids': ['MP0H1', 'MP0'], 'send_data_players': ['MP0'], 'side_ids': [0, 1], 'successive_ids': ['none', 'MP0'], 'teacher_checkpoint_paths': ['none', 'C:\\Users\\Steve Lu\\Downloads\\di-star\\di-star\\distar\\bin\\sl_model.pth'], 'teacher_player_ids': ['none', 'model'], 'update_players': ['MP0'], 'z_path': ['3map.json', '3map.json'], 'z_prob': [0.25, 0.25]}
run worker for token: MP0model at <IP_ADDRESS>: 18992
 * Serving Flask app 'distar.ctools.worker.coordinator.coordinator' (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
  We are working on adding support to all maps and all versions on Windows.
2. selfplay means the agent playing against itself. The log shows 2 errors, first one(learner_run) is fixed with the new commit, please pull the new code; 2nd one comes from 'terran' is not included in the statistics Z file, we release a new training guidance please check how to generate new statistics Z file here. Or you can switch back to zerg, the Reinforcement learning should be working fine.

Thanks for the info! I'll read the guide and try it out when I have time

All maps and versions are supported now on Windows! I think it will work fine with most replays now.  need to add a few new units in some new maps i.


refactor(meta): reduce meta store write in hummock service

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Refactor hummock service:

Remove 2 persistent states: stale_sstables and sstable_id_infos.
Replace hummock_versions with checkpoint_version.
Simplify vacuum.

TODO:

(next PR) GC orphan SSTs in object store

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
https://github.com/singularity-data/risingwave/issues/4037

@lmatz FYI the optimization from https://github.com/singularity-data/risingwave/pull/2315 is reverted in this PR, because we now only maintains min_pinned_version for each node, rather than the accurate pinned versions. As a result we cannot identify SSTs qualified for deletion in versions whose id > min_pinned_version_id any more.

@lmatz FYI the optimization from https://github.com/singularity-data/risingwave/pull/2315 is reverted in this PR, because we now only maintains min_pinned_version for each node (since https://github.com/singularity-data/risingwave/pull/3850), rather than the accurate pinned versions. As a result we cannot identify SSTs qualified for deletion in versions whose id > min_pinned_version_id any more.

Thanks!Good to know👍


"Dismissed" event is triggered for the side panel when closing ui.notification bar.

Wave SDK Version, OS
Wave SDK: Nightly (76a76fb942757119d31b3e5b11a5945b55206e6b) , OS: MacOs
(Testing Fix of https://github.com/h2oai/wave/pull/1813)
Actual behavior
Dismissed event is triggered for the side panel when closing ui.notification bar.
https://user-images.githubusercontent.com/12801761/220541091-735be2e9-e5a2-4361-8837-6b95b03dee12.mov
This is visible in the evidence video of the fix. https://github.com/h2oai/wave/pull/1813#issue-1558216706
Expected behavior
Its behaviour should be to close the ui.notification bar only and trigger Dismissed event for the notification bar instead of the side panel.
Steps To Reproduce
You can run the following code to create the issue:
@app('/')
async def serve(q: Q):
    if not q.client.initialized:
        # Create an empty meta_card to hold the notification bar
        q.page['meta'] = ui.meta_card(box='')
        # Display a button to show the notification bar
        q.page['form'] = ui.form_card(box='1 1 2 4', items=[
            ui.button(name="show_side_panel", label="Show side panel")
        ])
        q.client.initialized = True

    # Was the show_notification_bar button clicked?
    if q.args.show_notification_bar:
        # Create a notification bar
        q.page['meta'].notification_bar=ui.notification_bar(
            text='You can close me!',
            name="my_bar",
            # Get notified when the notification bar is dismissed.
            events=['dismissed'],
        )
    if q.args.show_side_panel:
        q.page["meta"].side_panel = ui.side_panel(
        title="",
        items=[ui.text("Side panel text"), ui.button(name='show_notification_bar', label='Show notification bar')],
        name="side_panel",
        events=["dismissed"],
        closable=True,
    )

    # Did we get events from the notification bar?
    if q.events.my_bar:
        # Was the notification bar dismissed?
        print("events triggered")
        if q.events.my_bar.dismissed:
            # Delete the notification bar
            print("dismissed")
            q.page['meta'].notification_bar = None

    if q.events.side_panel:
        if q.events.side_panel.dismissed:
            print("side panel dismissed")
            q.page["meta"].side_panel = None

    await q.page.save()

Thanks for reporting @senalw!

Its behaviour should be to close the ui.notification bar only and trigger Dismissed event for the notification bar instead of the side panel.

While I agree this is a bug, the scenario (having notification_bar and side panel rendered at the same time) shows a poor UI design. In general, there should always be at most 1 overlay component at a time.
Will be fixed as part of #1833


summary of v20 account forbidden

If i try to access the summary of my v20 account, it fails with HTTP error 403.
$ curl -v  -H "Content-Type: application/json"  -H "Authorization: Bearer "  "https://api-fxpractice.oanda.com/v3/accounts"

Trying <IP_ADDRESS>...
Connected to api-fxpractice.oanda.com (<IP_ADDRESS>) port 443 (#0)
TLS 1.2 connection using TLS_DHE_RSA_WITH_AES_256_GCM_SHA384
Server certificate: api-fxpractice.oanda.com
Server certificate: thawte DV SSL CA - G2
Server certificate: thawte Primary Root CA

GET /v3/accounts HTTP/1.1
Host: api-fxpractice.oanda.com
User-Agent: curl/7.49.1
Accept: /
Content-Type: application/json
Authorization: Bearer

< HTTP/1.1 200 OK
< Access-Control-Allow-Headers: Authorization, Content-Type, Accept-Datetime-Format, OANDA-Agent
< Access-Control-Allow-Methods: PUT, PATCH, POST, GET, OPTIONS, DELETE
< Access-Control-Allow-Origin: *
< Connection: keep-alive
< Content-Length: 52
< Content-Type: application/json; charset=UTF-8
<
Connection #0 to host api-fxpractice.oanda.com left intact
{"accounts":[{"id":"xxx","tags":[]}]}

$ curl -v  -H "Content-Type: application/json"  -H "Authorization: Bearer "  "https://api-fxpractice.oanda.com/v3/accounts/xxx/summary"

Trying <IP_ADDRESS>...
Connected to api-fxpractice.oanda.com (<IP_ADDRESS>) port 443 (#0)
TLS 1.2 connection using TLS_DHE_RSA_WITH_AES_256_GCM_SHA384
Server certificate: api-fxpractice.oanda.com
Server certificate: thawte DV SSL CA - G2
Server certificate: thawte Primary Root CA

GET /v3/accounts/101-001-129737-001/summary HTTP/1.1
Host: api-fxpractice.oanda.com
User-Agent: curl/7.49.1
Accept: /
Content-Type: application/json
Authorization: Bearer

< HTTP/1.1 403 Forbidden
< Access-Control-Allow-Headers: Authorization, Content-Type, Accept-Datetime-Format, OANDA-Agent
< Access-Control-Allow-Methods: PUT, PATCH, POST, GET, OPTIONS, DELETE
< Access-Control-Allow-Origin: *
< Content-Length: 0
<
Connection #0 to host api-fxpractice.oanda.com left intact
$

You received a 403 because the token being used in this v20 REST call is from the legacy REST API.
You needs to revoke this token and then generate a new one, and this will work with both the legacy and v20 REST calls.

ah. this is very confusing, as they state that tokens work for both the legacy api and v20. thanks


fix/openrpc-consistent-naming

see https://github.com/open-rpc/spec/issues/72

lol this commit message doesn't pass linting


[Question] WorldItem multi type not specified

spheremultis.scp
[WORLDITEM m_stone_and_plaster_house]
SERIAL=04001d6f9
LINK=04001d689
ATTR=010
MORE1=01e134
P=4022,521
REGION.FLAGS=0190
OWNER=01e134
ADDCOMP=04001d6f8
ADDCOMP=04001d689
LOCKDOWNSPERCENT=50
BASEVENDORS=10
BASESTORAGE=489

Shouldn't the type of multi be specified here? TYPE=T_MULTI , type=t_multi_custom or type=t_ship

Afair, the type info is saved only if it's different from the base item type, as for other properties.


Remove obsolete Swagger Validation Report comments

Now that TypeSpec folks are onboarding onto reviewing PRs, some of the cruft we have is causing trouble. E.g. @allenjzhang pointed out the Swagger Validation Report doesn't have TypeSpec Validation. I shall remove this report altogether, it is just tightly coupled in the codebase:
E.g.

Related:

https://github.com/Azure/azure-sdk-tools/issues/8612

Pull Request 564390: Remove "Swagger Validation Report" to reduce pressure on GitHub API rate limits


Update to privacy page

Some language fixes, updated change date, added link to Github

Fixed in #551 since this PR was behind after the switch to nx


cmd/go: "executable file not found" error if -out file exists

What version of Go are you using (go version)?
go version devel +67295d6 Thu Dec 14 22:12:09 2017 +0000 linux/amd64
Does this issue reproduce with the latest release?
It's on tip
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ernado/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build292560017=/tmp/go-build -gno-record-gcc-switches"

What did you do?
$ go test -cpuprofile cpu.out . # first
$ go test -cpuprofile cpu.out . # second (cached?)

What did you expect to see?
$ go test -cpuprofile cpu.out .
ok  	github.com/ernado/bug-donotuse	0.203s
$ go test -cpuprofile cpu.out .
ok  	github.com/ernado/bug-donotuse	0.203s

What did you see instead?
$ go test -cpuprofile cpu.out .
ok  	github.com/ernado/bug-donotuse	0.203s
$ go test -cpuprofile cpu.out .
exec: "DO NOT USE - link": executable file not found in $PATH
FAIL	github.com/ernado/bug-donotuse	0.000s

The package file I have used to reproduce that issue:
# bug_test.go
package bug

import "testing"

func TestNothing(t *testing.T) {
	if true == false {
		t.Fatal("-")
	}
}

upd: works even when "-o" is passed and the output file already exists.

I've bisected to 29be20a111d87b41b91e79a59fd3df95062ce91a
/cc @rsc @ianlancetaylor @bradfitz
https://go-review.googlesource.com/c/go/+/81895

/cc @dominikh

Is this the same as #23150?

@ianlancetaylor yes, it is the same issue.

Closing as dup.

Technically 23147 can't be dup of 23150 because first was created earlier.
But I see that cl84376 is already linked to 23150 so I got it :)


Implement Click as argparser

Going to be adding a few args; might as well do it right the first time.

Click didn't click, but argparse did. New CLI done.


Switch hillshading layer to HTTPS

The subdomains a.tile. etc. are not covered by the SSL certificate, but using tile.wmflabs.org seems to work correctly.

Thanks for the hint!


Add loading state to hotspots tab

Since the hotspot tab is the first tab when you load the app, it would be good to have a loading state with an activity indicator. that way we don't flash the "no hotspots" state while we're fetching your hotspots

This actually only seems to be an issue in development, since we hold the splash screen until we've completed the initial fetch


HPPCRT-49: Heaps wrongly use Comparable/Comparator for contains(), removeAll(), equals().

One of my user remarked that it was either wrong to use the Comparable or Comparator for methods involving equality tests in heaps, such as:

contains(),
removeAll(),
```equals()``

because indeed object equality is a different concept as object ordering, so that compareTo(obj) == 0 cannot be used in place of an object equals() when testing equality.
Consequently make so that the above methods for heaps use equals() instead of the comparison criteria as before.
Besides, that allows

contains(),
removeAll()

to be consistent with the parent KTypeContainer behaviour.

Done in https://github.com/vsonnier/hppcrt/commit/9552bcf33e31c049d2cd89be812cc77e1d58ce91


support pan & zoom control

please consider support adding
controller.zoom(Double scale)
and
controller.pan(Offset position)
so we can reset to original position & zoom easily

Please refer to #41


Is the state gradient not implemented yet for the CUDA kernel? (hence bptt_truncated_learning still forced to be True?)

Thanks for sharing this great project!
        # Add warning that bptt_truncated_learning is forced to be true
        # due to incomplete implementation of CUDA kernel for bptt_learning
        #
        # @TODO : remove this warning once the CUDA kernel, with state gradient, is implemented
        if self.bptt_truncated_learning == False:
            print("====================================================================")
            print("[WARNING]: bptt_truncated_learning is set as true (was configured as false), due to incomplete implementation of CUDA kernel for bptt_learning")
            print("====================================================================")
            self.bptt_truncated_learning = True

https://github.com/RWKV/RWKV-infctx-trainer/blob/70d02c4997578a027d110e3acb03a523d3986448/RWKV-v6/src/model.py#L291C1-L300C1
Just to confirm, when doing tbptt, this is essentially similar to the gradient estimator used in TransformerXL right?

use https://github.com/JL-er/RWKV-PEFT


"example" project is not helpful for beginners

There is an example (https://github.com/tarantool/grafana-dashboard/tree/master/example) in our repo. It's not trivial to understand how does it works for person who first time run Grafana and Prometheus.
First of all, "project" application doesn't expose any metrics.
User need manually add to init.lua:
local metrics = require('cartridge.roles.metrics')
metrics.set_export({
    {
        path = '/metrics/prometheus',
        format = 'prometheus',
    }
})

Secondly it's not obvious how to install and run Prometheus and Graphana.
I think it's a lack of simple readme.md file that says:
brew install prometheus
brew install graphana
brew services start grafana # grafana will be available on :3000
prometheus --config.file="prometheus/prometheus.yml" # prometheus will be available on :9090
# Here we need to verify "Status" -> "Targets" page.

Also it's better to change target urls from "example_project:" to "localhost:" - because basically user will run application using cartridge start that starts applications on the localhost.
Please add replicasets.yml file to test project. It will help to avoid some excess actions from user - all we need to call cartridge replicasets setup.
Finally, it would be great to add a small description about steps how to import dashboard from grafana.com. It's easier if user understand that "job" option means. It wasn't so in my case and I spent some time to understand what's wrong.

Thank you for your comments!
The example cluster was meant to be run with docker. There is a README section with short description https://github.com/tarantool/grafana-dashboard#experimental-cluster . If you use docker-compose up, it will start up containers and a network which will make up two configured and fully functional monitoring stacks connected to a Tarantool example app. Tarantool container is a container which runs luatest that starts up a cluster, configure it and send some requests to make up some load for graphs (https://github.com/tarantool/grafana-dashboard/tree/master/example/project/cluster). Metrics output is configured with clusterwide configuration with luatest, and that is the reason why we don't have metrics.set_export code for now. The other reason is that this cluster was introduced before set_export and there wasn't any obvious reasons why we should upgrade. The cluster runs with luatest instead of cartridge-cli because it was built before introduction of replicasets.yml and it was not convenient to works with cluster as I wanted to (for example, set up roles and replicasets automatically).
So if you want to use this example cluster to test only Grafana-related functionality, you can simply run docker-compose up and go to localhost:3000 to set up some boards. You do not need to set up Prometheus or Tarantool metrics output. If you want to change example app code, you can change example app code in folders and, if needed, changed enabled roles or cluster topology here if you're familiar with luatest https://github.com/tarantool/grafana-dashboard/tree/master/example/project/cluster (you may need to run docker-compose build --no-cache to reload code if you're have started a cluster before). If you want to monitor some app that already runs and have exposed metrics on your localhost, you may also use docker containers (Prometheus + Grafana) but configure your instances host:port in Prometheus config.
The reason that configuration uses example_project: is also so we could run it with docker-compose (which build a network for containers) and it will set up without any manual configuration.
It was planned to migrate to using cartridge-cli instead of luatest (#34) someday to set up example cluster. Maybe it is the time. But I still plan to run in inside docker container for now. The main reason for example cluster is to create an environment in which it is convenient to develop a dashboard. You can call a single docker-compose up and then go to localhost:3000 to check up your new panel. So we won't change target urls until something changes.
Finally, we have a documentation page on how to import a dashboard and set up parameters https://www.tarantool.io/en/doc/latest/book/monitoring/grafana_dashboard/ . I have a link in README, but it seems like it's easy to miss. And looks like it is still not obvious how to find a job from here.
I hope that I can use this issue to build up such a README so its crucial points will be hard to miss.


Update StringFormatFmt to StringFormat

Changes Proposed:

Updated deprecated StringFormatFmt to StringFormat

Fixes compile errors:
'StringFormatFmt': is not a member of 'Acore'
'StringFormatFmt': identifier not found

Issues Addressed:

SOURCE:

Tests Performed:

Built and compiled with no errors (Windows 11)

How to Test the Changes:

Commit changes, build, and compile

First time sending a pull request, let me know if you need anything else.


per-TenantShard read throttling

Slack thread: https://neondb.slack.com/archives/C033RQ5SPDH/p1707499237721679
Problem
part of epic https://github.com/neondatabase/neon/issues/5899
Specifically, RFC https://github.com/neondatabase/neon/pull/5648
tl;dr: we currently serve GetPage@LSN requests as fast as possible.
This sets the wrong incentives and poses operational and economic problems.
Summary of changes

Add a throttling mechanism based on the leaky-bucket
Make throttle reconfigurable via tenant conf (=> durable & runtime-changeable)
Throttle is implemented at the top of Timeline::get
The throttle applies to a configurable set of TaskKinds;

To limit only Timeline::get requests from clients, specify only PageRequestHandler
This will include Timeline::get requests from basebackup, so, it might slow down basebackup.

The rationale for placing the throttle at the top of Timeline::get is, and for the TaskKind selector, is that it enables experimentation. E.g., we can also use it to rate limit compaction.
Example
curl -X PUT --data @override.json localhost:9898/v1/tenant/config
# override.json
{
    "tenant_id": "f76997305d46b20964d2a5636aa302c3",
    "timeline_get_rate_limit": {
            "fair": true,
            "initial": 0,
            "interval_millis": 1000,
            "interval_refill": 1000,
            "max": 1000,
            "task_kinds": [ "PageRequestHandler" ]
    }
}

Testing & Performance
I tested manually using pagebench that the throttling works as expected by pointing pagebench at a single tenant initialized with pgbench -i 100 .
Adding --num-clients while at rate limit,

doesn't cause throughput to drop, i.e., no severe scalability issues
TODO: measure CPU utilization

TODOs

[ ] metrics to quantify the degree of throttling for a given tenant (per-tenant metric, like getpage latency histo!)
[ ] figure out what the page_service.rs metrics should mean on a rate limited tenant

currently, they will observe the rate-limited performance; that means our 4 Golden Signals dashboard & SLO will be affected by the rate limit
ideas:

A: duplicate these metrics, one rate-limited version, one not-rate-limited version
B: move the rate limit upwards into page_service.rs

if we don't want the experimentation capability that we gain by placing at top of  Timeline::get

Future Work

[ ] Regression test using pagebench: assert that the rate limit is enforced by driving at  limit rate with one client, then adding clients increases latency ~linearly.

Non-Goals
As pointed out in the RFC, this is a mechanism to prevent noisy-neighbor problems in pageserver. It's not a product feature / something we'll sell. The mechanism will be used to put an artificial ceiling on how many operations a single TenantShard can serve.

Did manual benchmarking.
Notes in https://github.com/neondatabase/neon/pull/6706/commits/ab88016d007aff4b1bcefdb50184e4296078b48a
Flamegraphs mentioned there
At 10x throttle

Below throttle


Improve command for finding saved chats in the saved chats picker

Should allign more with telescopes buildin see here to make it possible for users without fd to use

closed wrong issue


Label Today and Tomorrow are not shown

I'm expecting to see "today" and "tomorrow" in the list instead of just Sunday, Monday,...

Fixed on develop.


Add a diagram of the how the components of the system interact with eachother

A full system diagram is needed to make it easier for new contributors to start developing.
This is long due.


Added incrementOnFilenameConflict Option

Added incrementOnFilenameConflict option to optionally avoid overwriting existing output files.
For example, if pandoc is about to output to "MyFile.docx" but that file already exists, then this option will increment the filename to "MyFile1.docx," or whatever the appropriate increment number is like "MyFile6436.docx".

IMO, it's better to stay consistent with pandoc's behavior and overwrite and instead pop up a modal asking the user if they want to overwrite. What do you think?

That's certainly an option, but it wouldn't work for my use case.
I'm outputting a document into a locally mounted google drive, then (using a macro) following the pandoc command with a command that gives me the gdrive share link for that document, so if I want to get feedback on an obsidian document I hit Cmd+Shift+S and it automatically gets formatted, uploaded, and I get a share link to give out.
The problem I was solving with this is that the original behavior silently overwrote whatever file had been there before, which includes whatever comments readers had made on the google doc. So for example if I shared one version one day and tried to share another version on a different day, the new version would silently blow away the old one, along with whatever edits and comments existed. No bueno.
So for me it's not a matter of "Overwrite? Yes/No," it's a matter of "The share command should never destroy data."
A different solution I considered, that may still be useful for others, is something like a Save As modal. Right now the filename can only be exactly whatever the note filename is (modulo extension), but a popup offering Overwrite or Save As or Cancel would be a natural addition to the tool. This using the OS's Save As dialog via something like remote.dialog.saveSaveDialog().
I guess the full feature would have:

a separate command that gave you as Save As dialog no matter what, so users could optionally save the file with whatever name, regardless of currently existing files
the existing command now checking for the file's existence and offering the Overwrite/Save As/Cancel dialog

this should be an option that defaults to false, in favor of the existing behavior of silently overwriting, to avoid breaking people's workflow.

Still, for my usecase where I never want the tool to be able to overwrite anything, and expect to generate multiple versions over time, and I want it to "Just Work" without any additional user interaction, the increment is better than Save As.

Your patch is totally understandable given your use case but given that we want to migrate to this fork being the "official" one, I'm a bit hesitant to merge something like this (first time maintainer here). Perhaps use your own releases via BRAT for now? argenos is fairly busy but I'd probably want her opinion on this as well.
@argenos, if you wouldn't mind chiming in.

I think there are three different options when the output file already exists, And there are arguments / use cases for all three:

overwrite without confirmation (pandoc's default behavior)
confirmation-to-override modal (what @AB1908 suggested)
increment instead of overwrite (@PeteMichaud's PR)

So I think having a setting where the user can select one of the three would make most sense?

just so it doesn't get lost: here is a guide on how to use modals in Obsidian plugins

Ok, just pushed a crack at the feature as discussed. I have only tested on desktop, but I don't anticipate any issues, the code is pretty straightforward.

Ok, I may have uncovered an issue while helping Laura on the discord channel, but I can't reproduce it. I'd like either of you, @AB1908 or @chrisgrieser to try the following steps:

Don't use my fork, just use the current community release of the pandoc plugin
Remove any custom pandoc output path from your settings, just leave it blank.
Try to generate any file via pandoc that is not from the vault root directory.  It probably isn't important, but it happened to be the case that the generated file in Laura's case was a pdf, so maybe try that.
Now use my fork and do step 2 again.

That's it. The correct behavior is for the new file to end up in the folder you generated it from, so like generating from Vault/MyFolder/MyFile.md should produce Vault/MyFolder/MyFile.pdf
The behavior that Laura found while using my fork is that MyFile.pdf ended up in the vault root folder.
I'd like to know whether this bug is new to my fork or if it exists in the current version. I'll fix it either way, but I want to address it separately if it's separate!


Add TFRecordWriter

Please add support for writing TFRecords. I believe the corresponding python class is tf.io.TFRecordWriter.

These are notes for whoever tries to implement this.
The Python class calls through to the internal C++ API which doesn't really do much other than add the appropriate header & footer bytes to an open file handle (https://github.com/tensorflow/tensorflow/blob/5dcfc51118817f27fad5246812d83e5dccdc5f72/tensorflow/core/lib/io/record_writer.h), and those header & footer bytes are basically the length, a crc of the length, the data as a byte array and then a crc of the data (https://www.tensorflow.org/tutorials/load_data/tfrecord#tfrecords_format_details). The data is the byte array from a TF Example protobuf which we already have in TF Java. As usual the C++ API class isn't exposed in the C API, but at least in this case we can use Java's built in CRC32 and then mask it using the formula from the docs, and then we should be able to write it out fairly easily. They encode the length using this code (https://github.com/tensorflow/tensorflow/blob/5dcfc51118817f27fad5246812d83e5dccdc5f72/tensorflow/core/platform/coding.cc) which we should also be able to replicate pretty easily.

Or, it's also possible to access the C++ API using JavaCPP:
https://github.com/bytedeco/javacpp-presets/blob/master/tensorflow/src/gen/java/org/bytedeco/tensorflow/RecordWriter.java

See also https://github.com/tensorflow/ecosystem/blob/master/hadoop/src/main/java/org/tensorflow/hadoop/util/TFRecordWriter.java

Ah I'd missed that it was CRC32 but using a different polynomial than the one in Java. Bit tedious to have to depend on Apache Commons Codec to get access to that version of CRC, but at least it doesn't bring anything else in with it. We could probably bring that class across as it's part of TF already and fix up the entry points so they are a little nicer (i.e. accept an Example protobuf rather than a byte array). @karllessard any objections to adding Apache Commons Codec as a dependency of tensorflow-framework?

FWIW, it would be less maintenance to use the C++ API, since that's what Python uses anyway, the code is already there, and it guarantees compatibility, which is the goal here.

I'm reluctant to depend on the internal TF C++ API as we don't have a team of people to track it when it changes. The C++ API changes at the whims of the Python library, it's not depended on by other projects or languages, so we've got little scope for influencing it. If they stabilise it as part of the modular TF or TF-core initiatives then it's much more palatable.
In the case of RecordWriter I wouldn't expect it to change too much, but Apache Commons Codec isn't going to change either, and the overall code size changes for including Apache Commons Codec are probably a lot smaller given how big the preset is you linked to.

We don't need to wrap everything, it's probably going to work just by including only that one small header file.

Should we also have TFRecordReader?

Reader and Writer both will be useful!

In any case, if someone wants to use what's available in TF Core just like the Python API is doing, let me know and I'll be happy to add and maintain the necessary couple of header files for JavaCPP!

Wh can't we copy from  https://github.com/tensorflow/ecosystem/blob/master/hadoop/src/main/java/org/tensorflow/hadoop/util
TFRecordReader.java
TFRecordWriter.java
Crc32C.java

Crc32C uses org.apache.commons.codec.digest.PureJavaCrc32C


Order of Statements Causes Foreign Key Integrity Error

I have several relations in my DB, which when exported using cloudflare-d1-backups lead to "FOREIGN KEY constraint failed" and "Table not found" errors when imported into another DB. My current fix is to manually change the order in the export such that tables which rely on foreign keys are created last.

Can you provide a SQL example?

Also ran into this @nora-soderlund. Here's an example, of my schema where the tables are not getting created in the right order.
CREATE TABLE IF NOT EXISTS  migrations(
		id         INTEGER PRIMARY KEY AUTOINCREMENT,
		name       TEXT UNIQUE,
		applied_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP NOT NULL
);
INSERT INTO "migrations" ("id", "name", "applied_at") VALUES (1, '0000_user_table.sql', '2023-10-25 22:20:57');
INSERT INTO "migrations" ("id", "name", "applied_at") VALUES (11, '0001_seasons.sql', '2023-12-02 21:47:44');
INSERT INTO "migrations" ("id", "name", "applied_at") VALUES (12, '0002_tournaments.sql', '2023-12-02 21:47:45');
INSERT INTO "migrations" ("id", "name", "applied_at") VALUES (13, '0003_results.sql', '2023-12-02 21:47:45');
CREATE TABLE IF NOT EXISTS  results
(
    `tournament_id`             INTEGER NOT NULL,
    `user_id`                   INTEGER NOT NULL,
    `runner_deck_identity_id`   INTEGER NOT NULL,
    `runner_deck_faction`       TEXT    NOT NULL,
    `runner_deck_url`           TEXT,
    `corp_deck_identity_id`     INTEGER NOT NULL,
    `corp_deck_faction`         TEXT    NOT NULL,
    `corp_deck_url`             TEXT,
    'rank_swiss'                INTEGER NOT NULL,
    'rank_cut'                  INTEGER,
    `runner_deck_identity_name` TEXT,
    `corp_deck_identity_name`   TEXT,
    `points_earned`             INTEGER NOT NULL DEFAULT 0,

    FOREIGN KEY (user_id) REFERENCES users (id),
    FOREIGN KEY (tournament_id) REFERENCES tournaments (id),
    PRIMARY KEY (tournament_id, user_id)
);
INSERT INTO "results" ("tournament_id", "user_id", "runner_deck_identity_id", "runner_deck_faction", "runner_deck_url", "corp_deck_identity_id", "corp_deck_faction", "corp_deck_url", "rank_swiss", "rank_cut", "runner_deck_identity_name", "corp_deck_identity_name", "points_earned") VALUES (3811, 23764, 26066, 'anarch', 'https://netrunnerdb.com/en/decklist/78248', 30043, 'jinteki', 'https://netrunnerdb.com/en/decklist/78249', 1, 7, 'Hoshiko Shiro: Untold Protagonist', 'Jinteki: Restoring Humanity', 2.32293714379668443e+02);
CREATE TABLE IF NOT EXISTS  seasons
(
    `id`         INTEGER PRIMARY KEY NOT NULL,
    `name`       TEXT                NOT NULL,
    `started_at` TEXT                NOT NULL,
    `ended_at`   TEXT
);
INSERT INTO "seasons" ("id", "name", "started_at", "ended_at") VALUES (0, 'Hello World', '2023-01-01T00:00:00Z', NULL);
DELETE FROM sqlite_sequence;
INSERT INTO "sqlite_sequence" ("name", "seq") VALUES ('migrations', 13);
CREATE TABLE IF NOT EXISTS  tournaments
(
    `id`            INTEGER PRIMARY KEY NOT NULL,
    `name`          TEXT                NOT NULL,
    `location`      TEXT                NOT NULL,
    `concluded`     INTEGER             NOT NULL,
    `format`        TEXT                NOT NULL,
    `type`          TEXT                NOT NULL,
    `players_count` INTEGER             NOT NULL,
    `date`          TEXT,
    `season_id`     INTEGER,

    FOREIGN KEY (season_id) REFERENCES seasons (id)
);
INSERT INTO "tournaments" ("id", "name", "location", "concluded", "format", "type", "players_count", "date", "season_id") VALUES (1, '2016 Hungarian Nationals', 'Hungary, Budapest', 1, 'standard', 'national championship', 20, '2016-09-17T00:00:00.000Z', NULL);
CREATE TABLE IF NOT EXISTS  users (
	`id` integer PRIMARY KEY NOT NULL,
	`name` text,
	`email` text,
	`password` text
, `is_admin` integer NOT NULL DEFAULT 0);
INSERT INTO "users" ("id", "name", "email", "password", "is_admin") VALUES (76, '3Jane', NULL, NULL, 0);

Thanks, I'll have a look tonight.

Thank you. I have published 1.1.1 which fixes this.
An original schema with:
CREATE TABLE movies
(
    id INTEGER PRIMARY KEY NOT NULL
);
INSERT INTO movies (id) VALUES (123);

CREATE TABLE movie_tags
(
    movie_id INTEGER NOT NULL,
    tag TEXT NOT NULL,
    FOREIGN KEY (movie_id) REFERENCES movies (id)
);
INSERT INTO movie_tags (movie_id, TAG) VALUES (123, 'Action');

Results now in:
CREATE TABLE IF NOT EXISTS  movies
(
    id INTEGER PRIMARY KEY NOT NULL
);
INSERT INTO "movies" ("id") VALUES (123);
CREATE TABLE IF NOT EXISTS  movie_tags
(
    movie_id INTEGER NOT NULL,
    tag TEXT NOT NULL,
    FOREIGN KEY (movie_id) REFERENCES movies (id)
);
INSERT INTO "movie_tags" ("movie_id", "tag") VALUES (123, 'Action');


[Jetchat] Fix UI covered by navigation bars

Happens when the device is in landscape.
Screenshots

this added an extra padding to the portrait screenshot

🤔 what padding? The left/right values will be 0 in portrait, so the change will no-op.

Definitely no extra padding with this PR in portrait. I think you might be seeing the padding on the GitHub comment? If you click the image, it will be shown larger.


support newer versions of python

add support for python 3.12, 3.13
drop support for older versions of flask

Thanks, Will!


Universal C Runtime

Hello, very nice article. I was wondering if you can clarify the Universal C Runtime requirements for Windows Server 2016. The article says that Universal C Runtime should be installed on all machines. However, going to the download page for this run time, it does not have it for Windows Server 2016. I believe Windows Server 2016 and greater already includes this runtime. If so, I would add a statement on this page to avoid confusion. Thank you!

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

ID: 23e000f0-84ce-7ec7-e5ed-4bd9c49f5d49
Version Independent ID: e636b9d1-be81-0f00-97c4-ff4fa9078be3
Content: Deploy Azure AD password protection - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-password-ban-bad-on-premises-deploy.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@josuvil Thanks for your feedback! We will investigate and update as appropriate.

Per your feedback, Win10 and Windows Server 2016 do both in fact come with Universal CRT installed. Currently the author is looking into this and will update the docs with a note or something similar accordingly.

@jay98014 Thoughts on this?

Closing this issue as issue is being addressed. Thanks again @josuvil !

Glad to assist!


Adjustment to position of CannonJS HeightmapImpostor

I hope this works!  :)

Hi guys,
sorry for a late reply as always :smile:
Can you explain what was wrong with the old implementation and why you think this is the right way to go?
Cannon has z as the up vector, which is not needed until you get to very specific object types like the one we are handling here. Any playground you want to show me?

Hi R, thx for asking.  I guess it was #8 in the series that was first seen as an issue.  Camera aiming +x (I changed to +z aimed cam soon after).
Notice that physics impostor only active in southwest quadrant of terrain.  Notice impostor extends southwest BEYOND edges of terrain, too.  It is a 500x500 terrain, and I believe the impostor need repositioning 250 units north (+x) and 250 units east (+z)  (or +y if cannon heightmap has z vertical)
Thank god you're here!  I can use all the help I can get.  But don't let this stand in the way of real life stuff, R.  I think boundingBox.extendSize changed from returning DIAMETER-like values... to returning radius-like values (half of what they once were).  But I have no proof of that, and nobody is saying I am correct.  So, likely I'm wrong.  :)  Thanks for advice/input, R.  Sorry if I'm causing you hassles, or lost on wild goose chase.

DOA


save iD editor tags to a new database

Hello everyone, I am editing iD editor so i can put some more tags on it. i would like to have the my new tags saved in my own database and the tags from OSM saved in OSM database.
After my idea was to display all tags in one webpage using openlayers. Can someone tell me how can i do this?
Thank you for all the help

Sounds like a big project!   There's no easy way to do what you're trying to do.. You're just going to have to do some programming to keep the OSM and custom tags separate.
I'd suggest:

Make sure all of your custom tags have some kind of namespace like NOT_OSM:name.
Edit the DiscardTags action so that any tag like this is discarded before saving to OSM.
This block of code in save.js is where iD decides whether there is anything worth saving.  I'm not sure how you will split this up to save some tags into OSM and some tags into another database.  Maybe keep history.changes as OSM's changes, and make a new function to get the not-osm-changes?

What if instead of uploading all the tags i only upload mine .
I mean in the "new" iD editor only have the new tags that i need and for the other tags i would simply redirect the user to osm id Editor?
then when i was going to display all the tags i just have to search in my database and in osm ? what do you think?

I mean in the "new" iD editor only have the new tags that i need and for the other tags i would simply redirect the user to osm id Editor?
then when i was going to display all the tags i just have to search in my database and in osm ? what do you think?

Yes you could do that too.  It might be easier to make a separate iD editor with separate tags, rather than trying to make one iD editor that does everything and keeps tags separate.

how can i do to to send the new tags to my own database?


Update Testframework to handle username password for non-multi factor auth tenants

Enable login to non-multifactor auth tenant using username and password.
Adding tests

A minor comment, then LGTM


Remove informed-consent and test-descriptions from the ooni-spec repo

Their new canonical home is the ooni-web repository inside of the
following directories:
* ooni-web/content/nettest/
* ooni-web/content/about/

Looks good merging.


Bulk actions break simple list

Steps to reproduce:
Got to the simple example, display the list of comments, resize to mobile, see the console.

Environment

Admin-on-rest version: next
Last version that did not exhibit the issue (if applicable): before bulk actions
Browser: chrome
Stack trace (in case of a JS error):

React does not recognize the `hasBulkActions` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `hasbulkactions` instead. If you accidentally passed it from a parent component, remove it from the DOM element.
    in ul (created by List)
    in List (created by WithStyles(List))
    in WithStyles(List) (created by SimpleList)
    in SimpleList (created by WithStyles(SimpleList))

Fixed by #1536


Improve comments appearance

A couple of minor fixes and tweaks, and a bigger one: Render markdown in comments as we do for the desciption.
The last change means that only the header line of new comments is printed in bold, but IMHO this is enough for them to stand out.
And the readability improvement is quite significant:

I had originally implemented only bolding the new comment string like you did but found that it was useful to have the whole message bolded with MRs with lots of comments.
I'm on the fence on this.  I'll leave it @bmeneguele or @zaquestion to make a decision on this one.

One thought: It might be nice to have an MR format test of some kind.  Some of your commits are for indentation and readability, and it might be good to have a test that checks for that the output format is correct.

I had originally implemented only bolding the new comment string like you did but found that it was useful to have the whole message bolded with MRs with lots of comments.
I'm on the fence on this. I'll leave it @bmeneguele or @zaquestion to make a decision on this one.

IMHO it's better to keep to whole comment bold, mostly due to new comments in old discussions that are shown before newer comments in that MR. But the other changes you did indeed look pretty nice. Is it possible to keep both? :)

But the other changes you did indeed look pretty nice. Is it possible to keep both? :)

My first reaction was to say no, because printit() is still set up to use bold style, but the rendered markdown overrides it.
But after taking a closer look, I think it'll be possible to come up with something.

One thought: It might be nice to have an MR format test of some kind. Some of your commits are for indentation and readability, and it might be good to have a test that checks for that the output format is correct.

Good point, I'll amend the tests to check for the indent.


Add docs about library name

cc @osanseviero @philschmid @julien-c
There is no library_name mentioned in https://huggingface.co/docs/hub/models-cards
The library name is added to the tags of the model on the hub. It's determined as such:

The user can force library_name in the model card
If one of the tags in the model card already is a library name, then do nothing
We try to autodetect based on the presence of files for diffusers, keras, sklearn and nemo
Lastly, if nothing was detected and there is a valid config.json file, the library name is set to "transformers"

Totally! I can take this one :)


Route Group Nested Properties

Hello, thanks for this great framework!
I just want to know, is there a way to have a nested group having the same properties as parents?
I have this code:
Route.group('version1', function () {
  Route.group('publicApiV1', function () {
    Route.resource('places', 'V1/PlaceController')
        .only(['index', 'show'])

    Route.resource('articles', 'V1/ArticleController')
        .only(['index', 'show'])
  })

  Route.group('adminApiV1', function () {
    Route.post('places', 'V1/PlaceController.store')
    Route.put('places/:id', 'V1/PlaceController.update')
    Route.delete('places/:id', 'V1/PlaceController.destroy')

    Route.post('articles', 'V1/ArticleController.store')
    Route.put('articles/:id', 'V1/ArticleController.update')
    Route.delete('articles/:id', 'V1/ArticleController.destroy')
  })
  .middleware('auth')
})
.prefix('api/v1')
.formats(['json'], true)

What I expect from this will generate these routes:
a. GET places with prefix of api/v1 and formats of json
b. the same for GET articles
c. POST places to have prefix api/v1 and formats of json also middleware of auth
But what I really have is this:
a. GET places is written to /places not /api/v1/places
b. the same for GET articles
c. POST places doesn't have auth middleware
For now it will work if I don't write nested groups, but it will be very useful to have this work so routing will be simpler.
Thanks, waiting for your reply!
Regards,
Bagaskara Wisnu Gunawan.

Possible duplicate of https://github.com/adonisjs/adonis-framework/pull/469 and https://github.com/adonisjs/adonis-framework/issues/331
Nested groups are not supported intentionally, the idea is to keep your routes readable

okay then @thetutlage, I got your point, thanks for answering and sorry for late reply, thanks!


Split Component doesn't work for me

The instructions in this link do not work for me.
Here are the result of those codes:

may i ask you to retest with 0.17.0?

It is working with v. 0.17.0, thanks!

@MSzturc I am having this issue, on 0.17.8.  Does a setting need to be enabled to use the <split> tag?


Packet not sent until other packet received

On Android my UDP broadcast packets aren’t getting sent until the device receives some other UDP packet.  If nothing on the network is sending packets then no packets ever leave my device. (Confirmed with wireshark).  Has anybody else seen this kind of behavior?

maybe this is related to my issue: #82?

sorry guys, no time to help at the moment, but if you figure it out, PRs are welcome

This problem exists in my application too.
I'm using this library in a real-time application, and I have an interval that send and receive packet every 1000 msec.
If packets can't receive, the application stop sending packets.
and after a few minutes the app crashes and I see this error in log tab:
java.util.concurrent.RejectedExecutionException: Task android.os.AsyncTask$3@1b53ea0d rejected from java.util.concurrent.ThreadPoolExecutor@718cdc2[Running, pool size = 9, active threads = 9, queued tasks = 128, completed tasks = 934]


chore: move e2e tests to language tools repo

closes #477

Is it already documented where "which kind of" tests live?
If no, let's do that in this PR.

The e2e tests before publishing and those after publishing are the same, which live in the packages/vscode/src/test folder with the test schemas living in packages/vscode/testFixture.

Thanks, please add that do some readme to persist it.

Note: after merging this, the repo https://github.com/prisma/vscode-e2e-tests can be deleted.


DatetimeIndex accessors and reorganize the datetime accessors for Series

[x] Closes #193
[x] Tests added

test_timefuncs.py:test_datetimeindex_accessors

Reorganized the accessors to correspond to the pattern used in defining them for DatetimeArray.  Used a lot of Generic types and TypeVar to get all the return types right that are different between Series.dt.xxx and DatetimeIndex.xxx for various accessors xxx.
Removed deprecated methods and associated tests.

wow, we have to do all of this because of inherit_names? I would like to remove inherit_names but it is moving slowly since it also requires implementation changes https://github.com/pandas-dev/pandas/pull/48015

wow, we have to do all of this because of inherit_names? I would like to remove inherit_names but it is moving slowly since it also requires implementation changes pandas-dev/pandas#48015

It's really because the way that the properties are hooked in to Series.dt and into DatetimeIndex is via the properties on DatetimeArray , which is using inherit_names so that if you call Series.dt.month or DatetimeIndex.month (for example), it delegates that to DatetimeArray.month, but returns a different result (Series versus Index respectively).
The other option is that we just document all the properties for Series.dt and DatetimeIndex, but that then becomes different than the implementation.

Happy to keep your current approach - it works :) removing inherit_names will take quite a while.

Happy to keep your current approach - it works :) removing inherit_names will take quite a while.

Let me know if I should make any other changes.

Thanks @Dr-Irv hope it becomes easier to type in the future when pandas changes.


[BUG] - api-webapp is not fetching the docker image

Remember to label the issue with the appropriate area, and also assign it to yourself if you plan to work on the issue in the near future.
Describe the bug
the web app is failing to fetch the docker image.
This is due to a bug (probably) that fails fetching the image using managed identity
Steps to reproduce

deploy tre
check the deployment center tab (Azure)
verify the image was not pulled

correct


Bug I found

Challenge http://www.freecodecamp.com/challenges/waypoint-learn-how-script-tags-and-document-ready-work has an issue.
description:
first of all, in the end of the code it is written "<--" that came out of nowhere
second, if I have a typo (in the screenshot I wrote "documet" instead of "document"), all the missions' levels disappear, untill the typo is gone..

Sincerely,
Tzahi

@Tzahile thanks for posting this issue!
I can't find that "<--" neither in source code nor in production (I've tried in FF and Chrome Incognito modes).
But I can't modify any line of code below 20. As I remember this was mentioned somewhere and it could a bug in CodeMirror.

Please review the Guidelines for Contributing, thank you!.

well, I did "Inspect elements" (I'm using Chrome)
and there it is.
Anyway, it's a minor problem..
more important problem is about what I mentioned - the missions on the left side disappear if you do something like typo.
Another bug I noticed is that sometimes when starting a new mission, part of the code is (probably) invisible, and you need to click on the code-window or write something, and then the rest of the code reappears..

@Tzahile you typed documet but it should be document in your script tag.. This error is caused by you typo. Editor tries to execute the code you wrote but there is no documet object in jQuery so it gives you an error.


[CH-239]Support queries on struct fields

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Currently, ClickHouse backend has supported struct type, but we cannot run queries on specified struct fields. for example
seelct s.a from t

where s is a struct column.
In this pr, we make a change on NamedStruct.names based on type.proto's comments.
// A message for modeling name/type pairs.
//
// Useful for representing relation schemas.
//
// Notes:
//
// * The names field is in depth-first order.
//
// For example a schema such as:
//
// a: int64
// b: struct<c: float32, d: string>
//
// would have a `names` field that looks like:
//
// ["a", "b", "c", "d"]
//
// * Only struct fields are contained in this field's elements,
// * Map keys should be traversed first, then values when producing/consuming
message NamedStruct {
  // list of names in dfs order
  repeated string names = 1;
  Type.Struct struct = 2;
  PartitionColumns partition_columns = 3;
}

In previous implementation, the names only contains columns' top level names. In this pr, we visit named structs in depth-first order, and put all names in a flatten array. This may have different behaviors with previous implementation, and we should be careful with this.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
unit test
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
test by CH[[244]]

backend's change see: https://github.com/Kyligence/ClickHouse/pull/244

Not merge this pr, there are still some problems in the backend.


doc: Add necessary information in README.md

Currently , the README needs to be updated for a newcomer to contribute to the project.

@hamhaingaurav Please check the PR  #17 to fix this issue


Should be able to generate volumes of random data

As at 19/12/2018:

Priority: High - this is part of the MVP for the product, as a user i want to be able to create a database with indicative data with random values
Complexity: Medium - Random data can already be produced, data can already be produced to a limit (max-rows). It may already be possible to achieve this output. The complexity comes from the way data needs to be generated.

? Do you provide (random) data for all rules except the last and then keep producing random data for the last rule until you have the required number of rows emitted?
? Do you try and 'share' the number of rows across your rules and say for a profile with 5 rules and million required rows, emit 200,000 rows of random data per rule.
? How do you prevent being stuck on conditions in the first rule which are not satisfied as the values produced are random (dealing with contradictions)

Questions/Assumptions:

Assume this requirement doesn't preclude duplicate values.

? Is this a requirement for the MVP ?
Change to Medium-High


Test anti-emulation measures of Classic NES Series

There are a couple of nasty tricks there naive emulators expected to fail.
https://mgba.io/2014/12/28/classic-nes/

Some of them are rather trivial and don't require testing imo. Setting up the LDM / STM weirdness might be annoying. Memory mirroring, VRAM code execution and prefetch abuse could be done rather easily. It's hard to find motivation to write those tests as my emulator already does those things :)

I have added memory tests which check

memory mirrors
video memory byte stores
sram weirdness

and nes tests which check

vram code execution
two stage pipeline
stm decrease write order

Audio fifo irregularities are missing because I haven't tried emulating audio yet and save type masquerading can't be tested imo. I consider this issue solved.

Thank you!


An unsubscribed event may change store state

Code to reproduce the issue:
state changes even after the event was unsubscribed
Expected behavior:
If you do store.off(event) it should unsubscribe any handlers for the event
Actual behavior:
If you subscribed a store to the same event more than once using .on(event, handler),
calling .off(event) does not guarantee that the event won't affect store again.
If you call the event, the state of the store may be changed, because only the very last event handler (reducer) is unsubscribed. It wouldn't be an issue if a new reducer replaced the previous one for the same event (#8)
Versions of packages used:
Effector v0.18.4

Related #8


[SPARK-8353] [DOCS] Show anchor links when hovering over documentation headers

This patch uses AnchorJS to show deep anchor links when hovering over headers in the Spark documentation. For example:

This makes it easier for users to link to specific sections of the documentation.
I also removed some dead Javascript which isn't used in our current docs (it was introduced for the old AMPCamp training, but isn't used anymore).

/cc @pwendell, this might be nice to port back to the 1.4.x docs as well, since it's a big doc UX improvement.

Merged build triggered.

Merged build started.

Test build #34856 has started for   PR 6808 at commit f518b6a.

Test build #34856 has finished for   PR 6808 at commit f518b6a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

By the way, I've filed https://issues.apache.org/jira/browse/SPARK-8351 as an umbrella to track ideas for a few similar non-content documentation improvements, including improvements to other navigational aids.

The underline on hover looks a little strange, so I'm going to push a CSS fix to address that.

Merged build triggered.

Merged build started.

Test build #35153 has started for   PR 6808 at commit e59d8a7.

Test build #35153 has finished for   PR 6808 at commit e59d8a7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

I'm going to merge this into master and branch-1.4 so that we have the nice links for 1.4.1.  If we want to make more CSS tweaks, we can do it in a followup PR.


Lesson 210: improve Node.js app performance by usage of Fastify and uWebSockets.js

improve performance by usage of Fastify and uWebSockets.js (alternative node.js http and ws server which is more performant than node:http)

@pasha-vuiko Thank you for the PR. Could you please move it to the fastify-app folder and keep the original code in place for future reference? I'll merge it afterward.

@antonputra done


[SPARK-20631][PYTHON][ML] LogisticRegression._checkThresholdConsistency should use values not Params

What changes were proposed in this pull request?

Replace getParam calls with getOrDefault calls.
Fix exception message to avoid unintended TypeError.
Add unit tests

How was this patch tested?
New unit tests.

Test build #76548 has started for PR 17891 at commit 098e262.

Test build #76549 has started for PR 17891 at commit 098e262.

Test build #76548 has finished for PR 17891 at commit 098e262.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76548/
Test PASSed.

Test build #76549 has finished for PR 17891 at commit 098e262.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76549/
Test PASSed.

cc @jkbradley

LGTM, merged into master and branch-2.2. Thanks!

Thanks @yanboliang!

@jkbradley It shouldn't. It is not a correct test #18085


Prometheus Integration with Node-Problem-Dectector which IP to assign for Prometheus Exporter?

Hi Guys please throw some light here...I am trying to integrate Prometheus with Node-Problem-Dectector and want to know which IP to assign to Prometheus Exporter in values.yaml ?
Applying Prometheus Node Exporter IP gives error that IP Cannot be binded. May I know what IP to assign for Prometheus Exporter. Please Help Guys.

We can deploy the NPD with Prometehus Address as "<IP_ADDRESS>" but actual Prometheus Node Exporter address will not get binded? Please advise?

Guys any help would be appreciated!

@max-rocket-internet Please could you explain which Prometheus IP address to bind to NPD? I am using your Helm chart and it deploys the daemonset but doesn't work when I pass Prometheus Node Exporter as they both want to consume same 9100 port. Please help

Sorry but I don't know. PRs are welcome if/when you work out what you need 🙂

Closing due to inactivity. Feel free to reopen if you have any updates.


Update Zcash consensus branch ID

Describe the bug
ZCash went thought a network upgrade and consensus branch ID being changed: https://zips.z.cash/zip-0250
https://github.com/trustwallet/wallet-core/commit/586da8074c1c0825a84247779696c2139d3120ab#diff-a353960cd4b32e9bc718b45be79b50feR32 - This value needs to be updated, only for zcash

no need to change wallet core code, we can pass branch id from plan


Failover-Clustering vs. Storage Module

When used in Failover Cluster, cmdlets from the Storage module operate on cluster level (all servers in the cluster).

I think we need more details here - why does 'get-disk' not list some disks when Failover-Clustering is enabled?
How can we enable 'get-disk' or the storage module in general to work with disks again when Failover-Clustering is enabled?
If it's because a disk is available to a cluster, how do we make that disk NOT available to the cluster?
When I enable failover-clustering on my SQL server, I run get-disk and suddenly only 2 of my 8 total disks are showing.  The other 6 disks are showing in disk management.  I dont plan on making those shared cluster disks, how can I make the storage module start recognizing them as normal disks again?

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

ID: 3504bf98-0bd8-b3b5-082b-49f8609dd417
Version Independent ID: d417c085-9397-5847-79dd-8836f0b1115d
Content: Get-Disk (Storage)
Content Source: docset/winserver2022-ps/storage/Get-Disk.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

then after reboot:


Update version to 3.0.0 in doc and build/VERSION file

What type of PR is this:
/kind documentation
/area documentation
What does this PR do / why we need it:
See PR title. Needed to officialize the 3.0.0 release.
Which issue(s) this PR fixes:
Fixes #6011
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[x] Documentation

How to test changes / Special notes to the reviewer:

/override v4.11-integration-e2e

/override Kubernetes-Integration-Tests/Kubernetes-Integration-Tests
/override OpenShift-Integration-tests/OpenShift-Integration-tests
/override windows-integration-test/Windows-test

/override ci/prow/v4.11-integration-e2e

/approve
/lgtm

/override ci/prow/v4.11-integration-e2e


Translation not working on homepage

Translation on the homepage is not functioning as expected when the default locale is set to either Brazilian Portuguese or Spanish. It only works after logging in and manually refreshing the page.

Resolved in PR: Fix/Default-locale-language

This was fixed in v1.1.21


HHH-12866 : apply "for update of" with aliases in Oracle dialect

Similar to the issue HHH-5654, which had been fixed in PostgreSql81Dialect, when lock mode was specified with an alias, the generated SQL for Oracle database is not using the alias at all.
According to Oracle's documentation of for-update clause, the syntax allows user to specify "table.column" in order to lock the select rows only for a particular table or view in a join.
Tests have been added for Oracle dialects including 8i, 9i, 10g, and 12c. The fix is only in Oracle8iDialect as all the other dialects extends from this one.
Please refer to the description of JIRA ticket HHH-12866.

@gsmet Good point. I've added a test (in the latest commit) to verify the query can be executed by Oracle. Please have a look and let me know your thoughts. Thank you.

@jedichenbin I made a couple of minor adjustments in your commits (mostly formatting but also some minor tuning), squashed some of them and force pushed to your branch.
Could you get a fresh copy of your branch and rerun the test suite?

@gsmet Thank you so much for reviewing and updating the commits.
I thought the Travis build would run those tests against various databases, obviously I was wrong. Bad news - there are a few tests failed, including:

Gradle [documentation:test]

ExplicitLockingTest

Gradle [hibernate-core:test]

UpgradeSkipLockedTest (extends AbstractSkipLockedTest)
PessimisticWriteSkipLockedTest (extends AbstractSkipLockedTest)
PessimisticReadSkipLockedTest (extends AbstractSkipLockedTest)

The root cause is that, these test cases were trying to create a query, apply a pagination, and also specify lock options. The generated SQL (see below) did not expose alias properly hence the for update of [alias.column] cannot pass in Oracle.
-- Caused by: java.sql.SQLSyntaxErrorException: ORA-00904: "ABSTRACTSK0_"."ID": invalid identifier

select *
  from (
        select abstractsk0_.id as id1_0_,
               abstractsk0_.processed as processed2_0_
          from BatchJob abstractsk0_
       )
 where rownum <= :1
   for update of abstractsk0_.id skip locked

I am working on a fix, which will pass down the aliases only if the aliases intended to be locked are not equal to all the tables used in the query. Hopefully I'll be able to push something tomorrow.

The Travis build only runs the tests with H2.
About locking and pagination, I recently removed the SQL 2008 limit handler as Oracle did not support for update with it: https://github.com/hibernate/hibernate-orm/commit/d85831f5435a804e4a3654b9ced9c81c9b2f3222 .
I'm not sure there's a perfect solution for these limitations, especially when we don't have enough information to take the right decision in the dialects.

@gsmet - thanks for the clarification about the Travis builds. Apologise for my naive assumptions. Just curious - is there any other CI which is configured to run the tests, which are annotated with @RequiresDialect, e.g. for MySQL/PostgreSQL/DB2?
About that commit related with HHH-12848 - yes, I noticed that commit yesterday. It looks good because according to Oracle 12c document one of the restrictions of using row_limiting_clause is that,

you cannot specify this clause with for_update_clause.

So without that commit the for update of won't work with pagination at all.
I've pushed a commit that fixes all the failed tests mentioned in my previous
comment. Please refer to javadoc of the new method
Oracle8iDialect.needToSpecifyAliasesInForUpdate() for details.

@gsmet I have both Oracle 11g and 12c installed on my machine. Would you like me to test and validate this? If it works, I could integrate it to master and 5.3 if you want.

superseded by #9330


Could not introduce x, as it is not allowed by constraint * from project

What version of Go (go version) and dep (git describe --tags) are you using?
go version go1.8.3 linux/amd64
dep version v0.1.0-292-g7d36525 (up to date as of today)
What dep command did you run?
dep ensure -v github.com/a8m/tree

Root project is "github.com/ncw/rclone"
 75 transitively valid internal packages
 46 external packages imported from 26 projects
(0)   ✓ select (root)
(1)	? attempt bazil.org/fuse with 2 pkgs; at least 1 versions to try
(1)	    try bazil.org/fuse@master
(1)	✓ select bazil.org/fuse@master w/3 pkgs
(2)	? attempt github.com/aws/aws-sdk-go with 10 pkgs; at least 1 versions to try
(2)	    try github.com/aws/aws-sdk-go@master
(2)	✓ select github.com/aws/aws-sdk-go@master w/27 pkgs
(3)	? attempt github.com/Unknwon/goconfig with 1 pkgs; at least 1 versions to try
(3)	    try github.com/Unknwon/goconfig@master
(3)	✓ select github.com/Unknwon/goconfig@master w/1 pkgs
(4)	? attempt github.com/billziss-gh/cgofuse with 1 pkgs; at least 1 versions to try
(4)	    try<EMAIL_ADDRESS>(4)	✓ select<EMAIL_ADDRESS>w/1 pkgs
(5)	? attempt github.com/go-ini/ini with 1 pkgs; at least 1 versions to try
(5)	    try<EMAIL_ADDRESS>(5)	✓ select<EMAIL_ADDRESS>w/1 pkgs
(6)	? attempt github.com/VividCortex/ewma with 1 pkgs; at least 1 versions to try
(6)	    try github.com/VividCortex/ewma@master
(6)	✓ select github.com/VividCortex/ewma@master w/1 pkgs
(7)	? attempt github.com/ncw/go-acd with 1 pkgs; at least 1 versions to try
(7)	    try github.com/ncw/go-acd@master
(7)	✓ select github.com/ncw/go-acd@master w/1 pkgs
(8)	? attempt github.com/google/go-querystring with 1 pkgs; at least 1 versions to try
(8)	    try github.com/google/go-querystring@master
(8)	✓ select github.com/google/go-querystring@master w/1 pkgs
(9)	? attempt github.com/jmespath/go-jmespath with 1 pkgs; at least 1 versions to try
(9)	    try<EMAIL_ADDRESS>(9)	✓ select<EMAIL_ADDRESS>w/1 pkgs
(10)  ? attempt github.com/pkg/errors with 1 pkgs; at least 1 versions to try
(10)      try github.com/pkg/errors@master
(10)  ✓ select github.com/pkg/errors@master w/1 pkgs
(11)  ? attempt github.com/nsf/termbox-go with 1 pkgs; at least 1 versions to try
(11)      try github.com/nsf/termbox-go@master
(11)  ✓ select github.com/nsf/termbox-go@master w/1 pkgs
(12)  ? attempt github.com/mattn/go-runewidth with 1 pkgs; at least 1 versions to try
(12)      try<EMAIL_ADDRESS>(12)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(13)  ? attempt github.com/rfjakob/eme with 1 pkgs; at least 1 versions to try
(13)      try github.com/rfjakob/eme@master
(13)  ✓ select github.com/rfjakob/eme@master w/1 pkgs
(14)  ? attempt github.com/ncw/swift with 1 pkgs; at least 1 versions to try
(14)      try github.com/ncw/swift@master
(14)  ✓ select github.com/ncw/swift@master w/1 pkgs
(15)  ? attempt golang.org/x/crypto with 4 pkgs; at least 1 versions to try
(15)      try golang.org/x/crypto@master
(15)  ✓ select golang.org/x/crypto@master w/10 pkgs
(16)  ? attempt golang.org/x/time with 1 pkgs; at least 1 versions to try
(16)      try golang.org/x/time@master
(16)  ✓ select golang.org/x/time@master w/1 pkgs
(17)  ? attempt golang.org/x/oauth2 with 2 pkgs; at least 1 versions to try
(17)      try golang.org/x/oauth2@master
(17)  ✓ select golang.org/x/oauth2@master w/5 pkgs
(18)  ? attempt cloud.google.com/go with 1 pkgs; at least 1 versions to try
(18)      try<EMAIL_ADDRESS>(18)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(19)  ? attempt github.com/skratchdot/open-golang with 1 pkgs; at least 1 versions to try
(19)      try github.com/skratchdot/open-golang@master
(19)  ✓ select github.com/skratchdot/open-golang@master w/1 pkgs
(20)  ? attempt github.com/jlaffaye/ftp with 1 pkgs; at least 1 versions to try
(20)      try github.com/jlaffaye/ftp@master
(20)  ✓ select github.com/jlaffaye/ftp@master w/1 pkgs
(21)  ? attempt github.com/pkg/sftp with 1 pkgs; at least 1 versions to try
(21)      try github.com/pkg/sftp@master
(21)  ✓ select github.com/pkg/sftp@master w/1 pkgs
(22)  ? attempt github.com/kr/fs with 1 pkgs; at least 1 versions to try
(22)      try github.com/kr/fs@master
(22)  ✓ select github.com/kr/fs@master w/1 pkgs
(23)  ? attempt golang.org/x/net with 2 pkgs; at least 1 versions to try
(23)      try golang.org/x/net@master
(23)  ✓ select golang.org/x/net@master w/3 pkgs
(24)  ? attempt github.com/stretchr/testify with 2 pkgs; at least 1 versions to try
(24)      try github.com/stretchr/testify@master
(24)  ✓ select github.com/stretchr/testify@master w/2 pkgs
(25)  ? attempt github.com/davecgh/go-spew with 1 pkgs; at least 1 versions to try
(25)      try<EMAIL_ADDRESS>(25)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(26)  ? attempt github.com/spf13/cobra with 2 pkgs; at least 1 versions to try
(26)      try github.com/spf13/cobra@master
(26)  ✓ select github.com/spf13/cobra@master w/2 pkgs
(27)  ? attempt github.com/cpuguy83/go-md2man with 1 pkgs; at least 1 versions to try
(27)      try<EMAIL_ADDRESS>(27)  ✓ select<EMAIL_ADDRESS>w/1 pkgs
(28)  ? attempt github.com/ncw/dropbox-sdk-go-unofficial with 2 pkgs; at least 1 versions to try
(28)      try github.com/ncw/dropbox-sdk-go-unofficial@master
(28)  ✓ select github.com/ncw/dropbox-sdk-go-unofficial@master w/4 pkgs
(29)  ? attempt github.com/a8m/tree with 1 pkgs; 1 versions to try
(29)      try github.com/a8m/tree@master
(30)  ✗   github.com/a8m/tree@master not allowed by constraint *:
(30)      * from (root)
(29)    ← no more versions of github.com/a8m/tree to try; begin backtrack
(28)  ← backtrack: no more versions of github.com/ncw/dropbox-sdk-go-unofficial to try
(27)  ← backtrack: no more versions of github.com/cpuguy83/go-md2man to try
(26)  ← backtrack: no more versions of github.com/spf13/cobra to try
(25)  ← backtrack: no more versions of github.com/davecgh/go-spew to try
(24)  ← backtrack: no more versions of github.com/stretchr/testify to try
(23)  ← backtrack: no more versions of golang.org/x/net to try
(22)  ← backtrack: no more versions of github.com/kr/fs to try
(21)  ← backtrack: no more versions of github.com/pkg/sftp to try
(20)  ← backtrack: no more versions of github.com/jlaffaye/ftp to try
(19)  ← backtrack: no more versions of github.com/skratchdot/open-golang to try
(18)  ← backtrack: no more versions of cloud.google.com/go to try
(17)  ← backtrack: no more versions of golang.org/x/oauth2 to try
(16)  ← backtrack: no more versions of golang.org/x/time to try
(15)  ← backtrack: no more versions of golang.org/x/crypto to try
(14)  ← backtrack: no more versions of github.com/ncw/swift to try
(13)  ← backtrack: no more versions of github.com/rfjakob/eme to try
(12)  ← backtrack: no more versions of github.com/mattn/go-runewidth to try
(11)  ← backtrack: no more versions of github.com/nsf/termbox-go to try
(10)  ← backtrack: no more versions of github.com/pkg/errors to try
(9)	← backtrack: no more versions of github.com/jmespath/go-jmespath to try
(8)	← backtrack: no more versions of github.com/google/go-querystring to try
(7)	← backtrack: no more versions of github.com/ncw/go-acd to try
(6)	← backtrack: no more versions of github.com/VividCortex/ewma to try
(5)	← backtrack: no more versions of github.com/go-ini/ini to try
(4)	← backtrack: no more versions of github.com/billziss-gh/cgofuse to try
(3)	← backtrack: no more versions of github.com/Unknwon/goconfig to try
(2)	← backtrack: no more versions of github.com/aws/aws-sdk-go to try
(1)	← backtrack: no more versions of bazil.org/fuse to try
  ✗ solving failed

Solver wall times by segment:
     b-source-exists: 18.131155182s
         b-list-pkgs:  2.611433135s
  b-deduce-proj-root:  2.269181913s
              b-gmal:  1.345081685s
         select-atom:   23.978823ms
             satisfy:   22.843665ms
            unselect:   22.679971ms
         select-root:   21.986503ms
            new-atom:    1.318785ms
           backtrack:     194.886µs
               other:      41.211µs
           b-matches:       6.519µs
     b-list-versions:       5.439µs
          b-pair-rev:       3.897µs

  TOTAL: 24.449911614s

ensure Solve(): No versions of github.com/a8m/tree met constraints:
	master: Could not introduce github.com/a8m/tree@master, as it is not allowed by constraint * from project github.com/ncw/rclone.

$ dep hash-inputs
-CONSTRAINTS-
bazil.org/fuse
b-master
github.com/Unknwon/goconfig
b-master
github.com/VividCortex/ewma
b-master
github.com/aws/aws-sdk-go
b-master
github.com/ncw/dropbox-sdk-go-unofficial
b-master
github.com/ncw/go-acd
b-master
github.com/ncw/swift
b-master
github.com/pkg/errors
b-master
github.com/pkg/sftp
b-master
github.com/rfjakob/eme
b-master
github.com/skratchdot/open-golang
b-master
github.com/spf13/cobra
b-master
github.com/spf13/pflag
b-master
github.com/stretchr/testify
b-master
golang.org/x/crypto
b-master
golang.org/x/net
b-master
golang.org/x/oauth2
b-master
golang.org/x/sys
r-cd2c276457edda6df7fb04895d3fd6a6add42926
golang.org/x/text
b-master
google.golang.org/api
b-master
-IMPORTS/REQS-
bazil.org/fuse
bazil.org/fuse/fs
github.com/Unknwon/goconfig
github.com/VividCortex/ewma
github.com/a8m/tree
github.com/aws/aws-sdk-go/aws
github.com/aws/aws-sdk-go/aws/awserr
github.com/aws/aws-sdk-go/aws/corehandlers
github.com/aws/aws-sdk-go/aws/credentials
github.com/aws/aws-sdk-go/aws/credentials/ec2rolecreds
github.com/aws/aws-sdk-go/aws/ec2metadata
github.com/aws/aws-sdk-go/aws/request
github.com/aws/aws-sdk-go/aws/session
github.com/aws/aws-sdk-go/service/s3
github.com/aws/aws-sdk-go/service/s3/s3manager
github.com/billziss-gh/cgofuse/fuse
github.com/jlaffaye/ftp
github.com/ncw/dropbox-sdk-go-unofficial/dropbox
github.com/ncw/dropbox-sdk-go-unofficial/dropbox/files
github.com/ncw/go-acd
github.com/ncw/swift
github.com/nsf/termbox-go
github.com/pkg/errors
github.com/pkg/sftp
github.com/rfjakob/eme
github.com/skratchdot/open-golang/open
github.com/spf13/cobra
github.com/spf13/cobra/doc
github.com/spf13/pflag
github.com/stretchr/testify/assert
github.com/stretchr/testify/require
github.com/xanzy/ssh-agent
golang.org/x/crypto/nacl/secretbox
golang.org/x/crypto/scrypt
golang.org/x/crypto/ssh
golang.org/x/crypto/ssh/terminal
golang.org/x/net/context
golang.org/x/net/html
golang.org/x/oauth2
golang.org/x/oauth2/google
golang.org/x/sys/unix
golang.org/x/text/unicode/norm
golang.org/x/time/rate
google.golang.org/api/drive/v2
google.golang.org/api/googleapi
google.golang.org/api/storage/v1
-IGNORES-
-OVERRIDES-
-ANALYZER-
dep
1

What did you expect to see?
I expected it to add the dependency
What did you see instead?
This rather cryptic error message
ensure Solve(): No versions of github.com/a8m/tree met constraints:
	master: Could not introduce github.com/a8m/tree@master, as it is not allowed by constraint * from project github.com/ncw/rclone.

Reproduction

Check out the tree branch of github.com/ncw/rclone

Gopkg.toml
Gopkg.lock

Run dep ensure -v github.com/a8m/tree

Here is a walkthrough of the above in a clean gopath
$ cd /tmp
$ mkdir testgopath
$ export GOPATH=/tmp/testgopath
$ cd testgopath/
$ go get github.com/ncw/rclone
$ cd src/github.com/ncw/rclone/
$ git checkout origin/tree
HEAD is now at ff1ee18f... Implement tree command - fixes #1528
$ dep ensure -v github.com/a88m/tree
Root project is "github.com/ncw/rclone"
[snip - as above]
ensure Solve(): No versions of github.com/a8m/tree met constraints:
	master: Could not introduce github.com/a8m/tree@master, as it is not allowed by constraint * from project github.com/ncw/rclone.

I expected it to add the dependency

That's an incorrect assumption, at least currently. If you want to add a new dependency either import it in your code or add it to required in Gopkg.toml.
I'm not able to reach github.com/a88m/tree atm though.

@ibrasho the dependency is imported in the code already.
I haven't seen that bit of text you linked to before - is it new? I've been adding dependencies using
dep ensure package/name.

Maybe this is just user error then, apart from the baffling error message!

What output do you get from dep ensure -v?

yeah, the confusion there is in how we stringify different constraint types. "*" is an open semver constraint that allows any semver tag; "" (which it's basically impossible to ever see in output) allows any version of ANY type - semver tag, branch, not semver tag, revision.
this is definitely a suboptimal situation 😢. I'm hoping to be able to get to working on  errors soon


Bugfix/382 support using folkehelseinstituttetstyle in monorepo

@yleaxeman Split this in two new issues as discussed.