[AutoPR storagesync/resource-manager] [Hub Generated] Review request for Microsoft.StorageSync to add version 2019-02-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5210

(message created by the CI based on PR content)

Codecov Report

Merging #4371 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4371      +/-   ##
==========================================
- Coverage   53.41%   53.27%   -0.14%
==========================================
  Files       10460    10454       -6
  Lines      219328   219797     +469
==========================================
- Hits       117153   117096      -57
- Misses     102175   102701     +526

Impacted Files
Coverage Δ

...application_gateway_autoscale_configuration_py3.py
61.53% <0%> (-9.9%)
:arrow_down:

...zure-servicebus/azure/servicebus/common/message.py
28.63% <0%> (-7.2%)
:arrow_down:

azure-servicebus/conftest.py
33.1% <0%> (-4.72%)
:arrow_down:

...ge/azure/mgmt/storage/storage_management_client.py
42.95% <0%> (-3.25%)
:arrow_down:

...te/azure/mgmt/compute/compute_management_client.py
47.29% <0%> (-2.96%)
:arrow_down:

.../mgmt/containerservice/container_service_client.py
57.74% <0%> (-2.78%)
:arrow_down:

...t/authorization/authorization_management_client.py
66.3% <0%> (-2.39%)
:arrow_down:

...erregistry/container_registry_management_client.py
71.79% <0%> (-2.23%)
:arrow_down:

...servicebus/control_client/_common_serialization.py
51.41% <0%> (-2.15%)
:arrow_down:

...t-legacy/azure/servicemanagement/_serialization.py
81.31% <0%> (-2.02%)
:arrow_down:

... and 32 more

Continue to review full report at Codecov.

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

See https://github.com/Azure/azure-sdk-for-python/pull/2734


SSL Socket option needed

I was trying to intercept the flow between a client and server using HTTP protocol. Everything was good before client set up another SSL-socket without HTTP header to the server.
Mitmproxy hints: "HTTP protocol error in client request: Bad HTTP request"
Client sent JSON-like data could be decoded by mitmdump like:" { "request":"John" , "name":"foo" }".
Then mitmproxy closed the connection.
So, is there any way to get an option to disable HTTP protocol parser and just forward these socket packets?

This should work for you: http://docs.mitmproxy.org/en/stable/features/tcpproxy.html

Thanks!  It works.
Although I noticed that interception and modification are not available, I still appreciate your decent work.
I will temporarily make some dirty patches to modify these traffic.
Do you have plans to implement the TCP data inline script hook in the future?

You can modify TCP messages in scripts: https://github.com/mitmproxy/mitmproxy/blob/master/examples/complex/tcp_message.py
It's very difficult to get right considering TCP fragmentation etc., but should be good enough locally.


Added Merge Sort in cpp folder

Please merge it and provide hacktoberfest accepted tag

PLEASE READ NOTE ON README


Change link for local docker startup in readme.md

When the docker build was changed to a multi-stage build a step got added that renames the README.html to index.html.
Currently, following the link in the readme.md you receive a 404.
Relevant commit: https://github.com/domnikl/DesignPatternsPHP/commit/be66dcc037f8b2293134efe3cc3f2e87a25b8f80

Duplicates PR #404


Need an extra line after the sourceMappingURL in ractive.js 0.7.1 to make it compatible with r.js optimizer

At the end of ractive.js 0.7.1, this line exists:
//# sourceMappingURL=ractive.js.map
I have a personal project that runs r.js and concatenates JS libraries together into a single file.  r.js adds a semicolon at the end of whatever file is it concatenating, so it ends up like this:
//# sourceMappingURL=ractive.js.map;

When I view the site in Chrome, the console always says:
http://SOMESITE.NET/j/ractive.js.map; not found (404)

When I went into the ractive.js file and hit the return button to create a new line, and then ran the r.js optimizer, the error went away in Chrome.
The concatenated file now looks like this:
//# sourceMappingURL=ractive.js.map
;
/* NEXT concatenated file's code */

So, is it possible to add a new line to the end of ractive.js?

Thanks - I've reported the bug in r.js (https://github.com/jrburke/r.js/issues/799) and will add a workaround to the tool that combines sourcemaps (https://github.com/Rich-Harris/sorcery/issues/4) - it'll be fixed in the next release

Rich, thanks for providing a workaround in the next version and alerting the r.js developer.  You guys are staying on top of things!
I'll stay on 0.6.1 until the next release.

This has been fixed upstream, so current edge (and future stable) builds will have the trailing newline. If you get a chance to test it out with http://cdn.ractivejs.org/edge/ractive.js to make sure that it works with r.js, I'd be grateful!

Rich, I downloaded the revised file, and r.js puts the semicolon on the new line you created.  The browser no longer shows the "sourcemapping not found" error.   I think you're good to keep using this, until rburke fixes r.js.
I'm currently using Bower to manage my front-end dependencies, so I'll have to wait until you make a new release before actually using this change.

Thanks a lot for confirming that it works.

I'm currently using Bower to manage my front-end dependencies

I'm so sorry :) Just so you know, it is possible to use the edge version with bower, if that's what you'd prefer:
# to get the latest successful build:
bower install ractive#edge

# to lock to a specific build (from https://github.com/ractivejs/ractive/commits/build)
bower install ractive#4243006e1a96c76ef5ed455f527cb2f7cdf47d73

