rac menu: unexpected placement behavior

Provide a general summary of the issue here
I cannot get correct placement for hamburger menu placed as bottom left corner of screen using RAC menu in first of 5 cases, where trigger button is nearest bottom left corner, and there is an animation effect being applied in other cases even though no animations have been declared.
Note: There is plenty of room for the menu to appear in all cases, so this is not (or should not) be a case of menu auto-repositioning to avoid overflowing off of screen.
Menu implemented as follows:
import {MenuTrigger, Button, Popover, Menu, Item} from 'react-aria-components';

export function HMenu() {
  return (<MenuTrigger>
  <Button aria-label="Menu">☰</Button>
  <Popover placement="right bottom">
    <Menu onAction={(id) => console.log(id)}>
      <Item id="open">Open</Item>
      <Item id="rename">Rename…</Item>
      <Item id="duplicate">Duplicate</Item>
      <Item id="share">Share…</Item>
      <Item id="delete">Delete…</Item>
    </Menu>
  </Popover>
</MenuTrigger>)
}

<img width="899" alt="Screenshot 2023-10-02 at 6 51 05 PM" src="https://github.com/adobe/react-spectrum/assets/3402123/c5a602e1-7ae2-4c51-9
https://github.com/adobe/react-spectrum/assets/3402123/fad13ad5-a7a4-462f-9fed-be4b41651eaa
e1b-7e5580a30838">
🤔 Expected Behavior?
When menu appears, expect to see left edge of popup aligned to right of trigger button, and bottom aligned with bottom of button, in all cases provided (see repro /video) and w/o animation.
😯 Current Behavior
Varying behavior depending on location of trigger button.
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
Repo here: https://github.com/rolanday/rac-menu-placement

Clone
Install deps
npm run dev
Click menus, observe behaviors

Version
1.0.0-beta.1 (RAC)
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
macOS (Sonoma, v14.0)
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Seems like for the first case, the maxHeight is getting set to 0px which is what is causing its weird placement. We might need to revisit our calculations for getMaxHeight. Our current stories for useOverlayPosition seem to test when position is relative, maybe would be good to add when position is absolute.
As for the animation, that is definitely weird and will need to investigate that more. I also made a codesandbox very similar to yours which also demonstrates the animation you’re seeing as well. I thought it could have been related to Tailwind but seems that’s not the case.

