Confusion about a couple JsApi methods/documentation

I have been analyzing the way svgo works for the past few days... and I have noticed some inconsistencies with the documentation for a few methods.
I want to clarify how each method is intended to behave before I submit a pull request to update the documentation.
Question 1
The hasAttr method takes a string as its first argument:  https://github.com/svg/svgo/blob/ceccf1ffdcdcba0f422a49685c2f60520579e46a/lib/svgo/jsAPI.js#L134-L152
However, the removeAttr method calls hasAttr and in some cases passes an array as its first argument: https://github.com/svg/svgo/blob/ceccf1ffdcdcba0f422a49685c2f60520579e46a/lib/svgo/jsAPI.js#L235-L258
So my question is which of the two methods should be updated? Should the hasAttr documentation be updated, saying that it can take an array as an argument as well? Or should the removeAttr method be updated so that it never passes an array to the hasAttr method?
Question 2
The documentation for the computedAttr method says that it should return an object or undefined, which makes sense: https://github.com/svg/svgo/blob/ceccf1ffdcdcba0f422a49685c2f60520579e46a/lib/svgo/jsAPI.js#L215-L233
However, as you can see on line 128, it is possible for the method to return elem.hasAttr(name, val), which is a boolean: https://github.com/svg/svgo/blob/ceccf1ffdcdcba0f422a49685c2f60520579e46a/lib/svgo/jsAPI.js#L228
Should the documentation be updated to indicate that a boolean value can be returned as well? Or should the method be updated so that it is not possible for a boolean value to be returned?

hasAttr is just a method to check for a certain attribute. removeAttr was introduces for convertShapeToPath, and it's just a convenient way to remove a bunch of attributes in one line. Second parameter was added just for the sake of completeness. I don't think it's much use for an array in hasAttr.
computedAttr  is used to check for an attribute existence or a certain value, so boolean is a valid response, need to update the description.

So to clarify, should it be possible to pass an array to hasAttr? Because currently it is possible for an array to be passed as an argument.

I don't think it worth it. Just can't think even a case where it could be useful. Unlike removeAttr, where removing a bunch of attrs is one of the primal cases.
What do you mean by “it's possible for an array to be passed”?

Right now if you call removeAttr(name) (where name is an array of attr names), the method will end up calling this.hasAttr(name) on line 248, right?

Ah, yeah, it should have return earlier.

OK, great... that is kinda what I thought but wasn't sure. :)
I'll file a pull request to fix that.

Actually wait... I have one more question...
On line 246 you call name.forEach(this.removeAttr, this): https://github.com/svg/svgo/blob/ceccf1ffdcdcba0f422a49685c2f60520579e46a/lib/svgo/jsAPI.js#L246
According to the documentation for Array#forEach, it takes a function callback(currentValue, index, array) as its first argument.
On line 246, you pass this.removeAttr, which doesn't seem right (because the second argument for this.removeAttr is a string, but the second argument for the callback passed to Array#forEach is an index number.
Is this also a mistake?

Yep, that's why on line 250 it checks for the 3rd parameter recursive—it would be an array in case of forEach.

OK, thanks.
I created a pull request here with what I assume should be the fix... LMK if I should change anything though: https://github.com/svg/svgo/pull/855


[WiP] WebHook-to-WebSocket forwarder

I think I have it mostly working now, will test with https://github.com/pdsinterop/solid-nextcloud/issues/105


downloadable visio or pptx files of the images

[Enter feedback here]
hi,
Many thanks for updating this page with the end to end security map. This is super useful. Kindly help to provide the visio or pptx (or both) file, which will help us to build this into our existing security models. Hopefully this will be updated regularly as there are new acquisitions and features in each of your products/solutions.

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

ID: e017b080-4692-bb90-8fd4-8ce1f65f4b4c
Version Independent ID: 980eb134-f777-4f09-db6a-a744038e061c
Content: End-to-end security in Azure
Content Source: articles/security/fundamentals/end-to-end.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @TerryLanfear
Microsoft Alias: terrylan

@knssub
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@knssub - Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to act on your suggestion in a timely manner and we sincerely apologize for the delayed response. We've created an internal work item to check on your suggestion. If we are able to provide the images, the article will be updated with guidance on how to get the images.
#please-close


fix(Icon): Use BEM for Icon modifier classes

Process

I've confirmed that upgrading to the latest cfpb-icons library is the source of the breakage, causing the updating icons to no longer be animated
I've confirmed that other icon modifiers (ex. Success alert icon shown in green) seem to be working as expected.

How to test this PR

Visit https://deploy-preview-364--cfpb-design-system-react.netlify.app/?path=/story/components-draft-icons--status-icons
Confirm that the updating icon is animated

@shindigira Thanks for the quick review!  Requesting re-review because I needed to update broken unit tests.


fix: CliCommand Context is default to indexed object.

closes #186

:tada: This PR is included in version 2.4.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Force re-layout not working

Version: weaveworks/scope:release-1.5-1667984
Docker for Mac
Sockshop
Standalone Scope
Please see:
http://recordit.co/cZshm6AwXh

@abuehrle What was your expected behaviour? Take a look at my comment here: https://github.com/weaveworks/scope/issues/2564#issuecomment-307779912
I think people have very different ideas what the refresh button should be doing, see the issue https://github.com/weaveworks/scope/issues/2377 and my "fix" https://github.com/weaveworks/scope/pull/2407. :)

I expected the containers to centre themselves within that view so that you can see all of the containers including the ones that have fallen out of the main view.  When you click the force layout, it appears as if nothing happens.  So the zoom correction was removed? What should I be looking for when I force re-layout?

Yeah, most people seemed very opinionated about this :)
The idea with the refresh button is that it would force relayouting on the graph, meaning the nodes would rearrange if they were not positioned "optimaly", while the zoom state is currently left unchanged by design.

Ok... I will leave that one.


For me it’s not working