Although this is a tangent to the original issue, is there something wrong with Bower?  I just find it a little  easier and more lightweight to use bower than npm (although I do use npm for the Grunt dependencies - I'm not on Gulp either).
I just didn't know there was a way to get the edge version in Bower, so thank you for letting me know how to target edge and specific builds.  (Bower documentation didn't mention this, so I didn't know this was possible.)

It's a personal opinion, and I probably shouldn't use this as a soapbox, but after using it for a long time I realised it simply doesn't do the things that a package manager needs to do. With an npm package, you can guarantee that a) if you download package x and require it, the resolver will be able to find x because if it's not in the default location there'll be a main field in the package.json that points to where it is, and b) you're almost certainly getting a CommonJS module (deviations from that are rare).
Bower gives you no such guarantees - a lot of bower packages have incomplete/incorrect bower.json files, and the library could be in any format (CJS, AMD, globals...), and because there's no bower equivalent of node's resolver, you have to wire up all the locations yourself (<script src='bower_components/...'> or a convoluted require.js config, or whatever) which totally defeats the object of using a package manager in the first place!
I have other issues with it, but that's the main one. Eventually I switched to npm+browserify, and I've never looked back.

Rich, thanks for sharing your opinion on this.  You're way more experienced than me, so it's good to hear what some of the problems.  Since my personal project (and even some corporate work I've done!) isn't overly complicated, I've never really saw the shortfalls.  I just used Bower, because it interfaced with Github repos, as sometimes I couldn't find an equivalent npm module (but I'm sure that's changing).
That's the end of this tangent.  Back to Ractive.js!


User-defined webpack aliases are not resolved

Version
3.0.0-beta.6
Reproduction link
https://codesandbox.io/s/k2noq6mqy5
Steps to reproduce
Just run the app, the error will appear on compilation.
What is expected?
The homepage of the default vue-cli template should display.
What is actually happening?
A compilation error: "Could not find dependency: 'src'"

I added a webpack alias, src, to the vue.config.js file, then changed the App component to import HelloWorld via that alias.
When I tried this locally:

I looked at inspect and found that my webpack config was being merged as I expected
There were some other aliases provided by vue-cli, including @ which pointed to the same directory as my src alias
When I used @ in my code, it compiled, but src gave me a compilation error.

codesandbox does not run on vue-cli 3

Will close this as invalid.


Revert "Fix spelling mistake for copy-to-clipboard plugin page"

Reverts PrismJS/prism#1866

Since it was just a docs change, I don't think we need to revert it. We do need to merge the fix into master though or it won't show up on our site.

Agreed. Let me do that real quick.


ValueError: num_samples should be a positive integer value, but got num_samples=0

Dear All, I got this error could anyone solve it?
ValueError: num_samples should be a positive integer value, but got num_samples=0

If you just want to run this on custom images, open demo.sh and uncomment the command corresponding to # Test your own images (Line 32). Comment out the rest, and it should work.

thank you ， I got another error it's
'rm' is not recognized as an internal or external command,
operable program or batch file.
how I can solve it please

@abbaskali Are you using Windows or Ubuntu operating system?

@haikunzhang95 windows operating system

@haikunzhang95 windows operating system

Maybe you should use the Ubuntu operating system?

@haikunzhang95 thank you Dr.Zhang i will try using Ubuntu and run the code

self.loader_train = dataloader.DataLoader(
MyConcatDataset(datasets),
batch_size=args.batch_size,
shuffle=False,
pin_memory=not args.cpu,
num_workers=args.n_threads,
)
maybe you can try to change the "shuffle = True" to "shuffle = False"


Board Review: Introducing Azure EASM

Contacts and Timeline

Service team responsible for the client library:

<EMAIL_ADDRESS>

Main contacts:

Nate Falke<EMAIL_ADDRESS>@nathanfalke
Brian Zak<EMAIL_ADDRESS>@eqtip
Nisha Bhatia<EMAIL_ADDRESS>@nishabhatia-msft

Expected stable release date for this library:

About the Service

Link to documentation introducing/describing the service:

External Attack Surface Management

Link to the service REST APIs:

2022-04-01-preview/easm.json

Is the goal to release a Public Preview, Private Preview, or GA?

Private Preview

About the client library

Name of client library:

Azure-EASM
Azure-Mgmt-EASM

Link to library reference documentation:

Azure-EASM
Azure-Mgmt-EASM

Is there an existing SDK library? If yes, provide link:

No.

Step 1: Champion Scenarios

Data Plane Champion scenarios

Create and view discovery runs
Create discovery groups from templates
Use a saved filter to synchronize queries
Tag assets with external IDs
Create Sentinel Data Connection

Control Plane Champion scenarios

Create a workspace
Create labels

Step 2: Quickstart Samples (Optional)

Data Plane Quickstart Samples
Control Plane Quickstart Samples

Drafted

@nathanfalke, please include any materials like package names, champion scenarios, etc. for other programing languages.

We currently only have materials for a Python SDK

@nathanfalke it would be helpful if you could upload the API View links for the python libraries. Thank you.

Recording (MS INTERNAL ONLY)
Approved names:

.NET: Azure.Analytics.Defender.Easm
Java: com.azure.analytics.defender.easm
Python: azure-defender-easm
JS/TS: @azure/defender-easm


Update Composer dependencies (2024-05-09-00-17)

Visual regression test passed!
View the visual regression test report


Update R8 and then make a new release of Gr8

          It should be released by now. What we need to do to update R8 in here and then make a new release of Gr8 and then use that new Gr8 release in `main` and then we can remove the limitation. Bootstrap all the way down 😅

Note that in this specific case, toolchains wouldn't help anyways because R8 takes the runtime libs from the current JRE (unless we used a separate JavaExec task to run R8 itself but there are so many hours in a day sadly).
Originally posted by @martinbonnin in https://github.com/GradleUp/gr8/pull/22#discussion_r1804750655

Almost there but it was getting late yesterday. Will try to finish this today

Fixed in https://github.com/GradleUp/gr8/pull/27


Add option for only flattening nested maps, not arrays

Hi! Not sure if this library is being maintained, but I'm using it for https://github.com/alexwennerberg/json2csv (currently a WIP). I think something that would be valuable would be an option to only flatten nested maps, but not flatten arrays, or to only flatten nested arrays, but not nested maps. I may submit a pull request!

Hi @alexwennerberg, I honestly forgot I built this :). I'll start maintaining it now. Would be happy to take a PR!

I put together a PR but realized that it is not necessary for my use case, so I closed it. I'll be keeping an eye on this repo though during development


Py3.6+ detaches from python 3.5

Better typing
Mass MyPy fixes
f-strings

Pull Request Test Coverage Report for Build 559

87 of 93 (93.55%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 98.257%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

logwrap/log_wrap.py
38
44
86.36%

Totals

Change from base Build 558:
0.03%

Covered Lines:
451

Relevant Lines:
459

💛  - Coveralls


Can EIP-Bot report a neutral status instead of a failure?

Not sure if this'll work with kodiak, but it would be nice if EIP-Bot reported a neutral status until all required approvals have arrived. That way the check result will only show a red "X" when there's an actual issue that needs addressing.

Completed in #109.


Addition of home icon to various games

PR Description 📜
As we click on one of the listed games we need to click on the arrow to get back to home .So, I have included the home icon to various games except few left as I am not able to load the games on my laptop.
Fixes # 1620

Mark the task you have completed ✅
Addition of home icon completed

[x] I follow CONTRIBUTING GUIDELINE & 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 have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional)
Issue to be addressed

After resolving issue

Thank you soo much for contributing to our repository 💗

Thank you @KanchandeepK ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @KanchandeepK,
Please make sure to link the relevant issue using the appropriate syntax, such as "#issueNumber" 👀.
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @KanchandeepK , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

@kunjgit Do I need to again create a pull request as I have mentioned the correct issue .

@KanchandeepK  dont keep a space between the # and the issue number, otherwise it won't count.

@5h0ov is it required to add the screenshot in assets or is it fine.


[Backport release-24.05] thunderbird: 128.4.2 -> 128.4.3

https://www.thunderbird.net/en-US/thunderbird/128.4.3esr/releasenotes/
(cherry picked from commit b2e2858135d8c95b04e485a9c8e0cd897b926bd6)
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

Partial backport of PR #356089 to release-24.05.


Visualize FlowNode graph

This is significantly easier to understand than the antecedent tree view:

This does a BFS on the FlowNode graph (via antecedent / antecedents) and produces a graph spec using dot format. It then uses viz-js to turn this into an SVG for display.
If this is of interest, we can work on tuning the display, options and interactivity.
cc @JoshuaKGoldberg, who was interested in this

Oh my god. This is glorious 😍  I was dumping flowNode.__debugToString and interpreting ASCII diagrams for mooonths.
Thank you ❤

Thanks @Andarist! Regardless of what happens to https://github.com/microsoft/TypeScript/pull/57465, this is at least a nice, tangible win that came out of it. I found flow nodes completely incomprehensible until I did this :)


Research common testing framework for Core based SDKs

yes - in my mind this is super high priority. not sure that it is relevant to /proposals though


Upgraded Firebase to v9 and made some small tweaks

This pull request fixes #17

@Prashoon123 the build is failing

@avneesh0612 I have made some changes let me know if everything works.


[Feature] Model Specific Computed Properties

In a few cases i've found my self really wishing there were model specific computed properties.
Basic example:
Post with Comments. Have a computed property on the Post model to cache number of unique users in the comment discussion.
eg: Post.numOfCommentors
Easy work around for now i typically wrap a model from a list into a vue instance to pass to a template that I need the getters in. Doing so is definitely dirty and hacky so it would be great to get this worked directly in so they are computed as soon as a model is created.

This is already possible using es5 accessors.  You can use both getters and setters:
service('post', {
  instanceDefaults: {
    description: '',
    isComplete: false,
    comments: [],
    get numberOfCommenters () {
        // Put your logic here.
    },
    set someOtherProp () {
        //  Setters also work
    }
  }
})

I've added this to the documentation, here: https://feathers-plus.github.io/v1/feathers-vuex/common-patterns.html#Model-Specific-Computed-Properties
Thanks!


Add supported node version

Adds the required node version to the setup instructions and the engines field in the package.json
Issue: https://github.com/patternfly/patternfly-next/issues/40

very nice @cliffpyles!  didn't know about the engine param in package.json...learned something new.  neat!


IPv6 not preferred when -Djava.net.preferIPv6Addresses=system is set

We're setting -Djava.net.preferIPv6Addresses=system and we've found Netty doesn't honor the setting, because Netty implements IPv4/IPv6 loopback address/interface detection itself, causing different results from InetAddress.
In recent JDKs, LookupPolicy takes care how addresses are resolved, but the logic is essentially the same in all releases since JDK 9, but it's harder to follow in releases before LookupPolicy was added. In short:

If -Djava.net.preferIPv4Stack=true is set, IPv4 is forced
If the system is IPv4 or IPv6 only, the available protocol is used
Otherwise both IPv4 and v6 are enabled. The preferIPv6Addresses value determines the ordering of addresses:

For false IPv4 addresses are sorted first
For true IPv6 addresses are sorted first
For system addresses are provided in the order getaddrinfo returns them in

If no preference is specified via either flag, IPv4 addresses are ordered first. Equivalent to preferIPv6Addresses=false

The NetUtil implementation differs to this, and doesn't implement the system value. The real problem is the address reordering caused by the implicit preference:

https://github.com/netty/netty/blob/4eda913d998898d8489565904b2a5d133d081413/resolver-dns/src/main/java/io/netty/resolver/dns/DnsAddressResolveContext.java#L63-L67
https://github.com/netty/netty/blob/4eda913d998898d8489565904b2a5d133d081413/resolver-dns/src/main/java/io/netty/resolver/dns/PreferredAddressTypeComparator.java#L30-L39

The system value is required for mixed environments, because the combination of true/false for the two settings causes connectivity issues depending on the combination of source/destination being IPv4-only, IPv6-only or dualstack:

We chose system because it avoids the complexity of deciding whether it's safe to set -Djava.net.preferIPv6Addresses=true.
Expected behavior
Netty's selection of IP version, loopback address and name resolution order is identical to the JDK.
Actual behavior
The preferIPv6Addresses=system is not honored, causing addresses to be reordered based on the default preference, making it impossible to safely prefer IPv6 in mixed-protocol environments.
Steps to reproduce
Run with:
-Djava.net.preferIPv6Addresses=system

Note the failure:
io.netty.util.internal.SystemPropertyUtil: Unable to parse the boolean system property 'java.net.preferIPv6Addresses':system - using the default value: false

I don't quite follow why this is implemented explicitly, rather than calling io.netty.util.internal.SocketUtils#loopbackAddress and checking the instance of the returned address, that seems safer, with a check for the system value to determine if address reordering should be disabled.
Netty version
At least 4.1 and later.
JVM version
JDK 9 and later.
OS version
All operating systems.

It also seems that the class RoundRobinInetAddressResolver re-order the list of IPs with no consideration with the inet family of those addresses.


Fix config directories in getting started Doc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @kaushikb11 ！First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you are Chinese or have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


⚠ Spryr has degraded performance

In d384500, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4057 ms

Resolved: Spryr performance has improved in 30161ff after 20 minutes.


Testing if thread are joinable in destructor

Closing as the problem has been identified and will be fixed in a dedicated PR.


Question: How to have a fixed footer?

I used the footer to show monitoring status. I want to have it fixed on scroll. How can achieve this?

I got it.
.main-footer {
bottom: 0;
left: 0;
position: fixed;
right: 0;
z-index: 999;
}
Thanks.


Ubuntu 20 support

Heyo, I'd like to add support for Ubuntu 20 but I'm not sure where to start.  FWICT I need to add an image_alias but I can't seem to locate where the mapping between aliases and AMIs are located.  Could I get some pointers?

The image aliases are stored in the server side dynamodb store, which you (thankfully!) don't need to access directly as the API allows CRUDdy managament:
$ fuzzbucket-client create-alias trusty ami-08bc77a2c7eb2b1da
# fuzzbucket:INFO:2020-06-09T111127:: created alias for user='meatballhat' alias=trusty ami=ami-08bc77a2c7eb2b1da
[image_aliases]
trusty = ami-08bc77a2c7eb2b1da

$ fuzzbucket-client la
[image_aliases]
centos6 = ami-03a941394ec9849de
centos7 = ami-0affd4508a5d2481b
centos8 = ami-0e7ad70170b787201
opensuseleap = ami-05120304eb067176e
rhel6 = ami-0b1d4c689c7949a64
rhel7 = ami-0916c408cb02e310b
rhel8 = ami-0c322300a1dd5dc79
sles12 = ami-1eeab909
sles15 = ami-0b1764f3d7d2e2316
trusty = ami-08bc77a2c7eb2b1da
ubuntu16 = ami-08bc77a2c7eb2b1da
ubuntu18 = ami-046842448f9e74e7d

$ fuzzbucket-client delete-alias trusty
# fuzzbucket:INFO:2020-06-09T111423:: deleted alias for user='meatballhat' alias=trusty

Ah!  Thanks :)


