Use safeUp/safeDown methods by default for migration templates

In #13827 was added behavior to generate safe methods by default.
Maybe also add this to any migrations templates?
I think this good idea.
Also MySQL from 8 version will be with  transnationally DDL.
https://github.com/yiisoft/yii2/blob/master/framework/views/dropTableMigration.php#L30
https://github.com/yiisoft/yii2/blob/master/framework/views/dropTableMigration.php#L42
https://github.com/yiisoft/yii2/blob/master/framework/views/dropColumnMigration.php#L32
https://github.com/yiisoft/yii2/blob/master/framework/views/dropColumnMigration.php#L45
https://github.com/yiisoft/yii2/blob/master/framework/views/createTableMigration.php#L31
https://github.com/yiisoft/yii2/blob/master/framework/views/createTableMigration.php#L44
https://github.com/yiisoft/yii2/blob/master/framework/views/createJunctionMigration.php#L31
https://github.com/yiisoft/yii2/blob/master/framework/views/createJunctionMigration.php#L73
https://github.com/yiisoft/yii2/blob/master/framework/views/addColumnMigration.php#L33
https://github.com/yiisoft/yii2/blob/master/framework/views/addColumnMigration.php#L46

Agree.

@samdark ok. I will fix it..


silly question, but how to delete cached environment variables that verba's using?

hey, openai reset everyone's api keys and moved them to project-based keys, and i can't for the life of me figure out how to get verba to update to the new env var. I've tried the following:
update .env
export OPENAI_API_KEY=newkey
delete browser cookies and cache
change to a different browser (chrome to edge)
create a new virtualenv
uninstall goldenverba, reinstall with --no-cache flag
seems like no matter what I do i'm getting the unserializable object error (which iirc means my api key is incorrect, but it's difficult to tell for certain as i'm not displayed the error code.

Hey! I also struggled with this 🥲 Here are some tips I used to get it working, I hope they can help:

Make sure that the OPEN_API_KEY is not stored in ~/.zshrc, ~/.bash_profile, etc. python_dotenv will always prioritize global set variables to local .env files
You can use printenv outside your working dir to print all environment variables
If OPEN_API_KEY is only stored  in your .env, try to use source .env before starting verba, it will make sure that it uses the updated keys instead of any cached one.

Let me know if this helped fix things 🚀


Unable to open any files

I received this error when I attempted to open a file on the Mac version.

I have a folder containing the image and the mp3 for the chart.

The standalone file browser has not bind to the macOS in unity editor, waiting for the latest pull request to merge in order to solve this issue

Same issue to you!
https://github.com/yojohanshinwataikei/Arcade-plus/issues/11#issue-1434052928

This should be already fixed in #7, which will be included in the next release.


Clear previous search and hash on push/replace with pathname only

When history.push or history.replace is called with string URL without search or hash part the ones from previous location are kept.
history.push('https://reacttraining.com/with-query?query=hello');

// https://reacttraining.com/with-query?query=hello -> This is OK
console.log(history.createHref(history.location));

// new route is pushed without search
history.push('https://reacttraining.com/no-query');

// https://reacttraining.com/no-query?query=hello -> Current result keeps the old search
console.log(history.createHref(history.location));

This change adds default empty values for search and hash which are replaced if parsePath function finds some.
If the previous search or hash have to be reused PartialPath object can be passed with omitted keys.

Fixes https://github.com/ReactTraining/history/issues/814 ?

Fixes #814 ?

Yes, I have missed that is already reported.


⚠ Punto de Fuga has degraded performance

In 4b81893, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 15110 ms

Resolved: Punto de Fuga performance has improved in 80e9453.


Settings Button Covers Disconnect Button

When I have this mod enabled, its settings button sits on top of my Disconnect button in the pause menu. This prevents me from disconnecting at all without closing my client window entirely. I highly recommend switching to a Mod Menu interface for your settings.

Thanks for the report. Could you provide any more information, like what GUI scale you're using and maybe your monitor resolution? In the meantime, you could see if playing on a different GUI scale would fix this.

I'm on vacation for like a week but I will work on a patch as soon as I'm home. 👌

I play on gui scale 2/Normal, but testing it at Auto, 1/Small, and 3/Large yielded the same results. As did changing the size of the window arbitrarily. I've narrowed it down to the Mod Menu mod, which is essential for accessing the settings of various other Fabric mods. It adds an extra button over the Disconnect button, shifting it down.

I see. So it seems cabbagescape is merging poorly with another mod. Unfortunately this is an issue with no perfect fix, some mods just aren't going to be compatible with one another. But I will look at moving the settings button somewhere different for your convenience 😉

Thank you. It's really a good framework for doing mod settings, because of conflicts like this. There's only so much screen space to tack on extra buttons and messing about in config files is too much for a lot of users (plus it requires a whole restart, which can take a while in other situations). Never really understood why Fabric doesn't have this feature built-in like Forge does.

Will be added to the to-do list for future features but, because it is not a bug with the mod and instead a compatibility issue, I will be closing this topic. Your idea will most likely be finished in version a1.4 so you'll just have to wait for that.


bug: Strange input results in false positive

Expected behavior
When I input the following string:
    "" ""
    "" ""
    "" ""
Assamese -> Assam

I expect that there should be no censoring.
Actual behavior
However, Assam becomes A*sam.
Strangely, modifying parts of the string, such as the quotes ("), results in no censoring.
Minimal reproducible example
import {
  RegExpMatcher,
  TextCensor,
  englishDataset,
  englishRecommendedTransformers,
  keepStartCensorStrategy,
  keepEndCensorStrategy,
  asteriskCensorStrategy
} from 'obscenity'

const matcher = new RegExpMatcher({
  ...englishDataset.build(),
  ...englishRecommendedTransformers
})

const strategy = keepStartCensorStrategy(keepEndCensorStrategy(asteriskCensorStrategy()))
const censor = new TextCensor().setStrategy(strategy)

const input = `    "" ""
    "" ""
    "" ""
Assamese -> Assam`

const matches = matcher.getAllMatches(input)
console.log(censor.applyTo(input, matches))

Steps to reproduce

Run the above code
View console

Additional context
No response
Node.js version
N/A
Obscenity version
0.2.0
Priority

[X] Low
[ ] Medium
[ ] High

Terms

[X] I agree to follow the project's Code of Conduct.
[X] I have searched existing issues for similar reports.

const input = `

Assamese -> Assam`

Additionally, removing the quotes and leaving just the whitespace (4 spaces on each line) still results in the unexpected censoring. If you delete any of these spaces, no censoring occurs.

The error seems to be in the whitelisted term matching logic. In particular, we are using an index into the original input where we should instead be using an index to the transformed input, resulting in the second assa to be skipped over. The following diff seems to fix it, if this is urgent for you:
diff --git a/src/matcher/regexp/RegExpMatcher.ts b/src/matcher/regexp/RegExpMatcher.ts
index 7f4fdb1..af31d87 100644
--- a/src/matcher/regexp/RegExpMatcher.ts
+++ b/src/matcher/regexp/RegExpMatcher.ts
@@ -161,7 +161,7 @@ export class RegExpMatcher implements Matcher {
                                }

                                matches.insert(indices[startIndex], endIndex);
-                               lastEnd = endIndex + 1;
+                               lastEnd = startIndex + whitelistedTerm.length;
                        }
                }

I will hold off on a patch release until I have time to look at this more carefully, though. The matching logic is fairly complex and I would like to refamiliarize myself with the implementation to ensure this is fully correct first (particularly in cases with non-ASCII characters.) Unfortunately, as I said in #46, this may have to wait until late this month or early February. Apologies.


config: set default region for EC2 SD

If Prometheus is running on EC2, most of the case region might be same region where Prometheus is running.
I want to set default region for the case.

I forgot to mention maintainers.
@brian-brazil Would you check this PR?

Change seems fine to me in general if that's how AWS clients work with regions in general.

I think..., the aws-sdk-go should assure that.

@fabxc @brian-brazil How I can proceed this PR?
I should fix something?

My bad, looks all good. Thanks!

Thanks!!


Failing to converge core-site

================================================================================
Error executing action run on resource 'ruby_block[node.run_state[:core_site_generated_values]]'
================================================================================
NoMethodError
-------------
undefined method `[]' for nil:NilClass

Cookbook Trace:
---------------
/var/chef/cache/cookbooks/bcpc-hadoop/recipes/core_site.rb:80:in `block (2 levels) in from_file'

https://github.com/bloomberg/chef-bach/blob/bee4c26390fda7d64366b6409f718691706d5500/cookbooks/bcpc-hadoop/recipes/core_site.rb#L78-L91

I am able to output run state from a ruby block that executes ruby puts node.run_state

I think I found the vector :) https://github.com/bloomberg/chef-bach/commit/b9ad223ae7e01fde21b05b5889a6d8f5d014ec03#diff-9b58b641088e25d6d1218774d01fc2d9


