UndoManager Improvements (Firefox Freezing Fix)

Second attempt of #346.
This PR only address the failing test by avoiding passing options to scribe in the test file undo-manager.spec.js. Passing options in the tests is known to cause freezing, then a timeout, which end up failing the test. Now, the interval is not passed as an option. Instead, it is changed before starting the tests. It was successfully tested on Firefox 31 on Mac OS X (10.10).
Hopefully it will pass all other browsers/platforms this time around.
Edit: I've just tested it on Firefox 32, and I think there is still freezing issue. I'll try to identify the source of the issue. Actually not only aaalsaleh/scribe:master freezes, but also guardian/scribe:master freezes on Firefox 32 on Mac OS X (10.10)!

@rrees Firefox 31 problem is fixed, and it was due to the helpers as mentioned. With the last commit it can be considered fixed. However, Firefox 32 on Mac OS X 10.10 freezes with both guardian/scribe and aaalsaleh/scribe. I made fresh clones of both of them and verified that just now.
I tried the new version in #347 just to see, and both guardian/scribe and aaalsaleh/scribe went through successfully. To double check, I down graded back, and the same issue came back.
If this is a problem with the master branch, then I think it is a problem in the testing suite with this particular browser version. Can someone else confirm this manually for guardian/scribe and aaalsaleh/scribe with Firefox 32 (possibly on Mac OS X)?

@aaalsaleh I'll try and merge with master but I think we can live with disabling FF32 if that is the only problematic browser for now

@rress, technically it is most probably a problem with the tests suite, not the browser. The test freeze right from the beginning, and you can tell by the browser not even loading the local server URL to start the tests. I don't know what is the problem, but the tests run fine with #347. I've also made quick manual tests yesterday with both branches (before merge), and I didn't see any problems with FF32.

@rrees you merged both PR, but for some reason you kept the undo options (or was kept due to the separate two PRs?). Please remove the options, it is the cause of the freezing/failing that I fixed with this PR.

Yes, the order of the merges was to blame, I created a branch to make it a bit easier for me to try and keep the change in sync with master.


Use different base images for frauddetectionservice

Changes
This PR changes the base images used for building & running frauddetectionservice:

For the builder it uses the gradle image, so downloading gradle is not necessary anymore since it is part of the image. This improves the build time a little bit (for me from ~70s to 60s), especially when doing repeated builds
For the final image it uses a distroless java image, which reduces the overall size down to 271MB from 509MB.

Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[x] CHANGELOG.md updated to document new feature additions
[x] Appropriate documentation updates in the docs folder

Maybe to add this as a note: it looks like that python3.10 builds faster overall than python3.11 because of the availability of certain "wheels": if I go for 3.11 it starts to compile a bunch of things while for 3.10 it looks like they can easily be downloaded, maybe something that can also help with other python services.


cargo run --example whisper --release compile error!

rustc 1.71.1 (eb26296b5 2023-08-03)
error: failed to run custom build command for ring v0.16.20

new version:

I feel that this is more a general rust setup issue rather than something related to this crate (maybe something to do with some cross compilation setup). Maybe you can try building the ring crate separately and I would think that the issue would happen there too without candle being involved.

Closing this as it doesn't seem candle specific but feel free to re-open if you run into further issues / can provide more details.


Vulnerability roundup 102: bento4-1.5.1-629: 1 advisory [5.5]

search, files

[ ] CVE-2020-23912 CVSSv3=5.5 (nixos-20.09)

Scanned versions: nixos-20.09: 76ed24ceab9.
Cc @makefu

NixOS 20.09 is EOL, so this issue isn't actionable anymore.


crypto/x509: Add /etc/ssl/certs to certificate directories

This will allow certificate verification from system root CAs work for distros such as SLES10/SLES11 which have certificates stored there.
Originally asked about this in https://groups.google.com/forum/#!topic/golang-nuts/lEInOWNOVxk

It is OK to add /etc/ssl/certs to crypto/x509/root_unix.go.


item not updating

Hi! maybe i didnt understand how it works, but if i update an item in the compendium and a player has it, shouldnt it update itself automatically? Is there maybe a macro i need to press?

Yes it should, if the item is linked. Some fields won't update automatically, like "quantity" for instance.
Could you provide me some screenshots? What version of Foundry are you running?

Please try out v1.0.9

hi.
now its updating but is not calculating the amount of items in the world anymore (the number on the side in the compendium)

i even tried deleting the linked item and dragging a new item from the compendium to the character sheet, but no number is appearing.
the old numbers are still there.
Also, i opened a linked item in a character sheet and changed the quantity. It was a normal shield (that had a template in the compendium). The moment i changed the quantity from 1 to 2, it transformed the item from Shield to Armor of the Gnoll Lord.

i had a waterskin linked, i changed the quantity to two and it became a:

i reverted to 1.08 and this issue is gone. it shows the numbers again. (but obviously it doesnt update linked items)

The item listing number is a visual bug, should be resolved in v1.0.10

Please try out v1.0.10

perfect!


Puppeteer Does Not Pass CA File Configs when Downloading Chromium

I am attempting to download Chromium as part of the puppeteer install process with PUPPETEER_DOWNLOAD_HOST set to my organization's Artifactory server. The server has a self-signed certificate. After setting  NODE_EXTRA_CA_CERTS to a valid PEM file I was able to install pupeteer, but Chromium failed. So, I consulted the following articles:
https://github.com/puppeteer/puppeteer/issues/1994
https://stackoverflow.com/questions/9626990/receiving-error-error-ssl-error-self-signed-cert-in-chain-while-using-npm/35510347#35510347
https://docs.microsoft.com/en-us/windows-hardware/drivers/install/trusted-publishers-certificate-store
After following all the suggestions in the issue threads above, I continue to see the issue of Chromium failing to download because of the self-signed certificate.
Perhaps the CA file configurations are not respected when attempting to download Chromium.
Steps to reproduce
Tell us about your environment:

Puppeteer version: 2.0.0
Platform / OS version: Win 10 Version 1809 (OS Build 17763.973)
URLs (if applicable):
Node.js version: v12.14.1

What steps will reproduce the problem?
Please include code that reproduces the issue.

Set PUPPETEER_DOWNLOAD_HOST
Set NODE_EXTRA_CA_CERTS and/or CAFILE to a valid CER or PEM file
Run NPM install

What is the expected result?
Installation of Puppeteer and appropriate Chromium revision for Puppeteer version.
What happens instead?
The following error:

<EMAIL_ADDRESS>install [Path to project folder]\node_modules\puppeteer
node install.js

