 drops part of a string

Do you want to request a feature or report a bug?
bug
Bug: What is the current behavior?
Using <Highlight /> parts of a match after the highlighted tag will be dropped. Could be related to having a - directly after the last word.
Can be seen on https://yarnpkg.com/en/packages?q=react-instant, where the first three results are actually different packages (relatively react-instantsearch, react-instantsearch-theme-algolia and react-instantsearch-vl).
The whole string will show up when a query that ends at the dash is given (e.g. https://yarnpkg.com/en/packages?q=react-instantsearch)
Bug: What is the expected behavior?
The complete string to be visible.
Bug: What browsers are impacted? Which versions?
All browsers
What project are you opening an issue for?

react-instantsearch

What is the version you are using? Always use the latest one before opening a bug issue.

Awesome, this was an issue internal to the engine, and has been fixed by @speedblue @Nagriar and @ercolanelli-leo  💯

Fixed on API side 👍


impl DoubleEndedIterator

This looks good. I'd like to add a proptest for double eded check, and to compare benchmarks before and after this change to measure the perf delta.
We're in the middle of a rather large benchmark refactor to use RoaringBitmap/real-roaring-datasets. Lets hold off on this until that's settled to avoid churn.

#192 unblocked. Merge master and we'll have forwards iter benchmarks to compare against.

Thinking out loud: high level steps to get this merged:

Validate implementing DoubleEndedIterator has not caused perf regression for iteration
Add proptest for double ended iteration
Implement DoubleEndedIterator for RoaringTreemap + test, bench

Ok, so the CI crash clearly shows an issue in the Iterator implementation. We are retrieving the numbers from valid RoaringBitmaps and creating Vec<u32> by using .iter(): Numbers should be output in order, this is not the case here.
https://github.com/RoaringBitmap/roaring-rs/blob/d37d20242d55e8accc97999f7c6e99b71582e97c/benchmarks/benches/lib.rs#L143-L156

Found the problem, was a off by one bug. I also changed the tests so they test over a full bitmap store to catch that.

Found the problem, was a off by one bug. I also changed the tests so they test over a full bitmap store to catch that.

Good catch!
I wonder if we can rewrite in terms of double ended-iterator over the bit indices of individual u64s, and a flatmap over the collection of them. 🤔
flatmaps already "know how" to iterate double ended, so we'd get the tricky part of all this for free and could delete a bunch of code.


Add documentation for new bridge driver option

https://github.com/docker/libnetwork/pull/2317 introduced a new bridge option com.docker.network.bridge.inhibit_ipv4 that should be documented
File: network/bridge.md

These are being included in "master" through https://github.com/docker/docker.github.io/issues/10298, so should be documented for the 20.03 release


Is there support for the expand feature of the API?

Hi,
I am looking to use this to pull invoices with expanded customers which is possible with the api.  By the looks of the type for customer (long?), i would have to make subsequent call(s) to get the customer data. Is this correct?

Yes, that is correct. The expand parameter is not supported by the SDK due to the different data structure of the response.


linter config: reformat the lists

Issue #, if available:
Description of changes:
Making it easier to diff changes in future
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

Comparison is base (752114b) 72.93% compared to head (1e1ed65) 72.93%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #545   +/-   ##
=======================================
  Coverage   72.93%   72.93%
=======================================
  Files          26       26
  Lines        1456     1456
=======================================
  Hits         1062     1062
  Misses        322      322
  Partials       72       72

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


Issue 49976 switch condition

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

fixed(Issue 49976) switch condition problem

fixed(Issue 49976 switch condition)

Thank you for opening this pull request.
We’ve reviewed your pull request and decided not to merge it; However, We would welcome future pull requests from you.
You are welcome to read these guidelines to learn more about contributing to this project.
Happy contributing.


Add support for uwsd / WebSocket operation

This PR introduces a number of changes into the tree:

Package the uwsd HTTP/WebSocket server
Add client side support for requesting JavaScript class files and calling ubus RPC procedures via a WebSocket connection
Add a new luci-uwsd collection shipping appropriate default configuration and uwsd server backend for LuCI

This is a work in progress and currently experimental.

@jow-: Have you progressed on your PR?


Cross-package dependencies

I have a multi-repository and would not like to duplicate commands
It would be great to be able to call a task from another directory
a great example is https://github.com/google/wireit?tab=readme-ov-file#cross-package-dependencies
for now if you try to do something like
[tasks.repo1]
dir = "./repo1"
run = "mise run command1"

We will see several levels of nesting
[repo1] $ mise run command1
[repo1] [command1] output

Is there some way to specify which configurations should be loaded?

I think a solution might be to be able to "import" or "include" a config from a sub directory.
Then prefix tasks from the subdirectory like how Taskfile does it.

Is there a roadmap on how to achieve this? or a workaround, even though I have the same task names in different repositories

@jdx, could you add a PR that fixes this, or a link in the documentation or describe how it can be fixed?

https://mise.jdx.dev/tasks/#task-configuration


Support * lists in changelog.md for getChangelog task

Is your feature request related to a problem? Please describe.
I have automated my release workflow including patching the changes in the plugin automatically by the changelog.md of the repository.
This failed for my last release. I didn't realized that I used * list item instead of - list item, as some of the GitHub generated automatic changelog components (new contributors and merged pull requests) already used * for listings.
Describe the solution you'd like
As both symbols can be used in Markdown to create a list, * should be treated equal - when extracting the changelog.
Describe alternatives you've considered
Additional context

You can use the itemPrefix configuration property to specify the prefix for list items. The default value is - but can be set to *. There are details in the help in the README

You can use the itemPrefix configuration property to specify the prefix for list items. The default value is - but can be set to *. There are details in the help in the README

Thanks for your reply!
Is it possible to allow both?

No, you need to specify one.


Copper6SSP: new adapter

🏷 Type of documentation

[x] new bid adapter
[ ] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[ ] Related pull requests in prebid.js or server are linked -> Paste link in this list or reference it on the PR itself
[ ] For new adapters check submitting your adapter docs

I'm going to merge this as prebid.js is already released.


Log Spam with Vanilla IceCream Fix

The detection of Xareo's mods versions should be improved, please use ">=" with versions.
latest.log

please use ">=" with versions.

No. XaeroPlus is explicitly only compatible with a single WorldMap and Minimap version pair per release.
Failing the version check is supposed to crash your game. If this ice cream mod wants to try to prevent the crash, log spam is the least of your problems. I don't see this as something I can or should improve.


OTP modal

The user is unable to paste the OTP while accepting gifts. (In Android )

Will test this in next release

Completed


I really like this theme, but I can’t find how to set a gradient color for strong text.

So far there is no such option, maybe in the next updates.


Fix 404 error crashing server

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:400:5)
at ServerResponse.setHeader (node:_http_outgoing:663:11)
at ServerResponse.header (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:794:10)
at ServerResponse.send (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:174:12)
at ServerResponse.json (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:278:15)
at D:\code\killingsworth-kid-backend\routes\UserRoutes.js:24:17
at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
code: 'ERR_HTTP_HEADERS_SENT'
}
node:internal/errors:491
ErrorCaptureStackTrace(err);
^
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:400:5)
at ServerResponse.setHeader (node:_http_outgoing:663:11)
at ServerResponse.header (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:794:10)
at ServerResponse.send (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:174:12)
at ServerResponse.json (D:\code\killingsworth-kid-backend\node_modules\express\lib\response.js:278:15)
at D:\code\killingsworth-kid-backend\routes\UserRoutes.js:27:29
at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
code: 'ERR_HTTP_HEADERS_SENT'
}
Node.js v18.13.0
[nodemon] app crashed - waiting for file changes before starting...

moved res.status(200).json(obj) into an else statement to prevent from reaching finished state


Basic installation/running question

Hi, thanks for all the awesome work on this library!
I'm relatively new to pytest in general and had a question on general setup. Where does pytest need to be installed/run (my local machine, the HASS docker container, the Appdaemon docker container)?
So far whenever, I call assert_that('my_entity_id'), and try to run a test, I receive an appdaemontestframework.assert_that.EitherOrAssertionError error on the pytest console.
Here's an example of a simple test I've tried:
from apps.front_porch_lights import FrontPorchLights
import pytest
from appdaemontestframework import automation_fixture

@automation_fixture(FrontPorchLights)
def front_porch():
    pass

def test_light_turns_on_at_sunset(given_that, front_porch, assert_that):
    given_that.passed_arg('entities').is_set_to('light.front_porch_sconces')
    given_that.state_of('light.front_porch_sconces').is_set_to("off")
    front_porch.lights_on(None, None)
    assert_that('light.front_porch_sconces').was.turned_on()

Thanks in advance for any help in getting up and running!

So I think I've answered the first part of my question on my own - I initially wasn't sure how pytest would actually test my Appdaemon script. I think I've started to wrap my head around how purest works now.
That said, I still haven't been able to get past the EitherOrAssertionError. It always seems to get hung up on turn_on() calls. Here's a portion the output from debugging a test:
...
...

At least ONE of the following exceptions should not have been raised
E
E           The problem is EITHER:
E           call_service('ANY_DOMAIN/turn_on', {'entity_id': 'light.bathroom_vanity_lights'}) call not found
E
E           OR
E           turn_on('light.bathroom_vanity_lights') call not found

Ok, I managed to figure the issue out on my own - it was an error in my assert_that()...turn_on() call. In my automation, I set the light to turn on to a specific brightness_pct and then in my assertion, I didn't specify that brightness.
Closing.

Hi,
I'm glad you could find the solution 😊
I also believe that more expressive error messages would be very welcome, but I'm the sole maintainer and I don't have too much time to dedicate to this project at the moment.
But either way, kudos for solving your issue 👍🙂

Sorry to resurrect this, but how do you run the tests? I'm using the VS Code supervisor plugin, is it going to be possible to run the tests from within VS Code? Or do I need to ssh into a container?


Issue when sorting by created -- Keeps loading forever

