YouTube links not loading the title and description of the video in bookmark form

Adding a YouTube link in bookmark form never loads the actual title and description of the video, only some stub text. I could reproduce this issue both in a hosted instance running version 1.34.0 and in Linkding's demo instance (https://demo.linkding.link), currently in version 1.36.0.
Interestingly enough, I couldn't reproduce this problem while running the master branch (at commit 6548e16) in a dev-container in my machine. It displays as also running version 1.36.0.

Mhh I can't reproduce this either. But I remember Youtube sometimes returning some generic metadata, so I assume this is just Youtube sometimes preventing access from bots.
In any case, this will probably not be fully solvable with server-side scraping. I'd suggest you use the browser extension for saving such bookmarks, which now supports loading the metadata from the current browser tab. Or wait until the same is supported for the bookmarklet, which should be covered by https://github.com/sissbruecker/linkding/issues/118.

I just want to note that I have been experiencing this same issue for some time (a few months, I would estimate) with the browser extension.
Browser: Chromium Version 130.0.6723.116 (Official Build) for Linux Mint (64-bit)
Extension: Version 1.12.1

Apologies, I did not see the note about the extension using the server to fetch the info by default. I found the option to enable browser metadata and it now works for me.


Chinese version of the documentation

The Chinese version of the documentation is now available at https://shaobin-jiang.github.io/jsPsych-Chinese-Documentation/.
Wondering if it would be nice to integrate it into the README.md of the current repository

Wow, this is great @Shaobin-Jiang! Thanks very much for putting this together! Yes, we can link to this in the README and on the website.
Just so you know, we're working on a major release (7.0) which will be out in the next month or so. This will involve some changes to how jsPsych experiments are set up, which means there will be changes to the documentation. The general structure of the documentation pages will remain the same, but all of the code snippets will be a little different, and there will probably be some additional pages/sections. I'd be happy to tag you on these changes as we make them, so that you can update your Chinese version accordingly. Or you could keep your documentation as-is, since I'm sure some people will still be using v6.3.1 for a while.

That would be great @becky-gilbert! Appreciate that. I think I will be keeping the Chinese documentation updated, and I will probably get to working on that upon the release of version 7.0.

Back again, and apologies for this delay in my work progress. I have now finished with the translation of version 7.1 documentation into Chinese, which is available at https://shaobin-jiang.github.io/jsPsych-Chinese-Documentation/latest/.
The documentation for version 6.3 can still be viewed. Only I have not figured out how to use mkdocs  + mike to add a version selector to the page, so instead people can view the v6.3 docs at https://shaobin-jiang.github.io/jsPsych-Chinese-Documentation/6.3/ now.


Double Login after Registration ByRole

Hi programmers, i don't know is it a problem or a question.
There is two roles in my application, and after registration i give them roles like this:
UserManager.AddToRole(userId: user.Id, role: "Teacher");
and it redirects to action of Controller which is only for "Teachers" role, made like this
[Authorize(Roles = "Teacher")]
public class TeacherController : Controller

The problem is that application wants users to relogin to the application, and only then gives them permission to view webpage.
How to make sure that he doesn't pass authorization again?
Thanks

Solved the Problem)
After adding User to new Role just signIn him using code like this
UserManager.AddToRole(userId: user.Id, role: "Teacher");
ApplicationUser applicationUser = UserManager.FindById(tutor.UserId);
SignInManager.SignIn(applicationUser, isPersistent: false, rememberBrowser: false);


Creating an issue template for WinUI toolkit bugs

Description

New issue template for filing bugs related to the WinUI design toolkit.
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

/azp run


Updated postman settings