ERROR: Failed to download Chromium r706915! Set "PUPPETEER_SKIP_CHROMIUM_DOWNLOAD" env variable to skip download.
Error: self signed certificate in certificate chain
at TLSSocket.onConnectSecure (_tls_wrap.js:1321:34)
at TLSSocket.emit (events.js:223:5)
at TLSSocket._finishInit (_tls_wrap.js:794:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:608:12)
-- ASYNC --
at BrowserFetcher. ([Path to project folder]\node_modules\puppeteer\lib\helper.js:111:15)
at Object. ([Path to project folder]\node_modules\puppeteer\install.js:62:16)
at Module._compile (internal/modules/cjs/loader.js:955:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
at Module.load (internal/modules/cjs/loader.js:811:32)
at Function.Module._load (internal/modules/cjs/loader.js:723:14)
at Function.Module.runMain (internal/modules/cjs/loader.js:1043:10)
at internal/main/run_main_module.js:17:11 {
code: 'SELF_SIGNED_CERT_IN_CHAIN'
}
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node install.js
npm ERR! Exit status 1

I ran into this issue to and it comes down to the fact that the npm process invokes a sub-process of node install.js and doesn't share the same environment. We ended up just setting PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true as we didn't actually need that dependency for our use case. Now running into this again on another project where we actually need the Chromium.
We need to bubble the ENV vars and/or config file into this new node sub-process somehow.

I do see that in https://github.com/puppeteer/puppeteer/issues/1994#issuecomment-367153013 and https://stackoverflow.com/questions/9626990/receiving-error-error-ssl-error-self-signed-cert-in-chain-while-using-npm/35510347#35510347 it says that we have to add the below to /etc/environment:

NODE_EXTRA_CA_CERTS=/etc/pki/ca-trust/source/anchors/yourCerts.pem

I have a similar issue. Using yarn and Artifactory. I don't have any workaround so far.

FYI NODE_EXTRA_CA_CERTS seems to work for me:
NODE_EXTRA_CA_CERTS=/etc/my-cert.pem npm i puppeteer
but without NODE_EXTRA_CA_CERTS, the following does NOT work:
npm config set cafile /etc/my-cert.pem
The above resulted in the same error as reported.


Turn STAC Transaction API endpoints on again

We disabled the STAC API Transaction extension because we currently did not have a mechanism for authorization.
What are the steps for turning it back on? Do we need to wait for the API gateway (Tyk) setup to be ready, so we can secure those endpoints with Tyk? Or can we set up some simple intermediate auth mechanism?
Related ticket:

https://github.com/developmentseed/eoapi-k8s/issues/33

Acceptance criteria

[ ] The eoAPI deployment has working STAC Transaction endpoints that are secured with some (basic) form of auth

It might be rrelated to https://github.com/stac-utils/stac-fastapi-pgstac/issues/166/

Is there already a ticket for the Tyk configuration to protect these endpoints?
In the meantime I could configure the ingress controller to protect the transactions endpoint with some BasicAuth. What do you think?

We had this placeholder ticket last quarter:

https://github.com/EOEPCA/data-access/issues/81

We probably need a fresh plan for it from @jankovicgd.
Any interim simple protection of the Transaction API endpoints that we can remove later when the API Gateway is ready would be welcome.
We currently do not have our own runtimes in eoapi-k8s. I am not convinced we need them, longer-term. Our needs should not really deviate from what the runtimes included with the individual libraries provide.

I’ve looked into this further, and it turns out that adding basic authentication is more complex than initially anticipated. The STAC API Transaction Extension uses HTTP methods like POST, PUT, and DELETE, and HTTP method-based routing is only supported if we use Traefik as the ingress controller, rather than Nginx.
After discussing with @alukach (🙏), the following approach seems to be the most recommended and should provide a robust, extensible solution for the EOEPCA project—eliminating the need for future changes:

Build a custom stac-fastapi runtime for EOEPCA that integrates eoapi-auth-utils and leverages EOEPCA’s Keycloak for authentication.

The main tradeoff here is that we’ll need to set up a pipeline for building and deploying this custom runtime. However, this approach offers greater flexibility and scalability, allowing us to easily add additional functionality as needed in the future.

Build a custom stac-fastapi runtime for EOEPCA

If it was only for auth, I would prefer that we stick to the default stac-fastapi-pgstac image and solve auth via ingress. But we will also need things like metrics (probably via OpenTelemetry), multidimensional dataset (Xarray) support, etc. and the default runtimes do not include these, so let us take the step now to set up our own runtimes.
Let us time-box this. One sprint and the new runtimes are up and include eoapi-auth-utils?

I have published a chart where the control is implemented on database user level, taking advantage of a read user and ingest user that exist in pgstac by default. Then each of the users is connected to a different k8s deployment which is connected to a different k8s service and in the end a different k8s ingress. https://gitlab.eox.at/vs/stacapi-helm/
The idea with this approach is that there is a stac-read.example.com endpoint where read clients can be attached and whenever any write is attempted it fails at the database level, and a stac-ingest.example.com endpoint where every operation is protected by basic auth but here the read/write clients could be attached.
There are some drawbacks to this approach and may not be a good fit with the system.

if a more finegrained access is needed this doesn't work
ensuring clients are properly routed
increased infrastructure
database bottleneck
monitoring 2 deployments
updates of 2 systems

My bad, completely missed the transaction extension. Back to the drawing board I guess

This is an interesting approach, @jankovicgd! Thanks for sharing! For EOEPCA, I saw Basic Auth as a temporary solution, with the intention of eventually integrating it into a more robust authentication system. By leveraging eoapi-auth-utils in conjunction with EOEPCA's Keycloak, we're already taking a step toward a proper solution for the project


TableLoader: Option to load pointing information

It would be useful to have an option for the TableLoader to add (interpolated) pointing informations to the table.

Or at least have it constuct an interpolation function that the user can use to add pointing columns.
That is a more general thing: we need a standard way to do that:  a simple function or Component in ctapipe that is a  PointingInterpolator (it's also needed eventwise during reconstruction as well as for computing the table columns later).   I'll open a new PR for that.

closed since covered by #1902


Print webview with position:absolute

Hi,
The window doesn't want to print when the webview is with position:absolute.
I've done a test case available at https://app.box.com/electron-1681.
By default, Command+P won't print but if you comment the line 13 position: absolute;, it will print!
It has been test on osx with 0.22.3 and 0.26.0.
Thx
Daiyam

The layout of printed web page is calculated on the Chromium side, so there is nothing we can do to fix it on Electron side. I suggest transforming your app into a Chrome extension and report this bug to Chromium's issues.


Add message begging for help lol 🐢

jk I ask politely 😤

@conradludgate how'd you think this sounds?

Nice hacktoberfest spam... /s


Doc/link to spark functions

Some issues found trying to document (via link) these functions:

org.apache.spark.sql.functions.locate
org.apache.spark.sql.functions.split
org.apache.spark.sql.functions.to_utc_timestamp
org.apache.spark.sql.functions.from_utc_timestamp

Related to issue #135


[Bug]: The rendering indicator is not always visible

Attach (recommended) or Link to PDF file
This is not specific to any PDF file.
Web browser and its version
Firefox 133
Operating system and its version
Not relevant
PDF.js version
master (308ca2a16f2aa982ffc4bc825ff076307e01cc25)
Is the bug present in the latest PDF.js version?
Yes
Is a browser extension
No
Steps to reproduce the problem

Open https://www.gtt.to.it/cms/risorse/urbana/mappa/mapparete.pdf
Set the zoom to 200%

What is the expected behavior?
While PDF.js is rendering pages, it shows a little loading spinner, that disappears once the page is done rendering.
The steps above should show a loading spinner.
What went wrong?
The loading spinner is added to the document, however it's added to the center of the page. The center of the page is outside of the browser viewport, so the spinner is not visible.
Link to a viewer
No response
Additional context
I noticed this while working on https://github.com/mozilla/pdf.js/pull/19128. The spinner should be in the middle of the visibleArea computed in that PR.

The spinner in the pageNumber-input is however always visible, which make this a lot less problematical (and possible even a non-issue).

Not on Firefox for Android, since the toolbar is hidden there :)


Tutorial's link

Hi,
The tutorial's link doesn't work

I have a backup copy here. The original was either deleted or made private some time ago.


Question about codegen for arraylit node

Describe the bug
Version
swc_ecma_ast = "0.78.0"
swc_ecma_codegen = "0.107.0"
swc_ecma_parser = "0.104.0"
swc_common = "0.18.1"

Question Description
When I attempt to parse [1, [2]] to expr and then generate js code, I get the following error:
position 5 dose not resolve to a source location

I noticed that it is caused by the need to generate commas, but I'm a bit confused how to fix it.
Screenshots:

Input code
use swc_ecma_ast as ast;
use swc_ecma_codegen::{Node, Emitter, text_writer::JsWriter};
use swc_ecma_parser::{lexer::Lexer, Parser, StringInput};
use swc_common::{SourceMap, FileName, sync::Lrc};

fn parse_value(value: &str) -> ast::Expr {
    let cm: Lrc<SourceMap> = Default::default();
    let fm = cm.new_source_file(FileName::Anon, value.to_string());
    let lexer = Lexer::new(
        Default::default(),
        Default::default(),
        StringInput::from(&*fm),
        None,
    );

    match Parser::new_from(lexer).parse_expr() {
        Ok(expr) => *expr,
        Err(_) => panic!("{} had something wrong.", value),
    }
}

fn to_value(expr: &ast::Expr) -> String {
    let mut buf = vec![];
    let mut emitter = Emitter {
        cfg: Default::default(),
        cm: Default::default(),
        comments: None,
        wr: Box::new(JsWriter::new(Default::default(), "", &mut buf, None)),
    };
    // dbg!(expr);
    if expr.emit_with(&mut emitter).is_err() {
        panic!("{:?} had something wrong.", expr);
    }
    String::from_utf8(buf).unwrap()
}

#[test]
fn it_should_works() {
    let expr = parse_value("[1, [2]]");
    let codegen_expr = to_value(&expr);
    assert_eq!(codegen_expr, "[1, [2]]");
}

Config
No response
Playground link
Download this repo and exec cargo test.
Expected behavior
assert_eq!(codegen_expr, "[1, [2]]"); should be success.
Actual behavior
No response
Version
Look up the top of this issue.
Additional context
No response

This is not a bug

There are examples in swc repo
I'm currently AFK

@kdy1  I had noticed this discussions https://github.com/swc-project/swc/discussions/2300, thanks for your awesome work.


In the reports page the number "2" title is not matching with the document.

please follow the document.

finished


Add Edit this page link

Since this content is supposed to have a community feel, I've added an 'edit this page' link to the top right of each page.

I think that makes editing slightly easier than in the actual wiki, where the edit link is just a 'fork this repo' link at the very bottom of the page.

Oh dear, #131 has actually broken the surge preview entirely.


Documentation for "lob *" columns output

The following output columns "lob logical reads", "lob physical reads", "lob read-ahead reads" will also have output for the columnstore indexes pages regardless of the data type.

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

ID: 00651e98-1d8e-4c17-5f8f-ea50136128eb
Version Independent ID: 158bf686-4304-aa99-c2d2-c7749f87e913
Content: SET STATISTICS IO (Transact-SQL) - SQL Server
Content Source: docs/t-sql/statements/set-statistics-io-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@magasvs - thanks for the submission. I'm checking a few things out and will get back to you.

@bluefooted - would you mind looking into this issue. I can help update the docs.

Thank you @magasvs for raising this issue! It sheds light on this subject. We are updating the article.


Snips dataset

Add reader and (intent and NER) iterators for the https://github.com/snipsco/nlu-benchmark dataset

@dilyararimovna Sorry, I don't understand what you mean here.

About what @dilyararimovna said: there are configs for snips in classifiers directory.
They use basic_classification_reader but could use the new one

@yoptar @dilyararimovna I see. For this, the original snips dataset needs to be uploaded to files.deeppavlov.ai first, I think.

@alexeyr is http://files.deeppavlov.ai/datasets/snips.tar.gz ok?

@yoptar Yes, though *_metrics.json and Snips_metrics_full.json files can be removed. I'll update the request with configs (and automatic download) later then.

@yoptar Added downloading and updated configs.

@yoptar Done.

@yoptar Note that I rebased to combine the commits, and this apparently happened between you approving the changes and merging.


Release notes updates - SFN3.5

Update release notes for 3.5
Linked with #59

Updated in c66c91c0e8207a6852daeb7941e2a6ab78fb1ec7


feat(udf): support always_retry_on_network_error config for udf functions

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Closes https://github.com/risingwavelabs/risingwave/issues/15137.
The main issue is that when UDF call fails, the function call returns null instead.
If the function call updates a key, e.g. for join or group by, it is possible for the state to be inconsistent.
As such we provide the option to define a udf function which will always retry on network errors, since we cannot tolerate the UDF failing non-deterministically in such cases.
Here are the changes:

Introduce a new CreateFunctionWithOptions.
Since it's an AST object, we provide option for each of its fields. This is to make it compatible to Display (otherwise no WITH options, or only some options set, we still display all CreateFunctionWithOptions. See tests for examples). This also preserves the AST structure.
Flatten the CreateFunctionWithOptions into parameters to Function, UserDefinedFunction definitions.
Initialize the udf expr with always_retry_on_network_error.
Add tests for display of CreateFunctionWithOptions.
Add tests to test MV contents when there's no retry, vs when there's retry.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.
Users can now do:
CREATE FUNCTION ... WITH ( always_retry_on_network_error = true );

This means network errors will always be retried for function calls of that function.
Note that the entire stream graph will be blocked when UDF server goes offline, until it is back online.

qq: What's the relationship with #15171?

This PR makes it configurable as an with_option. In the other PR, it just hardcodes it to always retry.

Can @xxchan and @wangrunji0408 PTAL at this? I want to include this in 1.7. Since it's a breaking change, so users can upgrade their UDF functions ASAP, since they have to recreate all dependent MVs.

Can review the code first, there's some separate failure due to some migration issues.


Carousel: fix init via JS, jQueryInterface and dispose method

Hi @iprzybysz
What are the problems you've found? Right now I'm using a Carousel in Svelte, and it works the first time I open the page, but then when coming back from another page, it doesn't start showing images in big, but if I click the thumbnail images, it displays them.


SA1011 - Issue with nullable arrays

Using <PackageReference Include="StyleCop.Analyzers" Version="1.1.118">, with a .Net Core 3.1 project with nullables enabled (<Nullable>enable</Nullable>).
The following code:
byte[]? byteData = null;

generates the following:

Error	SA1011	Closing square bracket should be followed by a space.

I think this is a false positive.

Duplicate of #2927


Any file opened after neovim starts with vim-startify starts does not get a swap file

Hello! When starting up neovim using vim-startify and opening any file (either using a shortcut on the vim-startify screen or using :e any_file.txt, the file does not get a swapfile (i.e. running :swapname yields "No swap file"). This issue was originally reported to SpaceVim/SpaceVim#2966.
Environment
Ubuntu 19.04
neovim 0.3.4 (installed from the Ubuntu repositories)
vimrc:
if &compatible
  set nocompatible
endif
set runtimepath+=~/.cache/dein/repos/github.com/Shougo/dein.vim
if dein#load_state('~/.cache/dein')
  call dein#begin('~/.cache/dein')
  call dein#add('~/.cache/dein/repos/github.com/Shougo/dein.vim')
  call dein#add('mhinz/vim-startify')
  call dein#end()
  call dein#save_state()
endif
filetype plugin indent on
syntax enable

Steps to reproduce

open neovim with nvim (note: it's important to not give a filename)
edit a file with :e any_file.txt
check for swapfile with :swapname

Observed (incorrect) behavior
Status line says "No swap file"
Expected behavor
Status line says "/home/motevets/.local/share/nvim/swap//%tmp%directory%any_file.txt.swp" (or something similar)

I remember there were some swap-related issues around that Nvim version that were introduced and later fixed by backporting a few patches from Vim. Please try upgrading your Nvim (v0.3.8 is the current release).
For double-checking you could test if you can reproduce this with Vim. If not, then you know it's not a problem with the plugin but with that particular Nvim version.

You're right, upgrading to vim 0.3.8 fixed the issue when using vim-startify by itself. Interestingly however, it did not fix the issue when running vim-startify with SpaceVim (and disabling vim-startify in SpaceVim "fixes" the issue.) Feel free to close this issue, but if you have time, it would be wonderful if you could check out SpaceVim/SpaceVim#2966 to see if you can be of assistance.


[BUG] Bloxstrap Menu still takes me to Roblox Player

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Bloxstrap Menu, is one of the currently biggest problems right now. Here's why.
First up, when I open up Bloxstrap Menu, it starts loading Roblox Player and takes me there, instead of the ACTUAL Menu.
I used the search bar for Windows. I can't believe the only time I had access to the Menu was when I first installed Bloxstrap.
Please just fix this issue for my sake. It's starting to get annoying and out of hand.
Here's even MORE proof. When I try to find a way to uninstall, nothing works. I reinstall Bloxstrap, yet Menu takes me to Roblox Player.
Still don't believe me? Here's actual video evidence
Also this is my real computer.
https://github.com/pizzaboxer/bloxstrap/assets/169791544/eac8099b-ede2-4186-bd73-6b35d5e81a23

Send the latest log file from %localappdata%\Bloxstrap\Logs please

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen


Generic changes

Some changes for use on switchroot

Makes lockdown a bit more generic so it isn't so tied to the reswitched setup.
First attempt at reaction pinning cog. Needs a bit more work to integrate it with a pinboard and an unpin command.

I don't get why some cogs hardcode these roles anyways

Looks good to me. I'd rather have overflowing pins on a gist etc rather than a pinboard channel, though.
and @tumGER, please don't merge config-breaking PRs without consulting me first. Our usual workflow on RS is getting the changes added to config (which can take a day or two, people are busy, it happens), then pushing config-breaking changes.


Noor Grewal Review Ticket Week 2

Crossover Grade 2.7/3.0

Python List in Jupyter Notebooks
Reversed Python List
Extremely Fun Calculator with Python
Other Productive Blogging

Updated Vocab Table
Remote Theme Change
Noor mode after looking at Vunsh's Darkmode blog

In the next week, I plan to manage my time better so that I am able to resolve issues and work with my pair without a time crunch. I was proud of being able to reverse the lists/dictionaries I made, creating a calculator to learn more about actually coding with Python, and of being able to change the theme of my blog with color coding.

2.7+/3
Great collaborative and creative use of lists, dictionaries, and color/dark mode personalization. The Fun Calculator shows understanding of basic python coding, as well as reversed python list.


Local Policy Editor Stops Working After Applying Configuration

Hey there Dave,
I'm attempting to use this module for configuring local policies and running into the following issue after I apply a config.  The error I have is as follows:

I'm assuming it's because I did something horrible.  I'm just not quite sure syntactically what I did.

        cAdministrativeTemplateSetting NoWindowsHotKey
        {
            KeyValueName = '\Software\Microsoft\Windows\CurrentVersion\Policies\Explorer\NoWinKeys'
            PolicyType = 'User'
            Data = '1'
            Ensure = 'Present'
            Type = 'Dword'
        }

Pulling from the spreadsheet in another logged issue, here's the info that I was able to find on this specific policy.

1703
windowsexplorer.admx
Turn off Windows Key hotkeys
User
Windows Components\File Explorer
HKCU\Software\Microsoft\Windows\CurrentVersion\Policies\Explorer!NoWinKeys
At least Windows Server 2003
Turn off Windows Key hotkeys.  Keyboards with a Windows key provide users   with shortcuts to common shell features. For example, pressing the keyboard   sequence Windows+R opens the Run dialog box; pressing Windows+E starts File   Explorer. By using this setting, you can disable these Windows Key   hotkeys.  If you enable this setting,   the Windows Key hotkeys are unavailable.    If you disable or do not configure this setting, the Windows Key   hotkeys are available.

Halp! :P

It's probably that leading backslash on your KeyValueName.  Try setting Ensure to Absent and see if the problem goes away, then apply the configuration again without the slash.
If that works, I'll update the code to strip any leading slashes since they're not helpful anyway.

I'll give it a shot and let you know the result.  Thanks!

It was definitely the leading backslash that did it.  Working great now.  Thanks man!

Released 3.0.1 with the fix; new version is on PSGallery.


Deploy a development instance demonstrating the new WIT area

### Tasks
- [x] create feat/wit branch
- [x] Build migrations that create the new area
- [x] Implement the reassignments at https://mailarchive.ietf.org/arch/msg/ietf/qK_TwOiniQWxonzhkE5AQeK45mg/ in the migrations
- [x] build the new witarea area group
- [x] Deploy branch to dev.ietf.org
- [x] add grouphistory and groupevents
- [x] change the color of WIT in the schedule editor to match tsv
- [ ] leave tsvarea in tsv and close it
- [ ] make Francesca and Zahed chairs of witarea (coinflip for who is responsible AD)

We should leave GroupEvent objects noting the moves.

The rest of this moved to manual action by the secretariat


FreeRDP does not allow to cancel a running connection attempt

Currently there is no (clean) way to abort a running tcp_connect.
The timeout of the connection attempt must pass before a connection can be aborted.
Where does it matter:
Connection attempts with mobile clients (android, ios, ...) to hosts currently down can not be cleanly aborted.
Source of the issue:
freerdp_uds_connect and freerdp_tcp_connect use connect on blocking sockets.
A possible solution:

Use sockets in non blocking mode and listen to a appropriate event to abort, if desired.

Implemented with the freerdp_abort_connect


"NOASSERTION" license on License Rankings

https://innovationgraph.github.com/global-metrics/licenses
I think this should be removed or changed to something more obvious like "Unknown". Because it's capitalized I thought "NOASSERTION" might be something about not asserting some copyrights or something and googling "noassertion license" didn't produce immediately obvious results.

You're right, it's a term that only people very involved in open source licensing details would recognize, apologies for missing that (because I have been one of those people at times).
We'll change the display in the chart or at least add on-page documentation shortly. 💭

Added an explanation to the datasheet. @khxu will make a corresponding update to the website to close this out. Thanks again for reporting @verhovsky!

Fixed with nthe latest quarterly data release https://github.blog/2024-04-09-explore-the-seasons-of-software-development-with-four-full-years-of-data/#noassertion-changed-to-other-in-the-licenses-bump-charts


Switch back to hosted version of package:gap

It was switched to a custom version in https://github.com/gskinnerTeam/flutter-wonderous-app/commit/e30e2ebf48b99ef3f0324153205d5ca1c5467c07 due to an issue with the pre-release channels. This has been fixed in the latest package:gap release: https://pub.dev/packages/gap/changelog#301

Actually already made this change this morning before seeing this, closing this as redundant. Thanks!


Add KDNET well known BCD elements

@DaveEM Thanks for this PR!
I've cherry-picked the latest commit directly into master (skipping the merge commit) and pushed an updated version to NuGet: https://www.nuget.org/packages/Quamotion.DiscUtils/1.0.3-r118

Thanks @qmfrederik. What is the procedure you use for building the NuGet package?  I'd like to reproduce it locally and update it to include PDBs.

@DaveEM The packages are built via AppVeyor. The build definition is in appveyor.yml.
The code is built via project.json and not via the Visual Studio solution (something I hope we can fix once VS2017 RTMs). So I guess you'd need to update the project.json file.


[Suggestion] Make fertilizer work in dispensers

It's just unexpected that fertilizer works the same as bone meal in everything except for its use in dispensers. I understand that we already have the Plant Fertilizer, though, so this may not be worth the effort.

For a bit more context as to why this might be of interest: I'm making a personal modpack where I'm disabling all range addons. This basically makes the Plant Fertilizer block into a slightly more convenient dispenser, and so I was seriously considering removing the Plant Fertilizer until I realized that fertilizer couldn't be used in vanilla dispensers.


Feature arc42 02constraints

Added constraints to the docs.

The following should be taken into consideration: https://github.com/Arquisoft/viade_en2b/pull/12/files#r380154044
Apart from that, I would add an organizational constraint about the budget available (that is, none).

~The following should be taken into consideration: https://github.com/Arquisoft/viade_en2b/pull/12/files#r380154044~ Solved!
Apart from that, I would add an organizational constraint about the budget available (that is, none).

I think that is a good idea to take into account that type of constraint into account as it could be an important factor on the development process.

Added a budget constraint following the suggestions of @UO264345 and @uo264207.


Python deprecated? Issue after updating to FD22 22.00.005

After updating to FD 22.00.005 python library example stopped working.
No error logs from python.
Example python scripts launch FD but nothing happens after that.

@Marco-Pellegrino
I'm using the script example from:
https://github.com/strusoft/femdesign-api/tree/master/FemDesign.Examples/Python

Python script launches FD 22.00.005, no error log.
Script keeps on running indefinitely.

I'm using Python.NET.
I have used FD Python API before and I had it working with FD 22.00.004 a couple of weeks ago.
I'm just guessing that it has with the update to 22.00.005(changes in core.dll?), but I'll reinstall everything and try again when I have the time.
When are the .dll files going to be updated for FD 23?

I'll try to test the script and come back to you as soon as possible.
Are you also updating the .dll or just using the one in the script folder?
Do you have special characters in your file path?
.dll for FEM-Design 23 are already available:)
Have a look at the latest release.

@twangrt
Just tried the script with FEM-Design 23 and the latest .dll and it does work (you find them in femdesign.csharp)
https://github.com/strusoft/femdesign-api/releases
I will update the .dll in the python example folder at the end of march.
Meanwhile, have a look at the attached file
pythonnet.zip

Thank you. Attached files worked for FD23.
So new compiled cores are attached assets in releases and is not updated regularly in the repo?

@twangrt
I will need to update the .dll in the python example. It was my bad
I have just forgot to do it as we are not really promoting the use or PythonNet :)
I would like to know more on how you actually use it

No worries.
Ive done in the past some calculations with construction stages and support displacements to determine the preload on a existing structure.  But that was to much work and would be quicker to do by hand.
Now I'm thinking of using it to copy loads and loadcases. But i used a regular python script and the in-built export/import loads instead.


Only several questions are shown when user printed the exam transcript as pdf

One solution to solve this problem is users need to scale down the text. But that was annoying

Hi @anggalol . I experienced the same problem, using Windows 10, and Chrome browser.
The following two PDF are from a very simple exam: the first is at normal (100%) scale, and the second is 70% scale.
X-t22-1 - preview - Numbas at mathcentre.ac.uk-100percent.pdf
X-t22-1 - preview - Numbas at mathcentre.ac.uk-70percent.pdf
In larger (typical) exams, I have to use about 50% of the normal scale, or even less.
As a reference: I've an stand-alone exam, generated by march, 2nd. and it works fine. But if I test-run the same exam in the current Numbas editor, then the problem that you are reporting, appears again.
My current "solution" is to divide my normal exam in several one-question exams. But I hope we can find a solution to this problem, especially because I'm currently using the stand-alone exam mode.
Thank you in advance to any member of the development team, for your support concerning this issue.
Best regards,
Enrique Comer

It looks like this was a result of a recent change where I noticed that 'invisible' elements were still affecting the page layout.
I've made another change, and checked that everything that should be shown in print view, is shown. Thanks for reporting this!

Hello Christian:
Thank you very much for the good news about this issue.I did a test with my exams, and now the PDF preview is working fine.
Congratulations for all your great work on the Numbas editor.
Best regards,Enrique Comer
On Tuesday, April 5, 2022, 01:06:36 AM PDT, Christian Lawson-Perfect<EMAIL_ADDRESS>wrote:

It looks like this was a result of a recent change where I noticed that 'invisible' elements were still affecting the page layout.
I've made another change, and checked that everything that should be shown in print view, is shown. Thanks for reporting this!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Fixed spacing bug with article aside and headings.

@mikemai2awesome could you update this PR to point to the develop branch please? Once you do that this should be good to merge in

Travis automatic deployment: https://92-pr-boltdesignsystem.surge.sh


Implement get_ipv6_neighbors_table() getter

Merge get_ipv6_neighbors_table to develop

Coverage increased (+0.09%) to 95.462% when pulling 945e59bcd029748931fc22464f165be74d3eb0c6 on get_ipv6_neighbors_table into 195c924d5c318ecc4d9eba9f2d2cef1fc8f6b537 on develop.


Animation prop cancels other props

Hi,
I am in Next JS.
I have a component like this:
      <YStack
        {...{
          borderRadius: 3,
          backgroundColor: 'white',
          hoverStyle: { backgroundColor: 'var(--light-grey)' },
          animateOnly: ['background-color'],
          tag: 'button',
          borderColor: 'black',
          borderWidth: 1,
          borderStyle: 'solid',
          animation: 'fast',
        }}
      >

It outputs this:
<div class=" is_Stack  _bc-0hover-rytrrm _display-flex _bc-d0t1q780h6 _btlr-3px _btrr-3px _bbrr-3px _bblr-3px _minHeight-0px _flexShrink-0 _ai-stretch _transition-d0t1n15dys _flexDirection-column _minWidth-0px _minHeight-0px _position-relative _boxSizing-border-box _flexBasis-auto _display-flex _flexShrink-0  is_YStack">

It keeps only border radius and background color props. The transition works. The button tag is ignored.

I created animations like this
import { createAnimations } from '@tamagui/animations-css'

export const animations = createAnimations({
  fast: 'ease 0.3s',
})

I use the 1.9.0 version of all tamagui packages.

I tested this  a few different ways and couldn't replicate, though the tag likely is hard to fix with RNW not allowing it...

Can you add a repro and I'll re-open

Hi @natew,
Please find a repro here: https://github.com/SofianeDjellouli/animations-bug-repro. It is likely related to the config, as it was working with the original config of the template.
Thanks for the great library.


Network architecture keyword argument consistency

This is a minor issue but was a bit confusing when first encountered. Should be an easy fix.
In common.policies.MlpPolicy the network-architecture keyword argument is net_arch, whereas it is layers in sac.MlpPolicy.

Hello,
This is in fact an inconsistency that is fixed in SB3. The original reason for it was that sac.MlpPolicy is quite different from the common policies (you cannot pass dict(vf=[], pi=[]) but only a list).
I would appreciate a PR that solves that issue ;) (but that keep backward compatibility)

