suspenders help should provide actual help ...not generate an app...

Just made this mistake, expecting help. If the app is actually called "help", there should probably be a --name option or something.

:+1: thanks for your suggestion!

You can get the help "page" by running suspenders --help, I don't think there is any reason to expect suspenders help to open it as well. If we do want that, then probably moving to a rails like interface of suspenders new app_name.
One thing we can do regardless, is showing the help "page" when we run suspenders without any arguments as the current "No value provided for required arguments 'app_path'" is not very helpful.
What do you think?

I agree that the new namespace would be nice. Probably a good long-term solution.
For the specific case I ran into, a short-term solution could be to have a confirmation prompt before creating the new app. E.g:
Suspenders help can be found with 'suspenders --help'
Would you like to continue creating a new app named "help"? [y,N]

Yes the help page should probably show when no arguments are passed. (Somewhat separate issue though?)

Suspenders actively avoids prompting the user, so that I'm afraid would not
be a possible solution.
Anything else is up for discussion, I think.
On Mon, 14 Mar 2016 at 20:57, Loren Rogers<EMAIL_ADDRESS>wrote:

I agree that the new namespace would be nice. Probably a good long-term
solution.
For the specific case I ran into, a short-term solution could be to have a
confirmation prompt before creating the new app. E.g:
Suspenders help can be found with 'suspenders --help'
Would you like to continue creating a new app named "help"? [y,N]
Yes the help page should probably show when no arguments are passed.
(Somewhat separate issue though?)
—
Reply to this email directly or view it on GitHub
https://github.com/thoughtbot/suspenders/issues/730#issuecomment-196518089
.

You can get the help "page" by running suspenders --help, I don't think there is any reason to expect suspenders help to open it as well. If we do want that, then probably moving to a rails like interface of suspenders new app_name is a first step.

:+1:

One thing we can do regardless, is showing the help "page" when we run suspenders without any arguments as the current "No value provided for required arguments 'app_path'" is not very helpful. What do you think?

I like this. Will appreciate @mike-burns input: is this expected command-line utility behavior?

For the specific case I ran into, a short-term solution could be to have a confirmation prompt before creating the new app.

@lorentrogers what are other utilities that have the  help option output a help page?

Doing something useful when run with no arguments seems nice. Showing --help then might be a good idea.

Here's an example of how it was solved in https://pragprog.com/book/dccar/build-awesome-command-line-applications-in-ruby

All these produce the same help info:
rails help
rails -h
rails --help

I believe this is a documented design pattern for command line apps, but I can't seem to find the ref for it. I'll post when I come across more concrete info.

~% ls help
colorls: help: No such file or directory
~% ls -h
some-file
~% ls --help
colorls: unknown option -- -
usage: colorls [-1AaCcdFfGgHhikLlmnopqRrSsTtux] [file ...]

It may or may not be documented, but it is NOT how unix commands work. Let's instead focus on making suspenders usable. Do you need help with your pull request?

Interesting! You're 100% right that it's not a UNIX pattern -- I wouldn't expect things like cd, ls, or cat to respond to a help command. Seems like it's pretty common in ruby gems though? E.g. Bitters supports it:
bitters help
Commands:
bitters help [COMMAND]  # Describe available commands or one specific command
bitters install         # Install Bitters into your project
bitters remove          # Remove Bitters
bitters reset           # Reset Bitters
bitters version         # Show Bitters version

I assume that this is because they're not documented with man? Is there a common understanding around when the different options are used?

Long-term, we're hoping to drop the command-line aspect of suspenders -- see #676 for what we're trying to get working.
I'd rather concentrate on better output on ARGV.empty? and on #676 than on re-arranging the arguments for the existing command. You'll want to focus on bin/suspenders for your patch, I suspect.

Aah - makes sense. I'll put together a PR with just the ARGV.empty? output updated.
Out of curiosity, is there a standard set of guidelines for designing ruby cli apps? I've been looking through that Copeland book -- is there a better resource for this?


Server Hello should not contain supported_groups extension (extension.SupportedEllipticCurves)

Your environment.

Version: v2.0.9
Browser: N/A

Pion includes both the supported_groups (also called elliptic_curves) and ec_point_formats extensions, whether sending a Client Hello or a Server Hello. (Refs: flight 1, flight 3 for Client Hello, flight 4 for Server Hello). ec_point_formats is fine in a Server Hello, but supported_groups is only defined for Client Hello.
The presence of supported_groups in Server Hello has emerged as a concrete distinguisher used to block Snowflake WebRTC connections in Russia:

Russian DPI check supported_groups extension in ServerHello payload (byte 0x5a in udp packet). It looks for DTLS packet header "magic" "16 FE FD" and then looks for "1D 00 17 00 18" at 0x5a offset.

I think that supported_groups does not belong in Server Hello. These are my reasons for thinking so:

Comparison with browser DTLS.

Go crypto/tls: clientHelloMsg contains supportedCurves and supportedPoints fields which may be marshaled, but serverHelloMsg only contains and may marshal supportedPoints.

RFC 8422 specifies both extensions for Client Hello, but only ec_point_formats for Server Hello.

5.1. Client Hello Extensions
This section specifies two TLS extensions that can be included with the ClientHello message as described in [RFC4366]: the Supported Elliptic Curves Extension and the Supported Point Formats Extension.

5.2. Server Hello Extension
This section specifies a TLS extension that can be included with the ServerHello message as described in [RFC4366], the Supported Point Formats Extension.

(Note, though, that in TLS 1.3, Server Hello may contain supported_groups—though TLS 1.3 also removes ec_point_formats. I don't think it's relevant for DTLS 1.2.)

This sounds very reasonable to me. If you're able and willing, feel free to submit a PR (with a test please) that fixes this inconsistency.


add known issue

https://issues.redhat.com/browse/ACM-6230
https://issues.redhat.com/browse/ACM-7461

/cc @clyang82 @hchenxa

maybe we also need to log a task for ID to add them into official document. what do you think?

maybe we also need to log a task for ID to add them into official document. what do you think?

Yes, created.
https://issues.redhat.com/browse/ACM-8264

Thanks
/lgtm
/approved


Enable access to the checkout url or params within Checkout UI Extensions API

Is your feature request related to a problem? Please describe.
Some Shopify Plus merchants would like to show/hide checkout UI components based on the presence or absence of a URL param.
Describe the changes you are looking for
Currently there doesn't seem a way within Checkout Extensibility to read the url/params of a checkout. It would be great to have an API that allows for this, something like useUrl that would return the full url, params, and url-related data
Describe alternatives you’ve considered

Storefront API allows fetching the Cart object that has the URL, but it requires a cart ID that doesn't seem to be available within Checkout Extensibility.
Checkout UI Extensibility APIs don't seem to have access to this url data
Making a request to a 3rd party server may work as I can parse out the URL from there and return it back, but that would require a network request that I'd prefer to avoid due to adding an additional dependency and point of failure.

Something that was helpful in our case - the URL parameters can read out as checkout attributes if you build them into a direct checkout link like this:
yourstore.com/cart/VARIANT_ID:1?attributes[fieldLabel]=fieldValue

Any way to read parameters in the checkout that are not from the cart attributes?

this is very much needed


How to save deck that is automatically populated while playing?

I start an arena match, and the player tracker automatically shows whatever cards I draw (very cool).  When I start another arena match with the same deck, it clears the existing deck though.  How do I save the deck so that it will remember the cards that it already saw?
Thanks!

This is not yet implemented... I'll add this on my (very long) todo-list

I think this is here:
https://github.com/bmichotte/HSTracker/pull/461


autoprefixer (or other css prefixer) with vueify

I want to run autoprefixer when building vue projects. My build includes browserify with vueify as transform. Can you give an example of how to include css post-processors in this setup? Thanks!

This question doesn't really have anything to do with Vue. In this case you'd just use a build tool like gulp, and create two separate tasks, one that runs your .js files through browserify with the vueify transform, and one that runs your .css through a CSS prefixer.

Thanks. The problem is that I don't have any explicit css files. Vueify transform extracts the styles from a .vue file, and inserts them with the insert-css module in the document's head.

Ah sorry I misunderstood your question. I suspect a gulp plugin that prefixes HTML will work on Vue components, like this one for example: https://www.npmjs.com/package/gulp-html-autoprefixer
If that doesn't work you could always have separate CSS for your components, prefix them with gulp, and then require that CSS as the template property of your component options using Browserify (template: require('style.css'))

Currently the pre-processors supported in Vueify are hard-coded, I will think about how to make it more extensible.

Moving the discussion to vue-component-compiler.

Thanks for the quick and great comments! gulp-html-autoprefixer will do for now.

quick gist for people who want to run autoprefixer using custom pre-processors:
https://gist.github.com/kzima/d0f173fc8f7f5bce9979

Thanks!
On Sun, May 17, 2015 at 7:23 AM, Kris Zima<EMAIL_ADDRESS>wrote:

quick gist for people who want to run autoprefixer using custom
pre-processors:
https://gist.github.com/kzima/d0f173fc8f7f5bce9979
—
Reply to this email directly or view it on GitHub
https://github.com/yyx990803/vue/issues/732#issuecomment-102785452.


Give another solution of problem #66.

Give another solution of problem 66, and fix the old solution as #142 did.

Sure. I've replace the solution with mine and set the difficulty in all the relevant file. Thanks for your appreciation.

Thanks.


fix(invariants): respect fail_on_revert properly and do not populate logs/traces twice

Fixes #5726, fixes #6082, fixes #6024
fail_on_revert was not getting respected correctly after a recent refactor, in which we stopped populating the error in InvariantFailures. This was causing mainly failures in handlers to not be detected leading to erroneous [PASS] states.
We were also collecting the logs and traces of the last error twice, which is not necessary as we replay the entire run. This could lead to situations where it seems an invariant gets called twice, but it's just the error log popping up twice.