I have a list of documents in Sanity and when I change sorting to "created by", I get a the loading animation, but it never finishes.
When I reload the Studio in the browser it does seem to work sometimes, but when I change the sorting back to any other option, the same issue occurs.
I'd be happy to provide any additional details to help investigate this issue. Let me know.

We are having the same issue (v3.1.4)
Happens when sorting by created by, or custom ordering
This is how we get the menuitems
.menuItems([...S.documentTypeList(docType).getMenuItems()])

Can you recreate the behavior if you're using npx sanity dev, or is it isolated to when using next-sanity/studio?


Data clean up prior to making cohort data available

The following data needs cleaning up in the cohort field (in both MONGO and oracle databases), prior to making this data available to users.

The following should be changed to 'UKB':
UKBB
UK Biobank
UKbiobank

'multiple cohorts' should be 'multiple'

The following should all be changed to 'RS'
Rotterdam
Rotterdam Study I
Rotterdam Study II
Rotterdam Study III
RS_I
RS_II
RS_III
RS1
RS2
RS3
RSI
RSII
RSIII

4.'GenScot' should be 'GS:SFHS'

Any commas within the cohort field should be changed to pipes

Any forward slashes within the cohort field should be changed to pipes
The only exception would be for the study (PMID 33414549) which gives a URL containing slashes: ("see cohorts in the educational attainment (excluding 23andMe) and cognitive performance GWAS by Lee et al. 2018 https://doi.org/10.1038/s41588-018-0147-3")
It might be better to change the specific studies instead:
(PMID 33547301 - 3 studies) - 'UKBB/BBJ'
(GCST90179151) - 'FTC/NAG-FIN'

Trailing spaces should be removed

'special characters' eg. 'Ôªø' should be removed
Note: Letter case should be disregarded when it comes to consolidating

Smaller issues, fix if possible:
All New Diabetics In Scania (ANDIS)' should be 'ANDIS'
'Malmö Diet and Cancer (MDC)' should be 'MDC'
'Children's Hospital of Philadelphia (CHOP)' should be 'CHOP'
'Baependi Heart Study' should be 'Baependi'
The following should be changed to 'Estonia':
Estonia_Chip
Estonia_WGS
'EstBB' should be changed to 'EB'
'Generation R' should be 'Generation_R'
The following should be 'GERA':
GERA_EA_AFRchip
GERA_EUR_LATchip
The Resource for Genetic Epidemiology Research on Aging (GERA) Cohort

Please refer to @earlEBI for any questions about the data

Done in dev db and verified, needs re-running in prod

Two more secondary issues I noticed, if the other 'smaller issues' above are also going to be fixed:
'RS Study I' should be 'RS'
Special characters in 'The ‚ÄúEuropean NAFLD Registry‚Äù Metacohort' should be removed


Added CI tests (#8)

Updated README.md

Added github workflow

Added github workflow

Added tests

 CI/CD tests


Remove warning about boolean query params

The code converts them into integers anyway, so there is not point in
showing warnings.
Signed-off-by: Marcos Paulo de Souza<EMAIL_ADDRESS>

Thank you :+1:


String+SHA1.swift does not compile in Xcode 8.3b2, Swift 3.1

/swifter/Sources/String+SHA1.swift:92:74: error: expression was too complex to be solved in reasonable time; consider breaking up the expression into distinct sub-expressions
                let temp = (rotateLeft(a, 5) &+ f &+ e &+ k &+ words[i]) & 0xFFFFFFFF
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~

hello,
tried but I can't reproduce it on my side. Let me know, if changing the expresssion to the one below helps:
let temp = (((rotateLeft(a, 5) &+ f) &+ (e &+ k)) &+ words[i]) & 0xFFFFFFFF

Splitting the first part up worked for me:
var temp = (rotateLeft(a, 5) &+ f &+ e &+ k &+ words[i])
temp = temp & 0xFFFFFFFF

Odd that you don't see it on your side.

Alan, this is most likely because they are not using the "-solver-memory-threshold" command line parameter.

@rudro what's the value you use?

Oh right I forgot about that. We do -solver-memory-threshold 300000.

ok, thanks for sharing 👍

@rudro what's the value you use?

I use whatever Alan tells me to use ;) (we work on the same codebase)


Country-specific speed limits

I am using GraphHopper to import OSM data and transform it to graph-based format for a certain program that we are currently using.
As we only work with one country's data at any time, I would like to use an class AreaSpecificCarFlagEncoder extends CarFlagEncoder that would receive the speed limits for a given area in beginning of work of the program.
(More sophisticated) alternative: input for AreaSpecificCarFlagEncoder would be N sets of (WKT geometries and sets of speed limits). AreaSpecificCarFlagEncoder would make sure that the geometries are non-intersecting; and if given way is fully in some area, then the given speed limit is used. If way is not inside any of the given areas, default values are used.
I do not know what would be the best course of action if way is partially in one or more areas. Perhaps it would be most suitable to use default speeds then.
Please comment.

A country specific speed limit implementation should also allow handling of country specific access restrictions. It should provide the possibility to configure tables similar to http://wiki.openstreetmap.org/wiki/OSM_tags_for_routing/Access-Restrictions. The evaluation of these tables shoud be possible for all kinds of encoders, not just for cars.

I did some investiagtions how this feature could be implemented and found the following library, which does what is needed for reverse geocoding into a country code:
https://github.com/bencampion/reverse-country-code
This library does not use OSM data, but rather data from thematicmapping.org . This data is not as precise as the OSM, but I think that it would be sufficient for our purpose. Also the license should be OK.
A totally different appraoch would be to extract the polygons on our own during parsing of the OSM data on the fly. In comparison to using the libraray mentions or a similar library, I see a lot of draw-backs:

It does not work for OSM extracs
It increases the reader complexity
It slows down graph Generation
It will be problematic because from the sea-polygon discussions I know that the country releations will be frequently broken in OSM.

First I thought that the reverse geo-coding task has similarities to the inner/outer city speed limit of issue #230, but after having looked on the code provided by jansoe I came to the conclusion that this is something quite different.
Could you take a look on the library with your GIS expertiese? What approach do you prefer? I made some performance measurements with the lib and came to the conclusion that it is able to look up a lat/lon in an order of magnitute of about 0.1 ms on a 3 years old PC. I think that such performance should be sufficient.

Thanks for this idea!

It does not work for OSM extracs

valid point. We could still solve this with preprozessed and verified (less often updated) OSM geometry hosted somewhere at graphhopper.com and we fetch this data dynamically like we do with the elevation data.
The biggest problem I see with the library is the strange commons license, which is very uncommon for software and we need to avoid that someone is bound to mention this library everywhere where GraphHopper is used. But we could ask the author if he would willing to relicense to some proper LGPL or Apache License (preferred)

I made some performance measurements with the lib and came to the conclusion that it is
able to look up a lat/lon in an order of magnitute of about 0.1 ms on a 3 years old PC.

Thanks, can you re-do your tests with 100 mio random points spread over EU and US? This way we know better what the impact for the import could be.
The idea I have in mind will be faster as it will be based on a simple lookup via a kd-tree or quadtree. It is nearly identical to the procedure used in the already implemented city limits branch.
I also tend to prefer similar techniques used for city and country specific stuff.

First I thought that the reverse geo-coding task has similarities to the inner/outer
city speed limit of issue #230, but after having looked on the code provided by jansoe
I came to the conclusion that this is something quite different.

Ah, yes. No, it should be very similar. This is already very fast as no "in-polygon" check has to be done like in the proposed library, as there is just a boolean array where the index is calculated from the coordinates and this array is pre-calculated only once from any polygon list.

The 100mio test of reverse-country-code with following 50 mio runs in Europe and 50 mio runs in America
// Europa
lookup.getCountryCode(49+10*(rand.nextDouble()-0.5), 21 + 20*(rand.nextDouble()-0.5));
// North America
lookup.getCountryCode(46+10*(rand.nextDouble()-0.5), -107 + 30*(rand.nextDouble()-0.5));

took 96 minutes 23 seconds on an Intel i7<EMAIL_ADDRESS>GHz. Do you think this is fast enough?

Hmmh, not that good. The full GH import of full Europe is faster so with that this would slow down everything too much, but this is also not suprising. So I would instead create a QuadTree (or an array at the beginning like done for the city limits) where we can check with a simple "array[index]" request which should be a lot faster

BTW: Thanks for this test!

This issue should be fixed with the SpatialRuleIndex #926. Will close here for now.


multiprocessing runner spawns way too many processes

This was actually cause by a weird interaction with mkl, maybe warn users who are using the multiprocessing runner without setting mkl threads to 1


What' the relation between CanvasRenderingContext2D.beginPath() and CanvasRenderingContext2D.moveTo()

What's the meaning of "Note: To create a new sub-path, i.e., one matching the current canvas state, you can use CanvasRenderingContext2D.moveTo()."?
Answer:

There must be a moveTo() after beginPath(). Because after beginPath(), the canvas state is last point of last sub-path is empty, which means the start point of next sub-path must be specified by moveTo().

There must be a moveTo() after beginPath(). Because after beginPath(), the canvas state is last point of last sub-path is empty, which means the start point of next sub-path must be specified by moveTo().


Semantic syntax checker confuses public and private fields

This does not produce a syntax error, but it should because there is no #x property.
class C {
  x = 1;
  method() {
    obj.#x;
  }
}

It seems syntax checker is confusing public property x with private property #x. If you rename the public property, it works as expected.
Oxc playground

question: surely this is ok because you are accessing #x on obj not on this?
TS doesn't report a syntax error in this case:
var obj: any;

class C {
  x = 1;
  method() {
    obj.#x;
  }
}

instead you get "Property #x does not exist on type 'any'"

Private properties are lexically scoped. It's always a syntax error to use #x, unless that code is inside a class which defines a property #x. Try running the example above in NodeJS.
The fact that obj may not be an instance of a class which defines a private property #x is a separate problem - but that's a runtime error, not a syntax error. TypeScript makes it a type error too, but that's a TS-specific thing, and again a separate issue.
I'll admit, none of this is obvious! Everything around private properties is fiendishly complicated. Hence why I've been working on the class properties transform for weeks, and it's still not complete...


