prepare for Readthedocs

Readthedocs is a complete all-in-one documentation framework. To get started, i would run sphinx-quickstart in this repo, or a copy to test.

CCB has a tutorial to get started with a older version. https://github.com/ccb-hms/sphinx-tutorial


maxVariantFrac

I wanted to filter out all the possible SNPs,  both homozygous and heterozygous. My understanding is that if no SNPs, no non-G should be expected, and thus I set this to 0, then almost all the CpG sites were excluded (22Million out of 23 Million were excluded). When I set it to 0.1, most of the CpG sites are retained. Is this filtering so harsh? Do you have any suggestions about filtering the SNPs?
--maxVariantFrac means the fraction of Non-G on the opposite strand of C compared with the coverage at this given base or only the coverage of the opposite strand of C? For instance, 10 reads from the C strand and 10 reads from the non-C strand, there are 3 non-G reads from the non-C strand, then the variant fraction here is 0.3 or 0.15?
Thanks a lot :)

0.1 is fairly reasonable. Please note that you end up filtering out any sequence artifacts and stuff like that, which will randomly appear with longer reads.

Thanks for the reply! My other question is which count, the count of total reads at this base or count of reads on the non-C strand, is the fraction of non-G reads in the non-C strands compared against?

It's the count on the non-C strand, since it's easier to assess whether there's a variant using it.


⚠ Bofrost.at has degraded performance

In e0403cb, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2168 ms

Resolved: Bofrost.at performance has improved in 832ec15.


added rotl function

Dear Arkworks team,
As I needed a rotl function myself (and rotating to the right 31 times seemed wasteful), I created a rotl function entirely analogous to the rotr function. Kind regards,
Sam

Description

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (master)
[ ] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

@prismaman Thanks for the PR and sorry for late review. Would you mind including a unit test for rotl too?


Feature: Support to change the chart type

🔖 Feature description
Now Devtron supports multiple workloads to deploy as a Devtron application, but for the existing application, users cannot change the workload type, If anyone wants to switch from rollout to deployment type then they can't.
🎤 Pitch
Since the deployment is a native workload supported by Kubernetes so for many applications we want to change the workload type from rollout to deployment.
✌ Solution
NA
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#3812

Now API is supported for the same, but it is still not available on UI.


🚧 WIP 🚧  - Use PeripheralModel

This is my initial attempt at using Model that can be instantiated on the fly.
While this sort-of works, most Views rely on CoreBluetooth-specific data to update their UI, which still makes it difficult to work with actual previews.
I think for this to work properly, I'd need to write protocols encompassing most of CoreBluetooth's types (CBManagerState, CBService, CBCharacteristic, ...) so that I can then instantiate mocks (this should also help with writing tests later on).
So something like:

protocol ManagerState
protocol Service
protocol Characteristic
protocol Peripheral
protocol Manager

Then:

Extends related CoreBluetooth types to make them conform to those protocols
Write Mocks which also conform
Refactor views to rely on types which implement the protocols
Refactor previews to use mocks
???
profit

Let me know if this makes sense, it's quite a substantial effort but would definitely come in handy imo. If it does, I'll use this PR to work on this :)

Sorry for the long answer, it's WWDC week :) I'll check it and return with the feedback.

Yeah haha, saw your tweets ;) no worries been busy on my end too anyways

@artemnovichkov Thanks for the review. Before I jump into addressing your comments, could you please review my suggestion for the higher-level refactor ? Because I feel this will actually invalidate this PR altogether

I've checked the whole PR and I agree with the refactoring. Initially, I didn't want to add extra abstractions, but it will help us to check the layout faster with previews and write tests as well.

Sorry for the delay, hectic at work! I'll give this a whirl when I have some time :)

Sorry for the force pushes, messed up my git config 😅 .
At any rate, this is so far what I managed to get working, the mocks still need a bit of work but at least, functionality-wise, nothing broke 🤓

All good, great progress! Ping we when the PR will be ready for review

Alright, so with all of this done, here's a quick debrief of what went well and what could still do with improvements:
The good:

The wrapping layer is correctly abstracting CoreBluetooth logic; it still works with CB while also allowing to write mock implementations;
We can now instantiate mock data for our previews quite easily

The bad:

Dat boilerplate doe
Views now need to be generic to accommodate which data source is in use (actual BT or mock). Could be alleviated by using an intermediate struct that wraps only the values the view needs (title, id, etc.)

So all in all, it's a small win but could still do with some more work.
I also need to still write some more mocks and overhaul a few more previews.

Hi, @MrSkwiggs! Thanks for your work! I'm starting to the final changes 🥳

Thanks for the review @artemnovichkov ! Will get back to it as soon as I've got some time :)


Troublesome index settings[BUG]

Describe the bug
A 5xx was caused by creating an index with these settings
To Reproduce
Create an index with these settings:
index_settings = {
                    "index_defaults": {
                        "treat_urls_and_pointers_as_images": True,
                        "model": 'open_clip/coca_ViT-L-14/laion2b_s13b_b90k',
                        "normalize_embeddings": True,
                        "text_preprocessing": {
                            "split_length": 10,
                            "split_overlap": 2,
                            "split_method": "sentence"
                        },
                    },
                'number_of_shards':60,
        }

this error can be reproduced on `mainline` using the provided `index_settings`

I can't reproduce the error on the mainline. maybe it's fixed in PR (https://github.com/marqo-ai/marqo/pull/387)


Make it available for Jupyter Notebook

Hi, it is really useful and interesting.
Wondering if can have it for Jupyter Dash?

It should just work with Jupyter Dash, yes :)

@Function-X0 Have you ran into any trouble using it with Jupyter Dash? If so, please let us know!


Uncaught SyntaxError: Cannot use import statement outside a module

Hi. I have attempted to add Buffer to my browser extension in the name of replacing the deprecated btoa method so that I may submit a pull request to Buildspace, whom have provided the extension's template. I have tried on multiple occasions in multiple ways to get the buffer to work but to no avail. The file in which I am trying to use buffer is the index.js at the root level of the app. Below I have attached screenshots and a link to my extension's repository so you can see the various changes I have made in attempting to get this to work. If you need any further info, please let me know. Thank you for your assistance.
https://github.com/SMSonGitHub/gpt3-writer-extension-sms

I have recently installed webpack and that seems to make the errors go away. I am currently testing the app, but it seems there may still be some issues. Any and all good help would be appreciated.

I have installed a webpack manager recently that may be the solution to my issue.


GH-44479: [CI][Archery] Add missing Flight integration targets

Rationale for this change
Go, Rust -> C++, Java, C# don't exist.
What changes are included in this PR?
Add Go, Rust -> C++, Java, C# pairs.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

GitHub Issue: #44479

https://github.com/apache/arrow/actions/runs/11775797590/job/32796921021?pr=44691#step:10:13686
Integration: Test: Flight: C++ -> C++
Integration: Test: Flight: C++ -> Java
Integration: Test: Flight: C++ -> C#
Integration: Test: Flight: Java -> C++
Integration: Test: Flight: Java -> Java
Integration: Test: Flight: Java -> C#
Integration: Test: Flight: C# -> C++
Integration: Test: Flight: C# -> Java
Integration: Test: Flight: C# -> C#
Integration: Test: Flight: C++ -> Go
Integration: Test: Flight: C++ -> Rust
Integration: Test: Flight: Java -> Go
Integration: Test: Flight: Java -> Rust
Integration: Test: Flight: C# -> Go
Integration: Test: Flight: C# -> Rust
Integration: Test: Flight: Go -> C++
Integration: Test: Flight: Go -> Java
Integration: Test: Flight: Go -> C#
Integration: Test: Flight: Rust -> C++
Integration: Test: Flight: Rust -> Java
Integration: Test: Flight: Rust -> C#

C++, Java , C# <-> C++, Java, C#: All pairs exist: OK
C++, Java, C# -> Go, Rust: All pairs exist: OK
Go, Rust -> C++, Java, C#: All pairs exist: OK
Go, Rust <-> Go, Rust: No pairs exist: OK

+1


Mention udev in the custom base image doc page

The custom base image doesn't contain any information about starting udev and running udevadm trigger - these are vital steps for a working system so we should mention this at least.

this info should perhaps be mentioned in the runtime environment

Documented here: https://docs.resin.io/runtime/runtime/#access-to-dev


Add: Neon Devnet and Neon Mainnet

This PR adds the Neon Devnet and Mainnet to Snapshot
Neon EVM was launched to public on 17th July 2023 during ETHCC
References:
Mainnet: https://chainlist.org/chain/245022934
Devnet: https://chainlist.org/chain/245022926

Just want to also double check that start: refers to the starting block to index from? If so then my implementation should be correct.

Hey @ChaituVR
Just submitted 2 requests, one for each network
Just an FYI, the IPFS field doesn't accept ipfs:// addresses, I had to use a https:// address via cloudflare
Thanks

Hey @joehquak To support these networks the network request should be coming from the Neon EVM team itself

Hey @joehquak To support these networks the network request should be coming from the Neon EVM team itself

Hi @ChaituVR I'm a partner of Neon however if you like I can get one of the team members to comment on this PR?

Better to respond directly on the form in docs, they can add dummy data on network fields, I will merge these both requests into one


What if there are multiple replicates for control condition?

Hi,
I was trying to analyse a CRISPR screen data with multiple replicates of control condition. I have 4 replicates of control conditions (and 4 for treated/test condition). So I was using this command:
python BAGEL.py fc -i {path to input} -o {path to output} -c 1,2,3,4

Then I got this error
Traceback (most recent call last):
  File "BAGEL.py", line 165, in <module>
    ctrl_column = int(arg)
ValueError: invalid literal for int() with base 10: '1,2,3,4'

It seems to me that bagel (in the current version) is not applicable to data with multiple replicates of control condition. Please correct me if I am wrong.
Thank you. (bagel works very well if there is only one control condition.)

Hi again,
What if I analyse the 4 replicates of control conditions and 4 replicates for treated/test condition in a paired manner. i.e.
1st control vs 1st test condition
1st control vs 1st test condition
1st control vs 1st test condition

then aggre

Hi rraadd88,
You can use 4 replicates when you use updated version of BAGEL. It is on the branch of this github. build_111
Thank you.

Hi @rooeikim ,
Thank you very much for the reply. build_111 branch indeed supports multiple controls.
I find bagel very useful in CRISPR screens. Great work!
Thanks again.


how to set response header easily?

how to set response header easily?
i know ctx.HTML() sets content type html but how do i override c.String() coz i want output as text/html

All renders are automatically populated with content-type, ex. c.String() -> "text/plain; charset=utf-8"; c.Json() -> "application/json; charset=utf-8"