yep will do—just haven't finished adding the test :D


get self executor id from DB as per host:port

The acceptance tests need to specify the target executor ID, which has to be fetched from DB directly. This pull request writes a new API to have it.
Verified in localhost.

Found an issue to block this PR to merge. Since we don't know the port in advance, this API might be difficult to use.

Your idea sounds tons make sense to me! Will work on this when having bandwidth.

I renamed the API and added isAlive parameter to here.


CSIP1-10 requirements fix

Fixing many issues detected during validating against corpus: #44 #45 #46 #47
However issue #44 rule 3 and 4 and issue #45 rule 2 requires more attention, because they cannot be fixed simply by creating a proper schematron test.

CSIP1: The validator does not check if the OBJID is empty
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP1/invalid/mets-xml_mets_OBJID_attribute_value_empty/

Result prepatch
Fail: No entry in the schematron results array for CSIP1 despite missing @OBJID
Fix result
Pass: Missing OBJID error correctly reported
{
  "rule_id": "CSIP1",
  "severity": "Error",
  "location": {
    "context": "/mets:mets",
    "test": "@OBJID != ''",
    "description": "/*[local-name()='mets' and namespace-uri()='http://www.loc.gov/METS/']"
  },
  "message": "The mets/@OBJID attribute is mandatory, its value is a string identifier for the METS document. For the package METS document, this should be the name/ID of the package, i.e. the name of the package root folder. For a representation level METS document this value records the name/ID of the representation, i.e. the name of the top-level representation folder."
}

Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP1/valid/minimal_IP_with_1_representation

Result prepatch
Pass: No warning for valid IP METS
Fix result
Pass: No warning for valid IP METS
CSIP1: The validator does not check if the OBJID is equal to the name of the representation or package folder
This issue has not been addressed in the fix. As Jakub notes, it's not something that can be asserted in schematron. Consider moving this check into the structural checks.
CSIP2: The validator does not check if the mets/@type has value from VocabularyContentCategory
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP2/invalid/mets-xml_mets_TYPE_attribute_value_incorrect

Result prepatch
Fail: No warning for incorrect TYPE attribute value.
Fix result
Pass: Incorrect type value correctly caught
{
  "rule_id": "CSIP2",
  "severity": "Error",
  "location": {
    "context": "/mets:mets",
    "test": "((@TYPE = 'Textual works – Print') or (@TYPE = 'Textual works – Digital') or (@TYPE = 'Textual works – Electronic Serials') or (@TYPE = 'Digital Musical Composition (score-based representations)') or (@TYPE = 'Musical Scores - Print') or (@TYPE = 'Musical Scores - Digital') or (@TYPE = 'Photographs – Print') or (@TYPE = 'Photographs – Digital') or (@TYPE = 'Other Graphic Images – Print') or (@TYPE = 'Other Graphic Images – Digital') or (@TYPE = 'Microforms') or (@TYPE = 'Audio – On Tangible Medium (digital or analog)') or (@TYPE = 'Audio – Media-independent (digital)') or (@TYPE = 'Motion Pictures – Digital and Physical Media') or (@TYPE = 'Video – File-based and Physical Media') or (@TYPE = 'Software') or (@TYPE = 'Software and Video Games') or (@TYPE = 'Email') or (@TYPE = 'Datasets') or (@TYPE = 'Geospatial Data') or (@TYPE = 'Geographic Information System (GIS) - Vector Data') or (@TYPE = 'GIS Raster and Georeferenced Images') or (@TYPE = 'GIS Vector and Raster Combined') or (@TYPE = 'Non-GIS Cartographic') or (@TYPE = '2D and 3D Computer Aided Design') or (@TYPE = 'Design (schematics, architectural drawings) - Print') or (@TYPE = 'Scanned 3D Objects (output from photogrammetry scanning)') or (@TYPE = 'Databases') or (@TYPE = 'Websites') or (@TYPE = 'Web Archives') or (@TYPE = 'Collection') or (@TYPE = 'Event') or (@TYPE = 'Image') or (@TYPE = 'Interactive resource') or (@TYPE = 'Moving image') or (@TYPE = 'Sound') or (@TYPE = 'Still image') or (@TYPE = 'Text') or (@TYPE = 'Physical object') or (@TYPE = 'Service') or (@TYPE = 'Mixed') or (@TYPE = 'Other')) and (@TYPE != 'Other' or (@TYPE = 'Other' and @csip:OTHERTYPE != ''))",
    "description": "/*[local-name()='mets' and namespace-uri()='http://www.loc.gov/METS/']"
  },
  "message": "The `mets/@TYPE` attribute MUST be used to declare the category of the content held in the package, e.g. \"Datasets\", \"Websites\", \"Mixes\" , \"Other\", etc.. Legal values are defined in a fixed vocabulary. When the content category used falls outside of the defined vocabulary the `mets/@TYPE` value must be set to \"OTHER\" and the specific value declared in `mets/@csip:OTHERTYPE`. The vocabulary will develop under the curation of the DILCIS Board as additional content information type specifications are produced."
}

Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP2/valid/minimal_IP_with_1_representation

Result prepatch
Pass: No warning
Fix result
Pass: No warning
CSIP2: The validator does not check if mets/@csip:OTHERTYPE exists when mets/@type = "OTHER"
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP2/invalid/mets-xml_mets_TYPE_attribute_value_OTHER_and_csip-OTHERTYPE_attribute_not_exist

Result prepatch
Fail: No warning
Fix result
Pass: Missing OTHERTYPE attribute caught
{
  "rule_id": "CSIP2",
  "severity": "Error",
  "location": {
    "context": "/mets:mets",
    "test": "((@TYPE = 'Textual works – Print') or (@TYPE = 'Textual works – Digital') or (@TYPE = 'Textual works – Electronic Serials') or (@TYPE = 'Digital Musical Composition (score-based representations)') or (@TYPE = 'Musical Scores - Print') or (@TYPE = 'Musical Scores - Digital') or (@TYPE = 'Photographs – Print') or (@TYPE = 'Photographs – Digital') or (@TYPE = 'Other Graphic Images – Print') or (@TYPE = 'Other Graphic Images – Digital') or (@TYPE = 'Microforms') or (@TYPE = 'Audio – On Tangible Medium (digital or analog)') or (@TYPE = 'Audio – Media-independent (digital)') or (@TYPE = 'Motion Pictures – Digital and Physical Media') or (@TYPE = 'Video – File-based and Physical Media') or (@TYPE = 'Software') or (@TYPE = 'Software and Video Games') or (@TYPE = 'Email') or (@TYPE = 'Datasets') or (@TYPE = 'Geospatial Data') or (@TYPE = 'Geographic Information System (GIS) - Vector Data') or (@TYPE = 'GIS Raster and Georeferenced Images') or (@TYPE = 'GIS Vector and Raster Combined') or (@TYPE = 'Non-GIS Cartographic') or (@TYPE = '2D and 3D Computer Aided Design') or (@TYPE = 'Design (schematics, architectural drawings) - Print') or (@TYPE = 'Scanned 3D Objects (output from photogrammetry scanning)') or (@TYPE = 'Databases') or (@TYPE = 'Websites') or (@TYPE = 'Web Archives') or (@TYPE = 'Collection') or (@TYPE = 'Event') or (@TYPE = 'Image') or (@TYPE = 'Interactive resource') or (@TYPE = 'Moving image') or (@TYPE = 'Sound') or (@TYPE = 'Still image') or (@TYPE = 'Text') or (@TYPE = 'Physical object') or (@TYPE = 'Service') or (@TYPE = 'Mixed') or (@TYPE = 'Other')) and (@TYPE != 'Other' or (@TYPE = 'Other' and @csip:OTHERTYPE != ''))",
    "description": "/*[local-name()='mets' and namespace-uri()='http://www.loc.gov/METS/']"
  },
  "message": "The `mets/@TYPE` attribute MUST be used to declare the category of the content held in the package, e.g. \"Datasets\", \"Websites\", \"Mixes\" , \"Other\", etc.. Legal values are defined in a fixed vocabulary. When the content category used falls outside of the defined vocabulary the `mets/@TYPE` value must be set to \"OTHER\" and the specific value declared in `mets/@csip:OTHERTYPE`. The vocabulary will develop under the curation of the DILCIS Board as additional content information type specifications are produced."
}

Note: There is an incongruity between CSIP2 and CSIP3 tests due to a casing issue. We cannot satisfy both at once with the same test package. All further test corpus packages that use @TYPE=OTHER fail CSIP2 in tests moving forward.
CSIP2: The validator does not check if mets/@csip:OTHERTYPE is set to some value when mets/@type = "OTHER"
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP2/invalid/mets-xml_mets_TYPE_attribute_value_OTHER_and_csip-OTHERTYPE_attribute_has_no_value

Result prepatch
Fail: No warning
Fix result
Fail*: CSIP2 throws an error warning, but only because it fails the TYPE vocab check. The value in the METS is "OTHER" while the schematron checks for "Other" per the vocabulary.
CSIP4: The validator does not check if the mets/@CONTENTINFORMATIONTYPE has value from VocabularyContentInformationTypeSpecification
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP4/invalid/CONTENTINFORMATIONTYPE_value_incorrect