@yihuiliao , appreciate your taking a look at this. FYI, I used absolute for repro only for ease of showing the various placement cases. My actual  impl uses flex-box and align to achieve similar placement to the second case (bottom left placement). Can also confirm not related to Tailwind (I had previously ruled that out, also suspecting may be related. Thanks again!

Hi @devongovett , @yihuiliao , checking in on this since I'm at a point where will need to finish hooks implementation instead of using RAC if not resolved soon. I saw RAC 1.0 may be soon, and this should be P0 to fix IMO since placing hamburger menu at bottom corner is common design pattern.
Thanks!
-Roland.

@rolanday We ran into something similar for our SubMenu implementation in React Spectrum and are tracking those two issues via https://github.com/orgs/adobe/projects/19/views/20?pane=issue&itemId=43393367 and https://github.com/orgs/adobe/projects/19/views/20?pane=issue&itemId=45014178. Unfortunately these fixes will not make it for the RAC 1.0 release but we hope to pick them up soon in our next sprint after the holiday break.

@LFDanLu , thanks for letting me know. Much appreciated.

@rolanday I believe this has been fixed by https://github.com/adobe/react-spectrum/pull/5660. I verified locally in your example repo with the 1.0.2-nightly.4419. Feel free to try that nightly yourself or when this fix goes out with the next release (roughly at/near the start of next month).  Closing as resolved but feel free to reopen if something still breaks.

Hi @LFDanLu , thanks for the follow-up! I'm still seeing the issue on my end, but will wait for the next release and, if I still see the issue with it, then will work on providing an alternate repro. I should add that the "top *" placements work as expected (1.0.2-nightly.4419 and previous builds), but all other placements result in popover shrinking in heigh (w/ scrollbar) so stay in view instead of clipping (as opposed to adjusting placement).

but all other placements result in popover shrinking in heigh (w/ scrollbar) so stay in view instead of clipping (as opposed to adjusting placement)

Just to make sure, you are saying the popovers are adjusting their max height instead of just being cut off by the boundaries of the screen? Not entirely certain what is meant by clipping here, do you happen to have a screenshot to share?


Watermelon

Problem Link - https://www.codechef.com/problems/WATMELON

Please verify the problem and assign it to me. I will do it in c++.

I can do this in C

@gh4abhi Complete your all previous tasks.

I first write the code and after that ask for contribution. I'm only required to make PR for these issue.

@Nivedita967 i would like to contribute in python

@Nivedita967 I have completed my previous tasks now please assign this task to me.

Open for Java.

@Abhichal18 Are you working on this ?

@Abhichal18 Are you working on this ?

yeah i am


Setting up MTA-STS for legalservices.gov.uk

Background

We are setting up MTA-STS policy for all out email domains. However we do not manage the legalservices.gov.uk email domain. This is managed by the LAA Team from another AWS account.
We need to figure out how the MTA-STS policy can be implemented for that domain.
Contact LAA to identify someone to carry out changes on behalf of the OPG Team.
Approach

TBC
Questions / Assumptions

LAA will need to manage their own policy file and DNS.
Acceptance Criteria

TBD
Reference
How to write good user stories

Need to check with LAA is this is actually used for email. There do not appear to be any associated MX records.

Appears this domain is not used for email. Therefore no policy required.


Move aria label to match design system

The primary nav in the header (from the design system) has the aria-label on the UL, not the nav. This makes the two consistent.

 Move aria label in nav to match design system


sending empty params when using jsonEditor: true

When using jsonEditor: true, and the validation schema has some optional params swagger will always send all empty fields as empty (""), this will cause a 400 error.
Example:
validate: {
        payload: Joi.object({
          firstName: Joi.string().min(1).max(255).required().description('Person first Name').example('Jennifer'),
          lastName: Joi.string().min(1).max(255).required().description('Person first Name').example('Lawrence'),
          age: Joi.number(),
          address: Joi.object({
            street: Joi.string(),
            city: Joi.string(),
            zipCode: Joi.string()
          })
        }).label('Person creation schema')

in this example only firstName and lastName are required, using jsonEditor: false, I can send only the required params and don't even send the empty ones, but with jsonEditor: true, all params will be sent as empty, and this will cause a 400 error because the other params when enabled cannot be empty.
ex:
{
  "firstName": "Jennifer",
  "lastName": "Lawrence",
}

vs
{
  "firstName": "Jennifer",
  "lastName": "Lawrence",
  "age": 0,
  "address": {
    "street": "",
    "city": "",
    "zipCode": ""
  }
}

Is there a way to either:
-disable jsonEditor per route?
or to
-avoid sending empty params and instead just don't send them?

+1 this is really annoying

Anyone found a workaround?

Adding a note that https://github.com/jdorn/json-editor/ is no longer maintained (which this project uses). Development has moved to https://github.com/json-editor/json-editor.

Closing https://github.com/glennjones/hapi-swagger/issues/539#issuecomment-496004576


More unit tests for HMC building blocks

[ ] metrics
[ ] abstraction related to NUTS

More unit tests to resolve the ESS issue (https://github.com/TuringLang/Turing.jl/issues/840).
Some standard unit tests that we should try:

compare AHMC and DHMC on a set of hand-coded target distributions (e.g. https://github.com/TuringLang/Turing.jl/issues/840). This allows us to rule out potential interactions with Bisectors, VarInfo.

compare ESS of HMC configurations with mass matrix adaptation, step-size adaptation, u-turn independently, one at a time. This would allow us to rule out interactions between step-size, preconditioning and u-turn termination.

compare AHMC and DHMC with vanilla HMC with manual step-size and no preconditioning. This allows us to rule out potential bugs with leapfrog integrator, MH correction etc.


No erase button for recordings in Record activity

Currently there is no way to delete a picture,video or audio recording in the record activity.

Why not? But the UI should be simple and usable with a mouse or on a touch screen.

Fixed in https://github.com/llaske/sugarizer/pull/345

Closed in https://github.com/llaske/sugarizer/commit/dd3c58d56981e4dfa6d94601b95d7117a3bc9241


Azure Search service search on two fileds with special characters

Hi
I want to search on two fileds with deifferent search. It should search work for partial search and full text search. When my search text has special characters like "-".It is not working properly
Ex : I want below functionality in Azure search
SkuNumber = '2333IPTS1USD-01' & SkuDecription =  Setup&Cnfig Srvc'
SkuNumber = '%SL1%' & SkuDecription = '%Setup%'

@AbhishekAdusumilli Please create a new question on Stack Overflow and tag it with "azure-search". The Azure Search team monitors that tag and automatically assigns engineers to answer questions there.
I'm going to close this issue now since it's a question about the Azure Search service itself, not the .NET SDK per se.


better plugins documentation

i refer mainly on the upload plugin (the others are self-explanatory). there's nowhere any documentation to understand what data can be passed on upload success?
it seems that - by default - you should pass only message (success/custom error(?)) and url called file, but i really don't understand the function getDeep...
PS - what is the purpose of upload.success? it doesn't seems to pass any parameter to possible user-defined function
-- great work by the way! --

response from server must be an json ["success": 0||1, "file" : "path/to/file" ]

I've just updated upload plugin, so I don't take the time to write a correct documentation. Don't care about getDeep.
That's the basic usage from https://github.com/Alex-D/Trumbowyg/blob/gh-pages/js/main.js#L5 (used by demo page).
$.extend(true, $.trumbowyg.upload, {
    serverPath: 'https://api.imgur.com/3/image',
    fileFieldName: 'image',
    headers: {'Authorization': 'Client-ID 9e57cb1c4791cea'},
    urlPropertyName: 'data.link'
});

Here data.link is the property of your JSON response from server which contains the image path.
In this case, server response means look like that:
{
    "data": {
        "link": "http://domain.com/example.jpg"
    },
    "message": "Error or success message"
}

fileFieldName is the name of field which contains the file. Upload plugin use it to build AJAX request. In the above case, imgur server wait for a request with image attribute which contains image data.
I hope it's help you. I need to write a better doc than that, but I need time.

I want to write a plugin which pops a modal with a text field and and inserts whatever text is entered into the current position of the editor after wrapping the text with configurable styles. I have two questions:

Which of the included plugins could be a base something like this?
How do get my plugin to show up on the menu.

The upload file is only for php and in absence of any documentation of the plugin, I am unable to run this for asp.net. I have created a .vb file, and the image gets uploaded correctly, but the JSON that I return is probably not playing it correctly.
Can someone please guide me with this.
Thanks

Soon online with the 2.8.0 :)


soryBySelector is not sorting entries on the Front end.

I have setup a replica indice in Algolia dashboard as per what has been written on the algolia documentation.

On seeing the filter/sort in the frontend I go to click filter on Price high or price low and nothing happens, I have even checked the console and nothing is happening on the algolia api calls.

https://viaduct.frb.io/products/bedroom-furniture
What I would expect to see is the items filtering

OS: mac
Browser chrome/safari/firefox
Version latest

Here is my code for the functionality:
		var searchProducts = instantsearch({
		  // Replace with your own values
		  appId: 'NHF40KMICV',
		  apiKey: '91615faeab7c9c7e761f22279d5d1513', // search only API key, no ADMIN key
		  indexName: 'www.viaduct.co.uk_allProducts',
		  urlSync: false,
		  searchParameters: {
    		hitsPerPage: 500
  		  }
		});

	// types.level2.title

	if( $('#types').length ){
		searchProducts.addWidget(
		  instantsearch.widgets.refinementList({
		    container: '#types',
		    attributeName: 'types.level2.title',
		    operator: 'or'
		  })
		);
	}
	if( $('#selectedFilters').length ){
		searchProducts.addWidget(
		  instantsearch.widgets.currentRefinedValues({
		    container: '#selectedFilters',
		    onlyListedAttributes: true,
			attributes: [{name: 'types', label: 'Types'},{name: 'types.level2.title', label: 'Type'},{name: 'productPrice', label: 'Price'},{name: 'material', label: 'Material'}],
		    clearAll: 'after'
		  })
		);
	}
	if( $('#price-refinement').length ){
		searchProducts.addWidget(
			instantsearch.widgets.sortBySelector({
			    container: '#price-refinement',
			    indices: [
			    	{ name: 'www.viaduct.co.uk_allProducts', label: 'Price' },
			      	{name: 'products_by_price_low', label: 'Price lowest'},
			      	{name: 'products_by_price_high', label: 'Price highest'}
			    ],
			    label: 'Price'
			  })
		);
	}

I can see requests being done, is the sort by different in the dashboard if you go to the different indices?

Hmm really @Haroenv

Would it be possible to make a minimal reproduction in CodeSandbox and give access to your index for Algolia employees on your dashboard? Will be useful in finding what's going on here.

Would it be possibly because the value we are sorting by is a string not an integer?

Unfortunately there's no way to change that in the dashboard :(

@Haroenv so I've changed the numbers to be integers like so:

I've added the correct attribute to sortBy in the Ranking for the main indice but in the replicas it does not seem to order/reorder by price here

The js code has not changed at all from above and it still doesn't work:
https://viaduct.frb.io/products/bedroom-furniture

So looking at the requests it seems like they are actually completely correct (different results), but it doesn't update visually. What does your hit code look like? I guess it's custom.

@mdunbavan any update? Thanks :)


request for turning option name vacuum back to entity

Hello, instead of providing the vacuum name can we go back to providing the entity name of the map itself.
The getMapEntityName makes some assumptions about the naming scheme that aren't true by default (at least not for me).
I posted the function below which is the culprit.
getMapEntityName(vacuum_name) {     return "camera." + vacuum_name + "_map_data";   }
My map entity was just called camera.map_data so the card didn't display anything after the update. I got it working after changing the entity name but I only figured this out after looking at the source code which isn't the end of the world but it shouldn't be necessary.

The assumption IS true by default for recent Home Assistant versions, as written in the release notes.
But yes getting the option back to override this was requested by multiple people.

ah so home assistant and their breaking changes strike again. Thank you for letting me know. Maybe you can make it so that it uses either vacuum or entity, I do not know if that is a lot of work.

I'd like to have this option as well.
My map entity is also still named camera.map_data, after updating the card (and vacuum).

I did those steps. Deleted the vacuum from Home Assistant.
I see the vacuum and map re-appear when turning off/on mqtt on the vacuum.
It always comes back as camera.map_data.
As a workaround, I manually changed the entity as @Klabost wrote, that seems to work, but it still would be nice to have this option added back again.

I also followed the instructions. I also tried:

Remove device in Homeassistant
Turn off MQTT on Vacuum
Restart Homeassistant
Turn on MQTT on Vacuum

The camera entity appears again as camera.map_data

And you are both certain you are running the latest version of Valetudo? Because if so I really cannot explain that behaviour.
Anyway, as with all requested features, I may get to it eventually but if you want it faster send in a pull request.

I do run:
Valetudo on my Roborock S5: 2022.01.0

Map integration: v2022.01.2

I just repeated the steps again. Same result. My versions:
Valetudo

Homeassistant:

Valetudo Map Card:

@eifinger You need to to upgrade your vacuum to Valetudo 2022.01.0 first. The newest Map card addon does not work with Valetudo versions older than 2022.01.0 anymore. See releasenotes https://github.com/TheLastProject/lovelace-valetudo-map-card/releases/tag/v2022.01.0

@wimb0 Thank you for pointing it out.
Here is why I didn't realize it:

HACS notified me that the valetudo map card has a new release
I installed the new release and saw the instructions.
I logged into my Valetuo and did click the button to check for new releases, I installed it and was under the impression that I am under the newest release. I didn't realize it only upgraded from 2021.11.0 to 2021.11.1
Still under the impression I am on the latest release I came here
I now iterated through the updater to get to the latest release, repeated all steps and can confirm it is now working

Good it's working for you at least.
It's really ironic how that went, given Valetudo was changed to only update 1 release at a time because... people kept not reading the release notes properly and complaining about things being broken when they skipped releases 😓

I understand the dilemma and am sorry I piled onto that as well.
When I have an idea on how the update process can be handled "more user friendly" I will share it 😉


[FP]: CVE-2024-29133 matches commons-configuration:1.8, but affected versions are >= 2.0, < 2.10.1

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:apache:commons_configuration:1.8:*:*:*:*:*:*:*
CVE
CVE-2024-29133
ODC Integration
{"label"=>"Gradle Plugin"}
ODC Version
9.0.7
Description
Same as #6555

This is a duplicate, same as https://github.com/jeremylong/DependencyCheck/issues/6555#issuecomment-2029666371 - upstream OSSIndex data problem.


Will you be releasing the knowledge base training script?

Hi
This is exciting. There's a chance we'll use it at CNN. I'm interested in the knowledge base training, however. We'd want to be able to retrain on occasion, even if it takes a few weeks on 8 gpus as you mentioned. Can you provide more insight into whether you'll open those scripts?

@petulla Yes, we plan to update with instructions on how to train the model as well.

Cool, just for my sake, is that something you estimate you'll release in 1 week, 1 month or 1 year in terms of order of magnitude?

@petulla We should be able to update that in the next couple of weeks.


QR Code Types

I would like to ask on how QR Code types are generated?
Like this site: https://www.the-qrcode-generator.com/
They classify QR Codes as Free Text, URL, Contact, Phone, and so on.
How can we specify the type of QR Code that is being generated by this library?
Thanks for the answer!

There is no type. The website just uses text templates to gather the information to fill into url, vcard, tel:url, sms:url to generate the final text that will will be fed to the QRcode creator.


Rename Siphash::as_u64 to Siphash::to_u64

Resolves #3114

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.216%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hashes/src/siphash24.rs
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
19683

Relevant Lines:
23653

💛  - Coveralls

Just flagging that we may want to look at all the breaking changes we have made right before we release and re-add deprecated things back in. We really really do not want a repeat of the rust-bitcoin v0.29 to v0.30 upgrade hell that we created.
However, ACK 5a91719755f3d57ac64a3c44d2285e11a8245d78

Yeah, this one we probably should've just required a deprecation before approving. It was an obscure enough corner of the library that I let it slide, but it's not like a deprecation would've been difficult.


how to connect to local dynamo DB in docker compose?

Hi,
it can connect to separate dynamoDB, while when all the service and DynamoDB in one docker compose, then local seems not work,
maybe need to update some logic update fixed localhost  when region is local?

can not connect to dynamoDB from docker-compose:
{
  credentials: {
    accessKeyId: "DynamoDBLocal",
    secretAccessKey: "DoesNotDoAnyAuth",
    sessionToken: "preferTemporaryCredentials"
  },
  region: "local",
  host: "dynamodb",
  port: 8001,
  cache: {
    _credentialScope: "",
    _signingKey: null,
    _accessKeyId: "",
    _sessionToken: "",
    refresh: [AsyncFunction: refresh],
    signingKey: null,
    credentialScope: "",
    accessKeyId: "",
    sessionToken: ""
  },
  method: "POST",
  endpoint: "http://dynamodb:8001/"
}

got error:
error: Uncaught Http: error sending request for url (http://dynamodb:8001/): error trying to connect: tcp connect error: Connection refused (os error 111)
    at unwrapResponse ($deno$/ops/dispatch_json.ts:43:11)
    at Object.sendAsync ($deno$/ops/dispatch_json.ts:98:10)
    at async fetch ($deno$/web/fetch.ts:591:27)

it works without docker-compose,
the only difference I made is, in docker-compose, i use dynamodb, and use localhost without docker-compose,


--values and --set options are absent for install?

Hi,
Environment:

Ubuntu 18.04
Gravity 6.1.15
Platform GCP(but using generic flag)

The documentation mentions that I can use --values and --set during the install process
unpacked-image$ ./gravity install --values=custom-values.yaml --set=nginx.image=1.9.1 --set=postgres.registry=internal.registry.io

But it gives me an error to use it.
sudo ./gravity install --advertise-addr=<IP_ADDRESS> --token=abcd --cloud-provider=generic --config=/home/rabraham/tmp/fr/fr-config.yaml --values=/home/rabraham/tmp/fr/custom-values.yaml

[ERROR]: unknown long flag '--values'

Look in the help for install, I don't see those options? Is there any other way to pass this info?
root@on-prem-host:~/fifteenrock# ./gravity install --help
usage: gravity install [<flags>] [<path>]

Install cluster image on this node.

Flags:
      --help                   Show context-sensitive help (also try --help-long and --help-man).
      --debug                  Enable debug mode.
  -q, --quiet                  Suppress any extra output to stdout.
      --insecure               Skip TLS verification.
      --state-dir=STATE-DIR    Gravity local state directory.
      --log-file="/var/log/gravity-install.log"
                               Path to the log file with diagnostic information.
      --advertise-addr=ADVERTISE-ADDR
                               IP address this node will advertise to other cluster nodes. Must be present on the node. Will be auto-selected if not specified.
      --token=TOKEN            Unique install token to authorize other nodes to join the cluster. Generated automatically if unspecified.
      --cloud-provider=CLOUD-PROVIDER
                               Cloud provider integration: [generic aws gce]. Auto-detected if not set.
      --cluster=CLUSTER        Cluster name. Will be auto-generated if not specified.
      --flavor=FLAVOR          Cluster image flavor to install.
      --role=ROLE              Role of this node.
      --config=CONFIG          Kubernetes configuration resources, will be injected at cluster creation time.
      --wizard                 Start installation using web wizard interface.
      --mount=MOUNT            One or several mount overrides in the following format: <mount-name>:<path>, e.g. data:/var/lib/data.
      --pod-network-cidr="<IP_ADDRESS>/16"
                               Subnet range for Kubernetes pods network. Must be a minimum of /16.
      --service-cidr="<IP_ADDRESS>/16"
                               Subnet range for Kubernetes service networ.
      --vxlan-port=8472        Custom overlay network port.
      --dns-listen-addr=<IP_ADDRESS> ...
                               Custom listen address for in-cluster DNS.
      --dns-port=53            Custom listen port for in-cluster DNS.
      --storage-driver=STORAGE-DRIVER
                               Docker storage driver. Overrides the one specified in the cluster image manifest. Recognized are: devicemapper, overlay, overlay2.
      --docker-opt=DOCKER-OPT ...
                               Additional arguments to Docker. Can be specified multiple times.
      --service-uid="1000"     Service user ID for Planet container. "planet" user will created and used if none specified.
      --service-gid="1000"     Service group ID for Planet container. "planet" group will created and used if none specified.
      --gce-node-tag=GCE-NODE-TAG ...
                               Override node tag on the instance in GCE required for load balanacing. Defaults to the cluster name.
      --dns-zone=DNS-ZONE ...  Specify an upstream server for the given zone within the cluster. Accepts <zone>/<nameserver> format where <nameserver> can be either <ip> or
                               <ip>:<port>. Can be specified multiple times.
      --remote                 Do not use this node in the cluster.

Args:
  [<path>]  Path to the directory with the unpacked cluster image. Defaults to the current directory.

Hi @RAbraham, yeah, this feature will be a part of the upcoming 7.0 release, sorry about the confusion.
In 6.x versions what you could do is use the --config parameter to pass arbitrary Kubernetes resources which will be created in the cluster. So you could create, say, a ConfigMap with your custom values, pass it via --config parameter and have your install hook mount the ConfigMap and consume values from it.
Let me know if you have any questions.


Have SafeTest produce test items in a stable order.

Passing through set caused each run to have a different ordering of personas.

Makes sense to me. Is this worth writing a test for as well? Normally I'd say yes, so we have an explicit expression of the expectation. But maybe the assert's enough here?

Adding some rough testing.


Warn developer when using un-registered hook

Feature Request
Warn developers when they are attempting to hook onto an event that the app/installation is not subscribed to.
I don't remember where I saw this, but I believe there is an endpoint that returns all the events an installation (app) is subscribed to.
Is your feature request related to a problem? Please describe.
This would warn users in situations like #969 #933 #857 and any other related issue below 857 (i'm not gonna search all day :smile: ).
Example
app.on('issues.opened', async context => {
  const issueComment = context.issue({ body: 'Thanks for opening this issue!' })
  return context.github.issues.createComment(issueComment)
})
// logged to console:
// [WARN]: Installation is not subscribed to `issues` event.
//       : See <link here> on how to subscribe to 'issues.opened' event.

Yeah I wish that existed, but I don't know how it would work (efficiently). It will require an extra request, maybe we could add a --debug flag to add additional checks?

At best, this requires one additional request as the application starts up. I think it should be on by default.
And of course this feature should be disabled in production mode or with the passing of an environment variable (similar to DISABLE_STATS).

ok, I think if you or someone else wants to start a PR to discuss over code, that'd be great

The integrations API includes endpoints to view the events/permissions of an installation.
Sadly, there are no "free" or guaranteed endpoints that Probot could access to get the current installations' permissions & events.

Shouldn't that one work?
https://developer.github.com/v3/apps/#response-1
GET /app responds with an array of events the app is subscribed to


Package appmetrics-dash into NodeRED Bluemix Starter

@knolleary @tobespc Here's the PR for adding Appmetrics Dash into the Node-RED Bluemix starter. It can be enabled by setting the variable useAppmetrics in bluemix-settings.js to true, or by setting an environment variable NODE_RED_USE_APPMETRICS to true. Appmetrics-dash will attach to NodeRED's web server instance, and the dashboard will be on hostname:port/appmetrics-dash. Happy to write docs!
Signed-off-by: MattColegate<EMAIL_ADDRESS>

@mattcolegate thanks for this.  I'll need to port this change over to the node-red-bluemix repository later today and get the boilerplates updated in the catalog.
As discussed, I'll look at added a promo for it in the start-up screens.


d2 Interactive can't achieve, it seem just like a picture

when i use d2 playground, i hover the mouse on it and i got the Tooltips

but this plugin seem like just convert code to svg, but can't interactive.

Thanks for raising this. I was unaware of this interactive functionality in D2.
I am able to replicate your issue. It appears to be because the SVG is converted to base64 and then inserted as the source for an image element.

I shall look at creating a fix to use the SVG element itself instead of the base64 image.

This issue should now be resolved with the latest release (1.0.3). Working on the example site as shown below.


Members of ES6 classes not showing in documentation

As an example: http://pixijs.download/v4.3.0/docs/index.html and more specifically, http://pixijs.download/v4.3.0/docs/PIXI.DisplayObject.html
If you check the source code of that class via the inbuilt docs link, you can see in it's constructor that it declares member variables such as alpha, visible and renderable. But these do not show up in the generated documentation, only functions do (which makes this issue different from https://github.com/jsdoc3/jsdoc/issues/1293 I believe). Even if you add the --private flag for jsdoc, then private class functions show up in the documentation, but again, still no class member variables.

I have this same issue! been struggling with it for HOURS. I have a class that's defined inside of a module. And No matter WHAT I do, I can't get the memebers of my class to show up in the documentation.

Found a work around. Look at my last comment here
https://github.com/jsdoc3/jsdoc/issues/1293

@Native-Coder your workaround is to specify @memberof explicitly on every member.  This does already in work in PIXI docs, for example take a look at InteractionManager.moveWhenInside.
This is issue is about not having to specify it every time, as the properties should be automatically included the same way the methods are.

@Native-Coder your workaround is to specify @memberof explicitly on every member.  This does already in work in PIXI docs, for example take a look at InteractionManager.moveWhenInside.
This is issue is about not having to specify it every time, as the properties should be automatically included the same way the methods are.

in 3.4.3:

class methods are showing up in the global namespace
they will show up in the class namespace IFF export default is removed

This sounds like a duplicate of #1293, which is now fixed on GitHub master. The fix will be released in JSDoc 3.5.0. If you're still able to reproduce this with the current GitHub master, please provide a code sample that demonstrates the problem, and I'll reopen the issue.

@hegemonic, is there donation thing for this project?  it's not much, but i'd like to slap a few bucks in this project's direction.  i searched the README and issues looking for prior discussion

@cdaringe, no, there's no way to donate to JSDoc, but it's very kind of you to offer! I really appreciate your support.


Usage inconsistency

Usage is inconsistent between synopsis and examples. In the synopsis, <CACHE_LOCATION> precedes commands such as --clear, but in the examples it comes after. Both formats work (at least as of 3.0), but some might find this confusing.

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

ID: a5725b81-7ebf-e9c5-51b7-f6d1cf5d5dc2
Version Independent ID: a69419b1-6994-3310-6bde-d787b87db13f
Content: dotnet nuget locals command - .NET Core CLI
Content Source: docs/core/tools/dotnet-nuget-locals.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @karann-msft
Microsoft Alias: mairaw

Thank you for opening this issue. I'll add it to the backlog. You can always submit the fix yourself by editing the original article. Click on the Content Source link at the bottom of your original comment. To learn how to edit, see the Editing files in a repository article from GitHub.
Thanks again!


test

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Codecov Report

Merging #1718 (2e18b87) into master (4c17460) will decrease coverage by 25.01%.
The diff coverage is 30.50%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##           master    #1718       +/-   ##
===========================================
- Coverage   74.33%   49.33%   -25.01%
===========================================
  Files         296      451      +155
  Lines       36086    74181    +38095
===========================================
+ Hits        26825    36596     +9771
- Misses       7539    34692    +27153
- Partials     1722     2893     +1171

Impacted Files
Coverage Δ

master/disk_manager.go
42.22% <0.00%> (ø)

metanode/inode.go
0.00% <0.00%> (ø)

metanode/manager.go
0.00% <0.00%> (ø)

metanode/manager_op.go
0.00% <0.00%> (ø)

metanode/metanode.go
0.00% <0.00%> (ø)

metanode/metrics.go
0.00% <0.00%> (ø)

metanode/partition.go
0.00% <0.00%> (ø)

metanode/partition_free_list.go
0.00% <0.00%> (ø)

metanode/partition_store.go
0.00% <0.00%> (ø)

metanode/server.go
0.00% <0.00%> (ø)

... and 177 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add Type Hints for Python Package

Following https://github.com/dmlc/xgboost/pull/7707, I plan to resolve most mypy errors across the python package if not all.
We should probably populate compat.py with stubs for even more packages, such as cupy.

Sorry I wasn't able to follow up recently... Can I ask what cat_codes is? For example, https://github.com/dmlc/xgboost/blob/master/python-package/xgboost/core.py#L1151

@bridgream

Sorry I wasn't able to follow up recently... Can I ask what cat_codes is? For example

No worries, mypy needs to mature and we are not rushing into it.  cat code is the encoded categorical feature from cudf. Let's just mark it as a list for now.

Hi, @bridgream could you please resolve the conflicts?

Hi @trivialfis , could you please restart the Travis CI job? It looks the job failed at random.

Please ignore it for now.  It's SSH policy is changed.

Rebased master. There are 15 mypy errors remaining but they all look a bit complex to tackle.

@bridgream Hi, do you think it's ready to be merged?

Hi @trivialfis , thanks for the commits! Yes, I believe we're ready.

Great work!


Update basic.py to decrese line count

Number of total lines decreased 1

Closing as a duplicate of https://github.com/zserge/tinylangs/pull/4/files, but thanks for the contribution!


Guide for running Gardener on Minikube

The following scenario is supported - running the Gardener control plane components in Minikube and running the vagrant-provider on the host machine for creation of Shoot clusters locally. This setup simplifies the initial deployment setup.
@vasu1124 @rolpat can you test this setup on your machines?

@mvladev have approved, will wait for approval of @vasu1124 or @rolpat before final merge.

ping @vasu1124 @rolpat, could you please give feedback so that we can go ahead with this PR?

Thanks @vasu1124. @mvladev, squash please.

Squashed and changed gardener version to 0.2.0-f15944037e1784bada87bd4bff51b5a334b3b066

@mvladev want to merge, but you need to rebase on master, please.

Rebased


[BUG] Bot disconnects itself before playing the song

Describe the bug
When the user uses the /play command the bot disconnects itself before playing the song
To Reproduce
Steps to reproduce the behaviour:

Start the bot
Use the /play command
See error

Expected behaviour
The bot plays the song normally
Actual behaviour
The bot says that it was disconnected and in the console there is this warning: (node:243730) [DeprecationWarning] Warning: useMasterPlayer() hook is deprecated, use useMainPlayer() instead.

Hey! Thanks for the report, I'll look into this. Could you please provide me with some more information? The following will help me to understand this issue:

How you're hosting the bot (on your PC, with a hosting provider, etc.)
If you're using your PC, what operating system do you use? (Windows, mac, etc.)
The song that you are trying to play

Thanks :)

Sure! Here's the information you asked:

The bot it's honest on my Oracle vm
The os is Oracle Linux 8
I've tried many things, the latest 2 are: Mustard - Ballin’ ft. Roddy Ricch (YouTube) and EARFQUAKE (Spotify)

Both of those songs seem to work on my instance, which would suggest that it's operating system related. I've never used Oracle Linux, nor do I really have a way to deploy it to try and fix this issue. I am currently working on rewriting the entire Melody bot however, which may mean that once the new version is finished this issue may just disappear. For now though, I'm going to close this issue. If anyone who stumbles across this would like to contribute to fix this issue, feel free to reopen this issue, or make a new one. Apologies 😕


Rewrite merge script and stuff

Now with much more sparkle!  The only thing I didn't do was rewrite to use jq, because that seemed non-trivial at first glance (because of how we're using json), so I'm putting it off until I have more time.
Fixes #24 #7 #8 #22

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


feat(embedded-SDK): enable CSV download

SUMMARY

this will enable csv download with this fix #20261
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

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

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

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

Be great if this can get into the 2.0.0 release along with #20261

I think the change has not been published to npm, the lib/index.js file in 0.1.0-alpha.7 version only have 4  iframe.sandbox.add  in mountIframe which not include allow-forms and allow-popups, will it be published soon after?

Oh, I found use @preset-sdk/embedded is OK. = =


Verbose information during the build process

Hi Experts
I am new to Apollo and I would like to know how to extract the detail build commands for each object.  I searched a lot of related documents. However, I didn't find the solution.
In other words, I would like to know the linked library, the compiling option, the optimization flags, etc. during the compiling process for each object (Like the verbose information in GCC/Clang).
Anyone know how to extract this information. Many Thanks

It is auto generated by bazel, you can find the compile command in xx.params file. for example
bazel-bin/modules/control/control_component_test-2.params


Add auraBAL

Now that CowSwap routing for auraBAL has been fully integrated, this PR adds auraBAL to the token list.

I have read the CLA Document and I hereby sign the CLA

recheck


adding gemma dates model

Maybe you could also add some flags, like:
* `IF(EXTRACT(DAYOFWEEK FROM date) NOT IN (1,7), true, false) AS is_weekday`

I believe it would become even clearer like this EXTRACT(DAYOFWEEK FROM date) NOT IN (1,7) AS is_weekday

One more thing: Would it make sense to change the id column from a serial to the format yyyymmdd?
Rationale: Then the primary key is identical for the same date irrespective of start dates, which can then more easily be changed. On the flipside, you then can't just take deltas between ids to get number of days between dates. wdyt?

Thank you all for the comments!
@soltanianalytics I do like the idea of having a primary key, which is identical for the same date irrespective of start dates. As for getting the number of days between dates, I assume they could also be calculated from the dates themselves. :/


🌐 Add Chinese translation for docs/zh/docs/tutorial/static-files.md

Here also a same PR #3828, but it was two years ago, and it seems that the translator has not been to GitHub for a long time.

很好！Well done！

很好！Well done！

兄弟, 像Lenclove那样approved一下

Awesome, thank you @wdh99! 🚀
And thanks for the reviews @Lenclove and @ChoyeonChern 👏


Alexa stops responding a few minutes after startup

After a couple of minutes, Alexa stops responding.  It works for the first couple of minutes, and I'm able to ask questions and receive responses, but then it stops.  This happens regardless of whether I ask any questions.
At first, at failure time, I was receiving the following log:
src/hostapi/alsa/pa_linux_alsa.c :3636 :
PaAlsaStreamComponent_BeginPolling: assertion « ret == self->nfds » failed.

I then recompiled portaudio/avs-device-sdk with NDEBUG to turn assertions off, now I'm getting this:
2018-02-24 15:48:28.689 [ 15] I AbstractKeywordDetector:readFromStreamFailed:reason=readerTimeOut
2018-02-24 15:48:29.690 [ 15] I AbstractKeywordDetector:readFromStreamFailed:reason=readerTimeOut
2018-02-24 15:48:30.690 [ 15] I AbstractKeywordDetector:readFromStreamFailed:reason=readerTimeOut

This message repeats until I kill the process, restart, and it all happens again.
My microphone is shared via dsnoop.  I tried ending the other process that is listening, but the issue still occurs:
/etc/asound.conf
pcm.!default {
  type            asym
  playback.pcm    "hw:0"
  capture.pcm {
    type          plug
    slave.pcm     "dsnoop:2,0,0,S16_LE,16000"
  }
}

What version of the AVS Device SDK are you using?
1.5.0
Tell us what hardware you're using:
Raspberry Pi 3
Playstation Eye for microphone
Tell us about your OS (Type & version):
ArchLinux Arm
Edit: Changing from dsnoop to hw seems to resolve the issue, but I would like to continue using dsnoop.  I don't believe this was an issue with alexa-avs-sample-app.

Hi @ofuangka!
It seems that even having dsnoop does not allow two applications to record simultaneously. Does the simultaneous recording work with other apps?
In general, SDK uses the default recording device. The comments you provided show that the problem is either in PortAudio or in the OS audio configuration. Anyway, can you answer several questions to understand the problem better:

Is another device actually recording at the moment SDK stops. Is the other device able to record after SDK stops?
Is it always reproducible?

@mavamazon

Is another device actually recording at the moment SDK stops. Is the other device able to record after SDK stops?
By device, do you mean application?  If so, then yes the other application is recording both before and after the SDK stops.  However, I had tried stopping the other application before starting the Alexa sample app, and the same result occurred as described above.

Is it always reproducible?
Yes.

When I get some time this weekend I'll try the compiling against the nightly PortAudio to see if that fixes it.  If that doesn't work I'll probably go back to the Java alexa-avs-sample-app, which as stated before, seemed to be working with dsnoop.

Hi @ofuangka,
I'm finding a handful of references to the same assertion you are seeing in various audio application forums, but no one seems to have chased it to root cause yet.  I did finally find a bug report filed directly against pulseaudio here.
Of particular interest in the bug report: "this happens only with Alsa <IP_ADDRESS> (and 1.1.4), but does not happen with the older 1.1.3".  If you have any ability to roll back to an older ALSA release, it may work around the problem until the PulseAudio team gets to the bottom of it.
Ken

@kencecka
Thanks for finding that.  I'd rather not roll back my ALSA, but will consider it if all else fails.

I understand.  This sounds like a bug in ALSA or PulseAudio, and presumably they'll get to the bottom of it in a future release, but that may take some time.  You might want to chime in on the pulseaudio bug report to see if you can get them more interested in fixing it.
Let us know if there's anything else we can help with.
Ken

I also met this problem when I change the capture device from "hw" to "dsnoop".
@ofuangka How did you fix this problem if keep using "dsnoop"? Thanks.

I also get same problem:
2019-09-14 06:38:41.431 [  3] I AbstractKeywordDetector:readFromStreamFailed:reason=readerTimeOut

Same problem here using Raspberry Pi Zero W, Linux raspberrypi 4.19.75, Raspbian Buster Lite. Would be great if someone could give some hints on how to solve it...

me too ,I using Raspberry Pi Zero WH,I don't known how to solve it....

Install pulseaudio


Add grudginglyAcceptLongLines option

An import line such as
import aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa from '../aaaaaaaaaaaaaaa/aaaaaaaaaaaaaaa/aaaaaaaaaaaaaaa';

can't necessarily be split at all, so this adds a grudginglyAcceptLongLines option to quiesce those cases.

I would argue that simply checking whether the import has an opening curly brace in it is a bit of a primitive way to check this case.

It is, but it's also pretty effective. 😁 (Barring edge cases such as import foo from './{whywould}/{youdothis}';, but if those happen, they could be tackled separately.)

In my understanding, the issue is specifically with default imports, so I would rather not add an option for this and simply adjust the check to see if the line only contains a single default import.

Can do. Just a moment!

Thank you for your contribution, the changes have been merged and released as v1.3.0


node-version-file should support absolute paths

Description:
If we pass a absolute path to node-version-file, for example, when using with Composite Actions, the resulting path is resolved incorrectly, due to a path.join() call.
https://github.com/actions/setup-node/blob/5e21ff4d9bc1a8cf6de233a3057d20ec6b3fb69d/src/main.ts#L96-L100
Action version:
v3
Platform:

[x] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[x] Hosted
[ ] Self-hosted

Tools version:

Repro steps:

In a composite action with .nvmrc, use following as input:node-version-file: ${{ github.action_path }}/.nvmrc

Example broken action run: https://github.com/sidvishnoi/mediasession/actions/runs/6158104300/job/16710218833?pr=1
Expected behavior:
If it can check whether provided node-version-file exists as is, or perhaps use path.resolve(),  it won't break with absolute paths.
Actual behavior:
Error: The specified node version file at: /home/runner/work/mediasession/mediasession/home/runner/work/_actions/w3c/spec-prod/fix-nvmrc/.nvmrc does not exist
                                                                                ^^^

Hello @sidvishnoi. Thank you for your report. We'll take a look on it.

Hi folks, I'm running into this issue as well, trying to use ${{ github.action_path }}/.tool-versions but it tries to append the path to the current directory.
It's a pretty frustrating and unexpected behavior as far as I can tell. Any hope to see a fix to this soon? 🙌

Can confirm would be nice if it could be looked into. @w3c has a workaround here but less than ideal 🙏

@actions/setup-actions-team @HarithaVattikuti
Apologies for the pings, but it looks like Dmitry hasn't been active on this project in a long time, and this issue may have slipped through the cracks. It would be awesome if someone could take a look at this and any other issues that may have been buried.

Hi @sidvishnoi, Thank you for your suggestion to support absolute paths in the node-version-file. Although we do not have any plans to add this feature at this time, we will consider it as a feature request for the future.
In the meantime, please use the following workarounds:

Use a script to determine the relative path of the action directory and pass it to the node-version-file input:

id: action_path
run: echo "action_path=$(node -p 'require("path").relative(process.env.GITHUB_WORKSPACE, "${{ github.action_path }}")')" >> $GITHUB_ENV
shell: bash
- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version-file: ${{ env.action_path }}/.nvmrc

This script calculates the relative path from the GitHub workspace to the action directory and sets it as the node-version-file, ensuring the path is resolved correctly within the repository context.
2. Set the node version directly in the action, bypassing the need for a version file.
I hope this clarifies the issue and provides a suitable workaround. If you have any further questions, please feel free to ask.

Hi @sidvishnoi, Just a gentle reminder regarding this issue, If you have any updates or need further assistance, Please let us know.

@gowridurgad the workaround you shared above is what I created for W3C 😉
Further assistance would be not requiring this workaround.

Hi @sidvishnoi, Here is an additional workaround you can use:
Use symbolic links:
Create a symbolic link: ln -s ${{ github.action_path }}/.nvmrc
Add the symbolic link creation step in your composite action before setting up Node.js. Here’s how you can update your composite action:
- name: Create symbolic link for .nvmrc
   run: ln -s ${{ github.action_path }}/.nvmrc
   shell: bash

 - name: Set up Node.js
   uses: actions/setup-node@v4
   with:
     node-version-file: ./.nvmrc

This script will create a symbolic link from the root of your repository to the .nvmrc file in your action directory before setting up Node.js.
Let us know if this helps or if you have any further questions!

Hi @sidvishnoi, Just a gentle reminder regarding this issue, If you have any updates or need further assistance, Please let us know.

@gowridurgad There's no point in asking me for updates, as you've said "we will consider it as a feature request for the future." Asking over and over will not get a different answer from me, given it was me who suggested the workarounds you're sharing in one way or other.
I hope you're reading answers and not just asking questions. In this AI age, doesn't feel very human.

Hi @sidvishnoi, Thank you for your feedback. As mentioned previously, we have noted this as a feature request for future consideration. We have sent a reminder to check if the second workaround worked for you. Apologies if it feels repetitive, but our intention is to ensure clarity and keep the discussion transparent. We truly appreciate your patience and continued engagement.


Michael: Asynchronous-Programming, 3 Weeks.

stop watch project

Learning Objectives

🥚 You understand the JavaScript Event Loop, and can demonstrate this by using setTimeout and setInterval to schedule simple tasks.
🥚 You can explain why Asynchronous Programming is important for programs that have blocking and non-blocking tasks.
🥚 You can explain the basics of the Client/Server model and HTTP requests and can fetch data from RESTful APIs.
🐣 You can break down an asynchronous problem into smaller tasks and solve it using promises. This includes identify which tasks depend on each other and which are independent:

dependent tasks: The return value from one task is required to start the next task, these must be completed in a specific order - .then
independent tasks: These tasks do not use each other's return values, they can be completed at the same time - Promise.allsystem.

🐣 You can fetch data from an API and render it into the DOM using /api-calls, /handlers and async/await syntax.
🐣 You can write unit tests for functions that return promises using async/await syntax.

How was you first week of asynchronous programming @MichaelMugaiga?

Week 2
I Need Help With:

I don't need help at the moment

What went well?

understanding Promise.

What went less well?

having some challenges with user API project.

Lessons Learned

DOM manipulation:- creating elements
Promise consumers;-  .then .catch
Promise states(pending, fulfill, reject)}