Document the purpose of the chairman tests

This is to address this ticket: https://github.com/input-output-hk/cardano-node/issues/671

bors merge


STOR-4305 Swagger: update goswagger/swagger image version

Changed swagger to v0.17.0.
swagger's 'make all' ran successfully.
Also ran (successfully) make for external-endpoint-manager with new temporary container image created by swagger's make.
Also reorganized files by removing make-generated files and adding their directories to .gitigore (these directories match swagger's 'make clean').
go-template will be updated with new swagger tag once the image is in the Stratoscale's "docker hub" server.

@drorg-stratoscale  why did you delete the example?


C_CPP : Relative Complement of two Sorted Arrays in C++

Description
This documentation will explain Relative Complement of two Sorted Arrays with the help of a problem in C++
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Multiple servers applying immediate upgrades

How do we stop multiple servers from applying the same upgrades #46.

Upgrades have to be applied chronologically.  If not then the patches assumption about the state of the world is wrong.
Perhaps only one server at time can apply patches.  This implies some locking mechanism at the database level.


Use functools.wraps

functools.wraps does all the right things for wrapper-decorators. it's only available in Py 2.5 and newer, so please reject this if you support Py 2.4 or less.

Closing as deprecated (duplicated of #14 which was merged).


feat: add appProtocol and name to var.port_mappings

what

Adding missing fields to var.port_mappings (appProtocol, name, containerPortRange)

why

So that devs have access to all of the available fields: https://docs.aws.amazon.com/AmazonECS/latest/APIReference/API_PortMapping.html

references

/terratest

/terratest

Just merged earlier PR with the same feature https://github.com/cloudposse/terraform-aws-ecs-container-definition/pull/168

the other PR had the additional requirement of setting the name field, which is used in the ECS Service Connect api

Hey @max-lobur @dudymas! #168 added name and appProtocol to var.port_mappings, which is the main change I needed! In this PR I also added containerPortRange since I noticed it was missing.
Would you like me to refactor this PR to just contain the port range change? We would also want to make containerPort optional since it's optional in the AWS config.


Data cleanup/archiving

Hi,
For now we are using local storage for structurizr on premise and we are starting to get out of disk space errors.
We are uploading very often new versions of structurizr workspaces in our automated pipelines. Often even without changes. So a few questions on what are operationally our options:

It appears the structurizr workspace versions are all kept, even though there are no changes. Is there any way to optimize this ?
What are the ways we can archive/delete old versions of our workspaces ? Can we disable versioning or at least limit to a maximum amount of workspaces ? Do we create our own cleanup process to remove?

There's a scheduled job built in to the on-premises installation that removes all workspace versions older than 365 days. But this can be changed, and perhaps doing something like keeping the most recent X versions (where X is configurable) would be better. Would that work for you?

That would work. making the this '365 days' setting configurable would solve our issues

This will be available in build 2905+ (later today) ... a property named structurizr.maxWorkspaceVersions can be used to customise the number of workspace versions stored (default=30).


[Bug]: Unusual delay at Restore Your Wallet page

Describe the bug
A recent change has introduced a long wait at the Restore Your Wallet page once 'Restore' button is pressed. This wait may be needed but it should be moved to the rocket screen stage.
Reproduce

Go to Restore Your Wallet
Enter valid seed
Press 'Restore' button
Observe ~10 second delay before rocket screen appears.

Screenshots / Recording
https://github.com/user-attachments/assets/cfbc9f45-a195-4caa-9074-05d1ba9bb73c
Operating system
Android 14
Bitkit version
c7432721e886dc49b57d9b138f25031a257ab415
Log output
No response

Verified in 1.0.5 (137)


Update index.html

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Does your submission pass tests?

Note: please do NOT include build files (those generate by the build-xxx commands) with your PR,

Thank you for your help in advance, much appreciated !

@heristhesiya thanks for the change.
can you provide more information on the change? Why was the change made?

I added one more example in an index file.

Changing something just to change something doesn’t mean much. This change doesn’t add value to this index, which by the way isn’t used today.


feat(Order/ConditionallyCompleteLattice): lt_ciSup_iff and variants

Looks like CI passes on this – are you happy to go ahead? If so, please remove the "awaiting author" tag and I'll flag it for maintainer-merging.

maintainer merge


In Chrome:
bundle.js Failed to load resource: the server responded with a status of 404 (Not Found)
app.js:72 Uncaught ReferenceError: twinql is not defined
at onTheClient (app.js:72)
at runQuery (app.js:103)
at HTMLButtonElement.document.querySelector.addEventListener (app.js:95)

This happens with the demo site (stopping at step 3). This may not be a bug since WebPack creates bundle.js. Solved. Do:  npm run build:umd in step 4 with WebPack before trying npm start in step 3.

Hey @bshambaugh. Thanks for testing things out! Sounds like you figured this one out yourself. If you're running locally, you have to build the bundle before starting the server.


PCA fit method

I need to fit PCA model with many entries and transform for single entry next.
I think separate method to Fit and Transform is a good idea.

LGTM, could we retain the FitTransform method (even if it just calls one after the other) so that it doesn't cause stuff to break? Otherwise looks good

@Sentimentron What you mean about "retain the FitTransform method"? What I should do?

@Sentimentron method FitTransform already have in PR


0.29.2 breaks tests on DT

From https://dev.azure.com/definitelytyped/DefinitelyTyped/_build/results?buildId=155148&view=logs&j=9d906afe-3eb5-5d79-316d-86f3f3fa360c&t=09716206-abc7-5ba2-2ea2-6c6fe97a3aad&l=12352:
Error in ramda
Error: /home/vsts/work/1/s/types/ramda/test/find-tests.ts:5:30
ERROR: 5:30   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; }[]' is not assignable to parameter of type 'readonly Record<2, any>[]'.
  Property '2' is missing in type '{ a: number; }' but required in type 'Record<2, any>'.
ERROR: 6:30   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; }[]' is not assignable to parameter of type 'readonly Record<4, any>[]'.
  Property '4' is missing in type '{ a: number; }' but required in type 'Record<4, any>'.

/home/vsts/work/1/s/types/ramda/test/findIndex-tests.ts:5:35
ERROR: 5:35   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; }[]' is not assignable to parameter of type 'readonly Record<2, any>[]'.
  Property '2' is missing in type '{ a: number; }' but required in type 'Record<2, any>'.
ERROR: 6:35   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; }[]' is not assignable to parameter of type 'readonly Record<4, any>[]'.
  Property '4' is missing in type '{ a: number; }' but required in type 'Record<4, any>'.

/home/vsts/work/1/s/types/ramda/test/findLast-tests.ts:8:34
ERROR: 8:34   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; b: number; }[]' is not assignable to parameter of type 'readonly Record<1, any>[]'.
  Property '1' is missing in type '{ a: number; b: number; }' but required in type 'Record<1, any>'.
ERROR: 9:34   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; b: number; }[]' is not assignable to parameter of type 'readonly Record<4, any>[]'.
  Property '4' is missing in type '{ a: number; b: number; }' but required in type 'Record<4, any>'.

/home/vsts/work/1/s/types/ramda/test/findLastIndex-tests.ts:8:39
ERROR: 8:39   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; b: number; }[]' is not assignable to parameter of type 'readonly Record<1, any>[]'.
  Property '1' is missing in type '{ a: number; b: number; }' but required in type 'Record<1, any>'.
ERROR: 9:39   expect<EMAIL_ADDRESS>compile error:
Argument of type '{ a: number; b: number; }[]' is not assignable to parameter of type 'readonly Record<4, any>[]'.
  Property '4' is missing in type '{ a: number; b: number; }' but required in type 'Record<4, any>'.

/home/vsts/work/1/s/types/ramda/test/propEq-tests.ts:10:11
ERROR: 10:11  expect<EMAIL_ADDRESS>compile error:
Type '(obj: Record<"bar", any>) => boolean' is not assignable to type '(obj: Record<"foo", "bar">) => boolean'.
  Types of parameters 'obj' and 'obj' are incompatible.
    Property 'bar' is missing in type 'Record<"foo", "bar">' but required in type 'Record<"bar", any>'.
ERROR: 12:11  expect<EMAIL_ADDRESS>compile error:
Type '(obj: Record<5, any>) => boolean' is not assignable to type '(obj: Record<"baz", number>) => boolean'.
  Types of parameters 'obj' and 'obj' are incompatible.
    Property '5' is missing in type 'Record<"baz", number>' but required in type 'Record<5, any>'.
ERROR: 14:11  expect<EMAIL_ADDRESS>compile error:
Type '(obj: Record<"qwerty", any>) => boolean' is not assignable to type '(obj: Record<number, string>) => boolean'.
  Types of parameters 'obj' and 'obj' are incompatible.
    Property 'qwerty' is missing in type 'Record<number, string>' but required in type 'Record<"qwerty", any>'.
ERROR: 29:22  expect<EMAIL_ADDRESS>compile error:
Argument of type 'Obj' is not assignable to parameter of type 'Record<1, any>'.
  Property '1' is missing in type 'Obj' but required in type 'Record<1, any>'.
ERROR: 30:22  expect<EMAIL_ADDRESS>compile error:
Argument of type 'Obj' is not assignable to parameter of type 'Record<4, any>'.
  Property '4' is missing in type 'Obj' but required in type 'Record<4, any>'.
ERROR: 43:28  expect<EMAIL_ADDRESS>compile error:
Argument of type 'A' is not assignable to parameter of type 'Record<"", any>'.
  Property '""' is missing in type 'A' but required in type 'Record<"", any>'.

I'm surprised that not all the tests from DT have been copied here.

@sandersn Those tests broke specifically because propEq had it's argument order swapped between 0.28.0 and 0.29.0. The most recent release of types-ramda included typing updates to reflect that change. They have since been updated in this MR: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/65286

I'm surprised that not all the tests from DT have been copied here.

