chore(Dependencies) : update jsoniter-scala-core, ... from 2.20.7 to 2.30.9

About this PR
📦 Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.20.7 to 2.30.9
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.github.plokhotnyuk.jsoniter-scala" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #748.


Fix markdown link in components.md

😅 I made a huge mistake when writing this, confused between internalization and internationalization

https://github.com/neoforged/Documentation/blob/main/docs/misc/components.md?plain=1#L10
This uses [internationalization] so this one is definitely broken now


feat: add Ruby version file and Rubocop config files

Fix spelling and lint the 2.5 Gemfile

Closing as test fails and this has been sitting for too long


[WIP] Centralize documents and check out views for annotation

This PR adds a script to re-download all documents using pubget, and if there are any differences in the text, update the relevant jsonl labelbuddy files.
If any annotations are found that highlight text that mismatches the new documents, the operation is aborted.
In the future we could add a function to adjust annotations, but currently the main issue would be annotations which add tables at the bottom, so all existing annotations would not be affected.

Thanks @jeromedockes. Will keep working on this and I think remove the one time migration script from this PR and just keep the new structure, plus any scripts to manage assuming that existing structure.

thinking about it a bit more, adjusting the annotations so they are transferred to a new version of a document sounds complicated -- there is a chance of getting it wrong and in some cases it may not even be possible (eg if the formatting of tables were to change significantly that could be hard).
labelbuddy identifies documents and links them to associations through the hash of their content, because that is the part relevant for the annotation: what the annotator sees and annotates. of course we can use a different strategy in the litmining repo but using hashes is both simple and safe.
rather than editing the annotations and transferring them to the new documents, another approach could be to accept that having several versions of a pmcid is going to happen. the aggregation can be done when using the annotations, because when we need to pool annotations from several projects I think we will not care about their exact positions anymore so they don't need to be on the same document (I might be wrong about this though) -- we can just extract the information that each annotation brings (discarding the exact start and end positions) and group by pmcid

I think that's probably a fair enough assumption, but for the purposes of encouraging denser annotation (i.e. document re-use) and reducing storage needs, it would still be nice to convert annotations if they are compatible.
Right now there are many cases where documents are simply shifted at the start or at the end, and in principle the annotation should be adjustable.
I'm going to look into it a bit more but I may abandon if its too complicated. Either way, I think accepting that different versions may exist is a safe assumption.
I suppose my question is if we assume that different versions exist, should we still centralize? My inclination is to still do it because then we can minimize duplication and a lack of overlap, even if sometimes distinct version of documents exist (and we would always prefer the latest version for a new project).

I think that's probably a fair enough assumption, but for the purposes of encouraging denser annotation (i.e. document re-use) and reducing storage needs, it would still be nice to convert annotations if they are compatible.
Right now there are many cases where documents are simply shifted at the start or at the end, and in principle the annotation should be adjustable.

That makes sense; among the things to adjust in the annotation files you would also need to update the checksum to put that of the new document. I guess we should think of it as creating automatic annotations for the new document and discarding the manually created ones.

I'm going to look into it a bit more but I may abandon if its too complicated. Either way, I think accepting that different versions may exist is a safe assumption.
I suppose my question is if we assume that different versions exist, should we still centralize? My inclination is to still do it because then we can minimize duplication and a lack of overlap, even if sometimes distinct version of documents exist (and we would always prefer the latest version for a new project).

Yes that's what I was thinking. we have a central place with jsonl files for all documents. a document can only ever be removed if it was never annotated. when we want a new version of a pmcid we add it to that central place rather than replacing -- either by appending to the jsonl file or by creating a new folder or something. When choosing what to annotate in a new project or discussing overlap between projects we think more in terms of pmcids, but in practice when a project starts we have to settle on a concrete version of a pmcid, and typically we pick the most recent one.

Hi @jeromedockes
This is ready for another review. I've implemented the following:
One time migration to new folder structure:

All document that have been annotated are stored in documents/ with the following format pmcid_{id}.jsonl or pmid_{id}.jsonl for the rate instances that there are documents not sourced from pubget directly (as Kendra did a few times)
In each file, all necessary versions (i.e. md5 hashes) of that document are stored
Within each project, the {project_name}/documents folder is not tracked by git

Checking in documents

This must be done prior to creating the DB. I deliberately did not want to rely on the db for the script, as it became confusing which should be done first.
scripts/checkin_docs.py will look in project document directories, and identify all documents that are present AND annotated by that project. Then all those documents are moved to the central storage. By default this deletes the local copies of those files, but can change this default  behavior.
Note that because the documents for a project are computed on the fly (based on which docs are references in annotations) there is actually no need to keep a pmcids.txt file. I worried that would be one more thing to keep in sync.
For now, if annotators want to add new documents, they can simply add more labelbuddy.jsonl files to the project documents folder and annotate. The ones that are annotated will then be checked in to the central storage.

Creating database

Not much changes here except that all documents in {root_dir}/documents are read in, as they are asummed to only be checked in if annotated by at least 1 project.
I believe since all other scripts use the db directly, no other changes are necessary

Checkout documents

If you want to get the annotate documents for a given project (assuming you don't already have them because perhaps you just cloned this repo), you can use scripts/checkout_docs.py. This will create a copy of the labelbuddy jsonl docs in {project_name}/documents with a given maximum batch size (default=200)

Things I did not do that I may implement as separate PRs:

Converting annotations that refer to older documents to newer versions. This turned out to be tricky because many things changes in new versions, such as how new lines are handled (/n vs  ), header and footer, etc.... This could be done but I think its riskier and can be tackled separately as a rare "clean/migrate" operation. Could be done on the more conservative side and keep old versions if necessary
Scripts to help users find overlapping documents for new annotations. This shouldn't be too hard once DB is created. Alternatively, could integrate with pubget to take a complete output and take a subset of target length (e.g. n=200), which maximally overlaps with annotated documents.

This PR also takes down document storage from ~200MB to ~50MB.
One potential downside is that documents that were not yet annotated for a given project but an annotator meant to (i.e. downloaded those specific documents intentionally) have been deleted, but that seem like a minor drawback.

@jeromedockes @koudyk If you have any feedback lmk, otherwise I'm OK to merge

Feedback:

Add pre-commit hook to run checkin docs
Add pre-commit hook check to check if and documents are missing
Run checkin docs prior to running scripts that depend on database

Delete part of the github history

Merging this. Future fixes and improvements can come as new PRs


3.x: Upgrade to OCI SDK 3.12.1 or newer requires changes

After attempting to upgrade OCI SDK to 3.12.1 (or later) I ran into a couple build issues:

module io.helidon.integrations.oci.metrics.cdi now needs requires java.desktop; // For java.beans

 [ERROR] /Users/jdipol/.m2/repository/com/oracle/oci/sdk/oci-java-sdk-monitoring/3.12.1/oci-java-sdk-monitoring-3.12.1.jar(/com/oracle/bmc/monitoring/model/MetricDataDetails.class):
warning: Cannot find annotation method 'value()' in type 'ConstructorProperties': class file for java.beans.ConstructorProperties not found

Looks like  MockMonitoring in OCI Metrics CDI needs to be updated:

ERROR] /Users/jdipol/GitHub/barchetta/helidon-3.x/integrations/oci/metrics/cdi/src/test/java/io/helidon/integrations/oci/metrics/cdi/OciMetricsCdiExtensionTest.java:[164,11]
error: MockMonitoring is not abstract and does not override abstract method useRealmSpecificEndpointTemplate(boolean) in Monitoring

After fixing item 2, item 1 above is now just a warning. Even prior to the change to upgrade the OCI SDK, the warning
[WARNING] /Users/klustria/.m2/repository/com/oracle/oci/sdk/oci-java-sdk-monitoring/3.8.0/oci-java-sdk-monitoring-3.8.0.jar(/com/oracle/bmc/monitoring/model/MetricDataDetails.class): warning: Cannot find annotation method 'value()' in type 'ConstructorProperties': class file for java.beans.ConstructorProperties not found

has always been there.

This will be resolved by: https://github.com/helidon-io/helidon/pull/7163

PR https://github.com/helidon-io/helidon/pull/7163 is merged hence closing


[Task] Add /services page

Need to create a /services page for listing all our services. Let's discuss on this issue under the discussions.
cc: @wh0sumit

resolved in #28


generate CLI returning an error with a module, reflect-metadata

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ x] 3.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I made a new project, then ran: ionic g page login.
I got:
[ERROR] Error with ./node_modules/ionic-angular/package.json file: FILE_NOT_FOUND
Error: Cannot find module 'reflect-metadata'

Note that I got this error with ionic info as well.
Expected behavior:
It would generate a new page.
Steps to reproduce:
Generate a new page via the CLI.
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
[ERROR] Error with ./node_modules/ionic-angular/package.json file: FILE_NOT_FOUND

global packages:

    @ionic/cli-utils : 1.4.0
    Ionic CLI        : 3.4.0

local packages:

    @ionic/app-scripts              : 1.3.11
    @ionic/cli-plugin-ionic-angular : 1.3.1
    Ionic Framework                 : not installed

System:

    Node       : v8.1.2
    OS         : Linux 4.4
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 5.0.4

I'm having the same problem.

I had the same problem. For me the following fixed it:

rm -r node_modules
npm install

Hello, thanks for using Ionic! @danbucholtz fixed this last night. This was caused by a classic "dependency of a dependency" issue that npm 4 allowed you to get away with but npm 5, rightly, complains about. If you start a new project now things work swimmingly. Thanks, everyone!


New Bluesky user

Title
Jordi Adoumie
Bluesky profile url
https://bsky.app/profile/joadoumie.bsky.social
Twitter profile (url or username)
joadoumie
Category
windows
Type
microsoft

Adding myself to the list - love this project. New to bluesky but happy to migrate away from X.