Result prepatch
Fail: Not caught by schematron rule but is caught by the schema itself
Fix result
Pass: Correctly caught invalid CONTENTINFORMATIONTYPE value not from vocab
{
  "rule_id": "CSIP4",
  "severity": "Error",
  "location": {
    "context": "/mets:mets",
    "test": "((@csip:CONTENTINFORMATIONTYPE = 'ERMS') or (@csip:CONTENTINFORMATIONTYPE = 'SIARD1') or (@csip:CONTENTINFORMATIONTYPE = 'SIARD2') or (@csip:CONTENTINFORMATIONTYPE = 'SIARDDK') or (@csip:CONTENTINFORMATIONTYPE = 'GeoData') or (@csip:CONTENTINFORMATIONTYPE = 'citscarchival_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'cscarchival_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'citserms_v2_1') or (@csip:CONTENTINFORMATIONTYPE = 'citserms_v3_0') or (@csip:CONTENTINFORMATIONTYPE = 'citspremis_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'cspremis_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'citsehpj_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'citsehpj_v2_0') or (@csip:CONTENTINFORMATIONTYPE = 'citsehcr_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'citssiard_v1_0') or (@csip:CONTENTINFORMATIONTYPE = 'citsgeospatial_v3_0') or (@csip:CONTENTINFORMATIONTYPE = 'MIXED') or (@csip:CONTENTINFORMATIONTYPE = 'OTHER')) and (@csip:CONTENTINFORMATIONTYPE != 'Other' or (@csip:CONTENTINFORMATIONTYPE = 'Other' and @csip:OTHERCONTENTINFORMATIONTYPE != ''))",
    "description": "/*[local-name()='mets' and namespace-uri()='http://www.loc.gov/METS/']"
  },
  "message": "Used to declare the Content Information Type Specification used when creating the package. Legal values are defined in a fixed vocabulary. The attribute is mandatory for representation level METS documents."
}

Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP4/valid/valid_IP_with_SHOULD_MAY_1_rep

Result prepatch
Pass: No warning
Fix result
Pass: No warning
CSIP4: The validator does not check if mets/@csip:OTHERCONTENTINFORMATIONTYP exists when mets/@CONTENTINFORMATIONTYPE = "OTHER"
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP4/invalid/CONTENTINFORMATIONTYPE_OTHER_and_OTHERCONTENTINFORMATIONTYPE_not_exist

Result prepatch
Fail: No warning
Fix result
Fail*: Test package uses type "OTHER" not type "Other". This is because of the casing inconsistency. With the value "Other", it works correctly.
CSIP4: The validator does not check if mets/@csip:OTHERCONTENTINFORMATIONTYPE is set to some value when mets/@CONTENTINFORMATIONTYPE = "OTHER"
Result prepatch
Fail: No warning
Fix result
Fail*: Same issue as above.
CSIP9: The validator does not check if the mets/metsHdr/@csip:OAISPACKAGETYPE has value from VocabularyOAISPackageType
Test
eark-validator eark-ip-test-corpus/corpus/CSIP/CSIP9/invalid/mets-xml_metsHdr_OAISPACKAGETYPE_attribute_value_incorrect

Result prepatch
Fail: No warning, schema catches it
Fix result
Pass: Catches warning, but the message could be more descriptive
Additional Notes

CSIP1 test related to matching attributes to filesystem names can't be addressed in schematron, could we consider moving it to code? @carlwilson
CSIP2 and CSIP4 tests related to "OTHER" type and corresponding "OTHERTYPE" attribute are failing due to casing issues (OTHER vs Other), but as @dockmd notes this is really a problem between the spec, the vocabulary and the test corpus rather than the validator.  This needs to be standardized across the specifications and implementations.
Consider making error message more descriptive for CSIP9, but again of course that is non-trivial.
CSIP2 and 3 have redundant crossover which can make parsing validation results confusing, same as above though since both requirements describe enough scope to warrant comprehensive validation of the OTHERTYPE attribute individually.

Many thanks @Sunday-Crunk that's a better job than I could have done.

Thanks for testing @Sunday-Crunk, great find with 'Other' inconsistency. I think we should stick to values defined in vocabulary as otherwise testing if value belongs to vocabulary would always fail. I've fixed CSIP 2, 3 and 4  - 'Other' issue and created PR as testcase needed adjustments too.
Also I've fixed more issues: #49 #50 #51 #52 #53 #57 #59

@dockmd @carlwilson I've switched to putting my notes on the commits, couple of things to check in CSIP20, 24 and 40, other than that all seems great

@carlwilson Did you see my note about the CSIP20 change?

@Sunday-Crunk @carlwilson
Good finding with the status warning behaviour. Fixed so that when status is not present, a warning will be generated, but when it is present but with an invalid value, an error will be generated. The same applies for CSIP34 and CSIP47 (#61). Also it required CSIP20 testcase fix - https://github.com/DILCISBoard/eark-ip-test-corpus/pull/496


KVM: implement SYZOS_API_MSR, minor fixes

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

@dvyukov PTAL


Support for hard links

For full linux backups, support of hard links

DUplicate of #152.

Thanks for raising this issue, it is indeed a duplicate so I'm closing this issue. You can subscribe to #152 if you'd like to be notified of the process.


Clone panel overflow

When both SSH and HTTPS are enabled, the clone panel is not large enough to hold all its contents.
I suggest to increase the size from 350px to 400px.

@0x1793d1 Thank you for your PR 😄 are you able to include a screenshot of before/after changes?

Sure!
Before:

After:

Codecov Report

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

@@            Coverage Diff            @@
##             master    #5003   +/-   ##
=========================================
  Coverage          ?   37.41%
=========================================
  Files             ?      306
  Lines             ?    45374
  Branches          ?        0
=========================================
  Hits              ?    16978
  Misses            ?    25943
  Partials          ?     2453

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 2e2eacf...d332924. Read the comment docs.

I think the fix proposed in https://github.com/go-gitea/gitea/issues/5072 is better

I'd agree with @adelowo .

files conflicting.

It seems that the 1.6 version fixed this. I don't have the problem anymore.

So let's close this one and feel free to reopen it.


openai: embeddings: supported chunk_size when check_embedding_ctx_length is disabled

Chunking of the input array controlled by self.chunk_size is being ignored when self.check_embedding_ctx_length is disabled. Effectively, the chunk size is assumed to be equal 1 in such a case. This is suprising.
The PR takes into account self.chunk_size passed by the user.

@efriis is there something I could do to impove the PR? Thanks

reassigning to @baskaryan who recently refactored this!

Hi, @baskaryan! Do you have any suggestions?


set HOG true,but errors occurs that cvFilter2D

I use the KCV to track the moving object.When I set HOG false,it behaves very well,and peak_value set originally is 2.When set HOG false,the compiler returns the following errors.
OpenCV Error: Assertion failed (src.size() == dst.size() && src.channels() == dst.channels()) in cvFilter2D, file /opencv-2.4.13/opencv/modules/imgproc/src/filter.cpp, line 3359
terminate called after throwing an instance of 'cv::Exception'
what():  /opencv-2.4.13/opencv/modules/imgproc/src/filter.cpp:3359: error: (-215) src.size() == dst.size() && src.channels() == dst.channels() in function cvFilter2D.
I check the cvFilter2D,and image I use to init the KCFTracker and used to update the Rect are all gray images, and the size are also right.I am curious.
Do some friends know the reasons?
Thank you in advance!

OK,when I have asked the question,I get the answer.Also to let more people know the reason I met the question.
https://github.com/joaofaro/KCFcpp/pull/16


Logical not formatting fix

addresses #997
The ! operator was handled by appending with whitespace, which seemed to be the issue.

Extracted it into its own handle_logical_not_case
Added a basic test to test various scenarios

Thank you for the contribution @binggh! Is there still outstanding work for this PR?

Thank you for the contribution @binggh! Is there still outstanding work for this PR?

Nope, I'll convert it to be ready for review.

Lgtm nice work


nested tags are not working

Thanks for developing this nice plugin. The plugin doesn't work with nested tags like #a/b.

I fixed it in 1.0.3


Is there an API to query how many instance existed and iterate them?

such as I've created many Instance. how can I iterate them and get their names? i can't find any related api in documents.Please help me. thanks!

No, unfortunately not.
You should have a "master" instance to store such information and maintain those info on your app's side.
Maybe on v2 we could do something like this inside LF.
Duplicate of #719

OK! Thanks for your response!


Debug TorchScript error from moco

Repro - python torchbench.py --training --devices=cuda --accuracy-ts --only=moco
This ones has a DistributedDataParallel module, so it might be something we can table for now.
Error is pretty long, the important section is as follows
	First diverging operator:
	Node diff:
		- %mod : __torch__.torch.nn.parallel.distributed.DistributedDataParallel = prim::GetAttr[name="mod"](%self.1)
		+ %mod : __torch__.torch.nn.parallel.distributed.___torch_mangle_596.DistributedDataParallel = prim::GetAttr[name="mod"](%self.1)
		?                                                ++++++++++++++++++++
ERROR: Tensor-valued Constant nodes differed in value across invocations. This often indicates that the tracer has encountered untraceable code.

cc eellison

I don't think traced graphs are guaranteed to be consistent generally... Not sure this is a TorchScript error


cmd/go: go mod tidy noop, even though indirect dep versions are wonky

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

$ go version
go version go1.21.0 darwin/arm64

Does this issue reproduce with the latest release?
Yes
What did you do?
Our project accumulated a lot of dependencies and indirect dependencies in our go.mod file over the years. Whenever touching the deps, we always did go mod tidy to ensure everything's on a correct version. A few weeks ago we've been receiving bug reports about funky version issues and while investigating, we've realised there are indirect dependencies in our go.mod file... that are not actually used at all in our code, neither in any actual dependency.
Go mod tidy is a noop, it doesn't detect nor remove these. If I manually remove all indirect deps from the go.mod file and do a go mod tidy, it resulted in a huge diff with versions changing all over the place compared to the original go.mod and go.sum file. It seems that either something is not detected by go mod tidy correctly, or perhaps the command has some subtlety to it that is not documented?
A repro:
# Check out the go-ethereum repo and the weird commit where we noticed this
$ git clone https://github.com/ethereum/go-ethereum
$ cd go-ethereum
$ git checkout 41ee96fdfee5924004e8fbf9bbc8aef783893917

# Verify that there's no diff and that go mod tidy is a noop
$ git diff
$ go mod tidy
$ git diff

# DO THIS MANUALLY: delete all the //indirect lines from go.mod, rerun go mod tidy
$ go mod tidy
go: finding module for package gotest.tools/v3/assert
go: finding module for package github.com/tklauser/go-sysconf
go: finding module for package github.com/StackExchange/wmi
go: finding module for package github.com/naoina/go-stringutil
go: found github.com/naoina/go-stringutil in github.com/naoina/go-stringutil v0.1.0
go: found github.com/StackExchange/wmi in github.com/StackExchange/wmi v1.2.1
go: found github.com/tklauser/go-sysconf in github.com/tklauser/go-sysconf v0.3.12
go: found gotest.tools/v3/assert in gotest.tools/v3 v3.5.0

$ git diff
[... 400 lines of diff ...]

What did you expect to see?
My expectation is for go mod tidy to generate the same indirect dependency versions, no matter how it is run and with what input as far as indirect deps are concerned.
What did you see instead?
Seems that the indirect dependencies listed in the go.mod file play a role in the result of go mod tidy, which IMO they should not.

you can upgrade indirect dependencies, and the result of is recorded in your go.mod.
that won't be replicated if you remove any record of your upgrades by deleting the go.mod lines.


Thread with Concierge auto pinned

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Reply a message of Concierge
Re-open this thread

Expected Result:
Only main chat with Concierge should be pinned
Actual Result:
Thread with Concierge auto pinned
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: no
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/0c70639d-0631-44ca-9477-65247b79628a
https://github.com/Expensify/App/assets/93399543/c8bc834a-cbd7-41cc-9894-ba7b56e78fc8
Expensify/Expensify Issue URL:
Issue reported by: @dukenv0307
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686212619362709
View all open jobs on GitHub

@joekaufmanexpensify server side is returning isPinned: true for concierge reports. I think there should be default condition if parentID is not null then set isPinned: false by deflaut.


Drop Down On hovering on the items in Nav Bar

Describe the feature
As it can be seen in the screenshots below , the project displays sub-items on hovering on the item headings like clothing etc. in the navbar , but it is not visually appealing and does not match the original design of the flipkart website .
Below is the screenshot of the original website -

Add ScreenShots

Record

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

@arghadipmanna101 sir please have a look and if this looks good to you , consider assigning it to me . Thankyou :)