As Web API Controller Class does not exist, I took Controller Class - empty. It has no templated class methods so I always received a 404 :-( Therefore I've added a HTTPGet so I get the 200 :-) Additionally it does not work for me with localhost:5001 but with the port specified in ssl-port.

Thank you @K232 for your thorough input. Please can you confirm that these changes apply to VS Mac? The original steps were based on this environment?

Hi @mikeparker104, no idea how it looks on VS for Mac. Prerequisites say it's for Mac or Windows so I took windows. I don't know whether wording is different on Mac and Windows (which would be surprising?). If so, maybe add information about the naming on windows. I just tried to follow the instructions and wanted to let you know of the differences I had found on Windows.

Hi @K232. Really appreciate you pointing out these challenges. I can see how the following wording might be ambiguous: 'The steps provided are for Visual Studio for Mac but it's possible to follow along using Visual Studio 2019'. Making this clearer and adding further information specific to Windows, where it makes sense, will make it easier to follow. Thank you again for taking the time to share this feedback.

Thanks for the feedback @K232. I've adapted and combined it with other feedback into a single commit.

We'll close the PR. Thanks!


Update dh-backend-ci.yml

Description

References

#0000

Codecov Report

Merging #972 (154a4d8) into main (cb07480) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #972   +/-   ##
=======================================
  Coverage   62.75%   62.75%
=======================================
  Files          20       20
  Lines         290      290
  Branches       26       26
=======================================
  Hits          182      182
  Misses         94       94
  Partials       14       14

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


Fix action in tf2 diagram and add sentence.

Have added a sentence about 2 unreachable states.
@Nikoleta-v3 could you take a look at the change I've made to the tf2 diagram please, based on this line from the FSM
25      (3, C, 0, C), (3, D, 1, D),

Thank you for pointing that out. I will go through them once again now to make sure the rest is good to go!

It was the tests that caught this one when I added the strategies to the library, so thanks to whomever added those!

@Nikoleta-v3 and I have checked all the fsm diagrams and they are correct:
import collections
import imp
players = imp.load_source('players', '../src/players.py')

transition = collections.namedtuple("transitions", ["current_state", "input_action",
                                                    "next_state", "output_action"])

def tikz_line_to_transition(line):
    current_state = int(line[line.index("(") + 1: line.index(")")])
    next_state = int(line[line.index("(", line.index("(") +1) + 1: line.index(")", line.index(")") +1)])
    #input_action, output_action = eval(line[line.index("{") + 1, line.index("}")])
    input_action, output_action = line[line.index("{$") + 2: line.index("$}")].split("/")

    return (current_state, input_action, next_state, output_action)

def convert_tex_fsm(fsm_file):
    fsm = []
    with open(fsm_file, "r") as f:
        for line in f:
            if "\\draw" in line and not "(s)" in line:
                fsm.append(tikz_line_to_transition(line))
    return fsm

for i, tex_file in enumerate(["fsm_one.tex", "fsm_two.tex", "fsm_three.tex"]):
    print(sorted(players.fsm_transitions[i]) == sorted(convert_tex_fsm(tex_file)))

output:
True
True
True

Ok so the numbering is off by one, and in fact states 5 and 6 are unreachable for TF2?


Animefenix cloudflare

Source information and language
Animefenix 13.11 (Español)
Steps to reproduce
open source
Expected behavior
load animes
Actual behavior
failure to avoid cloudflare
Aniyomi version
<IP_ADDRESS>
Android version
Android 12
Other details
it opens in webview, the problem is when opening the extension directly

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.13.5.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

For the Failed to bypass Cloudflare error, refer to the Solving Cloudflare issues section at https://tachiyomi.org/help/guides/troubleshooting/#solving-cloudflare-issues. If it doesn't work, migrate to other sources or wait until they lower their protection.


?

This does not work I dragged both of the files into the console and why is it obfuscated?

Reason it is obfuscated is so TTRS can't fix it.

It doesent work

It doesent work

made this as an alternative (you have to refresh after starting a game)


Add new eusart driver for silabs MCU

based on Terezvent work (https://github.com/Terezvent/zephyr/tree/silabs-gecko-spi-eusart)
Fix spi_config function, add disable clock before re-init SPI
Add 16 bits data support

Hello @thomasvKM,
Could you rebase your work so the merge commit won't be needed?
Then, could you split your commit in two parts, one for the support for 16bits access, and another one for the clock.
For the patch related to the clock, don't forget to mention the issue you encountered and t is now solved.


"lime rebuild windows" does nothing in haxelib version

I tried building from official source today, but lime rebuild windows exited without showing error message. Then I tried haxelib run hxcpp Build.xml -Dwindows and it worked.
What is causing this issue? I'm using Visual Studio 2015. hxcpp version is 3.2.205.

Did you manage to solve your issue?

Well I just forgot to switch to git repo with haxelib command. It would probably be better to show some error messages when you try to do lime rebuild from haxelib version.
haxelib set lime 2.9.1
lime rebuild windows
outputs nothing.

Like #417

This is intended behavior, openfl test MyProject windows -rebuild will walk through and rebuild dependencies, a release version of Lime does not get rebuilt


(doc): use starlight for the doc

Description
Astro doc is using starlight, It would be really nice if this plugin also uses it for:

Tabs:

Installation for [npm] | [pnpm]
Eslint config for [as of v9]  | [legacy]
...

Copy to clipboard button for code
Rule search
...

Sounds good to me! Could you please send me a PR?

I guess I will have to make a checklist/plan for myself and for people who want to track on this:

[ ] convert logo to a standalone SVG file

👆 The starlight uses a standalone SVG file instead of an Astro component. While the eslint-plugin-astro use an Astro component for the logo, to remain it the same, need to create svg file with the same parameters

--color-base-blue: 212, 100%;
--color-blue: var(--color-base-blue), 61%;
--theme-text-accent: hsla(var(--color-blue), 1);

the SVG color: --theme-text-accent = hsla(212, 100%, 61%, 1) = #5AA6FF
the SVG size: width = height = 40px

Thank you!!!
Just to be clear, here are some things to know:

I don't have any preferences for the current website design. Feel free to change the colors and logo.

Demo/Playground(now I realize it's broken :sweat: ) doesn't need to be embedded in the website anymore. We can link to the more flexible Playground instead.
https://eslint-online-playground.netlify.app/#eslint-plugin-astro

We can also remove the Playground from the rules documentation (this is also now broken :sweat:).
If possible, it would be useful to use Twoslash ESLint instead of the Playground to see how the rules warn, but you don't need to include it in your initial PR.
https://twoslash.netlify.app/packages/eslint


Note that session() should not be used

According to requests/api.rst, the Session class was originally named session in 0.x. When the class name was capitalized in 1.x, the session() method was added for backwards compatibility.
This adds a docstring note that requests.Session should be used instead

Codecov Report

Merging #4685 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4685   +/-   ##
=======================================
  Coverage   66.79%   66.79%
=======================================
  Files          15       15
  Lines        1563     1563
=======================================
  Hits         1044     1044
  Misses        519      519

Impacted Files
Coverage Δ

requests/sessions.py
76.27% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@nateprewitt Since the method shouldn't be used in new code, should I add a DepreciationWarning?

On closer inspection, it seems session was created after Session. (Aug 16, 2011 vs Aug 15, 2011)

I'm not sure this is worth the grief we'll receive for adding this DeprecationWarning.

I’m in agreement with @sigmavirus24. The DeprecationWarning will likely be more of a harassment for existing users than deterrent for new users.
I think the initial scope of a minor docstring change is likely the extent of what we’d want in this PR.

Noted. I've removed the depreciation warning and updated the docstring.

Should I squash these commits together?

@Stevoisiak, please.

This looks good to me, thanks @Stevoisiak!

There are a few files in Requests which call session() instead of Session(). Should these be updated?


webpack external script tags, automatically

Suppose one uses react or jquery, then it's good idea (sometimes) to use this externals feature of webpack:
webpack.config.js
module.exports = {
    // ...
    externals: {
        "react" : "React"
    }
};

For instance, my index.js went from 640kB to 4kB. With that one line! Yes I know I must include a <script /> tag for react & assume React global does not clash. But it's nicer way sometimes.
It would be cool if HtmlWebpackPlugin had a way to inject those externals almost automatically. Even this would be a improvement:
package.json
{
  "dependencies": {
    "react": "0.13.2"
  },
  "external-scripts" : {
    "react" : "https://cdnjs.cloudflare.com/ajax/libs/react/0.13.2/react.min.js"
  }
}

Notice that I intentionally would like to put my external script URLs in package.json because it includes version numbers already.

I made a world ugliest workaround to inject external scripts:
var HtmlWebpackPlugin = require('html-webpack-plugin')
var html = new HtmlWebpackPlugin();
html.htmlWebpackPluginAssets = function () {
    var assets = HtmlWebpackPlugin.prototype.htmlWebpackPluginAssets.apply(this, arguments);
    assets.chunks['react'] = {
        id : -1,
        entry: 'https://fb.me/react-0.13.2.min.js'
    };
    var i = assets.chunks['index'];
    delete assets.chunks['index'];
    assets.chunks['index'] = i;
    return assets;
}
module.exports = {
    // ...
    entry: {
        index: ['./index.js']
    },
    output: {
        path: './build/',
        filename: './[name].js'
    },
    plugins: [
        html
    ],
    externals: {
        "react" : "React"
    }
};

Why don't you use the common chunks plugin?
http://webpack.github.io/docs/code-splitting.html#split-app-and-vendor-code

At some point if I have a vast vendors code to run in common chunk, it takes a long time to recompile but I think by natural of vendor code is always static that shouldn't be compiled every time when I run with Webpack I think to work with external feature should be better

@jantimon The common chunks plugin does not solve this issue.
What if you want to use a cdn? splitting your code into app and vendor does not really help with that.


⚠ Bofrost.be has degraded performance

In abfd436, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1211 ms

Resolved: Bofrost.be performance has improved in e281178 after 6 minutes.


/usr/bin/env: resty: No such file or directory

use order: lua /usr/local/bin/lord -h
$ lua /usr/local/bin/lord
lor v0.2.6, a Lua web framework based on OpenResty.

Usage: lord COMMAND [OPTIONS]

Commands:
 new [name]             Create a new application
 start                  Start the server
 stop                   Stop the server
 restart                Restart the server
 version                Show version of lor
 help                   Show help tips
 path                   Show install path

resty是openresty的一个命令行工具，需要把它配到环境变量


read info field

Description
update the Rakefile to create the active policy list to read the info field hash.
Issues Resolved
[List any existing issues this PR resolves]
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

          1 Error

:no_entry_sign:
Please add labels to this Pull Request

  Generated by :no_entry_sign: Danger

Danger has errored
[!] Invalid Dangerfile file: (eval):1: unterminated string meets end of file
(eval):1: syntax error, unexpected end-of-input, expecting }
{version: 10, service:"
^. Updating the Danger gem might fix the issue. Your Danger version: 5.16.1, latest Danger version: 6.1.0
 #  from Dangerfile:136
 #  -------------------------------------------
 #    # convert the string to hash
 >    info = eval(info_body) if !info_body.nil? && info_body!="" #&& info_body.any?
 #    version = info[:version] || nil
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger

Danger has errored
[!] Invalid Dangerfile file: No such file or directory @ rb_sysopen - azure_idle_compute_instances.pt. Updating the Danger gem might fix the issue. Your Danger version: 5.16.1, latest Danger version: 6.1.0
 #  from Dangerfile:91
 #  -------------------------------------------
 #  has_app_changes.each do |file|
 >    pp.parse(file)
 #    category = pp.parsed_category
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger

@douglaswth  I made the changes for the require_relative.  can you give one more review/approve?


Simple generators

Proposal of simple generators for

analysis options
metadata (new) https://github.com/dart-lang/linter/issues/1934

For rules it creates also metadata file that other apps can process better (and with more information).
TODO:

[x] generating linter rules
[x] generating analyzer severity (#23)
[x] generating metadata
[x] CI: analyze, format

Hi, @pq, did you have any time to look at the changes, e.g. if we can use metadata to a better generation of links, etc. at http://dart-lang.github.io/linter/lints ? :-)

Sorry!  Let me loop in a few more folks since their ideas would be good here too.  (ping: @kwalrath @bwilkerson)
My initial reaction is a few worries:

that we make advice overly formal and put too much purchase in descriptors like "prefer" vs. "don't"
that the metadata is burdensome to maintain and keep in sync w/ lints as they are added and evolve

Having said all that I've lost some of the initial context so maybe we can go back to dart-lang/linter#1934 and re-establish what we're trying to do and who would benefit from it.
Thanks again for pushing on this and sorry for the delayed responses!


chore(apps/prod/greenhouse): increase greenhouse data size to to 15Ti

/hold

@wuhuizuo: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

debug/prow-call-nested-jenkins-job
daf596209c795e4b0225ca92b13bbf74e64a8021
link
true
/test prow-call-nested-jenkins-job

Full PR test history.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wuhuizuo: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

debug/prow-call-nested-jenkins-job
daf596209c795e4b0225ca92b13bbf74e64a8021
link
true
/test prow-call-nested-jenkins-job

debug/prow-call-jenkins
daf596209c795e4b0225ca92b13bbf74e64a8021
link
true
/test prow-call-jenkins

Full PR test history.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Emit trace and span identifiers per W3C Trace Context

Switch to generating trace and span identifiers in hexidecimal form, meeting requirements of the "traceparent" header from the W3C Trace Context Specification.
Closes #96.

This is looking good.
I'd like to inject some locality into the span identifiers (perhaps overwriting the top 2 bytes with two bytes from the MAC address? Hash of the hostname?). 6 bytes of nanoseconds is 78 hours. That's a lot; in terms of comparing spans between participants in a trace might be better to have a bit of differentiation about which host it came from. Won't be airtight but the top two bytes of timestamp are never going to differentiate anything. Or even just two bytes random.
For the trace identifier, I'm considering potentially directly combine nanosecond precision (8 bytes) timestamp with MAC address (6 bytes) leaving 2 bytes for randomness, which would actually seem to better meet the spec than just using v1 UUIDs.

Instead of the MAC address, decided instead to overwrite the high 2 bytes of the timestamp with 2 bytes of random data.
Split the identifier code and bit fiddling out into Core.Telemetry.Identifiers, mostly to keep the documentation separate from the otherwise usage guide in Core.Telemetry.Observability.


[esp] Add status screen to all-clusters-app on M5Stack

Problem
It can be difficult to glean basic information about a physical Matter node.
Change overview
Add a "Status" screen to ESP / M5Stack platform.
This screen displays whether the device is commissioned or not, and if it is shows info about the first fabric in the table:

Fabric Index
Fabric ID
Node ID
UDP Port
IPv4 address
IPv6 Link Local address
IPv6 ULA address

Testing
Tested manually on desk:

Erase/flash.
Verify Fabric : none
Verify IP : none
Commission over BLE.
Verify Fabric : 1
Verify IP : [live address]
Factory reset.
Verify Fabric : none
Verify IP : none
Recommission over BLE.
Verify Fabric : 1
Verify IP : [live address]

CI should cover the rest.

/rebase


Apply Theme

uses the SB Admin 2 Bootstrap theme on index.php
breaks the theme into template parts head.php, header.php, and footer.php for reuse on other pages

hi Mike thanks for your effort! do you have an online example of the results? if not I suppose I can try to pull a local copy and take a look. Perhaps we can try merging into a branch for the time being, and after we've checked it out look into merging into the master branch?

I pulled the mike/theme branch from your fork and tested locally, it's looking good, I just fixed some of the links in the navigation sidebar before completing the pull. I also issued the pull request against the bootstrap_theme branch (#3 ), so that before merging into master we can continue testing or even complete the UI overhaul


Revert lmdb file to octree files

In your data preparation section you are referencing the "revert_octree_data" tool that can revert a lmdb file to the original octree files. The problem is I can't find this tool anywhere in caffe/tools. The only tool present is "convert_octree_data". Do you still have the original "revert_octree_data" tool?
Thanks in advance

I have uploaded the cpp file just now. Please pull the latest code.

thank you, much appreciated :)


"Server login has been failing" when using pgbouncer

Steps to reproduce
Not quite sure, seems to happen when doing a lot of concurrent writes.
Expected result
Successful login to db
Actual result
Write/connection fails
Environment
Autoscaling read write compute. 2-7. AWS lambda using node-postgres and pooling with 5 max connections.
Logs, links

server login has been failing, try again later (server_login_retry)

Also, I am connecting to the pooling endpoint of the compute node.


How do i unlink the workspace from Databricks?

[Enter feedback here]
Is there an unlink equivalent for the link capability? OR are there any alternatives/steps to follow for unlink?
Thank you.

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

ID: b33cb27c-0786-efe9-75ef-b0665eaebde9
Version Independent ID: 0301fab0-ec41-5b28-4c51-1b73dcc79036
Content: MLflow Tracking for Azure Databricks ML experiments - Azure Machine Learning
Content Source: articles/machine-learning/how-to-use-mlflow-azure-databricks.md
Service: machine-learning
Sub-service: core
GitHub Login: @nibaccam
Microsoft Alias: nibaccam

Thanks for the feedback! We are currently investigating and will update you shortly.

@nibaccam Hello Nina, I don't see how to unlink a workspace is mentioned. Is there any way we can do it?

@YutongTie-MSFT  I've reached out to the product team for help with this one. I'll post here once I learn more.

Thanks a lot.

Hi @aravishdatabricks, there currently isn't a way to unlink the workspaces via the UI. However, you do have a couple options to unlink programmatically:

If you no longer need the Azure ML  workspace, you can delete the Azure ML workspace, which then results in the Azure Databricks workspace to be unlinked from the Azure ML workspace

If you want to keep both workspaces, then follow the below template to unlink them

{
    "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "workspaceName": {
            "type": "string",
            "metadata": {
                "description": "The name of the Azure Databricks workspace to create."
            }
        },
        "pricingTier": {
            "defaultValue": "premium",
            "allowedValues": [
                "trial",
                "standard",
                "premium"
            ],
            "type": "string",
            "metadata": {
                "description": "The pricing tier of workspace."
            }
        },
        "location": {
            "type": "string",
            "defaultValue": "[resourceGroup().location]",
            "metadata": {
                "description": "Location for all resources."
            }
        },
        "amlWorkspaceId": {
            "type": "string",
            "defaultValue": "",
            "metadata": {
                "description": "Resource uri of the AML workspace id."
            }
        }
    },
    "variables": {
        "managedResourceGroupId": "[concat(subscription().id, '/resourceGroups/', variables('managedResourceGroupName'))]",
        "managedResourceGroupName": "[concat('databricks-rg-', parameters('workspaceName'), '-', uniqueString(parameters('workspaceName'), resourceGroup().id))]"
    },
    "resources": [
        {
            "apiVersion": "2018-04-01",
            "location": "[parameters('location')]",
            "name": "[parameters('workspaceName')]",
            "sku": {
                "name": "[parameters('pricingTier')]"
            },
            "comments": "The resource group specified will be locked after deployment.",
            "properties": {
                "ManagedResourceGroupId": "[variables('managedResourceGroupId')]",
                "parameters": {
                    "amlWorkspaceId": {
                        "value": null
                    }
                }
            },
            "type": "Microsoft.Databricks/workspaces"
        }
    ]
}```

Thank you for the update @nibaccam.


[dataprotection] Add support for Cross Region Restore for Backup Vaults

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

az dataprotection: Now supports Cross region restore (CRR) for Backup vaults.

az dataprotection backup-vault create: New parameter -cross-region-restore-state/--crr-state that can be set to Enabled/Disabled.
az dataprotection backup-vault update: New parameter -cross-region-restore-state/--crr-state that can be set to Enabled/Disabled.
az dataprotection backup-vault list-from-resourcegraph

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

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

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

dataprotection

Please fix CI issues

Please fix CI issues

all CI issues have been fixed, could you please re-run the pipeline so the Cred scan passes this time?

@yonzhan @evelyn-ys can you please review and merge the PR?

There are some breaking changes which are not documented

Can you confirm it's necessary to make these breaking changes?

There are some breaking changes which are not documented  Can you confirm it's necessary to make these breaking changes?

Removal of dataprotection backup-instance update is intentional. It had been incorrectly exposed due to misconfiguration in AAZ parameters. I am planning to re-add it with a better options in a PR later this month.


Add --no-wait support to various commands

Adds --no-wait support to various VM and VMSS extension, NIC and Deployment commands. Adds some wait commands where not previously supported (except VMSS extension, which did not work with generic wait).

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

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

Codecov Report

Merging #6684 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           dev   #6684   +/-   ##
===================================
  Coverage    0%      0%
===================================
  Files       11      11
  Lines      145     145
  Branches    11      11
===================================
  Misses     145     145

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 2431ed7...75ed583. Read the comment docs.


On Create Cluster, can't unselect an image after selecting it

I select a node image.  Then after changing my mind, I got to select a different one, there are none listed.
I expect I could see the original list.

Have you tried to clear the text box? I think the dropdown list will only list images that match the contents of the text box, so you should probably clear it for getting a full list of images...

Closed because of inactivity. Please reopen if you want.


Remove xfail in windows jitify test

This PR blocks #5407.
Currently, the following XFAIL is passing unexpectedly in Windows CI (cc: @leofang)
XPASS cupy_tests\core_tests\test_raw.py::TestRawJitify_param_0_{jitify=False}::test_jitify1 macro preprocessing in NVRTC is likely buggy

LGTM. This was added in #4771 but was unnecessary 😅

pfnCI, test this please.

Jenkins CI test (for commit 54604f896de79c96f26b1590fd07956634c8fcf4, target branch master) succeeded!

pfnCI, test this please.

Jenkins CI test (for commit cb1116db23cba5d7920081fb7b93b6f6e12dce01, target branch master) succeeded!

This is strange...Could it be passing because the cubins are now catched on Windows CI? We were pretty sure it had an error and couldn't figure out a way to fix or work around it, so marked it xfail: https://github.com/cupy/cupy/issues/4383#issuecomment-792475784

It seems that the test was passing at the time of merging #4771: https://ci.preferred.jp/cupy.experimental.win.cuda100/69484/
XPASS tests/cupy_tests/core_tests/test_raw.py::TestRawJitify_param_0_{jitify=False}::test_jitify1 macro preprocessing in NVRTC is likely buggy

Kernel cache was not available at that time.
I will recheck this on Windows locally.

I guess I figure out why (to be verified): when self. jitify is False, the test should pass (as we correctly capture the exception), so xfail should be only marked when self.jitify is True.

@leofang I see, that makes sense. Could you update the PR @asi1024?

I guess @asi1024 already did (cb1116d)! For some reason I was staring at the commit before the force push...

Ah, ok I overlooked that the PR was updated. LGTM.


Improper syntax highlight for dollar sign $

Describe the bug
The highlight for the dollar sign $ is improper
Steps to reproduce

open the following example org file test.org

#+title: Test

example 1: the verbatim should be respected
=$= the text should not be highlighted =$=

example 2: text surrounded by dolloar signs that are distanced by over 2 lines should not be highlighted
$2 billion
hello
oh not good
$3 billion

Expected behavior

when $ is closed by two =, i.e =$= then verbatim should be respected. i.e there should be no syntax highlight
when two $ are distanced by more two lines, text between them should not be highlighted.

Emacs functionality
see description here
https://orgmode.org/manual/LaTeX-fragments.html

Text within the usual LaTeX math delimiters. To avoid conflicts with currency specifications, single ‘$’ characters are only recognized as math delimiters if the enclosed text contains at most two line breaks, is directly attached to the ‘$’ characters with no whitespace in between, and if the closing ‘$’ is followed by whitespace, punctuation or a dash. For the other delimiters, there is no such restriction, so when in doubt, use ‘(...)’ as inline math delimiters.

Minimal init.lua
vim.cmd [[set runtimepath=$VIMRUNTIME]]
vim.cmd [[set packpath=/tmp/nvim/site]]

local package_root = '/tmp/nvim/site/pack'
local install_path = package_root .. '/packer/start/packer.nvim'

local function load_plugins()
    require('packer').startup {
        {
            'wbthomason/packer.nvim',
            { 'nvim-treesitter/nvim-treesitter' },
            { 'kristijanhusak/orgmode.nvim', branch = 'master' },
        },
        config = {
            package_root = package_root,
            compile_path = install_path .. '/plugin/packer_compiled.lua',
        },
    }
end

_G.load_config = function()
    require('orgmode').setup_ts_grammar()
    require('nvim-treesitter.configs').setup {
        ensure_installed = { 'latex' },
        highlight = {
            enable = true,
            additional_vim_regex_highlighting = { 'org' },
        },
    }

    vim.cmd [[packadd nvim-treesitter]]
    vim.cmd [[runtime plugin/nvim-treesitter.lua]]
    vim.cmd [[TSUpdateSync org]]

    -- Close packer after install
    if vim.bo.filetype == 'packer' then
        vim.api.nvim_win_close(0, true)
    end

    require('orgmode').setup {
        org_highlight_latex_and_related = 'native',
    }

    -- Reload current file if it's org file to reload tree-sitter
    if vim.bo.filetype == 'org' then
        vim.cmd [[edit!]]
    end
end

if vim.fn.isdirectory(install_path) == 0 then
    vim.fn.system { 'git', 'clone', 'https://github.com/wbthomason/packer.nvim', install_path }
    load_plugins()
    require('packer').sync()
    vim.cmd [[autocmd User PackerCompileDone ++once lua load_config()]]
else
    load_plugins()
    load_config()
end

Screenshots and recordings
the first is neovim

the second is emacs

OS / Distro
macOS 12.6
Neovim version/commit
0.8.1
Additional context
No response

I cannot reproduce this. The highlighting for me looks as you in the Emacs screenshot you posted. @kristijanhusak is a latex parser needed here?

I cannot reproduce this. The highlighting for me looks as you in the Emacs screenshot you posted. @kristijanhusak is a latex parser needed here?

Sorry, I missed the option org_highlight_latex_and_related = 'native', and ensure_installed = 'latex'
I have updated the minimal config to make it reproducible.

When you set org_highlight_latex_and_related = 'native' it actually loads the whole syntax file from vim's tex filetype. Support for latex is generally lacking and not stable. Maybe you could disable the native and just rely on what's there with treesitter/markup highlighter. Recently some highlights were added (https://github.com/nvim-orgmode/orgmode/pull/428), but not all of them.

When you set org_highlight_latex_and_related = 'native' it actually loads the whole syntax file from vim's tex filetype. Support for latex is generally lacking and not stable. Maybe you could disable the native and just rely on what's there with treesitter/markup highlighter. Recently some highlights were added (#428), but not all of them.

Thanks for your advice. I decide to go with org_highlight_latex_and_related = 'entity' right by now


create 020_homodyned-Jaynes-Cummings-emission.md for qutip tutorial 5

create 020_homodyned-Jaynes-Cummings-emission.md for qutip tutorial 5 for the unitary hack

Thanks @GustavoPetronilo for submitting this, but I accepted #98.
We would welcome more porting of tutorials outside of UnitaryHACK if you have the time and inclination to do so!

Thanks @GustavoPetronilo for submitting this, but I accepted #98.
We would welcome more porting of tutorials outside of UnitaryHACK if you have the time and inclination to do so!

Thank you!


Add strict option

This commit also contains code style updates, documentation improvements
and a rename to itin-validator.
Refs https://github.com/seegno/is-valid-itin/issues/2.

We will be deprecating this PR in favour of removing options. This will allow us keeping the module simpler and considering value as valid if it includes numbers only (in the expected range) or a formatted string in the format XXX-XX-XXXX.

Deprecated by https://github.com/seegno/is-valid-itin/pull/5.


Dockerfile  user non root

Is it possible to run the container with another user other than root?

The code here is open, you can do what docker offers by changing the code here.
Not sure I understand the question...

You may use the --user flag, as described in https://docs.docker.com/engine/reference/run/#user

I understand, but can I use a non-root user to run the application?

Try and let us know... We haven't tested it.

right!
I did the test adding the parameters below and it worked. Thank you all.
RUN addgroup   --system --gid 1001 graphhopper && \
    adduser  --system --uid  1001   --group graphhopper && \
    chown -R graphhopper:graphhopper /graphhopper
USER graphhopper


Setting up new pi failed

First issue seems to be that the logstream file is not writable.
After that the whole install process stops.

What pack did you try?

The basic one

I tried to reproduce:
RPi: Raspberry Pi 2 B v1.1
image: 2021-01-11-raspios-buster-armhf-lite.img
pack: basic
This runs as expected.
Which image was used?
Was the network connection working?


set hostname from cloud-init

@ibuildthecloud

@ibuildthecloud I think i've addressed all of your comments

closes #100


feature request: LIKE operator in JSON_MATCH

Hello,
I was wondering if it was planned to add the LIKE operator to JSON_MATCH ? I’m currently using
REGEXP_LIKE(JSONEXTRACTSCALAR("labels", '$.demande_intention', 'STRING'), 'terminal')

but it’s very slow (even with small number of scanned documents (21). And I was thinking that maybe having it directly with JSON_MATCH could speed-up this operation?
JSON_MATCH("labels", 'demande_intention LIKE ''terminal''')

edit: the labels column already have a JSON index
Thank you
cc @atris

Hi @Eywek how slow is "very slow"?
Is there a natural way for you to filter the JSON documents before applying the JSONEXTRACTSCALAR and REGEXP_LIKE functions? Without an index structured to support these functions, they are naturally quite expensive.
Have you considered extracting $.demande_intention into a text column during ingestion so you can create a text index on it, instead of storing it embedded in JSON?

how slow is "very slow"?

~600-800ms for 21 documents (vs 22-40ms without this filter)

Is there a natural way for you to filter the JSON documents before applying the JSONEXTRACTSCALAR and REGEXP_LIKE functions? Without an index structured to support these functions, they are naturally quite expensive.

I'm using a JSON index on the column and operations like JSON_MATCH(labels, 'demande_intention  = ''foo''') are fast (30-40ms)

Have you considered extracting $.demande_intention into a text column during ingestion so you can create a text index on it, instead of storing it embedded in JSON?

Yep I know it would be the best way to do it but since the labels column contains dynamic data, we don't want to add each field of the JSON object to the schema

Supported added in #12568


[QUESTION] Is the project still maintained?

Hi all,
quick question as in the title - is the project actually alive? The last change was commited on 16 Aug 2020, there's no active dev branch, there are 28 open pull requests with no answers, some crucial issues like no support for Click 8.x are not addressed... It looks very alarming. CLI is the crucial part of many apps, so if the project is abandon - it would be really valuable to say it out loud and recommend people to switch back to Click.
Cc: @tiangolo

https://github.com/tiangolo/typer/issues/263

I wonder more what's the point of yet another library to handle CLI interaction when there's already a ton out there that could simply be improved.
We are tired of having to relearn a new API for the same problem again and again.
And no. I don't choose the library. Whatever project I have to modify choses it.

While I really liked typer, but as the same time I am glad I used it only one experimental project so far, where it would not be more than a couple of hours to remove it.
Projects relying on a single person do have a high risk of suddenly becoming unmaintained. I know a couple of teams that questioned even maintainability and API stability of click itself. Sometimes is better to stick with argparse.
I hope the situation will change for the better.

I wonder more what's the point of yet another library to handle CLI interaction when there's already a ton out there that could simply be improved.

I for one thing typer has a fresh take on this. Yes there are other libraries (and for that matter, the stdlib itself has two!), but typer's reliance on type annotations to make the syntax clean and expressive I think is a different take on this that really isn't "yet another library" - it's a unique solution to an old problem.
With that in mind, I do think the question in this post is important to get answered: the long-term sustainability of this project is critical for people wanting to use it downstream. But I think it's totally legit for you to answer "I need some other contributors/maintainers to help out with maintaining it", @tiangolo. I know that can be hard - I've struggled with it myself as a maintainer of open source packages.  But I hope the interest in this thread and all the PRs show that there's enough excitement with the great work here to make it happen!  And regardless of the answer, thanks for your work on this @tiangolo - new carefully thought-out ideas like this are the beating heart of the Python community, IMHO!

A recent tweet from @tiangolo leads me to believe that development might speed up again.

Hey all! Thanks for the interest and discussion.
Yep, Typer is still maintained, in fact, I just released Typer 0.4.0, which is compatible with both Click 7 and Click 8. :tada:
But of course, it all depends on how much time I have available to dedicate to it.
A lot of my time is spent handling issues, replying to questions, etc. (here and in the other projects).
If you would like to see faster progress, you could help by answering questions from others, sponsoring, etc. https://typer.tiangolo.com/help-typer/#help-others-with-issues-in-github

@tiangolo great to hear you are still with us! 😄 You haven't mentioned adding new maintainers as an option - what's the reason?

The lack of answer to the most important question combined stats from https://github.com/tiangolo/typer/graphs/contributors do show that there is no desire to transform this into a community project. Typer is a wonderful showcase/portfolio project of @tiangolo and is likely to remain like this forever.
The fact that is hosted under a personal account should be another clue. In the end, it would be up to each one to decide if they want to make it a dependency of not. Sadly very few authors mention clearly the scope of their projects as they like the fame but they don't fancy the cost of becoming popular.

@bristea Yes, I believe you're right. It could nevertheless be forked and continued as a project. I have been considering that myself. (It helps that it's licensed under MIT.)

@tiangolo it's been a year without update and some important PR are ready to be merge.
Would it be possible to add some maintainer, as suggested, to keep this nice project alive?
I'm sadly considering moving back to click on my projects due to slow maintenance.

@RomainBrault FYI it's been a while since my last release, but if tiangolo is not keen (or able to) maintain this project, I'm happy to accept PRs or even freature requests on my fork. I've already enhanced functionality a fair bit, and fixed several important bugs.


Wolverine Event->Handler cache issues: wrong handlers being called

Describe the bug
Wolverine calls the wrong handler when using subclasses for events with a duplicated event name.
Given the following code:
marten
    .SomeConfig()
    .IntegrateWithWolverine()
    .PublishEventsToWolverine("internal-events");

public static class OrderCommands
{
    public sealed record Create(string Id);
}

public static class OrderEvents
{
    public sealed record Created(string Id);
}

public static class BasketCommands
{
    public sealed record Create(string Id);
}

public static class BasketEvents
{
    public sealed record Created(string Id);
}

public sealed class MyHandler
{
    public static void Handle(BasketCommands.Create command, IDocumentSession session, ILogger logger)
    {
        logger.LogInformation("Handling Basket Trigger command.");
        session.Events.StartStream(command.Id, new BasketEvents.Created(command.Id));
    }

    public static void Handle(OrderCommands.Create command, IDocumentSession session, ILogger logger)
    {
        logger.LogInformation("Handling Order Trigger command.");
        session.Events.StartStream(command.Id, new OrderEvents.Created(command.Id));
    }

    public static void Handle(IEvent<OrderEvents.Created> @event, ILogger logger)
    {
        logger.LogInformation("Handled Order Created event.");
    }

    public static void Handle(IEvent<BasketEvents.Created> @event, ILogger logger)
    {
        logger.LogInformation("Handled Basket Created event.");
    }
}

The first time an event is published, it gets routed to the correct handler. After that, Wolverine seems to cache the eventName to handler mapping without the outer class.
When sending the Create Order command, it's all looking good:
2024-07-26T12:56:47.6419280 info: WolverineIssues.ExceptionHandlingRetryIssue.OrderCommands.Create[0]
      Handling Order Trigger command.
2024-07-26T12:56:48.3121470 info: Marten.Events.IEvent[0]
      Handled Order Created event.

But then running the Create Basket command:
2024-07-26T12:56:56.1440730 info: WolverineIssues.ExceptionHandlingRetryIssue.BasketCommands.Create[0]
      Handling Basket Trigger command.
2024-07-26T12:56:56.7267350 info: Marten.Events.IEvent[0]
      Handled Order Created event.

Interestingly, the command is actually routed to the correct handler while it has the same setup (outer class + same class name)
Now the other way around, we first send a Create Basket command:
2024-07-26T13:02:09.8472780 info: WolverineIssues.ExceptionHandlingRetryIssue.BasketCommands.Create[0]
      Handling Basket Trigger command.
2024-07-26T13:02:10.6736290 info: Marten.Events.IEvent[0]
      Handled Basket Created event.

It handled it correctly, but now Order Created events are send to the cached handler:
2024-07-26T13:02:43.6185640 info: WolverineIssues.ExceptionHandlingRetryIssue.OrderCommands.Create[0]
      Handling Order Trigger command.
2024-07-26T13:02:44.5568680 info: Marten.Events.IEvent[0]
      Handled Basket Created event.

Expected behavior
The right handlers to be called

@jorik At a first guess, you've got a serialization issue due to using the same event class name. If you're going to do that, see this: https://martendb.io/events/versioning.html#event-type-name-mapping
Or make things much easier and use CreateBasket and BasketCreated instead of the inner types.


Monitoring v2 k8s-prometheus-adapter log spam "apiserver was unable to write a JSON response: http2: stream closed"

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Deploy a Rancher v2.5.3 instance and a custom cluster with 3 all role nodes
Deploy the monitoring chart from Apps & Marketplace in the Cluster Explorer UI

Result:
When the monitoring chart has finished installing, observe repeating messages of the following format in the rancher-monitoring-prometheus-adapter Deployment logs in the cattle-monitoring-system namespace.
E0106 15:15:10.658751       1 writers.go:163] apiserver was unable to write a JSON response: http2: stream closed
E0106 15:15:10.658799       1 status.go:71] apiserver received an error that is not an metav1.Status: &errors.errorString{s:"http2: stream closed"}

Other details that may be helpful:
The upstream k8s-prometheus-adapter issue for these log messages is at https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/292.
When the following is set in the values YAML on installing the monitoring chart (per https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/292#issuecomment-671786467) the log spam does not occur:
prometheus-adapter:
  rules:
    default: false

The upstream issue identifies a potential Kubernetes fix in v1.20 (https://github.com/kubernetes/kubernetes/pull/94589), per https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/292#issuecomment-741777382, and expects the issue resolved once the API machinery dependencies in prometheus-adapter are bumped to v1.20 (https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/292#issuecomment-741870086).
Environment information

Rancher v2.5.3

gz#14188

QA, please test this in the newest version of 2.6.x

Tested in 2.6-head Commit ID 8a31d3c
Error is still present in logs.

install monitoring v2
workload > pods > rancher-monitoring-prometheus-adapter-xxxx
right click 3 dots
view logs

E0518 18:41:29.145364 1 writers.go:130] apiserver was unable to write a fallback JSON response: http2: stream closed
E0518 18:41:29.147069 1 writers.go:117] apiserver was unable to write a JSON response: http: Handler timeout

This needs to be reproduced in 2023-Q1-2.6x and 2023-Q1-2.7x

Reproduced in 2.7-head 9376ad2 in an RKE2 v1.25.6 +rke2r1 downstream cluster (non custom, cluster created through Rancher) with monitoring 102.0.0+up40.1.2 installed.
E0216 23:54:57.688602       1 status.go:71] apiserver received an error that is not an metav1.Status: &errors.errorString{s:"http: Handler timeout"}: http: Handler timeout
E0216 23:54:57.689789       1 status.go:71] apiserver received an error that is not an metav1.Status: &errors.errorString{s:"http2: stream closed"}: http2: stream closed
E0216 23:54:57.692041       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0216 23:54:57.693205       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0216 23:54:57.694298       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0216 23:54:57.695515       1 writers.go:131] apiserver was unable to write a fallback JSON response: http2: stream closed

Also reproduced in 2.6-head 6d1bd94 in an RKE  v1.24.9 downstream cluster (non custom, cluster created through Rancher) with monitoring 100.2.0+up40.1.2 installed.
E0217 00:26:10.139403       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0217 00:26:10.140917       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0217 00:26:10.143083       1 status.go:71] apiserver received an error that is not an metav1.Status: &errors.errorString{s:"http2: stream closed"}: http2: stream closed
E0217 00:26:10.144196       1 writers.go:131] apiserver was unable to write a fallback JSON response: http: Handler timeout
E0217 00:26:10.145433       1 timeout.go:141] post-timeout activity - time-elapsed: 20.250622ms, GET "/apis/custom.metrics.k8s.io/v1beta1" result: <nil>
E0217 00:26:10.147089       1 timeout.go:141] post-timeout activity - time-elapsed: 23.421925ms, GET "/apis/custom.metrics.k8s.io/v1beta1" result: <nil>
E0217 00:26:10.148219       1 writers.go:131] apiserver was unable to write a fallback JSON response: http2: stream closed

Replaced image rancher/mirrored-prometheus-adapter-prometheus-adapter:v0.10.0 with most recent upstream image gcr.io/k8s-staging-prometheus-adapter/prometheus-adapter:master. Issue still persists. I'm thinking this is because they are using an older version of golang (1.18).
@sanjay920 @MKlimuszka @prachidamle

I have confirmed that this was fixed in v0.11.x.
Unfortunately v0.11.x also deprecated the logtostderr flag, and the chart does not expose the container args as a value, so the only way to run v0.11.x is using kustomize to override the args.

On https://github.com/kubernetes-sigs/prometheus-adapter/issues/292 there are some discussions about this.

Upstream ticket was closed, and we've rebased. Let's test this again after we make our new mirrored images.

Testing shows the behavior still persists after the rebase. The upstream ticket was closed since the originally reported panic has been resolved, but several comments report still having those logs in the prometheus-adapter deployment.

Just added ASML as a customer having this issue on 2.9.2


32-bit Linux and Windows build failure with embree-aarch64

Godot version:
3.x at 1209ee0df43d585d6645dfbcaa5e1120243f7c38, regression from #48455.
OS/device including version:
Ubuntu 16.04 i386 (Linux build), Fedora 33 x64 (Windows build with MinGW-GCC).
Issue description:
32-bit Linux and Windows builds are failing since #48455, the PR removed the conditions for not building on x86 (32-bit) but embree-aarch64 supports x86_64 and aarch64, but not x86.
Steps to reproduce:

Build for Linux or Windows 32-bit

Minimal reproduction project:
n/a

Fixed by #48483.


Use amp / FP16

Hi, since the sentence transformers support FP16 training (by setting use_amp=True)
can this also be build in here?
Not sure if this is a FR or a question. :-)

Should be easy to add it here:
https://github.com/huggingface/setfit/blob/7735e8e3b208edb8dfb549beb16e585453c5f44e/src/setfit/trainer.py#L276

see PR #134

closing this because PR is merged


Android Java: 10.00 Bluetooth tutorial in android

Description 📜
Android Java: 10.00 Bluetooth tutorial in android
Fixes #6098

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[x] Android Dev (Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

and also please try to give some short comments in your code to make it understandable.

ok


Cannot Change Header names for csv sometimes database column names dont make sense if using in excel

Interesting. I could use a few more specifics if you're looking for help. What does your model look like? Can you provide a sample of the file? What error message are you getting?

I am closing this ticket pending more information.


🐛 [firebase_performance] [iOS] Plugin crash (Fatal Exception: NSInvalidArgumentException)

Bug report
Fatal Exception: NSInvalidArgumentException
Object for handle already exists: 32

Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x1903f3298 __exceptionPreprocess
1  libobjc.A.dylib                0x1a414d480 objc_exception_throw
2  Runner                         0x100d36d6c +[FLTFirebasePerformancePlugin removeMethodHandler:] + 61 (FLTFirebasePerformancePlugin.m:61)
3  Runner                         0x100d366a0 -[FLTFirebasePerformance newHttpMetric:result:] + 67 (FLTFirebasePerformance.m:67)
4  Runner                         0x100d361c4 -[FLTFirebasePerformance handleMethodCall:result:] + 34 (FLTFirebasePerformance.m:34)
5  Runner                         0x100d36c34 -[FLTFirebasePerformancePlugin handleMethodCall:result:] + 44 (FLTFirebasePerformancePlugin.m:44)
6  Flutter                        0x101d1a49c (Missing)
7  Flutter                        0x1017e541c (Missing)
8  Flutter                        0x101abc454 (Missing)
9  Flutter                        0x1017ee844 (Missing)
10 Flutter                        0x1017f07b0 (Missing)
11 CoreFoundation                 0x1903730f0 __CFRUNLOOP_IS_CALLING_OUT_TO_A_TIMER_CALLBACK_FUNCTION__
12 CoreFoundation                 0x190372cf0 __CFRunLoopDoTimer
13 CoreFoundation                 0x1903721c4 __CFRunLoopDoTimers
14 CoreFoundation                 0x19036c4fc __CFRunLoopRun
15 CoreFoundation                 0x19036b818 CFRunLoopRunSpecific
16 GraphicsServices               0x1a6a71570 GSEventRunModal
17 UIKitCore                      0x192c970e8 -[UIApplication _run]
18 UIKitCore                      0x192c9c664 UIApplicationMain
19 Runner                         0x10090c870 main + 10 (AppDelegate.swift:10)
20 libdyld.dylib                  0x19004a140 start

Steps to reproduce
Unknown.
I suspect that due to running several FlutterEngines different plugin instances may start clashing. Every FlutterEngine registers all plugins via calling GeneratedPluginRegistrant.register(with: flutterPluginRegistry)
Expected behavior
Plugin doesn't crash.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel unknown, 1.22.6, on macOS 11.4 20F71 darwin-x64, locale en-NL)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.5.1)
[!] Android Studio (version 4.2)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[!] Android Studio (version 4.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
[✓] Connected device (1 available)

! Doctor found issues in 2 categories.

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.10.5
Flutter SDK 1.22.6

- firebase 7.3.3 [http http_parser js]
- firebase_analytics 7.0.1 [meta flutter firebase_core firebase_analytics_web firebase_analytics_platform_interface]
- firebase_analytics_platform_interface 1.0.3 [flutter meta]
- firebase_analytics_web 0.1.1 [flutter flutter_web_plugins firebase firebase_analytics_platform_interface meta]
- firebase_core 0.7.0 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_core_platform_interface 3.0.1 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.1+3 [firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_crashlytics 0.4.0+1 [flutter stack_trace firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface]
- firebase_crashlytics_platform_interface 1.1.6 [flutter meta collection firebase_core plugin_platform_interface]
- firebase_performance 0.5.0+1 [flutter firebase_core]
- firebase_remote_config 0.6.0 [flutter firebase_core]

Hi @sergey-triputsco
What code triggers the crash? Are you able to provide a minimal complete reproducible code sample?
Thank you

Hi @markusaksli-nc
We are in the process of updating our dependencies. That may take some time. I hope soon I can test with the latest plugin version.
The next code shows how we use the plugin to collect metrics:
class _MetricsClient extends BaseClient {
  _MetricsClient({
    @required this.inner,
  }) : assert(inner != null);

  final Client inner;

  @override
  Future<StreamedResponse> send(BaseRequest request) async {
    final url = request.url.toString().replaceAllMapped(
          RegExp('(.*)\/api\/\\w*\/(.*)'),
          (match) => '${match.group(1)}/api/***/${match.group(2)}',
        );
    final metric = FirebasePerformance.instance.newHttpMetric(url, _fromMethod(request.method));
    await metric.start();

    StreamedResponse response;
    try {
      response = await inner.send(request);
      metric
        ..responsePayloadSize = response.contentLength ?? 0
        ..responseContentType = response.headers['Content-Type']
        ..requestPayloadSize = request.contentLength ?? 0
        ..httpResponseCode = response.statusCode;
    } finally {
      await metric.stop();
    }

    return response;
  }

  HttpMethod _fromMethod(String method) {
    switch (method.toLowerCase()) {
      case 'connect':
        return HttpMethod.Connect;
      case 'delete':
        return HttpMethod.Delete;
      case 'get':
        return HttpMethod.Get;
      case 'head':
        return HttpMethod.Head;
      case 'options':
        return HttpMethod.Options;
      case 'patch':
        return HttpMethod.Patch;
      case 'post':
        return HttpMethod.Post;
      case 'put':
        return HttpMethod.Put;
      case 'trace':
        return HttpMethod.Trace;
    }

    return HttpMethod.Get;
  }
}

Crash doesn't happen if app has a single UI Flutter engine. But for some cases app starts headless Flutter engine to handle system events like geofences. Then both Flutter engines use FirebasePerformance and collect metrics of http calls. That seems a condition when crash happens.
I cannot provide now a small sample where crash is reliably reproducible. I try to implement something when I have more free time.


LAB_06-Implement_Network_Traffic_Management.md.pdf

English:
Implement Azure Load Balancer
Current Translation:
Mettre en oeuvre l'équilibreur de charges Azure
Suggested Translation:
Mettre en oeuvre Azure Load Balancer
Description:
Several occurrences

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run bugfix tool and waiting for result.


Milestone 1: Create Class

Milestone 1: Create Class
Changes Made In This Milestone:

Created class Person with  the following:

Instance vars: @id, @name, and @age.
Constructor with name,  age, and parent_permission as parameter. name and parent_permission are optional and have default values of "Unknown" and true.
Getters for @id, @name, and @age.
Setters for @name and @age.
Private method is_of_age? that returns true if @age is greater or equal to 18 and false otherwise.
Public method can_use_services? returns true if person is of age or if they have permission from parents.

Create class Student with the following:

Inherits from Person.
Constructor extends parent's constructor by adding @classroom and a parameter for it.
Method play_hooky that returns "¯\(ツ)/¯".

Create class Teacher with the following:

Inherits from Person.
Constructor extends parent's constructor by adding @specialization and a parameter for it.
Override can_use_services? so it always returns true.

Made Changes as Suggested


Spanish translation improved

Description

I made some grammar improves in the Spanish translation.

Awesome thank you! @ucdstudent95618 would you mind taking a quick look at this and approving the changes? Appreciate you all

@all-contributors please add @JoseNoriegaa for translation


"lint" script should use the locally installed esw

As eslint-watch package is part of the dependencies, esw should be referred to the local installation.
Currently, the "lint" script is using the globally installed version, if avalable:
"lint": "esw webpack.config.* src tools --color"
This small update includes:
"lint": "node_modules/.bin/esw webpack.config.* src tools --color"

Coverage remained the same at 98.501% when pulling d5ccf57d62ccd45b334c61d69d8d2916ffe884cf on visvadw:patch-1 into 64afbea59f7943d3760cf353bb64f38b836470f5 on coryhouse:master.

This shouldn't be necessary. Node adds any scripts run in npm scripts to the path automatically, so the local version is executed.

Hi Cory,
I understand and thanks for the reference. I've misinterpreted the error, thrown after Ctrl+C:
npm ERR! node v5.11.0
npm ERR! npm  v3.8.6
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>lint: `esw webpack.config.* src tools --color "--watch"`
npm ERR! Exit status 1

Sorry for the inconvenience. And thank you for the nice course on Pluralsight ("Building Applications with React and Redux in ES6") and this starter! 👍


2.4.1 not installing properly on haxelib

It unpacks, then says "current version is now 1.0.0" and creates a directory called unknown.  Then when I type haxelib upgrade again, it asks if I want to upgrade again and does the same thing.  Rinse, repeat.

Oh, haxelib :sob:
You can also try installing like this:
1.) Download from here: http://lib.haxe.org/p/lime
2.) Run haxelib local lime-2.4.0.zip (or whatever file name it is)


add automatic link detection

https://github.com/bryanwoods/autolink-js

See #52

added link language option

cant link inside a language-none


SQLite 3.35.0, build failure

SQLite 3.35.0 introduced math functions, of which at least signFunc collides with the one provided by extension-functions.c.
This is a minimal patch just removing this function for 3.35.0 onwards so that it at least compiles. I haven't checked yet if other functions became obsolete, but I'm following this up "upstream": https://gitlab.com/liamh/extension-functions/-/issues/4

Thanks for the fix. I'll prepare 3.35.0 release


Summary Text is being trimmed in Picture Message

Expected Behaviour
When notification is expanded. The image is showed and summary text is expanded too.
Actual Behaviour
When notification is expanded. The image is showed but summary text is truncated / trimmed.
Reproduce Scenario (including but not limited to)
Here is my code (using firebase admin nodeJS SDK):
admin.messaging().sendToTopic('someTopic', {
    data: {
      "title": "Big Picture",
      "message": "This is my big picture message",
      "style": "picture",
      "picture": "http://bisnisupdate.com/images/posts/2016/07/01/LABBAIK-CHICKEN.png",
      "summaryText": "How can we expand our chicken business so fast? ready to hear the secret?"
    }
  })
    .then(function (response) {
      res.send(response);
    })
    .catch(function (err) {
      res.send(err);
    });

Steps to Reproduce
Send notification as explined above.
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.0
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Samsung Galaxy S5
Cordova CLI version and cordova platform version
cordova --version  = 7.0.0
cordova platform version android  = android 6.2.3

Plugin version
cordova plugin version | grep phonegap-plugin-push = 1.10.5

Sample Push Data Payload
{
    data: {
      "title": "Big Picture",
      "message": "This is my big picture message",
      "style": "picture",
      "picture": "http://bisnisupdate.com/images/posts/2016/07/01/LABBAIK-CHICKEN.png",
      "summaryText": "How can we expand our chicken business so fast? ready to hear the secret?"
    }
  }

Sample Code that illustrates the problem
admin.messaging().sendToTopic('someTopic', {
    data: {
      "title": "Big Picture",
      "message": "This is my big picture message",
      "style": "picture",
      "picture": "http://bisnisupdate.com/images/posts/2016/07/01/LABBAIK-CHICKEN.png",
      "summaryText": "How can we expand our chicken business so fast? ready to hear the secret?"
    }
  })
    .then(function (response) {
      res.send(response);
    })
    .catch(function (err) {
      res.send(err);
    });

In current condition, when notification is expanded. The image is being show but the text summary is being trunctated / trimmed.
Any workaround to make the summary text is expanded too?

@dionalfa sorry, you'd have to create a custom view to have multi-line summary text in a big picture message and that is out of scope for this plugin.


Add a new parent package for the CDI full tests

Add a new parent org.jboss.cdi.tck.tests.full package to use as the root for all CDI tests with the cdi-full group. This should be done as test group is updated.

PR #274  introduces a new org.jboss.cdi.tck.tests.full package.

The various reworkings of the tests has moved cdi-full tests to under a org.jboss.cdi.tck.tests.full root package


IndexedParallelIterator impl for JoinParIter

It would be convenient to have a way of iterating only over some subset of matched tuples in a parallel join e.g.
// only process up to 32 entities matched
const MAX_PROCESSED_PER_TICK: usize = 32;
(&component_a, &component_b).par_join().take(MAX_PROCESSED_PER_TICK).for_each(do_thing)

I looked at possibility of implementing IndexParallelIterator when I was implementing par_join, but I don't think it's possible as there is no way that I know to cheaply index BitSet which is used to filter joined entities.

Sounds like the only workaround would be to iterate the components, take the first 32, collect them and then process that @HybridEidolon

Err..well not sure how to proceed here, any suggestions?

I don't think its possible to cheaply do indexing on BitSet, which kind of breaks this.


Unsigned types

I propose supporting unsigned types such as UInt (and the others)

:+1:
PR's are welcomed


Update .gitmodules

fixed a missing line break, that was causing this library to not properly build
Description
It seems that there was a line break that was accidentally deleted from the .gitmodules file.
When I was trying to build a skeleton project, I was getting the following error:
fatal: No url found for submodule path 'meson' in .gitmodules

meson.build:1:0: ERROR: Git command failed: ['/opt/homebrew/bin/git', 'submodule', 'update', '--init', '--checkout', '--recursive']

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] 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 not work as expected)
[ ] This change requires a documentation update

Thanks for finding this and opening the PR. I have fixed it in the monorepo and pushed the update back out. I also checked all the other monorepo associated repos - looks like only this one had the error.


code is sent to the ipython console but not run

Hello,
I have got this working as i want it in terms of splits and everything but i cant get the code i send to the IPython colsole i have to run. I send it and then it just sits there and i have to switch to the split and then go into instert mode to press enter and then tab back out to the python file i was in before. Im pretty sure this is not intended. Anybody know why this might be happening?
I have posted a pic of the setup below.

https://github.com/Vigemus/iron.nvim/assets/110160546/16a5412a-94bf-46d3-9b6a-e55c8f10b35c
Any help appreciated.
Thank you

Alan Manning

+1 I have the same issue here

+1 I have the same issue here

found that there is a plugin called toggle term and you can set up a keymap for ipython console. This also has commands to sent lines, visual selection etc to the terminal.  here is a video showing it with the setup, he uses python but can easily be changed to ipython, https://youtu.be/5OD-7h7gzxU.
The docs for toggle term show the ipython console example there with sending line etc to it, https://github.com/akinsho/toggleterm.nvim#termselect.
Hope this helps.

+1 I have the same issue here

found that there is a plugin called toggle term and you can set up a keymap for ipython console. This also has commands to sent lines, visual selection etc to the terminal. here is a video showing it with the setup, he uses python but can easily be changed to ipython, https://youtu.be/5OD-7h7gzxU. The docs for toggle term show the ipython console example there with sending line etc to it, https://github.com/akinsho/toggleterm.nvim#termselect.
Hope this helps.

Thanks! I will check it out. It's a shame this issue is holding me back from using this plugin.

Hiya,
I had this issue and I found a workaround that is suitable for me. Until the ipython issue is fixed, you can switch to use the python executable instead. Like instead of typing "ipython" to the terminal you will get a REPL that looks like as if you have entered "python" or "python3". Here is what I did:
I use lazy as my package manager on windows so my plugins are downloaded to this folder:
C:/Users/"myusername"/AppData/Local/nvim-data/lazy/
Here I opened this file:
C:/Users/"myusername"/AppData/Local/nvim-data/lazy/iron.nvim/lua/iron/fts/python.lua
And edited line 31 (for me that is), to point ipython to the python executable.

Now when I open IronRepl It opens a "python" terminal and I can send code to it and it will execute.
Hope this helps and you can keep using the great plugin! :)

Hiya,
I had this issue and I found a workaround that is suitable for me. Until the ipython issue is fixed, you can switch to use the python executable instead. Like instead of typing "ipython" to the terminal you will get a REPL that looks like as if you have entered "python" or "python3". Here is what I did:
I use lazy as my package manager on windows so my plugins are downloaded to this folder:
C:/Users/"myusername"/AppData/Local/nvim-data/lazy/
Here I opened this file:
C:/Users/"myusername"/AppData/Local/nvim-data/lazy/iron.nvim/lua/iron/fts/python.lua
And edited line 31 (for me that is), to point ipython to the python executable.

Now when I open IronRepl It opens a "python" terminal and I can send code to it and it will execute.
Hope this helps and you can keep using the great plugin! :)

I also switched to Toggle term. Exactly what I needed.

@g-plus-theworld @PyDataBlog @Alan-Manning Are you guys still using the pluggin and if so are you having the same problem. I am unable to reproduce the error you guys are having.

@g-plus-theworld @PyDataBlog @Alan-Manning Are you guys still using the pluggin and if so are you having the same problem. I am unable to reproduce the error you guys are having.

i am not using this anymore. just using toggle term and launching ipython console through that. I think that there may be some ipython settings for smart return/ make a new line and not run if command is incomplete, i cant recall what its called. i have had some issues with other setups ive tinkered with because of this and might not be specifically related to this plugin, However if you do find out what causes this, id imagine a section in the readme about that might be usefull to others

@Alan-Manning Hey how were you sending code to the terminal. where you using visual_send?

Use toggleterm plugin

@PyDataBlog mainly asking to see if the problem can be solved.

@PyDataBlog mainly asking to see if the problem can be solved.

I switched plugins so can't verify

@nickeisenberg I still have the issues described in this thread. My environment:

Windows 10 (Powershell 5.1.19041.429),
IPython 8.24.0
Windows Terminal 1.19.10821.0.

The behaviour seems to be the same, regardless if I use iron or toggleterm. I didn't do detailed tests with iron yet, but with toggleterm, the code is not executed regardless whether I use visual send or send current line.
If I launch with ipython --simple-prompt, the code is executed correctly.
@PyDataBlog did you do any specific configuration for toggleterm to get direct execution or did it work out of the box?
I'd be happy to see this issue resolved and can provide further input if needed.

Hey @roteiro, I think I fixed the issue. If you have time, could you do me a favor and test it on your end?  Can you use your package manager to install my fork of iron.nvim and see if the changes fix the ipython issue you were having. My fork is located at nickeisenberg/iron.nvim. I tested the changes on my end with windows11 and it seemed to be working. I think at the moment, only ipython is going to work with windows, at least that has been my experience when using powershell.

@nickeisenberg great, it works! I tested send line, send file and visual send. As I understood from your commit, you deferring the final  call. Could you elaborate on the root of the problem?

@roteiro What do you mean by many lines? I thought that the defer should only be used if you send one line. I am unsure what the real issue is because it doesnt occur on linux or mac. I can try and mess around with it some more.

https://github.com/jpalardy/vim-slime/issues/297#issuecomment-1657289152
This issue sounds very related and the linked comment has a possible solution in vimscript


fix: Java version error; Add missing #! line

This PR solves the following problems:

Upgrade Java version from 8 to 11, since GitHub runner macos-latest no longer supports Java 8 (see actions/setup-java#625).

Add missing #! line in the file ~/.local/bin/jython, which may cause an error when running jython by Python subprocess module.

Please review the changes. Thank you!

And this action seems not to work on Windows. When I just run "jython --version" using pwsh, I got the following error messages:

How can I fix it?

And this action seems not to work on Windows. When I just run "jython --version" using pwsh, I got the following error messages:

This is a known problem and is tracked by #8

Thank you for your submission @ZZy979!
I will review and eventually merge this PR as soon as I have spare time. Did you test that upgrading the Java version does not break compatibility?
I would like not to since almost all companies I know that use Jython also are stuck to Java 8. I am keen to providing a way to specify Java version (defaulting to 8)...
Do ytou know if it is possible to install legacy Jav 8 any other way?

This is a known problem and is tracked by #8.
Feel free to investigate the problem 😄

According to GitHub docs, different syntax should be used to add a system path on Windows PowerShell. I will try it later.

Did you test that upgrading the Java version does not break compatibility?

All CI workflows have succeeded (except for version 2.0 and 2.1, but I noticed they also failed in your last commit). As far as I know, only macos-latest doesn't support Java 8. Perhaps it's better to allow specifying Java version.

I've updated the changes by switching to the zulu distribution, and tests have succeeded in macos-latest. Please take some time to have a look!
As for the failing tests for Jython 2.0 and 2.1, I suggest you drop support for that old versions. Do you really need to keep the compatibility?

I've updated the changes by switching to the zulu distribution, and tests have succeeded in macos-latest

This looks good!

As for the failing tests for Jython 2.0 and 2.1, I suggest you drop support for that old versions

As already said here i cannot drop support for Jython 2.1 since my action is used in some projects of mine and in the toolchain of my workplace production build system.
I had to set for using windows runners instead of ubuntu (which are way faster) but it used to work perfectly, allowing me to discover bugs before they even went in production. I know that it's way too old but it's my work 😄


Fixing compile errors with bs-platform 2.2.0

Instead of using Js.Null.to_opt, we use Js.Null.toOption. Without this, compilation fails when using bs-platform 2.2.0.

Thanks! Unfortunately you've also heavily reformatted the modified files, so I'll apply this separately instead to reduce noise. But thanks for the heads-up!

Any time :)


Add WIN211 Branding

Add Win211 logo "Win211 Web Search" to top left and full logo to bottom of page.

Completed with 15a5741436f89a11a23a5bee418cedf7d19a21e7


Move nulling of controller before processing response

In certain cases on Safari, the app code utilizing <auto-check> might throw a abort error when trying to read the response. This error doesn't seem to happen on other browsers.
The hope is that this change will make sure that Safari will not raise a AbortError when trying to read a cloned, aborted response. But since this is really hard to test, we're not sure if this will actually fix the issue.
I'd like to ship this change as a prerelease on npm and update on dotcom and see if this change fixes it. If it does I can merge this and make a real release, if not I'll just revert the change in dotcom and close this PR.

This ended up not working.


Lasse's proposed cache invalidation strategy

Review PR #2362 and reopen it when ready for merge.

For the moment I see three problems with the PR:

It seems like there's not much memory usage difference when running a "normal" family-wide check. Instead it's rather only useful when doing e.g. a check run over the whole collection, where actually many cached ttFont objects make the memory usage really high.
The reference counting at the moment counts to much references, because it also counts usage that's indirect, i.e. When a condition A uses another condition B, B would be counted but in fact the cache of condition A would prevent B from being used each time when A is used. Hence, some caches are invalidated later than they have to.
Needs proper cleanup and maybe better design.


Precision of HuggingFaceEmbeddings.embed_query changes

System Info
Langchain version: 0.0.173
numpy version: 1.24.3
Who can help?
@hwchase17
Information

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

Related Components

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

Reproduction
from sentence_transformers import SentenceTransformer
import numpy as np
from langchain.embeddings import HuggingFaceEmbeddings

t = 'langchain embedding'

m = HuggingFaceEmbeddings(encode_kwargs={"normalize_embeddings": True})
# SentenceTransformer embeddings with unit norm
x = SentenceTransformer(m.model_name).encode(t, normalize_embeddings=True)
# Langchain.Huggingface embeddings with unit norm
y = m.embed_query(t)

print(f'L2 norm of SentenceTransformer: {np.linalg.norm(x)}. \nL2 norm of Langchain.Huggingface: {np.linalg.norm(y)}')

Expected behavior
Both of these two L2 norm results shoud be 1. But I got as blow:
L2 norm of SentenceTransformer: 1.0.
L2 norm of Langchain.Huggingface: 1.0000000445724682

I think the problem came from this code. When converting array to list, the numbers got bigger

In my case, when I used this embedding in FAISS vector store, the relevance_score I got cannot be limited between 0 and 1


OpenJDK: Fix incorrect version for macOS aarch64

It looks like using 17.30.19 for aarch64-darwin is wrong.
There is currently no 17.30.19 in https://cdn.azul.com/zulu/bin/
The sha256 seems to match the 17.30.15 macOS aarch64 file.
My guess is that either:

there was a version 17.30.19 with the same hash that was removed.
the .30.19 for aarch64 was copied over from 16.nix (there is a 16.30.19), the
sha256 was calculated using the 17.30.15 (perhaps with nix-prefetch-git) and
the file did not need to be downloaded when testing since it was already in the store.

merged https://github.com/NixOS/nixpkgs/pull/150006


Facilitate the use of Minify to output static files

Note: this is not a defect, but an enhancement suggestion.

By design, Minify and similar solutions represent a small performance
overhead, since even with filesystem caching the PHP script must be
executed, check if the files were changed, check if the cache exist, etc.
(I may be wrong for some parts, since i don't know the code and i'm not a
PHP developer.) This is OK for small-to-medium-sized websites, but might be
a problem for high-traffic websites.

I'm not criticizing Minify or the optimization efforts of its developers,
but the fact is serving static files is simply more efficient.

While testing Minify (which is great), i used it to configure some groups,
then used a simple shell script to wget the resulting files and write them
to disk.

wget http://media.domain.tld/min/?g=common.css -O /some/path/css/common.css

It works fine. But maybe this could be a feature of Minify, as a
configurable option?

You may want to mark this suggestion as WontFix or Invalid, not a problem
with me. I just wanted to suggest this and have your opinion. :)

Original issue reported on code.google.com by fverschelde on 26 Apr 2009 at 10:01

The PHP overhead is certainly a concern (though in my limited testing I found
Minify
outperformed Apache with mod_deflate, which apparently doesn't cache encoded
versions). Also high-traffic sites should probably use a reverse proxy cache
like
squid, which would cache Minify's output.

My issue with releasing workflows with Minify is that *everyone* has a unique
workflow/build process with different variables to consider: files hosted on
separate host/CDN? CDN have mod_rewrite (so you can rev URLs)? Should source
files
be monitored for changes? How would we do this if PHP is out of the loop? cron?
What
about IIS?

Maybe a simple tool to output all the groups defined in groupsConfig.php and
an .htaccess file would be a nice first step...

If you have more ideas, hit up the Google group.

Original comment by<EMAIL_ADDRESS>on 29 Apr 2009 at 8:58

Changed state: WontFix
Added labels: Type-Enhancement
Removed labels: Type-Defect


discover step fail if beakerlib library not found in the repo

If there's an existing repo on github.com/beakerlib/<COMPONENT> but the library is not there the discover step fails as git command cannot find the library there. This way it even does not try to satisfy the requirements from possibly enabled dnf repo.
I.e. if I want to move one of the distribution libraries to github I will break the others.
The correct behaviour should be to continue and try to satisfy the library requirement in different execution phases.

This should be already fixed by #382. What version of tmt are you testing with?

$ rpm -q tmt
tmt-1.0-1.20201007135237127416.master.37.gb5844a7.fc31.noarch

$ tmt run -vvvddd -a provision -h connect -g <IP_ADDRESS> prepare -h shell -s 'dnf config-manager --enable beaker-tasks' execute -h beakerlib.tmt tests --name .
...
    discover
        workdir: /var/tmp/tmt/run-011/plans/default/discover
        how: fmf
        order: 50
            Run command 'git rev-parse --show-toplevel'.
            out: /home/dapospis/test/git/SP_components/tests
        directory: /home/dapospis/test/git/SP_components/tests
            Copy '/home/dapospis/test/git/SP_components/tests' to '/var/tmp/tmt/run-011/plans/default/discover/default/tests'.
            Check metadata tree in '/var/tmp/tmt/run-011/plans/default/discover/default/tests/'.
            Detected library 'library(distribution/dpcommon)'.
            Fetch library 'distribution/dpcommon'.
            Run command 'git clone https://github.com/beakerlib/distribution /var/tmp/tmt/run-011/plans/default/discover/default/libs/distribution'.
            err: Cloning into '/var/tmp/tmt/run-011/plans/default/discover/default/libs/distribution'...
            err: remote: Invalid username or password.
            err: fatal: Authentication failed for 'https://github.com/beakerlib/distribution/'
            Command returned '128'.
            Repository 'https://github.com/beakerlib/distribution' not found.
            Detected library 'library(epel/epel)'.
            Fetch library 'epel/epel'.
            Run command 'git clone https://github.com/beakerlib/epel /var/tmp/tmt/run-011/plans/default/discover/default/libs/epel'.
            err: Cloning into '/var/tmp/tmt/run-011/plans/default/discover/default/libs/epel'...
            Run command 'git checkout master'.
            err: error: pathspec 'master' did not match any file(s) known to git
            Command returned '1'.

The problem above is that the epel repository does not have a master branch. It only has the main one, which seems to be the new github default for main branches. So far (according to the spec) we expect only master. Let's fix the repo. We could also check for main in the future but that needs some discussion.

I did not even notice that. Shouldn't tmt look at the default branch instead of hard-coded master?

ok, adding master branch moved me a bit forward
...
go
    discover
        workdir: /var/tmp/tmt/run-012/plans/default/discover
        how: fmf
        order: 50
            Run command 'git rev-parse --show-toplevel'.
            out: /home/dapospis/test/git/SP_components/tests
        directory: /home/dapospis/test/git/SP_components/tests
            Copy '/home/dapospis/test/git/SP_components/tests' to '/var/tmp/tmt/run-012/plans/default/discover/default/tests'.
            Check metadata tree in '/var/tmp/tmt/run-012/plans/default/discover/default/tests/'.
            Detected library 'library(distribution/dpcommon)'.
            Fetch library 'distribution/dpcommon'.
            Run command 'git clone https://github.com/beakerlib/distribution /var/tmp/tmt/run-012/plans/default/discover/default/libs/distribution'.
            err: Cloning into '/var/tmp/tmt/run-012/plans/default/discover/default/libs/distribution'...
            err: remote: Invalid username or password.
            err: fatal: Authentication failed for 'https://github.com/beakerlib/distribution/'
            Command returned '128'.
            Repository 'https://github.com/beakerlib/distribution' not found.
            Detected library 'library(epel/epel)'.
            Fetch library 'epel/epel'.
            Run command 'git clone https://github.com/beakerlib/epel /var/tmp/tmt/run-012/plans/default/discover/default/libs/epel'.
            err: Cloning into '/var/tmp/tmt/run-012/plans/default/discover/default/libs/epel'...
            Run command 'git checkout master'.
            err: Already on 'master'
            out: Your branch is up to date with 'origin/master'.
    finish
        workdir: /var/tmp/tmt/run-012/plans/default/finish
        summary: 0 tasks completed
        status: done
        Write file '/var/tmp/tmt/run-012/plans/default/finish/step.yaml'.
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/tmt/beakerlib.py", line 105, in fetch
    library = self.parent._library_cache[self.repo]
KeyError: 'epel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/tmt", line 8, in <module>
    tmt.cli.main()
  File "/usr/lib/python3.7/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.7/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.7/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.7/site-packages/click/core.py", line 1290, in invoke
    return _process_result(rv)
  File "/usr/lib/python3.7/site-packages/click/core.py", line 1224, in _process_result
    value = ctx.invoke(self.result_callback, value, **ctx.params)
  File "/usr/lib/python3.7/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.7/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/lib/python3.7/site-packages/tmt/cli.py", line 254, in finito
    context.obj.run.go()
  File "/usr/lib/python3.7/site-packages/tmt/base.py", line 944, in go
    plan.go()
  File "/usr/lib/python3.7/site-packages/tmt/base.py", line 536, in go
    step.go()
  File "/usr/lib/python3.7/site-packages/tmt/steps/discover/__init__.py", line 124, in go
    plugin.go()
  File "/usr/lib/python3.7/site-packages/tmt/steps/discover/fmf.py", line 165, in go
    test.require, test.recommend, parent=self)
  File "/usr/lib/python3.7/site-packages/tmt/beakerlib.py", line 180, in dependencies
    library = Library(dependency, parent=parent)
  File "/usr/lib/python3.7/site-packages/tmt/beakerlib.py", line 91, in __init__
    self.fetch()
  File "/usr/lib/python3.7/site-packages/tmt/beakerlib.py", line 140, in fetch
    self.tree = fmf.Tree(directory)
  File "/usr/lib/python3.7/site-packages/fmf/base.py", line 75, in __init__
    self._initialize(path=data)
  File "/usr/lib/python3.7/site-packages/fmf/base.py", line 101, in _initialize
    os.path.abspath(path)))
fmf.utils.RootError: Unable to find tree root for '/var/tmp/tmt/run-012/plans/default/discover/default/libs/epel'.

just for the record, previously I was using tmt-0.21-1.fc31.noarch

Ok, we should nicely handle when there are no fmf metadata in the library repository. Should be fixed by #399.


Neural Networks

I want to write about this topic. @the-ethan-hunt Please assign.

Assigned to @mohit2016


HTTPS is not filtered on Android on dailyuploads.net

Discussion:
https://forum.adguard.com/index.php?threads/dailyuploads-net-pop-up-android.25103/
Problematic link:
https://dailyuploads.net/tzqsqvj247cm
We should re-check it once we're close to release.

Works okay in the CoreLibs integrated Android build


feat: Add initial .editorconfig

Adds indentation and whitespace styling

Added a few more possible config options, thinking being

if (...) then on same line, not a fan of
braces on same line, not a fan of
single line statements/blocks seems good for get/setters otherwise I'd go against
keep indentation consistent
Sort imports is nice (is there a way to remove redundant ones? couldn't see one)
Agree with 5 lines there right now

Had a look at the dotnet preferences as well and set true any that seemed like a sensible thing to be favouring, that just improve readability
What I really wanted to add to this config file, that I couldn't see any editor config options for, was the formatting of chaining methods on multiple lines, and aligning those in a consistent way - (something like the final comment here) but yet to find a way to add that

All changes look good 👍
is there a way to remove redundant ones? couldn't see one
Yes and no.
If you change the severity of the warning to error, then dotnet format will automatically fix them.
E.g.
## Set warnings to errors to be auto-fixed by dotnet format

# Using directive is unnecessary
dotnet_diagnostic.IDE0005.severity = error
# Duplicate using directive
dotnet_diagnostic.CS0105.severity = error

followed by
dotnet format plan-technology-for-your-school.sln
Will remove any unnecessary using directive.
On a side note, I think having both IDE0005 and CS0105 is unnecessary, as IDE0005 also covers CS0105.

If you change the severity of the warning to error, then dotnet format will automatically fix them.
On a side note, I think having both IDE0005 and CS0105 is unnecessary, as IDE0005 also covers CS0105.

Great shout, done


Add volume property to player

There is no volume property within the player. Maybe can you add one?

What do you mean by "within the player". There is already volume control for a per player basis

Yes, I understand you can adjust the volume, but there's no way to get what the current volume is set to.

The best way that I do this is to have the volume stored in my database that stores settings per server, and uses that to set the volume and return what the volume is. But i see now what you mean


Add unit tests for database queries

How to: https://www.oreilly.com/library/view/essential-sqlalchemy-2nd/9781491916544/ch04.html

https://github.com/yzernik/squeaknode/pull/1488


Fix Number.is* to accept unknown again

Fixes #34931
This essentially reapplies #24436 which incorrectly updated the generated files rather than the sources, and so was wiped out by 2f73986b44a4ec10c7ebe7188e23863c3879d54e.
Note that this is specifically not the same as #4002, which pertains to the global functions that coerce their arguments and therefore should not accept unknown.  This change was already accepted 18 months ago, but was applied incorrectly.

I looked for a test that could be updated but I have no idea what would be appropriate - I found no unit tests that pertained to any of these declarations.  If there's something I can update to ensure it sticks this time, please advise.

Time to add new tests!

Guys. I'm sorry. But what are you waiting for? Why don't you merge PR? The correct behavior has been broken and needs to be restored.

Cool, checked over the history and these fixes look good IMO