Im using premium resendial proxys but the views don’t show up, on the contrary, YouTube starts deleting my real Klicks.. what’s happening? Looks like it do not work like it should :(

Post your config.json here.. on how many videos you were running the bot?

First of all thanks for your fast answer and second here is the config
{
"http_api": {
"enabled": true,
"host": "<IP_ADDRESS>",
"port": 5000
},
"database": true,
"views": 2500,
"minimum": 30.0,
"maximum": 35.0,
"proxy": {
"category": "f",
"proxy_type": false,
"filename": "privatresendial",
"authentication": false,
"proxy_api": false,
"refresh": 0.0
},
"background": false,
"bandwidth": false,
"playback_speed": 1,
"max_threads": 5,
"min_threads": 2
}
do you need to see something else?
Thanks for your help

Actually I’m just trying to run on one video with many links to it, but since this don’t work I tried also with more videos.. but seems like not working like it should to :/
Just bought expensive privat resendial Proxies for that..

you should specify the proxy type since you know your proxy type "proxy_type": false, change false to http/socks4/socks5
can you tell me how many proxies you are using?

Yes, I am using privat resendial proxies, with own ports. They are 25, but each of them should change their IP every 5 Minutes. So they are really good Ones.

thanks again for your fast answer. I will try it. I tried it but seems like not adding really views to my video, you may have anydesk support or something? I also would pay you.
I am using privat resendial proxies, with own ports. They are 25, but each of them change their IP every 5 Minutes. So they are really good Ones. I think you know what I mean.

Coud you please provide link to the proxy website?
Regards

@albocino  contact me over mail then


[ENH] fresh command

hi,
it would be nice to include a fresh command in Laravel to remove models, controllers , ... created by default in basic and advanced templates.
For example :
// execute command in basic template
php yii fresh
it remove the following files in folders:

assets/*
commands/*
controllers/*
mail/layouts/*
models/*
tests :

tests/codeception/acceptance/*
tests/codeception/functional/*
tests/codeception/unit/models/*

views :

view/si1te/*
layouts/main.php

web :

web/favicon.ico
css/*

this folders should be custom in config/console.php

Isn't it better to create empty application template?

This does not make any sense. New application template should be created for this.

I think it makes sense for crackers - to break someone site by one command :)

Laravel has fresh command and it seems nice for yii... but it is not priority, i close issue.
Thanks for comments :)


/admin/prizes should show top 8 prizes for league as well as new players

The actual competition pages for leagues do this in perpetuity so I've decided it's not necessary.


Completion not working in recent Jupyter Lab and Jupyter Notebook

I am using recent JupyterLab/Notebook and can not get the completion get to work. One of the test cases is failing
($pos,$end,$completions) = $r.completions('if "hello world".WHA');
is-deeply $completions, $("WHAT", ), 'WHAT method';

but appart from that there are no errors I can identify. The Binder demo also has no working TAB completion.

Interesting.  For the bug, perhaps this will address it: 54163f422c5caa7fcd3199cd07b0e033a9896ef3 since there appears to be duplicate WHATs in the output.  I can confirm that Tab completion is not working in Binder, though the kernel does seem to be sending the responses, yet they are being ignored.  Completion does work on a local notebook. cc @sumanstats for any thoughts

Oh, I see it is not just binder, but JupyterLab.  Yes, I am able to reproduce this.  There seems to have been some sort of backwards-incompatible change to the protocol; I don't see any documentation about this here so it may take some investigation to see what has changed.

Interestingly, this seems specific to JupyterLab -- completions work fine in a regular notebook or a terminal console (and the kernel is sending the same response in all those cases).


(Bug) Error when destroying and initializing the Button component

Cannot set property 'dispose' of undefined.

https://drive.google.com/file/d/1c5N11_0IASIReuOWg8K-Q6GyargrHHxr/view?usp=sharing

The error applies to all components.
Presumably the main component is removed.

My apologies, incorrectly initialized. It was necessary to use new


New devices

Added:
class SLAMonitor
class PhysicalInterface
class IPv4StaticRoutes
class DeviceHAPairs
class DeviceHAMonitoredInterfaces
class DeviceHAFailoverMAC

@parkerbrother1 ipv4 static routes is proving problematic, so I am removing it for now.  Other tests have been added for the new class features.


Overview graphic and updated references to ChannelWorm2 in README.

Coverage remained the same at 69.386% when pulling b2e94402297909743998cd352b6eaf51f2308e5c on gsarma:master into 477ae0b5453109785d9711f1e270510681d9585f on openworm:dev.

Are any of these diagrams going to update to show "ChannelWorm2" vs "ChannelWorm"? If not, the then lgtm.

Yeah, good point Mark.  I noticed that after I created them, and then thought that maybe they should be consistent with the published version in the paper coming out.  If you're ok with it, let's go ahead and use these for now, and then I'll update all of them en masse, but it might take a week or so before I get to it.


Added GetHashCode override to PartialDateTime. Fixes warning CS0659.

Hi Christiaan, the PartialDateTime has been totally changed internally, so this code is outdated ;-)


Add new Apple Watch Series 4 sizes

Hope no one minds that I added newlines to the Watch sizes, was getting too hard to read and modify!
Also fixes the broken tests!
Closes #44

Any updates on this PR? 😏

Fixed via #47


Fixes the bug when upgrading to Python 3.7

Python 3.7 seems to handle the async keyword differently than 3.5 and 3.6, thus syntax errors occurred when upgrading to Python 3.7.
This renames the async parameter in the call_api method of the api_client to asynchronously in order to fix this bug.
This fixes #39 and #64

These are auto generated files that will be overwritten as soon as I regenerate the API.  The fix here needs to happen with the Swagger generation.  I'm going to work on this soon, but until then, 3.7 won't work.


Many improvements

Config system (file for unsecured options, hidden input for authentication keys)
Build system that configures and runs the Docker container for you
Minecraft will be gracefully shutdown over RCON instead of abruptly stopping the instance and crashing the server
Guide on how to set the proxy server up
Small fixes here and there to bring the project up to date and more user friendly

Thanks for the contributions @logwet :)


go.mod, vendor: bump k8s deps from 0.24.0-alpha.0 to 0.24.0

Looks like this needs an update in github.com/cilium/cilium/pkg/slim/k8s first.


Assert device clipboard

Describe your idea
Suppose an app has - in one of its screens, a button that reads Copy to clipboard. It could make a lot of sense to validate that the button indeed copies text to the device's clipboard.
We can introduce API's into Detox that would assert the clipboard's content (provided that it's text-based).
cc @smokiyeno-wix

Refs: https://carmelosantana.com/accessing-android-clipboard-via-adb-4350/

Is this anyhow related to Pasteboard on iOS?
https://github.com/wix/Detox/pull/658

Is this anyhow related to Pasteboard on iOS?
#658

Nice catch. Might be, though it looks different. @asafkorem wdyt?


(BUG) The "Share" screen is displayed after selecting the "Copy to clipboard" method

Precondition:
Create a send transaction and stop on the screen with the "Share" button
Steps to reproduce:

Click the "Share" button
Select the "Copy to clipboard" method
Pay attention to displaying the "share" page

Actual result: The "Share" screen is displayed after selecting the "Copy to clipboard" method
Expected result: The dashboard is displayed after selecting the "Copy to clipboard" method
NOTE: reproduced for QA and native only Android

Checked on  QA 1.10.1-1
Samsung S8 // Android 8
Samsung S9 // Android 10
Video: https://www.screencast.com/t/gvTeTqGxR

https://stackoverflow.com/questions/49663206/navigator-share-wont-resolve-nor-reject-when-user-cancels-native-selector-on-an?answertab=active#tab-top
working on fix

@sirpy Unfortunately the fix doesn't works as expected - window doesn't triggers focus event just after sharing popup closed. You need also to click somewhere on the app UI, then event appears and sharing could be considered as done
I propose to use another workaround for Androind

call navigator.share inside try..catch
if any sync error happened - reject
otherwise race sharing result promise with 100-500ms delay (to wait for possible async errors). if share() call haven't resolved or rejected during that time - resolve anyway

From UI this will look the following way

we clicked share
sharing popup has slided up
the app immediately started redirecting to Dashboard
after we confirming / cancelling share we are on the Dashboard now

of course we could find (experimental way) Android / Chrome versions where bug reproduces and use workaround only for them

Moved to backlog, PR changed to the Draft

@serdiukov-o-nordwhale in the bug link you shared this appears as solved for chrome > 80
@vldkh please retest on mobile with chrome >80

The issue is not reproduced on QA env. The "Share" screen is not displayed after selecting the "Copy to clipboard" method.
Environment:
QA 1.29.1-17
Devices:
Samsung A80 // Android 10
Attachments:
https://www.screencast.com/t/dN1kcG4wGjh

@sirpy @serdiukov-o-nordwhale
The issue is not reproduced on QA env. The "Share" screen is not displayed after selecting the "Copy to clipboard" method.
Environment:
PROD 1.30.2
Devices:
Samsung A80 // Android 10
Attachments:
https://www.screencast.com/t/HZDY2hWFalI7


Speedup output serialization

This changes do not break compatibility if you use tf.make_ndarray:
future = stub.Predict.future(request, 5.0)
tensor = future.result().outputs["output_layer_name"]
values = tf.make_ndarray(tensor)

resolves https://github.com/IntelAI/OpenVINO-model-server/issues/124

@mzegla, @dtrawins, please review

Addressing this issue in https://github.com/IntelAI/OpenVINO-model-server/pull/119


Drop components on mouse position

When dragging components onto the canvas, drop it on the current mouse position

Fixed in commit e5b8458


Lina

This pull request so far has only Lina's bio in it.  Can you add the info for the rest of team2, commit it to the same branch, and push it back up?


MQTT Light Add brightness_command_template?

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.63.1
Python release (python3 --version):
3.6.3
Component/platform:
mqtt.light
Description of problem:
Brightness Command Template
There seems to be no templating to pass the brightness command to a custom MQTT light. I want to pass '%brightness', ie symbol '%' and then the brightness. Could someone modify the mqtt_light to be able to accept jinja format brightness out values just like we have templating for RGB values. This is more of a feature request than a bug.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum. Thanks!

Btw, I think you can already do what you want. You have access to the brightness in the command_on_template. Is that not enough?


incorrect hour/minute behaviour in batch date edit header

Occasionally, when changing hours or minutes in the batch edit header new days get generated incorrectly.

Assigning to myself to have a look

Assigning to @bp323 for review

Verified and merged.


How to add the specific location point names in postwrf

Hi all , am seeking the guidance how to add specific location point names into contour plots in Postwrf. Specifically, I am interested in adding location names based on latitude and longitude coordinates.

Information of your lat/long and location names are read from the WRF/ERA5 DATA EXTRACTION (ON SURFACE).
Then if you have enabled Location_point_mark_on_off, they will be plotted on your map.
Take a look at https://postwrf.readthedocs.io/en/latest/contour.html#example-1

Information of your lat/long and location names are read from the WRF/ERA5 DATA EXTRACTION (ON SURFACE). Then if you have enabled Location_point_mark_on_off, they will be plotted on your map. Take a look at https://postwrf.readthedocs.io/en/latest/contour.html#example-1

Thank you for quick reply , i have got your point

Thankyou so much for the quick reply I resolve the issue it it amazing tool
On Mon, 30 Oct, 2023, 13:11 Amirhossein Nikfal, @.***>
wrote:

Information of your lat/long and location names are read from the WRF/ERA5
DATA EXTRACTION (ON SURFACE).
Then if you have enabled Location_point_mark_on_off, they will be plotted
on your map.
Take a look at
https://postwrf.readthedocs.io/en/latest/contour.html#example-1
—
Reply to this email directly, view it on GitHub
https://github.com/anikfal/PostWRF/issues/23#issuecomment-1784638489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2HPVPLQPVXUHOY6FRI6UQDYB5K3JAVCNFSM6AAAAAA6VQW256VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBUGYZTQNBYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


How can I reloaded data when I click the tabBar button?

Before You Start...
The issue list is reserved exclusively for bug reports and feature requests. That means we do not accept usage questions.
For usage questions, please use the following resources:

Read the docs
Ask on the Reactiflux (#react-navigation)
Look for / ask questions on StackOverflow
Also try to search for your issue - it may have already been answered or even fixed in the development branch. However, if you find that an old, closed issue still persists in the latest version, you should open a new issue.

Current Behavior

If describing a bug, tell us what happens. Provide a screenshot when possible and full code sample
If suggesting a change/improvement, explain the difference from current behavior

Expected Behavior

If you're describing a bug, tell us what should happen
If you're suggesting a change/improvement, tell us how it should work

Your Environment

Include as many relevant details about the environment you experienced the bug in

software
version

react-navigation

react-native

node

npm or yarn

Hi. It looks like this is a usage question. We're trying to reduce the number of open issues on GitHub so we're asking users to please use the Reactiflux #react-navigation channel or posting to Stack Overflow to get their questions answered.
There is currently no tab onPress API that react-navigation provides. You could try a work-around by providing a custom tabBarIcon that has an onPress method.


Link Module, VM and User

Module:

[ ] Link to User
[ ] Link to VM

VM:

[ ] Link to User
[ ] Link to Module


Model refactor

The models are old and dusty. A refactor is required using the following convention:

The primary key of a table is the id
Any foreign key of a table is a table_id.

example:
In the Tournament class, the owner of the tournament, which is a user, is reffered by  user_id, not user.

Any entity related to another must be pointed by an homonymous method.

example:
In the Tournament class, the owner of the tournament, which is a user, should be returned by a user() public method.

The relationships should follow the laravel convention in order to simplify them.

New Data Model definition:
Entities:

Game
Ground
Map
Participant
Player
Report
Round
Scenario
Tournament
User

Game

A Game has several Report

Ground

A Ground has several Map

Map

A Map has several Scenario
A Map has several Game

Participant

A Participant has several Report

Player

A Player has several Participant

Round

A Round has several Game

Tournament

A Tournament has several Participant
A Tournament has several Map
A Tournament has several Round

User

A User has several Player
A User has several Ground
A User has several Tournament

Ground is the old Map
Map the old Tournament - Map pivot
Participant is the new Player - Tournament pivot
Game no longer belongs to Ground but to Map
Report no longer belongs to Player but to Participant


add a .gitignore

Can you please explain this?

Sorry for the mess, I wanted to make these commits to dev branch and ended up making a pull-request for merging with master branch.


Imports with @ prefix

Hello,
Thank you for this plugin, it is really an awesome tool!
Would it be possible to add support for '@' in the import path?
I am not 100% sure, but I think my errors are coming from the plugin not processing those. Each time I try to do somethingn like  {import('@xxx/xx').xx}, I am getting a parse error from jsdoc.
Example:
/** @type {import('@google-cloud/firestore').DocumentData} */
const data = {}

ERROR: Unable to parse a tag's type expression for source file lib/firestore.js in line 2 with tag title "type" and text "{import('@google-cloud/firestore').DocumentData}": Invalid type expression "import('@google-cloud/firestore').DocumentData": Expected "!", "=", "?", "[]", "|", or end of input but "(" found.

In the case of firebase, I can work around the issue by using import('firebase-admin'), but for other libs, it is not an option.

Also getting this issue, Only thing holding back right now is My packages use '@'

Thanks for the PR!


Introduce roles_path to support nested roles in OIDC config (eg. keyclock nested roles)

We should introduced roles_path
`
if (jsonRolesPath != null) {
                Object roles = JsonPath.using(jsonPathConfig).parse(claims.asMap()).read(jsonRolesPath);
                return splitRoles(roles);

This way customer using the "." in their key in payload will not be affected
Originally posted by @palashhedau in https://github.com/opendistro-for-elasticsearch/security/pull/698#discussion_r566357711

We are doing some "spring cleaning in the fall", and to make sure we focus our energies on the right issues and we get a better picture of the state of the repo, we are closing all issues that we are carrying over from the ODFE era (ODFE is no longer supported/maintained, see post here).
If you believe this issue should still be considered for current versions of OpenSearch, apologies! Please let us know by re-opening it.
Thanks!


Dependencies might be missing

Duplicates

[X] I have searched the existing issues

Latest version

[ ] I have tested the latest version

Current behavior 😯
Wanted to use the FormatColorFillIcon from https://mui.com/material-ui/material-icons/?query=fill&selected=FormatColorFill
so i npm installed "@mui/icons-material" because the code to import is :
import FormatColorFillIcon from '@mui/icons-material/FormatColorFill';
While loading my react page on which I displayed the icon, I got these errors :
Module not found: Can't resolve '@emotion/react' in 'F:\dev\video-spinning-manager\node_modules\@mui\styled-engine'
Module not found: Can't resolve '@emotion/styled' in 'F:\dev\video-spinning-manager\node_modules\@mui\styled-engine'

Sure enough, when I installed these 2 libraries, it works.
But shoudn't npm i @mui/icons-material already install these ? If it needs them to work, I guess they should be dependencies on that library, shoudn't they ?
Why aren't they ?
Expected behavior 🤔
Not having to install another dependency on my project when it's supposed to be a dependency of one of my dependency.
Steps to reproduce 🕹
The commit that was needed to display an icon : https://github.com/pierreminiggio/video-spinning-manager/commit/0c34f2ac3fdbb973812a513854bf7959ca8f57e3
You could refer to my package.json for versions :
https://github.com/pierreminiggio/video-spinning-manager/blob/0c34f2ac3fdbb973812a513854bf7959ca8f57e3/package.json
Context 🔦
I'm just displaying a meterial UI Icon on a react project, and I don't expect having to install 3 dependencies just to do that.
Your environment 🌎

npx @mui/envinfo
  System:
    OS: Windows 10 10.0.19043
  Binaries:
    Node: 14.15.5 - F:\fichiers-et-logiciels\node-js\node.EXE
    Yarn: 1.22.5 - F:\dev\yarn\bin\yarn.CMD
    npm: 8.12.1 - F:\fichiers-et-logiciels\node-js\npm.CMD
  Browsers:
    Chrome: 102.0.5005.115
    Edge: Spartan (44.19041.1266.0), Chromium (103.0.1264.37)
  npmPackages:
    @emotion/react: ^11.9.3 => 11.9.3
    @emotion/styled: ^11.9.3 => 11.9.3
    @mui/base:  5.0.0-alpha.86
    @mui/icons-material: ^5.8.4 => 5.8.4
    @mui/material:  5.8.5
    @mui/private-theming:  5.8.4
    @mui/styled-engine:  5.8.0
    @mui/system:  5.8.5
    @mui/types:  7.1.4
    @mui/utils:  5.8.4
    @types/react: ^17.0.9 => 17.0.9
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    styled-components: ^5.3.0 => 5.3.0
    typescript: ^4.3.2 => 4.3.2

I just did a portfolio project to get some MUI experience and had the same problem. Would be interested to know why this is, or to help fix it if this turns out to be a bug.

Nextjs example with mui5. You can install entire thing and dont need to worry which packages are missing.

Nextjs example with mui5. You can install entire thing and dont need to worry which packages are missing.

That's not really the point.
The point is to not to have to do that. Our projects shoudn't have to deal with the npm dependencies or one of its npm dependencies.
My project shoudn't need to know that @mui/icons-material depends on both @emotion/react @emotion/styled, that shoudn't be my role as a dependency user to clean them up later if for some reason later down the road @mui/icons-material don't rely on these dependencies anymore.

@pierreminiggio , yes i understand what you mean initially. I encountered that problem every time i use MUI5. Its annoying and i just follow NextJS doc. I honestly dont mind if it takes little bit more storage for additional packages. For me its the fastest temporary fix!

The requirement for the installation of emotion packages is documented on our site (https://mui.com/material-ui/getting-started/installation/).
It's necessary because you may choose to install styled-components instead. This gives developers the freedom to choose a library they want to utilize.

The requirement for the installation of emotion packages is documented on our site (https://mui.com/material-ui/getting-started/installation/).

Okey I got it, it makes sense then. I guess to add the dependency requirements to the package, you'd need to build a package for each engine that installs mui + the engine, which might not be the greatest idea, idk.
But then i don't think it's that conventional to have a package requiring us to install another npm package before being able to use it.
I think installing '@mui/icons-material' alone would be better throwing us a link to the documentation to help us choose an engine rather than a missing dependency error. Installing a package and getting a missing dependency error as the first thing when installing is weird to me.
It would make the installing process more understandable if we discover the package via the code displayed in the icon list on the MUI website : (Example for me : https://mui.com/material-ui/material-icons/?query=fill&selected=FormatColorFill)


Convert Jaeger trace exporter to OTLP trace exporter

Our local Fermilab observability stack, Landscape, now has support for OpenTelemetry traces.  We should switch the tracing library to use an OpenTelemetry trace exporter, rather than a Jaeger one.
See https://opentelemetry.io/docs/languages/go/instrumentation/

Same as #96.  Closing.


[Wix] -> [HTML & CSS Templates]

Wix
Free Website Builder with the freedom to create the Websites you want.
Link: https://www.wix.com/
Checklist:

[x] I have performed a self-review of submitted resource and its follows the guidelines of the project.

Wix is a great tool however I don't see a free plan on the site.
It demands a premium version for some of the features like our own domain name. It's, however, free for creating and designing our site.

Also can you move it to the Online design tools section?
Okay, I'll do it.


Return only the observable

Wasn't totally sure what the subject was for here. This works as is with:
var runtimes = require('shinx')
runtimes.subscribe(console.log);

This is actually true! Sorry, this was more of a newb issue on my part.
Awwww yeah! Less lines of code!

Deletion is my favorite.


Meltano installation within docker container running airflow

I wanted to check if this has been tried before and if not what approach the community would suggest:
I have airflow running as docker containers and wanted to install meltano inside these containers so that most of my taps and targets would be available.
Any ideas around this ?

I am running meltano in docker(k8s) which is triggered via airflow KubernetesPodOperator. I did not want to have 2 airflows, one in meltano, so I manage meltano schedules with my primary airflow.

@halilduygulu  any tips on how you did this ( meltano in docker ) . I was able to install meltano in the container but end having pip version related issues when installing the taps or targets inside the container .

@abhi-quinyx were you able to sort this out? Meltano installs plugins in their own virtual environments which helps with the dependency mismatch you're experiencing. We've had some users install Meltano with Airflow separately but that's not our recommended path.

No luck yet @tayloramurphy . Here are the steps I followed :

run airflow (version - 2.1.4) locally through docker
ran a pip install meltano
meltano init to create a project
meltano discover extractors lists various extractors
meltano add extractor tap-salesforce results in the following error

2022-08-08T15:21:14.605023Z [warning  ] The `discovery.yml` manifest received from https://www.meltano.com/discovery.yml has version 20, while this version of Meltano requires version 19.
2022-08-08T15:21:14.606545Z [warning  ] Please install the latest compatible version of Meltano using `meltano upgrade`.
2022-08-08T15:21:14.608044Z [warning  ] Falling back on the cached `discovery.yml` manifest...
Added extractor 'tap-salesforce' to your Meltano project
Variant:	meltano (default)
Repository:	https://gitlab.com/meltano/tap-salesforce
Documentation:	https://hub.meltano.com/extractors/salesforce.html

Installing extractor 'tap-salesforce'...
Extractor 'tap-salesforce' could not be installed: failed to upgrade pip to the latest version.
Will not install to the user site because it will lack sys.path precedence to setuptools in /home/airflow/my_test_project/.meltano/extractors/tap-salesforce/venv/lib/python3.6/site-packages
You are using pip version 18.1, however version 21.3.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Failed to install plugin(s)

The error complains about the pip version from the location - /home/airflow/my_test_project/.meltano/extractors/tap-salesforce/venv/lib/python3.6/site-packages . The pip version for tap-salesforce is 18.1 while the pip under airflow container is - 21.3.1.
I think when you say

Meltano installs plugins in their own virtual environments which helps with the dependency mismatch you're experiencing

the above error should not occur , correct ?  I wonder what is wrong in the setup if that is the case.

@abhi-quinyx I'm guessing that whatever docker image you're using as a base may just have an old version of pip and/or meltano. I'm not seeing this on a clean install of meltano. @meltano/engineering any ideas?

@tayloramurphy @abhi-quinyx I see what's going on with the old pip version. Meltano only upgrades its own version of pip (i.e. the one in its venv) when the meltano install --clean flag is used. I think when running meltano add and the automatic subsequent installation, Meltano should also upgrade pip.
That's a quick fix, I think.

@edgarrmondragon  thanks and that's interesting to know . so is it a fix on my end ? or something which will be handled as a bug fix ?

@edgarrmondragon thanks and that's interesting to know . so is it a fix on my end , if yes how can I proceed ? or something which will be handled as a bug fix ?

@abhi-quinyx Yeah, I wasn't all that clear 😅. I see you're using Python 3.6 which is EOL and hasn't been supported by Meltano for some time. I think if you use a newer Python, it should also come with a newer pip and could fix your issue.

@edgarrmondragon  gave another try with python - 3.7.13 now . It now fails at pip install meltano 😓  , tries to install meltano-2.4.0 but complains in the end with
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /home/airflow/.local/lib/python3.7/site-packages (from requests<3.0.0,>=2.23.0->meltano) (1.26.9)
Requirement already satisfied: certifi>=2017.4.17 in /home/airflow/.local/lib/python3.7/site-packages (from requests<3.0.0,>=2.23.0->meltano) (2020.12.5)
Collecting ruamel.yaml.clib>=0.2.6
  Using cached ruamel.yaml.clib-0.2.6.tar.gz (180 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [3 lines of output]
      sys.argv ['/tmp/pip-install-9fndr2jc/ruamel-yaml-clib_0a2bf18c0237428cbad0a24e0890d837/setup.py', 'egg_info', '--egg-base', '/tmp/pip-pip-egg-info-rdfq06fs']
      test compiling /tmp/tmp_ruamel_edup5rpk/test_ruamel_yaml.c -> test_ruamel_yaml compile error: /tmp/tmp_ruamel_edup5rpk/test_ruamel_yaml.c
      Exception: command 'gcc' failed with exit status 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

all right I was able to fix the previous error by adding gcc in my docker image
RUN apt-get update
RUN apt-get -y install gcc

but now the same tap-salesforce fails with the same error. Meltano complains again about pip version issue.
airflow@ce2173db40a9:~/my_test_project$ meltano add extractor tap-salesforce
2022-08-09T08:27:31.018765Z [info     ] Environment 'dev' is active
Added extractor 'tap-salesforce' to your Meltano project
Variant:	meltano (default)
Repository:	https://gitlab.com/meltano/tap-salesforce
Documentation:	https://hub.meltano.com/extractors/tap-salesforce

Installing extractor 'tap-salesforce'...
Extractor 'tap-salesforce' could not be installed: failed to upgrade pip to the latest version.
ERROR: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.
WARNING: You are using pip version 22.0.4; however, version 22.2.2 is available.
You should consider upgrading via the '/home/airflow/my_test_project/.meltano/extractors/tap-salesforce/venv/bin/python -m pip install --upgrade pip' command.

Failed to install plugin(s)
airflow@ce2173db40a9:~/my_test_project$ /home/airflow/my_test_project/.meltano/extractors/tap-salesforce/venv/bin/python -m pip install --upgrade pip
ERROR: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.
WARNING: You are using pip version 22.0.4; however, version 22.2.2 is available.
You should consider upgrading via the '/home/airflow/my_test_project/.meltano/extractors/tap-salesforce/venv/bin/python -m pip install --upgrade pip' command.
airflow@ce2173db40a9:~/my_test_project$ python --version
Python 3.7.13
airflow@ce2173db40a9:~/my_test_project$ pip --version
pip 22.2.2 from /home/airflow/.local/lib/python3.7/site-packages/pip (python 3.7)

@edgarrmondragon  now I tried with python3.7 and meltano-2.40

Extractor 'tap-salesforce' could not be installed: failed to upgrade pip to the latest version.

@abhi-quinyx - Can you try with --log-level=DEBUG in case additional info can be found about why pip is not able to be updated?

@abhi-quinyx I think what would also help is a look at the Dockerfile. It seems that the base (Airflow?) image being used restricts virtualenvs somehow.


Change wladyslaw_of_gielniow_priest to ladislas_of_gielniow_priest

Is it ready to be merge?

Is it ready to be merge?

Yes, though I’ve just fixed an issue, but it is ready to merge. :smiley:

You often put 👍 everywhere and I'm not always sure about the meaning.
Does it mean here that you just read it, or you approve and we can merge it?

Ok let's do it. Thanks for this PR!

Oops I copy pasted my comment in the description of this commit, I'll have to change that.

Last commit amended. Situation fixed. 😸

You often put +1 everywhere and I'm not always sure about the meaning.
Does it mean here that you just read it, or you approve and we can merge it?

You’re right, it is ambiguous.
Mostly, I use them to say I’ve read the particular comment and have nothing to say. Sometimes it also means I agree with the comment. Sorry about that.
Do you have any requirement of the emojis? :smiley:

I don't know what's the best. Maybe avoid them when it's possibly ambiguous.

Okay, I’ll do that. :smiley:


Ender Pearls not correctly handling curse

Seems that Ender Pearls dont handle curse correctly when the pearl lands ontop of a block.
On the side it seems to work due to you being pushed through the block and therefore moving.
Might simply be that it doesnt trigger handleCurse for this reason

cant recreate the issue for me even if the pear land on top of the block the curse trigger

It did seem abit odd and inconsistent when i tried it. The check should also not fail
I'll double check and close the issue if I cant reproduce it consistently either.
Otherwise I'll give you a more detailed explanation. Feel free to pick another issue in the meantime 👍

ok all good


Upgrade kotlin gradle

Assoicated with #80

Upgrades Kotlin to 1.3.20
Upgrades Gradle to 5.3.1
Removes plugin java-library. (Issue fixed)

looks good for me, will check if publishing is not broken and then merge


Release 0.30.1

Description
Removes affinity property from RoleGroup

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

bors merge


test-listener-manager can fail if the kernel doesn't support MPTCP

Describe the bug
The Debian CI reported a errors when validating the unit tests for mptcpd 0.11: the test test-listener-manager fails with:
Unable to open MPTCP listener: Protocol not supported
test-listener-manager: test-listener-manager.c:106: test_listen: Assertion `mptcpd_lm_listen(_lm, sa) == 0' failed.

To Reproduce
I guess it can be reproduced using a non MPTCP kernel
Expected behavior
Like some other tests, the test should be skip if we notice the kernel doesn't support MPTCP.
Desktop (please complete the following information):

Linux Kernel: I don't know :-/ at some points, they were using v4.19 kernels
Compiler: gcc 12

Thanks! :)