Fixed in SB3: https://github.com/DLR-RM/stable-baselines3


K8SPG-435 maximum size of tmp data can be 1G

vmagent stores tmp data in tmp dir and maximum size can be
1G. We need to increase the size of this dir.

CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?


How to use it?

How to use it?
Is there any usage summary like this, https://github.com/go-easygen/wireframe#-wireframe?
How do you config & start it? Thx.

Thanks!
I think one thing is missing, at github.com/LeKovr/teleproxy,
ln -s cmd/teleproxy/teleproxy

In order to use make run

What I'm doing wrong?
$ cd $GOPATH/src/github.com/LeKovr/teleproxy
$ make run
[ -d .git ] && GH=`git rev-parse HEAD` || GH=nogit ; \
  GOOS= GOARCH= go build -v -o teleproxy -ldflags \
  "-X main.Build= -X main.Commit=$GH" ./cmd/teleproxy
./teleproxy --log_level debug --group $GROUP --token $TOKEN --command ./commands.sh --template messages.tmpl
[ debug ] 2018/05/13 00:23:18 main.go:63: teleproxy v 0.4. Telegram proxy bot
...

Umm..., is your above the output before fix cddd9f6 or after?
After the fix to this issue, #6, this is the command I'm seeing:
$ make run -n
./teleproxy --log_level debug --group $GROUP --token $TOKEN --template $TEMPLATE --command ./commands.sh --template messages.tmpl