[IMPROVE] Add TQDM/Progress bar on tickers fetched

So far the TQDM/Progress Bar only shows the progress as 0/3 due to it being applied on a list of the 3 different financial statements. Wouldnt it be more appropriate to apply it on the loop that fetchesthe data for each individual ticker instead? That way you actually get a feeling for how far into the process it actually is.
An example being wrapping ticker_list_iterator in tqdm() at line 148 in fundamentals_model.py

I get it but it is a bit trickier to implement. For example, a somewhat similar result you wish to see can be achieved by calling the financial statement functions separately.
Feel free to open a PR though!


Rhino throws uncaught JavaScript runtime exception: ReferenceError: "window" is not defined

I cloned the repo and tried to run Yslow in Rhino with the following:
make
cd build/rhino
java -jar rhino.jar yslow.js http://example.com/

and got this:
js: uncaught JavaScript runtime exception: ReferenceError: "window" is not defined.

In several Google groups some extra parameters were suggested, for example -opt -1 which I tried too.
java -jar rhino.jar yslow.js -opt -1 http://example.com/

Same thing.
Is this a bug or am I doing it wrong? My skills in env.js is too brief for me to find it out myself.
If it works, please describe the correct use with Rhino here and update the README with Rhino instructions.

Rhino is still experimental, thus is not list in the official flavors on README. Moreover it's supposed to work with .har files and not directly fetching URLs.
rhino yslow.js foobar.har

Where rhino is an alias for java org.mozilla.javascript.tools.shell.Main -opt -1
I've just created an issue #21 to finish Rhino support.

Thanks for the reply. Will read more about .har files, then.


Update free-programming-books-zh.md

add Go语言高级编程
https://github.com/chai2010/advanced-go-programming-book

Great addition! Take a look at #2503 for some info on how the linter prefers to order Chinese titles.

PTAL

Great, thanks! And welcome to free-programming-books.


chore(requirements): update Django to v1.11.3

See https://docs.djangoproject.com/en/1.11/releases/1.11.3/

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1312      +/-   ##
==========================================
- Coverage    86.9%   86.88%   -0.03%
==========================================
  Files          45       45
  Lines        3941     3941
  Branches      682      682
==========================================
- Hits         3425     3424       -1
- Misses        345      346       +1
  Partials      171      171


Add azure pipeline

What kind of change does this PR introduce?
CI

Did you add tests for your changes?
N/A

Does this PR introduce a breaking change?
N/A

What needs to be documented once your changes are merged?
nothing

Blocked by facebook/jest#6965

Blocked by #8015 because of a memory leak


[UPDATE] v1.0.0 alpha1

Description
Updates to v1.0.0-alpha1 (Initial release).
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update
[x] Update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new errors or linter warnings
[x] My changes generate no problems with other code
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing tests passed locally with my changes

This PR will be merged tomorrow by adding the Approved and Can be merged labels.

@Mergifyio update


[FIX]Show some Warnings when login or refresh

pascalprecht.translate.$translateSanitization: No sanitization strategy has been configured. This can have serious security implications. See http://angular-translate.github.io/docs/#/guide/19_security for details.


Add tests for json storage

Closes #6

Codecov Report

Merging #14 into master will increase coverage by 4.88%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   28.45%   33.33%   +4.88%
==========================================
  Files          15       15
  Lines         239      246       +7
  Branches       38       39       +1
==========================================
+ Hits           68       82      +14
+ Misses        171      164       -7


Create new Invoice -> please make it scrolable

Create new Invoice -> please make it scrolable

Espo: NDEFC3-H4VAIU-FQ5GFT-IK3IJR-4DGKF7-PEQKRW-LPZ5

The Keyboard is over the "create" Button. It would help to slide scroll the popup.
Or what did you mean with "question" :)

"Question" mean that K.Miliak need to agree that this is a bug. This screenshot was done on IPhone 4 This screen size is not supported by the specification.

The taken screenshot above is from my Iphone 5S

Guys, please check last build (41), developer added scroll to the area with inputs. Let us know if this solution will work as we choose easiest way. Adding scroll to screen would take much time, as developer will need to add scroll to the whole section (three screens)

This solution is very good. It's just that what I had wished in my first post. :)

fixed

Bounty paid. Tx hash:9a201f16342af550f15866ef80a884f12d27dc0d489240e4c7157f2bab4f62d2


 remove the year toolbar

Remove the year toolbar and revise the whole date grid design accordingly.

Date picker initial mockup of the stretched layout with year toolbar removed.

Moved to https://github.com/vaadin/vaadin-date-picker/issues/97


Fix memory leak in mbedtls_x509_csr_parse

Description
Fix memory leak in mbedtls_x509_csr_parse
Status
READY
Requires Backporting
Yes
Which branch? I did not check
Migrations
No
Additional comments
Found using oss-fuzz
Todos

[X] Tests
[X] Documentation
[X] Changelog updated
[X] Backported

Steps to test or reproduce
Allocated memory in previous call to mbedtls_pem_read_buffer does not get freed without this patch

CLA has just been submitted through mail

@catenacyber Thank you for your contribution! As sent in mail, the CLA is being proccessed

@catenacyber I have just opened on your name bug #1623 for tracking purposes

Could this be written in a shorter way?
#if defined(MBEDTLS_PEM_PARSE_C)
    mbedtls_pem_init( &pem );

    /* Avoid calling mbedtls_pem_read_buffer() on non-null-terminated string */
    if( buf[buflen - 1] == '\0' )
    {
        ret = mbedtls_pem_read_buffer( &pem,
                               "-----BEGIN CERTIFICATE REQUEST-----",
                               "-----END CERTIFICATE REQUEST-----",
                               buf, NULL, 0, &use_len );

        if( ret == 0 )
            /*
             * Was PEM encoded, parse the result
             */
            ret = mbedtls_x509_csr_parse_der( csr, pem.buf, pem.buflen );
        mbedtls_pem_free( &pem );
        if( ret != MBEDTLS_ERR_PEM_NO_HEADER_FOOTER_PRESENT ) //not sure if this line is necessary
            return( ret );
    }
#endif /* MBEDTLS_PEM_PARSE_C */

Needs backporting to mbedtls-2.7 and mbedtls-2.1.

@andresag01

the value of ret after line 290 satisfies the following condition ret != 0 && ret == MBEDTLS_ERR_PEM_NO_HEADER_FOOTER_PRESENT

Function call in the line 287 could return MBEDTLS_ERR_PEM_NO_HEADER_FOOTER_PRESENT, therefore memory leak is possible. The issue existed since polarssl days.
Refactored code in this message should never leak memory. If necessary, a new PR could be created.

@andresag01
There is no such leak (for now) :

pem context is initialised in lines 281

There is no allocation, just a memset
The allocation take place in mbedtls_pem_read_buffer but only after we are sure we will not return MBEDTLS_ERR_PEM_NO_HEADER_FOOTER_PRESENT

The code proposed by @irwir looks good to me.
This pull request was the minimum change to fix the leak.
I let you decide if you want refactoring in the same or another pull request

Hi @catenacyber,
I'd like to go with @andresag01's suggestion, but I'm conscious that this is your PR. You can either fix it yourself, and resubmit, or if you're having more fun finding the problems than fixing them, @andresag01 can do the re-work. Either way, we will credit you for the work you did here!
Please let us know.

Here you go with one more commit to simplify the code.
I let you review it

I've backported this to mbedtls-2.7 with #1772, and mbedtls-2.1 with #1174.
Once those are approved, this can move to 'ready for merge'.


add log rotation

Are there plans to support log rotation?

There is.
Please check core/logx. Search keyword RotateRule.


lua-language-server: 3.9.2 -> 3.9.3

Description of changes
Release: https://github.com/LuaLS/lua-language-server/releases/tag/3.9.3

Things done

Built on platform(s)

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Result of nixpkgs-review pr 319132 run on x86_64-linux 1

3 packages built:

lua-language-server
lunarvim
vscode-extensions.sumneko.lua


multiple orientdbs in the same JVM?

I'm having serious problems when running tests that generate test OrientDB instances over and over, are there any known problems in this area?
I can't give you a minimal reproduction, but I can help you run my project. Tests pass independently, but fail when all run together. I'm not using any global mutable state, so I assume you must be and it is the cause of the problem.

apologies, this report is exceptionally vague... let's just stick to solid things like #5490 and #5491


Can not close modal?

Hello,
I use this code to create a modal with content
$('<a/>').modal({                 target: "#overlayDishSpec",                 fitViewport: true,                 immediate: true});
It work fine, but how can I close it by calling javascript or jquery function?
And one more thing is how can I customize modal size? "EX: width: 50%"

Hi @tuan072090
If you call the target again passing the method hide you will be able to close the model
e.g.
// Init
var $trigger = $("<a/>");
$trigger.modal({ target: "#overlayDishSpec", fitViewport: true, immediate: true});

// Later on...
$trigger.modal("hide");

http://responsivebp.com/javascript/modal/#methods
To change the width you have to use CSS.
.modal-ajax.container, .modal-iframe {
    max-width: 50%;
}

Hope that all makes sense.
Cheers
James

Thank @JimBobSquarePants , It's working now.


Webpack 5 compatibility

FetchCompileWasmTemplatePlugin has been renamed
Array changed into Set

@gluck I tried this PR and I still get one issue. Did you see this too?
TypeError: ParserHelpers.addParsedVariableToModule is not a function
        at /home/paulus/dev/hass/frontend/build-scripts/webpack-plugins/worker-plugin.js:171:27

@balloob I didn't encounter this error (because of my setup ?), but it does seem that this method was removed, so usage should be reviewed.


Range on errors not displayed

Sometimes only the opening bracket is highlighted when displaying the issue in the uploaded file view. Is there a way to highlight the entire range.