Federate a private room from Element to Mattermost does not work

refers to feature added in https://github.com/mattermost/matrix-as-mm/issues/14
I followed the documentation to create a private room from Element and have Mattermost users in there.

Create the room without E2EE
Invite Mattermost users in Element and the mattermost connector user
Mattermost users look like it has accepted the invitation, but the connector does not look like it has accepted the invitation
As a result no room created for the Mattermost users in Mattermost and no messaging across platforms

@janostgren

This works in my environment !.
You must invite Mattermost users.
Channel will be created in MatrixRooms team.
[image: image.png]
Den ons 20 sep. 2023 kl 14:24 skrev Murat @.***>:

refers to feature added in #14
https://github.com/mattermost/matrix-as-mm/issues/14
I followed the documentation to create a private room from Element and
have Mattermost users in there.

Create the room without E2EE
Invite Mattermost users in Element and the mattermost connector user
Mattermost users look like it has accepted the invitation, but the
connector does not look like it has accepted the invitation
As a result no room created for the Mattermost users in Mattermost and
no messaging across platforms

@janostgren https://github.com/janostgren
—
Reply to this email directly, view it on GitHub
https://github.com/mattermost/matrix-as-mm/issues/20, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG7INMARSNZBC5Q2ZR3LPZLX3LN6TANCNFSM6AAAAAA472S3IA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok thank you I will review the test environment and see if we can update/fix the issue