That can't. At least not copied and pasted. How the tests work in DT is kind of special to DT. That teams has suggested that we use thetsd package to test here
We specifically kept the existing testing in @types/ramda to catch regressions. We do plan to, overtime, manually migrate over all the tests to be re-written with tsd. See this issue: https://github.com/ramda/types/issues/2


Remove  element from w3c validator

Depends on – https://github.com/validator/validator/issues/1569


Restore "this" name to context module

what

Restore "this" name to context module

why

#110 accidentally changed the name to "label". All modules that use it depend on it being called "this"

/test all


Disable part of 8c4fcc9 when running on an iOS device

This is my attempt to fix https://github.com/openseadragon/openseadragon/issues/952
Solution

Introduce configuration variable iOSDevice that defaults to true when running on an iOS device, false otherwise
If iOSDevice === true disable the iOS-specific broken part of https://github.com/openseadragon/openseadragon/commit/8c4fcc9 (special handling of >100% zoom)
Update comments documentation to reflect the new configuration variable
The configuration variable iOSDevice is a variable such that

It can be overwritten if a user requires that behaviour
It can be read on viewer

Test Suite
Running "qunit:normal" (qunit) task
Testing http://localhost:8000/test/test.html ......................................................................................................................................................................................................OK
>> 2132 assertions passed (51213ms)

Done, without errors.

Wow, beautiful pull request! All looks great.
One minor thing: maybe it's worth including in the docs for smoothTileEdgesMinZoom that it gets disabled on iOS and why; possibly even worth including a link to #952.

 Great project!

Awesome, thanks! Good catch on the spacing as well.

I think I may do a release soon so that we can get this fix out there!


Test/pr logic test

DON'T MERGE - Just testing

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from Kudzai-moj

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@richgreen-moj Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform


Better site analytics

GitHub has its own analytics, but I don't think this sufficiently covers GitHub Pages / Jekyll.
Google Analytics comes first to mind of course, but I'm open to (free) suggestions.
The process described here seems straight forward enough.

First, I had to register my website through Google Analytics. Google gave me this javascript tracking code to embed on every website: (...)

This is ours:
<script>
  (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
  (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
  })(window,document,'script','//www.google-analytics.com/analytics.js','ga');

  ga('create', 'UA-66145978-1', 'auto');
  ga('send', 'pageview');

</script>

I put this code in the file _includes/google_analytics.html Finally, I included this tracking code on all of the pages of my website by modifying the file _layouts/default.html to include the line:

{% include google_analytics.html %}

Why not use cloudflare for analytics as well? We can get some free SSL at
the same time...
On Mon, Aug 10, 2015 at 10:05 AM Erlend Sogge Heggen <
<EMAIL_ADDRESS>wrote:

GitHub has its own analytics
https://github.com/nahtnam/learndiscourse/graphs/traffic, but I don't
think this sufficiently covers GitHub Pages / Jekyll.
Google Analytics comes first to mind of course, but I'm open to (free)
suggestions.
The process described here
http://joshualande.com/jekyll-github-pages-poole/ seems straight
forward enough.
First, I had to register my website through Google Analytics. Google gave
me this javascript tracking code to embed on every website: (...)
This is ours:

I put this code in the file _includes/google_analytics.html Finally, I
included this tracking code on all of the pages of my website by modifying
the file _layouts/default.html to include the line:
{% include google_analytics.html %}
—
Reply to this email directly or view it on GitHub
https://github.com/nahtnam/learndiscourse/issues/25.

Oh, I didn't know cloudflare had analytics. Sure, we can give that a try. Wanna implement it? :grinning:

Sure. Would require some nameserver changes. I dont think cloudflare lets
multiple people have permissions to a single domain so we will probably
have to create a shared account.
Should we get<EMAIL_ADDRESS>and then share the password VIA
https://keybase.io/nahtnam (encrypted) or something?
On Mon, Aug 10, 2015 at 10:37 AM Erlend Sogge Heggen <
<EMAIL_ADDRESS>wrote:

Oh, I didn't know cloudflare had analytics. Sure, we can give that a try.
Wanna implement it? [image: :grinning:]
—
Reply to this email directly or view it on GitHub
https://github.com/nahtnam/learndiscourse/issues/25#issuecomment-129543801
.

Hmmm, right, the lack of shared accounts is a bit annoying.
Since nameserver changes are required I'd like to give @fantasticfears and @mcwumbly a chance to chime in on this one before we move forward with that kind of change.
I wouldn't mind testing the simple Google Analytics setup in the meantime though. I've sent you an invite to our GA account.