init malfunctioning after one wrong password attempt

lib init stops working after one wrong password attempt.
Expected Outcome
$  lib init -f

Welcome to stdlib! :)

To use the stdlib registry, you must have a registered StdLib account.
It will allow you to push your services to the cloud and manage environments.
It's free to create an account. Let's get started!

Please enter your e-mail to login or register.

? E-mail<EMAIL_ADDRESS>
Welcome back, omarchehab98!
Please enter your password.

? Password ********               <- wrong password

Error: Invalid credentials

? Password ********               <- correct password

Logged in successfully! Retrieving default Library Token (API Key)...
Active Library Token (API Key) set to: Library Token

Congratulations!
Your stdlib development environment has been initialized.

Use lib create <service> to create a new (local) service package.
or type lib get <service> to download an existing service package.

Additionally, use lib help to see more commands.

Happy building! :)

Actual Outcome
$ lib init -f

Welcome to stdlib! :)

To use the stdlib registry, you must have a registered StdLib account.
It will allow you to push your services to the cloud and manage environments.
It's free to create an account. Let's get started!

Please enter your e-mail to login or register.

? E-mail<EMAIL_ADDRESS>
Welcome back, omarchehab98!
Please enter your password.

? Password ********               <- wrong password

Error: Invalid credentials

? Password ********               <- correct password

Error: Invalid credentials

$  lib init -f

Welcome to stdlib! :)

To use the stdlib registry, you must have a registered StdLib account.
It will allow you to push your services to the cloud and manage environments.
It's free to create an account. Let's get started!

Please enter your e-mail to login or register.

? E-mail<EMAIL_ADDRESS>
Welcome back, omarchehab98!
Please enter your password.

? Password ********               <- correct password

Logged in successfully! Retrieving default Library Token (API Key)...
Active Library Token (API Key) set to: Library Token

Congratulations!
Your stdlib development environment has been initialized.

Use lib create <service> to create a new (local) service package.
or type lib get <service> to download an existing service package.

Additionally, use lib help to see more commands.

Happy building! :)

I'll look into this early next week: there are a few outstanding items here that we need to ship fixes for on the CLI end. :)


narsil: 1.3.0-350-ga51756908 -> 7ee652d7a1284002f38ae653a91f17ceae55ba90

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for narsil is: Unofficial rewrite of Sil, a roguelike influenced by Angband
meta.homepage for narsil is: https://github.com/NickMcConnell/NarSil/
meta.changelog for narsil is: https://github.com/NickMcConnell/NarSil/releases/tag/7ee652d7a1284002f38ae653a91f17ceae55ba90
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7ee652d7a1284002f38ae653a91f17ceae55ba90 in filename of file in /nix/store/diwmas92yrdfjwyqzrx76vrmcw4kp2si-narsil-7ee652d7a1284002f38ae653a91f17ceae55ba90

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

narsil

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/diwmas92yrdfjwyqzrx76vrmcw4kp2si-narsil-7ee652d7a1284002f38ae653a91f17ceae55ba90 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A narsil https://github.com/r-ryantm/nixpkgs/archive/1966965e7d4923eeb271d9f960310bf0c58965de.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1966965e7d4923eeb271d9f960310bf0c58965de#narsil

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/diwmas92yrdfjwyqzrx76vrmcw4kp2si-narsil-7ee652d7a1284002f38ae653a91f17ceae55ba90
ls -la /nix/store/diwmas92yrdfjwyqzrx76vrmcw4kp2si-narsil-7ee652d7a1284002f38ae653a91f17ceae55ba90/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

narsil

Maintainer pings
cc @nanotwerp @x123 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Closing this, still waiting for someone to approve and merge https://github.com/NixOS/nixpkgs/pull/360714 as it's the latest proper release.


Support for gloabal default headers in axios !!

Hi I'm setting default axios headers in request interceptor but these headers are not accessible in another function... in axios documentation it is mentioned that global-axios-defaults are global...below is my sample code need help
 axios.interceptors.request.use(function (config) {
 axios.defaults.headers.accesstoken= "some_access_token"
  axios.defaults.headers.client = "some_client"
 axios.defaults.headers.uid = "some_uid"
 return config;
  },function (error) {
return Promise.reject(error);
 });
On page load componentDidmount executes but axios default headers are undefined in this function
 componentDidMount: function() {
console.log(axios.defaults.headers) #its giving me undefined
axios.get("http://some_url_for_get_request.json", {
 headers: {
accesstoken: axios.defaults.headers.accesstoken,
 uid: axios.defaults.headers.uid,
client: axios.defaults.headers.client
 }
})
 }

Default headers shouldn't be set in interceptors. You'd usually set them after loading axios (using require or whichever method).
If you set them outside the interceptor you should be able to access them from your component.


Store blocks using Dry::Core::ClassAttributes

This PR adds the ability to store blocks with Dry::Core::ClassAttributes:
class MyClass
  extend Dry::Core::ClassAttributes

  defines :say_hello
  say_hello do
    'hello world'
  end
end

MyClass.say_hello.call #=> "hello world"

I look forward to your feedback. Thanks! 😄

Just checking in on this :)

@psparrow 👋
I'm not fully convinced tbh. I can imagine where you want to have blocks that will be resolved on every access but the idea of simply capturing blocks and returning them back doesn't feel aligned with other dry gems. For instance, dry-container allows registering dependencies either with container.register('key', value) or container.register('key') { value } but in any case container['key'] returns value, not a proc.
Also keep in mind that you can register a lambda with
  say_hello -> do
    'hello world'
  end

which is not significantly longer compared to the proposed variant.
Do you have real-world use cases for this?

@flash-gordon Thanks for the feedback. That makes sense. After reviewing my use case, I realized building a DSL for defining a method is sufficient.


Action/PR: Avoid $version in checkver script

This is a patch to help us avoid using $version in checkver script, which can cause some serious errors like #8494 and #8498.

Another way to implement this is through schema.json,
but I think implementing this in PR handler is simpler and more readable.

I think we should really first investigate why the $version variables gets leaked out of the Invoke-Expression call here: https://github.com/ScoopInstaller/Scoop/blob/ac2fb38722fffca9dcea6e7faeda6f305f04ad9b/bin/checkver.ps1#L224, that too in the global scope (because it affects all subsequent checkver calls thereafter).
/cc @niheaven

Fixed by https://github.com/ScoopInstaller/Scoop/pull/4941.


No.10 page should not require / use Adobe Flash

As discovered by @joelanman, the No.10 history page appears to require Adobe Flash - with it blocked you get the below warning. It also has non standard navigation.

https://github.com/alphagov/whitehall/blob/a41ef2edaec2ce63facb2998d3d62c56252a70fe/app/views/histories/10_downing_street.html.erb#L326
As far as I'm aware, we avoid / don't allow flash on GOV.UK. Hoping this can be escalated.

Link to page: https://www.gov.uk/government/history/10-downing-street

Will discuss with the Content Community

Chatted to Abigail Waraker and they have raised this for their team to look at.

A WIP branch with the tour deleted:
https://github.com/alphagov/whitehall/compare/delete-tour
Currently awaiting new copy for page.
Undoes #474


Refactor: update eslint and it's packages

Motivation/Description of the PR

Description of this PR, which problem it solves
A link to the corresponding issue (if applicable).

Updated packages

eslint
eslint-config-airbnb-base
eslint-plugin-import
eslint-plugin-mocha

Type of change

[x] Dependencies changes/updates
[ ] Breaking changes
[ ] New functionality
[ ] Bug fix
[x] Documentation changes/updates
[ ] Hot fix
[ ] Markdown files fix - not related to source code

Checklist:

[ ] Tests have been added
[x] Documentation has been added (Run npm run docs)
[x] Lint checking (Run npm run lint)
[x] Local tests are passed (Run npm test)

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 25

Clones added
============
- lib/command/init.js  2
- lib/command/generate.js  2
- lib/plugin/puppeteerCoverage.js  1
- lib/command/configMigrate.js  1
- lib/plugin/stepByStepReport.js  1
- lib/plugin/screenshotOnFail.js  1

Clones removed
==============
+ lib/helper/SeleniumWebdriver.js  -1
+ lib/command/gherkin/steps.js  -1
+ test/rest/REST_test.js  -1
+ test/rest/ApiDataFactory_test.js  -1
+ lib/command/gherkin/snippets.js  -1
+ lib/helper/Nightmare.js  -1

See the complete overview on Codacy

So far I like it, however, merging it will make conflict for next PRs.

Which one?
Write after merges, so I'll update this

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 62

Clones added
============
- lib/command/init.js  2
- lib/plugin/screenshotOnFail.js  1
- lib/command/configMigrate.js  1
- lib/plugin/stepByStepReport.js  1
- lib/plugin/puppeteerCoverage.js  1
- lib/command/generate.js  2

Clones removed
==============
+ lib/command/gherkin/steps.js  -1
+ lib/helper/SeleniumWebdriver.js  -1
+ test/rest/REST_test.js  -1
+ lib/command/gherkin/snippets.js  -1
+ test/rest/ApiDataFactory_test.js  -1
+ lib/helper/Nightmare.js  -1

See the complete overview on Codacy

pulled master, please, re-review @DavertMik , @PeterNgTr

Thank you!


How can I change the port when I used the Administration API

Expected Behavior / New Feature
when I used the admin Api,bind the path http://localhost:57063,not the 5000,I succeed to get a token,but when I want to get the config, it it error.  where is it I must change the default port 5000?
WebHost.CreateDefaultBuilder(args)
.UseStartup().UseUrls("http://localhost:57063")
Actual Behavior / Motivation for New Feautre
Steps to Reproduce the Problem
1.IOException: IDX10804: Unable to retrieve document from: 'http://localhost:5000/administration/.well-known/openid-configuration'.
Microsoft.IdentityModel.Protocols.HttpDocumentRetriever+d__8.MoveNext()
InvalidOperationException: IDX10803: Unable to obtain configuration from: 'http://localhost:5000/administration/.well-known/openid-configuration'
Specifications

Version:
Platform:
Subsystem:

@dasajia2lang this is in the global configuration BaseUrl
{
    "ReRoutes": [],
    "GlobalConfiguration": {
        "BaseUrl": "http://localhost:57063"
    }
}

Hope this helps!

Going to close the issue, let me know if still have problems.

@dasajia2lang I’m not sure, how does the authorization header look?

@TomPallister ,Authorization:Bearer {{token}},this is the format.my get method is ok,

@dasajia2lang if you set logging level to trace, can you paste the log here?

@TomPallister ，I used postman to send a request ,it do not work ,but I used the code to request  it is ok!!
thanks help!!