Win & Linux importing.

So on windows the importing looks like this if i have the following directory structure:

- source/
  - app/
       -controllers/
            - a.js
            - b.js
            - c.js

import * as whatever from "app/**/*.js";
I receive an object that looks like this:
controllersa, controllersb, contorllersc ...
however on linux the output is
controllers$a, controllers$b, controllers$c, ...
I am not sure what is the reason behind this, but is there a way with your plugin to have the import mappings work such as
whatever.controllers.a
whatever.controllers.b
whatever.controllers.c

or something similar?

I receive an object that looks like this:
controllersa, controllersb, contorllersc ...
however on linux the output is
controllers$a, controllers$b, controllers$c, ...

I'd expect the behavior to be the same. We're not testing this on Windows though, so a good start would be to add AppVeyor. An example file is https://github.com/novemberborn/babel-plugin-files/blob/7803a37a7d094ab97c574570a697de1316c35c35/appveyor.yml.

s there a way with your plugin to have the import mappings work such as
whatever.controllers.a
whatever.controllers.b
whatever.controllers.c

or something similar?

The idea is that you should be able to import a specific file (import {controllers$a} from 'app/**/*.js'). Creating a nested object structure goes counter to that.
It's an interesting thought though. @lukescott what do you think?

It's unfortunate travis doesn't have Windows support. Adding Appveyor wouldn't hurt. I suspect it has something to do with \ vs /, although we use path.sep, so it's quite odd that $ is missing.
Nested objects: I typically avoid paths that give me $ anyway, so I don't have a preference on it. It could be an option. In fact it may even be easier to work with anyway. Gives you an option to target a specific folder. Generating the structure may be a bit tricky though.

Path sep is quite works odd on windows in my experience altought unix style path separators work just fine on windows.
I am using this module to load all my angular controllers into one file such as
Controllers/user/
profile.js
Password.js
Search.js
Etc..
So it generates bice controller names:
UserProfile.js however on linux i got User$Profile and had no idea whats causing the issue. Anyways a quick replace solved temporary but i think it shouls be fixed somewhere in the code or dependencies also would be really cool if we couls get a pretty directory structure object out from this plugin. (I just loved it how nicely its done and working :))


Update README.md

Adds newer functional React syntax

@googlebot I signed it!

Thank you @iRyanBell, these docs updates are a huge help!


Crashing on json::at

Hello, I'm trying to parse json objects sent from the client to my server. I completely undertsand that the client at times might not send valid json data in case of packet manipulation but I assume if I try catch json parse and json.at then I shouldn't have any issues. But it seems when I try to catch json::at get an out of range exception.
The client is sending non valid empty json like this
{}
This is the code for checking the valid json
    message[length] = 0;
    const std::string payload(message);

    json packet;
    try {
        packet = json::parse(payload);
    } catch (json::parse_error& e) {
        std::cerr << e.what() << std::endl;
        return ;
    }
    uint32_t packetid = 0;
    try {
        packetid = packet.at("id");
    } catch (std::exception& e) {
        std::cerr << e.what() << std::endl;
        return ;
    }
    std::cout << payload << std::endl;

When I send an invalid json the code should return but instead it just crashes.
This is where it seems to be crashing.
#1	0x000000010003bfb5 in std::__1::__throw_out_of_range(char const*) at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdexcept:262
this	std::__1::map<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, nlohmann::basic_json<std::__1::map, std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, bool, long long, unsigned long long, double, std::__1::allocator, nlohmann::adl_serializer>, std::__1::less<void>, std::__1::allocator<std::__1::pair<const std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, nlohmann::basic_json<std::__1::map, std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, bool, long long, unsigned long long, double, std::__1::allocator, nlohmann::adl_serializer> > > > *	0x000000010082ea10 size=0	0x000000010082ea10
    0x10003bf15 <+37>: mov    rax, qword ptr [rbp - 0x20]
    0x10003bf19 <+41>: cmp    qword ptr [rax], 0x0
    0x10003bf1d <+45>: jne    0x10003bf2f               ; <+63> at map:1544:40
    0x10003bf23 <+51>: lea    rdi, [rip + 0x16e620]     ; "map::at:  key not found"
    0x10003bf2a <+58>: call   0x10003bf50               ; std::__1::__throw_out_of_range at stdexcept:260