Issue when using find() on embeded model

Laravel Version: 6.20.32
jenssegers/mongodb Version: 3.6.8
PHP Version: 7.3
Database Driver & Version: Mongo 4

Description:
I'm using an EmbedsMany relationship between my Proof and ProofStage models, like this:
namespace App\Models;

use Jenssegers\Mongodb\Eloquent\Model as Eloquent;

class Proof extends Eloquent
{
    public function stages(): EmbedsMany
    {
        return $this->embedsMany(ProofStage::class);
    }

    // More ...
}

namespace App\Models;

use Jenssegers\Mongodb\Eloquent\Model as Eloquent;

class ProofStage extends Eloquent
{
    // ...
}

Based on the docs I should be able to do something like ProofStage::find('some-id') and get an instance of the embeded model by its id. But the reality is that I'm always getting null when doing so. Of course I'm passing a valid id.
Expected behaviour
The corresponding model instance is retrieved.
Actual behaviour
null is always returned, even when passing a valid id.

have you got a solution on it yet?

have you got a solution on it yet?

Nope.

The only way i think is - proof()->stages()->where(...)->first()

Just don't use  magic embedded relationships.
This has been discussed multiple times already.
If you know how to fix it, go ahead push PR.
Thanks!


det() and solve() take self rather than &self

Currently the signature is fn det(self) -> T. This consumes the Matrix so we cannot do anything else with it. This doesn't seem necessary, it should be changed to fn det(&self) -> T instead.

This was made as part of the most recent breaking change. Basically you can retain the previous functionality by calling mat.clone().det().
Previously we took a reference but then in the lup_decomp function, which is called within det, we would clone self. This is poor practice as we force the user to clone their data. By consuming self the choice is given to the user.
I'd like to provide an implementation which does not consume the matrix, but for now we decided it was best that the API was transparent.

Oh and I forgot to say - we will soon be updating the lu_decomp function so that we can have a persistent solve function. You can check progress in #94 .

As far as I know, it's not possible to compute the determinant of a general matrix without using at least O(n2) memory. The idea is then to take self and reuse its memory, so that the additional memory costs are much less (once the LU rework in #94 is complete, only O(n) additional memory will be necessary for LU decomposition). My point is that consuming self here is the right choice, and I think we cannot ever provide a function which only takes &self while avoiding excessive memory overhead.
@AtheMathmo: I think perhaps new users coming from e.g. NumPy might be a bit confused as to why these functions consume self. We should probably improve the docs in that regard. Once again, this will be easier with #94 in place...