@dasajia2lang awesome! I will close this issue for now :)


This issue is not an issue!

Hi @kevin-powell!
When I go into the fonts folder, I see a file called alaska.tff.
You bought this font. And, it's available for anyone who is on this repository for free. Shouldn't this be unaccessible for people who don't pay for the font, @kevin-powell?
EvanWebDev

I Googled it and the Alaska font is free to download for anyone, so this issue is incorrect!

Actually, the regular Alaska font can be downloaded for free. However, for the variable font, you have to pay! And, the font is available to anyone to download for free!

Thanks for pointing that out, will be fixing that now 👍


Unsupported png file format

While creating an event, you can upload a picture only in the jpg file format. We use png format everywhere else; therefore, I need to export the design in two different file formats.
Moreover, when you want to edit the event, you have to upload the picture once again (unlike the title, description, etc.).

Adding support for PNG files sounds useful. As far as I know, it should also be quite a simple fix.
As for the bug with readding event images, we are aware of that (#110) and plan to fix it.

it should also be quite a simple fix

The backend is hacky af when it comes to storing pictures, I wouldn't be so sure about it being a simple fix.


✨ Update version references for release 0.1.4

Summary
This PR updates the various version references in preparation for release 0.1.4.
Related issue(s)
Fixes #

BTW, I do not understand why the Makefile says version 0.1.1 instead of 0.1.3.

/cc @nirrozenbaum
/cc @pdettori
/cc @vMaroon

@MikeSpreitzer the commit is not signed so merging is blocked..

@nirrozenbaum: thank you for the LGTM and approval. I do not understand what is going wrong wrt the signing. I signed and pushed this commit the same way I sign all my other commits:
mspreitz@mjs13 ocm-transport-plugin % git commit -a -s
[prep-for-0.1.4 3e0df23] Update self-references for release 0.1.4
 3 files changed, 88 insertions(+), 5 deletions(-)

mspreitz@mjs13 ocm-transport-plugin % git push -u origin prep-for-0.1.4
Enumerating objects: 9, done.
Counting objects: 100% (9/9), done.
Delta compression using up to 12 threads
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 4.09 KiB | 4.09 MiB/s, done.
Total 5 (delta 4), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (4/4), completed with 4 local objects.
remote:
remote: Create a pull request for 'prep-for-0.1.4' on GitHub by visiting:
remote:      https://github.com/MikeSpreitzer/kubestllar-ocm-transport-plugin/pull/new/prep-for-0.1.4
remote:
To https://github.com/MikeSpreitzer/kubestllar-ocm-transport-plugin.git
 * [new branch]      prep-for-0.1.4 -> prep-for-0.1.4
branch 'prep-for-0.1.4' set up to track 'origin/prep-for-0.1.4'.

And Git seems to recognize that this commit is indeed signed by me:

There does seem to be something different between the commits in this repo vs those in ks/ks: the former are all marked "Verified" but the latter are not all marked "Verified".

/lgtm
/approve


The widget rebuilds too much times when scrolling

The Swiper rebuilds too many times when scrolling which cause performance issues when the widgets inside of the swiper are not just images.

Same here, got any fixes?


Create LICENSE

It would be great if the project would include a license file to make it immediately clear which license this project is under.

Thanks @Findus23


chore: attempt to upgrade to latest docusaurus

lord save me
Please describe the changes this PR makes and why it should be merged:
If applicable, please reference Discord API Docs PRs or commits that influenced this PR:

This flopped and might not happen. Who knows!


Dribbblish theme sidebar icons broken with recursive playlist folders

Hi,
Recently, the Dribbblish theme has stuffed up, the icons on the sidebar aren't showing. I've seen other users with the same issue but you've closed all their issues and told them to reinstall it.
I have reinstalled the theme, following all the commands in the readme. I even downgraded to an older version of Spotify. I've reinstalled Spicetify. The sidebar icons are still broken. I'm using Windows 10.
Looking at the console, there appears to be an error in the loadPlaylistImage function in the dribbblish.js extension.
Uncaught TypeError: Cannot read property 'href' of undefined
    at loadPlaylistImage (dribbblish.js:30)
    at dribbblish.js:54
    at i (zlink.bundle.js:59)
    at waitForElement (dribbblish.js:4)
    at i (zlink.bundle.js:59)

Line 30 of the .js file is const href = link.href.replace("app:", "");
Something somehow has broken and I can't seem to fix it (and I'm guessing other users too).
Hopefully you are able to provide a fix.

Ok I found the issue. One of my playlist folders when opened, stuffs up all the icons, and they are fixed when it is closed. I think it's because that playlist folder has other playlist folders inside of it.

Huh, I think I well tested both playlist folder and recursive folder cases. Can you temporarily disable dribbblish.js and screenshot your playlist folder structure?
Also you don't have to downgrade Spotify version. It's just make it harder to debug.

I am experiencing the same issue

I also have this issue.
Sidebar icons are not showing they just show a small text

I am having the same issue as everyone else... only text is showing : (

@BehdadEs , @vandan-school  https://github.com/morpheusthewhite/spicetify-themes/tree/master/Dribbblish#how-to-install

Had earlier experienced the same issue at first and tried going through the installation again to no avail.
Took hint from this issue that it could have had something to do with the dribbblish.js file. After further inspection into what was being run, I noticed that there was an error raised with regards to this particular dribbblish.js javascript file. Upon further investigation into the file itself, I realised that for some reasons, when pulling from github, another file is pulled instead of the javascript file that was required.
What I did to solve my issue was to locate the corresponding file on github and replaced the one that is in the Extensions. I suggest that @BehdadEs and @vandan-school give this a try to see if this would fix their issue.

Having the same issue, tried this but didn't work for me. Also tried reinstalling

Had earlier experienced the same issue at first and tried going through the installation again to no avail.
Took hint from this issue that it could have had something to do with the dribbblish.js file. After further inspection into what was being run, I noticed that there was an error raised with regards to this particular dribbblish.js javascript file. Upon further investigation into the file itself, I realised that for some reasons, when pulling from github, another file is pulled instead of the javascript file that was required.
What I did to solve my issue was to locate the corresponding file on github and replaced the one that is in the Extensions. I suggest that @BehdadEs and @vandan-school give this a try to see if this would fix their issue.

@Sparoney You are correct in that the bug is in dribbblish.js, a very simple traceback through the commits with 0 js knowledge and you can guess where the error was merged in. Fixed mine but would prefer author to fix this issue him/herself as this is her repo.

@khanhas i was able to fix the issue on my end by adding a if statement after link is defined.
`
for (let i = 0; i < sidebarItem.length; i++) {
const item = sidebarItem[i];
const link = item.getElementsByTagName("a")[0];
if (link === undefined) { continue }
const href = link.href.replace("app:", "");
        if (href.indexOf("playlist-folder") != -1) {
            const button = item.getElementsByTagName("button")[0]
            button.classList.add("Button", "Button--style-icon-background", "Button--size-28",);
            item.setAttribute("data-tooltip", item.innerText);
            link.firstChild.innerText = item.innerText.slice(0, 3);
            continue;
        }

        Spicetify.CosmosAPI.resolver.get({
            url: `sp://core-playlist/v1/playlist/${href}/metadata`,
            body: { policy: { picture: true } },
        }, (err, res) => {
            if (err) return;
            const meta = res.getJSONBody().metadata;
            item.firstChild.className = "playlist-picture"
            item.firstChild.style.backgroundImage = `url(${meta.picture})`;
            item.firstChild.setAttribute("data-tooltip", item.textContent);
        });
    }