First version release

Let's wait until the Node.js pull request is merged to release the v1.0.0
Fixes https://github.com/ada-url/ada/issues/207
Fixes https://github.com/ada-url/ada/issues/142

@anonrig Tests fail with a ccache error?

It seems Azure is having some issues:
Err:1 http://azure.archive.ubuntu.com/ubuntu focal/main amd64 ccache amd64 3.7.7-1
[27](https://github.com/ada-url/ada/actions/runs/4104094755/jobs/7079124017#step:3:28)
    Connection failed [IP: <IP_ADDRESS> 80]

Re-running the workflows one by one fixed the issue. It seems it was a temporary issue with Azure networks.


Add advent-of-code-2024

Repository URL
https://github.com/robprestezog/advent-of-code-2024.git
Crate name
No response
Toolchain version
No response

It looks like this was failing due to my inclusion of .git in the Repository URL. I tried again with the .git (see Issue 206) and was successful.


Allow random gender for personal data

Currently one of 'male' or 'female' must be given (or defaults to 'female') for personal data. It would be nice if the gender would be randomly generated.
I have made a sample implementation allows to pass gender=None to all functions of the personal provider. If None is given, the gender is randomly generated. The implementation can be found here:
https://github.com/jrast/elizabeth/tree/random-gender
The changes can be seen in this commit: https://github.com/jrast/elizabeth/commit/d746eaef612ac328cbc249eb8533ac4366b9d50c
If there is interest, I can create a pull request.

Looks good. PR, please.

PR https://github.com/lk-geimfari/elizabeth/pull/155 is out.
Please see also the note about the default in the comment on the pull request.

@jrast Look at comments by @wikkiewikkie in PR #155, please.


Update readme after code transition.

use Apache Dubbo (incubating)
update official site to dubbo.incubator.apache.org

Not only for incubator-dubbo, we also need to update readme for all apache/incubator-dubbo* repos.

Closed. Other repos should track with new issues separately.


Colorscheme does not work with NeoVim in certain environments

Problem
Colorscheme does not work with NeoVim in certain environments. It turns black as shown in the picture below.

Envirionments
I tested some environments. In every case, the latest commit:
5a195858657e1c494e77ea868073d2f3664cdb60 was cloned and compiled by myself.
Tested .nvimrc is here

Envirionment
on terminal
on tmux

Max OS X 10.10.4
o
x

ArchLinux
o
o

Ubuntu 14.04 (on Docker)
x
x

o: colored, x: black

Close due to inactivity.

I have also found this to be a problem. It makes using neovim or lightline mutually exclusive, and both are great.

+1

I tested on tmux 0.1.4, Mac OS X 10.11.5, the latest version of lightline with the following vimrc but all things seem to be working well. If you have problem on this, please describe your environment and configuration precisely so that I can reproduce the same environment.
set nocompatible
filetype off
set rtp+=~/path/to/lightline
let g:lightline = {}
set laststatus=2
filetype plugin indent on
syntax enable

Sorry to reopen old issues:
On Ubuntu 17.04, using tmux 2.3, vim 8.0, and latest lightline, i have the same coloring issues:
Works well outside tmux
No colors inside tmux
My complete .vimrc:
execute pathogen#infect()

set nocompatible
filetype off
set rtp+=~/.vim/bundle/lightline.vim
let g:lightline = {}
set laststatus=2
filetype plugin indent on
syntax enable

lightline is currently the only plugin installed

Please let me know the result of :echo [&t_Co, $TERM] in Vim.

There we have the culprit:
Outside tmux: ['256', 'xterm-256color']
Inside tmux: ['8', 'screen']
Now, vim in  general doesn't seem to mind for syntax highlighting works inside tmux with all colors of the rainbow....

There's a note for $TERM and t_Co in README.md.

Ah... another case of RTFM. I actually followed the part above with laststatus=2 but then assumed the rest doesn't apply to me b/c its coloring by default... I was guessing in a completely different direction...
Sorry to bother you and thank you for your help

Never mind, good luck.


How to Skip Booting

i want enter to windows without booting

I'm in the midst of refactoring the codebase to implement the Switchable UI feature. I will consider adding the fast boot option later. Thanks for your suggestion.


Computer Science Primer- Link not working

http://www.grokit.ca/spc/computer_science_review/
The URL doesn't host any content, can it be replaced with the correct version ?

I decided to remove it


Option to process links and images?

Hey, great library, thanks for this.
I noticed that when you call process_html to get the text to send to the LLM, you don't provide options to (not) ignore images and links. I need to get links and images from the page as well.
Would you consider adding this option? If so, I can try to put together a PR.

Fixed in https://github.com/Blacksuan19/scrapy-ai/pull/2.


deposit: set accordion style

requires https://github.com/inveniosoftware/invenio-theme/pull/235


Feature request: Ability to see progress

In README there is not noticed methods or callbacks to see progress. Basically, I want to show user that something is being downloaded and hide it when all downloads are finished.
Could you add callback "finished" and "progress" (with count of remaining)?

Progress is straight forward:
In the actual Ajax request you could set:
indexValue: i,
based on the loop.
From there in .done for each request:
this.indexValue would return the right index.
You can output what ever you need based on that.
MY ISSUE:
The issue I have is that I can not seem to get .when.then method to work after this process. IT seems to not hold all the items in the ajaxQueue


Include elm in package.json so it gets a symlink

I think you also need to add the "elm" binary in the "bin" section of package.json to make npm create a symlink to it in the user's bin directory.

Thanks, yes :).


Path not found error installing EA App

Trying to install the EA App and I'm getting a "Path not found" error message pop-up.

@sysmoon14 yeah usually that fixes a lot at the expense of deleting games... maybe try using the Uninstall button that may fix a particular one... I know currently in the script there is a weird bug that happens if you were to select like 6 launchers in a row to download the script exits after Amazon or EA App for some reason I still have to fix that.

This is a workaround rather than a fix, I'm having this issue, "starting fresh" is not a solution. I'm currently going through this and unable to use the EA app, it worked fine for a while after starting fresh, but then I got the EA updater message at which point it no longer works, so I uninstalled EA app through Nonsteamlaunchers and tried reinstalling, but getting the path not found error.


GM: Fix noisy brake pressed

PR moved from https://github.com/commaai/openpilot/pull/23617
Looking at all available data for the last 6 months, looks like this is a good change: https://imgur.com/a/DIeGHXs
Platform: CADILLAC ESCALADE ESV 2016
Checked 10 unique dongles, 119 unique routes, and 513 segments
Mismatches: 58573 frames, 493 segments (any frame)
---
Platform: GMC ACADIA DENALI 2018
Checked 1 unique dongles, 55 unique routes, and 483 segments
Mismatches: 16448 frames, 445 segments (any frame)
---
Platform: CHEVROLET VOLT PREMIER 2017
Checked 66 unique dongles, 2844 unique routes, and 10802 segments
Mismatches: 248608 frames, 7649 segments (any frame)

@sshane can you also include how many active (last 3 months or so) dongle IDs we know of for each of these cars?
@JasonJShuler have you shipped this for the Silverado and other ports? Are you confident it'll work there too?

@adeebshihadeh from Verylukyguy:

His PR was created due to fix the unwanted disengagements on the Silverado and Suburban; the Tahoe is the next port that he will be working on and it has the same architecture as the Suburban
The changes have been tested and working on the Silverado and Suburban.

@sshane can you also include how many active (last 3 months or so) dongle IDs we know of for each of these cars?
@JasonJShuler have you shipped this for the Silverado and other ports? Are you confident it'll work there too?

Bolt, Acadia and Suburban have been using successfully

@sshane can you also include how many active (last 3 months or so) dongle IDs we know of for each of these cars?

These are the dongle IDs for the vehicles that JShuler is currently working with and using his fork:
dea4d5b8d82f83fe
e38a02cae1ef1e0c
38aa7da107d5d252
87347aa6099ae6a6
004eb40f69c5f94e


Upgrade solidity version and add new features

Upgraded solc 0.4.* to 0.8.*
Changed Proxy contract to @openzeppelin/contracts:0.4.5
Added new feature and edit interfaces

LGTM


Feature Request: (Re-)Implementation of  different color settings for light- and darkmode

Feature Requested
Hello,
I'd like to ask for the (re-) implementation of different color hues for light and dark mode. I say reimplementation because I am pretty sure it was a feature before the big update, but I am not entirely sure.
The reason is that I usually work in dark mode, and have a color tone that suits me pretty well - I can stare at it for a long time.
However, every now and then, and especially when working with others, I require the light mode because dark mode just becomes too obstructive at that point. I also prefer light mode for my workshifts every now and then.
The issue is that my color setting chosen for dark mode results in a too aggressive light-mode. Dropping the Brightness-setting all the way down to -10 for light mode resolves this to some extend, but to be really comfortable for longer periods I need to adjust the color hue to a warmer color. Additionally, dropping the brightness improves the background, but suffers the text clarity itself a bit as well.
For reference, I use color 255 in dark-mode, and regularly need to shift to orange-ish around 55,60 on light mode.
Thank you.
Sincerely,
~Gw
Relevant Screenshot
/ I'm on mobile right now and for a while, not applicable.
Checklist