Sunday Prep Work

Not started yet, still finishing with the user API project.

Hi @MichaelMugaiga,
Congrats on the great work!
Feel free to contact me on Slack if you need some help, will try to be as responsibe as possible

Week 2

Hi @MichaelMugaiga, Congrats on the great work! Feel free to contact me on Slack if you need some help, will try to be as responsibe as possible

Hi @raymundovr,

thanks for reviewing my work and the feedback. i will contact you as soon as i have any challenges with code writing. thanks alot.

Week 3

ToDo list


KAFKA-5376: Ensure aborted transactions are propagated in DelayedFetch

This patch seems to fix the test failures due to duplicates we saw in the the new system tests in KAFKA-5366.

@apurvam Thanks for the review. Here is the JIRA: https://issues.apache.org/jira/browse/KAFKA-5383.


Bug fix: replace scale.x by scaleX

The property scale is a number. so try to access scale.x is invalid.

Perhaps it would be helpful if I marked the top of each file to show which hasn't been converted yet? (as this one is only partially done)

Yes, it could be a good idea. I find these bugs adding the types for the TypeScript definitions. I don't do a code review.

Mehmet Kızılaslan


Done with changes in searchBar

Just a update to my last issue #1256

@Pratik2401 Thank you for contributing to this repo. Looking forward for more 🚀


