Error each Godot start at DisplayServerWindows::cursor_set_shape()/VulkanContext::window_resize()

Godot 4 b3310a00743d65d8baa2b5730ecfaed8bf8842a6
Windows 10 64 bits
GeForce GTX 1060 6GB/PCIe/SSE2
Compiled with MSVC
Every time I open Godot, be it project manger or editor, the following errors happen in the console:
Godot Engine v4.0.dev.custom_build.b3310a007 - https://godotengine.org
ERROR: Index p_shape = -1163005939 is out of bounds (CURSOR_MAX = 17).
   at: DisplayServerWindows::cursor_set_shape (platform\windows\display_server_windows.cpp:1197)

ERROR: Condition "!windows.has(p_window)" is true.
   at: VulkanContext::window_resize (drivers\vulkan\vulkan_context.cpp:722)

Can you check if this patch works(I don't have for now ability to check this on Windows)?
It just move place of setting value of cursor_shape at the top of constructor, because probably this variable is used somewhere in constructor uninitialised. (probably same as #37321)
diff --git a/platform/windows/display_server_windows.cpp b/platform/windows/display_server_windows.cpp
index ebe9a7d27a..aba8194c81 100644
--- a/platform/windows/display_server_windows.cpp
+++ b/platform/windows/display_server_windows.cpp
@@ -2797,6 +2797,8 @@ DisplayServerWindows::DisplayServerWindows(const String &p_rendering_driver, Win
        drop_events = false;
        key_event_pos = 0;

+       cursor_shape = CURSOR_ARROW;
+
        alt_mem = false;
        gr_mem = false;
        shift_mem = false;
@@ -2936,8 +2938,6 @@ DisplayServerWindows::DisplayServerWindows(const String &p_rendering_driver, Win
                SetThreadPriority(GetCurrentThread(), THREAD_PRIORITY_TIME_CRITICAL);
        }

-       cursor_shape = CURSOR_ARROW;
-
        _update_real_mouse_position(MAIN_WINDOW_ID);

        joypad = new JoypadWindows(&windows[MAIN_WINDOW_ID].hWnd);

@qarmin your patch fixes the first error about the cursor. The second error remains.

Hello,
I just build Godot (Godot Engine v4.0.dev.custom_build.ff5dfcdf6) on Windows 10 64bits.
Same thing for me, a few issues when Godot 4.0 was launched :
ERROR: Some texture bindings were not properly freed (leaked canvasitems?
   at: ~RasterizerCanvasRD (servers/rendering/rasterizer_rd/rasterizer_canvas_rd.cpp:2564)
ERROR: Condition "!windows.has(p_window_id)" is true.
   at: window_destroy (drivers/vulkan/vulkan_context.cpp:761)
ERROR: Condition "!windows.has(p_window)" is true.
   at: window_resize (drivers/vulkan/vulkan_context.cpp:729)

The above issues only occur once during startup.
Then the following error repeats itself indefinitely :
ERROR: Condition "!uniform_set" is true.
   at: draw_list_bind_uniform_set (drivers/vulkan/rendering_device_vulkan.cpp:5960)

Finally when I leave the editor, I have these issues again :
ERROR: Some texture bindings were not properly freed (leaked canvasitems?
   at: ~RasterizerCanvasRD (servers/rendering/rasterizer_rd/rasterizer_canvas_rd.cpp:2564)
ERROR: Condition "!windows.has(p_window_id)" is true.
   at: window_destroy (drivers/vulkan/vulkan_context.cpp:761)

I'm pretty sure this has been fixed since then, so closing. Please comment if it's still valid.


Angular Connect

27/28 of september
CFP closes on May 31
http://angularconnect.com/ CFP

Sounds like fun, but I'm not really that much into Angular any more - sorry


ARTEMIS-1208 Do not use reconnect-atempts=-1 in tests

If a connection to a broker is configured with parameter reconnect-atempts=-1 and something goes wrong with the broker, the test may hang indefinitely. It happens in cases where the session is created in main test thread.

How about other test areas, such as MultiThreadRandomReattachTestBase, ConsumerTest etc, doing a quick search it seems there's about 40 files with setReconnectAttempts(-1), i assume a large proportion of these are tests?

You are right. Tomorrow I'll take a better look and update all occurrences of setReconnectAttempts in tests.

@dudaerich  did you run the whole testsuite on this?

@clebertsuconic Yes, I did. Currently there is a known issue with replicated HA and the number of failed tests in this area differs with each run. It is hard to say if this commit breaks some tests or not. I took a brief look on test results but I didn't identify any failure caused by some kind of error that connection was closed or something like that. All issues what I saw where related to either set up phase (backup wasn't synchronized with live in timeout) or clients didn't receive response on packet.
To be 100% sure we can increase reconnect-attempts or reconnect-delay. Currently reconnect-delay=50 and reconnect-attempts=200 what gives clients 10s to reconnect. Maybe we can increase the time to 30s. E.g. reconnect-delay=100 and reconnect-attempts=300 wdyt?

I've changed the reconnect-attempts attribute in all tests. I've set it in a way to give clients 30s for reconnection.
Don't merge this please. I want to run the whole testsuite first.

this increased the failure rate on a lot of FailoverTests, ReplicationTests.. etc.. failing because of this.
Are you sure this is an issue? ThreadLeak would find any threads still trying to reconnect.. that's actually why i developed that rule on ThreadLeaks.

mainly failures are added by testCreateNewFactoryAfterFailover and testTimeoutOnFailoverConsumeBlocked...
a lot of the extending failover tests because of that...
Maybe it's easy to fix.. but I would like to see it solved before we merge.. otherwise it's hard to pinpoint the failure.
I want to understand also if this is really an issue.. as the ThreadLeak thingy would catch them.

@clebertsuconic good catch! I'll take a look why testCreateNewFactoryAfterFailover fails.
I've attached thread dump from hanging test to https://issues.apache.org/jira/browse/ARTEMIS-1208.
Bellow are two threads which block the test. Main thread is waiting for failoverLock which is held by thread doing reconnection with reconnect-attempts=-1.
"main" Id=1 WAITING on java.util.concurrent.locks.ReentrantLock$NonfairSync@6e460863 owned by "Thread-4 (ActiveMQ-client-global-threads)" Id=5564 (in native)
	at sun.misc.Unsafe.park(Native Method)
	-  waiting on java.util.concurrent.locks.ReentrantLock$NonfairSync@6e460863
	at java.util.concurrent.locks.LockSupport.park(LockSupport.java:186)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.parkAndCheckInterrupt(AbstractQueuedSynchronizer.java:847)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireQueued(AbstractQueuedSynchronizer.java:881)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquire(AbstractQueuedSynchronizer.java:1210)
	at java.util.concurrent.locks.ReentrantLock$NonfairSync.lock(ReentrantLock.java:220)
	at java.util.concurrent.locks.ReentrantLock.lock(ReentrantLock.java:296)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.lockFailover(ClientSessionFactoryImpl.java:230)
	at org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQClientProtocolManager.lockSessionCreation(ActiveMQClientProtocolManager.java:163)
	at org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQClientProtocolManager.createSessionContext(ActiveMQClientProtocolManager.java:272)
	at org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQClientProtocolManager.createSessionContext(ActiveMQClientProtocolManager.java:237)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.createSessionChannel(ClientSessionFactoryImpl.java:1284)
	-  locked java.lang.Object@18ee7eb5
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.createSessionInternal(ClientSessionFactoryImpl.java:670)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.createSession(ClientSessionFactoryImpl.java:323)
	at org.apache.activemq.artemis.tests.integration.cluster.failover.FailoverTest.createSession(FailoverTest.java:94)
	at org.apache.activemq.artemis.tests.integration.cluster.failover.FailoverTest.simpleFailover(FailoverTest.java:693)
	at org.apache.activemq.artemis.tests.integration.cluster.failover.FailoverTest.testFailBack(FailoverTest.java:534)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:95)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:55)
	at java.lang.reflect.Method.invoke(Method.java:508)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
	at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
	at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
	at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
	at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:48)
	at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
	at org.junit.rules.RunRules.evaluate(RunRules.java:20)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
	at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
	at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:283)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:173)
	at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
	at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:128)
	at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203)
	at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)

"Thread-4 (ActiveMQ-client-global-threads)" Id=5564 TIMED_WAITING on java.util.concurrent.CountDownLatch$Sync@160a2aa1 (in native)
	at sun.misc.Unsafe.park(Native Method)
	-  waiting on java.util.concurrent.CountDownLatch$Sync@160a2aa1
	at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:226)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedNanos(AbstractQueuedSynchronizer.java:1048)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.tryAcquireSharedNanos(AbstractQueuedSynchronizer.java:1339)
	at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:288)
	at org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQClientProtocolManager.waitOnLatch(ActiveMQClientProtocolManager.java:136)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.getConnectionWithRetry(ClientSessionFactoryImpl.java:819)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.reconnectSessions(ClientSessionFactoryImpl.java:744)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.failoverOrReconnect(ClientSessionFactoryImpl.java:614)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.handleConnectionFailure(ClientSessionFactoryImpl.java:504)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl.access$500(ClientSessionFactoryImpl.java:72)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl$DelegatingFailureListener.connectionFailed(ClientSessionFactoryImpl.java:1165)
	at org.apache.activemq.artemis.spi.core.protocol.AbstractRemotingConnection.callFailureListeners(AbstractRemotingConnection.java:67)
	at org.apache.activemq.artemis.core.protocol.core.impl.RemotingConnectionImpl.fail(RemotingConnectionImpl.java:208)
	at org.apache.activemq.artemis.spi.core.protocol.AbstractRemotingConnection.fail(AbstractRemotingConnection.java:210)
	at org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl$CloseRunnable.run(ClientSessionFactoryImpl.java:996)
	at org.apache.activemq.artemis.utils.OrderedExecutorFactory$OrderedExecutor$ExecutorTask.run(OrderedExecutorFactory.java:101)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1153)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.lang.Thread.run(Thread.java:785)

	Number of locked synchronizers = 2
	- java.util.concurrent.locks.ReentrantLock$NonfairSync@6e460863
	- java.util.concurrent.ThreadPoolExecutor$Worker@e8c525c7