that doesn't look right, does it? (two --template last being the Russion version)

Oh, one more thing that's a typo
[ error ] 2018/05/12 22:13:19 main.go:131: Template load error: open messages.en.html: no such file or directory
I.e., the make default should be messages.en.tmpl instead of messages.en.html

Back to two --template, I thought it isn't right so I started from bash myself using:
./teleproxy --log_level debug --group $GROUP --token $TOKEN --template $TEMPLATE --command ./commands.sh

with TEMPLATE=messages.en.tmpl
however, during execution, I'm seeing:
[  warn ] 2018/05/12 22:23:13 app.go:46: template info exec error: template: no template "messages.tmpl" associated with template "messages.tmpl"
[  warn ] 2018/05/12 22:23:13 app.go:46: template message exec error: template: no template "messages.tmpl" associated with template "messages.tmpl"

that doesn't look right, does it?

Offer about ln -s cmd/teleproxy/teleproxy still needs explain. Others was fixed in e965c6f and 420f3de

Offer about ln -s cmd/teleproxy/teleproxy still needs explain

I built myself teleproxy, with go, thus teleproxy is under cmd/teleproxy as cmd/teleproxy/teleproxy, not under github.com/LeKovr/teleproxy. Thus I thought ln is necessary, and I didn't look closely how the build was done by make to build it under github.com/LeKovr/teleproxy.
Hope clearer this time.
Thanks to other fixes!


Move "members" key to end of "workspace" table

While not specified in the style guide, it feels clear enough that the "members" key should be placed at the end of the "workspace" table. The "members" list can be long, so the end of its section feels like the only natural place for it.

I don't think this is non-controversial. e.g., cargo's Cargo.toml doesn't satisfy this.

I stand corrected! I will close this PR.


Basic math operations

Write a script that asks a user for two numbers.
Print out the results for each math operation with these numbers.
E.g.
>>> enter number a:
10
>>> enter number b:
5
>>> a + b = 15
>>> a - b = 5
>>> a * b = 50
>>> a / b = 2.0
>>> a // b = 2

Hi @shorodilov I would like to work on this issue. Please Assign it to me. Thankyou

Hi @tarun212001.
The overall development process is described in comments to #87.
I don't thinks about the directory to place the entire scripts for now.
Just create a module within src/calc.
Note, this ticket to related to the beginners' challenges, so no functions - just a simple script code.
It's challenging to test the entire script modules. So, you may skip this step. Just let me know, or open an issue yourself to apply tests for your code in the future.


test | Fang Zhang's Blog

https://fang-zhang.github.io/2020/10/22/test/
test | Fang Zhang's Blog

1234

The arrow of "Full Screen" doesn't work.
Please remove this arrow, and recovery the upper-right square "Full Screen".

Please move the content in the Home page to the About page.

Fix the review from mobile phone.

Using the "C# developer Career Path" as the Home page for now. That would be a guide for myself to build and collect knowledge.

Add some helpful resource in one page.


root: Ignore default JetBrains IDEA module configuration

Motivation for this change
Avoid accidentally committing IDE configuration.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

Could you please open your PRs against the correct branch? This pinging seems to happen to all of your PRs.

Could you please open your PRs against the correct branch? This pinging seems to happen to all of your PRs.

Sorry, I keep tripping over the GitHub UI.

@Artturin Would you be OK to review this?

should be fine as we already have .idea/


Typescript transition

[x] desktop hot (reload of tsx works)
[x] desktop cold
[x] desktop prod
[x] ios
[x] android
[x] jest
[x] storyshots
[x] storybook
[x] rn-storybook
[x] no flow in ci
[x] no flow in precommit
[x] tsc in precommit
[x] tslint (just using babel-eslint for now)
[x] prettier to minimize