Please add Metasploit

Link to Tool
https://github.com/rapid7/metasploit-framework
Why?
Because it is very helpful for attacking systems using malware.

This is out of the scope of the simple tools leveraged in this package.


Bugfix: DryrunOption

Adds DryrunOption to graph for SousDeploy.
Adds missing godoc.

@alrs @nyarly I'm self-merging this in order to progress, post-review appreciated.


Separate repo for Objective-C parser

Hello, from your Contributing.md file:

Gherkin3 is implemented in several different languages, and each language implementation lives in a separate git repository.

Is that right? Can I find it in a separate repo? As I need it for one personal project. I'm thinking about how to integrate it as 3rd party without getting the entire gherkin3 repo.
Also is the current implementation ready to be used in real projects, or it is a work in progress?
Thank you.

The separate git repo for the Objective-C version is cucumber/gherkin-objective-c.


Should probably have a method to remove a Parent and unbind all its event handlers

Possible solution to #73.

Closed in 18de04d


Git is not updating after file change

Issue Type: Bug

Git is not detecting changes in files. Changes in code are highlighted but source control tab is empty. Manual refresh is necessary.

Git is also not updating after committing changes. After change is committed, source control tab still shows files that were committed. Manual refresh is necessary. Also, code changes are still highlighted in opened files after committing.