I found out why testCreateNewFactoryAfterFailover failed. I changed the default value of reconnect-attempts from 0 to 300 and the test needed reconnect-attempts=0.
I've updated the PR. I threw away previous changes and I did it again. I didn't change locator created in FailoverTestBase. I changed only code which set up reconnect-attempts to -1.
I ran the modified test cases locally on my laptop and all tests passed.


update /about page

including removing Clarifai logo and blurb. Jane will provide updated text here before reassigning to Breno

please update with final text in [google doc](please update with final text in google doc sent in email -- not open to public.) sent in email -- not open to public.

fix typo -- add space "strategy and roadmap for CC Search hereand" so it reads "strategy and roadmap for CC Search here and" while retaining link

also need a space here "For more details, read theCC Terms of Use." between the and terms of use

It's easier to do it myself, and let @brenoferreira focus on other stubborn issues! 😄


Fix redirect path after Cancel is clicked in VM Migrate

Use the URL available in breadcrumbs for redirect instead of constructing a new URL.
https://bugzilla.redhat.com/show_bug.cgi?id=1339924

looks good.


document the dhcp packet

I am more used to the wording of the RFC (yiaddr, chaddr, etc.) so I was a little bit confused by the attribute names. I think documenting them and makes it easier to relate them to the RFC.
Most of the text is adapted from the RFC, and from wikipedia, with some big chunks copy/pasted.
I hope it's ok.

Awesome, thanks a lot!


Fix warning of unsafe YAML loader

Compilation issues warning: "YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details."
This commit specifies the loader as recommended in https://msg.pyyaml.org/load

Thanks for bringing this to my attention! There was an extra ) in your commit, which caused the build to fail.
I removed the extra close-paren in 8553de2f335baab337bbff09ff79bd512dbc27fb and merged this.

And now deployed to PyPI :tada: https://pypi.org/project/pandoc-mustache/0.1.0/


Streams

I'm trying to add the stream functionality in the toPdf method. The goal is to be able to work with streams instead of files, when it's needed. Rendering in files is still supported.
Currently the toPdf function doesn't work because:

I don't know how to set custom html that comes not from a file but from a string value.
I don't know how to pipe the result pdf into a stream, not into a file.

However, the toHtml is working well. If no output field is specified in options, then the callback result will be the html string:
require("http").createServer(function (req, res) {
    myInvoice.toHtml(function (err, data) {
        res.end(data);
    });
}).listen(8000);

This will fix #4. :fire:

Maybe node-html-pdf by @marcbachmann could help here!  :)
Surprised by the fast responses and interest in the submitted issue! :)
Never happened to me before, really appreciate it!
fredag 7 augusti 2015 skrev Ionică Bizău<EMAIL_ADDRESS>
Maybe node-html-pdf https://github.com/marcbachmann/node-html-pdf by
@marcbachmann https://github.com/marcbachmann could help here! I'm
gonna give a try. [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/IonicaBizau/nodeice/pull/5#issuecomment-128745603.

@idlyapp Hehe, I'm always trying to do my best. I'm really happy when people appreciate the work I've done. Donations are always welcome! Thanks! :four_leaf_clover:

@idlyapp I pushed a fix for the toPdf function. Please update your code (see the examples in test/index.js) and see what happens. Some tmp files are still created in /tmp dir. Hopefully, Heroku accepts that.
I opened an issue about that here: https://github.com/marcbachmann/node-html-pdf/issues/61

Something like this:
require("http").createServer(function (req, res) {
    myInvoice.toPdf({ output: res });
}).listen(8000);

Sounds great will get back to you in a couple of h! Thx!
fredag 7 augusti 2015 skrev Ionică Bizău<EMAIL_ADDRESS>
@idlyapp https://github.com/idlyapp I pushed a fix for the toPdf
function. Please update your code (see the examples in test/index.js) and
see what happens. Some tmp files are still created in /tmp dir.
Hopefully, Heroku accepts that.
I opened an issue about that here: marcbachmann/node-html-pdf#61
https://github.com/marcbachmann/node-html-pdf/issues/61
—
Reply to this email directly or view it on GitHub
https://github.com/IonicaBizau/nodeice/pull/5#issuecomment-128749968.

Is res the file i should pass to s3?
fredag 7 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS>
Sounds great will get back to you in a couple of h! Thx!
fredag 7 augusti 2015 skrev Ionică Bizău<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>:

@idlyapp https://github.com/idlyapp I pushed a fix for the toPdf
function. Please update your code (see the examples in test/index.js)
and see what happens. Some tmp files are still created in /tmp dir.
Hopefully, Heroku accepts that.
I opened an issue about that here: marcbachmann/node-html-pdf#61
https://github.com/marcbachmann/node-html-pdf/issues/61
—
Reply to this email directly or view it on GitHub
https://github.com/IonicaBizau/nodeice/pull/5#issuecomment-128749968.

No sorry that was http response.
As i understand it the variable i put after output: will be the file?
fredag 7 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS>
Is res the file i should pass to s3?
fredag 7 augusti 2015 skrev Stellan Haglund<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','sh@idly.se');>>:

Sounds great will get back to you in a couple of h! Thx!
fredag 7 augusti 2015 skrev Ionică Bizău<EMAIL_ADDRESS>
@idlyapp https://github.com/idlyapp I pushed a fix for the toPdf
function. Please update your code (see the examples in test/index.js)
and see what happens. Some tmp files are still created in /tmp dir.
Hopefully, Heroku accepts that.
I opened an issue about that here: marcbachmann/node-html-pdf#61
https://github.com/marcbachmann/node-html-pdf/issues/61
—
Reply to this email directly or view it on GitHub
https://github.com/IonicaBizau/nodeice/pull/5#issuecomment-128749968.

No, it's the response stream which comes from the createServer callback.

Great news! now it's showing in browser! :)
But how would I send the file to for example s3?

The invoice rows doesn't seem to be rendered though..

I figured it out,
myInvoice.toPdf({
output: res
}, function (err, data){
console.log(err, data.path);
});

If you pass a stream to in the output field, then the pdf stream will be piped there. That means you can pass the request stream:
myInvoice.toPdf({
    output: Request("http://domain.com/upload-invoice")
}, function (err, data){
    console.log(err, data.path);
});

It should work I guess... maybe with some changes. :smile:

any idea why task rows isn't rendered?

@idlyapp Do you have an example (online)?

I actually did it a little different way, using express, so I simply do a post call to the url and the pdf is rendered and uploaded to where I want it.
This way I can simply put the invoice data as the body of the post, so I don't have a url where you will see the pdf just by loading it.
Is it working for you?

@idlyapp Thanks for the heads-up! I pushed a fix regarding the description rows.

Awesome! :)

:ship: 2.0.0 :tada:
@stellanhaglund Sorry it took so long to get this fixed, but finally it's happening! :smile:


Reconstruction Error

Very nice work! I have a question, while EDICT is invertible by design, why the reconstruction error is not zero as shown in Table 1?

Thank you! The reconstruction error from Table 1 is pixel-level so the VAE encoding/decoding process introduces some level of error (which is why the LDM VAE and 'EDICT` columns are all equal). EDICT is exact at the latent level (up to floating point precision, but that's too small to register in the table) but since the pixel measurement involves the VAE it inherits that level of error. Does that make sense?

Thanks for the reply, that makes sense! Is it possible to modify the VAE to fix the error?

Thanks for the reply, that makes sense! Is it possible to modify the VAE to fix the error?

I also want to know how to reduce the error further by modifying the VAE. Can anyone provide some suggestions?

Thanks for the reply, that makes sense! Is it possible to modify the VAE to fix the error?

I also want to know how to reduce the error further by modifying the VAE. Can anyone provide some suggestions?

The VAE is trained separately from the diffusion model. If you were to modify the VAE, it would make the outputs inconsistent with the diffusion model.
So, to improve the reconstruction error from VAE you would have to train both the VAE and diffusion model from scratch.
This is exactly what was done in latest Stable Diffusion versions. The latest versions of Stable Diffusion have improved their VAE, see the table below from SDXL paper.


Product VAT percentages

Products have a VAT percentage, at the end of the year the treasurer needs to correct the VAT prices in our accountancy system (exactonline). Normally this is an easy query since alcohol is the only product with 21% VAT. But the new cabinet is looking into increasing the VAT for products with high sugar %. A possible solution would be to give every product a VAT % or per category, but this is up to the programmer.  Furthermore, it would be nice to have some overview where you can see the total price of products bought grouped by VAT Percentage for the treasurer.

Done in a7246d4bbc08127e7c15c2a8851a7bb1e053a826


Updated  comments

I updated the comments to it easier to understand for newbies.

I don't see any real changes here.


Properly handle interleaved messages in Flowee

The Flowee binary protocol allows for long messages to be split into chunks of 8KiB or so. We support this properly. The protocol does, however, allow for other messages to be transmitted in between these chunks. We don't handle that correctly at the moment as the code in Flowee doesn't seem to utilize this yet. We should be robust here, though.
For completeness we should perhaps also split outgoing messages to handle messages larger than 32KiB. Currently, we don't ever to this, so it is not a pressing issue.
This would, however, need some refactoring of the interface to Flowee.

Flowee is no longer up-to-date with BCH consensus rules, so closing this for now.


Clarify how to deal with CHANGELOG.md in contributor docs

What should we add?
Very minor, but the contributing docs currently state: "All changes must be documented in the changelog." It might be helpful to specify that the user doesn't have to update the version --- just the additions/fixes.
I was initially a bit confused by this when contributing to the slurm plugin.

Ah, closing because actually my confusion was stemming from the PR template in the slurm plugin.


Javascript error: Cannot access 'player' before initialization

ReferenceError: Cannot access 'player' before initialization
    at restartPlayer (player.js:66)

@gingervitis is refactoring all of the JS so I'm just going to close this.


Add versioning properties

Problem statement
Following DCAT 3, DCAT-AP 3 introduces a set of versioning properties, replacing the corresponding ones in earlier versions of DCAT-AP (see the DCAT-AP 3 specification, §A.1).
In particular:

