allOf not interpreted ?

Hi,
Not sure if the issue is on the importer or API Gateway side.
I am using allOf in my swagger Model definitions.
Simple example:
  GenericUser:
    type: object
    required:
      - data
    allOf:
      - $ref: '#/definitions/Generic'
      - properties:
          data:
            $ref: '#/definitions/User'

The GenericUser Class located in the generated SDK, only contains the "data" field and not the fields from the Generic Model. There are not combined.
If models are referenced only in the allOf, then no Model class will be generated at all.
For example:
UsersList:
    type: object
    description: List of Users object as returned by Orchestrate.io
    allOf:
      - $ref: '#/definitions/Generic'
      - type: object
        required:
          - data
        properties:
          data:
            $ref: '#/definitions/Users'

This will not generate a "Users" model at all but the Model is still referenced in the UsersList generate class:
import tv.sportarchive.model.Generic;
import tv.sportarchive.model.Users;

/**
 * List of Users object as returned by Orchestrate.io
 **/
public class UsersList extends Generic{
    private Users data = null;

    /**
     * Gets data
     *
     * @return data
     **/
    public Users getData() {
        return data;
    }

    /**
     * Sets the value of data.
     *
     * @param data the new value
     */
    public void setData(Users data) {
        this.data = data;
    }

}

There is no object "Users" in my SDK.
Please advise
thank you
koxon

Can you confirm the model "Users" is created in API Gateway?

Swagger/OpenAPI import is now generally available in the API Gateway REST API, the AWS CLI and all AWS SDKs. You can also import and export Swagger definitions using the API Gateway console. This release addresses many of the open issues and feedback in this repository.
I would encourage you to migrate your workflow to the standard AWS tools. aws-apigateway-importer will receive minimal support from the API Gateway team going forward.
Any feedback or issues with the new importer should be directed to the official API Gateway forums.
Thanks,
Ryan

@koxon were you able to resolve your issue? I am trying to use the "allOf" in my swagger file and it fails to deploy to API Gateway even though it does generate the models fine.

I also see the same thing; and I'm using Swagger as well. The object which contains the allOf directive doesn't seem to get read at all. In the web console, the model simply does not show this object.


@conda-forge-admin please update version

Additional comment:
No response

Hi @bollwyvl, could you please re-trigger the version update? The release has been fixed and uploaded, now: https://pypi.org/project/pyodide-build/0.29.0/

Also, would it be alright if Hood, Gyeongjae, and I could join you as recipe maintainers for this feedstock?

Of course! :rocket: :unicorn:
I'll do a manual PR once the PyPI sdist is up, and for the forms to be observed, would probably need comments by the interested parties there to confirm the change.

This version is updated. Still awaiting additional issues/PRs for maintainers.


Release 2.1.105

On the docket:

[x] Fail on atomic_directory work_dir collision. #1905
[x] Fix build time venv pex path. #1906

Broken off from "Release 2.1.104 #1873" to make way for Pip 22.2.2 support.

Closed by #1913


remove geolite2, no longer accessible

geolite2 has been inaccessible since December 30th, this removes the crashing wget call

Re https://github.com/django/django-docker-box/issues/14 - we can potentially see if we can re-enable it later, but removing it makes sense for now.


Issue with text tags in all the application

Hi,
I've upgraded from Beta 13 to Beta 14, directly from the application, using the "Check for updates" button.
See what happened with all the text tags has an initial "!!!".
See screenshots attached.
Franz Version used when wrote message: 5.0.0.beta.14
Operating System and version: Windows 10 Pro Fall Creators Update

Ah this might be an issue with the language restructuring. Can you please try to set your language again?

Could it be because you don't have Spanish language (my system language) in your list of available languages in Franz?