VS Code version: Code - Insiders 1.34.0-insider (d234937aedcbc265d2272ba5861125293239f9c7, 2019-04-05T05:18:43.251Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4210M CPU @ 2.60GHz (4 x 2594)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (8.48GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

better-comments
aar
2.0.3

color-info
bie
0.5.0

bracket-pair-colorizer-2
Coe
0.0.28

EditorConfig
Edi
0.13.0

auto-close-tag
for
0.5.6

svg
joc
0.1.4

vscode-styled-components
jpo
0.0.26

wordcounter
kir
1.5.0

contextualduplicate
laf
0.2.0

vscode-language-babel
mgm
0.0.22

postcss-language
mhm
2.1.0

dotenv
mik
1.0.1

vscode-csscomb
mrm
5.2.2

vscode-duplicate
mrm
1.2.1

react-proptypes-intellisense
OfH
1.0.3

material-icon-theme
PKi
3.7.0

multi-command
ryu
1.3.0

code-settings-sync
Sha
3.2.8

stylelint
shi
0.48.0

code-spell-checker
str
1.6.10

code-spell-checker-czech
str
0.1.0

vscode-icons
vsc
8.4.0

dictionary-completion
yzh
0.8.1

(1 theme extensions excluded)

I also have the same problem with few latest versions of insiders, most of the time I have to click on the Git Refresh icon. However in couple of occasions the refresh made it worst and it showed commit differences that were already commited and pushed to origin, after restarting VSCode, it came back to correct diff...

The issue seems to be fixed now.
Version: 1.34.0-insider
Commit: 0ab39f4148f242e7b0802330385fc99b4845aa31
Date: 2019-04-08T05:19:58.162Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.18362


Add NCCLBroadcast to correct net

Otherwise was always added to main net instead of param_init_net when
desired (i.e. initial param sync)

Uh, this was pretty bad bug!


Bracket Colourisation doesn't work in Azure Pipeline files

When using the Azure Pipeline extension associated with .yaml/.yml files it only seems to colourise brackets if they are in the parameters area. Any brackets within the tasks don't seem to get colourised.
All other types of files colourise correctly.
Attached two screenshots to show it working in the parameter definitions  but not in the tasks:

Hi @tap1992, thanks for reporting this issue.
We are currently working on higher-priority issues, but we will investigate this issue later.


Change Application handler to get semester id by default

Loaded schema into a db, figured out how to get all dummy data inserted for testing, got a insert statement to work for creating/submitting applications without the front end needing to know the current semester. Realized some issues with the current schema and models, removed them and some duplicate info, implemented the db call for posting applications in the dao.
[t: 140]

Went over pr with other devs, realized some bugs and fixed them. Issue came up with needing another variable in the application table, added it. Realized another bug with checking the length of a field that should be accepted to be empty and fixed it.

Figured out how to build it with docker so that I could use dummy data and test the endpoints using postman.  Proceeded to spend a lot of time trying to debug it all. Eventually got it to work.
[t:120]


Add support for the TOUCH command

Search before asking

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

Motivation
TOUCH command is useful feature in Redis, allows updating LRU time of one or more keys without modifying their values.
Note: I have just started working around Kvrocks, so not sure if this would be needed. Although I can think of some possible solution. Needed review/guide here.
Solution
will update
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Kvrocks doesn't keep the last access time of keys and doesn't support key eviction.
So it's not so meaningful to support this command.
See if other guys have any comment.
cc @git-hulk

Kvrocks doesn't keep the last access time of keys and doesn't support key eviction.
So it's not so meaningful to support this command.
See if other guys have any comment.
cc @git-hulk

Kvrocks doesn't keep the last access time of keys and doesn't support key eviction.
So it's not so meaningful to support this command.
See if other guys have any comment.
cc @git-hulk

Yes, exactly.


jackett module: add more options and refactor

Motivation for this change
makes user, group, config location, and opening the firewall port for the webui configurable (similar to modules like sonarr or nzbget) in a backward-compatible manner.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test jackett

@elseym Thanks for this one and the Radarr one! Nice to have them synced up :-)


Status

Hey man, :wink:
Lovely work, do you pin the rocket version to a git one for a reason?
I forked the project and added some stuff, if you're in on have a PR I'm happy to help out :beer:

Hey!  The version is pinned just for a personal project I was using it with that was pinned to a Rocket version.  I'm totally down to unpin it now; I can update the project.  Would love to see your changes as well

Glanced at your branch and everything there looks like great changes; I'd be happy to upstream if you wanna make a PR :)

Sure, will remove the fork stuff from the readme and create a PR.
Any reason why the package isn't on crates.io?

I was hoping that it would be integrated into rocket itself, but I didn't see much interest/traction in the few/limited comments I made about it in issues on the rocket repo.  I could push it up there if you think it'd be useful to you or other people

Would be lovely.
Would also like to know if this action https://github.com/marketplace/actions/publish-crates really works that well with https://github.com/marketplace/actions/semantic-version and git tags
Well sergio seems to have had a difficult time personally but he will be back adcording to hist post https://github.com/SergioBenitez/Rocket/discussions/1957

Ok sounds good, I'll push it up to crates.io once your changes are merged in.  I'll probably skip the GH actions just because I don't foresee there being frequent changes to this code in the future.

Thanks for the improvements btw!


Masquerade rule bug on reboot

I have a basic wireguard setup alongside PiHole on a Raspberry Pi 4 8GB with just 4 clients. When rebooted, communication between wireguard clients work but no Internet connection. When pivpn -d is used upon reboot, it asks me to add the masquerade rule again. It works! Not sure what's going on.

https://github.com/pivpn/pivpn#feedback--support


Tear downs of event loops should stop async generators

What is wrong?
The official documentation recommends this pattern:
try:
    loop.run_forever()
finally:
    loop.run_until_complete(loop.shutdown_asyncgens())
    loop.close()

See: https://docs.python.org/3/library/asyncio-eventloop.html?highlight=loop#asyncio.loop.shutdown_asyncgens
How can it be fixed
We need to double check if and where this would apply to our code base but probably worth to investigate.

Yeah, there's quite a few of these. As of 185bcb8330e :
$ git grep -n run_forever
scripts/chainsync.py:129:    loop.run_forever()
scripts/peer.py:113:    loop.run_forever()
trinity-external-plugins/examples/peer_count_reporter/peer_count_reporter_plugin/plugin.py:62:        loop.run_forever()
trinity/main.py:154:        loop.run_forever()
trinity/main.py:195:        loop.run_forever()
trinity/main_beacon.py:127:        loop.run_forever()
trinity/plugins/builtin/ethstats/plugin.py:131:        loop.run_forever()
trinity/plugins/builtin/json_rpc/plugin.py:103:        loop.run_forever()
trinity/plugins/builtin/peer_discovery/plugin.py:181:        loop.run_forever()
trinity/plugins/eth2/beacon/plugin.py:83:        loop.run_forever()
trinity/server.py:440:    loop.run_forever()
trinity/sync/full/service.py:190:    loop.run_forever()
trinity/sync/full/state.py:401:    loop.run_forever()

0 instances of shutdown_asyncgens.
Using asyncio.run() is a no-go, since it requires py3.7 and we will likely be stuck with py3.6 for a while.


"The format of the phone number provided is incorrect.

I keep getting this error.
"The format of the phone number provided is incorrect. Please enter the phone number in a format that can be parsed into E.164 format. E.164 phone numbers are written in the format [+][country code][subscriber number including area code]."
here is my code:
PhoneAuthProvider.provider().verifyPhoneNumber("+13231234567", completion: {(_ verificationID: String?, _ error: Error?) -> Void in
            if error != nil {
                // Verification code not sent.
                print()
            }
            else {
                print()
                // Successful.
                // Show the Screen to enter the Code.
                // Developer may want to save that verificationID along with other app states in case
                // the app is terminated before the user gets the SMS verification code.
            }
        })

What is wrong with that format? Plz anyone help. I am using US phone number as example
Now am getting this...
We have blocked all requests from this device due to unusual activity. Try again later.
Great job firebase...

Problem here that phone number is not real.
We are validating phone number with carrier provider.
@yarodevuci Have you tried using real phone number?

btw you're using headless library (FirebaseAuth) calls, thus please create related tickets in there repo: https://github.com/firebase/firebase-ios-sdk

@yramanchuk sorry about that. And yea i was using real number. Of course I didn't want to share it hee

@yramanchuk this problem persists. im using a real phone number and still get the invalid phone number error. why?

I tested with Vietnam phone number, and put +84989440000 is good.

For firebase to Validate any phone number. The country prefix code has to be added.

Adding plus gives indication of country code (+91 for india OR add "(0120) for landline code") that is E164 compliance.

im have this problem by creating a user whit a colombian number "+573234686974"

"+1<PHONE_NUMBER>" Try sending with this format. you will get VerificationID for sure


A test for various URLs that end in with a broken percent encoding

For your consideration, the test for URLs that contain a broken percent encoding.  Some of these conditions will throw an exception.  The pull request for the fix will follow.

Thanks!


An error occurs when calling Azure's open ai gpt-4o in GitHub using the latest version of Claude dev

When I use the open ai gpt-4o large model in GitHub markplace, this problem also occurs, how should I deal with it?
It worked fine a week ago, but after updating the latest version of cluade dev, error code 424 was called
https://github.com/saoudrizwan/claude-dev/issues/300

I just released v1.8.1 that lets you set the Azure API version. Can you pls see if setting that fixes the issue for you?

Closing for inactivity, pls feel free to reopen if you're still running into problems.


👩🌾 3 ➡ 4

➡ Forward port
Port ign-gazebo3 to ign-gazebo4
Branch comparison: https://github.com/ignitionrobotics/ign-gazebo/compare/ign-gazebo4...ign-gazebo3
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

https://github.com/osrf/buildfarmer/issues/161

The user commands test failures are expected (#634)


A certain letter is not working

Type: Performance Issue
Ever since I started coding c language on vs code I can't type letter "h" in the coding panel. It also doesn't shows a problem specificly that I can fix. My all other letters are working except letter "h". When I was coding html on vs code at that time letter "h" worked but when I started c language , letter "h" stopped working in html as well. I can't understand where is the problem.
VS Code version: Code 1.80.0 (660393deaaa6d1996740ff4880f1bad43768c814, 2023-07-04T15:06:02.407Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-5005U CPU @ 2.00GHz (4 x 1995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
3.92GB (1.03GB free)

Process Argv
--crash-reporter-id e282023f-cc4b-46e9-9fd2-e12958146682

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

code-runner
for
0.12.0

cmake-tools
ms-
1.14.34

cpptools
ms-
1.16.3

cpptools-extension-pack
ms-
1.3.0

LiveServer
rit
5.7.9

cmake
twx
0.0.17

(1 theme extensions excluded)

/confirmPlease

Please perform the following three tasks to diagnose the root cause of the issue:

[ ] 1.) Disable Extensions

Select View and pick Command Palette...
Run Developer: Reload With Extensions Disabled
👉 See if the issue reproduces

[ ] 2.) Disable Configuration

Select View and pick Command Palette...
Run Profiles: Create a Temporary Profile
👉 See if the issue reproduces

[ ] 3.) Try VS Code Insiders

Download VS Code Insiders
Install and Run it
👉 See if the issue reproduces

Then pick one of the three resolutions depending on which step has helped:

Disabling my Extensions helped
Please run the command Start Extension Bisect and follow the instructions to find the extension that is causing this issue.

Please report the issue to the extension causing this.

Disabling my configuration helped
Please report back more details about your configuration, including settings.

Using VS Code Insiders has helped
✅ This likely means that the issue has been addressed already and will be available in an upcoming release. You can safely use VS Code Insiders until the new stable version is available.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


fix: Persist SessionStarted and HandoffToComposerCompleted telemetry events

Description
Currently, there are several events - SessionStarted and HandoffToComposerCompleted - that occur before the user has had the chance to opt in to data collection.  To avoid losing telemetry information, we are adding the events in the persistedEvents array to the event pool; however, they are only logged to Application Insights after the user opts in to data collection.
Additionally, this PR adds the appId property to the HandoffToComposerCompleted event.
Task Item
Closes #7911

Coverage decreased (-0.01%) to 56.711% when pulling ac27b3044614361fb8221c2a7382e69f57864869 on durnford/feat/telemetry into 06e1b66d9bd31ad086505fc15cbed15253f6a6bd on main.


TypeError: node.children.at is not a function

Throwing this Error right away:
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info  - automatically enabled Fast Refresh for 3 custom loaders
event - compiled client and server successfully in 699 ms (164 modules)
wait  - compiling /_error (client and server)...
event - compiled client and server successfully in 69 ms (165 modules)
warn  - Fast Refresh had to perform a full reload. Read more: https://nextjs.org/docs/basic-features/fast-refresh#how-it-works
wait  - compiling / (client and server)...
[nextra] Error compiling my-app/pages/index.mdx.
error - ./pages/index.mdx
TypeError: node.children.at is not a function

Reproducing steps:

Fork the template https://github.com/shuding/nextra-docs-template
yarn
yarn start

Pretty frustrating when you just want to run a new library and check it out :)