[X] I have checked that the feature is not already available via the Style Settings plugin.
[X] The feature would be useful to more users than just me. (If it's only relevant to you, have you considered using a CSS Snippet?)
[X] My issue concerns Obsidian version 0.16+ and I am not using the Legacy Editor. (Older Obsidian versions and the Lecacy Editor are not supported anymore.)

good suggestion. I just added separate hue sliders, one for light and one for dark mode.

Thank you. That works wonderfully.


feature(zjow): impala policy for continuous action space

Add impala policy for continuous action space.
Support action reparameterized by dimension-wise independent gaussian disribution.

bipedalwalker-v3 by impala policy


Create a dev container to make development easier

Is your feature request related to a problem? Please describe.
Usually new developers, they have a hard time for contributing especially the first time because they need to set up their own development environment. The current process might look simple, but in reality it's much more complex than that, one of the most often problems that I encountered when doing this kind of setup is OS difference, people using Windows will have a different steps of setting it up with those who are using Linux based distro or even Mac.
Describe the solution you'd like
Dev container has been around for sometime to solve this problem.
Describe alternatives you've considered
Another way is using vagrant to spin up a VM inside the contributor's machine, but this is not preferable as it is consuming too much resources compared to dev container.
Additional context
No

@refeed Kindly assign this to me


Windows build batch files enhancements: better setlocal usage, use 'call' in locatevc.bat to run buildarch.ring

The setlocal change avoids that environment variables are polluted which was causing failure when building a lot of extensions in the same CMD session.
The locatevc.bat change fixes an issue when using "auto" as first parameter because running "RINGEXEPATH" at line 32 was causing an error in some cases: using call avoids this problem and it is more reliable.

Thanks for the update
Keep up the GREAT WORK


chore: bump containerd-wasm-shims to v0.9.0

Thanks for updating the shims, but 0.9.0 is more than just a version bump, with the shims being able to work with sidecars we can now safely drop crun+wasmedge as the default runtime for WasmEdge in favor of the runwasi WasmEdge shim. This simplifies the installer a lot.
I'd close this issue in favor of #42. Sorry for any inconvenience.


When playing HLS,  cannot read properties of null (reading 'startTime')

Have you read the FAQ and checked for duplicate open issues?
Yes
If the problem is related to FairPlay, have you read the tutorial?

Not related to FairPlay.
What version of Shaka Player are you using?

4.9.3
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
My own custom app
If custom app, can you reproduce the issue using our demo app?
Haven't been able to run the demo app yet.
What browser and OS are you using?
MacOS Sonoma 14.4.1 Google Chrome
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?

Not able to share
What configuration are you using?  What is the output of player.getConfiguration()?

default
What did you do?

Call player.load(url, 0) with url to be like https://www.xyz.com/test.m3u8
In a super bad network condition with high latency and slow speed
throw error {"stack":"TypeError: Cannot read properties of null (reading 'startTime') (../../../../../../node_modules/shaka-player/lib/hls/hls_parser.js:495:42)

What did you expect to happen?
Properly handle audio streams in m3u8 format - hls
What actually happened?

related log
2024-10-11_12:00:10.93834-0700 [2:2:1011/120010.938274:INFO:CONSOLE(1)] "[XXX] [sendPlaybackError] PlayerErrorUnspecifiedError {"stack":"TypeError: Cannot read properties of null (reading 'startTime')    at S1.h (***/_app/immutable/chunks/shaka-player.compiled-728d6550.js:33:5121)    at fo (***/_app/immutable/chunks/shaka-player.compiled-728d6550.js:6:3739)    at A1.next (***/_app/immutable/chunks/shaka-player.compiled-728d6550.js:6:4037)    at t (***/_app/immutable/chunks/shaka-player.compiled-728d6550.js:6:4266)","message":"Cannot read properties of null (reading 'startTime')","severity":1}", source: ***/_app/immutable/chunks/log-8cc35343.js (1)

related code
# lib/hls/hls_parser.js
async updateStream_(streamInfo) {
...
    const {segments, bandwidth} = this.createSegments_(        playlist, stream, mediaSequenceToStartTime, mediaVariables,        streamInfo.getUris, streamInfo.type);
...
    const oldSegment = segments[0];
    goog.asserts.assert(oldSegment, 'Should have segments!');
    streamInfo.minTimestamp = oldSegment.startTime; // startTime undefined
...
}

createSegments_(playlist, stream, mediaSequenceToStartTime, variables,      getUris, type) {....}

Are you planning send a PR to fix it?
Not yet, seek for help to see if there is a good way to fix it

This has been fixed in version 4.10.0. Upgrade to a newer version please!


[zh] Localize 'bare_metal_machine'

[ ] Localize the content/en/bare_metal_machine.md to content/zh-cn/ in dev-zh.

Repeated


Document @<json_file> --settings parameter.

Fixes #8723.

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

[ ] 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.

Thanks for the whitespace fix. I edited it on Github web and missed it.

No worries @VictorioBerra :)  I just didn't want to ask you to make even more nit-picky changes. This should make the CI green and we'll get it in. Thank you very much for the contribution!


Updated requirements for 'python-neutronclient' to prevent crash

The latest 'python-neutronclient' (>=10.0.0) removed features used by VMTP, specifically in commit openstack/python-novaclient@01fb1653. This commit updates the requirements to python-neutronclient>=3.0.0,<10.0.0.
For example, instance.py#L200 calls self.instance.add_floating_ip which was removed from python-novaclient in openstack/python-novaclient@01fb1653.
This causes:
2018-04-16 20:23:00,475 INFO [TestServer1] Associating floating IP <IP_ADDRESS>
2018-04-16 20:23:00,476 ERROR Traceback (most recent call last):
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/vmtp/vmtp.py", line 475, in run
    self.setup()
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/vmtp/vmtp.py", line 318, in setup
    self.net.vm_int_net[0])
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/vmtp/vmtp.py", line 169, in create_instance
    init_file_name=user_data_file))
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/vmtp/perf_instance.py", line 47, in create
    init_file_name)
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/vmtp/instance.py", line 200, in create
    self.instance.add_floating_ip(self.ssh_access.host, ipv4_fixed_address)
  File "/home/user/vmtpenv/local/lib/python2.7/site-packages/novaclient/base.py", line 180, in __getattr__
    raise AttributeError(k)
AttributeError: add_floating_ip

VMTP should probably be updated to deal with this properly, but for now the requirements file should be updated to use a compatible version of python-novaclient.