->  0x10003bf2f <+63>: mov    rax, qword ptr [rbp - 0x20]

I'm not sure if this is a bug with the source code or I'm doing something wrong but the way I see it, in case of an invalid json the code shouldn't continue. And if it does continue then catching std::exception should prevent the crash.

Does it work if you try to catch a json::out_of_range exception?

As far as I can tell, the code you work ought to work, as a json::out_of_range derives from json::exception which derives from std::exception.
It seems like the first part of your code directly catches a json::parse_error, and because you didn't mention that it doesn't work, I assume it does. So does the second try block work if you try to catch a json::out_of_range exception directly?
If so, you might have an issue where you have multiple definitions of std::exception floating about, and the two types (std::exception at your try block and the std::exception that the compiler finds when parsing exceptions.hpp) might be different?

Hello. Thank you for the reply. I can try catching out_of_range exception directly sure. I'm confused about what you mean right here though.

If so, you might have an issue where you have multiple definitions of std::exception floating about,

Is it not good to try catch std::exception in several different locations?

In order to asses your issue, we need the following information:

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

Which compiler and operating system are you using? Is it a supported compiler?

Did you use a released version of the library or the version from the develop branch?

In order to asses your issue, we need the following information:

Please describe the steps to reproduce the issue. Can you provide a small but working code example?
Which compiler and operating system are you using? Is it a supported compiler?
Did you use a released version of the library or the version from the develop branch?

I apologise, the crashing was my fault. It's not a bug in your code. This thread can be closed.

Thanks for checking back!

Hi IreliaNina,
I am also getting the same issue with no obvious problem in my code. Can you please share your experience how was it solved?
Thanks in advance


Outdated phar available from deployer.org

I've recently switched to using the official Deployer GitHub Action, but this broke my deployment. The action will download Deployer as a .phar from deployer.org, while our old CI setup would instead install it via Composer (composer global require "deployer/deployer:~7").
I could trace the error back to a missing feature of the magento2.php recipe, that was initially introduced with Deployer 7.3.0. The GitHub Action will install Deployer 7.4.0, but the recipe seems to be outdated.
Steps to reproduce:

Download https://deployer.org/releases/v7.4.0/deployer.phar
Download https://github.com/deployphp/deployer/releases/download/v7.4.0/deployer.phar
Extract both .phar files
Compare magento2.php recipe

Expected behavior:

Both .phar files should contain the correct magento2.php recipe

Actual behavior:

.phar loaded from deployer.org contains an old version of the recipe (likely from version 7.1.3)

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

It looks identical to the file from version 7.1.3.

curl -LO https://deployer.org/releases/v7.4.0/deployer.phar
chmod +x deployer.phar
deployer.phar --version
// Deployer 7.1.2

The v7.4.0 version is instead the v7.1.2


Replace the social image for Getting Started

Currently, our "Getting Started" image is a green cylinder, and it's what's getting shown on social media. We need to replace this with something better just for social media.

Crossman's ticket w/ Learndot: https://help.learndot.com/servicedesk/customer/portal/1/SUP-10893

cc @SpicyThrow

LearnDot Support said to edit this page https://university.cockroachlabs.com/admin/system_setting/view/185.html following the guidelines on this page https://css-tricks.com/essential-meta-tags-social-media/

I've updated the LearnDot html to fix this issue. Directions on how to make this change can now be found here: https://cockroachlabs.atlassian.net/wiki/spaces/PROD/pages/233144602/Updating+LearnDot+Social+Media+Images


Windows 10 / Exchange / Error 502

Got this error the whole day

Hi ErwSim
Thank you for letting us know.
We believe this to be as a result of partner throttling limits and are working towards resolving this problem.
I will let you know once we've managed to address this

Same here since yesterday - Mac + iOS (both in latest, app also latest) + also webapp - error 502 while Exchange

Yes, this release went out slightly earlier than the server changes. the alternative exchanges will go live in a few hours.

still receiving this error..

Problem still unsolved by March 28th 2021

We are still looking at improving these error messages. In most cases, it's caused by the exchange amount being too low or a restriction on the region you're accessing it from.

This issue has been resolved. as over v1.1.23 which included improved error handling. If anyone is still receiving these error messages Ill reopen the issue


(IAC-590) Add the capability to create a local storage class for use with the SAS Information Catalog and VI products

Closed by #9

Closed with coding in #9


CORE-19887 Re-add filtered dependency notary checks

THIS PR IS FOR 5.2.1

Duplicate of https://github.com/corda/corda-runtime-os/pull/6089


ci: do not report coverage on pull requests

The code coverage reports on pull requests has been inconsistent and
confusing. Disable them for pull-requests.

Oh well..
On the plus side, it will save 30 seconds of CI time per PR

On the plus side, it will save 30 seconds of CI time per PR

10 minutes actually...


godep save error: go/src/golang.org/x/crypto/acme" is not using a known version control system

What version of Go are you using (go version)?
$ go version
go version go1.13.4 darwin/amd64
$ godep version
godep v80 (darwin/amd64/go1.13.4)
Does this issue reproduce with the latest release?
 As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Add Material Design

I hesitate to make a new theme for pelican but, with just a variable I can adapt https://github.com/FezVrasta/bootstrap-material-design on this theme.

How can I use this theme? If I copy material.min.css to static/css and rename it bootstrap.material.min.css, the theme gets screwed, probably because relative urls don't work anymore.
Am I missing something?

Yes, you have to update base.html and add the variable. You can directly take a look to all files updated in this PR.
Enjoy :)

Hi!
First of all, thanks for your pull request! I appreciate the contribution, but I cannot accept this PR at the moment. It's with heavy heart that I have decided that I can no longer maintain this theme due to lack of time and changing interests. To makes sure the theme lives on however and not goes to waste, I've donated it to the Pelican community. It can now be found in the official pelican-themes repository. I'd suggest you re-submit your pull request there.
Sorry for the inconvenience!


merging rows

Hi,
We use Docparser's table grid for parsing an invoice.  Docparser uses tabula.pdf for its table grid.
We are seeing many examples where two rows are getting merged inadvertently.   The conditions are pretty similar in all cases.  If the last row was partially filled out, started from the middle of the page to the end of the page, it is getting merged with the row above.
It does not happen when I parse the file with spaces as separators.  Only when table grid is used, the merging happens.
It does not matter where I place the column boundaries.
See attached examples of PDF and their subsequent row formation in Docparser using tabula.pdf.  I have attached three PDFs as well.
GFS- VR0207801- $1,712.44.pdf
Unilever GFS Montreal_5188213_9011 08_ART.pdf
Cash_Wa_Invoice100280_np.pdf
Let me know what else can I provide.

Any updates?

We don't maintain Docparser. Please either submit your issue to Docparser and see if they think it's a Tabula bug or not, or test your document in Tabula and see if the issue appears there.


[Donor Command] /repair works on Ras' Golden Lens and Lures

Describe the bug
Running /repair or /repair all will repair Ras' Golden Lens and Lures in your hand/inventory.
To Reproduce
Steps to reproduce the behavior:

Hold a Lure or Ras' Golden Lens that's been used at least once.
Type /repair
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Username: Grandeflora
Server name (all applicable): Orion
Logs:
Let me know if needed.
Additional context
Not sure if this is a bug or a feature, so I thought I'd report it!

this has been solved, thank you for reporting!


fix: Storybook error "future version of React will block javascript: URLs"

Contributes to #3593
Fixes a console error in Storybook.
What did you change?
Updated a Storybook page:
from <SideNavLink href="javascript:void(0)" ... />
to <SideNavLink href="(valid url)" ... />
How did you test and verify your work?

[x] Verified error no longer appeared in browser console.
[ ] Verified by @lee-chase that I didn't break anything.

Closing. Mixed merge was including unrelated previous commits. Replaced by PR https://github.com/carbon-design-system/ibm-products/pull/3652


Remove lua::running()

Its only (remaining) caller is a function that is only called from Lua
anyway, so it does not really make sense to check if the Lua thread is
running.
Signed-off-by: Uli Schlachter<EMAIL_ADDRESS>

Updated so that this actually compiles and so that the RUNNING static is completely removed.
I rebased this onto latest master and verified that each individual commit passed cargo test.

This interacts badly with #479, which adds a new kind of error to send(). That would basically mean reverting the commit which removed all the error handling. Back to the drawing board!


How to change default icon

Unable to find proper documentation for changing the default icon, can any one help out ?

In order to customize icons you need empty and full properties. The former configures which icon/s to use when the rating is not selected while the latter configures the icon/s when the rating is selected (hover). These properties are so flexible that allows you to create really custom ratings.  Both properties accept  a react element, inline style object, or classes. Even an array of such symbols that will be applied in a circular manner (round-robin).
A couple of examples:

Using bootstrap glyphicons classes:
<Rating empty="glyphicon glyphicon-star-empty big-star" full="glyphicon glyphicon-star big-star" />

Using react elements:
 <Rating start={0} stop={4} empty={[1,2,3,4].map(n => {n})} full={[1,2,3,4].map(n => {n})} />

There are more examples in the demo page.
My apologies for the lack of proper documentation... It is something pending and a repeating concern several issues. There is even an open issue requesting to improve docs. I know how important is a good documentation but I have been lazy addressing this issue so far 😅.

Thanks @dreyescat , Present documentation looks fine, only suggestion is need some code examples on each prop
My usage of your rating component
<Rating empty="fa fa-star-o" full="fa fa-star" fractions={2} stop={10} initialRate={movie.vote_average}/>
How to add half stars and custom styles ?

You are on the right path. To have half stars you just need to set fractions property to 2. You can customize the stars as you would do to any fontawesome icon with CSS. You can create new CSS classes and just assign them to the empty and/or full properties.
In this example I increased the size and changed the color of the symbols:
.custom {
  font-size: 2em;
}
.custom-empty {
  color: blue;
}
.custom-full {
  color: red;
}

<Rating empty="fa fa-star-o custom custom-empty" full="fa fa-star custom custom-full" fractions={2} stop={10} initialRate={3.5}/>

Please mark this as closed

@anandrikka please close this issue I believe @dreyescat answered sufficiently. otherwise I can help