use node 16

Thank you @B2o5T! That fixed it!


"API query failed" for files/photos, then downloads spam for topics

I'm not super tech savvy, but got a couple of techies to help me install Python, then Pip, and we finally got some movement with the downloader, but never managed to download anything from my group.
For files and photos I consistently get "API query failed", but then for topics I get a bunch of downloads, but it's all pornographic spam. Not a single legitimate message from the group.
It's late in the game, but I'm still hoping to download my archives in the next 24 hours, before Yahoo removes them. Is there anyone who has some insight into what could cause this, and how to fix it?
The command line specifiess the location of the archiver, then the location of Python, then: ./yahoo.py group -ct 'T cookie' -cy 'Y cookie' 'nameofmygroup'.
We got an error that 'group' was a necessary argument, so inserted it, and that's what finally made it do something....except not what we wanted it to.
Thanks for any last minute insight anyone might have.

Hello,
If your groups have files or photos (some do not have it) check the cokie, sometimes it seems fine but one skips copying a part or not closing the quotes that should surround each of the two values

We must have tried it a couple of dozen times, always very carefully checking everything. I even logged out, and logged back in, and copied the cookies from that session to see if maybe the cookies had timed out. We had three sets of eyes on it every time we tried.
I don't think it's anything as simple as a missing quotation mark, since we downloaded something, just not what we were aiming for. Everything we downloaded under 'topics' was junk. It seems like we downloaded a spam folder, which Yahoo Groups doesn't have. None of the data made any mention of the group name anywhere. Very strange.