SPARK-5570: No docs stating that `new SparkConf().set("spark.driver.memory", ...) will not work

I've updated documentation to reflect true behavior of this setting in client vs. cluster mode.

Test build #27663 has started for   PR 4665 at commit db47595.

This patch merges cleanly.

Test build #27663 has finished for   PR 4665 at commit db47595.

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

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

Hey @ilganeli thanks for doing this. Can you also do this for the other spark.driver.* options? Like extra java opts, class paths etc.

Sure @andrewor14 , I presume their behavior is identical ?

Yes, those won't work either if the user sets them through the conf.

Hi @andrewor14 - Can you please review the language for the other driver options and let me know if you agree with it? Thanks!

Test build #27736 has started for   PR 4665 at commit f149ba1.

This patch merges cleanly.

Test build #27736 has finished for   PR 4665 at commit f149ba1.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

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

What is a "MiMa test" failure??

MiMa is the check for changing public API signatures. This is a false positive since you didn't change any code, so don't worry about it, although a rebase might clear it up anyway.
Also, this may be purely a matter of taste, but I might have done
Note: to not put a blank line into the table boxes. Given the layout the extra whitespace makes it looks like there are extra rows somehow.

Got it - I wanted to break up the text. That's a good idea.

Test build #27752 has started for   PR 4665 at commit 5889d43.

This patch merges cleanly.

Test build #27754 has started for   PR 4665 at commit 85f7a08.

This patch merges cleanly.

Test build #27755 has started for   PR 4665 at commit a51700a.

This patch merges cleanly.

Hi @ilganeli thanks for reiterating on my comments quickly. I think this is pretty close to being merged.

I'm happy to help :)

Test build #27756 has started for   PR 4665 at commit 5d1c8dd.

This patch merges cleanly.

Ok, LGTM I'm merging this into master and 1.3 thanks.

Test build #27752 has finished for   PR 4665 at commit 5889d43.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkJobInfo(namedtuple("SparkJobInfo", "jobId stageIds status")):
class SparkStageInfo(namedtuple("SparkStageInfo",
class StatusTracker(object):

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

Test build #27755 has finished for   PR 4665 at commit a51700a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #27754 has finished for   PR 4665 at commit 85f7a08.

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

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

Test build #27756 has finished for   PR 4665 at commit 5d1c8dd.

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

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


CI builds are failing on AMD64 with "no space left on device"

Describe the bug
CI builds are failing on AMD64 with "no space left on device"
To Reproduce
Look at recent builds.
Expected behavior
Should not run out of space.
Additional context
We have seen this before: https://issues.apache.org/jira/browse/ARROW-12112

The fix for this is in https://github.com/apache/arrow-datafusion/pull/2495


Property 'ecRef' does not exist on type 'Sync'.  TS2339

能否把ecRef的代码贴一下？以及你用的address-picker的版本号？
Can you paste the code of ecRef? And the version number of the address-picker you used?
@creditcoder

@creditcoder 你好，如果你15天内不提供完整信息，这个issue将关闭

I'm also not sure about 'ecRef'.
I can't find in your project source.

太久了。就先关了哈。嘻嘻


Send greeting as private message

Would it be possible for shieldy to send greetings to the user as a private message after they pass the challenge/response?  This would be preferable to the greeting being in-channel.  Thanks either way!

There is no way to do so because Telegram does not allow bots to initiate dialogues with users. Use /deleteGreetingTime to make Shieldy auto-delete greetings.


balena-ssh: formatting fixups

Change-type: patch
Signed-off-by: Matthew McGinn<EMAIL_ADDRESS>

sighs and opens in CLI repo

https://github.com/balena-io/balena-cli/pull/1392/ for reference


Allow RDF::Vocabulary#send and #public_send

This reinstates #send and #public_send as a fix for #356.
We undefine many of the methods on Vocabulary::Term objects to clear the way for #method_missing; #send and other metaprogramming methods like #instance_methods are caught up in this.
:send doesn't seem to be a popular vocabulary term, so it's arguably harmless.
The behavior around this is pretty messy; honestly, the whole thing has me thinking about deprecating the method_missing handling and pushing users to #[] syntax much aggressively for 3.0.0. But I do think this is an improvement, since #send currently works on Vocabularies defined as classes (class MyVocab < RDF::Vocabulary('...'); end), and only errors when defining vocabulary instances (as v = RDF::Vocabulary.new('...')).

Coverage remained the same at 91.49% when pulling<PHONE_NUMBER>111b2b14ea70139dec4f726d223c44 on vocabulary-send into e7b328907124b23922a4446d481a8c3acb02ecb5 on develop.

Coverage remained the same at 91.49% when pulling<PHONE_NUMBER>111b2b14ea70139dec4f726d223c44 on vocabulary-send into e7b328907124b23922a4446d481a8c3acb02ecb5 on develop.

Coverage remained the same at 91.49% when pulling<PHONE_NUMBER>111b2b14ea70139dec4f726d223c44 on vocabulary-send into e7b328907124b23922a4446d481a8c3acb02ecb5 on develop.

Coverage remained the same at 91.49% when pulling<PHONE_NUMBER>111b2b14ea70139dec4f726d223c44 on vocabulary-send into e7b328907124b23922a4446d481a8c3acb02ecb5 on develop.


Message handling/history

Implementation for 1:1 connection, to be implemented for mesh later.

Message object?

Content
Sender
Timestamp
Other?

Displaying and saving message history (on runtime)

Message buffer?

In the case a node isn't in range when, say, a group message is sent, they should receive it as soon as they regain connection to the network

Adding messages visually to screen on runtime works

Saving/loading database functional


Resolve failed-precondition error

The deployed version cannot enlist the transactions and results in an error with code failed-precondition

Closed by https://github.com/usamazansari/expenses-tracker/pull/25#issuecomment-1585596899


[tables] re-order columns

Ability to pick up a column and drag it to a different order

Closed by #6570


chore: improve some of the deployment and build scripts

What this PR does / why we need it:

deploy-kubefed.sh: new envvar NO_JOIN_HOST_CLUSTER. When set to a
non-empty value prevents the script from joining the host cluster to
itself. Default behaviour doesn't change.
sync-up-helm-charts.sh: reduced verbosity of the script by letting
etcd and kube-apiserver log into files instead of stdout/stderr.
This should make it much easier to follow the steps the scripts is
exercising. Also improved the two messages printed when waiting for
etcd and kube-apiserver to get ready.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
n/a
Special notes for your reviewer:
No behaviour has been changed with respect to what the scripts do but only to how much information they log to stdout.

/approve

/lgtm


Factoring algorithm for resource estimation

This adds a factoring algorithm based on work from Ekara-Hastad with optimizations to modular arithmetic subroutines based on work from Gidney (see https://arxiv.org/abs/1905.09749)

I would add this sample to the readme at https://github.com/microsoft/qsharp/tree/main/samples/estimation, so that we document all samples uniformly.
Since this is a two-file sample (Jupyter notebook requires the .qs file to work), do we want to move both files to a separate subfolder to emphasize this? Not sure if the intent is to use them together only, or to allow using Q# file on its own as well - if it's the latter, we might want to keep them in this folder and document them separately in the README

I would add this sample to the readme at https://github.com/microsoft/qsharp/tree/main/samples/estimation, so that we document all samples uniformly.
Since this is a two-file sample (Jupyter notebook requires the .qs file to work), do we want to move both files to a separate subfolder to emphasize this? Not sure if the intent is to use them together only, or to allow using Q# file on its own as well - if it's the latter, we might want to keep them in this folder and document them separately in the README

Thanks, I addresses this comment in a recent commit.


add copy button to sprite_frames_editor

Add a copy button to sprite_frames_editor.
Easy to duplicate sprite in sprite frames editor,or copy to anywhere..

Note that this question mark next to your commit's author name and the lack of image means that the email address you used for git locally is not linked to your GitHub account. You might want to fix it by adding that address as secondary GitHub address, thus claiming back your commits made under this identity.

Could you also make the equivalent PR for the master branch if relevant?

I will make it a little later..


[AutoDiff] [stdlib] Add a JVP for 'differentiableMap(_:)'.

Add a JVP for 'differentiableMap(_:)'.

cc @marcrasi @rxwei

Fixed the suggested things in the code style!

@swift-ci Please smoke test

(Not sure if the test is passing now -- lit fails with  Aborted (core dumped) ... a.out at some point for me)

@swift-ci Please smoke test

@dan-zheng should we close this since https://github.com/apple/swift/pull/32854 was merged?

Yes, thanks - this PR should be closed because it's subsumed by https://github.com/apple/swift/pull/32840.


Live Deployment Issues

I'm having a couple of issues with a live deployment to Vercel that I don't have in the local environment. Requests are often ending with An error occurred with your deployment FUNCTION_INVOCATION_TIMEOUT. Looking in the Vercel function logs you can actually see responses are getting created, but just ending in a 504. Additionally when the request is returned successfully there is no live text printout, the answer just shows up when completed. Everything works well in the local environment however. Anyone else experiencing this?

"If the Serverless Function does not respond to the HTTP request within the timeout, then a 504 error status code is returned with the error code FUNCTION_INVOCATION_TIMEOUT."
Vercel has a serverless function timeout of approx. 10 seconds on the hobby plan: docs
Options would be either to try the Pro version, or other deployment platforms.

@darkomking Hey! What platform did you end up using. Im looking for one!

Did you change something in settings? I upgraded but the 10 sec limit still there

Did you change something in settings? I upgraded but the 10 sec limit still there

You need to redeploy before this takes effect  👍


Add flag to disable Syft hitting toolbox-data.anchore.io

What would you like to be added:
A flag to disable Syft's checking of the most recent version of Syft to prompt for potential update
Why is this needed:
Syft tries to hit Anchore's s3 toolbox on every run, which causes a long delay when running in an air gapped environment.
[0030] ERROR failed to fetch latest version: Get "https://toolbox-data.anchore.io/syft/releases/latest/VERSION": dial tcp <IP_ADDRESS>:443: i/o timeout

Additional Information
Grype provides this under the GRYPE_DB_AUTO_UPDATE environment variable

I think this exists
# enable/disable checking for application updates on startup
# same as SYFT_CHECK_FOR_APP_UPDATE env var
check-for-app-update: true

https://github.com/anchore/syft/blob/main/README.md?plain=1#L427


Example of 'Conditionally revealing content'

The provided example on this page is :
'
Do you know their National Insurance number?

  <legend class="visuallyhidden">Do you know their National Insurance number?</legend>

      <input id="radio-indent-1" type="radio" name="radio-indent-group" value="Yes">
      Yes

      <input id="radio-indent-2" type="radio" name="radio-indent-group" value="No">
      No

      National Insurance number
      <input class="form-control" name="national-insurance" type="text" id="national-insurance">

</fieldset>

'
The 'No' label is missing the data-target="example-ni-no"  attribute

The data-target="example-ni-no" attribute is present on the "Yes" label, so if you know the NI number, you can enter it and if you don't know the NI number, you aren't then asked to fill in an additional field.

Maybe that clarification should be added on the guidance?

The text above, says:
Conditionally revealing content

reveal additional questions, depending on the context

Click on "Yes" below to see how this works.

If I use that code and then I click 'No', the conditional field won't disappear

Hi @pietrodesiato, if you could share a link to your code with me - I can take a look and see if I can help.

For this to work, you'll need the JavaScript used by GOV.UK elements to add a .js-hidden class, which will hide this content.
The styles for this are set by the GOV.UK template:
.js-enabled .js-hidden {
  display: none;
  visibility: hidden;
}

The JavaScript responsible for adding the the .js-hidden class is here.
Hope that helps.

I copy and pasted the code in the example and it's not working for me: the conditional div is showing when clicking 'Yes' but it does not hide when clicking 'No'.
I looked at the html and the 'Yes' has this additional attribute that is not mentioned in the code example: aria-controls="example-ni-no"

The aria-controls attribute is there to provide additional information to screenreaders, so that they can set focus to the revealed content when it is opened.
It creates a relationship between the control and the content which is toggled.
http://tink.uk/using-the-aria-controls-attribute/
https://www.w3.org/TR/wai-aria/states_and_properties#aria-controls
Have you checked that your JS matches what is here?
https://github.com/alphagov/govuk_elements/blob/master/public/javascripts/application.js#L10

I checked the elements applications.js and the prototyping toolkit application.js (which I am currently using) and they have some significant differences that might affect this specific behaviour.
The kit is using show() and hide() while elements are adding or removing the CSS js-hidden class.

ELEMENTS:
` // Hide toggled content
$groupDataTarget.addClass('js-hidden');
// Set aria-expanded and aria-hidden for hidden content
$this.attr('aria-expanded', 'false');
$groupDataTarget.attr('aria-hidden', 'true');
});
      var $dataTarget = $('#' + dataTarget);
      $dataTarget.removeClass('js-hidden');
      // Set aria-expanded and aria-hidden for clicked radio
      $radio.attr('aria-expanded', 'true');
      $dataTarget.attr('aria-hidden', 'false');

    });`

PROTOTYPING KIT
` // Hide toggled content
$groupDataTarget.hide();
// Set aria-expanded and aria-hidden for hidden content
$this.attr('aria-expanded', 'false');
$groupDataTarget.attr('aria-hidden', 'true');
});
      var $dataTarget = $('#' + dataTarget);
      $dataTarget.show();
      // Set aria-expanded and aria-hidden for clicked radio
      $radio.attr('aria-expanded', 'true');
      $dataTarget.attr('aria-hidden', 'false');