I am going to close it. Feel free to reopen if needed.

not working in 2019

SPFx.docx


Validate faulting addresses are valid to fault on

This commit adds a defense-in-depth measure to Wasmtime which is intended to mitigate the impact of CVEs such as GHSA-ff4p-7xrq-q5r8. Currently Wasmtime will catch SIGSEGV signals for WebAssembly code so long as the instruction which faulted is an allow-listed instruction (aka has a trap code listed for it). With the recent security issue, however, the problem was that a wasm guest could exploit a compiler bug to access memory outside of its sandbox. If the access was successful there's no real way to detect that, but if the access was unsuccessful then Wasmtime would happily swallow the SIGSEGV and report a nominal trap. To embedders, this might look like nothing is going awry.
The new strategy implemented here in this commit is to attempt to be more robust towards these sorts of failures. When a SIGSEGV is raised the faulting pc is recorded but additionally the address of the inaccessible location is also record. After the WebAssembly stack is unwound and control returns to Wasmtime which has access to a Store Wasmtime will now use this inaccessible faulting address to translate it to a wasm address. This process should be guaranteed to succeed as WebAssembly should only be able to access a well-defined region of memory for all linear memories in a Store.
If no linear memory in a Store could contain the faulting address, then Wasmtime now prints a scary message and aborts the process. The purpose of this is to catch these sorts of bugs, make them very loud errors, and hopefully mitigate impact. This would continue to not mitigate the impact of a guest successfully loading data outside of its sandbox, but if a guest was doing a sort of probing strategy trying to find valid addresses then any invalid access would turn into a process crash which would immediately be noticed by embedders.
While I was here I went ahead and additionally took a stab at #3120. Traps due to SIGSEGV will now report the size of linear memory and the address that was being accessed in addition to the bland "access out of bounds" error. While this is still somewhat bland in the context of a high level source language it's hopefully at least a little bit more actionable for some. I'll note though that this isn't a guaranteed contextual message since only the default configuration for Wasmtime generates SIGSEGV on out-of-bounds memory accesses. Dynamically bounds-checked configurations, for example, don't do this.
Testing-wise I unfortunately am not aware of a great way to test this. The closet equivalent would be something like an unsafe method Config::allow_wasm_sandbox_escape. In lieu of adding tests, though, I can confirm that during development the crashing messages works just fine as it took awhile on macOS to figure out where the faulting address was recorded in the exception information which meant I had lots of instances of recording an address of a trap not accessible from wasm.

A high-level question as well: a pagefault from within a hostcall won't trigger the warning, right? Or do we enter the handler if an instance in the store is on the stack at all?

Correct, yeah. We've actually got tests which ensure that a host-triggered segfault isn't caught by Wasmtime and aborts the process (as you'd expect a segfault in a language like Rust to do so). The first thing the signal handlers do is test whether the faulting pc is a wasm pc, and it defers to the default behavior of each signal if it's not a wasm pc. If it's a wasm pc though we fully catch the trap assuming it's expected that the wasm instruction had the kind of fault.

Ok good to know it's expected! I'll disable this particular test for s390x in that case.


Allow symlinks and directories as custom conf items

Also adds a test for config customization

Already checked by https://travis-ci.org/simleo/hadoop-docker/builds/509443732. Merging right away


Large Rec File Memory and Speed Issues

Hi all,
I just had a question about the usage of recordio files in mxnet. I have a large .rec file (~35 GB) on a system with 32GB of RAM. Whenever I try training a model, I see my memory consumption overflowing into swap memory, eating around as much RAM as the .rec file itself. Changing prefetch_buffer and preprocess_threads constants don't seem to change this. Is mxnet loading the entire rec file into memory? I saw this issue https://github.com/dmlc/mxnet/issues/1411 which is similar to my issue, but working off of mxnet's latest commit didn't solve.
Also curiously, I am still able to train albeit slowly, but running the validation set between epochs takes >3 hrs when each epoch is only about 1 hr. Is this normal?
Thanks so much for the help!

hi, i trained 103G .rec file, and the RAM using is less than 20G.
if using one GPU, for your big dataset, it takes > 3 hrs is normal.

I have the same problem when i train ssd or imagenet and i check the usage of memory in top. At the beginning of training is 5.7% , after a while (3~4h) it takes 27% and finally it run out of my memory (waked up by the alarm message)

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


Fix home-assistant/core image URL

Image URL changed in https://github.com/home-assistant/core/commit/80b357262795a57dc267a313064266fd2682ca74

Thanks for catching this :)


Improve alby fee reserve calculation

Related to https://github.com/getAlby/nostr-wallet-connect-next/pull/130 which uses a hardcoded 1000 sat amount.
This might not be enough to open a larger channel? the max outgoing payment of 1M sats * 0.8% + routing fees could be over 8000 sats.

This might not be needed if we can remove the fee reserve for payments from LNDHub to Alby's LSP node.

Update: we can just remove the fee reserve because it's an internal payment

It's still slightly off, leaving up to 1000 sats (in larger migrations) in the user's LNDHub account. Probably a rounding error somewhere.


Figure out a way to show scene background image under dungeon layer, but background tiles over dungeon layer

Right now both are in BackgroundLayer and at the same z index, with the background image being set as bgPath and added as a child at position 0.
Canvas' draw() method sets the scene bg image as the bgPath on the background later:
      // Temporary workaround until "Scene Levels" are developed
      this.background.bgPath = this.scene.data.img;
      this.foreground.bgPath = this.scene.data.foreground;

Implemented in 0.11.0 as a separately-configurable background image in the dungeon layer. I.e., set it in the dungeon config, don't set it in the scene.


feat(a11y): add scripts for reporting a11y violations

This PR adds a few new scripts to help with identifying accessibility issues with our components. These scripts handle creating a sitemap file, which is processed by the aXe accessibility engine, which then outputs a report that can be used to guide developers on how to solve the issues being reported.
This introduces a nominal number of new dependencies, which are:

axe-core
axe-webdriverjs
selenium-webdriver

How to use:

npm run dev
In another terminal npm run a11y
Notice feedback produced by reporter

We'll soon want the ability for this to run as part of a CI job, this can be completed as part of follow up work. For now, the main thing is getting this tool into our hands so we can begin chipping away at the existing problems we have.
Should help close https://github.com/patternfly/patternfly-next/issues/466
add sitemap generator script
don't track generated files
add an audit runner
add basic reporters
add script for running as part of ci

Deploy preview for pf-next ready!
Built with commit a32f1ed88322bf436881ffa472a0e317087673da
https://deploy-preview-517--pf-next.netlify.com


transmission_4: 4.0.4 -> 4.0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/transmission/versions.
meta.description for transmission_4 is: A fast, easy and free BitTorrent client
meta.homepage for transmission_4 is: http://www.transmissionbt.com/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.0.5 with grep in /nix/store/6axqmr8if0ric66ab932m7jl79d7azlp-transmission-4.0.5
found 4.0.5 in filename of file in /nix/store/6axqmr8if0ric66ab932m7jl79d7azlp-transmission-4.0.5

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
libtransmission_4
transmission_4
transmission_4-gtk
transmission_4-qt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6axqmr8if0ric66ab932m7jl79d7azlp-transmission-4.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/342c7949e93c2b730ed9d6c244351391b3fe3412#transmission_4

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6axqmr8if0ric66ab932m7jl79d7azlp-transmission-4.0.5
ls -la /nix/store/6axqmr8if0ric66ab932m7jl79d7azlp-transmission-4.0.5/bin

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

4 packages built:

libtransmission_4
transmission_4
transmission_4-gtk
transmission_4-qt

Maintainer pings
cc @astsmtl for testing.

@astsmtl What's the issue with x86_64-darwin ?

transmission_4 has been failing on x86_64-darwin ever since this build (according to hydra-check): https://hydra.nixos.org/build/241182189 . Let's try to fix it in https://github.com/NixOS/nixpkgs/pull/271650 .


Verify Inquiry Emails

Hi Jeremy,
Emily suggested that I email you about an issue I am having with a family resource worker that inquires about children, but I only get some of her inquiries. I do not really know how to address the issue and I have told the woman that if she does not hear right back from me regarding an inquiry to email me directly, but here is an example:
A family inquired about Jeremiah Leathers the other day and when I went into his file to answer the inquiry I saw that there was an inquiry from a social worker that had not been checked off. I am pretty good about checking off my inquiries when I complete them, so I went into the inquiry and saw that it was from Caroline, so I went back in my email to see if I had sent her information on Jeremiah and I had not nor could I find a formal inquiry email from her for him. I do not recollect which other inquiries this has happened with for her, but it does seem to just be her.
If this makes any sense and you have any ideas, great; otherwise, it does not seem to be creating any major issues and is just something I wanted to mention. Thank you.

@jeremysMARE Investigate via Mandrill app dashboard.
https://mandrillapp.com/activity

This has been resolved through refactors of the inquiry process which now produce error notification when inquiries fail.


[core] Failed to decompile try-catch-finally - Try blocks wrapping queue limit reached

Decompiling this sample fails with an error in BlockProcessor pass: "Try blocks wrapping queue limit reached".
I tested with latest git revision 0f00fb9 and openjdk8. Same result with java-input and java-convert.
public class TestTryCatch1 {

  public void doSomething(int i) {
    try {
      doSomething1();

      if (i == -12) {
        return;
      }

      if (i > 10) {
        doSomething2();
      } else if (i == -1) {
        doSomething3();
      }
    } catch (Exception ex) {
      logError();
    } finally {
      doSomething4();
    }
  }

  private void logError() {
  }

  private void doSomething1() {
  }

  private void doSomething2() {
  }

  private void doSomething3() {
  }

  private void doSomething4() {
  }
}

This sample can be used in unit tests.

@nitram84 fixed, although it is not general enough so may break something else :smiling_face_with_tear:

@skylot Thank you a lot for fixing this issue. Usually my examples are taken from real code. I minimized the code to the root and obfuscated it - every change to my example makes it decompilable. Think of a complex method with 100 or 200 lines that can now be decompiled by this fix. I think the best way to break large apps with large methods is to report and fix all the smaller problems.

I think the best way to break large apps with large methods is to report and fix all the smaller problems.

You are right and this is the only way! Thank you for your work :+1:


Logic error of UIView's extension function: cornerRadius(corners: UIRectCorner, radius: CGFloat)