Try with email option "-e" it is slower but you can parallelize the download with the options --start and --stop, example:
yahoo.py group -ct 'T cookie' -cy 'Y cookie' -e --start 1 -stop 5000 'nameofmygroup'
yahoo.py group -ct 'T cookie' -cy 'Y cookie' -e --start 5000 -stop 10000 'nameofmygroup'
running two or more in paralell is quite fast

I am also having issues downloading files.  I'm using yahoo.py group -ct Cookie_T<value> -cy Cookie_Y<value> -f <groupName>
Also even though I'm using -f the script still runs for topics, attachments, etc

yahoo.py -ct "z=someLongText&a=someLongText&sk=someLongText&ks=someLongText&kt=someLongText&ku=someLongText&d=someLongText&af=someLongText" -cy  "v=1&n=someLongText&l=someShortText=someShortText&r=hp&intl=us" "nameofmygroup" -f


Update the schema diagram for the readme

Actioned in https://github.com/populationgenomics/metamist/pull/654


Fix Global Middleware Example

When using the package in a standard Laravel application, the Kernel is overridden from the base kernel.
Injecting the base kernel here creates a new instance instead of actually attaching the bound application Kernel.
Updated the example to inject the Kernel contract instead of the implementation.

@pdawson Thank you! That would be better indeed 👍 😄
Thanks again for the PR!