I'm going to close this issue now. Sorry that this was unclear @yongqli .
I agree that adding some more documentation for this might be worthwhile. However, given your reassuring comments I'll hold off for now. The changelog explains a little more about this change - I'll modify this post so that this information is more readily available and hope any other confused parties find their way here. If this proves futile in the future (I'm thinking after the #94 improvements land) then I will happily update the docs!


Tabprefix

#I would realy like if you make somthing where you can controll tabprefixes from rank + a option to change the max length of a tabprefix.
(sorry for the bad spelling I have dislexion)

LP doesn't manage Tablists...
It all depends on your Tablist-plugin.

You need an extra plugin to format the tab - LuckPerms just provides the data.


No way to detect elaboration message codes with quick fix API

Let's say I want to provide a quick fix for #17871. So far, I have a specialized elaboration:
    "Object literal may only specify known properties, but '{0}' does not exist in type '{1}'. Did you mean to write '{2}'?": {
        "category": "Error",
        "code": 2561
    },

However, because this is only an elaboration, I can't register
Diagnostics.Object_literal_may_only_specify_known_properties_but_0_does_not_exist_in_type_1_Did_you_mean_to_write_2.code

for my codefixes. Instead, I have to register a codefix for
Diagnostics.Type_0_is_not_assignable_to_type_1.code

which is suboptimal since it's so indirect and non-prescriptive.

I'm curious to hear if folks on the Roslyn team (@CyrusNajmabadi) have ran into this at all.

Nope.  We don't have that sort of concept for C#/Vb.

We can revisit if needed for some other case


Allow rails 7

As we are unable to find and change the base branch toallow_rails_6_1 so we are closing the MR and we will create MR again from the allow_rails_6_1. Thanks for understanding


[SPARK-8639] [Docs] Fixed Minor Typos in Documentation

Ticket: SPARK-8639
fixed minor typos in docs/README.md and docs/api.md

Can one of the admins verify this patch?

Merging after manually verifying the docs build.


The "Before you Start" Section has incorrect licensing info on it

The top bullet point "Acquire a license for Enterprise Mobility + Security 5 (EMS E5) directly via the Microsoft 365 portal or through the Cloud Solution Partner (CSP) licensing model." is not factually correct.
Azure ATP may be procured as a standalone product by a customer. EM+S E5 is not required. However, if a customer owns EM+S E5, then yes, they will also own Azure ATP as well. Same thing as Microsoft 365 E5 -- if you own that license, then you also already own Azure ATP (because M365 E5 includes the EM+S E5 suite).

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

ID: 3e880e6e-9cae-b122-14d9-963eee86d9cc
Version Independent ID: c8ab5d85-f3a4-2c24-d6c6-5940e26aea42
Content: Azure Advanced Threat Protection prerequisites
Content Source: ATPDocs/atp-prerequisites.md
Service: azure-advanced-threat-protection
GitHub Login: @mlottner
Microsoft Alias: mlottner

@WorldMir Thanks for your feedback. I'll update the article to make the licensing more clear. Appreciate you taking the time to contribute to the docs community.

#please-close


WebSocket not working on new messages

See : https://github.com/pagermon/pagermon/blob/2f24061fd6f45dd01bfb0dce7f226dcccbd3e409/server/routes/api.js#L712-L756
This portion of code does not work inside the plugin callback.
Does it need to be in this callback? Can't it be output?
I think it should be out of the callback, as the message is already stored in the database, it can be sent to the socket.io

It's in the callback to force it to be synchronous - you're right though, probably doesn't need to be for the after plugins. Curious why it would be failing though, broken plugin?

I don't know, but I realized that it worked well out of the plugin callback :)

I'm unable to replicate this issue in the current master branch.
Do you have any local plugins that could be causing this issue?

Maybe, I was in development on https://github.com/pagermon/pagermon/pull/283
Now it seems to be working properly, really sorry.
It can be closed.


[stable/rabbitmq] Update image to new bash container

Signed-off-by: Alejandro Moreno<EMAIL_ADDRESS>What this PR does / why we need it:
Updates the container to use the new container with bash logic.
Special notes for your reviewer:
As the chart has its own logic in a custom command, the changes to the underlying image shouldn't affect the chart. To upgrade, it maybe needed to use the forceBoot option, but as far as I know that is also the case with the current version.
This is a copy #14529.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Following https://github.com/helm/charts/pull/14529 after a rebase issue, @alemorcuq applied all the suggestions in the previous PR

/ok-to-test
/lgtm


Add THEN and ELSE as logical operators

Make the lexer know about the existence of the THEN and ELSE word as logical operators to tokenize and sanitize them.
Part of: https://github.com/appsignal/support/issues/270

We can also add the INTERVAL operator and sanitize whatever it takes, but I'm not sure if the info that can be passed to it must be considered sensitive. WDYT?

We can also add the INTERVAL operator and sanitize whatever it takes, but I'm not sure if the info that can be passed to it must be considered sensitive. WDYT?

Dates can be considered sensitive, but it'll be more difficult to determine what the date is without a column, but I think we'll sanitize the values it modifies already so let's leave it.


rancherctl service enable: when using a custom service, it's annoying that the file must be placed in a specific folder location

It's annoying that I have to save a folder in a specific folder location (/var/lib/rancher/conf/) AND I have to name the the service /var/lib/rancher/conf/example.yml in order for my service to work.

enforce needing the location of a service enabled to either /var/lib/rancher/conf or /home or /opt

@deniseschannon What is needed here?

yes, basically an error message indicating that the yaml file needs to be in /var/lib/rancher/conf/ if it can't be found.
right now, we add in the name of the service into the rancher.yml even if the file isn't located in that directory  like i did rancherctl service enable test.yml which obviously wouldn't work.

I think enable should validate that the service is

A name that contains no /
A file that exists and starts with /var/lib/rancher/conf
A url that starts with http:// or https://

@ibuildthecloud If you look at the code change I made, the above three cases are taken care of.
https://github.com/rancherio/os/pull/273

I didn't realize #1, #2 was taken care of.  I'll try to run the code.

@ibuildthecloud  -- If #1 is not met, then it throws an error
It doesn't enforce #2, but we have been following the convention of putting these files in /var/lib/rancher/conf. I should enforce it. I will make that change now

@ibuildthecloud I've pushed the changes to enforce #2.

Also, Im guessing #1 means no trailing / right?

Just tested this out with rancherctl service enable test.yml. The error message was "could not load service". Would it be okay if we made the change so that if the service ended in .yml, it shows the message that the file needs to be in /var/lib/rancher/conf?

We can check if it doesn't start with http then print that message.

@ibuildthecloud just made the PR.


[BEAM-5723] Do not use the default shadow closure when building CassandraIO

This addresses https://issues.apache.org/jira/browse/BEAM-5723
A part of https://issues.apache.org/jira/browse/BEAM-6620
Instead of extending the default shadow closure, replace it.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Java PreCommit

@kennknowles it seems we need to exclude this from validateShadedJarDoesntLeakNonProjectClasses too.

This actually exposed a huge bug in our build, if I understand it. It isn't just the Guava classes used here, but all of the ones relocated by any other module.

Ah, no it is just that excludes are not additive to the default set.

Hmm, I am not exactly sure why, but the failure is listing essentially every class. It may have to do with not having this block:
dependencies {
  include(dependency(project.library.java.guava))
}

Everything else in the DEFAULT_SHADOW_CLOSURE is just relocations of Guava.
Almost everything causing the relocation error should be OK, and should not be in the shaded jar.

Thanks for checking so thoroughly!

It looks like this may have broken :beam-sdks-java-io-cassandra:test

Well, it passed in presubmit. It could be this or Etienne's or a combo of them.

It passes locally for me, maybe it's a combo or Cassandra-related tests flakiness


Update Twenty extension - support self-hosted Twenty

Description
I'm self-hosting Twenty so updated the extension accordingly. Previously, when I reviewed PR # #14778 I did a find and replace to my self hosted instance for the testing.
I added a new useApiUrl hook that generates the URL using new URL to prevent having to check for trailing slashes and the like. We really need a native Raycast Preference of type URL 👀.
Screencast
N/A as there's data which would be leaked but don't change the Preference and that will allow you to test using Twenty Cloud.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for the update @xmok
@chrisdadev13 do you want to check this?


Improve XRT test suite

The test script currently only tests a limited set of inputs which sometimes results in bugs being merged into the project without us realizing. The script is also turning quite large and crashes on the first error. We should look into switching to a unit test framework like GoogleTest.
Improvements

[ ] Test all data types and multiple data sizes
[ ] Use different types of buffers for all tests (unaligned, aligned, bo buffer)
[ ] Continue testing on error
[x] Switch to unit test framework (i.e. GoogleTest)

WIP switch to gtest in #138


[No ticket] Stop linking users to the old ember-osf-registries repo

Ticket: [No ticket]
Feature flag: n/a

Purpose

Link users to the up-to-date repo for registries related stuff

Summary of Changes

Update places where we are linking users to the old ember-osf-registries repo with ember-osf-web

Screenshot(s)
Affected links are here:

osf.io/registries

Side Effects
None
QA Notes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.157%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5176

Relevant Lines:
6812

💛  - Coveralls


Travis not building on pull requests

Looks like travis is not set build on pull requests. Is there any reason why this was turned off, or can we re-enable it?

Fixed


Initial connection to Kubernetes cluster failed with error Get

os  Rocky Linux 9.1
rancher 2.7.8
after create a cluster  in rancher,  when joining the first node to the cluster , the blow error occurs:
Oct 23 14:59:05 n1 rancher-system-agent[1774903]: time="2023-10-23T14:59:05+08:00" level=info msg="Initial connection to Kubernetes cluster failed with error Get "https://rancher2.cnzhonglunnet.com/version\": x509: certificate signed by unknown authority, removing CA data and trying again"
Oct 23 14:59:05 n1 rancher-system-agent[1774903]: E1023 14:59:05.799839 1774903 memcache.go:206] couldn't get resource list for management.cattle.io/v3:
the certificate for the domain is valid. the https://rancher2.cnzhonglunnet.com/version cann accessed nornally.
any fix, thank you .

os Rocky Linux 9.1
rancher 2.7.8
after create a cluster in rancher, when joining the first node to the cluster , the blow error occurs:
Oct 23 14:59:05 n1 rancher-system-agent[1774903]: time="2023-10-23T14:59:05+08:00" level=info msg="Initial connection to Kubernetes cluster failed with error Get "[https://rancher2.cnzhonglunnet.com/version](https://rancher2.cnzhonglunnet.com/version\)": x509: certificate signed by unknown authority, removing CA data and trying again" Oct 23 14:59:05 n1 rancher-system-agent[1774903]: E1023 14:59:05.799839 1774903 memcache.go:206] couldn't get resource list for management.cattle.io/v3:
the certificate for the domain is valid. the https://rancher2.cnzhonglunnet.com/version cann accessed nornally.
any fix, thank you .

How to solve the same problem when encountering it? Or which version of the ranger does not have this issue?

logs ...
[root@k8s-prod-m2 opt]# journalctl -u rancher-system-agent.service -f
Jun 05 15:10:39 k8s-prod-m2 systemd[1]: Started Rancher System Agent.
Jun 05 15:10:39 k8s-prod-m2 rancher-system-agent[1094]: time="2024-06-05T15:10:39+08:00" level=info msg="Rancher System Agent version v0.3.6 (41c07d0) is starting"
Jun 05 15:10:39 k8s-prod-m2 rancher-system-agent[1094]: time="2024-06-05T15:10:39+08:00" level=info msg="Using directory /var/lib/rancher/agent/work for work"
Jun 05 15:10:39 k8s-prod-m2 rancher-system-agent[1094]: time="2024-06-05T15:10:39+08:00" level=info msg="Starting remote watch of plans"
Jun 05 15:10:39 k8s-prod-m2 rancher-system-agent[1094]: time="2024-06-05T15:10:39+08:00" level=info msg="Initial connection to Kubernetes cluster failed with error Get \"https://rancher.test.com/version\": tls: failed to verify certificate: x509: certificate signed by unknown authority, removing CA data and trying again"
Jun 05 15:10:40 k8s-prod-m2 rancher-system-agent[1094]: time="2024-06-05T15:10:40+08:00" level=info msg="Starting /v1, Kind=Secret controller"

There are now 3 issues with this same pattern but no response yet and they are all open.
It's the end of 2024, can a little attention be given to this.
issue 1
issue 2
issue 3


Use interpreter selected by Python extension

The Python extension allows you to select which interpreter executable to use. We should ensure this is used when running pydoctest.

This describes how to create a dependency on another extension:
https://blog.lextudio.com/different-ways-to-create-vscode-extension-dependencies/
This describes how to get interpreterPath from the vscode-python extension:
https://github.com/microsoft/vscode-python/issues/18727
Since the vscode-python API is still in development, we should also allow specifying it manually as a setting.

This is implemented, however there are changes in the vscode-python API from 2022.14.x.


Web App - Navigation Layout Changes For Administrator depending on whether Viewing Administrator or Volunteer Components

The navigation bar on the website changes depending on which page is viewed, either an administrator page or a volunteer page, and does not show all functionality at once requiring multiple jumps for navigation.
Terminology:
administrator 'request' page (create/edit requests): /Requests
volunteer 'request' page (view open/self assign/complete): /Requests/VolunteerRequestIndex
Only one of these is viewable by the administrator at a time.
In order to view the administrator 'Request' page from the volunteer 'Request' page (or the 'View Checkins' or 'View Data' pages), the administrator must go to their homepage. In order to access the volunteer 'Request', the administrator must navigate to either the 'View Checkins' or 'View Data' pages.
I believe the administrator should be able to access both of the pages at any time.

Hi @pgeeh, I noticed this too.  I thought I fixed this issue with #663.   Can you make sure that you have the latest code pulled down and check again?  Are you looking at the test website or are you running the solution locally?


feat(signs): enable staged signs by default

I was updating my plugins and noticed a significant behavioural change in gitsigns that might be related to this pull request.
I started seeing staged commits that previously displayed nothing in the gutter, were now displaying icons I did I was not using, such as the pipes for add.
It would have been beneficial if this change was highlighted in the sample config, with an explanation about the new signs_staged feature.
For example:
require('gitsigns').setup {
  signs = {
    add          = { text = '┃' },
    change       = { text = '┃' },
    delete       = { text = '_' },
    topdelete    = { text = '‾' },
    changedelete = { text = '~' },
    untracked    = { text = '┆' },
  },
  -- [Missing Section]
  -- Icons can be configured separately for staged content
  -- signs_staged = {
  --   add          = { text = '┃' },
  --   change       = { text = '┃' },
  --   delete       = { text = '_' },
  --   topdelete    = { text = '‾' },
  --   changedelete = { text = '~' },
  --   untracked    = { text = '┆' },
  -- },
  signcolumn = true,  -- Toggle with `:Gitsigns toggle_signs`
  numhl      = false, -- Toggle with `:Gitsigns toggle_numhl`
  linehl     = false, -- Toggle with `:Gitsigns toggle_linehl`
  word_diff  = false, -- Toggle with `:Gitsigns toggle_word_diff`
  watch_gitdir = {
    follow_files = true
  },
  auto_attach = true,
  attach_to_untracked = false,
  current_line_blame = false, -- Toggle with `:Gitsigns toggle_current_line_blame`
  current_line_blame_opts = {
    virt_text = true,
    virt_text_pos = 'eol', -- 'eol' | 'overlay' | 'right_align'
    delay = 1000,
    ignore_whitespace = false,
    virt_text_priority = 100,
  },
  current_line_blame_formatter = '<author>, <author_time:%R> - <summary>',
  sign_priority = 6,
  update_debounce = 100,
  status_formatter = nil, -- Use default
  max_file_length = 40000, -- Disable if file is longer than this (in lines)
  preview_config = {
    -- Options passed to nvim_open_win
    border = 'single',
    style = 'minimal',
    relative = 'cursor',
    row = 0,
    col = 1
  },
}

It might have been smoother if the signs_staged icons were defaulted to the initialized icons in signs for a more seamless transition for existing configurations.
Thanks for all your open source contributions, I added the default entry to the read me in https://github.com/lewis6991/gitsigns.nvim/pull/1077


Set remote-state.outputs to null

what

Set remote-state.outputs to null

why

Avoid Error: Inconsistent conditional result types

references

Previous PR https://github.com/cloudposse/terraform-yaml-stack-config/pull/32

/test all


Move db passwords to config file

Mike and I were discussing this and we'd like to implement this change now.
I found the following thread about this:
http://stackoverflow.com/questions/2397822/what-is-the-best-practice-for-dealing-with-passwords-in-github

This issue was moved to MetropolitanTransportationCommission/CAPVMT#21


Improve the initial user experience

Improve OOBE (Out of Box Experience)
Background & Purpose
On Purpose is hard to figure out and know how to use for a new user. This greatly limits the impact and limits the ability for On Purpose to help others because they don't understand how to use it and be successful. The purpose of this work is to make it more straightforward for someone to understand and try out On Purpose so they can successfully evaluate if it is right for them. This means having a video that summarizes the features, and it means improving On Purpose to be more usable.
Measurement of Success
A newcomer who is interested in trying out On Purpose attests that they believe they understand the program and how it works and as they use the program it can be observed that they grasp how to use the program.
Implementation Details
Do the following:

Make the 'Esc' key always go back. In places were ESC does something more turn that into a menu option.
Remove todo! from the code that cause the program to just crash when a menu item is selected. But don't just delete these, make sure that an issue is filed for them as previously this was used to state and track work to do.
Make a getting started video that explains the concepts of On Purpose and shows how to use it
Make it so you can always enter a date or a time
Get feedback from others who try the program on improvements to make
Look for ways to improve the terminology. For example, call it purpose or motivation rather than parent.
Do we want some kind of tutorial to happen when they first use the program that guides them through the experience?

I have done more work on this on Saturday removing various unimplemented menu options and moving those to be GitHub Issues.

I also made the starting menu, the Bullet List have two more menu options one for refresh and one for go to the main menu. Previously these two pieces of functionality required pressing the Esc key.


meta: add daavoo to maintainers

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Adds @daavoo to maintainers

@conda-forge-admin, please rerender


Fixed typos

@mluypaert the tests are never going to run for this PR however we still require both these tests to pass... now sure how to fix this other than to force merge the PR?


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-console-menu.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "nunomaduro/laravel-console-menu": "dev-l11-compatibility",
    }
}