if corners.contains(.bottomLeft) {
                    cornerMask.update(with: .layerMinXMinYCorner)
                }
                if corners.contains(.bottomRight) {
                    cornerMask.update(with: .layerMaxXMinYCorner)
                }
                if corners.contains(.topLeft) {
                    cornerMask.update(with: .layerMinXMaxYCorner)
                }
                if corners.contains(.topRight) {
                    cornerMask.update(with: .layerMaxXMaxYCorner)
                }

Should be:
if corners.contains(.bottomLeft) {
                    cornerMask.update(with: .layerMinXMaxYCorner)
                }
                if corners.contains(.bottomRight) {
                    cornerMask.update(with: .layerMaxXMaxYCorner)
                }
                if corners.contains(.topLeft) {
                    cornerMask.update(with: .layerMinXMinYCorner)
                }
                if corners.contains(.topRight) {
                    cornerMask.update(with: .layerMaxXMinYCorner)
                }

in iOS 10:

in iOS 11:

Code:
cornerRadius(corners: [.topLeft, .topRight, .bottomRight], radius: 16)

Thanks! Why not open a pull request? 😄
Just a question, they changed the Y asix on iOS 11?

Haha.. My source control software having some issue on push code to GitHub..
I don't think they changed the Y axis stuffs on iOS 11 due to coordinate x: 0, y: 0 is top-left corner. And in your original code you state .bottomLeft is minimum X and minimum Y, so it's not correct obviously.

Ahah ok I will fix it.
I don't understand your previous comment, both the images are generated by the same code cornerRadius(corners: [.topLeft, .topRight, .bottomRight], radius: 16)?
Or is a "How is now" and "How has to be"?

@kokitang News about that issue?


add authors page

So I am trying to add a more prominent author information section into the post page. But I guess it's useful to implement #114 as a first step. Currently, data/authors only has a display name field, but I am planning to add more things to it in the future.
However, there is something to discuss:
I personally feel that rather than keeping using the author field, it would be best to use an authors field, which takes a list of authors as its value. Because this would match the convention of tags and categories, and it's also potentially possible that an article might have multiple authors. However, there is already a singular author field in the past. So simply removing it might cause some compatibility issues. On the other hand, if we support both fields, it would be a little bit cumbersome. Not sure what is the best strategy here.

Thanks for your great work!
To my opinion, keep it as simple as before is better, that keeping using the author field.
Multiple authors is extra feature. If someone need this feature and authors field is necessary, we could add a description document.

Okay. The current implementation is to use the author field.


feat(services/vercel_blob): support vercel blob

fixes #2191

@Xuanwo blob fails to pass the test of test_write_with_special_chars, after debugging, i feel that it is a vercel error. Maybe you can help me check it

@Xuanwo blob fails to pass the test of test_write_with_special_chars, after debugging, i feel that it is a vercel error. Maybe you can help me check it

Not found in docs. Maybe we can just ignore this test for vercel blob.

Sorry for the mistake

Why does seafile container not start?

Seems seafile pushed a new tag serval hours ago:

Maybe we can change the tag in docker-compose file to use the old one until they addressed. Would you like to submit a new PR for this?

Fixed in https://github.com/apache/opendal/pull/4107, please try again,.


⚠ Samsung has degraded performance

In f8d3d3c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2566 ms

Resolved: Samsung performance has improved in 76e002d.


Review3: Schemas and PropTypes

Overview
The focus for this code review will be centered around structure and organization of schemas and propTypes in the project. For better re-usability and maintenance, we should consider storing all our schemas and propTypes in one location.
Please pay attention to:

Code organization.
File structure.

Review Branch
review-70
Files to review

PropTypes.js,
DBSchemas.js.
FormSchemas.js

Checklists

Architecture
Design
JS
React
Testing

Due date
12, February 11:55pm
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

DBSchemas.js
JS-01:
allowedValues in the type should be [Charitable, Church/Religious, Private Foundations, Political Organization, Other]
'mission' can be omitted
'location' can be named as address of the organization to avoid confusion
add contactPerson with contactInfo
PropTypes.js
JS-01:
'mission' can be omitted
'location' can be named as address of the organization to avoid confusion
add contactPerson with contactInfo

/propTypes
AR-01: /propTypes directory should be written /prop-types.
DBSchema.js
The Collection should be split into 2: Organization & Individual.
JS-01: type should be updated with new values or removed.
PropTypes.js
JS-01: Update the propType in accordance with DBSchema.js

PropTypes.js
JS-01: What does 'location' refer to: the organization's or event's location?
DBSchemas.js
JS-01:

What does 'location' refer to? Add a better descriptive name

L6 what's the difference between 'Business' and 'Organization'?

JS-08: Add some comments to avoid confusion

PropTypes.js
JS-01:
Mission should be clarified. Nonprofits may have multiple missions. Is it a description of the nonprofit?
DBSchemas.js
JS-01:
What is the difference between Business and Organization?
Mission should be clarified (same as above).
Should an Individual have a mission?


All components in the loading state should be unclickable and disabled

TuSimple/naive-ui version (版本)
2.22.0
Vue version (Vue 版本)
3.2.26
Browser and its version (浏览器及其版本)
chrome
System and its version (系统及其版本)
window
Node version (Node 版本)
16
Reappearance link (重现链接)
https://codesandbox.io/api/v1/sandboxes/define
Reappearance steps (重现步骤)
1.The loading state of the switch component is still clickable
2.The style in loading state should be is disabled style
Expected results (期望的结果)
All components in the loading state should be unclickable and disabled  ,
The style in loading state should be is disabled style
Actual results (实际的结果)
1.The loading state of the switch component is still clickable
2.The style in loading state should be is disabled style
Remarks (补充说明)

At present, the switch can still be clicked in the loading state. I think all components should be consistent

select loading也是吧

@07akioni @Talljack  Yes, and I think that in loading state, it should be in the same style as disabled

select loading也是吧

Don't change select now. It needs further consideration.

select loading也是吧

Don't change select now. It needs further consideration.
okay

@07akioni  @Talljack  是否考虑先把loading的css样式改为disabled的样式

@07akioni @Talljack 是否考虑先把loading的css样式改为disabled的样式

我们需要先对一下哪些组件需要修改，在考虑修改方案

@07akioni @Talljack 是否考虑先把loading的css样式改为disabled的样式

这个不能一蹴而就，要循序渐进，一点一点来

@07akioni @Talljack 是否考虑先把loading的css样式改为disabled的样式

我们需要先对一下哪些组件需要修改，在考虑修改方案 @07akioni

Switch 是合理的，改了没问题


Fix gaussian update

Fixes #198 , big thanks to @bvdmitri and @bartvanerp

Codecov Report

Merging #203 (f4005e9) into master (c96dac5) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   87.14%   87.12%   -0.03%
==========================================
  Files          97       97
  Lines        4614     4613       -1
==========================================
- Hits         4021     4019       -2
- Misses        593      594       +1

Impacted Files
Coverage Δ

src/engines/julia/update_rules/gaussian_moments.jl
92.00% <100.00%> (ø)

src/engines/julia/update_rules/delta_conjugate.jl
68.69% <0.00%> (-0.87%)
:arrow_down:

src/ForneyLab.jl
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Readme update

Responding to feedback posted in #4 and doing an overall read through and edit.

@iceLearn can you check over these changes.
We could also include the updated image.


Add persistent connection support

It uses pfsocketopen instead of fsocketopen. Did some initial testing (with hundreds of thousands of jobs and lots of clients) and socket reuse worked great.

This looks pretty good to me.
I assume it would be fully backwards compatible, since a constructor flag is required to opt-in.

Thanks for this feature and congratulations...

I'm glad someone's persistent connection work finally got merged, way to go @sraka1 :)

Not merged yet.

Let's see if I can fix that today…

Merged, with a couple of extra commits:

77c94716a04385be767f0980fc6c2192315404c6 as per https://github.com/pda/pheanstalk/pull/90#discussion_r26013943
1427066c72b7e51047ee5c8146ebfec87f5971ac to get minimal test coverage on the persistent connection code.


[NL] Add some Dutch sentences for climate

Just some simple few sentences to start with. The naming conventions weren't very clear to me, may need some work.

Hi Berend,
Besides the sentences themselves, you also need to add tests for the sentences.
It was a bit of a puzzle for me as well, but I managed to get my first PR merged earlier today. You can have a look at the files changed in my PR to see what needs to be added/changed
See: https://github.com/home-assistant/intents/pull/64#event-8097226710

Hi Berend,
Besides the sentences themselves, you also need to add tests for the sentences.
It was a bit of a puzzle for me as well, but I managed to get my first PR merged earlier today. You can have a look at the files changed in my PR to see what needs to be added/changed
See: https://github.com/home-assistant/intents/pull/64

Hi Fes,
Thanks for your mention 😊 will look into it!

Looks like we just need some more test sentences for HassClimateGetTemperature

Looks like we just need some more test sentences for HassClimateGetTemperature

Incoming :)


Finer grained permission for writing commit status

Feature Description
Currently, to generate a token that can create commit statuses via the API, it must have write:repository.
Proposal: create a new permissions for commit statuses (read, and read+write) and update the API to only require commit statuses privileges to read/write commit status. Additionally, for backwards compatibility and sanity leave the repository permission as sufficient for commit statuses as well.
If this idea is acceptable, I'd be happy to implement it myself.

Exactly. And probably a read:commit_status although I don't know of a great use for that without read:repository access (which would still be sufficient for reading commit statuses)... thoughts?

Yup, the "read" permission for a commit status seems quite strange, no idea whether it should really be introduced (while it is also strange not introducing it .....)

Overall I think your idea is pretty good (acceptable to me). The real challenge is that how to implement it (the token scope system is complex 🤣 )

Ok cool, I'll start digging into the code in the next few weeks and see what I can come up with. Thanks!

For reference GitHub as a read-only commit statuses option:

https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28#get-the-combined-status-for-a-specific-reference


Shuffleboard-metadata widgets are added to tab even if they already exist

Hi, I'm testing Elastic with code-generated layouts, and I'm seeing that every time I restart sim, all the widgets are added again to the tab, so that I have several sets of widgets all piling up in the corner grid square. I'm not using size or position metadata, just PreferredComponent. Is this intentional?

I'm a bit confused, do they keep adding even if they already exist on the tab or is the issue that they're not going to any position? I discovered an issue recently that's similar to this and I've patched it for the next update, but could you send me the code so I can try to reproduce this?
With the positioning, it doesn't automatically find a free space to place the widget, currently it just puts it at 0, 0 which is the top left corner. I've been thinking of ways to fix it but at the moment it doesn't really do anything.