Add all-contributors bot to repo

User Story:
As a software developer, I want this repository to utilise the all-contributors bot so that all types of contributions made by contributors to the project are fairly recognised.
Acceptance Criteria:

All-Contributors bot is installed on this repository
Contributions to the project are visible in README

Notes
https://allcontributors.org/

@all-contributors please add AlexanderTheGrape for code test

Note that this issue is tied to this pull request https://github.com/se701g2/Doto/pull/266
and this issue https://github.com/se701g2/Doto/issues/204 and needs those pull requests to be merged first.


Added section explaining Windows PowerShell vs PowerShell Core

We continue to get bug reports for Windows PowerShell 5.1.  Updating the readme will provide clarity to users on the difference particularly in relation to this repo.
Fixed semantic linefeeds in first paragraph.

Markdown test failed in AppVeyor CI build:
Describing Common Tests - Validate Markdown Files
NPM is checking Gulp is installed. This may take a few moments.
C:\projects\powershell\README.md: 4: MD013 Line length
See https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md for an
explination of the error codes.
 [-] Should not have errors in any markdown files 19.52s
   Expected: {0}
   But was:  {1}
   at line: 120 in
C:\projects\powershell\test\common\markdown\markdown.tests.ps


New Github design shows oldest release as latest?

The new side bar seems to get the order of releases wrong. It lists the v0.1 as the latest release:

(Not sure if this is something that can be fixed in this repo or if this is a bug with Github.)

That's odd. Clicking on the +12 releases link shows things in the right order, so I'm guessing this is an issue with the redesign. I submitted a GitHub support request for this since I'm not sure how else to inform them of this.

Thanks for noticing this!
As a quick fix, I removed the old releases now there are only tags.
I looked at a few Dask projects (distributed, dask, dask-kubernetes) and none of them seem to be using releases, maybe this would be a conversation to have in dask/community?
I have to admit I don't really know the the advantages of having Github releases besides looking better in the web UI.
One thing I noticed is that since we are using only tags, it is harder with the new Github interface to access the list of releases, i.e. there does not seem to be any direct link that you can click on.

I looked at a few Dask projects (distributed, dask, dask-kubernetes) and none of them seem to be using releases, maybe this would be a conversation to have in dask/community?

Agreed. I'll switch the PR to a draft to avoid merging before this is clearer.

On this note: While Github releases feel comfortable, they're a big move to vendor lock in. As we don't need them, because we rely on PyPI for distribution anyway, sticking to tags is the way to go for me.

OK closing this issue then!


Proxying uploads and downloads

Is this library capable of proxying uploaded and downloaded files between Plug and the HTTP client?  If so, are their any examples of these?
Thanks,
Lee

Hi @Lazarus404! Sorry for the delay.
I do not have any examples of this, and as of now, this isn't something that I've tested or looked into at any depth. That being said, I'd be more than happy to help correct any issues that arise.

Hi slogsdon,
Do you have any notion of how to get started with it?  Any ideas for pointers?  There's a disconnect in my brain of taking the stream from one down/upload and piping it to another down/upload.  It'd be a pretty powerful feature, though, imho.

I believe this should happen automagically via regular HTTP requests and responses since Plug only considers request/response bodies as being of type iolist.
For uploads, we read the request body from the client and pass it to the upstream. Granted, we may not be reading the entire request body at the moment.
For downloads, the upstream responds, and we take the body of the upstream response and set it as the client response.

Thanks. I'll have a play and will report back with something that works
when I have it.


Reset GL _State when rendering outer skybox

Fixes #1047.

I confirm it fixes the global fog issue from #1047.


Bump release

This PR updates all the course notebook links to point to their respective languages in the notebooks repo

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.