Finally, run: composer update


feat: Re-enable rar support

This re-enables attempted support for .rar files. It will attempt to decompress with Rar5 first, and fallback to the older Rar method if that fails.
If both methods fail, the user will get an error notification with the error messages from both attempts.
I tested with the main .rar files from https://github.com/ssokolow/rar-test-files. This successfully extracted the testfile.rar5.rar file, but still fails on the testfile.rar3.rar file, so support will probably still be spotty.

Ill tentatively re-add this for 2.0. seems to fail on some of my test files but some support is better than none!


solana: fix any broken solver examples

With the latest Solana devnet upgrade, we need to make sure the solver examples still work.

I've been facing issues running the solver example. Is this issue stale for now or would it be worked upon? Thanks.

Would be great if these were updated


Disable keyboard input

Set up a hotkey to disable keyboard input or automaticly disable when focus on window is lost

Thanks for taking the time to look into it

I understand this takes a lot of effort to fix, but I also am facing the same issue.
I often leave my game running while doing some other things on the side on the same PC, which makes Link move, or the cam move unintentionally because of inputs captured from outside the game.

@mistermark Yes, it is something I have in mind, but as I already said, it'll take a lot of time because I have to rewrite the entire input system. Right now the solution I see is hooking into GetMessage to at least receive the message of when the window is on focus, but this means I have to update memory-rs to be able to easily hook to functions outside of Cemu.


Add file decompression function when not in Windows system

Features
Add file decompression function when not in Windows system

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9e49e22) 85.31% compared to head (7e93cf9) 85.41%.
Report is 2 commits behind head on dev.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #811      +/-   ##
==========================================
+ Coverage   85.31%   85.41%   +0.09%
==========================================
  Files         183      183
  Lines        9788     9788
==========================================
+ Hits         8351     8360       +9
+ Misses       1437     1428       -9

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

@garylin2099 mark this pr as approve (in files tab) if it's ok


VSCode should suggest cleaning up unused workspaces

See the attached image for size context: (the workspace .vscode folders are gone for these workspaces)

VS Code version: Code - Insiders 1.32.0-insider (507312a3e3b34b084b467dfd983263bc7c9d87e0, 2019-03-06T10:06:48.833Z)
OS version: Windows_NT x64 10.0.17763

I'm not sure how we can tell when you'll be done with the workspace entirely. What's using all that space, is it in workspaceStorage? You must have an extension caching huge amounts of data?

Hey @rob, I’m happy to dump more data - it’s likely the cpp extension intelligence database. What info should I capture for you? Sample scenario here would be you have a mapping from the workspace storage to a workspace folder and the workspace folder gets deleted, I think that would be good grounds for concidering storage deletion, or prompting the user if storage is low (my disk was full). Simple edge case is someone moves or renamed a folder (but that’s not a intractable)

@kevindiclemente it would help if you can drill into the folder to see which one (with parents) is the large one


test: add linter to all packages and enable vitest

Relates to:

Risks
Low
Background
What does this PR do?

Enable linter to all packages and add the vitest plugin to the eslint configs to enable linter on test files.
Add a global lint script that run all packages lint command all at once.

What kind of change is this?

Linter improvement to improve code quality across the packages.

Documentation changes needed?

No

Testing
Where should a reviewer start?
Detailed testing steps

Run:
pnpm lint

and it will run the linter against all the packages.

$ pnpm lint

> eliza@ lint /tmp/eliza
> bash ./scripts/lint.sh

Linting package: plugin-video-generation
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_17_348Z-debug-0.log
Running lint for package: plugin-video-generation

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-video-generation/src/index.ts
  122:46  error  'message' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

✖ 1 problem (1 error, 0 warnings)

Lint failed for plugin-video-generation
Linting package: client-auto
Running lint for package: client-auto

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-auto/src/index.ts
  55:15  error  'tokenInfos' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  90:18  error  'runtime' is defined but never used. Allowed unused args must match /^_/u              @typescript-eslint/no-unused-vars

✖ 2 problems (2 errors, 0 warnings)

Lint failed for client-auto
Linting package: plugin-node
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_19_343Z-debug-0.log
Running lint for package: plugin-node

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-node/src/services/browser.ts
  265:21  error  Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free  @typescript-eslint/ban-ts-comment
  278:21  error  Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free  @typescript-eslint/ban-ts-comment

/tmp/eliza/packages/plugin-node/src/services/image.ts
  97:38  error  'reject' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-node/src/services/llama.ts
  184:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-node/src/services/pdf.ts
  12:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-node/src/services/speech.ts
    2:25  error  'ISpeechService' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
   79:17  error  Expected an assignment or function call and instead saw an expression             @typescript-eslint/no-unused-expressions
  115:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u         @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-node/src/services/transcription.ts
  30:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars
  71:22  error  'error' is defined but never used                                          @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-node/src/services/video.ts
   28:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars
  138:95  error  Unnecessary escape character: \/                                           no-useless-escape