Rename library and client to use new oid4vc name

Double check that this is the appropriate new name.

Rename to "oid4-client" instead to cover oid4*. There's a possibility that oid4* will become OIDI (I = issuance) and OIDP (P = presentation), but we'll just have to decide what to do about that later if it comes to pass.

Addressed by #6.


foundry: add chain id 31337 while forking chain

Description
Addresses issues #524
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Related Issues
_Closes #524
Your ENS/address: jrcarlos2000.eth

Tysm @jrcarlos2000 🙌 !!

"fork": "anvil --fork-url ${0:-mainnet} --chain-id 31337 --config-out localhost.json",

${0:-mainnet} will take the name of the network you declare in foundry.toml and use it for the fork. you can also pass an RPC_URL directly.

I was wondering if there a direct way to setup to this defult in foundry.toml file ?

anvil wiill override the value of the chain id specified in toml to the chain id of the RPC specified for the fork. for now the only way of using 31337 would be pass --chain-id @technophile-04

got it thanks Carlos merging this 🙌


Server isn't automatically started when Sublime is already open

Hi, I'm using Sublime Text 3.2.2 (build 3211) on Arch Linux. Here's the output of uname -a:
Linux mars 5.3.13-arch1-1 #1 SMP PREEMPT Sun, 24 Nov 2019 10:15:50 +0000 x86_64 GNU/Linux

I'm using LSP v.0.9.0 and I installed it through Package Control.
Ever since I started using this plugin I've experienced a little issue while using my language servers.
Basically, if I have Sublime Text open with no projects in it and then I decide to load in a project, the language server associated with that project won't start automatically. In order to get the server running I have to open a project, then close down Sublime, and then reopen it again. Thanks to Sublime's hot exit feature the project will already be there and the server will be started as intended.
Same thing happens if I have a project already open and then decide to switch to another one: if a project has a language server running and then I switch to another one via the Quick Switch Project feature, the server for the newly-open project won't start as well. However, if I then go back to the first project - always via Quick Switch Project - the server will still be there running.
So basically, also as a recap, intuitively it seems like LSP doesn't shut down the current language server and starts the other one when switching to a new project, neither does it start the language server when opening a project in an already-running Sublime Text instance.
I recorded a video that hopefully should clarify what the issue is.
For reference, this is my LSP configuration:
{
  "auto_show_diagnostics_panel_level": 2,
  "log_debug": true,
  "log_stderr": true,
  "show_diagnostics_count_in_view_status": true,
  "clients": {
    "rls":
    {
      "command": ["rls"],
      "scopes": ["source.rust"],
      "syntaxes": [
        "Packages/Rust/Rust.sublime-syntax",
        "Packages/Rust Enhanced/RustEnhanced.sublime-syntax"
      ],
      "languageId": "rust"
    },
    "clangd":
    {
      "command": [
        "clangd",
        "--background-index",
        "--compile-commands-dir=build",
        "--header-insertion=iwyu",
        "--pch-storage=memory",
        "-j=6"
      ],
      "languages": [{
        "scopes": ["source.c"],
        "syntaxes": ["Packages/C++/C.sublime-syntax"],
        "languageId": "c"
      },{
        "scopes": ["source.c++"],
        "syntaxes": ["Packages/C++/C++.sublime-syntax"],
        "languageId": "cpp"
      },{
        "scopes": ["source.objc"],
        "syntaxes": ["Packages/Objective-C/Objective-C.sublime-syntax"],
        "languageId": "objective-c"
      },{
        "scopes": ["source.objc++"],
        "syntaxes": ["Packages/Objective-C/Objective-C++.sublime-syntax"],
        "languageId": "objective-cpp"
      },
    ]
    }
  }
}

and here are the .sublime-project files for the projects I used in the video:
// Rust project
{
    "folders":
    [
        {
            "path": "."
        }
    ],
    "settings":
    {
        "LSP":
        {
            "rls":
            {
                "enabled": true
            }
        }
    }
}

// C project
{
    "folders":
    [
        {
            "file_include_patterns":
            [
                "compile_commands.json"
            ],
            "folder_exclude_patterns":
            [
                "*"
            ],
            "path": "build"
        },
        {
            "folder_exclude_patterns":
            [
                "build*"
            ],
            "path": "."
        }
    ],
    "settings":
    {
        "LSP":
        {
            "clangd":
            {
                "enabled": true
            }
        }
    }
}

With "log_debug" enabled in my LSP preferences, I opened the console on a new instance of Sublime Text (with no projects open) and these are the only entries related to LSP:
LSP: global configs ['ra-lsp=False', 'phpls=False', 'typescript-language-server=False', 'haskell-ide-engine=False', 'eslint=False', 'reason=False', 'flow=False', 'metals=False', 'bashls=False', 'rls=False', 'gopls=False', 'vscode-css=False', 'ocaml=False', 'javascript-typescript-langserver=False', 'clangd=False', 'lsp-tsserver=False', 'pyls=False', 'jdtls=False', 'golsp=False', 'dart=False', 'intelephense-ls=False', 'polymer-ide=False', 'cquery=False', 'ruby=False']
LSP: window 2 starting 1 initial views

When I open the Rust project (with the same steps as shown in the video) these are the only lines of output added to the console:
git: tracking working dir /home/gianluca/Projects/rust-sandbox

and this is what appears on the console when I switch to the C project via Quick Switch:
git: untracking working dir /home/gianluca/Projects/rust-sandbox

The C project isn't a Git repo so that's why the same messages don't appear for it as well.
So basically nothing seems to be happening, as far as LSP can tell.
If I try to restart Sublime Text with the project already loaded in, as mentioned earlier, this is what I get on the console:
LSP: global configs ['polymer-ide=False', 'eslint=False', 'cquery=False', 'jdtls=False', 'vscode-css=False', 'ra-lsp=False', 'haskell-ide-engine=False', 'reason=False', 'lsp-tsserver=False', 'javascript-typescript-langserver=False', 'flow=False', 'ocaml=False', 'phpls=False', 'ruby=False', 'clangd=False', 'gopls=False', 'rls=False', 'golsp=False', 'intelephense-ls=False', 'typescript-language-server=False', 'bashls=False', 'metals=False', 'pyls=False', 'dart=False']
LSP: window 2 has override for clangd {'enabled': True}
LSP: window 2 starting 1 initial views
LSP: window 2 requests clangd for /home/gianluca/Projects/hash-test/src/main.c
LSP: starting in /home/gianluca/Projects/hash-test/build
LSP: starting ['clangd', '--background-index', '--compile-commands-dir=build', '--header-insertion=iwyu', '--pch-storage=memory', '-j=6']
LSP:  --> initialize
LSP: window 2 added session clangd
I[02:48:13.842] clangd version 9.0.0 (tags/RELEASE_900/final):
I[02:48:13.842] Working directory: /home/gianluca/Projects/hash-test/build:
I[02:48:13.842] argv[0]: clangd:
I[02:48:13.842] argv[1]: --background-index:
I[02:48:13.842] argv[2]: --compile-commands-dir=build:
I[02:48:13.842] argv[3]: --header-insertion=iwyu:
I[02:48:13.842] argv[4]: --pch-storage=memory:
I[02:48:13.842] argv[5]: -j=6:
Path specified by --compile-commands-dir does not exist. The argument will be ignored.:
I[02:48:13.842] Starting LSP over stdin/stdout:
I[02:48:13.843] <-- initialize(1):
I[02:48:13.843] --> reply:initialize(1) 0 ms:
LSP:  --> initialized
I[02:48:13.843] <-- initialized:
I[02:48:13.843] unhandled notification initialized:
LSP:  --> textDocument/didOpen
I[02:48:13.845] <-- textDocument/didOpen:
I[02:48:13.846] Failed to find compilation database for /home/gianluca/Projects/hash-test/src/main.c:
I[02:48:13.846] Updating file /home/gianluca/Projects/hash-test/src/main.c with command clangd fallback:
[/home/gianluca/Projects/hash-test/src]:
/usr/bin/clang /home/gianluca/Projects/hash-test/src/main.c -fsyntax-only -resource-dir=/usr/lib/clang/9.0.0:
I[02:48:13.912] --> textDocument/publishDiagnostics:
LSP: <--  textDocument/publishDiagnostics

and now it works as intended with no issues.
I tried to make this as comprehensive as I could. Let me know if there's any other thing I can do to try and figure out what's going on :slightly_smiling_face:

Thanks for the detailed report! The issue is unique to Quick Switch Project and having servers enabled per project - we are missing the logic to update the project overrides on an existing window after switching.
It's probably a good time to finish pull request #708 as it will help fix this bug.
In the meantime, having servers globally enabled will make Quick Switch work.

I understand. Thanks for the quick reply and for the alternative solution until this is fixed! :)


luci-app-vnstat: Add ability to set datadir

Once pending pull request for vnstat is accepted in packages
we will have uci option to set the data directory, which is
exposed here.
Signed-off-by: Daniel Dickinson<EMAIL_ADDRESS>Depends on https://github.com/openwrt/packages/pull/3815

For personal reasons no longer participating in OpenWrt/LEDE


navbar-fixed-top causes font styling changes in Safari 8

http://jsbin.com/sehozemoda/3/
It seems like any JS action in a navbar-fixed-top causes font changes to occur. Does not happen in Chrome.

Also, this is found with base Bootstrap downloaded today...

Duplicate of #15199.


FR: Add highlights to the signcolumn / scrollbar

Only an aesthetic thing, but would it be possible to extend the highlights to the side? To the left with the sign column (and to the right, if you have a scrollbar plugin).
Without extending the highlight there, the resulting "gaps" in the background coloring look somewhat off.

While I see why it could be interpreted as "off", I prefer the current highlighting. I don't want things that are not text in the buffer to be highlighted as if they were.

This can, of course, just be implemented as an option.

This can, of course, just be implemented as an option.

That someone needs to implement and maintain.
Sorry, but I have no interest in supporting this.


feat: add type hints

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our commit message format.

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

The project does not currently have type hints.
Issue Number: N/A
What is the new behavior?

This PR adds type hints, supporting classes, and changes to CI so we can run mypy and ignition-api-stubs against the code.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

This PR depends in an unreleased version of ignition-api-stubs


Talk with user input

Hi,
I added again option in valid cheking process for fixes these issues: npm/npm/issues/4700, npm/npm/issues/5044. After that I'll write a patch for init-package-json like this: patch.diff. I hope this modification make npm init more useful.
Thanks.

@isaacs could you please any thoughts?

Seems work without this PR. thanks!