@grahamegrieve The ValidationMessage is only providing me with a line and ccolumn number.
I'm going to drop v1.0 of the website for now, and we can revisit this.


Snapshot download not working after upgrade to 0.77

Home Assistant release with the issue:
0.77.2

Last working Home Assistant release (if known):
0.76
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io on Raspberry Pi 3 Model B

Component/platform:
Auth

Description of problem:
System snapshots cannot be downloaded by GUI. The download link points to:
http://hassio.local/api/hassio/snapshots/e49dbe52/download?api_password=undefined
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
http:
  api_password: !secret http_password

Traceback (if applicable):

Additional information:

I'm seeing this as well.

/move to home-assistant/hassio

Is this still happening in 77.3?

Yes, it is still happening in 77.3.

This issue has been moved to https://github.com/home-assistant/hassio/issues/679


[W-14954399] Swap deployment option PNGs with SVGs

After rolling out the new lume icons, we got some feedback from writers about the deployment options tags listed on pages like this one. These PNG images were missed in the search for SVG icons, so I'm updating those now.
Swapping some of the icons around as well based on a discussion with writers and design.
Before:

After:

Note that the cloud-ide and desktop-ide badges are not being touched per a discussion with design and writers.

Yikes, thank you so much for catching that. Looks like I didn't push my last commit. Would you mind trying now?

Yikes, thank you so much for catching that. Looks like I didn't push my last commit. Would you mind trying now?

looks good now 😄


Hardcoded "probe-run" error messages

elf2table prints many error messages telling the user to update probe-run etc. elf2table is used in tools that are not probe-run, such as cargo-embed, which leads to very confused users.
I'm aware elf2table is supposed to be only internal to probe-run, but there's real demand for using defmt in other tools. It would be great if this use was officially recognized, stabilizing the API and fixing minor papercuts like these error messages.

I think this is a duplicate of https://github.com/knurling-rs/defmt/issues/412


"Edit this page" link

Would it be possible to have an "edit this page" link on every generated doc page and in each page of the exposition? I guess it should link to the GitHub repository, I believe GitHub has some API that lets you edit a file in the repo.

Ooh, that sounds cool (and pretty feasible, I think). However, the trick would be preserving the changes after another doc build. Without some relatively fancy mechanism (and I have no aversion to relatively fancy mechanisms when they do something valuable!), the manual changes would get clobbered in the very next build (and would have to manually get reapplied). Does this all make sense what I’m saying? Or did I misunderstand what you’re asking about?

uhhhhh...GitHub has an inline editor which submits the change as a pull request

Oh, I think I did misunderstand! You’re talking about editing the source files! Duh, brain is a little slow today. 😅 Yes, we should be able to make this work. Devil in the details, but any obstacles have got to be surmountable.

The link is formed like this
https://github.com/boostorg/docca/edit/develop/LICENSE_1_0.txt

This is starting to sound lame... lol

There's no way to bring the edit box focus directly to the line in question, and there's no way for the xslt to know which line in the source file it corresponds to in the first place. Not very streamlined.


each time an action is triggered in search bar, the map adds data

I haven't seen this issue occur in quite a while. Will re-open if it re-occurs.


Fix Centos 8 build

With CentOS 8 changing to an upstream release build RHEL/CentOS/Oracle Linux 8 builds to use Rocky Linux 8 as the base.

Closed by #6


OutputManager singleton

It works but apologies that this PR is more plus than negative. We're not reaping the benefits yet – I'm only taking advantage of it in a few places.  Will do those in subsequent PRs.
Note that this changes the behavior slightly with app.run. Previously this used to print output:
with app.run():
    f.remote()

However going forward you need this:
with enable_output():
    with app.run():
        f.remote()

This is intentional! IMO when using Modal "as a library" we should never output anything by default – libraries shouldn't do that. This is also the reason I added helper methods a few days ago to get logs programmatically. You can use app._logs() and image._logs() (although those are experimental still).
When using Modal "as a framework" we output by default though – modal run works just like usual.

However going forward you need this:
with enable_output():
    with app.run():
        f.remote()

The rationale here makes perfect sense but this feels a little cumbersome especially if enable_output mostly exists to be used with app.run. Could we maybe have
with app.run(output=True):
    ....

Maybe worth spending a few minutes thinking about the best name.

The rationale here makes perfect sense but this feels a little cumbersome especially if enable_output mostly exists to be used with app.run. Could we maybe have

We used to have the show_progress bool which I just deprecated in the same PR. Could potentially add it back if we want to? I'm somewhat leaning towards the context manager syntax since it would be useful in other app-less contexts:
with modal.enable_output():
    sandbox = modal.Sandbox.create(...)

One option is to keep show_progress for how but emit a warning.

Are there usecases for enable_output beyond (a) app.run() and (b) Sandbox.create?

Are there usecases for enable_output beyond (a) app.run() and (b) Sandbox.create?

Not that I can think of today

And do you think there's a usecase for separating the scope of enable_output from the scope of app.run?

Btw one general challenge with something like app.run(output=True) is that the output manager is now a singleton, so app.run(output=True) wouldn't necessarily be concurrency safe since it modifies the global scope.
I.e. if one thread does
with app.run(output=True):
    ...

And another thread does