@MarcoPolo pushing fixes here

[ ] osx prod build
[ ] ios prod build
[ ] android prod build

@keybase/react-hackers this is ready


tfupdate: 0.6.8 -> 0.7.1

Description of changes
https://github.com/minamijoyo/tfupdate/releases/tag/v0.7.0
https://github.com/minamijoyo/tfupdate/releases/tag/v0.7.1
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.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.

Rebased + updated to 0.7.1


Reduce allocations while loading deferred resources

What does the pull request do?
This PR aims to remove most unnecessary allocations related to deferred content inside resources dictionaries.
While loading the Fluent theme, there's a bit less than 1450 deferred items created, and that's normal.
However, they come with several allocations, both very short lived ones, and some longer ones, stored.
This PR removes more than 15000 allocations while loading the FluentTheme, half of which were long lived.
Resuls to load the FluentTheme:

Method
Mean
Error
StdDev
Gen0
Gen1
Allocated

FluentTheme (Before)
1.670 ms
0.0192 ms
0.0160 ms
66.4063
64.4531
1.07 MB

FluentTheme (After)
633.4 us
12.60 us
6.59 us
13.6719
12.6953
228.17 KB

Note that it's still the JIT that takes most of the time when starting up the application (use NativeAOT when you can!), but I'll still take those gains :)
As part of this PR, StaticResourceExtension.ProvideValue() was also improved (not that it was slow to begin with):

Method
Mean
Error
StdDev
Gen0
Allocated

StaticResource (Before)
238.5 ns
1.45 ns
1.35 ns
0.0153
256 B

StaticResource (After)
160.6 ns
1.50 ns
1.33 ns
-
-

How was the solution implemented (if it's not obvious)?
All numbers below are for loading the Fluent theme.
Buffer resource nodes
In XamlIlRuntimeHelpers, a reusable buffer is now used to store the parent resources nodes and get an array out of that, without using Linq. The array is stored instead of the list.
Removes ≈4350 allocations:

1450 List<IResourceNode> (long lived)
1450 OfType<IResourceNode> enumerator (short lived)
1450 IResourceNode[] backing array from List having to grow (short lived)

Extra enumerator allocations happening when the deferred resource will be built have also been removed.
Cache default parents
DefaultAvaloniaXamlIlParentStackProvider now caches its parent as an array, since it's only the Application.Current instance, which is very unlikely to change between two calls.
Removes ≈1450 allocations:

1450 Parents enumerator (short lived)

List parents without allocating
A specialized version of IAvaloniaXamlIlParentStackProvider is introduced IAvaloniaXamlIlEagerParentStackProvider: it can iterate the parents without any allocation. This interface is implemented by the runtime XAML context.
Removes ≈3500 allocations:

3500 ParentStackEnumerable.Enumerator (short lived)

As a bonus, using this interface makes StaticResource.ProvideValue() allocation-free.
Avoid delegate allocations
XamlIlRuntimeHelpers.DeferredTransformationFactoryV3 now returns an IDeferredContent instead of a Func<IServiceProvider, object>.
The returned closure directly implements this interface, so we can get rid of the delegate.
ResourceDictionary is now aware of IDeferredContent and can store it directly.
The XAML compiler now passes a function pointer to DeferredTransformationFactoryV3 to build its content instead of creating a delegate.
Removes ≈4350 allocations:

2900 Func<IServiceProvider, object> (long lived)
1450 DeferredItem inside the resource dictionary (long lived)

Reuse parent nodes
DeferredTransformationFactoryV3 reuses the parent resource nodes if they're the same as the last time it was called. This method is often called with the same parents (e.g. all items in the same resource dictionary), so this change provides nice gains. Out of 1450 items in the fluent theme, there are only 20 different lists of parents.
Removes ≈1430 allocations:

1430 IResourceNode[] (long lived)

Set resource dictionary capacity
ResourceDictionary.EnsureCapacity() is called.
This is https://github.com/AvaloniaUI/Avalonia/pull/15069 which is the only one I could really make separate (all other modifications are intertwined), but that I measured as part of this PR.
Removes ≈30 allocations, but larger ones:

30 Dictionary+Entry[] (short lived)

Notes
ResourceDictionary.AddDeferred(IDeferredContent) and Add(IDeferredContent) are equivalent. I didn't bother with disallowing one or the other, but this might be worth mentioning.
Dependencies
This PR is complete, but is marked as a draft because it contains merges from several other PR:

https://github.com/kekekeks/XamlX/pull/107
https://github.com/kekekeks/XamlX/pull/108
https://github.com/AvaloniaUI/Avalonia/pull/15069

When these get merged, I will rebase this PR to have a clean diff.

@MrJul @maxkatz6 It seems that this PR has introduced a breaking change. IAvaloniaXamlIlEagerParentStackProvider isn't supported by XAML compiled by Avalonia 11.0.x, so attempting to cast to it will trigger exceptions on XAML load.

Hmm, there is a type check here.

You are only doing a type check for the current stack provider, but EagerParentStackEnumerator assumes that all providers in the chain support the new interface


Blink

This is just a question is it possible to maybe make it blink compatible instead of edgeHTML ?
When doing so platform testing is much easier and you can use the api of the blink engine instead of 4 different api's for 3 different platforms.

@andreabosman16 I think the point of this lib is to use whatever is most likely to be installed on a given system (this lib promotes re-use).  On linux, it is highly likely that webkit is installed, on MacOS WebView (webkit-based), and on Windows it has its own built-ins (MSHTML/edgeHTML).
There are other libs that wrap Chromium that you can bundle and ship your app with instead, if you want to use Blink, for example.

But won't windows be using blink as it s built-in this April ? and isn't it possible to let a user download it when not available on the platform ?

But won't windows be using blink as it s built-in this April ? and isn't it possible to let a user download it when not available on the platform ?

Windows will use Edge.  Edge is name of the browser. And Blink will be the engine of this browser.So nothing will change internaly

I know but the engine is multi-platform right ?

Blink engine is multiplatform. But not native browser. Safari and Edge is default browser for Windows and Mac.
You want to make user download entire browser just for your app? Event on Win 7 or 8 or even Win 10 without updates?

Why not blink isn't that big and when using it you use one api for all the platforms

Users will never download browsers for you.  Noone will do that. I will never download any browser for any application. Its just strange

it's not the browser you download. it's just the engine that runs it like electron does but using rust or go as io. something like pwa but with io and more window permission. I am looking for a xul replacer.

I think there's some miscommunication here, as ignishub seems confused over browser/engine semantics.
edgeHTML is dead - it was officially discontinued last year, and we've now got public builds of the Blink/Chromium backed Edge.
As such, realistically if webview is to survive it'll have to support Blink/Chromium and kill off edgeHTML (which is only going to get less secure and more unsupported over time). It's either that or essentially sunsetting this project

Yes, that was my thought as well. I installed a current 'development' build of the new Edge browser backed by Blink. And of course, it doesn't replace the current Edge browser. So I guess it's too early to make any statements about that.
I imagine that once the new blink-backed Edge browser is rolled out, it will replace the 'default' Edge browser. And then the question is: Does the API change at all?...


Fix 'is final tx' rule for komodo s1 december hardfork condition

See komodo IsFinalTx with komodo_hardfork_active() function use (which is actually s1 hardfork)

Few useful links about IsFinalTx, CLTV, nSequence:

BIP: 65 OP_CHECKLOCKTIMEVERIFY
Why is OP_CHECKLOCKTIMEVERIFY disabled by maximum sequence number?
Introduction to Bitcoin Programming with BitcoinJS and Bitcoin Core Timelock Transactions
What is TxIn's sequence?

It looks like the changes are ready and can be found in the patch-isfinaltx branch. We will no longer be using lock_time_has_passed/ tx.lock_time() because this method is not clear as it returns a value based on certain conditions. Instead, we will be using a new function called is_final_tx_komodo, which is a step-by-step replication of the IsFinalTx function from komodod and uses the previously introduced tx.raw_lock_time() to read the "raw" nLockTime field. All possible cases have been covered in the tests.


Build user api

This PR builds the user API and restructures the controller's tests

This PR is being closed due to all the changes made since the college project was over


Role class must derive from UcoObject

The Role class is a subclass of Facet and it's not possible to create a Relation between an Identity (IdentityAbstraction subclass of UcoObject) and a Role , because the ObservableRelationship class expects an UcoObject as source and target properties.

Thank you for reporting.  I'll post a fix.

Fixed in #73 .  Thanks again!

The code you pushed doesn't seems to solve the issue. If I put the following code in the example.py:
role_created_time = datetime.strptime("2024-09-24T21:38:19", "%Y-%m-%dT%H:%M:%S")
role_df_expert = uco.role.Role(
    description="Digital Forensic Mobile Expert",
    name="Digital Forensic Incident Response",
    created_time=role_created_time
)
bundle.append_to_uco_object(role_df_expert)
has_role_relation = uco.observable.ObservableRelationship(
    source=object_identity,
    target=role_df_expert,
    kind_of_relationship="Has_Role",
    directional=True,
)
bundle.append_to_uco_object(has_role_relation)

It raises the TypeError: The target of an ObservableRelationship must be an Observable.
Moreover I realised that the relation "Has_Role" is not included in the list.

Roles are not natively ObservableObjects.  If you instantiate a uco.core.Relationship instead of uco.observableObservableRelationship, you should be fine.
Has_Role is known to be omitted, but that vocabulary is only suggestive.  You can use any string you wish.

Thanks Alex!


auto-import unresolved identifier

perhaps it is worth adding auto-import of unresolved identifier? like here https://github.com/vshaxe/vshaxe/wiki/Code-Actions#import-actions

Well, just try it maybe (Kode Studio uses vshaxe and all of that stuff works).

Kode Studio uses vshaxe and all of that stuff works

is there any way to update vshaxe extension in the already installed kode studio?

As I said, that stuff works, I just checked. See the description in your link to see how. vshaxe in Kode Studio is newer than the last change of that wiki page.

Oh, sorry, it's my fault. I used 17.1 version of Kode Studio. In 17.9.1 this feature works well.


Truncate ids to fit within max OS file name length

C++ projects in particular with longer names in benchmark.yaml cause an error to be raised in workdir generation due to long file names

JOB: https://pantheon.corp.google.com/kubernetes/job/us-central1/llm-experiment/default/ofg-pr-98-ggryan
REPORT: https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-02-12-98-ggryan-comparison/index.html

@DonggeLiu we might want to see if we can come up with some better way to generate this, in case we introduce clashes by truncating.

Yep, this is a temp solution.
@trashvisor and I discussed using raw_function_name as the directory name.
However, our current directory name is tied with the name in the report, and having benchmark name like "_ZN4absl19str_format_internal13FormatArgImpl8DispatchIxEEbNS1_4DataENS0_24FormatConversionSpecImplEPv" is not user friendly.
We can mitigate this by matching raw_function_name with the function signature during report generation (e.g., in web.py), but that takes a bit more extra work and can wait until this finishes.

JOB: https://pantheon.corp.google.com/kubernetes/job/us-central1/llm-experiment/default/ofg-pr-98-ggryan REPORT: https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-02-12-98-ggryan-comparison/index.html

Killing this as its code is outdated : )