`

@ChoogleGrome What exactly did you do? Because I have the exact same thing as you showed in the screenshot.

@ChoogleGrome Thanks for the fix. I don't know how that case would ever happen but if it works on yours, maybe it would work for others.
dribbblish.js update was pused. If you have this problem, please git pull or download newest theme package to update.

@ChoogleGrome What exactly did you do? Because I have the exact same thing as you showed in the screenshot.

I added a if statement after link was defined in dribbblish.js. I saw in the console that the link variable was coming up undefined, so I basically just skipped one iteration of the loop if it was undefined and it seemed to work. @vandan-school

That makes sense. Thanks again.


Hello World

I can create "Hello World" prgram in several languages like C, CPP, JAVA, JS, PY, etc..etc.. inside sepeate folders of the language.
For example, inside C folder -> HelloWorld.c
Inside Java folder -> HelloWorld.java
and so on.

YES @saubhagya-gupta

Hi @PratishShrivastava, Added this PR : https://github.com/Code-Outside/hacktober-fest-2022/pull/6, Please review.

You just need to add 1 languages Hello world program, please change it. @saubhagya-gupta


Unable to Scrape Movie for R18

I'm trying to match some JAV from R18 start with STARS but returned nothing.. Noticed this behaviour since last mmonth.. I checked the actress is still at R18. Any idea why it failed?
STARS-435

Hi there,
I also not receive any result when find with "SCUTE-1015" (code with 4 number) from R18.

@aka-fork You have to search it with the exact ID: Javinizer -Find 'SCUTE-1015-kotone' -R18

Rename your file to SCUTE-1015-kotone and you can sort it with Javinizer -Path . -Strict


Remove date from pages front matter, normalize last_review_date

Closes https://github.com/giantswarm/giantswarm/issues/14401
Changes the templates to use the last commit date from the git repo for each file.

This currently breaks in our build process.

I think that with the way we copy files from the original path to a different build path, we are missing the .git dir.

Closing, will start over


also test push_str ptr stability (the original report)

@bors r+

:pushpin: Commit 177c0d3baa1d44db6ff574147a62b8b487d65f1e has been approved by RalfJung

:hourglass: Testing commit 177c0d3baa1d44db6ff574147a62b8b487d65f1e with merge d25589f0bdcbde0b270c6d251f3fe850e7759233...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing d25589f0bdcbde0b270c6d251f3fe850e7759233 to master...


sync projects

monthly repo sync
Test URLs:
Homepage

Pre Migration Link: https://adobe.com/express/
Before: https://main--express-milo--adobecom.hlx.page/express/
After: https://sync-repos--express-milo--adobecom.hlx.page/express/

Quick Action Page

Pre Migration Link: https://adobe.com/express/feature/image/remove-background
Before: https://main--express-milo--adobecom.hlx.page/express/feature/image/remove-background
After: https://sync-repos--express-milo--adobecom.hlx.page/express/feature/image/remove-background

Templates

Pre Migration Link: https://adobe.com/express/templates/
Before: https://main--express-milo--adobecom.hlx.page/express/templates/
After: https://sync-repos--express-milo--adobecom.hlx.page/express/templates/

Create

Pre Migration Link: https://adobe.com/express/create/story/instagram
Before: https://main--express-milo--adobecom.hlx.page/express/create/story/instagram
After: https://sync-repos--express-milo--adobecom.hlx.page/express/create/story/instagram

Extra wide feature added to columns

Before: https://main--express-milo--adobecom.hlx.page/drafts/echen/education
After: https://sync-repos--express-milo--adobecom.hlx.page/drafts/echen/education

New Banner Variant

Before: https://main--express-milo--adobecom.hlx.page/drafts/kennethlum/banners-demo
After: https://sync-repos--express-milo--adobecom.hlx.page/drafts/kennethlum/banners-demo

Codecov Report
Attention: Patch coverage is 64.70588% with 18 lines in your changes missing coverage. Please review.

Project coverage is 58.11%. Comparing base (f8f8497) to head (8c882e8).
Report is 10 commits behind head on main.

Files
Patch %
Lines

express/blocks/banner/banner.js
52.38%
10 Missing :warning:

...ss/blocks/fullscreen-marquee/fullscreen-marquee.js
77.77%
4 Missing :warning:

express/scripts/branchlinks.js
0.00%
3 Missing :warning:

express/scripts/instrument.js
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
- Coverage   60.42%   58.11%   -2.32%
==========================================
  Files          23       22       -1
  Lines        3694     3791      +97
==========================================
- Hits         2232     2203      -29
- Misses       1462     1588     +126

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


NDTensors.jl incompatible with Optim.jl

Hello everybody,
I am trying to minimise a function of some variables and quantum state by ITensors.jl's means to measure an MPS. However, the Optim.jl package has a dependency on FiniteDiff.jl which in turn has a dependency on ArrayInterface.jl which in turn overloads the method setindex of NDTensors.jl. This throws an error when using the elementary method setelt() in ITensors.jl
An example:
julia> using ITensors, Optim

julia> i = Index(4)
(dim=4|id=70)

julia> setelt(i=>1)
ERROR: MethodError: setindex(::NDTensors.Tensor{Float64,1,NDTensors.Empty{Float64,NDTensors.Dense{Float64,Array{Float64,1}}},IndexSet{1,Index{Int64},Tuple{Index{Int64}}}}, ::Float64, ::Int64) is ambiguous. Candidates:
  setindex(T::NDTensors.Tensor{var"#s79",N,StoreT,IndsT} where StoreT<:NDTensors.Empty where IndsT where var"#s79"<:Number, x, I...) where N in NDTensors at ~/.julia/packages/NDTensors/YYVnF/src/empty.jl:99
  setindex(x::AbstractArray{T,1} where T, v, i::Int64) in ArrayInterface at ~/.julia/packages/ArrayInterface/hIHGL/src/ArrayInterface.jl:106
Possible fix, define
  setindex(::NDTensors.Tensor{T,1,StoreT,IndsT} where StoreT<:NDTensors.Empty where IndsT where T<:Number, ::Any, ::Int64)
Stacktrace:
 [1] setindex!!(::NDTensors.Tensor{Float64,1,NDTensors.Empty{Float64,NDTensors.Dense{Float64,Array{Float64,1}}},IndexSet{1,Index{Int64},Tuple{Index{Int64}}}}, ::Float64, ::Int64) at ~/.julia/packages/NDTensors/YYVnF/src/empty.jl:106
 [2] setindex!(::ITensor{1}, ::Float64, ::Int64) at ~/.julia/packages/ITensors/ka2Nl/src/itensor.jl:706
 [3] setelt(::Pair{Index{Int64},Int64}, ::Vararg{Pair{Index{Int64},Int64},N} where N) at ~/.julia/packages/ITensors/ka2Nl/src/itensor.jl:473
 [4] top-level scope at REPL[22]:1

I think it would be great to be able to use Optim.jl in conjuction with ITensors.jl. I have however no idea if the proposed solution by the compiler Possible fix, define   setindex(::NDTensors.Tensor{T,1,StoreT,IndsT} where StoreT<:NDTensors.Empty where IndsT where T<:Number, ::Any, ::Int64) would be suitable
Cheers
Jan

Sorry, I just noticed this. I'm not sure why I don't get notifications about issues raised, I'll have to look into my settings.
Thanks for pointing that out. It would be great to have ITensors working with Optim. I'll look into the best way to get it working (it may be just adding the solution that Julia recommends).

This should be fixed on master and will be included in the next ITensors release (v0.1.38), please let me know if you continue to have problems like this.

Hi,
I'm not sure whether I should be posting in this Issue or opening a new one - apologies if the latter. Today I tried using ITensors.jl to do some calculations and had this same problem. If I use a package which depended on ArrayInterface.jl and then tried to run, for instance, the first half of the DMRG example script https://itensor.org/docs.cgi?vers=julia&page=getting_started/dmrg :
using ITensors
let
  N = 100
  sites = siteinds("S=1",N)

  ampo = AutoMPO()
  for j=1:N-1
    ampo += "Sz",j,"Sz",j+1
    ampo += 1/2,"S+",j,"S-",j+1
    ampo += 1/2,"S-",j,"S+",j+1
  end
  H = MPO(ampo,sites)
end

I would get the same error. I fixed it by extending your fix to accept two integer arguments
# This is needed to fix an ambiguity error with ArrayInterface.jl
# https://github.com/ITensor/NDTensors.jl/issues/62
@propagate_inbounds function setindex(T::EmptyTensor, x, I::Int, J::Int)
  return _setindex(T, x, I, J)
end

I assume this happens because when constructing the MPO from the AutoMPO, ITensors needs to set two indices at once when instantiating the local operators, and the previous solution only fixed it when setting one index.

Thanks for pointing that out, I'm fixing it now (more generically by defining setindex(T::EmptyTensor, x, I::Int...)).


Rewrite 3.10 security for migration to 4.0

Rewrite the existing 3.10 security framework to utilize the new 4.0 security framework. This needs to incorporate migrating to current users to the new 4.0 sign-up page to create a new account and update their profile.

This needs to go the other way. Migration should be to the new security model and pages to old pages, selectively. Not making old pages to use new security model. Those old pages need to be rewritten as 4.0 versions using 4.0 security model.

rewriting 3.10 security is being scraped.
3.10 security is being dumped completely
4.0 security will handle all permissions for 3.10 and 4.0


Enclave: Test exits and withdrawals

For this, the client/test Client needs to be extended to also support sending exit and withdrawal transactions.
Can then also use this in #39

Resolved by #49


Fixed point decimal multiplication for DictionaryArray

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

We can do fixed point decimal multiplication for DecimalArray. We need to extend it to DictionaryArray.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

label_issue.py automatically added labels {'arrow'} from #4136


LGTM.com - false positive c# This assignment to is useless, since its value is never read.

Description of the false positive
The variable is clearly used, I think it's probably confused because of the type dynamic
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/ekonbenefits/dynamitey/snapshot/32d21b116695780312da079f180861c8195c3944/files/Dynamitey/Internal/Optimization/InvokeHelper.cs?sort=name&dir=ASC&mode=heatmap#x819805487acb5956:1

Thank you for your report, we appreciate it.
I can confirm this is a false-positive, and we're working on a fix.

The bug actually lies in the way we compute the control-flow graph. We didn't consider the cases after the default: to be reachable, but actually they are.

The fix submitted by @calumgrant should eradicate the false positive you've filed. Please let us know if the problem persists. Thanks!


@Positive and @Max

I found out that you do not support @Positive and @Max annotations at the same time. I know that @Positive could be changed to @Min(1), but it's definitely is not the idiomatic way.

Unfortunately it is not possible to "combine" two (or more) randomizers for a single field (in the end, a single randomizer will be selected to randomize each field). This is a known limitation documented here.

So maybe you need to change architecture? When user of your library use those 2 annotations he expects some behaviour. Justifying bugs with we can't fix it it's poor way to resolve it.

Saying "Currently it is not possible to do X.. This is a known limitation documented here" is not justifying a bug. It is acknowledging a known issue.
If you look at Java's know issues, you can see for example:

Currently it is not possible to programmatically invoke default interface methods using the JDI API.

If you programmatically invoke default interface methods using the JDI API and it would not work as you expect, do you think Oracle folks are poorly justifying a bug with that statement?
Combining randomizers is not a bug, it is a feature that has been requested in #285. I did my best to come up with a clean solution but I could not find one. If you have an idea of how to do it, you are welcome to suggest it, I am always open to any improvement.

Maybe add handling of @Max annotation to PositiveAnnotationHandler?

That would lead to code tangling between annotation handlers. If we follow this approach, we would also need to handle @Min in NegativeAnnotationHandler and all other combinations which is obviously not the way to go.
I should mention that Easy Random provides several way to cover your use case:

You can use a custom RandomizerRegistry where you implement the combinations you need by leveraging existing randomizers
You can use a custom a RandomizerProvider which is the main entry point for providing randomizers (this is more powerful than RandomizerRegistry as it give you access to the randomization context).
You can also register a custom randomizer for your field. This is the simplest solution IMO. Here is an example:

Predicate<Field> positiveWithMaxFieldPredicate =
        FieldPredicates.isAnnotatedWith(Positive.class)
                .and(FieldPredicates.isAnnotatedWith(Max.class));
EasyRandomParameters parameters = new EasyRandomParameters()
   .randomize(positiveWithMaxFieldPredicate, new MyRandomizer());

EasyRandom easyRandom = new EasyRandom(parameters);


feat: Display whether an error happened during render or init

Codecov Report

Merging #201 into master will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master     #201   +/-   ##
=======================================
  Coverage   98.14%   98.14%
=======================================
  Files          27       27
  Lines         270      270
=======================================
  Hits          265      265
  Misses          5        5

Impacted Files
Coverage Δ

src/components/base/bind.js
100% <100%> (ø)
:arrow_up:

src/components/base/render.js
94.44% <50%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Added Shazam

Proposed Changes

Added Shazam

Checklist

[x] Tested locally
[x] Ran yarn ci to test my code
[x] Did not add any unnecessary changes
[x] All my changes appear after other changes in each file
[x] Included a screenshot (if adding a new button)
[x] 🚀

Thank you!


Configuration step for DJGPP/Watt32 does not work

Hi there, attempting to build OpenSSL on linux using the DJGPP/Watt32 toolchain does not work as intended
Simply running the following configure statement:
# ./Configure no-threads -DOPENSSL_DEV_NO_ATOMICS --prefix=$HOME/djgpp DJGPP
Configuring OpenSSL version 1.1.1u (0x1010115fL) for DJGPP

renders the following output:
#  perl configdata.pm --dump

Command line (with current working directory = .):

    /usr/bin/perl ./Configure no-threads -DOPENSSL_DEV_NO_ATOMICS --prefix=/root/djgpp DJGPP

Perl information:

    /usr/bin/perl
    5.34.0 for x86_64-linux-gnu-thread-multi

[cut]

Config target attributes:

[cut]
    CFLAGS => "-fomit-frame-pointer -O2 -Wall",
[cut]
    cflags => "-I/dev/env/WATT_ROOT/inc -DTERMIOS -DL_ENDIAN",
[cut]
    lflags => "-L/dev/env/WATT_ROOT/lib",

[truncated]

the above configuration obviously does not work.
I'm not sure what distribution/os exposes /dev/env/* but certainly not ones I'm familiar with

that being said, the following proposed patch corrects what I believe the configuration developer intended
diff --git a/Configurations/50-djgpp.conf b/Configurations/50-djgpp.conf
index a8853a81..b2809054 100644
--- a/Configurations/50-djgpp.conf
+++ b/Configurations/50-djgpp.conf
@@ -7,9 +7,9 @@ my %targets = (
         inherit_from     => [ asm("x86_asm") ],
         CC               => "gcc",
         CFLAGS           => "-fomit-frame-pointer -O2 -Wall",
-        cflags           => "-I/dev/env/WATT_ROOT/inc -DTERMIOS -DL_ENDIAN",
+        cflags           => "-I$ENV{'WATT_ROOT'}/inc -DTERMIOS -DL_ENDIAN",
         sys_id           => "MSDOS",
-        lflags           => add("-L/dev/env/WATT_ROOT/lib"),
+        lflags           => add("-L$ENV{'WATT_ROOT'}/lib"),
         ex_libs          => add("-lwatt"),
         bn_ops           => "BN_LLONG",
         perlasm_scheme   => "a.out",

this also applies to the 3x branches as well

This works in the CI cross-builds since the Watt-32 include path there is added
by a gcc.specs file.  The /dev/env paths are then ignored - they only exist
in a native build.
Proposed patch looks good to me, that should work for all configurations.

That would work, yes.  Even nicer would be to omit the -I/-L flags
altogether in that case.

The version 1.1.1 is not supported anymore. Closing. If this is still an issue with currently supported releases, please open a new issue.


Add DynamicTable Schema

This issue is blocked by the following issue:

#320
#272
#111

ref: https://github.com/pdfme/pdfme/discussions/148
https://github.com/MP70/pdf-lib-draw-table

@hand-dot Can I ask how you're planning to draw the table on the pdf once you have added this to the schema?

@MP70
Thank you.
Firstly, regarding the support for links, I hadn't planned to include it in V4, but if it's necessary, I can add it.
https://github.com/pdfme/pdfme/issues/319
The idea of performing drawing via callbacks is very interesting. This means that the table schema would be limited to calculating the layout and style of the table, and the content of the table would be drawn through callbacks, right?
As for measuring the height and width of text, I'm not yet sure which method to use, but generally, the text schema of pdfme, including the dynamic font size feature, works well. It functions correctly not only with Latin characters but also with Japanese and Chinese fonts. However, it's not perfect. There was a recent bug report titled 'Line breaks in RTL languages'.
What I want to clarify is what features this DynamicTable Schema will have. I want to define the requirements before writing any code. I also want to make sure that these features are not overly complex.
Pdfme aims to be a user-friendly PDF library. Therefore, I think it's necessary to discard overly specialized features. The code you've written was originally developed for in-house use and then extracted. Before integrating it into the core library of pdfme, I want to clarify what is necessary and what is not.
Is there a way to check what features the current pdf-lib-draw-table supports and what requirements it had?

No worries. Apologies for the delay in response, it's been a busy New Year!
"The idea of performing drawing via callbacks is very interesting. This means that the table schema would be limited to calculating the layout and style of the table, and the content of the table would be drawn through callbacks, right?"
Yes. Schema-wise in draw table, it could be a type of cell content to add, similar to how we have customStyledText, or image/link. We might add something like customDrawnContent { MeasureCallback: void, DrawCallback: void}. It requires some thought, but that's what I was vaguely picturing. On the other hand, draw table already exposes a way of drawing for all primitives except shapes, e.g.,

Text,
MultiLine Text (where you choose the line breaks),
Custom Styled Text,
Images,
Links,

which you could use today with no modification needed to the draw table library.
"As for measuring the height and width of text, I'm not yet sure which method to use, but generally, the text schema of pdfme, including the dynamic font size feature, works well. It functions correctly not only with Latin characters but also with Japanese and Chinese fonts. However, it's not perfect. There was a recent bug report titled 'Line breaks in RTL languages.'"
Fair enough, draw table has only been tested in LTR Latin languages, but RTL and Japanese/Chinese fonts should, in theory, work. A corner case might be on word splitting.
"Pdfme aims to be a user-friendly PDF library. Therefore, I think it's necessary to discard overly specialized features. The code you've written was originally developed for in-house use and then extracted. Before integrating it into the core library of pdfme, I want to clarify what is necessary and what is not."
As it happens, this was actually written from the start as an open-source package. The general idea was that it should be simple to use for a basic use case but enable a decent amount of customization and overriding where the user needed. As such, all 'options' in draw table are exactly that - optional with defaults built in so you can choose to pass an empty options object if you like. It's up to you if you only want to expose a subset of draw table's features to your users. I would recommend exposing some bare minimum things like font, text size, has header, header background, border color, and so on, but it would be your choice. You can see this in the project readme.

@MP70
Thanks for your reply.
I have put together some ideas on how to implement a table in the following article, so please refer to it if you have time.
Implementation Ideas for Dynamic Tables in pdfme
I'm not sure how much of the above ideas can be applied to draw table at this point, but it seems necessary to clarify that.
I will find time to simply incorporate into pdfme what is already achievable with draw table and give it a try.
I will check what kind of UX it will be and, if it looks good, I would like to implement it using draw table as it is.
I will update you if there are any changes.

I have a question.
Is it possible to add width to drawTable?
It is possible to calculate width using startX and pageMargin, but by adding width to drawTable, I think it will be simpler.
In pdfme, the size of the table is determined using the x, y coordinates and width, so it is necessary to be able to specify the width.
Also, it would be more user-friendly if there is an option like cellWidth in jsPDF-AutoTable to specify the width of columns.

@MP70
The ability to set a background color for the 'td' element would be beneficial.
https://github.com/pdfme/pdfme/assets/24843808/887ae745-6063-4eb9-bbe9-d993829f20f7
ref: https://github.com/pdfme/pdfme/pull/405

@MP70 The ability to set a background color for the 'td' element would be beneficial.
My.Movie.mp4
ref: #405

Totally agreed! I think from memory the only reason why I didn't was there was no clear place in the schema to pass it because it's not a table setting, nor a cell content :) It's trivial to add in once I work out the best place for the setting to go as it's already done for the header.
Would per row setting be fine or is per cell be better?

@hand-dot

I have a question.
Is it possible to add width to drawTable?
It is possible to calculate width using startX and pageMargin, but by adding width to drawTable, I think it will be simpler.
In pdfme, the size of the table is determined using the x, y coordinates and width, so it is necessary to be able to specify the width.

Yeah I mean the margins were designed to be a more user friendly way of saying 'stop x' and 'stop y' but performing the calculations for the user so they didn't have to.
So for now you can just do page width - startx - table width to arrive at the page margin to pass to draw table. Maybe stopX and stopY would of been simpler after all, could probably add them in as an OR type with page margin, thanks so much for the feedback!

Also, it would be more user-friendly if there is an option like cellWidth in jsPDF-AutoTable to specify the width of columns.

This already exists in overrideWidths: number[] where each number in the array is the width of each column. If you're manually setting column widths then you must provide a width for every column. Is there something about this interface that doesn't work for you, anything I can improve?

@MP70

This already exists in overrideWidths: number[] where each number in the array is the width of each column.
I missed it! Thank you.

Can i see  the source code of https://pdf-lib-table-demo.vercel.app/?
I'd like to know how to achieve page break.

@hand-dot could be this one: https://github.com/MP70/pdf-lib-draw-table

@hand-dot could be this one: https://github.com/MP70/pdf-lib-draw-table

this is a source code of the pdf-lib-draw-table library, i'd like to know source code of demo app.
i couldn't find behavior of a page break from https://github.com/MP70/pdf-lib-draw-table.

@hand-dot
Page Break
You're right in that we let users do the page breaking because then they can do it the way they like, rather than imposing 'the one way' but it does provide the tools to make it super simple, and I will be happy to share some examples with you. If we can find a common setting that most users want, without overcomplicating the library it could potentially be in scope for inclusion.
The demo does page breaking if a render fails due to overflowing table, so it is quite simplistic. However I would expect you'd want to know if you're going to overflow a page before you actually send off for rendering the table on a pdf because you might want to know for the designer?
Background Color
I have just popped this in the table options under options.row. I don't really love that home for it, but it gets you what you need. It's an array of color or undefined.
so this
      table.options.row = {
        ...table.options.row,
        backgroundColors: [table.options.header.backgroundColor,undefined,table.options.header.backgroundColor],
      }

would set row 1 to have a color, row 2 no color, and row 3 a color.
NB: This setting is only for non header rows, so index 0 color will be the color applied to row 0 in a no header table, and row 1 in a header table.

Thats released as 0.75, you'll need to upgrade.

Maybe just use simple approach instead of pushing it automatically, maybe let user define max row for each table in a page.
We might need to create repeat for many page but I think it's more straight forward.
Then we just need to check total row and count which page need to be print.

Hey @wiryonolau
Interesting. Can you elaborate on this?

I mean let the user handle all the row count, cell width , row height design adjustment.
If it break the page they will need to readjust the design.
For example :
page 1 - table only 5 row
page 2 - table only 10 row
page 3 - table only 10 row
we then can create 3 version , a template that only use page 1, a template that only use page 1 and 2 and so on.
When given data with same identifier that has 12 row, it means it require template with 2 page. first page will only replace first 5 row, second page the rest of the row.
It's a bit stupid but I think it will works for the user.
I would love to have the auto page break when adding new row like excel, but I think it's over complicated for a printout report.

Hey @wiryonolau,
Thank you for the proposal.
I am considering offering an implementation for auto page break.
I have already made some progress: https://add-dynamic-table-schema-332.vercel.app/
It's complex and there are bugs, but I want to proceed in this direction.

One thing I wanted to add in case it hasn't been tracked, but I'm noticing that the placement of the tables in the pdf from Generate PDF sometimes differs from what's on the Designer. This is whenever I add the property to the table schema "readOnly: true", then call "designer.updateTemplate(template)"

The DynamicTable Schema has been released as Beta in pdfme V4.
For more details, please refer to the following documentation:
https://pdfme.com/docs/tables

Hello , @hand-dot  do you have any example to share on how to generate the pdf using a template that has a table ?


ValueError: Cannot align indices with non-unique values on Query 05

Running with internal dataset both SF100 and SF1000 with success. But using a self-generated dataset from bigBench dataGen's raw files runs into the following error:
Encountered Exception while running query
Traceback (most recent call last):
File "/fsx/install/tpcx-bb-nvidia/tpcx-bb/tpcx_bb/xbb_tools/utils.py", line 280, in run_dask_cudf_query
config=config,
File "/fsx/install/tpcx-bb-nvidia/tpcx-bb/tpcx_bb/xbb_tools/utils.py", line 61, in benchmark
result = func(*args, **kwargs)
File "queries/q05/tpcx_bb_query_05.py", line 266, in main
results_dict = client.compute(*ml_tasks, sync=True)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/client.py", line 2930, in compute
result = self.gather(futures)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/client.py", line 1992, in gather
asynchronous=asynchronous,
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/client.py", line 833, in sync
self.loop, func, args, callback_timeout=callback_timeout, **kwargs
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/utils.py", line 340, in sync
raise exc.with_traceback(tb)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/utils.py", line 324, in f
result[0] = yield future
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/tornado/gen.py", line 735, in run
value = future.result()
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/distributed/client.py", line 1851, in _gather
raise exception.with_traceback(traceback)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/dask/optimization.py", line 963, in call
return core.get(self.dsk, self.outkey, dict(zip(self.inkeys, args)))
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/dask/core.py", line 151, in get
result = _execute_task(task, cache)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/dask/core.py", line 121, in _execute_task
return func((_execute_task(a, cache) for a in args))
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/dask/dataframe/methods.py", line 271, in assign
df[name] = val
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/contextlib.py", line 74, in inner
return func(*args, **kwds)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/cudf/core/dataframe.py", line 750, in setitem
self.insert(len(self._data), arg, value)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/contextlib.py", line 74, in inner
return func(*args, **kwds)
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/cudf/core/dataframe.py", line 3001, in insert
self._index, how="right", sort=False
File "/home/ec2-user/anaconda3/envs/aws-rapids-tpcx-bb/lib/python3.7/site-packages/cudf/core/series.py", line 4161, in _align_to_index
raise ValueError("Cannot align indices with non-unique values")
ValueError: Cannot align indices with non-unique values
More analysis into the dataset found that the new dataset has an extra column of  null_dask_index for the index and 2 num_row_groups instead of 1 in the shared parquet file.

Thanks for filing. Likely due to upstream changes in the parquet writer in between now and when the succeeding dataset were generated.

I can reproduce this error with a fresh clone and environment

This error arises due to the reset_index(drop=True) call here. Dask now uses a __null_dask_index when the parquet metadata includes it as a placeholder, but it's not playing cleanly with the setitem + reset index on the series.
I've just verified that we can safely remove this and still pass correctness checks for this query, so I suspect it only exists due to an old bug we had to work around.


No autocompletions for Parent class when using ES2015 mixins (function returning the Class)

Consider this code:
class Parent {
  parentMethod() {  }
}

const Mixin = Base => class Mixin extends Base {
  mixinMethod() {  }
}

class Child extends Mixin(Parent) {
  childMethod() {
    this.
  }
}

For the Child class, all three method are available, but typescript doesn't detect the parentMethod to be available.

There may be a better way to do this, but the main problem is that Base is of type any for your mixin. If you give it a generic type so it can flow through, parent methods will show up.
type Constructable = new (...args) => any;

class Parent {
  parentMethod() {  }
}

const Mixin = <T extends Constructable>(Base: T) => class Mixin extends Base {
  mixinMethod() {  }
}

class Child extends Mixin(Parent) {
  childMethod() {
    this. // all three show up here in playground
  }
}

Nice! I tried to implement this using jsdoc syntax, but without luck, any ideas?
/** @typedef {new (...args) => any} Constructable */

class Parent {
  parentMethod() {  }
}

/**
 * @template T extends Constructable
 * @param {T} Base
 */
const Mixin = (Base) => class Mixin extends Base {
  mixinMethod() {  }
}

class Child extends Mixin(Parent) {
  childMethod() {
    this. // still only two autocompletions
  }
}

I think the problem here is that TypeScript doesn't have a way of expressing constraints on type parameters in JSDoc.
In other words:
/**
 * @template T extends Constructable
 */

doesn't actually add a constraint.
@dimvar I see that you mentioned some work on bounded generics here for Closure. Any word on that?

No progress on that yet.

Is there anywhere that "types I can describe in TS but not JSDoc" are listed?

OP works in latest on Playground

@RyanCavanaugh
Please explain more details


chore: extend the tuple type

Extends the 'TupleType' so it conforms to Typescript

@aladdin-add you merge this if you are happy :)


Consider adding CSSComb to automatically format stylesheet

Link - https://github.com/csscomb/csscomb.js
There would need to be a conditional to limit it to the unminifed stylesheet only.
Here's how I'm currently using it - example.
WPCS config:
{
    "remove-empty-rulesets": true,
    "always-semicolon": true,
    "color-case": "lower",
    "block-indent": "\t",
    "color-shorthand": false,
    "element-case": "lower",
    "eof-newline": true,
    "leading-zero": true,
    "quotes": "single",
    "space-before-colon": "",
    "space-after-colon": " ",
    "space-before-combinator": " ",
    "space-after-combinator": " ",
    "space-between-declarations": "\n",
    "space-before-opening-brace": " ",
    "space-after-opening-brace": "\n",
    "space-after-selector-delimiter": "\n",
    "space-before-selector-delimiter": "",
    "space-before-closing-brace": "\n",
    "strip-spaces": true,
    "unitless-zero": true,
    "vendor-prefix-align": true
}

Yes, seems like something which could be included in the default build:css task as long as the stylesheet is outputting in expanded style. The WPCS config could also be stored within the toolkit as with .eslintrc, .stylelintrc etc with an optional override if the file exists in the theme root.
Might be good to use the PostCSS version, and do something like:
    if ('expanded' === outputConfig.outputStyle) {
      postProcessors.push(csscomb( /path/to/custom/config ));
    }

https://www.npmjs.com/package/postcss-csscomb

Awesome, I'll get started on this.

Working on this here https://github.com/seothemes/gulp-wp-toolkit/tree/csscomb
Not really sure how to go about testing this so any feedback would be much appreciated!

@seothemes what is the source of the .csscomb.json file that you pasted above? Some of the rules seem to conflict with the stylelint-config-wordpress package which is used for linting.
For example, the ruleset above converts string quotes to single, but the Stylelint config is looking for double.

I'm not sure to be honest, it was a while ago. Single quotes aren't right so it may need to be double checked. I just did a search and found this old gist which appears to be what's needed https://github.com/cedaro/grunt-wp-css/blob/develop/tasks/config/default.json

BTW, thanks for adding this so quickly! Really nice work. Going to switch all themes over to this now (and give credit of course). Much cleaner than using the big old Gulp file.


Update ome-advanced-expiration.md

Changed O365 to Office 365.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@KCCross Please merge this PR, thanks.


Uncaught expression when opening entry point

Discord username
No response
What type of issue is this?
Permanent – Ocurring repeatably
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[X] Yes, previous version of Enso did not have this issue.

What issue are you facing?
Clicking on the link on the page with entry points leads to an error popup.
The chosen entry point is opened afterward without issues.
Expected behaviour
No error visible
How we can reproduce it?

Open entry points list: Enso -startup.entry=_
Click on any link in the list

Screenshots or screencasts

Enso Version
0abeb354e29a8490242690ee0b344dae3616edb8
Browser or standalone distribution
Standalone distribution
Browser Version or standalone distribution
standalone
Operating System
MacOS
Operating System Version
MacOS Ventura 13.0.1
Hardware you are using
M1 Mac

Closing as it should have been fixed by #6507.


update-ami does not honor auth config

Describe the bug
when defining a region in global auth:
general:
  auth:
    me-south-1: mes1

taskcat ignores this profile and uses the default profile, fails with:
[ERROR  ] : Your templates use the following regionsand no credentials were detected for them
[ERROR  ] : ME-SOUTH-1
[ERROR  ] : This can lead to inconsistent results. Not querying the API.
[ERROR  ] : Verify your taskcat auth config or opt-in region status
[ERROR  ] : - https://aws-quickstart.github.io/taskcat/#global-config
[ERROR  ] : - https://docs.aws.amazon.com/general/latest/gr/rande-manage.html

Closing for now. If you can confirm this is present in the latest development release, I'm happy to dig at it.


Update output-clause-transact-sql.md

Fix markdown oddity: @MyTableVar``table -> @MyTableVar table.

@watfordgnf : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Looks good, thank you for the update!


Hi, Is this superoptimizer hardware irrelvant?

Hi, want to know is this superoptimizer hardware relvant or irrelvant.

Oh, I mean will souper optimize differently on different target cpu, let make different optimization decision based on target. Like hardware-aware

no, it won't do anything like that, it works at the level of LLVM IR and has no specific knowledge of the target

you might look instead at this project, however https://github.com/minotaur-toolkit/minotaur

Thank you, yes, minotaur! The problem is I am working on arm specific, so probably need to figure out how to move minotaur on arm.


Update GitHub Actions dependencies

build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

8a1afee5b70e5dc41cf5c21c01bfcee77ddb6364


Continuous Deployment to functions failing with "Failed to fetch Kudu App Settings. Unauthorized (CODE: 401)

I set up continuous deployment in resource creation. My networking access restrictions are set to "Allow all." But, I get the following errors when it tries to publish:
Execution Exception (state: ValidateAzureResource) (step: Invocation)
When request Azure resource at ValidateAzureResource, Get Function App Settings : Failed to acquire app settings from https://<scmsite>/api/settings with publish-profile
Failed to fetch Kudu App Settings. Unauthorized (CODE: 401)
      Error: Failed to fetch Kudu App Settings. Unauthorized (CODE: 401)     at Kudu.<anonymous> (/home/runner/work/_actions/Azure/functions-action/v1/lib/appservice-rest/Kudu/azure-app-kudu-service.js:69:23)     at Generator.next (<anonymous>)     at fulfilled (/home/runner/work/_actions/Azure/functions-action/v1/lib/appservice-rest/Kudu/azure-app-kudu-service.js:5:58)     at processTicksAndRejections (node:internal/process/task_queues:96:5)

Is this related to the recent issues with pull requests and actions or am I missing something I need to do in addition to setting up continuous deployment in the azure portal?

I'm facing same issue. My github actions file is:
name: .NET Core アプリケーションを関数アプリ fun-mockapitest にビルドしてデプロイする
on:
  push:
    branches:
    - main
env:
  AZURE_FUNCTIONAPP_NAME: fun-mockapitest
  AZURE_FUNCTIONAPP_PACKAGE_PATH: publish
  CONFIGURATION: Release
  DOTNET_CORE_VERSION: 6.0.x
  WORKING_DIRECTORY: src\CQAKBMaintenanceApi\CQAKBMaintenanceApi
jobs:
  build:
    runs-on: windows-latest
    steps:
    - uses: actions/checkout@v3
    - name: Setup .NET SDK
      uses: actions/setup-dotnet@v3
      with:
        dotnet-version: ${{ env.DOTNET_CORE_VERSION }}
    - name: Restore
      run: dotnet restore "${{ env.WORKING_DIRECTORY }}"
    - name: Build
      run: dotnet build "${{ env.WORKING_DIRECTORY }}" --configuration ${{ env.CONFIGURATION }} --no-restore
    - name: Publish
      run: dotnet publish "${{ env.WORKING_DIRECTORY }}" --configuration ${{ env.CONFIGURATION }} --no-build --output "${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}"
    - name: Publish Artifacts
      uses: actions/upload-artifact@v3
      with:
        name: functionapp
        path: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
  deploy:
    runs-on: windows-latest
    needs: build
    steps:
    - name: Download artifact from build job
      uses: actions/download-artifact@v3
      with:
        name: functionapp
        path: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
    - name: Deploy to Azure Function App
      uses: Azure/functions-action@v1
      with:
        app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
        publish-profile: ${{ secrets.fun_mockapitest_3837 }}
        package: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}

It was generated by Visual Studio 2022.

The root cause of this error was disable Basic Authentication on Azure Functions' settings.
I enabled it, and the error was fixed.

I'm facing the same issue. Azure Functions consumption plan and GitHub actions.

Running into the same issue although it worked last week, Python function with an Azure Functions consumption plan:
name: XYZ

on:
  push:
    branches: ["dev", "tst", "prd"]

env:
  AZURE_FUNCTIONAPP_NAME: 'our-app-name-${{ github.ref_name }}-location'
  AZURE_FUNCTIONAPP_PACKAGE_PATH: '.'
  PYTHON_VERSION: '3.10'

jobs:
  build-and-deploy:
    runs-on: ubuntu-latest
    steps:
    - name: 'Checkout GitHub Action'
      uses: actions/checkout@v3

    - name: Setup Python ${{ env.PYTHON_VERSION }} Environment
      uses: actions/setup-python@v4
      with:
        python-version: ${{ env.PYTHON_VERSION }}

    - name: 'Resolve Project Dependencies Using Pip'
      shell: bash
      run: |
        pushd './${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}'
        python -m pip install --upgrade pip
        pip install -r requirements.txt --target=".python_packages/lib/site-packages"
        popd

    - name: 'Run Azure Functions Action'
      uses: Azure/functions-action@v1
      id: fa
      with:
        app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
        package: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
        publish-profile: ${{ secrets.AZURE_FUNCTIONAPP_PUBLISH_PROFILE }} # Remove publish-profile to use Azure RBAC
        respect-funcignore: true
        scm-do-build-during-deployment: true
        enable-oryx-build: true

AZURE_FUNCTIONAPP_NAME resolves to our-app-name-dev-location

@jeremydvoss @runceel @aarsan @yflement
Basic auth with publish profile must have been turned off for your org, so please use RBAC credential to run this deployment task: https://github.com/Azure/functions-action#using-azure-service-principal-for-rbac-as-deployment-credential


update articles and annotations languages when UI lang is not the default

when UI locale is not the default value, all articles titles are undefined.
The source is that CVReader.onLanguageComponent() was called once, before the articles are loaded (maybe it's a race condition and doesn't happen every time, didn't investigate).
My change does two things :

set CVReader.language to not throw an error if CVLanguageManager doesn't exist yet. I belive it is the intended behavior as it's generally discouraged to have properties getters throw errors and I couldn't find anywhere it was supposed to.
call CVReader.updateArticles() if the language manager is ever loaded before the meta component. Prevents an hypothetical race condition
if articles are updated and the language is loaded, set the language using CVReader.updateLanguage()

I made the minimal viable change but there is more to do on this subject. I'll make more pull requests when I can.

Made very similar changes for annotations, I thought they should be together.
For annotations, this.language seems always defined. It made me question my assumptions, maybe there is some deeper bug here? Let me know if you think it deserve more investigations.

Hi @sdumetz , thanks for highlighting the inconsistent behavior. I'd like to investigate a little further to make sure we are consistent on all fronts.

Hi @sdumetz finally had a chance to look at this. Noticed there was also an issue with UI tooltips in this case and found that the solution for all of the issues is even simpler. There's a bug in the language manager.
Remove this line: https://github.com/Smithsonian/dpo-voyager/blob/1a1e7185e3a577be42d903fbe7d8bdcfd7fbff82/source/client/components/CVLanguageManager.ts#L113
and everything should work as expected. Please let me know if it works for you as well.

Works like a charm, thanks!

Great, thanks for highlighting the issue. Fix will be included in the next release.


In the generated site, when searching with Ctrl + K, the results are not complete

Hello, oleeskild
In the generated site, when searching with Ctrl + K, the results are not complete.

Thanks for letting me know @Gonglja! I've released a fix for the template. Instructions for how to update it can be viewed here: https://dg-docs.ole.dev/getting-started/06-updating-the-template/
Let me know if that fixes it for you or not.

Yes, I just updated and it shows up normally
thanks


Add badges and fix codecov

Codecov Report

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

@@            Coverage Diff            @@
##             master     #804   +/-   ##
=========================================
  Coverage          ?   20.11%
=========================================
  Files             ?      767
  Lines             ?    62831
  Branches          ?    10980
=========================================
  Hits              ?    12641
  Misses            ?    48520
  Partials          ?     1670

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 38ad2ef...4c55b11. Read the comment docs.


error TypeError: chatMessage._getType is not a function. node_modules/langchain/dist/chains/conversational_retrieval_chain.js:67:33

I got this error from this function: ConversationalRetrievalQAChain.fromLLM

Could you provide additional context about how you're using the "ConversationalRetrievalQAChain.fromLLM" function? Understanding the flow of your code and the context in which this function is called will greatly help in diagnosing and resolving the issue.


Shared library example

Hi,
In your example "cmake-examples/01-basic/D-shared-library/", what happens when we call main? Where does it look for the library? Can we define it somehow? When we package the software, is putting the library in the same folder as main enough?
Thank you for the examples,

By default when CMake builds a library it will set the RPATH in the binary with the location of the library. If you later do an install or generate an installer CMake will strip the build time RPATH from the binary
See https://gitlab.kitware.com/cmake/community/wikis/doc/cmake/RPATH-handling
You can see this being set when doing a VERBOSE build
make VERBOSE=1
...
/usr/bin/c++     CMakeFiles/hello_binary.dir/src/main.cpp.o  -o hello_binary -Wl,-rpath,/path/to/workspace/cmake-examples/01-basic/D-shared-library/build libhello_library.so
...

And if you inspect your binary with objdump you can see it set

objdump -p hello_binary
objdump -p hello_binary
hello_binary:     file format elf64-x86-64
...
Dynamic Section:
NEEDED               libhello_library.so
NEEDED               libc.so.6
RUNPATH              /path/to/workspace/cmake-examples/01-basic/D-shared-library/build


Dark alerts in dark mode border should be 100% gray 10

RE: https://modus-bootstrap.trimble.com/components/alerts/#basic-alerts

cc @soniakaukonen @egunther39

I made a PR for this:
https://bitbucket.trimble.tools/projects/TMDS/repos/modus-bootstrap/pull-requests/282/overview

Fixed in v1.5.5


Make components optional

Also turn everything including processor and annotator into revisors.  Time them all.

FYI @BeckySharp, you might want to check the end result, definitely not each commit.  In particular, see the EidosSystem class and the components and refiners packages.


Add Django Bridge

Project Information

Project Name: Django Bridge

Project URL: https://github.com/django-bridge/django-bridge and https://django-bridge.org/

Description:
The simple way to build React frontends for Django applications

Criteria

Is the project new?

[ ] Yes
[x] No

How long has the project been maintained?
4 years as a closed source project, 6 months as open source

How many releases has it had if it's a library or package?
14 as an open source project (including pre-releases)

Are you the author or are you submitting the project on behalf of a company?

[x] I am the author
[ ] I am submitting on behalf of a company
[ ] Other (please specify)

What makes it awesome?
It's provides simpler way to build React frontends for Django, which is often overcomplicated when done manually. This library allows you to use regular Django views as a backend for a vanilla, standard React frontend. This gives you the ability to use these two frameworks together with very little overhead.

It's been used to build SaaS applications and internal enterprise applications.

Looks good @kaedroho. Thanks for the PR.


Refactor CMakeLists.txt so that it will be possible to build cpp tests with different options, e.g. with OpenMP support

Refer to https://github.com/microsoft/LightGBM/pull/3555#discussion_r579187716.
Possibly related:

https://github.com/dmlc/xgboost/pull/4323#issuecomment-480162720
https://github.com/dmlc/xgboost/pull/4897
https://github.com/dmlc/xgboost/pull/5871

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.

Opening this issue due to active PR: #4489.


Update to 0.66.0

@beepsoft Can you please check that nothing has changed for you in react with the latest release? I've stopped assuming that all of fontawesome has been imported and the component now uses targeted imports with the vue-fontawesome component.
So - you should be able to remove the global fontawesome import from your component to.
https://github.com/describo/crate-builder-component/releases/tag/v0.66.0

https://github.com/describo/crate-builder-component-react/blob/main/src/DescriboCrateBuilder.tsx#L13-L15
should no longer be needed

Thanks, I will check it!


Go thru potential XSS strategies to look for vulns

Useful resources from the OWASP cheat-sheet:
https://cheatsheetseries.owasp.org/cheatsheets/DOM_based_XSS_Prevention_Cheat_Sheet.html
https://cheatsheetseries.owasp.org/cheatsheets/XSS_Filter_Evasion_Cheat_Sheet.html#dom-based-xss


v1.2.0

Coverage remained the same at 100.0% when pulling 1668550d95769645df9b070d0444217e60c42daa on dev into 978cf44a68442645b9581f9de82dae8e8152f498 on master.


bug: Blank line at EOF with powershell_es

Neovim version (nvim -v)
0.9.4
Operating system/version
Windows 10.0.19045
Add the debug logs

[X] I have set log_level = vim.log.levels.DEBUG and pasted the log contents below.

Log file
Log file: C:\Users\****\AppData\Local\nvim-data\conform.log
          13:09:11[DEBUG] Running LSP formatter on c:\****\****\Set-Up.ps1
          13:09:11[DEBUG] Converting full-file LSP format to piecewise format
          13:11:29[DEBUG] Running LSP formatter on c:\****\test.ps1
          13:11:29[DEBUG] Converting full-file LSP format to piecewise format
          13:11:31[DEBUG] Running LSP formatter on c:\****\test.ps1
          13:11:31[DEBUG] Converting full-file LSP format to piecewise format
          13:11:47[DEBUG] Running LSP formatter on c:\****\test.ps1
          13:11:47[DEBUG] Converting full-file LSP format to piecewise format

Formatters for this buffer:
LSP: powershell_es

Other formatters:
stylua ready (lua) C:\Users\****\AppData\Local\nvim-data\mason\bin\stylua.CMD

Describe the bug
When using format on save, a new line gets added at EOF. Disabling conform.nvim and formatting with vim.lsp.buf.format() doesn't show this behavior.
What is the severity of this bug?
minor (annoyance)
Steps To Reproduce

Open .ps1 file
Save with format_on_save enabled.

Expected Behavior
Code gets formatted but no new line added at EOF.
Minimal example file
Write-Output -InputObject "Hello, World!"
Minimal init.lua
Minimal lazy spec:

{
    "stevearc/conform.nvim",
    dependencies = { "williamboman/mason.nvim" },
    event = { "BufWritePre" },
    cmd = { "ConformInfo" },
    opts = {
        formatters_by_ft = {
            lua = { "stylua" },
        },
        format_on_save = { timeout_ms = 500, lsp_fallback = true },
        log_level = vim.log.levels.DEBUG,
    },
},

Additional context
No response

No biggy. The main reason I was using conform.nvim was because of how easy custom formatters (mainly stylua) can be integrated. I just figured lua_ls provides me with the necessary options I use for stylua out of the box, so I was able to ditch conform.nvim alltogether. You might still want to look into this. Otherwise cool plugin, I might come back later in case I have something that has no built in formatting via lsp. Cheers.

I have a suspicion about what is happening, but I'll need some more logs to confirm it. Could you format a small file that exhibits this error with log_level = vim.log.levels.TRACE and provide the logs? I just need to see what the raw output of the LSP format operation is.

I know this was originally raised for the PowerShell LSP formatter, but I'm having a very similar problem with the Bicep LSP with conform where new lines are inserted just before the final closing brace. For example, the file looks like this before saving:
resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {
    name: 'deleteLock'
    scope: resourceGroup()
    properties: {
        level: 'CanNotDelete'
    }
}

After saving, it looks like this:
resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {
    name: 'deleteLock'
    scope: resourceGroup()
    properties: {
        level: 'CanNotDelete'
    }

}

Then from the second save onwards there are two new lines at this position:
resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {
    name: 'deleteLock'
    scope: resourceGroup()
    properties: {
        level: 'CanNotDelete'
    }

}

The resulting logs of three saves in a row on the example above:
07:06:14[DEBUG] Running formatters on C:\Users\Scott\Git\Windots\test.bicep: {}
07:06:14[DEBUG] Running LSP formatter on C:\Users\Scott\Git\Windots\test.bicep
07:06:14[DEBUG] Converting full-file LSP format to piecewise format
07:06:14[TRACE] Applying formatting to C:\Users\Scott\Git\Windots\test.bicep
07:06:14[TRACE] Comparing lines { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {", "    name: 'deleteLock'", "    scope: resourceGroup()", "    properties: {", "        level: 'CanNotDelete'", "    }", "}" } and { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {\r", "    name: 'deleteLock'\r", "    scope: resourceGroup()\r", "    properties: {\r", "        level: 'CanNotDelete'\r", "    }\r", "}" }
07:06:14[TRACE] Diff indices { { 1, 6, 1, 6 } }
07:06:14[TRACE] Applying text edits: { {
    newText = "\r\n    name: 'deleteLock'\r\n    scope: resourceGroup()\r\n    properties: {\r\n        level: 'CanNotDelete'\r\n    }\r",
    range = {
      ["end"] = {
        character = 5,
        line = 5
      },
      start = {
        character = 84,
        line = 0
      }
    }
  } }
07:06:14[TRACE] Done formatting C:\Users\Scott\Git\Windots\test.bicep
07:06:17[DEBUG] Running formatters on C:\Users\Scott\Git\Windots\test.bicep: {}
07:06:17[DEBUG] Running LSP formatter on C:\Users\Scott\Git\Windots\test.bicep
07:06:17[DEBUG] Converting full-file LSP format to piecewise format
07:06:17[TRACE] Applying formatting to C:\Users\Scott\Git\Windots\test.bicep
07:06:17[TRACE] Comparing lines { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {", "    name: 'deleteLock'", "    scope: resourceGroup()", "    properties: {", "        level: 'CanNotDelete'", "    }", "", "}" } and { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {\r", "    name: 'deleteLock'\r", "    scope: resourceGroup()\r", "    properties: {\r", "        level: 'CanNotDelete'\r", "    }\r", "\r", "}" }
07:06:17[TRACE] Diff indices { { 1, 7, 1, 7 } }
07:06:17[TRACE] Applying text edits: { {
    newText = "\r\n    name: 'deleteLock'\r\n    scope: resourceGroup()\r\n    properties: {\r\n        level: 'CanNotDelete'\r\n    }\r\n\r",
    range = {
      ["end"] = {
        character = 0,
        line = 6
      },
      start = {
        character = 84,
        line = 0
      }
    }
  } }
07:06:17[TRACE] Done formatting C:\Users\Scott\Git\Windots\test.bicep
07:07:08[DEBUG] Running formatters on C:\Users\Scott\Git\Windots\test.bicep: {}
07:07:08[DEBUG] Running LSP formatter on C:\Users\Scott\Git\Windots\test.bicep
07:07:08[DEBUG] Converting full-file LSP format to piecewise format
07:07:08[TRACE] Applying formatting to C:\Users\Scott\Git\Windots\test.bicep
07:07:08[TRACE] Comparing lines { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {", "    name: 'deleteLock'", "    scope: resourceGroup()", "    properties: {", "        level: 'CanNotDelete'", "    }", "", "", "}" } and { "resource rgDeleteLock 'Microsoft.Authorization/locks@2016-09-01' = if (deleteLock) {\r", "    name: 'deleteLock'\r", "    scope: resourceGroup()\r", "    properties: {\r", "        level: 'CanNotDelete'\r", "    }\r", "\r", "}" }
07:07:08[TRACE] Diff indices { { 1, 8, 1, 7 } }
07:07:08[TRACE] Applying text edits: { {
    newText = "\r\n    name: 'deleteLock'\r\n    scope: resourceGroup()\r\n    properties: {\r\n        level: 'CanNotDelete'\r\n    }\r\n\r",
    range = {
      ["end"] = {
        character = 0,
        line = 7
      },
      start = {
        character = 84,
        line = 0
      }
    }
  } }
07:07:08[TRACE] Done formatting C:\Users\Scott\Git\Windots\test.bicep

Let me know if any further info is required.

Thanks for the logs! This confirms my suspicion that it is related to the line endings. I've pushed up what I believe to be a fix. Could you give it a try?

Brilliant! That's fixed it for the Bicep LSP. @MarcoBuess may be able to advise RE PowerShell.

Will close as fixed. Feel free to re-open if issue persists


Short text recognition

Hi,
thank you for providing this library! I am interested in very short texts like "capital Italy". With the other version of this library, i.e. https://github.com/shuyo/language-detection I got quite good results.  With this version it is different. Is it a matter of configurations? Do you have an idea what it can be?
I use:
TextObjectFactory textObjectFactory = CommonTextObjectFactories.forDetectingShortCleanText();
Here are some examples that were working in the "previeus version":

Persone nate a padova (italian)
actors from canada (english)
attori canada (italian)
Was ist die hauptstadt von kanada (german)

You can check my forked version which I've added the build.gradle for building pure Java library with Android Studio 2.2: https://github.com/eclectice/language-detector
In my version, I have added more shorttext language resources and added more shorttext data in the DataLanguageDetectorImplTest.java which needs TestNG test framework to test upon (need to enable test option useTestNG() and disable useJUnit() in the build.gradle):
    @DataProvider
    protected Object[][] shortCleanTexts() {
        return new Object[][] {
                {"en", shortCleanText("This is some English text.")},
                {"fr", shortCleanText("Ceci est un texte français.")},
                {"nl", shortCleanText("Dit is een Nederlandse tekst.")},
                {"de", shortCleanText("Dies ist eine deutsche Text")},
                {"km", shortCleanText("សព្វវចនាធិប្បាយសេរីសម្រាប់អ្នកទាំងអស់គ្នា។" + "នៅក្នុងវិគីភីឌាភាសាខ្មែរឥឡូវនេះមាន ១១៩៨រូបភាព សមាជិក១៥៣៣៣នាក់ និងមាន៤៥៨៣អត្ថបទ។")},
                {"bg", shortCleanText("Европа не трябва да стартира нов конкурентен маратон и изход с приватизация")},
                {"it", shortCleanText("Persone nate a padova")},
                {"it", shortCleanText("attori canada")},
                {"de", shortCleanText("Was ist die hauptstadt von kanada")},
                {"pl", shortCleanText("I Kanadyjczycy")},
                {"en", shortCleanText("actors from Canada")},
        };
    }


Fix 2 bugs related to AuthenticationTokenHandler (Refresh semaphore, NotifyExpiredSession)

GitHub Issue:
Proposed Changes

Bug fix

What is the current behavior?

When multiple concurrent request are unauthorized, one of the request will automatically logged out the user,
When multiple concurrent request has its session expired, it triggers NotifySessionExpired multiple times.

What is the new behavior?

When multiple concurrent request are unauthorized, we make sure that the first request refresh the authenticationtoken and the other one use the refreshed token.
When multiple concurrent request has its session expired, it triggers NotifySessionExpired only once per TAuthenticationToken.

Checklist
Please check if your PR fulfills the following requirements:

[ ] Documentation has been added/updated
[x] Automated Unit / Integration tests for the changes have been added/updated
[x] Contains NO breaking changes
[ ] Updated the Changelog
[ ] Associated with an issue

Other information

After discussion, I'm going to close this PR for the moment, becasue we're not sure that the solution is not optimal (static variables). I've started another branch with an alternate solution dev/keta/multiplerefreshwithconfig