`

There is a pull request which fixes those differences for the prototype kit, it hasn't been merged yet.

Hi @pietrodesiato, @joelanman merged that PR, so if you update your version of the prototype kit - that example will work for you.
There's also another working example here: http://govuk-prototype-kit.herokuapp.com/examples/elements/toggled-content

#212 fixes this.


Safari extension doesn't allow export to Obsidian

Hi,
Even though I have installed the Advanced Obsidian URI plugin, and have enabled the Markdownload plugin setting to export to Obsidian, the plugin button doesn't show me any way to actually export to Obsidian. Only one button exists ('Download') which puts a Markdown file in my Downloads folder.
Best,
Lyndon

Should work with a right mouse click inside the browser window (like in the chrome plugin), but it still doesn't work. Also it was mentioned in the release note in AppStore: - Fixes for Obsidian integration in Safari (thanks @aancw !).
So what is fixed?

I am also not able to use the "Send Tab to Obsidian" feature that is available in the R-click context menu (click on webpage body) >  Markdownload - Markdown Web Clipper.  I get a pop up alert asking if Do you want to allow this website to open “Obsidian”?, but I get no new note in Obsidian.
Strangely, I just copied to clipboard the alert text and got a new Obsidian note with the webpage title as the note title and the copied alert text in the body. I haven't looked at the source code at all, but perhaps the markdown text is either not getting copied to clipboard or not being pasted?

Umm... in a strange breaking update, it now works. I don't understand why the behavior changed, though...
To aid in diagnosing, I have the Vault Name set as <vault_name> and the Folder Name set to Web Clips with no slashes or full paths.

@mjs271 really strange, I just try it again and it works. So fare I see there was no update until my last try. Suddenly the entries rin the context menu are displayed. So looks like I'm fine. Still have to check about the contents of the note, because currently it only capture the title and excerpt.

I am here to report that the Obsidian integration seems to be broken again on Safari 17.3 (and on MacOS 14.2 and 14.3). I have already confirmed that the advanced URI community plugin is active and functional with other browsers such as Firefox, but it is not cooperating with the Safari plugin.
Do you have any insights regarding what might be happening?

I am here to report that the Obsidian integration seems to be broken again on Safari 17.3 (and on MacOS 14.2 and 14.3). I have already confirmed that the advanced URI community plugin is active and functional with other browsers such as Firefox, but it is not cooperating with the Safari plugin.
Do you have any insights regarding what might be happening?

Hello, I've the same problem ... Thank you for your help (Safari 17.2.1, Ventura 13.6.3) !

+1
Safari Version 17.6 (196<IP_ADDRESS>.5) too.
@deathau could you please take a look?


add concurrency CI

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#concurrency=
Didn't work well with matrixes, but examples are not good for matrixes, might be possible.
styfle/cancel-workflow-action works well.
Review only adding the styfle/cancel-workflow-action, the rest is code format.
#skip-changelog

Codecov Report

Merging #808 (415155f) into main (36b4cfa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #808   +/-   ##
=======================================
  Coverage   89.83%   89.83%
=======================================
  Files         114      114
  Lines        3670     3670
=======================================
  Hits         3297     3297
  Misses        373      373

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 36b4cfa...415155f. Read the comment docs.


Setup guide

Please wait 3rd bin sync complete, after thus, then you can do follow to run cpp-tests:
mkdir build
cmake -A Win32 ..
cmake --build . --config Debug --target cpp-tests


Any plan to add typescript support?

Type definitions for public API

Yeah, we do have plan to add typescript support, We will finish this in the first quarter of 2020.


Froala, Unlicensed

My company has purchased a fully fledged license, but we still get the little red block in the bottom left which says Unlicensed Froala.  Any ideas?

Also, we do insert the license number into the configuration...
    this.froalaEditorOptions = {
      theme: 'custom',
      placeholderText: 'Write Something...',
      charCounterCount: true,
      key: 'KEY GOES HERE',
      heightMin: 300,
      heightMax: 400,
      toolbarBottom: true,

@ditproz please contact us via the support form and our support team will help you with this matter. Thanks.

I did contact you via the support form and never received an email back.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Stefan<EMAIL_ADDRESS>Sent: 05 February 2018 13:20
To<EMAIL_ADDRESS>Cc: Daniel<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [froala/angular-froala] Froala, Unlicensed (#166)
@ditprozhttps://github.com/ditproz please contact us via the support formhttps://www.froala.com/wysiwyg-editor/contact and our support team will help you with this matter. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/froala/angular-froala/issues/166#issuecomment-363082370, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVx6l9hjLHo_izA9w6OmBq44jN8wtDoHks5tRwApgaJpZM4R5H0L.

@ditproz could you please let me know the ticket number you got?


API Analytics Plugin

Use ALF as the default format for all HTTP logging.

+1

@Nijikokun @SGrondin @kennethklee @ahmadnassri please tell us more info about the ALF Plugin.
Where should the data be sent to: http, tcp? What address? @thibaultCha will take care of the implementation.
Renaming this issue to: API Analytics plugin

all the info is documented here

There is no such thing as "default logging". There is a plugin for each logging capacity. ALF appears to be one of the available logging plugin, but not the default because all plugins are equal...
Thus why it changed to the ALF logging plugin ticket. I knew for the ALF repo, but it does not provide info son how to send the data. Will follow up in 1-1

you have tcplog and udplog plugins... what formats are those using?

I see your point. So rather to separate the plugins by logging format, it is true they are separated by protocol they're using to send the data. In that case, if I want to send data to apianalytics, can't I just use one of those and set the format to "ALF", or what other value would APIanalytics give me? More than logging? What do we need to build now? An ALF serializer for our current logging plugin or an API analytics plugin? There is no clear roadmap

yes, API Analytics integration can simply be the TCP plugin (potentially) or UDP, whatever .. its more about the format (and setting a token)
currently API Analytics is built on ZMQ, so TCP is certainly a preferred method, as long as its in ALF format + an API Analytics token is set, just send the data over TCP / HTTP and that's it.
I could see this same setup be used for any TCP based logging platform (splunk) hence why I'm more interested on the format since you already got a TCP plugin working.
if it is possible for plugins to "stack" in a chain, where one plugin creates log data, and another sends it over a protocol, then that would be epic actually ... #IdealWorld

To recap, we want a new plugin called API Analytics that creates the ALF payload and send it to a configurable ZMQ server.

what formats are those plugins using:

http://getkong.org/plugins/udp-log/
http://getkong.org/plugins/tcp-log/
http://getkong.org/plugins/file-log/

we want a new plugin called API Analytics that creates the ALF payload and sends it to a configurable ZMQ server.

That's the point, we have different ways of doing that.

Each logging plugin is tied to a protocol, and has different formats. tcplog sends data over tcp and can have an ALF format. And if you send data to API Analytics, it can also have a key. Which is kind of strange, but totally doable. In that case, the default format would be "ALF", and other formats could be "raw", however it is currently (not any specific format).

We have an API Analytics plugin, with a documentation of its own, a configuration of its own, and that plugin sends data over, say, tcp only.

What's the preferred approach?> we want a new plugin called API Analytics that creates the ALF payload and sends it to a configurable ZMQ server.
That's the point, we have different ways of doing that.

Each logging plugin is tied to a protocol, and has different formats. tcplog sends data over tcp and can have an ALF format. And if you send data to API Analytics, it can also have a key. Which is kind of strange, but totally doable. In that case, the default format would be "ALF", and other formats could be "raw", however it is currently (not any specific format).

We have an API Analytics plugin, with a documentation of its own, a configuration of its own, and that plugin sends data over, say, tcp only.

What's the preferred approach?

The new plugin will create the ALF payload (only ALF) from the Lua object that Kong provides, and then send it to TCP. So it's your second option.
In the future we might want tcplog and updlog to have configurable log formats, but that would another issue. When we do that, we could unify/change the way logging works.

As stated in the ApiAnalytics docs, the server accepts both ZMQ and HTTP. For something like Kong, it's up to you to either include the ZMQ instructions into the install scripts (homebrew, .deb, etc etc) or, I think better in this case, use the batch HTTP input. Simply batch ALFs 1000 at a time into an array and send it off to the server.

you can batch both the ALFs + the entries within the ALFs, whaver makes the most sense for your agent's performance of course


Add warnings for Media Elements with the same base name

Currently the Media Elements use the base name of the file as the key for the StaticResource which means that two different file types with the same name will clobber each other and create a sea of red in the console. I see a couple of different options:

A stopping error message in the console when BaseName of one Media Element is the same as another (e.g. Loon.mp3 and Loon.jpg)
Using the full name of the file including extension as the key of the StaticResource (e.g. Loon.mp3 would become LoonMP3)

I'm leaning toward the latter because errors are never fun. However, it does make the syntax of the StaticResource a bit odd.

Resolved in Version 1.2


Programmatically set position and direction

Gradle version updated to 2.3.3
Build tools version updated to 25.0.3
Support library version updated to 25.4.0
Set position and direction methods added.
Minor improvements

wow, thanks a lot ;)


Root readme khadija

name: solution review
about: A template PR for code review with a checklist

Behavior
Files

[ ] The file name describes the function's behavior
[ ] There is a module docstring in the function file
[ ] The test file's name matches the function file name -
/tests/test_file_name.py
[ ] There is a module docstring in the tests file

Unit Tests

[ ] The test class has a helpful name in PascalCase
[ ] The test class has a docstring
Every unit test has

[ ] A helpful name
[ ] A clear docstring
[ ] Only one assertion
[ ] There is no logic in the unit test

[ ] All tests pass
[ ] There are tests for defensive assertions
[ ] There are tests for boundary cases

Function Docstring

[ ] The function's behavior is described
The function's arguments are described:

[ ] Type
[ ] Purpose
[ ] Other assumptions (eg. if it's a number, what's the expected range?)

The return value is described

[ ] Type
[ ] Other assumptions are documented

The defensive assertions are documented using Raises:

[ ] Each assumption about an argument is checked with an assertion
[ ] Each assertion checks for only one assumption about the argument

[ ] Include 3 or more (passing!) doctests

The Function

[ ] The function's name describes it's behavior
[ ] The function's name matches the file name
[ ] The function has correct type annotations

Strategy
Do's

[ ] Variable names help to understand the strategy
[ ] Any comments are clear and describe the strategy
[ ] Lines of code are spaced to help show different stages of the strategy

Don'ts

[ ] The function's strategy is not described in the documentation
[ ] Comments explain the strategy, not the implementation
[ ] The function does not have more comments than code

If it does, consider finding a new strategy or a simpler implementation

Implementation

[ ] The code passes the formatting checks
[ ] The code passes all Ruff linting checks
[ ] The code has no (reasonable) Pylint errors

In code review, you can decide when fixing a Pylint error is helpful and
when it's too restricting.

[ ] Variables are named with snake_case
[ ] Variable names are clear and helpful
[ ] The code follows the strategy as simply as possible
[ ] The implementation is as simple as possible given the strategy
[ ] The code includes defensive assertions
[ ] Defensive assertions include as little logic as possible

Thanks @olumide-AI for your feedback! I did it because we have asked Evan on slack if it's suitable to structure the solution tree as the old way and this was his response: "I would suggest doing it like in the examples and exercises folders.   Someone using your code would want to know what it does, not who wrote it.    It’s still possible to know who wrote what by reading through the git/github history: /solutions
/tests
test_deep_flatten_list.py
test_find_largest.py
deep_flatten_list.py
file_largest.py"
I hope it's clear now for you, please let me know if you'd like to discuss this issue further.

Couldn't understand what are the changes in README.md . What are they for?

Didn't see the comments above from Khadija. I understnd now what for. :)


Get more information from ethtool, like speed

https://softwarefactory-project.io/r/6320

https://softwarefactory-project.io/r/#/c/6320/


onSubmit not triggered on vanilla JS

We are using typeform vanilla JS in production and onSubmit method is no more triggered.
A test case is available here.
Related to https://github.com/Typeform/embed/issues/453

Hello @DanFaudemer, thanks for reporting this.
This is not related to #453 - that one was caused by the form not sending events to embed SDK. The onSubmit callback works in HTML snippets via data-tf-on-submit but seems to be broken in JS embeds.
We will investigate.

You have enableSandbox: true in your typeform embed. Submissions are disabled in sandbox mode and therefore the event is not dispatched.
@DanFaudemer  did this work in sandbox mode before?

@DanFaudemer we are looking into it and we will reactivate this for sandbox too. Sorry for the inconvenience.


[Errno 13] Permission denied in python3-pkg-resources when trying to install python3-opencv through apt

I hope this is the correct github repo for this bug - I'm trying to install opencv-python through apt-get in the default python docker image with the following Dockerfile :
FROM python:3

RUN apt-get update
RUN apt-get install -y python3-opencv

Having this Dockerfile in a folder, simply building it with docker build -t example/opencv ..
This seems to have worked previously, but since recently it's giving the following error :
...
Selecting previously unselected package python3-pkg-resources.
Preparing to unpack .../217-python3-pkg-resources_52.0.0-4_all.deb ...
Unpacking python3-pkg-resources (52.0.0-4) ...
Selecting previously unselected package python3-numpy.
Preparing to unpack .../218-python3-numpy_1%3a1.19.5-1_amd64.deb ...
Unpacking python3-numpy (1:1.19.5-1) ...
Selecting previously unselected package python3-opencv.
Preparing to unpack .../219-python3-opencv_4.5.1+dfsg-5_amd64.deb ...
Unpacking python3-opencv (4.5.1+dfsg-5) ...
Selecting previously unselected package va-driver-all:amd64.
Preparing to unpack .../220-va-driver-all_2.10.0-1_amd64.deb ...
Unpacking va-driver-all:amd64 (2.10.0-1) ...
Selecting previously unselected package vdpau-driver-all:amd64.
Preparing to unpack .../221-vdpau-driver-all_1.4-3_amd64.deb ...
Unpacking vdpau-driver-all:amd64 (1.4-3) ...
Setting up libgme0:amd64 (0.6.3-2) ...
Setting up python3-pkg-resources (52.0.0-4) ...

Errors were encountered while processing:
 python3-pkg-resources
 python3-numpy
 python3-opencv
E: Sub-process /usr/bin/dpkg returned an error code (1)

Host system is latest Ubuntu 22.10.
python:3 base image seems to run on Debian GNU/Linux 11 (bullseye)
I apologize if I posted this in the wrong repo - I've been trying to find the correct place to post this error. Maybe you can guide me to the correct place if this is not the right one.
Any advice is appreciated!

Seems to be related to docker being installed through snap instead of apt which breaks things.
https://forums.docker.com/t/bug-on-apt-install-permission-denied/100196/6


Question for reproducing Table 2

Hi, I am very interested in your paper and experiments and I want to reproduce Table2 for Set9 Dataset.
I have one question. Actually, I already reproduced original deep image prior code for denoising and I tested it into  Set9. However, I got only 29 PSNR and 0.85 SSIM, worse than the result in Table2.
Would you explain what is different from DIP setting on your experiment?
Thank you.

Hi @gihyunkim, thanks for your interest and sorry for the late response.  When using DIP-like methods, please check these subtle aspects: 1) the inputs: whether it's from an uniform or Gaussian distribution, and the corresponding hyperparameters (e.g., mu, sigma). This could critically influence the results and the speed of overfitting, sometimes regardless of the rest of the architectural components (discussed in more detail in our new ECCV paper). 2) the upsampling type. Nearest neighbor fits faster than bilinear.  3) the random noise added to the input, i.e., "reg_noise_std", please set it properly according to different noise levels.
As I have not fully cleaned up the code, please ignore the "train_unsup_morph.py".  We did not propose a particular type of architecture but presented rationales for more principled architectural tuning for DIP. The architecture we used as comparisons with DIP in the experiments is also an encoder-decoder architecture.

Thanks for your answer@YilinLiu97. I understand the aspect of DIP-like methods (three things above), however, I could have not reproduced the score for original DIP in table 2 of your paper, although I have not changed any of the option from the default setting.
Would you check my setting for your code below and let me know how can I reproduce it? (only --decay and --reg_noise_std are different from the default config)
============================================================================
python -u denoising.py --special DIP_orig --task denoising --folder_path ../data/denoise/set9 --model_type DIP_orig --num_iters 3000 --norm_func bn --decay 1e-5 0 --reg_noise_std '1/30' --lr 0.01 --noise_type u --noise_method noise
it gaves me average 30.23 PSNR and only 0.86 SSIM (0.90 in table2) for Set9 dataset.
**also I use ssim metric from piq module since the compare_ssim doesn't work in my environment (it shows much lower ssim than piq).
Thank you.

Find out that decay set 1e-7 or 1e-5 in the script is not applied since the args.reg_type=0 and args.sr=0. Could be this problem?


Replace requests with pycurl-requests

Description
Replaced requests with pycurl-requests.
Motivation and Context
"The requests library is broken in many ways as to be useless for any server that is strict or slightly non-conforming. There are requests that work perfectly fine with curl but that get rejected when made with the requests library due to its various unnecessary sanitizing"
Encountered delays in response at several times with an instance of trying to get balance while in server. Was also unable to create a connection reliably using HTTPS. Such delays weren't experienced while using cURL to make request from the server.
@Flutterwave/Corvus97 What do you think about these updates?

Hello @chibie,
Thanks for taking out time to contribute to the Library.
Were you able to test these changes?

@Corvus97 yes i tested.
i currently use my fork in production.


Using new TrackedArray<string>() in TypeScript

Hi,
Is it expected to be able to write new TrackedArray<string>() in TypeScript?
I'm seeing the following:
// This works fine. The type of foo is string[].
const foo = TrackedArray.of<string>()
foo.push("item")

// This gives the error: "Property 'push' does not exist on type 'TrackedArray<string>'. ts(2339)".
// The type of bar is `TrackedArray<string>`
const bar = new TrackedArray<string>()
bar.push("item")

Should TrackedArray<T> be extending Array<T> in the type definitions?
Thanks.

hmm, I'm not sure it should, because we don't want to deopt the array constructor unintentionally by forcing an unrelated object to use its constructor (it'll have a different shape, so it could cause the array constructor, which is pre-JITed, to deopt when it sees its not a standard array).
It may be that we need to define TrackedArray as an interface and use some tricks to make TS think the class matches the interface. @dfreeman had a similar suggestion for TrackedObject: playground

The following seems to work for me:
declare module "tracked-built-ins" {
  interface TrackedArray<T> extends Array<T> {}

  export const TrackedArray: new <T>(array?: T[]) => TrackedArray<T>
}

However, I'm not certain how to get from() and of() in there as well. For my use case this doesn't matter though.
If I use the extended tracked wrapper in this package and do const myArray = tracked([]) then I have the same typing problems because that function returns a TrackedArray<T> which has essentially no methods on it. If it returned T[] I think it would work, should the return types here in decorator.ts be changed?

Actually the following is sufficient (I'm currently only using TrackedArray from this package):
declare module "tracked-built-ins" {
  export class TrackedArray<T> extends Array<T> {}
}

I understand that the library doesn't want to do class TrackedArray extends Array {...} internally for performance reasons, but as far as typings go the above seems potentially simplest?

Ended up adding the following to prevent a subtle error I encountered while using this library. Any thoughts on this?
declare module "tracked-built-ins" {
  export class TrackedArray<T> extends Array<T> {
    /**
     * This member protects against implicitly converting a variable of type
     * `Array<T>` to a variable of type `TrackedArray<T>` which would silently
     * lose autotracking. It prevents errors like the following:
     *
     *   class Foo {
     *     private names = new TrackedArray<string>();
     *
     *     setNames(newNames: string[]) {
     *       this.names = newNames;
     *     }
     *   }
     */
    private readonly _trackedBuiltIn: never
  }
}

I believe this was resolved by the changes made in #56 and #161 and released in 2.0. Can you confirm that that resolves the issues you were having, @richard-viney?

Yep all seems to be sorted in 2.0. Thanks!


add path-parameters-function go example

Follow up of https://github.com/OpenFunction/functions-framework-go/pull/52#issuecomment-1165214944
before merging the PR:

release ff-go and update the version in this PR.
update the default ff-go version in go-builder (optional, define the version in go.mod  should be enough)

To do: link the example to openfunction.dev
side topic:
I found that to enable dapr sidecar, user need to define the outputs.  Here I use the annotation to enable manually. Can refine this part in the future.

I found that to enable dapr sidecar, user need to define the outputs. Here I use the annotation to enable manually. Can refine this part in the future.

This is a good point, we can discuss how to handle this in the community meeting @tpiperatgod @lizzzcai @wanjunlei

Perfect!


Bundle 'amirh/HTML-AutoCloseTag' not found

I did not find Bundle'amirh/HTML-AutoCloseTag' when I installed.
I changed it to Bundle 'vim-scripts/HTML-AutoCloseTag'.
Whether to consider revising the problem？

#1053,check it

If only someone merged the fix. Right now it's blocking non-interactive installs.


Follow browser theme

Currently if you are opening the export for the first time and have the browser set to dark theme, theme switches to a dark one as expected, but in a few seconds it switches back to light.
And also, you can't switch the theme on mobile at all, you are stuck with light one

The theme switching is not based on browser theme but on the theme you exported with and there is no way around this flash because it has to start on some style. The reason it changes is likely because it is matching the system theme, which afaik is the only theme I can respect.
As far as changing theme on mobile, I will eventually add mobile support to the sidebars.

In the next update I will remove the feature that switches the theme to match the system theme because of this flashing. Instead it will just match the theme upon export by default.

@hobojoe720  Under appearance > Base color scheme is set to dark for me, but it always exports with the default as light mode, assuming that is a bug then. As I was going to post a request to have the default here selectable

The webpage caches the state of the theme toggle, so if you switched the theme to light then even if you export in dark mode if will switch to light mode automatically until you manually switch it to dark mode. Then if will stay in dark mode. Is this what is happening or is it actually starting in light mode even during the page load?

@hobojoe720  Under appearance > Base color scheme is set to dark for me, but it always exports with the default as light mode, assuming that is a bug then. As I was going to post a request to have the default here selectable

The webpage caches the state of the theme toggle, so if you switched the theme to light then even if you export in dark mode it will switch to light mode automatically until you manually switch it to dark mode. Then it will stay in dark mode. Is this what is happening or is it actually starting in light mode even during the page load?

Even on a fresh PC the export always come in light mode. It will remember for the file, but a new file will reset it back to light mode

Yes, that's the same behavior I am observing

This feature has been added in 1.8.0! :)
If you find any issues please create a new issue
Thanks!


Update sigh version

New parameter for sigh resign and resign action which changes current application Bundle ID

:+1:


为什么没有原始SiamFC_AlexNet的结果呢?SiamFC作为孪生网络的代表作应当被考虑在论文中

Thanks for your question

As a pioneer of the Siamese tracker, this work notices the far-reaching impact of SiamFC, and it has been reviewed in detail in our article, which you can refer to in the provided link. However, since the official Python code does not release,  for fairness, this work evaluates trackers with open source code.
The purpose of this work is to provide a Python code library for usage by the tracking community as well as practical UAV-based tracking deployments.
This work includes as many leading-edge and representative Siamese trackers as possible for evaluation. The results could be found in the 'results' folder or in our article.

感谢您的问题
1）作为 Siamese 跟踪器的先驱，这项工作注意到了 SiamFC 的深远影响，并在我们的文章中进行了详细回顾，您可以在提供的链接中参考。但是，由于官方 Python 代码没有发布，为了公平起见，这项工作使用开源代码评估跟踪器。
2）这项工作的目的是提供一个 Python 代码库，供跟踪社区以及实际的无人机跟踪部署使用。
3）这项工作涵盖尽可能多的前沿和有代表性的Siamese追踪器进行评估。结果可以在“results”文件夹或我们的文章中查阅到。


Performance issues with custom nodes (React)

What platform were you using when you found the bug?

React Flow / Svelte Flow version: React Flow<EMAIL_ADDRESS>Browser and version: Firefox 131.0 // Chrome Version 129.0.6668.90
OS and version: macOS 15 // Windows 10

Live code example
https://codesandbox.io/p/sandbox/blissful-sun-forked-mqs4fd
Describe the Bug
I'm getting rather intense frame rate drops when dragging the background grid when using custom nodes. Interacting with individual nodes or changing their parameters performs very well as expected, however moving the background or zooming causes frame drops.
Steps to reproduce the bug or issue

Go to the codesandbox above or here
Wait for sandbox to load and then in the preview panel press "Add Nodes"
Move around the background and monitor FPS meter. Note: The larger you make the preview window (or if you open the window into its own page) the worse the performance.

Expected behavior
I expected no frame drops and performance to stay steady considering the operation (to my knowledge) shouldn't be that intense.
Screenshots or Videos

Note the above gif sort of destroys the visible FPS discrepancy however you can still see the FPS meter drop frames as I move the canvas around. You can try it on your end as well, I've had multiple people try this out on all sorts of hardware and browsers and we all experience the same issues.
Additional context
No response

I am hitting almost 120 fps with 300 nodes on screen. Looking at the perfromance graph, most of it is spent on layouting 300 overlapping nodes. I would say this is a very unrealistic benchmark, and you are hitting the limits of what browser compositing is capable on your machine.

Interesting observation. I think you are right that this is an unrealistic benchmark, as spreading out the nodes more such that they are not heavily layered reduced the lag almost entirely! Is this just a browser thing or is there maybe some way to increase performance in the case of layered nodes? If not, all good. I think I overestimated the scope of the lag based on this heavily layered example thinking it was the number of nodes vs their location.

Got it thanks for the insight. I'll build conscious of that fact.

@MrgSub You might have forgot to use React.memo.
And what is happening in your flow? It is very hard to reason about this when there a million things changing on screen. How did you determine this is specifically connected to your custom nodes and not something else?

I had that issue, for me it was some css properties. To be specific backdrop-filter: blur(2px);. Removing that property solved it
See here: https://github.com/nanoapi-io/napi/pull/19/files#diff-496b310be54684663993206686f697dcf04e6a5fb3eccd97af1555a21eda8863L21


Create a release for v0.1.0

Create a unity package zip for the basic functionality and create a release on GitHub.
May want to add an Action to build the package. Check to see if a package builder action already exists here.

https://github.com/MichaelJBradley/hex-grid/releases/tag/v0.1.0


Latest Development Changes

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Does clang has cache mechanism？

When I use it parse a large project source code, for example:
header1.h include header2.h
when i modify header1.h but header2.h not modified, can It just re-parse header1.h and use header2.h  cache parse result(saved by first times parse)?
It can save many time !

Yes, Clang has AST cache that can save much time when parsing.
I started adding some support for it in https://github.com/mono/CppSharp/pull/1164 but more work still needs to be done.


dumpbag: use with open to simplify the file I/O logic

Please add your changes to the Apollo 3.0 dev branch. Thanks!


[CLI]add cloud/recorder debug cli

Phenomenon and reproduction steps
Root cause and solution
to import debug efficiency
Impactions
Test method
cli test
Affected branch(es)

main

Checklist

[ ] Dependencies update required
[ ] Common bug (similar problem in other repo)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


No templates matched the input template name: blazorserver.

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
dotnet new blazorserver -o BlazorApp --no-https
No templates matched the input template name: blazorserver.

It looks like you don't have blazorserver template. Please make sure that you have the latest .NET SDK https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/install


csv: fieldsPerRecord: -1 doesn't seem working

The description says the below, but it seems to error on short number of fields even when it's set to a negative number.

If negative, no check is made and records may have a variable number of fields.

I did a quick fix in https://github.com/denoland/std/pull/5605/commits/cd43e7836c365a4723ea50be0dd489e00e744d32

No this isn't #5606 but #5605

Ah, ok. thanks!


Update spdx library versions and fix purl

use proper purl syntax
remove unecessary rdf store creation
update example again

k we're still missing the repository parameter on purl, but we can add that in after


Wrong UnitedKingdom holidays for 2012

workalendar==4.0.0 from PYPI
python 3.6 on LINUX
Hi, thanks for this library that is exactly what I'm looking for.
One issue I have is with holidays in the UK that seem to be wrong for years when the government "randomly" decided to add some. The last example is the Queen’s Diamond Jubilee.
Here is the official link: https://www.gov.uk/bank-holidays with 2012 at the end of the page.
The great thing about this year is:

There is an additional bank holiday on June 5
The usual Spring bank holiday is not the last Monday of May (May 28) but  June 4.

I don't have a complete list (yet) but there are other years with similar holiday like 2002.
Are these holiday something you see being in the scope of workalendar?
I'll probably fork your repo to manage them anyway so I'm interested to know if you are open to a PR to manage these cases. If it's the case there may be a way you prefer me to go to manage this.
Bellow the code I run with the missing and wrong date:
from workalendar.europe import UnitedKingdom
cal = UnitedKingdom()
cal.holidays(2012)

Gives me:
[(datetime.date(2012, 1, 1), 'New year'),
(datetime.date(2012, 1, 2), 'New Year shift'),
(datetime.date(2012, 4, 6), 'Good Friday'),
(datetime.date(2012, 4, 8), 'Easter Sunday'),
(datetime.date(2012, 4, 9), 'Easter Monday'),
(datetime.date(2012, 5, 7), 'Early May Bank Holiday'),
(datetime.date(2012, 5, 28), 'Spring Bank Holiday'),
(datetime.date(2012, 8, 27), 'Late Summer Bank Holiday'),
(datetime.date(2012, 12, 25), 'Christmas Day'),
(datetime.date(2012, 12, 26), 'Boxing Day')]
Thanks.

I've made the pull request, let me know if there is anything I need to change.

Your patch is part of the v4.1.0, released and published on PyPI. Enjoy!


bug-ish: 1.4.1 hasn't been published to npm

According to the commit history and the repo's releases/tags, 1.4.1 is currently the latest version - however, 1.3.5 is the latest version that has been published to npm.
(No judgement here, by the way - forgetting to publish a version is perfectly understandable, especially as it's been pushed to the repo ^^)

@Rexogamer 1.4.2 has now been published to NPM.


Admin Panel: Handle

At the moment it's not obvious that the user needs to add @ before whatever handle they want.

Isn't this going to be tied to Onename's blockchain ID? If so, then I guess we'll be circling back to this issue.

Yes it is but this is a temporary hack-in for development so validation is fine being missing right now.


Form Recognizer: Preview 3 Renames

[ ] TrainingFileFilter -> TrainModelOptions ?
[x] trainingFiles param -> trainingFilesUri
[x] useLabels -> useTrainingLabels
[x] Error.Code -> Error.ErrorCode

@annelo-msft I see some things are checked in the description of the issue but I don't see the changes in Master.
Is there a PR I am missing? or are u currently working on it?

@maririos They're updated in local changes in a branch of mine. I haven't assigned the issue to me because I'm not sure what to do with the first item, and wanted to make it available for other people to grab it.

There is an issue for that => https://github.com/Azure/azure-sdk-for-net/issues/11930
So if you want just remove the point from here and assigned the issue to you so once your PR is merged we can close it here


Docker for development environment

Steps

npm install in back end folder
npm install in front end folder
docker compose -f docker-compose.dev.yml up --no-recreate

PS: do not forget to export the environment variable for the ports or create a .env file containing the variable of:

FE_PORT
BE_PORT


Player pause race condition

There is a race condition that can occur when the VAST code attempts to call pause on the player, but the player has not loaded the video and began playing yet. This PR adds an interval that does not proceed until the player has actually been paused.

can you provide an example that triggers the race condition ?

Here's a test URL: http://www.wootables.com/videojs-alex/bower_components/videojs/dist/examples/simple-embed/test.html
Note: this is code running WITHOUT this PR (so the ad plays over the video)

Sorry, this PR was originally from the wrong branch. Only 1 file should be changed. Fixing now.

@massa84 Are you able to confirm this issue with my test?

hi @alex-phillips , yes i've  confirmed your issue but I'll propose you an alternative solution (as I don't really like that 1ms interval approach)
can you verify that the branch pause-race-condition-fix fix your issue ?

@massa84 Yup, it looks like that solution fixes it as well! Any estimate on when that might be merged?


Refactor update_icons

Description
Some preliminary refactoring needed to implement adaptive icons.
Related Issues
Enables https://github.com/flutter/flutter/issues/68016
Tests
No tests needed, since the script produces the exact same material icons.dart file.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

What's the plan to get from here to adaptive?

What's the plan to get from here to adaptive?

A private class to make Icons.adaptive.X possible, and an update to the script to auto-generate that private class' getters based on icons that include "ios" in the name

What's the plan to get from here to adaptive?

A private class to make Icons.adaptive.X possible, and an update to the script to auto-generate that private class' getters based on icons that include "ios" in the name

Do all the adaptive icons have '_ios'?

What's the plan to get from here to adaptive?

A private class to make Icons.adaptive.X possible, and an update to the script to auto-generate that private class' getters based on icons that include "ios" in the name

Do all the adaptive icons have '_ios'?

Yes, except a special case for more (more_vert, more_horiz)

LGTM


Apple Silicon (M1) - backend won't start (dyld: missing symbol called) - After upgrading to 1.0.x

Expected Behavior
When running yarn start-backend on a backstage app created by npx @backstage/create-app it should start up on an Apple Silicon (arm64) machine.
Actual Behavior
The backend fails to start with the following message:
yarn run v1.22.17
$ yarn workspace backend start
$ backstage-cli package start
Build succeeded
dyld[89372]: missing symbol called

Steps to Reproduce

Run npx @backstage/create-app (choose sqlite)
After the app is ready go to the root and run yarn start-backend (on an Apple Silicon machine)

Context
It has worked previously on  "version": "0.69.1" (from backstage.json).

Reinstalling nodejs using x86_64 architecture (using this guide for example: https://gist.github.com/adrienjoly/e29a6e73fb7e701eefd80ff9bde9abeb) is currently the workaround.

Same here, is it a new package that backstage is using which is incompatible with M1?

Hmm so this is pretty weird. I got a new M1 yesterday, and it works fine using 14.x, but 16.x throws this. Can you both confirm that it works fine on 14.x? And maybe help out trying to work out what library it is that doesn't like 16?

Hmm so this is pretty weird. I got a new M1 yesterday, and it works fine using 14.x, but 16.x throws this. Can you both confirm that it works fine on 14.x? And maybe help out trying to work out what library it is that doesn't like 16?

When I install node 14.x using nvm it actually installs the x86 version. You can check that by running node -p process.arch when using the 14.x. That would explain why it works.

Nice catch, so this is where it gets interesting. A new create-app throws this error, but the main repo works fine in arm64. Guessing yarn.lock installs later versions of some binary dependency, but not sure which one it is. Need to find all the packages that have bindings and add logs before and after requiring them to see where it bombs out and lock down the dependency somewhere to one that we know works in arm64 I guess.

Hello there, I just faced this very same problem, so for now the solution is to run node 14.x? Asking so I can unlock myself and move forward. Please let me know, thanks a lot for your time and help.

Hello there, I just faced this very same problem, so for now the solution is to run node 14.x? Asking so I can unlock myself and move forward. Please let me know, thanks a lot for your time and help.

You can install the intel version of node 16.x, that works too, guide here: https://gist.github.com/adrienjoly/e29a6e73fb7e701eefd80ff9bde9abeb

Ran into this in a separate setup, even though I used a compatible node-gyp, so disregard the previous comment.
I narrowed it down to a problem with dockerode, it seems to happen when trying to instantiate a new Docker() client. I'd expect the actual issue to be in one of the lower level transitive dependencies, so probably https://github.com/mscdex/ssh2/issues/1146 and https://github.com/mscdex/cpu-features/issues/7

A newer version of better-sqlite3 is available (7.5.1).
If you bump to this newer version, the problem should be fixed !

We believe this is fixed now - please reopen if the problem persists after the mentioned upgrades.


FR: Add option to 'remember' headers

So when you open a new query tab, the same headers would be sent. Cannot think of any use case where this isn't desired, so maybe even make this the default behavior.

I wonder if global headers (shared across tabs) wouldn't be more useful. If you have 12 tabs already and you need to change your auth header, it gonna be painful. And if an header is redefined locally for the tab, it would override the global one.


Private mount not found

Hi @m1k1o,
Sorry maybe got wrong last time I checked but do I have to create the folder prior to mount private paths?
I'm having an error:
invalid mount config for type "bind": bind source path does not exist: /root/meetnav-browser/neko-rooms-data/rooms/qcc-qemv-nrw/mozilla
when creating a new room which I want its "mozilla" profile to be stored and host
Thanks

My bad!
Forgot to update docker-compose.yml after changing data location
volumes:
      - "/var/run/docker.sock:/var/run/docker.sock"
      - "/root/meetnav-browser/neko-rooms-data:/data"

Fixed! Sorry


fix filename sanitization/spelling

fixes spelling error?, #36, and removed unnecessary code

Ah, I see. This was probably written when we were trying to sanitize the full path, but now that we're sanitizing only the file name we don't need to reserve the start of the string anymore.
Thanks!


Fix bug in hangup

I accidentally introduced a bug in the hangup function. If you run it without arguments, it kills everyone elses session on the box.
This is great and all, but not exactly what I intended to do.
I should probably rewrite this function to work better.

Example:
$ getidle
PTY 0 is 25621 seconds idle and owned by user1
PTY 1 is 0 seconds idle and owned by root ** this is us **
PTY 2 is 663 seconds idle and owned by user2
PTY 4 is 15657 seconds idle and owned by user3
$ hangup
This is seriously rude...
Terminating PTS/
Owner of PTS is root
SSH PID is 24893
36610
42622
sh: 433: cannot create /dev/pts/: Is a directory
$

In this case I was not in over SSH, but over another PTY spawning backdoor - this also begs the question of why I am only targeting SSH connections - there are times when we might want to kill a user who is accessing via telnet or another remote access tool which allocates a PTY, for example.

MESS WITH THE BEST
i'll have a look when i'm home and throw in some handling to stop it killing every single session

I have added some checks in the hangup function in the pull request #30.
Please test it.
I will try to add argument 'all' for all sshd clients and a list of PTS numbers.

In pull request #31 special arguments were added:
"hangup all" to terminate all and "hangup all other" to terminate all by not our PTS is now supported.

Appears to work fine on Debian/Ubuntu hosts I've been testing on. I'll probably have to add CentOS and such to my set of VM's though going forward as per your notes in https://github.com/zMarch/Orc/issues/21


Install Dependencies failing on Ubuntu 22

Hi,
The install_dependencies.sh script is failing on Ubuntu 22 because of python-dev and libdc1394-22-dev

Thank you for reporting, we will take a look at it! Also sharing @hardesh fixed script:
luxonis_installation_script.zip

Downloading the script and simply replacing libdc1394-22-dev with libdc1394-dev fixed the issue, I was able to install dependencies without any errors.
Found solution from this stackoverflow answer.


Shell Script here document syntax highlighting fails to recognize the end delimiter in some cases

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

VS Code Version: 1.87.2 (user setup)
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Open a new text file Ctrl+N (no need to save it).
Paste a problematic example content like the following one:

#!/bin/sh

cat <<- E+OF
here document
E+OF

# Normal highlighting should resume here but in VS Code it does not.

Once the word defining the here document delimiter contains a character like + or = (probably there are additional problematic characters), the redirection operator is the variant with removing of leading tabs <<- and there is a space between the operator and the word then VS Code does not recognize the end of the here document delimited by the line with the delimiter (here E+OF).

Example of incorrect highlighting - the comment (and possible following lines) is highlighted as if it was part of the here document:

These slight modifications are highlighted correctly:
#!/bin/sh

cat <<- E-OF
here document
E-OF

# Normal highlighting resumes if "+" is replaced by "-".

#!/bin/sh

cat <<-E+OF
here document
E+OF

# Normal highlighting resumes if there is no space between the redirection operator and the word.

Note: I have encountered this bug because I normally use +++EOF+++ as a delimiter.

Thanks for providing clear screenshots and code samples! We just made the stable release yesterday and the fix should be available there.

Thanks. I have upgraded to 1.88.0 stable and the highlighting works correctly.


negative training loss

Hi Jia, Thanks for sharing your well-written codes. Just a kindly reminder that when exploring your repo, I noticed some negative training loss appeared after ~50 epochs. I guess it might be caused by some typo, which should not affect the algorithm correctness (as their validation loss looks all good). Looking forward to your comments, and wish all the best.
Some log info:
epoch: 58/384, train loss: 0.0207
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000058.pth
epoch: 59/384, train loss: 0.0062
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000059.pth
epoch: 60/384, train loss: -0.0002
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000060.pth
epoch: 61/384, train loss: -0.0171
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000061.pth
epoch: 62/384, train loss: -0.0477
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000062.pth
epoch: 63/384, train loss: -0.0609
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000063.pth
epoch: 64/384, train loss: -0.0646
Saving exp/scannetv2/pointgroup/pointgroup_run2_scannet/pointgroup_run2_scannet-000000064.pth
Start Evaluation: epoch: 64/384, val loss: 2.0182

Hi,
The offset direction loss could have a negative value.  It is to be expected to have a negative training loss.

Hi Liang, Thanks for your quick reply and comments!


Arrays of numbers print as empty objects

Given this JSON input ...
"address": {
"address1": "3925 Brookside Pwky",
"city": "Alpharetta",
"region": "GA",
"zip": "30040",
"timezoneId": "America/Chicago",
"loc": [-84.2479169,34.0592993],
"country": "US"
}
The plugin prints the "loc" values (each a number) as two empty objects ...
address : {
address1 : 3925 Brookside Pwky ,
city : Alpharetta ,
region : GA ,
zip : 30040 ,
timezoneId : America/Chicago ,
loc : [             {             } ,             {             }          ] ,
country : US
Converting the numbers to strings makes them print as expected. Note that the empty object error happens whether the number is an integer or a float.

@cephirothdy2j,
Sorry for the delay to fix this issue.
I solved the problem, please, confirm if it works to you.


Doesn't work on first page load

Service worker is not yet active when page is loaded for first time, thus image is not loading on first page load.
How to reproduce: open in incognito mode.

Should work now. Note that this requires page restart (only on first visit) because FetchEvent for e.g. <img> tag will be made by the browser much earlier than Service Worker is activated. Library code handles this automatically, also you can override reload behavior with onUpdate option.


Add docs for operator : filtering - skipWhile

Add docs for operator skipWhile under group filtering
Below is minimum details needed

[ ] name
[ ] operatorType
[ ] signature
[ ] parameters
[ ] short description
[ ] Technical description
[ ] Visual example  (Marble diagram)
[ ] Real world example (Non-framework)
[ ] Related Operators

Closing as duplicate of #99


Add short installation/configuration section to README.md

This documents how to install both from PyPI and from source.  NOTE however, Zino is not yet published on PyPI, but the guide should work once it's published.
Closes #156

Since we added the small configuration section, I realized we should also document how to configure user accounts for the API, so I've added yet another section for that.  Feel free to have another look :)


Export all hash types

During the 0.30.0 release we removed the re-exports of hash types. This upset some folk and since the aim of our re-exports is not exactly clean as well as the fact that the public API surface is not yet fixed just re-export all the hash types at the crate root again.
This is #1792 but does not use a wildcard and also grabs the other hashes that we recently moved.

cc @stevenroose because I closed #1792 and re-did it here.

Yes I think we should accept that we have not got the re-exports (and the public API surface) exactly as we want it and don't know exactly what that is yet - we should not break downstream with half thought out changes. (That was all me by the way.)

This PR is a candidate for bikeshedding but at its core is super trivial. IMO it qualifies for the one-ack carve-out, if no one comments or nacks please consider for merge after the 25th of August.


Patch for SAMOA-58 (Samoa AvroFileStream from HDFSFileStreamSource stops at end of first file)

FileStreamSource seemed to support multiple files but during my testing it turned out otherwise - Samoa AvroFileStream from HDFSFileStreamSource stops at end of first file. I had to change AvroFileStream, ArffFileStream and their parent FileStream in order to make this work.
See following JIRA for additional detail:
https://issues.apache.org/jira/browse/SAMOA-58
Additionally, I modified bin/samoa, pom.xml, SystemUtils (as well as added a resource) to fix reading from HDFS on my cluster.
A seemingly unrelated change is the explicit test for supported Avro types so as to filter out any fields that are not supported instead of assuming all non-nominal (non-enum) fields are numeric and failing during reading.

Thanks for the patch @edi-bice !
There are a few changes that go beyond the scope of the patch, and their goal is not entirely clear to me.
For the rest, apart from some small doubts, the patch looks good.
Please have a look at my comments, and see if they make sense.
I'd like to get the patch in soon.

@gdfm apologies for the delay - was away at a conference for a week.
Thanks for reviewing my patch and being receptive to the changes. See my comments for answers to your questions. Will push the changes you've asked for hopefully in a couple of hours.

Pushed the changes you requested @gdfm

+1 and thanks for the contribution!

@abifet @nicolas-kourtellis could you also review the patch?

+1
I didn't run the Avro fix, but the rest of the additions look good. Thanks edi-bice!
Note: What I am not sure is if all these additions should have been committed as a single patch, or independent ones. The patch covers several different things and if someone wants to roll out one of them, it may not be easy..

+1 I like the idea of having different weights for different classes. The F1ClassificationPerformanceEvaluator is giving also recall and precision, that's a nice contribution. We may think in the future to add a windowing version of it. Thanks @edi-bice!

I agree with you @nicolas-kourtellis about the patch being multiple seemingly unrelated things. Typical rookie mistake - took off on my fork making all the changes I needed to get things working for me. Didn't squash either. Had it not been so long since I last worked on this, I'd offer to start splitting things.
@abifet let me know what the windowing version should look like and I can work on it.

@edi-bice  It's like WindowClassificationPerformanceEvaluator. This is a windowing version of BasicClassificationPerformanceEvaluator that takes into account the performance on the last instances, instead of all the instances from the beginning of the stream.


test JWS against go-jose

We need to ensure Aries-framework-go JWS messages are interoperable with Go Jose using DER key format, if it isn't, then we need to add IEE_1363 key format support as well.

IEEE_P1363 key format is now added in the KMS (#1700).
The only work remaining is interop testing between go-jose and keys with this new key type.


[Bug] Don't work json schema with model creation

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
https://microsoft.github.io/monaco-editor/playground.html?source=v0.41.0#example-extending-language-services-configure-json-defaults
Monaco Editor Playground Code
// Configures two JSON schemas, with references.

var jsonCode = ["{", '    "p1": "v3",', '    "p2": false', "}"].join("\n");
var modelUri = monaco.Uri.parse("a://b/foo.json"); // a made up unique URI for our model
var model = monaco.editor.createModel(jsonCode, "json", modelUri);

// configure the JSON language support with schemas and schema associations
monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
	validate: true,
	schemas: [
		{
			uri: "http://myserver/foo-schema.json", // id of the first schema
			fileMatch: [modelUri.toString()], // associate with our model
			schema: {
				type: "object",
				properties: {
					p1: {
						enum: ["v1", "v2"],
					},
					p2: {
						$ref: "http://myserver/bar-schema.json", // reference the second schema
					},
				},
			},
		},
		{
			uri: "http://myserver/bar-schema.json", // id of the second schema
			schema: {
				type: "object",
				properties: {
					q1: {
						enum: ["x1", "x2"],
					},
				},
			},
		},
	],
});

monaco.editor.create(document.getElementById("container"), {
	model: model,
});

Reproduction Steps
try type json, Monaco editor autocomplete doesn't work
Actual (Problematic) Behavior
I expect json autocomplete. Autocomplete and syntax highlight not work.
If I use creation without model - all is fine, but, I can't use URI for editors windows
for example
var jsonCode = ["{", '    "p1": "v3",', '    "p2": false', "}"].join("\n");

// configure the JSON language support with schemas and schema associations
monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
	validate: true,
	schemas: [
		{
			uri: "http://myserver/foo-schema.json", // id of the first schema
			fileMatch: ["*"], // associate with our model
			schema: {
				type: "object",
				properties: {
					p1: {
						enum: ["v1", "v2"],
					},
					p2: {
						$ref: "http://myserver/bar-schema.json", // reference the second schema
					},
				},
			},
		},
		{
			uri: "http://myserver/bar-schema.json", // id of the second schema
			schema: {
				type: "object",
				properties: {
					q1: {
						enum: ["x1", "x2"],
					},
				},
			},
		},
	],
});

monaco.editor.create(document.getElementById("container"), {
	language: 'json',
	value: '{\n\t\n}',
});

Expected Behavior
I expect syntax highlight and json autocomplete when I create editor with model
Additional Context
Bug in 0.41 version and higher. Works good in 0.40. Critical.

I can confirm the same behavior:
Up to v0.40, the remote fetch of a schema worked well:

Since v0.41 the schemaRequest is not transmitted anymore, no schema is loaded and hence editor highlighting and automcomplete features are not available.
The onMount configuration of the editor has not changed in the meantime and looks as follows:
`  private onMount(editor: editor.IStandaloneCodeEditor, monaco) {
const defaultOptions: editor.IEditorOptions & editor.IGlobalEditorOptions = {
tabSize: 2,
readOnly: options.readonly,
};
editor
  .getAction('editor.action.formatDocument')
  .run()
  .then(() => {
    if (options.schemaUrl) {
      monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
        validate: true,
        enableSchemaRequest: true,
        schemas: [
          {
            uri: options.schemaUrl,
            fileMatch: ['*'],
          },
        ],
      });
    }

    editor.updateOptions(defaultOptions);
  });

}`

This is fixed now.


Adding organisationSize column to service export

Trello: https://trello.com/c/Abra4gwN/454-export-g10-services-data-for-ash
Adds organisation size to export.
The files are generated separately by lot, so we shouldn't need to add that in.
Paired with @agalamatis 👍

Looks good to me, just needs flake8 to be happy

@lfdebrux @agalamatis After feedback from Ash I've added in a helper function to extract the service categories for Cloud Support. Also renamed the script and tidied the docstring 😸

I like the changes 👍 Have they been tested?

@lfdebrux I generated a new file, Ash is happy 👍


Description of an action to long in svg

Hi,
Some of the descriptions of my actions are to long for the .svg detailedfiles. With the .PNG it looks fine, but the .SVG doens't format them correctly. Is there a way I can make the word document use the .PNG as standard?
This is the .SVG:

This is the .PNG:

Hi @Ambesima , just added some code (https://github.com/modery/PowerDocu/commit/464f5f1f559e82c36a887609929116437d3e2e38) to introduce line breaks into the descriptions (Notes). For example:

This will be part of the next release (sometime in the next 2 weeks), I'll update this issue here again once I published it

Hi @modery , thanks for the update!
I’ve downloaded the self contained version and tried to verify to fix. Unfortunately I’m still having the issue with the flow-detailed.svg. I’ve tried multiple ways (see below) of writing my notes, but they all seem to have the same result. Anything else I can try or provide for you to solve the issue?

Write the notes without enters in the action.
Write the notes with enters in the action.
Write the notes with ‘’ in the action.
Write the notes with ‘’ in the action.
After each option, I’ve published and exported my solution.

Hi @Ambesima, you were a bit too fast - the code thar fixes this problem is already available, but I haven't published a new release that contains the fix yet. In final stages of testing at the moment, new version should go live in the next few days

Release is now available!
https://github.com/modery/PowerDocu/releases/tag/v-2.1.0

Thanks, works like a charm!


Fix code consistency between AudioStreamPlayer2D and AudioStreamPlayer3D

This started with the intention to fix\ https://github.com/godotengine/godot/issues/99699 but it doesn't work that way.
I saw some very noticeable difference between 2D and 3D, in both play() and seek(). These came from https://github.com/godotengine/godot/pull/91382

Thanks!


How to remove a battery from the chart?

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
How to remove a battery from the chart?
I don't have energy storage and I would like to use this flow card with my inverter
Expected behaviour
add function
Possible Solutions
No response
Mode
both
Context / Reason
show_battery: false

Will take a look at adding this

Added to 2.5.1


BUG Don't filter modules by type

Fixes https://github.com/silverstripe/silverstripe-framework/issues/7948
Also fixes an issue where root expose was ignored if the type wasn't a silverstripe- prefix.

Is there any possibility of other modules using extra: [ expose: [] ] that we might accidentally do something odd with here? Otherwise yay, this won't be needed: https://github.com/silverstripe/silverstripe-framework/pull/7950

Maybe but it's something we'll have to deal with if we came across it in the future.


OSError: It looks like the config file at './pretrained_model/wav2vec2-base-960h/config.json' is not a valid JSON file.

I have downloaded wav2vec2-base-960h/config.json from https://huggingface.co/facebook/wav2vec2-base-960h/blob/main/config.json, execute python -m scripts.app and get this error
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py", line 659, in _get_config_dict
    config_dict = cls._dict_from_json_file(resolved_config_file)
  File "/usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py", line 751, in _dict_from_json_file
    return json.loads(text)
  File "/usr/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/AniPortrait/scripts/app.py", line 43, in <module>
    a2m_model = Audio2MeshModel(audio_infer_config['a2m_model'])
  File "/AniPortrait/src/audio_models/model.py", line 25, in __init__
    self.audio_encoder_config = Wav2Vec2Config.from_pretrained(model_path, local_files_only=True)
  File "/usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py", line 547, in from_pretrained
    config_dict, kwargs = cls.get_config_dict(pretrained_model_name_or_path, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py", line 574, in get_config_dict
    config_dict, kwargs = cls._get_config_dict(pretrained_model_name_or_path, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py", line 662, in _get_config_dict
    raise EnvironmentError(
OSError: It looks like the config file at './pretrained_model/wav2vec2-base-960h/config.json' is not a valid JSON file.

Have you tried changing this to the exact path in your environment?
./pretrained_model/wav2vec2-base-960h/config.json

It is not a path problem, I have put the json in ./pretrained_model/wav2vec2-base-960h/. I even added in /usr/local/lib/python3.10/dist-packages/transformers/configuration_utils.py, in line 659 this
            with open(resolved_config_file, "r", encoding="utf-8") as reader:
                print(reader.read())

And print what is inside the json. So it is not a path problem

I ran it in a docker which is documented here https://github.com/maximofn/aniPortrait_docker, in case you want to take a better look at it

You can check with the following steps:

make sure pretrained_model/wav2vec2-base-960h/config.json is complete
make ure pretrained_model/wav2vec2-base-960h is complete, or you can manual download wav2vec2-base-960h
update the veision of json

Thank you, I fixed the error.
The problem was that I was downloading the json like this wget https://huggingface.co/facebook/wav2vec2-base-960h/blob/main/config.json and now I download it like this wget -q https://huggingface.co/facebook/wav2vec2-base-960h/raw/main/config.json.
That is, I changed blob to raw.


Feature: Open projects in dev containers?

In our team we use devcontainers for our projects. Would be nice if there's a possibility to open projects in containers from the dashboard view.

I have submitted a pull request for this issue.
https://github.com/DanielNoveo/vscode-dashboard/pull/2


League\Interfaces\Component\Query + ArrayAccess

It would be great if the Query interface could support the ArrayAccess interface, which would allow $url->query['pants']:
interface Query extends Collection, Component, ArrayAccess

You can then raise a BadMethodCallException when offsetSet or offsetUnset are called for immutability.

I had a better look at this and whipped up a version on my own fork: https://github.com/shabbyrobe/url/tree/arrayaccess
I don't know whether it's worth merging or not, I just prefer the array notation as less verbose and less visually noisy, particularly when there are a lot of values to be extracted from a query. Please let me know if it's worth submitting (pending any cleanup) or if I should close it off.

HI @shabbyrobe,
The package new version is now developped on  https://github.com/thephpleague/uri to avoid dependency hell and for better clarity.
That being said..I'm a little on the fence for adding the ArrayAccess interface to the QueryInterface. Because users may think the Query will return the same data has PHP's parse_str which is not true. in v4 the Query returns the key pairs so for instance using the following code would be unintuitive:
$query = new Query('foo[bar]=1&foo[baz]=2');
$query->getValue('foo'); // will return null
$query->getValue('foo[bar]'); //will return 1
$query['foo']; //people would assume ['bar' => 1, 'baz' => 2];

I may be wrong about it but this is what I think ? What do you think ?
This is the same reasoning behind not implementing the ArrayAccess Interface for the Host object for instance

ArrayAccess has been added to the  https://github.com/thephpleague/uri package


Rename log_model_version_metadata to log_model_metadata

Describe changes
I Renamed log_model_version_metadata to log_model_metadata to make it more consistent with log_artifact_metadata + i made it part of the public api so you can import it like;
from zenml import log_model_metadata

Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

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

@coderabbitai review


minor change for patch_local_cluster_details

minor change to patch_local_cluster_details in upgrade path to overwrite labels if they exist.
this is needed to ensure update manifests does no error during upgrade from v1.2-head to v1.3-head
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

Solution:

Related Issue:
Test plan:

@mergifyio backport v1.2


Need to restrict once user done the setup of the app

Need to restrict once user done the setup of the app. Currently I am able to get back on the setup screen which is not the ideal flow @akshay8551
Check
https://github.com/bithyve/bitcointribe/assets/107177634/54b717d0-60f2-43ca-ac37-8bf1161770cd

Closing this since we have new repo for New Tribe RGB


Doc - Fix typo metrics pages

Remove indentation that make the table CASRead not well generated

Merged in https://github.com/apache/cassandra/pull/1909


OFBIZ-11468 Improved: Convert ShipmentReceiptServices.xml mini lang to groovy

(OFBIZ-11468)
Also converted getTotalIssuedQuantityForOrderItem in IssuanceServices, because it is used in ShipmentReceiptServices and needed more return values.

Commit https://github.com/apache/ofbiz-framework/pull/61/commits/ebe1c38c06176a6d259fc11bc45e5a2ed25a436a introduces a change to the readme that should not be part of this PR.

I will provide a new PR, to avoid the merge conflicts.