Seeing this error:
https://pantheon.corp.google.com/logs/query;cursorTimestamp=2024-02-12T13:20:40.335827856Z;duration=PT12H;query=resource.type%3D"k8s_container"
resource.labels.project_id%3D"oss-fuzz"
resource.labels.location%3D"us-central1"
resource.labels.cluster_name%3D"llm-experiment"
resource.labels.namespace_name%3D"default"
labels.k8s-pod%2Fbatch_kubernetes_io%2Fcontroller-uid%3D"f475de05-eccc-4e51-8a9b-a15efc705afa" severity>%3DDEFAULT
timestamp%3D"2024-02-12T13:20:40.335827856Z"
insertId%3D"egk006exzw7vw80x"?project=oss-fuzz
No concrete proof yet, but since it happened after the 'Uploading the report.', I guess it relates to uploading directories with truncated name.

double pointers sorry, see https://cloud.google.com/storage/docs/wildcards
Match zero or more characters across directory boundaries. When used as part of a local file path, the ** wildcard should always be immediately preceded by a directory delimiter. For example, my-directory/**.txt is valid, but my-directory/abc** is not.

double pointers sorry, see https://cloud.google.com/storage/docs/wildcards
Match zero or more characters across directory boundaries. When used as part of a local file path, the ** wildcard should always be immediately preceded by a directory delimiter. For example, my-directory/**.txt is valid, but my-directory/abc** is not.

Yes, it could be.
As a temp mitigation, I will switch to raw_function_name, given we have that info now.
Let me test the change in my PR.

Would you mind if I steal your code to mine?
I want to merge, but copying is probably simpler than resolving conflicts in this case : (

haha go for it!!

Manually 'merged' into #97


NeedsBrowser error with get_tokens.py

Migrated from this gist comment https://gist.github.com/rithvikvibhu/952f83ea656c6782fbd0f1645059055d#gistcomment-3586276
@cicero200272:

I am requested to log in (using a browser) first. When I copy & paste the URL to a browser window, I can provide my credentials, but then I am stuck on a "One moment please..." page which does not reload. Does anybody else have this kind of problem?

@rithvikvibhu:

Not sure what you mean @cicero200272. Which URL did you copy? This script doesn't need any browser or a website to visit at all.

@cicero200272:

Have you set the USERNAME and PASSWORD fields in the script?
Verify:

Do you have 2FA enabled? If so, then create an app password and use that instead.
Make sure the USERNAME variable is just the username, not the complete email address (@gmail.com).
Are the overrides unchanged? (all of them set to None is fine).

Have you set the USERNAME and PASSWORD fields in the script?
Verify:

Do you have 2FA enabled? If so, then create an app password and use that instead.
Make sure the USERNAME variable is just the username, not the complete email address (@gmail.com).
Are the overrides unchanged? (all of them set to None is fine).

Hi @rithvikvibhu, thanks for looking into this! I did replace USERNAME and PASSWORD with my credentials.

No 2FA
I use USERNAME without email extension
All set to None

Hi @rithvikvibhu, thanks for looking into this! I did replace USERNAME and PASSWORD with my credentials.

No 2FA
I use USERNAME without email extension
All set to None

did you solve this? I'm getting your same error.

did you solve this? I'm getting your same error.

hi guys do you have any update on this? I get cicero's same error

hi guys do you have any update on this? I get cicero's same error

Okay this has worked from people at https://github.com/kiwiz/gkeepapi/issues/20:

Set the device_id override to a random value with _get_device_id() (so that it doesn't change between script runs)
Run the script - if it works, all good, otherwise continue
Visit https://accounts.google.com/DisplayUnlockCaptcha and complete captcha
Try the script again - should work now

This applies to accounts without 2FA also.

Okay this has worked from people at https://github.com/kiwiz/gkeepapi/issues/20:

Set the device_id override to a random value with _get_device_id() (so that it doesn't change between script runs)
Run the script - if it works, all good, otherwise continue
Visit https://accounts.google.com/DisplayUnlockCaptcha and complete captcha
Try the script again - should work now

This applies to accounts without 2FA also.

Visiting the captcha page did the trick! Thanks again for your help.

Visiting the captcha page did the trick! Thanks again for your help.

Hi!!
I have been trying to get the tokes but I'm experiencing the same issue reported here.
The Google web captcha is not working any more.
I have tried all other solutions with no luck. ..


Dev

Hey @Jovial248,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 5525a340-b489-11e9-a5f8-ad46f5ea24b0


Remove the natasha time system because it doesn't do anything.

Remove the useless code!
Fixes #8

Approved


bootstrap-datepicker

https://www.npmjs.com/package/bootstrap-datepicker

Just like your other PRs - could you fix the names?

Yup the two files you should have are these:
/bootstrap-datepicker_v1.x.x/flow_>=v0.30.x/bootstrap-datepicker_v1.x.x.js.flow
/bootstrap-datepicker_v1.x.x/test_bootstrap-datepicker_v1.x.x.js.flow

The second is for adding tests to have a sanity check.

fixed file names

This hasn't been updated for quite a while and I can't add commits to the PR.
If this is still relevant please submit a new PR.


Coverage reporting

Codecov Report

:exclamation: No coverage uploaded for pull request base (beta@9b546c6). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             beta      #20   +/-   ##
=======================================
  Coverage        ?   90.90%
=======================================
  Files           ?        5
  Lines           ?       55
  Branches        ?        6
=======================================
  Hits            ?       50
  Misses          ?        0
  Partials        ?        5

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Use stdlib ensure_packages for installing ca-certificates package

Hi there!
Ran into a common issue with installing packages in Puppet for duplicate resources when using the puppet/nodejs module:
https://github.com/voxpupuli/puppet-nodejs/blob/master/manifests/repo/nodesource/apt.pp#L9
Feb 22 14:17:18 exampleserver puppet-agent[18599]: Could not retrieve catalog from remote server: Error 400 on SERVER: Evaluation Error: Error while evaluating a Resource Statement, Duplicate declaration: Package[ca-certificates] is already declared; cannot redeclare at /etc/puppetlabs/code/environments/staging/modules/ca_cert/manifests/init.pp:70 at /etc/puppetlabs/code/environments/staging/modules/ca_cert/manifests/init.pp:70:5 on node exampleserver.example.com

For now we've opted to set install_package: false with this module but it makes our setup more complicated than necessary. Puppetlabs stdlib provides ensure_packages to consistently handle this situation (available since stdlib 2.6):
https://forge.puppetlabs.com/puppetlabs/stdlib#ensure_packages
Can you please switch to ensure_packages and release? Very much appreciated. Thanks!

I've added or $package_ensure == installed to the if condition. Please let me know if you'd like anything else changed. Thanks!

I just pushed v1.2.0 with this fix up to the forge.
Thanks for the patch!


WIP - [es] Add concepts/storage/persistent-volumes.md

Adding new content to concepts/storage/persistent-volumes.md
Related to https://github.com/kubernetes/website/issues/13948#issuecomment-1007875517
Work still in progress but it's huge!

bad user info clossing this ! sorry


Development/json types

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Thunder/1433/rdkcentral/Thunder

Commit: 071c318002a74a0aa0e7ce2d05fee4b0d6480444

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/Thunder/1433/rdkcentral/Thunder

Commit: 071c318002a74a0aa0e7ce2d05fee4b0d6480444

Report detail: gist

Closing the PR but keeping the related branch.


[Bug] Toolbox magic type feature doesn't indicate default chart state

Version
5.3.0
Link to Minimal Reproduction
https://nccrd.saeon.ac.za/reports
Steps to Reproduce

Go to the link
Look at the toolbox - the 'magic type' feature indicates no current state despite that the chart state does reflect one of the possible options (in this case stacked bar)

Current Behavior
The magic type feature allows for toggling between line/bar, and stacked/not stacked. In my case I have a bar chart that is stacked. When I include the magic type in the toolbox, none of the icons are selected. Because of this:

If a user clicks the 'line' option, then the chart changes to a line. This looks good. But
If a user clicks the 'bar' option, then nothing happens (except that the 'bar' option becomes emphasized'). The next click on the bar option changes the chart. This feels like a bug
Same with the stacked icon

Expected Behavior
The magic type icons represent all possible 'states' of the chart. By default the charts render in one of these 'states'. The correct icons should show as 'on' on initial chart render
Environment
- OS: ALL
- Browser: ALL
- Framework: ALL

Any additional comments?
No response

feature: {
magicType: {
show: true,
type: ['line', 'bar'],
iconStatus: { line: 'normal', bar: 'emphasis' }
}
}
You can set default indicate using the iconStatus bar: 'emphasis'


fix : 중복 일정 뷰에서 이동시간이 있을 경우 다른 일정과 겹쳐서 일정이 표시되는 현상 수정(#Dooray-캘린더/1141)

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has description for the breaking change

Description

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Commit log must be written in english.

[09/19] Reviewed. Nice work.

@dongsik-yoo
From now on, the commit log will be written in English.😮
Thank you for the review.


Add schema.yaml for helm3 compliant chart validation

Closes #1329 by mimicing https://github.com/jupyterhub/zero-to-jupyterhub-k8s/pull/2033 that converts a schema.yaml file into values.schema.json before publishing and package it with the Helm chart so helm automatically can perform validation logic whenever helm template, helm install or helm upgrade is run.
My process to create the schema.yaml file was to look in values.yaml and add schema entries for everything there. I copied various schema.yaml entries from z2jh's schema.yaml file to save me some work, but a lot of descriptions is still just having a TODO note. I consider it out of scope for this PR to also write descriptions for everything in the schema.yaml. The descriptions are currently not used anyhow, but they can be used to generate a configuration reference in the future.
I've tried this schema against mybinder.org's Helm chart, adjusted the schema by adding hpa and networkPolicy to not break mybinder.org-deploy and because these configuration options may very well be upstreamed to BinderHub itself.

Thanks, @consideRatio!

I think this PR broke the helm chart publishing action. It looks like the fix would be to install a Python yaml library. Do you want to follow up this PR with a fix?


the time consumption of the  forward

I tested the googlenet on my ubuntu pc (cpu i7), when I use the cat.jpg as the input of the net, the run consume 6s (cpu mode). It's too long , is it normal？
Almost same time consuming both using cpp interface and using python interface.
Here is my core code.
`void run() {
if (FLAGS_init_net.empty()) {
LOG(FATAL) << "No init net specified. Use --init_net=/path/to/net.";
}
if (FLAGS_predict_net.empty()) {
LOG(FATAL) << "No predict net specified. Use --predict_net=/path/to/net.";
}
caffe2::NetDef init_net, predict_net;
CAFFE_ENFORCE(ReadProtoFromFile(FLAGS_init_net, &init_net));
CAFFE_ENFORCE(ReadProtoFromFile(FLAGS_predict_net, &predict_net));
auto predictor = caffe2::make_unique(init_net, predict_net);
cv::Mat im = cv::imread("./cat.jpg");
if (!im.data) {
std::cout << "failed to read image" << std::endl;
return;
}
cv::resize(im, im, cv::Size(224, 224));
im.convertTo(im, CV_32FC3);
cv::Mat mean = cv::Mat(im.size(), im.type(), cv::Scalar(104.00698793,<PHONE_NUMBER>2,<PHONE_NUMBER>4));
im = im - mean;
std::vector input_shape{1, 3, 224, 224};
TensorCPU input_tensor(input_shape);
float *input_tensor_data = input_tensor.mutable_data();
std::vectorcv::Mat bgr_channels;
for (int c = 0; c < im.channels(); c++) {
cv::Mat single_channel(im.size(), CV_32FC1, input_tensor_data);
bgr_channels.push_back(single_channel);
input_tensor_data += (im.rows * im.cols);
}
cv::split(im, bgr_channels);
std::cout << "Checking that a null forward-pass works" << std::endl;
Predictor::TensorVector inputVec, outputVec;
inputVec.push_back(&input_tensor);

double time_start = getTimeOfMSeconds();
for (int i = 0; i < 10; i++) {
    predictor->run(inputVec, &outputVec);
}
double time_end = getTimeOfMSeconds();
std::cout << "time used: " << time_end - time_start << std::endl;

CAFFE_ENFORCE_GT(outputVec.size(), 0);

TensorCPU *output = outputVec[0];
cv::Mat pre(1, output->size(), CV_32FC1, output->mutable_data<float>());
cv::Point max_idx;
cv::minMaxLoc(pre, nullptr, nullptr, nullptr, &max_idx);

}`

USE_OPENMP=OFF
USE_MPI=OFF
Blas: OpenBlas, and set the thread = 1

You should use the convnet_benchmarks.py script to test the speed to make sure the speed is reasonable - 6s sounds too long for a normal C2 build.

I have tested the performance using the convnet_benchmarks.py script.
I got a normal result when testing on GPU
$ PYTHONPATH=./:$PYTHONPATH python ./caffe2/python/convnet_benchmarks.py --batch_size 16 --model Inception --forward_only --warmup_iterations 2 --num_workers 4

Inception: running forward only.
I0315 14:26:22.314787  3824 net_dag.cc:123] Operator graph pruning prior to chain compute took: 0.000254785 secs
I0315 14:26:22.315145  3824 net_dag.cc:374] Number of parallel execution chains 49 Number of operators = 141
I0315 14:26:22.315433  3824 net_dag.cc:536] Starting benchmark.
I0315 14:26:22.315441  3824 net_dag.cc:537] Running warmup runs.
I0315 14:26:23.073134  3824 net_dag.cc:547] Main runs.
I0315 14:26:23.371788  3824 net_dag.cc:558] Main run finished. Milliseconds per iter: 29.8606. Iters per second: 33.4889

But the performance on CPU is poor, similar to the previous results.
$ PYTHONPATH=./:$PYTHONPATH python ./caffe2/python/convnet_benchmarks.py --batch_size 16 --model Inception --forward_only --cpu --warmup_iterations 2 --num_workers 4

Inception: running forward only.
I0315 14:20:01.430321  3718 net_dag.cc:123] Operator graph pruning prior to chain compute took: 0.000249852 secs
I0315 14:20:01.430660  3718 net_dag.cc:374] Number of parallel execution chains 49 Number of operators = 141
I0315 14:20:01.430908  3718 net_dag.cc:536] Starting benchmark.
I0315 14:20:01.430917  3718 net_dag.cc:537] Running warmup runs.
I0315 14:22:16.696028  3718 net_dag.cc:547] Main runs.
I0315 14:33:38.261889  3718 net_dag.cc:558] Main run finished. Milliseconds per iter: 68156.6. Iters per second: 0.0146721

Looks like you might be building with an non-optimized build. Could you do the build with
cmake -DCMAKE_VERBOSE_MAKEFILE=1
make
and paste the build log?

Yeah, I just forgot to set CMAKE_BUILD_TYPE=Release. Now the result looks good.
$ ./:$PYTHONPATH python ./caffe2/python/convnet_benchmarks.py --batch_size 1 --model Inception --forward_only --cpu  --num_workers 1 --iteration 100

Inception: running forward only.
I0315 15:53:10.486758 22070 net_dag.cc:123] Operator graph pruning prior to chain compute took: 3.9578e-05 secs
I0315 15:53:10.486845 22070 net_dag.cc:374] Number of parallel execution chains 49 Number of operators = 141
W0315 15:53:10.486850 22070 net_dag.cc:391] Number of workers is 1: this means that all operators will be executed sequentially. Did you forget to set num_workers in the NetDef?
I0315 15:53:10.487021 22070 net_dag.cc:536] Starting benchmark.
I0315 15:53:10.487027 22070 net_dag.cc:537] Running warmup runs.
I0315 15:53:11.895370 22070 net_dag.cc:547] Main runs.
I0315 15:53:25.632608 22070 net_dag.cc:558] Main run finished. Milliseconds per iter: 137.372. Iters per second: 7.2795

Thank you very much for your great work.

Glad it worked!


Poor man's simulcast implementation

Per-commit review is suggested (tried to keep the commit history clean and "atomic").
As discussed, I'm submitting this PR to simplify the review of the simulcast that would otherwise have been too overwhelming (over 1000 LOC PR).
This is stage 1 of simulcast implementation (fixes https://github.com/matrix-org/waterfall/issues/31).

I have to admit that some of the abstractions are becoming a bit unclear to me again (e.g. Subscription vs SubscribedTrack). I wonder if the naming could be improved here

We don't have a SubscribedTrack structure if I'm not mistaken 🤔, but we do have a Subscription which refers to a single subscription (subscription to a single track).
Or did you mean that you would prefer Subscription called SubscribedTrack? - I thought that it might be confusing because it would mean that SubscribedTrack encapsulates a track to which someone is subscribed (to me it feels like it gives a notion that there is one SubscribedTrack, i.e. if asdf is a track, to which someone is subscribed, does it have to be a type of SubscribedTrack and stored in a map, so that we can differentiate tracks to which someone is subscribed vs tracks that no-one is subscribed to).
Whereas with Subscription it's just a subscription (has a connotation that it is a new entity) to an existing published track, which means that there could be many of them, i.e. we may have 3,4 or 10 subscriptions to a single existing track.

I have to admit that some of the abstractions are becoming a bit unclear to me again (e.g. Subscription vs SubscribedTrack). I wonder if the naming could be improved here

We don't have a SubscribedTrack structure if I'm not mistaken thinking, but we do have a Subscription which refers to a single subscription (subscription to a single track).
Or did you mean that you would prefer Subscription called SubscribedTrack? - I thought that it might be confusing because it would mean that SubscribedTrack encapsulates a track to which someone is subscribed (to me it feels like it gives a notion that there is one SubscribedTrack, i.e. if asdf is a track, to which someone is subscribed, does it have to be a type of SubscribedTrack and stored in a map, so that we can differentiate tracks to which someone is subscribed vs tracks that no-one is subscribed to).
Whereas with Subscription it's just a subscription (has a connotation that it is a new entity) to an existing published track, which means that there could be many of them, i.e. we may have 3,4 or 10 subscriptions to a single existing track.

Ah, yes, the whole Subscription, Subscribers and TrackSubscribers has me confused...

LGTM
Do we wait to resolve the Firefox issue before merging?
@dbkr, do you want to have another look?

Yep, the issue with grey frames seems to be a bug, I need to fix it either prior to merging or after that.


added solar drag

Seems like everything's working for now

Also, this needs a rebase on current master.

Merged! Thanks @nikita-astronaut :clap:


Clean-up NAT-PMP and PCP message structures.

NAT-PMP and PCPv2 messages have been refactored and tested.


Masking st

Purpose of this pull request
This pull request...
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[X] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Thanks for review team. Will be performing the release today and merge this PR in few hours - post the release.

Thanks for review team. Will be performing the release today and merge this PR in few hours - post the release.

Awesome! Thank you for the update.


[core,redirection] fix length of redirection strings

length field is in bytes, when converting from UCS-2 use size in WCHAR

@freerdp-bot test again please!


Sync missing serviceIds for service models

This adds serviceIds for all missing services.  There's also 5 that are remaining:
Missing: appsync
Missing: iot-jobs-data
Missing: mgh
Missing: pinpoint
Missing: transcribe
Given we're not using these for anything just yet, I think it's ok to add these to start, and then tracking down the missing 5 services.

Codecov Report

Merging #1490 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #1490   +/-   ##
=======================================
  Coverage     71.4%   71.4%
=======================================
  Files          128     128
  Lines        14165   14165
=======================================
  Hits         10115   10115
  Misses        4050    4050

Continue to review full report at Codecov.

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


PHPUnit 10 support

PHPUnit v10 just got out. Is the support planned?

@simPod
See https://github.com/johnkary/phpunit-speedtrap/pull/98, which may need an update.

Quick update: for those of you coming here and looking for an alternative that works with

phpunit/phpunit:^8.5.26
phpunit/phpunit:^9.6.0
phpunit/phpunit:^10.4.0

as well as

PHP 7.4
PHP 8.1
PHP 8.2
PHP 8.3

I just released ergebnis/phpunit-slow-test-detector:2.6.0 with support for all of these.


let the public and static folder catch new files

For context, when you have a script that generates a new static file at build time (sitemap.xml for example), it has to be inside .next/static, and then you'll need a Now route for it, with this change you could generate the file inside public/static and the builder will now take care of it.
The util includeOnlyEntryDirectory is no longer being used after this change, should I remove it?

Codecov Report

Merging #3096 into canary will increase coverage by 2.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           canary    #3096      +/-   ##
==========================================
+ Coverage   11.33%   13.34%   +2.01%
==========================================
  Files         268      268
  Lines       10064    10308     +244
  Branches     1314     1221      -93
==========================================
+ Hits         1141     1376     +235
- Misses       8830     8870      +40
+ Partials       93       62      -31

Impacted Files
Coverage Δ

src/commands/deploy/latest.js
0% <0%> (ø)
:arrow_up:

src/util/handle-error.ts
38.88% <0%> (ø)
:arrow_up:

src/commands/deploy/legacy.ts
0% <0%> (ø)
:arrow_up:

src/util/deploy/process-deployment.ts
0% <0%> (ø)
:arrow_up:

src/util/deploy/create-deploy.js
0% <0%> (ø)
:arrow_up:

src/util/errors-ts.ts
47.48% <0%> (+0.02%)
:arrow_up:

src/util/dev/builder-cache.ts
80.74% <0%> (+3.47%)
:arrow_up:

src/util/get-files.ts
91.66% <0%> (+3.88%)
:arrow_up:

src/util/dev/router.ts
82.69% <0%> (+4.12%)
:arrow_up:

src/util/dev/builder.ts
76.92% <0%> (+4.4%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Meets 'ImportErrot' when import taso

Hello All
I build TASO on ubuntu16.04(no cuda) and meet error when 'import taso' in python3.
here is error info:
Python 3.5.2 (default, Oct  7 2020, 17:19:02)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.

import taso
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.5/dist-packages/taso-0.1.0-py3.5-linux-x86_64.egg/taso/init.py", line 1, in
from .core import *
ImportError:  sdk/TASO/build/libtaso_runtime.so: undefined symbol: _ZN4taso10Activation7forwardEb

THX

It seems that you have not build the libtaso_runtime.so


Switch to allow config file to exist in a different directory

Softether works great in containers (docker).  But it would be even better if we could mount a volume to keep the config file in.  This would only be practical if softether would allow the config file (via a switch) to be located in a different directory than the main binaries.
Please add a command line argument to allow this.

https://github.com/SoftEtherVPN/SoftEtherVPN/issues/341

Just FYI, it's a bit hackish but you can also put the config file elsewhere by symlinking.  Here's a dockerfile that works fine with a persistent disk in Kubernetes for storing the config:
FROM alpine
ENV LANG en_US.UTF-8
RUN apk update && apk add openssl ncurses readline dumb-init iptables
RUN apk add build-base readline-dev openssl-dev ncurses-dev && \
        wget -qO - http://www.softether-download.com/files/softether/v4.24-9651-beta-2017.10.23-tree/Source_Code/softether-src-v4.24-9651-beta.tar.gz | tar -xzf - && \
        cd v4.24-9651 && \
        ./configure && \
        make install && \
        apk del build-base readline-dev openssl-dev ncurses-dev && \
        cd .. && rm -rf v4.24-9651 && \
        mkdir -p /etc/vpnserver && \
        ln -sf /etc/vpnserver/vpn_server.config /usr/vpnserver/vpn_server.config
VOLUME /etc/vpnserver
EXPOSE 443/tcp
ENTRYPOINT ["/usr/bin/dumb-init", "--"]
CMD ["/bin/sh","-c","touch /etc/vpnserver/vpn_server.conf && exec /usr/vpnserver/vpnserver execsvc"]


Link to documentation is not working

Link to documentation raises a bad request error with a 400 response code.

I guess the correct link is:
https://py-search.readthedocs.io/en/latest/


bug:  Timestamp is not working. It redirects the user to the main page.

Timestamps is redirecting user to the main page.
video:
Screencast from 2024-05-17 21-57-26.webm
Expected behavior
Timestamps should redirct to the specific part of video

fixed in #602


Support tree-sitter-markdown highlight groups

MDeiml/tree-sitter-markdown is a new Markdown parser for Treesitter that (among other things) improves speed and uses Treesitter to highlight embedded code blocks.The parser is still experimental, but it's already a big improvement over traditional syntax highlighting, so it would be great to have support for it here. I asked the author on Reddit how colorschemes can support it, and they said:

They would need to add colors for the highlight groups TSTitle, TSLiteral, TSEmphasis, TSStrong, TSURI, TSTextReference, TSPunctSpecial and TSStringEscape.

For reference, this commit shows how Catpuccin implemented support.

Thanks for letting me know and for the extra details! This looks really nice, I really like the highlighting in code blocks. These groups already exist so I think it looks decent now but I added some colors for TSStrong and TSEmphasis, what do you think:

Looks great! I should have been more specific about what seemed to be missing, and bold / italic colors make a big difference. I also wonder if a different color for link titles would help. Here's a couple of links with Treesitter:

And without:

The latter seems a bit easier to parse, but what do you think?

Ah I see, yeah I like the cyan more! Do you have any other suggestions for the markdown colors, either with or without Treesitter? I just have those 3 changes so far:

Looks great as-is IMO!


Programmable Switch Events

Need to see about working with programmable switch events and stateless switch events in Homebridge.

Don't think there is anything I need to add for this, as long as you specify the right characteristics.  See https://github.com/ebaauw/homebridge-hue/issues/54 for details.


Cant open folder mounted locally with sshfs

Cant open remote folder mounted with "sudo sshfs -o allow_other ....." .
Sublime and others -  can.
Just receive Editor is not responding message.
Linux.

Thanks for the report! This looks like a duplicate of https://github.com/atom/atom/issues/2456 / https://github.com/atom/atom/issues/1766 – please subscribe there for updates.


Feat/templates cli

Thank you so much for contributing to Datapane, please help us by filing out this PR template and ensure you have read the CONTRIBUTING.md.
It may take a few days to review your PR and merge it - please bear with us!
Thanks!
Prerequsites

[x] Has been tested locally
[ ] If a bugfix, have included a breaking test if possible

Proposed Changes
This PR adds template support to the Datapane CLI. The high-level steps of the template feature are:

Clone a Datapane template repository and delete the .git directory.
Change current working directory to the new template directory.
Run the template.py script, which will open the locally saved template.html upon completion.

Usage
From CLI: datapane template <repo_source>, e.g.

datapane template dp-template-classifier-dashboard
datapane template https://github.com/datapane/dp-template-classifier-dashboard.

Supports both relative (for the datapane organization) and absolute repository locations.
Repository template for creating Datapane templates
Included in this review is the new repository template at https://github.com/datapane/dp-template. This includes a directory structure and instructions on creating new Datapane templates.
This has been used to create the new Datapane template, Performance dashboard for classifiers (https://github.com/datapane/dp-template-classifier-dashboard).

A suggestion for a QoL improvement is a quick/minor change to the expected form of relative repo locations.
Currently, it requires the full repo name, e.g. dp-template-classifier-dashboard. If we maintain the naming convention specified in the template repository, we could have our usage code as datapane template classifier-dashboard instead.

On the template repo itself,

should we remove the dependency on poetry, as it requires users to use another build tool - can we get by with just pip only to keep things simple?
should we use pipdeps to build that requirements.txt?
pip does actually support pyproject.toml these days, we could still that file rather than requirements.txt if needed (see https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html - there's a whole bunch of PEPs that configure how this all works that I can't quite recall now! :) )

The pyproject.toml is for template developers not for template users – this supports both pip and poetry. I've made sure to delete this (and poetry.lock) when using datapane template.

how does this work for folks who have installed datapane via conda? Would they need different docs?

At the moment running a template with missing packages prompts the user that the packages need to be installed. The templates could require any package, and especially in the case of third-party templates, I thought it's best to let users manage their own environment.
pipreqs does look like a better way to do this, i.e. print out the missing packages and let them know we've created a requirements.txt they can use for convenience.

There is both a notebook and a python script in the repo - the template code here expects there to be a python script which it runs. Is the template author responsible for ensuring that the script is up-to-date with the notebook? Or we could convert the notebook to a script here in the client before running? See extract_py_notebook from datapane.client.apps.config for a sample function we could reuse
Perhaps we need a small Makefile in the template repo to automate some of these tasks (i would say taskfile but wouldn't want to add too many external dependencies)

A template can currently be script-first or notebook-first, the developer instructions in the template repository has a notice and steps for notebook-first templates.
I thought about adding a Makefile, but decided against it in case of the following scenario:

Developer creates a new template from dp-template
Adds their template code to template.py
Runs Makefile which overwrites their template.py with the contents of template.ipynb.

Shall we continue with notebook to script being the template developer's responsibility, or should we remove template.ipynb from the dp-template repo and update the instructions?

This is really cool! Couple of questions,

does this replace the current vendored template system in datapane.resources.templates and executed via datapane.client.write_templates?

write_template supports post-processing the files after cloning/copying, via template substitution - is that something we want to retain?
e.g. could change the contents of the generated report to personalise it to the user, etc.

I hadn't considered template substitution as a requirement for our Datapane templates. I think users will want to use them as starting points and load in their own data/visualisations/assets after seeing the template script/notebook and output.
I've had a look – I don't think it replaces it. The changes are mostly around cloning a template repo. The execution works the same as the current _run_script(), with the difference being: using template_path as cwd, ModuleNotFound handling, and not copying the script before running it.

LGTM - nice one!
Few comments, but unrelaed to merging this,

Has this been tested on Windows? Is there any path manipulation that wouldn't work there? Worth doing a quick double-check on the code (or perhaps we add a unit test here as we run tests in CI on Windows as well)

It's not tested on Windows – don't have a development machine with that OS/VM. Running tests in CI on Windows sounds like a great idea! @antdking is that something we can get up and running?

re the existing template system? Is that something we're dropping as part of the cycle (and can do in future commit?)

As a nice-to-have (hopefully end of this pitch) we can move init and hello-world to this new template system, and remove the old code. I think we're waiting on some confirmation on how init is used and where.

It's not tested on Windows – don't have a development machine with that OS/VM. Running tests in CI on Windows sounds like a great idea! @antdking is that something we can get up and running?

@shahinrostami CI runs in windows, macos, and ubuntu, across python 3.7, 3.8, and 3.9

It's not tested on Windows – don't have a development machine with that OS/VM. Running tests in CI on Windows sounds like a great idea! @antdking is that something we can get up and running?

@shahinrostami CI runs in windows, macos, and ubuntu, across python 3.7, 3.8, and 3.9

Thank you @antdking! I misread part of Mandeep's message. Will look into this for a test in future!