Thank you for contributing to openstack/vmtp!
openstack/vmtp uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Fix rcfile-new and bash syntax (#51)

Fix broken syntax of rcfile-new
Restrict bash syntax to bash-only to avoid colliding with sh

Sweet, looks good!  Brain-simulation of regex looks good & non-malicious! I'm too lazy to implement proper CI testing, so that will have to do.
Thank you for your hard work!


Deflake pgsql_trace_bpf_test

//src/stirling/source_connectors/socket_tracer:pgsql_trace_bpf_test has a <90% pass rate in CI.
De-flake the test, run locally with runs_per_test set to a large number to ensure that the pass rate is higher than 90% and then re-enable the test in CI by removing the disabled_flaky_test tag.

The source of the flakiness is due to how the pgsql stitcher works. It previously assumed that when it sees a request that the matching response exists during the same invocation to StitchFrames. For our test harness, this is true the majority of the time. However, there are times where stirling sees the request in isolation, calls StitchFrames and then receives the response by the second call to StitchFrames. The previous stitcher implementation discards all requests at the end of a run, so when this happens we lose the request and can never match the lone response on the next StitchFrames invocation. This results in all of the test cases occasionally seeing the following error:
	src/stirling/source_connectors/socket_tracer/pgsql_trace_bpf_test.cc:219: Failure
Expected equality of these values:
  tablets.size()
    Which is: 0
  1

While this test was flaky, this detected a real bug in our stitcher and further demonstrates why having a true end to end test is valuable even if it's not as reproducible.
I have a phab open for this and verified that the test passes 20/20 times, so we should be able to re-enable this test shorly


Add an HTML comment to the DOM

There should be a way to append an HTML comment  to the DOM.  It should be documented.

FWIW, you can do that by passing a function to selection.append, selection.insert or selection.select:
d3.select("body").append(function() { return document.createComment("This is a comment."); });

That works.  Thanks!


python3 training_S3DIS.py - data processing

Hi, thanks for your work. I have encountered some problems and may need your help, thank you. When running python3 training_S3DIS.py and processing the S3DIS data set to the third room of Area_5, it appears
File "training_S3DIS.py", line 202, in
dataset = S3DISDataset(config.input_threads)
File "/home/disk1/dj/downloads/KPConv-master/datasets/S3DIS.py", line 152, in init
self.prepare_S3DIS_ply()
File "/home/disk1/dj/downloads/KPConv-master/datasets/S3DIS.py", line 203, in prepare_S3DIS_ply
object_data = np.array([[float(x) for x in line.split()] for line in f])
File "/home/disk1/dj/downloads/KPConv-master/datasets/S3DIS.py", line 203, in
object_data = np.array([[float(x) for x in line.split()] for line in f])
File "/home/disk1/dj/downloads/KPConv-master/datasets/S3DIS.py", line 203, in
object_data = np.array([[float(x) for x in line.split()] for line in f])
ValueError: could not convert string to float: '185\x00187'

Hi @juzi-dj,
I remember having seen this error before. I think it was because one of the file of S3DIS dataset was corrupted. To find it, I printed the name of the object file just before line 203 in S3DIS.py, and then the line index where this error happens. And I just corrected the file myself

Thank you for your reply, I will check again.

感谢您的答复，我将再次检查。
Hello, I have the same problem as you. Have you solved it?

This error may be caused by copy or download.     the error is that that line has < >  you can print all line number
for example :
import numpy as np
object_file = "S3DIS/Area_5/office_19/Annotations/ceiling_1.txt"
print(object_file)
with open(object_file, 'r') as f:
for idx, line in enumerate(f):
print(idx)
for x in line.split():
data  = np.array([float(x)])


Phalcon can only be compiled with zephir build on Ubuntu 18.04

From @dschissler on May 11, 2018 21:17
Installation hangs on Ubuntu 18.04 when compiling from tarball using the ./install method.  However it does compile with zephir builid using latest zephir.  I tested that Phalcon\Version::get() works.
Ubuntu 18.04 uses PHP 7.2 and GCC 7.3.0.
Copied from original issue: phalcon/cphalcon#13376

From @dschissler on May 17, 2018 1:18
Those are the same steps that I do for all install methods after it has successfully built.  My problem was that the build process hung for a very long time and then failed.

From @glaucomonteiro on May 17, 2018 1:3
I was able to install it using the ./install method, but had to do this:
Create a file in /etc/php/7.2/mods-available called phalcon.ini with the content as:
extension=phalcon.so
Then run the command:
ln -s /etc/php/7.2/mods-available/phalcon.ini /etc/php/7.2/cli/conf.d/50-phalcon.ini
Doing this it will create a symbolic link to the file you just created, to be loaded with priority 50, or in other words, it will be the last extension loaded so you should not have any conflicts when starting the php service.

Implemented.
root@c94f22d87b2c:/# php -v
PHP 7.2.8-1+ubuntu18.04.1+deb.sury.org+1 (cli) (built: Jul 25 2018 10:52:19) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.2.8-1+ubuntu18.04.1+deb.sury.org+1, Copyright (c) 1999-2018, by Zend Technologies

root@c94f22d87b2c:/# lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.1 LTS
Release:	18.04
Codename:	bionic

root@c94f22d87b2c:/# curl -s https://packagecloud.io/install/repositories/phalcon/stable/script.deb.sh | sudo bash
Detected operating system as Ubuntu/bionic.
Checking for curl...
Detected curl...
Checking for gpg...
Detected gpg...
Running apt-get update... done.
Installing apt-transport-https... done.
Installing /etc/apt/sources.list.d/phalcon_stable.list...done.
Importing packagecloud gpg key... done.
Running apt-get update... done.

The repository is setup! You can now install packages.

root@c94f22d87b2c:/# apt-cache policy php7.2-phalcon
php7.2-phalcon:
  Installed: (none)
  Candidate: 3.4.0-5+php7.2
  Version table:
     3.4.0-5+php7.2 500
        500 https://packagecloud.io/phalcon/stable/ubuntu bionic/main amd64 Packages

root@c94f22d87b2c:/# apt install php7.2-phalcon
Reading package lists... Done
Building dependency tree
Reading state information... Done
Suggested packages:
  php7.2-mysql php7.2-sqlite3 php7.2-pgsql php7.2-memcached
The following NEW packages will be installed:
  php7.2-phalcon
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 1,233 kB of archives.
After this operation, 5,035 kB of additional disk space will be used.
Get:1 https://packagecloud.io/phalcon/stable/ubuntu bionic/main amd64 php7.2-phalcon amd64 3.4.0-5+php7.2 [1,233 kB]
Fetched 1,233 kB in 4s (285 kB/s)
debconf: delaying package configuration, since apt-utils is not installed
Selecting previously unselected package php7.2-phalcon.
(Reading database ... 24205 files and directories currently installed.)
Preparing to unpack .../php7.2-phalcon_3.4.0-5+php7.2_amd64.deb ...
Unpacking php7.2-phalcon (3.4.0-5+php7.2) ...
Setting up php7.2-phalcon (3.4.0-5+php7.2) ...


Added duration proprety to messages

Pull Request Checklist

[ ] Pull request is based on the develop branch
[ ] Pull request updates CHANGES.rst
[ ] Pull request includes a sign off

@artemkrachulov did I miss it, or there's no change to the actual message content (body)?
Keep in mind that we cannot send floating point values in Matrix (according to Canonical JSON, room version 7).
Probably we'll have to send the duration in milliseconds as UInt64.


Order code terms to match order of equation terms

Reverse the order of the 2nd and 3rd terms in the code corresponding to the finite difference estimate of the 2nd derivative of a function to match the order of the terms in the equation.

Link checker failure is unrelated to this PR (maintenance on Pythia binder).


Add Homebrew.jl/ImageMagick's bin directory to PATH

This allows issues like https://github.com/timholy/Images.jl/issues/240 to work with Homebrew.jl.
Contents of bin directory on OSX 10.10 with ImageMagick 6.8.9:
Magick++-config
Magick-config
MagickCore-config
MagickWand-config
Wand-config
animate
compare
composite
conjure
convert
display
identify
import
mogrify
montage
stream

Can you try pushing to this branch again? Travis is complaining about an unknown commit. I'm not sure what's up with that.

Rebased onto master.

Coverage remained the same when pulling b31d5ecc489f1cde2394c6c4cd09f732192e6043 on staticfloat:sf/binpath into 82fb13daf16244e75f77944dce77a21762702560 on timholy:master.

Thanks!


[feature] Optimize computeErasParse and computeMonthsParse

summary
Optimize computeErasParse and computeMonthsParse , reduce unnecessary regular escaping  and loop traversal.

Coverage increased (+0.03%) to 88.524% when pulling 3afc8d5cef3834dce122e80e935344939fd09de0 on Alanscut:optimizeCode into e3c67905f9d714d43a28dc627ee8efdab06dbce3 on moment:develop.

Merged in a3babc50afba073cb377ef7ce4b77803e928078c

I tweaked a bit the eras code, was double-escaping the regex for the mixed case.


Add cancel() method to interrupt a stream

Fixes #599.
Thanks for all your work on this project!

please accept this pr @abetlen

Actually.. I found an issue with this method.. this will only cancel after a token is generated but if the llm is slow or gets stuck processing the prompt, this doesn't cancel it..
We need a better method.

I'm coming back to this because I need to figure out a better method to interrupt the generation programmatically..
For a console-based scenario it's pretty easy in python, all I have to do is surround the code with try except KeyboardInterrupt: .. then I can just press ctrl+c at any point to gracefully interrupt the llm..
But.. if I'm using a front-end user interface, I haven't managed to make it work properly let's say with a button "Stop generating" that can call a python function.. because of the issue I mentioned in the previous post..
@abetlen sorry to bother again but do you have any suggestions/ideas on how to accomplish this?

Why not add it now and improve if there is a better solution. For now this would work in most cases.

has anyone found a reasonable solution for this? Or am I the only one not willing to wait until the model finishes without killing the job and losing context?

Any chance this gets merged for now?

It indeed blocks until the first token is produced, but cancelling it after that is trivial. The other similar issue is cancelling a model that is loading.

gpt4all python bindings offer a similar way which allows stopping with the next token

+1 can we merge this?

Take a look at https://github.com/ggerganov/llama.cpp/issues/10509 which should permanently solve this problem on lcpp's side


wasi posix semaphore issue

does it support wasi semaphore api, or how to fix, Thanks.
##running
./iwasm --map-dir="/::./" --max-threads=20 --heap-size=65535 ./test-apps.wasm
[21:31:09:159 - 7F13E76E5B80]: failed to check signature '($iii)i' and resolve pointer params for import function (env sem_open)
[21:31:09:143 - 7F13E76E5B80]: warning: failed to link import function (env, sem_open)
[21:31:09:224 - 7F13E76E5B80]: warning: failed to link import function (env, sem_open)
[21:31:09:398 - 7F13E76E5B80]: warning: failed to link import function (env, sem_open)
[21:31:09:652 - 7F13E76E5B80]: warning: failed to link import function (env, sem_open)
Exception: failed to call unlinked import function (env, sem_open)

source
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <semaphore.h>
#include <pthread.h>

#define MAX_NUM_THREADS 3

sem_t *sem;

static void* routine(void* arg){
    sem_wait(sem);
    printf("Hello from thread %ld\n", (long)arg);
    sem_post(sem);
    return NULL;
}

int main(int argc, char *argv[])
{
    pthread_t threads[MAX_NUM_THREADS];
    long i;

    sem = sem_open("/semaphore_example", O_CREAT, 0644, 1);
    if(sem == SEM_FAILED){
        perror("sem_open");
        return TEST_FAILED;
    }

    for(i = 0; i < MAX_NUM_THREADS; i++){
        pthread_create(&threads[i], NULL, routine, (void*)i);
    }

    for(i = 0; i < MAX_NUM_THREADS; i++){
        pthread_join(threads[i], NULL);
    }

    sem_close(sem);
    sem_unlink("/semaphore_example");

    return 0;
}

Hi, it seems that wasi-sdk doesn't support posix semaphore APIs so the compiled wasm app imports sem_xxx APIs, there may be two ways:
one is to use WAMR lib-pthread instead, please refer to:
https://github.com/bytecodealliance/wasm-micro-runtime/tree/main/samples/multi-thread (note that sem_open/sem_wait/sem_close are used in wasm-apps/main.c)
https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/doc/pthread_library.md
the other is to keep using WAMR wasi-threads, but to implement the native apis for sem_xxx, maybe you can refer to:
https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/core/iwasm/libraries/lib-pthread/lib_pthread_wrapper.c#L1134-L1176

Hello @wenyongh, Thanks for your comments
due to I will use wasi other apis like socket and file/directory,  so the last approach is reasonable or implement on wasm app side like:
#include <pthread.h>
#include <stdlib.h>

typedef struct {
    pthread_mutex_t mutex;
    pthread_cond_t cond;
    int count;
} sem_t;

int sem_init(sem_t *sem, int pshared, unsigned int value){
    if(pshared > 1) {
        /* this implementation does not support
		   process-shared semaphores */
        return -1;
    }
    sem->count = value;
    pthread_mutex_init(&sem->mutex, NULL);
    pthread_cond_init(&sem->cond, NULL);
    return 0;
}

int sem_destroy(sem_t *sem){
    pthread_mutex_destroy(&sem->mutex);
    pthread_cond_destroy(&sem->cond);
    return 0;
}

int sem_wait(sem_t *sem){
    pthread_mutex_lock(&sem->mutex);
    while(sem->count == 0){
        pthread_cond_wait(&sem->cond, &sem->mutex);
    }
    sem->count--;
    pthread_mutex_unlock(&sem->mutex);
    return 0;
}

int sem_post(sem_t *sem){
    pthread_mutex_lock(&sem->mutex);
    sem->count++;
    pthread_cond_signal(&sem->cond);
    pthread_mutex_unlock(&sem->mutex);
    return 0;
}

then waiting new wasi-sdk that supporting semaphore.
do you know the roadmap of wasi-sdk about IPC api, even including mqueue, rwlock ...

Hi, I am afraid I have no idea about the roadmap of wasi-sdk, maybe you can open issue in wasi-sdk's community.

is there any approach to use buildin sem_open that you list:
https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/core/iwasm/libraries/lib-pthread/lib_pthread_wrapper.c#L1134-L1176
and other api can also use wasi lib's  like pthread/file/socket?
need a prefix like _priv_ or not?
then I use sem_open after #define _priv_sem_open sem_open in wasm app
#define REG_NATIVE_FUNC(func_name, signature) \
    { _priv_#func_name, func_name##_wrapper, signature, NULL }
/* clang-format on */

static NativeSymbol native_symbols_lib_pthread_sem[] = {
#if WASM_ENABLE_LIB_PTHREAD_SEMAPHORE != 0
    REG_NATIVE_FUNC(sem_open, "($iii)i"),
    REG_NATIVE_FUNC(sem_close, "(i)i"),
    REG_NATIVE_FUNC(sem_wait, "(i)i"),
    REG_NATIVE_FUNC(sem_trywait, "(i)i"),
    REG_NATIVE_FUNC(sem_post, "(i)i"),
    REG_NATIVE_FUNC(sem_getvalue, "(i*)i"),
    REG_NATIVE_FUNC(sem_unlink, "($)i"),
#endif
};

If to use the builtin sem_xxx apis, you should enable the lib-pthread library, but it seems that you also enable the lib-wasi-threads? I think you had better just pick one, if to use the latter, you may implement the native api for sem_xxx apis by yourself:
https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/doc/export_native_api.md


Guide the newbee user to the included help

Before this, a plain lymph would just show
Usage: lymph [options] <command> [<args>...]

which leaves no clue about how to use lymph help

@Drahflow can you squash your commits pleases ? a commit that introduce a broken state is a useless  commit IMHO :)

LGTM


Add files via upload

Please add my school

@saiefzaier Pull request merged. Thank you!
For the record: proof of the domain ownership: https://www.4c.tn/Evenement/DetailEvenement?id=261

@saiefzaier Pull request merged. Thank you!
For the record: proof of the domain ownership: https://www.4c.tn/Evenement/DetailEvenement?id=261


[WIP] update rl_pong app

Two things before merging this.

When running this app, the worker logs stuff to the screen. Why is it doing that?
This prints too much. We should probably print only once per pass through the while loop.

Ok, I think the answer to my question "Why is the worker logging so much to the screen as opposed to the log file?" is that Gym is also using the Python logging module, and these interact.
For example,
import ray
import gym
ray.services.start_ray_local(num_workers=1)

@ray.remote([], [])
def f():
  pass

@ray.remote([], [])
def use_gym():
  import gym

If you call f() several times, it prints nothing.
If you call use_gym() once, it prints
[2016-07-28 21:20:11,235] Finished executing function use_gym, it took 0.0978379249573 seconds

Then if you call f() again it prints
[2016-07-28 21:20:58,972] Calling function f
[2016-07-28 21:20:58,972] Finished executing function f, it took 9.53674316406e-07 seconds

The logging problem is fixed in #321.


Fixing the routing issue that prevents servers from provisioning

The default route configuration on the server causes two issues:

With default route in post-up will cause apt to fail installing during ansible deployment
Without default route in post-up will cause server to fail ping after provisioning

I added a more specific post-up route to work around this problem.

Watch the repository changes... in helper_scripts/oob-mgmt-server-netq-setup.sh should probably point to cumulus networks repo.


Jax Code

Hi,
Thank you very much for your contribution.
I would like to ask is it possible to release the code based on Jax
Best

Hi! This code provided as is for now and I don't have plans for future maintenance, so rewriting it on jax is not my priority at the moment. My research interests are also shifting, so TT is not that interesting to me anymore.
If I would write it on jax tho, this repo should be a reasonable base: https://github.com/jenkspt/gpt-jax


Cannot use dict base datasets with keras.Model.fit.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0
Python version: 3.6.4
CUDA/cuDNN version: 10.1
GPU model and memory: 7.6.2

In order to make both dataset and keras model have good structures, I create a dataset and a vanilla model like this.
# dataset is something like <BatchDataset shapes: ({input: (None, 100)}, {output: (None, 10)}), types: ({input: tf.float32}, {output: tf.float32})>

# subscale model is something like
class VanillaModel(Model):

  def __init__(self, num_units, **kwargs):
    super(VanillaModel, self).__init__(**kwargs)
    self.num_units = num_units

    # One linear projection layer.
    self.dense_proj = tf.keras.layers.Dense(num_units, activation='relu')

  def call(self, features):
    """Forward pass."""
    output = self.dense_proj(features['input'])
    return {
        'output': output
    }

When I use the dict based dataset (from tfds) with keras.Model.fit, the first call will cause expection as
# Compile model using dict with same keys.
model.compile('adam', {'output': 'mse'})

# The errors
 File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/training_utils.py", line 1248, in cast_if_floating_dtype_and_mismatch
    if target.dtype != out.dtype:
AttributeError: 'str' object has no attribute 'dtype'

I checked the code and found that, when dict is passed, iterating through zip(targets, outputs) will just get the keys of the dict, so the string keys have no dtyle.
(https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/keras/engine/training_utils.py#L1246)

So how can I use dict based dataset and model with keras.Model.fit?

@npuichigo, Will it be possible to provide the standalone code to replicate the reported issue. Thanks!

@npuichigo, Any update on code. Thanks!

The problem is that, if our model and dataset return dictionaries, how can we compile and fit the model.
import tensorflow as tf
import numpy as np

class VanillaModel(tf.keras.Model):

  def __init__(self, num_units, **kwargs):
    super(VanillaModel, self).__init__(**kwargs)
    self.num_units = num_units

    # One linear projection layer.
    self.dense_proj1 = tf.keras.layers.Dense(num_units, activation='relu')
    self.dense_proj2 = tf.keras.layers.Dense(num_units, activation='relu')

  def call(self, features):
    """Forward pass."""
    proj1_output = self.dense_proj1(features['input'])
    proj2_output = self.dense_proj2(features['input'])
    return {
        'proj1_output': proj1_output,
        'proj2_output': proj2_output
    }

input_tensor = np.random.normal(size=(50, 32)).astype(np.float32)
output_tensor1 = np.random.normal(size=(50, 16)).astype(np.float32)
output_tensor2 = np.random.normal(size=(50, 16)).astype(np.float32)

dataset = tf.data.Dataset.from_tensor_slices(({'input': input_tensor}, {'proj1_output': output_tensor1, 'proj2_output': output_tensor2}))
model = VanillaModel(16)

model.compile('adam', {'proj1_output': 'mse', 'proj2_output': 'mae'})
model.fit(dataset)

any updates?

I am facing the same issue when I am retrieving data from TFRecords.

Assigning to Tom who will be working on this as part of ideal fit/compile change.

For a quick workaround, note the Keras expects Datasets passed to fit to be of the form (features, labels or (features, labels, sample_weights)
Here's an example of grabbing MNIST from tensorflow_datasets and formatting it in the way Keras expects:
import tensorflow as tf
import tensorflow_datasets as tfds

def convert_dataset(item):
    """Puts the mnist dataset in the format Keras expects, (features, labels)."""
    image = item['image']
    label = item['label']
    image = tf.dtypes.cast(image, 'float32') / 255.
    return image, label

mnist_data = tfds.load('mnist')
mnist_train, mnist_test = mnist_data['train'], mnist_data['test']
mnist_train = mnist_train.map(convert_dataset).shuffle(1000).batch(100).repeat()
mnist_test = mnist_test.map(convert_dataset).batch(100)

model.fit(mnist_train, epochs=10, validation_data=mnist_test)

@omalleyt12 That's true. But sometimes we have complicated models with multiple inputs and outputs, so it's more elegant if we can use dict based model and dataset.

@npuichigo For those Models, each of features and labels can be a dictionary:
ds = tf.data.Dataset.from_tensor_slices(
  ({'my_feature_1': ..., 'my_feature_2': ...}, {'my_label_1': ..., 'my_label_2': ...}))

@omalleyt12 So it looks like my example.
dataset = tf.data.Dataset.from_tensor_slices(({'input': input_tensor}, {'proj1_output': output_tensor1, 'proj2_output': output_tensor2}))

But it seems that keras restrict the input and output names to something like input_1, input_2, output_1 and so on. Can you give a correct example?

If I use  tf.feature_column.categorical_column_with_vocabulary_list I wasable to specify the column name in keras. But I faced the issue when I called model_to_estimator API to convert back to tensorflow model as the converted model was expecting inputs like input1,input2,... as you mentioned

same problem

I faced same issue. My workaround is not using the subclass way.  When I change to tf.keras.Model(inputs=...,outputs=...), the problem solved.
In fact I found there's quite some limitation on subclass model, and there's no good guidline about how to avoid these limitations.Hope tensorflow can improve the usability of subclass model. I really prefer subclass model than sequence model becuase that's the way how we can easily debug the model with eager.

Thanks for the issue all! This is now fixed in the latest tf-nightlym you can use any arbitrary nested structure of data with subclassed Models.
Code below works (note I had to add a call to dataset.batch, this is needed to create batches of data when using from_tensor_slices):
class VanillaModel(tf.keras.Model):

  def __init__(self, num_units, **kwargs):
    super(VanillaModel, self).__init__(**kwargs)
    self.num_units = num_units

    # One linear projection layer.
    self.dense_proj1 = tf.keras.layers.Dense(num_units, activation='relu')
    self.dense_proj2 = tf.keras.layers.Dense(num_units, activation='relu')

  def call(self, features):
    """Forward pass."""
    proj1_output = self.dense_proj1(features['input'])
    proj2_output = self.dense_proj2(features['input'])
    return {
        'proj1_output': proj1_output,
        'proj2_output': proj2_output
    }

input_tensor = np.random.normal(size=(50, 32)).astype(np.float32)
output_tensor1 = np.random.normal(size=(50, 16)).astype(np.float32)
output_tensor2 = np.random.normal(size=(50, 16)).astype(np.float32)

dataset = tf.data.Dataset.from_tensor_slices(({'input': input_tensor}, {'proj1_output': output_tensor1, 'proj2_output': output_tensor2}))
dataset = dataset.batch(10)  # This needs to be called to create batches of data.
model = VanillaModel(16)

model.compile('adam', {'proj1_output': 'mse', 'proj2_output': 'mae'})
model.fit(dataset)

@omalleyt12
When eager execution is disabled, the code snippet you posted above still fails under version 2.2.0-dev20200501 with the same error (AttributeError: 'str' object has no attribute 'dtype').

@jgmakin Disabling eager execution is a v1 compatibility feature, in general we're only making critical security or regression fixes in the v1 compatibility code for tf.keras. I'd recommend migrating to the TF2 style

Ah, ok, thanks.  I tend to disable eager execution under the assumption that this improves performance, but perhaps that isn't the case anymore?

@jgmakin When using Model.fit, everything runs inside a graph anyway (a tf.function graph) so performance should be good
For more, check out this guide

Hello, is there any way to do this too for validation_data?
I can't get it work for validation after every epoch end


feat: Notify involved project developers when project gets destroyed

I am adding ProjectDeveloperMailer.project_destroyed which is send when project is deleted. Email is send as part of controller action to project_developer/owner and all involved_project_developers of project.
Tricky part was to tweak ProjectInvolvement  model and its after_create/after_destroy callbacks which also sends collaboration emails so users are not getting two emails when project is destroyed (one that their collaboration ended, second that project was destroyed). Hopefully I have managed to catch all use cases --> please see tests ;).
During adding of new email, I have also added localization to already existing custom emails.
Testing instructions
Delete project via Rswag doc or Backoffice --> you should receive information that project was deleted. No other email should be received.
Tracking
https://vizzuality.atlassian.net/browse/LET-752
https://vizzuality.atlassian.net/browse/LET-759

I like how you handled the email localisation, although I'm not entirely sure about all cases where account language should have precedence over ui language. Just wondering what you think about this approach I took to localise devise_invitable emails? I'm actually not even sure how to test this. #407

Yeah, me neither, not sure about which language should be preferred at every use case, but it should be quite straightforward to change it at future if we decide to prefer first or second option.


BUG: Duplicate About Us html file

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There are two about us html files
-about.html
-about1.hmtl
Duplicate pages can create confusion and make it harder for users to navigate

https://github.com/anuragverma108/SwapReads/assets/169146901/f17c63a3-a1c9-448a-9bc3-26837747d080
i would like to work on this issue please assign it to me under GSSOC'24
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108  @huamanraj  @RitiChandak please assign this issue to me under GSSOC'24

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Bug: Word "I" not Capitalized when First Letter Capitalization Used

Describe the Bug
"Headings should be formatted with capitalization" lowercases words that should always be capitalized when "First letter" is selected.
In practice, this may only mean the word "I".
How to Reproduce
Steps to reproduce the behavior:

Write a heading such as ### What did I learn?
Lint
The heading is changed to ### What did i learn?

Expected Behavior
No change, i.e.
### What did I learn?

Device

[ ] Desktop
[x] Mobile

Additional Context
I realize I can add "I" to the list of ignored words, but I shouldn't have to.

I don't think that is necessarily as that seems to be the intended affect of the rule, but it likely can be updated to use a word list for what to capitalize for it.

The rule does not care for words that should be capitalized since it was asked to lowercase everything except the first letter. The only way I can see getting around this either adding a list of words that should either be ignored or a list or words that should be capitalized.

There is already a list of Ignore Words. I suppose that you could just add I to the defaults that are already in there rather than hard-coding it.

Apparently it is. But there's no reason it should be. There's nothing in the settings to say that it is. So I think this list should be used for both rules, and "I" added to it.

The logic here would need updating to work on a word by word basis instead of a header by header basis for other capitalization cases besides the "Title Case": https://github.com/platers/obsidian-linter/blob/master/src/rules/capitalize-headings.ts#L76-L127

Would it be a correct understanding to say that the ignored words would only affect "Title Case" and the capitalization of the first letter? That is to say it would not affect forcing all caps on the header. Or should ignore words affect all of the capitalization types for the heading rule?
Thanks for helping me flush this out before I take a stab at this.

Dev note: I can improve performance here since we can match all headers and then iterate from there. We can use a custom regex replace argument and allow it to work that way.
It would look something like this:
return text.replace(headerRegex, (headerText: string) => {
 let firstWord = true;
 let capitalizeJustFirstLetter = false;
 switch (options.style) {
    case 'First letter':
      capitalizeJustFirstLetter = true;
  case 'Title Case':
    const headerWords = headerText.match(/\S+/g);
    // split by comma or whitespace
    const keepCasing = options.ignoreWords;
    const ignoreShortWords = options.lowercaseWords;
    let firstWord = true;
    for (let j = 1; j < headerWords.length; j++) {
      // based on https://stackoverflow.com/a/62032796 "/\p{L}/u" accounts for all unicode letters across languages
      const isWord = headerWords[j].match(/^[\p{L}'-]+[.?!,:;]?$/u);
      if (!isWord) {
        continue;
      }

      const ignoreCasedWord = options.ignoreCasedWords && headerWords[j] !== headerWords[j].toLowerCase();
      const keepWordCasing = ignoreCasedWord || keepCasing.includes(headerWords[j]);
      if (!keepWordCasing) {
        headerWords[j] = headerWords[j].toLowerCase();
        const ignoreWord = ignoreShortWords.includes(headerWords[j]);
        if ((!ignoreWord !capitalizeJustFirstLetter) &&  || firstWord === true) {
          // ignore words that are not capitalized in titles except if they are the first word
          headerWords[j] = headerWords[j][0].toUpperCase() + headerWords[j].slice(1);
        }
      }

     firstWord = false;
   }

   headerText = escapeDollarSigns(`${headerWords.join(' ')}`),
 case 'ALL CAPS':
   headerText = headerText.toUpperCase(); // convert full heading to uppercase
   break;
  }

  return headerText;
});

Would it be a correct understanding to say that the ignored words would only affect "Title Case" and the capitalization of the first letter? That is to say it would not affect forcing all caps on the header. Or should ignore words affect all of the capitalization types for the heading rule?
Thanks for helping me flush this out before I take a stab at this.

I think I have answered my own question in my head on this. Uppercasing the whole heading should happen irrespective of the words present as ignore words and short words should all be entirely capitalized.
I will see about making this change this evening.

Would it be a correct understanding to say that the ignored words would only affect "Title Case" and the capitalization of the first letter? That is to say it would not affect forcing all caps on the header. Or should ignore words affect all of the capitalization types for the heading rule?

So…I'm talking about "First letter," not "Title Case." "First Letter" should not lowercase words that are always capitalized, such as "I." It should also not lowercase any of the words in the "Ignore Words" list. The fix to my mind is to either make "Ignore Words" apply to "First letter" as well as "Title Case" (if that doesn't cause other conflicts), or to create an "Uppercase Words" list that never get lowercased.

Would it be a correct understanding to say that the ignored words would only affect "Title Case" and the capitalization of the first letter? That is to say it would not affect forcing all caps on the header. Or should ignore words affect all of the capitalization types for the heading rule?

So…I'm talking about "First letter," not "Title Case." "First letter" should not lowercase words that are always capitalized, such as "I." It should also not lowercase any of the words in the "Ignore Words" list. The fix to my mind is to either make "Ignore Words" apply to "First letter" as well as "Title Case" (if that doesn't cause other conflicts), or to create an "Uppercase Words" list that never get lowercased.

The best way to incorporate this would be to just use the same logic under the hood for both of them. So using ignore words makes the most sense.

This should be fixed on master now and it will go out with the next release. Please let us know if there is an issue with the change.


Cannot step into execfile'd file that doesn't have .py extension

If you make a file called foo.cfg and you execfile it, any breakpoint you set inside of the file will not be hit, even if you can observe side effects of the file (for example put a print statement in it).
On the other hand, if you rename the file to foo.cfg.py and then execfile it, your breakpoint will be hit.

This sounds like something we may be able to support cleanly, but it may turn out to be complicated when we don't own the filetype. For example, what if you exec foo.cs and the C# editor tries to take over?
Worth taking a look at in case it is easy, but I worry that edge cases will make it safer for us to only allow debugging Python files.

I might be missing something, as your Python skills are obviously better than mine, but execfile is just a builtin function that compiles and runs python code, right?  Why would it matter if the file has a .cs extension?  This isn't the same as a shell execute scenario, this is you telling the interpreter "this file has python code in it, compile and run it, regardless of what the extension is".  I would imagine if you passed a .cs file with C# code in it, the interpreter would just raise an exception because it expected Python code, right?

It matters within Visual Studio if we enable those files to interact with the debugger. Python obviously doesn't care about the file type, but if it breaks the debugging experience in VS because of conflicts with other parts of the IDE then we do not want to enable it :)

Ahh, I see what you mean.  I would hope the IDE wouldn't be that finnicky.  PDB at the very least supports a source language flag in the debug info so you can tell the debugger what language something is in.  You can have cpp files with a .cs extension and everything works.  Obviously you python files don't have PDBs, but I assume you've hooked into the debug info provider somehow to provide similar information to what a PDB provides, so maybe there's some call that asks what language a file is in that you can respond to?
Worst case scenario, you could just blacklist specific extensions that you aren't going to support (i.e. stuff the IDE is finnicky about), and then let everything else through.
Also, it would be nice if you could log a warning to the console if a file is ignored due to an incompatible editing experience, at least that way people would be scratching their heads less when it doesn't work :)


SRT output not disabled

Which version are you using?
latest
Which operating system are you using?

[ ] Linux amd64 standard
[ *] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
I'm trying to disable SRT output via MEDIA_MTX_SRT_ENABLED=false env variable but stream seems keep working

my env vars
ps output in container show ffmpeg process with srt :

Describe how to replicate the issue

start the server
publish with ...
read with ...

Did you attach the server logs?

yes / no
Did you attach a network dump?

yes / no

SRT cannot be disabled because it is used as the communication protocol from the internal processes INTO MediaMTX


[Question]: How to connect to MySQL database in Agent?

Describe your problem
I am playing around the app in demo.ragflow.io
I want to try the component that can connect to MySQL component. I thought I have put the correct config as below:

Please help

The host with Mysql should have an IP which is accessible for ragflow server.


runtime: GC & debug.FreeOSMemory calls don't return memory to the OS

What version of Go are you using (go version)?

$ go version
go version go1.15.5 linux/amd64
CentOS Linux release 7.7.1908 (Core)
kernel-3.10.0-1062.el7.x86_64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/usr/lib/golang"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/var/www/virtual/service.extractor/service/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build181628151=/tmp/go-build -gno-record-gcc-switches"

What did you do?
A programm allocates some memory and than release it.
https://play.golang.org/p/Ljr0c6zRasB

What did you expect to see?
The programm releases allocated memor and after runtime.GC() & debug.FreeOSMemory() calls  I expect that values of VmSize/VmData drop, but nothing happen.
What did you see instead?
Values of VmSize/VmData remains the same
-- (comment) the programm started
START VmSize:725Mb  VmLck:0  VmPin:0  VmRSS:1  VmData:717  VmStk:0  VmExe:0  VmLib:1
-- (comment) the programm allocated memory for buffers
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:21  VmData:1313  VmStk:0  VmExe:0  VmLib:1
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:22  VmData:1313  VmStk:0  VmExe:0  VmLib:1
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:22  VmData:1313  VmStk:0  VmExe:0  VmLib:1

END
END
....
END
-- (comment) the  programm released memory and shoud return it to the system
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:22  VmData:1313  VmStk:0  VmExe:0  VmLib:1
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:22  VmData:1313  VmStk:0  VmExe:0  VmLib:1
  VmSize:1321Mb  VmLck:0  VmPin:0  VmRSS:22  VmData:1313  VmStk:0  VmExe:0  VmLib:1

I  tried to use "env GODEBUG=madvdontneed=1 ./test2" but it didn't help, it seems that kernel 3.10.0 always use MADV_DONTNEED and GODEBUG=madvdontneed=1 changes nothing
pmap also saw allocated memory

[root@localhost go]# pmap -x  5008
5008:   ./test2
Address           Kbytes     RSS   Dirty Mode  Mapping
0000000000400000     828     672       0 r-x-- test2
00000000004cf000     776     396       0 r---- test2
0000000000591000     100      68      40 rw--- test2
00000000005aa000     204      64      64 rw---   [ anon ]
0000000002089000     132       4       4 rw---   [ anon ]
000000c000000000  655360     480     480 rw---   [ anon ]
00007f3e3a1d8000   20964   18100   18100 rw---   [ anon ]
00007f3e3b651000       4       0       0 -----   [ anon ]
00007f3e3b652000    8192       8       8 rw---   [ anon ]
00007f3e3be52000       4       0       0 -----   [ anon ]
00007f3e3be53000    8192       8       8 rw---   [ anon ]
00007f3e3c653000       4       0       0 -----   [ anon ]
00007f3e3c654000    8192       8       8 rw---   [ anon ]
00007f3e3ce54000       4       0       0 -----   [ anon ]
00007f3e3ce55000   44100    2148    2148 rw---   [ anon ]
00007f3e3f966000  263680       0       0 -----   [ anon ]
00007f3e4fae6000       4       4       4 rw---   [ anon ]
00007f3e4fae7000  293564       0       0 -----   [ anon ]
00007f3e61996000       4       4       4 rw---   [ anon ]
00007f3e61997000   36692       0       0 -----   [ anon ]
00007f3e63d6c000       4       4       4 rw---   [ anon ]
00007f3e63d6d000    4068       0       0 -----   [ anon ]
00007f3e64166000    1804     264       0 r-x-- libc-2.17.so
00007f3e64329000    2048       0       0 ----- libc-2.17.so
00007f3e64529000      16      16      16 r---- libc-2.17.so
00007f3e6452d000       8       8       8 rw--- libc-2.17.so
00007f3e6452f000      20      12      12 rw---   [ anon ]
00007f3e64534000      92      56       0 r-x-- libpthread-2.17.so
00007f3e6454b000    2044       0       0 ----- libpthread-2.17.so
00007f3e6474a000       4       4       4 r---- libpthread-2.17.so
00007f3e6474b000       4       4       4 rw--- libpthread-2.17.so
00007f3e6474c000      16       4       4 rw---   [ anon ]
00007f3e64750000     136     108       0 r-x-- ld-2.17.so
00007f3e6477f000     512     416     416 rw---   [ anon ]
00007f3e647ff000     512       0       0 -----   [ anon ]
00007f3e6487f000       4       4       4 rw---   [ anon ]
00007f3e64880000     508       0       0 -----   [ anon ]
00007f3e648ff000     396      44      44 rw---   [ anon ]
00007f3e64970000       4       4       4 rw---   [ anon ]
00007f3e64971000       4       4       4 r---- ld-2.17.so
00007f3e64972000       4       4       4 rw--- ld-2.17.so
00007f3e64973000       4       4       4 rw---   [ anon ]
00007ffd8d92f000     132      20      20 rw---   [ stack ]
00007ffd8d9c6000       8       4       0 r-x--   [ anon ]
ffffffffff600000       4       0       0 r-x--   [ anon ]
---------------- ------- ------- -------
total kB         1353356   22948   21420

cc @aclements @mknyszek

@AlexBlack772 VmSize is virtual memory footprint and for a Go application it always grows monotonically; the Go runtime never unmaps heap memory. It appears that VmData is just VmSize with a few things subtracted out (https://ewx.livejournal.com/579283.html), so I expect it to have ~the same effect. In generally, address space is cheap on modern systems and does not reflect actual physical memory resources used. When we return memory to the OS, we mark that memory as "unused" to the OS and it's up to the OS to take that memory back. The result is that VmRSS goes down.
VmRSS is the actual physical memory footprint, and Linux won't actually take that memory back until your system is under memory pressure in Go 1.15, because we use MADV_FREE by default (as you point out, though, Linux 3.10.0 doesn't have MADV_FREE, so we fall back on MADV_DONTNEED anyway). In Go 1.16 and above, GODEBUG=madvdontneed=1 is the default.
Tentatively closing this issue as WAI. Please reopen if you feel this is in error.


Remove Offset in Static crop type

Is it possible to remove the Offset in the Static crop type?

Other croppers have this offset too. I will add option to set zoom to match drawing area to be able to crop with no offset on any sides if needed

Other croppers have this offset too but i can add a flag to remove offset. Also i think i should adjust zoom to be less than 1f match drawing area to be able to crop with no offset on any sides if needed.

in most cases, I think of, I don't think that scale should be less than 1f, regarding the offset I was trying to implement Instagram-like cropping this is why it would really be nice to have it without the crop 👍

Overlay ratio option in settings menu can scale overlay between 50 and 100%.


Style :term: references to a glossary

Done with https://github.com/TYPO3-Documentation/t3SphinxThemeRtd/commit/ff64289c2734fe526b49fe4f7c0b63d287dd581f


Version 6

⭐ MkDocs Exporter is now a single plugin (there is no longer the need to register exporter-pdf, exporter-extras...), see for yourself: https://github.com/adrienbrignon/mkdocs-exporter/blob/7f49ac6e1ebba5d32035cc1313c422b6c5410c35/mkdocs.yml#L61-L88

📃 Introduced the aggregator feature, it can be used to combine PDF documents as a single one (fixes #)

The behavior of the aggregator with cover pages can be configured

all (default): will include all cover pages
none: will exclude all cover pages
limits: will include only the front cover of the first page and the back cover of the last page

🐛 Miscellaneous fixes and enhancements
🐍 Dropped Python 3.8 support

I am still working on a workaround to ensure page numbers are coherent across files.

I am still working on a workaround to ensure page numbers are coherent across files.

✅ Done!
I am still polishing some rough edges.


fixes:  #27511

Fixes: #27511

Screenshots and screen captures:

Self-review checklist

[x ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x ] Explains differences from previous plans (e.g., issue description).
[x ] Highlights technical choices and bugs encountered.
[x ] Calls out remaining decisions and concerns.
[x ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x ] Each commit is a coherent idea.
[x ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x ] Visual appearance of the changes.
[x ] Responsiveness and internationalization.
[x ] Strings and tooltips.
[x ] End-to-end functionality of buttons, interactions and flows.
[ x] Corner cases, error conditions, and easily imagined bugs.

this could be my first open source contribution

Thanks, but what's shown in your screenshot does not look to be a tooltip.


Flaky test: Method_with_constant_queryable_arg

This test fails occasionally with the following error:
Assert.Equal() Failure
Expected: 1
Actual:   2
   at Microsoft.EntityFrameworkCore.Query.SimpleQueryTestBase`1.Method_with_constant_queryable_arg()

@maumar to check test is disabled (or fixed!)

disabled in a9859a73f375765cf274d66259d05b20fa4e3e3c

This is no longer relevant due to the query pipeline changes.


[Azure] Add vnet to paraglider

AttachResource function to add a resource with a specified resourceID and namespace to the paraglider namespace by creating a peering to gateway vnet.
Added unit tests for the AttachResource function
Does not rename the vnet, but would tag the resource (to be implemented in another PR)
Refactor some existing tests and functions to enhance reusability

#AttachResourceAzure

Just a few minor comments!
For the future though, maybe it would be good to separate refactoring/reorganizing from new functionality as much as possible in PRs just so they are a little easier to read.

Well noted! Thanks Sarah