Didnt get the invite. :(
On Mon, Aug 10, 2015 at 11:09 AM Erlend Sogge Heggen <
<EMAIL_ADDRESS>wrote:

Hmmm, right, the lack of shared accounts is a bit annoying.
Since nameserver changes are required I'd like to give @fantasticfears
https://github.com/fantasticfears and @mcwumbly
https://github.com/mcwumbly a chance to chime in on this one before we
move forward with that kind of change.
I wouldn't mind testing the simple Google Analytics setup in the meantime
though. I've sent you an invite to our GA account.
—
Reply to this email directly or view it on GitHub
https://github.com/nahtnam/learndiscourse/issues/25#issuecomment-129553277
.

Hmm. I tried sending you another mail now. In any case, the tracking code pasted above should be all you need to get it working. You just won't be able to confirm that it's working without the login.

I didn't receive the invite either<EMAIL_ADDRESS>And the GA code was applied.

There you go, finally got it :P
On Fri, Aug 14, 2015 at 3:30 AM Erick Guan<EMAIL_ADDRESS>wrote:

I didn't receive the invite either<EMAIL_ADDRESS>And the GA code was applied.
—
Reply to this email directly or view it on GitHub
https://github.com/nahtnam/learndiscourse/issues/25#issuecomment-131061773
.


Rewrite SocksProxy based on states

This PR is also used to solve the issue found at https://ci.trafficserver.apache.org/clang-analyzer/master/2019-04-25-012255-12399-1/report-4282fc.html#EndPath
@zwoop

+1
nice state diag ascii arts, lovely old school style


Spoken message appears as a short gap of silence if streaming/playing

Issue with sonos2mqtt see https://github.com/svrooij/sonos2mqtt/issues/73

create a fork of node-sonos-ts.
run npm install in the repository
Change the code you want
Press F5 (in VS Code), while the play-notification sample is open.

That should build (actually transpile) the typescript to javascript, and run the current open sample. The code in the repository is only TypeScript and the lib-folder is automatically ignored.

That won't help because the AsyncHelper isn't used when setting the Volume. Maybe it should be used. However that's above my experience level.

I mean that it might be better to put the delay function in the AsyncHelper class. That code might be needed in other places in the library. Something like
static async Delay(milliseconds: number): Promise {
  return await new Promise(resolve => setTimeout(resolve, 500));
}

Then you can use it by calling:
await AsyncHelper.Delay(150);

@besynnerlig if you have fixed this issue be sure to put the following in the commit message, so it will automatically release after merge.
fix: TTS not playing

Fixed #38

The fix:  will signal a new patch version. The Fixed #38 will close this issue after merging.

:tada: This issue has been resolved in version 1.2.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


After Response is ending (res.end()), still receive the data from client (res.onData)

Expected Behavior
When res.end is is executed, res.onData event will not be emit.
Actual Behavior
I sent the sample image (3.9mb), after res.end is is executed, res.onData still emit and receive the data from client.
server
% node index.js
limit 1048576
current size: 65328 bytes isAborted: false isLast: false
current size: 589616 bytes isAborted: false isLast: false
current size: 1113904 bytes isAborted: false isLast: false
current size: 1511620 bytes isAborted: true isLast: false
current size: 2035908 bytes isAborted: true isLast: false
current size: 2560196 bytes isAborted: true isLast: false
current size: 3084484 bytes isAborted: true isLast: false
current size: 3474944 bytes isAborted: true isLast: false
current size: 3999232 bytes isAborted: true isLast: false
current size: 4106429 bytes isAborted: true isLast: true

client
% http post localhost:3000 < sample.jpg
HTTP/1.1 413 Payload Too Large
Content-Length: 0
uWebSockets: v0.17

How to Reproduce
httpie command
http post localhost:3000 < sample.png

my code
const uWS = require('uWebSockets.js')

const app = uWS.App()

app.any('/*', res => {
  const limit = 1 * 1024 * 1024 // 1 mega bytes
  console.log('limit', limit)

  let data = Buffer.allocUnsafe(0)
  let isAborted = false

  res.onData((chunk, isLast) => {
    data = Buffer.concat([data, Buffer.from(chunk)])

    console.log('current size:', data.length, 'bytes isAborted:', isAborted, 'isLast:', isLast)

    if (isAborted) {
      return
    }

    if (data.length > limit) {
      res.writeStatus('413 Payload Too Large')
      res.end()
      isAborted = true
      return
    }

    if (isLast) {
      res.writeStatus('202 Accepted')
      res.end()
      isAborted = true
    }
  })

  res.onAborted(() => {
    isAborted = true
    console.log('aborted')
  })
})
app.listen('<IP_ADDRESS>', 3000, () => {})

smaple.jpg

Environment

Node.js v14.2.0
uWebSockets.js v17.4.0
httpie v2.1.0
macOS (Catalina v10.15.4)

MacBook Pro (Retina, 13-inch, Early 2015)
2.7 GHz Core x2
8 GB DDR3 Ram x1

You should only respond when isLast is set. Http does not allow the server to refuse a big payload, you still have to receive it (but you can ignore it). That's as far as I know at least.

According rfc7231
The server MAY close the connection to prevent the client from continuing the request.
Such as Nginx, go gin
Close the connection, when a request is over the limit

What if you use res.close() instead? That one should "Immediately force closes the connection." as per doc.
If that doesn't work either (for whatever reason, since it should), just move data = Buffer.concat([data, Buffer.from(chunk)]) down a few lines, after the isAborted check. That should do it as a workaround...

@mikeandtherest If use res.close() instead of res.end(), it will be close immediately. But the client side will not get any response like status code 413 Payload Too Large.

You can res.close in process.nextTick but that's a hacky solution. This feature is being tracked in the main repo.

I have thought about it and the most reliable and simple solution I can think of is to add res.endAndClose() as an alternative to res.end() where once drained, the connection is shutdown. You still have to ignore any further data you might get above your limit.
You can't really make good heuristics such as "if res.end() is called before seeing all data then make it res.endAndClose()" because some users simply ignore the data and in that case it would be really bad to close down the connection rather than keep it keep-alive. And in cases where you would like to stream back whatever data you got, it becomes messy.
I think the simplest solution is res.endAndClose(). The name is debatable - endWithClose?

I have thought about it and the most reliable and simple solution I can think of is to add res.endAndClose() as an alternative to res.end() where once drained, the connection is shutdown. You still have to ignore any further data you might get above your limit.
You can't really make good heuristics such as "if res.end() is called before seeing all data then make it res.endAndClose()" because some users simply ignore the data and in that case it would be really bad to close down the connection rather than keep it keep-alive. And in cases where you would like to stream back whatever data you got, it becomes messy.
I think the simplest solution is res.endAndClose(). The name is debatable - endWithClose?

Another solution is to have a second argument like res.end("data", shutdown true/false);

Another solution is to have a second argument like res.end("data", shutdown true/false);

You can't really make good heuristics such as "if res.end() is called before seeing all data then make it res.endAndClose()"
I totally agree that no assumptions should be made, since they could indeed start as good intentions but can end up as restrictions and make the whole thing less flexible.
I think the simplest solution is res.endAndClose(). The name is debatable - endWithClose?
Another solution is to have a second argument like res.end("data", shutdown true/false);
Tbh I would look at it from the opposite angle, and by that I mean that what I think it's missing is a "close with a response/payload". So maybe res.closeWithResponse() or res.closeWithData() could be better?

You can't really make good heuristics such as "if res.end() is called before seeing all data then make it res.endAndClose()"
I totally agree that no assumptions should be made, since they could indeed start as good intentions but can end up as restrictions and make the whole thing less flexible.
I think the simplest solution is res.endAndClose(). The name is debatable - endWithClose?
Another solution is to have a second argument like res.end("data", shutdown true/false);
Tbh I would look at it from the opposite angle, and by that I mean that what I think it's missing is a "close with a response/payload". So maybe res.closeWithResponse() or res.closeWithData() could be better?

Those are good names but they can be difficult to connect with "end" which is really the same thing. I ended up making it res.end("data", true) for closeConnection since I did not really want to add more functions than needed. This is committed and working in main repo now, so next js release should have it supported.

Those are good names but they can be difficult to connect with "end" which is really the same thing. I ended up making it res.end("data", true) for closeConnection since I did not really want to add more functions than needed. This is committed and working in main repo now, so next js release should have it supported.

Ok, as long as the default value for the new arg is false (which I see it's already the case in your commit), I think we're all good since it fixes the problem while also keeping it safe for existing code. Thx!

Ok, as long as the default value for the new arg is false (which I see it's already the case in your commit), I think we're all good since it fixes the problem while also keeping it safe for existing code. Thx!


[Linear] Allow login with multiple emails and choose email then project for task creation

Extension
https://www.raycast.com/raycast/linear
Description

When managing multiple accounts with raycast from separate emails, this extension only allows one email sign in at once
This means that if you wanted to connect raycast with another linear account, you would need to sign out and in to the other account with a different email
This extension could be augmented to allow logging into multiple emails, then exposing a field for selecting which linear workspace to complete an action for, for example, when creating a new task, give the option to select the workspace (which it cannot do currently), then project the task is for, and associated details of the task (like it does now)

Who will benefit from this feature?

Anyone using linear with multiple emails, who wants to manage multiple workspaces or projects in one place

Anything else?
No response

Hey 👋
Great idea. Supporting multiple accounts has popped up with other extensions as well. We have plans to add first-class support for this but there might be a way to support this with the existing API. The tricky bit is to find a UX that works across different command types.

Related to https://github.com/raycast/extensions/issues/269.


sleep not work

I added sleep according to the manual so it doesn't throw an exception and it's still throwing it. It's falling into this if.
  protected function resolveLimits(): array     {         return [             Limit::allow(60)->everyMinute()->sleep()         ];     }
` protected function handleExceededLimit(Limit $limit, PendingRequest $pendingRequest): void
{
if (! $limit->getShouldSleep()) {
$this->throwLimitException($limit);
}
    $existingDelay = $pendingRequest->delay()->get() ?? 0;
    $remainingMilliseconds = $limit->getRemainingSeconds() * 1000;

    $pendingRequest->delay()->set($existingDelay + $remainingMilliseconds);
}

`

Hey @leonardyrj thank you for opening this issue - I'm very sorry for the slow reply on this one, been very busy lately!
Could this be throwing an exception because of the automatic 429 detection? Try disabling that and see if it works. You have to configure the limit to be slightly lower than the real API limit.
https://docs.saloon.dev/plugins/handling-rate-limits#429-too-many-attempts-detection

Confirmed this is happening to me too. The stack trace shows that it is ignoring the sleep here. However if I dump and die the Limit at an earlier stage, it shows that sleep is set to true.
I don't know if I have time to work it out but I will need to figure this out before the xmas rush as its causing downtime for us when it gets busy.

Ahhh so it is the fallback "too many attempts" limiter kicking in. I have no idea how its leaking an additional 200 requests and going over the defined limit, but for now this is my work around for making the sleep work on the custom limiter. Hopefully there is no knock-on issues from the change but for now this gets me out of the hole while I work out where the extra requests are going.

    /**
     * Get the limits for the rate limiter store
     *
     * @return array<Limit>
     * @throws LimitException
     */
    public function getLimits(): array
    {
        $tooManyAttemptsHandler = $this->detectTooManyAttempts === true ? $this->handleTooManyAttempts(...) : null;

        $limits = LimitHelper::configureLimits($this->resolveLimits(), $this->getLimiterPrefix(), $tooManyAttemptsHandler);

        foreach ($limits as $limit) {
            if (!$limit->getShouldSleep()) {
                $limit->sleep();
            }
        }

        return $limits;
    }
    ```

It happens me too.

@ClaraLeigh bit late, but could the leak be because the requests are concurrent? 200 requests hitting an endpoint at exactly the same time would make them all read the same rate limit right?
But then again, we hit a 429 too and we're not doing anything concurrent on our end, just a command with a loop doing requests.


"tightenco/ziggy": "^0.9.4" cannot upgrade to 1.0.2

Hi,
I am using ziggy 0.9.4 . composer cannot upgrade to 1.0.2. if I manually change composer.json to ^1.0, will it break anything? also does the 1.0.2 require php 7.4 (the arrow function)? as i am using php 7.3. Thanks.

Yes, v1 contains breaking changes, which you can find in the Upgrade Guide. Ziggy's dependencies are listed in composer.json, PHP 7.3 will work fine.


chore: Deprecate facility in the favor of metro

Changes Title (replace this with a logical title for your changes)
Deprecate facility in the favor of metro API
Description

Support Metros location in libcloud interfaces and deprecate facility api.
Also, remove implementation of storage APIs as they were removed from Packet years ago. there is no direct replacement. Equinix Metal offers Pure and NetApp storage devices today but there is not equivalent functionality as far as the Equinix Metal API is concerned with directly attaching storage to servers and equinix metal doesnot supports only 2 storage providers and

For more information on contributing, please see Contributing
section of our documentation.
Status
Ready For Review
Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[x] ICLA (required for bigger changes)

@aayushrangwala perhaps the removal of strorage should be done in a separate PR from the facility deprecation and introduction of metros.

nice i"ll puul this PR into my work area and make sure PR #1970 & PR #1971 are all thats needed to tqeak EqunixMetal back into prime shape.

@aayushrangwala perhaps the removal of strorage should be done in a separate PR from the facility deprecation and introduction of metros.

Sure, will raise another PR

Reviewers, please note that Equinix Metal removed the facilities/facility request field in June for new users, and the field will be entirely unavailable for all users after December. metro is the replacement.  Facilities reside within a metro. Legacy facility values are not compatible with the metro field.

@Kami Can you please review and approve this PR as well

@aayushrangwala Looks like tests and some other CI checks are failing. Can you please look into it when you get a chance?
Thanks.

Sorry for the delay - the PR has finally been merged into trunk.
Thanks again for your contribution.


feat: pimps up the DMG build

see  https://github.com/tillt/KompleteSynthesia/releases/tag/v<IP_ADDRESS>


Problem Installing atom-beautify

I get this error message anytime I try installing atom-beautify
Installing<EMAIL_ADDRESS>failed.Hide output…
npm WARN deprecated<EMAIL_ADDRESS>JSCS has merged with ESLint! See - https://medium.com/@markelog/jscs-end-of-the-line-bc9bf0b3fdb2
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm ERR! Unexpected end of JSON input while parsing near '...remark-man":"^2.0.0",'
npm ERR!      C:\Users\ATagoe.atom.apm_logs\2022-03-11T07_11_21_767Z-debug.log

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


API update - mount() for mounting, patch() for patching

As mentioned in #89, the current patch() API essentially does double duty: mounting to the DOM at a single node, and updating the tree as the application updates.
This PR aims to introduce a separate mount() function and updates the patch() function's API to support patching at any level of the tree.
/** @jsx h */
import { h, mount, patch } from 'picodom'

const App = props => <h1>{props.title}</h1

let root = <App title='Hello world!' />

mount(document.body, root)

// after state updates
patch(root, (root = <App title='New title' />))

As you can see, it adds a few lines, but perhaps there is a more compact way 🤔
As far as introducing a new method to the API, I think it's a good idea. "Mounting" and "patching" are different concerns, and a single API for both would be confusing IMO.
TODO
If you like this direction, I'm happy to make any updates necessary to get it integrated 🎉
The tests are failing currently, I could probably use some direction there: parentNode is undefined. Throughout the suite patch would need to be updated to mount and the params slightly rearranged.
Thanks!

@estrattonbailey Thanks, but I went with this instead:
import { h, patch } from "picodom"

document.body.appendChild(
  patch( h("h1", {}, "Hello") )
)

As you can see, the patch function now returns an element, when none is given, allowing you to append, if that's the behavior you prefer. Otherwise, if you pass an element, patchdom will attempt to patch that element, instead of using it as a container like in previous versions.

@jorgebucaran beautiful 😍
I should have renamed this PR, lol. My main goal here was was enable patching a VDOM node at a DOM node target. My tweak to patch made a new mount interface useful. It looks like your new version does exactly this, using similar assumptions to mine.
Can't wait to dive into this, 99% positive this is exactly what I needed. Thanks for your work!


..

Closing, see: https://github.com/microsoft/vscode-azurecontainerapps/pull/688#issuecomment-2158897892


Add CVE categorisation labels for the node-problem-detector image

How to categorize this PR?

/kind enhancement
What this PR does / why we need it:
This PR adds CVE categorisation labels for easier triaging of vulnerabilities. It also adds responsibles so that it is clear who should triage vulnerabilities for this image.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

NONE

/cc @rishabh-11

/cc @ashwani2k


working with visval sterminal

Issue Type: Performance Issue
when i am working with angular framework i am using shortcut to terminate (ctrl+c) then its showing terminate yes or now option but irrespective of my choice it terminating my terminal
VS Code version: Code 1.34.0 (a622c65b2c713c890fcf4fbf07cf34049d5fe758, 2019-05-15T21:59:37.030Z)
OS version: Windows_NT x64 10.0.14393

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693


Use accelerometer to provide more accurate pitch indications

Can also improve indications for uncoordinated maneuvers.

Closed with commit 4bed8783540c6567b4495bb7387040e4e3937623 in goflying.  Further tweaks are possible, but this is the substance of the solution.  Next step is a full Kalman filter.


Create a template for data-backed services

Description:
Create a template that demonstrates data-backed services and add it to the templates directory. It should be designed as a module template so that it can be pushed to Ballerina Central.

Fixed https://github.com/wso2/ballerina-integrator/pull/412


Abstract the whole JSON serialization/deserialization so that one can decide between using Jackson or GSON or others.

The problem of not being able to switch from Jackson to other without modifying the sources, other than performance and customization, is that other libraries also come with hardcoded libraries (like GSON) so that in the end we have to keep all of them.

duplicated with https://github.com/couchbase/couchbase-lite-java-core/issues/676


Feature: Added mortality

Description of PR
DEATH percentage column in table
Type of PR

[ ] Bugfix
[x] New feature

Relevant Issues
Fixes #...
Checklist

[x] Compiles and passes lint tests
[x] Properly formatted
[x] Tested on desktop
[x] Tested on phone

Screenshots

May have to rename Death percentage as Mortality rate. Rest looks good.

column title updated

well done to all involved! I had this in mind as an unfortunate but real outcome measure. Next step cud b to statistically predict. Know margin of error. we can use stats conf levels and keep restricted. Any one to prioritise. With over 20 yrs of corp biz prcess & six sigma experience, happy to collaborate. Nube on tech though but srong on logic. lemme know pls

Closing due to inactivity.

Sorry naive question - is it deployed as connot see online right now?

what is the use of data if we cant compare it with difference sources/states ?


use ctrl + click jump to definition as informative as pycharm can do

Vscode can't jump to definition as deep as pycharm can do. For example, if
I have
class base_class(ABC):
    ...
    def test_function(): # pychram will show this, vscode will only show this
          pass

class sub_class(base_class):
    ...
    def test_function(): # ctrl+click the test_function here
          pass

def another_function():
     test_case = sub_class()
     test_case.test_function()  # pychram will show this

ctrl+click the test_function the comment part, in pychram, I will be able to get a list includes both the test_function in the base_class and the usage in the another_function while vscode will only get the one in the base_class.

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


doc: fix up Ubuntu apt-get install command

The libmnl package in Ubuntu/Debian packaging system is named as libmnl0, not libmnl.
$ sudo apt install libmnl
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libmnl

We should install libmnl-dev or libmnl0 instead:
$ sudo apt install libmnl-dev libmnl0
Reading package lists... Done
Building dependency tree
Reading state information... Done
libmnl0 is already the newest version (1.0.3-5).
The following NEW packages will be installed:
  libmnl-dev
0 upgraded, 1 newly installed, 0 to remove and 68 not upgraded.
Need to get 11.0 kB of archives.
After this operation, 88.1 kB of additional disk space will be used.
Do you want to continue? [Y/n]

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin.
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed -->

This change is

Coverage decreased (-0.01%) to 41.259% when pulling a5a17f0b82505478bccce9853d5c4be03f7e18fc on liuqun:patch-1 into 3b7d8bf831349e556087ec65ff98ed6de521bb2c on cilium:master.

test-docs-please


Wrap columns_in_query query in select statement

Summary
Appending LIMIT 0 to query that already have a LIMIT results in a syntax error:
... limit 10000 LIMIT 0 FORMAT JSON ...
>>> Code: 62. DB::Exception: Syntax error: failed at position 6358 ('LIMIT') (line 200, col 17): LIMIT 0
 FORMAT JSON.

This is resolved by wrapping the query inside a select statement and limiting this clause.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the fix!  (Sorry about the formatting stuff, it's overkill imo but we haven't loosened it up yet.)


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.19.222",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=04181973c38f3d6a353f9246dcf7fee08024fd9e"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "04181973c38f3d6a353f9246dcf7fee08024fd9e",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "scsi: scsi_debug: Sanity check block descriptor length in resp_mode_select()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.222 by commit 04181973c38f3d6a353f9246dcf7fee08024fd9e. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002661


grey foldcolumn when after installing lsp-zero.nvim

I installed lsp-zero.nvim via lazy.nvim as soon as I restart and lsp-zero installs I get this grey border on the left.
I tried:

:set foldcolumn=0 as mentioned on this stack overflow post here

i also tried to set the color manually by:

:lua vim.cmd[[highlight FoldColumn ctermbg=none guibg=none]]

Before Installation:

After Installation:

I am pretty new to neovim would be nice if some one could help me out a bit.

That's the signcolumn. Its a spot reserved for line indicators. Neovim can show error/warning indicators in that space. It can also be used by other plugins (is not just for LSP related stuff).
If you want to disable it you can use the option signcolumn.
vim.opt.signcolumn = 'no'

If you just want to change the color, the highlight group is called SignColumn. You can try to do the same thing you did for FoldColumn.

It works. Thank you very much for your help, I appreciate your work.


Reapply "sched/spinlock: remove nesting spinlock support"

Summary
since https://github.com/apache/nuttx/pull/14200 fix the dead lock.
we reapply https://github.com/apache/nuttx/pull/14190
Impact
none
Testing
Build Host(s): Linux x86
Target(s): sim/smp
After that, there will be no warnings for compilation
make distclean -j20; ./tools/configure.sh -l sim:smp;make -j20
Copy files
Select CONFIG_HOST_LINUX=y
Refreshing...
CP: arch/dummy/Kconfig to /home/hujun5/downloads1/vela_sim/nuttx/arch/dummy/dummy_kconfig
CP: boards/dummy/Kconfig to /home/hujun5/downloads1/vela_sim/nuttx/boards/dummy/dummy_kconfig
LN: platform/board to /home/hujun5/downloads1/vela_sim/apps/platform/dummy
LN: include/arch to arch/sim/include
LN: include/arch/board to /home/hujun5/downloads1/vela_sim/nuttx/boards/sim/sim/sim/include
LN: drivers/platform to /home/hujun5/downloads1/vela_sim/nuttx/drivers/dummy
LN: include/arch/chip to /home/hujun5/downloads1/vela_sim/nuttx/arch/sim/include/sim
LN: arch/sim/src/chip to /home/hujun5/downloads1/vela_sim/nuttx/arch/sim/src/sim
LN: arch/sim/src/board to /home/hujun5/downloads1/vela_sim/nuttx/boards/sim/sim/sim/src
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/examples/mcuboot
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/examples
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/crypto
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/audioutils
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/logging
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/system
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/fsutils
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/benchmarks
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/database
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/interpreters/luamodules
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/interpreters
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/boot
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/testing
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/mlearning
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/graphics
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/netutils
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/sdr
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/canutils
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/lte
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/inertial
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/math
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/wireless/ieee802154
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/wireless/bluetooth
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/wireless
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/videoutils
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/games
mkkconfig in /home/hujun5/downloads1/vela_sim/apps/industry
mkkconfig in /home/hujun5/downloads1/vela_sim/apps
Loaded configuration '.config'
Configuration saved to '.config'
Create version.h
LN: platform/board to /home/hujun5/downloads1/vela_sim/apps/platform/dummy
Register: hello
Register: nsh
Register: sh
Register: taskset
Register: ostest
Register: getprime
Register: smp
CP:  /home/hujun5/downloads1/vela_sim/nuttx/include/nuttx/config.h
CP:  /home/hujun5/downloads1/vela_sim/nuttx/include/nuttx/fs/hostfs.h
LD:  nuttx

[Experimental Bot, please feedback here]
The PR summary is missing some key details:

Why: It mentions re-applying a previous PR due to a deadlock fix, but doesn't clearly explain what the original PR aimed to achieve.
What: It lacks specifics about the functional code changes being made.
How:  No explanation on how the re-applied changes work.

Impact section is too brief. While stating "none" might be accurate, it's better to explicitly confirm no impact on:

User experience
Build process (despite showing build commands, it's not a confirmation)
Hardware compatibility
Documentation needs
Security implications
Backward/forward compatibility

Testing section has issues:

Only mentions build hosts and target; needs more details about the testing environment (OS versions, compiler versions).
It claims "no warnings for compilation" but doesn't prove it.  The provided logs are build logs, not compilation logs.
No "before change" logs are present, making it impossible to assess the impact of the changes.

Verdict: This PR does not meet the NuttX requirements. It lacks crucial information in the summary and impact sections, and the testing section doesn't provide sufficient evidence of verification.

Wonder why this telnetd build is failing? It doesn't seem to be caused by the PR.
https://github.com/apache/nuttx/actions/runs/11313625049/job/31462630855
Configuration/Tool: zkit-arm-1769/nxhello,CONFIG_ARM_TOOLCHAIN_GNU_EABI
2024-10-13 11:11:50
telnetd.c: In function 'telnetd_main':
Error: telnetd.c:57:5: error: 'CONFIG_SYSTEM_TELNETD_SESSION_STACKSIZE' undeclared (first use in this function); did you mean 'CONFIG_SYSTEM_TELNETD_SESSION_PRIORITY'?
   57 |     CONFIG_SYSTEM_TELNETD_SESSION_STACKSIZE,
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |     CONFIG_SYSTEM_TELNETD_SESSION_PRIORITY

@lupyuen fix by https://github.com/apache/nuttx-apps/pull/2716

since https://github.com/apache/nuttx/pull/14200 fix the dead lock.

@anchao @hujun260 @xiaoxiang781216
I tried this PR with https://github.com/apache/nuttx/pull/14200, the nsh deadlock issue that I reported in https://github.com/apache/nuttx/pull/14079#issuecomment-2408714889 was resolved.
However, if I run nxplayer repeatedly with spresense:rndis_smp, deadlock happens.
I think we would find other deadlocks if we test more complicated use cases.
As I commented in https://github.com/apache/nuttx/pull/14079#issuecomment-2408714889, I think it's too early to remove nesting spinlock support. So again, I suggest we revert this PR.

since #14200 fix the dead lock.

@anchao @hujun260 @xiaoxiang781216
I tried this PR with #14200, the nsh deadlock issue that I reported in #14079 (comment) was resolved.
However, if I run nxplayer repeatedly with spresense:rndis_smp, deadlock happens. I think we would find other deadlocks if we test more complicated use cases.
As I commented in #14079 (comment), I think it's too early to remove nesting spinlock support. So again, I suggest we revert this PR.

@masayuki2009
OK, please review this PR whether works for you, thanks


Allow for multiple IOs to be specified in _A and _ro algorithms

[ ] Reconfigure alg_config and JSON standard

[ ] Allow musc option to be a cell
[ ] Make sure old syntax is backwards compatible
[ ] Defaults: IO_A = HouseQR, and IO_1 = IO_2

[ ] Refactor RunKappaPlots
[ ] Come up with succinct naming-scheme for plots
[ ] Update tex_reports

Based off v3

Regarding json config.  v2 puts muscle options in the second level as follows:
{
  "skel": {
    "musc-1": [],
    "musc-2": {
      "param": {
        "chol": "chol_free"
      }
    }
  }
}

alg_config expects skel to be a top-level key, and the second-level (musc-1, musc-2) should also be keys populated with values.  All keys are expected to be strings, so we can't simply replace musc-1 itself with a cell.
I thought we could just use the key cellqr (or similar) to signal that we want to specify a cell of IOs for a _a algorithm.  The problem here is that we might want to compare many such configurations, so they can't all have the name cellqr.
A possible solution to kill two birds with one stone is allow for any key with a cell value to be vetted as a possible cellqr-type configuration.  Whatever strings are provided then become a succinct label for the plots.  This could of course be abused, and built-in muscles will also be excluded from potential key strings.  However, the abuse would be at the user's own peril.  The README could specify wise key names, e.g.:

cellqr-1, cellqr-2, etc.
houseqr-cgs, mgs-cholqr, etc.

Regarding defaults:

bcgs_a only accepts 2 IOs: {IO_A, IO}.  The first will be taken as IO_A and the second as the iterated IO.  For the purpose of backward compatibility, when only one IO is provided, then IO_A = HouseQR and the provided IO = iterated IO.
As for _iro_a algorithms: {IO_A, IO_1, IO_2}.  For backward compatibility, when only one IO is provided, IO_A = HouseQR, and IO_1 = IO_2 = provided IO.

Reconfiguring the json is tricky-- each muscle is itself a dictionary, and to provide a list of muscles, we need a list of dictionaries, i.e., [{key:val}, {key:val}].  The following appears to be the right thing:
{
  "bcgs_a": {
    "cellqr-1": [
      {
        "houseqr": []
      },
      {
        "houseqr": []
      }
    ],
    "cellqr-2": [
      {
        "houseqr": []
      },
      {
        "cholqr": {
          "param": {
            "chol": "chol_free"
          }
        }
      }
    ]
}

should be equivalent to
{
  "bcgs_a": {
    "houseqr": [],
    "cholqr": {
      "param": {
        "chol": "chol_free"
      }
    }
  }
}

Before merging, this needs to be rebased off master.  Wait until BCGS-PIPI+ work is done.


Cannot run without --bypass-dns-resolution=true

I'm experiencing troubles with kubelet-csr-approver without passing the option --bypass-dns-resolution=true
"Denying kubelet-serving CSR. DNS checks failed. Reason:The SAN DNS Name could not be resolved, denying the CSR"
$ dig A k8snode03u @<IP_ADDRESS>
$ dig A k8snode03u.cluster.local @<IP_ADDRESS>
Doesn't resolve to the node IP address.
Am I doing something wrong on my cluster?
Is having core-dns resolving the node's hostname a usually expected behavior?
How do you enable such feature on your cluster?

This doesn't have anything to do with your cluster config.
The node name is only resolvable if you have an external DNS provider (be it a DNS server, your router in some cases that are built-in with DNS resolution features, your cloud provider), or if you use the "bonjour" protocol (avahi-daemon on Linux), which permits to locally resolve node name through appending the .local suffix
resolving a node IP with the cluster.local suffix will not work, as there is no (to my knowledge) mechanism in Kubernetes and in its CoreDNS integration to make the node IP available through a DNS query.
in summary, CoreDNS will resolve your node's IP if the upstream DNS servers can resolve your node IP. Alternatively, you may find one of CoreDNS's plugins useful in helping you achieve DNS resolution of your nodes: https://coredns.io/plugins/
I hope this answers your question.

Ok.
This does not seem to be a usual DNS feature on Kubernetes clusters in production.
Do you agree that this check can be safely disabled as long as a decent provider regex is configured?

yes, fully agree. the most important part is that the regex is properly defined, to prevent a node from requesting a oauth.company.org type of DNS name 😅


website improvements

[x] jump to anchor tags links
[x] copy code snippets
[ ] add multilingual snippets in guide / modules too
[ ] add miniguides

i think a primate cookbook would be cool to work on. cookbook.primatejs.com


cmd/compile: implement duffcopy on ppc64x

implement duffcopy on ppc64x.
See TODO in code and stale pending CL https://go-review.googlesource.com/c/9684/ which could be resurrected.
/cc @minux @laboger

Are there benchmarks to test the performance of duffzero and duffcopy? I don't see any.
I'm not sure the duff implementations would perform better than memclr and a memcpy (similar to current memmove) on ppc64x.

Probably src/runtime/memmove_test.go


Revert the tag api change introduced in 430bf12

Pull the tag change from 430bf12 into #2013 so it can be discussed there.
Currently the dev API is half-wrong or half-fixed (depending on your perspective)

Since the other PR was intended, this is unnecessary


[Drawer] Mini Variant Example with discoverable menus on button links.

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
A new version / example of the Mini Variant drawer, in which we could demonstrate how to include a discoverable menu on each button link of the side bar.
My current progress:
https://codesandbox.io/s/material-demo-su5e4
I added the menus component to the mini variant.
What I need to add to it:
Whether it's opened or closed, the associated menu would pop up aligned right to the button and would have different buttons for each menu (the former I am currently trying to figure out, any suggestions would be great).
There is already a pull request for including a nested menu in the side bar, in which there is now talk of keeping it separate to the initial plain mini variant:
https://github.com/mui-org/material-ui/pull/19632
So I am suggestion keeping this as a third example. Aternatively, we could just add this feature to the mini variant with nested menu, as there is 3 left over buttons that have not been used to demonstrate this. If we choose the alternative route, I am willing to accept this as blocked for the time being until the nested menu version is completed and merged.

Examples 🌈
Already suggested briefly in issue #19108

Motivation 🔦

I think it is important to include both alternatives for the Mini Variant drawer.

I am adding an example of nested (3 levels) 'flyout' menus. https://codesandbox.io/s/material-demo-forked-mqm2s

Coming back after some time, I'd be happy to review the new demo if someone wants to add it. I honestly don't think it's worth adding this new demo, as it doesn't really look like a common use-case. @sleepless000 did a good progress on his codesandbox, so it could be used as an inspiration.


Added tests for the public methods in the Multiselect

Slowly increasing code coverage and preparing to do some cleaning in the Multiselect. The expand/collapse functionality will need to be tweaked to be testable, due to the aspects of jsdom, it's difficult to test for changes triggered by focus.
Additions

Added tests!

Removals

Removed unnecessary functions that were previously replaced by string utilities

Testing

gulp test:unit:scripts

Review

@anselmbradford
@sebworks

Todos

Rewrite the expand/collapse functionality to remove the dependency on focus

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

👍

👍


Don't hardcode bucket for Transcribe

See https://github.com/superwerker/aws-organizations-vending-machine/blob/7d004ff2ddc458a7c23961c1967d74e5e9ce39e4/code/create.js#L272
Also, implement auto-purge on the bucket.

invalid, since audio captcha is gone


Fix for Laravel's semantic versioning

Laravel 6.1.0 and up were not supported due to the ~6.0.0 constraint.

Good catch, thanks!


fix(plugin-chart-table): Resetting controls when switching query mode

SUMMARY
As a result of changes in PR https://github.com/apache/superset/pull/19039, when user switched query mode in table chart between Aggregate and Raw Records, the controls in currently inactive mode were being reset. This PR fixes that problem by introducing a new prop resetOnHide, which is true by default and set to false in table chart controls.
Please consider this PR as a hotfix - we should probably figure out a more sturdy, foolproof solution.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
https://user-images.githubusercontent.com/15073128/164252120-0641eae7-c268-4fbe-bd3c-9f50cd98656d.mov
After:
https://user-images.githubusercontent.com/15073128/164252307-2b3af4a7-fd52-40fb-9d87-bca68e81f51a.mov
TESTING INSTRUCTIONS

Open a table chart
Add some values to controls in aggregate mode
Add some values to controls in raw records mode
Switch between the modes a few times, verify that controls are not being reset
Run query, verify that values of currently hidden controls are not included in the query

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

🏷  preset:2022.15


Trouble with Python3/basestring and None values.

I had to hack in a few things to get kifield.py to work for me under anaconda python3.
one was:
    if USING_PYTHON2:
        bstr = basestring
    else:
        bstr = str
    if isinstance(collapsed, bstr):

Instead of
    if isintance(collapsed, str) or isinstance(collapsed, basestring):

The other was adding a line to strip None:
    possibilities = [x for x in possibilities if x is not None]

Just after def lc_get_close_matches(...)
Because it was crashing on str.lower a few lines lower when it hit any None values.
These are total hacks of course, but they got me going. I hope someone finds this useful.

I implemented your fixes. Thanks for the help!


Camera snapshot service from chosen stream source (sub/main)

Describe the bug
When the sub stream is selected for a camera, the HA snapshot service grabs a frame from the main profile.
Also the entity image is from the main stream.
Additional context
Snapshot from the sub stream when chosen as source in the setup would be great for external object processing speed.

Hello,
i have the issure that i set up SUB stream in integration setting, but nothing changes, it works from MAIN stream also afetr restart of HA

For as far as I know, the snapshot has no option to choose quality. Check here, there is no parameter included. The integration use the snapshot option from the CGI api, which has only those params:
'''
param = {"cmd": "Snap", "channel": self._channel}
'''
Can you choose snapshot quality anywhere else, like the WebUI or Reolink Windows application?

Also the command /cgi-bin/api.cgi?cmd=onvifSnapPic&channel=0 has no quality option.
I'm closing this issue, please inform me if you find out how to do this.


Add example for fallback function to README.md

I was searching for some explanation how I can set a fallback because I come from the Java version which uses self-written classes, see the Java version. So when I searched for the fallback I found some information, e.g. in the HystrixCommand implementation. It seems like I can just pass some delegate but a nice example for that in the global README.md would be great. If you agree I could do a pull request as well.

Sounds great, feel free to issue a PR adding an example to the README.


Works with Node v10 not v14.2

This works:
nvm use 10.0.0 64
npm i
This doesn't, and fills the console with compiler errors related to v8:: functions:
nvm use 14.2.0 64
npm i
Why?

Just incase anyone actualy looks at this:
Failure on 14.6.0 confirmed on Arch Linux, debug log in case it helps:
2020-07-29T15_47_52_946Z-debug.log
And 10.0.0 also fails for me, at least using asdf, averything is up to date:
2020-07-29T16_17_26_295Z-debug.log


[ADN] Use PhantomJS to get the subtitle encryption key (closes #12724)

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Use the PhantomJS wrapper class from the openload extractor to get the second half of the encryption key for subtitles.
(see #22035)
Make sure to add phantomjs in your PATH environment variable.
Should we refactor this wrapper class in the utils.py file (or make a phantomjs.py class)?
Thank you!

i see other extractors like pornhub.py just import the wrapper class from openload.py
thats kinda strange design; but i guess i'll just comply with this design and update this pull request accordingly
if a little refactoring affecting 3 extractors and the main YoutubeDL.py file doesn't bother you, just tell me and I'll do it in another PR
fixed, and thank you very much for the feedback!

@dstftw Heya, sorry for the mention.
Do I need to do something so that my PR gets reviewed again or do I have to wait?
Sorry if I'm rushing the process
Thank you!

Hello
please can u know how to use this pr please ?


affected/package: (os/exec) The stdin of the subprocess is closed abnormally on windows.

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

$ go version
go version go1.18.9 linux/amd64

Does this issue reproduce with the latest release?
I don't know.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="off"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/jun/.cache/go-build"
GOENV="/home/jun/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/jun/Projects/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/jun/Projects/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/golang"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18.9"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2999689779=/tmp/go-build -gno-record-gcc-switches"

What did you do?
The parent process is developed by go and is compiled on WSL(centos 7).
The subprocess is develop by C++ and compiled by either MSYS2 or Visual studio.(This is critical. If the subprocess is developed by go, the problem doesn't happen.)
The parent process works like this:

create the subprocess
read data from a file( This is critical.)
write some data to stdin of the subprocess

The subprocess works like this:

read data from stdin and print the amount of the data read

problem:
In the subprocess, after reading 2 or 3 times for some data, nothing can be read after that. The read always returns 0 then.

The parent process is developed by go and is compiled on WSL(centos 7).
The subprocess is develop by C++ and compiled by either MSYS2 or Visual studio.(This is critical. If the subprocess is developed by go, the problem doesn't happen.)
The parent process works like this:

create the subprocess
read data from a file( This is critical.)
write some data to stdin of the subprocess

The subprocess works like this:

read data from stdin and print the amount of the data read

problem:
In the subprocess, after reading 2 or 3 times for some data, nothing can be read after that. The read always returns 0 then.
What did you expect to see?
The subprocess reads all data from the parent.
What did you see instead?
The subprocess reads only a few data and then can read nothing.

This doesn't happen on Linux. It only happens on Windows.


Aplicando en Laravel

Sorry I could not see this topic again.
Look in the controller of the post path where the file should be received so I have it.
 public function postUpload()
    {
        echo json_encode(Input::all());
        die;
    }

This is what i get:
{items: [{}]}
items:[{}]
0:{}

If I put it like that:
 public function postUpload()
    {
        echo json_encode(Input::file('items'));
        die;
    }

I get this:
[{}]
0:{}

If I put it like that:
    public function postUpload(Request $request)
    {
        echo json_encode($request->allFiles());
        die;
    }

I get this:
{items: [{}, {}, {}]}
items:[{}, {}, {}]
0:{}
1:{}
2:{}

Sorry, I only speak spanish and I'm using google translate

Hi there,
I think you are not initializing the component properly. I've uploaded the two demo files for you to have a look at:
https://github.com/abarta/vue2-multi-uploader/blob/master/demo/App.vue
https://github.com/abarta/vue2-multi-uploader/blob/master/demo/main.js
Please try to include the uploader this way. Also, as you mentioned Laravel, are you including the CSRF token? Here's is the documentation:
https://laravel.com/docs/5.4/csrf
Let me know if I can help any more.
Thanks!

Subi los archivos a un repositorio para que los puedas ver como lo tengo implementado los js
(I upload the files to a repository so you can see how I have implemented the js)
https://github.com/asarmiento/tempUploadFile/blob/master/resources/assets/js/components/Example.vue
https://github.com/asarmiento/tempUploadFile/blob/master/resources/assets/js/app.js
Aqui te muestro como lo tengo en php (
Here I show you how I have it in php)
https://github.com/asarmiento/tempUploadFile/blob/master/resources/views/home.blade.php
https://github.com/asarmiento/tempUploadFile/blob/master/routes/web.php
https://github.com/asarmiento/tempUploadFile/blob/master/app/Http/Controllers/HomeController.php

Listo Solucionado.

Glad to hear it works now.
Have a good day in Chile!


RBAC Plugin

Kismatic Kubernetes role-based access control (RBAC) plug-in

Yes :) these guys are cool, talked to them at mesoscon. Check it out and
we'll meet with them...also need it for some frameworks
On Aug 23, 2015 5:17 AM, "Alexander Litvinenko"<EMAIL_ADDRESS>wrote:

Kismatic https://kismatic.io Kubernetes role-based access control
(RBAC) plug-in https://github.com/kismatic/kubernetes-rbac
—
Reply to this email directly or view it on GitHub
https://github.com/CiscoCloud/kubernetes-ansible/issues/60.

+1


Defense Bonus not sent on Initialization

For the time being, we are going with a defense bonus of 1.5.


fix: show custom model filename in chat panel

The model selection combo box above the prompt text area was always showing the hugging face model, even when the active selection is a custom model.

@HarukaMa Can you check if the changes https://github.com/carlrobertoh/CodeGPT/commit/7c067d9edd546375be419907b068d1b003e0db25 since 2.3.0 fixed your problem?

Oh, yes, it has already been fixed some time ago but I forget to follow up here.

No problem, glad that this bug has been fixed, thank you for reporting and providing an implementation 👍


Error while using Keycloak with JHipster 5.8.0

I created one microservice application using JHipster , with keycloak oAuth2 server. Then trying to signup error is showing
Error occured while trying to proxy to: localhost:9000/oauth2/authorization/oidc
JHipster + keycloak + angular able to sign up and  its working fine, but not working with vue js?

You can copy the solution from https://github.com/jhipster/jhipster-vuejs/pull/378/files, I believe that should work

If this fix works for you @sreerajr137, it will be fixed by https://github.com/jhipster/jhipster-vuejs/pull/378.

Hum that's many people who encounter that issue. Let's merge this as this is the solution in the main generator.

@sreerajr137 Could you try again please with the fix now provided ?

@sreerajr137 I noticed the JHipster version you mentionned. Latest version of this blueprint uses JHipster 6.0.X, if you'd like to use a prior version, please use blueprint version prior 1.0.0 release (but I can only advise you to use our latest versions^^)

Its not working with latest version(6.0.2) also...

Are you using the latest blueprint version or the master branch of it? This should be fixed within the master branch? If so, I could do a release soon.
Thank you for your feedbacks

JHipster Version(s)
--<EMAIL_ADDRESS>--<EMAIL_ADDRESS>

Ok, so it should be fixed within the next blueprint release. Check #378 for quick fix if you want.

As there is no more updates on this, I close.


Add Support for Vim 7.4+

Here is my early job to add support for Vim 7.4+ by using Vim's Channel. #44

Would you @imaben @przepompownia like to make some test?