dcat:hasVersion replaces dct:hasVersion (deprecated).
dcat:isVersionOf replaces dct:isVersionOf (deprecated).
dcat:version replaces owl:versionInfo (deprecated).

Proposal
To be aligned with DCAT-AP 3 and DCAT 3, GeoDCAT-AP 3 should deprecate the use of dct:hasVersion, dct:isVersionOf, and owl:versionInfo, and replace them with dcat:hasVersion, dcat:isVersionOf, and dcat:version, respectively.

dcat:hasVersion and dcat:version were added. dcat:isVersionOf was removed due to #90


Build a Python version SDK for Casdoor

Current Casdoor SDKs have no Python version, and we can build one for Python dev.

@hsluoyz we can open a new repo under Casdoor


Bump bridge-method-annotation from 1.18 to 1.23

Bumps bridge-method-annotation from 1.18 to 1.23.

Commits

4682998 [maven-release-plugin] prepare release bridge-method-injector-parent-1.23
a2c094f Merge pull request #27 from basil/asm
cd1a8ef Bump ASM from 9.0 to 9.2
1567095 [maven-release-plugin] prepare for next development iteration
37632e6 [maven-release-plugin] prepare release bridge-method-injector-parent-1.22
a778d55 Merge pull request #26 from jglick/listClassNames-JENKINS-66247
02cd7e4 jenkinsci/lib-annotation-indexer#12 released
2788ba4 [JENKINS-66247] Call Index.listClassNames
762976d Merge pull request #20 from matthiasblaesing/missing_dependency_central
f9a8222 Merge pull request #23 from infradna/dependabot/maven/injector/junit-junit-4....
Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

@dependabot rebase will rebase this PR
@dependabot recreate will recreate this PR, overwriting any edits that have been made to it
@dependabot merge will merge this PR after your CI passes on it
@dependabot squash and merge will squash and merge this PR after your CI passes on it
@dependabot cancel merge will cancel a previously requested merge and block automerging
@dependabot reopen will reopen this PR if it is closed
@dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
@dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Author: dependabot[bot]. Timestamp: 2022-11-16T22:10:24Z

Superseded by #63.


multitouch and pinch zoom?

Hi!
Can I use this project to detect multiple touches at the same time? I want to detect the touch of two fingers to zoom the camera.

Right now there is no multi-touch support, neither pinch zoom gesture.


A code definition window like visual studio

Hi
Can your team add a code definition window like visual studio already has.

I know there is already a peek window in vscode, but it is not enough, for :

it needs ctrl+click to call peek window
peek window can not be seen when user scroll screen up or down

what I am expecting is :

the definition window shows the symbol definition automatically when I put the cursor to the symbol (like visual studio)
the definition window is a independent window but not build-in control in the editor (like visual studio)
when I scroll down and up in the definition window, I can see the other part of codes in the file where the symbol defined.(like visual studio)
when I double-click anywhere in the definition window, a new editor opened to the 2nd editor group, to show the file where I clicked
when I click other symbol in the definition window, then window jumps to the new clicked symbol's defined location

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Typeorm returns a date string instead of a date object for date column

If I set a date column with   @CreateDateColumn({type:'Date'}) in an entity. Typeorm would return a string {created: "2018-05-18"} instead of a date object
Is it possible for type-graphql or typeorm to automatically convert that date string into a date object? I thought Typeorm would automatically do so like a dateTime column but it won't.
@Entity()
@ObjectType()
export class Rate {
  @PrimaryGeneratedColumn()
  readonly id: number;

  @Field()
  @CreateDateColumn({type:'Date'})
  created: Date;

}

AFAIK, this should be a Date instance:
https://github.com/typeorm/typeorm/blob/master/docs/decorator-reference.md#createdatecolumn
Please ask on TypeORM gitter or create an issue on TypeORM repository about the type of @CreateDateColumn.


Fix virtual attribute selection

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1461939
When expanding a virtual attribute that is an array of values (ie, power_states => ["ok","ok"] on a service), it is currently failing with the error:
{"error":{"kind":"internal_server_error","message":"undefined method `id' for \"on\":String","klass":"NoMethodError"}}

This is due to these values being passed to Rbac.filterer, which is looking for id of object. This fix avoids sending the value of the virtual attribute to filterer.
@miq-bot add_label api, bug
@miq-bot assign @abellotti
cc: @AllenBW

🌮 💃 ❤

I love this pr. SUPER love this pr.  🌮 🎉

This LGTM!!  Thanks @jntullo for fixing this.  👍

@miq-bot add_label fine/yes

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1466114

Fine backport details:
$ git log -1
commit d4fdfebcd331fe352eda12fef64f73917bc40ca3
Author: Alberto Bellotti<EMAIL_ADDRESS>Date:   Tue Jun 27 11:01:17 2017 -0400

    Merge pull request #15387 from jntullo/bz/fix_virtual_attr_expansion

    Fix virtual attribute selection
    (cherry picked from commit e92afb191454382639fb6f69423ef3419dec31c6)

    https://bugzilla.redhat.com/show_bug.cgi?id=1468294


url('file') is undefined with url end with no extension

version: 2.0.2
case 1, end with no extension or forward slash:
var urlString = http://localhost/news/list
window.url('file', urlString): undefined		//should be "list"
window.url('filename', urlString): undefined	//should be "list"
window.url('-1', urlString): list

case 2, end with dot, no extension:
var urlString = http://localhost/news/list.
window.url('file', urlString): undefined		//should be "list."
window.url('filename', urlString): undefined	//should be "list"
window.url('-1', urlString): list.

case 3, end with forward slash:
var urlString = http://localhost/news/list/
window.url('file', urlString): undefined
window.url('filename', urlString): undefined
window.url('-1', urlString): list				//should be ""

case 4, regular:
var urlString = http://localhost/news/list.html
window.url('file', urlString): list.html
window.url('filename', urlString): list
window.url('-1', urlString): list.html

This is intended behavior in all cases. Particularly with just a /something path. Actually a system has no idea really whether "you" mean path/directory or file.
You could perhaps do:
var file = window.url('file', urlString) || window.url('-1', urlString);

Or you can fetch the entire parsed url with:
window.url('[]', urlString);