/tmp/eliza/packages/plugin-node/src/vendor/vits.ts
  1475:36  error    'mean' is assigned a value but never used. Allowed unused vars must match /^_/u                                      @typescript-eslint/no-unused-vars
  1475:42  error    'stdDeviation' is assigned a value but never used. Allowed unused vars must match /^_/u                              @typescript-eslint/no-unused-vars
  1909:9   error    Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free  @typescript-eslint/ban-ts-comment
  2148:14  error    'newValue' is defined but never used. Allowed unused args must match /^_/u                                           @typescript-eslint/no-unused-vars
  2635:38  error    Unnecessary escape character: \,                                                                                no-useless-escape
  2635:40  error    Unnecessary escape character: \.                                                                                no-useless-escape
  2641:32  error    Unnecessary escape character: \-                                                                                no-useless-escape
  2641:34  error    Unnecessary escape character: \–                                                                                no-useless-escape
  3061:26  error    Do not access Object.prototype method 'hasOwnProperty' from target object                                            no-prototype-builtins
  3110:29  error    Do not access Object.prototype method 'hasOwnProperty' from target object                                            no-prototype-builtins
  4156:9   error    Unexpected aliasing of 'this' to local variable                                                                      @typescript-eslint/no-this-alias
  4416:13  error    'fragments' is assigned a value but never used. Allowed unused vars must match /^_/u                                 @typescript-eslint/no-unused-vars
  4417:13  error    'phonemizedFragmentsSubstitutions' is assigned a value but never used. Allowed unused vars must match /^_/u          @typescript-eslint/no-unused-vars
  4691:5   warning  'detectedLanguageProbabilities' is never reassigned. Use 'const' instead                                             prefer-const
  4747:10  error    'int8ToBuffer_Slow' is defined but never used. Allowed unused vars must match /^_/u                                  @typescript-eslint/no-unused-vars
  4761:10  error    'bufferToInt8_Slow' is defined but never used. Allowed unused vars must match /^_/u                                  @typescript-eslint/no-unused-vars
  4776:10  error    'int16ToBufferLE_Slow' is defined but never used. Allowed unused vars must match /^_/u                               @typescript-eslint/no-unused-vars
  4794:10  error    'bufferLEToInt16_Slow' is defined but never used. Allowed unused vars must match /^_/u                               @typescript-eslint/no-unused-vars
  4840:10  error    'int32ToBufferLE_Slow' is defined but never used. Allowed unused vars must match /^_/u                               @typescript-eslint/no-unused-vars
  4858:10  error    'bufferLEToInt32_Slow' is defined but never used. Allowed unused vars must match /^_/u                               @typescript-eslint/no-unused-vars
  4873:10  error    'float32ToBufferLE_Slow' is defined but never used. Allowed unused vars must match /^_/u                             @typescript-eslint/no-unused-vars
  4891:10  error    'bufferLEToFloat32_Slow' is defined but never used. Allowed unused vars must match /^_/u                             @typescript-eslint/no-unused-vars
  4906:10  error    'float64ToBufferLE_Slow' is defined but never used. Allowed unused vars must match /^_/u                             @typescript-eslint/no-unused-vars
  4924:10  error    'bufferLEToFloat64_Slow' is defined but never used. Allowed unused vars must match /^_/u                             @typescript-eslint/no-unused-vars
  5298:14  error    'e' is defined but never used                                                                                @typescript-eslint/no-unused-vars
  5298:17  error    Empty block statement                                                                                no-empty
  5828:11  error    'startTimestamp' is assigned a value but never used. Allowed unused vars must match /^_/u                            @typescript-eslint/no-unused-vars
  5832:11  error    'simplifiedText' is assigned a value but never used. Allowed unused vars must match /^_/u                            @typescript-eslint/no-unused-vars
  5866:11  error    'voiceGender' is assigned a value but never used. Allowed unused vars must match /^_/u                               @typescript-eslint/no-unused-vars
  6477:12  error    Unnecessary escape character: \-                                                                                no-useless-escape
  6551:15  error    'isLastUpdate' is assigned a value but never used. Allowed unused vars must match /^_/u                              @typescript-eslint/no-unused-vars
  6783:16  error    'commandExists' is already defined                                                                                no-redeclare
  7173:11  error    'startTimestamp' is assigned a value but never used. Allowed unused vars must match /^_/u                            @typescript-eslint/no-unused-vars
  7298:5   warning  'mappedTimeline' is never reassigned. Use 'const' instead                                                            prefer-const
  7661:5   error    'insertSeparators' is defined but never used. Allowed unused args must match /^_/u                                   @typescript-eslint/no-unused-vars
  7662:5   error    'useKlatt' is defined but never used. Allowed unused args must match /^_/u                                           @typescript-eslint/no-unused-vars
  7678:19  warning  'segmentTimeline' is never reassigned. Use 'const' instead                                                           prefer-const
  7679:16  warning  'espeakVoice' is never reassigned. Use 'const' instead                                                               prefer-const
  7837:10  error    'multiplyTimelineByFactor' is defined but never used. Allowed unused vars must match /^_/u                           @typescript-eslint/no-unused-vars
  8026:14  error    'e' is defined but never used                                                                                @typescript-eslint/no-unused-vars
  8320:10  error    'getAllLangCodesFromVoiceList' is defined but never used. Allowed unused vars must match /^_/u                       @typescript-eslint/no-unused-vars

✖ 53 problems (49 errors, 4 warnings)

Lint failed for plugin-node
Linting package: core
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_20_683Z-debug-0.log
Running lint for package: core

><EMAIL_ADDRESS>lint
> eslint . --fix

(node:99234) ESLintIgnoreWarning: The ".eslintignore" file is no longer supported. Switch to using the "ignores" property in "eslint.config.js": https://eslint.org/docs/latest/use/configure/migration-guide#ignoring-files
(Use `node --trace-warnings ...` to show where the warning was created)

/tmp/eliza/packages/core/src/tests/token.test.ts
  8:33  error  Describe callback cannot be async  vitest/valid-describe-callback

✖ 1 problem (1 error, 0 warnings)

Lint failed for core
Linting package: plugin-solana
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_21_998Z-debug-0.log
Running lint for package: plugin-solana

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-solana/src/actions/pumpfun.ts
    7:5   error  'DEFAULT_DECIMALS' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  237:5   error  Unreachable code                                                                    no-unreachable
  278:22  error  'runtime' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars
  278:46  error  'message' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-solana/src/actions/swap.ts
  328:22  error  'e' is defined but never used   @typescript-eslint/no-unused-vars
  334:26  error  'e2' is defined but never used  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-solana/src/actions/takeOrder.ts
   6:5   error  'ActionExample' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
   9:13  error  'fs' is defined but never used. Allowed unused vars must match /^_/u             @typescript-eslint/no-unused-vars
  10:10  error  'settings' is defined but never used. Allowed unused vars must match /^_/u       @typescript-eslint/no-unused-vars
  35:15  error  'text' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-solana/src/providers/token.ts
  895:19  error  'ownerPercentage' is assigned a value but never used. Allowed unused vars must match /^_/u    @typescript-eslint/no-unused-vars
  896:19  error  'creatorPercentage' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-solana/src/providers/trustScoreProvider.ts
   32:11  error  'RecommendationGroup' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  608:9   error  'state' is defined but never used. Allowed unused args must match /^_/u                @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-solana/src/providers/wallet.ts
  36:11  error  'BirdEyePriceData' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

✖ 15 problems (15 errors, 0 warnings)

Lint failed for plugin-solana
Linting package: adapter-sqlite
Running lint for package: adapter-sqlite

><EMAIL_ADDRESS>lint
> eslint . --fix

Successfully linted adapter-sqlite

Linting package: client-github
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_24_593Z-debug-0.log
Running lint for package: client-github

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-github/src/index.ts
   13:5   error  'Content' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  233:18  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

✖ 2 problems (2 errors, 0 warnings)

Lint failed for client-github
Linting package: plugin-trustdb
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_25_922Z-debug-0.log
Running lint for package: plugin-trustdb

><EMAIL_ADDRESS>lint
> eslint . --fix

Successfully linted plugin-trustdb

Linting package: create-eliza-app
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_27_079Z-debug-0.log
Running lint for package: create-eliza-app

><EMAIL_ADDRESS>lint
> eslint . --fix

Successfully linted create-eliza-app

Linting package: plugin-starknet
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_28_305Z-debug-0.log
Running lint for package: plugin-starknet

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-starknet/src/actions/swap.ts
  19:30  error  'validateSettings' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  83:46  error  'message' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/actions/takeOrder.ts
  54:15  error  'text' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/actions/transfer.ts
  19:5   error  'validateSettings' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  61:46  error  'message' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/actions/unruggable.ts
  19:5   error  'validateSettings' is defined but never used. Allowed unused vars must match /^_/u        @typescript-eslint/no-unused-vars
  24:5   error  'EKUBO_TICK_SPACING' is defined but never used. Allowed unused vars must match /^_/u      @typescript-eslint/no-unused-vars
  25:5   error  'LiquidityType' is defined but never used. Allowed unused vars must match /^_/u           @typescript-eslint/no-unused-vars
  27:5   error  'RECOMMENDED_EKUBO_FEES' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  85:46  error  'message' is defined but never used. Allowed unused args must match /^_/u                 @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/providers/token.ts
   12:5   error  'TokenTradeData' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  407:37  error  'tradeData' is defined but never used. Allowed unused args must match /^_/u       @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/providers/trustScoreProvider.ts
   23:10  error  'walletProvider' is defined but never used. Allowed unused vars must match /^_/u       @typescript-eslint/no-unused-vars
   26:7   error  'Wallet' is assigned a value but never used. Allowed unused vars must match /^_/u      @typescript-eslint/no-unused-vars
   35:11  error  'RecommendationGroup' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  608:9   error  'state' is defined but never used. Allowed unused args must match /^_/u                @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/providers/walletProvider.ts
    2:10  error  'Connection' is defined but never used. Allowed unused vars must match /^_/u        @typescript-eslint/no-unused-vars
    2:22  error  'PublicKey' is defined but never used. Allowed unused vars must match /^_/u         @typescript-eslint/no-unused-vars
   37:11  error  'BirdEyePriceData' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  109:31  error  'runtime' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars
  166:23  error  'runtime' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars
  178:19  error  'tokens' is assigned a value but never used. Allowed unused vars must match /^_/u   @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-starknet/src/utils/ERC20Token.ts
  1:19  error  'Call' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

✖ 23 problems (23 errors, 0 warnings)

Lint failed for plugin-starknet
Linting package: adapter-sqljs
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_29_686Z-debug-0.log
Running lint for package: adapter-sqljs

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/adapter-sqljs/src/types.ts
  138:5  error  Duplicate name 'each'  no-dupe-class-members

✖ 1 problem (1 error, 0 warnings)