I changed to a different language, and then back to English (as there's no Spanish available) and now it looks fine...

Oh yes, thank you for pointing this out. Seems like Spanish slipped through.

No, thank you for your great and helpful application!
Can live with English language, no worries.... At least the spellcheck is working (in Spanish only for now, I mean).


[Phoenix Patch] Move DummyLM to dsp/modules

This fixes a bug with Phoenix instrumentation. Phoenix is currently wrapping calls to every LM in LM.__subclasses__(), and we have a DummyLM in dspy/utils/ that makes the team for this wrapper,
From https://github.com/Arize-ai/openinference/blob/main/python/instrumentation/openinference-instrumentation-dspy/src/openinference/instrumentation/dspy/init.py
Line 68
language_model_classes = LM.__subclasses__()
for lm in language_model_classes:
            wrap_object(
                module=_DSP_MODULE,
                name=lm.__name__ + ".basic_request",
                factory=CopyableFunctionWrapper,
                args=(_LMBasicRequestWrapper(tracer),),
            )

This PR moves the DummyLM to dsp/modules to work around this.
@mikeldking @axiomofjoy @isaacbmiller @thomasahle

I think I don't really understand the philosophy for what belongs in dsp vs dspy

Thanks @CShorten!

Nice one!
On Thu, Mar 28, 2024 at 3:46 PM Xander Song @.***>
wrote:

Thanks @CShorten https://github.com/CShorten!
—
Reply to this email directly, view it on GitHub
https://github.com/stanfordnlp/dspy/pull/735#issuecomment-2026186628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLBDSHWMKPO2WKOMOFUHQTY2SFT5AVCNFSM6AAAAABFNNHIC2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGE4DMNRSHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @CShorten, just checking if this is still in draft or ready to merge? Thanks for the contribution!

Thanks @arnavsinghvi11, merged! 🚀


Update to latest swagger-parser.

This fix some issues with a bundle command JSON results.

I just released Swagger-CLI v1.0.0, which includes the latest version of Swagger-Parser.


datamanager-AnnotationPreview not working

@makseq, @hlomzik,hello
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to projects list page
Click on 'Grid' button in the project list page
See error: Spinner loaded only

Expected behavior
Display annotaions in grid view.
Environment (please complete the following information):

OS: [win10]
Label Studio Version [master]

Additional context
'https://github.com/HumanSignal/dm2/blob/master/src/components/Common/AnnotationPreview/AnnotationPreview.js'
generatePreview methed not exist

Hi @lackray, we are going to need a bit more information to reproduce this. Can you share your labeling interface, data, steps to reproduce, and possibly some screen shots? Thanks!

Hi @lackray, we are going to need a bit more information to reproduce this. Can you share your labeling interface, data, steps to reproduce, and possibly some screen shots? Thanks!

According to this file 'https://github.com/HumanSignal/dm2/blob/master/src/components/DataGroups/ImageDataGroup.js' line 12,when set root.showPreviews = true,there are no labels displayed in the gridview preview.
1.Labled someting in the labeling interface page:

Supposed to show labels preview in the project list page after showPreviews setup, but got none:

Am i missing something?


Nebula Studio can't login

Describe the bug (must be provided)
Nebula Studio can't login
Your Environments (must be provided)

OS: Linux
Studio-version: 3.2.3

How To Reproduce(must be provided)
Steps to reproduce the behavior:

I depoly Nebula Graph and Nebula Studio by Docker.
I can access nebula by nebula console.

But when I use nebula studio, I click the link button, I can't login and there's no any error message.

I have restart the service but it doesn't seem to work.

Please provide the nebula version and studio version used. and you can open chrome to check any messages, or if the interface returns data

too many session created
you need to clear the nebula session manually or automatically after the session expires. Or modifying 'max_sessions_per_ip_per_user' in nebula-graphd.conf like the message suggests
refer to doc

Thanks.


how can i change the size of a cell

something like
layout.itemSizeOptions.VerticalSize = 200
OR
layout.itemSizeOptions.VerticalSize = YBSlantedCollectionViewLayout(VerticalSize:200, HorizonSize: 200)
always made errors  like "YBSlantedCollectionViewLayout has no member VerticalSize" or
"YBSlantedCollectionViewLayoutOptions cannot be constructed because it has no accessible initializers"
so whats the right way to change the size of a cell
thanks a lot !!

Sorry it is an oversight. I released a new version to fix it.
Now with the v 2.1.1 you can set the size options. e.g :
override func viewDidLoad() {
    super.viewDidLoad()

    let layout = collectionView.collectionViewLayout as! YBSlantedCollectionViewLayout
    layout.itemSizeOptions = YBSlantedCollectionViewLayoutSizeOptions(verticalSize: 200, horizontalSize: 200)
}


Redhat init scripts for all nfp processes.

@AkashDeepSrivastava, @mak-454, @dpaks, @ashutosh-mishra - Please check the start up command in this PR and comment if there are any issues.

orchestrator script changes are fine


Support complex numbers

It would be nice to support complex numbers in typst calculation methods, which is not the case currently.

I would even argue that the generalization of complex numbers (vector2, vector3, vector4, ...) would be even more useful, specially if doing a library that draws stuff (2D/3D plots, for example)

more generalized would be to allow user to define arbitrary data type with operator overloading, but making a new whole programming language is a huge effort and easy to do incorrectly.
(Side remark, I think creating a Typst programming language was absolutely necessary because of the amount of syntactic sugar needed -- you need [...] and $...$ special syntax inside code, as well as #set, automatic concatenation etc. -- but it would also have been possible to make a precompiler into e.g. Lua -- and I think it's possible to sandbox the runtime environment of Lua to disallow interaction with operating system as well)

If operator overloading is an option, I would recommend to keep following Haskell's way and using typeclasses (similar to Rust's traits). But it would be out of scope for any kind of word processing software, you can always write some actual Haskell and generate Typst code from there

As for Lua sandboxing, it is just disabling os.execute and io.popen and you no longer can execute shell commands. Changing I/O is a bit more involved though, but still fairly simple (BTW, I would compile to a Scheme instead)

are there any plans to support it?

There are currently no plans to support this in the core language. I think this is best left to packages.


[Bug][grafana] current grafana version 9.5.2 authentication bypass vulnerability

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
our internal qualy scan reports a grafana authentication bypass vulnerability
Grafana has released patch addressing the issue. For more information please refer to  Grafana Security Advisory(https://grafana.com/blog/2023/...-cve-2023-3128/)
Patch: Following are links for downloading patches to fix the vulnerabilities:Grafana Security Advisory (https://grafana.com/blog/2023/...-cve-2023-3128/)
What do you expect to happen
due to my investigations this can be fixed by using grafana version 9.5.5 or higher
I expect a change of the used dockerfile
https://github.com/apache/incubator-devlake/blob/main/grafana/Dockerfile#L26
FROM grafana/grafana:9.5.5
How to reproduce
please see also
https://grafana.com/blog/2023/06/22/grafana-security-release-for-cve-2023-3128/
Anything else
No response
Version
v0.19.0-beta6
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for reporting. Please look at it @klesh .


AttributeKeyCategory::exportList does not exist anymore

See https://github.com/concrete5/concrete5/blob/75e5da3317b8700459265e0ff56f4a22f49bb957/concrete/src/Backup/ContentExporter.php#L73

@mlocati What are you using this for? If it's just a heads up to me, I think we ought to kill this entire library. We don't use ContentExporter any longer – we use the migration tool which works in a different, more extendable/less monolithic way.

I just have a tool that uses that class, but I'm fine if we remove it or if we fix it.
Leaving it broken is not kind :wink:

The functionality for all of these items still exists in the core but it's being called differently by the migration tool, because the migration tool lets you choose which specific items you want to export.


Does langchain support Oracle database as VectorStores?If yes, how to use the Oracle as VectorStore?

Does langchain support Oracle database as VectorStores?If yes, how to use the Oracle as VectorStore?

Generated by a 🤖
Answer
Yes, langchain supports Oracle database as VectorStores. However, the code in this repository does not specifically mention Oracle database integration. To use Oracle as a VectorStore, you would need to create a custom VectorStore class that connects to the Oracle database and implements the required methods for storing and retrieving vectors. You can use the existing VectorStore classes in langchain as a reference for creating your custom Oracle VectorStore class.
Sources
Files

docs/integrations/lancedb.md
docs/integrations/pgvector.md
docs/integrations/atlas.md
docs/modules/indexes/vectorstores/examples/vectara.ipynb
docs/integrations/vectara.md
docs/modules/indexes/vectorstores/getting_started.ipynb
docs/modules/chains/index_examples/vector_db_qa.ipynb
docs/integrations/myscale.md

Issues

#2816
#3191

PRs

#3267

This response is meant to be helpful and save users time. It is not meant to be a precise solution.
Please leave a 👍 if this is helpful and 👎 if it is irrelevant

got it. thanks. please close it.


Please take the SqlClient GitHub survey

We're surveying you, the members of the community in this repo, to learn how we were doing as maintainers. Whether you've been involved for years or you're relatively new please tell us about your experience participating and contributing to the project. The survey covers many dotnet repos, so be sure to answer for dotnet/sqlclient. It should only take a few minutes:
https://www.surveymonkey.com/r/8QTQTGJ?Source=sqlclient
Thanks!

@Wraith2 we ended up updating the original survey. If you already did that survey, it might not let you do it again. But the survey is repeated up to 3 times so that people can answer for up to 3 repos. It's possible you might be able to jump to 3rd one if you replace the code above with 8TWNFZL?Source=sqlclient

The second code worked, thanks :)

@David-Engel could you please close and un-pin this as the survey closed. Thanks to the 13 folks who responded for this repo.

Closing the issue as the survey as it is closed.


Resolves #104. Squishes layers into a Map keyed by name instead of id

This does not remove layers that do not have content, but it removes duplicate layers by condensing files with multiples of the same layer names to a single named layer with multiple ids.

We have it implemented how PDFStitcher does today (verified that this morning), so I'd say let's just get it out there. I'm not even 100% sure how we're going to detect whether a layer has objects in it yet anyway.

Ah, why are PDFs so messy?!

This pretty much applies to any protocol or standard that's been implemented ever, lol - definitely not unique to PDFs! If it's allowed, people will do it!

 Merging now.


Provide real, live examples

Participants looked for a real life example to see:

If GCDS fit the requirements they were looking for
understand further what use cases a component would be used for
a quicker way to get a project started (using multiple components together)
a way to investigate bugs in their own project.

Based on this, the need for templates and patterns are evident - though, these need may also be met using real live examples.
Related research
Developer onboarding 06/2024 - DoveTail

Note: We started this repository in the meantime. We only have one "published" example (vue3) but are working on others, some are WIP https://github.com/cds-snc/gcds-examples


Load libraries with SO version on Linux and OSX.

This is now fixed as of devel. Unified backend will still have issues because of upstream bug


Addition of multiclass adaboost algorithm

Added multiclass adaboost: added resultsToEvaluate field to classifier::Parameter, all classifiers are moved to the next interface, logistic regression resultsToCompute parameter is deprecated, removed logistic_regression::prediction::Result class.

make test

"make  test"
won


Implemented BoundedColumnValue partition function

Description
This PR adds a new implementation of PartitionFunction which is used to partition segments. The new partition function named BoundedColumnValue can be used to partition segments on column values and still generating partitionId of integer type.
Example Usage -
  "segmentPartitionConfig": {
    "columnPartitionMap": {
      "subject": {
        "functionName": "BoundedColumnValue",
        "functionConfig": {
          "columnValues": "Maths|English|Chemistry",
          "columnValuesDelimiter": "|"
        },
        "numPartitions": 4
      }
    }

PartitionId is generated based on position in columnValues. PartitionId would 1 for Maths, 2 for English and so on.
PartitionId 0 is reserved for any other subject which are not present in given columnValues. The different column values can be specified using pipe (|) separation. This additional partition functionConfig is persisted in metadata.properties and segment metadata in zookeeper.  Broker can also use this function to prune segments.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] No

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] No

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes, Adds new configuration option and changes to public interfaces.

Release Notes
Added new partition function called BoundedColumnValue to be able to partition segments based on column value.

Documentation
Yes, will create another PR in pinot-docs repo.

Codecov Report

Merging #8224 (bec3162) into master (de1cdf2) will decrease coverage by 42.18%.
The diff coverage is 7.04%.

@@              Coverage Diff              @@
##             master    #8224       +/-   ##
=============================================
- Coverage     70.98%   28.80%   -42.19%
=============================================
  Files          1626     1616       -10
  Lines         85004    84704      -300
  Branches      12791    12765       -26
=============================================
- Hits          60340    24395    -35945
- Misses        20514    58075    +37561
+ Partials       4150     2234     -1916

Flag
Coverage Δ

integration1
28.80% <7.04%> (+0.10%)
:arrow_up:

integration2
?

unittests1
?

unittests2
?

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

Impacted Files
Coverage Δ

...ot/controller/helix/core/util/ZKMetadataUtils.java
67.85% <0.00%> (-32.15%)
:arrow_down:

...processing/partitioner/TableConfigPartitioner.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ocal/indexsegment/mutable/IntermediateSegment.java
0.00% <0.00%> (-70.07%)
:arrow_down:

...local/indexsegment/mutable/MutableSegmentImpl.java
0.00% <0.00%> (-58.70%)
:arrow_down:

...ltime/converter/stats/MutableColumnStatistics.java
0.00% <0.00%> (-55.00%)
:arrow_down:

...verter/stats/MutableNoDictionaryColStatistics.java
0.00% <0.00%> (-45.00%)
:arrow_down:

...ment/creator/impl/SegmentColumnarIndexCreator.java
0.00% <0.00%> (-86.33%)
:arrow_down:

.../impl/stats/AbstractColumnStatisticsCollector.java
0.00% <0.00%> (-92.50%)
:arrow_down:

.../loader/defaultcolumn/DefaultColumnStatistics.java
0.00% <0.00%> (-76.00%)
:arrow_down:

...java/org/apache/pinot/segment/spi/V1Constants.java
0.00% <ø> (-14.29%)
:arrow_down:

... and 1192 more

Continue to review full report at Codecov.

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

Hi @kmozaid thanks for taking the time to make this contribution. Can you explain what problem this solves? Is it because  you already have a partitioning and you want to maintain locality within partitions?

Hi @kmozaid thanks for taking the time to make this contribution. Can you explain what problem this solves? Is it because you already have a partitioning and you want to maintain locality within partitions?

Hi @richardstartin , We have a table where data is being ingested from multiple sources. (these multiple sources pushes data to same kafka topic). Data is kept for 5 days in realtime table and then moved offline table by minion task. We want to keep data from these sources in separate segments for offline table. There is a column which identifies the source. BoundedColumnValue partition function provides capability to keep data from different sources in respective partitioned segments. Later if we want to backfill the data of just one source, then we will be able to do so because we would know what are the segments for given source and replace them by backfill. The main use case is to be able to backfill data of particular source. This is also discussed in slack thread - https://apache-pinot.slack.com/archives/CDRCA57FC/p1643286670255700

Thanks @Jackie-Jiang and @richardstartin for your prompt reviews, appreciate it!


google.cloud hardcoded into node snip

https://github.com/googleapis/gapic-generator/blob/e3501faea84f61be2f59bd49a7740a486a02fa6b/src/main/resources/com/google/api/codegen/nodejs/version_index.snip#L12

@alexander-fenster How do we handle this for Ads (which I assume isn't under cloud namespace)?

We don't handle this. JavaScript generation is being deprecated in gapic-generator and is moving to https://github.com/googleapis/gapic-generator-typescript. We are not going to onboard any new users to JavaScript and/or fix any issues here.

Closing the issue for there's no action item. Please feel free to continue the discussion.


Unable to set cursor caret to start of a wrapped line using the mouse.

Tested versions

Reproducible in: v4.2.1.stable.mono.official [b09f793f5] and v4.3.dev5.mono.official [89f70e98d]
Not sure if it was not reproducible in any older versions

System information
Godot v4.3.dev5.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated GeForce GTX 870M () - Intel(R) Core(TM) i7-4710HQ CPU @ 2.50GHz (8 Threads)
Issue description
Can't use the mouse to left-click and position the edit cursor/selection caret at the front of a new line that was formed from wrapping text. Should be able to click to the left of the center of the first character of word wrapped lines to position or selection from the start of the line.
Steps to reproduce
When working with a TextEdit with word wrap on, and with text that has word-wrapped to multiple lines, try left-clicking with the mouse to position the cursor into the front position of each line.  This should be doable by clicking anywhere to the left side of the starting character on the line.  On the first line, the caret position appropriately sets at the front of the line.  For any subsequent wrapped line, it positions the cursor after the first character.  You can use keyboard arrow keys to move to the correct position.  Also, if you left-click with the mouse at the end of a line that wraps, it will correctly position the caret to the start of the next line.  You just can't directly click there as would be expected.  This also means you can't select from the start of the line in a natural manner.  You have to start selecting from the end of the line above.
Minimal reproduction project (MRP)
TextEdit_MouseCursor.zip

Can you please try in 4.2.2?

duplicate of https://github.com/godotengine/godot/issues/88632


Could not resolve all dependencies for configuration ':app:debugRuntimeClasspath

Current behavior
npm i reac-native-video
react-native link react-native-video
react-native run-android
when I use do this ,it is cause an error
FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':app:debugRuntimeClasspath'.

Could not determine artifacts for com.google.android.exoplayer:exoplayer-ui:2.9.1
Could not get resource 'https://jcenter.bintray.com/com/google/android/exoplayer/exoplayer-ui/2.9.1/exoplayer-ui-2.9.1.aar'.
Could not HEAD 'https://jcenter.bintray.com/com/google/android/exoplayer/exoplayer-ui/2.9.1/exoplayer-ui-2.9.1.aar'.
Connect to d29vzk4ow07wi7.cloudfront.net:443 [d29vzk4ow07wi7.cloudfront.net/<IP_ADDRESS>, d29vzk4ow07wi7.cloudfront.net/<IP_ADDRESS>, d29vzk4ow07wi7.cloudfront.net/<IP_ADDRESS>, d29vzk4ow07wi7.cloudfront.net/<IP_ADDRESS>] failed: Read timed out

Reproduction steps
A 1, 2, 3, etc. list of what's needed to see the issue
happen.
Expected behavior
Describe what you wanted to happen
Platform
Which player are you experiencing the problem on

Android

Video sample
If possible, include a link to the video that has the problem that can be streamed or downloaded from.

添加maven仓库即可解决.原因是aliyun和谷歌的仓库中还没有最新版的exoplayer.具体操作如下:
android目录下的build.gradle中添加
maven{
url "https://jcenter.bintray.com"
}

It appears the repo is currently missing:
https://github.com/google/ExoPlayer/issues/5225
This issue has a temp solution using a different maven url.
maven {
url 'https://google.bintray.com/exoplayer/'
}

I have the same issue, was you able to resolve?
Failed to resolve: com.google.android.exoplayer:exoplayer-core:2.9.1
Failed to resolve: com.google.android.exoplayer:exoplayer-dash:2.9.1

turn off the proxy,and 'react-native run-android' maybe 5 or 10 times, I success

see the issue link and answer above

see the issue link and answer above

I have solved this problem , when my net speed is good I download the pack, I will close this issues

see the issue link and answer above

I have solved this problem , when my net speed is good I download the pack, I will close this issues

Please where in your project do you store the pack after you downloaded


LLM - Tx alerts

Add a toggle in the notifications settings to accept or not to receive transactions alerts

📝 Description
Brings tx alerts to LLM :

Add a toggle in the notifications settings to accept or not to receive transactions alerts
Manage subscription of accounts addresses to the backend service that watches for transactions and triggers the notifications

❓ Context

Impacted projects: ledger-live-mobile
Linked resource(s): LIVE-5688 LIVE-7100

✅ Checklist

[ ] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

👋  ESLint config has been unified on develop after we merged https://github.com/LedgerHQ/ledger-live/pull/3622 – this will improve the project consistency, remove eslint dependencies duplicates, and overall simplify the maintenance. This comes with a prettier library upgrade and unification of the prettierrc too, so you likely have conflicts with develop.
Hopefully, here is the procedure to easily fix this. please proceed as follow:
Step 1: sync your branch with develop up to the config changes:
2 cases

if your branch were continuously rebased with develop and don't have any merge commit, you can do

git rebase aa1e0ebc0a0498b4350754899b6a368fbd42e811

otherwise prefer

git merge aa1e0ebc0a0498b4350754899b6a368fbd42e811

then, apply the config
pnpm i
pnpm lint:fix --force

if there are remaining errors that lint couldn't fix, please fix them and pnpm lint again.
then commit in your branch.
git commit -a -m "chore: lint fixes"

Step 2: finalize the rebase/merge with develop
2 cases

if your branch were continuously rebased with develop and don't have any merge commit, you can do

git pull --rebase origin develop

otherwise prefer

git merge origin/develop

you should now be up to date. 👍

Thanks and sorry for the inconvenience,
gre


@redocly/config is missing a license

Is your feature request related to a problem? Please describe.
New internal dependency @redocly/config doesn't have a license set via package.json or a LICENSE(.md) file.

Describe the solution you'd like
An explicit license is set for the package for the avoidance of doubt.

Additional context
This was flagged by dependency license scanning.

Thanks for letting us know!


TCP Port Monitoring

⚠ Please verify that this question has NOT been raised before.

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

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hello, I have a problem with a host that constantly gives a refused connection, but I use Uptime Robot Monitoring which works very well
📝 Error Message(s) or Log
Flip UP to DOWN
connect ECONNREFUSED
🐻 Uptime-Kuma Version
1.23.11
💻 Operating System and Arch
Ubuntu 22.04
🌐 Browser
Chrome
🖥 Deployment Environment

Runtime:
Database:
Filesystem used to store the database on:
number of monitors:

1.12.11 is ancient.
We don't support such old software => please upgrade
With the level of detail you have given, that could literally be anything from docker networking to a typo in the monitor…
=> Could you fill out the form? (including the deployment environment and the error messages + log). Without this remote debugging is way more painfull than it needs to.
Have you tried https://github.com/louislam/uptime-kuma/wiki/Troubleshooting?

Sorry my version is:
Versão: 1.23.11

"Could you fill out the form? (including the deployment environment and the error messages + log)"
Where I do this?

Please edit the issue-description above and fill out the parts of the form which you ommited

Hello,
These are the errors that are constantly appearing on some monitors, which I don't understand why, if you need anything more, help me know how I can provide you with so we can resolve this problem, as the application is wonderful.
Flip UP to DOWN
connect ECONNREFUSED

Could you fill out this part of the form as well?

Deployment Environment
Runtime:
Database:
Filesystem used to store the database on:
number of monitors:

Where do I get this information? I don't know...

Do the examples in the form help? What are you stuck on precisely?

Update the description

Okay, so not docker. That closes a few tangents that I was thinking off and not old node which also closes some tangents.
Have you tried to debug if this might be related to firewalls on the network path?
You can do tcp port pings via executing telnet ip port.

No debugging, because UpTime Robot I have these same monitors and they work perfectly, you know...
That's why I didn't try your help.

Via telnet, there are times when this message is given, and there are times when it isn't...
Unable to connect to remote host: Connection refused

So you have your first hint that there is some firewall/.. in place blocking the requests. I would recomend you to walk the network path towards the monitored service and re-check where this firewall/router doing DPI is.

OK.
Now could you help me in another situation, I have a reverse proxy on it, and I changed my DNS appointment to Cloudflare, but I'm getting the error:
ERR_TOO_MANY_REDIRECTS

I disable proxy from cloudflare, set only dns, and work...

Thanks for all.


Exception during Logger instantiation in v5.3.0-v5.3.1

Since 5.3.0 version it is impossible to instantiate the Logging -  application throws an exception:
    for (var i = 0, resolved; i < filename.length; ++i)
                                          ^
                                          TypeError: Cannot read property 'length' of undefined

at Root.load (___/node_modules/protobufjs/src/root.js:192:44)
at Root.loadSync (___/node_modules/protobufjs/src/root.js:235:17)
at Object.loadSync (___/node_modules/@grpc/proto-loader/build/src/index.js:230:27)
at GrpcClient.loadFromProto (___/node_modules/google-gax/build/src/grpc.js:118:44)
at GrpcClient.loadProto (___/node_modules/google-gax/build/src/grpc.js:145:21)
at new ConfigServiceV2Client (___/node_modules/@google-cloud/logging/build/src/v2/config_service_v2_client.js:121:28)
at new Logging (___/node_modules/@google-cloud/logging/build/src/index.js:142:30)

this is because the filename variable can also be undefined because in config_service_v2_client.js
in here:
    const protos = gaxGrpc.loadProto(
      opts.fallback ? require('../../protos/protos.json') : nodejsProtoPath
    );

gaxGrpc.loadProto() method is called with just a single parameter.

Seems, the problem is in old google-gax dependency.

@koldoon I tried with 5.3.0 and couldn't reproduce. Could you share the code as well as an npm ls? It might not hurt to try a fresh rm -rf node_modules && npm install as well.


Implementing hidden friends

Implementing spec maintenance change described in issue #173 (https://github.com/KhronosGroup/SYCL-Docs/pull/42)

there are a lot of merge conflicts, probably because of the move to the new code structure. Can you resolve them, please?

Oh sorry. they are resolved now.

I think this is good, but would like to play some more with it due to the size of the changes.

I just noticed that accessor seems to be missing both of the equality operators altogether. Maybe this could also be added here..? :)

Thank you!


Plot precision-recall curve for support vector machine classifier

Hello I want to plot a precision-recall curve for SVC (support vector machine classifier), but the scikit-learn svm classifier does not implement a predict_proba method. How can I do that in scikit-plot (as far as I can see in the documentation it accepts prediction probabilities to plot the curve)?
Note that the scikit-learn documentation page has an example of precision-recall curve for SVC
Thank you,
Nikos

@foo123 For SVC in SKlearn, if you want to get the probability of each class, before you fit your model, you have to set SVC's parameter "probability" to True, by default it's False. After setting that, you train your model, and you can use SVC to predict_proba() method to get probability of each test dataset. So you can use the plot_precision_recall to get it. Try it.

@lugq1990 I use LinearSVC as well and this does not have a parameter probability. So I would need a more generic method. If using decision_function is an option I would like to learn how to use it but as far as I can tell from the documentation it returns confidence scores based on signed distance from hyperplane and not probabilities.
Furthermore using the probability parameter in SVC how am I supposed to access the probabilities from predict method? It does not have a predict_proba method. But I need predict method to return actual classes since I also plot the confusion matrix between predicted and actual classes. I am a little lost here any help or working example is highly appreciated.
Thanks

@foo123 For SVM, there is the parameter probability , so you can use it to predict the probability and prediction both. But for LinearSVC in sklearn is not supported for predict_proba.  But there is two ways to do this, one is use the basic SVC using the linear kernel, another is use the Probability calibration, it is a more general solution for estimator not supports predict_proba. Here is the link for it:http://scikit-learn.org/stable/modules/calibration.html.
For you want to get the code example, I write some basic code for you:
`from sklearn.datasets import load_iris
from sklearn.svm import SVC, LinearSVC
from sklearn.calibration import CalibratedClassifierCV
import matplotlib.pyplot as plt
import scikitplot as skplt
iris = load_iris()
x, y = iris.data, iris.target
if you want to get the probability, set probability to be True
clf = SVC(probability=True)
clf.fit(x, y)
get the prediction
pred = clf.predict(x)
get the probability
prob = clf.predict_proba(x)
you can plot the precision_recall_curve and confusion matrix
skplt.metrics.plot_precision_recall_curve(y, prob)
skplt.metrics.plot_confusion_matrix(y, pred)
plt.show()
Here is for LinearSVC
first way is to set the kernel of SVM to be 'linear' to replace the LinearSVC
clf = SVC(probability=True, kernel='linear')
clf.fit(x, y)
others is same
second way is use the CalibratedClassifierCV for more generally useful
for that not support predict_proba
svm = LinearSVC()
clf_cal = CalibratedClassifierCV(svm)
clf_cal.fit(x, y)
pred = clf_cal.predict(x)
prob = clf_cal.predict_proba(x)
others is same`

@foo123 For SVM, there is the parameter probability, so you can use it to predict the probability and prediction both. But for LinearSVC in sklearn is not supported for predict_proba. But there is two ways to do this, one is use the basic SVC using the linear kernel, another is use the Probability calibration, it is a more general solution for estimator not supports predict_proba. Here is the link for it:http://scikit-learn.org/stable/modules/calibration.html.
For you want to get the code example, I write some basic code for you:
`
from sklearn.datasets import load_iris
from sklearn.svm import SVC, LinearSVC
from sklearn.calibration import CalibratedClassifierCV
import matplotlib.pyplot as plt
import scikitplot as skplt
iris = load_iris()
x, y = iris.data, iris.target
if you want to get the probability, set probability to be True
clf = SVC(probability=True)
clf.fit(x, y)
get the prediction
pred = clf.predict(x)
get the probability
prob = clf.predict_proba(x)
you can plot the precision_recall_curve and confusion matrix
skplt.metrics.plot_precision_recall_curve(y, prob)
skplt.metrics.plot_confusion_matrix(y, pred)
plt.show()
Here is for LinearSVC
first way is to set the kernel of SVM to be 'linear' to replace the LinearSVC
clf = SVC(probability=True, kernel='linear')
clf.fit(x, y)
others is same
second way is use the CalibratedClassifierCV for more generally useful
for that not support predict_proba
svm = LinearSVC()
clf_cal = CalibratedClassifierCV(svm)
clf_cal.fit(x, y)
pred = clf_cal.predict(x)
prob = clf_cal.predict_proba(x)
others is same
`

@foo123 For SVM, there is the parameter probability, so you can use it to predict the probability and prediction both. But for LinearSVC in sklearn is not supported for predict_proba. But there is two ways to do this, one is use the basic SVC using the linear kernel, another is use the Probability calibration, it is a more general solution for estimator not supports predict_proba. Here is the link for it:http://scikit-learn.org/stable/modules/calibration.html.
For you want to get the code example, I write some basic code for you:
from sklearn.datasets import load_iris
from sklearn.svm import SVC, LinearSVC
from sklearn.calibration import CalibratedClassifierCV
import matplotlib.pyplot as plt
import scikitplot as skplt
iris = load_iris()
x, y = iris.data, iris.target
if you want to get the probability, set probability to be True
clf = SVC(probability=True)
clf.fit(x, y)
get the prediction
pred = clf.predict(x)
get the probability
prob = clf.predict_proba(x)
you can plot the precision_recall_curve and confusion matrix
skplt.metrics.plot_precision_recall_curve(y, prob)
skplt.metrics.plot_confusion_matrix(y, pred)
plt.show()
Here is for LinearSVC
first way is to set the kernel of SVM to be 'linear' to replace the LinearSVC
clf = SVC(probability=True, kernel='linear')
clf.fit(x, y)
others is same
second way is use the CalibratedClassifierCV for more generally useful
for that not support predict_proba
svm = LinearSVC()
clf_cal = CalibratedClassifierCV(svm)
clf_cal.fit(x, y)
pred = clf_cal.predict(x)
prob = clf_cal.predict_proba(x)
others is same

@foo123 For SVM, there is the parameter probability, so you can use it to predict the probability and prediction both. But for LinearSVC in sklearn is not supported for predict_proba. But there is two ways to do this, one is use the basic SVC using the linear kernel, another is use the Probability calibration, it is a more general solution for estimator not supports predict_proba. Here is the link for it:http://scikit-learn.org/stable/modules/calibration.html.
For you want to get the code example, I write some basic code for you:
`from sklearn.datasets import load_iris
from sklearn.svm import SVC, LinearSVC
from sklearn.calibration import CalibratedClassifierCV
import matplotlib.pyplot as plt
import scikitplot as skplt
iris = load_iris()
x, y = iris.data, iris.target
"if you want to get the probability, set probability to be True"
clf = SVC(probability=True)
clf.fit(x, y)
pred = clf.predict(x)
prob = clf.predict_proba(x)
"""you can plot the precision_recall_curve and confusion matrix"""
skplt.metrics.plot_precision_recall_curve(y, prob)
skplt.metrics.plot_confusion_matrix(y, pred)
plt.show()
"""
Here is for LinearSVC
first way is to set the kernel of SVM to be 'linear' to replace the LinearSVC
"""
clf = SVC(probability=True, kernel='linear')
clf.fit(x, y)
"others is same"
"""second way is use the CalibratedClassifierCV for more generally useful
for that not support predict_proba"""
svm = LinearSVC()
clf_cal = CalibratedClassifierCV(svm)
clf_cal.fit(x, y)
pred = clf_cal.predict(x)
prob = clf_cal.predict_proba(x)`

@lugq1990 Thanks, I use both SVC(linear) and LinearSVC() as they provide for different solutions with subtle differences. I dont understand how the documentation for SVC does not mention the predict_proba method even exists, but I tested it and it works. As far as calibrated probabilities are concerned, i keep it in mind but i use the train data as test data as well (in some cases) and it is mentioned that the data for calibrated probabilities should not be same as train data. So I do not use this method for LinearSVC(). Instead i used a combination of the precision-recall example from sklearn here and code from scikit-plot to make a precision-recall curve based on decision_function method. Closing this


Improve "Getting Started" page?

Original issue created as eslint/eslint#8420.
Quoting from the original issue:

I am also hoping to see a better explanation of the Getting Started article, including:

Explicitly say each rule do not support multiple options.
Explicitly say each rule has the potential to support unique severity beyond what we have.
A list of all options that each rule has. Currently, the options are written in the docs, but they are hard to distinguish against other similarly formatted text or code.
Windows users are being left out. Some of us, like me, are restricted to use Windows due to work. What is lacking is, for example, where the configuration files are at, and how to create a new .eslintrc manually without using eslint --init command (renaming a file as .eslintrc., with 1 period at the beginning and end of the filename each).

I'm very certain there may be a duplicate issue, but the most important one is the point in boldfaced text.

I agree that the Getting Started section deserves another look. Or, maybe we should keep Getting Started very high-level and then point to more of a "cookbook" page that goes into some more detail about how to do some setup and tasks. We definitely need to give this some thought, at any rate.

I haven't heard anything from the rest of the team on this, and this hasn't been a priority for me either, so I'm going to go ahead and close this.


Update requirements.txt

You haven't given me any context for this PR? What exactly are you 'fixing'? I'm rewriting singlerss to use a combination of AWS Lambda, CloudWatch, and S3, so this repo is sorta out of date.

There is no justification. I did a fork and since I took the trouble to update the dependencies myself, I thought it would be interesting to share them.
I have tested that it works fine with the new versions.
I take this opportunity to thank you for sharing the code, it has helped me a lot and I have learned some good practices from your code.

Hey, sorry for the late reply. I was really humbled to see my code helped you make your own project. This project is moving in the 'Serverless' direction at the moment. I'm gonna merge this PR, thanks! The idea with the new direction is that a user can hit the Lambda, which gets the RSS feeds as it does now, and puts it into an S3 bucket. Then the user can call the Lambda on a different API Gateway endpoint, and use it as a frontend to the S3 bucket's singlerss file. It's going to be renamed to single-rss.
Thank you again!

A very interesting objective/project.
Thanks to you!


Populate quickfix with prettier parsing errors

When prettier fails to execute we should populate the vim quickfix with parsing errors, this behaviour should also be user configurable.

Fixed by: https://github.com/mitermayer/vim-prettier/commit/2d5d3cd2585e9558ad896cbe97398315266dc030


Upgrade Inquirer

Inquirer just released a proper v1.0.0 based on promises. I would like to use this as it means we can get rid of our promise wrapper for prompts.
https://github.com/SBoudrias/Inquirer.js/releases/tag/v1.0.0

Closing because duplicate #77


feature: Fallback to bunny URL if main url doesnt work for a user

Ref - https://github.com/code100x/cms/commit/91d008e185d1eec02a683d8baf9b5c628952c3d8
https://github.com/code100x/cms/commit/d7baa2428acda6cf0b95292be025d0f47725f347

On it.


how to make clang-cl use lld?

how to make clang-cl use lld?

I'm not supporting using lld (for now at least), but you should be able to copy lld-link.exe from %programfiles%\llvm\bin to %programfiles%\llvm\msbuild-bin under the name link.exe, I belive that this will make msbuild use the lld as the linker

Some option like -Xclang -fuse-ld=lld might work, see the documet at https://lld.llvm.org/.


Matmul Example incorrect if use float32 & large matrices.

Using an Triton 2.1.0, Pytorch 2.0.1, Python 3.10, Cuda 11.8 on a pc running Ubuntu 22.04 with an nvidia 4090, if I take the example https://github.com/openai/triton/blob/main/python/tutorials/03-matrix-multiplication.py and just change the following line i.e. to use float32 format.
line 244 ->  c = accumulator
line 339 ->  a = torch.randn((M, K), device='cuda', dtype=torch.float32)
line 340 ->  b = torch.randn((K, N), device='cuda', dtype=torch.float32)
the result differs from the pure pytorch by more than the defined tolerance. This error seems to increase the larger I make M,K, and N. Particularly problematic appears to be if I make M and N very large and K small.
Using torch.float16 this works perfectly.

Are you sure that you're not comparing TF32 vs FP32 ?


KeyShared consumers with sticky hash range generate backlog with messages not fall within consumer hash range -  v2.9.2

Describe the bug
KeyShared consumers with sticky hash range generate backlog with messages not fall within consumer hash range, and that cause consumers to not receive any messages even for the ones that fall in its range .
To Reproduce
Steps to reproduce the behavior:

create Keyshared consumer with sticky has range example 35891,35899
create producer to generate message with Murmur3_32Hash hashing schema , and the key hash value is outside the consumer range , ex : 35770
Consumer will not receive message as expected , however , the message remains in the backlog
Now , let producer generate message with key hash fall within the consumer range
Consumer still not received the message

Note : IF consumer disconnect and reconnect , it will receive the expected message , however , the same scenario will be repeated for any new messages as long as the consumer has backlog for messages that does not fall within its range
Expected behavior
Consumer should have backlog for only messages that fall within its range , and should be able to consume that message
Desktop (please complete the following information):
OS: Ubuntu 18.04
Java 8
Pulsar running as systemd service
6 brokers
6 bookies
5 ZK
Additional context
Pulsar  v2.9.2
No k8 deployment , and client accessing brokers DNS directly without Pulsar proxy.

@GBM-tamerm You can't only have a key hash range [35891,35899] but don't have [0, 35890], [35900, 65535]. The key_shared consumer will not work for this case. If you only want to receive parts of messages, you can use Reader API with specific key hash range.

@GBM-tamerm You can't only have a key hash range [35891,35899] but don't have [0, 35890], [35900, 65535]. The key_shared consumer will not work for this case. If you only want to receive parts of messages, you can use Reader API with specific key hash range.

Thanks codellipenghui for reply , the mentioned issue is happening regardless ,other clients registered to other different ranges  or not ,  that issue does exist in v2.7.4 , it only appear after we upgrade to v.2.9.2


disable process.on('SIGTERM') SIGQUIT SIGINT

Discussed in https://github.com/autovance/ftp-srv/discussions/246

Originally posted by inkeytech March 10, 2021
Hello,
Would it be possible to include an option to disable the "quit" handlers of ftp-srv?
process.on('SIGTERM', quit);
process.on('SIGINT', quit);
process.on('SIGQUIT', quit);
I would like my app, using the excellent library, to be able to gracefully close the FTP server IN ADDITION TO closing other connections and processes.
With the ftp-srv event listeners in place, my own handlers fail to function.
I'm currently circumventing this by commenting out the 3 lines above, but this is not a sustainable solution.
Any pointers would be much appreciated.
Thanks,
Ian

Originally posted by **trs** March 10, 2021

This is a great point. From a library perspective it doesn't make a lot of sense to hijack the process and end it like that. Do you think keeping the SIG- events to call close would make more sense? Or just remove them completely? Either way would be a breaking change. A boolean option could be added to the constructor that would only call each process.on if true. This wouldn't be breaking and would satisfy your requirements.
class FtpServer extends EventEmitter {
  constructor(options = {}) {
    super();
    this.options = Object.assign({
      ...,
      endProcessSig: true // Better name?
    }, options);

    ...

    if (this.options.endProcessSig) {
      process.on('SIGTERM', quit);
      process.on('SIGINT', quit);
      process.on('SIGQUIT', quit);
    }
  }
}

Hello Matt,
Thank you for this.
I'm new to commenting or contributing to Github, so please excuse any faux pas.
I like your suggestion above. It would work for me, and continue to work for any existing users.
As for the option name, "endProcessSig" would work if documented.
Or possibly a more layman-ish: "handleQuitSignals"
Either way, thank you.
Regards,
Ian

These events trigger errors MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 SIGQUIT listeners added to [process]. when using new FtpSrv and then server.close() multiple times. (I'm using it in a testing context).
I'll open the PR with endOnProcessSignal as suggested here


Say commands, URL not clickable

Actual behaviour:
When you send a link url over the "Say", "Whisper" or even "Action" commands the links appear not clickable in twitch but just as raw text.
Expected behaviour:
Is there a way to make them clickable like regular link sent on web interface?
Server configuration

Operating system:
Node version (if applicable): 6.10.3
NPM version (if applicable): 3.10.10
tmi.js version: 1.2.1

tmi.js has nothing to do with how links appear on Twitch.tv. Ensure that the links are fully-formed and try again. If you can show in the frames of the websocket connection used on the Twitch website that inputted text comes in differently than it was sent from the tmi.js client, that would be useful.
How-to: Open a Twitch channel page. Open the dev tools and switch to the Networking tab. Filter by Websocket (WS) connections and click the one labeled "irc-ws.chat.twitch.tv" or similar. If you don't see any items and/or the chat has already been loaded and connected, refresh the page. Click the "Frames" tab on the details and scroll to the bottom of the list. Send an average link through the tmi.js client and see if the message shows up different in the raw IRC message (excluding escaping).

Ok sorry, it was my mistake. I tried to do it with the req headers, which was localhost on my dev environment, so it was not parsed by twitch. But trief with a real one (google.com) and it works well.
But thanks for the tips with Websocket on devtools it's really useful for debugging!
Etienne


Get Go recognized as an acceptable language for FOLIO modules

As required by MODULE_EVALUATION_TEMPLATE.MD
See https://wiki.folio.org/display/TC/Officially+Supported+Technologies
This is a bureaucratic rather than technical task, but someone needs to get on it.

The remaining part of this is #43, which is now closed as the relevant document has been written and is under review. I'll leave this open, though, until a decision is made.


webrtc integration Vs go2rtc for the picture glance

Hi,
just upgraded to 3.0.1, and after some adjustment seems working fine. A typical config I'm using is the following:
type: custom:webrtc-camera
entity: camera.cam_strada
style: 'video {aspect-ratio: 16/10; object-fit: fill;}'
mode: webrtc
ui: true

Now the question is:
how can I use picture glance card? It requires an entity to define the stream, but my entities as they are are generic cameras not processed by webrtc.
should I migrate to go2rtc addon?
Thanks

Picture Glance support formats: HLS, sync WebRTC, MJPEG
WebRTC Camera support formats: async WebRTC, MSE, MP4, MJPEG
I recommend to use WebRTC Camera card. But if you want to use Picture Glance, you can install:

WebRTC Camera integration OR go2rtc addon OR Frigate addon
RTSPtoWebRTC integration with link to http://localhost:1984/

And your Picture Glance will play Generic cameras via WebRTC technology.

Thanks Alex for clarifications. Just realized that shortcuts can be supported also on webrtc-camera, one of the reason I was evaluating Picture glance. I'll stay with webrtc integration for now. When will be more clear to me the advantages of the Add-on will eventually switch

But the state of these icons (shortcuts) will not change (unlike in the Picture Glance card), right?


implement menu designs on the frontend

I generally like the icon set we are using, but is there any alternative to this one that is more generic? like a gear icon @memosys

Let's switch to a gear icon for Settings


Update F-Droid relase

The fdroid version seems to be abandoned. Who can sort out update for the fdroid repo?

checking https://f-droid.org/wiki/page/libretasks.app looks allright.
But then searching fdroiddata emerges this:
https://gitlab.com/fdroid/fdroiddata/merge_requests/3039
So the good place for further discussion is right there.

Given that 0.7.1 seems to be available via fdroid, this.issue can be closed now?!?

Yep, closing


Automatically inserted closing bracket not skipped

I've just started using this extension and so far it's awesome! Thank you very much for your time and effort!
One problem that I have:
Usually, when I type (, vs code inserts the closing bracket automatically and if I type something like a function parameter and then type the closing bracket ) it just skips the already inserted one. Same goes when using suggestions from vs code, between the two brackets, when tabnine is disabled. On the other hand when I use tabnine suggestion and type the closing bracket, the automatically inserted bracket doesn't get skipped, but a new closing bracket is inserted like so (param)). This is quite inconvenient, since I have to move my hand to the arrow keys, or type a lot to enter vim normal mode and then skip the closing bracket and continue typing, or even use the trackpad/mouse. Could this be fixed?
I'm sorry if this issue has been posted already, but I couldn't find it in the main repo nor this one.

(looks to be already reported in #47 and #60)

(looks to be already reported in #47 and #60)

You're right. I was searching with parenthesis instead of parentheses and #60 doesn't even mention any of the keywords I searched for.

the fix was released in version 2.2.3

I'm on version 3.6.4 and experiencing the exact problem.


Blank Docx file when trying to insert HTML

I thought that I find a way to insert the content of a html, but I tried in a Python 3.12.0 with python-docx 1.0.1 and the bs4 4.12.2 and it generate the context, but when I tried with another version of Python, the Python 3.11.6 with python-docx 0.8.8 it didnt't work, then I upgraded to the latest version but still didn't work, Idk if it could be problem of the Python 3.11.6 version, I hope that's no the case.
This is according with the issue #1266.
from docx import Document
from docx.oxml import OxmlElement
from docx.oxml.ns import qn
from docx.opc.part import Part
from docx.opc.constants import RELATIONSHIP_TYPE as RT

def add_alt_chunk(doc: Document, html: str):
    package = doc.part.package
    partname = package.next_partname('/word/altChunk%d.html')
    alt_part = Part(partname, 'text/html', html.encode(encoding='utf-8'), package)
    r_id = doc.part.relate_to(alt_part, RT.A_F_CHUNK)
    alt_chunk = OxmlElement('w:altChunk')
    alt_chunk.set(qn('r:id'), r_id)
    doc.element.body.sectPr.addprevious(alt_chunk)

from bs4 import BeautifulSoup

route = "my.html"

with open(route, 'r', encoding= 'utf-8') as f:
    html_content = f.read()

soup = BeautifulSoup(html_content, 'html.parser')

doc = Document()

add_alt_chunk(doc, str(soup))

doc.save('test.docx')

I find out that the file it's correct, and it worked but only appears in Office Word, not in LibreOffice because LibreOffice doesn't have that option I guess.


lang: Build should error when PDA payer is not marked mutable

We should be able to parse the accounts at compile time and inspect the payer field to ensure it's marked mutable, giving the developer a more informative error than the current Cross-program invocation with unauthorized signer or writable account.

-> Anchor should check that the Payer is both mutable and a Signer at compile time to avoid this issue.
Ran into this problem earlier when executing an instruction with CPI using Multisig Program. This mutable/signer check does not show up when executed with the Anchor client, so some discrepancy there.

closed by https://github.com/project-serum/anchor/pull/1271


⚠ Bofrost.lu has degraded performance

In 7c3128c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1430 ms

Resolved: Bofrost.lu performance has improved in 6c807e6.


Upload: Add button and panel to upload survey

A small MR to add the upload button and panel. This can't work for now (unless combine with #97) but i check (using cherry-picking) and the upload works properly. Let me know if i should change the css tho

I change a bit the css for smaller screen than mine, could you recheck quickly? ty


Feature/dynamic errors

This PR will allow users to configure custom error messages, instead the default ones.
This also adds tests to the "tasks.js" file, as I couldn't find any.
We had a need to display custom error messages, so I thought this PR couldn't hurt.

Hi @IamTheHttp, thanks for your PR. I'm not sure about the custom error messages, is this common in tools like release-it? Any other examples?
In any case, the tests for tasks.js are obviously a very good idea so I'm definitely interested in that part.

We had issues with the current errors not being descriptive enough for all levels of developers, some were struggling with things like "no upstream" or "working directory must be clean", we wanted to change it to something more helpful such as "please add and commit before releasing" (instead of working dir..)
I didn't want to assume that our error message was correct enough to fit everyone, so I thought a configuration option/escape hatch would be useful.
Tools similar to release-it do not document any configuration for errors, but they also don't document the errors themselves most of the times.

Well, maybe we can try and let everyone benefit from descriptive and clear error messages? I'm definitely interested in improving them where possible.

Thanks @IamTheHttp! Great idea, and I've merged your PR. I took your idea a step further, I hope you like it! Let me know if you have any feedback.


[BUG] EventHubProducerClient (Azure.Messaging.EventHubs) doesn't correctly protect against oversized batches

Describe the bug
I was working on upgrading from the legacy (V4) version of the library to the new Azure.Messaging.EventHubs v5 library, and noticed that the documentation here (https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/eventhub/Azure.Messaging.EventHubs/MigrationGuide.md) claims that the new design of the library means that the TryAdd will automatically fail if the batch size is too large.  This is a nice improvement, so I assumed it worked as I refactored my code to run against v5.  My testing is done against a Basic pricing tier instance, and at first execution I immediately got an error about the size of the batch as follows:
I suspect the issue here is that the library is not factoring in the overhead required to actually send the batch, which my experience puts at 6-8%
Expected behavior
The library should adjust its sizing behavior to account for overhead to ensure that it can actually send the amount of data it plans to send.
Actual behavior (include Exception or Stack Trace)
When submitting the first batch after TryAdd fails, I receive the following:
Unhandled exception. Azure.Messaging.EventHubs.EventHubsException: The message (id:64727435, size:430284 bytes) is larger than is currently allowed (262144 bytes). (client)
at Azure.Messaging.EventHubs.Amqp.AmqpProducer.SendAsync(Func1 messageFactory, String partitionKey, CancellationToken cancellationToken)    at Azure.Messaging.EventHubs.Amqp.AmqpProducer.SendAsync(Func1 messageFactory, String partitionKey, CancellationToken cancellationToken)
at Azure.Messaging.EventHubs.Amqp.AmqpProducer.SendAsync(EventDataBatch eventBatch, CancellationToken cancellationToken)
at Azure.Messaging.EventHubs.Producer.EventHubProducerClient.SendAsync(EventDataBatch eventBatch, CancellationToken cancellationToken)
... [my code paths redacted]
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Create a batch using the standard parameters (using CreateBatchAsync());
Add Limit minus a few bytes to the batch (so for Standard tier, a couple bytes short of 1MB, for Basic tier, a a couple bytes short of 256KB).
Attempt to send batch, error appears.

Environment:

Name and version of the Library package used: Azure.Messaging.EventHubs 5.0.1
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): .NET Core SDK (reflecting any global.json):
Version:   3.1.100
Commit:    cd82f021f4

Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18363
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.100\
Host (useful for support):
Version: 3.1.0
Commit:  65f04fb6db

IDE and version : [e.g. Visual Studio 16.3]
Visual Studio Enterprise 16.4.2

Believe this one will likely start with @jsquire

Actually, after further testing, I need to expand the scope of this report.  I changed my testing Event Hub tier to Standard, and queued up a few MB of events to be transferred.  At first run, I got the following:
The message (id:20974680, size:1111395 bytes) is larger than is currently allowed (1046528 bytes). (client) | Azure.Messaging.EventHubs.EventHubsException
So in this case it looks like it's failing in either case (Basic or Standard) if you push it all the way to the limit.  Is it possible that the checks in TryAdd are not including some buffer space for the message metadata overhead?  The previous documentation talked about that, and my own internal testing previously indicated that reserving ~10% of the size for overhead would always be enough (actual numbers seem to be between 5-8%.
Also possible I'm missing something here since I had ported my code over from the prior version, but any ideas would be appreciated.  For now I've re-implemented my own size limit checks and will just stick with those.

Ok, some additional feedback here, it looks like this was ultimately a race condition in my own code where I missed awaiting an Async task and thus the TryAdd and SendBatch were being called many times over in certain cases.  However, that would seem to still point to an issue with the thread safety around TryAdd, the code in there seems to imply that it's thread safe, but my experience indicates otherwise at least in terms of testing for space in the batch before adding.  Might be worth looking into

Hi @OperatorOverload.   We're sorry that you're having difficulties and appreciate you reaching out to let us know.  You're correct in that the EventDataBatch in v5.0.1 was intentionally not thread-safe, leaving any synchronization to callers but not explicitly documenting that.
The good news is that this was something that we had recently revisited and made the decision to hold the responsibility for being thread-safe within the batch.  #11203 included those changes and will appear in our next release.

Discussion seems to have concluded, so I'm going to close this out.   Please feel free to reopen if the is additional feedback that you'd like to discuss for this topic.

Agreed @jsquire, my issues were all related to concurrency, I've implemented my own protection for now, but glad the library will get it moving forward.


Rule MD013 fails with long blocks comments in v0.27.0

Problem Description
What is actually happening
With new markdownlint v0.27.0 the MD013 rule fails with long block comments more than line length configured.
What is the expected behavior
Work as previous version markdownlint v0.26.2: MD013 rule does not fail with comment blocks longer than the maximum configured line.
Context
We are using pre-commit in our CI, and we have a workflow to detect new pre-commit hooks versions. This workflow detects the new markdownlint-cli v0.33.0, and creates a PR.

In this PR, pre-commit runs with the new markdownlint-cli hook version v0.33.0 and fails checking the MD013:

But in this file there is a comment:

I investigate a little bit...

markdownlint-cli pre-commit hook use the same version of markdownlint-cli.
markdownlint-cli updated from v0.32.2 to v0.33.0 (changelog diff)

According to the changelog of v0.33.0 the markdownlint js dependency was updated by Dependabot in this PR to the markdownlint v0.27.0 where the MD013 rule was updated too.(changelog diff

Tested in pre-commit with DavidAnson/markdownlint-cli2 v0.6.0

I have tried to find another case like tables to ignore block comments, as described in doc/MD013.md describe. But I can't find anything, related to the MD013.md rule or other global configuration.
I do not found similar case to mine (long block comment) in the test/MD011-MD021.md file

Questions

Is it a feature or is it a bug?
What could we do to ignore this case? For now, we are not going to update the version.

Thank you very much.
Our problem was that we had many files with that error. With your answer, we have proceeded to change all the files.
It might be interesting to write a test case for this case to define its behavior ? If not, we could close this issue.

Yes, I will add a test when I investigate this.

Note to self: f0bb4c639b95468181d4ad37d448694f9297b35a introduces the regression according to git bisect. Very brief visual review suggests notSpaceCrLfRe is the cause.

After some thought, I am going to leave the new behavior alone. Someone who enables the line-length rule wants to know when their lines are too long. There are some constructs today that can get an exception (code, headings, tables), but they are all things that can be difficult to reduce the line length for. Most comment scenarios should be pretty amenable to shortening (as you seem to have done in your project), so I am NOT going to add an exclusion at this time. I WILL add a test case for this like you suggest and will use that to close the issue. Thank you for bringing this up!


📝 Update docs_src/async_tests

Overview
I was following this async tests example and I got this error
/Users/tyler/code/personal/sql-translate/.venv/lib/python3.11/site-packages/httpx/_client.py:1426: DeprecationWarning: The 'app' shortcut is now deprecated. Use the explicit style 'transport=ASGITransport(app=...)' instead.
    warnings.warn(message, DeprecationWarning)

So I updated the test to match what HTTPX says to do in this example

📝 Docs preview for commit 7fc1030780738c6e957897a3fc898645a886d538 at: https://2cf514e6.fastapitiangolo.pages.dev

Hi @TylerHillery.
Thank you so much for your time and interest in helping us update the docs about the async test. 🙇
We'll be reviewing/fixing it soon. 🤓

Hi @TylerHillery.
We saw another PR fixing the same use case and already merged it. 🤓
You can see it here https://github.com/fastapi/fastapi/pull/12084
Thank you again for helping us to improve the docs! 🙇 🚀


⚠ ClamAV has degraded performance

In 53abd53, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 458 ms

Resolved: ClamAV performance has improved in 9264d10.


Update Form_Input class

Set $_tagSelfClosing to false
Any HTML tag with the class="form-control" will be automatically closed, however, this causes problems for tags that should not be closed in this way, e.g. SELECT tags.  HTML5 does not require INPUT tags to be closed this way, so setting $_tagSelfClosing to false should not cause any issues.

Thanks for your changes, and for the helpful explanations that you provided. Very much appreciated :)


add COLCON_PREFIX_PATH env var

Workaround for ros2/ros2#653.
connects to ros2/ros2#653

@dirk-thomas  here's a package containing this change, installing it in a test container it sets the COLCON_PREFIX_PATH as expected and when I create a workspace on top of it and source its install/setup.bash it does appear to source the Crystal workspace too.
ros-workspace-prerelease.tar.gz
For anyone else who encounters this file, I cannot help you if this package destroys your computer. Do not trust it.


Improve theme handling

The themes at the moment are done in a totally weird random way (theme provider is nested number of times in some occasions with no option to override easily, this is especially true for the way we have to have 2 themes, one light for toolbar when adding plugins, one dark for bottom toolbar and few other places).
It's a real shame cause it's a strong feature of material ui that could definitely be used better.
I would like to see something like this:
<UiThemeOverrides lightTheme={lightTheme} darkTheme={darkTheme}>
    <Trash editor={editor} />
    <DisplayModeToggle editor={editor} />
    <Toolbar editor={editor} />
</UiThemeOverrides>

Where UiThemeOverrides is an optional (no breaking change) component that would inject these 2 theme variants down the line. It might however open a few cans of worms as I explore how this is done at the moment across the whole lib. Ideas, preferences, existing work on this?

Yeah I saw this in the PR but it was so huge it was difficult to spot all the places. The main problem is that we're using two themes - light and dark - in the UI right now. I think the bottom toolbar uses all dark while all other elements use white. Optimally, this would be the same theme everywhere.
It would be also cool if the editor didn't override an existing MuiThemeProvider:
<MuiThemeProvider>
  <Toolbar>
</MuiThemeProvider>

Here, Toolbar doesn't use its own styling but the provider given by the top component. If no MuiThemeProvider has been set (this can be checked from the context iirc), then it should use a default one. This would make things much easier and cleaner in the integration.

Yeah one theme would make much more sense. I think I'll have to look into this as we have corporate theme that we need to adhere to. And it would be crazy to override every part of this with some custom css.
 Now that I've seen this size of this, I am pretty happy to just port out theme to material-ui, fix the problems with 2 themes here and make it overridable.

Now that I've seen this size of this, I am pretty happy to just port out theme to material-ui, fix the problems with 2 themes here and make it overridable.

That sounds do-able to me!

Yep, I have few more PRs and ideas for 1 or 2 more, then I'll do this when everything is stable.

Ok I've got some progress on this. Right now, we are almost able to achieve theme overriding. There are 2 things missing.

Provider in UI package needs to get theme from props and pass it to theme provider. No brainer
Plugins themes should also be over-ridable. And this is more complicated. We shouldn't make assumptions about plugins so obviously the public api shouldn't assume people use material-ui. Yet it's used in most builtin plugins. And it's locked down. The plugins wrapped in Provider which injects the theme. Of the top of my head, we could maybe add "extraComponentProps" to plugin api (type would be Object and of course it's optional). These props should be then passed into the component. In these plugins we could then just provide theme here and pass it along in the component. If you like this approach, I should be able to code it up in no time.

Plugins themes should also be over-ridable. And this is more complicated

You can detect if a theme provider exists by checking the react context. If it doesn't exist we can set it. If it does, we skip it.

Okay that makes sense. I'll check how it's done then. Fingers crossed it will be just an easy change in the themeProvider.

i managed to refactor themeProviders at some point, but i still get sometimes some unexpected results.
Nevertheless, i close that for now


Initial commit of the search sample web part

Q
A

New sample?
yes

What's in this Pull Request?
This PR contains the sample of my Search SPFx Web Part created with React and the Flux pattern.

Hi @estruyf, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

thx Elio, will get processed today.

thx Elio, great sample which shows multiple nice aspects.


fix: use default styled-components export to restore styled-components@5 compatibility

@sanity/ui allows<EMAIL_ADDRESS>as a peer dependency.
In commit 1ad5a02 of @sanity/ui, we switched to the named styled export, which is only available in styled-components@6.
Therefore, some folks have<EMAIL_ADDRESS>installed in their Studio project, but the installed version of @sanity/ui isn't compatible. It can’t import styled.

In this branch, I've reverted to the default export, which is compatible with both styled-components@5 and styled-components@6.
This can be tested in alpha release 2.0.13-alpha.2. For testing purposes, you may need to set an override in your Studio project. For example, using pnpm:
"pnpm": {
  "overrides": {
    "@sanity/ui": "2.0.13-alpha.2"
  }
}

Does it also make sense to change the devDependency to styled-components@v5 so if someone by mistake uses v6 API it fails early?

@binoy14 Good point! It makes sense to develop using the lowest common denominator of Styled Components.

Not releasing this for now, as there's concern about breaking the package for ESM consumers.

In transpiled/commonJS environments that works as expected, but breaks server-side ESM usage of styled components.


Various Connect Wallet changes

add theme in ConfiguredWallet.connectUI's props - to use theme aware UI for wallets
add useWalletConfig hook to get the ConfiguredWallet object for active wallet
add hooks useSetConnectedWallet, useSetConnectionStatus
rename useActiveChain to useChain - keep the useActiveChain also with deprecated tag
make useSafe hook await-able by returning the promise of connect() call
add hook useSmartWallet
allow rendering custom wallet details button via <ConnectWallet detailsButton={} /> prop
Rename "Export" to "Backup" in local wallet UI

/release-pr


Enhance termchecker performance by using compiled XSLT

To increase the performance, an approach can be to compile the XSLT and export it in the compiled form. Then, this compiled stylesheet can be used from other Schematron file, or from other XSLT via xsl:use-package.

This is no longer on the agenda.


Offer beacons

This PR includes the new beacon for more expressive querying.

[x] Change protocol to use a single swap address for all swaps. Every user effectively gets their own DEX address.
[x] Change protocol to use a specific beacon policy for every asset being offered. The asset name for the beacon specifies what asset is being asked for. This allows querying all swaps that are offering a specific asset (no matter what asset is being asked for) and also all swaps for a specific trading pair.
[x] Reorganize off-chain code to make testing compositions easier.
[x] Update CLI.
[ ] Update Readme.
[ ] Update GettingStarted.
[x] Update Benchmarks.
[ ] Update Changelogs.
[ ] Add Architectural Decision Record (ADR).

Closing in favor of #10


prefer_ipv6 config file option

Currently it is possible to set prefer_ipv6 via command line option, but not by using a config file variable.  Is there some reasons why the latter could not be supported?

This question has been replaced by pull request https://github.com/alfredh/baresip/pull/692.


Performance on browse endpoint

Possibly index categories and cache browse.

 Adding an index fixed it completely without hardly increasing the database size.


Support http bearer tokens for authentication

Dependency
Requires Serialize authentication information tasks
Use case:
Requests with http bearer authentication is supported
Acceptance Criteria:

Tests for malformed/expired token, invalid principles, valid users

Prototyped:

https://github.com/opensearch-project/OpenSearch/pull/5611

@peternied I filed a related issue for secret key management for extensions: https://github.com/opensearch-project/OpenSearch/issues/6043. Should this milestone include both?

I've added it to the dependency at the top of this tasks


temperatureType is always shown in BOTH.

Issue reported in https://github.com/aldostools/webMAN-MOD/issues/713

wow I'm a rookie for creating this bug 😂

qiuck fix
https://gofile.io/d/htRdDh

Thank you!

Still the same issue after the update.

Still the same issue after the update.

I tested on my side and it is working when I use temperatureType: CELSIUS

It was my mistake. I had Cobra debug enabled. So it was loading the old version
It's working fine. Sorry for the wrong report.

In that case excellent. By the way how to you detect a debug version of Cobra?

Currently I only do the check Evilnat's CFW
#define isCobraDebug	(file_size("/dev_flash/sys/stage2.bin") > 124000)
It's not the best way, but it works.

your methods of achieving the end goal are not always the best but as long as it works 😊

LOL I know I do a lot of hacky things.


Fixing build issues due to ambiguous method call [lookupCredentials]

reference to lookupCredentials is ambiguous both method
  <C>lookupCredentials(java.lang.Class<C>,hudson.model.ItemGroup,org.acegisecurity.Authentication,com.cloudbees.plugins.credentials.domains.DomainRequirement...)
    in com.cloudbees.plugins.credentials.CredentialsProvider and method
  <C>lookupCredentials(java.lang.Class<C>,hudson.model.ItemGroup,org.acegisecurity.Authentication,java.util.List<com.cloudbees.plugins.credentials.domains.DomainRequirement>)
    in com.cloudbees.plugins.credentials.CredentialsProvider match

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

Stupid question: Against what version of the credentials plugin are you building?
I'm asking because this builds fine here and therefore there must be some recent change in the credtials plugin?!

@felfert I don't know which version he's using but downloading the source code from GitHub and trying to build directly from the POM does indeed generate build errors, maybe there's some version mismatch in the POM?
That's something I also had to address in my PR #113 just to get the code compiling.

@felfert It looks like credentials-plugin v1.24 (based on Manage Plugins after running mvn clean install hpi:run.
The ambiguous methods can be seen here and here when using null.

FYI, I've seen this issue when compiling with Java 1.8 -- It does not happen in 1.7


fix: skip template files that are not transpiled

Reverts https://github.com/asyncapi/generator-react-sdk/pull/243

/rtm


add ability to get non-option arguments to a command

Only saw https://github.com/jatoben/CommandLine/pull/40 after partially implementing this. Took the comment about interleaved arguments/options into account as well.
Thought about creating a new Option subclass to express this, but this was the easiest way. Open to discussion about the API.

Hmm, now that I think about it, I think I need to add an Option subclass for printing usage appropriately.

Not going to use an Option subclass, but preparing a better API for this.


Improved solution for a colorspace / transparency issue

Improved solution for a colorspace / transparency issue that affects some types of images.
Reference:
AFNetworking

Apologies for leaving this open for so long! As you can guess, I no longer use this code in any of my current projects. If you’re still interested in this PR, could you verify it’s compatible with Xcode 7 and provide a test image or two to demonstrate the types images it fixes, to prevent future regressions? Thanks, and I apologize for not responding before!


[Rendering] Images with Action.Submit on Teams

Target Platforms
Other
SDK Version
1.5
Application Name
Teams
Problem Description
Images with Action.Submit are not working when clicking on it
        {
          "type": "Image",
          "url": "${telephonyurl}",
          "selectAction": {
            "type": "Action.Submit",
            "title": "Telephony Rates",
            "data": {
              "action": "Telephony Rates"
            }
          }
        }

Screenshots
No response
Card JSON
{
              "type": "Image",
              "url": "${telephonyurl}",
              "selectAction": {
                "type": "Action.Submit",
                "title": "Telephony Rates",
                "data": {
                  "action": "Telephony Rates"
                }
              }
            }

Sample Code Language
No response
Sample Code
No response

Stopped working after updating Teams.

Yes! It was working for me last week after updating Teams this week stopped :(

Any news?

-> I opened my own issue as i have stumbled upon a bigger problem with this. I might also have a fix for you guys, provided you have an element behind your image. Here's the issue

Thanks for reaching out and I understand the frustration here. These are Teams specific issues, not on Adaptive Cards. To help get some progress here, we'll reach out internally and reference the Team's issues and see how we can get some traction for you internally. Sorry for your pain you've had here, hopefully we can help resolve this!

This issue has been copied over to Microsoft Teams repo here: https://github.com/MicrosoftDocs/msteams-docs/issues/5379


Checkout query parameters should be configurable

E.g. ttl and include. Currently these are hard-coded and some users may want to configure them differently.

Design pattern for options: https://commandcenter.blogspot.com/2014/01/self-referential-functions-and-design.html
func (m *Machine) Checkout(options ...func(CheckoutOptions) error) (*MachineFile, error) {
  opts := CheckoutOptions{Encrypt: true, Include: "license,license.entitlements"}
  for _, opt := range options {
    err := opt(opts)
    if err != nil {
      return nil, err
    }
  }

  // ...
}

This would require that we define other include relationship types, e.g. product, policy, etc.:
https://github.com/keygen-sh/keygen-go/blob/d6911045cd386479c3aefaa7893404f73542ce91/machine_file.go#L149-L167

Closed in 261833ff47104c9055299bac5b48dcdabe3c8942.


Program with enabled tco produces incorrect result.

We have a simple program
[<EntryPoint>]
let main argv =
    let zero = bigint(0)
    let one = bigint(1)
    let rec ack (m:bigint) (n:bigint) cnt =
            if m = zero then cnt(n + one)
            elif n = zero then ack (m-one) one cnt
            else (ack m (n-one) (fun x -> ack (m-one) x cnt))

    ack (bigint(3)) (bigint(1))  id |> printfn "%A"
    0 // return an integer exit code

This code works correct only when compiled with --tailcalls- option..
Repro steps
Compile in release mode or with --tailcalls+ option.and run this code as a console app
Expected behavior
Output:13
Actual behavior
Output:1
Related information
Provide any related information

Windows
.NET Runtime CLR v4.0.30319
Visual Studio 2015
F# Interactive version 14.0.23413.0
F# version 4.0 Fsharp.Core <IP_ADDRESS>

C:\app>fsc.exe Program.fs --tailcalls-
Microsoft (R) F# Compiler version 14.0.23413.0
Copyright (c) Microsoft Corporation. All Rights Reserved.

C:\app>Program.exe
13

C:\app>fsc.exe Program.fs
Microsoft (R) F# Compiler version 14.0.23413.0
Copyright (c) Microsoft Corporation. All Rights Reserved.

C:\Users\hodza\Documents\Visual Studio 2015\Projects\Ack\ConsoleApplication3>Program.exe
1

OK.... This may be a CLR bug (presumably involving structs and indirect tailcalls).  The only difference I see in the code we generate between
fsc --tailcalls+ --optimize+ -o a2.exe a.fs
fsc --tailcalls- --optimize+ -o a4.exe a.fs

is a single tail. instruction.  Which should always be safe to emit.

(It would be good to know the exact versions of .NET being used, and if this is a RyuJIT bug or not)

In mono vorks as expected
C:\app>Program.exe
1

C:\app>"C:\Program Files (x86)\Mono\bin\mono.exe" Program.exe
13

C:\Program Files (x86)\Microsoft Visual Studio 14.0>clrver

Microsoft (R) .NET CLR Version Tool  Version 4.6.1055.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Versions installed on the machine:
v2.0.50727
v4.0.30319

C:\Program Files (x86)\Microsoft SDKs\F#\4.0\Framework\v4.0\fsc.exe -o:obj\x64\Release\ConsoleApplication3.exe --debug:pdbonly --noframework --define:TRACE --doc:bin\Release\ConsoleApplication3.XML --optimize+ --platform:x64 -r:"C:\Program Files (x86)\Reference Assemblies\Microsoft\FSharp\.NETFramework\v4.0\<IP_ADDRESS>\FSharp.Core.dll" -r:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.5.2\mscorlib.dll" -r:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.5.2\System.Core.dll" -r:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.5.2\System.dll" -r:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.5.2\System.Numerics.dll" --target:exe --warn:3 --warnaserror:76 --vserrors --LCID:1033 --utf8output --fullpaths --flaterrors --subsystemversion:6.00 --highentropyva+ --sqmsessionguid:7f316f64-d341-4b4b-beb5-03310f53ed37 "C:\Users\hodza\AppData\Local\Temp\.NETFramework,Version=v4.5.2.AssemblyAttributes.fs" AssemblyInfo.fs Program.fs

tested different .net framework versions. from 4.5 to 4.6.1 and everywhere it is broken

Tested with  and it works correct. So problem in RyoJit.

The bug was fixed in RyuJIT in coreclr:
https://github.com/dotnet/coreclr/pull/5394
https://github.com/dotnet/coreclr/commit/f22b221718f3a243d4fc3fba4e037c2a9524718c


Made validation of name saner (allowing for spaces, dashes, e.g. for …

…exotic names

Stole it from http://stackoverflow.com/questions/2629422/rails-validating-full-name

Main reason for the validation is to prohibit numbers, and the other special characters as specified in the regex..


Add coursera screenshots for settings

Purpose/implementation Section
What changes are being implemented in this Pull Request?
Following along with the settings suggested by Ira in BaseCamp, I've added some more instructions about how to set these on the later part of https://github.com/jhudsl/DaSL_Course_Template_Bookdown/wiki/Publishing-on-Coursera
These required some screenshots to be added too.

The checks should not be affected at all by these screenshots. Merging.


Sugestion: Add Pagination

I am using the plugin and it is working good. I suggest to add Pagination to it so it will display X posts as configured but if there is more posts it will show the pagination links. Thank you.

I like the idea. Don't know when I'll have the time to implement it though. PR welcome.

Hey there, really great plugin so far – works perfectly for me where so many other "posts-by-categroy-plugins" failed. Your widget displays my content as it is and also runs visual composer elements, media galleries and so on.  I'm just missing a function like mentioned above by weidmaster: I use the widget to load specific posts and display them with full content on my blog-page. I know I can restrict the number of posts shown, but it would be great if the reader had the possibility to see older posts too – so a pagination or a "load more" button (ajax) would be really nice.


Feature: Preload disk cache to memory

I have a number of icons in my app that load remotely. Because I want to cache those longer term than other images, I set up a separate SDImageCache for those icons.
It'd be nice to be able to also preload the entire disk cache into memory in the background at app startup.
Haneke has this ability, for example.
I can probably find time to implement this, but just wanted to get some feedback / ideas first.

preload ? SDWebImagePrefetcher

SDWebImagePrefetcher loads a set of URLs. I want to load the a specific SDImageCache's disk cache into memory.
This would work somewhat similar to cleanDiskWithCompletionBlock in that it'd iterate through all files in the diskCachePath and load them into memory. Maybe the method could be called - (void)preloadDiskCacheWithCompletionBlock:(SDWebImageNoParamsBlock)completionBlock;

@johnboiles Current 4.x, SDImageCache contains a method called addReadOnlyCachePath:. To add the folder that SDImageCache will query secondlly after the original image cache path failed. However, your image's file name should be exact what SDImageCache using for cache key. It now using a MD5 of image cache key.
In 5.x, we introduce a more powerfull API additionalCachePathBlock. Which we pass the image cache key for you, you return a file path in the block, and we use that file path to query disk again. This can solve this issue.
And, we also support custom memory cache & disk cache in #2282 . You can create your own disk cache implementation add add a preload feature for your own business.
If you still have issue about it, feel free to reopen it :)


2 random results for same package

Hi,
awsome software thx for sharing.
I have random results with this package, sometimes reported with 2 deps, is there a option to fix this behavior?
> syft --version
syft 0.66.1
> curl -O 'https://repo1.maven.org/maven2/net/bytebuddy/byte-buddy/1.11.22/byte-buddy-1.11.22.jar'
> for 1 in $(seq 10); do syft byte-buddy-1.11.22.jar; done;
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]

NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]
NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]
NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [2 packages]
NAME            VERSION  TYPE
byte-buddy      1.11.22  java-archive
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [2 packages]
NAME            VERSION  TYPE
byte-buddy      1.11.22  java-archive
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [2 packages]
NAME            VERSION  TYPE
byte-buddy      1.11.22  java-archive
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]

NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]

NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [2 packages]

NAME            VERSION  TYPE
byte-buddy      1.11.22  java-archive
byte-buddy-dep  1.11.22  java-archive
 ✔ Indexed byte-buddy-1.11.22.jar
 ✔ Cataloged packages      [1 packages]
NAME            VERSION  TYPE
byte-buddy-dep  1.11.22  java-archive

Hi thx for early reply, I noticed it on Ubuntu 22 x86_64 and Debian 11 (bullseye) docker images

Hi, and apologies for the delay here. There was an issue identified causing nondeterministic behavior in Java archive identification, which was fixed in Syft 1.10.0. I'm going to close this issue as I am no longer able to reproduce the nondeterministic behavior, but please do let us know if you still have the issue!


Add snippet for @dd directive

I think that a snippet for @dd directive could be awesome suggested by b:dd~ (similar to b:dump~)

Sorry I didn't notice this issue. For coc-blade's directive completions and snippets, we use snippet data from this repository.

https://github.com/onecentlin/laravel-blade-snippets-vscode/blob/master/snippets/snippets.json

Create an additional PR for @dd in the snippet data in this repository, or add
your own snippet for @dd using coc-snippets, etc.

@BrayanCaro Thanks!


How to control go1 in mujoco?

Have a check here https://github.com/unitreerobotics/unitree_mujoco