when a url end with forward slash, it's usually a path/directory, which the file name is hidden/omitted, such as "index.html"; while a url end with file name without extension, it‘s usually because the rewrite rule.
in my case, i need to use History.pushState() to change the url query strings.
for example, change "http://localhost/news/list" into "http://localhost/news/list?type=1", and "http://localhost/news/list/" into "http://localhost/news/list/?type=1". so i have to discriminate these two situations.
in the "http://localhost/news/list/" case, window.url('-1', urlString) is giving "list", rather than "list/". than with var file = window.url('file', urlString) || window.url('-1', urlString);, it will finally turn into "http://localhost/news/list/list?type=1" :(
i haven't found window.url('[]', urlString); in readme, test it later. (is it similar to window.url('', urlString);, but store values in a array?)

Seems the issue is more about /list/ returning '' rather than list. But this was a conscious decision. I'm not sure about specific RFC's, but generally I see that if you have a route /list and /list/ they will serve the same request. Furthermore /list/ will return null if you try to do some kind of ...segment(2) call. We can also add that most webservers will strip the forward slash if nothing comes after it.
On a side note, on unix based systems you can not have a file and a directory with the same name at the same level. However on Windows you can so I guess there is some arguing.
However if the forward slash were to be implemented, then it would follow that /list/ would return '' (empty string) for -1 and not list/.
I think it's probably best to avoid having "files" and "directories" with the same name. Which would pretty much remove the issue altogether.

i agree it is a "conscious decision". and yes, the problem is caused by a .net project on windows. i wasn't aware it until our back-end engineer told me.
never mind, i replaced it with native window.location.pathname witch will give the both ".../list" and ".../list/".


Sort subjects consistently when doing nats subscribe --report-subjects

Previously, report subjects only sorted by message count, but if two subjects had the same message count, they would not be sorted consistently.
This change sorts the subjects first by count, then by subject

Thank you


{AKS - ARC} fix: Update DCR creation to Clusters resource group instead of workspace

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

This PR is updating the creation of the DCR to the clusters resource group instead of the workspaces resource group to make it consistent with our new addon experience (managed prometheus azure monitor metrics addon)
General Guidelines

[] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

Hey @bavneetsingh16 Can you please help me in merging this PR in?

@NarayanThiru , can you please help on reviewing and merging this PR?


Update Dockerfile

Updating the runmode from publish to author, on author image.

Hi @thiagosanches , thanks for the fix!


Update eslint, add prettier (with lint-staged and husky)

Codecov Report

Merging #365 into master will decrease coverage by 0.08%.
The diff coverage is 89.01%.

@@            Coverage Diff             @@
##           master     #365      +/-   ##
==========================================
- Coverage   93.07%   92.98%   -0.09%
==========================================
  Files          51       51
  Lines         332      328       -4
==========================================
- Hits          309      305       -4
  Misses         23       23

Impacted Files
Coverage Δ

src/packages/recompose/createEventHandler.js
87.5% <ø> (ø)
:arrow_up:

src/packages/recompose/renameProp.js
100% <ø> (ø)
:arrow_up:

src/packages/recompose/mostObservableConfig.js
100% <ø> (ø)
:arrow_up:

src/packages/recompose/withProps.js
100% <ø> (ø)
:arrow_up:

src/packages/recompose/onlyUpdateForPropTypes.js
100% <ø> (ø)
:arrow_up:

src/packages/recompose/flydObservableConfig.js
73.33% <0%> (ø)
:arrow_up:

src/packages/recompose/renderComponent.js
100% <100%> (ø)
:arrow_up:

src/packages/recompose/onlyUpdateForKeys.js
100% <100%> (ø)
:arrow_up:

src/packages/recompose/componentFromStream.js
100% <100%> (ø)
:arrow_up:

src/packages/recompose/createSink.js
100% <100%> (ø)
:arrow_up:

... and 25 more

Continue to review full report at Codecov.

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


Add MMRL token and Logo

Dear Author ✍ Kindly advise if logo has been aded🤔Regards,MariusOn 10 Mar 2023, at 11:06, MRabenda @.***> wrote:
Closed #895.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Expose actual docs instead of "Delegated .... from ..." in delegate_rng_trait macro.

"Delegated ... from ..." does not tell me anything. It requires me to open the docs of turborand in the browser just to see what a simple method is doing.
This is bad UX.

Frankly, the entire Delegated thing is bad DX, but one that arises from the mismatch of turborand being mostly &self methods and ECS access patterns for threadsafety and determinism favouring &mut self methods, given that on its own, turborand is not threadsafe.
Likewise, it has required doing the repetitious macros to get around this, but I have yet to figure out how to re-export the doc comments from the original type/method to the delegated one. Otherwise, I'd have to copy-paste manually all of the docs from one crate to the other, or go write some proc macros which I'm not really motivated to learn for this crate alone.
Stuff like this is why I wrote bevy_rand, which is just the better API. Just use the wyrand feature if you need the same PRNG algorithm as what turborand is using.


Requesting: Confirmation modal wording

What

The second modal that appears when confirming a request states that the rare materials room is on the second floor, it is actually on " Level 2"
Where

Confirmation modal on requesting journey

Modal text update below:
It will be available to pick up from the library (Rare Materials Room, level 3) for two weeks.
Before your visit: you will need to book a 'Rare Materials Room – all day entry ticket' by 10am the day before you wish to visit.


feat: add quiet flag

This PR is related to #10
Example
node lint.js '**/tekton-beta-mixed.yaml' --quiet

Result
regression-tests/tekton-beta-mixed.yaml:
error   (17,7,21,1): Pipeline 'tekton-alpha-mixed-pipeline' references task 'my-task', but parameter 'my-param' is not supplied (it's a required param in 'my-task')
error   (38,7,43,1): Pipeline 'tekton-beta-mixed-pipeline' references task 'my-task', but parameter 'my-param' is not supplied (it's a required param in 'my-task')

Before (without flag)
error   (17,7,21,1): Pipeline 'tekton-alpha-mixed-pipeline' references task 'my-task', but parameter 'my-param' is not supplied (it's a required param in 'my-task')
error   (38,7,43,1): Pipeline 'tekton-beta-mixed-pipeline' references task 'my-task', but parameter 'my-param' is not supplied (it's a required param in 'my-task')
warning (8,7,9,1): Task 'tekton-alpha-mixed' defines parameter 'my-param', but it's not used anywhere in the spec
warning (29,9,30,1): Task 'tekton-beta-mixed' defines parameter 'my-param', but it's not used anywhere in the spec
warning (20,11,21,1): Pipeline 'tekton-alpha-mixed-pipeline' is defined with apiVersion tekton.dev/v1alpha1, but defines an inlined task (my-task) with spec.params. Use spec.inputs.params instead.
warning (42,13,43,1): Pipeline 'tekton-beta-mixed-pipeline' is defined with apiVersion tekton.dev/v1beta1, but defines an inlined task (my-task) with spec.inputs.params. Use spec.params instead.
warning (8,5,9,3): Task 'tekton-alpha-mixed' is defined with apiVersion tekton.dev/v1alpha1, but defines spec.params. Use spec.inputs.params instead.
warning (29,7,30,3): Task 'tekton-beta-mixed' is defined with apiVersion tekton.dev/v1beta1, but defined spec.inputs.params. Use spec.params instead.
warning (2,13,2,32): Task 'tekton-alpha-mixed' is defined with apiVersion tekton.dev/v1alpha1, consider migrating to tekton.dev/v1beta1
warning (11,13,11,32): Pipeline 'tekton-alpha-mixed-pipeline' is defined with apiVersion tekton.dev/v1alpha1, consider migrating to tekton.dev/v1beta1

Thanks! 🎉


ciao-controller: check error on tenant out of bounds test.

If we don't add the tenant limit correctly, do not proceed
with the test.
Signed-off-by: Kristen Carlson Accardi<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 66.408% when pulling 871e45374928cce359ff84919fee8503229ba0d0 on kaccardi:topic/fix-tenant-out-of-bounds into 755a264544d0f7e1def6ce38613aa69288718548 on 01org:master.

Coverage decreased (-0.02%) to 66.408% when pulling 871e45374928cce359ff84919fee8503229ba0d0 on kaccardi:topic/fix-tenant-out-of-bounds into 755a264544d0f7e1def6ce38613aa69288718548 on 01org:master.


Horizontal Bar chart ?

hi, I just check on this issue:
https://github.com/chartjs/Chart.js/issues/73
And seem react-chartjs doesn't work with options = { invertXY: true }  as chartjs:
http://jsfiddle.net/xcchcaptain/ye2wdou0/4/
Maybe this option has not implemented yet. Please investigate.

please check Chart.js V 1.1.1 docs if that option is available. I think there are plugins that you can incorporate into Chart.js to achieve this though (https://github.com/chartjs/Chart.js/tree/v1.1.1/docs).
Either way, you may be using a later version of Chart.js in your fiddle.
You can always upgrade to Chart.js V2, there is a specific type:
var myBarChart = new Chart(ctx, {
    type: 'horizontalBar',
    data: data,
    options: options
});

Thank you, I changed to react-chartjs-2 package, it worked like charm!


Workaround for get-environment-information not providing class data f…

…or each entry.
The get-environment-information RPC output corresponds to the "show chassis environment" CLI command.  The output from this command in a multi-chassis EX switch environment is as follows:
> show chassis environment
Class Item                           Status     Measurement
Power FPC 0 Power Supply 0           OK
      FPC 1 Power Supply 0           OK
      FPC 2 Power Supply 0           OK
Temp  FPC 0 CPU Sensor               OK         37 degrees C / 98 degrees F
      FPC 0 PSU Sensor               OK         30 degrees C / 86 degrees F
      FPC 1 CPU Sensor               OK         38 degrees C / 100 degrees F
      FPC 1 PSU Sensor               OK         30 degrees C / 86 degrees F
      FPC 2 CPU Sensor               OK         44 degrees C / 111 degrees F
      FPC 2 PSU Sensor               OK         35 degrees C / 95 degrees F
Fans  FPC 0 Fan Tray 0               OK         Spinning at normal speed
      FPC 0 Fan Tray 1               OK         Spinning at normal speed
      FPC 1 Fan Tray 0               OK         Spinning at normal speed
      FPC 1 Fan Tray 1               OK         Spinning at normal speed
      FPC 2 Fan Tray 0               OK         Spinning at normal speed

That's all well and good on the command line, but the RPC output seems to correspond to the CLI output a bit too faithfully.  PrettyPrinting from within the napalm-junos module reveals the structure:
[   (   'FPC 0 Power Supply 0',
        [('status', 'OK'), ('class', 'Power'), ('temperature', None)]),
    (   'FPC 1 Power Supply 0',
        [('status', 'OK'), ('class', None), ('temperature', None)]),
    (   'FPC 2 Power Supply 0',
        [('status', 'OK'), ('class', None), ('temperature', None)]),
    (   'FPC 0 CPU Sensor',
        [('status', 'OK'), ('class', 'Temp'), ('temperature', 37)]),
    (   'FPC 0 PSU Sensor',
        [('status', 'OK'), ('class', 'Temp'), ('temperature', 29)]),
    (   'FPC 1 CPU Sensor',
        [('status', 'OK'), ('class', None), ('temperature', 38)]),
    (   'FPC 1 PSU Sensor',
        [('status', 'OK'), ('class', None), ('temperature', 30)]),
    (   'FPC 2 CPU Sensor',
        [('status', 'OK'), ('class', None), ('temperature', 43)]),
    (   'FPC 2 PSU Sensor',
        [('status', 'OK'), ('class', None), ('temperature', 35)]),
    (   'FPC 0 Fan Tray 0',
        [('status', 'OK'), ('class', 'Fans'), ('temperature', None)]),
    (   'FPC 0 Fan Tray 1',
        [('status', 'OK'), ('class', None), ('temperature', None)]),
    (   'FPC 1 Fan Tray 0',
        [('status', 'OK'), ('class', None), ('temperature', None)]),
    (   'FPC 1 Fan Tray 1',
        [('status', 'OK'), ('class', None), ('temperature', None)]),
    (   'FPC 2 Fan Tray 0',
        [('status', 'OK'), ('class', None), ('temperature', None)])]

Note that each power supply, temperature sensor, or fan after those in the first chassis are given a 'class' of None, likely taken straight from the human-readable output of the CLI command.
This patch modifies the behavior of the main loop within get_environment() so that if a sensor object is encountered with a 'class' of None, the previous valid class is used.
This works on all models of EX switch that I have access to test against.  Testing against other JunOS platforms is likely warranted.

Coverage decreased (-0.06%) to 83.556% when pulling bb5ab5dae35df1d1139473b0495194c48d42d032 on bkeifer:develop into e6519c6dcde1e502db4b4ddb1d329ece05de7aea on napalm-automation:develop.

Would you mind adding tests? Just add an extra folder here with the relevant mocked data:
https://github.com/napalm-automation/napalm-junos/tree/develop/test/unit/mocked_data/test_get_environment

Getting sucked into a meeting.  Will take care of the build errors later.

Getting sucked into a meeting.

Have fun @bkeifer

Coverage decreased (-4.7%) to 78.958% when pulling 6d5b4641d9848b38690ab5bdfade49f79bdf60ba on bkeifer:develop into e6519c6dcde1e502db4b4ddb1d329ece05de7aea on napalm-automation:develop.

I'm back to the original conditional logic.  Here's why:
This line, suggested by @dbarrosop will update the object with the correct class name.
structured_object_data['class'] = structured_object_data['class'] or current_class
However, since Juniper only gives us a 'class' for the first instance of an object of that class, we also need to track when we see a new class name to let us know that the following data is about a different class of object.  That's where the else: comes in.
I've added some comments to hopefully explain whats going on a bit better, and Travis seems happier now.


Minimal theme settings override some other themes (light mode background contrast high contast)

Before you open a bug report
Disable plugins and snippets that could be causing a conflict. Update to the latest versions of the following, and restart Obsidian:

Minimal Theme 6.3.11
Obsidian 1.3.5
Minimal Theme Settings plugin 6.3.1
Contextual Typography plugin No

Confirm that the bug is not present when Minimal theme is off
Yes
Describe the bug
A clear and concise description of the bug
Obsidian minimal setting are override other setting when I am using other themes, for example Things
To reproduce
Provide steps to reproduce the behavior

Install obsidian minimal settings
Install Minimal theme
set light mode background contrast to high contrast
Change theme to other theme (fot example things)
The sidebar has black/dark accents
this black accent is active even after deactivating minimal setting, only restart of Obsidian helps

Expected behavior
A clear and concise description of what you expected to happen
the  minimal settings will be not active when I change the theme (vanilla obsidian doesnt have this problem)
Screenshots
If applicable, add screenshots to help explain your problem

Debug info (required):
Paste your debug info below, which can be accessed in Obsidian with Ctrl/Cmd-P, enter "Show debug info", and click "Copy to clipboard"

Is is also possible to set the high contrast when the theme set is not minimal, which gives weird results sometimes

Minimal Theme Settings is intended to be used with Minimal so this can be solved by disabling the plugin.

@kepano disabling the plugin doesn't work, only restart of Obsidian remove the active "high contrast" css effect which might be considered as bug

Minimal Theme Settings is intended to be used with Minimal so this can be solved by disabling the plugin.

This is true enough in principle, but in practice many people will have Minimal Theme Settings installed when they try other themes. If they have High Contrast set, then other themes will render incorrectly, and it's not obvious to a typical user that Minimal Theme Settings would affect some other theme entirely. This isn't an abstract concern: I've personally just spent several days going back and forth with a user to figure out why my theme was rendering poorly for them. I was lucky to get a patient and understanding user who was willing to post screenshots and debug info to a forum thread, or I'd never have known!
In the short term, I've patched my own theme to override the Minimal Theme Settings behaviour by replacing
.theme-light {
 ...
}
```css
with
```css
.theme-light,
.theme-light .theme-dark {
  ...
}

which looks weird, but gets the job done. However, my theme happens to be in active development right now, whereas I suspect there are many other affected themes that don't get updates as frequently, and whose authors may not be aware of the problem.
Longer term, could I ask that Minimal Theme Settings be adjusted so that the High Contrast setting doesn't break other themes if users happen to leave it on? My suggestion would be that instead of applying .theme-dark to the ribbon and sidebar, it could use a new class name specific to Minimal (something like .minimal-subtheme-dark, say). Then have the Minimal CSS apply the same styles to .theme-dark and .minimal-subtheme-dark (similarly to what I've done). Perhaps you can see a better approach, but I hope that's useful.

Fixed in 8.0.0

With 8.0.0 I can reproduce the behaviour: I set light mode background contrast to High contrast, and the sidebar stays in dark mode when I switch to other themes, both with the Minimal Settings Plugin   turned off and turned on

Oops, good catch. Try 8.0.1

Zipped source code doesn't seem to be updated from 8.0.0, Obsidian  doesnt see new version

You should see it now

yes, it works now!


Cannot disable dragging because dialog behavior is private.

i wanted to disable the dialog dragging but i am not able to do that, i searched about this and get a solution where i have to access   behavior from dialog but behavior is a private variable in the SuperBottomSheetFragment.

This work for me. Override superBottomSheetDialog theme in your own app module
<style name="superBottomSheetDialog" parent="Theme.Design.Light.BottomSheetDialog">
        <item name="android:windowIsFloating">false</item>
        <item name="behavior_draggable">false</item>
    </style>


Run Install Docker for Windows Server script error

I follow Install Docker for Windows-Server step but show an error message:
PS C:\> Install-Module -Name DockerMsftProvider -Repository PSGallery -Force                                                                                NuGet provider is required to continue
PowerShellGet requires NuGet provider version '<IP_ADDRESS>' or newer to interact with NuGet-based repositories. The NuGet
 provider must be available in 'C:\Program Files\PackageManagement\ProviderAssemblies' or
'C:\Users\BruceChen_adm_bt\AppData\Local\PackageManagement\ProviderAssemblies'. You can also install the NuGet provider
 by running 'Install-PackageProvider -Name NuGet -MinimumVersion <IP_ADDRESS> -Force'. Do you want PowerShellGet to
install and import the NuGet provider now?
[Y] Yes  [N] No  [S] Suspend  [?] Help (default is "Y"): y

PS C:\> Install-Package -Name docker -ProviderName DockerMsftProvider
WARNING: Cannot find path
'C:\Users\BRUCEC~1\AppData\Local\Temp\2\DockerMsftProvider\DockerDefault_DockerSearchIndex.json' because it does not
exist.
WARNING: Cannot bind argument to parameter 'downloadURL' because it is an empty string.
WARNING: The property 'AbsoluteUri' cannot be found on this object. Verify that the property exists.
WARNING: The property 'RequestMessage' cannot be found on this object. Verify that the property exists.
Install-Package : No match was found for the specified search criteria and package name 'docker'. Try
Get-PackageSource to see all available registered package sources.
At line:1 char:1
+ Install-Package -Name docker -ProviderName DockerMsftProvider
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package], Ex
   ception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

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

ID: e09e1b22-1344-d7b9-9462-dc4c2e160e86
Version Independent ID: f6411e97-4419-1209-56ad-9140ceb37329
Content: Prep Windows operating system containers
Content Source: virtualization/windowscontainers/quick-start/set-up-environment.md
Product: windows-server
Technology: windows-containers
GitHub Login: @cwilhit
Microsoft Alias: crwilhit

Hey @kkbruce,
Can you state the successfulness and the outcome of the command Find-Package -ProviderName DockerMsftProvider?

@ashenm
test result:
PS C:\>Find-Package -ProviderName DockerMsftProvider
Find-Package : Unable to find package providers (DockerMsftProvider).
At line:1 char:1
+ Find-Package -ProviderName DockerMsftProvider
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (Microsoft.Power...ets.FindPackage:FindPackage) [Find-Package], Exception
    + FullyQualifiedErrorId : UnknownProviders,Microsoft.PowerShell.PackageManagement.Cmdlets.FindPackage

I find that the official documents have been changed here:  https://docs.mirantis.com/docker-enterprise/v3.1/dockeree-products/docker-engine-enterprise/dee-windows.html
Follow the steps above to install correctly.

Hey @kkbruce,
I'm writing on the assumption you are still having an broken installation following the Microsoft Docs despite the aforementioned Mirantis installation.

Import-Module -Name DockerMsftProvider -Force
Import-Packageprovider -Name DockerMsftProvider -Force
Install-Package -Name docker -ProviderName DockerMsftProvider -Verbose

It would be a great aid if you could attempt the invoke of above commands in exact order and update the outcome.

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


v6: Remove unnecessary Aliasses. Closes #3644

Remove unnecessary Aliasses. Closes #3644
This PR removes some unnecessary Aliasses. It also removes a few leftover references to those aliasses from some samples and some docs pages.
Some titles needed to be updated as well, to align with the current command names

I think it is ready to be merged. Will do it asap 👍
thanks @martinlingstuyl for you awesome work 🤩


Fix potential infinite loop when using global menu

It's always checking the same iterator 🤣
I think it's a copy paste error.

Thanks!


The mp3 link in demo fails with MediaError 4 => MEDIA_ERR_SRC_NOT_SUPPORTED

The link for mp3 seems to be wrong.
http://www.sample-videos.com/audio/mp3/crowd-cheering.mp3
redirects to
http://play.leadzupc.com/?m=URWWBANDOO2=930&a=9007&afc_count=1

Fixed in 1.1.3 👍

Thanks


SAM nodes partially broken

SAM nodes are partially broken
🔴 downloads in the wrong folder
for vit_b, sam loader node seems to be looking in ComfyUI\ComfyUI\models\sam instead of ComfyUI\models\sams where it downloads it
🔴 tensor size mismatch for vit_b and vit_l
if I fix paths manually for the Vit-B, I get things like
size mismatch for image_encoder.blocks.10.mlp.lin2.bias: copying a param with shape torch.Size([768]) from checkpoint, the shape in current model is torch.Size([1280]).
       size mismatch for image_encoder.blocks.11.norm1.weight: copying a param with shape torch.Size([768]) from checkpoint, the shape in current model is torch.Size([1280]).
       size mismatch for image_encoder.blocks.11.norm1.bias: copying a param with shape torch.Size([768]) from checkpoint, the shape in current model is torch.Size([1280]).
       size mismatch for image_encoder.blocks.11.attn.rel_pos_h: copying a param with shape torch.Size([127, 64]) from checkpoint, the shape in current model is torch.Size([27, 80]).

same with vit_l
🟢 got vit_h working with manual path fixing

I fixed the path issues, but I am not sure about the tensor mismatch. @moorehousew do you know what this is about? I think it's the tensor to sam compatible tensor conversion, but also not sure why their models are working on different tensors for different models. Seems odd from a usage standpoint.

cause in build_sam.py
build_sam = build_sam_vit_h

It always returns vit_h, that's why only vit_h is working.
U can add some if ...else... and make model_size to control which one should be returned.


Time range selector

Thanks for the tool!
On the time range selector can a url parameter be used to set the initial value?
Something like:
https://analytics.example.com/share/EUf5qmgL/example?time=last_7_days
To choose the Last 7 Days.

I'm not sure how that can be handled in the framework though.

Yea I think we can support query parameters.


[JS] Streaming output always null

Describe the bug
I'm unable to use the streaming functionality with structured output when running in a server environment (testing locally), as utilizing chunk.output() does not work. Note that we are using GenKit version 0.5.17, not the latest 0.9.* updates as we have not had a chance to update to that yet.
To Reproduce
I'm using the example provided in the 0.5.17 docs with an extra console.log to show that data is still streaming and that it's just the output that's not rendering: https://github.com/firebase/genkit/blob/genkit%400.5.17/docs/models.md#streaming-streaming
const MenuItemSchema = z.object({
    name: z.string(),
    description: z.string(),
    calories: z.number(),
  });

  const MenuSchema = z.object({
    starters: z.array(MenuItemSchema),
    mains: z.array(MenuItemSchema),
    desserts: z.array(MenuItemSchema),
  });

  type Menu = z.infer<typeof MenuSchema>;

  const llmResponseStream = await generateStream({
    prompt: "Suggest a complete menu for a pirate themed restaurant",
    model: gemini15Pro,
    output: { schema: MenuSchema },
  });

  for await (const responseChunkData of llmResponseStream.stream()) {
    const responseChunk = responseChunkData as GenerateResponseChunk<Menu>;
    // output() returns an object representing the entire output so far
    const data = responseChunk.data();
    const output: Menu | null = responseChunk.output();

    console.log("data", data);
    console.log("output", output);
  }

This is the console output:
>  data {}
>  output null
>  data [ { name: "Pirate's Booty" } ]
>  output null
>  data undefined
>  output null
>  data {
>    name: "Mermaid's Treasure Chest",
>    description: 'Assortment of hummus, olives, feta cheese, and pita'
>  }
>  output null
>  data {
>    name: "Sailor's Delight",
>    description: 'Creamy tomato soup served in a bread bowl'
>  }
>  output null
>  data [
>    {
>      name: "Blackbeard's Bounty",
>      description: 'Grilled steak served with roasted vegetables and mashed potatoes',
>      calories: 850
>    },
>    {}
>  ]
>  output null
>  data { name: "Captain's Curry", description: 'Aromatic' }
>  output null
>  data {
>    name: "The Kraken's Catch",
>    description: 'Linguine pasta with shrimp, mussels, clams, and calamari in a white wine sauce',
>    calories: 720
>  }
>  output null
>  data [
>    {
>      name: 'Rum Soaked Treasure',
>      description: 'Spiced rum cake with vanilla bean ice cream',
>      calories: 450
>    },
>    {
>      name: 'Caramel Apple Pie Doubloons',
>      description: 'Mini apple pies with a buttery caramel sauce'
>    }
>  ]
>  output null
>  data {
>    name: 'Walk the Plank',
>    description: 'Chocolate lava cake with raspberry coulis and fresh berries',
>    calories: 420
>  }
>  output null

As you can see, chunk.output() is always null, while chunk.data() has expected results. Am I configuring something wrong? I've checked this code against the
Expected behavior
Using chunk.output() should provide the full structured output up until the point of the chunk being rendered.
Runtime (please complete the following information):

OS: MacOS
Version: 14.6.1

** Node version
v18.17.1 (tested on v20.14.0 as well)

We went ahead and spent the several hours required to upgrade to 0.9.* but found that there are no examples of how to use streaming with dotprompt. We did our best to come up with how you're supposed to do this, and this was our best guess:
const menuGenerator = ai.prompt<
  typeof GenerateMenuInputSchema,
  typeof GenerateMenuOutputSchema
>("generateMenu");

export const exampleFlow = ai.defineStreamingFlow(
  {
    name: "exampleFlow",
    inputSchema: ExampleInputSchema,
    streamSchema: ExampleOutputSchema,
  },
  async (input, streamingCallback) => {
    await menuGenerator(
      input
      {
        streamingCallback: async chunk => {
          console.log("chunk", chunk);

          const output = chunk.output as z.infer<
            typeof GenerateMenuOutputSchema
          >;

          console.log("chunk.output", output);
          console.log("chunk.data", chunk.data);

          await streamingCallback?.(output);
        },
      },
    );

    return response;
  },
);

When we do this, we get none of those logs from the streamingCallback. We do get a final output but we don't get any of the streaming chunks.
We also tried just following the updated example in the docs, but we found that when it gets to the async iterable, it just completely hangs.
Any advice on what we are doing wrong would be appreciated.

Alright I finally figured this out. Even though the result of .prompt is a callable function, you can also access a property on it called stream like so:
const menuGenerator = ai.prompt<
  typeof GenerateMenuInputSchema,
  typeof GenerateMenuOutputSchema
>("generateMenu");

const res = menuGenerator.stream(...)

for await (const chunk of res.stream) {
  console.log("chunk.output", chunk.output);
}


Support _skip parameter for pagination

Paging with FHIR server currently uses a continuation token which is not meant to be directly manipulated by the client. This limits traditional paging GUIs other than an infinite scrolling list.
Ideally it should look something like below:
first page:  Patient
second page: Patient?_skip=10
third page:  Patient?_skip=20

I created a POC to enable this for the SQL Server provider. Please have a look.
https://github.com/almostchristian/fhir-server/tree/fhir/paging
The change involves converting the usages of TOP into OFFSET
  SELECT DISTINCT T1, Sid1, 1 AS IsMatch, 0 AS IsPartial
  FROM cte1
  ORDER BY Sid1 ASC
  OFFSET (@p4) ROWS
  FETCH NEXT @p5 ROWS ONLY

A lot of change had to be done to support sorting, which is still tricky. Since sorting can happen in two phases, the skip parameter in the POC resets when the second phase starts. I denote the second phase with a hyphen in the skip parameter,
The search urls when sorting is involved can look like below. In here, repurposed ct as the skip parameter.
1st page: Patient?_sort=family&_count=2
2nd page: Patient?_sort=family&_count=2&ct=2
3rd page: Patient?_sort=family&_count=2&ct=-1
4th page: Patient?_sort=family&_count=2&ct=-3

Since I am currently working with a fork of FHIR server, I plan to go introduce this to our implementation. The sorting issue is a bummer, but I don't think this can be avoided without a big redesign. I'd like to hear feedback on this implementation. The E2E integration tests all pass, but I'm not sure if there is some edge cases not covered in the integration tests that could break this.
In terms of performance, the generated query plan is the same with the original, but the OFFSET version seems to be heavier compared to the TOP version outside of the first page, but I'd need to do proper testing on a large db.
AB#105830

@almostchristian - Apologizes on delayed response. We would be interested to review.

@EXPEkesheth I created a draft PR here #3536

Thanks @almostchristian - have scheduled it in our queue for review. We will respond back once we have reviewed or have additional questions.

@almostchristian
Thanks for the time taken to submit the review. We reviewed the change and unfortunately, we won't be able to accept the change in the current form. The proposed change is not aligned with MSFT FHIR server approach with using snapshots for continuation token. The proposed change introduces a new behavior with implementing separate parameter from the continuation token. Please update the PR to align with MSFT FHIR server approach.


Make BucketOwner DisplayName optional

According to: https://docs.aws.amazon.com/AmazonS3/latest/API/API_Owner.html#API_Owner_Contents, only certain regions support DisplayName for the bucket. This causes problem for my bucket (eu-north-1) once I use bucket.exists(), it throws the following Error:
S3(SerdeXml(Custom("missing field DisplayName")))
Thus, it should be Option<String> to support all regions.

@erdoganege good, I've fixed tests up and its good to go


fix upload file by s3

[Current coverage][cc-pull] is 65.10%

Merging [#314][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.03%

@@             master       #314   diff @@
==========================================
  Files            28         28
  Lines          2920       2920
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1902       1901     -1
- Misses         1018       1019     +1
  Partials          0          0

Powered by Codecov. Last updated by [87458aa...df57e86][cc-compare]
[cc-base-branch]: https://codecov.io/gh/leancloud/javascript-sdk/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/leancloud/javascript-sdk/compare/87458aa2fddc29f86abd31a8a239c96eba6529f5...df57e8661f943cb644ee9147af11afa8dec26b44
[cc-pull]: https://codecov.io/gh/leancloud/javascript-sdk/pull/314?src=pr


[Ysystem] First commit to Ysystem

Changes / Comments
This an attempt to submit a blank sheet for the (new) Ysystem system
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Not sure how to fix that conflict with the .DS_Store file. Apparently I removed something that wasn't there so probably i made git to go nuts. Should I try to create a new branch and make a fresh pull request from there or there is something you can easily do from there?

Yeah, that might be the best way to go. I was going to check out the conflicts in Github's editor, but it says they are too complex for the web editor. @jorsermar


Error in the android manifest [Godot 3.2.3 rc2]

Error when trying to build for android
* What went wrong:
Execution failed for task ':processDebugManifest'.
> Manifest merger failed : Attribute application@allowBackup value=(false) from AndroidManifest.xml:30:68-95
  	is also present at [com.gameanalytics.sdk:gameanalytics-android:5.2.4] AndroidManifest.xml:14:18-44 value=(true).
  	Suggestion: add 'tools:replace="android:allowBackup"' to <application> element at AndroidManifest.xml:30:5-68:19 to override.

I should add I'm using this in conjunction with AdMob from Shinnil.
Solution
add:
tools:replace="android:allowBackup"

to the <application> element but I'm not sure whether that's the desired solution and if so, it might be worth adding to documentation.
Also, I'm excited about the official plugin from GA, it's absolutely fantastic.

Ok thanks for reporting this. I have updated the GA native android lib used by the plugin and I have removed the android:allowBackup value inside the manifest file their since the value by default is set to true anyway so now there shouldn't be any conflicts when building for Android after updating to the latest version of the plugin (v1.0.1)

Fantastic, thanks!


Directly open the result in a browser

This pull request adds an option cce compile --open ... to directly open the result in a browser.
Please note that I am not a Rust developer, so please ensure that I haven't made any mistakes in this regard.

Thank you again!


update heading tags to match cockroachlabs.com

Will be part of DivisionOf's style updates, so closing here.


allow configuring panel position

The default position of the panel was obstructing a menu so I made this modification to allow the panel to be moved to a more convenient location. For example:
<DevTools position={{bottom:0, right:0}} />

That's useful indeed, thanks!

Released as part of 4.0.1


vscode is not recognized by the gnome-software as an application

Neither the vscode repository nor the package itself provide any appdata, which is important to have gnome-software working properly.
The vscode repository should generate appstream data (xml) and provide them.
So the PackageKit can save it into /var/cache/app-info/xmls/.
The code.rpm itself also should install a com.microsoft.code.metainfo.xml into /usr/share/metainfo/.
Look here:
https://www.freedesktop.org/software/appstream/docs/

I believe we include code.appdata.xml in the root of the package, is that not the case?
https://github.com/microsoft/vscode/blob/554029f6e9e08dad385bf31ec41956b53d34598f/resources/linux/code.appdata.xml

Unfortunately, it is not the case. There's no /usr/share/metainfo/com.microsoft.code.metainfo.xml or /usr/share/appdata/com.microsoft.code.appdata.xml file.
Output from rpm -qlp code-1.65.2-1646927812.el7.x86_64.rpm: 12c4703f.txt
The preferred place for that appdata file is /usr/share/metainfo/com.microsoft.code.metainfo.xml
Then the repository creation should also include generating AppStream repository metadata. The mechanics of doing this is documented here: https://blogs.gnome.org/hughsie/2016/04/27/3rd-party-fedora-repositories-and-appstream/
(Ignore the --simple-md-filenames argument, that should not be used)

Hey @Tyriar, this issue might need further attention.
@cooperbang, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

@Tyriar This should be reopened, because it's still broken.

Confirmed that the deb package is good but that the rpm package has this issue.

@ronakay22 see https://github.blog/changelog/2022-01-12-unwatch-all-repositories-owned-by-a-given-user-or-organization/.

Verification steps:

Install the latest Insiders on a Linux distribution with rpm
Run ls /usr/share/appdata | grep insiders in a terminal
Confirm that code-insiders.appdata.xml shows up.


Fixed #24291 - Fixed migrations with unused swappable models

Changed ModelState from_model to use an empty list if the model was an unused
swappable model.

Flake8 doesn't fail on my local system. Could someone give me any pointers on how to fix it?

Here you go:
./tests/migrations/test_state.py:568:9: E303 too many blank lines (2)

I just double-checked and PEP8 allows single lines to separate blocks. Why did flake8 mark that single blank line as an error? Anyway, it works now.

buildbot, test this please.

buildbot, test this please.

Could you also make the use of quotes consistent (single quote everywhere should be fine, I think)

db/migrations/state.py seems to use double quotes almost everywhere, so I used them there as well. tests/migrations/test_state.py was more 50/50 so I switched to consistently using single quotes in the new test.

buildbot, test this please.

With this patch Django creates a migrations for auth that drops the managers from User and Permission.

buildbot, test this please

buildbot, test this please

You can squash commits, thanks.

buildbot, test this please.

Test failure due to failed checkout

Merged in 15dc8d1c9d3697170a2c59ecaa7a2b4ba58f5990. Thanks.

I slightly changed the test cases to not use an entire ProjectState but just ModelState.from_model() and thus moved them into ModelStateTests.


Adds allergy/disease sensor platform from Pollen.com

Description:
Adds a sensor platform for allergy and disease information from Pollen.com.
Related issue (if applicable): N/A
Pull request in home-assistant.github.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/4392
Example entry for configuration.yaml (if applicable):
sensor:
  platform: pollen
  zip_code: 80919
  monitored_conditions:
    - allergy_average_forecasted
    - allergy_average_historical
    - allergy_index_today
    - allergy_index_tomorrow
    - allergy_index_yesterday
    - disease_average_forecasted

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

I'm getting data from
    monitored_conditions:
      - allergy_index_today
      - allergy_index_tomorrow

but not from
      - allergy_average_forecasted

Anyone else experiencing this?

@grantalewis I'm getting data from all three. Could you share your entire config and I'll help  you debug?

@grantalewis Please open a new bug report and we'll be able to help debug.


(MCO-751) Manage PID files to avoid race conditions on process start

The daemonize_runner was not using an exclusive lock writing it's pid
file so you could get into a race condition with SysVInit and by running
service mcollective start & service mcollective start. This also adds
checking to make sure stale PID files get cleaned up.

This seems to undermine existing expectations in the unit tests :(

Working on test updates and some new tests now.

:+1: pending travis, this makes a lot of sense to me.

Looks OK, but it's not quite correct to call this an exclusive lock in the commit message.  This technique is an exclusive create.

Hey @richardc , I've addressed your comments/nits and updated the commit message, so let me know how this looks!

@richardc ok, updated again!

@underscorgan cool, thanks.


Issue#57

Feature: Sending Cookies and Asserting Cookies from Response
Library Added: lightweight
Files Changed:

Spec.d.ts.......................added declaration for withCookies(), expectCookies() and expectStrictCookies()
Spec.js...........................added implementation for withCookies(), expectCookies() and expectStrictCookies()
expect.js........................added _validateCookies() and _validateStrictCookies for cookies assertion
cookies.spec.js.............added tests related to cookies

NOTE: Changes in below also required

cookie implementation within mock server
documentation

For user-facing apps it's typical to keep authentication tokens hidden inside the application and issue an "opaque" cookie to represent the session to the user agent upon successful sign-in. In this case we'd like to verify that the Set-Cookie is as expected, which we can do with this PR, but also have a built-in way to store the cookie(s) and automatically send the right ones on subsequent requests, as a cookie-supporting HTTP user-agent would do.
Would be willing to work on a PR for this if you can provide a bit of guidance on how to integrate.


Add initial authorization

Add support for authorization Bearer token
Add argument for JWT public key
Validate JWT access token when authorization enabled

@argerus can this be closed, because the functional changes had been included in #477  already?

@argerus can this be closed, because the functional changes had been included in #477 already?

Yes


moved main component from div to button to work with mobile

I tried this out and it did not work on mobile Safari for me.

Try this one - I noticed the same thing the other day.  I changed it back to a div but enabled TouchEvents and used the onStartTouch to trigger it.  Appears to be working

I just realised - this library has no use on a mobile as there is no concept of drag and drop of files on mobile. To support file inputs, one can simply use a normal input tag. For this reason, I'll be closing this PR.


[release/8.0] Add OTEL env var to not redact HttpClient query string values

Customer Impact
Previous PR - https://github.com/dotnet/aspire/pull/3681 - updated OTEL packages to redact query strings. It includes env var to not redact query strings in local development for incoming HTTP requests.
However, there is a different env var for outgoing HTTP requests. Without the env var, outgoing HTTP requests see redacted query string values in local dev.
This PR:

Adds OTEL_DOTNET_EXPERIMENTAL_HTTPCLIENT_DISABLE_URL_QUERY_REDACTION env var
Moves .NET specific env vars to project resource so they're not provided to containers

Testing
Manual and unit testing.
Risk
Low
Regression?
Yes. Restore redaction in local dev that OTEL enabled.
Microsoft Reviewers: Open in CodeFlow

Was this change made in main? If not, what is the plan for getting it there?

Same question as @eerhardt - if it needs to be in main and you're just doing this first, might want to just make both PR's at the same time, so we don't forget main..

Main is here: https://github.com/dotnet/aspire/pull/3797


Allow for larger scale submissions in Inference when moving from Preview to Available

A number of Preview systems in MLPerf Inference v4.0 used fewer cards than would be typical in production due to a limited availability of cards at the time. Rather than benchmarking the systems with exactly the same, atypical number of cards as in Preview, it would be desirable to benchmark them in a more typical configuration, with a higher number of cards. Of course, for Available submissions the performance per accelerator would still need to be demonstrated to be equal or better than in Preview submissions.
We have a similar provision in the submission policies, but at the moment it only covers Training:

On each of the benchmarks that are previewed and are Compatible, the Available submission must show equal or better performance (allowing for noise, for any changes to the benchmark definition) on all systems for Inference and across at least the smallest and the largest scale of the systems used for Preview submission on that benchmark for Training (e.g. Available Training submissions can be on scales smaller than the smallest and larger than the largest scale used for Preview submission).

@psyhtest Previously we had similar discussions and there were issues raised on this (on a different context) when a single model is split across multiple GPUs and hence the performance per accelerator might be better on a larger scale system. So, a rule change for this might be tricky - but may be the WG can agree on the "similarity" of the available and preview systems?

when a single model is split across multiple GPUs and hence the performance per accelerator might be better on a larger scale system

I agree how Offline may be affected, but could Server latency constraints counterbalance that?

The same issue can happen for Server scenario too right? But if the model is not split across multiple GPUs may be we can do a rule proposal.

If the preview submission was on say 4 accelerators and available submission is on 6 accelerators and as well as say 2 accelerators, and in both cases if the performance per accelerator is greater than that of the preview system, then I think the 4 accelerator submission may not be needed.
Also, another proposal can be doing just offline scenario (may be even open) for the same number of accelerators if a larger scale system is submitted as available.

Instead of an amendment asking for permissions before submission, it might be worth having a change in the main rules itself to permit an available submission if both these conditions are satisfied:

Number of accelerators used in available is equal or greater than preview system
AND
Scaling of the system performance in available over preview is linear or better given the number of accelerators in both.

Another scenario to consider: having done a Preview submission with an old Available server (e.g. v5) equipped with new Preview accelerators, a submitter may want to do an Available submission with a newer server (e.g. v6) equipped with now Available accelerators.

WG notes:
Not permitted for power
Seek WG pre-approval for any other HW/system changes
@psyhtest to draft PR

@psyhtest @mrmhodak @mrasquinha-g The deadline is close by. Can Inference submitters assume that this rule change applies to v4.1 and thus save some effort in their preview->available submissions? What is the conclusion?

@ashwin I think it is better if the submitter requests and get a waiver from the WG for v4.1.


Adjust dependencies that are explicitly imported

These libraries are often installed by other dependencies, but we don't explicitly list them as dependencies here (warnings originating from conda-forge):

geopandas (version >=0.9.0 is listed, but >=0.13.0 is required)
platformdirs (listed in dev but should be in core deps)
typing_extensions (needed for Python older than 3.10)

@huard We should tackle this small change before the next release.

platformdirs is already in the core deps
geopandas is in the gis deps
typing_extensions is in the core deps


Barebones MEW for Jeff

Clean header / footer / links / stuff

This custom work that needs done for someone?  Let me know, perhaps I can complete

@gamalielhere Do you think this year-old issue can be closed? :D

Yeah. I'm closing it now.


Chrome Debug Extension killing browser

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T15:27:10.246Z)
OS Version: Darwin x64 15.0.0
Extensions: msjsdiag.debugger-for-chrome-1.2.1

Steps to Reproduce:

Launch Chrome against localhost, with sourcemaps
After Chrome browser launches, try opening the developer tools.

This is expected - when the debug adapter connection is kicked out by Chrome, its process is killed, and Chrome is killed too. If you have to switch back and forth I suggest using an 'attach' config.

Thank you for the quick turnaround.


Loopback as API

Proposing that there should be talk about Loopback for creating API.

Thanks @yeomann, that's a great proposal. Could you be more elaborate on the description of the talk? Where would you want to speak?


⚠ Adacta-fintech.com has degraded performance

In 348b64c, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1703 ms

Resolved: Adacta-fintech.com performance has improved in 87ede0f.


Fix Exception Firebird driver doesn't support lastInsertId

When trying to use PDO with firebase, the following exception is raised: SQLSTATE[IM001]: Driver does not support this function: driver does not support lastInsertId()
The Firebird driver doesn't support the lastInsertId function. This function is called by getLastGeneratedValue function in every insert, update and delete and throw an exception.
This fix removes the call to lastInsertId when using firebase, based on PDO’s driverName.

@cgrandval Thanks for your contribution!


Remove trailing colon from filename when adding issues for black tool.

I found this bug when running Statick against errbot. For the Statick configuration I used one from make_movie. But any level that has the black tool enabled will show the bug.
statick errbot --output-directory /tmp/statick-output --user-paths make_movie/statick_config --force-tool-list black

With the master branch there is an error when running black against README.rst. Note the trailing colon on the filename.
Traceback (most recent call last):
  File "/home/thomas/.local/bin/statick", line 46, in <module>
    main()
  File "/home/thomas/.local/bin/statick", line 28, in main
    issues, success = statick.run(path, parsed_args)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/statick.py", line 346, in run
    issues = self.exceptions.filter_issues(package, issues)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/exceptions.py", line 221, in filter_issues
    issues = self.filter_nolint(issues)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/exceptions.py", line 202, in filter_nolint
    lines = open(issue.filename, "r+", encoding="utf-8").readlines()
FileNotFoundError: [Errno 2] No such file or directory: '/home/thomas/src/errbot/README.rst:'

As an aside, this is the first time I learned that the Unix file command says that .rst files are Python files. Perhaps we should check for that extension and add it to a new file type so that documentation linters can scan the files. There is little value in running pylint or black against those files.

I found this bug when running Statick against errbot. For the Statick configuration I used one from make_movie. But any level that has the black tool enabled will show the bug.
statick errbot --output-directory /tmp/statick-output --user-paths make_movie/statick_config --force-tool-list black

With the master branch there is an error when running black against README.rst. Note the trailing colon on the filename.
Traceback (most recent call last):
  File "/home/thomas/.local/bin/statick", line 46, in <module>
    main()
  File "/home/thomas/.local/bin/statick", line 28, in main
    issues, success = statick.run(path, parsed_args)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/statick.py", line 346, in run
    issues = self.exceptions.filter_issues(package, issues)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/exceptions.py", line 221, in filter_issues
    issues = self.filter_nolint(issues)
  File "/home/thomas/.local/lib/python3.8/site-packages/statick_tool/exceptions.py", line 202, in filter_nolint
    lines = open(issue.filename, "r+", encoding="utf-8").readlines()
FileNotFoundError: [Errno 2] No such file or directory: '/home/thomas/src/errbot/README.rst:'

As an aside, this is the first time I learned that the Unix file command says that .rst files are Python files. Perhaps we should check for that extension and add it to a new file type so that documentation linters can scan the files. There is little value in running pylint or black against those files.

@xydesa I agree that the output does not look good. I'm open to suggestions on improvements. One thing we should do for sure is not print anything if the show-tool-output flag is set to False. Perhaps we could make the default output of the tool plugin be "Black encountered internal error while scanning "? Then the full output would show if show-tool-output is set to True. The docformatter output is similarly overwhelming.
The black plugin does detect those internal errors.
https://github.com/sscpac/statick/blob/master/statick_tool/plugins/tool/black_tool_plugin.py#L35
See the notes for the --check flag in black for where the return code of 123 comes from..
https://black.readthedocs.io/en/stable/installation_and_usage.html#command-line-options
During parsing of the errors the plugin differentiates between whether an internal error was found or a file would be reformatted. Both types of issues are reported at the same warning level.

@xydesa I agree that the output does not look good. I'm open to suggestions on improvements. One thing we should do for sure is not print anything if the show-tool-output flag is set to False. Perhaps we could make the default output of the tool plugin be "Black encountered internal error while scanning "? Then the full output would show if show-tool-output is set to True. The docformatter output is similarly overwhelming.
The black plugin does detect those internal errors.
https://github.com/sscpac/statick/blob/master/statick_tool/plugins/tool/black_tool_plugin.py#L35
See the notes for the --check flag in black for where the return code of 123 comes from..
https://black.readthedocs.io/en/stable/installation_and_usage.html#command-line-options
During parsing of the errors the plugin differentiates between whether an internal error was found or a file would be reformatted. Both types of issues are reported at the same warning level.


Allow configuring LiveBuilder durations

https://github.com/flashbots/rbuilder/blob/469f1dc4fd8c8313d77b1283103164738d626ed9/crates/rbuilder/src/live_builder/mod.rs#L40-L46
are hard coded assuming mainnet block times, they need to be adjusted for shorter L2 block time intervals.

This is fixed by https://github.com/flashbots/rbuilder/pull/244


Fix ES5-shim insertion issue

Page should reload or something without inserting the es5-shim if it fails on the first attempt.
https://github.com/sindresorhus/pageres/issues/136.

@kevva Any thoughts on how we can fix this?

Not any good solutions yet, no.

These seems to be fixed by https://github.com/kevva/screenshot-stream/commit/34410dc2e02ced49afb355bbb27069bf34408ffe
pageres http://www.nationalgeographic.com 320x480 1024x768 1366x768
pageres cnn.com 320x480

This one still throws though:
❯ pageres bbc.co.uk 1000x1000
ncat(array_slice.call(arguments))
                );

            }

        };

        // 15. If the [[Class]] internal property of Target is "Function", then
        //     a. Let L be the length property of Target minus the length of A.
        //     b. Set the length own property of F to either 0 or L, whichever is
        //       larger.
        // 16. Else set the length own property of F to 0.

        var boundLength = Math.max(0, target.length - args.length);

Is it related to the es5-shim?

@sindresorhus, should we just read the whole es5-shim file and compare the error output against it? lol.

Sure, better than it not working.


SW-6146 Add French CSV templates

Add first drafts of the French translations of the CSV template files.

I started off by machine-translating the English templates, then manually updated the lists of valid values for fields that map to enums so the enum values match the French enum translations from Phrase. Obviously we'll want these files to be proofread by native speakers but we can do that as part of the broader QA effort for French support.


refactor(tests): use testonnet instead of assert

Testonnet runs each test individually and outputs all test results, assert stops
execution on first failure.

rebased.


Installing documentation

We need docs on how to build/install/download this thing

And maybe a way to render it? If so I recommend slate

@kris-nova I think it's already fixed with #53
i'm just thinking should we explicitly write that you must have Go env installed until first version doesn't get released (and link to tutorial from golang.org)?

Closed with #53 and #147


Terraform script fails with InvalidPrincipalId

During creation of azure infrastructure with provided terraform definitions the following exception occurs:
Error: authorization.RoleAssignmentsClient#Create: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="InvalidPrincipalId" Message="A valid principal ID must be provided for role assignment."
with module.kv_st_services.azurerm_role_assignment.sinequa_kv_role_for_me,
│   on ....\modules\services\main.tf line 33, in resource "azurerm_role_assignment" "sinequa_kv_role_for_me":
│   33: resource "azurerm_role_assignment" "sinequa_kv_role_for_me" {

Hello, I solved the issue by adding a ">" in the conf.tf file
from
  required_providers {     azurerm = {       source  = "hashicorp/azurerm"       version = "=2.78.0"     }   } }
to
  required_providers {     azurerm = {       source  = "hashicorp/azurerm"       version = ">=2.78.0"     }   } }
I hope this will help.


chore(mysql2): adding TAV script

Which problem is this PR solving?
This will let us run test-all-versions scripts for mysql2 plugin, and make sure tests are green for all versions supported by the instrumentations.
Currently this is just a script in package.json which can be run manually on demand by user to verify compatibility. This is not integrated into CI or expected to impact project in any way.

I'm putting this PR in draft mode, I found some issues for <2.1.0 versions.
I'm investigating the issue

After fixing broken tests for older versions (v2.0.0, v2.0.1, v2.0.2, v2.1.0)
PR is ready for review

Might be worth making the fixes in a separate fix: PR so it can be properly detected as a bug by tooling

Might be worth making the fixes in a separate fix: PR so it can be properly detected as a bug by tooling

The fix is relevant only for the tests, it doesn't change the instrumentation code.


Crash of the App when selecting a category

Describe the bug

I use your Android-App for a look into my nextcloud recipes. If I choose the middle bottom button for a list of my categories and I click on a category with more than one recipies inside, the app crashes.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

I would have shown a list of my recipes related to the choosen category.
Screenshots

Smartphone:

Device: [e.g. Pixel 7]
OS: [e.g. Android 13]
Version [e.g. 22]
Device: Redmi Note 11S
OS: MIUI 14.0.3

Additional context

I have a category with only one related recipe. If I choose this category the related recipe will be shown. No app crash.

Hi @rtglobal, thank you for reporting this issue. I may already have solved this issue with e1d4724. I'll try to create a new release tomorrow.

Hi @rtglobal, can you confirm the issue is fixed with the lastest release v0.13.0?

Yes, I can confirm this. Wonderful work. I like the new look. Thank you in advance.


A user can delete himself

Later we can try to delete all
Messages and things from the browser too.

Added new API:
POST http://localhost:12002/rpc.UserService/DeleteMe
Authorization: Bearer AUTH-TOKEN
Content-Type: application/json

{}

POST http://localhost:12001/rpc.UserService/DeleteMe
Content-Type: application/json

{}


Mystery Box can give players the exact same weapon as slot 1 or 2

Mystery Box can give players the exact same weapon as slot 1 or 2 allowing them to run an entire game with two Rayguns! or two LSats makes the game very broken at Early and Late Rounds
Using Spigot 1.16
COMZ DEV comzombies-Universal-2.2.65.jar

Added in v2.2.86