Lint failed for adapter-sqljs
Linting package: plugin-bootstrap
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_30_850Z-debug-0.log
Running lint for package: plugin-bootstrap

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-bootstrap/src/actions/ignore.ts
  17:9  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars
  18:9  error  'message' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-bootstrap/src/actions/none.ts
  24:9  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars
  25:9  error  'message' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/plugin-bootstrap/src/providers/boredom.ts
  340:9  error  Unreachable code  no-unreachable

/tmp/eliza/packages/plugin-bootstrap/src/providers/facts.ts
  16:15  error  'embedding' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

✖ 6 problems (6 errors, 0 warnings)

Lint failed for plugin-bootstrap
Linting package: client-telegram
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_31_788Z-debug-0.log
Running lint for package: client-telegram

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-telegram/src/index.ts
  22:18  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

✖ 1 problem (1 error, 0 warnings)

Lint failed for client-telegram
Linting package: client-discord
Running lint for package: client-discord

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-discord/src/actions/chat_with_attachments.ts
  94:63  error  'state' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/actions/download_media.ts
  13:5   error  'Service' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  76:63  error  'state' is defined but never used. Allowed unused args must match /^_/u    @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/actions/joinvoice.ts
    1:1   error  Do not use "@ts-nocheck" because it alters compilation errors                       @typescript-eslint/ban-ts-comment
  170:19  error  'datestr' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/actions/leavevoice.ts
  93:33  error  'channel' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/actions/summarize_conversation.ts
  256:15  error  'datestr' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/actions/transcribe_media.ts
  76:63  error  'state' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/attachments.ts
  11:5  error  'Service' is defined but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/index.ts
  306:18  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-discord/src/voice.ts
  388:15  error  'monitor' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

✖ 11 problems (11 errors, 0 warnings)

Lint failed for client-discord
Linting package: client-direct
Running lint for package: client-direct

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-direct/src/index.ts
   14:5   error  'State' is defined but never used. Allowed unused vars must match /^_/u            @typescript-eslint/no-unused-vars
  225:23  error  'result' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars
  288:19  error  'runtime' is defined but never used. Allowed unused args must match /^_/u          @typescript-eslint/no-unused-vars
  295:18  error  'runtime' is defined but never used. Allowed unused args must match /^_/u          @typescript-eslint/no-unused-vars

✖ 4 problems (4 errors, 0 warnings)

Lint failed for client-direct
Linting package: client-twitter
Running lint for package: client-twitter

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/client-twitter/src/index.ts
  40:16  error  'runtime' is defined but never used. Allowed unused args must match /^_/u  @typescript-eslint/no-unused-vars

/tmp/eliza/packages/client-twitter/src/utils.ts
  273:42  error  Unnecessary escape character: \.  no-useless-escape
  273:45  error  Unnecessary escape character: \?  no-useless-escape
  273:50  error  Unnecessary escape character: \.  no-useless-escape
  273:53  error  Unnecessary escape character: \?  no-useless-escape
  273:60  error  Unnecessary escape character: \.  no-useless-escape
  273:63  error  Unnecessary escape character: \?  no-useless-escape

✖ 7 problems (7 errors, 0 warnings)

Lint failed for client-twitter
Linting package: adapter-postgres
Running lint for package: adapter-postgres

><EMAIL_ADDRESS>lint
> eslint . --fix

Successfully linted adapter-postgres

Linting package: plugin-image-generation
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_37_908Z-debug-0.log
Running lint for package: plugin-image-generation

><EMAIL_ADDRESS>lint
> eslint . --fix

/tmp/eliza/packages/plugin-image-generation/src/index.ts
   10:10  error  'generateCaption' is defined but never used. Allowed unused vars must match /^_/u   @typescript-eslint/no-unused-vars
   79:46  error  'message' is defined but never used. Allowed unused args must match /^_/u           @typescript-eslint/no-unused-vars
  151:23  error  'caption' is assigned a value but never used. Allowed unused vars must match /^_/u  @typescript-eslint/no-unused-vars

✖ 3 problems (3 errors, 0 warnings)

Lint failed for plugin-image-generation
Linting package: adapter-supabase
npm error A complete log of this run can be found in: /tmp/.npm/_logs/2024-11-21T17_58_38_863Z-debug-0.log
Running lint for package: adapter-supabase

><EMAIL_ADDRESS>lint
> eslint . --fix

Successfully linted adapter-supabase

Lint process completed.😎

LGTM! Thanks for working on this will be a gamechanger for maintainability :)

@snobbee @monilpat we hav two conflicts in:
packages/plugin-starknet/package.json
packages/plugin-trustdb/package.json

Yeah please address merge conflicts. We really need to address https://github.com/ai16z/eliza/issues/457 let me look into this.

nice work. Can you:

make a higher level eslint config file and have packages import relevant settings
can you add a precommit function to run linting and prettier and suggest fixes


Broken URLs

qlog 0.4.0 link is down. I'd recommend directly linking to the respective docs.rs page.
aioquic link is https://github.com/aiortc/aioquic
Haskell quic link is https://github.com/kazu-yamamoto/quic
Sadly, I couldn't find any archive of the now-gone article The New Internet: HTTP/3 — No More TCP, let’s QUIC fix it.

By the way, thank you for maintaining this list, it's helpful for a lot of people!!

Hi @Mennaruuk ,
Thanks for your contribution, I just fixed the broken URLs.
About the article "The New Internet: HTTP/3 — No More TCP, let’s QUIC fix it.", unfortunately I can't find the archive either, seems the author's account was banned. I removed this article from QUIC Weekly for now.

Appreciate your assistance! Hope you have a good day/night :)


swift integration

hi,
Please i need your help to integrate into swift code in a very easy way, with clear steps.
Thank you very much for your help

I'm not sure what do you want to do

I need CLImageEditor in swift , Any one help me?

see #163

I know about this answer , but my requirement changes in Image Editing controller as per client requirement , So its require in swift language.

You can write your own swift implementation using this library as example
You can pay someone who can write swift implementation


Added Pnp bipolar transistor

Fixes #40
updates:

remade svg for pnp_bipolar_transistor, previous svg had orientation and symmetry issues
generated and finalized ts files for pnp bipolar transistor
fixed incorrect names in svg object

cc: @seveibar

@mrudulpatil18 I can see that you have added npn_bipolar svg as well. Can you please remove that from the PR?

@mrudulpatil18 I can see that you have added npn_bipolar svg as well. Can you please remove that from the PR?

@imrishabh18
i have just made a minor change in the svg file of npn bipolar, name of the object in the earlier svg was 'photo_diode'. I fixed that , shall i still remove from this PR ?

Ohh okay, never mind looks great. Resolve the merge conflicts and we are good

Thanks!


Adds additional property null checking

Goes through existing toolkit components and adds additional null checks to properties.

Closing


[Bug] Plugins calling mango.raise would crash Mango on ARM32

Plugin
catmanga
Describe the bug
searching for nonexistent manga results in a "0" error and unexpected logout
To Reproduce
Steps to reproduce the behavior:

open mango plugins, select catmanga
search for any manga title from catmanga (cafe for instance)
results are returned
search for a manga title not on catmanga (nagatoro for instance)
Error popup is returned from mango
navigate away from the search page
see login screen

Expected behavior
graceful search failure
Screenshots
If applicable, add screenshots to help explain your problem.
Mango version
v0.21 in arm64v8 docker container
Additional context
similar search with another engine, manganelo, worked ok, so i dont believe it to be the install's fault.

Upstream issue opened: https://github.com/jessedoyle/duktape.cr/issues/66

Hi there, just letting you know I'm getting a similar issue on an aarch64 system so it doesn't seem to be ARM32 specific:
mango  | [INFO]    2021/12/05 21:08:13 | Scanned 0 titles in 1.668491ms
mango  | Failed to raise an exception: -741148928
mango  | [0xaaaaacc857a0] ???
mango  | [0xaaaaacac2f08] __crystal_raise +44
mango  | [0xaaaaacbb0378] ???
mango  | [0xaaaaacd5ee90] ???
mango  |
mango  | Tried to raise:: SSL_connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed (OpenSSL::SSL::Error)
mango  |   from usr/share/crystal/src/openssl/ssl/socket.cr:15:11 in 'new:context:sync_close:hostname'
mango  |   from usr/share/crystal/src/http/client.cr:784:5 in 'io'
mango  |   from usr/share/crystal/src/http/client.cr:664:19 in 'send_request'
mango  |   from usr/share/crystal/src/http/client.cr:599:18 in 'exec_internal_single'
mango  |   from usr/share/crystal/src/http/client.cr:587:5 in 'exec'
mango  |   from usr/share/crystal/src/http/client.cr:706:5 in '->'
mango  |   from ???

I'm using the Oracle Cloud free tier which comes with Ampere A1 processors.

As a workaround, I'm currently running the x86_64 container using qus which seems to be performing fine.

@Techno-coder thanks for sharing the workaround!


Terminal onLineData has unexpected line breaks

You can repro this with really any task on Windows, but I discovered this while building vscode:

Open your clone of microsoft/vscode
Start the build task with ctrl+shift+b
Open the developer tools and make your tasks terminal "small"
Put a break point here:

https://github.com/microsoft/vscode/blob/5246162662ffa9f16a70dc2b94f13b0d15511e64/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts#L804-L805

Make a change in a file that causes errors
Expected: onLineData will fire with wrapped lines unwrapped.
Actual: onLineData fires with individual line segments, broken along where they wrap.

Line wrapped file links also appear to be broken in the terminal, possibly related?
I have terminal.integrated.windowsEnableConpty set to true and I'm on Windows build 19043.

Thanks for the issue. It is working for me with windows build number: 21376. I'm going to adjust the conditional to make sure Windows Mode is used for versions below that.

Looks good, thanks!