```python
with app.run(output=False):
    ...

Then the second thread will override the first thread. Maybe a bit of an edge case. This PR breaks this behavior to be clear – we used to support it, but having localized output control is kind of annoying imo.

I'm not following how that's different than using the context manager in one thread but not the other. Won't the singleton pattern also make the state change leak?

I'm not following how that's different than using the context manager in one thread but not the other. Won't the singleton pattern also make the state change leak?

This PR introduces the singleton – there's no state leakage in current main. But the introduction of the singleton means I think it's semantically wrong to support app.run(show_progress=True) since keeping that flag would introduce stage leakage. That's why I think it makes more sense to remove that flag and make people use a separate context manager which makes the global state explicit.

I think there are two different effects that show_progess=False or enable_output=False could have:

It could toggle the singleton to False; I think this would cause the state leakage that you're worried about
It could just leave the singleton state unchanged, which should be identical to the case where you use a with enable_output context manager in one location but not the other.

I think either way you could have some state leakage where an app run that has not explicitly opted into seeing output would start seeing it. Maybe you think that's more confusing if you've explicitly spelled it as show_progess=False?
Personally I think this is kind of an edge case, and otherwise having it as a parameter in the two functions where you'd use it is more discoverable and simpler. But idk, maybe it's an argument against the singleton pattern altogether if you think that the concurrent-runs-with-different-output-desires usecase will be common.

I think this edge case with state leakage is very rare to be clear – that's why I think it's fine to use a singleton for output management. I think I just have some sort of philosophical aversion about object methods altering global state, so that's why I'm a bit against having a flag on app.run going forward.
Tbh the bigger user concern I have is that the default used to be to always output, whereas going forward it will default to not output. So I think some users who run app.run might be confused about that? We should probably add some more docs about it to app.run and to the changelog.

So I think some users who run app.run might be confused about that? We should probably add some more docs about it to app.run and to the changelog.

Yeah — I guess the "right" thing to do here would be to start showing a warning message that can be disabled by using the new syntax. Not sure how much that would hamper getting the rest of the changes in though.

Yeah — I guess the "right" thing to do here would be to start showing a warning message that can be disabled by using the new syntax. Not sure how much that would hamper getting the rest of the changes in though.

Right but you don't want to show a warning on app.run() I think – it's the default behavior change that worries me. Although I sort of expect that this isn't super common. My feeling is that most people use modal run

Right but you don't want to show a warning on app.run() I think

Why not? I mean in the show_progress=True case. Although, we should probably also warn on show_progress=False too since that code will break on an update.

Why not? I mean in the show_progress=True case. Although, we should probably also warn on show_progress=False too since that code will break on an update.

In those cases sure! But what if the flag isn't provided at all?

Given that we're changing default behavior — and that default behavior is "show some output" — we should still warn. But the forward-looking way of doing things (i.e., using the enable_output context manager) should also flip some state that disables the warning.

Actually nvm this is more complicated!

Scenarios:

You are relying on current default behavior and want to keep getting output or explicitly setting show_progress=True
-> you're happy getting a warning directing you to the new way of getting output
You are currently using show_progress=False because you don't want output
-> Showing a warning that this will break makes sense, but if you just remove the parameter to accept the future default, you'll get a warning that you can't silence! Bad!
You are currently accepting the default behavior but actually would prefer not to see output, you just didn't notice it was an option
-> You can't be that bothered by the output, so maybe it's fine to see a warning and know that things will change in the future

The second bullet clearly the hard one to solve for.
One not great solution is some kind of MODAL_APP_RUN_OUTPUT_WARNING=0 env var that you can set, but that's kind of yuck.
Another solution would be to change the default behavior now, and only deprecate show_progress later. Even slower migration process, but maybe fine?
I do suspect that more users are in the first bullet than the second bullet but it's hard to know for sure. I'll think about it a little more.

Ok thinking about it I think the most gentle way to roll this out would be to:

If show_progress is not provided (i.e. we default it to None):

If enable_output is not run, point out that output will be dropped and they should run enable_output or set show_progress=False

If show_progress is True

If enable_output is not run, run it automatically and point out that this will be required soon and that they should update their code

If show_progress is False: do nothing

... wait a few months. then change the logic to this:

If show_progress is not provided (i.e. we default it to None): do nothing
If show_progress is True: raise a deprecation error
If show_progress is False: print a deprecation warning that this has no effect and they should remove this flag

... wait a few months. then change the logic to this:

If show_progress is not provided (i.e. we default it to None): do nothing
If show_progress is True: raise a deprecation error
If show_progress is False: raise a deprecation error

Yeah that sounds right to me as the smoothest deprecation path.

Haha ok stay tuned for 100 lines code + test for all the different deprecation options. I'll look at it in the next few days.

Added a ton of deprecations. I chose the most aggressive route that still doesn't break code – but basically if show_progress is None or True then we'll not print output, but will print a warning about it asking the user to enable output. I think a slight change in behavior is justified since it's unlikely to actually break anything downstream (unless they run Modal in a subprocess and read the output or something?)

Had to update a ton of tests to get this working. It's a bit annoying because all I did was to add show_progress=False to suppress the warning and then we're going to have to remove it later. But it's super mechanical and pretty easy to do that later.


Regions

I noticed that if I try to pull out the somatosensory cortex layer 2 using the logic in the example it does not get everything
ids = rm.find('SS2', 'acronym', with_descendants=True)
np.count_nonzero(np.isin(brain_regions.raw, list(ids)))
The list of regions that it pulls out is (614454286 )is less than the true number of layer 2 regions, which should be:
{614454286,614454294,614454296,614454298,614454300,614454302,614454304,614454306}
This is more a consequence of how the json hierarchy file is organized rather than voxcell itself, but might be an indication that we need to rework the hierarchy file to accommodate such cases. I'm posting it here just so that we are aware of it.

I'm going to close this; we're also looking for a solution in https://bbpteam.epfl.ch/project/issues/browse/BBPP134-64


chore: deprecate dfx wallet redeem-faucet-coupon

Description
Upcoming PR will introduce functionality allowing for redeeming the coupon straight into cycles-ledger (no wallet involved), making wallet part of the command a bit confusing.
Additionally, we wish to steer people away from dfx wallet unless they really want to use one.
How Has This Been Tested?
covered by CI
Checklist:

[x] The title of this PR complies with Conventional Commits.
[ ] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

Absolutely, that makes complete sense. In the meantime, the same has been discussed on Slack with @dfx-json & @sesi200; the current plan is to:

put deprecation warning on dfx wallet redeem-... (by reworking this PR)
add new functionality under dfx cycles redeem (by reworking the #3515)
remove dfx wallet completely and turn it into an extension (future work)

Closing in favor of #3569


[PDI-19754] - Job entry "Delete files" fails if the files that needs to be deleted do not exist

PR will fix the below issue
If files exist, the delete is performed. But if files do not exist, then execution fails.

Closing as it's no longer to be backported to this branch.


Windows: install and upgrade via package manager

For macOS, we have embraced Homebrew as our method for distributing and upgrading Scalar. We should do similar behavior for Windows.
Options include:

Chocolatey (external, not super-widely adopted)
winget (Microsoft project, currently not production ready)

Might also want to explore release via the Microsoft Store.

Don't forget scoop :)
https://github.com/lukesampson/scoop

We are almost complete with this one. Keep posted.

@derrickstolee , out of curiosity, which package manager are you targeting?

@derrickstolee , out of curiosity, which package manager are you targeting?

@marklannucci: We are targeting winget as it will (eventually) be automatically included in Windows. For now, it can be installed by downloading the .appxbundle from the latest microsoft/winget-cli release.
We already have winget integration for the microsoft/git fork of Git, and can be installed via winget install microsoft-git.


implement closed-notebook search

Fixes #173726
An introductory PR for closed-notebook search. This only searches inputs in closed notebooks, and might have a few extra bugs with opening at the correct position, but it is behind an experimental flag (off by default for now).

I probably want to merge in what I have now for this, but my next work forward would look something like:

Improve performance


Create a watch api for secrets

The problem faced currently?
Many services use the secrets management feature to store confidential information. In a distributed system, we need a mechanism to propagate changes in the secrets to the various clients.
How can we solve it?
We can create a watch secrets http endpoint in the runner which can stream json strings down to the client. Subsequent versions will be seperated by a \n.
For K8s
In runner, we should create a Shared informer to watch the secrets of all SC namespaces. Clients can simply opt for changes in a particular secret.
For Docker
This should be pretty straight forward. Simply forward secret requests to all interested clients.

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

Will be solved by #1284


Django 4.0 compatibility

Django 4.0 beta is released and this release removes a few deprecated features.

Utility force_text was removed in favor of force_str
Utility ugettext_lazy was removed in favor of gettext_lazy

ImportError raised when trying to load 'slick_reporting.templatetags.slick_reporting_tags':
  cannot import name 'force_text' from 'django.utils.encoding' (.../django/utils/encoding.py)

Thank you !!


Timestamp issues(handling milliseconds,seconds, and date string) fixed for ciscoduo and other collectors, moved httpagents to top of code

Problem Description
Problem :
1)https://alertlogic.atlassian.net/browse/VTT-6616, going wrong messageTs value while sending to ingest. Ingest not ingesting data due to incorrect date format. There is changes in tsparse function in pr . Which expecting the milliseconds but cisco duo having timestamp in seconds, so it divided by 1000 and sending wrong value in sec while sending to ingest.
2) Create http and https agent outside of RestClient to create only once the agent instance and modify Maxsockets and add Max free sockets to config.
Solution Description

Fixed Timestamp issues(handling milliseconds,seconds, and date string) for ciscoduo and other collectors
Moved creation of http and https agent to top of the code to reuse existing http and https agents instead of creating new evertime creating rest client.
let DEFAULT_HTTP_HTTPS_AGENT_CONFIG = {
keepAlive: true,
maxSockets: process.env.MAX_CONNS_PER_SERVICE || MAX_CONNS_PER_SERVICE, // Maximum number of sockets to open
maxFreeSockets: process.env.MAX_FREE_SOCKETS || MAX_FREE_SOCKETS   //sets the maximum number of sockets that will be left open in the free state
};

const httpAgent = new http.Agent(DEFAULT_HTTP_HTTPS_AGENT_CONFIG);
const httpsAgent = new https.Agent(DEFAULT_HTTP_HTTPS_AGENT_CONFIG);

I don't think we should be linking to internal tickets on public github. The code looks good though, +1


[MINT-3254] Updated POCs for retiring individuals

MINT-3254

Update references for retiring individuals to Sara Fontaine

Description

How to test this change

Rebuild containers
Verify names are correct on local webserver
Verify database contents on points of contact for possible operational solutions 25 and 40

PR Author Checklist

[x] I have provided a detailed description of the changes in this PR.
[x] I have provided clear instructions on how to test the changes in this PR.
[x] I have updated tests or written new tests as appropriate in this PR.
[x] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
When approving a PR, provide a reason why you're approving it

e.g. "Approving because I tested it locally and all functionality works as expected"
e.g. "Approving because the change is simple and matches the Figma design"

Don't be afraid to leave comments or ask questions, especially if you don't understand why something was done! (This is often a great time to suggest code comments or documentation updates)
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.

I'm not sure what's causing this build test to fail:
PR Checks / run_tests / build_docker_images
It seems unrelated to this PR

I'm not sure what's causing this build test to fail:
PR Checks / run_tests / build_docker_images
It seems unrelated to this PR

@OddTomBrooks This PR should fix this issue:
https://github.com/CMS-Enterprise/mint-app/pull/1568

@OddTomBrooks , when you merge this, can you please make sure to update the MTO feature branch so that migrations don't conflict?


Add copyTree and renameKey

copyTree can be implemented with RegCopyTreeW, and renameKey can be implemented with RegRenameKey. Both functions are available since Windows Vista, and the wrapper should be straightforward.
RegRenameKey is not documented, but the function exists, unchanged, since Windows Vista, and is declared in the SDK headers. For such a basic operation, I think that it's worth using an undocumented function.

I'm a bit dubious about adding a binary dependency on undocumented functions, even if well known and declared in the headers. It smells of the actually internal, but court-ordered documented functions that you absolutely shouldn't use (mostly because they suck), but the real issue is that since it's compiled in you can't choose not to have that dependency, and I don't want to force that on people. Possibly the bigger issue is the semantic effect (e.g. key watches moving with the key!) may not be expected by some software. I don't officially have a minimum windows version, but requiring Vista seems fine enough (I still occasionally see a requirement for 7), and I probably already have that as a dependency anyway.
copyTree seems to mostly be a missed case, that at least is very uncontroversial (other than being added in Vista, as mentioned above). When I get some time, I'll look at this one at leats.

Guess what, it is documented now! :)
Turns out that Node dropped support for XP and Vista in v6.0.0 (2016), so I believe it's fine.
Regarding key watches moving with the key, I can't think of a problematic case for that. It should have the same effect as renaming a key with regedit.

Wow, that was fast!

Could you check out v0.3.5 in @next to let me know if this is good? I've got a few too many changes for me to be happy with an 11:30PM release to @latest 😊

I tried renameKey and it seems to be working as expected.


[Help] Display entire newest post on home page?

Mandatory Step

[X ] I am using latest production release of Elegant

I need help with
How do you display the entire newest post on the 'home' page? I don't want just a list of post links to take the reader elsewhere, I want have the home page always display the entire newest post, and have all other posts live on another page/archives.
This is a common scenario, possible in Jekyll (using Liquid), and trivial in WordPress. I tried RECENT_ARTICLES_COUNT = 1, but that just displays a link to the post.

Elegant is designed to be minimal. We thought "About me" is more important on the landing page than the recent blog posts.
But I see your point. Some users will definitely want to display the entire posts on the landing page. Unfortunately, it is not possible right now.


Always a Chance patch not working

The Always a Chance patch isn't showing up in the menu, nor is it working when tested (nat 1 attack roll misses despite having it).
combatLog.txt
GameLogFull.txt

Ups. I reorganised the patch and forgot to call it. Will be fixed soon.

Fixed. Please reply if not working.


Fixed quadruped and biped examples

This PR fixes #1207.

Hi ! This project doesn't usually accept pull requests on the main branch.
If this wasn't intentionnal, you can change the base branch of this PR to devel
(No need to close it for that). Best, a bot.


Issue with achieving the same result as with Threejs

Hi,
I'm trying to achieve the same result with angular-three as I got with using ThreeJs "vanilla" way. I prepared both examples on Stackblitz. There is a difference in color although I'm using the same parameters for all the lights, materials, and background. Could you please help me understand why there's a difference?
Using ThreeJs: https://stackblitz.com/edit/angular-ivy-k2e13k
Using Angular-Three: https://stackblitz.com/edit/angular-ivy-uefxeg
Also, I wasn't able to find a way to generate simple terrain without using physicBody. Therefore I defined geometry and material in code-behind. If there is some other way, I would much appreciate an example.

hi @MaticDiba, I'm not well versed in THREE to know for sure. But it seems like it's some Color Space issue. Are you sure both Stackblitz use the same THREE.js version? In some later version, THREE.js has ColorManagement automatically. On the Angular version, you can try using the following inputs on ngt-canvas to test it out: legacy, linear, and flat

with `linear`

with `linear` and `legacy`

with `linear`, `legacy`, and `flat`

Hi again, I just tested this the new Angular Three (in a different repo, different package, still in beta) https://stackblitz.com/edit/angular-three-demo-template-6c2vs4?file=package.json,src%2Fmain.ts,src%2Fapp%2Fscene.component.ts,src%2Fapp%2Fapp.component.ts
Also, I tested the same setup with React Three Fiber (https://docs.pmnd.rs/react-three-fiber/getting-started/introduction) and they seem to have the same output.
Both Angular Three and React Three Fiber have automatic color management (and rely on THREE.js ColorManagement prop)
Thoughts?

Thanks for your fast replay :)
I think that using all three:  legacy linear flat are getting closer to the result. But still not there.
I guess I would need to somehow figure it out how to achieve the same coloring with automatic color management. I am quite fresh with Threejs.

@MaticDiba it is easier to rewrite your example with vanilla JS without any framework then use my R3F codesandbox and ask about the issue in React Three Fiber github discussion. I think you'd get better answers there than here

@MaticDiba I think I figured out the differences but it doesn't apply to the current version of Angular Three. The new Angular Three (from Stackblitz I shared above) DOES apply. I just wanted to let you know the differences so you're aware
Both Angular Three and React Three Fiber set up some defaults:

ColorManagement.legacyMode = false on THREE.js 0.139 and older
WebGLRenderer.outputEncoding = THREE.sRGBEncoding
WebGLRenderer.toneMapping = THREE.ACESFilmicToneMapping

On this Stackblitz (https://stackblitz.com/edit/angular-three-demo-template-6c2vs4?file=package.json,src%2Fmain.ts,src%2Fapp%2Fscene.component.ts,src%2Fapp%2Fapp.component.ts), you can add [linear]=true to change the outputEncoding and [flat] = true to change the toneMapping. Then the result will be the same as the plain THREE.js sample. However, please be advised that the defaults Angular Three (and React Three Fiber) sets are to provide more correctness to the color space.


Restore can fail when at least one partition was empty

If you have a topic backup where the count of entries is smaller than the count of partitions. You will get an IndexOutOfBoundsException when you try to restore the backup of the topic. Error message is Could not find offset 0 in topic test01, segment segment_partition_005_from_offset_0000000000 Maybe you add a flag when storing the partition, that it is empty, or you modify how empty partitions are handled (e.g. printing a warning message to the log).
Full Stacktrace:
java.lang.IndexOutOfBoundsException: Could not find offset 0 in topic test01, segment segment_partition_005_from_offset_0000000000
        at de.azapps.kafkabackup.common.segment.SegmentReader.seek(SegmentReader.java:50)
        at de.azapps.kafkabackup.common.partition.PartitionReader.seek(PartitionReader.java:49)
        at de.azapps.kafkabackup.common.partition.PartitionReader.<init>(PartitionReader.java:34)
        at de.azapps.kafkabackup.source.BackupSourceTask.lambda$findPartitions$0(BackupSourceTask.java:85)
        at java.base/java.util.Iterator.forEachRemaining(Iterator.java:133)
        at java.base/java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
        at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:658)
        at de.azapps.kafkabackup.source.BackupSourceTask.findPartitions(BackupSourceTask.java:79)
        at de.azapps.kafkabackup.source.BackupSourceTask.start(BackupSourceTask.java:46)
        at org.apache.kafka.connect.runtime.WorkerSourceTask.execute(WorkerSourceTask.java:202)
        at org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:177)
        at org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:227)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:834)

I see whats the problem… The SegmentReader::seek() function should check whether the file is empty and in case the file is empty it should not throw the exception. (I do not think it should log a warning at all because this should be the intended behaviour)

Similar issue for me,
I have Kafka 2.3.1 cluster and able to take periodic backups from Kafkabackup tool on another standalone Kafka 2.4.0.
However when I try to restore this backup to either of 2.3.1 Cluster or 2.4.0 Standalone, the backup fails with following error.
Another thing that I have noticed that the backup process did not create /tmp/connect.offsets file... not sure if this is related???
Please advise.
[2020-02-25 10:58:45,467] ERROR WorkerSourceTask{id=backup-source-0} Task threw an uncaught and unrecoverable exception (org.apache.kafka.connect.runtime.WorkerTask:179) java.lang.IndexOutOfBoundsException: Could not find offset 0 in topic Test_Topic, segment segment_partition_028_from_offset_0000000000 ok Optional.empty         at de.azapps.kafkabackup.common.segment.SegmentReader.seek(SegmentReader.java:50)         at de.azapps.kafkabackup.common.partition.PartitionReader.seek(PartitionReader.java:49)         at de.azapps.kafkabackup.common.partition.PartitionReader.<init>(PartitionReader.java:34)         at de.azapps.kafkabackup.source.BackupSourceTask.lambda$findPartitions$0(BackupSourceTask.java:85)         at java.util.Iterator.forEachRemaining(Iterator.java:116)         at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)         at java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:580)         at de.azapps.kafkabackup.source.BackupSourceTask.findPartitions(BackupSourceTask.java:79)         at de.azapps.kafkabackup.source.BackupSourceTask.start(BackupSourceTask.java:46)         at org.apache.kafka.connect.runtime.WorkerSourceTask.execute(WorkerSourceTask.java:208)         at org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:177)         at org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:227)         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)         at java.util.concurrent.FutureTask.run(FutureTask.java:266)         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)         at java.lang.Thread.run(Thread.java:748)

Please review and merge if changes are OK.
https://github.com/itadventurer/kafka-backup/pull/19

I think the issue is here:
https://github.com/itadventurer/kafka-backup/blob/3c95089c699af6b42207925b05ff04a721da95e1/src/main/java/de/azapps/kafkabackup/common/partition/PartitionWriter.java#L34
Here you tell the SegmentWriter to write the startoffset of 0, but no content is written to the Record file.
The Problem is, that a startOffset of zero is a valid option since you start to count at zero so the Reader will later think there has to be some data it can read.
If the Data read is empty, you will receive the error posted above.
My Idea is to again have something that tells the reader, that there is empty data by changing the line from
currentSegment = new SegmentWriter(topic, partition, 0, topicDir);

to
currentSegment = new SegmentWriter(topic, partition, -1, topicDir);

and insert a check when reading the startOffset

Should work now (https://github.com/itadventurer/kafka-backup/commit/1c200daced7fc8f514afc4f6f5215954e4b819f4), tests will come with #20. Please reopen if issue persists


Flutter Pub Get and Flutter.dev not working in Uzbekistan.

Flutter Pub Get and Flutter.dev not working in Uzbekistan...
I tried many times flutter.dev but it worked only when I used Chrome Extensions such as VeePN and Winscribe.
I'm assuming that flutter pub get command is not working because of this

Hi @umidjonov
does using mirror sites help https://flutter.dev/community/china#configuring-flutter-to-use-a-mirror-site?

Hi @VladyslavBondarenko,
I've been using Flutter 1.17.3 on Ubuntu 20.04 and it kept asking for Upgrade...
But I switched to Windows 10 and simply reinstalled Flutter 1.17.4 and its "flutter pub get" is working just fine.

@umidjonov
then closing the issue. If you disagree please write in comments and I'll reopen


Update our CI to use v22.2

Update both unit testing and docbuild to use v22.2

@germa89, is the database feature broken on v22.2?

@germa89, is the database feature broken on v22.2?

Yes. @dts12263 found out. That's why nightly MAPDL fails. @FredAns needs to implement some changes.

We've gotta disable this feature or make it dynamically available. Implement a server version check and then raise a version error when attempting to start this.
Also, if we're not doing this already, let's implement nightly testing on our nightly builds "inside" the fence.

This works. In a follow up we need to raise when attempting to start the database.
Thanks @dnwillia-work for reminding us to do this.


refs #1142 - fixed HTTP PUT support for clientutils.sendAJAX()

This fixes sending HTTP PUT requests to REST webservices.

+1

Archiving PR using ARCHIVE label as CasperJS, like PhantomJS, is not actively maintained for now. These PRs are from 1-5 years ago but there just hasn't been maintainers to continue the work.


Enhancements/standard documents

Update CONTRIBUTING.md ✅
Update CODE_OF_CONDUCT.md email ✅
Update ISSUE_TEMPLATE to a more modern version ✅
Add SECURITY.md ✅
Add version checker script (you can use Python, JavaScript, or any other programming language) ✅

@sepandhaghighi a Kind reminder!


Can I connect to DB and get the details or store the details given by user to bot  ?

Issue: It is possible to store the user details given details in a DB given to our chat bot ! if yes can I have any example of that ?

This is a question regarding the use of rasa core no? This would be done using a custom action to query your DB, which you can read more about here: https://core.rasa.com/domains.html#defining-custom-actions

Sorry for asking in wrong place ! I will work on it ! Thank you :)


Package test

Hi @LogSkele , I wanted to do a test package before doing it on authenticlick. I have already created a package but can't seem to import it after downloading it.

You have to do it in a IDE with a terminal like VS Code?

Then in the same directory as dist,
python3 -m twine upload dist/*

If you are on windows.

Bro, I uploaded the package already. Just couldn't download it

:>


feat: hide tab bar when keyboard is shown

Closes #16
When the statusbar is not translucent, the view resizes when the keyboard is shown on Android. The tab bar stays above the keyboard. This PR makes the tab bar hide automatically when the keyboard is shown.
The behaviour is enabled by default and can be disabled with keyboardHidesTabBar: false in tabBarOptions

very cool 😎


Issue 121 - Feat: Completed Favorites page

Describe your changes

Added Favorites route

Implemented favorite context

If there is no item in favoriteContext, it will show a different page

Updated header component with the favorites/heart icon

Favorites/Heart Icon is now a component to be abstracted from the header component

Features of new favorite component inside header component:

if favorite object does not the product id

Can add product to favoritesContext onClick
Will change the SVG color fill

- if favorite object already have the product id
    - Can remove product to favoritesContext onClick
    - Will change the SVG color back to its original color

Issue ticket number and link
Resolves #121
Name of Contributor
Andersen
Type of change
Feature
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My code does not generate any new warnings or errors
[ ] New and existing unit tests pass locally with my changes (if relevant)

Went out of scope with the issue and refactored Items page and its updateBasket function. I refactored it to create new array to set the state instead of mutating the original array.


elm: 0.16.0 -> 0.17.0 (#15383), backport 16.03

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Any reason not to backport to 16.03? Just getting started on elm so I have no opinion either way.

Well this kind of breaks the API. But on other hand it's a simple upgrade plan.

@sheenobu I'd like to reject this as Elm 0.17 breaks backwards compatibility. You can easily use Elm from master for your projects and still use release-16.03 for your system. See https://github.com/domenkozar/kamdanes.si/blob/master/frontend/default.nix#L1
This is according to our NixOS stable policy of doing only minor bumps and security fixes.
Sorry for inconvenience :)


Improvements for icon/image and tooltip

It would be great if we could set an image/icon when the actual text content is empty.
An option to disable the tooltip completely would also be very helpful.

Yes. Thanks for making the issue, i will try fixing it right away,

Thanks for the quick update!
However there seems to be a typo, so it doesn't work with images yet.
https://github.com/budlabs/genmonify/blob/0a83c1805f0639a24d809c1794abe73da14244a2/genmonify#L58
It has to be this:
$message${__o[img]}${__o[icon]}

thanks, i only tested with icon, will fix and update again.


Cannot connect to neopyter extension

hi @me,
Im trying to create both test.ju.py in nvim an test.ipynb on jupyter lab. The neopyter is configured as instruction. However, when checkhealth for neopyter, it shows as below. Perhap I did something wrong, can you pls advise?
==============================================================================
neopyter: require("neopyter.health").check()

neopyter: config ~
- {
    auto_attach = true,
    auto_connect = true,
    file_pattern = { "*.ju.*" },
    filename_mapper = <function 1>,
    highlight = {
      enable = true,
      shortsighted = false
    },
    jupyter = {
      auto_activate_file = true,
      scroll = {
        align = "auto",
        enable = true
      }
    },
    mode = "direct",
    on_attach = <function 2>,
    parser = {
      line_magic = true,
      trim_whitespace = true
    },
    remote_address = "<IP_ADDRESS>:9001",
    textobject = {
      enable = true
    },
    use_default_keybindings = false
  }

neopyter: version ~
- neovim<EMAIL_ADDRESS>status: inactive

Thanks @AbaoFromCUG,
I think there was an issue with the socket connection. I got it fixed by following this discussion: https://github.com/zauberzeug/nicegui/issues/1178#issuecomment-1677678994

I now can see it listened to port 9001 but status is still inactive. I will try to debug more.


run 100 cause error "list index out of range"

Hello,
I followed the readme guide, and run the map server and reverie. But when enter option "run 100", after a few seconds, it cause a error. I log the exception, it shows "list index out of range".
(python3912) G:\ai\generative_agents\reverie\backend_server>python reverie.py
Enter the name of the forked simulation: base_the_ville_isabella_maria_klaus
Enter the name of the new simulation: test-simulation
Note: The agents in this simulation package are computational
constructs powered by generative agents architecture and LLM. We
clarify that these agents lack human-like agency, consciousness,
and independent decision-making.
Enter option: run 100
Error: Error communicating with OpenAI: HTTPSConnectionPool(host='api.closeai-asia.com', port=443): Max retries exceeded with url: /v1/embeddings (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1129)')))
Enter option: run 100
Error: list index out of range
Enter option:

And run 10 is OK

same error here

maybe the issue raise from running  out of mine api quota😂

兄弟问题解决了吗？是要用代理吗？

It seems that is the problem of network connecting to call LLM api


Global SSP2 version

Hi,
I need the global version of message_ix with SSP2 as baseline for my research.
Could you please tell me what is the baseline for the current version of message_ix. And when should we expect the SSP version of message_ix? Especially SSP2?
Thank you
Franklyn

Hi Franklyn,
While the documentation for the MESSAGE-GLOBIOM global model is online (see https://message.iiasa.ac.at/global/ and https://github.com/iiasa/message_doc), the input data workflow and setup- / run scripts are currently not public. The goal is to eventually make them public, but no specific date is set for this.
For research collaborations, please contact the IIASA Energy Program via its website.


Removed default ingress annotation

This PR removes the default nginx annotation from the values.yaml
Closes #54

We can add additional annotations by default indeed but they should be localstack specific and never assume any implementation outside the helmchart like which ingresscontroller is to be used. The only way to "remove" this annotation was to overwrite it. The default ingressClass however should have kicked in.
Default annotations could be specifying that each component is part of this helm chart, a practice that is rather common in other charts.

Thanks @bravecobra for the fix, and @LaikaN57 for the review! 👍

Forgot to mention - we'll need to bump the version to release a new version of the chart. Can either of you please help with that and create a small follow-up PR @bravecobra @LaikaN57 ? Please also add a short note to the changelog. Thanks!


[Question] Fail to use docker with mysql to deploy the Apache ShenYu gateway

Question
Refer to the deployment documentation https://shenyu.apache.org/docs/deployment/deployment-docker.
I successfully deployed Apache ShenYu admin using h2 to store data.
When I change the storage to mysql,it failed
Here is the wrong message: Web server failed to start. Port 9095 was already in use.
I'm sure no one is using this port.I have tested ports  and none of them work.
By the way,I think Binary Packages Deployment also has something wrong.
the file start.sh doesn't work on Linux server.This line of code seems to have a problem.
"PIDS=ps -ef | grep "$DEPLOY_DIR" | grep -v grep | pgrep -f java"
It seems to find out all the processes java started, causing the gateway to fail to start.

@zhanggyang  hi whats the version？

2.4.2

2.4.2

you can used by master , it will be fix

Thanks for your reply. I'll try it first

https://github.com/apache/incubator-shenyu/issues/3041

2.4.2

you can used by master , it will be fix

Hi , I tried it with the master image, and the problem was still there.

Steps To Reproduce

docker pull apache/shenyu-admin:latest
docker network create shenyu
netstat -tunlp | grep 9095   Port 9095 is not occupied
docker run -v /home/shenyu/conf:/opt/shenyu-admin/conf/ -v /home/shenyu/ext-lib:/opt/shenyu-admin/ext-lib -d -p 9095:9095 --net shenyu apache/shenyu-admin:latest  my work dir is /home/shenyu and the relevant configuration files are ready

The error massage is  “Web server failed to start. Port 9095 was already in use.”
docker exec -it nostalgic_payne /bin/sh  I exec into the container  to checkout netstat -tunlp | grep 9095

sh -x start.sh

It seems there is still a problem with the start command

You need to go into the container and modify the start.sh script. For details, please see https://github.com/apache/incubator-shenyu/issues/3041.

You need to go into the container and modify the start.sh script. For details, please see #3041.

I have change PIDS=ps -ef | grep "$DEPLOY_DIR" | grep -v grep | pgrep -f java to PIDS=ps -ef | grep "$DEPLOY_DIR" | grep -v grep. But it is not work for me.

Delete all screenshots.

Delete all screenshots.
Hi，Do you mean converting all screenshots into text or simply deleting them？

The h2 storage command docker run -d --name sy-admin -p 9095:9095 --net shenyu apache/shenyu-admin:latest  can start normally
But  docker run -v /home/shenyu/bin/start.sh:/opt/shenyu-admin/bin/start.sh -v /home/shenyu/conf:/opt/shenyu-admin/conf/ -v /home/shenyu/ext-lib:/opt/shenyu-admin/ext-lib -d --name sy_admin  -p 9095:9095  --net shenyu apache/shenyu-admin:latest can not.  think there is no problem with the master branch startup command itself, this feels like a mysql-related issue

Delete all screenshots.
Hi，Do you mean converting all screenshots into text or simply deleting them？

The h2 storage command docker run -d --name sy-admin -p 9095:9095 --net shenyu apache/shenyu-admin:latest  can start normally
But  docker run -v /home/shenyu/bin/start.sh:/opt/shenyu-admin/bin/start.sh -v /home/shenyu/conf:/opt/shenyu-admin/conf/ -v /home/shenyu/ext-lib:/opt/shenyu-admin/ext-lib -d --name sy_admin  -p 9095:9095  --net shenyu apache/shenyu-admin:latest can not.  think there is no problem with the master branch startup command itself, this feels like a mysql-related issue


Fix message.awaitMessage to handle command prompting from DMs.

Proposed Semver Increment Bump: [PATCH]
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Command prompting wasn't working for required arguments in DMs.

(If Applicable) What Issue does it fix?
Fixes this thing where a command with required arguments would throw when in DMs instead of prompting the user.

is ok

Technically obsolete in terms of master


Always On Top Feature

Is your feature request related to a problem? Please describe.
I use this tool for mobile development but there are time i wish it had an always-on-top feature
Describe the solution you'd like
I always have my mirror device on the right switch windows make it hard to work with
Describe alternatives you've considered
Hmmm there's a feature on window that currently helps hmmm window tab manger

https://github.com/Genymobile/scrcpy/blob/master/doc/window.md#always-on-top

Okays thanks

https://github.com/Genymobile/scrcpy/blob/master/doc/window.md#always-on-top

Okay thanks 🙏🏿👍🏿👍🏿👍🏿👍🏿👍🏿

Hi
can this feature be used in runtime using shortcuts  ?
thank you


feature support local tvf on shared storage

Proposed changes
Previously, local tvf can only query data on one BE node.
But if the storage is shared(eg, NAS), it can be executed on multi nodes.
This PR mainly changes:

Add a new property "shared_stoage" = "false/true"
Default is false, if set to true, "backend_id" is optional. If "backend_id" is set,
it still be executed on that BE, if not set, "shared_stoage" must be "true"
and it will be executed on multi nodes.

Doc: https://github.com/apache/doris-website/pull/494
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


TWITCH LEECHER IS DISCONTINUED

The story of Twitch Leecher ends today. I do not want to stir up unnecessary drama but give a little insight as to why I came to this dicision.

80% of the dicision has grown over time since april 2020. It's the month my daughter was born. Before that I could randomly invest 20 hours a week into TL and nobody really cared about it. That's not possible anymore and it's just how life goes. I am a happy dad now with different priorities than 2015.

Twitch as a software developer has degraded massively over the past 7 years. "This does not work", "this throws an error", "this is not documented", "this changed without an announcement" and the best one... "just wait a couple hours until it works again". The quality of the entire product is at most "fragile". Just ask yourself how often you can't login on Twitch compared to Google. An then imagine having to work with that in your own software. In the last 3 years most major issues that made TL stop working were caused by Twitch. That is not a reliable partner. And it is no fun anymore to work with their product.

Legal stuff... yea, yea, the mighty grey area of video downloads, right? I tried to make TL as "legal as possible" in terms of copyright and sub-only downloads. Therefor TL is one of the few downloaders where you have to login to Twitch in order to use it. I don't want to get into the details, but acting illegal by using the non-public GraphQL API is actually easier and leads to a much better user experience than the way TL does it. So at the end, implementing illegal stuff results in a way better software. Wow...

In the end all that does not matter anymore. The latest sign-in problem showed me I need to go forward and do something else or I lose my cool. Maybe you hear from me again in the future in the NodeJS/Electron domain, but definitely not in the Twitch domain.
So, all that remains is to say thank you!
The users, the contributers, the helping hands, the brainstormers, and the donators! It was a pleasure!
There are voices on the internet that say I got rich with TL. I want to be transparent: $2187.54 since the donation button was implemented on 03/19/2016. If anyone has a spare island I could buy, please contact me.
I wish you all the best... logging out
~ Franiac

Thank you for everything Sir.

Thank you for everything you've done for us. I wish you happiness and peace wherever you are.

Sad to hear this, but your reasons are completely understandable.

we are very sorry, and thanks for all the efforts. I was using Twitch Leecher for a long time, even downloaded whole channels. But as a programmer, I totally understand that there is no worse than fixing bugs which are caused by someone else, and always you are blamed.
And yes, twitch is a waste of skin. There was a competition streamed, and I thought "in a few days Im gonna watch it" - that video is gone, because they erase it after 7 days. Only 7 days?!

I understand your decision. TL has been of tremendous help to me recently, for one because I could get the VODs for some post-production, for two because it saved some VODs the streamer forgot to upload on their YouTube channel, and they probably would've been gone for good if it weren't for this app.
But Twitch's recent actions really cast a shadow on them. And I can understand not being able or willing to work on this anymore for personal or programming reasons. So, thank you for everything you've done. May you and your family be happy.

Came here to check if internal download speed would be possible to add but saw this message.
I hope you will be doing great in life and maybe I will be using some of your work in NodeJS/Electron.
And I will be using TL as long as it will work before switching to something other, because it has probably best UI for my needs from SW I tested.
Thanks again for this app.

Thanks for your work. I have full respect for your decision.

Nice to meet you. Greetings from Japan.
I used TL for many years until the problem occurred. It was very useful.
I am very sorry to hear about this issue, but I applaud the developers.
Thank you so much. Thank you for your hard work.

Good luck in the rest of your life :)
Does this mean the application will be disabled for good? Can't you make downloadable without an account like you used to? Downloading videos from subscribed channels would be impossible, but it would still work and you wouldn't have to do anything else :)
Anyway. Thank you for your great work.

Thank you for everything and good luck on your journey.

Do not cry because it is over, smile because it happened :)

thank you for everything. heart and soul

TL has been one of the most important tools for me for the past 5 years and it was free. Sad to see you go and such a great tool is discontinued. I can't thank you enough. All the best to you, sir.

Thank you for all you have made possible. All the best to you and your family.

All your reason are completely understandable. Have fun in your next project, it was really cool to have Twitch Leecher during the time it existed :)

Thanks for everything.

Congrats to you and your family.
Due to the source being avaliable, I do hope someone picks up the project and continues it.

All the best @Franiac ! Thanks for making awesome software.

Thanks for supporting it as long as you did. It was much appreciated.

Thank you for continuing it for as long as you did and congrats on parenthood :)

I also thank you very much, for all the years of support on this straightforward and handy app. I really hope that, when a new Twitch problem eventually arises, another Developer is able to continue your work, yes? Since it's open source
Anyway, until then, know that if you manage to get to your hands on your own user OAuth access_token, you just need to put it in C:\Users\%username%\AppData\Roaming\Twitch Leecher\runtime.xml, and when you launch the app you should be logged in ;)
<RuntimeData Version="2.0.7">
  <Authentication>
    <AccessToken> !!! token code goes here !!! </AccessToken>
  </Authentication>
  [...]
</RuntimeData>

if you manage to get to your hands on your own user OAuth access_token, you just need to put it in C:\Users\%username%\AppData\Roaming\Twitch Leecher\runtime.xml, and when you launch the app you should be logged in

I like my PC and figuring stuff out but i clicked that Link and i only see hieroglyphics.
Is there an easy explanation for this...?
Like i don't know how to code at all or sending strings to server's for a response or whatever.
I know how to find the xml File in my System but that's about it with my Tech knowledge.
I just wanna login again :(
None of these so called "Twitch downloaders" were as consistent as TwitchLeecher.
Help

@pehndaa Maybe try to read the title, the comments and the room. Just a hint.

The development is discontinued, you're correct there, but that doesn't mean the app has to cease running or users can't still endorse it or show support to it, so I'm afraid your comment is not helpful Sir :P I've edited my previous reply with the actual URL that is used to create the access token inside the app: id.twitch.tv/oauth2/authorize.
I'm still not sure how to get the access_token json after clicking Authorize though... I was hoping @aniv1re from #800 could explain how he got the token from the URL, because I've now read further into the authorization code grant flow and it looks like the URL gets you a code that you must use in a second step to get both access and refresh tokens (access tokens don't last forever and must be refreshed) by means of an HTTP POST request (example).

98% of TL users don't even know what an authentication token is. You are not thinking for "Average Joe"

Both flows ("second step" & eventual "refresh") require you to know the app's client_secret - which is, you guessed it, SECRET 😆 - therefore I'm not sure if the 2% of us who're willing to look further into this grant flow and keep using the app we love the most actually have a shot at it, or we're just going to crash and burn. The only one that can clear up that doubt is @Franiac himself; eventually my token will expire too so I'll be in the same boat as @pehndaa.

I tagged you reluctantly Dominik, as a last ditch effort, because I didn't want to put salt in the wound. But while we're at it... by now I've re-read the Twitch documentation three times, and sort of like an attorney on the devil's case I noticed a missing clue that might be helpful to solve the incident (dramatic music plays, attorney lingo intensifies).
Allow me to take you back to your Sep 26 roadmap and Oct 1 remark in issue #800, with focus on the custom url schemes fiasco: the documentation uses http://localhost:3000 as redirect uri, so I thought to myself: couldn't Twitch Leecher run a local server that listens on an arbitrary port, and use it to receive the authorization code?
Whilst I hope to spark your enthusiasm again, I do not expect you to implement that. I've also dedicated myself day and night to an online Community since Jan 2016, and I've recently been burnt out, so I feel like I can relate to you. A reply though would be deeply appreciated, and doing so doesn't imply you will have to sit down and code any of this, but rather that you're strong enough to stand up for your decision yet support others in their strive to greatness.

Maybe the time has come to let someone else play the stupid Twitch game

In this regard, a final thought when reading issue #800 - in particular the excerpt above - was if now that things have settled down and dust has cleared you might be able to identify a contributor, or a fellow developer... somebody that has helped you personally or has contributed to the project itself and has earned your trust, and invite them to the Team in order to pass them the figurative relay stick and enable them to officially continue supporting the project.

The time where I could randomly program 5, 6, 7 hours at night without anybody caring are long over

We're very grateful for the 7 years of support and dedication, as you can see by the many replies and reactions here, but what I'm trying to say is that the demise of your time as a night-owl programmer (I totally just made this term up 😁 but apparently if you google it, some interesting results show up) does not necessarily need to be the demise of your project too. And it doesn't have to stop being "yours" either: in my mind you'd become the Team Leader - sharing project mission as well as vision and experience - and the other person would be the new Lead Developer.
I do not have hands-on experience of how GitHub actually works, in terms of forking and open-sourcing, so I don't know if what I just said can happen on its own without your guidance or not; but from a company-oriented perspective I feel that's what I would suggest for your project (or, indulging further into this suggestion, your product/brand); it doesn't have to die with you! I really hope this message finds you well. Best wishes, Francesco

Sorry for writing a whole book, I'll summarize the open points I'd love you to reply to:

client_secret: how can we generate a user token that authorizes your app, if we don't know that parameter
theoretically, instead of custom url schemes would a localhost redirect uri solve the auth code passing problem
have you considered bringing another person on board, to pick up from where you left, under your brand name

client_secret: how can we generate a user token that authorizes your app, if we don't know that parameter
Wdym by not knowing the parameter?

App worked great (when I used it a few times), All the best in your future.

Thank you so much for your work

Thanks @AlmightyLks and @Shark-93 ! , now i can continue using it

@stalk4wooder
Wot? What did I do?

@AlmightyLks for posting the link i didn't see first on shark's comment, thanks !

I got a new pc a few months ago but didn't need twitch leecher until now. I previously used your program back in 2017-2020 to archive VODs of streamers I was watching then. Thank you so much for your hard work and wish you nothing but the best on what you'll do next.


chore: upgrading emulators

Emulator upgrade is needed to avoid system test failures while developing query count in firestore.
Changes:

Upgrade version of emulator to latest one
Add .github/emulator/start-emulator.sh to paths of workflows which use emulator

Tests

I tested locally that the docker file exists and I can start a docker for all products.
Tested that the emulator tests are working locally for some products, others I do not have a local setup.

I am sort of blocked on the test failure with the PubSub emulator. The failure is not reproable in PROD and I am clueless with debugging into pubsub emulators.
@bshaffer should we skip PublishAndPullTest::testAckAndModAckContainFailedMsgs on emulators for now and revisit this later?

I believe this solves the issue that we encountered for PubSub?


Article should link to main x:Bind document

There are important details to how x:Bind works in the main article here; however, this article is not linked to in this document.
It'd be great if there was a 'Related Topics' section pointing to the main x:Bind document.
For instance the "Path to function" section could also link to https://docs.microsoft.com/en-us/windows/uwp/xaml-platform/x-bind-markup-extension#property-path and 'indexers' to https://docs.microsoft.com/en-us/windows/uwp/xaml-platform/x-bind-markup-extension#collections.
I'll try opening a PR.

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

ID: 9fdf7014-70bd-f562-046a-432f5f5caf11
Version Independent ID: f45e3d08-66e0-1b74-2c0d-595345e7f3b1
Content: Functions in x:Bind - UWP applications
Content Source: windows-apps-src/data-binding/function-bindings.md
Product: uwp
Technology: data-and-files
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Changes implemented here: https://github.com/MicrosoftDocs/windows-uwp/pull/2627


Getting the tag value

Hey Matthew,
Played around a bit with your action and managed to get it to work like this 🙂
You can see the action "succeeding" here: https://github.com/jmayntzhusen/umbraco-forms-rte/runs/713194505?check_suite_focus=true
(The Nuget pack failed as v2.0.0-buildtest51 is not a valid version, but the rest works)
Submitted this PR for a quick overview of changes, feel free to grab what you need and then close this 😁

Thanks for this one step closer :)


Permanently undock some panes?

Issue Report Checklist

[ x] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x ] Tried basic troubleshooting (if a bug/error)

[x ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[x ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
This is related to #10649
I am using Spyder 5.0.5. I used multiple monitors and undocked the var explorer pane and console pane on separated monitors. But when I close the var explorer and console pane or reboot, they are automatically re-docked to the main window. Are there options to permanently undock var explorer and console panel to separate monitors?

Hey @shuai-zhoum, thanks for reporting. You said:

But when I close the var explorer and console pane or reboot, they are automatically re-docked to the main window.

I took a look at the problem with rebo

@ccordoba12
if we don't dock them back in the interface, how are users supposed to displayed them again?
Thanks for your clarification. Yes, it makes sense to re-dock the "var explorer" and "console" (or maybe other panes) to the main window when closing those panes without closing the main window.
Just want to clarify that what I want is to undock "var explorer" and "console" (or maybe other panes) to my separate monitors so I have an entire monitor for the main window for coding, and keep this layout after my computer reboots (that is what I mean "permanently", i.e., every time I open Spyer, there is the main window for coding on one monior, and the un-docked "console" and "var explorer" on separete monitors).
Anyway, looking forward to the new version. Thanks.

Ok, thanks for the clarification. That's what I plan to implement in 5.2.0, i.e. the ability to restore undocked panes after Spyder starts again.


chore: bump ark-ff to 0.4.1

This PR bumps ark-ff to 0.4.0 and resolves all the breaking changes.

Bumping ark-ff to 0.4.0 seems to cause the wasm module generated under debug mode to use more than 50,000 locals in a single function, causing the wasm tests to fail. If we merge this, it's gonna have the unwanted side effect of forcing downstream applications to also run their wasm tests in release mode.
I'm putting this PR as draft until a better solution exists.

Just tried benchmarking a bit and it looks like the upgrade actually causes a performance drop of around 5-10% across the board... Not sure if we should still upgrade.

Whatever... let's just force every downstream application to test their wasm builds in release mode.


Programming Exercises: Wizard Mode for Creation

Builds on top of #5755.
Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
During a user study for my Master's thesis it became apparent that it's confusing for new users to create new lectures with all their contents because they don't know how everything works yet. This has been addressed in #5755. The same issues arise when it comes to programming exercise creation. Just being thrown into the editor can be confusing and overwhelming for new users.
Description
Therefore, analogously to the lecture creation, there is now a step-by-step wizard mode for programming exercises. This makes the whole process less overwhelming because you only see the controls step by step and some info is being hidden before you need to see it. Additionally, there is a summary text for the selected grading options.
It's possible to switch back and forth between the normal and the wizard mode at all times during the creation.
Steps for Testing

Prerequisites:

1 Instructor
1 course where the user can add new programming exercises

Log in to Artemis
Navigate to Course Management
Navigate to Exercises
Click "Generate New Programming Exercise"
Click the "Switch to guided mode" button
Follow the dialog to create the new exercise

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Screenshots

Codecov Report
Base: 75.53% // Head: 84.66% // Increases project coverage by +9.13% :tada:

Coverage data is based on head (f059da8) compared to base (7bd90ea).
Patch has no changes to coverable lines.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #5834      +/-   ##
=============================================
+ Coverage      75.53%   84.66%   +9.13%
- Complexity         0    12367   +12367
=============================================
  Files           1105      997     -108
  Lines          43707    39945    -3762
  Branches        8261     4274    -3987
=============================================
+ Hits           33012    33818     +806
+ Misses          5576     3772    -1804
+ Partials        5119     2355    -2764

Flag
Coverage Δ

client
?

server
84.66% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...learning-goal-form/learning-goal-form.component.ts

...se/manage/course-management-exercises.component.ts

...ad/manage/file-upload-exercise-update.component.ts

...eling/manage/modeling-exercise-update.component.ts

...ge/update/programming-exercise-update.component.ts

...ises/quiz/manage/quiz-exercise-detail.component.ts

...ge/text-exercise/text-exercise-update.component.ts

...ebapp/app/lecture/lecture-attachments.component.ts

...chment-unit-form/attachment-unit-form.component.ts

...te-exercise-unit/create-exercise-unit.component.ts

... and 2092 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Codacy Static Code Analysis fails. Check these issues:

@sebastiannikles could you please resolve the merge conflicts in the translation files and in the client tests? Thank you

please ask for at least 6 re-approvals then, creating programming exercises is a core functionality of Artemis and must work 100% correctly


[Icon Request] Support for .rake

[x] I'm sure this issue is not a duplicate
[x] I want to create an icon request
- Type: rake file
- Icon Name: ruby
- Extensions: .rake

From what I read .rake files are Ruby files but they contain task instructions. We already support rakefile associated with the Ruby icon, but I believe that this may cause confusion to the Ruby users.
How about we switch rakefile and .rake to

This is going to be supported by vscode ruby language id. Still though I believe we should override the icon with the rake icon.


keep it running

making donation through bountysource
thanks for correcting autolocation problems

Thank you for using the service!
(I am going through the ancient issues and close some of them :)


Tables should be inside expanding boxes

The current dashboard allows the user to collapse the boxes which gives a clean, minimalistic view.
@LawrenceHunt I'd like to do the same, with the loss curve visible by default and the tables collapsed (except the 'Label' table at the top which shows the winner)

Done!


Refine data download

[x] Currently, download button is broken

[x] Enhance with view-specific download

@mmontesanonyc Added current view, full table, and raw data. Feel free to change anything.


bubblesort.c

Added bubble sort In C language

Please Indent the code

https://codingblocks.com/ide/#/s/2632
This is how a code should be indented. Please take a look here @090max.
Thanks for contributions.
Happy HactoberFest
Keep Contributing.


Toolbar buttons 3

Extract class for view buttons and move code to history item button.

@himdel : care to review, please?

@martinpovolny will do.. but unless it's urgent, will do tomorrow, feeling a bit under the weather today..

@himdel : not urgent. Monday is a nice day for the review ;-) Thx!

Apart from the possible issue in build_toolbar_hide_button, LGTM!

@himdel : fixes, thx for the review!

@martinpovolny Prefer that the person creating a PR does not merge his/her own PR.  Next time, please ping @Fryguy, @matthewd or me.

@chessbyte : also should have @himdel explicitly confirm here, that all was checked once again to make it clear that all was checked. He confirmed F2F.


feat(getNetworth): Add JSDoc

Description
Adds JSDoc for output object

was bored and wanted to test new Copilot features so yeah..

Cool, but what's it do?